User login

Rationale
We want to explore a user based login routine, similar to how the Heroku and Vercel toolchain/cli work. This enabled the following:

Stricter control for our customers. When a user leaves the organisation, the loose their access. The alternative is that organisations needs to cycle their API tokens.
User based login enabled users to switch between multiple Checkly accounts they might have.

Acceptance criteria

[ ] A user executes checkly login and is presented with a link to open in their browser.
[ ] On navigating to the link, the user can log in to Checkly.
[ ] After successful login, we show a screen that a user can close the browser.
[ ] The CLI confirms the successful authentication.
[ ] Any authentication errors are handled gracefully with a friendly error message (either in browser or in the CLI) that the authentication failed.

Implementation
We will most likely have to create a separate and distinct auth scheme for this feature. There are multiple options here.
Solutions like Stripe and Vercel automatically create a dedicated token for the CLI, which is listed in the account profile.
Stripe generates this after going thru the stripe login-flow on the command line

Vercel does something very similar. On vercel login a dedicated token is created.

We currently only have account-wide API tokens, we'll have to see what we need to do backend-wise to generate tokens on a user level for this (and future) use cases.

This will also require changes in the way we do authentication in the Public API. We could probably end with a mixed system that allows API KEYS or Token to perform auth.


Invalid index when including empty array as nested includes.

The logic here is slightly faulty, such that you will always get a Notice: Undefined offset: 0 error if passing an empty array as a nested include.
For example:
// Won't fail:
$book_includes = [
  'author',
  'library' => [
    'city',
  ],
];

// Will fail:
$book_includes = [
  'author' => [],
  'library' => [
    'city' => [],
  ],
];

Expected behavior should be that an empty array of nested includes sets the $name as $index and sets the $nested_includes to an empty array.

I'll leave it to @koenpunt or @jpfuentes2 to merge.


Responsive on desktop browser but not iphone

the site I'm just starting to build is responsive when I resize my browser, but not when viewed on mobile devices (iphone, ipad). not sure what's wrong. I'm compiling using codekit - it auto-compiles to "responsive.css" - you guys use "bootstrap-responsive.css" in your compiled version - is that specific name referenced somewhere else that would affect this?
site: http://dev.davidlynchfoundation.org.uk/art-page.html
Thanks!

There is a meta tag called viewport that is needed to make it work on your phone ...
I made the pull request to add it to the examples https://github.com/twitter/bootstrap/pull/1808
Here is the code, just put it in

yep - just add that meta tag, thanks!

Hi guys, the meta tag fixed one aspect of the issue, but I have another related issue that's cropped up - an image thumbnail resizes fine, but when reduced down to the mobile viewport it seems to expand to the original image size and you have to scroll to view the whole image. An example is when resizing my browser, the image size reduces at each media query step, but seems to grow bigger on the final one. page is here: http://dev.davidlynchfoundation.org.uk/art-page.php

@matalin I'm having the exact same issues as you -- did you find a resolution?

no fix as of yet..
On 27 February 2012 18:48, hbirch <
<EMAIL_ADDRESS>
wrote:

@matalin I'm having the exact same issues as you -- did you find a
resolution?

Reply to this email directly or view it on GitHub:
https://github.com/twitter/bootstrap/issues/1813#issuecomment-4200188

@matalin a temporary fix is to add "row" to the classes of the ul element.
i.e.

etc...

actually this looks better but is still problematic with multiple thumbnails (i just happened to have only one when i tested it) at small sizes it will make all thumbnails the width of their container, which at least makes it so they don't expand beyond the container, but they wont maintain their relative proportions.


Windows: Volumes PR rename UnmountWithSyscall

Signed-off-by: John Howard<EMAIL_ADDRESS>@tiborvass Fixing one of the nits you mentioned on IRC for #16433. Renaming UnmountWithSyscall to Unmount.

LGTM

LGTM


Issues med søkeboks

[ ] Klikk i høyre del av det som ser ut som et input-felt gir ingen respons (den faktiske boksen er usynlig og mindre?)
[ ] Trykk på forstørrelsesglass har ingen effekt, veksler mellom X og tilbake
[ ] Ved fokus kommer ikke søkersultater opp
[ ] Søkekriteriet blir hengende igjen i søkefeltet etter at man har valgt et resultat
[ ] URL som ikke eksisterer fører ikke lenger til at søkeresultboks vises eller redirect til semantisk url

The search function needs to be described
As for the menu button, also the button to expand the search function needs to be explained to users who cannot see it.
Suggested solution

[ ] We always recommend using a visible text “Search” together with the search icon. This makes the menu easier to spot for all users. If you won’t do that, you at least need to use aria-label=”Søk” to describe this button to screen readers.
[ ] Use aria-expanded=”true/false” to indicate whether or not the function is expanded or not.
[ ] Make sure that the search function appears AFTER the button to expand it in the code. Users will look for the expanded area beneath the button in the code.

Priority (1-4)
Critical (1)
WCAG relation
1.1.1 Non-text content (level A)
4.1.2 Name, role, value (level A)

Improve accessibility on the search function
The search function is possible to use both with a keyboard and with a screen reader, but there are some issues that make it difficult.

When using a keyboard, you need to tab out of the search field to get to the search suggestions. The normal interaction is to use arrow down to go down through the suggestions.
There are no descriptions to screen readers on the buttons to open the search function or the button to empty the search field. Use aria-label if you can’t use a visible text.
There is no search button. You must choose something from the suggestions. This is not very common, and users who are blind won’t understand this since there is no instruction to screen reader users, they can’t see that there is no button and they can’t see that suggestions are presented, you have to tell them how it works.
For some users who see this can also be a bit difficult to understand. One way to make it easier would be to move focus from the search field to the list with suggestions if the user presses “Enter” in the field. This will move the focus to the “results”. This is not a requirement in WCAG, but you need to at least make sure that blind users get enough information to complete the task to meet the requirements.
All icons used in the list have the same alt text: “logo”. This doesn’t describe what the icon is informing users who can see it. Note that the description of the icon should come after the text in the search suggestion, so instead of “Sted – Oslo” it would be better with “Oslo – Sted”.
The list is quite long, and we get the feeling that here are different categories that are grouped but without a heading. It could be easier to get an overview if you did something like this  [Tabell med otalt uakseptabel måte å fremstille datane på]

Suggested solution
See our suggestions in the list with issues above.
Priority (1-4)
High (2)
WCAG relation
1.1.1 Non-text content (level A)
2.1.1 Keyboard (level A)
3.3.2 Labels or Instructions (level A)
4.1.2 Name, Role, Value (level A)

Relatert til https://github.com/Artsdatabanken/nin-kart/issues/1548


Type aliases resolve to aggressively in the generated API documentation

What happened:
All the type aliases for ConfigMapsDefaultLocalObjectReference end up linking back to a single documentation block. We lose the unique documentation present at each alias declaration.
Fields that are an instance of the type alias are also collapsed into the underlying type, which is not what we want.

What you expected to happen:
Each type alias should get its own documentation block, ie. for each of these:
api/v1alpha1/gateway_types.go:type ListenerExtensionObjectReference = ConfigMapsDefaultLocalObjectReference
api/v1alpha1/gatewayclass_types.go:type GatewayClassParametersObjectReference = ConfigMapsDefaultLocalObjectReference
api/v1alpha1/httproute_types.go:type RouteMatchExtensionObjectReference = ConfigMapsDefaultLocalObjectReference
api/v1alpha1/httproute_types.go:type RouteFilterExtensionObjectReference = ConfigMapsDefaultLocalObjectReference
api/v1alpha1/httproute_types.go:type RouteActionExtensionObjectReference = ConfigMapsDefaultLocalObjectReference
api/v1alpha1/httproute_types.go:type RouteHostExtensionObjectReference = ConfigMapsDefaultLocalObjectReference

This looks to be a consequence of how type aliases behave in Golang. They don't show up as actual types.
Maybe it's better to just reuse the same type instead of alias for the time being?

xref https://github.com/ahmetb/gen-crd-api-reference-docs/issues/23

This looks to be a consequence of how type aliases behave in Golang. They don't show up as actual types.

The generator is using gengo, so I think that it can distinguish type aliases in the source.

Maybe it's better to just reuse the same type instead of alias for the time being?

One thing I was playing with is inlining the type rather than aliasing. This probably gets the same result in YAML, JAON and Go, but not in protobuf.
diff --git api/v1alpha1/httproute_types.go api/v1alpha1/httproute_types.go
index e290acc..ba22865 100644
--- api/v1alpha1/httproute_types.go
+++ api/v1alpha1/httproute_types.go
@@ -273,7 +273,9 @@ type TargetPort int32
 // route action within a known namespace.
 //
 // +k8s:deepcopy-gen=false
-type ForwardToTargetObjectReference = ServicesDefaultLocalObjectReference
+type ForwardToTargetObjectReference struct {
+       ServicesDefaultLocalObjectReference `json:",inline" protobuf:"bytes,1,opt,name=servicesDefaultLocalObjectReference"`
+}

 // RouteActionExtensionObjectReference identifies a route-action extension
 // object within a known namespace.

I filed https://github.com/kubernetes/gengo/issues/180 for gengo. I agree that we should probably look for a local workaround.

may be fixed by changing
type ForwardToTargetObjectReference = ServicesDefaultLocalObjectReference
to
type ForwardToTargetObjectReference ServicesDefaultLocalObjectReference

/assign @bowei

https://github.com/kubernetes-sigs/service-apis/pull/378
/close


Feature/plugins list

Looks good! I wonder if we should factor out the plugin handling code to a separate file. Plugin loading could then also move in there.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.inscitivops.com//job/cli-ruby/489/
Test FAILed.


Add support for TypeScript's Assertion Functions

One of Jest's weaknesses is that it doesn't use assertion functions based on the matchers.
See:
https://github.com/DefinitelyTyped/DefinitelyTyped/issues/41179

