Sync Queue GUI issues

[ ] we should rename the GUI to Sync Queue to better reflect what it does

[ ] Let's change the status sorting to In Progress , Queued, Failed... the rest as is

[ ] default column widths are a bit annoying and have to be expanded every time

[ ] This should say Re-sync Active Site and Re-sync Remote Site

[ ] This should maybe say Open in explorer, It's probably enough to imply it means active site

[ ] To make it clear, let's join the date columns with the site icons.

[ ] also when Site has failed, we should clearly mark it. Colour or similar would work .

That 'Open in Explorer' is there for cases studio <> local , in that case you should have access to both, studio and local versions.

Pushed implemented changes


Add partitioning information to Raco's catalog interface

This is a requirement for https://github.com/uwescience/myria/issues/766
On Thu, Aug 13, 2015 at 1:01 PM, Bill Howe<EMAIL_ADDRESS>wrote:
On Thu, Aug 13, 2015 at 12:46 PM, Brandon Myers<EMAIL_ADDRESS>wrote:
We just need to add partitioning to racos catalog interface.
Brandon: Can you take a stab at a design for this?
One thought is to leave get_scheme as is, but to just add another call, say get_partitioning(rel_key), that returns a list of attributes on which the relation is partitioned.
Not sure if we need the workers as well.  If so, I'd imagine making that a separate call get_workers(rel_key)
(We could also have a similar call get_order(rel_key) that returns a list of attributes on which the relation is sorted, but we can do that once it's motivated.)

see #467

Fixed in https://github.com/uwescience/raco/pull/450.


fix: react refresh module id

Summary
fix #3762

Test Plan

packages/playground/cases/react/basic/react-refresh.js

Looks good to me. Would you take a look @underfin?


docs(guide/Developer Guide): describe your change...

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Fix/Feature: Docs have been added/updated
[ ] Fix/Feature: Tests have been added; existing tests pass

Other information:

@Biplab-1991 Not sure what you wanna do with this PR but this looks like a mistake to me.
@Narretz You can close this one.

Don't waste keystrokes on a PR like this @frederikprijck ;)


python3Packages.dnspython: 2.1.0 -> 2.2.0rc1

Update of dnspython
Motivation for this change
needed for pgadmin4 init
https://github.com/NixOS/nixpkgs/pull/154764
This version includes a bugfix for missing /etc/protocols files and is needed for building pgadmin4
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

evaluation fails due to missing maintainer info #155411


Fix the issue of scheduling task on all tables

ControllerPeriodicTask.processTables() should only process the tables that are managed by the controller
PinotTaskManager is overriding processTables() to schedule tasks

Codecov Report

Merging #4563 into master will increase coverage by 0.01%.
The diff coverage is 62.5%.

@@             Coverage Diff              @@
##             master    #4563      +/-   ##
============================================
+ Coverage     64.44%   64.45%   +0.01%
  Complexity       20       20
============================================
  Files          1064     1064
  Lines         55348    55351       +3
  Branches       8068     8068
============================================
+ Hits          35669    35678       +9
+ Misses        17058    17049       -9
- Partials       2621     2624       +3

Impacted Files
Coverage Δ
Complexity Δ

...controller/helix/core/minion/PinotTaskManager.java
13.2% <0%> (ø)
0 <0> (ø)
:arrow_down:

...elix/core/periodictask/ControllerPeriodicTask.java
77.27% <83.33%> (+4.1%)
0 <0> (ø)
:arrow_down:

...egation/function/customobject/MinMaxRangePair.java
75.86% <0%> (-24.14%)
0% <0%> (ø)

...pache/pinot/core/util/SortedRangeIntersection.java
83.82% <0%> (-7.36%)
0% <0%> (ø)

...e/impl/dictionary/LongOnHeapMutableDictionary.java
89.13% <0%> (-6.53%)
0% <0%> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
89.28% <0%> (-5.36%)
0% <0%> (ø)

...ore/startree/v2/builder/BaseSingleTreeBuilder.java
90.52% <0%> (-4.74%)
0% <0%> (ø)

...pinot/core/operator/docidsets/OrBlockDocIdSet.java
84.9% <0%> (-3.78%)
0% <0%> (ø)

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
76.78% <0%> (-3.58%)
0% <0%> (ø)

...der/HighLevelConsumerBasedRoutingTableBuilder.java
90.9% <0%> (-3.04%)
0% <0%> (ø)

... and 12 more

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 d8d0bb5...4053116. Read the comment docs.


PIN code

Numbers padding and bigger sizing to allow better visibility

Number pad moves down after having typed first number.

Confirmed on iOS too.

Does it cause any issues or uneasiness, guys ?

Yes sir!
I don't use PIN for that reason. Whenever I want to log in quickly I sometimes enter my PIN incorrectly. Things like these are annoying for 'quick users'. Also it's not what a user would expect.
I'm a front-end developer. I don't mind helping out with coding/styling/usability but I don't really know how GitHub works.

Uneasiness... I would say to fix when you have time. Not urgent in my view.

v1.4.6 Android already has a fix, could you guys confirm if it is better?!


[libbson] [mongo-c-driver] [mongo-cxx-driver] Updated to latest

Describe the pull request

What does your PR fix?
Fixes #18730

Which triplets are supported/not supported? Have you updated the CI baseline?
all previously supported, No

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Related PR #17883

Have you tested the usage for these ports? Seems the usage has been changed now.

@NancyLi1013 I've a local project that uses the new targets and they work (there was a Boost::boost missing dependency I just fixed). The old targets are still there, I haven't changed them, so they should work as before. The usage is changed to point to the new targets though, as the other ones were marked deprecated in the upstream repo.

I made several of @NancyLi1013 's requested changes in https://github.com/microsoft/vcpkg/pull/20039/files

I made several of @NancyLi1013 's requested changes in https://github.com/microsoft/vcpkg/pull/20039/files

Thanks for your help. @BillyONeal
Could you please help further review this PR?
If there are no other points that need to be done here, we may consider merging this PR.

any update ?

any update ?

@westfly we're waiting for the response from @bucurb.
@bucurb
Could you please help confirm if you're satisfied with the status of this PR? Also, please resolve the conflicts.

@NancyLi1013 I won't have time to work on this PR, happy to go with whatever changes you make to it.

@NancyLi1013 I won't have time to work on this PR, happy to go with whatever changes you make to it.

Thanks for your response. I will update the changes to make this PR fine.


Request for scanning new channels EPG for horizon.tv Germany

Hello
By scanning their EPG sourcing, many new channels could be add
For
https://www.horizon.tv/de_de/tv/tv-programm.html
https://github.com/iptv-org/epg/tree/master/sites/horizon.tv
Please

Please specify the name of the channels that are missing so we can help you.

GEO
Sky Documentaries
Sky Replay
Sky Nature HD
Sky Schowcase

GEO - GeoTV.de
Sky Documentaries  -SkyDocumentaries.de
Sky Replay - SkyReplay.de
Sky Nature HD - SkyNature.de
Already in the channels xml
Sky Showcase
New channel not in the database, but it's not listed on Horizon's own site, so if we added it there would be no data to get
https://www.horizon.tv/de_de/tv/tv-programm.html


lxd/apparmor: Allow remount using noatime

Tentative fix for the problem discussed in https://github.com/lxc/distrobuilder/pull/558#discussion_r721780858

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

jenkins: test this please

Testsuite passed


Select and Duplicate style

Select is the basis for various things...

Looks like I'm not going to do this.


Build error MSB4057: The target "Restore" does not exist in the project.

Using Visual Studio 2019.
build engine 14.0.25420.1
from command prompt at the CsvReader-master folder (freshly unzipped from github archive)
Typing build I get back:
Checking Paket version (downloading latest stable)...
Hash file of version 5.242.2 found in cache.
Copying version 5.242.2 from cache.
Done in 2.14 second(s).
Paket version 5.242.2
Extracted Paket.Restore.targets to: C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master.paket\Paket.Restore.targets (Can be disabled with PAKET_SKIP_RESTORE_TARGETS=true)
Starting full restore process.
Performance:

Disk IO: 3 seconds
Runtime: 5 seconds
Building project with version: LocalBuild
Shortened DependencyGraph for Target Default:
<== Default
<== Test
<== Build
<== PackageRestore
<== SetVersion
<== Clean

The running order is:

Clean
SetVersion
PackageRestore
Build
Test
Default
Running build with 1 worker
Starting Target: Clean
Creating C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\build
Finished Target: Clean
Starting Target: SetVersion (==> Clean)
C:\Program Files\Git\cmd\git.exe log --oneline -1
Exception! (git log --oneline failed: fatal: not a git repository (or any of the parent directories): .git)
Created AssemblyInfo file "./code/SolutionInfo.cs".
Finished Target: SetVersion
Starting Target: PackageRestore (==> SetVersion)
Building project: C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln
C:\Program Files (x86)\MSBuild\14.0\Bin\MSBuild.exe  "C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln" /t:Restore /m  /nodeReuse:False    /p:RestorePackages="False" /p:Configuration="Release" /logger:Fake.MsBuildLogger+ErrorLogger,"C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\packages\build\FAKE\tools\FakeLib.dll"
Microsoft (R) Build Engine version 14.0.25420.1
Copyright (C) Microsoft Corporation. All rights reserved.

Build started 3/3/2020 1:27:00 PM.
1>Project "C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln" on node 1 (Restore target(s)).
1>ValidateSolutionConfiguration:
Building solution configuration "Release|Any CPU".
1>C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln.metaproj : error MSB4057: The target "Resto
re" does not exist in the project. [C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln]
1>Done Building Project "C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln" (Restore target(s))
-- FAILED.
Build FAILED.
   "C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln" (Restore target) (1) ->
     C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln.metaproj : error MSB4057: The target "Res
   tore" does not exist in the project. [C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln]

0 Warning(s)
1 Error(s)

Time Elapsed 00:00:00.13
Running build failed.
Error:
Building C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln failed with exitcode 1.

Build Time Report
Target           Duration

Clean            00:00:00.0044802
SetVersion       00:00:00.1475222
PackageRestore   Failure
Total:           00:00:01.3314902
Status:          Failure

Building C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln failed with exitcode 1.
MSB4057: C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master\CsvReader.sln.metaproj(0,0): The target "Restore" does not exist in the project.

C:\Users\Emery Premeaux\Dropbox\repos\CsvReader-master>

I am also encountering this build issue. @phatcher , could you offer some advice? Are there some minor changes needed? Could you also specify in the readme.md what OS and .Net versions are required to build successfully?
Thanks,
Adam

@lancasteradam , @DIYsciBorg please verify that based on my fork: https://github.com/drcolombo/CsvReader. I believe this repository is not maintained anymore and I've already updated the whole project to support FAKE 5 last year.


[Snyk] Upgrade org.freemarker:freemarker from 2.3.31 to 2.3.32

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.freemarker:freemarker from 2.3.31 to 2.3.32.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a year ago, on 2023-01-06.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Freemarker removed from POM.  PR no longer needed.


Add Privacy

https://github.com/pluja/awesome-privacy
[Explain what this list is about and why it should be included here]
This is a curated list of services and alternatives that respect users' privacy. I think this should be included here because privacy is a right and people need easy ways to learn and choose how to protect their privacy. This list provides awesome alternatives to a lot of services and provides knowledge, this is why I think it is awesome.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions.
You have read and understood the instructions for creating a list.
This pull request has a title in the format Add Name of List.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

Your entry should be added at the bottom of the appropriate category.
The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it.
Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
The default branch should be named main, not master.
Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

Non-generated Markdown file in a GitHub repo.
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate. Please search for existing submissions.
Only has awesome items. Awesome lists are curations of the best, not everything.
Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't use hard-wrapping.
Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.
Doesn't include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

unicorn

The linter shows some errors but the formatting of the list (even a bit different from typical awesome-lists) is good and follows the guidelines, it's just that there are some things that the linter does not expect.

I understand what you say. But this Privacy list would make much less snese if there are no services to avoid. It's a list of privacy services that serve as a replacement for existing tools which are harmful for users privacy, so they are needed to be listed there. I am sorry, but I am not removing the "avoids" from the list neither the description text for some sections.
The punctuation errors and improper use of some words/brands and also the Awesome badge problem can be fixed and will be fixed completelly when I have some free time this next week. Thanks for the feedback.
Also, there is a Contributing.md file and this file is linked on the readme header.

@pgray-hiro I understand what you say. But this Privacy list would make much less snese if there are no services to avoid. It's a list of privacy services that serve as a replacement for existing tools which are harmful for users privacy, so they are needed to be listed there. I am sorry, but I am not removing the "avoids" from the list neither the description text for some sections.
The punctuation errors and improper use of some words/brands and also the Awesome badge problem can be fixed and will be fixed completelly when I have some free time this next week. Thanks for the feedback.
Also, there is a Contributing.md file and this file is linked on the readme header.

The Contributing.md causes the linter to fail because it's not in the root directory of the repo.

I think it's important to add descriptions about why something should be avoided. You're not really convincing anyone without any reasoning or evidence.
For example:

Avoid: VSCO

I like the app. Unclear why I would want to avoid it.

Closing for lack of response.


Unit testing + bug fixing in GRPC Adapter

Description
Adding Mockito plugin for mocking final classes.
Increasing test coverage for ObjectSerializer
Fixing bug in GRPC Adapter while creating the envelopes, found during unit testing.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@artursouza can you please take another look at this PR?

@artursouza I already addressed your comments, plus added a little extra covera, please take another look


[9.x] Fluent has-x-through definitions.

The current hasOneThrough and hasManyThrough method definitions can be confusing, especially when having to modify the local & foreign keys, and—at least personally—has me reaching for the docs pretty much every time I need to use them.
For (an admittedly contrived) example, let's say I have photos and albums tables, and in my app I have decided to use photo_id and album_id as the respective primary keys, I would have to achieve that like so:
class User
{
    public function photos(): HasManyThrough
    {
        return $this->hasManyThrough(Photo::class, Album::class, 'photo_id', 'album_id', 'photo_id', 'album_id');
    }

    public function profilePhoto(): HasOneThrough
    {
        return $this
            ->hasOneThrough(Photo::class, Album::class, 'photo_id', 'album_id', 'photo_id', 'album_id')
            ->where('is_profile', true);
    }
}

This isn't super clear on what we're achieving or what any of the final 4 arguments are. Granted, some IDEs might add inlays, or I could use named arguments to clear this up, but the definition doesn't feel obvious from a glance.
I am proposing that we add a through method onto the HasOne and HasMany relation classes. This would allow a more fluent definition of the relationship and is a lot easier to understand what is happening:
class User
{
    public function photos(): HasManyThrough
    {
        return $this
            ->hasMany(Photo::class, 'photo_id', 'photo_id')
            ->through(Album::class, 'album_id', 'album_id');
    }

    public function profilePhoto(): HasOneThrough
    {
        return $this
            ->hasOne(Photo::class, 'photo_id', 'photo_id')
            ->through(Album::class, 'album_id', 'album_id')
            ->where('is_profile', true);
    }
}

--
Edit: I have only added a couple of tests so far, as I wanted to see what appetite there was for this — if we wanted I can add more tests.

Just realised my implementation might be slightly off, so I'll look into that. More tests would probably help confirm it.

So the old way of defining relationship would still work?

So the old way of defining relationship would still work?

Yup, nothing has changed in that respect. The old way would still be available, so no breaking change. Just a more fluent (and in my opinion, readable) alternative of defining them.

I actually find the specific of keys in this PR more confusing than just using the original method. For example, here are both side by side:

Note that in your PR you have to specify mechanic_id with the call to hasOne(Owner::class) - but mechanic_id is not on the owners table at all - it's on the cars table. Likewise, car_id isn't on the Car class, it's on the Owner class. Everything feels reversed.
In the "non-fluent" definition (current state of Laravel), you specify the keys in a logical order... joining table foreign key, final table foreign key, local key to be used on joining table, local key to be used on final table. It goes from joining to final both times.

I've proposed a slight alternative approach to this idea: https://github.com/laravel/framework/pull/45894


Pass all fetcher context via HttpAwareContext

CassandraFetcher and all of its subclasses receive authorizationService and dataStoreFactory via the constructor. This generates a very tedious refactoring if we need to add a new argument or remove an old one (there is no automated way to change all the constructors at once AFAICT).
Such context components should be passed via HttpAwareContext instead.
Incidentally, we should rename that class, it already handles more than HTTP-related stuff. CassandraFetcherContext feels appropriate.

@olim7t you can change all consturctors with refactoring in Intellij, right click on constructor Refactor -> Change signature. If you do it in a base class, it will be applied to all sub-classes as well.

I think I tried that, are you sure it applies to constructors as well?
But anyway, beyond "refactorability" I still think using the context is the better option.


Develop React/NodeJS template repo

💡 Description

This probably doesn't need to be as comprehensive as the Java/Python repos, but we should have something here to house some of our best practices and CI/CD stuff

How to build
How to test
Static code analysis requirements
Security scanning requirements prior to production release

Completed several weeks ago. https://github.com/NASA-PDS/template-repo-nodejs/

Thanks @jordanpadams! And if I may say so, it's fairly comprehensive too 😁


Traveler's bricks issues.

 First issue, with a jump boost item, if you jump wile running against the blocks, you actually get launched about 120 blocks into the air, depending on the jump boost level. The speed is ridiculous when mixed with speed boost items (Sojourner's Wyvern Armor, etc.). Also, because it's not a full bock or a slab, you get stuck inside of it alot. We had to teleport a few people because they were glitched into the air, and breaking the bricks didn't change anything.

Can you elaborate on the 'they were glitched into the air' and 'you get stuck inside of it a lot'? Screenshots would be ideal.

Well, there wasn't really anything we could do for screenshots, as each person saw the player as being above the blocks, like he was standing on a full block, but he was stuck there. We had to forcibly teleport him out of the 'prison' via op commands. This usually happened through the player running too fast back and forth along the road, mostly having a speed up item to mess with the speed. Also, I'm not sure if it's intended, but it also seems to exponentially propel items along the path, meaning if you toss a cobblestone along the path, it turns into the minecraft equivelent to a particle accelerator.


Turn ValidSidechains C Array into std::array

This PR fixes https://github.com/drivechain-project/bitcoin/issues/20

fixed spacing - ty for review.


Add competency questions

This PR adds 50 competency questions

41 can be inferred
9 cannot be inferred

The travis CI is adapted to check the inferences of the former category on every push and PR.

hopefully, you mean 41 can be inferred?

hopefully, you mean 41 can be inferred?

Yes, thanks! Changed it.


DT-384 - improve modal component to support form input

What was changed

add hasInput prop to modals - if true, wrap content and buttons in a <form /> and use on:submit
fix keyboard nav for modals with inputs and/or disabled buttons

Why?

better keyboard a11y

Checklist

Closes

How was this tested:

Any docs updates needed?

I considered breaking the buttons out of Modal into a new component - <ModalActions />, so that we could compose the form ourselves, like
<Modal>

Title

  <form slot="content">
    <Input />
    <ModalActions type="submit" />
  </form>
</Modal>

but that would introduce a fair amount of breaking changes.
@Alex-Tideman @laurakwhit @stevekinney
Thoughts?


Question re "Strip image sizes"

I am seeing this in my source code for posts:
<img width="768" height="432" src="/cdn-cgi/image/width=1920,height=1080,fit=crop,quality=80,format=auto,onerror=redirect,metadata=none/wp-content/uploads/2023/07/image-768x432.jpg" ....
My understanding was that the "Strip image sizes" setting would take care of this? Otherwise, the "resizing" function would only ever be working off the thumbnail/smaller version of the image, and not the full sized one (eg image.jpg).
Great plugin otherwise!

There are 2 possible reasons why this cannot work:

the config.php is not saved properly (CF_IMAGE_RESIZING_STRIP_SIZES)
the regex used in the plugin fails

I need more information to look into it. Regardless, the result of images are better and faster.

Actually I found your Cloudflare worker script and am now using that. Thanks!


Is backwardHover possible?

Simple, but great plugin!!
Is therea feature to scroll the items backward on hover?
Thanks.

You can add same lines as forwardHover, and in scrollBackward definition, last statement is nextScrollId = setTimeout(forward, config.delay * 1000); you just change forward to backward and thats it.

forwardHover is
    forwardHover = function() {
        config.autoPlay = true;
        paused = false;
        clearInterval(scrollingId);
        scrollingId = setInterval(scrollForward, config.speed);
    };

I defined backwardHover  as
    backwardHover = function() {
        config.autoPlay = true;
        paused = false;
        clearInterval(scrollingId);
        scrollingId = setInterval(scrollBackward, config.speed);
    };

backwardHover works fine but it stop and start forwardHover after one element

@dragan2234 Thanks you so much it works for me ;)


如何用我自己的图片来验证训练模型的结果

Read the dataloader part. One important thing is, you need to "tune" the images of the dataset in order to make it work as you might expect


Request C++17 by default, add fallback flag for now

Summary
There have been discussions about moving to C++17 for a while. Stan compiles fine under -std=c++17 since https://github.com/stan-dev/math/pull/2693, and C++17 is already used by RStan.
There so far hasn't been a push to actually use C++17 in our code base, and this is a proposal for the first step to do this:

We start requesting -std=c++17 in our Makefiles.
We provide a flag called STAN_USE_CPP14 to switch back to -std=c++1y.

That's it.
This is essentially designed to smoke out users who can't support C++17 while providing them with something they can put in make/local to continue compiling for a version or two, until we actually add some C++17-requiring source code, at which point we delete the opt-out.
I tried to find a more clever way to do this, but automatically detecting which standards a compiler supports seems to be a difficult/brittle thing to do.
Tests
None
Side Effects
Release notes
Stan now requests -std=c++17. C++14 can still be used by defining the make variable STAN_USE_CPP14. This variable will go away and C++17 will become a requirement in a future version.
Checklist

[x] Copyright holder: (fill in copyright holder information)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Should we put this in the 5.0 branch?

In my opinion no - the real breaking change would actually be when we remove the flag and start using C++17-only constructs in the code.
The utility of doing this beforehand would be letting users know if their environment is incompatible early, while we can still give them an escape hatch

@WardBrian do we want to merge this to include in this release?

@SteveBronder I think we will need another round of RCs for other changes, so we can!
Because I opened this PR I can't actually click 'approve', so if you can that would be great


feat(Usage Reports): Update query param for getting reports snapshotlist

PR summary

Add new query parameters to support pagination of API: /v1/billing-reports-snapshots
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.44.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added Neuralmind to frontend

Title and Issue number

Hey @pranjay-poddar I have added neuralmind project to the frontend. Please review my pr and merge it. Thank you!
Close #1057
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md

@pranjay-poddar I had already created the project previously, I only had to add it.

Thanks for approving.


dotnet-helix-machines-image-generation broken

This has been broken for almost 1month.
It seems to be another authentication case: https://dev.azure.com/dnceng/internal/_build/results?buildId=2266787&view=logs&j=3dc8fd7e-4368-5a92-293e-d53cefc8c4b3&t=45862c6e-83c5-515f-73e2-e7009eff9f9b

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

It is an auth problem but only since the 7th (when the key expired). There may be other problems, of course.


Astra DB, LLM cache classes (exact-match and semantic cache)

This PR provides idiomatic implementations for the exact-match and the semantic LLM caches using Astra DB as backend through the database's HTTP JSON API. These caches require the astrapy library as dependency.
Comes with integration tests and example usage in the llm_cache.ipynb in the docs.
@baskaryan this is the Astra DB counterpart for the Cassandra classes you merged some time ago, tagging you for your familiarity with the topic. Thank you!

again looks great, thanks @hemidactylus!


Investigate Issues with Reachability Not Restored

Getting a number of reports about reachability issues not being restored properly.

I REALLY wish Apple would put together a real reachability framework with a thread-safe objective-c front-end.  Until that day, SCReachabilityKit on GitHub works pretty great. There's also Andrew Donoho's more recent revisions to Apple's reachability class.
For what it's worth, I went with the former since it fit my project case better. I wrote a layer on top of it that looks primarily at known hosts, suitable for restkit type usage, and for any other unmonitored hosts, it just looks at general network reachability instead of doing another DNS lookup, etc.
The kicker is there's only one time where it uses the synchronous getFlags: method, on the initial watch. Everything else is asynchronous, to reduce blocking.
I can discuss more if needed, but if you find the SunlightLabs/StatesLege project, look in Classes/Helpers/SLFReachable.m
Don't bother trying to build the project though, not all submodules are open to the public yet.
G
Sent from my iPhone.
On Oct 14, 2011, at 1:47 PM, Blake<EMAIL_ADDRESS>wrote:

Getting a number of reports about reachability issues not being restored properly.
--
Reply to this email directly or view it on GitHub:
https://github.com/RestKit/RestKit/issues/408

Two big things that I'll bring up in the conference call today:

We wind up calling SCNetworkReachabilityGetFlags() a whole lot ... every time we check networkStatus or isNetworkReachable or isConnectionRequired.  That's a synchronous call that can block the thread up to thirty-seconds every time we use it.  What I discovered from using SCNetworkReachabilityKit is that we can get all the same utility out of reachability if we just catch those same flags passively from the userInfo dictionary (that kit handles all of that part).  But there should only be one event that prompts you to call the synchronous library functions, and that's when you're just starting to monitor a host.

This restructuring looks good to me.


fix: update devcontainer for azd feature #239

fix #239

Hi @crazy4pi314. Thanks for the PR draft. Do you have a status update?

Actually it was fixed with #384


inline-verify --verifications-out creates a file but it has no content

When pgpainless-cli inline-verify has a --verifications-out argument, it creates the file, but doesn't write the expected verification string in it:
pgpainless-cli generate-key test > test.key
pgpainless-cli extract-cert < test.key > test.cert
echo test> test.txt
rm -f verifications
pgpainless-cli inline-sign test.key < test.txt > test.txt.pgp
pgpainless-cli inline-verify --verifications-out verifications test.cert < test.txt.pgp
ls -l verifications

What i'm seeing is that verifications is a zero-byte file.  It should have a line of verification info in it.
I still don't have a good sense of which project to open bugs in, so feel free to port this issue if i've got it in the wrong place.
version info
$ pgpainless-cli version --extended
PGPainless-SOP 1.3.7
https://codeberg.org/PGPainless/pgpainless/src/branch/master/pgpainless-sop

Implementation of the Stateless OpenPGP Protocol Version 04
https://datatracker.ietf.org/doc/html/draft-dkg-openpgp-stateless-cli-04

Based on pgpainless-core 1.3.7
https://pgpainless.org

Using Bouncy Castle 1.71
https://www.bouncycastle.org/java.html

Oh boy, pgpainless-cli is more broken than I thought :O
Apparently my idea of using the interoperability test suite as a test suite for pgpainless-cli was not sufficient :D
I will look into it and try to get a fix out asap!

Annoyingly this is simply just a writer that was not flushed :angry:. Luckily the fix is easy.

Fixed with 8494f413e63e01614c0a381ec9c6fb0821793f84


Searchable tags

You should be able to list and search tags
Additionally, suggesting tags may be a useful feature

PR #141


Kahan 2x2 determinant computation reduces numerical imprecision.

Related to issue https://github.com/GEOSX/GEOSX/issues/1440: some tests testTensorOpsInverseTwoArgs and testTensorOpsInverseOneArg are failing due to large numerical imprecision.
Using the Kahan method to compute 2x2 determinants brings more precision.
3x3 case still untouched.
Warning, I sometimes get compilation warnings like
../coreComponents/LvArray/unitTests/../src/fixedSizeSquareMatrixOpsImpl.hpp(100): warning: calling a constexpr __host__ function("fma") from a __host__ __device__ function("determinant") is not allowed. The experimental flag '--expt-relaxed-constexpr' can be used to allow this.

which I find surprising since this fma function is supposed to be device friendly.
fma also seems pretty standard so one should not need any fallback.
So I need to double-check.
Any opinion?

Also this modification makes calculating the determinant significantly more costly. I'm not sure you can answer this question but maybe the kind of matrices that pop up in the unit tests aren't representative of the normal use case.

I could spot 3x3 dets in the code (FEM stuff), but only 2x2 in the unit tests.
I will double check.


Initialize project with components

Created the project with create-react-app
Added React Redux
Created Folder/Files structure using ducks pattern for Redux files
Added React Router for Books (the default view) and for Categories
Styled a little bit to be able to see better the elements (even though wasn't required)

Thanks for the response @SpaYco


Updated Global Header / Footer Elements + Global CSS Path

 You should sign our Contributor License Agreement in order to get your pull request merged.

@antiheld could you please merge with the latest of develop


syncCommand?

I'd like to use node_redis to set up a sync client and parse the incoming representation of the command and args into strings/json. Is this possible?
Thanks!

This is not possible with node_redis. That command is actually only meant to be used internally in Redis not not that a client uses that command. And neither the hiredis parser, nor the js parser is able to handle the response of the sync command.


Using IPP: Unable to get printer status

Hello,
I cannot print using IPP with my new Canon G620 printer, "Unable to get printer status" occurs when sending a print job.
My other printers works just great using IPP, other printers are mainly Brother and Pantum.
I was able to successfully add the Canon printer to CUPS running remotely, port forwarding are in place to forward the traffic back to the printer on the local network.
If I'm adding the printer without passing by the remote CUPS service, meant using the local CUPS running from Linux Mint, by entering it's local IP, I can send every print jobs that I sent to the printer.
It's working locally but not remotely.
Thanks,
Guillaume

Hello,
I enable the debug logging, and the error_log is attached to this message.
error_log.txt
It is quite long, this is the part displaying the error message:
error_log2.txt
Also, when printing from my Android phone, on the same network just to try to see what will happen, the phone display these two notifications:

Tapping Accept make the print to go through. It is possible that the error when printing remotely is related to a certificate?
Thanks,
Guillaume

It could be  - do you know if Android connects to your print server, or to your printer? I recall there was an issue about certificates in the past, so it might be related.
Either way, even the longer debug does not show where IPP backend on your client connects to, so we're not able to tell if you do connect to the server or to the printer. At least start of the job would be great to see.
And IIUC you have client - server - printer environment, so the corresponding debug logs from cups on the server help a great deal, if you talk with server.

It could be - do you know if Android connects to your print server, or to your printer? I recall there was an issue about certificates in the past, so it might be related.

Yes, in the screenshot, the Android phone was connecting directly to the printer, I made it that way just to see what will the result.
For the certificate, is there anything I should do with that part, what do you recommend me?

Either way, even the longer debug does not show where IPP backend on your client connects to, so we're not able to tell if you do connect to the server or to the printer. At least start of the job would be great to see.

I'm not sure but we can see the error message stated above in the logs. The G620 seem to attempting to connect.

And IIUC you have client - server - printer environment, so the corresponding debug logs from cups on the server help a great deal, if you talk with server.

Yes, the logs are the one from the server running cups. Here is thr topology:
Workstation is sending print job -> CUPS running in my server room is getting the job (network is isolated, only electrical is common as it is on the same property but in a different structure) -> Printer (via port forwarding)
NB. Printers are spreaded across my three locations.
NB. That current topology works great since years with Brother and Pantum printers. My Canon MX492 injet also work with that topology.
Thanks
Guillaume

For the certificate, is there anything I should do with that part, what do you recommend me?

I'm not sure how printing exactly works on Android, but AFAIK in Linux CUPS generates DBUS event about certificate changed, which should be picked up by desktop environment and generate notification about certificate has changed - and if user confirms it is okay, application should save the new certificate, so printing should continue after that. Do I get it right, @michaelrsweet ?
So if you see this every time you print, your printer either changes cert, or Android does not save it, or there is something in the middle.

I'm not sure but we can see the error message stated above in the logs. The G620 seem to attempting to connect.

AFAIK the printer does not initiate the communication, so printer does not attempt to connect to cupsd or IPP backend - IPP backend on the server initiates the connection and I don't see URI what it tries to connect to - the only job related logs are those about getting supported attributes failing with bad-request, which results into error you see.
Can you upload .txt file with logs so I can see what the backend connects to? You can provide the part of the logs where the job starts and then gets several first error messages.
On the top of that, please upload unencrypted traffic which happened between your server and the printer, so we can see we don't send something weird to the printer.

Workstation is sending print job -> CUPS running in my server room is getting the job (network is isolated, only electrical is common as it is on the same property but in a different structure) -> Printer (via port forwarding)

I'm curious about this, since I haven't heard about someone doing it for printers - so you implement port forwarding on your server via iptables, so you install the printer with different hostname/IP on the server, but when communicating with printer, iptables translates the hostname/IP to the correct hostname/IP where the printer listens to?
Or is there is another network member doing the translation?

NB. That current topology works great since years with Brother and Pantum printers. My Canon MX492 injet also work with that topology.

As Mike mentioned, it can be a printer specific.

@zdonal WRT certificates, you are correct. However, none of the logged requests are encrypted.

Hello,
I think that to something, I will add a locally hosted CUPS server that the remote CUPS will send jobs to. This local CUPS instance will send the job to the printer so the printer will technically receive the job from another host on the network, instead directly from another host from the internet, so if it was an issue caused because the job is originating from a different hostname, it should theoretically stop showing the error and start printing?
Will see...
Guillaume

It works with a local CUPS in between...
So, it might be related to the hostname mismatch. 😞

I'm sorry, I did not do port forwarding with printers, so I don't know of other ways how to bypass it (certainly not by CUPS, since the hostname change happens below it and the error happens on the printer - and IIUC for good reason as protection from DNS-rebinding attack), so I would only recommend something you've proven it works - connecting to remote print server which will have the correct printer hostnames...


chore: Refactor chat completion creation to a separate method

This allows to:

simplify the chat method
customize OpenAI completion arguments in subclasses.

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


feat: Card implementation.

fixes #18

Pull Request Test Coverage Report for Build 63

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 81.754%

Totals

Change from base Build 54:
0.1%

Covered Lines:
441

Relevant Lines:
528

💛  - Coveralls


Add server versioning proposal

I have not heard of a defect release before, but fix is common with breaking.feature.fix from Convential Commits, Semantic Release, and the prior art those are based on
I think any of the options that stray from SemVer are not a good idea.  SemVer is heavily used in automation tools like Dependabot and such.  While it might be nice to have a 4th option, such as release.breaking.feature.fix, there is just not much support for parsing and comparing those so it is guaranteed to break automation.  Hijacking PreRelease tags or BuildInfo to indicate that this is a different, compatible Stable release would not be properly handled by automation either.
There is, of course the option of versioning JS APIs within the Server.  K8s APIs are one example and applied to NATS could look something like core/v1, stream/v1, kv/v1beta1, and obj/v1alpha1.  This would be a large undertaking for the server, however.
Another option could be to simply keep versioning things as they are, but document that minor releases in the nats-server may have major breaking changes in JetStream APIs, so read the release notes on those.


ARM and ARM64 docker containers

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
With most cloud providers now having ARM/ARM64 based machines it would be helpful to have the Docker containers of Pulumi of said architecture to be able to run on those machines (Ideally with a multi-arch container image so the same image will run on all architectures)

@naorlivne Good news!  We just started publishing multi-arch (AMD64 and ARM64) builds for most of our Docker images!  We do not currently have an ARM64 build for the pulumi/pulumi image, but we don't recommend this image for production use because the image is so large (it contains every language/platform Pulumi supports).  Our images are all published to both Docker Hub and AWS Elastic Container Registry.
Does this meet your requirements?

Hi @jkodroff, my hope was creating a generic Drone plugin to run pulumi out off to support all 3 (AMD/ARM/ARM64) & seeing how I wouldn't know which language the enduser would use I was planning as using pulumi/pulumi as the base image.
So basically there's 2 requests here:

add multi-arch to pulumi/pulumi image
add ARM to multi-arch images

@naorlivne

Adding multi-arch to pulumi/pulumi is gonna be tough (but not impossible) due to the large number of packages in the image, some of which are tricky to install.  (You may notice that some of our multi-arch single-SDK images use e.g. python3.9:slim as a base image to avoid this pain.)
We'll definitely take this request under consideration for our future planning.  In the meanwhile, we would welcome a PR.

I'm not exactly sure what the ARM platform is, although I see it available on images in Docker Hub like Debian.  Is it equivalent to linux/arm/v7or something like that?  (I'm not as familiar with the particulars of platform strings as I would like.)  We're happy to take this one under consideration like #1 above, but of course the difficulty of the work depends on the availability of our dependencies on the target platform.  If we add this to our existing Debian-based, single-SDK images, is this helpful to you?

I'm not familiar with Drone.  Could you send me a link?  Always interested in learning more about how our Docker images are being used in the community.

Thanks!

@jkodroff

I understand completely
Basically yes, ARM is a 32bit architecture versus 64bit used in ARM64, because ARM is often used in IoT devices it's still a common architecture of it.
https://www.drone.io/ - It's my favorite CI/CD tool, the reason for this request is I've created a plugin for it to be able to use it to run Pulumi to automate IaC, I plan on raising a request to have it added to https://www.pulumi.com/docs/guides/continuous-delivery/ just as soon as I finish adding some smoke-tests to it & figure out to raise a PR to Pulumi docs


Feature/fb send audio

#270

Coverage decreased (-0.1%) to 93.923% when pulling 00e5123a5941189df780069cf6493ba75c967083 on feature/fb-send-audio into fcf9af54a93953effb2d0dadc47ecd1ac41cb418 on develop.


My VS code suddenly disappeared from my laptop.

I was using it frequently. After restarting the system as I do generally, I was unable to find VS Code. Kindly help.

what platform are you on? (mac, windows, etc)


Test against multiple jdks, improve build time

Use default container build for faster build start time.
Install latest node js version using the node version manager.
Cache npm and maven dependencies to reduce wait time on network calls.

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Coverage remained the same at 38.761% when pulling 3af7d9ef0aa5e1c92c7a81cc5c7afe228209611a on ChristianMurphy:update-travis-ci into b34385041b5fb5802b01bd37b1b6bba70ae46707 on UW-Madison-DoIT:master.


docs: describe Tracer#in_span arguments

These parameter descriptions are based on comments in the OpenTelemetry::SDK::Trace::SpanData Struct.
I came across this when I recently needed to add :kind to a Tracer#in_span call and the current RubyDoc didn't describe how to do so.

It looks like Rubocop is upset about ./ included in the file arguments for require_relative in the Semantic Conventions code. I made a new PR to address those: https://github.com/open-telemetry/opentelemetry-ruby/pull/1475
The other error is related to TruffleRuby and seems to also happen for the Metrics WIP PRs. Would you like me to make a bug report for this issue?

The other error is related to TruffleRuby and seems to also happen for the Metrics WIP PRs. Would you like me to make a bug report for this issue?

That would be appreciated!

The other error is related to TruffleRuby and seems to also happen for the Metrics WIP PRs. Would you like me to make a bug report for this issue?

That would be appreciated!

Great! Here's the bug report: https://github.com/open-telemetry/opentelemetry-ruby/issues/1476


new dev

@hariyebk Appologies got the below inconvenience but we had to make huge change and we request every PR to follow the below rule
https://github.com/chapimenge3/codenight-developers/blob/main/CONTRIBUTING.md#making-changes
Please review and update the PR accordingly


Is it possible to add support for MSSQL?

Hello, I know we can connect to mysql and postgresql but is it possible to add support for MSSQL?
Thank you so much for your hard work on this extension! ⭐

Hi, It's possible to support for MSSQL.
It's going to take a little while, so just wait and see.

Thank you so much!

I've changed to be able to use SQL Server in Ver 0.4.0.
Please try it after upgrading.

It does connect, and I can run queries to SQLServer, however I just wanted to report a few things that might be improved at some point.
Here is how it looks for me. This may be because I use an Integrated windows authentication to connect to SQLServer, but in this case I created a login just for DB Notebook.

Then here is the connection settings:

And to test it, I had to write fully qualified FROM statement like this:

This fully qualified FROM statement returns results as expected, but it would be nice if I didn't have to write the full path.
But I really appreciate you adding this feature. I'm still learning SQL but SQLServer is what they use at work. I'm hoping to get a job that pays more. I'm just a clerk at the office and I hope to get into IT using SQL. So I'm teaching myself.
Anyway, you helped me and when I get a better job, I owe you a coffee, so let me know if you have one of those "buymeacoffee" features on your github.
If you think I should setup SQLServer differently let me know. Maybe there is another way I can set it up so that I don't have to type in full paths in the FROM statement.
Thank you.

I will wait to see if you have more things you want me to try when creating the SQLServer login. If you don't have any suggestions, it's ok to close this thread. Thanks again.

UPDATE
I unchecked the "Show only default schema" and now I can browse the database and see the different tables. I still have to enter a fully qualified path to return results. I even tried "dbo.Person" but that didn't work. Only Person.Person or Person.Address or HumanResources.Employee for example will work.

@lundeen-bryan
Person.Person's first is the schema name, right?
I think the problem of not being able to SELECT by only table name that is a matter of specifying the default-schema for the connecting user.
As an example of my environment, access to the DEPT table that I have in the default schema of the connecting user does not require schema modification.
In contrast, the CUSTOMER table is owned by schema0, so access to it requires schema0 qualification.

DB: testdb

SCHEMA: schema0 ← My Default Schema!!

TABLE: customer

SCHEMA: schema1

TABLE: DEPT

Sample of creating a user by specifying a default schema

https://github.com/l-v-yonsama/db-drivers/blob/main/__tests__/setup/mssql.ts#L88-L121

I don't know any clerk in the office who can write SQL statements.
That's great!
I'm sure you'll become an IT engineer, so good luck.

Thank you so much. Yes, I am basically hired for administrative support, but I made friends with one of the programmers and he let me help out on SQL projects when he doesn't have time. I really appreciate the hard work you put into this Extension.
The problem may be the way I created a user in SSMS. So I will compare your screenshots and try to figure out the difference.

I really appreciate your work on this. I had a problem opening a SQL Server database and got the error "STRING_AGG" does not exist. Sorry I will write a better description when I get a chance to look at it on my work computer. But I wanted to keep this ticket open until I had a better chance to give you more details. I will do it on Monday.

Both MySQL and SQLServer use postgresql's for parsing SQL statements. (I should really change it for each DB product I use, but I don't have time, so I use postgresql's.)
SQL is programmed to be rewritten to match the Postgres syntax before parsing, but not all of it is supported. In this case, I assumed the error was in the WITHIN GROUP clause of "STRING_AGG".
And we have addressed that part of the problem, so please try to fix it when you have time.
Where to upload the error log?
Is it this Issue?
What is everyone else doing about it?
My personal Email?

Ok I think I understand. It only happens because postresqul doesn't use WITHIN GROUP clause. No problem, I will try to work around this. I appreciate you addressing this so quickly!
At some point I will learn typescript and maybe I can help people like you when you are loaded with work. I just want to emphasize I appreciate you very much.

Microsoft SQL Server support tested and works with only some features not working such as using "WITHIN GROUP" clause which is not supported in postgressql - the basis for most sql transactions.


Add Example Code for Multiple Channel Scan Mode SAADC of nrf52

The example code for the SAADC of nrf52 is a single channel one-shot mode which is not very helpful in practical use. For most of the projects, people need more than just one channel for their ADC tasks. In the ADC example or the API provided, there is less information on how to configure the ADC to Scan mode and retrive the results from multiple channels.
Please provide an example code for at least two channels in scan mode.
Thanks,

Any one of the available channels can be enabled for the ADC to operate in one-shot mode. If more than one CH[n] is configured, the ADC enters scan mode.


Benchmark pebble.NoSync vs pebble.Sync

Currently we use pebble.Sync, meaning that our db operations are immediately synced to the disk. NoSync can provide performance improvements.
// WriteOptions hold the optional per-query parameters for Set and Delete
// operations.
//
// Like Options, a nil *WriteOptions is valid and means to use the default
// values.
type WriteOptions struct {
	// Sync is whether to sync writes through the OS buffer cache and down onto
	// the actual disk, if applicable. Setting Sync is required for durability of
	// individual write operations but can result in slower writes.
	//
	// If false, and the process or machine crashes, then a recent write may be
	// lost. This is due to the recently written data being buffered inside the
	// process running Pebble. This differs from the semantics of a write system
	// call in which the data is buffered in the OS buffer cache and would thus
	// survive a process crash.
	//
	// The default value is true.
	Sync bool
}

// Sync specifies the default write options for writes which synchronize to
// disk.
var Sync = &WriteOptions{Sync: true}

// NoSync specifies the default write options for writes which do not
// synchronize to disk.
var NoSync = &WriteOptions{Sync: false}

Gave this a try and it looks like pebble.NoSync performs worse. Closing.


Shape inference bug

Describe the bug
I'd like to test some object detection model on the latest runtime when it runs into a shape inference exception with the message:
RuntimeError: [ONNXRuntimeError] : 1 : GENERAL ERROR : [TypeInferenceError] Graph attribute inferencing failed: [ShapeInferenceError] Incompatible dimensions.
If I change back to runtime 0.3.0, everything will be fine.
You can find the model here: model. It's a part of Faster-RCNN model.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
ONNX Runtime installed from (source or binary): source or from https://test.pypi.org/project/ort-nightly/
ONNX Runtime version: 0.3.1
Python version: 3.6
Visual Studio version (if applicable):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

To Reproduce
Describe steps/code to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

For any shape inference error, it's either a converter bug, or an ONNX bug. It's unlikely caused by onnxruntime code (unless the TypeInferenceError was thrown from a custom op node)
Do you know which op caused this error?

The issue doesn't look like it's an Onnx issue. The inference error was in a 'Div' op (which is confusing since there is no 'Div' in the graph). When I disable the graph transform 'EliminateSlice' it works without error.
Adding Ashwini since she knows about transforms and helped me to narrow it down.

Konstantinos is already looking into this

@lucienwang1009 you can turn off the transformers by setting the level to 0. Which API are you using?
CAPI:     OrtSetSessionGraphOptimizationLevel(session_options, 0);
c# api:  options.SetSessionGraphOptimizationLevel(0);
python: options.set_graph_optimization_level(0)

From an initial investigation, Slice Elimination is indeed causing the problem. As @askhade says, you can bypass this by setting the optimization level to 0.
Interestingly, the problem is not caused by the "visible" Slice in the top-level graph (this cannot be eliminated actually), but by a Slice that appears in a 74-node sub-graph (if I disable elimination in the subgraphs, it works). I will debug it later today.

PR https://github.com/Microsoft/onnxruntime/pull/918 fixes the issue.

Issue fixed with #918.


model = keras.models.Sequential() # I tensorflow/stream_executor/cuda/cuda_driver.cc:801] failed to allocate 72.19M (75694080 bytes) from device: CUDA_ERROR_OUT_OF_MEMORY: out of memory

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):pip
TensorFlow version (use command below):2.1.0
Python version:3.7.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:10.1.0
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
building an image classifier
n_rows = 4
n_cols = 10
plt.figure(figsize=(n_cols * 1.2, n_rows * 1.2))
for row in range(n_rows):
for col in range(n_cols):
index = n_cols * row + col
plt.subplot(n_rows, n_cols, index + 1)
plt.imshow(X_train[index], cmap="binary", interpolation="nearest")
plt.axis('off')
plt.title(class_names[y_train[index]], fontsize=12)
plt.subplots_adjust(wspace=0.2, hspace=0.5)
save_fig('fashion_mnist_plot', tight_layout=False)
plt.show()
model = keras.models.Sequential()
model.add(keras.layers.Flatten(input_shape=[28, 28]))
model.add(keras.layers.Dense(300, activation="relu"))
model.add(keras.layers.Dense(100, activation="relu"))
model.add(keras.layers.Dense(10, activation="softmax"))
Describe the expected behavior
model.layers
<tensorflow.python.keras.layers.core.Flatten at 0x7ff370af5780>,
<tensorflow.python.keras.layers.core.Dense at 0x7ff370af5c88>,
<tensorflow.python.keras.layers.core.Dense at 0x7ff330ab36d8>,
<tensorflow.python.keras.layers.core.Dense at 0x7ff330ab3828>]
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem.
If possible, please share a link to Colab/Jupyter/any notebook.
https://colab.research.google.com/github/ageron/handson-ml2/blob/master/10_neural_nets_with_keras.ipynb#scrollTo=KQiHD3LwTRDF
Other info / logs
2020-05-28 13:40:44.785958: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1241] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 72 MB memory) -> physical GPU (device: 0, name: GeForce RTX 2060 SUPER, pci bus id: 0000:29:00.0, compute capability: 7.5)
2020-05-28 13:40:44.787561: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x55ed4ff1eab0 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-05-28 13:40:44.787571: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): GeForce RTX 2060 SUPER, Compute Capability 7.5
2020-05-28 13:40:44.789256: I tensorflow/stream_executor/cuda/cuda_driver.cc:801] failed to allocate 72.19M (75694080 bytes) from device: CUDA_ERROR_OUT_OF_MEMORY: out of memory
2020-05-28 13:40:44.796521: F ./tensorflow/core/kernels/random_op_gpu.h:232] Non-OK-status: GpuLaunchKernel(FillPhiloxRandomKernelLaunch, num_blocks, block_size, 0, d.stream(), gen, data, size, dist) status: Internal: out of memory
Aborted (core dumped)

