Allow GAUGE segmental sub-labels to display mdi-icons

Description
Replacing the sub-label text with an icon, could identify the chart's usage, and therefore users could then leave the widget title blank, to maximize real-estate.
https://discourse.nodered.org/t/ui-gauge-is-available/85318/15

Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

As a quick win here, client-side, check if props.unit has mdi- at start, if so, show an icon in place of the text. May need to improve the styling/sizing of the icon too vs. text-version

Is that not liable to be limiting at some point if you allow icons other than mdi?

Is that not liable to be limiting at some point if you allow icons other than mdi?

Sure, but easy enough to support for now and override in the future.

Though you will be committed to supporting this for a long time, in order to avoid breaking changes as far as possible.

I see an additional config input box has been added to change the sub-labels - now 'units' & 'icons'.
I thought the original plan was;

check if props.unit has mdi- at start, if so, show an icon in place of the text.

so that there was only one input box, for either text or a MD icon, which sounded a cleaner approach, and makes things easier for new users
Especially, if the icon/text size is increased, there would not be enough room for both an icon & text.


Missing Stamen Toner Layer Sources

Describe the bug
Missing Stamen Toner as a Layer Source
OL Link to Stamen Toner Source

It has been implemented with v1.0.0-next.1.


Copy selection from queue to browse, and vice versa

Make each view aware if the other has selected items, and add a 'Copy here' entry to context menu.

Implemented in 2.3 branch.


how can we use this tool with a model that have multiple parts in input?

For example, my model takes two inputs like the following coding shows
        outputs = net(inputs, tr_angles)

For example, my model takes two inputs like the following coding shows
 outputs = net(inputs, tr_angles)

I have tried graph = hl.build_graph(net, inputs, tr_angles) and graph = hl.build_graph(net, (inputs, tr_angles)) and graph = hl.build_graph(net, [inputs, tr_angles])

Hello, have you solved thsi question?
I have the same question as you.


an Importerror

Hi,your code is great,but When i run demo.py,find an error:
File ".\python-objectness-master\c_segment\c_segment.py", line 1, in
from c_segment import segment_img
ImportError: cannot import name 'segment_img' from 'c_segment'
how can I do it，Thanks :)

Did you follow Step.4 in Get Started to build the python interface?

Did you follow Step.4 in Get Started to build the python interface?

thank you for the reply , but it says:
ModuleNotFoundError: No module named 'c_segment.build.c_segment' (ㄒoㄒ)

You need to build the python interface first:
$ cd c_segment
$ mkdir build
$ cd build
$ cmake .. && make

Then the module 'c_segment.build.c_segment' will exist.

You need to build the python interface first:
$ cd c_segment
$ mkdir build
$ cd build
$ cmake .. && make

Then the module 'c_segment.build.c_segment' will exist.

Thank you very much ，I solve it  👍

You're welcome.


Error while running drugZ_w_modules.py

Hi Medina,
When I was running  drugZ_w_modules.py, I got the error information:
File "drugZ_w_modules.py", line 347, in drugZ_analysis
fold_change =pd.concat(fold_changes, axis=1, sort=False)
TypeError: concat() got an unexpected keyword argument 'sort'
It is OK when I was using DrugZ.py
Best,
Wenjun

That's due python version.
DrugZ.py is python 2, while drugZ_w_modules.py is python 3.
I will update the FDR calculation in drugZ.py in a bit, and you can keep using drugZ.py then.

Thanks. Problem is solved after I upgrade the pandas to 0.24.

Great! Happy to hear it's solved.

The same error is still existing.
INFO:main:Initiating analysis
INFO:main:Loading the read count matrix
INFO:main:Normalizing read counts
INFO:main:Calculating raw fold change for replicate 1
INFO:main:Caculating smoothed Epirical Bayes estimates of stdev for replicate 1
INFO:main:Caculating guide-level Zscores for replicate 1
Traceback (most recent call last):
File "drugz.py", line 478, in
main()
File "drugz.py", line 475, in main
drugZ_analysis(args)
File "drugz.py", line 458, in drugZ_analysis
fold_change =pd.concat(fold_changes, axis=1, sort=False)
TypeError: concat() got an unexpected keyword argument 'sort'


Indicators CSS adjustments

Wrapped panes should inherit first & last child css (border-radius)

Lower one looks weird


COMP: Fix image pointer casting error

Fix image pointer casting error.
Fixes
In file included from
ITK/Modules/Core/FiniteDifference/include/itkDenseFiniteDifferenceImageFilter.h:194:0,
from
ITK/Modules/Filtering/AnisotropicSmoothing/include/itkAnisotropicDiffusionImageFilter.h:21,
from
ITK/Modules/Filtering/AnisotropicSmoothing/include/itkGradientAnisotropicDiffusionImageFilter.h:21,
from
src/Filtering/AnisotropicSmoothing/ComputePeronaMalikAnisotropicDiffusion/Code.cxx:22:
/.../ITKExamples-build/ITK/Modules/Core/FiniteDifference/include/itkDenseFiniteDifferenceImageFilter.hxx:
In instantiation of
'void itk::DenseFiniteDifferenceImageFilter<TInputImage, TOutputImage>::CopyInputToOutput()
[with TInputImage = itk::Image<unsigned char, 2>; TOutputImage = itk::Image<float, 2>]':
src/Filtering/AnisotropicSmoothing/ComputePeronaMalikAnisotropicDiffusion/Code.cxx:72:1:
required from here

ITK/Modules/Core/FiniteDifference/include/itkDenseFiniteDifferenceImageFilter.hxx:44:35:
error: conversion from 'itk::SmartPointer<itk::Image<float, 2> >::ObjectType*
{aka itk::Image<float, 2>*}' to non-scalar type
'itk::Image<unsigned char, 2>::Pointer
{aka itk::SmartPointer<itk::Image<unsigned char, 2> >}' requested

typename TInputImage::Pointer tempPtr =	output.GetPointer();
                              ^~~~~~~

raised at:
https://open.cdash.org/viewBuildError.php?buildid=7083938
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

Note that this was triggered by an ITKExamples build:
https://open.cdash.org/build/7083938
The same blaster.kitware machine builds ITK sources nightly; not sure why the error is not being reported in those.

@N-Dekker I'm fine if what you are proposing is the solution, but am not willing to push force that immediately to later find another alternative. So I will wait for a couple of days before adopting that change and push forcing to leave time for discussion if people find downsides to it.

I ran into a compile issue with SimpleITK's current master building against ITK's current master seemingly due to the changes in https://github.com/InsightSoftwareConsortium/ITK/commit/5b3fc71e874323253216bb2d146b2a7f7c9029ea as well. The SimpleITK build appears to have succeeded after applying the current changes proposed in this PR.
I'm currently testing to upgrade the ITK version used in 3D Slicer.

So I will wait for a couple of days before adopting that change and push forcing to leave time for discussion if people find downsides to it.

Sure, thank you for looking at the issue, i am waiting, can not build project with ITK master branch

@jhlegarreta

So I will wait for a couple of days before adopting that change and push forcing to leave time for discussion if people find downsides to it.

Thanks for your patience  😃 Again, I think the current commit https://github.com/InsightSoftwareConsortium/ITK/pull/2386/commits/57e5f81869b966425c4b1947664a330f5d20180e is just fine. Given the comments from James (@jamesobutler) and Mihail (@issakomi), I would prefer to have it merged today already. Would that be OK to you as well?
We could always discuss possible further style improvements afterwards, right?

@jamesobutler and @issakomi thanks for the report and @jamesobutler for giving the fix a try.

We could always discuss possible further style improvements afterwards, right?

@N-Dekker Please feel free to submit your proposal as a PR. Thanks for the effort to make the code even more modern and robust.


Error in Connection Example

Hey,
just wanted to thank you for this package. Lifesaver.
And there is a mistake in connecting example.
It should read const {AutotaskRestApi} = require('@apigrate/autotask-restapi');

Corrected typo. Thanks.


We have a FastfilePeeker and a FastfileFinder, does this make sense

The comments should explain the relationship between the 2. Maybe they should be the same class, but just have different methods? They're usually used together, right?

The responsibility of one is, given a repo (a path) find the fastfile and the other one, given a repo (branch) see what lanes are in there, so it needs both the finder and the repo

Haha, sorry I didn't mention this in the issue body: This issue can be closed by updating the source code to make it more clear, meaning ideally we add some comments to the code base explaining just that 👍

I tried to document a little bit more in #458

Is this ready to close? 😄


wrapBy doesn't work like expected

Version
Tell us which versions you are using:  react-native-router-flux v4.0.0-beta.24
Expected behaviour
Documentation says about wrapBy Router prop:

function to wrap each Scene component and nav bar buttons - allows easy MobX integration (by passing observer)

I'm trying to use it with mobx-react/observer with this store:
class MyStore {
  @observable list = []

  @action.bound
  async fetch() {
    this.list = await fetchList()
  }
}

And I use this Router:
<Router wrapBy={observer}>
    <Stack key="root">
      <Scene
        key="offers"
        component={TestScene}
        title="Offers"
        on={store.fetch}
        store={store}
      />
    </Stack>
  </Router>

And Test component:
const Test = ({ store }) => (
  <View>
    <Text>Size of list is: {store.list.length}</Text>
  </View>
)

and I expect that Test component is rerendered when list is fetched
Actual behaviour
Test component is not rerendered. But when I manually add observer to the component, like this:
const Test = observer(({ store }) => (
  <View>
    <Text>Size of list is: {store.offers.length}</Text>
  </View>
))

It does work and it is updated when list is fetched.
Any suggestions? Am I using wrapBy incorrectly?

I have the same issue.
How shoud i use wrapBy?
Thanks

Same issue here
`import React from 'react';
import {
Router,
Scene,
Modal,
Drawer,
} from 'react-native-router-flux';
import { observer } from 'mobx-react';
import Reg from './containers/Reg';
import DrawerComponent from './components/Drawer';
export default function Scenes() {
return (

);
}
`


Missing interfaces for Recipient.PersonName and SpecifiedLegalOrganization.TradingBusinessName

