Pagination.El not making attribute with React

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: EXACT RELEASE VERSION OR COMMIT HASH, HERE.

Live Link or JSFiddle/Codepen or website with issue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

When I assign
pagination = {{
     el: 'swiper-pagination-timeline-page',
}}

inside my react component the pagination div disappears. When I specify the element the whole thing doesn't even render. Please see my component code at the end of thread:
I feel like a need better clarification on this:

Expected Behavior
EXPLAIN WHAT IS TO BE EXPECTED, HERE.
I am expecting to see the element render with the class .swiper-pagination-timeline-page assigned to it.
Actual Behavior
EXPLAIN WHAT IS ACTUALLY HAPPENING, HERE.
when I assign this attribute it does not render the pagination element.
P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.
Before you open an issue, please check if a similar issue already exists or has been closed before.
Component Code
import SwiperCore, { Navigation, Pagination, Scrollbar, A11y, Lazy} from 'swiper';
import { Swiper, SwiperSlide } from 'swiper/react';
import React from 'react';

SwiperCore.use([Navigation, Pagination, Scrollbar, A11y, Lazy]);

export default class Timeline extends React.Component {

    constructor(props) {
        super(props);

    }

    componentDidMount() { }

    componentWillMount() { }

    render() {
        return (
            <>
                <Swiper
                loadPrevNext = {1}
                preloadImages = {0}
                lazy = {1}
                spaceBetween = {0}
                slidesPerView = {1}
                direction = 'vertical'
                pagination = {{
                    clickable: true,
                    // el: 'swiper-pagination',
                    type: 'bullets',
                    bulletElement: 'span',
                    bulletClass: 'timeline-icon',
                    bulletActiveClass: 'timeline-icon-active',
                    renderBullet: function (index, className) {
                        return '' + (index + 1)  + ' icon ' + '';
                    }
                }}
                >
                    <SwiperSlide>
                            1
                    </SwiperSlide>
                    <SwiperSlide>
                            2
                    </SwiperSlide>
                    <SwiperSlide>
                            3
                    </SwiperSlide>
                    <SwiperSlide>
                            4
                    </SwiperSlide>
                    <SwiperSlide>
                            5
                    </SwiperSlide>
                    <SwiperSlide>
                            6
                    </SwiperSlide>
                    <SwiperSlide>
                            7
                    </SwiperSlide>
                    <SwiperSlide>
                            8
                    </SwiperSlide>
                    <SwiperSlide>
                            9
                    </SwiperSlide>
                    <SwiperSlide>
                            10
                    </SwiperSlide>
                </Swiper>

            </>
        );
    }
}

Also, it would be great to see how you can use namespaces to uniquely name a swiper instance. I am looking to recreate this nested swiper demo:
https://react-id-swiper.ashernguyen.site/example/nested-swipers

pagination.el is for specifying pagination element, not for adding custom classes. You need to add:

inside of Swiper

that worked great thank you

great, this atleast shows it in the DOM, but now its not rendering anything inside the new

I feel this could be because next.js automatically assigns a custom jsx className like so:

There is very little documentation on this feature (the same for customizing Navigation). Are there any working samples?

I've achieved it re-writing the given classes, see: https://omarapollo.com/

I have also ran into this issue, what was the solution? @jonrrivera

Same here, as soon as i add el property, pagination is not working anymore.


"Enable the custom threat intelligence application" text no longer relevant

Custom TI app was removed so that text is no longer relevant.

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

ID: c443d8dd-5e54-c255-d3fd-6a60b5188ee9
Version Independent ID: 7c1c9819-e92a-d364-ee79-7129a73912ce
Content: Overview of management and APIs - Microsoft 365 Security
Content Source: windows/security/threat-protection/microsoft-defender-atp/management-apis.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@officedocsbot assign @jvsam

Thanks for the feedback @iadgovuser1. Much appreciated!
Hi @mjcaparas, can you please confirm or provide some insights on this? Thank you so much!
@officedocsbot assign @mjcaparas

content updated


Add PullerComponent to Mothroaches

About the PR

MOFF! MOFF PULL THINGS!!!
Why / Balance

The risk of ghostrole shitters on axolotl is pretty low. Might as well give them more liberties to have fun.
Technical details

Add PullerComponent to MobMothroach. Add id to PullerComponent.NeedsHands.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

none

i agree with myself


[REA-45] Stream pausing

This is a rewrite of #1. The goal is to be able to pause streams and partitions, and resume them when a publication occurs via a Liftbridge gRPC call.

PR created upstream, so closing this.
See https://github.com/liftbridge-io/liftbridge/pull/170, https://github.com/liftbridge-io/liftbridge-api/pull/18, and https://github.com/liftbridge-io/go-liftbridge/pull/49.


Allow actions to be used instead of states in ForEach state and Paral…

…lel State branches
Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[x] Specification
[ ] Examples
[x] Schema
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] Website
[ ] SDK
[ ] TCK
[ ] Other

What this PR does / why we need it:
Issue: https://github.com/cncf/wg-serverless-workflow/issues/39
Allows ForEach state and branches in Parallel state to chose to define either states or actions to be executed.
Special notes for reviewers:
Additional information (if needed):

This raises the question of why the language needs both states and actions. Wouldn't one of them be enough?

@falko I think there are use causes for both. There is an issue: https://github.com/cncf/wg-serverless-workflow/issues/40 which is like "part two" of this work to simplify the markup. It is similar scenario where you can either express the work that needs to be done via actions (list of operation states), or more complex scenario where control flow logic is needed.
Allowing actions is just a simplification for users (they do not have to define list of operation states). Both cases can be seen as equivalent.


WIP: Added ChoiceMultiplePreference

Just want to get some implementation feedback.
I am not sure merging with a seperator is the best idea for string preferences. Would it be a better alternative to just use json-serializing?

Hey, what is the state of this? I actually was about to request this feature today, so what can we do to get this into shape and released :)?

Closing as stale/too old. If someone wants to work on thisi again, you're welcome :)


Reporting a vulnerability

Hello!
I hope you are doing well!
We are a security research team. Our tool automatically detected a vulnerability in this repository. We want to disclose it responsibly.  GitHub has a feature called Private vulnerability reporting, which enables security research to privately disclose a vulnerability. Unfortunately, it is not enabled for this repository.
Can you enable it, so that we can report it?
Thanks in advance!
PS: you can read about how to enable private vulnerability reporting here: https://docs.github.com/en/code-security/security-advisories/repository-security-advisories/configuring-private-vulnerability-reporting-for-a-repository

Thanks for pointing out the private reporting link, we'll look into that.
In the mean time, our "new issue" page points to https://github.com/liquibase/liquibase/security/policy with the<EMAIL_ADDRESS>email you can use for the time being.

Hello @igibek ! Private vulnerability reporting has been enabled, thanks for  pointing out this feature.


Add Flashlight Tool

A tool for developing plugins for Spotlight based on Flashlight.

Thank you for the contribution. It needed some fixes, so they were made in c7ba93cb6cdf37cdc4b33a335610021771ddabe2. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Hey Vítor, I want to thank you so much for the help/feedback and advice you have shared with me in the past and now. Man, you rock!

Thank you for the kind words, and for the contribution.


Create User groups for admins

First User group should be moonshot collective

For new users to use tip.party, it must be possible for them to 'sign up' and right now only existing admins can enroll new users.
To make it possible to test with arbitrary external groups, we must add a ~'make me admin' feature.
Proposal:
New users may pay 0.1 eth (or 5GTC somehow? gas tho?) to become admin, and the balance (less contract and gas fees) is sent to moonshotcoordinator.eth
(Is it possible to make it completely free and permissionless, and then add a permissioning and value regenerating (for MC and others) layer on top?)

This feature has been implemented in the updated Tip.Party


Automate task management

Auto-assign the Developer Relations projects when a new issue is created. Same for documentation, PR & issues. Small details that will save many clicks...
I also have other things I wanted to automate, will add here when I rethink about them!

We use project board only for yearly planning, so it's not necessary anymore.


Revert https://github.com/atlassian-labs/react-resource-router/pull/98

What's the reason to revert #98 ?

🙄 that would be a breaking change. What is actually causes this change?


fix automatic versioning

Currently, automatic versioning only jumps by minor version regardless of the git history.
Maybe smth needs to be done to standard-version configuration

@mixvar, standard-version doesn't bump the version properly as we don't currently have a major version.
see this issue for more details https://github.com/conventional-changelog/standard-version/issues/429
I've set up a local branch with a major version set to 1, and feat/fix/BREAKING CHANGE commits properly bump major/minor/patch versions

awesome, closing the issue


Menus not responsive on Windows 10

So I created a fresh Windows 10 VM today to try out the Beta.  However, the application is sluggish to open which isn't a big deal but I never noticed it before.  The problem is that the menus are not working.  They will appear on clicking on the menu bar, but selecting a menu item does nothing.  If I try to open a project folder from the explorer VSCode opens but the folder does not.  I have restarted and reinstalled with a fresh download of the installer.  Am I the only one who is seeing this?

So I fixed the problem.  Installing the Windows 10 November update was the solution.  However, I also tried installing on an existing VM that does not have the November update installed so I'll leave it up to the MS folks to leave this issue open or close it.

Ok thanks for the update, will close for now.


Support for better state checking of Service stuff

There may be a need that for Service dependency we will need to check the health of objects that is selected by Service (service selector). Which will basically mean that we will replace internally dependency on Service by dependency on all objects that are selected by Service.
It will be much easier for user to depend on mysql service than on galera/mariadb components itself.

