[Bug] Fix bug of cumulative compaction and deletion of stale version

Proposed changes
When selecting candicate rowsets to do the cumulative compaction,
some rowsets may not be selected because the protection time has not expired.
Therefore, we need to find the current longest continuous version path in the candicate rowsets.
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)

Checklist

[x] I have create an issue on (Fix #4551), and have described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes

LGTM


Declare compatibility for localstack image substitute

Using a private registry to bypass docker hub rate limits currently fails when using this extension.
Exception
java.lang.IllegalStateException: Failed to verify that image '<private-registry>/localstack/localstack:1.3.1' is a compatible substitute for 'localstack/localstack'. This generally means that you are trying to use an image that Testcontainers has not been designed to use. If this is deliberate, and if you are confident that the image is compatible, you should declare compatibility in code using the `asCompatibleSubstituteFor` method.

@gsmet Can you please review this PR?


abnormal oligodendrocyte differentiation

J:234565
decreased oligodendrocyte differentiation
several existing annotations mention abnormal differentiation as well
should also review the other types of glial cells

consider also
abnormal oligodendrocyte precursor number J:264451

added term for differentiation

added term for abnormal oligodendrocyte progenitor morphology MP:0014325


Json Formatter overridden by _get_json_formatter

logzero version: 1.7.0
Python version: 3.7.9
Operating System: Windows 10

Description
Thanks for the great library!
I was setting up a custom JSON formatter for logging a specific json message (using a custom json default), however this was stymied through https://github.com/metachris/logzero/blob/bc19173105561650bfdd0ae22d1bb0d9dcb1be99/logzero/init.py#L139
    _formatter = _get_json_formatter(json_ensure_ascii) if json else formatter or LogFormatter()

Is this intended behaviour (happy to override the formatter in the handler after creating?
Otherwise would suggest checking if isinstance(formatter, JsonFormatter)?

Happy to contribute the change, want to check if the behaviour is intended first!


put a deprecation warning

this lib is deprecated, right?
or what is the difference btw this lib and https://github.com/garyb/purescript-codec-argonaut ?

No, rather the opposite - purescript-json is meant to/can replace purescript-argonaut, and this library is newer than purescript-codec-argonaut.

TODO: also other argonaut libs + affjax depends on argonaut-core


digest: false configuration being ignored

[x] I tried updating to the latest version.

[ ] I can't, there is an issue.

[ ] I Am on Windows
[ ] I Am on Bash on Windows
[ ] I Am on Linux
[x] I Am on OS X

Description
Lodging this as a bug after me trying this on two production Jekyll installs. The digest setting is set to false but still produces the digest on the asset file names.
Steps
The following is how my Jekyll site is setup with jekyll-assets version 3.0.2 along side jekyll version 3.6.2 relevant configuration as follows:
Gemfile for the site
source 'https://rubygems.org'

gem 'jekyll'
gem 'uglifier'
gem 'kramdown'
gem 'rouge'
gem 'liquid'
gem 'octopress-autoprefixer'
gem 'redcarpet'

group :jekyll_plugins do
    gem 'jekyll-watch'
    gem 'jekyll-assets'
    gem 'jekyll-feed'
    gem 'jekyll-seo-tag'
    gem 'jekyll-sitemap'
end

_config.yml extract:
plugins:
  - jekyll-assets
  - jekyll-feed
  - jekyll-seo-tag
  - jekyll-sitemap
  - octopress-autoprefixer

assets:
  digest: false
  caching:
    type: memory
  sources:
    - _assets/css
    - _assets/js
    - _assets/fonts
    - _assets/img

Output
            Source: /Users/devraj/Work/WebApps/zen
       Destination: /Users/devraj/Work/WebApps/zen/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/screen-f44e0adf8a74f346853740055ade3bf5c9166323544757605d7505d8c577ab49.css
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/site-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.js
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/apple-touch-icon-76-b466ea76a2cdafdd708a38600eb389a5596de296f73c6e32fe4369dd2804c7f0.png
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/apple-touch-icon-120-8fe4d735a9acea1a3727cb21eb6c149f7e9c91faadd436f9aca798a5db09199f.png
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/apple-touch-icon-152-97b224a6d37317f2a7361ea5c30607e71663765ec9c1f3a33ce7e7b87727af17.png
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/apple-touch-icon-180-d7033dcff948b8ebe8ee7c4a220961da1ded6a841326ad55dac2f81279c6d68a.png
           Assets:  Writing /Users/devraj/Work/WebApps/zen/_site/assets/favicon-041e452d40a0f4b812051c0ec4d0726bbde600f7a0781c5ac30f55e7e02e33b6.ico

Expected
Files being written without the digest.

We don't allow you to doable digesting in 3.0.  It still being listed as an option is the bug here. The lack of digesting only forces us to hack Sprockets, and it doesn't provide any useful benefit.

@envygeeks so does that mean there would be no way of exposing an asset with a particular "fixed" name? e.g /assets/screen.css ?

Why would you want that?

Assets that are built / hosted by us but referenced  by external sites
We use Jekyll to prototype interfaces and the output is consumed by other automated build processes

The feature required us to maintain a lot more than I wanted to, and to consider a lot more than I wanted to. Sprockets doesn't support what you want which meant I had to take over parts of Sprockets to get what you wanted in 2.x.
If somebody is willing to monetarily sponsor it's initial development and testing, I'm happy to add it into the new rewrite, other than that though I feel it would take too much development time.

The reason why is because now we have to write code that works with Sprockets 3, and 4. Can be back ported and supported for at least a year, with minimal effort, and can integrate into our workflow which is much more well designed than Assets 2.x

@envygeeks Anomaly would be willing to consider sponsoring it, do you want to drop us a line and carry on the conversation there?

They @devraj there are so many ways to contact ya'll I don't know which is best.   It might be easier if ya'll email me at jordon at envygeeks.io, I'll be available for the next few hours if you want to chat over the phone too, if that's easier.


Error occurs when set android:visibility="gone" in xml file

Error occurs when set android:visibility="gone" in xml file,but setVisibility(View.GONE) in JAVA code is fine.It happens to my phone with android4.4 but run normal in other phone with android8.0.

Fixed with: 4001bd5726fece7e46a222602b5215305f58c308.


sqlfluff: 0.6.4 -> 0.6.5

Motivation for this change
Update to latest upstream release 0.6.5
Change log: https://github.com/sqlfluff/sqlfluff/releases/tag/0.6.5
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137403 run on x86_64-linux 1

1 package built:

sqlfluff


time only passes in VEDA if you have the editor in focus

talk log on talk.lurk.org:
efairbanks 1:04 AM
@amagitakayosi I think `time` only passes in VEDA if you have the editor in focus

At least on Mac OS.

Is this intended? Is there a more reliable form of `time`?
I've got a radial display to show me how far I am through a 25 minute performance but I"m worried it's not super accurate because if I nav away from Atom and then come back 3.125 minutes later, it's not 1/8 of the way through, it's barely moved

bgold 1:51 AM
If I cmd-H atom it freezes, but not if I just have another app in front. I suspect the OS is stopping GL rendering when it knows there's nothing to see.
Strangely, the playback of videos doesn't seem affected (maybe it's on a different timer?)

echophon 2:18 AM
I suspect the same policies for background tabs apply https://developers.google.com/web/updates/2017/03/background_tabs

amagitakayosi 10:41 AM
oh it's not intended
it broke when I added rercoding feature https://github.com/fand/vedajs/blame/8a77a044d6a5d4dda69687392219f0380b5a0ab8/src/veda.ts#L644

Fixed in v2.11.3 😎


Make usage of openSSL optional

Add the possibility to disable the usage of openssl when building libpq from source.

Thanks for opening this PR. Could you explain the motivation for why the usage of openssl should be disabled by default? I believe that's the wrong choice for most of our users as that disables support for SSL/TLS postgres connections in libpq.

I think we also need to put a #[cfg(not(feature = "disable-openssl")] on top of the extern crate openssl_sys; statement here: https://github.com/sgrif/pq-sys/blob/master/pq-src/src/lib.rs#L1 to prevent linking openssl if we don't need it.

Thanks for the review, indeed the build script is still run even with
#[cfg(not(feature = "diable-openssl")]
extern crate openssl_sys

So the only solution is to make this opt-out through default-features = false. I've added a feature bundled_without_openssl to pq-sys, mirroring the vendored feature of openssl-sys. I've also added some CI jobs to avoid regression like the one above, including compatibility with diesel.
Please let me know what you think.

Published as 0.6.2


[[clang::optnone]] causes miscompiles

This is mostly an Apple compiler bug, but we might want to consider avoiding using things that both aren't in the MSL spec and change lots of otherwise unused (and therefore probably completely untested) options in the compiler
To reproduce:

Download and unzip OptNoneFail.zip
Run make -j2 run

The test will do a two-pass render, first with a depth-only pass and then with a compare zequal color pass that writes red to the output.  It will then compare the two images and count the number of pixels that wrote depth but failed the zequal test.  This should be zero, but on Apple GPUs, 2211 pixels will fail.  You can go into main.swift and uncomment the commented lines to take a capture if you're curious (change the destination and add an output if you don't want to import this project into Xcode).
The shaders VS0 and VS1 are lightly modified from spirv-cross, with as much [[clang::optnone]] usage as possible removed while still preserving the bug.
I disassembled the compiled shaders to see why the bug happens, for anyone curious:

Shader disassembly and analysis
For anyone following along, apply this patch to the apple gpu disassembler and run python3 compiler_explorer.py OptNoneFail/VS0.metal --no-fast-math

Full shader decompilation
GitHub seems to have a comment length limit of 65536 characters, so have a text file instead
The two outlined functions are as follows:
; @ 0x-80, spvFAdd<float2>
   0: 2aad56a22500         fadd32           r11, r11, r13
   6: 2ab158c22500         fadd32           r12, r12, r14
   c: 1402                 ret              r1
; @ 0x-40, spvFAdd<float3>
   0: 2aad56c22500         fadd32           r11, r11, r14
   6: 2ab158e22500         fadd32           r12, r12, r15
   c: 2ab55a022600         fadd32           r13, r13, r16
  12: 1402                 ret              r1

