Possibility to take IP from another gateway

Hi there,
at my home I have 2 connections. (ADSL and 5G)
I want to run the script in a server, but I want to take the IP from another router (gateway).
Is this possible please ?
Thanks in advance
Denis

Here are two ways to get the IP that way:
ifstatus wan | jsonfilter -e<EMAIL_ADDRESS>
ip route get <IP_ADDRESS> | awk '{print $7}'

Thanks a lot :-)


Difficulties when packaging your module with rollup (es6)

Trying to switch from gulp to rollup in my videomail-client project but build fails with a websocket-stream related error saying:
'default' is not exported by node_modules/websocket-stream/stream.js

the rollup config I am using is in this incomplete, experimental branch:
https://github.com/binarykitchen/videomail-client/blob/feature/rollupjs/env/dev/rollup.config.js
the whole output is
~/c/videomail-client ❯❯❯ yarn build:dev                                                                                ⏎ feature/rollupjs ✭ ✱
yarn build:dev v0.24.5
$ npm run std --silent
$ rollup -c env/dev/rollup.config.js -w
bundling...
(700): src/wrappers/optionsWrapper.js⚠   'default' is not exported by 'node_modules/global/window.js'
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
commonjs-proxy:/home/michael-heuberger/code/videomail-client/node_modules/global/window.js (1:137)
1: import * as window from "/home/michael-heuberger/code/videomail-client/node_modules/global/window.js"; export default ( window && window['default'] ) || window;
                                                                                ^

⚠   'default' is not exported by 'node_modules/global/document.js'
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
commonjs-proxy:/home/michael-heuberger/code/videomail-client/node_modules/global/document.js (1:145)
1: import * as document from "/home/michael-heuberger/code/videomail-client/node_modules/global/document.js"; export default ( document && document['default'] ) || document;
                                                                                ^

🚨   'default' is not exported by node_modules/websocket-stream/stream.js
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
src/wrappers/visuals/recorder.js (3:7)
1: import { Buffer } from "/home/michael-heuberger/code/videomail-client/node_modules/buffer-es6/index.js";
2:
3: import websocket from 'websocket-stream';
          ^
4: import Frame from 'canvas-to-buffer';
5: import util from 'util';

Done in 12.99s.

have you every managed to rollup an es6 enabled project with this library?
is there anything suspicious?
does websocket-stream come with a default export or will i have to tweak this with the namedExports option https://github.com/rollup/rollup-plugin-commonjs#custom-named-exports ?

Unfortunately no, and I have never used rollup as well.


v2: remove leading/trailing whitespace from html properties

The relevant portion of the parsing spec here is http://microformats.org/wiki/microformats2-parsing#parsing_an_e-_property which reads:

html: the innerHTML of the element by using the HTML spec: Serializing HTML Fragments algorithm, with leading/trailing whitespace removed.


Recompile user_config.proto with protoc v3.6.0

Just ran protoc --proto_path=. --python_out=. user_config.proto

lol that was fast


Don't put integer in environment

It doesn't work when calling putenv later on.

Does this env var continue to make sense?

Does this env var continue to make sense?

I guess not... Maybe I had some outdated docker images.


covid19: Compare Country Trajectories

You should be able to see the interactive plots here. Basic idea here is to compare growth trajectories with Italy, South Korea & Japan. User can use (click, shift+click) legend to add countries. And last chart provides a country dropdown for a focused view. Hopefully, these representations reduces clutter.
Let me know, if you find this a useful addition and anything else you'd like to see added?

Excellent!  Can you add a 33% Daily Growth Line similar to https://covid19dashboards.com/growth-bayes/ ?

@hamelsmu -- Good point! Added 33% reference line to the plots.

This is excellent work.  @jph00 You should see this work by @pratapvardhan  it is the same COVID-19 chart you see on the Financial Times, but interactive, updated hourly and also with multiple views.
@pratapvardhan I'm really impressed by this.  Also I would categorize this PR as "beautiful" and very well put together, you really paid attention to detail.

@pratapvardhan for some reason the charts are not rendering on GitHub pages, even though they are rendering locally.  Really baffling.  I have temporarily marked this as hidden until I can figure it out.  You can see this at https://covid19dashboards.com/compare-country-trajectories/

@pratapvardhan fixed it

Thanks @hamelsmu !

Hi @pratapvardhan would you be willing to do the same set of charts for the Death Rate?
Like this https://twitter.com/jburnmurdoch/status/1239998499758002182?s=21
Awesome work

@hamelsmu -- let me send a PR in sometime.


New version: YoYoGames.GameMaker.Studio.2 version 2<IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #10339.

Log
Updated: 2<IP_ADDRESS> → 2<IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

This package installs and launches normally in a Windows 10 VM.


Make backend tests fast again

They used to take like 1.5 min, now it's at ~6 min.

You can try running the tests in parallel:
https://www.tutorialspoint.com/pytest/pytest_run_tests_in_parallel.htm#:~:text=By default%2C pytest runs tests,to run tests in parallel.
https://docs.pytest.org/en/3.0.1/xdist.html

They hit the same DB, though, which is problematic. Would require partitioning into separate schemas or similar. I'm not sure if PostGIS can be installed outside the default schema.

I think postgis is installed per database, and a db can have multiple schemas inside it.
https://postgis.net/install/

PostGIS is an optional extension that must be enabled in each database you want to use it in

Related: https://github.com/Couchers-org/couchers/pull/571 (but does not close this issue)

There's a few more things with CI: we could do the gitlab DAG thing, so that backend and frontend could be built/tested at their own pace. Currently frontend is slow to build, fast to test, and backend is fast to build, slow to test.
Another thing is that we could move the frontend build out of docker, and this should speed it up a whole lot.

In my experience, running pytest in parallel lead to hung tests and increased error rate.

Related: https://github.com/Couchers-org/couchers/pull/885 opens up for running a single migration test instead of running all tests twice.


Shared folders not available

InSIFT 3 I enabled shared folders in vmware but /mnt/hgfs is empty.
The command /usr/bin/vmware-hgfsclient show the shared folder name.
How can I enable and access the shared folder from SIFT to my Windows host?
Thanks.
Enz

This is common with certain versions of VMware (and, more specifically, vmware-tools).  Try disabling the shared folders, closing the dialog box, re-enabling them, and then open a new terminal/file window in /mnt/hgfs/

Thank you Phil, you solved my problem.
I close this issue.

Thanks @philhagen for fielding the question!


Install obmd only when running integration tests

doing apt-get update and then dowloading go-lang (~100MB) takes up a minute
or so.

Edit:
Actually, it takes way more than a minute. From the build of this PR:
"Fetched 108 MB in 5min 26s (330 kB/s)"

LGTM, but as a sanity check I want to wait until #1007 is merged, in case this interacts badly with that. I don't expect it to, but...

closing this in favor of #1019


Allow passing configuration via environment variables

Is your feature request related to a problem or a nice-to-have?? Please describe.
I have a launch config in neovim which runs npm test. I'd like a mechanism to pass --inspect-brk to mocha. Setting NODE_OPTIONS=--inspect-brk isn't an option because it affects all tools that are executed by the test script.
Describe the solution you'd like
A MOCHA_OPTIONS environment variable variable similar to NODE_OPTIONS.
Describe alternatives you've considered
Yargs has support for env variables: https://yargs.js.org/docs/#api-reference-envprefix
However, using this feature would result in a separate env variable for each option:
Ex:

MOCHA_BAIL
MOCHA_ALLOW_UNCAUGHT
MOCHA_FAIL_ZERO
etc ...

PR https://github.com/mochajs/mocha/pull/4835

Duplicate of #4330.


Sign In Redirect

Reworked sign in redirect logic to use local storage instead of a query param since the query param didn't support the wallet modal redirect flow.

@marcinbodnar Thanks for the comment! I pulled out the requestSignIn and requestCreateAccount methods from the VmInitializer flow and now have a simple method provided by the new redirect hook. I think this is an improvement over what we currently have. This can always be refactored further down the line, but right now it's probably more important to make sure the redirect is working for our users.

@calebjacob preview env is broken

Also can't run it locally:

@shelegdmitriy Thanks - should be fixed now.


Re-enable quest icon on world map

A Guild Wars update a few years back wiped out the quest marker on the world map, so its difficult to see where the current quest is supposed to take you. Investigate the codebase to see if some kind of flag has accidentally been unset or omitted, maybe we can fix it client side

Its actually here, just stupidly small


Origins of the extension

Hi Kalita!
I wrote an article and was corrected about the origins of your extensions. Was your extension a fork of the RustyCode extension?
Thanks!

@booyaa,
Yes, it was. It's actually described at the beginning of README.

Cheers


Add tokyRT/Gasy-quotes-explorer

Repo URL:
https://github.com/tokyRT/Gasy-quotes-explorer
Checklist:

