Better embedding extraction

As pointed out in https://github.com/rustformers/llm/pull/291, the quality of embeddings produced by the models at present appears to be suboptimal.
Our current approach uses the embedding of the final token as a representation for the entire input sequence, which might lead to the omission of some semantic information. The approach employed by SGPT: GPT Sentence Embeddings for Semantic Search offers an alternative: they use a weighted mean sampling method to amalgamate the embeddings of all tokens in the input sequence. According to the MTEB-Benchmark, this method results in superior embeddings.
So, this poses the question: should we integrate this method into our implementation? Alternatively, should we leave it to users to manually extract the embeddings for each token and carry out the calculations themselves?

Good catch! I think we should integrate this, but separate it from the existing embeddings. I'm also not sure how we best expose this. Any ideas for API changes that are understandable and restricted to only where it makes sense? This would only make sense with feed_prompt, right?


My rule is not getting any hits?

Using CentOS 7 and python2.7, installation (python setup.py install) was successful.
However, running lastalert-test-rule shows with the following exception:
[root@elk elastalert]# elastalert-test-rule --days 5 example_rules/example_frequency.yaml
Successfully loaded Example rule

INFO:requests.packages.urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>
INFO:elastalert:Note: In debug mode, alerts will be logged to console but NOT actually sent. To send them, use --verbose.

Thanks !

Maybe because you have timestamp_field: Timestamp when in the your example document it's the default @timestamp?

I don't know how to config @timestamp . I think problem is "date":
timedatectl
Local time: Mon 2016-06-06 15:54:48 ICT
  Universal time: Mon 2016-06-06 08:54:48 UTC
        RTC time: Mon 2016-06-06 15:54:48
       Time zone: Asia/Ho_Chi_Minh (ICT, +0700)
     NTP enabled: yes
NTP synchronized: no
 RTC in local TZ: yes
      DST active: n/a

And when i run elastalert-test-rule:
elastalert_status - {'hits': 0, 'matches': 0, '@timestamp': **datetime.datetime(2016, 6, 3, 3, 48, 22, 780474, tzinfo=tzutc())**, 'rule_name': 'Example rule', 'starttime': datetime.datetime(2016, 5, 29, 3, 48, 22, 480387, tzinfo=tzutc()), 'endtime': datetime.datetime(2016, 6, 3, 3, 48, 22, 480387, tzinfo=tzutc()), 'time_taken': 0.2984790802001953}

remove timestamp_field: Timestamp. The timezone doesn't matter because it queries using UTC and Elasticsearch will do the necessary conversion.

hi,Qmando,I got the same problem,and I did not config the timestamp_field: Timestamp ,I think the problem is timezone,so there abviously cause alert,but it did not,can  you help me?thanks  a lot

Time zones should never matter. Everything is done using timezone-aware comparisons.
I can't help you unless you tell me what the problem is.

@Qmando
Ok, I have commited my problem in issue  #893 , sorry to disturb


