Opacity always renders at 100%

I have 100+ SVGs that I need to use and most of them use opacity for coloring/shading, but it seems like all semitransparent paths are rendered as 100% opaque. Is this a known bug?

Hi @flanakin,
FFImageLoading uses SkiaSharp under the hood for SVGs.
The class used for parsing SVG files is here: https://github.com/mono/SkiaSharp/blob/master/source/SkiaSharp.Svg/SkiaSharp.Svg/SKSvg.cs

Do you use 'fill-opacity' attribute for transparency? It should be supported: https://github.com/mono/SkiaSharp/blob/master/source/SkiaSharp.Svg/SkiaSharp.Svg/SKSvg.cs#L673
Does issue apply to all platforms?

The code used for SVG parsing is here:
https://github.com/luberda-molinet/FFImageLoading/blob/master/source/FFImageLoading.Svg.Shared/SvgDataResolver.cs#L80-L101

Looks like opacity does not work, but fill-opacity works. e.g.,
<svg width='50' height='50' viewBox='0 0 50 50'><rect width='50' height='50' fill='#000' opacity='0.5' /></svg>
renders black rect
but this:
<svg width='50' height='50' viewBox='0 0 50 50'><rect width='50' height='50' fill='#000' fill-opacity='0.5' /></svg>
renders proper grayish one
Should FFImage or SkiaSharp fix this?

@jj09 Thanks for the info. It should definitely be fixed in SkiaSharp, I always incroporate the newest SkSvg class from there into FFImageLoading.
BTW: Google released Skia roadmap and one of the point is to have advanced SVG parser, I hope it'll be done soon. https://docs.google.com/document/d/1C9w8qpPpdgNGThqmgNnTToLZ5UYK4TsUGl5X3B_q6oM/edit


Roll engine 3f4f606105a8..e4ad3a2dcd43 (10 commits)

