ci(workflow): speed up e2e tests

Motivation
See this review: https://github.com/dreammall-earth/dreammall.earth/pull/1180#discussion_r1648992425
I want to challenge why we have a sleep in our workflow after all. I don't think it's necessary. I will let this run a couple of times and see where it breaks.
How to test

Restart the e2e workflow a number of times
See if it fails

The workflow in main takes 7 minutes for setup and 2 minutes for testing.
7 minutes for setup is too much!
My suggestions:

merge this PR
merge #1079
remove docker-compose in the e2e workflow completely

Instead use service containers for postgres and redis

run frontend, backend + presenter natively

I think this should speed up the workflow significantly

sleep was removed in #1262, I'll close this.


Dockerfile

[x] Add a dockerfile for the app.
[ ] Upload the image to our repo.
[ ] Find a good way to import files into the image.

We want to publish this to public docker hub correct?

This is great – thanks for getting it started @AdityaHegde!

Closing in favour of #166


WAS_Text_Concatenate doesn't allow text_ widgets to be converted to inputs

Only the required input types (delimeter and clean_whitespace) can be converted to inputs.

https://github.com/WASasquatch/was-node-suite-comfyui/blob/ee2e31a1e5fd85ad6f5c36831ffda6fea8f249c7/WAS_Node_Suite.py#L10296

Nevermind. This is a weird artifact from the script I'm writing that creates workspace.json files.


Check conditions

I think the only condition you can check once and store the result is #2 - hardware. The API level could change if the user updates to a newer version of the OS. Granted, that's not going to happen often, and probably unlikely that someone has hardware WITH a fingerprint reader that is on API 22 or lower, but it's still a possibility. And who knows - someone could create an external fingerprint reader that plugs into the USB port or audio port, so maybe even the hardware check needs to be done every time.

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

ID: febd430b-faf9-e25d-1bc8-de82a8453423
Version Independent ID: 100f9740-a1f6-38de-57bb-e954c533dd92
Content: Getting Started with Fingerprint Authentication - Xamarin
Content Source: docs/android/platform/fingerprint-authentication/get-started.md
Product: xamarin
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

@jfren484 fair call, it's always dangerous to make assumptions about the future! Going to rewrite that paragraph to address your feedback, thanks.

Wording has been updated in the doc, thanks again.


Feature Request: Some way to refresh device list of useMediaDevices

While working with some webcam flows in React, I've found that permissions states can be really funky in browsers. Specifically, when first requesting permissions, you don't have access to the proper input device list until the user has granted permissions. Once the user does grant permissions, it doesn't seem to automatically re-trigger the useMediaDevices hook, since that's not directly tied to any permission change events (and can't effectively be since Firefox doesn't support camera/microphone as permissions in the permissions API).
My proposal:
Some sort of optional function that can be used to manually trigger an update on the useMediaDevices check, i.e.
const { devices, reload: reloadDeviceList } = useMediaDevices();
...
navigator.mediaDevices.getUserMedia({ audio: true, video: true }).then((stream) => {
      stream.getTracks().forEach((track) => track.stop());
      reloadDeviceList();
    })

Thanks for report! I will git it a try soon.

@bfaulk96 we support it in version 3.0.1 . We have made a breaking change to the return value of useMediaDevices by modifying it to be an array. Additionally, we have added a parameter to determine whether to request permission immediately.

@childrentime Just checked it out, thanks! This is great! I did notice that when checking it out on the docs page, the first time you accept permissions, it looks like it doesn't properly close the stream; it stays trying to record until you refresh.

oh, crap. I should not be hurry. I will recheck it.

@childrentime our team just learned that Safari doesn't support navigator!.permissions.query... Not necessarily asking for Safari support on this, but it might be good to throw that in the isSupported check

@bfaulk96 Hi, I want to know if it works correctly on Safari 15? If it does, It is just executing an additional unnecessary JavaScript code?

I see it from https://caniuse.com/?search=permission.query

if Safari 15 or below does not support this functionality at all, then i will add it to the useSupported check and return it.

if Safari 15 or below does not support this functionality at all, then i will add it to the useSupported check and return it.

Correct, it throws an error in the browser – TypeError: undefined is not an object (evaluating 'navigator.permissions.query')

I want to know if it can still access the camera properly. If the functionality works fine, then I should not set the value of useSupported to false."
throw error is expected, will not cause application crash.

and pr welcome🥰


Add tools/sync-bazelversion.sh, commit results

Description
Ensures Bazelisk uses the same Bazel version in every workspace in the project. Part of #1482.
The sync-bazelversion.sh script is currently in tools/. #1608 introduced the scripts/ directory, and I'm happy to move it there, but that directory seems focused on third_party/repositories/scala_*.bzl update automation.
Motivation
I discovered that the workspaces under the main workspace were running the latest mainline Blaze release I had installed, instead of 6.5.0 from .bazelversion. Now all the workspaces are synchronized on the same Bazel version.
As noted in the tools/sync-bazelversion.sh comments, symlinks or an import ../.bazelversion mechanism would be preferable. However, the former can surprise Windows users, and Bazelisk doesn't support the latter.

 If I'd've thought to check that before, that's where I would've put it to begin with. D'oh!
Just pushed a new commit to move it to scripts/.


added test for stat function

I've added a small test for the stat function.
For some tests I can only test that the value is a positive integer as some values would be different on each developers machine (e.g. user id, inode number, etc).
I've also added an import for the old posix::Path as this is currently needed for the File type. It will be removed as soon as the new io module is more stable and can create files.
If the stat call returns Err I panic! as this indicates a serious problem on that machine.
What do you think?
I will add more unit tests in the future.

Seems to break as std:.path:Path has now "as_vec" method. I will look into this.

One way to fix the error in utils.rs would be:
impl ToCStr for Path {
    fn to_c_str(&self) -> CString {
        CString::from_slice(self.to_str().unwrap_or("").as_bytes())
    }
}

Would this be an option or are there better and more efficient ways to do that?

I think this is a recent breakage in rust. However, you should look at #61 which is working to solve the Path problem in a better way. Besides that, looks good. Once it passes CI I will merge :smile:

OK, will get the latest version and look at #61.

Now it builds.
I've added the unlink function to  unistd. This allowed me to use only nix functions to create, close and remove the test file for the stat function.
I've added asserts to the unlink, close and open functions to make sure everything is fine with creating and removing the file.
I will add more unit tests for the functions in unistd in the coming days.
Hope this is ok like this. More will follow :-)

Followed Utkarsh's suggestion and used unwrap. This allowed me to get rid of the not really necessary variables without compiler warnings about unused code variables or return values.

Thanks! Squashed & merged: f898b825566ab0738dbfa3a1a45010d37060aed0


Add URL as a possible input option to Vimeo player

This is reasonable to process correctly private Vimeo links, like https://vimeo.com/25345658/fe46e209ac - that contains not only vimeo id, but also private key.
So, from this point, we can pass a vimeo url to data-video-id attribute, like we do in our project:
<div #plyrPlayer
        attr.data-type="{{ externalType }}"
        attr.data-video-id="{{ currentUrl }}">
</div>

Cheers. I need to merge this to develop somehow for the v3 release. Will take a look 👍


Skip To Content Unsuccessful in Mobile View

Bug Report
Description

While testing with screen readers in Mobile Browsers (Voiceover in iOS/Safari and Talkback in Android/Chrome with Pixel), the hidden 'skip to content' appears unsuccessful.
In iOS it will either open up the navigation or click on the logo (which in our app acts as a 'home link')
In Android, it has no effects
The skip to content button does not appear in mobile browsers.
Testing this morning on Safari with VoiceOver, Skip to Content was successful in the fullscreen view, but failed once the browser was shrunk down to a smaller size where the hamburger menu was present.

Steps to Reproduce

Navigate to https://engineering.cerner.com/terra-framework/#/ on mobile device
Turn on Screen Reader Capabilities on mobile device
Navigate to where 'Skip to Main Content' is read using screen reader and select
Note that click does not seem to have any affect.

Additional Context / Screenshots
Expected Behavior
Skip to Content should direct users to main content in all browsers
Possible Solution

Environment

Component Name and Version: terra-application-header-layout, 3.5.0 in our Component, also reproducible on https://engineering.cerner.com/terra-framework/#/
Browser Name and Version:  Chrome 74 in Mobile, Safari 12 in iOS, Desktop
Node/npm Version: lts/carbon in our app
Webpack Version: 3.11.0
Operating System and version: Reproducible on various android and ios devices, as well as Mojave on the Desktop

@ Mentions

Planning to investigate this and see what I can figure out on this issue.

It looks like the common markup pattern for skip links may be broken on iOS and Android when using their respective screen readers - (VoiceOver and TalkBack). More info in this article: https://axesslab.com/skip-links/

A bug has been logged to Apple about in-page links not working as intended when using VoiceOver
A bug has been logged to Google about broken skip links usage

@bjankord thanks for the investigation!
Looking at this

what is terra's standpoint on this? Do we wait for browser vendors to fix it, or implement one of the hacks they listed?

The recommendation we've received from @scottwilmarth on items like this is to code the common / recommended accessibility markup patterns. If there is assistive technology that doesn't work with the common accessible markup patterns, we shouldn't try to customize the code to solve for a specific browser, screen reader, device bug/inconsistency.
The way we've implemented the skip to main content functionality is a fairly standard way based on resources I've seen for coding this type of functionality.
Based on this, I'm planning to close this issue as a won't fix.  This stems from an issue on the browser side.  Sites created with the terra-application-layout should still be functional without a change.  Users can navigate to the main content landmark region due to the usage of <main role="main" /> element used in the layout. As browsers correct the issues that have been logged about inconsistencies with the skip to content functionality, the terra-application-layout should continue to work as intended.


Failed to build iOS app