@SarahLynnePu
Please share simple standalone code to replicate the issue faced or share a colab gist for us to analyse.


DEFAULT_VIEWPORT_WIDTH too narrow for non tty gitlab runners

What is your Test Scenario?
The problem occurs when testcafe is run by non-interactive runner. E.g. gitlab docker-machine runner. By default this runner runs its jobs in non-interactive mode(in other words non-tty). That is why this line fails:
get-viewport-width.js:
if (outStream === process.stdout && tty.isatty(1)) {
And DEFAULT_VIEWPORT_WIDTH is set. As a result all the formatted lines are cut to 78 chars. Which is very limiting. In my case this constant limits length of SELECTOR shown in exceptions to 1/2 of full length. That makes my debugging more complicated. I am suggesting an override to this constant so I am able to see whole selector.
What are you suggesting?
I am suggesting to be able to override this default value by env variable. It could be also placed in some env file if there is some?
What alternatives have you considered?
For non-tty runners there is no alternative then to switch to tty-runners.  That means register gitlab runner (docker-executor) as interactive.
Additional context
Whole proposed change:
get-viewport-width.js:
import tty from 'tty';

const DEFAULT_VIEWPORT_WIDTH = process.env.DEFAULT_VIEWPORT_WIDTH || 78;

export default function (outStream) {
    if (outStream === process.stdout && tty.isatty(1)) {
        const detectedViewportWidth = process.stdout.getWindowSize ?
            process.stdout.getWindowSize(1)[0] :
            tty.getWindowSize()[1];

        return Math.max(detectedViewportWidth, DEFAULT_VIEWPORT_WIDTH);
    }

    return DEFAULT_VIEWPORT_WIDTH;
}

Thank you for your suggestion. We are not sure if environment variables should be used for this purpose. Please give us additional time to research this.

Yes please! The output from our Jenkins is miserably narrow.
As far as I know, it is impossible start a TTY with a set width/columns. Just a quick Google search shows that trying to control this in a OS-agnostic way is a fools errand.
So having some way to override this DEFAULT_VIEWPORT_WIDTH value for CI servers would be a life-saver. Although I'd name it TC_DEFAULT_VIEWPORT_WIDTH to prevent any confusion as to what it's for.

Thank you for your suggestion. We would greatly appreciate your help. You are welcome to open a PR.

You are welcome to open a PR.

Done

Thank you, @PayBas, we will take a look!


Download ipadic only when it doen't exist

Thank you for great library! I'm using lindera for Plume, a blog engine.
I want to reduce time to waiting for downloading ipadic on every build during development. Can you consider this patch?

@KitaitiMakoto Thanks for your contribution! :)

