Kitematic window not showing OSX

Expected behavior
Open the application from either the Mac Docker app in the toolbar or from Apps folder and Kitematic will show
Actual behavior
App opens and the main App Menu appears in the normal location at the top of the screen but no application window is showing.
Information about the Issue
Kitematic has been working for a few weeks without any problem. But now there is no main app window.  I've tried Bring to Front, Close Windows, Toggle Chrome Dev tools - and I still can see any app window
Steps to reproduce the behavior

...Open Kitematic
...there's no window to see the app in

I have the same... tried reinstalling and zapping preferences in the Library folder but window does not show up.
Note: switched to external monitor when problem began.
I also would like to mention the Kitematic version but the About window is not displayed either.

Does anyone have a solution for this? Experiencing the same issue. Have removed preferences, reinstalled, rebooted, tried various things that normally bring offscreen windows onscreen, but no joy.

FOUND IT!!
This works best when you have the default blue desktop color on your mac.

Open Kitematic
Choose 'Hide others' from the Kitematic menu
Now choose Window > Minimize. You should see 'something' disappearing.
Choose Window > Kitematic.
Go over the bar with your mouse and resize it... It was a very small Kitematic window!

Hopefully this works for you too!

This worked - for me it was a tiny vertical bar on the left of the screen.


SFTP Progress not supressable

I can not disable the progress display for SFTP.
I have tried setting $ProgressPreference to SilentlyContinue to no avail (it has worked elsewhere in my scripts).
I can see that for Set-SCPFile you have a -NoProgress switch, but that is not available for Set-SCPFile

I've been trying to suppress this as well.  $ProgressPreference does not work. If Set-SFTPFile could get  a -NoProgress switch too, that would be fantastic.


[Diagnostics] Diagnose key path referring to init method

Addresses a diagnostic FIXME where key paths can't refer to init methods.
cc @xedin @hborla :)

@swift-ci please smoke test


Enable -Prelease for maven build by default

Alluxio Version:
2.5.0-SNAPSHOT
Describe the bug
-Prelease should be on by default or the output client jar may have issues to compile depending projects like Presto, due to the lack of reduced pom file
To Reproduce
build Alluxio without -Prelease
then build Presto using the Alluxio version built above
Expected behavior
It should pass
Urgency
Medium
Additional context
When turning on -Prelease, there should be only 1 working thread or the workers may hang

@Xenorith  is this still valid? otherwise help me close this please

current scenario: all devs can compile, but local compilation of jars + using compiled jars for presto causes issue
"fixed" scenario: local compilation of jars can be used for presto compilation, but devs that enable multiple threads for compilation will encounter a hanging process
the problem with the "fixed" scenario seems more likely to occur, and i'm not aware of any way to avoid it (ex. disable release profile if multiple threads are used), so i think we should not perform this fix.


ModPreview: Non-English mod title/desc

#26 -2 is implemented.
#26 -1 is still being worked on.
Steam's API is only returning English.
I've contact Steam support, so when this can be figured out, it shall be implemented.

Update: I've found where Steam has been hiding the localised texts, and figured out their seemingly undocumented mapping.
As of V3, this issue will be closed.


App crash on build, but works very well on Simulator...

2 days that I stuck on this... Build is SUCCESS, but when I open the app >> error
I have this...
WARNING: Logging before InitGoogleLogging() is written to STDERR
E0322 21:12:46.280941<PHONE_NUMBER> NativeToJsBridge.cpp:159] Attempting to call JS function on a bad application bundle: RCTLog.logIfNoNativeHook()
2019-03-22 21:12:46.284 [fatal][tid:main] Unhandled JS Exception: Unexpected token '{'. import call expects exactly one argument.

$ react-native info
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.1
      CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
      Memory: 605.42 MB / 8.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 11.10.0 - /usr/local/bin/node
      Yarn: 1.13.0 - /usr/local/bin/yarn
      npm: 6.7.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.0 => 0.59.0

Hey @steffi3rd, I suspect that there's actually an issue in your project's code. I think if you look at Metro, you'll see a build-time error.
A few tips:

Try resetting all of the caches with:

 watchman watch-del-all && rm -rf $TMPDIR/react-native-packager-cache-* && rm -rf $TMPDIR/metro-bundler-cache-* && rm -rf node_modules/ && npm cache clean --force && npm install && npm start -- --reset-cache

Go ahead and upgrade to 0.59.1

Without a repository of your source, it's hard to give specific advice. This seems like an environment issue, but please do follow up with more detail if my pointers above don't help.

Same Issue,
i was working with Expo version 2.11.x, and it was working fine,
then i upgraded to version 2.13.0 and i tried to run my app again, The build was successful but red screen with error,
and now it is not  running, Some tintColor error.
The release expo-cli 2.13.0 did not work for me.
i rolled back to 2.11.3 and it started working again.

Same as mine, but I don't use Expo but React Native CLi

@turnrye ,
Is this issue being watched ?
any updates ?

Hello there 👋 by using the repro / the list of steps to reproduce, we can't actually see the reported bug on RN 0.59.5 so we'll close this issue. It may have something to do with your local setup, please double check that you are using the correct version of Android Studio / XCode / Android NDK / Yarn / etc.


Add Unfolded Data SDK Py Jupyter notebook

see: https://docs.unfolded.ai/data-sdk/use-with-python

running it in vscode with Jupyter notebooks extension:


GUAC-800: Migrate to JSON for protocol descriptions.

This change:

Rewrites the old protocol XML as JSON.
Uses full Form and Field instead of Parameter for protocol descriptions, thus allowing logical sections (much cleaner).

This is not quite obviously relevant to GUAC-800, but it's a necessary step before exposure of user attributes can be rendered sanely within the UI. Those attributes will need similar support for logical sections.

Before

After


Verilog and SystemVerilog grammars need to be updated

Hello,
I just came across this project and noticed that it uses parsers generated by ANTLR4 from the grammars here.
I had manually written those grammars and they had many issues.
I've since updated the grammars (auto-generated from standard documents this time) and created a pull request. You can use the ones here until the changes are merged into grammars-v4.
If you experience any issue with the grammar, please let me know.
Thanks.

Hi,
Sorry for the late response.
I am using those grammars only to parse headers and interfaces, not implementations. As of today, they seem to work. I did not meet any problem whatsoever.
Updating those grammars is a lot of work due to the way parsing is implemented inside Dovado.
In case an update will be needed I would highly prefer to keep using grammars-v4 and not switch to any other repository because of the inherent complexity of the implementation
Please consider filing a pull request if you think the new grammars may improve the project.
I am sorry but documentation is greatly lacking this being an academic project.
Best,
Daniele


Unexpected invocation to startLinearAdMode

Hi, there is a bug with the PlayerWrapper class.
The bug occurs when you try to load a pre-roll ad that takes more than the time specified in the prerollTimeout setting to load.
The problem is that when the adtimeout event is triggered and then the adsready event is triggered, the pre-roll ad is displayed but UI is not updated to reflect that. For example, the vjs-ad-playing is not added to the player and the content playback is not resumed after pre-roll is ended.
The bug is caused because in the method onAdBreakStart the videojs-contrib-ads startLinearAdMode function is called, even if the adtimeout event was fired before.
This causes a WARN: 'Unexpected startLinearAdMode invocation (Preroll)', so this is an unexpected behavior for videojs-contrib-ads.
An easy way to reproduce this is set prerollTimeout in contribAdsSettings to a small value like 10.

Hi,
Were you able to reproduce the issue using vastLoadTimeout field of the videojs-ima plugin? Can you also provide the ad tag and the sample app you were using when you encountered the issue?

I am having the issue using any placement Id from AppNexus ad server.
An important thing that I didn't mention before is that I'm using autoplay feature by using videojs .play() method on player ready event:
myVideoJsPlayer.ready(function () {
  const playPromise = myVideoJsPlayer.play();
  // do things with the promise
});

I want to be emphatic in that problem only occurs when adtimeout event arrives before adsready event.

Hi,
Thank you for providing additional insights with regards to the issue. As the issue is with the videojs-contrib-ads plugin, I would recommend you to report it on their github issue tracker.

Nope, the bug is here because they say that you should call startLinearAdMode function only in specific scenarios and this project is not checking that scenarios.
According to videojs-contrib-ads plugin integrator docs:

startLinearAdMode() (METHOD) – Invoke this method to start an ad break.