Use lae.travis-lxc and bandage avahi issue (fixing #11)

This simplifies the test suite a bit more.

Needed to restart the cent6 build since it looks like it didn't get an IP in time (will look into this more later). Passed here: https://travis-ci.org/lae/ansible-role-cuda/builds/198908945

I restarted both CentOS7 (this failed idempotency test because of too slow mirror or something like that) but also the CentOS6 one.

CentOS7 passed after restart. CentOS6 failed again

Pushed an update to lae.travis-lxc for checking network connectivity differently, so the cent6 build should pass without issue (well, if the cuda repo doesn't time out). Just made that last commit for semantic reasons and to start another build.

Yeah it looks like the cent6 build failed on getting packages from the cuda repo this time

Same for CentOS7 after restarts - looks like cuda 8.0.61 was released last night.
I'll just wait a few days and try again. Thanks!

Merged. CentOS6 test play failed on the first run but succeeded/changed on the idempotency test.


Do not use sshagent when on zos

Fixes https://github.com/AdoptOpenJDK/openjdk-tests/issues/713
Signed-off-by: Mesbah Alam<EMAIL_ADDRESS>

Tested using Grinder in internal Jenkins.

FYI @llxia


[6.0][region-isolation] Make "unknown pattern error" always an error.

Starting the testing.

This shows an actual issue with the compiler where semantically we should crash. Rather than crashing (due to the broken invariants), we emit this error so that the user gets a nice error message at the problem place and can work around it instead of just getting a mysterious crash.
Previously, we were making this a warning in swift 5 mode... but given the issues, it makes sense to emit an error diagnostic so we get the feedback and the user cannot ship the code.
rdar://131482934
(cherry picked from commit b63781f7ba7796a8ad20b79c56f1ad961eac6286)

@swift-ci test


Fix checkPirvateLinkServiceVisibility in 2020-06-01

fix 202 in new release, #9800

/azp run automation - sdk

Can one of the admins verify this patch?

@00Kai0, could you take a look at the check errors?

/azp run automation - sdk

/azp run automation - sdk

Looks like a new response code (202) is being added to existing api/api-versions like 2019 versions. This is a breaking change and should only be made in a new api-version

@ravbhatnagar this PR targets a separate branch for new API version that's going to be merged into master in July

/azp run automation - sdk

As this change happens in a separate branch, this is not a breaking change so merge this PR.


[CWS][SEC-12495] Do not send auto-suppressed events

What does this PR do?
Suppress events that are part of a security profile if the corresponding rule allows it and the runtime_security_config.security_profile.auto_suppression.enabled config parameter is set to true.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


CupertinoDatePicker Date mode should also use minimumDate maximumDate.

The CupertinoDatePicker widget has 3 modes. time, date and dateAndTime.
When using the date mode, we can restrict the selection range across years provided as integers
  CupertinoDatePicker(
      minimumYear: 1980,
      maximumYear: 2015,
      mode: CupertinoDatePickerMode.date,
      // ..),

This "year only limit" behaviour is quite strange as for example in limiting birth date ranges, it also should take months and days into consideration.
More specifically:
We want to allow only birthdates where a user is 21 year old or older.
Meaning
int minimumYear = 2018 - 21; // 1997

But if the user was born in December and the current month is November the user can still select the under-21 birthdate.
The DatePicker should therefore use the DateTime minimumDate and DateTime maximumDate parameters ( which are used for the other 2 modes) to limit the range.
Doctor output
[✓] Flutter (Channel dev, v0.11.9, on Mac OS X 10.14 18A391,
    locale en-NL)
    • Flutter version 0.11.9 at
      /Users/joelhernandez/Documents/code/flutter
    • Framework revision d48e6e433c (7 days ago), 2018-11-20
      22:05:23 -0500
    • Engine revision 5c8147450d
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK
    28.0.2)
    • Android SDK at /Users/joelhernandez/Library/Android/sdk
    • Android NDK location not configured (optional; useful for
      native profiling support)
    • Platform android-28, build-tools 28.0.2
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 27.1.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.4)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 29.1.3
    • Dart plugin version 182.4999

[!] VS Code (version 1.25.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Cod
      e.flutter

[✓] Connected device (1 available)
    • iPhone X • 8968109C-4432-45F0-A010-94DAC8575087 • ios • iOS
      12.1 (simulator)

! Doctor found issues in 1 category.

Also to point out, in the case of the Material date picker, this is the case.
showDatePicker( firstDate: minimumDate, lastDate: maximumDate);

Please add the output of flutter doctor -v.

Please add the output of flutter doctor -v.

Whoops, thought I did. Done now. Thanks!

Hi guys, would you have any updates on this one, please?..

Yes, I also have same problem to limit the maximum date

@zoechi - any updates on this one? As pointed out by @mleonhard this feature is quite critical for any iOS app, using this date picker...

Any updates on this?

Woah, it's been almost a year since I opened this... 😬
What we ended doing as a hotfix while the Flutter team resolved the issue was using the material component.
This has been the case for several widgets that were broken/buggy on iOS, so our app is currently a zombie made out of iOS and Material parts...

Any updates?


Metrics persistence

Is your feature request related to a problem? Please describe.
When the exporter is restarted, all monitoring metrics will be reset.
At this time, the data obtained by prometheus will fall off a cliff, which is very unreasonable.
Describe the solution you'd like
Persist every once in a while (for example, every 10s), and read data from the persistent data when the exporter is restarted.

With what kind of metrics do you see that as a problem? I think that is the normal behavior for Prometheus metrics in general and Prometheus is ready to deal with that.
For example for counter metrics, it is normal that they may reset also without restart (they can't grow infinitely after all). Functions like rate and increase should be used to process counters, looking at the change in a certain time, and they are able to deal with counter resets.

Thanks for your reply.
Rate and increase are generally not greatly affected, but counter will be greatly affected.
For example, if I want to count the total number of visits in the last hour, day or week, if a reset occurs during this period, then I may get a negative number result.
To be more specific, assuming that the value of nginx_http_response_count_total was 10000 a day ago, and the exporter restarted today, and the value is 1 at this time, then the number of visits I get is -9999.

Not sure what you want to track here - it should be no problem using increase to for instance get daily request counts (or for some other time frame where metrics are retained).
If you want to have a counter of requests over a time that extends what is stored in Prometheus, then I think Prometheus might not be the right tool or you would at least need to process and store the information somewhere else.

Thank u. I made a mistake. When calculating the number of visits over a period of time, it is not simply subtracting, but using functions such as increase and rate.


summary(DataFrame) should have columns

currently it's just a single column of summary tables

I don't think this is relevant anymore as we've moved to describe.


Test Certificate Renew

Hi there,
Is there a way to trigger a renew to test that the certificate renews correctly?

Yes, you can do it:
1- Comment this lines
2-Build a new release running $ make docker DOCKER_IMAGE=gianrubio/kube-lego
3-Start lego with LEGO_MINIMUM_VALIDITY=1h
Be warned that letsencrypt can throttle you if you do this too often.

Thanks!


Chaos canary

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Work still remaining: https://github.com/awslabs/aws-well-architected-labs/issues/409


Improve README

It is a basic README I did in a short time. Once you give me your feedback, or feedback from users I would update it.

Looks great! Merging.


Card link chevron not aligned with divider line

Description
The card link chevron isn't aligned with the divider line.

This is the original design spec in Figma (not aligned):

https://github.com/nhsuk/nhsapp-frontend/issues/10#issuecomment-2250640870

we use a simple chevron with no background, but the basic card in the NHS design system has a blue background circle
i suspect using the blue background circle would be far too heavy for the amount of them that we’d have, but if we sort-of-invert them, with a light grey circle BG and a blue chevron, we’d add a bit of visual detailing and be kind of close to the main NHS DS?

I think it's worth noting that the chevron is aligned 'correctly' when including its bounding box.


Added support for macOS

Added support for macOS and fixed some minor bugs.

Updated the source code

make a release so i can test it.

make a release so i can test it.

https://github.com/G1aD05/SkyOS-bugfix/releases/tag/bugfix

maybe add how to run on MacOS into the readme now? (make another pull)


A way to pause and resume queue and/or workers?

EDIT: I have found out since writing this that test mode stops jobs from getting processed at all...That should really be documented.
I've create my own solution in a fork where i still process jobs in test mode and keep them in the test mode array. I'll make it based on a boolean in queue.testMode.enter(process) (I.E. testMode.enter(true) to still process the queue) and then i'll PR.
I'll also PR the Readme.MD to include the much needed documentation that testMode stops jobs from running.
The below functionality of pausing and resuming would still be nice but i think the above solution is much cleaner and nicer.
Hi there,
I'm trying to control the queue within my tests. Ideally what i'm looking to do is pause a specific queue worker (while still allowing to it to accept new jobs, just not process them) then create a job and make assertions against it, attach an 'on complete' callback so i can continue test logic once the job is completed and then resume the worker to process the job.~~
At the moment, i'm relying on using setTimeout and waiting a bit before i check if a job is complete and has executed correctly. Which is painful.
I'm aware of the workerCtx.pause and workerCtx.resume methods within the queue.process callback. Which from what i can see in the documentation is exactly what i need. But they're not accessible anywhere else.
I've tried playing around with queue.shutdown with a second argument of argument being my worker which is what workerCtx.pause uses. Which pauses the queue and lets me run the assertions. But resuming the queue has become a pain point. i've tried manually calling worker.resume then worker.start to resume the worker. But worker.start requires the processing method again. But even when i pass it the method again it fails to run. I'm assuming there's some other code that needs to be run i'm missing.
Any advice on how to approach this? I'd be happy to submit a PR with changes to allow this if you gave me some guidelines. Or am i just stuck with constant setTimeouts?
Cheers.

#821


Only receiving data of variables 0 to 18

This seems due to the master sending data too fast to the slave.
Need to introduce a delay between two send in DistantIO.request_all_variables()

SerialPort write operation should be done asynchronously (transferred into the threaded execution) with some flow control.


Add stepersy

Please add stepersy, thanks!

@persanoadorno this is an interesting PR. I would expect a single-line change 😄

Sorry, I don't understand😄. I did something wrong? I'm from phone, so it's a little difficult to use GitHub

Sorry, I diti it wrong. I'm from phone so it's a little difficult to use GitHub 😄

@persanoadorno your PR contains 79 file changes. Can you open a proper one with only registry.yaml? I am closing this one


Updated dependencies for Debian/Raspbian Buster

Debian 10 (Buster) is coming out soon and will have some packages named differently. Raspbian Buster is already out.
Tested on Buster on a RPi

Thanks for heads up, I will merge the PR after dotnet core ship a version that supports Debian 10.

@TingluoHuang .NET Core supports Buster now: https://github.com/dotnet/core/issues/3051 Any chance you could merge this?
My eagerness comes from the fact that I need this to run the agent on a RPi 4 :)

I've been trying to run an agent on a buster based image FYI:
https://github.com/dotnet/corefx/issues/33179#issuecomment-528911543

Sorry for the slow response here - looks like these changes got added separately in a different PR!


Feat/couchbase db support

What does this PR do?
DynamoDb Support - #242
If this PR affects any API reference documentation, please share the updated endpoint references
Providing the couchbase support
If this PR affects any API reference documentation, please share the updated endpoint references
NO
Env Require

DATABASE_URL - Couchbase DB URL
DATABASE_USERNAME - DB User name
DATABASE_PASSWORD - DB Password
COUCHBASE_BUCKET - Couchbase Bucket name
COUCHBASE_SCOPE -  Couchbase Scope name or that can be _default as well
DATABASE_TYPE - couchbase

@manojown this PR will need a newly introduced db provider method called GetUserByPhoneNumber
https://github.com/authorizerdev/authorizer/blob/1eb8965f98dcea421ab99b5626b46afe930b9c79/server/db/providers/providers.go#L22

[discussion]:
What if we set by default and not allow this env vars
COUCHBASE_BUCKET -> authorizer
COUCHBASE_SCOPE -> authorizer
Any side effects you see for them?

@lakhansamani  There is a problem if someone already has a database set up with some scope and wants to switch to an authoriser. In this case, they must make changes to their database and need to create a default scope. WYT?

@manojown this PR will need a newly introduced db provider method called GetUserByPhoneNumber
https://github.com/authorizerdev/authorizer/blob/1eb8965f98dcea421ab99b5626b46afe930b9c79/server/db/providers/providers.go#L22

Ok, I'll add this.

@lakhansamani Is there any additional effect of GetUserByPhoneNumber, i.e. anything else I could have missed?

@manojown no usage of GetUserByPhoneNumber is already covered.


add ocaml syntax highlighting

Fixes #25

I'm not sure to understand why the highlighting is not the same in a file and the documentation tab. The color scheme of the tab doesn't match my scheme github dark nor the default dark+

Thanks. I'll merge this as-is because it's better than nothing. We can continue to investigate why the highlighting is different separately (I expect this is something that would also effect other languages/grammars)


"Already up-to-date" Bug with brew update in macOS 13.2.1

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.0.3
ORIGIN: https://github.com/Homebrew/brew
HEAD: ba6d87eed9ce31f42fa2b33a4e6067d9cf17f13f
Last commit: 2 days ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 49fbe33829353620bcfa42b99e356ce6c78fb07e
Core tap last commit: 13 hours ago
Core tap branch: master
Core tap JSON: 22 Feb 16:28 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 14.0.0 build 1400
Git: 2.37.1 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.86.0 => /usr/bin/curl
macOS: 13.2.1-arm64
CLT: <IP_ADDRESS>.1.1668646533
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
brew update for daily update of the new formulae.
What happened (include all command output)?
Already up-to-date
What did you expect to happen?
I expect brew to output any changes and outdated software I have installed
Step-by-step reproduction instructions (by running brew commands)
Just run `brew update` and it says "Already up-to-date." I have tried `brew update-reset` and it didn't change anything. I can run `brew upgrade` and it will still upgrade outdated packages, but I just want the original functionality of `brew update` back.

Duplicate of https://github.com/Homebrew/brew/issues/14706


Add Aliases

This pull request...

[ ] Fixes a bug
[ ] Introduces a new feature
[X] Improves an existing feature
[ ] Boosts code quality or performance

Description
Adds shortcut aliases to commonly used commands
Purpose
Sometimes it is tiresome to always have to type !play for a command to work. Just like Rythm Bot, it might be nice to add a few shortcut aliases like !p for play, !s for skip, etc.
It's relatively easy to implement. You could even do it yourself instead of using this PR.
Relevant Issue(s)
None

These should not be set by default, as they may conflict with other bots or are unwanted by some users. Additionally, these particular shortcuts used may not be the shortcuts that everyone wants. If this feature is added, it needs to be fully configurable (custom aliases for any command) from the config file, with the default setting being the current aliases.

I'm going to close this for now. If you'd like to work on a config-file version of this, feel free to reopen this PR or open a new PR. Otherwise, I'll probably look into adding it myself fairly soon.


Archive 0.19 docs

This branch failed to rebuild the docs, the environment was not reproducible due to npm dependencies. Closing for now.


Update Blog “jkjfkdfds”

Automatically generated by Netlify CMS

👷 Deploy Preview for singular-buttercream-e141a8 processing.

Name
Link

🔨 Latest commit
5ff1be75c6c4f2e03248fa4b1cfdd7c5f099cf6f

🔍 Latest deploy log
https://app.netlify.com/sites/singular-buttercream-e141a8/deploys/640bae2c0aa6d00008f48561


Declaratively set forwarding rules on ImprovMX

The email service of Pulumiverse.com is managed by ImprovMX, a free e-mail forwarding service. The email account which manages our service is<EMAIL_ADDRESS>which is an alias (alias is currently to my email, so@bjerk.io).
There are open questions at the moment, at least for me:

Should we declare the forwarding rules as infrastructure as code? Hint: I've found a Terraform-provider.

@cobraz let's document it in the README for now as something configured manually. Bridging the TF provider is something we can add later as part of the Pulumiverse offering.


Openroad dft 3444

fixes #3444

@fgaray I tested this PR with your testcase (thank you for providing that) and I no longer see a crash.


Update Gradle 3.5 → 4.2.1.

Changelog https://github.com/gradle/gradle/releases

Could not find or load main class org.gradle.wrapper.GradleWrapperMain

Hm, lemme see, I thought it was only my local problem because I used Linux subsystem on Windows

the same problem on mac

Interesting, I've definitely used 4.2.1 at work, hmm. Either they've updated wrapper jar without new release and broke something or it's my Windows machine.
@nikitin-da can you please try to update on mac locally on master branch? ./gradlew wrapper --gradle-version 4.2.1?
For now I'll update it to 4.2, works fine for me.

dafq

Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain

After local update all works fine https://github.com/pushtorefresh/storio/commit/e63ee35e2cadf60c4cc3e9b3bbf3964917c0bd66

Yeah I think it's Windows uhhhhh, I'll update from my mac then, thanks man!

#826

Codecov Report

Merging #825 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #825   +/-   ##
=======================================
  Coverage   97.41%   97.41%
=======================================
  Files          91       91
  Lines        2707     2707
  Branches      305      305
=======================================
  Hits         2637     2637
  Misses         38       38
  Partials       32       32

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 9e53b45...36a5b35. Read the comment docs.


setting current session after successful third party login

after a successful third party authentication the user will be redirected to:
<your-site-url>/#access_token=xyzabc&expires_in=3600&refresh_token=abcdef&token_type=bearer
we need a mechanism (if supabase-js) is present on the page -> to check the page url on startup and if these values are present - to use them to set the current session
I find the # slightly strange, if it's better I can also alter gotrue to use a ? instead so it's more like parsing query params

I think a ? would definitely be easier because then I can just parse them as query params. I wonder why they made it with a #? That's a page anchor, but I can't think of a good justification

Ok I will update this today to a ?

as of KPS v2.3.2 included here in infrastructure: https://github.com/supabase/infrastructure/commit/367758cc97a71eabc208b4a319234dd443882f26 the url is now:
<your-site-url>/?access_token=xyzabc&expires_in=3600&refresh_token=abcdef&token_type=bearer

Closing - this is implemented in the new gotrue-js, although I dont think we needed the ? in the end - looks like the # might work better with React, although it's hard to say. I get an occasional 400 error on the /example, which I think is caused by the react dev build not dealing with the ? (not compiling properly).
Will revisit this if we see a problem later


nb-select does not correctly compare [(selected)] for number values

Issue type
I'm submitting a ...  (check one with "x")

[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Issue description
Current behavior:

Binding a field using the [(selected)] property of nb-select will not work for the default value on load for number types.
After selecting another option, the select reflects correctly.
Expected behavior:

[(selected)] should work on load for number values
Steps to reproduce:

Create a field that stores number
Bind it to nb-select with options with number values
Observe that the first number does not automatically get selected

Related code:

HTML
      <nb-select fullWidth [(selected)]="numberStore">
        <nb-option [value]="0">
          this should be selected
        </nb-option>
      </nb-select>

Typescript
 numberStore = 0;

Other information:
npm, node, OS, Browser
Node: v10.16.0
NPM: 6.10.2
Windows
Chrome

Angular, Nebular
"@angular/animations": "^8.0.0",
    "@angular/cdk": "^8.0.0",
    "@angular/common": "^8.0.0",
    "@angular/compiler": "^8.0.0",
    "@angular/core": "^8.0.0",
    "@angular/forms": "^8.0.0",
    "@angular/platform-browser": "^8.0.0",
    "@angular/platform-browser-dynamic": "^8.0.0",
    "@angular/router": "^8.0.0",
    "@nebular/auth": "4.1.2",
    "@nebular/eva-icons": "4.1.2",
    "@nebular/security": "4.1.2",
    "@nebular/theme": "4.1.2",

The temporary work around is to cast the selected value and option value to string before binding.
      <nb-select fullWidth [(selected)]="''+numberStore">
        <nb-option [value]="''+0">
          this should be selected
        </nb-option>
      </nb-select>

hi @xucito! Seems that issue doesn't exist. Could you please provide a stackblitz example with the issue?

I have had the same issue but only with numbers, the workaround I have been using is to cast the value as a string in patchValue
this.form.control.numerStore.patchValue(value.toString());


Add support for aborting/canceling requests

Is your feature request related to a problem? Please describe.
It's a common use case to cancel/abort a query when it's no longer needed (e.g user scrolling fast in a paginated list, or user leaving the page/ closing a dialog before the query returns).
Describe the solution you'd like
Add support to cancel requests just like the JS Library

Assuming http quest need to be cancelled, which unfortunately is not yet supported officially in http package to abort/cancel the http request  Design an API to support aborting requests.


Need more documentation

Thank you for your work
This seems very promising, as a small queue
Still, can you add a detailed example as an publisher and as a consumer
I want to be able to update the queue in one app and read it in another app
Thanks

Hi @ehayun ,
Thank for the feedback. I'll see what I can do to make things clearer in the documentation. In reference to your comment regarding updating in one application and reading in another - it won't make any difference to the existing samples.
Providing your two applications have some mechanism for agreeing on the path of the file, the updating application would open the queue as follows:
# open the queue
{:ok, producer} = Stagger.open("/path/to/the/file")

# write the message
:ok = Stagger.write(producer, msg)

And your reading application might subscribe like this:
# create the  consumer
{:ok, consumer} = Consumer.start_link()

# open the queue
{:ok, producer} = Stagger.open("/path/to/the/file")

# connect the consumer to the producer
{:ok, subscription} = GenStage.subscribe(consumer, to: producer)

Thanks
I will try that
The thing is that in the docs there is no sigh for Consumer
Now it is clearer


searching deeper than one association

  pg_search_scope :full_text_search, against: [:confirmation_code, :reference_number, :starts_at, :category, :load_category],
  using: {tsearch: {dictionary: "english"}},
  associated_against: {company_friendship: {company_friend: :name}, carrier: :name, broker: :name},
  ignoring: :accents

How to search through company_friendship which has one company_friend which has a name? Is this gonna be supported in the future?

You could possibly do this by adding a has_one :through association for the company_friend.


Search certs by public key

Right now it's hard to search for certs by public key. You can search by hash or timestamp in the normal protocol. We could add a new index to the CT log or write our own index/searcher.

This has been fixed over in rekor.


Think about allowed Windows setups

Should using the Linux subsystem be the only allowed setup?

From meeting: we will keep the dockerfiles related to Windows, but only write about the Linux subsystem in the setup.


Can't docker-install

docker build --tag=bikeshed:latest .

Fails with
/usr/local/lib/python3.7/site-packages/pip/_internal/commands/install.py:236: UserWarning: Disabling all use of wheels due to the use of --build-option / --global-option / --install-option.
  cmdoptions.check_install_build_global(options)
ERROR: Location-changing options found in --install-option: ['--prefix'] from command line. This is unsupported, use pip-level options like --user, --prefix, --root, and --target instead.
The command '/bin/sh -c pip install --install-option="--prefix=/install" -r /requirements.txt' returned a non-zero code: 1

Docker version 19.03.12, build 48a66213fe, on OSX.

I'm having the same issue, is there a workaround for this?

I don't have a single clue how Docker works; the current documentation and support for using Docker is based on one user who apparently understands it.  I can't meaningfully support this, unfortunately, but I'm happy to update docs as necessary if y'all find out how.

Ah no worries. To be honest I have no idea how docker works either. I decided to use it for the first time today because I wanted to see what all the fuzz was about.
I suppose pip3 install bikeshed is the workaround then.


Support for contentPadding

In my use case the table is full screen but as I've transparent navigation bars the table is displayed under the system bar. Would be nice if support for contentPadding: PaddingValues could be added so it can scroll from edge to edge.

Released in 1.1.0 🎉


Update translation-fr.json

ortho and typo
Changes proposed in this pull request:

"nouvel" and not "nouveau"
"élément" and not "Élément"

Thanks! I assume you also know about our Weblate interface when you can do those changes? It is also fine to submit PRs like this one of course.
https://hosted.weblate.org/engage/openrefine/


Ethereum address does not appear when using BIP39 tool to find my private key.

I sent Vechain to my Ethereum wallet on my ledger nano X by accident. Now I need the private key in order to get my funds back. Ive tried numerous times to find my private key and my ethereum address is not showing up. Ive tried bip44, bip32, and even bip32 while changing the derivation path to m/44'/60'/0'/0. I am using 24 words and also using my ledgers passphrase. Im completely lost at this point does anybody have any idea what I might be doing wrong?

Have you tried BIP32 derivation path m/44'/60'/0' already? So that the 1st key itself is m/44'/60'/0'/0? Ledger wallet is "special" and they are not 100% compatible with BIP44 separating receiving and change addresses.

This was working for me, thanks do much

pls.. who can help set up git clone https://github.com/iancoleman/bip39 . im confused asf


Group Custom Question Bank

It would be cool if- especially for corporate settings, but even for friends/family groups- there could be a custom question bank. We could even give an editable prompt to users to pass to chatGPT- this would save any AI API calls on Mailpals end and allow users to curate their own questions.
For example, if I were to make a mailpals group with coworkers, I'd want questions that are specific to our work environment/ culture.
This can be done by toggling an option in group settings under # of questions option, which would reveal a textbox. Questions can be parsed just by newline (to account for questions that are 2-in-1's).
Bonus: an additional toggle to opt between "only pick from this question bank" or "use a combination of these questions and mailpals-suggested ones"

Good idea. A part of this would also be to allow members to suggest questions for the question bank / next issue

Totally. Two questions-

Are you saying that you'd have the site ingest these custom questions and build a database for use in similar groups? No clue how that would work but sounds cool.
How to keep track of which questions were already asked to eliminate repeats? Do we care about repeats?


Add KACARE stations

SolarAnywhere benchmarking used a number of high-quality stations from KACARE (King Abdullah City for Atomic and Renewable Energy): https://www.solaranywhere.com/resources/data-validation/

Email sent to the Solar Energy authors on 30-01-2024 to enquire about which stations are still active.
The API provided only gives metadata.

More material here and the actual map of stations with photos is here.
Some info on instruments is here.
The network looks active.

What is your "looks active" comment based on? Have you seen that you are able to get recent data or?

No, I did not get any data (it required to make an account, etc. so I did not look into that). I said that it "looks active" because they have grayed out some stations in their atlas, showing that they are not in operation. So I assumed that the rest are in operation... of course, that makes sense as long as they have an updated website (which I am not sure of)! I am also ok to change them to "?" if you think it makes more sense.

Question mark'em and I will merge it!

Question mark'em and I will merge it!

Done!


load schema from server

Hi again, is there a good way to load the schema from a server? We're using this to build a customizable form that can be embedded on any site. New to Vue and not sure how to defer loading of the component until the schema is available.
Thanks!

So you want to prevent showing the component until you get the schema from server?

@cristijora correct!

If you use vue router you can prevent a route from loading until you tell it tohttp://router.vuejs.org/en/advanced/data-fetching.html

We're trying to keep this as light weight as possible given this is an embeddable widget. Is there a way to do it without the router?

Show a loading animation until you get the data from server. Other than the vue router option to really prevent data I don't know any way of doing it. Maybe load the schema in the parent component and render the widget only when data is present


custom user-data property

reopen the closed #80 (test report in it), because after a few days i feel this one is more convinent then an integer field, and another advantage is this way it won't deal with the RS side.
guess CI check will fail because of the ES5 option in the tsconfig.json.

ok, CI failed as i guessed. (used Map, ES6 required)
check what we discussed in #81 out, personally i think ES6 won't bring compat issue, because:
1, the current build of rapier.es.js needs a browser which supports es module import, and,
https://caniuse.com/?search=es module
such browsers are more newer then those which support ES6. (https://caniuse.com/?search=es6)
2, WebAssembly support:
https://caniuse.com/?search=webassembly
the same, higher requirement on browser then ES6. (https://caniuse.com/?search=es6)
3, the JS output with ES6 version is more effeicent then ES5.
but anyway, i don't know, maybe need some feedback from Rapier users first.

Hmm I'm wondering if it's worth supporting this natively in the lib. It's relatively easy to do in user land this with Map<handle, metadata> and there might be performance implications of adding this. I found this talk really interesting but not sure how much of this translates to JS

Hmm I'm wondering if it's worth supporting this natively in the lib. It's relatively easy to do in user land this with Map<handle, metadata> and there might be performance implications of adding this. I found this talk really interesting but not sure how much of this translates to JS

no, it won't affect performance, because it won't deal with Rust. yes as you said, user can add their own user-data map in their logic, but actually this should be part of the lib because i think most of the dev users need it.
btw, do you think upgrade to ES6 compiling will affect something or not?

no, it won't affect performance

The TLDR of the talk is to keep memory usage as low as possible, so it fits in CPU cache instead of memory which improves perf quite a bit. But yeah, don't feel like I understand enough of the internals yet to say if it makes a difference

btw, do you think upgrade to ES6 compiling will affect something or not?

Hard to say, but IMO it's fine. Especially since it's a pre-release/alpha. If it breaks for a consumer for whatever reason they can add the needed polyfill or configure their compiler to also compile the Rapier pkg

no, it won't affect performance

The TLDR of the talk is to keep memory usage as low as possible, so it fits in CPU cache instead of memory which improves perf quite a bit. But yeah, don't feel like I understand enough of the internals yet to say if it makes a difference

btw, do you think upgrade to ES6 compiling will affect something or not?

Hard to say, but IMO it's fine. Especially since it's a pre-release/alpha. If it breaks for a consumer for whatever reason they can add the needed polyfill or configure their compiler to also compile the Rapier pkg

good video! thanks for sharing.

I would prefer it to be in the user land. I agree that the feature itself is pretty common, but also not always mandatory. It is often needed for cases, when you are mapping a visual representation of an object in the renderer with the physical one in the physics world. On the other hand, there isn't always a visual representation. For server-authoritative physics in multiplayer games - there is no visual mesh. Also having it in user land, allows the dev to decide on the map structure, e.g. using Map vs WeakMap or other object that fits their design and configuration.

close this because there will be a plain userData field stub in PR #98
thanks guys for discussing.


fix(client): allow readonly array and object as JSON input

Allow both readonly and mutable arrays and objects as JSON input values.
Closes https://github.com/prisma/prisma/issues/10165

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

Hi @Akxe and thank you for your PR 👍💯
There are some failing tests on CI, seem to be the snapshot tests only. Could you please update the snapshots? If not, that's also fine, I can do that as well.

@aqrln Sadly I am not sure how I did the change in the browser...

Is the buildkite/test-prisma-typescript/pr error a false negative?

Yes, MongoDB failed to start:
[2021-11-25T10:16:27Z] ERROR: for mongo  Cannot start service mongo: driver failed programming external connectivity on endpoint buildkite51bb967838824f0ba115922e0cb4f79d_mongo_1 (b80373ee0cc3f374efbd0f4862e8340076e4b263219e8045a30572431fd13850): Bind for <IP_ADDRESS>:27017 failed: port is already allocated

Is it then good to go?

Added tsd tests as well.

Cool did not think of that :)

@Akxe

Is it then good to go?

Pending review from others, yes.
Assuming I didn't miss something important and nothing unexpected happens, this should hopefully make it into the 3.6.0 release.

This is quite a bit more work than anticipated :/


JBPM-5710 KIE Persistence Cleanup

This set of PRs is a set of modifications to allow multiple persistences without all of them depending on JPA:

Creating drools-persistence-api and jbpm-persistence-api
SingleSessionCommandService refactors
PersistenceContext and ProcessPersistenceContext refactors:
TaskPersistenceContext refactors
Changes in the current persistence model interfaces
TimerJobFactoryManager / TimerService refactors

looks good to me just if you could squash commits to single one with commit message with jira
@psiroky any comments from your end?

LGTM

@marianbuenosayres could you please squash all commits into one with jira commit message?


What is the meaning of the PASS value next to services in the main Services screen?

This is only suppose to show if posture checks are available, I have fixed that, PASS means they are passing the checks FAIL means they are failing.


Typo "| **Number **|"→"| Number |"

https://docs.microsoft.com/en-us/azure/defender-for-iot/how-to-set-up-your-network

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

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

#sign-off

#sign-off


WIP: Automatically derive nested pydantic class

Could we automatically derive nested pydantic classes?
Suppose someone has a repo using pydantic and wants to start using strawberry.
It is troublesome to manually create a new class for each pydantic basemodel and decorate it with @strawberry.experimental.pydantic.type. (Thats the issue i am facing now)
Could we automatically (optionally) derive strawberry types for nested pydantic classes?
My initial MR works for this basic example. I'm opening one early so i can find out what issues may occur.
from typing import List
import strawberry
from pydantic import BaseModel

class Child(BaseModel):
    value: str

class Parent(BaseModel):
    children: List[Child]

class GrandParent(BaseModel):
    children: List[Parent]

@strawberry.experimental.pydantic.type(model=GrandParent, all_fields=True)
class GrandParentStrawberry: # only need to decorate GrandParentStrawberry
    ...

@strawberry.type
class Query:
    @strawberry.field
    def hello(self) -> GrandParentStrawberry:
        return GrandParentStrawberry.from_pydantic(
            GrandParent(children=[Parent(children=[Child(value="123")])])
        )

Description

Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[ x] 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.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to James Chua for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

I'm not opposed to this :) I definitely like the idea of making it optional
Let's see what other people think! @la4de (this might be something we could, if we don't already, do for django), @jkimbo, @BryceBeagle, @marcoacierno

thanks,

I'm not opposed to this :) I definitely like the idea of making it optional
Let's see what other people think! @la4de (this might be something we could, if we don't already, do for django), @jkimbo, @BryceBeagle, @marcoacierno

thanks, i've fixed the tests and added options for registration of nested basemodels. Let me know if that is desirable and i can update the docs

I know we are working on first class pydantic support. However, this PR would benefit me highly!
Is there any reasonn this hasn't been commited? Other than a release.md ?

I know we are working on first class pydantic support. However, this PR would benefit me highly! Is there any reasonn this hasn't been commited? Other than a release.md ?

From my side, I don't use Pydantic much so I haven't been able to test it properly :)
@XChikuX would you be able to test this if I make a pre-release? :)