Should we make this implicit? I think I would rather make user define his graph in a way that explicitly shows the service underlying objects as parents.

we can't and shouldn't make it implicit.
When you deploy some app that for example depends on mysql, you shouldn't need changing graph when you change single mysql to galera cluster. The same for already existing stuff.
We may make this optional, or non default behaviour obviously.

@pigmej you need to change the graph to adjust it from single mysql to galera anyway. So I am still opting for setting the dependencies by user explicitly on underlying pods, in which case the need for checking state of Service is.
Another reason to do it this way is that  is no notion of Service state in Kubernetes core (AFAIK).

On the other hand, it might come in handy!


Unable to prepare setup for platform frigate.camera: Platform not found

Hi,
frigate is up & running for month, not much change, still working great. However, I'm having trouble with the custom_compontent after upgrading HomeAssistant to 2021.5.5
Bug description
The integration turns up in Settings -> Integration, activation is successful. However, the homeassistant log shows that the frigate.* sensors are not initalized, also not when restarting HomeAssistant.
I found zero google hits on "Unable to prepare setup for platform frigate.camera: Platform not found" and assume its a configuration issue on my end.
However, i:

remove the integration in WebUI,
restarted the homeassistant container
cloned latest git
deleted /config/custom_component/frigate
moved the new customer_component/frigate dir to where it belongs
ensured there is no other place in the container with a frigate integration dir

2021-06-22 19:57:40 INFO (MainThread) [custom_components.frigate]
-------------------------------------------------------------------
Frigate
Version: 0.0.1
This is a custom integration!
If you have any issues with this you need to open an issue here:
https://github.com/blakeblackshear/frigate-hass-integration/issues
-------------------------------------------------------------------

2021-06-22 19:57:40 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform frigate.sensor: Platform not found (cannot import name 'DeviceInfo' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/enti
ty.py)).
2021-06-22 19:57:40 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform frigate.camera: Platform not found (cannot import name 'DeviceInfo' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/enti
ty.py)).
2021-06-22 19:57:40 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform frigate.switch: Platform not found (cannot import name 'DeviceInfo' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/enti
ty.py)).
2021-06-22 19:57:40 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform frigate.binary_sensor: Platform not found (cannot import name 'AddEntitiesCallback' from 'homeassistant.helpers.entity_platform' (/usr/src/homeassistant/h
omeassistant/helpers/entity_platform.py)).

Version of the custom_component
# cat /config/custom_components/frigate/manifest.json |grep version
    "version": "1.0.4",

# grep 0.0.1 /config/custom_components/frigate/* 2>/dev/null
/config/custom_components/frigate/const.py:VERSION = "0.0.1"

Ok, fixed by using 2021.6.6 - stupid.


Possible to get the id of an Autosuggest Container

Hi there,
Is it possible to get the id of an autosuggest container that is calling a function? For instance, I have multiple autosuggest containers on a page, and would like to have just one version of each function (e.g. onSuggestionsUpdateRequested, etc.) that have logic within them to act accordingly (set the appropriate property in state, etc) by checking which autocorrect container is invoking the function.
I'm somewhat new at this, so maybe this is already possible, or perhaps there is a better way of handling this issue that I haven't thought of. Thanks!

@BigPrimeNumbers Could you please provide a real world use case for this, with a  code example (use this example as a starting point)? I'd like to better understand your motivation.

I'm trying to build a form that has multiple autocomplete inputs. Instead of having multiple similar functions such as onSuggestionsUpdateRequested and getSuggestionValue for each container, I was attempting to insert logic into a single version of these functions that could determine which container was calling the function.
My (pseudo-ish) code is here:

Incidentally, I have a couple of other questions:

How would one get multiple Autosuggest components on the same line? They all seem to render as divs, and so they appear underneath each other as opposed to side-by-side.
How would one use Autosuggest with forms? I'd like to use this in conjunction with a custom form component that includes validation, such as tcomb-form, but am not quite sure how that would work.

Thanks again for your input and hard work!

@BigPrimeNumbers I can't see a clear benefit of doing what you are trying to do. I'd just create 2 separate components: FirstNameAutosuggest and LastNameAutosuggest. They could still share some common logic if you need:
// FirstNameAutosuggest.js
import commonLogic from 'common-logic-file';
getSuggestions(value) {
  // use `commonLogic` here
}

// LastNameAutosuggest.js
import commonLogic from 'common-logic-file';
getSuggestions(value) {
  // use `commonLogic` here
}

I think it's much easier to reason about your app this way rather than having one component that is doing everything.
To have 2 Autosuggests side by side just give containers display: inline-block;. Example
Integrating with other 3rd party components/libraries could be tricky and need to be handled case by case. There is no way I could guarantee easy integration with all 3rd party libraries. Try and see what exactly do you need from react-autosuggest for that integration, and come back with more specific questions.

Okay thanks for the input; it'll make the code slightly more verbose, but I'll work around it.
As for integration into a 3rd party component, I'm specifically trying to use tcomb-form. I can't seem to get the two to play nicely, so any thoughts you'd have for getting the two cooperating would be great (the component supports custom templates for libraries, so it should be possible, but I'm having trouble getting it to work).

Sorry, I probably can't help you with tcomb-form integration unless you have a more specific question / non-working runnable example.

No problem at all. I've got it figured out; if anyone else needs help with this particular setup, feel free to contact me. Thanks for doing what you could with this one!

@BigPrimeNumbers can you share your solution with tcomb-form? I tried the example that gcanti gived to you on the tcomb-form repo but it won't work properly.
Thanks in advance.

We actually abandoned tcomb-form months ago in favor of just making our own custom forms, so I can't remember exactly what I was doing to make it all work, but here's some of the (modified) code from commits back then:
"FormTemplate.js"
import React from 'react'
import T from 'tcomb-form'
import Autosuggest from './AutosuggestComponent.js'

export default class AutosuggestFormTemplate {

	getTemplate(options) {

	    function renderInput(locals) {
	        const value = locals.value || '' // react-autosuggest doesn't like null or undefined as value]
	        const suggestions = options.suggestions
	        const label = locals.label
	        const temp = new Autosuggest(suggestions, label)
			return temp.getAutosuggestComponent(
				value,
				locals.attrs.placeholder,
				(evt, { newValue }) => {locals.onChange(newValue)}
			)
	    }
	    return T.form.Form.templates.textbox.clone({ renderInput })
	}
}

"Form.js"
...
import T from 'tcomb-form'
import AutosuggestTemplate from './AutosuggestFormTemplate.js'
...

// TCOMB FORM STUFF
const Form = T.form.Form
const item = T.struct({
  name: T.String,
  company: T.String,
});
const options = {
    fields: {
        name: {
            attrs: {
                placeholder: 'Type item name'
            },
            template: getTemplate(names),
            error:  Please enter an item name
        },
        company: {
            attrs: {
                placeholder: 'Type company name'
            },
            template: getTemplate(companies),
            error:  Please enter a company name
        }
    }
}

function getTemplate(suggestions = []) {
    let autocompleteTemplate = new AutosuggestTemplate()
    return autocompleteTemplate.getTemplate({suggestions})
}

export default class Form extends React.Component {

    createAutosuggestComponent(suggestions, value, placeholder, onChange, label) {
        let autosuggestComponent = new Autosuggest(suggestions, label)
        return autosuggestComponent.getAutosuggestComponent(value, placeholder, onChange)
    }

...

    render() {
        return (

                           {this.createAutosuggestComponent(
                                    names,
                                    this.state.name,
                                    "Enter an item name",
                                    this.onChange,
                                    "name")}
                        {this.controls()}

        )
    }

Hope this helps!

I'll give it a try! Thanks!


php "inc" showing on every Dashboard page

This shows on Dashboard pages...

View Source shows this...

this has already been fixed https://github.com/concrete5/concrete5/pull/4592

So it has. My apologies...

no problem, better have a duplicate bug report than a missing one!


CollectionProperty doesn't propagate delete events through .filter{_ in true}

Just noticed this, see the failing test case at the very bottom. When adding an always-true filter to a CollectionProperty, it no longer propagates delete events. It does still propagate update events, though.
(This is with v2.0.3)

import XCTest
import ReactiveKit
import Nimble

class DemonstrateFilterIssueTests: XCTestCase {

    var collection: CollectionProperty<[Cat]>!
    let bag = DisposeBag()

    let catA = Cat(value: ["name" : "Mr A"])
    let catB = Cat(value: ["name" : "Mr B"])
    let catC = Cat(value: ["name" : "Mr C"])

    override func setUp() {
        super.setUp()
        collection = CollectionProperty<[Cat]>([catA, catB])
    }

    override func tearDown() {
        bag.dispose()
        collection = nil
        super.tearDown()
    }

    func testStartingState() {
        expect(self.collection.count) == 2
    }

    func testUpdateWithoutFilter(){

        var updates = 0
        collection
            .observeNext { changes in
                updates += changes.deletes.count
            }.disposeIn(bag)

        collection.replace([catB, catA], performDiff: true)

        expect(updates).toEventually(equal(1))
    }

    func testUpdateWithFilter(){

        var updates = 0
        collection
            .filter {_ in true} // does nothing (in theory..)
            .observeNext { changes in
                updates += changes.deletes.count
            }.disposeIn(bag)

        collection.replace([catB, catA], performDiff: true)

        expect(updates).toEventually(equal(1))
    }