I'm using the Mustang framework to generate ZUGFeRD files for version 2 / profile EN16931.
I try setting PersonName for recpient and sender. For sender it works, but not for recipient.
Then I found out why: class ZUGFeRD2PullProvider (Line 147) only sets DefinedTradeContact for sender, but not for recipient.
Is this a bug or a feature?
Also, I didn't find an interface for the attribute TradingBusinessName (this attribute is directly under SpecifiedLegalOrganization). I grepped the whole project but found nothing.

The TradeContact is set also for recipient, but only if profile is EXTENDED or XRECHNUNG, not for profile EN16931.
Don't know if it would be legal to add profile EN16931 here.
I started working on the second issue.

TradingBusinessName is Extended Only but having TradeContact for recipients not in EN16931 was most probably my fault, in particular if it's enabled for XRechnung, only the SalesAgentTradeParty TradeContact is extended only

thank you guys for the quick reponse, awesome :)


Record/Document Test Coverage

Looks like JavaCoCo is a strong option. It has a Maven plugin and integrates well with Eclipse.

Do we want to toss in Find Bugs too?
I'm not expecting 100% coverage with zero bugs reported, but it could be fun to add these in.


Page geometry

Why is the page geometry different for draft option? I understand shifting the margins, but why would you make it a 10" wide page? This forces weird shrinkage when printing.
I have started this branch to discuss options. Not ready to push these changes yet.

The reason that I made this geometry choice was that it adds sufficient space around all the margins without changing the aspect ratio of the written text on the page. I can see the utility of keeping a letterpaper page size while shifting margins around (no changing font sizes, properly sized figures, etc.), but this seemed like a good way to get space and since it's a draft, the text size doesn't really matter. Also, this way seems intuitive because it emulates what happens when MS Word prints out a document with track changes in the margins.
I'm open to adjusting this, but I would need a compelling reason to switch.


Support for stitch clips

A node that shows stitch clips assets connected. This could be one node that some text designation of frame range (ex. 1-50)

dfc3571e2aca06cedb24f359c8451cc28e3065f1


[Storage] Rename options types to be consistent

When the high-level functions were moved into clients, their options type name
were not updated. This change renames them to be consistent with other options
types.
Resolves #5648

alright should be fixed now.


Add --master and --lazy-apps to uwsgi CLI options

After conducting some research it seems like these two CLI options are commonly used with uWSGI. If --master is used, we also require --lazy-apps for the contrast python agent to work correctly. This PR updates our default uWSGI flags to include both --master and --lazy-apps.

passing pipeline (internal only) https://github.com/Contrast-Security-Inc/python-agent/actions/runs/1219082817

@danilito19 yup, I linked the example pipeline so I'm sure it works


Switch HTTPS Certificate from self signed to CA signed

As the title states.
 It is much more recommended to use a CA for a locally hosted server so it is easier to send requests to and fro.

This can't be done. CA certificates can be only be issued to NS/MX registered records, and thus, are not viable for localhost, or any example.com domains that exist locally.


feat: ScreenShot

#3752

hello 😄 @doraemonxxx
I think you are right, but this library also has many stars, which shows that it also has many usage scenarios. The main reason is that I saw this issue and communicated with the project members, so I made this function


Change parameter user to owner for github createRelease

Fixed https://github.com/webpro/release-it/issues/61

Thanks @achichen, this was merged and released in 2.5.1.


In the event your CWD has been removed this cleans up the error messa…

…ge while running mbed-cli.
Current error message:
Traceback (most recent call last):
File "/usr/local/bin/mbed", line 11, in
sys.exit(main())
File "/usr/local/lib/python2.7/site-packages/mbed/mbed.py", line 2567, in main
cwd_root = os.getcwd()
OSError: [Errno 2] No such file or directory
New error message:
[mbed] ERROR: Unable to get current directory: [Errno 2] No such file or directory

WIP disregard


Runtime: external storage enabled by default

Needs https://github.com/rilldata/rill/pull/3546 before this can be merged.

duckdb driver now enables duckdb-ext-storage support by default.
Removed duckdb-ext-storage so db needs to updated after these changes are merged.
Added duckdb-int-storage which allows to disable external db storage in case its required.

Awaiting resolution of this issue: https://github.com/rilldata/rill-private-issues/issues/62


[sidebar-nav] add labs logo back in + CSS tweaks + Expand first three sections if there

Added functionality to expand the first three sections if we navigate to something in those sections.


3DES CMAC and supported key sizes

Hey guys,
I'll be very grateful if you could clarify one thing for me:
I added 3DES CMAC support to OP-TEE some time ago (almost everything was already
there actually, in tomcrypt and mbedTLS respectively):
https://github.com/OP-TEE/optee_os/pull/4381
I was recently challenged by my fellow engineer about the supported key
sizes and I don't have enough data to back up my point.
Let me explain (I'll use key sizes with parity bits from now on).
My current understanding is that TDEA CMAC supports 3 keys (K1-K2-K3) and 2 keys
(K1-K2-K1).
If, for instance, I need to use 2 key TDEA CMAC what I'm doing (as a consumer of
this implementation not as an author) at the moment is just copying K1 to K3 and
have K1-K2-K1 192 bit long 3DES key.
My fellow engineer challenges this implementation by saying that he should not
be obliged to do it himself and that this implementation of 3DES CMAC is wrong
(not wrong, but incomplete).
Unfortunately, I don't have any sound proof that this is not the case.
I checked mbedTLS implementation and it also supports only 192-bit 3DES keys, so
for my case I need to do exactly the same (copying K1 to K3) before I'm able to
calculate 3DES CMAC.
I also checked Nettle, which states in its documention that
"As a 112 bit key is large enough to make brute force attacks impractical, some
applications uses a “two-key” variant of triple-DES. In this mode, the same key
bits are used for the first and the last DES box in the pipe, while the middle
box is keyed independently. The two-key variant is believed to be secure, i.e.
there are no known attacks significantly better than brute force. "
I've got two options atm:

modify TDEA CMAC implementation in a way that if 128 bit keys are supplied,
it will transparently copy K1 to K3 by itself without requiring a user
to do anything at all.
It could be implemented by I don't think that this is the right thing to
do tbh.
allow only 192 bits keys to be used for TDEA CMAC.
Right now a user can pass 128 bit keys but he gets incorrect result as keys are
not automatically transformed to 192 bit representation.

But as I said I can't prove that this is the right choice (limiting the key
size) as my opponent states that this would be incomplete solution as he
thinks that he doesn't have to copy anything in case of 128 bit keys.
What was your rationale for allowing only 192 bit keys when you added TDEA CMAC support?
Thanks in advance.