@XChikuX rebasing this might take me a bit of time :D

I'm not really sure if this pattern works appropriately in all cases. Like
what happens if you want to manually add fields in the converted type. How
do you make sure the user has control over it.
On Wed, Jan 18, 2023, 6:22 PM Patrick Arminio @.***>
wrote:

@XChikuX https://github.com/XChikuX rebasing this might take me a bit
of time :D
—
Reply to this email directly, view it on GitHub
https://github.com/strawberry-graphql/strawberry/pull/1477#issuecomment-1386816837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHI2757ZM4QBZA5PMCZIHGTWS675BANCNFSM5JQ3GYLA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm not really sure if this pattern works appropriately in all cases. Like what happens if you want to manually add fields in the converted type. How do you make sure the user has control over it.

that's a fair point. We need to think more about this pattern. I also see a future we enable pydantic via something like this:
@strawberry.type # or pydantic.type
class MyType(BaseModel):
    ...

and then in that case it would work since we are exposing all the fields (we'd basically treat pydantic as a first class citizen).
But for how strawberry.experimental.pydantic works, we probably might want to have a different behaviour. Maybe we can add a flag on type (ie convert_nested=True)? it's a bit ugly though

I agree we shouldn't throw this into main without covering the edge cases.
Maybe can just leave it as nested=True for now, Conversion should be implied in the one word.
I don't mind waiting for pydantic support, if its coming in the next month or two.
Otherwise, I'd like to push for this since its already half implemented. I can try to cover the edge cases like what @thejaminator mentioned when I get time.
My code is quickly getting verbose using the experimental library.

Sure, go ahead. I'll suggest that instead of a parameter is_nested: bool,
make it something like nested_options: Optional[NestedOptions].
NestedOptions can be a data class that describes the settings for
conversions. That allows us to add more settings in the future.
The None case will indicate our current behavior.
There's a few options that i foresee being useful. Like a callable that
allows you to specify the type renaming behavior. And in the future maybe
something like a set of  Type[BaseModel] that the user can add to exclude
from automatic conversion.
On Wed, Jan 18, 2023, 7:13 PM Srikanth @.***> wrote:

I agree we shouldn't throw this into main without covering the edge cases.
Maybe can just leave it as nested=True for now, Conversion should be
implied in the one word.
I don't mind waiting for pydantic support, if its coming in the next month
or two.
Otherwise, I'd like to push for this since its already half implemented. I
can try to cover the edge cases like what @thejaminator
https://github.com/thejaminator mentioned when I get time.
My code is quickly getting verbose using the experimental library.
—
Reply to this email directly, view it on GitHub
https://github.com/strawberry-graphql/strawberry/pull/1477#issuecomment-1386884230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHI275Z4LJ367VUDBVSX6MTWS7F5DANCNFSM5JQ3GYLA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@XChikuX i'll commit to coming up with something during our strawberry sprint on 18 feb :)


Adding ability to download KBs when running Update-DbaInstance

Type of Change

[ ] Bug fix (non-breaking change, fixes  )
[x] New feature (non-breaking change, adds functionality, fixes #5043 )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Add -Download functionality to Update-DbaInstance
Approach
When enabled, function would download each needed KB to the temp/network drive and (if needed) copy it to the remote machine.
Commands to test
Added an example showcasing the parameter usage.
Screenshots

Learning
Start-BitsTransfer was annoying and didn't want to play nicely with PSDrive drives. Had to switch back to Copy-Item.

It Update-DbaInstance.ps1 is not compliant with the OTBS formatting style. Please run Invoke-DbatoolsFormatter against the failing file and commit the changes.

I did, what does it want from me huh

I don't understand. The tests pass locally. Invoke-DbatoolsFormatter leaves the file as is.
 $content = Get-Content -Raw ..\functions\Update-DbaInstance.ps1 -Encoding UTF8
 $result = Invoke-Formatter -ScriptDefinition $content -Settings CodeFormattingOTBS
 $result -eq $content
True

@niphlod can you take a look? this issue also occurred on another Draft PR and I couldn't resolve it.

k, i committed the code from dev which should have the fix 👍

Thanks for making it work, somehow @potatoqualitee ! It seems however that the fix broke the indentation - it's not what it's supposed to be now, as confirmed by
}
}

