Update arithmetic.sh

Please fill in this pull request template before submitting
1. This pull request resolves #10
2. Description
Created content for tutorial: arithmetic operators using Bash.
3. Fill in the checklist by marking [x]

[x] I've read the CONTRIBUTING.md
[x] I was assigned to this issue
[x] My code is formatted using required linting
[x] I've run my code locally and checked it works

@naz2001 Thanks, nice tutorial!
Just wondering before I merge, is there any particular reason why you have the let and expr code commented out?

Oh... I ran the ShellCheck linting from this site https://www.shellcheck.net/ and for expr and let it said that both were antiquated. It's got something to do with the style.

@naz2001 Okay that's no problem. Happy to merge :)


Server fails to startup (V 3.6)

Hi everyone, I'm trying to set up this server but I can't understand why it won't start. I am a noob with docker stuff so please be patient...
What is failing is:
2021-03-21 21:56:07.795 UTC [514] FATAL:  terminating autovacuum process due to administrator command
2021-03-21 21:56:07.797 UTC [29] LOG:  background worker "logical replication launcher" (PID 36) exited with exit code 1
2021-03-21 21:56:07.839 UTC [31] LOG:  shutting down...
2021-03-21 21:56:11.611 UTC [29] LOG:  database system is shut down
 done
server stopped
Here's the whole console log of the whole process. I'm following the README guide step by step.
PS: I know you don't officially support Windows but any help is appreciated.
nominatim 3.6 install log.txt

It seems like the data got imported. How do you start the image after the import?

I'm using the command on step 4 of the documentation
docker run --restart=always -p 6432:5432 -p 7070:8080 -d --name nominatim -v /home/me/nominatimdata/postgresdata:/var/lib/postgresql/12/main nominatim bash /app/start.sh

Ok, any errors here or is that the error you have posted above?

Sorry for the late reply. The console prints a hash of some sort
24152b1d8c15471dd942e43d2dedd194d4ba790d3d6ce6fcaf5c0a08e09a639e
but trying a request with url http://localhost:7070/reverse?format=json&lat=53.95090&lon=-6.37792 yields:

{"error":{"code":500,"message":"Failed to establish database connection:SQLSTATE[08006] [7] could not connect to server: No such file or directory\n\tIs the server running locally and accepting\n\tconnections on Unix domain socket "/var/run/postgresql/.s.PGSQL.5432"?"}}

We've released a huge upgrade with the 3.6 version. Can you try it again with the new one please and report back? Make sure to read the new Readme!
https://github.com/mediagis/nominatim-docker/blob/master/3.6/README.md


Edit Application Not Working - Something went wrong

@Dhrumil0310 could you take a look?

Upon debugging, I found that in the API /update_application, session[email] is used. However, the session object is none. @jayrajmulani Can you take a look as well?
I think this should resolve the issue.

Yes, good catch. Please create a new branch. Also check for other APIs. I suspect similar issues in other APIs as well.

Yes, found another issue in the delete API call. It was being referenced via the old /api proxy. Updated it to fetch the base URL from the config file


Unable to merge PR's from commandline

Great project, thanks!
I managed to successfully create a PR from the commandline, however when I try to merge that same PR (or any other PR I created) nothing happens. No error output either, it just stays open.
Is there any way I can see the debugging or output of the API response to see what goes wrong?

Hi @ohwhen , unfortunately merge is not supported at this time.
https://github.com/danshumaker/bitbucket-cmd/blob/8e3ece2eac8673e9ed899dd294937d9ef8c3a224/lib/bitbucket/pr.js#L88
I use the tool for a proceduralized pr submission.  We use wunderflow and thus need pr submissions on the dev flow and the release flow, and bitbucket-cmd greatly helps with easy PR submissions.   However we feel that merges should be handled with a bit more "manual" care and thus havn't implemented the merge feature on this tool - not to say that we won't in the future.
I believe Palash got it (merges) working in his fork https://github.com/palashkulsh/bitbucket-cmd , however the config setup is a little different and thus havn't merged in his fork yet.  Good luck!

thanks @danshumaker , its really saving a lot of my time on handling PRs . yes merging PR is working in my fork, but it wouldn't have been possible without your support and help.

Owen, did you setup from scratch ? because as Dan mentioned, fork works on
different config.
On Thu, May 3, 2018 at 12:28 AM, Owen van Dijk<EMAIL_ADDRESS>wrote:

Thanks @danshumaker https://github.com/danshumaker for the response, I
already found out it wasn't implemented.
@palashkulsh https://github.com/palashkulsh I tried to setup your fork
but wasn't able to get it working, kept on getting a 404 response.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/danshumaker/bitbucket-cmd/issues/7#issuecomment-386084668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFeCryi8XFa-7R1MWumOxohN2FB3kg9bks5tugHogaJpZM4TuAkV
.

--
Regards
Palash Kulshreshtha
paytm
+91<PHONE_NUMBER>

If you are still facing the issue please feel free to reopen the issue, and we'll solve it together : ) .


error in pipeline job for sonar analysis java.lang.NoSuchMethodError: No such DSL method 'withSonarqubeEnv'

Hi i tried to excute a example for sonarqube analysis by running docker image for sonarqube
step1

step2

step3

step4 i found below error

could you please suggest me on this

Below link might help you:
https://stackoverflow.com/questions/42763384/unknown-stage-section-withsonarqubeenv

please check script once

Please change your groovy script
withSonarQubeEnv('sonarqube')
not this withSonarQubeEnv('sonarQube')
Here the issue is name mismatch


Twitter bootstrap doesn't need a inline_checkbox

We need to remove it from the generator and the sample app.

Closing 099efb4d55d8d993961ff5c2446bb9715ae5df29


(WinUI 3) Color banding in Navigation View content when in dark mode

Describe the bug
There's a color banding in Navigation View when the application is in dark mode.
Steps to reproduce the bug
XAML code:
<Grid>
  <NavigationView>
    <Grid>
      <Frame x:Name="contentFrame">
        <Grid>
          <TextBlock Padding="56,16,0,0">Page content!</TextBlock>
        </Grid>
      </Frame>
    </Grid>
  </NavigationView>
</Grid>

Expected behavior
There shouldn't be any color banding.
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.0
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows Insider Build (22572)

This is probably yet another consequence of this bug: #6194, with these dupes: #6207 #6419 #6475, that has been open for over 4 months.
¯\_(ツ)_/¯
@StephenLPeters

Top left of the main pane has a rounded corner, the other 3 aren't rounded.

Top left of the main pane has a rounded corner, the other 3 aren't rounded.

Yeah but are those related to setting different corner radiuses for the 4 corners?