when i run ,debug console:
Launching lib/main.dart on iPhone X in debug mode...
Xcode build done.                                           10.4s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **
Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
    /Users/zhangzhengyu/Documents/HKFamily_Flutter/flutterHK/hongka_flutter/build/ios/Debug-iphonesimulator/Runner.app: resource fork, Finder information, or similar detritus not allowed
    Command /usr/bin/codesign failed with exit code 1
Could not build the application for the simulator.
Error launching application on iPhone X.
Exited (sigterm)

Run flutter doctor:
[flutter] flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v1.0.0, on Mac OS X 10.14.2 18C54, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.0)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.1.7)
[✓] VS Code (version 1.31.1)
[✓] Connected device (2 available)

• No issues found!
exit code 0

Looks like https://stackoverflow.com/questions/2017756/command-usr-bin-codesign-failed-with-exit-code-1

It seems a pure iOS problem instead of flutter one.
See answers from apple.
https://developer.apple.com/library/archive/qa/qa1940/_index.html


future directions, what does a user need?

i first wrote this in another issue, but it's not the place for this discussion, but you triggered it by a statement in issue #96:

I REALLY dislike the imperative "lineTo", and "bezierTo"

declarative is fine, and I am voting for it, but...
please keep in mind, that lineTo, moveTo, ... usually makes sense with relative movement.
lineTo with relative coordinates is about adding vectors instead of defining points.
One reason to use functions like this is the possibility to allow different kinds (like bezier or arc) for each segment.
Most declarative systems lack the possibility to define different properties for segments and nodes.
Also, I don't really agree that lineTo is imperative in nature (though I understand why you are saying this).
From my POV lineTo declares(!) a line just like start and end point do. It only reuses (=connects to) the end of the previous object. So it's not stateless in some way, which might look bad at first.
But, in graphics there is a difference between single, separate lines and connected lines. Connections are different from ends, they can have a radius or be smoothed, they could even have extra shapes like spheres. In general there must be some definition how to connect one object to the next, which is not always obvious (e.g. how do you connect two rectangular bars, with different rotations? The ends can also have their own shapes, e.g. spherical, rectangular, arrow, etc.
In this respect line and lineTo is something very different.
Actually, you can do both absolute and relative coordinates in both declarative and imperative systems.
But unfortunately most systems have only one of them.
Systems that have all of this usually need words for three kinds:

xxx (separate)
xxxTo (connected, absolute end)
xxxBy (connected, relative end)
with xxx = line, move, arc, ...

A library uses a mathematical model and usually prefers absolute points in a consistent way. And programmers tend to export this interface to the user. Less work, less errors.
But users don't think so. They think mostly in relative terms. Actually, they need both ways dependent on the situation. And they like to use definitions that can use the values they measured.
Let's imagine my right arm. It's connected to my shoulder = upper right edge of my body.
It has two parts,
the first is a kind of cylinder starting at a circumference of 50cm, 27 cm long, 30° to the right, 20° to the front both relative to my body axis, it ends in a circumference of 40cm.
the second is another kind of cylinder continuing the first, 25 cm long, ending at a circumference of 20cm, and it is directed horizontally to the front.
As a human you choose terms that fit the purpose. So it's mostly relative, but sometimes absolute.
You would certainly use measurements directly if you would be allowed to. At least in systems where this is possible I do it all the time, saving a lot of calculations.
Which especially means, you nearly never use a radius, because you never measure it this way. You are thinking in diameters or in some cases in circumference depending on your equipment. And you also don't position a sphere by its center.
I think, jscad would benefit from having more "relative" or other "easy" features.
Users do not really like to calculate (e.g. start points), especially if it's not necessary. The computer should do the work as much as possible.
There should be alternative ways to describe something (the python way is academic, not natural).
Let's look at a rounded rectangle size 100x200 at position 500,600 with radius 5 at the top corners and radius 0 at bottom corners.
Now write that down as a path 1. in absolute coordinates and 2. in relative coordinates. Say we want to describe a tube running along this path.

absolute points, counter clock:

500|600
500+100|600
500+100|600+200-5
arc center=500+100-5|600+200-5 radius=5 start=0° end=90°
5|600+200
arc center=5|600+200-5 radius=5 start=90° end=180°
500|600
=> lots of calculations, lots of repeated terms

relative positioning, absolute directions, counter clock:

jump 500|600
straight 0|100
straight 200-5|0
arc 90° r=5
straight -(100-5-5)|0
arc 90° r=5
straight 0|-(200-5)
=> MUCH quicker to write, some thoughts remain about absolute coordinate directions

relative positioning, relative directions, counter clock:

jump 500|600 direction=0°
straight 100
turn 90°
straight 200-5
arc 90° r=5
straight 100-5-5
arc 90° r=5
straight 100-5
=> even faster, nicer to think, more human, no more fiddling with x|y when I really think in lengths

(what you really want...) define the length without the radius and declare how it should be rounded

jump 500|600 direction=0°
straight 100
corner 90°
straight 200 (!)
corner 90° r=5
straight 100
corner 90° r=5
straight 100
=> faster again, no more differences (think of r=3.7564), the computer does all the work

For users it's definetly a lot easier to define a figure by method 4 than by method 1.
From another POV:
My usual 3D designs are full of calculations to find absolute coordinates. I have to do trigonometrics, pythagoras etc. even for very simple designs. Everything that is non-rectangular and cannot be expressed with rotation etc. quickly becomes a real mess. While I am thinking very simple constructions I have to compile it for the graphical language.
So, please try to keep mathematics and programming separate from what a user wants to use. Or would you program in assembler?
How would a user construct something, if not forced by a language to think in a special way?
Also find NEW ways to help users...
jscad already offers some of this:
rotation around a point (in some other software you have to move the object to 0|0 rotate and then move back, wtf...)
chainHull (very simple, but very useful, especially 3D, I programmed this little function myself for openscad)
connectors (huge possibilities, but from my first contacts it seems to get not much love, it heavily depends on the connection points available)
some future wishes:

constraints

this is difficult because of dependency cycles
it may be simpler to construct sequential, with priority for earlier or later constraints, or prefer constraints with less freedom
attach things, define the free variables, e.g. attach edge A to edge B -> orientation, angle
for constructions it is not necessary to have living constraints, simply derive some of the parameters from the constraint
for animations it's a whole different thing

more of composed objects or constructions that are usual in constructions

cylinder with an array of lengths and diameters
cylinders along a path with an array of lengths, diameters, angles, may be elliptical, with rounded connections (this builds on chainHull)
tubes generally derived from path based objects with a wall thickness

I think that the 'scad api' is a good approach to this problem.
We can provide a number of apis which provide convenience to the user.
This can include things like lineTo(), which can then do something line incrementally building a path using a declarative approach.
Which means that we can focus on keeping the core clean for development, and focus on keeping user apis convenient for users.


Separate email from image generation

These can be two Lambdas, the Email triggered by the success of the Generation.

I think I set this up right, to have the first job write to SNS, and the second job triggered by the SNS topic.
I'll leave it open until I see it run tomorrow.

It didn't work right, but I got it sorted out today manually.
I'll leave this story open until I see it work on its own, on schedule.

Had a permissions issue with the SNS topic, and fixed it. Looks like this is done now.


[Introp][SwiftToCxx] Fix enum implementation test related to stub struct

@hyp Could you please check this pull request? Thanks.

@swift-ci please test and merge


Comment:Step-Basic-Real-time

They are not sent to Feathers REST ot HTTP REST clients

Fixed i8n https://github.com/feathersjs/feathers-docs/pull/676 . Thanks!


[Feat]: adding language translator

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
i want to add language translator which will promote multi language support for the website.
Priority
High
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@Tejashri-Taral can you assign me this task please

@Tejashri-Taral can u assign me this aswell.

@Tejashri-Taral can u assign me this aswell.

Currently we don't need this feature to be included. you can still raise and find new issues


The blankscans extension not working properly

Source information
The blank scanlation 1.4.39
Source language
English
Steps to reproduce
I go to the website try to click on a manga only to find out that it is updating but it falsely says site is erased but it's still up

Expected behavior
What should happen is me being able to access the chapters but I cant
Actual behavior
I can't access the chapters and this is what it says

Mihon/Tachiyomi version
1.7.4
Android version
Android 14
Other details

As you can see the site is still up
https://theblank.net/
Link to the website above
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Add a :+1: reaction to issues you find important.

Duplicate of #6648, it's already been reported before. Search the existing issues first before opening a new one, and acknowledging that you did when you clearly didn't.

Indeed I did not but as I looked at it now ,it still does not have a solution so what now?

It means to shut up, wait for a fix to close the issue. Migrate to a different source if you can't wait, fix the extension yourself, or find somewhere that'll fix things on demand for you. Don't unnecessarily comment bump in the existing issue.


log messages for connecting / disconnecting from slack

The bot logs "Carebot is running. Invite it to a channel to begin" once it's started and now also logs when a keyboard interrupt disconnects it. I don't believe that the slacker API wrapper library has functionality to listen for connection events.
Closing for now unless I misunderstood?


[SPARK-27414][SQL] make it clear that date type is timezone independent

What changes were proposed in this pull request?
In SQL standard, date type is a union of the year, month and day fields. It's timezone independent, which means it does not represent a specific point in the timeline.
Spark SQL follows the SQL standard, this PR is to make it clear that date type is timezone independent

improve the doc to highlight that date is timezone independent.
when converting string to date,  uses the java time API that can directly parse a LocalDate from a string, instead of converting LocalDate to a Instant at UTC first.
when converting date to string, uses the java time API that can directly format a LocalDate to a string, instead of converting LocalDate to a Instant at UTC first.

2 and 3 should not introduce any behavior changes.
How was this patch tested?
existing tests

cc @MaxGekk @srowen

Test build #104430 has started for PR 24325 at commit f4f6c79.

Test build #104430 has finished for PR 24325 at commit f4f6c79.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/104430/
Test PASSed.

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/testing-k8s-prb-make-spark-distribution-unified/9792/
Test PASSed.

Test build #104462 has started for PR 24325 at commit f026348.

Test build #104462 has finished for PR 24325 at commit f026348.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/104462/
Test PASSed.

thanks, merging to master!


Wildcard route does not work

https://github.com/Bubuhubu87/bc-jokes/blob/28f5380d823112e6014773dbe4c4d93695f31daa/jokes/src/app/app-routing.module.ts#L7

That's true! I forgot about '/Categories', to fix.

fixed


uri method on mocked Net::HTTPOK incorrectly returns nil

require 'webmock'
include WebMock::API

WebMock.allow_net_connect!

target_uri = URI.parse('http://www.google.com')

real_request = Net::HTTP.get_response(target_uri) # => #<Net::HTTPOK 200 OK readbody=true>
real_request.uri # => #<URI::HTTP http://www.google.com>

WebMock.disable_net_connect!
stub_request(:get, "http://www.google.com/").
  to_return(:status => 200, :body => "", :headers => {})

mocked_request = Net::HTTP.get_response(target_uri) # => #<Net::HTTPOK 200  readbody=true>
mocked_request.uri # => nil

I believe the method is inherited from HTTPResponse#uri.
This problem occurred on Ruby 2.2.0 and webmock version 1.21.0 as well as master (currently 9e7162f6).

@abevoelker thank you for reporting that. I'd need to investigate more to find out what the root cause is, unless someone investigates that before me.

No rush, I was able to avoid it by restructuring my code. Thanks for the neat library!

I have something similar going on in my Net::HTTP wrapper.
I attach the last_response instance variable to the response body to know the last resolved url in case of redirects. And it seems to have uri unset:
(byebug) head.instance_variable_get(:@last_response)
#<Net::HTTPOK 200  readbody=true>
(byebug) head.instance_variable_get(:@last_response).uri
nil

Is this something that could get made into a PR and merged in? I just hit it in a code where I want to follow redirects and update the model's url to the last one. Monkey patch still works, but feels like this should be fixed in core, no?

FWIW, I built a test project that reproduces the nil uri behavior: https://github.com/Automattic/webmock-uri-demo.

@Nakilon's monkey-patching suggestion worked for me. Thanks! RuboCop suggested a slight change, implementing the lambda call rule:
# Monkey-patch WebMock to work around response.uri not being set.
# See https://github.com/bblimke/webmock/issues/469
WebMock::HttpLibAdapters::NetHttpAdapter.instance_variable_get(:@webMockNetHTTP).class_eval do
  old_request = instance_method :request
  define_method :request do |request, &block|
    old_request.bind(self).call(request, &block).tap do |response|
      response.uri = request.uri
    end
  end
end

I've just reused this patch again for another project.
I've included it in the little gem of mine together with one useful dirty feature -- executing the blocked request on stub missing, so when exception prints that long code boilerplate it shows the real reponse to copypaste as a stub, not just empty status: 200, body: "".
https://github.com/Nakilon/webmockdump/blob/master/lib/webmockdump.rb

The fix is now released in version 3.21.1


Ability to navigate with ctrl+n/p

Thanks for this awesome program :)
I would like to have the ability to navigate with control+n/p which are popular vim keybindings for navigating up/down the list.

