chore(docs): fixing link for concepts in working-with-freight.md

The current link for key concepts is broken in https://kargo.akuity.io/how-to-guides/working-with-freight

Thank you @ftmiro !


Requirements missing for temporal_model_videomae

The seemingly missing requirements are iopath and simplejson. I will fix it later.

@YingtianDt  thanks, we will update!

This is addressed here!  #1210


SA1026 unit tests

Also all occurrences of Task.FromResult(true) replaced with cached instance

:+1: Looks good

:bulb: Note that I very much prefer to see changes like the use of a shared CompletedTask be a separate pull request. I try not to make this a hard-and-fast rule, but it definitely slows down this PR's review process.

@sharwell splitted #911

Thanks for taking the time to split these. Like I indicated, it wasn't required but it really does help.
:question: I see you changed the implementation of SA1026 as part of this. Were these part of a simplification, or did you find bugs/limitations that needed to be fixed?


x/pkgsite: github.com/newrelic/go-agent/v3/newrelic does not recognize license

What is the URL of the page with the issue?
https://pkg.go.dev/github.com/newrelic/go-agent/v3@v3.9.0/newrelic?tab=doc
What is your user agent?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.83 Safari/537.36

What did you do?

https://pkg.go.dev/github.com/newrelic/go-agent/v3@v3.9.0/newrelic?tab=doc
What did you expect to see?
Go documentation.
What did you see instead?
A notice that says godoc could not identify the license for the package. Package is licensed Apache-2.0, which is on the list of accepted licenses.

The reason we did not identify a license is because the zip file returned by<EMAIL_ADDRESS>doesn't contain that license file, though I'm not sure why.
/cc @katiehockman @hyangah @heschik for input on this.

This module uses the major directory (not major branch) model, so its files are here: https://github.com/newrelic/go-agent/tree/v3.9.0/v3 and there is no license there.

I don't think there's anything for the pkgsite team to do here, so closing.


Can't pass usb-mic to docker

I have Linux host with pipewire and just installed Assistant-Satellite docker and looks like usb-mic is not detected.
And I can't  manually specify it because this device doesn't start with plughw.
It may be necessary to manually specify --mic-device plughw:...

Run arecord -L to list available input devices. Pick devices that start with plughw: because they will perform software audio conversions. Use --mic-device plughw:... to use a specific input device.

This mic is working fine on another linux system.

I managed to get it working.


Is it possible to provide a scikit-learn interface?

This project is interesting and I want to use it as the baseline algorithm for my paper. However, it seems that I need to take several steps in order to make a prediction. Consequently, is it possible to provide a scikit-learn interface for making a convenient comparison between different algorithms?

You can try skorch, a Scikit-Learn wrapper for PyTorch: https://skorch.readthedocs.io. It looks like exactly what you are looking for. Does it solve your problem?
If you decide to implement things manually, you can take some inspiration from our official example: https://github.com/yandex-research/rtdl/blob/main/examples/rtdl.ipynb
There are other libraries as well, but they do not follow the scikit-learn interface:

Poutyne (a Keras-like library): https://poutyne.org/index.html
PyTorch Lightning (a general and powerful library for many kinds of tasks): https://pytorch-lightning.readthedocs.io

Do you mean that I only need to wrap the FTTransformer using skorch?

NOTE: I have updated the previous answer, please, read it first.

Do you mean that I only need to wrap the FTTransformer using skorch?

In theory, yes. Note that rtdl.FTTransformer expects two arguments (numerical and categorical features), so you will need to read this section.

Feel free to reopen the issue if you have more questions on the topic.


Support difference type of measurement error mitigation

What is the expected behavior?
#506 only supports CompleteMeasFitter to calibrate measurement result.
It will be nice if we can support TensoredMeasFitter as well.

Done the QuantumInstance that has now been migrated to Terra Qiskit/qiskit-terra#6570


Is it possible to open synonym table, not just info view?

Is your feature request related to a problem? Please describe.
It is a bit frustrating that when i have synonym tables (MsSQL) and i can easily find them via ctrl+shift+d i cannot visit those tables unless i focus select correct database in the navigator window on the left. Instead i see only some info view which is not much usfeul to me.
Describe the solution you'd like
I would like it to behave this way that when i find synonym table and i click open i am moved to this table with all the features that table view has.
Thanks, i enjoy dbeaver much besides this :)

thanks for suggestion

@andrzejdziekonski You can see synonym target in the synonym editor:

Just click the link to jump to actual table.
PS I've added several improvements, including synonym delete support.

Link isn't available in case Synonym is created using a fully qualified table name
works ok, link is shown:
CREATE SYNONYM test23
FOR dimdate;

Doesn't work, link is N/A:
CREATE SYNONYM test23
FOR master.dbo.dimdate;

Fixed

verified


request for useful visualization code

can i get some useful visualization tool(or code) to reproduce and check this work?
and also- any jupyter-notebooks for demo ?

Sure! We will provide the ipynb code soon. Thanks.


[SR-11790] 'generate-xcodeproj' fails to provide '.headerSearchPath(…)' includes

Previous ID
SR-11790

Radar
rdar://problem/57246846

Original Reporter
--marc (JIRA User)

Type
Bug

Attachment: Download

Environment

macOS Mojave 10.14.6 (18G1012)

Xcode 11.2.1 (11B500)

Swift 5.1.2 (swiftlang-1100.0.278 clang-1<IP_ADDRESS>)

swift-tools-version:5.1

Additional Detail from JIRA

Votes
0

Component/s
Package Manager

Labels
Bug

Assignee
None

Priority
Medium

md5: a641106cd9052b5772cc527190ed20b8

Issue Description:
`generate-xcodeproj` fails to `-I` include from `CXXSetting.headerSearchPath("Realm/ObjectStore/src")`
A build of `swift package generate-xcodeproj` generated project is expected to provide includes base on any '.headerSearchPath(…)' just like `swift build` does.
Addition discussion can be found in the GitHub realm/realm-cocoa issue #6345. (https://github.com/realm/realm-cocoa/issues/6345).
Steps
1. An example project is provided as an attachment and as the GitHub repository marc-medley/RealmSPMXcode. (https://github.com/marc-medley/RealmSPMXcode).
2. Build the example project with `swift build`. The build should complete with a small number of warning not related to this issue. Observe that the expected includes are present: "-I.../RealmSPMXcode/.build/checkouts/realm-cocoa/Realm/ObjectStore/src". And, later in the log file more instances of "Realm/ObjectStore/src" are present like "-MF '.../RealmSPMXcode/.build/x86_64-apple-macosx/debug/Realm.build/Realm/ObjectStore/src". See attached log-swift_build.txt.
3. Create an Xcode project with `swift package generate-xcodeproj`. Build the generated Xcode project. Observe the absense of "Realm/ObjectStore/src" in the build log. See attached log-generation-xcode-build.txt.
Note that the expressly stated `headerSearchPath(...)` incluse are present in step 2 but not in step 3.

@tomerd this can be closed since generate-xcodeproj has been removed.


X/Y Offset in TileRetriever.getTile()

Hi,
I‘m using this library (v3.4.0) to display a fairly large GeoPackage file on Android (Samsung Galaxy A50 with OSMDroid, but not with osmdroid-geopackage, as that was not flexible enough for me).
The GeoPackage File is in EPSG:3857 and was generated with GDAL. The Tilematrix overviews were built with gdaladdo.
When viewing the file with a gdal-based application (e.g. Qgis), everything is fine.
However, on Android with this library, the file is slightly shifted in X direction.
E.g. when using TileRetriever.getTile(551010, 365532, 20) with a tile size of 627x627.
This translates to this bounding box (taken from the Android Debugger inside getTile()):
minLongitude=1021275.134918239
minLatitude=6067380.212706579
maxLongitude=1021313.3534323834
maxLatitude=6067418.431220721

The resulting Tile from android is this:

Trying the same with gdal:
gdal_translate -projwin 1021275.134918239 6067418.431220721 1021313.3534323834 6067380.212706579 -outsize 672 672 -of PNG in.gpkg out.png

The code is not much more than the a standard OSMDroid tile provider. Basically:
        int zoom = MapTileIndex.getZoom(pMapTileIndex);
        int x = MapTileIndex.getX(pMapTileIndex);
        int y = MapTileIndex.getY(pMapTileIndex);
        GeoPackageTile geoPackageTile = tileRetriever.getTile(x, y, zoom);

Is there anything I‘m doing wrong?
I‘m fairly certain, that the issue is in geopackage-android, not in GDAL.
Reason: When I‘m going outside with a centimeter-precise RTK GPS, I‘m seeing the same offset when the underlying map is drawn with geopackage-android.
Tile matrix of the input image:
sqlite> select * from gpkg_tile_matrix;
table_name|zoom_level|matrix_width|matrix_height|tile_width|tile_height|pixel_x_size|pixel_y_size
tiles|0|1|1|256|256|152.455059500864|152.455059500864
tiles|1|2|2|256|256|76.2275297504321|76.2275297504321
tiles|2|3|3|256|256|38.1137648752161|38.1137648752161
tiles|3|5|5|256|256|19.056882437608|19.056882437608
tiles|4|10|10|256|256|9.52844121880402|9.52844121880402
tiles|5|20|20|256|256|4.76422060940201|4.76422060940201
tiles|6|40|40|256|256|2.382110304701|2.382110304701
tiles|7|79|79|256|256|1.1910551523505|1.1910551523505
tiles|8|157|157|256|256|0.595527576175251|0.595527576175251
tiles|9|313|314|256|256|0.297763788087625|0.297763788087625
tiles|10|626|628|256|256|0.148881894043813|0.148881894043813

EDIT:
Note that the offset seems to be absolute in terms of world coordinates.
I.e. when zooming out, everything is still shifted in the same way (the same "amount of meters").
Only when zooming out even further into the next overview in the tile matrix, there is a sudden jump (and it then again stays the same offset for the complete zoom-range for that overview).
EDIT2:
Just noticed the same with other geo package files, too. However, it's not always in x-direction.  For some files, there is an y-shift.  All files were created with gdal.

The bounds taken from the debugger for that tile appear correct.
From a tile server perspective the GDAL one lines up, however the GeoPackage appears to have an issue.
http://www.google.cn/maps/vt?lyrs=s@189&gl=cn&x=551010&y=365532&z=20

Without seeing your GeoPackage or visibility into the Tile Matrix Set table "tiles" row, the Tile Matrix still looks off.  The pixel sizes between zoom levels are a factor of 2 which is common.  The matrix width and height however are not.
So at zoom 1 you have a width of
2 matrix width * 256 tile width * 76.2275297504321 pixel x size = 39028.4952322
Which is a different width from zoom 2
3 matrix width * 256 tile width * 38.1137648752161 pixel x size = 29271.3714242
Each tile matrix shares the same width (and height) as defined in the Tile Matrix Set row.  So this violates the following requirement.
Requirement 45

The width of a tile matrix (the difference between min_x and max_x in gpkg_tile_matrix_set) SHALL equal the product of matrix_width, tile_width, and pixel_x_size for that zoom level. Similarly, height of a tile matrix (the difference between min_y and max_y in gpkg_tile_matrix_set) SHALL equal the product of matrix_height, tile_height, and pixel_y_size for that zoom level.

All tile matrix zoom levels share the same tile matrix set bounds.  So with a factor of 2 pixel change between zoom levels, the tile matrix width and height should double (even though tiles are not required to exist in all locations).  Assuming zoom 0 is correct, the matrix width and height at zoom 10 should be 1024 x 1024.
See this example
So I'm not sure if the bug is in the GDAL creation.  I pointed out a related bug to @rouault which he fixed on Feb 22, 2017.
https://trac.osgeo.org/gdal/ticket/6826

Oh wow, thank your for your insight, I think you are right.
I shared the load to create all my GPKGs between two different servers. One running gdal 2.4.2, the other one running 2.1.2.
I'm not sure which one created which GPKG, but looking at it, I find several files having the issue you describe, and several other that are fine. So I assume that the issue is in the files created with gdal 2.1.2. Which would make sense, since the bug you referenced was only fixed in 2.1.4.
I will recreate the affected files with gdal 2.4.2 and report back if that itself fixes the issue.

Alright, instead of recreating all the GeoPackage files, which would take a couple of days, I just created a quick python scripts that corrects the entries in the tile matrix.
That seems to have solved the issue for me. Thanks again.


PowerToys Run ran into an issue

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
Simple update but something went wrong.
✔ Expected Behavior
To be able to load PowerToys
❌ Actual Behavior
Error message "Something went wrong" and PowerToys will not run
2023-05-09.txt
Other Software
No response

Can you please upgrade to latest 0.73 and verify if the issue still occur?
/needinfo


message to user

In inserted messages we see in top
From: <John Smith><EMAIL_ADDRESS>Is it possible to do a link to username to start private messaging with user John Smith)?
Link like somthing  <@johnsimth> if i click on this in mattermost it will open private channel with user @johnsimth.
Mattermost link looks like @sometext, i have another idea =)
May be insert some functionality like option to work\edit\remove headers...to modify it before message will be inserted into channel.
May be something like find (John Smith) find email<EMAIL_ADDRESS>modify
From<EMAIL_ADDRESS>to
From: <John Smith><EMAIL_ADDRESS>where <John Smith> will has link to @johnsmith
and insert modifyed message to channel.
So, after this, we can click to name <John Smith> and start private channel with John Smith.
It will be nice to have option to start private channel immediately  with person who wrote mails.
Of course this person must have registered mattermost server account )
P.S. Sry for bad english )

