Top nav

Utilize an array to store page names and corresponding routings dynamically.
Ensure the navigation component supports easy addition and removal of pages without hardcoding.
Utilize Material-UI (MUI) components for the design and styling of the navigation component.

I wasn't sure if I should create a new CSS style file, so I added the styles directly within the component.

Code looks good! Some minor comments:

In the file src/components/Nav.jsx -
Delete the empty line in the array (11)
Use the appropriate MUI component for the image and not the img tag, that way you won't need internal styling.
Changes to the package-lock file should be undone.
A link to the issue should be in the description and not in a separate comment.

good luck!

I made the changes.

The changes look perfect!
Pay attention to points 2 and 3 in the previous comment.

I saw this .
Thanks


🌱 Refactor test scripts

What this PR does / why we need it:
We have been seeing quite many flakiness in the unit test scripts.
This is a first set of patches that will be trying to fix that flakiness.

When running make generate, some of the autogenerated code ends up
in github/ dir, which should be outside of staging area. This patch
add github/ dir to .gitignore, so that we don't accidentally submit
a PR with those changes included.

Current install_kubebuilder.sh script downloads a tarball containing
other binaries apart from kubebuilder, such as etcd, kube-apiserver
and kubectl. We use kubebuilder binary only when running make generate
target, meaning there is no need to set up a test k8s env with those
binaries. Those are only needed to setup a fake k8s env while unit
testing, via make unit/test target.

Delete kubebuilder & kustomize installation when running go tests.
We currenrty install them on every single run of test/unit test
run, and in fact go test doesn't require them to perform tests.

Run go mod verify after tidy to ensure go.sum matches with
downloaded packages.

/retest

/cc @furkatgofurov7

/test-v1b1-integration
/cc @kashifest

/test-v1b1-integration

/zapataxuyem

@furkatgofurov7 @kashifest
Do we want this to be backported to release-0.5 branch ?

/cherry-pick release-0.5

/cherry-pick release-0.5


IE8 fallback icon - accordion

Description of the issue
While testing v4 on GOVUK noticed the fallback icon was missing
(not an issue on GOVUK support is IE11+)
Steps to reproduce the issue
Visit https://govuk-design-system-preview.netlify.app/ (In Browserstack > IE8)
Actual vs expected behaviour
https://github.com/alphagov/govuk-frontend/blob/main/package/govuk/components/accordion/_index.scss#L175

vs
Expected

Environment (where applicable)

Operating system: Windows
Browser: Internet Explorer
Browser version: 8
GOV.UK Frontend Version: 4

Low Priority

Having given this a quick looksee, this appears to be an issue specific to the Design System website's implementation of the accordion. The version in the Frontend review app displays as expected.
Specifically, the layout used for this example doesn't load the IE8-specific stylesheet (though it does load IE8-specific JavaScript).
To that effect, I'm going to alter this issue and move it over to the Design System repository.


Bluetooth: LE-Audio: Codec config data and meta data storage refactor

Is your enhancement proposal related to a problem? Please describe.
The current implementation of codec config and meta data is based on copying the LTV encoded data into an array of type bt_codec_data. This approach was appropriate at the time where all defined data types were 4 bytes or less in size as the overhead of having pre-allocated memory for all possible values were not huge.
But as more and more metadata types and potentially vendor specific codec config values are added and needs to be processed this approach is no longer suitable. One example is the metadata type 0x03 (program_info) this is a UTF-8 string - hence even for a small supported size of say 20 bytes the overhead of re-configuring the bt_codec_data type to be able to hold this amount of data is huge as all entries have the same size - hence you would also use a 20-byte entry to store a single byte value.
Describe the solution you'd like
I'll suggest to keep the LTV encoded data in a flat memory buffer and use utility functions to extract data as needed (you would need this anyway if using bt_codec_data[]).
One solution could be to treat CC-data and meta-data as adv-data is handled today. The data is provided to the application while still in the net_buf in which it was received and lives only while processing the call-backs after which it is deleted from the stack.
A link from the stream to the codec config and metadata could be done in several ways:

the application could have sole control of it - and the stack can never access it. (this could complicate future integration into an audio-subsystem.
The application could save a copy of relevant TLV encoded CC- and meta-data and the stack could have an api to register this data to a stream (like bt_audio_capability_register() does for PAC records). I would prefer a storage type that do not force the application to store the data in RAM as is the case for PAC records (struct bt_audio_capability) due to the presence of sys_snode_t node; in the data type. This entry is changed by the stack - to link multiple entries - hence prevents placing the PAC records in read only memory like flash)

Describe alternatives you've considered
I considered having a flat shared block of memory in the stream structure and use that for CC- and meta-data directly. That way you would not be limited by a size limit for the largest element, but rather the total amount of data - hence if you have many small entries and one large memory usage would be much less. But I'm not sure there is a good MAX_SIZE for this - it could of cause be user-configurable as it is today, but it would prevent an application to use dynamic memory allocation - which allows to share memory between multiple streams that might have different amount of metadata - for example the larger program_info Utf-8 string only applies to broadcast streams - hence no need to allocate memory for it for all unicast streams.

Is this issue still being looked at? The codec configuration is not compliant with the BAP spec. Subgroup codec configs (Level 2) need to be inherited into BIS-specific codec configs (Level 3). I agree that it is probably necessary and best to use the flat LTV array to encode information, but the utility functions, as well as the BASE parsing logic for codec info will need to verify consistency and compliance with the BAP spec, wrt codec config inheritance.
An alternate approach would be to duplicate a codec struct across each BIS and "externally" ensure consistency between all BISs within a subgroup.

Is this issue still being looked at? The codec configuration is not compliant with the BAP spec. Subgroup codec configs (Level 2) need to be inherited into BIS-specific codec configs (Level 3). I agree that it is probably necessary and best to use the flat LTV array to encode information, but the utility functions, as well as the BASE parsing logic for codec info will need to verify consistency and compliance with the BAP spec, wrt codec config inheritance.
An alternate approach would be to duplicate a codec struct across each BIS and "externally" ensure consistency between all BISs within a subgroup.

Hi @bssc2g, I don't think this issue is being looked at by anyone. W.r.t. your comment about subgroups and the codec configurations for the broadcast sink and source, we have the following PR awaiting review: https://github.com/zephyrproject-rtos/zephyr/pull/49946

Thanks for the update @Thalley, glad to see it's in progress! Great work on the existing framework, btw. This was only an issue when we tested against a non-Zephyr broadcaster that was putting additional channel config info in the BIS codec config.

As part of this refactoring, we should also consider splitting codec configuration and codec capabilities into 2 separate structs, as the current design is confusing for users.


[WIP] Add: dunst service

Service defintion for dunst.
Somehow I cannot build my system with this (nixos-rebuild build -I nixpkgs=... gives me error: The option services.dunst' defined in /nix/store/.../dunst.nix' does not exist).
Maybe someone can investigate?
Tests will follow...

Tests added. They do not really test the dunst service by now, but as you can see the dunstrc is not found. I do not know why.

I've tried using this on top of the current nixos-unstable channel: https://github.com/bendlas/nixpkgs/tree/dunst-service-fail
I get failures in documentation generation
building path(s) ‘/nix/store/zlq97dfabijy4ms5k27wrdacy5gcbag1-nixos-manual’
options-db.xml:7978: namespace error : Failed to parse QName 'http:'
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : error parsing attribute name
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : attributes construct error
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : Couldn't find end of Start Tag http: line 7978
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: namespace error : Failed to parse QName 'http:'
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : error parsing attribute name
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : attributes construct error
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
options-db.xml:7978: parser error : Couldn't find end of Start Tag http: line 7978
<http://developer.gnome.org/pango/stable/PangoMarkupFormat.html>.
      ^
man-configuration.xml:34: element include: XInclude error : could not load options-db.xml, and no fallback was found
./man-pages.xml:8: element refmiscinfo: Relax-NG validity error : Invalid sequence in interleave
./man-pages.xml:34: element include: Relax-NG validity error : Element refsection has extra content: include
./man-pages.xml:29: element refsection: Relax-NG validity error : Element refentry has extra content: refsection
./man-pages.xml:3: element reference: Relax-NG validity error : Element reference failed to validate content
./man-pages.xml fails to validate
manual.xml:39: element include: XInclude error : could not load options-db.xml, and no fallback was found
builder for ‘/nix/store/wnl9sr61cvzayjvs4rdzlj315rk3lcaa-nixos-manpages.drv’ failed with exit code 3
manual.xml:39: element include: Relax-NG validity error : Did not expect element include there
manual.xml:39: element include: Relax-NG validity error : Expecting element example, got include
manual.xml:39: element include: Relax-NG validity error : Expecting element bridgehead, got include
manual.xml:39: element include: Relax-NG validity error : Expecting element annotation, got include
manual.xml:39: element include: Relax-NG validity error : Element appendix failed to validate content
manual.xml fails to validate
builder for ‘/nix/store/pj7n5zzhfnkm4sqwqn9rkmcx40hl0h15-nixos-manual.drv’ failed with exit code 3

@bendlas : Afaik, this is not yet merged, right? Anyhow, how is the trace you provided related to what I've done in this PR? And how to reproduce?
I will work on this asap, though atm I have not that much time.