[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`)
[x] The format of the line is [username/repo](link) - description.
[x] The description doesn't contain emojis.
[x] Acronyms should be fully capitalized.

Sorry forgot the backticks on branch name

You can use any branch name, just correct the PR title


clear blockers

Current blocker:
CA rules not running on my PC the same way it's running on @Ystyr PC

Done!

working!


guide: add code explanations to js algorithm

typo in guide article for "create a controlled input" from the front end libraries certification for react

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR in an attempt to attempt to break out of the Travis build pending status.


Feature Request: Nested array/object parameters

Feature request
Support specifying array and object params within object params, e.g.:
params:
- name: event
  value:
    repository:
      owner: tektoncd
      name: pipelines

Use case

putting full event bodies into parameters, which could be helpful for supporting FRs like https://github.com/tektoncd/triggers/issues/1574 (FYI @skaegi)
producing results from matrixed taskruns of type array of array (where the matrixed taskrun produces array results) or array of object (where the matrixed taskrun produces object results) (FYI @pritidesai)

Challenges

determining whether there should be a cap on level of nesting or param size

Thanks @lbernick for creating this feature request 👍
There is an additional use case from our users for an array of object in matrix.
matrix works great with an implicit and explicit combinations. But matrix does not support dynamic list of combinations specified by the users through the pipelineRun or a trigger.
For example, with a shared pipeline, some users would like to specify 2 combinations while some would like to initiate the same pipeline using 3 combinations.
The following example works great where the pipeline controller generates all possible combinations of both platform and browser. The users can specify a list of platforms and browsers at run time. The same pipeline can be invoked for 2 platforms and also for 3 platforms.
    matrix:
      params:
        - name: platform
          value: $(params.platforms[*])
        - name: browser
          value: $(params.browsers[*])
  ...

We are missing the same level of dynamism with an explicit combinations. Taking an example of matrix to build an image with a specific path for each image would not work if the list of images are specified at the pipelineRun level. The following example is part of the pipeline and the pipeline will always create three instances of the same build task.
    matrix:
        include:
          - name: build-1
            params:
              - name: IMAGE
                value: "image-1"
              - name: DOCKERFILE
                value: "path/to/Dockerfile1"
          - name: build-2
            params:
              - name: IMAGE
                value: "image-2"
              - name: DOCKERFILE
                value: "path/to/Dockerfile2"
          - name: build-3
            params:
              - name: IMAGE
                value: "image-3"
              - name: DOCKERFILE
                value: "path/to/Dockerfile3"
  ...

The concept of nested array and object param will help solve this use case where the same pipeline can be invoked with any number of explicit combinations.
matrix:
  params:
    - name: list-of-platform-browser-object
      value: ($params.list-of-platform-browser-object[*])


6031 DEV Update Sample for new steps yml format

https://github.com/Shippable/pm/issues/6031
We have changed the steps in jobs.yml as described below
steps:
  - GET resource1

to
steps:
  - GET: resource1

Closing this , as PR with new jobs and resources is already in place.


feat: bump Windows CSE script package to v0.0.46

What type of PR is this?
/kind feature

What this PR does / why we need it:
bump Windows CSE script package to v0.0.46
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[x] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.069%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
2572

Relevant Lines:
3619

💛  - Coveralls


Softmax() is not working on GPU

Hello,
I have slightly modified the "mnist_mlp.py" example from Neon's distribution.
I'm using softmax as an activation function of the output layer instead of original logistic.
Also I added fprop to get trained network's predictions on the MNIST standard test set
This works well on CPU, and I obtained ~95% classification accuracy on this test.
But the same example goes wrong on GPU here only 1 example out of 1000 was classified correctly.
Meantime,  after training the network reported only 4.53% of misclassification!
We consider this behavior as  a bug, thank you in advance for fixing it.
Best wishes.

Hi,
by your own words, you get 95% accuracy on CPU and 5% misclassification on GPU, so it doesn’t seem like a bug.  We checked the code and did not find any issues.

Sorry for insisting, but  the main isssue is that such a network cannot predict the class when trained on GPU!!!!
The actual accuracy (what I measured using fprop()) was 95% and 0.1% on CPU and GPU respectively, so misclassification rate was 5% and 99.9%
On the contrary, if checked by mlp.eval(valid_set, metric=Misclassification()), the reported rate was close to 5% in both cases.

if you're using softmax as an output activation, your cost function has to be multiclass cross entropy, not binomial cross entropy.  we don't support softmax -> binomial cross entropy or logistic -> multiclass cross entropy.

In all this experiments I'm using SumSquared() as a cost.
It works perfectly on CPU but fails on GPU.
I think this error is something technical not algorithmic.

It's a bit hard for us to follow what is required to reproduce the behavior that you are seeing. For example if you are training with a SumSquared cost, it seems like the model is doing regression, and it's not clear how you measure misclassification.
Generally, if you find a bug, the best way to go is to provide a minimal example that reproduces it. Preferably this would be modifying one of the examples slightly, but still using the original example data so that we can run it. We will then do our best to provide a speedy fix.

Actually, this situation does not depend on the last-layer activation function (softmax or logistic)
I was able to fix it by using the following code to get predictions:
X0= X_test [0:1000,:]
#X1= X0.transpose()
xx= gp.array(X0, dtype=float32)
xx1=xx.T
prdc= mlp.fprop(xx1)
For some reasons that are unknown to me, if I transpose the array before putting it to backend, the prediction fails, as I described above


Provide more context and links for untracked commits and deployments in Grafana UI.

Here:

Use the info link ( top left corner ) to point to documentation
Provide the names of the deployments that are not tracked.

Related to #380

Investigate: when documenting (and now), define why something isn't tracked.


fix missing prefix routes bug when workload cluster is created under …

This PR:

fixes missing prefix routes when workload cluster is created under another account than management cluster's

Towards https://github.com/giantswarm/roadmap/issues/2140
Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

It turned out that the change with getting subnets from the workload cluster account wasn't strictly necessary for this change. However I think we can keep it in since it makes sense.


User interface change for manga pro

Source information
Manga pro 1.4.33
Source language
Arabic
Steps to reproduce
I'm not sure what it's called in programming, but there was an update to the user interface
The source was down due to a change in the site's programming.
Before:

After:

There was an old bug, which no one reported, on the latest updates page, it was showing the latest work added to the site, not the latest chapter.

Expected behavior
Please Fix it
Actual behavior
Fix it
Mihon/Tachiyomi version
0.17.1
Android version
Android 14
Other details
No response
Acknowledgements

[ ] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Add a :+1: reaction to issues you find important.

I found out that someone beat me to reporting it.


Enable drug search in suggested-search api

@apierleoni @elipapa
Included drug-synonym fields in free text search api

I'm merging this, but we might need to stop using the dots in the field names, because they have some issues with querying them... _ to the rescue


Cannot register function as middleware due to Debug trait requirements

Looking at the middleware documentation, there is a implementation for Fn(Request<State>, Next<'a, State>) -> BoxFuture<'a, Result>, which should allow me to register an async function as a middleware without implementing a struct and trait. Unfortunately the Server::middleware function additionally has a requirement that the middleware implements the Debug trait. So the following example fails
async fn handle_error<State: Send + Sync + 'static>(
    ctx: Request<State>,
    next: Next<State>,
) -> Result {
    ...
}

...

let mut app = tide::new();
app.middleware(handle_error);

error[E0277]: `fn(tide::request::Request<_>, tide::middleware::Next<'static, _>) -> impl std::future::Future {handle_error::<_>}` doesn't implement `std::fmt::Debug`
  --> src/main.rs:97:20
   |
97 |     app.middleware(handle_error);
   |                    ^^^^^^^^^^^^ `fn(tide::request::Request<_>, tide::middleware::Next<'static, _>) -> impl std::future::Future {handle_error::<_>}` cannot be formatted using `{:?}` because it doesn't implement `std::fmt::Debug`
   |
   = help: the trait `std::fmt::Debug` is not implemented for `fn(tide::request::Request<_>, tide::middleware::Next<'static, _>) -> impl std::future::Future {handle_error::<_>}`

This makes implementing a middleware much more complex, especially for rust beginners.

Yeah that's fair — maybe we should get rid of the debug requirement, but add a default method on the middleware trait that contains the middleware name.

It seems part of this was fixed in #545, to me as a rust beginner the signature there still looks overly complex:
fn user_loader<'a>(
    mut request: Request<UserDatabase>,
    next: Next<'a, UserDatabase>,
) -> Pin<Box<dyn Future<Output = Result> + Send + 'a>> {

I'm still wondering if it might be possible to register a simple async function that returns a Result. An explanation why this might not be possible would also be very appreciated.

https://github.com/http-rs/tide/pull/556 makes this easier by exposing Before and After middleware constructors. We currently cannot implement the middleware trait for async fns, so this seems to be about the best we can do until the language catches up.

Sounds reasonable. If you want you can close this issue (or I can close), unless you want to keep it open as a reminder.


Missing setPageLoader(false) in BillingSameAsShippingCheckbox

https://github.com/hyva-themes/magento2-react-checkout/blob/main/src/reactapp/src/components/shippingAddress/components/BillingSameAsShippingCheckbox.jsx#L64-L67
It seems it was forgotten to reset the page loader, especially when form validation (field x is required) is triggered again when you check and uncheck the checkbox.

This will fix it:
    } catch (error) {
      setPageLoader(false); // this was missing
      console.error(error);
      setErrorMessage(__('Billing address update failed. Please try again.'));
    }

See for comparison the other implementations:
https://github.com/hyva-themes/magento2-react-checkout/blob/84b534251d955aba00a3400b9d1d5acb77be536d/src/reactapp/src/components/couponCode/components/CouponCodeForm.jsx#L37
https://github.com/hyva-themes/magento2-react-checkout/blob/71c95fbc8e10acf545910bab7f552b3310f02899/src/reactapp/src/components/paymentMethod/components/PaymentMethodFormManager.jsx#L42

Thank you for reporting the issue. Fix is created and will be available in the next release.


Fix curb on debug

This PR fixing issue: https://github.com/bblimke/webmock/issues/621

https://travis-ci.org/pavel-jurasek-bcgdv-com/webmock should work

@pavel-jurasek-bcgdv-com thanks for that.
I'm wondering if "faking" on_debug actually makes sense.
This code just pretends to be a real Curl.
If someone wants to debug what's going on, I'm not sure seeing a data generated by WebMock,
is what user wants to see.
What would be the usecase of using on_debug with WebMock enabled?

