zip.hpp not compiling on clang

In Godbolt, I have imported zip.hpp (by manualling copy and pasting)
https://godbolt.org/z/vrcaP6jbj
Not compiling in clang :

 error: missing 'typename' prior to dependent type name 'detail::tuple_or_pair_impl<Ts...>::type'  using tuple_or_pair = detail::tuple_or_pair_impl<Ts...>::type;

This may be related to another issue I have had using zip with std::view and it failing to compile (in MSCV) because of some indecipherable templating error

This should be fixed as of https://github.com/TartanLlama/ranges/commit/40ba5057de775d3d7218ac9975196868d22bf052


QOL Improvements for Classes

Summary of the new feature / enhancement
I'd like to suggest a few small quality of live improvements to PowerShell Classes.

Allow chaining Constructors (instead of having to an additional hidden internal function)

class T {
  T() {$this.T("something")}
  T($something) {Do-Something}
}

Fix default value handling for methods and functions in classes (this syntax is currently valid, but the default values are just ignored, requiring to create extra function definition to specify the default values in a way that actually works):

class T {
  something($mandatory, $optional="", $optional2=$false) {}
}
[T]::new().something("mandatoryvalue") | Should -Work # ParentContainsErrorRecordException: Cannot find an overload for "something" and the argument count: "1".

Add support for parameter validation to avoid boilerplate validation code for simple things like ValidateSet or NotNullOrEmpty
Add support for typed properties that aren't FixedSize. This could be archived by adding a (virtual) generic for ArrayList to use it like [System.Collections.Arraylist[String]] instead of [String[]] to get a "Non IsFixedSize" version of it. The main annoyance currently is that .Add() doesn't work when the list IsFixedSize.
Add an Enum Type (or better a annotation or attribute) that serializes and deserializes as the value instead of it's index key to replace [ValidateSet('string1', 'string2','string3')][string]$variable with a proper type without requiring a bunch of custom serialization and deserialization code. So that ConvertTo-Json and ConvertFrom-Json continue to work, but that proper types can be used for the attribute instead of having to check for the string value within the script.
Document that having a constructor with a single parameter but without type class T{ T() {}; T($name) {}} breaks type casting if not implemented explicitely within that constructor. Currently the documentation only mentions the parameterless one, but not the one with a single parameter without a specified type (is this intentional or a bug?)

class T{
  [string]$a
  [string]$b = "some default"
  [string]$name

  T() {}
  T($name) {
    $this.name = $name
  }
}
[T]@{
  a="Test"
  b="Values"
  name="SomeName"
} # Should return an object of Type T with a, b, and name initialized to the specified values, but instead it'll pass the HashTable to the T($name) constructor, which is currently not documented in the constructor section of about_Classes

Not technically classes related, but as classes drop everything that is not explicitly returned, having a modifier for normal functions (outside of classes) to act the same would be great to get rid of $null = and | Out-Null in almost every line of such a function... [DefaultOut(Out-Null)] function Do-Something() { Do-Something -WithOutput; return "Only thing that is returned"}

Proposed technical implementation details (optional)
No response

Related #6418

Also:

#6652


Replace is-lowercase-alpha and is-digit individual checks for a range check

Currently is-lowercase-alpha and is-digit compares char individually with each different case. We can replace this with a range check like this:
;; Determines if a character is a digit (0-9).
(define-private (is-digit-test (char (buff 1)))
    (and
        ;; Checks if the character is between '0' and '9' using hex values.
        (>= char 0x30) ;; 0
        (<= char 0x39) ;; 9
    )
)

;; Checks if a character is a lowercase alphabetic character (a-z).
(define-read-only (is-lowercase-alpha-test (char (buff 1)))
    (and
        ;; Checks for each lowercase letter using hex values.
        (>= char 0x61) ;; a
        (<= char 0x7a) ;; z
    )
)

Further, we could use inequality <,> and increase/lower by 1 the ranges and optimize even further but at the risk of reducing clarity.

Thanks, will look into this


Use non-broken link to protocol/commandTypes

Micro-fix. Link out to CommandTypes was broken. Decided to link to the source code

Thanks!


Fixed toggling source buttons on mobile

This change is

 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.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


10.2 counting inversions (better approach merge sort)

My code is clear and well understanding
Please assign it to me

Need to add a description and your approach. Thanks, Closing as of now.


⚠ ClamAV has degraded performance

In f22d324, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 47cfd47.


pgdiff does not work with PostgreSQL 8

I'm trying to run pgdiff against two servers running PG 8.4. Unfortunately, pgdiff seems to make lots of assumptions that do not hold with PG 8. Example:
$ bin-linux64/pgdiff [...] table
[...]
2015/01/15 16:19:36 Error running querypq: syntax error at or near "COLLATE"

or:
$ bin-linux64/pgdiff [...] index
[...]
2015/01/15 16:20:49 Error running querypq: column con.conindid does not exist

or:
$ bin-linux64/pgdiff [...] role
[...]
2015/01/15 16:21:30 Error running querypq: column r.rolreplication does not exist

Etc, etc. Looks like nothing works.
It would be awesome if pgdiff worked with PG 8 as well as 9. But if it only works with 9, please at least say so in the README! Then I can just skip it and look for another tool.

Hi Gward, I'm embarrassed to admit that I didn't see your post until more than a year later.  I'm adding a filter to my gmail so I see these right away.
Since it's not likely I'll get this working with Postgres 8, I'll update the README.  I'm sorry you had to find out the hard way.


How to autogenerate Adaptors with dart?

Question
The hive docs outline how to autogenerate adaptors for flutter projects, the same instruction cannot be used for pure dart projects, I've done some searching but I hit a dead end any help would be appriciated.
Code sample

Any annotated class in a pure dart project

Version

Platform: Windows
Flutter version: 2.5.1 stable
Hive version: latest (started my proj day b4)

The instructions there do only list Flutter commands, but what happens if you do the dart variants? dart run build_runner build.

Hive generator doesn't depend on flutter. You can run build_runner either using dart or flutter tool cli.
flutter pub run
dart pub run
dart run

I haven't tested personally but AFAIK all of those should have same effect.

I assume we need to use build_runner from pub?


Is there a way to get Audio from the camera streams?

Hey there, and thank you for this great integration.
Mainly a question, secondly, if possible, a feature request.
I was wondering what's going on with sound. I have seen several posts in the community saying that several onvif streams have audio and others are muted by code.
What can i do about it?
Is there some way to enable sound?
Is it impossible?
Thank you

RTSP stream does support sound via VLC, I will look into this.

RTSP stream does support sound via VLC, I will look into this.

Audio from my other cameras work via Onvif integration (aac codec)
Audio from Tapo via Onvif integration (and this one) doesn't work.
So this is the codec information of Tapo camera:

This is the PR which added audio capability to Home Assistant.
Quote:
This PR muxes the audio stream into the output HLS stream if the codec is supported. If not, no audio stream is sent along with the video, saving bandwidth. Note that PCM codecs such as G.711 are not supported as they are not supported in mp4 containers.
Unfortunately not possible. Support will have to come from Home Assistant side.

Audio from my other cameras work via Onvif integration (aac codec)
Audio from Tapo via Onvif integration (and this one) doesn't work.
So this is the codec information of Tapo camera:

This is the PR which added audio capability to Home Assistant.
Quote:
This PR muxes the audio stream into the output HLS stream if the codec is supported. If not, no audio stream is sent along with the video, saving bandwidth. Note that PCM codecs such as G.711 are not supported as they are not supported in mp4 containers.
Unfortunately not possible. Support will have to come from Home Assistant side.

Since the PR got merged into dev: can we expect audio to be working in a future home assistant release?

@JurajNyiri sorry for bringing this up again, could you instruct me on what further steps i could take to make this happen?
If i understand correctly, we need the PCM ALAW codec to be muxed into the stream.
Right now only "aac", "ac3" and "mp3" codecs are supported.
So this could be solved by including PCM codecs in the stream, from the HA side, or Tapo firmware to use different codec right?
How could one go and request this? In the community forum of HA or somewhere in github, could you please point me to that direction?
I am guessing that changing the tapo firmware would be out of the question.

Yes you are correct, change would have to happen either from Home Assistant or Tapo firmware.
You can request a new feature here: https://community.home-assistant.io/c/feature-requests/13 . It might not be possible because browser themselves do not support PCM ALAW (afaik) in video streams - https://developer.mozilla.org/zh-TW/docs/Web/HTML/Supported_media_formats .

Thank you, i will go start a feature request on the HA-side.

But is it possibel just to extract this audio stream and record it in file?
I need only audio recordings not real time listening.

Yes but out of scope of this integration. You can use ffmpeg directly to output audio to file.

Yes but out of scope of this integration. You can use ffmpeg directly to output audio to file.

Thanks a lot!
But can i use your itegration (i like it very much!) together with ffmpeg? Will it make any interference\problems?
Thanks a lot!

Hi, I don't have a camera or HA yet - doing preliminary research. From what I was reading about showing streams from cameras using RTSP and showing the stream in browser, there is currently no direct ( and native+universal) solution (https://flashphoner.com/browser-based-webrtc-stream-from-rtsp-ip-camera-with-low-latency/).
So I gather that HA is already doing some transcoding (probably using ffmpeg since it is based on the stream integration). And from what I understand it is quite simple for ffmpeg to convert the PCM audio to something usable (AAC)?
I tried this today and had a proof of concept with working sound (but only with PCM ulaw stream, could not find a alaw stream), basically using https://video.stackexchange.com/questions/29887/ffmpeg-convert-g-711-audio-to-aac-audio .
And since my result was just an browser-consumable stream, which is processed by ffmpeg in the backgound, isn't it basically the same HA does? And if so, could I just point HA to this new stream and use it directly (ideally without another ffmpeg processing)? Or is there something I am missing?

@krasatos did you open a feature issue on Home Assistant? Would be nice to know the status

Awesome project! I'd love to start using it with C320WSs. I noticed this issue and want to share what I've learned over the years regarding PCM audio:
Upstream HomeAssistant cannot "fix" or change what audio codecs MP4 allows. It's defined by ISO 14496-14.
So it looks like there's at least two ways forward:

Transcode the audio using gstreamer, which is available in hass. I've done this for my C320WS. Happy to share the code if it would help.
Use a different container or stream protocol that supports PCM audio e.g. RTSP or WebRTC.

Let me know what sounds best to you (no pun intended).