Looking at the area around the two function calls (since we know that's where things are breaking), you can see this:
1706:<PHONE_NUMBER>00         push_exec        r0l, 2
170c: 10c074e8ffff         call             0x-80
1712: d21600000000         pop_exec         r0l.cache, 2
1718: 02be8e2229a401308015 fcmpsel          lt, r47h, r39.cache, r41.cache, u82l, 0
1722: 02bc8e2229a6513aa015 fcmpsel          lt, r47l, r39.cache, r41.cache, u83l, u82h
172c: 02908e2229a5613aa005 fcmpsel          lt, r4l, r39.cache, r41.cache, u82h, u83l
1736: 82944e222500413a2005 fcmpsel          lt, r5l.cache, r39, r41, 0, u82l
1740: e22d00000000         mov_imm          r11.cache, 0
1746: aa9dca000002         fadd32           r7.cache, r5l.discard, -0.0
174c:<PHONE_NUMBER>00         mov_imm          r5, 0
1752: 7e19d80a8000         mov              r6, r12.discard
1758:<PHONE_NUMBER>3f         mov_imm          r13,<PHONE_NUMBER>
175e: 7e39920a8000         mov              r14, r9.cache
1764: 7e3d920a8000         mov              r15, r9.cache
176a: 7e41d20a8000         mov              r16, r9.discard
1770: 7e31ce0a8000         mov              r12, r7.discard
1776:<PHONE_NUMBER>00         push_exec        r0l, 2
177c: 10c044e8ffff         call             0x-40
1782:<PHONE_NUMBER>00         pop_exec         r0l, 2

The calling convention seems to put inputs and outputs starting at r11
Weirdly, offset 1740 seems to overwrite r11 (the first output of the first function call) without saving it first.  Checking the shader, it is used:
r8.xy = spvONFAdd(r1.xy, -r8.xy);
// much later...
r0.z = dot(r8.xy, r8.xy);

r12 does get saved into r6, so if we search for the next use of that...
1c7a: 9a9d4aa22400         fmul32           r7.cache, r5, r5
1c80: 9aad4cc22400         fmul32           r11.cache, r6, r6
1c86: 2a99dce22d10         fadd32           r38, r14.discard, r15.discard
1c8c: aaadce622d00         fadd32           r11.cache, r7.discard, r11.discard

It seems the compiler thinks r8.x is in r5, which got zeroed in offset 174c
I guess something about the next function's input being zero confused it?

What is actually actionable here from SPIRV-Cross' end? This sounds like a bug Apple needs to fix.

This is probably more of an @cdavis5e or @billhollings issue if anything.


Optionally set docker --host

The 'docker' cookbook LWRP allows us to specify an array of 'host' parameters to allow local and remote access to the docker daemon. This is quite useful in some environments but opens security holes in others. Allowing this to be specified optionally gives us the flexibility of both operating modes.
This PR also updates to the latest version of the docker cookbook which installs version 1.9 instead of 1.8.

Hey @codebudo this looks great, it looks like there are a few merge conflicts, would you mind rebasing and we can get this merged in?
/cc @someara

I'm going to close this out at this point since we're going through a full refactor of this cookbook. Sorry this never made it in.


Basic Document Formatting

Provide document formatting
align amounts in the current document
try to mimic bean-format

completed aligning up numbers
still need to do normalizing indents

I will look into adding more of the black formatting, maybe with some settings to turn off certain formats


CMS plugin ManyToMany Field  save in the django admin middle table render a schema name error

Django:1.5.4
Django cms :2.4.0
I create a simple schema  in the models backend.
from django.db import models

class Person(models.Model):
    name = models.CharField(max_length=200)

    def __unicode__(self):
        return self.name

I  want to try the django cms plugin, so I define a models .
plugin models file:
from cms.models.pluginmodel import CMSPlugin

from django.db import models

class Hello(CMSPlugin):
    guest_name = models.ManyToManyField(Person)

cms_plugins.py
class HelloPlugin(CMSPluginBase):
    model = Hello
    name = _("Hello Plugin")
    render_template = "hello_plugin.html"

    def render(self, context, instance, placeholder):
        context['instance'] = instance
        return context

plugin_pool.register_plugin(HelloPlugin)

and when I try to save the instance indirectly in the backend.
the middle table schema name is cms_plugin_xxxx,
actually, the cms export sql without the "cms_plugin".

Likely caused by the plugin table rename logic.
This was deprecated on 3.0 and removed on 3.1+


Adjust KCP to Kyma 1.18

Description
Adjust KCP to Kyma 1.18
Changes proposed in this pull request:

remove istio-kyma-patch component
remove installer-local.yaml file, replacing it with a file installer.yaml
fix deprecated resources in installer.yaml
remove unused rule ci-create-release-artifacts from Makefile

/test pre-master-control-plane-integration

/test pre-master-control-plane-integration

/test pre-master-control-plane-integration

/test pre-master-control-plane-integration

I'd say let's add the profile: production into the Installer spec of Kyma

https://github.tools.sap/kyma/backlog/issues/1213

/test pre-master-control-plane-gke-integration


psycopg2.errors.InvalidTextRepresentation: invalid input syntax for type bytea

Hello,
I'm monitoring postgres with pg_activity but sometimes it stops work and throw this error on terminal. How to solve this?
Traceback (most recent call last):
  File "/usr/bin/pg_activity", line 11, in <module>
    load_entry_point('pg-activity==2.3.1', 'console_scripts', 'pg_activity')()
  File "/usr/lib/python3.6/site-packages/pgactivity/cli.py", line 299, in main
    ui.main(term, dataobj, host, options, dsn)
  File "/usr/lib/python3.6/site-packages/pgactivity/ui.py", line 196, in main
    pg_procs.set_items(data.pg_get_activities(ui.duration_mode))
  File "/usr/lib/python3.6/site-packages/pgactivity/data.py", line 265, in pg_get_activities
    "dbname_filter": self.filters.dbname,
  File "/usr/lib64/python3.6/site-packages/psycopg2/extras.py", line 146, in execute
    return super(DictCursor, self).execute(query, vars)
psycopg2.errors.InvalidTextRepresentation: invalid input syntax for type bytea

                                             version

PostgreSQL 12.14 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-16), 64-bit
python3-psycopg2-2.8.6-1.rhel8.x86_64
pg_activity 2.3.1

This got resolved in a later release (3.1 at least, preferably 3.2).

@dlax hello again,
try to update but getting error on both rhel7 and rocky 8, any idea?
Error: Package: pg_activity-3.1.1-1.rhel7.noarch (pgdg-common)
Requires: python3-attrs >= 18.1
Error: Package: pg_activity-3.1.1-1.rhel7.noarch (pgdg-common)
Requires: python3-humanize >= 2.6.0
Available: python36-humanize-0.5.1-8.el7.noarch (epel)
python3-humanize = 0.5.1-8.el7

Yes, it's a known packaging problem, see #322 and https://redmine.postgresql.org/issues/7717. Nothing we can do here unfortunately.
I'd suggest to install with pip, at least until the above issue gets resolved.


core: add CHANGELOG.md and update version in gradle files

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Adds a CHANGELOG.md for documenting future changes. Also increments the version for each gradle file.
Why
To release 1.0.12.
Known limitations
N/A

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check


Update ADLSGen2-HOWTO.md

Description
I had a hard time figuring out how to connect to a delta table that is stored in ADLS Gen2 and only found a way by digging into the source code. I would like to save other people the same trouble by adding this to the docs.
Related Issue(s)

Documentation

Abfss was what I tried initially, unsuspecting users that will not read the manual will most likely do the same.  After failing and not finding the right scheme I went into the code to see what it expects. To save anyone else the search I thought it was a good thing to add it to the docs.

@dgcaron do you want to include the python usage docs update in this PR as well?

Yes sure, I'll do that today
On Wed, 23 Feb 2022, 08:09 QP Hou, @.***> wrote:

@dgcaron https://github.com/dgcaron do you want to include the python
usage docs update in this PR as well?
—
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta-rs/pull/560#issuecomment-1048499778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACKP6IA6CLTL5J7B54HC6EDU4SB2BANCNFSM5OSXT7VQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I'd think fixing the docs isn't the issue, the implementation should be fixed to use either abfss if delta-rs knows how to talk to the adls endpoint or https if it talks to the blob storage endpoint, 'adls2' is never the right scheme to use

I'd think fixing the docs isn't the issue, the implementation should be fixed to use either abfss if delta-rs knows how to talk to the adls endpoint or https if it talks to the blob storage endpoint, 'adls2' is never the right scheme to use

Hi @zeevm, I addressed all these points above, stated again here with some more elaboration:

abfs[s]:// refers to the Hadoop File System driver, which is not being used by delta-rs. From the Azure docs: "The Hadoop Filesystem driver that is compatible with Azure Data Lake Storage Gen2 is known by its scheme identifier abfs (Azure Blob File System). Consistent with other Hadoop Filesystem drivers, the ABFS driver employs a URI format to address files and directories within a Data Lake Storage Gen2 capable account."

Using https URLs would be inconsistent with the fact that delta-rs accepts URIs with a cloud specific scheme (s3:// and gs://) for the Amazon and Google backends.

Based on those two points, it makes sense to use a cloud specific URI scheme in delta-rs, but not abfs[s]:// - hence: adls2://.
Also: the delta-rs Azure backend relies on the atomic rename functionality of Azure Data Lake Storage (ADLS) Gen2 and doesn't work with Azure Blob Storage.
The adls2:// URI scheme is an invention of delta-rs, but there are other libraries that have done something similar for Azure storage. The root cause of this need is that ADLS Gen2 doesn't define a URI scheme other than in the context of Hadoop. It's worth noting that the S3 Hadoop File System driver doesn't use s3:// but s3a:// - the reasons are complicated but this supports introducing adls2:// to distinguish from abfs[s]://.

did i break the build here?

did i break the build here?

Clippy errors can show up because of toolchain updates. Take a look at the error, it has a suggestion to try and fix the clippy error.

yeah, i'll wait for your merge as it is a linter error for files I didn't touch at all.

yeah, i'll wait for your merge as it is a linter error for files I didn't touch at all.

Actually, @houqp already fixed these yesterday. Just fetch upstream on your branch.

do i need to do something additional to get this merged?

do i need to do something additional to get this merged?

No, someone with merge privileges has to review, approve, and merge.

Sorry, there were some intermittent CI errors, I was waiting for the rerun of the CI before doing a merge. Thanks again @dgcaron for the fix! And than kyou @thovoll and @wjones127 for the review :)


[OCPP] info (Connection.cpp:54): Disconnected

I flash MicroOcpp (main.cpp) to ESP-WROOM-32. ESP connects to Wi-Fi successfully but won't connect to my SteVe server at all. The "Etrel Inch Pro" charging station connects to the SteVe without any problems. What am I doing wrong? Here are my the configurations:
SteVe "main.properties":
# Jetty configuration
#
server.host = <IP_ADDRESS>
server.gzip.enabled = true
# Jetty HTTP configuration
#
http.enabled = true
http.port = 6226