Interesting, I might need to sleep on it a bit, but seems harmless enough.

Hi @dlvhdr, I added this to the v1.2.0 release. Use the command below to upgrade.
curl -L "https://github.com/omranjamal/mono-cd/releases/latest/download/install.sh" | sh -

Have fun!


fix(privileges): run as non-root user by default

Welcome to Cryostat3! 👋
Before contributing, make sure you have:

[x] Read the contributing guidelines
[x] Linked a relevant issue which this PR resolves
[x] Linked any other relevant issues, PR's, or documentation, if any
[x] Resolved all conflicts, if any
[x] Rebased your branch PR on top of the latest upstream main branch
[x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[x] Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main

Fixes: #10
Description of the change:
This change allows an environment variable to be configured so that...
Motivation for the change:
This change is helpful because users may want to...
How to manually test:

Run CRYOSTAT_IMAGE=quay.io... sh smoketest.sh...
...

Seems like simply running as a non-root UID works out of the box. I assume SeaweedFS is already built with that expectation. I tested this with the Cryostat 3.0 smoketest (using rootless podman), crc via Helm chart, and kind via Helm chart.


Move rewards logic to hyperdrive/appconfig

Currently the rewards logic per Hyperdrive pool is in the UI. This can be moved to the appconfig where a manual whitelist can be maintained.

Closing in favor of #1562


Remove BTL_YIELD_SCALING

Remove parameter and mechanism, i.e., remove the wrong equation for yield calculation keeping only the correct one. Paper needs to be updated as well!

Solved by commit d0226fc1d12dd53b03864ecb68d985ce9d65aa44, TODO added to the paper for changes to be introduced in the text


"Shows" rail in partner profile overview has non-deterministic sort

Raised in Slack: https://artsy.slack.com/archives/partner-s-help/p1487001455004240
On a partner profile overiew page, successive page loads will sometimes yield a different result order for the Shows rail, resulting in closed shows appearing at the start of the carousel:

@briansw was this mean to be randomized? i have a feeling this might be intentional. if there is no explicit randomization code these would reliable appear in the same order they're returned from the server. if it's the case that it is intentional, perhaps we can split the shows into two groups, open shows and closed shows, and randomize within each while making sure that the first group always shows up before the second.

@1aurabrown @anandaroop Luca worked on this, but it seems like we'd want to prioritize current shows when possible and backfill with upcoming then past when there's room.

@briansw thanks, i'll try to do that.


On Hide callback

Replaced my debug menu with this, it's so much cleaner I like it a lot!, one thing I'm missing is a callback when the menu hides, currently I need to pause/unpause the game when I show the menu and that involves a couple of things.
Already added this on my local so let me know if PR's are welcome.

Go ahead and make a PR! That would be much appreciated.
On Fri, Jul 28, 2023, 20:57 afk @.***> wrote:

Replaced my debug menu with this, it's so much cleaner I like it a lot!,
one thing I'm missing is a callback when the menu hides, currently I need
to pause/unpause the game when I show the menu and that involves a couple
of things.
Already added this on my local so let me know if PR's are welcome.
[image: menu]
https://user-images.githubusercontent.com/1025751/256958215-34cd831b-4e39-4898-b24f-c4de65923914.gif
—
Reply to this email directly, view it on GitHub
https://github.com/macvogelsang/pd-options/issues/10, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCCDZQ5ZW3BASWKYJAGN5DXSRUYLANCNFSM6AAAAAA24FR3P4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I was also missing this in Strategies! I ended up fudging it for a one level menu, but my hack would not work for multi-level menus (that is, multiple options menus) and I'll also need this for my next little project.

My workaround was to do some checking in the BButtonUp function of my own input handler

BButtonDown happens in the Options inout handler
BButtonUp happens in your own handler

Done, thanks!


[Feature] Add "loading" state provider

Describe the solution you'd like
Similar to the theme provider, we should have a state provider. This will be used for route changes/data loading at the highest level.

Wont create, as its too specific to the app itself.


"required" attributes aren't pulled from $refs in schemata API description.

We've got an API schema set up using JSON API, and we've got some required fields which are shared between several different API methods. As such, we'd like to include a $ref  that defines the API attributes in one place. I dug a bit into the schemata templates which generate the documentation, and while it does attempt to recurse down some things, it seems that when it's deciding which attributes are required, it doesn't do so in a way which allows it to expand $refs.
I tried to strip down our schema a bit to illustrate the issue:
https://github.com/lukeschlather/nested-jsonapi-example/blob/master/docs/schemata/person.yaml
You can see for example that both the "create" and "batch create" operations are supposed to share the /schemata/person#/definitions/resource-without-revision properties, which define the "required" fields for a person. You can see here the generated markdown: https://github.com/lukeschlather/nested-jsonapi-example/blob/master/source/index.html.md
prmd combine --meta docs/meta.yaml docs/schemata/ | prmd verify  | prmd doc -o source/index.html.md

Note that all of the "Create" fields are listed as optional, rather than required.
I was curious if you could implement this in prmd, or if you have any suggestions about how to implement it I could also take a look.

I think attributes may nest those properties more deeply than I would expect? Outside that, one option might be to leave the attributes as they are, but just move the requires directly to the shared definition and pull it out to be along-side the other properties. My suspicion is definitely a mismatch with expected nesting levels of some sort, fwiw.


Hey I want list to navigate to data of that particular item

Hey I want list to navigate to data of that particular item  how can I do that please guide me

I am not sure if it is possible with the current version. Although I am guessing providing a list view controller might solve it.

You can add your custom ScrollController using scrollController attribute which will help in scrolling to an item. Try this blog for reference.
Thanks for using the package 🎉  Feel free to reopen issue if you face similar issue.


random data not working properly

I have integrated campaign chart which display the random data but it's not working properly some time it doesn't update the data. I have used react-chartjs2 library in my react application. Here is my codesanbox link https://codesandbox.io/s/rrlpr2nxjm

@ghost this seems to be related to how you are generating the random data. The conditional you have defined does not always pick a different object key, so some intervals will indeed not return a different dataset.
Closing for now.


Expose parameter to allow setting explode=True

Should be able to set explode=True in set_defaults() or show() so you don't have to keep enabling the explode checkbox after each render

@JonnyHaystack There is too much interaction with the animation part to add it to set_defaults.
It will only be available as a parameter to show, but not  as parameter to show_object (doesn't make sense to explode one object).
Would that work?

Ha, found a way to control the dependency between explode and animate.
The explode setting will be available in VS Code Workspace config, with set_defaults and with show.
Just not with show_object

Fixed in 1.0.0


Merge USAGE.md into comprehensive README.md

Actually, we'll do it the other way around: merge comprehensive README into USAGE and other appropriate files, replace README with a simple contents index


Consider changing flow id to from, to pair in the CSV files

What and Why
The graph is indexed on the (from,to) pair labels. The ids are not used or stored explicitly.
Implicitly they are required because the profiles and partition CSVs use the flow ids. This means that to add this information to the graph structure, we have to use the flow_data vector and look at its keys. This is the reason for the ugly code that @datejada comment in the other PR: flow_data[id][1]. The 1 is referring to the key.
Possible Drawbacks

Requires changing all files.
The data becomes more verbose.

Related Issues
No response

We always come back to the dilemma of using number (i.e., ids) of label (i.e., name) in the formulation 🤔
As we have discussed before, Ids are fine as long as we can have the names when we have an infeasible problem.

This is not in the model, this only affects the CSV files.

Ah! I get it now. So, let's change the files. Using the ids was utterly arbitrary, and we don't have any particular reason right now to use them instead of the ids. We are on time to do it.
So, if I get it right now, we have this in the CSV files for the assets profiles:
,,,p.u.
id,rep_period_id,time_step,value
3,1,1,0.11
And we want this:
,,,p.u.
name,rep_period_id,time_step,value
wind,1,1,0.11
Is that correct? And, of course, change the i/o, graph creating, etc...

Yes!

@abelsiqueira If you are not working on this, I can take it over and do this change 😄

I started it and - hopefully - will finish it soon. There are more changes that I predicted.


Update dockerclient to support network create --internal flag

Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM


first-session

acceptance test for user creation and its controller

Please remove the files from the .vscode folder. The ignore file was fixed by this PR: https://github.com/drogojan/dotnetdays-outside-in-tdd-workshop/pull/3

Please remove the files from the .vscode folder. The ignore file was fixed by this PR: #3

I've removed then but the check-in that fixed the git ignore file was done in the upstream and my forked repo doesn't see it (when syncing with git pull origin dojo-starting-point it's not bringing anything)

Please remove the files from the .vscode folder. The ignore file was fixed by this PR: #3

I've removed them but the check-in that fixed the git ignore file was done in the upstream and my forked repo doesn't see it (when syncing with git pull origin dojo-starting-point it's not bringing anything)

use the upstream remote to sync - git pull upstream dojo-starting-point


[LANG-1572] green implementation CharSequenceUtils.indexOf

also refine javadocs.

Coverage decreased (-0.003%) to 94.672% when pulling 7335badbc7e4654da6ef1487619a81346310c26c on XenoAmess:remake_indexOf into 1621a23d2bd99df845abae0beea0add84135a49d on apache:master.

Hi @XenoAmess
Rebase with master

@arturobernalg Got it. I got a really busy job recently, sorry for the late reply.

@garydgregory rebased. please find some time to review. thanks.

Would you, please, be so kind to rebase again? Sorry for that inconvenience.

Hm, why do we want something so much more complicated?

Hm, why do we want something so much more complicated?

mostly for performance reason

Would you, please, be so kind to rebase again? Sorry for that inconvenience.

@jochenw rebase done.

@aherbert
fixed.
please re-review.
thanks.


Poor performance while editing large files

Describe the bug
I have a file with a bunch of messages, the model is essentially the same config as for the authors in the netlify-cms dev-test.
While editing a message, the performance is very poor and results in a very janky UI.
I'm assuming this is due to the onChange handler being called on every edit, which results in validation, and re-rendring of the entire component tree (please correct me if I'm wrong, I do not have a good overview of the internals in netlify-cms).
If my assumption is (somewhat) correct I propose storing the values while editing locally in the widgets instead of calling the onChange handler on every event, and only call the onChange handler in an onBlur handler instead.
To Reproduce
The easiest way to reproduce is to start the dev-test locally with a lot of authors manually configured:

Clone my fork: https://github.com/sondrele/netlify-cms
In the cloned directory: yarn && yarn develop
Navigate to: http://localhost:8080/#/collections/settings/entries/authors
Start editing the name of the first author

Expected behavior
Editing content should not have that kind of performance problems.

Screenshots

I've included a gif of what it looks like when writing. Not sure how obvious it is from the video, but I can testify that I'm hitting the keys way faster than the characters appear :)

Applicable Versions:

Netlify CMS version: 2.3.3
Git provider: any
OS: Linux
Browser version: Firefox 65

Node.JS version: 11.1.0

CMS configuration
Here: https://github.com/sondrele/netlify-cms/blob/master/dev-test/config.yml
Authors data: https://github.com/sondrele/netlify-cms/blob/master/dev-test/index.html#L42
Additional context

@sondrele did you ever find a solution for this?

@erquhart would you be able to point me in the right direction in the repo and I will try and fix

Closing this in favour of https://github.com/netlify/netlify-cms/issues/3415

this is still happening I wouldn't have closed it, neither this one or #3415 seems to be fixed, my project needs a massive refactor for working around this putting each entry in its own file, when can we expect a fix please?

Hi @lupelius, we can track this issue under #3415.
Keeping duplicate issues open just makes it hard to address them.
The CMS is a community based project, so we would love if someone could contribute a fix for this.


Schema docs for Policy_Action_disable and Policy_Action_Persist_disable are missing

Environment

Application Services Version: 3.25
BIG-IP Version: 15.1.2

Summary
Per https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/refguide/schema-reference.html?highlight=persistencemethods#policy-action-persist
There is no documentation for schema elements Policy_Action_disable or Policy_Action_Persist_disable also, the actual schema does not seem to contain the values necessary for auto-completion within VS.Code.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "class": "Endpoint_Policy",
    "strategy": "first-match",
    "rules": [
		{
            "name": "appx_rule",
            "conditions": [
                {
                    "type": "httpUri",
                    "pathSegment": {"values": ["appx"]},
                    "index": 1,
                    "event": "request"
                }
            ],
            "actions": [
                {
                    "type": "persist",
                    "disable": {},
                    "event": "request"
                },
                {
                    "type": "forward",
                    "select": {"pool": {"use": "appx_pool"}},
                    "event": "request"
                }
            ]
        }

Observe the following error response:

Expected Behavior
Please provide the language construct for the "disable" actions.
Actual Behavior

Buried in https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/3.19/declarations/all-properties.html
the syntax is simply
"disable": {},

I believe the schema here is actually incorrect. I believe it was intended to be a boolean value.

Yes, that would make more sense, but at least I can get this deployed and I have confirmation that it addresses our need.

If you are still looking for a fix, please reach out to us at<EMAIL_ADDRESS>


Various improvements

Hi, thanks for the nice plugin.
Here are some improvements I found when using the plugin in the last few days. Each commit contains a single change. The last two commits are based on my personal preference, so please feel free to exclude them.

Hi! Glad it could be of some use :) I totally agree with your last two commits, in fact I've had it set to open folds via the toggle since I made it. And reverse sorting it works better I think.
I'm pretty new to making vim plugins so any and all contributions like this goes a long way, thanks!


Feature Request: Ability to set input voltage manually for PD Power banks (i.e. 12v, 15v ,20/20.2v)

I can see the option to manually select the number of cells is being incorporated, but what about those using a pd power bank -some input 12v,15v and even 20v or 20.2v) Thanks

I am sorry, but would be the point of that? Surely the power bank is voltage regulated, it will output the same voltage until it doesn’t.

I was thinking if there has been an addition to "add ability to manually specify lipo cell count", that a similar specification of PD input voltage might be required to help with low battery alarms etc. as PD voltage would otherwise be detected as a lipo within the same voltage range, but then trigger low alarm warnings etc. when not necessary...


Crashing after about 10 minutes

I am on a Q338 with one revision back firmware. 1742 I believe, I have the pulse update. I am running a Mac M3 Pro.
It works perfectly for about 10 minutes and then crashes and then if I reopen it sometimes it will talk to reaper sometimes not. However it still crashes. Once reopened it seems to crash faster, I haven't really timed those exactly.
I am running the latest build you have published.

Hi- Thanks for reaching out. Is it generating a crash log? If so can you send me the information from that crash log. Just double checking- you are running the ARM64 version of the application, correct?

Seems to only do it when connected to console. I ran it for a while last night, of course no crashes. I will report back this weekend. Thanks for getting back to me.

I have attached the crash dump. Hope this helps.
Crash Log.txt

Just FYI- this crash log is super interesting to me- for some reason your software is spooling tons and tons of threads which I imagine is leading to the crash. I've got this running on multiple shows on both mac and windows and they are never sitting at more than about 6-7 threads at a time- but they are all on Quantum 7s. So I've got to imagine it's something related to the Q338. I've got some ideas- but I've got to get in front of a 338- so I'll try to do that ASAP and get back to you. Sorry- I'm just get getting a show up and running here but I'll get back to this ASAP.

Could you try the latest build? I can't recreate your issue here, but I have a thought about what it might be about and I've made a change in the code accordingly. https://github.com/jms5194/Digico-Reaper-Link/releases/latest

Thanks for looking into it  I just got back from a fly date on an SD12-96 running 1972 or w/e the latest firmware is. Had the same problem as I have on the Q338. I think it might have to do with my Klang card? Any thoughts to that? Interestingly enough your old version of the software works perfectly on both the Q338 and SD12-96 with Klang. I am loading in a show right now and will check this new release and let you know!

Ok, Update from the current release. I recorded for 20 minutes - It never crashed. However as time goes on it gets really slow to respond to the markers. It hasn't missed one but it's a solid 3 seconds from the time I hit next to the time it sets a marker in reaper. It was as instant as always during the first 5 or so minutes.

And then it crashed, Attached is the log,
Crash Log 2.txt

Oh man- I totally understand what's happening now and I have to thank you because in theater world we never use Klang and I never would have found this issue myself. I assume Klang control is running on the same machine as Reaper? Are they set up to send/receive on different ports? (That's mostly just as a point of interest for me). In any case- I have two possible solutions to this- one's easy and one's harder, so I'm going to implement the easier one later today and I'll upload a release for you to test. In any case, next time you have a fly gig that takes you to NYC, I owe you a beer.