@matthiasbeyer I included your commits in my system, because a) I use dunst and b) I wanted to help you test. When I saw those failures, I bisected them back to your commits. For your convenience (and reference), I linked the current unstable channel branch + your 2 commits in my comment. The current unstable builds fine.
I saw the failures by trying to nixos-rebuild with those commits applied, but I assume that you could reproduce by just building the nixos-manual with nix-build (I don't know the command for that offhand)

You can build just the manual from the nixpkgs source root with: nix-build nixos -A config.system.build.manual.manual
The failure is reproducable by this

What's the status of this PR ?

@zimbatm I will have a closer look on what's missing tomorrow. I will then either close this PR unmerged or push it to be ready, though I cannot promise anything here!

Wah, took me a bit longer than 1 days, I'm sorry. I have no longer any interest in this, feel free to reuse commits.

I'm interested again. I rebased to latest master and will now start working on this, but based on unstable, so I'm able to test it in a VM.

So here we go.
The failure @bendlas reported was indeed caused by this PR, it was the descriptions where the markup was wrong. I simply removed the descriptions (they should be added later, as soon as this is working).

What's working by now: dunst gets started by systemd. Anyways, systemctl status dunst says that the service is dead (killed by SIGSEGV) but notify-send hello still works.
dunst does not find the configuration file written via the config, though.
For people testing this: Put this in your configuration.nix:
  services.dunst = {
    enable = true;

    global = {
      transparency = 50;
      padding = 18;
      horizontalPadding = 18;
    };
    frame = {
      width = 8;
      color = ''#FF0000'';
    };
  };

Dunst should then put the notifications on the right side of the screen and they should be styled rather ugly. The default config shows the notifications on the left side in light blue (when sending notify-send hello for example).

Here is help wanted ... I don't know why dunst does not find the configuration file and I really do not know why systemd reports it crashed when it is actually running.

Is dunst forking itself by any chance? That would explain why maybe the parent would segfault but the child is still active.

@zimbatm Dunst seems to be forking, yes. But I also set the Type = "forking" for the systemd options...

If you can avoid the fork it generally is a good idea. Maybe there is a command-line option for that.

nope, there is not, sadly.

Weird, I just ran dunst from the command-line and it's running as foreground process.

Yeah, it is really weird ... maybe you (or someone else who knows how services on nixos work) can find the problem... but I'm at my wits end.

Is this still being worked on?

@joachifm I'm still waiting for review here and suggestions what could fix the problem.

Okay, but this is a lot of code that could be replaced simply by adding dunst & to your xinitrc ...

Looks like upstream has a pretty strange systemd service which is the basis for this AUR package. Could that hold the key to dunst not dying?

@siddharthist that is not a systemd service file. I'm dubious that the aur package actually does anything helpful. See https://dbus.freedesktop.org/doc/dbus-specification.html#message-bus-starting-services for details.

That said, using a dbus service instead of a systemd service might make sense anyhow.

@joachifm I've tried a DBus service type, and that gets it to not crash! See #19183. Will add more documentation on the fields tomorrow and hopefully get it mergeable.

I would favor the new pull request.


[BUG] Set-PnPUserProfileProperty : Access denied when set property for user

Account running script has global admin rights. It's possible to update the property for current user executing the script, but access denied when try to update a property for a different account.
Set-PnPUserProfileProperty -Account $accountName -PropertyName "SPS-Birthday" -Value "2021-01-01"
Set-PnPUserProfileProperty : Access denied. You do not have permission to perform this action or access this resource.
At \Mac\Home\Desktop\Reset-UserProfiles.ps1:25 char:13

        Set-PnPUserProfileProperty -Account $p.AccountName -Prope ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : WriteError: (:) [Set-PnPUserProfileProperty], ServerUnauthorizedAccessException
FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.UserProfiles.SetUserProfileProperty

PnP Management Shell application has been granted correct permissions in Azure AD
What is the version of the Cmdlet module you are running?
V1.5.0
Which operating system/environment are you running PnP PowerShell on?

[ X] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

@matsla - To give you an update, this is not an issue with PnP PowerShell but has to do with how SharePoint Online handles authorization. We provided Microsoft with this feedback and are waiting for them to reply on it. We have no ETA when that will happen.
Note: Also related to issue #277.

Closing this as it is a duplicate and we are tracking it in #277


fix typo

Thanks @alex-strizhakov! This has been merged into main: https://github.com/elixir-toniq/regulator/commit/064c8412918575965ed53526ef8f1fe60e36d4ae.


Adding New Text - Cursor Focus

Bug Description
Can add text, but expected cursor focus to be in text field
Expected Behaviour
expected that cursor focus would be within the field (with any assistive text within the field highlighted - expected that user can just go ahead and type without haveing to highlight/remove text
Steps to Reproduce
Opt to add new text box
Screenshots

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

@csossi I don't quite follow what the issue is.
is it related to this? https://app.zenhub.com/workspaces/web-stories-5e94d3aced449034e1e9b226/issues/google/web-stories-wp/830

Related: #1608

I will close this in favor of #1608


How to migrate from sqlite3 to mysql？

Gitea version (or commit ref):
1.12.5
Git version:
Operating system:

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description

I want to upgrade from 1.12.5 to 1.13.x, but after updating the binary file, the CPU occupies 100% and cannot be used. I want to migrate to MySQL, but the database table fields seem to be different
Screenshots

Manually or jou can use the rest api with own script.

https://gitlab.com/markuman/gitea-mariadb-migration/

https://github.com/go-gitea/gitea/issues/12909


Data loss in the inter-device communication

Hi, I am using two devices (x86 and Nvidia Jetson xaiver AGX) using the same CYBER_IP to communicate between two devices.
To reproduce the issue, I am sending from x86 device is lidar point cloud which is 1.38Mb per frame and 10Hz (so 13.8Mb/s) to another device AGX via Cyber. The ethernet interface that connects two devices supports up to 1000 megabits and confirmed sending/receiving data on the connection side isn't a problem. However, on the AGX side, the frequency of the lidar point cloud channel drops to 5~8Hz not 10 Hz in the cyber_monitor (the frequency of lidar channel on the x86 side in cyber_monitor is full 10Hz though). I tried to change the reader/writer config using qos_profile() in apollo::cyber::proto::RoleAttributes, but it didn't solve the issue.
Is there any other way I can configure the cyber transport to solve this issue?
Thank you!

Hi, Is there any follow-up on this issue


Incomplete git installation packaged with cmder v1.2

Git is only partially functional from a clean installation of Cmder v1.2. Testing from a {cmd} shell, most git commands appear to work fine but git pull fails.
λ git --version
git version 1.9.5.msysgit.1

λ set git
GIT_INSTALL_ROOT=C:\opt\cmder\vendor\msysgit

λ where git
C:\opt\cmder\vendor\msysgit\bin\git.exe

λ git pull
git: 'pull' is not a git command. See 'git --help'.

After a bit of experimentation (see comments on #419), my guess is that some of the files that were recently removed (004761b32f8aa47fad3929babf53e3f8972f33eb) from vendor\msysgit\libexec\git-core are still required.

:+1: Can confirm this issue. Made new version of cmder useless.

:+1: Yeah, doing git fetch and then git merge origin/BRANCH is a pain.

git svn is another command that doesn't work.

Well this explains a lot of issues I was getting when I tried 1.2! I thought it was just something I had done wrong trying to remove my main install of msysgit so that only the one version bundled with cmder would be used, but I guess not

I had the same problem, but 29d1378 seemed to fix it for me :)

Adding back all the git-* files to vendor\msysgit\libexec\git-core restores git pull etc. So it seems like that is indeed it.

https://github.com/bliker/cmder/commit/004761b32f8aa47fad3929babf53e3f8972f33eb was a silly change in general, why remove random exe files when you don't even know what they do? :-1:

Temporary workaround for #503 and #521 is to download PortableGit-1.9.5-preview20150319.7z (or newer when available) from https://github.com/msysgit/msysgit/releases and extract it to vendor\msysgit.
cd cmder\vendor\msysgit
curl -k -L https://github.com/msysgit/msysgit/releases/download/Git-1.9.5-preview20150319/PortableGit-1.9.5-preview20150319.7z -o git.7z
7z x -aos git.7z

@iki Thanks, that worked!

@iki Yeah, it works, thank you

@iki: Still works, thanks :+1:

@iki Just ran into this issue, thanks for the clear solution! :smile:


Show breakpoints and logpoints in minimap

Currently, the minimap shows certain decorations in addition to the file's text, such as the current line (blue) and lines with errors (red). It would also be nice to dedicate a few pixels on the left of the minimap to indicate breakpoints, with different colors and/or shapes to distinguish between unconditional breakpoints, conditional breakpoints, and log points.

@rben01 in case you haven't already discovered this, it is already possible to show them in the Overview Ruler
"debug.showBreakpointsInOverviewRuler": true


Feature/theme change transition

PR Fixes:

Added theme change transition
https://github.com/user-attachments/assets/4074ebb1-b1f9-42a7-8312-abdf59d1312a

2 Enhanced the no bookmark page

Checklist before requesting a review

[x ] I have performed a self-review of my code
[x ] I assure there is no similar/duplicate pull request regarding same issue

@devsargam , i shortened my previous pr

Thank u


JLC PCB fabrication instructions no longer work

I followed the steps described here with version 2.0.4:
but after specifying the bom and pick&place file, I get the following error:
File processing failed, please check your file and upload again.

I also tried with 2.0 is there an update to the procedure to remain compatible with JLC PCB?

Hmm, they mush have changed the bom format.
There is probably a different order/naming of columns in the BOM files. The pick and place document should be fine probably.
I'll try to look into it in the next days.

Hi @eduardkieser,
Thanks for reporting this!
I've just checked and JLCPCB has changed a lot of their requirements for the fabrication files since I've ordered last shields. :D
The new BOM and CPL files are in available in the repo.
You might have to hurry up because there are just 24 L6234 available in the jlcpcb parts as of today (22/06/2024) :D

@askuric Thanks so much!
Will take a look as soon as I get a moment.

Tested last night. Works beautifully, thanks! closing.


⚠ Bofrost.lu has degraded performance

In f88698c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1635 ms

Resolved: Bofrost.lu performance has improved in d232133 after 10 minutes.


@Wither and inherited properties

Consider the following scenario:
@Getter
@RequiredArgsConstructor
public class MyBaseClass {
    private String baseProperty;
}

@Getter
public class MySuperClass extends MyBaseClass {

    public MySuperClass(@NonNull String baseProperty, @NonNull String superProperty) {
        super(baseProperty);
        this.superProperty = superProperty;
    }

    @Wither
    private String superProperty;
}

Expected: A withSuperProperty(String s) is generated that creates a new object with the same inherited baseProperty (by using the provided constructor).
Actual: The code does not compile, since no appropriate constructor with only the parameter java.lang.String is found.

I am not sure whether this it's right to start resolving this issue at @Wither - @XArgsConstructor for example does not support these inherited properties as well. Maybe, if the issue is first resolved there, it may be easier for @Wither? I haven't had a look at the codebase, so this is only a thought. If it's nonsense, just ignore it, please.

You are right, this is impossible at the moment. This is restriction of lombok, like here
Wither looks only for a constructor with parameters for all fields of the current class. It doesn't look for fields/constructors from the inherited classes.
Maybe you ask lombok authors (@rzwitserloot, @rspilker ) if they could make it possible. At the moment intellij-lombok-plugin can't do anything.

Oh wow. I actually wanted to post this on the lombok tracker and not here and never noticed. Thank you for explicitly pointing that out, I would not have noticed that otherwise. Wow.


Group security related pages under Security guide, add new docs page Authentication enforced by WS-SecurityPolicy

Thanks @rhaetor, I included the most of your feedback in 82ae5a1


Godeps: update libopenflow to latest version

Signed-off-by: Bill Robinson<EMAIL_ADDRESS>

build pr

build pr

@dseevr: Can you bump ofnet as well, please?

@unclejack I did a make godep-save and, while it pulled in the latest HEAD on ofnet (after merging your PR), there were no file changes except the Jenkinsfile stuff that @nbartos added.  Nested vendor directories get flattened out I guess, so updating libOpenflow to the latest HEAD is sufficient.


typescript support

Any plans on converting to typescript? ill be willing to contribute.
Also can setup husky  for #20 which can run test and linters before commit (pre-commit hooks)
sample of the template is with a boilerplate i made node-app-bp which has husky and eslint configured and is using esbuild to compile
-> esbuild performance <-

Hi @princejoogie, I have plans in converting it to typescript which is better for maintaining the project. If you can do it feel free to submit a PR to the dev branch referencing this issue.

Also, what do you think @jofftiquez regarding this?

This is a great suggestion. I was also thinking of suggesting esbuild, but I can't recall the name yesterday xD. But yeah, esbuild is better.

Let's implement esbuild, the same thing, I forgot the name of it lol.

Closing this now, thanks to @princejoogie for the contribution!


Few more fixes and improvements to data we are collecting

I'm not sure about this __version__ thing. What do you think @pmacik ?
Intention is to be able to change that field in scenario file whenever scenario significantly changes (e.g. new task is added) so performance of that scenario is expected to change.
It is slightly duplicated by a last git commit, but benefit of having it would be this version only changes when there is a significant change in scenario itself, not when somebody updates a readme or so.

@jhutar I like the idea of versioning scenarios

/lgtm
/approve


Add configuration to upload release candidates to PyPI

Add configuration to upload release candidates to PyPI.

Codecov Report

Merging #108 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #108   +/-   ##
======================================
  Coverage    78.1%   78.1%
======================================
  Files           5       5
  Lines         676     676
======================================
  Hits          528     528
  Misses        148     148

Impacted Files
Coverage Δ

mlblocks/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix  $search operator to actually work

Summary
I noticed in looking at the code that the $search operand appeared suspicious.  So I set up a test case and sure enough, it didn't appear to work.  So I went in and fixed the code and added a couple of test cases.

I think you were looking at an old version. parse.js (https://github.com/feathersjs/feathers-rethinkdb/blob/master/src/parse.js) now looks quite different and $search is being tests in https://github.com/feathersjs/feathers-rethinkdb/blob/master/test/index.test.js#L154.

Closing since it has been implemented already. Still much appreciated though 😄


24 Hour capture

What would I need to change to make the allsky program do 24hour captures and then convert them to the movie instead of just night time.

You can try to the the angle value in the portal to 90. That means start/stop the capture when the sun is less than 90 degrees above the horizon (i.e. all the time).
This should take 24hr of images. The cut off between days happens at noon.

I will try this setting and see how it works for me ... I have cloudy and city-lit nights so some daytime recordings would liven things up a bit! Cheers!

Yes I can confirm that the program does not run when angle value is set to 90.  I think my top two desired features are (a) Capture pictures during the day and (b) analyse for meteors. Having said this, the software is very very good already.

Mind you, set the angle to 89 and it seems to cheerfully run! I will see what the output is like over the next few days

Please send us a link on a full day when it finishes please.
On Mon, May 13, 2019, 9:58 PM ZebrasHaveBigEars<EMAIL_ADDRESS>wrote:

Mind you, set the angle to 89 and it seems to cheerfully run! I will see
what the output is like over the next few days
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thomasjacquin/allsky/issues/48?email_source=notifications&email_token=AK2MJOQXGOMG75ZNZHAA2ODPVJBGXA5CNFSM4GKKKTZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVKIONQ#issuecomment-492078902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2MJOWM6HVBM6ZLEFSXNQTPVJBGXANCNFSM4GKKKTZQ
.

Well, it started but something went wrong and now something is wrong when I set the angle back to -6.  I hope I will have some over the next day or so to investigate.

When ever I tried to get all day shots it would never record the day images
or combine them in time.  I think it is something in the logic and it also
takes a while to convert those images into startrails and MP4.  I would go
into the images folder and select delete so nothing is in there.. reboot
the device and watch the CPU %   mine would always Max out and then lock up
when I tried anything close to the 90-85 angle..  I eventually gave up on
an all day-night video.
On Tue, May 14, 2019, 3:43 PM ZebrasHaveBigEars<EMAIL_ADDRESS>wrote:

Well, it started but something went wrong and now something is wrong when
I set the angle back to -6. I hope I will have some over the next day or so
to investigate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thomasjacquin/allsky/issues/48?email_source=notifications&email_token=AK2MJOUKCZHR3BTKUGVKQULPVM6ADA5CNFSM4GKKKTZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVNAC5Q#issuecomment-492437878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2MJOUVQFWO43B7NP7SDH3PVM6ADANCNFSM4GKKKTZQ
.

You're right, I need to add a bit of logic in the capture.cpp file. but feel free to experiment by adding this line in saveImageDay.sh
# Save image in images/current directory
cp $IMAGE_TO_USE "images/$CURRENT/$FILENAME-$(date +'%Y%m%d%H%M%S').$EXTENSION"

This is what you need in order to save the current image to disk. However, the timelapse still happens at dawn (or whatever angle you set the sun at). Ideally, for 24hr capture, we should start the timelapse at noon to have the night centred in the middle of the video.

Thanks for this code which nearly worked but it certainly showed the way to do it. I got it working by being inspired from the code in saveImageNight.sh:

Adding the commands to make the directory $CURRENT first where $CURRENT is the current day  as as opposed to the day before
Using %Y%m%d   as opposed to  %Y%m%d%H%M%S

and note

It is easy to check if it is all happening right by determining if the images go into the new folder under images
Making the movie of the whole day and the ftp of the whole day takes quite a lot of time as the new movie is quite large!

I am happy to share the code if needed.

Please share the code if you wouldn't mind
On Mon, May 20, 2019 at 2:23 PM ZebrasHaveBigEars<EMAIL_ADDRESS>wrote:

Thanks for this code which nearly worked but it certainly showed the way
to do it. I got it working by being inspired from the code in
saveImageNight.sh:

Adding the commands to make the directory $CURRENT first where
$CURRENT is the current day as as opposed to the day before
Using %Y%m%d as opposed to %Y%m%d%H%M%S

and note

It is easy to check if it is all happening right by determining if
the images go into the new folder under images
Making the movie of the whole day and the ftp of the whole day
takes quite a lot of time as the new movie is quite large!

I am happy to share the code if needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thomasjacquin/allsky/issues/48?email_source=notifications&email_token=AK2MJOT7YE4YSM4ILCOJDJLPWMJF7A5CNFSM4GKKKTZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV2DS2A#issuecomment-494156136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2MJOUNOS3JJ6GYIRQVMQTPWMJF7ANCNFSM4GKKKTZQ
.

yes sure .. please give me 24 hours as I took it outside for the first time last night ... I got the day and night video this morning! Thanks to Thomas!

Where do we post photos of our ALLSKY camera build ?  I designed and 3d printed mine but its still a work in progress while I figure out cooling and air circulation.    Bill in southern california.          PS: Thanks for the wonderful software.

I left photos of mine here:
https://www.instructables.com/id/Wireless-All-Sky-Camera/

Hi there
To save images during the day and night, here is the code which goes in saveImageDay.sh and which has worked over a 24 hour period for me.  It creates the directory under Images which will later be used in saveImageNight.sh and then copies the daylight images into it. Please remember to save a copy of the original version. Please be aware it takes a lot of time when the images are compiled into the movie and ftp is used to transfer the files to the website.  Keograms and startrails should also work.
Cheers
ZHBE
#########################################
ADD THE CODE BELOW THIS LINE IN saveImageDay.sh
##cp $IMAGE_TO_USE "liveview-$FILENAME.$EXTENSION"
################## ADD THESE EXTRA LINES#############
CURRENT=$(date +'%Y%m%d')
mkdir -p images/$CURRENT
Save image in images/current directory
cp $IMAGE_TO_USE "images/$CURRENT/$FILENAME-$(date +'%Y%m%d%H%M%S').$EXTENSION"
##########################################

Did you get it working? It is working a treat on my system.

Haven't gotten to put the code in yet.. Am really busy with some other
things, but I will let everyone know how it comes out once I get the first
one online.
spacemanbob.com is my site.
On Sat, May 25, 2019 at 10:14 PM ZebrasHaveBigEars<EMAIL_ADDRESS>wrote:

Did you get it working? It is working a treat on my system.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thomasjacquin/allsky/issues/48?email_source=notifications&email_token=AK2MJOW7U4TSHLFJ2YNKQR3PXIMEDA5CNFSM4GKKKTZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWH6OKA#issuecomment-495970088,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2MJOSVUTPQW3FVAY5NHXTPXIMEDANCNFSM4GKKKTZQ
.

I added the code above to the saveImageDay.sh. Do I also need to set the angle camera setting to 90 degrees?

I tried this but it didn't work properly.
It saved individual images for the daytime but didn't create a video.
It also stopped recording at night time.

This is brillianyt and just what I wanted. ZHBE I've just added the code to my saveImageDay.sh and restarted Allsky and it's saving images to the newly created /images/26062020 folder straight away. Is there anything else I need to do or will the time-lapse be created without further intervention please?

@ZebrasHaveBigEars I tried this and it saves the daytime images. Thanks!
@minnican It does not create a timelapse for daytime.

@michaeloconnell78 If you leave the camera running 24-hours it creates the timelapse automatically when the sun is rising at -6 degrees below the horizon - in my case that's 03:45 hours at present. Otherwise you can create the timelapse manually by running the timelapse.sh in the scripts folder if I recall.

@minnican
At present, it only creates a timelapse of the night videos.
I'll try running that script.
Thanks.

Silly Question time.
Where do I add the code mentioned .
At the start, at the end, after a certain line?
Or is it a case of deleting the  Original file info and adding the code.
IF it cant be broken I will find a way to break it by accident.

Here's the code in my saveImageDay.sh:
#!/bin/bash
source /home/pi/allsky/config.sh
source /home/pi/allsky/scripts/filename.sh
cd /home/pi/allsky
IMAGE_TO_USE="$FULL_FILENAME"
cp $IMAGE_TO_USE "liveview-$FILENAME.$EXTENSION"
If upload is true, create a smaller version of the image and upload it
if [ "$UPLOAD_IMG" = true ] ; then
echo -e "Resizing"
echo -e "Resizing $FULL_FILENAME \n" >> log.txt
# Create a thumbnail for live view
# Here's what I use with my ASI224MC
convert "$IMAGE_TO_USE" -resize 962x720 -gravity East -chop 2x0 "$FILENAME-resize.$EXTENSION";
# Here's what I use with my ASI185MC (larger sensor so I crop the black around the image)
#convert "$IMAGE_TO_USE" -resize 962x720 -gravity Center -crop 680x720+40+0 +repage "$FILENAME-resize.$EXTENSION";

##cp $IMAGE_TO_USE "liveview-$FILENAME.$EXTENSION"
################## ADD THESE EXTRA LINES#############
CURRENT=$(date +'%Y%m%d')
mkdir -p images/$CURRENT
cp $IMAGE_TO_USE "images/$CURRENT/$FILENAME-$(date +'%Y%m%d%H%M%S').$EXTENSION"
##########################################
echo -e "Uploading\n"
echo -e "Uploading $FILENAME-resize.$EXTENSION \n" >> log.txt
lftp "$PROTOCOL"://"$USER":"$PASSWORD"@"$HOST":"$IMGDIR" -e "set net:max-retries 1; set net:timeout 20; put $FILENAME-resize.$EXTENSION; bye" &
fi
I hope it helps a little. I should just say I have no idea why "# If upload is true, create a smaller version of the image and upload it" that is appearing as a much larger , bold font in my reply. It's just normal text.

Thanks for the code, Ive noticed that on the pi the Image.jpg is updating and not the Live Image or Resize Image JPG which is what the Index page of the website uses and more importantly what the GUI uses as the live view.  I assume I just need to change a line from live-image.jpg to Image.jpg, but in which file!
Thanks

Further to the my last message.
I have changed the code to the 24 hour capture  / timelapse, but it only displays a live image on the GUI at the time of night time capture and doest seem to save the daytime images anywhere but the file image.jpg on the pi updates. Thanks Ian

I'm afraid I don't use the GUI so it's difficult to say. I have 3 different images called "image.jpg", "image-resize.jpg" & "liveview-image.jpg" all being created in the "/allsky" folder. In addition to this I have images with names formatted like this "image-20200707035328.jpg" saved constantly in /allsky/images/20200707, the folder name representing the date of course. In order to see the live image without the GUI I have to click on the "liveview-image.jpg" or easier still just navigate to my website at http://www.minnican.com I admit that I have on more than one occasion, completely gone back to the beginning of the programming process to ensure that any tiny error is eradicated. I even re-flashed the SD card at one point. Worth it in the end I guess.

I'm afraid I don't use the GUI so it's difficult to say. I have 3 different images called "image.jpg", "image-resize.jpg" & "liveview-image.jpg" all being created in the "/allsky" folder. In addition to this I have images with names formatted like this "image-20200707035328.jpg" saved constantly in "/allsky/images/20200707", the folder name representing the date of course. In order to see the live image without the GUI I have to click on the "liveview-image.jpg" or easier still just navigate to my website at http://www.minnican.com I admit that I have on more than one occasion, completely gone back to the beginning of the programming process to ensure that any tiny error is eradicated. I even re-flashed the SD card at one point. Worth it in the end I guess.

Thats whay=t should be happening, but the image-resize isnt  updating or the date.kpg isnt being created for day time, Its like the daytime capture switch has been turned ff

I'm afraid I don't use the GUI so it's difficult to say. I have 3 different images called "image.jpg", "image-resize.jpg" & "liveview-image.jpg" all being created in the "/allsky" folder. In addition to this I have images with names formatted like this "image-20200707035328.jpg" saved constantly in "/allsky/images/20200707", the folder name representing the date of course. In order to see the live image without the GUI I have to click on the "liveview-image.jpg" or easier still just navigate to my website at http://www.minnican.com I admit that I have on more than one occasion, completely gone back to the beginning of the programming process to ensure that any tiny error is eradicated. I even re-flashed the SD card at one point. Worth it in the end I guess.

Thats whay=t should be happening, but the image-resize isnt  updating or the date.kpg isnt being created for day time, Its like the daytime capture switch has been turned ff

At this point, because my programming knowledge is very poor, I'd go back to the beginning. I really didn't do anything different to what's in the this thread.

Hi all, instead of a 24h video would it be possible to have one video for daylight and one for night?

This issue is from an older release of the AllSky software and the original issue appear to have been resolved.  If you think it might still be relevant, please test it with the newest version and submit a new issue if needed.  Thanks.


error: module 'Alamofire' was not compiled for testing on carthage update

If you carthage update on the latest version of the swift-2.0 branch you will get error included:
Workaround: Release "Enable Testability" set to Yes, and carthage build
Error
*** xcodebuild output can be found in /var/folders/8y/3fnvzblx5m35h3mz7889s9vm0000gn/T/carthage-xcodebuild.aRoNfJ.log
*** Building scheme "Alamofire iOS" in Alamofire.xcworkspace
** BUILD FAILED **
The following build commands failed:
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/MultipartFormDataTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ServerTrustPolicyTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/CacheTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/URLProtocolTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/TLSEvaluationTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResponseSerializationTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/RequestTests.swift
CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(9 failures)
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
/Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift:23:18: error: module 'Alamofire' was not compiled for testing
A shell task failed with exit code 65:
** BUILD FAILED **
The following build commands failed:
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/MultipartFormDataTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ServerTrustPolicyTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/CacheTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResultTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/URLProtocolTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/TLSEvaluationTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/ResponseSerializationTests.swift
CompileSwift normal x86_64 /Users/ceco/src/Flowthings/Carthage/Checkouts/Alamofire/Tests/RequestTests.swift
CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(9 failures)

Thank you for reporting @appsmonkey. I just pushed 61d6ddf9 that addresses this issue. Here's the output from just running against the following Cartfile.
Cartfile
github "Alamofire/Alamofire" "fix/enabling_testability"

Carthage Output
cnoon:~/Desktop/CarthageChecker$ xcodebuild -version
Xcode 7.0
Build version 7A176x
cnoon:~/Desktop/CarthageChecker$ carthage version
0.7.5
cnoon:~/Desktop/CarthageChecker$ carthage update
*** Fetching Alamofire
*** Checking out Alamofire at "61d6ddf9c91da83a062cbe29dc6b7f6a0a5d8789"
*** xcodebuild output can be found in /var/folders/s7/qpm0b2pn3d56rjlpl7g3_4fh0000gn/T/carthage-xcodebuild.7T1Gz9.log
*** Building scheme "Alamofire iOS" in Alamofire.xcworkspace
*** Building scheme "Alamofire OSX" in Alamofire.xcworkspace
*** Building scheme "Alamofire watchOS" in Alamofire.xcworkspace
cnoon:~/Desktop/CarthageChecker$

Thanks again for reporting! I'm going to close this issue out. If you still have any issues, please comment on this issue and I'll gladly re-open it.
Cheers 🍻


Admin: More capabilities

[ ] Rename robots
[x] Delete robots
[x] Make users 'robot admins' (Issue #42)

Added deleting
https://github.com/mayacakmak/emarsoftware/commit/8132e7b8a337354e12e403fd91e14755f929a790

[ ] Decide on a format to name the robot versions
[ ] Clear and clean the different robot versions, i.e., keep only the versions that are working

[ ] Decide on a format to name the robot versions
[ ] Clear and clean the different robot versions, i.e., keep only the versions that are working


Issues when installing some extensions.

Good afternoon guys, I just started testing Bazaar and must admit I'm more than amazed with your work. Although, I have reached an issue. I successfully installed a few extensions but somehow there are some that it won't install.
Here you have the list of the extensions that I couldn't install:

BBCode Helper
Chat
Birthdays
Flagrow Split
Widgets

I am running Flarum on a dedicated server with CentOS 7 and just did a composer update to double check and reconfirm my issue.
Hope you can help me! Blessings!

Flagrow split isn't in active development (yet). The version by Davis should be stable though.
Installing these extensions using composer require did work?

Ohh, Split from David did work, although the rest remain the same.
Would you be so kind to show me how or where could I get the composer require for those ones?

Just installed the following successfully

Realtime Chat
Gravatar
FancyPants

Tried the following without luck:

Chat
Widgets
And the ones in my first post.

I have also updated Baazar to the last version you posted on Flarum, but still the rest is not working. What could be the possible reason?
By the way, I still haven't found the composer commands to try :(

I think all the ones you listed are untagged (dev-master version only).

I'm about to drop all dev stability packages from our listings.

https://discuss.flarum.org/d/5151-flagrow-bazaar-the-extension-marketplace/25

@AngellitoX can you confirm this now works in Bazaar? The unstable packages should no longer show up.

Confirmed! Sorry for delaying, just woke up. Thank you so much @luceos. Have a wonderful day!


Django 1.7 support w/auth.User Migration fix

This addresses an issue that got created when an auto migration was performed with South when auth.User is in use. By default South will hardcode auth.User references when they should be dynamic. This pull request builds on the Django 1.7 request I submitted and additionally fixes the migration issue using this technique.
http://kevindias.com/writing/django-custom-user-models-south-and-reusable-apps/
Closes #22

@ivanvenosdel I released new version v0.4.3 with your changes.. thank you for your effort.


Add New Metric to Show Number of Running PipelineRuns

added new gauge metric to show number of running pipelineruns. added tests and docs accordingly.
https://issues.redhat.com/browse/SRVKP-6375

Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/test

/test


Add an object to the package metadata to describe images.

For example,
{
  "images": {
    "icon-small": "http://some.org/foo/small.png",
    "icon-medium": "http://some.org/foo/medium.png",
    "icon-large": "http://some.org/foo/large.png",
    "screenshots": [
      "http://some.org/foo/screen-1.png",
      "http://some.org/foo/screen-2.png",
      "http://some.org/foo/screen-3.png"
    ]
  }
}

cc @ashenden

Wooo!  Thanks for adding this.  For now, can we start with these fields?

icon-small
icon-medium
icon-large
screenshot (can we make this an optional array of zero to n screenshots?)

Sure.  If we want to tighten this up we can specify the raster dimensions of each image type.  We can also specify supported image formats (for example PNG or JPG).  Could you give some guidance on that here?

Let's stick with PNG and JPG for now.
As for the dimensions, these may change.  If I give you something to start with, can we change them later?

Updated the example JSON snippet with your suggested values.

Yes, we can surely change them later.  I've started in on a more formal spec of the registry contents, so we can use words like SHOULD (instead of MUST) for aspects that are simply suggestions.  We should deep in mind that many packages will be community-contributed and we don't want to make things too restrictive.

True.  So, let's roll with these for now (forgive the formatting if it's off)...
{
  "images": {
    "icons": {
      "small": {
        "width": 48,
        "height": 48,
        "src": "http://some.org/foo/screen-1.png"
      },
      "medium": {
        "width": 128,
        "height": 128,
        "src": "http://some.org/foo/screen-1.png"
      },
      "large": {
        "width": 256,
        "height": 256,
        "src": "http://some.org/foo/screen-1.png"
      }
    },
    "screenshots": [
      "http://some.org/foo/screen-1.png",
      "http://some.org/foo/screen-2.png",
      "http://some.org/foo/screen-3.png"
    ]
  }
}

Thanks John!  I'll turn these into recommendations in the spec.  Probably no need to encode these in each package JSON.
Also, I'm guessing you've seen the iOS human interface guidelines but I just found a section on icons for mobile.  Could be helpful as a reference: https://developer.apple.com/library/ios/documentation/UserExperience/Conceptual/MobileHIG/IconMatrix.html.


chat api need one update config API to update conversation title and so on.

@liutianyi0801  Is this sill in our plan?

@liutianyi0801  Is this sill in our plan?

Inactive for a long time, close the issue


Clarity in documentation of class / methods regarding logging configuration settings ...

For those of us that are completely new to .Net / logging ...
Can a link to this excellent resource: Logging in .NET Core and ASP.NET Core  please be added to the following page:
ConsoleLoggerExtensions Class
The above page makes no mention of how the Console logger is configured by default.  For newbies such as myself, having the first link above added to the ConsoleLoggerExtensions Class page would have made life SO much easier!   Instead (again, remember, for those of us that are complete newbies), I could have saved several hours had the link to the fundamentals been included.
Thanks in advance.

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

ID: 7184a308-a254-9350-a5cd-5bced1f369ae
Version Independent ID: 726e3bf1-f367-d733-8933-bccc04da0e16
Content: Logging in .NET Core and ASP.NET Core
Content Source: aspnetcore/fundamentals/logging/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Sorry, @bdcoder2 ... We can't do it. The API docs are automatically generated.

Understood.
Perhaps it may be time to rethink how the documentation is written / generated?   Would it not be possible to generate (via an automated process) a MERGE of the API docs with such relevant "background" info as in the above example?
Anyone old enough to remember the DEC VAX/VMS documentation set? -- that was a gold standard. :)
Thanks again ( @guardrex / @Rick-Anderson ) for your replies!

Anyone old enough to remember the DEC VAX/VMS documentation set? -- that was a gold standard. :)

Yup, it was pretty good but not as extensive as all of docs.microsoft.com

... @Rick-Anderson - Yes, about 17 3-inch binders! (I still have the doc set on CD).   Every API had its own binder and everything was in one location (the relevant binder) -- that (documentation) paradigm made learning an API so much easier.    Don't get me wrong -- docs.microsoft.com is getting better with each version.  For those of us that jump into an framework / API cold -- for example, by looking up a class, its methods and properties, we expect to find everything in that one location, not in 10 or 20 different locations.   I am sure once I get my hands a little dirtier, it will start to come together;  however, I can certainly see why so many people struggle with .Net from the get-go.


Can't get audio to and from two Bluetooth Headsets simulataneously

I have tried using the dmix and dsnoop plugins to go about doing this similar to what is shown in the following post: ALSA Api: How to play two wave files simultaneously? And have looked at the following sources: https://www.alsa-project.org/main/index.php/Asoundrc#Software_mixing, and https://github.com/Arkq/bluez-alsa/wiki/Using-the-bluealsa-ALSA-pcm-plugin. All I want to do is use the arecord and aplay commands to input and output to these two Bluetooth devices at the same time.
When I ran: aplay -D mix_stream /home/pi/Desktop/noise.wav
Error: ALSA lib pcm_direct.c:1809:(_snd_pcm_direct_get_slave_ipc_offset) Invalid type 'bluealsa' for slave PCM
aplay: main:828: audio open error: Invalid argument
Current asoundrc file:
'''
pcm.mix_stream {
type plug
slave.pcm dmix_stream
}
pcm.dmix_stream {
type dmix
ipc_key 1234
slave.pcm y30
slave.pcm seshevo
}
pcm.y30 {
type plug
slave.pcm {
type bluealsa
device "FA:D8:78:FB:57:36"
profile "a2dp"
}
}
pcm.seshevo {
type plug
slave.pcm {
type bluealsa
device "5C:44:3E:54:E0:01"
profile "a2dp"
}
}
'''

dmix and dsnoop allow more than one application to share the same alsa device.  It looks like what you want is the multi plug-in which allows one application to output to play to two devices. There's a lot of advice around on that.  I haven't looked into the equivalent for combining input devices.
dmix and dsnoop only work with hardware devices.  They don't work with bluez-alsa.

There's a lot of advice around on that.

see, for example:
https://github.com/Arkq/bluez-alsa/issues/379#issuecomment-707309026
https://github.com/Arkq/bluez-alsa/issues/309#issuecomment-609427129

Okay, thanks that enabled me to play a wave file and output it to my two Bluetooth headsets. But do you have any suggestions as far as using arecord to record those two Bluetooth devices? My current . asoundrc file I will post below which I referenced off of @Arkq. When I run the following command: arecord -D bth-multi -c 2 -r 44100 -f s16_le test.wav. It outputs the following error: ALSA lib bluealsa-pcm.c:763:(_snd_pcm_bluealsa_open) Couldn't get BlueALSA PCM: PCM not found
arecord: main:828: audio open error: No such device.
'''
pcm.bth-1 {
type plug
slave.pcm {
type bluealsa
device "FA:D8:78:FB:57:36"
profile "a2dp"
}
}
pcm.bth-2 {
type plug
slave.pcm {
type bluealsa
device "5C:44:3E:54:E0:01"
profile "a2dp"
}
}
pcm.bth-multi {
type plug;
slave.pcm {
type multi;
slaves.a.pcm "bth-1";
slaves.a.channels 2;
slaves.b.pcm "bth-2";
slaves.b.channels 2;
bindings.0 { slave a; channel 0; }
bindings.1 { slave a; channel 1; }
bindings.2 { slave b; channel 0; }
bindings.3 { slave b; channel 1; }
master 1
}
ttable.0.0 1
ttable.1.1 1
ttable.0.2 1
ttable.1.3 1
}
'''

do you have any suggestions as far as using arecord to record those two Bluetooth devices?

Are you sure that the two devices actually have A2DP from their microphones? Very often bluetooth headsets use only SCO (HFP or HSP) for the microphone link, designed for use only when making 'phone calls - their A2DP support in that case is only for music playback.

@tristonn is this issue still of interest to you? There has been no activity here for several weeks now, so I would like to close it if no-one has anything else to add. I think that recording from two bluetooth devices is the same as recording from two soundcard devices - you just need 2 instances of arecord, one recording from each device. Have you tried that?

This issue appears to be abandoned. Closing.


tool xtensa-esp-elf-gdb version 12.1_20221002 is installed, but has reported version unknown (IDFGH-11698)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
idf 5.1.2
Operating System used.
Windows
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
What is the expected behavior?
can build
What is the actual behavior?
Setting PYTHONNOUSERSITE, was not set
Using Python in C:\Espressif\python_env\idf5.1_py3.11_env\Scripts
Python 3.11.2
Using Git in C:\Espressif\tools\idf-git\2.39.2\cmd
git version 2.39.2.windows.1
Setting IDF_PATH: E:\Project\esp-idf-v5.1.2
Checking Python compatibility
Adding ESP-IDF tools to PATH...
WARNING: tool xtensa-esp-elf-gdb version 12.1_20221002 is installed, but has reported version unknown
ERROR: tool xtensa-esp-elf-gdb has no installed versions. Please run 'C:\Espressif\python_env\idf5.1_py3.11_env\Scripts\python.exe E:\Project\esp-idf-v5.1.2\tools\idf_tools.py install' to install it.
WARNING: tool xtensa-esp32s3-elf version esp-12.2.0_20230208 is installed, but has reported version unknown
ERROR: tool xtensa-esp32s3-elf has no installed versions. Please run 'C:\Espressif\python_env\idf5.1_py3.11_env\Scripts\python.exe E:\Project\esp-idf-v5.1.2\tools\idf_tools.py install' to install it.
WARNING: tool riscv32-esp-elf version esp-12.2.0_20230208 is installed, but has reported version unknown
ERROR: tool riscv32-esp-elf has no installed versions. Please run 'C:\Espressif\python_env\idf5.1_py3.11_env\Scripts\python.exe E:\Project\esp-idf-v5.1.2\tools\idf_tools.py install' to install it.
WARNING: tool esp32ulp-elf version 2.35_20220830 is installed, but has reported version unknown
ERROR: tool esp32ulp-elf has no installed versions. Please run 'C:\Espressif\python_env\idf5.1_py3.11_env\Scripts\python.exe E:\Project\esp-idf-v5.1.2\tools\idf_tools.py install' to install it.
PS E:\Project\esp-idf-v5.1.2>
Steps to reproduce.
1、start ESP-IDF-5.1 PowerShell
2、can see  a lot error
Build or installation Logs.
No response
More Information.
No response

Hi @luacerbb , could you please remove your directory C:\Espressif and then run from PowerShell install.ps1 and export.ps1 from directory E:\Project\esp-idf-v5.1.2.
Post the commands output here in case it does not help.


self isn't highlighted in classes

I've searched and found issues about that
https://github.com/hdima/python-syntax/issues/21
https://github.com/hdima/python-syntax/issues/10
And I find you arguments very weak.
So your first argument:

I actually tried it and I don't really like how it looks. It adds too many colors to the code to the point that it's getting harder to follow.

Really? You think that fist example is less readable then the second? I (and obviously a lot of people) don't. I think it's more readable.

If you can easily filter class fields from other variables and stuff it is beneficial. You can't doubt that, don't you? I strongly think if you somehow did it so that when self is highlighted it is harder to you to follow code, you just did something wrong with colors, that's it.
You second argument.

self is actually just a conventional name for the first argument of a method. And if somebody uses for example just s instead it won't be highlighted and just will be confusing.

Well then s is should be highlighted also, or any other name that is the pointer to class instance. And pycharm is actually doing just that. But that's probably involve actually parsing file or something and is harder (or impossible) to do with regexp. If that is so, than you should state so.
You last argument

As it just an argument of a method it also can raise a question "why the other arguments not highlighted?" and also can add to the confusion.

I will answer this 'confusing' question. Well, I've already answered it actually, so I'll just repeat myself. Because it's beneficial to filter class fields from other stuff in methods.
So to state my point again.
It is obviously beneficial to filter class field from other stuff. If it's impossible, say so. If it's hard and I you doubt my arguments, so you won't do it yourself, than if you could guide somehow to the solution many will thank you for that (including me). If it's easy, but you won't do it, then again, it would be nice if you could provide help on how to do that.

This module implements the self and cls highlighting, which is a good feature.
Plus, it can be deactivated using a dedicated option.
Maybe you will found a compromise here.

self is actually just a conventional name for the first argument of a method. And if somebody uses for example just s instead it won't be highlighted and just will be confusing.

Precisely because its conventionnal (and a convention widely accepted), allowing it to be highlighted is a good feature. From experience, it's very appreciable.
Using s leads to confusing code, more than any highlighting.

But that's probably involve actually parsing file or something and is harder (or impossible) to do with regexp.

I think its impossible in vim. Needs a static code checker. Jedi could help, but it seems to be a little over-engineered.

python-mode also highlights self/cls and maps it to the highlight group pythonSelf. You could make it optional via a config variable.

@sscherfke I don't use python mode, because I don't like its take it all or leave it approach. I think that that it should be the set of instructions or a script or something else, but not the bunch of packages, that you can't remove if you don't need them.
If anyone interested in highlighting self, cls, default arguments and other improvements check out my fork of this project.

@purpleP Curretnly moving away from python-mode and missing the highlighted selfand cls … ;-)