@KitaitiMakoto Thanks for your contribution! :)

Thank you for merging.

Thank you for merging.


Unexpected error: "uninitialized constant Fluent::Plugin::TailInput"

In fluentd 0.14.8, the following error occurred.
2016-11-15 22:01:52 +0900 [error]: unexpected error error="uninitialized constant Fluent::TailInput"

It seems that Fluent::TailInputis not defined.
I added "require 'fluent / plugin / in_tail'" to in_sendmail.rb, but it was in the same state
thanks

I fixed it by editing like below.
git diff
diff --git a/lib/fluent/plugin/in_sendmail.rb b/lib/fluent/plugin/in_sendmail.rb
index 4bac3c3..1f59225 100644
--- a/lib/fluent/plugin/in_sendmail.rb
+++ b/lib/fluent/plugin/in_sendmail.rb
@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
-class Fluent::SendmailInput < Fluent::TailInput
+class Fluent::SendmailInput < Fluent::Plugin::TailInput
   Fluent::Plugin.register_input("sendmail", self)

   config_param :lrucache_size, :integer, :default => (1024*1024)

Is this a compatibility problem of fluentd v0.14?


aarch64 GDB step debugging fails with "Remote 'g' packet reply is too long"

Has anyone ever made this work?
68af229490fc811aebddf68b3e2e09e63a5fa475, gdb 7.11.1 built by Buildroot
An extremely similar aarch64 setup works for QEMU, and x86 and arm both work for gem5, just aarch64 does not.
gem5 command:
M5_PATH='/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/gem5-1.0/system'  '/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/gem5-1.0/gem5/build/ARM/gem5.opt'  '/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/gem5-1.0/gem5/configs/example/fs.py' --checkpoint-dir='./m5out/cpts/aarch64' --disk-image='/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/images/rootfs.ext2' --kernel='/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/linux-custom/vmlinux' --mem-size=256MB --num-cpus='1'  --command-line='earlyprintk=pl011,0x1c090000 console=ttyAMA0 lpj=19988480 rw loglevel=8 mem=256MB root=/dev/sda nokaslr norandmaps printk.devkmsg=on printk.time=y' --dtb-file='/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/gem5-1.0/gem5/system/arm/dt/armv8_gem5_v1_1cpu.dtb' --machine-type=VExpress_GEM5_V1

