Merge upstream tag v1.1.0

Operator SDK v1.1.0

Merge executed via ./UPSTREAM-MERGE.sh v1.1.0 upstream master

Overwritten conflicts:
CHANGELOG.md
cmd/operator-sdk/bundle/create.go
cmd/operator-sdk/bundle/validate.go
cmd/operator-sdk/main.go
go.mod
go.sum
hack/generate/changelog/internal/util/fragment_test.go
hack/generate/changelog/util/fragment_test.go
hack/generate/changelog/util/fragment_test.go.bak
hack/tests/e2e-ansible-molecule.sh
images/scorecard-proxy/bin/entrypoint
images/scorecard-proxy/bin/user_setup
internal/ansible/controller/reconcile.go
internal/cmd/operator-sdk/bundle/validate/validate_test.go
internal/cmd/operator-sdk/generate/bundle/bundle.go
internal/generate/collector/clusterserviceversion.go
internal/generate/collector/clusterserviceversion_test.go
internal/generate/collector/manifests.go
internal/olm/client/client.go
internal/plugins/golang/v2/init.go
internal/registry/labels.go
internal/registry/validate.go
internal/scaffold/ansible/dockerfilehybrid.go
internal/scaffold/ansible/go_mod.go
internal/scaffold/go_mod.go
internal/scaffold/helm/go_mod.go
internal/util/projutil/project_util.go
internal/version/version.go
pkg/ansible/run.go
pkg/helm/run.go
release.sh
test/e2e-new/e2e_suite.go
test/test-framework/build/bin/entrypoint
test/test-framework/build/bin/user_setup
website/config.toml
website/content/en/docs/building-operators/golang/tutorial.md
website/content/en/docs/golang/project_migration_guide.md
website/content/en/docs/installation/install-operator-sdk.md
website/content/en/docs/new-cli/operator-sdk_generate_packagemanifests.md
website/content/en/docs/olm-integration/generation.md
website/content/en/docs/scorecard/scorecard-alpha.md
website/content/en/docs/upgrading-sdk-version/v0.17.1.md
website/content/en/docs/upgrading-sdk-version/v0.19.0.md
website/content/en/docs/upgrading-sdk-version/v0.19.1.md
website/content/en/docs/upgrading-sdk-version/v0.19.2.md
website/content/en/docs/upgrading-sdk-version/v0.19.3.md
website/content/en/docs/upgrading-sdk-version/v0.19.4.md

/hold


Filter Modal Bug

Describe the bug
Remove mechanism is not the same as insert new selection in Filter Modal.
To Reproduce
Steps to reproduce the behavior:

Go to 'Course Searching Page'
Open Filter Modal (dept or course type)
Add some courses then click save.
Open again, remove selected by clicking again, then don't press "save", press "X" or anywhere outside Modal.
The selected list should not be changed because we do not press "Save". It should remain the same.

Expected behavior
However, it change directly.

Acknowledged.👌

Just a simple solution currently and I will refactor these filters in the future and try to keep closeOnOverlayClick.

resolved in #71


SecondaryCommands button bug

I deploy app using Template10 (Minimal) on Lumia 640 with OS build 10.0.10586.63
If I open (or reopen) Main Page and tap on SecondaryCommands button, then I will see bug above

This is already fixed. Try from git if you use latest NuGet version, otherwise wait for next version.

@MichaelPetrinolis, thank you!


Fix - Chrome autoplay policy

Added a little bit of code in the play() function to comply with the Chrome autoplay policy.
Now errors don't pop out in the console whenever the autoplay is prevented.
Referring to the issue I opened - it is now fixed.

Thx for the PR! I've made the changes and published this patch to npm (https://www.npmjs.com/package/uifx)


Deployment

Deployment
User Story
As a bootcamp student
I want to view the bootcamp schedule on a website
So that I can easily know what subjects are happening next
Acceptance Criteria
GIVEN a bootcamp calendar
WHEN I visit the bootcamp calendar website
THEN I can access my website from any browser on a pc or mobile device

Site is now live: https://sidneybasa.github.io/prework-study-guide/


Support zend-servicemanager v2+v3

This patch builds on the one originally submitted via #21, and expands it to provide support for both v2 and v3 of zend-servicemanager. Specifically, it introduces the following changes:

composer.json:
Updates PHP version to ^5.5 || ^7.0
Updates develop branch alias to 2.6
Updates zend-servicemanager requirement to ^2.7.2 || ^3.0
Code changes:
Updated LoaderPluginManager to implement both validate() and validatePlugin() (latter proxies to former).
Updated TranslatorServiceFactory to implement both v2 and v3 factory methods, with v2 method proxying to v3.
Updated HelperConfig:

define aliases + invokable factories;
alter configureServiceManager to call SM methods setters instead of configure() (for portability)
implemented toArray() (v3 method)

Updated TranslatorTest to use Config objects + configureServiceManager() method to seed SM and plugin manager instances.

Labeled as a WIP, as several dependencies are on components that also bring in SM refactors, but which have not been updated to support both v2 and v3 of the SM (only v3). As such, the CI checks on this will fail until those are updated.

@weierophinney can you update the composer.json and .travis to use zend-servicemanager 2.7.3?

@ezimuel Done! I made it test against the v2 and v3 versions of both the service manager and event manager while I was at it.


Fix for HHH-11284 require that this class is enhanced, not only its superclass(es)

https://hibernate.atlassian.net/browse/HHH-11284
This pull request includes both the fix and the test showing it works.

Updated both the javassist and bytebuddy implementations and tested both.

@barreiro Can you take a look on this PR? Thanks

@barreiro @vladmihalcea can I convince someone to take a look and (hopefully) merge this?

@barreiro @vladmihalcea bump, please take a gander? Test case and fix are included in this PR.

superseded by #1661


Task 45629821: Add support for GZIP Compression

Enabling GZIP compression for LibHttpClient using ZLib for Win32 and GDK Platforms initially. This work will expand to other platforms down the road.

HCHttpCallRequestBodyReadFunction requestBodyReadFunction{ HC_CALL::ReadRequestBody };

Is it supported to set a RequestBodyReadFunction and have gzip compression enabled?

Refers to: Source/HTTP/httpcall.h:60 in 36e12bc. [](commit_id = 36e12bc0089a7347a9a7eb60f0c14d3858ecf9d2, deletion_comment = False)


Render  in source mode

<pre> tags does not renders like foo bar

Notice: This plugin is no more maintained and it has been removed from Jetbrains plugins repository. There is a Markdown plugin officially supported by Jetbrains.


opensearch on FreeBSD does not reliably restart the service

ISSUE TYPE

Bug Report

ROLE VERSION
n/a

CONFIGURATION
n/a
OS / ENVIRONMENT
FreeBSD
SUMMARY
the restart handler does not reliably restart the service because /usr/local/etc/rc.d/opensearch does not wait sometime between stop and start.
STEPS TO REPRODUCE
run the test.
EXPECTED RESULTS
the test passes.
ACTUAL RESULTS
sometimes, the test fails.

https://github.com/trombik/freebsd-ports-opensearch/commit/628e07c6895301319b655c9ab34df94578081e93 should fix this


Add Validated in standard

Add Validated in standard with pure and apply

To be honest, not being very comfortable with Kotlin, and that the code is very complicated (for a layman; I'm not implying that it's your fault) but I don't feel that the semantics of "Validate" are correct. Generally, the opposition between Result and Validate is that the right-hand type of Validate is a traversable construct (commonly, a non-empty list) and respects the interface of a Semigroup, to accumulate errors (unlike Result).

(So a quick fix is only renaming Validation into Result)

Agree! The proposed construction is not the appropriate one for Validated. If you change it to Result the corresponding monad can also be defined.
Reference: https://typelevel.org/cats/datatypes/validated.html
Reference: https://hackage.haskell.org/package/validation-1.1.1/docs/Data-Validation.html

Agreeing with the missing behaviour, as some other pieces wasn't implemented yet.
Closing this issue, it was maily to try how to add a a struct in standard


bugfix: Parcellation.supported_spaces crash

fixes #230

In fact, a working version was already implemented in Region, but Parcellation (as a special region) still had an obsolete implementation of supported_spaces, which I just removed to solve the problem.


Dynamically render icons by a string name

How can I dynamically render icons in React based on a string value?

You've got a few choices. You can import the whole library and index into it by component name:
import * as P from "@phosphor-icons/react";

function App() {
  const Avocado = P['Avocado'];
  return <Avocado weight="fill" color="darkolivegreen" />;
}

This has perf consequences, but I'd do this for something like an electron app, where bundle size is not an issue and the simplicity is nicer.
Another way (for which the exact details depend on your build tools and specific environment) is with dynamic imports, using code-splitting and Suspense to lazily import icons as needed. This example works with React + Vite:
import { lazy, Suspense } from 'react';
import type { IconProps } from '@phosphor-icons/react';

// This is essentially a map of relative paths to dynamic import functions,
// but since we use named exports we need to transform them to default exports.
// Most bundlers don't support template strings in dynamic imports, so we essentially
// let it code-split the whole Phosphor lib and allow them to be loaded at runtime.
const importMap = Object.entries(import.meta.glob("../node_modules/@phosphor-icons/react/dist/ssr/*.mjs"))
  .reduce((acc, [path, fn]) => {
    acc[path] = (name) => fn().then((mod) => ({ default: mod[name] }));
    return acc;
  }, {});

function relativePathForIcon(name: string) {
  return<EMAIL_ADDRESS>}

export function LazyIcon(props: IconProps & { name: string }) {
  const { name, ...iconProps } = props;
  const Icon = lazy(async () => importMap[relativePathForIcon(name)](name));

  return (
    <Suspense fallback={null}>
      <Icon {...iconProps} />
    </Suspense>
  );
}

// THEN IN YOUR APP
function App() {
  return <LazyIcon name="Avocado" weight="fill" color="darkolivegreen" />;
}

In nextjs, I keep having: TypeError: (intermediate value).glob is not a function whether I add 'use client' or not :/