SteVe Run log:
Log file: /root/logs/steve.log
Starting.......... Done!
Hint: You can stop the application by pressing CTRL+C
Access the web interface using
- http://<IP_ADDRESS>:6226/steve/manager/home
SOAP endpoint for OCPP
- http://<IP_ADDRESS>:6226/steve/services/CentralSystemService
WebSocket/JSON endpoint for OCPP
- ws://<IP_ADDRESS>:6226/steve/websocket/CentralSystemService/(chargeBoxId)

main.cpp:
#define OCPP_HOST "occp-server"
#define OCPP_PORT 6226
#define OCPP_URL "ws://<IP_ADDRESS>/steve/websocket/CentralSystemService/espdemo"

ESP Serial Output:
rst:0x10 (RTCWDT_RTC_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0030,len:1344
load:0x40078000,len:13964
load:0x40080400,len:3600
entry 0x400805f0
[main] Wait for WiFi: . connected!
[OCPP] info (MicroOcpp.cpp:387): Added ConnectorPluggedInput. Transaction-management is in auto mode now
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (Connection.cpp:54): Disconnected
[OCPP] info (RequestQueue.cpp:45): operation timeout: BootNotification
[OCPP] info (Connection.cpp:54): Disconnected

You need to add port also in the OCPP_URL.

You mean that?
main.cpp:
#define OCPP_HOST "occp-server"
#define OCPP_PORT 6226
#define OCPP_URL "ws://<IP_ADDRESS>:6226/steve/websocket/CentralSystemService/espdemo"

I am getting the same error..

Hey! How you doing?
I'm facing the same problem. Have you solved? What should I do?
Thanks already!


fix(go-test): add space between cmd and path

For go-test runner, when running the TestFile command for a file
which is in a directory nested inside the current working directory,
there's a space missing between the command and file path.
Running the newly added test case without the fix would produce:
go test./go/...
       ^
       Here, a space is missing

Test Update
As there's a need to cd inside the test case, we cannot just use the cd - command. Instead, the current working directory will be stored in a global variable pwd and the after_each function will use that to cd back.

@dhruvmanila Great! Thank you!


can beanstalkd have password protected from client?

is the feature already implemented?

It's pretty unlikely beanstalkd will ever have a feature like this. It wouldn't be much use without an encrypted communication channel, which is more difficult. As some folks have observed, stunnel can help by providing both encryption and authentication.


Quick Accent: Enhancement: Add "¡" and "¿" to "!" and "?" quick accent menu

Description of the new feature / enhancement
When "!" +  is pressed, a menu with "¡" appears.
When "?" +  is pressed, a menu with "¿" appears.
Scenario when this would be used?
This is a set of characters used when writing in Spanish. It marks the beginning of an exclamation or question in a sentence.
Supporting information
No response

Both are available, but they are accessed from the comma (,) key, and only ¿ shows up when Spanish is selected in Quick Accent.
The fix to add ¡ to the Spanish options looks pretty simple, just adding it to the characters in Languages.cs for the LetterKey.VK_COMMA key. The current version of @damienleroy's version includes both characters in the Spanish characters.

It looks like it might be possible to have characters mapped to the question mark ? key with LetterKey.VK_OEM_2. I don't see that actually used for any other language, so maybe there's a technical reason why nothing is mapped to the ? key. Someone else would have to comment on that.

I understand there may not be a usable implementation of this request with the current version of Quick Accent: looking at KeysEnums.cs, it seems to only "listen" to numbers, letters, and comma (the reason for the current assignment to comma of "¡" and "¿" ?) But I am still missing something, because for "/" it offers "÷" as an option.
Maybe in the future, a "customizable" version could be possible, where the user may assign needed keys for their own keyboard, e.g. for me "¡" on the "1"/"!" key, and "¿" to the "/"-"?" key (and a german user assign the "¿" on the upper side of the keyboard to the "ß"/"?" key)
Regarding the comment above, about a technical reason for nothing being assigned to a "?" key, could it be that it is not listed in KeysEnums.cs? Maybe this is for a good reason: looking up some international keyboards, I was not able to find the "?" key on the Portuguese or Ukranian keyboard for example.
Also, Quick Accent seems to ignore the shift key, on a US keyboard the options presented to "3" are the same as for "#" (consistent with what KeysEnums.cs shows), and the "!" and "?" are usually "shifted" keys already, not "base" keys, located over different base keys on different international keyboards, when present.
To be useful, for example on a US Keyboard, see the precedent of the "US International" keyboard from Microsoft, and look up the positions for "¡" and "¿" (on the "1"/"!" and "/"-"?" keys with AltGr).

As new user of Quick Accent on a german Keyboard layout working with spanish characters, I hit "SHIFT + ? + SPACEBAR" as I assume that the qustion mark will would be upside down as a result. hope this helps.


Support completion for tspconfig.yaml file in vscode

Add support of completion for tspconfig.yaml file in vscode
Code refactor considering we are having more code/features in vscode extension
Enable unit test in vscode extension
Also fixed small bugs in restart LSP client and the tool generating license

related issues:
#2049, #2996

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/internal-build-utils

The following packages have already been documented:

:white_check_mark: @typespec/compiler
:white_check_mark: typespec-vscode

Show changes
@typespec/compiler - feature ✏

Support completion for tspconfig.yaml file in vscode

typespec-vscode - feature ✏

Support completion for tspconfig.yaml file in vscode


Problems using the online version of netron

My browser version: chrome-stable:91.0.4472.164
I use online netron: https://netron.app/
When I loaded the model.json model of tfjs, this message popped up:
Error loading TensorFlow model. Unknown weight data type size 'bool' in 'model.json'.
This is the link to my model.json model:
https://drive.google.com/file/d/16JxAZkcLqhKESop17O47BBTGhctuVn-P/view?usp=sharing

@Zengyf-CVer can you share the weights files as well?

@Zengyf-CVer can you share the weights files as well?
https://drive.google.com/file/d/16JxAZkcLqhKESop17O47BBTGhctuVn-P/view?usp=sharing


[Firestore] Remove FirestoreSwift

CI failures will be fixed by GDT min. OS version bump and updated binary artifacts.


Binary response breaks join

Hi
I am trying to use chunnel for connecting to a pheonix channel, but the server responds with a binary response instead of a json when join request is sent. what am I doing wrong? the response is not a proper json and chunnel breaks on channel join.
I took the url and parameters from the website that implemented the channel connection in react.

Problem was with the decode of recv function. server response was in gzip format.


Amcodec

Help with my setup, I have a raspberry Pi running hyperion on rasbian. the pi is just my hyperion server (stand alone), i have setup my hyperion.config with boblight ports enabled... I'm running an Amlogic S805 MXQ and the kernel has video cap enabled, if i install boblightd addon and boblight kodi addon I get LED's to flash for 2 seconds then nothing... If i remove the boblight addons and install the hyperion xbmc addon the LED's will light up but freeze and stay there, if I disable AMCODEC acceleration the LED's work perfect but video shutters
i guess what i'm asking is how can i implement AMCODEC acceleration with hyperion?
thanks for all the help

Interesting point, due to lack of coding skills i could not help
But maybe someone could help
If you found a solution, don´t hesitate to submit the fix ;)

Had the same issue some time ago : http://forum.kodi.tv/showthread.php?tid=219563
Mostly related to Kodi RenderCapture implementation. Couldn't make things change and now since official Android support is pretty dead since Koying left...

@pigbait
As a workaround you could install hyperion again on the MXQ and use the forwarder feature to forward the picture to your pi hyperion
take notice of https://github.com/hyperion-project/hyperion/issues/550#issuecomment-211119886 too

Great thanks for this update. Is that just using the hyperion add-on?

you need to compile hyperion on your own from the master branch für with enabled amlogic grabber (see howto.txt compile at the source) (on your amlogic box)
setting up hyperion on your own at your box with frame grabber enabled (in config.json) and setting up the forwarder (see wiki)
The amlogic grabber is from @redPanther , if it won´t work i can´t help.

We may need sources and some information of your system to setting up our crosscompile machine that we could add offcial install script support for the whole amlogic platform (that you don´t need to compile everything yourself and also to setting up the startup service etc)

new update for the compilehowto
Could you try and report back?
https://github.com/hyperion-project/hyperion/blob/Beta/CompileHowto.txt

Ill see what i can do.. will be probably tomorrow night

Forget what i wrote!
The Wetek.tar.gz is already a amlogic image. Could you test our install script? It should work for you too. The amlogic grabber is already enabled in this build

OK well my MXQ is running Librelec, I installed the hyperion addon from there repo. but now Im lost on how to forward the grabbed images to my raspi.. Ill register on the new support forums.
I also want to thank you and everyone involved on the incredible work done on hyperion up to date

https://hyperion-project.org/threads/hyperion-arduino-and-wetek-core.32/#post-84
maybe this helps you? grab with that, than only forward


WIP 리드미 초안

🤔 해결하려는 문제가 무엇인가요?
🎉 변경 사항
🙏 여기는 꼭 봐주세요!
사용 방법
🌄 스크린샷
📚 참고

Codecov Report
Patch and project coverage have no change.

Comparison is base (74417eb) 91.71% compared to head (d2e80ea) 91.71%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #418   +/-   ##
=======================================
  Coverage   91.71%   91.71%
=======================================
  Files          44       44
  Lines         326      326
  Branches       55       55
=======================================
  Hits          299      299
  Misses         27       27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Request] Dual target locations?

I'm having issues where my plotting seems to stop because the previous plot didn't copy over to the final destination fast enough and it causes the whole plotter to stop.  Could we add dual target plot locations?  Maybe alternating between the two or move to the other if its going to slow?  Losing lots of plotting time from this, even with usb 3.0 drives.

A commit was merged that adds -G, --tmptoggle      Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

That. Would. Be. Awesome.

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

Unfortunately that isn't what you want - the alternate temp toggle is used to alternate between which drive is used for temp 1 and which for temp 2, in order to spread the writes evenly between the two drives. It's got nothing to do with final destinations.

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

Unfortunately that isn't what you want - the alternate temp toggle is used to alternate between which drive is used for temp 1 and which for temp 2, in order to spread the writes evenly between the two drives. It's got nothing to do with final destinations.

Correct me if I am wrong.
Before this commit it would use Temp1 for certain phases and Temp2 for the remaning phases, right?
And now it would just alternate between them and hopefully bring better performance?

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

Unfortunately that isn't what you want - the alternate temp toggle is used to alternate between which drive is used for temp 1 and which for temp 2, in order to spread the writes evenly between the two drives. It's got nothing to do with final destinations.

