Update active-directory-v2-limitations.md

fixed small typo

@dstrockis It looks like this PR is now just a typo fix. Please @sign-off as soon as possible. Thanks.


add stateful Info to json format

solves #240

Should decide about whether to be consistent with already existing mechanism to compute unidirectional connections subset for endpoints connectivity analysis in the json format.
Should also decide about whether data_model changes are required.
For now, closing this PR.


'LEVIR_CD_Dataset' is not in the dataset registry

When I run tools/test.py, there exists an error?
2023-04-18 16:34:01,005 - mmseg - INFO - Multi-processing start method is `None`
2023-04-18 16:34:01,005 - mmseg - INFO - OpenCV num_threads is `48
Traceback (most recent call last):
  File "/home/user/open-cd/tools/test.py", line 321, in <module>
    main()
  File "/home/user/open-cd/tools/test.py", line 193, in main
    dataset = build_dataset(cfg.data.test)
  File "/home/user/mmsegmentation/mmseg/datasets/builder.py", line 85, in build_dataset
    dataset = build_from_cfg(cfg, DATASETS, default_args)
  File "/home/user/miniconda3/envs/opencd/lib/python3.9/site-packages/mmcv/utils/registry.py", line 61, in build_from_cfg
    raise KeyError(
KeyError: 'LEVIR_CD_Dataset is not in the dataset registry'

I didn't change anything in opencd/datasets/levir_cd.py. How can I fix this error. :sob:

emmm...
try from opencd.datasets import LEVIR_CD_Dataset in test.py ? I am not sure..

Thanks, it is work for me. 😊


Import from scanpy object or local files

Good day, wanted to ask what is the way to import objects from either a scanpy or from local h5 and image files. I want to run tutorials but not sure where to start loading my own data. Probably missing something obvious! :) Thanks!
Nenad

hi @nbartonicek , thank for the interest in squidpy.
If you want to read visium data, you can use scanpy.read_visium https://scanpy.readthedocs.io/en/stable/generated/scanpy.read_visium.html and for the large tissue image, use the squidpy image container https://squidpy.readthedocs.io/en/latest/auto_tutorials/tutorial_image_container.html
if you don't have visium data, you can still import molecular, spatial and image data in anndata + iamge container easily, here's a walkthrough on how to do that: https://squidpy.readthedocs.io/en/latest/auto_tutorials/tutorial_read_spatial.html
let us know if you encounter any problem!

Thank you for the prompt reply and all the information!
What solved my problem for small visium "hires" images:
image=adata.uns[spatial_key][library_id]["images"]["hires"]
And for big tiffs:
from PIL import Image
source_image_path="tissue_hires_image.tiff"
Image.MAX_IMAGE_PIXELS=None
im = Image.open(source_image_path)
image = numpy.array(im)
Followed by:
img = sq.im.ImageContainer(image_hires)
Thanks again! :)
Cheers,
Nenad

thanks for your reply. I see that works, but according to this code snippet

And for big tiffs:
from PIL import Image
source_image_path="tissue_hires_image.tiff"
Image.MAX_IMAGE_PIXELS=None
im = Image.open(source_image_path)
image = numpy.array(im)

Followed by:
img = sq.im.ImageContainer(image_hires)

you wont be using the big tiff file in the image container (as you are assigning the visium hires, which is way lower resolution that the tiff). Is that whhat you are in interested in doing ?

@nbartonicek is the issue solved?

@nbartonicek I'll close this, please reopen if you have further questions

Hi! I am relatively new to using squidpy as well. I just finished processing my 10X visium data using the scanpy tutorials. I am having the same issue, where I am unsure how to add my 10X highres image from the cell ranger output to the image container. Is it as simple as setting img = (path_to_tissue_highres_image.png)? I dont believe there is a tiff file in the cellranger outs folder I have for my experiment.
Thank you!!!

Thank you! When I save the processed 10x visium files, I initially re-imported it for analysis using the sc.read_h5ad(). Is that incorrect? Should I be reading in the saved, processed data using the read.visium function?
For reference, after processing my data I used the adata.write() to save it as a h5ad. Should I be doing something else?

no then it's fine, you can read and write h5ad as you like. the reading functions for spatial data are just to read the "raw" files. Sounds good for library_id, it just needs to be a unique id

Thanks again. I am still having difficulty creating the image container object from the high res image. I understand that the image is stored in the anndata object in the adata.uns['spatial'][<library_id>] slot. This means that I don't have to load in another dataset/file after loading in the processed H5ad file that I made, correct?  Both the H&E Visium tutorial and the Import spatial data in AnnData and Squidpy tutorials aren't informative on how to make the image container object after processing the 10X data yourself and having 1 processed anndata file from it.
The H&E tutorial has you load in a seperate file for the image to create the image container
The tutorial for loading the anndata writes an original image.
I may just have a fundamental misunderstanding of where this image comes from in order to put it into the image container. I watched your seminar on youtube that came out a week ago where you went over the "import spatial data in AnnData and Squidpy" tutorial (https://www.youtube.com/watch?v=btqPgui8gpg&t=3545s) (which was very helpful and informative in many regards) but it didn't help me with this problem.
Thanks so much again, sorry I just can't seem to find the way to extract the image from the H5ad processed file and make the image container

Yes this is great! thank you!!

Hi,
I am trying the https://squidpy.readthedocs.io/en/stable/auto_tutorials/tutorial_visium_fluo.html tutorial.
But I am having issues assigning the hires figure from my own visium data loaded via the command sq.read.visium() to img = in squidpy.
I could not figure it out using the info above.
Spent hours trying different things.
Unfortunately the tutorial uses this command img = sq.datasets.visium_fluo_image_crop()
thank you!


Application error

https://agroicapp.herokuapp.com/      not working plz update your project

i have down the deployment because this is not using anymore. thanks


Added a small script to auto-delete local branches which got merged into the main branch via a PR

This will be helpful for outside contributors to auto-delete unneeded local branches which already got deleted on GitHub due to a pull request merge.

Thanks for making this effort. I don't think we should do this though... It would likely be helpful as a small dot file that you could probably publish in a personal repo.

So you're saying this is better off in another repo entirely?

Yeah I agree with @li-boxuan -- I don't think we should delete branches from users' local workstations.
@temotskipa you might be able to make this script part of your personal git workflow


Remove unused light-client leftover

Remove some leftover light-client code.
SyncingEngine instantiated here: https://github.com/paritytech/substrate/blob/91e0d66258a5143e2f4fa9bbf591d0ded4cd44b0/client/service/src/builder.rs#L830
Can not be light client : https://github.com/paritytech/substrate/blob/91e0d66258a5143e2f4fa9bbf591d0ded4cd44b0/client/network/common/src/role.rs#L100-L107

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2575434


Support (some) Projection by normalizing them away

fixes #260
The main trick is to build on the support for instance (c.f. #115) and when given a def_id with Projection in it, use the rustc API to normalize away the projections prior to lower_fn_sig etc.

Nice so the advantage here is the regions are NOT erased?
On Fri, Dec 23, 2022 at 6:07 PM Nico Lehmann @.***>
wrote:

@.**** requested changes on this pull request.
In flux-middle/src/rustc/lowering.rs
https://urldefense.com/v3/__https://github.com/liquid-rust/flux/pull/264*discussion_r1056714222__;Iw!!Mih3wA!CiC3DbRD5QYYrbMt1vlalmQQPHCAxD4eW3mrJp7br2wEg9tHXzja283mQs4253c7gQQUS_zYafl3KJNW5OQwOdxJ$
:

let fn_sig = if !fn_sig.has_projections() {

   fn_sig

} else {

   let param_env = tcx.param_env_reveal_all_normalized(def_id);

   match tcx.try_normalize_erasing_regions(param_env, fn_sig) {

       Ok(fn_sig) => fn_sig,

       Err(_) => {

           return Err(errors::UnsupportedFnSig {

               span,

               reason: "Sorry, projections are not yet supported!".to_string(),

           })

       }

   }

};

We are not doing anything interesting with regions yet, but we want to, so
we shouldn't be erasing them. For example, the following fails right now
trait Trait {
type A;}
impl Trait for i32 {
type A = i32;}
fn foo(x: &i32) -> ::A {
*x}
I don't understand exactly what's going on with the obligations and
InferCtxt but the following seems to work without erasing the regions
pub fn lower_fn_sig_of(tcx: TyCtxt, def_id: DefId) -> Result<PolyFnSig, errors::UnsupportedFnSig> {
let fn_sig = tcx.fn_sig(def_id);
let span = tcx.def_span(def_id);
let param_env = tcx.param_env(def_id);
match tcx
.infer_ctxt()
.build()
.at(&ObligationCause::dummy(), param_env)
.normalize(fn_sig)
{
Ok(fn_sig) => {
lower_fn_sig(tcx, fn_sig.value)
.map_err(|err| errors::UnsupportedFnSig { span, reason: err.reason })
}
Err(_) => {
return Err(errors::UnsupportedFnSig {
span,
reason: "Sorry, projections are not yet supported!".to_string(),
})
}
}}
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/liquid-rust/flux/pull/264*pullrequestreview-1229494943__;Iw!!Mih3wA!CiC3DbRD5QYYrbMt1vlalmQQPHCAxD4eW3mrJp7br2wEg9tHXzja283mQs4253c7gQQUS_zYafl3KJNW5KJ0OSK4$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4OBDYD6AZEOZ4MGDJV3WOZLGTANCNFSM6AAAAAATHGZDCI__;!!Mih3wA!CiC3DbRD5QYYrbMt1vlalmQQPHCAxD4eW3mrJp7br2wEg9tHXzja283mQs4253c7gQQUS_zYafl3KJNW5AUabSf_$
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah, it seems that calling normalize on At doesnt erase regions, but you need and InferCtxt and an ObligationCause to get ab At|, which I don't quite understand.

Interesting, these were alluded to in the replies here
https://internals.rust-lang.org/t/accessing-the-defid-of-a-trait-implementation/
Still lets just use normalize as it seems like a strict and local
improvement over the status quo!

Hmm I can't get this to compile, something about adding
rustc_infer::infer::TyCtxtInferExt
to the dependencies?

NVM figured it out, have applied your patch.


Arvo

I really like Arvo ( http://www.google.com/webfonts/specimen/Arvo ).
What do you think about including it?

Maybe like this markup and blockquote by Leonardo da Vinci?
as last section in index.html

Simplicity is theultimate sophistication.
— Leonardo da Vinci

and also a scss for application.scss
// Vinci
// ***************************************************************************
&#vinci {
background: #121212;
padding: 300px 0;
.text {
  margin: 0 auto;
  width: 900px;
}

blockquote {
  color: #fff;
  font: 300 80px/80px "Arvo";
  text-align: left;

  span {
    font-size: 53px;

    span {
      margin-left: 70px;
    }
  }

  footer {
    color: f4f4f4;
    font-size: 26px;
    text-align: right;
  }
}

blockquote:before {
  content: open-quote;
}
blockquote:after {
  content: close-quote;
}

}

Yeah, looks good!
I have a small problem with Arvo, though: the word-spacing is to small.
I am not sure if you should twist that manually since people will expect the same output when using the font.


Interwiki

Interwiki implementation is going quite well, except it's making the already complex code more complex. Well, whatever.
Migrate to links.Link interface from links.LegacyLink. Progress:

[x] Rocket links
[x] Image galleries
[x] Transclusion (show error)
[x] Inline links

After migrating everything to the new interwiki-enabled type, gotta fix those things in the tools package

The rest of the interwiki stuff is on the Mycorrhiza's side, not Mycomarkup's. Closing the issue


fix(dw-150):Guild page broken on mobile

Fixes dw-150

Great work @c2j! Thanks for your contribution!


[wip] Default ApplyToKeys

What does this PR do?
Fixes #645
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Closing this, should be adressed as part of ongoing Data Pipeline V2 refactoring


Fix #62 - Use the packages entry if part of the config

In theory, everything should work ... in practice:
ImportError: no module named 'mip'

I've asked in MicroPython discord channel about this, let's see how that goes: https://discord.com/channels/574275045187125269/1022482520845070407/1164218269134237860

Closed in favor of https://github.com/pyscript/polyscript/pull/73


Fix a bug where different tags of WebClient are registered to Prometh…

…eus registry
Motivation:
After Armeria introduces serviceName #2780,
MetricCollecting{Service, Client} register a service name as a tag if the RequestLog.serviceName is not null.
The nullable serviceName could a problem when a key is registered with different tags.
For example, a gRPC client that send requests using com.example.Foo stub,
MetricCollectingClient recodes metrics such as foo_pending_acquisition_duration_seconds
with tags of http_status, method and service whose value is com.example.Foo.
However, a WebClient sends requests, it only records the metrics with  http_status and method tags.
This could cause problems at Micrometer's Prometheus registry.
Prometheus requires that all meters with the same name have the same set of tag keys.
There is already an existing meter named 'armeria_client_pending_acquisition_duration_seconds' containing tag keys [http_status, method].
The meter you are attempting to register has keys [http_status, method, service].

Modifications:

Add 'none' to the value of service tag if RequestLog.serviceName() is null.
Remove @Nullable annotation from RequestLog.name() and Request.fullName().
They do not return null values always.

Result:
A Prometheus registry does not throw an IAE, when you register a metric prefix key to the heterogeneous clients.

Thanks, @ikhoon


CAN mode and bitrate configuration in Ansible

This PR adds bitrate configuration for each CAN interface. This helps to properly configure the bitrate, specially in John Deere machines, where implement and tractor bus run at different speeds. Also adds the option to enable listen-only mode to avoid locking the CAN bus when the bitrate is set to a different value.

Are the changes to cell logger intended?


Improve file watcher handling performance

File watchers currently add some unnecessary overhead in several places:

for all OSs: we use Files.isDirectory() to check if the parent directory to watch is indeed an (existing) directory here: https://github.com/gradle/gradle/blob/3f9b7efefeddafdfb8dfd048d342011d34321abb/subprojects/snapshots/src/main/java/org/gradle/internal/vfs/impl/DefaultWatchingVirtualFileSystem.java#L82-L85
This is a slow operation, and it amounts to 130 ms of processing for ~1800 watched directories on macOS.
We could fix this easily for Windows, where in the end we only care about the drive letters to watch.
We could also get rid of most of the overhead on macOS (provided we continue using hierarchical FSEvents) by ignoring directories whose ancestors are already watched earlier.
On Linux we would probably need something more elaborate, reusing the VFS itself to find the first existing parent candidate. We'd need some handling for when the directory has been removed in the meantime. (The latter is a possibility with other OSs as well, but the window for the race condition is smaller for them.)

we canonicalize the paths we watch explicitly. This shouldn't be necessary, as the OS should determine the canonical path on its own. We will most probably switch (back) to using canonicalized paths in the VFS later anyway (to let the VFS work with symlinks). This seems to be responsible for a smaller part of the overhead.

See async profiler results for macOS: onlyVfsRetention.zip

cc: @gradle/build-cache

For 1) we also don't need to check if the parent is a directory unless the item we are checking is a missing file.

We could also drop any missing files from the VFS at the end of the build.

If we don't check for the existence of parent directories we know should be there, we are down to around 45-50 ms (from 130) for 1). Step 2) is still at 28 ms.

Another potential improvement is to change the API to native-platform to avoid conversion of the watched paths. Right now we do String -> Path -> String -> File -> File.canonicalFile -> String with each watched path.
If we removed the canonicalization on the native side, we could either use String or Path, though using Path make the feature unavailable on Java 5 and 6, which is still supported by native-platform.

Here's another async profiler result, after https://github.com/gradle/gradle/pull/11890/commits/c84814f902ccc0b3bd381d34824821c67655ed81: onlyVfsRetention-after-directory-lookup.zip

Here's how I took the Async profiler measurements using https://github.com/gradle/santa-tracker-performance:
$ gradle-profiler/build/install/gradle-profiler/bin/gradle-profiler \
    --project-dir santa-tracker-java \
    --scenario-file santa-tracker-java.scenarios \
    --gradle-version "$GRADLE_VERSION" \
    --warmups 2 \
    --iterations 10 \
    --profile async-profiler \
    onlyVfsRetention

We are getting closer to the results without the watcher registration overhead. Here's where we are on Linux:
Results without the optimizations in https://github.com/gradle/gradle/commit/c84814f902ccc0b3bd381d34824821c67655ed81:

Results with the optimizations:

Results with no registration overhead from end of November (i.e. our ideal goal):

I've ran async profiler on the Linux variant which shows the same behavior for registering the watchers (even though it needs to register a lot more than macOS):
Spent 29 ms figuring out what to watch
Watching 1836 directories to track changes between builds
Spent 47 ms watching 1836 directories for file system events

...but it takes a lot longer to actually read the changes:
Virtual file system retention is an incubating feature.
Marking as changed since last build: /home/lptr/Workspace/gradle/santa-tracker-performance/santa-tracker-java/village/src/main/java/com/google/android/apps/santatracker/village/SnowFlake.java
Received 1 file system events since last build
Spent 14 ms processing file system events since last build

I'm surprised that stopWatching() doesn't show up in the stacks despite taking a comparable amount of time.
See profiler output:
linux-with-no-directory-check.zip


Allow public access to ServiceJsonRpcDescriptor.CreateFormatter

There are current implementations that wrap an existing ServiceJsonRpcDescriptor into another ServiceJsonRpcDescriptor-derived class.
These implementations need to call CreateFormatter on the inner descriptor and they currently use reflection to do so.
Unfortunately CreateFormatter is protected internal, so it cannot be made public without breaking the compilation of extended classes that are overriding this method.
For this reason, I propose creating a new public method to make the CreateFormatter capability publicly accessible.

As I suspected, we already have a pattern to fix this. Have you seen the DelegatingServiceJsonRpcDescriptor class? When you are wrapping another descriptor with a custom one, your custom one is supposed to derive from DelegatingServiceJsonRpcDescriptor, which lets you call *your own* CreateFormatter` method and it invokes the inner one.


