Add english version

40 words might be misspelled, please check them: Cryptographic, FB, fidoalliance, FIPS, GitHub, GnuPG, gpg, GPG, HOTP, https, io, keybase, keychain, keytocard, Kraken, nd, NEO, Nitrokey, NitroKey, ok, openpgp, OpenPGP, paire, parfaite, partie, passwd, presque, ransonware, SmartCard, ssb, subkey, subkeys, TOTP, tthuon, usb, wikipedia, Yubico, yubikey, Yubikey, YubiKey

41 words might be misspelled, please check them: Cryptographic, FB, fidoalliance, FIPS, GitHub, GnuPG, gpg, GPG, HOTP, https, io, keybase, keychain, keytocard, Kraken, nd, NEO, Nitrokey, NitroKey, ok, openpgp, OpenPGP, paire, parfaite, partie, passwd, presque, ransomware, SmartCard, ssb, subkey, subkeys, TOTP, tthuon, usb, wikipedia, Yubico, yubikey, Yubikey, YubiKey, Yubikeys

41 words might be misspelled, please check them: Cryptographic, FB, fidoalliance, FIPS, GitHub, GnuPG, gpg, GPG, HOTP, https, io, keybase, keychain, keytocard, Kraken, nd, NEO, Nitrokey, NitroKey, ok, openpgp, OpenPGP, paire, parfaite, partie, passwd, presque, ransomware, SmartCard, ssb, subkey, subkeys, TOTP, tthuon, usb, wikipedia, Yubico, yubikey, Yubikey, YubiKey, Yubikeys


I'm attempting to post to the meetup api. Values in the url are accepted, a json body is not.

Using the API key authentication.
As an example, when creating an event:
Url: https://api.meetup.com/{groupname}/events?&sign=true&key={apikey}&name=New+event
works correctly. However, the following does not:
Url:
https://api.meetup.com/{groupname}/events?&sign=true&key={apikey}
Header:
Content-Type: application/json
Body
{
    "name" : "Test event"
    "description" : "Description from request body"
}

returns a 400 Bad Request with body:
{
  "errors": [
    {
      "code": "name_error",
      "message": "name is required",
      "field": "name"
    }
  ]
}

As far as I can tell, the api should accept a json body for the post. If this is not the case, this needs to be clearer in the api documentation. Is it that only url or x-www-form-urlencoded data is accepted?

Sorry this was not more clear x-www-form-urlencoded the form most data creating/changing services expected. In the few cases we support photo uploads, we also support multi mime form posts.

So post requests with a JSON body is not accepted?

Ran into this too. It would be nice to have this info on https://www.meetup.com/meetup_api/docs/


Fixed bugs reported on issue #20

The references and logic missing has been added.

No conflicts. Merge accepted.


Bug Report: stat_val containing date (yyyy-mm-dd) Transforms to Integer

Using TableColumnStats, stat_val containing dates are displaying as integers, with the values being the subtraction calculation of the date pieces.
The output from my Athena query looks like this:
"db","cluster","schema","table_name","col_name","stat_name","start_epoch","end_epoch","stat_val"
"mydb","mycluster","myschema","mytable","column1","distinct values","1603238400","1603238400","437553"
"mydb","mycluster","myschema","mytable","column2","average","1603238400","1603238400","84712"
"mydb","mycluster","myschema","mytable","column3","max","1603238400","1603238400","2020-10-20"
"mydb","mycluster","myschema","mytable","column4","min","1603238400","1603238400","1945-06-30"

Expected Behavior
Date values should render as strings, not integers.
Current Behavior
The stats for column1 and column2 display fine on the front-end.
But the stats for the date columns (column3, column4) are displaying as integers, with the values being the subtraction calculation of the date pieces.
"2020-10-20" => 1990
"1945-06-30" = > 1909

Checked neo4j for one of the values, and stat_val is showing as integer:
{
  "identity": 1833,
  "labels": [
    "Stat"
  ],
  "properties": {
"stat_name": "min",
"end_epoch": "1603238400",
"start_epoch": "1603238400",
"publisher_last_updated_epoch_ms":<PHONE_NUMBER>050,
"stat_val": 1909,
"published_tag": "unique_tag",
"key": "<removed for privacy>/min/"
  }
}

Context
So may be an issue with the data builder?
This is a blocker for our use as we need to display min/max of several dates in our datasets.
Screenshots (if appropriate)

Your Environment
elasticsearch:6.7.0
neo4j:4.0
amundsendev/amundsen-search:2.1.3
amundsendev/amundsen-metadata:2.5.4
amundsendev/amundsen-frontend:2.1.1
amazon/amazon-ecs-agent:latest

cc @ttannis @Golodhros @dikshathakur3119 @allisonsuarez

Posting context in case someone gets to the fix before I can. We introduced automatic number formatting of stats, and did not consider that something other than numerical values would be sent as a stat: https://github.com/amundsen-io/amundsenfrontendlibrary/pull/723. cc @dikshathakur3119
Instead of using const numberVal = +stat_val, we could explicitly check if stat_val is a number before formatting else render stat_val as is.

@ttannis FYI, the documentation here: https://github.com/amundsen-io/amundsenfrontendlibrary
includes a screenshot of stat values that are strings: https://github.com/amundsen-io/amundsenfrontendlibrary/blob/master/docs/img/column_details.png
min: aardvark
max: zebra

Working on this now @ttannis

Should be fixed with the PR mentioned above!


chore: Fix inconsistency in environment variables between the docs andd docker-compose.yml

Thanks for submitting a PR! Please check the boxes below:

[ ] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Updated the docker-compose.yml to use an environment variable that is actually read in the settings here: https://github.com/Flagsmith/flagsmith/blob/main/api/app/settings/common.py#L36C1-L41C6
How did you test this code?
Manually - started the container with the ENVIRONMENT variable set to a different value than any currently allowed.The warning was then triggered.

Great thanks!


Sign out from Peatio does not work.

User cannot sign out from peatio UI, they have to go to barong UI to sign out.
Under user profile, on clicking "Exit", user does not sign out. The session is still maintained even though the message is shown "Sign in with Barong".
Clicking, "Sign in with barong", redirects the user to the profile page.
workbench: master
peatio: 1-8-stable
barong: 1-8-stable

@umaimehm Yes because you are still signed in Barong
For example you sign out from some service which uses Google OAuth you still signed in Google
Reopen this issue if you still have question


OCPBUGS-1831: Improve error propagation

ovn-get dbs: set default output folder for must-gather, ignore optional input
pod-run-netns-command: catch user's ctrl+c while running command, don't log error in that case
utils: propagate some subshell errors
Signed-off-by: Nadia Pinaeva<EMAIL_ADDRESS>

/jira refresh

/assign @flavio-fernandes


shimv2: support attachable binary I/O

nerdctl uses binary I/O to support detached mode, but this is not sufficient because it does not allow reattachment. This PR introduces a new I/O attachablebinary that allows detaching and reattaching of I/O managed by the containerd-shim

maybe we can merge attachablebinary with existing one, binary, by using reserve param key like _attach=true.

cc @AkihiroSuda @dmcgowan
I intended to merge them but I think it will introduce a non-negligible complexity and more responsibilities to NewBinaryIO func so I prefer to make a seperate function

/retest

Really not a fan of these runc specific options that really have nothing really to do with runc.
That said this seems fine to unblock an existing workflow.
It would also be extremely helpful to include an explanation of what this does and how its expected to be used in code and the commit.


Post it to Winget also

Feature/enhancement description:

Install and update it by official pkg manager!

Sorry, no time for this.

Anyone is allowed to submit apps to WinGet, but note that support for apps distributed as ZIP archives is currently limited compared to EXE/MSI installers.


sched/wdog: Change the default value of MAX_WDOGPARMS from 4 to 2

Summary
since the maximum number of argument pass to wd_start in the whole
code base is 2 and change CONFIG_MAX_WDOGPARMS in some defconfig
from 1 to 2 oherwise pthread_condclockwait will fail
Impact
Save 8 byes for each wdog instance
Testing

@xiaoxiang781216 I like the idea of saving memory. But this would be unbeknownst reduction that will silently impact downstream users of Nuttx.
I believe what Greg has argued in the past that Kconfig changes need to be non-breaking/non changing.  Isn't this the same situation?
The default was 4 so in a case where 4 was what the user had selected their last 'make oldconfig' has deleted it.
Therefore there is no defconfig entry. On running 'make oldconfig' with this release the build will have a value of 2.

@xiaoxiang781216 Ahh PARMS! Not Dogs. Never mind. The compiler should catch the overflow assignment on 'wdparm_t           parm[CONFIG_MAX_WDOGPARMS].

I understand why you remove all occurrences of CONFIG_MAX_WDOGPARMS=1 when the default changed to 2, but whcy do you remove CONFIG_MAX_WDOGPARMS=1?  Those are tiny configurations that might depend on the smaller value.

I understand why you remove all occurrences of CONFIG_MAX_WDOGPARMS=2 when the default changed to 2, but whcy do you remove CONFIG_MAX_WDOGPARMS=1? Those are tiny configurations that might depend on the smaller value. I think you should restore those few cases.

Because pthread_condclockwait require two arguments:
https://github.com/apache/incubator-nuttx/blob/master/sched/pthread/pthread_condclockwait.c#L289

It's dangerous to set CONFIG_MAX_WDOGPARMS=1 but don't set CONFIG_DISABLE_PTHREAD=y
Anyway, I will provide a patch to remove wd_pool which will release 1KB by default, so the expansion here is safe.

I understand why you remove all occurrences of CONFIG_MAX_WDOGPARMS=2 when the default changed to 2, but whcy do you remove CONFIG_MAX_WDOGPARMS=1? Those are tiny configurations that might depend on the smaller value. I think you should restore those few cases.

Because pthread_condclockwait require two arguments:
https://github.com/apache/incubator-nuttx/blob/master/sched/pthread/pthread_condclockwait.c#L289

It's dangerous to set CONFIG_MAX_WDOGPARMS=1 but don't set CONFIG_DISABLE_PTHREAD=y
Anyway, I will provide a patch to remove wd_pool which will release 1KB by default, so the expansion here is safe.

So it is basically not really a configuration "option".  It must be set to two.  So, again, I think if the default is two (which seems okay), then the function prototype should change to avoid the variable number of arguments.  It is silly to have a non-optional configuration setting; it is silly to support variadic arguments when there is not more than two.

@patacongo removing variadic argument should belong another patch, we can merge this patch first if you don't have other concern.


membrane and membrane fusion issues - missing logical definitions