In nextjs, I keep having: TypeError: (intermediate value).glob is not a function whether I add 'use client' or not :/ (nextjs v14.0.4)
Also tried with dynamic(() => import(relativePathForIcon(name))) but I get a Cannot find module error

Found a workaround:
export function LazyIcon(props: IconProps & { name: IconName }) {
    const { name, ...iconProps } = props

    const Icon = dynamic(
        () =>
            import('@phosphor-icons/react').then((mod) => {
                return mod[name]
            }),
        {}
    )

    return (
        <Suspense fallback={null}>
            <Icon {...iconProps} />
        </Suspense>
    )
}

Yeah, unfortunately every bundler is gonna have a different syntax or trick to get dynamic import of a module known only at runtime, and some won't support it at all. This problem isn't unique to this library, or to icons.
If you find yourself needing this feature in a place where it simply won't work, the React icon components are probably not your best bet. Using @phosphor-icons/web can get you smaller overall footprint if you are only using 1 or 2 icon weights but possibly any of the icons. And since they are fonts, they are cachable and fast.

Thanks for the quick reply! My workaround works, but seems to be quite inefficient, I'll see if I can improve it. However, for the name type IconName, this is incompatible with both our approaches since IconName is kebab-cased while component names are PascalCase. To make the type compatible with the actual component name you'd have to create a mapping from kebab-case to PascalCase which isn't ideal imo. Do you have a better idea? Should I write this to @phosphor-icons/web ?

I don't know what part of your application depends on using the kebab-names, but if you must use them, you could just have a function instead of a mapping:
function pascalName(str: string): string {
  return str
    .split("-")
    .map((substr) => substr.replace(/^\w/, (c) => c.toUpperCase()))
    .join("");
}

Alternately, you can use the @phosphor-icons/core library which contains metadata for every icon, including both its kebab-name and PascalName.

You are right, that is what I meant by mapping sorry for the confusion. I ended up finding such function using regex:
function clearAndUpper(text: string) {
    return text.replace(/-/, '').toUpperCase()
}
function toPascalCase(text: string) {
    return text.replace(/(^\w|-\w)/g, clearAndUpper)
}
// toPascalCase('airplane-tilt') -> AirplaneTilt

However, I tried loading the icons using SSR and it failed. The svg and path are in the client html, no error shows up, but the path has a width and height of 0 and I can't get the icons to actually show..
// no 'use client' here

const Icon = dynamic(
    () =>
        import('@phosphor-icons/react/dist/ssr').then((mod) => {
            return mod.Briefcase
        }),
    {}
)
// ...
<Icon size={16} />


upgrade site and add docs/zh

Hi,
I saw Win Travis failed because out-of-memory.
Please apply the following patch
git apply travis.patch.txt

travis.patch.txt
It will split the website builder into a single container.

Hi,
please add the following content into pom.xml
After Line 569:
                            <!-- only for Travis CI with WinOS-->
                            <exclude>hadoopbin</exclude>
                             <exclude>windowssystem32</exclude>


Update Jupyter Labs to be DEA's default launcher for notebooks on the NCI

On the NCI, DEA currently recommends users run a shell script that creates a desktop icon which uses jupyter notebook to launch notebooks on the NCI (set up here: /g/data/v10/public/digitalearthau)
For compatibility with the DEA Sandbox and to reduce the documentation that needs to be written, we should switch the default launcher to jupyter lab instead.
Leaving this as an open issue for now, because we should do the change after we have updated the bulk of dea-notebooks to work on jupyter lab on the DEA Sandbox.

@caitlinadams: I think you can safely write dea-docs documentation to focus just on JupyterLab, and we'll change it over to the default as soon as the refresh has progressed


Added progress bar for Remove-Item cmdlet

PR Summary

Fixes #19538
Added progress bar for Remove-Item cmdlet.
PR Context

Currently there is no progress bar when running Remove-Item -Path somedir -Recurse -Force. I've included a progress bar similar to Copy-Item.
Some reason when using terminalizer the progress bar doesn't show up right and overlaps the status activity message. If you run this on its own it displays fine. Couldn't work around this to get a better terminal recording on Windows, open to suggestions 😄.

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Any update? Has an alternative been presented somewhere else?


Integration: belocalhero.com

Wordpress- API:
For Example: https://belocalhero.com/wordpress-api/live/wp-json/wp/v2/place?page=16

I contacted the owner today and asked for their permission to crawl their data.

Request was declined due to data-protection issues


pymdownx.superfences prevents HTML from being interpreted in diagrams

Hyperlinks, bold, etc. in diagrams stop working, when superfences are activated according to instructions.
This issue occurs when all conditions are satisfied:

Loose mode activated (allow HTML to be interpreted in diagrams).
pymdownx.superfences is activated.

In mkdocs.yml:
plugins:
  - ...
  - mermaid2:
      arguments:
        securityLevel: 'loose'

markdown_extensions:
  - ...
  - ...
  - pymdownx.superfences:
      # make exceptions to highlighting of code:
      custom_fences:
        - name: mermaid
          class: mermaid
          format: !!python/name:pymdownx.superfences.fence_div_format

An issue was opened on the pymdown repo.
@facelessuser responded:

Probably by creating your own custom fence and referencing it instead. Here is the default one which escapes: https://github.com/facelessuser/pymdown-extensions/blob/master/pymdownx/superfences.py#L161.

Simply stated, we must reimplement the function without the _escape(source) part. So that should be easy.
That DIY approach is in the philosophy of pymdown-extensions.  The WIRDOARD pattern applies ("when in Rome, do as the Romans do").
The Superfences / Custom fences approach is described here.

The way to define a function for pymdown, through YAML is described on this page.
This seems quite simple, really:
In the following example:
markdown_extensions:
  - ...
  - ...
  - pymdownx.superfences:
      # make exceptions to highlighting of code:
      custom_fences:
        - name: mermaid
          class: mermaid
          format: !!python/name:pymdownx.superfences.fence_div_format

The function fence_div_format belongs to the pymdownx.superfences package.
Indeed, you can write in Python:
>>> from pymdownx.superfences import fence_div_format

So what si really needed, is to add a function to the mermaid2 package that woud have this form:
def fence_div_raw(source, language, css_class, options, md, classes=None, id_value='', **kwargs):
    """Format source as div."""

    if id_value:
        id_value = ' id="{}"'.format(id_value)
    classes = css_class if classes is None else ' '.join(classes + [css_class])

    return '
%s
' % (id_value, classes, source)

And call it as:
          format: !!python/name:mermaid2:fence_div_raw

@fralau I think you mean (. not :):
                format: !!python/name:mermaid2.fence_div_raw

Thanks a lot for pointing that typo out!

@facelessuser: I found an interesting issue.
The function works perfectly as described (the HTML is rigorously intact, with the '<' and '>' symbols), and a print statement proves it.
However, when I check the HTML code produced in the browser, I still see the that '<' symbols have been transformed into the entity >.
Since the problem does not manifest itself without pymdownx.superfence, a reasonable hypothesis is that there is a second level of escaping in superfence, but downstream to the my custom function?
Of course, that's a just a guess, and I perhaps missed the obvious. What would you say?

@lacy-llana you mean the trick of implementing your own HTML-producing function?
It would be worth trying.

@facelessuser Thanks, I thought you would not.
(Interesting you called this "stashing" BTW, it totally makes sense; is it a usual term in that context?)
 If you look at the store function, it calls markdown's stash function which stores the HTML in a dictionary and returns a placeholder that is used as a key to retrieve the HTML in the final stages, granted I also store the original source in my own stash as well, but that is to work around shortcomings of Python Markdown and restore the original source if we got too greedy and processed something under HTML tags we shouldn't have.

@facelessuser Thanks for this precision. I was asking because I was faced with a similar problem (but in the context of pandoc) and I did not want pandoc to process stuff normally because it would mangle it.
And I worked out a concept of stashing, called it so and implemented it.
It's another instance of "originality is often ignorance of literature" 😄 (or perhaps convergent evolution).
Concerning the issue at hand, you could check the code of my on_page() method if you wish (link in my earlier message).
For my part, I will check tomorrow what comes in and what comes out of that procedure, and I will let you know.

I wouldn't definitely output the code before you process it and after you process it (before you return it). As make sure it gets there as it should and then is also returned as it should. Lastly compare what Python Markdown returns as the final product before any mkdocs plugins touch it.

I looked at your code, are you not using a custom fence? It looks like you are just changing pre to div and such post process.

It's possible that I'm just confused at what you are trying to do. Can you post raw source that your are trying to use in your Markdown source? I think maybe if I can see the source and run it myself, I can better understand what you are trying to do. At the very least, I can compare the output in the live mermaid editor: https://mermaid-js.github.io/mermaid-live-editor.

@fralau I was referring being able to note what theme you want mermaid to use in the plugin configuration as shown below. In the same way that indicating the security level no longer allows for links within the diagram, indicating the theme also gets overridden in the Superfences usage.
  - ...
  - mermaid2:
      arguments:
        securityLevel: 'loose'
        theme: 'dark'

@lacy-llana Got you! I tested and yes it works!

OK I cracked it. It was complex because it involved mermaid, mkdocs, and superfences. I had a few moments of confusion.
But in the end, it was not really complicated.
@facelessuser The problem is that the mermaid2 plugin was expecting a  tag, because that is what it should always get in the default case, when the pymdownx.superfences extension is not activated.
My first version of the custom fencing function produced a
 (which was, of course, the "intuititive" thing to do). As a result the plugin was failing to catch the ball. When I changed the function so that it produces , the plugin started catching the ball and everything went back to normal.
Here is the new version of the function, now called fence_mermaid (for simplicity): https://github.com/fralau/mkdocs-mermaid2-plugin/blob/5ccbe4d6a2a67052c303606cb74d30388b48a843/mermaid2/init.py#L4
@lacy-llana I am happy to to let you know that according to my tests, the new version (0.2.1) is also preserving the dark mode! (same cause, different effects).
Could you test with this new version (on github only, for the moment), and let me know if it works for you?

Glad you got it figured out.

