Empty files are missing

Hi,
I'm using wskdeploy to deploy a nodejs project. Some of my dependencies under node_modules contains empty files, that are currently ignored by wskdeploy:
https://github.com/apache/openwhisk-wskdeploy/blob/d653592d8b8e8b7e64f228049bfa2f74db07a1e1/utils/zip.go#L79-L81
Unfortunately, the missing files currently break my application.
Is the empty check really needed?
How can I ensure that empty files will be uploaded?
My workaround:
find . -type f -empty -name '*.js' -exec bash -c "echo '//' > \"{}\"" \;

@Wohllebe-dotSource Just curious... why does your code need to include zero-length files?  It would be good to understand the use case before any code changes are made. Also, feel free to submit a PR for review/discussion and try the removal of the zero size check locally to verify your assumption.


Gradle: Account for the kotest plugin to create Gradle run configs now

See https://github.com/kotest/kotest-intellij-plugin/commit/114ec28.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

Merging this despite the bogus Travis failure as other checks completed and the change was verified locally.


Dont buy this service they're just scammers

No Customer Support, They won't respond to you at all and they'll ban you from the discord server if you pinged them or asked for help
just dont buy their service

and this guy SoraDev#1337 will ban you if you pinged him asking for help and the other guy Jae#1234 is just a shadow he doesn't send a single message nor respond to any ticket


feat: db pool config #5935

Pull Request Checklist
Changelog Entry
A connection pool can now be specified using env variables DATABASE_POOL_SIZE, DATABASE_POOL_MAX_OVERFLOW, DATABASE_POOL_RECYCLE and DATABASE_POOL_TIMEOUT
Description

this PR adds 4 env variables so that we can configure a pool for the database. It follows the documentation of SQLAlchemy here
in case the env variable DATABASE_POOL_SIZE is not configured or set to zero, then it falls back to no pool used at all like before. So it should not introduce any breaking change.

Added

The ability to configure a connection pool.

Fixed

It fixes #5935

Great addition, Thanks!


IE8 support

Hi,
Do you have IE8 support?
Also would like to attach the right and left arrow key events to move the slider.

Well ;) i need to check, which angular version ?

AngularJS v1.2.18

well, I got a lot of errors too while testing on IE8 and debugging tools so frustrating, I will try again but it is difficult to spend time on this

looks like you are using pageX and pageY which is not defined in IE8. Used clientX and cientY, if pageX is undefined and worked in IE8

It could be a solution if you can make PR i will test it but code is quite tricky now and maybe few other properties not handle well by ie


kwmrc settings not being applied

This happened after running brew upgrade. 2.2.0 => 3.0.1
I tried running various combinations of:
pkill kwm
kwmc quit
brew services restart kwm
kwmc config reload
I just started having this problem. never happened in previous 6 months of use.

Just switched back to 2.2.0 and everything works
here is a copy of my config: https://github.com/scottgrayson/Dotfiles/blob/master/kwmrc
located at ~/.kwm/kwmrc on my system

just saw this.... https://github.com/koekeishiya/kwm/issues/386


Added an example of x-clock with React.

Yep, I haven't tried in IE8. I just copied the x-clock example. How might I test in IE8 (considering I don't have Windows on my machine right now)?
I also put up the demo here: http://trusktr.io:3237/x-clock-react.html

yeah it doesn't work, I've just tried ( from Linux ... all you need is VirtualBox or similar and a VM image )
So since I'm pretty sure React won't work there, and neither will Babel, just please drop those IE8 related parts, thanks.

http://netrenderer.com is pretty cool! It worked in IE9, but as you confirmed, it rendered nothing in IE8.
I (gladly) removed those parts. I personally have no intention of supporting anything lower than IE11 (maybe even Edge) in my personal projects. x]

By the way, the reason I made this demo was to prove that a DOM manipulation library like React works with custom elements, which in turn shows that the concept behind A-Frame (which uses Custom Elements) is awesome, and that we can make a 3D scene using any DOM helper like React, Mithril, Mercury, Om, CitoJS, etc.

Yeah, it's good that this polyfill is not obtrusive and it works well with also not so obtrusive technologies ;-)

Thanks. I've published this demo too in the test page

Sweet :+1:


Tranlsated EventSeries add-on to French

In addition to the actual translation, I also proposed a few minor edits to make the template section easier to understand.

@fjjulien I think you meant to make a PR on the Culture Creates repo: https://github.com/culturecreates/artsdata-data-model


Electron HTML form input un-editable

I had previously created HTML form elements to drive other beaker cells, this works well in the browser. When I open the notebooks in the Electron version they render correctly, but the input fields are not editable. Is this a bug or do I need to re-implement the way I am taking user input?

It's just a bug in the Electron version, thanks for the report.


 unable to cmd/ctrl+click to open in a new tab

version 3.1.0
When using the <Link/> component, you are unable to cmd+click (mac) or ctrl+click (windows) to open a link in a new tab.

My bad - this is a duplicate (https://github.com/preactjs/preact-router/issues/253)


Ignore deepcopy autogenerated files in codecov

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


Hide Django admin next url

Django next parameter shows in the URL, it'd be nice to hide this for security.
admin/login/?next=/admin/ should just be admin/ or admin/login

@jantaylor I have been looking high and low to figure this one out, it looks like it may have to stay that way because of the way admin is built in the app, it changes based on login and logout.

@dabidpoto My friend has his hidden. I wonder if he's using apache or IIS to hide it.

No need to complete.


Add enforceUniqueId parameter to material constructor

See https://forum.babylonjs.com/t/bad-lag-200ms-and-increasing-in-pushmaterial/16003/10

LGTM I ll merge tomorrow if no fire are starting due to our previous merge :-)


Filter /post/ API

Posts should be filtered from the API if they are unlisted.
They should also be filtered if they don't belong to our server

as long as the source and origin fields are populated in the database for all the posts, that would be easy to do

should be fine yeah, just check the posts "origin" url or something


[AutoRelease] t2-network-2022-11-29-34202(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3469
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1016
BuildTargetingString
azure-mgmt-network
Skip.CreateApiReview
true

/check-enforcer override


[Story] How to change incorrect financial institution details

Epic
Epic issue includes overview, complete list of user stories, technical and content requirements, user testing task ideation, and current design mock-ups.

[Epic] Sign and Submit

User story

As a filer, I want to know how to change any incorrect financial institution details I find in my review, so that I can update my financial institution details with correct information.

Acceptance criteria

Given a system user is in the sign and submit page, when they scroll down to the financial information details section the system will display information on how to make changes to their financial institution details if they find the information is inaccurate or incomplete, then they will be able to follow the instructions to make the changes.

Tasks
Team members should create tasks and add links (below) prior to sprint planning.
UX Design / Content

[ ] Confirm where a user must go to change incorrect financial institution details
[ ] Write content for where a user must go to change incorrect financial institution details

Front end

[ ] Task (link)
[ ] Task (link)

Back end

[ ] Task (link)
[ ] Task (link)

Tracking moved to this consolidated issue:

https://github.com/cfpb/sbl-project/issues/259


Can't set value outside of ViewDidLoad

Hi,
I'm having a strange bug, I use your very nice module in a Objective-C project. I want to setup the value of ratings after a small API call and I noticed I can't change the value outside of ViewDidLoad.
I did some tests and even trying to set the value in viewDidAppear is not working.
Kind regards

Hi Ashernor,
Please share your code snippet that how you are trying to setting the ratingBar value in viewDidLoad.
For me, its simply working perfect by setting value as ratingBar.value = 3 in swift.
Regards,


Resized company logo

What changes were proposed in this pull request?

SnappyData Logo on UI is resized.
Displaying spark version in version details pop up.
Code/Files clean up.
Updated CSS

How was this patch tested?

Manually Tested.

These are minor changes.
Merging them without review.


Avoid direct calls to fwrite() and fputs().

This PR was originally going to resolve what looked like a bug on Windows re: Unicode support, but testing shows that it actually isn't needed. However, the overall code changes make FileHandle a bit nicer/less-redundant to use, so I'm opening a PR with them anyway. 🚀
Ensure terminal output on Windows is interpreted as Unicode.
Right now, we use fputs() to write to stderr on all platforms. On Linux and Darwin, this is fine because we can generally assume terminals use UTF-8 there (technically this is an unsafe assumption, but in practice it hasn't been an issue.)
On Windows, the console may not be configured with the UTF-8 pseudo-codepage (65001) and may instead still be using a locale-specific codepage like Latin-1 (1252).
Microsoft specifically recommends using the "w" functions when reading from or writing to the console. By doing so, we can ensure our output is correctly interpreted as Unicode.
Microsoft also documents that fputs() doesn't work with streams opened as Unicode, but fputws() works with ANSI and Unicode streams, so that's another reason to switch.
[One line description of your change]
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test

@swift-ci please test Linux

@swift-ci please test Linux

The Linux CI failure appears to be infrastructural. Tests pass at desk on all platforms, so I'm proceeding with the merge.

Follow-up: the Swift stdlib explicitly sets the output code page to UTF-8 (see _swift_stdlib_configure_console_mode()), so that explains that.


Remove Traefik Disclaimer

I am not sure what this disclamer relates to?
I asked in the Kubernetes Slack, and no one remembers why this is here,
nor has had any issues.
I cannot find any issues relating to bugs re. Traefik + Helm in the
Traefik repo.
If there is any specific reason for this message being here, it would
be great to get more details so we can resolve any issues upstream.

The disclaimer had mainly to do with ACME support not always working flawlessly. I'm not sure what the current state of that is.


<IP_ADDRESS> Intel Vision Skills NuGet Packages Broken / Attribution Text Files marked as 'Compile'

FYI:
There's a problem with the <IP_ADDRESS> 'Intel' Vision Skills packages that were published by Intel. (not yet tried to see if the <IP_ADDRESS> updates suffer same problem).
All the embedded attibution 'text' files appear to have been included with the 'Compile' build action - which prevents any samples from being built once you download the packages from NuGet. (VS2019 throws a bunch of compile errors as it tries to treach all these text files as code).
To fix this I had to go into the NuGet package cache folder and manually delete all the .txt files from those subfolders before I could compile anything.
As per above - just saw 1.0.03 has just been published (with new skills) - will check to see if it's still a problem there (can't find any release notes for your stuff so I'm unable to see what was fixed).
thanks
Niall

The issue was addressed in the new version of Intel's NuGet packages already published and there should be a new version of their samples available shortly. In the meantime simply update the NuGet package to the newer version

Thanks - I guess I need to manually clear old version from my NuGet package cache as just directly updating to <IP_ADDRESS> doesn't seem to have fixed the issue.  Looking forward to seeing the new samples being published - hopefully Intel can do better job of keeping it in sync with NuGet releases (<IP_ADDRESS> binaries now nearly a week old and samples still at <IP_ADDRESS>).  It would be a lot better if their samples could just be placed on GitHub repo so we can provide feedback too if needed (and going through their sign in system each time to get .zip file is a pitb).

Hi,
I agree, we suggested this as well earlier and to my knowledge this is in the work :). I will relay your feedback to Intel devs to reinforce the benefit of going through GitHub. Thx for trying it out and we appreciate any additional feedback

Some Addition Feedback  8)