missing parent
nuclear membrane fusion (GO:0000740)
please add
nuclear membrane organization (GO:0071763)

can this be added? There is another ticket that is a bit more detailed. I will follow up on that one at the w/end

I dont understand why we dont get this from the logical definitions.
@balhoff Can we look at this together ?

Missing LD definitiond for membrane fusion and nuclear membranes:
nuclear membrane fusion
remove:
-is_a: GO:0090174 ! organelle membrane fusion
GO:0016020 membrane
add LD:
+intersection_of: GO:0016020 ! membrane
+intersection_of: part_of GO:0043226 ! organelle
GO:0031090 organelle membrane
remove:
-relationship: part_of GO:0043226 ! organelle
nuclear membrane
remove:
-is_a: GO:0031090 ! organelle membrane
-relationship: part_of GO:0005634 ! nucleus
add LD:
+intersection_of: GO:0016020 ! membrane
+intersection_of: part_of GO:0005634 ! nucleus

@balhoff
I just figured out that the relation used in the LD for membrane fusion and children is 'results in fusion of'

in the go-obo file, it looks like it's in GOREL:
[Typedef]
id: results_in_fusion_of
name: results_in_fusion_of
namespace: external
xref: GOREL:0002005
is_a: has_participant ! has participant

However I cannot find it in GOREL. Should I add it to RO ?
@cmungall

yes, go ahead and add it!

Added relation in RO: https://github.com/oborel/obo-relations/issues/430
RO:0012008 results in fusion of

Also, added LD to some high level terms:
GO:0043227 membrane-bounded organelle
organelle
and 'has part' some membrane
GO:0065010 extracellular membrane-bounded organelle
'membrane-bounded organelle'
and 'part of' some 'extracellular region'
GO:0043231 intracellular membrane-bounded organelle
'membrane-bounded organelle'
and 'part of' some 'intracellular anatomical structure'
GO:0043232 intracellular non-membrane-bounded organelle
'non-membrane-bounded organelle'
and ('part of' some 'intracellular anatomical structure')
GO:0043264 extracellular non-membrane-bounded organelle
'non-membrane-bounded organelle'
and ('part of' some 'extracellular region')

This should get fixed once the RO gets updated.

@balhoff Can we reload RO ?

Sure, I started a build for the RO import.

I suppose that as long as there is no PR this has not yet gone through?


docs: update README for jest-worker

Summary
README.md of jest-worker package is out of date since now the exposed class isn't Worker anymore but called JestWorker. This PR is to update that.
Test plan
N.A.

Codecov Report

Merging #9788 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9788   +/-   ##
=======================================
  Coverage   64.87%   64.87%
=======================================
  Files         288      288
  Lines       12206    12206
  Branches     3023     3025    +2
=======================================
  Hits         7919     7919
  Misses       3646     3646
  Partials      641      641

Continue to review full report at Codecov.

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

@SimenB

Hmm, it's technically a default export so it doesn't really matter what you import it as, but I guess it'd be better to use the original name (the function still has the name and it could turn up in errors/stacktraces etc.)

The changelog has a conflict, but I think for a change like this we can just drop the changelog entry (so minor that there is no need), then we can merge

Thanks I will drop the change for CHANGELOG.


[resolves #246] Cannot load components defined in system contexts

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/152/
Test PASSed.


chore(deps): upgrade eslint

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

In #50601 , renovate bot tried to update eslint. But it needed a little help. Upgrading eslint required a chain of other package upgrades, and some of those led to updated rules. So I mostly added one line eslint disable rule or made small changes to comply.
Closes #50601

also the automerge got removed since i pushed changes. fyi

Yep – That is intentional. It's a security thing, to prevent people from pushing dangerous changes once approved and ready to merge. That would bypass the protections.
You don't have to worry about it :)


(yagan) update cluster.yml; change etcd/controlplane to yagan01-05

Uhh... yamllint is picking up .yml in the branch name under .git.  It should probably be excluding .git.  I'm going to reopen the PR from a different branch name.


Wrong global orientation

def batch_orth_proj(X, camera, mode='2d'):
    camera = camera.view(-1, 1, 3)
    s = camera[:, :, 0].unsqueeze(-1)
    X_trans = X[:,:,:2].contiguous()
    if mode=='2d':
        X_trans = s * X_trans + camera[:, :, 1:]
        return X_trans
    elif mode=='v3d':
        X[:, :, :2] = s * X_trans + camera[:, :, 1:]
        return X
    elif mode=='j3d':
        X[:, :, :2] = s * X_trans/torch.abs(s) + camera[:, :, 1:]
        return X
    else:
        print('projection mode is not included')
        return X

https://github.com/Arthur151/DSD-SATN/blob/3fb61686c6161bf89a91aab7635eb57a58b04c25/src/model/model.py#L261
It seems that you directly manipulate the tensor passed in, so verts and verts_cam here are the same value, and vertices of the body model is coupled with the camera.

@Mercurialzhang Yes, you are right!
If you want to get vertices decoupled from the camera rotation, please just make the change like,
verts_camed = util.batch_orth_proj(verts.copy(), cam, mode='v3d')
The code will be revised later. Thanks for your kindness.

Thanks for your quick reply! I will have a try


Add new quirks for SOL001 parsing

This PR merges in a single commit all the changes originally discussed in PR#58
2 new quirks are added
imports.ignore.topology_template
ignore topology_template in imported units
substitution_mappings.requirements.allow_dangling
allow a requirement in substitution_mapping to point to an
undeclared requirement of a node in node_template

Hi, in my latest commit I did some cleanup and change the name of "substitution_mappings.requirements.allow_dangling" to ""substitution_mappings.requirements.permissive" (and also changed its implementation a bit). Also made sure all the documentation matched.


Improve representation of trace spans.

This is a sketch of a better interface for trace spans that does not involve a Tracer object. Instead, anything which needs to produce traces receives a SpanParent, which is a reference to a parent span, and can create children from that.
An additional goal of this refactoring is to eliminate the dependency on Jaeger which also allows us to eliminate the dependency on Protobuf. Jaeger is one system that these traces could feed into, but by abstracting the API we can potentially support many different trace collectors.
The classes are designed such that if no tracing is active, no extra memory allocations are needed.
This is very incomplete. I'm uploading it for discussion purposes.
The interface proposal is in trace.h. The rest is showing how some things would change to use this.

OK, this is complete as far as this repo is concerned. I need to update the internal codebase, obviously, especially re-introducing the Jaeger code there.

I removed the commit that drops Jaeger for now, since I wanted to actually implement the SpanObserver interface in those files. A later PR will move the files out of this codebase so that we can drop the protobuf dependency.
The  force-push only removes that commit. The new code is in the two added commits.
This is now passing internal tests.


fix: fixed crash when the Bitmap size is zero in the ClusterRenderer

This PR prevents a crash when the Bitmap size is zero in the ClusterRenderer.
measuredWidth and measuredHeight typically return 0 when we call them from onCreate, which does not seem to be the case. This is also not deterministic and not always reproducible. For some reason, the view has not been drawn when we hit this point.
This PR will prevent the crash, although the causes need to be determined.
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[X] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[X] Ensure the tests and linter pass
[X] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #297

🎉 This PR is included in version 2.11.5 🎉
The release is available on:

v2.11.5
GitHub release


Restore network configuration via kernel cli

Restores the ability to set the network IP address, gateway and subnet mask via the kernel commandline arguments -ip, -gateway and -mask.
Closes #522

I agree that would make sense, I will try to implement that :)


Fix tests

Description
Some tests are currently broken on main. Trying to fix those:

fix test_short_license_id (DEC-3-Clause' detected instead of BSD-3-Clause`) - thanks to @dholth (commit from https://github.com/conda/grayskull/pull/541)
fix --pypi-url: always raised a RuntimeError (args.url_pypi_metadata has a default value and will never be empty)
fix meta.yaml output directory (the meta.yaml file was always created in the current directory instead of recipe_dir)
fix test_poetry_langchain_snapshot: replace pypi.io with pypi.org (pypi.org is the official domain)

test_injection_distutils_compiler_gsw is failing on python 3.12 due to:
DEBUG    grayskull.strategy.py_base:py_base.py:460 Exception when executing setup.py as script: module 'configparser' has no attribute 'SafeConfigParser'
SafeConfigParser was removed in 3.12.
This comes from versioneer.py included in the sdist. Test should be updated to use another package/version.

thanks for fixing the tests, much appreciated! :)


add lagoon-cli to lagoon CLI images

Added a comment in the lagoon-cli issue above.
Do you think it's better have lagoon-cli in all cli images or just toolbox one?

discussed elsewhere


Add the option to have a different release for vivado_HLS and Vivado

Vivado HLS used to compile the actions coded in C is totally independent from the vivado used for building the chip. It could be helpful to add a variable so that we can have different released co-existing together. It is mandatory to display the HLS release used as soon as it becomes different

Implementation done in #868 of the display of the HLS release used when HLS compiler is called.

This is feasible in theory but using a vivado 2018.2 to use a vivado hls 2018.3 often reports some libraries issues, because new libraries are not known in 2018.2. Closing this issue since nothing can be done on this side.


Implement Horizontal Pod Autoscaler

policy should adjust up or down to as request demands change could be node, pod or both

This is no longer required.


fix: let eslint not to find config for virtual module

fixes https://github.com/storybookjs/builder-vite/issues/367
eslint tries to find config on executing isPathIgnored, and it errors if the config file does not exist.
This PR suppresses to call eslint method in transforming virtual module.

You can test this build by adding git url in place of version tag:
  "devDependencies": {
    "vite-plugin-eslint": "git+https://github.com/sunghwan2789/vite-plugin-eslint.git#d9540d8bd010612ce463530c1184e92f4c9bd37a"
  }

needed this to be merged

When will this be merged? Until this is merged I can't use eslint with Vite and Storybook

Hey there! You should consider vite-plugin-checker as a replacement to this plugin. It works with no problem.

Hey there! You should consider vite-plugin-checker as a replacement for this plugin. It works without any problem, but also supports type checking.

No one took this comment serious, but this is today an abandoned project, I mean 2 years (almost 3 w/o an update)


bug(card): box-sizing: border-box does not work when adding border

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/components-issue-drfdkd?file=src/app/example-component.css
Steps to reproduce:

Create a simple card with click event to add/remove the border

Expected Behavior
What behavior were you expecting to see?
border-box should be applied when toggle border visibility.
Actual Behavior
What behavior did you actually see?
border-box is not applying so the card jerks up and down when toggle border visibility.
Environment