@sscherfke by the way, for easy access here is the link to my fork

This has been fixed over at the fork vim-python/python-syntax.


Remove WML client steps.

Remove everything in 5.0 Deploy the model to Watson Machine Learning
and below. This is only for use with OpenScale, which is not currently
in use for this workshop.
The actual model save is in section 4.0 Save the model and test data,
and having secton 5.0 is therefore confusing, so remove it.

Out of date. Closing.


d3.permute should inherit the type of the passed-in array.

For example, if d3.permute a Int32Array, you should get back a new Int32Array.

would permutes = array.slice(0, i) be a solution? It passes all tests.
https://github.com/d3/d3-array/blob/master/src/permute.js#L2

my solution clashes with #93. Here's a solution that does both;
export default function(array, indexes) {
  var i = 0, permutes = array.slice();
  for (var p of indexes) permutes[i++] = array[p];
  return permutes.slice(0,i);
}

On second thought, this can’t work because the source can be an object (as shown in the README) rather than an array, and this wouldn’t be backwards-compatible if people were expecting d3.permute to return an array when they pass in a typed array (so that they can e.g. subsequently push new values onto it).


Unnecessary try/catch in __awaiter

The __awaiter helper includes try/catch blocks within the fulfilled and rejected functions. When an error is caught by these, it calls reject with the error.
TypeScript Version:  3.0.0-dev.201xxxxx
var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
    return new (P || (P = Promise))(function (resolve, reject) {
        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
        function step(result) { result.done ? resolve(result.value) : new P(function (resolve) { resolve(result.value); }).then(fulfilled, rejected); }
        step((generator = generator.apply(thisArg, _arguments || [])).next());
    });
};