https://github.com/flutter/engine/compare/3f4f606105a8...e4ad3a2dcd43
git log 3f4f606105a8f6217cfbe08e193aa9a54b19dbef..e4ad3a2dcd437186364ac53b73f2dbe2d02f7b22 --no-merges --oneline
e4ad3a2dc Roll src/third_party/skia 2c1662e24da7..9dbbf887cee1 (6 commits) (flutter/engine#9502)
d85599f03 Roll src/third_party/skia 4f35817f3c91..2c1662e24da7 (3 commits) (flutter/engine#9500)
9190a2284 Roll src/third_party/skia a7016ae7f03d..4f35817f3c91 (2 commits) (flutter/engine#9499)
2c8255f63 Roll src/third_party/skia 01cb2a422884..a7016ae7f03d (5 commits) (flutter/engine#9497)
da82361da Purge caches on low memory on iOS (flutter/engine#9491)
3122ecc62 Roll src/third_party/skia 0c4fbf1bbb93..01cb2a422884 (2 commits) (flutter/engine#9496)
2f1d4f03b fix build breakage on PlatformViews.mm (flutter/engine#9495)
1ab99fb35 fix a bug where the platform view's transform is not reset before set frame (flutter/engine#9490)
aa9b3a180 Reland "IOS Platform view transform/clipping (#9075)" and fix the breakage. (flutter/engine#9483)
d90573a9d Roll src/third_party/skia 69881fb0b5fb..0c4fbf1bbb93 (5 commits) (flutter/engine#9487)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5961665865056256

Commit queue failed; closing this roll.


Missing an expected key "NSPrivacyCollectedDataTypes" in Privacyinfo.xcprivacy

Description
When generating a privacy report for my project, an error is reported indicating that the NSPrivacyCollectedDataTypes key is missing in the PrivacyInfo.xcprivacy file of the swift-sharing package.
Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
The generated privacy report should not contain any information about swift-sharing since it doesn't collect data.
Actual behavior
The generated privacy report indicates that the expected key NSPrivacyCollectedDataTypes is missing in the PrivacyInfo.xcprivacy file of swift-sharing.
Reproducing project
Demo.zip
Demo-PrivacyReport 2024-12-10 11-21-34.pdf
Sharing version information
main branch 524e1a6, 1.0.2
Destination operating system
No response
Xcode version information
16.1.0
Swift Compiler version information
swift-driver version: 1.115 Apple Swift version 6.0.2 (swiftlang-<IP_ADDRESS>.2 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

@tatsuz0u Thanks for the report! Can you run things against this branch and report back if it fixes? https://github.com/pointfreeco/swift-sharing/pull/31

@stephencelis Thank you for the fix! I tested the changes on the main branch, and the key missing error is no longer occurring. 🥳


📃:  Text Translator

:red_circle: Title :  Text Translator
:red_circle: Aim :  Translates the entered text into desired language.
:red_circle: Tech Stack Used :  HTML CSS Bootstrap JS
Screenshots 📷

N/A

:white_check_mark: To be Mentioned while taking the issue :

Full name : Amruta Jayanti
What is your participant role?  GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@AmrutaJayanti go ahead.


Toggle upsert retry behavior

When transporting from Mongo to Mongo, it would be great if there was a way to simply no-op if the insert fails because of a duplicate document already existing.  Right now transporter tries updating the document.
I couldn't find anything in the docs that seemed to control this behavior, so if I missed it, please let me know.  Thanks.

is updating the document an issue? and are you using the bulk option for writes?

yes i'm using the bulk option for writes
the issue is that we have a listener on the oplog and it adds more overhead if it is an update since it needs to do another query to get the full document, whereas with an insert it has the full document in the oplog

here's my pipeline.js
var source = mongodb({
  "uri": "mongodb://user:password@box.dblayer.com:12345/mydb",
  "ssl": true,
  "bulk": true
})

ok, so, you only want to tail the oplog for inserts?

no, the other way around.  on the destination db, we only want to have inserts, and no upserts done on it

gotcha, you could put in a goja transform function that will skip any update operations, something along the lines of
function transform(doc) {
  if (doc['op'] === 'u') {
    doc['op'] = 's';
  }
  return doc
}

that's neat, i haven't worked with that before.  is there overhead loading the javascript vm to run the function?  or is that already being done anyway?

yes, there's definitely some overhead there which is why we've started building native, single purpose functions that don't come with the same performance penalty.

got it.  so would the doc['op'] be update and then noop, instead of the u and s you have in your example above?  those are the functions i'm seeing on https://github.com/compose/transporter/wiki/Messages
so something like:
function transform(doc) {
  if (doc['op'] === 'update') {
    doc['op'] = 'noop';
  }
  return doc
}

slightly off topic, but what's the expected rate gain with bulk: true?  i've been seeing around 65k/minute rates without vs around 85k/minute with it.  does this seem in line with what you'd expect?  just want to ensure i've got it running as expected.  thx

the incoming op would be update and you'd need to set it to skip

I'm actually surprised you're seeing that good of ops/sec without bulk. most of the testing I did showed bulk to have at least double the throughput

interesting.  i haven't been using a transform so maybe that's got something to do with it?    will try the update/skip out right now.  really appreciate all your help.

ah, yea, it's very possible the transform function is slowing things down enough for bulk to not have as big of an impact. I created a new issue https://github.com/compose/transporter/issues/342 to accomplish this without the JS function so keep an eye out for it.

hey JP!  qq on the transform fn.  i'm trying the suggestion you had above about skipping if the op === update but the docs are all being sent to the sink as an update.
function transform(doc) {
  if (doc['op'] === 'update') {
    doc['op'] = 'skip';
  }
  return doc
}

i also tried 'noop' instead of 'skip' as that's mentioned here, but still no luck https://github.com/compose/transporter/wiki/Messages
do you know how i  can print debug statements within the fn?  i've tried console.log but looks like that's not understood within goja.

ok, this is not the answer I'd prefer to give as it's counterintuitive, the docs are off, and it really should work this way. all that being said, here's what you'll need:
function transform(doc) {
  if (doc['op'] === 'u') {
    doc['op'] = 's';
  }
  return doc
}

hm i'm still seeing the updates go through.  i created a file skipUpsert.js that only contains your function, and i'm calling everything with:
t.Source("source", source, "/^balances/").Transform(goja({"filename":"skipUpsert.js"})).Save("sink", sink)

does that look right?

yes, that looks right, add the following flag when you run it:
-log.level=debug
that may give us some insight as to what's happening


Bypass curseforge linkouts

Link
https://www.curseforge.com/linkout?remoteUrl=https://bit.ly/PatreonXaero96
Version
Manifest-v3
What browser(s) are you using?
Firefox browsers
What platform are you using?
Computer (Windows, Linux, MacOS)
(Optional) Anything else?
Seems like a similar extension did this bypass already a few years ago, maybe their work could be reused here too.

https://codeberg.org/Amm0ni4/bypass-all-shortlinks-debloated


Log level option

Running this webhook in number of clusters, so far so good, thanks!
I deployed this webhook via helm chart. Webhook pods produce a lot of logs probably because they are logging with DEBUG level. It there a way to change log level, preferably via helm chart values?

The webhook is built using the kubernetes controller sdk, so as a result, it uses the kubernetes runtime zap logging library. It has a number of flags you can set to configure logging: https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/log/zap#Options.BindFlags - I believe passing these should configure the logging.
The helm chart supports extra arguments via the extraArgs values.
Is that what you are looking for?

Deploying chart with
extraArgs:
- --zap-log-level=info

worked, no more debug level logs. Thanks.


Use colspans in generating number of columns

One file contains an 8-column table.
We're mocking it in wales_composite.html. We mock it by inserting a row of 8 cells.
But, there are not any rows with 8 cells in the data. There are rows with 2, 4 or 6 cells. The 6 cell rows have two "colspan=2" cells - to force the layout engine to accept 8 columns.
I have mocked this pattern in this PR #513

Is this only present in one file? Can we edit the source, or put some kind of tag/note in the file to tell us to treat it as an 8 column table? The table conversion code quite reasonably assumes that the maximum number of columns in any row of the table is the number of columns that table has. The source HTML table makes no sense to have an 8 column table without 8 columns anywhere within it. I think that extending our table code to try and handle columns like this will make it a lot more fragile, and more likely to break on other tables that it currently processes correctly.
What do you think? Can we edit the file so that it only has the number of columns it actually uses?

Actually, I have one idea that might make it work without being too fragile. It'll rely on assuming that there is a cell that is colspanned across the entire width of the table. I'll have a little go at it when I've got time.

Yes, this idea works. I'll tidy it up and do a PR.

Fixed in #511


Update ancestry population names in gks method to always use v4

Annotating v4 genomes now fails since the v4 genomes table uses ancestry population names from v4. This fixes that.

can you clarify the issue? does it involve the change from oth -> remaining?
just switching to always use POPS v4 isn't suitable as currently written because it would drop 'ami' from genomes

Yes @klaricch this was because some of the v3 genomes population labels are no longer in the v4 genomes so it was getting a KeyError. I guess this change is too imprecise though because it might miss some that are there. Do you have an idea of how to better keep the population labels we reference in sync with what's in the tables?
Should another set of labels be added? If v3 genomes and v4 genomes and v4 exomes don't all have the same ancestry groups then do we need 3 sets of keys instead of just the two v3 and v4 in hardcoded in there now? Or can we infer the groups from whatever's in the table?

I think the issue you were seeing should be resolved with this PR (https://github.com/broadinstitute/gnomad_methods/pull/690), but if not, let me know

Thank you @klaricch !


Update catch2 and NetCDF

Closes #29.
Closes #13.

This was meant to also address #30 and #13 but it all became too complex. Let's just address the one issue for now and then come back to the others.


RIS-live stream error handling

We current need to handle two types of errors related to RIS-live stream:

[X] error messages directly from ris-live server (see #139)
[X] ~http disconnection error (see #82 and #116)~

this issue is handed off to wandio, see https://github.com/wanduow/wandio/pull/43

This is fine, but I'm not convinced that the wandio fix will deal with the case where the connection is terminated if we don't keep up. In that case we'll still get a 200 code, and then we'll consume data for a while and then the connection will be terminated, which will look like EOF.

Note that this PR handles only the error messages (more in RIS Live manual) coming from ris-live server, not network-layer errors between libbgpstream and ris-live server.
As @alistairking commented, this PR does not handle the case where disconnection happens during the processing (see issue #116). It will be handled in HTTP transport (#82).


Endpoint analytics

Feature Request
Is your feature request related to a problem?
As an administrator I would like to have insight into the device analytics, so I can determine where issues might exists
Describe the solution you would like
That data from the endpoint analytics feature be included in the report
Describe alternatives you have considered
N/A
Additional context
N/A

Triage - Removing from the v2.0 project this item will be included in a future release


initial kafka setup for rust and python

setup basic logic for connecting to Upstash kafka, creating producers ,consumers, toppics and consumption loops.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#241  👈
#237
#235
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @chrisaddy and the rest of your teammates on  Graphite


Cannot run binary file

CentOS 7 64bit
Mono installed from: https://www.mono-project.com/download/stable/#download-lin-centos
After Make command I have "Cannot run binary file" why?

Sorry, my mistake... somthing with paths...
Now I execute "mono odocxmerge file1 file2" - all ok
Thanks! Cool util

Solved


Fix build failures

This changes some parameters that were causing issues to vulkan-hpp types

@jherico  Thank you


Add term "Population"

Description of the issue
Population, and subclasses thereof, are used by various energy models to e.g. drive demand.  Also it's listed as missing in #192
Ideas of solution
Named missing classes (#192) so far:

Population
Working Population (synonym Labour Force)

There are many further subdivisions possible.  The UN population division for example subdivides by sex and age group (in five-year cohorts).  The OECD by different age groups (I think ten-year cohorts) and education level.  Other @OpenEnergyPlatform/oeo-domain-expert-energy-modelling may have additional needs.
So do we need a scheme for subdividing populations analogous to sectors, with sectors conforming_to sector divisions (#460)?
I think Populations would go under independent continuant -> immaterial entity, just like organisations.  But strictly speaking, populations are composed of people, so it would be independent continuant -> material entity -> object aggregate, which … feels weird. Will need input from @OpenEnergyPlatform/oeo-general-expert-formal-ontology here.
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[x] I already read the latest version of the workflow for this repository
[x] The goal of this ontology is clear to me

I am aware that

[x] every entry in the ontology should have a definition
[x] classes should arise from concepts rather than from words

Let's first find a definition, the classification arises from the definition.

"A population is the number of people in a city or town, region, country or the world."
–https://en.wikipedia.org/wiki/Population

To reuse terms we already have, what about: A population is the number of people in a spatial region.

I don't think we should define a population as the number of people because that would make, e.g. "25" a population. Could we define it as an aggregate of people?

Good idea, Janna! So: A population is an aggregate of people in a spatial region.
The number of this population (e.g. 80 million in Germany) would then be a data property a specific instance of the population linked with a specific instance of a spatial region (Germany) right?

A population is an aggregate of people in a spatial region.

So independent continuant -> material entity -> object aggregate …
What about subdivisions by sex and age?  Elaborate scheme or just a bunch of subclasses defined as needed, like
"A population (female) is a population of female people."
One subdivision of populations we are actually using is rural/urban.

What about:

A female population is a population of female people.
A rural population is a population in rural areas.
A urban population is a population in urban areas.

Probably there's no need to define "female" and "male".  How do you feel about "rural" and "urban"?
And what about intersection sets (male urban population age 31-40)?  Do we add all combinations?

If you need that much detail, I would see these more of instances of population than classes. You have population age classes of 31-40 but another model might have age classes of e.g. 18-60 instead.
@jannahastings : What do you think.

If you need that much detail

I don't.  But other's might ;)

Okay, at the moment, we should focus on what we really need and not on what someone in future might need.

Okay, at the moment, we should focus on what we really need and not on what someone in future might need.

I agree: let's focus on including the broader classes for now, and if we need specific sub-classes for annotation in the future, we can add them with a "just-in-time" approach as they are needed. These will be compositional in any case, so we can always add them dynamically as long as we can define or re-use from external sources the required properties such as age and urban vs. rural. (Incidentally, we should check ENVO for urban vs. rural, they may have definitions we could re-use)

Okay, at the moment, we should focus on what we really need and not on what someone in future might need.

My concern is whether we set up a system akin to what is happening with sectors (#30, #460, #461), where there could be different definitions of "divisions of population" (for lack of a better term), or if we expect to add a bunch of subclasses as needed.  Planning ahead could save us a lot of trouble.  But I don't have strong feelings either way.

I also prefer to stay simple now and concentrate on whats needed.
When looking at the definition ("A population is an aggregate of people in a spatial region.") I would prefer to solve this together with the long planned society class ("an object aggregate of people", #301, #302).
Right now following the definitions a population would be a subclass of a society, which sounds a bit weird?
Society is used in definitions like organisation role: "An organization role is a role of a society that has a collective goal and a set of organization rules. "

If society is defined as an object aggregate of people, then it should be a synonym of population.

Just jumping in while this popped up in my inbox: Wouldn't the difference between population and society be that society includes specific rules (laws, codes of conduct, governance) on how the people that make up the population interact with one another and as a whole?

I tend to agree that there should be more to the definition of society than just the people in it. So we need to reconsider the definition of organisation role: is it society that is needed there, or population (aggregate of persons)?

I don't think people have to have anything more in common than all being people to create an organisation.
So I propose to leave society out for now, make a comment in the concerned issues, and change organisation def from society to population?

That sounds good to me. And if "population" sounds (by implication rather than definition) too large given that organisations might be small, we could also consider explicitly saying "aggregate of people" instead (which would indeed be a true synonym for population).

I like the population definition.
society is not included yet. Unlike population, society and  organisation role are not restricted to or defined by a spatial region, from my point of view. So, I'd stick with the current def of oraganisation role.

@stap-m in the pull request I now pushed to just change the organisation role def to "aggregate of people", would you agree with that?

We use lower case for class names. Also the issue numer #479 should be referenced in the term tracker item. I'll correct that.


Application round service sector can be null

@kieferbonk This may not fix the issue you're having, and if it doesn't let's check your role permissions.


Documentation should take class extensions into consideration

As of right now, deno doc does not respect extensions. This means
export class B {
   public id: number;
}

export class A extends B {
  public userName: string;
}

if I run deno doc for class A, it will only document the property userName but will not for id part of the class extension.
deno doc should be able to read this, and respect the extensions of extensions (example, B extends C)

There was a feedback from std users that inherited properties not being shown in API docs is confusing https://github.com/denoland/deno_std/issues/4353

Can we consider listing all properties including inherited ones?

I think we should not list all properties, but instead hyperlink to the associated symbol (e.g. type alias/interface/class/etc.). This will prevent a significant amount of noise.
Ref: https://github.com/denoland/deno_std/issues/4353#issuecomment-1962927306

Hyperlink is already available in doc website (jsr). For example, see https://jsr.io/@std/csv/doc/~/CsvParseStreamOptions

This will prevent a significant amount of noise.

I'm not sure I agree these are noises.
If someone comes to the page for CsvParseStreamOptions, then they probably want to know the all available options in CsvParseStreamOptions. Currently they need to see 2 separate pages to know all options, and that is not convenient for them.

I'm not sure I agree these are noises.
If someone comes to the page for CsvParseStreamOptions, then they probably want to know the all available options in CsvParseStreamOptions.

I agree that in cases where the interface is fully bespoke, a reader would certainly want to understand the shape of the type in full (because it is completely unfamiliar).
However, there are many cases where types are derived from other types (e.g. extension, inheritance, intersection, etc.) — and in many cases the existing types are already familiar, so the expanded type information serves no purpose (it's only noise). Here's a contrived example for illustration:
TS Playground
type LinkedNode<T> = T & {
  child?: LinkedNode<T>;
  parent?: LinkedNode<T>;
};

type ResponseNode = LinkedNode<Response>;
/*   ^?
{
  arrayBuffer: () => Promise<ArrayBuffer>;
  blob: () => Promise<Blob>;
  readonly body: ReadableStream<Uint8Array> | null;
  readonly bodyUsed: boolean;
  child?: LinkedNode<Response>;
  clone: () => Response;
  formData: () => Promise<FormData>;
  readonly headers: Headers;
  json: () => Promise<any>;
  readonly ok: boolean;
  parent?: LinkedNode<Response>;
  readonly redirected: boolean;
  readonly status: number;
  readonly statusText: string;
  text: () => Promise<string>;
  readonly type: ResponseType;
  readonly url: string;
}
*/

In the example above, Response is a standard, built-in type, and well-known by many developers. I think it is noise to include all of the fields in Response, and — similarly — I think it is noise to include the properties from LinkedNode because they both already exist as named types and can be referenced. If the reader is already familiar with either/both of them, then inlining all of the content makes it more difficult to understand which fields are unique to the type in question (there are none in the example, but other types might have them).

Currently they need to see 2 separate pages to know all options, and that is not convenient for them.

If the user is not familiar with Response, then perhaps they're also unfamiliar with FormData, which is in the return type of the field formData:
formData: () => Promise<FormData>;

It's also inconvenient for the user to click FormData to see its type information, right? It seems arbitrary that we would expand and inline all of the type information for the top-level fields, but not any of the type information for nested fields. I'm not sure that the convenience reasoning is scalable.

Ok. The above case looks like a valid case where showing inherited props can be noisy.
Then how about showing some text link saying show inherited properties at the bottom of properties? and when a user clicks that button, the inherited props are expanded there. That would cater for both needs, I think

Then how about showing some text link saying show inherited properties at the bottom of properties? and when a user clicks that button, the inherited props are expanded there. That would cater for both needs, I think

I think that's a great feature idea!


Feature/explore topic data options screen - ticket#119

What
We have updated the explore data screen as per trello ticket#119
To review the changes visit the /categories/topicSlug ==> topicSlug =population-basics,health or go on /topicSlug and navigate to this screen using the header.

LGTM, just a few small changes suggested.
We should probably update the URLs as one opens and closes the accordion as well (right now the one you landed on sticks throughout all the clicking). This means adding another custom functionality / prop on top of a standard ONSAccordion component.

This is definitely something that needs to be discussed with Lynsey .


ITK build failed on MSVC + permissive- on windows

ITK failed when build with /permissive- by msvc on Windows, I use latset version b95b4c4 on master branch. Could you please help look at this?
You can repro this issue as the steps below:

git clone http://itk.org/ITK.git D:\ITK\src
open VS2017 x86 command prompt as admin and browse to D:\ITK
set CL=/D_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING /permissive- /Zc:twoPhase-
mkdir build_x86 && pushd build_x86
cmake -G "Visual Studio 15 2017" -DCMAKE_SYSTEM_VERSION=10.0.17134.0 -DBUILD_SHARED_LIBS=OFF -DBUILD_TESTING=ON -DBUILD_EXAMPLES=OFF ..\src\
msbuild /m /p:Configuration=Release;Platform=Win32 All_BUILD.vcxproj /t:Rebuild

Failures:
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369,1): error C2362: initialization of 'nrOfRecipientsLen' is skipped by 'goto err'
err:
^
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(247): note: see declaration of 'nrOfRecipientsLen'
DWORD nrOfRecipeints, nrOfRecipientsLen = sizeof(DWORD);
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369): note: see declaration of 'err'
err:
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369,1): error C2362: initialization of 'kekAlgLen' is skipped by 'goto err'
err:
^
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(235): note: see declaration of 'kekAlgLen'
DWORD kekAlgLen = sizeof(ALG_ID);
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369): note: see declaration of 'err'
err:
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369,1): error C2362: initialization of 'cbMessageTypeLen' is skipped by 'goto err'
err:
^
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(221): note: see declaration of 'cbMessageTypeLen'
DWORD dwMessageType, cbMessageTypeLen = sizeof(DWORD);
D:\ITK\src\Modules\ThirdParty\GDCM\src\gdcm\Source\Common\gdcmCAPICryptographicMessageSyntax.cxx(369): note: see declaration of 'err'
err:

@QuellaZhang thanks for the report.
The path forward is to merge a patch into upstream GDCM for the release branch, then we will merge it into ITK.
It sounds like a refactor to remove goto could help.
 The errors from GDCM library are appearing because it is one of the base libraries in the dependency graph. If those errors are fixed, I would be surprised if no new errors appeared somewhere else. Removing goto statements would be nice, but I don't think that will be the end of it.

Hello, any updates?

#261 should take care of compiling in conformance mode (/permissive-).
Why do you want to disable two-phase lookup?

It turns out that adding the switch /Zc:twoPhase- made no difference after #261 was merged. Closing the issue as fixed.


totalVI: using info from empty droplet to evaluate ambient protein expression

Dear authors,
We collected some CITE-Seq data and as part of the analysis I am trying to subtract protein expression background that we have. I very much like your idea in totalVI where you try to separate signal and background portions. And a question that I have is it all possible to tell your model what is the expectation of my background signal? From empty droplets I can get an idea of what it may be and then your model can maybe use it as some starring point...
Because if I run totalVI as is following your tutorial it gives me different results for two antibodies. And when I compare empty droplets versus cells counts it looks like for both of these I should expect mostly background for both of them. And totalVI only says it for one antibody out of two.
Best,
Sasha.

Hi Sasha,
Thanks for your interest in totalVI! totalVI does not use empty droplets to determine protein background, so it is not possible to use this information in the model. We found that empty droplets alone are not sufficient to model protein background because background also comes from non-specific antibody binding. How did you choose the empty droplets that you're comparing against here?
There are a couple of things that I recommend checking. Do you expect PD1 to be expressed in any cells in this dataset, or do you expect that it is all background? Is there any Pdcd1 RNA expressed in these cells?
Other things to check are your filtering criteria before running totalVI (e.g., protein library size and number of proteins detected). If there are some low quality cells that have very low library sizes, it might make it appear that the rest of the cells have foreground protein expression in comparison.
Next, it is good to see that the isotype control appears to be all background. Does what you are seeing for PD1 happen for other proteins in your dataset, or is this unexpected protein foreground only an issue for PD1? I suggest visualizing the denoised values to see if these make sense for PD1 (and other proteins). Even if cells have a high foreground probability for PD1, the denoised expression could still be low.
Zoë

Hi Zoë,
Thanks for a prompt reply. To call cells I used a union of cellranger cell calling and also an emptyDrops function from DropletUtils R package. And then some manual selection on top of that. And so everything outside of this union of cellranger+emptyDrops I called empty droplets.
PD1 is expected to be negative in this dataset and mRNA kind of supports that.
There are are 38 proteins in total in this dataset and what I am seeing for PD1 happens for a number of other antibodies as well: their level of expression doesn't really exceed that of the isotype control or of empty droplets and yet totalVI calls them as having mostly foreground protein expression.
As for cell filtering there are some strange cells that don't stain much of the antibodies but nevertheless have some meaningful mRNA in them. I wouldn't call them low quality, they just don't "like" antibodies.

Hi Sasha,

As for cell filtering there are some strange cells that don't stain much of the antibodies but nevertheless have some meaningful mRNA in them. I wouldn't call them low quality, they just don't "like" antibodies.

Indeed having a population of cells that do not stain well could inhibit totalVI's ability to properly disentangle the signal. Ideally, the input to totalVI would be those cells that are "high quality" in both modalities. That said, the denoised protein expression for this protein will not be very high. Are these strange cells the ones at the bottom? Could you plot the protein library size on the UMAP?

Hi Adam,
Yes, these strange cells are at the bottom. And those are potentially some T cells in semi-differentiated state. In this experiment PBMC cells were given some kind of a treatment that would cause T cells to differentiate so it makes sense to find those kind of cells.
Sasha.

On Jun 17, 2020, at 1:23 AM, Adam Gayoso<EMAIL_ADDRESS>wrote:
Hi Sasha,
As for cell filtering there are some strange cells that don't stain much of the antibodies but nevertheless have some meaningful mRNA in them. I wouldn't call them low quality, they just don't "like" antibodies.
Indeed having a population of cells that do not stain well could inhibit totalVI's ability to properly disentangle the signal. Ideally, the input to totalVI would be those cells that are "high quality" in both modalities. That said, the denoised protein expression for this protein will not be very high. Are these strange cells the ones at the bottom? Could you plot the protein library size on the UMAP?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/YosefLab/scVI/issues/684#issuecomment-645158338, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEBJNFLFG7EWBSWQ7TJJ6Z3RXBHONANCNFSM4N774YUA.

--
This message contains information which may be confidential and/or
protected by attorney-client privilege.  Unless you are the addressee, you
may not use, copy or disclose to anyone this message or any information
contained in this message.  If you have received this message in error,
please send me an email and delete this message.   Thank you.

Oops it didn't attach anything by email, let's try it by posting a message.

To call cells I used a union of cellranger cell calling and also an emptyDrops function from DropletUtils R package.

Doesn't cell ranger use emptyDrops?
How many cells are you using as input to totalVI?

PD1 is expected to be negative in this dataset and mRNA kind of supports that.

How do you make this conclusion? Based on what you shared I feel like I can make an argument that it is expressed. It would be helpful if you also looked at the UMI counts for this gene.

As for cell filtering there are some strange cells that don't stain much of the antibodies but nevertheless have some meaningful mRNA in them.

Are the cells at the very bottom distinguishable from the other cells with similar gene expression in a standard RNA only pipeline?
In general, we advise using the denoised protein expression and doing our differential expression instead of deriving gates based on the probability foreground alone. Could you share the denoised protein expression?
I also don't understand the difference between the first and second umap you just sent as they have the same title.

Cellranger implemented emptyDrops algorithm in their python code but they are not actually calling emptyDrops directly. There is significant overlap between the two depending on a dataset but still it is not the same. There are about 4000 cells that I used as an input to totalVI. We know we put 4000 cells so that number sounds about right.
In RNA only pipeline "cells at the bottom" become cluster 8 on the right. And yes they have some distinguishing markers based on UMI counts.
OK there is some PDCD1 expression but still it looks mostly negative again from UMI counts.
I see it's better to avoid using probabilities and use denoised counts. Attached are the umap plots.

![pd1_isotype_control_denoised](https://user-images.githubusercontent.com/16946837/84943096-da83eb80-b0b1-11ea-90b7-c1a213fbd52f.png)

OK there is some PDCD1 expression but still it looks mostly negative again from UMI counts.

Thanks for sharing. This is indeed a difficult case. Based on your Seurat umap plots, it does indeed seem that pdcd1 is expressed in many cells based on RNA, but lowly. This implies to me that the foreground prediction made by totalVI is reasonable.
In a way, this is kind of the reason we don't use the data to set a prior on the background -- because it's very sensitive to what's called an empty droplet, panel makeup, etc. If you even look at the original CITE-seq paper, they spiked in cells from a different species and you see that the background is not really global.

their level of expression doesn't really exceed that of the isotype control

This is not a sufficient condition to call something background as it depends on quantitative characteristics of your panel. In fact, it's strange that the one mode of the isotype control in the ambient barcodes has a higher median than the mode for the real cells. The tails of PD1 distribution on the log scale in your first figure is are fatter than the isotype control also, which suggests that it's not purely ambient.

their level of expression doesn't really exceed that of the isotype control

This is not a sufficient condition to call something background as it depends on quantitative characteristics of your panel.

It remains so even after adjusting for different antibody concentrations if that's what you mean.
There is something strange going on in this experiment. We suspect that many antibodies are either falling off their target and floating around in a soup or even get degraded but their barcodes are still intact. And that may explain why ambient barcodes frequently have similar level of expression as real cells which is illustrated in the first set of figures. It's true some of PD1 expression is non-ambient but it's nowhere near as clear as say CD4, one our better examples.

Hi Sasha,
Your data looks interesting. As Adam suggested, I don't necessarily think that something is wrong here. I'm intrigued by those three clusters at the bottom of your UMAP plot that appear to have lower ADT counts, but only two appear to have the T cell markers you pointed out. Do all three end up in the RNA-only Cluster 8? Or does each of the three smaller clusters have unique RNA markers? I wonder if those cells include a mix of cells of the same type - some that stained well and some that did not. In this case, I would apply a stricter filter on the ADT UMI counts before running totalVI. If not (and you are convinced that the low antibody detection in these cells is not an artifact), it appears that the result you're seeing reflects a unique phenotype for these cells and I'd suggest proceeding with your analysis.
PD1 does indeed look like a difficult case to distinguish, especially because in your plot of raw data, it looks like these cells have no counts while every other cell type receives some amount of counts (whether or not they are foreground or background). I see that these cells don't appear to express CD4. Can you detect other expected T cell markers like CD8? If not, it appears that their lack of protein staining is indicative of lower quality.
In conclusion, I'd recommend trying two things. First, investigate the quality of these "non-staining" cells and try removing them before running totalVI to see how this affects your protein denoising. Alternatively, use the model as is, but keep in mind your knowledge that protein DE on this cluster of cells might have unexpected results given that the staining in these cells did not work as expected for this cell type.
One note on your point that "ambient barcodes frequently have similar level of expression as real cells". This might be related to how you selected empty droplets. It is possible that droplets just below the cutoff of being considered cells contain dead cells or cell fragments/debris, and could therefore have very similar levels of antibody binding as cells (while RNA has degraded more quickly). If you're interested in comparing to droplets that are truly empty, I would instead choose an RNA UMI cutoff that is lower than all cells detected by emptyDrops (like 100 UMIs, as in the emptyDrops paper), and also exclude droplets with extremely low counts (lower than 20 UMIs), which could be the result of sequencing errors.
I hope this helps!
Zoë

Hi Guys,
So what I ended up trying is to actually add my beliefs about antibody backgrounds into your model. I modified two lines of code in totalVI constructor, after about line # 104
            self.background_pro_alpha = torch.nn.Parameter(
                torch.tensor([35.10129549,  21.84515754,<PHONE_NUMBER>8,  35.76444929,
        22.95088955,  64.47878612,  28.78123608,  48.10336538,
        67.90458643,<PHONE_NUMBER>6,<PHONE_NUMBER>1,<PHONE_NUMBER>7,
<PHONE_NUMBER>2,   3.29264328,   2.5421942 ,  31.00417083,
        39.87524728,  78.4401522 ,  40.69179156,  40.50559832,
<PHONE_NUMBER>1,  17.41751705,  63.92961816,  42.23915539,
        58.99817905,   8.83917139,  15.12586042,  28.85759479,
         4.9506835 ,   4.24656145,   5.7083231 ,  16.75638644,
        11.80848372,   7.40514496,  21.76840866,  10.93849064,
         8.89010248,   6.38084776])
#                torch.randn(n_input_proteins)
            )
            self.background_pro_log_beta = torch.nn.Parameter(
                torch.tensor([ 2.54918386,  4.97250783,  3.86991096,  2.54067511,  3.98271688,
        3.18437106,  2.33621802,  2.84790875,  3.27534542,  4.07635033,
        4.28914376,  4.10811267,  4.47967407,  0.34850747, -0.17728883,
        2.41893878,  2.6641811 ,  3.50375743,  2.68648521,  3.07641335,
        4.13333484,  1.92913417,  4.97954541,  2.73156826,  6.66715033,
        3.92785977,  3.4320138 ,  3.97616401,  3.39640639,  3.07835912,
        3.62657674,  4.73322823,  3.96942934,  3.32012492,  4.40690117,
        4.19658137,  4.25255106,  3.42346765])
#                torch.clamp(torch.randn(n_input_proteins), -10, 1)

First set of numbers is what I believe are the averages of my background counts. And second set are logarithms of standard deviations for those backgrounds. If I really want to add my priors is it the right way to do it?
PD1 happens to be the first numbers of each set, about 35 for the average and about 2.55 for the log of standard deviation. And when I re-run your model with the above changes my PD1 denoised counts become much better I think. In particular, they now match low mRNA counts a lot better. And yes, both metrics indicate a little bit of PD1 expression in the upper left corner.
For empty droplet selection Zoë yes my method can be optimized that's true. Peaks at 1 in what I called ambient counts are probably not real and can be filtered out. The rest though should be roughly correct I would think. One thing I noticed is that cellranger implementation of emptyDrops frequently keep dying cells in while the original emptyDrops tends to throw them out. And also for that PD1 ambient plot there are over 50,000 droplets (if I exclude peak at 1) that I call empty and if a few thousands are actually not empty for the reasons that you outline it probably won't change the overall shape of ambient distribution by much. And so things like average and standard deviation should still be good enough to use I think.
Assuming I put in my numbers in the right place in your model I don't mind doing it for other datasets as well. But of course it would be nice to have it as an API option.

Thanks for these tips! It was actually my intent to just provide initialization and then see what happens. I didn't realize I can actually look at model.background_pro_alpha and model.background_pro_log_beta after the training.
I tried different variations and so far my best version is to set beta just as you describe (I hope I implemented it right) but then for alpha take that average of the UMI count on the natural log scale and multiply it by a factor 10. If I don't multiply by 10 then for a number of antibodies denoising becomes suboptimal I would say. I am guessing that initializing with a large alpha is in some way assuming that there is a lot of background and if there is any signal then it's better be higher than this initial assumption.

Interesting, I would say it might also work to make the prior variance larger as it implies greater uncertainty, while keeping the mean at the empirical estimate.

My second best version does in fact have larger variances. But it still uses 10 times multiplication factor.
And so if I take this second best with larger variances and drop 10 times multiplication factor then it becomes not as good...


Constrained value optimizer.

Codecov Report

Merging #11 (ed9b0ab) into latest (5b19cb6) will decrease coverage by 0.25%.
The diff coverage is 67.50%.

@@             Coverage Diff              @@
##             latest      #11      +/-   ##
============================================
- Coverage     78.54%   78.28%   -0.26%
- Complexity     1503     1534      +31
============================================
  Files           120      124       +4
  Lines          8919     9119     +200
  Branches       1358     1401      +43
============================================
+ Hits           7005     7139     +134
- Misses         1409     1469      +60
- Partials        505      511       +6

Impacted Files
Coverage Δ

...a/com/msd/gin/halyard/strategy/BindingSetPipe.java
78.26% <ø> (ø)

...alyard/strategy/HalyardQueryOptimizerPipeline.java
83.33% <ø> (ø)

...va/com/msd/gin/halyard/common/ValueConstraint.java
41.17% <41.17%> (ø)

...n/halyard/algebra/ConstrainedStatementPattern.java
48.57% <48.57%> (ø)

...n/halyard/strategy/HalyardTupleExprEvaluation.java
67.76% <48.71%> (-0.30%)
:arrow_down:

...a/com/msd/gin/halyard/common/ObjectConstraint.java
54.83% <54.83%> (ø)

...ava/com/msd/gin/halyard/common/StatementIndex.java
85.76% <64.70%> (-2.09%)
:arrow_down:

.../halyard/optimizers/ConstrainedValueOptimizer.java
80.72% <80.72%> (ø)

.../com/msd/gin/halyard/common/HalyardTableUtils.java
75.15% <100.00%> (+1.68%)
:arrow_up:

...in/java/com/msd/gin/halyard/common/Identifier.java
80.55% <100.00%> (ø)

... and 7 more

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 5b19cb6...ed9b0ab. Read the comment docs.


Add workflow overview

Visualization of our GH action strategy.
depends on:
#18 ✅ to prepare for code coverage and doxygen
#12 ✅ for Code coverage
#20 ✅ for doxygen

also added the trigger conditions for the merge queue

We tried to capture the rules how GitHub references jobs to status checks here: https://otterdog.readthedocs.io/en/latest/reference/organization/repository/status-check/

I looked at that page already. Probably it's a good idea to just use unique names for all jobs.


Poetry

use poetry

Coverage decreased (-6.7%) to 93.333% when pulling 11886bb3389227cae28596f753be6bd1f02a5d99 on poetry into fd1f1e7ca5643eeda1467c305a3c95e7adba3b19 on master.


Carla from sources freezes after make launch (Ubuntu 18.04)

I have successfully built Carla-simulator from sources using documentation. I have tested UE4 before carla installation and it works well. After I make launch and after make PythonApi (it all finished with success!) UE4Editor is freezes after 2-3 seconds after I launched it.
How could I resolve that problem?
I actually need to build a custom map, so I need editor at first.

I know it's been a while since this issue was posted, but is there a workaround for it? I am facing exactly the same problem. Thanks in advance!


fix issue - host compiler uses wrong dmd.conf

remove dmd.conf before building dmd from host compiler
also use dmd.conf target of posix.mak

Wrong target branch.


Trait "hat" disappears .

Hello nathan ^^ .
When I restart the server. NPC's with the trait "hat" , turns back to "Spell hat unknown" .
What can I do?
Regards!

I just tried this myself to make sure it hasn't broken and it seems ok. You are trying to put a hat on an NPC, right- not give them the hat spell?
 Is that what you were doing?

Yes, I use .
/ trait magic /mtrait hat wool
by the moment, dont have problems right now when I reboot. So.. maybe was an a bug .
Thank so much ^^


brush size in 3D

Hi @maarzt ,
I just hit an issue with a collaborator using labkit for annotating objects. Can you tell us if it is possible to change brush size in x/y and z independently? In some situations we would like to annotated 3D in other situations, we would like to annotated slice-by-slice.
Thanks!
Cheers,
Robert

This is possible using a workaround:

Open an image as 2D+time stack -> slice by slice annotation
Open an image as 3D stack -> 3D annotation

I also plan to implement a simple button that allows to switch between simple 2D and 3D annotation mode.


Question about the non-strict IL setting

Hi,
In the non-strict IL setting, which the paper adopts, is it possible an image appears in both the base stage training and incremental stage training? For example, an image contains both the annotation of the base classes and new classes. The image is sampled in the base step to learn the base classes, and is also sampled in the incremental step to learn the new class.

thanks.


Add HTTP status code response helpers

Added IsHTTPStatus() and HasHTTPStatus() methods to autorest.Response
Thank you for your contribution to Go-AutoRest! We will triage and review it as soon as we can.
As part of submitting, please make sure you can make the following assertions:

[ ] I've tested my changes, adding unit tests if applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, except in the case of urgent bug fixes warranting their own release.
[ ] If I'm targeting master, I've updated CHANGELOG.md to address the changes I'm making.

Fixes https://github.com/Azure/go-autorest/issues/336

Fixes https://github.com/Azure/azure-sdk-for-go/issues/2175


Adds list of skills needed to each project card on listing.

This  adds "Skill Types" list above "Project Types" on the Project cards
Also:

adds title attribute to "and X more" element
use 'blue' for "Skill Types" to differentiate from "Project Type" indigo.
some small refactoring

Before

After

hmm could these be on one row?
 looking for

@sradu does this look ok?

What if we bold looking for and add : at the end. It feels a bit iffy right now. How does it look on mobile and if there are a lot of requests?

Ooh, bold does look better:

(I also thought about aligning "Skills needed" to the right, but gave up bc my flex skills are lacking)

Aligning sounds like a great idea, perhaps @adrianthedev or @OrlandoLee can look into it, but I feel it looks OK to me.

How does it look like on mobile?

On mobile it should be on a new row. It looks off anyway.

Sorry, meant to say it looks off otherwise. Apologies.

Ok, figured out the right-align thing -- sometimes I miss table layouts 😆
Desktop:

Mobile:

Should I commit that?

On Sat, Mar 21, 2020 at 7:00 PM Tieg Zaharia<EMAIL_ADDRESS>wrote:

Ok, figured out the right-align thing -- sometimes I miss table layouts 😆
Desktop:
[image: Screen Shot 2020-03-21 at 9 58 13 PM]
https://user-images.githubusercontent.com/5054/77240707-3fb56700-6bbf-11ea-9ac1-6e7af9aa5470.png
Mobile:
[image: Screen Shot 2020-03-21 at 9 58 50 PM]
https://user-images.githubusercontent.com/5054/77240706-3fb56700-6bbf-11ea-85f0-46dfc2a93b2f.png
Should I commit that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sradu/covid-volunteers/pull/47#issuecomment-602135698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAUCJSG7RGHWDJU2H3CXE3RIVWK3ANCNFSM4LRC37RQ
.

Great job! Left a tiny note on Discord.


OpenBMC GUI deliverables Testing and Regression run

Test Plan

[x]  Verify Setting the power cap  - @ssombar
[x]  Verify Date Time management - @rahulmah
[x] Verify SNP Read ONLY- @prkatti1
[x] Full GUI regression Testing - @rahulmah

@gtmills  Counterpart Developer to sync up on this.

Also would like a general regression test on the GUI. :)
Thanks!!

@rahulmah is doing regressing testing.


Errors in auto_debugging task

Error in output at line 40
https://github.com/google/BIG-bench/blob/6bf883cf02540a6eb6475704b5437cdeb92e9a70/bigbench/benchmark_tasks/auto_debugging/task.json#L40
This target output is incorrect, the correct output is [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]. This is easily verified:
>>> exec("\nx = [i for i in range(10)]\nfor x_elem in x:\n\tx_elem *= 3\n")
>>> print(x)
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]

Python's syntax for x_elem in x: ... references copies of each value in x, so the original list is never modified. I've tested this is in Python 2.7.16 and 3.8.13 .
Alternate error possible for line 116
https://github.com/google/BIG-bench/blob/6bf883cf02540a6eb6475704b5437cdeb92e9a70/bigbench/benchmark_tasks/auto_debugging/task.json#L116
With a fresh python install, the numpy module will not be installed, so the exception ModuleNotFoundError: No module named 'numpy' is equally valid. Since this example does not require any numpy functions for the expected exception, using a module from the python standard library such as sys would be a reasonable substitute.
Suggested fixes:

Update target output at line 40, or task at line 39. Update human and model evaluation metric values for auto_debugging accordingly.
List Python version used to execute code, as well as necessary dependencies (or, for the latter, change imports in programs to only use standard library modules). These specifications should be included in prompts to LLMs, otherwise there may be multiple equally correct answers - in addition to the numpy ModuleNotFoundError, a simple example is print 'hi', which will produce a SyntaxError in Python 3, but not in Python 2.

@motiwari

Thank you for catching both of these subtle bugs!!
Would you like to make a PR implementing your proposed fixes? If you do, I will merge it immediately. If I don't hear from you in the next day or so, I will make a PR myself implementing your suggestions.
Note, I would actually do something slightly different for the second case. I would make targets a list including both valid exceptions, like:
    "target": ["NameError: name 'numpy' is not defined", "ModuleNotFoundError: No module named 'numpy'"]

We (benchmark organizers) discussed this, and decided to put task correctness ahead of consistency with past evaluations. The consequence of your PR will be that the existing model evaluations will be slightly less accurate, and will potentially have a slightly lower score than they otherwise could have achieved. (note that this is already true for human eval for some tasks -- some tasks were modified during the human evaluation period)


[bug] <rh-table> --rh-table-background-color not working

Describe the bug
Setting --rh-table-background-color has no effect. the property is actively set in the lighdom stylesheet. if we want it to be customizable, it should be documented and use fallbacks
as well, it does not appear in the cem docs
Which Component?
rh-table
Steps to reproduce

add an rh table with content
add this css

:root {
--rh-table-background-color: red;
}

hover a row

Expected behaviour
bg is red
Screenshots
No response
Element HTML
No response
Operating System (OS)
Linux
Browser
No response
More browser information
No response
Additional Info
No response

--rh-table-background-color does not exist lightdom.css are you referring to --rh-table-row-background-color and --rh-table-column-background-color and --rh-table-row-border ?
That said, those values should also be set as _private and moved to a failover calls:
_row-background-color: var(--rh-table-row-background-color,  ...);
Then both should also be added to the cem docs as stated.


Variable Descriptions in Reports

Missing functionality
Ability to have descriptions of variables alongside their descriptive statistics. Most often when EDA is being performed we are unfamiliar with a dataset. Being able to incorporate variable descriptions into the reports would be useful.
Proposed feature
Ability to provide a variable dictionary {"var_name": "description"} to the ProfileReport function. These descriptions would then be alongside the variable name/statistics in a generated report.

Good point, see #232

This feature is included in the next release. The descriptions can be passed as a dict to the variables.descriptions parameter.


Extension gets installed but is grayed out on the Extensions menu

Hi!
I installed your SidebarUsingContentScripts extension following your instructions in your README.md file (for Chrome) but the extension is grayed out.  And yes, it is both enabled and I did reload the browser before testing.
Any help in this regard would be greatly appreciated...

Hi there! I don't think I understand what you are referring to, can you provide a screenshot to show the what's grayed out?

Thanks for the quick reply.  As shown below, the "Sidebar with ContextScripts" menu item is grayed out:

This menu item is only available if there is any action registered in the manifest, which will show a popup when clicked. The sidebar example is showing how you can have Blazor injected into a page when using content script. You can see this if you navigate to Google's home page, there is a button that is injected and will show the sidebar when clicked.

I understand.  That worked perfectly.  Thanks….
From: mingyaulee @.>
Sent: Sunday, January 22, 2023 3:30 AM
To: mingyaulee/Blazor.BrowserExtension.Samples @.>
Cc: Louis S. Berman @.>; Author @.>
Subject: Re: [mingyaulee/Blazor.BrowserExtension.Samples] Extension gets installed but is grayed out on the Extensions menu (Issue #2)
This menu item is only available if there is any action registered in the manifest, which will show a popup when clicked. The sidebar example is showing how you can have Blazor injected into a page when using content script. You can see this if you navigate to Google's home page, there is a button that is injected and will show the sidebar when clicked.
—
Reply to this email directly, view it on GitHubhttps://github.com/mingyaulee/Blazor.BrowserExtension.Samples/issues/2#issuecomment-1399429665, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABACOHF37G3MYWR6WVCOFDDWTTVYVANCNFSM6AAAAAAUCQOLZE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Nested list binding doesn't work

Recent NancyFX has support for list binding https://github.com/NancyFx/Nancy/wiki/Model-binding but it didn't work for me.
For collection model binding test I created a stackoverflow-like Question model https://gist.github.com/zelid/6636968
I bind it like: https://gist.github.com/zelid/6636972
In HTML form I tried different things:
https://gist.github.com/zelid/6636987
And also:
https://gist.github.com/zelid/6637003
But it doesn't bind collection of Answers to the Question object while question's Name and Id are bind correctly.
Later I wanted to try binding a collection of comments to both Question and Answer objects.
What is wrong with it and how to make this nested collection bind correctly? Is it a bug or is it intended to bind either one level collection or non-nested object?

has anything been done with this of late?


Update office-365-worldwide-endpoints.md

Verified changes with Teams PG

@MSTejas please note in the source file that this is a generated file. You can only make changes at https://endpoints.office.net and not via a PR here

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@MSTejas please note in the source file that this is a generated file. You can only make changes at https://endpoints.office.net and not via a PR here

@MSTejas please note in the source file that this is a generated file. You can only make changes at https://endpoints.office.net and not via a PR here


Added EmbraceTelemetryService

Goal

Count public API calls
Group other general telemetry, such as disk usage (to be done in the future) or app attributes.

The information is sent privately on the current session span, so we pass EmbraceTelemetryService as a parameter to the SpansService, and fetch and clear the telemetry info every time we flush the spans (when the sessions end).

A few minor things. Are you planning to add to the integration tests to verify this?

@bidetofevil I added the attributes we expect to the session-end integration test. Do you think we should add more tests?


Added qmeta API

Added qmeta as global search engine API - https://api.qmeta.net
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Hey, sorry for the delay.
Please link to https://www.qmeta.net/api.html.

Hey, sorry for the delay.
Please link to https://www.qmeta.net/api.html.

It redirects automatically.

Unfortunately no, it links to https://www.qmeta.net/ right now.

Unfortunately no, it links to https://www.qmeta.net/ right now.

Of course. Try it with some Redirect-Checker like: redirect-checker.org
Or simply click the link by yourself.

I mean that the link you provide inside the readme is not http://api.qmeta.net nor http://www.qmeta.net/api.html.

Thank you 😄


404 Error on homepage

http://martyjs.org/guides/containers/index.html
Under further reading, the link to separating containers and components results in a 404.

Thanks


Envoy 1.21 Upgrade

Description
This PR is best reviewed commit by commit.
It contains the following:

edits to envoy.mk to use the commit at the top of datawire/envoy.git branch aidanhahn/v1.21-2
removal of --bootstrap-version from entrypoint
updates to the invocation of the envoy generate_go_protobuf.py tool
updates to protobuf generation to match updates made to the protos in envoy
go_package added to kat and agent protos
removal of the Envoy V2 API and Envoy V2 tests
tweaks to IR generation and unit tests to enable proper V3 testing and Envoy API use.

Before I get to commenting on the content, I can tell you that I'm going to want the history to be cleaned up.

Please remove the pytest-kat-envoy2* targets from builder.mk (to be fair: they didn't exist in commit you started from).

Please remove the GetEnvoyAPIVersion function.

What triggered the cluster name change in t_grpc_stats.py?

I almost want to ask you to split this in to a first PR that drops the v2-api, and then a second PR that upgrades Envoy.

python/ambassador/config/config.py and python/tests/kat/abstract_tests.py are still doing things with AMBASSADOR_ENVOY_API_VERSION.

The config.py:envoy_api_version() function should go away.

What triggered the cluster name change in t_grpc_stats.py?

According to the gRPC statistics documentation it is simply grouped by method instead of name.
apiVersion: getambassador.io/v3alpha1
kind: Module
name: ambassador
config:
    grpc_stats:
        # this will be ignored
        all_methods: true
        services:
            - name: echo.EchoService
              method_names: [Echo]

The behavior seems correct to me so I fixed the test.

So the t_grcp_stats.py change is related to the Envoy upgrade, not to the v2-api→v3-api switch?
Since users do care about watching stats, should we call out the naming change in the changelog?

Ugg, I'm not sure why go is putting new deps in their own require block in go.mod rather than adding them to the existing require block. Can you fix that?

It looks like you made made api/{agent,kat}/*.proto changes in the make generate commit? You should keep the human-made changes separate from the machine-made changes.

It looks like you made made api/{agent,kat}/*.proto changes in the make generate commit? You should keep the human-made changes separate from the machine-made changes.

I made no changes by hand. the differences you see are the result of commit 701a2c5

The AuthenticationGRPCV2Test should probably not go away, because as I said we aren't killing ext_authz v2.

Likewise for RateLimitV2Test.

It looks like you made made api/{agent,kat}/*.proto changes in the make generate commit? You should keep the human-made changes separate from the machine-made changes.

I made no changes by hand. the differences you see are the result of commit 701a2c5

Right, but then one of the later "regenerate" commits makes this change:
diff --git a/api/agent/director.proto b/api/agent/director.proto
index 97f1fc1d9..182535580 100644
--- a/api/agent/director.proto
+++ b/api/agent/director.proto
@@ -10,7 +10,7 @@ import "google/protobuf/timestamp.proto";

 package agent;

-option go_package = "github.com/datawire/ambassador/v2/pkg/api/agent;agent";
+option go_package = "./agent";

 service Director {
   // Report a consistent Snapshot of information to the CEPC.  This
diff --git a/api/kat/echo.proto b/api/kat/echo.proto
index a53a3630c..8b06faf80 100644
--- a/api/kat/echo.proto
+++ b/api/kat/echo.proto
@@ -2,7 +2,7 @@ syntax = "proto3";

 package echo;

-option go_package = "github.com/datawire/ambassador/v2/pkg/api/kat;kat";
+option go_package = "./kat";

 service EchoService {
     rpc Echo(EchoRequest) returns (EchoResponse) {}

It looks like you changed the default ENVOY_REPO back from https://github.com to git://github.com (I suspect a mistake during a rebase). GitHub killed off git:// support, which is why I had changed it to https;//.

Closing in favor of https://github.com/emissary-ingress/emissary/pull/4080


Installation Fails on Ubuntu with tunneling socket could not be established

When running npm install<EMAIL_ADDRESS>--save-dev. it fails with the following error:
npm info install<EMAIL_ADDRESS>><EMAIL_ADDRESS>install /home/me/code/projects/myproject/node_modules/node-sass
> node scripts/install.js
tunneling socket could not be established, cause=140168125925184:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:795:
npm verb unsafe-perm in lifecycle true
npm info postinstall<EMAIL_ADDRESS>><EMAIL_ADDRESS>postinstall /home/me/code/projects/myproject/node_modules/node-sass
> node scripts/build.js
module.js:340
    throw err;
          ^
Error: Cannot find module '/home/me/code/projects/myproject/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:929:3
Build failed

We are behind a proxy, but this is the only NPM package I've encountered that won't install with our configuration/
I've gotten a few older versions to install with a precompiled binary, but it still can't seem to find the libsass bindings and recommends reinstalling node-sass.
npm info install<EMAIL_ADDRESS>><EMAIL_ADDRESS>install /home/o572014/code/projects/rsam-ember-cli-app-template/node_modules/node-sass
> node build.js
`linux-x64-v8-3.14` exists; testing
Binary is fine; exiting

$ ember s
ember s --port 4300
version: 0.1.15
`libsass` bindings not found. Try reinstalling `node-sass`?
Error: `libsass` bindings not found. Try reinstalling `node-sass`?
    at getBinding (/home/me/code/projects/myproject/node_modules/ember-cli-sass/node_modules/broccoli-sass/node_modules/node-sass/lib/index.js:22:11)
    at Object.<anonymous> (/home/me/code/projects/myproject/node_modules/ember-cli-sass/node_modules/broccoli-sass/node_modules/node-sass/lib/index.js:188:23)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/me/code/projects/myproject/node_modules/ember-cli-sass/node_modules/broccoli-sass/index.js:5:12)
    at Module._compile (module.js:456:26)

The tunneling error is back from the brink: https://github.com/sass/node-sass/issues/588.
//cc @xzyfer

@DVG, please show the output of:
# you may need to prefix these with `sudo`:
$ mkdir /temp; cd /temp

$ npm install npmconf
$ node

then:
// This is node.js REPL
// The following code is extracted from node-sass/scripts/install.js#applyProxy
require('npmconf').load({}, function (er, conf) {
  var options = {};
  var proxyUrl;

  if (!er) {
    proxyUrl = conf.get('https-proxy') ||
               conf.get('proxy') ||
               conf.get('http-proxy');
  }

  var env = process.env;

  options.proxy = proxyUrl ||
                  env.HTTPS_PROXY ||
                  env.https_proxy ||
                  env.HTTP_PROXY ||
                  env.http_proxy;

  console.log(options);
});

after exiting the REPL:
# final cleanup
rm -r /temp

@DVG could you also supply a gist with the output of the following two commands
npm config list

and
env

https://gist.github.com/DVG/79a6fe2299af491509e3

The result of your script is:
undefined

{ proxy: 'https://proxy.mycompany.net:8000' }

{ proxy: 'https://proxy.mycompany.net:8000' } should do the trick!
Does this help: http://stackoverflow.com/a/26054219/863980?

So I've done some experimenting on a fork, and I can get past the tunneling socket problem, however, the build.js script still fails on the node-gyp bit:
npm info install<EMAIL_ADDRESS>><EMAIL_ADDRESS>install /home/me/code/projects/testapp/node_modules/node-sass
> node scripts/install.js
npm verb unsafe-perm in lifecycle true
npm info postinstall<EMAIL_ADDRESS>><EMAIL_ADDRESS>postinstall /home/me/code/projects/testapp/node_modules/node-sass
> node scripts/build.js
Building: /home/me/.nvm/v0.10.36/bin/node node_modules/pangyp/bin/node-gyp rebuild
module.js:340
    throw err;
          ^
Error: Cannot find module '/home/me/code/projects/testapp/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:929:3
Build failed

For the record, the tunneling socket seemed sensitive to the trailing slash I had in my NPM config, which was the only difference between the NPM config and the shell-level HTTPS_PROXY variable. I don't really know if that sort of thing is something you even care about, but since it doesn't seem to matter for the average NPM package I wanted to mention it.

I was able to install node-sass 1.1.4 with the proxy fix. I'm still unable to install the 2.0 line with the issues above, so I'm happy to keep helping figure out what's going on there.

@DVG, excellent findings and glad to know that proxy issue is sorted out. :+1:
Note that not all npm packages download binary in the installation step. However, since node-sass relies on a native binary, which is downloaded after the package itself is installed (hint: npm tasks).
I am not sure if the trailing slash is something unnecessary in all proxy servers scenarios? If they all defy then it is the issue when npm config are set and should be reported to https://github.com/npm/npm/issues.
The node-gyp step should not be triggered in first place for Ubuntu, since we are offering pre-build binary for Linux. To help us diagnose the issue, please make a gist showing the output of this set of commands: https://gist.github.com/am11/e5de3c49c219f0811e1d#comment-1386582.

Here you go: https://gist.github.com/DVG/5c965dd371bbd03ff8fc

@DVG, actually we have recently change the naming convention and it seems like you are installing node-sass from fork? Can you try installing it from the npm install node-sass from the server instead (with and without --save-dev)?
# tunneling issue was sorted out,
# now lets try a clean install:
sudo mkdir /temp
cd /temp
npm install node-sass

# success!
node -p "console.log(require('node-sass').renderSync({data:'a{b:c}'}))"
# some CSS output!

# cleanup
cd ~/
sudo rm -rf /temp

Also for the forked installation, try these steps:
git clone https://github.com/dvg/nodess --recursive
cd node-sass
git remote add node-sass https://github.com/sass/node-sass
git pull --rebase node-sass master
git submodule --recursive --init
npm install

# the next one requires gcc/g++ v4.6+ and python 2.7.x
node scripts/build -f

# success!
node -p "console.log(require('./').renderSync({data:'a{b:c}'}))"
# ./ because you are in the node-sass directory

From a fork? I don't think so. I did temporarily to experiment with the proxy code upstream (which is how I spotted the trailing slash thing), but since then have been only trying to install the official package
In any case:
npm install node-sass
npm info install<EMAIL_ADDRESS>><EMAIL_ADDRESS>install /home/me/temp/node_modules/node-sass
> node scripts/install.js
npm verb unsafe-perm in lifecycle true
npm info postinstall<EMAIL_ADDRESS>><EMAIL_ADDRESS>postinstall /home/me/temp/node_modules/node-sass
> node scripts/build.js
Building: /home/me/.nvm/v0.10.36/bin/node node_modules/pangyp/bin/node-gyp rebuild
module.js:340
    throw err;
          ^
Error: Cannot find module '/home/me/temp/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:929:3
Build failed

$ node -p "console.log(require('node-sass').renderSync({data:'a{b:c}'}))"
/home/me/temp/node_modules/node-sass/lib/index.js:23
    throw new Error('`libsass` bindings not found. Try reinstalling `node-sass
          ^
Error: `libsass` bindings not found. Try reinstalling `node-sass`?
    at getBinding (/home/me/temp/node_modules/node-sass/lib/index.js:23:11)
    at Object.<anonymous> (/home/me/temp/node_modules/node-sass/lib/index.js:197:23)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at [eval]:1:13
    at Object.<anonymous> ([eval]-wrapper:6:22)

npm install node-sass --save-dev
npm info install<EMAIL_ADDRESS>><EMAIL_ADDRESS>install /home/me/temp/node_modules/node-sass
> node scripts/install.js
npm verb unsafe-perm in lifecycle true
npm info postinstall<EMAIL_ADDRESS>><EMAIL_ADDRESS>postinstall /home/me/temp/node_modules/node-sass
> node scripts/build.js
Building: /home/me/.nvm/v0.10.36/bin/node node_modules/pangyp/bin/node-gyp rebuild
module.js:340
    throw err;
          ^
Error: Cannot find module '/home/me/temp/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:929:3
Build failed

$ node -p "console.log(require('node-sass').renderSync({data:'a{b:c}'}))"
/home/me/temp/node_modules/node-sass/lib/index.js:23
    throw new Error('`libsass` bindings not found. Try reinstalling `node-sass
          ^
Error: `libsass` bindings not found. Try reinstalling `node-sass`?
    at getBinding (/home/me/temp/node_modules/node-sass/lib/index.js:23:11)
    at Object.<anonymous> (/home/me/temp/node_modules/node-sass/lib/index.js:197:23)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at [eval]:1:13
    at Object.<anonymous> ([eval]-wrapper:6:22)

Ah you mentioned you were experimenting with fork, I thought the latter error is coming from there.
The fact that pangyp is not available is because I made a big blunder by adding it under dev-dependencies as opposed to main dependencies. I remedy it by this PR some 15 minutes ago: https://github.com/sass/node-sass/pull/717. :)
But the most important thing is; it should NOT fallback to manual build step at all (where it requires pangyp), especially on Linux, Mac and Windows. It seems like either the binary is not successfully downloading OR it is not compatible; the latter one is very troubling because I tested all binaries on Ubuntu which I built on old CentOS (for backward compatibility) before releasing (and it is a big hassle!):
vagrant@precise64:/temp/test-install$ uname -r
3.11.0-15-generic

vagrant@precise64:/temp/test-install$ cat /etc/issue
Ubuntu 12.04.5 LTS \n \l

I have checked your log again. You can try this approach:
cd into/myFav/project/dir
wget https://github.com/sass/node-sass/archive/v2.0.1.tar.gz
tar zxvf v2.0.1.tar.gz
cd node-sass
npm install
# should not try to invoke manual build! but we can just hope..

node -p "console.log(require('./').renderSync({data:'a{b:c}'}))"
# unsuccessful! :(

rm -rf vendor/lin*
mkdir vendor/linux-x64-node-0.10
wget https://github.com/sass/node-sass-binaries/blob/v2.0.1/linux-x64-node-0.10/binding.node vendor/linux-x64-node-0.10
npm install

node -p "console.log(require('./').renderSync({data:'a{b:c}'}))"
# success! :)

have exactly same issue with iojs 1.4.x (from nvm) on macos 10.10.2.
npm i node-sass
-
><EMAIL_ADDRESS>install /Users/yamalight/Projects/muse/front-end/node_modules/node-sass
> node scripts/install.js

Can not download file from https://raw.githubusercontent.com/sass/node-sass-binaries/v2.0.1/darwin-x64-iojs-1.4/binding.node

><EMAIL_ADDRESS>postinstall /Users/yamalight/Projects/muse/front-end/node_modules/node-sass
> node scripts/build.js

module.js:322
    throw err;
          ^
Error: Cannot find module '/Users/yamalight/Projects/muse/front-end/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:320:15)
    at Function.Module._load (module.js:262:25)
    at Function.Module.runMain (module.js:485:10)
    at startup (node.js:112:16)
    at node.js:863:3
Build failed

I assume because of the absence of the compiled binaries? When is the pangyp fixing PR getting pushed to NPM? :)

have exactly same issue with iojs 1.4.x (from nvm) on macos 10.10.2.

No. Your issue is that you cannot download binary for io.js 1.4, because node-sass v2.0.1 does not support io.js v1.4 but only io.js v1.2. The OP was having issue with https-proxy, which was fixed.
This issue was fixed by the OP. For separate issues, please search the issue tracker and if there is no issue reported as yours, feel free to report one.


Database no longer in debugging toolbar when using Propel on some upgrades

Is there something that changed that would remove the database debugging/icon from the web toolbar? I have a number of legacy sites that no longer have the database icon on the toolbar after updating to 1.5.15. I have 1 propel site that does have it still but I can't see anything different in the configuration files, nothing else changed on these sites. web_debug is set to true in settings.yml, DebugPDO is used in databases.yml, and the controller is set to load the debugging environment. Sites running doctrine are unaffected.

If you mean the SQL queries icon, it works for me.

Sorry, yes sql queries. Screenshots are for different sites on the same server, the config on them is almost identical though.
X.13

X.15

The icon will only appear if there are any Doctrine Events:
https://github.com/FriendsOfSymfony1/symfony1/blob/master/lib/plugins/sfDoctrinePlugin/lib/debug/sfWebDebugPanelDoctrine.class.php#L30-L33
I guess the listener of the profiler not registered. This is the callstack for a preConnect event:

This should return the symfony_profiler listener. If something is not set correctly you should get an error here.
    $dbm = sfContext::getInstance()->getDatabaseManager();
    $database = $dbm->getDatabase($dbm->getNames()[0]);
    $profilerListener = $database->getDoctrineConnection()->getListener()->get('symfony_profiler');

I think I had the same issue in the past and solved it by disabling persistent db connections in dev.
    param:
      persistent:
        value:      false
      classname:    DebugPDO

According to Propel docs, persistent connections need to be disabled for correct query counting/logging.


bitbag_sylius_cms_plugin_shop_block_render requires 2 queries

Looking into some performance optimisations, and one of the issues is that for each of my blocks rendered, a second query is done to fetch the image information (and/or translation). It seems to me that that's not really necessary. In my case I started digging into this as I don't need the image there, but that might be different for others.
I'm trying to figure out a method to get rid of these 2 queries, and move them into one. The repository method involved is BlockRepository::findOneEnabledByCode.
Queries on this URL: http://sylius.local/app_dev.php/nl_NL/block/block_code_here (next to the default Sylius ones):
SELECT b0_.id AS id_0, b0_.code AS code_1, b0_.type AS type_2, b0_.enabled AS enabled_3 FROM bitbag_cms_block b0_ WHERE b0_.code = ?  AND t0.locale = ? LIMIT 1

Parameters:[▼   1   "nl_NL" ]

Any suggestions here? Willing to PR if I find the improvement :-)

Adjusting the repository method in (just for testing an hardcoded locale):
    public function findOneEnabledByCode(string $code): ?BlockInterface
    {
        return $this->createQueryBuilder('o')
            ->addSelect('translation')
            ->innerJoin('o.translations', 'translation', 'WITH', 'translation.locale = :locale')
            ->leftJoin('translation.image', 'image')
            ->andWhere('o.code = :code')
            ->andWhere('o.enabled = true')
            ->setParameter('locale', 'nl_NL')
            ->setParameter('code', $code)
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }

Changes the queries, but doesn't take the image one away:
SELECT b0_.id AS id_0, b0_.code AS code_1, b0_.type AS type_2, b0_.enabled AS enabled_3, b1_.id AS id_4, b1_.name AS name_5, b1_.content AS content_6, b1_.link AS link_7, b1_.locale AS locale_8, b1_.translatable_id AS translatable_id_9 FROM bitbag_cms_block b0_ INNER JOIN bitbag_cms_block_translation b1_ ON b0_.id = b1_.translatable_id AND (b1_.locale = ?) LEFT JOIN bitbag_cms_block_image b2_ ON b1_.id = b2_.owner_id WHERE b0_.code = ? AND b0_.enabled = 1
Parameters:
[▼
  "nl_NL"
  "block_code_here"
]

SELECT t0.type AS type_1, t0.path AS path_2, t0.id AS id_3, t0.owner_id AS owner_id_4 FROM bitbag_cms_block_image t0 WHERE t0.owner_id = ?
Parameters:
[▼
  1
]

IMO the first query returns all the required information at once here, but it doesn't seem to be taken into consideration for the second query. Might be something to do with the fact that for the image an oneToOne relation is used.

Using v1.4.3 of the plugin btw.

Hello Stefan!
Thank you for your effort. @damonsson, kind of makes sense for me. Can I ask you to take a look at it?

I consider this topic to be outdated - closing.


Serialize Javascript Issue

🐛 Bug report
Current Behavior
Razzle dependancy 'serialize-javascript' has high risk vulnerability

Expected behavior
Razzle should not have  'serialize-javascript'  high risk vulnerability
Reproducible example
npm audit
Solution
Known issue with copy-webpack-plugin. https://github.com/vuejs/vue-cli/issues/5782
Suggest updating from copy-webpack-plugin 5.1.1 to 5.1.2

fixed in dev


pl_privatelink-proxy and dest_privatelink-proxy not appearing in the dropbox when creating the router

name: pl_privatelink-proxy
name: dest_privatelink-proxy

Can you guide what could be the issue ?

hi @saurabhakumbhare ,
could you please give me more context on what you are trying, on which step the issue is?

hi @saurabhakumbhare ,
sorry for my late response.
Since the PrivateLink service needs a couple of additional steps to approve the connection from Azure/AWS side, the service should be first initiated (e.g. name tfe-cal-s4-privatelink). After that, once you deploy the router, it will be automatically bound to that instance (pl_privatelink-proxy). The destination on Subaccount you need to add afterward, based on the assigned private hostname.
Hope this helps,
Kind regards,
Harut


chore(main): release yazi 1.15.5

:robot: I have created a release beep boop
1.15.5 (2024-10-08)
Miscellaneous

main: release fish 1.16.3 (#124) (5799516)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

yazi-v1.15.5
:sunflower:


ipopt: 3.12.10 -> 3.12.11

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ipopt/versions.
meta.description for ipopt is: '"A software package for large-scale nonlinear optimization"'.

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.12.11 with grep in /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11

directory tree listing: https://gist.github.com/91c5d7d0622f712f9f537b1d40968738

du listing: https://gist.github.com/cee661863cb87bf231f2e9e71b1c5c74

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:
nix-shell -p cachix --run 'cachix use r-ryantm'

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/ab050719baa0e82fa2c3761bbc26139bd11c064e.tar.gz -p ipopt
ls -la /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11

cc @abbradar for testing.

Success on x86_64-linux (full log)
Attempted: ipopt
Partial log (click to expand)
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/build/Ipopt-3.12.11'
make[1]: Leaving directory '/build/Ipopt-3.12.11'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11
shrinking /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11/lib/libipopt.so.1.10.11
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11/lib
patching script interpreter paths in /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11
checking for references to /build in /nix/store/y2arxy4h89c6087vl7gb2b01h842dhgm-ipopt-3.12.11...

Success on aarch64-linux (full log)
Attempted: ipopt
Partial log (click to expand)
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/build/Ipopt-3.12.11'
make[1]: Leaving directory '/build/Ipopt-3.12.11'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/9y8jnrs7xqkcwhd4jc992l6f7nnz832n-ipopt-3.12.11
shrinking /nix/store/9y8jnrs7xqkcwhd4jc992l6f7nnz832n-ipopt-3.12.11/lib/libipopt.so.1.10.11
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9y8jnrs7xqkcwhd4jc992l6f7nnz832n-ipopt-3.12.11/lib
patching script interpreter paths in /nix/store/9y8jnrs7xqkcwhd4jc992l6f7nnz832n-ipopt-3.12.11
checking for references to /build in /nix/store/9y8jnrs7xqkcwhd4jc992l6f7nnz832n-ipopt-3.12.11...


adds issue number to recommended branch name examples:

resolves #10

Basic question, but will the # character cause any problems with git if its in a branch name?

It hasn’t for me thus far. I’ve been using it all along.
On Mon, Jun 26, 2017 at 11:03 AM, Scott Williams<EMAIL_ADDRESS>wrote:

Basic questions, but will the # character cause any problems with git if
its in a branch name?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Tallwave/guidelines/pull/11#issuecomment-311136001,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALW5StiDMyZriMgN8fjoUPl2ZiuYG6Y5ks5sH_JvgaJpZM4OFeQO
.


add side bar visibility toggle

does this need to be rebased against master?  confused why all the IIIF stuff is in this when that's now on master.
This doesn't seem like it should be it's own widget but rather part of the editing UI when the EditableSkeleton is in edit mode or a property that is passed to the FixedSkeleton and development time.

does this need to be rebased against master? confused why all the IIIF stuff is in this when that's now on master.
This doesn't seem like it should be it's own widget but rather part of the editing UI when the EditableSkeleton is in edit mode or a property that is passed to the FixedSkeleton and development time.

@paltman - My bad. I just rebased. It is a property in state:
const getDefaultState = () => ({
  rightOpen: true,
  leftOpen: true,
  leftVisible: true,
  rightVisible: true,
  widgets: {
    mainWidget: null,
    right: [],
    left: [],
  },
});

The widget is only meant for demo purpose to highlight that the sidebars can be removed.

Cool
Sent from my iPhone

On Jun 5, 2019, at 9:37 AM, Michael Herman<EMAIL_ADDRESS>wrote:
does this need to be rebased against master? confused why all the IIIF stuff is in this when that's now on master.
This doesn't seem like it should be it's own widget but rather part of the editing UI when the EditableSkeleton is in edit mode or a property that is passed to the FixedSkeleton and development time.
@paltman - My bad. I just rebased. It is a property in state:
const getDefaultState = () => ({
rightOpen: true,
leftOpen: true,
leftVisible: true,
rightVisible: true,
widgets: {
mainWidget: null,
right: [],
left: [],
},
});
The widget is only meant for demo purpose to highlight that the sidebars can be removed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


chore(dex): upgrade dex version so it supports machine-machine auth

Dex only supports machine to machine auth from v2.38.0 see:
https://github.com/dexidp/dex/releases/tag/v2.38.0
In my eyes it makes sense to directly upgrade to latest (as of now v2.39.1), as it will probably be used more and more once #1554  is merged.
The aligning version of the chart would be 0.17.1 : https://github.com/dexidp/helm-charts/releases
For testing:
#!/usr/bin/env bash
KUBERPULT_URL="https://kuberpult.com"
TOKEN="$(gcloud auth print-access-token)"
ACCESS_TOKEN_JSON=$(curl -X POST "${KUBERPULT_URL}/dex/token" -H "Content-Type: application/x-www-form-urlencoded" --user kuberpult-dex:kuberpult-dex-secret --data-urlencode "connector_id=google-oidc" --data-urlencode "grant_type=urn:ietf:params:oauth:grant-type:token-exchange" --data-urlencode "scope=openid email profile" --data-urlencode "requested_token_type=urn:ietf:params:oauth:token-type:access_token" --data-urlencode "subject_token=${TOKEN}" --data-urlencode "subject_token_type=urn:ietf:params:oauth:token-type:access_token")
ACCESS_TOKEN=$(echo -n "${ACCESS_TOKEN_JSON}" | jq -r .access_token)

url="${KUBERPULT_URL}/environments/staging/locks/staging-lock"

curl -X PUT "$url" -d '{"message": "test app lock"}' -H 'Content-Type: application/json' --cookie "kuberpult.oauth=${ACCESS_TOKEN}"

with config.yaml
kuberpult:
  auth:
    dexAuth:
      baseURL: https://kuberpult.com
      clientId: kuberpult-dex
      clientSecret: kuberpult-dex-really-secret
      enabled: true
      installDex: true
      policy_csv: |
        p, role:Developer, CreateLock, *:*, *, allow
        p, role:Developer, DeleteLock, *:*, *, allow
        p, role:Developer, CreateRelease, *:*, *, allow
        p, role:Developer, DeployRelease, *:*, *, allow
        p, role:Developer, CreateUndeploy, *:*, *, allow
        p, role:Developer, DeployUndeploy, *:*, *, allow
        p, role:Developer, CreateEnvironment, *:*, *, allow
        p, role:Developer, DeleteEnvironmentApplication, *:*, *, allow
        p, role:Developer, DeployReleaseTrain, *:*, *, allow
        g, testorg:testgroup, role:Developer
      scopes: openid, groups, email, profile, federated:id
  cd:
    db:
      dbOption: "NO_DB"
  dex:
    config:
      connectors:
      - config:
          clientID: id
          clientSecret: secret
          redirectURI: https://kuberpult.com/dex/callback
        id: google
        name: Google
        type: google
      - config:
          issuer: https://accounts.google.com
          userNameKey: sub
          getUserInfo: true
          scopes:
          - openid
          - email
          - profile
          - audience:server:client_id:kuberpult-dex
        id: google-oidc
        name: Google OIDC only for machine login - do not use
        type: oidc
      issuer: https://kuberpult.com/dex
      staticClients:
      - id: kuberpult-dex-public
        secret: public-really-secret
        name: kuberpult-public
        public: true
      - id: kuberpult-dex
        name: kuberpult
        public: true
        redirectURIs:
        - https://kuberpult.com/callback
        - https://kuberpult.com/dex/callback
        secret: kuberpult-dex-really-secret
        trustedPeers:
        - kuberpult-dex-public
      storage:
        type: memory

However the better way would be to use a public client which is allowed to be accessed by the private one:
ACCESS_TOKEN_JSON=$(curl -X POST "${KUBERPULT_URL}/dex/token" -H "Content-Type: application/x-www-form-urlencoded" --user kuberpult-dex-public:public-secret --data-urlencode "connector_id=google-oidc" --data-urlencode "grant_type=urn:ietf:params:oauth:grant-type:token-exchange" --data-urlencode "scope=openid email profile audience:server:client_id:kuberpult-dex" --data-urlencode "requested_token_type=urn:ietf:params:oauth:token-type:access_token" --data-urlencode "subject_token=${TOKEN}" --data-urlencode "subject_token_type=urn:ietf:params:oauth:token-type:access_token")

However, this adds e.g.
  "aud": [
    "kuberpult-dex",
    "kuberpult-dex-public"
  ],

to the claim which would've to be supportend in the verify method.
Then you can remove the public: true in the kuberpult-dex client.

Closed by #1588.


Bulleted list doesn't render properly in packaged apps

Bug Report
Steps
Add a bulleted list to a React application (<List bulleted><List.Item /></List>).
Expected Result
Expect to see a bulleted list:

Actual Result
This works for apps in development. It does not work once the app is packaged. The packaged app shows:

In production I can see this in the console:

but in development it looks like this:

Version
0.88.2
Testcase
n/a this is a bug with a packaged app vs a development app

@yarhamjohn those CSS files are not included in this project. Those come from the parent Semantic UI project. SUIR does not do any kind of processing to the CSS files you are using. It only applies SUI class names.
If there are inconsistencies in the CSS you are seeing, it is likely that your build tooling is modifying your CSS in some way. Usually your build system has configuration options to enable source maps that could help you locate where those compiled files are coming from.


Dev mode doesn't work when adding helm extension

When adding the Helm extension from the https://quarkiverse.github.io/quarkiverse-docs/quarkus-helm/dev/index.html, the dev mode does not work anymore and also tests have issues. I've seen some PRs regarding the fact things will change with Dekorate 2.9.0, but for the moment the Helm part is not working as expected for Quarkus. Any idea how we can tweak that to work for Quarkus?

Fixed


NavLink forgets its classes when set active?

I have a Blazor 0.3.0 standalone app i've been toying with for a while and i came across an issue.
This is based off the VS Blazor template, And my NavMenu.cshtml is almost the same in regards of structure. I retained the <NavLink Class="nav-link" .. > form, only changing the href and adding data attributes on it.
When the link is set to active, intead of adding 'active' to the class list - it overrides it, losing all the classes that were applied, including nav-link. This messes up with styling.
I didn't see anything applicable in 0.4.0 changes; I might have missed something, though.
Thoughts?

I believe this is related to #905

It is? It seems to discuss a different customization of navigation components/links, rather than possible buggy/odd behavior in the current one. Maybe i misunderstood it.

Since class is a DOM attribute rather than a component parameter, it needs to be given in lowercase as is the convention for DOM attributes. That is, use class="some-class" and not Class="some-class".
I appreciate that this is an obscure detail and can easily seem confusing. If more people report this over time, we'll try to change NavLink so that it's specifically insensitive to the casing of that attribute name.

Shouldn't this be a warning or at least documented, then? Maybe a syntax/linter check.

Maybe. We're building up information about what people trip up on as we go.