Angular: 10.1.1
CDK/Material: 10.2.1
Browser(s): Chrome, Firefox

The card gets moved around, because it doesn't have border to begin with and you add one afterwards. Closing since this is an issue with your style override, rather than the styles we provide.


Invalid route action: [count]

Laravel Version: 8.12.0
PHP Version: 7.4
Database Driver & Version:

Description:
After upgrade to 8.12 it is impossible to use "count" as function name for route
In routes.php:
Route::get('workers/count', [WorkersController::class, 'count']);
throw the error "Invalid route action: [count]."
If I change it to Route::get('workers/count', WorkersController::class . '@count');
it works correctly
On 8.11.2 the error is not reproduced

We're looking into this

Hmm, I can't recreate this. I do however hit that error if I don't import the controller properly in my routes file.
Have you "use App\Http\Controllers\WorkersController;` at the top of your file?

Fixed in upcoming patch.

Patch released.

It is working well now, thank you!


How to get text from a specfic reddit thread instead of hot posts on a subreddit

How do I edit the scrape reddit/ task.py so that I can input a specific reddit thread to get comments , instead of get hot posts , thanks.

Hi! Thank you for your interest! This was actually a feature I was planning on implementing but haven't gotten to it yet due to school. I think maybe we should divide this between two tasks: scrape_subreddit and scrape_post. We could call the right one depending on the parameters passed, so if thread_id is passed we scrape the thread, else if subreddit_name is passed then we scrape the subreddit instead. Then we need to adapt the data passed to the other tasks because they expect a post object.


TextBlocks updating incorrectly

I am having trouble with an app for work. I tried to create a minimal example, but it didn't reproduce the bug.
I have a list of TextBlock with onPointerSelected handlers.
for KeyValue (experimentId, experiment) in state.Model.Experiments do
    TextBlock.create [
        TextBlock.text experiment.Name
        TextBlock.onPointerPressed (fun e ->
            e.Handled <- true
            dispatch (Msg.SelectionChanged (NewSelection.Experiment experimentId)))
    ]

The Experiments fields that is being iterated through is a Map<int, Experiment>.
type Experiment =
    {
        Id: ExperimentId
        Seed: string
        TimeUnit: TimeUnit
        Duration: string
        Iterations: string
        Name: string
        SinkBuffers: string
        TheoreticalLimit: string
    }

When I click on the TextBlock, a Msg is dispatched that's supposed to update a selection. What's happening, though is that the text in the fields is getting swapped around. You will notice in this GIF how the first entry keeps having the number change. This should not be happening. I tried using the View.createWithKey trick to force a re-render, but that doesn't help.

I'm sorry for the lack of a fully reproduced example. This is an internal app and I can't share the source code. I tried reproducing a small example, but it didn't have the same bug.

Hey @matthewcrews 👋
I see 2 problems:

Iterating over the map Map<int, Experiment> will not return items in a stable order. Sort them by some stable value before rendering or items might change/swap places.

When items change places the backing controls might be reuses (if the type matches and no key it set that changes)

FuncUI has no idea if the onPointerPressed function changed. (if views swap places, it did)
You can give FuncUI the necessary context, by passing SubPatchOptions

@JaggerJo I'm actually fairly certain that a Map returns the elements in ascending order of the keys. Also, I was originally using a ResizeArray as the backing collection and I had the exact same issue.
I have the same problem with the UI when I change the code to this:
let orderedItems = state.Model.Experiments |> Seq.sortBy (fun (KeyValue(k, _)) -> k)
for KeyValue (experimentId, experiment) in orderedItems do
    TextBlock.create [
        TextBlock.text experiment.Name
        TextBlock.onPointerPressed ((fun e ->
            e.Handled <- true
            dispatch (Msg.SelectionChanged (NewSelection.Experiment experimentId)))
            , SubPatchOptions.Always)
    ]

Could you provide me with a stripped app so I can reproduce it locally?

I will try. I did try initially but a simple example did not reproduce the bug. I will try to take the original app and strip out the proprietary code.

Here is a repo that reproduces the bug:
https://github.com/matthewcrews/ConfusingTextBox
Run the app, go to the Experiment sheet by clicking on the icon that looks like chemistry beakers:

Add several Experiments by clicking on + button, and try clicking the different text boxes.

You'll see that the first experiment you click on will have the displayed name constantly change as you click between them.

@matthewcrews I've looked at your code and I know why it happens. Also I know you don't want to hear it but the component model is great and helps with things like this 😉.

So.. you have a list of experiments on the left. You render details of the selected experiment details on the right.
The problem is that the details view is not keyed, so FuncUI will patch it. By patching it (e.g. setting TextBox.text to the selected experiments name) the already existing handler TextBox.onTextChanged will fire. This handler does not have SubPatchOptions set, so it will never change. The first experiment you click on will set all the handlers - because the handlers close over state.
→ Use keyed views when you switch the views context. (otherwise you need to be careful with your handlers closing over state)

Adding a key here fixes the issue described above:

I am confirming this resolves the issue.


Simulate tumors using haploid samples

We have a Q script for in silico mixing, which we should port to wdl, and we have haploid exomes and genomes.  We should use this to make great truth data for testing Mutect.

Issue moved to broadinstitute/gatk #2974 via ZenHub


Adding a new rebus

Associated Issue: #
Summary of Changes

added a new rebus, "football"

Pull Request Test Coverage Report for Build 926

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.143%

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


First draft of integrators section added

Hopefully this represents a good base that others can build off of.  The references will have to wait until I get back from a conference and have access to my books.  Otherwise, please let me know where you think more details should be added and I can work on that more immediately.

Thanks! I'll have a look soon.

Will take a look more in-depth in the morning. First cursory glance seems good though

Great, thanks!  It'll be a little tight timing-wise, but I'll get to these on Saturday once I'm home and can also fix all my references while looking at physical copies of textbooks.

Yeah, I have the same kind of issue; I'm on the road and don't have any books here so I can't look anything up.


otomi cli calls run twice

Describe the bug: (a clear and concise description of what the bug is)
When running otomi cli calls they seem to get called twice. When running the following I see diff output twice:
DEBUG=1 otomi diff -l name=loki

DEBUG=1 otomi diff -l name=loki

Looks like DEBUGs meaning in the context of bin/otomi has changed to TRACE.

I checked out the master branch, and pulled the latest code.
For testing purposes, I changed the Gitea version number from 1.13.4 to 1.13.6
Then I both: DEBUG=1 otomi diff -l name=gitea and TRACE=1 otomi diff -l name=gitea
In both cases the output was the same and I did not find the code running, or seeming to run twice.
If this is still a bug, that I am not aware of, please amend the description, so it can be more easily reproduced. As for now it seems this issue does no longer exist in master

closing then...tnx


Remove markdown:guides task which doesn't exist anymore

I think this should have been removed as a part of db13a40. See https://github.com/atom/atom/issues/5569 for more context.
cc @kevinsawicki

:+1:


#50: Create or adjust endpoint for accepting and saving checkpoint data

This PR implement a functionality of saving data into the postgreSQL database using slickpg.
Added Fa-db library for communication with the database.
impelemented a checkpoint controller as PrototypeController and injected DatabaseService for communicating with the db.
Changed checkpoint service to DatabaseService.
Defined postgres engine in the application.properties in the server package.
Added postgres configuration in the AtumConfig.
Created and instantiated runs class for the server to be able to communicate with the sql function that create a checkpoint in the database.
Added the package for access provider and implemented access providers.
Closes #50

The code for PrototypeController and DatabaseService looks clean. I understand that at this point the whole checkpoint creation process in the db would be difficult to test. For the future work (other endpoints) I would suggest to include test cases against a database already.

Is here plan to focus the low code coverage of changed/touched classes?

I am missing unit tests for at least new code. Is their implementation planned?

After last commit build is failing:
08:14:07.848 [pool-8-thread-3-ScalaTest-running-MeasureTest] DEBUG io.netty.buffer.PooledByteBufAllocator - -Dio.netty.allocator.maxCachedByteBuffersPerChunk: 1023
[error] /Users/ab024ll/repos/absa/branches/atum-service_74/server/src/main/scala/za/co/absa/atum/web/api/service/DatabaseService.scala:36:46: missing parameter type
[error]     wroteCheckpointFuture.thenApply[Boolean](_ => TRUE)
[error]                                              ^
[error] one error found

@miroslavpojer tests will be part of DB test PR, it would be too big here; otherwise, no need to test model/ and server/ so far is actually not doing too much - no crazy data transformations etc, mostly just calling DB


Import funtion not working

Import Cocktail is not working properly.
the URL Scraping is working fine, but if you click on "continue with editing" on the bottom rigtht you get forwared to "http://SALTRIM/cocktails/form" which is blank , if you refresh the site the form appears but is empty

Issue still persists... v1.17.0 is working fine if i go to the latest v1.20.0 i get a blank screen after importing the cocktail to the cocktail form


Add KerbalKrashSystem 1:0.3.2

name is better in this version :+1:

Looks good to me, merging with my thanks.


Set CC and AR for specific targets. Fixes #17.

This should avoid setting HOST_CC to a cross-compiler in build.rs
scripts that consume the cc crate.

@rubdos I've invited you to collaborate on this repo.  Can you see if this addresses the problem you were seeing locally?