Awesome project! I'd love to start using it with C320WSs. I noticed this issue and want to share what I've learned over the years regarding PCM audio:
Upstream HomeAssistant cannot "fix" or change what audio codecs MP4 allows. It's defined by ISO 14496-14.
So it looks like there's at least two ways forward:

Transcode the audio using gstreamer, which is available in hass. I've done this for my C320WS. Happy to share the code if it would help.
Use a different container or stream protocol that supports PCM audio e.g. RTSP or WebRTC.

Which approach sounds best to you? (no pun intended)

Hi, I am interested in how you solved this, could you please share your solution with some explanation?
Does this solution work with the TAPA add-on from github? I have two C225 camera's.
Thanks in advance, kind regards Mark


Azgaviperr Addition And Cleanup

Changed some stuff to show who created/maintain the app and added some app I didn't yet shared.

Nice! We will need to have you share them through our https://shuffler.io for now as well, though. Could you upload them and publish them there?
Mapping it back to here will be a part of it later for sure


Define unique index on multiple columns in sequelize

Is this feature supported?

yes it is, just give the index the same name on the fields.
https://www.npmjs.com/package/sequelize-typescript#index
eg.

@Table
export class UserObject extends Model{

  @Index({
    name:'UserIdObjectId',
    unique:true,
  })
  @Column
  public userId!: number;

  @Index({
    name:'UserIdObjectId',
    unique:true,
  })
  @Column
  public objectId!: number;
}

could it be that the second method ({ fields: ['userId','objectId'], unique:true}) not only adds a composite index, but also 2 single indexes on the individual columns?
is this behavior intended or just not well documented ?


docs: Exec commandLine option repeated

This is an extremely minor bug but I thought I would report it anyway.
If you look at the gradle docs for task type Exec:
https://docs.gradle.org/4.0/dsl/org.gradle.api.tasks.Exec.html
Under methods, It says:
commandLine(args) and commandLine(arguments), and also args(args) and args(args)
Which as far as I'm aware is the same thing repeated twice each.
EDIT:
Actually now that I look over the docs more closely, I notice this in several places, is this intentional?

I think those are for different overloaded methods. But yeah, the description could be updated with the argument type to distinguish them.

What about args(args) repeated twice? I'm a total novice when it comes to both Java and Gradle but that just seems wrong to me in any language.

I believe you could submit a pull request bravely. Gradle guys are nice, if there's anything inappropriate, they will point out it.

If you click on the link of one of the methods you will see that the parameter types are different: https://docs.gradle.org/4.0/dsl/org.gradle.api.tasks.Exec.html#org.gradle.api.tasks.Exec:commandLine(java.lang.Iterable). The list of methods doesn't render the types.


Plans for nuxt-on-netlify?

Would be sweet if we can get a Nuxt equivalent for this library.

Nuxt in it's current state works very well on Netlify, especially with with target: static :) that said, we are hiring someone to work on some of the new features that will be coming out with Nuxt 3, so there will be one, yes!


Define rule DDF00007: Timing From/To - For anchor then To does not need to be set or could be set the same as From

@rbakercdisc1 @DianeWold I'm planning to create this rule with the following specification:
In an instance of the Timing class/entity representing an anchor (i.e., where the type relationship links to an instance of the Code class/entity with code/decode of C201358/"Fixed Reference"), the relativeFromScheduledInstance relationship must link to either to nothing or to the same thing as relativeToScheduledInstance.
Given that we're supposed to be defining checks for data created by the USDM API, I will need to update the test data template to allow linked id values to be specified (e.g., in relativeToScheduledInstanceId and relativeFromScheduledInstanceId columns on the Timing.xpt sheet).  Once this is done, the check will be specified as relativeFromScheduledInstanceId must be blank or match relativeToScheduledInstanceId in anchors.

@ASL-rmarshall I agree that the specification you wrote is an accurate representation of Dave IH's "For anchor then To does not need to be set or could be set the same as From."  However, I'd like to understand where this rule comes from.  I think I need to dig into the Timing section of the USDM IG.

I do not understand this rule. Can we rephrase it somehow?

@gerrycampion Agree the description is opaque. It's something like, "For a timing with type = "Fixed Reference", relativeFromScheduledInstance relationship must link to either to nothing or to the same thing as relativeToScheduledInstance."
Part of the problem is that the text "anchor" doesn't appear in either the model or the controlled terminology.


Arduino restarts when it tries to read RTC_DS3231::now()

I tried to use this function to read the RTC, but the Arduino Pro Mini crashes constantly.
I was only able to isolate the problem using WDT. I was isolating each part of the code until I got to this library function. If I comment out this line, there are no more crashes:
  i2c_dev->write_then_read(buffer, 1, buffer, 7);

Full function:
DateTime RTC_DS3231::now() {
  uint8_t buffer[7];
  buffer[0] = 0;
  i2c_dev->write_then_read(buffer, 1, buffer, 7);

  return DateTime(bcd2bin(buffer[6]) + 2000U, bcd2bin(buffer[5] & 0x7F),
                  bcd2bin(buffer[4]), bcd2bin(buffer[2]), bcd2bin(buffer[1]),
                  bcd2bin(buffer[0] & 0x7F));
}

In another project with RTC DS3231 I've been using this other library for a long time without having this kind of problem, in case someone wants to compare what might be happening:
https://github.com/JChristensen/DS3232RTC
// Read the current time from the RTC and return it in a tmElements_t
// structure. Returns the I2C status (zero if successful).
uint8_t DS3232RTC::read(tmElements_t &tm)
{
    i2cBeginTransmission(DS32_ADDR);
    i2cWrite(DS32_SECONDS);
    if ( uint8_t e = i2cEndTransmission() ) { errCode = e; return e; }
    // request 7 bytes (secs, min, hr, dow, date, mth, yr)
    i2cRequestFrom(DS32_ADDR, static_cast<uint8_t>(tmNbrFields));
    tm.Second = bcd2dec(i2cRead() & ~_BV(DS1307_CH));
    tm.Minute = bcd2dec(i2cRead());
    tm.Hour = bcd2dec(i2cRead() & ~_BV(DS32_HR1224));   // assumes 24hr clock
    tm.Wday = i2cRead();
    tm.Day = bcd2dec(i2cRead());
    tm.Month = bcd2dec(i2cRead() & ~_BV(DS32_CENTURY)); // don't use the Century bit
    tm.Year = y2kYearToTm(bcd2dec(i2cRead()));
    return 0;
}

Too soon, I replaced this library with the other one, but it's still crashing, so I'll leave this notification closed, in case I manage to find the problem, I'll try to comment later.

I found the problem, the other library that makes the display work doesn't allow the I2C port to work properly.
A library for driving the Freetronics 512 pixel dot matrix LED display "DMD", a 32 x 16 layout.
Library that does not support I2C:
https://github.com/freetronics/DMD2
Library that accepts I2C:
https://github.com/freetronics/DMD

https://github.com/freetronics/DMD2/issues/67


MLAT not working for FlightAware

Specifications

Image hash or version: 3b511184a1cb
Platform: Raspberry Pi
Subsystem:

Expected Behavior
Actual Behavior
FlightRadar24 is receiving MLAT from my station but FlightAware isn't.

Do you have any error, what is flight aware saying about MLAT sync ? I really don't know what could be wrong 😕


[MSINTEROP-723] Implement a data retention policy

Adds delete_old_issues function
Adds append_last_updated_arg function
Documents the data retention policy in README
Adds tests to support new functionality

/verified

/verified

/verified

/verified

/verified


Use babel for esm for core

WIP of tree shaking.
Currently, only affect core package.
The structure is not final, some files are redundant. Just demonstrate how it works.

Confirmed w. @bertya that this PR is outdated to be closed.


Changes to LEPCC accumulated in Scene Layer Library.

Are there tests for these changes? It would be really helpful to have some continuous integration connected with a test framework as part of this repository. As it is now, there's little to go on as to why changes are being made or what their impact might be.

@hobu  No worries. There is no intention to change anything. The main goal here is to fix build errors with other compilers. This is already done with the last, much leaner check in.
Closing this PR.


zotero: 4.0.29 -> 5.0.25

This uses buildFHSUserenv.  The source installation probably does not work until
node2nix supports nodejs-8.x.
Motivation for this change
Zotero does not come bundled with newer firefox versions anymore, only standalone is supported.
Things done

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

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

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

See also #31479

This would also close #10355 presumably

This package should also install a menu entry (a desktop file) and icons (they are available in data/chrome/icons/default/). @timor Would you do this?

@c0bw3b I have not tested all the things mentioned in #10355
@orivej Haven't done this before, is there a good example package?  Also, how to test that, I am not using a desktop environment?

I don't know how to test this without a desktop environment, so I've pushed a commit with the desktop entry.

This is amazing, thank you @timor! 🍻


[AutoPR billing/resource-manager] ARM spec review for validateAddress for BillingRP

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6010

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/3102


Allow hubot to send slash commands

The use case I am trying to implement is actually to just get hubot to /leave. Since 99% of users on our slack team can't /kick rebot, it'd be nice to be able to ask hubot to leave instead of asking an admin to come into the channel and kick it for us.
However, it seems hubot can only send raw text messages. I can't get it to send commands to slack. Not sure if I'm doing something wrong or what. I guess a mechanism to send any arbitray /slash commands to slack would be pretty useful.
Unless there are some security/abuse issues there. E.g. usage of the /kick command on slack teams where anyone can kick, or usage of commands like /msg to message other users via hubot without the receiver knowing who it was really from. Similar for /invite and not even sure what /feedback could be abused for but i'm sure it could.

@tigris, until there's something official, I've made a Hubot wrapper for the Slack web API that'd allow you to leave a channel with robot.slack.channels.leave({channel: "#somechan"}).

Under the hood, most of the slash commands you've cited are just API methods that you'd be able to call. The main restriction is that bot users can't call everything, as @ORBAT pointed out. channel.leave isn't one of those though. https://api.slack.com/bot-users is the list of function.
So, I think we can close this as a feature request to be able to run slash commands, and instead open something to make it possible for hubot to leave a room. Does that sound reasonable?

Still looking for a solutions to bot users being able to trigger slash commands. In my case, I want to trigger prompts to slash commands with cron, but maintain only one CLI in the form of the Slash command.

Unfortunately, the chat.command endpoint requires a scope that is no longer available to third party developers, so the only way to call it is by logging in as a user, with username and password…and then we veer into highly undocumented and super insecure territory 😢  (👋 ex-Slack platform engineer and developer relations here 😉)