gdb command:
/home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/host/usr/bin/aarch64-linux-gdb  -q -ex 'add-auto-load-safe-path /home/ciro/bak/git/linux-kernel-module-cheat/buildroot/output.aarch64~/build/linux-custom' -ex 'file vmlinux' -ex 'target remote localhost:7000'  -ex 'continue' -ex 'lx-symbols ../kernel_module-1.0/'

gem5 stdout says:
4107766500: system.remote_gdb: remote gdb attached
warn: Couldn't read data from debugger.
4107767500: system.remote_gdb: remote gdb detached

Error message:
Reading symbols from vmlinux...done.
Remote debugging using localhost:7000
Remote 'g' packet reply is too long:<PHONE_NUMBER>00000090a4f90fc0ffffff4875450ec0ffffff01000000000000000100000000000000000000000000000001000000000000000000000000000000ffffffffffffffff646d60616b64fffe7f7f7f7f7f7f7f7f0101010101010101300000000000000000000000ffffffff48454422207d2c2017162f21262820160100000000000000070000000000000001000000000000004075450ec0ffffffc073450ec0ffffff82080000000000004075450ec0ffffff8060f90fc0ffffffc073450ec0fffffff040900880ffffff40ab400ec0ffffff586d900880ffffff0068a20ec0ffffff903b010880ffffffc8ff210880ffffff903b010880ffffffccff210880ffffff050000200000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000

