text:
[Feature] Add Kinetic and Potential Energy Calculator:

Description
This calculator helps us to calculate the kinetic energy using the formula  K.E. = 1/2 * m * (v^2)
and potenial energy P.E. = m * a * h
where m= mass
a=acceleration
v=velocity
I want add this calculator
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Go ahead @Anshika14528


text:
[Feature] Add Astrological Age Calculator

Description
i want to add a calculator that would convert a persons age based on the time the planets take to revolve around the solar system and hence converting a persons age from earth years to some other planets years
please assign me this issue so that i can start working on this
@Rakesh9100
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Proceed @abhay-sen

thank you @Rakesh9100 i will start working on this asap!

source: gharchive/issue

license_type: permissive

license_source: github-api


text:
Safari extension doesn't always recognize feed

I have seen multiple cases where the Safari extension doesn't recognize the feed but when I do it manually within the app, then the feed is found.
Check the following link as an example -> https://www.nativedsd.com/blog/

NetNewsWire itself goes to great lengths to find feeds. In the case where the page does not declare the feed in the normal way, the Safari extension won’t see it.
We could have the extension go to the same lengths NetNewsWire does — but it would be doing this for just about every page you ever open, and it would be a performance and bandwidth hit.

repo: Ranchero-Software/NetNewsWire

license: MIT


text:
An error occurred while processing the “Feedbin” account: Not Found

I'm trying to delete the "MSFS Memory Error" feed—which doesn't exist on my Feedbin account anymore as the feed was previously deleted—and I'm getting this error.

Reopening because this needs additional verification.

authors: brentsimmons stuartbreckenridge


text:
qfl_loss一直升高是什么原因？

从coco2017的数据集中抽取两个类别的数据，然后使用nanodet_m.pth作为预训练网络，配置文件采用nanodet_custom_xml_dataset.yml，请问大佬这是什么情况呢？

@RangiLyu 另外通过将xml的数据转换为json格式的数据再进行训练之后，loss_qfl的值变成下降趋势

repo: RangiLyu/nanodet


text:
PSK -> .pem Files?

Hi, and thank you for this DTLS server.  It's the first I've tried that more or less works out of the box!
Context: I have some embedded devices acting as both CoAP servers and clients.
By default, enabling DTLS on these devices uses a pre-shared key phrase as the basis of the cipher.
How would I go about creating the .pem files for this module based on a known PSK string?

I have no idea. I'm not all that familiar with the different certificate formats unfortunately.
Also as a warning: This DTLS server is far from complete as the current state in the readme indicates. It does implement the basics of DTLS but many ciphers, extensions, etc. are missing as is renegotiation and message validation (validation as in handling error messages gracefully and returning alerts as per the spec instead of just ignoring these).

Ok, thanks!

authors: Rantanen msolters

repo: Rantanen/node-dtls

license: ISC


text:
Rasa X custom action found in domain but not in training data

Rasa version: 1.1.4
Rasa X version: 0.20.0
Issue:
I'm migrating existing bots to Rasa X and I'm receiving warnings under the Domain section of the UI for response templates/utterances associated with custom actions. The warning is:
The action utter_xyzzy was found in the domain but is missing in the training data
training data in this case is referring to the bot stories and the message is correct that I don't have any example stories that use utter_xyzzy.
The problem is, I've never seen any Rasa docs or example bots where utterances associated with custom actions are used in stories. The Rasa documentation for custom actions shows stories associated with the custom actions and these stories don't include the custom action utterances.
I also reviewed the recent medicare_locator example bot which has some custom actions and associated utterances and there are no stories that show the utterances.
I think either these warnings should be removed or the Rasa docs and migration/upgrade documentation needs to address the value or requirement for stories that use these custom action utterances. Is there any value in putting the utterances in these stories? My thinking is that it's not a good idea because the utterances are coded into the bots actions.py file and that putting it into stories is an unnecessary duplication.

Thanks for the issue, @tmbo will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

I think the warning is correct, it is referring to the actions section, not the templates:

you need to list all your utterance in the templates section
only if you are using the template directly as an action, you need to list it in actions

So in your case, you only need to list the templates in the templates section and remove them from the actions list (it has always been like that, in earlier versions there was no warning though).

Thanks, I think my confusion is in the use of the term training data in the message.
It would be more helpful to the user if the message said:
The action utter_xyzzy was found in the **actions** but is missing in the **templates**
I also have instances where my data is in the template but missing in actions so it would be more clear if that message said:
The action utter_xyzzy was found in the **templates** but is missing in the **actions**

I just updated to 0.20.2 and am having more problems with importing response templates. I have about 300 response templates in a domain.yml. Of those 10 are used as a template in an action.
If I remove the 290 that are not used as templates in actions, I get 290 errors stating that the template is missing from the domain. If I click on the Copy to the domain option next to the warning, it adds the template to the actions section but @tmbo stated above that it should not be in the actions section, only the templates section.

authors: rgstephens sara-tagger tmbo

repo: RasaHQ/rasa


text:
Support NATS alongside the HTTP API

Description of Problem:
For too many concurrent requests, when using RASA as a micro-service, many HTTP API calls and consequently opening many tcp sockets is not efficient.
Overview of the Solution:
NATS is a high performance messaging system that can work with a pub/sub architecture. Supporting the NATS subscriber in RASA helps optimizing the performance of the system and gain higher throughputs from RASA.

Thanks for submitting this feature request 🚀 @melindaloubser1 will get back to you about it soon! ✨

Hi @KamyarGhajar. Can you help me understand how and why you are using Rasa a microservice?

Hi @TyDunn . We use Rasa NLU as a sequence tagger and intent classifier in our services. In a micro-service architecture using a restful api is not that efficient you know specially when in a Kubernetes cloud solution with several Rasa pods running alongside each other.

authors: KamyarGhajar TyDunn sara-tagger


text:
ner_spacy not extracting PERSON entity.

Rasa NLU version:0.8
Operating system (windows, osx, ...):3.6.4
Content of model configuration file:
slots:
  all_issues:
    type: text
  bugs:
    type: text
  stories:
    type: text
  project_name:
    type: text
    initial_value: "FAE"
  med_pri:
    type: text
  low_pri:
    type: text
  high_pri:
    type: text
  PERSON:
    type: text
intents:
 - greet
 - goodbye
 - inform
 - anything_else
 - find_count
entities:
 - all_issues
 - bugs
 - stories
 - project_name
 - low_pri
 - med_pri
 - high_pri

templates:

  utter_greet:
    - 'Hello! How can I help?'
    - 'Hi!'
  utter_goodbye:
    - 'Talk to you later.'
    - 'Bye!'
  utter_anything_else:
    - "Can I help you with anything else?"
    - "Anything else?"
    - "Do you need anything else?"

actions:
 - utter_greet
 - utter_goodbye
 - utter_anything_else
 - actions.ActionIssue
 - actions.ActionCount

config.json
{
"pipeline" : ["nlp_spacy",
"tokenizer_spacy",
"intent_entity_featurizer_regex",
"intent_featurizer_spacy",
"ner_spacy",
"ner_crf",
"ner_synonyms",
"intent_featurizer_ngrams",
"intent_classifier_sklearn"

],

	"language" : "en",

	"path": "./models/nlu",

	"data": "./data/testData.json"

}

Issue:
I am unable to extract the PERSONS entity using ner_spacy. I have tried various examples but to no avail.

Can you test your utterance in the displacy tool and see if it can parse it?
https://explosion.ai/demos/displacy-ent

Also, is there a feature holding you back from updating your Rasa version?


text:
Add table of contents to the readme file

I would like to add table of contents section to the readme file for better readability and user experience.
Please assign it to me under gssoc-ext and hactoberfest

Currently not accepting changes in readme  file

authors: T-Fathima ratnesh-maurya

repo: Ratnesh-Team/Rehabify


text:
added support for *.sh files...

hi, i'm a total newbie about atom and github...
i just added a few lines  to block-comment.coffee so that it supports commenting blocks of shell scripts lines:
    ...
    else if extension is 'sh'
      commentStart = ''
      commentEnd = ''
    ...
    ...
    selectionText = selection.getText()

    if extension is 'sh'
      # add '# ' to the beginning of each line
      selectionText = selectionText.replace /^/, "# "
      selectionText = selectionText.replace /\n/g, "\n# "

nothing else, and works pretty good...

i've just sent my very first pull request...
hope you'll evaluate it!

repo: RayKwon/atom-block-comment

license: mit

license_source: bigquery


text:
希望能支持自定义openaiAPI代理地址

如标题，如果能像自定义API-KEY一样自定义API代理地址就更方便了，感谢！
As the title suggests, it would be more convenient if we could customize the API proxy address just like customizing the API-KEY. Thank you!

不太理解这个需求。Google Colab上运行，应该不需要设置代理？
On Mon, Apr 24, 2023 at 12:02 AM darknesscho @.***>
wrote:

如标题，如果能像自定义API-KEY一样自定义API代理地址就更方便了，感谢！
As the title suggests, it would be more convenient if we could customize
the API proxy address just like customizing the API-KEY. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/Raychanan/ChatGPT-for-Translation/issues/16, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWHSA3HPPOYR2DYYEEVMU3XCYCP5ANCNFSM6AAAAAAXJBMZUY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

authors: Raychanan darknesscho

repo: Raychanan/ChatGPT-for-Translation


text:
Added tinder clone

🛠 Fixes Issue
Closes #256

👨💻 Changes proposed

Added tinder clone web application using HTML, CSS, JS

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

This branch is 1 commit ahead, 6 commits behind Rayman-Sodhi/Clone-IT: main
@TG922

This branch is 1 commit ahead, 6 commits behind Rayman-Sodhi/Clone-IT: main @TG922

Ok will send PR again

Don't create a PR just fetch your upstream from branch Issue-256
@TG922

@gurjeetsinghvirdee Ok will do today ASAP

Is it ok @gurjeetsinghvirdee ??

source: gharchive/pull-request

repo: Rayman-Sodhi/Clone-IT


text:
Add dark mode

Thou shall not blast my eyes with #FFFFFF as the background color and #000000 as the text/foreground color.

Made it.
Currently, you can't change back to Light mode (which you shouldn't be able), and I haven't fully done it.