For a preroll ad, you can invoke startLinearAdMode after the readyforpreroll event if isWaitingForAdBreak() is true.
For a midroll ad, you can invoke startLinearAdMode during content playback if isInAdMode() is false.
For a postroll ad, you can invoke startLinearAdMode after the readyforpostroll event if isWaitingForAdBreak() is true.

That validations are not present in PlayerWrapper class.

Hi
Can you share a screenshot and a walking skeleton of the sample app you were using when you encountered the issue?

I'm having this same issue which I reported back on 3/6. Christian has done an exceptional job of documenting this problem with detailed descriptions of where it's happening and what is causing it so I don't understand how a screenshot will provide additional value. This is causing problems for a lot of publishers and needs to be fixed. Has Sean or Yuri had a chance to review this?
Thanks,
D

Hi,
The screenshot is to be sure that we're testing the same issue and we need a walking skeleton of the project capable of reproducing the behavior so that we can investigate further.
I'm asking these info cause it seems that I need to modify our sample app to reproduce the behavior.

I think that main issue is different timeout in ima and contrib-ads. You cannot handle it then. Ima uses one timeout value, but contrib two. You should sync one value for both. When you set 10ms preroll on contrib-ads and ima uses default 8sec, then you are facing race condition..

We've been encountering the same bug. @cristian-atehortua has done a great job at tracking down why the problem is happening.
@cristian-atehortua any luck on your end solving this? We're looking at cutting a branch to dig into a solution.
For us, and I'm sure many publishers, this bug is mission critical. For publishers that use video-js in digital-signage-like products, this is causing a lot of headaches.

Hi, @GavinThompson. In my case I can fix my specific problem adding this workaround to my code. Check it out! I could be good for you, too.
/* Work around (started pre-roll after timeout event) */
const AD_PLAYING_CLASS = 'vjs-ad-playing';
const EVENTS = {
  AD_STARTED: 'ads-ad-started',
  ADS_READY: 'adsready',
  AD_TIMEOUT: 'adtimeout'
};

const onAdCompleted = () => {
  player.removeClass(AD_PLAYING_CLASS);
};

const onAdStarted = () => {
  player.addClass(AD_PLAYING_CLASS);
  player.ima.addEventListener([
    imaSdk.AdEvent.Type.COMPLETE,
    imaSdk.AdEvent.Type.SKIPPED
  ], onAdCompleted);
  player.off(EVENTS.AD_STARTED, onAdStarted);
};

const onAdsReady = () => {
  player.on(EVENTS.AD_STARTED, onAdStarted);
  player.off(EVENTS.ADS_READY, onAdsReady);
};

const onAdTimeout = () => {
  player.on(EVENTS.ADS_READY, onAdsReady);
  player.off(EVENTS.AD_TIMEOUT, onAdTimeout);
};

player.on(EVENTS.AD_TIMEOUT, onAdTimeout);
/* End work around */

@cdatehortuab you are the best, thanks for this!
Going to implement this afternoon and see how it works; this looks like a great solution. Thank you!

What is the problem that you have? The above solution only works around adding vjs-ad-playing class to player when the issue occurs. Maybe you can edit some things to make it work for your case.

Hi @cristian-atehortua ,
I set the prerollTimeout key to 10, added the code below to the advanced videojs-ima sample, no WARN: 'Unexpected startLinearAdMode invocation (Preroll)' message was logged in the console and the content playback resumed successfully. The behavior seems to be with your implementation. Can you provide a sample app capable of reproducing the behavior?
this.player.ready(() => {
    const playPromise = this.player.play();
    // do things with the promise
  });

I'm having this problem too.
It seems the problem does only occur when using autoplay and a HLS source. Using a .mp4 for instance did resume playback after the ad.
It also does not happen when using the default "preroll" ad tag included in the examples. I'm using a real one, but in test mode.
Also, when it takes too long to receive an ad, it triggers the adtimeout, however, if it receives the ad afterwards, it still calls the startLinearAdMode(), which for some reason does not resume the content after the ad ended.
If you increase the prerollTimeout, it stops triggering the Unexpected startLinearAdMode() error. And it triggers the Starting ad break and Ending ad break messages. However, in my case, it still does not resume the content.

I've managed to get it working in two ways.

Downgrading videojs-contrib-ads to version 6.0.0. I can confirm this problem started happening somewhere between 6.0.1 and 6.1.0.

I've also managed to get it working by adding a check on SdkImpl.prototype.onPlayerReadyForPreroll to check if the player is not in resuming state, as it seems videojs-contrib-ads ask for it now, and also implicitly calling a resume/play function on videojs-contrib-ads endLinearAdMode() functions.
However, this last option wouldn't play ads if they loaded after the player was already resuming (unless you increased your prerollTimeout). And it seemed to happen quite a lot, so I stick with the first method.

