When compare locations, only path is compared, params are ignored

As you can see in TurboNavRule
    private fun locationPathsAreEqual(first: String?, second: String?): Boolean {
        if (first == null || second == null) {
            return false
        }

        return URI(first).path == URI(second).path
    }

This will cause unexpected pop presentation.  Shouldn't params be compared too? Since params also reflect page state.

Can you describe the scenario where this is causing issues in your app? I'd like a better understanding, to see if our apps differ in how we use query params. Currently, if the location paths are identical, but the query params are different, we'll REPLACE the current destination with the new destination.
We use query params sparingly in our apps, essentially only using them to filter or sort the display of items on a screen. But I can see how a liberal use of query params might lead to undesired behavior with the current logic.

First I visit  /ledgers, the ledger show page(displaying the first ledger id=1)
then I visit /ledgers/list, then I click on a link /ledgers?id=2, expect to visit /ledgers?id=2, but a pop presentation loaded.
D/TurboLog: visitProposedToLocation ........... [session: main, location: https://localhost.dev/ledgers?ledger_id=2, options: TurboVisitOptions(action=ADVANCE, snapshotHTML=null, response=null)]
D/TurboLog: shouldNavigateToLocation .......... [session: main, location: https://localhost.dev/ledgers?ledger_id=2, shouldNavigate: true, fragment: WebHomeFragment]
D/TurboLog: navigate .......................... [session: main, location: https://localhost.dev/ledgers?ledger_id=2, options: TurboVisitOptions(action=ADVANCE, snapshotHTML=null, response=null), currentContext: DEFAULT, newContext: DEFAULT, presentation: POP, fragment: WebHomeFragment]
D/TurboLog: navigateWithinContext ............. [session: main, location: https://localhost.dev/ledgers?ledger_id=2, presentation: POP, fragment: WebHomeFragment]
D/TurboLog: fragment.onStop ................... [session: main, location: https://localhost.dev/ledgers/list, fragment: WebHomeFragment]
D/TurboLog: fragment.onActivityCreated ........ [session: main, location: https://localhost.dev/ledgers, fragment: WebFragment]
D/TurboLog: fragment.onStart .................. [session: main, location: https://localhost.dev/ledgers, fragment: WebFragment]
D/TurboLog: visitLocation ..................... [session: main, location: https://localhost.dev/ledgers, options: TurboVisitOptions(action=RESTORE, snapshotHTML=null, response=null), restorationIdentifier: 84b63bd4-1225-4bc9-8606-8c401d78c143]
D/TurboLog: visitStarted ...................... [session: main, location: https://localhost.dev/ledgers, visitIdentifier: 5c9be816-84a2-4336-b923-68903b67d67d, visitHasCachedSnapshot: true]
D/TurboLog: visitRendered ..................... [session: main, visitIdentifier: 5c9be816-84a2-4336-b923-68903b67d67d]

I knew the route is weird, but it shouldn't cause a problem?

I agree that this is a bug @fffx , having similar problems on a project myself.

@fffx @jwark I've changed the default behavior in #178, but have added a new path property configuration option to restore the previous behavior for any desired paths.

@fffx @jwark I've changed the default behavior in #178, but have added a new path property configuration option to restore the previous behavior for any desired paths.

Thanks @jayohms for putting in the effort to get this fixed and for letting us know, I really appreciate it. :)


Added connection timeout

This PR add a connectionTimeout option for the simpleSignalClient.
Related: https://github.com/t-mullen/simple-signal/issues/23
It throws a timeout error when a connect process or an accept request takes too long to happen.
Also, I add a way to detect a premature close of the peer and if that happen it should reject the connect/request promise.

Sweet! LGTM.


Составить список эвристик для разбора

TODO: декомпозировать на задачи?
Взять за основу список доступных для включения в решателе SCIP heuristics primal heuristics LNS RINS DINS Fix and  Propagate etc
Первым шагом я бы предложил разобрать их алгоритмически, не реализовывая самим.
Список доступных эвристик SCIP

heuristic
description

actconsdiving
LP diving heuristic that chooses fixings w.r.t. the active constraints

adaptivediving
diving heuristic that selects adaptively between the existing, public divesets

advanced
advanced parameters

alns
Large neighborhood search heuristic that orchestrates the popular neighborhoods Local Branching, RINS, RENS, DINS etc.

bound
heuristic which fixes all integer variables to a bound and solves the remaining LP

clique
LNS heuristic using a clique partition to restrict the search neighborhood

coefdiving
LP diving heuristic that chooses fixings w.r.t. the matrix coefficients

completesol
primal heuristic trying to complete given partial solutions

conflictdiving
LP diving heuristic that chooses fixings w.r.t. conflict locks

crossover
LNS heuristic that fixes all variables that are identic in a couple of solutions

dins
distance induced neighborhood search by Ghosh

distributiondiving
Diving heuristic that chooses fixings w.r.t. changes in the solution density

dps
primal heuristic for decomposable MIPs

dualval
primal heuristic using dual values

emphasis
predefined parameter settings

farkasdiving
LP diving heuristic that tries to construct a Farkas-proof

feaspump
objective feasibility pump 2.0

fixandinfer
iteratively fixes variables and propagates inferences

fracdiving
LP diving heuristic that chooses fixings w.r.t. the fractionalities

gins
gins works on k-neighborhood in a variable-constraint graph

guideddiving
LP diving heuristic that chooses fixings in direction of incumbent solutions

indicator
indicator heuristic to create feasible solutions from values for indicator variables

intdiving
LP diving heuristic that fixes binary variables with large LP value to one

intshifting
LP rounding heuristic with infeasibility recovering and final LP solving

linesearchdiving
LP diving heuristic that chooses fixings following the line from root solution to current solution

localbranching
local branching heuristic by Fischetti and Lodi

locks
heuristic that fixes variables based on their rounding locks

lpface
LNS heuristic that searches the optimal LP face inside a sub-MIP

mpec
regularization heuristic for convex and nonconvex MINLPs

multistart
multistart heuristic for convex and nonconvex MINLPs

mutation
mutation heuristic randomly fixing variables

nlpdiving
NLP diving heuristic that chooses fixings w.r.t. the fractionalities

objpscostdiving
LP diving heuristic that changes variable's objective values instead of bounds, using pseudo costs as guide

octane
octane primal heuristic for pure {0;1}-problems based on Balas et al.

ofins
primal heuristic for reoptimization, objective function induced neighborhood search

oneopt
1-opt heuristic which tries to improve setting of single integer variables

padm
penalty alternating direction method primal heuristic

proximity
heuristic trying to improve the incumbent by an auxiliary proximity objective function

pscostdiving
LP diving heuristic that chooses fixings w.r.t. the pseudo cost values

randrounding
fast LP rounding heuristic

rens
LNS exploring fractional neighborhood of relaxation's optimum

reoptsols
primal heuristic updating solutions found in a previous optimization round

repair
tries to repair a primal infeasible solution

rins
relaxation induced neighborhood search by Danna, Rothberg, and Le Pape

rootsoldiving
LP diving heuristic that changes variable's objective values using root LP solution as guide

rounding
LP rounding heuristic with infeasibility recovering

shiftandpropagate
Pre-root heuristic to expand an auxiliary branch-and-bound tree and apply propagation techniques

shifting
LP rounding heuristic with infeasibility recovering also using continuous variables

simplerounding
simple and fast LP rounding heuristic

subnlp
primal heuristic that performs a local search in an NLP after fixing integer variables and presolving

trivial
start heuristic which tries some trivial solutions

trivialnegation
negate solution entries if an objective coefficient changes the sign, enters or leaves the objective.

trustregion
LNS heuristic for Benders' decomposition based on trust region methods

trysol
try solution heuristic

twoopt
primal heuristic to improve incumbent solution by flipping pairs of variables

undercover
solves a sub-CIP determined by a set covering approach

vbounds
LNS heuristic uses the variable lower and upper bounds to determine the search neighborhood

veclendiving
LP diving heuristic that rounds variables with long column vectors

zeroobj
heuristic trying to solve the problem without objective

zirounding
LP rounding heuristic as suggested by C. Wallace taking row slacks and bounds into account

https://www.researchgate.net/publication/258846101_Primal_Heuristics_for_Mixed_Integer_Programs


Code snippet in docs unexpectedly throws

Bug Report
The docs present an example code snippet which crashes the js process. See https://reactjs.org/docs/refs-and-the-dom.html#adding-a-ref-to-a-dom-element
In particular, notice the highlighted parts of the code snippet and the first sentence under the snipppet, which reads "React will call the ref callback with the DOM element when the component mounts, and call it with null when it unmounts.".

If the ref callback is called with null, the callback will be doing this.textInput = null;.
From that moment on, if a user clicks the second input (the button), the focusTextInput method will do this.textInput.focus();.
But since this.textInput is null, it's really doing this.null.focus();.
Yet focus is not a method of null, so this will throw

I would suggest adding a little guard for this in the code snippet so that people can copy-paste without worry, replacing this:
this.textInput.focus();

with this:
this.textInput && this.textInput.focus();

or something like this.
This was found in the docs for React v16.0.0

@shawninder are you planning on submitting a PR for this?

I don't think this is actually a problem, is it?
The code in focusTextInput would only run in response to the input onClick handler, which wouldn't be invoked after an unmount (b'c the input would have also been unmounted).
Null checks before accessing refs are a good idea if the code in question might execute before a mount or after an unmount (eg as the result of a timer, promise, or network request) but I don't think it's actually a problem in this case.
Maybe I'm misunderstanding this? 😄

Hmm I guess you are right @bvaughn, the problem I mention would only occur if you place the two inputs in different components, allowing the button to be mounted while the other input is not. So no, @Ethan-Arrowood, I will not submit a PR for this.

No worries at all. I appreciate you bringing this to our attention, just in case. 👍


secure schema.org

Summary
did a find/replace of all "http://schema.org" to make it https.
Context
Related to https://github.com/mmistakes/minimal-mistakes/pull/1969#issuecomment-441875298
jbr@Rummels-MBPr ~/D/GIT> find minimal-mistakes -type f -exec grep "http://schema.org" "{}" \; -print | grep minimal
minimal-mistakes/_includes/author-profile.html
minimal-mistakes/_includes/seo.html
minimal-mistakes/_includes/search/algolia-search-scripts.html
minimal-mistakes/_includes/archive-single.html
minimal-mistakes/_includes/comment.html
minimal-mistakes/_layouts/single.html
minimal-mistakes/_layouts/splash.html
minimal-mistakes/assets/js/lunr/lunr-gr.js
minimal-mistakes/assets/js/lunr/lunr-en.js
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_includes/author-profile.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_includes/seo.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_includes/search/algolia-search-scripts.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_includes/archive-single.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_includes/comment.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_layouts/single.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/_layouts/splash.html
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/assets/js/lunr/lunr-gr.js
jbr@Rummels-MBPr ~/D/GIT> sed -i '' 's|http://schema.org|https://schema.org|g' minimal-mistakes/assets/js/lunr/lunr-en.js
jbr@Rummels-MBPr ~/D/GIT> find minimal-mistakes -type f -exec grep "http://schema.org" "{}" \; -print | grep minimal
jbr@Rummels-MBPr ~/D/GIT> find minimal-mistakes -type f -exec grep "https://schema.org" "{}" \; -print | grep minimal
minimal-mistakes/_includes/author-profile.html
minimal-mistakes/_includes/seo.html
minimal-mistakes/_includes/breadcrumbs.html
minimal-mistakes/_includes/search/algolia-search-scripts.html
minimal-mistakes/_includes/archive-single.html
minimal-mistakes/_includes/comment.html
minimal-mistakes/_layouts/single.html
minimal-mistakes/_layouts/splash.html
minimal-mistakes/assets/js/lunr/lunr-gr.js
minimal-mistakes/assets/js/lunr/lunr-en.js

Thanks @justinrummel. Everything looks good to me.


[BUG] Fix Cards in About Page

Description of the bug
The cards in the about page are off alignment and makes the UI look bad, hence I will like to fix this issue
Please add screenshots (if applicable)

as you can see the cfp, events and conference are out of alignment.
Add any other context about the problem here
No response
Code of Conduct

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

This issue has already been assigned to another contributor and you can look into other issues.


A list of injustices

We've been curating a list here:
https://docs.google.com/document/d/163tK5z1Pvh31ixQbkk3ID6alY4x3Ss9RVc8tYthNE6o/edit#heading=h.umptjdrym7r1
Post it on the website.

We now have a resource page. Integrate the Google Doc above.


Kernel dies/fails to connect on Windows with Scilab 6.0.1

On Windows there is no scilab-adv-cli, so I tried setting SCILAB_EXECUTABLE to C:\Path\To\Scilab\bin\scilab.bat or to C:\Path\To\Scilab\bin\scilab.bat -nw (-nw option must have similar effect of scilab-adv-cli on MacOS and Linux): Scilab starts (with a visible window without -nw option), but Jupyter doesn't connect to it.

Hi @mobius-eng, I plan to add support for Scilab 6.0 next week.


stop accidently blowing away user supplied options

resolves #165
there's probably a cleverer way to do this, but this change ensures that we pass through user supplied options to geosearchCore when the default provider is utilized.

thx @nixta!


PA2 DecreDepth doesnt work properly

Summary
Description
I get this as an output even though i alredy increased and decreased after every ack
Idea s
thanks for the help!
Screenshots (if any):

Looks like you are on the right track, can take a look at your FuncStat implementation?

Looks like you are on the right track, can take a look at your FuncStat implementation?

my funcstat looks like this

purpose of decreDepth() should be to negate the shorter side of the tree. subtracting by 1 does not negate that side of the tree if it branches more than once.


Fix/28094 - Move 2FA steps from context provider to routes

Details

Move 2FA steps from context provider to routes.
Fixed Issues

$ https://github.com/Expensify/App/issues/28094
PROPOSAL: https://github.com/Expensify/App/issues/28094#issuecomment-1732636744
Tests

Click on Settings > Security > Two-factor authentication
Click on copy and next
Click on the system back button
Verify that the user is on Codes step

Click on Settings > Security > Two-factor authentication
Click on copy and next
Scan QR or add secret key
Input six digit code
Click on the system back button
Verify that the user is on Security page

Click on Settings > Security > Two-factor authentication
Click on copy and next
Scan QR or add secret key
Input six digit code
Click Got it button on success screen
Click on the system back button
Verify that the user is on Security page

Verify user can't directly go to /settings/security/two-factor-auth/verify
Verify user can't directly go to /settings/security/two-factor-auth/success
If 2FA not enabled, verify user can't directly go to /settings/security/two-factor-auth/enabled
If 2FA enabled, verify user can't directly go to /settings/security/two-factor-auth/disabled

[x] Verify that no errors appear in the JS console

Offline tests

Same as above.
QA Steps

Same as above.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/22430210/c4f19fef-ccf4-4ed7-b429-c6640f8cf654

Mobile Web - Chrome
https://github.com/Expensify/App/assets/22430210/cd460605-d7cf-4b62-bc1e-1ccba04e5d44

Mobile Web - Safari
https://github.com/Expensify/App/assets/22430210/0a8d2a03-880d-4b62-ad9f-3534eb617d5b

Desktop
https://github.com/Expensify/App/assets/22430210/d6125a8e-724c-406e-afeb-bcff8472abf3

iOS
https://github.com/Expensify/App/assets/22430210/55829ae4-f766-4264-b552-17f595627d8e
Warning was there before changes.

Android

https://github.com/Expensify/App/assets/22430210/fffff63b-7055-4807-9f2c-04a8a76a6933

I have read the CLA Document and I hereby sign the CLA

@robertKozik I just realised we can remove AnimatedStepProvider from everything since it's using routes. Mind if I update the PR with changes?

Go for it! @bojangabric

@robertKozik Removed it, and checked every platform, looks good. Also removed some constants and styles that were used only in these components.

I'm stepping into it right now, sorry for the extended delay on my end 🙇🏼
First of all, can you extend the test steps to cover all affected screens? So we will be sure that whole 2FA process is working as intended.

@bojangabric bump

@robertKozik I've pushed a fix for preventing direct access to deeplinks.

Yeah, I was thinking if user should be able to do that or not. User can only access verify directly if he clicked copy codes some time earlier.
I'll remove it, and I'll fix the input focus.

I believe that if we require the user to copy before proceeding further in normal flow, we should block it accordingly on deep links. Btw we have conflicts with main as well

Hi @bojangabric did you manage to tackle focus problem?

Hey @robertKozik, I did, but still looking into how to properly fix directly accessing deeplinks. Every solution I came up with I didn't like, it all felt like a hacky workaround.

Added TwoFactorAuthActions.setCodesAreCopied(false) to Expensify component, definitely not a good place to put it. Just wanted to push it so you can check out if it's working properly.
By the way, when I merged main into my branch, the input for verify went all the way to the bottom of the screen. Is this expected, or should it be fixed?

Added TwoFactorAuthActions.setCodesAreCopied(false) to Expensify component, definitely not a good place to put it. Just wanted to push it so you can check out if it's working properly.
By the way, when I merged main into my branch, the input for verify went all the way to the bottom of the screen. Is this expected, or should it be fixed?

It's expected - the same behaviour is occurring on staging.
I see the problem with the verify step. It's not worth fixing if it's such a hustle - as on staging user can refresh and still be in second step

It's expected - the same behaviour is occurring on staging.
I see the problem with the verify step. It's not worth fixing if it's such a hustle - as on staging user can refresh and still be in second step

I've merged main to this branch and removed code for fixing direct access to verify step.

Closing because one route was intentional https://github.com/Expensify/App/issues/28094#issuecomment-1807928210


[WIP] Major API and Packaging refactor

Very close to completion. Needs:

make global vars import as native symbols (not convenience vars)
make global vars (and function names) cache and change by hooks

Closes #4


Few-shot examples support?

Thanks for this refreshing take on LLM prompt generation and evaluation, it's very promising.
I was wondering if few-shot examples should have their own first-class support in BAML due to their powerful influence on teaching the model both syntax and semantics simultaneously?
I acknowledge that it's currently possible to use Jijna + messages to inject few-shot examples, but seeing first-class support for test cases made me think, why not few-shot? The few-shot examples provided are indeed usable both as test cases and few shot examples in the prompt. In fact, this is what we do for validating our few-shot dataset, is by doing a k-fold split where we pass the train split as few-shot examples, and the test split as test cases.

(I combed through the docs to see if there is a way to "instantiate" BAML classes into an array and return them from "FewShotExamples()" function, but not sure yet whether it's possible or not. Sorry if it is and I missed it.)

Ignore the bot, it s gone crazy. We dont have a specific syntax yet for it, but it is on our roadmap. We currently dont have a timeline yet unfortunately!
Thanks for the suggestion!

The way it can be done at the moment is to pass the examples as an argument to the function and render the examples in the prompt.
So they are basically constructed on the python or Ts side

Have had a few times where some special treatment for few-shot prompt engineering would be helpful. Looking forward to this feature!


Fatal error after git clone and npm run pull

When I clone the repo and run npm run pull, I get a fatal error after gulp runs in the master branch.
To reproduce:

git clone git@github.com:JDRF/design-system.git
cd design-system
npm run pull

Screenshot:

npm-debug.log:

@briansaycocie What's your version of Node and NPM?

@fuhton node v5.5.0, npm v3.5.3

I'd suggest updating node, npm, and global gulp to the latest version and retest.

I've updated to node v6.2.0, npm v3.8.9, and gulp v3.9.1. Same result :(

Figured out the issue. The latest version (v0.4.0) of gulp-scss-lint threw a fatal error but did not specific what the issue was. I reverted it back to v0.3.9 and ran npm run pull. It failed at the same spot, except this time I got a helpful error message: Error: You need to have Ruby and scss-lint gem installed.
Running gem install ruby && gem install scss-lint fixed it for me.


not sure what this means but i go into a join loop

05.10 16:44:03 [Server] Server thread/ERROR Could not pass event PlayerJoinEvent to Multiverse-Core v2.4-b527
05.10 16:44:03 [Server] INFO org.bukkit.event.EventException: null
05.10 16:44:03 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:319) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:589) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:576) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.PlayerList.a(PlayerList.java:200) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.LoginListener.c(LoginListener.java:153) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.LoginListener.tick(LoginListener.java:52) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.NetworkManager.a(NetworkManager.java:219) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.ServerConnection.c(ServerConnection.java:130) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.MinecraftServer.b(MinecraftServer.java:1126) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.DedicatedServer.b(DedicatedServer.java:352) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.MinecraftServer.a(MinecraftServer.java:1007) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.MinecraftServer.w(MinecraftServer.java:846) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at net.minecraft.server.v1_16_R2.MinecraftServer.lambda$0(MinecraftServer.java:164) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO at java.lang.Thread.run(Thread.java:748) [?:1.8.0_242]
05.10 16:44:03 [Server] INFO Caused by: java.lang.NullPointerException
05.10 16:44:03 [Server] INFO at com.onarandombox.MultiverseCore.listeners.MVPlayerListener.playerJoin(MVPlayerListener.java:138) ~[?:?]
05.10 16:44:03 [Server] INFO at sun.reflect.GeneratedMethodAccessor111.invoke(Unknown Source) ~[?:?]
05.10 16:44:03 [Server] INFO at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_242]
05.10 16:44:03 [Server] INFO at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_242]
05.10 16:44:03 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:315) ~[spigot-1.16.2.jar:git-Spigot-b5a13e6-0d8350a]
05.10 16:44:03 [Server] INFO ... 14 more
05.10 16:44:03 [Connect] User Ytsharpbelt636,
05.10 16:44:03 [PebbleHost] Ytsharpbelt636 ran command Message of the Day
05.10 16:44:03 [Server] User Authenticator #64/INFO UUID of player Ytsharpbelt636 is b4d23fa0-5544-4d28-a085-2fc52685bade
05.10 16:44:12 [Server] Server thread/INFO CURRENT: 0
05.10 16:44:12 [Server] Server thread/INFO PLAYER: 0

You are using a version of mv. get latest from here: http://ci.onarandombox.com/view/Multiverse/

nope, the log shows v2.4-b527, the latest is v4.1.1-b796

nope, the log shows v2.4-b527, the latest is v4.1.1-b796

i updated mv core im still in the join loop but says nothing much just 05.10 17:14:00 [PebbleHost] Ytsharpbelt636 ran command Message of the Day
05.10 17:14:01 [Server] User Authenticator #79/INFO UUID of player Ytsharpbelt636 is b4d23fa0-5544-4d28-a085-2fc52685bade
05.10 17:14:01 [Disconnect] User Ytsharpbelt636 has disconnected, reason: Disconnected
05.10 17:14:01 [Server] Server thread/INFO CURRENT: 0
05.10 17:14:01 [Server] Server thread/INFO PLAYER: 0

There is no errors with mv now in your logs. Only error I see its Could not pass event PlayerJoinEvent to StaffPlus which means a possible issue with your StaffPlus plugin then. I don't see any issue with MV.
I suppose things you can try:

Get the latest paper /spigot build
start your server with only mv installed


jams.trim()

Do we have any functionality for trimming jams files? I couldn't find any, so wrote a quick example of what trimming might look like for a pitch_midi annotation:
def trim_jams_pitch(input_jams_path, output_jams_path, start_t, end_t):

    jams_in = jams.load(input_jams_path)

    if end_t > jams_in.file_metadata.duration:
        warnings.warn("End time provided (end_t) is greater than the duration of the "
                      "audio file.")

    onsets = []
    offsets = []
    pitches = []
    confidences = []

    # get pitch annotation (we assume there is only 1)
    pitch_ann = jams_in.annotations[0]

    # find all notes that are within the range and trim them appropriately
    for t, d, p, c in zip(pitch_ann.data.time, pitch_ann.data.duration, pitch_ann.data.value, pitch_ann.data.confidence):
        onset = t.total_seconds()
        offset = onset + d.total_seconds()

        if onset < end_t and offset > start_t: # this should catch all overlapping notes
            t_on = max(onset, start_t)
            t_off = min(offset, end_t)
            onsets.append(t_on)
            offsets.append(t_off)
            pitches.append(p)
            confidences.append(c)

    # Create a new annotation with the trimmed notes
    trimmed_ann = jams.Annotation('pitch_midi')
    for onset, offset, pitch, conf in zip(onsets, offsets, pitches):
        trimmed_ann.append(time=onset, duration=(offset-onset), value=pitch, confidence=conf)

    # Create new jams object to store new annotation
    new_jam = jams.JAMS(annotations=[trimmed_ann])

    # Copy over file metadata, change duration field
    new_jam.file_metadata = jams_in.file_metadata
    new_jam.file_metadata.duration = end_t - start_t

    # Save new jam file
    new_jam.save(output_jams_path)

We could probably write a generic trimmer for any annotation namespace, or if that looks too convoluted we could have a per-namespace trimmer.
Thoughts?

this is implemented now, ya? close now or next minor release?

Addressed by PR #136, closing out.
Note: discussion led to the implementation of two functions:

trim: only keep observations within a specified time range, leaving their start times unchanged
slice: similar to trim, but time axis reset such that the start time of the trim range is mapped to time 0 in the new sliced annotations.


Add Proxy

Hey, Is there any way to add proxy?

Hi @sairash!
You can do this by setting custom transport to client, like this example:
    proxyURL, _ := url.Parse("http://localhost:8080")

    client := fastshot.NewClient("https://api.example.com").
        Config().SetCustomTransport(&http.Transport{
             Proxy: http.ProxyURL(proxyURL),
        }).
        Build()

@sairash, the new version v0.8.0 has the simplified solution for define proxy:
client := fastshot.NewClient("https://api.example.com").
    Config().SetProxy("http://my-proxy-server:port").
    Build()


no Go source files always appear after trying build the project

Hello, i have problem when trying build the project
make && make install
   Compiling hyper-timeout v0.4.1
   Compiling in-toto v0.3.0 (https://github.com/Xynnn007/in-toto-rs.git?rev=7f69799#7f69799d)
   Compiling git2 v0.15.0
   Compiling tonic v0.8.3
   Compiling shadow-rs v0.19.0
   Compiling attestation-service v0.1.0 (/home/yaoxin/attestation-service)
error: failed to run custom build command for `attestation-service v0.1.0 (/home/yaoxin/attestation-service)`

Caused by:
  process didn't exit successfully: `/home/yaoxin/attestation-service/target/release/build/attestation-service-415e692c8a604cbc/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-changed=/home/yaoxin/attestation-service/target/release/build/attestation-service-7acaabdf8a946b93/out
  cargo:rustc-link-search=native=/home/yaoxin/attestation-service/target/release/build/attestation-service-7acaabdf8a946b93/out
  cargo:rustc-link-lib=static=cgo

  --- stderr
  ERROR: build github.com/open-policy-agent/opa/capabilities: cannot load github.com/open-policy-agent/opa/capabilities: no Go source files

warning: build failed, waiting for other jobs to finish...
make: *** [Makefile:22: grpc-as] Error 101

The error above always appears.  Could you help me to resolve it?

We no longer use the Go OPA engine, so I think we can close this. Please re-open if you still have this problem.


archivemount: 0.8.3 -> 0.8.7

[x] built on NixOS
[x] ran binary on NixOS (version output matches 0.8.7)

@GrahamcOfBorg build archivemount

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘archivemount-0.8.7’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/tools/filesystems/archivemount/default.nix:18 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
make[1]: Leaving directory '/build/archivemount-0.8.7'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7
shrinking /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7/bin/archivemount
gzipping man pages under /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7/share/man/
strip is /nix/store/xmpjypwjmp2qi1chs5kr0hacnh161ls4-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7/bin
patching script interpreter paths in /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7
checking for references to /build in /nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7...
/nix/store/4ainpharjklz8rzys3v4hycwcf3qgp4f-archivemount-0.8.7

Success on x86_64-linux (full log)
Partial log (click to expand)
make[1]: Leaving directory '/tmp/nix-build-archivemount-0.8.7.drv-0/archivemount-0.8.7'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7
shrinking /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7/bin/archivemount
gzipping man pages under /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7/share/man/
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7/bin
patching script interpreter paths in /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7
checking for references to /tmp/nix-build-archivemount-0.8.7.drv-0 in /nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7...
/nix/store/m2a6qjaii7xfaw8w5kjx4dgamriayw8k-archivemount-0.8.7


Add screenshots or screencasts to the main README.md

It might be a personal preference but it could be useful to see really quickly what an extension looks like, even before clicking on it.
We should keep the table of contents, but right below it there could be a list of screencasts (in the same order / structure)

Fixed by #44.


how to provide independent scope for each svg component?

in two svg file ,all define the same id ! but then later id cannot working ! so how to provider independent scope for each svg component ?

id MUST be unique in all html page (or encapsutate svg in iframe)
you cannot violate this.
BUT in order to avoid problem you can use config param propsMap to revrite id to another propertyname :
 example :
  propsMap: {
                     fillRule: 'fill-rule',
                     id: 'data-id'
                 },

ok, have solved this problem with svgo-loader.


[Bug]: Beam SQL Extension raised an error when the input row contained iterable fields

What happened?
Upon utilizing the Beam SQL within our pipeline, we encountered a noteworthy exception. It appears that when the input row encompasses fields of iterable types, the process fails irrespective of whether the iterable fields are included in the SQL filter condition or not. This issue stems from the fact that the Beam SQL extension attempts to construct an output row schema based on the input schema, and unfortunately, it currently lacks support for iterable types.
Consider the following example:
Given an inputRow in the schema:
field1: String
field2: Integer
field3: Array<String>
field4: ITERABLE

And ANY  Beam SQL condition such as :
field2 > 1
or Even
1 = 1
The pipeline will invariably fail, yielding the following error: Exception in thread "main" java.lang.UnsupportedOperationException: Unable to get ITERABLE at org.apache.beam.sdk.extensions.sql.impl.rel.BeamCalcRel$InputGetterImpl.getBeamField(BeamCalcRel.java:603).
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Create Matic Faucet for the Mumbai Network

ETH Frontend (MVP site first):

Build a button thats able to request "MATIC" on Mumbai Polygon from the smart contracts used by Polygon Faucet or build a faucet yourself!
If the above is not possible; see if you are able to console.log() the wallet address when connecting to your site with the metamask wallet and use the wallet address to send a request to the polygon faucet for however many MATIC; should not require a on chain tx
list it on https://app.honey.finance/playground/mumbai temporarily

A bounty has been minted for this issue at https://app.openq.dev/bounty/I_kwDOHB1JPc5L_866/0x8B7Bccd58D9c14384882B18D0698260b0f4A2b1A


Add Async UDFs

Add a new 'AsyncMapOperator' that applies an async udf to its input.
Async UDFs use the existing interface for defining the UDF and are called in SQL the same as non-async UDFs. Options are specified as a TOML configuration block in the special comment with the dependencies.
Here's an example of an Async UDF:
/*
[dependencies]
tokio = { version = "1", features = ["full"] }
tokio-postgres = "0.7"

[udfs]
async_results_ordered = true
async_timeout_seconds = 5
async_max_concurrency = 10

*/

use tokio_postgres::{NoTls, Error};

pub async fn user_name_from_id(id: i64) -> String {
    let conn_str = "host=localhost user=arroyo password=arroyo dbname=my_db";

    let (client, connection) = tokio_postgres::connect(conn_str, NoTls).await.unwrap();

    tokio::spawn(async move {
        if let Err(e) = connection.await {
            println!("connection error: {}", e);
        }
    });

    let rows = client
        .query("SELECT name FROM users WHERE id = $1", &[&id])
        .await
        .unwrap();

    return if let Some(row) = rows.get(0) {
        row.get(0)
    } else {
        format!("Unknown-{}", id)
    };
}

which can be used in a query like this:
select user_name_from_id(person.id) as name from nexmark where person is not null;

I've left these things for a future commit:

Retries: if the UDF panics we should allow retrying it certain number of times
Async UDF Context: A trait that users can implement to perform setup and shutdown behavior, that each UDF invocation will have access to

This query is panicking in the planner:
create table logs (
    ip TEXT,
    city TEXT GENERATED ALWAYS AS (get_city(ip))
) with (
    connector = 'sse',
    endpoint = 'http://<IP_ADDRESS>:9563/sse',
    format = 'json'
);

select city
from logs;

with
2024-01-11T23:52:08.418511Z ERROR arroyo_server_common: panicked at arroyo-sql/src/expressions.rs:269:21:
async udf appeared in a non-projection context panic.file="arroyo-sql/src/expressions.rs" panic.line=269 panic.column=21
2024-01-11T23:52:08.418876Z  WARN arroyo_api::pipelines: failed to generate SQL program

Another panic if you try to group by an async udf:
select
    get_city(logs.ip) as city,
    count(*)
from logs
group by 1;

2024-01-12T00:06:33.748816Z ERROR arroyo_server_common: panicked at arroyo-sql/src/expressions.rs:269:21:
async udf appeared in a non-projection context panic.file="arroyo-sql/src/expressions.rs" panic.line=269 panic.column=21
2024-01-12T00:06:33.749935Z  WARN arroyo_api::pipelines: failed to generate SQL program

This is really great! I'm so excited to see how people use this.


[QA-117]: Adding SPOT perf test scripts

QA-117
What?
Added the SPOT perf test k6 script
Changes:

Added environment variable for SPOT SQS queue in CF template.
Added the SPOT sample request template.
Added k6 script to send messages to the SPOT SQS queue using the request template.

Why?
SPOT happy path scenario script added in the performance test repo.

Related:

Old IPV Perf Test Repo

With this new folder, you should also update the .github/CODEOWNERS file

With this new folder, you should also update the .github/CODEOWNERS file

Done


Illuminaparser Error

Hello, I am having an issue when running the following command on a .bam file from cellranger v2.0
time java -Xmx15000m -jar /home/user/sicelore/Jar/IlluminaParser-1.0.jar -i /home/user/Downloads/possorted_genome_bam.bam -t /home/user/data/scdm/barcodes.tsv -o /home/user/data/javasc/output/par.obj -b CB -u UB -g GN
The error:
Exception in thread "main" java.lang.ClassCastException: class java.lang.Character cannot be cast to class java.lang.String (java.lang.Character and java.lang.String are in module java.base of loader 'bootstrap') 	at com.rw.bamparsermaven.Parser.parse(Parser.java:127) 	at com.rw.bamparsermaven.Main.main(Main.java:99)
Also, here are few lines from the .bam file I am using:
`NB501663:146:HVFCCBGX5:2:11102:20995:14314	16	1	10486	0	14M124804N84M	*	0	0	GC[...]C	6/////E<E/A//EE//E//A//EAA/EE/EEAEEAEA/EE<AEEEEEEEEEEEEEEEEEAEEE/EEEEEEEEEEEEEEEEEEEEEEEEEEEEAAAAA	NH:i:5	HI:i:1	AS:i:84	nM:i:5	RE:A:I	CR:Z:TTCTTAGGTCAATACC	CY:Z:AAAAAEEEEEEEEEEE	CB:Z:TTCTTAGGTCAATACC-1	UR:Z:TCTTCCAGGA	UY:Z:EEEEEEEEEE	UB:Z:TCTTCCAGGA	RG:Z:count_GRCh38:MissingLibrary:1:HVFCCBGX5:2
`
Any idea what could be causing this and how to fix it?
I realise that the bam file is missing the GN tag,  but was expecting the software to recognize the missing tag. Is this the problem, or could it be something else?

The explanation for the “java.lang.ClassCastException: class java.lang.Character cannot be cast to class java.lang.String” is, that you have one or several BAM entries that have a gene name that has just one character (quite unusual).
A missing GN tag is no problem.
We updated the software and uploaded a new version (IlluminaParser-1.002.jar) that won’t throw an exception if a one character gene name is encountered.

Great, works now! Thank you


Create doc and example recipe for the TokenSelector

This PR adds documentation for my new component, TokenSelector, implemented in scaffold-eth/scaffold-eth-2#822. It adds the documentation for the component, along with a recipe demonstrating how to customize the button by creating a Uniswap-like button that shows the currently selected token.
I'm not sure if a documentation page for useToken is necessary, as it is mostly self-explanatory, so I have not added it; I'd be more than willing to add it if needed.

Checkout : https://github.com/scaffold-eth/scaffold-eth-2/pull/822#issuecomment-2069255793


Mention docker-compose file bundle in Containerization guide

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2339
Originally created by @DouweM on 2020-09-03 20:34:46

See https://gitlab.com/meltano/files-docker-compose/-/merge_requests/2#note_406954306

View 2 previous comments from the original issue on GitLab


Add instance, volume, network-interface as resource tag defaults

what

Add instance, volume, network-interface as resource tag defaults

why

Reasonable tagging defaults

references

https://github.com/cloudposse/terraform-aws-eks-node-group/issues/131#issuecomment-1319252027
Other module also contains this as a default

/test all


Server Security

Secure the ES administration and/or adapt submission code appropriately.

So the first part securing elk on submission port is done.
curl -XDELETE https://elk.liguros.net:9200/funtest
401 Unauthorized

Which methods are actually needed by the reporter? I have HEAD, GET, POST, PUT enabled.
The second part is to secure kibana frontend.

we are currently using POST only :
my $response = $http->request( 'POST', $url, \%options );

I might have a solution for kibana frontend. Take a look at https://liguros.net
I am embedding an iframe.
Could this work? Or is it just security/obscurity?

iframe is just a webpage shown through a window. It is just obscurity. For the browser to view the iframe contents, they must also be able to view the page within the frame.. even separately.

OK, so for now Kibana is secured and ES submission should not allow DELETE method.


[Announcement] Apollo's official blog is launching on Medium.com! Subscribe it today!

Dear Apollo Developers,
We are proud to announce that Apollo's official blog is launching on Medium today at medium.com/apollo-auto. Read our first blog post: Deep Dive on Apollo 2.5. Subscribe us if you'd like to be notified for future posts!
We start this blog to better serve our developer community. New articles will be posted 1-2 times per month, and it will focus on various technical topics that will help our developers to better understand and use Apollo. Please leave a comment here if you have particular topics that you would like us to cover, and vote for your favorite topics!
Yours,
Zhenni
Community Manager | Apollo Platform

I want apollo team to have a meet up in Japan.

Hi @poutyface ， can you please send me an email at<EMAIL_ADDRESS>where we can connect and discuss the potential of possibly doing a meet up in Japan? :)

Happy to see there are more and more activities both online and offline.
@Zhenni17 , you might know that from PRC, we cannot access medium.com, so could you also have a counterpart of that for PRC local users?

Hi @Peter9606 , thank you for pointing out that information. I actually was not aware of that issue. I will work with our China team to publish the blog on our official WeChat account. Hope that will help! Please let me know if you have any additional question.


refactor: make obvious that checkpoint_id is required

Describe your changes
We've currently got old deprecated code paths and things will be simpler if we clear them up. As far as I can see the worker is always sending a checkpoint ID to a snapshotting container.
 Backward/forward compatibility checks

Check these boxes or delete any item (or this section) if not relevant for this PR.

[x] Client+Server: this change is compatible with old servers

Old workers are already sending the checkpoint ID and have been for a while.

[x] Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.

Oh nice this breaks tests. Will fix.


hopefully final deduping, added consulting to all service categories

What changed

combined hospital and surgical equipment (46500 | 47500), changed from general services to medical services and public safety categories
added consulting to all service categories

Closes #97

Does it make sense for all of the consulting services to have the same examples?

@bsmithgall:
2 options:

Remove "consulting services" from all broad categories, and keep the examples associated with it in human-friendly categories that make more sense (ex: "urban planning consulting" is in "planning, development, and surveying services").
Differentiate "consulting services" associated with each broad category by removing examples (ex: for "consulting services" under "Events and recreation...", remove examples that don't have to do with events and recreation).  This sounds like the more tedious option, so I recommend going with the first, as long as it doesn't f up the data.

No opinion.

@hamhands if you get a chance, can you address the comments from @shellicious above? I'd like to get this merged.

@bsmithgall @shellicious at this point do the examples really matter? We're not displaying them anywhere yet right?

What did we decide to end up doing with this? @shellicious ?

Let's just keep it as it is and forget about the duplicate examples for now.


Validate schemas overlooking the nullability fields

This PR includes:

[x] Changes to the validate_schema Dataset method to compare schemas between dataframes by looking at the names.

The functionality is essentially the same, only that we iterate on the column names, instead of on the properties of each schema field.
There are 2 checks in place: a) checking that the observed schema does not have any extra field; b) checking that the mandatory fields dictated by the expected schema are present in the observed_schema

[x] Added 2 unit tests for each of the functionalities I describe above

Notes:

I also had to tweak the pytest_generate_tests function. TIL that when we use pytest's metafuncs, we are defining a fixture that will be used in every test, unless specified otherwise.
Mypy doesn't check the script that tests the schema. This is because mypy is crashing when checking this script, even with the newer version, hence I was not able to commit. It's not that the script is raising a typing error, but mypy is failing to deserialize something. I've found an issue in their repo that might be related. Full trace:

Traceback (most recent call last):
  File "/Users/irenelopez/.cache/pre-commit/repomwnp37wt/py_env-python3.8/bin/mypy", line 8, in <module>
    sys.exit(console_entry())
  File "/Users/irenelopez/.cache/pre-commit/repomwnp37wt/py_env-python3.8/lib/python3.8/site-packages/mypy/__main__.py", line 15, in console_entry
    main()
  File "mypy/main.py", line 95, in main
  File "mypy/main.py", line 174, in run_build
  File "mypy/build.py", line 193, in build
  File "mypy/build.py", line 276, in _build
  File "mypy/build.py", line 2903, in dispatch
  File "mypy/build.py", line 3284, in process_graph
  File "mypy/build.py", line 3362, in process_fresh_modules
  File "mypy/build.py", line 2101, in load_tree
  File "mypy/nodes.py", line 397, in deserialize
  File "mypy/nodes.py", line 3689, in deserialize
  File "mypy/nodes.py", line 3630, in deserialize
  File "mypy/nodes.py", line 274, in deserialize
  File "mypy/nodes.py", line 854, in deserialize
  File "mypy/types.py", line 189, in deserialize_type

  File "mypy/types.py", line 2038, in deserialize
KeyError: 'unpack_kwargs'

Codecov Report

Merging #71 (f1a3fc1) into do_hydra (0450163) will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           do_hydra      #71      +/-   ##
============================================
+ Coverage     77.99%   78.13%   +0.13%
============================================
  Files            41       41
  Lines          1127     1125       -2
============================================
  Hits            879      879
+ Misses          248      246       -2

Impacted Files
Coverage Δ

src/otg/dataset/dataset.py
90.90% <100.00%> (+5.19%)
:arrow_up:

I agree with @d0choa, we should only drop the nullability check not the type check.

@d0choa @d0choa Thank you so much for your comments. They were all on point. The new changes include:

types of each field are now validated, not only the names (leaving apart only the nullability property)
compatibility for nested validation. this is achieved by flattening the schema of both dataframes to extract their fields. the function is heavily inspired in the one we have to calculate the metrics.
unit tests for: missing field, extra field, field of different type. run both in nested and not nested mock data.

Last change: added an ad hoc check for duplicated fields in https://github.com/opentargets/genetics_etl_python/pull/71/commits/d0b34891e16bfa48aaf2c85170d562d8a70fe47c


Smoke test failed on ubuntu 20.04

see https://github.com/apache/cloudstack/pull/6031#issuecomment-1061431788
Trillian test result (tid-3520)Environment: kvm-ubuntu20 (x2), Advanced Networking with Mgmt server u20Total time taken: 36070 secondsMarvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6031-t3520-kvm-ubuntu20.zipSmoke tests completed. 91 look OK, 1 have errorsOnly failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_migrate_VM_and_root_volume
Error
70.02
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
49.82
test_vm_life_cycle.py

ISSUE TYPE

Bug Report

COMPONENT NAME

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

Fixed on #6116


Possibility to stay the app in the background

Add a possibility to get the app in the task bar and in background.

Added on v1.2.0


pylance forces the import from typehint.

Environment data

Language Server version: 2023.4.40
OS and version: macOS Ventura 13.3.1
Python version (& distribution if applicable, e.g. Anaconda): pyenv 3.10.11 + poetry venv

Expected behavior
### before
from sqlalchemy.types import TypeDecorator

class Test(TypeDecorator):
    def dialect_impl

### after
from sqlalchemy.types import TypeDecorator

class Test(TypeDecorator):
    def dialect_impl(self, dialect: Dialect) -> TypeEngine[Any]:
        return super().dialect_impl(dialect)

Actual behavior
### before
from sqlalchemy.types import TypeDecorator

class Test(TypeDecorator):
    def dialect_impl

### after
from typing import Any
from sqlalchemy.engine.interfaces import Dialect
from sqlalchemy.sql.type_api import TypeEngine
from sqlalchemy.types import TypeDecorator

class Test(TypeDecorator):
    def dialect_impl(self, dialect: Dialect) -> TypeEngine[Any]:
        return super().dialect_impl(dialect)

Logs
in 2023.4.40
(3191) Background analysis message: setFileOpened
(3191) Background analysis message: markFilesDirty
(3191) Background analysis message: setFileOpened
(3191) Background analysis message: markFilesDirty
(3191) [FG] parsing: .../test.py (1ms)
(3191) [FG] binding: .../test.py (1ms)
(3191) Background analysis message: analyze
(3191) [BG(1)] analyzing: .../test.py ...
(3191) [BG(1)]   parsing: .../test.py (1ms)
(3191) [BG(1)]   binding: .../test.py (0ms)
(3191) [BG(1)]   checking: .../test.py (18ms)
(3191) [BG(1)] analyzing: .../test.py (19ms)
(3191) Background analysis message: resumeAnalysis
(3191) Background analysis message: getSemanticTokens delta
(3191) [BG(1)] getSemanticTokens delta previousResultId:1683022949096 at .../test.py (1ms)
(3191) Background analysis message: getDiagnosticsForRange
(3191) Background analysis message: getDiagnosticsForRange
(3191) Background inlayHint message

in 2023.4.30
(3065) Background inlayHint message
(3065) Background analysis message: setFileOpened
(3065) Background analysis message: markFilesDirty
(3065) [FG] parsing: .../test.py (1ms)
(3065) [FG] binding: .../test.py (1ms)
(3065) Background analysis message: analyze
(3065) [BG(1)] analyzing: .../test.py ...
(3065) [BG(1)]   parsing: .../test.py (0ms)
(3065) [BG(1)]   binding: ...test.py (0ms)
(3065) [BG(1)]   checking: .../test.py (7ms)
(3065) [BG(1)] analyzing:.../test.py (7ms)
(3065) Background analysis message: resumeAnalysis
(3065) Background analysis message: getSemanticTokens delta
(3065) [BG(1)] getSemanticTokens delta previousResultId:1683022783984 at .../test.py (1ms)
(3065) Background analysis message: getDiagnosticsForRange
(3065) Background analysis message: getDiagnosticsForRange
(3065) Background inlayHint message

more
I downgraded to 2023.4.30 and it worked fine.
"python.analysis.autoImportCompletions": false has already been added to the setting.

Thanks for the issue but this is by design. It's importing the missing imports when you complete the method. Without those imports it wouldn't run.
This was actually an enhancement:
https://github.com/microsoft/pylance-release/issues/4200

Thank you for your answer @rchiodo .
If it's by design, how can I turn it off?
and, Is this also intended to ignore TYPE_CHECKING?
ex:
## before
from typing import Any, TYPE_CHECKING
from sqlalchemy.types import TypeDecorator

if TYPE_CHECKING:
    from sqlalchemy.engine.interfaces import Dialect
    from sqlalchemy.sql.type_api import TypeEngine

class Test(TypeDecorator):
    def dialect_impl

### after
from typing import Any, TYPE_CHECKING
from sqlalchemy.engine.interfaces import Dialect
from sqlalchemy.sql.type_api import TypeEngine
from sqlalchemy.types import TypeDecorator

if TYPE_CHECKING:
    from sqlalchemy.engine.interfaces import Dialect
    from sqlalchemy.sql.type_api import TypeEngine

class Test(TypeDecorator):
    def dialect_impl(self, dialect: Dialect) -> TypeEngine[Any]:
        return super().dialect_impl(dialect)

If it's by design, how can I turn it off?

There's no way to turn it off. I guess don't use completions for methods would be the way to not have it happen at the moment.
The part where it skips TYPE_CHECKING seems like a bug though. It should recognize those imports I believe.

Thank you for your answer @rchiodo .
If this is a bug, would it be better to keep this issue open?
I'm not used to using github community yet, so I'm not sure what to do.

I think the TYPE_CHECKING part is a bug

I don't think this is a bug. Auto-imports shouldn't be added to import statements in conditional blocks because those import statements may not be executed, which will result in a runtime exception when the symbol is later referenced.
I suppose it would be possible to determine whether pylance is introducing a use of the symbol only for purposes of a type annotation and the use of that type annotation happens to be deferred (e.g. the file contains a from __future__ import annotations or is a stub file) and the conditional block is guarded only by a TYPE_CHECKING test. In that particular circumstance, you could safely special-case it and allow auto-import for an import statement within the conditional block. But that's a very specific circumstance that involves a coding practice that I would not recommend in the first place.
So I think the current behavior is "by design", and this is a feature request.

@erictraut
When I define task in airflow, I have to delay the point where import is executed.
In that case, I use TYPE_CHECKING if I want to add a type hint.
I think that's definitely the special case you mentioned.
I don't want it to be added under the conditional blocks, but I think it's okay if it's not duplicated.


[5.3][AST] Restore getSourceRange() on DefaultArgumentExpr

Cherry-pick of https://github.com/apple/swift/pull/31861

Explanation:
This restores getSourceRange() on DefaultArgumentExpr after it was removed in
https://github.com/apple/swift/pull/31184. While source tooling and diagnostics don't care about default arg expression locations (nothing can reference them and any issues in them should be diagnosed on the callee rather than caller side), their locations are output in the debug info, so their removal resulted in a debug info regression (rdar://problem/63195504).
It was originally removed to solve the issues it was causing when computing the source range of TupleExprs containing defaulted arguments after a trailing closure. The implementation assumes its argument expressions with valid source locations appear in source order, which default argument expressions violate (they point to the start of their parent tuple expression). This change works around that issue with a more targeted fix instead by updating TupleExpr::getSourceRange() to simply ignore DefaultArgExprs.

Scope of issue: This results in having no debug info for frames corresponding to default argument expressions.

Origination: This regressed due to a part of the fix to stop tuple expressions being incorrectly flagged as implicit when they contained defaulted arguments: https://github.com/apple/swift/pull/31184 (rdar 62118957)

Risk: Low. This restores the previous functionality and adds a more targeted fix for the original issue it was resolving.

Testing: Added a regression test to catch the debug info regression in future. All regression tests pass.

Reviewer: @xedin (on the master PR)

Resolves rdar://problem/63195504

@swift-ci please test

@swift-ci please nominate


narwhal: update narwhal pointer to a188b870eb1a4b601a58e5744290b7d56c…

…cd4039
This updates the narwhal dependencies to commit
a188b870eb1a4b601a58e5744290b7d56ccd4039 which includes the following
changes:

Conversion to gRPC network interfaces
Conversion to using the multiaddr format for describing network
endpoints.

Fixes: #1452

This should largely solve the weird port allocations issues we were seeing when deploying sui to production.


References to .scss files in _index.scss files are causing issues

This is at least spotted with the latest library on stackblitz, but might not ben an issue with the node-sass parser.

Issue reported at Stackblitz: https://github.com/stackblitz/core/issues/966

it's building now, does this need test?


apt install cassandra by default installs Cassandra v4 which comes with cqlsh v6 that breaks DESCRIBE

Install on Unix/Linux/Mac:
Per documentation, if I were to apt install cassandra, apt will by default get me the latest version (4.0.5). Currently, there's have Cassandra 4.0.5 and 3.11.13 on apt.
rico@ricos-studio:~$ apt show cassandra -a
Package: cassandra
Version: 4.0.5
Priority: extra
Section: misc
Maintainer: Eric Evans<EMAIL_ADDRESS>Installed-Size: 58.8 MB
Depends: openjdk-8-jre-headless | java8-runtime | openjdk-11-jre-headless | java11-runtime, adduser, python3 (>= 3.6), procps
Recommends: ntp | time-daemon
Suggests: cassandra-tools
Conflicts: apache-cassandra1
Replaces: apache-cassandra1
Homepage: http://cassandra.apache.org/
and storage of structured data.

Package: cassandra
Version: 3.11.13
Priority: extra
Section: misc
Maintainer: Eric Evans<EMAIL_ADDRESS>Installed-Size: 40.4 MB
Depends: openjdk-8-jre-headless | java8-runtime, adduser, python (>= 2.7) | python2 (>= 2.7), procps
Recommends: ntp | time-daemon
Suggests: cassandra-tools
Conflicts: apache-cassandra1
Replaces: apache-cassandra1
Homepage: http://cassandra.apache.org/
Download-Size: 30.7 MB
APT-Sources: https://downloads.apache.org/cassandra/debian 311x/main amd64 Packages
Description: distributed storage system for structured data
Cassandra is a distributed (peer-to-peer) system for the management
and storage of structured data.

The problem with this is, Cassandra 4.0.5 comes with cqlsh v6 which breaks DESCRIBE. [CASSANDRA-16652] "desc" on cqlsh 6.0.0 not working with a Cassandra 3.x server - ASF JIRA (apache.org)
To fix this, I had to specify apt install cassandra=3.11.13 which comes with cqlsh 5.0.1 that works. This is also the version which the Data-Explorer-integrated Cassandra shell uses.
Can we revise the docs on this?

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

ID: b6c47387-6172-e49b-5144-c843c91815a8
Version Independent ID: 93cad71b-c859-3ae7-32c2-931ef46dd603
Content: Apache Cassandra features supported by Azure Cosmos DB Cassandra API
Content Source: articles/cosmos-db/cassandra/cassandra-support.md
Service: cosmos-db
Sub-service: cosmosdb-cassandra
GitHub Login: @iriaosara
Microsoft Alias: iriaosara

Hi @ricowang-msft ,
Thanks for sharing your feedback. We will investigate and work as appropriate.

Hi @iriaosara ,
Could you please look into this document update request. Thanks

We could update this document: https://learn.microsoft.com/en-us/azure/cosmos-db/cassandra/cassandra-support
And, instruct how to obtain the supported cqlsh 5 version 5.0.1 with cql spec version 3.4.4.
Go to Apache Cassandra Download  |  DataStax Enterprise and from the Version dropdown box, choose 'DataStax Distribution of Apache Cassandra'. One should expect below in cqlsh shell:
(cosmosdb cassandra acct)@cqlsh> show version
[cqlsh 5.0.1 | Cassandra 3.11.0 | CQL spec 3.4.4 | Native protocol v4]

Update done. Thank you

@shaktisingh-msft Please can you help close this issue. Update in and it has been merged. Thank you


fix app.go

apparently https://github.com/cosmos/cosmos-sdk/tree/master/x/genaccounts is in "github.com/cosmos/cosmos-sdk/x/genaccounts"

Same comment as https://github.com/cosmos/sdk-application-tutorial/pull/111#issuecomment-506937102, Will be updating the tutorial this coming week with the upcoming release.

updating the tutorial in this PR: #120, closing this in favor of it as it has the most recent version of the Cosmos-SDK


Enable support for RTL Text

Motivation
Since we have users with arabic background we would like to be able to show text in arabic on the map.
For web maplibre js provides a RTL Plugin: https://maplibre.org/maplibre-gl-js/docs/examples/mapbox-gl-rtl-text/
For native it seems like there is no support. I even couldn't find.
Design Alternatives
The feature could be provided with a separate plugin
Design
The feature couldn't be implemented directly
Mock-Up

Concept
I don't have really design or concepts for this, since i don't know much about what has to be done but probably the features that are included in the js library could be ported to the native library

Hey you can try if it works if you prefix the UTF RTL code:
\u{200F}
you can also try:
\u200F
https://unicodeplus.com/U+200F
https://stackoverflow.com/questions/59319997/how-to-set-direction-property-of-a-view-to-rtl-in-react-native

okay it was basically an issue with the font glyphs on my side. Only changed on our web platform but not for native.
Thx for the hint

oh that works? :D
was just a guess based on the standard


StrictHostKeyChecking

The application is bothered by this entry in the config that I need to use. Is there anything I can do about it?
2023/12/06 11:36:26 /Users/xxx/.ssh/config:0: config variable before Host variable
StrictHostKeyChecking=no

Hey! That should not be a problem anymore in 4.0.0

Thank you, it looks better. But it now has a problem with the commented text in the config file. It stops at it and doesn't read other servers.

I'm also experienc

Thank you, it looks better. But it now has a problem with the commented text in the config file. It stops at it and doesn't read other servers.

I'm also experiencing this as well.

Woops - It really needs some tests. Should be fixed in 4.0.1!

@quantumsheep confirmed that the issue is fixed. Thank you for resolving it!


callback for pushing CupertinoModalSheetRoute

Is there a way to get a callback after the modal has been popped?  I've tried subclassing the CupertinoModalSheetRoute but the callbacks are happening immediately when modal is being dismissed:
CustomCupertinoModalSheetRoute({
    required WidgetBuilder builder,
    RouteSettings? settings,
    bool swipeDismissible = true,
    Color? barrierColor,
    this.onDismissed,
  }) : super(
          builder: builder,
          settings: settings,
          swipeDismissible: swipeDismissible,
          barrierColor: barrierColor,
        );

  @override
    bool didPop(T? result) {
      final didPop = super.didPop(result);
      if (didPop && onDismissed != null) {
        onDismissed!();
      }
      return didPop;
    }

  @override
    void didComplete(T? result) {
      super.didComplete(result);
      if (onDismissed != null) {
        onDismissed!();
      }
    }
  }

  Future<T?> showSheet<T>(
    BuildContext context, {
    required Widget content,
    EdgeInsets padding = EdgeInsets.zero,
  }) async {
      Navigator.of(context).push(
      CupertinoModalSheetRoute<T>(
        swipeDismissible: true,
        builder: (BuildContext innerContext) {
          return SafeArea(
            child: Center(
              child:  Sheet(
                  content: content,
                  padding: padding,
              ),
            ),
          );
        },
        onDismissed: (){
          print('dismissed');
        },
      ),
    );
  }
}