I investigated it in the past. Currently, assertion functions are quite fragile (ie they don't work as part of the objects etc), which makes them impossible to utilize with the current API.


Prometheus http_sd discovery and unit test

why 22222 int pkg/admin/util/monitor_utils.go?
ans: After communicating with the relevant person in charge, he asked me to write 22222 to death first, and the follow-up monitoring port will be put into the registration center.

Codecov Report

Merging #1070 (8b2c61a) into refactor-with-go (8e28727) will increase coverage by 0.31%.
The diff coverage is 88.00%.

@@                 Coverage Diff                  @@
##           refactor-with-go    #1070      +/-   ##
====================================================
+ Coverage             69.46%   69.78%   +0.31%
====================================================
  Files                    30       31       +1
  Lines                  2361     2386      +25
====================================================
+ Hits                   1640     1665      +25
  Misses                  604      604
  Partials                117      117

Impacted Files
Coverage Δ

pkg/admin/services/prometheus_service_impl.go
13.17% <85.00%> (+13.17%)
:arrow_up:

pkg/admin/util/monitor_utils.go
100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@chickenlj PTAL


Auto bump version for monorepo packages

When updating main package version, sometimes it is required to bump same in other package.json files in same monorepo such as examples, playground, fixture, other monorepo packages, etc. Example h3 playground

Looking forward for this feature 🚀


A small update

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

I think that it's more friendly now, thanks! could you solve DCO check and update also update the file in v2.15?


feat(healthcheck): provides API Route for status of vault and db

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

Codecov Report

Merging #469 (3fa58db) into main (1c36e03) will decrease coverage by 0.40%.
The diff coverage is 14.63%.

:exclamation: Current head 3fa58db differs from pull request most recent head 7374199. Consider uploading reports for the commit 7374199 to get more accurate results

@@            Coverage Diff             @@
##             main     #469      +/-   ##
==========================================
- Coverage   50.23%   49.83%   -0.41%
==========================================
  Files          51       51
  Lines        3581     3622      +41
  Branches      690      703      +13
==========================================
+ Hits         1799     1805       +6
- Misses       1612     1647      +35
  Partials      170      170

Impacted Files
Coverage Δ

src/utils/SecretManagerService.ts
16.66% <0.00%> (-1.12%)
:arrow_down:

src/utils/constants.ts
41.42% <5.88%> (-33.58%)
:arrow_down:

src/models/RCS.Config.ts
100.00% <100.00%> (ø)

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 1c36e03...7374199. Read the comment docs.


[MLIR] Bumping release tag to rocm-5.3.0

Regression tests have all passed for ROCm 5.3.0 tag. Pushing this as MLIR go to development complete.

@jerryyin Will this work with ROCm 5.1 that we currently use in our dockers?

@atamazov Yes, and we are bumping to ROCm 5.1 now as well.
FYI, all CI tests have passed: http://micimaster.amd.com/blue/organizations/jenkins/MLLibs%2FMIOpen/detail/mlir-bump-rocm5.3.0/15/pipeline


Add possibility to pass textController

After setState on widget my inputted text in TagsTextField became deleted. In my simple TextField I can manage it with TextController. Please Add possibility to pass textController to TagsTextField or how can I save the text inputted in TagsTextField before called setState() on Widget?

@Dn-a How can I set the text to Edit Text field from code?

Is this observation already considered @Dn-a?
I'm running into a similar issue while trying to allow the user to submit inputted text as a tag with space not just with an enter, which also can be managed by the textController.


Delayed resync raises NotImplementedError

Here's a sample mdstat output triggering the error:
Personalities : [raid0] [raid1] [raid6] [raid5] [raid4]
md1 : active raid1 sdb2[1] sda2[0]
      16760832 blocks super 1.2 [2/1] [U_]
        resync=DELAYED

md0 : active raid1 sdb1[1] sda1[0]
      4190208 blocks super 1.2 [2/1] [U_]
      [==========>..........]  recovery = 54.0% (2264768/4190208) finish=0.4min speed=68084K/sec

unused devices: <none>

This situation occurs when a single disk has two md raids and both are being resynced at the same time. The second one is delayed until the first finishes. I fixed it locally with the following patch:
diff --git a/mdstat/device.py b/mdstat/device.py
index 1a4dfbb..d894b5d 100644
--- a/mdstat/device.py
+++ b/mdstat/device.py
@@ -28,6 +28,8 @@ def parse_device(lines):
             bitmap = parse_device_bitmap(line)
         elif line.startswith("      ["):
             resync = parse_device_resync(line)
+        elif line.startswith("        resync=DELAYED"):
+            resync = None
         else:
             raise NotImplementedError("unknown device line: {}".format(line))

If I read https://github.com/torvalds/linux/blob/master/drivers/md/md.c properly the resync=DELAYED should be prefixed with a tab (\t) character, can you confirm that?  Could you email me your mdstat file as-is by any chance?

Can you pull and test my last commit? I removed the sync_request property on the device and put the "resync=PENDING" and "resync=DELAYED" on the "operation" property of "resync".

The new version seems to be working correctly. Thanks again for your quick fix :)


mitmdump not exiting after serving a request

Steps to reproduce the problem:
Hey I'm running mitmdump in a docker container on a VM. Over time, the memory overflows and disk swap starts, rendering the proxy unusable. This is easily fixed by a system reboot.
I start the execution with the following command:
sudo docker run -d --restart unless-stopped -p 80:8080 -v ~/HttpReorder.py:/tmp/HttpReorder.py mitmproxy/mitmproxy mitmdump -s /tmp/HttpReorder.py -R https://<my-url-here>:443 -p 8080
and the script file's contents are:
def reorder(headers, header_order=["Date", "Server", "Connection", "Pragma", "Accept-Ranges", "Expires", "Cache-Control", "Content-Disposition", "Content-Length", "Vary", "Content-Range", "Content-Type"]):
    lines = []
    for name in header_order:  # add existing headers in the specified order
        if name in headers:
            lines.append([name, headers[name]])
            del headers[name]
    #lines.extend(headers.fields)  # all other headers
    return lines

def response(flow):

    reordered = reorder(flow.response.headers)

    for pair in reordered:
        flow.response.headers[pair[0]] = pair[1]

    if "/download/" in flow.request.path and "Content-Type" not in flow.response.headers:
        flow.response.headers["Content-Type"] = "text/plain;charset=UTF-8"

def requestheaders(flow):
    flow.request.headers["NewHost"] = flow.request.headers["Host"]

def request(flow):
    if "NewHost" in flow.request.headers:
        requested = flow.request.headers["NewHost"]

        if "dev.data.something" in requested:
            flow.request.host = "dev.admin.something"
        elif "data.something" in requested:
            flow.request.host = "admin.something"

        del flow.request.headers["NewHost"]

What am I doing wrong? There seems to be a new python process in memory for each request. They never quit, only add up used memory.

Hi @turowicz,
Thanks for the detailed issue. Is this also happening without your script? If so, what's the output when you send SIGUSR1 or SIGUSR2 to mitmproxy after it's running for a while?

Hi @mhils
Unfortunately I'm unable to test this for you as the proxy is serving a production system and a cron task with docker restart every few hours seems to do the trick. We're aiming to remove the need for having a proxy so it's not our immediate plan to fix this problem.
That said I thought you'd find this interesting.


fix: translate3d style case

Found in using with cdkDrag directive

Codecov Report

Merging #90 into master will decrease coverage by 55.44%.
The diff coverage is 20%.

@@             Coverage Diff             @@
##           master      #90       +/-   ##
===========================================
- Coverage   95.97%   40.53%   -55.45%
===========================================
  Files           4        4
  Lines         298      301        +3
  Branches       29       30        +1
===========================================
- Hits          286      122      -164
- Misses          8      172      +164
- Partials        4        7        +3

Impacted Files
Coverage Δ

src/resizable.directive.ts
40.07% <20%> (-55.68%)
:arrow_down:

src/resize-handle.directive.ts
31.25% <0%> (-65.63%)
:arrow_down:

Continue to review full report at Codecov.

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


Add support for braces / position_before_control_structures

This option allows to force a line break before "else", "elseif", "finally" and "catch" to preserve verticality between all the control structures of conditions and exceptions handling.
I found this kind of prettyprint in a project and realized it was really helpful when you need to duplicate or comment a whole case (several lines including braces) without making the code broken:
if (/* first case */) {
    // whatever
}
// elseif (/* case you want to patch (commented so easy to compare) */) {
//     // what was done before your patch
// }
elseif (/* copy of the case that you're patching */) {
     // your patch experiments
}
else (/* ... */) {
    /// ...
}

This participated to make my workflow really smoother and promotes a rule trait which is rare in mainstream standards even if it's a plus for readability, I think: verticality.
It also makes it shorter to convert the "if {}" and "else {}" statements to "elseif {}" in both ways.
Hoping you'll consider this behavior and thank you very much for your work!

There is an option corresponding to it in the options of Scrutinizer:
Configuration   →   Coding-Style   →   PHP / Braces / Braces in ifs / Place else on a new line

Hi @jclaveau and thank you for creating this PR,
The scope of the project is to help out formatting code based on standards and promote such standards.
Your PR does not fit the scope. Friendly reminder, even if not past of the core of the project, you can always publish and use custom rules!

@SpacePossum this formatting is required by Drupal coding standards and Drupal has a sizable developer community. I hope you'll reconsider including this feature.
https://www.drupal.org/docs/develop/standards/coding-standards#controlstruct

It is just an option, do not see why it will be a bad thing to merge this feature.

I did reopened this after @jstoller informed me this is a rule within the Drupal community, I don't know all the rules and I did reopened afterwards the moment I saw it.
@elgandoz please let me know when you don't agree with a comment, as a down-vote thing doesn't trigger a notification in github so it might go unnoticed, secondly if someone else didn't tell me what the issue was I would have never know the issue based on your down voted only.
This PR is in the mixer with all other things about the braces fixer and not really on my plate, ping @julienfalque for consideration about the refactor of the rule.

I think this is covered by #4884 which introduces a new (private) ControlStructureContinuationFixer and #4885 which makes it public. This new fixer has a keyword_position option that allows to tweak the behavior as proposed in this PR. Those PRs still need work though, any help welcome :)
Since BracesFixer is on its way to deprecation, I would suggest closing this PR as it would make things more complicated.

@jclaveau Hello, I love this feature but seem they're not accepting this request, could you tell me how can I use it with registerCustomFixers?

How I made it work using a custom fixer:

create a src\App\Fixer\Basic\CustomBracesFixer.php class with the content from this file
register it under easy-coding-standard.yml

services:
  ...
  App\Fixer\Basic\CustomBracesFixer:
    allow_single_line_closure: true
    position_before_control_structures: 'previous'

disable the default BracesFixer in easy-coding-standard.yml

parameters:
   skip:
      # We have our own custom braces fixer, so don't use the default one!
      PhpCsFixer\Fixer\Basic\BracesFixer:
        - '*'

@costinel-draculet Seems you're modifying in the project directly, is it still ok to update? And I can't find any instruction about using .yml file.