ProtectKeysWithCertificate only searches the "My" StoreName

From @DavidChenware on Thursday, March 3, 2016 7:37:21 AM
We push various certificates to workstations and server groups via GPO, and sometimes these end up in the Trusted Root Certification Authorities store.
ProtectKeysWithCertificate(thumbprint) seems to only search StoreName.My by default, e.g. the Personal store for both the Current User and the Local Computer accounts.
This means that we cannot use certificates in anything but the Personal stores to encrypt keys at rest.
A) Can this be a feature added? The ability to either automatically search all stores, or the ability to search specific StoreNames. e.g. StoreName.Root.
configure.ProtectKeysWithCertificate("thumbprint", StoreName.Root);
B) Or...is it the intention that if I need to search for a certificate by thumbprint in a different StoreName, that I just duplicate the code already in the CertificateResolver and search my own stores with the X509Store.
var store = new X509Store(StoreName.Root, StoreLocation.LocalMachine);
try
{
    store.Open(OpenFlags.ReadOnly);
    var matching = store.Certificates.Find(
        X509FindType.FindByThumbprint,
        "thumbprint",
        validOnly: true);
}
finally
{
    store.Close();
}

Thanks.
Copied from original issue: aspnet/DataProtection#125

From @muratg on Thursday, March 3, 2016 9:11:56 AM
@blowdart thoughts?

From @blowdart on Thursday, March 3, 2016 9:12:47 AM
By design. You can of courser provide your own X509Certificate2 when configuring.

From @soeron on Wednesday, March 23, 2016 7:33:55 AM
Providing certificate doesn't solves anything.
Using ProtectKeysWithCertificate( X509Certificate2 ) still searches the stores and if the certificate is not there raises Cryptography_Xml_MissingDecryptionKey (System.Security.Cryptography.Xml.EncryptedXml).
If the certificate is in the store then the exception becomes Bad key\r\n which is guess is because of the certificate purposes.

From @pksorensen on Monday, March 13, 2017 1:56:34 PM
I also ran into:
System.InvalidOperationException: A certificate with the thumbprint 'C03BB5A6410741CDD2927B4FF88C3E67215A393B' could not be found.

and looking at the code here - i get the impression that it should be implemented such it first look in User Store, then Machine. Therefore I am confused that it fails with above when below works- where I find the certificate myself.
services.AddDataProtection()                     .ProtectKeysWithCertificate(X509.LocalMachine.My.Thumbprint.Find("C03BB5A6410741CDD2927B4FF88C3E67215A393B", validOnly: false).FirstOrDefault())

Anyway, the latter solution is fine for me - just expected the overload to work also

From @natemcmaster on Monday, May 8, 2017 12:44:03 PM
Remarking for triage: .NET Core 2.0 adds support for EncryptedXml, however, at the moment, the X509Store on Linux will only support the 'LocalMachine\Root' and 'LocalMachine\CertificateAuthority' stores.
See this issue: https://github.com/dotnet/corefx/issues/3690 - currently marked as up-for-grabs

From @natemcmaster on Monday, May 8, 2017 3:26:22 PM
To clarify my last comment, CurrentUser\My is supported on Linux in .NET Core 2.0. It's LocalMachine\My that isn't which means there's no good machine-wide place to add store certs.

Closed as by design.


fixes typings for typescript usage (both CommonJS and ES6 modules)

The types are broken when using typescript or a a project with a jsconfig.json, this is related to https://github.com/fastify/fluent-schema/issues/62 and this PR solves that issue.
In addition to that, I added the package-lock.json to the project since it ensures the correct version of each dependency will be installed.

Can you please remove package-lock? Thanks.
Also, can you add some sort of test so that we do not regress this?

@mcollina I removed the package-lock.json and added a basic test file, the way it works now is much like axios, requiring the module works (as before), but using require('fluent-schema').default returns the type definitions. It works with ES6 import as well.

LGTM but all my projects have "esModuleInterop": true
Anyone use MS Code and can double-check if this helps for code hint as well? @jnv related to #59

LGTM but all my projects have "esModuleInterop": true
Anyone use MS Code and can double-check if this helps for code hint as well? @jnv related to #59

@javi11


[CTIS QSF tools] use item name remapping to generate matrix subquestion names in differ

In the differ, the names in the list of matrix subquestions should ideally be generated using remapped base question names from item_rename_map.csv. Not doing so could lead to errors in the changelog since we're joining the codebook onto the diffs using the generated matrix subquestion names (for matrix questions).

Addressed in https://github.com/cmu-delphi/covidcast-indicators/pull/1564


[Backend] Update PrivacyExperienceConfig fields to support new requirements

Is your feature request related to a specific problem?
See design doc here

Removing concept of delivery mechanism, replacing with banner_enabled field
Removing banner_title and banner_description in favor of component_title and component_description
Rename link_label to open_modal_link
Add privacy_policy_link
Add privacy_policy_url
Remove validation on how notices must be delivered in a banner

Related to https://github.com/ethyca/fides/issues/3320
Describe the solution you'd like
A description of what you want to happen.
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@pattisdr updated designs here: https://www.figma.com/file/dmEwdK3xZwjKfGVQ9t66xe/Fides-v.2-Master-Working-Files?type=design&node-id=6980-351576&t=Q8vzrc7cfau41H2J-4

@mfbrown do we not have the option to link regions here though?

Ah ok never mind I understand the design better -

moving to done


Fix yucky error when target asm doesn't put labels into the data section

https://decomp.me/scratch/qnEHC

This is much less yucky now - just an asm-differ error.


Feat: Show Nested Table Columns

Summary of Changes
In addition to showing nested columns as strings in a modal, we now support showing nested columns as individual rows in the list of columns. Also fixes some bugs where sorting columns would mess up descriptions and nested types.
Tests

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

This is awesome, do you have a screenshot here that you could share?

I have one request, Can we have sample data as well ? because we are getting such a good features but can't get visual or understanding better without having sample data.


docs: Add VS Code WSL extension install step for dev envelopment setup

This commit adds a step to the CVAT development setup guide for users working with WSL (Windows Subsystem for Linux). The added instruction guides users to install the VS Code extension for WSL, ensuring that Visual Studio Code opens correctly inside the WSL environment.

This change addresses an issue where users might encounter a 'DEBUG STOPPED' error if the extension is not installed, improving the overall setup experience.

Related to issue #8313.

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

Documentation

Added instructions for installing a Visual Studio Code extension for WSL to improve the development environment setup.

@mdacoca can you please guide me know why linter fails. As much I understand is commit message has length more than 72 characters which I fixed. Now its showing me something like this "237:211  warning  Line must be at most 120 characters  maximum-line-length  remark-lint. Can you please kindly guide me.

@azhavoro  can you please guide me know why Linter check fails. As much I understand is commit message has length more than 72 characters which I fixed. Now its showing me something like this "237:211 warning Line must be at most 120 characters maximum-line-length remark-lint. Can you please kindly guide me.

Thanks for the contribution!


Enhance Packaging, Testing, and Infrastructure with Modern Tools and Features

Before submitting a pull request make sure you have:

[x] The pull request title contains a meaningful title

Short and informative: serves as a summary

[x] At least skimmed through the coding conventions used in the project
[x] Checked the code with flake8
[x] Performed a self-review of my own code
[x] The code is commented, particularly in hard-to-understand areas
[x] The documentation associated was created or updated
[x] The changes generate no new warnings
[x] Add tests that prove the change is effective or that the feature works
[x] New and existing unit tests pass locally with the changes
[x] Any dependent changes have been merged and published in downstream modules

Contents of the Pull Request
This pull request introduces significant updates and new features to the project:
New Features

Added .github/CODEOWNERS file for automatic reviewer assignments.
Introduced comprehensive issue templates for bug reports and feature requests.
Added SECURITY.md to outline the security policy, supported versions, and disclosure practices.

Infrastructure Enhancements

Migrated to pyproject.toml for modern Python packaging, replacing setup.py and setup.cfg.
Introduced Rye for dependency and virtual environment management.
Updated .python-version to include Python versions 3.12.8 and 3.13.1.

CLI Improvements

Enhanced gsppy/cli.py with:

Verbosity control using a --verbose flag.
Type annotations for better maintainability.
Streamlined logging and error messages.

Testing & Quality

Reorganized test structure: moved tests to the root tests directory.
Added static type checks using mypy and pyright.
Replaced linting and type-checking configurations in workflows for improved accuracy.

Stacked PR Chains (Optional)

N/A

How Has This Been Tested? (Required)
The following tests and validations were performed:

Unit Tests:

All new features and changes were covered by new or existing unit tests.
Mocked functions to simulate edge cases in the CLI.
Verified backward compatibility.

Integration Tests:

Verified functionality with real datasets.
Tested end-to-end workflows, including CLI commands and output validation.

Manual Testing:

Validated logging verbosity for CLI operations.
Ensured smooth transitions when using Rye for environment setup.

Test Configuration:

Firmware version: N/A
Hardware: Standard development machine
Toolchain: pytest, pyright, ruff, tox
SDK: Python 3.8 to 3.13

Other Notes

This PR deprecates requirements-dev.txt and migrates all configurations to pyproject.toml.
It also removes legacy CI workflows and replaces them with modern, scalable alternatives.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.75%. Comparing base (46eca97) to head (a491432).

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   99.50%   99.75%   +0.24%
==========================================
  Files           6        6
  Lines         405      409       +4
  Branches       22       20       -2
==========================================
+ Hits          403      408       +5
+ Partials        2        1       -1

Flag
Coverage Δ