    func testDeleteWithoutFilter(){

        var deletes = 0
        collection
            .observeNext { changes in
                deletes += changes.deletes.count
            }.disposeIn(bag)

        collection.replace([catA], performDiff: true)

        expect(deletes).toEventually(equal(1))
    }

    func testDeleteWithFilter(){

        var deletes = 0
        collection
            .filter {_ in true} // does nothing (in theory..)
            .observeNext { changes in
                deletes += changes.deletes.count
            }.disposeIn(bag)

        collection.replace([catA], performDiff: true)

        expect(deletes).toEventually(equal(1)) // failed: expected to eventually equal <1>, got <0>
    }
}

// demo model
class Cat: Object {
    dynamic var name = ""
    dynamic var miceEaten: Int = 0

    dynamic var id: String = NSUUID().UUIDString
    override static func primaryKey() -> String? {
        return "id"
    }
}

Hah, blame on me for not unit testing those parts of the code! Thanks for reporting this @iandundas!
It should be fixed now in v2.0.4.

thanks! 👌🏻😎


Create a Language Learning App Landing Page

Description: We're promoting language learning. Your task is to design a landing page for a language learning app with a multilingual and engaging design.
Difficulty: intermediate
Estimated Time: 5-6 hours

Hi User,
Hi there! I'm interested in working on this issue. Could you please assign it to me? I believe I can make a valuable contribution to this task.
Looking forward to your response. Thanks!

Hello Jayesh,
Thank you for your interest in working on this issue. I've assigned the issue to you, and you can now start working on it. If you have any questions or need any assistance, feel free to ask. Good luck with your contribution!
Best regards,
User


find match highlight border color

when i typed on find widget and hit enter, white color dotted border appears around matched words.
I want to change color of this with  "workbench.colorCustomizations".

today i updated vscode to version 1.75. (everyday, I apply update if available). it haven't happened by yesterday.
my setting is "workbench.colorTheme": "Default High Contrast"

I uncommented settings above (e.g. editor .findMatchHighlightBorder), but nothing changed

actually, I want to make dotted border transparent or erase, rather than changing color of it

I would agree this should be an opt-in. Was there a reason disseminated to users why this would be coerced? If it's a good reason I'd understand but for the worlds most customizable editor it feels strange for this not to have a configuration or opt-in.
My issue is not that it occurs on find and replace or find matching symobls, you could sell me on that if it stays within the bounds of the current symbol. My issue is that it additionally shows the border on unfocused and not active editors. This has made me think I'm editing a file my cursor is not actually in on multiple occasions; like when I Alt + Tab applications.
For example, just simply focusing on editor group one (picture left) and then focusing on editor group 2 (picture right) the border populates. It's very misleading and confusing on what the purpose of it is in my opinion.

Given that I would second the request to have a configuration or way to disable it.

This can be turned off via "editor.occurrencesHighlight": false. The root cause is an incorrect CSS rule.

This can be turned off via "editor.occurrencesHighlight": false. The root cause is an incorrect CSS rule.

@alexdima Thanks so much, I looked for a setting for hours! Looks like I tried everyone but that one!!


ENH: Update CI ExternalDataVersion to 5.0.1

It looks like GitHub releases is failing to serve 5.0.0 ExternalData tarballs. This may not resolve the issue, but it will not hurt.

GitHub is currently experiencing some issues according to:
https://www.githubstatus.com


Revert "[FLINK-20033] Ensure that stopping a JobMaster will suspend the running job"

Revert #13978 because of https://issues.apache.org/jira/browse/FLINK-20065.

AZP gives green light. Merging this PR now.


GCE: Forwarding Rules and Target Pools remain after kube-down

This, combined with the fact that the target pools are not updated when creating an external load balancer, means that your next cluster could have a misconfigured LB.

Create cluster set NUM_MINIONS=4
Create a service "foo" w/ external load balancer.
Tear down cluster with kube-down.
At this point you can still see your forwarding rules and target pools.
Repeat step 1 w/ NUM_MINIONS=6
Repeat step 2
Note the "foo" target pool only has 4 members. kubernetes-minion-{5,6} are missing.

Possibly addressed by a proper resolution to #1836?

I just ran into this!

This is related to #4630 and I'm working on it.

Closing as a dupe of #4630.


Pug example does not support locals

The pug loader used in this Wiki page does not say anything about passing in local variables.
Is it possible with this package? If yes, an example with that option would be great in the Wiki page. If not, maybe another package should be suggested.

I looked at the source and it seems like the option for locals is data and should be used like this:
...

rules: [

...

{
 test: /\.(pug|jade)$/,
 use: [{
  loader: 'pug-html-loader',
  options: {
   data: { foo: 'bar' }
  }
 }]
},

...

]

...


Feature request: Include official helper for building prompt from OpenAI messages[] source

Include official helper for building prompt from OpenAI messages[] source.
I believe there is many who would love this and quite surprised it was not done before. :)

Can you share an example snippet of what you'd like to see?

Sure thing Alex.
Here is my version in TS:
// Build a prompt from the messages
function buildClaudePrompt(
    messages: { content: string; role: 'system' | 'user' | 'assistant' }[],
) {
    let prompt = '';

    // Extract system prompt
    let systemPrompt = '';
    if (messages[0].role === 'system') {
        systemPrompt = messages.shift()!!.content;
        prompt = `${Claude.HUMAN_PROMPT} ${systemPrompt}${Claude.AI_PROMPT} OK.`;
    }

    // Build the prompt from the messages
    for (const message of messages) {
        if (message.role === 'user') {
            prompt += ` ${Claude.HUMAN_PROMPT} ${message.content}`;
        } else {
            prompt += `${Claude.AI_PROMPT} ${message.content}`;
        }
    }

    prompt += `${Claude.AI_PROMPT}`;

    return prompt;
}

I also happen to have a PHP implementation for it too
// Build the prompt. We will replicate OpenAI input standard to Anthropic API.
// https://console.anthropic.com/docs/prompt-design
//! Notice how the prompt always begins with "\n\nHuman:" and ends with "\n\nAssistant:"
$finalClaudeInput = collect($messages)
    // Claude doesn't support System role, so we will simulate as the beginning of the conversation.
    ->when($messages[0]['role'] === 'system', function (Collection &$collection) {
        $systemMessage = $collection->shift();
        // Rename the role to user
        $systemMessage['role'] = 'user';

        // Prepend the acknowledge "Assistant" message
        $collection->prepend([
            'role' => 'assistant',
            'content' => 'OK.',
        ]);

        // Prepend the system aka "Human" system message
        $collection->prepend($systemMessage);
    })
    ->add([
        'role' => 'assistant',
        'content' => ' ',
    ])
    ->reduce(function (string $carry, array $message) {
        return $carry .
            sprintf(
                "\n\n%s: %s",
                match ($message['role']) {
                    'user' => 'Human',
                    'assistant' => 'Assistant',
                    default => throw new \Exception(
                        'Invalid role provided: ' . $message['role'],
                    ),
                },
                Str::of($message['content'])
                    ->replace(['\n\n', '\r\n'], "\n")
                    ->toString(),
            );
    }, '');

