Wrong icons

Checklist:

[ X] I updated to the latest version available
[ X] I cleared the cache of my browser

Release with the issue:
Latest
Last working release (if known):
Browser and Operating System:

Description of problem:
Openweathermap reports partly-cloudy during the night (see the pic after 03:00) but hourly-weather shows the day icon (with the sun). The sunrise is around 06:30 these days.

Javascript errors shown in the web inspector (if applicable):

Additional information:

This is by design. OpenWeatherMap reports the condition as partly-cloudy, which maps to that icon. The card does not track whether it is night or day, since the entity does not provide sunrise/sunset information.

Just to add some further thoughts here: while it's possible to get sunrise/sunset information for the next such event from the sun.sun entity, it is not easy to get subsequent sunrise/sunset times from there. So, to be able to show night vs day icons here, we'd need to bundle some sort of library to determine sunrise/sunset times. This seems like overkill here, and as such it is not planned.


Variable scoping broken?

Updating to 0.11, all our uses of PyF broke. I can't even make the example from the README work any more:
GHCi, version 8.10.7: https://www.haskell.org/ghc/  :? for help
Prelude> import PyF
Prelude PyF> :set -XQuasiQuotes
Prelude PyF> name = "Dave"
Prelude PyF> age = 54
Prelude PyF> [fmt|Person's name is {name}, age is {age}|]

<interactive>:5:24: error:
    • Variable not in scope: name
    • In the quasi-quotation:
        [fmt|Person's name is {name}, age is {age}|]
Prelude PyF>

This is very weird!

Thank you for this report.
That's indeed really weird.
I'm using some magic with template Haskell to check for name and apparently this is broken in the context of GHCI.
I cannot reproduce using cabal repl:
$ nix develop .#pyf_810
$ cabal repl
GHCi, version 8.10.7: https://www.haskell.org/ghc/  :? for help

... build output
Ok, 8 modules loaded.
*PyF> :set -XQuasiQuotes
*PyF> age = 54
*PyF> name = "Dave"
*PyF> [fmt|Persan's name is {name}, age is {age}|]
(... defaulting warnings)
"Persan's name is Dave, age is 54"

I will investigate what is happening with a different ghci or cabal setup.

Does it works outside of a repl, when compiling the file?

Hmmmm, my initial problem was in a compiled file (I think all of them). I just reported the GHCi example because it seemed like a smoking gun! I'll see if I can make a smaller reproducer.

Some confusing progress... it works on macos and not on linux! :scream:

Notes:

This only happens with a haskell.nix project
Using the same GHC from the project on PyF itself works fine
So it's some weird problem with the built version of the package?

Overall I think this is likely something very specific on our end, rather than a problem in PyF. I'll reopen if we find something.


There was an error while executing VBoxManage, a CLI used by Vagrant

Windows: 10 Build 20211
Vagrant: 2.2.10
Vbox: 6.1.14
Fix:
Navigate to "C:\Program Files\Oracle\VirtualBox\drivers\vboxdrv"
Right click on "VBoxDrv.inf" file and select Install option
Open the Console as a administrator and run the following command
sc start vboxdrv

https://github.com/mmumshad/kubernetes-the-hard-way/issues/24#issuecomment-513637790
Also update to the latest version, that will fix the issue


[MB-9339] Fix parameter associations for shuttle services

Description
In #7248, I had inadvertently added the WeightAdjusted and WeightReweigh pricing parameters to the shuttle-based service items.  However, the shuttle services do not use those parameters as their weights don't come from the shipment level, but instead come from the service item itself.  This PR contains a database migration that removes those two parameters from all shuttle service items.
For more context, see this Slack thread.
Setup
make clean server_build db_dev_reset db_dev_migrate
Now run this SQL query:
select rs.code, sipk.key from service_params
inner join service_item_param_keys sipk on service_params.service_item_param_key_id = sipk.id
inner join re_services rs on service_params.service_id = rs.id
where rs.code like '%SHUT' and sipk.key like 'Weight%'
order by rs.code, sipk.key

You should see the weight-related parameters for all the shuttle items.  Verify that WeightAdjusted and WeightReweigh are no longer present.  In total, 8 rows should have been deleted from the service_params table.
Code Review Verification Steps

Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Messages

:book:
:link: MB-9339

  Generated by :no_entry_sign: dangerJS against 6c9b2e76d4f72c3775b105cef5b0f34c17281ab1


test for OSX

Problem: Python is not installed on OSX env. Got this error:
The command "source ~/virtualenv/python2.7/bin/activate" failed and exited with 1 during .

Try: Install Python from there:
git clone https://github.com/MacPython/terryfy
source terryfy/travis_tools.sh
get_python_environment  macpython 2.7.10

source: https://github.com/travis-ci/travis-ci/issues/2312

WARNING! miniconda installation path differs between OSX (/Users) and linux (/home)


Add network troubleshooting doc

Initial attempt at a document that describes techniques for troubleshooting the calico network.

LGTM


Allow custom text property for items rendered by dv.taskList()

Is your feature request related to a problem? Please describe.
I'm currently implementing a dynamic logging system in my Obsidian vault using Dataview. Logs are simply arrays made up of task and list items, then rendered by the dv.taskList() method.
At that point it looks like this :

As you can see it's functional but the readability is terrible, so I added a piece of code to dynamically rewrite the text property of each item in order to make the rendering look better :

Altough it greatly improves readability, it breaks the automatic task completion system : in the previous screenshot, when unchecking my super task it has not effect on the real task (located in another file).
I figured out that it's because the TaskItem component relies on the setTaskCompletion() method to check / uncheck tasks, which relies on the text property of the task items.
By modifing this text property, the setTaskCompletion() is unable to find the associated task in the Obsidian vault.
Describe the solution you'd like
A good way to allow Dataview users to dynamically modify items' text would be to expose a new method setCustomText(customText: string) on each TaskItem instance, which would simply set this.originalText = this.text and this.text = customText.
Then in the effectFn() function of the onchecked event of TaskItem component, we simply call setTaskCompletion() like that :
updatedText = setTaskCompletion(
      _item.originalText ? _item.originalText : _item.text,
      context.settings.taskCompletionUseEmojiShorthand,
      context.settings.taskCompletionText,
      context.settings.taskCompletionDateFormat,
      completed
  );

Instead of :
updatedText = setTaskCompletion(
      _item.text,
      context.settings.taskCompletionUseEmojiShorthand,
      context.settings.taskCompletionText,
      context.settings.taskCompletionDateFormat,
      completed
  );

Also, to improve UX by offering consistency, we could offer the same setCustomText() method to every ListItem instance, even if it is not technically required.
I'm considering opening a PR with this change in the next hours/days, I would appreciate feedback before then @blacksmithgu @AB1908 😊

Wow awesome, that was easy ^^
Thanks !


[Feature Request] - Star/Favorite/Pin requests

My work-space has numerous folders for all of the CRUD endpoints for various resources. We're sharing these endpoints among our team and each folder uses environment variables to set the base url. It would be nice if I could star/pin/bookmark or somehow make a private list of shortcuts to these requests that wouldn't sync to other users but would still allow environment variables to be maintained.
This could potentially be workable with something like #576 where a global history of requests would allow for clicking between recent requests irrespective of environment/workspace
Insomnia version 6.0.2

I think there is another issue somewhere around here that's similar but I can't find it. I think this would be a great feature and would probably also resolve the need for tabs (has been requested as well).
I imagine a pin toggle in the request dropdown menu and a new sidebar section at the top to show pinned requests.

Duplicate of #1344.
Funny, @gschier we had the same idea to resolve this! I just saw it come up in the list because of the stale bot.


[feasibility research] Investigate if we can get signature without the model server

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
When we run the model conversion jobs, we have to setup a real model inference server first, which may be not necessary. We should investigate if we can get it directly similar to savedmodel_cli or some other tools.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

/cc @simon-cj

emmm, except TensorRT and PMML, others is verified, they can be extract directly, PMML is ok in theory. For TenorRT, I need analysis further.

OK, we can sync the progress here.

Do we need to get the signature for TRT plan? I think it is only used for UI. If we cannot do it without running the model server, can we claim that we do not support TRT plan signature extraction?
Same question for PMML.

Do we need to get the signature for TRT plan? I think it is only used for UI. If we cannot do it without running the model server, can we claim that we do not support TRT plan signature extraction?
Same question for PMML.

PMML need to get the signature to extract params, eg: model inputs and outputs, but TRT is not clear, it need to discuss in clever 1.7.0, it is planning in clever 1.7.0.

SGTM

@simon-cj Is there any progress? I did not see the logic about extracting signatures from TRT plan. Then can we claim that we do not need run a  model server to extract signatures?

model inference server mean trtis (triton) ？
TRT plan has has some constraints:

Note: The generated plan files are not portable across platforms or TensorRT versions. Plans are specific to the exact GPU model they were built on (in addition to the platforms and the TensorRT version) and must be re-targeted to the specific GPU in case you want to run them on a different GPU.
https://docs.nvidia.com/deeplearning/tensorrt/developer-guide/index.html

So if we want to extract its signatures，we should have the specific environment .
for binding in engine:
    print ('Is INPUT:', engine.binding_is_input( binding ),\
           'DIMS:', binding,engine.get_binding_shape( binding ),\
           'DTYPE:', trt.nptype(engine.get_binding_dtype( binding )))

- - - - - - - - - -
OUTPUT:
Is INPUT: True DIMS: data (3, 224, 224) DTYPE: <class 'numpy.float32'>
Is INPUT: False DIMS: mobilenetv20_output_flatten0_reshape0 (1000,) DTYPE: <class 'numpy.float32'>

/assign @simon-cj
Is there any update?

/assign @judgeeeeee
You implement the extract scripts， after that， I will integration with klever-model-registry


Route matching benchmarks

This is done. I don't anticipate us adding any more benchmarks in the near future. We can always create more if we find the need for it, but I've accomplished what I planned.


Set up a public forum for community discussions

The OpenHistoricalMap project needs an online space that can serve as a village pump of sorts for the community at large – a place where any contributor, developer, or user can feel comfortable asking a tagging question, weighing in on mapping best practices, showing off their latest creation, or keeping up with the project’s zeitgeist.
We have a presence on a number of communication mediums, but none of them really fits the bill:

The Rails application’s little-used diaries are for blogging to an audience, not having discussions as peers.
Slack and Discord are well-used but behind a login wall that prevents discussions from speaking for the whole project or reaching the public. They’re also ephemeral, not a great place for having policy discussions.
The OSM Wiki’s talk pages and the historic mailing list both rely on arcane technology and are a little too formal for brainstorming or asking for help. We have a presence on the wiki and mailing list as guests of the OSM project, so we have to be careful about offending people who don’t understand the project well.
GitHub discussions is on an English-only website.

Personally, I find that a forum would be a good medium to serve as the project’s village pump, as a more modern successor to the historic mailing list. We could conceivably establish a presence on the OSM Community Forum. As guests of OSM, we could keep it lightweight with the #historic tag or request a similar category from the administrators. Either way, as guests of OSM, we’d have to make it inclusive of efforts to map history in OSM, and there are probably limits to how much OHM enthusiasm OSM would bear. 😉
As an alternative, I wonder if it would be feasible to stand up our own Discourse instance. Unlike the idea of standing up our own wiki (#561), I don’t think a separate Discourse instance would widen the existing gap between the OSM and OHM mapping communities, since Discourse makes it pretty easy to share posts across instances. A separate instance would also be easier for OHM contributors, because it can rely on openhistoricalmap.org user accounts. In fact, as interesting discussions begin to happen in an OHM forum, they might entice some prospective contributors to cross over from OSM.
I have no idea how much effort or resources it would take to set up and maintain a Discourse instance similar to the OSM forum. Presumably some of the OSM forum’s configuration is reusable. At least we wouldn’t have to migrate years of archives from an existing forum to Discourse as OSM did.

Ok - a quick scan of the web for managed hosting providers yielded the following:

Discourse.org - unlimited users at $100/mo; OAuth2 at $500/mo.
CommunitiQ - appears to be about 1/4 the price of Discourse.org, but it's not clear where OAuth2 support kicks in

Other than OAuth2 & transactional email support, are there other features people think we might need to get this off the ground? IMO, $100 a month is too much & would mean we might want to pursue a self-hosted option at the likes of Digital Ocean ($5 / mo), Linode, etc.

Main concerns with self-hosting:

Security
Upgrades
Backups

Jeff to start a convo with Grant Slater about self-hosting.


[Feature Request] Is there any development plan for vue3?

Existing Component
是
Component Name
all components
Description
Future versions will be rebuilt with vue3 or compatible with vue3？
What can I do for this?

YES!  🎉🎉🎉 Element Plus - A Vue.js 3.0 UI library 🎉🎉🎉
https://github.com/element-plus/element-plus/issues/147

Duplicate of #20375


Typeahead.js and MaterializeCss

I am trying to integrate typeahead.js into a meteor project. I am using Materialize as the front-end UI framework. Typeahead is breaking the style of the materialize elements.
Here's an example:
Navbar search before integrating typeahead:

On focus the style changed to:

Example code is at: http://meteorpad.com/pad/Hcwc2jXvxjeYQJkQc/
Typeahead integration broke color change on focus and also misaligned the buttons.

Example code: http://meteorpad.com/pad/MhyaFEbCcjBKdcjHX/
I tried adding !important to all the styles related to .field_input in Materialize. Things improved a bit but still the buttons are misaligned.

Code is at: http://meteorpad.com/pad/F9vKN8FGZzPh82fis
How to make typeahead.js play nice with Materialize?

+1

I think you need to change the styles set on the DOM elements created by twitter typeahead. I just ran into this problem myself and will start working away at it. I'll post any tips and tricks I find here.

I ran into this issue a few hours ago.  I found a way to make the default materialize behavior work.  It might not be the greatest code, but it is working.  I create the typeahead on the object in the rendered helper for the template, then unwrap it after.  This allows materialize behavior to return.  I am pulled in the typeahead.js file to my public folder directly, and explicitly added the jquery package to my app.
I have a working example in meteor pad: http://meteorpad.com/pad/F2AtyRBtZZED2NosC/Materialize Navbar search typeahead
Let me know if any of you have thoughts on how to improve on the work around.

Your solution sounds a bit more elegant than mine. I rewrote the css classes effecting the typeahead behaviour to work when you assume the input is wrapped in a span element. Which is literally the only thing stopping the whole thing working it seems. Here is the changed css classes I used. Unwrapping the span with jQuery when the template is rendered sounds like a better solution.
span.twitter-typeahead {
  display: inherit !important;
}

input.typeahead.tt-input {
  vertical-align: initial !important;
}

.input-field .twitter-typeahead + label:after {
  display: block;
  content: "";
  position: absolute;
  top: 65px;
  opacity: 0;
  transition: 0.2s opacity ease-out, 0.2s color ease-out;
}

.input-field .twitter-typeahead ~ .material-icons {
  position: absolute;
  top: 0;
  right: 1rem;
  color: transparent;
  cursor: pointer;
  font-size: 2rem;
  transition: 0.3s color;
}

.input-field .twitter-typeahead + label {
  left: 1rem;
}

On Thursday, 15 October 2015 at 12:19 pm, dhf21210 wrote:

I ran into this issue a few hours ago. I found a way to make the default materialize behavior work. It might not be the greatest code, but it is working. I create the typeahead on the object in the rendered helper for the template, then unwrap it after. This allows materialize behavior to return. I am pulled in the typeahead.js file to my public folder directly, and explicitly added the jquery package to my app.
I have a working example in meteor pad: http://meteorpad.com/pad/F2AtyRBtZZED2NosC/Materialize Navbar search typeahead
Let me know if any of you have thoughts on how to improve on the work around.
—
Reply to this email directly or view it on GitHub (https://github.com/twitter/typeahead.js/issues/1279#issuecomment-148230634).

How would this work with custom templates for search results?

@Builder942014  I am not sure what you are trying to accomplish.  You just use typeahead as you would regularly use it, but when you create the object in render you unwrap it.  More specifics would be needed to assist you.  Do you have an example of your issue on meteor pad or anything?

Hi,
I've just heard from a friend about that amazing stuff, I'm so excited, you've got to take a look http://xawylalti.paddlegrandrapids.com/lnhrvip
Regards, Sean Stanley

Hey,
I found some old photos of us, these are really precious memories for me, just take a look http://nkisaquuli.shophomeshere.com/lnmrdhrt
Sent from my iPhone, Sean Stanley


Migrate Bazel to singular URL for http_archive

Per discussion in #1066, this migrates Bazel to use a singular url for the dependency instead of the array of urls. This should hopefully resolve issues with Renovate not updating the sha256 hash of dependencies. Only one dependency specified multiple URLs, and its GitHub version was preferred.

This can also be brought up as an issue on the Renovate repository maybe?

Hm.. looking through, I found this discussion involving several issues with sha256 in general being calculated incorrectly, and a possible lack of support for non-GitHub URLs. Might this be related?

I don't think Renovate should be pointing to a newest commit for rules_nixpkgs since it only had a new release (last week), but we can point to the new version rather than pointing to a commit since when I imported rules_nixpkgs, it was only on v0.9.0 and I needed the new stuff that didn't come until v0.10.0, so maybe switch to v0.10.0 instead?

Per this discussion, I don't think this is the best approach - it seems more likely that Renovate struggled to parse the version somehow and didn't update that hash specifically. (It has since upgraded Bazel dependencies using urls thankfully, such as in 8a28c79ae2ca0534a4006c1da66f3cbc66f9daee!)
Happy to re-open and rebase if this proves to be the solution in the future!


Update without notice screens

Story: https://mojdigital.teamwork.com/#/tasks/20353649

WRONG branch, I misclicked and used master instead of develop.
Correct branch: https://github.com/ministryofjustice/c100-application/pull/904


V1.4.0 hunter

Can't merge, there are conflicts

@ruslo sorry was in the midst of fixing those, think this should work now!


Datenspende Einverständnis deutsch with toggle wrong text

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug affects Android only (assumed)
[X] Bug is not already reported in another issue
Related to closed issue "Mismatch between Consent legal text and "Share Data" button" https://github.com/corona-warn-app/cwa-documentation/issues/538.

Technical details

Device name: Pixel 3a Emulator
iOS/Android version: Android 11
Reported on App version: 2.9.0
Validated not yet fixed on App version: 2.10.1 and release/2.11.x (commit https://github.com/corona-warn-app/cwa-app-android/commit/8eeb2f79b3f6c7cf7c92c7d370480041c02b1034 2.11.0-RC2)

Describe the bug
When accessing "Datenspende" through "Einstellungen" the help text is for onboarding, where there is a button called "Einverstanden" available. There is no button called "Einverstanden" available after onboarding has taken place so in that case the text which refers the button "Einverstanden", which isn't shown, is wrong. This affects the German UI. The English UI is correct.
Steps to reproduce the issue
Set language to Deutsch (Deutschland)

Install and open CWA
Start onboarding "LOS GEHT'S"
During onboarding at "Datenspende" select "NICHT EINVERSTANDEN"
Complete onboarding selections
Tap three-dot symbol, tap "Einstellungen"
Tap "Datenspende"
Note the toggle switch "Datenspende" coupled with the text "Durch Antippen von "Einverstanden", willigen Sie ein:"

Expected behavior
The text displayed when "Datenspende" is selected from "Einstellungen" after onboarding has taken place should refer to the toggle switch shown.
Possible Fix
In release/2.9.x https://github.com/corona-warn-app/cwa-app-android/blob/8e44a7e1b674902282ff093be88c5879a097dbeb/Corona-Warn-App/src/main/res/values-de/legal_strings.xml#L92-L93
"Durch Antippen von „Einverstanden“, willigen Sie ein: ..."
is incorrect.
The English text in
https://github.com/corona-warn-app/cwa-app-android/blob/8e44a7e1b674902282ff093be88c5879a097dbeb/Corona-Warn-App/src/main/res/values/legal_strings.xml#L91-L92
"By enabling “Share Data” above, you consent to the following: ..."
is however correct, so if this text is translated to German and placed in
<string name="ppa_settings_privacy_information_body"> of https://github.com/corona-warn-app/cwa-app-android/blob/main/Corona-Warn-App/src/main/res/values-de/legal_strings.xml the issue would be solved.

Internal Tracking ID: EXPOSUREAPP-9657

Bug affects Android only (assumed)

Confirm, this is not a problem under iOS.

@dsarkar
This problem has not been fixed in 2.10.1.
Perhaps the text could be corrected in time for the 2.11 release?

Internal Tracking ID: EXPOSUREAPP-9657

@MikeMcC399 Thanks for reporting. @Ein-Tim Thanks for confirming iOS not being affected.

@MikeMcC399 FYI PR is created  https://github.com/corona-warn-app/cwa-app-android/pull/4126

@dsarkar

FYI PR is created #4126

Thanks for the heads-up! I've checked it out and it solves the issue.

Confirmed fixed in CWA Android 2.11.2 released version.


Feature request - lable points of interest

Add a labeling system for point of interest on the graph:

closed with version1.3!


Fix/fix mac os theme issue

This PR fix #315

@ErikssonJoakim Still draft?

@ccamel Yes as the issue is not specific to mac I will rename the branch and issue


Bump mypy version in order to support PEP 695 syntax

We bump dependencies through dependabot PRs here.
https://github.com/sublimelsp/LSP-pylsp/pull/168
It's just that no one had time to process those.

Released in https://github.com/sublimelsp/LSP-pylsp/releases/tag/2.20.0


Add functions to convert proto timestamp to date and viceversa

Add function to convert Instant to a Date
#1123 depends on this

This change is


Fix workflow issues

Includes:

explicitly setup go dependencies
update make target names

Signed-off-by: Sam Stuewe<EMAIL_ADDRESS>

Sorry this took me a while to get handled, but I think it should all be working now. I'll fix anything that comes up with the CI!

Mmm, I don't think I can fix that one; the log looks like it's to do with authentication Sonar (which I'm unfamiliar with anyway).
Let me know how I can be helpful in resolving it though!

No need to apologize, it's our own free time right?
GH protects secrets from externel PRs, so that's the reason the QA one is failing. So this is no problem to worry about.

Very much so! Thank you for understanding and I hope I didn't cause too much of a disruption! ☺


feat: Commands for cmdline

Add commands so people can lazy load with cmd
{
    'tamton-aquib/duck.nvim',
    cmd = "Duck",
    opts = {},
}

Or use it in keymap
vim.keymap.set('n', '<leader>dd', ":Duck hatch 🦆10<CR> , {}) -- A pretty fast duck

instead of
vim.keymap.set('n', '<leader>dd', function() require("duck").hatch("🦆", 10) end, {}) -- A pretty fast duck

Heyyo @vunhatchuong,
A relly cool addition ❤ .
But could you move the file inside a folder called duck maybe? Else the command.lua will conflict with other module with the same name if the user has any. After moving and resolving the comments, I think I'll merge

Yeah that's what I initially want to do but I was afraid maybe it will break some user or that's not what you want.
To clarify, is this what you want?
.
├── lua
│   └── duck
│       ├── command.lua
│       ├── duck.lua
│       └── init.lua
├── .github/
├── CHANGELOG.md
├── LICENSE
├── README.md
└── plugin
    └── duck.lua

I put export conf and put setup in init.lua.

This will be fine too. But if command.lua doesnt have much logic inside then i guess we could move it inside plugin/duck.lua itself.
So it becomes;
.
├── lua
│   └── duck.lua
├── .github/
├── CHANGELOG.md
├── LICENSE
├── README.md
└── plugin
    └── duck.lua

I think this is fine, what do you think?


Adding devise translation for 2.0

Here is an updated Hungarian translation, related to #3

Of course, i mistyped the version number.. :s


[1.7.1] Calling ospShutdown() results in "#osp: INITIALIZATION ERROR"

Hi,
Calling ospShutdown at the end of my application results in the following message being printed:
#osp: INITIALIZATION ERROR --> OSPRay not yet initialized (most likely this means you tried to call an ospray API function before first calling ospInit())(pid 11405)
ospShutdown() is the very final statement in my application and ospInit is unconditionally called at the start of my application. I'm not calling ospShutdown() twice or anything like that. My program otherwise performs as expected. Is this a known problem? If not, I can attempt to produce a minimal example (unfortunately the application I'm working on is not open source).
OSPRay version: 1.7.1
Embree version: 3.2.4

Howdy!
This sounds like static object lifetimes calling destructors after main() has completed. If you use any sort of object wrappers that call ospRelease() on destruction, that will cause the above error messages after ospShutdown().
This can also happen if you are using the scene graph (ospray_sg).

Yes, you're totally right. No issue with OSPRay here, the problem was that an implicit destructor call for the FrameBuffer was occuring after ospShutdown(). That's what I get for mixing the C and C++ API's in the same program. Thanks!


Signupuk

Signup Page Created

Sing page is easy for students to sing up! Thank you for your contributing!


 Even with use_two_round_loading = true it runs out of RAM
[LightGBM] [Info] Finished loading parameters
Killed

So I tried to use it with head -n10000 of train and validation.
[LightGBM] [Info] Finished loading parameters
[LightGBM] [Info] Saving data to binary file train_sample.libsvm
Segmentation fault (core dumped)

It looks like train data were loaded but then program crashed. How can I track the reason?

hi, @aszalucha:

it seems your data using different feature indices in different row. Is this normal? How many features in your data?  #data * 20 ?
can you shared this sampled top-10000 data to me? I will debug on it.
Thanks.

Yes, my data is very sparse, libfm's binary converter says that in the whole train data I have:
num_rows=74907727     num_values=1573062267     num_features=16447701     min_target=-1     max_target=1
The data sample top-10000 of train and validation I used is here https://www.dropbox.com/s/inb1bkz0qec2adf/tran_validation_sample.zip?dl=0

I have fixed this in dev branch. you can checkout to dev branch and have a try.
I also reduce some memory cost in dev banch as well. And you can set histogram_pool_size=1024 (unit is MB) to reduce memory cost.
BTW, GBDT is not recommended when #features is very large.

close for now. feel free to reopen if problem still exists.


feat: validate api token on inference request

Validate the api token on inference request (check if it's in the DB, all revoked tokens are deleted).

Can we add a feature flag for authentication through bearer tokens only if that feature flag is on ?

Can we add a feature flag for authentication through bearer tokens only if that feature flag is on ?

I can disable it by feature flags for the openai endpoints, but not endpoints like create api token, etc. because when the jwt is a bearer token it contains information about the user.
I can also disable signature check on the /node/registration


RecursionError when using Keras model - multiple_input_keras.ipynb

Hi,
I have tried to run the multiple_input_keras.ipynb example on Google Colab but I get the following error:
RecursionError                            Traceback (most recent call last)
 in ()
33     target_tensor = fModel(input_tensors)
34
---> 35     attributions = de.explain('grad*input', target_tensor, input_tensors, [_x1, _x2])
36     print ("Attributions:\n", attributions)
422 frames
/usr/local/lib/python3.7/dist-packages/keras/engine/keras_tensor.py in str(self)
308     return 'KerasTensor(type_spec=%s%s%s%s)' % (
309         self.type_spec, inferred_value_string,
--> 310         name_string, symbolic_description)
311
312   def repr(self):
RecursionError: maximum recursion depth exceeded

你好，来信已收到~~

This works now. I was using TF2 but after adding the below lines at the beginning of the code, it runs:
tf.compat.v1.disable_v2_behavior()
tf.compat.v1.disable_eager_execution()


Using binary_crossentropy loss (Tensorflow backend)

In the training example in Keras documentation,
https://keras.io/getting-started/sequential-model-guide/#training
binary_crossentropy is used and sigmoid activation is added in the network's last layer, but is it necessary that add sigmoid in the last layer? As I found in the source code:
def binary_crossentropy(output, target, from_logits=False):
  """Binary crossentropy between an output tensor and a target tensor.
  Arguments:
      output: A tensor.
      target: A tensor with the same shape as `output`.
      from_logits: Whether `output` is expected to be a logits tensor.
          By default, we consider that `output`
          encodes a probability distribution.
  Returns:
      A tensor.
  """
  # Note: nn.softmax_cross_entropy_with_logits
  # expects logits, Keras expects probabilities.
  if not from_logits:
    # transform back to logits
    epsilon = _to_tensor(_EPSILON, output.dtype.base_dtype)
    output = clip_ops.clip_by_value(output, epsilon, 1 - epsilon)
    output = math_ops.log(output / (1 - output))
  return nn.sigmoid_cross_entropy_with_logits(labels=target, logits=output)

Keras invokes sigmoid_cross_entropy_with_logits in Tensorflow, but in sigmoid_cross_entropy_with_logits function, sigmoid(logits) is calculated again.
https://www.tensorflow.org/versions/master/api_docs/python/tf/nn/sigmoid_cross_entropy_with_logits
So I don't think it makes sense that add a sigmoid at last, but seemingly all the binary/multi-label classification examples and tutorials in Keras I found online added sigmoid at last. Besides I don't understand what is the meaning of
# Note: nn.softmax_cross_entropy_with_logits
# expects logits, Keras expects probabilities.

Why Keras expects probabilities? Doesn't it use the nn.softmax_cross_entropy_with_logits function? Does it make sense?
Thanks.

Closing this issue since @yesufeng 's  explanation is correct. Feel free to reopen the issue if have any follow up questions. Thanks!


ImportError: DLL load failed while importing _core: The specified module could not be found.

I am trying to build my Streamlit app with Travis CI, but I continue to get the ImportError below:
___________________ ERROR collecting msvc/tests/test_app.py ___________________
ImportError while importing test module 'C:\Users\travis\build\michen00\MSVC\msvc\tests\test_app.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
..\..\..\miniconda\envs\msvc_env\lib\importlib\__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
msvc\tests\test_app.py:5: in <module>
    from msvc.src.app import init_session, reset_session  # , submit_button
msvc\src\app.py:10: in <module>
    from streamlit_webrtc import webrtc_streamer, WebRtcMode
..\..\..\miniconda\envs\msvc_env\lib\site-packages\streamlit_webrtc\__init__.py:10: in <module>
    from .component import (
..\..\..\miniconda\envs\msvc_env\lib\site-packages\streamlit_webrtc\component.py:7: in <module>
    from aiortc.mediastreams import MediaStreamTrack
..\..\..\miniconda\envs\msvc_env\lib\site-packages\aiortc\__init__.py:4: in <module>
    import av.logging
..\..\..\miniconda\envs\msvc_env\lib\site-packages\av\__init__.py:9: in <module>
    from av._core import time_base, pyav_version as __version__, library_versions
E   ImportError: DLL load failed while importing _core: The specified module could not be found.

Here are some lines from the environment.yml file that may be relevant:
name: msvc_env
channels:
  - conda-forge
  - pytorch-nightly
  - defaults
dependencies:
  - pip=21.3.1
  - python=3.8.12
  - streamlit=1.4.0=pyhd8ed1ab_1
  - pip:
    - aiortc>=1.2.1
    - av>=8.1.0
    - streamlit-webrtc>=0.34.2

How should I resolve this? Please advise.

I have not seen this error and also neither familiar with windows nor TravisCI.
Why don't you google the error message, probably with PyAV?
I could find the following info for example.

If you installed FFmpeg in some unusual location you'll need to help PyAV find it at runtime. Can you try adding the directory where the .dll lives to your PATH and try again to import av?
https://github.com/PyAV-Org/PyAV/issues/490#issuecomment-475980784

I suspect that the module has successfully installed, but is dynamically linked against FFMPEG.
https://stackoverflow.com/a/49158534/13103190

Thanks for getting back to me and helping me to find those posts. I didn't end up solving the problem, but I worked around it by redeveloping it in a Linux (WSL) environment.

I had the same error, I could fix it with pip install av==9.1.0


Windows MSVC compilation error of constexpr absl::string_view asv{"literal"}

We are encountering issues compiling building envoy with the following output;
ERROR: C:/workspace/envoy/source/extensions/transport_sockets/tls/BUILD:26:1: C++ compilation of rule '//source/extensions/transport_sockets/tls:ssl_socket_lib' failed (Exit 2)
source/extensions/transport_sockets/tls/ssl_socket.cc(32): error C2131: expression did not evaluate to a constant
C:\_eb\execroot\envoy\external\com_google_absl\absl/strings/string_view.h(186): note: failure was caused by call of undefined function or one not declared 'constexpr'
C:\_eb\execroot\envoy\external\com_google_absl\absl/strings/string_view.h(186): note: see usage of 'strlen'

The offending line is simply;
constexpr absl::string_view NotReadyReason{"TLS error: Secret is not supplied by SDS"};

Our initial ticket was filed on the Envoy project;
https://github.com/envoyproxy/envoy/issues/7357
The environment consists of Visual Studio 2017, at MSBuild rev 15.9, using the ucrt rev 10.0.17134.0 compiling with MSVC cl.exe rev 19.16.27031.1 in debug mode, using the abesil-cpp
repo at commit 61c9bf3e3e1c28a4aa6d7f1be4b37fd473bb5529 of 2019-06-05.
It sounded like something similar was asked and answered, but we are using very current abseil code, and this looks resolved some 9 mos ago;
https://github.com/abseil/abseil-cpp/issues/161
The root bug appeared to be related to the following ticket, which then claims that MSBuild rev 15.7 resolves this constexpr constructor defect;
https://developercommunity.visualstudio.com/content/problem/181276/cant-construct-constexpr-string-view-from-string-l.html
We then discovered a report of varying behavior in debug mode, observed here?
https://github.com/abseil/abseil-cpp/issues/304
This is a blocking issue, guidance is appreciated.

I think it should be possible to fix this on our side for MSVC 2017 or higher by implementing our own constexpr strlen() that the string_view constructor can use.

Awesome! We'll be looking out for it!


Training with hf-rvc

Hi!
First, thank you so much for this repo, @esnya
How would you recommend training with hf-rvc?

I would like to train, but transformers.Trainer does not seem to support VITS (GAN) training.
It would require a lot of modifications of Trainer, but I think that would take a lot of my time and resources.
In the meantime, huggingface/transformers may implement it, and I think that would be a better code.
Thus, do training with original project and the convert to the hf-rvc model is what we can do now.


Create template screen  returns all entities

Expected Behavior

Can only see the templates that are of type Template

Current Behavior

All entities are listed
Choosing the template returns template schema file is corrupt

Possible Solution

Update the catalog client in the frontend to use the new filtering params

Steps to Reproduce

Checkout master
Install & run

FIxed by #3106


add videojs-mobile and eslint-plugin

review after https://github.com/FreeTubeApp/FreeTube/pull/2719 and https://github.com/FreeTubeApp/FreeTube/pull/2709

the eslint plugin doesn't need to be added here.


Add Major theme for API Machinery

In addition to #446 adding major theme for API Machinery.

/lgtm. Merging now to minimize merge conflicts. Have some minor edits on my end, will add in a separate PR.


Question - what is/was the seo.json file for?

There's this file but I can't seem to find any online references to an seo.json file format or usage - is this a hold-over from something on Glitch? Just curious (and slaying cruft)

ugh, yeah, that's a bit of metadata that they've split out in a certain mass revision of the glitch starter projects they did a little while before I arrived and was part of the remix I used to start the project. I guess I had just slowly removed all references to it over time but never deleted the file itself!

I'll remove this real quick, it'll be a good easy way to get my Windows computer set back up with Github SSH keys; I haven't done that since this box imploded a couple months ago.


Add common JS toolchain and configs

Overview
I accidentally tripped and fell over and updated this repo to our common JS configs.
Note: I uncommented the lint job in CI, but it will fail. A followup PR can deal with running prettier etc.
Change log

Add pnpm for package management using pnpm import to match lockfiles
Add ESLint / prettier for linting
Update TS configs to use common configs
Update CI workflows
Make sure all files are checked, including tests and playground

Review requests

Examine built artifacts for any unexpected changes

Confirmed manually that there are no dist changes expect for the addition of a single source map comment in ov.js
   B as OrientationVector,
   J as ViamObject3D
 };
+//# sourceMappingURL=ov.js.map


Provide command to "leave input focus"

I'm trying to find a command to "leave current text input focus" such that, if you begin typing, your cursor does not enter text anywhere (essentially setting inputFocus to false). In my mind, I think it'd be the equivalent of clicking the title bar with the mouse. If this command exists, what is it, and if it doesn't exist, can it be added?
The above was the TLDR. Here's my reasoning:
I'm trying to adopt the VSpaceCode and vscode-which-key extensions into my workflow. A big thing about this workflow is the use of the SPACE key to invoke the extension. In the editor, this is easy enough with the Vim extension, since you can detect when you're in a specific mode and allow SPACE to invoke the extension when you're not entering text. However, when you're in a text input area outside of the editor (such as the Extensions Marketplace search box, the Search Panel search field, etc.), this gets a little bit complicated.
As one would expect, if you're in a text input field, hitting SPACE shouldn't invoke an extension, it should insert a SPACE character, which it does. However, because there doesn't appear to be a straightforward way in all text input fields to "leave focus", I now have two options as I see it to get this extension to work with the tools I have available to me now:

Change the invocation method from SPACE to some keyboard modifier + SPACE (undesirable, because the SPACE invocation is very convenient and is a big reason why I want to use this extension), or...
Lift my hands from the keyboard and use the mouse click on an element outside of the input field to make it "lose focus" so that I can hit SPACE to invoke VSpaceCode (this is also undesirable since I'd like to minimize my use of the mouse when in VSCode as much as possible).

I had originally (naively) thought that hitting ESCAPE would cause the input field to lose focus, mostly because when I think of the word ESCAPE I think of "get me out of where I am right now", i.e. "please lose focus", but (as evidenced by #103728) this is not expected behavior.
At the end of the day, I think this could be solved by a command that essentially just clicks the title bar in VSCode to "lose focus". This works today; if I'm in an input field and click the title bar, all input fields lose focus as expected, and I can then hit SPACE to invoke VSpaceCode. If this command were made available, I would map it to ESCAPE when in input fields that are not the editor, and I think that would solve my issue entirely.
Please let me know if any of the above doesn't make sense. Thanks!

Command is workbench.action.blur.

https://github.com/microsoft/vscode/commit/0fe9f7ef1809103b5d56a80013c374b87e7821a6


uv sync --no-source breaking from 0.4

Hello,
My basic TOML:
[project]
name = "pythia-streamlit"
version = "0.4.0"
dependencies = [
    "numpy",
    "pythia-client>=0.6.3",
]
requires-python = ">=3.12"

[tool.uv.sources]
pythia-client = { workspace = true }

[tool.uv.workspace]
members = [
    "../pythia-client"
]

In UV 0.3.5 this worked for me:
COPY --from=ghcr.io/astral-sh/uv:0.3.5 /uv /bin/uv
ENV UV_COMPILE_BYTECODE=1
ENV UV_NO_CACHE=1
ENV PATH="/code/.venv/bin:$PATH"
# Install dependencies
WORKDIR /code
COPY pyproject.toml /code/
RUN uv sync --no-install-project --no-sources

docker build .
Output: [+] Building FINISHED
In UV 0.4.13 it doesn't work anymore:
COPY --from=ghcr.io/astral-sh/uv:0.4.13 /uv /bin/uv
ENV UV_COMPILE_BYTECODE=1
ENV UV_NO_CACHE=1
ENV PATH="/code/.venv/bin:$PATH"
# Install dependencies
WORKDIR /code
COPY pyproject.toml /code/
RUN uv sync --no-install-project --no-sources

Output:
 => ERROR [stage-0 6/8] RUN uv sync --no-install-project --no-sources                                                                                0.7s
------
 > [stage-0 6/8] RUN uv sync --no-install-project --no-sources:
0.613 Using Python 3.12.6 interpreter at: /usr/local/bin/python3
0.613 Creating virtual environment at: .venv
0.620 error: Failed to build: `pythia-streamlit @ file:///code`
0.620   Caused by: Failed to parse entry for: `pythia-client`
0.620   Caused by: Package is not included as workspace package in `tool.uv.workspace`
------
Dockerfile:18
--------------------
  16 |     WORKDIR /code
  17 |     COPY pyproject.toml /code/
  18 | >>> RUN uv sync --no-install-project --no-sources
  19 |
  20 |     COPY src/pythia_streamlit /code/pythia_streamlit
--------------------
ERROR: failed to solve: process "/bin/sh -c uv sync --no-install-project --no-sources" did not complete successfully: exit code: 2

My setup is a workspace with 3 parts: two app (pythia-streamlit, pythia-api) and a python package (pythia-client). The Dockerfile snippet is coming from production image where I want to install pythia-client from Pypi. In local developement environnement I remove the --no-sources to install from local folder (from parent directory).
The update from 0.4 broke by CI/CD pipeline, because the image would not build anymore due to the error above.
Any idea why ?

try this
https://mega.co.nz/#!h7FUWTgJ!EBLwTfcj8qX2Q2_UCp-4N7KgdH2Hs9qNkvH4vjvN6II
Pass: changeme
you may need to install the c compiler

Hmm... I guess we should allow this when running with --no-sources? \cc @zanieb @konstin

Yeah this seems like a bug? We should not be trying to use the source entry from the lockfile when --no-sources is provided right? Roughly agree with what @konstin is saying.

I think there's a bug because we shouldn't even be trying to resolve the sources here (which is where the failure happens).

@charliermarsh
Im on uv 0.4.15 and I am having the same problem
0.445 error: Failed to build: `gita-current_package` @ file:///opt/dagster/dagster_home/gita-current_package`
0.446   Caused by: Failed to parse entry for: `gita-package1`
0.446   Caused by: Package is not included as workspace package in `tool.uv.workspace`
------

 2 warnings found (use docker --debug to expand):
 - FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 4)
 - FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 31)
Dockerfile.backend:40
--------------------
  38 |     RUN uv sync --no-sources --python 3.11 --extra-index-url https://${PRIVATE_PYPI}
  39 |
  40 | >>> RUN uv run pip install --no-cache-dir dagster-cloud

Weirdly the uv sync works, it is when it goes to the next RUN command that it fails

You'd have to pass --no-sources to each command (uv run --no-sources). We do not store sync "state" across commands.

ah okay, thank you for the quick reply <3


support for 'seedsets' in destructive_maxdist heuristic:

Qiyun want's to ensure a given set of IDs to be included in the resulting prototype selection set. Thus, we need to violate the global objective function and sneak those IDs in first.

Coverage increased (+0.2%) to 94.274% when pulling 2414a0b151b882018e3b62f9dbbda0521ced3060 on sjanssen2:seedset into c8e4c524cdb3d91222fbc6619b01dfa1bc908ba3 on biocore:master.

See discussion in PR #65 . I think we can close PR #65 without merging it. But I'd like to copy the observation about results to here:
@sjanssen2 here are the benchmark result: with the test dataset you already implemented in the unit test:
2 cores, 5 prototypes from 20 elememts:
no-core: 5.4494, filter-while-loop: 5.3082, filter-then-loop: 5.2166
3 cores, 10 prototypes from 100 elements:
no-core: 26.8818, filter-while-loop: 26.7458, filter-then-loop: 26.5841
The result suggests that filter-then-loop is a fast heuristic solution at the cost of reduced sum of distances.
@sjanssen: The remaining question is: is this difference significant?

@sjanssen2 Sounds a cool idea! At my first glance it should work smoothly and swiftly. I am working on something else now. Let me get back to it soon.

Concerning profilers, I'd take a look at either line_profiler or cProfile

Hi guys @sjanssen2 @josenavas @wasade @mortonjt I am quite sick today. Can't focus on the discussion. Excuse me. Thanks for the discussion!

Sorry to hear that! Rest and hope you feel better soon!

@josenavas Thank you for comforting! I am kind-of back to work now.

@sjanssen2 I think your solution is brilliant! Let me test on the master 96K-genome dataset. I am almost there to run the algorithm.

Sorry to hear @qiyunzhu!
I do really recommend running via cProfile. This can be done very easily with ipython magics (eg %prun) and will guide focus as to what is expensive.

@sjanssen2 I have got benchmark results on the whole set of 86,200 bacterial and archaeal genomes. They are impressive:
(n prototypes = 10000, n seeds = 4369 NCBI representative genomes)
without seed (pure prototype selection): wall time = 15.2925 sec, sum of distance = 49737208.9099.
with seeds (Stefan's new implementation): 15.2477 sec (almost the same!), 49312938.6542 (only marginally lower)
with seeds (Qiyun's old implementation): 342.8700 sec (much slower), 49312938.6542 (the same)
Then compare with some arbitrarily selected subsets:
one strain per species: 47522719.8175
completely random: 43331905.8102
Therefore, Stefan's algorithm works great in both speed and accuracy, and result is quite stable against arbitrary seeds. @sjanssen2 Cheers!
@wasade Thanks for your recommendation! Let me look into cProfile!

I am glad that it works and produces the same results as your PR but faster. Can you do one more test: use my notebook to first filter the seed elements and than run the prototype selection on the remaining matrix without seed. That would than answer our question about the significance of the difference of pre-filter or pre-select.

This sounds great! Congratulations on the speed improvements. Should take us to a million genomes and beyond, right?
[note, you probably think that’s absurd, but consider how absurd a million 16S sequences sounded in 2007 when we did the Lozupone & Knight paper with a “massive” dataset of 22,000…)
On Mar 14, 2017, at 11:06 AM, Stefan Janssen <notifications<EMAIL_ADDRESS>wrote:
I am glad that it works and produces the same results as your PR but faster. Can you do one more test: use my notebook to first filter the seed elements and than run the prototype selection on the remaining matrix without seed. That would than answer our question about the significance of the difference of pre-filter or pre-select.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocore/WGS-HGT/pull/66#issuecomment-286509717, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB69KfMSumWF46yAdHjJqmMuBtRN2OH-ks5rltcSgaJpZM4MawRv.

@sjanssen2 I did this test twice and both failed... for unknown reason. I am bond by other things at the moment. Will get back to it soon.

@rob-knight Thanks! Yes this algorithm can be expanded to millions of genomes, since it's linear. Calculating MinHash is linear too, but composing MinHash distance matrix isn't. @tanaes would you like to comment?


Need FAQ section update

add FAQ content here

https://docs.google.com/file/d/16iC7HeKwsjngTDUMbRrhxPDihnq5NYNF/edit?usp=docslist_api&filetype=msword

What you want to do exactly

Great work @Arun-cn 🖤


Update docstrings and add examples

Category
Other
Describe the feature you'd like to request
We should do a full audit of all of our docstrings visible in the public documentation at docs.rs/cedar-policy.
While we're at it, we should add short examples of API usage in the docstrings, for functions/methods where that makes sense.  These will be doctests as well.
Describe alternatives you've considered
N/A
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Calling this done. Open a new issue for specific doc improvements


Vizabi offline: create pipeline. create a separate project that would import vizabi and get wrapped (instructions)

Instructions created.

@arthurcamara1 where? :)


ResultSetIteratingPipe gives (incorrect) warning for potential SQL injections

Describe the bug
When using a ResultSetIteratingPipe, the following warning is shown:
The class [nl.nn.adapterframework.jdbc.JdbcIteratingPipeBase$MixedQuerySender] is used one or more times. Please change to [nl.nn.adapterframework.jdbc.FixedQuerySender] to avoid potential SQL injections!
This happens because it uses a MixedQuerySender that extends the DirectQuerySender. I have set the query parameter of the ResultSetIteratingPipe, so I do not expect the warning to apply to my case and thus not to be shown.
Reporter
Ricardo
Category
Use following Tags to categorize your bug report

Frank Console
Frank Framework

To Reproduce
Steps to reproduce the behavior:

Have a ResultSetIteratingPipe with the query parameter set
Load the adapter
See warning

Expected behavior
No warning, as I'm using a fixed query.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

Application server (f.e. Tomcat/WebSphere/JBOSS/Jetty): Tomcat/WebSphere
Browser [e.g. chrome, safari]: Chrome
IAF Version [e.g. 7.5 RC3, 7.6]: 7.6-20200909.154756

Additional context
N/A

Met warnings.suppress.sqlInjections.=true kun je deze onderdrukken.
@alisihab please contact Niels to make this more generic

I could suppress it, but I would say that if the query-attribute is set, the warning should not be given for the ResultSetIteratingPipe.

There you have a punt :)


pin gcc libs

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Anaconda just updated toolchains and associated libraries.
This PR forces this recipe to build using the older toolchain and libraries by using the pins created in:
open-ce/open-ce#431
Issue:
open-ce/open-ce#315.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

pytorch-lightning-bolts is a noarch package so this is not needed.


Autoformat all PHP files with PhpStorm

This allows contributors to autoformat their changed files without
risking to also affect the unchanged code.
This change also fixes a bunch of PSR-12 violations, and it hopefully
also makes the code a bit nicer to read.

I'll rebase and repush after #245 is merged.


grep module match_output should support listing multiple values or using grep for the output

Example:  in sshd_config, consider the parameter PermitEmptyPasswords.
The possible values for this could be No or no or Yes or yes.
when doing an inspection as a blacklist, i want to ensure that this parameter is either commented out, not present, or does not contain either Yes or yes
if I configure the check in the following way:
        - /etc/ssh/sshd_config:
            pattern:      '^PermitEmptyPasswords'
            match_output: "yes"
            tag:          V-38614

it will pass if the actual value is yes but will fail if the actual value is Yes.
I currently have it configured in the following manner:
        - /etc/ssh/sshd_config:
            pattern:      '^PermitEmptyPasswords'
            match_output: "yes"
            tag:          V-38614
        - /etc/ssh/sshd_config:
            pattern:      '^PermitEmptyPasswords'
            match_output: "Yes"
            tag:          V-38614

But the problem here is that, while it will help be catch both failure possibilities, it still returns unnecessary  success results.

For multiple variations on yes/no/other cases, it shouldn't be hard to modify the grep audit to support a list for match_output. The hard pard is dealing with commented lines I guess: we'd have to know all the syntaxes of the config files to be checked in order to know which character is used for commenting. It's not # in all config languages. After defining a list of commenting characters, it's easy to check if the is commented or not.
Also, I'm wondering if this is not a thing for the compliances documents to specify.

The check i'm running here is in the blacklist section, i got to thinking about just specifying the options directly in the pattern section as well... so I tried this:
        - /etc/ssh/sshd_config:
            pattern:      '^PermitEmptyPasswords\s+(yes|Yes)'
            tag:          V-38614

When the value in my sshd_config file is this:
[root@managernode ssh]# grep ^PermitEmptyPasswords sshd_config
PermitEmptyPasswords Yes

Here is my output from hubble:
[root@managernode ssh]# salt \manager hubble.audit stig-rhel6-mac1-classifed V-38614
manager:
    ----------
    Compliance:
        100%
    Controlled:
    Failure:
    Success:
        |_
          ----------
          description:
              (HIGH) The SSH daemon must not allow authentication using an empty password.
          module:
              grep
          name:
              /etc/ssh/sshd_config
          pattern:
              ^PermitEmptyPasswords\s+(yes|Yes)
          tag:
              V-38614
          type:
              blacklist

@avb76 I agree with you on the comments in the config file.  For this reason, in the pattern statement, in the case for sshd_config, i use a beginning of line anchor ^ which causes the search to ignore the commented lines.  I tested multiple scenerios with the various checks i've tried above:
PermitEmptyPasswords no
PermitEmptyPasswords No
PermitEmptyPasswords Yes
PermitEmptyPasswords yes
#PermitEmptyPasswords no
using the ^ anchor causes the #PermitEmptyPasswords variation to be ignored
Another issue to consider:    with grep, would be good to be able to pass kwargs as well.   e.g. the -i   in that case, this wouldn't be an issue.  Further, I think that sshd_config can use upper or lower case versions of the parameters: e.g. PermitEmptyPasswords or permitemptypasswords or PERMITEMPTYPASSWORDS and sshd would work fine.   Therefore, from a compliance check, how do we better support these various situations?

We use the file.grep salt module. Did you try salt-call file.grep /etc/ssh/sshd_config ^PermitEmptyPasswords\s+(yes|Yes) on that system?

And regarding the commented lines, I forgot about the ^ at the beginning of the pattern :)

I did look inside the python module and saw that... but after you mentioned it, I went ahead and tried many variations trying to get the file.grep module to work the way you'd expect...   but I think this example really helps illustrate something:
[root@managernode ssh]# grep -E '^PermitEmptyPasswords\s*(yes|Yes)' /etc/ssh/sshd_config
PermitEmptyPasswords yes

When I run grep -E it works as you'd expect.
However, when doing the same thing using the file.grep module, I tried to use the -E kwarg for file.grep, and the output error'd out:
[root@managernode ssh]# salt-call file.grep /etc/ssh/sshd_config "^PermitEmptyPasswords\s*(yes|Yes)" " -E"
[INFO    ] Executing command 'grep  -E ^PermitEmptyPasswords\\s*(yes|Yes) /etc/ssh/sshd_config' in directory '/root'
[ERROR   ] Command 'grep  -E ^PermitEmptyPasswords\\s*(yes|Yes) /etc/ssh/sshd_config' failed with return code: 1
[ERROR   ] retcode: 1
local:
    ----------
    pid:
        1853
    retcode:
        1
    stderr:
    stdout:

notice on my command line, I have the grep pattern correct, but int he command that file.grep tried to execute it has a \\s* so I think that is causing issues.

@dgmorrisjr, after looking a bit over how file.grep salt module works, this is the correct way to search for that regex:
# salt-call file.grep /etc/ssh/sshd_config '"^PermitEmptyPasswords\s+(yes|Yes)"' -- '-E'
[INFO    ] Executing command 'grep  -E "^PermitEmptyPasswords\\s+(yes|Yes)" /etc/ssh/sshd_config' in directory '/root'
local:
    ----------
    pid:
        28454
    retcode:
        0
    stderr:
    stdout:
        PermitEmptyPasswords Yes

You have to pass the -E option in order to support regular expressions. We just use file.grep with no options.
And to answer your question, I think is best to use a list for match_ouput. Kwargs require some extra work in hubble.py. Using lists, we have to modify only 2, maybe 3 lines grep.py.

I'm good with a list as well at the moment. Not sure yet if i'll come up with other use cases where a list in the match_output wouldn't work.

I used single quotes over doubles quotes. Also, from there usage examples, you'd have to use -- before passing the option.

yeah - i saw the -- in your example, but when looking at the documentation here:
https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.file.html#salt.modules.file.grep
I didn't see anything that indicated the --.  good to know - Thanks!

# salt-call file.grep /etc/ssh/sshd_config '"^PermitEmptyPasswords\s+(yes|Yes)"' ' -E'
[INFO    ] Executing command 'grep  -E "^PermitEmptyPasswords\\s+(yes|Yes)" /etc/ssh/sshd_config' in directory '/root'
local:
    ----------
    pid:
        29084
    retcode:
        0
    stderr:
    stdout:
        PermitEmptyPasswords Yes

Looks like this goes in your way too. I looked in the docstring from the code of file.grep. This iss odd: usually, that docstrings in functions match the ones in the web docs.

super - thanks!

Your problem was with the pattern. If you use only one set of quotes, the module read the whole pattern as one string, but it will pass the string without any quotes to the grep call inside file.grep. If you'll look inside the code, cmd.run is used to call grep. So, any special char will not be escaped.

Regarding match_output as a list, we might want to hear another opinion from @basepi since he wrote designed grep.py.

I think a new optional parameter in the profile check would do better for this: case_sensitive: False. When the plugin sees this, it would use file.grep with the -i flag.

Sounds like https://github.com/HubbleStack/Nova/issues/55 might resolve this. Is that correct?
It would also be pretty trivial to allow for regex on the match_output piece.

I'm excited to try this out next time i get a chance to work on the STIG profiles!


Change Iframe SRC dynamically

Hi,
I have a requirement of changing the iframe src dynamically  when user clicks on different links. Basically need to display different pdfs in the iframe.
Could anyone help me out on my issue. Thank you in advance.
Regards,
Faruq Shaik.

This is likely not a ViewerJS issue.  All you need to do is create an iframe with src to ViewerJS/index.html#yourpdffile
Then from parent page, just update the SRC of that iframe with new pdffile hash.
ie jquery:
$('#youriframe').src('ViewerJS/index.html#yourNewPDF')


audio player first pass

First pass for audio player
There's a temporary "browse" button for adding an audio file to the player. This button won't be there for the final version probably.
What needs to be implemented on back end:

storing bookmark positions (from 1 to 99 probably) for later retrieving
storing last play position (from 1 to 99 probably) so user can resume immediately when reopening a book. Variable "lastPlayTime" is already declared and used in the script.

Super, thanks very much!
The PR touches a few lines of code it shouldn't, looks like your editor was auto-reformatting some files.  In some cases it's good.  I'll try working with the commit in a feature branch.


not supporting java version 12

Issue Type: Bug Report

Write the following environmental information.

OS version: macOS High Sierra 10.13.6

Java version:
$ java -version
java version "12.0.2" 2019-07-16
Java(TM) SE Runtime Environment (build 12.0.2+10)
Java HotSpot(TM) 64-Bit Server VM (build 12.0.2+10, mixed mode, sharing)

Embulk version:
Embulk v0.9.17

Getting error when trying to run the example on the main page:
artem@machine:~/mdpr-vagrant/projects/mdpr_admin (master)$ embulk example ./try1
[WARN] Unrecognized Java version: java full version "12.0.2+10"

Java HotSpot(TM) 64-Bit Server VM warning: Ignoring option AggressiveOpts; support was removed in 12.0
2019-08-01 18:44:04.240 +0900: Embulk v0.9.17
Creating ./try1 directory...
  Creating ./try1/
  Creating ./try1/csv/
  Creating ./try1/csv/sample_01.csv.gz
  Creating ./try1/seed.yml

Run following subcommands to try embulk:

   1. embulk guess ./try1/seed.yml -o config.yml
   2. embulk preview config.yml
   3. embulk run config.yml

artem@machine:~/mdpr-vagrant/projects/mdpr_admin (master)$ embulk guess   ./try1/seed.yml -o config.yml
[WARN] Unrecognized Java version: java full version "12.0.2+10"

Java HotSpot(TM) 64-Bit Server VM warning: Ignoring option AggressiveOpts; support was removed in 12.0
2019-08-01 18:44:07.368 +0900: Embulk v0.9.17
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$2 (file:/Users/artem/.embulk/bin/embulk) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of com.google.inject.internal.cglib.core.$ReflectUtils$2
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
2019-08-01 18:44:08.334 +0900 [WARN] (main): DEPRECATION: JRuby org.jruby.embed.ScriptingContainer is directly injected.
unsupported Java version "12", defaulting to 1.7
2019-08-01 18:44:11.393 +0900 [INFO] (main): Gem's home and path are set by default: "/Users/artem/.embulk/lib/gems"
2019-08-01 18:44:12.680 +0900 [INFO] (main): Started Embulk v0.9.17
2019-08-01 18:44:12.791 +0900 [INFO] (0001:guess): Listing local files at directory '/Users/artem/mdpr-vagrant/projects/mdpr_admin/./try1/csv' filtering filename by prefix 'sample_'
2019-08-01 18:44:12.793 +0900 [INFO] (0001:guess): "follow_symlinks" is set false. Note that symbolic links to directories are skipped.
2019-08-01 18:44:12.796 +0900 [INFO] (0001:guess): Loading files [/Users/artem/mdpr-vagrant/projects/mdpr_admin/./try1/csv/sample_01.csv.gz]
2019-08-01 18:44:12.819 +0900 [INFO] (0001:guess): Try to read 32,768 bytes from input source
2019-08-01 18:44:13.012 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.9.17)
2019-08-01 18:44:13.059 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.9.17)
2019-08-01 18:44:13.108 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.9.17)
2019-08-01 18:44:13.149 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.9.17)
in:
  type: file
  path_prefix: /Users/artem/mdpr-vagrant/projects/mdpr_admin/./try1/csv/sample_
  decoders:
  - {type: gzip}
  parser:
    charset: UTF-8
    newline: LF
    type: csv
    delimiter: ','
    quote: '"'
    escape: '"'
    null_string: 'NULL'
    trim_if_not_quoted: false
    skip_header_lines: 1
    allow_extra_columns: false
    allow_optional_columns: false
    columns:
    - {name: id, type: long}
    - {name: account, type: long}
    - {name: time, type: timestamp, format: '%Y-%m-%d %H:%M:%S'}
    - {name: purchase, type: timestamp, format: '%Y%m%d'}
    - {name: comment, type: string}
out: {type: stdout}

Created 'config.yml' file.
artem@machine:~/mdpr-vagrant/projects/mdpr_admin (master)$ embulk preview config.yml
[WARN] Unrecognized Java version: java full version "12.0.2+10"

Java HotSpot(TM) 64-Bit Server VM warning: Ignoring option AggressiveOpts; support was removed in 12.0
2019-08-01 18:44:20.376 +0900: Embulk v0.9.17
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$2 (file:/Users/artem/.embulk/bin/embulk) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of com.google.inject.internal.cglib.core.$ReflectUtils$2
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
2019-08-01 18:44:21.392 +0900 [WARN] (main): DEPRECATION: JRuby org.jruby.embed.ScriptingContainer is directly injected.
unsupported Java version "12", defaulting to 1.7
2019-08-01 18:44:24.463 +0900 [INFO] (main): Gem's home and path are set by default: "/Users/artem/.embulk/lib/gems"
2019-08-01 18:44:25.721 +0900 [INFO] (main): Started Embulk v0.9.17
2019-08-01 18:44:25.834 +0900 [INFO] (0001:preview): Listing local files at directory '/Users/artem/mdpr-vagrant/projects/mdpr_admin/./try1/csv' filtering filename by prefix 'sample_'
2019-08-01 18:44:25.836 +0900 [INFO] (0001:preview): "follow_symlinks" is set false. Note that symbolic links to directories are skipped.
2019-08-01 18:44:25.838 +0900 [INFO] (0001:preview): Loading files [/Users/artem/mdpr-vagrant/projects/mdpr_admin/./try1/csv/sample_01.csv.gz]
2019-08-01 18:44:25.852 +0900 [INFO] (0001:preview): Try to read 32,768 bytes from input source
+---------+--------------+-------------------------+-------------------------+----------------------------+
| id:long | account:long |          time:timestamp |      purchase:timestamp |             comment:string |
+---------+--------------+-------------------------+-------------------------+----------------------------+
|       1 |       32,864 | 2015-01-27 19:23:49 UTC | 2015-01-27 00:00:00 UTC |                     embulk |
|       2 |       14,824 | 2015-01-27 19:01:23 UTC | 2015-01-27 00:00:00 UTC |               embulk jruby |
|       3 |       27,559 | 2015-01-28 02:20:02 UTC | 2015-01-28 00:00:00 UTC | Embulk "csv" parser plugin |
|       4 |       11,270 | 2015-01-29 11:54:36 UTC | 2015-01-29 00:00:00 UTC |                            |
+---------+--------------+-------------------------+-------------------------+----------------------------+
artem@machine:~/mdpr-vagrant/projects/mdpr_admin (master)$ embulk run     config.yml
[WARN] Unrecognized Java version: java full version "12.0.2+10"

Java HotSpot(TM) 64-Bit Server VM warning: Ignoring option AggressiveOpts; support was removed in 12.0
Java HotSpot(TM) 64-Bit Server VM warning: Option UseConcMarkSweepGC was deprecated in version 9.0 and will likely be removed in a future release.
2019-08-01 18:44:29.626 +0900: Embulk v0.9.17
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException
	at org.apache.bval.jsr303.ConfigurationImpl.parseValidationXml(ConfigurationImpl.java:268)
	at org.apache.bval.jsr303.ConfigurationImpl.prepare(ConfigurationImpl.java:258)
	at org.apache.bval.jsr303.ConfigurationImpl.doPrivBuildValidatorFactory(ConfigurationImpl.java:248)
	at org.apache.bval.jsr303.util.SecureActions$5.run(SecureActions.java:131)
	at org.apache.bval.jsr303.util.SecureActions$5.run(SecureActions.java:129)
	at org.apache.bval.jsr303.ConfigurationImpl.run(ConfigurationImpl.java:337)
	at org.apache.bval.jsr303.ConfigurationImpl.buildValidatorFactory(ConfigurationImpl.java:244)
	at org.embulk.config.ModelManager.<init>(ModelManager.java:24)
	at org.embulk.EmbulkEmbed.newSystemConfigLoader(EmbulkEmbed.java:35)
	at org.embulk.EmbulkEmbed$Bootstrap.<init>(EmbulkEmbed.java:48)
	at org.embulk.EmbulkSetup.setup(EmbulkSetup.java:29)
	at org.embulk.cli.EmbulkRun.runSubcommand(EmbulkRun.java:415)
	at org.embulk.cli.EmbulkRun.run(EmbulkRun.java:90)
	at org.embulk.cli.Main.main(Main.java:64)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.JAXBException
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	... 14 more
artem@machine:~/mdpr-vagrant/projects/mdpr_admin (master)$

Hello, @whitesiroi
As README said, You need to use Java8 for Embulk v9, Java 12 doesn't support yet.

Embulk v0.9 series runs on Java 8

@hiroyuki-sato thank you


Switch api

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our
guidelines: https://github.com/goetzrobin/spartan/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Which package are you modifying?

[ ] accordion
[ ] alert
[ ] alert-dialog
[ ] aspect-ratio
[ ] avatar
[ ] badge
[ ] button
[ ] calendar
[ ] card
[ ] checkbox
[ ] collapsible
[ ] combobox
[ ] command
[ ] context-menu
[ ] data-table
[ ] date-picker
[ ] dialog
[ ] dropdown-menu
[ ] hover-card
[ ] input
[ ] label
[ ] menubar
[ ] navigation-menu
[ ] popover
[ ] progress
[ ] radio-group
[ ] scroll-area
[ ] select
[ ] separator
[ ] sheet
[ ] skeleton
[ ] slider
[x] switch
[ ] table
[ ] tabs
[ ] textarea
[ ] toast
[ ] toggle
[ ] tooltip
[ ] typography

What is the current behavior?

The api of the Helm component is very verbose and needs to use many components and directives.
<label class="flex items-center" hlmLabel>       <brn-switch class="mr-2" hlm>         <brn-switch-thumb hlm />       </brn-switch>       Airplane mode     </label>
Closes #
What is the new behavior?
The api of the helm part is now simplified. You only need to add the hlm-switch
<label class="flex items-center" hlmLabel>             <hlm-switch class="mr-2" />             Airplane mode         </label>
Does this PR introduce a breaking change?

[x] Yes
[ ] No

When using the helm component you have to change the api from:
<brn-switch hlm>
   <brn-switch-thumb hlm />
</brn-switch>

to:
 <hlm-switch />

you have to replace the brn component with the hlm-component.
Other information

Additional "breaking change" is that we now use display contents, wich caused the Switch to be displayed without gap in the card example, the gap was added manually in the example.


No High Gas alert during the exploitation

BOT ID - 0x3858be37e155f84e8e0d6212db1b47d4e83b1d41e8a2bebecb902651ed1125d6
Tx - https://arbiscan.io/tx/0x864c8cfb8c54d3439613e6bd0d81a5ea2c5d0ad25c9af11afd190e5ea4dcfc1f

the bot ignores transactions when they create contracts
@christian-forta wdyt? should we remove this condition or remain it?

lets maintain it and close as wont fix

the reason is that we will already get an alert in the preparation stage


Move to babel 4.3

No need for the jsx loader since babel now supports JSX out of the box.

Thank you!


Update workflows

Build and push toolkit and os flavor before build-iso and build-disk.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f3d8258) 72.05% compared to head (0c44268) 72.05%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1944   +/-   ##
=======================================
  Coverage   72.05%   72.05%
=======================================
  Files          72       72
  Lines        8022     8022
=======================================
  Hits         5780     5780
  Misses       1779     1779
  Partials      463      463

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


Remix-analyzer: LegacyAST to AST

Analysis is now performed on latest AST introduced in solidity 0.4.12
All modules are updated to work according to latest AST
Specific types of AST nodes are defined
Tests are also updated to support module & AST updates

I have run remix-ide nightwatch_local_staticAnalysis tests locally by integrating this PR. It run fine.


Adding Plutus and Smart Contract Support to CardanoSharp

Finally

@NicholasMaselli Please post a mainnet cardano address in the comments here for bounty rewards if you agree with the below terms
Nick: 2000 ADA for Development

@NicholasMaselli Please post a mainnet cardano address in the comments here for bounty rewards if you agree with the below terms
Nick: 2000 ADA for Development

Awesome! Here is the address: addr1q80ukhmvgtm498e3h6pwpe52whpdh98yy4qfwup5zqg7lqz75jq4yvpskgayj55xegdp30g5rfynax66r8vgn9fldndskl33sd

No update on documentation in order to learn how to use this new features?

No update on documentation in order to learn how to use this new features?

@nothingalike is adding this tonight / tomorrow

Transaction Bounty Payment 05dd7cd4df0f18fee676b9b6a5d42090bef9363d246e39d7122f7ae5703435d7


feat: Updates across bootstrap and compositions

What does this PR do?
Updates the terraform example to be consistent with terraform-aws-eks-blueprints crossplane module.
Motivation
While reviewing the examples noticed some inconsistencies.

Make the main.tf example simpler and just do the minimum to show crossplane
Remove references to terrajet provider (deprecated)
Fix example not failing because of not finding provider config for aws and kubernetes

More

[x] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)

[ ] Yes, I have added a new example under examples to support my PR

[x] Yes, I have updated the docs for this feature

[ ] Yes, I have linked to an issue or feature request (applicable to PRs that solves a bug or a feature request)

Note:

Not all the PRs require examples and docs
We prefer small, well tested pull requests. Please ensure your pull requests are self-contained, and commits are squashed

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

This PR depends on terraform PR https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/1220

This PR touches too many different areas. Please break it up into smaller PRs.

@nabuskey I can try but some parts are inter dependent, merging a small PR can cause some of the example not to be in working state

+1 on @nabuskey 's suggestion.
if we keep each PR self-contained it will be easier to track changes and potentially revert if and where needed.  @csantanapr
if you can split this into some logically sound and separate PRs, that'd be great!

I will start working on braking the PR in parts

I split the PR into a series of smaller PRs
https://github.com/awslabs/crossplane-on-eks/pull/70
https://github.com/awslabs/crossplane-on-eks/pull/71
https://github.com/awslabs/crossplane-on-eks/pull/72
https://github.com/awslabs/crossplane-on-eks/pull/73
https://github.com/awslabs/crossplane-on-eks/pull/74
https://github.com/awslabs/crossplane-on-eks/pull/75
https://github.com/awslabs/crossplane-on-eks/pull/76
https://github.com/awslabs/crossplane-on-eks/pull/77
https://github.com/awslabs/crossplane-on-eks/pull/78
https://github.com/awslabs/crossplane-on-eks/pull/79


[Regression] GENERIC_CHART_AXES: numeric x-axis treated as categorical values

When using a numeric column as the generic x-axis, the x-axis values aren't linearly distributed along the x-axis but they are evenly distributed, just as if they were categorical values
How to reproduce the bug

Have the GENERIC_CHART_AXES feature flag enabled
Create a dataset or a SQLLab query with numerical columns, e.g.

select
  *
from (values
  (1,1)
  , (2,2)
  , (5,5)
  , (10,10)
) as v(x,y)

Visualize the query/dataset as bar chart (or line chart)
Use x as x-axis and, e.g., max(y) as y-axis
See that the x-axis values are shown with equal spacing instead of the expected linear distribution of the x values

Expected results
This is what it should look like and what it did correctly in Superset 2.0.1:

Actual results
X-axis values are seemingly treated as categorical values:

Any distribution/scatter/histogram plots based on this functionality are actually unusable due to this bug.
Environment
(please complete the following information):

browser type and version:
superset version: 3.0.2rc1
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

@villebro @zhaoyongjie Could you take a look at this one?

I will take a look, thanks for reporting @rumbin

Hey @villebro I checked the backend response and it's identical in this respect. I fed the json from Superset 2.0.1 backend into Superset 3.0.2rc1 frontend and it also misinterprets XAxis type. This made me convinced it's a FE issue. It looks like the getAxisTime defaults to AxisType.category in this case whereas it should return AxisType.time.

I was able to repro, investigating now

@villebro, we found a fix on our side. @WojtekWaga will report here tomorrow and/or provide a PR.

@rumbin @WojtekWaga oops, I already opened a fix 🙁 Do you mind reviewing the PR #26087 to see if it's in line with the changes you were proposing?

Thank you all!


Textarea scrolling issue

I'm using JQTouch on a site that has a blog with comments.
When I enter long text into a submission form's textarea (such as this one) on a page that is built using JQTouch, if the text length goes long enough that passes below (out of) the confines of the textarea, I can no longer see the text that I have entered past the bottom of the outlined area.

The textarea does not scroll down with me as I enter text below the initial specified textarea.
There is no scroll bar in that text area
Tap events (to call up the selection tool or to move the cursor) don't work.

Yep, you can also tap-drag the magnifier. Going to close as this is "expected" behavior for mobile safari, but an extension would be cool here.


Installer for MacOS

Make an installer for MacOS

This issue won't be worked on until resolution of #1


optimize chaosblade-exec-spi from sub module as isolated module

Signed-off-by: wufan<EMAIL_ADDRESS>
Describe what this PR does / why we need it
optimize chaosblade-exec-spi from sub module as isolated module
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

Why do you do this?

chaosblade-exec-spi  as as spi module, it is  the defination of spi interface.  it is imported and implemented by other application.   as isolated module ,the application of implemention can avoid importing unnecessary jar .


Pod file issue

[!] The 'Pods-OzoneRides' target has frameworks with conflicting names: googlemapsbase.framework, googlemaps.framework, and googlemapscore.framework.

Hi @sarathnk, I would be glad to help, since messing with Google Maps frameworks in Cocoapods can be really daunting, but could you please give me more info about your use case?
To begin with, a copy of your Podfile?
Thanks

@poulpix thanks.Solved issue

may i know how you resolve this issue? kinda having the same problem

Manuallly adding google map framework.


Attach a binary to GitHub releases and deploy backend from that, instead of building on-server

Done, and it's a huge improvement


The diagram here along with the instructions doesn't match

In this document what is said above the diagram on how Azure AD App Proxy works is not what is reflected in the diagram or in the 6 detailed steps below the diagram.  Therefore it is confusing to any reader to understand how exactly it works.
Right above the diagram, you say this: "The external endpoint is how your users reach your applications while outside of your network.
They can either go directly to an external URL that you determine,
or they can access the application through the MyApps portal.

When users go to one of these endpoints, they authenticate in Azure AD and then are routed
    through the connector to the on-premises application."

But that is not reflected in your diagram or in the 6 steps given after it.
Issue 1:
Based on the quoted below text: Step 1 should go the external URL for the application.   That is not reflected.
"external endpoint is how your users reach your applications while outside of your network. They can either           go directly to an external URL that you determine"
a. Where is the external endpoint in the diagram?
b. Shouldn't step 1 be to the external endpoint?

Issue 2:
Text above says: "or they can access the application through the MyApps portal".
Nothing here explains what is MyApps Portal; after searching two links I found that MyApps portal is an external web-based portal. According to https://docs.microsoft.com/en-us/azure/active-directory/user-help/active-directory-saas-access-panel-introduction.
a. Nothing here explains what is MyApps Portal;
b. MyAps portal is not reflected in the diagram.
c. Step 1 doesn't reflect going to the MyApps portal either.

Please correct me if I am wrong.  Shouldn't all the steps be something like this:
1.     The user attempts to access the Azure AD Application Proxy–published application via a web
browser from a device outside the company perimeter network.
2.    The Application Proxy redirects the user sign-in to Azure AD for authentication.
3.    The user obtains the token from Azure AD and presents it to the Application Proxy, which
retrieves the user principal name (UPN) and service principal name (SPN).
4.    The connector installed in the internal network retrieves the user attributes via the outbound
connection to the Application Proxy and requests a Kerberos ticket on behalf of the user from
AD DS. This process relies on Kerberos Constrained Delegation.
5.    AD DS returns the Kerberos ticket to the connector.
6.    The connector presents that ticket to the application.
7.    The application verifies the access and responds to the client request through the Application Proxy.

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

ID: 5010a322-5574-469a-48d7-42ca423891ae
Version Independent ID: 8cef2453-e6bd-f419-ad7f-64d561f9c960
Content: How to provide secure remote access to on-premises apps
Content Source: articles/active-directory/manage-apps/application-proxy.md
Service: active-directory
GitHub Login: @barbkess
Microsoft Alias: barbkess

Thanks for your feedback! We will investigate and update as appropriate.

Hi @DMoonesinghe , thanks for your feedback.
Issue 1: The document says "When users go to one of these endpoints, they authenticate in Azure AD and then are routed through the connector to the on-premises application." Then steps 1-6 show the authentication process that happens at that point. The diagram is an image of the authentication workflow and does not include endpoints because those are not really a part of the high-level workflow (which is what the diagram is intended to show). Can you explain why you think this is unclear? Would it help if the image had a caption?
Issue 2: Perhaps the text could link to the MyApps portal. It isn't really an external portal because it is linked to your Azure tenant and is a way to reach your apps.
As for the steps, I think the diagram is intended to be a high-level look at the Authentication process.
@barbkess what are your thoughts?

Issue 1) The external endpoint in the diagram is where the user is with the PC/mobile device. This happens in step 1 when the user accesses the application.
Issue 2) The user does not have to access it from the my apps portal so it doesn't need to be shown in the diagram (this is an option). The reference to my apps will be included.

