[Feature]: support tailwindcss plugin

What problem does this feature solve?
Support tailwindcss by a rspress plugin, then the user can apply tailwindcss styles in the rspress site.
What does the proposed API look like?
// rspress.config.ts
import { tailwindRspressPlugin } from '@rspress/plugin-tailwindcss';

export default {
  plugins: [tailwindRspressPlugin()]
}

Rsbuild will automatically read postcss.config.js, users just need to follow the Tailwind CSS official guide: https://tailwindcss.com/docs/installation/using-postcss

ok, then i think we just need add a guide document for rspress to tell user how to integrate tailwindcss.

yeah that would be nice


Changeset.validate_change behavior question

It seems a little weird to me that all the built-in validation functions (inclusion, exclusion, etc.) update the errors list with a {field, error} tuple, but the validate_change function doesn't pass the field to the custom validator, so you have to jump through a few hoops to write a reusable validator that can also return the errors in this form.
That is, say I had a validator that ensured that value wasn't just whitespace (ignoring the fact we could probably do this with a regex and validate_format):
def blank?(value) do
  length = value |> String.strip |> String.length
  if length == 0, do: ["cannot be blank"], else: []
end

So if I did validate_change(changeset, :whatever, &blank?/1), the returned changeset's error's list would be ["cannot be blank"], rather than [{:whatever, "cannot be blank"}].
Additionally, if this same validator is used for two separate fields in the same changeset, the changeset's error list becomes: ["cannot be blank", "cannot be blank"]
It seems to me that it might make sense to either have validate_change store the field name and the validator function's error's as a tuple, as the other validate functions do, or to pass the field and value to the validator function, rather than just the value.

@jordan0day So we are theoretically supporting the same validator to attach multiple error messages in different fields, that's why it works the way they do. We could though "pass the field and value to the validator function, rather than just the value".
/cc @drewolson @jeregrine @ericmj thoughts?

I struggled with exactly this problem after reading the docs. I did some source reading and it seems to me that validate_change is intended to be used as an internal helper function for custom validators. It should be used to write custom validators that only fire on changes.
@jordan0day I would probably write your validator as:
def validate_present(changeset, field) do
  validate_change(changeset, field, fn val ->
    length = value |> String.strip |> String.length
    if length == 0, do: [{field, "cannot be blank"}], else: []
  end)
end

If you wanted to reuse blank? you could partially apply it.
def validate_present(changeset, field) do
  validate_change(changeset, field, &blank(field, &1))
end

You could use it in your changeset like so:
def changeset(model, params \\ nil) do
  params
  |> cast(model, ~w(...), ~w(...))
  |> validate_present(:field1)
  |> validate_present(:field2)
end

At the end of the day, I think we should focus on making the docs for validate_change clear that it is intended to be used as an "internal" helper for writing custom validations.
That said, for reusability I do think it might make sense to have the function passed to validate_change have an arity of 2 and accept the field name.

I have changed validate_change to pass the field as argument too. Pull requests to improve the documentation are always welcome!

:+1:
Thanks @josevalim and @drewolson!


Feature Request - expose a method to display the binary version of the app

Add the ability to obtain the current build version even when no update is available.
This can perhaps be implemented by modifying getCurrentPackage to return an actual object containing the appVersion instead of null. Another option is to expose it through any new API.
https://docs.microsoft.com/en-us/appcenter/distribution/codepush/cordova#codepushgetcurrentpackage
The purpose of this request is to allow users to always view the version number of the currently installed app. This can be easily implemented by combining the app binary version along with the update label (managed by codepush).

Hey there!
Thanks for the feature request. Our getCurrentPackage was made to retrieve metadata about the currently installed "package", which means even when no update is available you shouldn't be getting null if there has been a CodePush release (in most cases, there are a few exceptions).


SS4: Gridfield RelationEditor search results escaped

SilverStripe4-beta4: The search results in Gridfield's relation editor (to add new link) are displayed as escaped html.

Fixed with https://github.com/silverstripe/silverstripe-framework/pull/7502


edit Composition and Inheritance

Does "Override the schema name by overriding the property with a new value," mean override the schema name by giving the property a new value? If so, it might be better not to use "overriding" again.


Submit sample for app

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

What's in this Pull Request?
This is a lightweight admin tool to get an overview of guest users in your tenant.
Features:

List all guest users in your tenant
See when they last signed in
Regenerate guest invitation link
See which groups they are a member of
See their sign in history
Block/unblock guest users
Open their profile in Entra
Open AAD groups in Entra
Delete users
Get a quick indication of which users are active and which are not via simple color coding
Edit the users profile information

@Label: Teams global hack

Great job @jeppesc11 and Dan Toft! Works like a charm. I have added gitignore file, telemetry url (new one) and a template for search in adoption site.
All yours to merge @BobGerman :)

NOTE - I just realized the name for this sample didn't follow the naming convention. Samples with only a tab should begin "tab-"; "app-" is for samples that include more than one Teams app capability such as a tab plus a message extension. Please LMK if I was incorrect and this supports more than the tab.
SO I just renamed the sample tab-teams-guest-user-overview
Apologies in advance for any confusion @jeppesc11 @rabwill
And thanks for a great sample - super useful and a great idea!

You are totally right @BobGerman - I think I misread the guide :-)


Add chainerx.moveaxis

Reference : #6423
TODO:

[x] Add Tests
[x] Add Documentation

Jenkins, test this please.

Jenkins CI test (for commit 79d1baa3a011f3fd9c258c36990eb11b01f6ceb4, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit cd527c53d20dc183ca01f930d3b02d1acc2fb901, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 155f9c079e2a2f8c6c5ee4607953ac8124da4af7, target branch master) failed with status FAILURE.

Jenkins failure seems unrelated to this PR. LGTM!
Thank you for the PR!


Mini-flash of old todos is seen before showing new todos

See the following GIF 👇 (~3MB)

this is a known-issue.
will be fixed in the next version.


i got an api key  from the marvel API but i don't know how to format it in the secrets file

Can you provide a sample of the correct format?

I found this in the gradle file which i presume is the right format but I got the following error with this -->
app.marvel_base_url=developer.marvel.com
app.marvel_public_api_key=f044d9ab0e9745a30fd905a67bf4
app.marvel_hash=68abde2aada9617f04d7dbdc876a7026dc95f
I cut off a few characters to keep the key private.  Do i have to use a keychain tool?  What tool and hashing algorithm should I use?

I managed to get the strings to work but it fails to find BaseViewModel file.  Is this from a dependency?

I think it is. Unfortunately, this is also an outdated sample so I can't say that I still agree with how this project is structured. If you'd like, I think Now In Android is a better resource to see what's in Android Development nowadays.


Reset yarn cache

Some experiments corrupted it, making the v3 cache no longer usable

bors try

bors merge

bors merge


As GitHub generates the PR title from the first commit, it would be nice to allow this locally.

Nice!


update JSON serialization to use CIMI metadata

Evaluation done in #363.  Implementation being tracked in other tickets.


Change API to be able to pass array(:,:,:) kind of objects

There is also a fix for calculation of IBS forces in LAPW. The radial wave functions should not be recomputed.

Hi @mtaillefumier ! This might break the cp2k interface in favor of a more Fortran-ish style. I'm testing it now with Exciting and QE.


Invalidate throws if the cache is empty

@oliverjash and I spent ages debugging an issue only to realise it was our mistake and that the cache wasn't populated.
This PR changes the behaviour of invalidate to throw if the trace cache is empty. I understand that throwing might be controversial, and I'd happily consider swapping for console.log.
@guybedford what do you think? I think this could help with debugging.

Invalidate returns the array of modules that were invalidated - I think that should be enough information to debug if that list is empty?

That doesn't let you distinguish between the two scenarios:

You had the right module name, but the cache is empty
The cache is populated, but the module name isn't correct

This PR immediately spots and alerts you to (1), hopefully saving some debugging time. The more I think about it the more I'd swap the error for a console log mind.

Clearing a cache is a safe guaranteeing operation. That is I can run it aggressively to know that the cache has been cleaned. Throwing an error in some cases removes the ease of that use case. Imagine if clearing your trash can on a mac gave an error when there was no trash for example. Also the only time we output log messages is for deprecation warnings in this project.
If you're looking at avoiding your original mistake, it may help to understand what the original issue was that you still thought you needed to be clearing the cache properly?

@guybedford :+1: fair points, particularly on being able to invalidate regardless of the state of the cache.
I can't remember the exact issue @oliverjash and I had, but I remember commenting saying that if SystemJS had warned us the cache was empty it would have been useful.


speed up suggestions?

Hello,
I'm thinking about using the sentiment neuron to create sentiment-colored, blocks of text like they did in the paper in Figure 4. The code I'm using to calculate sentiment is simple, something like:
`from encoder import Model
model = Model()
text = ['This is a terrible product. Very bad!']
text_features = model.transform(text)
text_features[:, 2388]`
I've got a quad-core macbook pro and no GPU. Any suggestions on how I could speed up the calculations? Is there a way to specify that all available cores should be used?

I can't remember exactly how but you should be able to set the number of cores/threads you want to be used for BLAS


Some words confuse movie name parsing.