Please ask them to add GPU Learning Device Model support - their stuff is quite slow without it - I think all their skills are CPU only right now.

Also - the amount of redistributables for their Skills is too large - they aren't models (which would be acceptable as it would be more comprehensive training data resulting in better Skill). It seems to just be supporting library dll's. I think there's something like 170mb of extra files just for a single Skill of theirs (not sure if they all share same ones) - which gets quite unmanageable for redistributing the App. So ask them to please try to get some lightweight version of this - these skills can't possibly need everything that's in those DLL's.

Also feedback for the Microsoft Vision Skills : the ObjectDetectorPreview is awesome - so much better than what we can produce in Custom Vision. One issue though is also the file size (model is 340Mb) - which again means each new version of our app is this much larger. Is there any way you to provide (in future release) a way to decouple the NuGet package from the model so it can be supplied via code when initialized (like the CV API does - ability to take an IStorageFile or Uri as model source). Our App is able to work with models downloaded into the Application Storage folders (for CV Models) - which makes redistribution of updates so much easier (smaller).

@mobilewares Intel GitHub: https://github.com/intel/Intel-AI-Skills
I'll reply to your feedback soon

Hi again,

For Intel Windows skills, I believe Intel integrated GPUs (Iris, Intel HD, etc.) are supported but not GPUs from other manufacturers for the moment. I did pass along that feedback for supporting other GPUs
I'll relay this feedback to Intel in our next meeting, app footprint is indeed important.
Glad you like the results you get from the object detector, indeed the model is big but note that if you distribute your app through the store, only differential updates are pulled in (not the whole app assets if unchanged at every update) see this blog entry
We could thinker around passing specified model files but we have to take into consideration why we do it this way for now as well (encrypt the model to avoid tempering, need to be usable offline, etc.)


Bug#1404 fixed Hilo concurrency error

Fixes concurrency error with HiloSequence generator in high concurrency case.

Added test that consistently reproduces the problem

Updated get_next_hi sproc update condition to include check that the current hilo value hasn't changed since the last select. If it did, then return -1

Updated transaction isolation to read committed.  We do not need to run in serialization isolation now because of #2.

Removed try..catch block surrounding sproc call. It's put in place before to address exceptions regarding concurrency issues in serialization isolation mode, hence no longer necessary.

Things for reviewer to look at

I'm not sure about is whether those who already have Marten running live would get the updated sproc (e.g. AutoCreate option is set to none).  How do we force sproc update?

It might be a better idea to retry getting next hi within the sproc.  Depends on maintainers' stance on this and sproc updates to deployments, we can implement it in this way instead

Throwing type Exception.  Should I create a new exception type for HiloSequence?

@mysticmind thanks. added commit

@jeremydmiller @oskardudycz @jokokko could you please review this PR?

@mysticmind @Miggleness I think this looks good to me. I like the optimistic concurrency check. Good call.

Looks legitimate to me.
@Miggleness: Regarding the question:

I'm not sure about is whether those who already have Marten running live would get the updated sproc (e.g. AutoCreate option is set to none). How do we force sproc update?

If the AutoCreate none is set then user should manage the updates manually and prepare the sql script. Currently, it's possible to do it with ToPatch method (see https://github.com/JasperFx/marten/blob/fcdecf7aadd889ed2e55cb421b166da8cda79295/src/Marten/Schema/IDocumentSchema.cs#L44)


added snake game pygame

Added the snake game for Hacktober fest using the pygame module

Added the snake game for Hacktober fest using the pygame module


[Question/suggestion] plugin download to build/ instead of src/

Is it possible to configure the plugin to put downloaded files in build/main/assets/<modid>/lang instead of src/main/assets/<modid>/lang?
I'd like use processResources { } to process my source lang file, and I don't want the local version with placeholder tokens to be overwritten by the version on crowdin with those tokens already replaced.
build/ also seems to me like a more fitting place for auto-downloaded files used in built jars, but that's just preference.

Sorry, there's no such option. You'd have to create a fork that replaces the rootDir string in line 414 of CrowdinTranslate.java.
I like to have the lang files in src/ where they normally belong, so I have them in git, can see their history, and also compile a mod when crowdin is down if necessary.


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2933 ms

Resolved: Samsung performance has improved in 18c7b33.


Provide script to setup ad-hoc Metastore per branch

User story: users want to define a Spark cluster per branch, so that the runtime environment controls which branch an end-user is working with. User should be able to use the same Hive table names and schemas regardless of the branch being used, to allow for code and query reuse.
Proposed solution: Provide a script that starts a containerized Hive Metastore and does the following:

Accept a source metastore address
Accept source and target branch names
Accept an optional filter for schema and table names
Scan source metastore for all tables that meet the filter
For each table, create a corresponding table in the newly created Metastore, rewriting paths to use the target branch instead.

The resulting metastore should contain the exact same set of tables as the source, only mapped to the target branch instead.
Note: this is an interim solution, to be deprecated when Metastore support in lakeFS improves.

A repository containing the proposed script and docker-compose could be found here


Allow passing options to the Slack API