at the end

mannnn how on earth does that happen? i made some changes that were suggested by vscode or whatever. reverted changes. can you help @niphlod ? D:

whoops, I didn't receive the notification for this ! totally gonna inspect in a few hours

Should be fixed now.

That seems to have worked, @wsmelton ! It didn't break formatting at the bottom either, like mine

nope, i was wrong. gosh this is a miserable experience

That can be ignored I think, and is likely coming from the script block. It is something to do with PSSA rule in indentation.

@nvarscar can you download the latest change and test in yours?

My version shows no indentation issues. If you look at the problems panel on yours @potatoqualitee it will show you the rules it thinks are violated.

It doesn't change the formatting on my end after @wsmelton 's commit. Thanks!

oh yay, than you both! 💯 🚀 !

love it!!


Allow touchableType

Why not allow touchableType to enable a Slack-like drawer? Can default to 'opacity'

i think this is quite limited compared to allowing users to provide a touchable component, eg: https://reactnavigation.org/docs/en/bottom-tab-navigator.html#tabbarbuttoncomponent

Closing due to no response.


VictoriaLogs

Does the operator support VictoriaLogs? If not, are there plans too?

Hello, yes, we've such plans, but there is not ETA for it.
And it'll be challenging. Current CRD bundle almost exceed 1MB limit.

