STM32F030 clock control: software timers depend on special clock settings

A simple project with just one software timer and LED feedback was used. The used MCU is the STM32F030C8T6R.
The timers depend to the rcc and/or pll frequencies, which actually should independent. The following configuration lead to working software timers.

By changing the frequency of the rcc or using the pll with a separate and valid configuration the timers do take more or less time than expected. The clock settings were checked with the clock tree of CubeMX.
Tested on:
Ubuntu 20.04.5 LTS
Zephyr 3.2
Related to discord post: https://discord.com/channels/720317445772017664/969266602925252638/1075368071713259531

@danielSchaeble An example of configuration leading to PWM issue ?

The pwm does also work with the clock settings above, but it is also mandatory to specifiy the st, prescaler. If left out the pwm does not work at all. The pwm is used for the RGB LEDS.

The actual problem is also the dependency to the rcc/pll configuration and the mandatory st,prescaler. (The st,prescaler was not mandatory with stm32wl MCU.)

it is also mandatory to specifiy the st, prescaler. If left out the pwm does not work at all. [...] The st,prescaler was not mandatory with stm32wl MCU.

When not specified, st,prescaler is set to default value (0). Depending on RCC configuration (which might differ on your WL target), st,prescaler might require tuning, this is expected.

The timers depend to the rcc and/or pll frequencies, which actually should independent.

Timer clock freq depend on APB domain frequency (equal if APB =1, 2xAPB Freq otherwise).
This is handled internally in PWM driver.
So to me this is normal, unless there is a valid configuration which isn't working

When not specified, st,prescaler is set to default value (0). Depending on RCC configuration (which might differ on your WL target), st,prescaler might require tuning, this is expected.

What does it mean to set the prescaler to 0? Isn't it a multiplicator/divider and setting it to 0 would set the frequency to 0?
All clock prescalers, also in the WL target project, were set to default project settings of a CubeMX project, so it should be valid.

Timer clock freq depend on APB domain frequency (equal if APB =1, 2xAPB Freq otherwise).
This is handled internally in PWM driver.
So to me this is normal, unless there is a valid configuration which isn't working.

So does this mean it is normal that there is only one valid rcc/pll setting to make all zephyr timers and pwm work? Of course only considering valid clock settings in terms of it is valid for the target in general.

What does it mean to set the prescaler to 0? Isn't it a multiplicator/divider and setting it to 0 would set the frequency to 0?
All clock prescalers, also in the WL target project, were set to default project settings of a CubeMX project, so it should be valid.

See bindings doc dts/bindings/timer/st,stm32-timers.yaml:
  st,prescaler:
    type: int
    required: true
    description: |
      Clock prescaler at the input of the timer
      Could be in range [0 .. 0xFFFF] for STM32 General Purpose Timers (CLK/(prescaler+1) )

it is normal that there is only one valid rcc/pll setting to make all zephyr timers and pwm work?

You should be able to get PWM working with every valid RCC/PLL configuration ... BUT it might require to tune st-prescaler

Okay then this seems to be normal. Does that also apply to zephyr timers? Is it correct that a timer can take less time than specified, when a prescaler is not set with a valid clock configuration?

Yes, it will be the same if you're using counter_ll_stm32_timer (same IP, similar driver).

Okay thanks, then this is not a bug.


Add Release Manager command to add a new API

This gets all the information it can from service configs and protos.
We might want to fix our terminology and consolidate our tools at
some point: "add library" isn't obviously part of a release
manager, and we've conflated "API" and "package" quite a lot. But
that can all happen later.

Sample session:
$ ./prepare-release.sh add Google.Cloud.BigQuery.Connection.V1 google/cloud/bigquery/connection/v1
Added Google.Cloud.BigQuery.Connection.V1 to the API catalog with the following configuration:
{
  "id": "Google.Cloud.BigQuery.Connection.V1",
  "version": "1.0.0-beta00",
  "type": "grpc",
  "productName": "BigQuery Connection API",
  "description": "Allows users to manage BigQuery connections to external data sources.",
  "tags": [],
  "dependencies": {
    "Google.Cloud.Iam.V1": "2.0.0"
  },
  "generator": "micro",
  "protoPath": "google/cloud/bigquery/connection/v1"
}

Will work out the AppVeyor issue and sort it before merging.


Add coherent_gain()

Fixes #305

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

Project coverage is 78.16%. Comparing base (3a61a50) to head (35bcef7).

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #307      +/-   ##
=================================================
+ Coverage          78.12%   78.16%   +0.03%
=================================================
  Files                 57       57
  Lines               3410     3416       +6
=================================================
+ Hits                2664     2670       +6
  Misses               746      746

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


Trying to fork a modified version of this to my website: Which branch do you publish?

The master branch..


Add loadMoreIds field to comment

Open to suggestions on the name of this field. I also did some light cleaning up while digging through the code.

:eyeglasses: @schwers

:fish: lgtm, had some minor but non-blocker notes, I'll leave those to you


v3.1.1 release

Quick release to get out node 5.x support (with binaries)
TODO:

