@Authorize is not called if user is logged in?

Information

Version: 5.x
Packages: passport

I have used the @Authenticate and successfully authenticated and now I would like to link my Facebook, google or twitter accounts. My understanding is to use the @Authorize to get the account information back from the third provider. Once authenticated, I can not get information back from the other providers because the middleware is stopping the redirect?
https://github.com/TypedProject/tsed/blob/c6424c4a6c2199722d63223fa9798e5acef5fef4/packages/passport/src/middlewares/PassportMiddleware.ts#L8-L15
If I do not @Authenticate and use the @Authorize, I am redirected always to the callback and information is returned in req.account which is correct and I can use onVerify perfectly.
Acceptance criteria

[ ] If @Authenticate has been successful and req.user is set, endpoints that have @Authorize should redirect me to the callback.

I hope this makes sense @Romakita!

If I remove the:
     if (request.user && request.isAuthenticated()) {
       return;
     }

@authorize is working as it should but @authenticate is now not maintaining the user as you have probably guessed.
I was going to do a PR for this but I am just copying this file and having 2 middlewares - one with the if statement and one without. I feel there is probably a better way?
Many thanks,
Cal

Hello @virus2016
If you think it should be the right behavior, yes make a PR :). Your solution seems to be good for me ;)
Thanks @virus2016  ;)
Romain

Hello @virus2016
Have you the time to work on the PR?
See you :)

As I understand, you use @authenticate decorator then  @authorize to link connected user with a social profile.
The usecase work if you use a localStrategy and the user have already an account. But what about the usecase where the user is new? It must create his account then link his profile?
In my usecase, I use only authorize decorator because all authentication are delegated to an oidc server. So current middleware implementation works as expected.
I think we have to describe all possible usecase / usage the decorator to determine the right implementation :)
What do you think?

Hi @Romakita!
I haven't have time to work on it at the moment.
PassportJs doesn't really explain the authorize very well. The best description I can give it is:
Authorize allows users who are already logged in to access our third party providers while maintaining the original user account. On successful login of the third provider, req.account is populated with the third party profile information. If the user has already logged in with Authenticate then req.user is maintained even after the authorize callback has happened using req.user. ￼
￼￼
This works if I remove those lines as mentioned however, I'm not sure of the best implementation for tsed.
I felt that another middleware with the if logged in check removed would do it?
Hope this helps!


Add default fetch size support

Make sure that:

[x] You have read the contribution guidelines.
[x] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
[x] You use the code formatters provided here and have them applied to your changes. Don't submit any formatting related changes.
[x] You submit test cases (unit or integration tests) that back your changes.

Issue description
Fixes #256
New Public APIs

PostgresqlConnectionConfiguration.builder().fetchSize()
PostgresqlConnectionFactoryProvider.FETCH_SIZE

Additional context

Refactored some classes to pass and store PostgresqlConnectionConfiguration instead of duplicating the fetch size and forceBinary fields. Should make it more future proof to future additions
I probably need more tests, but not sure where or what I should add so some guidance would be good here.

Thanks for the review. I think I've addressed all your feedback. Please take a look again.

Thank you for your contribution. That's merged, polished, and backported now.


dev-master requires sorich87/bootstrap-tour dev-master -> no matching package found.

Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for mycademy/yii2-bootstrap-tour dev-master -> satisfiable by mycademy/yii2-bootstrap-tour[dev-master].
- mycademy/yii2-bootstrap-tour dev-master requires sorich87/bootstrap-tour dev-master -> no matching package found.
What do I have to do?

Hi Arjen,
can you put the following as workaround in your composer.json
"sorich87/bootstrap-tour": "dev-master",

Before or after the tour? Does it matter?

Not sure it matters, but just to be sure I would put it before.

Thanks


Added HateSpeach dataset

Please accept my PR

Good one @MayuriKolhe-2003
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

https://www.clueless.tech/profile/admin/100678575974637348384
Merged By @nikhil25803


Unfortunately, SocialX has stopped

I have just downloaded it via PS and when I launched, it says "Unfortunately, SocialX has stopped".
Android version: 4.4.4 KTU84P
Model number: MI 4LTE-CU
MIUI version: MIUI 7.0 | Stable <IP_ADDRESS> (KXDMICI)

Hey @Bakhid , thanks for opening this issue. Unfortunately we can't support very old Android versions like Android Kit-Kat. Please try to install it with a more recent phone btw. Android version.
Thanks


Make assemble() respect max_shots

Summary
Currently the assemble() function was not respecting the value of the
max_shots property from the backend. It takes in a shots argument which
could exceed the maximum number of allowed shots reported by the backend
and not do anything. This will result in an error when running on the
backend and is something that should be caught before we send a job to
the backend. This commit adds this check, if the backend has the
max_shots property set and the value of shots is greater than that it
will raise a RuntimeWarning and switch the number of shots to be
max_shots. This behavior was chosen because some backends (namely from
qiskit-aqt-provider) have a max_shots value less than the default of
1024.
Details and comments

We should also make assemble respect meas_level as well.


Fix tracing execution of parallel preloads in Ecto

This PR is based on conversations in following PR and a git commit
https://github.com/elixir-ecto/ecto/issues/2843
https://github.com/spandex-project/spandex_ecto/commit/a321d5f435fb2fb72c3ebde1b17104e2e611c9f1#diff-424a006e2a1dac6f34a878614aa2b57b7bd46e86221b5f0e07327c22dc135381
In NewPhoenixBackendWeb.UserController I have this function
  def index(conn, _params) do
    users = Accounts.list_users()

    users = users |> Repo.preload([[posts: :comments], :pins, :mobile_phones], in_parallel: true)

    render(conn, "index.json", users: users)
  end

This is with fix in the PR is applied.

This is without the fix in the PR applied.

As per Ecto source, Ecto does parallel preloads when there is 2 or more tables to preload
https://github.com/elixir-ecto/ecto/blob/master/lib/ecto/repo/preloader.ex#L124
Context
In this ecto commit Ecto removed adding caller_id to log_entry https://github.com/elixir-ecto/ecto/commit/8d09248c357d98e4096af449f6d0c20bc415fb92#diff-152ab0727a1d450c0fe74d178e80d1f0
that broke Spandex ecto functionality as issue created here by Spandex ecto author https://github.com/elixir-ecto/ecto/issues/2843
Later Elixir had added this $callers feature https://github.com/elixir-lang/elixir/pull/8510
Discussions about the issue
https://github.com/spandex-project/spandex_ecto/issues/12
https://github.com/spandex-project/spandex_ecto/pull/16
In above PR, author of Spandex Ecto mentions
I agree. This isn't really conventional, and it's possible, as far as I know, for the parent process to be dead while this log entry is happening, or for the parent span process to not be the most recent caller (think parallel preloads). I'd rather see an implementation that uses a registry of some kind, either in spandex core or in here.

can now this be closed?

Yes, this issue can be closed.
They didn't want Process.get(:"$callers")  based solution. If I remember correctly, authors of spandex ecto said, this would be solved in https://github.com/open-telemetry/opentelemetry-erlang (1.0 has released Jan 5, 2022) as solution required much cleaner design than what they have in spandex_ecto.
Related discussions https://github.com/spandex-project/spandex_ecto/commit/a321d5f435fb2fb72c3ebde1b17104e2e611c9f1#diff-424a006e2a1dac6f34a878614aa2b57b7bd46e86221b5f0e07327c22dc135381
https://github.com/spandex-project/spandex_ecto/issues/12 https://github.com/spandex-project/spandex_ecto/pull/16


Bundles Typeshed

Bundles Typeshed
Improves options for customising or disabling Typeshed

@MikhailArkhipov You'll need to update the VSC build to both include the entire Typeshed directory (so we get any modifications we need to make) and detect and provide the path in the initialization options. The analyzer deliberately doesn't pick it up automatically - it has to be specified as a type stub directory.

Note that #4460 is required for changes to these options to be noticed without reopening the project.

Ideally I would see us both (PTVS+VSC) use a submodule that points to our own fork of typeshed.
So that if we want to make patches to it and not wait for typeshed to accept the changes, we can do it in one place and have both products pick it up. Also both products would have more control on when we take updates, and we'll reduce user reports of IntelliSense working for some things in PTVS but not VSC or vice-versa when it just amounts to different typeshed.
Is adding all the files to this repo just a temporary thing until we set up our fork?

To be clear, my main objection is to PTVS and VSC not bundling the same thing.

I’m not even that keen to fork the repository, I’d rather just take a copy. We don’t need yet another repo floating around the Microsoft org for this.
I’m also opposed to submodules for the same reason. It’s in its own folder already, so it’s easy enough to copy changes back to a personal fork to send a PR if we need to. But I think we'll need to carry our own changes here, and since our model is richer than MyPy (we support simple code in these files) we may not merge back anyway.

VSC takes the latest but later may list specific commit to sync to for the submodule since typeshed releases are spotty. @brettcannon and @DonJayamanne can comment more.

You won't have the opportunity for any customization, or be able to pick up any changes to the files copied in this repo, if you submodule from typeshed repo directly.
Do you have any way of embedding the files from the ptvs repo, without making another copy that needs to be kept in sync?