Entry title is "dan in real life 2007", parser stops at "real"
2016-05-01 09:32 DEBUG    imdb_lookup   library_movies_cleanup_test lookup for d
an in real life 2007
2016-05-01 09:32 VERBOSE  imdb_lookup   library_movies_cleanup_test Searching fr
om imdb `dan in real life 2007`
2016-05-01 09:32 DEBUG    parser_internal library_movies_cleanup_test Parsing mo
vie: `dan in real life 2007` kwargs: {}
2016-05-01 09:32 DEBUG    movieparser   library_movies_cleanup_test parts: [u'da
n', u'in', u'real', u'life', u'2007'], cut is: real
2016-05-01 09:32 DEBUG    movieparser   library_movies_cleanup_test after parts
check, cut data would be: `dan in` abs_cut: 6
2016-05-01 09:32 DEBUG    movieparser   library_movies_cleanup_test data cut to
`dan in` - this will be the name
2016-05-01 09:32 DEBUG    parser_internal library_movies_cleanup_test Parsing re
sult: <MovieParser(name=dan in,year=2007,quality=unknown)> (in 0.92 ms)
2016-05-01 09:32 DEBUG    utils.imdb    library_movies_cleanup_test smart_match
name=dan in year=2007
2016-05-01 09:32 DEBUG    utils.imdb    library_movies_cleanup_test Searching: d
an in

i think thats related to proper detection. i remember seeing some old, and still open, bugs about that relating to series names confusing the detection of propers: http://flexget.com/ticket/2948 , http://flexget.com/ticket/2935

yeah, that one is super tricky though. try using guessit in the meanwhile

Waiting on #1138


Automatically loading a new configuration incorrectly when creating it

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Version: 1.86.2 (system setup)
Commit: 903b1e9d8990623e3d7da1df3d33db3e42d80eda
Date: 2024-02-13T19:40:56.878Z
Electron: 27.2.3
ElectronBuildId: 26908389
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

Steps to Reproduce:
Currently when creating a new profile vscode will switch to the new profile and reload, but I just want to back it up,
This is inconsistent with the semantics of create profile, maybe you need to add create and switch new profile option

I think this is as designed. For backup I suggest you use the Export option.
https://code.visualstudio.com/docs/editor/profiles#_share-profiles

The described behavior is how it is expected to work. If you disagree, please explain what is expected and what is not in more detail. See also our issue reporting guidelines.
Happy Coding!


Expired NuGet key

error: Response status code does not indicate success: 403 (The specified API key is invalid, has expired, or does not have permission to access the specified package.).
https://ci.appveyor.com/project/MxLabs/anviz-ebvkk/builds/35410606
Can you take a look? Thanks

New keys added.
Thanks for your informing.


InteractionManager throws 'cannot read property click of undefined' after removeChild

It's hard to make some very small selfcontained example showing this, I can try though, but what I do is calling removeChild in batches, then I always get the error as described in the title:
on this line =
if (item[clickFunction] || item[upFunction] || item[upOutsideFunction])

in
PIXI.InteractionManager.prototype.onMouseUp

I've fixed it by just putting a little escape plan in place
var item = this.interactiveItems[i];
	if (! item){
	    continue;
	}
        if (item[clickFunction] || item[upFunction] || item[upOutsideFunction])

Yo! this should be sweet on v3 now :+1:


Image caption size increased

@shataxi
This is good. Can you

in the issue comments add before/after images of the rendering so that I can see the benefit from this pull request
rename class to something more meaningful, say large-caption-text

Earlier the caption of images were small in size and light in colour in both normal mode and lightbox mode.
Now the caption of images are large and bold.
For adding CSS to the captions in normal mode, a span is added below each image that contains the caption.
For adding CSS to the captions in lightbox mode, description is added for each image.

@shataxi
Can you add further details as done in https://github.com/probml/pyprobml/pull/1112
You mainly want to show images before your PR and after your PR. That way I can directly merge without the need to test. You can do so for both the regular and the lighbox mode.

Before PR
https://github.com/nipunbatra/lab/assets/67477720/507541a2-3bb0-4a92-96ee-a9d16baa0de9
Normal Mode :

Lightbox mode :

After PR
https://github.com/nipunbatra/lab/assets/67477720/54ab32d3-084a-4d12-9c89-55a3757d792c
Normal Mode :

Lightbox Mode:


Enable the Timber Sales Manager name field if the applicant is BCTS.

Describe the task
For FOM "Timber Sales Manager name" field, only show the field when it is BCTS FOM, if not hide the field.
Acceptance Criteria

[x] Conditionally show (when Holder matches Timber Sales Manager...) "Timber Sales Manager name" field and hide otherwise .
[x] On display side (admin/public frontend), only show this field if value present.

Looks good


Search form interactivity

[x] don't submit the form when a checkbox is selected
[x] only submit fields that have defined values (otherwise querystring is needlessly long)
[x] overlay the spinner image on the page when search is active?

"only submit fields that have defined values (otherwise querystring is needlessly long)" no wonder why it was so slow before! so much better now!
Is it easy to retain the scrolling position after page refreshing? it will be really helpful if I can stay where i've been scrolled after the page refresh instead of having to start from the top and go down again.

@xinyil don't think I understand. wouldn't you have to scroll to the top to change a filter anyway?

but you don’t have to scroll back all the way to the banner. also originally the filter is designed to be fixed as you scroll... we might not have time for this at this point.

On Apr 5, 2018, at 2:03 PM, Nick Budak<EMAIL_ADDRESS>wrote:
@xinyil https://github.com/xinyil don't think I understand. wouldn't you have to scroll to the top to change a filter anyway?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Princeton-CDH/derrida-django/issues/196#issuecomment-379025245, or mute the thread https://github.com/notifications/unsubscribe-auth/AQ6bOOlxLPfVgubAQsFw1F41Od0Dxbu7ks5tllyCgaJpZM4THDyL.

would the the filters be "sticky"? i.e. when you scroll further down the header stuff goes away, but the filters stay fixed to the top? it seems like a good way to show more content, and theoretically preserving scroll in that situation would be possible. don't think we have time right now though.

Yes filters were designed to be sticky but this function must be overlooked somehow. I'll add it to the wishlist.
Maintaining scroll position between page load and when going back to the previous page is a different question, and I feel it's a fairly common behavior. If it's not easy to incorporate now we can add to wishlist. But it really disturbs the experience especially when you are browsing a list, and wanting to go back to the list from a detail page.

I'll do some digging about how to implement it - there's not a standard way I know of, but I've never really tried it, so worth looking up

Adding sticky filters to the wishlist; everything else here is done.


DNM: first try on modal for keymaps

You can try it here: http://kasperpeulen.github.io/dart-pad/ (by pressing ctrl-? or cmd-?)

The interaction is getting there :-), but the UI needs some further development.

I really dislike anything that looks like a modal dialog box on the web. Could we style it to look more like the overlay? I.e. a slide up from the bottom?

The UI style is very different to the rest of DartPad, could you make the theme more consistent?

I get that you're trying to give the suggestion of keys for the things on the right -  I don't think it's quite working visually at the moment, they look more like slightly weird buttons - As this bit may eventually options in it, could you make them more explicitly keyboard actions, e.g.  + g, style just as text rather than buttons.

An 'open in new window' tearoff would be great :)

It's not obvious how you close this UI

@lukechurch Yeah I haven't put much on the UI yet. I just copied the style of github setting dialog a bit.
What is the difference between a modal and an overlay ? Do you mean that you don't want the background to be grayed out ? Shall I style it like the toast ?
About the keys, for mac, I would like to style them with unicode symbols for mac like this:

@kasperpeulen This gets a little subtle. The difference is in connotation. Modal dialogs were a very very over-used feature of desktop systems. The interaction is similar, but the visual appearance is very different.
Having it appear from the bottom with slight transparency helps - it means that you don't completely loose the context of what's under-neath it.
I like the visual appearance of the one in your comment much more.
So can we try styling it like that, vertically aligned on the bottom of the page?

 Things should either have rounded corners or not and should be consistent thoughout the whole UI. We're in rectilinear land at the moment, so lets make this have sharp corners as well please.

I still don't get how tihs "overlay" should look like :) Could you show an example ? Should the background be grayed out for example ? Do you want it look like the toast ?

We're in rectilinear land at the moment

I'm fine with sharp cornes, but I actually see rounded corners throughout the UI: the buttons, the selection menu for demos on the right, the completion popup, the issue labels. I would say most things are rounded now.

I still don't get how tihs "overlay" should look like :) Could you show an example ? Should the background be grayed out for example ? Do you want it look like the toast ?

Sure, np, I'm being a little abstract - I'll mock up an example for you in a bit.

I'm fine with sharp cornes, but I actually see rounded corners throughout the UI: the buttons, the selection menu for demos on the right, the completion popup, the issue labels. I would say most things are rounded now.

Heh. Good point - I should be more precise. Look at the 'radius of curvature' of the corners. Compare e.g. the run button or the new button with the corners on the popup. On my screen the run button is about 1 px of rounding and the corners on the menu are around 3px? This was most glaring here because of the very strong contrast.
The completion dialog I find hard to tell - it's somewhere in between it has too many layers in it. It's due for some visual simplification soon.
The issues and labels are more rounded I agree.
So perhaps this would better expressed as: We need visual consistency on corner rounding throughout the UI. Lets make everything 'as round' as the corners of the Run button. Would you be up for taking a crack at doing this?

The new toast UI might serve as a good template for dialog UIs:

@kasperpeulen @devoncarew +1 for using the toast as a template.
To be specific, lets try something then: A toast, centered in the middle of the screen horizontally, aligned with the bottom. With enough lines to display the needed settings etc.
Hyperlinks for 'open in new window' and 'close' in the top right of the toast.
Does this make sense?

FYI, I have a branch I'm working on to refactor the anon sharing UI a bit. I adds a dialog and a glass pane; I think a lot of the code would be useful in this PR as well. I'll probably be done with it in a day or so -
https://github.com/dart-lang/dart-pad/tree/devoncarew_sharing_ui

@devoncarew Nice, I've forked that branch for a new PR (which I haven't finished yet).
It looks like this now (on windows):


[MRG][FIX] Fixes to psd_topo and added megsim data download to circle.

I don't know when plot_psd_topo stopped working, but here is a fix to it. Also,  seems that megsim downloading wasn't included in circle, so it's included here.

Test updated. I wonder if we should at some point update the testing data to have ias channels.
Ready for review.

thx @jaeilepp


State of the projet

Hi,
I'm really interested in the state of the project as I'd like to use Druid Batch Indexing but I'm very reluctant to deploy a Hadoop cluster for this sole purpose.
Is this working ? Will be maintained in the future ? Is there any doc on how to use it ?
Regards,

Hi,
First of all sorry, i missed this until now. Wasn't expecting interest in this at such an early stage.
I am hoping to have a procedure in place for indexing in a few days. Stay tuned for a detailed example on how to index tpch.  This indexing functionality is meant to be used with the spark-druid-olap project which then uses the index to speedup queries against the indexed star schema.
Feel free to reach out to me if you want to discuss more details.
regards,
Harish

This functionality is available as create/insert of OLAP Indexes in our SNAP BI Platform


search box for services losing characters

go to https://microsoft.github.io/jacdac-docs/services/
quickly type the word "character" in the search box
See that typed characters are lost:

fixed.


docker stats all

There needs to be a cleaner way to get docker stats for all running containers from the cli.
docker stats `docker ps | awk '{print $NF}' | grep -v NAMES`

is just way too much to type.
simplest is to just have docker stats provide stats for all running containers instead of usage