@bblimke well i add on_debug statements just because simplecov do not hit then all code.
Use case:
Building abstract business client (internally using curb). If this client will contain debug option (Foo.client.debug = true) and within the class will be specified:
@curb.on_debug do |foo, bar|
  >> loging to NewRelict in my case
end

test coverage will be not 100% since on_debug is not implemented in WebMock. It will produce 100% coverage without WebMock. I can simplify on_debug statements but i tried to produce as close as possible results.

I see. Thanks for explaining.
What you implemented, is it exactly what curb would output every time?
I wouldn't want to mislead people who use on_debug to trace some issue, with incorrect data.

@bblimke as close as i can go it not handle follow location requests but for intention (e.g. on_debug handling) should be fine.

@bblimke removed :) thanks to check code ! i over looked this.

Excellent :)

released as 2.1.0


Remove dates from job posts

This removes dates from our job postings. The rationale for this is that job postings may be up for a long time if we can't find a suitable candidate, or if multiple candidates are hired under one posting. If people see an old date then they may be discouraged from applying just because it is in the past. Since the date is not strictly related to the job posting (as long as it is active), then we can juts remove it to avoid this confusion.
It also fixes a boolean check in Hugo to choose whether to show templates. TOML treats true and false as different from true and false. I think we want the booleans explicitly instead of strings.

thanks @sgibson91 !


fix double dtype missing on CLANG

Related: https://github.com/tinygrad/tinygrad/issues/1548
double is introduced in this PR: https://github.com/tinygrad/tinygrad/pull/1525
but it's missing type mapping to CLANG
This PR fixes the issue

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/runtime/lib.py                    1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0


Some Channels throw error: "Cannot read property 'items' of undefined"

Describe the bug
A clear and concise description of what the bug is.
Some channels such as PewDiePie throw error.
(node:18860) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'items' of undefined
    at Channel.load (E:\WatchSquad\server\node_modules\youtubei\dist\classes\Channel.js:46:58)
    at Client.<anonymous> (E:\WatchSquad\server\node_modules\youtubei\dist\classes\Client.js:93:53)
    at Generator.next (<anonymous>)
    at fulfilled (E:\WatchSquad\server\node_modules\youtubei\dist\classes\Client.js:5:58)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:18860) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:18860) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

To Reproduce
Steps to reproduce the behavior:
// PewDiePie Channel ID is used.
client.getChannel("UC-lHJZR3Gqxm24_Vd_AJ5Yw").then(res => {
    console.log(res)
})

Expected behavior
A clear and concise description of what you expected to happen.
Should return Channel object.

Thanks for reporting this, will be fixed in next version


jump points based on current project

Is it possible to make the jump point list project dependent? As for now, the jump point list global in sublime text.
Thanks

Well, the answer is yes, it is probably possible, but why? project has folders, so normally you get list of them when call FileBrowser.
If you use ST3 you can add folder from within FileBrowser into project f, on ST2 you have to drag&drop folder(s) from Explorer/Finder onto ST.
Project dependent jump points would add too much complexity with small benefit, imho.

Then I probably misunderstand the purpose of jump points. I thought it was a quick way to jump to specific locations in a project instead of hard bookmarks in the whole filesystem. I understand that the benefit is small, it was more of a convenient if it was available. I usually just load the project folder instead of separate folders, some folder have a deep folder structure due to frameworks convenient so thought it would be great to set jump points, but as you can see the list get pretty long and polluted overtime and there are chances that I accidently jump outside the project folders.
--
Calvin Choy
On 22 Mar 2015 at 00:26:44, Vova Kolobok<EMAIL_ADDRESS>wrote:
Well, the answer is yes, it is probably possible, but why? project has folders, so normally you get list of them when call FileBrowser.
If you use ST3 you can add folder from within FileBrowser into project f, on ST2 you have to drag&drop folder(s) from Explorer/Finder onto ST.
Project dependent jump points would add too much complexity with small benefit, imho.
—
Reply to this email directly or view it on GitHub.

You can add subfolders into project as you like, e.g. let’s say you got
▾ app\
   ▸ libs\

you can add both into project as roots; and then if add "project": true argument to keybinding you’ll always get list of project folders.

Cool, I will try that out :) Thanks!


⚠ Samsung has degraded performance

In 95bca57, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2272 ms

Resolved: Samsung performance has improved in d6e7718.


Support 'has_one' relationship

Allows you to recursively support single association relationship types.
If you have JSON like
{
  "name": "Jeff"
  "address": {
    "street": "123 Fredrick St.",
    "city": "Metropolis",
  }
}

and classes like this
class Address < ActiveRestClient::Base
  def full_string
    return "#{self.street}, #{self.city}, #{self.region}, #{self.country}"
  end
end

class Person < ActiveRestClient::Base
  get :find, "/people/:id", :has_one => {:address => Address}
end

It will instantiate the address class instead of the Person class and you can then call things like
puts @person.address.full_string
If there is anything I can do to help get this merged I'd love to work with you guys on doing that, thanks!

Coverage decreased (-0.18%) to 98.99% when pulling 544a384f1a25296b4041dc9c21f893e271fa1898 on nathanhoel:has-one-association into ea0a7af1ec848e21a92e126f53a2071e1427a5b0 on whichdigital:master.

Coverage decreased (-0.18%) to 98.99% when pulling 544a384f1a25296b4041dc9c21f893e271fa1898 on nathanhoel:has-one-association into ea0a7af1ec848e21a92e126f53a2071e1427a5b0 on whichdigital:master.

Forgot to check specs again, will do then let you know

Coverage increased (+0.01%) to 99.18% when pulling fc56ec40e52633d17dc7383c537fb18789e6d50f on nathanhoel:has-one-association into ea0a7af1ec848e21a92e126f53a2071e1427a5b0 on whichdigital:master.

Coverage increased (+0.01%) to 99.18% when pulling fc56ec40e52633d17dc7383c537fb18789e6d50f on nathanhoel:has-one-association into ea0a7af1ec848e21a92e126f53a2071e1427a5b0 on whichdigital:master.

Nice work @nathanhoel, thank you!


Feature Request / Question

Hi,
Would it be possible to add lobby features or networking? I believe they're documented in the Game SDK documentation and would make life a lot easier. If this would take too much time, please could you point me in the direction of adding them - I have tried to compile the library on my own using the latest Game SDK, to no avail.
Thanks! :)

That is a really interesting idea, but I am afraid that it will take a lot of time as there are a lot of functions to implement.
So if you want to try it yourself, here is a short guide on how to set up a development environment:

Install a C compiler:
As the library relies on native code included with JNI, a C compiler is required to build it. On Windows I'd recommend using the one from Visual Studio (but not actually using Visual Studio). On Linux it's a lot easier as you can just use gcc.
Install CMake (https://cmake.org):
The project uses CMake to build the native libraries (which makes it possible to use mostly the same build process for Windows and Linux). Installing it should be easy on Linux, but might be a bit more difficult on Windows. Mainly make sure that it is in your %PATH%, so you can access it from anywhere.
Install Maven:
The project is built using Apache Maven. If you are using an IDE, you can probably skip this step tho, as it should provide its own version of Maven.
Download and extract the Discord Game SDK:
Download the Game SDK and extract it to ./discord-game-sdk (relative from the repository root). After this step the directory structure should look something like this:

Run mvn test:
This Maven goal will build the native libraries, compile the Java code and also run a few tests. Some tests will probably fail, because I included some hard coded user IDs and stuff like that. Running mvn compile is not enough tho, as it will not copy the native libraries to the right place. But you can skip the tests by doing mvn test -DskipTests.
Now you should have a development environment which should be able to compile the library. I will add a little guide on how I'd recommend getting started with including new features tomorrow.

If you encounter any problems with this setup, please tell me and I will try my best to help you revolve them. I know that this guide is pretty messy, so don't hesitate to ask if you have any questions or something isn't clear :)

I've got it building using cmd, I just couldn't really figure out how to implement new functions - haven't exactly created Java bindings for the Game SDK before.
The files (e.g. this) aren't exactly the easiest to understand at first glance, and so I was having some slight issues in trying to figure out how it was implemented so that I could add the lobby functionality.

Ah, I see.
The main idea is that the "normal" Java functions call native C functions which then call the Game SDK.
For callbacks it works exactly the other way around: The SDk calls a provided C function which then calls a Java function. This for example happens in line 35.
Another thing that needs to be done by the native library is converting between C structs and Java objects. That e.g. is what create_java_relationship (line 7) is for. To do so, it first needs to find the corresponding Java classes (FindClass) and their constructors (GetMethodID(..., "<init>", ...)) and then finally create the objects (NewObject). For some objects there is a static method that is called instead of the constructor (e.g. createRelationship), but the overall procedure is similar.
Another confusing thing might be this CallbackData. Those structs are passed to the Discord SDK and then to our callback function. They provide an easy way to tell the C callback which Java method (usually of a Consumer or Predicate) it is supposed to call.
I hope this could help a bit to get a general overview.
I'm sorry if this is a bit confusing, but where I am it's currently 3:45, so I might be a little sleepy already.

I made a little example of how I would implement those functions:
You can find it in 27a435db208dc1f0fabde47cc689a00082bcfab6.
If you want to work on it, maybe just fork and use the lobby-stub branch.
I'd even welcome little changes, so if you want to, maybe just implement a few easy functions and make a pull request.
I also hope that this example is a little easier to understand than the messy relationship code.
But if you have any particular questions or need help to understand it, feel free to ask :D
If you don't want to work on it or don't have time to get into my messy code, just tell me, so I will work on implementing the functions.