And it'll be challenging. Current CRD bundle almost exceed 1MB limit.

I think you may actually be okay. There are multiple CRD objects, and the cumulative size is not relevant to the 1M size limit. Only if an individual object exceeds 1M will it be a problem.
https://github.com/kubernetes/kubernetes/issues/82292

I've made some research.
Current  the only way for adding VictoriaLogs - completely remove any k8s server side validation. CRD will have only spec definition with preserve unknown true annotation.
For actual validation, webhook must be used.
I propose, to add VictoriaLogs as Deployment and use the same behavior as VMSingle does.
cc @Amper @Haleygo

Current the only way for adding VictoriaLogs - completely remove any k8s server side validation. CRD will have only spec definition with preserve unknown true annotation.
For actual validation, webhook must be used.

I think it may leads problems for some users due:

they use CRD specification with different handy k8s tools (like kubectl explain / kubectl explore).
they doesn't use webhooks because:

it block resource deploy when the operator is not running - pipelines will be failed
therefore, this requires HA mode deployment and load scaling
require certificates

released in v0.47.0


Use New-AzStorageContainer instead of alias New-AzDatalakeGen2FileSystem

New-AzDatalakeGen2FileSystem is an alias. It may make confuse. Please use formal name New-AzStorageContainer on online doc.
Original question is from https://github.com/Azure/azure-powershell/issues/12038
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 562aa838-243f-bed8-c3cb-9164af5efe71
Version Independent ID: 369544f3-5260-3773-68f2-3479c6e9587e
Content: Azure Data Lake Storage Gen2 PowerShell for files & ACLs
Content Source: articles/storage/blobs/data-lake-storage-directory-file-acl-powershell.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@dingmeng-xue  Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

Thanks @dingmeng-xue . I switched out the command as per your request. The change should appear in the live version of the documentation by tomorrow morning.

#please-close


Update README.md: Fix quickjs urls to original repo

Oh good catch! All you need to do is remove the () from the first link. The QuickJS reference is at the bottom of the file.

Fixed in https://github.com/quickjs-ng/quickjs/commit/bc1e3cd00946482d7467be6c6433a74422755950


⚠ Bofrost.es has degraded performance

In 745a8ef, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1504 ms

Resolved: Bofrost.es performance has improved in 932e217 after 10 minutes.


Smoke Test Dev Review

It would be great if each of you can familiarise yourself with smoke test area that you are responsible of (this involves code review, general automated smoke test operation). Smoke test areas are defined as here - https://github.com/Microsoft/vscode/wiki/Smoke-Test.
This is required to ensure we reduce any potential flaws in our smoke tests. Moreover, going further you may need to implement some tests if you add new feature to VS Code, or redesign some of existing smoke test parts due to any UI changes. Hence, it is important that you are familiar with the way automated smoke test is working now.
Here is a general readme on the code - https://github.com/Microsoft/vscode/tree/master/test/smoke.
When reviewing your area make sure you understand:

Code architecture
What to do when smoke test fails (looking it up on VSTS website, downloading test artifacts after build has finished, debugging)
Weak points

Also think of the way you would improve the smoke test, and write your thoughts here. Please check yourself off once you became familiar with it.

[ ] @alexandrudima
[x] @weinand
[ ] @chrmarti
[x] @tyriar
[ ] @dbaeumer
[x] @bpasero
[x] @egamma
[ ] @isidorn
[x] @joaomoreno
[x] @jrieken
[ ] @kieferrm
[ ] @aeschli
[ ] @rebornix
[ ] @ramya-rao-a
[x] @roblourens
[ ] @sandy081
[ ] @mjbvz

All feedback is welcome and appreciated, skepticism should be transformed into suggestions together with contribution to automated smoke test implementation, ensuring it is always reliable! 🙂

I haven't mentioned it anywhere else, hence adding it here.
If you see test failure that you cannot reproduce locally, you might want to check out screenshots generated by smoke test. You can do it on VSTS websites, by going to build and dowloading build artifacts as on the image below.

Multi root smoke test: https://github.com/Microsoft/vscode/pull/31059

Why do we often send a 'NULL' key? https://github.com/Microsoft/vscode/blob/master/test/smoke/src/areas/search.ts#L20

@roblourens it indicates KEYUP event to the webdriver.

I see, a helper for that would be useful

Closing - the internship of @michelkaporin is over and we are now on our own 😄

Some smoke tests depend on each other. e.g. a git test depends on an exact number of created or modified files. If you disable a test that creates or modifies files, you will have to adapt the git test, other wise it will fail.

@weinand this is correct, as the automated smoke test was implemented with the same scenario as in manual smoke test we did before. The latter one had the same dependency as you've mentioned above. If we split each area not to be dependent, we lose end-to-end scenario, thus reducing scope of testing. As a solution every test can become self-contained, but the time it would take to perform such tests would be much bigger.


Mr.

Description
[Enter feedback here]
Good afternoon sir/ma,  I want to ask the details of my shares,
Page URL
https://learn.microsoft.com/en-ca/aspnet/core/host-and-deploy/azure-iis-errors-reference?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/host-and-deploy/azure-iis-errors-reference.md
Document ID
daf78291-6119-23c6-e1b1-0606a74bd510
Article author
@Rick-Anderson

Good afternoon sir/ma I bought a  shares some years ago, but unfortunately I lose my phone, but since that time, I never received details on that shares, please I need it,


CI: Xvfb not starting

This might actually be better suited for an issue in PX4/containers, but here it is.
On my local Jenkins machine I can check the output from running Xvfb in the Docker entrypoint script. Here's what I see:

_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.

I'm thinking we have a permission problem. The docker plugin for Jenkins automatically runs the container as the current user. Currently in CI this is uid:gid 1000:1000. On my machine it is different, but it doesn't matter. I think the same thing is happening in CI. This user isn't root. So, Xvfb can't start.
Additionally, adding the 1001 jenkins user in the docker file doesn't seem proper knowing this now.
@dagar if you can get into Jenkins master and execute docker logs with a Jenkins container you can see the output from the Xvfb command.

I'm seeing a similar problem (errors) when running trying to set up SITL in a Docker for Windows.  I don't think I'm using Xvfb in my application anyway, but something isn't happy with how I was installing and running things in the container.

What is your error?

What is your error?

@Antiheavy That has been fixed with https://github.com/PX4/containers/commit/31dc86bd112d3aa28ad44c6dee7f0cbd93c83728. We need to update the container tags for it to be realized on your end.

After some digging I found two problems. I'm just going to note them here before I work on an actual fix.

Permission error for xvfb -- this can be fixed pretty easily
DISPLAY env var isn't persisted when set in the entrypoint script -- This requires more thought on the best way to do this. We need another way of setting it and therefore a different way to trigger Xfvb to run in the entrypoint.

After some hackery, I got it to work as it should, but the method I used is not good for mainstream. More to come.

There are two ways to approach this. Either use it all the time (Fix in dockerfile) or only want to used in Jenkins (fix in Jenkinsfile and entrypoint). Any thoughts on this?

@lamping7  thanks for looking into this.
FYI @dagar

See https://github.com/PX4/containers/pull/179. It was decided to use Xvfb by default, but allow users to override the DISPLAY with docker run -it -e DISPLAY=${DISPLAY} ... as show in the existing README. This means Jenkins will be able to use it without any changes over here in Firmware.


The rsync array options should be merged with defaults

Deployer version: 7.0.0-rc3

Deployment OS: Mac OSX Monterey
Please, provide a minimal reproducible example of deploy.php

https://deployer.org/docs/7.x/contrib/rsync
When defining rsync options only with exclude:
set('rsync',[
    'exclude' => [
        '.git',
        'deploy.php',
        'assets',
    ],
]);