@ncalexan: Could you comment on the patch I just "had to" do on my project in my build script?
diff --git a/backend/build.rs b/backend/build.rs
index feb2792..4c6f700 100644
--- a/backend/build.rs
+++ b/backend/build.rs
@@ -72,7 +72,7 @@ fn main() {
 }

 fn get_gcc_system_include_dirs(target: &str) -> Result<Vec<PathBuf>, String> {
-    let gcc_cmd = match env::var("CC") {
+    let gcc_cmd = match env::var("RUSTC_LINKER") {
         Ok(path) => path,
         Err(_) => target.to_owned() + "-gcc",
     };

I wonder: RUSTC_LINKER is always the target linker, not the host linker? For me, this gets linked through to build/linker-wrapper/linker-wrapper.sh
This also worked:
    let gcc_cmd = match env::var(&format!("CC_{}", target)).or(env::var("CC")) {
        Ok(path) => path,
        Err(_) => target.to_owned() + "-gcc",
    };

What is preferred, and why?

Am I correct that you're trying to compile Rust code both for your host and for your target against the same jni.h?

No, no. I'm only building for Android here. This function calls clang and figures out where to locate the jni.h for Android. Apparently, jni.h differs quite a bit between Android and regular hosts.

And sorry for breaking you: I'd usually wait for feedback but I have some internal deadlines to hit for this work. I'd like to get a test suite in place so that we assert more of our expected behaviour but it's surprisingly hard to actually test this plugin. Help appreciated: #22.

You broke nothing, no worries. I updated the version, so it was on me. And it was more or less documented! Biggest "break" was the libname to name.


Sigstore v1.9 not compatible with previous v.1.8 integration

Description
Hi! In slsa-github-generator, we use Sigstore to generate and verify our signing tokens. Our workflows were working with Sigstore v1.8, however, when we upgraded to v1.9, we experienced issues with the generation and verification of the signing tokens. We were curious if this backwards incompatibility was intentional or not.
The first issue for us lies within the sign function. In this workflow, we see that in v1.9, our  sigstore.sign function fails, not identifying that it is in a GHA workflow.
However, the attest function also has the same issue. In this workflow run, we use v1.8 version of sign but use the v1.9 version of sigstore.attest, and have the same issue of the not identifying that it is in a GHA workflow on a later job.
Is this functionality the intended behavior? Thank you!
Version
Issues with Sigstore v1.9
Expected behavior with Sigstore v1.8

@enteraga6 there was a little refactoring of the way the identity token is retrieved in v1.9. Looking at your code, I think you just need to remove the signOptions altogether:
https://github.com/enteraga6/slsa-github-generator/blob/sigstore-bug/.github/actions/sign-attestations/src/main.ts#L20-L23
Those options were really only intended to be used in scenarios where you want to use the OAuth provider to retrieve an OIDC token.
You can see here that when those options are omitted, it will fallback to using the CI provider (which is the one which knows how to retrieve an OIDC token from GHA:
https://github.com/sigstore/sigstore-js/blob/sigstore%401.9.0/packages/client/src/config.ts#L130-L139
I think the change that may have broken this for you was that it used to always try the CI provider by default as the first strategy for retrieving a token and now it only falls back to that when some other strategy hasn't been configured.


Health checks from Aliases don't get added to Haproxy Loadbalancer

Test case: Create service with healthcheck, add it to lb, check Haproxy stats page and see that Haproxy configuration is indeed has helthchecks (green line on service).
Now create alias to that service and put it to lb instead of service itself. Check stats pages and see that there is no health checks on lb anymore.
Use case: Our CI creates new stacks, pass health checks and then change alias, and that alias is already set in lb. That way we can ensure that lb config is not disturbed. But using aliases disable health checks on lb side.

Useful
Info

Versions
Rancher v1.3.3 Cattle: v0.175.9 UI: v1.3.5

Access
localauth admin

Orchestration
Cattle

Route
stacks.index

Close it as there is a status/autoclosed label


[ Deploy ] Plurk

Service

Plurk (https://plurk.com) is a popular social network service in Taiwan, India and Japan.
https://en.wikipedia.org/wiki/Plurk
Link to your repository

https://github.com/irvin/Plurk-for-Franz
Additional Information

Plurk is already available in Franz

good to know, thanks!


Further usecases

This PR includes a multitude of use cases based on the defined layers and features.

One remark so that we do not get confused with terminology. In my review I wrote "Zones" but it would be better to use the word "Area", to keep the word zones for the "Zones layer". So the zones layers uses areas, like e.g. also the animal layer.
In case of doubt, we can also meet in the next days for discussions/clarifications.

Contributor

Got it! Will adjust these changes


[flake8-bugbear] Fix mutable-contextvar-default (B039) to resolve annotated function calls properly

Summary

Fix #14525
Test Plan

New test cases

Thanks!


angularfire2 configuration problems

In one tutorial:
 import {FIREBASE_PROVIDERS, DEFAULT_FIREBASE} from 'angularfire2' platformBrowserDynamic().bootstrapModule(AppModule, [routing,provide (DEF.....etc]);
in another:
 import {FIREBASE_PROVIDERS, defaultFirebase} from 'angularfire2' platformBrowserDynamic().bootstrapModule(AppModule, [routing,defaultFirebase('url')]);
and non of them works.
Any suggestions?

Are you using RC5 or RC4? There's a mismatch of our docs that uses RC5, but the default package uses RC4.
Please try using the pre-release version for RC5 using npm install angularfire2@next.

Should be fixed with the new version and documentation.


Doesn't work on High Sierra

Mac: 10.13 Beta (17A315i)
iPad 10.3.2

I try it worked

How can I use in version : 10.12.6
Can you help me with that.


Passing method to component changes this

I am passing the component an options object that contains a callback, which should be invoked from the component and emit an event. Currently, when the method is invoked, this.$dispatch('event_name') do not point to the original view, and hence, do not have the $dispatch method.
From the view:
<template>
    <div v-component="fm-bind" v-with="options: bind_device_options">
    </div>
</template>
<script>
//view itself is a component
    data: function() {
        return {
            bind_device_options : {
                bindModifiedCallback : function(devices) {
                    this.$dispatch('modify_step', 3); //No longer dispatches the event
                }
            }
        };
    },
</script>

On the component itself
<script>
//somewhere in the script
    methods: {
        onComplete: function() { this.options.bindModifiedCallback(); }
    }
</script>

I don't think you can define a method in the data property of a Vue component, as it seems you've done.

@nadirabid then how can I pass a method into the component?

btw, I'm currently going around the problem by creating a closure and capture this value and save it as another variable

Could do the following:

var PopupComponent = Vue.extend({
   methods: {
      open: function() {
         // method definition
      }
   }
});

Then instantiate the PopupComponent with additional methods:

var popup = new PopupComponent({
   methods: {
      close: function() {
         //method definition
      }
  }
});

Vue does not auto-bind functions inside data objects. But binding it yourself is fine.


.httpAuth not adding basic auth header to request

I've tried every way I can find and nothing results in basic auth being added to the request. Any help would be greatly appreciated. Below is the test and the log output:
import { Selector } from 'testcafe';
import { RequestLogger } from 'testcafe';

const logger = RequestLogger('http://localhost:8080/document/create.jspa?containerType=2020&containerID=1', {
	logRequestHeaders: true
});

fixture `Getting Started`
    .page `http://localhost:8080/document/create.jspa?containerType=2020&containerID=1`;

test
	.httpAuth({
        username: 'username',
        password: 'password'
    })
	.requestHooks(logger)
	('My first test', async t => {
    await t.expect(logger.contains(record => record.response.statusCode === 302)).ok();

     const logRecord = logger.requests[0];

     console.log(logRecord);
     console.log(logRecord.request.headers);
});

Running tests in:
 - Chrome 76.0.3809 / Mac OS X 10.14.5

 Getting Started
{ id: 'YOQTjdiaC',
  testRunId: 'A2jNLCeiv',
  userAgent: 'Chrome 76.0.3809 / Mac OS X 10.14.5',
  request:
   { url:
      'http://localhost:8080/document/create.jspa?containerType=2020&containerID=1',
     method: 'get',
     headers:
      { host: 'localhost:8080',
        connection: 'keep-alive',
        'upgrade-insecure-requests': '1',
        'user-agent':
         'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.100 Safari/537.36',
        accept:
         'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3',
        'accept-encoding': 'gzip, deflate',
        'accept-language': 'en-US,en;q=0.9' } },
  response: { statusCode: 302 } }
{ host: 'localhost:8080',
  connection: 'keep-alive',
  'upgrade-insecure-requests': '1',
  'user-agent':
   'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.100 Safari/537.36',
  accept:
   'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3',
  'accept-encoding': 'gzip, deflate',
  'accept-language': 'en-US,en;q=0.9' }
 ✓ My first test

 1 passed (4s)

@shannon-fluellen-aurea
Hello,
If I understand it correctly, you have an unsuccessful testcase with "Basic Auth".
I tried to reproduce the issue in the following example, but it passed successfully:
server.js
const http = require('http');

const server = http.createServer(function(req, res) {
    console.log(req);

    const auth = req.headers['authorization'];
    console.log("Authorization Header: ", auth);

    if(!auth) {
        res.statusCode = 401;
        res.setHeader('WWW-Authenticate', 'Basic realm="Secure Area"');

        res.end('!auth');
    }

    else if(auth) {
        const tmp = auth.split(' ');

        const buf = new Buffer(tmp[1], 'base64');
        const plain_auth = buf.toString();

        const creds = plain_auth.split(':');
        const username = creds[0];
        const password = creds[1];

        if((username == 'test') && (password == 'test')) {
            res.statusCode = 200;
            res.end('OK');
        }
        else {
            res.statusCode = 401;
            res.setHeader('WWW-Authenticate', 'Basic realm="Secure Area"');

            res.end('NOT OK');
        }
    }
});

server.listen(8080, function() { console.log("http://localhost:8080/"); });

test.js
fixture `Fixture`
    .page `http://localhost:8080/`;

test
    .httpAuth({
        username: 'test',
        password: 'test'
    })
    ('Auth test', async t => {
        await t
            .wait(1000000)
            .expect(true).ok();
    });

Could you please provide us with your "Basic Auth" sample project (or a public URL with required credentials)?

Hi @Farfurix thanks for taking a look at this for me! I'm working on a company project so I don't really have anything I can share as far as that goes. Can you please help me understand why my logger is showing no BasicAuth header in the request? I tested my application using Postman's BasicAuth functionality and it's able to use that to directly access login protected pages without redirection to the login page itself which has led me to believe that Testcafe is not including the basic auth header in my test. Do you have any reason to believe the .httpAuth feature is actually working in the test file I provided? Thanks again!

Hi @shannon-fluellen-aurea, you can debug your test by logging all resources of your page via RequestLogger in console.
However, we are not able to find the cause of the issue without researching a simple example showing the problem. Could you please correct the example @Farfurix provided above?

I thought I was running into this same issue @shannon-fluellen-aurea but I checked our server config we are using digest authentication not basic. I see this might have been a request at one time? https://github.com/DevExpress/testcafe/issues/2630
I'm wondering if there is any workaround for this.

Hi @shannon-fluellen-aurea, you can debug your test by logging all resources of your page via RequestLogger in console.
However, we are not able to find the cause of the issue without researching a simple example showing the problem. Could you please correct the example @Farfurix provided above?

Hi @LavrovArtem I'm not sure exactly what you're asking me to do, can you please explain more?

@shannon-fluellen-aurea
Hello,
We cannot examine this issue without any example. Could you please provide us with a sample project (or some public URL)?


How to send jmeter test results(*.jtl) to hygieia dashbard .

I have setup hygieia dashboard and installed hygieia jenkins plugin and would like to send jmeter load test results (*.jtl) to hygieia dashboard.can you please let me know the steps .Is this the right way to set it up in Jenkins ? Please let me know.

@satishc1 @tabladrum can you please advise how to proceed ? Are jmeter test results supported by jenkins hygieia plugin ?

@krishs123 Not yet. But wip.

ok thanks for update .

@krishs123 Is your jmeter running on the same server as jenkins? If not how do you get the results to jenkins ? What format is your jmeter output file ?

hi satish,
yes jmeter is running on the same server as jenkins and jmeter output format is xml .
thanks.

Do you store the xml in jenkins workspace?

yes ,  all the results are stored in jenkins workspace (same local machine).

Hi @krishs123 , Where you able to solve this problem. I am also looking at the similar use case where I want to have the performance data available in Hygieia. Currently I see there is only AppDynamics plugin available under performance .


gtf indexing

Add GTF to indexing.

Would you mind also adding typical value for GTF file argument into the usage documentation?


[documentation]: update Pytests README

Description
Updated README (and .gitignore) in pytests because I found it confusing.
Checklist

[ ] Tech writers review
[x] @AlexStroke review

Reviewed and polished README.md for clarity. The file still has issues that need to be resolved (see Issues below).
Changes

Restructured hierarchy of the subtopics.
Wording and formatting.
Additional external links (e.g., xfail, Allure).
Added a short note on Allure Report.
Added details to the "Iroha 2 Test Model".

Arguable: see Issues: 1 below.

Added table of contents for ease of use by users re-accessing README.md on multiple occasions.

Issues

"Iroha 2 Test Model" subtopic:

In my opinion, the original usage of the term "model" implies a deeper explanation of the provided .py files.
This is why I added a description for every file. If these should be removed or explained further, please let me know.
The original "Iroha 2 Test Model" subtopic contains a Configurations entry that only provides a short prompt and does not have a corresponding directory (unlike, e.g., Accounts -> /account), which can be adding to the confusion (at least, mine).

If this refers to the poetry.lock or pyproject.toml configuration files, this entry could be removed from this subtopic, considering we have a separate explanation of these files.
If this refers to the conftest.py file in the /test directory, I would like some more information on how this should be described as a part of this subtopic.

We have a mention of the pytest.ini file, however it does not exist.

Is this intentional? Is this file generated once we have the dependencies installed?

@AlexStroke, please review and provide feedback when possible.

Reviewed and polished README.md for clarity. The file still has issues that need to be resolved (see Issues below).
Changes

Restructured hierarchy of the subtopics.
Wording and formatting.
Additional external links (e.g., xfail, Allure).
Added a short note on Allure Report.
Added details to the "Iroha 2 Test Model".

Arguable: see Issues: 1 below.

Added table of contents for ease of use by users re-accessing README.md on multiple occasions.

Issues

"Iroha 2 Test Model" subtopic:

In my opinion, the original usage of the term "model" implies a deeper explanation of the provided .py files.
This is why I added a description for every file. If these should be removed or explained further, please let me know.
The original "Iroha 2 Test Model" subtopic contains a Configurations entry that only provides a short prompt and does not have a corresponding directory (unlike, e.g., Accounts -> /account), which can be adding to the confusion (at least, mine).

If this refers to the poetry.lock or pyproject.toml configuration files, this entry could be removed from this subtopic, considering we have a separate explanation of these files.
If this refers to the conftest.py file in the /test directory, I would like some more information on how this should be described as a part of this subtopic.
Currently, I left both versions of this subtopic in the file.

We have a mention of the pytest.ini file, however it does not exist.

Is this intentional? Is this file generated once we have the dependencies installed?

@AlexStroke, please review and provide feedback when possible.

Test Model is the solution for organizing our tests and test suites. Ok, it's not necessary to mention. Configurations means a test suite with tests that check different Iroha's configurations. I just have not developed this suite yet.
You're right. I don't use the pytest.ini solution anymore.

Updated the README.md:

Added Custom Test Environment with Docker Compose subtopic.
Added suggestion from #4176.
Minor tweaks.

I have an alternative version of the Using Test Suites instructions that implements the steps to set up a custom environment with Docker Compose in its body: Draft on HedgeDoc
If this one seems clearer to the reviewers, I suggest using the alternative.

LGTM


Conflicting sub-dependencies with other libraries

Hello. First of all, I´d like to thank you guys for the awesome library !
I am having an issue while trying to use it, I´ve managed to fix it by manually searching for conflicting dependencies inside pipenv, which leads me to believe this issue may be related to undeclared dependencies withing the build-system you guys are using on the package !
Here is the issue I face to run the app after installing pyxirr:
PS C:\codebase\my_path> & C:/Users/myuser/.virtualenvs/my_path-r724wWIu/Scripts/python.exe my_path/main.py
Traceback (most recent call last):
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\requests_toolbelt\_compat.py", line 48, in <module>
    from requests.packages.urllib3.contrib import appengine as gaecontrib
ImportError: cannot import name 'appengine' from 'requests.packages.urllib3.contrib' (C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\urllib3\contrib\__init__.py)
    from ..util.multipart_stream import MultipartStream
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\boxsdk\util\multipart_stream.py", line 3, in <module>
    from requests_toolbelt.multipart.encoder import MultipartEncoder
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\requests_toolbelt\__init__.py", line 12, in <module>
    from .adapters import SSLAdapter, SourceAddressAdapter
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\requests_toolbelt\adapters\__init__.py", line 12, in <module>
    from .ssl import SSLAdapter
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\requests_toolbelt\adapters\ssl.py", line 16, in <module>
    from .._compat import poolmanager
  File "C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\requests_toolbelt\_compat.py", line 50, in <module>
    from urllib3.contrib import appengine as gaecontrib
ImportError: cannot import name 'appengine' from 'urllib3.contrib' (C:\Users\myuser\.virtualenvs\my_path-r724wWIu\Lib\site-packages\urllib3\contrib\__init__.py)

steps to reproduce:

with pipenv, try to install pyxirr while having a conflicting package that uses an undeclared dependency. When running the main.py script you will start having these errors due to uncompatible libs (in my case, boxsdk 1.8-2.2)

Additional info: I´ve notticed also that even removing the package that caused the dependency does not solve the issue. I have to regenerate my virtualenv using pipenv to fix it, this is another indication to me that it is due to an undeclared dependency. I´m available to provide extra info if you guys need it ! :)

Hi @pmenegas. pyxirr has no external dependencies. You probably have issues with other libraries (or pipenv).

You should probably use --keep-outdated flag when using pipenv.

Hey @Anexen , thanks for the quick reply.
It is not a problem with other dependencies (or pipenv)
This happened with several other colleagues trying to use your library as well. It also happens across OSs.
I understand there are no external dependencies, but this may be due to the fact that the build system you guys use turns out to be imcompatible with other libraries. Believe me we have been tackling this problem for some time now and it is definitely coming from pyxirr.

I suggest you take a closer look at requests-toolbet. You may need to pin urllib3 to specific version.

I used pipenv a few years ago and gave it up because unacceptable dependency updates were a frequent problem. There are a lot of issues related to unexpected package updates (1, 2, 3)
I assume the following has happened:

You ran pipenv install pyxirr.
pipenv installed pyxirr and updated all dependencies to the latest allowed version (in Pipfile).
The latest version of urllib3 turned out to be incompatible with your requests / requests-toolbelt version.

You need to carefully examine Pipfile.lock and pin the necessary versions in the Pipfile.
if you are sure that the reason is pyxirr, please provide a reproducible example.


Customizable IAM policies

Issue
#200
What I did
Added custom value for specifying policy JSON files.
Without specifying policy files Zappa will use its default hard-coded policy.

Coverage decreased (-0.2%) to 89.356% when pulling c2400fa0b8b0dfac7fc734fbdbcd7cc9bd5b4cc0 on puhitaku:feature/custom_iam_policies into c264cfea2acd9cda1238a5bc12dd4881f9d6874e on Miserlou:dev.

Coverage increased (+4.9%) to 94.493% when pulling 6b50dfcb144fb64a0f43db2473fa5093b8fe4820 on puhitaku:feature/custom_iam_policies into c264cfea2acd9cda1238a5bc12dd4881f9d6874e on Miserlou:dev.

Added an event in test_settings.json for increasing coverage.

Coverage increased (+4.9%) to 94.493% when pulling c602e1d92c1de09c85a3e3a194991814bf6443d2 on puhitaku:feature/custom_iam_policies into c264cfea2acd9cda1238a5bc12dd4881f9d6874e on Miserlou:dev.


Support formatting more than 16 arguments with printf API when using variadic templates

Helper class TypeList holds a reference to array with packed type data (living on the stack)

:+1:

Great stuff, thanks for working on this! I'll need to take a closer look before merging though (hopefully later today or tomorrow).

Merged, thanks.


querystring.parse is legacy, upgraded to newer recommendations

NodeJS docs state "The querystring API is considered Legacy. While it is still maintained, new code should use the  API instead" https://nodejs.org/api/querystring.html
Upgraded this to the recommended solution

I have not been able to run `pnpm test`, because I get this error. Any idea on why might this be happening ?

@talentlessguy hey you need to update the dev deps, it breaks for newer node versions

@aldy505 ok I will later today

@aldy505 @vipulbhj I have bumped dev deps, plz pull from master

@aldy505 Yes, the test was failing as you had mentioned, I have fixed the object shape and it's working fine now.

@talentlessguy This can be merged, whenever you get sometime 😄


stable-diffusion.dll is much larger now (14MB vs. 140MB)

The June 1st release was the last one with the smaller size of 14MB, the June 28th release already has 140MB. I wonder, if it is neccesary, that the size increased that much and why, if so...

Anyway, not really an issue, if it needs to be that big.

I am assuming this is concerning cuda builds? yea upstream ggml got huge, since it contains specialized assembly for multiple platforms.
(also a duplicate of https://github.com/leejet/stable-diffusion.cpp/issues/311 :) )


Clicking on X to close the post window, the keyboard doesn't close smoothly

Hi,
Clicking on X to close the post window, the keyboard doesn't close smoothly. See the video.
https://github.com/PrimalHQ/primal-android-app/assets/1178856/6342996f-50a8-486d-b557-84016f341660

Closing the ticket. Please reopen if necessary.


i keep exiting out of the modes

i keep pressing my left grip, can you make it so the button is always a or something please

sure


[auto-bump] dependency by JCOR11599

:coffee: An Automated Dependency Version Bump PR :crown:
Initial Changes
The initial changes contained in this PR were produced by `go get`ing the dependency.
```
$ cd ./go
$ go get github.com/dolthub//go@
$ go mod tidy
```
Before Merging
This PR must have passing CI and a review before merging.
After Merging
An automatic PR will be opened against the LD repo that bumps the bounties version there.

This PR has been superseded by https://github.com/dolthub/bounties/pull/2191

This PR has been superseded by https://github.com/dolthub/bounties/pull/2192


Document acknowledgements option to s3 source

Description
Document acknowledgements option to s3 source. Added acknowledgements option to s3 source README file.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.
[X ] New functionality has been documented.

[ ] New functionality has javadoc added

[ X] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2530 (6b5ea07) into main (b9ee4b9) will increase coverage by 0.05%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2530      +/-   ##
============================================
+ Coverage     93.39%   93.44%   +0.05%
- Complexity     2120     2149      +29
============================================
  Files           252      254       +2
  Lines          5921     6012      +91
  Branches        479      486       +7
============================================
+ Hits           5530     5618      +88
- Misses          265      266       +1
- Partials        126      128       +2

see 9 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[SL] fixes weather and person untranslated states

Why does your PR touches so many languages and files and yet has no descriptions?

@andrejs2 do you have a Discord account? Are you in the Home Assistant community?
If yes, give me your user name, I want to help you.
I see you already tried twice to make some changes on this repo and included works from other people in your PR
First try was : https://github.com/home-assistant/intents/pull/1851
Git is super complex, I'm no expert but I can show you a workflow that works for a repo that is simple like this one.
PS: This PR will never be merged in this state.
Better to close it

@andrejs2 do you have a Discord account? Are you in the Home Assistant community?
If yes, give me your user name, I want to help you. I see you already tried twice to make some changes on this repo and included work from other people in your PR First try was : #1851
Git is super complex, I'm no expert but I can show you a workflow that works for a repo that is simple like this one.
PS: This PR will never be merged in this state. Better to close it

Thanks @jlpouffier for your message and help.
Yes, I'm aware of that, I closed  https://github.com/home-assistant/intents/pull/1851, this one will follow as well.
My discord: andrejs2, but do not worry about it, I would be super grateful if you showed me way out but don't want to bother you too much. As I mentioned in https://github.com/home-assistant/intents/pull/1851, I can wait for the matter to be sorted out and merged into the repository before opening new PR for sl.
Contributions for sl are my first serious work on Git :) and as I'm very inexperienced with Git, but there was no interest from more experienced native speakers from Slovenia who could translate to sl, so this task fell to me :).
Thank you, JLo, for your contributions to HA!


add new workflows and also add service templates etc. for metrics port.

Added service yaml and also service monitor
Added lint and test workflows for helm
Updated version and README

I guess it is normal that all checks are failed because they will be added with this PR.


taking size of array from user in c programming language

Issue Type: Bug
int x;
printf("Enter the size : ");
scanf("%d",&x);
// the below error is a bug in vs code it does not harm the functioning of the program
bd b[x];
on writing the above given code i found out that a red line appears on x inbd b[x]; which denotes an error but on running the program it does not disrupts the functoning of the program
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-10210U CPU @ 1.60GHz (8 x 2112)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.83GB (23.66GB free)

Process Argv
--crash-reporter-id 53e61994-1cd4-451a-a4ff-08743a6389ee

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
33.2.2

code-runner
for
0.11.6

c-snippets
Har
1.3.1

better-cpp-syntax
jef
1.15.10

cmake-tools
ms-
1.9.2

cpptools
ms-
1.8.0-insiders2

cpptools-extension-pack
ms-
1.1.0

material-icon-theme
PKi
4.11.0

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vscop804:30404766
vscop453:30404998
vsrem710:30416614
vsccppwtstudy:30421978

If you disable all extensions, do you still get the red line? If that causes the red line to disappear, please use the Help: Start Extension Bisect command to identify the extension that is flagging an error, and report this issue on that extension's issue tracker.

If you disable all extensions, do you still get the red line? If that causes the red line to disappear, please use the Help: Start Extension Bisect command to identify the extension that is flagging an error, and report this issue on that extension's issue tracker.

yes just found out an extension was causing this error , have reported the issue thanks


Fixed Code Execution on tensorflow/models

📊 Metadata *
Arbitrary code exec vulnerability
Bounty URL:  https://www.huntr.dev/bounties/1-other-models
⚙ Description *
Arbitrary Code Excecution in Tensorflow/Models.The TensorFlow Model Garden is a repository with a number of different implementations of state-of-the-art (SOTA) models and modeling solutions for TensorFlow users. We aim to demonstrate the best practices for modeling so that TensorFlow users can take full advantage of TensorFlow for their research and product development
💻 Technical Description *
This package was vulnerable to Arbitrary code execution due to a use of a known vulnerable function load() in pyyaml. Changing that to safe_load or using SafeLoader will fix the issue.
🐛 Proof of Concept (PoC) *
Install the package and run the below code:
// poc.js
import params_dict
exploit = params_dict.read_yaml_to_params_dict('exploit.yml')
print(exploit)

//exploit.yml
!!python/object/new:type
  args: ["z", !!python/tuple [], {"extend": !!python/name:exec }]
  listitems: "__import__('os').system('xcalc')"

🔥 Proof of Fix (PoF) *
After applying the fix, run the PoC again, calc wont pop and no code will be executed. Hence code exec is mitigated.
👍 User Acceptance Testing (UAT)
Only SafeLoader is used, no breaking changes introduced.

Congratulations Asjidkalam - your fix has been selected! 🎉
Thanks for being part of the community & helping secure the world's open source code.
If you have any questions, please respond in the comments section, or hit us up on Discord. Your bounty is on its way - keep hunting!
Come join us on Discord

Congratulations Asjidkalam - your fix has been selected! 🎉
Thanks for being part of the community & helping secure the world's open source code.
If you have any questions, please respond in the comments section, or hit us up on Discord. Your bounty is on its way - keep hunting!
Come join us on Discord


In the menu in the Russian localization there is not a translated item

VSCode Version: 1.20.0
OS Version: macOS High Sierra 10.13.3

Steps to Reproduce:

In the editor, click on the bottom left of the settings icon
All menu items will be localized, except one

Does this issue occur when all extensions are disabled?: Did not try

It takes us ~ 1 week to translate the strings into all language. So a x.0 release can ship with partly English strings and we try our best to have the first recovery build fully translated. Closing since we received full translations for Russian yesterday.


[RFC] [d3d9] Add option to disable the explicit frontbuffer

I'm not too happy about this, especially the wording. "Explicit frontbuffer" was the best term I was able to come up with. Maybe you can suggest a better name if the patch is acceptable in general.
Maybe there are better approaches to fix the issue, but this one seemed the easiest to implement so I gave it a try, and it fixes #1368 for me.
I reproduced the same ZUSI 3 bevhior on Windows by setting the game to "DirectX fullscreen" (aka exclusive fullscreen). So I'm almost convinced now that Zusi 3 does not redraw the full screen in every frame, only regions that have changed vs. the previous frame. So there is probably nothing other than a workaround similar to this one to fix it in dxvk. Exclusive fullscreen is marked as "special use only" in the game so I guess there are no plans to fix the game itself.

I would like to see if we can actually fix this issue without an app profile. I think it is related to the swap effect that the game is using -- currently we are entirely disregarding that and I know there are some quirks wrt. buffer count/front buffer with those (probably the same in D3D11 too @doitsujin)

As far as I can tell (I don't have a lot of apitrace experience though, so you may want to double check - trace is in the bug report), the game uses the DISCARD swap effect to be able to use MSAA. MSAA is only allowed when using DISCARD.
However, DISCARD requires the game to always draw complete frames: "An application that uses this swap effect should update an entire back buffer before invoking a Present operation that displays it.". So it's perfectly legal for dxvk to flip buffers, or even throw away old buffers, or fill them with random noise, when a game uses DISCARD.
ZUSI 3 apparently ignores that part and gets away with it because most Windows drivers implement "COPY" swapeffect semantics ("the [Present] operation leaves the content of the back buffer unchanged") when requesting DISCARD in windowed mode, because that is apparently the most efficient thing to do when working with the Windows compositor (?).
Initially I also assumed that the bug is caused by unimplemented "COPY" swapeffect in dxvk, but it looks like ZUSI 3 really requests DISCARD and then fails to follow DISCARD rules. This is also why the same bug occurs on Windows when setting ZUSI 3 to use exclusive fullscreen.

Merged as ba41a5219ae69753b4c7106bed99d310a5b28e01


Cannot download pdf format from renderer

When trying to download a Cairo plot from the jupyter notebook renderer (image/svg+xml from ms-toolsai/jupyter-renderers) I'm unable to download the pdf format (I click save as pdf but nothings happens). The SVG save file works fine.
When opening the Plots using "Expand image" button there is another save button which can save in SVG, PNG (both working fine) and also PDF which gives an error
19:29:06.896 [info] exporting plot...
19:29:14.669 [info] Attempting pdf write...
19:29:14.669 [error] [TypeError: Invalid host defined options
	at iL (~\.vscode\extensions\ms-toolsai.jupyter-2024.1.1-win32-x64\dist\extension.node.js:362:62189)
	at kI.exportPlot (~\.vscode\extensions\ms-toolsai.jupyter-2024.1.1-win32-x64\dist\extension.node.js:362:61723)]

This is the code I used to replicate the bug (it uses the igraph library with pycairo)
import igraph as ig
graph = ig.Graph.Lattice((3, 3))
ig.plot(graph, layout="grid")

Versions
vscode 1.86.1
jupyter v2024.1.1
jupyter notebook renderers v1.0.17
python-igraph 0.11.3
pycairo 1.25.1

Another issue arises when trying to rerun the cell. If I try to open again the "Expand image" I get the following error:
19:44:13.823 [error] No SVG or PNG Plot to open ~\##myfolder##\issue_igraph.ipynb, id: 2e3711a7-dbf9-4bce-be8a-fae6257d88f5

I first have to clear the output then I have to click again and it works. This issue is not present for the matplotlib inline rendering.

I'm very sorry for missing this issue, I can replicate this issue at my end as well,
Will look into this.


Clean install throws error: ImportError no module named chardet

Python version:  3.5.2
Bandcamp-dl version: latest from pip install bandcamp-downloader
Bancamp-dl options: -h
url:  none given
options:  none given
Describe the issue:
Using pyenv with Python 3.5.2 and a clean install with pip install bandcamp-downloader, it throws the following error (simply testing with the -h flag):
Traceback (most recent call last):
File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/bandcamp_downloader-0.0.7.post6-py3.5.egg/bandcamp_dl/utils/requests_patch.py", line 2, in
ImportError: No module named 'cchardet'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/bin/bandcamp-dl", line 9, in <module>
    load_entry_point('bandcamp-downloader==0.0.7.post6', 'console_scripts', 'bandcamp-dl')()
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/pkg_resources/__init__.py", line 542, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2569, in load_entry_point
    return ep.load()
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2229, in load
    return self.resolve()
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2235, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/bandcamp_downloader-0.0.7.post6-py3.5.egg/bandcamp_dl/bandcamp_dl.py", line 50, in <module>
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/bandcamp_downloader-0.0.7.post6-py3.5.egg/bandcamp_dl/bandcampdownloader.py", line 11, in <module>
  File "/Users/thibaultmaekelbergh/.pyenv/versions/3.5.2/lib/python3.5/site-packages/bandcamp_downloader-0.0.7.post6-py3.5.egg/bandcamp_dl/utils/requests_patch.py", line 4, in <module>
ImportError: No module named 'chardet'

Apparently I'm retarded, chardet isn't standard lib and I never added the requirements. Do: pip install chardet I'll fix this permanently in a bit so users don't have to manually install it.

Here's the output of requirements:

Hm, chardet is never installed like it should be, I have it listed just like all the other dependencies.
Interestingly enough this is happening for me as well, only reason I didn't notice is I always run from source and have chardet.
Looks like I need to dig around to find out what might cause this.

Alright fixed it, when I pushed the changes with the requests monkey-patch using chardet I didn't update the requirements.
There is a fix release on the releases page as well as on PyPI so a simple pip install --upgrade bandcamp-downloader should suffice.


Integrate audio query into UI

Description
Adds audio querying capability to the MultimodalQnA UI. Also changes the chat textbox into a multimodal textbox in preparation for image queries.
Issues
MMQnA Images & Audio RFC
Type of change
List the type of change like below. Please delete options that are not relevant.

[x] New feature (non-breaking change which adds new functionality)

Dependencies
N/A
Tests
Manual testing with the backend branch/PR (omar/speech-audio)

A comment about those print statements that got commented out - I saw some microservices using a LOGFLAG env var that's a boolean to turn on/off extra log messages (example). Would it make sense to do something similar here, or does it not work to turn them all on at once because of timeouts?

I updated the PR with this logflag. Does it look right?


Build failure (Android)

Unity editor version: v2019.1.0f2
Firebase Unity SDK version: firebase_unity_sdk_6.0.0
Firebase plugins in use (Auth, Database, etc.): Auth & Analytics
Additional SDKs you are using (Facebook, AdMob, etc.): Facebook, Gamesparks
Platform you are using the Unity editor on (Mac, Windows, or Linux): Mac & Windows
Platform you are targeting (iOS, Android, and/or desktop): iOS & Android
iOS builds just fine but Android build process fails:

Hi @pparreno, have you tried the same setup with our quick start test app
https://github.com/firebase/quickstart-unity/tree/master/analytics/testapp
https://github.com/firebase/quickstart-unity/tree/master/auth/testapp
to see if they have the same issue in case of 1) no additional sdk and 2) add additionals sdks separately, to narrow down the cause?

@pparreno since we haven't heard from you I'm closing this issue.  Let us know if you were still having issues.


stats.truncnorm variance works incorrectly when input is an array.

When trying to compute variance of stats.truncnorm it gives incorrect results when shape parameters a and b are arrays.
It works correctly when a and b are floats.
I am working with scipy-1.6.1 and Python 3.9.1.
Reproducing code example:
import numpy as np
from scipy import stats

a = np.arange(10, dtype=float)
b = a + 1

# This will print correct mean
print('mean:', stats.truncnorm.stats(a, b, moments='m'))

# This will print incorrect variance
print('var:', stats.truncnorm.stats(a, b, moments='v'))

# This will print correct variance
print('var:', [stats.truncnorm.stats(aa, bb, moments='v') for (aa, bb) in zip(a, b)])

Output of this program:
mean: array([0.45986223, 1.38316905, 2.31582133, 3.26045429, 4.21683078,
       5.18314709, 6.1572109 , 7.13706716, 8.12118899, 9.10845629])
var: array([-8.60292538, -8.60292538, -8.60292538, -8.60292538, -8.60292538,
       -8.60292538, -8.60292538, -8.60292538, -8.60292538, -8.60292538])
var: [array(0.07965182), array(0.07274289), array(0.06152078), array(0.04927779), array(0.03829029), array(0.02945243), array(0.02274838), array(0.01779288), array(0.01414854), array(0.01144881)]

Thanks. I think this is a duplicate of gh-13580, which will be fixed by gh-13581. Reviews welcome.

Thank you for your answer! Really looking forward for this to be merged.

@hocop You can help make that happen! Reviews from users are valuable, too. Please take a look at gh-13581. If you can check out  that branch and confirm that it fixes the issue for you, that would be great. If you can only look at the code and confirm whether it looks right, that would be valuable too. Please add your comments/approval there. Thanks!

@mdhaber thank you for your work! I checked it out and it does fix my issue. Left my comment in gh-13581 approving your pull request.

Thanks!


Add GSI for new transferToAnalysisBucketJobId field

As part of the work to make the error handling for decrypt during data transfer more robust, we need to add the batch job id to the database, and query by it. This new Global Secondary Index will allow us to do the query.
When this value is set, code in the ticf-integration stack will be able to find the relevant record using this index.

Realised I probably don't need this now, figured out another way to make this work


search_command.py: Failed to parse transport header

When I run the basic example of FilterCommand I am getting this error when enable
enableheader = true
or
passauth = true
in commands.conf for this command.
External search command 'filter' returned error code 1. Script output = "'splunkVersion:6.5.1\n' chunked 1.0,244,0 {"inspector":{"messages":[["ERROR","RuntimeError at \"/Applications/Splunk/etc/apps/App/bin/splunk-sdk/splunklib/searchcommands/search_command.py\", line 858 : Failed to parse transport header: splunkVersion:6.5.1\n"]]},"finished":true}"