This seems unnecessary, as per standard, the Promise implementation should do the rejection if an error is thrown.
From Promises/A+:

promise2 = promise1.then(onFulfilled, onRejected);
<IP_ADDRESS>  If either onFulfilled or onRejected throws an exception e, promise2 must be rejected with e as the reason.

And the reason I write this issue is that, not only does it appear unnecessary, but it prevents my custom Promise implementation from recognizing the difference between a true runtime error and an Error object that was created and passed to reject. This affects my ability to log errors as I'd like.
Expected behavior:
Allow the Promise implementation to be exposed to any errors and reject the promise.
var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
    return new (P || (P = Promise))(function (resolve, reject) {
        function fulfilled(value) { step(generator.next(value)); }
        function rejected(value) { step(generator["throw"](value)); }
        function step(result) { resolve(result.done ? result.value : new P(function (resolve) { resolve(result.value); }).then(fulfilled, rejected)); }
        step((generator = generator.apply(thisArg, _arguments || [])).next());
    });
};

Edit: As I look at this more, I realize it maybe requires a little more than just removing the try/catch statements, as the promises weren't directly chained together. I made a subtle change to step which I think maybe chains them all together.

The helper is written to align with the ECMAScript specification for async functions and the evaluation behavior of await, not the Promise/A+ specification. Although Promise/A+ specifies the chaining behavior, the ECMAScript specification does not require the chaining behavior when adopting the state of a foreign promise-like (such as jQuery's deferred). The only requirement for an operand to await that is "promise-like", is that it has a callable then method that accepts two callbacks. The ECMAScript specification does not observe or use the return value of then.


[ENG-2324] Enables 0 OSFStorage Used to be Displayed in the UI and Admin App

Purpose
Checks for storage_usage on a node were structured as if self.storage_usage; this leads to issues if there is a calculated 0 value for storage used.
Changes

osf/models/node.py: Checks storage_usage against None instead of 0
website/settings/defaults.py: Adds NOT_CALCULATED to the Enum
admin/nodes/serializers.py: Checks storage_usage against None instead of 0
website/project/views/node.py: Checks storage_usage against None instead of 0

QA Notes
Verify that storage usage for projects with no OSFStorage usage is displayed as 0.0B while other storage usage indication works as anticipated in the UI and in the Admin App.
Documentation
N/A
Side Effects
N/A
Ticket
JIRA Ticket

Cherry-picked in as 0895296dc0884c65ac6cabeb8efe3c021a17d06c, consider this merged :octocat:


Performance doesn't seem to be improving

After running the program and comparing it, I found that the performance has hardly improved. Could you please help me find out the reason?
When I looked at the data from the runtime console, I noticed several suspicious things:

BOLT-WARNING: split function detected on input : _ZL28delete_dictionary_tablespacev.cold/1. The support is limited in relocation mode.
PERF2BOLT: wrote 16614 objects and 0 memory objects to /home/lyf/Desktop/ocolos-data/perf.fdata
I looked at the sysbench_output.txt file and found that the performance dropped after 20 seconds and improved again after 130 minutes, but the performance was the same as when I started：
[ 4s ] thds: 16 tps: 2787.88 qps: 33449.53 (r/w/o: 27874.77/0.00/5574.75) lat (ms,95%): 8.90 err/s: 0.00 reconn/s: 0.00
[ 26s ] thds: 16 tps: 1918.98 qps: 23029.77 (r/w/o: 19191.81/0.00/3837.96) lat (ms,95%): 12.08 err/s: 0.00 reconn/s: 0.00
[ 167s ] thds: 16 tps: 2628.97 qps: 31538.64 (r/w/o: 26280.70/0.00/5257.94) lat (ms,95%): 8.74 err/s: 0.00 reconn/s: 0.00
I have added these two files to the attachment.
Hope to get your help.
sysbench_output.txt
runtime_console_infomation.txt

i rebuild mysql and after executing ./tracer anage,i got followering error: Error reading BOLT data input file: line 2, column 17: malformed field PERF2BOLT-ERROR: failed to match build-id from perf output. This indicates the input binary supplied for data aggregation is not the same recorded by perf when collecting profiling data, or there were no samples recorded for the binary. Use -ignore-build-id option to override. Could you please help me find out the reason? Hope to get your help.

i add the flag "-ignore-build-id",but performance still hasn't improved.

PERF2BOLT-ERROR: failed to match build-id from perf output. This indicates the input binary supplied for data aggregation is not the same recorded by perf when collecting profiling data, or there were no samples recorded for the binary. Use -ignore-build-id option to override.

This error message should not be there if you didn't turn on the continuous optimization mode. Please check if your perf.data in your tmp_data_dir is stale. I suspect you use the stale perf.data with your newly generated mysqld.
Also, in your  tmp_data_dir, there should be a mysqld.bolt binary.
(1) Copy that mysqld.bolt to /usr/local/mysql/bin/ (or your mysql root dir)
(2) run mysqld.bolt with /usr/local/mysql/bin/mysqld.bolt --user=ocolos --port=3306 --basedir=/usr/local/mysql --datadir=/usr/local/mysql/data --plugin-dir=/usr/local/mysql/lib/plugin --log-error=ocolos.err --pid-file=ocolos.pid
(3) after mysqld.bolt is successfully setup, in another terminal, run sysbench's read only workload first using this command and then using this command
If the performance of the BOLTed binary is not better than the original one, it means BOLT cannot improve the performance of mysqld on your machine.

PERF2BOLT-ERROR: failed to match build-id from perf output. This indicates the input binary supplied for data aggregation is not the same recorded by perf when collecting profiling data, or there were no samples recorded for the binary. Use -ignore-build-id option to override.

This error message should not be there if you didn't turn on the continuous optimization mode. Please check if your perf.data in your tmp_data_dir is stale. I suspect you use the stale perf.data with your newly generated mysqld.
Also, in your tmp_data_dir, there should be a mysqld.bolt binary. (1) Copy that mysqld.bolt to /usr/local/mysql/bin/ (or your mysql root dir) (2) run mysqld.bolt with /usr/local/mysql/bin/mysqld.bolt --user=ocolos --port=3306 --basedir=/usr/local/mysql --datadir=/usr/local/mysql/data --plugin-dir=/usr/local/mysql/lib/plugin --log-error=ocolos.err --pid-file=ocolos.pid (3) after mysqld.bolt is successfully setup, in another terminal, run sysbench's read only workload first using this command and then using this command
If the performance of the BOLTed binary is not better than the original one, it means BOLT cannot improve the performance of mysqld on your machine.

Thank you very much! When I extend the duration of sysbench, there will be a boost at around 230s. Maybe it wasn't long enough.


peek showing 2 definitions on functions with one assignment

VSCode Version: Version 1.24.0 (1.24.0)
OS Version:  10.12.6 (16G1314)

Steps to Reproduce:

Create a JS function declared as const func = () => {} or const obj = { func: () => {}}
Create call site for that function
Go to Definition

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

Since the latest update when I go to definition the editor goes to the correct location but also opens PEEK on top of the definition. It happens consistently with arrow functions assigned to vars or object keys. It doesn't seem to be happening with function declarations.
This has never happened before, and its happening consistently since the last update, which leads me to believe it isn't an issue with the language provider.

You can see here that VS code has found 2 definitions, even though they are for the same line and assignment. Peek is showing the function on top of the function in the editor

This is not a duplicate of the provided issues

Duplicate of https://github.com/Microsoft/TypeScript/issues/24861

@mjbvz This is not happening in typescript files, its happening in js files. Is there some way I can stop TS bugs from affecting my JS code?

Typescript powers both JS and TS intellisense. This is the correct duplicate bug. You can workaround it by manually upgrading your workspace to typescript@next by following these instructions

Good to know, thank you for the fix instructions


[web]: PWA not showing option to install

I have launched about 3 Flutter web projects in the past couple of months. I use HTML as my preferred web renderer, but one thing I have noticed is that I no longer see that button on Chrome search bar or on the more menu to install the app to my PC as a PWA. I literally have to save the web page as a shortcut for each of the projects.

Hi @Kings-Samuel, sorry for replying late.
But I can not reproduce the issue on the latest stable and master channels.
I build Web app with HTML renderer and the can see PWA install button on the Chrome search bar.

Demo
https://reproduceissues.web.app

flutter doctor -v

Please provide a completed and minimal reproducible code sample so that we may verify this. Also, please provide the output of flutter doctor -v as well.
Thank you!


Add Test Reporter

Test: CI

Unfortunately, this seem to require to have the workflow checked in on main before the CI picks it up...


fix#366 Fixing the alignment of button group in appbar and background color of buttons

PR Fixes:

1 Button Group alignment in Appbar
2 Background Color of all buttons in the appbar

Resolves #366
Screenshot

Checklist before requesting a review

[X] I have performed a self-review of my code
[X] I assure there is no similar/duplicate pull request regarding same issue

hey @hkirat will it be okay If I work on cleaning the codebase meaning changing this

into something like this

with this, it will be easier for newcomers to quickly understand the codebase

Too old + has conflicts. Closing for now feel free to open and discuss


Cannot initiate connection to backend

I face an issue that my request not reach to backend... Can you help in this?
also the same subscription work in Apollo graphql website.
 import 'package:flutter/material.dart';
import 'package:graphql_flutter/graphql_flutter.dart';

class TrackingScreen extends StatefulWidget {
  const TrackingScreen({Key? key}) : super(key: key);

  @override
  State<StatefulWidget> createState() {
    return _TrackingScreenState();
  }
}

class _TrackingScreenState extends State<TrackingScreen> {
  @override
  Widget build(BuildContext context) {
    final httpLink = HttpLink(
      'https://......',
    );

    final authLink = AuthLink(
      getToken: () async => 'Bearer token',
    );

    var link = authLink.concat(httpLink);

    final websocketLink = WebSocketLink(
      'wss://......',
      config: const SocketClientConfig(
        inactivityTimeout: Duration(seconds: 30),
        delayBetweenReconnectionAttempts: Duration(seconds: 10),
      ),
    );

    link = Link.split(
      (request) => request.isSubscription,
      websocketLink,
      link,
    );

    final client = ValueNotifier<GraphQLClient>(
      GraphQLClient(
        cache: GraphQLCache(),
        link: link,
      ),
    );

    return GraphQLProvider(
      client: client,
      child: CacheProvider(
        child: Subscription(
          options: SubscriptionOptions(
            document: gql(
                r"""subscription Tracking($frequencyInSeconds: Int!, $packageNumber: Int!) {
                  tracking(frequencyInSeconds: $frequencyInSeconds, packageNumber: $packageNumber) {

                    timestamp
                  }
                }"""),
            variables: {
              "frequencyInSeconds": 3,
              "packageNumber": 10,
            },
            operationName: 'Tracking',
          ),
          builder: (result) => result.isLoading
              ? const Text('Loading...')
              : Text(result.data.toString()),
        ),
      ),
    );
  }

we need an example for this!
You can use our endpoint https://api.chat.graphql-flutter.dev/graphql

Hi @vincenzopalazzo
Yes I try it with your endpoint but it the same issue keep show loading only.

if you push the example that you try somewhere I can reproduce it, because I'm currently use in production the client and my code works

Hi
Here is the full code exactly.
`import 'package:flutter/material.dart';
import 'package:graphql_flutter/graphql_flutter.dart';
class TrackingScreen extends StatefulWidget {
const TrackingScreen({Key? key}) : super(key: key);
@override
State createState() {
return _TrackingScreenState();
}
}
class _TrackingScreenState extends State {
@override
Widget build(BuildContext context) {
final httpLink = HttpLink(
'https://api.chat.graphql-flutter.dev/graphql',
);
final authLink = AuthLink(
  getToken: () async => 'Bearer token',
);

var link = authLink.concat(httpLink);

final websocketLink = WebSocketLink(
  'wss://api.chat.graphql-flutter.dev/graphql',
  config: const SocketClientConfig(
    inactivityTimeout: Duration(seconds: 30),
    delayBetweenReconnectionAttempts: Duration(seconds: 10),
  ),
);

link = Link.split(
  (request) => request.isSubscription,
  websocketLink,
  link,
);

final client = ValueNotifier<GraphQLClient>(
  GraphQLClient(
    cache: GraphQLCache(),
    link: link,
  ),
);

return GraphQLProvider(
  client: client,
  child: CacheProvider(
    child: Subscription(
      options: SubscriptionOptions(
        document: gql("""subscription ChatCreated {
          chatCreated {
            id
            description
            name
          }
        }"""),
      ),
      builder: (result) => result.isLoading
          ? const Center(child: Text('Loading...'))
          : Center(child: Text(result.data.toString())),
    ),
  ),
);

}
}`

@vincenzopalazzo
check this screenshot of what is printing in the debug console.

Mh I think you need to change the protocol.
Check how I do this https://github.com/zino-hofmann/graphql-flutter/pull/1204

Hi @vincenzopalazzo
The issue fixed after upgrade to graphql_flutter 5.1.1-beta.4  and add subprotocol to Graphql-transport-ws.

Coo! :) the beta is not so reckless is it?


[Request]support to Woodworks mod

https://www.curseforge.com/minecraft/mc-mods/woodworks

I will try to implement

implementation done, now just wait for the 1.4.6 compilation | https://github.com/vindocel/Dark-Everywhere/commit/02a450e4b3a47f1ac7caf48449fa67075ff88e1d


net_buf issue leads to unwanted elem free

Describe the bug
Current net_buf structure shared the first word in a union between sys_snode_t node;  and struct net_buf *frags;.
This has been introduced a very long time ago : https://github.com/zephyrproject-rtos/zephyr/commit/553af7cd52f5726d1318338c545fcafdff2e44b5
However, net_buf allocation are handled by a LIFO, and the current documentation of k_lifo_put(), clearly states that the first word of the element must be reserved for the kernel, which is not the case here.
https://github.com/zephyrproject-rtos/zephyr/blob/2637c0ed22f817d39f206721d09ae01c8baf018c/include/kernel.h#L2302
It appears, that this shared first word works great in many cases as the kernel mostly use it only when the element is in the queue and therefore the element is not allocated at this time.
However, in a edge case (probably when the queue is empty and a new element is added), the last element is modified, even if it is currently in used.
At that moment, the value of node is written, hence the frags value is not NULL anymore, and bad things happens in my use case.
To Reproduce
I've currently see this using mcumgr and smp_bt, however I expected it might be reproducible more easily when the edge case as been localized more accurately.
Impact
On my case, this leads to a mess in the net_buf pool, that leads to bt_conn ref not being properly released, witch means bt_le_adv_resume() is never called in bt_conn_unref().
Environment:

Zephyr SDK,
nrf52840,
zephyr 2.4 but as neither net_buf nor queue implementations s changed since, I really think main is also affected.

This is a eventually a duplicate of https://github.com/zephyrproject-rtos/zephyr/issues/32579.
See https://github.com/zephyrproject-rtos/zephyr/pull/38830#issuecomment-929118977 for more details.


cc-wrapper: fix bool handling for empty and zero values

Motivation for this change
Before the code would fail silently for zero values and with some output for
empties. We now currently handle both via defaulting value to zero and making
let return success error code when there's no syntax error.
Test case:

nix-shell -A hello
export NIX_ENFORCE_NO_NATIVE=""
gcc --version (observe error)
export NIX_ENFORCE_NO_NATIVE=0
gcc --version (observe silent failure)

Things done

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

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

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

@Ericson2314 Updated with your suggestions, thanks!


Unable to send message

Hi, I logged in successfully. I tried to message you. When I click "Send" nothing happens.
On a Mac.

Is your window empty or filled with Messages from other users like this?

I forgot to enter amount of SBD/STEEM. It seems now to default to 0.001 after I entered it once, but it was not working initially.


Plugin DNS lookups (Kubernetes, SRV records, incorrect port, search domains)

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
2.8.3 and 3.1.0
Current Behavior
When a plugin is configured to point a Kubernetes ClusterIP Service that has:

more than one port
a hostname with a trailing dot, to prevent additional DNS lookups for search domains

Then Kong:

uses a different port from the ClusterIP Service, which causes plugin errors and data loss
makes additional DNS lookups for search domains, which uses additional resources

I've tested this with two plugins, to see if the behaviour was coming from the HTTP client. The configs for these are linked further down.
zipkin
DNS lookups for SRV and search domains:
IP <IP_ADDRESS>.33336 > <IP_ADDRESS>.53: 13718+ SRV? jaeger.kube-system.svc.cluster.local. (54)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.33336: 13718*- 3/0/1 SRV jaeger.kube-system.svc.cluster.local.:4318 0 33, SRV jaeger.kube-system.svc.cluster.local.:9411 0 33, SRV jaeger.kube-system.svc.cluster.local.:16686 0 33 (382)
IP <IP_ADDRESS>.33216 > <IP_ADDRESS>.53: 57462+ A? jaeger.kube-system.svc.cluster.local.default.svc.cluster.local. (80)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.33216: 57462 NXDomain*- 0/1/0 (173)
IP <IP_ADDRESS>.56435 > <IP_ADDRESS>.53: 2101+ A? jaeger.kube-system.svc.cluster.local.svc.cluster.local. (72)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.56435: 2101 NXDomain*- 0/1/0 (165)
IP <IP_ADDRESS>.34735 > <IP_ADDRESS>.53: 55158+ A? jaeger.kube-system.svc.cluster.local.cluster.local. (68)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.34735: 55158 NXDomain*- 0/1/0 (161)

Connection to 4318 instead of 9411:
IP <IP_ADDRESS>.34136 > <IP_ADDRESS>.4318: Flags [S], seq<PHONE_NUMBER>, win 64240, options [mss 1460,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
IP <IP_ADDRESS>.4318 > <IP_ADDRESS>.34136: Flags [S.], seq<PHONE_NUMBER>, ack<PHONE_NUMBER>, win 65160, options [mss 1460,sackOK,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>,nop,wscale 7], length 0

Plugin error because it's used the wrong Jaeger port:
[error] 1127#0: *7949 [kong] handler.lua:99 reporter flush failed: 404 Not Found, context: ngx.timer

datadog
DNS lookups for SRV and search domains:
IP <IP_ADDRESS>.60069 > <IP_ADDRESS>.53: 32873+ SRV? metrics.kube-system.svc.cluster.local. (55)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.60069: 32873*- 2/0/1 SRV metrics.kube-system.svc.cluster.local.:1234 0 50, SRV metrics.kube-system.svc.cluster.local.:8125 0 50 (296)
IP <IP_ADDRESS>.37335 > <IP_ADDRESS>.53: 61911+ A? metrics.kube-system.svc.cluster.local.default.svc.cluster.local. (81)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.37335: 61911 NXDomain*- 0/1/0 (174)
IP <IP_ADDRESS>.36719 > <IP_ADDRESS>.53: 48+ A? metrics.kube-system.svc.cluster.local.svc.cluster.local. (73)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.36719: 48 NXDomain*- 0/1/0 (166)
IP <IP_ADDRESS>.54753 > <IP_ADDRESS>.53: 57483+ A? metrics.kube-system.svc.cluster.local.cluster.local. (69)
IP <IP_ADDRESS>.53 > <IP_ADDRESS>.54753: 57483 NXDomain*- 0/1/0 (162)

Connection to 1234 instead of 8125:
IP <IP_ADDRESS>.49517 > <IP_ADDRESS>.1234: UDP, length 75
IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP <IP_ADDRESS> udp port 1234 unreachable, length 111
IP <IP_ADDRESS>.49517 > <IP_ADDRESS>.1234: UDP, length 77
IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP <IP_ADDRESS> udp port 1234 unreachable, length 113
IP <IP_ADDRESS>.49517 > <IP_ADDRESS>.1234: UDP, length 79
IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP <IP_ADDRESS> udp port 1234 unreachable, length 115

Curiously it appears to retry against the other port:
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 75
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 70
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 77
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 78
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 79
IP <IP_ADDRESS>.42676 > <IP_ADDRESS>.8125: UDP, length 75

Plugin error because it's used the wrong statsd port:
[error] 1128#0: *70 send() failed (111: Connection refused), context: ngx.timer
[error] 1128#0: *70 [kong] statsd_logger.lua:82 failed to send data to metrics.kube-system.svc.cluster.local.:8125: connection refused, context: ngx.timer

Expected Behavior
Kong should:

use the port specified in the plugin configuration
not perform additional DNS lookups for FQDNs that have trailing dots

We have worked around this by setting dns_order to LAST,A,CNAME but the default behaviour was very surprising.
Maybe SRV lookups should come last or be opt-in in a future major version?
Steps To Reproduce
You'll need to use 3 concurrent terminal windows so I've broken the instructions into those groups.
The YAML manifests can be found here: https://gist.github.com/dcarley/888134d7f4eea62efeb49c4bf33a1406
The whole process can be repeated for datadog.yaml instead of zipkin.yaml.
Terminal 1
Start minikube:
minikube start

Start minikube tunnel so that we can access the LoadBalancer Service (this might be Mac specific):
sudo --validate && minikube tunnel

Terminal 2
Deploy the chart:
helm upgrade test kong/kong --install --version 2.14.0 --wait

Export the address using the instructions from the Helm output:
HOST=$(kubectl get svc --namespace default test-kong-proxy -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
PORT=$(kubectl get svc --namespace default test-kong-proxy -o jsonpath='{.spec.ports[0].port}')
export PROXY_IP=${HOST}:${PORT}

Deploy an example app, backend, and plugin from:
kubectl apply -f httpbin.yaml
kubectl apply -f zipkin.yaml

Note the ClusterIP of the backend:
kubectl get service -n kube-system

Terminal 3
Start a debug container to watch network traffic from the proxy:
kubectl debug -it test-kong-6879b698d4-f4dhh --target proxy --image alpine
apk add tcpdump
tcpdump -nt port 53 or host <ClusterIP>

Terminal 2
Make a request to trigger the plugin(s):
curl $PROXY_IP/status/201

Anything else?
Minikube version:
% minikube version
minikube version: v1.25.2
commit: v1.25.2

Kubernetes version:
% kubectl version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5", GitCommit:"c285e781331a3785a7f436042c65c5641ce8a9e9", GitTreeState:"archive", BuildDate:"1980-01-01T00:00:00Z", GoVersion:"go1.17.13", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.3", GitCommit:"816c97ab8cff8a1c72eccca1026f7820e93e0d25", GitTreeState:"clean", BuildDate:"2022-01-25T21:19:12Z", GoVersion:"go1.17.6", Compiler:"gc", Platform:"linux/arm64"}

Hi @dcarley! Thanks for your detailed report and reproduction steps.
We are already working to fix this behavior, but it's been challenging. As you mentioned, you can workaround this issue by changing dns_order. Another option is not using FQDN hostnames, but that can decrease name resolution performance.
We will update this issue as soon as we come up with a fix.

That's great, thanks!
We use FQDNs everywhere because we've been bitten by Kubernetes DNS performance and Kong timer exhaustion, both separately and combined, a few times in the past.

The timer exhaustion issue has been fixed in Kong 3.x, but you are right, if you can stick to using FQDNs the performance is way better.

Dear contributor,
We're closing this issue as there hasn't been any update to it for a long time. If the issue is still relevant in the latest version, please feel free to reopen it. We're more than happy to revisit it again.
Please have a look at our pledge to the community for more information.
Sincerely,
Kong Gateway Team


Move end of line comments to their own line to improve readability

Replaces and closes #309 (sorry for the delay :bowing_man:)

But this triggers a warning on docker build, which says that empty continuation lines will be an error in a future release. Has that future been saved yet?

That was fixed in https://github.com/moby/moby/pull/35004

That was a bug in a particular version of Docker that should've now been long-since fixed where this was detected incorrectly as an empty continuation line. :+1:
See https://github.com/moby/moby/pull/35004 for the PR which fixed this (Docker 17.10+).

But it seems like we use the default docker version on Travis CI, which is really old. So that error is shown.


Archive this addon

Last commit: 5 years ago
Codebase: very far from the current paradigm
I recently removed this addon from a project, the code we replaced it with was simple and easier to maintain than an external dependency
### Tasks
- [ ] Update README to include information about what to do instead of using this addon

@adopted-ember-addons/admin Any objections here?

nope. looks like vertical collection would be better

Let's archive it 👍

Shall I archive it? Anything we need to do before doing so?

I don't think there's anything you need to do first

I don't think there's anything you need to do first

I think we should say something in the README and maybe deprecate the npm package?


regression adjusted estimator for propensity_score_weighting

Thank you for developing this package.
I am looking for regression adjusted estimator for propensity_score_weighting.
Is that something the package would be supporting in the future?
Thanks.

That will be a very good addition. If I understand correctly, you are referring to the doubly robust estimator.
It may be a while before we get to it. Would you like to contribute and add it to the package?

I am using this package for a project at work. I would have to see how the approval process works for such contributions. Not clear at the moment.

Thank you for considering contributing to DoWhy.  In case it is useful for your work to consider, here is the section on contributing to DoWhy that is included on our project page, such as the contributor license agreement and FAQ.
Contributinghttps://github.com/microsoft/dowhy#id25
This project welcomes contributions and suggestions. For a guide to contributing and a list of all contributors, check out CONTRIBUTING.mdhttps://github.com/microsoft/dowhy/blob/master/CONTRIBUTING.md. You can also join the DoWhy development channel on Discord: [discord] https://discord.gg/cSBGb3vsZb
Most contributions require you to agree to a Contributor License Agreement (CLA) declaring that you have the right to, and actually do, grant us the rights to use your contribution. For details, visit https://cla.microsoft.comhttps://cla.microsoft.com/.
When you submit a pull request, a CLA-bot will automatically determine whether you need to provide a CLA and decorate the PR appropriately (e.g., label, comment). Simply follow the instructions provided by the bot. You will only need to do this once across all repos using our CLA.
This project has adopted the Microsoft Open Source Code of Conducthttps://opensource.microsoft.com/codeofconduct/. For more information see the Code of Conduct FAQhttps://opensource.microsoft.com/codeofconduct/faq/ or contact<EMAIL_ADDRESS>with any additional questions or comments.
From: adg-ci @.>
Sent: Wednesday, April 20, 2022 11:08 AM
To: microsoft/dowhy @.>
Cc: Subscribed @.***>
Subject: Re: [microsoft/dowhy] regression adjusted estimator for propensity_score_weighting (Issue #407)
I am using this package for a project at work. I would have to see how the approval process works for such contributions. Not clear at the moment.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fdowhy%2Fissues%2F407%23issuecomment-1104260492&data=05|01|emrek%40microsoft.com|2735a7f40f3b4316a6ce08da22f8b305|72f988bf86f141af91ab2d7cd011db47|1|0|637860748801303912|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Eimmop8WuNxXtEGViQexLrB%2B1%2BKW0KJryqP3BPq3GHM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABNUPUCZXKEQ7MJ53BF2MZDVGBBXZANCNFSM5TJNAB6A&data=05|01|emrek%40microsoft.com|2735a7f40f3b4316a6ce08da22f8b305|72f988bf86f141af91ab2d7cd011db47|1|0|637860748801303912|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=KAWl8MHnXLS7Vjd6YHI3NIkzAnmh4ZNPtwtbtUSqFGc%3D&reserved=0.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

Hi! I was thinking of working on this -> implement the doubly robust estimator in doWhy. Do you have any pointers so I could get started?

Sure. At a high level, the doubly robust estimator is a combination of the regression estimator and the inverse propensity weighting estimator. So,  you will need to create a new Estimator class having these models as part of it (e.g., as attributes).
For the exact way in which these two estimators should be combined, you can refer to, https://academic.oup.com/aje/article/173/7/761/103691. Let me know if you have any questions.