Ok, I've implemented most lobby functions now (see https://github.com/JnCrMx/discord-game-sdk4j/tree/lobby-stub).
I'm currently working on networking (which seems to be a bit complicated, but I was able to set up a minimal working example already).

Sorry for closing, it was a misclick. I will keep it open until I'm done with lobby and networking functions.

Thank you for helping with this - I didn't really understand the conversion, but am incredibly grateful for your continued help in using this SDK.

Update:
I'm currently working on the Javadoc for Lobby functions, but I have finished implementation for Lobby as well as Network bindings.
Javadoc is usually one of the less interesting tasks, so I'm not sure how quick I will have it finished.
If you want to test/use the new functions right away, either checkout networking (or lobbies, but networking is mostly built upon lobbies, so you get both when choosing networking) and build the library yourself, or if you don't want to bother with compiling, tell me, so I build it and make some pre-releases or something like that. :D

Update:
I'm currently working on the Javadoc for Lobby functions, but I have finished implementation for Lobby as well as Network bindings.
Javadoc is usually one of the less interesting tasks, so I'm not sure how quick I will have it finished.
If you want to test/use the new functions right away, either checkout networking (or lobbies, but networking is mostly built upon lobbies, so you get both when choosing networking) and build the library yourself, or if you don't want to bother with compiling, tell me, so I build it and make some pre-releases or something like that. :D

I'm done with the documentation for the Lobby functions now. It took me a lot longer than I expected though.
I merged the Lobby branch into master and will hopefully be able to make a new release today.

I'm done with the documentation for the Lobby functions now. It took me a lot longer than I expected though.
I merged the Lobby branch into master and will hopefully be able to make a new release today.

Thank you for helping so much with this - it means a lot, and I hope I can somehow make it up to you in future!

Thank you for helping so much with this - it means a lot, and I hope I can somehow make it up to you in future!

networking was merged into master by #15 and 7d91e88358fdb14f0d78421ffb0ac750b7c4cbc0.
I will include it in the next release (0.5), which I'm working on at the moment.

networking was merged into master by #15 and 7d91e88358fdb14f0d78421ffb0ac750b7c4cbc0.
I will include it in the next release (0.5), which I'm working on at the moment.


opt links to versioned document

:tada: This issue has been resolved in version 0.0.5 :tada:
The release is available on:

v0.0.5
GitHub release

Your semantic-release bot :package::rocket:


Allow for multiple linked notes in inline data

Hello,
Thank you for this update! Inline data really opens up even more possibilities. No need for the meta data to be hidden.
Would it be possible to support multiple linked notes in inline data ?
For now it only supports one linked note (I even tried various array styles but to no avail).

Have you tried updating to the latest version? It should have explicit support for lists of links as desired.

Oh sorry, I missed that one. Awesome. Thank you for this amazing plugin and your work.


Errno::ECONNRESET: Connection reset by peer - SSL_connect

Running 1.9.3 and Rails 3.2.14
I've been getting
Errno::ECONNRESET: Connection reset by peer - SSL_connect
I've tried this on multiple dev machines, made sure it wasn't an issue with openssl, all to no avail. Anyone experience the same / have a suggestion for what could be causing it?
Thanks

I'm in China, I get the same problem because of the GFW(Great FireWall).

I was looking for a while already, to get this nasty bug fixed while running most recent versions of ruby and gem on Unbuntu 14.04. Then I read zhizouxiaos comment. Sitting in China as well, I should have tried with a VPN earlier. Made my day. Thanks for giving this hint!

Mario you can just use Chinese mirror http://ruby.taobao.org/

Thanks for this advice lammersons. Tried, and works flawlessly without VPN.

@zhizouxiao @lamersons ,Thanks for your responses, I met the same problem and tried several ways but no effect, your advice and solution save my day.

@lamersons thank you very much , it worked!

Same here, thank you @lamersons!

What finally did it for me. Shutting down IPv6!
networksetup setv6off "Wi-Fi"

All is good now.


Missing flag: AN

Flag of Netherlands Antilles (country code AN) is missing.

Why was this closed, without any explanation? Thanks

Ok, thanks for explanation.


Regex documentation is confusing

The module documentation does not tell you how to get a Regex object.
The Regex class documentation shows you illegal code: Regex("[0-9]+ years") is illegal for two reasons – Regex is abstract and sealed.
Nothing points to the regex function you actually use to get a Regex object.

The combination of these three circumstances can be very confusing to users.

Oh, and…
    """Applies the regular expression to the given string. This call affects the
       value returned by {@link #getLastIndex()} if the global flag is set.
       Produces a [[match result|MatchResult]] if the string matches, else `null`.
       """

Javadoc syntax in Ceylondoc‽

Thanks @quintesse!


(model): standalone py/rust packages for capnproto generated code

Separate workflows should generate code from capnproto, so that it is not needed as a system dependency for hugr use.
It may be convenient to make the generated code standalone-separately versioned packages.
See: https://github.com/cqcl-dev/quantinuumapis/