The beauty of having one from you guys would implicitly mean that you choose best performing template. For example, the ambiguous handling of the system message. In some cases (php version) I convert system to user with Claude acknowledging it with "OK". Others have suggested to inject as Claude's first reply. Your own docs suggest either injecting all before or appending the first user message.
It'd be nice to have an input support OpenAI's messages to allow easily replacing GPT models with Claude (there's definitely good use cases and wins for that). It'd save a lot of mental overhead for us developers dealing with it while making it more attractive for more users to give a Claude a chance.
Let me know what you think!

@x5a thoughts?

Hi @flexchar. I appreciate your examples and ideas, as well as the thought you put in sharing them.
Currently, we're not able to offer drop-in compatibility with other foundation model APIs. You are welcome to build and independently release such a helper.
I encourage you to continue watching our public announcements for updates and improvements as we continue to evolve our APIs and SDKs.
Speaking of which, our newest model version, claude-2.1, was released today and offers first class support for system prompts. Because many use cases for Claude are unique, the evaluation of prompt performance will be somewhat specific to your individual needs. We provide some prompt engineering resources on our docs page, which you may have already seen.
Hopefully this addresses your feedback; I encourage you to continue sharing your ideas with our team.

@x5a Zaa Lee, I hope this message finds you well.
I understand you're a competition company to OpenAI and I have first hand experienced that Claude is suitable for a variety of use-cases where chat flow may not be used as much.
Nevertheless the completion endpoint enforces structure to include a chat flow with Human and Claude turns. It still forces us developers to adhere to it. Please do consider making a compatible, easy to swap in SDK/API endpoint. It'd be greatly appreciated.
By the way, amazing work with the documentation. I have greatly benefited from the knowledge resources on prompting.


observing unexpected regular changes for some users

hi, i'm reporting this issue on behalf of a coworker after doing some initial investigation
the git repo in question is https://github.com/ManageIQ/integration_tests
the user even after reinstalling pre-commit faced the following detail issues
diff --git a/.isort.cfg b/.isort.cfg
index f00acd605..be837f5e0 100644
--- a/.isort.cfg
+++ b/.isort.cfg
@@ -1,4 +1,4 @@
 [settings]
 line_length=100
 # NOTE: never change known_third_party by hand its managed by a hook
-known_third_party = Crypto,IPython,OpenSSL,PIL,_pytest,attr,bottle,bugzilla,cStringIO,cached_property,click,coverage,cycler,dateparser,dateutil,debtcollector,deepdiff,diaper,docker,dockerbot,dump2polarion,fauxfactory,freeze,gevent,git,github,glanceclient,humanfriendly,importscan,iso8601,jenkins,jinja2,jira,jsmin,keystoneauth1,kwargify,lxml,lya,manageiq_client,matplotlib,miq_version,navmazing,novaclient,numpy,paramiko,pathlib2,pdfminer,pika,pkg_resources,py,pygal,pytesseract,pytest,pytz,requests,riggerlib,rope,scp,selenium,sentaku,setuptools,six,slumber,sphinx,sphinx_rtd_theme,sqlalchemy,tabulate,tzlocal,urllib2,wait_for,werkzeug,widgetastic,widgetastic_patternfly,wrapanapi,xmltodict,yaml,yaycl,yaycl_crypt,zmq
+known_third_party = Crypto,IPython,OpenSSL,PIL,_pytest,attr,bottle,bugzilla,cached_property,click,concurrent,coverage,cycler,dateparser,dateutil,debtcollector,deepdiff,diaper,docker,dockerbot,dump2polarion,fauxfactory,freeze,gevent,git,github,glanceclient,humanfriendly,importscan,iso8601,jenkins,jinja2,jira,jsmin,keystoneauth1,kwargify,lxml,lya,manageiq_client,matplotlib,miq_version,navmazing,novaclient,numpy,paramiko,pathlib,pathlib2,pdfminer,pika,pkg_resources,py,pygal,pytesseract,pytest,pytz,requests,riggerlib,rope,scp,selenium,sentaku,setuptools,six,slumber,sphinx,sphinx_rtd_theme,sqlalchemy,tabulate,tzlocal,wait_for,werkzeug,widgetastic,widgetastic_patternfly,wrapanapi,xmltodict,yaml,yaycl,yaycl_crypt,zmq

the particular exchange in packages leads me to believe that pre-commit isn't quite sure which python version to use - and the seed tool adds certain packages based solely on whether its invoked on python3 or python2

this is intentional (and partially addressed with the latest version https://github.com/asottile/seed-isort-config/pull/19).  it is suggested to use language_version: ... (if you're using pre-commit) to select the version you want to classify imports for.  pre-commit by default will use whatever interpreter version you're running as unless you specify otherwise

the core of it is classify_import uses the current running interpreter to determine whether something is "standard library"

I notice that ManageIQ/integration_tests is using 1.2.0 -- maybe try under 1.3.0 -- should be more consistent there :+1:

oops, closing this as it's working as intended.  -- try out 1.3.0 and language_version: python3!


add ShowFullScreenButton Property

i know that there is a property (IsFullScreen), but ISFullScreen property remove noneclientarea

in generally, the demand for FullScreen is rarely or at least less than maximization and minimization relatively, hence we don't need a new button, we could provide a prop named "FullScreenMode", hide or show noneclientarea.

So it is better to close this PR


build.sh produces unusable wofi-emoji script

Hi! First off, thanks so much for making this. It's awesome.


Searching for a percent won't work correctly.

Even when parameterized, symbols that have meaning to SQLite (percent, underscore) will need to be escaped. Escaping is pretty easy, just map:

  "%" => "^%"
  "_" => "^_"
  "^" => "^^"

in all user-input LIKE strings, and append an ESCAPE '^' clause to every query.

Fixed by f4e54a0.


Low mach number correction for HLLC Riemann solver

Description
The HLLC Riemann solver is known to have an accuracy issue at the low Mach number limit. The new feature addresses this issue using two different methods (Chen et al. 2022 and Thornber et al. 2008)
Type of change

[x] New feature (non-breaking change which adds functionality)

Scope

[x] This PR comprises a set of related changes with a common goal

If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

[x] 2D_GreshoVortex for Ma = 0.001 (added to examples)

Test Configuration: 2D Gresho vortex is an exact solution to the incompressible N-S equations. The flow field should remain the same unless there is a numerical dissipation. The figure attached below shows the simulation results at Mach number 0.001 with low_Mach = 0 (no correction), low_Mach = 1 (Chen et al. 2022) and low_Mach = 2 (Thornber et al. 2008). Both correction methods work well.

What computers and compilers did you use to test this:

Richardson (cpu)
Bridges2 (cpu)
Delta (cpu/A100x4)

Checklist

[x] I have added comments for the new code
[x] I added Doxygen docstrings to the new code
[x] I have made corresponding changes to the documentation (docs/)
[x] I have added regression tests to the test suite so that people can verify in the future that the feature is behaving as expected
[x] I have added example cases in examples/ that demonstrate my new feature performing as expected.
They run to completion and demonstrate "interesting physics"
[x] I ran ./mfc.sh format before committing my code
[x] New and existing tests pass locally with my changes, including with GPU capability enabled (both NVIDIA hardware with NVHPC compilers and AMD hardware with CRAY compilers) and disabled
[x] This PR does not introduce any repeated code (it follows the DRY principle)
[x] I cannot think of a way to condense this code and reduce any introduced additional line count

If your code changes any code source files (anything in src/simulation)
To make sure the code is performing as expected on GPU devices, I have:

[x] Checked that the code compiles using NVHPC compilers
[x] Checked that the code compiles using CRAY compilers
[x] Ran the code on either V100, A100, or H100 GPUs and ensured the new feature performed as expected (the GPU results match the CPU results)
[ ] Ran the code on MI200+ GPUs and ensure the new features performed as expected (the GPU results match the CPU results)
[x] Enclosed the new feature via nvtx ranges so that they can be identified in profiles
[x] Ran a Nsight Systems profile using ./mfc.sh run XXXX --gpu -t simulation --nsys, and have attached the output file (.nsys-rep) and plain text results to this PR
nsys.zip
[ ] Ran an Omniperf profile using ./mfc.sh run XXXX --gpu -t simulation --omniperf, and have attached the output file and plain text results to this PR.

MFC does not recognize --omniperf
[hlee3@gpua001 gpu]$ ./mfc.sh run case/p001/p001_gs2_001/low_Mach1/case.py -t simulation --gpu -c delta --omniperf
mfc: OK > (venv) Entered the Python 3.11.6 virtual environment (>= 3.8).

usage: ./mfc.sh [-h] {run,test,build,clean,bench,bench_diff,count,count_diff,packer} ...
./mfc.sh: error: unrecognized arguments: --omniperf

[ ] Ran my code using various numbers of different GPUs (1, 2, and 8, for example) in parallel and made sure that the results scale similarly to what happens if you run without the new code/feature

@sbryngelson Could you re-run the benchmark test? It failed with exit code 1 without any useful messages. If it is really a problem, do you have any idea what went wrong? Thanks!


Changing ”quit” to ”menu”

Really love your game so far.
Just a thought, but your ”quit” button actually does not quit, but instead bring you to the menu. Where you can actually press ”continue” to come back to your game. That seems a bit misleading to me. Again, not a major issue.

Hi @luigibrosse !
Thanks for reporting, I think this is a neat suggestion :)
Want to submit a PR for it? Otherwise I can do it 👍


Allow clicks on the %100 bosses, with rate-limiting so boss lasts 1 hr

Enable clicks on the %100 bosses, to keep getting gold to buy, e.g., HP upgrades.  They don't need to go dying fast, however, so this calculates the damage per sec which would kill the boss in 1 hr, and then caps the click rate.  At low boss HP levels, this should end up setting the click rate to 0, just like now.
A bit messy, as I didn't refactor any of the surrounding code.

We want to make the bosses last as long as possible, this is, sadly, contrary to that goal :-(

For context, high-level bosses have so much HP that this only affects low levels.  The highest-powered player in 4600 currently has EDPS of 216,329,222,140,200.  The current Gold Helm in lvl 56,135,700 has over 80,000S HP.  If 1500 of the top player clicked 20 times per second, it would take 3.4 million hours of clicking to kill the boss.  Unless clicks do damage in other ways that the EDPS formula doesn't capture.
The longest natural boss plateau I can see from the drx.pl graphs is about half an hour, so I put the 1 hr damage target in as just a sanity check.
Still, if you'd rather not mess with it, I understand.


bug fixes and code improvement

bug fixes, random user agent, cross-platform auto-download driver, code improvement

forgot to add recoveryname in csvData (line 274)

Please remove data inside of csv file


sendmail: don't notify if no addresses are valid

If all the addresses available for sendmail are invalid, log that but don't
send any emails.
Signed-off-by: Mark Langsdorf<EMAIL_ADDRESS>

Not updating release notes draft because this is bugfix to an unreleased feature.


Hide ability to create a ShareLeaseClient for a Share or ShareSnapshot

/azp run net - storage - ci


Remove Study ID from prep info file

This mod will remove the study_id from the beginning of each row in column
'Orig_ID', if there exist numbers at the beginning of the string that
match the proper Study ID. If not, it will leave the entry untouched.

Duplicate of https://github.com/jdereus/labman/pull/446


research lerna-changelog

https://github.com/lerna/lerna-changelog

@mwcz Is this completed now?

PoC for using standard-version: https://github.com/patternfly/patternfly-elements/pull/853

PoC for using standard-version: https://github.com/patternfly/patternfly-elements/pull/853

Closing this issue with a reference to it in the release action issue.


Made Rakefile more detailed and added part one tests of second homework

Hi,
I've modified Rakefile to make test of each homework and part seperately and added part one of homework two.
Regards.

I've inspected a bit, these commits has missing lines.


Audio support in near future?

Original ticket:
https://sourceforge.net/p/djv/support-requests/4/

Duplicate of:
https://github.com/darbyjohnston/DJV/issues/18


BigNumber usage with undefined.

Nano Node v24.0 returns no values for unused accounts rather than 0 in pre v24.0
Nano Node PR: https://github.com/nanocurrency/nano-node/pull/3790
To Reproduce:
Use v24.0 API node in Nault
If there is an unused account in nault the issue can be seen directly/on reload
create a new account or a new wallet to make the issue appear as well.
Pull request has been opened which assumes 0 values when the api response didn't supply the values.
PR: 573

This issue will also be fixed by a newer node version in the short term. https://github.com/nanocurrency/nano-node/pull/4075


Fix for #132

What title says - fix for #132. Re-use the detection logic of the legacy processor in the modern one.

Codecov Report

Merging #133 (3b4f833) into master (567729c) will increase coverage by 0.00%.
The diff coverage is 88.88%.

@@           Coverage Diff           @@
##           master     #133   +/-   ##
=======================================
  Coverage   92.26%   92.26%
=======================================
  Files         254      254
  Lines        5392     5394    +2
  Branches      871      871
=======================================
+ Hits         4975     4977    +2
  Misses        247      247
  Partials      170      170

Impacted Files
Coverage Δ

src/ast/legacy/function_definition_processor.ts
88.46% <85.71%> (+0.22%)
:arrow_up:

src/ast/modern/function_definition_processor.ts
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


is there list canDrag support ?

hi, thanks so awesome package.
i wont's drag my list move ~ only use items.
how can i do it ?

Hi, glad you like it, and thanks for the suggestion!
I've added the canDrag functionality to lists now, you can find it in version 0.1.0. To use, just set canDrag in your DragAndDropList or DragAndDropListExpansion.
It did require an interface change so if you've made your own custom list class (implementing DragAndDropListInterface) you'll need to add the canDrag field to it.
I've also included a new example for Fixed Items.

aha~ your code nice read. i give up my fork and use 0.1.0.


Usage in npm module

Hey there,
I had a question about how best to use api-check when using it in an npm module I'm developing. Currently I'm developing the module as a local module within a larger project, so it is just using the same api-check setup file as is used in the larger application. However, I'd like to soon split this local module out of my project to have it be its own stand-alone npm module.
I would still like to have api-check running for my stand-alone npm module, which I think means I'll need to duplicate the api-check setup file, and instead require it explicitly within the module this time.
I'm wondering if there will be a way to disable api-check for my now npm-required module when building my app in production. I guess the question boils down to: does api-check get disabled if its included in node_modules when building an app in production?
Hope this question makes sense. Thanks a bunch for the excellent package!
Thomas

Glad you're using and benefitting from api-check! So api-check doesn't have any mechanism for disabling itself during production (maybe it should?) so you have to do so manually. In my app, when I build for production, what I do is:
var apiCheckFactory = require('api-check');
apiCheckFactory.globalConfig.disabled = process.env.NODE_ENV === 'production'; // <-- do this before anything else to get maximum perf benefits
var myInstanceOfApiCheck = apiCheckFactory();

So when you build your app, just set the NODE_ENV to production and you're set! I hope this helps.

Hmm thanks for your reply. I wonder what the best way would be to allow the user of the npm module to control this functionality. Maybe they'd want it to be disabled via the node_env variable, but maybe not... It seems like there must be a better way than hardcoding it into the module. Do you know of any other approaches that would allow the user to specify when/how it should be disabled?
Thanks again for all your work!

Most of the time, they'd want to disable api-check globally. So you can
document how to do that before your library is loaded (they can do it how
they want). Or, if you want them to be able to disable your instance
specifically, you could expose an api for them yourself.
Let me know if I'm understanding you incorrectly.
On Fri, Sep 11, 2015, 12:03 PM Thomas Rich<EMAIL_ADDRESS>wrote:

Hmm thanks for your reply. I wonder what the best way would be to allow
the user of the npm module to control this functionality. Maybe they'd want
it to be disabled via the node_env variable, but maybe not... It seems like
there must be a better way than hardcoding it into the module. Do you know
of any other approaches that would allow the user to specify when/how it
should be disabled?
Thanks again for all your work!
—
Reply to this email directly or view it on GitHub
https://github.com/kentcdodds/api-check/issues/33#issuecomment-139614702
.


Not OOP enough

This is a horror. There should be a Five class, and methods should be stackable.
Number --------------Five (also final)
|
|
|
|
Impossibility
|
|
|
Four

All methods of Four should super call to Impossibility, which destroys the universe. Universe destruction can be tested with a docker build.

As silly as this sounds. This would be something that I would see taught in a Brainwashing educational attempt by the college board


Debugger silently calls next() on iterable iterators that inherit from list

Try to debug this code:
import os
class MyIterator(list):
	def __iter__(self): return self
	def next(self): os._exit(1)
a = MyIterator()
print(a)   # put a breakpoint here

You will find that the debugger will quit before you even attempt to do anything.
This should not happen. If the debugger insists on iterating through members of every subclass of list silently, then it should really call list.__iter__(instance) rather than iter(instance). Otherwise it breaks debugging on the entire codebase, since the program no longer behaves correctly when stepping through.

More generally, I think the debugging experience of subclasses of basic types needs improvement.
(This bug probably touches on similar aspects of the debugger as #3505.)

I am having this same issue with sqlalchemy query results. I tried the experimental debugger in 15.7 preview 2.0, and its still a problem there as well.

This now works properly with ptvsd 4.x (I tested 4.1.3).


Display the app icon, from which the text was copied

Before Submitting Your Feature Request

[X] Check that there isn't already a similar feature request to avoid creating a duplicate.
[X] I have seen the FAQ.

Problem
Hello! I’ve used Maccy for a long time, but the main issue I faced was not being able to instantly see, from where I copied the item. Later, I discovered I can see the app, from where I copied, but it isn’t displayed next to it, so I must click on the item to get this information.
Solution
The solution I want is the app icon, from where the item is copied, right next to it. In this way, I can instantly see, from where I copied anything.
Alternatively, you could provide an option in the settings to display app icon next to the copied item, instead of the keyboard shortcut to paste that item. It would be more handy for me personally.

You can see the source app in the preview, which can be shown instantly by changing Preview Delay. Is this enough for your needs?

Well, I’ve used this feature. However, in the preview the app is shown by the text and it still isn’t that handy. With app icon right next to the copied icon, I would be able to see it instantly and it would be more handy to me.

@p0deje you could add an option to the settings to show the app icon, instead of shortcuts, so people can decide what’s more handy for them. What do you think about it?

I am open to merge PRs adding support for this, but I don't have capacity to implement it myself at the moment.

I have this actually implemented in the branch where I keep my custom changes: ab9fb0aa666b64bfcf17964d3397d34297520798
I don't have a lot of time to clean it up and connect it to the settings, however it is a good starting point @Romachamp

I have this actually implemented in the branch where I keep my custom changes: https://github.com/p0deje/Maccy/commit/ab9fb0aa666b64bfcf17964d3397d34297520798

Screenshots or didn't happen.

Awesome, this is exactly what I wanted. It would be awesome, if it were in the app.

@p0deje Will you implement this feature now, since there is a pull request?

@Romachamp There is no PR yet.
@weisJ Are you up to create a PR with this? We'd need a checkbox in Appearance setting to turn it on (with off by default) and localizations for it in all languages (I can help with that).

I can create a PR. Still have to do some cleanup. Currently not satisfied with how I am handling missing icons. Also the cache needs to have some eviction strategy to ensure icons get updated when they change.
I would also add the icon to the preview popover next to the application name. Should this be toggled through the same settings switch or always be visible?

I can create a PR. Still have to do some cleanup. Currently not satisfied with how I am handling missing icons. Also the cache needs to have some eviction strategy to ensure icons get updated when they change.

I haven't looked through the code, but I'll trust your judgement with whatever is the best here.

I would also add the icon to the preview popover next to the application name. Should this be toggled through the same settings switch or always be visible?

I think it's fine to always show in the preview, maybe with a different size to make it more compact.

Good point, I also think it’s fine to show the icon in the preview as well, but a bit smaller.

@weisJ Will you be able to implement these app icons into Maccy, after polishing the code a bit?

@p0deje There is now a PR. Will there be this feature any soon?


Improve file system rights

Currently the myuser has little to no rights which prevent easy copying of files and running scripts in Dockerfiles that extend our base images. We should enable common operations like running typescript builds, or react builds, copying .npmrc files etc. without the need to change file permissions all the time.

I guess this is no longer valid (as building TS projects work fine nowadays), so closing.


V9 -- Osmium not rendering properly in Storage Drawer

When I place osmium blocks into a compacting drawer the ingot and nugget do not show up.  If I place the ingot in all three do.

Not sure if this is an issue with the new texture so I thought I would post anyways.

I should also mention that when you try and take an ingot or a nugget out it doesn't allow you to.

Not really sure why that would be the case.  I really doubt this is on my end, though; everything is properly registered on my end with OreDictionary.  Ask the Storage Drawers team if they have any ideas, if they link it back to me, I'll take another look.

I noticed oredict was the same so I'm not sure either.  I've opened an issue on the SD github as well.
For reference: https://github.com/jaquadro/StorageDrawers/issues/290


Bootstrap site doesn't work

http://getbootstrap.com/ is not loading.

It works for me using Chrome 56. Does the link https://getbootstrap.com work? If so, it might be a http -> https redirect error

Please check your internet connection, including any intermediate proxies. If you continue to have problems, please post the output from cloudflare.com/cdn-cgi/trace.

I am having the same issue even on Chrome and IE. Here is the output from https://www.cloudflare.com/cdn-cgi/trace
fl=59f3 h=www.cloudflare.com ip=<IP_ADDRESS> ts=1479286550.247 visit_scheme=https uag=Mozilla/5.0 (Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0 colo=MAA spdy=h2 http=h2 loc=IN
Thank you!

Looks like we're good to go, site seems completely accessible on my end.


Inital x86 support

I've successfully built MCUBoot as a 3rd stage loader and have been able to use it to validate and boot a Zephyr OS image.

https://github.com/apache/incubator-mynewt-newt/blob/master/newt/image/image.go#L472 for the other tool's location for the header skipping.


geocoder.osm error cod 403

When using geocoder.osm('Berlin'), python returns an error.
Status code 403 from https://nominatim.openstreetmap.org/search: ERROR - 403 Client Error: Forbidden for url: https://nominatim.openstreetmap.org/search?q=Berlin&format=jsonv2&addressdetails=1&limit=1

Same here (only on Windows computer). On macOS everything's working as usual.

Add the following function in the class OsmQuery(MultipleResultsQuery) in osm.py
def _build_headers(self, provider_key, **kwargs): """Will be overridden according to the targetted web service""" return {"User-Agent": 'My User Agent 1.0'}
This should be work.
Reference: https://help.openstreetmap.org/questions/74205/nominatim-usage-policy-http-referers-and-user-agents

def _build_headers(self, provider_key, **kwargs):
    return {'User-Agent': 'DenisCarriere-geocoder/1.32.1'}

Definitely helps. Nominatim API policies must have changed since the package was last updated.


Make remote verification more prominent

Moves the enable remote verification button out to the main screen
Closes #35

Here

I tried to follow this comment
https://github.com/GrapheneOS/Auditor/pull/262#issuecomment-2067363814


x/exp: build failure on darwin-arm64-11_0-toothrot due to C type mismatch in dmitri.shuralyov.com/gpu/mtl

2020-12-15T15:35:30-b5a6e24/darwin-arm64-11_0-toothrot
# dmitri.shuralyov.com/gpu/mtl
In file included from mtl.m:5:
/tmp/buildlet/gopath/pkg/mod/dmitri.shuralyov.com/gpu/mtl@v0.0.0-20190408044501-666a987793e9/mtl.h:3:21: error: typedef redefinition with different types ('signed char' vs 'bool')
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/objc/objc.h:78:18: note: previous definition is here

CC @golang/release @dmitshur

Does such a complex dependency really belong in x/exp? It feels like it will mean a continuous investment of time to keep it working on all the platforms we have builders on, which is probably the kind of work that belongs outside the x repos.

I've already prototyped a fix locally and intend to clean it up and push it.

@mvdan, that's an interesting point, because last I checked that package was not actually build as a dependency of x/exp (see https://github.com/golang/go/issues/37175#issuecomment-585424741). I wonder what has changed since that analysis.

@bcmills In that comment, you ran go list [...] all presumably on Linux or another non-Darwin OS, which did not match the Darwin-only golang.org/x/exp/shiny/driver/mtldriver and golang.org/x/exp/shiny/driver/mtldriver/internal/coreanim packages that import mtl.
The mtl package is a real dependency of shiny when it is built with -tags metal on macOS.


Add Domain BPK Penabur Jakarta

https://bpkpenabur.or.id/
https://bpkpenabur.or.id/cities/jakarta

@frans-bfi First stage of verification passed, official domain is bpkpenabur.or.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://bpkpenabur.or.id/cities/jakarta/hubungi-kami

@frans-bfi Thank you for your patience while waiting for a review and processing your request. Pull request merged.


fix: We had two unnecessary comparisons.

We can simplify the code slightly.

@anonrig This will not affect speed but it will prevent compilers from warning us. Merged.


Ability to represent relationship without including the related resource

This is similar to some thinking reflected here:

https://github.com/jeregrine/jsonapi/issues/193
https://github.com/jeregrine/jsonapi/issues/174

We have a service boundary in our application but want to represent the JSON:API relationship. Relationships from within the service might be "includable", whereas relationships where another endpoint or service own the resource might not be.
I'll base an example off of the JSON:API example here: https://jsonapi.org/examples/
Let's say we have a CMS service that hosts our actual articles, but a separate service that really owns user information (authentication, authorization, profile data, etc.). Each article in the CMS would likely still have an author_id.
Using JSON:API. We'd still want to represent that as a relationship in a response from /articles, but we may never want to be able to include the full resource since its owned by another service.
It would be great to be able to make one request to something like /articles and see the relationship even though we have to make a subsequent request to get the actual people record
GET /articles HTTP/1.1

HTTP/1.1 200 OK
Content-Type: application/vnd.api+json

{
  "data": [{
    "type": "articles",
    "id": "1",
    "attributes": {
      "title": "JSON:API paints my bikeshed!",
      "body": "The shortest article. Ever.",
      "created": "2015-05-22T14:56:29.000Z",
      "updated": "2015-05-22T14:56:28.000Z"
    },
    "relationships": {
      "author": {
        "data": {"id": "42", "type": "people"}
      }
    }
  }]
}

GET /people/42 HTTP/1.1

HTTP/1.1 200 OK
Content-Type: application/vnd.api+json

{
  "data": {
    "type": "people",
    "id": "42",
    "attributes": {
      "name": "John",
      "age": 80,
      "gender": "male"
    }
  }
}

It would be helpful to be able to represent a relationship without exposing the ability to include that resource. Perhaps we could have the ability to hide the related view or to otherwise include the relationships portion of the payload without serializing the resource on the includes portion of the response.

@jeregrine Does this make sense? Is it on the radar?

We could also potentially solve the problem by allowing the relationship name to be overridden similarly to how fields are in the view.
Override author relationship in the view
def author(article, _conn) do
  %{ id: article.author_id }
end

It would also be good to be able to potentially disable including the data but still reflect the relationship, but that looks like a bit of a refactor by comparison.

Not sure if this addresses your use case, but the client can restrict which fields are returned on included resources by using sparse fieldsets.
eg:  GET /articles?include=author&fields[people]=id,age
should return something like this for you:
{
  "data": [{
    "type": "articles",
    "id": "1",
    "attributes": {
      "title": "JSON:API paints my bikeshed!",
      "body": "The shortest article. Ever.",
      "created": "2015-05-22T14:56:29.000Z",
      "updated": "2015-05-22T14:56:28.000Z"
    },
    "relationships": {
      "author": {
        "data": {"id": "42", "type": "people"}
      }
    }
  }],
  "included": [
    {"id": "42", "type": "people", "attributes": {"id": "42", "age": 80}}
  ]
}


PNG files missing for HERE example

See examples/images folder in the folium repo.

Ouch! Sorry about that. I'll move them ASAP.

Great! Some pointer from the folium repo to these examples here might be a good idea, too, isn't it? At least I couldn't find any...

It will be here and in the README. I just need to finish writing the docs and migrate more notebooks. Some of them are actually tests and need to stay in this repository, but most as duplicate "as tests" and have value only as examples. So it may take some time for me to finish figuring out which is which.


Configure Bulkrax

Attempting to import the sample CSV that the client provided caused the importer to fail with this error:
CSV::InvalidEncodingError - Invalid byte sequence in UTF-8 in line 3.

After some digging, I'm pretty sure the file is encoded in WINDOWS-1252 (primarily due to seeing <92> in vim instead of '). Running the following seems to have resolved the importer error:
iconv -f WINDOWS-1252 -t UTF-8 input.csv > output.csv

While configuring Bulkrax's field mappings using the CSVs provided by the client, it became apparent that WCU's field mappings conflict with other existing HykuUP tenants. Because of this, @orangewolf and I decided that the best short-term[^1] solution was to add a new Hyku Account setting that would allow for per-tenant field mappings. That Hyku PR can be found here:

https://github.com/samvera/hyku/pull/2384

[^1]: A better long-term solution would be if Bulkrax allowed you to edit a given importer's field mappings. That feature would live entirely within Bulkrax and thus wouldn't have to "bleed" into Hyku. However, that's too much of a lift for the scope of this ticket

The Hyku feature that allows for per-tenant field mappings has been merged into Hyku main. The next steps for this ticket are to:

Update the submodule in HykuUP knapsack to latest Hyku main
Deploy to prod
Configure the WCU and UNCA tenant field mappings

The Mobius field mappings should also be updated and ensured that they still work


[Enhancement] Deployment documentation

This PR aims to help improve documentation for Chibisafe in regards to deployment. I have included a basic example using Docker and Caddy, but the goal is to have far more examples for users to choose from.
Note that the format which I have used should not be regarded as a strict requirement to follow. It can be as simple as a step-by-step document without any fluff.

Isn't it a bit complicated? Not sure what a beginner should do ("Quick setup" or "Basic setup").
Recommending HSTS to beginners is begging for trouble.

Currently it's undergoing a bit of a structure change to simplify the steps a bit further. HSTS is recommended for making sure none of the resources are being fetched over HTTP. Thankfully, Caddy makes it relatively easy to change configs, so you can feel free to omit that step. I will be marking it as optional once I clean everything up.

HSTS, as it is currently recommended, makes the domain inaccessible over HTTP for 365 days, if the user decides to reuse it.
You can ensure no resources are accessible over HTTP using simple redirect - this is much more noob-friendly method.
In my eyes, it is very unlikely someone who needs to follow this guide is realistically going to benefit from HSTS.
At the same time, it is easy to imagine people asking why they can no longer access their domain after testing chibisafe.

I'll keep the HSTS recommendation for now while marking it optional. In exchange, I have reduced the max age to 24hr.  I'll also leave a note that it is intended for permanent deployments.


Python double free or corruption

Hi,
I try tensorflow-gpu 1.9.0rc1, but got the following error. I use ubuntu 16.04 (fresh install last night), numpy=1.13.3 (as required by 1.9.0rc1). Anyone tell me what the reason for this error? Thanks in advance.
Tuan
*** Error in python': double free or corruption (out): 0x00007ff8980021b0 ***
*** Error in python': double free or corruption (out): 0x00007ff898001d20 *** ======= Backtrace: ========= ======= Backtrace: ========= /lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7ff9781137e5] /lib/x86_64-linux-gnu/libc.so.6(+0x8037a)[0x7ff97811c37a] /lib/x86_64-linux-gnu/libc.so.6(cfree+0x4c)[0x7ff97812053c] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so(_ZN10tensorflow8EventMgr10PollEventsEbPNS_3gtl13InlinedVectorINS0_5InUseELi4EEE+0x6fb)[0x7ff943fcb14b] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow7GPUUtil18CopyGPUTensorToCPUEPNS_6DeviceEPKNS_13DeviceContextEPKNS_6TensorEPS6_St8functionIFvRKNS_6StatusEEE+0x3ff)[0x7ff93e95241f] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow16GPUDeviceContext21CopyDeviceTensorToCPUEPKNS_6TensorENS_11StringPieceEPNS_6DeviceEPS1_St8functionIFvRKNS_6StatusEEE+0x82)[0x7ff93e953a72] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x5f59ef)[0x7ff93e97e9ef] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow10CopyTensor6ViaDMAENS_11StringPieceEPNS_13DeviceContextES3_PNS_6DeviceES5_NS_19AllocatorAttributesES6_PKNS_6TensorEPS7_St8functionIFvRKNS_6StatusEEE+0x67a)[0x7ff93e97f53a] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow22IntraProcessRendezvous18SameWorkerRecvDoneERKNS_10Rendezvous9ParsedKeyERKNS1_4ArgsES7_RKNS_6TensorEPS8_St8functionIFvRKNS_6StatusEEE+0x4cc)[0x7ff93e9c7cac] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x63f2bd)[0x7ff93e9c82bd] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow19LocalRendezvousImpl9RecvAsyncERKNS_10Rendezvous9ParsedKeyERKNS1_4ArgsESt8functionIFvRKNS_6StatusES7_S7_RKNS_6TensorEbEE+0x961)[0x7ff93e7ebaf1] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN10tensorflow22IntraProcessRendezvous9RecvAsyncERKNS_10Rendezvous9ParsedKeyERKNS1_4ArgsESt8functionIFvRKNS_6StatusES7_S7_RKNS_6TensorEbEE+0x473)[0x7ff93e9c8903] /lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7ff9781137e5] /lib/x86_64-linux-gnu/libc.so.6(+0x8037a)[0x7ff97811c37a] /lib/x86_64-linux-gnu/libc.so.6(cfree+0x4c)[0x7ff97812053c] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x6036b9)[0x7ff93e98c6b9] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x606ba8)[0x7ff93e98fba8] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x60c8c3)[0x7ff93e9958c3] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(+0x60ca2a)[0x7ff93e995a2a] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZN5Eigen26NonBlockingThreadPoolTemplIN10tensorflow6thread16EigenEnvironmentEE10WorkerLoopEi+0x21a)[0x7ff93e9f3fba] /home/anhnt/tf-190/venv/lib/python3.5/site-packages/tensorflow/python/../libtensorflow_framework.so(_ZNSt17_Function_handlerIFvvEZN10tensorflow6thread16EigenEnvironment12CreateThreadESt8functionIS0_EEUlvE_E9_M_invokeERKSt9_Any_data+0x32)[0x7ff93e9f3062] /usr/lib/x86_64-linux-gnu/libstdc++.so.6(+0xb8c80)[0x7ff921537c80] /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba)[0x7ff97846d6ba] /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d)[0x7ff9781a341d]

Hi again,
This is the full stack trace.
Tuan
error-1.9.0rc1.txt

@anhnt1 - thanks for the report. Is it possible for you to create a reproducible example?
Thanks

@asimshankar - I got the error twice yesterday. But the work-around is here https://github.com/tensorflow/tensorflow/issues/6968, compile numpy from source.
But I still got segfault, another issue :)
Tuan

@anhnt1 : Not sure I followed - do you have some code to share that can reproduce the problem?

@asimshankar
Hi,
The code is attached, but data is quite big. I do not know how I can share it with you.
Tuan

@asimshankar Change the bug.txt to bug.py. I will zip data files later.
Tuan

@asimshankar
Data files (zip version) is still too big (125M). Not sure how I can share it with you.
Tuan

@anhnt1 : I was hoping for a much smaller, minimal, verifiable, complete example. Is there something you could do to isolate the issue further?


Remove prefixMatch from CacheQueryOptions

Implementor feedback from both Chrome & Firefox suggests this will be a de-opt case.
I added it to the spec to implement the kind of prefix matching appcache had in FALLBACK rules, but I can't say I've used it in practice in demos and apps.
We can add it back later if we see devs recreating this & we think the browser can do it faster.
@wanderview is getting further Mozilla feedback, if he gives the go-ahead, let's remove it.

Yea verily, make it so:
https://bugzilla.mozilla.org/show_bug.cgi?id=1130452


Proposal to add option to Disable [NavigationDrawerDestination]s

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
Any kind of routing that is permission based.
Proposal

It is basically the same issue as this: https://github.com/flutter/flutter/issues/127108, but now for the NavigationDrawer instead of the NavigationRail.

PR for this issue: https://github.com/flutter/flutter/pull/132349


Please remove onPrem scope from ExcelBuffer procedure CreateValidationRule.

Dear Developer.
Until Microsoft_Base Application_20.2.41144.43129 the  [Scope('OnPrem')] on ExcelBuffer table, procedure CreateValidationRule.
Is still there, can you please remove it?
I see on CreateRange is already removed.
I need it to export excel and make a drill down option.
Thank You.

Already done


add SO_KEEPALIVE

On our proxy server, after adding SO_KEEPALIVE, memory consumption is reduced from 23G to 16G.

Thanks! I think I'm too afraid to ask why your instance even consumes 16G, but this looks like a sensible patch. A recent update to pytest broke our builds, I'll retrigger CI here once #3077 has landed.

Generally reducing our memory footprint is always a good improvement.
However, in this case I don't really see WHY this might benefit us? Do you have a solid explanation or evidence on what is happening here?
As far as I understand SO_KEEPALIVE, it should not have any user-facing changes. It just puts the kernel into a different checking mode...? So why does this reduce memory consumption?

ref http://tldp.org/HOWTO/TCP-Keepalive-HOWTO/usingkeepalive.html


installation problem under node v4.0.0 and npm v2.14.3

After updating to node 4, local-npm ceased to work at some point.
Uninstall was ok, but when npm install -g local-npm, I got this error.
Here is npm-debug.log file.

I can repro. Seems to be related to out-of-date leveldown packages.

Should be fixed in 1.4.5, thanks for reporting!

Indeed fixed) Thanks for quick update!


📦 Update Python dependencies

No update to simple-salesforce, as it conflicted with cumulusci requirements.

@wlonk When I build I see a number of other conflicts with CCI requirements (and I've seen those for weeks now) -- was there something about simple-salesforce in particular that's cause for concern?

The only problem was that pip tools couldn't make a working set of frozen requirements.


Updated swift version to 4.2.1

This PR adds support to swift 4.2.
Many static vars were open, while a static is implicitly final. Silenced relative warnings by changing them to be public.

Just saw the approved PR for swift 4.2 (approved yesterday) this one has no reason to exist any longer.
Thank you everyone for updating this.


Use fully asynchronous implementation for DNS lookups

Right now we use blocking to run getaddrinfo, on a threadpool, but we can do better. I created async-dns as a way of doing DNS asynchronously. It needs improvements in robustness and optimization, but it usually works.
Another option would be trust-dns.

Probably out of scope for this crate. Maybe this would be a job for a higher-level crate, or something locked behind a feature flag.


⚠ Samsung has degraded performance

In 35a37b5, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2215 ms

Resolved: Samsung performance has improved in 5353f3f.


Hash Pricing view for All.

Summary
Add the Hash Pricing View for Users who don't hold any assets.

Problem Definition
Currently if you create a brand new wallet, your Portfolio view is sorta just blank.

Proposal
I suggest to add the pricing view to this page for All cases, or at least for those with zero Assets, as I think this could help drive interest into the utility token for those who are not aware. Provenance benefits from this as a marketing tactic for hash as near term solution until a more robust 'get Hash' feature can be implemented into the wallet App.

For Admin Use

[x] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

@mhellwinkel @jdfigure So, display a value (even if $0.00) and clicking on that portfolio value sends you to the Hash Pricing View?

The latest pull request adds an asset for Hash with an amount of 0 if there are no assets in the account.
If you have no Portfolio Balance when the assets are finished loading, the balance will be reflected as $0.00.
Some other fixes were made to the graph, to make it look more like the design.


In Dropdown Menu parent element is not clickable

Thanks!

How you solve this?

On Bootstrap the parent item of a dropdown menu is, by design, clickable only to open the dropdown and not as a link.
Here is the short version of why:

In Bootstrap, we use a mix of both hover and click though–hover for tooltips and popovers, clicks for dropdowns and more. What it really boils down to is user intent. The purpose of a hover state is to indicate something is clickable (underlined text) or to provide some quick information (full URL in a tooltip). The purpose of a click is to actually do something, to take an explicit action. Opening a dropdown is an explicit action and should only happen on click.

Here is the long version, explained directly from the author of Bootstrap: https://markdotto.com/2012/02/27/bootstrap-explained-dropdowns/

I solved it with the code below


chore: setup nighty build in CircleCI

Creating a new workflow called nightly that will run every night. It will remove package-lock.json, then run the build.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   99.82%   99.82%
=======================================
  Files           7        7
  Lines         580      580
=======================================
  Hits          579      579
  Misses          1        1

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 a0ec13a...7bfa418. Read the comment docs.


removes OpenAIAudioToText from config

Fix for extensions.android_packaging pipeline in preparation for ORT 1.19 release.

@skottmckay , is it OK to remove this OpenAIAudioToText since the build already is broken?


Replace old Animation Code with Sausage64

The old animation code should be replaced with the newly adopted Sausage64 library. Old Model exports should be re-exported via the appropriate tools and the old exports & tools removed

resolving this issue includes:

removing the old animation code
remove legacy tools for animation export and eventually include sausage64 tools
handle animated objects in level file structure separately

Just a heads up that Sausage64 had a big update recently which adds in binary model asset support. Might be worth pulling in the new changes :slightly_smiling_face:

@buu342 thanks for the heads up :) I created Issue #12  for that and will have a look at the updates!


lots of images are available by default, why only 26 listed on index?

v86 has 81 images in https://github.com/copy/v86/blob/master/src/browser/main.js  (I counted them all because I'm crazy) but index.html only lists 26, maybe there should be a "other options" dropdown which lists all the options? I thought windows nt4 was quite cool and its sort of a bummer more people dont know about the other options

It's supposed to show a few highlights in the homepage, so people don't get overwhelmed by the amount of options there are. Yeah, an Other Options dropdown would be a great idea :)