fixes #9
[PR to #15 to avoid conflicts]

I'll fix conflicts + merge + tag on Monday


AWS: Configure SSL certificate alternate-names

GCE does this in its per-provider scripts; this does the same for AWS and lets
other providers do the same; I believe kube2sky requires <IP_ADDRESS> as a SAN.

This is supposed to be the AWS equivalent of #10107.  Hence cc @roberthbailey
kube2sky was failing to contact the API without this (it was getting an certificate not valid for IP error).  I believe this is now required on all clouds that use kube2sky?  (@thockin is that true? - I saw some comments from you on other kube2sky issues that suggest you understand this area; I'm not really sure why this is broken now but wasn't broken previously)

We we did this for GCE, I did not realize it was not the same for all platforms.  yeah, this is the right way to deal with this, as far as I know.

LGTM

@roberthbailey @quinton-hoole

I think this actually solves the problem for all of the other salt-based providers. Thanks.

@roberthbailey I think that (sadly) the other providers still need to do the same work as I did in cluster/aws/..., but hopefully that should be straightforward (hopefully a copy-and-paste and then translate to their variable names, parameter passing style etc).

True, but I think it's just a matter of setting a few extra variables now that you plumbed them through to the script that generates the certs.

LGTM.


ASCII control characters escaping in error messages

Line feed literals, carriage returns, and tabs are all printed as their actual characters rather than the escaped sequence in error messages.
Note in the reproducer that a new line seperates '1' and '2', a sequence of spaces seperates '3' and '4', and that '6' has been printed at the start of the line as the terminal's cursor has interpreted the carriage return to overprint text.
Reproducer:
(sphinx) PS S:\Development\sphinx> ruff -V
ruff 0.0.199
(sphinx) PS S:\Development\sphinx> type bug.py
if token == "1\n2":
    pass

if token == "3\t4":
    pass

if token == "5\r6":
    pass
(sphinx) PS S:\Development\sphinx> ruff bug.py --select S105
bug.py:1:13: S105 Possible hardcoded password: `"1
2"`
bug.py:4:13: S105 Possible hardcoded password: `"3      4"`
6"`.py:7:13: S105 Possible hardcoded password: `"5
Found 3 error(s).

A

Sidenote: We do not want to escape all error message arguments: e.g. the messages (.msg) of the new banned-api setting should be interpolated into the error message without being escaped (so that they can contain newlines).


more consistent assigner

The assign function in every assigner is more consistent with the new label indexing (0-based) method.

doc strings are modified for the consistency
base, atss, max_iou, approx_max_iou and point assigners are all modified
the background_label will be passed to the assign function at all Anchorhead when it is called.

Can you merge the newest V2.0 and see whether the changes pass the CI?


Create module for logging and log to console more often

can we make the loudLogger the default?

We could change the main functions of generate-packages, parse-definitions, and publish-packages to use loud logging. package-generator and package-publisher should use quiet logging because these functions are run in parallel with others. How do you mean to make it a default though?

in stead of requiring the loudLogger from the package, you just require logger, and this is by default is loud. but i see your point about the parallel operations.

👍


Migration locks: lock_for_migrations

I noticed that lock_for_migrations doesn't have an implementation.
Am I correct in concluding that the Clickhouse adapter does not support running migrations on multiple nodes?
If not, are there plans to support this? Such as using a table in Clickhouse for locking purposes?
If there are no plans, why not? Would a PR for this be accepted?

👋 @ecarlson94

Such as using a table in Clickhouse for locking purposes?

How would it work?

I don't know if this is a good strategy. You'll have to tell me if you see something wrong with it. But I was thinking a table with one column that only ever has one record in it. Here is some sudo code for the lock_for_migrations function:
def lock_for_migration(_meta, _options, f) do
  guid = random guid
  record = "SELECT TOP 1 guid FROM mirgation_lock"

  if record exists do
    repeat on interval with retry policy
  else
    "INSERT #{guid} INTO migration_lock"
    f.()
  end

  "DELETE * FROM migration_lock WHERE guid = #{guid}"
end

Or something similar, but I think this gets the idea across.

Would you be able to explain it further? I don't understand what exactly prevents race condition here.

I was thinking the existence of the record would indicate that a lock already exists. I guess, technically, there could still be a race condition if the second node performs the SELECT before the first node inserted its guid into the lock table. Though highly unlikely. This could be mitigated if there is a way to see if the record exists and insert one if not within the same transaction. But I'm not sure how that could be accomplished with Clickhouse. Any ideas?

Any ideas?

I don't think there is much we can with no transactions support in ClickHouse. Distributed DDL is already possible however, with ON CLUSTER clause. And Sidtributed data migrations can be done semi-manually, like in Plausible.

I'm going to close it for now. We can revisit it if ClickHouse starts supporting transactions.


[FYI] Recent models are not supported

Hi, thanks for developing interesting module 😃
This is just for your information.
I have tried to automate KI-PX* series, but recent models cannot be operated from the COCORO AIR app and must use the COCORO HOME app: https://jp.sharp/support/cloud/air/air_purifier.html
The API endpoint for COCORO HOME app is https://cocoroplusapp.jp.sharp instead of https://hms.cloudlabs.sharp.co.jp, in addition, the whole process of issuing JSESSIONID seems to be different, therefore your module could not be used for recent models.
Fortunately, the COCORO HOME application is partially a Web application and can be operated from a browser on a PC, so we can easily inspect HTTP communications by developer tools: https://cocoroplusapp.jp.sharp/air/welcome
On my side, to automate my air purifier using Python, I wrote the code with Selenium and requests (It works well, but it is made in an unmannerly way that requires logging in with ID and password every time, so I will not make it available to the public 😞).
If you are interested, try COCORO HOME. Please feel free to close this issue. Thanks!

Thanks.
Yes, I also have a new humidifying air purifier that only works with COCORO HOME, so I can't manage it with this tool.
However, recently I've been leaving it on all the time, including the older models, so I haven't updated this tool.
If there is a demand or if I feel like using it myself, I might update it sometime.
I will leave this issue open as a reminder for the support of COCORO HOME.


Add support for DISPLAY_CYD2USB

Like CYD but with ST7789V

Thanks for the PR. I'm in the middle of a refactoring and need to change your code to fit the API changes. I'll add the entry with the name DISPLAY_CYD_2USB to match the other CYD names.


0.3.6 doesn't patch a clustered register as 0.2.8 did

My svd has a peripheral PWM with a cluster SM and register SMOCTRL
In 0.2.8 I could patch fields in SMOCTRL with...
PWM?:
  SMOCTRL:
    _clear:
      - PWMXFS
      - PWMBFS
      - PWMAFS

With 0.3.6 it can't find the SWOCTRL register and its unclear to me how to select it (in the cluster SM)
Actual yaml patch
https://github.com/imxrt-rs/imxrt-ral/blob/master/devices/imxrt1176_cm7.yaml#L105
Actual SVD
https://github.com/imxrt-rs/imxrt-ral/blob/master/svd/imxrt1176_cm7.svd

Could you try to find first breaking release/PR?
What exactly is the error?
Also with last release you could run
svdtools patch --show-patch-on-error your_patch.yaml

to see full patch with all _includes merged.

Oh. I see. It is https://github.com/rust-embedded/svdtools/issues/144 related.
Forgot about it.
Should be easy to fix.

With 0.3.6 it can't find the SWOCTRL register and its unclear to me how to select it (in the cluster SM)

Should be something like this. Although I've never used it.
PWM?:
  _clusters:
    "SM[%s]":
      SMOCTRL:
        _clear:
          - PWMXFS
          - PWMBFS
          - PWMAFS

When I try this with 0.3.6
PWM?:
  _clusters:
    "SM[%s]":
      SMOCTRL:
        _clear:
          - PWMXFS
          - PWMBFS
          - PWMAFS

I get
[2023-12-19T04:23:51Z ERROR svdtools::cli] by svdtools (0.3.6)

    Caused by:
        0: Processing device `MIMXRT1176_cm7`
        1: According to `PWM?`
        2: Processing peripheral `PWM1`
        3: According to `SM[%s]`
        4: Could not find `PWM1:SM[%s]. Present clusters: SM[%s].`

That's pretty confusing to me I have to say!

Try to escape brackets:
SM\[%s\]

Hi @burrbull @SpinFast
SM\[%s\] only works on Linux, but failed on Windows with same version 0.3.8

Looks like we need to enable backslash_escape feature. See https://docs.rs/globset/latest/globset/#syntax.
Make PR, please, if you can.

Just wanted to pop in here to say to get this to work on macos, i had to double escape the brackets:
I2C0:
  _clusters:
    "I2C0_GPRCM\\[%s\\]":
      _strip:
        - "I2C0*"


Fatal error: grunt.sprite requires a src, dest (img), and destCss property

Hi,
I am running the grunt task config as defined in your example and am getting "Running sprite:build(sprite) task Fatal error: grunt.sprite requires a src, dest (img), and destCss property". I noticed that the example given for grunt-spritesmith has the properties it is looking for each on that JSON object. Do you know why I am getting this error?
Thanks,
Joe

Hi,
It's because spritesmith was updated recently and some options was changed. I'll update my mixins and example config soon.
Thanks for noticing.

Great. Thanks.

I've fixed mixins for the latest grunt-spritesmith version. Also grunt options changes a little. Please check new options in example or readme.


Wrong number of NA rows in the output?

Hi,
first of all thank you for this great tool.
If I run skimpy on this 999 rows CSV I have 1000 NA rows.
Thank you

Huh, that's strange! I will look into it. Before, I do, can I just check it's not the case that one row is being counted as the column header (ie there are 1000 rows in the csv file but one of those is being used as the column names so only 999 rows of data)?

This is occurring because _round_dataframe applies to all numeric columns, regardless of whether or not they are integers. This has some unfortunate consequences, such as rounding 988 to 990, 999 to 1000, and so on. As a solution, rounding should probably only be applied to specific columns that need it, ie it should be applied within the type-specific functions (eg _string_variable_summary_table) to only the relevant columns.

I didn't remember this issue
Thak you very much @aeturrell


docs: add missing ci_activity to Notification Reasons

Why:
Closes https://github.com/github/docs/issues/10578

What's being changed:
This PR adds a missing possible ci_activity notification reason to the relevant section of the Notifications REST API documentation along with its description.
Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@codebytere Thanks for opening this PR ✨ I have linked this PR with an issue that was opened internally 🐬

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
I have added the 'writer impact' label

I have added the 'writer impact' label


Subtractive lighting not supported

Reporting a bug/missing feature:
When using Subtractive lighting https://drive.google.com/uc?id=1r5fO4GJOwFskzgW2L4UOHaXcUEYRrB36, the toon shader from this Github doesn't receive the Subtractive shadow as demonstrated here https://drive.google.com/uc?id=1EWzjEFLrqJQTN0HGQgcFCnqUr0LwEI4A
My project uses Subtractive lighting so would be great if this feature could be supported!

Hey there! I've updated the main (URP v12+) branch to add support for this~
(I've also added the SubtractiveGI_float function to the older branch (for v8 - v11) in case anyone needs it, but that's probably as much as I'll do there)
I'm not that familiar with Baked GI (hence why this wasn't included earlier) so hopefully I've handled it correctly (seems to work okay on my end now at least!). But let me know if there's any issues.
Thanks!


Tokio 1.0.0

Unable to complete this task at this time.
Major blockers appear to by tonic and warp:
https://github.com/hyperium/tonic/issues/443
https://github.com/seanmonstar/warp/pull/753

Oh are we using warp here? Starting to lose track of dependencies

Oh are we using warp here? Starting to lose track of dependencies

We decided to use Warp for the http server. We could potentially change frameworks if it looks like it is not being actively maintained.

We decided to use Warp for the http server. We could potentially change frameworks if it looks like it is not being actively maintained.

@kflansburg Oh no, warp is great! We use it over in bindle too. I just forgot we used that for our server

@kflansburg Oh no, warp is great! We use it over in bindle too. I just forgot we used that for our server

Bumped the Tonic version to 0.4 and Prost to 0.7.
Warp appears to be very near cutting 0.3: https://github.com/seanmonstar/warp/pull/777
I suspect that the Warp server encounters a runtime error due to the differing Tokio versions.
Finally, now that the providers compile, I have identified wascc-httpsrv and its dependency actix-codec to require Tokio 0.2 still. actix-codec appears to have a beta release out for 0.4 which will update to Tokio 1.0, however I'm not sure how quickly wascc-httpsrv will update. @thomastaylor312

Bumped the Tonic version to 0.4 and Prost to 0.7.
Warp appears to be very near cutting 0.3: https://github.com/seanmonstar/warp/pull/777
I suspect that the Warp server encounters a runtime error due to the differing Tokio versions.
Finally, now that the providers compile, I have identified wascc-httpsrv and its dependency actix-codec to require Tokio 0.2 still. actix-codec appears to have a beta release out for 0.4 which will update to Tokio 1.0, however I'm not sure how quickly wascc-httpsrv will update. @thomastaylor312

@thomastaylor312 I may not be able to fix the Windows errors myself. Having trouble getting cross compiling to work.

@thomastaylor312 I may not be able to fix the Windows errors myself. Having trouble getting cross compiling to work.

I can help out with any windows problems once we get everything updated. We can talk to the ~wascc~ WasmCloud people about tokio 1.0

I can help out with any windows problems once we get everything updated. We can talk to the ~wascc~ WasmCloud people about tokio 1.0

Ok, so we can probably help upgrade or at least open an issue here once actix has a full release: https://github.com/wasmCloud/capability-providers

Ok, so we can probably help upgrade or at least open an issue here once actix has a full release: https://github.com/wasmCloud/capability-providers

With Warp 3 released, I think the only blocker is now Actix 4 and the subsequent wasmCloud update.
@thomastaylor312 I think that stuff is orthogonal to the Windows updates, which are mostly related to the CSI modules.

With Warp 3 released, I think the only blocker is now Actix 4 and the subsequent wasmCloud update.
@thomastaylor312 I think that stuff is orthogonal to the Windows updates, which are mostly related to the CSI modules.

@thomastaylor312 You should be good to address with Windows issues. It looks like there is something preventing E2E tests from finishing, so I will investigate this tomorrow.

Not sure what is going on with the e2e tests. They complete locally with no issues.

Found another location where there's a blocking call: https://github.com/deislabs/krustlet/blob/33ad0846f4717c2d97939ed0bf4d42f1c235f641/crates/kubelet/src/volume/persistentvolumeclaim.rs#L248
Should probably be replaced with tokio::fs::remove_dir_all(path).await?; so that the thread can yield back to the runtime.


Please add NtCreateProcess to NTDll

Is your feature request related to a problem? Please describe.
No
Describe the additions or enhancements you'd like
I'd like NtCreateProcess to be added to NTDll
And thanks for this very nice OSS project!

I have no way of testing these definitions, but I've created the following to match, best as I can guess, what I saw in your link and some others. If you can test these and ensure they work, I'll include them in the library.
[DllImport(Lib.NtDll, SetLastError = false, ExactSpelling = true)]
public static extern NTStatus NtCreateProcess(out HPROCESS ProcessHandle, [In] ACCESS_MASK DesiredAccess, in OBJECT_ATTRIBUTES ObjectAttributes,
	[In] HPROCESS ParentProcess, [In, MarshalAs(UnmanagedType.U1)] bool InheritObjectTable, [In, Optional] IntPtr SectionHandle,
	[In, Optional] IntPtr DebugPort, [In, Optional] IntPtr ExceptionPort);

[DllImport(Lib.NtDll, SetLastError = false, ExactSpelling = true)]
public static extern NTStatus NtCreateProcess(out HPROCESS ProcessHandle, [In] ACCESS_MASK DesiredAccess, [In, Optional] IntPtr ObjectAttributes,
	[In] HPROCESS ParentProcess, [In, MarshalAs(UnmanagedType.U1)] bool InheritObjectTable, [In, Optional] IntPtr SectionHandle,
	[In, Optional] IntPtr DebugPort, [In, Optional] IntPtr ExceptionPort);

///
[Flags]
public enum PROCESS_CREATE_FLAGS : uint
{
	///
	PROCESS_CREATE_FLAGS_BREAKAWAY = 0x00000001,
	///
	PROCESS_CREATE_FLAGS_NO_DEBUG_INHERIT = 0x00000002,
	///
	PROCESS_CREATE_FLAGS_INHERIT_HANDLES = 0x00000004,
	///
	PROCESS_CREATE_FLAGS_OVERRIDE_ADDRESS_SPACE = 0x00000008,
	///
	PROCESS_CREATE_FLAGS_LARGE_PAGES = 0x00000010,
}

[DllImport(Lib.NtDll, SetLastError = false, ExactSpelling = true)]
public static extern NTStatus NtCreateProcessEx(out HPROCESS ProcessHandle, [In] ACCESS_MASK DesiredAccess, in OBJECT_ATTRIBUTES ObjectAttributes,
	[In] HPROCESS ParentProcess, [In] PROCESS_CREATE_FLAGS Flags, [In, Optional] IntPtr SectionHandle,
	[In, Optional] IntPtr DebugPort, [In, Optional] IntPtr ExceptionPort, uint JobMemberLevel);

[DllImport(Lib.NtDll, SetLastError = false, ExactSpelling = true)]
public static extern NTStatus NtCreateProcessEx(out HPROCESS ProcessHandle, [In] ACCESS_MASK DesiredAccess, [In, Optional] IntPtr ObjectAttributes,
	[In] HPROCESS ParentProcess, [In] PROCESS_CREATE_FLAGS Flags, [In, Optional] IntPtr SectionHandle,
	[In, Optional] IntPtr DebugPort, [In, Optional] IntPtr ExceptionPort, uint JobMemberLevel);

Yep, these work 👍


DEP-2632 - Display Mendix SSO constants in Developer Portal

Asked 6/7
Written 7/7
Informed 9:45 7/7 that this is not now being released in current release.
Added flag for Future Release.

After discussion with the Identity Services team this change was not released to production.


feat: add pnr validators (refs KDK-48741)

Adding validators for pnr:

notSamePnrValue - Check that one pnr field value isn't the same as another.
compareAgePnrValue - Check that one pnr field value isnt't older/younger than another.

Validatorerna behöver dokumenteras här:
https://forsakringskassan.github.io/designsystem/latest/components/validation/validators.html


Responsive sprites (relative positions using percentages)

PORTED from css-sprite issues.
Thanks for this nice tool.
It would be great if you extend it with an option to use relative positioning. Here is the bash code I use to post process your output (NOTE: this version is only for vertical sprites, but supporting horizontal ones should be straightforward):
!/bin/bash
IFS='
';
tech_re="^(vertical|horizontal)-(background|imgspacer)$";
if [ $# -ne 2 -o echo $2 | grep -c $tech_re -le 0 ];then
echo "ERROR: Two parameters required. CSS-file and Orientation-Technique.";
echo -e "\tCSS-file with absolute positions in background-position";
echo -e "\tOrientation-Technique: $tech_re";
exit -1;
fi
numimages=grep -c "background-position" $1;
orientation_technique=$2;
orientation=echo $technique | cut -f 1 -d"-";
technique=echo $technique | cut -f 1 -d"-";
numincrements=$((numimages-1));
percentage=bc <<< "scale=4; 100/$numincrements";
count=0;
for line in cat $1;do
if [[ $line =~ .background-position. ]];then
echo "$count>> $line" >&2;
curr_rel_percentage=bc <<< "scale=4; $count*$percentage";
if [[ $count -eq 0 ]];then
echo -e "\tbackground-position: 0% 0%;";
else
#echo -e "\tbackground-position: 0% -${count}00%;"; for imgspacer
echo -e "\tbackground-position: 0% $curr_rel_percentage%;";
fi
count=$((count+1));
else
echo "$line";
fi
done
Let me know if I can help.

Hey @hllorens could you please have a look at sprity's how to write style processors and see if all values you would need to do that with a style processor are available? If so you could it is probably easier to write one for you then it was writing the bash script ;-)

Hey @aslansky. The code is quite cryptic to me. What I want to modify is background position (as you can see above). Could you show me a very simple example for me to understand? e.g., imagine I want just add 1 to each Y value.
If I can understand how to do that, I will probably be able to do the rest.
This responsive alternative is quite useful (note that most websites are migrating to more responsive alternatives), maybe it could be added as another style processor in sprity. What do you think?
Next question will post it in sprity.
Thanks in advance for your help
Hector

@hllorens could you please post a short example output of your script. Sadly I am not really a css guy and never heard of using percentages till now. So an example would make it much easier to understand what the output should look like.

Sure, here it is.
Original (not responsive):
.wordimage {
background-image: url('../../../afan-app-media/img/wordimg-sprite.png');
}
.wordimage-lluvia {
background-position: -4px -4px;
width: 150px;
height: 150px;
}
.wordimage-abeja {
background-position: -4px -162px;
width: 150px;
height: 150px;
}
...
Modified (responive):
.wordimage {
background-image: url('../../../afan-app-media/img/wordimg-sprite.png');
background-size: 100% auto;
}
.wordimage-lluvia {
background-position: 0% 0%;
}
.wordimage-abeja {
background-position: 0% .6289%;
}
...
The operation applied to calculate the percentage is:
increment=$((numimages-1));
percentage=100/increment;
count=0;
for each image in a vertical sprite {
rel_percentage=count*percentage;
}
As you can see width and height are no longer needed since size is relative.
Thanks in advance!

This looks fantastic. Any more progress with this since June?

Sadly I had neither enough knowledge to implement this in sprity, nor the time to get such knowledge.
It would be really great if someone could help. Could you @jakeg ?
I know what needs to be done, but now how.
For now I have a bunch of sh scripts that I run after sprity to get the responsive output.

This seems to be feasible using "only" custom templates (see https://www.npmjs.com/package/sprity#templates), as long as you output something to be parsed by a CSS pre (or post) processor (i.e. leaving the math to the preprocessor, as Handlebars doesn't allow you to have logic in templates).
This is just a quick prototype I did, using the Sprity CSS template as the basis:
{{#each layouts}}
	{{#each sprites}}

		{{#if dpi}}
		@media (-webkit-min-device-pixel-ratio: {{ratio}}), (min-resolution: {{dpi}}dpi) {
		{{/if}}
		.{{cssesc ../classname}} {
			background-image: url('{{escimage url}}');
			{{#if dpi}}
			background-size: {{baseWidth}}px {{baseHeight}}px;
			{{/if}}
		}
		{{#if dpi}}
		}
		{{/if}}
	{{/each}}

	{{#each layout.items}}
	.{{../classname}}-{{meta.name}} {
		background-position: percentage({{baseDim x}}/({{../layout.width}}-{{baseDim width}})) percentage({{baseDim y}}/({{../layout.height}}-{{baseDim height}}));
		width: {{baseDim width}}px;
		height: {{baseDim height}}px;
		background-size: percentage({{../layout.width}}/{{baseDim width}}) percentage({{../layout.height}}/{{baseDim height}})
	}
	{{/each}}
{{/each}}

In this scenario, the template outputs a LESS file which should then be processed to yield the actual percentage values.

I had more luck with this:
`{{#each layouts}}
{{#each sprites}}
    {{#if dpi}}
    @media (-webkit-min-device-pixel-ratio: {{ratio}}), (min-resolution: {{dpi}}dpi) {
    {{/if}}
    .{{cssesc ../classname}} {

        position:relative;
        {{#if dpi}}
        background-size: {{baseWidth}}px {{baseHeight}}px;
        {{/if}}
    }
    .{{cssesc ../classname}}:after {
            content:"";
            display: block;
            height: 100%;
            position: absolute;
            top: 0;
            left: 0;
            width: 100%;
            background-image: url('{{escimage url}}');
        }
    {{#if dpi}}
    }
    {{/if}}
{{/each}}

{{#each layout.items}}
.{{../classname}}-{{meta.name}} {
    padding-bottom: percentage( {{baseDim height}}/{{baseDim width}});
}
.{{../classname}}-{{meta.name}}:after {
    background-position: percentage({{baseDim x}}/({{../layout.width}}-{{baseDim width}})) percentage({{baseDim y}}/({{../layout.height}}-{{baseDim height}}));
    background-size: percentage({{../layout.width}}/{{baseDim width}}) percentage({{../layout.height}}/{{baseDim height}});

}
{{/each}}

{{/each}}
`


vi modal indicator

Readline supports the vi editing mode, with these options in ~/.inputrc
set editing-mode vi
set show-mode-in-prompt on
set vi-cmd-mode-string :
set vi-ins-mode-string +

The show-mode-in-prompt feature is a really helpful bit of user feedback for a modal editing system like Vi.  You can test this feature by writing the configuration above, and running Bash.

I think a good way would be to follow NVim's way: using a line cursor ('\033[6 q') for insert and emacs mode and switch to a block cursor for command mode.

I disagree @AdminXVII.  The issue is that not all terminals support cursor modes.
The default behavior of iPython (which doesn't use readline anymore) is to show [ins] in insert mode, and [nav] in normal/navigation mode.  I think that's pretty good and self explanatory.

I should also say that the default indicators of readline are (cmd) for normal/command mode and (ins) for insert mode.  Maybe it's better to use the readline defaults given the library's lineage.

FWIW this is the config I use for IPython, to get cursors:
   3   │ import sys
   4   │ from prompt_toolkit.key_binding.vi_state import InputMode, ViState
   5   │
   6   │
   7   │ def get_input_mode(self):
   8   │     return self._input_mode
   9   │
  10   │
  11   │ def set_input_mode(self, mode):
  12   │     shape = {InputMode.NAVIGATION: 2, InputMode.REPLACE: 4}.get(mode, 6)
  13   │     raw = u'\x1b[{} q'.format(shape)
  14   │     if hasattr(sys.stdout, '_cli'):
  15   │         out = sys.stdout._cli.output.write_raw
  16   │     else:
  17   │         out = sys.stdout.write
  18   │     out(raw)
  19   │     sys.stdout.flush()
  20   │     self._input_mode = mode
  21   │
  22   │
  23   │ ViState._input_mode = InputMode.INSERT
  24   │ ViState.input_mode = property(get_input_mode, set_input_mode)
  25   │ c.TerminalInteractiveShell.editing_mode = 'vi'
  26   │ c.TerminalInteractiveShell.prompt_includes_vi_mode = False

The issue is that not all terminals support cursor modes.

Well can't we have a graceful fall-back for those that don't?  Or allow this as a configuration option somewhere?

Just for reference:

https://github.com/judah/haskeline/pull/78
https://github.com/prompt-toolkit/python-prompt-toolkit/issues/192

For reference, replxx uses a callback to update prompt dynamically:
https://github.com/AmokHuginnsson/replxx/blob/master/examples/cxx-api.cxx#L228-L232


Added Yellow to community projects

The plugin uses Twemoji v2.0.0 by Twitter, which supports about 800 colorful images.

I think the current count is about 1600 emoji ;-)

Thank you, fixed it right away  :smile:


[FLINK-34707][tests] Update base version for japicmp check

This is japicmp configuration update for master branch

@JingGe @Myasuka @MartijnVisser  Can someone of you help review this?

@JingGe @Myasuka @MartijnVisser Can someone of you help review this?

@Myasuka Thanks for reviewing! Please help review the cherry-pick one: https://github.com/apache/flink/pull/24514


Touch Bar Server "crashes" touchbar

When I launch the touch bar server app my touch bar goes black and doesn't display anything. When I hit fn none of the function keys show up either.

yeah, latest update has been since 2015. don't think it's gonna work on this version of macosx.
the least version that works with this is el capitan (10.11)

No, the Touch Bar only exists since 2016 and macOS 10.12 Sierra

okay, well it's just not gonna work on any modern version of macosx.


pull request for e2e test at 02/05/2017 05:56:09

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


suggested rarefied level

From @clozupone. Rewriting: add a way to know the sequences per sample in a meta-analysis before defining the rarefaction level.

What if you store the number of sequences per sample in the database, and then when the user selects their samples you could display the histogram of number of sequences per sample? To assist the user in making their choice, you could annotate (e.g.) the 10th, 20th, 25th, 50th, 75th, 80th, 90th percentiles in that distribution and note what fraction of their samples they'd discard with each level.

Great idea!
On Mar 11, 2015, at 7:23 PM, Greg Caporaso <notifications<EMAIL_ADDRESS>wrote:
What if you store the number of sequences per sample in the database, and then when the user selects their samples you could display the histogram of number of sequences per sample? To assist the user in making their choice, you could annotate (e.g.) the 10th, 20th, 25th, 50th, 75th, 80th, 90th percentiles in that distribution and note what fraction of their samples they'd discard with each level.
—
Reply to this email directly or view it on GitHubhttps://github.com/biocore/qiita/issues/650#issuecomment-78412691.

Just an extension, I like the thought of storing within the BIOM table
similar to how we store stats in the demux files. Since we're pending a
minor release for biom, we could actually just get this lumped in as a new
option to the table summarizer (--save-stats). Fetching the stats would not
require a parse of the table, just opening the h5py file. However, storing
in the db is light as well, but it puts the stats further away from the
actual data they represent.
On Wed, Mar 11, 2015 at 8:27 PM, Rob Knight<EMAIL_ADDRESS>wrote:

Great idea!
On Mar 11, 2015, at 7:23 PM, Greg Caporaso<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
What if you store the number of sequences per sample in the database, and
then when the user selects their samples you could display the histogram of
number of sequences per sample? To assist the user in making their choice,
you could annotate (e.g.) the 10th, 20th, 25th, 50th, 75th, 80th, 90th
percentiles in that distribution and note what fraction of their samples
they'd discard with each level.
—
Reply to this email directly or view it on GitHub<
https://github.com/biocore/qiita/issues/650#issuecomment-78412691>.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/650#issuecomment-78413026.

I have been thinking about this and I can see the benefits of the described approaches. However, thinking pragmatically about this, the histograms need to be good but not prefect. With this in mind, what about adding a step in the processing step (otu picking) so a default histogram is generated and we add this information to the process data table as an array or ints? The default histogram can be from 0->50K (we can discuss this or we can create a histogram of all the current biom tables so we get a better range) with 100 or 1K points. With this approach we can quickly display the histogram of one biom but more importantly can show histograms of each process data and the combination of all for metaanalyses. Finally, this solution is really easy to implement and has minimal DB changes. What do you think?

This should break down to:
Table = load_table(foo)
Res = hist(table.sum(axis='sample'))
Recommend storing the bar positions and height instead of a PDF so that the
visual can be changed in UI if needed, but path of least resistance is just
generate the image for display
On Oct 23, 2015 07:59, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:

I have been thinking about this and I can see the benefits of the
described approaches. However, thinking pragmatically about this, the
histograms need to be good but not prefect. With this in mind, what about
adding a step in the processing step (otu picking) so a default histogram
is generated and we add this information to the process data table as an
array or ints? The default histogram can be from 0->50K (we can discuss
this or we can create a histogram of all the current biom tables so we get
a better range) with 100 or 1K points. With this approach we can quickly
display the histogram of one biom but more importantly can show histograms
of each process data and the combination of all for metaanalyses. Finally,
this solution is really easy to implement and has minimal DB changes. What
do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/650#issuecomment-150598761.

Yes, simple addition. We could store positions and height with the default 10 values and on merge we simply display the combination. Note that this might not work well when you start filtering down samples by metadata ...

I've got code for this (and for making the plots interactive) - code here
https://github.com/gregcaporaso/q2d2/blob/master/q2d2/__init__.py#L265,
example image attached. Feel free to use - just throw an attribution in as
a comment.
[image: Inline image 1]
On Fri, Oct 23, 2015 at 8:09 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:

Yes, simple addition. We could store positions and height with the default
10 values and on merge we simply display the combination. Note that this
might not work well when you start filtering down samples by metadata ...
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/650#issuecomment-150603489.

Sounds great!
On Oct 23, 2015, at 9:09 AM, Antonio Gonzalez <notifications<EMAIL_ADDRESS>wrote:
Yes, simple addition. We could store positions and height with the default 10 values and on merge we simply display the combination. Note that this might not work well when you start filtering down samples by metadata ...
—
Reply to this email directly or view it on GitHubhttps://github.com/biocore/qiita/issues/650#issuecomment-150603489.

Solved in https://github.com/biocore/qiita/pull/1738.
Now, every artifact has it's own summary. For example a biom has:

Which should help you decide your rarefaction level.


oddjob: init at 0.34.7

Description of changes
The odd job dameon for dbus. https://pagure.io/oddjob
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Bump for visibility. Waiting on this for realmd

@s1341 Please can I get write access to this branch? (assuming you are no longer wanting to work on this)

@SohamG you can always branch off of their fork and create your own PR. In that case you should keep the existing commits to retain credits to the original author of this PR, and can commit your own changes seperately.

Apologies. I don't have time to get to this at the moment. @SohamG if you still want write perms, let me know.

@s1341 Yes please can I have write perms?


PathVerifier fails on whitespace filename under Linux and .NET Core

Describe the bug
When using FileSystem and MockFileSystem I noticed there is different behavior when creating a new IFileInfo using only whitespaces.
The "error" trails down to the PathVerifier that the MockFileSystem uses
To Reproduce

// Both throw in Linux and windows
var mfs = new MockFileSystem();
mfs.FileInfo.New("   "); // ASCII space
mfs.FileInfo.New("\u00A0"); // Unicode char that's treated as whitespace

// Both pass in Linux
mfs.FileInfo.New("./   ") // Explicit relative path
mfs.FileInfo.New("/   ") // Explicit absolute path

// Passes in Linux
var fs = new FileSystem();
fs.FileInfo.New("   "); // ASCII space

// Passes in Linux and Windows (.NET Core); throws on NET4.8
fs.FileInfo.New("\u00A0"); // Unicode char that's treated as whitespace

Expected behavior
Both calls throw an exception when they should not.
For Linux   (whitespace) is a valid file name
For Windows and Linux \u00A0 is a valid file name. However under .NET Framework it also fails
Additional context
In PathInternal.OS.cs .NET has a bool IsEffectivelyEmpty(string? path) method which behaves differently to linux and windows, just to cover those cases.
Currently PathVerifier uses the string.Trim() method, which causes the two explained scenarios

valid unicode chars get trimmed (\u00A0)
a whitespace-only relative path is considered to be illegal when it actually is not under linux.

Thanks for reporting! Happy to accept a PR to fix this 🙇


Fix 404 initial load

Fixes an issue where the 404 would "flash in" on the initial project load (closes https://github.com/jbx-protocol/juice-interface/issues/1167)
Fixes it by defaulting v2 contract loader "loading" state to true.
Also adds a skeleton loader for the Reserved Tokens number.

@johnnyd-eth  🏓

Beautiful fix here ser. LGTM


[Task] Replace require statements with imports - dependencies.js

this file doesn't actually have any require statements


WebSocket is closed before the connection is established.

I tried to run port 6001 on my local and it worked but not work in production.
I am using Cloudflare CDN with Flexi SSL configuration.
I had also double check that I allowed port 6001 in my firewall and it is running.

===========================================================================================
Here is my settings:
websockets.php
'apps' => [
[
'id' => env('PUSHER_APP_ID'),
'name' => env('APP_NAME'),
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'path' => env('PUSHER_APP_PATH'),
'capacity' => null,
'enable_client_messages' => false,
'enable_statistics' => true,
],
],
'ssl' => [
'local_cert' => null,
'local_pk' => null,
'passphrase' => null,
],
broadcasting.php
'pusher' => [
'driver' => 'pusher',
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'app_id' => env('PUSHER_APP_ID'),
'options' => [
'cluster' => env('PUSHER_APP_CLUSTER'),
'encrypted' => false,
'host' => '<IP_ADDRESS>',
'port' => 6001,
'scheme' => 'http',
'useTLS' => false,
'curl_options' => [
CURLOPT_SSL_VERIFYHOST => 0,
CURLOPT_SSL_VERIFYPEER => 0,
]
],
],
client side
window.Echo = new Echo({
broadcaster: 'pusher',
key: "mykey",
cluster: "mt1",
wsHost: window.location.hostname,
wsPort: 6001,
wssPort: 6001,
disableStats: true,
enabledTransports: ['ws', 'wss'] // <- added this param
})

I have same problem,
any problem when i try to connect from "http" client website .. but  this problem occurred when i try to connect with "https" client website..

I tried to run port 6001 on my local and it worked but not work in production.
I am using Cloudflare CDN with Flexi SSL configuration.
I had also double check that I allowed port 6001 in my firewall and it is running.

===========================================================================================
Here is my settings:
websockets.php
'apps' => [
[
'id' => env('PUSHER_APP_ID'),
'name' => env('APP_NAME'),
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'path' => env('PUSHER_APP_PATH'),
'capacity' => null,
'enable_client_messages' => false,
'enable_statistics' => true,
],
],
'ssl' => [
'local_cert' => null,
'local_pk' => null,
'passphrase' => null,
],
broadcasting.php
'pusher' => [
'driver' => 'pusher',
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'app_id' => env('PUSHER_APP_ID'),
'options' => [
'cluster' => env('PUSHER_APP_CLUSTER'),
'encrypted' => false,
'host' => '<IP_ADDRESS>',
'port' => 6001,
'scheme' => 'http',
'useTLS' => false,
'curl_options' => [
CURLOPT_SSL_VERIFYHOST => 0,
CURLOPT_SSL_VERIFYPEER => 0,
]
],
],
client side
window.Echo = new Echo({
broadcaster: 'pusher',
key: "mykey",
cluster: "mt1",
wsHost: window.location.hostname,
wsPort: 6001,
wssPort: 6001,
disableStats: true,
enabledTransports: ['ws', 'wss'] // <- added this param
})

Sorry, know I'm a bit late, but if you try  NGINX and reverse proxy, this will work better.


Learning page

this is the first version of the learning page
you have a text box (which you enter a word there)
and two buttons
translate to sign text button
and text to speech button (react-native-tts libary installed so use npm ci to update the package json)

fix #82

i alredy did the merge but anyway
if you want any changes tell me.

Amazing


Mistake in the function

There is a mistake in the function, extra if statement exist.
https://www.freecodecamp.com/challenges/comparisons-with-the-logical-and-operator

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

if (val) {
  if (val) {
    return "Yes";
  }
}
<!-- should be this way: -->
if (val) {
  return "Yes";
}

Screenshot

@aseelsky Thank you for reporting this issue. Yes, there should only be one if statement, which goes along with what is asked for in the instructions.

Combine the two if statements into one statement which will return "Yes" if val is less than or equal to 50 and greater than or equal to 25. Otherwise, will return "No".

This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on Free Code Camp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


SamplePulse data type should be np.complex64 and not np.complex128

What is the expected enhancement?
The pulse module currently uses np.complex128 for SamplePulses. This resolution is excessive for modern AWG resolution and np.complex64 would be better suited.
Thoughts @ewinston and @dcmckayibm?

When it gets converted to list of list for qobj, it is going to be two doubles anyway since that is the precision of python floats.

Actually I guess you could if you wrote out the int32 dtypes directly to the json.

That is a good point. Python+JSON makes this difficult.

Closing following @nonhermitian's comment on Python floating point precision.  If we move away to a structured data description format in the future with stricter typing we should reconsider.


fix sphinx build

Overview
Sphinx build was failing due to this errors:

AttributeError exception having no message on Python 3.6.
AnyValue missing value.

Note, AnyValue is not really implemented yet.
Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x ] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

@huard Please, check if this fix is ok.

Coverage increased (+1.0%) to 76.068% when pulling 9368c764fbe1f13d76ede1749572fbcf615d575e on bird-house:fix-sphinx-build into 62a31d178edc3205bad0c159776bebca7a36a08a on geopython:master.

Coverage increased (+1.0%) to 76.068% when pulling 9368c764fbe1f13d76ede1749572fbcf615d575e on bird-house:fix-sphinx-build into 62a31d178edc3205bad0c159776bebca7a36a08a on geopython:master.


Does Paket have any culpability in VS intellisense failing issue?

Before I blast this message publicly, I would like some feedback from people more familiar with the intricacies of paket/dotnet. I have only every used this combination for F# projects, and never pure nuget/dotnet. Thanks.
The message:
I had high hopes for F# developer tools in VS 17.8 Preview 2, and at first I thought all was well. I was wrong.
You never know if intellisense is going to work or not, but you can be certain of one thing. If it is working, it will soon break. F12 works everywhere, then doesn't work.
I don't even know how to begin a bug report on this. If you can get intellisense and F12 to work, sooner or later you will type or paste in a code snippet that does not build and that is the end. Closing and reopening the file does not fix it. Closing and reopening VS does not fix it. Rebooting does not fix it.
The only thing that might work (and this doesn't even always work) is:
close vs
git clean -xfd
paket update -f (I think install might work better, but I have not experimented)
dotnet restore
Is the problem at the juncture of Roslyn tools and dotnet SDK?
Migrating F# development to Roslyn tools was probably a mistake in retrospect. What did F# developers get out of it? Answer: light bulbs.
Oh, and guess what? When everything else about the development tools is broken, the light bulbs don't work either.
What I'm talking about is real world solutions in production breaking the tools.
Everyone at Microsoft involved with F# needs to stop whatever they are working on and fix the developer tools.
I will stop now before I write something I regret.

@jackfoxy Paket is not supported with implicit restore at design-time, so you must restore before you open in Visual Studio. Otherwise, tooling will simply not light up because references cannot be resolved. This is due to design constraints both in the project system and in Paket, which are very difficult to reconcile on either side. This general problem isn't new to 15.8, though if you've first adopted paket for larger solutions with 15.8 previews it's understandable that it would surface these problems for you.

Is the problem at the juncture of Roslyn tools and dotnet SDK?

Doesn't sound like it. We also dogfood our own codebase with a mix of .NET SDK and non-SDK and do not encounter what you're describing. We do not use Paket, though.

You never know if intellisense is going to work or not, but you can be certain of one thing. If it is working, it will soon break. F12 works everywhere, then doesn't work.
I don't even know how to begin a bug report on this. If you can get intellisense and F12 to work, sooner or later you will type or paste in a code snippet that does not build and that is the end. Closing and reopening the file does not fix it. Closing and reopening VS does not fix it. Rebooting does not fix it.

Unfortunately, this is not specific enough to be actionable. I've never personally encountered pasting a code snippet that fails to compile and then having all tools fail across the entire codebase (nor has this been reported before), but if you can reproduce this, please do send a bug report, as that would be a high priority issue to fix.
Same goes for Go to Definition, unless Go to Metadata from the VFPT extension (Note: not officially supported) is what you are referring to. This is an open feature request with a candidate PR, but that PR has tremendous issues where the output often fails to write a metadata file with complete information. So it's not merged since it's quite broken w.r.t the actual result it produces.
Really sorry that you're seeing these issues, but unfortunately, without reproducible scenarios for what you've described, no action can be taken unless it can be observed in our daily work. And with regards to Paket integration, serious work in the project system and/or Paket will be required before any acceptable experience surrounding implicit restore can occur.

I don't know how much of this is due to paket - I can only try to analyze concrete repros. ;-(

Paket is not supported with implicit restore at design-time

That's not true. That actually worked long time ago. Maybe we regressed.
Phillip Carter<EMAIL_ADDRESS>schrieb am Mi., 27. Juni 2018,
20:20:

@jackfoxy https://github.com/jackfoxy Paket is not supported with
implicit restore at design-time, so you must restore before you open in
Visual Studio. Otherwise, tooling will simply not light up because
references cannot be resolved. This is due to design constraints both in
the project system and in Paket, which are very difficult to reconcile on
either side. This general problem isn't new to 15.8, though if you've first
adopted paket for larger solutions with 15.8 previews it's understandable
that it would surface these problems for you.
Is the problem at the juncture of Roslyn tools and dotnet SDK?
Doesn't sound like it. We also dogfood our own codebase with a mix of .NET
SDK and non-SDK and do not encounter what you're describing. We do not use
Paket, though.
You never know if intellisense is going to work or not, but you can be
certain of one thing. If it is working, it will soon break. F12 works
everywhere, then doesn't work.
I don't even know how to begin a bug report on this. If you can get
intellisense and F12 to work, sooner or later you will type or paste in a
code snippet that does not build and that is the end. Closing and reopening
the file does not fix it. Closing and reopening VS does not fix it.
Rebooting does not fix it.
Unfortunately, this is not specific enough to be actionable. I've never
personally encountered pasting a code snippet that fails to compile and
then having all tools fail across the entire codebase (nor has this been
reported before), but if you can reproduce this, please do send a bug
report, as that would be a high priority issue to fix.
Same goes for Go to Definition, unless Go to Metadata from the VFPT
extension https://github.com/Microsoft/visualfsharp/issues/1939 (Note:
not officially supported) is what you are referring to. This is an open
feature request with a candidate PR, but that PR has tremendous issues
where the output often fails to write a metadata file with complete
information. So it's not merged since it's quite broken w.r.t the actual
result it produces.
Really sorry that you're seeing these issues, but unfortunately, without
reproducible scenarios for what you've described, no action can be taken
unless it can be observed in our daily work. And with regards to Paket
integration, serious work in the project system and/or Paket will be
required before any acceptable experience surrounding implicit restore can
occur.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-400781633,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNJIYb01nNc-wY9THYc9XZnUVwYn-ks5uA8z_gaJpZM4U6Awx
.

Interesting. I wonder if this was back when the SDK project system was still super new, and it happened to work before the proper machinery was in place?

It would be kind of awesome if you could detect that scenario and pop open a yellow information bar at the top.
This is how rider does it since end of 2017: https://twitter.com/0x53A/status/924421382653476864?s=09

No. We just did what @davkean told us at that time and it worked.
Lukas Rieger<EMAIL_ADDRESS>schrieb am Mi., 27. Juni 2018,
21:34:

It would be kind of awesome if you could detect that scenario and pop open
a yellow information bar at the top.
This is how rider does it since end of 2017:
https://twitter.com/0x53A/status/924421382653476864?s=09
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-400803037,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNPey9SiCm75hecwDYuZqnyBUE7B9ks5uA944gaJpZM4U6Awx
.

That was targeted at @cartermp and @davkean, paket can't and shouldn't open any message boxes.

Seriously: we were told that referencing a targets file and loading the PackageReferences in memory is a good option and would work with VS. We did that and it worked.
Don't know why it doesn't anymore. Did not test for long time.

@davkean I remember there was a change in the order of the MSBuild targets for dependencies restore. Maybe we need to hook it up in a new one for background restore to see it?

@forki I've indicated many times that implicit restore with paket requires paket to do more work than they are currently, my last time was here: https://twitter.com/davkean/status/1003972527926886401. You've acknowledged that in the past, but keep coming back to this "VS has changed behavior part".
@0x53A I really like how Rider has done it, we should be better about this. At minimum paket should probably put an error in the error list to manually run it.

@davkean I did not say VS has changed/regressed. But I think the combination of paket/NuGet targets/VS did. In our very old email thread we already had it working. I even remember tweeting gifs about it. Imho a plugin in VS is just not a good solution. We were always looking for a MSBuild-only solution, so that things work ootb for everyone.

@davkean it seems there is a bit of a misunderstanding here. I was referring to @cartermp. He said you need to restore before opening VS. But that should work since it's "initial restore" and does not need file watchers.
In your tweet I think you were referring to the situation when paket files were touched and this does not trigger an additional background restore. For that part we set a MSBuild property RestoreSuccess to false. That seemed to trigger it, but maybe it's not enough. I was wondering if nuking the assets file would work. Do you know if that's watched? Thanks

Are we talking about the same scenario?

Open project in VS
Edit paket.dependencies/paket.references and save.
???

How do you delete the assets file in step 3? What code of yours is running in this situation?

I see 2 scenarios that I hope should work ootb without a plugin.

git clone
open VS
restore is triggered and intellisense works

after 1) edit paket.dependencies + paket.references
call paket.exe install from cmd line outside of vs
(paket nukes assets file)
VS triggers background restore and intellisense works

I think if we get these two working reliably then it would be great.

Two things:
In the case of 1) - does paket itself (not NuGet) need to hit the network?
In the case of 2), deleting the assets file will trigger restore - but it's heavy hammer and will result in us throwing away the world instead of just the diff. I would prefer a different mechanism, such as touching a file[1] that we're watching which will cause us to reevaluate -> design-time build -> restore if there's a change in package references.
[1] We can get to what that looks like later.

Yes paket needs to download. Same as nuget would do. It would also need to download from other sources, since we support more than just nupkgs.
What if we touch the csproj with a zero diff (changing the timestamp)

Well, it has to download eventually. That's just the way it is. If someone
was calling restore before then thongs are already cached, but after git
clone it would take a while.
David Kean<EMAIL_ADDRESS>schrieb am Do., 28. Juni 2018, 08:23:

If paket needs to touch the network during 1) then it will never be a good
experience via MSBuild. You'll be delaying all operations in VS waiting for
the packages to download, for old project system, even worse, you'll be
blocking UI while this occurs. NuGet restore is background via an extension
and never blocks build inside VS.
Touching the project file will not be enough, we will not trigger a
design-time build in 15.7. We can dig into this when I understand how this
stuff is supposed to work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-400925613,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNIMQnB3R4ZrwSs4mPzx4zOlP8EXpks5uBHZrgaJpZM4U6Awx
.

I'm still hazy on how this works - are you saying that you are not letting NuGet download the packages? Why is that?
This is my concern; you will be running 3rd party code hitting the network in a path that is supposed to take under 200ms. Then devs start blaming VS because it's not working as expected, like the start of this thread.

For nupkgs we could probably let nuget do parts of it (if we find a way to specify nuget source url per package). But what would that improve? Nuget can't download large packages in 200ms.

NuGet doesn't download packages during design-time builds. It's all background and we don't block any parts of VS based on restore.

As a reminder a design-time build is described here: https://github.com/dotnet/project-system/blob/master/docs/design-time-builds.md.

I think the best next step here is make sure we're all the same page around how things work in NuGet today, and why they work as such. Based on all the perf work we've been doing, I'm extremely sensitive to 1st and 3rd party (increasing design-time build times. These affects almost all VS operations and makes the entire VS seem slow and sluggish. I have a mental model in my head of how paket should plug in, but it doesn't seem to match your mental model of it. I think next steps should be set up a call and just discuss scenarios and so forth.
How's that sound?

OK. We could probably benefit from that when PackageReferences has an attribute for the nuget source (it's important for us) and if background build sets a MSBuild property (I guess you already do that). Then I could let paket ignore the downloads and let nuget do it.
In other situations we can't let nuget do it, since we do a lot of stuff on top - like generating load scripts for F#/C# interactive.

Hey, I'm a rando Paket user that jumped on this thread.
Before expecting  anything to compile, we run ".paket/paket.exe restore" from command line in the root of our deep, multi-solution application.
There is no scenario where Visual Studio would be downloading anything. If the scenario where you edit paket.dependencies and paket.references while a solution is open, and re-do "paket restore" from command line works (i.e. VS prompts for reload), that's already good enough.

Hey, I'm a rando Paket user that jumped on this thread.
Background restore works just fine in Code.

@cartermp I can now confirm. scenario 1) from above where you just open VS after clone doesn't work anymore. It doesn't trigger paket restore anymore

actually scratch that. I accidentally removed the gitignore paket's targets file. Scenario 1) still works and it doesn't block any UI. it works as expected. After a while the packages are restore and squigglies go away. so that's good

@Krzysztof-Cieslak regarding VS Code. Unfortunately it only works for F# there. Omnisharp doesn't automatically restore at all, they only pop up one window per project that asks if they should restore. that's a bit annoying if you have more than zero C# projects.

and tbf it only works because there is a plugin - and installing plugins is much more common in VS code world than in VS world.

@davkean @enricosada quick question: does VS set a MSBuild variable when background build/restore is running? which one would that be?

Use the DesignTimeBuild (CPS-based projects) and BuildingProject (legacy project system) properties to differentiate between when a target is run in a design-time build versus a normal build.

From https://github.com/dotnet/project-system/blob/master/docs/design-time-builds.md

Ok, I think I found a way to allow 2):
We already create a *.csproj.paket.props file in /obj in order to put CliReferences there. I can dump the PackageReferences there as well. VS is watching the file and triggers restore/resolution.
I didn't do that before since there is a timing issue when you run dotnet build from cmd line. In that case MsBuild doesn't see the packagesreferences since loading of props files is already done. But with yet another Msbuild switch we can keep current behaviour (PackageReferences in memory) for cmd line builds and move the references to props file for background builds. Does this sound reasonable?

I just spent an entire day in VS 15.8 Preview3 on the big solution for my day job with no problems.
Then I switched to https://github.com/fsprojects/FSharpx.Collections/commit/37d582a5d49edbe5ce41ce23b767a2a72961fc73
and in a file of FSharp.Collections.Experimental.Tests found the project in a state where the in-solution referenced projects were not accessible by intellisense. Then I did the following:

exited VS

command line paket restore

opened VS, in file in question waited until intellisense had finished doing its thing

3a) Damn! still not recognizing projects

closed VS again, with intention to do git clean, etc.

4a) remembered "I should really try building in debug first"

returned to VS, waited for intellisense to complete again

5a) this time projects are recognized and intellisense is working
cc @forki  @cartermp
I don't know if this provides you any useful information, but that's my story.

@vivainio Thanks, so your expectation is that VS doesn't just automatically pick up these changes like it does with PackageReference? Do you expect to be able to open newly cloned repo in VS and have it "just work"? It seems like the expectation by @jackfoxy is that it does.
@Krzysztof-Cieslak Missing the relevance to this issue?
@forki That is one approach albeit a bit annoying to have two paths, at what point to do you "generate" this file? If during the build, in worse case this will result in 3 design-time builds before you get full IntelliSense, which might seem to work okay for a single project, does not work well for a reasonable sized solution.

@davkean frankly, I have no expectations. This is all black magic to me. When I find what works, I'm happy.

@davkean yes, for freshly cloned repository I'm totally fine with having to do command line build (that does restore) before being able to compile any of the solutions within the repository. That's how other systems (like npm) work anyway.

I don't think what @jackfoxy describes is completely due to paket
integration. If you restore from cmd line and open VS afterwards as
described then everything is already restored and references are properly
populated. No manual build is needed. I think that must be something else.
Ville Vainio<EMAIL_ADDRESS>schrieb am Fr., 29. Juni 2018,
07:58:

@davkean https://github.com/davkean yes, for freshly cloned repository
I'm totally fine with having to do command line build (that does restore)
before being able to compile any of the solutions within the repository.
That's how other systems (like npm) work anyway.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-401255708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHtjnTgpEZdKj38iEdjfZfzZTJ0Dks5uBcHvgaJpZM4U6Awx
.

@vivainio I agree running cmd line build after clone is always a good idea. The package restore is usually only the smallest of the things that need to be prepared. Most of the projects that I have need so much more. Btw: even visualfsharp needs a command line preparation build, so that you have proto compiler ready.
Anyway, if you keep that habit then the proposed change will probably have no downside for you. During that first restore all props files will be generated and when you start VS everything should be done already.
@davkean regarding large solutions. I'm not completely sure if the new approach will actually be bad. That depends a bit on timing and I need to check it. This first project that is restoring in background build will restore for all projects in the solution. So all props files will

@vivainio I agree running cmd line build after clone is always a good idea. The package restore is usually only the smallest of the things that need to be prepared. Most of the projects that I have need so much more. Btw: even visualfsharp needs a command line preparation build, so that you have proto compiler ready.
Anyway, if you keep that habit then the proposed change will probably have no downside for you. During that first restore all props files will be generated and when you start VS everything should be done already.
@davkean regarding large solutions. I'm not completely sure if the new approach will actually be bad. That depends a bit on timing and I need to check it. This first project that is restoring in background build will restore for all projects in the solution. So all props files will

We've been using paket in combination with the new project system and have had no significant problems, at least nothing that cannot be fixed by triggering a build. This is using a repo with 6 solutions and ~60 projects.

@jackfoxy there is a prerelease version available. please report all bugs. (since we did not fix cache invalidation after paket's own upgrade: you need to remove paket-files folder the very first time after you upgrade)

easiest way to test the prerelease is to put version 5.174.0-alpha001 on first line of deps file

by deps file you mean paket.dependencies?

Yes
Jack Fox<EMAIL_ADDRESS>schrieb am Fr., 29. Juni 2018, 16:30:

by deps file you mean paket.dependencies?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-401371923,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNEq9aj1qjxBB9AogKoKb7NxsKRSWks5uBjoQgaJpZM4U6Awx
.

I will likely have some sort of report by Monday.

@jackfoxy I think you need to run paket install in order to update the targets file.

@forki I ran paket restore and it updated the targets file.
I just ran paket install and it did not change. So restore appears to work too.

@forki I have not encounter any problems with the alpha release.
I cannot determine if it had any impact on my issue. I still cannot reliably replicate the problem on demand, or replicate how to fix it. And it always does eventually get fixed.
You can close this if you are satisfied with the alpha release testing.
@cartermp  I'll probably set aside attempting a repeatable report of my problem until Preview 5,

@jackfoxy personally I'm not convinced that paket is the reason. in most (all?) former cases when intellisense issues were reported, we usually found a way to repro without paket as well.

@forki  I'm pretty sure you are right. I opened this issue only to make sure I had covered all possibilities. Glad the issue was useful for something I did not expect.

I had to restore intellisense functionality in VS 15.8 Preview 3 again yesterday. I still do not have a good reproducible way to restore intellisense. It seems to be a combination of clearing obj folders, "restoring" (in a generic sense) with paket, and then opening and closing VS and attempting to force intellisense by making it do a dot completion.
This is only anecdotal, not scientific, but I think I eventually get it right by starting with
git clean -xfd
paket update -f
whereas
git clean -xfd
paket restore -f
does not work. And this is with paket alpha.

Actually gut clean + just opening vs should already give you intellisense.
Jack Fox<EMAIL_ADDRESS>schrieb am Di., 3. Juli 2018, 17:26:

I had to restore intellisense functionality in VS 15.8 Preview 3 again
yesterday. I still do not have a good reproducible way to restore
intellisense. It seems to be a combination of clearing obj folders,
"restoring" (in a generic sense) with paket, and then opening and closing
VS and attempting to force intellisense by making it do a dot completion.
This is only anecdotal, not scientific, but I think I eventually get it
right by starting with
git clean -xfd
paket update -f
whereas
git clean -xfd
paket restore -f
does not work. And this is with paket alpha.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-402196878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNFFt-E-UyIOVVamsPIEJneTn9Sezks5uC40wgaJpZM4U6Awx
.

I'll try that next time.

Amazing. This time I only did paket restore -f with the alpha.
I still had to open and close VS twice to get intellisense working. If this consistently works it's a big step forward, and I can at least report a VS bug with some consistent work-around. Thanks.

If you did restore before opening VS, then everything works as vanilla
nuget. There is no paket in memory fiddling going on anymore. All
PackageReferences are real PackageReference tags in the paket.props file.
You can inspect that file in your /obj folder. If that file contains the
correct PackageReference tags then it's up to VS to provide intellisense
for you.
Jack Fox<EMAIL_ADDRESS>schrieb am Di., 3. Juli 2018, 23:10:

Amazing. This time I only did paket restore -f with the alpha.
I still had to open and close VS twice to get intellisense working. If
this consistently works it's a big step forward, and I can at least report
a VS bug with some consistent work-around. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3270#issuecomment-402292937,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNGQpbOmpKda3xFwtODxvHmSXaOiNks5uC93agaJpZM4U6Awx
.

This is starting to make more sense to me now. In other words, VS does not know when paket needs to do a new restore. VS is classic nuget centric.
My most recent incident triggered when I added a new table to the database FSharp.Data.SqlProgrammabilityProvider is monitoring. The TP caches the DB schema, so when you change the schema you have to restart VS. I also added code to process the new table. Somehow this broke intellisense requiring a paket restore and 2 more VS restarts.

In other words, VS does not know when paket needs to do a new restore.

No, whenever a nuget restore is done, then targets files call paket to do the actual restore. It's not bound to nuget at all. Also we now generate real PackageReferences in the props file. So it IS Nuget now.
Now that you mention type providers, I think we now moving to real root cause.

Now that you mention type providers, I think we now moving to real root cause.

Except the other solution that exhibits this problem, FSharpx.Collections test projects, have no TPs. They do have Expecto and FsCheck, which could be bringing along some strange baggage.

I haven't experienced this in collections with the new alpha release, yet.


There were errors during two-stage execution

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
With the latest update, it shows the following error:
(mini.deps) There were errors during two-stage execution:

...deps/opt/blink.cmp/lua/blink/cmp/fuzzy/download/init.lua:3: module 'blink.cmp.fuzzy.download.git' not found:
	no field package.preload['blink.cmp.fuzzy.download.git']
	no file './blink/cmp/fuzzy/download/git.lua'
	no file '/usr/share/luajit-2.1/blink/cmp/fuzzy/download/git.lua'
	no file '/usr/local/share/lua/5.1/blink/cmp/fuzzy/download/git.lua'
	no file '/usr/local/share/lua/5.1/blink/cmp/fuzzy/download/git/init.lua'
	no file '/usr/share/lua/5.1/blink/cmp/fuzzy/download/git.lua'
	no file '/usr/share/lua/5.1/blink/cmp/fuzzy/download/git/init.lua'
	no file './blink/cmp/fuzzy/download/git.so'
	no file '/usr/local/lib/lua/5.1/blink/cmp/fuzzy/download/git.so'
	no file '/usr/lib64/lua/5.1/blink/cmp/fuzzy/download/git.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'
	no file './blink.so'
	no file '/usr/local/lib/lua/5.1/blink.so'
	no file '/usr/lib64/lua/5.1/blink.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'

Relevant configuration
No response
neovim version
Nightly: v0.11.0-dev+1397-gfd05c7f19
blink.cmp version: branch, tag, or commit
main

Fixed by https://github.com/Saghen/blink.cmp/commit/f7bef25052820d4d7604a296c739ba9d885117f8


Colored text possible?

Looking at #218, I can't quite figure out what the point of \textcolor is. If I do:
\textcolor{red}{hi}\textcolor{blue}{ there}

all letters are italic as if it was math, and there is no space between the words. Is there away to achieve colors in a string of text that is upright, not italic?

This is the correct behavior, and is consistent with how LaTeX processes the commands you listed.
If you want to color non-mathematical text, you can use
{\color{red}{\text{hi}}\color{blue}{\text{ there}}}

You can , of course, define your own macro to make it easier, e.g.,
\def\textcolor#1#2{{\color{#1}{\text{#2}}}

which should make your expression produce the result you are looking for (both with MathJax's initial \color command, and the one from the color extension; that's the purpose of the extra braces).

Ah perfect then. What are the supported colors?

If you use the original \color macro (i.e., don't load the color extension), then any HTML color name (e.g., red) or color value (e.g., #F00 or #00FF80) is allowed.  If you use the color extension, then it depends on which color model you use (and you are allowed to define your own colors).

Just to be clear, it's HTML and hex that works, not the default from LaTeX or dvipsnames option?

Can more colors be loaded by loading in e.g. \usepackage{xcolor} or is this package unsupported?

Not because I desperately need them, I just want to be clear on what's available :-)

Just to be clear, it's HTML and hex that works, not the default from LaTeX or dvipsnames option?

If you don't load the color extension, then yes, HTML and hex RGB values.  If you load the color extension, either in your MathJax configuration, or through \require{color} within a math expression, then you can use the divps names.  You can also use several of the other color models (rgb, RGB, grey).

Can more colors be loaded by loading in e.g. \usepackage{xcolor} or is this package unsupported?

No, but you can use \definecolor to specify your own named colors.

Thanks.

I have an issue which is related.
I use color{Cyan} some text and the result from MathJaX doesn't match the Cyan of HTML.
Where are the colors defined?

Where are the colors defined?

The ones for the v3 color extension are in ts/input/tex/color/ColorConstants.ts.  They are the dvips color names from the xcolor LaTeX package, and they use the color values from that package, not the HTML colors of the same names.
Note, however, that the color names are case sensitive, and that if you use a name that is not in the list, it will be passed along as an HTML color name, so you can use
\color{cyan} some text

or
\color{#0FF} some text

to get the HTML cyan colors.


Add Deployment configurations

Add configs for vercel to help with ssr

Merging this by myself since it's boilerplate generated by astro itself


Polly describe_voices(LanguageCode="hi-IN") doesn't return any voices

Cause: Polly describe_voices(LanguageCode="hi-IN") doesn't return any voices
This appears to be a problem with the Polly service.  There is at least one supported language for "hi-IN" for Polly according to the documentation (https://docs.amazonaws.cn/en_us/polly/latest/dg/voicelist.html).
Using "Aditi" to synthesize an audio track works fine.

Closing because workaround was merged in #386.


Added new theme for Real Time Segmentation

Another new theme, this one does require 0.8.1 ksqlDb which isn't really called out anywhere other than the fact that it includes a docker compose file that references it.

thanks for the PR @pdruley !