+kind/feature

Already reported in https://github.com/docker/docker/issues/10772


fix(theme): fix typo in swizzle component config

Pre-flight checklist

[x ] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

Thanks 👍


Feature request to help support Multi-Tenant scenarios

We have started using this to support Multitenant deployments where each tenant has its own AzureAD Tenant for showing people to pick from, and at the moment we are making custom packages inheriting from the base AzureCP and overriding the CheckIfShouldProcessInput method to add a hardcoded filter against the context parameter.  This works great to limit the people pickers output based on the tenant url to just their AzureAD, but for every new tenant we bring on board, we need to create a new custom AzureCP with a hardcoded URL to filter by.  What I would like to ask for an upcoming version would be an additional data string field on the IAzureCPConfiguration so that we could actually pass in additional data from the settings UI, so then we could put in some JSON or simply the specific tenants URL in this case, and then we can use that in custom implementations versus hardcoding.  I realize I would still need to make custom packages for each tenant but I could then make AzureCPCustom01 - 50, deploy them all at once and deploy in a new farm and then our Ops guys just need to go in and configure each adding an extra bit of information (the url) as they bring new tenants online over time reducing our need to wait for outage windows (since deploying the hardcoded tenant version restarts the farm).

If I understand, you want a custom field in the persisted object that you can use for your own needs.
What would be the type of this field?

That's it exactly... a simple String property for additional data would be perfect as then I can customize the settings UI to set it and read from it in the CheckIfShouldProcessInput to get the URL (or anything more complex I could parse it as JSON), and apply the filter.

It's not urgent, as we currently use the data hardcoded in each WSP and that does work.  It's more a matter of ops process improvement for us.  Whenever you update next would be fine, as knowing its coming at some point at least allows us to plan ahead.

@lunarridge The release is now published: https://github.com/Yvand/AzureCP/releases/tag/14.0.20190307.660

Awesome.  Thanks so much, I will grab the latest and test it out in the next week or so.

@lunarridge were you able to test if the new release fits your needs?


shutdown old GH pages site for quickstarts dashboards

References
https://newrelic.github.io/quickstarts-dashboard-library/#/
https://github.com/newrelic/quickstarts-dashboard-library
Acceptance Criteria

[x] old GH page site is no longer available

this is done.


Update for Dockerfile

This addresses issue #17 , and should provide a cleaner conda environment.

Aha cool - so the entire Dockerfile needed replacing! Sorry, I thought that #17 was just about adding the one conda clean command..

Ok, changed / merged things. Sorry, should've updated my fork beforehand -.-

No worries!

:rofl:


instancetype: Label ControllerRevisions with stored object metadata

/area instancetype
What this PR does / why we need it:
ControllerRevisions are used to store point in time copies of instance
type and preference objects for each VirtualMachine making use of the
feature, see the following user-guide docs for more on this:
Instance types and preferences - Versioning
https://kubevirt.io/user-guide/virtual_machines/instancetypes/#versioning
This change adds labels to these ControllerRevision, exposing metadata
of the stored objects. With this in place it becomes easier to track the
objects stored within these ControllerRevisions and in the future
orchestrate the migration of these objects to newer versions of the API
and CRDs as these are introduced.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #9925
Special notes for your reviewer:
Release note:

`ControllerRevisions` containing `instancetype.kubevirt.io` `CRDs` are now decorated with labels detailing specific metadata of the underlying stashed object

/retest-required

/cc @0xFelix

Thanks!
Looks good to me. But this only helps converting newly created ControllerRevisions, right? We still need to figure out how to convert older existing ones?

Yes I think any conversion is going to assume that these labels aren't present to be begin with.

I also like that his allows for deduplication of ControllerRevisions!

Yeah hopefully allows us to look things up quickly without relying on the names etc.

I assume functests are not needed at this point?

I didn't think so at this point but happy to add something if required.
I'm still not sure about the names tbh, I think these could easily conflict in their present form so I might make them a little more specific to the ControllerRevisions this morning.

/cherry-pick release-1.0


fix pip installation

@rsignell-usgs , this would solve the problem you were seeing. You can test, if you like.

@martindurant , the environment file that allows me to run the examples is:
name: xrviz
channels:
  - conda-forge
  - defaults
dependencies:
  - cartopy
  - hvplot
  - intake
  - intake-xarray
  - nb_conda_kernels
  - netcdf4
  - panel
  - pip
  - xarray
  - pip:
      - git+https://github.com/intake/xrviz

I assume this is for your general work - xrviz does not call intake stuff.
Looks like I was probably wrong to assume that listing panel also got hvplot or that listing xarray automatically got the cdf and carto stuff.
Which package comes from defaults?

You are right.  Intake is not needed for the current demo.   But everything else is.

OK, so I'll merge this and make a release on conda-forge, just to show we can.


Typo in documentation of time_periodic argument of Fieldset constructors

Hey there, I think there is an typo in the docstrings. I think it should be allow_time_extrapolation instead allow_time_interpolation in

time_periodic (bool, float or datetime.timedelta) – To loop periodically over the time component of the Field. It is set to either False or the length of the period (either float in seconds or datetime.timedelta object). (Default: False) This flag overrides the allow_time_interpolation and sets it to False

Cheers + many thanks for developing this awesome tool!!

Good catc, @leoluecken! Thanks so much for reporting. I've fixed it in #1383


Add renderer argument to TagWidget to match Django widget interface

Starting with Django 1.11, if a widget's .render() method does not have a renderer argument, a warning is issued of the form:

RemovedInDjango21Warning: Add the renderer argument to the render() method of <class 'taggit.forms.TagWidget'>. It will be mandatory in Django 2.1.

Handle this argument to be future compatible and squelch the warning.
Added workaround for older Django versions as it is not always supported.

Not supported: Django < 1.11
Supported: Django 1.11 and 2.0
Required: Django master

Tests against the master branch now pass.
For additional information see the Django release notes:
https://docs.djangoproject.com/en/1.11/releases/1.11/#id2

The renderer argument is added to the Widget.render() method. Methods that don’t accept that argument will work through a deprecation period.

And the Widget.render() documentation:
https://docs.djangoproject.com/en/1.11/ref/forms/widgets/#django.forms.Widget.render

Changed in Django 1.11:
The renderer argument was added. Support for subclasses that don’t accept it will be removed in Django 2.1.

Codecov Report

Merging #516 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #516   +/-   ##
======================================
  Coverage    87.8%   87.8%
======================================
  Files           5       5
  Lines         541     541
  Branches       96      96
======================================
  Hits          475     475
  Misses         52      52
  Partials       14      14

Continue to review full report at Codecov.

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

If you agree with the approach taken in #518 then this PR can be closed.


Failing build with C# VS 2013 Project