@fralau - Tried to clone the repo and install manually/locally and I'm hitting some snags with my version of Python/access to the correct directory structure. It is likely user error given that I've only been working with referencing it via pypi. If you've got tips on that, I will definitely take them but otherwise will definitely be ready to try it out when it is pushed to pypi!

@lacy-llana No problem: as soon as it's ready on pypi, I will let you know!

It was almost solved.
I hate escaping... 😢 I found another error, this time caused by beautiful soup. It does not happen all the times.
When there is this in a mermaid diagram:
metadata --> pandoc["1. Pandoc"]

It transforms it into this:
   metadata --> pandoc["1. Pandoc"]

This mangles the file, and crashes the mermaid script (javascript). 😠
So I need to work out a better approach. Back to the drawing board.

@fralau Why would that break anything? Mermaid can read escaped > and < if they are not part of HTML tags. If you remember, I escape those characters (as all people do for regular text in HTML) and I still have diagrams that render with Mermaid: https://facelessuser.github.io/pymdown-extensions/extensions/superfences/#custom-fences.

@facelessuser  One thing that I can say now with greater certainty, is that superfences is not to blame. So far it has been behaving exactly as advertised.
It's somewhere within the mermaid plugin in the on_post_page() method.
To be honest, I have one mkdocs project where the plugin works perfectly (the one that is published with the project) and the mermaid diagram is displayed properly; and one where it doesn't (more complicated).
I still have no clear idea why this happening. But at least I have a testing process: I will keep adding stuff to the plate of this test project that works, until I manage to break it.

While Mermaid is super powerful, it is not without its bugs.

@lacy-llana The new version 0.2.2 of the plugin is uploaded on pypi. It seems to work on basic tests. Could you let me know how it works for you?
https://pypi.org/project/mkdocs-mermaid2-plugin/0.2.2/

Still isn't rendering the theme on my end, unfortunately. :/

@lacy-llana Did you update your mkdocs.yaml file?
https://github.com/fralau/mkdocs-mermaid2-plugin#using-mermaid-and-code-highlighting-at-the-same-time

I pushed a new version (c52b25d). No change of functionality, but improved documentation.

That worked! Missed that update. Thank you so much!

@lacy-llana Great! I'm doing to do a few more tests.
If everything is fine, I'm going to close that issue.

Testing it out a bit further, and it looks like self closing tags aren't working. So in order to do  you have to actually do . Is this the intended behavior?

@lacy-llana No, it isn't. I had no problem with . Are you sure the problem is with that tag and not another?
The way to check, is simply to visualize the generated HTML code in your browser. The closing tags pile up at the end.

Pretty certain - if I replace the
 that is in the mermaid diagram with

 it works, but will crash if they aren't replaced and render a 404 error.

Granted, after I get it live it will let me convert them back...which is interesting. I'm doing some more digging.

@lacy-llana
As @facelessuser noted:

While Mermaid is super powerful, it is not without its bugs.

I'd say it is a little temperamental 🙂

Example of idiosyncracy: mermaid does not accept a box called default.
Unless someone strongly opposes, I'm going to close this issue.


Its taking more time to close the exisiting client connection

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
I'm have opened a connection like following inside a function.
// Create a client and connect to http://<IP_ADDRESS>:9201
client, err := elastic.NewClient(elastic.SetURL("http://<IP_ADDRESS>:9201"))
if err != nil {
// Handle error
panic(err)
}
defer client.Stop().
Issue is it takes, more than 2 min to close the function.
I understand that client.Stop() will be executed only after function exits. The outcome of the function I'm getting less than 10 seconds.
Is there any way to specify time to close the connection?

Any suggestions ?

Hmm... I usually create one client in the main function of the application and don't call Stop, as the application will exit anyway when main exits. How do you setup your client?

Hi Olivere..Thanks for response....
Actually, my application does not exists. It is server which listens for some inputs,
and based on those inputs I will open a Elastic client, fetch data from elastic and need to close the
elastic DB connection.......
Elastic Client is called inside a Handler function.
Now after I quit that handler I expect that, DB connection to get closed since I use client.Stop().
It it stopping, but it takes more 1 to 2 min to close the connection.
Handler expection is completing with less than 5 sec, but closing the DB connection is taking much time.

If you use Elastic on a request scope, you should definitely switch to elastic.NewSimpleClient. It opens a connection much more quickly and closing is not necessary because it doesn't start any other goroutines for e.g. healthchecks and sniffing. Use elastic.NewClient for a long-running client that can be shared by many goroutines.

Ya sure, I will try using it..
Just on doubt, whether NewSimpleClient will support all the configurations supported by NewClient.
List of configurations are given below.
https://github.com/olivere/elastic/wiki/Configuration

In general, yes. See here. But e.g. enabling the sniffer when the sniffing process isn't started is certainly a no-op.

Thanks ..I will try the elastic.NewSimpleClient() with few options then.

I'll close this for now. If you keep having issues, just let me know and I'll re-open.

Hi .. I tried working with elastic.NewSimpleClient(). I can still see connection is not getting closed immediately, after querying. With the command "netstat -tunlpa" I see that 9200 port is taking few delay around 2 to 3 min on average to close. I see the status as "TIME_WAIT" .. Is there any other suggestions ?

That might be normal behaviour with HTTP connections in the standard library. We use the normal HTTP client, but you may specify your own with different timeouts etc.

It was taking around 1 min to close the connection now.  As you said that was dependent on HTTP settings defined in file /proc/sys/net/ipv4/tcp_fin_timeout.
Thanks for the info.  :-)


TypeError: Cannot read properties of undefined (reading 'keywords')

I receive this error while editing some members
TypeError: Cannot read properties of undefined (reading 'keywords')
	at e.exports.getDocs (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:35328)
	at processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async Object.provideDocumentSymbols (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:43951)
	at async o.provideDocumentSymbols (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:129081)

followed by
Error parsing /wfisrc/qrpgsrc/prp03c.rpgle: Cannot read properties of undefined (reading 'keywords')
TypeError: Cannot read properties of undefined (reading 'keywords')
	at e.exports.getDocs (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:35328)
	at runMicrotasks (<anonymous>)
	at processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async Object.provideDocumentSymbols (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:43951)
	at async o.provideDocumentSymbols (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:129081)

Program is a **FREE source member at 4871 lines of code.
The outline and other features seem to be disabled while I see these error.

@m-tyler There are really two ways to solve this:

Share some source with me privately with me so I can recreate it
Recreate the issue in a small file

@m-tyler Sounds like the issue could be in one of the copybooks from the base library you copied from then. Let me know if you eventually recreate the issue! I'd love to tackle this.

I am not certain that is it.  I only copied the main program source into my developer temp member.   I currently have NO members in my developer /COPY source file.  We use only one source file, QCPYSRC, for /COPY members.

I have to correct my original statement.  I thought this issue was from a **FREE source version because that is the source I was working with when I noticed the issue.  However, in my attempt to recreate the problem and discovered which source is causing the error.   I should have noticed which source caused the error because in the second image in the first post, sorry for the misleading information.

The source that causes the error (same program just the production version) is a mix of fixed and free-format RPGLE which has free-form internal procedures.   This is the same program source I copied into my developer temp member, I talked about above which does not cause the error.
We have four source environments, developer, integration, quality, and production.  The error only appears with this source n the production environment.  I have put the source into the other environments and opened the source and the error does not appear.
Is there a setting (in my current installation) I can add to make the logging be more verbose?  Maybe so it can report the source part that is causing the error?

There isn't more verbose logging (yet). That is something that could be looked into, though.

So I am now able to recreate the issue in my developer environment.  I copied all the /COPY members into my environment and opened the temp member source of the production version.  I get the same errors.  I am going to one by one rename the /COPY members to see if I can find the one causing the issue.

Okay, I think I have recreated the issue.
This /COPY member causes errors to appear when opened.
F4DATE.RPGLE.txt
When included in a source member it gives the reported error.  I only need to include this member in a temp source file for the reported error to appear.
Error when /COPY member opened.
Error parsing /WIASP/USER/QCPYSRC/F4DATE.RPGLE: Cannot read properties of undefined (reading 'keywords')
TypeError: Cannot read properties of undefined (reading 'keywords')
	at e.exports.getDocs (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:35328)
	at Object.provideDocumentSymbols (c:\Users\USER\.vscode\extensions\halcyontechltd.vscode-rpgle-0.7.2\dist\extension.js:1:43959)
	at o.provideDocumentSymbols (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:129102)
	at c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:159280
	at te._withAdapter (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:158424)
	at te.$provideDocumentSymbols (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:83:159258)
	at s._doInvokeHandler (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\
api\node\extensionHostProcess.js:86:13828)
	at s._invokeHandler (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:86:13512)
	at s._receiveRequest (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:86:12121)
	at s._receiveOneMessage (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:86:10843)
	at c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:86:8949
	at g.invoke (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:145)
	at b.deliver (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:2265)
	at v.fire (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:1843)
	at a.fire (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:19025)
	at c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:102:34863
	at g.invoke (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:145)
	at b.deliver (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:2265)
	at v.fire (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:1843)
	at a.fire (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:19025)
	at r._receiveMessage (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:23606)
	at c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:21140
	at g.invoke (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:145)
	at b.deliver (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:2265)
	at v.fire (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:58:1843)
	at v.acceptChunk (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:15856)
	at c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:66:14986
	at Socket.R (c:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:102:13798)
	at Socket.emit (node:events:390:28)
	at addChunk (node:internal/streams/readable:315:12)
	at readableAddChunk (node:internal/streams/readable:289:9)
	at Socket.Readable.push (node:internal/streams/readable:228:10)
	at Pipe.onStreamRead (node:internal/stream_base_commons:199:23)

@m-tyler Great work on tracking it down. I have been able to get the same error.

0.7.3 coming out now with a fix. Test case was also created for this issue.


Write more client DNS tests

When #1762 has a working Split DNS implementation, we should make sure those core components are tested:

[x] DNS integration tests in CI