ok empty mailboxes should work so this is a bug ill take a look at this.
About the '@' i thought it would be nice since you then know that you can click the person.

Its not a big problem, yes, "@" - shows us that we can click.
All is ok, service is working on production server, thx a lot )
ср, 5 февр. 2020 г. в 15:55, Carsten Seeger<EMAIL_ADDRESS>
ok empty mailboxes should work so this is a bug ill take a look at this.
About the '@' i thought it would be nice since you then know that you can
click the person.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cseeger-epages/mail2most/issues/33?email_source=notifications&email_token=AFRMOJJHQDLHJVXEQ3PXBNTRBKZKJA5CNFSM4KNU4I6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK3KB6Y#issuecomment-582394107,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFRMOJJVTYLI5G25PJ533Y3RBKZKJANCNFSM4KNU4I6A
.


Modify auto-increment id behaviour

Coverage remained the same at 87.468% when pulling c700e77fd3c4f3fd94fef6e04c62ad072c6b6645 on Kratious:auto-increment-fix into 37152d00d5780b5c5778b297d93a4262929996f4 on CS2103-AY1819S1-W16-2:master.


pbis authenticate-user working, but not login [Ubuntu 18.04 LTS]

Version: <IP_ADDRESS>7 from PPA
OS/Distro: Ubuntu 18.04 LTS
Issue/Impact:
after joining domain, pbis authenticate-user works, but not login or su to domain users. Always says "authentication failure" as if password wrong.
I had to add lsass to group and passwd lines in /etc/nsswitch.conf before id username would work, but otherwise this is a fresh install
two other 18.04 machines working fine.

systemctl status lwsmd.service
Says domain is online
/opt/pbis/bin/lwsm list
all running except usermonitor and dcerpc
/opt/pbis/domainjoin-cli query
outputs correct name, domain and DN
pbis status
says online
/opt/pbis/bin/enum-users
lists users
attach logs
lsass.log

These entries were all missing
/etc/pam.d/common-account:account       [success=ok new_authtok_reqd=ok default=ignore]         pam_lsass.so unknown_ok
/etc/pam.d/common-account:account       [success=2 new_authtok_reqd=done default=ignore]        pam_lsass.so
/etc/pam.d/common-auth:auth     [success=2 default=ignore]      pam_lsass.so smartcard
/etc/pam.d/common-password:password     [success=2 default=ignore]      pam_lsass.so
/etc/pam.d/common-session:session       optional        pam_lsass.so
/etc/pam.d/common-session-noninteractive:session        optional        pam_lsass.so


chore(README.md): add info on using body parser for POST to work

This is related to the issue where the app did not use body-parser and create service was failing with bad stack trace. This PR adds the information to add body-parser in the README.

Sweet, thanks for squashing!


M-06 - Risk-Free Trades By Credit Manipulation

Description

Orders can be executed after commit time + delay and till the expiry of the order. The price at commit time + delay is used even if the order is executed later than that (somewhere between this timestamp and the expiry timestamp)
An Order can not be settled if the delegated liquidity by the pool to this market is smaller than the required liquidity to back the open interest + this order sufficiently.

These conditions can be abused for risk-free trades:

Attacker is an LP of the pool behind the perps market
The attacker successfully creates an order
Attacker mints snxUSD to reduce the delegated amt to the perps market
The order can not be settled as there is no longer enough liquidity to back the perps market sufficiently if this trade goes through
The attacker waits if the price changes so that the order would be instant profit and if so burns the snxUSD tokens to increase the delegated amount again and settles the order

Recommendation
Cancel orders that fail because the delegated amount is smaller than the required credit.


In quick info, show all jsdoc tags

Fixes #19284

Check with @minestarks before merging this. She designed this API for a specific use-case which only wanted to see non-standard tags. I can't remember the case, but it will be broken if standard tags start being included.

@sandersn @andy-ms actually standard tags were OK in my use case - I just ignore them. As long as I'm still getting nonstandard ones it's fine.


Changes rst and readability

Rst markup altered.
Readability changed and specific info block created.

nice changes iphutch


Excellent! Can we have more complete vignettes?

Hi,
Blather seems a good library to speed up fine-tuning models. Would you mind to show how would we fine-tune code-generation based on a set of examples of prompt/completion?

Hey,
Thanks for the interest in the library! I think your idea of supporting fine tuning models for code generation is a great idea and I might add it in the future! In the mean time if you were interested I might check out https://github.com/wangcongcong123/auto_coding


Typo in the example

Hi!,
I tested the code in my computer with the example files and there is a typo that gives back an error.  The separation between the replicates in each condition should be without spaces after the comma.
Rscript MethyLasso.R --n1 normal --c1 WGBS_normal_1_chr1_meth.bed.gz,WGBS_normal_2_chr1_meth.bed.gz,WGBS_normal_3_chr1_meth.bed.gz --n2 cancer --c2 WGBS_cancer_1_chr1_meth.bed.gz,WGBS_cancer_2_chr1_meth.bed.gz,WGBS_cancer_3_chr1_meth.bed.gz --cov 4 --meth 5 -q 0.05

Hi,
Thank you for pointing out the typo!
Delphine

Dear Miguel-M-O,
For your information, if you were using bismark output files as input to MethyLasso, we just fixed an issue in MethyLasso.R to take the correct columns as input so this file should be replaced or options "--mC 5 --uC 6" specified.
I'm very sorry for that.
Best,
Anaïs


Add support for vagrant 1.8.1

Vagrant 1.8.1 depends on net-ssh (~> 3.0.1) and the chef gem depends on net-ssh (~> 2.6). In order to support vagrant 1.8.1 the chef gem had to be dropped. That means that uploading environments and roles as *.rb won't work anymore (and that's actually what the README always said).
Fixes issue #75

ping @priorityinterrupt and @mlawson1986 could you please build the gem form source (from this PR branch) rake build, install the resulting gem and test it?

Sure, I'll do that as soon as possible

Solves the issue for me, hopefully this gets merged soon.

Thanks @zakcrawford I'll try to get this merged and a new gem released in the next few days


Reversort

I would like to add solution to a DSA problem of codejam i.e. reversort engineering

They are not considering dsa,cp,algorithm repository :(
all the pr will not be considered in hacktoberfest but your contribution will be for lifetime, this will help many students to practice dsa .
Thank you for your contribution :)
you can contribute to other repo


To sync workflow with libusb release

https://github.com/pyusb/pyusb/blob/master/.github/workflows/run_tox.yml
The workflow is still using version libusb 1.0.23.
Not so sure if this is necessary but it may be good to test with latest release of libusb.

Probably not a huge deal, but it's certainly better to use the latest libusb release.  Thanks!

Probably not a huge deal, but it's certainly better to use the latest libusb release.  Thanks!


George/config from parameter store

What does this change?
This fetches configuration from AWS Parameter Store, instead of using environment variables.
Heavily inspired by https://github.com/guardian/mobile-purchases/blob/master/typescript/src/utils/ssmConfig.ts! ❤ 🙌

Should we load the parameter at server startup rather than api call?

Should we load the parameter at server startup rather than api call?

We could do, but I figure that since we are not passing the parameter to readNewslettersSheet(), we can just load it when it's needed. What do you think?