Hello,
I'm using this project to automate builds for AX2012 R3 on TFS 2015.
As far as I understand included Visual Studio C# Projects - once they are added to the AOT and to TFS repository - will be compiled by the build mechanism and no other steps should be necessary to make the resulting DLL “visible” to the X++ code. Is this statement correct?
I’m asking this because it seems that C# projects are not automatically compiled, thus causing compiling errors on X++ code:
\Classes\FAB_CustomerVerification\verifyCustomersActivity, line 3, column 5 : Variable FAB_CrifIntegrationV2 has not been declared.
This is the verifyCustomersActivity code:
public void verifyCustomersActivity(Set customersToBeVerified)
{
    FAB_CrifIntegrationV2.ServiceReference.SearchClient searchClient;
    FAB_CrifIntegrationV2.ServiceReference.CompanySearchData searchData = new FAB_CrifIntegrationV2.ServiceReference.CompanySearchData();
    FAB_CrifIntegrationV2.ServiceReference.InquiryCompanyDetails inquiryDetails;
    FAB_CrifIntegrationV2.ServiceReference.CompanyItem[] companyList;
    FAB_CrifIntegrationV2.ServiceReference.CompanyItem item;
                …

while the object declared are defined in FAB_CrifIntegrationV2 C# project, included in the AOT and the correct model (I'm using VS 2013). I also wrote a job that executes that method and it works correctly if executed both on client side or server side.
Please, could you point out my mistake or suggest checks that could fix the issue?
Thank you in advance for any help/suggestion.

Try setting the CodeCrib.Build.AOTCompilePaths build pipeline variable to an array (PowerShell notation) of the projects you need to compile.
@('\Visual Studio Projects\C Sharp Projects\abc_Project','\Visual Studio Projects\C Sharp Projects\SRSDrillThroughCommon','\Visual Studio Projects\C Sharp Projects\xyz_Report.BusinessLogic')
This compiles the VS projects using the ax32 client prior to building everything with axbuild.  It's a bit of a holdover from previous versions where it had to run these compiles in a specific layer to successfully save the build output back to the AOT in the right place.  Now axbuild offers a layer parameter to run in a specific layer, but the build libraries don't use that option yet.

Sorry @dan-we , where do I set that variable? in the process XAML?

For a XAML build, it's a parameter on the build definition named AOT nodes to compile (no changes needed in the build process template XAML file).  It's a TFS string list type so you can enter multiple AOT paths using the ... button.


feat(policy): DSP-51 - deprecate PublicKey local field

resolves: https://github.com/opentdf/platform/issues/1358

reserved fields are not supported in oneOf
https://github.com/protocolbuffers/protobuf/issues/5338
https://github.com/protobufjs/protobuf.js/issues/1008


[Bug] Entries per page are not remembered (persistent table - true)

Bug report
What I did
Config persistentTable=true
I chose a different number of items per page than default on datatable. I edited some item and then chose save and back.
What I expected to happen
To return to items listing on the same page with same number of items per page.
What happened
But I was returned to the first page with the default number of items per page.
What I've already tried to fix it
let $dtCachedInfo = JSON.parse(localStorage.getItem('DataTables_crudTable_/{{$crud->getRoute()}}')) ?? [];
var $dtDefaultPageLength = {{ $crud->getDefaultPageLength() }};

if($dtCachedInfo.length !== 0 && $dtCachedInfo.length !== $dtDefaultPageLength) {
   //localStorage.removeItem('DataTables_crudTable_/{{$crud->getRoute()}}');
}

I commented the line above in datatables_logic.blade.php, and seems to be working well on this issue.
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 7.4.11 (cli) (built: Sep 29 2020 13:18:06) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
LARAVEL VERSION:
v8.28.1@73dd43d92fcde6c6abc00658ae33391397ca119d
BACKPACK VERSION:
4.1.34@d9ec59ab1f9fe03c7106911b5667d912c1cfe1f7

Hello @spajz
Thanks for the bug report and sorry for the trouble.
We introduce a few changes (lots of them) in paginator recently, this workaround was there to prevent datatables to sort data by unexistent pagination option.
For example, you setup your paginator in controller. You go to crud, test it out, you go to controller and change all the numbers. Go back to the page and you will be paginating by an unexistent option.
What I did in the PR, is adding the user selected pagination into a variable in localStorage. That way, if it was the user who changed pagination, we are not going to clear it anymore.
Once again, thank you. I am going to close this issue so we keep any relevant discussion in the PR thread. #3566
Wish you the best,
Pedro


Handle reset identities event

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@addb another use-case we should handle is the async processing of the server response:

a network request was initiated
a reset event was received and processed - resetting identities
the response for the network request (1) was received and processResponseData was executed after reset
Step 3 should not set any identities that were previously reset in step 2.

@emdobrin I wanted to get some feedback on changes I made in Analytics Drop hit being processed if reset is called and Dispatch response only for events triggered after reset


Update as-supplemental-lesson-dynamic-security.md

Fixed code in step 9 to reflect correct names.

@donkolap : Thanks for your contribution! The author, @minewiskan, has been notified to review your proposed change.

@donkolap - Thank you for catching this!
#sign-off


Support Rize ETF/ARK Europe funds (ARKI, ARKG, ARKK, CYBR, CYCL, FOOD, LIFE, LUSA, NFRA & PMNT)

https://europe.ark-funds.com/ark-acquisition
https://europe.ark-funds.com/funds
Bear in mind that ARKG & ARKK are also available on the US site, but looking at their info page you can see they have different ISIN codes in US than they have in Europe:

US00214Q3020 vs. IE000O5M6XO1 for ARKG
US00214Q1040 vs. IE000GA3D489 for ARKK

Looking at the holdings CSVs, these also have slightly different holdings in the US than in Europe:

ARKG - US00214Q3020
ARKG - IE000O5M6XO1
ARKK - US00214Q1040
ARKK - IE000GA3D489

Inception date if also different
All this to say that I'm quite certain that the Europe specific versions are actually different funds, they just use the same base code to show that the fund "rules" are the same I guess 🤔
CC/ @NexVeridian

@NexVeridian I see you already included CYBR, CYCL, FOOD, LIFE, LUSA, NFRA & PMNT, but ARKI can already be included as well
Only ARKG & ARKK have a US counterpart as well


Intellij IDEA hangs on initialization

I've been trying to use this plugin a few times. The first few times the "Clojure LSP startup - LSP: Initializing" step caused the IDE to crash. I updated my version, unfortunately I cannot remember from what, but now It doesn't seem to hang, but the task has now been running for 15 minutes.
If this is normal, you should notify users of this expected wait (so they dont go posting issues). And if it is not normal, please advise.
This is my configuration.
IntelliJ IDEA 2023.1.5 (Community Edition)
Build #IC-231.9392.1, built on July 25, 2023
Runtime version: 17.0.7+10-b829.16 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 6.2.0-32-generic
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 16
Registry:
    debugger.new.tool.window.layout=true
    ide.experimental.ui=true

Non-Bundled Plugins:
    com.github.clojure-lsp (0.12.2)
    IdeaVIM (2.5.1)
    PythonCore (231.9225.4)

Kotlin: 231-1.8.21-IJ9392.1
Current Desktop: i3

I am going to try disabling the plugins one by one and report back if that has any effect.

The culprit was IdeaVim!
My workaround was to disable the plugin, let the first initialization complete, and then reenable IdeaVim. And now they seem to play nice.
I reckon this issue can either be solved programmatically or by informing users of this incompatibility and its workaround.

Additionally, I increased the memory to 8192mb, but that alone had no effect.

that's pretty weird, clojure-lsp should start ins max 1 or 2 mins usually and next startups should be even faster given the cache, but another plugin affects that? pretty odd... will try to reproduce it here, thanks for the report

@hallundbaek I really can't repro that, IdeaVim plugin + clojure-lsp-intellij work great for me when testing on some projects like https://github.com/ericdallo/clojure-sample

Maybe it's in conjunction with PythonCore, I also had like 15 clj files open at the time. And only 2048 mb heap.
I had previously installed Cursive plugin and had that enabled initially, and I disabled it before enabling closure-lsp, and then un-installed it on a subsequent reboot of IDEA.

Yeah, without a clean repro it's hard to know what it is, if you manage to repro in a public repo with specific plugins, LMK


Updates to support xcode 16

Short description
Adujsted project so it builds on Xcode 16
Proposed changes
Updated sentry to fix this: https://github.com/getsentry/sentry-react-native/issues/3883
Updated url launcher to fix this https://github.com/flutter/flutter/issues/152986
Side effects

May break anything related to any library

Testing
smoke test the whole app on iOS and android
check if the app builds on Xcode 16 and 15 for you please

Resolved issues
no issue, but will me merged into / is needed for #1431

@ztefanie
i made some smoke tests and everything except of deeplinking activation works.
Maybe you can merge the recent changes from main (where its working) your working branch
The issue i'm facing here is that the link doesn't work with trailing slash anymore (even on android) and on main it does
npx uri-scheme open berechtigungskarte://bayern.ehrenamtskarte.app/activation/code#CkkKHwoPQW5kcmVhcyBGaXNjaGVyEPGkARoICgIICSICCAASEDGjkYQBPaU2Dd6mI87eXtAaFHNsrKCgpGu20z2c06yyG7bU12ap/ --ios

I also tested deeplinking, works as expected. Will merge this


Deserializing a Dictionary<string, string> throws JsonReaderException

Yeah I forgot that I have to pass in a string to DeserializeObject<> and not a file name.
Source/destination types
Dictionary<string, string>

Source/destination JSON
{
  "FIRSTNAME": "First name"
}

Expected behavior
I expected to get an instance of Dictionary<string, string>.
Actual behavior
I got a JsonReaderException:
"Error parsing boolean value. Path '', line 1, position 2."
Steps to reproduce
JsonConvert.DeserializeObject<Dictionary<string, string>>("translations.json");

ok
thanks


Cement/concrete are not recycled

Cement and concrete never get recycled, in any scenario.
Current parameter 4_PY_EoL_RecoveryRate (V2.4) modules the rate of waste recovery from EoL products by multiplication with F_8_9 (the input of waste management industries). Some products actually have some concrete content recycled as  "construction waste, concrete, bricks, tiles, ceramics", but F_8_9 always have 0 flows for concrete, because concrete is split between cement and aggregates in the code. Since cement and aggregates are not waste of any product in 4_PY_EoL_RecoveryRate_2.4, they are also not recycled.
Even if the flows above were consistent, all the waste has to be remelted in process 12 before being available as secondary material, and there is no reference in 4_PY_MaterialProductionRemelting_V2.3 to any mineral. Despite the #"emelting" name might imply that only metals could become available as secondary material, you can find also wood and plastics as remelted products. I wonder if the parameter is used as Reclycing, and not Remelting.
Currently, only way to give second life to minerals is to activate the Re-Use ME strategy, but there is no baseline recycling.

Hi @CarrerF , we do have a small potential for the re-use of concrete, typically done by reusing concrete slabs.
Concrete reycling can be done, but only aggregates can be recoverd, and they need new cement as binder.
The current state of the knowledge is that recovering aggregates from concrete is a resource savings strategy, as it can reduce the amoung of sand and gravel needed, but not an emissoins reductoin strategy, see here: https://doi.org/10.1016/j.resconrec.2021.105767
Currently, it is implicitly assumed that all recovered crushed concrete either stays on the side as filling or goes into infrastructure construction. So more of an open system. As we move towards also including infrastructure in RECC v2.6, we will have to rethink this issue and add a scenario for recovering aggregates and send them to different end-use sectors.


ENH Convert bark.h into true C header

WDYT @Green-Sky ?

also, (dark knowledge), you can request reviews :)


remove meta tag view port that comes by default

Hi  all, how can I remove
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">

that comes with django-bootstrap4 template by default

If you don't want this viewport, you should use your own base template. This package does not depend on the default template, it is just here for convenience.


Difference with async-local-storage

Can someone explain what is the difference between this library compared to https://github.com/vicanso/async-local-storage ? That library seems super simple.
I checked other solutions as we are having issues with destroying the namespace.
Is destroying the namespace necessary on every request?
Here is our middleware:
return async function(ctx, next) {
      const namespace = continuationLocalStorage.createNamespace('session');

      try{
        await new Promise(namespace.bind(function(resolve, reject) {
          namespace.set('request_id', ctx.request.header['x-request-id']);

          next().then(resolve).catch(reject);
        }));
      } catch(error) {
        throw error;
      } finally {
        continuationLocalStorage.destroyNamespace('session');
      }
    };

Basically the same.  this module supports node 'Async Hooks' api compared with node-continuation-local-storage, so there is no need to wrap every asynchronous function like node-continuation-local-storage does.

At first glance and without comparing in detail the async_hook event state management of async-local-storage, I assume for the latest node 8 versions, it and cls-hooked would behave the same.
One clear difference is cls-hooked provides backward compatibility with node 6 & 7 using AsyncWrap when running in those versions. It also provides the same interface as node-continuation-local-storage to make it easier for devs to try.
As for destroying namespaces, its not required but I guess it depends how you're using it. I'm not sure what framework your middleware example is from, but usually namespaces are created sparingly, such as a "sessions" namespace, while contexts are created on that namespace for things like requests and sessions.
Btw, I love the simple and elegant code of async-local-storage... it makes me want to rewrite cls-hooked... when days start to have 36 hours in them...

@Jeff-Lewis It is a Koa middleware, and wanted to make request id available in subsequent calls.
How can i refactor it to use one namespace, but the context remains?

Take a look at these for examples:
https://github.com/othiym23/cls-middleware
https://github.com/madbence/koa-continuation-local-storage
https://github.com/clakech/cls-hooked-sample
Just create the namespace outside of the middleware handler function. I see most apps create the namespace once when initializing the server app.

@blacksonic Closing for now. Let me know if you have any issues.


Add Youtube and Vimeo embedded elemens

+1

:+1: looks good! :tv:

+1
Are these the only embed types supported by amp?

@dsager Nope, there's support for all the types we use, just wanted a not so big PR.