So yes and no. Ports are different. Currently I am running Reaper on one machine and Klang on the other. However, even when I do a blended machine I have the same results. My assumption is that Klang is also OSC and the network is just flooded when a scene changes. I am currently doing a 4 night stay and I can get you any information you need from my end. Sadly I was just in NYC, but yes the next time I am around we can get together. I VERY much appreciate all you've done with this. P.S. It only lagged during a 3 hour show last night, didn't crash. Don't know if that's helpful.

OK last night I was only able to get through about half of the show with it. Crashes a lot. Also crashes as soon as you open it the first time. Also noticed it was VERY laggy. I didn't have time to catch the crash report, if it does it today where I can export it I will send it to you.

Crash Log 3.txt

Ok. Bummer. Harder fix it is. Working on it.


problem with current grunt-contrib-imagemin version in package.json

Hi Guys,
It seems that there is a problem with the current grunt-contrib-imagemin, when you do a build it fails.
You'll need to upgrade the version from 0.9.4 -> 1.0.0 and then the grunt builds work again.
Tom

Hi @tommaton, we are aware of the issue & have fixed it in the imminent v6.0.0 release. I'm going to close this because dupe of #65

@tommaton have you seen the work we're doing for v6.0.0? the new generator can be installed using npm i -g generator-kickoff@next or the next branch here has everything too. We are very close to releasing it and would value your opinion.