If you go to debug (/debug.html) on the site there are all of them, but it will show a ton of debug info

I'm planning to redo the front page at some point (similar to https://winworldpc.com/library/operating-systems).

That would be lovely, on that note actually, a good friend of mine was suggesting adding removable/hotswappable CD images so you can transport more than just the current removable floppy's, might be something to consider adding if you do a redesign I think

Done, thanks for the suggestion: https://copy.sh/v86/


Checkbox/Radio button does not have any indicator to show its focus state

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Checkbox/Radio button has focus ripple in focused state
What is the current behavior?
Checkbox/Radio button does not have any indicator to show its focused state
What is the use-case or motivation for changing an existing behavior?

Usability
Material spec: https://material.io/guidelines/components/selection-controls.html

Which versions of Angular, Material, OS, browsers are affected?
Material 2-beta1

This has been fixed in https://github.com/angular/material2/pull/1730 and https://github.com/angular/material2/pull/1731 although those PRs should be freshened up a bit.


Use pool with free list for large allocations

@oerling
A number of " error: pointer is missing a nullability type specifier (_Nonnull, _Nullable, or _Null_unspecified)" in macos compilation for the new method in MmapAllocator.h


site: change log links kaput

since site/api migration

the api -> npm connection was bad.  fixed that code.  now, however, I think there is an environment problem, where the api host is unknown -- either in CI or on prod box... I think it has to be in both locations (one for static build, the other for client dynamic reqs)