Tried to add set tdesc filename aarch64.xml as mentioned at: https://stackoverflow.com/questions/27411621/remote-g-packet-reply-is-too-long-aarch64-arm64/27430836#27430836 but no change.
This is the exact reproduction setup: https://github.com/cirosantilli/linux-kernel-module-cheat/tree/a1a6a5d6fb417238ce4064ef7b3319686f02fd6b#gem5-gdb-step-debug-kernel-aarch64

I am currently closing all issues on github and disabling the issue support.
If you have general gem5 questions, please use the gem5-users mailing list (http://www.gem5.org/mailman/listinfo/gem5-users).
Please report any issues/bugs to the gem5-dev mailing list (http://www.gem5.org/mailman/listinfo/gem5-dev). However, gem5 is an open source project, so you are encoraged to fix any bugs/issues you find an submit a review request on our review site: https://gem5-review.googlesource.com
More details can be found in the CONTRIBUTING file: https://gem5.googlesource.com/public/gem5/+/master/CONTRIBUTING.md


Review changes related to validator set

At the moment, we are hijacking the validator manager's update function to return a static validator set. We should update the code so that we bootstrap using a static validator set from the genesis information instead, and then simply disable the updating of the set subsequently.
This would allow us to re-enable the genesis bootstrapping testing of validators, and would integrate the Flare logic more cleanly into Avalanche.

The latest version already improves on this, by instead hijacking the validator manager at the level of getting the validators. It bypasses the call to the chain state to get the list of validators, and instead returns a static set initialized on start-up. We can probably improve on this further, though.


Alert profiles still not populating management zones

Issue still seems to exist with 1.4.12. Neither updating an existing configuration nor adding a new configuration applies the management zone to the alert profile. Export shows the management zone with proper ID in SaaS but never gets applied to the configuration.

I'm sorry about that
Did you re-run "Execute OneTopology & TerraComposer" using the new version?
To re-create the tf files with the new code change
And then in the alerting module, in your .tf files, do you see something like this?
management_zone = "${var.dynatrace_management_zone_v2_MZ_NAME.value.id}"

Yes. Re ran the whole extract including execute OneTopology and terracomposer. Still failed to include the management zone.

I would need to see the logs and your .tf files to understand better why it doesn't work
Just to make sure, you are running on either linux or windows, right?
I will soon update darwin (Apple Silicon), but it is very out of date v1.3.1
Can you open a support ticket with your files please?
from the tool:
data/work/#-#/

history/complete//*.log
config/modules/alerting/aaa.tf  <-- the alerting profile that is missing a management zone variable
config/modules/management_zone_v2/bbb.tf   <--- the management zone that you expected to see in the alerting profile

Make sure you name me, David Cryans, so they assign the ticket to me directly
Thank you!

Fixed temporarily in v1.4.14
Waiting for real fix from the Setting 2.0 Devs
https://github.com/Dynatrace/Dynatrace-Config-Manager/releases/latest

This is now completely fixed


Does ArangoSearch support custom analyzer

Dose ArangoSearch support custom analyzer like elasticsearch ?
The analyzer would be better if it could be written in JavaScript

Hi,
YES! with ArangoDB 3.7 we added support for custom analyzers.

Thank you. I have been waiting for this feature for a long time


Detection of Smart View Mirroring on TV

Has anyone had any luck with this? I ask because my device is being constantly update in the graph/device link of the SmartThings API but it still is forever OFFLINE. I know not why.
When I Smart Mirror, the listing in Home Assistant just shows up as my last user HDMI source (even if I come from the YouTube app to Screen Mirroring, it will show up the last used HDMI source). This suggests to me that it's an App that doesn't appear in the App List.

It may not be an app at all, it may be a Tizen OS feature. Regardless, if you go in the SmartThings API Web Panel (where u created the API key) and there is no relevant information u can find in there that can identify if Smart View Mirroring is active or not, then there's nothing we can do about it.. This component can only interpret the data that is available in the SmartThings API (or the local APIs, but I'm pretty sure none of the local APIs has any info about this)

It may not be an app at all, it may be a Tizen OS feature. Regardless, if you go in the SmartThings API Web Panel (where u created the API key) and there is no relevant information u can find in there that can identify if Smart View Mirroring is active or not, then there's nothing we can do about it.. This component can only interpret the data that is available in the SmartThings API (or the local APIs, but I'm pretty sure none of the local APIs has any info about this)