@mrmartineau never saw the new version, I'll take a look as just starting a new project so will give it a go and give you any feedback I have.

@tommaton That’d be cool – we’re looking at releasing it early next week, so if you find any issues let us know :+1:

@mrmartineau @ashleynolan when updating the generator to v6 it still seems to be installing v5 when running the yo kickoff command.
I've uninstalled the generator, cleared the npm cache etc and then reinstalled.
Any ideas?

umm, not sure. Was going to suggest you uninstall. I am about to push another @next version very shortly so I'll respond again when I do. Sorry mate

Hi @tommaton, I have just published a new, tagged version of the generator. Please try installing it again..


Updated Get-PnPAAdUser to allow retrieval of all users

Type

[X] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
N/A
What is in this Pull Request ?
Updated Get-PnPAAdUser to allow retrieval of all users. The current implementation only returns the first 1000 users in an Azure Active Directory due to the default EndIndex of 999 in the underlying PnP Framework. Changed the implementation in PnP Framework through PR #157 to allow the EndIndex to be nullable in which case it will return all available results from Azure Active Directory. For backwards compatibility in PnP PowerShell when using Get-PnPAadUser, I have made it so that the current cmdlet still returns the first 1000 results and only if -EndIndex $null is provided it will return all users.

@erwinvanhunen Since the documenation stated that it should return ALL users already but in reality it only would return the first 1000 users, perhaps its even better to rewrite this PR to just "fix" it instead of introducing -EndIndex $null that needs to be used to retrieve all users? Don't actually think fixing it would fall into the break backwards compatibility scenario. What do you think?


CSV processing bug fixes

This fixes three problems:

CSV processing could fail silently
CSV processing could fail due to a race condition, where the Sidekiq processing job ran before the data to be processed is available in every Mongodb node
Tests that were incorrectly passing

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Superseded by #681. I've copied over what's necessary to fix things. There were a couple of changes in this PR that weren't necessary to fix the bug or the build. Thanks Bill! ⭐


How to integrate inside ProcessStep and/or Pipeline?

Hey, I'm trying to integrate with a pipeline that I'm building that's based on the Abalone example (https://docs.aws.amazon.com/sagemaker/latest/dg/define-pipeline.html)
I think that the .run method executes things outside the pipeline. Is there a way to orchestrate this inside the Sagemaker Pipeline?

Hi @jonathanglima, thank you for taking an interest. At the moment, SageMaker workflow cannot run this sample framework processor because this processor does not override the get_run_args().

Hi, I'm going to close this issue, since the get_run_args() is implemented (and tracked) by the SageMaker PR #2251.


LightCycles don’t need to know about android.

For applications, it allows presenters (LightCycles) to use views - which can be implemented by Fragment or Activity classes.

Dispatcher: dispatch the Android life cycle (Activity, SupportFragment, Fragment) to LightCycles.
Host: the activity/fragment as viewed by the light cycle. It can be a view.
LightCycle: notified of the life cycle with the Host as a parameter.

https://github.com/soundcloud/lightcycle/issues/48

Just for confirmation, the view here is view in MV* Architecture right? not Android View?
While in this topic, do you think supporting Android View would be a good option?

@NikoYuwono

The View is indeed the V of the MVP.
I never thought about adding light cycle to Views, neither to Application because we just don't need it at SoundCloud. I gave it some thoughts and I can't tell it would be easy: it would mean subclassing a view to use a ViewDisaptcher, unless the view tree can be used.

Long story short, it is not part of the V2 roadmap as of now.

@glung

Sorry but I don't think I can come with better name, let's keep with this rule to use View to mention V of the MV* architecture and Android View for Android's View.
Yes that's why I ask, would we got a merit from it or not, for now I don't think it can be really useful use case for the user, so let's let it go for now.

Can you release this, since there are more people looking for this feature?

Check 1.6.0. I will communicate the release after more testing but you can already use it


Better error message on import when missing common dependencies

Steps to produce the error:

docker run -it ubuntu:latest /bin/bash
apt-get update --yes && apt-get upgrade --yes && apt-get install git build-essential curl wget --yes
wget https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh
bash Miniconda3-latest-Linux-x86_64.sh -b
export PATH=/root/miniconda3/bin:$PATH
hash -r
pip install dask
python -c "import dask.array"