[x] Add a test.httpbin.docker.local service in docker-compose.yml
[x] Add ?.httpbin.docker.local and *.docker.local as Resources in the test env seeds.exs
[x] Test that client can ping them
[x] Test that they returns CGNAT IP
[x] Test that lookups for e.g. github.com still work and do not return with CGNAT IP

[ ] Unit tests

[ ] Test more combinations of DNS lookups using mocks in Rust: Single name resolving to multiple IPv4 and IPv6 addresses. Single IPv4/6 being resolved to by multiple different subdomains and TLDs.

Here are the integration tests for firezone-connection: https://github.com/firezone/firezone/tree/main/rust/connection-tests
The main difference to the main docker-compose.yml is that this one is setting up a network topology where both clients are behind a NAT.
To test DNS, we could add a another container (a resource) next to the gateway (in the test called listener) that the client will try to reach via a DNS name.

Should be implemented as part of #1762

Maybe this should be in the Windows smoke test too?

Maybe this should be in the Windows smoke test too?

The issue with testing the other clients is that docker isn't available on those GH runners :-(
There are service containers but those must be specified on the job and are Linux-only

I am hoping that we can more easily test this once the tunnel is sans-io as it should be possible to run a client and a gateway tunnel next to each other in-memory without having them any actual IO.

What's the next step for this one?
I think we have one integration test that uses Docker Compose to reach httpbin.test over DNS.

I think we can save further DNS tests for another PR, marking this as done (feel free to reopen otherwise)


Harry Potter and the Chamber of Secrets (Issue)

I used your latest version 0.5.1. The game splash screen starts and the keyboard freezes... We can't control the game.
The latest version of the program that doesn't have this issue is DDrawCompat v0.2.1. But this old version ddraw.dll doesn't work well with the game. The game lags already in the options, if you move the cursor of the game.
Please fix. because it is impossible to play. And we really need  for the game the performance, stable fps in all locations.
The game uses DirectX7 renderer.
In default.ini
[Engine.Engine]
GameRenderDevice=D3DDrv.D3DRenderDevice

The game was tested on Windows 7 (64-bit)... The game's resolution was set to 1280x960 (4:3)

log
DDrawCompat-Game.log

It was broken by https://github.com/narzoul/DDrawCompat/commit/2f52e1028b45f3874dcd99fef74b95ab36c2a334
Oh well, I guess there is no good solution to that other problem, but it's not that important.
I'll fix this for the next release, for now here's a quick fix you can test:
ddraw.zip (diff.txt compared to v0.5.1)

This fix works!!! Thank you very much !!!

Fixed in v0.5.2.


vm/vmss create: remove human names from the admin name blacklist

Fix #4257.
Compute RP is starting to deploy the fix this week, so starting this week, David and John should be able to create vm/vmss w/o problems.
General Guidelines

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4283 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4283      +/-   ##
==========================================
- Coverage   70.09%   70.01%   -0.09%
==========================================
  Files         475      475
  Lines       29676    29742      +66
  Branches     4556     4599      +43
==========================================
+ Hits        20802    20823      +21
- Misses       7421     7459      +38
- Partials     1453     1460       +7

Impacted Files
Coverage Δ

...cli-vm/azure/cli/command_modules/vm/_validators.py
78.48% <ø> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
69.73% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.26% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

...li-cloud/azure/cli/command_modules/cloud/custom.py
13.79% <0%> (ø)
:arrow_up:

...nd_modules/azure-cli-interactive/azclishell/app.py
31.47% <0%> (+0.04%)
:arrow_up:

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 ce10f47...3e17339. Read the comment docs.


ES-Hadoop with SSL/PKI security issue. Key/Truststore clear text passwords exposed in Spark UI

Version Info
OS:         :   Any
JVM         :  1.8
Hadoop/Spark: Hadoop 2.7.3/Spark 1.6.2
ES-Hadoop   :  2.10
ES          :  2.4.0
Issue description
Clear text password in Spark UI for ES-Hadoop with SSL/PKI configuration
Steps to reproduce
Configure ES-Hadoop with SSL/PKI. It require the next properties (among others):
es.shield.ssl.keystore.password
es.shield.ssl.truststore.password
which holds clear text passwords for key and trustsore
Take a look at the picture attached (marked red) - these passwords exposed in Spark UI.
Business case scenario (production mode):
Actors:
Support representative - responsible for support customer in case any issue with Spark application
Customer representative - responsible for Spark application results from Customer side
Sustaining representative - second/thrird level of support
IT representative - responsible for Hadoop/Spark cluster set up including keys, passwords etc.
All actors have its own area of responsibility and appropriate permission to access sensitive information. Among all actors the only IT rep has permission to access key/trust store passwords.
Support acting on some Customer issue related to Spark application. In order to analyze the state of application he need an access to Spark application environment screen (among others). This screen could be shared with Customer and/or Sustaining engineer(s). At this moment is critical to hide (do not expose) some sensitive information - key/truststore password is one of it.
From Spark point of view there is no concept of sensitive properties since it is specific for appropriate add-on/plugin.
There is no way to hide/scramble these passwords since Spark does not expose such functionality (limited UI access does not help since user have to see some properties any way). Custom password scrambling cannot be implemented since these passwords are using deep inside ES-Hadoop (in org.elasticsearch.hadoop.rest.commonshttp.CommonsHttpTransport)
Suggested solution

Provide password scrambling functionality with some static algorithm (like encoding). This require some tool to scramble password before pass it to SparkConf.
Current approach suggest to have some place to store keystore and truststore accessible from all Spark executors (Hadoop nodes). Keystore and truststore have to be well protected (using file system permission for example). The only person who have access to it (key/truststore and passwords for it) is IT representative.
Due this i don't quite understand why we need to pass password(s) from Spark client side (with all related issues)? I'd rather suggest to implement concept of "secure store" which is some well protected location accessible for each Hadoop node containing keystore, truststore and passwords for it.
In simplest case "secure store" is a file directory with mask 100/400 for directory/files. Inside it we have keystore, truststore files with fixed names and some property file that contains passwords for it.
In that case we are passing the only "secure.storage.path" Spark property.
More sophisticated implementations, for example,  could provide something like LDAP/DB based storages (better accessibility and management) -> leads to (3) below.
Implement custom Java key management that implements all requirements.

I'd prefer solution (2) as most balanced in terms of effort/result. I've implemented directory-based storage for my application (require org.elasticsearch.hadoop.rest.commonshttp.CommonsHttpTransport patching among others) and can share it if you are interested.

The biggest item that needs to be accounted for is that the solution to this configuration issue must work across all supported technologies. Thank you for opening the request and providing some suggestions on implementations.

Looks like Spark guys have fixed security issue -> https://issues.apache.org/jira/browse/SPARK-16796
For now it became low priority improvement related to configuration, you can close it

Closed per user feedback


cellxgene-schema must validate cell_type_ontology_term_id

Design
See cell_type_ontology_term_id.
cell_type_ontology_term_id

Key
cell_type_ontology_term_id

Annotator
Curator

Value
categorical with str categories. This MUST be a CL term.If no appropriate high-level term can be found or the cell type is unknown, then it is STRONGLY RECOMMENDED to use "CL:0000003" for native cell. The following terms MUST NOT be used:

"CL:0000255" for eukaryotic cell

"CL:0000257" for Eumycetozoan cell

"CL:0000548" for animal cell

@jahilton / @corismall ready for testing!

Passes QA. Good-to-go
QA notebook


Compile for MacOS

Did someone successfully compile braw-decode for MacOS? Is it even possible?

Not that I am aware of and I don't own a new enough Mac to try it on so I can't really help out for now.


Getters for RepeatStep.emitTraversal / untilTraversal and LoopTraversal.maxLoops

These getters are required for providers to access the state of the RepeatStep when optimizing it.

This PR is for https://issues.apache.org/jira/browse/TINKERPOP3-907

This looks good to me. Simple getters added.
VOTE +1.
@pietermartin --- you are really introspecting deep into the traversal. Note that we have yet to publish the virtual machine step library and thus, things like LoopTraversal may not be in the specification. Thus, we can't guarantee you stability of the API in this area. Just a heads up.

+1

+1
RESULT: +1 (3) - i'll do the merge

Thanks for the PR @pietermartin - merged


Unpin numpy as a host-dependency

By having >= 1.14 in the host section, this disables our mechanism for building against the oldest supported numpy. This means the recipe will always pull in the newest numpy (currently 1.24), and by the way the numpy C API works, this means the all pysr builds need the newest numpy to run[^1].
Effectively, this turns >=1.14 into >=1.24 (soon 1.25), which is clearly not the intent. Please have a look at the relevant section in the knowledge base.
[^1]: numpy doesn't bump the relevant marker for every version, but almost.

Thanks. Just to confirm, I should leave the numpy pin in run as is?

~Thanks. Just to confirm, I should leave the numpy pin in run as is?~
Nevermind, I had forgotten the difference but just re-read the docs page.