+1 we also have to find a way how to remember which amp extensions are used, for the required js includes

@christofdamian yes, I thought about adding a test for the parser, to check all which are included


Update uuid to 4.0.0

:wave: I don't have the rights to modify your PR, so I've finished it on my own locally f928dc88cba25e5e6be05cfcc1a37a07b11dafc3. I'll therefore close this :)

@rrousselGit Can you please release the uuid change when you get a chance?


Playwright error: Unknown engine "internal:testid"

We use a custom test data attribute during testing, which we configure using Playwright's inbuilt testIdAttribute key in the config file. See https://playwright.dev/docs/api/class-framelocator#frame-locator-get-by-test-id
When I run my Playwright tests using the latest moon version (1.28.1 as of this date), the tests work fine. But when I try using an older Moon Playwright image (1.27.1, 1.26.1, 1.25.1 etc.) I receive the following error:
locator.click: Unknown engine "internal:testid" while parsing selector internal:testid=[data-object-name="FilterRadioButton-application-option-0"s]
Using an older Moon Playwright image is important for us to test compatibility with older browser versions.
I start the tests on Moon using the following endpoint
wsEndpoint: `wss://moon.infra.bitgrip.berlin/playwright/${BROWSER}/playwright-${PLAYWRIGHT_IMAGE_MOON}?headless=false`,
I could reproduce this issue with both Chrome and Firefox as browsers.

@anupamck at least on Playwright documentation page they note that this was implemented in Playwright 1.27.

@vania-pooh - Thank you. I guess it's mostly an issue with the Playwright version then. You can close this ticket.
Btw, the moon Playwright images seem a little outdated? The latest image, 1.28.1, dates back to Nov 2022 release from Playwright.

@anupamck in fact we automated build for Playwright images and all recent versions are available. We forget to update HTML document.  https://quay.io/repository/playwright/chromium?tab=tags

Great job! Thank you. Perhaps the update to the HTML doc should be automated too :D


Show function-loading errors

Atm chatcraft just pretends nothing is wrong and ignores the url in the response..but in console can see that stuff went wrong.

So this is when you use a fn-url and it fails to parse what it gets, due to some bug in the code?

yes

I guess I could store the err on the ChatCraftFunction as an extra piece of data (i.e., put it on object, not store in db), and if it exists at runtime, show it in the UI somehow?
Have to think about when and where to show that info.

i think it should be same as openai api failing. currently we flash a red error message for those. Ideally we'd have a more permanent error


Feature Request: array.sortBy

I know this isn't part of the Ember.computed namespace, but sortBy would be an extremely useful addition. I could try my hand at a PR. :slightly_smiling_face:

I should check the docs first before making a wish list. Haha. :rofl:


Fix README: client-go issues should be reported at k8s.io/client-go now

Fix https://github.com/kubernetes/client-go/issues/84.

This change is

LGTM, thanks


Setup my own LibreTranslate instance

Hi,
With Google and Bing, there are a lot of limitations and it's too slow. To avoid network issues with Libre Translate, is there an option to setup the IP/port of my own Libre Translate?
Best Regards,
LeChuck

Hi @Monkey-Island ,
There is not an option for doing it for not, but that would be a great improvement for a package.
I would like to implement it soon.
Thank you.


Allow ApolloClient to be Proxied

https://github.com/spring-projects/spring-boot/issues/14879#issuecomment-430777067
Can ApolloClient please not be final, or maybe have a single interface that composes the 4 factory interfaces? Right now my client will be fine with just mutation, but if I wanted to do read/write in the same interface I'd have to inject 2 or more of them. Interface method is preferred, imho.

And final makes it hard to mock via mockito. Using mock-maker-inline as work around.

you can make a single interface by extending the 4 interfaces. I'm not sure if conflating interfaces together for  general use is what we want.

yes, but I can't make your class use my interface, and it's final, thus I can't extend it... so what exactly do you expect? me to reimplement a full delegate?

Misunderstood the issue sorry. Reopening

Closing out of date. You can always mock raw http server request if needed with https://github.com/square/okhttp/tree/master/mockwebserver


Store Simulation Configuration with Simulation Data

dconner
https://jira.jpl.nasa.gov/browse/AERIEQS-419
Moderate

#725 does copy the simulation configuration into a simulation dataset prior to running simulation - which I think satisfies this ask. There may be utility in exploring ways to display this information in the UI, but that should be a follow up issue.


Make 2.18 the default branch

I don't have the access to do this.
Also see https://github.com/FasterXML/jackson-jakarta-rs-providers/pull/26#issuecomment-2039229847

Ah. Thought I did that, thank you for ping.


Separate user and group management

Supersedes #321

Added sidemenu items for users and groups
removed 'back to users' button from groups toolbar
removed 'groups' button from users toolbar

Just waiting for @petehalverson to help me out with an issue with our testing setup first and then I'll be able to review this

Awesome, works great @joakimbo! Thanks for fixing that @petehalverson, makes my life so much easier 😄


Add draft for binary switch based on extended CAPSSession

Extends CAPSSession with SPIP API via mixin trait. The trait and necessary adapters are bundled within a single package. This allows a seamless switch between SPIP and OKAPI API in a single graph app by switching the session implementation. See GraphApp for an example.
Pro:

no additional module
seamless binary switch between CAPS and SPIP
CAPS spark-cypher can be renamed to morpheus-spark-cypher or morpheus

Cons:

as with v1, it feels a bit weird that similar named methods return equally named results but with different semantics, but I guess that's what we just need to deal with

Needs apache/spark#24297 installed locally to compile and run.

I think I like this one better than the alternative, but I'm not sure I understand all of the implications. Very nice examples though!


[WIP] Fix #2479 - Check migration at server start

Describe your changes :

This PR fixes https://github.com/open-metadata/OpenMetadata/issues/2479.
The idea behind this is being able to help users during the upgrade of OpenMetadata by checking the flyway versions on MySQL before starting the server.
As we are controlling flyway with the TableInitializer, here we can reuse its validate command and check the exit status to either show the message that users need to migrate, or we can continue normally.
One question I would have here is that with this behavior we will also be giving this notification to users that are installing OpenMetadata for the first time, as openmetadata.sh does not run migrate-all. @harshach are we OK with this behavior? If so, should we update this doc to show how to run the migration as well before starting the service?
Thanks!!

Type of change :

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@pmbrull this could work, was expecting this to be part of CatalogApplication.java file. Users could use some other startup mechanism instead of using ./bin/catalog.sh then this approach might not work. But should be ok.
For users who are just installing for the first time then they won't be having DATABASE_CHANGE_LOG table as thats created by flyway. You'll be running this tool before invoking the migration so that case should work

@pmbrull this could work, was expecting this to be part of CatalogApplication.java file. Users could use some other startup mechanism instead of using ./bin/catalog.sh then this approach might not work. But should be ok. For users who are just installing for the first time then they won't be having DATABASE_CHANGE_LOG table as thats created by flyway. You'll be running this tool before invoking the migration so that case should work

Hi @harshach, makes sense. I came up with this as I was following the docs myself. I can add the check on CatalogApplication.java as well

Hi @harshach, added the validation in the Catalog class as well. Let me know if this is the approach you had in mind 🙏  Thanks!


Name of the Vendor

If decided to expand the TransmitterEquipment specification to cover also the chassis, it would be proposed to fill the attributes of the actualEquipment of the chassis alike the ones at the ODU (see second table in §31 ActualEquipment in the current TransmitterEquipment specification).

Additional: The vendor should be identified by its IANA Private Enterprise Number: https://www.iana.org/assignments/enterprise-numbers/enterprise-numbers
Reason: there is a working registration available and the number are maintained. It avoids for machines the impelementation for abstracting (for example) "highstreet", "highstreet technologies", "highstreet technologies GmbH".

Unfortunately, there are different definitions of vendor numbers or vendor organizationally unique identifiers (OUI). E.g. SFPs use the IEEE Company Identifier.
Furthermore, the IDs are often not unique. That is, some companies have more than 1 identifier.

The uniqueness is the beauty of IANA PEN. There is a registration and maintenance process involved that' why I bring IANA PEN into the game.

Proposal to the 5G-xhaul call on 12th of October 2022:
Change on TransmitterEquipment in the section ActualEquipment of chapter 9 Information
from
For ODUs, this shall include at least the following information
to
For ODUs and Chassis, this shall include at least the following information

Starting from TransmitterEquipment_i.j.k-tsp.231213.1100+spec.1 changed as decided.

Has been updated and published in TransmitterEquipment_1.0.0-tsp.200715.1225+spec.1.doc


Add builder classes, make them the only parameter to macro methods

This is still a WIP, but I wanted to get it out there for an initial review.

