Replace single line PHP statements

Summary
I noticed a bug in Laravel that happens when using a single line @php directive followed by a @endphp tag anywhere in the template file. I created an issue on the laravel/framework repository but I was told it is a know issue for quite a while and there is no plan to fix it.
Because the default template files of Sage are using this syntax, it is impossible to use raw PHP inside the loop.
For example this would throw a compilation exception:
@extends('layouts.app')

@section('content')
  @while(have_posts()) @php(the_post())
    ...

    @php
    $var = 'something';
    @endphp

    ...
  @endwhile
@endsection

I would suggest to update the default template files with the multiline syntax so it doesn't happen. The short syntax is easier to read but at least the full one compiles correctly :)
@extends('layouts.app')

@section('content')
  @while(have_posts())
    @php the_post() @endphp
  @endwhile
@endsection

I also suggested to the Laravel team to add a warning to the docs about this.
Additional context
No response

https://discourse.roots.io/t/blade-php-endphp-why-is-it-not-working/23713/11 has more context on this.
I understand that not everyone agrees, but this won't be getting changed in Sage's out of the box views unless it were to stop working all together.


Fix my schedule page tab

If user register only one day sessions, the tab in my schedule page is single.

I think it is not bad. But there might be more good solution.

I think it does not need to do


Update bifrost-wallet.providerlist.json

updated change flare address

Added logo, thanks.

FYI, we've updated our contribution guidelines: https://github.com/TowoLabs/ftso-signal-providers#how-to-add-your-signal-provider and clarified how to run tests locally.


Implement JSON API

We broke all the routes: https://github.com/hoodiehq/hapi-couchdb-account-api
It's now all JSON API, following the specification at http://docs.accountrestapi.apiary.io
We need to adapt the tests & implementation of the client accordingly

I made separate issues for it


electron_25-bin: 25.2.0 -> 25.3.1, electron_24-bin: 24.6.1 -> 24.6.4, electron_23-bin: 23.3.9 -> 23.3.10, electron_22-bin: 22.3.15 -> 22.3.18

Description of changes
https://github.com/electron/electron/releases/tag/v22.3.16
https://github.com/electron/electron/releases/tag/v22.3.17
https://github.com/electron/electron/releases/tag/v22.3.18
https://github.com/electron/electron/releases/tag/v23.3.10
https://github.com/electron/electron/releases/tag/v24.6.2
https://github.com/electron/electron/releases/tag/v24.6.3
https://github.com/electron/electron/releases/tag/v24.6.4
https://github.com/electron/electron/releases/tag/v25.3.0
https://github.com/electron/electron/releases/tag/v25.3.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2455


screenshots with browser ui

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
I have 2 devices: macbook pro and mac mini, the version of chrome is the same. When I take a screenshots on mac mini I get the screenshot with browser ui, and on macbook without.
What is the expected behavior?
On both devices screenshot should be without browser ui.
Specify your

operating system: macbook pro(macos high sierra ver. 10.13.4), mac mini(macos high sierra ver. 10.13.2)
testcafe version: 0.19.1
node.js version: 9.6.1

Hi @AndreyBelym, do you have some new info related to this bug?

Hi @jdc0m, could you please open chrome://flags/#force-color-profile in Chrome on your Mac Mini, change value of the Force color profile flag to sRGB:

Then start your tests using chrome:userProfile as a browser:
testcafe chrome:userProfile $YOUR_TESTS

thx for quick reply @AndreyBelym.
Ok, i change the value to sRGB and the browser ui is still there. then i try to change color profile to Display P3 D65 and the ui is any more there.

Hi, @jdc0m
We discussed the issue with @AndreyBelym and came to the conclusion that we can reproduce it only with direct access to your computer. We can do this using TeamViewer, Skype, or some other means. If direct access is suitable to you, let us know the preferred way for it.

I will close the issue because no activity has been encountered for a long time.
Feel free to reopen the issue if your can provide us the direct access to your computer.


doesn't show copied images

I'm guessing this is intentional (powertoys limitation?)

Hi @ASong5,
Yeah, since PowerToys Run is a text based interface and the main benefit of ClipboardManager is the searchability it wouldn't be possible to include images in the result set.


markAspectCrop default value is incorrect

Setting a default crop on an image with an aspect ratio.
The default crop shown on the image is as expected, but when I hit save, the resultant cropped image is the bottom left quarter (or smaller) of the image.
If I resize the image, the crop works correctly. This is only a problem with the crop set in onImageLoad
import React, { useState, type ReactElement, useRef } from 'react';
import { type DimensionsModalProps } from './DimensionsModalProps';
import { Modal } from '../Modal/Modal';
import ReactCrop, { centerCrop, makeAspectCrop, type Crop } from 'react-image-crop';
import { Buffer } from 'buffer';
import { Button } from '../Button/Button';
import { Loader } from '../Loader/Loader';

export const DimensionsModal = ({ url, file, onComplete, show, setShow, x, y }: DimensionsModalProps): ReactElement => {
  const [crop, setCrop] = useState<Crop>();
  const [imageLoaded, setImageLoaded] = useState(false);
  const imgRef = useRef(null);
  const saveCroppedImage = (): void => {
    if (!file) {
      return;
    }
    if (!crop) {
      return;
    }
    if(!imgRef.current){
      return;
    }
    const canvas = document.createElement('canvas');

    // get the image element
    const image = imgRef.current as HTMLImageElement;

    // draw the image on the canvas
    if (image) {
      const scaleX = image.naturalWidth / image.width;
      const scaleY = image.naturalHeight / image.height;
      const ctx = canvas.getContext('2d');
      const pixelRatio = window.devicePixelRatio;
      canvas.width = crop.width * pixelRatio * scaleX;
      canvas.height = crop.height * pixelRatio * scaleY;

      if (ctx) {
        ctx.setTransform(pixelRatio, 0, 0, pixelRatio, 0, 0);
        ctx.imageSmoothingQuality = 'high';

        ctx.drawImage(
          image,
          crop.x * scaleX,
          crop.y * scaleY,
          crop.width * scaleX,
          crop.height * scaleY,
          0,
          0,
          crop.width * scaleX,
          crop.height * scaleY
        );
      }

      const base64Image = canvas.toDataURL(file.type); // can be changed to jpeg/jpg etc

      if (base64Image) {
        const buffer = Buffer.from(
          base64Image.replace(/^data:image\/\w+;base64,/, ''),
          'base64'
        );
        const newFile = new File([buffer], file.name, { type: file.type });
        onComplete(newFile);
        setShow(false);
      }
    }
  };
  function onImageLoad (e: any): void {
    console.log({imgRef})
    if(!imgRef.current){
      return;
    }
    const { naturalWidth: width, naturalHeight: height } = (imgRef.current as HTMLImageElement);
    const crop = centerCrop(
      makeAspectCrop(
        {
          unit: '%',
          width: 100,
          height: 100
        },
        x  / y,
        width,
        height
      ),
      width,
      height
    );
    setCrop(crop);
    setImageLoaded(true);
  }

  return <Modal setShowModal={setShow} show={show}>
    <Loader loading={!imageLoaded}>
    <ReactCrop aspect={ x / y} crop={crop} onChange={c => {
      console.log(c);
      setCrop(c);
      }}>
      <img ref={imgRef} src={url} onLoad={onImageLoad}/>
    </ReactCrop>
    <Button onClick={saveCroppedImage}>Save</Button>
    </Loader>
  </Modal>;
};

Yes, I'm also facing the same issue.

Yes, I'm also facing the same issue.

I don't have this issue in the Codesandbox demo. I recommend you look there to see how your code differs: https://codesandbox.io/p/sandbox/react-image-crop-demo-with-react-hooks-y831o
P.S. You shouldn't use canvas.toDataURL as it freezes up the main thread which is noticeable for larger images on mobile devices, and a data URL is also less efficient than a blob

Running into this now @elmarti  do you remember what your solution was?

No solution unfortunately and I now don't have access to the project unfortunately. Although I recall there was very little difference in my code to the example


Replacing Jobs with Pods in the building/signing features.

We are not using any of the Job's features and we are reconciling the operator artifacts (jobs/pods) anyway, so it is better to reconcile pods directly.

/cc @yevgeny-shnaidman
@yevgeny-shnaidman @qbarrand @mresvanis Do you think we should also rename the job packages?

/cc @yevgeny-shnaidman
@yevgeny-shnaidman @qbarrand @mresvanis Do you think we should also rename the job packages?

yes, otherwise it is confusing. We should rename job package under build and sign, and rename the "jobhelper" file to "podhelper"

Codecov Report
Patch coverage: 87.50% and project coverage change: -0.11% :warning:

Comparison is base (37217d6) 81.58% compared to head (15353b0) 81.48%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #504      +/-   ##
==========================================
- Coverage   81.58%   81.48%   -0.11%
==========================================
  Files          42       42
  Lines        3758     3748      -10
==========================================
- Hits         3066     3054      -12
- Misses        564      566       +2
  Partials      128      128

Files Changed
Coverage Δ

api/v1beta1/module_types.go
100.00% <ø> (ø)

controllers/hub/managedclustermodule_reconciler.go
73.56% <0.00%> (ø)

controllers/preflightvalidation_reconciler.go
71.65% <0.00%> (ø)

controllers/module_reconciler.go
85.97% <40.00%> (ø)

internal/build/job/manager.go
76.92% <85.71%> (ø)

internal/build/job/maker.go
90.27% <89.65%> (-0.27%)
:arrow_down:

internal/sign/job/manager.go
87.67% <90.00%> (ø)

internal/sign/job/signer.go
84.11% <90.00%> (-0.37%)
:arrow_down:

internal/utils/jobhelper.go
94.38% <92.00%> (-2.25%)
:arrow_down:

internal/cluster/cluster.go
91.61% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

yes, otherwise it is confusing. We should rename job package under build and sign, and rename the "jobhelper" file to "podhelper"

Done.

/lgtm

/lgtm

"msg"="problem running manager" "error"="failed to wait for Module caches to sync: timed out waiting for cache to be synced for Kind *v1beta1.Module"

Doesn't seems to be related to the PR, re-triggering the CI.

/lgtm


Primus can't alt+tab in Diablo III (wine). Works with optirun

Hey,
When using primus I can't alt tab from Diablo III. The game minimizes fine but refuses to maximize again. The game music is played and the cursor is displayed, but it doesn't actually actually draw anything, it maintains the same picture as the last image before trying to alt tab back (like the browser).
However it works perfectly fine when the game (the battle.net launcher) is started with optirun.
When the game fails to render with primus and I kill it the following is written in the console:
primus: fatal: failed to acquire direct rendering context for display thread

And sometimes
primus: warning: glXSwapBuffers: respawning threads after context change

This didn't use to happen until recently (beginning of December) when alt-tabbing an application that was run with primus and wine didn't work at all (the application remained in foreground and you needed to switch to another desktop or hit the "Show desktop" hotkey). I don't know if a change has happened in primus or wine, but I can investigate.  If you have any pointers/suggestions for debugging?
Cheers,
Nick

alt-tabbing an application that was run with primus and wine didn't work at all

This is the first time such problem is reported to me; with which applications did you observe it?

I don't know if a change has happened in primus or wine

In all of 2014, changes to primus happened only on December 28th.

If you have any pointers/suggestions for debugging?

An easy thing to do first is to export PRIMUS_VERBOSE=2 before running primus, and see how perf dumps change after alt-tabbing back.