Correct me if I am wrong.
Before this commit it would use Temp1 for certain phases and Temp2 for the remaning phases, right?
And now it would just alternate between them and hopefully bring better performance?

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

This information is not in the Readme. Does another location exist that documents this?

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

This information is not in the Readme. Does another location exist that documents this?

https://github.com/madMAx43v3r/chia-plotter/pull/503
You're welcome to write the documentation! You're right, it should be written down somewhere.

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

This information is not in the Readme. Does another location exist that documents this?

#503
You're welcome to write the documentation! You're right, it should be written down somewhere.

Sure, I might throw something together. Where should it get published? The Wiki here? The Discord?

Best way would be to clone this repo,edit the readme to say whatever you want, then open a pull request. There are probably some guides that you an find if you google around. Worst case scenario, just post your edit as a comment here and I'll open the pull request.

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

Unfortunately that isn't what you want - the alternate temp toggle is used to alternate between which drive is used for temp 1 and which for temp 2, in order to spread the writes evenly between the two drives. It's got nothing to do with final destinations.

Correct me if I am wrong.
Before this commit it would use Temp1 for certain phases and Temp2 for the remaning phases, right?
And now it would just alternate between them and hopefully bring better performance?

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

So what is the idea behind this then?

A commit was merged that adds -G, --tmptoggle Alternate tmpdir/tmpdir2 (default = false) to the options. However, I did not find any more information on its usage.

Unfortunately that isn't what you want - the alternate temp toggle is used to alternate between which drive is used for temp 1 and which for temp 2, in order to spread the writes evenly between the two drives. It's got nothing to do with final destinations.

Correct me if I am wrong.
Before this commit it would use Temp1 for certain phases and Temp2 for the remaning phases, right?
And now it would just alternate between them and hopefully bring better performance?

Almost. Usually you specify one drive for temp 1 and one drive for temp 2. Now, this flag just swaps which drive is used for temp 1 and which is for temp 2 every plot. It won't give any better performance.

So what is the idea behind this then?

As is stated in the readme, about 75% of all writes happen on tmp2. This would mean that, if using separate SSDs for tmp1 and 2, the tmp2 drive would wear out much faster. The purpose of this toggle is to distribute that wear evenly across the drives, so they fail around the same time, instead of tmp2 failing 3x faster. Personally, I don't see the benefit: it doesn't save the drives from failing, it just prolongs the failure of temp2 and hastens the failure of temp1 to bring them in line. But anyway. To each their own.

The premise of this thread, however, is to add a feature to have the plotter switch to a secondary final folder/drive if the first one is behind or has an error, that way the plotting process doesn't stall due to some glitch (I've experienced several).


Netron not displaying tensor attributes' shape for .PB files

Netron app and version: 7.1.3 Desktop

Steps to Reproduce:
Tensor attribute shapes not displayed properly for .PB files. I cannot attach the .PB for security reasons but have an image for debugging. I noticed this happened when my Netron was automatically updated to use 7.1.0

@lutzroeder I see this on all .PB files, not .ONNX or .PTH

It was saved using TensorFlow


[HTTP] Move all headers(general,request/response and entity headers) inside ‘Entity’

Move headers inside mime Entity

Fixed > https://github.com/ballerina-lang/ballerina/pull/5339


Fix missing string formatting and return msg as an error

String formatting is missing if job is not found. Instead of logging an error, return the error with the message. Jobs need to be restarted to be executed again, but might be deleted already because of their TTL. In this case this is not an error and should be handled properly by the calling code.

Just noticed https://github.com/openstack-k8s-operators/lib-common/pull/449 fixes this as well. Closing.


[Suggestion] - Admin Commands (!VerifyRoles, !VerifyUsernames, !Ping)

Very simple commands that only someone with a linked admin role can use (maybe a website permission or website owner only or even discord permission set)
!VerifyRoles - Verifies all the roles in the discord and syncs them with the website (Pulls from the website to update) {Maybe even update plugin roles with website.}
!VerifyUsernames - Similar to !UpdateUsername (I beleive) in that it updates all users in the discord to ensure their names are the same as their Minecraft Account Usernames.
!Ping - Unlike normal ping commands. This checks the connection to the API Server and the Plugin (if active)

There are definitely some good ideas here. Could you create separate issues fot each comment so I can keep track of progress easier?

There are definitely some good ideas here. Could you create separate issues fot each comment so I can keep track of progress easier?

See #87 #88 #89

See #87 #88 #89


Menu issue

Since yesterday when i deploy the site, menu goes like this but everything is fine in local
Any idea?

This is page source on internet after deployed:

This is how the source looks on local:

Try to delete public/storage folder then run php artisan storage:link

Hey. The issue is printing JSON in source not storage lol

The issue template is not optional when opening issues.

Any idea to resolve this?


Classic Mode Custom Condition with parameters

I've looked through a number of pages and it really isn't very clear. Most of these docs seem to be based around YAML mode even though there is a Custom conditions for a task.
I have a parameter to my Task Group called Components. If Components contains "blah" then I want to run a powershell task
I've put a condition
contains(parameters.Components, 'cube')

but it puts a red squiggly underneath but no error. Is this right or wrong? Where is the doc that will explain this?
Specifically this document talks about how to refer to variables and then says you can also use parameters that are typed, but doesn't point to how I refer to parameters. Also parameters in task groups do not seem to be typed.
This is an area that badly needs more comprehensive documentation and examples for Classic mode. Thanks

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

 I'll make that clearer in our docs. Thanks for flagging this!
For specific implementation concerns and feature requests, I'd recommend that you ask in the Developer Community.


Use imexamine automatically given an image and a list of x,y coordinates

I'd like to use imexamine to emulate IRAF's psfmeasure task, which is useful to estimate the average FWHM of an image.
To do this, I need to pass to imexam an image, and a list of x,y star center coordinates. Then I have to emulate pressing the a key on each of those x,y coordinates, and log the results either to screen or a file.
Is this possible? Or must I press the a key manually star after star to obtain the estimated FWHMs?

nope, you can just feed the data and x,y positions in and save the information.... You probably don't need the full aperture photometry though, yes? You can save the aperture photometry similar to the example below, but if you're mostly interested in the gauss fit results,  you can use the Gaussian line fit function in either x or y ( attached to the j and k keys) and return the fit directly, then save however you like. It could look something like this:
from imexam.imexamine import Imexamine
from astropy.io import fits

#get your data into an array
data=fits.getdata(filename)

#get an object up with your data attached
plots=Imexamine()
plots.set_data(data)

If you have a text file with columns of x, y, star id, then one option is to read the information and save the results as a dictionary, you might also try an astropy table, here's a simple read and save, just as a quick example
starlist=list()
with open(filename) as fh:
    for line in fh:
        starlist.append(tuple(map(float,line.strip().split())))

Now you have a  list of tuples which are the float values of x,y,star id
You can continue and feed this to the plots object and save to a dictionary. I'll be a little obtuse here and just save a dictionary indexed by star id with a list of values in order x, y, std, mean, amp, fwhm
If you were doing this in an astropy table you could do math on the columns
#make a function to calculate the fwhm
g = lambda x: x * np.sqrt(8.0 * np.log(2.))

or you can import the function from imexam, note that this function currently
returns a tuple (fwhm,fwhm) because it supports accepting the stddev from x and y
assumes they are equal when only 1 value is given
from imexam.math_helper import gfwhm
fwhm=gfwhm(stddev)[0]  #to get just the x fwhm

#continuing on ...

results=dict()
for star in starlist:
    x,y,sid=star
    gauss=plots.line_fit(x,y,genplot=False)
    results[sid]=[x, y, gauss.stddev, gauss.mean, gauss.amplitude, g(gauss.stddev)]

Now you have a dictionary, indexed by the star id with all the information, you can use this to make plots, for example plot of the x vs fwhm to see how it changes across your image, or calculate the average fwhm for all the stars... you can save the results in any form you like :-)
As long as centering is on, line_fit and column_fit will first center the object using a gaussian 2D fit and then calculate the 1D fit at that center in either the x or y direction.
Does this example help? I believe all/most of the functions accept genplot=False and instead return useful information.

Thank you very much for your detailed answer @sosey! A couple of comments:

Importing imexam, even if it is not used, makes the plt.show() window crash (or immediately close). Try the following small script for example. In my case, the plot window never appears (I think it appears and is closed so quickly I can't even see it).

import matplotlib.pyplot as plt
from imexam.imexamine import Imexamine

plt.scatter([2., 3.], [2., 3.])
plt.show()

I'm using Python 2.7.12 and imexam v0.6.2 (installed via AstroConda).

Why does line_fit requires the x,y coordinates as integers? Doesn't it make more sense to accept floats?
The line_fit Notes state: "If centering is True in the parameter set". Where is this "centering" parameter? It is not listed as a parameter from line_fit.
Calling plots.line_fit(x, y, genplot=False) writes the xc, yc values to screen. It is a minor annoyance, but I'd like to turn this off if possible.
If I understand correctly, plots.line_fit() returns the 1D FWHM of a star in the x axis. To obtain the y axis FWHM I'd have to use

gauss_y = plots.column_fit(x, y, genplot=False)
fwhm_y = gfwhm(gauss_y.stddev)[0]

Is this correct?

Sometimes I get negative FWHMs using the method described in your answer. What is the meaning of such values?

I'm in the middle of a conference this week @Gabriel-p, but I will get to these questions as soon as I can. In the mean time, if you are using a conda virtual environment, will you dump it to a text file and send me the result so I can repilcate it?   conda env export > environment.yml

Of course @sosey, take as much time as you need.
Here's the environment.yml.zip file. Let me know if there's anything else you need.
In the meantime, I'll add another question:
7- What FWHM does the a key return? The averaged x, y (line, column) FWHMs? The FWHM of a 2D fitted Gaussian?

Thankyou. The "a" key is returning photometry from photutils, but I use the sigma from the 2D gaussian fit which recenters the star to calculate the FWHM
You can call this function yourself,  imexam.math_helper.gauss_center, it takes a data array which is assumes is centered on the object and then calculates a better 2d center with the fit.

Thanks @sosey. Updating status of the questions:

Still an issue.
Resolved. I see that it expects the column/line number, which is why it needs an integer.
Resolved.
I could "fix" this wrapping calls to column_fit or line_fit around

sys.stdout = open(os.devnull, "w")
...
sys.stdout = sys.__stdout__

It feels hackish though.

Unless I understood something wrong, this is correct.
Here's a comparision of line_fit and column_fit FWHM values, with those returned by psfmeasure for the exact same group of stars:

# psfmeasure  gfwhm_line_fit  gfwhm_column_fit
4.684   -18.212 34.701
4.992   202.577 7.016
0.906   39.609  -585.165
1.018   23.748  -36.779
0.809   -1.603  50.768
5.428   6.345   5.861
4.948   0.853   4.783
5.076   -5.210  5.225
4.352   3.209   -4.828
5.268   1.266   0.450
4.392   -5.525  4.550
5.268   1.266   0.450
4.496   -5.193  6.898
5.588   5.948   5.174
5.588   5.948   5.174
5.068   -5.683  4.860
5.096   5.544   5.346
5.096   5.544   5.346
5.024   -0.357  -5.041
4.052   -5.909  -4.805
5.024   -0.357  -5.041
5.024   -0.357  -5.041
4.052   -5.909  -4.805
5.5 -5.429  5.551
2.244   13339.176   628.546
5.26    -5.435  5.176
5.716   7.686   -2.709
0.969   1.051   5.259
4.82    0.723   -4.977
3.856   -5.134  4.826
3.414   4.721   4.228
3.594   0.648   0.689
4.376   5.480   7.224
5.04    -83.771 -5.033
4.628   5.032   4.890
4.956   -5.244  -5.252
4.908   6.064   -5.513
4.908   6.064   -5.513
4.672   -0.698  4.755
0.993   1.405   1.056
4.756   -5.871  -5.100
5.3 5.808   -6.084
5.032   6.905   -0.034
4.864   -5.313  0.890
4.84    -5.099  4.864
0.941   1.067   0.181
5.292   -5.882  0.643
4.716   5.091   4.874
4.308   -5.071  -4.741
4.476   4.918   4.941
6.02    -5.945  -5.896
1.244   33.442  0.261
5.276   5.743   -5.184
4.712   5.154   4.880
2.796   4.779   -4.766
5.232   6231.627    -5.071
4.912   4.910   5.102
4.82    5.272   0.873
4.788   5.600   5.024
4.032   -0.401  4.818

I understand that x,y FWHMs can differ a bit, but I don't understand the negative values, or when line_fit and column_fit differ by a lot.

"The "a" key is returning photometry from photutils", could you expand a bit on how the FWHM is calculated with a? Is this a 2D Gaussian (or Moffat, etc) fit to the data?

@Gabriel-p  Some more answers:
2) Why does line_fit requires the x,y coordinates as integers? Doesn't it make more sense to accept floats?