Static analysis now is clean
JsonCodable works, haven't tried the other ones
I still need to get all the tests passing (I am sure many fail, haven't actually checked), will address this tomorrow
All builder classes still have the same query API, which needs to be evaluated and hopefully restricted per phase in some way.
The QueryClass macro was actually breaking the rules. It is no longer allowed to, so I changed a bit how it works.

@davidmorgan I made some substantive changes to fix some of the tests that you should look at - lazy merged maps are now considered equal if their left/right maps are equal, and _keyof and getParents also were modified for lazy merged maps.

To fix the remaining tests I had to change the expectations, the qualifiedNameOf function now technically works for cycles and re-used nodes (although the result could be surprising especially in the re-used nodes case).
I am not sure if this is a bad thing or a good thing... but these nodes shouldn't be re-used anyways.

Thanks! Still LGTM.
Re: lists and cycles, both seem fine for now.


Avoid Symbol to disappear on zoom out

Hello,
I have many different symbols on my maps. And they are all important. The problem is when I zoom out and Symbol get closer, some of them disappear as you can see below

I tried to play with zIndex but it does not change anything. I did not have that issue when I was using Markers but now they are deprecated so I moved to Symbols.
How to let my Symbols always displayed on the map ?
Thanks

I found the answer while posting this issue...
           mSymbolManager.setIconAllowOverlap(true);


[Snyk] Fix for 4 vulnerabilities

Snyk has created this PR to fix 4 vulnerabilities in the maven dependencies of this project.
Snyk changed the following file(s):

master/pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230368
  53
org.springframework:spring-webmvc:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230364
  40
org.springframework.data:spring-data-jpa:  2.5.3 -> 3.2.11  org.springframework.security:spring-security-core:  5.4.7 -> 6.2.7  org.springframework:spring-webmvc:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230365
  40
org.springframework.data:spring-data-commons:  2.5.3 -> 3.2.11  org.springframework.data:spring-data-jpa:  2.5.3 -> 3.2.11  org.springframework.security:spring-security-core:  5.4.7 -> 6.2.7  org.springframework:spring-aop:  5.3.18 -> 6.1.14  org.springframework:spring-web:  5.3.18 -> 6.1.14  org.springframework:spring-webmvc:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230366
  40
org.springframework:spring-web:  5.3.18 -> 6.1.14  org.springframework:spring-webmvc:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

:warning: Snyk checks are incomplete.
:warning: security/snyk check encountered an error. (View Details)
:warning: license/snyk check encountered an error. (View Details)


fix: build with glibc 2.28 for x86_64_gnu

An attempt to fix https://github.com/pact-foundation/pact-js-core/issues/502 using cross.

I think the failure before was due to cross re-using deps in the target folder
the macos failure, is indeed due to binary content matching, and is due to macos-latest switching to arm based runners.
we can revert back to original behaviour by switching the macos-latest tag to macos-12

That makes sense! I saw your PR which is all green. I will close this one.


Feature/initial data model

Cleaned up the data and merged lyrics, genres and lang information into the tracks dataset - the english tracks are contained in tracks_genres_lyrics_en.csv.zip - See notebooks/data_cleanup.ipynb for details.
The modeling work is detailed in notebooks/data_modeling.ipynb.
The app/data_model_usage.py file contains sample/test code using the data_model.find_songs module.


SQLNexus crashes on WIndows 10

SQL Nexus crashes with a .Net runtime error on WIndows 10. Using .Net 4.8 patched with KB5034468

@Imprecator can you provide info from the Detalles tab - possibly a call stack?
Also, which version of SQL Nexus are you using?
Finally, how can we reproduce this issue? Do you have some repro steps

Closing. Please reopen if you have a way for us to reproduce


set bucket acl error

设置存储桶acl报错

The minio gateway native does not support the bucket acl feature.
https://github.com/juicedata/minio/blob/c4373ef290344fd01981864cb9feca9410cf5b21/cmd/gateway-unsupported.go#L124

We plan to implement bucket ACLs when we provide POSIX ACL.

done by #4030


Refactor helm installs

We recently landed a change to bump the operator-sdk version, which
resulted in breaking apart the roles and cluster roles for the
compliance operator into separate files.
This commit updates the Helm installation documentation and links to
work with the new location after the operator-sdk update.

/lgtm

/label docs-approved

/px-approved

@rhmdnd I am not sure with the test procedure. Correct me if I am wrong.
Seems installation succeed. But the suite will return error. Details seen from: https://privatebin.corp.redhat.com/?21c8e41fceaeb256#EMVPWyjUcwUQaFfrYSvoBiSuMeJMtsa9C8trJtaQ68pp

@rhmdnd I am not sure about the test procedure. Correct me if I am wrong.
Seems installation succeed. But the suite will return error. Details seen from: https://privatebin.corp.redhat.com/?21c8e41fceaeb256#EMVPWyjUcwUQaFfrYSvoBiSuMeJMtsa9C8trJtaQ68pp

I believe the issues you're hitting is unrelated and it's currently being addressed in the CaC/content repository.
https://github.com/ComplianceAsCode/content/pull/9435 contains some context from @jhrozek. A separate fix just landed https://github.com/ComplianceAsCode/content/pull/9436

/label qe-approved

/label px-approved
/label docs-approved

/label qe-approved


Gophish logs "Email open" and "Clicked link" immediately after the campaign is started, can Goreport solve this?

Hi, Thanks for the great tool you have made! But I have a problem where Gophish logs "Email open" and "Clicked link" immediately after the campaign is sent, is there an option or some config configuration that removes these false positives?
(For example remove/ignore the first 3 minutes or remove/ignore any addresses except Norway)
Under you can see an example where someone actually opened and clicked the link but it registers multiple times that shows in the high level results shown in the bottom. I started the campaign 2022-05-23 T10:56:46

False positive with real user later

Only false positive

High Level Results
Total Targets: 352
The following totals indicate how many events of each type Gophish recorded:
Total Open Events: 1644
Total Click Events: 780
Total Report Events: 0
Total Submitted Data Events: 24
The following totals indicate how many targets participated in each event type:
Individuals Who Opened: 351
Individuals Who Clicked: 351
Individuals Who Reported: 0
Individuals Who Submitted: 23

I believe this is more of a GoPhish issue rather than a GoReport issue


Improve the README

Improve the readme

I will do this

I would like to claim this. This would be my first open source pull request.


import { ACLGuard } from '@delon/acl';

When used the "import { ACLGuard } from '@delon/acl';", the prompt error : has no exported member 'ACLGuard'

i mark sure that have got export ACLGuard (index.ts), pls trying reinstall package again.

What is the instruction to download the package?

npm i @delon/acl

I tried reinstall package again, but the file was not updated

# remove node_moudes & package-lock.json
npm i --save-dev @delon/acl
npm i

Thank you very much, I found that its version is relatively low, upgraded to 0.6.6 later is correct


Remove version: Alibaba.aDrive version 4.3.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://cdn.aliyundrive.net/downloads/apps/desktop/update/4.3.0/win32/ia32/aDrive-4.3.0.exe
Status Code: 200
(Automated message - build 797)


Lost

so ive cloned this and installed rust, but im not sure how im supposed to compile? reading/searching was getting me nowhere, so i started poking around and came up with switching into the project directory and running cargo build. but it eventually crashes trying to compile crypto, complaining about "no method named write_hex" and aborting due to a previous error. am i even close?

Hey, thanks for the heads up.
First of all, sorry for the bad documentation about building the project, I added a new section to README.md describing how to build.
Secondly, the error you mentioned came from a breaking change in a dependency and has been fixed in 55aa95a41cf2462df2a0cda255b29e7dbd5a7b37.
You might want to try git pull and then cargo build --release.
You should know, I no longer use this solution myself, since acme-dns is the superior solution, at least for my use case. You are still welcome to use this tool. There are no known problems aside the ones mentioned under Security Considerations

Closing due to inactivity


[ANDROID] browserInfo doesn't return from DropIn

Describe the bug
After upgrading the library from v1.2.0 to v2.0.0 BETA2, Android version is not working properly.
Previously, the data parameter of the onSubmit function had browserInfo and the data.amount.value was an integer. Now, there is no browserInfo data, and data.amount.value comes as string. DropIn is dismissed after the first step.
To Reproduce
Steps to reproduce the behavior:

on Android, fill the credit card details on DropIn
Click on "Confirm preautorization" button (amount.value = 0)
See the DropIn is dismissed

Expected behavior
The 3DS auth screen should appear on the screen after completing the credit card details
Smartphone (please complete the following information):

Device: Pixel 4
OS: API 34
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Should we populate browserInfo prop manually on Android?

Fix for data.amount.value released on 2.0.0-rc.1

I am going to close this ticket, since remaining part is a duplicate of https://github.com/Adyen/adyen-react-native/issues/164


Native MessageBox API

It would be great to have a native Message box API. Win32 on Windows and GTK on Linux / OSX (or Cocoa). Because Cocoa would need to probably be used on iOS anyway, it makes sense for OSX.

I think we should take inspiration from UWP and implement MessageDialog and ContentDialog

+1 for @nc4rrillo

will require #624 to also be completed.

I know it's a workaround but what about this: https://habr.com/en/post/454386/

Hi @danwalmsley ,
I see that https://github.com/AvaloniaUI/Avalonia/issues/624 was closed... Is this workable now?
Thanks, Kevin

Definitely possible now

Given that this issue is about native message boxes, I've done a bit of research into the various native message box APIs in the platforms we support:
Native Message Boxes
Win32
MessageBox
MessageBox shows a simple message box. It has the following elements:

A title
A text message
A fixed choice of buttons (localized):

OK
OK, Cancel
Yes, No
Yes, No, Cancel
Abort, Retry, Ignore
Cancel, Try Again, Continue
An optional Help button an be added to each of the above

An icon:

Exclamation,
Warning
Information/Asterisk
Question
Stop/Error/Hand

This is the API used by WPF's MessageBox.
TaskDialog
TaskDialog shows a "vista"-style messagebox. It has the following elements:

A title
A message ("main instruction")d
Content (additional text that appears below the main instruction, in a smaller font)
Any combination of buttons (localized):

OK
Yes
No
Cancel
Retry
Close

An icon:

Error
Information
Shield
Warning

There is a proposal to add an API for this to WinForms: https://github.com/dotnet/winforms/issues/146
TaskDialogIndirect
TaskDialogIndirect shows an advanced "vista"-style messagebox. It has the elements from TaskDialog plus:

A set of buttons that can be:

Radio buttons
Command links

A verification checkbox
An expander to show additional information
An arbirary footer icon
A footer message
Hyperlinks in the content/expander/footer
A progress bar

OSX
NSAlert shows a message box. It has the following elements:

A message
Informative text
A type

Critical
Informational/Warning

A custom icon
An optional help button
Arbitrary buttons
A supression checkbox
An accessory view containing arbitrary content

Localization of buttons?
X11
On X11 message boxes are provided by the toolkit so we would have to implement our own managed message boxes, but for context here's what GTK and QT's message box APIs provide.
GTK
gtk_message_dialog_new is simply a convenince function for creating simple dialogs. It can be modified arbirarily, but the base API has the following elements:

A title
A message
Secondary text
An icon

Information
Warning
Question
Error
Other (No icon)

Buttons:

OK (discouraged)
Close
Cancel
Yes, No (discouraged)
OK, Cancel (discouraged)

QT
QMessageBox shows a message box. It has the following elements:

Primary text (plain or rich)
Informative text (plain or rich)
Optional detailed text
An icon

Question
Information
Warning
Critical
A custom bitmap

Buttons:

OK
Open
Save
Cancel
Close
Discard
Apply
Reset
RestoreDefaults
Help
Save All
Yes
Yes to All
No
No to All
Abort
Retry
Ignore

Two APIs for using QMessageBox are provided, the property-based API, and the static functions. Calling one of the static functions is the simpler approach, but it is less flexible than using the property-based API, and the result is less informative. Using the property-based API is recommended.
Android
AlertDialog shows a message box. It has the following elements:

Title
Content area (can be text, a list or other custom layout)
Up to 3 buttons with arbitrary text

iOS
UIAlertController shows a message box. It has the following elements:

A title
A message
A style

Action sheet (lays out actions vertically)
Alert

Arbitrary actions (buttons)

Given that this issue is about native message boxes, I've done a bit of research into the various native message box APIs in the platforms we support:
Native Message Boxes
Win32
MessageBox
MessageBox shows a simple message box. It has the following elements:

A title
A text message
A fixed choice of buttons (localized):

OK
OK, Cancel
Yes, No
Yes, No, Cancel
Abort, Retry, Ignore
Cancel, Try Again, Continue
An optional Help button an be added to each of the above

An icon:

Exclamation,
Warning
Information/Asterisk
Question
Stop/Error/Hand

This is the API used by WPF's MessageBox.
TaskDialog
TaskDialog shows a "vista"-style messagebox. It has the following elements:

A title
A message ("main instruction")d
Content (additional text that appears below the main instruction, in a smaller font)
Any combination of buttons (localized):

OK
Yes
No
Cancel
Retry
Close

An icon:

Error
Information
Shield
Warning

There is a proposal to add an API for this to WinForms: https://github.com/dotnet/winforms/issues/146
TaskDialogIndirect
TaskDialogIndirect shows an advanced "vista"-style messagebox. It has the elements from TaskDialog plus:

A set of buttons that can be:

Radio buttons
Command links

A verification checkbox
An expander to show additional information
An arbirary footer icon
A footer message
Hyperlinks in the content/expander/footer
A progress bar

OSX
NSAlert shows a message box. It has the following elements:

A message
Informative text
A type

Critical
Informational/Warning

A custom icon
An optional help button
Arbitrary buttons
A supression checkbox
An accessory view containing arbitrary content

Localization of buttons?
X11
On X11 message boxes are provided by the toolkit so we would have to implement our own managed message boxes, but for context here's what GTK and QT's message box APIs provide.
GTK
gtk_message_dialog_new is simply a convenince function for creating simple dialogs. It can be modified arbirarily, but the base API has the following elements:

A title
A message
Secondary text
An icon

Information
Warning
Question
Error
Other (No icon)

Buttons:

OK (discouraged)
Close
Cancel
Yes, No (discouraged)
OK, Cancel (discouraged)

QT
QMessageBox shows a message box. It has the following elements:

Primary text (plain or rich)
Informative text (plain or rich)
Optional detailed text
An icon

Question
Information
Warning
Critical
A custom bitmap

Buttons:

OK
Open
Save
Cancel
Close
Discard
Apply
Reset
RestoreDefaults
Help
Save All
Yes
Yes to All
No
No to All
Abort
Retry
Ignore

Two APIs for using QMessageBox are provided, the property-based API, and the static functions. Calling one of the static functions is the simpler approach, but it is less flexible than using the property-based API, and the result is less informative. Using the property-based API is recommended.
Android
AlertDialog shows a message box. It has the following elements:

Title
Content area (can be text, a list or other custom layout)
Up to 3 buttons with arbitrary text

iOS
UIAlertController shows a message box. It has the following elements:

A title
A message
A style

Action sheet (lays out actions vertically)
Alert

Arbitrary actions (buttons)

If we were to implement a native MessageBox API, I can see 3 choices:

Implement a common subset. This would appear to be:

Title (implemented on all platforms except OSX - could just not display it there)
Message in plain text
Win32 MessageBox button choices if we use that API, or a arbitrary buttons if we use TaskDialog

Does TaskDialog look out of place in Win10? Does that matter?

"Content" text could be displayed using the relevant feature on platforms that support it, and appended to the message on platforms that don't?

Have some sort of capabilities query

Would somewhat overcomplicate just showing a message box in a cross-platform manner

Just provide the bare minimum somewhat like JS:

alert displays a message and an OK button
confirm displays a mesage and OK, Cancel buttons

If we were to implement a native MessageBox API, I can see 3 choices:

Implement a common subset. This would appear to be:

Title (implemented on all platforms except OSX - could just not display it there)
Message in plain text
Win32 MessageBox button choices if we use that API, or a arbitrary buttons if we use TaskDialog

Does TaskDialog look out of place in Win10? Does that matter?

"Content" text could be displayed using the relevant feature on platforms that support it, and appended to the message on platforms that don't?

Have some sort of capabilities query

Would somewhat overcomplicate just showing a message box in a cross-platform manner

Just provide the bare minimum somewhat like JS:

alert displays a message and an OK button
confirm displays a mesage and OK, Cancel buttons

Note that we need some kind of localized text database anyway since we have default context menus textboxes.

Note that we need some kind of localized text database anyway since we have default context menus textboxes.

Discussion about a new TaskDialog API in WinUI: https://github.com/microsoft/microsoft-ui-xaml/issues/2313 - though this is only really a "native" MessageBox in the sense that MS would be shipping it.

Discussion about a new TaskDialog API in WinUI: https://github.com/microsoft/microsoft-ui-xaml/issues/2313 - though this is only really a "native" MessageBox in the sense that MS would be shipping it.

In terms of UI thread lock, I just worked around this issue in WPF with this simple function.
Can now run sync UI methods in a fully asynchronous way and await!
/// <summary>
/// Runs a synchronous action asynchronously on the UI thread.
/// </summary>
/// <param name="window">Any window to get the dispatcher from.</param>
/// <param name="action">The action to run asynchronously.</param>
/// <typeparam name="T">The return type of the action.</typeparam>
/// <returns>The result of the action.</returns>
public static Task<T> RunUiAsync<T>(this Window window, Func<T> action)
{
    if (window == null) throw new ArgumentNullException(nameof(window));
    TaskCompletionSource<T> completion = new();
    window.Dispatcher.BeginInvoke(new Action(() => completion.SetResult(action())));
    return completion.Task;
}

In the Avalonia port of MvvmDialogs, I need to have a common MessageBox API to share between WPF, Avalonia, UWP, WinUI3, Blazor or anything else (because the ViewModel shouldn't need to know which UI it's running on)
What we first need is a basic MessageBox with standard features which should be simple enough to implement.
Later on, you could add an ExtendedMessageBox in Avalonia with extra features, which will take more time and it will be harder to come with a stable cross-platform API for.
ExtendedMessageBox extension could later be added to MvvmDialogs with support for WPF/Ookii, and for Avalonia if that's added. But it's really difficult for these extended feature settings to be platform-agnostic.
My vote is that extended features do not need to be part of the core library but can be added through a separate project, but basic MessageBox features DO need to be part of the core library.

NEED this

NEED this

@Anequit, while we wait for integration of this feature, have you checked out MessageBox.Avalonia?

NEED this

@Anequit, while we wait for integration of this feature, have you checked out MessageBox.Avalonia?

Yes I have, but I'm not a huge fan of it personally. I think for the time being I'll opt for an alert popup instead of a message box

have you checked out MessageBox.Avalonia?

I've just gave it a spin, and it's not resizing vertically for a long texts:
https://github.com/AvaloniaCommunity/MessageBox.Avalonia/issues/104
It also has no API docs for MessageBoxStandardParams etc., looks like something unpolished / buggy to me =\
Would be happy to see out-of-the-box platform-agnostic solution in Avalonia.

Yes it has a lot of unpolished rough edges. It had no option for setting a default button; they added it but the implementation is badly designed and looks more like a hack.

I'm having a nice laugh finding out a few hours into coding that I basically have to write my own custom message box to give the user feedback. Incredible.

As an option, with Prism.Avalonia you make use of the DialogService which provides the ability to make it modal and non-modal with ease. It's pretty much 60 seconds of copy/paste as per the end-to-end sample

or using MvvmDialogs for Avalonia

Simple MessageBox should be higher on the implementation priority list. Cross Platform UI framework without a MessageBox!!


Update netty-incubator-transport-native-io_uring to 0.0.10.Final

Updates io.netty.incubator:netty-incubator-transport-native-io_uring from 0.0.8.Final to 0.0.10.Final.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.netty.incubator", artifactId = "netty-incubator-transport-native-io_uring" } ]

labels: library-update

Superseded by #223.


Use "ON CONFLICT UPDATE" on matrix & trend element precalculation result update in DB

Currently this uses an auto incrementing primary key and simply, deletes all staled rows then inserts all fresh rows. It just looks nasty because the sequence will not reset on cache reset and it's just eventually gonna go wild.
Composited Primary Key or simply ON CONFLICT UPDATE in a transaction shall solve this easily.

![7D6C0A47556DD6C449F37F907265A348](https://user-images.githubusercontent.com/25188552/172468415-1190079d-eee6-46e8-9d18-ef00b401b20d.jpg)


[ktest] negative tests

Currently, ktest compares outputs of krun only, not of kompile. If kompile fails, the whole test fails. Also, krun's output is compared only if krun succeeded. So, it is not possible to do negative-tests using ktest.
See also #1945 #1982 #1987

@daejunpark, added you so we have someone tracking this issue.


Missing Security policy

Hello 👋 ,
Are there plans to create a Security Policy?
Currently that tab appears empty: https://github.com/agilepathway/label-checker/security
Thanks in advance

hi @w0rmr1d3r, thanks for raising this.   It's a good suggestion indeed, I'll add this tomorrow. The policy will be to have private vulnerability reporting, I'll reply to this thread when it's setup.  Thanks, John

Really great! thank you for adding it in such a short time! 🔝


Cant install who is on my wifi because it lacks "WMI" but I get an error when trying to install wmi!

import who_is_on_my_wifi
Traceback (most recent call last):
File "", line 1, in
File "C:\Program Files\JetBrains\PyCharm Community Edition 2019.3.3\plugins\python-ce\helpers\pydev_pydev_bundle\pydev_import_hook.py", line 21, in do_import
module = self._system_import(name, *args, **kwargs)
File "C:\Users\Frederik\PycharmProjects\Check_Network\venv\lib\site-packages\who_is_on_my_wifi.py", line 13, in
import wmi
File "C:\Program Files\JetBrains\PyCharm Community Edition 2019.3.3\plugins\python-ce\helpers\pydev_pydev_bundle\pydev_import_hook.py", line 21, in do_import
module = self._system_import(name, *args, **kwargs)
ModuleNotFoundError: No module named 'wmi'

Then I try and install WMI
pip install wmi
Collecting wmi
Using cached https://files.pythonhosted.org/packages/ee/b9/a80d1ed4d115dac8e2ac08d16af046a77ab58e3d186e22395bf2add24090/WMI-1.5.1-py2.py3-none-any.whl
Could not find a version that satisfies the requirement pywin32 (from wmi) (from versions: )
No matching distribution found for pywin32 (from wmi)

I am using Python 3.8 and Windows 10 in Pycharm

Hello,
this error means that you can't install pywin32 (just as my library depends on wmi, wmi depends on pywin32)
Please try:
pip install pywin32

If another error appears, let me know.
Here are links to pywin32 issues and stack overflow where you could find a solution to your problem.
I hope it helped at least a little.


Make build system pack-compatible.

Updated and simplified the build system to ensure pack will install support files. Makefile is based off stefan-hoeck/idris2-linux.

I will have to make a custom build makefile that includes cloning this tls repo, and build and copy so files into apps build folder.
So this is very much needed and wanted.


support ATTACH statement for Iceberg catalog (similar to UC_CATALOG)

The uc_catalog extension supports the following:
INSTALL uc_catalog from core_nightly;
INSTALL delta from core;
LOAD delta;
LOAD uc_catalog;
CREATE SECRET (
	TYPE UC,
	TOKEN '${UC_TOKEN}',
	ENDPOINT '${UC_ENDPOINT}',
	AWS_REGION '${UC_AWS_REGION}'
)
ATTACH 'test_catalog' AS test_catalog (TYPE UC_CATALOG)
SHOW ALL TABLES;
SELECT * FROM test_catalog.test_schema.test_table;

This would be extremely helpful to have for Iceberg.

hey @mike-luabase yea supporting more catalogs in this way is definitely something we are looking into, the iceberg catalog seems like a sensible candidate for sure


FPGA Implementation on TensorFlow

Hi all,
Recently, I read some papers about implementing Intel Xeon Phi on Tensorflow. Therefore, I want to make TensorFlow support FPGA board. But there is no information in Google and other forum/community.
For my understanding, we should implement the FPGA supporting in a class of TensorFlow which distributes tasks to different devices (like GPU/CPU). Is possible that implementing the FPGA supporting in that class? Or is there any good suggestions to make the progress to support FPGA on TensorFlow?
I am looking forward any help or advice!
Thanks,
Kevin

How about implement an ops of Tensorflow using FPGA?
https://www.tensorflow.org/extend/adding_an_op

Related: https://github.com/tensorflow/tensorflow/issues/8820

Hi @jiawen11 !
I do see an implementation of Tensorflow on FPGA board (1.8 version though) through LeFlow. will connect with team on it though for feasibility of 2.x implementation.
Closing this issue  as duplicate to #8820  (Please comment in #8820)
Feel free to raise a PR from your end.
Thank you!


Update WASI-tutorial.md

Extend doc on how to compile

I'll change it to macos specific compilation. Would have helped me, so I think others too :)

Anything still to update?

Looks good, thanks!


Latest beta can't write to config file on windows

What is your rclone version (eg output from rclone -V)

~ rclone.exe version
rclone v1.36-151-g48508cb5

Which OS you are using and how many bits (eg Windows 7, 64 bit)

Windows 10 64 bit

Which cloud storage system are you using? (eg Google Drive)

All

The command you were trying to run (eg rclone copy /tmp remote:tmp)

All

A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)

~ rclone.exe ls gdrive:
2017/06/01 HH:mm:ss Failed to create temp file for new config: open C:\Users\*redacted*\AppData\Local\Temp\C:\Users\*redacted*\.config\rclone\rclone.conf059243011: The filename, directory name, or volume label syntax is incorrect.

Any command that requires writing to the config, for example rclone config or any command involving a remote like google drive which requires writing to the config fails as a result.

@breunigs this is probably related to your changes.

This had an easy fix - please find a beta here:
https://beta.rclone.org/v1.36-154-gb0474022/ (uploaded in 15-30 mins)


Adde getDeviceIDFA,getMobileService,getCurrentBatteryPercent,getCurre…

Added getDeviceIDFA,getMobileService,getCurrentBatteryPercent,getCurrentNetworkType method for iOS

Current network type is exposed via the react-native netinfo APIs

Pretty sure IDFA is in core (AdSupportIOs)

Hi there!
I'm a new maintainer on this project and I'm trying to clear the PR queue.
Are you still available to bring this PR up to date with latest changes and discuss it?
Please let me know, thank you!

No, it's been over a year, I don't work in tech anymore.

On 6 Feb 2018, at 5:06 pm, Mehdi Achour<EMAIL_ADDRESS>wrote:
Hi there!
I'm a new maintainer on this project and I'm trying to clear the PR queue.
Are you still available to bring this PR up to date with latest changes and discuss it?
Please let me know, thank you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/rebeccahughes/react-native-device-info/pull/145#issuecomment-363471034, or mute the thread https://github.com/notifications/unsubscribe-auth/AAB4awVMc5ILbrpWTu_7D1rNPkvyJMNWks5tSHiZgaJpZM4Lqybw.

@ThisIsMissEm are you the original pull request submitter?


IE9 compatibility issue

I had some problem in resource visualization con IE9 (Masonry js lib error).

Some text in the page was overlapped.

I think this is due to a compatibility issue that can be fixed adding a meta-tag just after the  tag

to fix it, as you privately suggested, I added

https://github.com/dvcama/LodView/blob/master/src/main/webapp/WEB-INF/views/inc/header.jsp#L8


multithreading use from_msgpack  leading very slow

Description

I have many to_msgpack binary file, like 1-1000;

I use multithreading read these binary file through from_msgpack;

void ff(int i) {
    std::ifstream infile(std::to_string(i), std::ios_base::in | std::ios_base::binary);
    std::istreambuf_iterator<char> iter(infile);
    std::vector<uint8_t> v_msgpack{};
    std::copy(iter, std::istreambuf_iterator<char>(), std::back_inserter(v_msgpack));
    auto start = std::chrono::system_clock::now();
     j = json::from_msgpack(v_msgpack);
    auto end = std::chrono::system_clock::now();
    std::chrono::duration<double> elapsed = end - start;
    printf("%d load cost= %f \n", i, elapsed.count());
}

int main() {
  {
   // change thread_num get different load cost time
    int thread_num = 20;
    vector<thread> th_pool;
    for (int i = 0; i < thread_num; ++i) {

      th_pool.emplace_back(ff, i);
    }
    for (int i = 0; i < thread_num; ++i) {
      th_pool[i].join();
    }
  }
  return 0;
}

when thread_num =2

0 load cost=7.471617
1 load cost = 7.226486

when thread_num = 20

0 load cost=40.xx
1 load cost = 40.xxx
.
.
.
x load cost 53.xx

Reproduction steps
see description
Expected vs. actual results
I think when use multithreading, the load cost time shoule almost the same.  it looks like multithreading competition
Minimal code example
No response
Error messages
No response
Compiler and operating system
Linux DESKTOP-E1TD784 <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Apr 2 22:23:49 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Library version
3.9.1
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[X] I can successfully compile and run the unit tests.

The library does not use any static memory that would hinder multithreading - maybe the operating system has limits on opening files? Can you share the files to have a reproducible setup?

my test file is  large，average size is 300M，so i can not share，but you can construct array
with 100M items,I think is because allocate memory too many times ,


Scale multiple paragraphs to the same size

I have got three text blocks next to each other each in its own div.
How can I achieve that the text size of each of the text blocks is the same
and fits into each div?

Hey I did not understand exactly what you want to accomplish, this project objective is to make texts fit fixed-sized divs perfectly, no matter the div size. As such there is no way to make the plugin do what you ask, but you could just apply the plugin to one of the blocks, read its font-size and line-height and apply it to the other. Another option is to use the maximumFontSize option so at least if it is a value smaller than the div size both blocks would have the same size.


Contributing docs

These are just some fixes and additions to CONTRIBUTING.md. We bumped golang to v 1.10 since that is what the bosh release is compiled with. Any of these that you don't want let me know and I will rebase them out.

Thanks for the fixes, @jasonkeene ! These seem great to me, prioritizing for the team to evaluate.
Best,
Eric


Fix SC2053 for shellcheck

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Fix SC2053 for shellcheck
Which issue(s) this PR fixes:
xref https://github.com/kubernetes-sigs/kwok/issues/470
Special notes for your reviewer:
Does this PR introduce a user-facing change?
NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/ok-to-test

/lgtm


Add be_within_range matcher

Ouch!...
Apparently, trait RangeBounds was stabilized in rust 1.28 only.
@zummenix, would you agree to bump the min rust version up to 1.28?

@bekh6ex, yes, let's bump to 1.28.

@bekh6ex, I took the liberty and fixed an error related to imports in 1.28.
Thank you for the contribution!


Set a explicit nodePort

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-28 here


Is this repository abandoned?

The mkinitcpio-dropbear package hasn't received an update in about eight years.
There are issues and pull requests that should be considered.

Duplicate of #12. My offer there still stands. I suggest closing this and keeping the focus on one issue where discussion and background is already in place.

The original author apparently doesn't use this any more, but has just added me as a collaborator, and I still use this, so no it is not abandoned. I'll work or facilitating PRs and tinker with improvements going forward until such a time as I no longer use it.


[stable/prometheus-operator] Use new API version for ingress

What this PR does / why we need it:
Use new API for ingress if it is available

/ok-to-test

/lgtm


CBL-2144 : Fix default conflict resolution

Fixed default conflict resolution logic for replictor.
Added private API to get generation of the document.
Added tests.

@borrrden Can you review this PR when you have a chance?


Server does not shutdown gracefully when interrupted (on Windows)

As reported in https://github.com/flexxui/flexx/issues/525#issuecomment-440926361, the running server, when interrupted either:

stops like it should.
stops with an error (File "..\python37\lib\selectors.py", line 314, in _select).
does not stop until a connection is made, and then does the above.

(I am able to reproduce this with both flx.run() and flx.start(). Maybe something changed in Tornado or asyncio, breaking our use of it?

FYI, on Windows ctrl-c and ctrl-break are different, (ms ref). I didn't find what looks like an idiomatic python handling method. there is this though:

https://stackoverflow.com/questions/24426451/how-to-terminate-loop-gracefully-when-ctrlc-was-pressed-in-python
https://stackoverflow.com/a/1364409/14420
https://mail.python.org/pipermail/python-dev/2017-August/148800.html (explains why it's not straightforward)