libvlcpp: Init 0.1.0-unstable-2024-02-04

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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/) header only
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@FliegendeWurst no idea what made me think it needed vlc i rebased to master and removed the vlc dependency and retested it again and it works without


Can we update caching strategy for page-data.json

as per gatsby caching guidelines https://www.gatsbyjs.org/docs/caching/ can we add cache-control: public, max-age=0, must-revalidate to page-data.json files

@rsbh have you figured out a way to do that with nginx?


Support .ruby-version

This PR is an implementation for  #31  by reading the target version from .ruby-version before using version parameter in workflows.

Proposed to the upstream https://github.com/actions/setup-ruby/pull/37


ionic build copies www/assets to platforms and not src/assets

Short description of the problem:
ionic build copies to platforms folder from www/assets and not from src/assets. Same for index.html. So in order run to develop on device we need to:
rm -rf www
ionic build
ionic run
What behavior are you expecting?
All for the run on device folder should be taken from src.
aka ionic run should perform ionic build first.
Steps to reproduce:

create unique file in your src/assets folder
look at assets folder
ionic run
watch the unique file in src/assets folder not being copied over to platforms folder.

Which @ionic/app-scripts version are you using?
0.0.36
This whole topic needs some more testing on Ionic teams side in my opinion.

This issue was moved to driftyco/ionic-cli#1599

Hello, thanks for opening an issue with us! Since this is a cli issue and not an app-scripts repo I have moved this issue to the cli repo. Thanks!


Small refactoring changes in Stride.GameStudio

PR Details
This PR refactors Stride.GameStudio and replaces Xenko images with Stride images in CrashReport form.
Types of changes

[x] refactoring / Bug fix

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hey @Jklawreszuk is this PR alright, I'm seeing a big contradiction here

What's the status on this PR?

Sorry, Honestly I am almost forgot about it, but I think now is ready :)

Thanks !


Yahoo! Stocks

https://greenido.wordpress.com/2009/12/22/yahoo-finance-hidden-api/

Yahoo's YQL might be a better option for this. It has a REST api that returns results as either XML or JSON which would be easier to parse than a csv file.
You can interact w/ it here and try out different queries: https://developer.yahoo.com/yql/console/
Documentation: https://developer.yahoo.com/yql/

@rhyolight
Can this be closed?

Yes
Sent from my MegaPhone

On Aug 21, 2015, at 8:26 PM, Jared Weiss<EMAIL_ADDRESS>wrote:
@rhyolight
Can this be closed?
—
Reply to this email directly or view it on GitHub.

Closed by #50 and #92


What is the purpose of:  FSharp-4.0.30603.0

We have 3 lkgs in our lkg folder
FSharp-2.0.50726.900
FSharp-4.0.30319.1
FSharp-4.0.30603.0
sentinel
FSharpSource.Settings.target is configured to use 4.0.30319.1
4.0.30319.1
To be clear .... 4.0.30603.0 will not compile FSharp.Compiler-proto.fsproj because of resource id collisions, which amazingly the previous lkg is quite relaxed about.
Additionally the FSharp-4.0.30603.0\ appears to be dependent on a non-existent runtime.  When I run fsc.exe I get this dialog:

I propose we delete it from the tree, it seems to be nothing but clutter, unless someone understands it's divine purpose.
Thanks
Kevin

agreed!

Closing - fix by @KevinRansom above


Limited default length of status-left and status-right

Limited default length of status-left and status-right
I wanted to have the following scheme for my status line:

However, when the session name is larger it overflows its segment. Here the session is called '0123456789':

After reading the nova scripts I thought that the problem was directly from tmux and found the status-left-length option. It seems tmux limits the length of status-left, and right, to 10 characters by default. After setting this option to 100 the problem was solved:

I think nova should either handle this for the user or at least mention it in the documentation so that the user can customize their theme without having to read the original tmux status line docs.

Hi @diogoDSN, thank you for using my plugin and opening an issue! I added a FAQ section in the README that explain and address the issue. I don't want to overwrite these values via tmux-nova to give the user more flexibility.


fix: exclude PNG from lossy webp compression

Purpose
Hugo uses a lossy compression when WebP is specified. That makes PNG
images such as a screenshot blurry.
$ identify content/post/.../myimage.png
-> PNG 738x607 738x607+0+0 8-bit sRGB 118157B 0.000u 0:00.000

$ webpinfo docs/post/.../myimage_..._1200x1200_fit_q75_h2_lanczos_3.webp \
    | egrep 'Height:|Width:|Format:'
->
  Width: 738
  Height: 607
  Format: Lossy (1)

Exclude those PNGs from a conversion and use the original lossless files
as is alongside of SVG or GIF.

PNG is converted by Hugo to WEBP on purpose by popular user request and Google (Lighthouse) recommendation.
If you want control over the WEBP algo or parameters, consider submitting an issue or PR on the Hugo repo.


make chaincode failing

Failing to build chaincode from the root directory. But I'm able to do the go build in trade_workflow directory manually.

make chaincode works for me. @ldesrosi any idea what she's missing?

Still seeing this problem? If not, can we close the issue?

Closing due to inactivity.


winch: Epoch-based interruption

Closes https://github.com/bytecodealliance/wasmtime/issues/8091
This commit introduces support for epoch interruption to Winch. The heuristics around epoch check emission are identical to Cranelift's except for the fact that the current implementation doesn't introduce a local-based cache for the current epoch deadline. This is an intentional decision given Winch's focus on compilation performance. However, if needed in the future, knobs could be introduced to optionally introduce a local cache at the cost of reduced compilation performance.

@fitzgen agreed with all the suggestions,thanks! I've also opened https://github.com/bytecodealliance/wasmtime/pull/9737, which applies the suggestions in this PR to fuel checks.


Another "classification" highlighting issue

Description
(VSCode 1.91.0 , Ruby LSP 0.7.12)
Related to #2269, I recently noticed another issue with words containing class being highlighted as keywords. This time, it is words ending with class.
I noticed it in a Slim template similar to this:
div class=wrapper_classes data-something="value"

div class=wrapper_class data-something="value"

Supposing wrapper_classes and wrapper_class are some ruby methods, the first example is highlighted correctly while the second is messed up:

Adding a word boundary check to the beginning of the class regexp in the grammar file seems to fix it:
- "match": "(class)\\s+(([a-zA-Z0-9_]+)((::)[a-zA-Z0-9_]+)*)\\s*((<)\\s*(([a-zA-Z0-9_]+)((::)[a-zA-Z0-9_]+)*))?",
+ "match": "\b(class)\\s+(([a-zA-Z0-9_]+)((::)[a-zA-Z0-9_]+)*)\\s*((<)\\s*(([a-zA-Z0-9_]+)((::)[a-zA-Z0-9_]+)*))?",

I can add a PR if you like. Thanks!

@borama when you are editing this file, does VS Code show { } Ruby in the footer?

@andyw8 No, it says "Slim" as the file is a Slim template but, in Slim notation, everything after the equal sign in an attribute is embedded ruby and that's where I think the ruby LSP comes into play and that's also where the class keyword gets matched via the regexp. See the Editor tokens inspection:

BTW I also managed to replicate the same problem in a HAML template: suppose I have a some_class helper method available, then the following HAML template renders correctly:
def some_class(item)
  "some_class #{item}"
end

hi = "hi!"
engine = Haml::Template.new { "%p #{some_class hi}" }
engine.render

# outputs:
=> "
some_class hi!
\n"

but when I write the same HAML code in a HAML template file and have one of the HAML extensions installed in VS Code, then it does not get highlighted properly:

Again, it is embedded ruby inside the parens and due to the class regexp not being properly left-bounded, it highlights the part of the helper class as a ruby keyword.

Oh I can actually replicate this even in a ruby file, at least for a moment. When I paste the following code to a bigger ruby file (that takes VS Code some time to parse), the incorrect highlighting is briefly visible before the semantic highlighting takes place and corrects it:
Záznam obrazovky z 2024-07-26 00-10-52.webm

Which Slim extension are you using, this one?

No, I'm using my own extension that I made recently :) but both of them behave similarly and fail in slightly different situations, based on what the extension recognizes as ruby (the older one e.g. does not highlight ruby in HTML attributes).

Ah, I see. I was able to reproduce the issue with your extension.
This should solve it, if you want to give the branch a try:
https://github.com/Shopify/ruby-lsp/pull/2371
Note that you'll need to run via the Extension Development Host so it uses the updated grammar.

Another way to illustrate the problem is with a Ruby heredoc string, e.g.:

Ah, indeed, this seems perfect, thank you!

Fixed in #2371 :pray:


translate  content_zh/docs/setup/kubernetes/helm-install into Chinese

original code original page
@rootsongjc @fleeto please review, thanks.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: xiaosuiba
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
Assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @xiaosuiba. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: xiaosuiba
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fleeto
Assign the PR to them by writing /assign @fleeto in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@xiaosuiba CI failed, to fix this please rebase the master.
- ./public/blog/2017/0.1-using-network-policy/index.html
  *  External link https://www.envoyproxy.io/docs/envoy/latest/operations/admin.html failed: response code 0 means something's wrong.
             It's possible libcurl couldn't connect to the server or perhaps the request timed out.
             Sometimes, making too many requests at once also breaks things.
             Either way, the return message (if any) from the server is: SSL connect error
- ./public/docs/tasks/telemetry/fluentd/index.html
  *  213:53: ERROR: Unexpected end tag : p (line 213)
- ./public/help/bugs/index.html
  *  External link https://preliminary.istio.io/ failed: response code 0 means something's wrong.
             It's possible libcurl couldn't connect to the server or perhaps the request timed out.
             Sometimes, making too many requests at once also breaks things.
             Either way, the return message (if any) from the server is: SSL connect error