This is to accommodate the imexam loop.  The integer pixel location of the pointer is used to make a subarray cut of the data to localize the fit, then with centering on,  a better floating point center is found assuming that the object is close to the center of the smaller array. This gives a little more flexibility for people visually interacting with an image and clicking on things within a few pixels of the objects true center.  In reality, you could pass a float number to line_fit, but it will get converted to make this smaller array from the data ...actually in looking at the code I see a bug here too where the int isn't getting applied, so I'm going to fix that :)  So my first suggestion is to pass a float or int to line_fit and with centering on you should be fine. Secondarily, I would suggest using the astropy models  directly, you can take a look at the code in the math_helper.gauss_center function, ignore the first part that guesses the center, and use the rest of the code to do the 2d fits for yourself.
4) Calling plots.line_fit(x, y, genplot=False) writes the xc, yc values to screen. It is a minor annoyance, but I'd like to turn this off if possible.

I think this is part of the logging that I have enabled. Currently a log message at debug level is sent to the screen, you could set the logging to a higher level. From the full object, if you are using the connect() the loggins sets up and there is a setlog() function which you can pass the logging level to, setlog(level=logging.WARNING) I think would turn off the print to screen. But this logging function isn't attached to the Imexamine() object. Let me think about the best way to do this a little bit.
If I understand correctly, plots.line_fit() returns the 1D FWHM of a star in the x axis. To obtain the y axis FWHM I'd have to use column_fit

yup, these are the 1D, line or column fit functions
5) gauss_y = plots.column_fit(x, y, genplot=False)
fwhm_y = gfwhm(gauss_y.stddev)[0]
Is this correct?

that should be correct
6) Sometimes I get negative FWHMs using the method described in your answer. What is the meaning of such values?

Most likely the centering is off and the model fit is barfing, often getting good starting conditions can be tricky and data dependent. I have them set to try and cover the average dataset. Since you are doing something very specific I think you should consider just using the astropy models directly like discussed above? If you run into issues there, you can send me more questions, or ask on the astropy models page.

I had the above typed up when your update came in, lemme think on the psfmeasure differences a bit and I'll get back to you


(enhancement) Configs

More configs pls
There are config files that have one or two configs where several machines are still missing Config files.
Missing: AnimalFarm, AnimalGym, AnimalReleaser, CropCloner, CropFarm, ElectricButcher, FluidSolidifier, LiquidXPStorage, FarmerMachine, PoweredPump, and Sewer.

well... yeah... but the specific config files are usually for custom recipes...
the rest of the machines don't have any kind of recipes... so not sure what would be in those configs.

Good point....


[Bug]: no output of profile when VLLM_TORCH_PROFILER_DIR is enabled for vllm serve

Your current environment

The output of `python collect_env.py`
Your output of `python collect_env.py` here

Model Input Dumps
No response
🐛 Describe the bug
I start a vlllm server with
VLLM_RPC_TIMEOUT=1800000 VLLM_TORCH_PROFILER_DIR=***/MLinference/language/llama2-70b/vllm_profile_output  vllm serve ${CHECKPOINT_PATH}   --tensor-parallel-size 4 2>&1 | tee vllm_serve_profile_opt-13b_log.log &

then use scripts from MLinference/language/llama2-70b/main.py to feed request to the server. After the scripts is finished I cannot see any output of profile in the given folder.
But I can see from the vllm_serve_profile_opt-13b_log.log that the torch profiler is enabled.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

see:https://docs.vllm.ai/en/latest/contributing/profiling/profiling_index.html

Yes, I followed the exact steps on this page.

which vllm version are you using?
BTW, could you plz provide your serving script?Perhaps you forgot to add the --profile argument

I'm using vllm0.6.4
the script i used is
mkdir -p ***/MLinference/language/llama2-70b/vllm_profile_output
echo start vllm server
VLLM_RPC_TIMEOUT=1800000 VLLM_TORCH_PROFILER_DIR=***/MLinference/language/llama2-70b/vllm_profile_output  vllm serve ${CHECKPOINT_PATH}   --tensor-parallel-size 4 2>&1 | tee vllm_serve_profile_opt-13b_log.log & #--pipeline-parallel-size 2
server_pid=$!

sleep 100s  # wait for the server to start

echo start main_profile.py
python3 -u main.py --scenario Offline \
        --model-path ${CHECKPOINT_PATH} \
        --mlperf-conf mlperf.conf \
        --user-conf user.conf \
        --total-sample-count 10 \
        --dataset-path ${DATASET_PATH} \
        --output-log-dir offline-logs \
        --dtype float32 \
        --api-model-name facebook/opt-13b \
        --vllm \
        --api-server http://<IP_ADDRESS>:8000 \
        --device cuda:0 2>&1 | tee vllm_profile_opt-13b_log.log

kill $server_pid

When I tried to add the --profile command in the vllm serve, it gives error message to me

what's the error messgae?

I'm not using openai server, I'm starting a local server with the command
VLLM_RPC_TIMEOUT=1800000 VLLM_TORCH_PROFILER_DIR=***/MLinference/language/llama2-70b/vllm_profile_output  vllm serve ${CHECKPOINT_PATH}   --tensor-parallel-size 4

If I add --profile to this command, it will show
vllm: error: unrecognized arguments: --profile

I'm not sure if the script you're using can accept this arg, but you can try:
python3 -u main.py --scenario Offline \
        --model-path ${CHECKPOINT_PATH} \
        --mlperf-conf mlperf.conf \
        --user-conf user.conf \
        --total-sample-count 10 \
        --dataset-path ${DATASET_PATH} \
        --output-log-dir offline-logs \
        --dtype float32 \
        --api-model-name facebook/opt-13b \
        --vllm \
        --api-server http://<IP_ADDRESS>:8000 \
        --profile ## add profile
        --device cuda:0 2>&1 | tee vllm_profile_opt-13b_log.log

You can first try using the vllm script to see if you can generate profile files, and  you should understand how your script(main.py) calls the vllm service.

Ignore the above links, I typed in the wrong PR number.


.ipynb has no PowerShell support beyond basic kernel execution

Describe the bug
Reproduction steps
Install Polyglot Notebooks and recommended packages.
Set the default language to PowerShell.
Set the default format to .ipynb.
If I run the following command from the command pallet:
Polyglot Notebook: Create default notebook
I get this:

Please consider the language in the bottom right of this picture. ☝
The kernel is set to pwsh - PowerShell and the language is set to Code.
The language should be PowerShell and not Code.
As a result syntax highlighting is broken, autocompletion is broken, and extensions for PowerShell are not applied.
What is the intended functionality here?
In my humble opinion:
If .ipynb is not supported, don't list it as supported.
If  pwsh - PowerShell does not work in combination with .ipynb format, then don't list it as a kernel for the .ipynb format.
Thanks for your time and consideration.
Please complete the following:

OS

[x] Windows 11

Frontend

[x] Visual Studio Code

The issues you're seeing aren't related to the file extension, and PowerShell is working generally. There are two different, unrelated issues I think you might be hitting.

As a result syntax highlighting is broken...