That's the problem with submodules - you can't customize. I doubt we'll ever need to pin to an older release of typeshed (unless they start adding incompatible features to MyPy, which... okay... that seems pretty likely at some point), but we already want to replace parts of it with our own versions.
To do that we either need to put it in the language server repository (currently a directory in PTVS, but will be split out eventually) or in our own fork and then pull it into the language server repository. My vote is obviously to put it into the language server.
It should build and be part of the language server just as our other pregenerated files are (which we'll be keeping because they include docs, and typeshed does not).


Adding preconnect tags to speed up connections

I'm adding <link rel="preconnect"...> to some external dependencies (fonts.google.com and cdnjs.cloudflare.com) to speed up connections.
See more on web.dev

@dennistobar please raise an issue and add here your issue no.
otherwise, if you do only PR it wouldn't be tracked under Hacktoberfest.


button next could be on the right part

hey! great job guys! so as a user when I play specially with the phone I found confusing that the button "next" is on the left and not on the right. I think the logic there should be on the right to give a better user experience because is known that the eyes  are going to find the buttons, menus, etc on the right. I've actually clicked by accident the first time on the exit thinking that was the next. And also when you hold the phone I think is easier because you press the button with the fat finger.

Ok. It has been modified.


Content is removed when creating URL in chrome.

When aliasing an URL in chrome the alias is removed when you press space or enter. When starting to type again the first character of the word will have the href tag on it, but not the whole word.
It looks like this

Only the Y becomes clickable after we try to write the name again after it has been removed.
This behavior does not exists in Firefox, edge dev or edge.
I'm not sure if this is a mui-rte issue or a draftjs issue though.

@Destidom This issue was addressed in version 1.1.1. Can you confirm this is happening in the latest version (>= 1.1.1)?

Ah, yeah it is working in version 1.1.1. Sorry for the inconvenience and thanks for a swift reply.


⚠ Bofrost.fr has degraded performance

In e04f76d, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1452 ms

Resolved: Bofrost.fr performance has improved in f045445 after 6 minutes.


Page up/down with selection mode

If you have text selected and you press page up/down, everything in between the old cursor position and the new one becomes selected. Is this a bug or a feature?

Both :)
It's a cover/workaround for a bug.
Desired results

with text selected pressing page down should deselect the existing selection
with text selected pressing shift + page down should extend the selection a 'page' down

Problem: I haven't gotten shift + page up/down to work consistently.
Workaround: have page down act like shift + page down was intended to act like.
Having shift + page up/down work, would be great. I'm open to other workaround ideas if the desired behavior continues to be elusive.

It works on my Ubuntu 16.04 when I try shift + pageup/pagedown. I am using the standard ubuntu terminal. I think mine is GNOME Terminal 3.18.3.


Can someone help me fix Travis CI?

If you look at Travis you'll see that it no longer works, complaining about the command eval yarn no longer working. I haven't changed anything, it just decided to stop working and I'm not entirely sure why.
Does anyone know or can figure it out? Here's an example error:
https://travis-ci.org/photonstorm/phaser/builds/372712165

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Looks like travis is using node 10 and the upath module requires <=9:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=4 <=9".
error Found incompatible module

You should be able to set the version in the travis.yaml
https://docs.travis-ci.com/user/languages/javascript-with-nodejs/

Done. Fingers crossed.

It seems to work, no longer erroring on the upath module but on regular linting errors.

Awesome, thank you :) that's been bugging me for days.