@shhlkien I am using it in a Symfony 4.4 project.
Of course that any updates need to be done manually, as it is outside de library.

@costinel-draculet Hmm, I see... But I'm using .php_cs file for configuration. It's hard to understand when they provide a custom function without any instruction for use :confused:

Any update on this? Deal-breaker to using PHP CS Fixer imho

@jclaveau Could you rebase the pull request for avoid the conflicts? Thanks

Coverage increased (+0.002%) to 91.931% when pulling df6afd79883fb2d219576e6ea6e1e074e74b7f8d on jclaveau:2.13-position_before_control_structures into 6a18fd75962f12db61384991797ba7c99de19bf4 on FriendsOfPHP:master.

@shakaran Done! ;)

@SpacePossum @julienfalque As https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/4884 seems blocked, wouldn't it create more value for the users to simply merge this PR (which is only 1 commit long)?

Can this conflict fixed and this be merged?
@jclaveau thanks for your great work and your contribution. It's really sad to see this really useful addition was not merged three years ago.

Please merge :)   (?)

@jclaveau Can we please get another rebase for v3? Thanks.

For anyone still looking for a solution to get "uncuddled elses" (like Stroustrup):
You can apply 'control_structure_continuation_position' => ['position' => 'next_line'], which will put else, elseif, catch etc in new lines and not together with the closing braces.

Are other control structures like try / catch / finally supported?
Using control_structure_continuation_position next_line caused these also to go to a new line, as expected. So I believe the answer is yes.

Can I close this PR as it doesn't seem useful anymore?
I found your PR trying to do exactly what I could get out of control_structure_continuation_position in the end. I would say it's not required anymore and can be closed.

Are other control structures like try / catch / finally supported?
Using control_structure_continuation_position next_line caused these also to go to a new line, as expected. So I believe the answer is yes.

Can I close this PR as it doesn't seem useful anymore?
I found your PR trying to do exactly what I could get out of control_structure_continuation_position in the end. I would say it's not required anymore and can be closed.

Good to me, thanks!


Add support for SMS conversations

Closes #4.
This does make the deployment process a little more complicated though, and it also wouldn't be safe to publish the endpoint URL publicly (since someone who knew the incoming SMS endpoint could forge requests from Twilio and trick the Hotline into 'replying' to a third party).

ODI5NzM1OTkxODExMTEzMDQw.YG8d3A.qPb8omO-GBGD-FRffcSBO0LRTUw


Error installing bcbio

Hi,
I've been having trouble installing bcbio.  Could you kindly advise me on how I can overcome this?
My installation command is:
python bcbio_nextgen_install.py  --tooldir=  --genomes GRCh37 --aligners bwa --aligners bowtie2
The following is the error I got.
Resolving ftp.broadinstitute.org (ftp.broadinstitute.org)... <IP_ADDRESS>
Connecting to ftp.broadinstitute.org (ftp.broadinstitute.org)|<IP_ADDRESS>|:21... connected.
Logging in as gsapubftp-anonymous ...
Login incorrect.
gzip: stdin: unexpected end of file
Traceback (most recent call last):
File "/home/ubuntu/bcbio/stable/anaconda/bin/bcbio_nextgen.py", line 221, in
install.upgrade_bcbio(kwargs["args"])
File "/home/ubuntu/bcbio/stable/anaconda/lib/python3.6/site-packages/bcbio/install.py", line 106, in upgrade_bcbio
upgrade_bcbio_data(args, REMOTES)
File "/home/ubuntu/bcbio/stable/anaconda/lib/python3.6/site-packages/bcbio/install.py", line 348, in upgrade_bcbio_data
args.cores, ["ggd", "s3", "raw"])
File "/home/ubuntu/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 354, in install_data_local
_prep_genomes(env, genomes, genome_indexes, ready_approaches, data_filedir)
File "/home/ubuntu/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 480, in _prep_genomes
retrieve_fn(env, manager, gid, idx)
File "/home/ubuntu/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 856, in _install_with_ggd
ggd.install_recipe(os.getcwd(), env.system_install, recipe_file, gid)
File "/home/ubuntu/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 30, in install_recipe
recipe["recipe"]["full"]["recipe_type"], system_install)
File "/home/ubuntu/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 62, in _run_recipe
subprocess.check_output(["bash", run_file])
File "/home/ubuntu/bcbio/stable/anaconda/lib/python3.6/subprocess.py", line 356, in check_output
def repr(self):
File "/home/ubuntu/bcbio/stable/anaconda/lib/python3.6/subprocess.py", line 438, in run
4) Backslashes are interpreted literally, unless they
subprocess.CalledProcessError: Command '['bash', '/home/ubuntu/bcbio/stable/genomes/Hsapiens/GRCh37/txtmp/ggd-run.sh']' returned non-zero exit status 1.
Traceback (most recent call last):
File "bcbio_nextgen_install.py", line 288, in
main(parser.parse_args(), sys.argv[1:])
File "bcbio_nextgen_install.py", line 45, in main
subprocess.check_call([bcbio, "upgrade"] + _clean_args(sys_argv, args))
File "/usr/lib/python2.7/subprocess.py", line 190, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/ubuntu/bcbio/stable/anaconda/bin/bcbio_nextgen.py', 'upgrade', '--tooldir=/home/ubuntu/tools', '--genomes', 'GRCh37', '--aligners', 'bwa', '--aligners', 'bowtie2', '--data']' returned non-zero exit status 1

Sorry about these issues. This is an intermittent problem with the Broad FTP servers. They can report this when they get oversubscribed or are otherwise unhappy. Unfortunately just re-trying is the best approach. If you re-run the upgrade command:
/home/ubuntu/bcbio/stable/anaconda/bin/bcbio_nextgen.py upgrade --tooldir=/home/ubuntu/tools --genomes GRCh37 --aligners bwa --aligners bowtie2 --data

it should pick up where it left off and hopefully the FTP server at Broad will be happier. Hope this helps get your install finished.

Thanks everyone, closing this issue as this is almost assuredly the problem. Please reopen if you continue to have problems. Thanks!


Object reference not set to an instance of an object.

I tried to use it in my project but as soon it starts throws an Object reference not set to an instance of an object. Specifically in PerRequestLifetimeExtension.cs line 27

When I call:
container.RegisterInstance(new NamuDbContext(optionsBuilder.Options), new PerRequestLifetimeManager());

Are you using it in a WCF project? Because the OperationContext.Current is only available in WCF. If you are using in web API or a web project then, it would be HttpContext.Current. If you are interested in submitting a patch to support both then, do the fix and send me a pull request.


Scan all forc.toml's in this repo to check if we have any implicit std dependency on git version of sway-lib-std

As a special case (as this repo is the home of sway-lib-std), any forc project hosted in this repo should not depend on the sway lib std implicitly. If they do in release PRs such as #5452 we will see a failure in CI. Because forc tries to fetch std versioned with its own version. For example if we are bumping to 0.49.0, forc will search for std by fetching this repo and going to the 0.49.0 tag. The problem is at the time of that PR is open, 0.49.0 won't be available as it will only be available once it is merged and a new tag is created.
So as a solution projects that do not need std should have implicit-std = false in their forc.toml and projects that need them should have it explicitly added as a path dependency:
...
[dependencies]
std = { path = "../../../sway-lib-std" }

IIRC @ironcev did an amazing job to ensure this in test suite, we can create a script out of it to run it in CI to ensure other test projects are not depending on it either.

Yes there is a single module in the test suite harness checking exactly that (and few other points). I assume it shouldn't be to difficult to extract the logic and use it elsewhere.


Hotkeys no longer depend on abapGit installation #2629

When this commit is applied hotkeys also work with
the single abapGit report. Limitation is still that users
only can override hotkeys with the installed abapGit repo.
#2629

:+1:
P.S. I’ll be able to check it just on Sunday from my side. Sorry. Fill free to merge if anyone else confirm

Now it's also possible to override the hotkeys within the single file abapGit report.

how so? what is the new approach?

Inspect local classes which implement interface ZIF_ABAPGIT_GUI_PAGE_HOTKEY with FM SCAN_ABAP_OBJECTS_CLASSES.
https://github.com/christianguenter2/abapGit/blob/4bc1c234beb5953e0e3259d923fa3d7fa2404140/src/ui/zcl_abapgit_hotkeys.clas.abap#L64-L69
https://github.com/christianguenter2/abapGit/blob/4bc1c234beb5953e0e3259d923fa3d7fa2404140/src/ui/zcl_abapgit_hotkeys.clas.abap#L155-L187

consider the following scenario:
1: I have the development version installed which contains syntax errors
2: run the merged report, will it try to instantiate the pages from the development version?
anyhow, something like
https://github.com/larshp/abapGit/blob/ff6725e7596bedd1f5b3fb3610df266246e576f0/src/zcl_abapgit_serialize.clas.abap#L163 should solve this issue, if it is an issue

Yeah, you're probably right. Should we make is_merged reusable? What's the right class for the method?

suggest adding it as new method in class zcl_abapgit_environment


Fixing small bug when generating URI for single resource and adding URI property to other resources.

Update
This PR fixes a bug with how the uri property on some resources was being output. There was a wrong reference to the id and thus the uri value was not being generated correctly. This PR also adds the uri property to the rest of the Campaigns, Kudos, Reportbacks, ReportbackItems resources so all have data regarding how to access them via their respective individual URIs.
The PR also does some extra cleanup and adds some indentation formatting to the ReportbackController.php file which somehow was lost before.
@aaronschachter @DFurnes

:+1:

hehe, I'm working on one more update to this PR and then will request another review @aaronschachter :) Sorry, forgot to mention it was still a work in progress! But thanks for the initial check :dancer: Have the code review is completed now :)

:+1:


Update deps

Learning to do things is good!

This has been updated

Question: when doing return within render(), why do we add whitespace after render, but not return?

@phated Maybe, it's better to add a .jscsrc for this repo

@coderhaoxin I agree but that wouldn't solve the issue of adding unnecessary whitespace to the diff


Automatic swiping not possible when the page containing react-swipy is not the current Active Tab

My implementation of react-swipy requires that the song will swipe on its own after a set interval of time.
This works great when the tab is active in the browser, however when the tab is inactive the Swipe will fail.
I have replicated this behavior in the following codepen fork:
https://codesandbox.io/s/react-swipe-cards-wplqd
You will notice that the Cards will swipe every 1s while the tab is active, however will pause swiping when the tab is inactive - and resume once the tab is active again.
Any thoughts on how to fix this within the source?