htmlproofer 3.9.1 | Error:  HTML-Proofer found 3 failures!

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rootsongjc, xiaosuiba
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [rootsongjc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Wrong cursor position when entering insert mode with a large cursor jump after binding autocmd on InsertEnter

Hi, I am using im-select with this awesome project to change input method when leaving/entering insert mode. However, the cursor's position after entering insert mode is wrong when there's a "large jump" of cursor position.
Let | denotes cursor position, here's a example
(foobar"foobarfoobarfoobarfoobar|foobar"foobar)
↓ ci"
(foobar""foobar)|

However small jump works fine (when the jump is less than 8 characters)
(foobar"foobarfo|obarfoobarfoobarfoobar"foobar)
↓ ci"
(foobar"|"foobar)

Here's a screen record:

MWE is:
-- init.lua
vim.api.nvim_create_autocmd("InsertEnter", {
    callback = function ()
        vim.fn.system({"im-select"})
    end,
})

Maybe time delay of the function call is the reason. I managed to reproduce this issue by calling sleep command to delay 0.1 second.
-- init.lua
vim.api.nvim_create_autocmd("InsertEnter", {
    callback = function ()
        vim.fn.system({"sleep", "0.1"})
    end,
})

The problem is invoked when the cursor before the motion is on the right of the last character of current line after the motion. Or in another word, cursor before the motion is in a invalid position after the motion.
Works as intended:
(foobar"foobarfo|obarfoobarfoobarfoobar"foobar)
↓ ci"
(foobar""foobar)  # for position reference
(foobar"|"foobar)

Wrong behavior:
(foobar"foobarfoo|barfoobarfoobarfoobar"foobar)
↓ ci"
(foobar""foobar)|

Not sure if the autocommand is the culprit. In that case, dup of https://github.com/vscode-neovim/vscode-neovim/issues/1080


Still active?

Hey, messing around with prismic right now, and having some trouble with your library, but I see you haven't committed for about 8 months, so figure that probably makes sense. :)
The new (?) GroupDoc type for one seems to have broken your Group code, and unordered lists are breaking the StructuredText conversion.
I'm using Nextjs and I'm also pretty new to React (ng2 guy), so I may be missing something, but figured I'd check in just in case. If you need help with the library let me know.

I noticed Prismic have upped their game and started shipping new features at some point in the past couple of months, so no surprise that this broke :)
I will certainly need this lib to support the latest Prismic things to avoid breaking the project I originally wrote it for. I'm surprised the customer hasn't complained already! If you (or anybody) want to do it I would be super grateful. Shouldn't be much work unless they made some nasty breaking changes to existing fragment types. Let me know. Otherwise I'll try to get around to it soon.
Thanks for bringing this to my attention 👍

No problem, and thanks for getting back to me. Been poking around in your code, any tips for getting li's rendered safely within ul? I know I could just send html to dangerouslysetinnerhtml but I'm new to React and from what I can tell that's a no no, so trying to do it right.
Curious what brought you to prismic over cosmisjs or contentful? For me, it's their free plan. :)

I think the project I was doing ended up with Primsic because that's just what other people in my company had used before, rather than any sensible comparison with the competition.
FWIW I thought Prismic was rather lacking in many respects when I used it. The SDK was buggy, there was no write API, the query API was not well-rounded, there was no way to version document types, the docs were miserable, the UI was miserable, no singleton documents, no real help for  doing l18n, and so on and so on.  From what I've seen and heard of Contentful, it solves most (all?) of those problems and more. Seems like a really tight product, and I'd probably pick that next time I need an API-based CMS. Haven't looked into cosmicjs.

Ok, good to know. Might just end this pain now then. Thanks for your help!

For anyone else reading this: I probably won't bother maintaining this library after all. If you implement support for the new stuff, please send a PR :D Maybe I'll even grant you ownership.

So funny, I was just reading this issue and your comment popped up @ds300.
I can't get the library working with Next.js (getting a Cannot read property 'prototype' of undefined in fragments.js). I'll try to get it working though and I'll definitely send a PR if this library proofs to be useful for the website we're developing :).

I suppose I was commenting because I saw that the project had just been starred (by you, it turns out) and I was like "damn I hope that was an accident and not somebody actually intending to use the lib"  :)
Best of luck!

Haha, that makes sense. I was just searching for "react + prismic" and this library popped up and seemed to be useful. But I haven't had the time yet to really dig into it (or to even get it working in the first place ;)).


Redux and immutable stores

I'm starting to wrap my head around Redux and was previously using a basic flux architecture based around immutable stores.  It seems like getting Redux to work nicely with an immutable store is a bit of work with lots of gotchas (from connect, to reducers, to selectors, to the dev tools, etc).
Since changes always flow through reducers and reduces shouldn't mutate state, are most people just going without an immutable store when they switch over to Redux? I'm concerned that this will make shouldComponentUpdate harder to write since you can't immediately tell what state has changed using strict equality which will lead to lots of unnecessary renders.
Was there a reason that Redux wasn't built with an immutable store?

Redux makes no assumptions about the type of state you return from the reducer. You can use plain objects:
function reducer(state = { counter: 0 }, action) {
  switch (action.type) {
    case: 'INCREMENT':
      return { counter: state.counter + 1 };
    case: 'DECREMENT':
      return { counter: state.counter - 1 };
    default:
      return state;
  }
}

immutable data structures:
function reducer(state = Immutable.Map({ counter: 0 }), action) {
  switch (action.type) {
    case: 'INCREMENT':
      return state.update('counter', n => n + 1);
    case: 'DECREMENT':
      return state.update('counter', n => n - 1);
    default:
      return state;
  }
}

or any other value type, like a number:
function reducer(state = 0, action) {
  switch (action.type) {
    case: 'INCREMENT':
      return state + 1;
    case: 'DECREMENT':
      return state - 1;
    default:
      return state;
  }
}

Since changes always flow through reducers and reduces shouldn't mutate state...

In fact, that's exactly why immutable data and Redux go so well together. If you use persistent data structures, you can avoid excessive copying.
You may have misunderstood because of the way selectors work in react-redux. Because the result of a selector is passed to React's setState(), it must be a plain object. But this does not prevent the store's state from being an immutable data structure, so long as the selector maps the state to a plain object:
function select(state) {
  return {
    counter: state.get('counter')
  };
}

Hope that makes sense!

This is a good topic for the docs

Thanks for the explanation.  Seems like there are a few other gotchas like Issue #153.  Would be great to have a place in the docs for the proper way to take advantage of an immutable store.
There are also projects like https://github.com/gajus/redux-immutable which seems like they shouldn't even need to exist at all based on your comments.  And I'm not sure how well things like redux-react-router or the redux-devtools or other middleware will work when the store is immutable.  And then there is redux-persist which then needed redux-persist-immutable.  And redux-reselect needs a custom comparator (maybe...).  Basically every repo has at least one if not more issues raised with how do I get this working with Immutablejs?.
Seems like everyone is touting the benefits of immutable stores, but in this case the path of least resistance is looking like using plain objects instead.

#153 shouldn't be an issue. If you want the entire state tree to be an immutable object (rather than just the values returned by sub-reducers), it does require a bit more work, but nothing prohibitively difficult. You need to either use a specialized form of combineReducers() that returns an immutable value:
function combineImmutableReducers(reducers) {
  // shallow converts plain-object to map
  return Immutable.Map(combineReducers(reducers));
}

or eschew combineReducers() entirely and construct the top-level state object yourself.
As for the specific projects you mentioned, redux-devtools will work just fine regardless of the type of state. redux-react-router currently expects the router state to be accessible at state.router, but won't once this proposal is implemented. Not sure about the others.

Ah, so that's what redux-immutable implemented.  That makes sense now as does the proposal for redux-react-router.  Be great if everyone that interacts with the store can be pushed in that direction.  Appreciate all the help.

Didn't mean to close if you are using this issue for docs.

:+1: for adding a section in the docs for this - or maybe create a specific repository with all the instructions about how to use Immutable.js, and link it to the docs.
We also decided to use immutable.js, and for now what we did was to create a (very simple) custom combineReducers, didn't think about using the default one and then just transform it to Immutable.Map() Great suggestion @acdlite :)

Am I right to say that if you always return a new state in the reducer and therefore you don't mutate the state, it makes no difference to an immutable data structure performance wise? Or are there some other benefits beside less error prone?

@ms88privat the performance advantage of using an immutable state is in the rendering of React components: with immutable state, you can compare references to learn if the state has changed inside shouldComponentUpdate, without any complicated (and potentially slow) deep comparisons.

@ms88privat as @acdlite said in this comment, the improvement of using immutable data structures over plain js objects and arrays is the ability to reuse the unchanged things as much as possible, and avoiding excessive copying. By assuming the data is immutable, the library can do structural reusing of the collection, and share the unchanged parts between different objects, and still making it look to the outside as completely different structures.
@danmaz74 the performance improvement you mentioned also applies to using plain objects or arrays without mutating (ie creating a new state with the changes). The difference between something like immutable-js and POJOs is what I described in the previous paragraph.

@leoasis Thats what i wanted to hear. So react performance wise it should be the same, but it has less copying / memory-footprint or something like that.
What is the default status of shouldComponentUpdate with Redux? I read something about pureRenderMixing ... which i think should apply to redux, like we said. But how to implement in ES6?

@ms88privat react-redux's connect method creates a Connect higher order component that implements shouldComponentUpdate: https://github.com/gaearon/react-redux/blob/master/src/components/createConnect.js#L82-L84
As per your last question regarding ES6 and pureRenderMixin, that mixin is just implementing shouldComponentUpdate as it is done in the Connect component above. So that's really what you need to do if you want to have the pure render behavior in your ES6 component classes.

@leoasis thx, makes sense - so it's good to go.

@leoasis: @ms88privat wrote: "If you always return a new state in the reducer and therefore you don't mutate the state, it makes no difference to an immutable data structure performance wise". That's not correct, because if you always return a new state in the reducer (ie, also when there is no change) you can't just compare the reference - the comparison will always return the "new" state is different from the "old" state. That is in addition to the fact that using Immutable to create a new changed state is faster than creating a new deep copy.

@danmaz74 Well, if you do something like this:
return {...state, something: 'changed'}

then the root state and the something property will the only different objects, while the other properties in state will be the exact same reference and will be able to take advantage of the shouldComponentUpdate optimization.
It is true though that the Immutable.js library checks if the value you are setting is already set, and does nothing if that's the case, but that's fairly simple to do with POJOs as well:
return state.something === 'changed' ? state : {...state, something: 'changed'}

@danmaz74 Ok, i have to correct my spelling. If there is no change at all (= no action), the reducer will return the last state and there is no need for a new state.

@ms88privat ok no problem, I just interpreted that literally.
But in using Immutable (or something similar), there's still the advantage of easily getting what @leoasis pointed out in his last comment - without a library you can still do it using using ES7 spreads, but it looks much more complicated/less maintainable that way (at least to me).

We need “Usage with Immutable” in docs Recipes folder.