Idk why they are applying gradients for some reason, I just used ApplicationPageBackgroundThemeBrush (or sth i don't remember the exact name)

It looks fine in light mode

Another duplicate of this same issue which is actively being looked into.

@codendone and @gegao18 FYI

I agree the cause is the same as #6194. Please track progress via that issue.


Moving mimized Panel above SoftKeyboard

Can you please suggest any way or idea to implement this thing as I am using your library in very high commercial app.
When Soft keyboard is shown I want to move minimized panel at the top of the keyboard and Vice Versa.  I have tried windowSoftInputModes also but it does not seem to be working.

If that approach is not working you can try detecting when the keyboard is going to be shown and then change the size of the view or move the y position. Is this the answer to your question?

Hi Perdrovgs,
Thanks for your reply. I couldn't reply before coz I was working on some other thing. As you suggested I have tried setting the Y scale. Below is the code for that.
if (NavigationManager.getInstance().getDraggablePanel() != null && NavigationManager.getInstance().getDraggablePanel().isMinimized()) {
NavigationManager.getInstance().getDraggablePanel().setYTopViewScaleFactor((float)100);
}
But it does not work and however it does not seem to be a right way though as it may cause other issues like clicking on view and maximizing it again etc.
If you could suggest any other option that would be very helpful and also It is a must have feature also as your library is popular worldwide now. So hopefully if you could spend sometime on it.
Thanks

Hi @Taranlayal10 what my suggestion was to move the y position of the draggable pannel view, not invoking setYTopViewScaleFactor method. Sorry if I didn't explain my suggestion properly. Could you try changing the top value of the layout params associated to the draggable panel? Doing this you'll move the entire view Y position instead of change the top view scale factor.

Hi,
I have a solution for this in my fork.

hi @b00n12, I check in your fork I could see that draggable panel still behind the keypad. It can't move up of keypad. It'd be really helpful if you point me where to find the change in your fork.

Hi, i use the KeyBoardVisibilityEvent library in order to move the draggableview above the keyboard. So you have to set a listener for keyboardvisibilityevent to get it working:
KeyboardVisibilityEvent.setEventListener(this, draggableView);
I will provide an example and issue a pull request as soon as I have time.

Hi, i use the KeyBoardVisibilityEvent library in order to move the draggableview above the keyboard. So you have to set a listener for keyboardvisibilityevent to get it working:
KeyboardVisibilityEvent.setEventListener(this, draggableView);
I will provide an example and issue a pull request as soon as I have time.

Hi @b00n12,Thanks for update, It would be great if you will give pull request as soon as possible.


COMP: Update for changes to VTK target names

The targets have changed in VTK Git master.
Also call "vtk_module_autoinit" on the test driver, which is required on
recent VTK to avoid runtime errors such as;
vtkRenderWindow.cxx:35    WARN| Error: no override found for
'vtkRenderWindow'.
Closes #739

@thewtex What does it mean when only CDash is holding this up?

@thewtex What does it mean when only CDash is holding this up?

We can ignore the duplicate CDash status checks that are not transitioning from In progress to succeeded and merge -- this is an issue with the hook that I need to address.
Thanks for the review.

I can confirm this fixed the issue I reported to @thewtex, thanks!


cannot download a paid chapter

When I downloaded ch prologue and ch 1, it worked. But when I tried to download ch 3 which I have already paid for, it became an error.

Can you tell me the info?

language
comic name
range

Even I'm having this problem. I was able to download the ch1 but not the later paid chapters.
Language: English
Comic Name: Pian Pian
Range: I didn't add any range, just the name.
Downloader Version: 2.7.0

@yumeinmoon
Did you try to download the paid episodes?
This can download the episodes that your account can access.
If you want to download them, first of all, you should buy.

Oh so that's how it is!! Thank you so much for the clarification😁 I didn't knew that😅


[Feature Request] TSV output format with same columns as Table

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I cannot create a project..
I wish to output work items into Excel. This would be trivial by using --output tsv, but this format gives me columns such as ID, None, 3, URL.
What I want, is the columsn given by --output table, so what I end up doing is manually replacing spaces with tabs and getting rid of the headers. But the other problem with --output table is that it truncates fields.
Describe the solution you'd like
A clear and concise description of what you want to happen.
e.g. If you want a new command to be added, what would it do?
A new output format (since changing TSV would be a breaking change, I guess) which outputs the fields I select, but in TSV format.
So in the below command:
az boards query --wiql "select system.id, system.title, system.assignedto, system.state from workitems" --output tsv

Would give me multiple rows, with each row containing ID, Title, Assigned To, State, separated by tabs. Today it gives ID, None, 3, URL.
Additional context
Add any other context or screenshots about the feature request here.

Hi @SaqibS ,
This is the limitation from az-cli for tsv format.
As stated in this az-cli documentation , there is no special handling for tsv format. Also ,  it can't output the nested entries.
You might want to pass the json results to additional scripts to get the output in your required format.

Hi @SaqibS ,
I hope you were able to get the results with some additional scripts.
If you are still facing issue related to output format, you can create an issue in Azure CLI repo
I am closing this as of now.


dm: update to latest tidb to fix lightning loader

What problem does this PR solve?

Issue Number: close #4486, close #4489
What is changed and how it works?
Update tidb dep to the newest version. pingcap/tidb#32011 will fix these two bugs.
Check List
Tests

chaos test
manually trigger chaos test result can be seen in https://github.com/glorv/ticdc/pull/1

Code changes
Side effects
Related changes
Release note
None

/hold
Waiting for the test result of https://github.com/glorv/ticdc/pull/1

/unhold
chaos test result can be seen at https://github.com/glorv/ticdc/runs/5137105665. Load phase doesn't trigger failure anymore. One of the case (base(network emulation base)) failed, but this is caused by sharding ddl. So I think the bug in the load phase is fixed.

/run-verify
/run-leak-test

/run-all-tests

/verify

/hold
Need to fix the tests first

blocked by #4582

/run-kafka-integration-test

/run-integration-test

/run-all-tests

@lance6716  Explicitly disabled new collation in https://github.com/pingcap/tiflow/pull/4557/commits/caa5050d7b6043723bb6be72faec740c04b81bd2 for schema tracker, PTAL again.

/unhold

/merge

/run-integration-test


Reorganizing the headers

By popular demand.
@PipKat although there is some renaming, I did not change anything in API directory. Have a look at the other files (docker, wsl, etc)
I'm merging tonight so if you cannot have a look today, feel free to open a later PR.
@Revathyvenugopal162 ... Finally you got it! xD
Close #1688

Regarding #1688
I have decided to NOT join both sections. The WSL section is more specific than docker and it has its complexities.


Short solution needed: "How to set image size" (php-gd)

Please help us write most modern and shortest code solution for this issue:
How to set image size (technology: php-gd)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to set image size


Design a mechanism to update the soda with all the existing CSI driver deployment in k8s.

Issue/Feature Description:
Design a mechanism to update the SODA dynamically with the metadata of all the existing deploymen of CSI drivers in k8s.
Why this issue to fixed / feature is needed(give scenarios or use cases):
We need to fix this issue so that SODA can automatically decide with CSI driver needs to be picked for volume provisioning.

We can have 2 approach here :

We can make use of sidecars to update the soda-api-server with all the details of (StorageClass, PV, PVC ) whenever these sidecars gets a request to change the state of these objects. Problem with this approach might be that we might not get full state of K8s env unless and until any request lands to sidecar. And also we have to tweak all the sidecars of CSI to get this working.
We can develop a soda-syncer which will watch on the objects(SC, PV, PVC, CSINodes, CSIDrivers) and update the state of these objects to soda-api-server, this appraoch will help to get the full state of k8s env and also will help soda-csi-provisioner to enable the intelligence while picking the driver. This approach will require user to deploy this syncer in k8s env with proper permissions to connect with k8s-api-server.

@sushanthakumar @AmitRoushan @kumarashit @skdwriting @rajat-soda
Please give your suggestion on which approach will be better for this requirement.


feat: allow forced local execution regardless of cluster flags

Description
This PR adds a --force-local command line argument that forces local execution regardless of other flags. This is useful when you have a default profile environment variable but you want to test a workflow locally. This is my first contribution so hopefully I am following all the conventions/directions!
QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.

I added a test case, but there may be a cleaner way to test.

[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

If I understand the code correctly the new command line argument will add its help information to the cli page in the docs.

Hi @johanneskoester,
The workflow/profiles/ you recently added removed enough of the boiler plate options I was including with my more complex pipelines that I don't feel the need for this so much anymore, and I agree with some of the risks you pointed out, so I am going to close unless someone objects.
Thanks for taking a look!


License question

Could you please specify what license your app is published under? Without a license named, it would default to "all rights reserved" – which is not exactly what Open Source means :wink:
Further it would be nice if you could expand the app description a little, e.g. stating what features your app supports. The two screenshots do not tell very much – and even browsing all commits does not reveal a lot in this context.
Thanks a lot!

The source code is under MIT license https://github.com/AndreMiras/QrScan/blob/v20190303/LICENSE
Does this answer your question?
Regarding the app description apart from scanning codes it doesn't do much at the moment. But I'll try listing this in the readme then.

Thanks! Yes, license question is definitely answered then. Can you give me another ping when the description is updated? I'm asking because :wink:

I see, thanks for sharing more context about the request. I'm not sure how they're trying to pull the data such as the license https://apt.izzysoft.de/fdroid/index/apk/com.github.andremiras.qrscan. I can't recall there's something in the Play store console to tell about the license. Maybe it's some metadata in the APK.
Anyway, yes I'll update the README.md and Play store to describe furthermore the app and update the ticket

Well, that I can answer (the repo is mine): I go by what is visible. Now that you've added the license, it is clearly visible – so I've updated it (would go online with the next sync tomorrow usually, but I've just triggered one now to speed that up). Will update the description as soon as you tell me it's there.

But the license has always been there, it's literally the initial commit https://github.com/AndreMiras/QrScan/commit/40e3a5380ebda0cfd6bd06d7bf270227fd6f2e07#diff-9879d6db96fd29134fc802214163b95a
So do you need to do this manually or you have a scraper that does it? If you have a scraper then I don't know how it missed the license :thinking:

Funny. Then how did I miss it? Must have had crossed eyes then. Apologies for that!
Yes, initialization is manual (fetching screenshots is semi-automatic). Updates are automatic (if you tag a new release and attach the APK to it, it should pop up in my repo within 24h – and whoever uses my repo with the F-Droid client should get an update notification).

Thanks! Description updated on my end (goes life tomorrow). Added a comment on the commit with a little "request for clarification", though. Hope I don't get annoying :wink:


POST should put data into request body

Hi, thanks for maintaining this library @tasoskakour .
Quick question, maybe I am also doing it wrong, but I wonder why exchangeCodeForTokenServerURL is getting called with GET query parameters, when exchangeCodeForTokenMethod is explicitly set to POST. I think this is not correct, it should be given as request body to the POST request or make it configurable how the actual payload is getting send (https://github.com/tasoskakour/react-use-oauth2/blob/master/src/components/use-oauth2.ts#L119).
At least goauthentik doesn't support GET, but POST expects the content to be in body and not in query parameters. Cross-checked doing it in Postman and it worked.
So something like this in the useOAuth hook:
const exchangeCodeForTokenUseBody = responseType === 'code' && props.exchangeCodeForTokenUseBody; // bool

// ....
if (responseType === 'code' && exchangeCodeForTokenServerURL) {
	let response;

	if (exchangeCodeForTokenUseBody) {
		response = await fetch(exchangeCodeForTokenServerURL, {
			headers: {
				'Content-Type': 'application/x-www-form-urlencoded'
			},
			method: exchangeCodeForTokenMethod || DEFAULT_EXCHANGE_CODE_FOR_TOKEN_METHOD,
			body: new URLSearchParams({
				client_id: clientId,
				grant_type: 'authorization_code',
				code: payload?.code,
				redirect_uri: redirectUri
			})
		});
	} else {
		response = await fetch(
			formatExchangeCodeForTokenServerURL(
				exchangeCodeForTokenServerURL,
				clientId,
				payload?.code,
				redirectUri,
				state
			),
			{
				method: exchangeCodeForTokenMethod || DEFAULT_EXCHANGE_CODE_FOR_TOKEN_METHOD
			}
		);
	}
	payload = await response.json();
}

// ....

Hi @v4rakh thanks for bringing this up!
To be honest I'd like this library to be as simple as possible.
An introduction of a new prop like exchangeCodeForTokenUseBody could be a solution to what you describing, or maybe a custom fetch function passed as a prop would give even more flexibility to the users of this library.
When I find some free time, I'll investigate on how we could proceed with this, in the meantime feel free to open any PR that you mind find useful.
Cheers :muscle:

+1. Keycloak will give errors unless the parameters are given in the POST body. They cannot be provided as query parameters in the URL.

Further to my comment above, I have put in several hours trying to implement the change provided above. I am currently in npm link hell. As much as I would like to use this library, I am not capable of making the required change in the context of the NPM package, and I cannot use it because Keycloak will not accept this incorrectly formatted request.

Thanks for the feedback guys!
FYI, I'm already working on a solution at this PR: https://github.com/tasoskakour/react-use-oauth2/pull/101 in order to make exchangeCodeForToken query fully customizable so you'll be able to shape it however it fit your needs.
I estimate that the new release will be ready in 1-2 days. :+1:

Hello everyone! :wave:
:tada:  You can now use version v2.0.0 where you can utilize a new property named exchangeCodeForTokenQueryFn in order to take full control over how you send the query to your backend.
Please check here for instructions: exchangeCodeForTokenQueryFn
Basically, you will be able to do something like:
const { ... } = useOAuth2({
      // ...
      // Instead of exchangeCodeForTokenQuery (e.g sending form-urlencoded or similar)...
      exchangeCodeForTokenQueryFn: async (callbackParameters) => {
        const formBody = [];
        for (const key in callbackParameters) {
          formBody.push(
            `${encodeURIComponent(key)}=${encodeURIComponent(callbackParameters[key])}`
          );
        }
        const response = await fetch(`YOUR_BACKEND_URL`, {
          method: 'POST',
          body: formBody.join('&'),
          headers: {
            'Content-Type': 'application/x-www-form-urlencoded;charset=UTF-8',
          },
        });
        if (!response.ok) throw new Error('Failed');
        const tokenData = await response.json();
        return tokenData;
      },.
      // ...
    })


DF-501: Consistantly use Record objects to represent records

The long-awaited sequel to #345
The APIManager is gone... There is now only KongClient, and its methods return more useful values... with lazy conversion to boot.
With Record instances used consistently to represent records, we can remove a bunch of horrible code for dealing with different types of values in different places, and have a single component that is responsible for data conversion.
Please thank the following bits of code for their service:

The (necessarily) over-complicated {% record_url %} tag
Those special custom template filters in search_tags.py for extracting record details out of dicts
APIManager and all of the tests for it
Unnecessary methods on APIModel to create APIManager instances

@TNA-Allan I think everything you raised has now been resolved. Would you mind having another glance over to make sure?

@TNA-Allan I think everything you raised has now been resolved. Would you mind having another glance over to make sure?

@ababic
Just took a glance

Could no find Resolved in a0ccd87
The fake API responses are still showing.
Did you push these changes?

Hi @TNA-Allan. Sorry for the confusion:
a0ccd87 was replaced by 0533fed

The fake API responses are still showing.

I'm not following what you mean here, sorry. Could you clarify?

@TNA-Allan If you're referring to your comment from a while ago:

I think we should not fake the original response, but rely on original values. Better if we get the values at real time at some point.

I don't think we ever agreed this should be done as part of these changes. With respect, I think calling the API directly would be a huge mistake, as we'd be introducing some major flakiness into our test suite.

Hi @TNA-Allan. Sorry for the confusion:
a0ccd87 was replaced by 0533fed

The fake API responses are still showing.

I'm not following what you mean here, sorry. Could you clarify?

Hi @ababic ,
This is related to my previous comment on your commit re-

Ensure fake API responses have the same ['hits']['total']['value'] in both the 'agreggations' and 'results' responses, as the latter is now used by ResultList
I think we should not fake the original response, but rely on original values.

Those json files  are original responses from the API, we already have a factory method to create fake responses factories->create_record. Therefore without any changes made to these json files, the tests should work.

Observed a warning in CI test
/home/runner/work/ds-wagtail/ds-wagtail/etna/records/tests/test_blocks.py:7: RemovedInWagtail50Warning: Importing from wagtail.tests.utils.form_data is deprecated. Use wagtail.test.utils.form_data instead. See https://docs.wagtail.org/en/stable/releases/3.0.html#changes-to-module-paths   from wagtail.tests.utils.form_data import nested_form_data, rich_text, streamfield
It was not there previously for this PR.


VsCode

great work
would be nice to have RTVS work with VS Code and support non windows platform such as Mac.
Thanks,
M.

Thanks MikeXL - it's on the todo list!

And now, with today's announcements, support for VS for Mac would be great too.

Which one of the two would you consider of more immediate utility to you?
(This is a question for everyone who would like one or the other or both.)

I think it depends on the implementation -- if you can get the full set of tools available in vscode, that'd be great.  If the offering there is going to be less than what's in VS for Windows, then I'd prefer a full implementation in VS for Mac.
Has anyone tried pulling the repo and just building it in the new VS for Mac?  I'm assuming you'd need new version specs for the Nuget packages, but are there other known reasons why the build (without the installer) wouldn't work?  (Haven't tried this myself...)  I'm assuming you'll need a different solution for the installer...  thoughts on that?

VS for Mac is based on Xamarin Studio, and so the extensibility API there is completely different.

OK, then I think focusing on vscode first makes more sense.

Related in VSC https://github.com/Microsoft/vscode/issues/14393

Is there any progress?

Any news? Would we soon be able to install this for VSCODE as an extension or some other form?

No immediate plans. It is basically at prototype stage/shape at the moment. VS Code does not have extensible console/REPL with intellisense, does not have ability to create custom windows for plots or help and there is no connection between editor and the REPL.

Forgive me if I missed this but could someone who could help identify issues in VSCode that would allow for this to happen? VSCode features are almost completly driven by upvotes if we could add a list of those issues that would help this to happen we could all go vote for them. I've looked but I don't know much about programming and didn't find anything. Thank you for your work on this I am really looking forward to using VSCode in the future for all of my R work.

Please also consider Linux for RTVS for Visual Code

I'd point followers of this issue to this extension for VS Code: https://github.com/Ikuyadeu/vscode-R
Depending on the reasons you are hoping for RTVS for VS Code, this may satisfy some/all of your needs. It supports highlighting, linting, and REPL among a couple of other things. And if you're itching to build out some functionality for VS Code, that extension is maintained by a lone developer who I suspect would be very happy to get contributions.

I am developer of https://github.com/Ikuyadeu/vscode-R.
Sorry, I will end supporting R extension by the end of this month for finding part-time job.
Could RTVS team or someone take over developing that?

@Ikuyadeu  - unlikely. This repo actually has implementation of R for VSCode in https://github.com/Microsoft/RTVS/tree/master/src/VsCode , there is no reason to have another one.

@MikhailArkhipov
Hm, looks like this repo has not been updated in almost a year.
Is it working on VS code?
Is there documentation I could give to me PhD students?
Thanks!

It is possible to build your own from sources but extension is not officially published or supported.

@MikhailArkhipov Good! Thank you for your fast reply.
So, I can close my repository free, and looking forward to using new one.

Speaking for myself --- I would probably stick with RStudio for teaching purposes, though that isn't necessarily what's going on with PhD students --- I use VS Code with @Ikuyadeu's R extension because VS Code is so far superior as a text editor compared to RStudio.
Basic REPL support is so trivial to add to VS Code (and indeed, @Ikuyadeu has done that with the extension) that in some respects VS Code becomes better than RStudio for R development. The extension also included linting and through a related extension, LSP support. I am very thankful for @Ikuyadeu's work on it and respectfully ask that the extension be kept available, perhaps with a notice in the README that it is no longer maintained.
I'll have to give the RTVS team's version a go, though that will require me to mess around trying to figure out how to install it...

@jacob-long - when I said REPL I didn't mean R terminal. Yes, terminal is trivial. However, both R Studio and RTVS provide proper REPL with colors, history, intellisense, snippets, ability to execute code from the editor selectively and shared variable declarations between REPL and editor.

@MikhailArkhipov
Re: Rstudio.
Our team:

writes C/C++ code in VSC.
writes Python code in VSC.
writes Latex in VSC.
writes and runs build scripts from VSC.
writes and runs Azure deployment scripts within VSC.
relies on Azure DevOps VC from within VSC.

Why would we use Rstudio for a single language?

Because it has rich feature set? It all depends on you expectations of the R extension feature set. What would be the min bar of acceptable functionality? The extensions you've mentioned have millions (and Python over 18M downloads and probably close to a million regular users).  R extension has about 200K downloads, active users maybe 20K?

The minimum feature set is exactly what is currently in Ikuyadeu's extension.

Then the extension already exists, why create another one?

No need to create another. But it seems like it will be unsupported soon.
Looking back over this whole thread, I am confused about Microsofts planning. You seemed to promise Rtools for Visual Studio for Mac on some point but then pivoted to Rtools for VSC. Now it seems neither is working/supported/being developed?

I don't think anything was actually promised. There was some prototype work done since RTVS project was ported to .NET Core anyway in order to support of Linux in Azure and Docker over remote. A prototype LS was implemented as a personal side project. That is about it.  This is an open source project and external contributions are welcome.

Just echoing @stegmueller's comments. VSC  is a great platform. There are so many features that you gradually build a workflow for multiple languages. It would be great to use for R. RStudio is also great (and now has nice support for python including REPL). https://rstudio.github.io/reticulate/
Microsoft has widespread use of R https://blog.revolutionanalytics.com/2018/02/what-does-microsoft-do-with-r.html. So, R support in VSC makes a bit of sense?
@MikhailArkhipov I'm not sure the use of R vs Python extensions holds too much water if there isn't great R support in VSC.
REPL support is the critical feature.

Also would like to echo that R in VSC would be great - I've switched to writing basically everything in VSC (Python, ReStructured Text, LaTeX, Julia, even toying with C++/CUDA).
Having (almost) the same interface for everything is great. The extension from @Ikuyadeu has most of the features already (big thanks to him!), I'll just stay with it for now (compiling this repo's extension from source is not really a solution as I don't have any js/ts experience).

+1 - would love to be able to use this on a Mac.

https://marketplace.visualstudio.com/items?itemName=Mikhail-Arkhipov.r
This is personal effort, not official from Microsoft. Thanks.

Per https://marketplace.visualstudio.com/items?itemName=Mikhail-Arkhipov.r - closing


Memory limit config not working

Hi i am deploying qdrant using argo. And i want to limit the memory usage of qdrant. I have this config
qdrant:
  persistence:
    size: 10Gi

  config:
    storage:
      optimizers:
        # Maximum size (in KiloBytes) of vectors to store in-memory per segment.
        # Segments larger than this threshold will be stored as read-only memmaped file.
        # To enable memmap storage, lower the threshold
        # Note: 1Kb = 1 vector of size 256
        # If not set, mmap will not be used.
        memmap_threshold_kb: 100000

        # Maximum size (in KiloBytes) of vectors allowed for plain index.
        # Default value based on https://github.com/google-research/google-research/blob/master/scann/docs/algorithms.md
        # Note: 1Kb = 1 vector of size 256
        indexing_threshold_kb: 23000

      # Default parameters of HNSW Index. Could be overridden for each collection individually
      hnsw_index:
        # Minimal size (in KiloBytes) of vectors for additional payload-based indexing.
        # If payload chunk is smaller than `full_scan_threshold_kb` additional indexing won't be used -
        # in this case full-scan search should be preferred by query planner and additional indexing is not required.
        # Note: 1Kb = 1 vector of size 256
        full_scan_threshold_kb: 50000
        # Store HNSW index on disk. If set to false, index will be stored in RAM. Default: false
        on_disk: false

  updateConfigurationOnChange: true

  resources:
    limits:
      cpu: 1500m
      memory: 3000Mi
    requests:
      cpu: 1000m
      memory: 2500Mi

I can see that this values are used because pvc has 10Gi, but pod is still beikng killed by k8 because of the memory limit.
I have already indexed something about 1M points not shure exactly. But the qdrant pod is in crash loop and it doesnt seem like changing memmap_threshold_kb helps.
When i described the productin.yml i got this
production.yaml:
----
cluster:
  consensus:
    tick_period_ms: 100
  enabled: true
  p2p:
    port: 6335
storage:
  hnsw_index:
    full_scan_threshold_kb: 50000
    on_disk: false
  optimizers:
    indexing_threshold_kb: 23000
    memmap_threshold_kb: 100000

i didnt realise that the setting has to be active before creating collection


Um hello

There is a problem whenever I go to config.js and type npm install it doesn't work pls help

also have same issue

me too

same here


Look into getting our own analytics or understand the GA better

I would like to see if we can spin up our own open source analytics if possible, to keep in our mission for open source

@vim-usds is this a duplicate of #1322?

Yes, I might add the title of this ticket to 1322
Vim Shah (he/him/his)
Software Engineer
United States Digital Service
Work:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
From: Beth Mattern @.>
Reply-To: usds/justice40-tool @.>
Date: Wednesday, March 2, 2022 at 3:10 PM
To: usds/justice40-tool @.>
Cc: "Shah, Vim K. EOP/OMB" @.>, Mention @.***>
Subject: Re: [usds/justice40-tool] Look into getting our own analytics or understand the GA better (Issue #1309)
@vim-usdshttps://github.com/vim-usds is this a duplicate of #1322https://github.com/usds/justice40-tool/issues/1322?
—
Reply to this email directly, view it on GitHubhttps://github.com/usds/justice40-tool/issues/1309#issuecomment-1057333886, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUSCJV5YG3PJ6LMSSVPH6I3U57DLPANCNFSM5OYRWXRA.
Triage notifications on the go with GitHub Mobile for iOS<%3ehttps:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675%3c> or Android<%3ehttps:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub%3c>.
You are receiving this because you were mentioned.Message ID: @.***>


Provide azd init flag to "Use Code in Current Directory" without having to prompt user

We want to be able to run azd init without it requiring user input.
@ellismg has the context.

I'm closing this as duplicate of #3361 based on the title. However, I suspect that the scenario is to fully run azd init without any prompts. We can continue that discussion on #3341.


Update documentation regarding primitives and ISA circuits

From Slack
Some questions on primitives moving to ISA-only circuits
It sounds like the immediate plan is to advise users to do the observable manipulation locally. For SparsePauliOp, apply_layout makes this easy. But there isn't a built-in equivalent for the other ObservableLike types (str, Pauli, or dict). Is there a way to cover these other cases?   Need documentation on how users should handle the other cases themselves.
Possible SME @jakelishman

since qiskit 1.1 Pauli now has apply_layout, and generally speaking we always recommend using SparsePauliOp in our docs. I think we should focus on encouraging people to use SparsePauliOp as the "best" way to do things, and if we come to a point where we have to document these other types then we can reopen this issue


Doesn't seem to work with artifactory 4.7.4 and jspm 0.16.29

Hi. This looks like exactly what we need to download github packages from JSPM. However all packages are returning a 404 when I do a jspm install after configuring the plugin. I followed your directions to the letter (I think) and no packages can be resolved. Here is the artifactory log file. Any idea?
'null' Cached key has been found for request: '/artifactory/api/vcs/github/artifactory/api/vcs/tags/github-remote/twbs/bootstrap' with method: 'GET'
Entering request GET (<IP_ADDRESS>) /api/vcs/github/artifactory/api/vcs/tags/github-remote/twbs/bootstrap.
Could not transform artifactory/api/vcs/tags/github-remote/twbs/bootstrap to a valid module info.
Could not transform artifactory/api/vcs/tags/github-remote/twbs/bootstrap to a valid module info.
Could not transform artifactory/api/vcs/tags/github-remote/twbs/bootstrap to a valid module info.
github is not a valid module info -  'artifactory/api/vcs/tags/github-remote/twbs/bootstrap': not enforcing snapshot/release policy.
Could not transform artifactory/api/vcs/tags/github-remote/twbs/bootstrap to a valid module info.
Could not transform artifactory/api/vcs/tags/github-remote/twbs/bootstrap to a valid module info.
github-cache is not a valid module info -  'artifactory/api/vcs/tags/github-remote/twbs/bootstrap': not enforcing
Check isNewerThan. resourceLastModified=0, roundedResourceLastModified=0, modificationTime=0
Sending HTTP error code 404: Not Found
Exiting request GET (<IP_ADDRESS>) /api/vcs/github/artifactory/api/vcs/tags/github-remote/twbs/bootstrap

Yep, I've assumed that the github's remote repository on artifactory is named 'github-remote' (as mine was)
So if you have a remote repository named 'github-remote' you should register the plugin as with your artifactory server url and not the repository path.
jspm config registries.github.remote <artifactory-server-url>
i.e. jspm config registries.github.remote https://yourartifactorycompany.com
Let me know if it helps, and I will update the readme file.
If you wish to register a different name, I will need to change the code accordingly, but it will take me some time.
Many thanks :)

Thanks! That worked.

Should it be https://artifacts.company.com or https://artifacts.company.com/api/vcs/downloadBranch/github.com/

Tried both dint work for me


bruno: 1.17.0 -> 1.18.0

Description of changes
Release: https://github.com/usebruno/bruno/releases/tag/v1.18.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313904 run on x86_64-linux 1

1 package failed to build:

bruno

I got a hash mismatch building on aarch64-darwin.
specified: sha256-vYN245vMt/NjISaaFSXOkELONVld6knaKbi5FiN/0tA=
got:    sha256-RwTJhMMdJHgsJYaFSRgdyFEOOosgIkQArUs+6iYXNnE=

Not sure why...

Something strange is that gepbird and kashw2 were able to build on Linux; lucasew hit a failure when running nixpkgs-review on Linux; I failed to build on Darwin; and lastly, I was able to build on Linux as well.

I wonder if this is related to something in master but not merged into this branch. It crops up for a nixpkgs-review` but not a regular build against this branch.

I wonder if this is related to something in master but not merged into this branch. It crops up for a nixpkgs-review but not a regular build against this branch.

It fixes the package.lock at fetch time. It's a workaround but it normally works.

Not sure if I was saying what I meant clearly; I wonder if between when this branch was created and now, a change to a derivation or tool on master caused the hash generated by pulling the source & fixing the lock file to be different.

Not sure if I was saying what I meant clearly; I wonder if between when this branch was created and now, a change to a derivation or tool on master caused the hash generated by pulling the source & fixing the lock file to be different.

Not sure but I don't think so. BTW normally r-ryantm does the job on this package and we only test if it runs then merge using the bot if OK.

BTW normally r-ryantm does the job on this package and we only test if it runs then merge using the bot if OK.

@lucasew If r-ryantm works I'll stop doing manual version bumps on bruno. But when the bot fails to build the updated package for some reason and doesn't post a PR, how do we know it happened and we need to fix something? Is there some hydra-like UI for the bot that shows which packages it failed to build?
I see that @mattpolzin got sha256-RwTJhMMdJHgsJYaFSRgdyFEOOosgIkQArUs+6iYXNnE= on aarch64-darwin and @lucasew got sha256-QIew... (on aarch64-darwin too or x86_64-darwin?) hashes which are different. Do you consistently get those hashes or they are different every time? If they are consistent, we could provide different hashes for different architectures, just like how binary packages do it.
It builds on both linux arches for me, so it must be a darwin issue, but I don't have that to test it. Can you check the diff of the generated package-lock.json files?
Result of nixpkgs-review pr 313904 run on x86_64-linux 1

1 package built:

bruno

Result of nixpkgs-review pr 313904 run on aarch64-linux 1

1 package built:

bruno

@lucasew If r-ryantm works I'll stop doing manual version bumps on bruno. But when the bot fails to build the updated package for some reason and doesn't post a PR, how do we know it happened and we need to fix something? Is there some hydra-like UI for the bot that shows which packages it failed to build?

https://r.ryantm.com/log/bruno/

FWIW, I just rebuilt on aarch64-darwin off of master post merge of this PR and it worked. I'm confused, but not disappointed.


Enhancement: add antiX as a demo distro

I tested the antiX distro on copy v86, and it seems to work very well. It is an established lightweight distro based on Debian and comes in 32 and 64-bit versions. The "base" version seems to have reasonable functionality for use on copy v86. Would be nice if it was demo version so that users could then select antiX and install a few additional packages and then "save state" to be able to rapidly restart work in that distro in their browser.
It would then be additionally useful if that demo antiX browser would be able to access the user's local filesystem (similar to how the ArchLinux demo distro does).
Thanks for considering this request.

This sounds really cool from my point of veiw but I'm not a v86 dev. I do have the following concerns:

The AntiX image is quite big for the web
AntiX is going to be really slow once booted
AntiX by default uses a kernel optimized for old hardware and would require a custom kernel to use virtio
apt is network intensive and might put to much demand on the proxy

If you are looking for a light distro you may want to consider compiling buildroot with only the minimal functionality configured.
There are plenty of buildroot configs to base off of but here is mine https://github.com/Darin755/browser-buildroot

Thank you very much, @Darin755. I'll take a look at your buildroot as a base :-)

Your live demo does boot pretty fast, @Darin755. Probably be too hard to add R into your buildroot without a package manager to access a binary package of R? I imagine trying to compile R from source inside of buildroot in the emulator in the browser might take a while.

Let's close this, as I don't plan to upload an antiX profile.

Your live demo does boot pretty fast, @Darin755. Probably be too hard to add R into your buildroot without a package manager to access a binary package of R? I imagine trying to compile R from source inside of buildroot in the emulator in the browser might take a while.

You don't compile it inside v86. You build the image on a local machine


Searching in Search creates an endless loop

Steps to reproduce:

Open the app
Click on the search icon, type something and click the keyboard's search icon
The search results appear
Click on the navigation bar (or word you typed before), type something else and search again.
The UI is inconsistent: there's a "No results for newword" message in the background and the UI spins forever. In the app, that means that there's a endless loop in the SearchActivity + SearchViewModel

After debugging it, it seems like when you search for the second time and the SearchActivity gets new search results (L112), the InfiniteScrollListener gets triggered (L161) and calls loadMore again.

Happens when connectivity is slow so that makes it flaky


make uWebsockets available as library

hello,
thanks for this great code,
however:

can be made a wrapper with C extern to make it compiled as library or static object (.so linux is enough)?
have done stress load tests or realcase scenario for scalability and reliability? Any results to look?
best regards
R.

You can find stress tests in benchmark folder. C is great and all, but this library is a C++ one, so you can't really easily expose it as a C library. Well maybe by using lots o macros but that is not a direction I want to take. You could try and wrap it yourself and see if it turns out any good.


Question: AutoLispDebugAdapter not found

Hey, I'm getting error below. I can't find any information online, how to fix it. My autocad path and process are set correctly.
I'm using version Autocad 2020. How do I know which versions supports this adapter?

AutoLispDebugAdapter.exe doesn’t exist. Verify that the file exists in the same folder as that for the product specified in the launch.json file.

We support the vscode extension debugger since AutoCAD 2021. You can check the sysvar lispsys. If it is available then it should support the vscode debugger.

@Sen-real , can we update the error message to indicate it only works after 2021?

@LanDong-adsk
good idea

Close this issue. Solved in PR129.


Convert should be a ScriptableObject

This one I am less convinced on...
I am unsure how exactly this would work. i.e. how do you set parameters of the converter?

Can we find a way to support MultiValueConverters?
https://www.c-sharpcorner.com/UploadFile/87b416/wpf-multivalue-converters/

I think the converters should be ScriptableObject because,

they don't use the functionality of the components
This will allow you to reuse one converter between different bindings and scenes.
For example on the loading screen in the menu and when passing a level, I show the competition percentage
And here I can use the same converter

I can see the value here for re-use, however I think it is really useful to be able to easily create instances in the editor and change parameters easily.
Maybe we can have it both ways?
Make...
SerializedValueConverter : SerializedObject : IValueConverter {}
Then change       public ValueConverterBase Converter; in DataBinding.cs  to       public IValueConverter Converter;


Some docs & examples use top-level Date.now() for cookie expiration

What version of Remix are you using?
1.4.0
Steps to Reproduce
(Problem is the docs, see below)
Expected Behavior
Hi and thanks for the great work on Remix !
In the docs https://remix.run/docs/en/v1/api/remix#using-sessions and in some examples https://github.com/remix-run/remix/tree/main/examples/redis-upstash-session the proposed session storage code uses calls to top-level Date.now() for cookie expiration date.
While it may seem to work in development because it looks like the modules are re-imported for every request, it does not in production. Date.now() then does not have much sense, since it is only the timestamp when the server was started, not the timestamp of the request.
Also, I found it confusing to create a cookie outside the request/response lifecycle. As I understand, it's not actually a cookie, but cookie options instead. But it's weird, because the expires option doesn't make sense. (Unless we recreate the session storage for every request. And the docs & examples don't seem to go that way).
Hope I'm not missing something...
Cheers !
Actual Behavior
(Problem is the docs, see above)

Fixed by #3091


[protobuf] build

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

First options:
https://github.com/microsoft/vcpkg/issues/42582

In vcpkg CI:
Elapsed time to handle protobuf:x64-linux: 21 min


Add middleware

Our users might want to run some code before and after their endpoint handlers; something like:

Middleware in Fresh
Middleware in Next.js
Python WSGI middlewares
Ruby Rack middlewares

This also came up in #1102 and #997.

This also came up in #1102 and #997.


Typo in PublishInternetEndpoint

I have found a little typo
checkout - section "Configure a route-specific endpoint"
checkout "powershell" tab, 2nd bullet point
PublishInternetEndpointTo
correct: ... set the -PublishInternetEndpoint parameter to true

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

ID: 117e636b-4a91-084d-e52b-7b0a11d3b11d
Version Independent ID: 460ea36c-23d0-bc5a-28d1-15b9c16eb639
Content: Configure network routing preference - Azure Storage
Content Source: articles/storage/common/configure-network-routing-preference.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@lorenz-haenggi-jarowa Thanks for the feedback. We are investigating the issue and will update you shortly.

@@lorenz-haenggi-jarowa  The PR has been initiated, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Move _gettimeofday_r call in clock_gettime (IDFGH-1525)

Thank you for your report. I placed it in your internal repo, will be merged as soon as possible.

@raldone01 Thanks for the contribution. The PR has been merged, thanks.


Location in photo upload throws error

The loaction chips are throwing errors:

Invalid argument(s)

Please provide more details of it. What errors are they throwing. When they are throwing errors

This is the exception that I'm getting:
The following ArgumentError was thrown building FutureBuilder<List>(dirty, state: _FutureBuilderState<List>#ed3b5):
Invalid argument(s)
The relevant error-causing widget was:
FutureBuilder<List> file:///Users/tyler/AndroidStudioProjects/Instagram-Clone/lib/ui/insta_upload_photo_screen.dart:152:22
When the exception was thrown, this was the stack:
#0      _StringBase.+ (dart:core-patch/string_patch.dart:262:57)
#1      _InstaUploadPhotoScreenState.build. (package:instagram_clone/ui/insta_upload_photo_screen.dart:176:42)
#2      _FutureBuilderState.build (package:flutter/src/widgets/async.dart:732:55)
#3      StatefulElement.build (package:flutter/src/widgets/framework.dart:4623:28)
#4      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4506:15)
...


Fix: Settings/Portfolio UI

Closes #17191
Closes #17199
It is not a quick implementation to link from the project form labels to the associated projects on learn. I would have to re-write a component that will disappear shortly.
Home

Settings

Portfolio

@bouncey I agree with @raisedadead about the public / private instead of on / off.


Promote Retiarii to NAS (step 2) - update imports

Description
Update the imports to make the code runnable and backward compatible.
Test Options

[ ] fast test
[ ] full test - HPO
[ ] full test - NAS
[ ] full test - compression

Checklist

[ ] test case
[ ] doc

How to test
This PR should pass pipeline.

How do the IDEs behave?

How do the IDEs behave?

IDEs will prompt you to write the full path like nas.nn.pytorch.xxx.


Initializers for Swift data types

Hi,
I add initalizers for swift data types and added some NSData properties and functions.
I am not quite sure how to make Initializers for Arrays and Dictionaries from MessagePackValue, maybe someone can help me there, but the other work well so far.

This looks good, but I would make the initializers that take MessagePackValue not take an Optional. That's what Optional.map is for.


RE-146 Ensure the apt cache is updated

Before we can install the fontconfig package, we need to
update the apt cache so that apt is aware of where it
can be installed from.
Issue: RE-146

recheck_release
recheck_artifact

recheck_all


Generalize and improve message API

Message driver API was modified, to better match drivers like MQTT.
I also removed the IInOutDriver base für specific drivers. Combining them in the interface provides no benefit over simply implementing all supported interfaces.

I also removed the IPlcDriver since it is basically just a message driver and simply the predecessor.


74 - Add Pagination To Search Result Page

Issue Number
#74
The initial commit of this PR starts at 2144187.
Purpose/Implementation Notes

Based on the design, we'll add the pagination to the search result page.

Implemented the pagination and its necessary logic on the search result page

Files related to this PR

[ ] UI review w/ @dvenprasad  - please visit the latest here, thank you!
(After passing the UI review, @davidsmejia will be added to the reviewers.)

NOTE: For this PR, due to the limitations for building and testing UI, the mock data was removed and replaced with the data fetched from the actual API (the same url used in the current refine.bio front-end local dev).
Types of changes
New feature (non-breaking change which adds functionality)
Functional tests
List out the functional tests you've completed to verify your changes work locally.
Checklist

[x] Lint and unit tests pass locally with my changes## Issue Number

🗒 As mentioned in 1:1, due to the changes made in the recent PRs (API integration for search, experiments, and samples (in progress) endpoints),  I’ll be closing any PRs that are no longer relevant. Since the pagination was updated during the API integration process in the search manager and the samples table manager I am closing this PR as well.


All: Use Rollup and Babel to build

Addressing #707. This will enable us to use ES6 via Babel and build our application via Rollup, meaning no more reliance on specified concat order.
Some interesting things to note:

The code in src should mostly be free from QUnit expressions, other than in src/core and src/export which is where the definition and export of the global QUnit object occurs.

JSHint now runs on the individual source files and is more strict to ensure that we are explicit about dependencies.

Anything in reporter/runner does not depend on any code in src other than src/core which exports QUnit and src/globals which provides access to global objects.

There are some files in reporter/runner that have their contents wrapped in an IIFE to allow them to return early. These files should be converted to export functions that execute similar code. This will eventually make it easy to ship QUnit separately from the runner and reporter which are specific to the browser.

Improvements for future PRs:

Remove all IIFEs, these shouldn't be needed anymore

Breakup some of the larger modules to simplify dependency graph (e.g., src/test and src/html)

Move all source code into src. This might be contentious, but I'd like to see a structure similar to the following:
qunit/
   src/
     qunit.js
     qunit.css
     core/
       index.js
     reporter/
       index.js
     runner/
       index.js

   The `qunit.js` file will import `core/index.js`, `reporter/index.js`, and `runner/index.js` and those will then be responsible to act as entry points for the functionality encompassed within. `index.js` could also be something like `main.js` or `exports.js`. The idea here is to have just one directory for our source code that can also mirror how we'd like to think about our dependency graph.

Test failure is due to Node 0.10 not having Map support 😢  will fix in a bit

I want to prioritize this as we're in a moment we can release a new version without any new features in the API or bug fixes.
This will give us some time to check everything.

Test failure is due to Node 0.10 not having Map support :cry: will fix in a bit

With this and the comments I've mentioned above getting fixed I believe we'll be good to roll.

So apparently Rollup doesn't support Node 0.10. Even running with the --harmony flag doesn't seem to work properly. It also looks like Webpack only supports >0.12 versions of Node.
This only really affects the build pipeline. So maybe we can revamp Travis to build using a supported version and then use nvm to change to 0.10 to test against? Ideally I'd like to just drop 0.10 of Node, but LTS doesn't end until October.
Would like your thoughts @leobalter

I'm ok to drop support for 0.10 in this case if that won't represent any problem for other projects using QUnit. This might require a new major version which we could attach the PR dropping QUnit.push so we something else for a new 3.0.0 version.

I think we should definitely slate 0.10 (and probably 0.12 as well) for dropping in QUnit 3.0, but doing that now seems a bit pre-mature since QUnit 2.0 is still new.
Instead I updated the Travis configuration to build against a recent version of Node but to still test against our supported versions. We shouldn't need to support all the same versions for our development/build environment as we do for the distributed version of QUnit.

Instead I updated the Travis configuration to build against a recent version of Node but to still test against our supported versions.

Note that this means you can't have any native dependencies or devDependencies as then you can't install packages with one Node & run them with another one. You're also now using a different npm version to install packages and a different one to run npm test. This seems risky but at the same time I don't see another easy way: you'd basically have to npm install with a new Node, build QUnit, switch to the old Node, re-install all dependencies but retain the build QUnit files and then run the tests.

you can't have any native dependencies or devDependencies

By native, you mean global? Regardless, I don't foresee this being a problem, but good to keep in mind.

you'd basically have to npm install with a new Node, build QUnit, switch to the old Node, re-install all dependencies but retain the built QUnit files and then run the tests.

I would actually be okay with this, except running npm install also runs prepublish which would attempt to re-build the files causing an error and I don't see a good way around that.

Having both src/qunit.js and src/core/index.js seems strange. Otherwise I like the suggestion to move all source files into a src folder.
As for dropping node 0.10, I wouldn't worry too much about it. Whoever is still on 0.10 is probably not going to touch a (current) QUnit release. October isn't far away, might as well drop that immediately. Also, http://qunitjs.com/ only mentions browser support, not node.

Whoever is still on 0.10 is probably not going to touch a (current) QUnit release

The only issue here is that will require a major release (by dropping support) and @trentmwillis pointed it 2.0 has got released recently.
What we can do is land this without a release, focus on the cli (#1024) with a built in reporter and release 3.0.0 when we get all of them done.
Patches (2.0.x) might still have progress for bug fixes, but we would stall QUnit for any minor version (2.1.0+).
This strategy will require us to avoid much further breaking changes, like removing something from the API, allowing us to remote the only method we already have deprecated, .push

If this plan works, we can expect a new major version being released around October, otherwise, we need keep the restraints with 0.10.

I don't understand how dropping 0.10 in our build pipeline is equivalent to a major, breaking change. A few more thoughts there:

Did we ever officially support 0.10? If not, we're not dropping support
Is there any actual compat issue with 0.10 with the changes we're discussing? It seems like dropping 0.10 on Travis makes it easier to build/test on all other platforms, but that's it
Holding off on releases to sprint to 3.0 sounds like a bad idea. I'd like to see js-reporters land as part of 2.x, and that takes time
Patch releases for old versions can also eat a lot of time. Better to be able to always release from master.

@gibson042, can you give your feedback on this one?
From the main projects I take in consideration for support, we have lodash, which still supports node 0.10. This might be a PITA support to keep, but I would also prefer having more feedback from @jdalton.

I'm cool with whatever you all need to do. Lodash will drop Node 0.10 support in v5 (Jan 2017). I only ever use the QUnit dist build though so changes to your build requirement don't effect my use.

Ok, the way @trentmwillis fixed the CI process, it looks good to me landing this PR.

Having both src/qunit.js and src/core/index.js seems strange. Otherwise I like the suggestion to move all source files into a src folder.

I like the src/qunit.js as the starting point. We can move the src/core/index.js to a better naming in a follow up.

Is there any actual compat issue with 0.10 with the changes we're discussing? It seems like dropping 0.10 on Travis makes it easier to build/test on all other platforms, but that's it

@jzaefferer exactly. This "drop" was primarily talking about our build/testing processes. The distributed library itself should still work in Node 0.10.
@leobalter I would like to hammer out what we want our Node support policy to be. We currently have a "rolling" support policy for browsers and I tend to think we should probably adopt something similar with Node. Ex: we support Node LTS releases, but any QUnit released after LTS ends is not guaranteed to be compatible (similar to how we don't test against all versions of Chrome since 2.x was released, we only care about the 2 most recent). That way, when 0.10 LTS ends in October we can drop it without having to do a major version bump.
As for the file structure, we can debate that when I open the PR to make the change.

surprisingly we have no specific support mentioning Node 0.10. We can't say we actually had any support as some previous versions of QUnit didn't work on Node at all.
Similar to the browsers I want to say we should support the last 2 LTS versions if more than one is active. That would target us to Node 4 and 6. At some point this will make more sense.
The alternative is to say we support the current Node LTS versions from the time of each release.

Similar to the browsers I want to say we should support the last 2 LTS versions if more than one is active. That would target us to Node 4 and 6.

Currently that would target 0.12 & 4, not 4 & 6. Node 6 is not an LTS, it will become an LTS at the time of Node 7 release.
https://github.com/nodejs/LTS

There's only one LTS and one stable at a time, right? That's two versions, similar to Chrome's last two versions support. So Node 4 and 6, right?
I don't know if we really need to support two LTS versions plus a stable, do we? (Honestly not sure, not trying to persuade folks.)

@platinumazure No, there are more supported versions. Just click the link I provided, it's all explained there.

A couple of nits, but on the whole this looks great! :clap:

Updated to address @gibson042's comments.
I think we should land this. Since we have no official Node support policy as of this moment, I don't see a need to do a major version bump. Additionally, we're only dropping Node 0.10 from our build environment which should not concern our consumers.
I do think we should figure out an official policy for Node support, but we should likely do that in a separate issue.

Woops, looks like Node 0.10 and 0.12 don't like the template literal in the rollup config. Changing it back...

Curious why the choice to go with Rollup? I've found Webpack to be super flexible for builds.

@jdalton honestly, we could have gone with either. After looking at both Webpack and Rollup, they both support everything that we need. I wound up choosing Rollup because it seemed to more closely align with the architecture of QUnit, in that we only have one real module we distribute. We hope to change that in the future so that Node users don't need all the Browser-specific code, but even then we're still only likely to have a small handful of modules. Rollup also seemed a bit easier to get up and running.
At the end of the day, the primary goal is to support ES6 and start using a standard module syntax, so if needed we could switch to Webpack (or some other tool) in the future.

I have no objection to landing this.

@leobalter would like your :+1: before landing.

👍 let's do this!

For a heads up on the rollup/webpack, I agree with the answer from @trentmwillis. I believe it should not be any problem to switch it later. I have some experience with webpack but I'm interested to try rollup for now.

Merged, but build error'd. Not sure why, investigating now...

Update: there was a back-compat issue in Babel that messed up the preset for es2015. Find out more in this issue: https://github.com/rollup/rollup-plugin-babel/issues/72
Gotta hang tight until a new release is cut. At which point I'll kick the build again.


Can't install in react-native 0.74 using NPM I

Environment

System:   OS: macOS 14.2.1   CPU: (8) arm64 Apple M1   Memory: 98.03 MB / 16.00 GB   Shell:     version: "5.9"     path: /bin/zsh Binaries:   Node:     version: 21.5.0     path: /opt/homebrew/bin/node   Yarn: Not Found   npm:     version: 10.2.4     path: /opt/homebrew/bin/npm   Watchman:     version: 2<IP_ADDRESS>     path: /opt/homebrew/bin/watchman Managers:   CocoaPods:     version: 1.14.3     path: /opt/homebrew/bin/pod SDKs:   iOS SDK:     Platforms:       - DriverKit 23.2       - iOS 17.2       - macOS 14.2       - tvOS 17.2       - watchOS 10.2   Android SDK: Not Found IDEs:   Android Studio: 2022.3 AI-223.8836.35.2231.11005911   Xcode:     version: 15.1/15C65     path: /usr/bin/xcodebuild Languages:   Java:     version: 17.0.9     path: /usr/bin/javac   Ruby:     version: 2.6.10     path: /usr/bin/ruby npmPackages:   "@react-native-community/cli": Not Found   react:     installed: 18.2.0     wanted: 18.2.0   react-native:     installed: 0.74.1     wanted: 0.74.1   react-native-macos: Not Found npmGlobalPackages:   "*react-native*": Not Found Android:   hermesEnabled: true   newArchEnabled: false iOS:   hermesEnabled: true   newArchEnabled: false
Platforms
IOS and Android

Versions

Android:  API 33
iOS: iOS 17.2
react-native-modal-datetime-picker :17.1.0
react-native-community/datetimepicker:
react-native: 0.74.1
react: 18.2.0

Description

I can't install this package, I try to do ''npm i'' but I get this error:

Reproducible Demo

i have the same problem how can i fix it pleaaase


Add Item Filtering

Can you add item filtering/ban, or like denie offers if there item below x ammount.
Just to prevent people to add cases or 3 cent skins to do the over pay.
like 1$+0.03+0.03+0.04 = 1.1$ (simply denies items below 3 cents [can be configured on config])

mhm i would suggest actually Looking through the readme, instead of telling people what to do. youve made so many issues for this, if you actually wanted to add something either wait or do it yourself.


Create website

We should create a simple website for Lazybones with:

A short description of what it is
Perhaps a short screencast showing it in use
Links to documentation

Getting started
Creating templates

Details of how to contribute

JBake and GitHub Pages play really nice together for static project information sites - Baking Your Blog with JBake, Groovy and GitHub. I'd be willing to take an initial stab at a site unless you know of someone else already working on it.

I have a simple starting point site implemented as a sub-project using JBake. I am on the fence about whether or not the site should build with the overall build or only when specifically requested. I am leaning towards having the site be outside the normal overall build... any thoughts or preferences? Publishing, would of course, require specific action.

I would give the site project a build target that generates the HTML, but I wouldn't link it to any of the other projects.


in actionmode you can press on stuff in the actionbar

When using actionmode in ABS4.0 beta 5 I can click on my spinner and action items below the action mode bar. I'm using Android 2.3.4.

This appears to be because the animation is not being cleared from the ActionBarView after it's done animating visibility.
The ViewGroup does the following check before passing on a touch event:
if ((child.mViewFlags & VISIBILITY_MASK) == VISIBLE || child.getAnimation() != null)
getAnimation() on ActionBarView never returns null on <=2.3, while it correctly does so on 3.0+.

Damn. That's going to be tricky...


Uses only 1 gpu in system.

Uses only 1 gpu in system, need cmd option to set platform_id and device_id.

Thanks.
If you know anything about GPU dev, we'd appreciate some help. Otherwise, it might take us a few days or weeks to get to this.

you can now select which gpu the miner uses, you can use multiple gpus by running multiple instances of the miner


Fix hop field serialization for large IFIDs

Hop field serialization would break whenever egress IFID values larger than 255 were used.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Disable drag and drop on avatar

Some avatar (on chat and order details windows) have drag and drop enabled. When dragged, they show, for instance, a link like the following:
file:///C:\Users\User\AppData\Local\OpenBazaar\app-1.1.5\resources\app.asar\index.html - userPage/a06aa22a38f0e62221ab74464c311bd88305f88c/store
This link is useless outside the software and should be disabled.
OB v1.1.5 win 7 64 bits

I'll fix it


Missing deepspeech dependency in requirements.txt

I wanted users to first install deepspeech as per their hardware requirements(with/without GPU) and verify it using command prompt, then use this server to serve it on web. Same is mentioned in readme of this project.
Therefore, I intentionally didn't include it in requirements.txt.


Add token instructions into getting_started and fix spelling in qiskit notebooks

Added basic token instructions into getting started -
a) Added a line pointing new users to the QX API token generate page, just to smooth the flow
b) Made the instructions compatible with Q Network setup flow, so we can have a single getting started notebook compatible with both QX and Q Network.
Context: For the last x months we've used a separate notebook for Q network setup that we sent to client administrators manually. The qiskit-tutorials getting started notebook wasn't compatible with Q Network before, and many new users encountered errors during setup because they didn't know about our Q Network setup notebook or were simply using the Qiskit tutorials and didn't know the nuances of where to change code for Network usage. This was the source of 3 different escalations from ORNL. The recent registration flow changes make it much simpler to maintain a single "getting started" point for both QX and Q Network users.

I copied every notebook in the qiskit folder into word to spellcheck them, and fixed spelling errors.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Donny seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Can you sign the cla


improve NativeModuleRegistryBuilder.java

Motivation
I met the error Native module xyz tried to override xyz for module name xyzModuleName. If this was your intention... after something went wrong during react-native link - one module somehow ended up being included twice in the getPackages method, as in:
@Override
        protected List<ReactPackage> getPackages() {
            return Arrays.<ReactPackage>asList(
                    new MainReactPackage(),
            new WowPackage(),
            new WowPackage(),

Since I have > 20 native modules it took me a little while to find out what the problem was. The improved error message should make the problem clearer to anybody who may encounter it. I did try to refactor the code a little more, by extracting the whole part of:
        String name = moduleHolder.getName();
        if (namesToType.containsKey(name)) {
          Class<? extends NativeModule> existingNativeModule = namesToType.get(name);
          if (!moduleHolder.getCanOverrideExistingModule()) {
            throw new IllegalStateException(getModuleOverridingExceptionMessage(
              type.getSimpleName(),
              existingNativeModule.getSimpleName(),
              name
            ));
          }

          mModules.remove(existingNativeModule);
        }

        namesToType.put(name, type);
        mModules.put(type, moduleHolder);

out into a separate method since there were two places where nearly identical code was written.
Test Plan
I have built RN from source and used it in a very simple app with RN vector icons (the package creates a native module as can be seen here).
After including the module twice, ie.
        @Override
        protected List<ReactPackage> getPackages() {
            return Arrays.<ReactPackage>asList(
                    new MainReactPackage(),
                    new VectorIconsPackage(),
                    new VectorIconsPackage()
            );
        }

I get the improved error description, as seen in the screenshot.

[ANDROID] [MINOR] [NativeModuleRegistryBuilder] - Improve error message and refactor putting native modules to module maps

@hramos this PR should be good to merge

@hramos sorry to tag you again; this PR is ready to be merged imho

Hey @vonovak - I don't see any reviews on this PR yet. Let me know once it has been accepted and I can assist with the merge. Thanks!

@AaaChiuuu hello, you seem to have authored much of the code around this PR, do you think you could review it? Thank you!

just to illustrate this can be useful, because people do run into this problem: https://github.com/rebeccahughes/react-native-device-info/issues/243#issuecomment-336739204

Good catch @janicduplessis, I implemented the requested changes. Thanks for review.

@facebook-github-bot shipit


Creating multiple resources using the API

Describe the bug
{
"type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
"title": "One or more validation errors occurred.",
"status": 400,
"traceId": "00-26b70131132fac2aeb0061ff19e00c27-461a02b87ac1001c-00",
"errors": {
"$": [
"'c' is an invalid start of a value. Path: $ | LineNumber: 0 | BytePositionInLine: 0."
],
"request": [
"The request field is required."
]
}
}
To Reproduce
Steps to reproduce the behavior:
Create a request in this form:
curl -X 'POST'
'https://aznamingtool-sbsa.azurewebsites.net/api/ResourceNamingRequests/RequestName'
-H 'accept: /'
-H 'APIKey: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
-H 'Content-Type: application/json'
-d '[
{
"ResourceType":"nw",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessUnit":"marketing",
"ResourceRegions":"san"
}
},
{
"customComponents":{
"ResourceType":"kv",
"ResourceEnvironments":"nonprod",
"BusinessSegment":"marketing",
"ResourceRegions":"san",
"ResourceIdentifier":"infra"
}
},
{
"ResourceType":"rtt",
"customComponents":{
"ResourceLocations":"san",
"BusinessSegment":"marketing",
"ResourceEnvironments":"nonprod",
"ResourceIdentifier":"default"
}
},
{
"ResourceType":"as",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessSegment":"marketing",
"ResourceRegions":"san"
}
},
{
"ResourceType":"vnet",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessSegment":"marketing",
"ResourceRegions":"san"
}
},
{
"ResourceType":"rt",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessUnit":"marketing",
"ResourceRegions":"san",
"ResourceIdentifier":"default"
}
},
{
"ResourceType":"subn",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessUnit":"marketing",
"ResourceIdentifier":"db"
}
},
{
"ResourceType":"nsg",
"customComponents":{
"ResourceEnvironments":"nonprod",
"BusinessUnit":"marketing",
"ResourceRegions":"san",
"ResourceTypeSubset":"subn",
"ResourceIdentifier":"db"
}
}
]'
Expected behavior
Multiple names to be create for an example:
nsg-san-marketing-nonprod-subn-db
Screenshots
If applicable, add screenshots to help explain your problem.
Installation Method
A description of the installation environment for the Azure Naming Tool.
Additional context
I am trying to create multiple resources using this tool, I have managed to create one but I am wondering if I can have a single PS script that can generate all names.
The script below does what I want perfectly.
Azure Naming Tool API URL
$apiUrl = "https://aznamingtool-sbsa.azurewebsites.net/api/ResourceNamingRequests/RequestName"
$keyVaultName = "kv-san-nonprod-poc"
$secretName = "APIKey"
Retrieve the API key from Key Vault
$apiKey = (Get-AzKeyVaultSecret -VaultName $keyVaultName -Name $secretName).SecretValueText
Request Body
$requestBody = @{
ResourceType = "nsg"
customComponents = {
ResourceRegions = "san"
BusinessUnit = "marketing"
ResourceEnvironments = "nonprod"
ResourceTypeSubset = "subn"
ResourceIdentifier = "db"
}
}
Convert the request body to JSON
$jsonBody = $requestBody | ConvertTo-Json
Make the API request with the API key in the header
$headers = @{
"APIKey" = $apiKey
"Content-Type" = "application/json"
}
$NSGresponse = Invoke-RestMethod -Uri $apiUrl -Headers $headers -Method Post -Body $jsonBody
Output the response from the API
$NSGresponse
Extract the NSG name from the API response
$NSGname = $NSGresponse.name  # Replace 'name' with the actual JSON key for the NSG name
Output the NSG name
Write-Host "The NSG name is: $NSGname"

Hi @blaqvikin,
Thank you for your feedback. The current API only supports a single resource name creation action. We will add the ability to create multiple resource names in a single request to the backlog.

Bryan

Ok, now I know, I was banging my head on this one, thanks @BryanSoltis , I will keep an eye on the product for new updates.


unit test:s3test random path busy to produce data and timeout

Describe the bug
s3test random path busy to produce data and timeout
Environment
docker unit test
To Reproduce
python s3 script run failed always

#1517


--watch don't work with git bash on Windows and files as list of entires

Bug report

What is the current behavior?
running webpack --watch from git bash din't respond to file changes. the same happen running:
npm run webpack:watch

and
./node_modules/.bin/webpack --watch

the same running from cmd.exe works fine.
node builtin fs.watch works fine when I run example code in git bash, so the problem is in config and in webpack, not in git bash like in #2949
this code works (it also works for directories, but for unknown reason it execute twice):
var fs = require('fs');
var fname = 'C:\\projects\\idaapp\\inst\\shiny\\www\\js\\ga.js';
fs.watch(fname, { encoding: 'buffer' }, (eventType, filename) => {
  if (filename) {
    console.log(filename);
  }
});

I've check the source code and found this project: webpack/watchpack
this code works in git bash:
var fname = 'C:\\projects\\idaapp\\inst\\shiny\\www\\js\\ga.js';

var Watchpack = require("watchpack");

var wp = new Watchpack({});

wp.watch([fname], [], Date.now() - 10000);

wp.on("change", function(filePath, mtime) {
    console.log(filePath);
});

and gives single console unlike fs.watch.
If the current behavior is a bug, please provide the steps to reproduce.
I think that the problem is list of entries instead of require or import in files with single entry point
module.exports = {
  mode: 'production',
  entry: {
    'style': [
      path.resolve(__dirname, 'inst/shiny/www/css/analysisTab.css'),
      ..
   'other files': []

Here is my whole config file (sorry it's complicated, it's simple app with 4 files but webpack was giving me hard times and it was not working until it looks like this).
https://gist.github.com/jcubic/216e56027d5ad1010f47aae267ac3228

What is the expected behavior?
I expect that if I change one file that's inside entry then webpack will rebuild that entry when I run webpack --watch from git bash on Windows.

Other relevant information:
webpack version:
"webpack": "^4.17.1",
"webpack-cli": "^3.1.0"

Node.js version: v10.9.0 installed using nvm from older version
Operating System: Windows 7
Additional tools: (package.json included in gist).

Sorry for this issue, it seems that it work but there is longer delay then in cmd.exe

in your package.json add --watch-poll
"scripts": {
    "dev": "webpack -- --mode development --watch --watch-poll"
  },


i want to play this but the repo for cindy (a library) is gone

plz fix, i'm bored

The itch version should still work: https://bepvte.itch.io/meowrio


Wrong Value in $args in a Delete Method Handler

Can someone confirm whether or not this is a Bug or correct behaviour? Perhaps, my routes are not being defined correctly, but my other routes using GET, POST and PUT are working as expected.
I'm using Slim4 Beta and have a route defined demousers/{id} which handles the http-method DELETE
In my Handler Class
public function deleteDemouser(Request $request, Response $response, $args): Response
In the parameter $args I was expecting the value given in the UR, e.g. "id" => 5. This is what happens with other Http-Methods (GET, PUT and POST). Instead $args has the Key/Value pair "id" => "delete"
For routing I'm using Slim\Middleware\RoutingMiddleware Also  Slim\Middleware\MethodOverrideMiddleware is being used to convert the Html POST request to a DELETE request.
With XDebug I was able to locate the Method where the data seems to be mixed up. \FastRoute\Dispatcher\GroupCountBased::dispatchVariableRoute

Why are you using the beta? Please update to 4.2 and then try to see if the issue persists.

I've upgraded to Slim 4.2
For routing I am now using
$app->addRoutingMiddleware();
The same problem persists

We are going to need to see a gist of your code.

When I went to create the Gist, I discoverd the problem was in my code.
Problem solved (was never there in the first place)
Sorry


Duplicated section heading on Production Checklist page

There are two headings on https://www.cockroachlabs.com/docs/stable/recommended-production-settings.html titled "Basic Recommendations", so it's impossible to link to the second one. If a user clicks on the link-ify button next to the second such section, they'll be given a link that actually directs people to the first one: https://www.cockroachlabs.com/docs/stable/recommended-production-settings.html#basic-recommendations

@a-robinson, thanks for catching this. I suspect that we'll keep adding to this content, so do you think it would make sense to break each section of the "Production Checklist" into its own page? It could be a collapsible section in the sidenav. Or do you think users would prefer to have everything on a single page?

I like it as one page at it's current size, but don't feel very strongly about it.
If you do want to keep it as one page, it seems that renaming the offending sections to "Basic Topology Recommendations" and "Basic Hardware Recommendations" wouldn't be too ugly.


iOS CoreML-model convert report

Hi.
Lightweight-OpenPose (checkpoint_iter_370000.pth) could be converted to a CoreML-model.
I did well thanks to your research results.
This is a report.
Can Lightweight-OpenPose Pytorch-Model be used with SwiftOpenPose?

Thanks, really great to see it!


MVP - making mosaic public

exposing mosaic to public development, trying to commit to 01.11.2020:
Primary Goals

[x] running raw mode application like bandwhich in mosaic
[x] scrolling inside a pane
[x] IPC bus
[x] fix borders
[x] possibility to close a pane

Secondary goals:

[x] github action
[x] WIP Readme.

[x] Roadmap
[x] Plugin research goals

[x] Org Icon - Done!

We went public, yay!


Google Analytics cookie domain hardcoded

It appears that the Google Analytics (GA) cookie domain is hardcoded to umn.edu (app/views/shared_analytics.html.erb). This may cause issues with session tracking or other features of GA when hosted in other domains. Can the cookie domain be made a configuration option?

Yup.  We will address this in the next release.


"No remotes could be found." on version 11.3+ with Remote SSH

GitLens Version: 11.3+
Git Version: git version 2.17.1
VSCode Version: 1.55.1
OS Version: Windows 10 20H2 + Remote SSH Ubuntu 18.04.4 LTS

Version 11.2.1 works as expected.

Please enable output channel logging by running from GitLens: Enable Debug Logging from the command palette (ctrl+shift+p or cmd+shift+p). This will enable logging to the GitLens and GitLens (Git) channel in the Output pane.
Once enabled, please reproduce the issue, and attach the log from the both GitLens and GitLens (Git) channels.

Thanks for the instruction, here's log in the GitLens channel.

GitLens
[2021-04-17 11:34:00:483] GitLens (v11.4.1): syncedVersion=11.4.1, localVersion=11.4.1, previousVersion=11.4.1, gitlens:views:welcome:visible=undefined, gitlens:views:updates:visible=false
[2021-04-17 11:34:00:484] [1] GitService.initialize
[2021-04-17 11:34:00:484] [2] GitService.getBuiltInGitApi
[2021-04-17 11:34:00:498] [2] GitService.getBuiltInGitApi completed • 14 ms
[2021-04-17 11:34:00:532] Git found: 2.17.1 @ /usr/bin/git • 33 ms
[2021-04-17 11:34:00:532] [1] GitService.initialize completed • 47 ms
[2021-04-17 11:34:00:534] Starting repository search in 1 folders
[2021-04-17 11:34:00:536] [3] LineAnnotationController.resume
[2021-04-17 11:34:00:536] [4] GitLineTracker.start
[2021-04-17 11:34:00:536] [4] GitLineTracker.start completed • 0 ms
[2021-04-17 11:34:00:536] [3] LineAnnotationController.resume completed • 0 ms
[2021-04-17 11:34:00:536] [5] LineAnnotationController.refresh
[2021-04-17 11:34:00:537] [6] GitLineTracker.stop
[2021-04-17 11:34:00:537] [6] GitLineTracker.stop completed • 0 ms
[2021-04-17 11:34:00:537] [7] GitLineTracker.start
[2021-04-17 11:34:00:537] [7] GitLineTracker.start completed • 0 ms
[2021-04-17 11:34:00:553] GitLens (v11.4.1) activated • 71 ms
[2021-04-17 11:34:00:554] [5] LineAnnotationController.refresh completed • 17 ms
[2021-04-17 11:34:10:027] [8] GitService.getOrderedRepositories
[2021-04-17 11:34:10:027] [9] GitService.getRepositories
[2021-04-17 11:34:10:028] [9] GitService.getRepositories completed • 0 ms
[2021-04-17 11:34:10:028] [8] GitService.getOrderedRepositories completed • 0 ms
[2021-04-17 11:34:13:751] [a] GitService.resetCaches — branches, remotes
[2021-04-17 11:34:13:751] [b] GitService.getRepositories
[2021-04-17 11:34:13:751] [b] GitService.getRepositories completed • 0 ms
[2021-04-17 11:34:13:751] [c] GitService.getRepositories
[2021-04-17 11:34:13:751] [c] GitService.getRepositories completed • 0 ms
[2021-04-17 11:34:13:751] [a] GitService.resetCaches completed • 0 ms
[2021-04-17 11:34:13:751] [d] RemotesView.refresh — true
[2021-04-17 11:34:13:752] [e] RemotesViewNode.refresh — true
[2021-04-17 11:34:13:752] [e] RemotesViewNode.refresh completed • 0 ms
[2021-04-17 11:34:13:752] [f] RemotesView.triggerNodeChange
[2021-04-17 11:34:13:752] [f] RemotesView.triggerNodeChange completed • 0 ms
[2021-04-17 11:34:13:752] [d] RemotesView.refresh completed • 0 ms
[2021-04-17 11:34:13:801] [10] GitService.getOrderedRepositories
[2021-04-17 11:34:13:801] [11] GitService.getRepositories
[2021-04-17 11:34:13:801] [11] GitService.getRepositories completed • 0 ms
[2021-04-17 11:34:13:801] [10] GitService.getOrderedRepositories completed • 0 ms

However I am not seeing GitLens (Git) channel:

BTW git remote --verbose writes the content below:
origin  git@github.com:makeflow/boring-router.git (fetch)
origin  git@github.com:makeflow/boring-router.git (push)

@eamodio gentle ping

@github-actions [magic] remove inactive tag.

@vilic Sorry for the long delay (I've been swamped lately -- and continue to be).
Can you ensure that you have "gitlens.outputLevel": "debug", set in your settings.json? The GitLens (Git) channel should exist if that setting is set to debug. Can you see if it shows up after a restart?

@eamodio Yes, but here's something new. After opening a file in the project, GitLens then loads the remotes successfully and the GitLens (Git) channel appears.
[2021-05-31 02:26:44:396] [/root/workspace/boring-router] git rev-parse --show-toplevel • 59 ms
[2021-05-31 02:26:44:469] [/root/workspace/boring-router] git ls-files -- /root/workspace/boring-router/CHANGELOG.md • 45 ms
[2021-05-31 02:26:44:513] [/root/workspace/boring-router] git remote -v • 42 ms
[2021-05-31 02:26:44:721] [/root/workspace/boring-router] git blame --root --incremental -- CHANGELOG.md • 107 ms
[2021-05-31 02:26:44:750] [/root/workspace/boring-router] git config --get-regex ^user\. • 28 ms
[2021-05-31 02:26:44:773] [/root/workspace/boring-router] git check-mailmap placeholder<EMAIL_ADDRESS>• 21 ms
[2021-05-31 02:26:58:551] [/root/workspace/boring-router] git for-each-ref --format=%3ch%3e%(HEAD)%3cn%3e%(refname)%3cu%3e%(upstream:short)%3ct%3e%(upstream:track)%3cr%3e%(objectname)%3cd%3e%(committerdate:iso8601) refs/heads refs/remotes • 23 ms

Unfortunately, I'm not sure what is going on here and I can't reproduce an issue with a similar remote. Can you or someone in the community try debugging this to see what is going on? Thanks!


[HUDI-2451] on windows client with hdfs server for wrong file seperator

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
The problem it seems has been repaired and merge in  HUD-1420
This issue was fixed in HUD-1420, but seems to have been accidentally overwritten by hud-1486 and is still happening path：/apps/hive/warehouse/dw_t1.db/test_hudi_table/.hoodie.temp/20210918131318
I changed some of the overwritten code to keep my program running, to see if it helps HUDi
https://issues.apache.org/jira/browse/HUDI-2451

(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

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

@hudi-bot run azure


Updating to new Stripe Checkout

Hi, Wanted to ask if you are planning on updating this example to use the new Stripe Checkout as its a lot more versatile.

Hey,
Yeah I think this would be a great idea!
I don't have time right now to put to this, but would be more than open to a PR.
Cheers


disable termistore reading

hello how to disable thermistor reading !

I don't think I understand your question. Why, and under what conditions, do you want to avoid reading the thermistor?
By design, a thermistor is always associated with a hotend. So you cannot disable the thermistor without also disabling the hotend. This is a safety measure, as dumping power into a heating block without having a temperature reading/feedback system is not smart.
However, you can disable both the hotend and the thermistor by editing your config file. For example, you could remove the hotend/thermistor from the firepickdelta machine by making the following change to lines 591-617 of https://github.com/Wallacoloo/printipi/blob/master/src/machines/rpi/firepickdelta.h:
inline std::tuple<Fan, TempControl<RCThermistor2Pin, PID, LowPassFilter> > getIoDrivers() const {
            return std::make_tuple(
                Fan(IoPin(PIN_FAN_INVERSIONS, PIN_FAN, PIN_FAN_PULL), PIN_FAN_DEFAULT_STATE, FAN_PWM_MULTIPLIER, FAN_IDEAL_PWM_PERIOD),
                TempControl<RCThermistor2Pin, PID, LowPassFilter>(
                    iodrv::HotendType,
                    IoPin(PIN_HOTEND_INVERSIONS, PIN_HOTEND),
                    RCThermistor2Pin(
                        IoPin(NO_INVERSIONS, PIN_THERMISTOR),
                        IoPin(NO_INVERSIONS, PIN_THERMISTOR_CHARGE),
                        THERM_RCHARGE_OHMS,
                        THERM_RSERIES_OHMS,
                        THERM_RUP_OHMS,
                        THERM_C_FARADS,
                        VCC_V,
                        THERM_V_TOGGLE_V,
                        THERM_T0_C,
                        THERM_R0_OHMS,
                        THERM_BETA),
                    PID(HOTEND_PID_P, HOTEND_PID_I, HOTEND_PID_D),
                    LowPassFilter(3.000),
                    HOTEND_IDEAL_PWM_PERIOD
                )
);
}

to
inline std::tuple<Fan> getIoDrivers() const {
            return std::make_tuple(
                Fan(IoPin(PIN_FAN_INVERSIONS, PIN_FAN, PIN_FAN_PULL), PIN_FAN_DEFAULT_STATE, FAN_PWM_MULTIPLIER, FAN_IDEAL_PWM_PERIOD)
);
}

But I doubt this is what you want to do, as you obviously cannot print anything without a hotend. Please clarify your problem.


fix: Sync typescript unlock condition types

Sync typescript unlock condition types.
Rust uses slot not slotIndex, you can see this for instance in TimelockUnlockConditionDto.

Please add a changelog entry

Done, did I do it correctly? :eyes: