GOnetstat/port resource fail to notice net.ipv6.bindv6only - validation fails when dual mode ipv4/6 port is available

When sysctl net.ipv6.bindv6only is 0 (the default value), ports shown as tcp6 in netstat output are may be actually listening on both protocols. If I start with a Goss check like this:
port:
  tcp:12345:
    listening: true

And I then create a simple container, exposing a port on all interfaces (ipv4 and ipv6), like so:
$ docker run -d -p 12345:80 --name goss-test nginx

Then I'll find that I can correctly use the listening port via ipv4:
$ nc -z <IP_ADDRESS> 12345 ; echo $?
0

However, goss will not validate the port correctly:
$ goss validate
F

Failures/Skipped:

Port: tcp:12345: listening:
Expected
    <bool>: false
to equal
    <bool>: true

Total Duration: 0.016s
Count: 1, Failed: 1, Skipped: 0

What this boils down to is: goss is ensuring netstat's output, not the actual state of the ports.
Isn't netstat generally deprecated in favour of ss, because ss does a better job of not confusing the user in ipv6 situations like this one? e.g.
$ sudo ss -lapn | grep 12345
tcp    LISTEN     0      512                   :::12345                :::*      users:(("docker-proxy",25454,4))

versus
$ sudo netstat -lapn | grep 12345
tcp6       0      0 :::12345                :::*                    LISTEN      25454/docker-proxy

(ss is more correct; this is a tcp port, not a tcp6 specific one)

GOnetstat isn't really "netstat" per say, but rather a low level library that parses /proc/net/* files.
I'll look into the impact of net.ipv6.bindv6only on the proc files and see if there's a way to better handle this setting. But I can't make promises on timeframe, since I'm not sure how easy it will be for me to find this information.
That said, if you or anyone else knows how net.ipv6.bindv6only is handled behind the scenes in ss or impacts /proc/net/, it would be a major help and speed up the fix greatly.

Maybe it will be useful:
Try to get info about port 2200
$ netstat -lapn | grep 2200
tcp        0      0 <IP_ADDRESS>:2200            <IP_ADDRESS>:*               LISTEN      -
tcp        0      0 <IP_ADDRESS>:2200     <IP_ADDRESS>:53786    ESTABLISHED -
tcp        0    200 <IP_ADDRESS>:2200     <IP_ADDRESS>:52288     ESTABLISHED -
tcp6       0      0 :::2200                 :::*                    LISTEN      -

Convert decimal 2200 to hex
$ printf "%x\n" 2200
898

Find out information about 0898 in /proc/net/tcp
$ cat /proc/net/tcp | grep 0898
  16: 00000000:0898 00000000:0000 0A 00000000:00000000 00:00000000 00000000     0        0<PHONE_NUMBER> 1 ffff88345d94e0c0 99 0 0 10 -1
  94: DEC6940A:0898 8568800A:D21A 01 00000000:00000000 02:000087B6 00000000     0        0<PHONE_NUMBER> 2 ffff8801327640c0 48 3 30 10 -1
 242: DEC6940A:0898 EA52800A:CC40 01 000000BC:00000000 01:00000017 00000000     0        0<PHONE_NUMBER> 4 ffff88188853c7c0 24 3 17 10 15

And in /proc/net/tcp6
$ cat /proc/net/tcp6 | grep 0898
   4:<PHONE_NUMBER>0000000000000000000000:0898<PHONE_NUMBER>0000000000000000000000:0000 0A 00000000:00000000 00:00000000 00000000     0        0<PHONE_NUMBER> 1 ffff885fd7238080 99 0 0 2 -1

Headers of /proc/net/tcp
$ cat /proc/net/tcp | head -n 1
  sl  local_address rem_address   st tx_queue rx_queue tr tm->when retrnsmt   uid  timeout inode

Headers of /proc/net/tcp6
$ cat /proc/net/tcp6 | head -n 1
  sl  local_address                         remote_address                        st tx_queue rx_queue tr tm->when retrnsmt   uid  timeout inode

st value for LISTENING state is 0A
and local_address keep IP:PORT data in hex
so try to find out listening ipv6 ports from /proc/net/tcp6:
$ cat /proc/net/tcp6 | grep " 0A " | awk '{print $2}' | cut -d: -f2  | xargs -I %% printf "%d\n" 0x%%
5672
3306
11211
22
2200

and verify this information with netstat:
$ netstat -tunap6 | grep LISTEN | awk '{print $4}' |rev |cut -d: -f1|rev
(Not all processes could be identified, non-owned process info
 will not be shown, you would have to be root to see it all.)
5672
3306
11211
22
2200

similar approach with /proc/net/tcp for ipv4
And with docker ports
# docker ps
CONTAINER ID        IMAGE                                                COMMAND                  CREATED             STATUS              PORTS                     NAMES
1f142a08c558        mysql:5.6                                            "docker-entrypoint.sh"   9 hours ago         Up 9 hours          <IP_ADDRESS>:29999->3306/tcp   some-mysql

# sudo netstat -lapn | grep 29999
tcp6       0      0 :::29999                :::*                    LISTEN      7631/docker-proxy

# cat /proc/net/tcp6 | grep " 0A " | awk '{print $2}' | cut -d: -f2  | xargs -I %% printf "%d\n" 0x%% | grep 29999
29999

ss does not print out the correct values either, it just says tcp for all, for example:
netstat:
$ netstat -lna|grep 8080
tcp6       0      0 :::8080                 :::*                    LISTEN

ss:
ss -lna |grep 8080
tcp    LISTEN     0      128      :::8080                 :::*
tcp    LISTEN     0      0        :::8080                 :::*

Copied from duplicate thread:
Random thought.. maybe it would be simpler to get rid of the whole tcp: vs tcp6: thing all together.
We can collapse the tests into two categories: tcp (default) and udp, if the user wants to make more specific assertions, they can use the ip: attribute.
The only downside is if a user wants to test one or the other, it would be an awkward syntax:
port:
  tcp:22:
    listening: true
    ip:
      and:
      - not:
          contain-element: '::'
      - contain-element: <IP_ADDRESS>

Thoughts on this approach? I'm just thinking out-loud on this.. not 100% sure it's a good idea yet.
@frezbo Brought up a good point about this complicating the syntax. It seems like a trade off, the syntax would be simpler for just checking if a port is listening, but more complicated if you're trying to test that it's listening only tcp but not tcp6 and vice versa.

Okay, I have an idea:
Things we know: If net.ipv6.bindv6only is 0 then anything bound to all interfaces will be presented to the OS as listening on the IPv6 address: "::" but it's also listening on the IPv4 address "<IP_ADDRESS>".
Processes bound to specific addresses aren't bound to both IPv6 and IPv4 unless specified.
So, my thought is that the simplest way to get the expected behaviour from Goss would be to do something like this in the port code:
....
	v6only, _ := sysctl.Get("net.ipv6.bindv6only")
	....
	net = "tcp6"
	for _, entry := range netstat {
		if entry.State == "LISTEN" {
			port := strconv.FormatInt(entry.Port, 10)
			ports[net+":"+port] = append(ports[net+":"+port], entry)
			if entry.Ip == "::" && v6only == "0" {
				entry.Ip = "<IP_ADDRESS>"
				ports["tcp:"+port] = append(ports["tcp:"+port], entry)
			}
		}
	}
....

I know this may not look ideal immediately, but considering the alternatives (try to find and integrate another go package or use the above syntax) I think it probably is the MVP of achieving the output we expect from the test. I would need to investigate around the errors thrown by sysctl.Get() and the value returned on error.
Any thoughts or critique?

Processes bound to specific addresses aren't bound to both IPv6 and IPv4 unless specified.

Are you sure this is true? I haven't tinkered enough with it to know.. but if the bindv6only affects only wildcard IPs, then your solution would be sufficient.

By that statement I mean that if I run say tomcat bound to an interface such as: <IP_ADDRESS>:8080 it wouldn't be (this is an assumption) bound to an IPv6 address unless specified.
I may have to do some testing around this though tbh. Maybe it's possible to bind to a specific IPv4 address but bind to all IPv6 interfaces which would result in "::" but shouldn't output <IP_ADDRESS> in tcp port binding. as I would expect the process to report that port 8080 is already bound to.
I'll double check it, I'm confused too now...