@gaearon I agree. I started playing with redux over the weekend trying to port a current app over from a flux implementation. I've been slowly figuring out how to use immutable but hit snags in the reducers and the tests for them. I've been able to use chai-immutable for my tests (https://github.com/astorije/chai-immutable) but it took some bumbling around by me to figure it out.
That being said, it hasn't been a terrible burden to figure out. It would just be nice to have a smoother onramp.

@acdlite after a deeper analysis, unfortunately you can't simply do Immutable.Map(combineReducers(reducers)); because combineReducers() returns a function, not an object. This is less terse, but works:
let combineImmutableReducers = reducers => {
  var combined_reducers = combineReducers(reducers);

  return (state,action) => Immutable.Map(combined_reducers(
      Immutable.Map.isMap(state) ? state.toObject() : state,action
  ));
}

@danmaz74 Yes you're right, my mistake. It was meant to be a naive example; e.g. even with your fix, it returns a new Immutable map every time, regardless of whether any of the keys have changed. A more complete implementation would update the previous state map rather than creating a new map. I think using map.merge() would suffice.

@acdlite You're right about the new map every time, but I think Map#merge always returns a new map anyway, too (at least, that's what the docs say).
Before testing the code above, we used this one:
let app_reducers = (state=Immutable.Map({}), action={}) => {
  for (let reducer_name of Object.keys(reducers)) {
    state = state.set(reducer_name, reducers[reducer_name](state.get(reducer_name), action));
  }
  return state;
};

This wouldn't change the root state if nothing changes below, but it wouldn't do the checks that combineReducers does. Honestly it's not clear to me which approach would be better; I guess that changing the root map would usually do very little damage, but of course it wouldn't be 100% clean. Any thoughts?

I also would love more documentation/examples on using immutable.js with Redux. Also, has anyone used normalizr with immutable.js? I was thinking of using it in some api middleware and was wondering if it was viable.

@dfcarpenter Normalizr is mostly useful for normalizing API responses before they become actions, so there's no difference: you can use it regardless of what you use for state.

@dfcarpenter We do use normalizr with redux, I have a createEntityReducer factory :
import Immutable, { Map } from "immutable";

export default function createReducer(reducer, entitiesName) {
  return (state = new Map(), action) => {
    const response = action.response;
    const data = response && response.data;
    const entities = data && data.entities;
    const collection = entities && entities[entitiesName];
    if (collection) {
      return reducer(state.merge(Immutable.fromJS(collection)), action);
    } else {
      return reducer(state, action);
    }
  };
}

I realize now that I could probably improve it by making it a middleware to which I would pass all my schemas..
To not be too much off topic, I'm also really interested in having redux working with Immutable from the grounds up. I like redux-immutable but I'm not a fan of having to use https://github.com/gajus/canonical-reducer-composition by default..

@chollier I was wondering if you can confirm if I understand correctly about using both normalizr and Immutable.js with redux.
I am just starting to get deeper into React/Flux, but it seems like the benefit of Immutable.js with React is to reduce graph traversal through something like the PureRenderMixin. If your store stores each model independently/unlinked using normalizr with references through ids to other models, it seems like there is little opportunity to know if the graph needs rendering (eg., maybe only at the leaves) since Immutable.js will not actually store the graph, but the individual links so the simple, instance immutable checks will not cover the changes lower in the graph.
Another way to ask this is: are there any common ways to optimize React rendering using normalizr or are you going to have to build the full virtual DOM whenever changes occur in a hierarchical model graph?

One small potential gotcha that you might encounter when using normalizr, immutable.js and numeric IDs...
If you have a collection: [{ id: 1, name: 'A' }, { id: 2, name: 'B' }]
normalizr will generate something like: { 1: { id: 1, name: 'A' }, 2: { id: 2, name: 'B' } }.
If you pass normalizr's output to Immutable.fromJS, you'll only be able to access the value with the string ID. Maps differentiate between numbers and string: collection.get(1) !== collection.get('1').
This can lead to unexpected bugs, for example you might have seemingly harmless code like:
const user = userCollection.get('1')
const account = accountCollection.get(user.get('account'))

But that won't work, and account will be undefined.

@kmalakoff I think one answer to that is to use cursors.
Also because of the way immutable.js store thing, even if part of your tree change, you can still get equality comparison on other parts of the tree. Example :
> a = Immutable.fromJS({ a: 1, b: 2, c: { d: 4, e: 5, f: [6, 7]}})
Map { a: 1, b: 2, c: Map { d: 4, e: 5, f: List [ 6, 7 ] } }
> var c = a.get('c')
undefined
> c
Map { d: 4, e: 5, f: List [ 6, 7 ] }
> c == a.getIn('c')
true
> a = a.setIn('a', 12)
Map { a: 12, b: 2, c: Map { d: 4, e: 5, f: List [ 6, 7 ] } }
> c == a.getIn('c')
true

interesting topic. Would like to see more examples.

Yeah,need more examples for immutable and normalizr

@chollier makes sense. From what I understand with normalizr and your example, if the List in 'f' referred to related models in the store, you would only be able to check if the reference to the ids changed, but not is the underlying models changed (since they are not embedded in 'f') so you wouldn't know if the tree needs rendering and would need to render it regardless of whether the ids changed, right?

Closing, as there doesn't appear anything actionable for us here. If somebody wants to write “Usage with Immutable” please do! Unfortunately I don't have experience in using them together so I can't help.

More references:
Plain Redux with ImmutableJs:
https://github.com/indexiatech/redux-immutablejs
A project that combines Redux with Seamless-Immutable:
https://github.com/Versent/redux-crud

Although this was closed back in September, I still haven't found any clear examples of using Redux with ImmutableJS, especially when it comes to shouldComponentUpdate and with normalised data.
Demo
Please find a fun little JSbin here:
http://jsbin.com/tikofel/4/edit?js,console,output
I think I was initially hitting the same hurdle that @kmalakoff was touching on. That with the normalised data, how does the container record "know" to update when one of it's children update - as it's just a sequence of numbers/strings/ids.
In the example above, I have created the data in a normalised way, but could also be achieved using Dan's nifty normaliser library.
Notes

Everything is kept as immutable until the latest possible moment (when toArray) is called
shouldComponentUpdate is bound to isDiff which runs over each immutable prop for a component and does an inexpensive === equality check (because each prop is immutable, this is cheap and accurate)
id lists, in this case, a persons list of pet id's are resolved only when creating the person component
With the console tab open, you can see when clicking each pet you are only updating that pet, and that person

I feel like this is the right approach, but am still learning redux/react/immutable, so would love to get anyones feedback. It's not a recipe as @gaearon mentioned, but hopefully this is on the right track.


Updating Single-Day Schedules (Thursday/Friday) To Be Chronological Ordering/Tabular Format

Can we modify the Thursday and Friday tabs to be chronological ordering?
Ideally, this would be:
Session Time (EST), Content Type (Tutorial, Keynote, Doctoral Symposium, Research Talk, Workshop Talk), Title (Paper/Talk Title, or "Doctoral Symposium"), Session Speaker Name(s), CHIL Moderator Name(s), Content Page Link, Zoom Session Link

The blurb for interaction:
"All sessions are meant to be interaction Q/A with authors, and most will have pre-recorded talk/tutorial presentations that should be viewed prior to attending the Zoom sessions.
For all content other than tutorials and socials, there will be a single Zoom webinar link for the entire conference for both days. Registered participants will receive a password for the webinar session.
Tutorials and socials will have separate Zoom links, and passwords for these sessions will again be emailed to registered participants.
When you move from the main conference Zoom webinar to a tutorial session or social, leave the main conference webinar session and rejoin.
Please do not share/tweet/post any passwords, as sessions are likely to be Zoom-bombed if these are leaked.
Realtime questions and coordination can be done over RocketChat.
"
The actual daily content for THURSDAY is:
10:00 AM - 10:50 AM (EST) | Tutorial | A Tour of Survival Analysis, from Classical to Modern  | George H. Chen, Jeremy C. Weiss  | Irene Chen |  LINK HERE | TutorialZoomLink

10:00 AM - 10:50 AM (EST) | Tutorial | Medical Imaging with Deep Learning | Joseph Paul Cohen | Irene Chen |  LINK HERE | TutorialZoomLink

10:00 AM - 10:50 AM (EST) | Tutorial | Public Health Datasets for Deep Learning: Challenges and Opportunities | Ziad Obermeyer, Katy Haynes, Josh Risley | Irene Chen |  LINK HERE | TutorialZoomLink

11:00 AM - 11:10 AM | Introduction | Opening Remarks | Marzyeh Ghassemi | TBD | N/A  | MainZoomLink

11:10 AM - 11:30 AM | Keynote | Title Here | Yoshua Bengio | Tristan Naumann |  Link | MainZoomLink

10:00 AM - 10:05 AM | Research Talk | Defining Admissible Rewards for High-Confidence Policy Evaluation in Batch Reinforcement Learning | Niranjani Prasad | TBD | Link | MainZoomLink

10:05 AM - 10:10 AM | Research Talk | Variational Learning of Individual Survival Distributions | Zidi Xiu | TBD | Link | MainZoomLink

10:10 AM - 10:15 AM | Research Talk | Interpretable Subgroup Discovery in Treatment Effect Estimation with Application to Opioid Prescribing Guidelines | Chirag Nagpal | TBD | Link | MainZoomLink

10:15 AM - 10:20 AM | Research Talk | Adverse Drug Reaction Discovery from Electronic Health Records with Deep Neural Networks | Wei Zhang | TBD | Link | MainZoomLink

10:20 AM - 10:25 AM | Research Talk | CaliForest: Calibrated Random Forest for Health Data | Yubin Park | TBD | Link | MainZoomLink

10:25 AM - 10:30 AM - 10:25 AM | Research Talk | BMM-Net: Automatic Segmentation of Edema in Optical Coherence Tomography Based on Boundary Detection and Multi-Scale Network | Ruru Zhang | TBD | Link | MainZoomLink
12:00 PM - 11:00 PM (EST) | Social Lunch | Social Lunch | N/A | Irene Chen |  N/A | SocialZoomLink

1:00 PM - 1:10 PM | Introduction | Welcome Back | Marzyeh Ghassemi | TBD | N/A  | MainZoomLink

>>> PUT IN DETAILS FOR Paper Talks 7 - 12
>Research Talk 7
Survival Cluster Analysis
Paidamoyo Chapfuwa

Research Talk 8
An Adversarial Approach for the Robust Classification of Pneumonia from Chest Radiographs
Joseph D. Janizek