The deploy will fail:
[<IP_ADDRESS>]  ErrorException  in rsync.php on line 257:
[<IP_ADDRESS>]
[<IP_ADDRESS>]   Undefined array key "flags"
[<IP_ADDRESS>]

Not sure if it would be possible to define defaults somewhere else so I need not configure all rsync options.
PS: I hope its fine that I created seperate issues and not did pack all into one big issue.

Can't you do:
set('rsync', function () {
  return array_merge(get('rsync'), [
    'exclude' => [
      '.git',
      'deploy.php',
    ],
  ]);
});

Hello,
Same issue when importing in the deploy.php a yml file with custom conf
Placing it in the dpeloy.php makes no sens as the list of excluded files can changed from projects to another.
    rsync:
      exclude:
        - '.git*'
        - .bundler
        - deploy.php

We are obliged to put the whole thing
    rsync:
      exclude:
        - '.git*'
        - .bundler
        - deploy.php
           flags: 'rzcE'
      options: ['delete', 'delete-after', 'force']
      timeout: 3600
      include: []
      filter: []
      exclude-file: false
      include-file: false
      filter-file: false
      filter-perdir: false


package.sh: upload raw binaries to S3

/cc @beckettsean @gunnaraasen how do we feel about distributing linux binaries of telegraf? I was thinking that this might be good for people who want to try out telegraf without installing a .deb or .rpm which also installs an init service on the server.
Also nice for people who just want to manage the service themselves

Sounds good to me. I think there was a request for something similar for InfluxDB but I can't find the issue.

Let's do it. We get requests for binaries for chronograf and influxdb fairly frequently, it's just a matter of time for telegraf.

great! will also probable need an influxdb.com update for the downloads page, they'll start getting uploaded with 0.1.8

I added a note to https://github.com/influxdb/influxdb.com/issues/267 to document the telegraf tarball, too.

Cool, I'll update this PR to actually tar -zcf the binary rather than zip, just for consistency with the DB

@sparrc agreed. I think gzipped tarballs are more standard than just gzipped files.


external control plane

Enable sriov operator in the cluster w/o master nodes (k8s apiserver and other control plane components are managed externally).

@zshi-redhat can you explain abit more about the use-case ?

Can you put your first comment in the commit message? Or something more elaborate?

Can you put your first comment in the commit message? Or something more elaborate?

added the comment in the commit message.

/lgtm

Hi @zshi-redhat will you please be able to rebase this PR?

Hi @zshi-redhat will you please be able to rebase this PR?

rebased, thanks for reminding!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 60 (18.33%)  changed or added relevant lines in 2 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.04%) to 15.784%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

controllers/sriovoperatorconfig_controller.go
3
6
50.0%

pkg/utils/cluster.go
8
54
14.81%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovibnetwork_controller.go
2
67.33%

controllers/sriovnetwork_controller.go
2
67.0%

api/v1/helper.go
6
34.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
1162

Relevant Lines:
7362

💛  - Coveralls

@zshi-redhat We could label the workers in guest cluster with label "master" (such that the role will be master,worker). In that case, this would not be needed. For the sake of my understanding, is that correct? I still think it's nice to allow to run on workers so we will still want to have this PR in.

@zshi-redhat We could label the workers in guest cluster with label "master" (such that the role will be master,worker). In that case, this would not be needed. For the sake of my understanding, is that correct?

@bn222  This may not be desired from my understanding, since the node pool is supposed to only contain worker nodes, not sure if there will be any side effects to other pods.
btw, do we still need this PR? or is it going to be integreted to Tyler's?

@adrianchiris  PTAL.

/test-all


Clicking Go Live with Sound does nothing.

The daemon recieves a invalid type
   ~/source/Tuxphones/daemon    main    cargo run                                                                                 ✔  4s 