This is related to how browsers handle javascript code execution in inactive tabs, probably you can use requestAnimationFrame, you can see more on this issue https://stackoverflow.com/questions/5927284/how-can-i-make-setinterval-also-work-when-a-tab-is-inactive-in-chrome


RFC: use compatible name for prefix for device-specific API

A number of devices including ams,ccs811 and maxim,ds3231 have capabilities that are not supported by generic subsystem APIs like sensor or counter.  Access to these capabilities is provided by device-specific extensions like:

int ccs811_envdata_update(struct device *dev,
                          const struct sensor_value *temperature,
                          const struct sensor_value *humidity);
int maxim_ds3231_synchronize(struct device *dev,
￼			     struct sys_notify *notify);

Zephyr does not have a global documented policy on naming that takes into account future conflicts, e.g. by placing things in namespaces.  This has recently caused several issues between external libraries and internal and public Zephyr code, and as Zephyr's in-tree capabilities expand the potential for conflict will only increase.
I propose that public API related to device-specific capabilities should use the device compatible for a prefix, e.g. ams_ccs811_ and maxim_ds3231_.  This has several benefits:

The longer name reduces the chance of conflict if two vendors produce devices with similar names;
The vendor prefix will group the custom API in documentation indexes.

I don't propose that API for existing drivers like CCS811 be renamed (though I would support that), but only that new drivers like DS3231 (#17631) follow this practice.

I couldn't agree more @pabigot . Adding this to today's API meeting.

API meeting:
General consensus towards enforcing (and documenting) this. #24741 will include a patch to document this.


essay: How ckb pool prioritizes transactions

I'll submit a new post about fee rate priority and fee bumping later.


when will you release the training code?

thank you.

Thank you for your interest in our project! The training code is now available. If you find our work helpful, please star our repository and cite us in your work.


Parent comment button

Is your feature request related to a problem? Please describe.
When you find a user's comment you want to know the context of, you have a button that takes you to the comment in the thread. Unfortunately this doesn't show the parent comment, when there is one.
Describe the solution you'd like
I've seen in other interfaces, when you follow the link of the comment to the thread, a button which expands either the parent comment, or the whole tree the comment is a part of. Either or both buttons would be a very nifty feature!

If you mean going from an user's comments page to the original thread, there's actually an invisible "context" button between the "save" (floppy disk) and "go to post" (fenced square) buttons. Just tap between those and it'll send you to the comment inside the thread with a context of 3 parent comments.

I didn't know about this hidden button, but even then, when I tap that, it shows that comment as the parent of the displayed thread, and never the previous comments, unless you tap "view the rest of the comments" which just shows the whole comments section of the post.
I do see a new (recent?) button which seems to do exactly what I was looking for, to the right of the floppy disk :)


remove scrcpy+ menifest with older publisher name

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

This PR aims to remove older publisher named manifest as publisher name got changed

Needed to remove older published manifests with older publisher name.

The pull request contains more than one manifest. Either singleton or multi file manifest under one version directory.

The pull request contains more than one manifest

Does targeting multiple versions of the same package mean 2 different manifests? It's same publisher in the end tho.

Either singleton or multi file manifest under one version directory.

I am not sure i understood what to do.

That means 2 different versions manifests.

That means 2 different versions manifests.

so what should I do if I want to remove both versions?

Split into two pr

Oh got it.


Patch 1

I wanted the limit specifier for strings (e.g. "%16.s") to be usable in situations when zero termination isn't guaranteed. Otherwise it may end up crashing. As a simple fix I added lenght limitation to _strlen.

Codecov Report

Merging #36 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   99.34%   99.34%   +<.01%
==========================================
  Files           1        1
  Lines         306      307       +1
==========================================
+ Hits          304      305       +1
  Misses          2        2

Impacted Files
Coverage Δ

printf.c
99.34% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@cz7asm, thanks a lot! This is really a very useful patch! Appreciate that.
Your suggestion is basically the c++11 strnlen_s() (secure version of strlen) function. Indeed, this prevents a possible crash.
I merge your patch and add the according test cases shortly.


European Parliament

All speeches of the European Parliament are archived, translated to English. Potentially, this will be added as a corpus in 2024.

There is a scraped version, converted to linked data, available from https://ssh.datastations.nl/dataset.xhtml?persistentId=doi:10.17026/dans-x62-ew3m. The suggestion was to start with this dataset, and then use the English version. The corpus information should include a warning that the speeches can be in a different original language.
Metadata is limited: the vocabulary is rich, but all speeches seem to have lpv:unclassifiedMetadata as a predicate.
Data that can be extracted:

date
order in debate
not systematic: original language  -> some objects of lpv:translatedText start with a language isocode in brackets, e.g., "(IT)"
not systematic: speaker -> object of lpv:unclassifiedMetadata predicates - though this contains data such as President, and may either have full names or just the surname
not systematic: speaker party -> object of lpv:unclassifiedMetadata contains party indication in brackets (e.g., "(PSE)"

@jgonggrijp this was the corpus I told you about. As we need to read turtle to process this corpus, I'm currently thinking to integrate rdflib in I-Analyzer and make RDFExtractors based on it. Or @lukavdplas is this something to rather start in the new repo?

If you want to merge things within the next month or so, then you should do it in I-analyzer. Otherwise, it will save time to build it in the ianalyzer-reader package from the start.


Hi

Hi friend, could you rate my repositories and I'll rate yours at the same time? Thanks!

Hi Evgeniy, I like your portfolio site. It has a lot of fun design elements that are coded instead of just animation. Do you use a JavaScript compiler or is that hand written?


Angular Language Service extension abandoned and buggy?

VSCode Version: Code - Insiders 1.19.0-insider (d8989c95964ce06c64ffebc2fb400a0327bd447d, 2017-12-08T10:45:40.870Z)
OS Version: Windows_NT x64 10.0.16299
Extensions:

Extension
Author (truncated)
Version

ng-template
Ang
0.1.7

path-intellisense
chr
1.4.2

vscode-svgviewer
css
1.4.3

githistory
don
0.2.3

tslint
eg2
1.0.24

auto-rename-tag
for
0.0.15

Angular2
joh
2.11.2

json-to-ts
Mar
1.5.4

markdown-shortcuts
mdi
0.8.1

code-beautifier
mic
1.7.2

prettify-json
moh
0.0.3

csharp
ms-
1.13.1

material-icon-theme
PKi
3.1.0

view-in-browser
qin
0.0.5

(2 theme extensions excluded)
Any chance that the Angular Language Service can either get rolled into the core VS Code, or the extension get some attention? There's been no updates for a while, and it has major bugs (intellisense issues, crashing, etc.)

@jtsom We'll reach out to the angular team. We won't bring Angular support into the core though.


Add transaction fee estimation and show fee in view.

This isn't a strict port of the fee estimation from btcwallet as #10 would indicate.  It's actually better, since the actual fee/kB of the estimated transaction size is used to determine whether enough fee was added, instead of incrementing the fee by the relay fee for every full kB boundary.

There is also a small display issue in the create transaction view that is introduced by this change.  If a transaction can not be constructed (either because one of the outputs is invalid or there is not enough previous output value to spend), the Transaction Fee and Remaining Balance labels will show just "BTC" with no amount, instead of hiding those values completely.

Fee estimation is also slightly off when comparing the actual fee/kB (I just sent a transaction with an actual fee/kB of 991 satoshis, when it should have been >=1000).  This is because the transaction overhead constants are best case scenarios.  These have to be changed to be worst case, since this fee estimation algorithm does not do the iterative fee incrementing like btcwallet does.

Fixed the fee calculation to use a better (worst case) serialize size estimate.  Transactions are now created with actual fees/kB which are just over the relay fee.

EstimateSerializeSize is wrong.  The output count needs to be increment if a change output is being added.

I'm currently moving all of the transaction construction code from the AccountViewModel (in RecalculatePendingTransaction) into a static, pure function in the Bitcoin project.  Because that construction code requires a way to fetch inputs for some target amount, I am passing in that functionality using a delegate rather than depending on the RPC client directly.  The advantages of this design are testability (testing this code in the view model would be tricky, due to other issues with the view models depending on classes like MessageBox), flexibility (any input fetching code can be substituted here, not just rpc results), and modularity (the code could be imported by a command line tool, for example, and be able to construct transactions with the same smart fee estimation).

In retrospect, I'm going to modify the RPC API to allow incremental utxo selection all under a single database transaction, using gRPC streams.  This closely models the new txauthor.InputSource btcwallet type, and it should fit in very well with this Paymetheus code, since this Paymetheus PR was the inspiration for that package.

Closing, overwrote this branch with the decred fee estimation work by accident.


Skype having a problem with screen sharing

I've had one problem several times when screen sharing: someone could not see your screen or vice verse.
The bug happens on Skype application both on linux and mac.
When I choose to share my screen sometimes other people in the room can't see the screen shared, although some folks also in the same room can. We've found a solution that either the user who is sharing or the users who can't see quit the application and come back in.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


gather_nd InvalidArgumentError

2017-07-24 10:35:25.247357: W tensorflow/core/framework/op_kernel.cc:1152] Invalid argument: flat indices[12287, :] = [127, 42] does not index into param (shape: [32,512,100,1]).
[[Node: conv-maxpool-3/GatherNd = GatherNd[Tindices=DT_INT32, Tparams=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"](ExpandDims, conv-maxpool-3/stack)]]
2017-07-24 10:35:25.247422: W tensorflow/core/framework/op_kernel.cc:1152] Invalid argument: flat indices[12287, :] = [127, 42] does not index into param (shape: [32,512,100,1]).
[[Node: conv-maxpool-3/GatherNd = GatherNd[Tindices=DT_INT32, Tparams=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"](ExpandDims, conv-maxpool-3/stack)]]
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1039, in _do_call
return fn(*args)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1021, in _run_fn
status, run_metadata)
File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/contextlib.py", line 89, in exit
next(self.gen)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status
pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.InvalidArgumentError: flat indices[12287, :] = [127, 42] does not index into param (shape: [32,512,100,1]).
[[Node: conv-maxpool-3/GatherNd = GatherNd[Tindices=DT_INT32, Tparams=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"](ExpandDims, conv-maxpool-3/stack)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "mix.py", line 66, in
autoen_nn.clf_train_epoch(x_train,y_train,batch_size,dropout_rate = 0.5)
File "/Users/MilesZhao/Desktop/ORNL/data/epath_data/yong/cnn.py", line 248, in clf_train_epoch
feed_dict = feed_dict)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 778, in run
run_metadata_ptr)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 982, in _run
feed_dict_string, options, run_metadata)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1032, in _do_run
target_list, options, run_metadata)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1052, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: flat indices[12287, :] = [127, 42] does not index into param (shape: [32,512,100,1]).
[[Node: conv-maxpool-3/GatherNd = GatherNd[Tindices=DT_INT32, Tparams=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"](ExpandDims, conv-maxpool-3/stack)]]
Caused by op 'conv-maxpool-3/GatherNd', defined at:
File "mix.py", line 64, in
num_classes)
File "/Users/MilesZhao/Desktop/ORNL/data/epath_data/yong/cnn.py", line 125, in init
filter_size)
File "/Users/MilesZhao/Desktop/ORNL/data/epath_data/yong/cnn.py", line 217, in conv_layer
local_in_doc_mat = tf.gather_nd(in_doc_mat,temp)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/ops/gen_array_ops.py", line 1321, in gather_nd
name=name)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 2336, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1228, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): flat indices[12287, :] = [127, 42] does not index into param (shape: [32,512,100,1]).
[[Node: conv-maxpool-3/GatherNd = GatherNd[Tindices=DT_INT32, Tparams=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"](ExpandDims, conv-maxpool-3/stack)]]

I apologize but I am having a hard time understanding what the problem is, where the problem is, and what version it affects. Please resubmit and pay attention to the issue template (https://github.com/tensorflow/tensorflow/issues/new) . Please provide all the information it asks, in particular detailed instructions to reproduce the problem. Thank you.


Parameterises the url for nexus

Some projects need https://s01.oss.sonatype.org/service/local/ instead of https://oss.sonatype.org/service/local/.
This allows to override the url if necessary.

Arrow account was moved to new infra?

No, but I am also re-using this on a non-Arrow project and we might also want to re-use this from non-Arrow org projects.
This config is quite re-useable for many people I think, no?

Right, I would rename the properties to indicate they are nexus repositories, i.e: nexus.repository.snapshot

Updated with your suggestion @JavierSegoviaCordoba. Thanks for the feedback!


[v2.9] AKS: Nodepool values validation

Setup
Rancher version: v2.9-6d87a11ea46b7571646d7c3d7af704584c39fd62-head
Browser type & version: Chrome latest
Describe the bug
AKS (Vue UI):
Nodepool values on UI like Count, Max surge need validation for nil, 0 value, as in old UI
To Reproduce
Provision AKS cluster from Rancher UI with Nodepool count 0 and max surge empty
Screenshots

I am moving to done as the Hosted clusters automation is still on the backlog and is not clear if some other team owns it cc @yonasberhe23
Unit test is enough in this case.


modals border-radius 0

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Needs Design check

@exezbcz please review

Context

[x] Closes #6604
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at f25665c
Refactored NeoModal component to enhance UI and UX. Removed border-radius from modal header in NeoModal.scss.

🤖 Generated by Copilot at f25665c

NeoModal refactored
Responsive and accessible
Border-radius zero

Cc wdyt @daiagi ?


Export flow map does not produce output

Tried all three variations of "export flow map".  Didn't seem to produce a .png or .jpg in the specified folder.

 Also, did you place
at least 3 flow points? Both steps are necessary for export to succeed. A
screenshot could help to see if it looks ok. Thanks in advance.
On 18 August 2016 at 19:20, dhomburg<EMAIL_ADDRESS>wrote:

Tried all three variations of "export flow map". Didn't seem to produce a
.png or .jpg in the specified folder.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sadaszewski/flowed/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUI3cfvj6oj7gQMsiQuhUReqS5bX-YBks5qhJRVgaJpZM4JnuA8
.

Ah, I didn't have a background image loaded.  Works fine now that I do.  Thanks for the quick response!


Reference the dataset that's actually used

Fixes #446 (it's a good idea to open an issue first for discussion)

[ ] Tests and linter pass N/A
[ ] Code coverage does not decrease (if any source code was changed) N/A
[x] Appropriate docs were updated (if necessary)

I signed the CLA!

@wboykinm nice catch, thanks for the PR!


[Feature]  Adding Deployment link

Description
i will add a deployment link in readme.md file at title of project
Motivation
it will be easier for others to view preview of project
Screenshots
No response
Additional Context
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct.
[X] The changes don't break the code.
[X] I will attach screenshots and videos of the development work once it is done.

Priority
High

@GarimaSingh0109 assign me with is issue


Player drifting to the right

When staying still, the player is constantly drifting to the right and also a bit backwards, with a speed of around 0,1 m/s. Also when pressing the move up and move down keys, the player moves quite a bit right.
I'm experiencing this using a Samsung HMD Odyssey Plus and my new key bindings for Mixed Reality controllers from PR #28. Is this happening only for me or with other configurations too?

I never had that problem on the Quest 2 myself.
You don't think it is controller drift maybe?
I wanted to implement a deadzone for the analog inputs into the mod but I think you can do that aswell in the steam vr input.

It could be controller drift in the left joystick used for movement. It's good to know that it doesn't happen for everyone.
However when using the move up and down keys, the player is not just moving slightly to the right but at full speed (in addition to moving up or down). Could this be caused by controller drift too? One explanation could be that when the player is moving both up and sideways, the sideways velocity would not depend on the joystick anymore but just be at full speed.

I tested it: yes, it is controller drift. My joysticks were showing values between -0.05 and 0.11 while in neutral position. They required a 16% deadzone from SteamVR settings to completely stop drifting.
Indeed, it is also true that while moving both up/down and sideways, the sideways velocity always goes to full speed, regardless of joystick position. This caused the huge sideways drift while moving up/down.

I think the 100% up/down velocity issue should have its own issue.


Unbound breakpoint popup

Type: Bug

Launch a node process that has a "attach" debugger
try to set a breakpoint

If the breakpoint is an "unbound breakpoint" there is a popup screen that appears; the popup has a link to "troubleshoot your launch configuration", but as soon as I move my mouse, the pop up disappears!
VS Code version: Code 1.79.2 (Universal) (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:58:52.392Z)
OS version: Darwin arm64 20.6.0
Modes:

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
1, 1, 1

Memory (System)
8.00GB (0.11GB free)

Process Argv
--crash-reporter-id bb98004e-8ecb-417d-b574-ae93e552ea43

Screen Reader
no

VM
0%

Extensions (32)

Extension
Author (truncated)
Version

vscode-erb-beautify
ali
0.4.0

rails
bun
0.17.8

vscode-gemfile
bun
0.4.2

solargraph
cas
0.24.0

path-intellisense
chr
2.8.4

erb
Cra
0.0.1

githistory
don
0.6.20

prettier-vscode
esb
9.19.0

font-switcher
eva
4.1.0

auto-rename-tag
for
0.1.10

latex-workshop
Jam
9.12.2

rspec-snippets
kar
0.0.4

theme-1
Mar
1.2.6

ruby-rubocop
mis
0.8.6

vscode-docker
ms-
1.25.2

remote-containers
ms-
0.295.0

live-server
ms-
0.4.8

emoji
Per
1.0.1

material-icon-theme
PKi
4.28.0

ruby
reb
0.28.1

vscode-yaml
red
1.13.0

LiveServer
rit
5.7.9

ruby-lsp
Sho
0.3.5

slim
sia
0.1.2

shopify-liquid
sis
3.2.2

code-spell-checker
str
2.20.5

learn-vim
vin
0.0.28

simple-ruby-erb
vor
0.2.1

vscode-icons
vsc
12.4.0

vscode-ruby
win
0.28.0

markdown-all-in-one
yzh
3.5.1

html-css-class-completion
Zig
1.20.0

(7 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30776702
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
a2ce3375:30757347
7ij38806:30736111
pythonfmttext:30731395
pythoncmv:30756943
fixshowwlkth:30771522
pythongtdpath:30769146
bgfeh915:30780428
pythonnosmt12:30779714
pythonidxpt:30772539
pythonnocebcf:30776496

This is probably more due to my inability to work with the debugger, but the fact that the popup disappears when I move my mouse, I'm not able to link to the recommended troubleshooting docs

It is a hover but should stick around when you move your cursor over the hover widget. Is it in the wrong place? Can you share a screen recording?

ya know what, i'm going to close this issue.
This was occurring because of how I was configuring the launch.json... and through trial and error, I was able to properly attach the debugger to the correct file/process.


Typo "**Output readable model file **"→"Output readable model file"

https://docs.microsoft.com/en-us/azure/machine-learning/algorithm-module-reference/train-vowpal-wabbit-model

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

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

Looks good. Thanks for this fix @hyoshioka0128.

Looks good. Thanks for this fix @hyoshioka0128.


"Close Other Panes" seems to close other tabs

The "Close Other Panes" closes most but not all my other tabs for some reason. I would expect split panes to be closed, not other tabs inside the tab group of my current file.

Are you running v1.0.2? If not, please download it and test again.
Can you send me a screenshot of "before" and "after" showing the panes that did not close? And also your console output if there is any?

I'm running whatever's in master.  I think it's v1.0.2 but manifest.json wasn't updated so I'm not 100% sure.
Before:

After:

My other tab got nuked.

Thanks for letting me know about the manifest- yes I forgot to update it. I'll fix shortly.
As for the actual behavior, that actually looks correct to me. The intention of that close other function is to close all other leaves, without regard for whether they are tabs or separate panes.
If you want stacked tabs within the currently active pane to remain open, I will need to make a setting for that.

Not sure why you're calling this "stacked tabs".  To me those are regular tabs.
These are stacked tabs:

You really think the current behavior makes sense?  All I want to do is create a split and the plugin kills all the other tabs.  What do tabs have to do with splits?  I mean, sure, create a setting—that'll work for me.  But don't you think the current behavior is surprising and unhelpful to users?

I used the wrong terminology for stacked tabs. I meant "tabs that are grouped together into a single pane" (leaf). In your case, if you want to activate the side-by-side view without closing other leaves, does turning off the toggle below not do what you want?

Obviously I don't think the behavior is "surprising and unhelpful" 🙀 otherwise I wouldn't have published it. But, I am open to feedback from you and others on how to better implement this or re-word the descriptions so there are no surprises.
One thing I suppose I could implement is, when that toggle is off, we could iterate all the leaves and only close leaves that are part of the current grouping. In that way, repeated pressing of the hotkey would not open 2,3,4...duplicate tabs—it would simply reset the current split view for the active tab without interfering with other leaves.

I just want "splits" (aka "split panes" or simply "panes") closed, not "tabs". "Leaves" is not a common term—never seen it, so I assume that must come from the Obsidian API. Maybe you mean "Close other files"?

repeated pressing of the hotkey would not open 2,3,4...duplicate tabs

Not sure what you mean here.  I thought this plugin only opened a split pane. How can there be duplicate tabs.
I feel like I have a completely wrong model of what this plugin is trying to do.

Yes, leaves are what the views are called internally (API). Yes, I think maybe you are looking for a different feature entirely here. Are you saying you want a feature to close all splits (sort of a "combine all open splits/views into a single pane")? That is definitely not the purpose of this plugin. If I am misunderstanding, please manually arrange your layout "before" and "after" and paste screenshots here so it's clear.

What my friend and I miss from before Obsidian v1 is this: always having 2 splits, one for editing view and one for reading view.
The problem is that when tabs were added in v1, the "linked" reading view on the right doesn't change when one switches tabs in the left split.  To me this is a UX flaw, and there are feature requests for Core to change this, but who knows if and when they'll get to it.
I thought your plugin was going to provide a nice workaround: the ability for me to, once i switched to a new tab, hit a command to quickly open a split reading view; and because i only have so much horizontal real estate, I would want all existing splits on the right to be closed to make room.
Does this make sense?

I keep asking you for screenshots... please post screenshots of what you are asking for. Before and After. I will see if this is something that can be done.
I think you are saying, if you have a random arrangement of tabs, panes, splits, etc, you want the plugin to collapse ALL of them to the "left" and then open a single split pane view to the right of just the active tab. Is that right?

So if I start out with:

If I use core functionality by ⌘-clicking this button:

I get:

With your plugin and "Close other panes" set to on, I would get:

What I was expecting, and what I was asking for is:

But actually, to me this would be ideal as I would lose no tabs:

Now, if there happened to be more than two split panes to begin with, then it would be ok for the third, fourth, etc. splits to be closed, as that would free up horizontal real estate for the reading view.
In any of the above solutions, it would be ideal if when the plugin closed any split, any of the tabs in that split were shifted into the right-most split.  But if that's harder to implement, then I can live without that, and losing the tabs in the closed splits would be fine for me (the reason is that I tend to have ephemeral notes in my non-main split pane; of course, other users may have different schemes).

Screenshots were helpful. I think I understand what you're after now. Basically, preserve tabs you have open no matter what, but always set up the view as a single-split with the active file on the left and right.
That's a challenge, but I will see if I can figure out a way to get it done. It will take some time, I'm open to PRs if you have any ideas on how to implement it.

Thanks for considering it.  I'll try to take a look at it once I'm done looking at the Obsidian Plugin Manager.


Handle outgoing UDP messages also without connect or bind

With UDP, it's possible to send messages with sendto without calling the optional connect and bind first.
We need to make sure outgoing UDP messages are intercepted even if connect or bind are not called.
Also, currently if the application binds port 0, the connection is bypassed and not intercepted because 0 is an ignored_port.
Sockets that bind port 0 should either be intercepted, or it should be documented why they are ignored.
Documentation should be updated accordingly.

Update: hooks were added for sendmsg (#1778) and sendto (#1520), but only for forwarding DNS resolution to the agent.


Added a check for the rust version being within a date range

Sadly, this would break compatibility with Rust 1.0.0 and later versions where Bounds wasn't stabilized. I'm working on a revamping of the API that includes an is_max_date function, however, that would allow for this functionality while still supporting every version of Rust >= 1.0.0.


Update AGL security blueprint

LGTM however the figure  docs/sec-blueprint/App-flow.png  misses the fact that the framework uses also a direct socket connection without D-BUS for connecting to services provided by the framework

LGTM too. Thanks! +1


Suppress and check for deprecated warning

https://trello.com/c/9FmE3rPn/281-double-opt-in-create-api-adapter-to-send-double-opt-in-email
This stops the many 'deprecated' warnings for Rummager cluttering the
test output when running 'rake'.

Can we instead stop using the deprecated functionality?

@issyl0 indeed, although I'm not sure where that might be. @barrucadu @sihugh do you know if we're still using these adapters anywhere?

Superseded by https://github.com/alphagov/gds-api-adapters/pull/966


Disable F11 fullscreen toggle

F11 seems to default to a fullscreen toggle, even if I havent enabled it. How do I disable this? or change it to a different key? I cant find anything in the documentation about it. I believe I am using 3.1.2

This functionality is provided by your window manager, not GLFW. Check the settings for your window manager and/or desktop environment.

Ah, you are correct, toggle fullscreen is configured in ~/.config/openbox/*.xml
Thankyou


Multiple neobundle#begin()/neobundle#end() blocks

Hi,
Is it possible to have multiple neobundle#begin()/neobundle#end() blocks, each in a different .vim file?
I tried this but only the last added .vim file loaded plugins successfully.
Thanks,
Sheng-Loong

Unfortunatelly, I cannot support the feature.
Because, neobundle#begin() resets the RTPs to support multiple .vimrc loading.
@sushengloong Why do you need the feature?   Please explain it.

Thanks for answering, @Shougo.
I am using vim config files from https://github.com/neo/vim-config, of which the .vimrc looks like the following:-
set rtp+=~/.vim/bundle/neobundle.vim/

runtime! custom_preconfig/*.vim
runtime! common_config/*.vim
runtime! custom_config/*.vim

... (truncated)

This setup allows user to customize config or add custom plugins in ~/.vim/custom_preconfig/*.vim or ~/.vim/custom_config/*.vim, as mentioned at https://github.com/neo/vim-config#user-content-customizing.
There is already a neobundle#begin()/neobundle#end() block in the common_config folder which includes a bunch of plugins by default. So calling NeoBundle in custom_preconfig or custom_config will cause the warning below on startup:-
[neobundle] `NeoBundle` commands must be executed within a neobundle#begin/end block.  Please check your usage.
What are your thoughts on this? Thanks.

OK.  I will add neobundle#append() block.
You can use neobundle#append() instead of neobundle#begin().
It does not reset rtps.  But you must call neobundle#begin() before call neobundle#append().
neobundle#begin() initializes neobundle.

Implemented it.

Awesome, it works like a charm.
Many thanks, @Shougo!


Change spark- and mesos-tools to create GP2  volumes instead of standard ones

Pull Request in cgcloud here:
https://github.com/BD2KGenomics/cgcloud/pull/52

Pull request merged, closing the issue


generateTexture signature is inconsistent

I've noticed that:
DisplayObject.prototype.generateTexture = function (renderer, scaleMode, resolution)
and
Container.prototype.generateTexture = function (renderer, resolution, scaleMode)
or
Graphics.prototype.generateTexture = function (resolution, scaleMode)
have different order of arguments (or number), there is no real need to do this  and also this upsets the google-closure-compiler which reports following error:
Running "closure-compiler:dev" (closure-compiler) task
>> grunt-google-closure-compiler: src/externs/pixi-3.0.8-externs.js:2745: WARNING - mismatch of the generateTexture property type and the type of the property it overrides from superclass PIXI.DisplayObject
>> original: function (this:PIXI.DisplayObject, (PIXI.CanvasRenderer|PIXI.WebGLRenderer|null), number, number=): (PIXI.Texture|null)
>> override: function (this:PIXI.Container, (PIXI.CanvasRenderer|PIXI.WebGLRenderer|null), number, number): (PIXI.Texture|null)
>> PIXI.Container.prototype.generateTexture = function (renderer, resolution, scaleMode) {};
>> ^
>>
>> src/externs/pixi-3.0.8-externs.js:3360: WARNING - mismatch of the generateTexture property type and the type of the property it overrides from superclass PIXI.DisplayObject
>> original: function (this:PIXI.Container, (PIXI.CanvasRenderer|PIXI.WebGLRenderer|null), number, number): (PIXI.Texture|null)                                                                                >> override: function (this:PIXI.Graphics, number, number, number=): (PIXI.Texture|null)
>> PIXI.Graphics.prototype.generateTexture = function (resolution, scaleMode) {};
>> ^
>

https://github.com/pixijs/pixi.js/search?utf8=✓&q=generateTexture
Looks like just the DO one is wrong:
https://github.com/pixijs/pixi.js/blob/962f5bdff1ed2d96cfeaef22897ed83b7cce79bd/src/core/display/DisplayObject.js#L483

Hmm, yes, I assume it is not worth to have a type test and swap the params for "old" use?
probably resolution and scaleMode can't be recognised from eachother?

both are numbers :(

maybe based on the valid ranges? surely there is distinct difference between resolution and scaleMode?

scaleMode is a constant value in the range [0, 5] and the resolution is generally in the range [1,3].
Hard to tell the difference...

Thanks for highlighting @LukaszGrela ! generateTexture has been moved to the renderer now for v4. This means all objects get rendered through a single function :+1:


Auth files in laravel 7

Laravel Version: 7x
PHP Version: 7.2.5
Database Driver & Version:

Description:
After laravel upgrade to v7 I install a fresh laravel from composer and there is a problem about auth files. actullay the auth folder is empty so I install the "composer require laravel/ui" and after that I run the "php artisan ui bootstrap --auth" and the controllers and views ... were made buuuut the traits which are located in "Illuminate\Foundation\Auth*" for example the AuthenticatesUsers , RedirectsUsers , ... are not exists and I don't know what to do .
my composer.json :
"name": "laravel/laravel",
"type": "project",
"description": "The Laravel Framework.",
"keywords": [
    "framework",
    "laravel"
],
"license": "MIT",
"require": {
    "php": "^7.2.5",
    "fideloper/proxy": "^4.2",
    "fruitcake/laravel-cors": "^1.0",
    "guzzlehttp/guzzle": "^6.3",
    "laravel/framework": "^7.0",
    "laravel/tinker": "^2.0",
    "laravel/ui": "^2.0",
    "spatie/laravel-activitylog": "^3.13"
},
"require-dev": {
    "facade/ignition": "^2.0",
    "fzaninotto/faker": "^1.9.1",
    "mockery/mockery": "^1.3.1",
    "nunomaduro/collision": "^4.1",
    "phpunit/phpunit": "^8.5"
},
"config": {
    "optimize-autoloader": true,
    "preferred-install": "dist",
    "sort-packages": true
},
"extra": {
    "laravel": {
        "dont-discover": []
    }
},
"autoload": {
    "psr-4": {
        "App\": "app/"
    },
    "classmap": [
        "database/seeds",
        "database/factories"
    ]
},
"autoload-dev": {
    "psr-4": {
        "Tests\": "tests/"
    }
},
"minimum-stability": "dev",
"prefer-stable": true,
"scripts": {
    "post-autoload-dump": [
        "Illuminate\Foundation\ComposerScripts::postAutoloadDump",
        "@php artisan package:discover --ansi"
    ],
    "post-root-package-install": [
        "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
    ],
    "post-create-project-cmd": [
        "@php artisan key:generate --ansi"
    ]
  }
}```

I find the problem.
the command
php artisan ui bootstrap --auth doesn't work in laravel 7 but  the php artisan ui vue --auth works fine and it's very werid

just tested by running the following commands and everything worked as expected:
laravel new playground

composer require laravel/ui

php artisan ui bootstrap --auth

yarn

yarn run dev

Regarding the auth traits, those traits files are now owned by the ui package, but namespaced as usual, you can find those files in the /vendor/laravel/ui/auth-backend folder in your project vendor folder.
As they have the same namespace as before you can import them using the Illumintate\Auth\... namespace as before. Composer allows mapping different folders to the same namespace.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


keras imdb data set is not converged  when maximum size of the word sequence is 500

i am trying to use mxnet to analyze the keras imdb dataset.   using attached python script (imdb_mx2.py), it works well when maximum size of the word sequence  is 150, once i increased the length to 500, it does not converge anymore.
imdb_mx2.py.txt

following are outputs when maxLen = 150
dyld: warning, LC_RPATH ${ORIGIN} in /Library/Python/2.7/site-packages/mxnet/libmxnet.so being ignored in restricted program because it is a relative path
Preparing train set...
Preparing test set...
Trimming to 5000 max-features
Padding to length 150
Epoch 0, Training ('accuracy', 0.6598865089514067)
Epoch 1, Training ('accuracy', 0.8490249360613811)
Epoch 2, Training ('accuracy', 0.8255274936061381)
('Accuracy: ', 0.83776)

following are outputs when maxLen = 500
dyld: warning, LC_RPATH ${ORIGIN} in /Library/Python/2.7/site-packages/mxnet/libmxnet.so being ignored in restricted program because it is a relative path
Preparing train set...
Preparing test set...
Trimming to 5000 max-features
Padding to length 500
Epoch 0, Training ('accuracy', 0.5082320971867008)
Epoch 1, Training ('accuracy', 0.523497442455243)
Epoch 2, Training ('accuracy', 0.5347266624040921)
('Accuracy: ', 0.5114)

i tried it in keras with tensor_flow back end,  even with max_len = 500, it still generated decent result

i found the issue.  the pre-padding instead of post-padding solved the issue.
dyld: warning, LC_RPATH ${ORIGIN} in /Library/Python/2.7/site-packages/mxnet/libmxnet.so being ignored in restricted program because it is a relative path
Preparing train set...
Preparing test set...
Trimming to 5000 max-features
Padding to length 500
Epoch 0, Training ('accuracy', 0.7589114450127877)
Epoch 1, Training ('accuracy', 0.8890664961636828)
Epoch 2, Training ('accuracy', 0.9126838235294118)
Epoch 3, Training ('accuracy', 0.9197969948849105)
('Accuracy: ', 0.81308)


Fourth Korean l10n work for release 1.18

Translate docs/reference/issues-security/_index.md into Korean (#21145)
Translate tasks/administer-cluster/kubeadm/adding-windows-nodes.md into Korean (#21003)
Translate manage-resources/memory-default-namespace.md into Korean (#21089)
Translate tasks/configure-pod-container/pull-image-private-registry in Korean (#21036)
Translate contribute/advanced.md into Korean (#21002)
Update to Outdated files in the dev-1.18-ko.4 branch. (#21102)
Translate manage-resources/cpu-constraint-namespace.md into Korean (#21048)
Translate manage-resources/memory-constraint-namespace.md into Korean (#21091)
Translate setup/release/notes.md in Korean (#21158)
Translate manage-resources/quota-pod-namespace.md into Korean (#21087)
Translate tasks/administer-cluster/kubeadm/upgrading-windows-nodes.md into Korean (#20999)
Translate tasks/administer-cluster/kubeadm/kubeadm-certs.md into Korean (#21000)
Translate manage-resources/quota-memory-cpu-namespace.md into Korean (#21088)
Translate manage-resources/cpu-default-namespace.md into Korean (#21090)
Translate contribute/suggesting-improvements.md into Korean (#21001)
Translate quality-service-pod in Korean (#21081)

Co-authored-by: Jerry Park<EMAIL_ADDRESS>Co-authored-by: Jesang Myung<EMAIL_ADDRESS>Co-authored-by: DongMoon Kim<EMAIL_ADDRESS>Co-authored-by: Jesang Myung<EMAIL_ADDRESS>Co-authored-by: Yuk, Yongsu<EMAIL_ADDRESS>Co-authored-by: bluefriday<EMAIL_ADDRESS>

/assign @ysyukr

@gochist 고생하셨습니다.
/lgtm
/approve


Remove jwe

What is the context of this PR?
https://trello.com/c/ribJzayk/173-remove-unused-jwt-encryption-code-and-tests-3
The JWT encryption that was written was never used so it should be deleted.  This PR removes this code and updates the tests to reflect it.
How to review
Everything should function as normal, unit and acceptance tests should pass.  It would probably be a good idea to send a message, read a thread list and look at a thread to double check that everything is okay as is.

Manually tested and it all works for me


Sort query params

It would be nice if the query params were always displayed in a consistent order. Alphabetical seems ideal.

What, you don't like how every time you start the server the parameters change order? It's like a seek and find!
Can you move the sorting under ToSwaggerParameters at the end of case KindObject: that way paths and headers get the same treatment?

@jakecoffman Done, if you wouldn't mind taking a quick look.


Add variable and function renaming option

Adds the APPLYRESULTS configuration option that parses out the GPT suggested function and variable names and attempts to apply them. Only works on listing based variables (like parameters and data references), I'm not sure how to get access to the decompiler ones right now. Also adds a formatting request to the prompt so the result can be parsed.

thanks for this! i'll kick the tires a bit on monday, and if nothing breaks, i'll try and get it merged by the end of the day.

Okay, it's looking pretty good. I had to make a few tweaks so that HighVariables could be renamed -- there's a whole other mechanism for this that took some time to dig up, but it's there.
# note that you need to pass a HighVariable object and not a Symbol to this function
# for everything to work properly
def rename_high_variable(hv, new_name, data_type=None):
    """takes a high variable object, a new name, and, optionally, a data type
    and sets the name and data type of the high variable in the program database"""

    if data_type is None:
        data_type = hv.getDataType()
    return HighFunctionDBUtil.updateDBVariable(hv,
                unicode(new_name),
                data_type,
                SourceType.ANALYSIS)

I've gotten myself tangled up in a git thicket right now, so i'm going to go ahead and approve this merge, and then merge my fix.


Potential execution time improvement for compareSamples

Hi,
I have noticed that the runtime of the script can be improved for local execution. The subprocess library can run jobs in parallel. Moving the p.wait() out of the _run method, and later checking for errors could significantly improve execution time.
In compareSamples.py
def _run(cmd):
    p = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
    out, err = p.communicate()
    p.wait()
    if p.returncode != 0:
        return err
    return out

Though I believe this would require the TSV making step be moved out of the compareGenomes script, to avoid multiple instances of compareGenomes.py script trying to access and write to the same file.

Yes.  The serial local execution was meant to support easy debugging.  Any optimized running of this is workflow is done in AWS Batch.  I never considered optimizing for running locally.


chore(main): release 0.1.34

Auto generated release
0.1.34 (2024-11-12)
Build System

deps-dev: bump phpstan/phpstan from 1.12.7 to 2.0.1 (#118) (5a1f90a)
deps: bump nesbot/carbon from 3.8.1 to 3.8.2 (#119) (426d9c9)
deps: bump symfony/var-dumper from 7.1.6 to 7.1.7 (#120) (16a44d9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.34
:sunflower:


Custom logs with frame object as the 2nd param

Hi there,
this is a kind of a feature request. I am aware that it's possible to override the logs via client.debug = (log) => ...myLogic. It is cumbersome to do anything with the plain text. You could send the text and, as the 2nd parameter, the frame object. Then we developers can strip headers, shorten some access tokens, etc. ...
At the moment the debug methods accepts a vararg ...args. Every method in src/client.js is using only 1 arg (the message). See the code:
debug(...args) {
        if (this.hasDebug) console.log(...args);
}

I will create a PR in a moment so that you can verify it will make no harm, you'll see.

Here is the promised PR - https://github.com/JSteunou/webstomp-client/pull/56

Hey @JSteunou, what is your opinion?

If I understand well you just need more info in the log like the content of a frame. I could do that, but not the way you did in your PR.

@JSteunou I need to customize the data that is logged and the text is not the best fit. Of course, I don't mind if you do it in a different way.

Could you test #58 and tell me if it matches your need?

@JSteunou Right in the morning I'll check it!


Allow compiling without python

The Python dependency is marked as REQUIRED, but the library can compile just fine without it; this MR turns it into optional.
Also duplicates this fix for missing DESTINATION: https://github.com/graphql/libgraphqlparser/pull/105
So you probably want to merge that one first!

Nah, I complete misread the cmake file, it definitely needs python -- closing this.


@patternfly/react-core breaks the indentation of TreeView

@patternfly/react-core has floating dependency on @patternfly/react-styles, which bumps the version to ^4.7.5 every time we generated new package-lock.json. As mentioned in https://github.com/patternfly/patternfly-react/issues/5165, the identation of TreeView is kinda broken. To fix this issue quickly for 9.2.0 release, I'll pin the version of @patternfly/react-styles to a version that works.

As of 2012.14, @patternfly/react-core now specifies
    "@patternfly/react-styles": "^4.7.16"

which means that patternfly's react-core will now use the buggy 4.7.16, despite our attempt to override with 4.7.12

PatternFly 2012.14 breaks the indentation of TreeView again. We fixed this issue by pinning @patternfly/react-styles to 4.7.12 previously, but @patternfly/react-core now has"@patternfly/react-styles": "^4.7.16", which resulted in two copies of @patternfly/react-styles in our node_modules. Thus, we decided to  override the treeview css rules instead.


Add warning when used remote icon

I sometimes forgot to install local collection when used new icon. It works nobody will notice it.
Could you add optional warning when remote icon was loaded, so I can see it in the dev server console and in CI while building?
Option:
export default defineNuxtConfig({
  icon: {
    remoteWarning: true
  }

How this warning could look like:
ℹ Nuxt Icon server bundle mode is set to local
✔ Nuxt Icon discovered local-installed 5 collections: fontisto, heroicons, material-symbols, ph, simple-icons
 WARN  [Nuxt Icon warn]: Loaded remote icon i-ion-example

Maybe you can try
export default defineNuxtConfig({
  // development time only override
  $development: {
    icon: {
      serverBundle: 'local',
      fallbackToApi: false
    }
  },
  icon: {
    // ... your production settings
  }
})

to enforce you to not using remote on dev time?


There is some error with Computation service. Here is the Error message:  Cannot read properties of undefined (reading '__wbindgen_add_to_stack_pointer')

Describe the bug
Integrate PyGWalker with Shiny-Python  following https://github.com/ObservedObserver/pygwalker-shiny/tree/main.
But when I change ui.HTML(pyg.walk(df, spec="./viz-config.json", return_html=True, debug=False)),
to ui.HTML(pyg.walk(df, spec="./viz-config.json", use_kernel_calc=True,   return_html=True, debug=False)),
bug happens.
and also there is no data in Data Tab。
Without the option: “use_kernel_calc=True”，the program runs normally.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions

pygwalker version:
python version
browser

Additional context
Add any other context about the problem here.

with the option: use_kernel_calc=True

another interesting thing is that the program adding the option: “use_kernel_calc=True” runs normally in jupyter

Hi, @qingfengwuhen , thanks for feedback.
Currently, pygwalker does not implement communication feature on Shiny, so it cannot use python kernel for data calculation.
You can use the following code to run on Shiny:
ui.HTML(pyg.to_html(df, spec="./viz-config.json"))

im strongly looking forward the support of communication feature on Shiny

In todo-list.
Thanks for supporting pygwalker.

I have the same issue, but with dash.


Support Carthage

Expected Behavior
it should be possible to include the framework as a dependency via Carthage
Actual Behavior
Carthage cannot build MMSProfileImagePicker
Steps to Reproduce
Include
github "miller-ms/MMSProfileImagePicker" == 1.1.0
in a Cartfile. run carthage bootstrap

fixed in release 1.3.1


CSmartTable table & column filters won't work if data loaded after setup

(Version v4.1.1)
If you define a CSmartTable and load the items data via an API call (i.e. data is not already present at the time the CSmartTable component is instanced) then the table and column filters will not work.
These filters leverage itemsDataColumns which is defined at setup, and which compares against the keys found in items.value[0].
Example:
<CSmartTable cleaner column-filter table-filter :items="items" :columns="columns" :loading="is_loading">

<script>
export default {
  data() {
    return {
      is_loading: true,
      items: [],
      columns: [
        { key: 'citation' },
        { key: 'title' },
      ],
    };
  },
  mounted() {
    api.load().then((r) => {
      this.is_loading = false;
      this.items = r.data;
    });
  },
}
</script>

The table will load the data, but any time a character is entered into a filter the table will blank out with 0 matches because no field keys can be found.
Workaround:
Give items an initial dummy record with the shape of your data, e.g.:
items: [{ id: 0, citation: '', title: '' }],

(Version v4.1.1)
If you define a CSmartTable and load the items data via an API call (i.e. data is not already present at the time the CSmartTable component is instanced) then the table and column filters will not work.
These filters leverage itemsDataColumns which is defined at setup, and which compares against the keys found in items.value[0].
Example:
<CSmartTable cleaner column-filter table-filter :items="items" :columns="columns" :loading="is_loading">

<script>
export default {
  data() {
    return {
      is_loading: true,
      items: [],
      columns: [
        { key: 'citation' },
        { key: 'title' },
      ],
    };
  },
  mounted() {
    api.load().then((r) => {
      this.is_loading = false;
      this.items = r.data;
    });
  },
}
</script>

The table will load the data, but any time a character is entered into a filter the table will blank out with 0 matches because no field keys can be found.
Workaround:
Give items an initial dummy record with the shape of your data, e.g.:
items: [{ id: 0, citation: '', title: '' }],

I dont know how you made it work to this point , i cloned coreui-pro-vue-admin-template-v4.1.0 example and this code is not working
<template>
    <CSmartTable :items="items" :columns="columns"/>
</template>

<script>
export default {
  data () {
    return {
      columns:[ {key:"test"} ],
      items: []
    }
  },
  mounted:function() {
    this.items = [ {test:1}, {test:2} ]
  },
}
</script>

Table doesnt render any changes that were made to "items" after setup, wasted hours , am i missing something?

Fixed in v4.1.3

Confirmed working in v4.1.3, thanks @mrholek !


x509 Certificate.CheckSignatureFrom() inconsistency with OpenSSL

What version of Go are you using (go version)?

$ go version
go version go1.15.8 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"

Issue Summary
Go's CheckSignatureFrom() function differs from OpenSSL's signature verification. This is because Golang does not check for exact byte match between the signatureAlgorithm field in the TBS and the signatureAlgorithm field outside the TBS. Golang ignores the outside-TBS signatureAlgorithm and populates the Certificate struct with the in-TBS field. OpenSSL seems to perform this check and fails signature validation if it is not the case.
One additional consideration is that Certificate Transparency logs that rely on CheckSignatureFrom()
will validate the signature chain for any certificate that has its outside-of-TBS signatureAlgorithm trivially modified or extended. This is a way to circumvent CT's chain verification spam-control mechanism.
I've discussed this issue briefly with @rolandshoemaker.
What did you do?
Ran CheckSignatureFrom() on a x509.Certificate that has mismatching in-TBS/outside-TBS signatureAlgorithm fields. As long as the in-TBS sigAlgorithm works, the outside-TBS sigAlgorithm ASN.1 sequence can contain nearly anything. The CheckSignature() function also exhibits this divergence from OpenSSL, but it takes in a TBSCert as its argument, so can't enforce OpenSSL-consistent behavior.
Example: https://play.golang.org/p/xegQcq0dU7f
To compare with OpenSSL (tested on LibreSSL 2.8.3 and OpenSSL 1.1.1f): openssl verify -CAfile parent.pem child.pem
What did you expect to see?
An error indicating that the signature is not valid.
What did you see instead?
The signature validates with golang, but throws an error with OpenSSL.

Similar to #21118 (the missing parameters version of this), should be fixed by https://go-review.googlesource.com/c/go/+/235118/.

cc @FiloSottile


DateTime (de-)serialization in different Clients with C# DateTime.MinValue (or values before 1900 depending on timezones)

NSwag toolchain v<IP_ADDRESS>
Certain DateTime values are deserialized differently by the C# and and the Typescript client depending on local Timezones.
TL;DR; Is there a way to provide a custom implementation for the (de-)serialization of Date-object in the Typescript-client? If so we could solve our problem ourselfs with a somewhat hacky work-around.
If not, here is our problem:
Due to a difference in the specification of ECMAScript-Date and C#-DateTime, the serialization of certain date-values seems to be off in JS.
This is due to the fact that different timezones have been changed in the past.
ECMAScript-Spec expect the browser to adjust this to the local time when creating Date-instances.
For example, Berlin, Germary (my timezone), used to be UTC+00:53:28h before April 1893, when it changed to UTC+1h.
Which causes the following output:
new Date("1893-04-01T00:00:00+01:00") => Fri Mar 31 1893 23:53:28 GMT+0053
new Date("1893-04-01T01:00:00+01:00") => Sat Apr 01 1893 01:00:00 GMT+0100

Note that the browser adapts the local timezone that was active at that time, generating an offset of 6:32 min, which will be mirrored in any inputs/displaying of the date.
We have an ASP.NETCore WebApi and two clients generated by NSwag, one C# (Silverlight) and one Typescript (Angular). We are currently migrating towards the Angular-client.
During migration we will have to support both clients, displaying the same Date-values in both application. The Silverlight-Runtime does not change the UTC Offset to UTC+00:53:28h and displays the expected date.
We are using C#'s-DateTime.MinValue across our backend-achitecture and we unfortunatly can not refactor that.
This value apparently uses local time, so our min-value is actually "0001-01-01T00:00:00+01:00"
We also use the date "0001-01-01" to store time of day values (like 7:30 am), if they are not connected to a certain date, also in our local timezone ("0001-01-01T07:30:00+01:00").
These are displayed with a 6:32min offset and cannot be edited correctly.
I am (at least now) aware that using C#'s DateTime.MinValue may be bad design, but there is no way to change this right now.
Is it possible to teach the typescript client to deserialize "0001-01-01T00:00:00+01:00" to Mon Jan 01 0001 00:00:00 GMT+0053 (which would be like "0001-01-01T00:06:32+01:00") and serialize it back to "0001-01-01T00:00:00+01:00" (or UTC "0000-12-31T23:00:00.000Z")?
We can provide a typescript implementation for that if there is a possibility to provide a custom implementation for Date-parsing.
Of course it would be ideal if we could just toggle something like a "override historic timezone"-feature.

Maybe it should be possible to adapt the javascript to json serialization in the typescript client.
Instead of
const content_ = JSON.stringify(input);
we should have something:
const content = customSerializer ? customSerializer.stringify(input) :  JSON.stringify(input)
The customSerializer could be injected in the constructor, analogous to http and baseUrl. However, this would need some small changes to the code.


Dev 1.7.0

Pull Request Test Coverage Report for Build 282

563 of 579 (97.24%)  changed or added relevant lines in 24 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 98.006%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pymcmcstat/chain/ChainStatistics.py
43
44
97.73%

pymcmcstat/plotting/MCMCPlotting.py
51
52
98.08%

pymcmcstat/ParallelMCMC.py
22
24
91.67%

pymcmcstat/chain/ChainProcessing.py
38
40
95.0%

pymcmcstat/settings/ModelParameters.py
24
26
92.31%

pymcmcstat/settings/ModelSettings.py
11
13
84.62%

pymcmcstat/MCMC.py
98
104
94.23%

Files with Coverage Reduction
New Missed Lines
%

pymcmcstat/plotting/MCMCPlotting.py
1
96.26%

pymcmcstat/procedures/CovarianceProcedures.py
2
93.41%

Totals

Change from base Build 245:
-0.2%

Covered Lines:
2212

Relevant Lines:
2257

💛  - Coveralls