Yup. Just putting it out there in the hopes that someone else has it. Because my TV is permanently OFFLINE and I can't even get App IDs for other Apps. It's only because others like you have already documented what they've found that I've got most of what I need done.

Yup. Just putting it out there in the hopes that someone else has it. Because my TV is permanently OFFLINE and I can't even get App IDs for other Apps. It's only because others like you have already documented what they've found that I've got most of what I need done.


What is the status of Hover Editor: abandonware or Active?

There does not seem to have been any involvement recently from the dev, @nothingislost. Instead, @pjeby has been answering questions and closing issues as if he has some involvement. Unfortunately, he refuses to elucidate the situation.

Please do not open duplicate issues.  See #172 for resolution.


[SR-7334] Fix crash and warn about duplicate dependency definitions

SR-7334
This provides a warning message when the same dependency is declared multiple times in a package, and fixes a crash caused by this

Closing this PR due to inactivity. Please feel free to reopen anytime!


Add Bonan plots to documentation

Purpose

Add plots reproducing Figs 6.3(a),(b) from Bonan (2019) to existing documentation.

[ ] I have read and checked the items on the review checklist.

This PR reproduces Figs 6.3(a),(b) from https://doi.org/10.1017/9781107339217.007. Plots render OK - forms of stability functions (as noted in the linked document) are from Dyer and Hicks 1970; Dyer 1974; Brutsaert 1982, pp. 68–71; Garratt 1992, pp. 52–54. With an updated description this PR LGTM.