Wahhh! And now it's erroring again, even though I'm specifying node 9 :(
At least it worked for a few hours. Bloody thing.

remove 'node' just 'keep the '9'
my bad :(
It's now doing two builds.
language: node_js
node_js:
  - '9'


Issue/venv fixes

Description
Fixes for problems introduced in previous PR
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Linking previous PR for bookkeeping reasons: #7956

Processing this pull request

Merged into branches master in 8c97eae6e5bd4b2e2f2098ada65f67881dbb257a

Processing #8083.


Improve Forth test

The original version of the test only checks for positive numbers.
if(/^\d*$/.test(functionName)) throw new Error("Invalid definition");
An implementation like above will work for positive numbers, but not for negative numbers.

/format

@tgujar is this something you could do for us <3? If not, that's okay, but please let us know so we can take action instead.

Oop I forgot about this, will get it done

Here's the PR in exercism/problem-specifications. I have added the UUID to .meta/tests.toml

(can you rebase?)

I am not quite sure how to proceed since the files were moved. Should I create a new commit?

Eh. Good question. This is what you can do!
git mv exercises/forth exercises/practice/forth

Commit that, THEN rebase. If you can't work it out, I will do it for ya, no problem!

Thank you for the suggestion, I did manage to rebase my branch. I however ended up affecting two other files when I did not intend to. Unfortunately I only realized this after I had already pushed to remote. I tried to recover from there but the only solution I found was to reset to a previous commit(hope that's alright)

Looks great @tgujar, than you a million.
If you wish, you can PR yourself into this list. I forgot to mention it before (it's super new) but I want you to be able to have the credit for improving this exercise.


question

@weidingerhp I'm sorry but I needed to recreate the repo. I made a comment about your contribution in procgov - hope it's fine with you.

yes, sure ... just wanted  to have the code in the codebase ...
Thanks for you library - it does a great job.


Open API

[ ] user can generate API token
[ ] APIs

[ ] GET|articles
[ ] GET|articles/:uuid
[ ] POST|articles

Third party can get/create articles for user using user's API Token.


helm chart 0.15.1 is not available in the chart repository

What steps did you take and what happened:
The latest helm chart version in the aquasecurity/helm-charts registry is still 0.15.0.
Steps to reproduce:

Add aquasecurity helm repository

$ helm repo add aquasecurity https://aquasecurity.github.io/helm-charts/
"aquasecurity" has been added to your repositories

Make sure repository is updated

$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "aquasecurity" chart repository
Update Complete. ⎈Happy Helming!⎈

Check latest version of trivy-operator helm chart in the repository

$ helm search repo aquasecurity/trivy-operator

NAME                            CHART VERSION   APP VERSION     DESCRIPTION
aquasecurity/trivy-operator     0.15.0          0.15.0          Keeps security report resources updated

What did you expect to happen:
GitHub Action Publish Helm chart #89 should have updated the helm repository index.yaml.
Anything else you would like to add:
It appears the helm chart 0.15.1 upload worked fine, however the Index helm chart task incorrectly indicated that the old index.yaml was already up to date:
Run ./cr index -o aquasecurity -r helm-charts -c https://aquasecurity.github.io/helm-charts/ -i index.yaml
Using existing index at index.yaml
Index index.yaml did not change

Environment:

Trivy-Operator version (use trivy-operator version): 0.15.1
Kubernetes version (use kubectl version): 1.21.14
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): Amazon Linux 2

Have the same issue with Postee helm chart
helm search repo aquasecurity/postee --version v2.13.0

No results found

While v2.13.0 was released a week ago. Looks common issue.

@nbarnum @rtalipov  thanks for reporting this issue. I have republished it again. should be ok now

Thank you for the fast response @chen-keinan, I've confirmed that I can now install the trivy-operator chart at v0.15.1


Support load data from *.safetensors

Fix #575

Hi @LaurentMazare , can you take a moment of time to this, I need this feature for debugging on some models.

I have already started working on it, some weeks ago. Here is the branch: https://github.com/A2va/tch-rs/tree/safetensors
I'm currently finishing the unit tests.
The main difference between your implementation and mine is that I use the v0.3.0 version of safetensors. This allows to the remove the unsafe part ( unsafe { std::slice::from_raw_parts(cpu_tensor.data_ptr() as *const u8, length) }).
Related: Discussion around the new trait api huggingface/safetensors#173

Close by #636


Add support for Ubuntu Zesty

provision.py has zesty in the unsupported platform list cause it doesnt support tsearch_extras.
I guess we should add it back to the supported platform list as it works well on my laptop (Ubuntu Zesty).

The blocker issue for this is build tsearch_extras for Zesty.

Closing as wontfix; we aren't supporting non-LTS releases (And Zesty is reached EOL anyway).


test(Dashboard): move app.json to inline, fixing failing jenkins system tests

Description:
In the test-dashboard.js system test, we created services by using an app.json file. The tests are broken on Jenkins because this file is not found. Since this file is small, move it inline to the cypress test file. This will fix the issue on Jenkins.
Closes DCOS-17383.
Testing:
Everyday I usually create an open cluster named "andrew-(random-id)-open". Feel free to use it for testing.
docker run -it --rm --ipc=host \
    -v `pwd`:/dcos-ui \
    -e CLUSTER_URL=$OPEN_CLUSTER \
    mesosphere/dcos-ui dcos-system-test-driver --only "dashboard" \
    ./system-tests/driver-config/dev.sh

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

run integration tests

run integration tests


Incompatibility with latest JAX version

Hi,
it appears that JAX has deprecated jax.experimental.host_callback as of March 21, 2024 (google/jax/issues/20385). I've implemented a working solution below for the TqdmWrapper class, located in solver.py. This resolved all issues for me.
class TqdmWrapper:

    def __init__(self, tqdm, reference_time, total, *args, **kwargs):
        self.reference_time = reference_time
        jax.experimental.io_callback(lambda total: self._create_tqdm(tqdm, total, *args, **kwargs), None, total)

    def _create_tqdm(self, tqdm, total, *args, **kwargs):
        self._tqdm = tqdm.tqdm(total=total, *args, **kwargs)

    def update_to(self, n):
        jax.experimental.io_callback(lambda n: self._tqdm.update(n - self._tqdm.n), None, n)

    def close(self):
        jax.experimental.io_callback(lambda _: self._tqdm.close(), None, None)

    def __enter__(self):
        return self

    def __exit__(self, exc_type, exc_value, traceback):
        self.close()`

Thanks for the report and apologies for the (significant) delay in fixing this! Your suggestion above was helpful; I found that I also needed to add the option ordered=True in #16 to get the progress bar to update correctly.


Config with only subscribers

Hi,
I have a config file with about 3k queues, bindings and subscriptions.
Every time the process starts I have a warmup time of about 15 minutes. The greatest time is for checking the queues and the bindings.
A little time is spent to subscribe the consumer to the queue.
Is it possible to have a config for another process with queues and bindings and a config for a process with only subscriptions to speed up the time for the consumers to listen the queues?
Thanks,
Nico.

Hi Nico,
Having two configs doesn't make much sense to me, they would both run in parallel, so you would risk subscribing to a queue that didn't exist.
Alternatively if you are confident the queue exists and are configured correctly, you can disable checking the queues / exchanges.
Another thing that might help is if there was an async option. Rascal this serially because it makes it far easier to work out where problem occurs, but I have no problem performing the checks asynchronously or even in batches.
If you're happy to have a go, I think it should be a case of changing this line to use async.eachLimit.
The default limit should be 1 for backwards compatibility. I'm not sure how I would specify a different value through config though, as the "queues" config object contains a set of queues, with each key being the queue name. It might have to go at the top level instead, so a little more thought required there.
Rinse and repeat for https://github.com/guidesmiths/rascal/blob/master/lib/amqp/tasks/assertExchanges.js#L6 and https://github.com/guidesmiths/rascal/blob/master/lib/amqp/tasks/applyBindings.js#L12

Unfortunately making the calls asynchronous doesn't help. The reason is that behind the scenes amqplib is waiting for a response before sending the next call.
With a creating 1,000 exchanges with a concurrency of 10 resulted in the following debug (I added a post success message locally)
rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e1 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e2 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e3 +1ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e4 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e5 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e6 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e7 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e8 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e9 +0ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e10 +0ms
  rascal:tasks:assertExchanges Successfully asserted exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e1 +3ms
  rascal:tasks:assertExchanges Asserting exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e11 +0ms
  rascal:tasks:assertExchanges Successfully asserted exchange: a0490c6c-3814-453d-b0bf-253cb28bf693:e2 +3ms

However using wireshark the Exchange.Declare and Exchange.Declare-Ok messages still come in sequence
No.	Time	Source	Destination	Protocol	Length	Info
41	0.298930	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e1
43	0.301350	::1	::1	AMQP	88	Exchange.Declare-Ok
45	0.302188	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e2
47	0.304333	::1	::1	AMQP	88	Exchange.Declare-Ok
49	0.305039	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e3
51	0.306679	::1	::1	AMQP	88	Exchange.Declare-Ok
53	0.307083	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e4
55	0.308682	::1	::1	AMQP	88	Exchange.Declare-Ok
57	0.309298	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e5
59	0.311408	::1	::1	AMQP	88	Exchange.Declare-Ok
61	0.311851	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e6
63	0.314042	::1	::1	AMQP	88	Exchange.Declare-Ok
65	0.314599	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e7
67	0.332144	::1	::1	AMQP	88	Exchange.Declare-Ok
69	0.332802	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e8
71	0.339870	::1	::1	AMQP	88	Exchange.Declare-Ok
73	0.340720	::1	::1	AMQP	141	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e9
75	0.343136	::1	::1	AMQP	88	Exchange.Declare-Ok
77	0.343977	::1	::1	AMQP	142	Exchange.Declare x=a0490c6c-3814-453d-b0bf-253cb28bf693:e10
79	0.346291	::1	::1	AMQP	88	Exchange.Declare-Ok

Therefore the best option I can think of is to set the assert attribute to false and remove the bindings. This will speed up your start-up significantly, but you will need another way of asserting new queues/exchanges and binding them.
The other option is to change amqplib to support the nowait option, but the change looks complicated, and as one of the maintainers, I don't think anyone will get to it quickly. Especially as it's a mostly useless feature - if you are going to the trouble of creating a queue, it is hard to imagine a scenario where you wouldn't want to wait until it had been successfully created.

I've thought of another way that might help. If I use multiple channels, then amqplib will be able to create the queues and exchanges concurrently. I ran a quick test, and with 10 channels the time to create 1,000 exchanges dropped from 1,794ms to 465ms. I'll continue to experiment.

I've just published<EMAIL_ADDRESS>You can control the concurrency / number of channels used during initialisation by setting the concurrency attribute of the vhost. e.g.
{
  "vhosts": {
    "/": {
      "concurrency": 10
    }
  }
}

I suspect it will still take some time to create thousands of queues, but should make a significant improvement

Removed some debug from v16.1.1

Tested with the same config (6k log rows of "rascal:tasks:applyBindings" with a test configuration file of 450 queues)
Now the warm up time is about 30 seconds!
Before (About 4+ minutes):

start: 2022-09-28T10:33:50.768Z
end: 2022-09-28T10:38:00.839Z

After (About 30 seconds):

start: 2022-09-29T07:08:56.234Z
end: 2022-09-29T07:09:25.387Z

Inside the config I have 5 vhosts with param "concurrency": 10

About 11 seconds with concurrency = 30

start: 2022-09-29T07:25:16.214Z
end: 2022-09-29T07:25:27.162Z

Great! Thanks for updating. OK to close?

Thank you too. I think this is a great improvement to Rascal, and wouldn't have crossed my mind if you hadn't taken the time to share it.


[MINOR] Fix the issue when handling conf hoodie.datasource.write.operation=bulk_insert in sql mode

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Currently we only accept hoodie.sql.bulk.insert.enable to control whether bulk_insert is enabled or not in spark sql mode, so during building buildHoodieInsertConfig(https://github.com/apache/hudi/blob/f30b3aef3e5e12e9119d6b5294c94a0282719e00/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/ProvidesHoodieConfig.scala#L97), we use this param to decide insert operation. But it could be overridden at last: https://github.com/apache/hudi/blob/f30b3aef3e5e12e9119d6b5294c94a0282719e00/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/ProvidesHoodieConfig.scala#L177
So if we set hoodie.datasource.write.operation=bulk_insert, enableBulkInset is still false, and this will set ENABLE_ROW_WRITER to false too.
https://github.com/apache/hudi/blob/f30b3aef3e5e12e9119d6b5294c94a0282719e00/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/ProvidesHoodieConfig.scala#L131
but method withSparkConf could override this, which change OPERATION.key back to bulk_insert
https://github.com/apache/hudi/blob/f30b3aef3e5e12e9119d6b5294c94a0282719e00/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala#L251
So as a result, the operation is still bulk insert, but doesn't use spark row writer.
Brief change log
Here we accept both hoodie.sql.bulk.insert.enable and hoodie.datasource.write.operation=bulk_insert to control whether we should allow bulk insert or not in sql mode .
Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure


update publishing/print_preview_toolbar_button to 4.x

Not sure if anybody uses this at all ?

Nor me!
On 8 Oct 2015 5:25 pm, "Juergen Hasch"<EMAIL_ADDRESS>wrote:

Not sure if anybody uses this at all ?
—
Reply to this email directly or view it on GitHub
https://github.com/ipython-contrib/IPython-notebook-extensions/pull/351#issuecomment-146603079
.

There is a menu for exactly this, so let's remove this extension.

Fair enough, I'll edit.

Although, this seems to have been written when the menu item already existed, so I guess someone just wanted a button. It might be better though, to have an extension which would just provide toolbar buttons for arbitrary actions.


Would love to see an update to the capabilities list here.

For example it seems like script access to OAUTH tokens is supported in YAML now: https://github.com/Microsoft/vsts-agent/blob/master/docs/preview/yamlgettingstarted-token.md

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

ID: 87686bea-7c46-cd47-d1a9-128e16b63a23
Version Independent ID: ace78317-e562-e35d-781d-23cd7d2dfff2
Content: CI Build in code using YAML
Content Source: docs/build-release/actions/build-yaml.md
Service: unspecified
GitHub Login: @douge
Microsoft Alias: alewis

Thank you for the feedback. We're working on an update.

We've added info about it to this topic and here: https://docs.microsoft.com/en-us/vsts/build-release/concepts/process/phases?view=vsts&tabs=yaml#access-to-oauth-token


creating json annotations for new images

Are there any scripts to create annotations in the json format for new images?

No, we don't have the script for it.


change ZED_LAKE_HOST to ZED_LAKE and -host to -lake

ZED_LAKE and -lake both take a Zed lake service URL.

Great!


fix bench test on 32bit architectures

Fixes:
./config.go:277:42: constant<PHONE_NUMBER>0 overflows int
./config.go:277:48: constant<PHONE_NUMBER>00 overflows int
./config.go:277:54: constant<PHONE_NUMBER>000 overflows int
while running internal/tool/bench test.

Hi. Thanks for the fix. Can you rebase onto the development branch?

Done: https://github.com/dsnet/compress/pull/64 .

In case you find this information helpful @rathann, it's possible to change the base branch of a PR, without having to make a new one. See https://blog.github.com/2016-08-15-change-the-base-branch-of-a-pull-request/.


How to display commands grouped by category when running using --help argument

First Check

[x] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Typer but to Click.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import typer

# Troubleshooting and Debugging Commands:
def describe(foo: str):
    print(f"{foo}")

def logs(foo: str):
    print(f"{foo}")

def attach(foo: str):
    print(f"{foo}")

def _exec(foo: str):
    print(f"{foo}")

def port_forward(foo: str):
    print(f"{foo}")

def proxy(foo: str):
    print(f"{foo}")

def cp(foo: str):
    print(f"{foo}")

def auth(foo: str):
    print(f"{foo}")

def debug(foo: str):
    print(f"{foo}")

# Advanced Commands:
def diff(foo: str):
    print(f"{foo}")

def _apply(foo: str):
    print(f"{foo}")

def patch(foo: str):
    print(f"{foo}")

def replace(foo: str):
    print(f"{foo}")

def wait(foo: str):
    print(f"{foo}")

def kustomize(foo: str):
    print(f"{foo}")

# Settings Commands:
def label(foo: str):
    print(f"{foo}")

def annotate(foo: str):
    print(f"{foo}")

def completion(foo: str):
    print(f"{foo}")

def main():
    # console = Console()
    app = typer.Typer()

    # Troubleshooting and Debugging Commands
    app.command(short_help="Show details of a specific resource or group of resources")(
        describe
    )
    app.command(short_help="Print the logs for a container in a pod")(logs)
    app.command(short_help="Attach to a running container")(attach)
    app.command("exec", short_help="Execute a command in a container")(_exec)
    app.command(short_help="Forward one or more local ports to a pod")(port_forward)
    app.command(short_help="Run a proxy to the Kubernetes API server")(proxy)
    app.command(short_help="Copy files and directories to and from containers")(cp)
    app.command(short_help="Inspect authorization")(auth)
    app.command(
        short_help="Create debugging sessions for troubleshooting workloads and nodes"
    )(debug)

    # Advanced
    app.command(short_help="Diff the live version against a would-be applied version")(
        diff
    )
    app.command("apply", short_help="Apply a configuration to a resource by file name or stdin")(
        _apply
    )
    app.command(short_help="Update fields of a resource")(patch)
    app.command(short_help="Replace a resource by file name or stdin")(replace)
    app.command(
        short_help="Experimental: Wait for a specific condition on one or many resources"
    )(wait)
    app.command(short_help="Build a kustomization target from a directory or URL")(
        kustomize
    )

    # Settings
    app.command(short_help="Update the labels on a resource")(label)
    app.command(short_help="Update the annotations on a resource")(annotate)
    app.command(
        short_help="Output shell completion code for the specified shell (bash, zsh, fish, or powershell"
    )(completion)
    app()

if __name__ == "__main__":
    main()

Description
Hi, first of all let me tell you I've only been using typer for 20min and I love it, this is a really modern, powerful and very intuitive python package, so congrats about that :)
I've got a question:

I'm trying to figure out how to achieve a similar behaviour to kubectl --help cli in a way that if run python my_typer_cli.py --help I'd be able to see all my commands grouped by categories. How can I achieve a similar behaviour using typer?

Pasting the kubectl output:
$ kubectl --help
kubectl controls the Kubernetes cluster manager.

 Find more information at: https://kubernetes.io/docs/reference/kubectl/

Basic Commands (Beginner):
  create          Create a resource from a file or from stdin
  expose          Take a replication controller, service, deployment or pod and expose it as a new
Kubernetes service
  run             Run a particular image on the cluster
  set             Set specific features on objects

Basic Commands (Intermediate):
  explain         Get documentation for a resource
  get             Display one or many resources
  edit            Edit a resource on the server
  delete          Delete resources by file names, stdin, resources and names, or by resources and
label selector

Deploy Commands:
  rollout         Manage the rollout of a resource
  scale           Set a new size for a deployment, replica set, or replication controller
  autoscale       Auto-scale a deployment, replica set, stateful set, or replication controller

Cluster Management Commands:
  certificate     Modify certificate resources.
  cluster-info    Display cluster information
  top             Display resource (CPU/memory) usage
  cordon          Mark node as unschedulable
  uncordon        Mark node as schedulable
  drain           Drain node in preparation for maintenance
  taint           Update the taints on one or more nodes

Troubleshooting and Debugging Commands:
  describe        Show details of a specific resource or group of resources
  logs            Print the logs for a container in a pod
  attach          Attach to a running container
  exec            Execute a command in a container
  port-forward    Forward one or more local ports to a pod
  proxy           Run a proxy to the Kubernetes API server
  cp              Copy files and directories to and from containers
  auth            Inspect authorization
  debug           Create debugging sessions for troubleshooting workloads and nodes

Advanced Commands:
  diff            Diff the live version against a would-be applied version
  apply           Apply a configuration to a resource by file name or stdin
  patch           Update fields of a resource
  replace         Replace a resource by file name or stdin
  wait            Experimental: Wait for a specific condition on one or many resources
  kustomize       Build a kustomization target from a directory or URL.

Settings Commands:
  label           Update the labels on a resource
  annotate        Update the annotations on a resource
  completion      Output shell completion code for the specified shell (bash, zsh, fish, or
powershell)

Other Commands:
  alpha           Commands for features in alpha
  api-resources   Print the supported API resources on the server
  api-versions    Print the supported API versions on the server, in the form of "group/version"
  config          Modify kubeconfig files
  plugin          Provides utilities for interacting with plugins
  version         Print the client and server version information

Usage:
  kubectl [flags] [options]

Use "kubectl <command> --help" for more information about a given command.
Use "kubectl options" for a list of global command-line options (applies to all commands).

Operating System
Windows
Operating System Details
Windows 10 pro
Typer Version
0.7.0
Python Version
Python 3.10.1
Additional Context
No response

If you have enabled rich support in Typer, you can do this by adding a rich rich_help_panel parameter to your app.command calls. See the example in the link

I think this "issue" can be closed, the answer is straightforward (as per the link to the manual shared above).


vitest: fix several tests

This PR for #662:

Updates the password queries for sign in page to select correct elements
updates PersonalProfile tests to correctly select navigation and tab elements
Makes the tests in CivicProfile and PersonalProfile todo instead of commenting out entire file

Resolves the following tests:

test/components/CivicProfileForms/FinancialInfo.test.jsx
test/components/Signup/PodRegistrationForm.test.jsx
test/pages/CivicProfile.test.jsx
test/pages/PersonalProfile.test.jsx
test/pages/Signup.test.jsx
test/components/Contacts/ContactListTableDesktop.test.jsx
test/components/Contacts/ContactListTableMobile.test.jsx

Future Steps/PRs Needed to Finish This Work (optional):
I believe this fixes all the tests.

😈😈😈😈😈😈😈😈😈😈😈😈😈😈😈

We don't have a component called PersonalProfile so PersonalProfile.test.jsx can probably be renamed Profile.test.jsx
I can look closer tomorrow. I haven't compared the our different tests for Signup yet

The unit test for PersonalProfile.test.jsx does not test the Profile component, but the CivicProfile component (see what the test is trying to render, there no imports for the Profile component).
We should merge the tests in there as part of CivicProfile.test.jsx.

@AJSterner Any updates?

@andycwilliams @leekahung consolidated the CivicProfile tests that were in the PersonalProfile.test.jsx profile into CivicProfile.test.jsx.


do not overrun when quotes are unescaped

When using double_quotes_escape, an unescaped quote can lead to an overrun if it appears at the end of the string.
For example, with the following code(test.cpp):
#include <iostream>
#include "csv.h"

int main(){
  io::CSVReader<3, io::trim_chars<' ', '\t'>, io::double_quote_escape<',', '\"'>> in("test.csv");
  in.read_header(io::ignore_extra_column, "vendor", "size", "speed");
  std::string vendor; int size; double speed;
  while(in.read_row(vendor, size, speed)){
    std::cout << "Vendor: " << vendor
              << ", size: " << size
              <<", speed: " << speed
              << std::endl;
    // do stuff with the data
  }
}

And the following csv file(test.csv):
"vendor","size","speed"
"escaped ""quotes""","1","2"
"unescaped "quotes"","1","2"

I get:
$ clang++ -std=c++11 test.cpp -lpthread -o test && ./test
Vendor: escaped "quotes", size: 1, speed: 2
Segmentation fault (core dumped)

With the proposed fix:
$ clang++ -std=c++11 test.cpp -lpthread -o test && ./test
Vendor: escaped "quotes", size: 1, speed: 2
Vendor: unescaped "quotes", size: 1, speed: 2

Hi,
thanks for the bug report.
I agree that the library should SegFault. However, is
"unescaped "quotes"","1","2"
not illformed and an exception should be thrown?
Best Regards
Ben Strasser
On 11/16/2016 04:19 PM, Yuval Hager wrote:

When using double_quotes_escape, an unescaped quote can lead to an overrun if it appears at the end of the string.
For example, with the following code(test.cpp):
|#include  #include "csv.h" int main(){ io::CSVReader<3, io::trim_chars<' ', '\t'>, io::double_quote_escape<',', '"'>> in("test.csv"); in.read_header(io::ignore_extra_column, "vendor", "size", "speed"); std::string vendor; int size; double speed; while(in.read_row(vendor, size, speed)){ std::cout << "Vendor: " << vendor << ", size: " << size <<", speed: " << speed << std::endl; // do stuff with the data } } |
And the following csv file(test.csv):
|"vendor","size","speed" "escaped ""quotes""","1","2" "unescaped "quotes"","1","2" |
I get:
|$ clang++ -std=c++11 test.cpp -lpthread -o test && ./test Vendor: escaped "quotes", size: 1, speed: 2 Segmentation fault (core dumped) |
With the proposed fix:
|$ clang++ -std=c++11 test.cpp -lpthread -o test && ./test Vendor: escaped "quotes", size: 1, speed: 2 Vendor: unescaped "quotes", size: 1, speed: 2 |

    You can view, comment on, or merge this pull request online at:

https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32
    Commit Summary

do not overrun when quotes are unescaped
File Changes

M csv.h https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32/files#diff-0 (2)
Patch Links:

https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32.patch

https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32, or mute the thread https://github.com/notifications/unsubscribe-auth/ALaAj6Cnfv6tMG8LycGLLmg4NFWWIuzqks5q-x8KgaJpZM4Kz_NK.

Possibly. But the fix was so easy, so might as well let it through (unless you see a problem with the code)

Hi,
the problem is that if I accept the patch as is now and change the behavior later to throwing an exception, then the later change will be a breaking change. :/
However, as I already said, a seg fault is not acceptable. Sometime has to change. I'm just not sure what yet.
Best Regards
Ben
On 11/17/2016 01:45 PM, Yuval Hager wrote:

Possibly. But the fix was so easy, so might as well let it through (unless you see a problem with the code)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ben-strasser/fast-cpp-csv-parser/pull/32#issuecomment-261237874, or mute the thread https://github.com/notifications/unsubscribe-auth/ALaAjzMUiuq4C_eaDPQqxXtVJdUIDPXSks5q_Ex3gaJpZM4Kz_NK.

That's correct. Let me know if you want me to try and modify the patch so that it throws an exception instead. For my case, I need to keep parsing other lines, even if the current line is ill formed.

Hi,
I merged the patch.
The reason is that currently strings such as """ is read as " even though the quotation mark is not escaped. One can argue that this should throw an exception. However, it currently does not and throwing an exception would therefore potentially break existing code. Throwing an exception is therefore of the table and your patch is the only remaining solution that I can see.
Best Regards & Thanks for the patch
Ben Strasser


adding an -o & an "%(title)s - %(id)s.%(ext)s" argument failes

if i add extra arguments to name the outputfile as i like it to become, somehow the naming argument get f*d over:
i tried adding the following commands through the config.htm-screen and directly in the config.yml file
- -o
- "%(title)s - %(id)s.%(ext)s"
this has to be capsulated in double quotes as i'm on Windows (and i'm sure Linux will go broke on it as well) as else the space behind %(title)s will cause '-' to be seen as a next commandline option which will make the full youtube-dl.exe ... command faulty
however  "%(title)s - %(id)s.%(ext)s" becomes "\"%(title)s - %(id)s.%(ext)s\"" if i go through the config-screen
and for the next downloads with that profile the file(s) get a starting and trailing # around it.
imho that somehow comes from the \ being added.
if i save it directly into the config.yml as such the double quotes get stripped on reading the file in config.go and that makes the full commandline fail for the 1st stated reason (no double quotes = spaces cause the '-' to be seen as next commandline option).
if i try to use single quotes around it making it: '"%(title)s - %(id)s.%(ext)s"'  (singlequote, double quote ... double, single) it again excapes the double quotes on reading the configfile, resulting again in the added #'s at the start and finish of the full filename(s) e.g.:  #name - id.mp4#
if i try the other way around: "'%(title)s - %(id)s.%(ext)s'"  (double quotes on the outside, single qoutes on the inside), it keeps the single quotes and strips the qouble quotes, but the resulting filename also has those single quotes at start and finish
e.g.: 'name - id.mp4'
The command i hope to get (with the default and other args) is
youtube-dl.exe -o "%(title)s - %(id)s.%(ext)s" <url>
which is according to the youtube-dl readme a correct optionline
with a resulting filename: name - id.mp4 without any # or ' around that.
anyone having an idea how to tackle this?

To give you a better idea of what the correct, single quoted and excaped double quoted version result in i copy-pasted all 3 and the first 7-ish resulting outputlines in the following pastebin:
https://pastebin.com/pdjttyMv

Hi @soulmaestro, gropple does not invoke the shell, so shell style escaping / quoting is unnecessary. Spaces inside arguments should be ok and the double quotes should be unnecessary. Give that a try and let me know if it works.

well damned :o
The one option that i never tried - no shell style escaping/quoting - is working 👍
the cmd.Start() - and especially the 'cmd'-part - made me think escaping/quoting was necessary.
(windows cmd vs Golang cmd variable name. )
tyvm for the quick response and preventing some of my hair to turn grey combating this what looks like a non-excisting problem/issue :)

No problem :-) I will open a separate issue to improve the documentation, because I think this is probably surprising behaviour for many people.


DeleteTR - Delete Trust Relation endpoint

Use Case
Success

❌ Receive DeleteTR request
❌ Validate external request data match request schema (has all needed required fields)
❌ Save TR
❌ Respond with success

Exceptions

❌ throw InvalidRequestError if external request data validation fails
❌ throw PersistenceError if fails to delete TR

Tasks

[ ] Create Controller (handle request, throws InvalidRequestError)
[ ] Create Interactor (service) (execute action)
[ ] Create repository (persistence, throws PersistenceError)
[ ] Create Façade (sync)
[ ] Create server endpoint

CRUD operations are done directly to oxtrust/config apis that handle persistence. Closing.


[WIP] Handled instrumentation configuration later at the end of block ...

... for return value tracing and line by line tracing

Pull Request Test Coverage Report for Build 663

0 of 0   changed or added relevant lines in 0 files are covered.
123 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 66.158%

Files with Coverage Reduction
New Missed Lines
%

dist/plugins/config/TraceableConfig.js
7
78.43%

dist/opentracing/instrument/Instrumenter.js
116
7.68%

Totals

Change from base Build 655:
-0.1%

Covered Lines:
2322

Relevant Lines:
3242

💛  - Coveralls


README.md Sponser image is missing

It seems Sponsor image from README.md is missing.

Fixed by #13.


Option to keep delimiter values in model output

So currently the delimiters seem to get stripped out of the model output.
https://jsfiddle.net/v3aLq6x6/1/
Is it possible to keep them in the model output? Or ideally have an option to set them to be retained?

https://jsfiddle.net/nosir/5ysgqrpc/
I guess this is what you need?
Will close this, feel free to open if you still have problem.


Problem with Unique Key with Oracle

I've created a table on Oracle with Migration and with a UNIQUE KEY.
When trying to generating the model via Gii, I am receiving an 'UNDEFINED INDEX_NAME' error.
I opened the file \db\oci\Schema.php and, at line 540 I realized that it is not considering the ATTR_CASE property.
In my case, I am using CASE_LOWER, which explains why the query is not finding the INDEX_NAME key.

@rickgrana would you like to fix it and submit a pull request?


Excluding regular attributes and nested class attributes

Is there a way to exclude an attribute from the docs for a class?
I have a class with a class attribute. The documentation for the class attribute is showing up nested, and I'd like it not to.
The code looks something like this:

class A:
    x = SomeClass

    def __init__(self, ...):
        """
        Arguments: ...
        """

class SomeClass:
    """
    The docs for this class appear in the docs for class A
    """

I believe you're using the legacy Python handler. This issue has been reported here: https://github.com/mkdocstrings/pytkdocs/issues/141. Could you confirm that it describes your own issue, so I can close as duplicate?

Yes that looks like the same issue


Update Bluemix CLI to IBM Cloud CLI

Update site references from Bluemix CLI to IBM Cloud CLI.

I can work on this issue, but I'm not able to change assignees.

@filipecorrea assigned it to you. Thanks.

Thanks, @mrshah-at-ibm.
I wasn't able to create a branch for this issue, but I opened the merge request:
https://github.com/IBM-Blockchain/ibm-blockchain.github.io/pull/78

The website has been archived; closing.


Added emote Sensing data classification example

Checked docs_build without 'plot_'

Codecov Report

Merging #13 into 0.2.X will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            0.2.X      #13   +/-   ##
=======================================
  Coverage   98.83%   98.83%
=======================================
  Files           4        4
  Lines         171      171
=======================================
  Hits          169      169
  Misses          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update abf41c3...82f93bc. Read the comment docs.


ajaxSniffer.off does not work on after callbacks

The stored reference is not the one of the callback.

cc @samaingw


runtime/race: detect when program synchronizes with time.Sleep

There is a common pattern of "synchronization":

var result ...
go func() {
  ...
  result = ...
}()
time.Sleep(time.Second)  // 1 second should be enough for everyone
if bad(result) {
  complain()
}

C++ ThreadSanitizer prints a useful "As if synchronized via sleep" message
with a stack trace of the sleep:
http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/tsan/lit_tests/sleep_sync.cc?view=markup

It would be useful to do the same for Go.

This is a very old proactive improvement request, nobody ever asked for it and there are no active plans to implement this, so closing.


Fix broken EJScreen Link

Our Cypress tests found a dead link to EJScreen's technical documentation (404). This PR fixes that dead link (in multiple places) and updates the associated test snapshots.

Our Cypress tests found a dead link to EJScreen's technical documentation (404). This PR fixes that dead link (in multiple places) and updates the associated test snapshots.

Nice catch! A good reminder to run the cypress test locally when we make any FE changes. :)

I'm just doing the things you told me to do in the FE instructions! They're good! 🚀


Self sign up: When account confirmation is expired a blank page appears

Tried in:
Google Chrome Version 63.0.3239.108
Steps

Self registered for an account
Let the account confirmation link be expired

Clicked the link and the following page loaded. Proper error message should be given.

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


AsyncMessage now implements the Message interface

AsyncMessage should extend the Message interface as it's API is expected inside of AsyncSwitchMessageRouter.
We're upgrading our system to use async messaging but we don't use the DomainMessage but we did implement AsyncMessage and the system fell over because we were unaware we had to implement this API.

Coverage decreased (-0.08%) to 98.054% when pulling d6b59f1fa9771f4349e512de90b58df2c15bdf18 on iainmckay:master into 5062478d5e24e958e7c13fd4fd46eb3d23bc0060 on prooph:master.


Footer icons

Fixes #131

Hover color + Hover text (added)

@AkhilaSunesh
why are there too many files under assets/font-6 if only one file is linked?

Fixed


ci: Cleanup ci.yml

CI Vulkan-ExtensionLayer build queued with queue ID 32264.

CI Vulkan-ExtensionLayer build # 693 running.


.

1- Python 3.9 support: Added to the list of versions searched for in the find_python function.
2- Python installation on macOS: Python 3.9 is included as an option to install using Homebrew if it is not present.
3 - brew faiss

why installing both python version

I've changed it now if you have 3.9 the code will look in the system for Python 3.9 and install brew of that version and if you have 3.10 it will install brew of 3.10 that way the user doesn't have to download unnecessary stuff

we don't support anymore python 3.9

I don't know if you noticed but there is a little bug in 3.10 for Mac you can't make inferences and the support channel was reading that some users downgraded to 3.9 and guess what, the problem is solved, just saying

gradio 5 needs python 3.10

Let me change a few things in the code, because Applio on Mac doesn't support the inference section for now.

better


[CARBONDATA-3400] Support IndexSever for Spark-Shell in secure Mode(kerberos)

Problem
In spark-shell OR Spark-Submit mode, Application user  and  IndexServer User are different .
Application user is based on Kinit user OR based on spark.yarn.principle user whereas Indexserver user is based on spark.carbon.indexserver.principal . it is possible that both are different as Indexserver should have it's own authentication principle and should not depend on Application principle so that any application's Query(Thrifserver,Spark-shell,Spark-sql,Spark-Submit)  can be served from IndexServer.
Solution
Authenticate the IndexServer by it's own principle and keytab.
keytab is required so that long run application (client and indexserver ) does not impacted on token expire.
Note:- Spark-default.conf of Thriftserver (beeline), spark-submit ,spark-sql  should have both spark.carbon.indexserver.principal and spark.carbon.indexserver.keytab.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?
NA
[ ] Any backward compatibility impacted?
NA
[ ] Document update required?
NA
[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
Manually tested
[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3340/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11607/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3541/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3364/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3567/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11631/

LGTM


Chore: Added wallet_connector_id to wallet_connected event context

What changed? Why?

added a property to the wallet_connected event context to track wallet connector id for additional granularity

Notes to reviewers
How has it been tested?
locally

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Segfault on target process when attaching for the second time under docker

A segfault happens when trying to attach for the second time to another dotnet process in docker.
This could be either an issue on the profiler implementation side or a nasty bug in the dotnet runtime itself.
In order to invalidate an issue on the runtime itself, we need to reproduce with some simpler code (without the rust layer).
The CLR tests include some small integration tests for the profiling API. Those could be tweaked to run not once but twice per test, and eventually run them in docker.
Building the CLR under MacOS ARM64
brew bundle --no-lock --file eng/Brewfile
sudo ./build.sh --subset clr
sudo ./build.sh --subset libs
sudo ./src/tests/build.sh -test:profiler/gc/gc.csproj /p:LibrariesConfiguration=Debug
sudo dotnet run --project gc.csproj -c Debug

See issue https://github.com/dotnet/runtime/issues/80683


[BUG] Duplicate Schemas screws up the +Populate page on the website

If I copy the schema to multiple apps, the autogenerated website will only display a code snippet for one of the tables.
It will not auto-generate the population script for all the tables with duplicated schemas.

Ough, sorry. I somehow just see this issue now. Can you elaborate more, please, so I can try to reproduce the issue?
I take it you use the word schema for children of Django models / EspressoDB Base models? You create a new app inside the same project and copy these models from one app into another? Did you change the names of the copied models? And did you add the new app to the installed apps?


[IMPAC-654] IE: Javascript errors on widget settings and display

@cesar-tonnoir
-Fixed some js method errors in IE with lodash (there are inconsistencies with this throughout the project that should be fixed at some point)
-Allow settings changes in expense to turnover widget
-Fixed salaries summary and workforce summary widgets
-Made input range bars (globally) visible in IE

@cesar-tonnoir Hey fixed all but the Range input... Should work everywhere. Need to make sure I've covered all the widgets it is used in.. otherwise other fixes are pushed


⚠ Bofrost.lu has degraded performance

In 9ac58d5, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1079 ms

Resolved: Bofrost.lu performance has improved in ba6244c.


Fix renaming the old_relation in Oracle 12c+ when using DBT table materialization with missing database name in profiles.yml

Hi,
first of all, thank you very much for releasing an official DBT Oracle DB adapter/profile with an up-to-date DBT version!!! 😍
I tried this adapter today locally together with some Airbyte modifications and found that version 1.0.0 does not rename the already existing destination table to a backup table when using DBT table materialization and therefore I get the following error:
1 of 1 ERROR creating table model staging.test................................................................. [ERROR in 0.96s]
2022-05-04 08:41:55 normalization > 08:41:55  oracle adapter: Oracle error: ORA-00955: name is already used by an existing object

With dbt run --debug
Database Error in model test (models/generated/airbyte_tables/staging/test.sql)
2022-05-04 08:42:00 normalization > 08:41:55.566833 [error] [MainThread]:   ORA-00955: name is already used by an existing object
2022-05-04 08:42:00 normalization > 08:41:55.567157 [error] [MainThread]:   compiled SQL at ../build/run/airbyte_utils/models/generated/airbyte_tables/staging/test.sql

This is caused by the fact that the database name is passed to function adapter.get_relation() which should not be the case for Oracle DB.
https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/include/oracle/macros/materializations/table/table.sql#L22
In Oracle DB the database name is never a prefix when making a select as it does not even work.
✅ schema.relation
❌ database.schema.relation
Also at the DBT/Jinija macro oracle__list_relations_without_caching the filter for the table_catalog (database) should be removed as usually no one ever will pass the database name in the profiles.yml.
Fixes #4
Fixes techindicium/dbt-oracle#8
Fixes techindicium/dbt-oracle#32

Thank you @ThoSap for testing and contributing.
While we review to get this merged, could you sign the Oracle Contributor Agreement (OCA) ?
The bottom of your commit message must have the following line using your name
and e-mail address as it appears in the OCA Signatories list.
Signed-off-by: Your Name<EMAIL_ADDRESS>
This can be automatically added to pull requests by committing with:
git commit --signoff

I just wanted to create an Individual OCA but the oracle/dbt-oracle repo is not available for selection and it is a required field.
Can you help @aosingh ?

Sure, let me check and I will get back to you

Hi @ThoSap
Thanks once again for your help!
I would like to better understand the problem and the fixes. The reason is, I am trying to reproduce the issue but it works as expected with the current version of dbt-oracle.
Test Model - https://github.com/oracle/dbt-oracle/blob/main/dbt_adbs_test_project/models/promotion_costs.sql
Oracle  Database version - 19c
I ran the model 3 times and each time it completed successfully. With --debug flag enabled i could also see the backup table being dropped using the below PL/SQL.
DECLARE
     dne_942    EXCEPTION;
     PRAGMA EXCEPTION_INIT(dne_942, -942);
     attempted_ddl_on_in_use_GTT EXCEPTION;
     pragma EXCEPTION_INIT(attempted_ddl_on_in_use_GTT, -14452);
  BEGIN
     SAVEPOINT start_transaction;
     EXECUTE IMMEDIATE 'DROP table dbt_test.promotion_costs__dbt_backup cascade constraint';
     COMMIT;
  EXCEPTION
     WHEN attempted_ddl_on_in_use_GTT THEN
        NULL; -- if it its a global temporary table, leave it alone.
     WHEN dne_942 THEN
        NULL;
  END;

The thing with this change is that it would no longer be necessary to specify the database name (in profiles.yml) which normally is never needed in any other application using OJDBC or cx_Oracle when connecting to an Oracle DB.
The database name (select SYS_CONTEXT('userenv', 'DB_NAME') from dual;) is not required for OJDBC, cx_Oracle and not even in SQL Developer, but only the service name or SID instead.
I don't think the database name is needed at all as the service name should suffice with these PR changes, maybe we can eliminate this property altogether.
But it could be that I don't get the technical reason why the database name is really needed in DBT Oracle 🤔
This probably has historic reasons and also there is the confusion, where if one specifies the profiles.yml property dbname but not database, the property dbname with alias database gets to be the service name and then the connection works anyways.
https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/adapters/oracle/connections.py#L129-L134
Example profiles.yml:
config:
  partial_parse: true
  printer_width: 120
  send_anonymous_usage_stats: false
  use_colors: true
dbt_test:
  target: prod
  outputs:
    prod:
      type: oracle
      protocol: tcp
      host: yourhost.example.com
      port: 1521
      schema: test
      user: test
      pass: SECRET_PW
      # Connection wors anyways as in connections.py#L83 the dbname property gets alias database and if the service property is not specified the database internally is the service in the end
      # https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/adapters/oracle/connections.py#L134
      dbname: test.service.name.example.com
      threads: 4

For my use case using Airbyte (I plan to do there a PR also with the new oracle/dbt-oracle adapter), the database property in profiles.yml would pose a breaking change in the Oracle Destination Connector, as suddenly users need to specify the database name (select SYS_CONTEXT('userenv', 'DB_NAME') from dual;) as a mandatory field if they want to use the Oracle Basic Normalization which is already fully implemented in Airbyte but currently not enabled due to the fact that the current Airbyte Oracle normalization implementation is still using DBT 0.19.1 from techindicium/dbt-oracle:0.4.3.

Also the OracleIncludePolicy for database is False anyways 😁
https://github.com/oracle/dbt-oracle/blob/1.0.0/dbt/adapters/oracle/relation.py#L32

Could we revert the change in dbt/adapters/oracle/impl.py because in OracleIncludePolicy the property database is False. Additionally IncludePolicy is configurable in dbt_project.yml ?

Without this change it is not working, even though OracleIncludePolicy the property database is False.

Could we revert the change in dbt/adapters/oracle/impl.py because in OracleIncludePolicy the property database is False. Additionally IncludePolicy is configurable in dbt_project.yml ?

Without this change it is not working, even though OracleIncludePolicy the property database is False.

Ok, I will check this and let you know.

Thanks for explaining this to me. I understand the following.

You would like to keep dbname or database as optional parameter in connection config. This way it will not be a breaking change for Airbyte. Also, you can create a PR there to use the new dbt adapter
Secondly, dbname or database is redundant in the WHERE clause in oracle__list_relations_without_caching macro and we can safely remove it. Thus we can leave it empty or None

Let me know if this understanding is correct.

I would like to remove dbname, database and env var DBT_ORACLE_DATABASE alltogether as it makes no sense at all. It does not matter if one connects using OracleConnectionMethod -> HOST, TNS or CONNECTION_STRING, you will always be on the same database (select SYS_CONTEXT('userenv', 'DB_NAME') from dual;) and therefore also the function impl.py#verify_database can never throw an exception, not even with a DB link.

The WHERE clause in macros
oracle__list_relations_without_caching
and
oracle__get_columns_in_relation
also do not make sense at all due to the first point.
We do not have to filter a column just for the sake that it exists in the first place.

Maybe I'm missing the technical reason but why was this dbname, database and env var DBT_ORACLE_DATABASE introduced in the first place?
Could it be that with the Oracle Autonomous Database Service (ADBS) it is possible to have multiple database (select SYS_CONTEXT('userenv', 'DB_NAME') from dual;) names with the same service/connection as I never used ADBS before.

Thanks for explaining this to me. I understand the following.

You would like to keep dbname or database as optional parameter in connection config. This way it will not be a breaking change for Airbyte. Also, you can create a PR there to use the new dbt adapter
Secondly, dbname or database is redundant in the WHERE clause in oracle__list_relations_without_caching macro and we can safely remove it. Thus we can leave it empty or None

Let me know if this understanding is correct.

* I would like to **remove** `dbname`, `database` and env var `DBT_ORACLE_DATABASE` alltogether as it makes no sense at all. It does not matter if one connects using `OracleConnectionMethod` -> `HOST`, `TNS `or `CONNECTION_STRING`, you will always be on the same database (`select SYS_CONTEXT('userenv', 'DB_NAME') from dual;`) and therefore also the function [impl.py#verify_database](https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/adapters/oracle/impl.py#L101-L112) can _never_ throw an exception, not even with a DB link.

* The WHERE clause in macros
  [`oracle__list_relations_without_caching`](https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/include/oracle/macros/adapters.sql#L344)
  and
  [`oracle__get_columns_in_relation`](https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/include/oracle/macros/adapters.sql#L185-L187)
  also do not make sense at all due to the first point.
  We do not have to filter a column just for the sake that it exists in the first place.

Maybe I'm missing the technical reason but why was this dbname, database and env var DBT_ORACLE_DATABASE introduced in the first place? Could it be that with the Oracle Autonomous Database Service (ADBS) it is possible to have multiple database (select SYS_CONTEXT('userenv', 'DB_NAME') from dual;) names with the same service/connection as I never used ADBS before.

agree and would love to see this simplification/fix getting merged.

Maybe I'm missing the technical reason but why was this dbname, database and env var DBT_ORACLE_DATABASE introduced in the first place?

This was present in Techindicium's version and we have kept it for compatibility reasons. I do see your point. Even in ADBS we will not have multiple values for db_name.

Also, even in dbt-core v0.19 (one used by Airbyte), database is a mandatory parameter in the Credentials class. These contracts are defined by dbt-core. Maybe this is also why database or dbname existed in dbt-oracle
https://github.com/dbt-labs/dbt-core/blob/0.19.latest/core/dbt/contracts/connection.py#L120

Currently, as part of this PR, we are making changes at a lot of places just to get one parameter removed. I am evaluating between making so many changes v/s simply specifying the correct value of database. You can always get the correct value. It will be helpful if you could help me understand the following things

database was always a mandatory parameter. How is this a breaking change for Airbyte ?

Based on the code changes why is this the case ? -> database can be equal to schema. The user is always the same as schema. Not database.

For the below error, it has got to do with how inheritance is handled in Python dataclasses

TypeError: non-default argument 'user' follows default argument

The superclass Credentials has 2 mandatory positional attributes -> [database, schema]
As part of OracleCredentials we add [database=None, schema=X, user, pass, ...] This ordered list of attributes is passed to the __init__() method and you get an error because user is a positional argument which follows schema which is keyword argument

I just wanted to create an Individual OCA but the oracle/dbt-oracle repo is not available for selection and it is a required field. https://oca.opensource.oracle.com/api/v1/projects/enabled
Can you help @aosingh ?

Do you have an update on this @aosingh ? I would sign the OCR in the meantime.

I have raised this internally so that dbt-oracle becomes available for selection.

Currently, as part of this PR, we are making changes at a lot of places just to get one parameter removed. I am evaluating between making so many changes v/s simply specifying the correct value of database. You can always get the correct value. It will be helpful if you could help me understand the following things

database was always a mandatory parameter. How is this a breaking change for Airbyte ?
Based on the code changes why is this the case ? -> database can be equal to schema. The user is always the same as schema. Not database.

For the below error, it has got to do with how inheritance is handled in Python dataclasses

TypeError: non-default argument 'user' follows default argument

The superclass Credentials has 2 mandatory positional attributes -> [database, schema] As part of OracleCredentials we add [database=None, schema=X, user, pass, ...] This ordered list of attributes is passed to the __init__() method and you get an error because user is a positional argument which follows schema which is keyword argument

You are right, we are making a lot of changes but in my opinion, this major 1.0.0 release is very fresh and probably not many people are using this release already.
So it would be the perfect opportunity to drop this useless parameter altogether as it is internally only used in two where clauses where it is not really needed at all.
1. The database parameter was never specified in Airbyte because of the following edge case when dbname is specified but service is not (see quoted example below).
Don't ask me why, I just recently started to use Airbyte and just want to contribute so that the Oracle normalization (JSON to normalized table with columns) works in Airbyte with dbt-oracle 1.0.0.
See the Airbyte code and history https://github.com/airbytehq/airbyte/blob/v0.38.1-alpha/airbyte-integrations/bases/base-normalization/normalization/transform_config/transform.py#L250-L264

This probably has historic reasons and also there is the confusion, where if one specifies the profiles.yml property dbname but not database and also omits service, the property dbname with alias database gets to be the service name and then the connection works anyways. https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/adapters/oracle/connections.py#L129-L134 Example profiles.yml:
config:
  partial_parse: true
  printer_width: 120
  send_anonymous_usage_stats: false
  use_colors: true
dbt_test:
  target: prod
  outputs:
    prod:
      type: oracle
      protocol: tcp
      host: yourhost.example.com
      port: 1521
      schema: test
      user: test
      pass: SECRET_PW
      # Connection works anyways as in connections.py#L83 the dbname property gets alias database
      # and if the service property is not specified the database internally is the service in the end
      # https://github.com/oracle/dbt-oracle/blob/v1.0.0/dbt/adapters/oracle/connections.py#L134
      dbname: test.service.name.example.com
      threads: 4

Also if a user specifies the wrong database  property value, let's say the user specifies the database name with a trailing space by accident -> 'mydatabase ' you will get the following error with dbt-oracle:
https://github.com/dbt-labs/dbt-core/issues/3470
This error tells the user absolutely nothing about what is going on and why it is not working.
2. You are absolutely right, this makes no sense for Oracle.
This was just some copy pasta from dbt-mysql and dbt-clickhouse, sorry about that 😁
I specified a default value for every property, so the TypeError: non-default argument 'argument' follows default argument is not an issue anymore.

Thank you @ThoSap for testing and contributing.
While we review to get this merged, could you sign the Oracle Contributor Agreement (OCA) ?
The bottom of your commit message must have the following line using your name and e-mail address as it appears in the OCA Signatories list.
Signed-off-by: Your Name<EMAIL_ADDRESS>
This can be automatically added to pull requests by committing with:
git commit --signoff

I signed off all commits using my OCA agreement using rebase, sorry for the commit spam here.
https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/changing-a-commit-message
https://hyperledger-indy.readthedocs.io/projects/sdk/en/latest/docs/contributors/signing-commits.html#signing-your-current-commit

Superseded by https://github.com/oracle/dbt-oracle/pull/6


Find & Replace: Allow regular expressions as replace string

VS allows for regex patterns use as replacement value and I'd like to be able to do just that and avoid having to open VS just to perform some find and replace I can't do with VSCode.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

100% agree with this one. Having the syntax supported in find, but not replace, just drove me a little crazy trying to figure out what syntax it used just to discover it isn't implemented

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Exporting the Custom Vision Models

Hi ,
We have also  tried the other options of exporting the Custom Vision Model
It appears that , when models are exported from Custom Vision Projects , those may be built in the older versions
For Azure ML :
We are unable to find the steps shared in the zip file
I am able to create the image with latest version of Azure ML but while publishing as a service model crashes as  this could be due to models created in older version.
For Tensorflow:
Though I have updated it to the latest version Tensorflow 2.0 , the documentation shared for both Classification & Object detection projects does not work.
This has been posted in another thread.
https://github.com/MicrosoftDocs/azure-docs/issues/38490
For ONNX:
Though the image is created , it crashes while publishing as the webservices, This could be due to similar reasons ,
Best Regards,
Swati

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

ID: 45c36ea5-8258-c917-65f3-f545f48aa7ec
Version Independent ID: 0ede94d2-c3ab-3f7b-661c-e1c2d24c0b13
Content: Tutorial: Use an ONNX model with Windows ML - Custom Vision Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/custom-vision-onnx-windows-ml.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @LarryFr
Microsoft Alias: larryfr

@jaiswati Thank you for the detailed feedback.
Could you please point us to the documentation page so that we can assist you better.

Hi Kalyan,
I did not find any specific documentation for hosting Custom Vision Packages (for Docker & ONNX export options in Customvision.ai portal)  with Azure ML Service . and then followed the  image creation with deployment as azure ml webservice as described here.
https://docs.microsoft.com/en-us/azure/machine-learning/service/how-to-deploy-and-where
Step -1
After the Custom Vision Classification project was trained in compact domain , it was exported
with 'DockerFile'  (IoT,AzureFunction,AzureML)
Step - 2
I have tried to export the custom vision models and host them as Azure  ML service (which is one of the option in the custom vision's export functionality), and there is a readme file generated within the exported zip file which contains the CLI commands for hosting the models as ML services,  but it seems the CLI commands are for older versions and is no longer supported now.
Following are the some of the steps  from the readme file
How to build and deploy an AzureML Image using this package
First you need a few files from the docker app.
Assuming you are in the azureml subdirectory run the following:
copy ..\app\requirements.txt
copy ..\app\predict.py
copy ..\app\model.pb
copy ..\app\labels.txt
These 4 files along with the score.py file will make up the assets needed to create an AzureML image.
Using the Azure ML Command Line Interface you can create and deploy a service using the following steps.
If you haven't previously setup a Model Management account, please follow the instructions
"  etc etc...
Similar is the case with ONNX models as well.
These are the links for ONNX (that I have referred for image creation and deploying as the webservice.)
https://github.com/Azure/MachineLearningNotebooks/blob/4c0cbac8348f18c502a63996fdee59c3fe682b79/how-to-use-azureml/deployment/onnx/onnx-inference-facial-expression-recognition-deploy.ipynb
https://github.com/Azure/MachineLearningNotebooks/blob/master/how-to-use-azureml/deployment/onnx/onnx-inference-mnist-deploy.ipynb
Would like to Request  for the documentation for this one (if not available)
Best Regards,
Swati

@jaiswati Thanks for the details. Can you please share the service.get_logs() details to troubleshoot.
Please follow the below sample that shows how to deploy your custom vision model to azure machine Learning.
https://github.com/datashinobi/customvision2AzureML
@areddish Can you please help on this.

Why my Github login is mentionned on this ticket ?


Added Parse: A Static Security Scanner

Checkout https://github.com/psecio/parse

Thanks @andygrunwald. 🎆


ValueTuple documentation should discuss serialization

ValueTuple documentation should discuss serialization
General
The documentation should be revised to note that:

.NET Standard, including 2.0, does not define serialization support for ValueTuples.
ValueTuples are not serialized in .NET Core 1.x or the .NET Framework 4.7 and earlier.

@ViktorHofer

https://docs.microsoft.com/en-us/dotnet/api/system.valuetuple


ValueNotifier setter not recognized as a function

ValueNotifier setter
The set value(T newValue) on ValueNotifier doesn't seem to be working. When I try to use it as I would expect I get the following error:
IDE Error: 'value' isn't a function

As a workaround I copied the ValueNotifier code into my file and just renamed to FooValueNotifier with a foo (just something other than value) method to act like the setter. I had to remove the set keyword and this worked.
  foo(T newValue) {
    if (_value == newValue)
      return;
    _value = newValue;
    notifyListeners();
  }

Steps to Reproduce
Instantiate a ValueNotifier and then try to set a new value vis the set value(T value method.
Logs
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following NoSuchMethodError was thrown building MyApp(dirty):
Class 'String' has no instance method 'call'.
Receiver: "initial string"
Tried calling: call("new string")

When the exception was thrown, this was the stack:
#0      Object.noSuchMethod (dart:core-patch/dart:core/object_patch.dart:46)
#1      MyApp.build (/Users/atown/git/misc/value_change_reproduce/lib/main.dart:11:20)
#2      StatelessElement.build (package:flutter/src/widgets/framework.dart:3678:28)
#3      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3625:15)
#4      Element.rebuild (package:flutter/src/widgets/framework.dart:3478:5)
#5      ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3605:5)
#6      ComponentElement.mount (package:flutter/src/widgets/framework.dart:3600:5)
#7      Element.inflateWidget (package:flutter/src/widgets/framework.dart:2890:14)
#8      Element.updateChild (package:flutter/src/widgets/framework.dart:2693:12)
#9      RenderObjectToWidgetElement._rebuild (package:flutter/src/widgets/binding.dart:852:16)
#10     RenderObjectToWidgetElement.mount (package:flutter/src/widgets/binding.dart:823:5)
#11     RenderObjectToWidgetAdapter.attachToRenderTree.<anonymous closure> (package:flutter/src/widgets/binding.dart:769:17)
#12     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2205:19)
#13     RenderObjectToWidgetAdapter.attachToRenderTree (package:flutter/src/widgets/binding.dart:768:13)
#14     BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&RendererBinding&WidgetsBinding.attachRootWidget (package:flutter/src/widgets/binding.dart:657:7)
#15     runApp (package:flutter/src/widgets/binding.dart:699:7)
#16     main (/Users/atown/git/misc/value_change_reproduce/lib/main.dart:3:16)
#17     _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:279)
#18     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)
════════════════════════════════════════════════════════════════════════════════════════════════════

flutter analyze:
➜  value_change_reproduce flutter analyze
Analyzing /Users/atown/git/misc/value_change_reproduce...
  error • 'value' isn't a function at lib/main.dart:11:20 • invocation_of_non_function
  error • 'value' isn't a function at lib/main.dart:14:18 • invocation_of_non_function
2 issues found.
(Ran in 6.8s)

Flutter Doctor
Paste the output of running flutter doctor -v here.
➜  flutter doctor -v
[✓] Flutter (Channel beta, v0.1.5, on Mac OS X 10.13.3 17D102, locale en-US)
    • Flutter version 0.1.5 at /Users/atown/git/flutter/flutter
    • Framework revision 3ea4d06340 (3 weeks ago), 2018-02-22 11:12:39 -0800
    • Engine revision ead227f118
    • Dart version 2.0.0-dev.28.0.flutter-0b4f01f759

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/atown/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.4.0

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] Connected devices (1 available)
    • iPhone X • 87F12481-BCBA-422D-B2C0-5C9F10698EAB • ios • iOS 11.2 (simulator)

• No issues found!

Hey @ashtonthomas : The set keyword defines value in ValueNotifier as a setter and not a regular method on the class. It's a convenience method that allows you to set the value of value on the class instance directly:
ValueNotifier<String> stringNotifier = new ValueNotifier<String>('initial String');
stringNotifier.value = 'new string';

This might explain it better: https://www.dartlang.org/resources/dart-tips/dart-tips-ep-10

Heh, and I decided to remove the disclaimer that I'm one week new to Dart... Sorry about the needless issue!
Thanks, @xqwzts.

We've all been there, good luck!

ValueNotifier<List> topBarCardss = ValueNotifier<List>([]);
how to set array list


Add a delimited text file

text file as a possible file to store and process on HDFS

@PiJoCoder : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Checking with @uc-msft

#sign-off


Is it possible to share some experiment details about model training ?

Thanks a lot for sharing this awesome project !
I really want to have a try for the training part, but didn't find any experiment details in README or your arxiv paper. Is it possible for you to kindly share some info about the questions below ?

How many disk/memory space should I prepare for the training part?
How long it takes to do supervised/rl train with single or multi gpus?
What level can the trained agent achieve? Can the trained agent beat easy/median/hard bot in sc2 ?

Hi, our journal paper has recently been notified of acceptance. I will put the arxiv version of the paper on Arxiv recently. At that time I will notify you! The journal paper contains all the experiment details!
At 2022-08-24 10:59:36, "nicklhy" @.***> wrote:
Thanks a lot for sharing this awesome project !
I really want to have a try for the training part, but didn't find any experiment details in README or your arxiv paper. Is it possible for you to kindly share some info about the questions below ?
How many disk/memory space should I prepare for the training part?
How long it takes to do supervised/rl train with single or multi gpus?
What level can the trained agent achieve? Can the trained agent beat easy/median/hard bot in sc2 ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


The video is not updated with  Sharepoint Admin webpage

@carias78 Which document you are referring to. Please indicate. Thanks

@carias78 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Limiting user login instance.

Devise allows to login user from multiple browser same time. Is there is any method to limit this. Suppose I have  2 Organizations as  clients and I want not more than 5 simultaneous login for one organization and 10 simultaneous login for another. How do I achieve this ? I search over internet and no suitable method found. Please help. Thanks in advance.

Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


Rework site tree in page editor.

Optimize and extend the site tree in the left-hand pane of the page editor:

make it more compact (smaller font? less line-spacing?)
make nodes expandable/collapsible
store state per user (what nodes were expanded/collapsed)
use different Icons depending on page type (e.g. page, internal redirection, external redirection)
distinct (via color or icon) between states "inactive" and "hidden in navigation"

Very good points, Roland!

Thank you, Roland.

No problem, you are welcome.

So are you.

Great, thanks.

E-Mail spaaaam.

Thank you, @rolandschaub! Very good points @rolandschaub!

Gäll.


Added min allowed SquareSize

Set min allowed square size for calibration to be greater than 2.2 cm.

Is it the case that below 2.2cm that the calibration is not sufficiently accurate?

Is it the case that below 2.2cm that the calibration is not sufficiently accurate?

It looked like it. User in Discord had good depth image but values were off. Calibrating by increasing square size from 1.8cm to 2.4+ approx fixed issue for him.

Got it, thanks.  We should add that to the calibration documentation as well then @saching13 so that folks know the why.
And thanks for the catch/debug with the Discord user!


[Backport 1.3] Add windows build script changes here

Signed-off-by: Naveen<EMAIL_ADDRESS>Description
Add Windows build.sh script changes
Cherry-picked from https://github.com/opensearch-project/k-NN/commit/195d26d0f4b1f2d1bdb89eee7cb51897b33b3077
Issues Resolved
https://github.com/opensearch-project/k-NN/issues/635
Check List

[x] All tests pass
[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #646 (eaf945e) into 1.3 (bcce85d) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.3     #646      +/-   ##
============================================
- Coverage     83.91%   83.90%   -0.01%
  Complexity      895      895
============================================
  Files           127      127
  Lines          3867     3865       -2
  Branches        358      358
============================================
- Hits           3245     3243       -2
  Misses          463      463
  Partials        159      159

Impacted Files
Coverage Δ

...n/index/codec/util/KNNVectorAsArraySerializer.java
62.50% <0.00%> (-2.21%)
:arrow_down:

...index/codec/KNN80Codec/KNN80DocValuesConsumer.java
82.35% <0.00%> (-0.18%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Test GHA cache with build-push action

Test cargo chef and GHA cache to speedup builds. Results show some success, but not as much as we would like. GHA cache with the build-push action still makes the build slower over all, compared to simple docker build.

Closing, as this was a test only and did not result in a clear break-through.


[FEATURE REQUEST] Make graph simple as possible

Is your feature request related to a problem? Please describe.
There are too many generated edges/vertices after parsing OSM (via edge expansion). Contraction hierarchies algorithm could take very much time to process all set of vertices. I'd suggest to implement Edge contraction to reduce size of graph for further contraction hierarchies algorithm
Describe the solution you'd like and provide pseudocode examples if you can
Find all vertices with degree == 2. Then recursively:
while exists vertex v with degree 2:
    - remove v and the 2 outgoing edges
    - add a new edge between the neighbours of v
    - the weight of the new edge is the sum of the weights of the deleted edg

reference - https://stackoverflow.com/questions/54954644/algorithm-to-simplify-reduce-graph
Additional context

Would be cool to have an optional flag for CLI (for debugging purposes) also
Some numbers for "before" and "after" (kinda benchmark) on specific graph

Some kind of workaround could be found in https://github.com/LdDl/osm2ch/pull/12

Hello,
OSM data has directed and undirected ways. So a simplifying pseudocode would be as follows:
while exists vertex v with undirected-degree 2:
    - remove v and the 2 edges
    - add a new undirected edge between the neighbors of v
    - the weight of the new edge is the sum of the weights of the deleted edges
while exists vertex v with inward-degree 1 and outward-degree 1:
    - remove v and the 2 edges
    - add a new directed edge between the neighbors of v
    - the weight of the new edge is the sum of the weights of the deleted edges


Update samsungApi.ts

smartthings api device name has changed
"name":"Samsung Room A/C"

that solve the issue @teodoradima plz merge to master.