I have a made a fork with dark mode.
If you're wondering why the commit changes a lot, it's because I didn't clone the repo, I downloaded the source as a ZIP file...
Which means that I didn't have git source control.


text:
Unable to extract Samsung Firmware

Results in exponential expansion that eventually fills the disk.
https://www.samsungimaging.com/file/download?XmlIdx=280&file=C200GLU0APE4_160519_1848_REV00_user.bin

Try using the -r flag to delete intermediate files that binwalk can't process. This should prevent the disk from filling up.
https://github.com/ReFirmLabs/binwalk/wiki/Usage#-r---rm

repo: ReFirmLabs/binwalk


text:
history.push() stoppped working after upgrading to v5

I have recently upgraded history module from 4.10.1 to 5.0.0 and noticed that history.push() stopped working. After reverting it back to v4 it starts working.
Using hash history with react-router-redux

Same issue: Uncaught Could not find router reducer in state tree, it must be mounted under "router"

I am facing the same issue. Its look like there is some issue in the newest version "5.0.0" of history package. For now we can change its version to "4.10.1" which works for me.
Use bellow command for chnage history version
npm install history@4.10.1

Looks like a duplicate of #803

Here is the minimal code to reproduce this issue in the browser:
import React from 'react'
import { Router, Switch, Route } from 'react-router-dom'
import { createBrowserHistory } from 'history'

const history = createBrowserHistory()

export default () => (
  <Router history={history}>
    <Switch>
      <Route path="/target">

Target

      </Route>
      <Route path="/">

Home

        <button onClick={() => history.push('/target')}>Push</button>
      </Route>
    </Switch>
  </Router>
)

URL is changed, but it seems that no route is matched after history.push.

react-router-dom@5.2.0
history@5.0.0

Downgrading to history@4.10.1 fixed the problem.

Duplicate of #804 and #803

same thing, we are using hash router and history.push() is no longer working

Have you guys check the migration list?
https://github.com/ReactTraining/react-router/blob/dev/docs/advanced-guides/migrating-5-to-6.md

@brendonco I think this issue is regarding upgrading to v5, not v6

authors: PulkitAgg avindra brendonco danutzcodrescu etc-tiago mrlubos nikdo saadbinsaeed

repo: ReactTraining/history


text:
SecurityError while testing

I'm testing a react HOC which checks if there's a token present in localstorage or not. If there is, it redirects the user to the Home component else the user will be redirected to the Login component.
However, i'm getting this strange error in the last test where it tests for calling componentDidMount

SecurityError
at HistoryImpl._sharedPushAndReplaceState (node_modules/jsdom/lib/jsdom/living/window/History-impl.js:87:15)
at HistoryImpl.pushState (node_modules/jsdom/lib/jsdom/living/window/History-impl.js:69:10)
at History.pushState (node_modules/jsdom/lib/jsdom/living/generated/History.js:72:31)

app.test.js
import React from 'react';
import { shallow, mount } from 'enzyme';
import { expect } from 'chai';
import sinon from 'sinon';

import App from '../containers/App.jsx';

describe('<App />', () => {
  beforeAll(() => {
    global.localStorage = {
      i2x_token: 'someToken',
      getItem() {
        return 'someToken';
      }
    };
  });

  it('renders without exploding', () => {
    shallow(<App />);
  });

  it('renders children when passed in', () => {
    const wrapper = shallow(
      <App>

      </App>,
   );
    expect(wrapper.contains(
)).to.equal(true);
  });

 it('calls componentDidMount', () => {
    sinon.spy(App.prototype, 'componentDidMount');
    const wrapper = mount(<App />);
    expect(App.prototype.componentDidMount).to.have.property('callCount', 1);
    App.prototype.componentDidMount.restore();
  });
});

`
app.jsx
import React, { Component } from 'react';
import { browserHistory } from 'react-router';

class App extends Component {
  componentWillMount() {
    if (localStorage.i2x_token) {
       browserHistory.push('/home');
    } else {
      browserHistory.push('/');
    }
  }

  render() {
    return (

        {React.Children.toArray(this.props.children)}

    );
  }
}

App.propTypes = {
  children: React.PropTypes.node,
};

export default App;

package.json
  "dependencies": {
    "express": "^4.15.2",
    "react": "^15.4.2",
    "react-dom": "^15.4.2",
    "react-router": "3.0.0",
    "whatwg-fetch": "^2.0.3"
  },
  "devDependencies": {
    "babel-core": "^6.24.1",
    "babel-jest": "^19.0.0",
    "babel-loader": "^7.0.0",
    "babel-polyfill": "^6.23.0",
    "babel-preset-es2015": "^6.24.1",
    "babel-preset-react": "^6.24.1",
    "chai": "^3.5.0",
    "cross-env": "^3.1.3",
    "css-loader": "^0.28.0",
    "enzyme": "^2.8.2",
    "eslint": "^3.19.0",
    "eslint-config-airbnb": "^14.1.0",
    "eslint-plugin-import": "^2.2.0",
    "eslint-plugin-jsx-a11y": "^4.0.0",
    "eslint-plugin-react": "^6.10.3",
    "file-loader": "^0.11.1",
    "jest": "^19.0.2",
    "moment": "^2.18.1",
    "moment-timezone": "^0.5.13",
    "node-sass": "^4.5.2",
    "react-addons-test-utils": "^15.5.1",
    "sass-loader": "^3.2.0",
    "sinon": "^2.1.0",
    "style-loader": "^0.16.1",
    "webpack": "^2.2.1",
    "webpack-dev-server": "^2.4.2"
  }

https://github.com/tmpvar/jsdom/issues/1372

The issue linked here leads to a long rabbit hole but I finally found the solution. If you're using jest with jsdom, the fix is to add this to your configuration:
{
  // This configures the jsdom document with a URL.
  testURL: 'http://localhost',
}

In my case I was using jsdom directly (without jest) and created jsdom like so:
const document = jsdom('', {
  url: 'http://localhost/',
});
global.document = document;

@kumar303 thanks so much for sharing, you've saved my sanity ! ;-)

I was facing the issue :
(node:6392) UnhandledPromiseRejectionWarning: SecurityError: Could not parse url argument "../../ListMapView" to pushState against base URL "about:blank".
at HistoryImpl._sharedPushAndReplaceState (D:\KD1\KrossDeliveryWebApp\node_modules\jsdom\lib\jsdom\living\window\History-impl.js:73:15)
at HistoryImpl.pushState (D:\KD1\KrossDeliveryWebApp\node_modules\jsdom\lib\jsdom\living\window\History-impl.js:54:10)
at History.pushState (D:\KD1\KrossDeliveryWebApp\node_modules\jsdom\lib\jsdom\living\generated\History.js:88:21)
at D:\KD1\KrossDeliveryWebApp\node_modules\history\createBrowserHistory.js:175:23
at Object.confirmTransitionTo (D:\KD1\KrossDeliveryWebApp\node_modules\history\createTransitionManager.js:44:7)
at Object.push (D:\KD1\KrossDeliveryWebApp\node_modules\history\createBrowserHistory.js:166:23)
at D:\KD1\KrossDeliveryWebApp\src\scenes\account\login\login.Container.js:149:50
at
I'm using HOC
import { connect } from 'react-redux';
import { withRouter } from 'react-router-dom';
export default withRouter(connect(mapStateToProps, mapDispatchToProps)(Login));
This is my react code
`
if (this.props.user && this.props.user.userRole !== "" && this.props.user.loggedIn)  {

        let url = ApiUrls.Routes.ACNT_C + ApiUrls.Routes.CVLDUSR_M + "?currentPartyRoleId=" + this.props.user.partyRoleId + "&userName=" + this.props.user.userName;
        Api.Get(url, this).then((responseData) => {
            if (responseData.success) {
                if (this.props.user.userRole === Constants.APPROVAL_MANAGER) {
                    this.props.history.push("../Notifications");
                } else if (responseData.status === "Confirmation Pending") {
                    this.props.history.push("../../Verification");
                } else if (this.props.user.userRole === Constants.CUSTOMER) {
                    this.props.history.push("../../Customer/CustomerLanding");
                }else if (this.props.user.partyRoleId > 0) {
                    this.props.history.push("../../ListMapView");
                }
            }
        });
    }