[x] Write a CONTRIBUTING.md doc so that @Mithgol @ErisDS could follow it in the future to push out a release (will pull from https://github.com/mapnik/node-mapnik/blob/master/CONTRIBUTING.md)
[x] Bump the version and get latest deps - https://github.com/mapbox/node-sqlite3/commit/77d51d5785b047ff40f6a8225051488a0d96f7fd
[x] Ensure binaries built okay
[x] Ensure changelog is up to date
[x] Tag and publish

@Mithgol @lvivier - I see you've added a few things to master, can you update the changelog with the key details?

v3.1.1 is now tagged and published and supports Node 5.x (with binaries).

:+1:


Node, gulp and sass are out of date

As the title says, this repo is essentially way out of date. Should be updated for manual compiling to work. Otherwise, you have to use outdated versions of node.

Fixed in dev.


Map feature abundances per metadata column on tree

Example: https://itol.embl.de/help.cgi#bar

Update:

generate legends (clade color legends, sample metadata legends)


[2.x] Production build not working in Vite projects

When using the library in a Laravel project that uses Vite, you'll find this error in the browser console: currentRenderingInstance is null. The test suite, a regular Laravel app with Laravel Mix, uses Webpack and works fine.
From searching the web, it seems the library is using the wrong Vue instance (at least not the one from the app). On the other hand, it looks like the Vite config is set up correctly.
https://github.com/protonemedia/inertiajs-tables-laravel-query-builder/blob/e8c38d170f9f0ab52f19e4c456aa026dbbba1f88/vite.config.js#L20
From inspecting the build manually, I can see Vue is not included in the library, which is good.
Any help would be appreciated!

There is a workaround! Add this to your Vite config:
https://github.com/protonemedia/inertiajs-tables-laravel-query-builder/blob/52085954bd48e2ac1df97688b98202f84aa0c10c/app/vite.config.js#L7-L11
It increases the size of the app.js build but decreases the size of the inertia-tables-etc.js build, so it seems to be moving stuff around.
% npm run build

> build
> vite build

vite v2.9.13 building for production...
✓ 395 modules transformed.
public/build/manifest.json                                                  1.27 KiB
public/build/assets/CustomTable.4c10cd7f.js                                 0.50 KiB / gzip: 0.34 KiB
public/build/assets/TwoTables.0d923f56.js                                   0.52 KiB / gzip: 0.31 KiB
public/build/assets/Users.31bc86fb.js                                       0.42 KiB / gzip: 0.32 KiB
public/build/assets/app.15ef2060.css                                        22.71 KiB / gzip: 4.78 KiB
public/build/assets/inertiajs-tables-laravel-query-builder.es.bfea688d.js   125.59 KiB / gzip: 43.15 KiB
public/build/assets/app.797b0635.js                                         184.36 KiB / gzip: 66.05 KiB

% npm run build

> build
> vite build

vite v2.9.13 building for production...
✓ 400 modules transformed.
public/build/manifest.json                                                  1.27 KiB
public/build/assets/CustomTable.65cbee34.js                                 0.50 KiB / gzip: 0.34 KiB
public/build/assets/TwoTables.5e1487f0.js                                   0.52 KiB / gzip: 0.31 KiB
public/build/assets/Users.7ed78ef6.js                                       0.42 KiB / gzip: 0.32 KiB
public/build/assets/app.15ef2060.css                                        22.71 KiB / gzip: 4.78 KiB
public/build/assets/inertiajs-tables-laravel-query-builder.es.db97e4c2.js   97.90 KiB / gzip: 32.35 KiB
public/build/assets/app.eeb46a26.js                                         248.28 KiB / gzip: 91.79 KiB

I don't know if it's related, but I had to add ïnertia=$inertia to the Table tag:
<Table :resource="users" :inertia="$inertia">

I can confirm that with vite 3 I don't have this issue

I tagged a new npm package (v2.1) which works correctly with Vite 3. The demo has also been updated to Vite 3.

Hello @pascalbaljet, Projects with Laravel mix won't work anymore after the latest version released?

any help please ?
326 modules transformed.
[vite]: Rollup failed to resolve import "@popperjs/core/lib/popper-lite" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
build.rollupOptions.external
error during build:
Error: [vite]: Rollup failed to resolve import "@popperjs/core/lib/popper-lite" from<EMAIL_ADDRESS>This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
build.rollupOptions.external
at onRollupWarning (file:///C:/Users/Moi/code/airgestion/node_modules/vite/dist/node/chunks/dep-557f29e6.js:45907:19)
at onwarn (file:///C:/Users/Moi/code/airgestion/node_modules/vite/dist/node/chunks/dep-557f29e6.js:45705:13)
at Object.onwarn (file:///C:/Users/Moi/code/airgestion/node_modules/rollup/dist/es/shared/rollup.js:23225:13)
at ModuleLoader.handleResolveId (file:///C:/Users/Moi/code/airgestion/node_modules/rollup/dist/es/shared/rollup.js:22352:26)
at file:///C:/Users/Moi/code/airgestion/node_modules/rollup/dist/es/shared/rollup.js:22313:26


feat!: Implements the agent's internal file server

Details:

Adds an internal file server for serving the UI from within the VM/Agent. The directory holding these files is exposed through the www-dir configuration parameter.
Prefixes the internal /credentials and /status API endpoints with /api/v1/... to differentiate them from the  /login endpoint which is used to display the login form
Adds an additional field to the Credentials payload: isDataSharingAllowed for future use as part of that feature.

Signed-off-by: Jonathan Kilzi<EMAIL_ADDRESS>

Adds an internal file server for serving the UI from within the VM/Agent. The directory holding these files is exposed through the www-dir configuration parameter.

Why do we need this?

Adds an internal file server for serving the UI from within the VM/Agent. The directory holding these files is exposed through the www-dir configuration parameter.

Why do we need this?

In order to serve the agent's UI


Let us know what you think of the new guides section!

Discussed in https://github.com/github/docs/discussions/2779

Originally posted by simpsoka January 11, 2021

The docs team has been busy iterating on the new landing page system for products, and we've added an all guides section that breaks down the content in a new way: paths.
Think of paths like a Spotify playlist, they're there to help guide you through a learning experience, but you can still skip around to your favorite guides(tracks) or something new. We also offer multiple ways to navigate and find what you're looking for based on who you are as a developer.
Check out the demo of this new feature, and please let us know what you think about this new page by replying in this discussion. What's coming next is an all guides section towards the bottom of the page for the more CYOA (choose your own adventure) folks


Link to Amazon Step Functions

Link Step Functions to list of services that can invoke Amazon Lambda.
Description of changes:
Add a link to AWS Step Functions in the list of possible Lambda triggers.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks!


Refactor search form to use form_with and proper Bootstrap styling classes

No change to the form or styling, just a refactor

Coverage remained the same at 92.624% when pulling c877491ab0925fbd1ca64d0b4f0af3504f7f8aef on nimmo-search-form-with into 8a7e9961cdc8698c3a64caacec064054dea7c098 on master.

Should be ready to go

It's gone. Thanks for your work on this.


503 Error - After install

Thanks for the StackScript!  I created a VM (Linode) using the stackscript.  Lora appeared to run normally and completed, but I'm getting SSL errors and a 503 Service unavailable error after running the scripts.
I've tried updating the SSL cert after correcting the FQDN and fixing my domain records, but certbot can't connect.  Any thoughts?

After the script finished.  I updated the hostname and hosts file to correct the server name and also validated the DNS A/AAAA records.  I also added a CAA record to allow letsencrypt.com to issue the cert.
IMPORTANT NOTES:

The following errors were reported by the server:
Domain: www.centralsw.com
Type:   unauthorized
Detail: Invalid response from
https://www.centralsw.com//.well-known/acme-challenge/wPta7eAk8H-dWiU86G9wQ-GJUeqTzPxba8BlCoHMFcY
[<IP_ADDRESS>]: "\n\n404 Not
Found\n\nNot Found\n<p"

After creating using stackscript
503 Service Unavailable
The connection pool named: |apex|| is not correctly configured, due to the following error(s): IO Error: The Network Adapter could not establish the connection
To fix these errors, please make sure that your domain name was
entered correctly and the DNS A/AAAA record(s) for that domain
contain(s) the right IP address.

I was also providing /usr/share/tomcat as the webroot for certbot.

@jdv68 What version of the database did you attempt to install?

I used <IP_ADDRESS> 64-bit
https://www.dropbox.com/s/rerpga0xyurhokt/oracle-database-xe-18c-1.0-1.x86_64.rpm?dl=0

Apex 18.2
https://www.dropbox.com/s/bfn69qly3own6ht/apex_18.2.zip?dl=0
ORDS 18.3
https://www.dropbox.com/s/3fepp85qoj4w5q3/ords-<IP_ADDRESS>0.1456.zip?dl=0

@jdv68 Are you able to validate that the database was installed and configured correctly by the script? I have to apologise. I won't be able to really look into this until after Kscope. Hope that's okay.

It does look like it's running.  No problems waiting.  I appreciate the help.  Have a great Kscope!

@jdv68 I was having the same issue, but on DigitalOcean. What I found was that the issue only occurred when running ORDS on Tomcat, but running ORDS in Standalone mode worked just fine with the same settings.
After turning debug mode on in ORDS, I saw these Java exceptions in the stack trace:

Caused by: java.io.IOException: Permission denied, socket connect lapse 6 ms. /<IP_ADDRESS> 1521 1500 1 true
at oracle.net.nt.TcpNTAdapter.connect(TcpNTAdapter.java:229)
at oracle.net.nt.ConnOption.connect(ConnOption.java:172)
at oracle.net.nt.ConnStrategy.execute(ConnStrategy.java:508)
... 61 more
Caused by: java.net.SocketException: Permission denied
at sun.nio.ch.Net.connect0(Native Method)
at sun.nio.ch.Net.connect(Net.java:454)
at sun.nio.ch.Net.connect(Net.java:446)
at sun.nio.ch.SocketChannelImpl.connect(SocketChannelImpl.java:648)
at java.nio.channels.SocketChannel.open(SocketChannel.java:189)
at oracle.net.nt.TimeoutSocketChannel.connect(TimeoutSocketChannel.java:108)
at oracle.net.nt.TimeoutSocketChannel.(TimeoutSocketChannel.java:86)
at oracle.net.nt.TcpNTAdapter.connect(TcpNTAdapter.java:188)
... 63 more

To turn debug tracing on: https://docs.oracle.com/en/database/oracle/oracle-rest-data-services/18.3/aelig/troubleshooting-REST.html#GUID-459D10EB-4E62-4D28-92EB-BE1886615E00
In my case SELINUX is blocking Java from connecting to port 1521.
Run this to check if that's the case for you:
cat /var/log/audit/audit.log | grep denied
If so, you should see something like this:
type=AVC msg=audit(1561154618.063:2909): avc:  denied  { name_connect } for  pid=32318 comm="java" dest=1521 scontext=system_u:system_r:tomcat_t:s0 tcontext=system_u:object_r:oracle_port_t:s0 tclass=tcp_socket permissive=0
To confirm, temporarily set SELINUX to permissive mode:
setenforce 0
Then restart Tomcat and see if that at least changes your debug messages.
I'm not very familiar with SELINUX so I'll have to research some more on how to allow that connection to the database from Tomcat.
In the meantime, I'm just running ORDS in Standalone mode.

Redeployed to a brand new Linode and had the same issue, 503 ORDS message on a fresh install.
Ran setsebool -P tomcat_can_network_connect_db 1 then restarted Tomcat and the APEX login page came right up.
Seems like it's due to new SELinux policy booleans in  RHEL/CentOS 7.6: https://dev.to/goober99/selinux-has-a-ui-problem-4knj

Thanks.  I'll check that out this afternoon.

Thanks @hanielburton . I'll add and test the additional boolean flag for SELinux. It was a pleasure meeting you at Kscope19!

Thanks!  @hanielburton
I ran setsebool -P tomcat_can_network_connect_db 1, restarted tomcat it came right up.
@fuzziebrain If you need a tester, please let me know.  I'll be happy to help.


xorm does not support struct inherited?

for example,
type a struct  {
Id int64 `orm:"pk"
}
type B struct {
a
UserName string `orm:"column:user_name"`
}

func (engine *Engine) mapType(v reflect.Value) (*core.Table, error) {

there are some common fields for many tables like id or create_time and so forth.
but common fields in a hidden struct because it can not be used by others.
when using B to query,  the func mapType will raise error at engine.go:1031
if _, ok := fieldValue.Addr().Interface().(core.Conversion); ok {
will this be fixed to support inheritance for common fields?

I added some code to skip processing hidden properties  and error disappeared,
but it can not find common fields as well, always tells me no primiary key...

Find query multiple records from database, also you can use join and extends
type User struct {
    Id int64
    Name string
    Salt string
    Age int
    Passwd string `xorm:"varchar(200)"`
    Created time.Time `xorm:"created"`
    Updated time.Time `xorm:"updated"`
}

var users []User
err := engine.Where("name = ?", name).And("age > 10").Limit(10, 0).Find(&users)
// SELECT * FROM user WHERE name = ? AND age > 10 limit 10 offset 0

type Detail struct {
    Id int64
    UserId int64 `xorm:"index"`
}

type UserDetail struct {
    User `xorm:"extends"`
    Detail `xorm:"extends"`
}

var users []UserDetail
err := engine.Table("user").Select("user.*, detail.*").
    Join("INNER", "detail", "detail.user_id = user.id").
    Where("user.name = ?", name).Limit(10, 0).
    Find(&users)
// SELECT user.*, detail.* FROM user INNER JOIN detail WHERE user.name = ? limit 10 offset 0

but Detail and User is real table in your case,
I just defined some common fields not a real table as struct
so I do not think it can resolve my problem.
thank you for your help.


Connection to drachtio is failing

while connecting to drachtio from call provider we are getting connection errors, below two different kinds are logs show up without any config change, and we never recieve sip invites, please help
Type 1
2023-01-18 03:33:40.929704 tport.c:2806 tport_wakeup_pri() tport_wakeup_pri(0x55dcf0c76a00): events IN
2023-01-18 03:33:40.929872 tport_type_tcp.c:202 tport_tcp_init_secondary() tport_tcp_init_secondary(0x55dcf0c81600): Setting TCP_KEEPIDLE to 30
2023-01-18 03:33:40.929895 tport_type_tcp.c:208 tport_tcp_init_secondary() tport_tcp_init_secondary(0x55dcf0c81600): Setting TCP_KEEPINTVL to 30
2023-01-18 03:33:40.930054 tport.c:1088 tport_register_secondary() tport_register_secondary(0x55dcf0c76a00): register secondary tport 0x55dcf0c81600 from tls/147.X.X.X:39893, count(wss) is 0, count(tcp) is 0
2023-01-18 03:33:40.930067 tport_type_tls.c:620 tport_tls_accept() tport_tls_accept(0x55dcf0c81600): new connection from tls/147.X.X.X:39893
2023-01-18 03:33:40.930082 tport_tls.c:970 tls_connect() tls_connect(0x55dcf0c81600): events NEGOTIATING
2023-01-18 03:33:40.968575 tport_tls.c:970 tls_connect() tls_connect(0x55dcf0c81600): events NEGOTIATING ERROR HANGUP
2023-01-18 03:33:40.968692 tport.c:2891 tport_hup_event() tport_hup_event(0x55dcf0c81600) tls/147.X.X.X:39893
2023-01-18 03:33:40.968706 tport.c:2201 tport_shutdown0() tport_shutdown0(0x55dcf0c81600, 2) tls/147.X.X.X:39893
2023-01-18 03:33:40.968715 tport.c:2132 tport_close() tport_close(0x55dcf0c81600): tls/147.X.X.X:39893
2023-01-18 03:33:40.968890 tport.c:2208 tport_shutdown0() tport_shutdown0(0x55dcf0c81600): after tport_close refcount is now 0
2023-01-18 03:33:40.968904 tport.c:2311 tport_set_secondary_timer() tport(0x55dcf0c81600): set timer at 0 ms because zap
2023-01-18 03:33:40.968916 tport_tls.c:987 tls_connect() tls_connect(0x55dcf0c81600): tport was closed during connect. Returning, but set secondary timer first.
2023-01-18 03:33:40.968924 tport.c:2311 tport_set_secondary_timer() tport(0x55dcf0c81600): set timer at 0 ms because zap
2023-01-18 03:33:40.968937 tport.c:1155 tport_zap_secondary() tport_zap_secondary(0x55dcf0c76a00): zap tport 0x55dcf0c81600 from tls/147.X.X.X:39893, count(wss) is 0, count(tcp) is 0

Type 2
2023-01-18 03:49:41.953757 tport.c:2806 tport_wakeup_pri() tport_wakeup_pri(0x55dcf0c76780): events IN
2023-01-18 03:49:41.954108 tport_type_tcp.c:202 tport_tcp_init_secondary() tport_tcp_init_secondary(0x55dcf0c81600): Setting TCP_KEEPIDLE to 30
2023-01-18 03:49:41.954126 tport_type_tcp.c:208 tport_tcp_init_secondary() tport_tcp_init_secondary(0x55dcf0c81600): Setting TCP_KEEPINTVL to 30
2023-01-18 03:49:41.954180 tport.c:1088 tport_register_secondary() tport_register_secondary(0x55dcf0c76780): register secondary tport 0x55dcf0c81600 from tcp/147.X.X.X:59649, count(wss) is 0, count(tcp) is 1
2023-01-18 03:49:41.954190 tport.c:2700 tport_accept() tport_accept(0x55dcf0c81600): new connection from tcp/147.X.X.X:59649
2023-01-18 03:49:41.954204 tport.c:2832 tport_wakeup() tport_wakeup(0x55dcf0c81600): events IN
2023-01-18 03:49:41.954212 tport.c:2929 tport_recv_event() tport_recv_event(0x55dcf0c81600)
2023-01-18 03:49:41.954254 tport.c:3270 tport_recv_iovec() tport_recv_iovec(0x55dcf0c81600) msg 0x55dcf0c97400 from (tcp/147.X.X.X:59649) has 247 bytes, veclen = 1
2023-01-18 03:49:41.954285 tport.c:2345 tport_set_secondary_timer() tport(0x55dcf0c81600): reset timer

@davehorton this was due to tls certificate issue, adding fresh certificate solved it.


change test.sh to test_local.sh and add memory profiling

The local tests should always be run with test.sh (to be renamed test_local.sh) and should include a basic memory profile to see that there is no memory build up from code changes.

This is now handled. Benchmarks for std of the memory test and max memory have been added and an exception is raised in test_local.sh unless met.


Without $LANG var being set weather was not shown

Issue Label:

[ ✔ ] enhancement

Description of bug or feature enhancement:
Without $LANG var being set weather was not populated
OS and OS version:

[✔ ] Mac

OS Version:
10.12.5 (16F73)

Fixed here https://github.com/alexanderepstein/Bash-Snippets/pull/14

Really? This variable should be a builtin so every system should have a value there, but I guess it cannot hurt to have a default value

Merged 4f44a45ba19932da064988bc4189ac9444f29b9b

Yes, unfortunately it was empty I am using latest zsh version zsh 5.2 (x86_64-apple-darwin16.0.0). But anyways, thanks for the merge! 👍


Exception when startin RateMyApp for native ratings

Describe the bug
When calling the showRateDialog method with ignoreNativeDialog: false, an exception is being thrown:
E/flutter ( 6167): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: type 'Future<bool?>' is not a subtype of type 'FutureOr' in type cast
E/flutter ( 6167): #0      RateMyApp.showRateDialog (package:rate_my_app/src/core.dart:128:46)
E/flutter ( 6167): #1      _StartScreenState._showRateMyApp (package:paintmaster/screens/start_screen.dart:342:24)
E/flutter ( 6167): #2      _StartScreenState.initState. (package:paintmaster/screens/start_screen.dart:335:9)
E/flutter ( 6167):
To Reproduce
Steps to reproduce the behavior:

Call the method, see an exception

Expected behavior
No exceptions, I am not sure if the rate dialog works right now
Screenshots
The app runs fine though

I think the fix is simple. The as Future<bool> should be Future<bool?>

You're right, thanks for reporting !


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1293 ms

Resolved: Bofrost.lu performance has improved in ef5d916 after 8 minutes.


BEF Major Refactor

We have four major refactoring ideas/goals.
1. Bloated Core Plugin
The intent of the Core plugin is to be the integration layer to/from Eclipse APIs. The other plugins have no exposure to the Eclipse APIs, which make them reusable (if anyone ever wanted to build a language server) and more easily testable.
But over time, the Core plugin code base has grown and includes too much. The classpath container is a perfect example. About half of that file has nothing to do with Eclipse.
2. Code share with Language Server effort
Another Salesforce team is working on a language server. They are using BEF code and we would like to share work.
bazel-jdt-ls
3. Non-java projects
We plan to support other languages than Java in the future. I envision have a set of plugins, like:

plugin-java
plugin-scala
plugin-go

that would adapt the Bazel rules for each language into Eclipse projects
4. Bazel Java SDK
The idea of the non-Core plugins is to keep most of the code base free of Eclipse dependencies. One of my hopes it to isolate it as a general purpose SDK for anyone who wants to create Bazel build tooling in Java.

Bear in mind #131 (find a faster way to compute the dependency graph) which is itself a big refactoring story.

I accomplished item 4 in the refactor list, culminating in PR #148

I accomplished item 1 in the refactor list culminating in this commit which moved the classpath computation from the Eclipse specific plugin to the bazel-java-sdk:
https://github.com/salesforce/bazel-eclipse/commit/515f818792ca7894f408ced25cd061195d8b567c

This is done as far to the extent that we need. Follow up Issues noted above track additional work.


Seems CSS not loading

Successfully ran the backpex into my project but it seems CSS not loading. Also, showing empty label buttons.

Also, it shows large icons.

Hey, are Tailwind CSS and daisyUI installed correctly?

Hi, I installed it using the installation guide and I'm not sure if it installed correctly or not.
Do you know how I can verify?

I'm having a similar issue.

I've been testing out adding Backpex (v0.5.1) into an existing project as we need an admin ui. But the existing project was built as a no html project, as it's just been used for API endpoints until now. So I have had to add in tailwind manually.
I was testing out kaffy and live admin as well, which all rendered ok. It might just be that i've messed something up with the install steps for daisy ui/tailwind config changes.
I'm going to test out the install with a fresh phoenix project and try installing Backpex into that to see if I can work out what i've missed.

I tested the install steps in a fresh phoenix project and I'm not having the issue. Just trying to figure out what i'm missing in my initial implementation.

My issue was due to missing this in the :browser pipeline in router.ex.
 plug :put_root_layout, html: {YourAppWeb.Layouts, :root}
Adding that in, and my UI displays ok now.

I noticed the below error in the console in my browser, if you see that, maybe the above will fix your issue.

Yes, I noticed this and added this line in the router it starts loading layouts but still does not load CSS and JS files.

Does your static_paths function in YourAppWeb have assets included?
def static_paths, do: ~w(assets fonts images favicon.ico robots.txt)

Yes, I added assets in the static_paths in my AppWeb.

Thanks, The Problem is resolved.
I missed some configuration in the dev.ex for tailwind and  esbuild.


Support waiting for ACK response

TODO:

Reuse the logic to convert vector<pair<MessageId, ResultCallback> to a map.
Fix the callback in AckGroupingTrackerEnabled not triggered in time

TODO: handle the response in ClientConnection.

TODO:

Fix the code format
Handle the case when ACK response is disabled and ACK grouping time is positive


koord-manager: add controllers options

Ⅰ. Describe what this PR does

Move the controller registration of the koord-manager into the options pkg instead of the main.go.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


onboarding_step: Remove hotspot as an onboarding_step

Fixes: #29296

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

The test failures are the flaky star puppeteer & installation failure.


Fix with_routing not working with get :index

As guys previously mentioned in the thread the problem is that #get is called on other context then #with_routing.
It is caused by https://github.com/rspec/rspec-rails/blob/master/lib/rspec/rails/example/controller_example_group.rb#L5.
ActionDispatch::Assertions::RoutingAssertions is adding #with_routing to an assertion_instance, and ActionController::TestCase::Behavior is adding #get to ControllerExampleGroup.
But they are included to different classes and do not affect each other.
So I decided to add an ControllerAssertionDelegator which will include both of them. Actually AssertionDelegator missed some methods, so I featured them in that delegator.
Closes #1652

Hi @samphippen. This is my first try for rspec and also just a working draft rather then full solution.
Just want to be sure that I'm on right track)

On the other hand we can do it like that https://gist.github.com/sclinede/3dc0565e5ef9cb9d99bbcf6719302c5d

@sclinede Can you please add a test that would fail without the patch?

@pirj You could find failing spec in an additional commit https://github.com/rspec/rspec-rails/pull/1777/commits/a9adc657db9e0d79ef5b072d36c18c5ed7aa7189

The solution was tested against  https://github.com/samphippen/test_issue_817 provided by @penelopezone

I don't get it. The test suit is failing around at teardown method somewhere deep. If you have any suggestions I'll appreciate.
Meanwhile, trying to figure it out by myself...

So... it looks like I need more time to invest in that change. Hope I'll find it soon :)
I don't give up

@sclinede Have you had a chance to take a closer look at teardown problem?


Fix network info

Update parsing of inbound/outbound network info of images.
Fix #149.

waiting for codacy....


GL example [GL-1]

Please check if the PR fulfills these requirements

[x] The pull request title contains the ticket number in the title. For example [CB-1]

What kind of change does this PR introduce? (Bug fix, new feature...)
finished post, delete, and get route.

What is the current behaviour? What does the code do before you made the new changes?
before it didn't have post and delete butit only has get route.

What is the new behavior?
The pull request has all the finished routes.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR? Run a new command?)
none

Other information:

none

Not related to your code itself, but your index.js should be in a folder you created named GL (for your initials). There may be issues if in the current state your index.js gets merged into the one in the main branch rather than in the one in your folder.


Lock down pipenv to v2022.12.19 to prevent further Pipfile.lock issue…

Summary

Hello @neflyte,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


Add translated textarea and input components

Description
At the moment, we have quite a few translation pages that render textareas and inputs then the english translation below them.
We've decided that we're going to migrate the h2 and english translation for both to a details component.
While doing this I think it makes sense to add reusable components which handle spacing etc. so they're all consistent. This PR adds 2 components which handle this.
If details text, but no heading it will default to English translation and have for {label_name_with_(required)_stripped} in a visually hidden span.
Guidance for review
I've potentially made some fault assumptions about the correct behaviour of this component.

If there is no english translation for the field, i've hidden the details component. It might be that there be something that states there's no translation for that field
i've added a default title for the details component which simply states "English translation" with a visually hidden span which adds the label to the end ("English translation for Title" etc.)

It might be that these pages should have more specific titles.
I probs need to speak to Nik about this when he's back off leave before i'll know how to proceed.
Screenshots
Translated textarea component

Translated input component

Trello card
https://trello.com/c/8vp9mX1f/365-ui-revisit-translation-page-update-original-language-inside-the-details-component
https://trello.com/c/S0kuPlXk/470-create-new-endpoint-for-new-translation-page
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Hi @minhngocd
I just left a very minor comment on this, otherwise looks fine.


ARROW-540: [C++] Build fixes after ARROW-33, PARQUET-866

@xhochy Travis repro's the OS X build failure you experienced -- will attempt to fix tomorrow

Well,  https://github.com/apache/arrow/pull/325/commits/9070baf552f221cf9a2fea983cb34e962c38dd39 fixes the tests on OS X for me locally. I'm not sure why this bug does not occur on Linux -- I'm not satisfied merging this patch until I understand the discrepancy. Will investigate tomorrow...

I realized it's because the PRNG is slightly different because it's a different standard library. I will play with the data generation to make it slightly less corner-case-y.

Thanks. The test case that was failing is with a deeply nested list (63 levels deep) — I am not going to bother with tweaking it any further since what's being tested with that test case is something fairly unrelated to slicing (it only needs to not fail)


TouchScreenButton Rotation behavior

Godot version:
3.0.5
OS/device including version:
Linux Mint 19 x64
Issue description:
Why TouchScreenButton rotates around Position instead of own center (in that time Sprite rotates around center)? Is it possible to rotate TouchScreenButton around center of it's current selected texture?
Thanks.

_would be cool to get feature like a rotation offset and unify rotation behavior for all objects, who supports rotation and _

Closing in favor of https://github.com/godotengine/godot-proposals/issues/3976


[R114] Define "JSON-LD format sympathetic"

R114 requires that messages are "JSON-LD format sympathetic", but it is not stated what that means in the spec, nor is that term used in the JSON-LD spec.
Please define the meaning, or change the requirement.

There is a reference given as to what it means. We can repeat that in the glossary if that is required. I think a reference is sufficient.
cc @chaals

The reference says what JSON-LD means, and that is sufficient. But I cannot find the term "format sympathetic" defined in either this spec or the JSON-LD spec.
Either defining it in the glossary, in the requirement, or changing the wording, would be helpful.
If it means "either a valid  JSON-LD document, or a JSON document that can be interpreted as JSON-LD by associating a context through HTTP headers, as described in https://www.w3.org/TR/json-ld11/#interpreting-json-as-json-ld" then we should make an explicit reference to that section of the JSON-LD spec.

@chaals ... very good wording. will include in PR I am working on ... see current working branch for PR https://github.com/eea-oasis/baseline-standard/blob/pr/101/core/baseline-core-v1.0-psd01.md

closing issues per meeting 12/2


On WIndows 8.1 x64 it doesn't detect net desktop runtime even though ive reinstalled it twice.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Please fix the bug.

Requires more information, such as what version of .net you have.

Please fix the bug.

This is not my problem, this is .NET's problem.
If you open a Command Prompt window and run the following command: dotnet --info - what does it show?

Same issue for me and I'm on Windows 10 (Pro I think?)

@Acuelo-Azure Do what I asked OP to do please.

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


Update to work with fugue-support report

What

Updates the fugue_installer_iam policy to provide appropriate permissions to fun fugue-support report and upload the report to Fugue Support.

Thanks! Merging!


Update() vs FixedUpdate()

Issue Type

[ ] Bug
[ ] Enhancement / Feature
[x] Question
[ ] Other

Description
In your NetworkedTransform component, shouldn't you use FixedUpdate() instead of Update()? As far as I have understood, when it comes to rigidbody dynamics and changes in an object's transform, we should use FixedUpdate(). I think it has to do with the fact that FixedUpdate() is executed in sync with Unity's physics engine, whereas Update() can vary out of step with the physics engine, either faster or slower. Could you please explain me why you decided to use Update() instead?
Thanks

FixedUpdate is for physics. Not transform.


merged master into dev-1.16

tide/merge-method-merge

/label tide/merge-method-merge

/tide merge-method-merge

closing in favor https://github.com/kubernetes/website/pull/15679


Review all catch(...) in code for silencing exceptions that shouldn't be silenced like bad_alloc

closed by #4472


Question: Why not generate interfaces based on sequelize definitions?

First of all many thanks for the development of sequelize-auto-ts - I would love to use it ;) The reason that I can't use it right now is that we use postrgres as our database, so I think we would need to write a new schema dialect layer.
I was wondering why one wouldn't base the generation of interfaces directly - after all that's what most (all?) users of sequelize already have and it is what sequelize is drawing all its information from? Could this be a way to avoid having special code for every db or is this a bad idea?

The reason I pull from the database is precisely because I do not have the sequelize definitions. sequelize-auto-ts provides to features.. it creates interfaces for all your table definitions, and it creates sequelize model definitions. sequelize itself is based around model first which admittedly is the direction most ORMs have gone. I'm more traditional and prefer to work on my schema in my database, not in my model, and thus I created a tool to support database first with sequelize.
There were other people interested in Postgres too and I think one of them started working on a patch but never got it done or submitted. I'm too busy now to do it myself, since I don't have a personal need, but if someone comes up with one that works transparently I'll be happy to merge it.
Thanks,
Sam

Hi Sam,
thank you for explaining the reasons. I understand better now and know the time constraints you speak about all too well. I'll have a look at generating interfaces from sequelize definitions (not making any promises here...).
Thank you
Paul

Sure, shouldn't be all that difficult, but that's usually the last word of any software developer before jumping down some bottomless rabbit hole, right?
Anyways, sequelize-auto-ts defines an object model for storing the schema definition and builds this up from the schema defined in the database. If you build the same object model from existing sequelize definitions, then you can use the rest of sequelize-auto-ts as is.
HTH,
Sam

Yeah, famous last words ;) OK, it does sound manageable indeed, the one
problem is that I don't know how sequelize represents its definitions, so I
guess I have to figure that out ;)
Anyway, thanks for your input - if I manage to get this going I'll let you
know.
On Sat, Apr 9, 2016 at 3:23 PM, Samuel Neff<EMAIL_ADDRESS>wrote:

Sure, shouldn't be all that difficult, but that's usually the last word of
any software developer before jumping down some bottomless rabbit hole,
right?
Anyways, sequelize-auto-ts defines an object model for storing the schema
definition and builds this up from the schema defined in the database. If
you build the same object model from existing sequelize definitions, then
you can use the rest of sequelize-auto-ts as is.
HTH,
Sam
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/samuelneff/sequelize-auto-ts/issues/9#issuecomment-207788737


Google search automation

please check and update me of any changes are required from my side 😄

Hi @ssm0801. I have made another contri to this repo. Hope you go through these changes and give your feedback for this pr as well. Thanks


Upgrade flake.nix to produce a package with the binary

At the moment the flake.nix is used only to create the dev environment, it would be more useful to produce also the peerswap binary in a package so that can be properly referenced in a nix env

Thank you for your suggestion.
I believe the priority is low unless specific requirements or requests arise, but we will consider addressing it.


v10 compatibility

I see that the json says its compatible but when loading in Foundry it has the "Compatibility Unknown" flag and in-game doesn't function at all

@Rayuaz any updates on this? It seems like there's no new release for download still

This module changed ownership, use this version from now https://github.com/UnoriginalDM/spell-level-buttons-for-dnd5e


SCSS hot reload is not working

@EisenbergEffect thanks for doing the release, unfortunately the new feat in #12 doesn't seem to work for me. @niieani, @jods4 did you try it out at the time?
I'm submitting a bug report

Library Version:
0.2.0

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
8.2.1

NPM Version:
5.4.2

JSPM OR Webpack AND Version
Webpack 3.6.0

Browser:
Chrome 61.0.3163.100

Language:
TypeScript 2.5.2

Current behavior:
All my dependencies are up to date
{
  "version": "1.0.0",
  "sasslintConfig": "node_modules/siopa-package-tools/.sass-lint.yml",
  "private": true,
  "scripts": {
    "test": "jest",
    "start": "webpack"
  },
  "dependencies": {
    "aurelia-bootstrapper": "^2.1.1",
    "aurelia-dependency-injection": "^1.3.2",
    "aurelia-framework": "^1.1.5",
    "aurelia-pal": "^1.4.0",
    "aurelia-router": "^1.4.0"
  },
  "devDependencies": {
    "@types/extract-text-webpack-plugin": "^3.0.0",
    "@types/jest": "^21.1.2",
    "@types/node": "^8.0.32",
    "aspnet-webpack": "^2.0.1",
    "aurelia-hot-module-reload": "^0.2.0",
    "aurelia-polyfills": "^1.2.2",
    "aurelia-webpack-plugin": "^2.0.0-rc.5",
    "css-loader": "^0.28.7",
    "extract-text-webpack-plugin": "^3.0.0",
    "file-loader": "^1.1.5",
    "gulp": "github:gulpjs/gulp#4.0",
    "import-glob-loader": "^1.1.0",
    "node-sass": "^4.5.3",
    "sass-loader": "^6.0.6",
    "siopa-package-tools": "^1.0.9",
    "style-loader": "^0.19.0",
    "ts-loader": "^2.3.7",
    "ts-node": "^3.3.0",
    "tslib": "^1.7.1",
    "tslint": "^5.5.0",
    "typescript": "^2.5.2",
    "webpack": "^3.6.0",
    "webpack-hot-middleware": "^2.19.1"
  }
}

Given the following component, when changing the color, an error is displayed in console.
// relevant webpack config line
{ test: /\.scss$/i, issuer: /\.html$/i, loader: 'css-loader!sass-loader' },

<template class="c-header-bar">
  <require from="./header-bar.scss"></require>
  My Header Text
</template>

.c-header-bar {
  color: blue;
}

process-update.js:115 [HMR] Cannot check for update (Full reload needed)
handleError @ process-update.js:115
process-update.js:116 [HMR] TypeError: Cannot read property 'resources' of undefined
    at HmrContext.29.HmrContext.reloadCss (http://localhost:5000/dist/app.js?v=QNzl5O3zgddSQAkYZUauFSbxSVlDVeKMHEZPOYHMqfk:10355:29)
...ETC
handleError @ process-update.js:116
aurelia-hot-module-reload.js:235 Uncaught (in promise) TypeError: Cannot read property 'resources' of undefined
    at HmrContext.29.HmrContext.reloadCss (aurelia-hot-module-reload.js:235)
    at Object.hot (aurelia-hot-module-reload.js:79)
    at aurelia-loader-webpack.js:167
    at Object.hotApply [as apply] (bootstrap 8fa308dbf2e403c4d37c:580)
    at cb (process-update.js:52)
    at process-update.js:68
    at <anonymous>

Expected/desired behavior:
Given the following component, when changing the color, then the rendered text should change color without console errors.
If I use .css files instead with the same setup and following webpack config line everything works
        { test: /\.css$/i, issuer: /\.html$/i, loader: 'css-loader' },

I did the change in #12 mostly because it's the same code as another PR I did in aurelia-templating-resources, to keep things consistent.
I think I had a basic LESS example working, but to be honest that was long ago and I'm not 100% sure. There has been major Webpack releases since then so even if it worked it might have been broken in the meantime.
I'm confident this could work with HMR but someone needs to debug the error and fix it.

Found some time to look into this. The culprit is obvious. It's fixed as a css-resource-plugin but the resource is under scss-resource-plugin.

Any thoughts on the best way to determine the plugin to look for? I assume the same code exists elsewhere.

To clarify:

do you transpile your scss into css with a Webpack loader? So at runtime, css-resource-plugin is right.
or do you pass scss code to the runtime, hoping it will use a converter?

I'm thinking 1. is the right way to do it with Webpack and should work?

See the details in the issue above. I think I'm doing 2??
FWIW I just changed the distributed code as follows and it works.
From:
        var cssPluginModuleId = this.loader.applyPluginToUrl(moduleId, 'css-resource-plugin');
        console.log("Handling HMR for " + moduleId);
        delete this.loader.moduleRegistry[moduleId];
        delete this.loader.moduleRegistry[cssPluginModuleId];
        var analyzedModule = this.moduleAnalyzerCache["css-resource-plugin!" + moduleId];

to
        var cssPluginModuleId = this.loader.applyPluginToUrl(moduleId, 'scss-resource-plugin');
        console.log("Handling HMR for " + moduleId);
        delete this.loader.moduleRegistry[moduleId];
        delete this.loader.moduleRegistry[cssPluginModuleId];
        var analyzedModule = this.moduleAnalyzerCache[cssPluginModuleId];

Picture vs thousand words:

The plugin name is convention-based upon the file extension name so perhaps we can ignorantly build the plugin name within hot-module-reload
...but part of me says there should be an interface we can use in the place where that convention has been implemented. i.e. a viewEngine.getResourcePluginName should probably exist

Having a static build step with webpack, I don't quite see why you would compile your resources at runtime. Is there a benefit of doing so?
I'm using LESS and I compile everything down to minified CSS before pushing to browser. In my head this is the main/only(?) scenario.
We could try to support both use cases, but that would require more thinking I guess.

I guess I misunderstood (and still do misunderstand) what #12 was for if not this. I'm new to webpack and this is part of my migration from JSPM..
I understood the resulting bundle to contain css not scss. Thus not compiling scss at runtime as you seem to suggest. Is the reason for referencing the scss not just to trigger the compilation into css on change and serve that?  Thats what 'css-loader!sass-loader' is doing right?
This is taken from my resulting app.js bundle. Which shows css and not the variable/mixin containing scss that I began with (albeit with \n new lines):
// module
exports.push([module.i, ".c-header-bar {\n  align-items: center;\n  display: flex;\n  flex-wrap: wrap;\n  color: #FF5113;\n  -webkit-border-radius: 10px;\n  -moz-border-radius: 10px;\n  border-radius: 10px; }\n  .c-header-bar__logo {\n    flex: 0 0 auto;\n    margin: 1rem; }\n  .c-header-bar__heading {\n    flex: 0 1 auto;\n    margin: 0 1rem; }\n  .c-header-bar__loader {\n    flex: 1 1 auto;\n    font-size: 2em;\n    margin: 0 1rem; }\n  .c-header-bar__nav {\n    flex: 1 0 100%;\n    font-size: 1.1rem;\n    text-align: left; }\n", ""]);

Minification is a step I'm yet to do but I wanted to get this working before I complicate things!
Again I'm new to webpack but as far as I can tell I have everything working sensibly.

Ah, I've just thought to re-read your earlier question:

do you transpile your scss into css with a Webpack loader? So at runtime, css-resource-plugin is right.

I was wrong with my earlier reply, which may have confused matters (sorry)... yes, I do transpile my scss into css with a webpack loader. But the code added in #12 adds a plugin named scss-resource-plugin for files loaded with an .scss extension. So when the file gets referenced and loaded by aurelia it's loaded with that but the HMR assumes it was loaded with css-resource-plugin.
I don't think I'm doing anything wrong/weird here, I just think this was missed.
Sorry for wall of text.

Having re-read everything again I am definitely doing the method you suggest as the right way of doing things.

Yeah, I thought about this a bit more as well.
I'm not even sure aurelia-webpack-plugin supports client-side loaders.
For now I think that if you use webpack to build your project, you should transform everything at build-time and not use loaders at runtime.
Unless a compelling use-case comes up, I think it's better to leave it at that and not support loaders in aurelia-webpack-loader.
Going back to the initial issue: we need to modify hot-module-reload to not apply loader, right? Because right now your css goes through an automatic scss loader because of the extension?

@jods4 sorry for delayed response. Ignoring the rest (cause it's not important) I'll focus on this:

Going back to the initial issue: we need to modify hot-module-reload to not apply loader, right? Because right now your css goes through an automatic scss loader because of the extension?

I'm not sure what you mean by "to not apply loader" or what you're asking in your following question.
The problem is really simple. As far as I can tell I'm just trying to make it work as you initially intended, you just missed an additional required change. I urge you to look at the proposed fix in #17. The facts are this:

The code you added in #12 and https://github.com/aurelia/templating-resources/pull/279 instructs the application to use the existing CSS resource plugin implementation for less, sass, scss, styl files

As you know, there is no special handling within aurelia for those extensions beyond plain CSS. This is fine because webpack has already transformed the input files to CSS. You said this yourself in aurelia/templating-resources#279

viewEngine.addResourcePlugin uses the extension to create a specific resource plugin name

Don't be fooled by seeing scss-resource-plugin. Any less/sass/scss/style-resource-plugin uses the same implementation as the css-resource-plugin, just a different instance with a different name. This is the change you created in #12

The code in this repo assumed the plugin name to be css-resource-plugin when applying the hot reload. As per the above bullet, this is not true.

My fix in #17 removes this assumption and fixes the issue.


Not able to verify my email address

Network telemetry
Not able to verify my email address since no emails are sent to neither my inbox nor spam folder

What is your PyPI username?

youssef_sherif
From: Ernest W. Durbin III<EMAIL_ADDRESS>Sent: Friday, 17 August 2018 5:43 PM
To: pypa/warehouse<EMAIL_ADDRESS>Cc: Youssef Sherif<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [pypa/warehouse] Not able to verify my email address (#4580)
What is your PyPI username?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pypa/warehouse/issues/4580#issuecomment-413906171, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AULYcwoFkE-ng9ryMwIwZOiY8j8eyi2Jks5uRuSSgaJpZM4WBx5w.

Hi @youssefsharief I'm seeing recent attempts to verify two email addresses.
Can you email me at<EMAIL_ADDRESS>with information on which email address you're unable to receive your verification email.

The issue has been resolved
Thank you
On Fri, Aug 17, 2018 at 11:07 PM, Ernest W. Durbin III <
<EMAIL_ADDRESS>wrote:

Hi @youssefsharief https://github.com/youssefsharief I'm seeing recent
attempts to verify two email addresses.
Can you email me at<EMAIL_ADDRESS>with information on which email
address you're unable to receive your verification email.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pypa/warehouse/issues/4580#issuecomment-413988485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AULYc0GlEuyEIo5bFpj03nf9_VMBy4gqks5uRzCGgaJpZM4WBx5w
.

--
Best Regards,
Youssef Sherif


BigDecimal

I had a situation where I wanted to coerce into a BigDecimal/Numeric (I was actually using a T::Struct). The reason for a igDecimal was that the API sometimes receives strings, and sometimes integers (JSON). I want to convert both to a numeric without data loss.
TypeCoerce[T.nilable(Integer)].new.from('123') => 123
TypeCoerce[T.nilable(Integer)].new.from(123) => 123

TypeCoerce[T.nilable(Float)].new.from('123.2') => 123.2
TypeCoerce[T.nilable(Float)].new.from(123.2) => 123.2

TypeCoerce[T.nilable(Float)].new.from('2823111111111111111111111111111111111111111111111111111111111111111111111111111111111111.12').round ==><PHONE_NUMBER>111111211310091539389163649541789162060078943821790215899541495486114917515264

Given I am using these values for pricing information I can not loose precision (although would be happy with some form of Numeric class with arbitrary precision.
Currently I am parsing the input as a string and then converting to a BigDecimal before usage, however I would rather convert to a BigDecimal during the conversion to a T::Struct.

Hi! I think BigDecimal is supported:
TypeCoerce[T.nilable(BigDecimal)].new.from('2823111111111111111111111111111111111111111111111111111111111111111111111111111111111111.12')
=> 0.282311111111111111111111111111111111111111111111111111111111111111111111111111111111111112e88

This is only a problem on Ruby 2.7+ since BigDecimal.new was deprecated on Ruby 2.6.
I've created a small PR (#50) with a test case for this and a fix, but I'm happy to discuss other alternatives there. 🙂


NPE on Graphite sink

Description
It looks like when heapster went to startup, it was unable to setup a graphite sink, which resulted in an NPE.
I0425 21:24:15.289715       1 heapster.go:78] /heapster -v=3 --source=kubernetes:https://kubernetes.default --sink=influxdb:http://monitoring-influxdb.kube-system.svc:8086 --sink=graphite:tcp://relay.asg.company.com:2003?prefix=k8s.asg --sink=graphite:tcp://relay.asg.company.com:2003?prefix=k8s.asg.default
I0425 21:24:15.289743       1 heapster.go:79] Heapster version v1.5.2
I0425 21:24:15.289884       1 configs.go:61] Using Kubernetes client with master "https://kubernetes.default" and version v1
I0425 21:24:15.289916       1 configs.go:62] Using kubelet port 10255
I0425 21:24:15.320675       1 reflector.go:187] Starting reflector *v1.Node (1h0m0s) from k8s.io/heapster/metrics/util/util.go:30
I0425 21:24:15.320803       1 reflector.go:236] Listing and watching *v1.Node from k8s.io/heapster/metrics/util/util.go:30
I0425 21:24:20.328314       1 influxdb.go:312] created influxdb sink with options: host:monitoring-influxdb.kube-system.svc:8086 user:root db:k8s
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x15f7423]

goroutine 1 [running]:
k8s.io/heapster/metrics/sinks.(*SinkFactory).BuildAll(0xc420749cd8, 0xc4202b1d40, 0x3, 0x4, 0x0, 0x0, 0x7f393be10000, 0x0, 0x0, 0x0, ...)
	/go/src/k8s.io/heapster/metrics/sinks/factory.go:90 +0x563
main.createAndInitSinksOrDie(0xc4202b1d40, 0x3, 0x4, 0x0, 0x0, 0x4a817c800, 0x100, 0x0, 0xc420435480, 0x0, ...)
	/go/src/k8s.io/heapster/metrics/heapster.go:194 +0x8d
main.main()
	/go/src/k8s.io/heapster/metrics/heapster.go:89 +0x458

I humbly suggest a quick review of this code in the stack trace to ensure that NPEs do not happen and errors are gracefully handled.
Output of heapster --version:
1.5.2

I explicitly take issue with this line on factory.go line 44:
func (this *SinkFactory) Build(uri flags.Uri) (core.DataSink, error) {
In go, its best to return concrete types.  Accept interfaces, but return concrete types.  Because a interface was returned here, it became hard to return "nothing" in the case of an error.  Because of that, the developer chose to return a nil.  Because a nil was returned, the code blew up somewhere.
Looking at the codebase now, it looks like this case is handled.  I am assuming this was fixed in master but is not in the image for Heapster 1.5.2...


Add Elixir support

This PR adds Elixir support
The implementation is based on https://github.com/elixir-lsp/vscode-elixir-ls which adds some wrappers on top of https://github.com/elixir-lsp/elixir-ls
The VSCode implementation exposes mix_task name(because it relies on the mix tool) but I decided to go with the elixir name.
It might be a bit confusing(I can add a note in the README regarding that) but at the same time, it kind of makes the naming more consistent.
But I am happy to change it to mix_task

Using elixer is fine by me. I'll probably add support for alternative type names so vscode configurations would still work with the mix_task type would still work.


Update encoding.md

All changed packages have been documented!

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/2903/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/2903/


Add max_generated_tokens_length for window service

Needed for this for PaLM and for GPT-4-turbo.

Duplicate of #2086


Account Password Reset

Change Type
Correction
Proposed Changes
https://docs.cloudmydc.com/account-and-pricing/account-password-reset
***Unable to replicate the diagram. Needs to remove Jelastic Logo under Reset Password article.

Closed the issue. No screenshot saved.


Allow traefik.port to not be in the list of marathon ports

Fixes #261
This is needed when using --host and the port cannot be provided to the marathon app.

@nicgrayson ouch, I think broke everything, your commits have disappeared :fearful: Could you push -force from your local repo?


Basic how-to?

Can someone created a simple Getting Started section in README?
I assume I need to install NodeJS, NPM, then this, and create a custom https://github.com/bitcoinjs/indexd/blob/master/test/index.js-like config file? Does that config file replace the main index.js file, or is it supposed to exist in a separate directory like here?
How to execute?
How to query the index?

@unsystemizer great questions,  I think for now I am a slightly time-limited on this,  but what I will do is add some examples of how to query the indexes in test/.

Thanks.  What is your configuration if I may ask?
I just need a couple of high level pointers.

@unsystemizer I expect to have some examples added by tomorrow,  but incase that is delayed:

Build a node.js server (e.g using express)
Install this module
Add express routes for your desired end-points, calling adapter.* functions to retrieve the data you need.

For example, using an router = express.Router(),  typeforce-middleware and body-parser,  the following address_transactions route returns an { $address: [txIds] } object result of transactions for each address, since block 0.
  router.post('/address_transactions', bodyParser.json(), typeforceM([bitcoin.address.toOutputScript]),
 (req, res) => {
    let addrs = req.body
    let tasks = {}

    addrs.forEach((address) => {
      let script = bitcoin.address.toOutputScript(address)
      let scId = bitcoin.crypto.sha256(script).toString('hex')
      tasks[address] = (next) => indexd.adapter.transactionsByScript(scId, 0, next)
    })

    parallel(tasks, (err, resultMap2) => {
      if (err) return respond(req, res, err)

      let tempMap = {}
      for (let address in resultMap2) {
        Object.assign(tempMap, resultMap2[address])
      }

      res.status(200).json(tempMap)
    })
  })

@unsystemizer please see https://github.com/bitcoinjs/indexd/tree/master/example for an in-depth example using express.


Teleporting up and down changes base height

Changing heights by teleporting onto colliders with different heights eventually changes the overall base height. This results in floating in air or stuck in the ground. It's not very predictable when this happens (last time it worked properly 3 times teleporting, but failed the forth time trying to get from a sofa onto the floor and staying at sofa level)

Having the same problem. Needs a Body Physics implementation similar to that of VRTK Body Physics.

Ahh, hmm. The attempt at preserving height during a teleport is done by raycasting straight down from the camera right before the teleport. I could see a situation where you've teleported on top of a taller object, moved your head to no longer be directly over the object, and then teleported again, which would raycast past the object and back to the floor (or otherwise something at a different height).
A better height detection method would be preferable. Thinking about it, an app in RoomScale might no longer need to check height, since Y = 0 becomes real-world floor level. Stationary space-type apps will still need something though.

If its true that a raycast from the head measures the height every time a teleport occurs, then I'm wondering why, once the height is off, it stays off?

 Any subsequent teleports/raycasts will simply return the same height.

Just one question. Will or is it possible to get the height of the HMD above the ground from windows? Or is that something you guys are working on?
A good workaround would be to either populate a layermask or an override collider against which the casts are performed.

Ok, I fixed the height issue by changing two methods in MixedRealityTeleport.cs:
`        public void SetWorldPosition(Vector3 worldPosition)
{
// There are two things moving the camera: the camera parent (that this script is attached to)
// and the user's head (which the MR device is attached to. :)). When setting the world position,
// we need to set it relative to the user's head in the scene so they are looking/standing where
// we expect.
Vector3 newPosition = worldPosition - (CameraCache.Main.transform.position - transform.position);
newPosition.y = worldPosition.y;
transform.position = newPosition;
	}`

`        private void FinishTeleport()
{
if (currentPointingSource != null)
{
currentPointingSource = null;
            if (isTeleportValid)
            {
                fadeControl.DoFade(0.25f, 0.5f, () =>
                {
                    SetWorldPosition(teleportMarker.transform.position);
                }, null);
            }

            DisableMarker();
        }
    }`


Add esp32 support

This MR adds support for ESP32. Adding another target as well as switching to latest esp-hal made for significant changes, therefore I decided to merge it directly into main branch as opposed to dev branch. Comments were incorporated to this branch, except for using heapless::Deque instead of heapless::spsc::Queue, due to performance reasons on esp32c3 (rustc 1.64.0-nightly).

In order to compile both feature and target has to be used:
 cargo build --features esp32 --target xtensa-esp32-none-elf --release

If anyone happen to know how to specify target in .cargo/config.toml based on provided feature, please share.

@DNedic @jessebraham @jessebraham  PTAL


Fix integration tests

Move to a pattern where we check that the right prompt is shown before attempting to input text, that should allow us to track down issues easier
Fix the command names for creating apps/modules, was changed in #366 (to be pushed after I've tested how these failures look in Jenkins)

Warnings

:warning:

src/commands/alloyGenerate.ts#L45 - src/commands/alloyGenerate.ts line 45 – Forbidden non-null assertion. (@typescript-eslint/no-non-null-assertion)

Messages

:book:

:white_check_mark: All tests are passing
Nice one! All 32 tests are passing.

:book:

:floppy_disk: Here's the built vsix for this change.

  Generated by :no_entry_sign: dangerJS against fb7e41d7b4a0ade7f274bf9c0aa17f368d8b2711


Release 4.1.0: index.js header reports v4.0.0

Minor nitpick, it seems that the 4.1.0 release of Bootstrap didn't bump the version of js/src/index.js, that is still reporting:
 * Bootstrap (v4.0.0): index.js

This header line appears also in the following files, that I think are derived from the one in src:

js/dist/index.js
dist/js/bootstrap.js
dist/js/bootstrap.bundle.js

Yup, confirmed.
@mdo: you reverted the whole index.js after the version bump and we missed this.
I guess it doesn't matter for 4.1.0 but we'll make sure it won't happen in the next release.


Fix auth token HTTP request not containing username in "account" GET parameter

When pushing a multi-arch image to a private registry that is attached to authentication frontends like Portus or monitoring tools like registry-web-ui, it is necessary to add the username to the URL parameters of the digest token request.
This is because if the account parameter is not present, the JWT access token issued will contain an empty sub field. When the registry then generates a notification for the frontend, the frontend cant correlate any user because the actor field is empty.
The regular docker push does this correctly, however (and thats how I found this bug) Docker BuildX does not do this when pushing a multi-arch image. This is because buildx uses containerd via buildkit.
This pull requests adds the required parameter to the token request URL. Also see the discovery of this bug: https://github.com/docker/buildx/issues/177
Signed-off-by: Christoph Honal<EMAIL_ADDRESS>

@dmcgowan @tonistiigi ptal

This is not a specified parameter. Portus should be using the basic authentication credentials. Portus should also be audited to ensure that the sub is authenticating with the same value as provided in the basic auth username.

When the registry then generates a notification for the frontend, the frontend cant correlate any user because the actor field is empty.

Not quite sure how the registry is authenticating the request if there is no subject to authenticate. This seems like something is wrong in Portus or the Registry configuration.
I am going to close this as there is no different in the current implementation from the specification. Nor do we want to duplicate information being sent to the authorization server as that just gives the authorization more work to do for validating the account is matching the auth user. See https://github.com/docker/distribution/blob/master/docs/spec/auth/token.md for what is specified.

Thank you for the clarification, I opened a pull request in Portus (https://github.com/SUSE/Portus/pull/2286).
However, only out of interest, why does Docker CLI do this?

why does Docker CLI do this

Probably something legacy there with the hub.
I discussed this with @tonistiigi, after the Portus folks take a look at it, if that value is being properly handled, doesn't present a security issue and rolling out the update is burdensome, then we can consider a temporary workaround to add that value. Normally when we accept a workaround we look for acknowledgement of the issue from the upstream registry, what versions are affected by it, then add the workaround with a comment with that information and the date.

This sounds reasonable. Portus development appears kind of stalled since last fall, but I will keep an eye on it.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1521 ms

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


solution for october challenge 2021- THREE BOXES

I've added a solution in C++ for February Challenge 2021 Division 3 - MEET problem. Link to the problem statement is present in the README.md file.
Attaching a screenshot for the Sub-Task Info:

i have done a pr, owner please accept this under, hacktober fest


uv pip list --outdated does not use UV_INDEX_URL setting

Trying to use the new uv pip list --outdated feature with a custom UV_INDEX_URL I get an error that a random private pacakge cannot be found in the registry. Such as error: Package `mypackagename` was not found in the registry
Activating verbose mode the problem seems to be that packages are not looked up in the custom registry but on pypi
❯ uv pip list --outdated --verbose
DEBUG uv 0.5.0 (8d665267c 2024-11-07)
DEBUG Searching for default Python interpreter in virtual environments, registry, or search path
DEBUG Found `cpython-3.12.7-windows-x86_64-none` at `somepath` (active virtual environment)
DEBUG Using Python 3.12.7 environment at .venv
DEBUG Using request timeout of 30s
DEBUG No cache entry for: https://pypi.org/simple/adlfs/
DEBUG No cache entry for: https://pypi.org/simple/aiohttp/
...

A minimal code snippet that reproduces the bug. run uv pip list --outdated --verbose with UV_INDEX_URL configured to use an index different from pypi. Notice that packages are looked up on pypi.
The command you invoked: uv pip list --outdated --verbose
The current uv platform. Windows 11
The current uv version (uv --version). uv 0.5.0

This is similar to #8926 but I guess a separate issue.

Thanks, that's a bug.


Unifying links and actions

Would it be an idea to unify link, link-templates and actions into one single "links" element?
With this change I hope it becomes easier for client developers:

Because the same link name/relation cannot exist in both links, templates and actions.
It becomes easier to move from a simple "HAL-like" links-only representation to a representation with more meta data about the links (without breaking existing clients).
Client behavior should be simplified (but that is mostly a gut feeling - no hard facts).

Changes

All links, templates and actions are now moved to @links.
The "type" property now designates the type of link.
The "target-type" property now designates target content type hint (this is the old "type" for links).
The link "template" is now "href" (since "template" is used in the actions for a different purpose)

Expected client behavior

Assume having a (possibly empty) JSON data object for the operation (the payload or template variable set).
Read resource, find @links.
Find link by name.
If link type is "link" then simply follow it with a GET and retry.
If link is "link-template" then merge the "href" URL template with the JSON data object and follow the URL with a GET.
If link is "json" and method is GET then form-url-encode the JSON data object, merge the result into the href URL and follow the URL with a GET.
If link is "json" or "json+files" then merge data object with data template from the link description and send it to the server as either JSON or multipart (using the designated method).

Examples
Simple link
Here we have a simple non-templated link
@links:
{
  "foo:a":
  {
    // "type" designates the type of link
    type: "link",
    href: "...",
    title: "...",
    // Old "type" (target content type hint) is now "target-type"
    target-type: ["text/vcard"],
    // "alt" is still alternative links (usually having different target types)
    alt: [ ... ]
  }
}

Templated link
@links:
{
  "foo:b":
  {
    type: "link-template",
    // "href" is now the URL template
    "href": "http://.../issues-query?text={text}&severity={severity}&project={pid}",
    "title": "Search for issues",
    "description": "This is a simple search that does not check attachments.",
    "parameters": [
      {
        "name": "text",
        "description": "Substring search for text in title and description"
      },
      {
        "name": "severity",
        "description": "Issue severity (exact value, 1..5)"
      },
      {
        "name": "pid",
        "description": "Project ID"
      }
    ],
  }
}

JSON action
The actions have not changed much. Only things is that they now live under @links instead of @actions
"@links":
{
  "is:update-project": {
    "type": "json",
    "href": "...",
    "method": "POST",
    "title": "Update project details",
    "description": "...",
    "schemaUrl": "...",
    "template": {
        "Code": "SHOP",
        "Title": "Webshop",
        "Description": "All issues related to the webshop."
    }
}

The version 2 specification will combine all links, link templates and actions into a single object. Right now the name is "@navigation" but I am looking for something better.

How about @transitions?

"@requests"?  The point is that if you consider HTTP, what these translate into are different ways for formulating a request to the server.  The point is you are going beyond a link (which simply identifies a resource) and elaborating more on the method and potentially the payload.  That's a request.

Version 2 is now official - and uses @controls for all hypermedia elements together.


Application only worked the first time for each controller

After using the program to change the colors of each my Joy-Cons, I closed it out and put them onto my Switch and saw that it had indeed worked. However, I wanted to make some tweaks to the colors and upon trying to open JC Toolkit again it simply will not open. I constantly get the "Connection Error!" screen, and no amount of disconnecting and reconnecting Bluetooth, or even restarting my computer does anything about this.

Hi @JimbertHimbert
(assuming you have windows), go to the bluetooth settings and remove the joy con pairing from the list, the reconnect.
Bit of a hassle to do each time but it works from me.
Hope that helps


Construct new report incorporating resistance genes and plasmids

This report should incorporate both resistance genes and plasmids, but unlike summary.tsv, each resistance gene/plasmid should be on a separate line of the table.

Got feedback from Laura, she says:

Each isolate tested should have the datatype plasmid or resistance marked
- If none exist then the gene would indicate None

Issue has been resolved/implemented


[Expanded params] Should be an option to load file contents as a list

Today we have @file to load contents from a file (loads it as one chunk, useful for SSH keys, etc.).  We should also have an option to load contents as a list, say with #file.  This would be especially useful in Windows where shell substitution isn't supported.  Windows users could then pipe IDs to a file and execute a command like az vm show --ids #file

Cleaning u old issues, reopen if this is still active.


DeprecationWarning: pkg_resources is deprecated as an API

Hello, just wanted to flag this warning I've begun seeing:
/usr/bin/vcs:6: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
  from pkg_resources import load_entry_point

I believe it comes from:
https://github.com/dirk-thomas/vcstool/blob/daf389377310f7f31b2171f51a79af82c31bf3e2/vcstool/commands/help.py#L4
Happy to look into a fix if it's something that would be merged.

Seeing this issue as well

Same issue on Ubuntu 24.04.


AR dependency locked to =4.2.0

hi there, I noticed something strange when doing a library update. The new version of PR seems to have a hard dependency on the rather old AR 4.2.0:
$ gem dependency permanent_records
Gem permanent_records-3.3.0
  activerecord (>= 3.0.0)
  rake (>= 0, development)

Gem permanent_records-4.0.0
  activerecord (= 4.2.0)
  activesupport (= 4.2.0)
  database_cleaner (>= 0, development)
  pry (>= 0, development)
  rspec (~> 2.14.1, development)
  sqlite3 (>= 0, development)

I see this also reported on rubygems: https://rubygems.org/gems/permanent_records
I don't see anything in the project that could explain this. Any clue as to what's going on?

+1 ... seems like it could be a >=

Everything is fine when installing the gem from github master.
@JackDanger It seems before building the gem to be uploaded to rubygems, an unset AR_TEST_VERSION is required; otherwise the gem metadata is built the locally set AR version. Maybe you could bump to 4.0.1, tag, build and upload to rubygems again?

This shouldn't be an issue any more.


Suggestion: use a wiki for Addict recipes

I think it would be a great help if we had a wiki where we could document recipes for Addict, like the Devise wiki which is a great help.
And then some of those, once reviewed, can be incorporated into the documentation.

That would be helpful, I mostly run into code, which not friendly to new comers.


Minor refactors and typo fixes

Hi guys,
I was curious to find out how you implemented the connection between the extension and the react app, so I checked out your project to take a look around.
While doing so, I fixed several typos in the docs and some other minor improvements which resulted in this PR.
Cheers,
Goce

Thanks for leaving the code in a better state than you found it. Hope you were able to make the connection


Fix broken link to react-router history docs

What is says on the label 🏷

Terribly sorry I didn't see this (since it was first), but I merged #2538 which seems to fix this as well.
I hope to see more from you in the future!


"Frăsinet" is listed in the drop-down menu

Description:
"Frăsinet" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Frăsinet" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Frăsinet, CĂLĂRAȘI ".


Introduce describedby link relation

The describedby link relation can be used to identify a description document, which may be an OpenAPI definition, a JSON Schema document, or another description format.
It's proposed that the describedby member can appear in two places:

A) in the top-level links object, in which case it provides a description for the current document.
B) inside any link object, in which case it provides a description for the target of the link.

We could optionally allow this member inside any links object that can also contain a self link - namely, resource links and relationship links. This would be consistent with (A) but also somewhat redundant with (B). I'd like to make a decision about this explicitly prior to merging.
Another question is whether there should be a central definition of a "description document" within the JSON:API spec, or whether using parenthetical examples (e.g. OpenAPI or JSON Schema) is sufficient.
related: #1281
/cc @philsturgeon @gabesullice

🚢 👍

Thanks @dgeb @philsturgeon @gabesullice  for initiating this effort.
I'd vote for adopting a central default description format to avoid any confusion between API providers and consumers. Any other format can be handled through profiles.
In this case, I would prefer OpenAPI because:

Large community
Industry wide support
Tooling
Alignment with JSON Schema

PS: We are already using OpenAPI for describing JSON:API resources. This spec is used by both the client and server to handle requests and responses.

@jugaadi Thanks for your feedback! I agree that OpenAPI is a solid choice and a good starting point.
While we don't want to restrict the content type of the description documents pointed to by describedby, we definitely want to provide some recommendations in the non-normative section of the site that will cover best practices. We can include example usage of OpenAPI and JSON Schema to start, and would appreciate your contributions to the recommendations based on your experience.

@gabesullice I think this is ready to review. Please let me know what you think about my question from the PR description:

We could optionally allow this member inside any links object that can also contain a self link - namely, resource links and relationship links. This would be consistent with (A) but also somewhat redundant with (B). I'd like to make a decision about this explicitly prior to merging.

@dgeb and I discussed this today. WRT to the question in the description, we decided against allowing describedby links in resource or relationship objects because it may cause confusion about whether the description document is intended to describe the object as it appears in the current response or the description document of the document you expect to receive by following its self link. The describedby target attribute is a link hint that clearly indicates that the description document it targets describes the target URL.
I think this PR is ready to merge.

There is also this format: https://tools.ietf.org/html/draft-nottingham-link-hint
bit different from how this was done, but does the same thing nonetheless

@evert, afaict, we could integrate that RFC w/ JSON:API fairly easily. They're not mutually exclusive. To do that, we would add hints as an allowed member of a link objects both as a sibling of and similar to the params member.
If you're saying that we could have added describedby under hints, I don't think that works because that RFC is about providing hints related to HTTP headers, statuses, etc. I suppose it would be a proxy for the Link header, but for better or for worse the JSON:API spec, talks about links exclusively about links represented in the JSON response body.
Perhaps you're referring to the links hint from that RFC since we could have done something like:
links: {
  self: {
    href: /articles/1,
    hints: {
      links: {
        describedby: /schemas/article.json
      }
    }
}

Instead of what we chose:
links: {
  self: {
    href: /articles/1,
    describedby: {
      href: /schemas/article.json
    }
}

We considered something like that (i.e. adding a links target attribute to link objects themselves) but decided against it to keep complexity to a minimum.
PS. It's funny you mentioned that RFC, I just mentioned it in a post within 30 minutes of your reply 😺

Yea just for context, I do believe that your use-case is one of the things link-hints hopes to solve, but it would indeed be a deeper nested structure.
I believe the equivalent syntax would actually be:
links: {
  self: {
    href: /articles/1,
    hints: {
      formats: {
        'application/vnd+api.json': {
           links: {
             describedby: {
               href: "/schemas/article.json",
               type: "application/schema+json"
           }
        }
      }
    }
}

This makes the most sense for links that are explicitly not self links. For a self link, I would just take the format information and lift it to its own link:
links: {
  self: {
    href: /articles/1,
  },
  describedby: {
    href: "/schemas/article.json",
    type: "application/schema+json"
  }
}

Whoaaa, I had not grokked that bit of the RFC. That's a very thought provoking idea... It's interesting how the format is specialized by a schema/profile.

For a self link, I would just take the format information and lift it to its own link:

This PR does exactly that, but only for top-level links.

Probably not worth opening a new issue, but just thought I'd note that link describedBy is generally about describing a resource (so JSON Schema fits well here), but an OpenAPI document generally describes the surface area of a whole or partial API.
The link rel service-desc might be more appropriate for OpenAPI?

I expect implementors to link to the HTML representations of their OpenAPI spec where they can add a fragment to the URL. For example, https://petstore.swagger.io/#/user


Fixed bug with connecting to console same user

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/477

This change is

Please add commit message describing the issue and its fix, including link to the bug.

@mareklibra Done.

@mareklibra I added user check for more safety.


Amend RFC0070 with DeclBuffer TVMScript syntax updates

As a followup on the previous discussion https://github.com/apache/tvm-rfcs/pull/70#issuecomment-1149308146, this PR updated the DeclBuffer RFC with TVMScript related changes.
cc @Lunderberg @wrongtest-intellif @Hzfengsy @junrushao1994 @cyx-6 @tqchen

lgtm!


Dashboard graph does not scroll to full-screen contents

Where to find the issue

https://www.coronawarn.app/en/analysis/ "Corona-Warn-App (CWA): Key performance indicators"
https://www.coronawarn.app/de/analysis/ "Corona-Warn-App (CWA): Kennzahlen"

Describe the issue
Full-screen display of any graph on https://www.coronawarn.app/en/analysis/ does not automatically scroll to display all the graph.

This follows on from the related issue https://github.com/corona-warn-app/cwa-website/issues/2321.

Steps to reproduce

Open browser and resize to take one half of screen height and one half of screen width (this effectively uses one quarter of the screen's available area)
Open https://www.coronawarn.app/en/analysis/ in the existing browser window
Scroll down to Downloads graph
Click on the full-screen icon of the Downloads graph
Note that the graph is not scrolled so it is fully visible in the window

Reproducible with:

Google Chrome
Mozilla Firefox

Expected behavior
The full-screen display option in the analysis graphs should display the full graph inside the available browser window display area. It should not be necessary for the user to have to scroll vertically in order to see the graph after full-screen has been selected.
Suggested change
After resizing the graph to full size according to the available browser area, scroll the window so the graph is fully visible.

This issue is resolved by PR #2721.


[BACKLOG-40923] Unit test fixes for JDK17

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 13m 9s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
plugins/edi2xml/impl

:ok_hand: All tests passed!
Tests run: 1, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Step 3 Not Correct

Step 3 #1 States: In the Microsoft 365 admin center, go to Settings > Services & add-ins;
There is no longer a Services & add-ins link at this location.  Can't find where to go for this step now.

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

ID: f737e278-7c89-e048-b60b-287615a829fb
Version Independent ID: 1df35258-7360-74c4-680a-e154c54291fe
Content: Microsoft Teams guest access checklist - Microsoft Teams
Content Source: Teams/guest-access-checklist.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

Hello @Jason-ACESGroup
Document has already been updated :) https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/commit/bae9ca1e471638df851982b87059c1f539d4868e
Thank you!

@scanum please close this thread.

@Jason-ACESGroup Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.