Output:
Traceback (most recent call last):
  File "/root/miniconda3/lib/python3.6/site-packages/dask/array/core.py", line 21, in <module>
    from cytoolz import (partition, concat, join, first,
ModuleNotFoundError: No module named 'cytoolz'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/root/miniconda3/lib/python3.6/site-packages/dask/array/__init__.py", line 4, in <module>
    from .core import (Array, block, concatenate, stack, from_array, store,
  File "/root/miniconda3/lib/python3.6/site-packages/dask/array/core.py", line 26, in <module>
    from toolz import (partition, concat, join, first,
ModuleNotFoundError: No module named 'toolz'

You probably want pip install dask[array] or pip install dask[complete]. See http://dask.pydata.org/en/latest/install.html#pip.

I see. But should it not give a more meaninful error?

Sure. That'd be a good issue for the scipy sprint this weekend (or any new contributor :) ).
The code should look similar to that for dask.distributed (https://github.com/dask/dask/blob/master/dask/distributed.py#L6-L11).
Should look something like a try-block around all statements in __init__, and on ImportError raise a new ImportError explaining the common install issue here (users installing pip install dask instead of pip install[something]), with the original error message concatenated at the end.
We'd want to do this for dask.array, dask.dataframe, dask.bag, and dask.delayed (around the import for toolz) - this mirrors the structure in extras_require: https://github.com/dask/dask/blob/master/setup.py#L10-L17.

Fixed by #3771. Closing.


[Enhancement] backtrack aimbot

Backtrack aimbot would be a good enhacement.
I need help also, for some reason RCS misses the first shots always

There's no recoil in first shot?

There's no recoil in first shot?

First shots not first shot
First 4 shots are mostly misses


Fancybox Label Alignment

Hey All,
Awesome plugin, love this thing! Running into a unique issue when using this map selector inside of a fancybox where the labels don't line up with their backing. I've added the map to the regular page and it functions perfectly but in the fancybox there seems to be some kind of initialization issue happening, please check out the image.
If there's any chance someone can give me a hand with this today it'd be hugely appreciated.

+1

I have this problem as well, I think it could be due to bootstrap or that the labels cannot be places properly when then map is hidden.

+1 would like to see this issue resolved.

Okay did some research:
The misalignment of the labels is caused by loading in the map when map isn't visible.
An example of this could be that you've loaded your map in a pop up, modal, carousel or simple the element just isn't visible on it's own because of your custom code.
To fix this, rather than initialing the map on page load, initialize it when the element becomes visible.
Using the setTimeout() function could help with this (because all animations/fades need to be completed before loading in the map.


Header font appears very dull and small

I have not changed any font settings from your theme, and I feel the header I am seeing is not the same as what you get on your demo.  Here is what I am seeing..

Additionally, I noticed on mobile devices the normal paragraph text is super small. I would say this needs to increase in size a bit for general readability on mobile devices. How can I best do that with your theme? I assume I need to change the CSS somewhere but it would only need to change for mobile devices.
Thanks for the help.

Not sure on the difference in font you're seeing between the demo. The font family set uses "system fonts" so depending on the environment you're looking at it will change as macOS, Windows, Linux, Android, etc. all use different system fonts.
As for bumping up the font-size. Do a search through the closed issues. I've covered how to change them too many times to count.

Closing as this is a duplicate of #1352, #1219, and #1184.

Thank you. You respond to these questions/bugs super fast! Appreciate it.


[Docs]: Release notes for 1.25.0-rc1

Is there an existing issue already for your request/idea?

[x] I have searched for an existing issue, and could not find anything. I believe this is a new documentation enhancement to be evaluated.

What problem in the existing documentation this issue aims to solve?
Add release notes for CloudNativePG first RC version of 1.25.0
Describe what additions need to be done to the documentation
No response
Describe what pages need to change in the documentation, if any
No response
Describe what pages need to be removed from the documentation, if any
No response
Additional context
No response
Backport?
No
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Created a spreadsheet: https://docs.google.com/spreadsheets/d/1gVxVkQW6fveCUyi1d6fPYANvXRAQ99rYhEyHdFCecVI/edit?usp=sharing

Started drafting content here: https://docs.google.com/document/d/15AzdCoJRMtXxc8fresLpYdwX6DxOlKWJtdX0pcimpgU/edit?usp=sharing


Rebased onto upstream master

Did you mean to open this PR on your fork, @rogthefrog ?

I did—please ignore. Sorry.
From: Thomas Orozco
Sent: Wednesday, September 27, 2017 0:55
To: aptible/docker-logstash
Cc: Roger; Mention
Subject: Re: [aptible/docker-logstash] Rebased onto upstream master (#16)
Did you mean to open this PR on your fork, @rogthefrog ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

:+1: no worries


Improve the documentation about the filtering support for the organization list endpoint

Describe the issue:
$subject

Fixed Documentation: https://is.docs.wso2.com/en/next/apis/organization-mgt-rest-api/#tag/Organization/operation/organizationsGet


Apply CELL_PAD to replace

replace has a somewhat recent option to specify padding.

I notice a few other differences vs the openroad regression tests:
openroad applies a left and right pad, and openlane applies only one (left for cts, right otherwise). This might make a small improvement to routability I think. Could we apply CELL_PAD/2 left and right?
openroad also applies 2x padding in global placement vs detailed placement, I'm not sure why.

I wonder if this will cause some working configurations to fail during global placement

It does seem to cause previously passing configurations to fail.

@antonblanchard: We set these values with experimentation over the test set we provide in the repo: https://github.com/efabless/openlane/tree/master/designs I don't think it will make a huge difference if the padding is to the left or to the right or divided over both. However, if you have a test case that proves the significant difference we'll be happy to reconsider.
It seems that replace's cell padding didn't work well with the current designs and flow setup (maybe it needs to be 2x as you said?). Perhaps we can look into this later. But, for now, I will not be able to merge this PR (check the CI for more details).

Agreed, lets close it for now and we can try tuning it sometime later.

Agreed, lets close it for now and we can try tuning it sometime later.


Module not found

Hi, when trying to install the plugin as per the docs, I was getting the following error using Laravel Mix (Webpack):
Module not found: Error: Can't resolve '@ryangjchandler/alpine-tooltip' in …

I fixed it by editing your package.json directly, changing "main": "dist/alpine-tooltip.cjs.js" to "main": "dist/module.cjs.js" and "module": "dist/alpine-tooltip.esm.js" to "main": "dist/module.esm.js".
I would create a PR, but unfortunately, I have no idea to do this properly. 😅 Also, maybe I was just doing something wrong in the first place. Anyways, I hope this will help someone else.

Ah, rookie error on my part. I'll get this fixed & a new release tagged - thanks @RobertCordes!

Closed by https://github.com/ryangjchandler/alpine-tooltip/commit/d5cb1b09381df2af9cc47d452d910bad2b6a015a

@RobertCordes Please upgrade to v0.1.2 which has the correct entrypoints.

Just did. Thanks for the fix!


-k option for API key not working, needs swapped

https://github.com/GreyNoise-Intelligence/pygreynoise/blob/e818d952f006723cd06dea26789d02b54c553038/src/greynoise/cli/subcommand.py#L39
Click documentation seems to suggest the long argument should always be first so "--api-key", "-k" instead.

@Obsecurus I see examples in the documentation that use the same ordering for short/long options. Could add more information about what is not working and how to reproduce it?

@Obsecurus Since it looks like now it cannot be reproduced, I'm going to close this issue. Feel free to reopen it if it happens again.


No module named '_swigfaiss'

any idea how i can solve this?

OutOfRangeError                           Traceback (most recent call last)
 in
11 ffn_weight_file=None,
12 bert_ffn_weight_file=bert_ffn_weight_file,
---> 13 embedding_file=embedding_file)
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/docproduct/predictor.py in init(self, pretrained_path, ffn_weight_file, bert_ffn_weight_file, embedding_file)
208             pretrained_path=pretrained_path,
209             ffn_weight_file=ffn_weight_file,
--> 210             bert_ffn_weight_file=bert_ffn_weight_file
211         )
212         self.faiss_topk = FaissTopK(embedding_file)
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/docproduct/predictor.py in init(self, hidden_size, dropout, residual, pretrained_path, batch_size, max_seq_length, ffn_weight_file, bert_ffn_weight_file, load_pretrain, with_question, with_answer)
78         answer = 'fake' if with_answer else None
79         self.predict(questions=question, answers=answer, dataset=False)
---> 80         load_weight(self.model, bert_ffn_weight_file, ffn_weight_file)
81
82     def _type_check(self, inputs):
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/docproduct/predictor.py in load_weight(model, bert_ffn_weight_file, ffn_weight_file)
21 def load_weight(model, bert_ffn_weight_file=None, ffn_weight_file=None):
22     if bert_ffn_weight_file:
---> 23         model.load_weights(bert_ffn_weight_file)
24     elif ffn_weight_file:
25         loader = checkpoint_loader(ffn_weight_file)
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/tensorflow/python/keras/engine/network.py in load_weights(self, filepath, by_name)
1468         save_format = 'h5'
1469     if save_format == 'tf':
-> 1470       status = self._trackable_saver.restore(filepath)
1471       if by_name:
1472         raise NotImplementedError(
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/tensorflow/python/training/tracking/util.py in restore(self, save_path)
1057     try:
1058       object_graph_string = reader.get_tensor(
-> 1059           base.OBJECT_GRAPH_PROTO_KEY)
1060     except errors_impl.NotFoundError:
1061       # The object graph proto does not exist in this checkpoint. Try the
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py in get_tensor(self, tensor_str)
428         from tensorflow.python.util import compat
429         return CheckpointReader_GetTensor(self, compat.as_bytes(tensor_str),
--> 430                                           status)
431
432     swig_destroy = _pywrap_tensorflow_internal.delete_CheckpointReader
~/anaconda3/envs/tfgpu/lib/python3.7/site-packages/tensorflow/python/framework/errors_impl.py in exit(self, type_arg, value_arg, traceback_arg)
546             None, None,
547             compat.as_text(c_api.TF_Message(self.status.status)),
--> 548             c_api.TF_GetCode(self.status.status))
549     # Delete the underlying status object from memory otherwise it stays alive
550     # as there is a reference to status from this from the traceback due to
OutOfRangeError: Read less bytes than requested

It looks like it's not finding Faiss. Did Faiss install without any errors?

Am I missing anything? Seems the issue content dose not match the title.
It seems the OutOfRangeError is caused by loading the model weight. Please make sure the model weight file is good and its path is correct.

Getting the same error.

ModuleNotFoundError: No module named '_swigfaiss'

Getting the same error.

ModuleNotFoundError: No module named '_swigfaiss'

Can you show the full error message? This looks like a FAISS install error.

try the api: https://pocket-derm.firebaseapp.com/

conda install -c pytorch faiss-cpu/gpu for faiss install. It worked for me

@masapasa did you install faiss on windows or mac or linux?

Faiss is not supported on windows

@harshit-saraswat try the api you dont have to install anything just send a post request :)


PowerShell Guide Container.start.ps1 should be the entry point

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


issue with webpack + ES6/babel and sourceMaps

Hey!
When using transform-loader with babel looks like it breaks the source maps.
Here's my config :
loaders: [{
    test: /\.(js|es6|jsx)$/,
    exclude: /node_modules/,
    loaders: [
        'transform/cacheable?brfs',
        'babel'
    ]
}]

when transform enabled, chrome directs me to the ES5 transpiled code.
if i comment the line, i have the correct sources in chrome.
any idea how to fix this ?
Thanks!

(tried with devtool: 'eval-source-map', and every other option without luck)

This doesn't work for two reasons:

brfs doesn't emit SourceMaps
transform-loader doesn't support input SourceMaps (nested SourceMap)

Thanks for the feedback ! so is there any way to fix this ?
when using browserify/brfs i think i had the sourcemaps working;

when using browserify/brfs i think i had the sourcemaps working;

I tried it again, but brfs doesn't emit a SourceMap. coffeeify does.

ok thanks man!

for a quick and dirty workaround: https://github.com/thaiat/webpack-babel-sourcemap

@thaiat solution worked for me ! move babel to the first loader and add a query block
module: {     loaders: [{       test: /\.js$/,       loader: 'babel',       exclude: /node_modules/,       include: __dirname,       query: {         retainLines: true,         cacheDirectory: true       }     }]


regl.buffer type option need to be documented.

The type option of regl.buffer is not documented in API.md. But it should be.

I will take care of this.

Resolved with PR  #293.


Regtest 2.0:   sleepless launch, graceful shutdown

I added conversations on the PR itself, but none were blocking:
https://github.com/zingolabs/zingolib/pull/26
Please check out those comments for ideas for minor improvements.
This issue is to track two tasks:
(1) sleepless launch
(2) graceful shutdown
sleepless launch:

remove all sleeps
report initiation of zcashd launch
report initiation of lightwalletd launch
provide proof of life (e.g.  periods regularly showing up on the screen...)
monitor progress of zcashd (launch)
monitor progress of lightwalletd (launch)
when lightwalletd provides complete launch signal, expose UI
IF zcashd is not ready:

provide concise report of zcashd state in UI
expose only relevant commands

IF zcashd is ready:

provide 1-char "zcashd ON" "light"
expose all commands

graceful shutdown:
When the user instructs the zingo-cli to shutdown it must:
* announce that it was instructed to shutdown
* issue proper shutdown commands to each spawned process
* report progress to the user
* monitor the behavior of its spawned processes
* report success-or-failure of spawn shutdowns
* shut itself down

ACK.
I am going to start two new issues for these. I plan to begin with graceful shutdown.


Think about cloning behavior of different errors.

By cloning an instance a new stack should be generated. By cloning a prototype, no stack must be generated, so we need an if-else block to check this or a prototype check can do the same: this === this.constructor.prototype.
By native errors we clone only the wrapper, so the stack should stay the same, just as the name and the message and other properties. By the current implementation the build() is empty, so it works properly now.

Closing this in favor of #94 , since I moved the error api to a separate project...


GPU

Maybe also check for discrete GPU utilization (and TDP if we can find where it is) and adjust power plan based on that (also adjusting clockspeed if possible)

Yes!!

GPUs take up a lot of power

Oh maybe we can also limit the FPS to an set number when on battery for games

I feel like we should also work on this now because the gpu soaks up a lot of power

on laptops with dedicated nvidia chips, we can do like a mux switch warning (or nvidia optimus) because it's about 2 times less efficient with it off
we can't change it though because it's locked by the BIOS chip, so the user will have to do that

I feel like we should also work on this now because the gpu soaks up a lot of power

Sounds good!


selectively add back 508 checks

Pull requests into Cypress require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[ ] This pull request describes why these changes were made.
[ ] Internal ticket for this PR:
[ ] Internal ticket links to this PR
[ ] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Admin

Checklist

Measure_test

Products New

Products Show

Master Patients

Vendor Patients

Login

Vendors

view xml


Querying vsphere_vm table results in error

Attempting simple query of the vsphere_vm table results in errors
`steampipe query "select name from vsphere_vm;"
Error: rpc error: code = Internal desc = hydrate function listVms failed with panic runtime error: invalid memory address or nil pointer dereference (SQLSTATE HV000)`
plugin-2023-02-22.log

Hey @matthewalr I'm unable to replicate your issue unfortunately, is there any thing else you can tell me about your setup?


dropwizard-db 0.8.1 closed connections are returned to pool

Hello guys, we have migrated to Dropwizard 0.8.1 from 0.6.2 recently and faced the following issue while using JDBI:
after database restart (Postgres), our application wasn't been able to restore connectivity to DB, all connections were closed and every time, after failed attempt to execute some SQL statement, returned back to pool. Pool is configured with all tests (on borrow, on return, on connect, while idle) disabled. Dropwizard 0.8.1 is using Tomcat DBCP 8.0.18 vs 7.0.37 on 0.6.2. Database configuration we are using is the same for both Dropwizard versions.
Dropwizard 0.6.2 is using 7.0.37 DBCP and is able to restore from DB restart, because PoolableConnection class performs explicit check if underlying SQL connection is closed:
 public synchronized void close() throws SQLException {
    if (_closed) {
        // already closed
        return;
    }

    boolean isUnderlyingConectionClosed;
    try {
        isUnderlyingConectionClosed = _conn.isClosed();
    } catch (SQLException e) {
        try {
            _pool.invalidateObject(this); // XXX should be guarded to happen at most once
        } catch(IllegalStateException ise) {
            // pool is closed, so close the connection
            passivate();
            getInnermostDelegate().close();
        } catch (Exception ie) {
            // DO NOTHING the original exception will be rethrown
        }
        throw (SQLException) new SQLException("Cannot close connection (isClosed check failed)").initCause(e);
    }
   ...

There is no such check in DBCP 8.0.18.
Have anybody else faced this? We don't want to enable on borrow/on return validation checks just to overcome it.
P.S.: I've asked in Tomcat mailing list why they have removed this underlying connection check, will update this issue with response.

My question got answered on Tomcat mailing list and it don't relate to DBCP.
The thing is that dropwizard-db 0.6.2 is using Tomcat DBCP's GenericObjectPool, which have this closed connection check upon return to pool, while dropwizard-db 0.8.1 is using Tomcat JDBC ConnectionPool which is more straightforward and don't have such check.
What was the reason of moving to Tomcat JDBC pool from DBCP?

Because of its better perfomance in concurrent environments, little size and more configuration options.  I would recommend to enable connection validation on borrow and to set an apropriate validation interval.

@arteam Is this going to be efficient? I thought that the Dropwizard 0.6.2 approach was to be as performant as possible.

@pparth I guess it's more about correctness than performance. If we know the database can be restarted during the work of the application, we should work in assumption that a connection can be invalid in any time. But we don't want to make validation every time a connection is borrowed, that's why a validation interval is needed.
By default Dropwizard configures the pool to make checks only in background, so this
should be most performant approach.
I guess, we can get the old mode by enabling a check on returning a connection to a pool, but I've never used this feature.

@arteam may be you could add validator, validatorClassName properties from Tomcat JDBC pool properties to DataSourceFactory, then we could write our own validator instead of just using SQL validation queries.

I can create PR with corresponding changes if you will take a look at it

Sure, I will gladly review it.


Define meta-data pipeline for PlantCV, not embedded in filename

Ideally this will be embedded in meta-data uploaded with images
@nfahlgren @max-zilla will discuss
meta-data will be provided at time of upload

I have some code I can reuse for this in the Clowder uploader we build for the Danforth Center data stream. Below are the metadata terms and formats I have proposed using for uploads to iPlant Data Store and BisQue. Happy to rename/reformat, add/subtract as necessary as the data standards committee develops a framework. Examples below are from our published Setaria experiment.
snapshot_id (internal ID number for the group of images taken for one plant at one time point)
plant_barcode (internal alpha-numeric code for each plant)
plant_age (units: days after planting)
zoom (camera zoom setting, units: X optical zoom)
perspective (camera position: side-view or top-view)
rotation_angle (the angle the plant was rotated for the picture. For side-view images we have 0, 90, 180 and 270 degree images, units: degrees)
camera_type (visible/RGB, near-infrared or photosystem II fluorescence)
treatment (e.g. 100%: 217 ml water (48% VWC), 66.5%: 144.5 ml water (31% VWC), 33%: 72 ml water (14% VWC), or 0%: 0 ml water. These treatment values will be different for the sorghum experiments and could also include temperature and other environmental values)
geometry (image pixel length/width dimensions, units: pixels)
image_type (8-bit/color RGB non-interlaced, 8-bit grayscale non-interlaced, or 16-bit grayscale non-interlaced)
imagedate (date-time of the snapshot: YYYY-MM-DD HH:MM:SS)
experiment (e.g. Setaria pilot)
experimenter (lead PI: Ivan Baxter)
project (Drought response in Setaria)
laboratory (Baxter Lab Donald Danforth Plant Science Center)
growth_medium (MetroMix360 potting mix with Osmocote)
filename (name of the image file)
species (e.g. Setaria italica, Setaria viridis or Setaria viridis x Setaria italica)
genotype (e.g. A10, B100, RIL102, RIL128, RIL133, RIL161, RIL187, RIL20, RIL70, RIL98)
sha1 (checksum value for file integrity)
publication_url (No value until published, maybe not needed?)
publication_citation (No value until published, maybe not needed?)

Need to figure out how data will go to BETYdb.
either extract from images or have Noah populate BETYdb with experimental meta-data. Need to also look at ICASA meta-data formats so that we can support those.
Here are some ideas for metadata mapping to BETYdb:
plant_barcode, species, genotype, --> unique identifier / natural key
plant_barcode --> entities
treatment --> treatment, managements
experimenter --> citations.author
experiment --> citations.title
plant_age --> covariate
planting_date --> managements
growth_medium --> managements
camera meta-data --> methods

I've started to path out some pseudo-queries to push these data into BETYdb. Here's an Excel file I created based on the images in snapshot42548 of the sample-data:
sample_metadata.xlsx
I'm trying to sort not only what queries we'll need, but the order to execute them, and which (if any) should be done by Danforth prior to sending the image metadata that will be loaded (e.g. adding treatment descriptions).
@gsrohde I have some questions about best practices in terms of what dictates a suitable method name and description, for instance. I generally assume that we want to check for existence of some of these things before inserting duplicates, but not sure in all cases yet what should be used as criteria for 'uniqueness' if the primary key is a sequentially generated ID.
Here's the work in progress, if we pretend we're loading the first row of my sample file.

Add plant_barcode to entities if not present

	INSERT INTO entities (id)
	SELECT (500)
	WHERE NOT EXISTS (SELECT 1 FROM entities WHERE id=500);

Add camera parameters to methods if not present

How do we typically name a method, based on a set of inputs into its description?
I imagine that we give some unique name to the method, and the description is the concatenation of (zoom, perspective, rotation, camera_type, etc)

	INSERT INTO methods (name, description)
	SELECT(<SomeName>, <zoom> <perspective> <rotation_angle> <camera_type>)
	WHERE NOT EXISTS (SELECT 1 FROM methods WHERE name=SomeName);

Add treatment details to treatments if not present

Is name likely to be unique? Should this be checked in combination with user_id?

	INSERT INTO treatments (name, definition)
	SELECT ("66.5%", "144.5 ml water (31% VWC)")
	WHERE NOT EXISTS (SELECT 1 FROM entities WHERE name="66.5%");

Add management details to managements if not present

May also require updates to managements_treatments table after both management.id and treatment.id are available.

	INSERT INTO managements (mgmttype, level, units)
	SELECT ("water", "144.5", "ml");

Add experiment details to citations if not present

	INSERT INTO citations (author, year, title, journal, pg, url, pdf, doi)
	SELECT ("Max Burnette", 2015, "Max test data", "Danforth unpublished data", 0, "www.danforthcenter.org", "www.danforrthcenter.org", 0)
	WHERE NOT EXISTS (SELECT 1 FROM citations WHERE author="Max Burnette" and year=2015 and title="Max test data");

Add plant_age to covariates if not present, creating entry in variables if necessary

Need to discuss what a trait actually is and how we should represent e.g. plant age

An example of best practices - the only constraint on treatments table is that ID, name and definition are not null, and ID will be automatically assigned if not provided. However, it seems plausible that two different experiments could use two different treatments, but want to give them the same name.
In this case, checking for an existing ID doesn't really make much sense.

We could check to see if there's already a record with that same name and description, but if that's for a different experiment do we want to generate a new treatment.id anyway?
Is checking for name+desc sufficient?
Do we not even worry, and insert a record for the new treatment regardless?

For this particular problem my first reaction would be to check for redundancy in name+description and insert if there's not an existing record with those values, but there are similar questions for e.g. methods that may be more complex.

@max-zilla I may have sent you these before, but here's some documentation we have on contraints:
https://docs.google.com/spreadsheets/d/1fJgaOSR0egq5azYPCP0VRIWw1AazND0OCduyjONH9Wk/edit?pli=1#gid=956483089
https://www.overleaf.com/2086241dwjyrd#/5297403/
Many of these constraints haven't been implemented, and some may have changed.
Also, some information about the BETY database semantics can be gleaned from the data entry workflow document here: https://www.authorea.com/users/5574/articles/6800/_show_article
Some random comments on your last comment:
entities: I know we said at the meeting entities should be used for bar codes, but this seems rather different from how they been have used up until now, which is as a way grouping trait measurements that were made on the same plant at the same time.  (In nearly all cases, there is no information associated with an entity other than it's uniqueness (and the standard Rails timestamp columns created_at and updated_at)—the parent_id, name, and notes columns are usually blank.)  I would double-check with @dlebauer that it make sense to use them to identify particular plants (regardless of measurement time).
In any case, it is important that the id column not be used for any semantic information—that it only serve as a surrogate key.  So if barcode has any real-world significance (as a product barcode does), it shouldn't go into the id column.  (It wasn't clear in your example if you meant 500 as a barcode.)
methods: Again, I see that the meeting notes have camera meta-data being mapped to methods.  I had thought "method" meant experimental method, so I was baffled by this, but looking at the existing table entries, it does seem that these indeed do refer to measurement methods.  But  I would check with @dlebauer to make sure this mapping makes sense.
The naming has seemed rather arbitrary.  Since you have 255 characters available, I would err on the side of being descriptive.  The constraint documents suggest that the names should be unique per citation (i.e., (name, citation_id) should be unique (and each non-null!)), but we don't yet enforce this (except for the name being non-null).
treatment: We don't enforce unique names.  But the Overleaf document suggests that if two rows have the same name value, they should not be associated with the same citation (via citations_treatments).  The boolean attribute control should be non-null, but we don't yet enforce this.  Also, the Overleaf document states that there should be only one "control" treatment for a given citation and site, but it's not clear exactly what this means (that is, how the association of a treatment with a site is delineated).
managements: One or more managements may be associated with a given treatment and they help to define that treatment.  But a given management may be used to help define other treatments as well, which is why the association is many-many and not many-one.
I'm pretty sure the bulk upload script uses (atomic) transactions in cases where it is necessary to get the id of an inserted row and then use it to make an insertion into a join table.
`

I think we should not be using SQL statements for this. We should be using the rest endpoints provided by BETY and if the right ones don't exist add them. We can not guarantee that the extractor has access to the database, most likely it will not.

@gsrohde is it correct that currently the API only supports search, not data entry? I see information about API queries in the Data Access page, but no mention in the Data Entry Workflow article.
I also took a look at lib\simple_search.rb to see how parameters are handled there. If we're willing to have an external script handle the logic of when to upload the data, a new API endpoint could be very simple - maybe even calling something like UPSERT to simplify, although thats PSQL 9.5 only. Essentially an endpoint to insert values into fields of some table that throws an exception if constraints aren't met. Then, a script at Danforth or an extractor at NCSA could make sure data is added in a sensible order.

@max-zilla As far as I know, the API currently only supports searching.
Re Rob's remark: It seems to me it would be OK to think in terms of SQL statements for now until we firm up exactly what we want.  At that point, we could add the application interfaces we need to do what we need without opening up full SQL access.
The code in simple_search.rb is a bit convoluted and ugly, and I've often thought of changing it but have so far contented myself primarily with trying to document it.  Just as a caveat.  I wouldn't necessarily be using it as a model.
I'm not familiar with UPSERT but may look into it.

@gsrohde how can we document the API? Is there something like swagger (what
Clowder uses) for Rails?
On Wed, Dec 16, 2015 at 8:38 AM Scott Rohde<EMAIL_ADDRESS>wrote:

@max-zilla https://github.com/max-zilla As far as I know, the API
currently only supports searching.
Re Rob's remark: It seems to me it would be OK to think in terms of SQL
statements for now until we firm up exactly what we want. At that point, we
could add the application interfaces we need to do what we need without
opening up full SQL access.
The code in simple_search.rb is a bit convoluted and ugly, and I've often
thought of changing it but have so far contented myself primarily with
trying to document it. Just as a caveat. I wouldn't necessarily be using it
as a model.
I'm not familiar with UPSERT but may look into it.
—
Reply to this email directly or view it on GitHub
https://github.com/terraref/computing-pipeline/issues/36#issuecomment-165168770
.

One other thing to think about is if we want to separate the API from the web part. In clowder we have a special subset called api that contains all the api calls. Here is a nice article:
https://labs.kollegorna.se/blog/2015/04/build-an-api-now/

@dlebauer Ruby has RDoc, and the comments I have in many of the Rails files (simple_search.rb, for example) are RDoc-compatible.  But we may want something with different features.  I'll look at what Rob sent.
And don't forget that there is some extensive existing manually-written API documentation.

manually-written API documentation

This is useful to me, but @robkooper said developers need something more
like https://clowder.ncsa.illinois.edu/clowder/assets/docs/api/index.html
Probably we can reuse some of the text from the data access tutorial.
On Wed, Dec 16, 2015 at 8:54 AM Scott Rohde<EMAIL_ADDRESS>wrote:

@dlebauer https://github.com/dlebauer Ruby has RDoc, and the comments I
have in many of the Rails files (simple_search.rb, for example) are
RDoc-compatible. But we may want something with different features. I'll
look at what Rob sent.
And don't forget that there is some extensive existing manually-written
API documentation.
—
Reply to this email directly or view it on GitHub
https://github.com/terraref/computing-pipeline/issues/36#issuecomment-165173164
.

@nfahlgren one task that remains here is to provide experiment-level metadata. I am not sure what is necessary, but things like planting_date, medium, container size, fertilizer / other treatments, irrigation_rate should be included.

@dlebauer right now I added experiment-level metadata to the dataset itself. This is what I added for the pilot experiment, we can add more and edit what's there:
author: Noah Fahlgren
growth_medium: MetroMix360 potting mix with 14-14-14 Osmocote
title: Sorghum Pilot Experiment - Danforth Center Phenotyping Facility - 2014-05-27
project: TERRA-REF
instrument: Bellwether Phenotyping Facility
location: Donald Danforth Plant Science Center
planting_date: 2014-05-27

What about

genotypes
watering and weighing logs
anything else that could help interpret the data?

Please continue evolving the experiment meta data structure. I will update here. Regarding experimental details I need input from experimentators in Maricopa.

Any idea how to include the imaging script (text file) in the meta data?

The metadata is currently uploaded as JSON but I need to update it to JSON-LD eventually. In the JSON-LD file I can provide links to the sources of the information. One new metadata value could be the processing script information with a link to the version (on GitHub) that was used.


Groovy 5 mode is not usable

The Groovy 5 mode uses Spock 2.3-groovy-4.0.
This is not compatible as Spock checks the compatibility.
You could set the spock.iKnowWhatImDoing.disableGroovyVersionCheck system property though to ignore the version check and hope it works, or remove Spock until a 5.0 variant exists.
Currently even if you execute an empty script, you get the version error from Spock.

You don't get an error. You only get the output from spock, because that property is set.

Executing Spock 2.3.0-groovy-4.0 with NOT compatible Groovy version 5.0.0-alpha-8 due to set spock.iKnowWhatImDoing.disableGroovyVersionCheck system property. This is unsupported and may result in weird runtime errors!

Ah, sorry, misinterpreted.
But probably confusing for people just wanting to do something with Groovy.


Add CRMScript.DataStructure.yml to reference Library

Describe the solution you'd like
Adding CRMScript.DataStructure.yml to the Reference-library.
Second request would be to move Vector and Map from Native to Datastructure, to not have them defined twice in the documentation.
Additional context
Snippet from toc.yml:

That's an interesting suggestion! Will give this serious consideration.

Not to include more 'stuff' into this, but I've also noticed you have added some .md-files for 'Data Types' under CRMscript:

This includes Generic and EventData, and the description in these looks to be more extensive then the ones under references.
Generic is added as a class (CRMScript.Native.Generic), but it doesn't contain any valuable information/description


fontforge: fix error: Core python package 'pkg_resources' not found: Cannot discover plugins

Tested both fontforge and fontforge-gtk. No problem anymore. I can still run Python script using Fontforge GUI.
First report of this problem: https://github.com/NixOS/nixpkgs/pull/163439#issuecomment-1062928123. @risicle suggested (#291941) that we fix this.
@erictapen Could you please see whether my fix fits the semantics of the withPython argument?
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

[ ] 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
[ ] 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
[ ] (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.

Needs to target staging due to the number of rebuilds.


[Question] How to add new tests or suite of tests to the ui without reloading everything?

Hey man I am back asking annoying questions. So here is the thing, could you explain me how you add new tests to the ui when the user adds a new test in a file? the same if he adds a completely new file. I am getting those tests from my test enviroment but I am not able to add them to the test explorer UI without triggering a full reload.

You can only send the full test tree to Test Explorer. If you don't want to do a full reload, you should keep a copy of the test tree (i.e. TestLoadFinishedEvent#suite) in your adapter, add the new tests to that and then send a TestLoadStartedEvent followed by a TestLoadFinishedEvent containing the updated test tree.

yeah you re right my bad, I was doing that but I was using an eventEmitter without registering it in the disposable collection, that's why it wasn't working. Now I need to implement some kind of logic to only load if something changes(cause karma returns all the files every single time even if you re just running the tests). Thanks :)


Link point to wrong place

On the document the line 'Azure Batch basics for an overview of the Azure Batch service.' having wrong link point on 'Azure Batch basics'.

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

ID: a73a1ff2-94e8-a570-ff7b-7a57acca06e6
Version Independent ID: 9ec244a7-d939-3918-fc87-34ab1eec5293
Content: Use custom activities in a pipeline - Azure Data Factory
Content Source: articles/data-factory/transform-data-using-dotnet-custom-activity.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

@manish-jain-alept, Thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

@manish-jain-alept  The PR for the fix has been created and it will be merged soon. Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.