bors try

bors r+

bors r-

@gdecker1 can you please rebase and squash the commits? Once this is done I can merge it.

bors r+


Filtering pandas df

Currently, there's only one option for reading pandas DataFrames from TTrees that have JaggedArrays: flatten=True. However, another use case would be to have a simple way to filter JaggedArrays out.
Currently, this looks like:
good_keys = [key for key in t.keys() if not isinstance(t[key].interpretation, uproot.asjagged)]
t.pandas.df(good_keys, ...)

This might be useful in the function call instead. Ideas: filter=True, filter_jagged=True ,flatten=None, flatten='filter', or jagged='filter' (jagged='flatten' or multi-index being another policy) are some ideas.
(This is because I'm dealing with everything-but-the-kitchen-sink NTuples...)

Actually, this is broader than the DataFrame interface: it affects the set of branches that are loaded if none are specified. The branches parameter already takes a number of options, one of which is a function from branch (object) to interpretation or None. You can right now do something like
t.pandas.df(lambda b: b.interpretation if not isinstance(b.interpretation, uproot.asjagged) else None)

though this is far from obvious. Maybe there should be a shorthand object that the branches parameter understands to define such policies in a more user-friendly way? One I can think of: suppose any Interpretation class object like asdtype is taken as
lambda b: b.interpretation if isinstance(b.interpretation, asdtype) else None

But then, you want the inverse of asjagged.

I did notice that it took a function, but I haven't taught my students lambda functions and I have taught them comprehensions, so that's why I structured my example that way.
However, .df already has a parameter that is designed for jagged arrays. What about having flatten take True (Current behavior), False (new: filter out jagged arrays), and None (the default, current behavior, crash)

Sorry, the default is jagged=True, and the crash is not because it's not trying to flatten, it's because:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-13-2689fd22c539> in <module>()
----> 1 df = t.pandas.df(entrystop=100_000)

~/.local/lib/python3.6/site-packages/uproot/_connect/to_pandas.py in df(self, branches, namedecode, entrystart, entrystop, flatten, cache, basketcache, keycache, executor, blocking)
     37     def df(self, branches=None, namedecode="utf-8", entrystart=None, entrystop=None, flatten=True, cache=None, basketcache=None, keycache=None, executor=None, blocking=True):
     38         import pandas
---> 39         return self._tree.arrays(branches=branches, outputtype=pandas.DataFrame, namedecode=namedecode, entrystart=entrystart, entrystop=entrystop, flatten=flatten, cache=cache, basketcache=basketcache, keycache=keycache, executor=executor, blocking=blocking)

~/.local/lib/python3.6/site-packages/uproot/tree.py in arrays(self, branches, outputtype, namedecode, entrystart, entrystop, flatten, cache, basketcache, keycache, executor, blocking)
    517         # if blocking, return the result of that function; otherwise, the function itself
    518         if blocking:
--> 519             return wait()
    520         else:
    521             return wait

~/.local/lib/python3.6/site-packages/uproot/tree.py in wait()
    482
    483                             df = outputtype(index=pandas.MultiIndex.from_arrays([entries, subentries], names=["entry", "subentry"]))
--> 484                             if interpretation.asdtype.todims == ():
    485                                 df[name] = array.content
    486                             else:

AttributeError: 'asjagged' object has no attribute 'asdtype'

That's an idea. It'll fix this bug (I wish Python was strongly but dynamically typed for booleans, as it is for all other types) and make meaningful use of three states of the parameter. That's the kind of thing that would absolutely have to be documented, though. It's on my to-do list.

I wish Python was strongly but dynamically typed for booleans, as it is for all other types

How are booleans different from other types? All types have a truthiness, that just happens to also be the main use for booleans. You can say x is False just like x is None.  I would not have None the default, True or False would be better (since it is would be user-unfreindly to remove the jagged columns without their asking).

Currently, False and None both do not crash, and they do the same thing - they are both "falsy". My suggestion is to make them do different things, just because I'd like three behaviors. The crash occurs when True is passed, which is also the current default. So to cause the crash above, just read a TTree with jagged columns into a dataframe using t.pandas.df().
Python convention for None checking is to use is not None; there was a long-winded PEP a while back to even add a set of operators for None checking. The convention for if x: replaces if x == True, which is uglier and can crash if x does not support comparison.
I've got several issues with booleans. True and False act like numbers to a tea; while I like to sum over booleans to count Trues, this is really odd: ~True is -2, and therefore truthy. But on arrays, we use ~ for not. However, what I'd really love to be able to do is have not, and, and or be overloadable to be used on arrays, which would also fix statements like 2 < array < 3.

Do you have a simple jagged example in tests/samples? I don't see one... I'd like to find a couple of normal columns and a couple of jagged ones.

Nevermind, I found one: sample-6.10.05-uncompressed.root. Verified that it crashes when you call .pandas.df(). I can add a test.

Can I highly recommend you use pytest? It's the gold standard for Python testing and is much easier to read and maintain. It will also read unittest style tests until tests are converted.

First PR was #157, then migrated to #160 and merged. This will be version 3.2.3 of uproot.


fix problem for php 7 (bcmath) to work to AMQP

this is for fix a problem with rabbitMQ, to fix the problem with phpamqplib bcadd() the only we can do is add list line  RUN docker-php-ext-install bcmath  in Dockerfile-70

@armandojose0009 could you add your fix in a PR?

hi  @philtrep how can do it ?? i create a branch in my git but whe try push to the repository don't have permission in the repo..

@armandojose0009 follow the instructions from https://help.github.com/articles/creating-a-pull-request-from-a-fork/

Hi thanks you
El 21/12/2016 3:10 p. m., "Mahmoud Zalt"<EMAIL_ADDRESS>escribió:
Closed #453 https://github.com/laradock/laradock/issues/453 via #502
https://github.com/laradock/laradock/pull/502.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/laradock/laradock/issues/453#event-902046704, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASzP27jJldJmpYl3yyn1e6z9Cw3hP5dOks5rKXmagaJpZM4K-6Vr
.


A local version label starts with a '+' sign

As per https://www.python.org/dev/peps/pep-0440/#local-version-identifiers:
Local version identifiers MUST comply with the following scheme:

<public version identifier>[+<local version label>]

But EGG_FRAGMENT's character set does not contain the plus sign, which seems to be a defect.

This looks good to me. If you add an entry to the CHanges.rst file, that'll save me time getting this out. Probably us an 0.1 bump. Thanks.

I found '!' also needs to be accepted on a further investigation:
https://www.python.org/dev/peps/pep-0440/#public-version-identifiers
The canonical public version identifiers MUST comply with the following scheme:

[N!]N(.N)*[{a|b|rc}N][.postN][.devN]

https://github.com/pypa/packaging/blob/master/packaging/specifiers.py#L301

Brilliant. I love it.


Unable to connect to RabbitMQ over SSL

I'm trying to connect to RabbitMQ using SSL. The 5671 port is open on RabbitMQ and accepting connections, but when I try to connect to it through the sink, I get a "None of the specified endpoints were reachable" error in .NET and "handshake_timeout,handshake" error in the RabbitMQ log. Is there somewhere I need to tell the sink to initiate the handshake for SSL?
Here is my code:
        RabbitMQConfiguration config = new RabbitMQConfiguration
        {
          DeliveryMode = RabbitMQDeliveryMode.Durable,
          Hostname = "myhost.com",
          Port = 5671,
          Username = "username",
          Password = "password",
          VHost = "/",
          BatchPostingLimit = 100,
          Exchange = "local.engines.logs.exchange",
          ExchangeType = "direct"
        };

        LoggerConfiguration loggerConfiguration = new LoggerConfiguration()
           .Enrich.FromLogContext()
           .Enrich.WithExceptionDetails()
           .Enrich.WithMachineName()
           .Enrich.WithProperty("Application_Name", ConfigurationManager.AppSettings["ApplicationName"])
           .Enrich.WithProperty("Application_Version", Assembly.GetExecutingAssembly().GetName().Version);

        if (config != null)
          loggerConfiguration.WriteTo.RabbitMQ(config, new JsonFormatter());

        Log.Logger = loggerConfiguration.CreateLogger();

I'm using:
.NET 4.7
RabbitMQ.Client 4.1.3
Serilog 2.8.0
Serilog.Sinks.RabbitMQ 2.0.2

Hi @dmajcherlandstar
In v3.0.0 - so you need to upgrade to be able to have access to the  SslOption. It's not sufficient to just change the port.
Your can find help on configuring the Sink correctly on RabbitMq.Clients website
Good luck.

steffenlyng, thanks for the response.
Ok, I upgraded to version 3.0.6 of Serilog.Sinks.RabbitMQ. I updated my code to the following, but I'm still not seeing messages in RabbitMQ. I turned off peer validation in RabbitMQ. Am I missing anything else?
Thanks
 RabbitMQClientConfiguration config = new RabbitMQClientConfiguration
        {
          DeliveryMode = RabbitMQDeliveryMode.Durable,
          Hostname = "myhost.com",
          Port = 5671,
          Username = "username",
          Password = "password",
          VHost = "/",
          BatchPostingLimit = 100,
          Exchange = "local.engines.logs.exchange",
          ExchangeType = "direct",
          SslOption = new SslOption() { ServerName = "myhost.com", Enabled = true, AcceptablePolicyErrors = SslPolicyErrors.RemoteCertificateNotAvailable | SslPolicyErrors.RemoteCertificateChainErrors | SslPolicyErrors.RemoteCertificateNameMismatch }
        };

        LoggerConfiguration loggerConfiguration = new LoggerConfiguration()
           .Enrich.FromLogContext()
           .Enrich.WithExceptionDetails()
           .Enrich.WithMachineName()
           .Enrich.WithProperty("Application_Name", ConfigurationManager.AppSettings["ApplicationName"])
           .Enrich.WithProperty("Application_Version", Assembly.GetExecutingAssembly().GetName().Version);

        if (config != null)
          loggerConfiguration.WriteTo.RabbitMQ(config, new JsonFormatter());

        Log.Logger = loggerConfiguration.CreateLogger();

I was able to get it working. I wasn't specifying the SslOption.Version property. Once I did that, it started working.
 RabbitMQClientConfiguration config = new RabbitMQClientConfiguration
        {
          DeliveryMode = RabbitMQDeliveryMode.Durable,
          Hostname = "myhost.com",
          Port = 5671,
          Username = "username",
          Password = "password",
          VHost = "/",
          BatchPostingLimit = 100,
          Exchange = "local.engines.logs.exchange",
          ExchangeType = "direct",
          SslOption = new SslOption() { ServerName = "myhost.com", Enabled = true, Version = SslProtocols.Tls12  }
        };

        LoggerConfiguration loggerConfiguration = new LoggerConfiguration()
           .Enrich.FromLogContext()
           .Enrich.WithExceptionDetails()
           .Enrich.WithMachineName()
           .Enrich.WithProperty("Application_Name", ConfigurationManager.AppSettings["ApplicationName"])
           .Enrich.WithProperty("Application_Version", Assembly.GetExecutingAssembly().GetName().Version);

        if (config != null)
          loggerConfiguration.WriteTo.RabbitMQ(config, new JsonFormatter());

        Log.Logger = loggerConfiguration.CreateLogger();


Deployment should only work on pushing of certain tags?

Right now, pushing any tag will result in deployment. However, we probably only want pushing a certain type of tag, e.g. 'v1.0' to trigger deployment? If someone pushes an old tag, the deployment will happen as well and that is not desired.
https://github.com/ietf-ribose/bibxml-service/blob/131d6f74bf695c399805cf171d35247ea4970c35/.github/workflows/main.yml#L6

I support this, this helps avoid situations where someone pushes a change but forgets to increment version. Version is shown in GUI and people may rely on it in some cases, so it’s undesirable.

relaton-py does it this way.

One problem here is that I don’t think Git tags are included in the PR. This creates a situation where we may have different version numbers on staging and production.

edit: Which may be OK"