As the docs explain, you should use - {{ pin_compatible('numpy') }} in run (which I don't see in #85)


Don't retry when retry_max_times == 0

Which issue(s) this PR fixes:
Fixes #3161
What this PR does / why we need it:
retry_max_times == 0 should also work as expected.
Docs Changes:
None.
Release Note:
Same with the title.

When I revert the fix, the added test fails expectedly:
=======================================================================================================================
Failure: test: Do not retry when retry_max_times is 0(BufferedOutputRetryTest::bufferd output for retries with periodical retry): <false> is not true.
/home/aho/Projects/Fluentd/fluentd/test/plugin/test_output_as_buffered_retries.rb:747:in `block (2 levels) in <class:BufferedOutputRetryTest>'
     744<EMAIL_ADDRESS>== 1)
     745<EMAIL_ADDRESS>== 1)
     746:       p "hoge"
  => 747:       assert(@i.log.out.logs.any?{|l| l.include?("[error]: failed to flush the buffer, and hit limit for retries. dropping all chunks in the buffer queue.") && l.include?("retry_times=0") })
     748<EMAIL_ADDRESS>== 0)
     749<EMAIL_ADDRESS>== 1)
     750:       assert(@i.buffer.queue.all?{|c| c.empty? })
=======================================================================================================================


Bug in JavaScript syntax highlighting, with less-than symbol on new line

Issue Type: Bug
const a = 1;
const b = (1
< a + 1);
const c = 1;

Syntax highlighting issues start on line 3: a should be light blue, + should be white, 1 should be light green, closing paren and semicolon should be white, and on line 4: const should be darker blue etc.
If you concat line 3 on to the end of line 2 then it's fine - the syntax highlighting works as expected.
Note: also reproducable on a new install of Insiders with no extensions.
VS Code version: Code 1.31.0 (7c66f58312b48ed8ca4e387ebd9ffe9605332caa, 2019-02-05T22:32:14.164Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 3, 2

Memory (System)
16.00GB (1.00GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.8.0

prettier-vscode
esb
1.8.1

haml
kar
1.1.0

syncing
non
2.1.0

ruby
reb
0.21.0

rails-i18n
sha
0.0.10

vscode-open-in-github
sys
1.10.1

(1 theme extensions excluded)

Does not repro in master.

Great! Thanks for looking at this. Do you have any idea when the fix will be released?


chore: Allow custom JDK version for Dev Server

Thank you for your interest in contributing! For general guidelines, please refer to
the contributing guide.
Before submitting a pull request, please make sure to:

[x] Identify an existing issue to associate
with your proposed change,
or file a new issue.
[x] Describe any implementation plans in the issue and wait for a review from the repository maintainers.

Allow users to allow a pass in the JDK version as part of the DevServer's Run Configurations. This may be different from the JDK specified in JAVA_HOME. By default, this plugin should generally be using the java.specification.version System Property. If needed, customers can use the Run Configuration's JDK version to override.
Fixes #916 🛠

@blakeli0 @suztomo Can you give this PR an initial review? Talked with Etan and we think this approach is simple enough to allow customization from their app.

Few nice-to-fixes.

Setting automerge and will release once checks are green.


Cropped cursors from hyprcursor-util

Hey there, I've been playing around with hyrpcursor-util, converting breeze cursors, and eventually some other major cursor themes, but I've found some of the cursors have cropping. Or there's some other rendering issue that unrelated to the creation.

I'd suspect an issue with xcur2png then. Are the png files in the working theme cropped too?

The cursors are good in the original theme, and in the png files created during the hyprcursor-util extraction, they are only cropped in the final working state. The default cursor, and a lot of other cursors are fine in the working state; it's only cursors that exceed a certain height and/or width that are cropped.

Hey, @v4u6h4n!
Can you check out the meta.hl files nearby cursors and compare the px sizes? I suspect you might've hit the same issue I did here: https://github.com/hyprwm/Hyprland/issues/6517.
I've converted my xcursor theme with hyprcursor-util too, and didn't notice that difference in meta.hl files with actual cursor image sizes.

Hey @Gigas002 yeah sure, I'll have a look, but I can't see a meta.hl file; all I have is a manifest.hl file in the parent directory and a bunch of .hlc files in the hyprcursors subdirectory.

These .hlc files are an archives, you can open them and see the cursor png images and meta file

to be precise, .hlc is just a .zip

likely that hyprcursor-util is extracting them incorrectly from XCursor. X's spec allows for this mismatch and does... some weird stuff there. Anyways, this doesn't seem like a case I want to support. Manual intervention is part of life, util's X -> hc conversion is very approximate anyways. Closing :)


(BSO) Bugged Item breaking =b --names command

🐛 Bug Report
My friend received an item from a mystery box (located at A3 in the screenshot) that appears to be "Chicken Legs" but the item doesn't respond to search commands and when attempting to do =b 4 --names the bot returns "An unexpected error occurred"
When attempting to find the problematic item using =b --text --full we found an item by the name of "WTF-11018"
I believe this is what the item is actually named.
He is unable to sell/drop or trade this item.

When did your friend received that item?

This issue was fixed early today.


some bug fixes

fix all warnings in g++ -Wall -Wextra
fix bug in "inline double rho_pX" function
replaced pow(somethig, 0,5) with sqrt
replaced pow(somethig, 2) with something*something
std::vector::push_back replaced with reserve + emplace_back
some approaches to vectorize calculations in Region5::T_p(double p)
added const modifier in constant methods
some little optimizations

Nice! Thanks for taking the time to put together this PR

@rafuck @ibell OK, for my own edification:

Why is reserve + emplace_back better coding than std::vector::push_back?  What does it achieve?
What's the benefit of making all the base region functions const?  This is a new concept for me.
I get that sing PI = p/p_star, etc. creates another operation and variable and that p_star can be put into the interpolating equation directly, but this is how the IAPWS formulas are written.  Is that small optimization better than maintaining consistency with the IAPWS documents (self-documenting)? Same thing with vectorization in Region5::T_p(double p) - doesn't this make it harder to read and debug for consistency with the IAPWS docs?

About 3) - now this function works 10x faster than initial func on my system.

Ok, about const:

https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#S-const
If the compiler knows that the fields of a class instance are not modified across a const member function call, it doesn't have to reload any fields that it may have kept in registers before the const function call. Yes, on not so deep call tree compiler can poove, that some of object properties will not change, but if we say it explicity we can make compilation process little faster %)

About reserve+emplace vs push_back. It may be faster were size of vector is very big ( http://coliru.stacked-crooked.com/a/83d23c2d0dcee2ff ). But if the vector length is known in advance, why did not immediately allocate memory for it?

Excellent.  Thanks.  I'll read up!

@ibell Is CoolProp really off Python 2.7 now and can now use C++11 conventions?  Is it worth putting the C++11 push_back code back in to add some speed?

@ibell - Bump.

I think that C++11 should be safe. In other projects I have moved to C++20.


EmeraldEnchants2 Not appearing in GUI

1 - Disappears and reappears at random from GUI
2 - When available does not allow for return to main GUI option no back book to click on.
No errors in console
Install EmeraldEnchants2 along with SF and CSLIb. - Restart Server -
1 - Open SF guide or Cheats Look for EmeraldEnchants (not visible).
2 - Open SF guide or Cheats Look for EmeraldEnchants (visible) click to access and then attempt to return to main GUI (no option or return to book provided)

Should be visible in GUI at all times
Should have a back to main GUI feature like other SF features do.
No error log or errors given in console - Pastebin not required
sf versions

1.14.4
vDEV - 637 ( git aa9652c )
vDEV - 78 ( git 199e1b1e )

Additional note I am unable to pull up the EmeraldEnchants GUI section to show the lack of back-book or back option.  There for at this time I can not provide a picture or screen clip of this issue. I will however continue to attempt to have EmerldEnchants show and grab the required screen shot.

Continued in depth testing finds that /sf guide shows the EmeraldEnchants in the GUI. While the use of /sf cheat does not, as noted above.

I am not sure if this additional information will help in diagnosing the issue but I do hope it helps.

Adding this here - but unsure if related or requires a separate ticket. If a separate ticket someone please let me know and I will open one asap.
This is an error log section that I get in the console for EmeraldEnchants
[13:15:55 ERROR]: Could not pass event BlockBreakEvent to EmeraldEnchants vDEV - 5 (git 6adbbb87)
java.lang.IllegalArgumentException: Cannot drop air
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:122) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.EntityItem.setItemStack(EntityItem.java:429) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.EntityItem.(EntityItem.java:41) ~[patched_1.14.4.jar:git-Paper-234]
at org.bukkit.craftbukkit.v1_14_R1.CraftWorld.dropItem(CraftWorld.java:700) ~[patched_1.14.4.jar:git-Paper-234]
at org.bukkit.craftbukkit.v1_14_R1.CraftWorld.dropItemNaturally(CraftWorld.java:717) ~[patched_1.14.4.jar:git-Paper-234]
at me.mrCookieSlime.EmeraldEnchants.listeners.EnchantmentActionListener.onDig(EnchantmentActionListener.java:96) ~[?:?]
at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor661.execute(Unknown Source) ~[?:?]
at org.bukkit.plugin.EventExecutor.lambda$create$1(EventExecutor.java:69) ~[patched_1.14.4.jar:git-Paper-234]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:80) ~[patched_1.14.4.jar:git-Paper-234]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[patched_1.14.4.jar:git-Paper-234]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:545) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PlayerInteractManager.breakBlock(PlayerInteractManager.java:306) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PlayerInteractManager.a(PlayerInteractManager.java:265) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PlayerInteractManager.a(PlayerInteractManager.java:239) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PlayerConnection.a(PlayerConnection.java:1307) ~[?:git-Paper-234]
at net.minecraft.server.v1_14_R1.PacketPlayInBlockDig.a(SourceFile:40) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PacketPlayInBlockDig.a(SourceFile:10) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.PlayerConnectionUtils.lambda$ensureMainThread$0(PlayerConnectionUtils.java:23) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.TickTask.run(SourceFile:18) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeTask(IAsyncTaskHandler.java:136) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeNext(IAsyncTaskHandler.java:109) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.MinecraftServer.aX(MinecraftServer.java:1029) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.MinecraftServer.executeNext(MinecraftServer.java:1022) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.awaitTasks(IAsyncTaskHandler.java:119) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.MinecraftServer.sleepForTick(MinecraftServer.java:1006) ~[patched_1.14.4.jar:git-Paper-234]
at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:929) ~[patched_1.14.4.jar:git-Paper-234]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_202]

EmeraldEnchants is not an Addon to Slimefun, Slimefun simply integrates EE's guide into Slimefun's.
And there isn't a way to cheat in the items from EmeraldEnchants because it doesn't have any, it only adds Enchantments, so it is completely intentional that it doesn't appear in /sf cheat
Make a seperate issue for the error though and use pastebin for reporting errors.


Database: User Repo Index

We need to be able to get a list of repositories for a user:
https://github.com/drone/drone/blob/bolt/datastore/bolt/user.go#L53
This means we need to manually maintain an index that associates repositories to a user. So we store a key/value where the key is the username and the value is an array of keys ([][]byte):
There is code in place that expects this index to exist:
key := []byte(login)
raw := t.Bucket(bucketUserRepos).Get(key)
keys := [][]byte{}
err = decode(raw, &keys)
if err != nil {
	return repos, err
}

The problem is that we are not currently maintaining this index when creating a repository:
https://github.com/drone/drone/blob/bolt/datastore/bolt/repo.go#L45
We should also maintain a reverse index. A list of user keys that have access to a particular repository (in the bucketRepoUsers bucket).

I'm not sure if this is the optimal solution, however, I'm storing the user repositories inside a map inside the user structure. Pro less code than maintaining and index. Cons less ideal for database plugins that are relational. may not be consistent with other index approaches (like repo statuses)
type User struct {
	Login    string `json:"login,omitempty"`
	Token    string `json:"-"`
	Secret   string `json:"-"`
	Name     string `json:"name,omitempty"`
	Email    string `json:"email,omitempty"`
	Gravatar string `json:"gravatar_id,omitempty"`
	Admin    bool   `json:"admin,omitempty"`
	Created  int64  `json:"created_at,omitempty"`
	Updated  int64  `json:"updated_at,omitempty"`

	// Repos contains a list of subscriptions
	// to repositories the user is watching.
+	Repos map[string]struct{} `json:"-"`
}


Adjusting ci_make for community.general > 6.5.0

Collection requirement was pinned to version 6.5.0 as it is the first one to include adjusted make module.
The module no longer checks for presence of the 'command' key in the returned dictionary, as such situation can not arise since version 6.5.0.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

/approve
the dry_run is actually tested in the integration tests via ansible-test - seems to be green, so we're good, apparently.

/lgtm


binary module modifies String.prototype

During attempts to modernize/es6ify the default modules, i came around this: the binary module modifies String.prototype by adding the methods toByteArray and toByteString. I certainly see the convenience, nevertheless it took me quite a while to figure out why these methods are available in RingoJS shell, but not in applications run using the standard method (it's because the binary module is - unnecessarily - loaded in io module, which is a dependency of ringo/term).
Since we're heading towards 3.0 i propose to clean this up (and break backwards-compatibility), and replace it by binary.toByteArray(string) and binary.toByteString(string). An alternative could be to move these two methods into a separate string module or into the globals module (the latter would mean no changes to the java source), though i don't think that's necessary.
Please share your opinions.

This should be located in the binary module and keep the overall count of modules the same. Never touch the String prototype 👍

Never touch any of the standard prototypes ;-)


Update MigrationCommand.php

line 82 : Unless there is a better way to point path to /database/migrations rather than /app/database/migrations
line 90 : points to package view

Thanks @rgrigga Switched to use base_path() instead for ln80. Used 90 as-is.  Thanks for the addition.


Updated link to SDL testcontroller and added entry for AntiMicroX

Any reason in particular for adding AntiMicro ? I believe the SDL version is out of date and I don't think we need to link two of Travis' projects for the same purpose.

Mostly because it's the one I've heard people talk about most when discussing controller mapping software. Feel free to disregard that portion of the patch.


Add support for multi-class (multi label) training

Possibly would need separate model.
https://sbert.net/docs/sentence_transformer/training_overview.html


Optimize String.repeat() (3.x)

This backports the optimization done in master.
Testing project: test_string_repeat_3.x.zip
Requires readding and exposing a repeat_slow() method with the old code to work.
Benchmark
OS: Fedora 36
CPU: Intel Core i7-6700K @ 4.4 GHz
Compiler: Clang 14.0.5
Performed on an editor debug build, but a release build should also benefit significantly.

Type
Before
After

len1 x4
5.967ms
5.267ms

len1 x40
17.193ms
5.445ms

len1 x400
126.042ms
5.735ms

len1 x4000
1213.496ms
7.698ms

len5 x4
10.426ms
5.284ms

len5 x40
61.751ms
5.6ms

len5 x400
569.243ms
6.278ms

len45 x4
50.869ms
5.38ms

len45 x40
462.394ms
6.103ms

len250 x4
266.615ms
5.668ms

Thanks!


Make a flutter gallery for Official plugins

The official flutter plugins are really useful.
Would it not be smart to have a flutter gallery plugins app tmso developers can get firsthand experience and the plugin team get more cross device testing ??

cc @mit-mit

@gedw99 do you mean on flutter.io, or on Pub? Or really on either, just make these more showcased?

@mit-mit I mean an app that runs on Android and IOS. And a repo of the source code of that app.
Come to think of it, It could be easier to add to the existing Flutter Gallery too if thats easier, which i think it is.
Like a new Nav Section called "Plugins", and then a Page for each once showing a typical use case for using each plugin.
But it might be easier for the Flutter team to decouple and have a separate app too.
Either is good :) JUst as long as it gets in people faces and you can collect success / failure data on lots of mobile !! Very useful to get a matrix of what plugins work and dont work on which phones.
But also good because it helps devs see how thee plugins really help solve problems too.

Did you notice that each plugin has a small example app located inside it?

yes of course :)
But this does not help my concerns.
Is there data somewhere that shows a regression test matrix that cross references the official plugins to mobiles ( and their environment) ??  Is it public ?

Similar to #16736

@gedw99 I put together an example of using the Flutter Gallery with SauceLabs to run the instrumentation test provided with Flutter Gallery (android only). You can see a video (and logs) of it running on SauceLabs here:
https://app.saucelabs.com/tests/0953cb723204478ab206b7b8f0097554/watch
and source here:
https://travis-ci.com/mmcc007/test_flutter_saucelabs/builds/102025976
This can be used as a starting point to maintain a test matrix on various real devices.
It seems more work is needed to make integration testing on real devices in the cloud a practical option.
See further details on #7087

Thank you.
This is a great start @mmcc007
Currently using internal CI and rolling it out to internal people for beta testing for now to try to get a device matrix going. All manual still. No saucelabs automation for the tests yet
Fledge also looks useful to help standardise the provisioning aspects and CI.

Thank you.
This is a great start @mmcc007
Currently using internal CI and rolling it out to internal people for beta testing for now to try to get a device matrix going. All manual still. No saucelabs automation for the tests yet
Fledge also looks useful to help standardise the provisioning aspects and CI.

@gedw99 Glad you found the above experiments and fledge helpful.
If seems there is no CI/CD framework or reference for mobile development in general. Every dev has to kinda invent there own. Fledge is an attempt to accumulate some best practices for Flutter and use as a starting reference point (as you mention).
I have made a start, as mentioned above, with SauceLabs, and Genymotion.
BTW: I also made a start on access to AWS Device Farm (currently android only):
https://github.com/mmcc007/sylph
(the Flutter team recently made a fix that should make testing in device farms easier #28662)
Mobile testing (independent of dev framework) in a CI/CD context depends on a number of things including the suitability of the services available (Saucelabs, Genymotion, AWS Device Farm, alternatives, etc..).
There seems to be room for improvement.

I'm going to close this, as I don't think there's a compelling benefit to it that would outweigh the work of maintaining it.
In particular:

Is there data somewhere that shows a regression test matrix that cross references the official plugins to mobiles ( and their environment) ??

A gallery app that almost nobody would have any reason to run wouldn't give us meaningful data here, not are most issues we see in practice things that would show up in metrics. Real-world issues are gathered via real apps.
And:

This can be used as a starting point to maintain a test matrix on various real devices.

Automated tests on real devices are done via the example apps that we already have.


Subcommands with hyphen not working with autocompletion

When generating autocompletion script, subcommands with hyphen are stripped of it and are no longer matched for autocompletion.
As discussed in #178 , contained example can be found in comment.

This is now fixed in master. Please verify if you have the chance.
Thanks for the bug report!


Getting started page

When starting Monokle without a folder or cluster configured we should show a welcome page with options to

select an existing folder/project
create a new folder / resource
configure a context/kubeconfig

with links to documentation

Some new flows here... we'll want to talk through at the UX call.

Figma:
https://www.figma.com/file/h2p93QZ0Q6toI4wb4Q37EL/%23619-Getting-started-page?node-id=3%3A2050


Use zero-based numeric indices for much faster instrumented code

Closes #21. Makes instrumented code run much faster by making V8 use “fast” mode when accessing stats objects, which wasn't triggered before because of non-0-based indices.
Here's are the timings from mapbox-gl-js render suite:

no coverage: 1m 6s

nyc current: 4m 37s

nyc updated: 1m 20s

[ ] update tests

cc @bcoe @mramato

CLA is valid!

@bcoe Travis passes, so this ready to go.

@bcoe @gotwarlost anything I could help with to get this merged? A 15x speedup seems like something that's worth prioritizing. :)

this is amazing! ❤

@dmitriiabramov @mourner I see no reason not to land this; but since I haven't heard back from @gotwarlost, I will be testing quite throughly; I'll start by publishing this to a next tag.

@mourner I've published these changes to the next tag on npm, if you'd like to help test a bit:
npm cache clear; npm i istanbul-lib-instrument@next

Oh this is awesome! Is anyone interested in trying this out on https://github.com/facebook/jest itself? :) (run jest with --coverage).

Thanks @mourner, I would love to try this out to confirm my issue in https://github.com/gotwarlost/istanbul/issues/556 is gone.  Unfortunately I currently rely on https://github.com/karma-runner/karma-coverage which hasn't updated to this library yet (but I'm hoping it happens soon or I can open a PR to do it for them).

@cpojer i tested it on jest itself, but we don't really instrument much code, so there isn't much we can optimize :)
although it seems to run 1 second faster, 28s on average instead of 29s

@bcoe thanks! The new version works perfectly in our project:

no coverage — test.sh takes 4m53s
nyc latest — test.sh takes 11m22s
nyc new — test.sh takes 5m56s

So sorry about my lack of response (this is getting old :( )
I see no reason off the top of my head why this should not be merged. 👍

ah, it is merged already. Great.


feat: twitch badges

i added commit: feat in progress: auto saving links from twtich

Please separate the Link logic into different PR. :)

I'm sorry about that. That's exactly what I wanted to do. I reverted the changes


Add option to shift click examine or do nothing for items in inventory

We're already able to shift click use, drop, wear, possibly others that I can't think of. It would be nice to be able to shift click either examine or do nothing on items in the inventory. This would be useful for actions such as cleaning herbs where shift click anti drag is used, so that herbs aren't dropped or chosen to be used on another item.

This is already possible via the Menu Entry Swapper plugin's Customizable shift-click feature

Attempting to change shift click option on an item e.g. molten glass to examine simply changes it back to drop, rather than examine or no option as I would like it to. Is there something else that I need to do? I have customisable shift click enabled.


Enabling TLS detects stack overflow in a few tests

Because TLS stores critical thread information at the top of each stack, overflows in subsequent stacks will write over that and cause general chaos. This means that enabling thread local storage will cause some tests to fail simply because they overflow their stacks in a way which was not detected without TLS enabled.
These three patches increase stack sizes in tests (probably by way more than necessary) to make them work when thread local storage is enabled.

ok, as #53590 shows, this series now gets all tests passing with TLS enabled, so this series should actually be ready to merge.


Translation issue in promotion.

Framework Version: 0.1.6

Framework Version: 0.1.8

Fixed

Framework version v 0.1.8
getting same issue  while selecting condition for those columns where we need to enter value when trying to edit cart rule

fixed in new cart rule


#15 Requirement of Proper Input Handling in Login/Register form

Description

Added a small warning using html and css whenver user enter a password and cursor points on password input box
validated the backend form validation used special character regex expression to prevent symbols that lead to client side injection attacks such as xss and other attacks.
Related Issues

None
Type of PR

[X] (Security patch update feature with proper exception and error form validation)

Screenshots / videos (if applicable)

Wrote this Exception handling  to ensure proper validation

Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
Atfirst, Tried using the alert method but it was conflicting the google app authenticator then switched to this method of warning based change with change in the signup form validation code

@pand-coder please update the branch


Nested items drag and drop

Hello there,
I'm doing some research, before starting to use react dnd,
i have the following structure

task1
task2
a.task 3
b.task 4
c.task 5
task 6
a.task 7
task 8

i want to be able to drag any one of those elements for example
taking task 3 and nest it under 8
or take task 4 and put it above 1
or even moving task 2 with all the tasks bellow.
like this

task2
a.task 3
b.task 4
c.task 5
task1
task 6
a.task 7
task 8

my questions is if this is possible, and if so how would you try to solve such problem

I am still learning this library my self and still have some bugs with my code but I am doing something similar and it is working. I have multiple containers and multiple positions with in those containers that "cards" can land. It works but running into an issue with one card replacing another.  If you looks at the Chess example they have it is overly complicated but if you strip it back I was able to get it to work.


Reading OpenDAP

I want to apologize in advance if this is not the right place to ask, if I'm asking a "silly" question or if this question has been asked hundreds of times.
I have been porting all of my Python work into Rust with a considerably high degree of success.
Currently, when using netCDF4.Dataset in Python, one can directly open an DAP connection directly and just request the variables needed.
The netcdf docs for Rust mention the capability of opening a DAP connection directly, but I haven't seen examples, and I have tried doing it in Rust with no success. It seems like I'd need to download the netcdf files and open them locally. Is there a way to mimic the "streaming" behavior of the netCDF4.Dataset using georust/netcdf ?
Thanks again!
-J.

It should work like in python, but it requires the dap support to be included in the library linked (netcdf-c). It should be possible to force this by adding netcdf-sys = { version="*', features =["dap"]} to your Cargo.toml

Hi @mulimoen
I have modified my Cargo.toml as follows:
cargo add netcdf --features static,ndarray

cargo add netcdf-sys --features dap,static,memio

But now I get:
 --- stderr
 thread 'main' panicked at /home/jreniel/.cargo/registry/src/index.crates.io-6f17d22bba15001f/netcdf-sys-0.6.0/build.rs:135:13:
 DAP requested but not found in this installation of netCDF
 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Any help is appreciated,
Thanks!

Could you check if cargo add netcdf-src --features dap enables dap support? If it does I can fix this in a new (patch) release of netcdf-sys

I fixed an incompatability for the static and dap features. Instead of the above, could you run cargo update netcdf-sys and try again?

That fixed it, thanks a lot!
You guys are rock stars!

OK, I reopened because although it compiled, I'm still having one issue. This works in Python:
from netCDF4 import Dataset
nc = Dataset("https://icdc.cen.uni-hamburg.de/thredds/dodsC/ftpthredds/hamtide/k1.hamtide11a.nc")

But the same URL won't work with the Rust NetCDF API:
use netcdf;
let url = "https://icdc.cen.uni-hamburg.de/thredds/dodsC/ftpthredds/hamtide/k1.hamtide11a.nc";
let nc = netcdf::open(&url.to_string());
dbg!(&nc);

Results in:
[src/bctides/src/hamtide.rs:56:9] &url.to_string() = "https://icdc.cen.uni-hamburg.de/thredds/dodsC/ftpthredds/hamtide/k1.hamtide11a.nc"
Error:curl error: SSL peer certificate or SSH remote key was not OK
curl error details:
Warning:oc_open: Could not read url
[src/bctides/src/hamtide.rs:58:9] &nc = Err(
    Netcdf(
        -68,
    ),
)

Any ideas on how to fix this?
Thanks!

Bit of a journey finding out what needed to be fixed here. The basic problem is curl not being able to find the certificate.
The solution is to tell it where to find them. This functionality was fixed in netcdf-c, but after the release of 4.9.2, which means another patch was needed, and bubbling this up to the top crate.

New versions are up, you might want to update using cargo update netcdf to pull in the fixes

Thanks for checking this, but unfortunately, even after doing cargo clean and double checking my Cargo.toml:
netcdf = { version = "0.9.2", features = ["static", "ndarray"] }
netcdf-sys = { version = "0.6.2", features = ["dap", "static", "memio"] }

I still get the same error:
[src/bctides/src/hamtide.rs:54:9] &this_url.to_string() = "https://icdc.cen.uni-hamburg.de/thredds/dodsC/ftpthredds/hamtide/q1.hamtide11a.nc"
Error:curl error: SSL peer certificate or SSH remote key was not OK
curl error details:
Warning:oc_open: Could not read url
[src/bctides/src/hamtide.rs:56:9] &nc = Err(
    Netcdf(
        -68,
    ),
)

Did you call netcdf::rc::set("HTTP.SSL.CAPATH", "/etc/ssl/certs/").unwrap(); before opening the file?

Did you call netcdf::rc::set("HTTP.SSL.CAPATH", "/etc/ssl/certs/").unwrap(); before opening the file?

Ah, that explains it! I added it and it worked as expected.
Thanks!
Note: I did read https://github.com/georust/netcdf/pull/144 but I misunderstood that this call had been added as an explicit call inside the library.

Happy to hear it works for you. I find it strange that the curl library does not use a sane default, I don't know what e.g. ncdump does to handle this.

I found https://www.github.com/alexcrichton/curl-rust/pull/446 which describes why a default path is not set for the static build.

Cool, thanks for the refs!
I hid the call on my struct constructor and made it conditional for #[cfg(unix)] and if it's not already set. This should be sufficient for the time being.
Rust is awesome! The fact that no one else had caught this means Rust is still growing, but the fact that it was fixed so quickly means it's growing fast!
Cheers!


Add license to packages

I see the license for Atomico in general is MIT, is it the same license for these packages? Could you add the license field on the packages?

Hi @mig8447 , thank you for reminding me. My goal is to ensure that all code related to the atomico package and its sub-packages under the @atomico organization is and always will be open source under the MIT license. Ignore the configuration in some repositories like this one. In the attached release, all packages from this repository are released with the MIT license for NPM and the license is declared in the repository.
https://github.com/atomicojs/hooks/releases/tag/v4.1.2
If you find any repo of the organization without these changes, let me know  or if you generate a PR that references this, I will be happy to approve it.
Greetings

Thank you for your quick reply

Thank you for your quick reply, I consider this closed


Add pagination to blog posts

Implemented pagination in the blog to display 3 posts per page. This change allows users to navigate through posts using a pagination control at the bottom of the page. The pagination follows the pattern: << < 1 2 3 ... n > >>, enabling navigation to the first, previous, specific, next, or last pages. This improves user experience by preventing long scrolls on pages with many posts.
Issue ID: FLAS-33

I've reviewed your code. Please address the comments. Thanks!


Unable to Bypass on URLs with Permanent Redirection(301)

What I did:
I got cookies and useragent that was provided by the module, then I used that information to enter the home page which results in complete bypass with status code 200.
HOWEVER, when I used the same information on a page that has 301 redirection, I got the interstitial page from cloudflare.
I wonder what's the root cause for this.
Kindly assist.

Do you have an example this could be tested against ?

I believe this should have been fixed in one of the recent updates.


Reth execution client reports of invalid JWT at init

Expected behavior
The execution clients shouldn't report any issues and should work normally.
Actual behavior
Running a Lisk node with op-reth execution client reports of Invalid JWT error at init but only once, as seen in the logs below.
The node however functions normally, syncs with the L2 network and reports no more issues later.
sameer@Sameers-MBP lisk-node % docker logs -f lisk-node-execution-1

------------------------------------------------------------------------------------------------------------
Starting op-reth execution node under '/data'...
------------------------------------------------------------------------------------------------------------

ts=2024-07-24T12:37:16.369169668Z level=info target=reth::cli message="Initialized tracing, debug log directory: /root/.cache/reth/logs/4202"
ts=2024-07-24T12:37:16.392493793Z level=info target=reth::cli message="Starting reth" version="\"1.0.3 (390f30aa)\""
...
...
ts=2024-07-24T12:37:21.429181337Z level=error target=engine::jwt-validator message="Invalid JWT: Authorization header is missing or invalid"
...

Steps to reproduce
CLIENT=reth docker compose up --build --detach

Which version(s) does this affect? (Environment, OS, etc...)
PR #27  (branch: 24-unable-to-build-reth-images, commit: 2166361e2bb96c3e0bde198112176dc6b880b299)
Additional Information
Both the op-node and the execution (op-reth in this case) client already share the same JWT token and the container has everything set properly for the jwtsecret (--authrpc.jwtsecret) to be used by the reth client.

manually added --l2.jwt-secret=/jwt.hex and --authrpc.jwtsecret=/jwt.hex to both entrypoints, mounted ./jwt.hex:/jwt.hex as volumes in docker-compose.yml , then booted op-node first, later reth and it worked


FileNotFoundError: No such file or directory: 'tmp.wav'

I am getting this error. Please help.

did you resolve this problem???

@oualido1212 No...

Dear @shubhi8256 and @oualido1212 ,
Did you read the blogposts linked in the README?
Thanks.

@Hrant-Khachatrian can you help me? how to create the tmp.wav file ?

Dear @oualido1212 ,
If you read the blogposts, you will see that the data for this code was not released by the organizers of the contest. So there are no publicly available wav files. This is just a code.

Was someone able to fix it if yes can you tell me how?


Missing Information

I guess, It is better to inform, after upgrading the kuberenetes cluster, user need to manually deploy the applications

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

ID: a14a3f84-28b4-0a2a-4da7-47cb4d66689f
Version Independent ID: d1ffdd88-ab9a-9f55-8689-526e7415b326
Content: Kubernetes on Azure tutorial - Upgrade a cluster
Content Source: articles/aks/tutorial-kubernetes-upgrade-cluster.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@AdimUser Hope you have self resolved and hence closed the issue. Thanks for the update.


Messages are getting removed from the queue when there are errors in the program

In the JMS server side, try to retrieve an integer value from the following method
system:println("Message received - " + messages:getStringValue(m,"states"));
Related to : https://github.com/ballerinalang/ballerina/issues/2112
This will throw an error
Check the queue

Issue
The messages will be removed even without acknowledging

@dilinisg - When there is a error, we call the jms session.recover(). As per the jms API, http://docs.oracle.com/javaee/7/api/javax/jms/Session.html#recover--, it should try to re-deliver the message. But seems like MB is adding it to the dead-letter channel. In ActiveMq, it tries to re-deliver the message again for certain number of times. @pamod @ramith - Can you comment on this behavior please?

The correct behavior in MB also should be moving the message to DLC after exceeding the maximum redelivery count (which is 10 by default).

This issue is not there in MB 3.2 Alpha.

@Megala21 JMS recovery() would result it rejecting all un-acked messages for the given session.
In the implementation of MB and as per the AMQP spec. The messages would be sent from the broker to the client as a batch. i.e if there're 10 messages in the queue all the 10 messages will be sent as once to the client as a batch. AMQP defines this as a mechanism to improve performance.
As a result all the messages sent to the client, despite it being received by the high-level API or not would be rejected resulting in all the messages in the batch to move to the DLC after the number of retries are breached.
Unfortunately JMS doesn't define a way to recover/rollback individual messages. We're discussing alternative ways to identify ways to distinguish the messages which are received by the API and the once which have not. So that we could maintain the retry count only for the messages which are received, so that the messages which are actually received by the API are sent to the DLC.
Will update  https://wso2.org/jira/browse/MB-1896 with the progress.

Created https://github.com/ballerinalang/connector-jms/issues/22 to track this


Screenshare support

We need to add screenshare support into electron.  Unfortunately we can't just drop the chrome extension in, but we should be able to port it over.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34107172-screenshare-support?utm_campaign=plugin&utm_content=tracker%2F19476668&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F19476668&utm_medium=issues&utm_source=github).

Duplicate of https://github.com/RocketChat/Rocket.Chat.Electron/issues/65


Support for Storybook 8

This addon is not yet compatible with Storybook 8

Any update on Storybook 8 support?

Sorry, I'm behind again. I'm going to work on this soon.
I'm having trouble doing it from the laptop I'm on
due to my setup with yarn and things (didn't look to far into it yet).
For reference (my own mainly):
Here's the branch https://github.com/Sirrine-Jonathan/storybook-source-link/tree/sb-v8
Docs for the migration: https://storybook.js.org/docs/addons/addon-migration-guide


Add blog posts to website

📚 Description
Replace this text with a short description of your feature/bugfix.
🔖 Changes

List individual changes in more detail

✅ To-do list

[x] Make sure that all the pipeline passes
[x] Make sure you have updated the documentation to reflect the changes or new features
[x] Make sure to update the CHANGELOG.md to reflect the new feature or fix

Keep in mind that since the web version releases are tied to the bashunit releases, to add new articles a release would be needed.
This has a solution since the articles could be read from main branch and rendered with the latest branch code 🌈 , but adding this behavior would add too much complexity to the PR.
Since the blog was in a functional state I decided to make the PR and will add this functionality later on.


Add doc comments for service collection extension methods

At least these classes and methods:

https://github.com/aspnet/Options/blob/dev/src/Microsoft.Extensions.Options/OptionsServiceCollectionExtensions.cs
https://github.com/aspnet/Options/blob/dev/src/Microsoft.Extensions.Options.ConfigurationExtensions/OptionsServiceCollectionExtensions.cs

Recommend following the general pattern we've used in other places, e.g. https://github.com/aspnet/Session/blob/dev/src/Microsoft.AspNetCore.Session/SessionServiceCollectionExtensions.cs

b4a9c2014c8eb567738cd170a3bee3fea7bfadc7


[Tracker]: Edge Deployments

Description
Investigate Edge deployments
Target Branch
f/edge
Requirements

User provides location for Model stored in S3 or Git
Kickoff the pipeline to build -> test -> push model in immutable container

Itemized UX Issues

https://github.com/opendatahub-io/odh-dashboard/issues/1811

Itemized Dev Issues
Foundation

https://github.com/opendatahub-io/odh-dashboard/issues/1933
https://github.com/opendatahub-io/odh-dashboard/issues/2061
https://github.com/opendatahub-io/odh-dashboard/issues/2251
https://github.com/opendatahub-io/odh-dashboard/issues/2247

Model Registry page

https://github.com/opendatahub-io/odh-dashboard/issues/2062
https://github.com/opendatahub-io/odh-dashboard/issues/2064

ML Ops Pipelines page

https://github.com/opendatahub-io/odh-dashboard/issues/2248
https://github.com/opendatahub-io/odh-dashboard/issues/2250

Related artifacts
No response

migrated issue tracked in jira
ref: https://issues.redhat.com/browse/RHOAIENG-216?page=com.docminer.jira.issue-links%3Acom.kintosoft.jira.links.tab-panel
/close


add go mod support

@zshi-redhat Thank for this PR. Do we have to continue support the vendoring or this is for a transitioning period?

@zshi-redhat Thank for this PR. Do we have to continue support the vendoring or this is for a transitioning period?

For a transitioning period.
This is mainly for backward compatibility support, if user

doesn't use new go versions(1.11 or 1.12),
GO111MODUELS is not enabled,
no internet connection
Then vendor pkgs are required to built sriovdp binary.

I think we will eventually remove all the vendor pkgs in the repo(it can be easily re-generated via go module),wdyt?

Ok, make sense.
So, can we remove the glide dependency and use "go mod -v" to generate vendor where needed or do we will still need glide for older Go? Just so that a developer do not need to update glide files any longer.

Ok, make sense.
So, can we remove the glide dependency and use "go mod -v" to generate vendor where needed or do we will still need glide for older Go? Just so that a developer do not need to update glide files any longer.

Yes, we can remove glide dependency.
I didn't try with older go version, but I think for developer who needs to update package dependency, we can enforce to use go module instead of glide, then remove all the glide files.
For user who wants to build sriovdp binary locally, we can keep vendor pkg there so they don't need to self generate vendor pkgs which requires new go versions.


Create signed request (signature)

Feature request for Cloudinary Go SDK
…(If your feature is for other SDKs, please request them in the relevant repo)
Explain your use case

I need to generate a signature on the backend for the frontend to create a signed request

Describe the problem you’re trying to solve

https://cloudinary.com/documentation/upload_images#generating_authentication_signatures

Do you have a proposed solution?

The python lib exposes a similar method: https://github.com/cloudinary/pycloudinary/blob/9950c7e8af5036e4129f2c8f11e62e98163ee8de/cloudinary/utils.py#L577-L580

After more digging of the code I found this:
https://github.com/cloudinary/cloudinary-go/blob/320ac57d59a12af4829096dd185ec0dbd7f8929b/api/api.go#L211-L224
Can I simply return that string to the frontend for them to use on a signed upload request?

Hi Danny,
Yes, the output from that function should be a valid signature for a signed upload, and passing the signature string from the backend to the frontend is the same method we recommend for similar cases, like using our Javascript Upload Widget with signed upload where the signature comes from some backend code.
That said, the parameters sent on the API call must match the parameters used to create the signature, and the highlighted function is also setting the timestamp that's used on the request, so you need to pass the same timestamp to your frontend also and use it on the API call ( e.g. https://github.com/cloudinary/cloudinary-go/blob/3ea19ca02dff04344198e0b53091935edbc6d65b/api/uploader/upload.go#L103 where the timestamp used in teh signing function is added to the API call)
If there's any problem with the signature you send on an API call, note that our error response shows the string we signed when checking the provided signature, so it's often easy to see why the signature didn't match
Thanks,
Stephen


Memory leak with unused cursor buildup

Running the queries on a loop leads to an obvious memory buildup here with: https://github.com/dodopizza/sqlalchemy-kusto/blob/main/sqlalchemy_kusto/dbapi.py#L109

Creating cursors in a loop is a bad idea. You allocate a new object each time when you call the method anyway. We save it to the array to be able to track all the allocated cursors and be able to close them when the connection will be closed. I believe the right way to solve this is to allocate a single cursor and reuse it in a loop.
I agree that the API may be a little bit confusing, but I do not know an easy solution for this. Probably, having a cursor pool of limited size could be an option, but it leads us to add complexity on a library configuration level. I do not think we need to follow this path at the moment.

why isn't the cursor just 1:1 with the connection?