Hey,
Then it must be a wine change that exposed the behavior. Could it be related to the compositor I am using? I use kwin (compositing suspended when primus is running).
 I don't really see anything here.
Cheers,
Nick

I am pretty sure this is related to wine-staging. Somehow some distributions just start shipping it.

Problems in issue #155 look very similar. Can you provide apitrace traces? Wine log dumps with WINEDEBUG=trace+wgl might be helpful as well.

@amonakov There you go:
log 1
log 2
Log one: Started Battle.net launcher, started the game. Alt tabbed. The game crashed, but not primus: fatal message.
Log two: Started the game, alt-tabbed 3 times successfully, the game crashed on the fourth.
I didn't see anything interesting in the logs but I don't really know what to look for.
@karolherbst , I am using arch linux vanilla wine, which as you can see is vanilla
Cheers,
Nick

I see, I though this issue was related to that, because I started using that. But maybe its just a wine change or something.

Just to reiterate, the problem doesn't appear if the application is windowed (or borderless fullscreen windowed) only if it is proper fullscreen.
Cheers,
Nick

@amonakov  I noticed some fullscreen issues with a lot of other wine applications, too, but never got enough time to create apitraces for that :/ Maybe it also makes sense to bisect wine to see what actually started to cause this.

Please retest after the recent fix for #160

Hey,
I just tested it, 5-6 alt tabs, the problem has disappeared. That seems to have fixed it, thank you!
Cheers,
Nick


Discussion: Supporting CSS variables / custom properties

Opening this issue as a discussion on how the system-props package could support generating CSS variables / custom properties - I haven't given this much thought yet but wanted to jot down a few initial ideas!
My first concept was around aligning on defining a theme object that uses the var(--X) references as theme values, e.g.:
let theme = {
  colors: {
    primary: `var(--primary)`
  }
};

(essentially what the early versions of this package was doing [has since been updated however]).
However this could be less than ideal for a few reasons:

It's clunky / non-standard currently
If you need to read the underlying value of the var (e.g. to compute contrast), you need to do some weird logic

This is even worse than less than ideal because it won't work in non-browser environments!