Is there a way to determine when the modal has completely dismissed?

nvm i found it in the docs https://api.flutter.dev/flutter/widgets/TransitionRoute/completed.html

  return route.completed.then((result) {
    if (onClose != null) {
      onClose();
    }
    return result;
  });```


More informative errors when node type is not recognized

This is an awesome start.  I do think though that once we get to that point we have lost an unfortunate amount of information (i.e. line number, parent context etc.).  Perhaps we could do these checks as part of .visit instead of .visitNode.  We could then add parent parameter to .visit that we could populate everywhere we call .visit.  That way we could look to see if that parent exists and use it for context.
That way we could have errors look like:

A child of {ParentNodeType} is {NodeValue}, expected a Jade AST Node.
...Line Number Info...

and

A child of {ParentNodeType} is of type {NodeType}, which is not supported by jade-code-gen
...Line Number Info...


Rename file to conform to psr-4 autoloading standard

This pull request renames the RunTimeException.php file to RuntimeException.php to conform to the psr-4 php standard.
Currently composer gives the below deprecated notice:
Deprecation Notice: Class CurrencyConverter\Exception\RuntimeException located in ./vendor/ujjwal/currency-converter/src/Exception/RunTimeException.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///usr/local/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201

This fixes the above issue

Thanks for the PR @knox2. Do you mind removing the existing RunTimeException.php too?

I removed the file myself as it was a minor change.
I also published https://github.com/ojhaujjwal/currency-converter-php/releases/tag/2.5.1.
Thanks for your contribution. :)


added rune testscase on hamming distance problem

I'm 💯 behind the goals of this PR: Go's handling of Unicode characters is special, and can lead novices into trouble if they don't understand it properly.
The fundamental issue, for those who haven't been following closely, is that you can process strings as either byte sequences, or rune sequences. Provided you have only ASCII characters, then those both amount to the same thing.

The hamming exercise is an early core exercise, and I'm not sure we should expect understanding of rune and UTF-8 so early in the Go language track.

I've changed my mind on this fairly recently. I used to tell students that in cases like this, where they can guarantee that only ASCII characters will be in the input, then it's fine to read strings by bytes (and index them directly with []). However, in other cases you need to use the other strategy (read runes).
Frankly, two things to learn is one too many, at this stage. My thinking now is that it makes sense to teach just one way, at least at the beginning: the right way.  Until that time, my fail-safe teaching strategy is always treat strings as rune sequences, not byte sequences.
@leenipper is absolutely right that it's an early exercise, and this is a somewhat arcane thing to understand, especially because few other languages do it this way. But if you're using the Bitfield Method(TM), you'll want to teach runes right away. Otherwise, you can bet someone will say "How come it was fine to use bytes before, but now we're doing runes?"
Maybe the answer is to move it later in the track; it's a trivial exercise in languages where bytes == runes, but not in Go, if you're doing it right.
The question here is not just "should mentors guide students towards a rune-based solution to Hamming?", but "should we actually enforce it with a test case?"
I think we've historically been rather against the idea of enforcing specific implementations with test cases, or even enforcing input validation by testing broken inputs. Also, bear in mind that this is not a generic comparing-random-strings problem. This is specifically a comparing DNA strands problem. The input consists only of As, Cs, Gs, and Ts. So what's an èà test case for? It's invalid input, and we explicitly don't test input validation.
Maybe this is an alternate way to achieve more or less the same goals:

Leave Hamming as it is for now, but move it a step or two further down the track.
Update the mentor notes (if they're not already) to point out the problems with reading strings as bytes, and to suggest that mentors guide students towards always using runes.

Optionally:

Introduce another exercise which involves counting or comparing runes, but not involving DNA strands, so that wide characters are now fair play, and add test cases which would fail a byte-oriented solution.

I'd love to hear what @kytrinyx thinks too.

@bitfield , thank you for your excellent points.
I agree that hamming may not be the exercise to emphasize rune awareness.  I also agree that it is best to teach one way, and using rune in a for .. range is best when dealing with string elements.  In whatever exercise is applicable, a hints.md to augment the README.md with Go specific instruction about rune vs. byte will be helpful.

I am closing all PRs that are over a year old. Please reopen if you are still working on this.


Check static limits for sequences

wip: check limits when creating/updating sequences. check for recursion as well.

I think this is ready for review.
596 tests completed, 1 failed, 19 skipped - swift action timed-out, unrelated to these changes.
Once I address your comments, I'll squash the second commit.

Beautiful Green Travis - @rabbah ready for a second round of comments. I think I fixed everything from the first round. Once I fix everything, I'll squash in one commit.

I've generally done for pairs of properties for convenience. For a singleton, we have generally referenced it by name in the map for the required properties.

ok, I'll move it then.

Adding @markusthoemmes as a reviewer - these changes are relevant for #239.

Thanks for the feedback.
The first thing I'll try to do is get rid of the components field and use instead the new class we have so we always store FQEN for components. I think this is the way to go, we shouldn't have used Strings to begin with. This involves messing around with @perryibm 's code (for sequences with pipe.js), I hope I'll iron out the differences so I can still have both versions of the code in place.

@rabbah @markusthoemmes I still have a couple of your comments to address + fixing a check that it's missing right now (+ test for it). Let me know if once I'm done, you'd like me to  squash or it's better to progressively see the changes for now.

Could you please install https://github.com/openwhisk/openwhisk/blob/master/tools/eclipse/scala.properties to avoid formatting mismatches?

@rabbah @markusthoemmes another look before I rebase and squash? I think I addressed all issues, let me know if I missed anything.

Closing this, subsumed by #1337


Extract the login logic as an independent service "LoginService" (Code level and not the deployment level)

Closed because we have introduced a new project turms-gateway. https://github.com/turms-im/turms/commit/75e8b7ffb83346ef6a8cf54088012364879a09c5


How to exclude functions matching a pattern from D100 error?

I have a few functions on a legacy code base named call that do not have a docstring. I would like to be able to specify that I do not want to check for docstring presence for the functions named call. It would be helpful to have an extra configuration with the whitelist of such functions. Adding # noqa comment is not an option.
Is it something already supported? If not, I would love to see this feature implemented.
Is it something that I could implement relatively easy in the source code for submitting a PR?

@Nurdok thanks for getting back! I gave you just an example of a functionality with a dummy function call. The actual code base has dozens scripts which are used as cli tools and each has boilerplate functions like main, read_cli_args, parse_args etc. The docstring will be pretty much the same as the function name with whitespaces, so I wondered whether those function names could be whitelisted. Adding # noqa takes as much effort as adding a docstring.
Another use case for this would be Python frameworks where you need to override magic dunder methods and adding docstrings for all those __str__() seems unnecessary. Does it make sense?

For magic methods there's a separate error code specifically designed for this common use case. But the main reason we handle that case is that the APIs for these methods is known from their name. That error
In your example, say, read_cli_args, I think there's still reason to document the function: does it return a value? In what format? What's the behavior for invalid CLI flags? etc., etc.
If your general opinion is "some constructs do not need docstrings because their name is self-explanatory", you can simply turn off all of the D1xx "missing docstring" error codes.

Good points @Nurdok , thanks! I don't want to disable D1 for the whole file as there are other functions for which I would like to enforce presence of docstrings :) Anyway, I guess I will need to add docstrings to them as this is the only option to me (apart from # noqa). Cheers,

@AlexArcPy that's great. Sorry for the push back, but as with any open source project, each user has their own unique use case and we need to make sure features are applicable to a large portion of them to avoid feature creep and to keep the code maintainable.
Cheers!


Add greater debug control

Currently the debug chain turns on all possible debugging tools - Notion API logs, NotionCMS console logs and (its turned off for now) fs output logs from the parser. If we instead used a debug object we could let end users specify which debugging features they want turned on.

One idea would be to add debug logs for plugins. I would want to see which plugin ran in what order for each hook. and possibly other things like run duration? For debugging performance specifically.


Fix Dashboard Tests for ODH and RHOAI 2.9

Including fixing dependent KW and tests:

Fix Verify RHODS Display Name and Version for ODH
Set $DASHBOARD_APP_NAME according to RHODS / ODH
Fix Verify Favorite Resource Cards test
Fix Sort Resources By KW
Fix Verify Jupyter Card CSS Style KW
Fix Upload Serving Runtime Template in ODHDashboard Settings
Update DEFAULT_NOTEBOOK_VER=2024.1 and PREVIOUS_NOTEBOOK_VER=2023.2
Search Pod KW returns a list of pods that match regular expression
Verify Jupiter Service Is Not Enabled should not wait long (app isn't present)
Remove Test Billing Metric test (5+ minutes) from Smoke

Noam, I like the structure of the commits to separate each part of work. I wish we have this for each PR. Although, the change with the kube/config file isn't correctly provided within these commits - the original commit that inadvertently introduced this file still contains it and file was removed in a different (much later) commit. As such, we should merge this with squashing the commits, unless you update this discrepancy in the commit history properly (which I would prefer, though).

Tested again after requested changes: rhods-ci-pr-test/2637/TestReport/

Thanks @jstourac , I squashed the commits.

JFTR - the commits aren't squashed (and I like it since they are well structured and thus can be merged as they are from my point of view). What was squashed/fixed-up was the change discrepancy - inadvertent change in first commit was fixed in a different commit - now this has been removed from the original commit directly and thus makes the commit history clean. Thank you.

I'd vote for squashing anyway, it makes the overall history cleaner, if everybody starts merging new 10 commits or more it becomes difficult.
Moreover, I believe that when PR are so big, you cannot just revert a commit without the risk of missing dependent code. Hence if there is a regression introduced by a such PR, it would be easier to revert the single squashed commit and raise the PR again with the fix

I guess I wouldn't squash. The individual commits have meaningful commit messages, so it would help with git blames and later investigations to keep them separate.

I guess I wouldn't squash. The individual commits have meaningful commit messages, so it would help with git blames and later investigations to keep them separate.

in case of regression it can become a hunt for the commits you need to revert. Even squashing you can see who edited a line with git blame. What's the problem?

GitHub webui gives you option to revert entire PR. And I guess I now see yet another good use of merging with merge commits.
In git blame, I like to be able to go to commit message and see whats the message associated with each change. Usually it's obvious but sometimes it isn't and then it's useful info to have.

I will squash the PR, since we can still see all the commits within it.
Take a look at another squashed commit: https://github.com/red-hat-data-services/ods-ci/commit/8910a4db011f6a9a38d5d869054b84d43c3de9fd - You can click on the PR https://github.com/red-hat-data-services/ods-ci/pull/1235 link in it and see all the squashed commits it includes.


Add a Gitter chat badge to README.md

cmosguy/react-redux-auth0 now has a Chat Room on Gitter
@cmosguy has just created a chat room. You can visit it here: https://gitter.im/cmosguy/react-redux-auth0.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Why not? :-)


GNU APL Compatability

Hey all. Is tatin gnu apl compatible?

I am said no: it is specific to Dyalog APL.
On Sat, 14 Oct 2023, 13:42 Timo Sarkar, @.***> wrote:

Hey all. Is tatin gnu apl compatible?
—
Reply to this email directly, view it on GitHub
https://github.com/aplteam/Tatin/issues/150, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5XJHL5MF32D3GQARRHCDDX7J3EHANCNFSM6AAAAAA6AGKP7Y
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Foo.prototype.isFoo = true breaks tree-shaking

Bug report

What is the current behavior?
Seems like Webpack tree-shaking breaks when adding a property to a class prototype.
Rollup however, seems to be able to tree-shake it.
If the current behavior is a bug, please provide the steps to reproduce.
Unsure if this is a bug or a feature request.

What is the expected behavior?
Webpack should be able to tree-shake that code. In Three.js we rely on prototype.isFoo so this means we'd have to recommend people to use Rollup if they want to produce small bundles.

Other relevant information:
webpack version:  5.23.0

Rollup maintainer had this to say:
https://twitter.com/lukastaegert/status/1363084395960221703

There are many edge cases which makes this unsafe. Rollup is a little more aggressive in optimization, but webpack usually tries to avoid unsafe optimization.
You can wrap it in a PURE flag instead:
export const Foo = /*#__PURE__*/ (() => {
  class Foo {}
  Foo.prototype.isFoo = true;
  return Foo;
})();

class properties work a bit different (they define a property instead of calling a setter) and allow a lot more optimizations:
class Foo {
  isFoo = true;
}


SSL Proxy with a HttpClient?

The docs describe how to set up a HttpClient that uses a proxy by setting the findProxy callback.
This works fine with HTTP proxies, but not when using a proxy with SSL.
When using a SSL server (for example, us5308.nordvpn.com:89), a get operation fails with the following:
Unhandled exception:
Connection closed before full header was received
#0      IOClient.send (package:http/src/io_client.dart:62:7)
<asynchronous suspension>
#1      BaseClient._sendUnstreamed (package:http/src/base_client.dart:91:38)
#2      BaseClient.get (package:http/src/base_client.dart:27:7)
#3      main (<DART_FILE>)
<asynchronous suspension>
#4      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

The same server can be connected to with no problems with OpenSSL:
$ openssl s_client -connect us5308.nordvpn.com:89

A low level connection with the dart:io SecureSocket API also works.

Any updates on this?


Bug28458

Need to fix CI...

I've solved what you comment locally, but before i push and you get the email notification (and then maybe you feel like you have to review it), see my comment in https://trac.torproject.org/projects/tor/ticket/22453#comment:32

The new PR is https://github.com/torproject/sbws/pull/294


Assignments: Get back to "type text" for bulk grading input

Related to Commit #10378
Jira: SAK-47700
Reason: incompatibility with decimal comma based languages.
It makes sense to use type “number” as the input will always receive a score, and results in less checks on the backend.
So the best option would be to modify the backend, but by the way it’s coded, a lot of extra inputs/logic should be modified, taking longer than expected. Then, this should be a temporal patch.

This is bad, I'd like to use the number instead of Text but I understand it will require many backend changes.

Could you please edit your commit and include SAK-47700?

Sure, forgot about it

Awesome @V100X !


Spectrum "module"

Will be a nice option to be able to pickup frequency and SR of a transmission to send them to Pluto-longmynd by a click right on the mouse or Ctrl click right.
I am not a programmer but I have seen on your wb code that you have started...
Thank you for your work for the community.
73 qro de michel hb9dug

hey, please see latest commit


[WIP] Rewrite from scratch

@emmenko thx for pushing this ;)

Sure ;)

@hajoeichler you should also encrypt the credentials for travis :)


Return multiple read responses as per prometheus/prometheus#2588

/cc @juliusv
See prometheus/prometheus#2588

:+1:


fix greenkeeper + travis config

Coverage remained the same at 67.47% when pulling d3cf4def11d1c408f94f0180a46758cf6dd72af9 on greenkeeper-lockfile into bcca805f6a753420539f79c789747fcf6f23753f on master.


⚠ Bofrost.be has degraded performance

In 94efefb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 853 ms

Resolved: Bofrost.be performance has improved in f3e6824.


Updated PropTypes for React 16

React.PropTypes => PropTypes

Thanks for the PR @blaremc!  If I merge this PR and publish, will this break anyone on <16?  I'm assuming the answer is "yes" and I'll bump the major version.  I just wanted to make sure.

This would be a breaking change for anyone on React <15.5.0 (more details here).
Love this package. A merge would be great!

@BrianGenisio Fantastic! Thanks so much

Just got around to integrate 2.0.0 to a project. It looks like package.json refers to "main": "dist/index.js" (permalink), but the NPM release is missing the dist folder.
See https://registry.npmjs.org/react-balance-text/-/react-balance-text-2.0.0.tgz
1.2.0 had it:
https://registry.npmjs.org/react-balance-text/-/react-balance-text-1.2.0.tgz
.npmignore looks fine.


PERF-#6702: don't materialize axes when calling to_numpy

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6702
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

the obvious thing to mention is that we are losing the ability to catch errors when index/columns are not materialized, but I guess it's okay?

Not really, when the index is not materialized, we calculate it from internal partitions, in this case the check does not make sense. We do the same for to_pandas.

@dchigarev let's merge it


Add Stranglehold

AKA John Woo Presents Stranglehold. This title is only available on GOG, as far as I know.
https://www.gog.com/game/stranglehold?pp=2863d7ae605104eeef364e3f164d3404e20f680c

I added the game—albeit a little late ;)
Feel free to add the changes by yourself. That way I can simply approve the additions or changes which makes it a bit easier.
Thanks for your help! I really appreciate every contribution.


[SMALLFIX] Ignore the fuse tests if the libfuse library is missing

So developers won't see test failures on mac if they do not have osxfuse installed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17763/
Test PASSed.

Shouldn't libfuse dependency be added as part of build step in docs rather than ignoring test cases ?

@gurmeetb many people on mac may not want to install the third-party osxfuse, so it's better to be opt-in. Also most flavors of Linux (Alluxio jenkins included) have libfuse preinstalled, so the pull request can be checked on the fuse tests.


Add documentation for selector_tabulator app demo

Does this need its own repo? Is it a demo that should stay in our current repo?
Update README.md in this folder and one level up to document what this is if it stays in our current repo.

Closed by #548


Create some base level of De-ID for Kafka

we need to ascertain the De-ID at the KAFKA level.  need a baseline approach such at ARK which is apache 2.0
https://arx.deidentifier.org/downloads/

Data anonymization for stored JSON in Kafka.  How to balance the ability to retrieve the exact JSON stored for downstream integration vs the need to anonymize in Kafka?

additional references here:
https://arx.deidentifier.org/development/api/
and a novel ** game theoretic ** approach:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5977602/


bug: hideDisabled option doesn't work correctly

Hello,
since version 1.13.4 hideDisabled option doesn't work as should. Hidden positions from dropdown menu are hidden, but they are still visible (or counted if selectedTextFormat option is set to count them) on dropdown button.
examples - click 'Hide selected' button and toggle bootstrap-select menu:
1.13.3 - works: https://jsfiddle.net/8tLzwdjy/
1.13.4 - doesn't work: https://jsfiddle.net/6ntzyhu8/
1.13.9 - still doesn't work: https://jsfiddle.net/2aLzdjg0/

jQuery's .val() doesn't take into account disabled options, so we need to use the getSelectValues in togglePlaceholder, and update that function accordingly so it doesn't count disabled options, making it a closer recreation of .val() (which is what togglePlaceholder was using before to check if there were any options selected). This will be available in the next release.

Released in v1.13.10!