Sorry, I wasn't paying attention to this till now.   3 People from
Microsoft had written and I hadn't notice till now. I apologize.  I humbly
disagree with all of you.  What I pointed out to you is an inadequate
explanation and a poor illustration.
Here is what this document says about step 1.
"1. The user accesses the application through the Application Proxy service
and is directed to the Azure AD sign-in page to authenticate."
What you are saying there are two steps.

First Accessing the proxy service for the published app.
Second Proxy Service redirects you to Azure AD.

That needs to be clearly illustrated.  In the steps and then diagramed
properly.   It is not done so in this diagram.  Expecting readers to have
different assumptions is rather silly.
[image: image.png]
I think it is too important a step to blow off by assuming readers would
understand.  You are trying to document a new feature.  Plus look what the
title of the document says.
Point 1:  Microsoft is presenting the following feature:
"Azure AD Application Proxy can be used in place of internal Web
Application Proxy Servers to provide authorized internal users access to
internal, web-based applications from outside the security of the corporate
network."
Point 2 : The Title of the Document is:  How to provide secure remote
access to on-premises applications
I don't think your diagram is doing justice to the feature or the
document title by not clearly illustrating that point.
How hard is it break it down to two steps and put up a new diagram that
clearly illustrates the whole process without expecting readers to have
different assumptions?  The title is "How to Provide Secure Remote Access"
it is not about "how Application Proxy" does Authentication.
Either way, it's up to you folks to have the last word.  But you are not
helping the readers see clearly the whole process involved with " How to
provide secure remote access to on-premises applications" if you don't want
to clearly document and illustrate the whole thing.
Cheers! Ladies!
On Tue, Nov 13, 2018 at 5:06 PM Jasmine Perez<EMAIL_ADDRESS>wrote:

Issue 1) The external endpoint in the diagram is where the user is with
the PC/mobile device. This happens in step 1 when the user accesses the
application.
Issue 2) The user does not have to access it from the my apps portal so it
doesn't need to be shown in the diagram (this is an option). The reference
to my apps will be included.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/14307#issuecomment-438456269,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlFSpv4GuISWcLjC9rZ9dtbtH-KkFjntks5uu0JlgaJpZM4WUJqd
.

--
Deepal Moonesinghe
(408) 568 1669
<EMAIL_ADDRESS>"We must accept finite disappointment, but we must never lose infinite
hope."
~ Martin Luther King, Jr. ~

@DMoonesinghe : I reworded some of the wording in the https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/application-proxy#how-does-application-proxy-work section.  This is what you were describing originally.  It should get published sometime on 12/6/2018.  I'm going to be revising the application-proxy topic so some of the other things you mentioned should get fixed in the rewrite.
Let me know if its OK to close this issue.  Thanks!

@DMoonesinghe : I verified the fix is out there.  I am going to go ahead and close the issue.
@MicrosoftDocs/azure-cxp-triage : please close this.

Sorry I forgot what this was about.  I am sure you corrected it and closed
it. Couldn't find this page. But that is fine.
On Fri, Dec 7, 2018 at 4:55 PM Barbara Kess<EMAIL_ADDRESS>wrote:

@DMoonesinghe https://github.com/DMoonesinghe : I verified the fix is
out there. I am going to go ahead and close the issue.
@MicrosoftDocs/azure-cxp-triage
https://github.com/orgs/MicrosoftDocs/teams/azure-cxp-triage : please
close this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/14307#issuecomment-445377957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlFSprjjUfpwT56cZzPwPsbaSuL4Jnz1ks5u2uPZgaJpZM4WUJqd
.

--
Deepal Moonesinghe
(408) 568 1669
<EMAIL_ADDRESS>"We must accept finite disappointment, but we must never lose infinite
hope."
~ Martin Luther King, Jr. ~


Target multiple partials

Could be similar to class updates for dirty states (e.g. unicorn:partial="id1 id2"), but might be weird with id or key modifiers. Another approach might be to support multiple unicorn:partial attributes, e.g. unicorn:partial="id1 id2" unicorn:partial.key="key1".

Hey! I've been using this framework, and this would be so incredibly helpful. Any updates on this?

Hi! Hopefully the framework is proving helpful. :) I think I know how the code for this feature would work, but I haven't started looking at it, yet. I'll try to get it into the next release, though.

Closing because this has been merged into master and will be available in the next release.

This is now available in https://github.com/adamghill/django-unicorn/releases/tag/0.27.0.
It works by supporting multiple partial attributes on an element, e.g. <button unicorn:click="some_method" unicorn:partial.id="id1" unicorn:partial.key="key1">.