Research Talk 9
Explaining an increase in predicted risk for clinical alerts
Michaela Hardt

Research Talk 10
Fast Learning-based Registration of Sparse 3D Clinical Images
Kathleen Lewis

Research Talk 11
Multiple Instance Learning for Predicting Necrotizing Enterocolitis in Premature Infants Using Microbiome Data
Thomas Hooven

Research Talk 12
Hurtful Words: Quantifying Biases in Clinical Contextual Word Embeddings
Haoran Zhang

1:40 PM - 2:00 PM | Keynote | Title Here | Elaine Nsoesie | TBD |  Link | MainZoomLink

2:00 PM - 2:20 PM | Keynote | Title Here | Nigam Shah | TBD |  Link | MainZoomLink

2:20 PM - 2:30 PM | Introduction | Day 1 Closing Remarks | Marzyeh Ghassemi | TBD | N/A  | MainZoomLink

@mghassem the functionality to display this information is now ready (e.g. see: https://www.chilconference.org/calendar.html#tab-thu).
Please could someone add the schedule to the following file?: https://github.com/acm-chil/acm-chil-website/blob/master/sitedata/daytoview.yml
The times in the post above seem to be mixed up (10am reappears after 11am), so this schedule needs a review before adding!

Thanks @tompollard !
Could we modify to:
0) Remove the "Read More" Column.

Change the "Video link" Column to be "Session Link".
Include two link fields for A) the session docpage and B) the session Zoom link.
Make the "Title" Column text link to the specified session docpage link.
Make the "Session Link" Column text link to the specified Zoom link.

Sure. Just see you can see how this works (and tweak if necessary), the components.html file includes the following macro:
{% macro day_to_view(day) -%}

      Time
      Type
      Title
      Speaker
      Moderator
      Read more
      Video link

    {% for row in day %}

      {% if row.time %}{{ row.time }}{% endif %}
      {% if row.type %}{{ row.type }}{% endif %}
      {% if row.title %}{{ row.title }}{% endif %}
      {% if row.speaker %}{{ row.speaker }}{% endif %}
      {% if row.moderator %}{{ row.moderator }}{% endif %}
      {% if row.doclink %}{{ row.doclink }}{% endif %}
      {% if row.videolink %}{{ row.videolink }}{% endif %}

{% endfor %}

{%- endmacro %}

This iterates the yaml content (which is loaded as a python dictionary) and creates the HTML that you see at: https://www.chilconference.org/calendar.html#tab-thu
So the template just needs modifying so that the link variables (e.g. {{ row.doclink }}) are included in href tags.

@mghassem I've updated the template. We now have the following fields in the YAML file:
  - time: "10:00 AM - 10:50 AM"
    type: Tutorial
    title: A Tour of Survival Analysis, from Classical to Modern
    speaker: George H. Chen, Jeremy C. Weiss
    moderator: Irene Chen
    doclink:
    sessionlink:
    sessionlinktext: Coming soon

If there is a doclink, then this will be linked to the title.
If there is a sessionlink then this will be linked to the sessionlinktext.

Hope this helps!


Move repo to github

BFO GoogleCode to Github migration
Source code migration:
Preserving commit history for files/directories under releases directories
Both Github import and GoogleCode export only migrate the files and directories under trunk directory. The files under releases cannot be migrated. It can be done by move the directory under tags directory using SVN move in GoogleCode. Using this approach, the commit history of the files and directories under releases will be lost after migration. Following approach was used for keeping commit history of BFO releases directory.
Step 1: migration
a. Migrate files/directories under trunk using github import:
https://bfo.googlecode.com/svn/
b. Migrate files/directories under releases using github import:
https://bfo.googlecode.com/svn/releases/
Now two repositories were created in the Github
Step 2: merge
Merge two created repositories following commands shown in page:
http://stackoverflow.com/questions/2949738/git-merge-different-repositories
If you have two projects, proj1 and proj2 and want to merge changes of proj1 into proj2, you would do it like this:
in proj2:
git remote add proj1 path/to/proj1
git fetch proj1
git merge proj1/master # or whichever branch you want to merge
After merge, all files/directories under releases and trunk are merged in one repository as sibling not in trunk or releases directory as in Google project
Step 3: add tag
Add 'releases' tag to the files/directories under original GoogleCode releases directory.
Find github commit # of the files and directories and add 'releases' tag using following commands:
git tag -a releases commit-number -m "tag message" // create a new tag and tag a specific commit
git tag -f releases commit-number -m "tag message" // force to use an existing tag and tag a specific commit
git push origin releases //push tag to the server
Step 4: clean up
Create a releases directory and move the release files/directory under new created releases directory using 'git mv'
Now, full commit history of the files under master/releases directory can be found by using command: 'git log -follow path/filename' and cannot be browsed on the website.
https://github.com/BFO-ontology/BFO/tree/master/releases
The commit history of the files/directories can be viewed on the website via tagged directory, see
https://github.com/BFO-ontology/BFO/tree/releases
This approach can preserve the committers information who contribute to the files/directories under releases directory in the Google code.
Link committers to their github account
During migration, github will list all the authors, the mapping to github account can be done by providing committer's github account name or registered email. However, it does not always work since github require the user to set account configuration for linking your commits to your account. Following commands can be run for account configuration:
git config --global user.name "your user name"
git config --global user.email "your email address"
This setting will apply to all the github repository you involved in. Details please see:
https://git-scm.com/book/en/v2/Getting-Started-First-Time-Git-Setup
After code migration, following approach can be tried to link your commit to your account. To browse the commit history on the website, when you find any of your commit, click that commit. When you view the committed file, it will associate a committer with question mark. click the question sign, you can add the email to associated with your account.
Here is an example page

Related material:
https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/
Issue tracker migration:
Using issues-migrator Python script:
https://github.com/arthur-debert/google-code-issues-migrator
The option '--synchronize-ids' is used to ensure that every Github issue gets the same ID as its original Google Code issue. When Google Code skips issue numbers, this option fills the gaps with dummy issues to ensure that the next real issue keeps the same numbering.
Since github only allow to remote access 100 times in an hour, it needs to run twice for BFO project. Second time, '--starts-at' option need to be used to specify the starting issue ID.
Notes: Github issue only allow to attach image file. If the google code issue has attachment other than image will be lost after migration.
Wiki pages migration:
Using googlecode2github wiki conversion Python script, details see:
https://github.com/trentm/googlecode2github

Things need to be done:

move issue attachment (e.g. https://code.google.com/p/bfo/issues/detail?id=189, copied to https://github.com/BFO-ontology/BFO/issues/189)
Update links in the wiki pages and issues if they link to BFO google code page
Update PURL redirect link
Update BFO link on the OBO Foundry and ontobee website

Attachments added
Wiki links updated
PURL redirect links updated

Change foaf:homepage to point to https://github.com/BFO-ontology/BFO or something other than google code


Can add Freeze column and row Footer ?

Expected Behavior
Can add Freeze column and row Footer ?
excel  need sum total and freeze bottom，last row
Actual Behavior
Steps to Reproduce the Problem

Specifications

Xcode version: ?
iOS version: ?
Dependency manager + version: ?


locked and unlocked visual for gui

What does your feature do?

How would people use it?
in the config of quests.yml

This is a feature that could be implemented in QuestsGUI. Please follow support methods for that plugin.


Fix move leader shard.

We now wait until all followers except the old leader are in sync.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2206/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2209/
New try, forgot a commit.


[Discussion] Is there a way to estimate serialized json string size before actual serialization?

The scenario is I'm trying to using the elasticsearch-rs crate to index json data by using bulk API(sending multiple json objects in one TCP request). However I don't wanna make a bulk request too big since that may cause Elasticsearch's index performance to drop.
Now I could only prevent that by controlling how many documents(json objects) to be send by a single request, but that is not precise enough. Sure I could do a serialization operation first to control request size, but that means I have to make an extra copy of the json string or do an extra serialization operation, which is kind of annoying.
So is it possible to do some estimation about how big the serialization result is?

Yeah here is a way to count just the size without producing a heap allocated json string:
use serde::Serialize;
use std::io::{self, Write};

struct ByteCount(usize);

impl Write for ByteCount {
    fn write(&mut self, buf: &[u8]) -> io::Result<usize> {
        self.0 += buf.len();
        Ok(buf.len())
    }
    fn flush(&mut self) -> io::Result<()> {
        Ok(())
    }
}

fn main() {
    let mut ser = serde_json::Serializer::new(ByteCount(0));
    let value = vec!["3", "2", "1"];
    value.serialize(&mut ser).unwrap();
    let len = ser.into_inner().0;

    assert_eq!(len, serde_json::to_string(&value).unwrap().len());
}

Awesome mate! Thx! I'll give it a shot :)


update new.haml undefined method active_users_count. should be active…

…_users.count

Thanks for catching this @rajeevkannav. This was actually an incomplete counter cache. I've taken care of that and a number of other issues in the latest updates. Curious that tests were passing previously.


Migrate to CircleCI?

My PRs have been stuck at Travis CI for a couple of days. Tried to rebase and close/reopen, still can't get the CI checks to go green---sometimes even when it has successfully built in Travis itself.
Should we consider moving to other CI tools? Almost all the popular open source projects I checked have been, or are now using CircleCI, including React, Vue.js, Gatsby, Redash, MetaBase...
A quick search on GitHub has found many GitHub projects migrated from Travis to CircleCI at some point, but seldom the other way around. There were chatters about Travis going through some hardship, too.
CircleCI offers free builds for open source projects just like Travis. They also have an official migration guide, which should make things easier.

Can those who used CircleCI before comment on their experiences?

@ktmud I believe the travis choice is due to to apache's rules. I think the main issue with travis is the limited resources for apache owned repos. If there are various other builds occuring in the apache org then things can grind to a halt and it can be quite frustrating. While CircleCi is probably better maintained than travis and their tooling and performance is likely better I'm not sure the migration would completely solve these pains

Actually, looking at your PRs it appears the issue is travis isn't updating the github status check after it passes. That's definitely a headbanger. Maybe reopen the PR in another branch?

Just created a new PR with a new branch and it's still stuck...
As @kristw mentioned to me offline, another choice is to use GitHub Actions. Indeed, it seems no Apache projects are currently using CircleCI. Some of them had CircleCI configs checked in the code, but it's not active running, at least not on GitHub.
There are some Apache projects already using GitHub Actions, including Spark,  incubator-dolphinscheduler, incubator-echarts,  and Groovy. It's also completely free and seems pretty easy to set up.
Should we give it a try?

@ktmud I agree that things could be better with CI right now. I don't have strong feelings about CI platforms, but anything that gets the job done quickly, reliably and is ok'ed by Apache gets my vote.

Github actions looks really promising, 👍 from me

+1 for github actions. We can go side by side for a while and see how it goes


add README.jp.md

https://github.com/spectresystems/spectre.console/pull/107
I add a localized README for Japanese.

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


Disable searches against less than 3 characters

You should probably disable searches for 1-2 characters as it cannot utilize the trigram index meaning its a vector for abuse and arguably useless.
Only consideration I can see is CJK users doing searchs for 1-2 characters but hopefully its not too restrictive for them.

3 Characters or more sounds good with me

What if you want to search for episode numbers?
Also how is the limit determined? utf-8 bytes? unicode codepoints?

unicode codepoints is my understanding.
As for the episode numbers there is no need to limit every search term to 3 or more characters, just to prevent searches of only less than 3 characters.
You can play around with the explain plans yourself to see how things break down just to double check.

Due to how search works searching for "some-anime 04" will also trigger the performance problem with trigrams
(SELECT * FROM torrents WHERE torrent_name LIKE '%some-anime%' AND torrent_name LIKE '%04%')

You know I learn something every day. Unfortunately it appears postgres makes some wrong assumptions with respect to GIN indices and will try to merge all predicates into one type of scan (index or table) rather than considering separating the predicates into different nodes. (Should be noted that the sequential scan is in fact much faster than the index only scan on the combined predicate so at least is partially right!)
You can force an index scan followed by a filter which is what I expected it to chose with something like
with q as (select * from torrents where torrent_name like '%some-anime%')
select * from q where torrent_name like '%04%' and torrent_name ...

or apparently
select * from torrents
where torrent_name like '%some-anime%' and (torrent_name||'') like '%04%' and (torrent_name||') ...

Pretty ugly, but it appears to be the only way to make up for the query planner. Maybe someone can be woo'ed to improve the query planner for pg11.

Is it still a thing? Since we use ES now

If you are no longer using postgres indices for accelerating these searches then this is probably better off closed.


dynamic import file load twice in build mode

Environment

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.2.3
Nitro Version:    2.2.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-nfcl83?file=pages%2Fthird.vue
npm run build
npm run preview

Open /third page reload page (SSR load) and see DevTools -> Network, static_file loaded twice.
Describe the bug
dynamic imports in application hooks load file twice in build mode with SSR. In SPA and/or dev mode all works correct

Additional context
No response
Logs
No response

Hey! Can't reproduce via StackBlitz.

Neither by navigating to the page ("client-side") nor hard reloading on /third.
Can you give more info? 🤔

Hey! Can't reproduce via StackBlitz.

Neither by navigating to the page ("client-side") nor hard reloading on /third.
Can you give more info? 🤔

you must make npm run build  after that npm run preview open /third and click reload
5 min and i will upload the video

you must make npm run build after that npm run preview open /third and click reload

That's what I did. Maybe a browser-specific issue or based on your Anti Virus software installed? 🤔

I dont think so, because it's reproduce always in other 3 PC

you must make npm run build after that npm run preview open /third and click reload

That's what I did. Maybe a browser-specific issue or based on your Anti Virus software installed? 🤔

https://user-images.githubusercontent.com/25429994/224297110-f899d575-07e8-423e-8bf7-36fd368c6991.mp4
you also must disable filtrating of request because file with type javascript not displayed when the filter JS is on

Looking into this, I don't think this is an issue. The file isn't being downloaded; it's being fetched from the prefetch cache, exactly as desired:

Looking into this, I don't think this is an issue. The file isn't being downloaded twice; it's being fetched from the prefetch cache, exactly as desired:

No, this requests not related, and browser start fetching first request and tries to load the next file, and you have 2 active requests. After that if you reload the page yeah, script file will be received from cache.

If your app hydrates and starts dynamically importing that file before it is finished prefetching, then it is expected behaviour that you will have two network requests.

Why another browsers  has only on network request?

Same problem, but second request not received from cache


Add node engine version in package.json

This will give developers a warning if they are using incorrect versions of node when installing the project.
I believe this project uses Node 18 or higher.
See the following for how to use it: https://docs.npmjs.com/cli/v10/configuring-npm/package-json#engines

Version >=20 was defined in this commit https://github.com/code4sac/opensac.org/pull/50/commits/204d438fab0a30731dfb3bbd72c2924c67a38bc5.
The project minimum Node version is 18.17 which is required by Next.js 14.


Update nx-remotecache-custom to 18

pnpm install on this:
	"devDependencies": {
		"@nx-dotnet/core": "2.2.0",
		"@nx/js": "18.1.3",
		"nx": "18.1.3",
		"nx-remotecache-gcp": "2.3.0"
	},

gives
 WARN  Issues with peer dependencies found
.
└─┬ nx-remotecache-gcp 2.3.0
  └─┬ nx-remotecache-custom 17.1.1
    └── ✕ unmet peer<EMAIL_ADDRESS>found 18.1.3

nx-remotecache-custom 18 expects peer of nx 18, so please, update nx-remotecache-custom.

Enjoy v2.4.0 :)


feat: send webhooks to Salient CompleteView events

https://trello.com/c/r4gqkBUA/1307-salient-vms-integration

@danleyb2
Don't forget to work on this.

@adolfoarmas regarding the folder structure, The larger idea is to also add OpenEye and Synology as a destination in the forwarder. This is because they are all basically a server that format the webhook payload then call and external API

@marcbelmont @danleyb2
I have only one point to highlight: I suggest the creation of bookmarks instead of events, along with the use of the latest NX API that does not require Digest authentication. The use of bookmarks makes the boards searchable in the system, in addition to providing a newer and recommended authentication method.
We have two NX implementations in our guides. The implementation below utilizes bookmarks:
https://github.com/parkpow/deep-license-plate-recognition/tree/master/webhooks/Webhook_nx
Is there any limitation for using bookmarks that made us choose events instead?

@ttayson @marcbelmont  @adolfoarmas
Suggestions:

[ ]  Keep existing REST API NX implementation  and delete the one in PR
[ ]  Maintain the implementations in separate folders and so refactor this PR to only contain Salient

Question:
How should we handle support for different NX VMS versions because the endpoint changes? It's easier and recommended to support latest only then advise users to upgrade
Based on NX API versions and their support from here
If the integration intends to support VMS 5.0+, it must prefer /rest/v1 which was introduced in this VMS version.
If it intends to support VMS 5.1+, it must prefer /rest/v2, and so on.
Originally (up to and including VMS 4.2), there was a mostly procedural-style API, with URLs mostly starting with /api/ and /ec2/ - these functions are now referred to as a "LEGACY & BETA" API.
Latest is /rest/v3


load usage help text for samples from file

Hi @ms-guizha, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Joe Zhang (SDE)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Current coverage is 42.94%

Merging #185 into master will not affect coverage as of 8fbd4b4

@@            master    #185   diff @@
======================================
  Files           40      40
  Stmts         3621    3621
  Branches       338     338
  Methods          0       0
======================================
  Hit           1555    1555
  Partial        103     103
  Missed        1963    1963

Review entire Coverage Diff as of 8fbd4b4

Powered by Codecov. Updated on successful CI builds.

LGTM.

@ms-guizha - @danielli90 made a valid point on formatting. Ignore my previous comment.


Steps not possible

This page needs a refresh, the portal interface has changed and you cannot follow these steps.
I tried to make a virtual network gateway in a separate subscription to simulate a 3rd party VPN.  Use it to connect to VWAN.
For external partners that may not have Azure and just a firewall of some kind, connecting to their VPN is complicated.
Perhaps show how to connect.  It seems you have to download the config, parse it and give them the settings they need to connect.  And with two hubs it is not clear where they need to connect.

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

ID: 6d62470b-a511-4b32-5498-2ae5182ec585
Version Independent ID: df2d4c2e-8afb-14ae-6667-9a504a5f8197
Content: Tutorial: Use Azure Virtual WAN to Create Site-to-Site connections
Content Source: articles/virtual-wan/virtual-wan-site-to-site-portal.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@ktleach
I have assigned the issue to the content author for further review and update the doc as appropriate.

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Added css preprocessor options to esbuild plugin

Linked to #18
Pretty much adds a preprocessing option so we can includePaths of our own and such.

Looks good, thank you!


Reorder deprecated features.

Hitting two bird with one stone, as per discussion here: https://github.com/D-Programming-Language/dlang.org/pull/976#issuecomment-96258429

so prolly this needs surgery as well after we undo if(arr)

Removed the if (arr) commit.

By the way @andralex @WalterBright - would there be any merit in adding a -transition for use of 'delete' and 'scope' classes?  Similar as per here https://github.com/D-Programming-Language/dmd/pull/4611

:+1: cc @MartinNowak


Speed up inference by providing more type annotations

The motivation here is to follow @sandersn advices (maybe @ahejlsberg will also give us some guidance on that matter)
https://github.com/Microsoft/TypeScript/issues/15443#issuecomment-300942212

Do we have any ideas where we should add the type annotations?
My project which uses fp-ts does not compile because of https://github.com/Microsoft/TypeScript/issues/15443. I'm very keen to unblock this, so I can continue using fp-ts in my project.
To get us started, it would be good to take a look at this simple test case, and figure out which type annotations are needed to get it compiling again.

@OliverJAsh The major problem in the test case is mixing Either and Validation
If you start from helpers/ip.ts
-task.of(validation.success('foo'))
+task.of(validation.success<any[], string>('foo'))

then in routes you get an error in getUserTwitterCredentials: ApiResponseNew<UserTwitterCredentialsT> is a Validation but t.validate returns a Either. If you remove the second map
export const getUserTwitterCredentials = (userId: string) => {
    const task = new fp.task.Task(() => Promise.resolve('{}'));
    return task
        .map(result => t.validate(result, UserTwitterCredentials))
        // .map((validation): ApiResponseNew<UserTwitterCredentialsT> => (
        //     validation.mapLeft(validationErrors => (
        //         [{ }]
        //     ))
        // ));
};

then you get 2 errors:

session.userId is not defined
req.ip is not defined

If you fake session and req
const session = {
    userId: '678'
}
const req = {
    ip: '387'
}

then you get the last error
credentialsM.chain(credentials => {
    // requestIpM :: Task<string>
    const requestIpM = normalizeRequestIp(req.ip);
    // y :: task.Task<ApiResponseNew<string>>
    const y = requestIpM.chain(a => getIanaTimeZoneFromIp(a))

    // this is not valid: y is a Validation but EitherT expects a Either
    // const ianaTimeZoneM = new fp.eitherT.EitherT(fp.task, y);
    // return ianaTimeZoneM
})

y is a Validation but EitherT expects a Either. Changing to
    const y = requestIpM.chain(a => getIanaTimeZoneFromIp(a))
        .map(v => v.toEither())

will let you compile the project

@gcanti That's really helpful. My project compiles now.
How did you deduce that the memory issue was because of this? I knew the project had type errors, I just didn't know what they were because of the memory/compilation issue.

The deduction comes from the following helpful comment by @sandersn

From poking at it with the debugger, it looks like type inference starts comparing various failure Monads like Left | Right and Succcess | Failure and gets caught in structural comparison of their various methods

Unfortunately always provide type parameters seems necessary (to get fail fast errors like in helpers/ip.ts) but not sufficient (to avoid crashes): if you put by mistake a wrong type (for example a Validation for a Either or a Either for a Option) canches are that you get stuck. As noticed by @sandersn I scrupulously provided phantom types (like _tag and _URI) to make assignability fail fast, however they don't help with the type inference issue:

There's no way of failing quickly out of an inference. It just fails to find any inferences for _tag: 'Failure' → _tag: 'Right' and moves on to the next property

Update from Microsoft/TypeScript#15443: It looks like we might have a fix on the typescript side so being explicit about type parameters probably won't be needed.

thanks @sandersn

Verified so I close

Just verified as well, amazing. Many thanks to the TypeScript team! You rock


Merge mdap-rc1 into production

Signed-off-by: Akash Srivastava<EMAIL_ADDRESS>

I need some time to go through the changes @Akash4927


[🐛 Bug]: ESLint rule no-app-not-found-runtime is exported as no-app-not-found

next-on-pages environment related information
System:
	Platform: darwin
	Arch: arm64
	Version: Darwin Kernel Version 23.0.0: Fri Sep 15 14:41:43 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T6000
	CPU: (10) arm64 Apple M1 Pro
	Memory: 16 GB
	Shell: /bin/zsh
Package Manager Used: npm (10.1.0)

Relevant Packages:
	@cloudflare/next-on-pages: 1.7.2
	vercel: N/A
	next: N/A

Description
The ESLint rule that is called no-app-not-found-runtime in the documentation is exported as no-app-not-found.
Reproduction

Install the ESLint plugin and make sure the rule next-on-pages/no-app-not-found-runtime is set.
Add export const runtime = 'edge'; to the not-found.tsx page.
Run linting.

No error. But if changing the name to next-on-pages/no-app-not-found it works.
Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
I can make a PR if you tell me whether it's best to change the documentation or to change the export.
Would you like to help?

[X] Would you like to help fixing this bug?

ah... yeah you're totally right! 😓
https://github.com/cloudflare/next-on-pages/blob/bf57ec9d01b354ea6c9b9bd17c683d592d5c6ae9/packages/eslint-plugin-next-on-pages/src/index.ts#L11
https://github.com/cloudflare/next-on-pages/blob/bf57ec9d01b354ea6c9b9bd17c683d592d5c6ae9/packages/eslint-plugin-next-on-pages/src/index.ts#L17
That's my bad, I think that the two lines above are the only places that need this changed, since you ticket that you'd like to help fixing this bug, please let me know if you want to apply the changes or if I should 🙂
(PS: updating this is technically a breaking change... but I think it could be added as a patch since the rule name is actually wrong (and the docs do indicate it with the correct name) 🤔)

I can do it. I was also wondering if this would be considered a breaking change or not. But I'm with you, the documented API will not change, it will just start working. :-)


Refactor data/queries module

          The `data/queries` module currently contains two types of functions.
          Roughly speaking, the newer functions (i.e. those we wrote there more recently)
          return `Query` objects.
          The older functions return objects that are returned by calling some query execution method
          on the `Query` object, like `.all()` , `.first()` or `.one_or_none()`.
          In my opinion, we should move the latter functions to `data/services`.

Originally posted by @Flix6x in https://github.com/FlexMeasures/flexmeasures/issues/314#issuecomment-1455939241
That is, functions in the data/queries module should return a Query object.

[x] Move other functions to data/services.
[x] Move queries from other places to data/queries.

A general comment:
One tricky thing about this kind of refactoring is that we have FlexMeasures plugins that import various classes and functions from flexmeasures. So moving these around within flexmeasures may break plugins. My strategy around this is to allow plugins to update asynchronously, by keeping the old definitions in place a little longer, throwing out the internal logic and letting them point to the new definitions instead, while logging a deprecation warning.

A general comment:
One tricky thing about this kind of refactoring is that we have FlexMeasures plugins that import various classes and functions from flexmeasures. So moving these around within flexmeasures may break plugins. My strategy around this is to allow plugins to update asynchronously, by keeping the old definitions in place a little longer, throwing out the internal logic and letting them point to the new definitions instead, while logging a deprecation warning.

Should this discussion not be on the PR?


Support for font shaping and subsetting

Work is underway on Raku bindings to the HarfBuzz font shaper.
Also separately underway on bindings to its font subsetting capability.
Would be good to integrate HarfBuzz for glyph selection at least. Ideally layout as well, but might require some work in PDF::Content to support an external shaper.
I think we need to combine it with Identity encoding and embedding.
Although both go well together. Font subsetting is somewhat independent of shaping.
Note: Most Linux distros don't support HarfBuzz's subsetting capability yet, which restricts its current use to those keen enough to build HarfBuzz from soruce, with subsetting enabled.

v0.4.1 uses the HarfBuzz::Subset module for optional font subsetting.
HarfBuzz's font subsetting is relatively new but seems pretty good. Not commonly packaged yet, so the HarfBuzz library needs to be built form source.


ICE candidates with docker IP

Dockerized go2RTC generates SDP answer with ice candidates containing the private docker IP. (<IP_ADDRESS>)
a=candidate:2516015194 1 udp<PHONE_NUMBER> <IP_ADDRESS> 44774 typ host
a=candidate:2516015194 2 udp<PHONE_NUMBER> <IP_ADDRESS> 44774 typ host

when runninc go2rtc via the binary the ip in these candidates (<IP_ADDRESS>) is correct
a=candidate:3078786767 1 udp<PHONE_NUMBER> <IP_ADDRESS> 39703 typ host
a=candidate:3078786767 2 udp<PHONE_NUMBER> <IP_ADDRESS> 39703 typ host

Not sure if this is something that can be fixed with code changes or different configuration in docker

Some docker IP ignored, but not all of them. It is very complicated thing for setup. I don't want the software to be for nerds.
https://github.com/AlexxIT/go2rtc/blob/12925a6bc5ed203582f465ef45ef4992270d8823/pkg/webrtc/api.go#L30-L32

For what it's worth, MediaMTX addresses this problem with a single config setting:
webrtcAdditionalHosts: [192.168.X.X]
https://github.com/bluenviron/mediamtx#connectivity-issues

go2rtc also has "additional hosts" via candidates setting


Method getUserChatBoosts doesn't work correctly

Bug Report

I have read:

Usage information
Help information

I am using the latest version of the library.
Expected Behavior
getUserChatBoosts method works correctly and returns UserChatBoosts for a successful request.
Actual Behavior
getUserChatBoosts method always rejects with an error:
ETELEGRAM: 400 Bad Request: invalid user_id specified
    at /app/node_modules/node-telegram-bot-api/src/telegram.js:316:15
    at tryCatcher (/app/node_modules/bluebird/js/release/util.js:16:23)
...

Steps to reproduce the Behavior
Just simply call getUserChatBoosts.

As a workaround for this issue instead of calling:
const userBoosts = await telegramBot.getUserChatBoosts(chatId, userId);

the following call can be used:
const userBoosts = await telegramBot._request('getUserChatBoosts', {
  form: {
    chat_id: chatId,
    user_id: userId
  }
});

Is not a issue, all the methods return a promise so you can get the result calling .then or use the aysnc / await


Whitespace management

Are there any linter rules for managing white space?  Like detecting tabs vs spaces, indentation issues, trailing spaces, etc.?

@rsmolkin Good question.  There are none at this time.  Due to the way tokens are currently lexed/parsed whitespace is discarded, checking whitespace is possible but not easy within the current framework.

I think this might be a potential useful addition for an unspecified future release - will reopen and track as enhancement.
In the meantime, @rsmolkin - I can recommend using http://editorconfig.org/ for helping you to manage tabs/spaces/indentation  etc. across IDEs and across team members etc.
It's not quite the same as having linting rules, but it'll help with consistency.

:+1: Consistency of tabs vs. spaces is the reason my team is investigating using a linter.

I have added a very basic implementation of the tabs and the moment it enforces spaces over tabs, however I still need to merge with the latest version. I will creating a pull request by the end of next week as the moment I stuck on another project.


Add SHIPPABLE_CONSOLE_API_URL env variable to microservices

https://github.com/Shippable/pm/issues/11458

this is done, closing


update twisted to 16.6.0

See #497.

It looks like the reason is https://twistedmatrix.com/trac/ticket/8692

Current coverage is 61.64% (diff: 100%)

Merging #539 into master will decrease coverage by 0.13%

@@             master       #539   diff @@
==========================================
  Files            41         41
  Lines          4320       4320
  Methods           0          0
  Messages          0          0
  Branches        571        571
==========================================
- Hits           2669       2663     -6
- Misses         1466       1468     +2
- Partials        185        189     +4

Powered by Codecov. Last update 8887ec4...407389d


Not working on IOS 17.5.1

Your Environment

Plugin version:
Platform: iOS or Android
OS version:
Device manufacturer and model:

Running in Simulator:
React Native version:
Plugin configuration options:
Link to your project:

Context

Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Context

Debug logs

Where you able to fix this for ios? @josefonte


Document version conflicts with max.in.flight.requests > 1

Hi,
we noticed that if we increase max.in.flight.requests to > 1 we get many problems with version conflicts. Meaning a newer version (offset) is already stored in opensearch.
I think it would be correct for the connector to ignore those exceptions, what do you think?
Best regards,
Daniel

waiting for next release, I think i misread the error message