There was also a secondary problem which seemd to happen sometimes.
When the HLS was resuming from ads, which it seemed to get stuck on loading. The main playlist was being fetched, but not the segments.
I've managed to fix it by adding a play event on adend. As such:
player.on('adend', function(e) {
    player.play();
})`

Workaround 1:
Use https://cdnjs.cloudflare.com/ajax/libs/videojs-contrib-ads/6.0.0/videojs-contrib-ads.js instead.
Workaround 2 (recent versions):
In videojs.ima.js, change:
SdkImpl.prototype.onPlayerReadyForPreroll = function () {
  if (this.autoPlayAdBreaks) {

To
SdkImpl.prototype.onPlayerReadyForPreroll = function () {
  if (this.autoPlayAdBreaks && !player.ads.inAdBreak() && !player.ads.isContentResuming()) {

In videojs-contrib-ads.js, wherever there is a endLinearAdMode() function, add this to the end:
var _this = this;
  this.afterLoadStart(function () {
  _this.resumeAfterNoPreroll(player);
});

You may want to increase your prerollTimeout in player.ima options too.

Hi,
I’m closing this issue because it has been inactive for more than a month. Please reopen if you still encounter this issue with the latest stable version
Thank you!

Workaround 1:
Use https://cdnjs.cloudflare.com/ajax/libs/videojs-contrib-ads/6.0.0/videojs-contrib-ads.js instead.

OMG it works! I'm dealing with this issue (player doesn't play after ad) from years... any downside to this workaround?


firefly-iii-data-importer: init at 1.5.3

Description of changes
Added Firefly III Data Importer. Closes #32942

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Any blockers on this?

@SuperSandro2000 @drupol Do you guys think anything else is needed here?


Helper contract that closes a channel and moves it into a newer TokenNetwork

This issue keeps track of creating a contract that calls the old TokenNetwork to shutdown a channel and then calls the new TokenNetwork to create a channel and sets up the channel state there.

The first step is to make sure that the necessary functions are available in TokenNetwork contract.


Make initial location POP optional

Makes the logic added back in https://github.com/supasate/connected-react-router/pull/172/files optional
Can be called like
<ConnectedRouter history={history} noInitialPop>

Discussed in https://github.com/supasate/connected-react-router/issues/144

Would be very useful, and cater to both use cases

This looks great @dr3 !  Thanks for your contribution to the change, the unit test, and the doc!


Issue #6777 fixed

Issue Reference
#6777

Add css for search box place holder  in arabic locale

@akhtarkhan1
Not fixed, the search icon should be on left side and the placeholder should be in the center of the search box, please find the screenshot Here

@akhtarkhan1
This is working fine, Please find the screenshot Here


Cb-7808 Flow events are missing from Structuctured events

We filtered the structured events to datalake and datahub only but before 2.25 the CB stored as stacks. The legacy resource type added

should this go to 2.25 ?

I think yes, it should go to 2.25, this is a regression in 2.25 and it worked well in 2.24.

retest this please


fixed #listview.js

listview.js line 250
      if ( typeof colref.resizable === 'undefined' || colref.resizable === true ) {
        if ( i < (l-i) ) { //err
          resizer           = document.createElement('div');
          resizer.className = 'Resizer';
          label.appendChild(resizer);
        }
      }
      //change i to 1
      if ( typeof colref.resizable === 'undefined' || colref.resizable === true ) {
        if ( i < (l-1) ) { //ok
          resizer           = document.createElement('div');
          resizer.className = 'Resizer';
          label.appendChild(resizer);
        }
      }

@zqqq Thanks. I think I might have been slightly intoxicated when writing that. I uncovered a completely separate bug related to this that needs to be fixed first for this to work (Invisible columns are counted and that makes i < (l-1) always true)
I am going to look into the column resize some more because it does not behave nicely. Like when you resize the "MIME" column in File Manager the column moves to the left and expands at the same time. It should only expand.


Code blocks in Wiki are rendered with arbitrary syntax highlighting instead of none

Gitea version (or commit ref): 1.11.5
Can you reproduce the bug at https://try.gitea.io:

[X] Yes: https://try.gitea.io/qwertfisch/markdown-test/wiki/Code-blocks
[ ] No
[ ] Not relevant

Description
When adding a code block on a page in the built-in wiki, but with no language given for syntax highlighting, this block will be rendered using an arbitrarily chosen highlighting. Code blocks without selected syntax I will set like this:
```
This is simple text rendered with fixed-width font,
but no syntax highlighting.
```

Even if the text inside the block would contain keywords for some languages, I do not expect them to be highlighted or rendered other than as fixed-width font. In fact when given no language for the code block, I expect the language to be None.
Actual result
When selecting the preview window, or after storing the page, a second later a seemingly arbitrarily chosen syntax highlighting is applied to the block. On the example page I added some keywords (for some languages), and the two blocks are rendered as Cal and Vim, respectively. It seems the highlighting is selected automatically given the content of the block.
Workaround
I can enclose the block with <pre> / </pre> instead of Markdown. Is there otherwise a possibility to have a non-syntaxed code block with Markdown syntax?
Edit: <pre> does not set the content to a fixed-width font, so this is unfortunately no solution.
Is this a bug for Gitea? Or should I file a bug on the Markdown interpreter used (goldmark?).

This is not the fault of goldmark.
Gitea uses highlightjs for syntax highlighting of code blocks and as far as I understand highlightjs will automatically attempt to detect the language.
It's probable that if you put plaintext after the backticks that would turn it off.

We should put the plaintext class on <code> to prevent hljs from autodetection.

An automatically selected highlighting is welcome – just not when it is not expected. I could not find a documentation of this feature before.
plaintext works and selects the plaintext syntax from hljs, thanks for the advice. Also any other name unknown for hljs would work and use a language-[name] class, which defaults to no highlighting.


Resolve actions with xstate.stop event

This got superseded by https://github.com/statelyai/xstate/pull/3126


Move overkiz.event to the end of the coordinator logic

@tetienne wouldn't it make sense to have this code at the end of our coordinator logic? This means that it won't interfere with our core functions at least.

What do you mean by interfere?

What do you mean by interfere?

To make sure that issues like this https://github.com/iMicknl/ha-tahoma/issues/629, won't disable core functionality (like state etc.).

It should not happen anymore with my fix. But OK.

It should not happen anymore with my fix. But OK.

I agree! However, it just made me think that we should try to have our most important logic before other logic.


roboticscape device tree breaks usb network autoconfiguration in Linux

When connecting to a beaglebone black or green after loading the robotics cape device tree the usb network auto-configuration (both rndis and cdc-ether) is broken, and requires a static IP to be set on the host computer.  It would be nice if this wasn't required, and the network interfaces automatically were configured.

I observed this a long time ago but sadly have no idea why since I only enable hardware (eeqep, pwm, etc) in that device tree. Any help would be appreciated!
Thalaivar, yes, and I do have it installed on my computers. I added a 'make native' target to the library makefile so it will use the native compiler instead of building specifically for ARM. The examples always compile natively anyway. Try this:
cd Robotics_Cape_Installer/library/
make native && sudo make install
cd ../examples
make && sudo make install
Obviously the device tree and PRU code are worthless on a PC. But things like the math/timing/start_stop subsystems work, as will standard driver things like UART, I2C, GPIO, SPI

@RobertCNelson I put up a pull request to fix this on your boot-scripts repo here https://github.com/RobertCNelson/boot-scripts/pull/82
I copied the contents of am335x-boneblack.dts verbatim into am335x-boneblack-roboticscape.dts and DHCP goes back to normal. However changing ONLY the model name from "TI AM335x BeagleBone Black" to "TI AM335x BeagleBone Black RoboticsCape" breaks DHCP again so I knew to look for a startup script switch case and the above pull request seems to do the trick.


Fix: adds template for Memcached as a cacheStore

Fixes: https://app.zenhub.com/workspaces/commit-zero-5da8decc7046a60001c6db44/issues/commitdev/zero-backend-node/42

Adds Memcached to package.json
Adds touchmemcache functions in app.js
Creates folder for storing Memcached constants

Thanks for contributing!


Same question in different exams with different answers

In the below link, there is the same question, with different correct answer.
Can we fix it?
1. Which services can be used across hybrid AWS Cloud architectures? (Select TWO.)
    - A. Amazon Route 53
    - B. Virtual Private Gateway
    - C. Classic Load Balancer
    - D. Auto Scaling
    - E. Amazon CloudWatch default metrics

    Answer

    Correct Answer: AE

    Explanation:
    - <https://aws.amazon.com/cloudwatch/>
    - You can also use CloudWatch in hybrid cloud architectures by using the CloudWatch Agent or API to monitor your on-premises resources

https://github.com/kananinirav/AWS-Certified-Cloud-Practitioner-Notes/blob/master/practice-exam/practice-exam-13.md?plain=1#L156
43. Which services can be used across hybrid AWS Cloud architectures? (Select TWO)
    - A. Amazon Route 53.
    - B. Virtual Private Gateway.
    - C. Classic Load Balancer.
    - D. Auto Scaling.
    - E. Amazon CloudWatch default metrics.

    Answer
      Correct answer: A, B

https://github.com/kananinirav/AWS-Certified-Cloud-Practitioner-Notes/blob/master/practice-exam/practice-exam-9.md?plain=1#L442

The correct answers to this question are A and B. This is because AWS CloudWatch default metrics only measure metrics on AWS resources, not on On-Premises resources.

@LiorA1 Thanks I will update it. 🙏

https://github.com/kananinirav/AWS-Certified-Cloud-Practitioner-Notes/commit/dbfc6f748c372d35ab319d832ae74ac96c4b267f


Using document.location.href in config/environment.js incorrect.

The README as the following snippet:
/* jshint node: true */
// config/environment.js
module.exports = function(environment) {
  var ENV = {
    /* ... */
    torii: {
      // a 'session' property will be injected on routes and controllers
      sessionServiceName: 'session',
      providers: {
        'facebook-oauth2': {
          apiKey:      'facebook-app-id',
          redirectUri: document.location.href
        }
      }
    }
  };
  return ENV;
};

But that file is evaluated in the node context, and document.location.href will throw an error.
I'm happy to submit a pull request fixing, but I was unsure what to change it to...

I believe redirectUri: document.location.href is actually the [default behaviour of the oauth-2 adapter}(https://github.com/Vestorly/torii/blob/master/lib/torii/providers/oauth2-code.js#L87), so removing this should be fine.

Fixed in https://github.com/Vestorly/torii/commit/3ede6737a1ef48847966ba8b3b20ac78c8f42523 thanks @rwjblue  :-)


docker-multinode offline support

I'm working on modifying the scripts to make this possible, if anyone else is interested post here and I can push my modifications up. Once I get it working, I'll do a PR.
At the moment, my approach is to provide an offline.sh script that can download all the images from the internet, and optionally push them to a private registry, and then also pull them from a private registry and retag them as appropriate (due to https://github.com/kubernetes/kubernetes/issues/28189). The chief difficulty is enumerating all of the images required -- hyperkube downloads a variety of images when it starts. I'm hoping I can enumerate them all in a low-maintenance way -- but it still needs some work.

Migrated from kubernetes/kubernetes:

I'd like to keep the logic in common.sh at absolutely the minimum.
I see it as you have to transfer the images to the offline node in some way.
This might be achieved in a few ways:

On an internet connected node that's different from the offline node:

docker save  | ssh root@ "docker load"
docker pull ; docker save  > .tar.gz; scp .tar.gz user@
On the same node that should run docker-multinode
simply docker pull  in advance
I do not like the idea of patching images; that's clearly the wrong way to go; if Kubernetes setup get so complex that we have to modify (hack) images that contain "official" content, there should be a better official way, or we should try to see the problem from a new perspective.

I'll gladly take Pull Requests that targets the documentation around this; once we have documented common/useful ways to transfer images to nodes, things will get easier for users. So please change your PR to a docs PR in the first place, and from there we might continue.

Things will change a lot (I hope) in v1.4, Kubernetes will try to ease cluster setup issues/complexities. Follow sig-cluster-lifecycle for more info.

As mentioned in the other issue, my PR provides mechanisms that facilitate using the docker-multinode scripts in an offline manner. While I agree that keeping the contents of common.sh should be kept to a minimum, kubectl et al does not seem to provide the necessary functionality.
In particular, the difficult parts are:

Actually figuring out which images to pre-download -- there's a lot of work the PR needs to do
Pre-loading images onto the bootstrap docker daemon is annoying

Additionally, I think it should be a requirement to allow downloading from a private registry (as that PR allows) -- docker save/load over ssh is very slow compared to downloading from a registry. The PR provides this functionality -- and makes it very easy to use docker-multinode in an offline manner.

docker-multinode was removed in https://github.com/kubernetes/kube-deploy/pull/283 so I'm closing this issue as obsolete.


http connector - Allow to configure url as global configuration

So you can setup a base url for a http server and then maybe as the action add additional context path or something.
This requires some changes in the java code / connector to support this thing to assemble that into one httpUri option that the camel components uses.

This is a bit harder to do, as I need to build a custom http component for ipaas that has those global option to support this.

The test example now use global configuration of the http get/post connectors, so you only specify the last part of the context-path, eg hello / bye
        from("periodic-timer")
            .log("Timer is triggered")
            .to("http-get:hello")
            .log("GET: ${body}")
            .transform().constant("{ \"message\": \"iPaaS was here\" }")
            .to("http-post:bye");

However we need to store this global configuration in an easy way for the ipaas. Eg have some api like we have for writing the endpoint uri, but for the global options.
Or make it spring boot configuration per connector or something

I put them in spring boot now, but this is shared between get and post which you dont want
## global options of the get and post connector
camel.component.ipaas-http.scheme=http
camel.component.ipaas-http.hostname=localhost
camel.component.ipaas-http.port=8080
camel.component.ipaas-http.path=myservice


[Edit profile] User can not save allowed name in the Name field on User profile

Environment: Windows 10 Pro v.21H2, Google Chrome 109.0.5414.75 (64-bit).
Reproducible: always.
Build found: 19/01/23
Preconditions

The registered user is logged in.
'My habits' page is opened.

Steps to reproduce

Click the 'Edit' icon on the Profile block.
Enter allowed characters in the Name field.
e.g: Terter2023
Click on the 'Save' button.

Actual result
The name isn't saved on User profile. A pop-up"Oops, something went wrong. Please reload page or try again later." is shown.

Expected result
The name is saved on User profile.
Postconditions
User story and test case links
E.g.: "User story #1010
Test case
Labels to be added
"Bug", Priority ("High "), Severity ("Minor"), Type ("Functional").

Need to change back-end validator for profile name

@atsa21
Back-end validator for profile name will be changed. But I also found an incorrectness in the front-end. The hint for filling the field "Name" says that the dot can only be in the middle of the name and without repetition, for example Tester.123, but in this case the button "Save" is still disabled.

The following pattern replacement is planned for the backend:
^(?=.{6,30}$)([ЁёІіЇїҐґЄєА-Яа-яA-Za-z-'\d\s]+[.]?[ЁёІіЇїҐґЄєА-Яа-яA-Za-z-'\d\s]+)+$
For example these name variants will be allowed:

Tester
Tester123
Tester.123
Tester Testerov
tester.testerov 123

In fronted it will be fixed when we will merge this branch https://github.com/ita-social-projects/GreenCityClient/pull/2091
Check backend pattern that it allow to enter one letter name

@atsa21
Yes, I heard yesterday's discussion. Backend pattern allows to set the name from one character.


compiler  tag requires maven 3.6+

I was debugging a CI build failure, and I think the cause is the use of a new compiler tag that is only valid in maven version >= v3.6. Do we want/need that requirement now?
Assuming not, I removed the tag so older maven versions will still work.
(My CI build using an older maven version succeeded, so I'm pretty sure this PR fixes the issue).
Please ignore the unrelated commit comment below - side effect of merging upstream.

Hi Dan,
I was having problems getting groovy to compile on Java 11 when I added the tag.
I'll try it again without the tag to check it's not required.
Thanks,
Peter.

I have confirmed the tag isn't required.

Thanks Dan.


Feat: Password toggle button

Describe the feature
password toggling feature so that user can verify their entered password
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

go for it @Sourabh782

On it @RamakrushnaBiswal


General API design issues

NOTE: I will expand on some of the issues outlined below when I have more time.
It is particularly important that an API abstraction such as this has an API design that is as close as possible to "correct" from the outset, since it defines the structure of the code that uses it, and backwards-incompatible changes to an abstract API are even more difficult to manage than at the level of a concrete implementation.
I would like to raise a few issues I have with the current API:
Naming
These issues are all essentially down to unnecessary brevity, which will result in consuming code that is that little bit harder to work with. (N.B. this is a big fat IMHO)

ConnectionConfig#connectionString() should be getConnectionString() - method names need a verb. I realise this is effectively a property, but as we are currently stuck with using methods to implement these, the "rules" for method naming should be followed.
Link#transaction() should be beginTransaction(). Same basic reasoning as above, but more important in this case - the lack of a verb makes it slightly ambiguous, it's not possible to know about how this will behave from the name alone; it could be that if I call this when there is an existing transaction it will return a reference to the existing one rather than throwing an error, or queuing the request. Adding the verb removes the ambiguity and makes it easier to read the calling code.
Pool#getMaxConnections() is inconsistent with it's companion methods, which both use a ...Count() suffix in the names, It should be getMaxConnectionCount().

Interface segregation
The forgotten "I" in SOLID has been forgotten again. A number of interfaces explicitly define some unrelated operations, and even some functionally identical compatible methods are defined separately with no type relationship (todo: note examples)
The Operation interface
I suspect this interface doesn't make any sense in this abstraction layer. If it does make sense then there is definitely a naming problem.
As far as I am aware, the Operation interface is actually an implementation detail, essentially used to trigger code which frees underlying resources, and thereby avoid breaking encapsulation by exposing them directly. Both amp/mysql and amp/postgres have a matching interface, but the existing usage of it is inconsistent - postgres only uses it for transactions, whereas mysql also uses it for result sets and one of the concrete statement implementations (but not as part of the statement interface). This inconsistency betrays the fact that it is an implementation detail, and not part of a generic public API such as the one defined by this repo.
If it were concluded that it does make sense to codify it as part of the public API, then the naming should be changed. Operation#onDestruct() - one of these names is wrong, either the method should be named onComplete() or the interface should be named... something else more generic. "Destruction" is a generic event that can be applied to any type of object, not just those that encapsulate a logical operation. Given the way this relates to the other API elements defined in this repo, my gut says that the method should be named onComplete().
However, the semantics of this mechanism are identical to Promise#onResolve() and I wonder whether it might make more sense to simply turn it into interface Operation extends Promise {} (or get rid of it entirely and just use Promise directly), so that one could yield $operation.
No @throws tags
The exceptions thrown by a method are part of it's public API and should be defined at the interface level, so that code working with the generic interfaces knows what it needs to handle. The exceptions defined at this level should be abstract types (abstract classes or interfaces), requiring the implementor to define a specific concrete subtype.

Related to https://github.com/amphp/postgres/pull/14 and https://github.com/amphp/mysql/pull/74

Operation was always meant to be an implementation detail, but might be usable/required if someone wanted to implement their own version of Pool. Thus it entered the public API, but never something that a user of the library would be concerned with. Naming certainly can be improved, absolutely open to suggestions. The semantics are not identical to Promise. It is not a placeholder, but represents resources being used that should be released when the object is destroyed or the operation completes. Moreover, the objects implementing Operation are used to resolve promises, so implementing Promise will not work. Perhaps a name such as Resource with a method onComplete()?
I agree with your points on naming, reflecting some of my own thoughts. ConnectionConfig feels incomplete and perhaps should include some of the methods that could be common, such as getHost(), getPort(), etc.
Looks like some of the @throws tags were lost in translation or were missing in the first place. Definitely should be added here. Not sure if the exception classes need to be abstract though, would there be a reason to differentiate between a QueryError from Postgres and MySQL?

A few thoughts I like to add:

ConnectionConfig#connectionString() should be getConnectionString() - method names need a verb. I realise this is effectively a property, but as we are currently stuck with using methods to implement these, the "rules" for method naming should be followed.

I disagree with this, the get-prefix doesn't add any meaningful value. Tell - don't ask. Or to quote Vaughn Vernon: The method names of Side-Effect-Free Functions are important. Although these methods all return Values (because they are CQS query methods) they purposely avoid the use of the get-prefix JavaBean naming convention. This simple but effective approach to object design keeps the Value Object faithful to the Ubiquitous Language. The use of getValuePercentage() is a technical computer statement, but valuePercentage() is a fluent human-readable language expression.
So I'd vote for removing all get-prefixes, however I can live with it, because it's not a real problem for me to keep them.

I saw this newly added method in Pool: public function resetConnections(bool $reset). I don't like this at all, because I don't know what the bool parameter is for (I actually had to look in the code on how this is used, in order to understand it). I prefer more meaningful method-names (and no params)  in this case. f.e. public function enableResetConnection() and public function disableResetConnection(). BUT... see my next point.

Then there is this: public function setIdleTimeout(int $timeout) in the AbstractPool. This means that the pool can change its behaviour from the outside, someone can somewhere change this timeout value and hence affect other parts of the system. I would prefer to remove all setters completely and inject those value either in the constructor, or, when it's too many, use a configuration object that groups those values together (also only injected in the constructor).

I don't think there is any difference in the Exception (f.e. QueryError), so one class in the SQL package to rule MySQL and Postgres implementation should be good.

Adding those methods getHost() (or simply host()), port(), etc to the ConnectionConfig sounds reasonable.

About the Operation class:

Perhaps a name such as Resource with a method onComplete()?

I like this suggestions, I would also add we can add an @internal annotation above that interface, so it's clear that no userland implementation should worry about this interface and make any use of it.

Renaming transaction to beginTransaction - I'm okay with that.

Adding the missing @throws annotations sounds reasonable as well.

I agree that setting the idle timeout and reset connection flag should only be injected in the constructor. Will update.

Operation should be deleted IMO and the __destruct handling be implemented via decorators. It's a slight overhead, but I'm fine with that, because it allows a way nicer abstraction.

the get-prefix doesn't add any meaningful value.

It allows typing get and getting a list of all things you can ask for, that's at least some value people often forget.

Tell - don't ask.

That's all fine, but getters are always "ask", never "tell".

@kelunik Yep, I agree that decorators would be a much better way to implement resource freeing. I should have time tomorrow to implement this and then I'll drop Operation from this library.

Do we still miss the @throws tags? Otherwise we've addressed most points in this issue, I think.

Everything done.


Update to Jackson Data Bind <IP_ADDRESS> or later

For the current 2.9.9 version there is this vulnerability: CVE-2019-14379

2.9.10 seems to have only two vulnerabilities:
https://www.cvedetails.com/version-list/15866/42991/1/Fasterxml-Jackson-databind.html

Even for the latest 2.10.1 there seem to be two:
https://snyk.io/vuln/maven:com.fasterxml.jackson.core%3Ajackson-core

I would suggest upgrading to the latest 2.10.1.

PR plz 😺

@jelovirt on the develop branch? I will do this after I double check on my side the security reporter we have in the build process reports no longer problems with 2.10.1

@raducoravu I think ´hotfix/3.4.1´ since it's a security fix.


How to execute raw sql using RXQueryExecutor

This is not a bug but a question on how to execute raw SQL statements.
In jOOQ, one can do the following:
String selectQuery = ...;
Result<Record> res = dsl.fetch(selectQuery);

How can we do the equivalent when using RXQueryExecutor? I do not have the flexibility to build a Query object rather I'm given a select query statement (pulled from an external source) as a String and have to execute it using Rx and process the result.

If you are using the JDBC-variant, you can use the executeAny-method to execute arbitrary SQL.
If the result is of any type (e.g. not the record-type from the QueryExecutor) than I'm afraid you have to use the underlying delegate, e.g. the ReactivePGClient or the AsyncClient for query execution
If the result is the same record-type as defined in the QueryExecutor you have to somehow convert the String to a ResultQuery of that type. DSL.resultQuery("yourPredefinedStringQuery") comes closest, but you need to properly cast it somehow to your record-type.

Thank you. After browsing through the source code, here's what I'd come up as an interim until you responded with your suggestions. Do you see any pitfalls, apart from relying on some internal logic / api of vertx-jooq?
For the JDBC variant:
public Single<QueryResult> query(String sql) {
	return RXTool.executeBlocking(h -> h.complete(new JDBCQueryResult(dsl.fetch(sql))), vertx);
}

For the ReactivePGClient
public Single<QueryResult> query(String sql) {
	return pgClient.rxPreparedQuery(sql).map(res -> new ReactiveQueryResult(res.getDelegate()));
}

I've tried to keep the API similar to what's already present in the Rx QueryExecutors i.e.
public <R extends Record> Single<QueryResult> query(Function<DSLContext, ? extends ResultQuery<R>> queryFunction)

Looks good imho.


Document workaround for getting table-size in Citus

From @marcocitus noted in issue: https://github.com/citusdata/citus/issues/631.
Workaround:
CREATE OR REPLACE FUNCTION citus_table_size(table_name regclass)
RETURNS bigint LANGUAGE plpgsql
AS $function$
DECLARE
        table_size bigint;
BEGIN
        PERFORM master_update_shard_statistics(shardid)
        FROM pg_dist_shard
        WHERE logicalrelid = table_name;

        SELECT sum(shard_size) INTO table_size
        FROM (
                SELECT max(shardlength) AS shard_size
                FROM pg_dist_shard JOIN pg_dist_shard_placement USING (shardid)
                WHERE logicalrelid = table_name AND shardstate = 1
                GROUP BY shardid
        ) shard_sizes;

        RETURN table_size;
END;
$function$;

SELECT logicalrelid AS Name, pg_size_pretty(citus_table_size(logicalrelid)) AS Size
FROM pg_dist_partition;

            name            |    size
----------------------------+------------
 articles_hash              | 16 kB
 articles_single_shard_hash | 0 bytes
 test                       | 8192 bytes
(3 rows)

Edit: update to newer version

We should use max() instead of min() or eliminate invalid shards in the query.
This issue was brought up in citus-user.
https://groups.google.com/forum/#!msg/citus-users/n4N9gWz_Ssk/lUVdCP1TBwAJ

Was already fixed in #631, updated Sumedh's comment.

3 UDFs are implemented to get the size of distributed tables.

citus_total_relation_size(relation_name) : Returns a distributed table and its indexes' total relation size. (b) Adds up total sizes on primary nodes. (c) If the function can't connect to a node, it errors out. (d) Works with shard replication factor equals to 1 and reference tables.

citus_relation_size(relation_name) : Return size of the table excluding the size of the Free Space Map(FSM) and Visibility Map(VM).

citus_table_size(relation_name) : Returns a distributed table's total relation size including size of FSM and VM, but excluding size of indexes.

Explanation of corresponding UDFs on postgresql : https://www.postgresql.org/docs/9.6/static/functions-admin.html#FUNCTIONS-ADMIN-DBSIZE
**Example SQL **
Examples of these UDFs can be found on : https://github.com/citusdata/citus/blob/master/src/test/regress/expected/multi_size_queries.out
PR for the implementation : https://github.com/citusdata/citus/pull/1267


[FEATURE] Help needed with docs website CSS template

Detailed Description

Help is requested from contributors to create CSS stylesheets, header and footer templates for the doxygen based online site for the repo.

I want to work on this issue. Please assign me this issue

Please don't ask to be assigned. See the contributing guidelines.
If you want to help with this, submit a PR or contact the admins to work on it.
Thank you. 🙂

@brainless-coder Feel fret contribute and create a pull-request when ready. You can also reach us on the Gitter chat - link posted in the repo README.

@brainless-coder Any update over this?


example 3 for Linechart is not working anymore

Describe the bug
LineChart is not of Type Widget. See: https://github.com/imaNNeoFighT/fl_chart/blob/master/example/lib/line_chart/samples/line_chart_sample3.dart
To Reproduce
take example Widget with current fl_chart version and copy paste it into a project. :)
I really need some Quick Start Template otherwise this is not working out as a graph lib for me :)
Screenshots
screenshot
Versions
Flutter: stable latest
FL_chart:  ^0.2.0

Please use our latest version 0.8.7.
Thanks!


Make check on OS X

Thanks for adding this. Let's see if it will complete now. :four_leaf_clover:

Works great on Mac (and Linux).  This doesn't affect Windows (please consider adding [skip appveyor] to the commit message in the future if it doesn't affect Windows). Will merge as the platforms needing testing of this change have passed. Thanks @isuruf.


Mccalluc/index tweaks

Description

Move index.html to svg.html so index can be reverted.
Add event handlers.
Replace higlass.io html with higlass.io viewconf

Approved.
How do you feel about adding a note to the Development section of the README indicating that svg.html exists and that it should be used when developing or modifying new tracks?

I would like to move this over to the examples directory, rather than littering the top level.

How do you feel about adding a note to the Development section of the README indicating that svg.html exists and that it should be used when developing or modifying new tracks?

Will do.


Cal sorting

Describe your changes
I have implemented an algorithm that sorts out frames collected for either k-gain, non-linearity or em-gain calibration. The case of em-gain calibration is partly implemented: there is a mock set generated, though it is not yet sorted by sorting.py. This is an agreed work to do, and it is not necessary for v1.1
The sorting algorithm adds a keyword OBSTYPE to the sorted files with values MNFRAME, KGAIN, NONLIN, according to the calibration type selected (Notice that the frames to generate a mean frames are always sorted for any calibration type)
Type of change
I added two new functions:
sorting.py: sorting algorithm with I/O Datasets and the calibration type to be considered
The algorithm respects the constraints of the calibration data as defined in https://collaboration.ipac.caltech.edu/pages/viewpage.action?spaceKey=romancoronagraph&title=Calibration+functions+description
How to run the code
python test_sorting.py
Issues
In a mac system, set at least ulimit -n 800 before running the code
Checklist before requesting a review

[ x] I have linted my code: 9.02/10
[ x] I have verified that all unit tests pass in a clean environment and added new unit tests, as needed
[ x] I have verified that all docstrings are properly formatted and added new documentation, as needed

Please also fix the docstrings to make the tests pass.

I'll try again the docstrings issue. It's quite annoying tbh because I mimicked other modules I wrote that passed and still does not. Do you have a reference for us to follow if we can't have a minor flexibity?

I'll try again the docstrings issue. It's quite annoying tbh because I mimicked other modules I wrote that passed and still does not. Do you have a reference for us to follow if we can't have a minor flexibity?

I think this one is pretty self-explanatory if you read the error message produced by the build:

Basically your variable name in your function declaration em_emgain doesn't match the variable name you put in the docstring em_gain. You can also run these checks locally on your computer before you push the code (see the README file for the repo).

Issue when running e2e tests after merging branch obstype-switch: https://github.com/roman-corgi/corgidrp/issues/250

@kjl0025 in case he has some ideas:
I've implemented all the changes for the sorting algorithm and have written a unit test.
As for the e2e, it's been long and confusing. I realized that if I remove the cosmic ray detection from the recipe on non linearity, and I leave the other modules, the walker ends up successfully though the nonlinearity table is not good with errors of a few percent, instead of single precision. Not sure where the error comes from. The sorting algorithm seems to find the right combo: Dataset to calibrate NON-LIN. A sorting algorithm based on the constraints that NFRAMES, EXPTIME and CMDGAIN have when collecting calibration data for K-gain, Non-linearity and EM-gain vs DAC was applied to an input dataset from PUPILIMG visit files.The result is that Non-linearity has 242 frames with gains [1.649999976158142, 5.239999771118164, 8.600000381469727, 16.70000076293945, 27.5, 45.2599983215332, 87.5,<PHONE_NUMBER>035156,<PHONE_NUMBER>068359,<PHONE_NUMBER>070312,<PHONE_NUMBER>140625]
Consistent with https://collaboration.ipac.caltech.edu/pages/viewpage.action?spaceKey=romancoronagraph&title=Calibration+functions+description
If I add the cosmic ray, I get this long error message that I am not sure where it is coming from (https://github.com/roman-corgi/corgidrp/issues/251)
Running walker
Traceback (most recent call last):
File "nonlin_e2e.py", line 206, in
test_nonlin_cal_e2e(args.tvacdata_dir, args.output_dir)
File "nonlin_e2e.py", line 139, in test_nonlin_cal_e2e
walker.walk_corgidrp(nonlin_l1_list, '', e2eoutput_path, "l1_to_l2a_nonlin.json")
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 77, in walk_corgidrp
recipes = autogen_recipe(filelist, outputdir, template=template)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 153, in autogen_recipe
_fill_in_calib_files(step, this_caldb, first_frame)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 194, in _fill_in_calib_files
best_cal_file = this_caldb.get_calib(ref_frame, calib_dtype)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/caldb.py", line 314, in get_calib
return dtype(calib_filepath)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/data.py", line 853, in init
super().init(data_or_filepath, err=err, pri_hdr=pri_hdr, ext_hdr=ext_hdr, err_hdr=err_hdr)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/data.py", line 271, in init
with fits.open(data_or_filepath, ignore_missing_simple=True) as hdulist:
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 174, in fitsopen
lazy_load_hdus, ignore_missing_simple, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 411, in fromfile
lazy_load_hdus=lazy_load_hdus, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 1064, in _readfrom
fileobj = _File(fileobj, mode=mode, memmap=memmap, cache=cache)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/utils/decorators.py", line 536, in wrapper
return function(*args, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/file.py", line 170, in init
self._open_filename(fileobj, mode, overwrite)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/file.py", line 558, in _open_filename
self._file = fileobj_open(self.name, IO_FITS_MODES[mode])
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/util.py", line 390, in fileobj_open
return open(filename, mode, buffering=0)
FileNotFoundError: [Errno 2] No such file or directory: '/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/tests/e2e_tests/l1_to_kgain_output/CGI_EXCAM_L1_0000051867_kgain.fits'

And this is for K-gain: Considering K-gain:
Dataset to calibrate K-GAIN. A sorting algorithm based on the constraints that NFRAMES, EXPTIME and CMDGAIN have when collecting calibration data for K-gain, Non-linearity and EM-gain vs DAC was applied to an input dataset from PUPILIMG visit files.The result is that K-gain has 110 unity frames with exposure times [0.077, 0.769, 1.538, 2.308, 3.077, 3.846, 4.615, 5.385, 6.154, 6.923, 7.692, 8.462, 9.231, 10.0, 10.769, 11.538, 12.308, 13.077, 13.846, 14.615, 15.385, 1.538] seconds with 5 frames each.
It crashes :
Running walker
Traceback (most recent call last):
File "kgain_e2e.py", line 153, in
test_l1_to_kgain(tvacdata_dir, outputdir)
File "kgain_e2e.py", line 111, in test_l1_to_kgain
walker.walk_corgidrp(ordered_filelist, "", kgain_outputdir, template="l1_to_kgain.json")
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 77, in walk_corgidrp
recipes = autogen_recipe(filelist, outputdir, template=template)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 153, in autogen_recipe
_fill_in_calib_files(step, this_caldb, first_frame)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/walker.py", line 194, in _fill_in_calib_files
best_cal_file = this_caldb.get_calib(ref_frame, calib_dtype)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/caldb.py", line 314, in get_calib
return dtype(calib_filepath)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/data.py", line 853, in init
super().init(data_or_filepath, err=err, pri_hdr=pri_hdr, ext_hdr=ext_hdr, err_hdr=err_hdr)
File "/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/corgidrp/data.py", line 271, in init
with fits.open(data_or_filepath, ignore_missing_simple=True) as hdulist:
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 174, in fitsopen
lazy_load_hdus, ignore_missing_simple, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 411, in fromfile
lazy_load_hdus=lazy_load_hdus, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/hdu/hdulist.py", line 1064, in _readfrom
fileobj = _File(fileobj, mode=mode, memmap=memmap, cache=cache)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/utils/decorators.py", line 536, in wrapper
return function(*args, **kwargs)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/file.py", line 170, in init
self._open_filename(fileobj, mode, overwrite)
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/file.py", line 558, in _open_filename
self._file = fileobj_open(self.name, IO_FITS_MODES[mode])
File "/Users/srhildeb/miniconda3/envs/cgi_etc/lib/python3.7/site-packages/astropy/io/fits/util.py", line 390, in fileobj_open
return open(filename, mode, buffering=0)
FileNotFoundError: [Errno 2] No such file or directory: '/Users/srhildeb/Library/CloudStorage/OneDrive-JPL/Documents/GitHub/corgidrp/tests/e2e_tests/l1_to_kgain_output/CGI_EXCAM_L1_0000051867_kgain.fits'


errors.angularjs.org URL incredibly long.

I've attempted to change the log level in order to get rid of the http://errors.angularjs.org/1.3.14/$injector/modulerr?<crazy_long_params_here> URL, but I've had no luck. Is there an option to minify or remove this URL when running tests through karma? It's including many full file paths which happen to be very long making the tests almost unreadable.

@lethjakman totally agree, but really is it issue of karma?
I created the issue in angular.js about it

Ahh maybe it isn't a karma. I'll go weigh in there, thank you.
On Thu, May 14, 2015, 9:49 AM Maksim Ryzhikov<EMAIL_ADDRESS>wrote:

@lethjakman https://github.com/lethjakman totally agree, but really is
it issue of karma?
I created https://github.com/angular/angular.js/issues/10590 the issue
in angular.js about it
—
Reply to this email directly or view it on GitHub
https://github.com/karma-runner/karma/issues/1399#issuecomment-102079579
.

Closing as it's not an issue with karma itself.


Prepare the upgrade to the upcoming Gradle 8.x, there are some incompatible features for now:

It would be nice to prepare the upgrade to the upcoming Gradle 8.x, there are some incompatible features for now:

https://github.com/gradle/gradle/milestone/164
https://docs.gradle.org/current/userguide/upgrading_version_7.html

This is what gradle help --scan outputs:

Originally posted by @XVilka in https://github.com/NationalSecurityAgency/ghidra/issues/3050#issuecomment-947615608

Yes, we will have to address this in the coming months.

I think there are more Gradle 8 -related issues than those already fixed here.
Arch repos got Gradle upgraded to 8.0 a few days ago, and now attempting to run gradle --init-script gradle/support/fetchDependencies.gradle init on a freshly cloned Ghidra repo ends with the following error:
FAILURE: Build failed with an exception.

* Where:
Script '<path omited>/ghidra-src/gradle/root/jacoco.gradle' line: 77

* What went wrong:
A problem occurred evaluating script.
> Could not get unknown property 'JacocoMerge' for root project 'ghidra-src' of type org.gradle.api.Project.

I believe this is somehow related to this issue but I don't know Gradle enough to say for sure.


Expose all queries and mutations as resolvers

It seems like a common sentiment amongst Prisma users is that they just want to expose the entire generated CRUD API to the client, with a few custom pieces. See the following for evidence of that:

https://github.com/graphql-binding/graphql-binding/issues/40
https://github.com/graphql-binding/graphql-binding/issues/37

To achieve this, no one wants to copy over parts of the generated schema nor resolvers in any way if possible. That is cumbersome and messy. As a partial solution to this problem, I propose exposing mutation and query resolver objects that have all of the generated mutation and query bindings wrapped as resolvers. This might be done on the Binding class, exposing a queryResolvers property and a mutationResolvers property. Essentially, I think we need to incorporate functionality like the following:
export function prepareTopLevelResolvers(resolverObject: Query | Mutation) {
    return Object.entries(resolverObject).reduce((result, entry) => {
            const resolverName = entry[0];
            const resolverFunction = entry[1];
            return {
                ...result,
                [resolverName]: async (parent, args, context, info) => {
                    return await resolverFunction(args, info);
                }
            };
    }, {});
}

This function does the automatic wrapping and allows you to very easily expose the entire generated API in your application server. See this write-up for a full example application and explanation: https://medium.com/@lastmjs/advanced-graphql-directive-permissions-with-prisma-fdee6f846044

Thanks a lot for bringing this up @lastmjs. This seems to be indeed a very common pattern which we'll try to simplify a lot with some upcoming tooling in the next few weeks.
For everyone interested in this, please provide more information about your use case(s) in the comments. Things we have heard so far:

Adding authentication & authorization rules
Add input validation rules to mutations
Extending schema
Hiding certain fields / arguments (but keep majority of the schema)
Generate code based on Prisma GraphQL API

@schickling may you share a bit more about what you're planning for this if you can? Thank you!
P.S. The 5 points you mentioned are already super useful.

We need to append on extra geolocation related values determined by the server to a couple of queries and it should be invisible to the user that the returned response is filtered in any way. I guess this concept would allow us to not rewrite the entire schema for this use case? We are using Node & GraphCMS

Thanks for the write up @lastmjs! My experience with Prisma is very similar to what you've described. I'd like to expose the majority of the generated database API, with the addition of permissions/authentication. I'd like to keep duplication of the generated schema in my schema/graphql to a minimum. Ideally I'd be able to replicate the graphcool-framework API I had previously.
@schickling would it be possible to get an update on the upcoming tooling to simplify this approach?
It's tricky to track the progress on the above across #40, #83, and graphql-middleware.
If there's soon to be a recommended approach to this, with tooling to support, I'd like to go with that. Otherwise, I'll follow @lastmjs's approach in their blog.

i am evaluating prisma and also react-admin
react-admin has three graphql-adapters:

simple
graphcool
prisma/opencrud

the prisma one works against the prisma-server, but of course this is not ideal in practise as you want to add authentification, modifications, etc. as mentioned in this thread here.
I am now about to introduce a graphql server that uses the prisma client, but i am unsure which "dialect" for crud-operation i should take. I would like to stick with the prisma one, so I would also like to see a best-practise to forward prisma's crud api

Thank you for reporting.
In the last few months, since the transition of many libraries under The Guild's leadership, We've reviewed and released many improvements and versions to graphql-cli, graphql-config and graphql-import.
We've reviewed graphql-binding, had many meetings with current users and engaged the community also through the roadmap issue.
What we've found is that the new GraphQL Mesh library is covering not only all the current capabilities of GraphQL Binding, but also the future ideas that were introduced in the original GraphQL Binding blog post and haven't come to life yet.
And the best thing - GraphQL Mesh gives you all those capabilities, even if your source is not a GraphQL service at all!
it can be GraphQL, OpenAPI/Swagger, gRPC, SQL or any other source!
And of course you can even merge all those sources into a single SDK.
Just like GraphQL Binding, you get a fully typed SDK (thanks to the protocols SDKs and the GraphQL Code Generator), but from any source, and that SDK can run anywhere, as a connector or as a full blown gateway.
And you can share your own "Mesh Modules" (which you would probably call "your own binding") and our community already created many of those!
Also, we decided to simply expose regular GraphQL, so you can choose how to consume it using all the awesome fluent client SDKs out there.
If you think that we've missed anything from GraphQL Binding that is not supported in a better way in GraphQL Mesh, please let us know!

In the context of that particular issue - There are other, individual tools that generates queries and mutations from a given schema, no matter the source.
We are not fans of that approach but if you need it, there are tools for that that are not tied to how the schema and API were created.
We're looking forward for your feedback of how we can make your experience even better!

for reference: if someone uses prisma2 and react-admin, we built a new, maintained data-provider for it. It has a helper function for the backend that exposes all crud operations for a given resource
https://github.com/panter/ra-data-prisma


Add a screen rect check for bounds drawing to prevent incorrect lines…

… & ImGui triangle list exhaustion.
When the camera is inside the bounding box the boundary points can blow up, causing incorrect rendering and a large number of triangles added to ImGui triggering an assert in debug or potentially incorrect GUI rendering in release.
A proper fix would be to intersect the boundaries with the screen rect, but this simpler solution works reasonably well.
Note that the check:
stepCount = min( stepCount, 1000 );

is probably not required unless someone is using a very large monitor, but some form of primitive count check should likely be done here.

Merged. Thanks Doug!


Error: Missing template processor. Master is unusable.

The master version of the addon is unusable for me.
Including
    "ember-cli-async-button": "git://github.com/dockyard/ember-cli-async-button.git",

to package.json renders the app impossible to start. Running ember s results in an error "Missing template processor". Stacktrace.
This is reproducible on a fresh ember new foo project.

Very strange, TravisCI is passing. @rwjblue any idea what this is?

I just confirmed this with latest ember-cli (0.2.0-beta.1)

related: https://github.com/ember-cli/ember-cli/issues/3419

@lolmaus try master now

Will try later and report. Thank you, @bcardarella.

Still have the issue.
I did rm -rf node_modules/ember-cli-async-button and npm install.
npm list | grep async-button reports the latest commit on master:
$ npm list | grep ember-cli-async-button
├──<EMAIL_ADDRESS>(git://github.com/dockyard/ember-cli-async-button.git#8fb2fc28ee6140a4f2a71669e746cb3d849b91cf)

So do 0.5.0 and 0.4.0 releases.
0.3.1 works fine. All other addons in my project work fine.
:ambulance:

What version of Ember CLI are you running with?  0.4.0 and above requires ember-cli 0.2.0 or above.  I submitted https://github.com/dockyard/ember-cli-async-button/pull/36 to make sure a nice message is printed when using older Ember CLI versions.

I just tested 0.5.0 against an app using Ember CLI 0.2.0 and it does build properly...

I was on  0.2.0-beta.1. Upgrading to 0.2.0 resolved the issue. Thank you, @rwjblue.

:+1:


docs: Update commands.md to add detail for debug migrate-database

Add details of what debug migrate-database does, to provide information and reassurance. The text is close to the prompt that the command returns.

The Issue

#

How This PR Solves The Issue
Manual Testing Instructions
Automated Testing Overview

Release/Deployment Notes

I pushed a commit excluding symfony.com from link checks.


update: content changes as required

This update brings in the text changes ive attached a screen shot of the images ive picked there are more so let me know

I might just need to adjust either the image or code to get rid of that white space. Also I thinck 'tech' in the title shouldn't be capitalised.

Ive actually made a change to the image padding so the white top and bottom border are no longer present


net-mgmt/zabbix-agent: Service is not beeing restarted during upgrade procedure

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
I upgraded OPNsense from version 23.7.5 to 23.7.6. This upgrade did not require a reboot. During the upgrade zabbix5-agent was upgraded: 5.0.37 -> 5.0.38. The service has not been restarted during the upgrade procedure though and version 5.0.37 was still running. This was reproducible on three OPNsense instances. Restarting the service manually solved the issue.
To Reproduce
Steps to reproduce the behavior:

Install OPNsense 23.7.5 with the plugin zabbix5-agent 5.0.37
Upgrade OPNsense to 23.7.6
Check logs of zabbix-agent
See error: version 5.0.37 is still running, although version 5.0.38 has been installed

Expected behavior
Automatic service restart during upgrade procedure, if OPNsense will not reboot after upgrade.
Screenshots
None.
Relevant log files
Services --> Zabbix Agent --> Log File
Additional context
None.
Environment
OPNsense 23.7.6-amd64
FreeBSD 13.2-RELEASE-p3
OpenSSL 1.1.1w  11 Sep 2023

This is the intended behaviour.
Cheers,
Franco

Thank you for the feedback, but may I ask for the reasons for this being considered as the intended behavior?
I noticed this issue, because Zabbix announced several critical security issues on versions prior to 5.0.38 lately. I checked the installed versions on my OPNsense instances and was happy with it. I didn't notice, that OPNsense was still running the old versions of the agents in the first place. So I thought my systems were safe, but they weren't.

Restarting based on packages (and indirect dependencies) is a non-trivial task. Furthermore restarting (and reconfiguring) may not be side effect free. If you want the behaviour rebooting after a release as a best practice makes sense. Restarting services manually can be useful, but is also hard to define: restart software or just apply (possible) fixes in the configuration.
Restarting a service seems simple, but once you start defining what should be done when it’s going to be much more.
At least as it is intended now new versions don’t taint smooth operation. If you start restarting you will have negative impacts of varying sorts (VPN disconnecting, DNS cache flushing, etc.).
Cheers,
Franco

Thank you for the explanation. I will close this and open a new issue in opnsense/core, to request an "always reboot after update" option in System: Firmware: Settings or a new cronjob command "Automatic firmware update (always reboot)".


Use proper Docker secrets for the API token

Proper Docker secrets integrate well with Google Cloud Run as well, and are better suited than environment variables.

The token was moved to Google's Secret Manager and exposed from there as an environment variable to the service:

This gets us proper secrets management (it's recognized as a secret and therefore possibly redacted from logs etc (didn't test this)), while keeping the simplicity of environment variables.


docs: correct issue templates links

:tada: This PR is included in version 2.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sane defaults for terraform templates

As a:
Operator
I want:
Defaults for my terraform templates
So that:
I don't have to manually fill in every terraform variable
Describe the solution you'd like:
Defaults for all terraform variables that I must provide to make a bedrock deployment work.
Additional context:
Many of the variables required don't have any defaults, so they have to be provided manually.
Does this require updates to documentation?:
Yes

@bnookala do you think the bedrock infra scaffold workflow solves this?
Cc @yradsmikham @NathanielRose

@andrebriggs @NathanielRose From the looks of this, I think this seems to be strictly speaking of the Bedrock Terraform templates, not the bedrock CLI. It's not possible to provide a default value for every variable, especially if we expect users to "BYO" resources. I think as we currently stand, our main templates already provide as many defaults as we can.
Unless we provide a "starter" script to create these resources beforehand, and etc...


Introduce deploy command to bee

Task
Default timeout for service start on Windows is 30s (its hard to be increased, you should reboot the machine) and bee is failing during initial start due to wallet deployment. Idea is to deploy wallet during post-install so that services is started much faster even during initial start.
Acceptance criterea

There is a deploy command bee deploy that will deploy wallet.

This is going to be tricky. The chequebook initialisation requires a ton of dependencies which are created in node.go. Seems we either need to duplicate a lot of stuff from node.go or refactor NewBee from node.go, quite a bit.

@vandot , could you live without this issue fixed?

@paxthemax , this could be something. Pending reply from @vandot (above)

it's quite important to have this I think for the windows service stuff. and a bit of refactoring won't hurt node.go either. one big issue though is that that init code has also changed on storage-incentives branch and rebasing would be a pain. anyway I do have some prior work here. @paxthemax ping me if you start working on this.

@paxthemax , could you please assign your PR to this issue and assign the issue to the correct pipeline in the Active Sprint workspace?

Would it be good to separate the deployment command for the start command functionality?
Currently, when calling bee start the chequebook service is deployed, and with it the chequebook contract, amongst others. We could consider extracting this to a separate command - deploy that specifically does this, leaving start as a command that does not require on-chain deployments.

start should deploy (if required) and then start. Separately, we should also be able to deploy only.

I agree with @Eknir , there is runtime functionality that assumes we have Ethereum connectivity for the chequebook as we recently learned as slock.it is having issues right now.

done in https://github.com/ethersphere/bee/pull/1314


ConsoleRunHost Exception Logging

Hi!
We are using the OnException method to register a callback for handling exceptions. In that callback, we create a custom log entry like so:
service.OnException(ex =>
{
    // some more logic ...
    Log.Fatal(ex, "An unhandled exception occurred in service.") // our custom log entry (simplified)
});

When running the service as console application, unhandled exceptions are logged twice, because in the ConsoleRunHost implementation, an error is logged too:  https://github.com/Topshelf/Topshelf/blob/22baba8dff7ba47b12b761b03c15006a0d3cd44c/src/Topshelf/Hosts/ConsoleRunHost.cs#L106
Any thoughts on how to suppress the hardcoded _log.Error(...) call in the ConsoleRunHost?

@meinsiedler usually the logging of the internals of the library should be disabled - this is usually done with the logger configuration.

@maxcherednik Thank you for your response.
I'm not quite sure how to turn off only the internal log messages of the library. Do you have an example how to achieve that? (We are using Serilog btw.)
For our setup, it would actually be OK to log internal messages - especially the Information messages when a service starts or stops are relevant. So maybe there is a way to just ignore the above stated Error message in case of an exception?

This is how it's done to hide asp.net core internal logging.
https://github.com/serilog/serilog-aspnetcore#instructions

I am closing all issues that haven't seen any activity. You're welcome to submit a PR to resolve the issue or provide more information.


How to access enum values s.a EmploymentType.FULL_TIME ?

I import the client library as follows:
const talent = require('@google-cloud/talent').v4beta1;

sample use case: searching for jobs based on EmploymentTypes
const jobQuery = {
    // e.g:
     jobQuery.employmentTypes = [EmploymentType.FULL_TIME]
};

@kajiru for enums, I believe you should just be able to provide the string "FULL_TIME"; let me know if this does the trick for you?

@bcoe That works, thanks!

@kajiru awesome 👍 I'm glad you're unblocked.