VsCode extension fails install on OSX

Issue Description
When installing VsCode extension it reports "Failed to spawn 'dotnet --info'"; but, only for one specific user.  If I create a new user, create a dotnet console program, open application with vscode, follow the prompts to install extensions, then everything worked as expected.
Steps to Reproduce
Install the extension for a specific user.
Expected Behavior
Extension installs
Actual Behavior
Extension installation reports that it cannot spare  dotnet --info
Logs
OmniSharp log
Installing C# dependencies...
Platform: darwin, x86_64
Downloading package 'OmniSharp for OSX' (43667 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package 'OmniSharp for OSX'
Downloading package '.NET Core Debugger (macOS / x64)' (51526 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package '.NET Core Debugger (macOS / x64)'
Downloading package 'Razor Language Server (macOS / x64)' (57271 KB).................... Done!
Installing package 'Razor Language Server (macOS / x64)'
Finished
Failed to spawn 'dotnet --info'
C# log
Post the output from Output-->C# here
Environment information
VSCode version: 1.35.1
C# Extension: 1.20.0
OSX: 10.13.6
NodeJs: 8.11.1
dotnet:
.NET Core SDK (reflecting any global.json):
Version:   2.2.300
Commit:    73efd5bd87
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.13
OS Platform: Darwin
RID:         osx.10.13-x64
Base Path:   /usr/local/share/dotnet/sdk/2.2.300/
Host (useful for support):
Version: 2.2.5
Commit:  0a3c9209c0
.NET Core SDKs installed:
2.2.107 [/usr/local/share/dotnet/sdk]
2.2.300 [/usr/local/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.2.5 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.2.5 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.2.5 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Mono Information
        OmniSharp using built-in mono
Dotnet Information
A valid dotnet installation could not be found.
Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-vscode
1.20.0

HTMLHint
mkaufman
0.6.0

vscode-eslint
dbaeumer
1.9.0

Note that I have uninstalled all other extensions and had the same result for this user.
Running dotnet run from the Terminal window in VsCode works as expected.

@emarks It appears that you dont have dotnet on your path. Could you reinstall dotnet and check that your path variable has dotnet on it.

@akshita31 Yes, I checked that and it is in the path.  In fact, botnet --info from within the vscode terminal works fine.    I found that if I go into the directory with .csproj and run "code ." that it works all the time.   The install seems to be failing on line 85 of src/coreclr-debug/util.ts.    resulting in "Failed to spawn 'dotnet --info".  It's the spawn that is failing, not that it doesn't find dotnet.  I haven't had time to investigate it further.

@emarks Thank you for confirming that. We have a similar issue here https://github.com/OmniSharp/omnisharp-vscode/issues/2728. Did you recently install .NET? Can you restart your system and check again ?

Yes, I installed dotnet for the first time on this laptop within the last week.  I installed the C# extension immediately after validating that dotnet worked from the command line.
Restart made no difference.
Note that this is only happening with one user account on the laptop.  A second one worked fine.  I added a third just to test and it worked fine.  The effected user had already been using vscode for some time for C/C++ and NodeJS development.   I did uninstall all the extensions in that users vscode and installed only the C# extension and the problem persisted.

@emarks Did you figure this out? It sounds like the account that had problems might not have had dotnet on its PATH?

@emarks Did you figure this out? It sounds like the account that had problems might not have had dotnet on its PATH?

No, I was not able to get it to work with that account.  dotnet was in the PATH along and worked fine from the command line and from the vscode terminal view.  However, the extension could never find it.

I am going to close this issue since it isn't really about this extension, and this issue is rather old. Note that I am adding a help link to the error message with #3507, to hopefully make it easier to understand what is going on.


Provide other Threading strategy using Thread Pool, configurable per actor

It would be nice to allow one Actor to run on multiple threads, or does this break the concept?  You could implement this by defining a semaphore for each Actor like I did with ParallelIterable...

Correct, started an implementation of this. Can be useful in systems having lots of actors. Provide an "ActorPool" from which Actors can borrow threads. However the same Actor can never be active on multiple threads in parallel, but he can be active on Thread-1 now, and on Thread-2 in 10 seconds.

Cool! Looking forward to it! ;-)

Maybe you could also forsee a second attribute allowing you the customize the thread name prefix which is always handy when logging messages ;-)

How about @Actor(pool = CPU), @Actor(pool = IO), and @Actor(pool = SINGLE_THREAD) as a default?

Maybe rename it to poolStrategy? Values are fine! And add a second attribute named poolNumber to be able to fine tune it if you don't want to use predefined strategies? And not to forget threadNamePattern for logging purposes 😉

A configurableInterface would be nice to, in the situation that you want to set the value from a property file or spring context.


URL's not working

Hello,
the installation was fine, but every link (e.g. admin panel, about...) is ending in 404.

I had to set the URL Rewrite in IIS....
#problemsolved


fix: CSS class ordering in component rendering

🐛 Bug Report
When components have a Class attribute for specifying additional classes AND they have their own default class used as part of their styling, the custom class is listed first, and the default class is listed second. This means that it is not possible to override styles in the default class via another class - it must be done via style.
💻 Repro or Code Sample
<FluentBodyContent Class="custom-body-content">
    @Body
</FluentBodyContent>

.custom-body-content {
    margin-right: 10px;
}

🤔 Expected Behavior

😯 Current Behavior

💁 Possible Solution
It looks like the ordering would just need to be different in places where the CssBuilder is used, like this:
Current:
protected string? ClassValue => new CssBuilder(Class)
    .AddClass("body-content")
    .Build();

Changed:
protected string? ClassValue => new CssBuilder("body-content")
    .AddClass(Class)
    .Build();

🔦 Context
The current design makes it hard to avoid using inline styles.
A quick code search shows this impacts several components, but definitely not all of them (since many don't have a default style).

You're right. We'll change that as soon as possible.

That will be included in the next release, via #745

Thanks for the quick response!


feat: Support basicAttConstraints X.509v3 extension

This PR adds support for the basicAttConstraints X.509v3 extension defined in ITU-T Recommendation X.509 for usage in attribute certificates. This is different from, but similiar to the basicConstraints X.509v3 extension.
Checklist

[x] documentation is added or updated
[x] tests are added or updated

My PR is failing on style checks. I don't think it is possible for me to fix the styling problem: either I indent the arguments to those functions to column 61 and then the names overflow the 80-character column limit, or I indent them less to avoid it, which violates the rule. How do I proceed?

Alternatively, I could just put this in the same file as basicConstraints, since it uses the exact same syntax.

My PR is failing on style checks.

The style checking is new and somewhat experimental. Its currently coming up with quite a few false positives. Adding "style: waived"

All feedback addressed and ready for review. I am pretty sure the one check failure (other than the code style, I mean) is unrelated to my changes.

All seems good except there is a merge commit. Could you please rebase to drop it?

All feedback addressed and ready for review again. @nhorman would you be interested in reviewing this?

@mattcaswell would you have any interest in reviewing this one?

@t-j-h Would you be interested in reviewing this one, since it only needs one more approval?

@t8m It seems like a lot of people that are usually responsive are not. This can't be a coincidence. Is everybody busy with the OpenSSL issues cleanup?

@t8m It seems like a lot of people that are usually responsive are not. This can't be a coincidence. Is everybody busy with the OpenSSL issues cleanup?

More likely people are on vacations, etc.

@t8m @nhorman @FdaSilvaYY , all feedback addressed and this is ready for review again. (Just a few trivial style changes.)

Is this able to receive the approval: done label yet?

yep, has acks from myself and @t8m.  Not sure why the automation didn't flag it.

This pull request is ready to merge

pushed to master, thank you!


Client Allocation Request for: 上海童程文化传播有限公司（ Shanghai TongCheng culture communication co., LTD.）

Client Allocation Questions
Core Information

Name: 上海童程文化传播有限公司（ Shanghai TongCheng culture communication co., LTD.）
Website / Social Media: Wechat ID:gh_ae37b77400fe
Region: Greater China
DataCap Requested: 30TiB
Addresses to be Notarized: f13lwcalwguboqmquxqvyqqj22fiuyqvw3qucv4qa
Notary Requested: ozhtdong

@tanw2009 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Please provide your pubilc website. thanks!

There are three reasons for non-compliance:

There is no public website;
The data cannot be downloaded publicly;
The applied data capacity exceeds the standard.


Fix picking through gui panels

Bug fix
Fixed bug
The picking module was conflicting with egui and models behind the panel would be highlighted / selected when the user interacted with panels, making it almost impossible to interact with text boxes to change a model property (when clicking on it the model behind the panel would be selected, changing the panel itself).
Example video below:
https://user-images.githubusercontent.com/9111558/178931130-f894e92c-19bc-459b-bf1a-2827c6b1e58d.mp4
Fix applied
It seems we should also use is_pointer_over_area to figure out whether we are on top of a egui area or not, that helped but made the bug happen "sometimes" i.e. consecutive cargo run would behave differently.
I suspected it might be due to the fact that systems are executed in parallel in bevy unless clear dependencies are specified and my hypothesis is that since picking depends on egui being updated it should be executed afterwards.
I added an explicit order by running enable_picking after the egui update and it seems to be fine, but this kind of issues can be tricky to figure out and "it works on my machine" might not be the best metric :man_shrugging:
https://user-images.githubusercontent.com/9111558/178932355-6ae95d82-b6de-4c4c-9776-3a5ab72facad.mp4

Oops, same change is implemented in #42 :sweat_smile:, this can be closed if that is merged


Question for grid_coord

What is the meaning of the grid_coord variation? Thanks.

Hi, grid_coord = ⌊coord / grid_size⌋, also code here (https://github.com/Pointcept/Pointcept/blob/main/pointcept/datasets/transform.py#L810) might be helpful.


Dropbox API user limit reached

Tried to link my Dropbox account and reached this error saying that StackEdit has reached its user limit for the Dropbox API.

Tried to link my Dropbox account and reached this error saying that StackEdit has reached its user limit for the Dropbox API.

Tried to link my Dropbox account and reached this error saying that StackEdit has reached its user limit for the Dropbox API.

@anjigithub do you still have this error? Are you using stackedit.io or a custom instance?


[Cinder] Add reporting of max volume size

This patch adds the python-cinderclient as a requirement so that
the cinderbackend collector can pull the quota information from cinder.
The openstacksdk 0.61.0 release, which is the latest as of this patch,
does not support the quota functions for cinder.
This patch adds a new metric of cinder_per_volume_gigabytes, which is
the max volume size.

The openstacksdk doesn't have quota methods in the latest release.  This is why I had to do it this way.   The master branch of openstacksdk has it, but not the official release.
No quota_set.py here:
https://github.com/openstack/openstacksdk/tree/0.61.0/openstack/block_storage/v3
but it's here:
https://github.com/openstack/openstacksdk/tree/master/openstack/block_storage/v3

I got that part, I would just have called the API endpoint and took the JSON apart manually for that single key I would have been looking for. Shouldn't be more than 3 lines of code.


Update the footer design.

Describe the bug/feature

Want to add some some things to footer design..
To Reproduce

Expected behavior
A clear and concise description of what you expected to happen.
Logs
If applicable, add logs to help explain your problem (e.g. the error message and/or exception traceback).
Environment (please complete the following information):

OS:
Browser:

Additional context
Add any other context about the problem here.

Hi,I would like to work on this issue.

@tjanhvi go ahead.

okay :)
Please Mention the tag ...........

Hey Is This Still Open? If It is then Please Assign This Issue To Me Under LGMSoC'21!

Hey Is This Still Open? If It is then Please Assign This Issue To Me Under LGMSoC'21!

Hey, i'm doing this


Error loading up after upgrade to 0.43

Error loading up the app after upgrading to 0.43
After upgrading to 0.43 from 0.42.3 I get this error shown below. I have tried all suggestions from error message but none fixed the problem.
error: bundling: UnableToResolveError: Unable to resolve module react/lib/ReactComponentWithPureRenderMixin from ../app/node_modules/react-navigation/src/views/Header.js: Module does not exist in the module map or in these directories:
/app/node_modules/react/lib
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset packager cache: rm -fr $TMPDIR/react-* or npm start --reset-cache.
at p.catch.error (/app/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:366:19)
at process._tickCallback (internal/process/next_tick.js:109:7)
Bundling index.android.js  99.2% (509/511), failed.

Please note that I have tried this in both Ubuntu 16.10 and Windows 10.
This is how my package.json changed:

"react": "15.4.2",
"react-native": "0.42.3",

"react": "^16.0.0-alpha.6",
"react-native": "^0.43.0",

Reproduction Steps and Sample Code
I had NO conflicts during upgrade
I followed the suggested way to upgrade:

npm install -g react-native-git-upgrade
react-native-git-upgrade
react-native start
react-native run-android

Solution
I don't have a solution. I have navigated in the folder in question and I can see that Header.js exists.
The only thing that I can think of is creating a new app and moving over my src/ to the new project. Obviously I don't want to have to do that every time there is a new version.
Additional Information

React Native version: [^0.43.0]
Platform: [Android]
Development Operating System: [Ubuntu 16.10, Windows 10]
Dev tools: [IntelliJ IDEA Ultimate 2017.1, VSCode, Android SDK 23]

This problem is due to the react version        use   "react": "16.0.0-alpha.3",  is fine

I don't understand your comment, how is that fine? Effectively the upgrade path to the newer version is broken. Is there a suggested solution?

@sherlock221 and how do u force it to use a older version, when using react-native init [project name] ?

@MacVej I don't think you can, just change the version in package.json after initing. Hilarious that there's no documentation for this...

Same issue here, change to "16.0.0-alpha.3" is fine, but i don't think this is good solution.  So, we hope facebook developer can be fix this bug, thanks!

So changing the version to that and
rm -rf nodes_modules/ && npm install
is the suggested solution? I will try to verify tomorrow.
Indeed that's not very viable but good enough for now.

@Kennytian same here, and agreed

In order to use the alpha version, you need to run:
rm -rf node_modules && npm install
Clean ios/android build folder:
rm -rf android/build && rm -rf ios/build
After that, just run as usual:
react-native run-ios or react-native run-android

It seems odd to me that 0.43, a stable version of react-native, depends on a potentially unstable alpha version of react

Agree. And some important packages like react-native-router-flux and react-native-maps  use the latest stable React 15.4.2 and now incompatible with RN 0.43.1.

https://github.com/react-community/react-navigation/issues/923

@mikehobi 0.43 is not a stable version unfortunately as it's still version 0

@joemckie True, but:

Is there any official word on this? I upgraded to 43 to get past done 42 bugs, and now storyshots broken.
It seems like a pretty bad idea to upgrade to an alpha version of react. I think maybe I'll need to go to 41 instead.

react": "16.0.0-alpha.3"  works for me .I have be under the problem for serval days!thank you! @sherlock221

👍  on using a stable version

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


notifyMonthChanged does not work

Property months: List<CalendarMonth> in CalendarAdapter contains CalendarMonth instances which all have yearMonth: YearMonth property set to YearMonth.of(2000, 2) equivalent so getAdapterPosition(month) in method reloadMonth returns -1 for any YearMonth instance passed to notifyMonthChanged method.
This is calendar setup:
val today = YearMonth.now()
calendar.setup(
    today.minusYears(20),
    today.plusYears(20),
    DayOfWeek.MONDAY
)
calendar.scrollToDate(LocalDate.now())

 <com.kizitonwose.calendarview.CalendarView
     android:id="@+id/calendar_weekly"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
     android:background="?colorSurface"
     app:cv_dayViewResource="@layout/item_calendar_day_weekly"
     app:cv_hasBoundaries="false"
     app:cv_inDateStyle="allMonths"
     app:cv_maxRowCount="1"
     app:cv_orientation="horizontal"
     app:cv_outDateStyle="endOfRow"
     app:cv_scrollMode="paged"
     app:layout_constraintEnd_toEndOf="parent"
     app:layout_constraintStart_toStartOf="parent"
     app:layout_constraintTop_toTopOf="parent" />

This is because app:cv_hasBoundaries is set to false. The value for YearMonth  becomes somewhat undefined when the calendar has no boundaries. You need to set this to true. However, if you must use false, then you have to rely on notifyDateChanged instead.

Thanks!


Warn about non-explicit dependencies?

First, thanks for this cool project, it is so great to be able to fully nixify Purescript projects in such an easy way!
In contrast to e.g. spago, purs-nix does not display a warning if one is importing modules from packages that are not explicitly declared as a dependencies. Making those explicit helps e.g. when accidentally using an undeclared transitive dependency that might suddenly "vanish" when it is no longer used by one of the declared dependencies.

I'm glad you like it! I definitely understand the desire to have this feature, I would like it as well. Unfortunately I haven't been able to come up with a satisfactory way to do it yet. I am currently having a look at #42.


Support conditional requests/transactions

see https://firebase.google.com/docs/database/rest/save-data#section-conditional-requests

So this didn't get in?

Instead of adding suport for conditional requests you could provide access to the api client and http client of a reference. In that case we could use the client to manually call the correct endpoints with the correct headers.

@EECOLOR 4.21 is out and includes transactions/conditional requests 🎉


add typings

include typescript definitions

@mikolalysenko : let me know if you have questions
Best regards

@rreusser Good point!  I come back to you with a fix ASAP.
Best regards

Because https://github.com/scijs/ndarray/blob/master/ndarray.js is the only main file makes up this repository, then how about rewriting this javascript program into typescript completely? There could be no needs to maintain some snips for different module system or different es versions, like the practice in rxjs

When the PR will be ready?

@dmromanov  I saw there is already here, https://www.npmjs.com/package/@types/ndarray.


Is there any documents about how to custom models?

I find some custom model structs such as UserDefinedEmbeddingModel in the library, however i cloud not find any documents about them.

Hey. We're lacking docs there.
You can refer to the tests to see the usage.
https://github.com/Anush008/fastembed-rs/blob/main/tests%2Fembeddings.rs#L197


Clarify point in time restore geo-redundant backup

Point in time restore can be done in another server in a paired region so long as geo-redundant backup is selected.  This should be corrected to the effect of suggested changes, you can run a restore only on the same server => this inaccuracy slows down the process of restoration from customers who were impacted by recent outages.

@SudhirRaparla
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Can you please explain what you intend to change in the documentation?

In light of the recent outage we noted the following.

Database is set with PITR with geo-redundant backup storage

Database server was affected and therefore need to restore in a different region
We spun up a new database server in a paired region and created a new database from an existing PITR backup -- see image below

As such, the above statement you can run a restore only on the same server is incorrect, please re-clarify and correct this statement, this PR is just a suggestion.

2. geo-Restore can restore the backup to any region, it doesnt have to be paired region.

Happy to mod the PR to remove the paired part. Just need to agree if a correction is needed.

This section is meant for Point-in-time restore and we have a separate section for Geo-Restore in the document below
@SudhirRaparla I agree there's a section for the geo-restore in the same document, but the earlier statement using the "only" meant that geo-restore is not possible which contradict the later section that Point in time restore with geo-redundant backup storage means that you can restore PITR in another region.

You can perform Point-in-time restore only on hte same server. Cross-server and Cross subscription Point-in-time restore is not supported yet.

Can we remove the statement  only and refer to the Geo-Restore section for more info so early conclusions won't be made with regards to restore PITR in a different region?
Something like => You can perform point-in-time restore on the same server. Cross-server and Cross subscription Point-in-time restore is not supported yet. Refer to the Geo-restore section to restore to a different region for databases set with geo-redundant backup storage.

@SudhirRaparla made the changes - please review/suggest additional mods if needed.

@vicperdana - Cross-Server, Cross-Subscription and Cross-Geo PITR is not supported. With geo-restore you can restore a database to any region using only the last replicated backup(to the paired region). Created a new PR to address this to resolve conflicts: https://github.com/MicrosoftDocs/sql-docs-pr/pull/26893
Please check if the changes in above PR help clarify PITR capabilities.

Hi @vicperdana , thanks for your submission! We've processed your changes in a private repository and they'll be live later this week.
I'll go ahead and close out this PR but please feel free to comment should you need anything further.
Thanks!
Masha from the SQL Docs team


Can you add a way to set extra headers in url?

Webview_flutter package has a way to set headers in load url parameter just like this Controller.loadUrl('https://xxxx.com', headers: header);. I wonder if this can be added here it can help bypassing url a lots

hey any update on this??

I actually just use webview2 c++ manually for my work's case, Though I think I need to reopen this issue.

Hi ...
We need this too...

I hope this issue will have update early


PyPI, travis, test coverage, and dockerfile

Current coverage is 28.24%

Branch #1 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

:+1: looks good to me, let's wait for @r-marques feedback.

since we are at it lets use semantic versioning like we do with bigchaindb and tag the first release as v0.1.0

yes, thanks

I will close this PR so that PR #2 with branch 0.1.0 replaces it.


[Feature request] Download background image for artist

The logic would be the same as the folder picture.
Background image is foundable in member web page, in it's head.
Example: view-source:https://www.pixiv.net/member.php?id=198128
    <style>
    body {
        background: #EDF0F5 url('https://i.pximg.net/background/img/2015/10/25/08/45/27/198128_77ddf78cdb162e3d1c0d5134af185813.jpg') no-repeat fixed;
    }
    </style>

try https://github.com/Nandaka/PixivUtil2/releases/tag/v20170726

It works well, thanks!

With the new redesign, it looks like it has changed.
I found an artist with the new header (ex-background) working as an example: https://www.pixiv.net/member.php?id=8021957
The json is different, but still indicate 'background' (for now):
user: { 8021957: {"userId":"8021957","name":"yuzu-aki","image":"https://i.pximg.net/user-profile/img/2018/09/17/11/23/44/14786252_df3649c03d98c4d5dd655c5ef57f6f97_50.jpg","imageBig":"https://i.pximg.net/user-profile/img/2018/09/17/11/23/44/14786252_df3649c03d98c4d5dd655c5ef57f6f97_170.jpg","premium":false,"isFollowed":false,"isMypixiv":false,"background":{"repeat":null,"color":null,"url":"https://i.pximg.net/c/1920x960_80_a2_g5/background/img/2018/10/02/14/04/52/8021957_2328e2818081e71c25b3aabd82f23c48.jpg","isPrivate":false}


Add FTP server for users

Add an FTP server that users can log in to and administer their servers.

If security is concerned, perhaps consider WebDAV over HTTPS.


Update @solana/web3.js

Closing because this change is causing weird errors and so long as the ^ is next to version number in package.json it shouldn't affect people's use of peer dependencies who want to use more up to date versions..


Required Fields

JSON Schema has the ability to mark fields as required. This module supports this via the @JsonProperty(required = true) field annotation.
This is not working for me.  My object has the following field:
@XmlAttribute(name = "clientRequestId", required = true)
protected String clientRequestId;
and the generated output is :
"clientRequestId" : {
"type" : "string"
},
I'm using
	<dependency>
		<groupId>com.fasterxml.jackson.module</groupId>
		<artifactId>jackson-module-jsonSchema</artifactId>
		<version>2.5.1</version>
	</dependency>

Have you registered JAXB module? Otherwise Jackson does not make use of JAXB annotations, and can not propagate information (since it does not detect it) to schema generator.

Thanks, that fixed the issue.


Fix for AllowUnencrypted

fixed warning for new rspec

Fixes #26
With allow unencrypted first request comes encrypted and then next requests come unencrypted

:+1:  Thanks for your PR.

Thank you!  https://msdn.microsoft.com/en-us/library/Cc251576.aspx seems to agree.  It might be cleaner to check the protocol headers but I don't know the exact response that comes back.  If the multipart response is marked multipart/encrypted;protocol="application/HTTP-SPNEGO-session-encrypted";boundary="Encrypted Boundary" even though the body is unencrypted, then this may be the only clean way out.


Misc doc changes

Besides other documentation changes, this commit ensures the generated
HTML doc for HexDocs.pm will become the main source doc for this Elixir
library which leverage on latest ExDoc features.
Screenshot:

SourceLevel has finished reviewing this Pull Request and has found:

1 possible new issue (including those that may have been commented here).

See more details about this review.


Feature: apply align option for table

Similar to this PR: https://github.com/IonicaBizau/json2md/pull/50
But I add more flexibility to set align by column.
{
  headers: ["a", "b", "c", "d"],
  rows: ["col1", "col2", "col3", "col4"],
  aligns: ["", "center", "left", "right"],
}

Can you please write some tests for this? Thanks!

Sure, I add a test case for the table align.
And I found some test cases are broken. I believe the code is correct and the test assert is not. So I also update those items.


README: fixing typo

Codecov Report

Merging #1852 into development will decrease coverage by 0.17%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development    #1852      +/-   ##
===============================================
- Coverage        85.95%   85.78%   -0.18%
===============================================
  Files               78       70       -8
  Lines             2101     2075      -26
  Branches           273      273
===============================================
- Hits              1806     1780      -26
  Misses             202      202
  Partials            93       93

Impacted Files
Coverage Δ

src/utils/index.ts

src/carousel/index.ts

src/collapse/index.ts

src/popover/index.ts

src/tooltip/index.ts

src/alert/index.ts

src/positioning/index.ts

src/sortable/index.ts

src/component-loader/index.ts

demo/src/app/api-docs/index.ts
100% <0%> (ø)

Continue to review full report at Codecov.

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

Replaced with to enable ....
thanks for PR!


Powershell commands to Reset the Remote Desktop Services configuration

Article: https://docs.microsoft.com/en-us/troubleshoot/azure/virtual-machines/reset-rdp#reset-the-remote-desktop-services-configuration-1
The command will not work on any VM if the Microsoft.Compute.VMAccessAgent is not installed on the VM.
By default this agent is not installed on any windows VM, so the documentation should make clear that you will need the extension installed before.
Furthermore even with the extension installed if you run the command it will result on one error:
COMMAND:
Set-AzVMAccessExtension -ResourceGroupName "myResoureGroup" -VMName "myVM" -Name "myVMAccess" -Location WestUS -typeHandlerVersion "2.0" -ForceRerun
ERROR:
Set-AzVMAccessExtension: Missing an argument for parameter 'ForceRerun'. Specify a parameter of type 'System.String' and try again.
So the documentation is not correct for this command.

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

ID: 9bcca963-7ced-fe06-6b20-51984aa31ea1
Version Independent ID: c2c4516a-5f37-4dc3-71a9-f93abfb0465a
Content: Reset Remote Desktop Services or its administrator password in a Windows VM - Virtual Machines
Content Source: support/azure/virtual-machines/reset-rdp.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@joteromontes, Thanks for the comments. I agreed with you. We should add the information about the prerequisite for running the command. Will update the article,.

@joteromontes The Azure VM agent is required to run the command. When you run the Set-AzVMAccessExtension command that provided in the article, the Microsoft.Compute.VMAccessAgent will be install automatically.
We need to specify $true for parameter 'ForceRerun'.
Set-AzVMAccessExtension -ResourceGroupName "myResoureGroup" -VMName "myVM" -Name "myVMAccess" -Location WestUS -typeHandlerVersion "2.0" -ForceRerun $true


If kaleidoscope running and a message comes in which gets a NAK do not stop kaleidoscope

Need to compare incoming message to valid commands and not POP/Return if it is invalid.
Refactor command validation logic to a new subroutine.

how to keep kaleidoscope running when server disconnects from the client?


Move print_err macro to a common file

In order to avoid duplicating macros like that, you can simply put them in a module that gets added to your root first, and has a #[macro_use] annotation.

I signed it!

Thanks!


Sorting by Stake Weight is not supported

BALANCE_DESC is in query:
{"variables":{"limit":100,"sort_by":"BALANCE_DESC","query":{"epoch":0}},"query":"query StakingLedgersQuery(\n  $limit: Int = 10\n  $sort_by: StakeSortByInput!\n  $query: StakeQueryInput!\n) {\n  stakes(limit: $limit, sortBy: $sort_by, query: $query) {\n    balance\n    chainId\n    delegate\n    epoch\n    ledgerHash\n    nonce\n    pk\n    public_key\n    token\n    voting_for\n    delegationTotals {\n      totalStakePercentage\n      countDelegates\n      totalDelegated\n    }\n  }\n}\n","operationName":"StakingLedgersQuery"}

Result:

It would seem the "stake" column is mislabeled:

I think the STAKE column is mislabeled. This is the balance associated for that PK and not the stake. The stake can be queried like so:
{
stakes(query:{epoch: 77}) {
delegationTotals {
totalDelegated
}
}
}

Assigning to @trevorbernard as he has indicated this work might be in progress in some branch.

@trevorbernard api.minasearch.com is not out of date.

@n1tranquilla Please confirm that this is supported in the indexer (now that it is correctly deployed).

This is now supported in the indexer. The implementation of this will be in #501


Remove custom platforms setting

With bazel 7.x and rules changes we can use the native platforms instead

depends on https://github.com/maplibre/maplibre-native/pull/1956


Error:  attribute height: A negative value is not valid. ("-348")

Expected Behavior
A height and width are being set on my chart container. I expect no issues in the console.
Current Behavior
I'm seeing the following error in the console, even though the boundingClientRect has an assigned width and height. Please see screenshot.
Possible Solution
I've tried reordering my data ( ordering by both ascending date and ascending stack order ), removing higher-valued data points that might edge out smaller ones ( possibly generating the rendering error ), filling in stack gaps with zeroed data to make everything homogenous, updating Britecharts to 2.13 from 2.12, and forcing height and width with CSS and chart configuration. I cannot get the errors to disappear.
Steps to Reproduce
Providing the following data set to a stackedBarChart creates the error seen in the screenshot.
[{name: "Test1", date: "12/01", value: 9378, stack: 1}, {name: "Test2", date: "12/01", value: 135357, stack: 2}, {name: "Test3", date: "12/01", value:<PHONE_NUMBER>7, stack: 3}, {name: "Test4", date: "12/01", value:<PHONE_NUMBER>, stack: 4}, {name: "Test5", date: "12/01", value: 44731804, stack: 5}, {name: "Test6", date: "12/01", value: 142205244, stack: 6}, {name: "Test1", date: "12/02", value: 19335, stack: 1}, {name: "Test2", date: "12/02", value: 174201, stack: 2}, {name: "Test3", date: "12/02", value:<PHONE_NUMBER>7, stack: 3}, {name: "Test4", date: "12/02", value:<PHONE_NUMBER>6, stack: 4}, {name: "Test5", date: "12/02", value: 55913698, stack: 5}, {name: "Test6", date: "12/02", value: 148038887, stack: 6}, {name: "Test7", date: "12/02", value: 8374, stack: 7}, {name: "Test1", date: "12/03", value: 23630, stack: 1}, {name: "Test2", date: "12/03", value: 204863, stack: 2}, {name: "Test3", date: "12/03", value:<PHONE_NUMBER>1, stack: 3}, {name: "Test4", date: "12/03", value:<PHONE_NUMBER>, stack: 4}, {name: "Test5", date: "12/03", value: 49869637, stack: 5}, {name: "Test6", date: "12/03", value: 142239144, stack: 6}, {name: "Test7", date: "12/03", value: 4210, stack: 7}]
Screenshots

Your Environment

Britecharts version used: 2.12.0
Browser Name and version: Chrome Version 79.0.3945.88 (Official Build) (64-bit)
Operating System and version: MacOS 10.14.6 (18G103) Mojave

Hi @amahterian-github, thanks for opening this issue!
Could you try to reproduce your issue in a Codepen? You could use this one as a starting point.
Thanks!

Here's my codepen, but I'm using Britecharts in a Vue context. I'm not sure how to represent that in CodePen.
https://codepen.io/amahterian/pen/dyPRpMe?editors=1112

Hi @amahterian-github,
the first issue I see is that there is a value missing in your data. If I add
       {name: "Test7", date: "12/01", value: 8374, stack: 7},

it stops failing.
It is weird that it doesn't fail on Firefox.

Stranger still that I tried this before, as I mentioned above, ie " filling in stack gaps with zeroed data to make everything homogenous". I added the line you mentioned and the errors did stop. I'll try this with actual live data and verify that it's fixed. Thank you!


ID from stubbed relationship

DBFlow Version: 4.0.4
Issue Kind: Question
Description:
I have a one-to-many relationship similar to the queen and ants example, with the ant having @ForeignKey(stubbedRelationship = true) var queen: Queen? = null.
I want to get a two-dimensional list (ie. a list of queens, and each of them having their list of ants). If I just query for the Queens and have them load all ants, that can be quite a few queries. The best approach in my mind is to do two queries, one for the queens, and another for the ants, and then map them. I'm OK with 2 queries, which is always going to be less than N queries. A join would also work, but to my knowledge that would mean using a Query Model and mapping it out from there, which seems like a worse solution.
My question: is there a way to get the queen_id from the stubbed relationship without loading the Queen?

when it's stubbed we don't load the full object. It instantiates the object and fills it only with referenced fields. if its a single @ForeignKey you can replace the object with the type of the field it references such as an int. If there was an abstraction (aka ForeignKeyContainer), it would be just as efficient as this way, since we would create a new object anyways for every ant.

or! have your ant model implement LoadFromCursorListener, and load the field from the Cursor manually.
public interface LoadFromCursorListener {

    /**
     * Called when the {@link Model} is loaded from the DB.
     *
     * @param cursor The cursor that is loaded.
     */
    void onLoadFromCursor(@NonNull Cursor cursor);
}

@agrosner Appreciate the responses. Do you think it'd be safe to just use the ant.queen.rowid that gets set when the ants are loaded from the cursor? I realize it might change if the primary keys change. I'm not concerned about the created objects at this point, just the number of queries.
Otherwise, I think the LoadFromCursorListener approach would work too, a little more boilerplate, but not excessive.
Thanks again!

yes you can safely reference any of the primary keys on a stubbed object no problem. Just be careful as the reference may be null if the cursor or reference doesnt exist.

if you need the full object, you can call load().

@agrosner  saw that the issue get resolved, But it would be good if load() return the object of table like foreignkeycontainer was doing well, it will help to migrate 3.3+ version code to newer version easier


viewer.text_overlay.text

Hi,
What is the name of the methods to change the color and set the position of the viewer.text_overlay.text?
BTW, viewer.text_overlay.color = 'red' crashes napari.
Better yet, is there doc to find all the methods?
Thx.

psobolewskiPhD),
My mistake, I used 'Red" with capital R. But, it should've not crashed napari.
Also, there should be documentation.
I found in text_overlay.py the following
visible: bool = False
color: ColorValue = (0.5, 0.5, 0.5, 1.0)
font_size: float = 10
position: TextOverlayPosition = TextOverlayPosition.TOP_LEFT
text: str = ""
So, the position can be changed but for selected places, found in _viewer_constansts.py
TOP_LEFT = 'top_left'
TOP_RIGHT = 'top_right'
TOP_CENTER = "top_center"
BOTTOM_RIGHT = 'bottom_right'
BOTTOM_LEFT = 'bottom_left'
BOTTOM_CENTER = "bottom_center"
I guess if there is no documentation, we need to find the details in the source code.
Thank you.

Thanks for the followup! I don't get why the API reference on napari.org doesn't have this info, everything does have proper doc strings. Will look into getting that fixed.
Also, I've updated an existing issue that we need some tutorial or guide to overlays in general:
https://github.com/napari/docs/issues/37
Regarding the crash, can you clarify that a bit more? For me the following doesn't cause a crash, just KeyError: 'Red'
viewer.text_overlay.text = "Hello World"
viewer.text_overlay.visible = True
viewer.text_overlay.color = 'Red'		# Note the capital, which is not correct on purpose

This is with napari 0.4.17
Does this cause a crash for you?

Regarding the API docs, I think it's because the overlays were not added here:
https://github.com/napari/napari/blob/main/napari/components/__init__.py
So the auto-built docs don't pick up on them.
@brisvag were the overlays still intended to be hidden?

Yeah, the overlay models are in abit of a limbo at the moment between private/public api, unfortunately. But maybe the pre-existing ones like text-overlay and axes should/could be made public.

My mistake, I used 'Red" with capital R. But, it should've not crashed napari.

Oops, at leas that should be easy to solve with a lower().

So, the position can be changed but for selected places, found in _viewer_constansts.py

Correct; as you've seen, this part is not really fully public api, so that's why it's not well documented or discoverable. So thanks for trying it out and reporting issues!

I tried to reproduce the 'Red' but it does not happen now.
Thank you all for you responses!

But maybe the pre-existing ones like text-overlay and axes should/could be made public.

I think this would be a good idea... lets add scale_bar to the list too.

So it would entail just adding to the __init__.__all__? Not very familiar with the docs generation :P

But can we do it in a way that shows the docstring of the TextOverlay model, rather than having to copy the docstring over to the property?

This is related to #7221


State Machine

What would you like sesh to do?
I think it could be really helpful to create a state machine for the logic that determines sesh connect "name".
Bonus: being able to visualize the state machine and add it to the README.md automatically.
Research

https://www.iamyadav.com/blogs/implementing-a-simple-state-machine-in-golang
https://towardsdatascience.com/writing-a-finite-state-machine-in-go-e5535e89d615
https://github.com/qmuntal/stateless

I think a simple sequence diagram will offer visibility to the logic of the project. Going to close this.


error

Note: you should install v2 branch.Could you please provide a detailed explanation of this sentence? I would be extremely grateful

Thanks for your issue!
Our code depend on the torch-radon library,as shown in picture,the torch-radon have two branch(master and v2)，and the master branch is default. But our code depend on v2 branch。so you should install v2 branch。


Maven plugin StackOverflowError in multimodule projects with extensions

In maven project with many modules and if one of them we use plugin as extension during "deploy" we got StackOverflowError.
Build generate many logs and don't stop.
You can test example project:
https://github.com/PayU-Tech/artifactory-test
Fragment of log:
Caused by: java.lang.RuntimeException: org.jfrog.build.extractor.maven.BuildInfoRecorder.sessionStarted() listener has failed:
        ... 1024 more
Caused by: java.lang.StackOverflowError
        at java.io.ExpiringCache.entryFor(ExpiringCache.java:103)
        at java.io.ExpiringCache.get(ExpiringCache.java:78)
        at java.io.WinNTFileSystem.canonicalize(WinNTFileSystem.java:406)
        at java.io.File.getCanonicalPath(File.java:618)
        at java.io.FilePermission$1.run(FilePermission.java:215)
        at java.io.FilePermission$1.run(FilePermission.java:203)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.io.FilePermission.init(FilePermission.java:203)
        at java.io.FilePermission.<init>(FilePermission.java:277)
        at sun.net.www.protocol.file.FileURLConnection.getPermission(FileURLConnection.java:228)
        at sun.net.www.protocol.jar.JarFileFactory.getPermission(JarFileFactory.java:166)
        at sun.net.www.protocol.jar.JarFileFactory.getCachedJarFile(JarFileFactory.java:136)
        at sun.net.www.protocol.jar.JarFileFactory.get(JarFileFactory.java:91)
        at sun.net.www.protocol.jar.JarURLConnection.connect(JarURLConnection.java:122)
        at sun.net.www.protocol.jar.JarURLConnection.getInputStream(JarURLConnection.java:150)
        at java.net.URLClassLoader.getResourceAsStream(URLClassLoader.java:238)
        at org.jfrog.build.extractor.maven.BuildInfoModelPropertyResolver.getMavenVersion(BuildInfoModelPropertyResolver.java:181)
        at org.jfrog.build.extractor.maven.BuildInfoModelPropertyResolver.resolveProperties(BuildInfoModelPropertyResolver.java:54)
        at org.jfrog.build.extractor.maven.BuildInfoRecorder.sessionStarted(BuildInfoRecorder.java:129)
        ... 1005 more

Hi @slawekjaranowski ,
There are 2 configuration issues in the plugin DSL configured in your pom.xml. Fixing them solves the issue.
This is the current configuration:
    <build>
        <plugins>
            <plugin>
                <groupId>org.jfrog.buildinfo</groupId>
                <artifactId>artifactory-maven-plugin</artifactId>
                <version>2.4.0</version>
                <executions>
                    <execution>
                        <id>build-info</id>
                        <goals>
                            <goal>publish</goal>
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <publisher>
                        <contextUrl>http://localhost/</contextUrl>
                        <repoKey>release-local</repoKey>
                        <snapshotRepoKey>snapshot-local</snapshotRepoKey>
                        <username>deployer</username>
                        <password>deployerpas</password>
                    </publisher>
                </configuration>
            </plugin>
        </plugins>
    </build>

You'll need to add the inherited tag to the DSL
<inherited>false</inherited>

Also, the configuration tag should be inside the execution tag.
After fixing these issues, your plugin configuration should look like this:
<build>
    <plugins>
        <plugin>
            <groupId>org.jfrog.buildinfo</groupId>
            <artifactId>artifactory-maven-plugin</artifactId>
            <version>2.4.0</version>
            <inherited>false</inherited>
            <executions>
                <execution>
                    <id>build-info</id>
                    <goals>
                        <goal>publish</goal>
                    </goals>
                    <configuration>
                        <publisher>
                            <contextUrl>http://localhost:8080/artifactory</contextUrl>
                            <repoKey>libs-release-local</repoKey>
                            <snapshotRepoKey>libs-snapshot-local</snapshotRepoKey>
                            <username>admin</username>
                            <password>password</password>
                        </publisher>
                    </configuration>
                </execution>
            </executions>
        </plugin>
    </plugins>
</build>

We recommend using the following references when configuring the plugin:

artifactory-maven-plugin-example
Maven Artifactory Plugin Documentation

Proposed solution with disabled configuration inheritance is not acceptable for us.
We have many projects which use artifactory, and we have one project called "Super POM" where we have global configuration.
So when we disable configuration inheritance we must copy configuration from one place to many other projects. Of course it will produce many work with maintenance.
Do you have another proposition how to maintain configuration for artifactory in one place for many projects?

Thanks for sharing this input @slawekjaranowski.
We will work on finding a solution for this issue.

I am running into the same limitation. We have a parent POM other projects are supposed to use, which is why we can't disable inheritance. Looking at the code I was assuming that to work but I am also running into the stack overflow.
A quick Google search reveals some blog post outlining how to build a plugin that only runs in the project root.

I recently ran into this issue as well on this project.
Everything was working as expected until this commit : travis build which consist to upgrade the version a plugin in a sub module
Reverting the commit fixed it : travis build
Going deeper, upgrading the version to the root POM of the project definitively fix it so that I was able to re enable the reverted commit : travis build
Hope this could help to diagnose this issue.

Hi @slawekjaranowski, @drautureau-sonarsource, @olivergierke,
We have released a new version (2.6.0) of the Maven Artifactory plugin which includes the fix for this issue.
You are welcome to use it.

thanks @romangurevitch for the notification.
FYI, I just tried it and I am getting this error:
[ERROR] Failed to execute goal org.jfrog.buildinfo:artifactory-maven-plugin:2.6.0:publish (build-info) on project sonarqube: Execution build-info of goal org.jfrog.buildinfo:artifactory-maven-plugin:2.6.0:publish failed: Plugin org.jfrog.buildinfo:artifactory-maven-plugin:2.6.0 or one of its dependencies could not be resolved: The following artifacts could not be resolved: org.jfrog.buildinfo:build-info-api:jar:2.6.x-SNAPSHOT, org.jfrog.buildinfo:build-info-extractor-maven3:jar:2.6.x-SNAPSHOT: Could not find artifact org.jfrog.buildinfo:build-info-api:jar:2.6.x-SNAPSHOT -> [Help 1]

Thanks @drautureau-sonarsource,
We have fixed the dependencies, you can use it now.

Did you redeployed same version of artifact on JCenter ?
If so, users will have to remove it from their Artifactory/Nexus/Archiva but also from local repositories on build machines/slaves.

BTW, I confirm that it fixes my use case (after removing the JCenter cache from the Artifactory instance).
2.6.1 would be welcome to avoid any confusion !

Thanks @hgomez-sonarsource , @drautureau-sonarsource,
Your feedback is very valuable and welcome.
We have released new version, 2.6.1.

<executions>
                <execution>
                    <id>build-info</id>
                    <goals>
                        <goal>publish</goal>
                    </goals>
                    <configuration>
                        <publisher>
                            <contextUrl>http://localhost:8080/artifactory</contextUrl>
                            <repoKey>libs-release-local</repoKey>
                            <snapshotRepoKey>libs-snapshot-local</snapshotRepoKey>
                            <username>admin</username>
                            <password>password</password>
                        </publisher>
                    </configuration>
                </execution>
            </executions>

I'm facing the same issue what is the solution?


created_at timestamp in OAuth2 refresh_token response

Summary
It would be great if we can get the 'created_at' or 'refreshed_at' timestamp in the OAuth refresh_token response as well. We currently see the 'created_at' attribute only in the initial OAuth token creation response Having this attribute will be useful to enforce the 'expires_in' attribute properly.

Contributions welcome!

I could tackle it, but I would like to clear some points:
The current response from the Token Endpoint looks like:
{
    "refresh_token": "MYHpsGzPJ0Nu5sMcg44BAiXGNZTh2zLj",
    "token_type": "bearer",
    "access_token": "pft312LDczVLBzcngdvkyAXkXSDbyTDm",
    "expires_in": 7200
}

The expires_in field is connected to the expiration time of the access_token. The default expiration time for refresh tokens is 2 weeks and cannot be configured.
We could add a field refresh_expires_in with a value of two weeks, but I looked a bit through OAuth2 docs and couldn't find a specification for such field. So this would be an optional field, which isn't specified in the standard OAuth Token Endpoint specification.
IMO, what would be more useful: add a config field to set the expiration time for the refresh token. WDYT?

Sorry if my initial request confused things up.
In order to ask for a refreshed access_token one would make an api call using the refresh_token within the 'expires_in' time limit but to be precise if an attribute called 'created_at' was also added to this same response it would be more accurate to ask for a new access_token using both created_at and expires_in fields. What I am trying to get as is relying on the application system time may add a few seconds of delay and so a time reference (created_at) would be helpful to make better use of the expires_in attribute (One could also argue that the refresh call can be made preemptively well ahead of the expires_in time to compensate for any delays caused while generating the tokens)

Ok, for custom TTLs in refresh tokens there is already an issue (#2024).
The expires_in field tell you how long the token can be used in seconds. It's not a timestamp. When saving the refresh token somewhere locally you can store also the timestamp from your system with it and add the expires_in value, so you get the timestamp, after which the refresh token is not valid. I don't see any use for the timestamp on Kong.
Besides, I don't think it's a biggie to try getting a new access token with a expired refresh token. You will simply get a error message, which means the refresh token isn't valid and you have to authenticate again. It's only one call.


Add support for vertical multiple display setup.

Set the Top property of the KeyShowView based on the selected display property.

Any progress on this?


Make dynamic require imports return unknown

Currently, require faces a similar problem to JSON.parse, where it returns any. Like JSON.parse, I think this would be another great place to reset the return type to be unknown.
I.e.
var require: NodeRequire (id: string) => any

becomes
var require: NodeRequire (id: string) => unknown

To be fair: require has intellisense on exported instances if I'm not mistaken.
So what exactly is the usecase of making it return an unknown type?

To be fair: require has intellisense on exported instances if I'm not mistaken.
So what exactly is the usecase of making it return an unknown type?
To eleborate:
When importing like so:
const hi = require("pkg");

The variable hi will contain all exported stuff and will show it in intellisense, where this may seem as any because it says so, it does not harm the typesafety in my opinion.

Yeah this is true when doing static imports, but if you're using require dynamically, for example
const importFile = (path: string): any => {
  return require(path);
};

The default behaviour of this require is to return any.
You're right about the intellisense stuff though so I'll investigate to see if theres a way to test this behaviour is maintained in the PR

Yeah this is true when doing static imports, but if you're using require dynamically, for example
const importFile = (path: string): any => {
  return require(path);
};

The default behaviour of this require is to return any.
You're right about the intellisense stuff though so I'll investigate to see if theres a way to test this behaviour is maintained in the PR

I've tested it in JS and no luck, does not change when you return unkown, the wrapper function will also not have any idea what is in there since you put a path as import path.

Yep thats exactly what I mean. We can change the Require's default return type to be unknown when its given a dynamic string such as in the above example (see here), then this function we can change to return unknown which in my opinion makes more sense.
When doing top level require, JS's intellisense will still kick in. So this change would only affect the very limited use case of using require when not knowing the path being passed in until runtime

I think top-level require calls should be using the import = require() syntax anyway, so I think this change should be fine.


feat: add OpenCollective provider

This PR adds an OpenCollective provider.
Blocking
OpenCollective returns an avatar url even if the user doesn't have one (which fails on resolveAvatars), a default one should be provided instead.
Concerns

OpenCollective suppots multiple currencies which might confuse users since the key is called monthlyDollars
OpenCollective graphql api is not fully documented (eg. pagination)
Didn't fully test the other collective inputs (id & gh handle)

OpenCollective Graphql docs
OpenCollective API key page

Output (on babel with a 10 backers hard limit):

closes: #8

@GeopJr Is this PR ready? I'd love to use it!

@GeopJr Is this PR ready? I'd love to use it!
@JounQin

Kinda - I just fixed the conflicts. The main problem still is the lack of default avatar (#11). As per the original post, OpenCollective returns an avatar url even if the user doesn't have one (e.g. https://images.opencollective.com/kentcdodds/d219cb9/avatar/460.png) which causes sponsorkit to fail during the avatar downloading portion:
FetchError: 404 Not Found (https://images.opencollective.com/kentcdodds/d219cb9/avatar/460.png)

However here's a quick (and dirty) workaround (it's a git patch):
diff --git a/src/image.ts b/src/image.ts
index 911f31a..faf7e23 100644
--- a/src/image.ts
+++ b/src/image.ts
@@ -5,8 +5,15 @@ import sharp from 'sharp'
 import type { Sponsorship } from './types'

 export async function resolveAvatars(ships: Sponsorship[]) {
-  return Promise.all(ships.map(async(ship) => {
-    const data = await $fetch(ship.sponsor.avatarUrl, { responseType: 'arrayBuffer' })
+  const defaultAvatar = await $fetch("https://i.imgur.com/nk8ksF0.png", { responseType: 'arrayBuffer' })
+  return Promise.all(ships.map(async (ship) => {
+    let data;
+    try {
+      data = await $fetch(ship.sponsor.avatarUrl, { responseType: 'arrayBuffer' })
+    } catch {
+      data = defaultAvatar
+    }
+
     const radius = ship.sponsor.type === 'User' ? 0.5 : 0.15
     ship.sponsor.avatarUrlHighRes = await imageDataURI.encode(await round(data, radius, 120), 'PNG')
     ship.sponsor.avatarUrlMediumRes = await imageDataURI.encode(await round(data, radius, 80), 'PNG')

This will use a quick default avatar I made from the GitHub "private sponsor" icon when an avatar 404s.
Here's the full babel list (1212 members - too big of an svg but ok for the sake of testing)
example usage: SPONSORKIT_OPENCOLLECTIVE_KEY=<YOUR_API_KEY> SPONSORKIT_OPENCOLLECTIVE_GH_HANDLE=babel pnpm run dev

I think we could have an option to configure the fallback avatar (could be a function for generated ones). Using GH's fallback one as the default sounds good to me

Should SPONSORKIT_OPENCOLLECTIVE_KEY be renamed to SPONSORKIT_OPENCOLLECTIVE_TOKEN to match the other providers?
OpenCollective does call it API Key however:

Since #11 is done, I think this PR is ready for review!


coupons: stream

A seperate news-stream for coupons instead of coupons in the excisting news-stream

Duplicate of https://github.com/our-city-app/oca-backend/issues/492


Fix usage of OCI target with insecure transport

The --insecure flag wasn't respected in oci.Repo#Has. Now, whenever that flag is set, a tls.Config with InsecureSkipVerify set to true is set injected into the DefaultTransport used to connect to the OCI registry.
Before
$ ~/dev/charts-syncer/dist/charts-syncer --insecure -c ~/dev/charts-syncer/test-config.yaml sync --latest-version-only --insecure
I0718 17:22:47.287675 3905275 sync.go:34] Using config file: "/home/max/dev/charts-syncer/test-config.yaml"
E0718 17:22:48.564152 3905275 index.go:153] unable to explore target repo to check "grafana-6.58.4" chart: failed checking remote: Get "https://localhost:8443/v2/": tls: failed to verify certificate: x509: certificate is valid for garage.garage.svc, not localhost; Get "http://localhost:8443/v2/": EOF
W0718 17:22:48.564170 3905275 index.go:120] Failed processing grafana:6.58.4 chart. The index will remain incomplete.
W0718 17:22:48.564180 3905275 sync.go:34] There were some problems loading the information of the requested charts: failed checking remote: Get "https://localhost:8443/v2/": tls: failed to verify certificate: x509: certificate is valid for garage.garage.svc, not localhost; Get "http://localhost:8443/v2/": EOF
I0718 17:22:48.564189 3905275 sync.go:49] There are no charts out of sync!

After
$ ~/dev/charts-syncer/dist/charts-syncer --insecure -c ~/dev/charts-syncer/test-config.yaml sync --latest-version-only --insecure
I0718 17:30:51.796045 3919041 sync.go:34] Using config file: "/home/max/dev/charts-syncer/test-config.yaml"
I0718 17:30:52.060203 3919041 sync.go:47] There is 1 chart out of sync!
I0718 17:30:52.060215 3919041 sync.go:55] Syncing "grafana-6.58.4" chart...

superseded by #190


Feature/details random point

Adding details of the position.
FormatGeoJSON
{"type":"FeatureCollection","features":[
{
   "type":"Feature",
    "geometry":...,
     "properties":{
           "title": *string* | *object*,
            "description":*string* | *object*,
       }
}
]}

interface object {
    [languageInital: string]: string
}

Example :
{
    "type": "FeatureCollection",
    "features": [{
        "type": "Feature",
        "geometry": {
            "type": "Point",
            "coordinates": [2.0104348, 42.4411844]
        },
        "properties": {
            "title": "Sainte-Léocadie",
            "description": {
                "fr": "un ville",
                "en": "a city"
            }
        }
    }]
}

Screenshots :

Map exemple
French station (source https://ressources.data.sncf.com/) https://gist.githubusercontent.com/BilelJegham/891f976fac2017ae19e699991aba4f01/raw/19bfe072be8762e93142b56eddecf83b967f7d9d/referentiel-gares-voyageurs-min.geojson

Codecov Report

Merging #56 (07bdd9f) into master (b4ef456) will decrease coverage by 1.07%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   22.70%   21.62%   -1.08%
==========================================
  Files          21       21
  Lines         709      712       +3
  Branches      118      116       -2
==========================================
- Hits          161      154       -7
- Misses        548      558      +10

Impacted Files
Coverage Δ

src/components/Maps.vue
0.00% <0.00%> (ø)

src/pages/StreetView.vue
0.00% <0.00%> (ø)

src/components/home/DialogCustomMap.vue
24.19% <16.66%> (-0.81%)
:arrow_down:

src/store/homeStore.js
93.44% <100.00%> (-0.77%)
:arrow_down:

Continue to review full report at Codecov.

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


Make all non-modal dialogs user-movable

Currently only the debug window is movable, while other non-modal dialogs like the usernotes dialog can't be moved. Would be nice QoL change, but it's not major.

Personal Notes is also.   But I agree that it'd be nice now that we have a solid solution for movable windows.

This also applies to usernotes, domain tagger and maybe others.

Please note that I am still not sure all of these should be movable.
side note: history button windows probably needs some testing now

This functionality will allow all dialogs that use the TB.popup function to make their things movable. The only reason I turned it off by default, and only enabled it for history notes, was that history notes being movable was marked for 3.1, while the others weren't, and we're in a feature freeze

That's fine.  But the issue still isn't closed until it's done.  That's how we forget about things.

Fair point. I'll keep it in mind

I adjusted the title so we also don't forget to make the ones that are already dragable make make use of the shiny new "dragable: true" thingie.


Implement block sparsity support

Block sparsity is needed for many physics applications.

you can  assign me to this

@Thenerdstation @amilsted @viathor @MichaelMarien
I think we should have a serious discussion about how we are planning to implement this. I am working towards implementing DMRG, TDVP and a few other 1d tensor network algorithms into the library. Ideally I would like to build it such that only minimal changes are required once we support symmetric tensors. For this we have to first settle on how we want to support it.
The only sane way of organizing block sparsity is by packing blocks into a class SparseTensor. This class also needs to take a backend argument. SparseTensor holds an array of tensors of the according backend type. We could probably also repurpose Node to fill the role of SparseTensor. In any case SparseTensor implements the API that high-level operations are using.
An immediate issue is that all backend functions (ones, zeros, randn, tensordot, ...) are incompatible with SparseTensor, meaning that we would have to redesign each backend to be able to handle both sparse and dense tensors of the according backend.
I would suggest that in this case we resort to internally supporting only SparseTensor, and handling dense tensors as special cases of sparse tensors with only one block (these details can be hidden from the user). We then build a linear algebra module which implements the following backend independent linalg operations

decompositions: svd, qr, rq, eigh, eig, ...
sparse solvers: eigs, eigsh, lanczos, ...
contractions: tensordot, einsum, ...
initialization: ones, zeros, randn, ...
misc: conj, transpose, sqrt, diag, ...

which exclusively act on SparseTensor. Behind the scenes we will still use our current backends on the level of the individual dense tensors. This could be hard for e.g. sparse solvers if we want support TPU with  tensorflow or jax (due to possible complications under jax.jit or tf.function).
This approach also has a potential drawback: we likely need to store dense blocks in a list (occupying different memory locations). For high rank sparse tensors, the number of such blocks can become huge: e.g. for a rank 7 tensor (very common in MERA) with 10 charge sectors (a very typical situation in DMRG) on each leg, this typically yields 10^7 non-zero blocks. Operations like transpose, tensordot, svd, ... can become very  slow because we need to iterate through a very long list in python. People typically port such things to e.g. C++ to obtain considerable speed up. In our case, this may be very hard/impossible to do. In the long run, it thus would make sense to design a symmetric tensor class in C++ and expose its interface to python.
We should also decide if GPU/TPU is a priority for SparseTensor support or not. It is conceivable that sparse problems benefit substantially less from accelerated hardware. For example, in symmetric DMRG, one typically ends up with O(10) - O(100) charge sectors, but each dense block
has modest bond dimension O(100).

closing this issue because we have added the support


Update tutorial documentation

My PR is related to Documentation. Should I still follow the PR rules ?
Since th repository airbnb/airflow does not exist anymore. I have changed the URL https://github.com/airbnb/airflow/blob/master/airflow/example_dags/tutorial.py to https://github.com/apache/incubator-airflow/blob/master/airflow/example_dags/tutorial.py

I can merge this. Thanks @Raniazy !


Have a way to split a UnixStream into reading and writing halves.

I have a UnixStream that I want to read from in one thread and write to in another thread. It would be cool if UnixStream had a function to split it into two sides:
fn split(self) -> (UnixReader, UnixWriter);

The try_clone method will allow you to get a second UnixSocket that uses the same connection as the main one just like TcpStream, UdpStream, File, etc in the standard library: http://sfackler.github.io/rust-unix-socket/doc/unix_socket/struct.UnixStream.html#method.try_clone


Consistently use ubuntu-latest?

Currently, some actions use ubuntu-18.04, while others ubuntu-latest.
@krlmlr If you agree, I can make this change.

Thanks. I have a slight preference for fixing the version (e.g. 20.04 instead of "latest"). Will everything continue to work when we upgrade to 20.04? Should we maybe even deliberately test things on older images too?

Let's switch to Ubuntu 22. We also need to switch to Node 16, e.g., https://github.com/r-dbi/DBItest/actions/runs/3270079958. Can we take this in one swoop?

We also need to switch to Node 16

IINM, those warnings are coming from r-lib/actions: https://github.com/r-lib/actions/issues/626
We don't specify Node version anywhere.

Do we need to use a newer tag for actions/checkout?

No, I don't think we need to.

I'm glad we didn't use ubuntu-latest because R is no longer available by default on ubuntu-24.04 and some workflows rely on it.


Contrib guide update

This is mostly a clean up and doesn't change the gist of it, since it was already pretty well written. @thunterdb can you take a look? Thanks.

Codecov Report

Merging #246 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #246   +/-   ##
=======================================
  Coverage   91.59%   91.59%
=======================================
  Files          35       35
  Lines        3022     3022
=======================================
  Hits         2768     2768
  Misses        254      254

Continue to review full report at Codecov.

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


We should run on host network in order to be able to run normally in infra cluster

We should run on host network in order to be able to run normally in infra cluster
Currently after enabling ovn offload we can't run pods on infra cluster
that are not using host network, so each operator restart will cause it
to stuck.
Another issue is dns resolution to tenant cluster as currently we set in
etc hosts as workaround

Need to run make bundle manifests too. Maybe as a separate commit.

In controllers/ovnkubeconfig_controller.go you missed to add hostnetwork here:
-//+kubebuilder:rbac:groups=security.openshift.io,resources=securitycontextconstraints,resourceNames=anyuid,verbs=use
+//+kubebuilder:rbac:groups=security.openshift.io,resources=securitycontextconstraints,resourceNames=anyuid;hostnetwork,verbs=use

/lgtm

@pliurh can you set approved

It looks good to me. But I think it would be better to put the reason why we want to run the operator in the host network in the commit message.

/lgtm
/approve


Added an expand_shards script and sharding vs. single-run modes.

Need to:

[ ] Update docs
[ ] Test fully

Summary by CodeRabbit

New Features

Introduced a new script expand_shards to handle file path expansion with numeric suffixes.

Improvements

Updated the configuration file path for better organization.
Enhanced default data settings and output file path generation logic.
Added new configuration files for data sharding and single file handling.
Introduced options for specifying data standards and timestamp formats.

Added some docs in #39


Update suggestions

What type of PR is this?

[x] Refactor (because of calling load_dotenv)
[x] Documentation

Description
This change suggestion include 4 small parts :
1- update of the requirements-metr.txt file that was used before...
even if now it is not really needed but i had issues with poetry not installing all the packages and tests were failing because of that locally but they were working on docker image. So, i got the requirements from the docker image and put them in this file , this way, the dependencies can be installed with pip3 install -r requirements-metr.txt , if you prefer to remove that file i would be okay with it since it is just a temporary workaround
2- add .env-example file in order to help setting up the project locally since we usually have an example that we copy and fill ...
they have a command make .env that would create the .env  file but, not only it wouldn't be easy to notice, it also creates it without the DATABASE_URL variable , so, i thought it would be a good idea to have it
3- make sure the env variables in the .env file are loaded by calling load_dotenv...
I honestly don't know why they were not doing this already since we install the python-dotenv package and they introduce .env file
4- update the readme file with instructions to run both backend and frontend locally without docker.
Extra note
The pre-commit is failing but not because of our changes , so I chose not to fix line of codes that we don't touch in order to avoid a lot of conflicts with getredash code later .
How is this tested?

[x] Unit tests (pytest, jest)

Great work!
I would prefer to not rely on the requirements-metr and only use poetry to be as close as possible to their development structure.
Have you tried installing with poetry using poetry install --with dev?

Oh yes makes sens, this command seems to install the dependencies needed actually, thanks a lot, i think dev should already be included within the poetry install --only main,all_ds,dev command.
Seems like with poetry my issue would only be mysqlclient requirement that i forgot about because I skipped it and it does not affect tests , i documented it here . I will try to fix it and update this PR and the documentation issue :D

@thiagogds i am so happy , it works now with poetry now for me :D
I updated the readme and removed the requirements-metr file since it is not needed anymore


add support for rendering diagrams with mermaid

What
This adds support for rendering diagrams from any markdown code blocks with the language tag mermaid.
For example the code block:
```mermaid
sequenceDiagram
    Alice->>+John: Hello John, how are you?
    John-->>-Alice: Great!
```

Will now render an inline <svg> diagram of the sequence instead of the raw
<code> block.
Before

After

Why
Keeping diagrams as code in this way makes it significantly easier to keep diagrams up to date with the documentation, and can make reviewing changes to them easier.
We often use sequence or state diagrams in internal team manuals to give visual aids to complex systems or architectures.
An example can be seen in the Alertmanager documentation within the Reliability Engineering Team Manual ... if a minor change is required to this diagram, the author would need to have the original source or start again from scratch. But with the ability to use diagrams directly from markdown, the source of the diagram would be easily update-able and preview-able from the middleman server.
How
mermaid generates diagrams and flowcharts from text in a similar manner as markdown.
The implementation takes advantage of the existing project dependency on node.js to install and execute the mermaid cli tool that translates the various diagram code into SVG. A timeout is added to execution to workaround an issue where the cli tool fails to terminate on error.

Human-readable version-controlled diagrams in the docs - such an excellent addition ✨
I realise this is a work in progress and this is probably on the to-do list or out of scope, but it would be good if the diagrams could be made to look like GOV.UK.
Also for accessibility, it would be good to document how to add alt text to these diagrams.
Great work - this is a super useful feature 🙌

@bravegrape RE: looks...
I agree, the default theme/colors does clash a bit with the GOV.UK "brand" ... I don't fancy spending hours tweaking CSS to paint this particular shed ... however there is a grey-scale theme, which will likely be less controversial, I'll switch it to use that.

@chrisfarms We might be able to ask the Design System team to help us with colours (and make sure things like the colour contrast meet WCAG 2.1 requirements).
@bravegrape great point and sorry I should have added a note here that @chrisfarms and I had a quick chat about accessibility the other day. I'm planning to catch up with our accessibility team here to check where we'd stand on using Mermaid accessibility-wise.

@m-green and @chrisfarms I'm sure you're on top of things on the accessibility front 😄
Being on a mid-week-once-in-5-years national day off, I couldn't keep away from exciting new features being developed for the GOV.UK tech docs template. It would be great to find out more about accessibility for diagrams-as-code and if their "code" nature can help with accessibility 🤔


improvement: mail structure

no issue
In preparation for subscribers V2. We will have multiple mail adapters and right now the mail adapter we have was implemented in index.js.
Get rid of writing code in index.js (in this case for mail logic).
Create utils for mails and get rid of writing util code in api folder.

needs a rebuild.
after rebuild, ready to review 👍

Failed build has been restarted


Issue with screener

I tried the same code as in the README file and I got this on Google Colab:

I have tried on Google Colab myself which doesn't have the problem. Maybe try some other functions and/or update the packages.

I have tried on Google Colab myself which doesn't have the problem. Maybe try some other functions and/or update the packages.

I don't get it but I had: foverview.set_filter(filters_dict) instead of foverview.set_filter(filters_dict=filters_dict) now it's working correctly

I don't get it but I had: foverview.set_filter(filters_dict) instead of foverview.set_filter(filters_dict=filters_dict) now it's working correctly

Just one question before you close this, how can I get an Industry P/E ratio like here

Just one question before you close this, how can I get an Industry P/E ratio like here

It is under finvizfinance.group.valuation

It is under finvizfinance.group.valuation

You can view the example in the example folder of this package.

You can view the example in the example folder of this package.

Good, thank you very much!

Good, thank you very much!

This issue will be closed in one day. Thank you for your advice.
Tianning Li

This issue will be closed in one day. Thank you for your advice.
Tianning Li

FYI, filter_dict is not the first parameter in the function so you need to pass in the parameter name.

FYI, filter_dict is not the first parameter in the function so you need to pass in the parameter name.


[TB] We need to update the video in mkdocs

Problem/Concern
@lwnoble suggested that the video that we have in our documentation is out of date and that she has a new one available.
Proposed Solution
I will work with Lise by updating the mkdocs files with the new video.

Hi @aaronreed708 - can I add the videos from conference talks to the mkdocs files as well?  Is there a place where I should make those changes if you agree?

On the project call today, I shared that we can also work wtih Win Morgan at the Linux Foundation to have videos posted to the FINOS resources site and tagged for our project:  https://resources.finos.org/category/projects-sigs/a11y-theme-builder?projects-sigs=a11y-theme-builder&c=cG9zdDo5OTE2NTU%3D
For example, the Video and PDF of the slides from the FINOS/Discover GAAD Celebration are posted there - we could decide to centralize videos and other resources on the FINOS Resources site, and tag them to our project. Win is on the FINOS Slack and has been pretty easy to work with; the resources site includes blogs, videos, PDFs, and links to the ThemeBuilder FINOS Sandbox.


Updating GET section

Adding 204 http status code as a possible result for GET operation

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

#label:"aq-pr-triaged"

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

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

#sign-off
(I'm sure this will result in a message saying I can't do this ... but this is just me saying I'm okay bringing this in now.  Thanks!)

@ckittel: I'm sorry - only the author of this article, @EdPrice-MSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ckittel Thanks for the review.


Update Readme.md

Description
Going through all readme.md files for grammar issues. fixing, formatting issues, hyperlinks etc.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Sample Migration (Moving sample from old repository after completing checklist established)

@JoeOster -- changes to the .../Tools/Migration/* samples look fine.

@JoeOster -- changes to the .../Tools/Migration/* samples look fine.


Can't compile rusqlite

Hi, I've just started a new project with rusqlite (with version "0.18.0"). However, when I try to build my project, I get the following errors:
    Updating crates.io index
  Downloaded lru-cache v0.1.2
  Downloaded libc v0.2.54
  Downloaded linked-hash-map v0.5.2
   Compiling libc v0.2.54
   Compiling num-traits v0.2.6
   Compiling num-integer v0.1.39
   Compiling pkg-config v0.3.14
   Compiling memchr v2.2.0
   Compiling linked-hash-map v0.5.2
   Compiling fallible-iterator v0.2.0
error[E0658]: use of extern prelude names introduced with `extern crate` items is unstable (see issue #55599)
  --> /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/fallible-iterator-0.2.0/src/lib.rs:92:13
   |
92 |     pub use std::collections::{BTreeMap, BTreeSet, HashMap, HashSet};
   |             ^^^
   |
   = help: add #![feature(extern_crate_item_prelude)] to the crate attributes to enable

error[E0658]: use of extern prelude names introduced with `extern crate` items is unstable (see issue #55599)
  --> /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/fallible-iterator-0.2.0/src/lib.rs:93:13
   |
93 |     pub use std::hash::{BuildHasher, Hash};
   |             ^^^
   |
   = help: add #![feature(extern_crate_item_prelude)] to the crate attributes to enable

error[E0658]: use of extern prelude names introduced with `extern crate` items is unstable (see issue #55599)
  --> /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/fallible-iterator-0.2.0/src/lib.rs:94:13
   |
94 |     pub use std::prelude::v1::*;
   |             ^^^
   |
   = help: add #![feature(extern_crate_item_prelude)] to the crate attributes to enable

I have the following in my Cargo.toml:
[package]
name = "scheduler"
version = "0.1.0"
authors = ["crides<EMAIL_ADDRESS>edition = "2018"

[dependencies]
rusqlite = {version = "*", features = ["chrono"]}
chrono = "*"

Can anyone help fix this? Thanks in advance!

What is version of rustc ?
rustc --version

rustc 1.32.0-nightly (15d770400 2018-11-06)

You reminded me. No problem with 1.36.0 nightly.


typescript conversion

Question
Hi, can anyone please send me a code snippet on how to convert a typescript file to javascript using this library.

What is the language are you using?
swc is rust library and js library at same time.

What is the language are you using?
swc is rust library and js library at same time.

I am using it in a rust project
On Thu, Jan 14, 2021, 4:18 PM 강동윤<EMAIL_ADDRESS>wrote:

What is the language are you using?
swc is rust library and js library at same time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/swc-project/swc/issues/1323#issuecomment-760116376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYS7NBREPDVVULOBORGDSTSZ3DYVANCNFSM4V3YZAQQ
.

I am using it in a rust project
On Thu, Jan 14, 2021, 4:18 PM 강동윤<EMAIL_ADDRESS>wrote:

What is the language are you using?
swc is rust library and js library at same time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/swc-project/swc/issues/1323#issuecomment-760116376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYS7NBREPDVVULOBORGDSTSZ3DYVANCNFSM4V3YZAQQ
.

@Japroz-Saini
Then module.fold_with(&mut swc_ecma_transforms::typescript::strip()) would be enough.
Note that you should enable a cargo feature named 'typescript'.

@Japroz-Saini
Then module.fold_with(&mut swc_ecma_transforms::typescript::strip()) would be enough.
Note that you should enable a cargo feature named 'typescript'.

I didn't understand. For example ,i read a file and stored in a variable called code. How would I convert that typescript file to javascript and how can I enable the typescript feature in the cargo.toml file

I didn't understand. For example ,i read a file and stored in a variable called code. How would I convert that typescript file to javascript and how can I enable the typescript feature in the cargo.toml file

See ecmascript/parser/examples/typescript for parsing code, and you can just do module.fold_with(&mut swc_ecma_transforms::typescript::strip()) to convert it to javascript ast.
Finally use swc_ecma_codegen to convert it into the javascript code.

See ecmascript/parser/examples/typescript for parsing code, and you can just do module.fold_with(&mut swc_ecma_transforms::typescript::strip()) to convert it to javascript ast.
Finally use swc_ecma_codegen to convert it into the javascript code.


Added Typescript Typings

Not sure if you are interested in supporting Typescript users, but I did this while using your library for myself and thought I would share. Thanks!

Of course. The whole world is like TypeScript now mate!

Any updates on this? Would be nice to get this merged in

If the answer is no, I'll gladly submit a PR to DefinitelyTyped to get typings added there.

@PaitoAnderson Can you make PR to DefinitelyTyped?

@PaitoAnderson  any luck friend ?

This would be much appreciated

Any updates on this one? Thanks :)

bump?

bump

no stale


feat: improve support for relationship queries (nested SELECTs)

feat: complete nested FROM with parent-child relationships:
SELECT (SELECT Id FROM |) FROM Account
feat: complete nested SELECT with fields of the relationship object:
SELECT (SELECT | FROM Users) FROM Account
feat: complete nested ORDER BY with fields of the relationship object:
SELECT (SELECT Id FROM Users ORDER BY |) FROM Account
fix: don't propose more than one level of SELECT nesting
fix: don't propose aggregate functions or GROUP BY on nested SELECT
feat: only propose Id fields on semi-join SELECTs
W-8558994

Note: need branch feat/improve-nested-selects of salesforcedx-vscode for testing from the UI:
https://user-images.githubusercontent.com/152839/113082260-82d1ef00-91b0-11eb-8ef8-e277a0869b2e.mov

QA on this PR complete.


CLOUDP-263321: Unified development environment (Nix)

Problem
Managing dependencies in our development setup is challenging. Current issues include:

Ensuring up-to-date versions for security
Pinning specific versions
Handling non-Go binaries
Inconsistencies across different developer environments
We'll adopt Nix-based environment declarations to sole those.

Changes
This solution enhances flexibility, maintenance, and consistency. The implementation includes changes in the following areas to support this approach.:

Shell.nix (Includes a declarative approach to install all necessary dependencies)
Updated CI pipeline (Update the CI pipeline to remove all manual dependency installations, relying solely on Nix)
Makefile  (Remove all manual dependency installations)

[x] Have you signed our CLA?
[x] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
[x] Update docs/release-notes/release-notes-template.md if your changes should be included in the release notes for the next release.

👌 ok to move to review right?

The Dockerfile conflict needs to be addressed, rebasing from main should help. Basically is because of the ubi cache changes merged yesterday.

I’d like to close this PR in favor of using Devbox. I believe Devbox offers more benefits and provides an easier installation process compared to Nix(both on the CI, locally and on the IDE). While Nix remains a valuable tool and could serve as an alternative if Devbox were to become unavailable, Devbox currently simplifies our workflow significantly. It effectively acts as a wrapper around Nix, making it the best choice for us at this time.

I’d like to close this PR in favor of using Devbox. I believe Devbox offers more benefits and provides an easier installation process compared to Nix(both on the CI, locally and on the IDE). While Nix remains a valuable tool and could serve as an alternative if Devbox were to become unavailable, Devbox currently simplifies our workflow significantly. It effectively acts as a wrapper around Nix, making it the best choice for us at this time.

Agreed, feel free to close. We can just reference it as an experiment from the Jira ticket.

This experimental PR was fantastic! It gave us valuable insights into Nix. However, at this point, we've found that Devbox offers better functionality and is easier to use. If anything changes and Devbox becomes deprecated or goes out of business, we’ll revisit this option. For now, updating with flakes is still a solid approach to leverage some of Devbox’s benefits. We'll keep this as a starting point for the Unified Development Environment, which I believe will evolve significantly in the coming years due to its potential to enhance reproducibility, scalability, and usability across repositories.


More work on inline dialogs

This also fixes some issues with local vector (WFS) layers found during testing of the modals
closes #394
closes #389

Coverage increased (+0.2%) to 70.328% when pulling 0541fdb5487ac127d7ca451b569377ba676b2b3b on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.

Coverage increased (+0.2%) to 70.328% when pulling 0541fdb5487ac127d7ca451b569377ba676b2b3b on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.

Coverage increased (+0.2%) to 70.347% when pulling bbbb48ffdb6b29bf44144fde9a2cb67dd7f6dd5b on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.

Coverage increased (+0.07%) to 70.233% when pulling 820723ce757583f5b1ed0334f9bf55d94caa8991 on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.

Coverage increased (+0.05%) to 70.222% when pulling abf6d3ab643ddeff47cd3cef06c9b60fb45fd513 on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.

Coverage increased (+0.05%) to 70.222% when pulling abf6d3ab643ddeff47cd3cef06c9b60fb45fd513 on layerlist-modals into d5d969d64068b984285b474083956b5b77b3809f on master.


syntax error at importing

After upgrading cfscrape with pip, now I'm unable to import cfscrape

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\tools\python\lib\site-packages\cfscrape\__init__.py", line 107
    print "'%s' returned error %d, could not collect tokens.\n" % (url, resp.status_code)
                                                              ^
SyntaxError: invalid syntax

I get the same error on my windows (8.1) and linux (Ubuntu 14.04) machine and both running Python 3.4.2
Am I doing something wrong? I tried to uninstall cfscrape and installed it again
also PyExecJS and requests are the latest version

Looks like it's working on Python 2.7 and iPython. How ever, there are some changes on print. They have replaced print statement with print function.
https://docs.python.org/3.3/library/functions.html?highlight=print#print
Looks like init.py tries to print out the error message using print statement, so this is why it happens. I can fix and send a PR if needed.

Done, please see pull request #23.


Dynamic import errors with computed strings

The following code triggers a Flow error:
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBjOA7AzgFzCjjjAF4wBLAWwAc4AnfACgAMA6YWgQ3wAtg+OMAAkAbyx0ArvgCmAEwAKvPgF92AK1ysAlAG5UQA
const foo = import(`./path/to/${computedPath}.js`);
            ^ The parameter passed to import() must be a literal string.

Per the spec:

import() accepts arbitrary strings (with runtime-determined template strings shown here), not just static string literals.

You need to set module.ignore_non_literal_requires

Ah, got it. Is there a reason it does throw an error? Is it not spec?

We should add that config option to the error message but:
In general this is a limitation of Flow. Flow needs to understand the dependency graph in order to understand the type of a module's imports -- so, by default, Flow asks that you only use literal strings as those are the only statically identifiable option. A fully-typed way to do this kind of import would be to use a switch statement:
let foo;
switch (computedPath) {
  case "path1"
    foo = import("./path/to/path1.js");
    break;
  case "path2":
    foo = import("./path/to/path2.js");
    break;
  ...
}

If you use the config option, Flow will suppress the warning and type dynamic imports that it can't resolve as any
}

This error message is just a straight-up lie. It makes it look like import (x) does not allow non-string-literal arguments (plausible, since it is a language construct not a function, and so might have unusual requirements).
In reality, there's nothing wrong with specifying an expression, other than Flow throwing a hissy fit, and sending developers on an expedition across the internet trying to find why it isn't allowed.


Cost for data trasfert between Azure and Sharepoint Online (SharePoint Online using the Azure Data Box)

Hello,
In case of SharePoint Online migration using the Azure Data Box, what will be the cost for the data tranfering between the Azure VM and the SharePoint Online ? (Outgoing bandwith from Azure ?)
Regards

@Rikonimo - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@oficedocsbot close


Emulator: Microsoft.Azure.Cosmos.Compute.Host.ServiceFabric.EntryPoint.exe crashes on startup

Microsoft.Azure.Cosmos.Compute.Host.ServiceFabric.EntryPoint.exe crashes on Emulator startup, but Emulator seems to work just fine.

System.Collections.Generic.KeyNotFoundException: 'FabricDataPath'

To Reproduce
Launch emulator normally.
Expected behavior
No crash.
Actual behavior
Microsoft.Azure.Cosmos.Compute.Host.ServiceFabric.EntryPoint.exe crashes.
Environment summary
OS Version: Windows 11 build 25201 (Dev External Channel) - crashes on multiple builds already.
Emulator version: <IP_ADDRESS> (crashed on <IP_ADDRESS> as well).

Please report Emulator issues through the Emulator channels: https://learn.microsoft.com/en-us/azure/cosmos-db/troubleshoot-local-emulator, this repository is for the Cosmos DB .NET V3 SDK.


Pop-ups::YouTube Videos submitted through bookmarklet are not playing on the pop-ups.

@meeshmon @rchen152
Rebecca fixed the video problems with issue #887, however similar problems have been occurring again.
YouTube videos submitted through the bookmarklet are not playing in the pop-ups (as shown in the red box in the image below) and thumbnail images (yellow box) do not appear on the JDA site either. These videos play on Waku, so I would like to keep the admin page uris and attribution uris as they are.
I do not know how to phrase it but does it have to do something with the code for video bookmarklets  and the pop-ups?
For example,
http://jdarchive.org/admin/zeega/data/item/1993962/edit

The video works without a problem in Waku.

The reason for this is that some YouTube videos submitted through the bookmarklet have layer type "outside-YouTube" rather than "YouTube".  Pull request https://github.com/Japan-Digital-Archives/Japan-Digital-Archive/pull/1001/ fixes this for some items (e.g., http://jdarchive.org/admin/zeega/data/item/1990819/edit), but other videos (see http://jdarchive.org/admin/zeega/data/item/1992627/edit) are still not playing - need to look into this further.


EXTERNAL版本无法运行

Describe your bug here.
第一次运行出现图一这种情况

后面运行直接闪退
Crash Log or logcat logs (if existing)
No response
Are you modding a build from source or with Lua?
I am not modding
What is your build target?
Android
Did you edit anything in this build? If so, mention or summarize your changes.
no

system not support


GO:0000827 has an EC number as synonym

Hi,
GO:0000827 inositol-1,3,4,5,6-pentakisphosphate kinase activity
has broad synonym EC:<IP_ADDRESS>
This was probably meant to be a dbxref.
Note, EC:<IP_ADDRESS> is already a dbxref to three other terms in GO.
Thanks,
Paola

Probably my bad. I will remove the synonym.

Note it does satisfy one of the two reactions described in the EC entry, so I am keeping it as an xref. @hdrabkin, please correct this if I am wrong.  Rhea:10276 is a bit more specific that our def. Have you been changing these?

RHEA maps RHEA:10276  to GO:0033857 based on the EC, (which I have not added yet ( not
GO:0000827  There are no predictions to GO:0000827 (we would need to request)