warning: unused variable: gst_resize_rx
--> src/lib.rs:110:53
|
110 | ...                   let (gst_resize_tx, gst_resize_rx) = channel(10);
|                                           ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: _gst_resize_rx
|
= note: #[warn(unused_variables)] on by default
warning: field encoder is never read
--> src/gstreamer.rs:98:5
|
94 | pub struct GstHandle {
|            --------- field in this struct
...
98 |     encoder: Element,
|     ^^^^^^^
|
= note: #[warn(dead_code)] on by default
warning: tuxphones (lib) generated 2 warnings
Finished dev [unoptimized + debuginfo] target(s) in 0.10s
Running target/debug/tuxphones
2022-12-02T02:01:28.122999Z  INFO tuxphones: Daemon started
2022-12-02T02:01:47.400511Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:01:47.415168Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
2022-12-02T02:01:48.921137Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:01:48.934695Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
2022-12-02T02:01:49.941016Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:01:49.947888Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
2022-12-02T02:01:50.952460Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:01:50.965792Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
2022-12-02T02:01:56.615958Z ERROR tuxphones::socket: Failed to deserialize command: invalid type: null, expected a string

This is one of the issues we're having, try reloading Discord (Cmd/Ctrl+R) then try to go live again. It comes from us being unable to get an IP address that we need. Once I resume development on the plugin around mid-December I will try to fix it.

2022-12-02T02:21:54.097748Z  INFO tuxphones: Daemon started
String: {"type":"GetInfo","xids":[29360136,100663306,100666156,4194307,106954785,134217729,106954871,106954865,58720263]}
2022-12-02T02:22:04.828261Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:22:04.850224Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
String: {"type":"GetInfo","xids":[29360136,100663306,100666156,4194307,106954785,134217729,106954871,106954865,58720263]}
2022-12-02T02:22:06.360372Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:22:06.380296Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
String: {"type":"GetInfo","xids":[29360136,100663306,100666156,4194307,106954785,134217729,106954871,106954865,58720263]}
2022-12-02T02:22:07.386118Z  INFO tuxphones: [GetInfo] Command received
2022-12-02T02:22:07.427607Z  INFO tuxphones: [GetInfo] Command processed (applications found: 1)
String: {"type":"StartStream","pid":26752,"xid":134217729,"resolution":{"width":0,"height":0,"is_fixed":false},"framerate":60,"server_id":"1048061479917727825","user_id":"147892034932441088","token":"08cf4e400ec6991c","session_id":"58b06360538e03801016934d09971136","rtc_connection_id":"4165cf79-3f39-4534-b49b-717127022b22","endpoint":"us-east11072.discord.media:443","ip":null,"ice":{"type":"IceData","urls":["stun:global.stun.twilio.com:3478?transport=udp","turn:global.turn.twilio.com:3478?transport=tcp","turn:global.turn.twilio.com:3478?transport=udp"],"username":"4aac1e53ade1a5473f8b5da67be3b591113cad11a9c75f957537026f628111fa","credential":"dyH2YPGFDI8rgDcaAl73jJOR7ga/st4/YpNxsVJ498A="}}
2022-12-02T02:22:09.038634Z ERROR tuxphones::socket: Failed to deserialize command: invalid type: null, expected a string

Thanks for such a quick response!  Keep at it, you've given me a great reason to learn rust.  <3

Ahhhh I see the ip is null.  Interesting.

Is it the ip of the endpoint you need to resolve?

This should be fixed by e1ae4cb


blocksync: export errors

Contributes: #1140
Changes
Export errors in blocksync package. More context: #1140

hey @thanethomson, comments addressed, I think this PR is good to go!


Form schema fixes

Further work on https://github.com/silverstripe/silverstripe-framework/issues/4938, in preparation for a React-powered asset and campaign admin.
/cc @tractorcow @flashbackzoo

@silverstripe/core-team We're hoping to get this pull request over the line in the next 48h because we have an opportunity for staff to fix things during an internal hackday here at SilverStripe Ltd on NZ Friday, ideally without too much branching fragmentation. Can you please review in your respective areas of expertise (PHP and/or JS/React)? @flashbackzoo, @scott1702 and myself have largely peer-reviewed each others work here, but would be good to get the thumbs up from at least one other core committer since these pull requests lay the ground work for further React integration. You'll see some @todo tags, some of it is early stages, but we agree with @sminnee's guidance that we should try to integrate early and often in our product development cycle.

 Is it just the one $schemaDataType property that’s the problem? Can we work around it somehow?
public function getSchemaDataType() {
    if (!$this->schemaDataType) {
        $this->schemaDataType = 'Text';
    }
    return parent::getSchemeDataType();
}

Or possibly call $this->setSchemaDataType('Foo'); in the subclass constructors?

 For me, it's on the same level as creating a "section" in a class definition through some kind of PHP comment markers. I can't see a reuse for this trait outside of the FormField class. So the only thing it does is confuse devs with less powerful IDEs on where a certain method is defined.

I'm with @kinglozzer on this one. I think breaking down code into smaller pieces is a pretty fundamental piece of good software design.
I'd shift only the implementation of getSchemaDataType() (and the protected property) into FormField, and declare it as abstract in the trait.

OK three votes for traits (@sminnee, @kinglozzer, @tractorcow , one against (@chillu - in this context). I'll add the trait back in, and move getSchemaDataType() and getSchemaComponent() to FormField.

Jumping in for no good reason, while I agree that breaking code into smaller pieces is good design, using a trait to do it seems a bit smelly. You can't inject or replace traits, and they share the same access space as the class they're attached to. Why isn't this a seperate class with a decorator pattern? If the argument is "Form Fields should be able to override specific behaviour", the argument for moving that out into a different class is the same as the argument for moving the base trait behaviour into a seperate class.

I like traits but I don't care that strongly about it in this case. I'd have done it if it were up to me, but happy for it to be left out for this PR.

Why isn't this a separate class with a decorator pattern?

Also fine with me.

I'm going to merge, since "moving schema into a trait, extension, or other class" isn't something that we can't do at a later date. The tests pass so let's keep moving onwards.

Maybe I'll wait for @chillu to respond to the query on using constants; Since that seems to be the only un-addressed point here.

OK well, if Ingo and Damian are both okay with the solution, I'm not going to block merge of this.

However, it would be good to hear if @kinglozzer thinks we should hold fire and refactor this some more before merging...

Yeah moving it back into a trait is my preference, but I don’t feel strongly enough about it to block merge, so +1

OK, updated the PR according to feedback:

Added a comment regarding definition of $schemaDataType in subclasses
Using constants: FormField::SCHEMA_DATA_TYPE_*. A bit lengthy, but just TYPE might be too confusing.
Haven't re-instated the trait, as per latest feedback.


adding cluster name to alert message and dedup_key

added ability to add cluster name to alert message and dedup_key via CR environment key

Please link to issue, what issue was this fixing? We should be layering our changes on top of the opensource project. These changes make our operator fork API incompatible with upstream grafana-operator.

@drewwells, there was a general recommendation from management not to point to any internal issues in open source repos


Decide UI layout

Nick had a great animated webpage layout which we have decided works well for the purposes of this project.


OpenGL support goes weird in Win10 Home edition

It is known that mstsc graphic driver doesn't support opengl. When opening a game which needs opengl, it can not succeed in creating a window. You will get a error message like this.
Error launch application
Can't create gl window
More info:
GLFWError #65542 Happen, WGL: The driver does not appear to support OpenGL
(Something wrong when updating the screenshot, so I typed the error message here)
However, when I am using rdpwrap in Win10 Home edition, the window is successfully created in the remote session. I wonder how it happened, and if Pro edition can do something to support opengl.

Does your remote OpenGL application work correctly, or it only creates a window and no more?

I get the same error message using the glfw library. It does not create the window.
To repro:
import glfw
glfw.init()
window = glfw.create_window(50, 50, "Test OpenGL Window", None, None)

OpenGL support depends on graphics card. Intel and AMD support opengl without problems, NVIDIA support opengl through remote desktop only on expensive Quadro cards. On desktop cards it is blocked.

Closed as solved. If not - reopen it.


These are not translated yet. We are waiting for someone using that language to contribute. If you would like to help, you're more than welcome ;)


Replaced certain beam segment properties that had the wrong unit with…

… their correct units. Also, I needed to define some new KoQ's in CifUnits for this purpose.

I assume you're taking the deprecate/hide approach for these properties because the Custom-attribute way to change a property's KOQ to a different dimension didn't work?

Nope, I just forgot about that support. Thanks for the reminder. I'll push changes shortly to go that route.

@diegoalexdiaz I switched to using the schema update koq switch tags. The automatic checks seem to be passing now. I think it is ready for review again when you get the chance. Thanks!


Use Existing DLL and not Build csproj (for Larger Solutions)?

Thank you for your efforts with this project 🙏
Everything works well and I was able to generate the code for my application.
The only concern is build time.  I experience this with EFCore migrations/commands, too, and it's rather frustrating.  It takes about 90 seconds (or longer) for a build to occur, mostly because dotnet build does not leverage build acceleration.
Ideally, it would be valuable to have a parameter to use the existing .dll (if it can find it of course) and use that instead of rebuilding the project, which takes a lot of time for larger solutions.
Thank you for any consideration.

It should work if you pass existing dll path:
dotnet-generate-client MyApiProject.dll -o OutPath

It should work if you pass existing dll path

And it does!  AWESOME!  Thank you @Dreamescaper for that pointer and for your efforts. 🙏🙏🙏


Add worker-tidb

Add worker-tidb. https://github.com/cloudflare/workers-sdk/issues/2725
The mysql driver is copy from worker-tidb. And I optimize the driver to let it work with tidb.

This looks really cool! Would you be willing to move this over to the workers-sdk monorepo where the templates have been moved? The team can review and discuss it being added from there 😄
https://github.com/cloudflare/workers-sdk/tree/main/templates


xcode7 and linking issues with libstdc++  / libc++ ?

log_openvdb.txt
seems like openframeworks needs to be compiled with libc++ but openVDB needs libstdc++ , I've tried going into "build phases" and adding libstdc++.dylib but I can't get past this compile issue, have any ideas?
Undefined symbols for architecture x86_64:
"openvdb::v2_3_0::io::setGridBackgroundValuePtr(std::__1::ios_base&, void const*)", referenced from:

You'll have to either use an OF compiled with the same compiler or recompile the libs with the same compiler as that used with your OF.  I was using a 64-bit branch of OF.


Cant pass parameters whit "_" in name

First at all.
Thanx a lot for making this.
Looks like if you try to load reportviewer in view passing params whit name whic ontains "_".
I frame won't be loaded.

"" does not work when you are passing it inside an anonymous object. HtmlHelper.AnonymousObjectToHtmlAttributes call translates "" to "-". This is the reason you report fails.
You should pass IEnumerable<KeyValuePair<string, object>> instead:
@{
    var parameters = new List<KeyValuePair<string, object>>
    {
        new KeyValuePair<string, object>("Parameter1", "Value 1"),
        new KeyValuePair<string, object>("Parameter2", 123),
        new KeyValuePair<string, object>("Parameter3_Test", 12345),
        new KeyValuePair<string, object>("_Parameter4", null)
    };
}

@Html.MvcReportViewerFluent("App_Data/Reports/Products.rdlc").ProcessingMode(ProcessingMode.Local).ReportParameters(parameters).LocalDataSource("Products", Model.Products).LocalDataSource("Cities", Model.Cities).ControlSettings(settings).Attributes(new { Height = 900, Width = 900, style = "border: none" }).Method(FormMethod.Post)


no-img fix

Closes: #1151
Description of changes:

Default Profile Picture Rendering
In response to the absence of a user-uploaded profile picture, a sophisticated update has been implemented to seamlessly display a default profile picture everywhere.

Rectified Story Viewing for Users without Profile Pictures
A remedial intervention has been successfully executed to address the issue of story viewing for users lacking a profile picture. The newly applied solution guarantees uninterrupted access to previously unviewable stories, even in instances where a user profile picture is absent.

@narayan954 i have solved one more bug (story viewing for users with no profile pic) in this PR only. So ig it should be level 2.

@narayan954 i have solved one more bug (story viewing for users with no profile pic) in this PR only. So ig it should be level 2.

ohh alright, fixing that


The CI is broken since v3.0.0

The recent Travis CI runs for PRs 13, 14 and 15 didn't report status checks back to GitHub.
The runs for the first 2 PRs succeeded and didn't trigger a release (expected), but the last run failed and didn't block the merge.
We should:

address the CI issue (npm dependencies are failing to install with node 15)
ensure Travis reports status checks to GH
run the CI to trigger a release
add branch security to the repo settings, merging should be blocked unless status checks have passed

:tada: This issue has been resolved in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Expose the cell_size affecting VisibilityNotifier2D precision

Having the cell size hardcoded means that pixelart games are unable to use VisibilityNotifier2D since their whole screen might be way less than 100 pixels, in which case the visibility notifier would report entering/exiting the screen a few screens away.
Refs #4803.

I did not even know this was using cells.
Anyway, the change is great, it might reduce the confusion when you have a sprite going out off screen and no visibility signal triggered right away. The setting might help understanding that I guess.

Thanks!

Cherry-picked for 3.2.2.


new issue for random mode

I am using this program. But I think I found a bug. It seems to be a bug related to a random query patched in version 2.0.0.
bug log:
<IP_ADDRESS>:54867 - - [23/Mar/2019:15:43:31 +0000] "BLUR FOR DOMAIN. IN TXT"
panic: invalid argument to Intn
goroutine 22 [running]:
math/rand.(*Rand).Intn(0xc00008a180, 0x0, 0x13)
	/usr/local/go/src/math/rand/rand.go:169 +0x9c math/rand.Intn(...)
	/usr/local/go/src/math/rand/rand.go:329 github.com/m13253/dns-over-https/doh-client/selector.(*RandomSelector).Get(0xc0000931e0, 0xc00009cc20)
	/tmp/dns-over-https-2.0.0/doh-client/selector/randomSelector.go:45 +0x3f main.(*Client).handlerFunc(0xc0000955f0, 0x85b660, 0xc0000a05a0, 0xc0000fa090, 0xc00004be00)
	/tmp/dns-over-https-2.0.0/doh-client/client.go:365 +0x322 main.(*Client).udpHandlerFunc(...)
	/tmp/dns-over-https-2.0.0/doh-client/client.go:443 github.com/miekg/dns.HandlerFunc.ServeDNS(0xc000114490, 0x85b660, 0xc0000a05a0, 0xc0000fa090)
	/go/pkg/mod/github.com/miekg/dns@v1.1.6/server.go:37 +0x44 github.com/miekg/dns.(*Server).serveDNS(0xc000112500, 0xc000142000, 0x36, 0x1000, 0xc0000a05a0)
	/go/pkg/mod/github.com/miekg/dns@v1.1.6/server.go:603 +0x2b8 github.com/miekg/dns.(*Server).serveUDPPacket(0xc000112500, 0xc000028020, 0xc000142000, 0x36, 0x1000, 0xc000010010, 0xc000093280)
	/go/pkg/mod/github.com/miekg/dns@v1.1.6/server.go:549 +0xb2 created by github.com/miekg/dns.(*Server).serveUDP
	/go/pkg/mod/github.com/miekg/dns@v1.1.6/server.go:479 +0x27a

I will wait for a quick reply.

Would you please paste your configuration file? (You can censor your privacy info)
By the way, CC @Sherlock-Holo .

# DNS listen port
listen = [
    "<IP_ADDRESS>:5380"
]

# HTTP path for upstream resolver

[upstream]

# available selector: random or weighted_round_robin or lvs_weighted_round_robin
upstream_selector = "weighted_round_robin"

# weight should in (0, 100], if upstream_selector is random, weight will be ignored

## Google's productive resolver, good ECS, bad DNSSEC
#[[upstream.upstream_google]]
#    url = "https://dns.google.com/resolve"
#    weight = 50

## CloudFlare's resolver, bad ECS, good DNSSEC
#[[upstream.upstream_google]]
#    url = "https://cloudflare-dns.com/dns-query"
#    weight = 50

## CloudFlare's resolver, bad ECS, good DNSSEC
#[[upstream.upstream_google]]
#    url = "https://<IP_ADDRESS>/dns-query"
#    weight = 50

# CloudFlare's resolver, bad ECS, good DNSSEC
[[upstream.upstream_ietf]]
    url = "https://cloudflare-dns.com/dns-query"
    weight = 50

## CloudFlare's resolver, bad ECS, good DNSSEC
#[[upstream.upstream_ietf]]
#    url = "https://<IP_ADDRESS>/dns-query"
#    weight = 50

## Google's experimental resolver, good ECS, good DNSSEC
#[[upstream.upstream_ietf]]
#    url = "https://dns.google.com/experimental"
#    weight = 50

## CloudFlare's resolver for Tor, available only with Tor
## Remember to disable ECS below when using Tor!
## Blog: https://blog.cloudflare.com/welcome-hidden-resolver/
#[[upstream.upstream_ietf]]
#    url = "https://dns4torpnlfs2ifuz2s2yf3fc7rdmsbhm6rw75euj35pac6ap25zgqad.onion/dns-query"
#    weight = 50

[others]
# Bootstrap DNS server to resolve the address of the upstream resolver
# If multiple servers are specified, a random one will be chosen each time.
# If empty, use the system DNS settings.
# If you want to preload IP addresses in /etc/hosts instead of using a
# bootstrap server, please make this list empty.
bootstrap = [

    # Google's resolver, bad ECS, good DNSSEC
    "<IP_ADDRESS>:53",
    "<IP_ADDRESS>:53",

    # CloudFlare's resolver, bad ECS, good DNSSEC
    #"<IP_ADDRESS>:53",
    #"<IP_ADDRESS>:53",

]

# The domain names here are directly passed to bootstrap servers listed above,
# allowing captive portal detection and systems without RTC to work.
# Only effective if at least one bootstrap server is configured.
passthrough = [
    "captive.apple.com",
    "connectivitycheck.gstatic.com",
    "detectportal.firefox.com",
    "msftconnecttest.com",
    "nmcheck.gnome.org",

    "pool.ntp.org",
    "time.apple.com",
    "time.asia.apple.com",
    "time.euro.apple.com",
    "time.nist.gov",
    "time.windows.com",
]

# Timeout for upstream request in seconds
timeout = 30

# Disable HTTP Cookies
#
# Cookies may be useful if your upstream resolver is protected by some
# anti-DDoS services to identify clients.
# Note that DNS Cookies (an DNS protocol extension to DNS) also has the ability
# to track uesrs and is not controlled by doh-client.
no_cookies = true

# Disable EDNS0-Client-Subnet (ECS)
#
# DNS-over-HTTPS supports EDNS0-Client-Subnet protocol, which submits part of
# the client's IP address (/24 for IPv4, /56 for IPv6 by default) to the
# upstream server. This is useful for GeoDNS and CDNs to work, and is exactly
# the same configuration as most public DNS servers.
no_ecs = false

# Disable IPv6 when querying upstream
#
# Only enable this if you really have trouble connecting.
# Doh-client uses both IPv4 and IPv6 by default and should not have problems
# with an IPv4-only environment.
# Note that DNS listening and bootstrapping is not controlled by this option.
no_ipv6 = false

# Enable logging
verbose = true

I think I know what happened, it should be rand.Intn(len(rs.upstreams)) instead of rand.Intn(len(rs.upstreams)-1)
by the way, the config you posted doesn't use random mode, when you found this bug, did you use weighted_round_robin mode? It should use another selector not random selector

PR is created

Merged PR #41
Please report if the problem is fixed.

I'm not the author but I was experiencing the exact same error using the random selector.
PR #41 fixed it for me. Can you retag the 2.0.0 release to the working version? Thanks!

@Sherlock-Holo I also got an error when using random selection. However, in the configuration, I forgot that I had already changed to another setting and just uploaded it.

PR #41 fixed it for me. Can you re-tag the 2.0.0 release to the working version, or make a new one?

Tagged 2.0.1


React to Auctionator API change

Thanks for the PR. Its been merged and put in the next pbs release (pending processing by Curseforge)


Add GET /about API support

Hi thanks for this project!
I'm trying to use the about.get API to load some information about the user, so I thought I'd go ahead and submit a PR to make that an option in the library.
I tried to follow all the existing conventions and added tests as well (although I did not add this function to the example app). I also verified all tests (including old tests) pass.

Hello @idolize! Thanks for your great contribution. There is only one small issue with unit tests: GetAboutTests.testGetAbout is failing because of a wrong assertion against the request URL. It's easy to fix:
file: Tests/GoogleDriveClientTests/GetAboutTests.swift
line: 61
- let url = URL(string: "https://www.googleapis.com/drive/v3/about")!
+ let url = URL(string: "https://www.googleapis.com/drive/v3/about?fields=\(About.apiFields)")!

I guess you just forgot to include the fields query parameter to the expected URL. With the above change, tests will pass. I'm happy to merge the PR and release a new version that includes your changes once the issue is fixed.
Thanks again for contributing!

Thanks, I made a change and forgot to update that part of the test.
Should be fixed now!

Also, I wasn't sure if you want to add this function to the README as well or not

Thanks! I will update the README once merged.


nix 2.4: include lowdown in both buildInputs and nativeBuildInputs.

Motivation for this change
Lowdown is both called at compile time and statically linked into the nix binary, so we need both when cross-compiling.
With this, nix-build . --arg system '"aarch64-linux"' --arg crossSystem '(import ./. {}).lib.systems.examples.raspberryPi' -A nixFlakes works and produces aarch32 binaries that seem to run fine (on the 64-bit build machine, at least - I haven't tried an actual 32-bit ARM processor).
Things done

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

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

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

How about adding strictDeps = true to prevent future regressions of this kind?

I'm not sure that would help. As I understand it, strictDeps makes sure nothing in nativeBuildInputs ends up in the closure; nix statically links to lowdown, so it wouldn't get caught by strictDeps.

It does more than that: if I add strictDeps = true to nixpkgs without your patch, nixUnstable fails to build:
src/nix/markdown.cc:7:10: fatal error: lowdown.h: No such file or directory
    7 | #include <lowdown.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [mk/patterns.mk:3: src/nix/markdown.o] Error 1
make: *** Waiting for unfinished jobs....
builder for '/nix/store/y3l65349zf332vh1mjbxijxldfhxqm2z-nix-2.4pre20201118_79aa7d9.drv' failed with exit code 2
error: build of '/nix/store/y3l65349zf332vh1mjbxijxldfhxqm2z-nix-2.4pre20201118_79aa7d9.drv' failed

Ah, neat! Added.

Closing in davor of #107909

Closing in davor of #107909


Correction for raw affiliation King's University College at Western University, London, Canada

Correction needed for raw affiliation King's University College at Western University, London, Canada
raw_affiliation_name: King's University College at Western University, London, Canada
new_rors: ;02grkyz14
previous_rors: 048f5b183;02grkyz14
works_examples: W4319031149;W4323854287;W4362725590
contact: 14235d63a2b92d7c700b9126c02ca3c6:15de79d9345a1531 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Android app keeps crashing

I use the NewsBlur Android app on a Google Pixel 4a, and it keeps crashing for me on certain feeds. This particular story is one example of an article that crashes. I got this story from the Planet Python RSS feed. My app is also configured to load the feed in "Story" mode.

It looks like it doesn't let me load the story in the Story mode. Do you see the same?
Also does it crash on story load usually for you? @jgbishop


UI - Navigate around the 3 ssapps and register missing actions. Step 2 - implement

for instance, in a product list item, when we click a batch, we should be sent to that batch's screen.
One missing screen, for instance is the sale screen for the pharmacy. Only the create part is implemented, the manage is not

phase 1: identify missing navigation 'routes' in the applications;
phase 2: discuss;
phase 3: implement


super low GC bias in many samples results in n/a iRep values

I have 120 MAGs across 25 samples. The MAGs are good quality: most are >90% complete with <5% contamination. But for most of these nice MAGs, there seems to be pretty much a logistic distribution for GC bias across the samples. The value is either roughly between 0.1 to 1 OR something like 2.2E-22. What is up with these super low GC bias values? What does it mean?
For any MAG, the super low GC bias for any sample results in failure to produce the corrected iRep values, even though all samples have the non-corrected "raw" indices of replication. Maybe I should just use those?
Much appreciate the help!

Hi Paul,
That sounds quite peculiar. Did you notice anything else that would suggest variation in sequencing quality across your samples? Given the extremes of GC bias, I would be skeptical of the uncorrected values.
Chris

On Feb 23, 2020, at 4:27 PM, Paul Wilburn<EMAIL_ADDRESS>wrote:

I have 120 MAGs across 25 samples. The MAGs are good quality: most are >90% complete with <5% contamination. But for most of these nice MAGs, there seems to be pretty much a logistic distribution for GC bias across the samples. The value is either roughly between 0.1 to 1 OR something like 2.2E-22. What is up with these super low GC bias values? What does it mean?
For any MAG, the super low GC bias samples for any sample results in failure to produce the corrected iRep values, even though all samples have the non-corrected "raw" indices of replication. Maybe I should just use those?
Much appreciate the help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Remove timestamp parsing

Remove the ability to specify the --exp and --nbf token values as
timestamps.  This behavior was deprecated in a previous release because
we want the server time to be the source of truth.  Clients now pass
timestamp values as a duration and the server does the math.

Aditya and I decided to keep timestamp parsing in the CLI.  It's convenient to be able to create a token that expires at a given date and time.


Decouple Acceptance Tests from mainnet.json

Most acceptance tests use mainnet.json, so bugs on adding a new fork (eg cancun) may not get covered until much later in the SDLC.
to avoid finding bugs at the last minute, change this to holesky/sepolia (or make it configurable?)

this one would be nice to implement.


Scroll Fix on Terms of Service Page

fix #257

@just-ary27 @fredfalcon This pr will solve the current scroll problem on terms and condition page right now if reached the bottom of page you can't scroll up


Fix iOS CHIPTool compilation

Problem
Fixes: https://github.com/project-chip/connectedhomeip/issues/15375
Change overview
Fix usage of readAttributeList API
Testing

If manually tested, what platforms controller and device platforms were manually tested, and how?
verified compile works

/rebase