[BC Idea]: Allow setting SASToken as SecretText in Storage Service Authorization

BC Idea Link
https://experience.dynamics.com/ideas/idea/?ideaid=699c8b0d-c3f5-ee11-a73d-6045bd808754
Description
Currently you can set SASToken only as Text. Adding an overload with a SecretText parameter would allow to secure the value from isolated storage to Auth Codeunit.
I will provide the implementation for this BC Idea

[X] I will provide the implementation for this BC Idea
Internal work item: AB#524632

@ChrisBlankDe - I have some code that uses the Authorize procedure in Codeunit 9088 "Stor. Serv. Auth. Ready SAS". Everything worked ok in V24 but since the changes from this PR have been introduced in V25 the Authorize function returns a blank value. Are you able to test this on your end to check if it is a bug and not just an issue that I am experiencing?


Sprite3D and MeshInstance -> BillboardY property doesn't work

Hello together,
in Godot 1.0 stable the BillboardY property from Sprite3D seems broken.
To test create a Sprite3D node, add a texture and activate the BillboardY property in the editor and it should nothing happens. The normal Billboard property works fine.
I use Godot 1.0 stable under Kubuntu 14.04.

I had a look in the sources but couldn't find where it's implemented. Maybe a todo or overlooked.

I had a go at fixing this here #1368.

Hello not-surt, thank you for your help and work. :) A small additional information, the billboard_y property from the MeshInstance is broken too.

The bug is still valid in the current master branch according to https://github.com/godotengine/godot/issues/2061#issuecomment-156806560

I can confirm this bug too, Billboard Y doesn't seem to do anything.