Based on the error messages it looks like you're using search command protocol v2. This would be the case if you set chunked = true in commands.conf. Is that is the case neither enableheader or passauth are valid. Thankfully, they also aren't required for what you want splunkd to do: pass authentication-related facts.
Before I go on, please send your default and local commands.conf files and I'll confirm this. Thanks.

Thank you for a quick reply. I have the following in commands conf
[filter]
filename = exclude.py
streaming = true
retainsevents = true
overrides_timeorder = true
enableheader = true
passauth = true

So it looks like I was mistaken. You've not specified chunked = true. I'm assuming that you're setting passauth = true so that you can issue requests to splunkd. Is that true?
If it is true, try replacing passauth = true with requires_srinfo = true. You should then be able to use the service property on your command object to issue requests to splunkd.
One more question: Can you send a sanitized version of your app?
I will not be able to debug today, but can debug tomorrow, if need be, especially if requres_srinfo = true does not solve your problem. Having some form of your source code will make this easier.

Yes, my aim is to be able to query kv store from the command periodically. Command is meant to be run by triggered alert.
I changed conf to
[filter]
filename = filter.py
streaming = true
retainsevents = true
overrides_timeorder = true
requires_srinfo = true

and almost the same
``Failed to parse transport header: infoPath:/Applications/Splunk/var/run/splunk/dispatch/admin__admin__App__searchBarSearch_1481484455.226/externSearchResultsInfo.csv\n```
command code
from __future__ import absolute_import, division, print_function, unicode_literals
import os
import sys

sys.path.append(os.getcwd() + '/splunk-sdk')
from splunklib.searchcommands.streaming_command import StreamingCommand
from splunklib.searchcommands import dispatch, Configuration

@Configuration()
class CountMatchesCommand(StreamingCommand):
    def stream(self, records):
        for record in records:
            yield record

dispatch(CountMatchesCommand, sys.argv, sys.stdin, sys.stdout, __name__)

it selects v2 protocol as I see from the debug

I'll debug this tomorrow. Until then please:

send a tarball containing a sanitized version of your entire app
try setting chunked 'chunked = false'
if that doesn't work, try setting 'chunked = true' and removing all other settings but 'filename'

Closing due to inactivity


cleanup, sanitize and upgrade base images to dot net 4.8 for PaaS sup…

…port

Upgrade base images to 4.8 to add support back for Azure WAC.
Increased SA password complexity to allow SSMS into the SQL container (will need to retest later)
Disabled Mock Data on EshopModernizedMVC solution to force pulling from SQL DB (whether local container or external source)
Compiled a docker compose with no SQL container built-in and mock data enabled for rapid fire testings

Also, fixed the base docker image for WCF, missing mcr.microsoft prefix


[OM-91] Enable travis2 to test JIRA #comment new PR comment

[OM-91] Some text #comment Body comment

Boone and I were testing Jira integration with this, no longer needed.


php7.1 and const problems

Hey,
I just started to look for errors in phalcon to run it in php-7.1
and just found one cause of
--- test.zep ---
namespace Com\Test;

class Test
{
    const CON1 = 1;
    const CON2 = 2;
    const CON3 = 2;
    const CON4 = 2;
    const CON5 = 2;
}

compile ok
but when run php
--- test.php ---
$oClass = new ReflectionClass('\Com\Test\Test');
var_dump($oClass->getConstants());

i've got
array(5) {
  ["CON1"]=>
  NULL
  ["CON2"]=>
  bool(false)
  ["CON3"]=>
  bool(false)
  ["CON4"]=>
  bool(false)
  ["CON5"]=>
  bool(false)
}

zephir good interpreted
zephir_declare_class_constant_long(com_test_builder_ce, SL("CON1"), 1);
zephir_declare_class_constant_long(com_test_builder_ce, SL("CON2"), 2);

and how to crash and coredump
php script access to const like
var_dump(\Com\Test\Test::CON2);

or change type from int/long to string
then is SegmentationFault after run php or php -m

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/41512447-php7-1-and-const-problems?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

I created a patch for Zephir after which phalcon is compile and run for php7.1
please for feedback

Fixed in master branch.


color of collapse component goes out fix

closes #269
📑 Description
✅ Checks

[ ✅] I have updated the documentation as required
[✅ ] I have performed a self-review of my code

ℹ Additional context

@petar-cvit

@petar-cvit

this

@petar-cvit

Oh okay, I get it. Good job. Can you add index to the start of that forEach (line 383.) and use that to check if it's the first collapse panel?

Oh okay, I get it. Good job. Can you add index to the start of that forEach (line 383.) and use that to check if it's the first collapse panel?

sure ,made changes


Text highlight in 'form input with icon' offset

See GIF

the "lag" is the width of the padding/space set aside for the icon
Recording it, but don't think users have complained.

As far as I can tell the search box doesn't look like this anymore, therefore this issue should no longer exist.
Therefore I'll take the liberty of closing this issue (but do feel free to reopen it if I'm wrong).


make-json: generated file missing theme textdomain prefix

Bug Report
I'll keep it brief – might hopefully save someones time. ;)
Translations folder in a theme – just as expected by load_theme_textdomain():
languages/
    theme-slug.pot
    de_DE.po
    de_DE.mo

CLI Command
wp i18n make-json languages/*.po --no-purge

The result
languages/
    theme-slug.pot
    de_DE.po
    de_DE.mo
    languages/de_DE-a0623ef93657e56554c5ddda522ed690.json

The translation will not load.
wp_set_script_translations() expects the file to live in {textdomain}-{locale}-{md5}.json.
After renaming it accordingly the translation loads.

Related: https://github.com/wp-cli/i18n-command/issues/151#issuecomment-479493274

Just copying my comment from the above ticket, which still seems accurate:
wp i18n make-json does indeed not support a --domain because it is actually not used. That is indeed problematic.
The script just turns <source>.po into <source>-hash.json, which does not work as you mentioned.
I think the solution here should be:

Check --domain argument for text domain
If not passed, check PO file for X-Domain header.
Prefix JSON file with text domain in case it is missing.


single source of env script

Resolves: https://redhat-internal.slack.com/archives/C05M2GGKU7Q/p1707834629390729
/cc @petrpinkas

We can't rely on predefined values. The only valid method on how to get component URLs is to get them from component statuses
execute in your namespace
oc get fulcio -o jsonpath='{.items[0].status.url}'
oc get rekor -o jsonpath='{.items[0].status.url}'
oc get tuf -o jsonpath='{.items[0].status.url}'

@Gregory-Pereira @petrpinkas @bouskaJ how do we want to handle this PR

Pretty sure we dont need this anymore. Closing.


Check the extension of extracted file for backward matching

Fix #63
I have modified it to check the extension of extracted file for backward matching.

Thank you for very quick review and merging!!


Stack service "extends:" needs a way to ignore fields

For example:
services:
  foo:
    extends:
      file: docker-compose.yml
      service: foo
    env_file: .production

If the docker-compose specifies an env_file, it will be used instead of your .production, causing problems like dev env variables leaking to production.
Currently there's no way to unset / modify fields like env_file for example.
option a
Add reject to extends options so you can pick which fields to skip when extending.
option b
Flip the from.merge(service_config) in lib/kontena/cli/stacks/yaml/service_extender.rb around, making it possible for the stack file to overwrite anything coming from the external file. (actually it needs to be service_config.merge(from).merge(service_config['environment']).merge(service_config['secrets']) and not just flip around)
I think this is the expected outcome of the example in the beginning of this issue.

Not sure if option b is able to remove all keys...?
This is a definitely needed feature.


[FLINK-29633] Backport to 1.2

Backporting https://github.com/apache/flink-kubernetes-operator/pull/403 to the 1.2 release branch.

Ran failed test locally, worked fine 🤷 Could be flaky


ci: improve performance

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

@stebenz
errors:

[ ] error during start of container

missing:

[ ] (release.yml) push release to google registry (for cloud run): https://docs.docker.com/build/ci/github-actions/push-multi-registries/
[ ] switch between old and new pipeline (for the beginning run both pipelines or old/new manually) => i removed the old zitadel.yml, you need to add it again
[ ] (all.yml) add lint jobs to needs of release
[ ]

nice have

[ ] (release.yml) push sha to google if tests pass

untested

[ ] homebrew => should update https://github.com/zitadel/homebrew-tap/blob/main/Formula/zitadel.rb as soon as a new release is published


setup.py is modified.

PR for the issue #192
setup.py is slightly changed.
It seems to work.

Closed because it is behind a lot.


fix: python automated tests

Description
Fix bug introduced in commit 8d3c9b27047d721ea7e3ce7b262cfa9abe3087ea. With this PR, the automated tests of python will work.
Furthermore, I reduced the number of python versions to speed up the CI/CD, like in SyMPC. To improve this in the future, another github action could be created that is executed with more versions every day with a cron.
I used setup-miniconda
Close #20.
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests

@tholop The code is ready, I am requesting the review as you said

Thanks! Just one question: why only checking for Python 3.9? Is there any dependency that breaks on 3.7 or 3.8? (I don't quite remember at the moment)

I did not test them, but they should keep working. I have done that because I remember working in SyMPC on issue https://github.com/OpenMined/SyMPC/issues/244. They said that the CI/CD was slow, and that they wanted to reduce the number of tasks in a PR, and add a more exhaustive github action with all tests that was triggered daily in a cron job. So, I figured out that I could aswell reduce the tasks here, and as I commented at the beginning, maybe add the cron in the future. Of course, if you prefer that I recover the python 3.7, 3.8, I do not have any problem.

Oh yes, it makes sense, thanks!
I think that we should keep an exhaustive coverage, so maybe reactivate Python 3.7 and 3.8 for now (and we can remove them once we have a Cron job for slow versions).

Oh yes, it makes sense, thanks!
I think that we should keep an exhaustive coverage, so maybe reactivate Python 3.7 and 3.8 for now (and we can remove them once we have a Cron job for slow versions).

done!!


Some user-installed fonts not found at startup

Windows Terminal version
1.16.10262.0
Windows build number
10.0.22621.1265
Other Software
DejaVu font V2.37 in my case.
Steps to reproduce
DejaVu fonts are not bundled with Windows 11, so they have to be downloaded separately
Installation is done by right-clicking on the selected .ttf files then Install. No admin account is needed.
In Terminal settings, DejaVu Sans Mono should appear in the font selection listbox. It can be applied to any profile including the default one. The preview area should display the font correctly.
Restart Terminal. A warning message should appear.
Note : tried also with Inconsolata font (single .otf file), installed in the exact same way as DejaVu, but no issue with that one.
Expected Behavior
No warning, font displayed correctly in Terminal
Actual Behavior
Upon Terminal restart, a warning message appears :

Translation :
Sorry # C0 we can’t find the selected font "DejaVu Sans Mono". "Consolas" has been selected instead. Install the missing font or choose another one.
Workaround : by just opening the terminal settings, the font seems to be found again (no need to click on apply), then opening a new tab should display the font without warning.

Thanks for the report! This is a known issue with installing fonts into the user font directory. It's a bug that's unfortunately outside of our control and we're all patiently waiting for a fix for this as well. It's one of the most common bug reports we get. Here's how you fix it: https://github.com/microsoft/terminal/issues/14231#issuecomment-1280827973
/dup #3257


Dropdowns often broken since Angular 4 upgrade

I noticed in https://github.com/galaxyproject/cloudlaunch-ui/pull/12 but it keep happening outside that context. Switching clouds on an appliance page or choosing a different instance type will not always but often render the page useless without doing a refresh. I'm guessing this is in part due to the crapy dropdown component and possibly compounded by the Angular update.
As talked about earlier, we should migrate to using native angular components (e.g., https://material.angular.io/components/select/overview).

I'm getting this error locally when I try to launch Ubuntu:
ERROR TypeError: _co.cloud is undefined
Stack trace:
View_CloudCredentialsEditorComponent_3/<@ng:///MyProfileModule/CloudCredentialsEditorComponent.ngfactory.js:58:9
debugUpdateRenderer@http://localhost:4200/vendor.bundle.js:89836:12
checkAndUpdateView@http://localhost:4200/vendor.bundle.js:88983:5
callViewAction@http://localhost:4200/vendor.bundle.js:89343:21
execEmbeddedViewsAction@http://localhost:4200/vendor.bundle.js:89301:17
checkAndUpdateView@http://localhost:4200
...

This might be related to changes in the #12 pull request.  I'll investigate further.

I'm not sure if it is the same error but in Chrome the error on the console looks like this:
ERROR TypeError: Cannot read property 'name' of undefined
    at Object.eval [as updateRenderer] (CloudCredentialsEditorComponent.html:18)
    at Object.debugUpdateRenderer [as updateRenderer] (core.es5.js:13096)
    at checkAndUpdateView (core.es5.js:12243)
    at callViewAction (core.es5.js:12603)
    at execEmbeddedViewsAction (core.es5.js:12561)
    at checkAndUpdateView (core.es5.js:12239)
    at callViewAction (core.es5.js:12603)
    at execComponentViewsAction (core.es5.js:12535)
    at checkAndUpdateView (
...

This is related to the changes in pull request #12. I added the cloudType attribute to the cloud-credentials-editor component but I failed to update cloud-credentials-selector to set the cloudType on the cloud-credentials-editor.  I'm still running into an issue that when the cloudType changes the cloud-credentials-editor needs to recompute the availableClouds based on the new cloudType.

I pushed a fix for the cloudType problem to the gce branch, see #12 pull request

Now I'm looking into problem with UbuntuConfigComponent not implementing certain methods, for example getting this kind of error:
ERROR TypeError: get form must be implemented
Stack trace:
get@http://localhost:4200/common.chunk.js:18:19
../../../../../src/app/marketplace/plugins/base-plugin.component.ts/BasePluginComponent.prototype.ngOnInit@http://localhost:4200/common.chunk.js:57:13
checkAndUpdateDirectiveInline@http://localhost:4200/vendor.bundle.js:87576:9
checkAndUpdateNodeInline@http://localhost:4200/vendor.bundle.js:89072:17
checkAndUpdateNode@http://localhost:4200/vendor.bundle.js:89011:16


Add Pagination - Solution from @OwenRay

I made nothing - Credits on https://github.com/Bardavon-Health/actions-aws-ssm-params-to-env/pull/79

@dennybiasiolli Let me know if I can do something else :)


Enable checkin bots

We need to enable the bots to enforce the OWNERS file discipline in the istio.github.io repo in order to make sure the right people are involved in reviewing our docs.
Thanks.
cc @frankbu

istio.github.io is another good candidate to enable Tide.

Tide does not support required check now, it needs all checks to pass. I am working on it.


fix vertexdata serialize

The fix is not back compatible, I will push a new one in a sec. Thanks a lot for reporting it !!!

Here it is for reference #11517


Allow file to be dropped onto file input to select it for encryption/decryption

Allow file to be dropped onto file input to select it for crypto instead of just click to select.

Solved by https://github.com/HR/Crypter/commit/8b3638dee4df7df29e7b6c367710ecd53524b98f