Lack of code coloring is a known issue (#2683) and it's unrelated to the other issue you're seeing.

...autocompletion is broken, and extensions for PowerShell are not applied.

When code won't run or completions aren't working, it's usually because there's a pending VS Code update, which often then triggers a Polyglot Notebooks extension update. Please click the gear icon in the lower left of VS Code and select Check for updates. Then also check (after an update was applied, if there is one) in the extensions item in the sidebar to see if there's a Reload required button under the Polyglot Notebooks extension.

The language should be PowerShell and not Code.

The separation of the kernel chooser from the "Language Mode" chooser (Code vs Markdown) is a recent design change. It's possible in Polyglot Notebooks to add multiple subkernels for the same language (for example, multiple SQL connections to different databases). Choosing just the language is insufficient, so we added the more detailed chooser which shows both the kernel name and the language used by that kernel.

I've started an up-to-date Windows 11 Pro VM, installed VS Code and the Polyglot Notebook extension and created a new notebook.
As you said, the colorization and IntelliSense work just fine.
I assume this is update related on my VS code Server instance.
I've been running it since beta preview, but can't recall seeing any updates.

VS Code updates once a month, typically in the first week of the month. The updates are often very subtle, unless they break something. I'm not sure if the update cycle works differently for VS Code Server or if there's something about your server configuration that's preventing updates.

You don't see the updates on VS Code Server so you can't update it remotely.
VS Code and VS Code Server are the same executable; the latter is just code.exe tunnel with some additional setup.
I've updated VS Code by running it without the tunnel command, and checking for updates.
That fixed the IntelliSense and colorization, but only when I run code.exe.
When I run code tunnel only the IntelliSense suggestions work.
Colorization doesn't work, and error-highlighting doesn't work.

I've resolved this issue by uninstalling .NET interactive and switching to the baseline Python implementation instead.
That one works out of the box.


Type guards do not work when using the non-null assertion operator

TypeScript Version:  2.3 (Playground)
Code
Using strictNullChecks.
interface Foo {
    optional?: number;
}

interface Bar {
    foo?: Foo;
}

function test(bar: Bar) {
    if (bar.foo!.optional) {
        let num: number = bar.foo.optional;
    }

    if (bar.foo && bar.foo.optional) {
        let num: number = bar.foo.optional;
    }
}

Expected behavior:
No errors or warnings.
Actual behavior:

num in first if block is number | undefined
bar.foo in first if block can be undefined

this is behaving as intended. the ! operator is not a narrowing operator. it just muffles the error. so every time you use the variable you need to reapply !. if you want to narrow, use one of the narrowing patterns, like bar.foo && bar.foo.optional is suggested above.

Why? There is no way I can enter the first if block unless both foo and optional is not undefined.
And even so, TypeScript still thinks bar.foo!.optional inside the block can be undefined, even though I just checked that it's not. The very fact that I used ! on foo breaks the type guard on optional. See the second if block below.

The ! operator does not change the type.. it just tells the compiler, i know it is null | undefined, but i want to access it anyways.. it is similar to your regular type assertion, think of:
var x: number | string;

if (typeof (x as number).toFixed === "function") {
      (x as number).toFixed();  // need to cast again
      var y: number =  x;   // error, x is number | string
}

@mhegazy what's our excuse for not unwrapping this, though?

If we retrieved a property of an identifier, there's no reason to not consider that a truthiness guard of the same identifier. It's clearly not null / undefined

The assertion has no scope, it can appear in the middle of an expression, or in the middle of an if block, it is not clear what that means to the type for the rest of the block.. other narrowing constructs do have clear scoping semantics /control flow implications.
We did discuss this before, but i can not find the issue at the moment.

This isn't about applying the ! operator to the rest of the block. It's about this:
function test(bar: Bar) {
    if (bar.foo!.optional) {
        let num: number = bar.foo.optional; // Should not error!
    }

The dotted property access on bar.foo should act as an equivalent type guard to if (bar.foo) {, because the fact we got inside the if block without crashing means it's not null/undefined

related https://github.com/Microsoft/TypeScript/issues/9640

Well, it's actually about this:
    if (bar.foo!.optional) {
        let num: number = bar.foo!.optional; // "Type 'number | undefined' is not assignable to type 'number'"
    }

although it was also surprising that I have to use foo!. twice, even inside the if block.

If the ! operator is used before a . or (), it could be considered a narrowing operator.
For example, this:
function test (a: { b?: { c: string } }) {
  console.log(a.b!.c)
  console.log(a.b.c)
}

could be considered as being more-or-less equivalent to:
function test (a: { b?: { c: string } }) {
  if (a.b == null) throw new TypeError()
  console.log(a.b.c)
  console.log(a.b.c)
}

but the throw itself is done by the engine..... and is an expression throw.

Has this ever been resolved?


[Nox.Lib] Is depending on concrete database providers implementation

We need to dynamically add the necessary dependencies to the User Project regarding DatabaseProviders


Restyling/Case search (with new action bar)

Redesigned case search and action bar

The Java portions of this PR look good to me.

Added a bunch of smaller things, but two major points we need to clear up before we can pull

The huge "Everything is final" refactor doesn't really seem like it should be in here. It makes the PR very hard to review and I don't think it's a desirable change for now

Pre-Honeycomb it seems like we still need to rely on the old search bar, but it doesn't look like that's enabled here.


People can take items from gui

What is your Quests version / build number (do not say "latest")?
3.6.6
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
Hi, ive problem with quests on 1.13.2 with gui. If you click on item in gui you take the item and then you could take it to inventory :/
https://pastebin.com/4YVTN3dD
If applicable, can you provide an example quest from quests.yml using Pastebin.com?
https://pastebin.com/gXiEvNtv

I wasn't able to replicate this, but the error should now be caught in a way that will allow the inventory to close before any items can be taken.
Use http://ci.ac3-servers.eu/job/Quests/730/ once it's available

This happend for me one 1.14.4 too. I have 2 quests NPC's. One I can pick items from the GUI and one I can't

@Johannes173 Please open a new ticket including your Quests version and a copy of the problematic quest.
Sent with GitHawk


Added About Us feature to navigation

Description
About us page helps users to know more about the website and project, learns mission and vission of the project.
Related Issue
Fixes #376
https://github.com/user-attachments/assets/965a85b4-d0f1-46f6-8ff2-7edbdd84d141
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X ] My code follows the style guidelines of this project
[ X] I have performed a self-review of my own code
[ X] I have commented on my code, particularly in hard-to-understand areas
[ X] I have added tests that prove my fix is effective or that my feature works
[ X] New and existing unit tests pass locally with my changes
[ X] Any dependent changes have been merged and published in downstream modules

@subhadipbhowmik please review and allow merge
and kindly assign level 3, gssoc-extd labels


visibilitychange event does not fire

I ran into a strange bug(?) where the visibilitychange event does not fire in my app built on the Epic Stack. I created a minimal reproduction from a new Epic Stack app. I'm totally open to the idea that this is somehow my fault 😅 but I haven't found the explanation yet after a lot of debugging. Any help / advice / guidance would be much appreciated!
Here are some reputable blog posts that indicate that this isn't a remix problem:

https://sergiodxa.com/articles/automatic-revalidation-in-remix
https://sergiodxa.com/tutorials/keep-your-loader-data-fresh-in-remix
https://ben.page/remix-focus

The use case is reloading the page after a period of inactivity. I ended up using the focus / blur events on the window, which DO work. I'm really baffled by the visibilitychange event not firing though.
Reproduction:
https://github.com/NGimbal/epic-stack-visbilitychange-repro/commit/e0e54c0b796e932e341feeb18a0e3d7619ff682e

Hi @NGimbal!
I'm afraid I don't have time to look at this now (you might bring it to my office hours later this week if you like).
But as the first bit of debugging I would recommend is create a brand new remix project without the epic stack and see whether you're having the same issue. Try to determine whether it's really something in the stack. If it is then try to remove as much as possible until you find the cause.

No worries @kentcdodds - I just wanted to flag it because it was weirding me out. I will debug further when I have the chance.


IRGen: Use an invariant load to load metadata base offset

@rjmccall Is this correct?

@swift-ci Please test

It's not safe, no.  The load is invariant as long as it isn't hoisted before something that's guaranteed to trigger class metadata initialization, like fetching the class metadata, but it's conceivable for this load to appear in a function that triggers the class metadata to be initialized during its execution.
For example, there could be a loop like this:
var total = 0
for i in 0..<100 {
total += MyClass().number
}
invariant.load allows the number to be hoisted outside of the loop, but it's possible for the first iteration of the loop to be what triggers the computation of MyClass's metadata bounds.

@rjmccall Ok, I'll revert then.


Migrate Zulip Nagios plugins out of zulip_internal/

I haven't had a chance to fully test this, but preliminary tests are looking good.

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

Having now tested that this all works, I merged this.


Mandrill DNS records

Use TXT record instead of SPF record type
Mandrill SPF verification doesn't recognize current SPF value.
From Route 53 docs:

... we no longer recommend that you create resource record sets for
which the record type is SPF ... its use is no longer appropriate for
SPF version 1; implementations are not to use it ... Instead of an
SPF record, we recommend that you create a TXT record

(http://docs.aws.amazon.com/Route53/latest/DeveloperGuide/ResourceRecordTypes.html#SPFFormat)
Add mandrill DKIM record and change SPF/DKIM TTL to 300
Setting a short TTL in case we need to revert the records. DKIM
value copied from Mandrill setup instructions page:
https://mandrillapp.com/settings/sending-domains?q=digitalmarketplace

Looks right to me :+1:


Use resource variable for ODH Dashboard project name when checking for the ODH Dashboard title

To support running ods-ci against Open Data Hub deployments, this PR adds resource file variable to allow anyone to set the project name referenced in the Dashboard title xpath to support using Open Data Hub vs Red Hat OpenShift Data Science
This change should not affect existing functionality since the default value is still Red Hat OpenShift Data Science and can be set via the global variable ODH_DASHBOARD_PROJECT_NAME

Being tested here (running all Smoke):
https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/view/QE_Internal/job/rhods-ci-pr-test/1614/console

@jgarciao @bdattoma After checking the logs for rhods-ci-pr-test/1615, it appears the change is working and does not affect existing functionality based on the successful execution of Wait for RHODS Dashboard to Load in Test.Jupyterhub.Test.
Just to be absolutely certain, I ran this against a RHODS cluster to verify that the standard test to login to the dashboard, spawn a notebook and interact with JupyterLab still worked as intended

I've tested it in this job running all smoke using the long running cluster (rhods 1.23) but half of the tests failed:

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/view/QE_Internal/job/rhods-ci-pr-test/1615/TestReport/

I think the failures are not related to this change. @bdattoma could you take a look when possible?

not sure Jorge, we ran smoke for 1.23 live testing and it was 100% pass. Could you run once again?

Once Landon rebased this PR to master I've tested it running Smoke on a 1.24 cluster and 100% of the tests passed
https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/1629/


Incorrect pressure measurement (bug introduced since v2.0.5)

I have just migrated from v2.0.5 to v3.3.4 (latest).  The sensor operates as expected except that the pressure measurement is ~485 hPa when it should be ~985 hPA.  There is no issues with the humidity or temperature sensor.  The pressure sensor noise and response time are as expected; it is only the magnitude that is incorrect.
Here is my relevant code for reference:
// Initialise driver
bme280.dev_id = BME280_I2C_ADDR_PRIM;
bme280.intf = BME280_I2C_INTF;
bme280.read = &Bme280Read;
bme280.write = &Bme280Write;
bme280.delay_ms = &Bme280DelayMs;
bme280_init(&bme280);

// Set settings
bme280.settings.osr_p = BME280_OVERSAMPLING_16X;
bme280.settings.osr_t = BME280_OVERSAMPLING_2X;
bme280.settings.osr_h = BME280_OVERSAMPLING_1X;
bme280.settings.filter = BME280_FILTER_COEFF_16;
bme280.settings.standby_time = BME280_STANDBY_TIME_1_MS;
bme280_set_sensor_settings(BME280_OSR_PRESS_SEL, &bme280);
bme280_set_sensor_settings(BME280_OSR_TEMP_SEL, &bme280);
bme280_set_sensor_settings(BME280_OSR_HUM_SEL, &bme280);
bme280_set_sensor_settings(BME280_FILTER_SEL, &bme280);
bme280_set_sensor_settings(BME280_STANDBY_SEL, &bme280);

// Set power mode
bme280_set_sensor_mode(BME280_NORMAL_MODE, &bme280);

// Read data
struct bme280_data comp_data;
bme280_get_sensor_data(BME280_PRESS | BME280_TEMP | BME280_HUM, &comp_data, &bme280);

// Convert measurements to degrees Celsius, hPa, %RH
float temperature = (float) comp_data.temperature * (1.0f / 500.0f) + 24.0f;
float pressure = (float) comp_data.pressure * (1.0f / 25600.0f);
float humidity = (float) comp_data.humidity * (1.0f / 1024.0f);

The issue has now been resolved.
API v2.0.5 uses a pressure units of 256 lsb/hPa for 64-bit calculations.  These are also the units indicated on page 23 of the latest datasheet (Rev. 1.3, May 2016).  However, API v3.3.4 uses a units of 100 lsb/hPa.  This is already indicated in the repository README.
The correct pressure measurements are obtained when using the units indicated in the repository README.


Prevent 5xx error during evicting deployments that has small replica count

If your deployments are well replicated, then this is not a problem.
However, when they are not, then there's still be ALB 5xx errors while they are evicted.
These eviction can be triggered by kubectl drain, node termination, etc.
Evicted pods should be handled in this order:

The pod is requested to be evicted.
Increase the replica count of the deployment of the requested pod. However, you can't isolate the pod yet since it'll drain the pod.
When the new pods are ready, you can isolate the pod, restoring the replica count, and starts delayed deletion.

Related comment: https://github.com/foriequal0/pod-graceful-drain/issues/31#issuecomment-1091618985

Hi, I'm following up here with quotes from your linked comment.

However, the eviction process seems to be different. Pods are evicted from the node first (and pods are drained) then replicaset controller tries to reconcile the pod replica count later without being noticed by eviction processors. I didn't recognized this until now.

It's interesting that it worked during a cluster upgrade which did create new nodes but not when recreating a node group independently. Is this process different between both of those methods?

I might be able to temporarily increase the replica count of the deployment when the pod is requested to be evicted in later versions.

This seems pretty scary because with a manually sized cluster there might not be room to add +1 replica for every deployment at once, or would this happen 1 deployment at a time so there's only ever +1 replica for 1 deployment at a time? If it's the second option that might work but is still a little scary since you might have a node with 8GB of memory with 2 pods running that requires 3GB of memory each, the node couldn't fit a 3rd node -- even temporarily.

To mitigate this, I think we should have enough replicas and make sure they are distributed across multiple nodes.

At the moment I'm running a 2 node cluster across 2 azs with 2 replicas of everything. That's just enough safety for me to feel comfortable without breaking the bank.

Thank you for the kind words!
I am also not sure about the solution I came up with for this issue. Let's sit still and think for some time.
Since you have 2 replicas for everything, I guess that the downtime during the node group renaming was due to the skipping webhooks.
Actually, I couldn't reproduce downtime reliably with renaming the node group. That's why I think you might have been lucky when you upgrade the cluster.
I'll create another issue for this here #34 :sweat_smile: I think that would be more appropriate for this situation.


i get this error when i'm tring to use electron app with angular-localforage

Uncaught Error: Cannot find module 'localforage'
Uncaught Error: [$injector:modulerr] Failed to instantiate module intouch due to:
Error: [$injector:modulerr] Failed to instantiate module LocalForageModule due to:
Error: [$injector:nomod] Module 'LocalForageModule' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

Sounds like the module isn't being included by your module bundler (webpack/browserify/rollup/etc). Can you provide a bit more detail and maybe your main angular.module('yourModule', ['LocalForageModule']) declaration?

I am having the same issue and still doesn't know why...

Can you provide some more detail about your build process and your module declaration? I use this module in my Electron app with no problems, so I imagine there is something in your configuration that is a little off.

@mostafa-raafat and/or @Maximeesilv
I'm going to close this issue since I haven't heard back from either of you about this issue, so I'm going to assume it's now working for you.
Feel free to re-open it if you have more details to share.

Oh, actually, I just re-read the issue y'all were having:
You have to load localforage before you loading this library. If you're using a bundler, make sure you have localforage installed and that it is available to this module to consume. If you're not bundling, make sure you load the localforage script before this one. Open back up if you have anymore questions!
I'll add a little helpful error messaging around this as I've been bitten by this before, too! 👍


False positive Embed (Basic) API

I used the latest version (pulled the code one hour ago)
Try this key AIzaSyB3MeUSpCntta2BW8UeLmVlRgnM7tB6hio.
Then if you try the generated iframe code:
<iframe width="600" height="450" frameborder="0" style="border:0" src="https://www.google.com/maps/embed/v1/place?q=Seattle&key=AIzaSyB3MeUSpCntta2BW8UeLmVlRgnM7tB6hio" allowfullscreen></iframe>
You will get this:

I am saving the iframe code as html file on my local pc and opening it in the browser which successfully loads the iframe.
On your case, since you loaded iframe to www.w3schools.com, probably it is returning error.
Adding <meta name="referrer" content="no-referrer"/> to your HTML code will bypass that control & solve the issue.
Reference to that: https://ozguralp.medium.com/google-maps-api-not-the-key-bugs-that-i-found-over-the-years-781840fc82aa

Thanks for the info @ozguralp .  I will try harder next time.


Add optional data field to SourceItem

relevant discussion: https://github.com/build-server-protocol/build-server-protocol/discussions/691

Also please add a changelog entry


reminders: Clean up unfinished feature.

This is not for beginners, and the issue is deliberately vague about how to proceed.
We have a reminder feature that was started a couple years ago and is now in somewhat of an unfinished state.  AFAIK there were no major blocking issues on the feature other than competing priorities (school, other Zulip stuff was getting more attention, etc.).  See #8121 for an example PR related to this feature.
Long term we want to bring this feature to completion.
In the short term we may want to do some things like remove feature_flags.
More discussion here:
https://chat.zulip.org/#narrow/stream/6-frontend/topic/reminder.20UI/near/823022

Is this a duplicate of #11227?

No, reminders are different from send later.  I also think we should just finish the feature later this summer rather than removing the feature flag for it, so I think we can close.

Ok, so I assume the relevant feature request is #1739.

Yep!


Smooth switching between desktops in Windows 11

Description of the new feature / enhancement
Currently the default way to switch between desktops is: Windows key + Ctrl + Arrow Key
The sudden transition for me is a bit jarring.
I'd like an option that's similar to paperwm on gnome where it's like you are smoothly transitioning from one desktop to another.
Scenario when this would be used?
I'd like to quickly know the content of another desktop without having to mouse-over the task view icon but I don't like it to be as jarring as using the shortcut to switch between desktops (Windows key + Ctrl + Arrow Key)
Supporting information
No response

This is the type of work that needs to happen at the OS level


Floppy eject/insert

This patch implements two new methods on FloppyController, eject_fda and insert_fda.  These are not exposed to the UI yet, but from the browser console in debug mode they work for removing and inserting floppies in MS-DOS at least.
Caveats:

FloppyController does not implement the format track command, so you need to give it a pre-formatted disk image.
FloppyController does not seem to emulate all the relevant internal controller state, so I made a best-effort patch here (references: this website , osdev ).  I did what I could around reset behavior, extending existing commands, and reporting errors on seeks and reads, but this is pretty far outside my wheelhouse.

So, please see this PR as a request for testing and review towards part of #54 .

The first failing test is an openbsd floppy test that tests fdb behavior. Arguably the test should never have passed since fdb was never used in floppycontroller.

Thanks! This works pretty well so far. I've left a few comments above, but in general there's nothing major in the way of merging.
A few notes:

A couple of tests fail with "Unimplemented floppy command call" (Windows XP, React OS). I'm going to improve the command parser a bit and turn this into a warning (the commands executed by these OSes don't always make sense)
Currently the floppy disk is only really tested for OSes that boot from it (plus the one test for MS-DOS hard drive + floppy). It would be nice to survey which existing OSes support the current implementation, and make sure this PR doesn't break any of them. I can take care of that.
A test would be nice that demonstrates ejecting/inserting (i.e., boot MS-DOS, run dir A:, insert, run dir A:, eject, run dir A:)

Few stylistic things:

{ after function/if/else should go on its own line
Use === and !== over == and !=

Thanks for the review. I’ll address the style issues and wrong flag sets, and try to add an automated test using eject and insert.

I ended up adding an “extra_images” config option (and field on V86Starter) so that the starter can download/initialize all the floppies or CDs or whatever that will be necessary. This way it’s easy to code up something to swap disks by name (as I do in the integration tests).

I've added an async function on V86 to load an image file description as an image description, which I use from run.js.  In the future, this could be used to refactor continue_init (( settings["hda"] = this.disk_images["hda"] = await this.load_images(this.disk_images["hda"]); )) and allow for loading all the image files in parallel (using e.g. Promise.all).

I went ahead and slightly refactored continue_init in the way I described, just so we have only one code path for loading image files.  I left the put_on_settings function intact and did not use Promise.all since I thought it would complicate the progress bar-related code even more.

Sorry for the spam--if you don't want the last commit, feel free to just use the preceding one and keep the code duplication.  I thought adding the async function to load an image would be a smoother, nicer change than exposing v86util or the buffer types as exports of the library, since they kind of feel like implementation details.

I've refactored the tests and buffer loading. Also improved the command parser and implemented port 3F4 reset (for ReactOS and winxp, previously this code was never hit because we didn't run it without a floppy disk).
Will merge when tests pass.

Also pushed another change to expose it in the UI.

Thanks!


update to the table ids in preproduction and production

Overview
I've made an update of the tables ids in pre-production and production following this spreadsheet:
https://docs.google.com/spreadsheets/d/1X0g-cqjfGuFs0p2JRaR8x1PwowYV6zFIO0LFFXL4P1U/edit#gid=0
Demo
If applicable: screenshots, gifs, etc.
Notes
If applicable: ancilary topics, caveats, alternative strategies that didn't work out, etc.
Testing

Include any steps to verify the features this PR introduces.
If this fixes a bug, include bug reproduction steps.

which tables are affected and how can I test this PR?


clerk not compatible nextjs 15

Preliminary Checks

[X] I have reviewed the documentation: https://clerk.com/docs

[X] I have searched for existing issues: https://github.com/clerk/javascript/issues

[X] I have not already reached out to Clerk support via email or Discord (if you have, no need to open an issue here)

[X] This issue is not a question, general help request, or anything other than a bug report directly related to Clerk. Please ask questions in our Discord community: https://clerk.com/discord.

Reproduction
xxxxxx
Publishable key
pk_test_bmF0dXJhbC1zYXdmaXNoLTc1LmNsZXJrLmFjY291bnRzLmRldiQ
Description
I am creating a new project and I have the latest version of clerk, but it is not compatible with nextjs 15.
@clerk/nextjs": "^5.7.5",

Environment
xxx

To add to this, the stack trace shows that this is very like caused by the implementation of auth(), in my case used within a SignedOut component:
at new Request (node:internal/deps/undici/undici:9267:34)
at new NextRequest (.next/server/chunks/ssr/5a2f6_next_2cb2f2._.js:3350:14)
at buildRequestLike (.next/server/chunks/ssr/_2a1227._.js:48:16)
at auth (.next/server/chunks/ssr/_2a1227._.js:853:443)
at SignedOut (.next/server/chunks/ssr/_2a1227._.js:1043:433)

@drosi94 We as consumers of the Clerk SDK cannot fix it with the workaround, it's a problem the Clerk team has to fix since we receive the error to their underlying sync access.

Yes you are right they are working on it https://github.com/clerk/javascript/pull/4366

Clerk provider not awaiting headers throws error per @clerk/nextjs": "^5.7.5" and "next": "^15.0.1-canary.1" next --turbo.

@eifo As drosi pointed out, the fix is part of https://github.com/clerk/javascript/pull/4366 and will most likely be shipped in a major update anytime soon.

@eifo As drosi pointed out, the fix is part of #4366 and will most likely be shipped in a major update anytime soon.

This got worst with latest next canary, it's critical at this point.

Given that Next 15 just released with massive breaking changes it's not surprise. If you have a critical application you shouldn't migrate the same day and wait for a few days until the ecosystem catches up.

@LekoArts I just tried out the latest canary and at first glance it looks to be working well with all the relevant changes, but I had to apply the changes manually. I couldn't get your @clerk/upgrade codemods to work, but chances are it's a just an issue of me trying to use it.

@LekoArts I just tried out the latest canary and at first glance it looks to be working well with all the relevant changes, but I had to apply the changes manually. I couldn't get your @clerk/upgrade codemods to work, but chances are it's a just an issue of me trying to use it.

same, I guess it's for now...?

Latest canary breaks the middleware, specifically auth().protect() in clerkMiddleware
middleware.js (17:12) @ TURBOPACK__default__export.debug
TypeError: auth(...).protect is not a function

@ChristianIvicevic @cbmastery Could you share any more details on the codemod failures you encountered?

Latest canary breaks the middleware, specifically auth().protect() in clerkMiddleware
middleware.js (17:12) @ TURBOPACK__default__export.debug TypeError: auth(...).protect is not a function

If you read the changes thoroughly you'll notice that it's await auth.protect() now and not auth().protect().
@jacekradko Yesterday I attempted to run pnpm dlx<EMAIL_ADDRESS>and the CLI recognized I was using the Next.js SDK but then did not apply any changes at all without any additional output. Is this the intended way to invoke the CLI in the first place? Just to make sure it's not an error on my end.

Latest canary breaks the middleware, specifically auth().protect() in clerkMiddleware
middleware.js (17:12) @ TURBOPACK__default__export.debug TypeError: auth(...).protect is not a function

if you're using "@clerk/nextjs": "^6.0.0", just change auth().protect() to this.
 export default clerkMiddleware(async (auth, request) => { 	const { userId, redirectToSignIn } = await auth(); 	if (!userId && isPublicRoute(request)) { 		// Add custom logic to run before redirecting 		return redirectToSignIn(); 	} });

@ChristianIvicevic I am running exactly what is in the documentation on clerk website and have tried applying the mention async solution by @fprado-dev as well but once I import isPublicRoute it throws an error. Apologies, am noob. :)

@ChristianIvicevic I am running exactly what is in the documentation on clerk website for next.js and have tried applying the mention async solution by @fprado-dev as well but once I import isPublicRoute it throws an error. I also have a an app/auth/sign-in[[...sign-in]] and a sign-up[[...sign-up]] folders that are .tsx files, would those need to be async?Apologies, am noob. :)

import { clerkMiddleware, createRouteMatcher } from "@clerk/nextjs/server";
const isPublicRoute = createRouteMatcher(["/auth/sign-in(.)", "/auth/sign-up(.)"]);
your file must be page.tsx.
if doesn’t work try to upgrade your next to 15.0.1 and @clerk/nextjs": "^6.0.0"


When do Workbox / create-react-app PWAs update to new static assets?

I originally asked this as a question on Stack Overflow.  It gained zero traction there alas; however, I suspect the topic is important in terms of getting people to get comfortable to moving to using PWAs. So I thought I'd ask here too!
I have a PWA which was made with create-react-app.  The codebase of the PWA is regularly updated and we ship new versions of the app daily. Under the bonnet this is powered by the webpack workbox plugin.
I've noticed different behaviour with regards updates of our PWA in two different use cases.  I wanted to talk through what I've noticed and I'm hoping someone with workbox expertise can explain the why 😊
I should say that we optimise for clients updating as soon as possible with this:
          if ('serviceWorker' in navigator) {
            navigator.serviceWorker.addEventListener('controllerchange', function () {
              if (confirm('New version of app available; update now?')) window.location.reload();
            });
          }

Desktop Chrome Usage
Every time we release, anyone accessing the app subsequently sees the "New version of app available" prompt and gets to upgrade. This version is typically used by people who work access the app via the intranet of our company.
Android Added to Home Screen PWAs
The other use case is the same app installed on Android phones.  This version is accessed on the internet behind Cloudflare.
We've never yet seen the "New version of app available" in this context.  Rather, at some point the app updates and we see new content.
The Question
So the question is: what triggers the update of the PWA in the Added to Home Screen context?  I'm not certain but I've some circumstantial evidence to offer:
Updates in this context seem to happen within 7 days.  Interestingly, static assets (JS / CSS / HTML) are cached for 7 days in our app.
I'm wondering if maybe Cloudflare is a reason here?  Or the static asset cache policy?  Or both? Or something else?
If someone could shed some light I'd love to learn!

Hello @johnnyreilly—I've answered on https://stackoverflow.com/a/60939397/385997. We can follow up there as needed.


RTCMulticonnection V3 screensharing not working in firefox

this one not working in firefox - https://rtcmulticonnection.herokuapp.com/demos/Scalable-Screen-Broadcast.html
Can you please look on it

Please download this:

https://github.com/muaz-khan/Firefox-Extensions/tree/master/enable-screen-capturing

Modify index.js to include rtcmulticonnection.herokuapp.com here:

https://github.com/muaz-khan/Firefox-Extensions/blob/master/enable-screen-capturing/index.js#L11

Make ZIP and rename .zip to .xpi and install by dragging/dropping over Firefox address-bar.
Otherwise follow these steps:

https://github.com/muaz-khan/Firefox-Extensions/tree/master/enable-screen-capturing#how-to-deploy

Otherwise
Open about:config and search for media.getusermedia.screensharing.allowed_domains.
Now manually append at the end of the list: rtcmulticonnection.herokuapp.com

Open about:config and search for media.getusermedia.screensharing.allowed_domains.
Now manually append at the end of the list: rtcmulticonnection.herokuapp.com
This one made it working. Thanks !
So one doubt,  RTC multiconnection  V2 worked fine in firefox,  but  in V3  i have to do it like this to make it working, why is it so?
Why RTC multiconnection V3 is using a node server is it possible to do it without setting up node server just like V2 using firebase.js. which version il be better to use

All demos from v2 was deployed here:

https://www.webrtc-experiment.com/RTCMultiConnection/

However v3 demos are using a separate server:

https://rtcmulticonnection.herokuapp.com/

v3 is using socket.io to get maximum accuracy in the handshake and presence detection.
v3 supports Firebase+PubNub+WebSocket+many others as well. You can check demos in the "demos" directory.

Thanks you muaz for the quick reply.


multi-gpu hyperparameter issues

Hi, have u  use multi-gpu for training? I don't know what's the proper parameters to config.

如果你是单机多卡,你可以去看我的SSD.我打半天英语,一看是中国人...


Unable to generate erd

Hi, I'm trying to generate erd using prisma but getting the following error,
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Error:
✔ Generated Prisma Client (4.11.0 | library) to ./node_modules/.pnpm/@prisma+client@4.11.0_prisma@4.11.0/node_modules/@prisma/client in 673ms

could not parse datamodel

The prisma client was generated which suggested that the schema is valid. I've also tested the validity using npx prisma validate .
Following are the dependencies.
OS: Debian GNU/Linux 11 (bullseye)
node: v19.8.1

# packages
 "@prisma/client": "^4.9.0",
 "prisma": "^4.9.0",
 "prisma-erd-generator": "^1.3.1",
 "@mermaid-js/mermaid-cli": "^10.0.2",

The prisma schema is as follows:

// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

generator client {
    provider = "prisma-client-js"
}

generator erd {
    provider                  = "prisma-erd-generator"
    output                    = "./ERD.svg"
    includeRelationFromFields = true
    erdDebug                  = true
}

datasource db {
    provider = "mysql"
    url      = env("DATABASE_URL")
}

model User {
  id            String         @id
  email         String
  favoriteProducts  FavoriteProducts[]
}

model Product {
  id              String        @id
  title           String
  inFavorites  FavoriteProducts[]
}

model FavoriteProducts {
  userId      String
  user        User    @relation(fields: [userId], references: [id])
  productId   String
  product     Product @relation(fields: [productId], references: [id])

  @@id([userId, productId])
}

I'd really appreciate if someone can help me out :slightly_smiling_face: .

I was able to solve the issue by changing the directory that does not contain any spaces in its name. After seeing this issue I suspected that this may be the case. Hope this is fixed in future.


test/v3nametest.c: Add check for OPENSSL_malloc

As the potential failure of the OPENSSL_malloc(),
it should be better to add the check and return
error if fails.
Signed-off-by: Jiasheng Jiang<EMAIL_ADDRESS>
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged, thanks for the fix.