Deploy error

Error: supabaseUrl is required.
--
16:49:50.780 | at new ag (/vercel/path0/.next/server/chunks/856.js:1:109603)
16:49:50.780 | at aw (/vercel/path0/.next/server/chunks/856.js:1:112826)
16:49:50.781 | at 37446 (/vercel/path0/.next/server/app/api/webhooks/route.js:1:72564)
16:49:50.781 | at t (/vercel/path0/.next/server/webpack-runtime.js:1:127)
16:49:50.781 | at r (/vercel/path0/.next/server/app/api/webhooks/route.js:1:112889)
16:49:50.781 | at /vercel/path0/.next/server/app/api/webhooks/route.js:1:112916
16:49:50.782 | at t.X (/vercel/path0/.next/server/webpack-runtime.js:1:1181)
16:49:50.784 | at /vercel/path0/.next/server/app/api/webhooks/route.js:1:112902
16:49:50.784 | at Object.<anonymous> (/vercel/path0/.next/server/app/api/webhooks/route.js:1:112944)
16:49:50.784 | at Module._compile (node:internal/modules/cjs/loader:1358:14)
16:49:50.789 |
16:49:50.789 | > Build error occurred
16:49:50.791 | Error: Failed to collect page data for /api/webhooks
16:49:50.791 | at /vercel/path0/node_modules/.pnpm/next@14.1.0_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/build/utils.js:1258:15
16:49:50.791 | at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
16:49:50.791 | type: 'Error'
16:49:50.791 | }
16:49:50.807 | ELIFECYCLE  Command failed with exit code 1.
16:49:50.826 | Error: Command "pnpm run build" exited with 1

I have used the deploy button from github, followed all steps, as soon as i change anything in the github repo it fails, didnt touch the api/webhooks folder, just some footer changes and removing readme, and public files related to it. It worked before i changed anything.

anyone else with this error, supabase key, url are production only whereas the deployment was in preview therefore failure

I am facing the same issue. I will work on this again tomorrow morning and will get back to you with a fix if I find it.

anyone else with this error, supabase key, url are production only whereas the deployment was in preview therefore failure

The solution seems to be to create separate Supabase projects for staging and production environments, and use the relevant keys and url's from the staging environment.

Ok but how can I sync that? I was hoping to not create a testing Supabase project since I can use local to develop. If I have to support preview on Vercel, is there a way to sync 2 project?

I have created 2 environments ( DEV and PROD ) . I have 2 Supabase projects DEV & PROD and working on 2 github branches main ( PROD ) and DEV ( dev ).
I still encounter the same error:
Error: supabaseUrl is required.
    at new ay (/vercel/path0/.next/server/chunks/286.js:1:111494)
    at ab (/vercel/path0/.next/server/chunks/286.js:1:114738)
    at 91819 (/vercel/path0/.next/server/app/api/webhooks/route.js:1:77116)
    at t (/vercel/path0/.next/server/webpack-runtime.js:1:127)
    at r (/vercel/path0/.next/server/app/api/webhooks/route.js:1:117797)
    at /vercel/path0/.next/server/app/api/webhooks/route.js:1:117828
    at t.X (/vercel/path0/.next/server/webpack-runtime.js:1:1191)
    at /vercel/path0/.next/server/app/api/webhooks/route.js:1:117810
    at Object.<anonymous> (/vercel/path0/.next/server/app/api/webhooks/route.js:1:117856)
    at Module._compile (node:internal/modules/cjs/loader:1469:14)
> Build error occurred
Error: Failed to collect page data for /api/webhooks
    at /vercel/path0/node_modules/.pnpm/next@14.2.3_react-dom@18.3.1_react@18.3.1/node_modules/next/dist/build/utils.js:1268:15
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  type: 'Error'
}
 ELIFECYCLE  Command failed with exit code 1.
Error: Command "pnpm run build" exited with 1


Feature Request: log versions that got deleted

right now, it only logs the package id.

Endorse this. Would appreciate some more verbose logging.

We faced an issue today due to misuse of this action - having more verbose logs would definitely help troubleshooting.


[2.0.6] www. is stripped from domains

I'm using postcss-loader with webpack3 and sass-loader and adding this block causes the absolute address used for font loading to be stripped of www. prefix:
{
    loader: "postcss-loader",
    options: {
        plugins: function () {
            return [
                require('autoprefixer')('last 2 versions', 'ie 11'),
                require('cssnano')(_{normalizeUrl: {stripWWW: false}}_)
            ]
        },
        sourceMap: false
    }
},

I tried adding the option to cssnano hoping this will help, but even removing cssnano from plugins and leaving only autoprefixer causes the same effect. This option for cssnano also doesn't help... Any idea what I'm doing wrong?

Normally postcss-loader doesn't mess with paths by default, it is just a wrapper for  postcss and the rest is done via plugins. Could you show the 'full' relevant part rule > test.scss of webpack.config.js and maybe setup a small test repo? ⚠ Note that css-loader uses cssnano aswell internally

Of course! Here's the whole scss block. The address remains ok (with www.)  if I remove either  loader: "postcss-loader" block or both plugins from  loader: "postcss-loader" > plugins array...
{
    test: /\.scss$/,
    loader: ExtractTextPlugin.extract({
        fallback: 'style-loader',
        use: [
            {
                loader: "css-loader",
                options: {
                    sourceMap: false,
                    modules: false,
                    importLoaders: true
                }
            },
            {
                loader: "postcss-loader",
                options: {
                    plugins: function () {
                        return [
                            require('autoprefixer')('last 2 versions', 'ie 11'),
                            require('cssnano')({normalizeUrl: {stripWWW: false}})
                        ]
                    },
                    sourceMap: false
                }
            },
            {
                loader: "sass-loader",
                options: {
                    sourceMap: false,
                    includePaths: ["node_modules/foundation-sites/scss", "node_modules/motion-ui"]
                }
            }
        ]
    })
},

Sorry to bother you. I lack the knowledge to dig deeper and find the reason, but here's some more information, maybe you will be able to find the culprit...
I found a config that works for me, but it seems to be patching something that shouldn't need patching...

code being changed - www. is being stripped from font-face declaration:

@font-face{font-family:bb-icons;src:url(https://www.babyboom.pl/fonts/bb-icons/bb-icons.svg#bb-icons) format("svg"),url(https://www.babyboom.pl/fonts/bb-icons/bb-icons.ttf) format("truetype"),url(https://www.babyboom.pl/fonts/bb-icons/bb-icons.eot) format("eot"),url(https://www.babyboom.pl/fonts/bb-icons/bb-icons.woff2) format("woff2"),url(https://www.babyboom.pl/fonts/bb-icons/bb-icons.woff) format("woff")}

I have stripped options and require from above scss test, so I have only:
{
      loader: "postcss-loader"
},

and my postcss.config.js is:
module.exports = {
  plugins: {
    'autoprefixer': { browsers: ['last 2 versions', 'ie 11'] },
    'cssnano': {normalizeUrl: {stripWWW: false}},
    'postcss-urlrewrite': {
      imports: true,
      properties: true,
      rules: [
          { from: '//babyboom.pl', to: '//www.babyboom.pl' }
      ]
    }
  },
}

this url rewrite is necessary but not enough, I also need to disable:
//new webpack.LoaderOptionsPlugin({
   //    minimize: true
//}),

and this configuration works fine.
Just to make sure - even running "empty" postcss with no other plugins removes the www. Therefore I have to have the 'postcss-urlrewrite' plugin enabled. But then having LoaderOptionsPlugin minimize enabled, removes the www (again?)... This option doesn't seem to make any difference at the moment, so I can live with that. I just hate not knowing what's really going on...

I take a look into it soon :)

It's recommend to use the cssnano implementation in css-loader via options.minimize for the time being, since css-loader rewrites url() by default to resolve assets 'the webpack way'. Just remove cssnano from postcss.config.js. The @font-face issue is likely to be caused by css-loader aswell, bc of how it handles assets resolving url(). You could set options.url = false to disable this behaviour in css-laoder completely, but this affects all url()'s then 🙃. I recommend to include fonts via <link href="path/to/fonts.css"> in index.html instead (e.g via/like Google Fonts). Also postcss-urlrewrite seems to be unnecessary  and can be removed aswell, I personally never had the need for plugins like that, when using webpack


[UI] Cluster Templates - Hide "cattle-global-data:" in CT dropdown

Version: master-head (7/31/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create one or more cluster templates
Create a cluster, choose to do so via cluster templates, look at the drop-down

Result:
The drop down looks not very nice because of the mention of "cattle-global-data:" we should ideally just see the plain name of the revisions supplied for each cluster template. This will make it look better.

@davidnuzik This piggy-backs on a request vince had to break these out into two separate selects.  One select for cluster template and another for revision that is disabled until template is selected. I'll clean up the display as requested as well.

Version: master-head (8/2/19) bec76ef3
This changes the design of how the user must pick their CT/CTR in the UI. The cattle-global-data is now also not shown. This is much cleaner:

I tested and ensured I can create Clusters with the tempaltes without any issues. I created a CT with several revisions and ensured that the UI updates accordingly when change revisions and CT's. The HTTP/UI/API all look good.


Cannot use svgs with "","" elements.

Hi, thank you for the library, I'm trying to use this for svg coloring, but svgs with <rect>, <g> elements are not working. Any guidance for this is appreciated. Thank you!

Can you please provide a minimal working example of this issue? <rect> and <g> tags are supported; something else must be going wrong.

It would be helpful to add a SVG that illustrates the problem. Thanks!

Hi, if @AmilaWijayarathna can not provide the example, I can give it to you:

3.Icons/Outlined/sticker
Created with Sketch.

It will present as a black picture with Sharp, but show a "thumb" picture in browser (like Chrome)
@pflammertsma Looking forward to your reply!

This is still an issue


Card #538 Not Classified Under バレンタイン編 Collection

The Rin Pure UR card should be included in the バレンタイン編
Attached Image of the Rin card not showing

Thanks for reporting!
However, we do not insert any information manually. All the information come from various wiki.
The information about the collection comes from this page: http://www59.atwiki.jp/lovelive-sif/pages/34.html
If you're willing to help about this (thanks!), can you please try to see if you can update this wiki page?
Unfortunately, we do not have time to update all the wikis ourselves :/
Thanks again.


Extension unavailable in VSCodium

Seems like your wonderful extension is not available in VSCodium extension store.
See https://github.com/VSCodium/vscodium/blob/master/DOCS.md#extensions-marketplace for more details

Done - https://open-vsx.org/extension/ciarant/vscode-structurizr
Just waiting on my namespace to get verified.
First time hearing about VSCodium, btw - thank you!


Diff/Patch: Split view - enable line hover

Makes it a bit easier to track line changes.

Wow, that's very nice indeed. Thank you very. I updated the CSS and enabled it for unified view also. Though did go for a bit lighter grey.


Rerender and rebuild.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hm.It still seems to have left the building disabled.
I will merge this in to check and then possibly message someone for help.
Thanks for all of your help! :bowing_woman:

It was an intermittent smithy issue, I have the correct revision locally now.

:+1:

Something is really strange, now it's not working again :-/

I'll keep trying and will send a new PR when things start working again. Thanks for your time!

Thank you too! :bowing_woman:
This is good impetus to set up a conda-smithy environment. ^^;

Working now, #5 ! Indeed, I find the local rerendering more convenient. First time I've seen issues like this though.


Shorten build configuration name

By removing "Test Coverage".
So we can read the title more easily.
Before:

After:

@bot-gradle test and merge

@bot-gradle test and merge

OK, I've already triggered a build for you.


api-v2v3search-0 paging limit

Hi,
This is not really an issue but more a help request.
When using the api-v2v3search-0 endpoint, only 20 results are returned.
Is there any way to get more results? result data does not seem to contain information to retrieve additional packages even if it tells how many packages match the query.
Thanks

You can add the "skip" URL parameter, e.g. "skip=20" to ge the next page.

Thank you very much!!


does replication support mysql compress protocol?

I want to know whether the replication support the mysql compression protocol?
If yes, how to enable the compress protocol in BinlogSyncerConfig? thanks!

This library can receive compressed binlog event but won't decode it. Any program using this library should implement the decoding itself.
For example, https://github.com/pingcap/dm/issues/2192

Note that there is:

Compression in the MySQL protocol, which is used to transfer binlog events. The protocol has zlib and zstd compression. For regular MySQL to MySQL replication this is configured with SOURCE_COMPRESSION_ALGORITHMS as part of the CHANGE REPLICATION SOURCE TO... statement

Compressed binlog events as enabled by binlog_transaction_compression.

https://dev.mysql.com/doc/refman/8.0/en/change-replication-source-to.html

https://dev.mysql.com/doc/refman/8.0/en/replication-options-binary-log.html#sysvar_binlog_transaction_compression

https://dev.mysql.com/doc/dev/mysql-server/latest/page_protocol_basic_compression.html

https://dev.mysql.com/doc/dev/mysql-server/latest/classbinary__log_1_1Transaction__payload__event.html

There is already a ticket for the compressed event format: #533


fix: Error logs are events

Codecov Report

Merging #93 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #93     +/-   ##
=========================================
- Coverage   68.39%   68.29%   -0.1%
=========================================
  Files          24       24
  Lines        2063     2063
  Branches      280      280
=========================================
- Hits         1411     1409      -2
- Misses        543      544      +1
- Partials      109      110      +1

Impacted Files
Coverage Δ

sentry_sdk/integrations/logging.py
74.28% <100%> (-2.86%)
:arrow_down:

Continue to review full report at Codecov.

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

Blocked on https://github.com/huge-success/sanic/issues/1332


jnp.indices returns numpy.ndarray read-only arrays

Hey Jax team!
I have found that jax.numpy.indices() actually returns an un-writeable numpy array, rather than a DeviceArray.
I found a work around for the problem, but I doubt this is the intended behavior.
This was found using:
jax 0.1.77
jaxlib 0.1.57
which I know are not the current stable release, but I am working with a package that has not been patched to work with 0.2+.
import jax.numpy as jnp
import numpy as onp

# Numpy returns a writeable numpy.ndarray
a, b = onp.indices((5,5))
print(a.flags["WRITEABLE"])
# <class 'numpy.ndarray'>, True

a += 5
# Does not cause error and works fine

# Jax.numpy returns a read-only numpy.ndarray (NOT DeviceArray)
a, b = jnp.indices((5,5))
print(type(a), a.flags["WRITEABLE"])
# <class 'numpy.ndarray'>, False

a += 5
# Throws "ValueError: output array is read-only"

# Re-casting array to DeviceArray fixes problem
a = jnp.array(a)
print(type(a))
# <class 'jaxlib.xla_extension.DeviceArray'>
a += 5
# Now the array is writeable without error

Hi - thanks for the question! It turns out the culprit is not jnp.indices, rather this is a long-standing bug/wart in JAX (see #1583). jnp.indices returns a device array, but when you write a, b = jnp.indices(...) it calls the __iter__ method, which outputs static numpy arrays.
There's a fix in the works at #3821 (not sure why it stalled, maybe @mattjj knows?).
Until that is resolved, a workaround is to do something like this:
ind = jnp.indices((5,5))
a = ind[0]
b = ind[1]

I'm going to close this - we can continue to track the issue in #1583


doesn't appear

i have a github site, when i tried to embed nothing popped up. i inspected everything and the embed was there, just not functional.

I visited your site and the embed appears to be working – it’s configured by default to use a cookie so it only shows up once per user. Perhaps try clearing your cookies and check again.
Let me know if there’s still an issue. Thanks for your support on this campaign!
From: LinuxRocks2000
Sent: Sunday, November 19, 2017 9:21 AM
To: fightforthefuture/battleforthenet-widget
Cc: Subscribed
Subject: [fightforthefuture/battleforthenet-widget] doesn't appear (#131)
i have a github site, when i tried to embed nothing popped up. i inspected everything and the embed was there, just not functional.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Populate safeTxGas using the provided gas limit (SafeAppProvider)

Currently, it is not possible to populate the safeTxGas when using a safe app that communicates through the SafeAppProvider (e.g. WalletConnect).
This PR updates the Safe App Provider to forward the provided gas limit to safeTxGas. This is helpful in many cases where a gas buffer is required to avoid "Out of gas" errors.

I have read the CLA Document and I hereby sign the CLA

Could you please add a changeset?

@mikhailxyz Added 👍

@solimander we've discussed this internally and we're ready to accept this pull request if this behaviour gets properly documented. Could you please add something along these lines into safe-apps-provider readme?

Note on gas limit
The ethereum transaction gas limit passed to the safe-apps-provider will be treated as safeTxGas, which is an equivalent of the ethereum transaction gas limit in the Safe context. To learn more about the safe transaction gas, read here: https://github.com/safe-global/safe-contracts/blob/c36bcab46578a442862d043e12a83fec41143dec/docs/safe_tx_gas.md


internal server error when trying to submit rating

Happened after two ratings were submitted successfully. Unfortunately no log availiable.
Can't reproduce atm.
{
  "code": "internalServerError",
  "detail": "edu.kit.ipd.crowdcontrol.workerservice.InternalServerErrorException: Cause: Object service responded with: SQL [select crowdcontrol.Answer.id_answer, crowdcontrol.Answer.experiment, crowdcontrol.Answer.answer, crowdcontrol.Answer.timestamp, crowdcontrol.Answer.worker_id, crowdcontrol.Answer.reservation, crowdcontrol.Answer.quality, crowdcontrol.Answer.quality_assured, crowdcontrol.Answer.system_response, crowdcontrol.Answer.hash from crowdcontrol.Answer where crowdcontrol.Answer.id_answer \u003d (select crowdcontrol.Rating_Reservation.answer from dual where crowdcontrol.Rating_Reservation.idReserverd_Rating \u003d ?)]; Unknown column \u0027crowdcontrol.Rating_Reservation.answer\u0027 in \u0027field list\u0027"
}

Closing, since it's merged.


Having trouble running your sample even though the dll exists and using

Having trouble running your sample even though the dll exists and using latest gdal from
http://www.gisinternals.com/release.php. See below
Any ideas
"Can't load requested DLL: C:\Program Files\GDAL\gdalplugins\gdal_netCDF.dll\n127: The specified procedure could not be found.\r\n"} | System.ApplicationException

Hi, can you try installing the GDAL tools from https://trac.osgeo.org/osgeo4w/ ?

I did an install on a clean Windows installation, got it working by:

Install older (2.x) version of QGIS (http://download.osgeo.org/qgis/windows/QGIS-OSGeo4W-2.18.28-3-Setup-x86_64.exe). The more current 3.10 installation seems missing gcs.csv file.
Set GDAL_DATA enviroment variable to directory where gcs.csv is installed (default C:\Program Files\QGIS 2.18\share\gdal)

There might be a way to get it working by installing only GDAL not QGIS complete.

Closing


Retrieves all GPOs in a domain

Retrieves all GPOs in a domain by using LDAP request to the "groupPolicyContainer" and can display detailed information and all attributes about a specific GPO :
poetry run cme ldap <DC_IP> -d  -u  -p  -M get-gpo
poetry run cme ldap <DC_IP> -d  -u  -p  -M get-gpo -o gpo=<GPO_NAME>

Hello @Bebzounette can you make a PR on the new official repo https://github.com/mpgn/CrackMapExec ?


CLI init adds "testing" dependency instead of "testing-junit4"

When running fn init --runtime java javafn, the <artifactId>testing</artifactId>  is added by default instead of 'testing-junit4'.
The testing dependency doesn't contain the FnTestingRule class, which provokes a compile error.

Please try latest CLI

This was fixed with https://github.com/fnproject/cli/commit/60128b3211fa5305d206783e4d4a252b1cc9b2f2


update to readme

1st Version
I tried to fit to the K8s readme, and removed some categories like Plan and current status, Configuration and Installation details.
I think it's less consistent but maybe more efficient.
Tell me what can I add to it to be better.

If you want to get rid of the fixup commits by pre-commit, take a look at our contribution guide with the instruction how to use it locally. It brings various checks but also tools to reformat our code to look nicer.


BaseSettings - pylance complains about missing parameters

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.0
            pydantic compiled: True
                 install path: /home/user/repo/venv/lib/python3.10/site-packages/pydantic
               python version: 3.10.0 (default, Oct 11 2021, 05:33:59) [GCC 11.2.0]
                     platform: Linux-5.13.0-28-generic-x86_64-with-glibc2.34
     optional deps. installed: ['typing-extensions']

When using base settings in (even without Field see #3617 #3753)
from pydantic import BaseSettings

class MySettings(BaseSettings):
    option_1: str
    option_2: int

MySettings()

the resulting class expects the parameters to be provided by the program and not by the environment. Therefore Pylance seems to complain.

Btw: I use the basic setting:
"python.analysis.typeCheckingMode": "basic",
"python.languageServer": "Pylance",

The addition of parameters in BaseModel is welcome change, however I don't think the BaseSettings class should be affected as well.

closing as a duplicate of #3767 which has more details.


skeleton codes for arm system and tele op

Before issuing a pull request, please see the contributing page.

gottem


Calculate ortholog relationships in Reactome

This PR adds a script for identifying orthologous pathways in Reactome based on lexical matching. All pathway identifiers have the form R-{SPECIES CODE}-{PATHWAY CODE}, so any two species' pathways can be matched by splitting on the dash - then maching pathways codes.

@bgyori this has been cleaned up and now has a more specific scope. What do you think about adding these ~100K orthology mappings between Reactome pathways? In comparison to the 5K we've done manually, this vastly changes the scale of the data here. Do you think this is a problem? Maybe we should have yet another category for large scale calculated mappings that can automatically be trusted (as opposed to predicted, which still need to be checked) so we can keep these separate


Implement methods in org.jooq.lambda.Seq

Seq extends Stream and adds some more useful methods.
You may be interested in adding those.
See here: https://github.com/jOOQ/jOOL

Thanks for your input. I'll look into it. Do you have any specific requirement?

Nope. I'm currently not doing any web development. This is just a suggestion :)

A few methods are now available in Stream.js, like slice as well as takeWhile and dropWhile which seem to equivalent to limitWhile and skipWhile in jOOL.


HARK AgentType wrapper around Dolo model

The inverse of trying to export HARK to Dolang (see #659)
This issue is for an AgentType class that:

starts with a fully formed Dolo model, like the kind you get from a parsed Dolang YAML file
builds all the getStates, getShocks, etc. methods needed for simulation out of the materials in the Dolo model
Ideally, also setting things up so that a solution can be passed in...
and can be ran as a simulation.

What exists:
Dolang (yaml) -> Dolo (py-obj)
What this issue is about:
Dolo (py-obj) -> HARK (py-ob)

Others things that we might do, not in this issue:
HARK -> Dolo
Dolo (py-obj) -> Dolang (yaml)
HARK -> Dolo (py-obj) -> Dolang (yaml) -- might be the best implementation for #446


Housekeeping for data and endpoint exports

I was doing some diffs of the modules in github.cabal against those exported and found some things to fix:

Add missing data and endpoint exports
Move statuses exports under repositories
Remove duplicated enterprise exports (This one's my bad, I exported these from the GitHub.Enterprise module so we don't want to export them from the GitHub module.)

Thanks!


Link status panels to appropriate filters

Description of change
Link the status panels to the appropriate filter, or to the unfiltered view if the filter is already enabled.
Story Link
https://trello.com/c/3J0a9MnX/1772-smart-filtering-filter-by-clicking-on-status-bar
Screenshots
If you have made changes to the frontend please add screenshots
Decisions [OPTIONAL]
If you had to make any decisions between different ways of doing things, what where they and why?
Known issues [OPTIONAL]
Things you know need further follow on work but aren't in scope of this issue

issue1
issue2

Further testing or sign off required [OPTIONAL]
e.g.

product manager to sign off view templates

I was going to say that because all this does is use the normal search filters it's probably fine to just rely on the existing search tests, but it seems like the tests for which applications are being shown when searching/filtering are some of the flaky ones. I've checked that the panel is highlighted when selecting a relevant filter, and I think we can rely on the tests for the SearchFilterService if not the Capybara search tests.


Add Fit_transform to pipeline_base and component_graph

Pull Request Description
Fixes #2665

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

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

Note: Found out the LabelEncoder class overrides its fit_transform method with its own self.fit().transform(). Thus, when its mock is called, it actually triggers 2 calls to its transform() unlike the OHE and Imputer


[WIP] Run iOS UIKit tests on simulator

Run iOS UIKit tests on simulator
Implement basic keyboard appearing tests
Improvements needed:

It's impossible to get a screenshot while running tests.
Keyboard should be operated manually  when running tests.
There is no proper implementation of waitForIdle function. waitToEquals has been implemented and used instead.

Release Notes
Run iOS UIKit tests on simulator

cc @m-sasha as ui-test owner where you're inctroducing new public APIs

@igordmn , to be clear: we're implementing integration tests for iOS platform, which are not UI tests in their classic definition.

Without going too deep into the implementation itself, a few concerns:

Should this be in the ui-test module? Perhaps it's better to introduce a new module.
A test API with a "live" app is useful on all target platforms, not just iOS. If we want to release it publicly (even if experimental) perhaps we should design/discuss it upfront, so that we could have a common (or at least cohesive) API across platforms.

Without going too deep into the implementation itself, a few concerns:

Should this be in the ui-test module? Perhaps it's better to introduce a new module.
A test API with a "live" app is useful on all target platforms, not just iOS. If we want to release it publicly (even if experimental) perhaps we should design/discuss it upfront, so that we could have a common (or at least cohesive) API across platforms.

Test was extracted into another source-set uikitInstrumentedTest (analog to androidInstrumentedTest), to run integration tests having properly configured iOS environment.


Documentation for debugging modules

📝 How to debug (C#) modules such that symbols are loaded correctly
Folks who are familiar with C# WPF applications may want to contribute to a module like Fancy Zones, but out of the box it's not clear how to run the module with symbols loaded for debugging. If you try to run Fancy Zones by itself you're shown a message that it should not be run as a standalone application.
Though with some playing around you might find you can start up the runner project and open the module through settings. However at this point, by default, no symbols are loaded for breakpoints within Fancy Zones.
It would be a great to have some documentation around the right way to debug this (and other) modules.
Note: I am currently in the boat of having not figured out how to do this yet, so I'll take a quick version of how to accomplish this if one exists!

we're in the process of making each item independent of the runner.  This will allow for easier dev and debugging.  FZ has to bits, the editor (wpf) and the actual FZ snapping engine which is C++.
Are you trying to debug something from our EXE or via the VS and compiling locally?

@dedavis6797 fyi for improving docs

we're in the process of making each item independent of the runner. This will allow for easier dev and debugging. FZ has to bits, the editor (wpf) and the actual FZ snapping engine which is C++.
Are you trying to debug something from our EXE or via the VS and compiling locally?

Sounds like a great direction, thanks for the insight.
I'm trying to run & debug from VS (compiling locally).

What's the status for this issue?

I'm in the situation described. Anyone figure it out?


Different behaviour between dev and production builds

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
I'm experiencing different behaviour between dev and production builds. It seems to be related to library I added to the project. The following steps should reproduce the issue:
Expected behavior
I can see that AdminLTE is added to shim.js in production build. Also, if I run $.AdminLTE from console I can see the object is defined. My guess is that the order of scripts are different between dev and production. Ideally these should be the same.
To reproduce the error you can follow these steps

cloned the project on 2017-01-30 commit 5fb5a8b
npm install --save jquery bootstrap admin-lte
update project.config.ts

      this.NPM_DEPENDENCIES = [
        ...this.NPM_DEPENDENCIES,
        {src: 'jquery/dist/jquery.min.js', inject: 'libs'},
        {src: `bootstrap/dist/js/bootstrap.min.js`, inject: 'libs'},
        {src: `admin-lte/dist/js/app.min.js`, inject: 'libs'},
        {src: `bootstrap/dist/css/bootstrap.min.css`, inject: true},
    ];

update app.component.ts

declare var $: any;
@Component({
  moduleId: module.id,
  selector: 'sd-app',
  templateUrl: 'app.component.html',
})
export class AppComponent implements OnInit {
  constructor() {
    console.log('Environment config', Config);
  }
  ngOnInit(): void {
    $.AdminLTE.layout.fix();
  }
}

run npm start and the console doesn't show any error
run npm run serve.prod and you see that $.AdminLTE during AppComponent OnInit call is undefined. This can be seen from the error in the console and there is an error there.

Thanks for reporting this issue! I'll take a look at it today.

@mgechev Thank you!

It's not a problem in the seed but a race condition.
The $.AdminLTE.layout is initialized on document.ready (i.e. $(function () {...})) which in case of development happens before the application gets loaded (since it's non-optimized and requires a bit more time in order to fetch all the required assets). On the other hand, when the application is bundled, the ngOnInit is invoked before the document.ready event is fired. This causes call of the $.AdminLTE.layout.fix method but layout is undefined.
To fix the problem use:
/**
 * This class represents the main application component.
 */
@Component({
  moduleId: module.id,
  selector: 'sd-app',
  templateUrl: 'app.component.html',
})
export class AppComponent {
  constructor() {
    console.log('Environment config', Config);
  }
  ngOnInit(): void {
    $(function () {
      $.AdminLTE.layout.fix();
    });
  }
}

I'd not recommend you using Angular with jQuery in long term.

Thank you @mgechev. That fixed the issue. Regarding your suggestion about jquery, I fail to see a replacement with its vast ecosystem of code and plugins already available.

Yes, jQuery is convenient for rapid development. Hopefully at some point we'll have enough high-quality Angular components :)

@mgechev Thank you!


Remove empty lines on issues/pulls page

This does not fix the underlying issue of not showing (seemingly) random issues/PRs, but it does fix the positioning of the if-check.
The check starts inside the <li> but ends outside the closing </li>
https://github.com/go-gitea/gitea/blob/51432ebb9c6f59c618ff34147f4b644c8bfb52f5/templates/user/dashboard/issues.tmpl#L89-L90
https://github.com/go-gitea/gitea/blob/51432ebb9c6f59c618ff34147f4b644c8bfb52f5/templates/user/dashboard/issues.tmpl#L155-L156

Codecov Report

Merging #9232 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9232   +/-   ##
=======================================
  Coverage   41.08%   41.08%
=======================================
  Files         555      555
  Lines       72393    72393
=======================================
  Hits        29746    29746
- Misses      38901    38904    +3
+ Partials     3746     3743    -3

Impacted Files
Coverage Δ

modules/task/migrate.go
25% <0%> (-3.95%)
:arrow_down:

models/error.go
32.67% <0%> (-1.19%)
:arrow_down:

models/repo.go
47.87% <0%> (+0.05%)
:arrow_up:

modules/migrations/gitea.go
9.45% <0%> (+0.63%)
:arrow_up:

modules/log/event.go
65.64% <0%> (+1.02%)
:arrow_up:

modules/migrations/migrate.go
25.4% <0%> (+1.62%)
:arrow_up:

Continue to review full report at Codecov.

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

I'm already on this ... !

but thanks @jolheiser for the smal part - i dont have to do this now naymore ...

is there a open issue or should i create one?

There is no open issue. You can open one if you want. 😊


Add information on the current environment (conda) and provide a switcher

Display the current environment in use as a StatusBar Widget. Allow to select the environments found from there.
This work will tie with the projects refactoring
See https://github.com/spyder-ide/spyder/pull/5947 for reference

Unifying over https://github.com/spyder-ide/spyder/issues/9008

I realize this issue is closed, but I thought I'd report one issue and relevant information for a "switcher". For reference, this feature was added in #9778 for 4.0.0b4.

As of 4.0.0b5 the status bar does not update to reflect a change in the conda environment when switching python interpreters in the preferences.
Also, I don't know if it's possible to have the status bar reflect the conda environment of the current console tab? Similar to the git branch status of the active editor tab...
I ran into a corner case involving a package (fenics, a PDE solver) that relies on subprocesses where some paths were not found. I resolved this by using the following script in Preferences->iPython console->Startup

import os
import sys

if 'anaconda' in sys.executable:
    # user intends to use anaconda environment...
    os.environ.update({'PATH': os.path.dirname(sys.executable) + ':' + os.environ['PATH'],
                       'CONDA_PREFIX': sys.exec_prefix})

del os, sys

These were the minimum conda environment variables I needed to change to get everything working properly, but I can't be certain that it perfectly emulates all that's performed by conda activate, so there may be other edge cases out there. I thought this may be useful information for any implementation of a "switcher".

So is there an environment switcher in Spyder or is there not? I can see the current environment, but cannot see any switcher... Manually changing python interpreter makes me nervous, I would much prefer something safer that can read conda environments.

@milindsmart no it is not yet implemented.
@mrclary could you open a new issue with your suggestion (follow and show the current console env)? Also could you open an issue for the environment switcher, I guess the idea would be to not use preferences at all for that, but provide a click on the env and display found envs plus an option to select an env right there?
Thanks!

I created #13318

Actually, I already opened #12200, closing #13318


Update GOV.UK Frontend dependency to v2.1.0

Inline with the latest release of GOV.UK Frontend

You can preview this change here:
Built with commit 46f6f85487dec8b90b26831ee7ff3c1e3729b4fa
https://deploy-preview-565--govuk-design-system-preview.netlify.com


no option setting html5 through prop

Hi,
what would be the best way to set the howl to use html5 mode for the audio?
thanks a lot for the awesome package

Hey @walleXD,
I just added the html5 prop that lets you force HTML5 Audio.
If you update vue-howler to 0.2.0 it should be in there.
Cheers!


Question Regarding the Absence of Controller Settings on Steam Deck

Hi,
I'm currently following the Steam Deck ROS installation guide provided by you. You mentioned to go to "Settings -> Controller -> Desktop configuration." However, as shown in the screenshot above, we cannot find the controller option in the settings menu on our Steam Deck. We would like to ask where we should perform the settings.
Thank you for providing this guide.

You're missing the very first part: Tap the Steam icon in taskbar. It's not the system settings you should open, it's Steam settings.


KYLIN-4701 Front end change for Parquet Storage

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Here is what I change.
Remove Encoding and Length in CubeDesc Advanced Setting

Remove Streaming Source in Load Table

Remove Column Family & Advanced Dictionaries & Advanced Snapshots
N/A
Improved tip
N/A


UC Exchange Cydonia uses hospital map marker icon

What the title said

Details
Here's a self explanatory screenshot
https://imgur.com/xvYACHy
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Go to cydonia and check the place

This has been fixed officially.


chore(package): update version to 0.2.1

I realized we hadn't updated the version field in package.json to v0.2.1.
So, I've created a commit to update it and I will tag that commit (once merged) as v0.2.1.


clickhouse datetime64(3) shows wrong millisecond

Description
The query sql is select toDateTime64('2022-12-16 09:09:09.080',3), the executing result  in text format is:
toDateTime64('2022-12-16 09:09:09.080', 3)|
------------------------------------------+
              2022-12-16 01:09:09.80000000|

the result of millisecond is supposed to be 2022-12-16 01:09:09.08000000
DBeaver Version
Community 23.0.0
Operating System
windows server 2019
Database and driver
clickhouse server version  22.12.3
ClickHouse driver (server version 21.3+)
Steps to reproduce

query select toDateTime64('2022-12-16 09:09:09.080',3) in the sql edit
execute

Additional context
No response

Hello @xayjin
Unfortunately, I can't reproduce your issue.
What driver version do you use - legacy or not?

Can you provide your Data formats settings screen?

hi @LonwoLonwo , this is Data Formats. Timestamp Pattern is default value yyyy-MM-dd HH:mm:ss[.fffffffff], and Timestamp timezone is UTC+16. I try UTC+2, also the wrong millisecond.

this is dbeaver version.

this is clickhouse driver version.


API import not returning create IDs and not supporting updates

I need to map the created timesheet entry IDs from timeld to my timesheet entries locally, so I know I don't send them again for import. Can import endpoint return created entry IDs?
Also, if I don't have the IDs of the entries, I can't send update requests. I know the API doesn't support updates for now but probably a future extension would do it? Both prejorunal and Tiki support timesheet entry updates via the API.

Every imported JSON object is treated as an overwrite. So you can update an entry by sending its information again with some change.
For timesheet entries, you must include an external property which is the identifier in your system. If you send the same identifier again, it will overwrite the previous entry information. So, there's no need for you to know our internal ID.
(For Timesheets and Projects, you have to specify the @id anyway.)

I see. Then, we are all set here.


Include google maps api

Is there a way to use google maps with flutter_map ?
I realy want to use flutter_map librairy. But as i can see, it works with mapbox and others. It is possible to manipulate google maps with flutter_map package ?
Thanks!

no there's not. Google doesn't allow their tiles to be used as standalone tiles

Okay thanks. Let's go with mapbox.
I took a look at the mapbox doc, the map is pretty good with lots of places. Tell me is there some big differences in functionality between mapbox and google maps?

You'll have to be more specific... Google maps is actually a large API with places search, etc.  MapBox is mainly used for displaying map tiles but has other APIs as well. You'll have to look into the docs for your use-case to compare

Okay! Thanks a lot.


Modify typo of the specification document

I think maybe that's points are typo, because of used by example code.

@horiuchi, Thank you for fixing the typos. As I mentioned in the comment above, animation is obsolete and play is used instead.
Would you revert the line? Then we can merge your pull request. Thanks😃

@yoichitgy Sure, I reverted that line 😄

Thank you for the update!


MPD remote host

The ability to chose a specific host of the mpd module would be helpful for remote setups as it is currently impossible to do this with lemonbuddy.

You can do it when building the project. Got a cmake setting for it. Quite useless though when installing it from packages.
I'll move it to the user config.


bundler error after ruby upgrade to 3.3.0

The following error occurs after upgrade ruby 3.2.2 -> 3.3.0
$ bundle exec rails server
bundler: failed to load command: rails (/home/solinux/.rbenv/versions/3.3.0/bin/rails)
/home/solinux/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/shakapacker-7.2.1/lib/shakapacker/version_checker.rb:72:in `raise_differing_versions_warning': **ERROR** Shakapacker: Shakapacker gem and node package versions do not match (RuntimeError)
Detected: 7.1.0
     gem: 7.2.1
Ensure the installed version of the gem is the same as the version of
your installed node package.
Do not use >= or ~> in your Gemfile for shakapacker without a lockfile.
Do not use ^ or ~ in your package.json for shakapacker without a lockfile.
...

There is no >= or ~> in the Gemfile
...
gem 'shakapacker'
...

Removing the older version of shakapacker won't help
$ gem uninstall shakapacker

Select gem to uninstall:
 1. shakapacker-7.1.0
 2. shakapacker-7.2.1
 3. All versions
> 1
Successfully uninstalled shakapacker-7.1.0
WARN: Unresolved or ambiguous specs during Gem::Specification.reset:
      psych (>= 4.0.0)
      Available/installed versions of this gem:
      - 5.1.2
      - <IP_ADDRESS>
WARN: Clearing out unresolved specs. Try 'gem cleanup <gem>'
Please report a bug if this causes problems.

$ gem cleanup shakapacker
Cleaning up installed gems...
WARN: Unresolved or ambiguous specs during Gem::Specification.reset:
      psych (>= 4.0.0)
      Available/installed versions of this gem:
      - 5.1.2
      - <IP_ADDRESS>
WARN: Clearing out unresolved specs. Try 'gem cleanup <gem>'
Please report a bug if this causes problems.
Clean up complete

Expected behavior:
To be able to upgrade to ruby 3.3.0
Actual behavior:
Bundler error
Small, reproducible repo:
Setup environment:

Ruby version: 3.3.0
Rails version: <IP_ADDRESS>
Shakapacker version: 7.2.1

@s0rin I am closing the issue as resolved. Feel free to comment with further details if your issue didn't get resolved by following what I suggested.

Indeed after updating shakapacker version in package.json the error has been solved.
Closing the issue is fine, thank you for your support!


LOCK LAYER BUTTON BUG

Hi!
When clicking on the lock icon the layer does not lock immediately, it is necessary to make some object selection in Artboard.
Lock icon does not change when clicking, same with eye icon (visible / invisible).

hi @danielmalencar
Thanks for your reporting.
I checked it.
I will fix it and let me know to you.

I fixed it now.
please try to test again.

Now works!


[Feature] Usage as backend in Pytorch

Hi can we use the mscclpp for pytorch

Sure we can, but this would be a PyTorch's issue, not MSCCL++.


Get occasional error in the logs

Bug description
The device is a  Sonoff Dongle E flashed with router firmware from the Sonoff site
I get the following in the logs occasionally but everything seems to be working fine
Logger: homeassistant.components.zha.core.cluster_handlers
Source: components/zha/core/cluster_handlers/init.py:537
Integration: Zigbee Home Automation (documentation, issues)
First occurred: 3:12:18 PM (2 occurrences)
Last logged: 3:12:18 PM
[0xF9C9:2:0x0300]: [Min mireds is 0, setting to 153] Please open an issue on the quirks repo to have this device corrected
The ZHA GUI for the Sonoff Dongle E shows two switches that can change the LEDs on the Dongle PCB
Steps to reproduce
Just appears in the logs
Expected behavior
No errors, LOL
Screenshots/Video
Screenshots/Video
[Paste/upload your media here]

Device signature
Device signature
[Paste the device signature here]

Diagnostic information
Diagnostic information
[Paste the diagnostic information here]

Logs
Logs
[Paste the logs here]

Additional information
No response

Can you check what device 0xF9C9 is? You can open up the ZHA network map visualization and enter that in the search to see a (barely) highlighted device.

It's a Sonoff USB Dongle E
On Wed, Oct 11, 2023, 7:10 PM TheJulianJES @.***> wrote:

Can you check what device 0xF9C9 is? You can open up the ZHA network map
visualization and enter that in the search to see a (barely) highlighted
device.
—
Reply to this email directly, view it on GitHub
https://github.com/zigpy/zha-device-handlers/issues/2643#issuecomment-1758720376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYCKW76CKPCJZ3EZLWYJPDX64YPJANCNFSM6AAAAAA54YJ76Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

Z3RouterUSBDonlge_EZNet6.10.3_V1.0.0.gbl https://github.com/itead/Sonoff_Zigbee_Dongle_Firmware/blob/master/Dongle-E/Router/Z3RouterUSBDonlge_EZNet6.10.3_V1.0.0.gbl
Regards,
Darrell
From: MattWestb @.>
Sent: Thursday, October 12, 2023 3:06 AM
To: zigpy/zha-device-handlers @.>
Cc: def1149 @.>; Author @.>
Subject: Re: [zigpy/zha-device-handlers] Get occasional error in the logs (Issue #2643)
What exact firmware is the Sonoff USB Dongle E running ?
—
Reply to this email directly, view it on GitHub https://github.com/zigpy/zha-device-handlers/issues/2643#issuecomment-1759132985 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYCKW3CJ3IXGIJ5NKWEBNTX66QFDANCNFSM6AAAAAA54YJ76Y .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AAYCKW2FDDHKMPCHJ7O3IJDX66QFDA5CNFSM6AAAAAA54YJ762WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTI3I6TS.gif Message ID: @.*** @.***> >

itead have cooking one standard firmware from Silabs demo file nd have nit fixing all small things that the manufacture shall do.
Make one issue in there guthub and see if they can fixing it.
Alternative i think one x-employee wad doing one good working router firmware but i dont remember where it is.

https://github.com/xsp1989/zigbeeFirmware/tree/master/firmware/Zigbee3.0_Dongle-NoSigned/RouterForDongle

Looks like older version? 6.7.10 vs 6.10.3 that I'm using.
On Fri, Oct 13, 2023, 1:38 AM MattWestb @.***> wrote:

https://github.com/xsp1989/zigbeeFirmware/tree/master/firmware/Zigbee3.0_Dongle-NoSigned/RouterForDongle
—
Reply to this email directly, view it on GitHub
https://github.com/zigpy/zha-device-handlers/issues/2643#issuecomment-1760983426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYCKW2IIMVSWFZ3NASF25DX7DOXLANCNFSM6AAAAAA54YJ76Y
.
You are receiving this because you authored the thread.Message ID:
@.***>


sample ser/de for structs

Proof serialization fix
⚠ This code won't compile ⚠
Description
The aim of this pull request is to minimize the file size of .proofs caused by Field elements being stored as strings, code provided shows examples on how structures that use FieldElements need to implement serde::Serialization and serde::Deserialization need to accomodate to the fact that not all FieldElements can be serialized.
Serde documentation:
Struct serialization
Struct deserialization
Type of change

[x] Optimization

[x] This change requires new documentation.

[ ] Documentation has been added/updated.

If this approach is the one to be used, documentation on how to implement serialization and deserializaiton should be added.

Closed in favour of #657


Enable support for Moleculer service with version configuration

Currently if Moleculer service has version settings enabled, this plugin wouldn't work because it using service name to call actions. This pull request will check for version and $noVersionPrefix settings of service to build correct serviceName and queueName, so it will work with both version or unversioned services

@Hugome please take a look for my PR
Thanks

@Hugome please take a look for my PR
Thanks

Merged, thanks for your PR :+1:

Merged, thanks for your PR :+1:


pageto, page, etc. keywords are not being replaced

Is it possible that the keyword replacer for 'frompage', 'topage', 'page', 'webpage', 'section', 'subsection', 'subsubsection' is not working or not implemented correctly for the headerHtml and footerHtml. You know, where you should just type  and the inside the  there should be the page number.
I fixed this by inserting the subst() JS script and calling it onload of  in my html-file like this.
    ...

 ...

Maybe this helps somebody.

I got bit by this before. The header and footer html have to be a well formed HTML document, not just a snippet. It's WkHtmlToPdf, not this wrapper that has that restriction.


[REQUEST] Auto refresh preview pane

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Saving the contents of the .ps file (or pressing F5 for refresh?) causes the preview window to update.
Describe alternatives you've considered
The only way I figured out to refresh, is to close the preview pane, and reopen it, by clicking the icon in the upper right. I can assign a keyboard shortcut to PostScript Preview, but it opens up a second preview pane. It would be best if the existing pane simply refreshed.
Additional context
None.

Thanks!


Adds TypeScript type definitions

Addresses issue #3.
This PR primarily adds TypeScript types for the new v2 API. Unit tests for these type definitions were added, as well, using dtslint, which was added as a dev dependency.
The readme file was modified slightly to make mention of the package containing TypeScript definitions because that is something of immediate interest to TypeScript developers when they are searching for and evaluating packages to adopt.
Secondarily, there were a just few places where slight corrections were made:

package.json referenced the "lib" directory that no longer exists
a single space was added to a require statement in test/aws-arn-parser.js
a few spaces at the end of lines were removed

@sandfox, any thoughts on this PR?


[Product Catalog Service] add reflection register for grpcurl debug

Changes
I want to debug grpc method in productcatalog service, But I got error:

after with this PR enhancement:

Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[ ] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

please fix the markdown link check failed

the fails link seems right, can we retry the CI ?

@julianocosta89 could you help trigger the check again?


Fixes #92 and use of ' in spellchecks, and some cleanups.

Tested on most platforms. Read for merge I think.


Catching up with missing commits (not moved to the new branch), Fixing issue #1475: incorrect example in LogProfiles API documentation for CreateOrUpdate, fixing incorrect logProfiles example names.

Catching up with missing commits (not moved to the new branch), Fixing issue #1475: incorrect example in LogProfiles API documentation for CreateOrUpdate, fixing incorrect logProfiles example names.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Retry of closed PR #1476.

@gucalder ping! Could you please see whether we can fix this as part of this PR. Thanks!

@gucalder Thanks for fixing the example. I see that there is still one more invalid example/swagger model for operation LogProfiles_CreateOrUpdate. Could you please have a look?
https://travis-ci.org/Azure/azure-rest-api-specs/jobs/260424100#L663

I do not know how to fix this error reported by Linter:   "code": "DefinitionsPropertiesNamesCamelCase",
"message": "Property named: 'odata.type', for definition: 'RuleCondition' must follow camelCase style. Example:
'odata.type'.",

Those are 6 of the 10 errors.

@gucalder Looks like the rule should not have triggered when there is a odata int eh property name.
Would you please open an issue at https://github.com/Azure/azure-openapi-validator/issues and link to https://github.com/Azure/autorest/issues/2064 so that we can get it taken care. Thanks!
Secondly, Please also have a look at the build MODE:model PR_ONLY=true here. That build ensures that provided examples are matching the swagger models. Let me know if you need any help. Thanks!

Issues with the MODE:model PR_ONLY=true fixed. Waiting for the result of the validations.

Thanks @gucalder for incorporating the feedback.

No modification for AutorestCI/azure-sdk-for-node

 https://github.com/Azure/azure-sdk-for-ruby/pull/878

 https://github.com/Azure/azure-sdk-for-python/pull/1353

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-node


1.10: Copy for emails and section on the form

Copy is required for the following items

[x] "Cover page" section of the form

[x] Brief description on who this form is for
[x] Brief description of the process
[x] Link to more detailed info
[x] Privacy disclaimer
[x] How to contact for help

[x] Information on next steps before submission in the form

[x] Brief description of what to expect & next steps after form submission

[x] Confirmation message after form submission

[x] Brief description of what to expect & next steps after form submission

[x] Confirmation email after form submission to complainant

[x] Brief description of what to expect & next steps after form submission

[x] Confirmation email after form submission to recipient
[x] Email to be sent to PSA/LDB secure mailbox

Digital Journey Virtual Team - Assignment 728.docx

I believe copy has been provided for each of the above elements.


query with postgres function -> Extract(MONTH FROM column)

Related Issues: #IssueNumber if necessary

Introduction
I need to filter grouped data on fetch or fetchAll
Issue Description
I cannot find a way to run a query like the one below:
select EXTRACT(MONTH FROM "lastPaymentDate"), count( "lastPaymentDate") from "invoice" group by EXTRACT(MONTH FROM "lastPaymentDate")
Describe the bug found or the feature that you are proposing.
Steps to reproduce issue
This is not a bug, I could not found a stackoverflow space for this question
Expected behaviour
What is the behaviour that you were expecting after following the steps outlined above?
Being able to pass functions to the column method as well as the groupBy method.
Actual behaviour
What behaviour did you get instead?

You'll want to use bookshelf ability to opt in raw queryBuilder using QueryBuilder#select and QueryBuilder#groupBy (or QueryBuilder#groupByRaw).
Here's an example (not tested):
class MyModel extends bookshelf.Model {
  get tableName () {
    return 'invoice'
  }
}

const myModels = await MyModel.forge()
  .query(qb => {
    qb.select(knex.raw('EXTRACT(MONTH FROM ??)', 'lastPaymentDate'))
    qb.select(knex.raw('count(??)', 'lastPaymentDate'))
    qb.groupBy(knex.raw('EXTRACT(MONTH FROM ??)'), 'lastPaymentDate'))
  })

Hey thanks for your nice response.
Yes that should work, but sorry I think my problem is that I am using strapi-hook-knex and strapi-hook-bookshelf and I cannot find a way to retrieve the knex function from the contenxt.
my connection configuration is as follow:  "connector": "strapi-hook-bookshelf",
Man a bit of more documentation further than the simple case scenarios for the open source projects wouldn't harm eh ? :(

I think my problem is that I am using strapi-hook-knex and strapi-hook-bookshelf and I cannot find a way to retrieve the knex function from the contenxt

You'll have to ask on the strapi project for that. It's the first time I ever heard of that package.

Man a bit of more documentation further than the simple case scenarios for the open source projects wouldn't harm eh ? :(

Care to contribute the required documentation that's missing so that the next person doesn't feel like that?

Don't worry neither I , I am pretty new with this Strapi and all the integration we have in here.
Yes I know... I do my best whenever I bump onto stuff and towards providing  my feedback for the best as well as responding questions.
To be honest that is the risk we took by going fo a alpha solution, you become their tester and part of the evolution of them as you go.
It's not recommendable unless you can handle the risk I guess.
Thanks for your input @ricardograca

Closing this since it seems to have been answered already.


Rails Budget App

For this project I did the following

Use ruby gems as software packages system.
Install Ruby on Rails framework.
Understand Rails RESTful design and router.
Use controllers to handle requests and render empty views.
Use params from browser request in a safe way.
Use preprocessed html file with embedded Ruby code.
Use layouts and templates for shared content.
Use database migration files to maintain database schema.
Use validations for models.
Secure app from n+1 problems.
Understand what ORM is.
Write SQL queries with ActiveRecord.
Set up associations between models.
Build a webapp that requires the user to log in.
Use devise gem for authentication.
Limit access to webapp resources based on authorization rules.
Analyze in writing why you have made a coding choice using one structure over another.

Hi @Chimoneg27 👍,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project that used wrong flow to open your PR. Kindly use gitflow as stated in the requirement. 👍
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

@OyePriscilla how do I fix this issue

Hi @Chimoneg27 👍,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project that used wrong flow to open your PR. Kindly use gitflow as stated in the requirement. 👍
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

@OyePriscilla how do I fix this issue

@prolajumokeoni how do I fix this issue

@prolajumokeoni what did I do wrong

@Chimoneg27  Your default branch should be a development branch, you open a pull request in your feature branch and all your changes should be on your feature branch (the name of your feature branch should have a name of a feature or action related to your project) and only when your pull request on your feature branch is approved you merge to the development branch, this is the process to follow for solo capstones, for group capstones you merge all your pull request to the development branch and open a pr in your development branch into the main branch


Topic recognition

Super-resolution CNN model to generated high resolution images of the ADNI brain scan dataset with a downsample factor of 4. Following files are included in the project under Brain_MRI_SuperResolution directory.
dataset.py - For the purpose of preprocessing the data-sample  and feeding data to the model.
modules.py - CNN model consisting of residual block and convulational layers
predict.py - Functions used for predicting the low resolution images and display the upscaled images.
train.py - Utilised for training the super resolution model and plotting graphs for training and validation losses and PSNR metrics.
README.md - containing all information about the report

This is an initial inspection - no action is required yet
Difficulty: Normal
General Comments: Files have been implemented in the wrong folder. The student has combined SRCNN with PixelCNN, which results in good outcomes for the output images. However, they only downscale images to 100x100. Also, residual artefacts in the upscaled images are probably a result of the final layer being a ConvTranspose2D layer. Target image size is also 400x400. Overall problem was made easier by their chosen resolutions, but code and README seems ok.
[Recognition Problem]
"modules.py":

Implementation of Super Rez in Tensorflow.
Adds a residual block before upscaling block.
Adds a convolutional layer after the first PixelShuffle upsampling block.
Adds a final layer that is a ConvTranspose2D block
so upsampling 2x via pixelshuffle then 2x via convtranspose.

"dataset.py":

Creates some functions to downsample the images and store them to a folder, rather than computing on the fly.
Creates a function that then loads all of the images into an array and stores them in memory.
Student includes an image sharpen function, which might help in the reconstruction?

"train.py":

Loads AD and NC images separately, but then doesn't randomize before splitting into train and validation. It may be possible that either AD or NC are not in the training or validation set.
Uses checkpoints and early stopping.
Creates training plots.
"Target Size" for Low Rez is 100x100 and High Rez is 400x400. Seems the student is resizing both the input and ground truth, rather than just the input to 60x64. This makes the super rez problem easier, as the initial image is only ~2x downscaled.

"predict.py":

Correctly loads the model.
Provides some reconstructions, again starting from the 100x100 image size.
No metrics computed on the test set.

"README.MD":

Contains description of approach, however figure used is from SRCNN which doesn't itself use pixelshuffle or ConvTranspose2D.
Much lower validation scores compared to training.
Dependencies included.

[Commit Log]
Commit log looks ok.

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage, but no metrics on test -1
Difficulty: Normal -5
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe good missing usage -1
Model/technical explanation
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good

Cannot merge because of changes to repo README and adding DStores files to repo, please remove and revert to allow merge. Does not affect grade.

Sir, done with all of the deletion and reverted back README.md to original.

Feedback attempt and no feedback marks lost.
I think the adjustment is insufficient. All student-constructed documents should be packed in one document under the ‘recognition’ folder, which means you just need to create one folder under the original ./recognition/ folder without modifying any other original files and put all your files into the student-constructed folder. Try to fix it so that Shakes can merge your pull request, though you will not lose any marks.

Changed and put everything in recognition folder


INT B-20627 Fix circleci tests and bump dependency

B-20627
Summary
The package otelhttp:v.0.43.0 is outdated and needs to be bumped up to the latest version otelhttp:v.0.55.0.
Normally this is managed through dependabot automatically but each time dependabot attempted the bump there were test failures in circlci. You can find the trail of failed PRs by going here. The latest results show that the dependency bump was failing one of the automated tests run by circleci, server_test.
It appears that the test is failing due to the use of deprecated methods http.server.request_content_length and http.server.response_content_length. OpenTelemetry-Go Contrib removed these in otelhttp:v0.49.0 release. It appears that the new methods that need to be used are http.server.request.size and http.server.response.size.
After these tests have been fixed, the package should be able to upgraded in the go.mod file to the latest version otelhttp:v.0.55.0 with all circleci tests passing.
Verification Steps for the Author
These are to be checked by the author.

[x] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Login to GitHub.
Open up a DRAFT pull request for MAIN-B-20627-Fix-circleci-tests-and-upgrade-dependency to be merged into main.
Scroll down to the automated tests and wait for the server_test to finish.
Verify the test has passed.

[!NOTE]
Make sure to make the pull request a draft.

Screenshots

Draft PR created, all tests pass!


Edit this page in {repositoryURL}

Summary
I want to remove the Edit this page in {repositoryURL} link. Do you know how I can do that? I'm using this theme as a personal portfolio, and I don't want to have pages edited by visitors.
Relevant information
I attempted to modify EditGithub.js, but it didn't do anything other than break my build.

I commented out the repositoryURL in the gatsby-config.js file.


Support key binds mapping

fix #15

Thank you, I'm checking

Hi @lusingander,
Thanks for the review.  Now everything is good to go.

Thanks so much, please check only the ci error? (I think it's a missed fix in the test code)

Great!!


GH-38297: [C#] Fix build for .NET 4.7.2

What changes are included in this PR?
Fixes #38297
Are these changes tested?
Yes

Closes: #38297

As these are trivial changes aimed at fixing the build, I'm going to check in the changes without additional review and ask for forgiveness if required... .

@CurtHagenlocher Could you use our merge script instead of the GitHub's merge button?
https://github.com/apache/arrow/tree/main/dev#how-to-merge-a-pull-request

Oops, sorry :(. Guess I didn't read nearly enough.

We may need to create a document for new committer in https://arrow.apache.org/docs/ instead of using dev/README.md...


CU-8694vxbkh: Medcat v1.12 support

Remove use of deprecated method.
TODO:

[x] Update requirements to medcat==1.12 (waiting on MedCAT release)

NOTE:
There is an issue with this PR, regarding medcat/evaluate_mct_export/mct_analysis.py using the medcat.utils.meta_cat.data_utils.encode_category_values method (the signature of which was changed in MedCAT 1.12 due to PR 419).
Refer to CU task:
https://app.clickup.com/t/8695vz088

Added the change for below

NOTE: There is an issue with this PR, regarding medcat/evaluate_mct_export/mct_analysis.py using the medcat.utils.meta_cat.data_utils.encode_category_values method (the signature of which was changed in MedCAT 1.12 due to PR 419). Refer to CU task: https://app.clickup.com/t/8695vz088


Facing issue using 'Expand TreeItem' with multiple items.

ISSUE Description:
I'm working on a WPF application that use DevExpress library.
Sometimes and inside tree taht contain multiple treeitems (10000 items) an exception will be retrieved from the 'Expand TreeItem' function:
Full Name:	Test.Expand Folder10
Start / End / Elapsed:	20230802 17:55:48.422 / 20230802 17:56:11.477 / 00:00:23.055
Status:	FAIL
FlaUiError: Incorrect syntax usage 'self.current_treeitem.Expand()'
Test code:
${IODataServerTreeItem}          /Window[@Name="Movicon.NExT"]/Group[@AutomationId="dockingManager"]/Group[@AutomationId="rootLayoutGroup"]/Tab/Group[@AutomationId="UFProjectExplorerUIControl_Panel"]/Custom[@AutomationId="UFProjectExplorerUIControl"]/Tree/Pane[@AutomationId="dataPresenter"]/TreeItem[@Name="I/O Data Server"]
${folder1TreeItem}               /Window[@Name="Movicon.NExT"]/Group[@AutomationId="dockingManager"]/Group[@AutomationId="rootLayoutGroup"]/Group/Tab/Group[@AutomationId="I_O_Data_Server__emptyProject__Panel"]/Custom/Tab/TabItem[@AutomationId="DXTabItemtabAddressSpace"]/Custom/Tab/TabItem[@AutomationId="DXTabItemtreeListTab"]/Tree/Pane[@AutomationId="dataPresenter"]
Expand Folder10
FlaUILibrary.Double Click                      ${IODataServerTreeItem}
FlaUILibrary.Wait Until Element Is Visible     ${folder1TreeItem}            10    Element Not Loaded
FlaUILibrary.Expand Treeitem                   ${folder1TreeItem}    N:folder_10
FlaUILibrary.Wait Until Element Is Visible     ${folder1Variable2TreeItem}            10    Element Not Loaded
FlaUILibrary.Select Treeitem                   ${folder1TreeItem}    N:variable_02
FlaUILibrary.Collapse Treeitem                 ${folder1TreeItem}    N:folder_10
FlaUILibrary.Click                             ${addressSpaceTabCloseButton}
Anyone facing this kind of issue?

@albertorivi77
Which kind of exception do you receive?

Okay thanks for you response. Afterwards this issue should be easily to reproduce if many treeitems will be created. To investigate if it's a flaui issue here and how to solve it.

@albertorivi77
I can see from the code that if an exception occurs when opening a treeview, the error mentioned occurs.
                try:
                    if action == TreeItemAction.EXPAND:
                        node.Expand()
                    elif action == TreeItemAction.COLLAPSE:
                        node.Collapse()
                    elif action == TreeItemAction.SELECT:
                        node.Select()
                except ValueError:
                    raise FlaUiError(FlaUiError.FalseSyntax.format(
                        "self.current_treeitem." + action.value + "()")) from None
                except InvalidOperationException:
                    raise FlaUiError(FlaUiError.ElementNotExpandable.format(node.Name)) from None
                except Exception:
                    raise FlaUiError(FlaUiError.FalseSyntax.format(
                        "self.current_treeitem." + action.value + "()")) from None

Unfortunately, the error type of C# is suppressed and you cannot see which error is forcing it. I am currently also not able to reproduce the error.
Do you have an example of this?


Unable to redeploy Championship Management Platform

I followed the steps and deployed the Championship management platform, but then I had the circling error of not all the groups formed on the Champions Page, so I deleted the site, and then followed by application.
I am then trying to recreate the solution, importing the solution shows success but API access page no permission requests are made, assuming its already there I tried but with no success, after going through the issues here, I deleted the site permanently and now the site also not  getting created and am stuck , any inputs are appreciated to get this going.
Application is listed on the SharePoint Apps Page

API Access I don't see it.

When I sync, I get "sync failed to Teams"

also I dont see the SharePoint site created.

Hi @say-brad ,  Thanks for trying this app. We have updated the package to a new version V1.2 yesterday with few fixes and improvements. Can you please uninstall the existing package and try installing the new package from the same location - link.
You can see the updated ReadMe here for the 1.2 changelog and upgrading instructions as well under the section "Upgrade to version <IP_ADDRESS>".
You don't have to approve the graph permissions again if you have already done once. In your screenshot you can see the permissions are under "Approved" section.
IMPORTANT NOTE: Please wait for 15 minutes after you have completed first 4 steps under the section "Upgrade to version <IP_ADDRESS>" in ReadMe and then continue with step 5. This is to avoid cache related issues for the deleted assets.

@say-brad just wanted to check in to see if you were successful in getting 1.2 / updated package deployed?

Hello Josh, haven't got into installing it, will be doing this week and keep you updated.
Regards,
Sayee
From: Josh Leporati [MSFT] @.>
Sent: Wednesday, August 18, 2021 7:18 AM
To: OfficeDev/microsoft-teams-apps-champion-management @.>
Cc: Sayeenath Sehar-Varadaraj @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-champion-management] Unable to redeploy Championship Management Platform (#37)
[EXTERNAL]: Do not click links or open attachments unless you recognize the sender
@say-bradhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsay-brad&data=04|01|sayeenath_sehar-varadaraj%40bio-rad.com|063321bed48949cdb64808d96252f12b|ef0aebeb2af047a3b8ae5665f8a62d11|0|0|637648930637799807|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=N3YRDMykom1QbE7PDei%2FkJN6Zlz3nvdFFeae3iFuv0U%3D&reserved=0 just wanted to check in to see if you were successful in getting 1.2 / updated package deployed?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fmicrosoft-teams-apps-champion-management%2Fissues%2F37%23issuecomment-901153877&data=04|01|sayeenath_sehar-varadaraj%40bio-rad.com|063321bed48949cdb64808d96252f12b|ef0aebeb2af047a3b8ae5665f8a62d11|0|0|637648930637799807|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OKBjjBUKMDJKmbeZWsKjg9S5MqE8gBwf%2FxBoYu98vLU%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAU2SGTCI4ARDJHX6Y3AEAWDT5O6IJANCNFSM5AJUYCXA&data=04|01|sayeenath_sehar-varadaraj%40bio-rad.com|063321bed48949cdb64808d96252f12b|ef0aebeb2af047a3b8ae5665f8a62d11|0|0|637648930637809762|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ovE46oSFi25vGtsrv2NK9k2GwRW32u4A9WQVUrCQzXU%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|sayeenath_sehar-varadaraj%40bio-rad.com|063321bed48949cdb64808d96252f12b|ef0aebeb2af047a3b8ae5665f8a62d11|0|0|637648930637809762|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7O5C4HrCzoCy6xVrzAbGcL%2FAhBkLY1NNFoOYjGTkr0k%3D&reserved=0 or Androidhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26utm_campaign%3Dnotification-email&data=04|01|sayeenath_sehar-varadaraj%40bio-rad.com|063321bed48949cdb64808d96252f12b|ef0aebeb2af047a3b8ae5665f8a62d11|0|0|637648930637819719|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=r%2FykFzVxGj07mUzbAIiHLKX4Mg8wLIiXY%2BUPTkdgR%2Bw%3D&reserved=0.

Sounds good, thanks @say-brad !

@say-brad just wanted to check in to see if you were able to re-deploy? thanks!

@say-brad please feel free to open a new issue if you are still having any issues with the installation. Also please note we have version 1.3 out as of this post and that should be the latest for you to run the install against or upgrade to, thanks!


Exclude Helm manifests in get-started guide

The get started guide uses repo github.com/weaveworks/flux-get-started
which contains HelmRelease manifests, but the guide didn't instruct
the reader to deploy the Helm operator.
Fixes #1898

I think we should make it more clear to the user that installation of the Helm operator is optional (and only required because the flux-get-started repository makes use of it).
If the user now follows these steps but for example, has not installed Tiller on the cluster (because we did not tell them to), the apply of the HelmRelease will succeed but the roll-out of the release itself will fail (and the operator will only be logging errors).

That's true. Then, we either change the example repository to one without HelmReleases or we add instructions to install Tiller and indicate that helm is optional. Thoughts?

BTW, how could the guide have possiblly ever worked? I'll do some digging

From the git log in my head; there was a commit a while ago that just changed the repository URLs without matching the content.

Then, we either change the example repository to one without HelmReleases or we add instructions to install Tiller and indicate that helm is optional.

I think the latter would be good, with a note saying that for demo purposes we install the operator too (and thus Tiller must be installed) but it is completely up to the user to only make use of the Flux daemon.

Hum. Maybe I'm missing something, but AFAICS the get started tutorial is about "flux standalone" (cf this note, whereas the helm get started tutorial is about helm: the need to install Tiller is discussed there.

AFAICS the get started tutorial is about "flux standalone"

Correct, the problem is that the repository linked in this doc (and used as an example) contains HelmRelease resources, confusing users (see linked issue).

We can also change the repository.
On Fri, Apr 5, 2019, 12:06 Hidde Beydals<EMAIL_ADDRESS>wrote:

AFAICS the get started tutorial is about "flux standalone"
Correct, the problem is that the repository linked in this doc (and used
as an example) contains HelmRelease resources, confusing users (see
linked issue).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/flux/pull/1902#issuecomment-480221103, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACQOJCzeAzdeO7gHxTEkKgFp-mHEgYtzks5vdyA0gaJpZM4cdEfK
.

This is unfortunate. It'd be great if we could

maintain just one example
have a straight-forward example with clear instructions that you can easily tailor to your use-cases

It looks like we can't easily have it all this time.
My inclination would be to merge this PR, but make a note that we at least document why this is necessary in the future, and even better: make the use of the example repo here more "obvious"?

It looks like we can't easily have it all this time.

We can also restrict the repo to directories not including HelmReleases by configuring the flux deployment. I think I will do that.

@hiddeco @dholbach PTAL

Thanks for the fix - this looks clean. (Even if it adds another step to the "get started" experience - I guess I'll file a separate issue about this.)
Have you tried this locally? In my case the new podinfo gets deployed, but for some reason the color does not change.

In my case the new podinfo gets deployed, but for some reason the color does not change.

It does work for me. Do the logs show anything odd?
I don't see why any of the changes I made would cause flux not to update the deployment correctly.

Just started with a fresh minikube, same thing. The correct revision is deployed. Colour stays green.

I will take a deeper look. However I doubt that's caused by this PR. Does it work when deploying helm and the helm op?

The tutorials follow different paths:

standalone get started asks people to change PODINFO_UI_COLOR to blue in workloads/podinfo-dep.yaml - this seems not to work in either of the variants.
helm get started asks people to update the image rev of mongodb, this works.

Maybe we can just copy https://github.com/weaveworks/flux/blame/master/site/helm-get-started.md#L147 (147 and following) over to get-started.md?

/cc @stefanprodan on the above

I think the podinfo application no longer listens to the PODINFO_UI_COLOR environment variable (https://github.com/stefanprodan/k8s-podinfo/search?q=PODINFO_UI_COLOR&unscoped_q=PODINFO_UI_COLOR).

Can me merge this and create a separate issue for the color?

It does work for me.

I retried and it didn't, I would had sworn it did. ¯_(ツ)_/¯
Sorry @dholbach

Let's copy over the Helm Get Started bits into the other tutorial in a separate issues.

Is this ready to merge or is it blocked on something -- I'm not clear after reading the comments :sweat_smile:

Yes, let's do it, and fix the second part of it in a separate PR.


How to update  to the most recent MAIN (or nightly) release if I have used the one click installer for windows for my current installation?

Hi there, how can I update H2OGPT to the most recent MAIN (or nightly) release if I have used the one click installer for windows for my current installation?

I think others like @cherrerajobs are doing that.  Maybe he can contribute some docs to what he does.

The outline you are describing above does not seem to take advantage of the fact that you already have an installed version of H2OGPT using the installer.
What I was looking to identify was a method by which we could simply update a few files (which had been updated as part of a bug / feature release) manually, or simply copy the recently available main release folders into the installed file structure of H2OGPT (which had been previously installed using the on-click method).
The steps that I took, which did not ultimately work were to, download the latest main release and simply replace all of the H2O GPT folders on my laptop (e.g. C:\Users\xxx\AppData\Local\Programs\h2oGPT with the source code files of the main  release and then starting/clicking or executing the .exe file for H2O GPT. Unfortunately this resulted in the browser tab not appearing which obviously suggests that the copying of the files did not work. Maybe I am slightly naïve but I thought that would be sufficient as the updates were made in only a few files. Simply exchanging those source files within the installed H2O GPT would solve the problem and update the installation.
Could you take a few minutes to explain why that approach would not work? I could have done that at the beginning and that undermines the whole question of updating a current installation rather than doing a completely new manual installation. Thank you.

Hi There
Any update? I know, I Know. Its the holiday season! 😊

The installation often changes packages, so I'm not sure it's super easy to identify what to add to the one-click installer files.
However, the python env can be activated and used like normal, and packages can be installed, but i'm not sure this is reliable since the packages are often complex in changes.
Probably the best option is to have the one-click installer installed, but overwrite the installed env with a new env that is manual install.
I'll try to get a windows one-click up by Jan 4, but yes holidays slow things down.

Thank you for the update; and your continued efforts, they are much appreciated.

Hi, I updated windows support for both one-click and batch script (new) installation.  Should be easier to install "manual" way now.
I now include pip as well in the packages, and one can update to new versions of packages more easily now.  See README.md and the example of torch itself.


Upgrade to Go 1.21 and update all dependencies and container base images

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


Doesn't compile for x86

Describe the problem
I can't deploy to x86
To Reproduce
Steps to reproduce the behavior:

Try and deploy to x86

Expected behavior
All images to compile
Screenshots

Device information

Board (hardware info): Apple Mac Mini
Device type (as seen in balenaCloud): Generic x86_64
balenaOS version: balenaOS 2.54.2+rev1

Additional context
Add any other context about the problem here.

Forgot to add this one:

Hi @dewitpj, balenaSound currently doesn't support Generic x86_64 device type, the only x86 device with support is the Intel NUCs. That said, running balenaSound on a mac mini sounds cool, I'll try to build generic x86 images in the coming days if you are up for doing a bit of testing.

Totally up for testing it 👍

Hey @dewitpj the images should now be available for testing. Let me know how it goes!

shakes fist @ spotify
[Error]             Some services failed to build:
[Error]               Service: spotify
[Error]                 Error: The command '/bin/sh -c curl -sSL https://dtcooper.github.io/raspotify/key.asc | apt-key add -v - 2> /dev/null   && echo 'deb https://dtcooper.github.io/raspotify jessie main' | tee /etc/apt/sources.list.d/raspotify.list   && install_packages raspotify' returned a non-zero code: 100
[Info]              Built on x64_01
[Error]             Not deploying release.

Ah right, raspotify does not have an x86 build. I'll have to replace that with librespot which i think does. In the meantime you can comment out the spotify part from the docker-compose.yml file to see if it builds fine.

I'll give that a go and see - give me a few mins :)

Ah right, raspotify does not have an x86 build. I'll have to replace that with librespot which i think does. In the meantime you can comment out the spotify part from the docker-compose.yml file to see if it builds fine.

Yip - that deploys fine - I see all the services running, but AirPlay doesn't show up....not sure it will ?
Keeps saying 28.11.20 08:05:08 (+1300)  airplay  Waiting for audioblock to start...

...and then "audio" restarts - it does "see" the card thou

Hey @tmigone,
I'm trying to get it to compile but for some reason it keeps defaulting to ARM v7:
[spotify]            ---> 6a33bba29141
[spotify]           Step 2/6 : RUN install_packages librespot
[spotify]            ---> Running in 804c5fe76ceb
[spotify]           Here are a few details about this Docker image (For more information please visit https://www.balena.io/docs/reference/base-images/base-images/):
[spotify]           Architecture: ARM v7
[spotify]           OS: Debian Buster
[spotify]           Variant: run variant
[spotify]           Default variable(s): UDEV=off
[spotify]           Extra features:
[spotify]           - Easy way to install packages with install_packages <package-name> command
[spotify]           - Run anywhere with cross-build feature  (for ARM only)
[spotify]           - Keep the container idling with balena-idle command
[spotify]           - Show base image details with balena-info command
Any tips ?


Question-Commercial and ready to use devices?

Hi all,
This is not an issue but I cannot create anything else to post a question.
Is there any list of ready to use devices from known commercial manufacturers?

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

ID: 6acaea64-2587-d161-1ce7-053e6ab45967
Version Independent ID: ae15e7d0-48a7-8982-7c0a-f122b89b5af8
Content: Speech Devices SDK microphone array recommendations - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/speech-devices-sdk-microphone.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @erhopf
Microsoft Alias: erhopf

@jonmikeli Thanks for the question. We are investigating the issue and will update you shortly.

@jonmikeli Roobo Smart Audio Dev Kit for Microsoft speech services.
Please follow the below document for setup of the Roobo Dev Kit.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi @ram-msft,
Thank you for your answer.
Unfortunately, it does not answer my question (related to commercial devices and not development devices). Any known commercial device fulfilling all the requirements you detail in the link below?
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/speech-devices-sdk-microphone

@jonmikeli We have forwarded the same to the product team to check. @erhopf Could you please comment on the commercial devices fulfilling all the requirements in the doc.


20% hot reload benchmarks regression on Dart roll

https://flutter-flutter-perf.skia.org/e/?begin=1690415288&end=1690495744&keys=X6959ede456befbb073f0c12a7fe63d78&selected=commit%3D35993%26name%3D%252Carch%253Dintel%252Cbranch%253Dmaster%252Cconfig%253Ddefault%252Cdevice_type%253DMoto_G__4_%252Cdevice_version%253Dnone%252Chost_type%253Dmac%252Csub_result%253DhotReloadMillisecondsToFrameAfterChange%252Ctest%253Dhot_mode_dev_cycle__benchmark%252C&xbaroffset=35993
Seems like across the board there was a 20% regression. This happened on the manual dart roll here:
https://github.com/flutter/flutter/pull/131371

The Dart changes that went in during this time are https://dart.googlesource.com/sdk.git/+log/b1e82e2e55b2..b5710782b91c
The one change that probably caused this is
6ef967f [vm, reload] Delay enum forwarding until after instance morphing. by Ryan Macnak · 2 days ago
(https://dart-review.googlesource.com/c/sdk/+/316041, this change was reverted and reapplied at https://dart-review.googlesource.com/c/sdk/+/316501)
This was a correctness fix and would have impact on reloads done with shape changes
The trade off here is crashes during hot reloading Vs this regression because of the additional pass

//cc @rmacnak-google

Yes, the correctness fix involves an extra O(heap) step, so 20% isn't surprising.

@rmacnak-google Can we avoid this extra heap iteration? Maybe we can special case canonicalization of enum constants to avoid depending on the contents of instances, and still forward them along with other objects.

Re-opening for the question from @alexmarkov

There was such a special case before the "enhanced enums" changes: enums had only name/String and index/Smi fields that could not involve a shape change. But merely running the constant evaluator is unsafe with a pending shape change now since that runs canonicalization for each intermediate object, possibly one with a shape change.
In the absence of a shape change, or a shape change with no live instances, the become operation for the shape change should be a no-op and the enum forwarding should already be folded into the same become operation as class/library forwarding.

During hot reload, can we create placeholder empty objects for new enum values (only initializing index and maybe name), migrate old -> new objects on the heap and then fill in those empty objects with real contents by reading it from kernel?

I'm not sure what constitutes a state change. The benchmarks measure three different source code changes:

https://github.com/flutter/flutter/blob/master/dev/integration_tests/flutter_gallery/lib/gallery/app.dart#L148 The string is changed from 'Flutter Gallery' to 'Updated Flutter gallery'

https://github.com/flutter/flutter/blob/master/dev/integration_tests/flutter_gallery/lib/demo/calculator/home.dart#L258 The line is replaced with "fontSize: (orientation == Orientation.portrait) ? 34.0 : 24.0"

A newline is added to the end of https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/widgets/framework.dart

Looking through the numbers, it looks like only 1) and 2) regressed

A shape change is the adding or removing of instance fields. In such cases, instances need to be reallocated to their new size. None of these scenarios involve shape changes.

@alexmarkov Maybe. Filling in the values in the right order to prevent canonicalization from seeing uninitialized instances (Enum -> non-enum constant -> Enum) seems like it would require a whole parallel implementation of constant reading and canonicalization.

Golem reports its reload benchmark back to before the forwarding fix. So performance is as before for reloads without shape changes, but reload with shape changes will still incur the extra heap walk.


Custom Views

How do i go about Creating Custom Views for this tool? Love what it dose but would like to match the theme to my site? is there a generate command for this? documentation is very poor.

It doen't look like the is a generator for the views. Would be a nice feature though.
You can find the views here:
https://github.com/phatworx/devise_security_extension/tree/master/app/views/devise
Just add those folders and views to your project and customize them.