is still not working :(
even from code FLAG_BILLBOARD_FIX_Y

Fixed by #5962.


[BUG] "Incomplete" Filter doesn't do anything

Complete file "__init_cpp_exceptions". Shown with Relax Relocation Diffs off

Incomplete filter is off and complete file is shown

Incomplete filter is on and the file is still shown

It seems like the filter doesn't actually do anything
Hidden filter works as expected
All files in my project are Diffable, so I cannot test this one easily

Once you mark one of the objects complete (or “Matching”), this will filter them out.

I have no idea how I forgot about that
For some reason just assumed objdiff would know when it's matched
Thanks


Remove duplicate home tag in spark Chart

Spark chart contained duplicate home tag.
This caused Kubernetic to crash https://github.com/harbur/kubernetic-issues/issues/18
Not sure how if this should up the version number or not since it's clearly altering the hash of the release tarball.

I signed it!


The login button on the explore page is missing.

Describe the bug
The login button on the explore page is missing.
Steps to reproduce
The login button on the explore page is missing.
Screenshots or additional context
No response

Clicking the logo/icon takes you to the login/auth page

I don't want to open the login page by clicking on the logo. Because a lot of unregistered visitors don't do that.
It is better to be able to register directly out of the login button.

I think this issue has been resolved in the latest version.
Feel free to reopen it if needed.


Sail showns errors after cloning a project.

Sail Version: v1.12.10
Laravel Version: 8.75.0
PHP Version: 7.4.26
OS: Windows / Linux / macOS Windows (WSL2)

Description:
When starting with a new Sail project everything works as expected.
Every Sail command works as expected.
The problem beginns when I upload my project to a new Gitlab Repository and have to download it on another machine,
After doing a git clone **** on every other machine I get errors on every single sail command.

Here are the command I use when cloning the project:

git clone git@lsv-eddyn-gitlab-prd-01.local.gruppe.de:it-internet/my-project.git
cd my-project
composer install
cp .env.example .env
php artisan key:generate
sail up -d

Am I forgetting anything?
It is the exact same .env file I have on the machine, where Sail works without errors.

This seemed to be a problem in PHPStorm.
For some reasons it was changing the line endings and that was exactly the problem.

This seemed to be a problem in PHPStorm.
For some reasons it was changing the line endings and that was exactly the problem.

Same issue here, line endings were at CRLF and needed to be:

Changed to LF
Rebuild to copy new file across: sail build


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Zefau/ioBroker.nello

[x] [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: "npm owner add bluefox iobroker.nello"

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Added iko.soy

The webring icon is at the bottom of every page.
https://iko.soy

Better :) Thank you.
Welcome aboard Илка


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1582 ms

Resolved: Bofrost.lu performance has improved in 9f6c107 after 9 minutes.


Add verbose logging support during mountVolume func

To better debug various environments and distributions the mountVolume function should log more detailed information when in verbose.

Additional debug added to NFS mountVolume call.  In 0.13 release

Acctually I thought more about passing a -v flag to the actuall mount call to get more information about what happend. Means instead of
cmd = fmt.Sprintf("mount -t nfs4 -o %s %s %s", opts, source, dest)

do the follwoing when verbose flag is given to plugin
cmd = fmt.Sprintf("mount -t nfs4 -v -o %s %s %s", opts, source, dest)

Ok, gotcha thanks

Ok, version 0.14 released with verbose support in mount.  Re-open if it doesn't work.  I tested it and am getting the mount feedback now


Properly color pragma contents in hyperlinker

The hyperlinker backend now classifies the content of pragmas as TkPragma. That means that in something like {-# INLINE foo #-}, foo still gets classified as a pragma token.
Given the following source:
module Kaa where

kaa :: Int -> Int
kaa = (+1)
{-# INLINE [1] kaa #-}

koo :: Int -> ()
koo _ = ()

Output prior to this PR:

New output:

Apart from this, the only thing that remains to be done for the hyperlinker backend is to add a case to the classify/inPragma functions for the new ITcolumn_prag that https://phabricator.haskell.org/D4336 will introduce.

Alec, looks good. Do we need this in ghc-8.4? I convinced Ben to merge https://phabricator.haskell.org/D4336. Is this PR needed to make use of D4336?

Do we need this in ghc-8.4? I convinced Ben to merge https://phabricator.haskell.org/D4336. Is this PR needed to make use of D4336?

Yes, this is needed to support ITcolumn_prag (added in D4336). Thanks!

Oh boy, alright then!

This is a PR against master which does not pass travis right now. I applied your patches to ghc-8.4. Closing this PR.

@alexbiehl Thanks, and sorry for any confusion. I opened this against master since the lexer changes where at the time only there.
Now that I see what the release process is like, I'll definitely try to make my patches smaller.


Interceptor's request.body().writeTo(buffer); causes "java.net.ProtocolException: unexpected end of stream" with MultipartBody Post

I'm POSTing a MultipartBody using OkHttpClient - wire protocol is Http2.
With
        HttpLoggingInterceptor httpLoggingInterceptor = new HttpLoggingInterceptor();

        httpLoggingInterceptor.setLevel(HttpLoggingInterceptor.Level.BODY);

Every attempt to POST the MultipartBody fails with variations of java.net.ProtocolException: unexpected end of stream
As per the following traceback:
        at okhttp3.internal.connection.Exchange$RequestBodySink.close(Exchange.kt:244)
        at okio.RealBufferedSink.close(RealBufferedSink.kt:287)
        at okhttp3.internal.http.CallServerInterceptor.intercept(CallServerInterceptor.kt:63)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.kt:34)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.kt:95)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.kt:83)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.kt:76)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.logging.HttpLoggingInterceptor.intercept(HttpLoggingInterceptor.kt:221)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at org.asamk.signal.manager.config.ServiceConfig.lambda$getServiceEnvironmentConfig$0(ServiceConfig.java:46)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.connection.RealCall.getResponseWithInterceptorChain$okhttp(RealCall.kt:201)
        at okhttp3.internal.connection.RealCall.execute(RealCall.kt:154)

A workaround I have implemented and tested is to have an interceptor write the body to a buffer and then create a new Request with the MultipartBody represented instead as raw bytes. This is logged fine and completes fine, suggesting that somewhere inside MultipartBody is a hidden (unlabeled?) oneshot or oneshot-like abstraction.
An implementation of this workaround can be found at https://github.com/firedexofficial/signal-cli/blob/bad99253bc52bd16ae9065ed9f82bf62a952a2d9/lib/src/main/java/org/asamk/signal/manager/config/FixHttpLoggingInterceptor.java
Adding this interceptor before HttpLoggingInterceptor enables the request to complete successfully.
This resembles https://github.com/square/okhttp/issues/2738 in terms of issue/traceback and I see folks in the comments using HttpLoggingInterceptor.  See also: https://github.com/square/okhttp/issues/7779 and https://github.com/wordpress-mobile/WordPress-Android/issues/5690 and https://github.com/square/okhttp/issues/2738#issuecomment-515960916.
This appears to be at least a footgun with HttpLoggingInterceptor (incompatible with MultipartBody POSTs) and possibly a subtle implementation issue with OkHttp. None of the other workarounds (connection: close / retry on failure) worked, but setting "HttpLoggingInterceptor.Level.HEADERS" instead of BODY did the job, cluing me into where the issue existed.

version: com.squareup.okhttp3:okhttp:4.11.0

I think it's because one of the Parts of the MultipartBody is oneshot.  So I've put up a PR to check for this, which will stop logging.
Can you confirm that some part of the form is one shot in your case?

Hopefully this should be fixed in next 5.0 release.  I'll discuss with @swankjesse whether to backport to 4.x.
But if you are using that workaround, it might be simpler to ensure you are not using OneShot parts. Or if you are not, then we can reopen.

@yschimke @i-infra
I am facing this issue intermittently. When i use HttpLoggingInterceptor, its not happening or is very difficult to reproduce.
I am not really sure what is OneShot part.
Can you please let me know what is the workaround?
is it to do this:

Request request = chain.request();
        Buffer buffer = new Buffer();
        Request newRequest;
        if ( request.body() != null ) {
            request.body().writeTo(buffer);
            newRequest = new Request(request.url(), request.method(), request.headers(), RequestBody.create(buffer.readByteArray(), request.body().contentType()), request.getTags$okhttp());
        }
        else {
            newRequest = new Request(request.url(), request.method(), request.headers(), RequestBody.create(buffer.readByteArray()), request.getTags$okhttp());
        }
        return chain.proceed(newRequest);


About deleting OPENSSL_cleanse（）

Hello,
It is described in a nginx-1.6.2-async.l.0.1.0-008 document as follows:

Performance Optimization to OpenSSL (this is optional step)
This step is optional. During our testing, we've found OPENSSL_cleanse() to be the bottleneck limiting performance. OPENSSL_cleanse() is a type of memory scrubbing functionality that clears the memory that used to hold the keys. Customers that already use HW accelerators might have already come across this as a bottleneck and might already have it optimized in their systems. For customers that do not use HW accelerators they might not have this optimized already. We are currently evaluating different options to optimize the functionality.

I understood that I can delete OPENSSL_cleanse() because it is optional and unnecessary.
Is this same in OpenSSL1.1.0-pre5, too?
If it is a problem to delete, please advise a reason.
If a performance improves, I want to implement it like that.
I think that a performance improves around 2%.
Kind regards,
Tom.

Hi Tom,
The purpose of OPENSSL_cleanse() is to overwrite an area of memory containing sensitive data (e.g. a private key) before an OPENSSL_free()
This is done to avoid security issues when the area of memory is returned to the OS and potentially reassigned to another process in the system.
This operation has a cost and, as you said, it can impact the performance of the application.
From a coding point of view there isn't anything stopping the user to remove the OPENSSL_cleanse(), but this should be done only after carefully considering all the security implications.
Hope this helps!
Andrea

Hi Andrea,
Thank you for your reply!
I understand.
Do you not recommend deleting OPENSSL_cleanse()?
Let me see ... I'll think about it.
Regards,
Tom.


Canot use for InDesign, because of the gripper.

ERROR in ./node_modules/ruxp/dist/components/Panel.js 12:29-49
Module not found: Error: Can't resolve 'photoshop'

hey! thanks for reporting this. it's weird that you're getting this error since i put a bunch of safeguards to avoid it. i'll install InDesign and test this out when i get home. is everything else working fine?

so, for some reason a random error was being thrown instead of the promise being rejected, so the catch() didn't catch it. whatever, it's wrapped in a try..catch now. i'll work on getting a release ready


For an array of JSON type script attributes, allow to set default values

Given:
Test.attributes.add('testObjects', {
    type: 'json',
    array: true,
    schema: [{
        name: 'foo',
        type: 'entity'
    }, {
        name: 'bar',
        type: 'string',
    } ],
    default: [{foo: null, bar: ""}]
});

It would be great if we can autofill the default values of a script type with these types rather than just have an array of 0

You set the default property for each element in the schema not on the surrounding object as you have done. For your example, you would set it as
Test.attributes.add('testObjects', {
    type: 'json',
    array: true,
    schema: [{
        name: 'foo',
        type: 'entity',
        default: null,
    }, {
        name: 'bar',
        type: 'string',
        default: '',
    } ],
});

@kpal81xd Ah yeah, makes sense.
@yaustar I'm going to close this since it appears you specified the JSON attribute incorrectly. However, if you believe there's still a problem here, please let me know.

I guess there is still no way to specify default value for a whole JSON array? I guess this applies for any array-like attribute.


list-inline & col-offset not working

I am a newbie to GitHub and programming in general, so I apologize for any errors in the following report. I am creating a page on CodePen and the list-inline and column offset classes are not working properly in v4.0.0 alpha 6, however if i switch to Bootstrap 3.3.7 it is working as expected. Links to Bootstrap CSS files I am using:
https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css
https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/4.0.0-alpha.6/css/bootstrap.min.css
My CodePen project:
https://codepen.io/geminix/pen/PpMNzz

v4 has different HTML, CSS, and JS, so you might need to change a few things around. Take a look at the v4 docs at http://v4-alpha.getbootstrap.com. The grid classes have changed for offsets (now .offset-*), for example.


error TS2304: Cannot find name 'ErrorOptions