`
Solution :
I was missing to mention tesst url in package.json file

"jest": {
    "testURL" : "http://localhost"
  },

authors: Risanshita kumar303 lgandecki rizvified thom-nic timdorr

repo: ReactTraining/react-router


text:
[Experimental] Flip startTransition and isPending

facebook/react#20976

Is this released anywhere yet? I know it's an unstable API, so things can change whenever they want. But just so folks using this don't run into trouble.

It's already released as latest experimental version. The last version that works with react-router-dom@experimental is {react,react-dom}@0.0.0-experimental-78120032d.

There are even more changes unstable_ prefixes got removed

@RichardLindhout  What I know transition is still unstable_  other one gets stable with lastest React but transition not.. Only if this changed in last days...

In the experimental release I'm importing straight from   React.useTransition but I'm using ^0.0.0-experimental-46491dce9 maybe they've changed it back to unstable

Are you using typescript and last Types? I remember i was updating types month ago for this... But maybe I should check correctly again :P  But good to know the import is directly available...

Updated React again. react-test-rendered is not the latest because it fails.

Are you using typescript and last Types? I remember i was updating types month ago for this... But maybe I should check correctly again :P But good to know the import is directly available...

Types were not up to date yet I think. Btw React 18 is now released so hope react-router will update soon :)

Can we bump this to the @alpha releases, now that they are out? That should be a more stable target. Thanks!

Updated and now it warns about not using ReactDOM.createRoot, but when I change it then it fails. react-test-renderer still fails with alpha version so I left it at last experimental version that works.

Did you use React.createRoot().render()? That's the new API (you create a root and then render or hydrate it)

Did you use React.createRoot().render()? That's the new API (you create a root and then render or hydrate it)

Yes, like that:
 act(() => {
-  ReactDOM.render(
+  ReactDOM.createRoot(node).render(
     <Router initialEntries={['/home']}>
       <Routes>
         <Route path="home" element={<Home />} />
         <Route path="about" element={<About />} />
       </Routes>
     </Router>,
-    node,
   );
 };

And I get error:
/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:1543
    return doc.body;
               ^

TypeError: Cannot read property 'body' of null
    at getActiveElement (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:1543:16)
    at getActiveElementDeep (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:8017:17)
    at getSelectionInformation (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:8050:21)
    at prepareForCommit (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:10500:26)
    at commitBeforeMutationEffects (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:21737:27)
    at commitRootImpl (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:25273:45)
    at commitRoot (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:25168:5)
    at finishConcurrentRender (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:24511:9)
    at performConcurrentWorkOnRoot (/home/lis/code/react-router/node_modules/react-dom/cjs/react-dom.development.js:24389:5)
    at workLoop (/home/lis/code/react-router/node_modules/react-dom/node_modules/scheduler/cjs/scheduler.development.js:253:34)
    at flushWork (/home/lis/code/react-router/node_modules/react-dom/node_modules/scheduler/cjs/scheduler.development.js:226:14)
    at Immediate.performWorkUntilDeadline (/home/lis/code/react-router/node_modules/react-dom/node_modules/scheduler/cjs/scheduler.development.js:516:21)
    at processImmediate (internal/timers.js:461:21)

authors: RichardLindhout damikun phaux timdorr


text:
BUG [TypeScript] UnityEngine.Networking has no exported member UnityWebRequest

With the following script. Click "Generate Project TypeScript Models" using the menu. A ts error occurs.
public class MyBehavior : MonoBehavior
{
    public UnityEvent<UnityWebRequest> OnStartDownload;

}

react/node_modules/@reactunity/renderer/dist/src/models/generated/unity".UnityEngine.Networking' has no exported member 'UnityWebRequest'.
    811 |   hideFlags: UnityEngine.HideFlags;
    812 |   UpdateServer: string;
  > 813 |   OnStartDownload: UnityEngine.Events.UnityEvent<UnityEngine.Networking.UnityWebRequest>;

Added this and bunch of other types in @reactunity/renderer@0.16.4.
But the TS generator doesn't include all types, so this may happen for some other type in the future. In that case, you have some options:

Open a Github issue and ask me to add those types
Manually convert problematic types to any after running TS generator
Add [TypescriptExclude] attribute to exclude them from TS generator, or use [TypescriptRemap] to convert them to other types

authors: KurtGokhan xiaohai-huang

repo: ReactUnity/core


text:
Use of unresolved identifier 'combineLatest'

Hi guys,
I'm trying to use the combineLatest in a test project.
But i get the error Use of unresolved identifier 'combineLatest but i can't figure out what i'm missing. I don't find other example of this implementation.
I'm using the latest version 6.5.2 with swift 4 and iOS target 10+
Code:
import Bond

    @IBOutlet weak var emailTextField: UITextField!
    @IBOutlet weak var passwordTextField: UITextField!
    @IBOutlet weak var loginButton: UIButton!

    override func viewDidLoad() {
        super.viewDidLoad()

// this work
        _ = emailTextField.reactive.text.observeNext { text in
            print(text!)
        }

// this show Use of unresolved identifier 'combineLatest
        combineLatest(emailTextField.reactive.text, passwordTextField.reactive.text) { email, pass in
            return email.length > 0 && pass.length > 0
            }
            .bind(to: loginButton.reactive.enabled)
    }

Do you have any idea?

import ReactiveKit

Thanks i tried with it also but then i got a new errors.
In the first line i get:  Argument type '(_, _) -> Bool' does not conform to expected type 'SignalProtocol'
combineLatest(emailTextField.reactive.text, passwordTextField.reactive.text) { email, pass in
     return email.length > 0 && pass.length > 0
}.bind(to: loginButton.reactive.isEnabled)

Now what i did to bypass this error is to don't use the parameter.
combineLatest(emailTextField.reactive.text, passwordTextField.reactive.text) { (_, _) in
      return (self.emailTextField.text?.count)! > 0 && (self.passwordTextField.text?.count)! > 0
}.bind(to: loginButton.reactive.isEnabled)

But i definitely would to find a way to make it work like in the example in the readme (which should be adapted I think)

combineLatest(emailTextField.reactive.text, passwordTextField.reactive.text).map { email, pass in
return email.length > 0 && pass.length > 0
}.bind(to: loginButton.reactive.isEnabled)
combineLatest().map <---

Ok thank you that worked. I think an update of the Readme is needed.
Here the example working code for swift 4:
combineLatest(emailTextField.reactive.text, passwordTextField.reactive.text).map { (email, pass) -> Bool in
      return (email?.count)! > 0 && (pass?.count)! > 0
}.bind(to: loginButton.reactive.isEnabled)

Just a question why do we need to import Bond and ReactiveKit, shouldn't be Bond which make the link?

TBH Im not 100% positive... It looks like Bond imports ReactiveKit in itself, but the actual method for combineLatest lives in ReactiveKit.... seems strange to me as well.

authors: dcrobert omerozk

repo: ReactiveKit/Bond


text:
Added forkJoin operator

This is forkJoin() operator implementation based on RxJS 5 (selector function and it's available only as a static method). The tests copy mostly RxJS 4 with exception to selector function which is tested the same way as RxJS 5 tests.

Coverage decreased (-0.07%) to 95.767% when pulling 65eef5f5c8fd9109ffc1330746857d23247a12c3 on martinsik:forkjoin-observable into 7c7d8584dc8d92337fdaedbe74c4ced5f95e9a69 on ReactiveX:master.

Coverage decreased (-0.07%) to 95.767% when pulling 65eef5f5c8fd9109ffc1330746857d23247a12c3 on martinsik:forkjoin-observable into 7c7d8584dc8d92337fdaedbe74c4ced5f95e9a69 on ReactiveX:master.

@martinsik - Do you think you could add a test for throw inside the selector?

Coverage decreased (-0.06%) to 95.778% when pulling b135e86a3e6315740053b676859bd4072db7ee3a on martinsik:forkjoin-observable into 7c7d8584dc8d92337fdaedbe74c4ced5f95e9a69 on ReactiveX:master.

I incorporated everything you've mentioned.

authors: coveralls martinsik mbonneau

repo: ReactiveX/RxPHP


text:
Record perf results vs RxJS 4, then drop RxJS 4 from perf testing

It's time to move on. Future versions of this library need to be compared against previous versions of itself, not Rx4.
We need to

Record and keep the perf results vs RxJS 4, if for no other reason than documentation purposes.
Fix the perf tests to target a previous version of RxJS 5.

cc @kwonoj @jayphelps

Hi @blesh are you still planning to use Benchpress for future perf tests? I'm actually building a tool called Perfy, on top of Benchpress which allows to keep track of previous tests campaign, which is exactly what you need. The tool lives here :https://github.com/manekinekko/perfy.
If this is something you wanna try out, I can prepare a POC for RxJS.

@manekinekko that sounds awesome. I'd love to see it in action. I have a feeling our benchpress tests have probably fallen into disrepair.
Do note, though, that we have both micro perf and macro perf tests.

alright. I'll begin working on that just right after christmas 🎅

@blesh here is a quick draft... Please tell if this is what you had in mind?

@manekinekko I'm curious if we could create github webhooks to display some critical info per each PR (let's not think about ergonomics like where to run server, etcs for now)

@kwonoj yep I think it's doable.
Here are some facts about Perfy:

It runs after protractor/benchpress has ended executing specs tests.
It overrides benchpress config in order to make benchpress outputs JSON files.
It reads those JSON files and merges them into a single JSON file which serves as a database.
It has a CLI interface.

Here is the complete workflow:
npm run tests --> protractor --> benchpress --> perfy --> charts
One thing I'm missing here is how we'll be able to provide a subset of spec tests to benchpress. Typically, we will need to:

run only the tests that are affected by the specific PR that is being created.
Or do we need to run the whole tests?

I think 2) would be a good approach 🤔

Is benchpress still being supported? I don't see it in the latest modules in angular, it's supposed to be @angular/benchpress, but it's missing after angular 2.4. Seems like we need to overhaul all protractor and benchpress test code too if we want to move forward with this.

This is just old now.

Haha! I totally forgot about this experiment 😁

authors: benlesh blesh claudiordgz kwonoj manekinekko

repo: ReactiveX/rxjs


text:
TypeScript error in config.ts: Conversion of type 'undefined' to type 'PromiseConstructorLike' may be a mistake

RxJS: 6.5.3
TS: 3.6.2
src/internal/config.ts
export const config = {
  /**
   * The promise constructor used by default for methods such as
   * {@link toPromise} and {@link forEach}
   */
  Promise: undefined as PromiseConstructorLike,

Fails with compilation error:

TypeScript error in D:/Dev/OpDashBoard/FrontEnd/ClientApp/node_modules/rxjs/src/internal/config.ts(12,12):
Conversion of type 'undefined' to type 'PromiseConstructorLike' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.  TS2352

I think the fix would be to define it as:
  Promise: undefined as unknown as PromiseConstructorLike

Or maybe another, cleaner way.
Repro in Playground

I got this error when accidentally imported a RxJS file under src/internal/ into my project

In the interim, you could - should, IMO - specify skipLibCheck in the compiler options in your tsconfig.json. There really is little point in having TypeScript check the internals of packages within node_modules when those packages contains .d.ts files that have already passed through the TypeScript compiler.

I'm going to close this as it's a sub-case of a known issue: the TypeScript compiler options used to build RxJS are insufficiently strict. In particular, strictNullChecks is false. See https://github.com/ReactiveX/rxjs/issues/4968

authors: cartant kostrse


text:
CSRF cookie not set

Hello, i got this error when i try to authenticate myself with Google Auth:

Here's my settings.py, have i done something wrong ? many thanks.

"""
Django settings for ponynote project.

Generated by 'django-admin startproject' using Django 1.11.4.

For more information on this file, see
https://docs.djangoproject.com/en/1.11/topics/settings/

For the full list of settings and their values, see
https://docs.djangoproject.com/en/1.11/ref/settings/
"""

import os

from os.path import join, dirname
from dotenv import load_dotenv

dotenv_path = join(dirname(__file__), '.env')
load_dotenv(dotenv_path)

# Build paths inside the project like this: os.path.join(BASE_DIR, ...)
BASE_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))

# Quick-start development settings - unsuitable for production
# See https://docs.djangoproject.com/en/1.11/howto/deployment/checklist/

# SECURITY WARNING: keep the secret key used in production secret!
SECRET_KEY = os.environ.get("SECRET_KEY")

# DJANGO OAUTH TOOLKIT ID and SECRET used in github_social.views
CLIENT_ID = os.environ.get("CLIENT_ID")
CLIENT_SECRET = os.environ.get("CLIENT_SECRET")

# SECURITY WARNING: don't run with debug turned on in production!
DEBUG = True

ALLOWED_HOSTS = []

# Application definition

INSTALLED_APPS = [
    'django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',
    'rest_framework',
    'corsheaders',
    'oauth2_provider',
    'social_django',
    'rest_framework_social_oauth2',
    'rocket.apps.RocketConfig',
]

MIDDLEWARE = [
    'django.middleware.security.SecurityMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
    'corsheaders.middleware.CorsMiddleware',
    'django.middleware.common.CommonMiddleware',
    'django.middleware.csrf.CsrfViewMiddleware',
    'django.contrib.auth.middleware.AuthenticationMiddleware',
    'django.contrib.messages.middleware.MessageMiddleware',
    'django.middleware.clickjacking.XFrameOptionsMiddleware',
    'rocket.middlewares.AdminTimezoneMiddleware.AdminTimezoneMiddleware',
]

ROOT_URLCONF = 'settings.urls'

TEMPLATES = [
    {
        'BACKEND': 'django.template.backends.django.DjangoTemplates',
        'DIRS': [os.path.join(BASE_DIR, "templates"), ],
        'APP_DIRS': True,
        'OPTIONS': {
            'context_processors': [
                'django.template.context_processors.debug',
                'django.template.context_processors.request',
                'django.contrib.auth.context_processors.auth',
                'django.contrib.messages.context_processors.messages',
                'social_django.context_processors.backends',
                'social_django.context_processors.login_redirect'
            ],
        },
    },
]

WSGI_APPLICATION = 'settings.wsgi.application'

# Database
# https://docs.djangoproject.com/en/1.11/ref/settings/#databases

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.sqlite3',
        'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
    }
}

# Password validation
# https://docs.djangoproject.com/en/1.11/ref/settings/#auth-password-validators

AUTH_PASSWORD_VALIDATORS = [
    {
        'NAME': 'django.contrib.auth.password_validation.UserAttributeSimilarityValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.MinimumLengthValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.CommonPasswordValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.NumericPasswordValidator',
    },
]

# Internationalization
# https://docs.djangoproject.com/en/1.11/topics/i18n/

LANGUAGE_CODE = 'en-us'

TIME_ZONE = 'UTC'

USE_I18N = True

USE_L10N = True

USE_TZ = True

# Static files (CSS, JavaScript, Images)
# https://docs.djangoproject.com/en/1.11/howto/static-files/

STATIC_URL = '/static/'

REST_FRAMEWORK = {
    # Use Django's standard `django.contrib.auth` permissions,
    # or allow read-only access for unauthenticated users.
    'DEFAULT_PERMISSION_CLASSES': [],
    'TEST_REQUEST_DEFAULT_FORMAT': 'json',
    'DEFAULT_AUTHENTICATION_CLASSES': (
        'oauth2_provider.contrib.rest_framework.OAuth2Authentication',
        'rest_framework_social_oauth2.authentication.SocialAuthentication',
        'rest_framework.authentication.SessionAuthentication',
    ),
}

AUTHENTICATION_BACKENDS = (
    'social_core.backends.google.GoogleOAuth2',  # for Google authentication
    'django.contrib.auth.backends.ModelBackend',
)
SOCIAL_AUTH_GOOGLE_OAUTH2_KEY = os.environ.get('SOCIAL_AUTH_GOOGLE_OAUTH2_KEY')
SOCIAL_AUTH_GOOGLE_OAUTH2_SECRET = os.environ.get(
    "SOCIAL_AUTH_GOOGLE_OAUTH2_SECRET")

CORS_ORIGIN_WHITELIST = (
    'localhost:3000',
    'localhost:3001',
)
CORS_ALLOW_CREDENTIALS = True

#DJANGO OAUTH TOOLKIT EXPIRATION SECONDS  - DEFAULT IS 36000 WHICH IS 10 hours
OAUTH2_PROVIDER = {
    'ACCESS_TOKEN_EXPIRE_SECONDS': 36000,
}

well, i solved it by accident.
I've switched the position of the url in the urls.py like this:
Before:
from django.conf.urls import url, include
from django.contrib import admin
from django.views.generic import TemplateView

from rocket import endpoints as endpoints_rocket

urlpatterns = [
    url(r'^admin/', admin.site.urls),
    url(r'^rocket/', include(endpoints_rocket)),
    url(r'^', TemplateView.as_view(template_name="index.html"), name='home'),
    url(r'^auth/', include('rest_framework_social_oauth2.urls')),
]

After:
from django.conf.urls import url, include
from django.contrib import admin
from django.views.generic import TemplateView

from rocket import endpoints as endpoints_rocket

urlpatterns = [
    url(r'^admin/', admin.site.urls),
    url(r'^rocket/', include(endpoints_rocket)),
    url(r'^auth/', include('rest_framework_social_oauth2.urls')),
    url(r'^', TemplateView.as_view(template_name="index.html"), name='home'),
]

If someone know why, i'm waiting your explanation \o/


text:
@hsong-rh In the source object in the Sources API we have these extra fields

@hsong-rh In the source object in the Sources API we have these extra fields
Source(id: integer, name: string, uid: string, created_at: datetime, updated_at: datetime, tenant_id: integer, source_type_id: integer, version: string, availability_status: string, imported: string, source_ref: string, last_checked_at: datetime, last_available_at: datetime)

We need these extra vars
availability_status, last_checked_at, last_available_at
Originally posted by @mkanoor in https://github.com/RedHatInsights/catalog_inventory-api/pull/8#r546963605

PR #12


text:
cleanup: implement delete images.

Description

implement delete images with the following conditions:

images names are not in the list
updated_at of images is older than one week
image is not linked to any device.

implement delete of orphan images (orphan images are images that has image-set already deleted)
updated the main function to not return error if one of the features is not available

NOTES:

all images are soft deleted as described and tested in unittests.
Put it in do not merge , to allow updating the list to keep ImagesWithNamesToKeep.

FIXES:  https://issues.redhat.com/browse/THEEDGE-3490
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Patch coverage: 67.44% and project coverage change: +0.02% :tada:

Comparison is base (8ef0455) 60.99% compared to head (08390e3) 61.01%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2183      +/-   ##
==========================================
+ Coverage   60.99%   61.01%   +0.02%
==========================================
  Files          78       79       +1
  Lines       11615    11658      +43
==========================================
+ Hits         7084     7113      +29
- Misses       3920     3930      +10
- Partials      611      615       +4

Files Changed
Coverage Δ

cmd/cleanup/deleteimages/deleteimages.go
67.44% <67.44%> (ø)

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

authors: codecov-commenter ldjebran

repo: RedHatInsights/edge-api


text:
openapi: import from yaml instead of JSON

Use the yaml representation of the openapi used in Image Builder to load generate the RTKQ code.

What's the advantage of yaml over json? Also would it be useful to extend the api.sh script to pull the latest specs from upstream?

The advantage would be to be able to pull directly from upstream instead of getting the api representation from console.redhat.com. But that opens a few more questions about compatibility. I'm trying to wrap my head around these questions :sweat_smile:

Alright so changing the representation from json to yaml is the first step at automating the task for a github action that would live in IB. This action would check that if on the latest main commit API changes induce a change in the generated code from the frontend and if so, open a PR onto this repo.

authors: croissanne lavocatt

repo: RedHatInsights/image-builder-frontend


text:
new param in function

Description
new param in function
Fixes CCXDEV-7859
Type of change
Please delete options that are not relevant.
Testing steps
Please describe how the change was tested locally. If, for some reason, the testing was not done or not done fully, please describe what are the testing steps.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #249 (c2c37d6) into master (ca4e43d) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #249      +/-   ##
==========================================
- Coverage   90.85%   90.77%   -0.08%
==========================================
  Files          34       34
  Lines        1137     1138       +1
==========================================
  Hits         1033     1033
- Misses         89       90       +1
  Partials       15       15

Impacted Files
Coverage Δ

tests/helpers/rules.go
0.00% <0.00%> (ø)

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 ca4e43d...c2c37d6. Read the comment docs.

authors: codecov-commenter dpensi

repo: RedHatInsights/insights-operator-utils


text:
Unable to Add Manifest.

Steps to reproduce :

Login to : https://console.stage.redhat.com/beta
Navigate to Manifests using "Find an App or Service" -> "Manifests"
Click Create Manifests button
Provide Details [Anand-Test1, Satellite 6.9 ] and Save
Expected - A new manifest should be created.
Actual - Manifest is not Created
Close the Create Manifests View
Click on "Manifests" Menu in LHS Navigation panel
Verify Manifests page
Expected - Manifests page should be rendered correctly
Actual - Manifest page appears with errors

Environment: Stage/Beta

As to parts 1-4 -- as discussed, this appears to be related to a 403 error due to a misconfiguration of permissions for the test user involved. The error message in Splunk is "account does not have a Satellite subscription". Hence, the create manifest form should fail here. Separately there is an issue in the toast error message not displaying correctly; that has been resolved in this PR and merged; it should appear in stage by tomorrow morning at the latest. Let me know if that hasn't been deployed to stage by then.
Separately, as far as 6-8, it appears this bug occurs no matter what one does within the app -- if you navigate to manifests page and then click that side nav link again, it will show an error. There's a good chance this is due to the updates from c.rh.c team with the new side nav architecture. I created a new issue for this and will follow up separately, as I think this should be handled separately from this story.
Let me know if anything further needs to be done on this front.

Actually after digging in a bit further, it appears it is a bug within insights-chrome. I created an issue there - https://github.com/RedHatInsights/insights-chrome/issues/1570 and will keep an eye on it.

Was resolved with https://issues.redhat.com/browse/TEAMNADO-2617 and with user configuration changes.

authors: joeldmyers qatoolist

repo: RedHatInsights/subscription-central-ui


text:
Handle percent encoded repository_url

Now that the purls in erratum notes fields include repository_url, the purl has to be percent-decoded to match purls in Corgi.

@RedHatProductSecurity/corgi-devs Please review.


text:
$ref recursive/indefinite resolving for examples

Describe the bug
If

examples (in our case for responses) are added from a separate file with e.g.    example:
      $ref: '../../dir/otherdir/example.data.get.json'

and those example responses themselves contain a $ref like   {
          "field": {
                  "$ref": "https://www.some-url.dev/api/data?cursor=eyJvZmZzZXQiOjUwLCJsaW1pdCI6NTB9"
           }
   }

the bundle/lint also tries to resolve those references - which is basically wrong, as they are part of the response, not part of the spec!
In our case, with the pagination cursor being part of the response, that resulted in an (potentially indefinite) loop and the redocly-cli was requesting those API requests from our staging server, paginating over and over...
Note: using the configuration setting resolve.doNotResolveExamples: true does not help here, because it will then not embed our example response into the bundled file, because it resolves not even the first encountered $ref, wich is referring to our local example file.
Expected behavior
If a $ref in an example json file is encountered, do not try to resolve it.
Or: add a configuration like doNotRecursivelyResolveExamples or config something like recursiveExampleResolvingDepth: 1 to prevent that behaviour.
Additional context
Redocly cli version used:  1.0.0-beta.126

I wonder if this can be handled through a different way, a combination of the config:
resolve:
  doNotResolveExamples: true

And a preprocessor or decorator which actually resolves the first $ref in the example.

@wuuti what OpenAPI version is that?

what OpenAPI version is that?

that is 3.1

@christianseel for OAS 3.1, it's possible (and recommended) to use examples instead of example. Then you should be able to use $ref as a literal value like in the following example:
examples:
  MyExample:
    value:
      $ref: '../../dir/otherdir/example.data.get.json' # Will not be resolved by linter

More details on $refs (particularly in examples) here.
NB: There's currently a discrepancy between the behaviour of the linter and the bundler regarding this matter (here's the corresponding issue: https://github.com/Redocly/redocly-cli/issues/1371).

We should resolve refs that point to examples, but not refs inside example values, I think? @tatomyr I think your summary is very good, and that #1371 is definitely a related issue to this one.

authors: adamaltman christianseel lornajane tatomyr wuuti

repo: Redocly/redocly-cli


text:
mouse click handler crash by singularity detector

crash-2024-06-01_13.18.33-client.txt

Ok, I think I found the issue. I'll hopefully fix it soon. :)


text:
When starting in offline disable uploader

Or at least have a default that is NOT production

I would actually prefer the current behaviour. I basically develop in offline mode all the time and I need to check the upload to my private (local) server. So i usually use -o noop and -p http://localhost/myapi

A possible solution is when processing an offline file only send to configured private servers and not any configured public servers.

authors: Regner oxisto

repo: Regner/albiondata-client


text:
MVC 6 rewrite errors instead of redirect

Hi I have a little issue.
I though the error pages would rewrite instead of redirect, but when I get a 404 error I get redirected to /error/404.
I think google search craweler might have an issue with this as it is seeying a page as permanently moved instead of not available.

Are you using Azure? this issue may be related.

The httpErrors section has PassThrough set for existingResponse in Debug mode (which causes a redirect but is required for some debug functionality like Glimpse to work) but when you publish (Not build) in Release mode it sets the value to Replace instead (See Web.Release.config).

Merging your fix.

authors: RehanSaeed jhermsen


text:
Thank you!

Hi!
Thank you for great work.

Hha, thanks for your interests to our work


text:
Jenkins Pipeline Development

This project will be used to run my jobs in CI/CD pipeline.

This project will be used to run my jobs in CI/CD pipeline.

repo: RemTech/Angular


text:
Update isValidName regex to fix ISSUE-12

https://github.com/RemiRigal/Yape/issues/12
Regex was taken from https://stackoverflow.com/a/3824105

bump

Thank you for your contribution @joshuajcoronado ! Unfortunately, as the options page has been refactored this is no longer compatible with the current state of the extension. However I've taken your changes into consideration, so thanks again.

authors: RemiRigal joshuajcoronado

repo: RemiRigal/Yape


text:
Icon request: hashtag

Icon Request

Icon name: hashtag
Tags of this icon: tag, hash
Use case: before header or tag
Screenshots of similar icons:

收到~会在后续版本更新。
不过，v1.2.1版本的图标内容已经封版了，在后续几天会发布（目前还有一些bug没处理完）。所以 # 图标会更新在v1.2.2版本中。一般一个版本的发布时间大概在2-3周。

@xiaochunjimmy It's your repo and your rules, but I think it would be easier if we could keep it to english?
For me, the above says:

...v.1.2.1...v1.2.2...2-3

@jenstornell Yeah, you are right.
But what the words above are not important. He just wanted to tell me that he would do it in next releases.
And he found we are both from China, so maybe he thought it would be more convenient for me.

@jenstornell Hahaha, thanks for your reminder. Sorry for Non-Chinese friends.

Sorry for a long time waiting.
You can find it on our website remixicon.com now.
感谢云游君的长期支持 😊

authors: YunYouJun jenstornell xiaochunjimmy

repo: Remix-Design/RemixIcon


text: 大型动物图床


text:
feat: minecraft brigadier argument types

Lamp currently supports arguments provided by Brigadier library directly. There are, however more Minecraft-specific arguments included in game internals. All of them are handled by the client to support completions etc. (see wiki)
They can be accessed with a bit of reflection or NMS on supported platforms. Do note there is a Paper PR (https://github.com/PaperMC/Paper/pull/6695) open to expose them directly to the API which may, or may not be merged in the future.
Notes

Some existing arguments would need to be handled differently to mimic their brigadier parser behavior.

Lamp uses StringArgumentType for String which expects validation (see this)
EntitySelector<T> should refer to internal minecraft:entity type to support rich client-side completions.

Supporting some arguments (eg. minecraft:vec2 and vec3) could be a bit hacky and would most likely require internal changes.

Thoughts
Since that have more side effects than it sounds, I think that addition of BrigadierCommandHandler (or similar) is necessary. This would keep current users unaffected by these changes and also make brigadier easier to maintain.
Sponge
I know nothing about Sponge and can't really suggest anything in that regards. I can only say that since all (minecraft) platforms can use Brigadier, maybe a new module structure is needed. (example)
MinecraftCommandHandler<T> extends CommandHandler<T> { ... }
BrigadierCommandHandler<T> extends MinecraftCommandHandler<T> { ... }

MinecraftCommandHandler<Bukkit> handler = ... // for simple commands
BrigadierCommandHandler<Bukkit> handler = ... // for brigadier commands

MinecraftCommandHandler<Sponge> handler = ... // for simple commands
BrigadierCommandHandler<Sponge> handler = ... // for brigadier commands

Lamp already uses some reflection (through commodore) to access custom Minecraft argument types, EntitySelector being one example. Unfortunately, as you pointed out, arguments like Vec2 and Vec3 are not supported by the nature of Lamp, therefore they could be challenging to add, and simply registering them would serve no purpose.

This is super weird because last time I tried, I couldn't get EntitySelector to play nicely with completions. I'll try it once more in the coming days since it's most likely an issue on my own. Thanks for pointing that out!

arguments like Vec2 and Vec3 are not supported by the nature of Lamp, therefore they could be challenging to add
That's totally understandable, their syntax is pretty complex and could be complicated to handle, I agree.

Although, what do you think about drawing a line between Bukkit arguments and Brigadier arguments? They're both different in some ways and sometimes you want them to be handled differently from each other (like the String one I've pointed out before). I can't really think of a good structure for that stuff sunce I don't know Lamp much.
I'd really like the ability to override existing, or even register my own brigadier argument handlers with a type reference for the client (eg. brigadier:long or minecraft:namespace)

Should be fixed. Thank you for the report.

authors: Grabsky Revxrsal

repo: Revxrsal/Lamp


text:
Create Frog Sensor Installation Documentation

We need to create a documentation note about where to place the Frog Sensor once someone is ready to install it.

Done!


text:
IMPORTANT: Added Apache License

I added the Apache licence because we still were missing a license at all. If anyone has a more restrictive wish about how we should license our work, we should discuss it asap.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 27

See the complete overview on Codacy

Detto francamente: fanculo Codacy per questa volta. E' la licenza, non mi metto a modificarla.


text:
Resolves  RicoSuter/NJsonSchema#1322 - Added uuid format for schema validation

Created new uuid format validator
Added testing for format

Resolves RicoSuter/NJsonSchema#1322

Thanks for the PR.

authors: RicoSuter martin-shields-sage

repo: RicoSuter/NJsonSchema


text:
Added another overload to addURLToDownload which allows the ripper to…

… set the name of the file; Fixed 8muses filename length issue
Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[X] a bug fix (Fix #369)

Description
I Added an overload to addURLToDownload which allows rippers to set the file name and Fixed 8muses filename length issue
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

The test that failed was for furaffinity and it seems to have no been caused by these changes

repo: RipMeApp/ripme


text:
Cleanup custom tables, enable LinkToInstance

Makes Maid call any custom Disconnect or Destroy function within a table
Enable LinkToInstance (Makes Maid cleanup when Instance is Destroyed)
Removed __call metamethod hotfix, however any table passed to Maid without Disconnect or Destroy will be called like a function

Credit to Corecii for the instanceDestroyed Module

The InstanceDestroyed Module would be a nice Library to add to RoStrap's mainstream recommendations and could be a dependency of Maid (rather than just pasting the code straight in).
Also, the InstanceDestroyed Module / LinkToInstance function has a dependency of FastSpawn, seen below:
local QuickRobloxThreadSpawner = Instance.new("BindableEvent")
QuickRobloxThreadSpawner.Event:Connect(ChangedFunction)
QuickRobloxThreadSpawner:Fire(Reference.Value, Reference.Value.Parent)
QuickRobloxThreadSpawner:Destroy()

If these modules are added, they should call from each other, rather than having duplicated code.

Support.

authors: Validark devSparkle

repo: RoStrap/Events


text:
Receive should not pick receive points in the defence area's during normal play.

It does do this. Can probably just pick points on the line segment parts that are close but not in the defence area.

maybe we need a 'projectInSafeArea' control util?

authors: mrlukasbos rolfvdhulst


text:
Expand firmware example selection

Splitting issue #12 into this issue, which is adding additional firmware (rather than just having ANY firmware)
Bluegiga BLE112
Bluegiga BLE113
Bluegiga BLE121LR
SiLabs BGM111
SiLabs BGM113
Cypress EZ-BLE
Nordic 52 series
Anaren BLE (Broadcom WICED stack, or Anaren stack)

Probably not going to do this - lots of work, minimal return.

repo: RobotPajamas/Blueteeth


text:
Building on windows: locks up on sphinx build to read_config

It unpacks the tar for my chosen version and  gets to "partially running sphinx-build to read configuration for: master".
Then it blocks-waiting on sphinx_.py : read_config, in the child.join() invocation. (Could this do with a timeout in there to help it clean up after failure?)
From what I can see in the tmp directories, the .doctrees look like they're all made correctly (assuming this is what's meant to happen?). Sphinx reports: sources read 100%.
I realise that it's not returning after the call into the Sphinx api, so not sure how much you can do to fix what's going. Let me know if I can send log files/config/etc to help out.
Same results on Windows native or via cygwin.

I've got an issue tracking Windows support: https://github.com/Robpol86/sphinxcontrib-versioning/issues/3

Thanks for the quick response. Hit me up if you need testing help!

I changed a lot of stuff around multiprocessing which involved read_config() in the latest release. Windows is now officially supported. Let me know if you still run into this problem.
Thanks for reporting it.


text:
Prevent BagDiscoverer routine from running over itself

Is your feature request related to a problem? Please describe.
When BagDiscoverer runs over a large bag, it runs past the apache timeout and restarts over itself. This can compound and eat up resources.
Describe the solution you'd like
Implement a check on whether BagDiscoverer is already running. There are possibly additional solutions, but one is to save the bag to the database much earlier in the process, and add an additional processing status.

Another solution is to change the s3 downloader to create a database object and then base the BagDiscoverer routine off of the BaseRoutine.

@helrond I've been working on the solution to create a database object in the S3 download service, and am running into problems with Asterism's Bag model. bag_identifier and origin are required. Bag identifier is easy enough to move up assignment, but in order to discern origin we need to unpack the TAR file first. An option here is to create a default origin but that could be confusing if something breaks in a weird spot?

repo: RockefellerArchiveCenter/zorya


text:
Run Rocket chat on devices with API 17 or higher

I downloaded Rocket.Chat.Android (Kotlin) but minSdk of this code is on 21 but I need to run it on API 17 or higher.

Hi @sealiasheq We explained the reason here. Thanks.

authors: filipedelimabrito sealiasheq


text:
improved subfolder-documentation

After spending half a day in frustration before finally getting rocket.chat running in a subfolder, I felt that I should share my findings with others.
Thus I improved the - to that point not very helpful - documentation about subfolder-support.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@Iridias thanks for contributing these improvements.  Hopefully this will help others.  Left a couple of small notes

@geekgonecrazy Can you review it again, please?

@Iridias thanks!  Last thing is to click through and review the CLA and we'll be good to merge :+1:

authors: CLAassistant Iridias geekgonecrazy rafaelks


text:
Broken layout for macOS when toggled server list

Description:
The layout for the macOS client has a broken layout with Rocket.chat server version 0.59.3:

Server Setup Information:
Version of Rocket.Chat Server: 0.59.3
Operating System: macOS 10.13.1
Deployment Method(snap/docker/tar/etc):
Number of Running Instances:
DB Replicaset Oplog:
Node Version:
Steps to Reproduce:
On macOS click VIEW -> toggle server list
The view looks perfect with opened server list:

Hey @JannikZed, can you confirm which version of Rocket.Chat+ are you using? The electron version 2.10.1 already fixed this with the sidebar redesign.

Yep, updating to the newest version fixed the bug! Thank you.

authors: JannikZed gdelavald


text:
Wrong data field onSubmit react-select

Description of bug
I need submit Form on onBlur event of the field, but on react-select not working correctly. When I click on select field value is empty, but on second click is filled with previous value. See the gif bellow for information.

To Reproduce
Problem simulation gif
Screenshots
Select implementation
Select implementation continuos
Select usage

Hello @brunoJSX,
I made an example on codesandbox (check here) and it's working.
Check and let me updated if it works or not to look another solution.

authors: brunoJSX carlosallexandre

repo: Rocketseat/unform


text:
Rich presence API?

Will the new rich presence API be available here?
https://discordapp.com/rich-presence
https://discordapp.com/developers/docs/rich-presence/how-to
Best regards

Rich Presence being added to Discord.Net.Rpc would be nice. Without it, the Rpc lib will be incomplete, and not to mention the official RPC SDK wraps an unmanaged library. I personally feel it would be easier for .NET/Unity game developers (like me) to integrate Rich Presence into their game, and I might consider opening a PR regarding this.
Dylan P
HOO @ Ultz

authors: DurtyFree DylanFPS


text:
Update Fallout Equestria Dead Tree Studio PNP.html (calculation fix)

Updated Potency calculation

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[x] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


text:
New Feature - Ego Save added

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Ego Save added to the sheet, small increase to note text area size.

Character Sheet Info Roll20 Internal Use only.


text:
Starfinder HUD v0.041523636721

Bug Fixes

Needler Pistol added to internal compendium
weapon range no longer defaults to 5, and is now applied from the compendium
Statblock importer improved to accomodate more complex statblocks. More improvements coming in the future.
Logging of statblocks added for easier debugging of statblock issues
Incorrect addition of "+" to numerical attributes.

Feature Changes

The internal compendium is now accessed by a text search instead of a drop down. Search requires complete words that you are looking for. It will return the smallest subset of items that match any one term in the search box.

Known Issues

The statblock importer does not currently handle statblocks with multiple spell lists or multiple sources of spell like abilities. A fix is coming for this. Until the fix is done, simply combine all spell lists into one, and all spell like ability lists into one.
Weapons that have special properties (e.g. explode or burst) need to have their special properties at the start of the parenthesized data, and any commas within the parentheses need to be replaced with semicolons. Example:
utility scattergun +5 (blast 1d4+1 P) or shock grenade I +5 (explode [15 ft., 1d8 E, DC 12])
needs to be changed to:
utility scattergun +5 (blast; 1d4+1 P) or shock grenade I +5 (explode 15 ft. DC 12; 1d8 E)

@wing-it, You're gonna hate me, but the fix that you merged this morning had another issue in it (the incorrect addition of the "+" referenced in the change notes here). Any chance of a quick merge before the other one goes live?


text:
[HeXXen 1733][Shadowrun5th] Add new System and fixes

Changes / Comments
Shadowrun 5th used different colors for text. It looks like I forgot to create a pull request last year.
HeXXen 1733 was not available so far. So I created it.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[yes] Does the pull request title have the sheet name(s)? Include each sheet name.
[no] Is this a bug fix?
[yes] Does this add functional enhancements (new features or extending existing features) ?
-- It adds HeXXen 1733 as new System
[yes] Does this add or change functional aesthetics (such as layout or color scheme) ?
-- Shadowrun 5th used different colors for text, this one has been unified with better contrast to improve readability
[no] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[yes] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

The comments about Shadowrun 5E seems redundant as the PR doesn't edit any files for it. And Roll20 prefer people make separate PRs for each sheet, and would likely reject a PR that edits more than one sheet.

Hi @Anduh ,
that's interesting. I thought already that I did the changes a year ago. But github told me that it's a commit that's not merged.
If it doesn't change anything. Then it won't hurt, hopefully.

authors: Anduh pantherophis-bln


text:
New Character Sheet for See You In Hell Game

Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


text:
[Shadowrun5thA] calculate drain/nethits for rituals and spirits

Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

before merging this take a look at https://github.com/Roll20/roll20-character-sheets/pull/9702


text:
🛑 STEP is down

In 3754502, STEP (https://step.polymtl.ca/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: STEP is back up in 9d62f6d.


text:
Add Hari Kuduva to Contributors list

title says it all.
Great tutorial, thank you.

@sriharikuduva, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

authors: Roshanjossey sriharikuduva

repo: Roshanjossey/first-contributions


text:
adding vedabrat to list

Adding myself to the contributers list.

Hi  @vedabrat, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

authors: Roshanjossey vedabrat


text:
Strings

How do I add the base64 strings in the loader section? Do I add the base64 at the end of the url? I cant figure it out I am lost. Settings.ini is the file. I figured out the rest that is the only thing I don't get. I am new to injecting in this way.

hey c4retro, you find a way to fix crypted injection?

repo: Roshly/HyperHook


text:
Fix Selenium click error (on Ubuntu 22.04)

There were errors for lots of more or less recent homeworks.
The download part crashed with an error like:
Traceback (most recent call last):
  File "{$HOME}/.pyenv/versions/3.9.10/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "{$HOME}/.pyenv/versions/3.9.10/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "*/prpr/prpr/main.py", line 213, in <module>
    main()
  File "*/prpr/prpr/main.py", line 160, in main
    for results, homework in zip(
  File "*/prpr/prpr/download.py", line 88, in download_batch
    urls = _get_zip_urls(driver, homework.revisor_url)
  File "*/prpr/prpr/download.py", line 209, in _get_zip_urls
    element.click()
  File "{$HOME}/.pyenv/versions/3.9.10/envs/yapr-prpr/lib/python3.9/site-packages/selenium/webdriver/remote/webelement.py", line 80, in click
    self._execute(Command.CLICK_ELEMENT)
  File "{$HOME}/.pyenv/versions/3.9.10/envs/yapr-prpr/lib/python3.9/site-packages/selenium/webdriver/remote/webelement.py", line 633, in _execute
    return self._parent.execute(command, params)
  File "{$HOME}/.pyenv/versions/3.9.10/envs/yapr-prpr/lib/python3.9/site-packages/selenium/webdriver/remote/webdriver.py", line 321, in execute
    self.error_handler.check_response(response)
  File "{$HOME}/.pyenv/versions/3.9.10/envs/yapr-prpr/lib/python3.9/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)

selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it

I'll attach couple of examples of {homework_number}.{iteration} {cohort} to make possible some suggestions on the content version on the platform.
See also the thread in Pachca (limited access): https://app.pachca.com/chats?thread_id=1086996
OS: Ubuntu 22.04

The earliest fail I met on the 19.05.2023.
Here are some of the earliest examples:
25.05: 7.2 на 60 когорте
selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it

7.1 61
selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it

13.1 51
selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it

26.05: 10.2 57 - success

And the latest examples include:
21.06: 12.2 57 success
23.06: 1.3 69
selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it

12.3 19+
selenium.common.exceptions.ElementClickInterceptedException: Message: Element
 is not clickable at point (811,266) because another element
 obscures it


text:
🛑 Website (rubenfixit.com) is down

In 827ea59, Website (rubenfixit.com) (https://rubenfixit.com) was down:

HTTP code: 503
Response time: 536 ms

Resolved: Website (rubenfixit.com) is back up in 4ed3f00 after 2 hours.

repo: RubenFixit/upptime


text:
Refactor to update sre-engine to 0.4.1

replace #4033

@youknowone can you please publish and merge?

repo: RustPython/RustPython


text:
Fix benchmarks by replacing rust-cpython with pyo3

The benchmarks seem to have been broken since Python 3.10 deprecated the API
they were using to parse and execute CPython baselines. Since then
rust-cpython has been deprecated in favor of pyo3.

Replace cpython and python3-sys dependencies with pyo3.
Add html_report feature to criterion, it will be required in a
future release.
Remove anyhow. It was unused and cargo cleaned it up automatically.
Use compile and exec from within the CPython context to
compile and execute benchmarks. There's probably more overhead to that than
the internal API had, but that overhead should be consistent per
benchmark.

If anyone cares about hyperoptimizing benchmarks then they can optimize
the harness as well.
Fixes #3498

Use compile and exec from within the CPython context to
compile and execute benchmarks.  There's probably more overhead to that than the internal API had, but that overhead should be consistent per benchmark.

I have a minor question: How do we know that this won't inadvertently give RustPython an advantage? As far as I know, the API we used (from before CPython 3.10) ensured that compilation was kept separate from execution.

Also, if I recall correctly, the RustPython benchmarks will eat up a lot of memory because RustPython presently has no garbage collection more advanced than simple reference-counting and needs circular references in order to set up simple things like object and type.

When I originally tried to run the benchmarks they just failed to build because of features deprecated in Python 3.10, so my understanding was that they just haven't worked since the project moved past that point. My objective with this PR was to get something working so that I could benchmark some changes I was making.
From my cursory investigation, I couldn't find that pyo3 exposed a similar API to what was there previously. I think the difference between calling the CPython exec and hooking into whatever function it uses internally is small enough for now. If maintaining parity is important I can dig deeper.
I'm bummed about garbage collection too, but I don't see that it should block improving the benchmarking code.

The reason I warned about GC is so that you're not surprised if the benchmarks crash due to your system not having enough RAM. (From what I recall, RustPython chews through about 16 GB of RAM.)

Thanks for the warning! That is probably why I crashed my computer a few times testing this PR lol

authors: dchiquito fanninpm


text:
使用wechat_kit-3.1.0版本无法调起微信支付

监听到的PayResp：{
"errorCode": -1,
"errorMsg": null,
"returnKey": null
}
wechat_kit: ^3.1.0
wechat_kit_extension: ^1.0.0+1

在安卓手机和苹果手机中都无法调起

解决了吗

@sbiweeq 升级到3.2.0

authors: FLEasy sbiweeq


text:
[WIP] New Rx style

Changes rx_action to rx.action.
Fixes #50.
WIP because setting is currently borked. Consider:
button.rx.action = Action(...)

This produces an error cannot assign to property 'rx' is get-only. rx is a get-only property that is a struct value-type, which is causing the compiler to mix up. There's probably an easy way around this, I'm going to look through the RxSwift code to find it.

Hmm, no easy solutions here since we can't mutate the value returned from .rx. Which is a bit silly since this works:
var rx = subject.rx
rx.action = action

The value returned from .rx is computed, created fresh every time and serves as a proxy. There are a few solutions to this, but the ones I think of all require changing RxSwift, which is asking a lot. Solutions include:

Making Reactive (the .rx type) a class.
Making .rx property { get set } where the setter does nothing.

Both of these kind of suck. @bontoJR @kzaher do either of you have suggestions here?

I'm going to open an issue on RxSwift to discuss this further, it's a real head-scratcher.

@ashfurrow  +1, had a similar problem recently. Would be awesome if we manage to make it work.

Hi guys,
yeah, like @ashfurrow said, there are some workarounds for this. Probably the lest intrusive is making rx { get set }.
If this was c++, I think you could override = operator :)
There are other solutions like:

using a setter rx.setAction(action)
defining a custom operator rx.action operator action

... but yeah, non of them is particularly nice.

@kzaher your first proposition is, as you say, the less intrusive one. I ran a quick test an rx could be defined { get set } and then set defined as  set { self.rx }.
This said, when running tests I noticed that if you do:
obj.rx.action = someAction
The code from the set implementation gets called twice. Once to obtain the rx object then follow up on the action object, and a pair of set/ get on Rx after that:
testing rx.action
rx.{get} called
A.{set} called with value = 5
rx.{set} called
rx.{get} called

This can lead to performance issues (also I should move this discussion to RxSwift :))

@fpillet that would be awesome if you could open an issue, I'll try to get to it tomorrow if you don't beat me to it 👍

@ashfurrow @fpillet
I've pushed the necessary changes (added setter on rx) to develop branch. Hoping to release rc tomorrow.

Wow, thanks!

@ashfurrow I updated the Action code, demo and tests for finalize this update. All tests now pass, demo works and we should be good to roll for a first RC update.

authors: ashfurrow fpillet kzaher sunshinejr

repo: RxSwiftCommunity/Action


text:
Happy to help

Hey, I've been trying to fix MathJax preambles in Obsidian for a while! If you need any help with this plugin, please do let me know!

Thanks! To be honest, I don't know about MathJax very much so your help will be definitely appreciated.
And I do have a problem right now. Do you have any ideas on how to "cancel" an already-defined command? I wonder if this can be achieved by a more sophisticated approach than the current one.
The current approach of this plugin is very naive.
Namely, before every markdown post-processing or CodeMirror's update), this plugin

Determine which preamble file to load based on the current note path
Check if the preamble to be loaded is different from the previous one
If so, it just executes renderMath(preamble, false), which is equivalent to MathJax.tex2chtml(preamble, {display: false}).

This is super simple, but this way we can switch the definition of a single command (e.g. \P{...}) between two (e.g. \mathbb{P} \left( ... \right) and \mathrm{Pr} \left[ ... \right]) or more based on note paths.
But the problem is that this approach doesn't allow us to cancel a command that has been already defined by a preamble but not contained in another one. For example,
% preamble 1: applied to note1.md
\newcommand{\foo}{\mathbb{FOO}}
\newcommand{\bar}{\mathbb{BAR}}

% preamble 2: applied to note2.md
\newcommand{\foo}{\mathbf{FOO}}

I want \bar not to work in note2.md because preamble 2 doesn't contain it, but with the current approach, it does work both note1.md & note2.md if we load note1.md first.
Do you have any suggestions?

Also I'd appreciate any other suggestions not necessarily related to the point above.

For me, the main things I would like to do are:

Finer control over preambles
Changing maths delimiters so that it can be consistent with pandoc etc.
Be able to inject custom startupReady commands etc
Be able to inject custom extensions

I think points 2-4 are literally only possible with a more exposed MathJax. I'm no expert but I've tried hacking the config of the existing MathJax object and wasn't able to really get anywhere with those points. I guess point 1 is more relevant to this discussion.
I'm not actually aware of how to remove commands, it appears to be the big issue with this kind of approach. I've asked a question on the MathJax github itself in case anyone can point us in the right direction. I'll let you know is anything helpful comes up. The only alternative approach I can see is loading your own MathJax instance but I'm not sure how Obsidian would let you do that. Does it let you override the MathJax object? There isn't sandboxing so this may be possible and would actually help with points 2-4. I've not tried to do this inside a plugin to overwrite MathJax so please let me know how it goes if you try it. I may also try it if I find the time soon.

Finer control over preambles

Do you have any specific features that you want to have other than switching preambles from note to note?

Changing maths delimiters so that it can be consistent with pandoc etc.

This is extremely difficult because it's not only MathJax but Obsidian's markdown parsers that determine what delimiters to use.
(Indeed, you can find this in Obsidian's app.js:
            var iA = Db("/lib/mathjax/tex-chtml-full.js", {
                before: function () {
                    window.MathJax = {
                        tex: {
                            inlineMath: [],
                            displayMath: [],
                            processEscapes: !1,
                            processEnvironments: !1,
                            processRefs: !1
                        },
                        startup: {
                            typeset: !1
                        },
                        options: {
                            enableMenu: !1,
                            menuOptions: {
                                settings: {
                                    renderer: "CHTML"
                                }
                            },
                            renderActions: {
                                assistiveMml: []
                            },
                            safeOptions: {
                                safeProtocols: {
                                    http: !0,
                                    https: !0,
                                    file: !0,
                                    javascript: !1,
                                    data: !1
                                }
                            }
                        }
                    },
                        localStorage.removeItem("MathJax-Menu-Settings")
                }
            });

Notice inlineMath: [], displayMath: [].)
And here's a bad news. Live preview seems to use HyperMD, and its documentation clearly says this:

math
🎨 Type: boolean   📦 Default: true
Parse TeX formula wrapped by $ or $$.
🚩 Example:
Inline Formula: $\LaTeX$ and $$y=kx+b$$
Display Formula:
$$
\begin{bmatrix}
\cos \theta       &&     -\sin \theta    \
\sin \theta       &&      \cos \theta
\end{bmatrix}
$$

And to make matters worse, reading view uses a different parser from live preview and I don't know what it is. And if I knew, it would be hard to modify how it works.

I've asked a question on the MathJax github itself in case anyone can point us in the right direction. I'll let you know is anything helpful comes up.

Wow thanks! Much appreciated.
FYI, I think some parts of MathJax's config can be changed afterward, as I mentioned in this post on the forum:
https://forum.obsidian.md/t/links-from-within-math-blocks/69050/4?u=ush

The only alternative approach I can see is loading your own MathJax instance but I'm not sure how Obsidian would let you do that.

I think some plugins do that.

Excalidraw
TikZJax

Does it let you override the MathJax object?

I'm not sure if we can overwrite the global MathJax object without any issues, but we can at least modify what the renderMath function does by monkey-patching MathJax.tex2chtml. It's what I did in my plugin called Auto-\displaystyle Inline Math.
https://github.com/RyotaUshio/obsidian-auto-displaystyle-inline-math/blob/e080e89b465b71e2c473263f0593ee2348bebdc9/main.ts#L51-L60

Hey, I've succeeded in loading another MathJax object!
https://github.com/RyotaUshio/obsidian-mathjax-preamble-manager/blob/519b247f5bf1830398b9a292393534f6349af3e1/src/manager.ts#L32-L71

And now I can remove commands as well!
https://github.com/RyotaUshio/obsidian-mathjax-preamble-manager/assets/72342591/f576a23b-80df-4945-9336-399a47d37b6c

Now we can modify the MathJax config as well.
For example, the default config doesn't allow us to use javascript: protocol, and tags are displayed on the right side, but we can do this:
https://github.com/RyotaUshio/obsidian-mathjax-preamble-manager/assets/72342591/cf6a36eb-158c-4d74-972e-498428cbdcca

Be able to inject custom startupReady commands etc

Is this what you wanted?
https://github.com/RyotaUshio/obsidian-mathjax-preamble-manager/assets/72342591/944168b2-6acf-4f76-b830-7492cf3a84fb

authors: RyotaUshio mayurankv


text:
검색결과 목록 내 숨겨진 항목들의 자리에 빈 공간이 생기는  문제

우선 좋은 필터 덕분에 잘 사용하고 있습니다 감사합니다. 👍
크리티컬한 문제는 아니지만, 필터에 걸려 숨겨진 엘리먼트 상위 엘리먼트 (ul)에 패딩이 들어있어서 검색결과 항목간 빈 공간(갭)이 크게 떠보이는 문제가 있습니다. 타겟팅 필터 조건을 상위 엘리먼트까지 포함하게 만들어 갭을 없앨 수 있으면 더 깔끔할 것 같습니다.
사용환경은 Safari/Firefox용 AdGuard입니다. 양쪽다 동일하게 나타납니다.
모든 경우에 다 이런건 아니고,

이런식으로 동일 사이트내 하위 검색결과가 트리 형태로 나타나는 식에만 ul li 형태의 껍데기가 추가로 붙어서 생기는 문제가 아닐까 싶기도 합니다.

반영하였습니다.

authors: RyuaNerin zvuc

repo: RyuaNerin/only-stackoverflow


text:
Filter counts and results count after clicking filter.. don't match

For example, filtering on the "Temperature" keyword

https://catalogue.saeon.ac.za/graphql
{
  catalogue {
    summary(
      limit: 75
      filterByIds: []
      filterByText: ""
      fields: [
        {
          id: "keywords-filter"
          field: "subjects.subject.raw"
          path: "subjects"
          filters: [
            {
              field: "subjects.subjectScheme.raw"
              includeIfMissingField: true
              values: ["SANS1878 keywordType general"]
            }
          ]
          sortBy: "_count"
          sortOrder: "desc"
        }
      ]
      filterByTerms: [{ field: "subjects.subject.raw", value: "temperature" }]
    )
  }
}

That query gives a doc count of 759 records that include temperature. But that's is because the aggregation includes the fields.filters.
The records search on the other hand, doesn't include the filters resulting in slightly more records being returned.
In other words. This filter should be passed to the query builder as well

The best implementation for now is just to indicate that the facet counts may be lower than the document counts returned

Once example where this is still a problem is searching the 'southern ocean location' on the dev site. more records appear than are indicated. I think this is because the aggregations are not taking into account nested documents whereas the query now is.

Sometimes the counts in the facets don't match the search results because the keywords are tagged with the incorrect subject scheme. Looking at this - https://catalogue.saeon.ac.za/records/10.15493/DEA.MIMS.26052335

My last bit of work for this year was looking into why facet counts on the catalogue don't always match the number of documents returned. The list of facets is built by filtering the subjects object, and sometimes subjects have the same value for different subjectSchemes. But the document search just finds all documents with the clicked subject. So occasionally the documents returned are estimated higher than the facet count suggests.
The number of results is correct and can be relied on. The facet aggregation is a little more complex and while it's usually correct... can sometimes be a little under what it should.

repo: SAEON/data-portal


text:
CAP ex.4 - Error "Cannot find module 'sqlite3'"

Hello,
I am trying to finish ex. 4 of SAP Cloud Application Programming Model (CAP).
But whenever I try to click on "Books" at service web page I get an error.

The error in Business Application Studio is

GET /catalog/Books
[2020-11-21T20:08:46.513Z | ERROR | 1557126]: { Error: Cannot find module 'sqlite3'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at _loadSqlite (/extbin/npm/globals/lib/node_modules/@sap/cds-runtime/lib/sqlite/Service.js:9:15)
at Promise (/extbin/npm/globals/lib/node_modules/@sap/cds-runtime/lib/sqlite/Service.js:31:20)
at new Promise ()
at _new (/extbin/npm/globals/lib/node_modules/@sap/cds-runtime/lib/sqlite/Service.js:30:10)
at SQLiteDatabase.acquire (/extbin/npm/globals/lib/node_modules/@sap/cds-runtime/lib/sqlite/Service.js:156:19)
at SQLiteDatabase. (/extbin/npm/globals/lib/node_modules/@sap/cds-runtime/lib/sqlite/Service.js:110:29) code: 'MODULE_NOT_FOUND' }

I have installed sqlite and deployed db.
npm install sqlite3 -D
cds deploy --to sqlite:bookshop.db
I created db connection and connected to it without problem.
I think I followed instructions of this and previous exercises and I don't know what else to do.
Thank you!

Hi @dasast,
could you please upload your project as a repo to GitHub so I can run it myself?
Cheers,
Max

Hi,
I have decided to try lessons one more time. I deleted my dev space and created new one.
But this time I wasn't successful even with deploying.
npm install sqlite3 -D   ok
cds deploy --to sqlite:bookshop.db ended with error

I created repo https://github.com/dasast/cap_sqlite_error1  with tar.gz file with downloaded dev space. But is almost an initial one.
Thanks for help.

HI @dasast,
the repository is unfortunately completely empty.
Regards,
Max

Sorry, I probably forgot to commit it. Now it should be OK.
Dasa

Hm, that's interesting.
I tried npm install followed by npm start and it works for me - can you please try that once again as well?

Well, yes npm install followed by npm start works, but that is not where the problem is.
Until the database is deployed, "Books" link returns error "No database connection". So I need to use database.
npm install followed by cds deploy still doesn't work.
And when I earlier was successful with cds deploy, the  "Books" link returned error "MODULE_NOT_FOUND'

Good morning @maxstreifeneder and @dasast  - please permit me to join this discussion.
I am guessing from the screenshots you shared, @dasast , that you're doing this in the SAP Business Application Studio - is this correct? Assuming it is, I just tried it, with your tar and gzipped file in that repo you created, and it worked fine. Here's the output of a session just now:
user: user $ cd projects/
user: projects $ curl -L https://github.com/dasast/cap_sqlite_error1/raw/main/workspaces-ws-cl4f7.tar.gz | tar xzf -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   156  100   156    0     0   5571      0 --:--:-- --:--:-- --:--:--  5571
100 6231k  100 6231k    0     0  3458k      0  0:00:01  0:00:01 --:--:-- 7297k
user: projects $ ls
bookshop
user: projects $ cd bookshop/
user: bookshop $ npm install
npm WARN lifecycle The node binary used for scripts is /extbin/bin/node but npm is using /opt/nodejs/node-v10.22.1-linux-x64/bin/node itself. Use the `--scripts-prepend-node-path` option to include the path for the node binary npm was executed with.

> @sap-cloud-sdk/core@1.32.1 postinstall /home/user/projects/bookshop/node_modules/@sap-cloud-sdk/core
> node usage-analytics.js

added 137 packages from 106 contributors and audited 257 packages in 4.074s

5 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

user: bookshop $ cds deploy --to sqlite:bookshop.db
/> successfully deployed to ./bookshop.db

/> updated ./package.json
user: bookshop $ npm start
npm WARN lifecycle The node binary used for scripts is /extbin/bin/node but npm is using /opt/nodejs/node-v10.22.1-linux-x64/bin/node itself. Use the `--scripts-prepend-node-path` option to include the path for the node binary npm was executed with.

> bookshop@1.0.0 start /home/user/projects/bookshop
> npx cds run

[cds] - model loaded from 2 file(s):

  db/schema.cds
  srv/catalog-service.cds

[cds] - connect to db > sqlite { database: 'bookshop.db' }
[cds] - serving CatalogService { at: '/catalog' }

[cds] - launched in: 557.845ms
[cds] - server listening on { url: 'http://localhost:4004' }
[ terminate with ^C ]

and here's the successful retrieval of (no) books:
user: user $ curl http://localhost:4004/catalog/Books
{"@odata.context":"$metadata#Books","value":[]}
user: user $

I am therefore wondering whether you have the right tools installed into your Dev Space - did you start with the "SAP Cloud Business Application" type of Dev Space when you create it? Can you please try this flow (above) and let us know what happens?

Hello @qmacro,
you are right, I am working in the SAP Business Application Studio as described in Devtoberfest2020 Week 3 SAP Cloud Application Programming Model (CAP).
I am using my trial account, dev space of type SAP Cloud Business Application.

I have tried it  and I was successful !
The main reason is that you use command npm install which probably installs more packages than in the video SAP Cloud Application Programming Model - Ex. 03 - Associations & adding persistence used command npm install sqlite3 -D.
After npm install I was already successful with command cds deploy --to sqlite:bookshop.db
And also the "Books" link already works.
Thanks very much, I am very happy than I can continue with CAP  lessons.
Dasa

Hey Dasa, thanks, that's great news! Happy hacking!

For me unsetting node path before running CDS deploy worked
run "npm install sqlite3 -D"
run "unset NODE_PATH"
run "cds deploy --to sqlite"

authors: dasast maxstreifeneder qmacro vikjdk7