After added  the "spew.Dump(system.Ports())" in the system/port.go file then could get all  ports information on the machine.
func NewDefPort(port string, system *System, config util.Config) Port { 	p := normalizePort(port) 	spew.Dump(system.Ports()) 	return &DefPort{ 		port:     p, 		sysPorts: system.Ports(), 	} }
For example, the output of the ss command:
ss -lna|grep 9090 LISTEN     0      128                      :::9090                    :::*
the above will print like as the flowing:
....  (string) (len=9) "tcp6:9090": ([]GOnetstat.Process) (len=1 cap=1) {   (GOnetstat.Process) {    User: (string) (len=4) "root",    Name: (string) "",    Pid: (string) "",    Exe: (string) "",    State: (string) (len=6) "LISTEN",    Ip: (string) (len=2) "::",    Port: (int64) 9090,    ForeignIp: (string) (len=2) "::",    ForeignPort: (int64) 0   } ....
So the goss.yaml could as
port:   tcp6:9090:     listening: true     ip:     - '::'

Is it finally implemented ?

I've marked this ticket as "help wanted" a while back. I would be interested in knowing which tools handle this correctly. See my comment above regarding netstat and ss https://github.com/aelsabbahy/goss/issues/149#issuecomment-290194285


Update specs2-core, specs2-scalacheck to 4.12.9

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 4.8.3 to 4.12.9.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, semver-minor

Superseded by #295.


Increase verbosity of output

It would be nice if the program could display

"Treating sheet number X",
"Looking for doi / abstract for paper P",

and the like, possibly with checkmarks / crosses when the doi / abstract is found or missing.

Actually, even outputting the logs into a separate file could be quite useful. Recording the number of doi / abstract found would also be useful, cf.
https://github.com/popbr/abstract_doi_finder/blob/5c654607f9822e9839851e4312f5cb0ad979bd40/abstract_doi_finder/src/main/java/popbr/AbstractDoiFinder.java#L281-L282

This is now done:
https://github.com/popbr/abstract_doi_finder/blob/9fe507324975a1a7c5b1737db33eb4c1c1dea6dc/abstract_doi_finder/src/main/java/popbr/AbstractDoiFinder.java#L462-L469
We could give to the user the option record this information in a file (a log file), but leave that for future work.


Order price not correct

The market order price is always '0.00000000', for example:
{'symbol': 'BTCUSDT', 'orderId': 87970972, 'clientOrderId': 'vNG6LpxYRwdz9X8MIGCzvV', 'transactTime':<PHONE_NUMBER>813, 'price': '0.00000000', 'origQty': '0.01000000', 'executedQty': '0.01000000', 'status': 'FILLED', 'timeInForce': 'GTC', 'type': 'MARKET', 'side': 'BUY'}

what is the function you are calling?

I'm using order_market_buy() and order_market_sell().

I have the same issue when I call order_market_buy().

I think it's probably due to Binance's API. I have used ccxt, the same problem exists there.


Implementing Auto Login with SSO

Laravel Version: 5.8
Adldap2-Laravel Version: 6.0
PHP Version: 7.1
LDAP Type: ActiveDirectory

Description:
Hi,
I am currently developing an API which allows our company staff to login the website without entering their user credentials. The user credential will be fetch on the background when user open the web application.
I found that actually SSO can do the trick but I quite lost about how to implement it into the Laravel.
I check my Windows Authentication has been enabled but I don't know what to do next.
Do you have any tutorial which can help me to understand more how to implement it?

Steps To Reproduce:

Hi @WeeHong, I'd be glad to help you out, though I'll need to know a couple things first.
Are you using IIS as your web-server? If so, there's a walk through guide here to enable it on your IIS instance:
https://docs.microsoft.com/en-us/iis/configuration/system.webServer/security/authentication/windowsAuthentication/providers/add
As well as some documentation I created here:
https://adldap2.github.io/Adldap2-Laravel/#/auth/middleware
Basically, once you enable Windows Authentication on your IIS application instance and you visit that site while logged in on a domain joined computer that your server is also on, the users domain account will be authenticated against it, and if successful, PHP will set a server variable equal to the domain users username (`$_SERVER['AUTH_USER']).

@stevebauman
Thank you for providing such informative detail.
I would like to try it out first and get back to you as soon as possible.
I am also planning to write an article on Medium if possible.
Thank you.

No problem @WeeHong! Let me know if you need any more help. I'd love to read your article once your done too!
I'll re-open this issue if you need anything else.

Hi @stevebauman
Sorry I have a dumb question about this.
If I am using Ngnix or Apache2 to host my website, I should use another solution right?
Thanks.

Hi @WeeHong,
Unfortunately I'm not sure, I've never set up SSO on Nginx or Apache, only IIS on Windows Server. Sorry!


I added a doggo command! Hope it works.

Only posted the url rn tho so maybe file in the future.

url is fine


experimental unity files packaged, config removed in favor of default config to copy, modified readme and demo

I removed the individual unity files in favor of a unitypackage which is built using the demo project. This prevents duplicate files in the git and discourages making changes to the scripts without testing them in a Unity environment.

The Demo project has been updated accordingly
The Readme has been edited

Additionally the vinter.config.json has been ignored and removed so local changes like IPs will not trigger a change in git.

added another commit with readme changes for the new package


Update pages/about.md

Updated exec descriptions for Social Officer and Treasurer

Accepted and merged, thanks!


Add test case for createResponse() with non-empty reason phrase

Test case for the ResponseFactory::createResponse() method in case the reason phrase is non-empty.

Coverage increased (+0.1%) to 94.658% when pulling d76910f3d3f1090b766222c9cf907749f8ccfcfe on adriansuter:patch-responseFactory into 6b4729702e990cd9c72b5ed9115434e5f0af3df5 on slimphp:master.


Missing parameters for get request on list messages.

On the List Messages beta endpoint there's some new (I think added within the last week or two) pagination logic.
This requires some query params on the request, but there's not a way to add those currently. I monkeypatched it so I can continue testing, but I'd be happy to open a PR with a larger fix if that's worth doing.
Is there a way you'd like for me to approach this in general, and if so, how? Here's my quick and dirty code using the ActiveSupport Hash#to_query method.
module OpenAI
  class Messages
    # Add parameters to the list method.
    def list(thread_id:, parameters: {})
      query = parameters.present? ? "?#{parameters.to_query}" : ""
      @client.get(path: "/threads/#{thread_id}/messages#{query}")
    end
  end
end

Definitely open to a PR on this, thanks. Generally I prefer code that's

Simple
Reusable (eg if there are other endpoints that have pagination)
Works by default - the less config / setup users have to do the better
Easy to understand, especially in how to use it
Documented (in the README)
Written so a junior developer can understand it and use it

Thanks a lot!

Released in v7


Broken link issues

Came across two typos:

In SETUP.md's Intro section, the bookmark link for GCP is broken.
In CONTRIBUTING.md, the link to the FAQ page is broken.

I can take these up and submit a PR if needed.

Thanks for pointing those out and for the PR Anchal! Link stability on GH is a constant problem. :)


About the cross origin

I've also come across a cross origin issue.
Maybe you can try my solution: Set effect to true, so that logseq loads the plugin with the file protocol.
https://github.com/logseq/marketplace/pull/122

Thank you, @haydenull.  I've created a pull request.  I'll let you know how it works out.

@haydenull — I appreciate your taking the time to share this.  It appears to have solve my issue.


Library makes assumption that unsecured connection would never use authentication

I ran into this issue pretty quickly with my first test email.
It is possible for a server to require authentication, even if implicit TLS or STARTTLS is not used.
I would suggest that the client check for 250-AUTH from the server and use the credentials set by the user of the library regardless of port, TLS, STARTTLS, etc.

This is part of version 1.1.7.


How to manually issue a certificate

What is your question?

Hello, I am planning to use this package to replace go-acme/lego, but I don't know how to manually issue the certificate like lego:
// code to get a lego.Client

httpProvider, err := webroot.NewHTTPProvider(path/to/nginx/wwwroot)
if err != nil {
	return err
}

err = client.Challenge.SetHTTP01Provider(httpProvider)

request := certificate.ObtainRequest{
	Domains:    domains,
	Bundle:     true,
	MustStaple: false,
}

certificates, err := client.Certificate.Obtain(request)
if err != nil {
	return err
}

I only need to issue the certificate and do not use domain on this issuance process, can you give me a demo similar to lego?
What have you already tried?

I tried looking through the README and issue lists, and found GetACMEChallenge method, but I don't know how to use it.
Include any other information or discussion.

The code I wrote using go-acme/lego is here: https://github.com/TheTNB/panel/blob/main/pkg/acme/client.go
Bonus: What do you use this package for, and does it help you?

When using certmagic to replace lego, dozens of dependent packages will be deleted from my go.mod.

Ah I think you want ACMEz, which is the package CertMagic uses under the hood!
Here's an example you can run with pebble: https://github.com/mholt/acmez/blob/master/examples/porcelain/main.go

Thanks, I will try.

I guess you can also use the ObtainCert*() methods directly: https://pkg.go.dev/github.com/caddyserver/certmagic#Config.ObtainCertSync


Add missing meta tags to the landing page

Context
https://dfedigital.atlassian.net/browse/HFEYP-398
Changes proposed in this pull request
These meta tags are in the content pages, but not in the landing page
Guidance to review
View the source for the landing page and check that this is included
<meta property="og:url" content="https://help-for-early-years-providers.education.gov.uk:3000/" /><link href="https://help-for-early-years-providers.education.gov.uk:3000/" rel="canonical" />

Must it match exactly? this is what I'm seeing in page source:


Add live weather reports web page

Hi @avincodes  Please assign me this issue with Hacktoberfest Label. I will use simple HTML, CSS and JAVASCRIPT. It will gives you the weather information about the place you search.

Sure go for it!

I have added Live weather Web Page. Please Check it and accept my PR.


Unsupported Protocol 'workspace:^' in Dependencies Causing Deployment Failure on DigitalOcean

I'm encountering a build error when deploying my MedusaJS project on DigitalOcean due to the use of workspace:^ in my dependencies. The build fails because npm doesn't fully support workspace protocols, which are required by Yarn. Additionally, my package.json specifies engines.node with >=20, but DigitalOcean's build system interprets this as an ambiguous version, leading to further issues.
Error Message:
npm error Unsupported URL Type "workspace:": workspace:^
Deployment Platform: DigitalOcean
Node Version Specified: >=20

Hi @riqwan ,
I am facing same issue Please take up this issue and help me to deploy the B2B starter.


source native_units from daemon

Currently required in config.

I think we might go with something like self.native_units = config.get("native_units", self.client.get_units())
This is because I'd like to actually make native units optional everywhere, but delays in particular "native" is perhaps a bit of a misnomer... we want native to be "ps" (or at least "mm_delay") while the daemons are typically in "mm"

I think we require native_units for delays and disallow it everywhere else

do not like ambiguous behavior, or loosey goosey units


Add support for multiple ion servers in UI

Fixes https://github.com/CesiumGS/cesium-omniverse/issues/562.
Follow up to https://github.com/CesiumGS/cesium-omniverse/pull/578 that adds a cesium ion server selector in the UI. There are still a few UI components that are not addressed in this PR but will come in a follow up:

https://github.com/CesiumGS/cesium-omniverse/issues/563
https://github.com/CesiumGS/cesium-omniverse/issues/565
Dynamically populating the Quick Add assets based on the server

In addition, there are some bugs related to saving and loading:

Saving a .usda with multiple servers for the first time causes a server disconnect
Loading a .usda with multiple servers doesn't resume the connection

Projects with the single default ion server don't seem to have these issues, which is good.
Also...

Creating a new project always requires connecting to ion. That's because there's no existing connection to resume. I think this is expected behavior, but would have to verify with Cesium for Unreal.

Finally, while I started to implement something that would query the ion api url automatically, this didn't really work with the existing code. @r-veenstra any tutorial content will need to be updated so that the api url is filled in instead of left blank. E.g. for https://self-hosted.cesium.com/ it would be https://api.self-hosted.cesium.com/.
Obviously there are a lot of improvements to make here still, but it's certainly usable. Here I add CWT from ion.cesium.com and GTOPO from self-hosted.cesium.com.
https://github.com/CesiumGS/cesium-omniverse/assets/915398/e4216afa-c9aa-4800-b175-c6210bd10386

Creating a new project always requires connecting to ion. That's because there's no existing connection to resume. I think this is expected behavior, but would have to verify with Cesium for Unreal.

This is fixed in https://github.com/CesiumGS/cesium-omniverse/pull/594. It should no longer sign you out each time. What was happening was that the property change notifications here were triggering a disconnect here. Since property change notifications are now ignored after prim add notifications, this no longer happens.


reactor: reactor the code of leafModel

What type of PR is this?
/kind cleanup
What does this PR do?

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

/approve
/lgtm


Timestamps on directories not preserved

Hello,
When using rclone with sync option local to local,
The date and time from the destination is not the prseved as the original file

Thanks for your report.
I just tried this on linux and it does preserve the timestamp when I tried rclone sync local1 local2 and used ls -l to check the timestamps.
Can you tell me

which OS you are using?
which filing system?
how you tell the date and time isn't preserved?

Thanks
Nick

This is the "last modified" time stamp on directories
Using Raspbian, and did not remove empty directories when using sync

Ah ha!  As far as rclone is concerned directories don't actually exist, which means

timestamps aren't preserved
it doesn't remove empty directories when it should

I'd like to fix the second of those, but the first I don't have a plan to fix.
I'll add something to the docs to make it clear the limitations.
Thanks for the report
Nick

I've updated the rclone docs trying to explain these issues a bit better.  See http://rclone.org/bugs/
Thank you very much for the report
Nick


view created/won/entered giveaways

It'd be nice to be able to view your own created, won and entered giveaways (probably entered as most-important), similarly to http://www.steamgifts.com/giveaways/entered
It should also be possible to remove entries from within that list.

This is most likely going to end up being the default action for clicking the header, instead of viewing your public profile.
If it is implemented that way, you'd most likely need a link to the public profile from there.

See also #7


Missing add_ele (cumulative kWh) sensor for power metering socket.

First of all, thanks for this new integration which is MUCH easier than https://github.com/tuya/tuya-home-assistant :clap:
I just tried it because I was missing some power metrics in my circuit breakers and was happy to find them with this integration :) the connection worked well and I have more metrics than with https://github.com/tuya/tuya-home-assistant so far so good.
I only noticed one missing sensor in my case (add_ele) with some connected outlets with power metering (Konyks Priska+ Mini in my case). The add_ele attribute is returning the cumulative kWh, very useful for accurate energy metering.
I checked the code and it is supported for circuit breaker: https://github.com/tuya/tuya-smart-life/blob/33a6ecad44adf2ac2916671f5894c1bcc568145a/custom_components/smartlife/sensor.py#L821-L828
I do have one of those and it's working so cool :sunglasses: (see the kWh number in the sensors, along the 3 classic others W, V, A)

But it's not supported for outlets (only W, V, A here):

While the attribute is available the same way, I can see it in Tuya IoT debug UI:

So I would like to request for this attribute to be added for outlets too, in cz or pc I guess? or maybe kg.
FTR this sensor (add_ele) was also missing in the native Tuya integration so it's not a regression, just would be a nice addition :+1: otherwise I can't use my 3 power metering outlets for energy mesurement in the dashboard.
Thanks, and sorry if my issue is missing important details, I'm new to HA so let me know if there's more debug info/logs that would be useful.

Please provide the diagnostics file, we will check the product.

Actually I noticed a couple other issue with both my circuit breakers (already "supported") and outlets (for which I added "support" with the previously shown copy-pasted config):

Sometimes the add_ele event is missing from the time series, leading to inacurrate results. For example here between 01:00 and 02:00 I have no data point (if I hover I can only see one at 00:58 and one at 02:28):

While the consumption was definitely not 0 and the device seems to be sending those events at least every 30 minutes:

(screenshot from Tuya IoT platform for reference as I don't know where else I can find this).
This happen also with my circuit breakers but is more likely when the value is very stable, so I suspect some intentional "deduplication" logic here is removing "duplicate" entries, thinking it wouldn't change the end result but it does matter as this is only a delta.

I also found warnings in my logs that the sensor is using "TOTAL_INCREASING" while the values is not increasing, and this is very true, this value is only a delta of kWh since previous event, so it's definitely not a total increasing counter. And this might be the reason for 1 (because if it were really a total increasing counter, not storing duplicates would be acceptable). Unfortunatly I don't know to improve this because the only other option if it's not a globally increasing counter is "measurement", but this is apparently not OK with the "energy" type: 2023-12-03 16:19:47.375 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.bureau_add_ele (<class 'custom_components.smartlife.sensor.SmartLifeSensorEntity'>) is using state class 'measurement' which is impossible considering device class ('energy') it is using; expected None or one of 'total', 'total_increasing'; Please update your configuration if your entity is manually configured, otherwise report it to the author of the 'smartlife' custom integration

So I'm not sure how we are supposed to implement sensor values in kWh delta like this one. I was a bit suprised to noticed this type of counter which is implemented in many outlets and counter for years hasn't any obvious  implementation in HA? I'm currently trying to fiddle with some https://www.home-assistant.io/integrations/utility_meter/ and https://www.home-assistant.io/integrations/integration/ simply to get an accurate chart of consumption per hour like anybody would want (like in the Tuya app) but I couldn't get anything correct so far and I feel like I'm fighting with the system here. Maybe I'm missing something obvious here?
At the moment I changed the type to "total" instead of "total_increasing" because it's definitely not increasing, this way I don't have any warning, but this is still not correct because it's not a total and I guess it'll still create problems (like the deduplication).

Yes sure, I guess it's this one? let me know if not ^^

The category of your device is "cz", Not Supported add_ele.
Currently only supported the category of "dlq".
You can read the code.

Mmm yes saw and I did look at the code, that's exactly why I'm writting this ticket suggesting to add add_ele support. Because this outlet (and many others) does support it ^^

+1 for add_ele support.
In my case I have:
    "product_id": "bezptefgpqmax74x",
    "product_name": "6920HA",

Diagnostics attached.
smartlife-91f1e2d812d37fe02c3f041baf262bc3-Laundry socket-16fdc198c9c915024680d18e86a2dd56.json.txt


[BUG]: Having the plugin activated writes three " when typing one instead of two.

Hello, when I have the plugin activated, if I type one ", I get three instead of two. Regardless of the language I am writing in.

I see that, try this 1.1.1-rc1.

Yup, it looks like it's fixed!

Great, so I may release the stable version of 1.1.1 plugin in the next two days.


[Youtube] Cannot download videos using channel search result page

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl --version
2021.06.06
$ youtube-dl -v https://www.youtube.com/c/PBSNewsHour/search?query=Australia
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/c/PBSNewsHour/search?query=Australia']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.7.9 (CPython) - FreeBSD-11.4-RELEASE-p3-amd64-64bit-ELF
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1
[debug] Proxy map: {}
[youtube:tab] PBSNewsHour: Downloading webpage
ERROR: Unable to find selected tab; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/extractor/youtube.py", line 2827, in _real_extract
    return self._extract_from_tabs(item_id, webpage, data, tabs)
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/extractor/youtube.py", line 2752, in _extract_from_tabs
    selected_tab = self._extract_selected_tab(tabs)
  File "/local/file/path/youtube-dl-venv/lib/python3.7/site-packages/youtube_dl/extractor/youtube.py", line 2708, in _extract_selected_tab
    raise ExtractorError('Unable to find selected tab')
youtube_dl.utils.ExtractorError: Unable to find selected tab; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

$

Description

I am attempting to archive the a topical slice of a very large youtube channel by using a the URL of a channel-specific keyword search.  It doesn't work, and the log messages ask for a bug to be reported.  I can use the channel URL to download the whole channel, so it seems logical that I should be able to download from the channel search results.

I have a similar issue, not for a channel but for a search query e.g. https://www.youtube.com/results?search_query=ducks&sp=EgIoAQ%253D%253D.
I tried using ytsearchcc:"ducks" but it throws Unable to download webpage: <urlopen error unknown url type: ytsearchcc>.
What would be a correct way to construct a query search with filters in youtube-dl?

Duplicate of https://github.com/ytdl-org/youtube-dl/issues/29071

What would be a correct way to construct a query search with filters in youtube-dl?

Not possible since the devs search URLs have not been fixed yet (https://github.com/ytdl-org/youtube-dl/pull/27749)

PR #30570.


[feature] Supply agent DeviceIdentity via docker environment variable

I'm interested in this as a AWS/Azure bastion host solution. My only concern is if I have devices spinning up and registering as new clients I would like them registered with a friendly name rather than a mac address. The potential for name collisions can happen but that should be my problem to solve through SSoT or other mechanisms.

What kind of friendly name you would like to have?

Something that will be username friendly. [a-zA-Z0-9-]+ is plenty fine with me. I would prefer an arbitrary string. I would set it through AWS userdata on instance creation and use a script to create the docker container using the values from userdata.

This is indeed a nice idea. For us to design the feature we need to clarify a few aspects: for device identity we need it to be a unique value and this is the reason we adopted the MAC as the source for this data.
Considering you'll be setting this when launching the service, you need to make sure it is unique. Does it sound Ok?

Yup. I plan to use Terraform to generate the systems and have an incrementing serialization. Uniqueness won't be an issue. The collision should only exist within my tenancy on the Shellhub hosted service correct? I would assume the same would apply for the self-hosted option as well?

If you are looking for use cases to justify the LoE how about AWS instances identifying by AWS instances id extracted from userdata API within the host. Would simplify the identification of systems.
I prefer my previous example as a means to help reduce the number of AWS users who have unnecessary access to the corporate account by identifying the system by function, application or deployment tier (prod, int, dev).

@simora you mentioned you intend to use Terraform to install it. Maybe we could add the Terraform Provider at our organization here so more people can benefit from it.
Does it make sense to you?

I was just going to have it install via script after instance creation. Have docker installed and the ami prepped with Packer ready to run. Keep it simple. No need to rewrite what has already been masterfully crafted.


fix timeout being reached on eventlistener

wait for eventlistener to come up
increase timeout from 10s to deployment_timeout
don't return on the first successful/failed deployment
keep a buffer of events just in case deployments are done in subsecond speed (being a bit too cautious here I guess)

Thanks for finding this!


[feature request] Details for artists in Similar Artists view

The right panel could use some more info for each artist.
Currently, i just get a somewhat long list of artists with no idea what kind of music they make. I propose four things:

Make each line a bit higher, so it's easier to click on artists
Add a link/button to each artist that takes you (in a new tab) to the artist on spotify - so I can check them out and see if I like them
Add labels to each artist indicating which genre they belong to (maybe in the style of these labels on github issues).
Have the fuzz_ratio (i guess similarity?) as a setting in the app (with reasonable lower bounds?), maybe as a slider (Lidify.py, line 128)

The aim was to offer a basic search for well-known-ish artists that are not in your Lidarr library, hence the absence of additional information. Unsure about the necessary info for an informed selection.

Make each line a bit higher, so it's easier to click on artists

I'll have a look at that, tried to make it compact so I would also kinda work for smaller devices.

Add a link/button to each artist that takes you (in a new tab) to the artist on spotify - so I can check them out and see if I like them

Sounds reasonable enough.

Add labels to each artist indicating which genre they belong to (maybe in the style of these labels on github issues).

Not a bad suggestion, I'll keep it in mind.

Have the fuzz_ratio (i guess similarity?) as a setting in the app (with reasonable lower bounds?), maybe as a slider (Lidify.py, line 128)

This is only to make sure that the right artist is found when getting the mbid, the recommendations are as per Spotify Related Artists and there is no real control over that as far as I'm aware.
Feel free to jump in with PRs or fork it if you want


Add missing doc for Openshift Build Config for 1.2.x

Purpose

Resolves https://github.com/ballerina-platform/ballerina-lang/issues/22223

Codecov Report

Merging #509 into ballerina-1.2.x will not change coverage by %.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             ballerina-1.2.x     #509   +/-   ##
==================================================
  Coverage              64.26%   64.26%
  Complexity               950      950
==================================================
  Files                     98       98
  Lines                   4900     4900
  Branches                 503      503
==================================================
  Hits                    3149     3149
  Misses                  1530     1530
  Partials                 221      221

Continue to review full report at Codecov.

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


github.com/mailgun/ttlmap is deprecated

I'm trying to build Caddy 2, but Caddy 2 depends on this dependency (vulcand/oxy), which depends on github.com/mailgun/ttlmap, which depends on github.com/mailgun/minheap, which has tests that depends on launchpad.net/gocheck.
A quick breakdown of the dependencies
launchpad.net/gocheck is deprecated. It has been moved to github.com/go-check/check. This isn't the problem, but only one of the tiny things that sucks.
github.com/mailgun/minheap and github.com/mailgun/ttlmap are both deprecated. They are moved to a bigger monorepo in github.com/mailgun/holster. This annoyingly means that all those huge dependencies listed in the go.mod will be fetched along with holster.
github.com/vulcand/oxy (this project) uses ttlmap, which is not only deprecated (as mentioned above), but also drags down the Bazaar dependency hosted on Launchpad for whatever reason.
Solution
Change github.com/mailgun/ttlmap to github.com/mailgun/holster, which has holster.TTLMap.

Made a PR: https://github.com/vulcand/oxy/pull/197

Hello, the main problem with the holster is the dependencies of the holster it-self.
It's a "utils/helper" package with too many unrelated dependencies.

Yeah, I figured that's the problem, and I wrote it in the issue description. I even thought of copy-pasting code and attributing it the appropriate license, but that seems tedious as I don't know how licenses work.
I think it's still better than a Launchpad dependency though. Bazaar is bizarre.

FYI, this repo is not really maintained, I tried several times to get the rights but the admins of the vulcand organization don't answer.
So don't put to much hope in your PR.

Keeping around deprecated dependencies don't sound any better, either. I'm not sure what your solution would be.

I just realized vulcand owns this. Sorry.

Do you plan on forking this and maintaining it there while archiving/deprecating this repository?

It's an old repository: the core design need to be updated with the current Go practices, and the core itself can be rewritten in another way.

Given that Caddy's release candidates are pretty close to completion, is there any faster way than waiting for a rewrite? I personally think it's still usable as-is and just requires some updates.

So you have to fork.


[CHORE] Move FlareExample to framework

rename it to FlareView
it is a base view for showing flare files so it should be delivered within a framework

The reason for a change: People using a framework shouldn't need to implement their own views for loading artboards.

At the moment we're using the FlareCoreGraphics folder for everything related to rendering, thus it'd be better for FlareView.swift to live there as well.
FlareSwift is meant to be the most basic abstraction layer built in Swift.

@umberto-sonnino Done 👍


CAGR Doesn't Match Closely with Portfolio Visualizer

I've been try to get the utils.make_index function to work in Quantstats. First, I cannot get the rebalance parameter to work. Leaving the default value gives an error, so I have to set it to 'None'. The error message:
File "C:\Users\Techie\PycharmProjects\StockAnalysis\venv\Lib\site-packages\quantstats\utils.py", line 400, in make_index
return index[index.index <= last_day].sum(axis=1)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Second, when I set rebalance='None', it will work. But, when I run the reports.html function I get CAGR results that are very different from Portfolio Visualizer. Is there another parameter I need to tweak to match up with PV? Here is my code:
import matplotlib
import quantstats as qs
matplotlib.use('Agg')
qs.extend_pandas()
symbols = {
'SPY': 0.40,
'IEF': 0.40,
'TLT': 0.20
}
port = qs.utils.make_index(symbols, rebalance=None)
qs.reports.html(port, benchmark='SPY', title='Strategy vs. SPY', output='port.html')
Here is a link to the same portfolio when run in Portfolio Visualizer: https://tinyurl.com/2mbsvp4c
I realize PV does it monthly but I'm off by almost 5%. Plus, I cannot do monthly in Quantstats due to the error above on rebalance.
The results vary by quite a bit. I'm adding a screenshot of the results for the Quantstats run. Maybe I have something wrong?

Thanks!

Hi,
I'm facing the same issue with CAGR. and I think that i have found the problem.
the number of years is calculated as
(shortvol.returns.index[-1] - shortvol.returns.index[0]).days/252
the problem with this is that returns a wrong number of years. ex
dt.datetime(2023,7,6) - dt.datetime(2005,12,20)

datetime.timedelta(days=6407)

now 6407/252=25.42 which is wrong.
however 6407/365 = 17.55 which is correct
the fix is quite easy. probably just remove the periods=252parameter and hardcore 365 in the formula. the formula is working in days already, so having a periods parameter can only lead to problems. it should always be 365.
this has a great side effect, now all my backtests will have a higher CAGR 🤪
it's a bit late here, i'll try to submit a pull request in the next few days

Please see my reply to issue #275.
Regards
On Fri, Jul 14, 2023 at 4:31 PM gnzsnz @.***> wrote:

Hi,
I'm facing the same issue with CAGR. and I think that i have found the
problem.
the number of years is calculated as
(shortvol.returns.index[-1] - shortvol.returns.index[0]).days/252
the problem with this is that returns a wrong number of years. ex
dt.datetime(2023,7,6) - dt.datetime(2005,12,20)
datetime.timedelta(days=6407)
now 6407/252=25.42 which is wrong.
however 6407/365 = 17.55 which is correct
the fix is quite easy. probably just remove the periods=252parameter and
hardcore 365 in the formula. the formula is working in days already, so
having a periods parameter can only lead to problems. it should always be
365.
this has a great side effect, now all my backtests will have a higher CAGR
🤪
it's a bit late here, i'll try to submit a pull request in the next few
days
—
Reply to this email directly, view it on GitHub
https://github.com/ranaroussi/quantstats/issues/273#issuecomment-1636466247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJWQW5BU2W7QORWEJ3EPQHTXQG3DVANCNFSM6AAAAAA2A7JMRE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Feedback on RFC 2: Rabbit storage for containerized applications

Need approval from LLNL on the USER and ADMINISTRATOR roles as described in RFC 0002: Rabbit Storage For Containerized Applications.

Since a good chunk of the Containers RFC has been implemented, I'm inclined to close #28 and start converting it to documentation.
There are still some things to resolve, mainly #66 and #67.


remove big blob file type in ps v3

What problem does this PR solve?
Issue Number: close #5558
Problem Summary: We use a special type of BlobFile to handle write which is larger than BlobFile limit size which make the logic a little complicated.
What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-check-issue-triage-complete

/run-all-tests

/run-all-tests

/run-all-tests

/run-integration-test

try to add metric(wip)

need to show the metrics in the grafana panel

try to add metric(wip)

need to show the metrics in the grafana panel

Added. PTAL.

/run-all-tests

/run-all-tests

/merge

/run-unit-test


Why do you set APP_DIRS to False in production?

In the production settings, you do:
TEMPLATES[0].update({"APP_DIRS": False})

Why? This is just out of curiosity. Using git blame, I found it had something to do with using cached templates in production, but I'm not really lcear on how that works.

Sorry, I missed the fact that you have the cache loader in your template loaders there.


Improve README.md

Add more information and clean up the file.  More steps to come as I test on other systems.

Very nice


Errors in loading state_dict for RawGAT_ST

Hi. When I tried to load pretrained model 'Pre_trained_models/RawGAT_ST_mul/Best_epoch.pth',
I got those errors

Hi,
Actually I modified the variable names for git repo (more readable) and in pre-trained model was with usual variables name such in pre-trained model I used GAT_layer1 for spectral GAT and that I modified to GAT_layer_spectral in git clean code that's why you are getting this error.
There are many ways to store weights in new state_dict and then used. Anyways I update git repo to old code which I used for pre-trained model.
Let me know if need further help.
Thanks


feat: create a way to handle errors

considering solidity updates and good semantic practices, it can be:

if (owner != msg.sender) revert NotAllowed();

helps to make error messages more verbose and simple to view code for developers or feedback to users.

Hi @luislucena16
Referring to the identation, I think its a good idea to shorten characters in the code by removing the "{}" of the if else's, from this:
        if (swap.owner != msg.sender) {
            revert InvalidAddress(msg.sender);
        }

...to this:
        if (swap.owner != msg.sender) revert InvalidAddress(msg.sender);

hey @0xneves
yes exactly! it allows to handle errors more organized and with better semantics 💫

Hey guys this issue is available? If so, I would like to get it
@0xneves @luislucena16

hi @0xjoaovpsantos I'll take care of it!

Counting on you, this is a fast one, let's go!

done! PR: https://github.com/blockful-io/swaplace-contracts/pull/157

Could you speed up this resolution @luislucena16?  Since it is blocking issue #104 from developing.
Thank you

done! waiting for your approval


[PERF] Move with_column and exclude function logic to Rust side, add with_columns

This PR includes two things:

a performance improvement to calling (not execution of) DataFrame.with_column and DataFrame.exclude  where projection conversion is done on the Rust side so that costly conversions of the dataframe fields into Python objects are no longer necessary
the addition of the DataFrame.with_columns method which allows adding multiple columns to a dataframe at once. Will also improve performance over calling with_column multiple times. Since I was modifying the with_column function I figured I would add this as a drive-by

I also took this opportunity to clean up some of the rust builder code.

Quick performance results:
Péter's script in the community Slack went from an average of 49.02s to 5.84s per run on my machine, averaged over 5 runs.


Update README.md

Added 3 Resources in UI/UX Interview Section.
IMPORTANT: When creating your Pull Request, please use the following format for the title: Category - Resource Name & Section
PR Title samples:

Web development - CSS Portal & Tools
Web development - Leetcode & Challenges

Fixes Issue

Description:

Resource URL:

Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[ ] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[ ] I followed the prescribed PR title template. (Check this if you're adding a resource)

@jfmartinz Kindly check this PR


Allow flexible configuration of container affinity

Allow more flexible configuration of the container affinity. This change allows for better support of mixed scenarios where the regular worker nodes are protected by the node sensor and only certain containers should be protected by the container sensor, e.g. when using AWS Fargate, Azure Container Instances, Google Cloud Run. Resolves #134

Hello,
Thanks for opening this, but unfortunately, this PR doesn't work as described or intended since it only sets the affinity for the webhook injector service and not the actual sidecar injected sensor. The container sensor uses a mutating webhook which is cluster-wide so setting the affinity for the injection service won't actually tell the sensor to only inject into pods on say fargate-only nodes. This means that in mixed clusters the sensor will be injected into all pods.
Setting sensor.falcon-system.crowdstrike.com/injection to disabled in each namespace/pod or disabling injection by default and setting sensor.falcon-system.crowdstrike.com/injection to enabled in each namespace/pod is the way to do it which is pretty standard/common for kubernetes deployments.

Hello,
Thanks for opening this, but unfortunately, this PR doesn't work as described or intended since it only sets the affinity for the webhook injector service and not the actual sidecar injected sensor. The container sensor uses a mutating webhook which is cluster-wide so setting the affinity for the injection service won't actually tell the sensor to only inject into pods on say fargate-only nodes. This means that in mixed clusters the sensor will be injected into all pods.
Setting sensor.falcon-system.crowdstrike.com/injection to disabled in each namespace/pod or disabling injection by default and setting sensor.falcon-system.crowdstrike.com/injection to enabled in each namespace/pod is the way to do it which is pretty standard/common for kubernetes deployments.

Hi,
Is it actually possible to run falcon helm chart with:
node:
enabled: true
container:
enabled: true
Because in our setup we have issue with using both node sensors and container sensors - using argo for the deployment.
And as I've reached the falcon support they said to me that this is not possible - to have both sensors run simultaneously.
Thanks in advance!

bellaned

Yes. However, this would only be for mixed cluster like EKS with Fargate nodes. Which in that case, you would need to disable the sidecar injection in each namespace or pod not running on Fargate nodes. What is not supported is running both the sidecar in pods that run on nodes that have the sensor also installed.


Support DashJ Core 18

Unit tests are failing, there is a problem with the Linter.  Unit tests depend on Platform Testnet working.


Add RichText debugFillProperties TestCase

Description
Add RichText debugFillProperties test case, Enrich test case coverage.
Tests
I added the following tests:
test/widgets/rich_text_test.dart

@googlebot  I signed it!

@googlebot I fixed it.

@googlebot I fixed it.

Thanks for the PR, Change looks good to me.
Also CC @justinmc


CS8669 - #nullable enable does not work in razor files unless in code block

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
I'm writing a new component based on this:
https://github.com/microsoft/fluentui-blazor/blob/main/src/Microsoft.Fast.Components.FluentUI/Components/Search/FluentSearch.razor
Doing this I got the error CS8669 inside a razor file:

The annotation for nullable reference types should only be used in
code within a '#nullable' annotations context. Auto-generated code
requires an explicit '#nullable' directive in source.

I was able to disable the warning like this:
@{#pragma warning disable CS8669}
<fluent-search @ref=Element
               value=@Value
               current-value="@BindConverter.FormatValue(CurrentValue)"
               @onchange="@(EventCallback.Factory.CreateBinder<string?>(this, __value => CurrentValueAsString = __value, CurrentValueAsString))"
               @oninput="@(EventCallback.Factory.CreateBinder<string?>(this, __value => CurrentValueAsString = __value, CurrentValueAsString))"
               @attributes="AdditionalAttributes">
    @ChildContent
</fluent-search>
@{#pragma warning restore CS8669}

However this will not work:
@{#nullable enable}
<fluent-search @ref=Element
               value=@Value
               current-value="@BindConverter.FormatValue(CurrentValue)"
               @onchange="@(EventCallback.Factory.CreateBinder<string?>(this, __value => CurrentValueAsString = __value, CurrentValueAsString))"
               @oninput="@(EventCallback.Factory.CreateBinder<string?>(this, __value => CurrentValueAsString = __value, CurrentValueAsString))"
               @attributes="AdditionalAttributes">
    @ChildContent
</fluent-search>
@{#nullable disable}

It does not work to add  @{#nullable enable} on top of the file either.
If it is inside a code block everything works as expected:
@code {
    private Timer timer = null;

#nullable enable
    NewFluentSearch? searchTest;
#nullable disable
...
}

Expected Behavior
@{#nullable enable} and @{#nullable disable} should work everywhere inside a razor file where CS8669 and similar errors can occur.
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
7.0.203
Anything else?
ASP.NET Core version: 7.0.5
Visual Studio 2022 Version 17.5.5
.NET SDK:
Version:   7.0.203
Commit:    5b005c19f5
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22621
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\7.0.203\
Host:
Version:      7.0.5
Architecture: x64
Commit:       8042d61b17
.NET SDKs installed:
7.0.203 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

@333fred I assume this is because we wrap basically every chunk of emitted code with #nullable directives?

It appears to be this code that does that:
https://github.com/dotnet/razor/blob/852f223a1c5a7acc160fb85f36215cd0dfe89d55/src/Compiler/Microsoft.AspNetCore.Razor.Language/src/CodeGeneration/CodeWriterExtensions.cs#L750-L753
When I first saw this I was baffled. It disables nullability if SuppressNullabilityEnforcement is false. You'd think it would only disable it if the options said to suppress nullability.
But it appears that this setting actually reflects whether the version of C# in use understands nullability. That property might better be understood as DoNotEmitAnythingRelatingToNullability. It is true if you're on an old (before C# 8, i.e., pre .NET Core 3) language version.
Apparently Razor just has a baked in "we don't do #nullable enable assumption. So it always emits that #nullable disable unless it thinks the version of C# is so old that the compiler won't understand the directive. In either case, its intent is not to have nullability enabled.
But the slightly baffling thing is that in Visual Studio, some of the tooling seems to act as though nullability is in fact enabled. So you can end up getting diagnostics telling you that you've failed to deal with nullability. (Apparently the design-time tooling in the IDE doesn't understand that nullability is in fact disabled.)


fix: fixed is Ref not judge instance

fix issue : #4263

Thank you but isRef already works when it should


not enable to compile with openCL

Hi Martin,
by compiling
cmake .. -DETALER_ENABLE_OPENCL=ON
I always have the following errors (because Catch2 is header file only, I do not understand why do you want to link???)
%] Linking CXX executable etaler_tests_opencl
/usr/bin/ld.gold: error: cannot find -lCatch2::Catch2
collect2: error: ld returned 1 exit status
tests/CMakeFiles/etaler_tests.dir/build.make:122: recipe for target 'tests/etaler_tests' failed
make[2]: *** [tests/etaler_tests] Error 1
CMakeFiles/Makefile2:429: recipe for target 'tests/CMakeFiles/etaler_tests.dir/all' failed
make[1]: *** [tests/CMakeFiles/etaler_tests.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
/usr/bin/ld.gold: error: cannot find -lCatch2::Catch2
collect2: error: ld returned 1 exit status
tests/CMakeFiles/etaler_tests_opencl.dir/build.make:122: recipe for target 'tests/etaler_tests_opencl' failed
make[2]: *** [tests/etaler_tests_opencl] Error 1
CMakeFiles/Makefile2:392: recipe for target 'tests/CMakeFiles/etaler_tests_opencl.dir/all' failed
make[1]: *** [tests/CMakeFiles/etaler_tests_opencl.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

@Thanh-Binh Thanks for reporting the issue.
I don't know why CMake is trying to link against Catch2. CMake should discover that the Catch2::Catch2 target is actually a virtual target that only provides interface information.
Which version of CMake are you using and how is your Catch2 installed?

@marty1885 it works well with this Cmakefile.
I can understand this problem. Because your SW uses only Catch.hpp so that I copy it directly to your test folder (as in your README).
Commonly, we do NOT need to link to Catch2....
Thanks

fixes in #111


Changed serviceversion

Changed the request serviceversion to 1.

I suggest you stay consistent with the naming convention BP-XXXX ticket name ex: BP-1435 Change version 'Transfer' to 1


When i try to connect to the websocket i get this

Connecting(url='https://ws-quiz.hype.space/ws/46541')
Connected(url='https://ws-quiz.hype.space/ws/46541')
Rejected(response=<response HTTP/1.1 503 Service Temporarily Unavailable>, reason='Websocket upgrade failed (code=503)')
Disconnected(reason='closed', graceful=True)
Socket closed

Is anyone else getting this? It seems not. How do i fix this? nothing shows up on google

Sorry for this but i've been stuck on this for days now and this project was last updated months ago so @Exaphis do you know this? (since u made it)

The bot still works perfectly for me. Does this issue occur on another network?

I figured it out. Forgot to replace https with wss 🤦♀

Great! Thanks for letting me know.


Updated the button text

@ritz303

looks good, let's go ahead with prod and prod-beta.

@jeremyary : okay, sound good.  I'll get to work on them.  Thanks again for the help!


Group projects together using workspaces

For example the frontend projects could get grouped together using cargo workspaces

#14


Remove deprecated method calls within templates

Replace calls to boot with start per deprecation warning.

Fixed in #64. Thanks for reporting!


Make flex-padding-* classes more practical in use

Restrict the scope of flex-padding-* classes as follows:
.flex-padding-xs > .flex-col,
.flex-padding-xs > [class="flex-size-"] {
padding: 0.313rem;
}

Completed.


Update typing, tests, apply linters

low coverage of sampler.py (80%) is because pytest-cov doesn't capture functions decorated with numba.jit.


New Ripper ‒ weheartit.com

There's 3 areas I'd like to see supported if possible:

A user's hearts - this is similar to a tumblr blog as it shows any original/self uploaded content and any content the user has re-hearted (re-tweeted, re-blogged, etc.).
Example: https://weheartit.com/sad

A user's uploads - this is very similar to a user's hearts except only original/self uploaded content is shown.
Example: https://weheartit.com/sad/uploads

Collections - user's can create collections (albums) consisting of either their own and/or other user's content.
Example: https://weheartit.com/sad/collections/142270657-sad

There's some unofficial API documentation that might come in handy here.

@bored
Please make any site requests in the sites request issue here

@cyian-1756 apologies, I only made it in a separate issue because #710 was dealt with. Personally I think it makes more sense to handle them separately since issues don't scale well but I'll post it over there, thanks!

Personally I think it makes more sense to handle them separately since issues don't scale well

Yea, but if you have a different issue for every site then other issues end up getting buried


fix: use object for querystring generation

Allows passing an object to the MakeUrl function and using the querystring module from node to generate the string.  This is easier to manage and less error prone.
Still allows taking strings because in some cases we forward the query string from the UI
Fixes ADO#1143

:tada: This PR is included in version 0.321.2 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Access table displayname in templating

As title suggests, in some cases it's useful to be able access the table's displayname. Current use cases that we discovered:

In 2021:google-dataset annotation we have to provide a "name" and we decided to use the page's title for this. Page title in chaise are in <table-display-name>: <rowname> format. We currently have a way to access the rowname, but we had to hardcode the table displayname. So in this case we just need a way to access current table.
In saved query table we are storing the table's name. This is done for lookup to find all the saved queries of the table. We also would like to show this value but given that we're storing the raw table name (and not displayname), we decided not to show it. So in this case, we would need a "lookup" for the displayname by the schema and table name. I should mention that this lookup does not work in case of CFDE. In CFDE we're working with multiple catalogs, while ermrestjs is only introspecting the main catalog that the page is showing.

This is not directly related to https://github.com/informatics-isi-edu/ermrestjs/issues/825 but both discuss about adding extra attributes to templating, so we most probably should consider both while discussing it.

Moved to wishlist.


Inconsistency in $/setTrace

When building a language server I came across an inconsistency between the Specification and VSCode and thought I should log it here first to confirm if the specification or the IDE is at fault.
The specification for a Set Trace Notification says that the method will be
method: '$/setTrace'

In reality I receive the following message (from VSCode):
method: '$/setTraceNotification'

Please could you confirm if this is a bug I should report for VSCode or for the Specification?

Which version of the libraries do you use. This got fixed ~ 1 year ago with this PR https://github.com/microsoft/vscode-languageserver-node/pull/611


Fix upload directory for 0.39 docs

https://github.com/rapidsai/ucxx/pull/215 updated the docs to use the ucxx version, but during branch-0.39 creation, it was reverted due to the update-version.sh script.
This PR fixes the docs version and removes the line from the update-version.sh script.

/merge


Porting Issue

Describe your bug here. If you are modding, try testing your bug in a clean version of the engine instead. Also, if you're using an older version of JSE, please try the latest version/action build. Also, be sure to check the pinned 'JS Engine: Known Issues' issue, to check if your issue hasn't already been found!
So, I was trying to put a mod apparently made in Psych (Creepy Night Pasta) into JS, and I got a crash. I supposed something may be off with the engine or I did something I'm not supposed to, but I'd like a direct answer pls :3 . (BTW, I imported everything as it was, curious as to if it was possible or I'd have to port it a little more directly by editing the files of the OG mod)
Command Prompt/Terminal/Crash logs (if existing)
Uncaught Error: Invalid char 34 at position 198
/opt/hostedtoolcache/haxe/4.3.4/x64/std/haxe/format/JsonParser.invalidChar() [line 318]
/opt/hostedtoolcache/haxe/4.3.4/x64/std/haxe/format/JsonParser.parseRec() [line 96]
/opt/hostedtoolcache/haxe/4.3.4/x64/std/haxe/format/JsonParser.doParse() [line 57]
WeekData.getWeekFile() [line 224]
WeekData.addWeek() [line 192]
WeekData.reloadWeekFiles() [line 180]
FreeplayState.create() [line 83]
flixel/FlxGame.switchState() [line 639]
flixel/FlxGame.update() [line 726]
flixel/FlxGame.step() [line 679]
flixel/FlxGame.onEnterFrame() [line 558]
openfl/events/EventDispatcher.__dispatchEvent() [line 426]
openfl/display/DisplayObject.__dispatch() [line 1398]
Please report this error to the GitHub page: https://github.com/JordanSantiagoYT/FNF-JS-Engine
The engine has saved a crash log inside the crash folder, If you're making a GitHub issue you might want to send that!

Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Linux
Did you edit anything in this build? If so, mention or summarize your changes.
Nop
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.
Yes

that's json issue so check the json file

that's json issue so check the json file

doing just now but neither of them seem to be that weird, unless there are some values that aren't valid or something?

that's json issue so check the json file

doing just now but neither of them seem to be that weird, unless there are some values that aren't valid or something?
(only one I hadn't seen before in week jsons would be 'hiddenUntilUnlocked')

you could be missing a } or something similar

that's json issue so check the json file

doing just now but neither of them seem to be that weird, unless there are some values that aren't valid or something?
(only one I hadn't seen before in week jsons would be 'hiddenUntilUnlocked')

you could be missing a } or something similar

second check, everything seems just fine in the weeks jsons

that's json issue so check the json file

doing just now but neither of them seem to be that weird, unless there are some values that aren't valid or something?
(only one I hadn't seen before in week jsons would be 'hiddenUntilUnlocked')

you could be missing a } or something similar

second check, everything seems just fine in the weeks jsons

obviously something isn't, I can't see what you did wrong so send the file here

that's json issue so check the json file

doing just now but neither of them seem to be that weird, unless there are some values that aren't valid or something?
(only one I hadn't seen before in week jsons would be 'hiddenUntilUnlocked')

you could be missing a } or something similar

second check, everything seems just fine in the weeks jsons
obviously something isn't, I can't see what you did wrong so send the file here

bro I didn't do anything wrong. I literally just copy-pasted from the og mod. Anyway, here are the files.
creepy1.json
creepy2.json
creepy3.json
creepy4.json

creepy1 fixed.json
here I fixed it

creepy1 fixed.json
here I fixed it

well... uh thanks for the help though. Didn't know the difficulties thing had an issue, it seemed to work well on it's own. Sorry for posting an issue so easy to fix, hehe :sweat_smile:

creepy1 fixed.json
here I fixed it

well... uh thanks for the help though. Didn't know the difficulties thing had an issue, it seemed to work well on it's own. Sorry for posting an issue so easy to fix, hehe 😅

np


Add ability to trigger the recent-messages api without restarting the client.

Is your feature request related to a problem? Please describe.

When the connection is dropped and Chatterino reconnects. There's no way to see what messages were sent between the disconnect and reconnect.
Describe the solution you'd like

I would like a menu option and/or a command that shows historical messages obtained using the recent-messages service using a popup similar to the search one.
Describe alternatives you've considered

Not doing it at all
Automatically filling in the messages that couldn't be received. If the API doesn't have a parameter in the request to select the time we want the messages from, there would be a need to filter them based on the timestamps.

Additional context

Currently there is no way to see the recent messages without using logs or restarting. Logs don't show every users' actions.

How about automatically doing so when a reconnect is detected to fill the gap since the disconnect?
I don't know the limits of randers' log API but if an after-time can be specified, that would be 💯 👌 🔥

That would be even cooler.

How about automatically doing so when a reconnect is detected to fill the gap since the disconnect?
I don't know the limits of randers' log API but if an after-time can be specified, that would be 💯 👌 🔥

My API does not support such a feature, but you can easily filter the returned messages by time range/start time yourself using the rm-received-ts tag present on all returned messages.

Duplicates #915, will close the older one and post the conversation here for completeness:

RAnders00 commented on 7 Jan
When reconnecting, there is a short pause where chatterino is not connected to IRC. Messages that are sent during this time are missed by chatterino. Maybe chatterino could keep the old connection alive (maybe for a maximum of 5-10 seconds) OR until the new one is connected and has received the first message? Or maybe consult the endpoint for getting the last messages in a chat like it does on the initial connect, and merge the chat messages with the existing ones.

TETYYS commented on 22 Jan
I think account connections should be kept alive for a minute or two of no usage, this would prevent rapid reconnections if user wants to switch between two accounts multiple times.

@Evanito There is no need for after-time on my API. Messages are strictly in order of time received, and you can use @rm-received-ts to filter messages based on time received.

Would like to take a look into this, its pretty big annoyance for myself right now, but I am not sure what the ideal solution would be...
Keeping "old" connections active in the background would probably be the most seamless solution for the user, maybe actually keeping them open for some time considering the JOIN limits and the annoyances that introduced when switching accounts, if the connection would be kept active you could just immediately switch back to it without joining all channels again.
Not sure if its a actually a great idea to keep multiple duplicate IRC connections open though.
The other option I see is reloading from recent-messages, either on reconnect or by using a menu entry/command, probably a lot easier to implement and less resource-heavy but does not fix account-switching issues.
Would appreciate input on what others see as the best solution :)

re-polling recent messages on irc reconnect seems like the best-of-both-worlds solutions
Would need a way to filter which messages are interesting, and would need a different way to insert the messages. atm we just do "insert at top" which obviously wouldn't be correct on a reconnect


Rename Elastic Search to Elasticsearch (official name)

Thank you for adding support for the Elasticsearch JDBC driver.
I've made a small PR to use the proper/official name (Elasticsearch - one word, no camel case) and updated the JDBC driver to the latest version.
P.S. Each version of Elasticsearch comes with its own JDBC driver. Not sure whether it is possible, but it would be great in the maven artifact to use a placeholder like latest to avoid having to manually update the jar on each release:
<file type="jar" path="maven:/org.elasticsearch.plugin:jdbc:{latest}"/>
Cheers!

Ah, apparently artifact name was also changed (from jdbc to x-pack-sql-jdbc). Fixed.
Unfortunately I can't set version id to RELEASE. Elasticsearch Maven (Nexus?) doesn't support version listing (there is no maven-metadata.xml file in artifact root folder). That's why particular version has to be set.
(dbeaver works with Maven according to http://maven.apache.org/ref/3.2.5/maven-repository-metadata/)

For some reason I didn't get notified on your comments - sorry about that.
I'll see what can be done regarding the maven-metadata.xml and report back (or can create a new issue if you prefer that).
Cheers,

Yep, new issue would be preferable.
Regards


Dependencies and default implementations

Moving zipWithA into Control.Applicative from Control.Monad is a good idea, but it causes an issue.
Since we can't do default implementations in typeclasses yet we have <* and *> in Control.Applicative rather than being in the typeclass and forcing everyone to implement these as well. A similar thing happens in Data.Foldable and Data.Traversable also since we could give a default implementation of each of the other functions in terms of just one.
Now, this causes an issue. Control.Applicative defines zipWithA which requires sequence which is imported from Data.Traversable which imports Data.Foldable for its constraints which imports Control.Applicative for constraints on some functions: traverse_, for_, and sequence_..
So, we can either shuffle the definitions around, e.g. put zipWithA in Data.Traversable or something. We can move <* and *> into the typeclass and force everyone to implement these as well (the implementations that are there are fine, just would have to copy paste it everywhere :-1: ). Or we can try to get default implementations working.
Thoughts or other ideas?

Closing this as I guess we must have figured it out at some point in the last year :wink:


Skip installation of Upstream Community Tasks on Power/Z clusters

Current Behavior
Upstream community tasks are installed on all OpenShift Clusters irrespective of the architecture.
Upstream Community Tasks are maintained by non Red Hat contributors. Hence, it is almost impossible to ensure that the step images in those tasks are supported on all architectures (x86,_64, ppc64le, s390x).
Expected Behavior
Therefore the operator should check the architecture of the cluster (nodes) and skip installation of upstream community task if the platform is not x86_64

one way to handle this is to add a platform check in the fetchCommunityResources. We can skip the community resources if the platform is not x86_64.
https://github.com/openshift/tektoncd-pipeline-operator/blob/a59f72478cf7fc0aabad132fa9299b2dd7a99a48/pkg/controller/config/config_controller.go#L104

@bahetiamit - Could you please have a look at it?

@nikhil-thomas - Created PR https://github.com/openshift/tektoncd-pipeline-operator/pull/488 to skip upstream community tasks installation if arch is P OR Z.  Please review and do let me know if any changes are needed.

@snehlatamohite the check in you pr is correct, but it is in the addons block: https://github.com/openshift/tektoncd-pipeline-operator/blob/f2f3524f29632d9807fd19cc2feb864aede83ab6/pkg/controller/config/config_controller.go#L126
it should be in the fetchCommunityResources block: https://github.com/openshift/tektoncd-pipeline-operator/blob/f2f3524f29632d9807fd19cc2feb864aede83ab6/pkg/controller/config/config_controller.go#L105
could you make a patch.
@savitaashture @vdemeester

@nikhil-thomas - Yes I will so the needful and will raise PR for required changes.

@nikhil-thomas  - I have created PR https://github.com/openshift/tektoncd-pipeline-operator/pull/546 added check in fetchCommunityResources please let me know if I need to remove check from readAddons block?


debug hkube api

This change is

core/algorithm-debug/lib/algorithm-communication/ws.js, line 39 at r9 (raw file):
Previously, NassiHarel (Nassi Harel) wrote…

const encoding = new Encoding({ type: data.encoding });

You already have this._encoding....

It might be a different one by the second debugged algo.

core/algorithm-debug/lib/algorithm-communication/ws.js, line 41 at r9 (raw file):
Previously, NassiHarel (Nassi Harel) wrote…
We have a format to send messages on WS

I didn't think it should be part of the protocol between worker and wrapper. I thought it should act like a connection problem.
I"ll add it and deal with it.


Create &[u8] fast fields

They can use the double array strategy used for multivalued int.
This requires to add another type in the schema.
We can use base64 for its JSON encoding.

@jason-wolfe do you want to take over that one?

Double array strategy but without bitpacking/having to bitunpack the value array entries, right?
Sure! I'll take a look.


Create git tags

To easily see the changes between 2 versions, it would be nice to have the released versions here also as git tags.

We now have git tags and an up to date changelog! 🎉


Regenerate all APIs to pick up generator changes

This commit only contains changes to the client builders, which now
have partial methods to allow interception of the Build/BuildAsync
methods.

Self-reviewing, as this is purely generated. No API surface changes - merging.


Please add Equinix Metal to Cloud Providers List

File: docs/intro/cloud-providers/_index.md
Please add Equinix Metal to Cloud providers list. As well as update the menu item from Packet.net to Equinix Metal

Hi @etrosa
Equinix Metal already exists in this menu listing and packet.net has been removed
We will redirect the packet.net page to the Equinix Metal page this week
Paul

Awesome, Thank you Paul!


Add ColumnWiseEncoder

So we can assign a different encoder for different columns.

Thanks for raising this! Let me take on this. I think a promising approach is to add an argument, col_encoder_dict, in StypeWiseEncoder so that it uses the given StypeEncoder for specific columns while using the default StypeEncoder (specified in stype_encoder_dict) for the rest.
We can support ColumnWiseEncoder, but it sounds a bit overkill to specify the encoder for each column name.


Backport ECDb profile <IP_ADDRESS> to release/3.x

Still work in progress, lots of merge problems during cherry pick.

Can you edit the description to describe the motivation for updating 3.x this late in its maintenance cycle? It reaches end of life in 2 weeks.

Can you edit the description to describe the motivation for updating 3.x this late in its maintenance cycle? It reaches end of life in 2 weeks.

Our policy is to push profile upgrades back to the oldest supported version.  There is an update to BisCore that uses this profile update and there are applications that are waiting on the features enabled by the profile + BisCore update.
The goal of doing this update so late in cycle is to increase the number of apps that can easily support the new profile.  It'd be nice if they were all updated to 4.x but we know some are not and we do not want that to delay the use of the new features enabled by this profile.

After discussion with Colin, we will not do this. It seems too risky so close to end of life of the branch.


english versions of the new before and booking pages

i'm not done doing fixes, just setting it up: it does seem to have a standing conflict though, which i'm wondering after the source of, aside from heading things maybe?

It tells you here:  https://github.com/nextgentravel/travel-guidebook-client/pull/63/conflicts


Remove dbt-postgres and dbt-tests-adapter from dbt-core

Remove  dbt-tests-adapter and dbt-postgres from dbt-core but preserve the adapter one tests in the functional directory to make sure we don't lose any test coverage.
resolves #

Problem

Solution

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[ ] This PR includes type annotations for new and modified functions

In theory, the dbt-postgres tests shouldn't provide us with additional testing signal on dbt-core changes that we wouldn't be able to pick up in our existing functional tests. But the current state of things is less ideal than that 🙃
For example, test_dbt_show tests adapter-specific functionality, but I don't see another test  in tests/functional that includes the --limit argument.
We can't really know the full extent of this without auditing all the tests but we'll definitely lose test coverage by removing them outright at this point, so I agree with the decision to keep them + move to tests/functional.
My main concern is that these will drift from their source definitions in dbt-adapters/dbt-postgres over time as new tests are defined in those directories. Longer term, we should explore options for running the postgres tests on commit / before merge without having to actually define them in dbt-core.

Have you tested if the release will work without any changes?

@emmyoop testing here: https://github.com/dbt-labs/dbt-core/actions/runs/7759825156/job/21164841687

Added a follow-on tech debt issue for dbt-core to delete the adapter tests entirely, and instead trigger dbt-postgres adapter tests to run: https://github.com/dbt-labs/dbt-core/issues/9513


Produksjon

https://nextjs.org/docs/going-to-production

https://github.com/vercel/next.js/blob/canary/examples/with-dynamic-import/pages/index.js

https://github.com/w3bdesign/dfweb-v3/pull/576

https://user-images.githubusercontent.com/45217974/154784575-ec7c0df5-3724-4de0-b8ec-c0ee6ea42f6f.png


Learner read wait too long when region newly split

Bug Report

What version of TiKV are you using?
Almost all version
TiKV
Release Version:   4.0.0-beta.1
Git Commit Hash:   c3ea751b552570bc70940b44d5b81d3955261e01
Git Commit Branch: heads/refs/tags/v4.0.0-beta.1
UTC Build Time:    2020-02-28 12:47:23
Rust Version:      rustc 1.42.0-nightly (0de96d37f 2019-12-19)
Enable Features:   jemalloc portable sse protobuf-codec
Profile:           dist_release
What did happened?
When a region is split, its index is 5 initially. If send read index request to leader immediately, usually 6 will be returned. However, learner will receive raft entry with index 6 after 1~2 seconds. Like log in tiflash
2020.03.09 23:47:13.001333 [ 29 ] <Information> Region: [region 58] split into [79,58]
2020.03.09 23:47:13.328226 [ 87 ] <Debug> Region: [region 79, applied: term 5 index 5] need to wait learner index: 6
2020.03.09 23:47:14.986560 [ 87 ] <Debug> Region: [region 79] wait learner index 6 done
2020.03.09 23:47:14.986619 [ 87 ] <Debug> tpch10.lineitem (SelectExecutor): [Learner Read] wait and get data from memory cost 1658 ms

tikv log
[2020/03/09 23:47:12.982 +08:00] [INFO] [apply.rs:1693] ["split region"] [keys="key<PHONE_NUMBER>000000FF355F728000000000FF65DA1F0000000000FA"] [region="id: 58 start_key:<PHONE_NUMBER>000000FF355F728000000000FF5758320000000000FA end_key:<PHONE_NUMBER>000000FF3600000000000000F8 region_epoch { conf_ver: 2 version: 34 } peers { id: 59 store_id: 1 } peers { id: 60 store_id: 44 is_learner: true }"] [peer_id=59] [region_id=58]
[2020/03/09 23:47:12.982 +08:00] [INFO] [peer.rs:1549] ["insert new region"] [region="id: 79 start_key:<PHONE_NUMBER>000000FF355F728000000000FF5758320000000000FA end_key:<PHONE_NUMBER>000000FF355F728000000000FF65DA1F0000000000FA region_epoch { conf_ver: 2 version: 35 } peers { id: 80 store_id: 1 } peers { id: 81 store_id: 44 is_learner: true }"] [region_id=79]
[2020/03/09 23:47:12.982 +08:00] [INFO] [peer.rs:145] ["create peer"] [peer_id=80] [region_id=79]
[2020/03/09 23:47:12.982 +08:00] [INFO] [raft.rs:861] ["became follower at term 5"] [tag="[region 79] 80"] [term=5] [id=80]
[2020/03/09 23:47:12.982 +08:00] [INFO] [raft.rs:977] ["became leader at term 6"] [tag="[region 79] 80"] [term=6] [id=80]

tiflash proxy log
[2020/03/09 23:47:13.151 +08:00] [INFO] [peer.rs:143] ["create peer"] [peer_id=81] [region_id=79]
[2020/03/09 23:47:13.151 +08:00] [INFO] [raft.rs:723] ["[region 79] 81 became follower at term 5"]
[2020/03/09 23:47:13.151 +08:00] [INFO] [raft.rs:295] ["[region 79] 81 newRaft [peers: [80], term: 5, commit: 5, applied: 5, last_index: 5, last_term: 5]"]
[2020/03/09 23:47:14.984 +08:00] [INFO] [raft.rs:924] ["[region 79] 81 [term: 5] received a MsgHeartbeat message with higher term from 80 [term: 6]"]
[2020/03/09 23:47:14.984 +08:00] [INFO] [raft.rs:723] ["[region 79] 81 became follower at term 6"]

closed by https://github.com/tikv/tikv/pull/8820


Kth Largest

https://leetcode.com/problems/kth-largest-element-in-an-array/

I want to work on this issue..I can provide the solution in C/C++/Python

I want to work on this issue too... I'll use C/C++

https://github.com/HacktoberfestSIT/Problem-Solving/pull/55
@Gourav8152-ai  @deepdhar
Sir , please review

I want to work on this solution using C++

I would like to work on this using JAVA


Rename View() to Page()

This requires us to introduce another base class between RazorPageBase and
Page - you need this because you aren't allow to have Page.Page().

I accidentally made the PR based on dev, but this is actually for preview1.

/cc @Eilon

FYI I double checked this and it looks like there's no dependency in the Razor repo on this. None of the tests call this method.

b1b3a816cc808037d3e6bb45c955b344792a9343 in 2.0.0-preview1
0c70cdb8dfee85043c84a3656c5424992643406f merged into dev


Migrate apply Gradle plugin method from Gradle's legacy imperative apply script method to declarative block method

This project was built with SDK version prior to 3.16, hence it used this method to apply Flutter's Gradle plugins, so while building debug/release apk  said to migrate

Steps to recreate:

run flutter build apk

I would like to fix it so will u accept my PR

Sure.Thanks for the contribution.


[Feature request] Ability to expand path in ensure_no_debug_code

I was trying to do this:
ensure_no_debug_code(
            text: "fcontext(",
            extension: ".swift",
            path: path
)

where path is
"{Framework1Tests,Framework2Tests,Framework3Tests}"

so that it can match the different folders containing the tests for all my frameworks. Unfortunately this fails right now:
Couldn't find the folder at '.../project/{Framework1Tests,Framework2Tests,Framework3Tests}'

I think because it's currently checking if the path exists without expanding it first.
Am I missing an alternate way of doing this? Alternatively path could also take an array of paths.

Does it work if you do
["Framework1Tests", "Framework2Tests"].each do |path|
  ensure_no_debug_code(
            text: "fcontext(",
            extension: ".swift",
            path: path
  )
end

Just curios if the problem is the check because you pass multiple values using  {}

Oh, I guess I should learn how to :gem: first :sweat_smile:

Just curios if the problem is the check because you pass multiple values using {}

Yeah, but what you suggest is totally the way to do it! Thanks :D

@NachoSoto can you check: would this work without the additional raise in the verify block? I assume the Dir["..."] supports your notation, right?

Hey @NachoSoto and @KrauseFx! I am going to close this issue for lack of conversation, but obviously, please continue conversation, and we can reopen if you guys are still interested in discussing. #springcleaning. :sparkle:


Brightness after stop is not the same as settings.

After firing the node the first time everything works like a charm. When sending a stop command and a new start command the brightness has not been reset to the initial values. The color_temp instead is working correctly.

There is no information passed trough via previous nodes. The payload has only the string text off (the node before checks if the lights are off).
If i move the light transition node around and press the save button (only changed nodes save) everything is working correctly, until one time. After that it start over like above.

I will take a look, thanks.

I published a new version (1.4.2) to flows.nodered.org.  This should fix the issue as well as an underlying problem with not resetting any of the data when stopped.

Thanks for your fast response. I will test the new version and send you some feedback when the issue is still there.


Licensing

Is this library using JUCE under the hood? If so how does the licensing work in this regard!?

After having another look through the source, that doesn't seem to be case, which is great, as the JUCE licensing model certainly seems to be in opposition to the one provided by Unity.
If there's any more information to add, please do.

Hi @semsion, that's right -- this plugin does not use JUCE. The relevant licenses would be the MIT license on this repo, and the Cycling '74 License for Max-Generated Code for Export.

Hey @jinpavg - wonderful stuff; thanks for the clarification!


Task - comprehensive specimen - Recent button

Recent button takes forever. Did not return the result.

@jrflood, @proceps Can you provide steps to replicate this, I can't.  Check your developement server log to see if you can track it down further, likely due to some bad view rendering.

I don't observe this phenomenon.

This is not an issue with TaxonWorks but rather sandbox back end. Will be fixed when http:// resolves to https://.  Temporary fix is to use https:// to start.


Hello, I would like to my web application integrated cloud reader, how do I modify the UI reader UI UI style and UI location

General Instructions

Please use a short, but meaningful title.
Delete everything in italics before submitting issue
Make sure to fill in as much information as possible

This issue is a Bug|Feature Request|Enhancement|Question|Task|Other
(choose ONE from the above list and delete the others)
Related issue(s) and/or pull request(s)
Example: See also #NUMBER
Expected Behaviour
What you expected Readium to do...
Observed behaviour
Instead, Readium does...
Steps to reproduce

First, do this
Then do this
etc.

Test file(s)
If relevant, please indicate public EPUB URL(s), or whether you can share ebook(s) privately.  It is often next to impossible to reproduce bugs without the original file.
Product
Choose one of the following (and delete the others)

Readium Chrome extension

latest official version available from the Chrome Web Store
Version of the Chrome browser
OS and version

Readium cloud reader app

latest development build uploaded at https://readium.firebaseapp.com OR a custom integration (list public deployment URL, if any)
Browser and version
OS and version

Native application (Readium SDK C++)

official "launcher" sample OR propietary app
OS and version

Hybrid app (ReadiumJS)

Electron OR Apache Cordova OR ?
OS and version

Other?

Additional information
Please list any additional information that might help us
Hello, I would like to my web application integrated cloud reader, how do I modify the UI reader UI UI style and UI location

First, make sure you are able to build the existing cloud reader app by following instructions from the README:
https://github.com/readium/readium-js-viewer/blob/develop/README.md
Then, modify the CSS:
https://github.com/readium/readium-js-viewer/tree/master/src/css
...and the HTML templates:
https://github.com/readium/readium-js-viewer/tree/master/src/templates
...and rebuild.

Thank you very much for your solution

I have a question as follows:
Each time the highlight is loaded, the label position is always offset, which is why
for example:
reader.on(Globals.Events.CONTENT_DOCUMENT_LOADED, function ($iframe, spineItem) {
if (_initialized) {
_highlightsManager.attachAnnotations($iframe, spineItem, reader.getLoadedSpineItems());
if(spineItem.idref == "id-id2604743"){
//self.redrawAnnotations();
_highlightsManager.addHighlight(spineItem.idref,"/4/2/4,/1:361,/1:400",2,"test-highlight");
}
} else {
_initializedLate = true;
}
});


Removed Device-Addressable relationship

#1101
#946

Addressables are no longer associated with Devices, no need for uniqueness constraints
Removal of endpoints related to obtaining devices by their addressable
Includes RAML updates
Note associated blackbox test issue https://github.com/edgexfoundry/blackbox-testing/issues/169 It might make sense to leave this PR in place for testing with that issue. If we merge this PR before the tests are ready, the daily run will have errors in it.

Signed-off-by: Trevor Conn<EMAIL_ADDRESS>

Recheck


Release v0.14.1

v0.14.1 Sep. 29, 2020
Enhancements

Updated partial dependence methods to support calculating numeric columns in a dataset with non-numeric columns #1150
Added get_feature_names on OneHotEncoder #1193
Added detect_problem_type to problem_type/utils.py to automatically detect the problem type given targets #1194
Added LightGBM to AutoMLSearch #1199
Updates scikit-learn and scikit-optimize to use latest versions - 0.23.2 and 0.8.1 respectively #1141
Added __str__ and __repr__ for pipelines and components #1218
Included internal target check for both training and validation data in AutoMLSearch #1226
Add ProblemTypes.all_problem_types helper to get list of supported problem types #1219
Added DecisionTreeClassifier and DecisionTreeRegressor classes #1223
Added ProblemTypes.all_problem_types helper to get list of supported problem types #1219
DataChecks can now be parametrized by passing a list of DataCheck classes and a parameter dictionary #1167
Added first CV fold score as validation score in AutoMLSearch.rankings #1221
Updated flake8 configuration to enable linting on init.py` files #1234
Refined make_pipeline_from_components implementation #1204

Fixes

Updated GitHub URL after migration to Alteryx GitHub org #1207
Changed Problem Type enum to be more similar to the string name #1208
Wrapped call to scikit-learn's partial dependence method in a try/finally block #1232

Changes

Added allow_writing_files as a named argument to CatBoost estimators. #1202
Added solver and multi_class as named arguments to LogisticRegressionClassifier #1202
Replaced pipeline's ._transform method to evaluate all the preprocessing steps of a pipeline with .compute_estimator_features #1231
Changed default large dataset train/test splitting behavior #1205

Documentation Changes

Included description of how to access the component instances and features for pipeline user guide #1163
Updated API docs to refer to target as "target" instead of "labels" for non-classification tasks and minor docs cleanup #1160
Added Class Imbalance Data Check to api_reference.rst #1190 #1200
Added pipeline properties to API reference #1209
Clarified what the objective parameter in AutoML is used for in AutoML API reference and AutoML user guide #1222
Updated API docs to include skopt.space.Categorical option for component hyperparameter range definition #1228
Added install documentation for libomp in order to use LightGBM on Mac #1233
Improved description of max_iterations in documentation #1212
Removed unused code from sphinx conf #1235
###Testing Changes

Breaking Changes

DefaultDataChecks now accepts a problem_type parameter that must be specified #1167
Pipeline's ._transform method to evaluate all the preprocessing steps of a pipeline has been replaced with .compute_estimator_features #1231
get_objectives has been renamed to get_core_objectives. This function will now return a list of valid objective instances #1230

@angela97lin RTD builds are still backed up. Let's wait to merge this until we verify the docs are good to go. Hopefully it won't be more than 60min 😬

Thanks guys! Retriggering docs since the last one kept timing out, will merge when it's green and ready to go!