fixed by providing env vars in teamcity build.


toggle instead of hover to select spiders

this makes it less laggy because when you select a spider it lags so it isnt accidental anymore and it makes it easier to select spiders at higher speeds

camera clamp is lame


Allow to configure rollbax to contact api server via proxy

Why?
Because some security hardened production environments require to contact the "outside" internet through a proxy and hackney (the erlang http client used by rollbax) doesn't read the usual env vars used to config a proxy.
How?
Adding an optional entry to Rollbar.config allowing the specification of an URL to be passed to hackney to force the use of a proxy.
Caveats

The testing changes seems very complicated but are fast enough
hackney will ignore the proxy option in case the value is nil, which simplify a lot passing & using the config.

Kudos to @amuino for guiding me to this solution

Hey @GermanDZ, thanks for the contribution! I like it but all the infrastructure for testing it seems like overkill considering that we're basically testing a hackney feature. I would either:

not test proxying, again it's a hackney feature so it's supposed to work and I'm fine with not testing it
testing it through Mox (by mocking the HTTP client) and asserting that the proxy option is passed

Thoughts?

I cannot use Mox to mock :hackney, being it an Erlang module without a behaviour (ArgumentError) module :hackney is not a behaviour, please pass a behaviour to :for)
My options right now:

Keep the Proxy server for test (current solution)
Add an indirection, extracting :hackney from the Rollbax.Client and using something like http_client/0, then for production http_client/0 will return the :hackney module, but during test we can mock it to return an alternative. This also could lead to allow to use other http client via configuration (but not seems too desirable)
Remove the test for being only a simple config (but I don't code without test)

1 is what I would really prefer to avoid. 2 is what I meant when I said we could use Mox. I personally would go with 3, but either 2 or 3 are okay for me.

Working on 2 …


SeaTunnel job executed failed

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I reported an error when starting the seatunnel engine using my own configuration file
SeaTunnel Version
apche-seatunnel-2.3.2
SeaTunnel Config
env {
  execution.parallelism = 1
  job.mode = "BATCH"
}

source {
   jdbc {
        url = "jdbc:mysql://localhost:14306/student"
        driver = "com.mysql.cj.jdbc.Driver"
        user = "root"
        password = "mile123456"
        query = "select * from seatunnel_test"
	}
}

transform {

}

sink {
   jdbc {
        url = "jdbc:mysql://localhost:7306/student"
        driver = "com.mysql.cj.jdbc.Driver"
        user = "root"
        password = "mile123456"
        query = "insert into seatunnel_test(id,name) values(?,?)"
	}
}

Running Command
./bin/seatunnel.sh --config ./config/mysql_to_mysql_temp.conf -e local

Error Exception
Exception in thread "main" org.apache.seatunnel.core.starter.exception.CommandExecuteException: SeaTunnel job executed failed
        at org.apache.seatunnel.core.starter.seatunnel.command.ClientExecuteCommand.execute(ClientExecuteCommand.java:188)
        at org.apache.seatunnel.core.starter.SeaTunnel.run(SeaTunnel.java:40)
        at org.apache.seatunnel.core.starter.seatunnel.SeaTunnelClient.main(SeaTunnelClient.java:34)
Caused by: java.lang.RuntimeException: Plugin PluginIdentifier{engineType='seatunnel', pluginType='source', pluginName='jdbc'} not found.
        at org.apache.seatunnel.plugin.discovery.AbstractPluginDiscovery.createPluginInstance(AbstractPluginDiscovery.java:223)
        at org.apache.seatunnel.engine.core.parse.ConnectorInstanceLoader.loadSourceInstance(ConnectorInstanceLoader.java:61)
        at org.apache.seatunnel.engine.core.parse.JobConfigParser.parseSource(JobConfigParser.java:77)
        at org.apache.seatunnel.engine.core.parse.MultipleTableJobConfigParser.parseSource(MultipleTableJobConfigParser.java:298)
        at org.apache.seatunnel.engine.core.parse.MultipleTableJobConfigParser.parse(MultipleTableJobConfigParser.java:159)
        at org.apache.seatunnel.engine.client.job.JobExecutionEnvironment.getLogicalDag(JobExecutionEnvironment.java:155)
        at org.apache.seatunnel.engine.client.job.JobExecutionEnvironment.execute(JobExecutionEnvironment.java:147)
        at org.apache.seatunnel.core.starter.seatunnel.command.ClientExecuteCommand.execute(ClientExecuteCommand.java:140)
        ... 2 more

Zeta or Flink or Spark Version
Zeta version
Java or Scala Version
java8
Screenshots

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Please according to the deployment document to install connectors plugin.
https://seatunnel.apache.org/docs/2.3.3/start-v2/locally/deployment


Issue1041 data cleaning

Related issues
#1041
What was changed
Use ge argument in Cleaningbase.

This will be merged because #1045 will be prefered.


[FEA] Support parse_url function

It would be better to support parse_url function:
Codes:
scala> spark.sql("SELECT parse_url('http://spark.apache.org/path?query=1', 'HOST')").show

driverlogs:
@Expression <Alias> parse_url(http://spark.apache.org/path?query=1, HOST, false) AS parse_url(http://spark.apache.org/path?query=1, HOST)#3 could run on GPU
    ! <ParseUrl> parse_url(http://spark.apache.org/path?query=1, HOST, false) cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.catalyst.expressions.ParseUrl

This is a duplicate of https://github.com/NVIDIA/spark-rapids/issues/6969

@sameerz why was this closed?  #8761 was not merged, and #9481 is still pending.

@sameerz why was this closed? #8761 was not merged, and #9481 is still pending.

It's a dupe of https://github.com/NVIDIA/spark-rapids/issues/6969