As long as the workflow stays optional in the local dev process.  Having the capnproto cli tool around at devtime is quite easy to manage; integrating CI workflows when trying out new changes is a lot more tricky.
(Note that the capnproto toolchain isn't a runtime dependency, so it only has to be around when building the packages and not when using them)


add tests

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

#17293  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @xxchan and the rest of your teammates on  Graphite

avoid manipulating columns manually, which bypasses ColumnIdGenerator and can be problematic if we support ALTER TABLE with connector

https://github.com/risingwavelabs/risingwave/issues/9828

https://github.com/risingwavelabs/risingwave/pull/17336 change a position to unify the additional columns related logic

I finally found the reason of CI failure, and why I can't reproduce it locally: When using rpk topic produce --schema-id=topic, it looks for schema registry of the redpanda, but the schema is in confluent schema registry... 🤡
Unfortunately rpk topic produce returns 0 when it meet error..
unable to build value serializer using TopicNameStrategy for topic "avro_alter_source_test": unable to get schema with name "avro_alter_source_test-value" using TopicName strategy: unable to GET "http://<IP_ADDRESS>:8081/subjects/avro_alter_source_test-value/versions/latest": Get "http://<IP_ADDRESS>:8081/subjects/avro_alter_source_test-value/versions/latest": dial tcp <IP_ADDRESS>:8081: connect: connection refused


Multiple Resources

I have an existing maven project wich already defines its own src/main/webapp/META-INF/context.xml file.
In that file is already defined the <Resources>...</Resources> element.
When I start the Tomcat (my version is 8.5), the server.xml file handled by the plugin is rewritten with two <Resources>...</Resources> elements: the one of the maven project and the one generated by the plugin.
Tomcat starts without any error but the instruction declared in the maven project is ignored.
I think would be better try to detect the existance of that element, and if it exists append the <PreResources> and <PostResources> inside that element.
Intellij & SmartTomcat Version (Help -> About copy & paste below)
IntelliJ IDEA 2022.1.2 (Community Edition)
Build #IC-221.5787.30, built on May 31, 2022
Runtime version: 11.0.15+10-b2043.56 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 2024M
Cores: 8
Non-Bundled Plugins:
idea.plugin.protoeditor (221.5787.13)
com.poratu.idea.plugins.tomcat (4.2.0)
camtwoTomcatRunner (1.7)
Docker (221.5787.15)
Kotlin: 221-1.6.21-release-337-IJ5787.30

@matteopic It has been fixed in https://github.com/zengkid/SmartTomcat/commit/28c7f27b0f2249f0ceb718760658dcb621f6a410, will release soon.

Released in 4.3.4


Research a proper monorepo workflow

The amount of packages in the repository is growing but the publishing is done manually. This is not optimal. We should take inspiration from other monorepos (ethers, web3.js) and automate the process.

example: https://github.com/emotion-js/emotion

Some requirements I can think of:

Packages at the development branch are deployed as alpha versions
Packages at the master branch are deployed as beta versions
Tags are deployed as production versions
Only changed packages are deployed
If the updated package is a dependency of another package, it should be updated too
Versioning is managed automatically (semantic-release?)

cc @katspaugh


Bump to latest version

I appreciate the work you did here, but there was already a pull request that did this change.

Done in #4449


Problem mentioning users when sending messages

Hello
I use tdlib 1.8.3.
I want to send a HTML style text message and mention some users.
My text is completely correct because the test robot does not give any errors.
Only the main robot gives the following error:
[ 0][t 3][1655630390.567631244][MessageEntity.cpp:4171][#1][!MessagesManager][&r_input_user.is_ok()]	do_send_message [Error : 400 : User not found] TDLib fatal error:  b'[ 0][t 3][1655630390.567631244][MessageEntity.cpp:4171][#1][!MessagesManager][&r_input_user.is_ok()]\tdo_send_message [Error : 400 : User not found]\n' Aborted
Thanks for guiding me.

Could you send log with verbosity level 4 to https://t.me/tdlib_bot?

Could you send log with verbosity level 4 to https://t.me/tdlib_bot?

Yes, I can send it to you.

Did you send the log?


accept arbitrary urls

Describe the bug

Alt image text is checked, but there we should be a bit more lax and allow arbitrary urls or until #44 the alt image text should be ignored. Normally this is never shown anyways.
To Reproduce
Steps to reproduce the behaviour:

A file containing [![crates.io](https://img.shields.io/crates/v/cargo_spellcheck.svg)](https://crates.io/crates/cargo-spellcheck)
Run cargo spellcheck README.md

Expected behavior

Please complete the following information:

System: Fedora
Obtained: git

Additional context

This is an issue with the tokenizer too, it splits on . tokens. But we could whitelist valid urls in alt image names.
Note that this can most likely not be solved by a user dictionary, since it will never see the full url, but only subtokens.

Since this has so much overlap with #44, I am assigning this to you as well - maybe an additional test case will already be enough given your work on #113

it makes sense. So do you mean that the altext should not be corrected in case is followed by a link? or that the link should not be corrected?
This is what it looks in this branch: laysa-cmark-links-handled
error: spellcheck(Hunspell)
  --> /home/tmhdev/Documents/cargo-spellcheck/README.md:2
   |
 2 | ![altext](filenamve)
   |   ^^^^^^
   | - alt ext, alt-ext, textual, textural, external, or exalt
   |
   |   Possible spelling mistake found.

error: spellcheck(Hunspell)
  --> /home/tmhdev/Documents/cargo-spellcheck/README.md:3
   |
 3 | [![crates.io](https://img.shields.io/crates/v/cargo_spellcheck.svg)](https://crates.io/crates/cargo-spellcheck)
   |           ^^
   | - oi, Io, ii, ion, bio, Rio, is, or one of 7 others
   |
   |   Possible spelling mistake found.

I think the alt text should be checked if it is an url, if so, just accept it. If not, check the text like any other. I think that makes most sense. Usually alt text contains only a few key words or an url.

For the example,
[![crates.io](https://img.shields.io/crates/v/cargo_spellcheck. svg)] (https://crates.io/crates/cargo-spellcheck)
What should be checked in this case?highlighted in negrito.

[ ] [![crates.io](https://img.shields.io/crates/v/cargo_spellcheck. svg)](https://crates.io/crates/cargo-spellcheck)
[ ] [![crates.io](https://img.shields.io/crates/v/cargo_spellcheck. svg)] (https://crates.io/crates/cargo-spellcheck)
[ ] [![crates.io](https://img.shields.io/crates/v/cargo_spellcheck.svg)] (https://crates.io/crates/cargo-spellcheck)

Ignore that I added some spaces just to avoid showing images or make links.

I would expect that crates.io is checked if it is a valid url, and if so it is skipped to be spell checked.
Additionally, if requested by the user, both urls should be checked for existence https://img.shields.io/crates/v/cargo_spellcheck.svg https://crates.io/crates/cargo-spellcheck ).
Does that make sense?

To speed this up, we could borrow some code from https://github.com/deadlinks/cargo-deadlinks , which works ontop of html files. We could re-use the link check code, and apply it directly on the source, without having to deal with the html parsing. Just an idea though.


support F shortcut for full screen in videos

F is the youtube shortcut to full screen.

Fixed by https://github.com/microsoft/devicescript/commit/f923ab95d0b1cfaacbab17131b3ed0666874dfe2


Cannot find name 'AbortSignal'

I'm submitting a bug report

Library Version:
1.4.0

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
6.2.0

NPM Version:
3.8.9

JSPM OR Webpack AND Version
webpack 2.6.1

Browser:
Chrome 69.0.3497.81 (Official Build) (64-bit) (cohort: 69_win_81), Edge 11

Language:
TypeScript X.X

Current behavior:
node_modules\aurelia-fetch-client\dist\aurelia-fetch-client.d.ts
(111,12): error TS2304: Cannot find name 'AbortSignal'.
1.3.1 works fine. If the problem is that 1.4.0 requires other updates then it would be handy to list them.

What is the expected behavior?
Not barfing

What is the motivation / use case for changing the behavior?
Barfing is bad

@fkleuver So we should update this line?
"typescript": "^1.9.0-dev.20160622-1.0",
https://github.com/aurelia/fetch-client/blob/master/package.json#L88

@davismj Wow, 1.9.0-dev. Yeah, that ought to be updated..

I'm not sure how to solve this. Updating the dev dependency won't fix OPs issue. Adding a peer dependency ^2.7 might, but then will raise annoying warnings to non TypeScript users. Perhaps we add console output on install?

I only logged the issue as a courtesy. Just put it in the release notes if it's a huge PITA to fix. All you need to say is either update TS or use 1.3.1 if you can't update TS for some reason.


js.Global().Call("someFunc") - How to handle error responses gracefully ?

What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
How do you handle error responses from a JS function so that the app  does not stop execution and exit ?

What did you expect to see?
What did you see instead?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


[BUG]: Error processing condition on org.springframework.cloud.stream.config.BindingServiceConfiguration.binderFactory

Describe the bug
Hey guys,
this is my build.gradle:
build.gradle.txt
After starting of spring an exception occured.
Exception or Stack Trace
exception.txt
To Reproduce

Create new Spring boot project
Override build.gradle with my build.gradle
Start

Setup (please complete the following information):

OS: Windwos 10
IDE: IntelliJ
Library/Libraries: com.azure.spring:spring-cloud-azure-stream-binder-servicebus:6.0.0-beta.2
Java version: 17
App Server/Environment: Tomcat
Frameworks: Spring Boot

Hi @bitbaggi ， the Spring Cloud Azure 6.0.0-beta.2 is based on Spring Boot 3.0.0-M4 and Java 17, could you try to upgrade the Spring Boot version? Please check the reference doc https://microsoft.github.io/spring-cloud-azure/6.0.0-beta.2/reference/html/index.html#what-is-new-in-6-0-since-4-x

Nice, its working.
Thank you very much ☺

@bitbaggi , the 6.0.0-beta.2 of spring-cloud-azure-stream-binder-servicebus has a bug which blocks the sending feature. It has been fixed in https://github.com/Azure/azure-sdk-for-java/pull/31447. So if you want to use the sending function, please expect our next release.

@yiliuTo, should we update the IT to cover such case?

@saragluna yeah I agree, this issue https://github.com/Azure/azure-sdk-for-java/issues/31491 I think could cover the IT for such case

Hi @bitbaggi , the 6.0.0-beta.3 version is coming, since the 6.0.0-beta.2 is a new beta version based on Spring Boot 3 milestone version, I am wondering how you find this version for your application, this is just a survey for our side to improve the documentation or other aspect, any comments are welcome!

Hi @bitbaggi , the Spring Cloud Azure 6.0.0-beta.3 is available now, please check if you want to use this version, https://microsoft.github.io/spring-cloud-azure/6.0.0-beta.3/reference/html/

@bitbaggi I'm wondering where you saw the com.azure.spring:spring-cloud-azure-stream-binder-servicebus:6.0.0-beta.2 and add this into your build.gradle? the 6.x version is compatible with Spring Boot 3 not 2.7, maybe we have something on document to improve and avoid further mistake happen

It could be dependabot https://github.com/Azure-Samples/azure-spring-boot-samples/pull/555.

@saragluna  @moarychan with regard to this issue, since we don't know how to config dependabot to not promote our 6.0 to a spring boot 2.x application, I think we need to add clear error message when our 6.0 is used in a spring boot 2.x application to inform users how to fix it

@stliu thanks, good point. @moarychan will the compatibility verifier handle this? Or is the exception happens before the compatibility verifier?

Hi all, I re-examined this issue, sorted it out as follows, and concluded that there is a user scenario that we have not covered.
The brief summary is:
When using our 6.x(6.0.0-beta.2) spring-cloud-azure-stream-binder-xxx library, the Spring Boot version is locked at 2(2.7.4), and the Spring Cloud version(2021.0.4) is not locked, which will cause current problems for users.
The detailed analysis is as follows:
1 Overview
Since Spring Boot(SB) ships multiple major versions(2.7..x and 3.x), and Spring Cloud Azure(SCA) also ships multiple major versions(4.x and 6.x) to match the Spring Boot version, At the same time, there are multiple versions(2021 & 2022) corresponding to Spring Cloud(SC). If the user does not read this SCA version mapping carefully, it is likely to use the wrong version.
1.1 Customers use SCA 4 in an SB 3 application
When the customer uses the SCA 6 scenario in the SB 2 application, the error message is as expected. Assuming that the customer has a Gradle configuration similar to the following, when the customer starts the application, the following failure report should be reported to the user first.
Customer Gradle configuration:
plugins {
    id 'java'
    id 'org.springframework.boot' version '3.0.0-M4'
    id 'io.spring.dependency-management' version '1.0.9.RELEASE'
}

group 'de.test'
version '1.1'
sourceCompatibility = '17'

repositories {
    mavenLocal()
    mavenCentral()
    gradlePluginPortal()
    maven { url 'https://repo.spring.io/release' }
    maven { url 'https://repo.spring.io/milestone' }
}

dependencies {
    implementation 'org.springframework.boot:spring-boot-starter-web'
    implementation 'com.azure.spring:spring-cloud-azure-stream-binder-servicebus:4.4.1'
    testImplementation 'org.springframework.boot:spring-boot-starter-test'
}

1.2 Customers use SCA 6 in an SB 2 and SC 2021 application
If customers add the dependencyManagement with Spring Cloud 2021.0.4, Gradle configuration like below:
plugins {
    id 'java'
    id 'org.springframework.boot' version '2.7.4'
    id 'io.spring.dependency-management' version '1.0.14.RELEASE'
//    id 'org.springframework.experimental.aot' version '0.12.1'
}

group = 'com.example'
version = '0.0.1-SNAPSHOT'
sourceCompatibility = '17'

repositories {
    mavenLocal()
    mavenCentral()
    maven { url 'https://repo.spring.io/release' }
}

ext {
    set('springCloudVersion', "2021.0.4")
}

dependencies {
    implementation 'org.springframework.boot:spring-boot-starter-web'
    implementation 'com.azure.spring:spring-cloud-azure-stream-binder-servicebus:6.0.0-beta.2'
    testImplementation 'org.springframework.boot:spring-boot-starter-test'
}

dependencyManagement {
    imports {
        mavenBom "org.springframework.cloud:spring-cloud-dependencies:${springCloudVersion}"
    }
}

tasks.named('test') {
    useJUnitPlatform()
}

tasks.named('bootBuildImage') {
    builder = 'paketobuildpacks/builder:tiny'
    environment = ['BP_NATIVE_IMAGE': 'true']
}

then the below failure report is as expected:

But other situations may not be friendly to help customers.
2 Problem statement
The SCA 4 customers may notice the new version of SCA 6, they may try to upgrade to SCA 6, when they forget to lock the SC version to 2021. they will get the unclear error message encountered before the SB version compatibility validation.
This is because the AOT support of Spring Framework 6 will bring up lots of callback (Such as BeanRegistrationAotProcessor in org.springframework.cloud.stream.binder.BinderChildContextInitializer) process, and the implementations are referenced to the new Spring Cloud Stream(SCS) 2022.
3 User stories
3.1 Customers use SCA 6 in an SB 2 application
Customer Gradle configuration:
plugins {
    id 'java'
    id 'org.springframework.boot' version '2.7.4'
    id 'io.spring.dependency-management' version '1.0.14.RELEASE'
}

group = 'com.example'
version = '0.0.1-SNAPSHOT'
sourceCompatibility = '17'

repositories {
    mavenLocal()
    mavenCentral()
    gradlePluginPortal()
    maven { url 'https://repo.spring.io/release' }
    maven { url 'https://repo.spring.io/milestone' }
}

dependencies {
    implementation 'org.springframework.boot:spring-boot-starter-web'
    implementation 'com.azure.spring:spring-cloud-azure-stream-binder-servicebus:6.0.0-beta.2'
    testImplementation 'org.springframework.boot:spring-boot-starter-test'
}

a clear failure report should be output to the customer, not the error message with a long stack trace of Spring.
the clear error message should contain the below items:

The root cause of the error is using the wrong dependency SCA 6.
Prompt to downgrade to SCA 4.

@moarychan please help me understand this, the error messages from the screenshots you pasted are our current behavior?

I'm still a little bit lost here.
could you explain these scenarios, I can see some are discussed and some arent'
Secnarios
No Spring Cloud

[ ] Spring Boot 2 + Spring Cloud Azure 4
[ ] Spring Boot 2 + Spring Cloud Azure 6
[ ] Spring Boot 3 + Spring Cloud Azure 4
[ ] Spring Boot 3 + Spring Cloud Azure 6

With Spring Cloud

[ ] Spring Boot 2 + Spring Cloud Azure 4 + Spring Cloud x?
[ ] Spring Boot 2 + Spring Cloud Azure 6 + Spring Cloud x?
[ ] Spring Boot 3 + Spring Cloud Azure 4 + Spring Cloud x?
[ ] Spring Boot 3 + Spring Cloud Azure 6 + Spring Cloud x?

1 Overview
Since Spring Boot(SB) has released multiple major versions(2.x and 3.x), and Spring Cloud Azure(SCA) also has released multiple major versions(4.x and 6.x) to match the SB version, At the same time, there are multiple versions(2021.x & 2022.x) corresponding to Spring Cloud(SC). If the customer does not read this SCA version mapping carefully, it is likely to use the wrong version.
2 Problem statement
If customers use the below combination, the application will fail to start and the error message is not clear or actionable.

SB 2.x + SCA 6.x
SB 3.x + SCA 4.x

SCA should make the application fast fail with a clear message.
3 User scenarios
Note:

'SCA with SC component' means the following artifacts

spring-cloud-azure-starter-stream-eventhubs
spring-cloud-azure-starter-stream-servicebus
spring-cloud-azure-stream-binder-eventhubs
spring-cloud-azure-stream-binder-servicebus

'SCA not SC component' means all SCA artifacts except those listed above.

3.1 Scenario: SB 2.x + SCA 6.x(not SC component)
3.1.1 Current behavior and message
The application fails to start, the error message as below:
Description:

Your project setup is incompatible with our requirements due to following reasons:
- Spring Boot [2.7.4] is not compatible with this Spring Cloud Azure version.

Action:

Consider applying the following actions:
- Change Spring Boot version to one of the following versions [3.0.0].
You can find the latest Spring Boot versions here [https://spring.io/projects/spring-boot#learn].
If you want to learn more about the Spring Cloud Azure compatibility, you can visit this page [https://aka.ms/spring/versions] and check the [Which Version of Spring Cloud Azure Should I Use] section.
 If you want to disable this check, just set the property [spring.cloud.azure.compatibility-verifier.enabled=false].

3.1.2 Except behavior and message
The application fails to start, and a more accurate message should be output as below:
Description:

Your project setup is incompatible with our requirements due to following reasons:
- Spring Boot [2.7.4] is not compatible with this Spring Cloud Azure version.

Action:

Consider applying the following actions:
- Change Spring Boot version to one of the following versions [3.0.0-M4].
You can find the latest Spring Boot versions here [https://spring.io/projects/spring-boot#learn].
If you want to learn more about the Spring Cloud Azure compatibility, you can visit this page [https://aka.ms/spring/versions] and check the [Which Version of Spring Cloud Azure Should I Use] section.
If you want to disable this check, just set the property [spring.cloud.azure.compatibility-verifier.enabled=false].

3.2 Scenario: SB 2.x + SCA 6.x(with SC component)
3.2.1 Current behavior and message
The application fails to start, this is expected, but the error message is not clear and the customer is having a difficult time finding the root cause of the failure. See full stack trace from this issue description.
3.2.2 Except behavior and message
The application fails to start, and instead of outputting an error stack trace, it should output a clear message, as expected by 3.1.2.
3.3 Scenario: SB 3.x + SCA 4.x(not SC component)
3.3.1 Current behavior and message
The application fails to start, the error message as below:
Description:

Your project setup is incompatible with our requirements due to following reasons:
- Spring Boot [3.0.0-M4] is not compatible with this Spring Cloud Azure version.

Action:

Consider applying the following actions:
- Change Spring Boot version to one of the following versions [2.5.x, 2.6.x, 2.7.x].
You can find the latest Spring Boot versions here [https://spring.io/projects/spring-boot#learn].
If you want to learn more about the Spring Cloud Azure compatibility, you can visit this page [https://aka.ms/spring/versions] and check the [Which Version of Spring Cloud Azure Should I Use] section.
 If you want to disable this check, just set the property [spring.cloud.azure.compatibility-verifier.enabled=false].

3.3.2 Except behavior and message
Behavior and error messages as expected.
3.4 Scenario: SB 3.x + SCA 4.x(with SC component)
3.4.1 Current behavior and message
The application fails to start, the error message as below:
Description:

Your project setup is incompatible with our requirements due to following reasons:
- Spring Boot [3.0.0-M4] is not compatible with this Spring Cloud Azure version.

Action:

Consider applying the following actions:
- Change Spring Boot version to one of the following versions [2.5.x, 2.6.x, 2.7.x].
You can find the latest Spring Boot versions here [https://spring.io/projects/spring-boot#learn].
If you want to learn more about the Spring Cloud Azure compatibility, you can visit this page [https://aka.ms/spring/versions] and check the [Which Version of Spring Cloud Azure Should I Use] section.
 If you want to disable this check, just set the property [spring.cloud.azure.compatibility-verifier.enabled=false].

3.4.2 Except behavior and message
Behavior and error messages as expected.
4 Goals
Verify compatibility between the Spring Boot version and Spring Cloud Azure version.
5 Non-Goals
Not verify compatibility with Spring Cloud version.
6 Exit Criteria(TBC)
7 Follow up considerations
7.1 What's the alignment to choose
7.1.1 Alignment 1

Spring Cloud verifies Spring Boot version

Spring Cloud Azure verifies Spring Boot version

7.1.1 Alignment 2

Spring Cloud Azure verifies Spring Boot version
Spring Cloud Azure verifies Spring Cloud version

7.2 What phases of verification occur

On JVM startup

On Application Context bean initialization (SCA selected)

On Application Context other stages

7.3 SCA verifies which version

Only Spring Boot version

Spring Boot and Spring Cloud

7.4 Who to enable SB and SC verification when using SC component-based SCA

SCA
Spring Cloud Stream

Superseded by https://github.com/Azure/azure-sdk-for-java/issues/32593


Cannot PASTE into "Destination Address" field when sending.

I'm using a a MacBook Pro Running SIerra 10.12.5, pasting in the send address field isn't working.

This needs to be done for it
https://pracucci.com/atom-electron-enable-copy-and-paste.html

 [Wallet-GUI] OS X copy/paste

Fixed in #151


HIGH security risk using rtsp-simple-server

Hello, I'm using rtsp-simple-server and lately scanned it with X-Ray security scanner by JFROG.
The project has 15 published security vulnerabilities, some are HIGH risk.
Is it possible to update GO or erase the problematic files from the project?
These are the results, to my knowledge, all of them because of old GO version:
resources/cpprtspserver/rtsp-simple-server folder contains golang.org/x/net component, which contains the following vulnerabilities:
CVE-2022-41723
A maliciously crafted HTTP/2 stream could cause excessive CPU consumption in the HPACK decoder, sufficient to cause a denial of service from a small number of small requests.
CVE-2022-41721
A request smuggling attack is possible when using MaxBytesHandler. When using MaxBytesHandler, the body of an HTTP request is not fully consumed. When the server attempts to read HTTP2 frames from the connection, it will instead be reading the body of the HTTP request, which could be attacker-manipulated to represent arbitrary HTTP2 requests.
CVE-2022-27664
In net/http in Go before 1.18.6 and 1.19.x before 1.19.1, attackers can cause a denial of service because an HTTP/2 connection can hang during closing if shutdown were preempted by a fatal error.
resources/cpprtspserver/rtsp-simple-server folder contains golang.org/x/crypto component, which contains the following vulnerabilities:
CVE-2022-27191
The golang.org/x/crypto/ssh package before 0.0.0-20220314234659-1baeb1ce4c0b for Go allows an attacker to crash a server in certain circumstances involving AddHostKey.
CVE-2022-30636
Go Cryptography acme/autocert/cache.go Get() Function Path Traversal File Disclosure Weakness
CVE-2021-43565
The x/crypto/ssh package before 0.0.0-20211202192323-5770296d904e of golang.org/x/crypto allows an attacker to panic an SSH server.
resources/cpprtspserver/rtsp-simple-server folder contains github.com/lang/gogo component, which contains the following vulnerabilities:
CVE-2022-41725
A denial of service is possible from excessive resource consumption in net/http and mime/multipart. Multipart form parsing with mime/multipart.Reader.ReadForm can consume largely unlimited amounts of memory and disk files. This also affects form parsing in the net/http package with the Request methods FormFile, FormValue, ParseMultipartForm, and PostFormValue. ReadForm takes a maxMemory parameter, and is documented as storing "up to maxMemory bytes +10MB (reserved for non-file parts) in memory". File parts which cannot be stored in memory are stored on disk in temporary files. The unconfigurable 10MB reserved for non-file parts is excessively large and can potentially open a denial of service vector on its own. However, ReadForm did not properly account for all memory consumed by a parsed form, such as map entry overhead, part names, and MIME headers, permitting a maliciously crafted form to consume well over 10MB. In addition, ReadForm contained no limit on the number of disk files created, permitting a relatively small request body to create a large number of disk temporary files. With fix, ReadForm now properly accounts for various forms of memory overhead, and should now stay within its documented limit of 10MB + maxMemory bytes of memory consumption. Users should still be aware that this limit is high and may still be hazardous. In addition, ReadForm now creates at most one on-disk temporary file, combining multiple form parts into a single temporary file. The mime/multipart.File interface type's documentation states, "If stored on disk, the File's underlying concrete type will be an *os.File.". This is no longer the case when a form contains more than one file part, due to this coalescing of parts into a single file. The previous behavior of using distinct files for each form part may be reenabled with the environment variable GODEBUG=multipartfiles=distinct. Users should be aware that multipart.ReadForm and the http.Request methods that call it do not limit the amount of disk consumed by temporary files.
CVE-2022-2880
Requests forwarded by ReverseProxy include the raw query parameters from the inbound request, including unparseable parameters rejected by net/http. This could permit query parameter smuggling when a Go proxy forwards a parameter with an unparseable value. After fix, ReverseProxy sanitizes the query parameters in the forwarded query when the outbound request's Form field is set after the ReverseProxy. Director function returns, indicating that the proxy has parsed the query parameters. Proxies which do not parse query parameters continue to forward the original query parameters unchanged.
CVE-2022-41715
Programs which compile regular expressions from untrusted sources may be vulnerable to memory exhaustion or denial of service. The parsed regexp representation is linear in the size of the input, but in some cases the constant factor can be as high as 40,000, making relatively small regexps consume much larger amounts of memory. After fix, each regexp being parsed is limited to a 256 MB memory footprint. Regular expressions whose representation would use more space than that are rejected. Normal use of regular expressions is unaffected.
CVE-2022-41716
Due to unsanitized NUL values, attackers may be able to maliciously set environment variables on Windows. In syscall.StartProcess and os/exec.Cmd, invalid environment variable values containing NUL values are not properly checked for. A malicious environment variable value can exploit this behavior to set a value for a different environment variable. For example, the environment variable string "A=B\x00C=D" sets the variables "A=B" and "C=D".
CVE-2022-41724
Large handshake records may cause panics in crypto/tls. Both clients and servers may send large TLS handshake records which cause servers and clients, respectively, to panic when attempting to construct responses. This affects all TLS 1.3 clients, TLS 1.2 clients which explicitly enable session resumption (by setting Config.ClientSessionCache to a non-nil value), and TLS 1.3 servers which request client certificates (by setting Config.ClientAuth \u003e= RequestClientCert).
CVE-2022-2879
Reader.Read does not set a limit on the maximum size of file headers. A maliciously crafted archive could cause Read to allocate unbounded amounts of memory, potentially causing resource exhaustion or panics. After fix, Reader.Read limits the maximum size of header blocks to 1 MiB.
CVE-2022-41722
A path traversal vulnerability exists in filepath.Clean on Windows. On Windows, the filepath.Clean function could transform an invalid path such as "a/../c:/b" into the valid path "c:\b". This transformation of a relative (if invalid) path into an absolute path could enable a directory traversal attack. After fix, the filepath.Clean function transforms this path into the relative (but still invalid) path ".\c:\b".
CVE-2022-41720
On Windows, restricted files can be accessed via os.DirFS and http.Dir. The os.DirFS function and http.Dir type provide access to a tree of files rooted at a given directory. These functions permit access to Windows device files under that root. For example, os.DirFS("C:/tmp").Open("COM1") opens the COM1 device. Both os.DirFS and http.Dir only provide read-only filesystem access. In addition, on Windows, an os.DirFS for the directory (the root of the current drive) can permit a maliciously crafted path to escape from the drive and access any path on the system. With fix applied, the behavior of os.DirFS("") has changed. Previously, an empty root was treated equivalently to "/", so os.DirFS("").Open("tmp") would open the path "/tmp". This now returns an error.
golang.org/x/text:
CVE-2022-32149
An attacker may cause a denial of service by crafting an Accept-Language header which ParseAcceptLanguage will take significant time to parse.

I've enabled dependabot on the repository in order to automatically update dependencies and avoid these kind of warnings.


Update from template and update dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.183%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
395

Relevant Lines:
438

💛  - Coveralls


Don't check for aws keys when there is no datasource #minor

I only just noticed that if you run a command that does not require a datasource, like Truncate, without having AWS_ACCESS_KEY_ID and AWS_ACCESS_KEY_SECRET in your env the command wouldn't work but throw an error.
This PR fixes that minor by only checking for AWS credentials when there is a datasource

Awesome, thank you @ilyakava!  Would you mind adding a spec to cover this change?


consensus: mark envelopes non-exhaustive

Motivation
Ref #451

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

rebased to resolve conflict with #461


charts/openshift-metering: Update the annotations used in the Presto coordinator/worker statefulsets to use the secret template hashes.

This was left out after we made the switch from deployments to statefulsets.

@chancez I was under the impression I had already added those, but it appears that only the reporting-operator-presto-tls-secrets.yaml was included in the annotations. Updated.

/lgtm
/approve


DotNetCoreCLI@2 does not support configuration

Environment

Agent - Hosted:

Issue Description
When using DotNetCoreCLI@2 parameter configuration is ignored and not added to dotnet.exe call

Cannot reproduce this, how did you configure your task?

@akaFalsh  did you try out the suggestion from @JohannesDrexler . This should help you out.

@JohannesDrexler  this works for me either. However I was wondering why configuration parameter does not work.

can you show us some build logs and how you configured your task ?

@akaFalsh can you provide @JohannesDrexler with the logs for further investigations ?

@akaFalsh we are closing the issue now, if more help is needed, feel free to reopen or open a new issue.

I just ran up against this as well and using arguments solved it. The documentation here is not near as complete as what's in this project's readme files so it's probably leading to a lot of the confusion.
Having the configuration as an input for build and test commands would be a nice to have to keep things consistent across all 3 tasks.

Thanks @Kaneraz for the feedback. Adding @pulkitaggarwl to consider the feedback in the next version of the task. @pulkitaggarwl - Can you help in fixing the documentation referred above in the meanwhile?

Hopefully this is on it's way to being fixed. I'm not sure what the release timelines are for the docs, but I've had a PR accepted for it: https://github.com/MicrosoftDocs/vsts-docs/pull/3582

The PR appears to introduce a difference between configurationToPack and configuration.
It does not address the problem that configuration does not work in devops with dotnet core 3.1.
I would like to use:
configuration: $(BuildConfiguration)
Now I have to use:
arguments: '--configuration $(BuildConfiguration)'


Interfaces rules

I have interface Storage.

What should WPCS interfaces be called?

IStorage
I_Storage
InterfaceStorage
Interface_Storage
StorageInterface
Storage_Interface.

What should be called file names for interfaces?

interface-storage

We don't have an agreement about this atm.
Related: #1327, #1859

Closing as a duplicate of #1859. Please continue the discussion there.


key points detection

can apply key points detection?

you can do it with retinaface: https://github.com/serengil/retinaface
deepface uses retinaface in the background and uses its landmarks to align faces.

https://github.com/serengil/deepface/issues/41


Touchpad issue

Hi, i've got a problem with touchpad. It works really slowly, it looks like it has delays.
Tried to modify it, for example by changing coordinate type to float from int but it didn't help.
I want the cursor to move smoothly.
Can you help me?
Regards

Touchpad should work smoothly. You might have some other issues-

Are you connected to a very slow network? You can try directly connecting your lappy to android via phone hotspot.
Check on android-studio logcat if you are getting some exceptions.
You can try using a different device.

I was connecting to fast home Wi-Fi, also to mobile hotspot network and nothing changed.
There are no exceptions.
Running application on 2 phones with different android versions and still were delays. Don't have another PC, but this one is not old, so i don't think so it could be the problem.

Please note that touchpad will not be as smooth as real mouse touchpad. But practically it is good to go. I am not sure to what level of smoothness you are expecting or if you are really getting unacceptable delays which we are not facing. If you could optimize this, your Pull Request is welcomed.

I will try to fix it. But when i use other apps from Google Play, touchpad works there smoothly, so this is not device problem but some optimization.


Tests

Started in #32.

Expanded in #49, #54.

Regression because of missing tests for #76 in #95.

Another regression in https://github.com/sigwinhq/yassg/pull/103


Feature Request: Floating Action Button

Are there any plans for adding a floating action button?
I really like how easy it is to implement and all the cool feature, but I feel like it's missing a floating action button.
Thanks!

I'd suggest using https://github.com/mastermoo/react-native-action-button
It's well established so would make sense to use that than re-invent the wheel.


[SPARK-15911][SQL] Remove the additional Project to be consistent with SQL

What changes were proposed in this pull request?
Currently In DataFrameWriter's insertInto and ResolveRelations of Analyzer, we add additional Project to adjust column ordering. However, it should be using ordering not name for this resolution. This is how Hive does for dynamic partition.
How was this patch tested?
Existing tests.

Test build #60380 has started for PR 13631 at commit 5f4455a.

Test build #60380 has finished for PR 13631 at commit 5f4455a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60380/
Test FAILed.

@cloud-fan There is a test ("Detect table partitioning with correct partition order") in InsertIntoHiveTableSuite which is dedicated to test insertInto with this column re-ordering. What you think we should do about it? Remove it?

Test build #60403 has started for PR 13631 at commit c0500c2.

Test build #60403 has finished for PR 13631 at commit c0500c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/60403/
Test PASSed.

Test build #60481 has started for PR 13631 at commit f8c9ccf.

Test build #60481 has finished for PR 13631 at commit f8c9ccf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60481/
Test PASSed.

Merged build finished. Test PASSed.

Test build #60546 has started for PR 13631 at commit 3030144.

Test build #60546 has finished for PR 13631 at commit 3030144.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60546/
Test PASSed.

Merged build finished. Test PASSed.

@cloud-fan any other thoughts?

hi @viirya , we are auditing the insertion behaviour of spark sql, and will have an agreement this week. How about we revisit this PR  after that?

@cloud-fan no problem at all.

please track the process of https://issues.apache.org/jira/browse/SPARK-16032 and update your PR, thanks!

@cloud-fan Looks like a part of this PR is done by some PRs in that JIRA. I will update this.

@cloud-fan Updated. Please take a look. Thanks!

Test build #60795 has started for PR 13631 at commit be60027.

Test build #60795 has finished for PR 13631 at commit be60027.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/60795/
Test PASSed.


8 refactor for package

Description

Refactored repo for use as a package, and added a short set-up test to check this is done sucessfully:

.pre-commit-config.yaml updated pydocstyle to ignore D203 errors which is deprecated
CONTRIBUTING.md updated to show installation in editable mode, and pytest --runsetup.
conftest.py added to handle switching of setup tests
pyproject.toml updated to install src folder in editable mode
requirements.txt updated with pytest
tests/test_setup.pt added to check installation. marked to run only when --runsetup is added to pytest cli call
src/heimdall_transport added and refactored content in this dir for installation in editable mode

Fixes #8
Motivation and Context

Refactored the repo to all package installation in editable mode to made development easier. Also added a simple setup test, which pytest cli flag switching such that the setup tests are marked to only run when the --runsetup flag is added.
Type of change

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

How Has This Been Tested?

Pytest run (with --runsetup flag) and passes test as expected. Without flag, the test does not run - expected behaviour. src/heimdall_transport/utils/check_setup.py still runs as expected (see #7 and Additional Notes).
Test configuration details:

OS: macOS
Python version: 3.9.13
Java version: AdoptOpenJDK-11
Python management system: venv = conda, packages = pip

Advice for reviewer

Should now be able to install as a package in editable using pip install -e .. CONTRIBUTING.md updated to reflect the steps. Expected behaviour: Before installation, running pytest --runsetup should fail. After installation, is should fail work.
Checklist:

[X] My code follows the intended structure of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Additional comments

#7 can be added to test_setup.py to benefit from the --runsetup pytest cli flag.

Morning chaps. Is this being handled? I can see it's in the 'in review' list on the kanban, but nobody is assigned? Happy to pick up but just paused incase Serg was handling it.
@ethan-moss  @SergioRec

Morning @r-leyshon, @SergioRec! Yes, this one is ready for review but no-one is assigned just yet. Feel free to take a look. Thanks for your help and if I can clarify anything just let me know.


Discourse style forum pull request

Parameters below in the .env file must be provided to activate discourse style forum.
DISCOURSE_URL
DISCOURSE_API_KEY
DISCOURSE_ADMIN_USERNAME

Awesome, we are checking over a couple things before merging in today


[Feature]: Enable language selection from SignupView

Feature Summary
Now that it is possible to to select your preferred language, we should enable users to select a language before logging in.
Detailed Description
Currently users can select their preferred language from the UserMenu in TheHeader after they've logged in:

But new users who have not yet created an account can't select their language of choice until after signing up. We should improve this by adding an element to the top of the signup page for selecting your preferred language:

This could open the same language selection menu that's currently in use in TheUserMenu:

To do this, we should refactor the language selection functionality into a new TheLangSelectMenu component that is consumed by both TheUserMenu (as it currently works) and now newly applied to the top of the page directly in SignupView.
The result of this would be the same ability to select languages via TheHeader in TheUserMenu in the top left of the app after logging in, while additionally being able to select your language directly from SignupView

Looking into this atm


all previously uploaded global docs referenced instead of specific doc sources.

Describe the bug
After uploading multiple docs globally, when a query is sent in chat, instead of the specific docs that answer the question being references, ALL the global docs that have been uploaded are referenced.
To Reproduce
Steps to reproduce the behavior:

Start chat copilot via quickstart
Under documents tab, upload multiple different global docs.
Ask about information that should be found in only one or two of the docs.

Expected behavior
Only the docs that actually answer the question should be referenced.
Screenshots

Platform

OS: Windows 11
IDE: VS Code
Source: https://github.com/microsoft/chat-copilot/commit/5ba7ed01068a2ffa4894331e5a6bd9a4420bcc17

This is a side-effect of importing the same doc to the same index multiple times.  As such, you recieved multiple valid results.
The citation lists all results included in the prompt as there's no way of knowing which the model prefered.  Each result is a specific node or chunk of a document...so a given document might have several cititions.
One might assert that since the file name is the same the citations might be reconcile3d...but the document identifiers are different.  There's no enforcment to prevent a user from uploading different documents with the same name and i'm not sure we'd want that.
In short, there's a real risk of creating a bug farm by getting too fancy here.


Link to homepage repo from bottom of homepage. Remove everything else.

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/317
Originally created by @bencodezen on 2019-01-22 21:21:40

View 4 previous comments from the original issue on GitLab


Run Docker image with tmp/pids/docker.pid as pid file

I'm not too fond of the excessively long CMD, but I didn't see any way other than passing -P to set a pid file (I was hoping for RAILS_PID=tmp/pids/docker.pid or something).
It might be worth exploring whether having a smarter config.ru and changing the Dockerfile to have CMD ["bundle", "exec", "rackup"] could improve the situation, assuming this could be done in a way that wouldn't interfere with staging/production deployments of OpenFarm.

Coverage remained the same at 99.94% when pulling a2e79c9a61af7f359c02ad2f052fe67705ac2f66 on md5:docker-pid-file into 19e62e0ba589aa3037ac1011de4b98f90bdce289 on openfarmcc:master.


Add support for passing files and markdown directly to Show-Markdown

PR Summary
Previously you had to use ConvertFrom-Markdown and pipe that output to Show-Markdown making it cumbersome to use interactively.  Added -Path and -LiteralPath parameters as well as allowing markdown text to passed directly as -InputObject (or via pipeline) to Show-Markdown.
Internally, Show-Markdown uses ConvertFrom-Markdown to handle these new capabilities.  Removed the use of Out-Default which only passed the resulting vt100 text to the console and instead using WriteObject() so that the output can be redirected or captured by scripts or further in the pipeline.  The testhook is still needed when -UseBrowser is specified.
Note I had to fully qualify PowerShell and Path types due to conflict with Path parameter and PowerShell namespace.
Fix https://github.com/PowerShell/PowerShell/issues/7338
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

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

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[x] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

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

[ ] Add [feature] if the change is significant or affects feature tests

@daxian-dbw I think the ConvertFrom-Markdown issues you bring up should be a separate issue

@dantraMSFT CI is failing on macOS oslog tests.  I don't see any thing related to the changes I've made.  Can you take a look?


vertx generator should add the -cluster arg when a cluster.xml is given

Description
To cluster vertx applications (access the event bus), the fat jar needs to be launched with the -cluster argument. This could be provided by the vert.x generator if it finds a cluster.xml under src/main/.
On the other hand I cannot find a possibility to add the -cluster argument manually via config. Is this possible as workaround?
Info

f-m-p version : 3.4.2
Maven version (mvn -v) :

Apache Maven 3.3.9

Kubernetes / OpenShift setup and version : minishift 1.5
If it's a feature request, what is your use case : cluster vert.x application on openshift

It's a bit more tricky as the configuration file depends on the cluster manager used. What about looking at the vert.x maven plugin runArgs parameter to detect if cluster needs to be enabled?

Yeah. Realized it in the meantime that you don't always have a cluster.xml. But what if people want to use the maven shade plugin instead? Maybe we could have an optional property: fabric8.runArgs or sth. like that, too.

Fixed with #1035