unittests
99.75% <100.00%> (+0.24%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: Capture chain_type in sqlite chain table

Given ibctest supports many types of chains, we need to be explicit about the chain type (ecosystem). Any debugging tools can leverage this extra dimension to present more advanced debugging options for cosmos like the messages view from https://github.com/strangelove-ventures/ibctest/pull/133.
Supports: https://github.com/strangelove-ventures/ibctest/issues/123

I changed my opinion. I'm going to merge this one.

Therefore my loosely held preference is that we just change the CREATE TABLE call

The problem with this strategy is we CREATE TABLE IF NOT EXISTS. Therefore, if we add columns to a table on an existing schema, it will skip the create table call. And then downstream callers may expect new columns, but the columns won't be there resulting in potentially obscure errors.
But if we do CREATE TABLE without the existence check, the migrations will fail every time if the schema exists. I think that's terrible user experience - a database that holds only the first test case. The same problems applies to ADD COLUMN without ignoring the "duplicate column" error.
My hope was this database would not get too complex. If we feel the complexity grows more, it may be time to consider something like https://github.com/golang-migrate/migrate. With SQlite, my hope was to avoid it. (May need CGO for this migrate pkg.)
I think a simple, detailed error message to the user to rm or mv the old database is sufficient. I was going to attempt that but given we may move the database off the Interchain object, I will wait until that refactor is complete.


Error loading model after finetuning.

Thanks for the great work. I am however running into issue when training a Levit-128s model and then loading the trained model . It complains with lot of unexpected keys.
I am using following command to evaluate the model (checkpoint.pth)
python main.py --eval --model LeViT_128S  --batch-size 1
`        Missing key(s) in state_dict: "patch_embed.0.weight", "patch_embed.0.bias", "patch_embed.2.weight", "patch_embed.2.bias", "patch_embed.4.weight", "patch_embed.4.bias", "patch_embed.6.weight", "patch_embed.6.bias", "blocks.0.m.qkv.weight", "blocks.0.m.qkv.bias", "blocks.0.m.proj.1.weight", "blocks.0.m.proj.1.bias", "blocks.1.m.0.weight", "blocks.1.m.0.bias", "blocks.1.m.2.weight", "blocks.1.m.2.bias", "blocks.2.m.qkv.weight", "blocks.2.m.qkv.bias", "blocks.2.m.proj.1.weight", "blocks.2.m.proj.1.bias", "blocks.3.m.0.weight", "blocks.3.m.0.bias", "blocks.3.m.2.weight", "blocks.3.m.2.bias", "blocks.4.kv.weight", "blocks.4.kv.bias", "blocks.4.q.1.weight", "blocks.4.q.1.bias", "blocks.4.proj.1.weight", "blocks.4.proj.1.bias", "blocks.5.m.0.weight", "blocks.5.m.0.bias", "blocks.5.m.2.weight", "blocks.5.m.2.bias", "blocks.6.m.qkv.weight", "blocks.6.m.qkv.bias", "blocks.6.m.proj.1.weight", "blocks.6.m.proj.1.bias", "blocks.7.m.0.weight", "blocks.7.m.0.bias", "blocks.7.m.2.weight", "blocks.7.m.2.bias", "blocks.8.m.qkv.weight", "blocks.8.m.qkv.bias", "blocks.8.m.proj.1.weight", "blocks.8.m.proj.1.bias", "blocks.9.m.0.weight", "blocks.9.m.0.bias", "blocks.9.m.2.weight", "blocks.9.m.2.bias", "blocks.10.m.qkv.weight", "blocks.10.m.qkv.bias", "blocks.10.m.proj.1.weight", "blocks.10.m.proj.1.bias", "blocks.11.m.0.weight", "blocks.11.m.0.bias", "blocks.11.m.2.weight", "blocks.11.m.2.bias", "blocks.12.kv.weight", "blocks.12.kv.bias", "blocks.12.q.1.weight", "blocks.12.q.1.bias", "blocks.12.proj.1.weight", "blocks.12.proj.1.bias", "blocks.13.m.0.weight", "blocks.13.m.0.bias", "blocks.13.m.2.weight", "blocks.13.m.2.bias", "blocks.14.m.qkv.weight", "blocks.14.m.qkv.bias", "blocks.14.m.proj.1.weight", "blocks.14.m.proj.1.bias", "blocks.15.m.0.weight", "blocks.15.m.0.bias", "blocks.15.m.2.weight", "blocks.15.m.2.bias", "blocks.16.m.qkv.weight", "blocks.16.m.qkv.bias", "blocks.16.m.proj.1.weight", "blocks.16.m.proj.1.bias", "blocks.17.m.0.weight", "blocks.17.m.0.bias", "blocks.17.m.2.weight", "blocks.17.m.2.bias", "blocks.18.m.qkv.weight", "blocks.18.m.qkv.bias", "blocks.18.m.proj.1.weight", "blocks.18.m.proj.1.bias", "blocks.19.m.0.weight", "blocks.19.m.0.bias", "blocks.19.m.2.weight", "blocks.19.m.2.bias", "blocks.20.m.qkv.weight", "blocks.20.m.qkv.bias", "blocks.20.m.proj.1.weight", "blocks.20.m.proj.1.bias", "blocks.21.m.0.weight", "blocks.21.m.0.bias", "blocks.21.m.2.weight", "blocks.21.m.2.bias", "head.weight", "head.bias".

Update: create_model function from timm has fuse argument setting it to False , loads the model correctly. Is there a function to save the fused model though?

When I run
python main.py --eval --model LeViT_128S --batch-size 128

I get
...
Creating model: LeViT_128S
number of params: 7756466
...
Test:  [  0/261]  eta: 0:19:30  loss: 0.5133 (0.5133)  acc1: 85.9375 (85.9375)  acc5: 96.8750 (96.8750)  time: 4.4859  data: 4.0955  max mem: 780
Test:  [ 10/261]  eta: 0:02:25  loss: 0.5133 (0.5916)  acc1: 85.9375 (84.0909)  acc5: 96.8750 (96.4489)  time: 0.5778  data: 0.4961  max mem: 780
...
Accuracy of the network on the 50000 test images: 76.5%

The model in '--eval' mode first loads the weights, then removes the Batchnorm modules. You are free to save a copy of the weights for the fused model as you would for any PyTorch model.


Action selection based on parameters

sorry for not providing code samples. This is an issue I am seeing on a closed sourced project and trying to solve it now. I can get some reproducible steps for this if needed.

I have two controller actions which have the same route pattern but have different non-optional, simple type (e.g. string, int, etc.) parameters on the method signature. However, I am getting AmbiguousActionException.
Don't MVC do parameter based routing as Web API is doing?

No, the WebAPI compat shim package has a backwards compatible version of this as an IActionConstraint, but all parameters in MVC are optional.
Anything that we did in the future here will have to be opt-in. Personally, I like the idea of making query string patterns part of attribute routing. It's opt in, expressive, and lights up existing features like constraints, and view expanders. It also gives you a clear way to indicate "required" query string data.

I have two controller actions which have the same route pattern but have different non-optional, simple type (e.g. string, int, etc.)

This only works if they have different names in the query string in WebAPI 1-3x. WebAPI never had overloading based on type.

thanks @rynowak! So, the preferred way to do this is to do an inner routing inside the action itself based on the query strings to make it perform different operations based on query string values which are being passed in?
That's will end up being a bit dirty but not entirely bad. I assume I can extend action selection to employ this logic?

There's a few options here, I'll give some examples and you can decide what you like :+1:
1. Just let parameters be optional
If your actions represent distinct logical operations, then make sure they have a different route/path. The routing system will help you out here and you don't have to build any infrastructure to support it. All parameters in MVC are optional. If you want to treat some of them as required then validate them inside the action code and return BadRequest(..).
If you're doing this, you probably have a pretty clear separation between "this thing doesn't exist" (404) and "you didn't provide all the required data" (400).
Obviously you can't always do this because you might have compatibility requirements with the URL space. You also may not want to do this because you don't think it's the right thing to do, but this is basically what the framework provides out of the box.
2. Manually dispatch to different actions
If you want to write separate methods to handle distinct logical operations, but want them all to be routed to the same action, then you need to write some ugly dispatching code inside that action.
If you want to try and bind different parameters based on that "logical operation" that you're handling, the framework won't do anything to help you with this. This is a little painful, but depending on the scale you're operating at, might be an acceptable level of boilerplate.
The difference between 1 and 2 here is that in 1, your separation of logical actions is in your business layer, not in the controller layer.
3. Extend action selection
We actually planned for this, we just didn't want to put it in the box.
There's a new interface IActionConstraint which is a much more powerful version of ActionMethodSelectorAttribute from ye olde MVC. An IActionConstraint get to vote yes/no for a particular action, and having an IActionConstraint make an action "better" then a method without. This is how filtering based on HTTP verb works in MVC.
Here's where I'd start to implement this:
public interface IRequiredParameterMetadata
{
    // Just a marker
}

public class RequiredQueryStringAttribute  : Attribute, IBindingSourceProvider, IRequiredParameterMetadata
{
    public BindingSource BindingSource => BindingSource.Query;
}

public class RequiredQueryParametersConstraintConvention : IActionModelConvention
{
    public void Apply(ActionModel action)
    {
        var required = action.Parameters
            .Where(p => p.Attributes.OfType<IRequiredParameterMetadata>().Any())
            .Select(p => p.BindingInfo?.BinderModelName ?? p.ParameterName)
            .ToArray();
        if (required.Count > 0)
        {
            action.ActionConstraints.Add(new RequiredQueryParametersActionConstraint(required));
        }
    }
}

public class RequiredQueryParametersActionConstraint : IActionConstraint
{
    public RequiredQueryParametersActionConstraint(string[] required)
    {
        Required = required;
    }

    public string[] Required { get; }

    public int Order => 1000; // applied after HTTP methods

    public bool Accept(ActionConstraintContext context)
    {
        for (var i = 0; i < Required.Length; i++)
        {
            var request = context.RouteContext.HttpContext.Request;
            if (!request.Query.ContainsKey(Required[i]))
            {
                return false;
            }
        }

        return true;
    }
}

public class MyController
{
    public IActionResult Action1([RequiredQueryString] int required1, [RequiredQueryString] int required2, string optional)
    {
    }

    public IActionResult Action2([RequiredQueryString] int otherRequired, string optional)
    {
    }
}

Note that this won't disambiguate when multiple actions both have their requirements satisfied. How you want to resolve that is up to you. You could use another attribute to set the priority (and enforce it by setting Order in the action constraint) or you could resolve it based on number of required parameters, which is what webapicompatshim does

great explanation @rynowak :+1: thanks a lot!

Always a pleasure @tugberkugurlu - let us if know if you end up building it, it might be something that others ask for.


"No such file or directory" when trying to merge examples (IDFGH-6220)

Environment

Development Kit:      None
Kit version (for WroverKit/PicoKit/DevKitC): NA
Module or chip used:  ESP32-PICO-D4
IDF version (run git describe --tags to find it): ESP-IDF v4.3.1-dirty
Build System:        idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it): 8.4.0
Operating System:     Windows
(Windows only) environment type: ESP Command Prompt
Using an IDE?: Yes, Visual Studio Code
Power Supply:

Problem Description
I think what I'm dealing with is a user-error, but I can't find the proper way to do what I'm trying to do.
Long story short, I'm taking two examples from esp-idf/examples. One is a2dp_gatts_coex, the other is blehr in the nimble folder.
I've copied the entire example directory (C:\esp\esp-idf\examples\bluetooth\nimble\blehr) into a different location (C:\esp\myProjects\blehr).
My main.c file uses aspects of both examples, and there are #includes from both. The problem is, I can't build it. The debug log section shows the terminal window contents.
I've tried adding the missing inclusions directly into my project, but it's a fools errand that eventually leads me to an inclusion that isn't discoverable (e.g. event/callback.h). It's a bad solution anyway.
Expected Behavior
Project to be built.
Actual Behavior
No such file or directory found.
Steps to reproduce

Copy an example folder into a location outside the esp-idf directory.
Attempt to add requirements (e.g. #include "esp_a2dp_api.h")
Run "idf.py build"

// If possible, attach a picture of your setup/wiring here.
Code to reproduce this issue
GIST
Debug Logs
C:\esp\myProjects\blehr>idf.py build
Executing action: all (aliases: build)
Running ninja in directory c:\esp\myprojects\blehr\build
Executing "ninja all"...
[1/9] Performing build step for 'bootloader'
ninja: no work to do.
[2/7] Building C object esp-idf/main/CMakeFiles/__idf_main.dir/main.c.obj
FAILED: esp-idf/main/CMakeFiles/_idf_main.dir/main.c.obj
ccache C:\Users\aman.espressif\tools\xtensa-esp32-elf\esp-2021r1-8.4.0\xtensa-esp32-elf\bin\xtensa-esp32-elf-gcc.exe -DHAVE_CONFIG_H -DMBEDTLS_CONFIG_FILE="mbedtls/esp_config.h" -DUNITY_INCLUDE_CONFIG_H -DWITH_POSIX -Iconfig -I../main -IC:/esp/esp-idf/components/newlib/platform_include -IC:/esp/esp-idf/components/freertos/include -IC:/esp/esp-idf/components/freertos/port/xtensa/include -IC:/esp/esp-idf/components/esp_hw_support/include -IC:/esp/esp-idf/components/esp_hw_support/port/esp32/. -IC:/esp/esp-idf/components/heap/include -IC:/esp/esp-idf/components/log/include -IC:/esp/esp-idf/components/lwip/include/apps -IC:/esp/esp-idf/components/lwip/include/apps/sntp -IC:/esp/esp-idf/components/lwip/lwip/src/include -IC:/esp/esp-idf/components/lwip/port/esp32/include -IC:/esp/esp-idf/components/lwip/port/esp32/include/arch -IC:/esp/esp-idf/components/soc/include -IC:/esp/esp-idf/components/soc/esp32/. -IC:/esp/esp-idf/components/soc/esp32/include -IC:/esp/esp-idf/components/hal/esp32/include -IC:/esp/esp-idf/components/hal/include -IC:/esp/esp-idf/components/esp_rom/include -IC:/esp/esp-idf/components/esp_rom/esp32 -IC:/esp/esp-idf/components/esp_rom/include/esp32 -IC:/esp/esp-idf/components/esp_common/include -IC:/esp/esp-idf/components/esp_system/include -IC:/esp/esp-idf/components/esp32/include -IC:/esp/esp-idf/components/driver/include -IC:/esp/esp-idf/components/driver/esp32/include -IC:/esp/esp-idf/components/esp_ringbuf/include -IC:/esp/esp-idf/components/efuse/include -IC:/esp/esp-idf/components/efuse/esp32/include -IC:/esp/esp-idf/components/xtensa/include -IC:/esp/esp-idf/components/xtensa/esp32/include -IC:/esp/esp-idf/components/espcoredump/include -IC:/esp/esp-idf/components/esp_timer/include -IC:/esp/esp-idf/components/esp_ipc/include -IC:/esp/esp-idf/components/esp_pm/include -IC:/esp/esp-idf/components/vfs/include -IC:/esp/esp-idf/components/esp_wifi/include -IC:/esp/esp-idf/components/esp_wifi/esp32/include -IC:/esp/esp-idf/components/esp_event/include -IC:/esp/esp-idf/components/esp_netif/include -IC:/esp/esp-idf/components/esp_eth/include -IC:/esp/esp-idf/components/tcpip_adapter/include -IC:/esp/esp-idf/components/app_trace/include -IC:/esp/esp-idf/components/mbedtls/port/include -IC:/esp/esp-idf/components/mbedtls/mbedtls/include -IC:/esp/esp-idf/components/mbedtls/esp_crt_bundle/include -IC:/esp/esp-idf/components/app_update/include -IC:/esp/esp-idf/components/spi_flash/include -IC:/esp/esp-idf/components/bootloader_support/include -IC:/esp/esp-idf/components/nvs_flash/include -IC:/esp/esp-idf/components/pthread/include -IC:/esp/esp-idf/components/esp_gdbstub/include -IC:/esp/esp-idf/components/esp_gdbstub/xtensa -IC:/esp/esp-idf/components/esp_gdbstub/esp32 -IC:/esp/esp-idf/components/wpa_supplicant/include -IC:/esp/esp-idf/components/wpa_supplicant/port/include -IC:/esp/esp-idf/components/wpa_supplicant/include/esp_supplicant -IC:/esp/esp-idf/components/perfmon/include -IC:/esp/esp-idf/components/asio/asio/asio/include -IC:/esp/esp-idf/components/asio/port/include -IC:/esp/esp-idf/components/bt/common/osi/include -IC:/esp/esp-idf/components/bt/include/esp32/include -IC:/esp/esp-idf/components/bt/common/api/include/api -IC:/esp/esp-idf/components/bt/common/btc/profile/esp/blufi/include -IC:/esp/esp-idf/components/bt/common/btc/profile/esp/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/porting/nimble/include -IC:/esp/esp-idf/components/bt/host/nimble/port/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/ans/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/bas/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/dis/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/gap/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/gatt/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/ias/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/ipss/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/lls/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/services/tps/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/util/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/store/ram/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/nimble/host/store/config/include -IC:/esp/esp-idf/components/bt/host/nimble/nimble/porting/npl/freertos/include -IC:/esp/esp-idf/components/bt/host/nimble/esp-hci/include -IC:/esp/esp-idf/components/cbor/port/include -IC:/esp/esp-idf/components/unity/include -IC:/esp/esp-idf/components/unity/unity/src -IC:/esp/esp-idf/components/cmock/CMock/src -IC:/esp/esp-idf/components/coap/port/include -IC:/esp/esp-idf/components/coap/port/include/coap -IC:/esp/esp-idf/components/coap/libcoap/include -IC:/esp/esp-idf/components/coap/libcoap/include/coap2 -IC:/esp/esp-idf/components/console -IC:/esp/esp-idf/components/nghttp/port/include -IC:/esp/esp-idf/components/nghttp/nghttp2/lib/includes -IC:/esp/esp-idf/components/esp-tls -IC:/esp/esp-idf/components/esp-tls/esp-tls-crypto -IC:/esp/esp-idf/components/esp_adc_cal/include -IC:/esp/esp-idf/components/esp_hid/include -IC:/esp/esp-idf/components/tcp_transport/include -IC:/esp/esp-idf/components/esp_http_client/include -IC:/esp/esp-idf/components/esp_http_server/include -IC:/esp/esp-idf/components/esp_https_ota/include -IC:/esp/esp-idf/components/protobuf-c/protobuf-c -IC:/esp/esp-idf/components/protocomm/include/common -IC:/esp/esp-idf/components/protocomm/include/security -IC:/esp/esp-idf/components/protocomm/include/transports -IC:/esp/esp-idf/components/mdns/include -IC:/esp/esp-idf/components/esp_local_ctrl/include -IC:/esp/esp-idf/components/sdmmc/include -IC:/esp/esp-idf/components/esp_serial_slave_link/include -IC:/esp/esp-idf/components/esp_websocket_client/include -IC:/esp/esp-idf/components/expat/expat/expat/lib -IC:/esp/esp-idf/components/expat/port/include -IC:/esp/esp-idf/components/wear_levelling/include -IC:/esp/esp-idf/components/fatfs/diskio -IC:/esp/esp-idf/components/fatfs/vfs -IC:/esp/esp-idf/components/fatfs/src -IC:/esp/esp-idf/components/freemodbus/common/include -IC:/esp/esp-idf/components/idf_test/include -IC:/esp/esp-idf/components/idf_test/include/esp32 -IC:/esp/esp-idf/components/jsmn/include -IC:/esp/esp-idf/components/json/cJSON -IC:/esp/esp-idf/components/libsodium/libsodium/src/libsodium/include -IC:/esp/esp-idf/components/libsodium/port_include -IC:/esp/esp-idf/components/mqtt/esp-mqtt/include -IC:/esp/esp-idf/components/openssl/include -IC:/esp/esp-idf/components/spiffs/include -IC:/esp/esp-idf/components/ulp/include -IC:/esp/esp-idf/components/wifi_provisioning/include -mlongcalls -Wno-frame-address   -ffunction-sections -fdata-sections -Wall -Werror=all -Wno-error=unused-function -Wno-error=unused-variable -Wno-error=deprecated-declarations -Wextra -Wno-unused-parameter -Wno-sign-compare -ggdb -Og -fstrict-volatile-bitfields -Wno-error=unused-but-set-variable -std=gnu99 -Wno-old-style-declaration -D_GNU_SOURCE -DIDF_VER="v4.3.1-dirty" -DESP_PLATFORM -MD -MT esp-idf/main/CMakeFiles/__idf_main.dir/main.c.obj -MF esp-idf\main\CMakeFiles__idf_main.dir\main.c.obj.d -o esp-idf/main/CMakeFiles/__idf_main.dir/main.c.obj   -c ../main/main.c
In file included from ../main/main.c:21:
../main/bt_app_av.h:13:10: fatal error: esp_a2dp_api.h: No such file or directory
#include "esp_a2dp_api.h"
^~~~~~~~~~~~~~~~
compilation terminated.
ninja: build stopped: subcommand failed.
ninja failed with exit code 1

You're trying to use two different Bluetooth host stacks at the same time which is not supported

Would that be Bluedroid and NimBLE at the same time?

Correct

I'm not trying to use Bluedroid, but I am trying to have Bluetooth Classic and NimBLE coexist. Do you happen to see where I could "turn Bluedroid off" for lack of better wording?

Bluetooth classic features require bluedroid

Got it. I'm way off then. Thank you.


Seasonality extraction using weekly data

Hi,
looking into the prophet code, the yearly seasonality period is hardcoded at 365.25.
The paper also says

Let P be the regular period we expect the time series to have (e.g.
P = 365.25 for yearly data or P = 7 for weekly data, when we scale our time variable in days

Since MMM data is usually on a weekly basis, there should be some mismatch in seasonality extracted by Prophet unless it rescales the data as if it was on a daily basis with missing values in between?

Hi, looking at this thread from prophet, my understanding is that we're good/don't need to specify freq as long as we're not running make_future_dataframe.

please reopen if necessary


Reduce max speed when lift is above set point

Especially from reverse to forward, as this is the only action that seems likely to be a tipping hazard.


How can I change tab bar height? I think on README.md screenshot it is smaller

Download the last update on github and try again

Download the last update on github and try again

That is the last update on GH. I've cloned it just before the screenshot :D

This is on my Firefox 114.0.1

For some reason Monterey theme looks fine. I'll just stick to it probably


--shm-size: inconsistencies

$ docker run -d ubuntu top
09606c3c2e31df6c9c5c67783ef2d950c8ac836d9b734329cb6eddf0eb425b1a
$ docker inspect 096 | grep -i shm
        "ShmSize": 67108864,

So far, so good CLI side...
$ docker run -d --shm-size="-1" ubuntu top
docker: --shm-size: invalid SHM size.
See 'docker run --help'.

-1 is not even parseable by units.RAMInBytes so we get invalid SHM size.. This can be good maybe.
docker run -d --shm-size=0 ubuntu top
docker: --shm-size: SHM size must be greater than 0 . You specified: 0 .
See 'docker run --help'.

With 0, RAMInBytes passes but 0 is not valid. We are hitting https://github.com/docker/docker/blob/master/runconfig/parse.go#L211 and the condition cannot be clearly catch the strictly < 0 because it's catched by the error in RAMInBytes two lines above..
So the CLI applies the default 64MB only if the flag is not passed. It will error out (with different errors?!) if the value is invalid.
API side
$ curl -X POST -H "Content-Type: application/json" -d '{"Image":"busybox", "Cmd": "mount", "HostConfig":{"ShmSize": -1}}' http://localhost:2375/containers/create
{"Id":"975c16bf9e3fd94eb9afb1f7737f1c78bc90ff4d0939dc7090e5b979d1277923","Warnings":null}
$ docker start -i 975 | grep shm
shm on /dev/shm type tmpfs (rw,seclabel,nosuid,nodev,noexec,relatime,size=65536k)

On the API I can pass -1 and it won't bother about the fact it's invalid so it applies the default 64MB. Should it error out instead much like on the cli?
Plus:
$ docker inspect 7c125 | grep -i shm
        "ShmSize": -1,

Inspect shows an shm size of -1 which seems incorrect..(to me)
Same goes if I provide 0 to the API, inspect will report 0 and 64MB will be applied (instead of erroring out)
Lastly, if you don't provide an HostConfig at all it will default to 64MB and report 64MB in inspect :
$ curl -X POST -H "Content-Type: application/json" -d '{"Image":"busybox", "Cmd": "mount"}' http://localhost:2375/containers/create
{"Id":"975c16bf9e3fd94eb9afb1f7737f1c78bc90ff4d0939dc7090e5b979d1277923","Warnings":null}
$ docker start -i 975 | grep shm
shm on /dev/shm type tmpfs (rw,seclabel,nosuid,nodev,noexec,relatime,size=65536k)
$ docker inspect 975 | grep -i shm
        "ShmSize": 67108864,

(I'm sure I've even forgot something to report lol, but we'll see)
I think validation should be either the same for cli and API or API side only.. (much like we're currently doing with things in verifyPlatformContainerSettings, which will save the overwritten value so inspect is consistent)
ping @LK4D4 @calavera @duglin @cpuguy83  WDYT we should go with this? is the current behavior ok? I'll fix it if we find a common ground and this is not ok.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I vote for API side only, but I'm open to check both sides.

@calavera I agree with API side only.

+1 API

+1 to API side validations, keep the client thin and don't duplicate validation logic (which can result in getting out of sync)

What about what inspect should show? I think the issue here lies in the fact we let invalid values to use the default value.
IMO:

it should not be possible to fallback to the default if value provided is invalid, just error out everytime (no fallback) unless:
if no value is provided in the CLI, use the default, and inspect will show the default (64MB)
if no value is provided in the API (this is tricky), use the default, and inspect will show the default (64MB)
if no hostConfig is provided (which is a case above), use the default and inspect will show the default

I hope I have covered everything (I'll add integration tests as well since none are present right now)


[Bug]: forwardRef is not defined client-side error in Next.js production builds

Package Scope
@suspensive/react
Bug description
After updating to 1.20.10 the app throws a client-side error
ReferenceError: forwardRef is not defined

Expected behavior
No client-side error
To Reproduce
Use the ErrorBoundary component anywhere in the app. Only happens in production builds, webpack is used for both (no --turbo flag in dev).
Using Next.js 14.0.4 with standalone output and outputFileTracingRoot set (monorepo setup likely not required to reproduce).
Possible Solution
Could be issues with treeshaking/minification, that's what 1.20.10 introduced. Downgrading to 1.20.9 resolves the issue.
etc.
No response

I reproduced it in my next.js repo I found bug too. I made fixed new version in v1.23.2
but, Could you check whether v1.23.2 fix your issue please too?

If this version resolve your issue, close this issue please. Thanks for your help!

@minsoo-web @okinawaa Check it out please.


[BUG] Unable to restore latest backup from backup page

Describe the bug
We are unable to restore the latest backup from the backup page.

To Reproduce

Navigate to the backup page.
Attempt to restore the latest backup.
Observe that the restoration process fails or does not initiate.

Expected behavior
The latest backup should be successfully restored when selected from the backup page.
Support bundle for troubleshooting
supportbundle_7904f20b-1bf8-4f06-92ef-c69b71830e9c_2024-12-23T12-00-17Z.zip
Environment

Longhorn version: v1.8.x-head
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of control plane nodes in the cluster:
Number of worker nodes in the cluster:

Node config

OS type and version:
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context

Workaround and Mitigation

@roger-ryao Can it work by kubectl?

@mantissahz Please check the issue.

Hi @roger-ryao,
Can you check if it is still valid for v1.8.x-head because https://github.com/longhorn/longhorn-manager/pull/3393 just merged in 2 hours ago ?

Verified on master-head 20241224

longhorn master-head https://github.com/longhorn/longhorn/commit/7cd6412137171db36e06eeae82e97364558a877d
longhorn-manager master-head https://github.com/longhorn/longhorn-manager/commit/718b6b4d3db6e4b3a0a06e00d4053b915a2a9530

Verified on v1.8.x-head 20241224

longhorn v1.8.x-head https://github.com/longhorn/longhorn/commit/ede8f46a75704352e33fbc45b01f14fa2770a827
longhorn-manager v1.8.x-head https://github.com/longhorn/longhorn-manager/commit/e8b5292131205725745468c61d00200986107dbf

The test steps
https://github.com/longhorn/longhorn/issues/10046#issue-2755974865
Result Passed


ROCm updates due to cccl third_party and 6.1 test errors.

This PR updates the proper include files to be added on ROCm. The below are the changes performed.

This fixes hipRTC issues that occur due to CCCL.
Fixes for cupyx lapack import error.
Fixes for carray.cuh files.

The carray.cuh got modified to use CCCL thrust instead of the files that are already available in include/cupy folder.

Full test run:
https://amdcloud-my.sharepoint.com/:u:/g/personal/csrikris_amd_com/ES5G8Khs4qJFgQzIYaV4BE8BSDUQBJe3p-aH51TOUagNIA?e=OfSTYS


line-buffered stdout where appropriate

Based on the suggestion in #18, this change adopts grep_cli to detect
when to line-buffer output, and adds support for line-buffering to the
writers module.
This change accomplishes this by:

Plumbing through "flush on newlines" information into FileHandles for
stdout, when stdout is a tty.
Allowing write requests to trigger a flush of the underlying writer.

Note: Even after this change we do not have the same behavior as most Awk implementations, where in line-buffering mode the  program processes input one line at a time. We still buffer substantially on the input side. This PR unblocks needed changes in the parts of the code that handle writing output (and fixes the linked issue whereby we were not appropriately buffering writes to stdout). For full "interactive" output, we need to make some changes to the read side as well.


Fix Connections Instead of "Save-As"

In the guides for the flows provided for this solution it recommends you "Save As" each of the flows and then run them. Other than it being easier to save-as, is there any drawback to manually going into each flow and fixing broken connections so that the flow remains within the solution? Does the solution still function correctly if you fix connections and turn on the flow within the solution?

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

ID: 9fe69498-a485-5379-97ff-53fbe029f11a
Version Independent ID: 5b3f9705-9e49-1575-4ae7-0293d12b739c
Content: Set up core components - Power Platform
Content Source: power-platform/guidance/coe/setup-core-components.md
Service: power-platform
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

Yes, the solution will still function correctly if you turn on the flows inside the solution. The other drawback is that if you choose to make customizations to any of the flows, they would get overwritten during an upgrade.
Hope that helps, and thanks for using the CoE Starter Kit!


Add line marker snapshot testing

Add a liner marker ico for snapshot testing and with an action to navigate to the snapshot.
This is going to be snapshot testing provided by https://github.com/spatie/pest-plugin-snapshots plugin.
Few things that needs to be discovered before this can be completed

How are snapshot filenames actually generated? (with data providers and so on) @freekmurze 😉
How should the icon look like? @caneco 🙏

Snapshots are saved in a directory __snapshots in the base test directory.
General naming rule: <name of test file>__<name of test>__<order number>.txt
That order number is 1 is there's only one snapshot assertion in a test.
Some of the snapshots methods will produce other extensions (json, xml,  html, ...)
In the screenshot you can see a few examples (and also how snapshots in datasets are named)

If you have any further questions let me know.

Current issues I am having with this is figuring out what test it is part of.
I think we might do some limitations here with the icons, so it is only shown if it is inside the root of the test for first iteration.
Meaning the following will not work

This is currently part of the EAP.
If you are interested in trying it out, use the EAP channel or manual install the following file.
Install it manually using
Preferences > Plugins > ⚙ > Install plugin from disk...
PEST_PHP-1.1.0.zip


Add support for treating generic method parameters as dynamic in all contexts

The current implementation of generic method syntax in the VM and dart2js treats generic type parameters as malformed types, which get mapped to dynamic in some (but not all) contexts.  In particular, they are not usable in is or as checks.  In order to remove the comment syntax fully from strong mode, we will need an option in the VM and dart2js to enable a mode in which all uses of generic method parameters are mapped to dynamic.

I have sent a cl for review addressing this in the VM. I did not add another mode, and the new behavior is what you get with --generic-method-syntax.

Actually, I recommended a change to the vm implementation in #27437 such that it would treat a method type parameter as dynamic when performing checked mode checks. The rationale was that the intention behind type annotations is that the dynamic behavior of the program should satisfy the implied constraints. Hence, in a correct program there should never be a failure during such a check, and silently succeeding will not introduce bugs into correct programs.
An is check is different: It is perfectly plausible that a correct program uses an is check in such a way that it should evaluate to false in certain situations. This means that it would be misleading for an implementation to silently evaluate x is T to true in all cases, it would silently introduce logical bugs into correct programs. That's the reason why dart2js raises a runtime error on is and is! checks with method type parameters.
The as check falls between these two extremes. We could put it into any of the buckets ("it's just like a checked/strong mode check on a type annotation" vs. "it's a dynamic check, just like is"), and it will currently raise an error in dart2js.
In short, I'm not convinced that we should let x is T return true in all cases.

The CL https://codereview.chromium.org/2388843002/ implements a new semantics for o as T in dart2js: It is checked as if T had been dynamic. I haven't changed the semantics of o is T, based on the worries I described in the previous comment.

Note that we haven't actually settled what the semantics should be.

I'm not sure allowing is checks on generic method arguments is a good idea. It won't work in dart2js and in the VM and there isn't even a good default way of dealing with it. If it's really much easier to deal with in the VM, then we can/should go this route, but if it doesn't make a big difference, then I would rather discourage users from doing is/as checks on these arguments until the language actually handles them.
Independently reopening since only the VM would satisfy the original request (and this issue has also been filed for dart2js and the language).

Yes, it's really not ideal that we will get different behavior on the VM/dart2js vs DDC for these checks, but unfortunately we're already in that state because of the comment syntax.  I did some code searching, and I can't find any uses of the comment syntax for is checks (thankfully!).  There are many uses of the comment syntax for as checks.  So I think we could get away with the compromise position of treating uses of generic parameters in is checks as an error, but allowing them everywhere else.
We may want to consider issuing a static warning for is checks on generic method parameters for the time being as well (i.e. in the analyzer).

Removed area-dart2js and area-vm from the bug.
We shouldn't have bugs that cover multiple areas, and there is no reason to include them as long as we don't have an agreement in the language team.
Yes. I think is checks should not be allowed.
The as checks are probably to work around the type system and could be ignored. (Would be nicer to have Picard, since that's a better indication of "I don't actually want the check").

We discussed this in the language meeting this morning, and will follow up with clarification shortly.  Sorry for the confusion.

We have an informal spec for this. I believe that the VM implementation is already compatible with this. @crelier can you please verify, and close this bug if so?
https://docs.google.com/document/d/1nccOSkdSyB9ls7TD8gUqIu6Wl2J_9bTSOoMCEIoAiQI/edit#heading=h.dnanv15dn1me

According to the revised informal spec, the errors resulting from using a function type parameter in some contexts are always static errors, and not compile time or run time errors. Therefore, the implementation in the VM is free to map a function type parameter to dynamic in all contexts, which it does today.
Closing this issue.

Very good, thanks!


⚠ Bofrost.at has degraded performance

In c6f416a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1707 ms

Resolved: Bofrost.at performance has improved in 9b639f5 after 13 minutes.


Can We Configurable axis title location ?

according to #5218  now in chart js 3.5.1 can we configurable axis title location ?

not possible by default, can only use the align to make place it at the start, center or end of the scale.
You can use a custom plugin to draw it yourself: https://jsfiddle.net/Leelenaleee/L2vhwe6u/29/

Closing since this appears to be asking for the same thing as #5218

默认情况下不可能，只能使用 align 将其放置在比例尺的开始、中心或结束处。
可以使用自定义插件自己绘制：https : //jsfiddle.net/Leelenaleee/L2vhwe6u/33/

Ask a question, although this can achieve the effect, but it seems that it can not pass the type detection of ts

That's because there are no type declarations so you will need to add those if you want to use it in TS, or alternatively you can TS ignore it


TypeScript type of call's return value doesn't match runtime type

This compiles. My editor says the return type of call(print, ...) is [StackVar<"i32">].
const print = importFunc(
  { in: [i32, i32, i32, i32, i32, i32, i32], out: [i32] },
  () => {}
);

const printCentered = func(
  { in: [i32, i32, i32, i32, i32, i32, i32], out: [i32], locals: [] },
  ([text, x, y, color, fixed, scale, small]) => {
    const [width] = call(print, [text, 0, -6, color, fixed, scale, small]);
    const newX = i32.sub(x, i32.div_s(width, 2));
    return call(print, [text, newX, y, color, fixed, scale, small]);
  }
);

However, it fails at runtime with TypeError: call is not a function or its return value is not iterable.
Changing
-    const [width] = call(print, [text, 0, -6, color, fixed, scale, small]);
+    const width: any = call(print, [text, 0, -6, color, fixed, scale, small]);
+    console.info(width);

makes it work and prints { kind: 'stack-var', id: 16, type: 'i32' } which is indeed not an array.

Thanks for finding that, a type bug I will fix

Fixed and released as 0.2.3


Implement, test and document store=True and metadata=... mechanisms

I also haven't tested and documented the store=True and metadata=... mechanisms.
Plus there's no way to get BACK the metadata/stored content yet.

Originally posted by @simonw in https://github.com/simonw/llm/issues/191#issuecomment-1703564265

I'm going to modify the return value of collection.similar() to be a list of "entry" namedtuples, where each namedtuple has id and score and metadata and content.
Those last two can be null for entries that didn't store those things.

I really need to refactor the llm embed command to use the new Python API.

Updated docs: https://github.com/simonw/llm/blob/de6d257dde0fc2107707f94c2aa07f01b981d345/docs/embeddings/python-api.md#retrieving-similar-items


FastAPI errors, is the "mySql" the problem or problem in code? -->  File "/usr/local/lib/python3.8/site-packages/fastapi/utils.py", line 129, in create_response_field raise fastapi.exceptions.FastAPIError( fastapi.exceptions.FastAPIError: Invalid args for response field! Hint: check that <class 'sqlalchemy.orm.session.Session'> is a valid pydantic field type

This question may have been asked before, but as I have been googling it and not found any solutions, i feel i need to ask here..
So, im trying to my FastAPI restapi server(which is up and running), so my "mySql" database using SQLalchemy.
I think i have come pretty far, but im getting this Error:
File "/usr/local/lib/python3.8/site-packages/fastapi/utils.py", line 129, in create_response_field raise fastapi.exceptions.FastAPIError( fastapi.exceptions.FastAPIError: Invalid args for response field! Hint: check that <class 'sqlalchemy.orm.session.Session'> is a valid pydantic field type
So before posting some of the snippets from my code, i have some questions:

is my SQL string correct?
Do i need BaseModel? is it necessary?
How do i solve this error?

Code snippets:
my database_config.py:
from sqlalchemy import create_engine
from sqlalchemy.ext.declarative import declarative_base
from sqlalchemy.orm import sessionmaker

SQLALCHEMY_DATABASE_URL = "mysql+pymysql://user:pssword@ip/the_database"

engine = create_engine(
    SQLALCHEMY_DATABASE_URL
)

SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)

Base = declarative_base()

my database_model.py:
from database_config import Base

from sqlalchemy import Boolean, Column, ForeignKey, Integer, String
from sqlalchemy.orm import relationship
from datetime import datetime

class user_profiles(Base):
    __tablename__ = "user_profiles"

    user_ID = Column(Integer, primary_key=True, index=True)
    email_adress = Column(String, unique=True)
    age = Column(Integer)
    sex = Column(Integer)
    height = Column(Integer)
    weight = Column(Integer)
    main_goal = Column(Integer)
    level_experience = Column(Integer)
    profile_created_at = Column(default=datetime.utcnow())

    work_out_plan_task = relationship("workout_plan_task", back_populates="user_profiles")

my database_schema.py:
from pydantic import BaseModel

class a_user_profile(BaseModel):
    user_ID : int
    email_adress : str
    age: int
    sex: int
    height: int
    weight: int
    main_goal: int
    level_experience: int
    profile_created_at: str

    class Config:
        orm_mode = True

my main.py:
from sqlalchemy.orm import Session
from fastapi import FastAPI

from database_config import *
from database_schemas import *

Base.metadata.create_all(bind=engine)

from database_models import user_profiles

app = FastAPI()

# Dependency
def get_db():
    db = SessionLocal()
    try:
        yield db
    finally:
        db.close()

@app.get("/") # works perfectly, and i ge the "Hello" Fastapi message
def home():
    return {"Hello": "FastAPI"}

#@app.get("/user_profiles", response_model=a_user_profile)
@app.get("/user_profiles", response_model=a_user_profile)
def get_users(db: Session):
    return db.query(user_profiles).all()

You are returning a list of response model, use response_model=List[a_response_model]

Here you are trying to get all the records from user_profiles. But restricting to give response as a single record.
please change
response_model=a_user_profile --> response_model=List[a_user_profile]
this may resolve your problem

Seems i might have solved it!
I changed to this:
@app.get("/users/", response_model=List[schemas.a_user_profile])
def show_records(db: Session = Depends(get_db)):
    records = db.query(models.user_profiles).all()
    return records

Now the error i get is:
 File "/usr/local/lib/python3.8/site-packages/sqlalchemy/util/compat.py", line 182, in raise_
    raise exception
sqlalchemy.exc.NoForeignKeysError: Could not determine join condition between parent/child tables on relationship user_profiles.work_out_plan_task - there are no foreign keys linking these tables.  Ensure that referencing columns are associated with a ForeignKey or ForeignKeyConstraint, or specify a 'primaryjoin' expression.

So i just need to change a little bit at my model.

Edit: finally solved.

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue @vlori2k  👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Incorrect Links on Actions, Resources, and Condition Keys for AWS Resource Access Manager

In the Actions Defined by AWS Resource Access Manager section, all the links in the table (in the Action column, such as CreateResourceShare or DeleteResourceShare) that are supposed to link to the API actually all redirect here: https://docs.aws.amazon.com/AWSEC2/latest/APIReference/Welcome.html.
This is because all the RAM links have since been moved to a different section.  The old links still point to  https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_CreateResourceShare.html where they should point to https://docs.aws.amazon.com/ram/latest/APIReference/API_CreateResourceShare.html
(ram should be the new high level path directory instead of AWSEC2)

Thanks for finding this! The list of actions, resources, and conditions that we publish in the IAM user guide is automated based on content from each service. Thanks for letting us know that this service includes incorrect or out of date data. I will forward this information to the service's engineering team so that they can add the missing information.


Browser.origin missing from package

Please list the package(s) involved in the issue, and include the version you are using
@shopify/web-pixels-extension
Describe the bug
browser.origin interface missing from package build but present in the type defs
Steps to reproduce the behavior:
Inside of register callback, the BrowserAPI type definition includes a browser.origin.get() method to capture the origin of the URL (protocol, hostname, and port).. which is useful for capture the window location when the script loads on page.
Without this property, you can't get the window.location on initial load and have to try to disambiguate from subscribing to user interactions like page_viewed
Expected behavior
Have an API to get the original browser location to capture things like query parameters

I see now that the interface is accurately described here, although this is much different from type definitions in the package


Minor: add StreamlitCallbackHandler to import and all

I just wanted to add in the import for this callback handler. Thanks for making it!
Who can review?
@hwchase17
@agola11

fixed in other pr


Pipe error when calling vscode from WSL

Is it possible use vscode and pipes from WSL?
From a WSL, I called
alan@local:~$ cmd.exe /c 'code --list-extensions'|grep ms

and I get
net.js:320
      err = this._handle.open(fd);
                         ^

Error: EINVAL: invalid argument, uv_pipe_open
    at new Socket (net.js:320:26)
    at createWritableStdioStream (internal/process/stdio.js:179:18)
    at process.getStderr [as stderr] (internal/process/stdio.js:40:14)
    at Object.get (internal/console/constructor.js:158:40)
    at Object.Console.<computed> (internal/console/constructor.js:276:46)
    at Object.warn (internal/console/constructor.js:287:61)
    at writeOut (internal/process/warning.js:27:3)
    at process.onWarning (internal/process/warning.js:79:3)
    at process.emit (events.js:223:5)
    at internal/process/warning.js:52:24
    at processTicksAndRejections (internal/process/task_queues.js:76:11) {
  errno: -4071,
  code: 'EINVAL',
  syscall: 'uv_pipe_open'
}

VSCode Version: 1.50.0
OS Version: Windows 10.0.19041

It seems working on version 1.63.2.


Please invite me to the GitHub Community Organization

Name
Devarshi

@MrKrishnaAgarwal

Hey @devarshishimpi
I will send you invite myself
Before I had a github action, but I didn't want anyone to join.
You can join it.
Sending you 👋

Hey @devarshishimpi I will send you invite myself
Before I had a github action, but I didn't want anyone to join. You can join it. Sending you 👋

Okay! Thanks


Strange behavior when using WithLatestFrom on reactive properties

Example:
void Start()
{
    A = new ReactiveProperty<int>(1);
    B = new ReactiveProperty<int>(2);

    Debug.Log(A.Value);
    Debug.Log(B.Value);

    A.WithLatestFrom(B, (a, b) => (a, b))
        .Subscribe(ab => Debug.Log(ab.a + ab.b));
}

Expected output:
1
2
3
Actual output:
1
2
Strangely, if I add a DelayFrame operator on A as seen below, the expected output is produced:
void Start()
{
    A = new ReactiveProperty<int>(1);
    B = new ReactiveProperty<int>(2);

    Debug.Log(A.Value);
    Debug.Log(B.Value);

    A.DelayFrame(1).WithLatestFrom(B, (a, b) => (a, b))
        .Subscribe(ab => Debug.Log(ab.a + ab.b));
}

I'm sure this can be explained but I don't understand it and it's certainly not intuitive. Is this an issue or am I misunderstanding something about how WithLatestFrom works?

Based on the marble diagram it looks like this is how WithLatestFrom is supposed to work:
https://rxjs-dev.firebaseapp.com/api/operators/withLatestFrom
You don't get a value on subscription, you get it after you have subscribed and the left value (A) emits.  The reason your second example works is because the left value is emitting 1 frame after the subscription.


Compile error on CentOS 7

When following the build instructions from the README on a fresh CentOS 7 machine:
$ git clone git@github.com:Netflix/dynomite.git
$ cd dynomite
$ autoreconf -fvi
$ ./configure --enable-debug=yes
$ make
$ src/dynomite -h

The following build error appears:
In file included from dyn_core.h:131:0,
                 from dyn_server.c:26:
dyn_server.c: In function ‘server_deinit’:
dyn_server.c:123:25: error: ‘s’ undeclared (first use in this function)
     ASSERT(TAILQ_EMPTY(&s->s_conn_q) && s->ns_conn_q == 0);
                         ^
dyn_util.h:203:11: note: in definition of macro ‘ASSERT’
     if (!(_x)) {                                \
           ^
dyn_server.c:123:12: note: in expansion of macro ‘TAILQ_EMPTY’
     ASSERT(TAILQ_EMPTY(&s->s_conn_q) && s->ns_conn_q == 0);
            ^
dyn_server.c:123:25: note: each undeclared identifier is reported only once for each function it appears in
     ASSERT(TAILQ_EMPTY(&s->s_conn_q) && s->ns_conn_q == 0);
                         ^
dyn_util.h:203:11: note: in definition of macro ‘ASSERT’
     if (!(_x)) {                                \
           ^
dyn_server.c:123:12: note: in expansion of macro ‘TAILQ_EMPTY’
     ASSERT(TAILQ_EMPTY(&s->s_conn_q) && s->ns_conn_q == 0);
            ^
dyn_server.c: In function ‘server_rsp_forward’:
dyn_server.c:908:45: warning: conversion to ‘long unsigned int’ from ‘int64_t’ may change the sign of the result [-Wsign-conversion]
         uint64_t delay = dn_usec_now() - req->request_send_time;
                                             ^
dyn_server.c: In function ‘req_send_done’:
dyn_server.c:1011:41: warning: conversion to ‘int64_t’ from ‘usec_t’ may change the sign of the result [-Wsign-conversion]
     msg->request_send_time = dn_usec_now();
                                         ^
dyn_server.c: In function ‘req_server_enqueue_imsgq’:
dyn_server.c:1029:55: warning: conversion to ‘int64_t’ from ‘usec_t’ may change the sign of the result [-Wsign-conversion]
     msg->request_inqueue_enqueue_time_us = dn_usec_now();
                                                       ^
dyn_server.c: In function ‘req_server_dequeue_imsgq’:
dyn_server.c:1060:40: warning: conversion to ‘long unsigned int’ from ‘int64_t’ may change the sign of the result [-Wsign-conversion]
     int64_t delay = dn_usec_now() - msg->request_inqueue_enqueue_time_us;
                                        ^
dyn_server.c:1060:35: warning: conversion to ‘int64_t’ from ‘long unsigned int’ may change the sign of the result [-Wsign-conversion]
     int64_t delay = dn_usec_now() - msg->request_inqueue_enqueue_time_us;
                                   ^
dyn_server.c:1061:5: warning: conversion to ‘uint64_t’ from ‘int64_t’ may change the sign of the result [-Wsign-conversion]
     histo_add(&ctx->stats->server_queue_wait_time_histo, delay);
     ^
make[3]: *** [dyn_server.o] Error 1
make[3]: Leaving directory `/root/dynomite/src'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/root/dynomite/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/root/dynomite'
make: *** [all] Error 2

I have the following prerequisites installed:

autoconf-2.69-11.el7.noarch
libtool-2.4.2-21.el7_2.x86_64
libevent-2.0.21-4.el7.x86_64
openssl-devel-1.0.1e-51.el7_2.7.x86_64

System information:
Linux dynomite-01 3.10.0-327.36.3.el7.x86_64 #1 SMP Mon Oct 24 16:09:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

It seems I recently broke it: https://github.com/Netflix/dynomite/commit/e285e2ffd88348529857011d3783fa4415e626b0

BTW, please try without the debug flag as well.

Compile is now successful. Thanks!


UIMARCAUTH-359 Don't validate hrid start number for FOLIO source files (follow-up)

Description
Don't validate hrid start number for FOLIO source files because they don't have this field
Issues
UIMARCAUTH-359

Hey @Terala-Priyanka, this is a follow-up request, so the CHANGELOg entry is already there


Issue: Prisma internal error if providing filters without data

Here the issue is that if you have a filter with only null (undefined) values for fields that are non-nullable, Prisma will crash as that is not a valid query.
It would be nice if Prisma AppSync silently "collapsed" the filter to prevent a mistake, or alternatively, gave the proper error. The "cause" bit in that screenshot is my own change to the library in order to demonstrate the issue.
This only happened while dynamically generating the query through front-end code, of course.

@Tenrys In theory, you should already be able to access the appropriate underlying error from your Terminal (CloudWatch on AWS). Please let me know if this is not the case.
The GraphQL response is intentionally designed to provide only a high-level summary of the underlying error. We avoid displaying explicit details in the response for security reasons, as we do not want to expose internal information to anyone querying your API.
My initial thoughts on this issue are that we should not suppress errors silently, as users might assume their query was successful when it was actually invalid.
Ideally, we would report a more specific error, but this would be a significant undertaking since Prisma Client does not generate simple error messages. As a result, we would need to identify non-nullable fields from your schema, pass this information to the runtime, and parse all queries to verify and report their validity.

it is visible on cloudwatch, but for the other developers on my team trying to use the API I'm building for them, it is time consuming having to dig up the logs for that request every time something does crash. I would like to have the option to show more raw details at least if there was a way to flag the API as being "in development" or such.

We could imagine a new client option to enable unfiltered errors in GraphQL response. However, since it is not a good security practice, I would name it accordingly:
const prismaAppSync = new PrismaAppSync({ unsecureGraphQLErrors: true })


Cleanup error msg for interrupted system call

I've not been able to reproduce the original issue so I'm going to trust you that this fixes it. Thanks!


Feature: Support SHOW TABLE STATUS Statement

What would you like to be added:
Why is this needed:
https://dev.mysql.com/doc/refman/8.0/en/show-table-status.html
SHOW TABLE STATUS
    [{FROM | IN} db_name]
    [LIKE 'pattern' | WHERE expr]

@王亚峰

assigness me