The benefits however here being that it should _just work_™ out of the box!
My second idea (and something I'm tinkering with a bit) is to enable dependency injection and allow consumers to provide their own get function to createSystem which can be used in place of memoizedGet across the codebase. This is kind of clunky however, as developers may need to replicate a decent amount of logic possibly. Plus I'm sure there will be some edge cases that need to be accounted for (e.g. get used in places that you don't want to return a CSS variable for).
Maybe get is the wrong "unit" to expose and allow for this change however, maybe a new transform codepath can get introduced that gets called with:

value (the result of resolving the prop value against theme, e.g. after get(...)
theme
path
prop name (maybe?)
scale
... more?

and is expected to return the raw css value generated. This could default to essentially a pass through:
function defaultTransform(
  value: string,
/*
  theme: any,
  path: string,
  propName: string,
  scale: string,
  ...
*/
): string {
  return value;
}

Again, this is all sort of top of mind, thinking out loud on supporting CSS variables within system-props, curious if anyone else has other interesting ideas!

I started hacking on a fork / POC that was heavily inspired by this library that generates a CSS variable / custom property output instead of getting the value from theme: https://github.com/ds-pack/simple-props (still a work in progress, doesn't support media queries or pseudo props yet).

I started hacking on a fork / POC that was heavily inspired by this library that generates a CSS variable / custom property output instead of getting the value from theme: https://github.com/ds-pack/simple-props (still a work in progress, doesn't support media queries or pseudo props yet).


Update docker daemon SysV init scripts with upstream improvements

Non-Debian SysV changes to match upstream:

check_for_cleanup: https://github.com/docker/docker/commit/05d04843e6b80f131c07882c882f7e4da018a306
killproc -d 300: https://github.com/docker/docker/commit/640d2ef6f54d96ac4fc3f0f745cb1e6a35148607
unshare: https://github.com/docker/docker/commit/6bb65864589fbd720622cbd795763d108999a366

part of util-linux-ng package, installed on almost all CentOS/RHEL 6+ boxes by default

Debian SysV changes to match upstream:

DOCKER_SSD_PIDFILE: https://github.com/docker/docker/commit/c11fadb282ea7e96eef2280a5e422fd2c94e8c45
set -e / status_of_proc $DOCKER_DESC: https://github.com/docker/docker/commit/478fdaf93353c70a26c0043dadb650fd2f101c40
Should-Start / Should-Stop: https://github.com/docker/docker/commit/36d913078b7ea0138dbc5d561703d068e7e31934
ulimit: https://github.com/docker/docker/commit/bae79c3e7285c170f97a0f7f331a48fb47f0f40e and https://github.com/docker/docker/commit/4f0fbb885eb6b561fd6e4ba26b35cb30dd8d9121

Passed local regression. Merging for 0.37.0 release.


Leaner CI

we do not need to lint in different OS, only one OS, cleanup warning as well

I run lint on four OSes to make sure the app compiles on them and the files specific to that OS are lint-free. I don't recall if unpoller has any OS-specific files. Just lessons learned.

yeah I removed the OS specific files, these were constants that moved to a simple switch statement https://github.com/unpoller/unpoller/blob/master/pkg/poller/config.go#L26


update time to <1.15

Description
Update bounds of time to allow the new 1.14 release.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: build with --constraint='time == 1.14' passed locally

[ ] I updated the CHANGES.md file (n/a)

Actually, since time is a bootlib CI is still using 1.12.2. Have to see how to test it locally.

I believe @slotThe did the testing a couple times in the past when I pinged him on IRC after the Packdeps CI failed.
Perhaps we should add a note about this to MAINTAINERS.md so we don't need to remember (and also to decrease the bus factor).

I saw the

build with --constraint='time == 1.14' passed locally

edit and figured that the local tests ran through.

We could even make the Packdeps job point us to the relevant piece of documentation to minimise the brain function required to deal with this :-D

We could also look into using https://github.com/nomeata/cabal-plan-bounds to generate version constraints automatically, or https://github.com/nomeata/cabal-force-upper-bound to force CI to use the upper bound of a package

edit and figured that the local tests ran through.

They did, and I updated to show it. Should I have deleted that comment afterward? (or edited it, I guess)


DCOS-12971: Add heading to Containers section

This PR adds a heading to the Containers section in a multi-container form.
Before:

After:

Looks good, works great


Cannot restore 1wallet

When trying to restore the 1wallet I can input the wallet address and scan the QR code but it gets stuck at the "Restoring your wallet" page. Will not get past the "Recomputing proofs for each time interval" check.

Wallets created prior to 6/28/2021 may have incorrect parameters during setup. This is fixed by https://github.com/polymorpher/one-wallet/commit/7bc808c0c9a2f89b3cbd08e79d34c1adc0a78e98
If your wallet is a brand new wallet, can you share with me the address so I can take a look at the contract?

Let me create a new one and see if the issue persists.

Let me know if it happens again.


Check for "method has no arguments" considers py an argument

When deciding whether to use METH_NOARGS for a pymethod, the check only considers signature.arguments.is_empty() which includes the py argument.
I.e., argument-less methods with py get a different calling convention, which should not be necessary.
Before I endeavour to fix something: was it intended that py is in the arguments list?

I think this is indeed a bug, but at least one which only causes a de-optimization.


Adds export command

Description

Resolves #66

Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #78 into master will decrease coverage by 0.09%.
The diff coverage is 97.52%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
- Coverage   97.23%   97.13%   -0.10%
==========================================
  Files          20       20
  Lines        1047     1118      +71
  Branches      168      174       +6
==========================================
+ Hits         1018     1086      +68
- Misses         17       18       +1
- Partials       12       14       +2

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.43% <97.02%> (-0.10%)
:arrow_down:

src/spellbot/data.py
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 f1c5a41...9b0c0fa. Read the comment docs.


Bug fixes

patches for workshop

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (952ada7) 58.87% compared to head (2782577) 58.87%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   58.87%   58.87%
=======================================
  Files         138      138
  Lines       14893    14893
=======================================
  Hits         8768     8768
  Misses       6125     6125

Flag
Coverage Δ

tests
58.87% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

setup.py
0.00% <ø> (ø)

mth5/helpers.py
57.89% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Short solution needed: "Adaboost" (python-scikit-learn)

Please help us write most modern and shortest code solution for this issue:
Adaboost (technology: python-scikit-learn)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

Adaboost model usage example


Suggest: shortURL using Google FireBase Dynamic Links

I am trying out igv-webapp for our intranet. Looking great so far and thanks for sharing this app. One suggestion though for shortURLs. I believe google shorturl is now deprecated over firebase dynamic links and bitly seems unstable. It would be better if you can have user-supplied API token for either tinyURL or better, google firebase dynamic link. That way, shorturls are more private and restricted by user supplied api token.
I have been using firebase api for while and it's stable to generate shareable urls. Here is a bash wrapper to generate urls (where FBXK is an api token field). I will tinker with java script to add firebase links at somepoint but thought would be quicker on your end.
https://github.com/igvteam/igv-webapp/blob/0a013c9a7c1c2dd1e9760ee72331545584cc1278/js/urlShortener.js#L47-L61

We are happy with tinyURL.  If you wish to customize your own installation I will consider a pull request to make this customizable, that was actually the intent originally.

Sure and will do that sometime later in June. Thanks again - just setup functional igv portal and using caddy OAuth2 plugin for secure access.


GRPCRoute timeout - GEP-3139

What type of PR is this?
/kind gep
What this PR does / why we need it:
Staying consistent with the HTTPRoute timeout feature, opening a GEP to allow for GRPCRoute timeouts
Which issue(s) this PR fixes:

Fixes # https://github.com/kubernetes-sigs/gateway-api/issues/3139
Does this PR introduce a user-facing change?:

cc @robscott @arkodg

thanks for authoring this GEP @xtineskim and @gnossen for reviewing this in depth !
thinking out loud for gRPC timeouts, thoughts on the below semantics for GRPCRoute ?

If no timeout section is defined,  rely on grpc-timeout header for deciding a per request timeout https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md#requests
timeouts.maxStreamDuration which overrides grpc-timeout header timeout and instead enforces a HTTP/2 stream duration timeout

Thanks @arkodg 😄 !
for your point here:

timeouts.maxStreamDuration which overrides grpc-timeout header timeout and instead enforces a HTTP/2 stream duration timeout
I wonder if this should be the opposite - if a request were to propagate to another service, could it just continually be growing in duration 🤔

Thanks @arkodg 😄 ! for your point here:

timeouts.maxStreamDuration which overrides grpc-timeout header timeout and instead enforces a HTTP/2 stream duration timeout

I wonder if this should be the opposite - if a request were to propagate to another service, could it just continually be growing in duration 🤔

i meant the timeouts.maxStreamDuration would override the timeout value defined in the header, but not overwrite the grpc-timeout header itself

/remove-lifecycle stale


fix: Fix UPI integration test

Description

In recent PR accidentally update env variable for transformer that fail the test as the infer method is not correct for upi http predictor. This PR fix the infer method and parameterize the test whether it is using http predictor or not
Modifications

Update pyfunc_upi_integration_test.py
Tests

Checklist

[ ] Added PR label
[ ] Added unit test, integration, and/or e2e tests
[ ] Tested locally
[ ] Updated documentation
[ ] Update Swagger spec if the PR introduce API changes
[ ] Regenerated Golang and Python client if the PR introduces API changes

Release Notes

Thanks a lot for the fix @tiopramayudi ! Just curious, why is this test not run within the GitHub CICD pipeline though?

because it is  pyfunc predictor so require image building. We exclude pyfunc in github cicd due to resource consumption to run the test and some of the test using feast which complicate the setup


Adding Curiefense to sandbox

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and https://github.com/vmware-tanzu/carvel.dev has issues with stats fetching: Error: ETIMEDOUT
No cached entry, and Curiefense has issues with logo: curiefense.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/curiefense.svg'
Item Curiefense has no image_data

Deploy preview for landscape failed.
Built with commit 367803e9ebaa687cee43b65d6d1d8eaff34133ac
https://app.netlify.com/sites/landscape/deploys/60524eab0a803a0007ac4c8e

hey @jordinl83 what's going on here?

Logo was in the wrong folder

@amye it still doesn't show up =(

Was missing 'project: sandbox' -- it's in the landscape. https://github.com/cncf/landscape/pull/2042 resolves.


Try to fix windows builds

Well, I looked at the wrong version here. It's rattler-build-conda-compat that has >=1, not yet rattler-build itself. The error is:
Building recipe
error: unexpected argument '--extra-meta' found

  tip: a similar argument exists: '--experimental'

Usage: rattler-build build --recipe <RECIPE> --variant-config <VARIANT_CONFIG> <--output-dir <OUTPUT_DIR>|--use-zstd|--use-bz2|--experimental|--auth-file <AUTH_FILE>>

For more information, try '--help'.

ERROR conda.cli.main_run:execute(124): `conda run rattler-build build --recipe recipe -m .ci_support\win_64_.yaml --extra-meta flow_run_id=azure_20240812.1.1 remote_url=https://github.com/conda-forge/jolt-physics-feedstock sha=b97924a944103b24953286d3007d3064d7dd9e01` failed. (See above for error)

CC @baszalmstra @wolfv @nichmor

We are aware. There are two issues:

https://github.com/prefix-dev/rattler-build/issues/1017
And conda smithy not adding --extra-meta on linux and osx.

We will fix that in the next few days!

Closing this as obsolete


Some deploy mechanism is overriding our rest results on github pages

Describe the bug
Some deploy mechanism is overriding our rest results on github pages.
On branch gh-pages some workflow was deleting / overriding test results (852d30600e66ec4ab004b0dbcf2f9a88d6578fcf)
To Reproduce
unknown yet
Expected behavior
missing:
https://thin-edge.github.io/thin-edge.io/results/report-matrix.html
available:
https://abelikt.github.io/thin-edge.io/results/report-matrix.html

I know the reason because I had the similar issue of pushing version bump-up related commits by two bots. Both workflow push the commits to same branch. The solution is to specify to the branch where to push in at least one workflow.
We should use this option publish_branch. https://github.com/peaceiris/actions-gh-pages#-set-another-github-pages-branch-publish_branch
- name: Deploy
  uses: peaceiris/actions-gh-pages@v3
  with:
    github_token: ${{ secrets.GITHUB_TOKEN }}
    publish_branch: your-branch  # default: gh-pages

Ah, it's more complicated than I thought. We need to publish everything to gh-pages branch.

I think the problem is the gh-pages.yml doesn't have destination_dir to specify where to deploy, so it deploys always to the top directory, whilst the ci_pipeline.yml deploys to sub-directory by the key. See: https://github.com/thin-edge/thin-edge.io/blob/main/.github/workflows/ci_pipeline.yml#L1412


Error reporting in {N} AppBuilder Companion App is broken

When the deployed in app crashes with an error in its JavaScript, Ion becomes unresponsive and not debuggable.
To reproduce just add the following in app.js:
__enableVerboseLogging();

var path=require('path');

After crashing the logcat contains:
10-22 14:33:52.845: W/ActivityThread(29043): Application com.telerik.NativeScript is waiting for the debugger on port 8100...
10-22 14:33:52.847: I/System.out(29043): Sending WAIT chunk
10-22 14:33:52.918: I/art(29043): Debugger is active
10-22 14:33:53.047: I/System.out(29043): Debugger has connected
10-22 14:33:53.048: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:53.248: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:53.448: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:53.649: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:53.849: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:54.050: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:54.251: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:54.451: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:54.652: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:54.852: I/System.out(29043): waiting for debugger to settle...
10-22 14:33:55.053: I/System.out(29043): debugger has settled (1486)
10-22 14:33:55.104: I/LoadedApk(29043): No resource references to update in package common
10-22 14:33:55.104: I/LoadedApk(29043): No resource references to update in package com.cyngn.hexo
10-22 14:33:55.105: I/LoadedApk(29043): No resource references to update in package com.cyngn.hexo
10-22 14:34:12.999: I/TNS.Native(29043): NativeScript Runtime Version 1.4.0, commit bb5a2c9dc53f717981993781f6052e5418fcaa18
10-22 14:34:12.999: D/TNS.Native(29043): JNI_ONLoad
10-22 14:34:13.002: D/TNS.Native(29043): JNI_ONLoad END
10-22 14:34:13.503: D/TNS.Native(29043): lenNodes=62472, lenNames=650593, lenValues=661763
10-22 14:34:13.503: D/TNS.Native(29043): time=7
10-22 14:34:13.938: D/TNS.Java(29043): Module path not found. required from directory /data/data/com.telerik.NativeScript/files/app
10-22 14:34:13.963: E/TNS.Native(29043): Error: Module "path" not found
10-22 14:34:13.963: E/TNS.Native(29043): File: "/data/data/com.telerik.NativeScript/files/app/app.js, line: 0, column: 263
10-22 14:34:13.963: E/TNS.Native(29043): StackTrace:
10-22 14:34:13.963: E/TNS.Native(29043): 	Frame: function:'require', file:'/data/data/com.telerik.NativeScript/files/app/app.js', line: 2, column: 264
10-22 14:34:13.963: E/TNS.Native(29043): 	Frame: function:'', file:'/data/data/com.telerik.NativeScript/files/app/app.js', line: 3, column: 10
10-22 14:34:13.963: D/TNS.Native(29043): Error:
10-22 14:34:13.963: D/TNS.Native(29043): Error calling module function
10-22 14:34:13.963: D/TNS.Native(29043): Error: Module "path" not found
10-22 14:34:13.963: D/TNS.Native(29043): File: "/data/data/com.telerik.NativeScript/files/app/app.js, line: 0, column: 263
10-22 14:34:13.963: D/TNS.Native(29043): StackTrace:
10-22 14:34:13.963: D/TNS.Native(29043): 	Frame: function:'require', file:'/data/data/com.telerik.NativeScript/files/app/app.js', line: 2, column: 264
10-22 14:34:13.963: D/TNS.Native(29043): 	Frame: function:'', file:'/data/data/com.telerik.NativeScript/files/app/app.js', line: 3, column: 10
10-22 14:34:13.964: D/TNS.Native(29043): ObjectManager::GetJSInstanceInfo: called

It is expected that the runtime would save the error, restart the application and pass it to the registered error reporting activity.

Note that {N} Companion App with 1.4.0 is still not released and only available in a feature branch.

We have broken the {N} Companion App with one refactoring of the error handling. Fortunately the issue is resolved after a debug session.
closing this issue


[Bug]: error: TypeError: Cannot read properties of undefined

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Heute update auf 3.3.0 gemacht seitdem steht alles auf 0 adapter ist grün im log steht:
`

[ Adapter V:3.3.0 writeState ] error: TypeError: Cannot read properties of undefined (reading 'toString') stack: TypeError: Cannot read properties of undefined (reading 'toString') at Tankerkoenig.writeState (/opt/iobroker/node_modules/iobroker.tankerkoenig/build/main.js:865:42) at async Tankerkoenig.requestData (/opt/iobroker/node_modules/iobroker.tankerkoenig/build/main.js:117:11) at async Timeout._onTimeout (/opt/iobroker/node_modules/iobroker.tankerkoenig/build/main.js:74:13)

`
What did you expect to happen?
No response
How to reproduce it (minimal and precise)
No response
Adapter version
3.3.0
Environment
- Node:v16.19.0
- npm:8.19.3
- JS-Controller:4.0.24
- Operating system:Debian 11

What browsers are you seeing the problem on?
No response
Screenshots & Log
No response

Same here with 3.3.0 on each instance start I got this error:
2022-12-30 09:37:00.194 - info: tankerkoenig.0 (685640) starting. Version 3.3.0 in /opt/iobroker/node_modules/iobroker.tankerkoenig, node: v16.19.0, js-controller: 4.0.24
2022-12-30 09:37:00.228 - info: tankerkoenig.0 (685640) Adapter is already in daemon mode
2022-12-30 09:37:00.229 - info: tankerkoenig.0 (685640) Sync time set to 10 minutes or 600000 ms
2022-12-30 09:37:01.559 - info: tankerkoenig.0 (685640) Start first request
2022-12-30 09:37:01.904 - error: tankerkoenig.0 (685640) [ Adapter V:3.3.0 writeState ] error: TypeError: Cannot read properties of undefined (reading 'toString') stack: TypeError: Cannot read properties of undefined (reading 'toString')
    at Tankerkoenig.writeState (/opt/iobroker/node_modules/iobroker.tankerkoenig/src/main.ts:1033:35)
    at Tankerkoenig.requestData (/opt/iobroker/node_modules/iobroker.tankerkoenig/src/main.ts:156:6)
    at Timeout._onTimeout (/opt/iobroker/node_modules/iobroker.tankerkoenig/src/main.ts:96:7)

Until 3.2.5 all was fine.

After the update, you have to go to the UI again and then click on

and copy the ID once again, so that the data is taken over correctly, by clicking on add and save.

Ok, seams to solve this issue but now I got thsi for every station:
State value to set for "tankerkoenig.0.stations.7.postCode" has to be type "number" but received type "string"

Delete the state one time and restart the will regenerate newly created. The type has changed to number

Von github 3.3.1 installiert alles wieder ok


[SPARK-48444] Use withSQLConf in tests: Refactor SQLQuerySuite

What changes were proposed in this pull request?

Use withSQLConf in tests when it is appropriate.
Why are the changes needed?

Enforce good practice for setting config in test cases.
Does this PR introduce any user-facing change?

NO
How was this patch tested?

Existing UT
Was this patch authored or co-authored using generative AI tooling?

NO

@cloud-fan


AMBARI-24936. Fix suspicious code spotted while working on AMBARI-24936

What changes were proposed in this pull request?
Looked through critical warnings of static code analyzer and fixed some obvious issues
How was this patch tested?
unit tests

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4608/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4614/
Test PASSed.


Use versioning for the docker image

From @emirotin on August 31, 2016 10:46
The plan is simple: keep a version file for the docker image
It has to be bumped when the breaking changes are added to the dockerfile
It has to be read when building the docker image, and used to tag that image
It has to be read when using the docker image (to load the image with the corresponding tag).
As the file is available inside of this repo (that is used as a submodule) it should provide the seamless integration with the current process (as the submodule is updated the file will be updated, etc.)
Copied from original issue: resin-os/resin-yocto-scripts#32

What is this supposed to be a workaround for?

Sorry, I don't remember. It belongs to the yocto-scripts where it was originally reported. It has something to do with the way we're building images. I wonder if we have since introduced the proper versioning for the OS releases that supersedes this.


TypeError: Cannot set property readonly of #<Object> which has only a getter

Hi @MartijnR,
In light of this change, do you have any advice on how to handle the following?
https://github.com/kobotoolbox/enketo-literacy-test-widget/blob/839a31eae903e75c6fea5f245bf1a1557f0e2dee/literacywidget.js#L27
Right now, it's throwing a
TypeError: Cannot set property readonly of #<Object> which has only a getter
    at LiteracyWidget.get props [as props] (enketo-webform-bundle.min.js:1)
    at LiteracyWidget._getCurrentModelValue (enketo-webform-bundle.min.js:1)
    at LiteracyWidget._init (enketo-webform-bundle.min.js:1)
    at new Widget (enketo-webform-bundle.min.js:1)
    at new LiteracyWidget (enketo-webform-bundle.min.js:1)
    at Collection._instantiateSingleWidget (enketo-webform-bundle.min.js:1)
    at enketo-webform-bundle.min.js:1
    at Array.forEach (<anonymous>)
    at Collection.instantiate (enketo-webform-bundle.min.js:1)
    at enketo-webform-bundle.min.js:1

P.S.: The <Object> part of the exception message isn't getting escaped when presented to the user:

Hi @jnm,
For properties that are not determined by the Widget super class, we override the get props() method. See e.g the geo widget.
The readonly property is now determined in the Widget super class so the line that causes this error can just be removed.


Wizard: Update the File system step

Fixes #911. Fixes #912.
This updates the File system step. The changes are:

changed the toggle group for selecting the mode of partitioning to radio select and updated information for each radio button as mentioned in #911
added information to the "automatic" option of the file system configuration as mentioned in #912
updated information for the "manual" option of the configuration as mentioned in mocks

/retest


Add seek by count index

Add a seek index for each data object that stores an ordered list of
counts and offsets. The name of the new file is
[OBJECT_ID]-seek-count.zng.

There's no test for this feature but testing will be added when this feature is actually used in #3018


Adds serialization test

Fixes #631.

This depends on #633. I am adding the commits to that PR and closing this one.


Saving any scene on godot v4 beta 17 (google play version)

Godot version
4.0.0.beta.17
System information
Android 12 Mediatik helio g95 6 ram 128gb
Issue description
So I made a project scene of a floor with a character and simple movement system and I tried to save it to test it and it saved but kept giving me this error:
editor/progress_dialog.cpp:173 - Do not use progress dialog (task) while flushing the message queue or using call_deferred() ! editor/progress_dialog.cpp:206 - Condition "!tasks.has (p_task)" is true. Returning: canceled editor/progress_dialog.cpp:206 - Condition "!tasks.has (p_task)" is true. Returning: canceled • editor/progress_dialog.cpp:206 Condition "!tasks.has (p_task)" is true. Returning: canceled • editor/progress_dialog.cpp:206 Condition "Itasks.has (p_task)" is true. Returning: canceled
• editor/progress_dialog.cpp:206 Condition "!tasks.has(p_task)" is true. Returning: canceled
• editor/progress_dialog.cpp:235 Condition "!tasks.has(p_task)" is true.
And the fps was horrible like 5 fps or something then I tried on a different scenes even tried a scene with just node 3d and all gives the same error while trying to save the scene
Steps to reproduce
Just saving any scene
Minimal reproduction project

https://user-images.githubusercontent.com/115576911/218261716-69c7c4e8-95f4-4993-bc4c-9bcb6a6dbe93.mp4

Can you try the release candidate?

Duplicate of https://github.com/godotengine/godot/issues/73097.


:book: docs(README): remove publisher badge

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
814

Relevant Lines:
814

💛  - Coveralls


[CH] RewriteDateTimestampComparisonRule fails on some cases

Backend
CH (ClickHouse)
Bug description
[Expected behavior] and [actual behavior].
RewriteDateTimestampComparisonRule doesn't work on following case
SELECT
count(comment_uid) as n
FROM
mysql_tb.welog_tbl_video_comment
WHERE FROM_UNIXTIME(comment_time, 'yyyy-MM-dd') BETWEEN date_sub('${day}',14)  AND '${day}'

The corresponding filter expression is
 ((cast(from_unixtime(comment_time#7L, yyyy-MM-dd, Some(Asia/Shanghai)) as date) >= date_sub(cast(2024-01-29 as date), 14)) AND (from_unixtime(comment_time#7L, yyyy-MM-dd, Some(Asia/Shanghai)) <= 2024-01-29))

Spark version
None
Spark configurations
No response
System information
No response
Relevant logs
No response

Compare the filter expression use date and timestamp, we could see that, the timestamp comparison is more friendly to push down the filt condition.

comparison with casting timestamp to date

comparision with timestamp


This project could not be access

When I added this paths to my gradle file it will show error message telling "Failed to resolve:com.github.dbachelder:CreditCardEntry:v1.3.0" like that.

Did you add the repository?
maven { url "https://jitpack.io" }

Yes, I added it also under
repositories{
.......
maven { url "https://jitpack.io" }
}
But it gave that issue. What can be the issue..?
On 15 Jul 2015 19:45, "dan"<EMAIL_ADDRESS>wrote:

Did you add the repository?
maven { url "https://jitpack.io" }
—
Reply to this email directly or view it on GitHub
https://github.com/dbachelder/CreditCardEntry/issues/17#issuecomment-121629516
.

No idea... You'll need to post your entire build.gradle

@amila-rangana also I just released a new version. so try com.github.dbachelder:CreditCardEntry:1.4.0

Hi dan,
Now it is working great. Thank you very much. But extra attributes like
app:helper_text_color="@color/yellow_500"
is not working, then I have added
xmlns:app="http://schemas.android.com/apk/res-auto"
into root element. But it is also giving error like
Error:(9) No resource identifier found for attribute
'helper_text_color' in package 'com.devmarvel.creditcardentry'
Error:(9) No resource identifier found for attribute 'include_helper'
in package 'com.devmarvel.creditcardentry'
Error:(9) No resource identifier found for attribute 'include_zip' in
package 'com.devmarvel.creditcardentry'
Error:(15, 32) No resource found that matches the given name (at
'helper_text_color' with value '@color/yellow_500').
what should I do for that?
Regards,
Amila

Amila Rangana
Software Engineer
PHIZUU
Mobile   : +94 718310108
Office   : +94 117548675
Email<EMAIL_ADDRESS>Web      : http://www.phizuu.com
On Wed, Jul 15, 2015 at 10:48 PM, dan<EMAIL_ADDRESS>wrote:

@amila-rangana https://github.com/amila-rangana also I just released a
new version. so try com.github.dbachelder:CreditCardEntry:1.4.0
—
Reply to this email directly or view it on GitHub
https://github.com/dbachelder/CreditCardEntry/issues/17#issuecomment-121684524
.

@amila-rangana Hmm, that's fishy.. can you run the demo app?

Hi dan,
I could figured it out correctly with your latest release. It is working
perfectly. Thank you very much.
Regards,
Amila

Amila Rangana
Software Engineer
PHIZUU
Mobile   : +94 718310108
Office   : +94 117548675
Email<EMAIL_ADDRESS>Web      : http://www.phizuu.com
On Fri, Jul 17, 2015 at 10:08 PM, dan<EMAIL_ADDRESS>wrote:

@amila-rangana https://github.com/amila-rangana Hmm, that's fishy.. can
you run the demo app?
—
Reply to this email directly or view it on GitHub
https://github.com/dbachelder/CreditCardEntry/issues/17#issuecomment-122335618
.


Add date filter

This commit adds a date filter for the query, which defaults to the year zero to return all updates.

This is at least 7 kinds of fantastic. Thanks! :shipit: :rocket:


Refactor gpu interior face integration

This PR refactors the interior face integration routines for the gpu.
Summary of code changes
The changes include

Use Marc's approach from #78, which refactors and simplifies some of the mfem device macro usage
Reduce duplicate calculations by

Making each element responsible for interpolating its own state to the faces (rather than its own and its neighbors')
Evaluating the flux at the face quadrature points once and storing the result rather than recomputing it for both elements on a face

Clean up, mainly by using class data where possible rather than function arguments

These changes have only been applied to the interior face routines.  Analogous changes could be applied to the shared face calculations as well (one assumes), but to keep this PR small, that has not been attempted yet.
Performance changes
The modifications have improved performance somewhat.  Specifically, on 1 mpi rank on Lassen, a modified version of the cylinder test case goes from 17.7 seconds on main (in the solve) to 10.9, an improvement of about 38%.  The modifications to the test are 1) the order is increased to p=3, 2) the number of steps is increased to 100, and 3) the output frequency is reduced.  The input file diff relative to the regression test is below:
[oliver33@lassen708:test]$ diff inputs/input.dtconst.cyl.ini inputs/input.dtconst.cyl100.ini
6c6
< order = 1
---
> order = 3 # 1
9,10c9,10
< maxIters = 4
< outputFreq = 5
---
> maxIters = 100
> outputFreq = 20
21c21
< cfl = 0.80
---
> cfl = 0.12 # 0.80

The output from the grvy timers for this case is below:

On main before the PR:

-----------------------------------------------------------------------------------------------
TPS - Performance Timings:                              |      Mean      Variance       Count
--> solve               : 1.76863e+01 secs ( 79.2083 %) | [1.76863e-01  2.60817e-02        100]
--> restart_files_hdf5  : 6.86684e-02 secs (  0.3075 %) | [1.37337e-02  5.72839e-07          5]
--> GRVY_Unassigned     : 4.57388e+00 secs ( 20.4841 %)

    Total Measured Time = 2.23289e+01 secs (100.0000 %)
-----------------------------------------------------------------------------------------------

On this branch:

-----------------------------------------------------------------------------------------------
TPS - Performance Timings:                              |      Mean      Variance       Count
--> solve               : 1.09058e+01 secs ( 70.1072 %) | [1.09058e-01  2.59142e-02        100]
--> restart_files_hdf5  : 6.57170e-02 secs (  0.4225 %) | [1.31434e-02  6.02010e-07          5]
--> GRVY_Unassigned     : 4.58439e+00 secs ( 29.4703 %)

    Total Measured Time = 1.55560e+01 secs (100.0000 %)
-----------------------------------------------------------------------------------------------

Finally, snippets of nvprof output for this case are included here to show the reduction in time spent in kernels associated with interior face integration on the gpu:

On main before this PR:

==21552== Profiling application: ../src/tps -run inputs/input.dtconst.cyl100.ini
==21552== Profiling result:
            Type  Time(%)      Time     Calls       Avg       Min       Max  Name
 GPU activities:   45.91%  6.72093s       400  16.802ms  16.745ms  23.898ms  _ZN4mfem10CuKernel2DIZN15DGNonLinearForm19faceIntegration_gpuERNS_6VectorES3_S3_S3_S3_PKNS_15ParGridFunctionEP6FluxesRKiSA_SA_SA_SA_SA_SA_P10GasMixtureRK27volumeFaceIntegrationArraysSA_SA_EUliE_EEviT_i
                   22.53%  3.29769s       400  8.2442ms  8.1945ms  9.2603ms  _ZN4mfem10CuKernel2DIZN15DGNonLinearForm18interpFaceData_gpuERKNS_6VectorERS2_S5_S5_S5_PKNS_15ParGridFunctionERKiSA_SA_SA_SA_SA_SA_RKdSC_SC_SC_SC_RK27volumeFaceIntegrationArraysSA_SA_EUliE_EEviT_i
                   11.30%  1.65469s       400  4.1367ms  4.1190ms  4.7785ms  _ZN4mfem10CuKernel2DIZN9Gradients15faceContrib_gpuEiiiiRKNS_6VectorERS2_iiRK27volumeFaceIntegrationArraysRKiSA_EUliE_EEviT_i
                    7.87%  1.15157s       400  2.8789ms  2.8687ms  3.2078ms  _ZN4mfem10CuKernel2DIZN9Gradients20computeGradients_gpuEiiiiRKNS_6VectorERS2_iiRK27volumeFaceIntegrationArraysRKiSA_EUliE_EEviT_i
                    4.54%  518.67ms       400  1.2967ms  1.2894ms  1.6569ms  _ZN4mfem10CuKernel2DIZN6WallBC18integrateWalls_gpuE8WallTypeRKdRNS_6VectorERKS5_S6_S6_RKNS_5ArrayIiEESC_PNS_15ParGridFunctionESE_S6_S6_RSA_SF_SF_RK9EquationsRKiSK_SK_SK_P10GasMixtureEUliE_EEviT_i
                    1.86%  271.92ms        54  5.0356ms  1.3760us  120.89ms  [CUDA memcpy HtoD]
                    1.66%  243.64ms      2000  121.82us  118.62us  144.03us  _ZN8cusparse21load_balancing_kernelILj512ELj4ELm16384EiiNS_7CsrmvOpILi512EdLb0EEEJKiKdS4_didEEEvPKT3_T2_S5_S5_iPKS8_T4_DpPT5_
                    0.97%  141.28ms       400  353.19us  350.14us  434.75us  _ZN4mfem10CuKernel2DIZN6Fluxes17viscousFluxes_gpuERKNS_6VectorEPNS_15ParGridFunctionERNS_11DenseTensorERK9EquationsP10GasMixturePKS5_RK19linearlyVaryingViscRKiSK_SK_EUliE_EEviT_i
                    0.82%  119.78ms       400  299.46us  297.05us  354.91us  _ZN4mfem10CuKernel2DIZN6WallBC15interpWalls_gpuE8WallTypeRKdRNS_6VectorES6_RKS5_RKNS_5ArrayIiEESC_PNS_15ParGridFunctionESE_S6_S6_RSA_SF_SF_RKiSH_SH_SH_EUliE_EEviT_i
                    0.80%  117.37ms       400  293.43us  290.62us  328.16us  _ZN4mfem10CuKernel2DIZN9Gradients15multInverse_gpuEiiiiRNS_6VectorEiiRK27volumeFaceIntegrationArraysRKS2_RKNS_5ArrayIiEEEUliE_EEviT_i
                    0.62%  90.356ms       400  225.89us  223.45us  268.48us  _ZN4mfem10CuKernel2DIZN11RHSoperator18multiPlyInvers_gpuERNS_6VectorES3_RK27volumeFaceIntegrationArraysRKS2_RKNS_5ArrayIiEEiiiiiEUliE_EEviT_i
                    0.50%  72.477ms       400  181.19us  180.25us  255.39us  _ZN4mfem10CuKernel2DIZN6Fluxes20convectiveFluxes_gpuERKNS_6VectorERNS_11DenseTensorERK9EquationsP10GasMixtureRKiSD_SD_EUliE_EEviT_i

On this branch:

==63462== Profiling application: ../src/tps -run inputs/input.dtconst.cyl100.ini
==63462== Profiling result:
            Type  Time(%)      Time     Calls       Avg       Min       Max  Name
 GPU activities:   21.26%  1.65840s       400  4.1460ms  4.1182ms  4.7775ms  _ZN4mfem10CuKernel2DIZN9Gradients15faceContrib_gpuEiiiiRKNS_6VectorERS2_iiRK27volumeFaceIntegrationArraysRKiSA_EUliE_EEviT_i
                   18.27%  1.42513s       400  3.5628ms  3.5074ms  4.0475ms  _ZN4mfem10CuKernel2DIZN15DGNonLinearForm18interpFaceData_gpuERKNS_6VectorEiiiEUliE_EEviT_i
                   15.81%  1.23360s       400  3.0840ms  3.0610ms  3.2895ms  _ZN4mfem10CuKernel1DIZN15DGNonLinearForm19faceIntegration_gpuERNS_6VectorEiiiEUliE_EEviT_
                   14.78%  1.15312s       400  2.8828ms  2.8688ms  3.2065ms  _ZN4mfem10CuKernel2DIZN9Gradients20computeGradients_gpuEiiiiRKNS_6VectorERS2_iiRK27volumeFaceIntegrationArraysRKiSA_EUliE_EEviT_i
                    6.67%  520.04ms       400  1.3001ms  1.2891ms  1.6550ms  _ZN4mfem10CuKernel2DIZN6WallBC18integrateWalls_gpuE8WallTypeRKdRNS_6VectorERKS5_S6_S6_RKNS_5ArrayIiEESC_PNS_15ParGridFunctionESE_S6_S6_RSA_SF_SF_RK9EquationsRKiSK_SK_SK_P10GasMixtureEUliE_EEviT_i
                    6.62%  516.48ms       400  1.2912ms  1.2678ms  1.3494ms  _ZN4mfem10CuKernel1DIZN15DGNonLinearForm16evalFaceFlux_gpuEvEUliE_EEviT_
                    3.49%  271.97ms        54  5.0366ms  1.3760us  120.03ms  [CUDA memcpy HtoD]
                    3.07%  239.86ms      2000  119.93us  116.80us  141.76us  _ZN8cusparse21load_balancing_kernelILj512ELj4ELm16384EiiNS_7CsrmvOpILi512EdLb0EEEJKiKdS4_didEEEvPKT3_T2_S5_S5_iPKS8_T4_DpPT5_
                    1.84%  143.91ms       400  359.78us  355.42us  444.67us  _ZN4mfem10CuKernel2DIZN6Fluxes17viscousFluxes_gpuERKNS_6VectorEPNS_15ParGridFunctionERNS_11DenseTensorERK9EquationsP10GasMixturePKS5_RK19linearlyVaryingViscRKiSK_SK_EUliE_EEviT_i
                    1.54%  120.52ms       400  301.29us  298.20us  355.71us  _ZN4mfem10CuKernel2DIZN6WallBC15interpWalls_gpuE8WallTypeRKdRNS_6VectorES6_RKS5_RKNS_5ArrayIiEESC_PNS_15ParGridFunctionESE_S6_S6_RSA_SF_SF_RKiSH_SH_SH_EUliE_EEviT_i
                    1.51%  117.64ms       400  294.11us  290.36us  329.08us  _ZN4mfem10CuKernel2DIZN9Gradients15multInverse_gpuEiiiiRNS_6VectorEiiRK27volumeFaceIntegrationArraysRKS2_RKNS_5ArrayIiEEEUliE_EEviT_i
                    1.16%  90.609ms       400  226.52us  224.25us  269.05us  _ZN4mfem10CuKernel2DIZN11RHSoperator18multiPlyInvers_gpuERNS_6VectorES3_RK27volumeFaceIntegrationArraysRKS2_RKNS_5ArrayIiEEiiiiiEUliE_EEviT_i
                    0.93%  72.541ms       400  181.35us  180.03us  256.48us  _ZN4mfem10CuKernel2DIZN6Fluxes20convectiveFluxes_gpuERKNS_6VectorERNS_11DenseTensorERK9EquationsP10GasMixtureRKiSD_SD_EUliE_EEviT_i
                    0.92%  71.653ms       400  179.13us  177.60us  209.47us  _ZN4mfem10CuKernel2DIZN11RHSoperator20updatePrimitives_gpuEPNS_6VectorEPKS2_ddiiiRK9EquationsEUliE_EEviT_i
                    0.39%  30.584ms       400  76.459us  75.612us  98.492us  _ZN4mfem10CuKernel2DIZN7InletBC19integrateInlets_gpuE9InletTypeRKNS_6VectorERKdRS3_S5_S8_RKNS_5ArrayIiEESC_PNS_15ParGridFunctionESE_S8_S8_RSA_SF_SF_RKiSH_SH_SH_P10GasMixtureR9EquationsEUliE_EEviT_i

Performance update
I was tricked by the grvy timer output above into thinking that the restart and visualization file write step was negligible.  However, the restart_files_hdf5 timing only includes the restart file write, not the visualization file write (and time required to move primitives to the host and update the pressure).  As a result, changing flow/outputFreq significantly affects the time spent in solve (at least for this case).  For instance, setting flow/outputFreq = 100 (instead of 20 as above) and leaving everything else as before leads to the following times (still lassen, 1 mpi rank):

On main prior to this PR:

-----------------------------------------------------------------------------------------------
TPS - Performance Timings:                              |      Mean      Variance       Count
--> solve               : 1.44515e+01 secs ( 75.9066 %) | [1.44515e-01  2.83485e-06        100]
--> restart_files_hdf5  : 1.40920e-02 secs (  0.0740 %) | [1.40920e-02  0.00000e+00          1]
--> GRVY_Unassigned     : 4.57292e+00 secs ( 24.0194 %)

    Total Measured Time = 1.90385e+01 secs (100.0000 %)
-----------------------------------------------------------------------------------------------

On this branch:

-----------------------------------------------------------------------------------------------
TPS - Performance Timings:                              |      Mean      Variance       Count
--> solve               : 7.64890e+00 secs ( 62.3466 %) | [7.64890e-02  5.63179e-05        100]
--> restart_files_hdf5  : 1.54190e-02 secs (  0.1257 %) | [1.54190e-02  0.00000e+00          1]
--> GRVY_Unassigned     : 4.60403e+00 secs ( 37.5277 %)

    Total Measured Time = 1.22684e+01 secs (100.0000 %)
-----------------------------------------------------------------------------------------------

Now the performance improvement for solve is about 47%.


Fix i2c devices on avr32 (7bits/8bits addresses)

Hot fix after #264 .
IMU sensors are not properly responding. I think this is because of an issue with the address not being correct.
See issue #267

@GregoireH I will accept this PR because the previous PR #264 broke the avr32 board.
I guess we are both to blame for not testing on the hardware :-1: ^^

Arg sorry...
I tested the sonar but not the IMU,
my bad !
I ll have a look on thursday afternoon


Implement root workspace and organization workspaces

This PR lifts the kcp platform to have real (Cluster)Workspace support by adding a hierarchy of the

root workspace (replacing admin)
organization workspaces (= workspaces of type Organization defined in root workspace)
non-org workspaces (= workspaces of other type, Universal by default, defined in org workspaces.

For this, the following bigger changes are included:

all controllers and e2e tests are extended to not depend on an admin workspace anymore. I.e. they create real org workspaces and in there real universal workspaces to do the actual work (cc @marun this might open up the possibility to use shared fixtures in a number of tests).
logical clusters are separated by : instead of _, mostly for cosmetical reason, and to align with the use of : in Kubernetes, e.g. in RBAC roles and bindings. Underscore feels foreign. With that a cluster name is either a) root b) : or c) system:*. Org workspaces have logical cluster name of the shape root:<org>.
Workspaces and renamed to ClusterWorkspaces to allow us to have different sources of workspace definition in the future. The virtual personal workspace apiserver will return Workspaces which are simplified, supportable projections of ClusterWorkspaces. The latter can include low-level fields like location information, movement statemachine, quota, accounting data and more. We don't want to leak those as a service provider.
org workspaces are ClusterWorkspaces of type Organization. A clusterworkspacetype_organization controller is added and an admission plugin. The controllers bootstrap CRDs, and makes use of the initializer field of ClusterWorkspaces. The admission plugin makes the type field immutable and adds initializers defined by ClusterWorkspaceType objects. Providers of custom-workspaces can use that to have their own initialization flow.
the admin logical cluster is renamed to system:admin. In general, system:* logical clusters are supposed to be shard-local.
by default (through config/root) a root:default organisation is created on kcp startup. The default context in the .kcp/admin.kubeconfig points to root:default. In addition, there are cross-cluster(as before), root for the root workspace and system:admin for the former admin workspace.

@marun @ncdc addressed comments or replied above

LGTM. Switch to 'f' versions of require functions can be done separately (maybe once a ci check is available).


How to Mount USB/Micro SD on crouton?

I only have 32GB on my HP Chromebook 14 (AMD). I want to download games to play on the go without lugging around my huge laptop, but all "tutorials" never work. Any help would be great!

https://github.com/dnschneid/crouton/wiki/Access-external-drive-from-crouton

I have done that but It always says (drivename): is a read only partition. Even though the write protect is off.

send an image/screenshot of what it says


Diff blocks not appearing in Pull Requests for private repos

For some reason, diff blocks only appear in Pull Requesrs for public repos, but not for private repos. The side effect is that you cannot view different for private repo PRs.

Believe this was fixed somewhere along the way, works for me now.


Topbar cannot be allowed to horizontally overflow

The tabs/links/whatever they are in the grey thing on the header cannot be allowed to overflow onto the next line
Either reformat the grey bit so it can accommodate another row if need be, or just cut excess tabs off

This is fixed.


Switching between saved requests keeps old data form.

I open saved request A that has form data, Update some values, save it (overwrite), and send that request. Next I open request B, and instead of form data for request B I get form data from request A.
If I close ARC and reopen it and load request B, I get the correct form data.
Versions
App: <IP_ADDRESS>-stable
Chrome: 48.0.2564.97
Platform: Windows (7)

Hi,
Thank you for issue report.
I can't reproduce the problem. Is your directions accurate? Is it possible for you to send me request data for both requests? I'd like to imitate the same conditions on my environment and test it again.
Maybe you can see any error message in log viewer? (left nav > bottom buttons > More)

I have a video of the problem but I don't want to post it here. Is there an email address I can send it to?

Sure,
You can send it to<EMAIL_ADDRESS>I'll test it when I have a chance to do it.

Thanks for the movie and the issue report. It was fixed in latest release.


execution of "squid -z" missing on web cache activation

Hello,
Maybe I'm wrong, but it looks like execution of squid -z on cache activation is missing.
Thanks for your work.
Regards,
Renoow.

Hi Renoow,
It should be pulled in via setup.sh
https://github.com/opnsense/core/blob/master/src/opnsense/scripts/proxy/setup.sh
What issue are you seeing?
Cheers,
Franco

Hmm, it’s not run on restart/reconfigure, maybe that is your issue. Initial setup ok, cache add and not working?

Hi Franco,
When I check " Enable local cache (requires service restart) " squid restart hang.
/var/log/squid.syslog.log
OPNsense (squid-1): Failed to verify one of the swap directories, Check cache.log for details. Run 'squid -z' to create swap directories if needed, or if running Squid for the first time.
/var/log/squid/cache.log
2017/12/20 19:13:47 kid1| Squid Cache (Version 3.5.27): Exiting normally.
2017/12/20 19:13:48 kid1| Set Current Directory to /var/squid/cache
2017/12/20 19:13:48 kid1| Starting Squid Cache version 3.5.27 for amd64-portbld-freebsd11.0...
2017/12/20 19:13:48 kid1| Service Name: squid
2017/12/20 19:13:48 kid1| Process ID 46668
2017/12/20 19:13:48 kid1| Process Roles: worker
2017/12/20 19:13:48 kid1| With 57960 file descriptors available
2017/12/20 19:13:48 kid1| Initializing IP Cache...
2017/12/20 19:13:48 kid1| DNS Socket created at [::], FD 6
2017/12/20 19:13:48 kid1| DNS Socket created at <IP_ADDRESS>, FD 8
2017/12/20 19:13:48 kid1| Adding domain localdomain from /etc/resolv.conf
2017/12/20 19:13:48 kid1| Adding nameserver <IP_ADDRESS> from /etc/resolv.conf
2017/12/20 19:13:48 kid1| Adding nameserver <IP_ADDRESS> from /etc/resolv.conf
2017/12/20 19:13:48 kid1| Logfile: opening log stdio:/var/log/squid/access.log
2017/12/20 19:13:48 kid1| Unlinkd pipe opened on FD 13
2017/12/20 19:13:48 kid1| Local cache digest enabled; rebuild/rewrite every 3600/3600 sec
2017/12/20 19:13:48 kid1| Logfile: opening log stdio:/var/log/squid/store.log
2017/12/20 19:13:48 kid1| Swap maxSize 102400 + 262144 KB, estimated 28041 objects
2017/12/20 19:13:48 kid1| Target number of buckets: 1402
2017/12/20 19:13:48 kid1| Using 8192 Store buckets
2017/12/20 19:13:48 kid1| Max Mem size: 262144 KB
2017/12/20 19:13:48 kid1| Max Swap size: 102400 KB
2017/12/20 19:13:48 kid1| ERROR: /var/squid/cache/00: (2) No such file or directory
FATAL: Failed to verify one of the swap directories, Check cache.log
for details. Run 'squid -z' to create swap directories
if needed, or if running Squid for the first time.
Squid Cache (Version 3.5.27): Terminated abnormally.
Running "squid -z" manually and restarting squid solve the problem.
I've tried on two fresh installs and obtain same result each time.
Cordialement,
Renaud MEDICI
Inform'Ethic
8, Route de Saint Julien - 07200 Ucel
Tél. : +33 7 86 47 60 21
Fax : +33 9 55 24 01 54
De: "Franco Fichtner"<EMAIL_ADDRESS>À: "opnsense/core"<EMAIL_ADDRESS>Cc: "Renaud MEDICI"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Envoyé: Mercredi 20 Décembre 2017 20:57:10
Objet: Re: [opnsense/core] execution of "squid -z" missing on web cache activation (#2008)
Hi Renoow,
It should be pulled in via setup.sh
[ https://github.com/opnsense/core/blob/master/src/opnsense/scripts/proxy/setup.sh | https://github.com/opnsense/core/blob/master/src/opnsense/scripts/proxy/setup.sh ]
What issue are you seeing?
Cheers,
Franco
—
You are receiving this because you authored the thread.
Reply to this email directly, [ https://github.com/opnsense/core/issues/2008#issuecomment-353165405 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AXGwXZMFo0_86a3ty7NIndDX7F-FpIm1ks5tCWVNgaJpZM4RI1Wv | mute the thread ] .

Yes, local cache activation activated when web cache already activated.
Cordialement,
Renaud MEDICI
Inform'Ethic
8, Route de Saint Julien - 07200 Ucel
Tél. : +33 7 86 47 60 21
Fax : +33 9 55 24 01 54
De: "Franco Fichtner"<EMAIL_ADDRESS>À: "opnsense/core"<EMAIL_ADDRESS>Cc: "Renaud MEDICI"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Envoyé: Mercredi 20 Décembre 2017 20:54:57
Objet: Re: [opnsense/core] execution of "squid -z" missing on web cache activation (#2008)
Hmm, it’s not run on restart/reconfigure, maybe that is your issue. Initial setup ok, cache add and not working?
—
You are receiving this because you authored the thread.
Reply to this email directly, [ https://github.com/opnsense/core/issues/2008#issuecomment-353166168 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AXGwXVBZqw6VDdbLKME-ocpd75nTW-F2ks5tCWYRgaJpZM4RI1Wv | mute the thread ] .

Aha, should be an easy fix. Will ping you tomorrow morning :)

It looks like this is a problem in squid itself. If the service is running, squid -z -N says:
2017/12/21 06:49:36| Squid is already running!  Process ID 79343

and refuses to do the cache creation.

@renoow you can try this prior to starting to configure squid:
# opnsense-patch 25ca2ad 85c1e0a

However, the restart will always be necessary and I think manual stop + start already works.

There is more issues here, I'm circling around to fix this for 18.1.

This works now, but will not hit 17.7.x anymore.


Support building on "darwin" with golang.org/x/sys updated over 6fcdbc0bbc04dcd9e7dc145879ceaf9bf1c6ff03

https://github.com/golang/sys/commit/6fcdbc0bbc04dcd9e7dc145879ceaf9bf1c6ff03 has removed some "darwin" related code, so projects, which use higher versions of golang.org/x/sys can no longer be built for this platform.
Example:
cat <<EOF > go.mod
module foo

go 1.15

require (
        github.com/moby/term v0.0.0-20200611042045-63b9a826fb74
        golang.org/x/sys v0.0.0-20200828080924-6fcdbc0bbc04 // indirect
)
EOF

cat <<EOF > main.go
package main

import (
        "log"
        "os"

        "github.com/moby/term"
)

func main() {
        fd := os.Stdin.Fd()
        if term.IsTerminal(fd) {
                ws, err := term.GetWinsize(fd)
                if err != nil {
                        log.Fatalf("term.GetWinsize: %s", err)
                }
                log.Printf("%d:%d\n", ws.Height, ws.Width)
        }
}
EOF

GOOS=darwin go build

Results in:
# github.com/moby/term
../../../go/pkg/mod/github.com/moby/term@v0.0.0-20200611042045-63b9a826fb74/tc.go:13:28: undefined: unix.SYS_IOCTL
../../../go/pkg/mod/github.com/moby/term@v0.0.0-20200611042045-63b9a826fb74/tc.go:18:28: undefined: unix.SYS_IOCTL
../../../go/pkg/mod/github.com/moby/term@v0.0.0-20200611042045-63b9a826fb74/termios_bsd.go:24:31: undefined: unix.SYS_IOCTL
../../../go/pkg/mod/github.com/moby/term@v0.0.0-20200611042045-63b9a826fb74/termios_bsd.go:37:31: undefined: unix.SYS_IOCT

Note: the same now happens for older versions of github.com/docker/docker or github.com/moby/moby, but the latest master already depends on this code here.

I have same issue :(

Putting replace golang.org/x/sys => golang.org/x/sys v0.0.0-20200826173525-f9321e4c35a6 in go.mod is a workaround for it BTW. It pins the sys version to commit right before the breaking change.

Thanks for the quick fix!

Will be fixed by #16.

#16 is merged, so I guess this can be closed?

I'll close.


Kubelet publish node components' version

Forked from #4855, and required for rolling upgrade:
Kubelet should report cross-products version, such kernel, docker, kubelet, kube-proxy, image, etc. to API server.
cc/ @zmerlynn

So now that it's in NodeSystemInfo, is it queryable easily from kubectl at all?

+me

Awesome, thanks Dawn!


Limit three versions of KIS to KSP 1.7.0

Problem
See https://forum.kerbalspaceprogram.com/index.php?/topic/154922-ckan-the-comprehensive-kerbal-archive-network-v1260-baikonur/&do=findComment&comment=3613097
Some recent versions of KIS are the victims of unexpected stock API changes. They don't work on KSP 1.7.1, but their version files and CKAN metadata announce compatibility with 1.7.0 through 1.7.99.
Changes
Now they're marked as 1.7.0 only.

Awesome! Thank's a lot for the fix.


Added click-parser snap

This change is

Comments from Reviewable

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Leonlu2/error on geo values

process about #641
Prerequisites:

[X] Unless it is a documentation hotfix it should be merged against the dev branch
[x] Branch is up-to-date with the branch to be merged with, i.e. dev
[x] Build is successful
[X] Code is cleaned up and formatted

Summary

add valid geo_value validator in _params
modified test_covidcast and test_delphi_epidata for server geo_value validation passing
help wanted and added TODO  in test files for patch and mock the return values of geo validator in server

looks great! very nice job!

thanks :)


docker build fails when trying to install pyshtools

Hi there, what a great project. Thank you!!
I am on an M1 MacbookPro and am trying to use Docker (version 20.10.11, build dea9396) to make the utils/multires/generate.py script work. When running docker build -t generate-panorama . as instructed, I get the following error from Docker (sorry, a lot of info):
 => ERROR [3/4] RUN pip3 install pyshtools                                                                                4.0s
------
 > [3/4] RUN pip3 install pyshtools:
#6 0.725 Collecting pyshtools
#6 0.853   Downloading pyshtools-4.9.1.tar.gz (37.7 MB)
#6 3.981     ERROR: Command errored out with exit status 1:
#6 3.981      command: /usr/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-1buygh4k/pyshtools/setup.py'"'"'; __file__='"'"'/tmp/pip-install-1buygh4k/pyshtools/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-1buygh4k/pyshtools/pip-egg-info
#6 3.981          cwd: /tmp/pip-install-1buygh4k/pyshtools/
#6 3.981     Complete output (59 lines):
#6 3.981     *** pypandoc is not installed. PYPI long_description will not be formatted correctly. ***
#6 3.981     INSTALLING SHTOOLS v4.9.1
#6 3.981     get_default_fcompiler: matching types: '['gnu95', 'intel', 'lahey', 'pg', 'absoft', 'nag', 'vast', 'compaq', 'intele', 'intelem', 'gnu', 'g95', 'pathf95', 'nagfor']'
#6 3.981     customize Gnu95FCompiler
#6 3.981     Could not locate executable gfortran
#6 3.981     Could not locate executable f95
#6 3.981     customize IntelFCompiler
#6 3.981     Could not locate executable ifort
#6 3.981     Could not locate executable ifc
#6 3.981     customize LaheyFCompiler
#6 3.981     Could not locate executable lf95
#6 3.981     customize PGroupFCompiler
#6 3.981     Could not locate executable pgfortran
#6 3.981     customize AbsoftFCompiler
#6 3.981     Could not locate executable f90
#6 3.981     Could not locate executable f77
#6 3.981     customize NAGFCompiler
#6 3.981     customize VastFCompiler
#6 3.981     customize CompaqFCompiler
#6 3.981     Could not locate executable fort
#6 3.981     customize IntelItaniumFCompiler
#6 3.981     Could not locate executable efort
#6 3.981     Could not locate executable efc
#6 3.981     customize IntelEM64TFCompiler
#6 3.981     customize GnuFCompiler
#6 3.981     Could not locate executable g77
#6 3.981     customize G95FCompiler
#6 3.981     Could not locate executable g95
#6 3.981     customize PathScaleFCompiler
#6 3.981     Could not locate executable pathf95
#6 3.981     customize NAGFORCompiler
#6 3.981     Could not locate executable nagfor
#6 3.981     searching SHTOOLS in: build/temp.linux-aarch64-3.8
#6 3.981     fftw_info:
#6 3.981     customize UnixCCompiler
#6 3.981       libraries fftw3 not found in ['/usr/local/lib', '/usr/lib']
#6 3.981       fftw3 not found
#6 3.981     customize UnixCCompiler
#6 3.981       libraries rfftw,fftw not found in ['/usr/local/lib', '/usr/lib']
#6 3.981       fftw2 not found
#6 3.981     /usr/lib/python3/dist-packages/setuptools/dist.py:473: UserWarning: Normalizing 'v4.9.1' to '4.9.1'
#6 3.981       warnings.warn(
#6 3.981     Traceback (most recent call last):
#6 3.981       File "<string>", line 1, in <module>
#6 3.981       File "/tmp/pip-install-1buygh4k/pyshtools/setup.py", line 243, in <module>
#6 3.981         setup(**metadata)
#6 3.981       File "/usr/lib/python3/dist-packages/numpy/distutils/core.py", line 137, in setup
#6 3.981         config = configuration()
#6 3.981       File "/tmp/pip-install-1buygh4k/pyshtools/setup.py", line 194, in configuration
#6 3.981         fftw_info = get_info('fftw', notfound_action=2)
#6 3.981       File "/usr/lib/python3/dist-packages/numpy/distutils/system_info.py", line 444, in get_info
#6 3.981         return cl().get_info(notfound_action)
#6 3.981       File "/usr/lib/python3/dist-packages/numpy/distutils/system_info.py", line 696, in get_info
#6 3.981         raise self.notfounderror(self.notfounderror.__doc__)
#6 3.981     numpy.distutils.system_info.FFTWNotFoundError:
#6 3.981         FFTW (http://www.fftw.org/) libraries not found.
#6 3.981         Directories to search for the libraries can be specified in the
#6 3.981         numpy/distutils/site.cfg file (section [fftw]) or by setting
#6 3.981         the FFTW environment variable.
#6 3.981     ----------------------------------------
#6 3.999 ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
------
executor failed running [/bin/sh -c pip3 install pyshtools]: exit code: 1

The Docker container is not being built.
Any Idea? Maybe the Docker GitHub would be a better place to post this issue?

I was able to install the dependencies manually like so:
$ brew install<EMAIL_ADDRESS>$ brew install pillow
$ brew install numpy
$ brew install --cask hugin

Additional Steps

open and allow the Apps in /Applications/Hugin/
make nona available: $ export PATH="/Applications/Hugin/PTBatcherGUI.app/Contents/MacOS:$PATH"

But I still would like to know how to get it to work with Docker...

If I deactivate pyshtools like this:
# RUN pip3 install pyshtools

...the Docker build passes, but of course the shtHashes won't be generated.

The Dockerfile assumes that pyshtools will be installed as a binary Wheel. However, PyPI only has Wheels for x86_64, so installation on Arm requires building from source, which requires additional dependencies. Per the pyshtools documentation, adding gcc gfortran libfftw3-dev libblas-dev liblapack-dev to the end of the apt-get install line should install the dependencies required for building from source.

WOW @mpetroff , that worked!!!
I changed my Dockerfile as follows:
FROM ubuntu:20.04

# docker build -t generate-panorama .
# docker run -it -v $PWD:/data generate-panorama /data/image.jpg

ENV DEBIAN_FRONTEND noninteractive
RUN apt-get update && apt-get install -y --no-install-recommends \
    python3 python3-dev python3-numpy python3-pip python3-pil hugin-tools \
    gcc gfortran libfftw3-dev libblas-dev liblapack-dev \
 && rm -rf /var/lib/apt/lists/*
RUN pip3 install pyshtools --no-binary pyshtools

ADD generate.py /generate.py
ENTRYPOINT ["python3", "/generate.py"]

Thank you so much for your fast help!

I guess the ideal solution would be to check if a binary wheel exists for pyshtools for the architecture Docker is being run on. If it exists, the wheel should be installed. If not, the additional distro packages should be installed, and pyshtools should then be built from the source package. Unfortunately, there doesn't seem to be a good way to check this with pip. If pyshtools started uploading binary wheels for Arm, that would also solve this issue.

Hi @mpetroff , I tried @hirasso solution on my mac m1, and getting the following error:
Collecting pyshtools
Downloading pyshtools-4.11.10.tar.gz (41.5 MB)
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'done'
Installing backend dependencies: started
Installing backend dependencies: finished with status 'done'
Preparing wheel metadata: started
Preparing wheel metadata: finished with status 'error'
ERROR: Command errored out with exit status 1:
command: /usr/bin/python3 /tmp/tmp4ewa_eo9 prepare_metadata_for_build_wheel /tmp/tmp7uyhowwi
cwd: /tmp/pip-install-k33pyqxz/pyshtools
Complete output (30 lines):

meson setup /tmp/pip-install-k33pyqxz/pyshtools /tmp/pip-install-k33pyqxz/pyshtools/.mesonpy-4jxnlo60 -Dbuildtype=release -Db_ndebug=if-release -Db_vscrt=md --default-library=static --native-file=/tmp/pip-install-k33pyqxz/pyshtools/.mesonpy-4jxnlo60/meson-python-native-file.ini
The Meson build system
Version: 1.5.1
Source dir: /tmp/pip-install-k33pyqxz/pyshtools
Build dir: /tmp/pip-install-k33pyqxz/pyshtools/.mesonpy-4jxnlo60
Build type: native build
Program python3 found: YES (/usr/bin/python3)
WARNING: You should add the boolean check kwarg to the run_command call.
It currently defaults to false,
but it will default to true in future releases of meson.
See also: https://github.com/mesonbuild/meson/issues/9300
Project name: shtools
Project version: 4.11.10
Fortran compiler for the host machine: gfortran (gcc 9.4.0 "GNU Fortran (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0")
Fortran linker for the host machine: gfortran ld.bfd 2.34
C compiler for the host machine: cc (gcc 9.4.0 "cc (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0")
C linker for the host machine: cc ld.bfd 2.34
Host machine cpu family: x86_64
Host machine cpu: x86_64
Did not find pkg-config by name 'pkg-config'
Found pkg-config: NO
Did not find CMake 'cmake'
Found CMake: NO
Run-time dependency openblas found: NO (tried pkgconfig and cmake)
Run-time dependency openblas found: NO (tried pkgconfig and cmake)
Run-time dependency blas found: NO

../meson.build:15:9: ERROR: Dependency lookup for BLAS with method 'pkgconfig' failed: Pkg-config for machine host machine not found. Giving up.

@cyango I think you ran into the same issue that is described in #1205.
I just made some updates to the Dockerfile in 9beb4184e34dc0073ab35bbebf0eba9eac1d4e86 and badf8812c39aaf515e6a8aa2f5f034331b33aa18, and with these updates, it will hopefully just work.

It fixed it, thanks!


RBAC scoped to ADLS Gen2 Filesystem?

"You can assign RBAC roles scoped to a subscription, resource group, storage account, or an individual filesystem to a security principal or a managed identity for Azure resources. "
The Azure Portal does not provide an "Access Control (IAM)" pane for individual ADLS Gen2 Filesystems.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d5853661-e411-4030-9b7a-04b460a5f1c5
Version Independent ID: 3d89497f-a123-09bd-6c2e-da5ba663d563
Content: Azure Data Lake Storage Gen2 Storage security guide
Content Source: articles/storage/common/storage-data-lake-storage-security-guide.md
Service: storage
GitHub Login: @roygara
Microsoft Alias: rogarana

@cg6 Thank you for your query . We will investigate and update further.

@cg6 You can now change the owner and owning group when managing access for an ADLS Gen2 container, file, or folder using Azure Storage Explorer.
Note that the version of Storage Explorer that is embedded into the Azure Portal does not currently support viewing or managing Data Lake Storage Gen2 accounts with the hierarchical namespace feature enabled.
To view or manage Data Lake Storage Gen2 accounts by using Azure Storage Explorer, you must have at least version 1.6.0 of the tool which is available as a free download.
For more details, refer “Known issues with Azure Data Lake Storage Gen2”.
Hope this helps.

Hi, my comment was referring to Azure RBAC, not ACL's on objects in the
filesystem.
Databricks appears to require at least Storage Blob Reader to mount a Gen2
filesystem. I was looking to apply this role to a specific filesystem
rather than the whole storage account.
On Wed, 20 Mar. 2019, 17:03 CHEEKATLAPRADEEP-MSFT<EMAIL_ADDRESS>wrote:

@cg6 https://github.com/cg6 You can now change the owner and owning
group when managing access for an ADLS Gen2 container, file, or folder
using Azure Storage Explorer.
Note that the version of Storage Explorer that is embedded into the Azure
Portal does not currently support viewing or managing Data Lake Storage
Gen2 accounts with the hierarchical namespace feature enabled.
To view or manage Data Lake Storage Gen2 accounts by using Azure Storage
Explorer, you must have at least version 1.6.0 of the tool which is
available as a free download
https://azure.microsoft.com/en-us/features/storage-explorer/.
For more details, refer “Known issues with Azure Data Lake Storage Gen2
https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-known-issues
”.
Hope this helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/27576#issuecomment-474711216,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7o-sEe7HY_HlQ7ElhCK8D3n7xmY3csks5vYd0vgaJpZM4b-W3f
.

@cg6 You may follow the below steps to work with Azure Databricks and ADLS Gen2:
•	Assign Blob Reader role the Application ID you use for Databricks at IAM in Storage Account setting portal
•	Use Storage Explorer to give object id (remember object id not Application id, you can get it using az ad sp show --id your_application_id) appropriate read/write/execute access to parent and sub folder. For example, you can give read access to parent folder and give write access to a particular subfolder
Hope this helps.

@cg6 We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

Hi @CHEEKATLAPRADEEP-MSFT, is it possible to assign the permission to object id using AzCopy (or any PS module)? I was using ".\azcopy make "https://account.blob.core.windows.net/container-name"" to create a filesystem for Adls Gen2, which works fine but I also need to manage the permission policies using a PowerShell script. Thanks!

@stanupab Currently, you can set file and directory level permissions using Azure Storage Explorer with Azure Data Lake Storage Gen2.
Refer: https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-how-to-set-permissions-storage-explorer
Hope this helps.


Add loaders

Add simple loaders satisfying the theme of the app at the loading instances. This can be done using CSS and/or JS as per your convenience.

I would like to work on this issue. can i?

Sure! Go ahead!

Hello Sir, I would love to work on this issue.

@sanyam-aggarwal Go ahead!

Hey! Is this issue still up for grab?
I want to have a go at it. Please let me know if I can take this up.

No i am not working on this issue. If you want to give it to someone else
you can.
On Thu, Jul 2, 2020, 12:58 PM ritwik9857<EMAIL_ADDRESS>wrote:

Hey! Is this issue still up for grab?
I want to have a go at it. Please let me know if I can take this up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/chatbot/issues/2#issuecomment-652835914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMUCWBK5CPND2UI3JMS5XITRZQZJ7ANCNFSM4JTSM7FA
.

Go ahead, @ritwik9857 👍

Hi @rexagod
I installed the project in my local system but when I am running it is showing me an
Unhandled Rejection (TypeError): Cannot read property 'session' of undefined
at ../chatbot/src/App.js:206
Can you suggest anything here why is this coming. I can send the screenshot of the code file also if you want


Fixing recent issues #34, #35, #36;

PR overview
This PR includes:

Fix for Android ObjectDisposedException: Cannot access a disposed object on BottomNavigationView (issue #34 and issue #36);
Fix for Android badge display when Shell's TabBarBackgroundColor="White" (issue #35).

What's changed

Android platform's shell item renderer.

Actions required

Nuget package needs to be updated;
Sample project needs to be updated.

Package has been updated to  v1.0.20


Create isEmptyOrInvalid helper function

This helper has as purpose to validate and null-check structuredText fields returned from DatoCMS.
Situation

An admin decides to not fill in a certain structuredText field.
The returned value through GraphQL will be paragraph with an empty .
On the front-end you will end up with  which could extra invisible margins in some cases.

Hey! We've just added a similar method called isEmptyDocument in 2.0.4
https://github.com/datocms/structured-text/commit/7ab60ee323ee03c53a171becb47ef4cdad74fc8d


test(client): created unit test suite for OfflineWarning component

Created unit tests for OfflineWarning component.
Checklist:

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

Closes #XXXXX (NA)

I created a test suite with 5 unit tests for the Offline Warning component.

@huyenltnguyen thank you for the feedback! I've implemented your suggestions into the pull request.


[ALLUXIO-2574] Decouple Control Flow from Exception Logging

https://alluxio.atlassian.net/browse/ALLUXIO-2574

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13601/
Test PASSed.

@jsimsa For the second suggestion, I think we should add a utility (ie. V CallAndLog(Callable<V> callable)) which works with Java's Callable class. All our current Callables can extend Java's callable.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13618/
Test PASSed.

@jsimsa I would like to merge this one as-is. I've created https://alluxio.atlassian.net/browse/ALLUXIO-2577 to track the logging module consolidate and I plan to do that change in a separate PR.


Member images aren't loading on the "Member list" page

General Checkups

[x] Have you checked that there isn't already an existing issue that describes what you report below?
[x] Have you checked that there isn't already an open pull requests for this issue/update/change?

Description

The "Member list" page is not loading member images:

I see the image URL gives an "access denied" error:
https://open-humans-production.s3.amazonaws.com/member/mpball/profile-images/portrait_20140925.png
I seem to recall that we used a thumbnail manager with its own set of template tags to manage these images, but that seems to be a direct URL...

Ah, yes, I think that's the issue. In member-detail-content.html:
    {% if member.profile_image %}
    {% thumbnail member.profile_image '250x250' as image %}

    {% endthumbnail %}
    {% endif %}

vs. member-list.html:
        {% if member.profile_image %}

        {% else %}

        {% endif %}

That's weird, some of them are.
I removed the thumbnailer because it was adding a significant amount of processing time, causing the page to take a very long time to render, and occaisionally time out.

Whether they load or not may depend on when the user uploaded their profile image, if the S3 permissions were updated to be non-public by default (or vice versa, if old ones work and new ones don't work).

You could use the thumbnailer again if you added a scheduled task in Heroku to ensure the cache was always warm... (The improvement there would be the smaller amount of network data sent on page load, since the member list page is displaying the full-size image now)

Fixed.


Web, Anonymous mode - Login menu does not show

Noticed this started happening after upgrading to 18.2
Not sure what is going on and where to look. Browser console does not show anything suspicious.
Example what I am talking about:

I managed to repro it in SecurityTester solution after switching to New Web Style. It looks like the new style is getting more mature and does not accessing an object without read permissions. You need something like this for your anonymous role.
anonymousRole.AddObjectAccessPermission(SecuritySystem.UserType, "[Oid] = CurrentUserId()",SecurityOperations.ReadOnlyAccess);

Installer lab build 18.2.301.21 includes commit Web, Anonymous mode - Login menu does not show #262 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Excellent, thanks Tolis.

you still need to adjust permissions manually for your existing projects. The patch fixes the SecuritySystemExtensions

Apologies, used AddObjectAccessPermission instead of AddObjectPermission for XpandPermissionPolicyRole

hi tolis,
in 18.2.301.27 version i can't click anymore login popup

I didnt have time to come back to this but for me latest builds didnt resolve this either. A bit stuck on how to proceed with this.

@skybluex3 did you manage to fix this?

Yeah, i restored css it has been fixed

I my case didnt work due to configuration

Closing for now.


Design API(s) needed to support GUI

Description
The frontend needs an endpoint that lists allowed "next steps" to be available to navigate to.
Based on the authorization the list could only contain processes the user should be able to use from the given process/view.
Spec
From an API for example api/v2/allowedNextSteps, the httpclient will get an array similar to this:
[
   {
     "processId":  "form",
      "allowed": true,
      "otherAttributes": "may need to have more complexity later"
   },
   {
     "processId":  "submit",
      "allowed": false,
      "otherAttributes": "If the process is not allowed it might not be needed at all in this array"
   }
]

Considerations
The formLayout can be the source of how the actual rendering of the available steps is done.
The app-developer can define all the navigation in the formLayout given if all possible steps where available.
Therefore the need of sending more than the actual available steps is unnecessary.
If the formLayout definition can't be found in the API-response it can simply be ignored.
@TheTechArch do you have enough to get started on the backend from this?

@TheTechArch I'm not sure if this issue might be better being moved to one of the backend-repos?


arrondissement facet counts on member page never change

Fixes #629

Codecov Report

Merging #649 into develop will decrease coverage by 0.05%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           develop     #649      +/-   ##
===========================================
- Coverage    98.09%   98.04%   -0.06%
===========================================
  Files          218      218
  Lines        11599    11606       +7
  Branches        63       66       +3
===========================================
+ Hits         11378    11379       +1
- Misses         221      227       +6


Updated Viewcontroller by cleaning notices

Before, when you clicked in the button, the notice layers were being added on the top of each other.
Now, the notices are cleared before showing a new one

Sorry I can't merge this PR, because the piles of notices is on purpose :-D


Cannot traverse DeferredStructModel

Calling context.nrt.incref/decref on a deferred type raises exceptions. The root cause seems to be that DeferredStructModel cannot be traversed.
from llvmlite import ir
from numba import cgutils, types
from numba.datamodel import models, default_manager, register_default

class DeferredArrayType(types.DeferredType):
    pass

deferred_array_type = DeferredArrayType()
deferred_array_type.define(types.Array(types.float64, 1, 'C'))  # <- actual type should not matter
register_default(DeferredArrayType)(models.DeferredStructModel)

block = ir.Function(  # there might be a simpler way to create a block
    ir.Module(), ir.FunctionType(ir.DoubleType(), ()), 'dummy'
).append_basic_block()
builder = ir.IRBuilder(block)

dm = default_manager[deferred_array_type]
deferred_array = ir.Constant(dm.get_value_type(), None)

for mtyp, getter in dm.traverse(builder):
    print(getter(deferred_array))  # <- raises TypeError

fails with the following exception raised:
Traceback (most recent call last):
  File "<module1>", line 23, in <module>
  File "<path_to_numba>\numba\numba\datamodel\models.py", line 1340, in <lambda>
    lambda value: builder.extract_value(value, [0]))]
  File "<path_to_venv>\lib\site-packages\llvmlite\ir\builder.py", line 911, in extract_value
    instr = instructions.ExtractValue(self.block, agg, idx, name=name)
  File "<path_to_venv>\lib\site-packages\llvmlite\ir\instructions.py", line 619, in __init__
    typ = typ.elements[i]
TypeError: 'NoneType' object is not subscriptable

@asodeur thanks for reporting this! I can confirm that I can reproduce this locally.

Why are you trying to subclass from DeferredType?

DeferredType is an implementation detail in handling cyclic references between types. It is to provide a kind of forward declaration.
The datamodel for a deferred type is a IdentifiedStructType in llvm.  To traverse over it, it must be defined (not an opaque type anymore).  To resolve this issue, one can put a self._define in https://github.com/numba/numba/blob/60d2bddd0a9ddf84cf900a70e4b7e17bcbe5dc87/numba/datamodel/models.py#L1338.

Closing this as the original question appears to be answered and there have been no further updates for over a year.


Fix(docs/installation): Small typo in markdown in installation section

Was reading up a bit of Coolify's documentation and noticed a small typo in the Markdown.
Before:

After:

Thank you for the PR!