The world has pretty much stopped using any form of DES years ago outside of banking and people who want to implement all NIST-compliant algorithms. Mbed TLS isn't commonly used in banking, so DES has been a low-priority concern in Mbed TLS for a while. It was before my time, but I would guess that 2-key 3DES was never important enough to both with.
NIST stopped accepting 2-key 3DES except for legacy use a few years ago. (At least since 2017, with SP 800-67 rev2: see §3.1.)
If OP-TEE needs 2-key 3DES, I don't see anything wrong with copying K1 into K3. That's what psa_cipher_setup does. We'd accept a patch to make psa_mac_setup work for 2-key DES, if it currently doesn't (it probably doesn't since it has no special handling). But we'd probably accept a patch to remove single-key DES and 2-key 3DES support, too: I only added them because it validated that the interface and the implementation could handle quirks such as DES's.
(What I wrote here about the importance of DES is my personal opinion, not an official position of Arm.)

@gilles-peskine-arm, thanks a lot for you reply.

If OP-TEE needs 2-key 3DES, I don't see anything wrong with copying K1 into K3.

Ok, I also asked OP-TEE guys what they think and I guess it could be the way to go if there's no strong objection:)
I don't think that OP-TEE really needed that implementation tbh, but I had to make sure that it gets to upstream due to different reasons.


Cronjob fails to update certificate if there is an update to bncert

What steps will reproduce the bug?
When the cronjob that bncert creates runs an update of a certificate it will fail if there is an update to the bncert tool.
Is there some command we can run along with the cronjob, that only updates the bncert tool, but doesn't run an update of the certificates?
What is the expected behavior?
No response
What do you see instead?

Additional information
No response

Hi @CMeldgaard, AFAIK bncert does not execute with the cron job that gets created by the tool. Here can you see the related logic: https://github.com/bitnami/bncert/blob/main/project/bncert-letsencrypt-functions.xml#L305

@marcosbc What happens when i run bncert and there is a update ? Does it just run a update of lego ?

If you run a complete execution of Bncert (to create a new cert or renew an existing one), it will upgrade Lego if the installation's one is outdated, as well as replacing the cron line with a different timestamp.
But as far as the cron job goes, Bncert is not executed here and therefore nothing will be upgraded: Only the Lego tool is called.

Since there haven't been any updates to this issue for a few weeks, I'm proceeding to close it.


Add "SPIRV-Headers" package

I'm interested in package SPIRV-Headers
Here is the official site of package https://www.khronos.org/registry/spir-v

Official GitHub repository: https://github.com/KhronosGroup/SPIRV-Headers

Used build system: CMake

This package has no dependencies

Managed to upgrade the upstream directly:
https://github.com/KhronosGroup/SPIRV-Headers/pull/105
We just need to wait for their next release...


[0.12.0.x] More compatibility for old ipv6 format

Changes:

try old format ipv6 if new format failed to verify
sanitize log strings

good idea, utACK


Search field in navigation bar

Version
Tell us which versions you are using:

react-native-router-flux v4.0.0-beta.24(v3 is not supported)
react-native v0.50.3

I would like to ask on how to put a search field in the navigation bar, can't see any example. Hoping there are some people that already did this.

I haven't tried it but perhaps you could build out your own navbar component and add it to the Scene using the navbar prop, see https://github.com/aksonov/react-native-router-flux/blob/master/docs/API.md#scene

Custom navbar is solution, @rstims is correct


What does "MiniImagenet integrity check failed" mean?

What does "MiniImagenet integrity check failed" mean?

It could be one of two issues:

You didn't include download=True to download the dataset (by default, download=False, following the conventions of Torchvision). It would look something like dataset = miniimagenet(..., meta_train=True, download=True).
If you did include download=True, this means that there was an issue with the dataset conversion. If you have a subfolder miniimagenet in your data folder, then delete it and try to run it again.


Better Local Package Testing

Is your request related to a new offering from AWS?
Is this functionality available in the AWS provider for Terraform? See CHANGELOG.md, too.

No 🛑: please wait to file a request until the functionality is avaialble in the AWS provider
Yes ✅: please list the AWS provider version which introduced this functionality
Yes, v1 or earlier

Is your request related to a problem? Please describe.

I would love to be able to generate a build package locally using the method from package.py so I can peer into its contents during debugging. The 'pattern' regexp used is very difficult to debug and I don't find the debug logs to be as useful as having direct access to the actual zip directory.
Describe the solution you'd like.

Allow devs to run a command that accesses the same functionality behind package.py so we can generate zip files locally for debugging without having to run any terraform commands.
Describe alternatives you've considered.

rerunning terraform apply over and over again to debug.
Additional context

I'd be interested in this also. It would be useful to build the package locally without having to run the apply.

terraform apply should mean "create something".
If you don't want to deploy a Lambda Function, you can always use terraform apply -target=module.lambda_function.null_resource.archive (or something like that).

That's not a bad idea. I rarely use -target and forget it is an option. Thanks Anton!


Global / Application wide settings?

Hi,
Does it exists a way to make global / application wide settings with your  module?
Else can you implement it?
Regards

You are free to use the settings' values however you want.

Sorry for mistake ... I understand that settings was bind to each user. So my question is how to make settings per user?


apollo-router release assets were renamed in release 0.16.0

Describe the bug
The download paths for the release assets changed when the PR to fix this issue merged: https://github.com/apollographql/router/issues/1385
Steps to reproduce
% asdf install apollo-router 1.0.0
* Downloading apollo-router release 1.0.0...
curl: (22) The requested URL returned error: 404
asdf-apollo-router: Could not download https://github.com/apollographql/router/releases/download/v1.0.0/router-1.0.0-x86_64-macos.tar.gz
%

Expected behavior
apollo-router downloads and installs
Screenshots
N/A
Additional context
The changes were:
linux is at https://rover.apollo.dev/tar/router/x86_64-unknown-linux-gnu/v1.0.0
macos is at  https://rover.apollo.dev/tar/router/x86_64-apple-darwin/v1.0.0
windows is at https://rover.apollo.dev/tar/router/x86_64-pc-windows-msvc/v1.0.0

I've fixed it. Please try the latest version with  asdf plugin update apollo-router. Thank you for your reporting!


First pass at creating correctly-oriented polygons.

See #53

@frewsxcv I've had to rename the geo-types feature to geotypes  in order to set up multiple crates as a feature (AFAIK you can only use a crate name as a feature name for a single crate). Lmk if there's a better / non-breaking way to accomplish it…

From the GeoJSON spec:
   o  A linear ring MUST follow the right-hand rule with respect to the
      area it bounds, i.e., exterior rings are counterclockwise, and
      holes are clockwise.

   Note: the [GJ2008] specification did not discuss linear ring winding
   order.  For backwards compatibility, parsers SHOULD NOT reject
   Polygons that do not follow the right-hand rule.

I'm having a little trouble comprehending this, but it to me it sounds like:

When reading GeoJSON, allow clockwise or counterclockwise linear rings
When writing GeoJSON, output counterclockwise exterior linear rings, clockwise interior rings

Does that sound right? 🤔

Also, separate thought, but instead of pulling in the entire geo crate, we do have the ability to just add a [geo-winding-order] crate as a dependency. Just an idea!

Does that sound right? 🤔
Yep, that's my understanding! Good idea on the winding-order crate if it gives us a compilation speedup…

Does that sound right? 🤔

Yep, that's my understanding! Good idea on the winding-order crate if it gives us a compilation speedup…

So considering we'll need to potentially reverse the orientation of linear rings when reading and writing GeoJSON, it seems like we'll need to make a dependency on geo (or geo-winding-order) regardless of whether they have geotypes feature enabled.


Rule proposal - Spacing rule - Check indentation

Stylecop doesn't check the indentation of lines. I think it should.
There is a problem: lambda expressions. How should they be indented? I've seen AT LEAST three styles:
the same indent as line above:
Action a = () =>
{
};

the same indent as line above + 1 additional indent
Action b = () =>
    {
    };

indent with => operator + 1 additional indent
Action c = () =>
                   {
                   };

(technically the third style is used with the "java notation" of curly brackets, and with "MS" Notation it isn't very beautiful):
Action c = () => {
                    };

For lambda expressions the VS by default "suggests" the second solution (if you write => and new line the new line is indented), but if you change the indentation the VS won't reformat that block.
A similar "problem" is with switch/case+curly brackets as for example: switch (t) { case 1: { // } }
In this situation the VS forcibly reformats the curly brackets using the "second style".
Originally posted by xanatos http://stylecop.codeplex.com/workitem/6792
follow up discussion:
andyr wrote Feb 4, 2011 at 11:51 AM
Jason, please comment and I'll fix accordingly.
xanatos wrote Feb 4, 2011 at 1:15 PM
When I say "Stylecop doesn't check the indentation of lines" I mean even the "standard" indentation, for example this is legit:
if (true)
{
func()
}

(where the func() isn't indented)
Everything else in the comment was about two "special" cases (lambda and switch).
andyr wrote Mar 15, 2011 at 5:32 PM
From Jason, "This would be a great new rule to have"
berntie wrote Jan 12, 2013 at 6:30 PM
I think this is a duplicate of http://stylecop.codeplex.com/workitem/7337. Please delete one of the two. If possible, add the votes of the deleted to the remaining item.

IMHO rules for indentation would be nice, especially around "gray area" - formatting lamdas and wrapped arguments. I personally align { for statement lambda to ( in its parameters. This way it looks like embedded method.
On the other hand, I indent wrapped arguments as per "line above + one indent" rule

I'm closing this rule, not because it's a bad idea but because it's too broad and doesn't present a specific rule which can be discussed and later implemented. I've already opened two new rules related to indentation; I encourage you (or anyone) to do the same if you feel strongly about some aspect.


Expense - App returns to main chat after deleting expense when it has replies

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.33-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4953525
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to staging.new.expensify.com
Go to DM that has no unsettled expense.
Submit an expense.
Go to combined report.
Add a reply.
Click on the report header.
Click Delete expense.
Delete the expense.

Expected Result:
App will stay on the same thread because the report has replies (production behavior).
Actual Result:
App returns to main chat after deleting the expense when it has replies.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/603aa3a5-3496-4666-91de-5df86293522b

View all open jobs on GitHub

Production
https://github.com/user-attachments/assets/8ba7f573-7a46-4534-9179-7056aefcb6f6

@nkdengineer do you think this one is related to https://github.com/Expensify/App/issues/49052?

Let me check and testing when implementing the PR.

We're reverting the offending PR.

This is fixed on staging. Closing.


Image constantly resizes on imageView.blur(percentage)

I was trying out your pod today for some simple blur on an image.
Unfortunately I came to some fairly major issues, maybe a little hard to explain but I think i've figured out the issue.
I don't think the imageView is respecting the view contentMode (in my case 'Aspect Fit'), it works perfectly with 'Scale to fill' but will flicker between 'aspect fit' and 'scale to fill' as I adjust the percentage. I've copied your example code for the '0.0 to 1.0 parameter' example, so nothing strange in my code.
Besides this I also came to these issues pretty fast:
Crash when no image in imageView
Crash if not configured with a boxsize first for animation

You must set image and boxsize before committing animations.

hi you, I do follow "SABlurImageView" of you, but I want save image camera roll, then do just how. Thank you 👍

Hello :(


Add save function for statement_prompt

Fixed problem with statement_prompt not being saved in cache when applying language adapter to faithfulness

@miyaketomoya thank you so much for the fix and welcome to the contributors list 🙂 ❤


Sto doc bug fixes doc 3297

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ * ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/STO-6100
https://harness.atlassian.net/browse/DOC-3297

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ * ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Autogenerate a lints database

Observed behaviour
The Rust clippy lints page is awesome.
Expected behaviour
We should have a similar page generated from the rationales and check IDS of fontbakery checks.

Done: https://font-bakery.readthedocs.io/en/stable/fontbakery/profiles/universal.html
But it could be improved, for sure! That Rust page is indeed pretty cool. Maybe we could incorporate some aspects of that in our docs as well.

This is something that bothers me, though: #3313


Use Tesseract's Orientation detection

This PR replaces the orientation correction mechanism with Tesseract's. It resulted in ~60-70% reduction in time taken by the orientation correction step (1.862s -> 756.95ms).
The confidence returned by the Tesseract is based on the number of glyphs detected and can result in extremely low scores (<1) even for correct detections. This is why this PR doesn't have a confidence threshold, as finding a score that works universally is a hard problem.
Before this change, the test resulted in ~63% accuracy. The dataset was randomly rotated to either 0, 90, 180, or 270 degrees, indicated as "randomly rotated" in the table below. This dataset was then used to compare Tesseract OSD and the chunking strategy. Tesseract OSD produced almost perfect results (61%) coming in close to the original Zerox performance of 63% (This 3% step down is discussed later on). On the other hand, the chunking strategy produced 47.10% accuracy (25% down).

Orientation detection mechanism
Dataset
Accuracy

In-house chunking strategy
Zerox test
~63% (baseline)

Tesseract OSD
Zerox test - randomly rotated
~61%

In-house chunking strategy
Zerox test - randomly rotated
~47.10%

This randomly rotated dataset can be found here.
Why the 2% step down?
Well, Tesseract failed to identify orientation for two images with the message "Too few characters. Skipping this page". For one of the image, cropping in to the receipt fixed this problem.
Tesseract has a configuration min_characters_to_try, which was set to 0 but got the same result.
And...
This PR also adds the image correction (trimming and orientation correction) if the image (PNG) is passed, which were bypassed before. This required in change in the temp directory folder structure which is expected to have no user facing impact.
Note:
When running this for the first time, tesseract downloads a file osd.traineddata sized 10.6MB

That is correct: all the tests were run using the npm run test. With an ideal orientation detection system, we'll get the same accuracy on the original test dataset and the randomly rotated (i-e, all pages were corrected accurately). With Tesseract OSD, we are getting a 2% reduction in the accuracy compared to how Zerox performed with the original dataset that had all but one correctly oriented pages.

@tylermaran your question made me realize that it'll be more helpful to know the exact count of misclassifications across both approaches, and if've now added that to the PR's description.


Playfair issue when decoding

When I try to decode a playfair cipher, it tells me: 'ValueError: '__' is not in list' after attempting to run this line: arow, acol = int(self.key.index(a) / 5), self.key.index(a) % 5.
The '__' is whatever the first letter of the ciphertext is. I've tried several different texts and it always gives me that error. Is this user error?

Figured it out. Sorry!


Update the layout to match new default in Phoenix 1.4

Closes #66
I saw this and figured it was pretty easy.
I also noticed that the templates for the phx.gen.html.slime task also did not match the 1.4 defaults. Should this PR include those changes as well?
It looks like the current phx.gen.html.slime templates are using Bootstrap-like classes (i.e. btn and btn-primary)?

@JohnKacz would you be interested in helping with some other @slime-lang things? We'd love to have other contributors around these parts.

@doomspork Sure. I'm not sure how much I'll be able to give but give back here and there.


P12 Certificate File Upload (Build 270)

Hey there,
With the latest 270 build of October I am unable to upload .p12 certificate files for the Google Analytic plugin (version 1.0.3) within it's backend settings page.  The upload widget DropZone seems to be setup to only accept a restricted set of file types such as media or text.
I've tried adding fileTypes: p12 to fields.yaml but the issue seems to run deeper into a nest of validation checks.  I couldn't find anywhere to declare a set of MIME types in order to pass validation.
Thanks

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/22491534-p12-certificate-file-upload-build-270?utm_campaign=plugin&utm_content=tracker%2F1282891&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1282891&utm_medium=issues&utm_source=github).

Fixed in 1.0.4 + Build 271


Update README.md

adds instructions for setup in windows using wsl 2

Cool, thanks for figuring this out and sharing!
I think even if you have developer mode enabled and symlinks working (or if we remove the symlink in the future), this is a very useful guide, e.g. if you want to debug a UNIX error on a Windows machine. Maybe we can create a separate wiki page for it? Then in the README, we could link to that page.

If you asked me:
Wiki or readme, both is fine for me, only I think i am not able to propose a wiki change via PR (can i?) in the main repo. So if you add an article in the wiki, I could modify my proposed change to the readme. Maybe you add a wiki page for "Setup" or "Installation" with chapters for each OS. I guess my originally proposed change to the readme would fit a "Windows" chapter.
If I understood correctly, I can simply add another commit to the feature branch in my repo and it is added to this PR automatically.
I'd change the readme to something like
either
- enable developer settings and use git's symlinks capabilities, see [this stackoverflow answer](https://example.com) or
- use WSL with at least Ubuntu 22.04, see [detailed instructions in wiki](https://example.com)

Where the wiki page link points to the "Windows" Chapter on a wiki page

Yes, exactly. Maybe @bohning can enable wiki edits for non-contributors?

Yes, exactly. Maybe @bohning can enable wiki edits for non-contributors?

Just enabled the Wiki for non-contributors and also added @mjhalwa as a contributor.


docs: add ADR on keycloak authorisation service for multitenancy

Overview

Add an ADR on Keyclaok authorisation service for wallet permission management.
Checklist
My PR contains...

[x] No code changes (changes to documentation, CI, metadata, etc.)
[ ] Bug fixes (non-breaking change which fixes an issue)
[ ] Improvements (misc. changes to existing features)
[ ] Features (non-breaking change which adds functionality)

My changes...

[ ] are breaking changes
[ ] are not breaking changes
[ ] If yes to above: I have updated the documentation accordingly

Documentation

[ ] My changes do not require a change to the project documentation
[ ] My changes require a change to the project documentation
[ ] If yes to above: I have updated the documentation accordingly

Tests

[ ] My changes can not or do not need to be tested
[ ] My changes can and should be tested by unit and/or integration tests
[ ] If yes to above: I have added tests to cover my changes
[ ] If yes to above: I have taken care to cover edge cases in my tests

Hi @patlo-iog - should this supersede our previous ADR on using Keycloak in MT? https://github.com/hyperledger-labs/open-enterprise-agent/blob/main/docs/decisions/20230527-use-keycloak-and-jwt-tokens-for-authentication-and-authorisation-to-facilitate-multitenancy-in-cloud-agent.md

@davidpoltorak-io I see it as an extension to that ADR. It does agree on the JWT claims approach and tries to add more context how exactly the integration will look like based on a small POC we did.

@patlo-iog, good job!
I would elaborate on the considered options as these might not be understandable enough for engineers who are out of context.


Use different stars icon

How can we use different stars icon ?
I'd like for example to use font-awsome classes (fa fa-star etc), instead of ionic ones (ion-ios-star etc)
Is that possible ?

add in your template html

and change the ionic-rating.min.js
find and replace (ion-ios-sta) for ( fa-star)
Works for me!


Invalid code generated when input structure is used as regular function parameter

Paste the following shader into xsc_debugger:
struct VSIN
{
float3 position : POSITION;
};
float3 GetPosition(VSIN i)
{
return i.position;
}
float4 VSMain(VSIN i) : SV_Position
{
return float4(GetPosition(i), 1);
}
and set the entry point to VSMain.
This will generate the following shader:
// GLSL Vertex Shader "VSMain"
// Generated by XShaderCompiler
// 10/01/2017 20:33:34
#version 130
in vec3 position;
vec3 GetPosition(VSIN i)
{
return position;
}
void main()
{
gl_Position = vec4(GetPosition(i), 1);
}
This will obviously not compile.
I tried this because I run into a similar issue when trying to make geometry shaders to work.
The reason I write this as an issue is because I don't know how you want to handle it.
Solution 1: Fall back to what other compilers do and keep the structure:
#version 130
in vec3 position;
struct VSIN
{
vec3 position;
};
vec3 GetPosition(VSIN i)
{
return i.position;
}
void main()
{
VSIN x;
x.position = position;
gl_Position = vec4(GetPosition(x), 1);
}
Solution 2: Propagate the input access to other functions and kill the parameter
#version 130
in vec3 position;
vec3 GetPosition()
{
return position;
}
void main()
{
gl_Position = vec4(GetPosition(), 1);
}
The second solution looks nicer but it needs to remove the parameter of GetPosition()
In case of geometry shaders it is possible to call a function with an array element (the data of just one vertex) in which case the parameter would need to be replaced with just the array index.

I think I will handle this with the ReferenceAnalyzer, which must detect if the struct is used by another function than the main entry point.
This issue is some sort of a relict from the fashion I handled the I/O semantics, when I always used input/output blocks.
Back then the struct was translated into a shader block instead of a struct.
But now the compiler always translates I/O semantics into global in/out members.
I try to find a little time tomorrow to fix this.

It's still incomplete, but the code you posted should be translated correctly now.
The I/O semantic differences between HLSL and GLSL seems to most complex :-/

This issue should be solved now, at least for Vertex- and Fragment shaders.
Geometry and Tessellation shaders are still incomplete.


fix: data printed in console

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ x ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description
I fixed this bug when, just removing that line of code

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ x ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ x ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?
https://user-images.githubusercontent.com/37635083/180580438-d3413229-d86d-4f34-b69d-c99bc9e06b22.mp4

Please add the ticket this fixes in the Related Tickets & Documents section like:
closes #226 or fixes #226


remove wait login in kafka controller

Summary
Related issue(s)
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/test test-integration

/test test-integration

/test test-integration

/cc @clyang82 @yanmxa


County json

Looking for help on how to use us-atlas to create a JSON file that shows a state's counties. I want something like this (http://bl.ocks.org/mbostock/5349951) for Florida. I followed Mike's tutorial for a U.S. map, but I'm stuck at the state level. I'm not sure how to proceed.

Please try Stack Overflow if you require assistance. These issues are for feature requests and bug tracking. Sorry!


CLI generate provider/page issues

Short description of the problem:
In CLI, run the generate command for page and provider.  Does not create the appropriate files under source.
Ionic CLI :          3.1.2
Cordova CLI :    7.0.1
@ionic/cli-utils:  1.1.2
Ionic Framework:  ionic-angular 3.2.1
What behavior are you expecting?
Expect the new page or new provider I am wishing to create will generate the proper folder and files.  Specifically I was ran the following
ionic cordova plugin add cordova-sqlite-storage --save
then proceeded to generate the provider.
Steps to reproduce:

install ionic package
Open CLI
Go to project folder
type: ionic g provider myprovider
enter

insert any relevant code between the above and below backticks

Post the output of ionic info below please
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

You are missing your ionic info output at the bottom of the issue. Please append that.

In CLI, run the generate command for page and provider. Does not create the appropriate files under source.

What does it create then? What exactly happens?

ionic cordova plugin add cordova-sqlite-storage --save

How is this command connected to this?

To your question:  When I run the ionic g provider [name] command the command states completes successfully creating the new provider with [name]!  However, in my source code:  -->src-->providers-->[name folder]  is not generated.
This also occurs when trying to generate a page.  I was successful a few times during the same session to create a few pages, but now it no longer creates pages nor the needed provider.
Wondering if I am missing something or could be a potential issue.  First time developing using ionic.
thank you all in advance
ionic info---
global packages:
@ionic/cli-utils : 1.1.2
Cordova CLI      : 7.0.1
Ionic CLI        : 3.1.2

local packages:
@ionic/app-scripts              : 1.3.7
@ionic/cli-plugin-cordova       : 1.1.2
@ionic/cli-plugin-ionic-angular : 1.1.2
Ionic Framework                 : ionic-angular 3.2.1

System:
Node       : v6.10.3
OS         : Windows 7
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed

Very strange behavior! I'm not able to reproduce this on my end. Is your project in source control? Do you notice any file changes at all?

The project is not in source control.  The only file changes were to app/app.module.ts adding the provider reference.

I suggest you create a local git repo in the project directory and try
again - just to make sure.
2017-05-24 14:02 GMT+02:00 AndrewsDevTeam<EMAIL_ADDRESS>
The project is not in source control. The only file changes were to
app/app.module.ts adding the provider reference.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic-cli/issues/2336#issuecomment-303703458,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALNeU7bXXnqrJtt3vqHa2B-7PKhrghFks5r9BxvgaJpZM4NkEve
.

@AndrewsDevTeam Is this still an issue? I'm trying to clean up this repo's issues.

No

From: dwieeb<EMAIL_ADDRESS>Sent: Thursday, July 12, 2018 4:56:49 PM
To: ionic-team/ionic-cli
Cc: Todd Andrews; Mention
Subject: Re: [ionic-team/ionic-cli] CLI generate provider/page issues (#2336)
@AndrewsDevTeamhttps://github.com/AndrewsDevTeam Is this still an issue? I'm trying to clean up this repo's issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ionic-team/ionic-cli/issues/2336#issuecomment-404648107, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXycnWkc5BH4A9ORJ0-qCvV3JDxcUrfWks5uF7gRgaJpZM4NkEve.


Scoping Issue in CoffeeScript CQL Execution

In some cases, it appears there is a scoping issue in CQL execution -- specifically around aliases.  For example, if an expression uses an alias C in a query, and that query invokes a function that also uses an alias C for something different, one will overwrite the other.
I don't have a specific test case on hand at the moment, but did run into this on some internal CQL-related work.  Will try to create a test case (and maybe even a fix) when I have a moment.

There was a translator issue with this that was recently resolved, #126. There is a unit test that verifies it here

Ah, I remember that.  I'm not positive it's the same issue (there may really be a bug in CQL exec), but I'll try compiling w/ the new translator first to be sure.

This issue was moved to cqframework/cql-execution#46


add imagePullSecret

add imagePullSecret for algorithm-queue, worker, pipeline-driver, and tensorboard pods

This change is

/deploy

/deploy


Is there a way to target a variables.yml in a specific branch?

is there a way to specify the branch where the variable file is using the repo url? Currently it defaults to main.

 Let me take a quick look at it, something like this: org/repo/path/to/file@<ref> where ref is the branch/tag/sha

And if no ref is specified fallback to the default branch

@mattcowen https://github.com/lindluni/actions-variable-groups/pull/2/files
I released it as v2.0.0

good job, works perfectly


Restart device if no WiFi is available for 60 secs

Improved naming and logs from #154. Issue see #155.

See discussion #154
Thanks @dandjo


Codemirror languageserver

merge features added to the language server client so far.

checks failed due to incorrect build comand being used by netlify, should still be ok to merge


CORE-5081: Upgrade to Kotlin 1.7.0.

Kotlin 1.7.0 has been released.

We probably need to perform this sort of update of the following projects as well:
https://github.com/corda/corda-api
https://github.com/corda/corda-gradle-plugins
https://github.com/corda/corda-cli-plugin-host

We probably need to perform this sort of update of the following projects as well.

I am already updating corda-api and corda-runtime-os. However, any Corda Gradle plugin written in Kotlin will use Gradle's own version of Kotlin and not Corda's.

Thank you @chrisr3
As for upgrading https://github.com/corda/corda-cli-plugin-host project - @pmck91 could you raise a Jira for that so that we could prioritize?

This PR is intended to be a precursor to corda-runtime-os#1426.


Update documentation

Collecting ideas for updating documentation around node-mapnik after merging #605 and closing #604.

[ ] upgrade documentationjs to the newest version (currently at 4.0.0-beta)
[ ] generate mapnik theme starting with documentation-theme-default
[ ] research auto pushing to gh-pages if the /documentation folder has been updated
[ ] incorporate more guides and sample code (major update to https://github.com/mapnik/node-mapnik-sample-code)
[ ] "fromStringSync" is listed twice instead of "fromString" for the async function

:+1:

Not sure if this is the right place to leave this, but I noticed that you have "fromStringSync" listed twice instead of "fromString" for the async function.
https://github.com/mapnik/node-mapnik/blob/master/documentation/3.5/index.html#L5734
https://github.com/mapnik/node-mapnik/blob/master/documentation/3.5/index.html#L474
I originally noticed it here: http://mapnik.org/node-mapnik/documentation/3.5/#mapnik-map/fromstringsync

good eye @syork! That's undoubtedly a miss on my part. Added to the list of TODOs

Heya @syork the fromString and fromStringSync options have been fixed. Thanks for reporting :smile:
http://mapnik.org/node-mapnik/documentation/3.5/#mapnik-map/fromstring

Per discussion with @springmeyer, we are going to look into pulling Node Mapnik documentation generation into a separate repository.
The goals:

declutter the Node Mapnik repository (just source code)
make generating documentation much more simple. A user would still update JSDoc comments in the /src directory files, but that should be it.
give space for writing Node Mapnik guides using jekyll, which would add more cruft to the Node Mapnik source code repo, but this way we keep it all separated
move mapnik/node-mapnik-theme into a documentation repository, rather than a separate repo, which makes theme development simpler
ponder how we can more effectively update old versions of documentation (probably phase 2)

This would involve:

hooking into commits on mapnik/node-mapnik's master branch using the travis API based on either commit messages (i.e. message [docs]) or checking the git diff for any changes to JSDoc lines
once the above is detected, trigger another repository (something like mapnik/node-mapnik-docs) to build documentation and commit to its gh-pages branch.
move node-mapnik-theme into new repository

Downsides:

Abstraction can lead to confusion, but hopefully with a nicely updated CONTRIBUTING.md we can make this process clear
Domain name stability - if we move the documentation page to mapnik/node-mapnik-docs we lose the index.html file in the main Node Mapnik repository, which means we may be unable to host docs at mapnik.org/node-mapnik/documentation and it would instead exist at mapnik.org/node-mapnik-docs. I'm not sure if this is :100: true, so will look into customizing the subpage mapnik.org/node-mapnik to point to node-mapnik-docs.

cc/ @springmeyer @artemp @flippmoke @jakepruitt

:+1 The next steps of This would involve: are :100:
As far as domain name stability - I'm not too worried about it changing as long as you are happy with the final url name. Jekyll redirects could be used in a pinch to keep old links working if things change.

Closing this issue down - now that we are auto generating docs!


⚠ Secret Website 3 has degraded performance

In e06728e, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2065 ms

Resolved: Secret Website 3 performance has improved in b60c1da.


sql schema of pyBiocFileCache and r-BiocFileCache are not compatible

Hi,
I found one anther issue. Unfortunately, schemas of Python and R packages are not compatible due to different schema.
Python:

R:

The differences are only VARCHAR/TEXT
Could you please look into this issue?
Thanks!

Thank you so much for the contribution. Its been a while since I looked at this particular package to check if the schema has been migrated! This is merged and let me know if you run into any issues! Thanks again!

v0.4.0 of the package contains these changes.


Eufy Security Integration and Live stream

Good Morning,
I took some time to get the cameras integrated, but after I integrated them and after 1 week, they still don't update often. I need to reload the integration myself. when I do it, it updates for the last event, but I can never open the camera to see it live.
Upon integration I realized that I would have to set the cameras as US, as I was not able to use UK, which is where I live.
Can anyone help me with what I should do to have live stream when I open the card, and also for the integration to update more frequently?
thanks

Also interested as I cannot get live stream to work.

I used GB instead of UK and it accepted it.

https://github.com/fuatakgun/eufy_security/issues/592


Revert "chore(upgrade): update gatsby to v2 (#1126)"

This reverts commit 2f71d9d1f27e50a852ad20e59195fa0e65d8f4e4.

Deploy preview for pf-next ready!
Built with commit 3bed4c1d59431329701b338242472107ebf9505b
https://deploy-preview-1168--pf-next.netlify.com

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update Instructions for uploading user reference data in RDM UI

Will be done after milestone 1 meeting

To be reviewed by @boog-wur and @juanlaso-iiasa .

To do a review I need credentials. Who can provide me with the credentials?

Please register here

@santoshkaranam : I am trying to upload a geoparquet file (QGIS saves it as parquet). But the system does not allow this type

@Santosh: and how can I remove such entry? I am now forced to go back and enter the metadata again

@boog-wur  as per spec, the geoparqet file should end with .geoparquet  not jus .parquet , hence the error. Could you rename the file and try.
Geoparquet file is bit different than a parquet file as it has additional geometry column metadata. This data is mandatory to be present in the parquet file.

@santoshkaranam, would be nice to also allow .gpkg files for user dataset upload

I did a first review of the upload of data by users. See https://vitoresearch.sharepoint.com/:w:/r/sites/21717-ccn-world-cereal/Shared Documents/System/RDM/Evaluate_upload_private_user_data_set.docx?d=w5eb4ab24c2e64ecd8561f4d73478e624&csf=1&web=1&e=DyYIzw
@jdegerickx : could you check my proposed changes before your holidays?
@santoshkaranam : after Jeroen D. additions/comments we could go through the doc and discuss the improvements e.g. during the maps4gpp scheduled for 6 August

@jdries @boog-wur i have added support to .gpkg files and .parquet files.

@boog-wur, thanks for the review.
I have added my comments to the file directly.

@jdegerickx / @santoshkaranam : I see that you want to avoid the term "column". I wanted to do the same but then I saw that the whole UI uses the word column. So should we change column to attribute everywhere in the upolad UI (text, headers)?

I personally prefer "attribute", but that is maybe less intuitive for a user, I don't know. We can also keep using column. We can always adjust later based on user feedback...

Yes attribute is more general and less technical. But Santosh has to change it in many places. Let's hear from Santosh. If too much work we might keep column for now.

@boog-wur  @jdries I can change the names, it is not a problem for me to change in Website.  Shall I change Column--> Attribute ?

Looks good, let's discuss other proposed changes when I am back from holidays thus Tuesday 6 August

Today Santosh and I discussed the review done by Jeroen Degerickx and myself. See Word document (link is in one of previous comments). Santosh will process the changes. Some improvements will be done later (next version). These are:

support more co-ordinate systems
automated generation of data set name / collection id by only asking identifier/key word
ai-assisted mapping
add sampling

@boog-wur I have updated the website with version1 changes as discussed.

@santoshkaranam:  thanks, I still have a few issues. Could we meet tomorrow morning?

@boog-wur  sure

Today we discussed what issues still can be included in the first version of the upload. Before October 2024 we aim to include:

Automated generation of data set name / collection id by only asking identifier/key word
Assisted mapping within the UI
Add sampling
If ready before October we can include these improvements in the MOOC and other dissemination events

On  assisted mapping within the UI the following. There are two options:

AI assisted mapping which works good but depends on paid API service.
Solution based on google translate and egular expression and is free.

We might consider to first use option 2 as it is free and has no dependency on a third party. And we prefer to do the mapping within the GUI thus without the csv down and upload.
The issue to support more co-ordinate systems will be done later.

Concerning "Automated generation of data set name / collection id by only asking identifier/key word": according Santosh it is not easy to generate the datasetid from different elements. At this moment of the upload we do not know the different elements of the desired dataset id name. So we have to ask the user to come with a proposal, preferably based on our convention (available in separate document). In the editor we can show, in grey font, an example to support the user.
Note that the UI assisted mapping is ready and can be tested

added help text for ttitle, datasetId inputs.


removing email

Kindly improve this project by not publishing email addresses in clear text in code on github.

kinda useful ngl


How to use woth Source type : Library

Previous library https://github.com/mmackh/MAImagePickerController-of-InstaPDF has an option to select image from library and make editions . But present is only based for camera. Any insight into this ?

This library does not cover selecting an image. This needs to be implemented separately.


CXXCBC-128: fix build on Windows

I fixed build script (restored old bash script, and renamed portable version to .rb). Your fixes pulled into #143


Windows 11 won't complete startup

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
Tried to start computer
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 8/6/2023 2:06:17 PM
Exception:
System.IO.FileNotFoundException: Could not find file 'C:\Program Files\PowerToys\modules\launcher\Images\app.dark.png'.
File name: 'C:\Program Files\PowerToys\modules\launcher\Images\app.dark.png'
at Microsoft.Win32.SafeHandles.SafeFileHandle.CreateFile(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options)
at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable1 unixCreateMode)    at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable1 unixCreateMode)
at System.IO.Strategies.FileStreamHelpers.ChooseStrategyCore(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at System.Windows.Media.Imaging.BitmapDecoder.SetupDecoderFromUriOrStream(Uri uri, Stream stream, BitmapCacheOption cacheOption, Guid& clsId, Boolean& isOriginalWritable, Stream& uriStream, UnmanagedMemoryStream& unmanagedMemoryStream, SafeFileHandle& safeFilehandle)
at System.Windows.Media.Imaging.BitmapDecoder.CreateFromUriOrStream(Uri baseUri, Uri uri, Stream stream, BitmapCreateOptions createOptions, BitmapCacheOption cacheOption, RequestCachePolicy uriCachePolicy, Boolean insertInDecoderCache)
at System.Windows.Media.Imaging.BitmapImage.FinalizeCreation()
at Wox.Infrastructure.Image.ImageLoader.Initialize(Theme theme)
at PowerLauncher.App.<>c__DisplayClass19_0.b__0()
at Wox.Infrastructure.Stopwatch.Normal(String message, Action action)
at PowerLauncher.App.OnStartup(Object sender, StartupEventArgs e)
at System.Windows.Application.<.ctor>b__1_0(Object unused)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
✔ Expected Behavior
Windows to run
❌ Actual Behavior
White background, Task Manager works via key strokes, No taskbar or start menu. Shortcut keys do not work
Other Software
No response

Fixed in latest release.
Please update PowerToys to 0.72.0: https://github.com/microsoft/PowerToys/releases/tag/v0.72.0
/needfeedback

Thanks. But my problem is I cannot get Windows 11 running properly to uninstall and install. I have a completely white screen. I cannot get the start menu, file explorer, or taskbar to appear. I did get Microsoft Edge running from the GiThub error message. And Task Manager (Ctrl-alt-delete). But Windows shortcuts to start Run do not work. Nor other Win key shortcuts. I ran System Restore for before latest PowerToy install and it made no difference. I am hosed

That PowerToys error have been experienced by many users and has been fixed in the latest version.
How do you relate the startup issue to PowerToys? Is the issue started after installing PowerToys?

/needinfo

That was my impression from the Power Toys error message above. Could be wrong, but it popped up on my white screen.

Please update to the latest version.


Audit use of Seq and try to make it less restrictive.

As discussed on the sync call, we want to audit the Seq interface, as it currently contains a lot of methods that make extra assumptions about what kind of sequence is being dealt with, whether it has a fixed size, whether the underlying pointer can be resized, etc.
In the upcoming value-dependent type changes, we will have a Vector type with a fixed size that won't be compatible with Seq as it currently stands.
I would suggest reducing the number of methods in Seq and possibly adding other interfaces to cover Seqs that have a size that isn't fixed, and the ability to reserve more space in the pointed-to-buffer.

Is there anything this should be tagged as?

Thanks, I added some tags.  This is ready for someone to pick up if they want to do an audit.

Just an audit of usage in the standard library? Or anything else?

I'd say just do the standard library.  If there are conflicts between the proposed new interfaces and users existing code, hopefully those will come to light in the RFC process.

@jemc: Sounds good, here's an overview. I can do something more in depth in a day or so to see where and how the various returned Seq instances are used.

Implemented explicitly by:

String
Array
List Includes the following "dummy" methods to satisfy Seq:
create
reserve

Used as return in interfaces (other than Seq):

promises.ReadlineNotify.tab()

Used as return (or constraint on return) in concrete types:

encode.Base64.encode_url()
encode.Base64.encode()
encode.Base64.decode()
encode.Base64.decode_url()
regex.Match.apply()
regex.Match.find()
regex.Match.replace()

Used as parameter (or constraint on parameter) in concrete types:

collections.Sort._sort() utilizes:

Seq.apply

collections.Sort._swap()utilizes:

Seq.apply
Seq.update

@Perelandric thanks for the first pass audit. have you had a chance to dig in any further?

@SeanTAllen My attempt so far is probably naive. I've merely searched for explicit uses of Seq in the standard library, and noted those above. At this point I think I'd need a little direction if more data is needed.

Next step would be for someone to confirm which methods of Seq match Vector in ponyta and draw up an RFC that reduces the surface of Seq to those methods, and creates an additional interface that extends Seq to cover the other (resize/reserve-related) methods that Seq currently has.  Some methods could be removed from the interfaces entirely if they are not used or are not general enough.

@Perelandric do you want to continue on with this or should I reach out to the dev list and see if anyone wants to take it on?

@SeanTAllen Please put it out on the list. Thanks.

@Perelandric email sent

@jemc can you write up an issue on the RFC repo requesting the RFC and we can close this one out. I'm working on other RFCs now.

Closed in favor of https://github.com/ponylang/rfcs/issues/61.


[BUG] Sha Integrity Check fails for listr-update-renderer

Description
Currently, lint-staged depends on a forked version of listr-update-renderer, which is pointing at a branch owned by you. When we went to deploy after merging the latest version of lint-staged, we got integrity errors installing this version of listr-update-renderer. My hunch is that it was a version mismatch, pointing the v0.4.0 sha of listr-update-renderer at the GitHub branch version.
My initial suggestion was going to be to lock it to a specific commit hash. Then at least when you commit to that branch, we won't get integrity errors related to those changes. That said, I'm not entirely sure what the underlying issue was. Hopefully you get that fix merged upstream soon :).
Environment

OS: Heroku
Node.js: 10.4.0
lint-staged: v8.0.0

I’m not sure how I should resolve this one. Suggestions?

My initial suggestion was going to be to lock it to a specific commit hash.

Beyond that, I'm not sure. Maybe also bumping the version in package.json in that branch? The underlying cause of the problem isn't entirely clear to me.

We hit this too.  I ended up having to clear my npm cache locally, then npm i to update the package-lock.  However it broke again once 8.0.1-8.0.3 were published so I had to re-run npm i to update the package-lock again to fix it.

I'm afraid I can't do much about it until the PR with this change is going to be merged into the upstream repo. Alternatively I could publish my own package, of course. Ideas?

You could but I think you'd have to publish both listr and listr-update-renderer since the latter is a tertiary dependency.

This is currently causing npm audit to fail for Node 10.13.0 and npm 6.4.1 for any project that uses lint-staged as a devDependency.

I'm afraid I can't do much about it until the PR with this change is going to be merged into the upstream repo. Alternatively I could publish my own package, of course. Ideas?

Do you mean this PR?  It looks like it failed to build, which is probably why it's not been looked at.

Closing in favor of #533


Add format validation when loading a run (#34)

Fixes the issue of malformed runs.

That is a great idea, @ammsa, thanks for your contribution


[core v2] prepare for GA release

This PR updates core v2 libraries' version number to 1.0.0 and update the release date in CHANGELOG files.

🍾


Compile libusb on Windows via CMake.

This should avoid issue #12 and also allows us to move to non-released
libusb versions.

Any opinions @jonasdn or @ataffanel? Would be nice to get this repo's CI green again:-)


[feat] pages about "Play With Voxel"

[ ] Create Voxels!
[x] Upload Voxel File to Arweave by Permaweb Dropper on Arweave
[ ] Voxel to HTML by Github-pages Using Template
[x] Mint Voxel as an NFT!
[ ] Make Voxel NFT from Virtual to Actual One by 3D Print!

I will write "Voxel to HTML by Github-pages Using Template".

Using Github Pages to serve Voxel as HTML by using template
In this articles, we will show how to use Github pages to serve Voxel as HTML by using template. This HTML file will be used to mint our NFT.
In the previous article, we have upload the voxel file to Arweave with Permaweb Dropper. And we got the permanent link of the voxel file, for example https://5ywn6daenz6poefpjkgs3c2vdgtzsk2vevam6aozkfzzzwrmgi.arweave.net/nFF3J4VB0BP9eHt35gdffGuwCXaHfXwtJl2K6u4GyA0. Now Let's use Github pages to serve the voxel file as HTML.
Clone the repository
First, we can navigate to the WeLightProject/voxel-viewer-page project. This is a template repository. We will use this template to create our own project.

Click the "use this template" button, and you will see the following page. We need to fill the repository name and description. Then just click the "Create repository from template" button. Then the repository will be created in a few seconds.

Replace the voxel link and commit
Second, we need to clone the repository. We can click the "Clone" button. Then wen can copy the link of the repository. For example, https://github.com/qiwihui/MyDeerVoxel.git.

Then we can open a terminal and run the following command to clone the repository.
git clone https://github.com/qiwihui/MyDeerVoxel.git
cd MyDeerVoxel

We need to replace the voxel link in the handler.js file. We can open the handler.js file in your IDE, and in the 19th line, we need to replace the link in src tag to the permanent link of the voxel file, which is https://5ywn6daenz6poefpjkgs3c2vdgtzsk2vevam6aozkfzzzwrmgi.arweave.net/nFF3J4VB0BP9eHt35gdffGuwCXaHfXwtJl2K6u4GyA0.

Then we need to commit the changes and push it to Github.  We navigate to repository url and click Settings. Then we can click the Pages tab. We should choose the main branch. And then click Save.

The Github Pages will deploy your project automatically and you can see the website link when it is ready. For example, https://qiwihui.github.io/MyDeerVoxel.

We can now open the website in your browser. The website will show the voxel file.

Cool! Now we can mint our NFT using deployed url. We will show you in the next article.


RFC: switch Zip compression level from 6 to 4 by default, and add attribute to control it

Kinda related to #1002 but this one does not add any new libraries or compressors.
Change Zip compression level from default (6) to 4. In my test of 18 various exr files (total raw data size 1057MB):

zip 6: 2.452x ratio, 206MB/s compression
zip 4: 2.421x ratio, 437MB/s compression

So a tiny drop of compression ratio, but compression is more than 2x faster. This makes writing zip faster than writing uncompressed (386MB/s). Decompression speed unaffected.
Additionally, added the zipCompressionLevel header attribute to control the compression level.
Places I have not modified:

DWA compression (uses a mix of 6 and 9 levels today),
Pxr compression (uses level 6),
The new zip code path in OpenEXRCore I have switched to level 4, but did not make it pay attention to the zipCompressionLevel, partially because I could not quickly find out how to do that :)

A longer writeup of all this, with data graphs: https://aras-p.info/blog/2021/08/05/EXR-Zip-compression-levels/

Looks like ImfStandardAttributes.h is missing #include "ImfIntAttribute.h", which leads to the clang link error.

This is a very exciting change! The prospect of doubling write speed with a few lines of code change, no user intervention, and barely any detectable change in compression ratio is crazy awesome.

This does look promising. Could I suggest running these performance comparison tests on the standard OpenEXR test images as well as your larger test set so there's a more self-contained reproducible test?
https://github.com/AcademySoftwareFoundation/openexr-images/tree/master/ScanLines
https://github.com/AcademySoftwareFoundation/openexr-images/tree/master/Tiles
Also, somewhat pedantic point that might need clarification in comments: the presence of a zipCompressionLevel attribute in a file won't guarantee the file was compressed with that level of compression, or even that it was written with ZIP compression. A ZIP file might be read in and written out again as PIZ  preserving all metadata, so the zipCompressionLevel attribute is preserved but ignored. Equally an older version of the library might read in a ZIP file and write it back out at again. The old library won't interpret the zipCompressionLevel and will use the old default value, so there will be an unexpected change in file size.
That behavior seems fine to me, but it might cause confusion.

Could I suggest running these performance comparison tests on the standard OpenEXR test images as well as your larger test set so there's a more self-contained reproducible test?

Here's writing performance on the 11 files (76MB raw pixel data) from openexr-images ScanLines and Tiles folders:

The files are much smaller, and I guess that's why the write performance is lower overall, compared to the larger image set.

Uncompressed: 1.000x ratio, 343MB/s
Zip 6: 2.063x ratio, 166MB/s
Zip 4: 2.035x ratio, 290MB/s

So here changing Zip level from 6 to 4 does not quite make it 2x faster, but still a good speed up, for a really small ratio decrease.

Also, somewhat pedantic point that might need clarification in comments: the presence of a zipCompressionLevel attribute in a file won't guarantee the file was compressed with that level of compression, or even that it was written with ZIP compression

Right. Then the same gotcha should arguably be clarified for the already existing dwaCompressionLevel attribute. To me, using header attributes to pass compression level settings does feel a bit strange, TBH -- this is purely data for the compression process; the decompression does not need it at all, and one could argue that it should not be stored into the file header. But the API does not allow passing data to the compressor in any other way, that's why I guess Dwa passes it through a header attribute. I did the same for consistency.

Thanks for that graph: that's still a sizeable chunk of performance gain.
Yes, good point about dwaCompressionLevel.  Indeed, dwaCompressionLevel actually doesn't really tell you much at all. It controls how lossy the data was the last time the data was written, but doesn't tell you if an earlier step introduced more compression error. To understand how much compression error there is in an image you need to know the entire processing chain.
However, dwaCompressionLevel has been in OpenEXR since DWA compression was introduced, so no versions of the library that wrote DWA ignored the attribute. I do think it's worth clarifying that zipCompressionLevel is new so might have been ignored by the version of the OpenEXR library that compressed the data.

However, dwaCompressionLevel has been in OpenEXR since DWA compression was introduced, so no versions of the library that wrote DWA ignored the attribute

Good point, haven't thought of that! Something like https://github.com/AcademySoftwareFoundation/openexr/pull/1125/commits/f4036be1c then perhaps?

Do we really want the compression level as persistent attributes? as part of ImfHeader, sure, but the compression level doesn't seem right to have in the attribute list. I have identical feelings about both dwaCompressionLevel and zipCompressionLevel - both seems like something we should remove from the attribute list, and have the ImfHeader have an ephemeral list of options to control the compression (we can re-use the attribute type system for genericity)?

Do we really want the compression level as persistent attributes? as part
of ImfHeader, sure, but the compression level doesn't seem right to have in
the attribute list.

For historical purposes, dwaCompressionLevel seemed like the least-bad
option at the time, as there wasn’t really a mechanism to signal details to
the compressor otherwise.
Having something non-persistant seems like a good idea.

Do we really want the compression level as persistent attributes

It does feel a bit wrong to me too, but here I was just following what DWA compression did. Some sort of "non-persistent options" thing would feel better indeed, but that's perhaps a larger scope task/effort/decision.
It still feels to be that changing zlib level from current 6 to 4 (even without any attributes to control it) would be worth it. Compression gets 2x faster, at a tiny loss of ratio.

We discussed this at length at the last TSC meeting (which you are very welcome to join, they are open zoom calls).
We're very excited about these optimizations and wish to implement them, and 100% agree that doubling speed with just a tiny change in compression ratio is definitely the right thing to do.
We thought that just changing the level internally with no runtime control was sort of the lazy way out, and we preferred to make it externally controllable so that (a) we can more easily test across a range of images to verify that we're replicating your results before changing the default for everybody, (b) others can try all of the compression levels without altering or recompiling the source, as different users and their cases may have different preferences about what compression vs speed is the right trade-off, (c) we feared that there might be users who require bit-exact files, not just bit-exact pixels (for testing/validation replicability?) and so an option to keep the old behavior may ease the transition for them, even if they agree that 4 should be the default moving forward.

yes, yes, my comments are only about the mechanism, not that we should avoid the change :)
I will try to prototype an api to add to ImfHeader and exr_context that we can comment on and once happy we can merge the efforts. Thanks again for bringing this up!

@aras-p Do you recall whether your tests results posted here and on your blog here were using multithreading or not? I don't see any reference to the thread count. And also not 100% sure if you were doing the full I/O, or doing some trick like writing to /dev/null to minimize I/O overhead in the timing.
I tried doing an analogous change in OpenImageIO, but for TIFF files, and got good results, though not as good. I tested with 100 images randomly selected from the Animal Logic Lab USD scene (they are almost all 4k^2, a mix of 1- and 3-channel images), converted from half to uint16 TIFF images. Going from zip level 6 to level 4 resulted in 3% bigger files, and around 40% faster write times.
It's not apples-to-apples versus your tests... uint16 vs half, TIFF vs OpenEXR (different overheads), the texture files I used may have different compression characteristics than the set of images you used. And I was benchmarking using a real image manipulation program, not merely a test harness, so there was probably other overhead mixed in. Oh, and you said y our files were all 4 channel RGBA (and the thumbnails you showed looked like maybe A=1.0 everywhere), versus mine that were a mix of 1- and 3-channel images, so dunno if that also may have been an important difference.
But another thing that occurred to me is threading -- I was using 32 threads when I tested, because I mostly care about performance on our typical production workstations. But writing files consists of both compression (fairly well parallelized by threads), and the actual I/O (serialized). So I was thinking that if your tests were single threaded, or had a low thread count, the improvement in compression time would appear larger to you because it would have represented a larger portion of your total runtime.
Switching the default and a ~3% larger disk space still seems like a more than fair tradeoff for a 40% improvement in write speed, but I was curious to poke a bit more about why I was seeing less than the 2x you reported.

@aras-p After looking at your previous blog post and comparing the charts, I think I figured out that you are reporting results with 16 threads?

It would be nice to have similar controls for dwaa/dwab lossless channels which use zip compression.

It would help if I read the code before commenting.  Ignore me, this is already done in the request.

After looking at your previous blog post and comparing the charts, I think I figured out that you are reporting results with 16 threads?

@lgritz yes, on a 2019 MacBookPro with 16 hardware threads. My guess for why my numbers are slightly different: fewer threads in my case, so CPU time is more important; and Macs tend to have really fast SSDs, so IO time is less important.

Thanks!


⚠ mainnet - Auth API has degraded performance

In bc58d20, mainnet - Auth API (https://auth-api.pro.cf-explorer-mainnet.eu-west-1.beta.explorer.cardano.org/api/v1/user/info) experienced degraded performance:

HTTP code: 504
Response time: 10692 ms

Resolved: mainnet - Auth API performance has improved in 23e6758 after 3 minutes.


⬆  (deps<EMAIL_ADDRESS>+ 🔺 next.config.mjs

Migration to plaiceholder@3

[x] Move to ESM: next.config.mjs
[x] Create getImage within @jeromefitz/shared
[x] Migrate to use getImage instead of calling plaiceholder directly
[x] Allow for us to redo getImage on OVERRIDE_CACHE (not necessary)
[x] Remove @plaiceholder/tailwindcss, glyphhanger (wh00ps, from inter4)
[x] Remove unused types
[x] Fix Hydration error introduced by ListColumn

This pull request has been deployed to Vercel.

Latest commit:
ae98351

✅ Preview:
https://jeromefitzgerald-rqkj5ikw6-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/7LtxxUYaVPQ5SFNk9NnLbsDwP2DL

View Workflow Logs

This pull request has been deployed to Vercel.

Latest commit:
ab1ccd8

✅ Preview:
https://jeromefitzgerald-mp3rf4oj4-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/3Rt1aGZ9QuKuFdqtJZDmWs7ebmYd

View Workflow Logs

This pull request has been deployed to Vercel.

Latest commit:
1811fcd

✅ Preview:
https://jeromefitzgerald-m21srco5t-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/GJBeznLYe4AJ2JQAcntKWhAAaUHM

View Workflow Logs


bug(mat-form-field): unwanted border appears after mat-label.

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
v14
Description
When form field appearance is set to outline unwanted border appears at the end of mat-label. This happens only when field is focused.
I set the border color to red so you can see which element is at fault. By default it takes the primary color (screenshot 3).

Reproduction
The interesting this is I'm not sure why this happens because it works correctly in the material docs.
Expected Behavior
No unwanted side-border
Actual Behavior
Unwanted border is present
Environment
Angular CLI: 16.1.4
Node: 18.14.0
Package Manager: npm 9.4.2
OS: win32 x64
Angular: 16.1.4
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, material, platform-browser, platform-browser-dynamic
... router
Package                         Version
@angular-devkit/architect       0.1601.4
@angular-devkit/build-angular   16.1.4
@angular-devkit/core            16.1.4
@angular-devkit/schematics      16.1.4
@schematics/angular             16.1.4
rxjs                            7.8.1
typescript                      5.1.6

Possibly related: https://github.com/angular/components/issues/26102

@vbraun , indeed it is 👍🏼


Non Partitionable sequence

Hi Markus,
I have this error when i want convert large file

Thanks in advance

This happens if there are a large number of duplicates, see #427. In this case it seems like there are 42,498,386 duplicate points somewhere at the specified min/max coordinates. Could you try to confirm this?
If possible, send me a las file from around that region, e.g. by clipping with
las2las.exe -i file.las -o clipped.las -keep_xyz 1652175.0 8188675.0 51.4757 1652185.0 8188685 51.4908

ok, but i have 300 files, how i can do that ?

test.zip
here is the las2las on the 300 files.
Indeed a lot of the same points, in the same place.
How can we deal with this?

I will work on automatically dropping duplicates if there are too many of them, but it will take a week or two.

@m-schuetz That sounds great, but how did the old converter handle this? Just curious. I imagine it wrote the duplicates to each file, leaving them for Potree JS to handle.
Also, say someone runs a precision clamp algorithm to limit artifacts in WebGL or maximize compression (similar to Draco compression for 3D Tiles). It would be possible for duplicate point locations to exist with different attributes like color. I don't know under what conditions this may happen naturally, but I think it's worth consideration. What do you think would be the best way to handle identical locations with mismatched data?

I will work on automatically dropping duplicates if there are too many of them, but it will take a week or two.

Hi !
Some news about this error ?

See https://github.com/potree/PotreeConverter/issues/438#issuecomment-689477023 and http://potree.org/temporary/PotreeConverter_drop_duplicates_2020.09.09.zip
Please give it a try and let me know if it works.


Expand icon is lost when deleting a line using git and collapse the code.

VSCode Version: Code 1.3.0 (e724f269ded347b49fcf1657fc576399354e6703, 2016-07-07T16:56:12.476Z)
OS Version: Windows_NT ia32 10.0.10240

Steps to Reproduce:
1.Create a function with some lines inside. Git will tell you you've added these lines.
2.Sumbit the source to repository.
2.1.Collapse the created function and you'll see the expand icon.
3.Now remove any inside line from created function. Git will tell you you've removed a line.
4.Collapse the function. Notice how instead of the expand icon it is shown the git red triangle from removed line.

duplicate of #9222


feat: [POC] Return AssistantResponse in DefineForm endpoint

Return AssistantResponse in DefineForm endpoint
Description
Added a new field in the response called AssistantResponse that returns a human readable response from the AI Assistant. The response's Definition field is now optional as not every prompt makes changes to the definition.

Yes, using structured responses is the way it worked initially with ChatCompletion API (as in the examples).
Now we use Assistants API in order to support chat continuation with chat threads. Unfortunately providing the JSON schema stopped working with Assistants API with an error "Invalid parameter: 'response_format' of type 'json_schema' is not supported with model version gpt-4o."
This is a bit strange because exactly the same schema used to work with 4o model before but let's not forget Assistants API is experimental so it could have certain issues with specific models.
We now achieve the same result in practice as the response format is set as JSON object, having the response description inside. We only lack the defined schema.
In the long term it will be good to have it but with the current available options, this is the way to achieve all we wish.


Hide upgrade button, upgrade to premium menu entry and ad placeholder

yes please

If @JulienMaille is okay with it I could probably add an config option for this.

If @JulienMaille is okay with it I could probably add an config option for this.

give the green light @JulienMaille

Isn't this against the TOS?

We're technically just hiding those elements but not deleting them entirely from the app so I don't see why it should be against the ToS. Many other themes hide them as well.
Although, I'm now a lawyer so I can't say with 100% certainty that it's against the ToS or not.

paste this code at the end of the dribblish user.css file
/* Remove upgrade button*/
.main-topBar-UpgradeButton {
display: none
}
/* Remove upgrade to premium button in user menu /
.main-contextMenu-menuItemButton[href="https://www.spotify.com/premium/"] {
display: none
}
/ Remove ad placeholder in main screen */
.main-leaderboardComponent-container {
display: none
}
++ spicetify apply
++The upgrade button should be gone by now.
++Then reinstall blockthespot

You can use this as an extension to the dribbblish theme.
just copy it into the spicetify extensions folder and run
spicetify config extensions dribbblish-no-ads.js
spicetify apply

If Julien worries that blocking these elements is against TOS, then this is the best option, since it is not an "official" part of the theme.

@WelkinBunbury I've sent this PR for the same thing.
@SendNukez That would be pretty good but if it were to break the TOS of Spotify, it would still break it anyway. Also, it seems like extra work for the user to add another extra extension.

@SendNukez That would be pretty good but if it were to break the TOS of Spotify, it would still break it anyway. Also, it seems like extra work for the user to add another extra extension.

Yes, it would break TOS, but because the extension is not part of the main theme it shouldn't cause legal issues or whatever.
And if Julien is okay with it I can easily add this to the main theme, but that is not my call.

Don't bother creating a separate js file

So you're okay with including this in the theme?

Thank you for this PR!


Revert old link-validator.conf accidentally deleted in #241

Refs #216

Yep, also getting some output this time. There's also an overflow error, so not sure if everything is tested comprehensively but let's assume for now.


Make launcher.sh generated by js_binary more self-documenting.

I often find launcher.sh files are the entry point to some failed command. When I inspect launcher.sh, it takes some digging to figure out what js_binary it corresponds to. Some additional comments at the top of the script will reduce this debug time.
Before:
#!/usr/bin/env bash

set -o pipefail -o errexit -o nounset

export JS_BINARY__BINDIR="bazel-out/k8-opt-exec-2B5CBBC6/bin"
export JS_BINARY__BUILD_FILE_PATH="BUILD.bazel"
export JS_BINARY__COMPILATION_MODE="opt"
export JS_BINARY__PACKAGE=""

After:
#!/usr/bin/env bash

# This is a bash script generated to execute the js_binary target
#     @esbuild_linux-x64//:launcher
#
# This bash script is a wrapper around the NodeJS JavaScript file
# entry point with the following bazel label:
#     @esbuild_linux-x64//:launcher.js
#
# The template used to generate this script is
#     @aspect_rules_js//js/private:js_binary.sh.tpl

set -o pipefail -o errexit -o nounset

export JS_BINARY__BINDIR="bazel-out/k8-opt-exec-2B5CBBC6/bin"
export JS_BINARY__BUILD_FILE_PATH="BUILD.bazel"
export JS_BINARY__COMPILATION_MODE="opt"
export JS_BINARY__PACKAGE=""

Nice one. Thanks for the addition.
Can you run bazel run //js/private/test:write_shellcheck_launcher to green up CI? That updates a checked out launcher script that is used for shellcheck.

Pushed a commit after running bazel run //js/private/test:write_shellcheck_launcher and then rebase to HEAD and force pushed to branch


fix(pre-commit): add missing node_modules and release on merge

fix(pre-commit): add missing node_modules
Signed-off-by: Tiago Castro<EMAIL_ADDRESS>
fix: publish helm chart only when the chart changes
This is done by publishing only when the 'chart' is modified.
Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors try

bors merge


fix: Fixes the dsn behaviour

This fixes the behaviour to basically support running sentry.io on a sub path like:
https://mysupercooldomain.com/sentry

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@edd08f9). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1411   +/-   ##
=========================================
  Coverage          ?   84.95%
=========================================
  Files             ?       33
  Lines             ?      771
  Branches          ?      122
=========================================
  Hits              ?      655
  Misses            ?      110
  Partials          ?        6

Impacted Files
Coverage Δ

packages/core/src/dsn.ts
100% <100%> (ø)

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 edd08f9...3886d4a. Read the comment docs.


Serpentshrine Cavern

In the middle of the pool from which Lurker is summoned, mobs need to be able to climb out of the water seamlessly onto the platform and back again. No coords needed since you cant miss that part of SSC.

When underwater, mobs ignore obstacles completely.  They will happily swim right through the side of a ship, or a wall.


Update repositories.txt

LoRa EBYTE E70 CC1310 device library tested with Arduino, esp8266, esp32, STM32 and Raspberry Pi Pico (rp2040 boards)..

I try to di the fix @ArduinoBot


Querying series title and contributors for articles/webcomics

Gets series titles and contributors from Prismic, indexes them alongside articles/webcomics, updates the API to query them. Diff is a bit bigger than one might hope as I've moved some stuff around.

Can we add the article IDs to the queryable fields, alongside the new series IDs? I overlooked article IDs in the original mapping but they're now here as keywords in the prototype mapping and in the query

Yes, will do in a separate PR so as not to pollute the intent of this one


Predicate/Quantus, Level 10:  add additional warning about branch not yet covered

Players that start with unfold Odd, push_neg get a warning “Dieser Lösungsweg scheint mir zu schwierig …”.
Players that start with push_neg, unfold Odd do not get that warning.  I suppose that this latter approach is equally difficult. (I haven’t check this in detail, but I was only able to finish from here using the have tactic, which is not unlocked at this point.)

added the hint again after
push_neg
unfold Odd
intro n

that way it catches a unfold after intro too.