Trying this out.
"@ibm-generative-ai/node-sdk": "^1.6.0"
but getting the following errors.
For now to get by, I changed ErrorOptions to any.

18     constructor(message?: string, code?: string, cancelled?: boolean, options?: ErrorOptions);
                                                                                ~~~~~~~~~~~~

node_modules/@ibm-generative-ai/node-sdk/dist/index.d.ts:21:60 - error TS2304: Cannot find name 'ErrorOptions'.

21     constructor(message?: string, code?: string, options?: ErrorOptions);
                                                              ~~~~~~~~~~~~

node_modules/@ibm-generative-ai/node-sdk/dist/index.d.ts:28:180 - error TS2304: Cannot find name 'ErrorOptions'.

28     constructor(message: string, statusText: string, statusCode: number, extensions: ErrorExtensions | undefined, response: ErrorResponse, headers: IncomingHttpHeaders, options?: ErrorOptions);
                                                                                ~~~~~~~~~~~~

Found 3 errors in the same file, starting at: node_modules/@ibm-generative-ai/node-sdk/dist/index.d.ts:18```

We are currently working on SDK v2, so we will not fix this. Ensure you are running Node 16+

@Tomas2D, I'm running Node 18.17.1.
Also do I need to request an apikey?  I'm getting unauthorized with my apikey.  And is v2 available to try?

V2 is not available yet for NodeJS SDK.

Make sure your api key corresponds to the environment you're targeting, e.g. https://workbench.res.ibm.com. If unsure, generate a new one.

hmm, getting access denied with my IBMid https://workbench.res.ibm.com/.

You'll need to request access to the Workbench environment. Talk to someone at IBM Research.


[BUG] QTreeWidget editor is too small

Description
The editor for QTreeWidgets is too small and conceals half of the text it is editing.

Prerequisites
If applicable to the issue, here are a few things you should provide to help me understand the issue:

Qt version: 6.4
Extension: None
Theme: Dark and Light

Sure. I reproduce the issue on Windows 10 Pro.

I've fixed this by removing the padding from the QLineEdit in the widgets in #72. It should now look much better, and I've posted 2 examples below (both with Qt6, but they're identical between Qt5 and Qt6). Thanks for reporting, this was a very important fix.


feat: Allow SVG Icon viewBox to be set via configuration instead of reusing icon size

🙋 Feature Request
Icons sizes being used for svg viewBox is a a little problematic when trying to use third party svgs. Can we either make the view port numbers configurable.
🤔 Expected Behavior
Be able to set the viewBox on the svg manually
😯 Current Behavior
This viewBox is currently built from the Icon.Size property which is an enum with insufficient granularity and range.
💁 Possible Solution
Allow the viewBox to be set by either ints or singly a string e.g "0 0 73 30"
🔦 Context
I would live to be able to add my own icons. Currently I can create a class inheriting from Icon to create my own svg icon. Only problem is that I cannot set the viewBox.

After a quick check of the code, we were able to add an IconSize.Custom enumeration. In this case, the FluentIcon component will use the icon content as a "raw" value and display the content as is.
Example, using a SVG with a size of 124x124.
public class MyIcon : Microsoft.FluentUI.AspNetCore.Components.Icon
{
    private const string SVG_CONTENT = @"<svg xmlns=""http://www.w3.org/2000/svg"" viewBox=""0 0 124 124"">
                                            <rect width=""124"" height=""124"" rx=""24""/>
                                            <path d=""M19.375 36.7818V100.625C19.375 102.834 21.1659 104.625 23.375 104.625H87.2181C90.7818 104.625 92.5664 100.316 90.0466 97.7966L26.2034 33.9534C23.6836 31.4336 19.375 33.2182 19.375 36.7818Z"" fill=""white""/>
                                            <circle cx=""63.2109"" cy=""37.5391"" r=""18.1641"" fill=""black""/>
                                            <rect opacity=""0.4"" x=""81.1328"" y=""80.7198"" width=""17.5687"" height=""17.3876"" rx=""4"" transform=""rotate(-45 81.1328 80.7198)"" fill=""#FDBA74""/>
                                            </svg>";

    public MyIcon() : base("MyIcon", IconVariant.Filled, IconSize.Custom, SVG_CONTENT)
    {
        // Default Color
        WithColor("#F97316");
    }
}

And you could use this icon like another:
<FluentIcon Value="@(new MyIcon().WithColor("red"))" Width="32px" />
<FluentIcon Value="@(new MyIcon())" Width="48px" />

But certain restrictions will apply: FluentButton.Loading or Icon.ToMarkup could be affected.
What do you think?

This would work perfectly yes.

#1342 - Will be published in the next release


Minor RazorProjectService cleanup

With recent changes, this is unnecessary because a document add event will do the same job, but better.

I definitely consider the integration tests the main thing here, and verifying that the AddDocument method doesn't generate the c# feels odd, but you reminded me I should have added a test to ensure we don't regress that the OpenDocumentGenerator works on DocumentAdded, since that is the functionality that replaces this chunk of code. I have now added said test.


windows wheel files not available

I'm trying to install zeep[xmlsec] on a windows system with Python 3.6. It results in the following,
Installing collected packages: xmlsec
  Running setup.py install for xmlsec: started
    Running setup.py install for xmlsec: finished with status 'error'
    Complete output from command D:\home\python364x64\python.exe -u -c "import setuptools, tokenize;__file__='D:\\local\\Temp\\pip-install-l510dnt_\\xmlsec\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record D:\local\Temp\pip-record-p1y7cdcl\install-record.txt --single-version-externally-managed --compile:
    running install
    running build
    running build_ext
    Retrieving "https://ci.appveyor.com/api/buildjobs/hij3a6776pdv2007/artifacts/libxml2-2.9.4.win64.zip" to "build/extra\libxml2-2.9.4.win64.zip"
    error: HTTP Error 404: The specified blob does not exist.

    ----------------------------------------

to what I can tell, it looks like the libxml and xmlsec wheel files were hosted on appveyor, but they don't have them anymore. Can we please get them hosted again?
related install attempt,
D:\home\site\wwwroot>D:\home\python364x64\pip.bat install xmlsec
Collecting xmlsec
  Using cached https://files.pythonhosted.org/packages/35/42/d7cd323c91d4706f3cc32ffe7d5f851ab8ef9898ccb350f6ba593dd8b89a/xmlsec-1.3.3.tar.gz
Requirement already satisfied: pkgconfig in d:\home\python364x64\lib\site-packages (from xmlsec)
Requirement already satisfied: lxml>=3.0 in d:\home\python364x64\lib\site-packages (from xmlsec)
Installing collected packages: xmlsec
  Running setup.py install for xmlsec: started
    Running setup.py install for xmlsec: finished with status 'error'
    Complete output from command D:\home\python364x64\python.exe -u -c "import setuptools, tokenize;__file__='D:\\local\\Temp\\pip-build-_75yc_6u\\xmlsec\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record D:\local\Temp\pip-y7wv7xut-record\install-record.txt --single-version-externally-managed --compile:
    running install
    running build
    running build_ext
    Retrieving "https://ci.appveyor.com/api/buildjobs/hij3a6776pdv2007/artifacts/libxml2-2.9.4.win64.zip" to "build/extra\libxml2-2.9.4.win64.zip"
    error: HTTP Error 404: The specified blob does not exist.

    ----------------------------------------
Command "D:\home\python364x64\python.exe -u -c "import setuptools, tokenize;__file__='D:\\local\\Temp\\pip-build-_75yc_6u\\xmlsec\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record D:\local\Temp\pip-y7wv7xut-record\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in D:\local\Temp\pip-build-_75yc_6u\xmlsec\
You are using pip version 9.0.1, however version 18.0 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.

D:\home\site\wwwroot>

the appveyour changes its policy about retention build artifacts.
if someone knows where this files may be posted instead of appveyour - welcome.
I cannot just rebuild projects since build rules have been changed and build fails.
I anyone can fix this, please make PR for repository https://github.com/bgaifullin/libxml2-win-binaries.

@bgaifullin  if the issue is hosting, can we consider using github releases for the wheel files?

Alternatively, has pythonhosted.org been considered?
https://packaging.python.org/tutorials/packaging-projects/

@plasticav the github releases can be used, but need to change scripts for uploading into github after build. Can you do this?
about pythonhosted.org, there is libxml2 binaries for windows, and I am not sure, that it can be hosted on pythonhosted.org.

Not only the wheel. The pip build (pip install xmlsec) and manual install (python setup.py install) options do not work either.
Getting the following error:
zipfile.BadZipfile: File is not a zip file

You can see this is the same error in the wheel's build too:
https://ci.appveyor.com/project/bgaifullin/python-xmlsec/build/job/ed6uyd5t68gscyv3

This issue is not resolved. This is a circular issue reference if you read to the bottom of 35. It is currently a big blocker for me.

the latest windows binaries are available here https://github.com/mehcode/python-xmlsec/releases/tag/1.3.5

Any movement on this? I am getting the same issue. On python 2.7 I get a badzip error, and on python 3.7 I get ci.appveyor.com is no longer available.

Hi there, I don't see any release for Python 3.7.x environment on windows. Can you please please make a release?  Tried with Python 3.6 release did not worked:
ERROR: xmlsec-1.3.52.dev0-cp36-cp36m-win_amd64.whl is not a supported wheel on this platform.

Same demand and issue as @RSwarnkar; installing the 32-bit binary worked, though.

@neemzy Which one did you get to install. I have gone through all of them and get "not a supported wheel on this platform"

@klatessa xmlsec-1.3.52.dev0-cp36-cp36m-win32.whl

@neemzy : the win32 wheel package not work for me for python 3.7

@majvan try
pip install xmlsec --extra-index-url=https://hoefling.io/pypi

I will upload wheels for windows on PyPI soon, with the release of 1.3.7.


Move core component to 0.14.0 spec version | RTC approved

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Merging since the rum test case is fixed as part of another PR.


Bug 2120269: core: add support to rotate log based on logfile size

Description of your changes:
core: add support to rotate log based on logfile size
this commits add new field MaxLogSize inside LogCollectorSpec of
cephCluster cr which will take max size of log after which we want to
rotate the log.
core: fix logrotate bash check and periodicity logic
we need use != for string comparision in bash instead of
-ne. Also, need to correct periodicity if condition to
make it work as expected.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>(cherry picked from commit https://github.com/red-hat-storage/rook/commit/9d4d5bc7025a026a7f11c52f48f1db0c00c4d5c9)
Signed-off-by: subhamkrai<EMAIL_ADDRESS>(cherry picked from commit https://github.com/red-hat-storage/rook/commit/62f73dcd989dde83eca48f58c47a7d27d98b6bc1)
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/assign @travisn

/lgtm
/approved


Add Defer function to defer free HKDF object

I think this is an interesting idea, however, I'd recommend to propose this on master first because it could be applied to many other places and it would change flows. For HKDF, using constructor / destructor should be enough (and it uses those on the PR currently on review), and I prefer to use a general way if there's no big difference.

link to #5682. Thanks


Update traffic after deploy

From docs:
To restore sending traffic to the LATEST revision by default, run the [gcloud run services update-traffic](https://cloud.google.com/sdk/gcloud/reference/run/services/update-traffic) command with --to-latest.

Not sure when it started happening, but traffic is not being rerouted to the latest version after deploys. This happens even in a successful deploy job. Example
Hopefully this fixes it

not needed anymore, closing.

Switching the gcloud service to managed solved our issues of deploys not redirecting traffic to latest


[HxTooltip] Empty Text property causes error.

Hello,
We're using the HxTooltip in our Blazor Server .NET 8 application and we have some logic to change the text displayed in the tooltip based on some conditions - for example we allow the user to toggle a checkbox, which might change the content that should be shown in the tooltip.
If the text is empty though, after a re-render, we see the following crash.
We can fix it by ensuring that the tooltip always has text - is there something we should be doing other than excluding the HxToolTip tag completely if we don't want it shown? I notice there's only an "Enable/Disable" method, not a property for this component.
Error: Microsoft.JSInterop.JSException: TOOLTIP: Option "title" provided type "null" but expected type "(string|element|function)".
_typeCheckConfig@https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js:6:7196
_getConfig@https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js:6:68998
W@https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js:6:7409
cs@https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js:6:62513
@code {
    bool isChecked = false;
}

<Havit.Blazor.Components.Web.Bootstrap.HxCheckbox Text="Toggle me?" @bind-Value="isChecked">

</Havit.Blazor.Components.Web.Bootstrap.HxCheckbox>

@{
    var tooltip = "";
    if (isChecked)
    {
        tooltip = "Checked";
    }
    <Havit.Blazor.Components.Web.Bootstrap.HxTooltip Html="true" Text="@(tooltip)">
        <div>
            Hover over me to see the tooltip.
        </div>
    </Havit.Blazor.Components.Web.Bootstrap.HxTooltip>
}

If I understood your question correctly, you could try something along these lines in the code block of the razor file.
private string _tooltip = string.Empty;
private string tooltip
{
    get{
        return _tooltip;
    }
    set{
        if (string.IsWhiteSpaceOrEmpty(value))
        {
            hxTooltip.Enable = false;
        }
        else
        {
            hxTooltip.Enable = true;
        }
        this.StateHasChanged();
        _tooltip = value;
    }
}

I can no longer reproduce the Bootstrap JavaScript error in the browser console, but when I set Title to null (after initializing HxTooltip with some text), an arrow still displays:

Bootstrap doesn’t automatically disable the tooltip when the title is unset.
We could improve this by detecting when the tooltip needs to be disabled (and re-enabled) and toggling it by calling the appropriate JS method. The same approach could apply to HxPopover.

After some initial investigation into implementation options, we decided to stick with the current state for now.

This is how Bootstrap itself behaves when you set the data-bs-title="" attribute.
Detecting the removal of a Title value and calling the disable method on the tooltip/popover is quite complex. We need to respect explicit calls to EnableAsync() or DisableAsync() - meaning we can't re-enable the component if it was intentionally disabled and then the Title changes from null or String.Empty to some value.


rename base class => border

Closes #101

PR-Codex overview
This PR changes the color scheme of various UI elements in the QA admin app to improve accessibility and consistency.
Detailed summary

Changed the base color to a neutral shade for better contrast
Updated border colors to match the new base color
Changed various background colors to use the new border color with a lower opacity
Updated text and shadow colors to match the new color scheme

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (e967378) 71.00% compared to head (12fdcbc) 71.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #102   +/-   ##
=========================================
  Coverage     71.00%   71.00%
  Complexity      172      172
=========================================
  Files            78       78
  Lines          1221     1221
  Branches        104      104
=========================================
  Hits            867      867
  Misses          258      258
  Partials         96       96

Impacted Files
Coverage Δ

...rc/components/UI/ValidatedInput/ValidatedInput.tsx
70.76% <100.00%> (ø)

...t/src/components/categories/CategoriesListHead.tsx
80.00% <100.00%> (ø)

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


Add "compare" function and got error message

I just append a function "compare" to one of the module:
let compare _ _ = 0 (* implementation removed*)

In order to support Map, Something like this:
module PathMap = Map.Make(Path)

But got error message below. Is there any more explanation or solution to this?
  38 |   | Tunivar of string option
  39 |   | Tpoly of type_expr * type_expr list
> 40 |   | Tpackage of Path.t * Longident.t list * type_expr list
  41 |
  42 | and row_desc =
  43 |     { row_fields: (label * row_field) list;

It is unclear which name this signature has. At least two similar
signatures found, namely:

* Stdlib__map.OrderedType
* Stdlib__set.OrderedType

We were looking for a module signature name for the following shape:
[ compare ]
(a shape is a list of names of values and sub-modules)

We use the concept of shape to find the name of a signature for Coq.

Yes, the two signatures Stdlib__map.OrderedType and Stdlib__set.OrderedType are actually the same, so coq-of-ocaml gets confused. I think this error is fine then because both choices are the same.

By the way, does the generated code for this example looks fine?

yes, the content seems OK. But when "set" is not used, it still generate errors:
 $ cat main.ml

(*open Path*)

module PathMap = Map.Make(Path)

 $ cat path.ml
type t = int

let compare _ _ = 0

 $ cat Main.v
(** Generated by coq-of-ocaml *)
Require Import OCaml.OCaml.

Local Set Primitive Projections.
Local Open Scope string_scope.
Local Open Scope Z_scope.
Local Open Scope type_scope.
Import ListNotations.

Definition PathMap := Stdlib.Map.Make (existT (A := Set) _ _ (|Test123.Path|)).

What if you add a dummy definition to 'path.ml', and recompile everything?

Like 'let dummy = ()'

Definition dummy : unit := tt.

is generated, and still get error from 'coq-of-ocaml main.ml'

What do you get for 'main.ml' then? The hope is to change the interface of 'path.ml' so that it doesn't think it is a record but a plain module.


Feature: remote load specs from url in FunctionInputs

#1 It resolves the hassle of reploying the automate function, when there is a change in the test specs using httpimport package. It only works until python 3.11.

There is an open issue on httpimport about python 3.12 support https://github.com/operatorequals/httpimport/issues/58 we can keep an eye on

I have a package called od_import which is similar and is compatible with 3.12+, if you want to try that. The syntax is fairly similar.


Bluetooth: Cherry-pick upstream commits

[nrf fromtree] Bluetooth: host: Prevent race condition in pairing procedure
Moving the key check after checking that no pairing or encryption
procedure is started fixes a race condition that is seen in some PTS
tests:
GATT/CL/GAR/BI-04-C
GATT/CL/GAR/BI-16-C
GATT/CL/GAR/BI-21-C
GATT/CL/GAW/BI-05-C
GATT/CL/GAW/BI-12-C
Signed-off-by: Jonathan Rico<EMAIL_ADDRESS>[nrf fromlist] Bluetooth: L2CAP: Fix missing net_buf_unref()
In case of non-recoverable errors (e.g. the connection breaks while
transmitting), the l2cap_chan_tx_process deques the net_buf but does
not unreference it. As this is inside a work queue thread, the sending
thread gets no information on this error, relying on the tx_process to
ultimately free the buffer.
Upstream PR: zephyrproject-rtos/zephyr#36031
Signed-off-by: Patrick Rathje<EMAIL_ADDRESS>

Rebasing to make CI pass


Development of ConvertToXML on FixedWidthFlatFile

ConvertToXML function implemented and formatted to Fends.FixedWidthFlatFile. From community task (https://github.com/CommunityHiQ/Frends.Community.FixedWidthFlatFile).

Please also add the task documentation link to the root README.md: https://github.com/FrendsPlatform/Frends.FixedWidthFlatFile/blob/main/README.md


Spring Integration.

Hi,
I'm using spring with XML config and I have filter on responsbody using responsebodyadvice wherein i create a nice tree to filter/replace node values which are access restricted. Now how can I configure and invoke squiggly here..

Please read nice as NODE jsonNode.

The way I'm handling is in beforeBodyWrite where I convert the responsebody to jsonNode and traverse through each one to check if the node contains specific pattern . If I get the pattern i replaces it with masking text or value.

@atulmahajan05 I'm not sure squiggly would work with actual JsonNode objects.  Squiggly is implemented as a Property Filter, which works on Arrays, Collections, Maps, and POJOs/Beans.   My guess is you would have to convert it back to one of these types before you could invoke Squiggly.
There's a method on ObjectMapper called treeToValue that will help you do this.
If you're still stuck and willing to provide some sample code, I'd be happy to investigate.

@atulmahajan05 The other thing I want to mention is that in a couple weeks, I will be releasing version 2.0, which will allow you the ability to manipulate json response.  So, you will be able to do something to do something like:
/somepattern/.replace('*******')

I wil keep your situation in mind before I release the new version

That will be awesome.. this is a practical situation where some response nodes or value from api's need to be kept access restricted or dropped. Jackson filter does work on the level 0 nodes but does not work over nested objects and collections. I'm  look forward to use squiggly for such nice out of the box features... Thanks once again.
Regards,
Atul

One more feature i was looking for is the way to provide operators within the request parameters..We don't use Spring Data which has the capabilities to integrate with rsql .. if you can build it with squiggly, I'm sure this will help a lot of folks...

@atulmahajan05 Version 2.0 will have this.
For example,  you will be able to write a filter like the following:
friends.filter(firstName == "Ryan")


update compare table columns for no reference (issue #114)

This provides a fix for issue #114

Only creates the reference output column header if reference output is present for the data
Updates the test suite state settings so that the compare table is reloaded correctly. Before making this change, the correct column headers would only be displayed on refresh.
Had to update the versions of the types packages to build successfully

New table view for readability (no reference) test suite:

going to merge because these changes are all FE (and clearly our build is flaking because those are BE validations)


make scores.probability.functions public

scores.probability.functions contains functions that will be used by a user.
We need to discuss how to best call them. Options could be:
scores.probability.processing
scores.probability
scores.processing
scores.processing.probability
I'm interested in @rob-taggart's view

After talking to Rob, "cdf" rather than probability may be clearer.
My preference is now
scores.processing.cdf


Cost Explorer API (?/n): to be refined notes on remaining work

Use this issue as a todo list of work related to using the Cost Explorer (CE) API, as part of #4453.
### Tasks
- [ ] research: Figure out how Grafana will adjust requests based on UI configured time ranges
- [ ] code: Cache requests to CE to reduce costs (0.01 per request) and improve performance
- [ ] chart: Declare resource requests and limits
- [ ] chart: Add and validate NetworkPolicy resources deployed so that _only Grafana_ can work against the webserver

Thanks @consideRatio. As guidance for scope of tasks, please use the EPIC and create separate subtasks for each items than using a 'catch all'. The goal is to make sure we have things that can be sized during next sprint planning, and single catch-all tasks can not be. Sized tasks help us figure out team velocity to know how much we can deliver. This also means there are refined things to mark as comitted next time you're working, which was one of the issues this sprint.
So the specific ask is:

Use #4453 to track, rather than a catchall issue like this.
Create issues for each task (with just enough refinement for tracking progress), so we can look at what needs to be done by looking at the EPIC
Spend some time explicitly and intentionally refining remining tasks on the EPIC, now that you have more information. This is to 'bootstrap' refined tasks for you to pick up on next sprint, so you don't have to do unplanned work.

Sorry for the confusion @yuvipanda, I'm aligned with that intent.
I perceived it was a bit premature to define/refine individual tasks from this list now, that I mostly wanted as a overview of various things to define/refine.
In the spirit of how we are to do things, this issue can be seen as the task of doing the work of defining+refining misc followup tasks to the first four initially defined, ending with #4673. This refinment issue should itself not be considered refined until the list of followup tasks is defined.
I'll update it to reflect that it should be the task of defining+refining followup work after #4673.

I perceived it was a bit premature to define/refine individual tasks from this list now, that I mostly wanted as a overview of various things to define/refine.

@consideRatio that makes sense. I'd like you to use #4453 for that, rather than creating another issue. Can you move the tasklist here to that?

This refinment issue should itself not be considered refined until the list of followup tasks is defined.

+1, I think using this issue to say 'refine further' is appropriate.

This refinement effort is now complete!


[SUGGEST] Adding nick logs

That when some1 gets nicked save that as log in a file or in a db. Like that if some1 is reported or something you can know who was. Using db it can include a command like /checknick. For example: /checknick NormalNick
Justix - 15/12/22 10:43 to 12:26
CraciusConEse - 27/4/22 18:53 to 28/4/22 1:16

You can use LogNicknames  from the setup.yml to log the nicknames to the console. You can then just look through the log files. If you really need a database log, you can easily add it using the API: Click here for details


Payment.py: "import re"

Seems you made a typo here. Line 1 of Payment.py is "import re", rather than "import requests". However, this may just be the fork I was using, specifically the one by Jarvis Johnson.

Thanks for reaching out!
We're actually importing the re module intentionally - we use regular expressions to validate certain credit card attributes.
You can see the calls to re throughout the file - for example, see here: https://github.com/gamagori/pizzapi/blob/master/pizzapi/payment.py#L20.


parse contents, not name, of environment variable

fix to parse contents of, and not name of, environment variable for default region
(e.g. "AWS_DEFAULT_REGION" of euca2ools, see https://github.com/boto/requestbuilder/issues/6)

Ouch.  Good catch.  Thanks for the patch!


Issue on Colab: "Failed to connect Can't connect to display"

When I run AI2-THOR on Colab, I had such an issue. How to solve it?
The following builds were found, but had missing dependencies. Only one valid platform is required to run AI2-THOR.
Platform Linux64 failed validation with the following errors: Invalid display: :1. Failed to connect Can't connect to display ":1": [Errno 111] Connection refused
  Linux64 requires a X11 server to be running with GLX. If you have a NVIDIA GPU, please run: sudo ai2thor-xorg start

Hi, have you ever solved this problem? I tried running the suggested commandsudo ai2thor-xorg startbut got another series of errors

No. I give up running it on colab. There is no issue on local PC.

Okay, I'll try it on PC too. Thanks for your quick reply!

I encountered the same issue on the stock Collab notebook for procthor, so I think this functionality must just be broken right now. Maybe Collab's out-of-the-box config changed or something?

Could you please tell me did you solve this issue now?

This seems to be a larger issue, i cannot get X to run on any colab notebook, no matter what i try, it comes down to this error message when starting X: xf86EnableIO: failed to enable I/O ports 0000-03ff (Operation not permitted). The virtualhome project has the same problem on their colab. I cannot find a larger community that requires the X display system to run on Google Colab.


Add an .editorconfig

I've added a .editorconfig to help facilitate community contributions. Hopefully this will ensure consistent formatting for new contributors. It's also much easier to write pull requests when you're not fighting your code editor the entire way. :smile:
More info: http://editorconfig.org/

Looks nice. Thank you!

Yay! :+1: :shipit:


[FEATURE] Remove 'day' portion from dates

In most resumes, the exact day of a date is not important, only month and year are. Few people remember the exact day they started or finished a project / school / job, and this information is not relevant to employers. Plus, it makes the date take too much space in the title row.
Current implementation: June 08, 2022 - July 09, 2023
Proposed implementation: June 2022 - July 2023

You could set the date format for the whole resume, this may not be what you want, we should have individual customization.
Go to settings in the bottom of the right side bar > Date > choose one without days

That's quite helpful actually, I didn't know about that configuration. Cheers!


Alibaba/ecs-sd

Alibaba ECS Service Discovery feature.

rebase and pull request again.
https://github.com/AliyunContainerService/prometheus/pull/2


Blog Plugin Not Working?

Hello, enable the blog plugin, but then got a bunch of stuff spit back at me. Any idea?
  File "build/bdist.macosx-10.12-intel/egg/django_markwhat/templatetags/markup.py", line 61, in markdown ImportError: No module named markdown

Any thoughts here?

Hi wragen22,
it seems like your missing the Python Markdown package.
Copy the following into your command line (Terminal App) and press enter to install the Python Markdown package.
easy_install markdown


yafetch: init at unstable-2021-05-13

Motivation for this change
A minimal, and fast fetch clone.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer


CTCP-5431: Updated Additional Reference Number page to be a standard …

…input text page with a hint


Fix.warnings

Update to fix the warnings in the current main branch.  This does not adjust any of the text.

Review conflicts at your convenience

#12 included all the fixes in this PR.  Closing this, as not merging.


Placeholder is not working

Hello,
I tried to replace the placeholder but it has no effect.
<ReactTags
       value={tag}
       tags={tags}
       placeholder="Adicione uma nova Tag"
       suggestions={lista}
       onDelete={(e) => onDelete(e)}
       onAddition={(e) => onAddition(e)}
/>

I've tested in on https://react-tag-input-1nelrc.stackblitz.io and it's working for me with the latest version.

Do you mind in show me the snippet you use? When I access this url, I still see the Add new tag placeholder. I am developing an app in Portuguese, this is why I need to change it to Adicione uma nova tag.
Thanks for the attention.

Go to the link i've pasted above and try those props:
<ReactTags           tags={tags}           suggestions={suggestions}           delimiters={delimiters}           handleDelete={this.handleDelete}           handleAddition={this.handleAddition}           handleDrag={this.handleDrag}           handleTagClick={this.handleTagClick}           placeholder="Adicione uma nova tag"         />


Annotations dont work when @ decorator is used in service

I have a class written in ES6 style which is then exposed as angular service
Annotations are working fine until this class used @ decorator.
I am using mobx actions which requires methods to be decorated with @action. The moment I use @action inside my class annotations start breaking.
Is this a known issue?

I have this issue too, and it's very pity :(

It seems like we simply need to use this acorn plugin : https://github.com/angelozerr/acorn-es7
But how do we include it into the pipeline?


LoadBalancerOptions not skip DownstreamHostAndPorts is not work.

I have "DownstreamHostAndPorts" start in my server 3 port 8001 , 8003 , 8004
It can have any accident once day 8003 is stop work.
I need to let it auto skip / auto call to another DownstreamHostAndPorts index is work fine , It doesn't. Now it will call success 2 time , return 500 one time when it call to  8003  and  loop.
`
{
"ReRoutes": [
{
"DownstreamPathTemplate": “/test/{version}/hub/{everything}",
"DownstreamScheme": "http",
"DownstreamHostAndPorts": [
{
"Host": “localhost",
"Port": 8004
},
{
"Host": "localhost
"Port": 8001
},
{
"Host": "localhost
"Port": 8003
}
],
"UpstreamPathTemplate": "/api/test/{version}/hub/{everything}",
"UpstreamHttpMethod": [ "POST" ],
"LoadBalancerOptions": {
"Type": "CookieStickySessions",
"Key": "ASP.NET_SessionId",
"Expiry": 1800000
}
}
],
"GlobalConfiguration": {
}
}
`

same my prob

Ocelot team is not responsible for Heath of your downstream services. If you get error then it is normal to have the error in logs!
Apply Health Checks for your downstream services to keep them always online. This is DevOps task.

Not Ocelot issue!


feat: add google-gax status to exports

Fixes #902

I am also not able to get type completion to work. Look at these two examples:


TeXify emits improper HTML

HTML parameters must be enclosed in quotes (").  If they are not Jekyll will consider the entire image html to be text and it will not render properly.  For example:
What TeXify emits:

What it needs to be:

GitHub (I think) uses Jekyll to render pages and sites.  Even if there were a configuration to change Jekyll behavior, I would have access to the GitHub version of it.
I can reproduce this failure on my on laptop with using Jekyll locally.
Please fix this :) :)

Let me try this again but without the opening and closing brackets "<" and ">" so it won't try to render :)
What TeXify emits:

What it needs to be:
<img src="/docs/Tuple/tex/4b83ab92144c8118c40653189ab60df5.svg?invert_in_darkmode&sanitize=true" align="middle" width="81.54091274999998pt" height="22.831056599999986pt"/&gt:

I substituted < and >

Same issue here. Please fix it, thanks!


[Woptim] Compiler wrappers + update packages + use spack environments

[ ] What should we do with the content of our local modules.yaml file?

Summary

The compilers (wrappers) on Lassen have been updated to better support the use of gcc 8.3.1 toolchain.
The local spack packages have been updated to be as close as possible to what we are pushing in https://github.com/spack/spack/pull/41375.
We moved to Spack environments, allowing us to update Uberenv in the projects.

Notes
Packages in RADIUSS Spack Configs still have some specific changes compared to the PR mentioned above.
Tests
https://github.com/LLNL/Umpire/pull/875
https://github.com/LLNL/RAJA/pull/1617
https://github.com/LLNL/RAJAPerf/pull/438
https://github.com/LLNL/Caliper/pull/520
https://github.com/LLNL/CHAI/pull/255

@rhornung67 this is ready for review.


Veldrid can't be compiled without VALIDATE_USAGE define

@mellinoe
2>3rdParty\Veldrid\Source\Veldrid\CommandList.cs(800,36,800,61): error CS1061: 'Pipeline' does not contain a definition for 'GraphicsOutputDescription' and no extension method 'GraphicsOutputDescription' accepting a first argument of type 'Pipeline' could be found (are you missing a using directive or an assembly reference?)
More likely this is because of:
#if VALIDATE_USAGE
internal OutputDescription GraphicsOutputDescription { get; }
#endif

Fixed with https://github.com/mellinoe/veldrid/commit/9807394ddd540b100498006e6f1c4fbc442b0d76


Document procedure for Debian 11 (Bullseye)

Small update - Debian 11 is live and there's provision on the actual site as well; this should probably be reflected on documentation.

@gschizas : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Debian 11 support has not been added to the Microsoft ODBC Driver 17 for SQL Server. It's not just a matter of updating documentation when a new OS version releases.
#please-close

Fair enough. Sure explains some problems I had


feat(health): add health check

Signed-off-by: Felipe Zipitria<EMAIL_ADDRESS>Adds health check to the /healtz endpoint configured in all containers.
Fixes #136.
Note: I did try adding variables to the different times, but docker doesn't like it. :( My idea was to add 4 new variables HEALTHCHECK_, but looks like we can't.
❯ docker-compose build apache-waf
[+] Building 0.1s (2/2) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 9.17kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
failed to solve: rpc error: code = Unknown desc = failed to solve with frontend dockerfile.v0: failed to create LLB definition: dockerfile parse error line 190: time: invalid duration "${HEALTHCHECK_INTERVAL}"

No log output?

It is failing because of a typo

Cool. Added more output and now we can see the progress from starting to healthy.


Popup Color Palettes

I need a lot of colorpalettes (all with complete material color set) and it would be very nice not to have them loaded all in a row since they take a lot of room, just for one color value it takes a lot of space.
So a colorpalette just like the multi-color field, but only with the palette colors would be great!
See https://github.com/carloscabo/jquery-palette-color-picker
Greetings DWFee

Good call, though probably rarely needed. I'm closing this for now but consider re-opening it if we get more requests for this.


uniform matcher可以给一个更详细的注释吗

uniform matcher的有些难理解，希望有更多注释

Ok, I will add detailed comments in the next few days.

Please check this commit for more comments about the matching.


Add Features section to README

Attempting to clean up README.md more, giving a place for all of the features of the track. Hopefully, this will help users know more about what the tracker can do for them right up front. This will also help future collaborators to have a good place to add documentation on their features.
Oh yeah, I also added more links and references to BizHawk needing to be version 2.8 or higher... I held back on my snark... 😅

Added a version check near the beginning of the script to ensure that users are using version 2.8. If not, the script will end immediately. This will resolve a LOT of errors that people keep bringing up in Discord!
Note: Currently, it checks the output of client.getversion() explicitly as a string compare. So if there is a version upgrade, we would need to add the strings to the if check, which is not ideal or scalable. I'm sure there is a better alternative to how I did this, but this works for now.


Serialization separate

Created a separated serializer and unserializer for Pywren.
All functions' logic explained in serialize.py.

This was about to test some optimizations of object serialization methods, we should consider to test it again in future


ProfileReport fails when data frame contains date objects

Current Behaviour
Whenever a DataFrame contains a series of date objects, creating a profile report fails with the following stack trace:
File ~/misc/test/env/lib/python3.9/site-packages/ydata_profiling/model/pandas/describe_date_pandas.py:34, in pandas_describe_date_1d(config, series, summary)
     22 """Describe a date series.
     23
     24 Args:
   (...)
     30     A dict containing calculated series description values.
     31 """
     32 summary.update(
     33     {
---> 34         "min": pd.Timestamp.to_pydatetime(series.min()),
     35         "max": pd.Timestamp.to_pydatetime(series.max()),
     36     }
     37 )
     39 summary["range"] = summary["max"] - summary["min"]

TypeError: descriptor 'to_pydatetime' for 'pandas._libs.tslibs.timestamps._Timestamp' objects doesn't apply to a 'datetime.date' object

NB: #1394 mentions the issue, but it is not the main issue.
Expected Behaviour
The report should be able to handle comparing date objects, and not only datetime objects. We don't always know in advance what columns are in the data. Some of them are also of dtype object, so it is tedious to try to convert all dates to datetimes.
Data Description
Any column with datetime.date objects
Code that reproduces the bug
import pandas as pd
from ydata_profiling import ProfileReport
from datetime import date as dt

df = pd.DataFrame({"t": [dt.today(), dt.today()]})
report = ProfileReport(df)

pandas-profiling version
v4.5.1
Dependencies
pandas==2.0.3

OS
Tried with macOS and Docker (python:3.9 image)
Checklist

[x] There is not yet another bug report for this issue in the issue tracker
[X] The problem is reproducible from this bug report. This guide can help to craft a minimal bug report.
[X] The issue has not been resolved by the entries listed under Common Issues.

same issue here.
the workaround (for now) is to explicitly provide the parse_dates argument to pandas: pd.read_sql(..., parse_dates=date_columns)

Related to #1378


Possible to give puppeteer browser window it's own title/icon?

Just wondering how I would go about this. I initially suspected a chromium argument on startup would do the fix but to no avail.

You won't be able to do that.

For those looking at 2024. You can use command line argument https://peter.sh/experiments/chromium-command-line-switches/#window-name
`--window-name=MyWindow`


#23 - Add labels in the example form (master)

This improves the form's accessibility.

This is great, but to be honest I'd prefer to have the inputs nested inside the labels to get rid of the id and for attributes. I think this keeps the examples simpler. Do you mind changing that?
Thanks!

To be honest I didn't know that approach . I'll change it later.

El 9 sept 2016, a las 12:34, Eduardo Bouças<EMAIL_ADDRESS>escribió:
This is great, but to be honest I'd prefer to have the inputs nested inside the labels to get rid of the id and for attributes. I think this keeps the examples simpler. Do you mind changing that?
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This pr and the other one should be fine now.

Perfect, thanks!


Table Index Config settings not preserved when migrating to fieldConfigList

When setting table index configs, Pinot documentation recommends deprecating usage of single-column index configurations in favor of the Field Config List.
However, when attempting to migrate an existing table from the deprecated format, these settings are not respected. For example, using an old table config, dictionary indexes are disabled:
{
  ...
    "tableIndexConfig": {
      "noDictionaryColumns": [
        "col1",
        “range1”,
      ],
      ...
      "optimizeDictionaryForMetrics": true,
      "optimizeDictionary": false,
      "noDictionarySizeRatioThreshold": 0.85
    },
  }

Updating to a new definition:
{
...
  "fieldConfigList": [
    {
      "name": "range1",
      "indexes": {
        "dictionary": {
          "disabled": true
        },
        "range": {}
      }
    },
    {
      "name": "col1",
      "indexes": {
        "dictionary": {
          "disabled": true
        },
        "json": {
          "maxLevels": 1,
          "excludeArray": true,
          "disableCrossArrayUnnest": true
        }
      }
    }
  ]
}

Making the table config change and triggering a table reload creates new dictionary indexes even though they are specified as disabled. Adding the property "encodingType": "RAW" also does not resolve this issue.

Thanks for reporting the issue. I believe this is a bug and should be fixed with #14258. Can you try with the latest master?

Thanks for addressing. We'll test out the changes on the next release with 1.3.0 and report back with any issues.


Ramda: Error in type definition for Pipe/Compose

The definitions for Ramda for pipe and compose are incorrect.
The definitions specify that pipe takes a list of UnaryFns which pipe through to the next. The docs for ramda contradict this though:

The leftmost function may have any arity; the remaining functions must be unary.

Case in point, I have a function that looks like this: pipe(zip, map(sum)). This is valid Ramda code and passes my tests, but Flow is not having it.
As the first function pipe accepts can be variadic, I propose adding a new type alongside the UnaryFn definitions for a Variadic Definition:
declare module ramda {
+  declare type VariadicFn<A,R> = (...rest: A[]) => R;
  declare type UnaryFn<A,R> = (a: A) => R;

So Pipe (whose first function and returning function which can be Variadic) becomes:
  declare type Pipe = (<A,B,C,D,E,F,G>(ab: UnaryFn<A,B>, bc: UnaryFn<B,C>, cd: UnaryFn<C,D>, de: UnaryFn<D,E>, ef: UnaryFn<E,F>, fg: UnaryFn<F,G>, ...rest: Array<void>) => UnaryFn<A,G>)
    & (<A,B,C,D,E,F>(ab: UnaryFn<A,B>, bc: UnaryFn<B,C>, cd: UnaryFn<C,D>, de: UnaryFn<D,E>, ef: UnaryFn<E,F>, ...rest: Array<void>) => UnaryFn<A,F>)
    & (<A,B,C,D,E>(ab: UnaryFn<A,B>, bc: UnaryFn<B,C>, cd: UnaryFn<C,D>, de: UnaryFn<D,E>, ...rest: Array<void>) => UnaryFn<A,E>)
    & (<A,B,C,D>(ab: UnaryFn<A,B>, bc: UnaryFn<B,C>, cd: UnaryFn<C,D>, ...rest: Array<void>) => UnaryFn<A,D>)
    & (<A,B,C>(ab: UnaryFn<A,B>, bc: UnaryFn<B,C>, ...rest: Array<void>) => UnaryFn<A,C>)
    & (<A,B>(ab: UnaryFn<A,B>, ...rest: Array<void>) => UnaryFn<A,B>)

And compose becomes
  declare type Compose = & (<A,B,C,D,E,F,G>(fg: UnaryFn<F,G>, ef: UnaryFn<E,F>, de: UnaryFn<D,E>, cd: UnaryFn<C,D>, bc: UnaryFn<B,C>, ab: VariadicFn<A,B>, ...rest: Array<void>) => VariadicFn<A,G>)
    & (<A,B,C,D,E,F>(ef: UnaryFn<E,F>, de: UnaryFn<D,E>, cd: UnaryFn<C,D>, bc: UnaryFn<B,C>, ab: VariadicFn<A,B>, ...rest: Array<void>) => VariadicFn<A,F>)
    & (<A,B,C,D,E>(de: UnaryFn<D,E>, cd: UnaryFn<C,D>, bc: UnaryFn<B,C>, ab: VariadicFn<A,B>, ...rest: Array<void>) => VariadicFn<A,E>)
    & (<A,B,C,D>(cd: UnaryFn<C,D>, bc: UnaryFn<B,C>, ab: VariadicFn<A,B>, ...rest: Array<void>) => VariadicFn<A,D>)
    & (<A,B,C>(bc: UnaryFn<B,C>, ab: VariadicFn<A,B>, ...rest: Array<void>) => VariadicFn<A,C>)
    & (<A,B>(ab: UnaryFn<A,B>, ...rest: Array<void>) => VariadicFn<A,B>)

This isn't a perfect proposal, as these accepted functions can be unary, but the VariadicFn definition suffices for both.

cc @alexeygolev

@keithamus Good point! Could you please submit a PR with a relevant (zip) test case

I've been playing a bit with these and I can see a few holes in this, so I'm just going to work a bit on several parts, if thats okay? I'll make one single PR with a set of individual commits for each issue I find.

@keithamus amazing! thank you

FYI this is still very much on my radar, I'm filling out much of the TODOs as we speak. I hope to have a PR before the weekend.

Closing as it seems the changes were made at some point.


Security Providers Documentation

Basic documentation for Security Providers

Stackedit.io preview

@swankjesse Any thoughts on the format?  Table with links vs Separate sections of a single page?


Compiling libmodsecurity and nginx with it difficulties

Hello,
My task is to compile nginx with support for mod security. I tried both mod security and nginx-modsecurity-connector, but found some difficulties for which I searched in the issue tracker and google and would like to ask you for some help. I compiled libmodsecurity with the steps everybody other follows:
OS: CentOS 6.8
Modsecurity version is 2.9.1
nginx version: 1.12
Is there any incompatibility as I found the following topic:
https://serverfault.com/questions/772136/any-versions-of-modsecurity-that-works-with-recent-versions-of-nginx
216  git clone https://github.com/SpiderLabs/ModSecurity
217  cd ModSecurity/
218  git checkout -b origin/v3/master.  #also tried without this branch
219  git submodule init
220  git submodule update
224  ./autogen.sh
225   ./configure
226  make
227  make install
Note: Tried to compiled it also as a standalone module but the result is the same. Which is the right one?
Then compiled nginx with --addmodule=/path/to/modsec source, but after nginx is compiled it does not find any mod security command. Here is an example from the error log:
2017/06/05 20:42:39 [emerg] 11011#0: unknown directive "SecStatusEngine" in /usr/local/nginx/conf/modsecurity.conf:2
2017/06/06 11:35:16 [emerg] 25759#0: ModSecurityConfig in /usr/local/nginx/conf/modsecurity.conf:3: Unknown command in config: ModSecurityConfig
2017/06/06 09:34:02 [emerg] 12296#0: unknown directive "SecStatusEngine" in /usr/local/nginx/conf/modsecurity.conf:2
So it does not find at all mod security although it's compiled. I had the following directory structure in /usr/local/modsecurity,but I wonder why there is not include folder?
[root@activewaf-compile ModSecurity]# ll /usr/local/modsecurity/
total 8
drwxr-xr-x 2 root root 4096 Jun  6 10:26 bin
drwxr-xr-x 2 root root 4096 Jun  6 10:26 lib
Is it normal and should I export something to work properly? What is wrong in my steps?
Here is the result that show nginx is compiled with mod security:
[root@XXX]# /usr/local/nginx/sbin/nginx -V
nginx version: nginx/1.12.0
built by gcc 4.8.2 20140120 (Red Hat 4.8.2-15) (GCC)
built with OpenSSL 1.0.2g  1 Mar 2016
TLS SNI support enabled
configure arguments: --prefix=/usr/local/nginx --error-log-path=/var/log/nginx/error.log --with-XXXXX --add-module=/usr/local/src/ModSecurity/nginx/modsecurity
In nginx.conf I have the following includes:
include /usr/local/nginx/conf/modsecurity.conf;
include /usr/local/nginx/conf/crs-setup.conf;
include /usr/local/nginx/conf/rules/*.conf;
Where exists the following configuration options and others:
ModSecurityEnabled on;
SecStatusEngine on;
ModSecurityConfig /usr/local/nginx/conf/modsecurity.conf;
SecRuleEngine On
Thank you in advance!

Hi @intelbg,
You have to place the ModSecurity directives inside the modsecurity_rules block, like demonstrated here:
https://github.com/SpiderLabs/ModSecurity-nginx#usage-example-loading-rules-from-a-file-and-injecting-specific-configurations-per-directoryalias

Thank you about your reply. I tried this, but after place     modsecurity on; in the server block I receive:
nginx: [emerg] unknown directive "modsecurity" in /usr/local/nginx/conf.d/shared.conf:2
Also as I set modsecurity_rules_file and the path to the mod security.conf file in the location block as the article you show me I receive error:
unknown directive "modsecurity_rules_file"
I forgot to mention that I use nginx as proxy pass in front of apache, so in nginx/conf.d dir I have a ip.conf file for every IP on the server. The idea is after building configuration to bundle it in rpm, so I think that I should transfer /usr/local/modsecurity too (but I didn't receive an answer why include dir there is missing and if I should do some exports).

Make sure that your nginx binary is really the onde that you have compiled with ModSecurity support. Please give it a try on the example of the documentation and see if it is working or not.

@zimmerle I posted in my first reply the output from nginx -V where in the output you can see that this is the binary, compiled with mod security. Also in nginx log after restarting nginx there are messages like this:
017/06/06 16:20:32 [notice] 7775#0: ModSecurity for nginx (STABLE)/2.9.0 (http://www.modsecurity.org/) configured.
2017/06/06 16:20:32 [notice] 7775#0: ModSecurity: APR compiled version="1.3.9"; loaded version="1.3.9"
2017/06/06 16:20:32 [notice] 7775#0: ModSecurity: PCRE compiled version="7.8 "; loaded version="7.8 2008-09-05"
2017/06/06 16:20:32 [notice] 7775#0: ModSecurity: LIBXML compiled version="2.7.6"
2017/06/06 16:20:32 [notice] 7775#0: ModSecurity: Status engine is currently disabled, enable it by set SecStatusEngine to On.
I copied the example and it toes not work. I provided to you the errors. It does not find the modsec command. Any other suggestions?

Thank you about your reply. In my first reply I posted the version - it’s Modsecurity version is 2.9.1.
You wrote that the configuration directives in mod security 2.9.1 are different than the nginx connector. I do not want to use nginx connector because it’s limited in the rules and the functionals. I want to setup basic nginx environment with mod security and when it works to setup rules that works for our websites. What are the correct directives for mod security 2.9.X in nginx as the information I read in internet is the same? The examples in the GitHub for mod security are the same.

On Jun 8, 2017, at 2:07 PM, Felipe Zimmerle<EMAIL_ADDRESS>wrote:
@intelbg https://github.com/intelbg Are you working with ModSecurity v2.9.x or 3.x they are different things. You have to choose one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/SpiderLabs/ModSecurity/issues/1443#issuecomment-307072324, or mute the thread https://github.com/notifications/unsubscribe-auth/ALkouGqeBbCeQ7Xx_jEvuVF5-dGnUT69ks5sB9X3gaJpZM4NxBqr.

ModSecurity version 2.9.x configuration for nginx:
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#configuration-steps

This is the configuration I tried and the problem is that I receive the following errors in nginx configtest:
Unknown command in config: ModSecurityEnabled
Although nginx is compiled with mod security:
/usr/local/nginx/sbin/nginx -V
nginx version: nginx/1.12.0
XXXX --add-module=/usr/local/src/ModSecurity/nginx/modsecurity
This is the reason why I am writing to you. I am not sure why nginx does not find modes directives as it’s compiled with mod security by the way I posted in my first reply.

On Jun 8, 2017, at 2:33 PM, Felipe Zimmerle<EMAIL_ADDRESS>wrote:

ModSecurity version 2.9.x configuration for nginx:
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#configuration-steps https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#configuration-steps
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/SpiderLabs/ModSecurity/issues/1443#issuecomment-307077287, or mute the thread https://github.com/notifications/unsubscribe-auth/ALkouM-k-adVt6JGitKJavPbogb_iirnks5sB9wLgaJpZM4NxBqr.

What happens if you try: /usr/local/nginx/sbin/nginx  -t ?


Add a little icon next to overclocked items and saged items

This would help alot when farming for saged/overclocked items

do you happen to know the hypercube nbt for oc'd items?

found it


Wfa super pr

Super PR with pending PRs to unblock WFA certification.

Both the pending PRs merged now, closing this.


function name change IPs to IPv4s

I need to distinguish between IPv4 and IPv6.
Ip means both of them.
How about function IPs return all ips and only function IPv4s return IPv4 address.

Seems great :D
Could you contribute for them and send me a PR?

Related PR :  #16


Clarify usage of SSL in kube_apiserver_metrics and use Kube CA by default if present

What does this PR do?
Previously this module believed that ssl_verify was false by default. Though it's not the case.
From https://github.com/DataDog/integrations-core/blob/master/datadog_checks_base/datadog_checks/base/checks/openmetrics/mixins.py#L231
It's in fact true by default (for years apparently).
So changing the behavior to expect ssl_verify as true by default and use default Kube CA to target APIServer if possible.
Motivation
Check does not work out of the box at least in Kube 1.16 setup locally with kind.
Additional Notes
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

In fact, this is not the root cause of the issue, so we don't this PR, except if we want to move call to apiserver to ssl_verify+default cacert.


The attributes for variants are not removed in the UI, if they are not present in one of the variants

Example. PIM
variants created.
Bue XL
Blue L
Blue S
Red XL
the ui in the boilerplate, will generate variants for the sizes XL, L ,S, which works correctly, but when red is selected, L,S size attributes are still displayed, which does not exist for red. this causes and error in the boilerplate.

Hi @stunjiturner and thank you for this bug report. Fixed in main now


Pick pick 25550:make enable_stats session var

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall


Add support for TLS logging

Add support for TLS logging, including test suite. By far the
most tricky part here is getting certificates that golang
works with, and getting the TLS config correct.
Signed-off-by: Alex Bligh<EMAIL_ADDRESS>

Awesome!


Clicking the plus to expand a property with a nested resource does not work

receiving console error TypeError: undefined is not an object (evaluating 'e.map')
Jeremy believes #493 (merged PR) may solve this

PR #493 did not resolve this issue. Further testing in Chrome, getting this error:
Uncaught TypeError: Cannot read property 'map' of undefined
    at t.resourceTemplatePromises (bundle.js:51)
    at bundle.js:51
    at Object.<anonymous> (bundle.js:67)
    at p (bundle.js:67)
    at bundle.js:67
    at T (bundle.js:67)
    at P (bundle.js:67)
    at x (bundle.js:67)
    at A (bundle.js:67)
    at On (bundle.js:67)```