[Bug]: Input value not showing at the start, then it can not be edited

Reproduction
CodeSandbox
Describe the bug
When using Input component and setting its value I would expect it work as the other shadcn/ui (here is link for expected result). A.k.a. its value would be set at the beginning and it can be edited afterwards by the user. But this is not the current behaviour. Instead at the beginning empty input is shown to the user and when user tries to edit the Input value it instantly changes to value set in code and sticks to that, therefore user can't change it.
System Info
System:
    OS: Linux 6.1 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
    CPU: (2) x64 AMD EPYC
    Memory: 2.18 GB / 4.01 GB
    Container: Yes
    Shell: Unknown
Binaries:
    Node: 18.20.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.6.0 - /usr/local/bin/npm
npmPackages:
    @vueuse/core: ^10.9.0 => 10.9.0
    radix-vue: ^1.7.2 => 1.7.2
    shadcn-vue: latest => 0.10.4
    vue: latest => 3.4.25

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

Data binding should using v-model or modelValue

You should use defaultValue for the inital default value @Grolicek .
  <Input type="email" placeholder="Email"<EMAIL_ADDRESS>/>


Implemented Skip button functionality, added adaptive WillPopScope, and enhanced Signup/Login page UI.

Implemented Skip button functionality to redirect to the login page.
Added adaptive WillPopScope to exit the app instead of returning to the onboarding screen.
Enhanced UI of Signup and Login pages for better user experience.

Please add label of gssoc-ext


split() function not accepting empty space as a part of separator/

Description
No response
Reproduction

But working fine without empty space🤷🏻♂

Expected behavior
It should accept that separator text.
Environment
swift-driver version: 1.115
Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

Additional information
No response

" - " is no longer a character. The first example resolves to a newer overload overload that accepts a collection of the same element type. The second example resolves to this overload, which accepts an element (a Character for String).

" - " is no longer a character. The first example resolves to a newer overload that accepts a collection of the same element type. The second example resolves to this overload, which accepts an element (a Character for String).

Got it/ thanks a lot for explaining nicely!!


ID is None in model observer message when model objects are deleted with atomic transactions

I have a ModelViewSet with a custom destroy() definition decorated with @transaction.atomic.
When the model object is deleted, the message passed into the model_observer handler does not contain the ID.
I have checked that the ID is present in the instance when ModelObserver.post_delete_receiver() is called.
However, the ID has changed to None when ModelObserver.post_change_receiver() is called.
The ID does not change to None if I remove the @transaction.atomic decorator from destroy() in the ModelViewSet.
Is this behavior expected?
If so, are there any workarounds to retain the ID when model objects are deleted via atomic transactions?
I would like to avoid defining post_delete signal handlers for all my models to send the delete notifications via the AsyncAPIConsumer containing the model_observer definition.

Soory I did not spot this sooner.
This is interesting, are other fields still populated (is it just the ID that is lost?)


optimize createBlock - generate the labels map only once.

This reduces the CPU time in half.
cc @codesome
Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

yes that is right but with maps the order is randomised so it is essentially different series.
Anyway I added some more code to add one unique label so now it is clear that it creates unique series.

it is not in an immediate  needed so will close and revisit if needed.


LaTeX2AI not working for version2020

My version of AI is 2020. I used the LaTeX2AI for 2021 and placed it where the doc suggests.
However, I got an error - Exactly 6 input arguments are expected when I click the button of the plugin in the AI2020.
Is this version for 2021 not compatible with AI2020?

The error occurs during the interaction between the PlugIn and the GUI. One possible issue could be if you have special characters or whitespace characters in your document name or in your Windows user name.
I also uploaded a (not tested) build with Illustrator 2022. Please have a try if this fixes your problem.
LaTeX2AI_b5c0db9_release_Illustrator2020.zip

Thank you. I tried all three versions of LaTeX2AI - 2020 (you uploaded here), 2021, and 2022; however, none of them are valid and the same error occurs.
My user name is "zjj_0". The document is a newly created blank one.
The following is my OS information:
···
Operating System: Windows 10 家庭版 64-bit (10.0, Build 19044) (19041.vb_release.191206-1406)
Language: Chinese (Simplified) (Regional Setting: Chinese (Simplified))
System Manufacturer: LENOVO
Processor: Intel(R) Core(TM) i5-6300HQ CPU @ 2.30GHz (4 CPUs), ~2.3GHz
Memory: 16384MB RAM
Available OS Memory: 16212MB RAM
Windows Dir: C:\WINDOWS
DirectX Version: DirectX 12
···

Try the following version, it is based on the branch 111-debug-forms-error and should give a more detailed error output.
LaTeX2AI_328d8ae_release_Illustrator2020.zip

The version [LaTeX2AI_328d8ae_release_Illustrator2020.zip] returns an error -
Nuber of arguments: 1
Argument 0: "C:\Program Files\Adobe\Adobe Illustrator 2020\Presets\zh_CN\脚本\LaTeX2AI_328d8ae_release_Illustrator2020\LaTeX2AIForms.exe"

I thought the Chinses character "脚本" perhaps causes the error, so I placed the .exe and .aip files into "C:\Program Files\Adobe\Adobe Illustrator 2020\Presets", however, it does not work as well.

If you place both executable files in a folder without Chinese symbols, which error message do you get? When exactly does this message appear?

I placed both executable files in "E:\" and got the same error -
Nuber of arguments: 1
Argument 0: "E:\LaTeX2AIForms.exe"

Can you exactly specify when this error occurs? Then I can upload a detailed bug tracking version for you.

When I clicked the "LaTeX2AI.exe" to start/open/initiate it in AI, the error occurs. Below are two pics.

The LaTeX2aiForms.exe executable is only used from within the plug-in in Illustrator. It is not required to call it by hand. I will clarify this in the README.md with the next release.
Try to remove all LaTeX2AI executable files you placed on your machine and reinstall LaTeX2AI as described here (preferably in a directory path  with only ASCII characters). Does the LaTeX2AI Toolbox appear in your version of Illustrator? You can try with the official releases or the one from this comment:

The error occurs during the interaction between the PlugIn and the GUI. One possible issue could be if you have special characters or whitespace characters in your document name or in your Windows user name.
I also uploaded a (not tested) build with Illustrator 2022. Please have a try if this fixes your problem.
LaTeX2AI_b5c0db9_release_Illustrator2020.zip

If I placed it in the plug-in folder in AI, the path will contain Chinese character I mentioned above. Perhaps this error occurs due to the Chinese character, so I try to put it on the desktop to avoid Chinese character. Neither of them works; and the same error - "Nuber of arguments: 1
Argument 0: "E:\LaTeX2AIForms.exe" - occurs for both conditions. The following pictures show the condition that the LaTeX2AI Toolbox appears in my AI.

It seems like you are using the script folder to install the plug-in, this does not work.
Place both files (LaTeX2AI.aip and LaTeX2AIForms.exe) into an arbitrary directory with an ASCII path. Then this directory has to be set as the Adobe Illustrator Plugin directory in Illustrator via Edit/Preferences/Plug-ins & Scratch Disks.../Additional Plug-ins Folder. After a restart of Illustrator the following tool panel should appear in Illustrator:

Thanks for your help. I've changed from AI2020 to AI2021 and followed the instructions you provided. The LaTeX2AI was installed correctly and it can work well except for a newly default doc created because of its Chinese name in the Chinese version of AI2021. The ASCII path requirement in either path or name is not friendly for users of non-English version of AI. It sounds great that the LaTeX2AI can support UTF8. Is this possible in the future?

Good to hear that LaTeX2AI is now working on your system. Regarding the ASCII paths, there is already an open issue (#100).


[Sematext Docker Agent] Add support for tolerations and other minor improvements

/ok-to-test

/assign

/lgtm


[Question] What is the plan with regards to handling bindings for multiple c++ classes in the same project

@zicklag ... while I have you here maybe you can answer something for me?  What is the plan with regards to handling bindings for multiple c++ classes in the same project.  The idl file in the sample is called point.idl presumably because the only thing it handles presently is the Point class but if I wanted to  add a class (different header and implementation file) would I just append it to the same .idl file?  And if so I  suspect I will have to handle it somehow in the makefile?  Perhaps make is a bit light for a larger workload if that's the case? Perhaps the work presently being done by make should be handled by a webidl executable?  I'm not sure what the long term goals are for this project but it looks like something that I would like to make more industrial use of than just a few samples...

Originally posted by @WolfieWerewolf in https://github.com/ncannasse/webidl/pull/13#issuecomment-464375774

but if I wanted to add a class (different header and implementation file) would I just append it to the same .idl file?

Yes. All of the definitions that you want to go into the Haxe module would go into the same WebIDL file.

And if so I suspect I will have to handle it somehow in the makefile? Perhaps make is a bit light for a larger workload if that's the case? Perhaps the work presently being done by make should be handled by a webidl executable?

Yes, you would want more automation for that that this library doesn't currently provide. I'm actually using this library to automate C++ bindings for Kha projects. Kha has its own make system called Khamake, and I have recently submitted PRs for Khamake and Kha that completely automate the whole binding process.
For any other users of this library, though, a setup for automating the make process for other applications would be useful. What I setup was very specific to Kha, but you could still check it out at https://github.com/armory3d/haxebullet/issues/25#issuecomment-462210471.

@zicklag great, thanks.. I will take a look.

@zicklag .. I think we discussed this and I believe I've handled this in my PR.  Once you've tested I think this ticket  can be closed.

Sweet, I'll test it as soon as I get the time. :+1:


Invalid 8 Byte Files, Frequent Hangs of Applications, ERROR - "path/file.something" is not a valid cache file, deleting

Hello,
I love catfs. I'm using it to provide caching for a goofys s3 "mount". It makes a huge difference!
I do however have some nagging issues which happen on a reglar basis. I hope I have provided enough information and would like to help resolve this if it's a non-local specific issue.
System:

Gentoo linux
uname -a: Linux deleted 5.12.13-gentoo-x86_64 #1 SMP Thu Jun 24 12:24:13 DELETED 2021 x86_64 Intel(R) Core(TM) i7-7820X CPU @ 3.60GHz GenuineIntel GNU/Linux
16 cores
32GB RAM
Filesystem which the cache is using: 100GB, EXT4-fs (sde1): mounted filesystem with ordered data mode. Opts: errors=remount-ro,user_xattr. Quota mode: disabled.

catfs issues:

When viewing the data of a directory full of .tif files, the first one requested is always 8 bytes, which appears to be the bare minimum of data to get 'file' to report it is a tif image.

Indeed the first 8 bytes of real source file is the same, however, it is far larger than 8 bytes.

When viewing the data of a directory full of .tif files, first request or not, screenful of errors are printed out by catfs. Examples:

2021-06-30 16:53:22 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0902830w295700n.tif" is not a valid cache file, deleting
2021-06-30 16:53:23 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0902830w295830n.tif" is not a valid cache file, deleting
2021-06-30 16:53:23 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0902830w300000n.tif" is not a valid cache file, deleting
2021-06-30 16:53:24 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0902830w300130n.tif" is not a valid cache file, deleting
2021-06-30 16:53:24 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0903000w295530n.tif" is not a valid cache file, deleting
2021-06-30 16:53:25 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0903000w295700n.tif" is not a valid cache file, deleting
2021-06-30 16:53:25 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0903000w295830n.tif" is not a valid cache file, deleting
2021-06-30 16:53:26 ERROR - "2005_Hurricane_Katrina/aug31JpegTiles_GCS_NAD83/aug31C0903000w300000n.tif" is not a valid cache file, deleting

During this time, which can be several minutes, the application trying to use the data is hung and usually needs force killed.

While catfs is "deleting" files, the total size of the cache directory root grows

After these few minutes, everything works, except for that first invalid 8 byte file.

Often but not always after this delay, despite having fetched files before, they are fetched new again (increased inbound network traffic is observed)

Gwenview STDERR on the 8 byte file:
gwenview.libtiff: Error JPEGLib "Not a JPEG file: starts with 0xd5 0x7e"
org.kde.kdegraphics.gwenview.lib: Could not generate thumbnail for file "file:///media/s3-noaa-eri-pds-LOCAL-catfs/2005_Hurricane_Katrina/sep02JpegTiles_GCS_NAD83/sep02C0890130w290900n.tif"
gwenview.libtiff: Error JPEGLib "Not a JPEG file: starts with 0xd5 0x7e"
org.kde.kdegraphics.gwenview.lib: Could not generate thumbnail for file "file:///media/s3-noaa-eri-pds-LOCAL-catfs/2005_Hurricane_Katrina/sep02JpegTiles_GCS_NAD83/sep02C0890130w290900n.tif"

Sometimes after all the other images have been downloaded and thumbnails created, I can double-click on that first file that had been 8 bytes, and it's now downloaded, and will display without error. Other times, it stays 8 bytes.

Reproduction

Mount an aws s3 public bucket with goofys: goofys -o allow_other -o ro noaa-eri-pds /media/s3-noaa-eri-pds-LOCAL

Start catfs appropriately: catfs -o ro --free 1G /media/s3-noaa-eri-pds-LOCAL /home/myusername/.goofys-cache /media/s3-noaa-eri-pds-LOCAL-catfs

Explore the images with a tool such as gwenview that will show thumbnails

Optional: Start gwenview on the command line, outputting STDOUT and STDERR to their own files and then watch these files: gwenview 1> /tmp/gwenview-STDOUT.txt 2> /tmp/gwenview-STDERR.txt & tail -f /tmp/gwenview-STD*

I noticed I was getting complaints from catfs about too many open files. The default was 1024 for the user, so I increased it to 10240 and rebooted.
I'm not getting the too many open files errors anymore, however, I was checking how many open files there were for which processes, and here's the result. I am not entirely sure this is the right way to count "open files", however, I think these stats are interesting. Is this normal?
`
As root: lsof -n | grep myusername | awk '{print $1}' | sort -n | uniq -c | sort -nr | head -n 4
133961 catfs
78938 chrome
49162 thunderbi
16426 gwenview
`


DNF repo cost option working the oposite way as described in man with the reinstall subcommand

According to DNF Configuration Reference (man dnf.conf), a repository with a lowest cost should be picked when the priorities are equal.

cost   integer
The  relative  cost  of accessing this repository, defaulting to 1000. This value is compared when the priorities of two repositories are the same. The repository with the lowest cost is picked. It is useful to make the library prefer on-disk repositories to remote ones.The  relative  cost  of accessing this repository, defaulting to 1000. This value is compared when the priorities of two repositories are the same. The repository with the lowest cost is picked. It is useful to make the library prefer on-disk repositories to remote ones.

However, on Oracle Linux Server release 8.8, DNF reinstall is picking the repository with the highest cost.
STEPS TO REPRODUCE:

Mirror a repo locally (I mirrored the ol8_appstream)

Create a local_ol8_appstream repo with a lower cost than default (1000)

[local_ol8_appstream]
name=Local Oracle Linux 8 Application Stream ($basearch)
baseurl=file:///yum/OL8/ol8_appstream/
gpgkey=file:///yum/OL8/RPM-GPG-KEY-oracle
gpgcheck=1
enabled=1
cost=900

Try to reinstall a package available in ol8_appstream/local_ol8_appstream. DNF should pick the package from local_ol8_appstream (lowest cost), but picks it from ol8_appstream.

[root@localhost ~]# dnf reinstall wget
Local Oracle Linux 8 Application Stream (x86_64)                                                                                541 kB/s | 3.9 kB     00:00
Last metadata expiration check: 0:00:01 ago on Thu 16 Nov 2023 05:51:58 PM -03.
Dependencies resolved.
==============================================================================================================================================================================================================
 Package                                    Architecture                                 Version                                                    Repository                                           Size
==============================================================================================================================================================================================================
Reinstalling:
 wget                                       x86_64                                       1.19.5-11.0.1.el8                                           ol8_appstream                                       734 k

Transaction Summary
==============================================================================================================================================================================================================

Total download size: 734 k
Installed size: 2.8 M
Is this ok [y/N]: N
Operation aborted.

Change the cost of the local repo to a value higher than default.

[local_ol8_appstream]
name=Local Oracle Linux 8 Application Stream ($basearch)
baseurl=file:///yum/OL8/ol8_appstream/
gpgkey=file:///yum/OL8/RPM-GPG-KEY-oracle
gpgcheck=1
enabled=1
cost=1100

Try to reinstall the same package. DNF now picks the package from local_ol8_appstream.

[root@localhost ~]# dnf reinstall wget
Local Oracle Linux 8 Application Stream (x86_64)                                                                                919 kB/s | 3.9 kB     00:00
Dependencies resolved.
==============================================================================================================================================================================================================
 Package                                   Architecture                                Version                                                 Repository                                                Size
==============================================================================================================================================================================================================
Reinstalling:
 wget                                      x86_64                                      1.19.5-11.0.1.el8                                        local_ol8_appstream                                      734 k

Transaction Summary
==============================================================================================================================================================================================================

Total download size: 734 k
Installed size: 2.8 M
Is this ok [y/N]: N
Operation aborted.

Also reported on https://issues.redhat.com/browse/RHEL-25005


Construction only uses planarLength

The construction code only uses geometryEngines planarLength and not geodesicLength-method. This leads to wrong constructions when using WGS84 (wkid: 4326) or Web Mercator.
https://github.com/conterra/mapapps-sketching-enhanced/blob/11f75ff88e01e76b65b858d155252dfc52bce3d9/src/main/js/bundles/dn_sketchingenhanced-construction/SketchingConstructionInfo.js#L65
https://github.com/conterra/mapapps-sketching-enhanced/blob/11f75ff88e01e76b65b858d155252dfc52bce3d9/src/main/js/bundles/dn_sketchingenhanced-construction/SketchingConstructionInfo.js#L89

resolved. I have added the geodesicLength calculation as well


add a function to retrieve years in which a series crosses a threshold

Please confirm that this PR has done the following:

[x] Tests Added
[x] Documentation Added
[x] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
This PR adds a pyam.cross_threshold() function to retrieve a list of years in which a pd.Series (e.g., a timeseries row) crosses a given threshold.

this looks good @danielhuppmann, but it's a little confusing because the crossing could be in the positive (below to above) or negative (above to below). I think that information should be included in what is returned.
Also, maybe add a test for one that does not cross a threshold?

thanks @gidden!
Added a test where cross_threshold() returns an empty list (because not crossing the threshold).
On returning the direction of crossing, not sure what would be suitable return format - any ideas? Alternatively, the function could take an additional optional from=['above', 'below'] kwarg that filters the returned list.

Yes, I would add a kwarg like direction.

I was thinking of direction at first, too, but how would you then name the options? direction=up is ambiguous. That's why I thought from would be more intuitive...

@gidden, added a kwarg direction=['from above', 'from below'] and three more tests for that feature...
Should be good to go once the tests pass


Extreme Noob alert-failure to run

So I've successfully installed everything including the CA cert but after I start HTTP server I'm completely lost.  Does the "How to code it" section go into coffee/terminal or should i be able to start  a localhost to view traffic?

this is all I'm seeing
coffee> PokemonGoMITM = require './lib/pokemon-go-mitm'
[Function: PokemonGoMITM]
coffee> server = new PokemonGoMITM port: 8081
[+++] PokemonGo MITM Proxy listening on 8081
[!] Make sure to have the CA cert .http-mitm-proxy/certs/ca.pem installed on your device
PokemonGoMITM {
handleProxyError: [Function],
handleProxyRequest: [Function],
port: 8081,
debug: false }
coffee> starting server for pgorelease.nianticlabs.com
https server started for pgorelease.nianticlabs.com on 61674
starting server for accounts.google.com
https server started for accounts.google.com on 61679

The how to code and the examples show you how to use the api of this project.
https://github.com/justinleewells/pogo-optimizer i.e. uses the pogo-mitm proxy to get the pokemon an calculates their stats.
If you dont want to code - use the examples or try pogo-optimizer.

Well that makes me feel dumb. I got everything sorted out. Thanks for the help!


[DO NOT MERGE] Try setting dist: focal on travis

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Request for Tooltip: Sell complete sets

@page1 commented on Mon Jul 23 2018

I'm not sure what **sell complete sets** does, when I click... it asks me to sign on my ledger... Adding a tooltip might help increase my confidence in using this feature.
In the mean time, what is this suppose to do?

@page1 commented on Mon Jul 23 2018
Coming back to this cold after the weekend and just a little confused. I'm sure I'll get this in a few minutes.... but on the topic of tool tips maybe adding some to all of these numbers would be good

A little puzzling how I can have a negative net position and positive quantity.

@bthaile commented on Tue Jul 24 2018
You should see the ‘you can sell a complete set’ that at the bottom of the trading page (should have the you have 1 complete set, would you like to sell it? -- SELL COMPLETE SET)
If you have 1 complete set that means you have 1 Yes share and 1 No share. You can trade that in for 1 ETH.
1 Yes share + 1.3 No shares will give you a net position of -.3 shares (negative means No shares)

@page1 commented on Tue Jul 24 2018
@bthaile Thanks for letting me know what a complete set is and what its getting traded in for.

@petervecchiarelli commented on Tue Jul 24 2018
https://app.clubhouse.io/augur/story/14479/github-request-for-tooltip-sell-complete-sets

@sharkcrayon commented on Wed Jul 25 2018
I think this also comes back to the Position numbers being confusing overall, which needs more than a tooltip to solve. This is high on our priority list. I'll throw some mocks in here when we have them to get your feedback on possible changes.

Closing as a duplicate of #418


Switch to Pug 3.0

so, any plans to support Pug 3.0 ?

I don't use pug 3 for anything personally, but I've tested and this loader seems to work fine with it. Are you having problems using pug 3? If you link a repo I'll look into supporting your use case.


Update probe-rs to version 0.13.0

This patch doesn't implement 64-bit support, but only inserts
conversions from u32 to u64 where necessary.

Thank you!
bors r+


Fixed documentation syntax error.

"/" is a special character and it was making the dots...italic.

Thanks for the fix! :)


Adding a mingw based build toolchain for windows

This toolchain is similar to the one used in Rubyinstaller's
DevKit. It is slightly different in that it uses tdb-gcc so
that it can build both 32-bit and 64-bit binaries. It should
be almost identical to the things ksubrama did in omnibus.
There are some idempotence issues with this implementation:

Once a package has been downloaded, it won't be unpacked
again after the first try. This means if the first go
fails, the next chef run won't fix itself

Also, build failures probably aren't because of this, master is failing. I can't even run rspec tests locally, it trys to upload my workspace directory to hosted chef I think

Test failures should be fixed by https://github.com/chef-cookbooks/build-essential/pull/81

@schisamo moved the function into a library

cc @chef-cookbooks/engineering-services

:+1:

:+1:

@schisamo attribute now defaults to SYSTEMDRIVE/mingw

:+1:

unless I'm using this wrong, this immediately blows up on non windows systems as SYSTEMDRIVE env var does not exist so File.join fails due to inability to implicitly convert from nil to string.

this is a good point

also, the readme is now out of date https://github.com/chef-cookbooks/build-essential#attributes as this PR added default['build-essential']['msys']['path'].  I would pull request that, but given my issues w/the PR as it stands, I figure I should hold off

https://github.com/chef-cookbooks/build-essential/pull/83

thanks @jaym


doc: add version in outputs document

version is included in the outputs, and it would be helpful to use instead of combine with major, minor, and patch.
See also:
https://github.com/google-github-actions/release-please-action/blob/cc61a07e2da466bebbc19b3a7dd01d6aecb20d1e/src/index.ts#L162-L193
https://github.com/googleapis/release-please/blob/72b0ab360c3d6635397e8b02f4d3f9f53932e23c/src/manifest.ts#L273-L280

Please merge this, really makes it easier to implement the action. Before I found this I combined the version myself and was wondering why there is not such an output...


Output file

Write output file (fasta) with the edited sequence

Since we're using biopython already:
https://biopython.org/wiki/SeqIO
They offer a simple sequence to fasta function.

I just realized I already implemented this function right in the beginning, but forgot about it...

;)


derhuerst and juliuste

Try to link to all git project of derhuerst and juliuste
There are so many repos, like:

https://github.com/public-transport
A list of DB stations: https://github.com/derhuerst/db-hafas-stations

https://transport.rest
https://github.com/public-transport/friendly-public-transport-format
https://github.com/public-transport/hafas-client
https://github.com/public-transport/gtfs-utils
https://github.com/public-transport/gtfs-to-fptf
https://github.com/public-transport/generating-transit-maps
https://github.com/public-transport/state-of-the-open
https://github.com/juliuste/eurolines-de
https://github.com/juliuste/tallink
https://github.com/juliuste/interrail
https://github.com/juliuste/sncf
https://github.com/juliuste/flix
https://github.com/juliuste/zhv-de
https://github.com/juliuste/ouibus
https://github.com/juliuste/osm-transit-lines
https://github.com/juliuste/ecolines
https://github.com/juliuste/hafas-osm-line-colours
https://github.com/juliuste/korail
https://github.com/juliuste/zpcg
https://github.com/juliuste/train-ose
https://github.com/juliuste/bilkom
https://github.com/juliuste/mav
https://github.com/juliuste/comboios
https://github.com/juliuste/lvb
https://github.com/juliuste/slovenske-zeleznice
https://github.com/juliuste/oebb
https://github.com/juliuste/mdv-lines
https://github.com/juliuste/koleo
https://github.com/juliuste/deinbus
https://github.com/juliuste/db-flix-cities
https://github.com/juliuste/pkp-ic
https://github.com/juliuste/direkt.bahn.guru
https://github.com/juliuste/travel-price-map
https://github.com/juliuste/build-mav-gtfs
https://github.com/juliuste/build-cp-gtfs
https://github.com/juliuste/transit-map
https://github.com/juliuste/nahsh-hafas
https://github.com/juliuste/oebb-hafas
DB:
https://github.com/juliuste/bahn.guru
https://github.com/juliuste/db-wagenreihung
https://github.com/juliuste/db-perrons
https://github.com/derhuerst/db-netz-hafas
https://github.com/derhuerst/db-hafas
https://github.com/derhuerst/current-ice-speed
https://github.com/derhuerst/db-stations-autocomplete
https://github.com/derhuerst/fetch-db-operating-status
https://github.com/derhuerst/db-stations
https://github.com/derhuerst/db-cli
https://github.com/derhuerst/generate-db-shop-urls
https://github.com/derhuerst/db-hafas-stations
https://github.com/derhuerst/discover-db-stations
https://github.com/derhuerst/compute-db-station-weight
https://github.com/derhuerst/generate-db-graph
VBB:
https://github.com/derhuerst/vbb-modules
https://github.com/juliuste/vbb-change-positions
https://github.com/juliuste/vbb-platform-patterns


shader implementations should go into stdlib

it's fine to provide these as some defaults to get people started, but that's all they are. They should not be part of the core.

the glsl code should go into stdlib as well.

This was resolved as part of #70 . Haven't opened a PR with the changes yet though.


mill: enforce repo is absolute path

Hello @landism, @dmiller,
Please review the following commits I made in branch maiamcc/enforce-abs-repo:
82f8ab53ff1890c62dd166ee29efc0d270458d85 (2018-08-24 12:11:28 -0400)
mill: enforce repo is absolute path
e62010ab1889ca34c5b74a9275540fe2f331b93d (2018-08-24 11:51:23 -0400)
wip
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

ping @nicks @landism


Add jupyter client as a build req for the notebook files

@willingc I'm seeing a jupyter_client missing dependency error on RTD.  I'm not quite sure what of nbsphinx is triggering the error.  Let me know if you see it too.  If you do, we'll add it to the sphinx theme.

Ah here's some good information to know:
https://github.com/spatialaudio/nbsphinx/blob/0.2.6/nbsphinx.py#L389-L397
nbsphinx attempts to execute a notebook if no outputs exist and yet code cells exist.

Thanks Jon.

@jdfreder We should probably add a readthedocs.yml and environment.yml to build using conda on RTD

I don't use conda myself, but I do not object to switching

Which repo do you want me to do a pr against.

A PR against ipython/ipywidgets directly would be nice!

Thanks :grin:

okie dokie


SDK throws exceptions for serialized null values

Hello, i'm Norman, Software Engineer at idealo and i would like to draw your attention to a problem in this library. One of our applications printed some unexpected exception stack traces after we turned on one of our A/B tests. The problem is that the ConditionEvaluator is not able to handle serialized null values for all possible operators (e.g. $in, $nin, etc. ). This is a problem for us, because the evaluation process stops and our log-shipper expects a specific format for all messages/text that will be printed to stdout/stderr.
We would like to offer a possible fix for that use case (Pull Request #48), that should make the ConditionEvaluator more robust.
Feedback is highly appreciated and welcome!
Best regards,
Norman on behalf of idealo

Issue closed due to pull request successfully approved and merged.


Fixed confirmation dialog styles

They weren't mobile friendly, and were glitchy on Chrome/Windows.
Previous style was to have two elements, an overlay and container, where
the overlay gave a translucent background and the container was
center-positioned via table displays with a white background. This is
risky structure and a bit complicated.
Now, the overlay contains the container, and they use flexbox for
centering.

@juliusl This is what I meant by using flex.

Cool -- Thanks for taking care of it.


NPE on StartPreview after stopRecording

java.lang.NullPointerException: Attempt to invoke virtual method 'android.hardware.camera2.CaptureRequest$Builder android.hardware.camera2.CameraDevice.createCaptureRequest(int)' on a null object reference
at camera2.Camera2Fragment.startPreview(Camera2Fragment.java:1093)
at camera2.Camera2Fragment.stopRecordingVideo(Camera2Fragment.java:1270)

This sample has been migrated to a new location (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


[SPARK-23431][CORE] Expose the new executor memory metrics at the stage level

What changes were proposed in this pull request?
Collect and show the new executor memory metrics for each stage, to provide more information on how memory is used per stage. Peak values for metrics are show for each stage. For executor summaries for each stage, the peak values per executor are also shown.
How was this patch tested?
Added new unit tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #100275 has started for PR 23340 at commit d77612c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6245/
Test PASSed.

Test build #100275 has finished for PR 23340 at commit d77612c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100275/
Test FAILed.

Jenkins, retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6257/
Test PASSed.

Test build #100289 has started for PR 23340 at commit d77612c.

Test build #100289 has finished for PR 23340 at commit d77612c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100289/
Test PASSed.

Test build #100294 has started for PR 23340 at commit 42599fe.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6263/
Test PASSed.

Test build #100294 has finished for PR 23340 at commit 42599fe.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100294/
Test FAILed.

Test build #100324 has started for PR 23340 at commit 3dd8424.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6291/
Test PASSed.

Test build #100324 has finished for PR 23340 at commit 3dd8424.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100324/
Test PASSed.

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10153/
Test PASSed.

Test build #104844 has started for PR 23340 at commit 6d6f64c.

Test build #104844 has finished for PR 23340 at commit 6d6f64c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104844/
Test PASSed.

@squito and @rezasafi , could you please take another look? Thanks for your review and comments.

Ping—any chance someone could review this patch? These are useful metrics.

@squito @rezasafi any chance either of you could review this patch?

This is reopened according to the community request. I also removed the label, Stale.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@edwinalu thanks for getting this started so long ago.  Looks like there are some conflicts after so much time has passed.  Do you have time/interest to continue working on this or would you like some help getting this sorted out?

@stackedsax , I don't have time to continue working on this unfortunately. If you or someone else is able to help with adding stage level metrics, that would be great. Please let me know if there's any questions about the original PR.

@stackedsax, do you plan to work on this? Otherwise, I can start looking into this next week. Thanks!

@imback82 I'll be taking this up if you don't (on behalf of @stackedsax), but since I am not a Spark contributor at the moment it'd probably be a lot faster and easier if you did it. So please do look into it—thank you!

Thanks @itamarst. I started looking into this and will update this thread.

I created a PR here: #29020. I will also ask a reviewing committer to put @edwinalu as a co-author if that's the practice when a PR is "inherited". Thanks!

@imback82 Looks like #29020 is merged into 3.1.0.  Should this issue get closed now, or is it being left open for someone to do the same in 2.x versions of Spark?

This can be closed now. We can use changes in #29020 if backporting to 2.4 is needed. (@dongjoon-hyun / @gengliangwang can confirm, but I don't think this will be backported to 2.4.)


[Feature] Handle system packages for Mac OS X

Currently the system utils ic is handling Ubuntu and Windows packages only. We need to extend this to Mac OS X and non-debian linux flavours too.
https://github.com/octoml/cm-mlops/blob/main/ic/prototype-get-sys-utils-cm/customize.py

Hi Arjun,
We get the basic host OS info using Python whenever we start running any IC. We get this info here: https://github.com/mlcommons/ck/blob/master/ck2-repo-mlops/automation/utils/module.py#L55
cm get_host_os_info utils

This is not a very sophisticated method and it's just used to understand if IC is run on Windows (to use .bat files) or on any sort of Linux (to use .sh files).
I believe that we should add more sophisticated methods to detect OS in the IC "detect-os": https://github.com/octoml/cm-mlops/tree/main/ic/prototype-detect-os
So if a new IC relies on extended OS info (not just Windows or Linux but MacOS vs Debian OS), we need to add a dependency on detect-os which will return extra env related to the OS FLAVOR and related tools (apt yum; lscpu ...) ... What do you think?

I believe it's done now!


realtime access gets 403 in kong

Bug report
when accessed realtime websocket gets 403 in kong
Describe the bug
In default config and deployment realtime functions end up with 403 in kong
for example:
<IP_ADDRESS> - - [06/Nov/2024:22:01:41 +0000] "GET /realtime/v1/websocket?apikey=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyAgCiAgICAicm9sZSI6ICJhbm9uIiwKICAgICJpc3MiOiAic3VwYWJhc2UtZGVtbyIsCiAgICAiaWF0IjogMTY0MTc2OTIwMCwKICAgICJleHAiOiAxNzk5NTM1NjAwCn0.dc_X5iR_VP_qT0zsiyj_I_OZ2T9FtRU2BBNWN8Bu4GE&vsn=1.0.0 HTTP/1.1" 403 0 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
Same config works with supabase docker example. Tried both with and without ingress.

I have the same issue with the newest version.


Update geth merge fork block config

Update merge for block config. Geth changed the config name here.
Because of this mismatched, get node in eth2-merge-kurtosis which uses this generator does not generate correct fork id, which will disconnect nethermind's node, which is what I'm working on.

Tested manually, confirm fork id is now generated correctly.

@MariusVanDerWijden @holiman


uadk: schedule - modify schedule parameter assignments

Now the schedule design three patameters. 'type' is not a common setting.
'numa_id' is a patameter decided by users. So assign it in create 'sess'.
Signed-off-by: Yang Shen<EMAIL_ADDRESS>Signed-off-by: Haojian Zhuang<EMAIL_ADDRESS>

Thx. When I added this patch, I found the 'sched_key' is unused in aead/cipher/digest. So I only fixed the problem in 'hisi_zip_test'.

Thx. When I added this patch, I found the 'sched_key' is unused in aead/cipher/digest. So I only fixed the problem in 'hisi_zip_test'.

Please help to submit your PR with your new additional patch. Let's check the status now.


tcav python module updated recently making implementation incompatible

Traceback (most recent call last):
File "/gdrive/My Drive/braxai/ACE-master/ace_run.py", line 12, in
from ace import ConceptDiscovery
File "/gdrive/My Drive/braxai/ACE-master/ace.py", line 18, in
from tcav import cav, tcav_helpers
ImportError: cannot import name 'tcav_helpers'

Thanks a lot! The issue is fixed!


Why is "symfony/thanks" added to composer.json?

Lots of discussion about this on slack.
And if this is a skeleton why it has package that doesn't have really meaning ?

Just to invalidate a misconception I saw on Slack: the thanks plugin doesn't do ANYTHING unless YOU call "composer thanks".
The meaning of this package is to allow thankful users to discover the command easily, and then use it when they want. Of course, it's also to promote sending good vibes to OSS authors.

First of all, of course, it is very good initiative to "say thank you" in some way for all those developers who share their work for a community.
But I thought the meaning of Symfony is to set highly professional standards. Including "thanks" doesn't feels really professional.
Why should it be a "surprise" to see during installation process that there's some package installing that has nothing to do with the framework that you choose for your project?
Then you need to spend your time to discover what is this, why it's even included in the skeleton, what it's actually doing and then remove it, because you already supporting those developers with your way to "thanks" them.

Saying "thanks" is very professional. But I would also say it is one of the most missing interaction in enterprise world...
There are no single way to say thanks, and they are no exclusive to each others.
OSS is never cold code: it's built by actual human beings. See the talk from Compsoer's Jordi and many other OSS leaders to know how much negative vibes they get instead of good ones. I think we need to actively find ways to counter balance this. And every stones count to achieve this.
Last: nobody forces anyone doing anything. So if you don't like what some authors pushes forward (like this plugin), just don't use their code, or fork their code, that's fine.

Yes, let's fork symfony/skeleton and tell the world to use whatever/symfony-other-skeleton-without-thanks-because-it-is-not-professional. I'm sure everyone will totally agree that it's legit and professionnal.
I'm totally fan of the idea of including symfony/thanks: this leads to a better view of the open-source community when developers start a new project.
For years, I met a lot of companies that said things like "Symfony is too big/bloated/overkill/complex/for the elite/selfish" and other dump and false words, sometimes much harder and violent words.
By helping developers saying "thanks", it's like a worldwide educational project: everyone saying "thanks" will probably unconsciously realize that the open-source community is full of human beings that deserve to be thanked for their free work, because OSS is free, and built in the free time of every contributor.
Making the IT community grow worldwide to help all companies make more money, doesn't this deserve at least a single command-line instruction that you can use anytime?

@nicolas-grekas "nobody forces anyone doing anything"

This statement opposite to the fact that something like "thanks" package included in the skeleton.
This doesn't solve any problem, because people who don't care, for them this is just bull**it. And people who cares already doing everything they can to support those cool devs.

So, again. There's no sense in this.

The thanks package is a great idea. Low cost, low effort and big impact to our ecosystem that has a high net win. This issue and the comments on it are being overly pedantic for no good reason. This issue does more harm to the project and it’s maintainers than anything.

You're right @jwage, and I would even go further: this issue could be close right now without warning.
As nicolas-grekas said above, we could just search for a few of seldaek's blog posts about people yelling at him about something they don't like to see that people who don't accept to thank OSS maintainers for their hard work do not even deserve a publicly opened issue and discussion.
And I would go even more further by saying this:
symfony/thanks should even be mandatory (and even integrated to Composer), because when you use free projects to make money, it's mandatory to be kind with them. Period.

And it's a dev dep... It will not even reach your production server.

@nicolas-grekas Thanks for clear answer about this.
Seems like this still divides community - which is really shame. And of course everyone can has different opinion about this. But now that question has a answer it's pointless to continue this discussion within this issue, so closing this.
And to make clear, personally I haven't anything against that thanks package, it's really usefull tool for some people.

I'm locking as some comments were deleted, so the remaining ones are a bit out of context now.


Release 0.0.1

@matthew-andrews if I can be so bold, would you kindly merge this PR to update the package.json version, and run npm pub - to put #27 into effect.

We're at v2.1.1 already for this module and Travis has already released v2.1.1 it for me :relaxed:.

https://travis-ci.org/matthew-andrews/isomorphic-fetch/builds/71244301
[ https://mattandre.ws/2015/01/npm-releasing-codeship-jenkins-travis/ ]


Confused how this would ever work with a physical file (limit on)

Hi, thanks for this plugin. It works if I use it in combination with ExtractTextPlugin, where the urls get re-written on output.
However if it's using the normal blobs, then the paths in the blobs don't get re-written.
For example in a node_module I have a font-face decl. with src: url("../fonts/bla" and this is literally in the blob, but of course that's wrong and it needs to be based on where the assets have been copied to.
For example for loader: 'url-loader?name=assets/[name].[ext]', I would expect the url to be changed to be src: url("assets/fonts/bla", otherwise how would it find the file?
But also of course physical files don't get copied until I run this with ExtractTextPlugin, so I'm unsure how it would ever work in dev mode with a limit set?
Thanks for any pointers!

@DominicTobias Could you provide more info about your setup please? Maybe you are looking for something like copy-webpack-plugin or similar, which copys the physical files to the dest folder in case you don't require them within you're bundle

Hi np it's too long since I was looking at this now so I'm going to close it 👍


Enriched with examples to use the extracted xpub

The syntax of the BIP32 is tricky:
BIP32 utils wants "m" in the beginning.

Instead of place holders, can you use an actual example?

Absolutely. I just have to configure a new ledger, so probably not today :P.

Also, I think this would be better suited as it's own document instead of as part of the ledger instructions.

Like an examples.md in the doc directory?

I didn't know how to cancel my editing to Ledger.md so I removed them. If you want I could close and delete the PR and resubmit it if you tell me how.

Just use git rebase -i to squash/drop/reorder your commits as you want and the force push (git push -f) the changes.

Thank you.


Insert Local File instead of link

How do I train the model with my local files? Suppose I have a pdf in root directory and I want to add it like mygpt.add("pdf_file", "book.pdf"). Is it possible?

you could just read the code you know :)
my_bot.add("pdf_file",'/Users/youruser/Downloads/yourpdf.pdf')

thanks.


feat: add android gravity docs

Description

Adds docs for the gravity attribute for TextElement

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.90.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Duplicate type declarations on some libraries

Problem
While trying to import xss:
import { filterXSS } from 'https://esm.sh/xss'

I came across with this error:
deno run -r render.ts
Download https://esm.sh/xss?nocheck
Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Check file:///home/v1rtl/Coding/deno-libs/gql/graphiql/render.ts
error: TS2300 [ERROR]: Duplicate identifier 'OnTagHandler'.
      type OnTagHandler = (
           ~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:93:12

    'OnTagHandler' was also declared here.
          type OnTagHandler = (
               ~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:281:12

TS2300 [ERROR]: Duplicate identifier 'OnTagAttrHandler'.
      type OnTagAttrHandler = (
           ~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:99:12

    'OnTagAttrHandler' was also declared here.
          type OnTagAttrHandler = (
               ~~~~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:287:12

TS2300 [ERROR]: Duplicate identifier 'SafeAttrValueHandler'.
      type SafeAttrValueHandler = (
           ~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:106:12

    'SafeAttrValueHandler' was also declared here.
          type SafeAttrValueHandler = (
               ~~~~~~~~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:294:12

TS2300 [ERROR]: Duplicate identifier 'EscapeHandler'.
      type EscapeHandler = (str: string) => string;
           ~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:113:12

    'EscapeHandler' was also declared here.
          type EscapeHandler = (str: string) => string;
               ~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:301:12

TS2300 [ERROR]: Duplicate identifier 'OnTagHandler'.
      type OnTagHandler = (
           ~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:281:12

    'OnTagHandler' was also declared here.
          type OnTagHandler = (
               ~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:93:12

TS2300 [ERROR]: Duplicate identifier 'OnTagAttrHandler'.
      type OnTagAttrHandler = (
           ~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:287:12

    'OnTagAttrHandler' was also declared here.
          type OnTagAttrHandler = (
               ~~~~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:99:12

TS2300 [ERROR]: Duplicate identifier 'SafeAttrValueHandler'.
      type SafeAttrValueHandler = (
           ~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:294:12

    'SafeAttrValueHandler' was also declared here.
          type SafeAttrValueHandler = (
               ~~~~~~~~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:106:12

TS2300 [ERROR]: Duplicate identifier 'EscapeHandler'.
      type EscapeHandler = (str: string) => string;
           ~~~~~~~~~~~~~
    at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:301:12

    'EscapeHandler' was also declared here.
          type EscapeHandler = (str: string) => string;
               ~~~~~~~~~~~~~
        at https://cdn.esm.sh/v28/xss@1.0.8/typings/xss.d.ts:113:12

Found 8 errors.

Temporary solution
Add the ?no-check to module URL.

not an issue anymore in v47, closing


deps: bump sigs.k8s.io/karpenter to v0.35.10

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.165%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
5692

Relevant Lines:
7282

💛  - Coveralls


Bug SP-left

sp level bug in the skill upgrade form section. Exit buttons below and above do not work, if clicked they will not exit the form and SP-Left will increase...
like in a screenshot
https://www.dropbox.com/s/xmyl103o49ir5bh/Screenshot 2020-08-03 11.04.44.png?dl=0
thanks, this is cool plugin

Oof... Yes it is poorly tested, if tested at all. Thanks for the feedback I'll look into it

Should be good to go now at least that issue you pointed out 👍 Thanks


continue watching, might need new player

can do season / episode remember over local or future cloud. remembers video progress over local storage, probably cant do over cloud.


Restore NFC HCE documentation

Please consider restoring the documentation explaining Host Card Emulation which was deleted in commit 9c27bf.
The article serves as a good introduction to the public Windows.Devices.SmartCards.SmartCardEmulator APIs, which we are using in an embedded application. Additionally, it provides a good overview of HCE technology in general and complements other available resources on the Internet. The article is cross-referenced elsewhere in docs.microsoft.com and the NFC sample, and I would propose restoring it to maintain those links.
While it's still possible to read from the Git history, there would be no discoverability for such a good resource.

@tigerw - can you follow the instructions to sign the CLA here: https://github.com/MicrosoftDocs/windows-dev-docs/pull/4482#issuecomment-1585142279
#assign: @Karl-Bridge-Microsoft
@Karl-Bridge-Microsoft

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks @tigerw - Looking at fixing this in a private PR. Will update and close this PR when complete.

Hi @tigerw - ok, I addressed this in a private branch PR. Closing this PR.


Add component builder to the default prim validator

Changelog Description
Add component builder to the default prim validator.
This PR is made for future reference. For more info check this comment https://github.com/ynput/ayon-houdini/pull/163#discussion_r1852875084
Testing notes:

Create a Houdini component builder instance.
Turn on contribution workflow and set it to asset.
The validator should error if the rootprim of the component builder is not the same as the folder name.

Closing this as it's not needed.


RecursiveCharacterTextSplitter  for Chinese sentence

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
for the example sentence "天青色等烟雨，而我在等你。哈哈哈....."
when I use RecursiveCharacterTextSplitter  split it by ["\n\n", "\n", "。"]，the result shows that the "。"will be appended in the beginning of the sub-sentence.for example:[Document(page_content="。哈哈哈...")]
Error Message and Stack Trace (if applicable)
No response
Description
for the example sentence "天青色等烟雨，而我在等你。哈哈哈....."
when I use RecursiveCharacterTextSplitter  split it by ["\n\n", "\n", "。"]，the result shows that the "。"will be appended in the beginning of the sub-sentence.for example:[Document(page_content="。哈哈哈...")]
System Info
langchain 0.1.0

Text splitters have a keep_separator kwarg. I was able to reproduce your issue, but if I set this kwarg it appears to be fixed:
from langchain_text_splitters import RecursiveCharacterTextSplitter

input_string = "天青色等烟雨，而我在等你。哈哈哈....."

splitter = RecursiveCharacterTextSplitter(
    separators=["\n\n", "\n", "。"],
    chunk_size=20,
    chunk_overlap=0,
    keep_separator=False,
)
splitter.split_text(input_string)

['天青色等烟雨，而我在等你', '哈哈哈.....']

Please provide a minimal example reproducing the issue if this doesn't fix your problem.


Add note on hostname in /etc/hosts

MacOS users may have trouble running topologies locally if they don't add their machine's hostname to /etc/hosts. This PR adds a note about that to the Getting Started doc.
I also fixed a persistent issue with shortcodes that have been affecting the website build process. It turns out that shortcodes must be HTML and cannot be in the Ace template format.

LGTM.


PL-400 Changes June 2021

NOTE All of the diagrams use the old menu items. They all need to be changed to the new menu items.

Lab 00:
o	Step 7 – Enter<PHONE_NUMBER> into the phone number field.
o	Step 8 – Click Get Started.
o	Step 9 – Click Get Started.
o
Lab 2 - Exercise 1:
o	Task 1 – Step 3 – Change to + Add Component.
o	Task 1 – Step 4 – Change to + Add Component.
Lab 2 - Exercise 2:
o	Task 1 – Step 2 Change to + New

Thank you for reporting this. We are releasing an update in the next couple of weeks.


Update GoogleMapHeatmapController.h

change import file resolve ios build issue

Resolve IOS build issue

Resolve IOS build issue


Can't get initial example working, even on vanilla rails install - only last element renders

The example at https://phlex.fun under "Why render HTML with Ruby?" only renders the final child.
ruby 3.2.2 (2023-03-30 revision e51014f9c0) [arm64-darwin23]
I setup a blank app (tried on Rails 7.1.2 and <IP_ADDRESS>)
rails -v
# => Rails <IP_ADDRESS>
rails new TestingPhlex
bundle add phlex-rails
rails g phlex:install
rails g controller posts index

# app/views/components/nav.rb as per docs
class Nav < Phlex::HTML
  def template(&content)
  	nav(class: "main-nav") {
	  ul(&content)
  	}
  end

  def item(url, &content)
  	li { a(href: url, &content) }
  end
end

# app/views/posts/index.html.erb as per docs
<%= render Nav.new do |nav|
	nav.item("/") { "Home" }
	nav.item("/about") { "About" }
	nav.item("/contact") { "Contact" }
end %>

Output received

Contact

Expected - as detailed in docs.

Home

About

Contact

This code only works from another Phlex context, but you’re calling it from ERB.
render Nav.new do |nav|
  nav.item("/") { "Home" }
  nav.item("/about") { "About" }
  nav.item("/contact") { "Contact" }
end

To use it in ERB, you would have to use ERB output tags, e.g.
<%= render Nav.new do |nav| %>
  <%= nav.item("/") { "Home" } %>
  <%= nav.item("/about") { "About" } %>
  <%= nav.item("/contact") { "Contact" } %>
<% end %>

Well Joel - that's an outrageously fast reply.  Thank you!  It might be worth adding this to the docs although with hindsight it's completely obvious and doesn't seem to have tripped anyone else up!

Well Joel - that's an outrageously fast reply. Thank you!

We get notifications in the Naming Things Discord so I saw this straight away 😅
Totally agree about the docs. We’re preparing to release 2.0, which will have completely new docs and hopefully make this kind of thing much clearer. 🙏

Brilliant, I'm a convert!  It's not immediately obvious from the Github that clicking the logo takes you to the docs.  Maybe you could add a 'Read the docs' text link below.
Thanks for your work on this!  There's a beauty to it: I think it gets the closest yet in Ruby to the niceties of building UI in, eg. React.

Trying to replicate this with ViewComponents just doesn't feel natural to me: the concept of Slots isn't right to represent multiple different types of content.
To represent the same (using Github's Primer as an example):
render(Primer::Alpha::ActionMenu.new(select_variant: select_variant, anchor_align: anchor_align, anchor_side: anchor_side, size: size)) do |menu|
  menu.with_show_button { "Menu" }
  menu.with_item(label: "Copy link")
  menu.with_item(label: "Quote reply")
  menu.with_item(label: "Reference in new issue")
  menu.with_divider
  menu.with_item(label: "Edit")
  menu.with_item(label: "Delete", scheme: :danger)
end

<%# An effort with Phlex %>
<%= render UI::Dropdown.new do |dd| %>
  <%= dd.item { 'An item' } %>
  <%= dd.item { 'Another item' } %>
  <%= dd.separator %>
  <%= dd.item do %>
    And finally...
  <% end %>
<% end %>

Much better!  In my mind, ActionHelper is methods (one or two liners). Phlex is builders (classes, generators, components, whatever people want to call them).
Just a brain dump!  No reply needed!

Brilliant, I'm a convert! It's not immediately obvious from the Github that clicking the logo takes you to the docs. Maybe you could add a 'Read the docs' text link below.

👍 Agreed

I think it gets the closest yet in Ruby to the niceties of building UI in, eg. React.

You should see what’s coming in 1.10 with the new Kit feature! (previously called Buckets). There’s an example in this PR. https://github.com/phlex-ruby/phlex/pull/676


WIP: add comments in tutorial

Brilliant, thanks a lot! Let me know when this is no longer WIP.

You can have a look. I've marked with TODO @Eduardo the places that you may want to change or extend.
I've also added zfit and phasespace to the conda packages in the environment.yml

Thanks very much @mayou36! I will accept this as is and will later review everything altogether.


Add "tanzu context get-token" command to fetch a valid CSP token for the given context

What this PR does / why we need it
This PR adds "tanzu context get-token" command to fetch a valid CSP token for the given context
Changes Summary:

Add a hidden command "tanzu context get-token" to fetch a valid CSP token for the given context. This command would be used in the kubeconfig generated for UCP resource to fetch/refresh the access-token dynamically.
Updated the kubeconfig generation logic to include the exec plugin to fetch the access token dynamically
Command would return error if the context provided is not an UCP context

Which issue(s) this PR fixes

Fixes #
Describe testing done for PR
Created the UCP context and ran the tanzu context get-token and it successfully printed the ExecCredentials to stdout.
❯ ./bin/tanzu context list
Target:  kubernetes
  NAME           ISACTIVE  ENDPOINT                                                                         KUBECONFIGPATH                           KUBECONTEXT
  test-withcert  false     https://<IP_ADDRESS>:6443                                                      /Users/pkalle/.kube-tanzu/config         tanzu-cli-tkg-mgmt-vc@tkg-mgmt-vc
  tkg-mgmt-vc    false                                                                                /Users/pkalle/temp/tkgCluster_admin.kfg  tkg-mgmt-vc-admin@tkg-mgmt-vc
  myucp          true      https://api-dev.tanzu.cloud.vmware.com/org/bc27608b-4809-4cac-9e04-778803963da2  /Users/pkalle/.kube/config               tanzu-cli-myucp
Target:  mission-control
  NAME            ISACTIVE  ENDPOINT
  tt-test-selfmg  false     tmc-sm-main.local-dev.7infra.com:443
  mytmc           false     unstable.tmc-dev.cloud.vmware.com:443
  mytmc2          true      unstable.tmc-dev.cloud.vmware.com:443

❯ ./bin/tanzu context get-token myucp
{"kind":"ExecCredential","apiVersion":"client.authentication.k8s.io/v1","spec":{"interactive":false},"status":{"expirationTimestamp":"2023-09-27T22:08:24Z","token":"<REDACTED>"}}

Tested with context that is not of UCP type
❯ ./bin/tanzu context get-token tkg-mgmt-vc
[x] : context "tkg-mgmt-vc" is not of type UCP

Created a UCP context and verified the kubeconfig has user information with ExecConfig with reference to tanzu context get-token
❯ ./bin/tanzu context create myucp --endpoint https://api-dev.tanzu.cloud.vmware.com --type application-engine  --staging
[i] API token env var is set

[ok] successfully created a Application Engine(UCP) context

//kubeconfig generated

apiVersion: v1
clusters:
- cluster:
    server: https://api-dev.tanzu.cloud.vmware.com/org/bc27608b-4809-4cac-9e04-778803963da2
  name: tanzu-cli-myucp/current
contexts:
- context:
    cluster: tanzu-cli-myucp/current
    user: tanzu-cli-myucp-user
  name: tanzu-cli-myucp
current-context: ""
kind: Config
preferences: {}
users:
- name: tanzu-cli-myucp-user
  user:
    exec:
      apiVersion: client.authentication.k8s.io/v1
      args:
      - context
      - get-token
      - myucp
      command: tanzu
      env: []
      provideClusterInfo: false

Release note

Add "tanzu context get-token" command to fetch a valid CSP token for the given context

Additional information
Special notes for your reviewer

Might be good to document the behavior when the tanzu cli is found but get-token is not, and when tanzu is not found.
this is found during development, when I have the released and build binaries of cli named differently.
but it would be a potential issue for other customers
~> kubectl api-resources
NAME   SHORTNAMES   APIVERSION   NAMESPACED   KIND
Unable to connect to the server: getting credentials: decoding stdout: yaml: line 3: mapping values are not allowed in this context

this is a rarer situation, but could happen if tanzu is somehow not in the user's PATH
~> kubectl api-resources
NAME   SHORTNAMES   APIVERSION   NAMESPACED   KIND
Unable to connect to the server: getting credentials: exec: executable tanzu not found

It looks like you are trying to use a client-go credential plugin that is not installed.

To learn more about this feature, consult the documentation available at:
      https://kubernetes.io/docs/reference/access-authn-authz/authentication/#client-go-credential-plugins

Not to block this change, but we might want to take note to see if we can do some validation (by actually test running "tanzu context get-token" or something) before assigning the invocation to the kubeconfig's Exec

Not to block this change, but we might want to take note to see if we can do some validation (by actually test running "tanzu context get-token" or something) before assigning the invocation to the kubeconfig's Exec

Created an Issue#506 to track and address it


[components]: Create custom FlipCard

Review existing implementation in Meshery UI
https://github.com/meshery/meshery/blob/master/ui/components/FlipCard.js

[ ] Update FlipCard.js to use MUI v5 and its styles (inline or not)
[ ] Update imports to use existing components in sistent-components
[ ] Place custom component in custom directory, and export appropriately
[ ] Ensure that the component works properly locally before pushing PR for testing

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

Would like to request assignment to this issue. I think I have successfully migrated the component on my local machine.


Add verbose output of file offsets and virtual addresses for language-specific strings

This PR enhances the verbose output to display raw file offsets and virtual addresses for Rust strings.
Fixes: #925 #937
*The PR is in draft status because it currently only supports Rust strings.

do you think we should add column headers?

Currently looks like this in verbose:-


Combine notifyItemMoved() and notifyItemChanged()

Sometimes I have an item which moves position and also changes appearance at the same time. I've tried calling both notify methods but this totally messes up the animation.
I imagine this is quite a common scenario.
Ideally, the item would animate to the new appearance as it moves. However, the appearance changing suddenly at the start or end of the move would also be acceptable to me.
Is there any workaround?

Seems it is working as you expected. I've created branch for you in sample https://github.com/BelooS/ChipsLayoutManager/tree/test/notifyChange%26move. Try to move some item with move button.
Before move i call notifyItemChanged with the old position and then notifyItemChanged. See it in ItemsFragment.


vs2 feature

PLEASE READ THE GUIDELINES BEFORE MAKING A CONTRIBUTION

Please check if the PR fulfills these requirements

[x] The commit message are well described
[ ] Docs have been added / updated (for features or maybe bugs which were noted). If not, please update the needed documentation here. Feel free to remove this check if you don't need it
[x] All changes have fully been tested

What kind of change does this PR introduce? (Bug fix, feature, ...)
Feature

What is the new behavior (if this is a feature change)?
IntelligenceModding/Advanced-Peripherals-Features#89

Does this PR introduce a breaking change? (What changes might users need to make in their scripts due to this PR?)

Other information:

TODO:

[x] Player Detector
[x] Environment Scanner
[x] Geo Scanner
[x] Distance Sensor
[x] Chatbox
- [ ] Autometa Turtle (maybe another day)

I love that feature


Put T3_ES_PIC_BSC in production: Update CERN & FNAL resource control accordingly

Impact of the new feature
WMAgent
Is your feature request related to a problem? Please describe.
We decided to start testing T3_ES_PIC_BSC for production using regular production agents and we want our agents to work with this site.
Describe the solution you'd like
Update existing and incoming CERN & FNAL agents to include this site in their resource control.
Describe alternatives you've considered
None.
Additional context
None

Thanks, Hasan!
I'd like to add that the typical scale at which we would like to operate BSC is 5k CPU cores. (=about 2.5 times that of PIC).
As files produced by jobs running at BSC will be merged at PIC, we might need to rescale merge limits for PIC as well. What is the current total limit of merge for T1_ES_PIC?

@aperezca Hi Antonio, current limits for T1_ES_PIC are (pending slots vary according to the number of agents up and running with the same team name):
T1_ES_PIC - 106 running, 680 pending, 4820 running slots total, 723 pending slots total, Site is Normal:
  Merge - 0 running, 1 pending, 120 max running, 25 max pending, priority 4

so, 4.8k jobs in total and up to 120+25 merge jobs.
Regarding T3_ES_PIC_BSC, I have just added it to vocms0252 such that tests can proceed. It's set with 3k running and 2k pending jobs (a total of up to 5k jobs).
A permanent or better solution is still to be implemented.
Antonio, are these BSC resources going to be available for a somehow long period?

Hi Alan,
On PIC, thanks for the info, we might need to increase the amount of merge jobs, let's see as we advance in our expertise running PIC+BSC combined (all merge is done at PIC)
On BSC, thanks for adding it to the covms0252 agent and schedd. Yes, I already see some jobs in queue for T3_ES_PIC_BSC. And 5k jobs should be sufficient.
"A permanent or better solution is still to be implemented." => We have been discussing this with Stephan (see ticket). He pointed me to the Site Capacity data, which I have already edited for BSC: https://cmssst.web.cern.ch/cgi-bin/set/SiteCapacity/T3_ES_PIC_BSC
I hope this is the permanent solution we need, as indeed BSC CPU resources are expected to become part of the future pledge of our Tier1 at PIC. We already have a collaboration agreement signed so we, from PIC, can use BSC to run our LHC VOs tasks (ATLAS, CMS and LHCb). This allows us to request and get assigned some CPU fraction regularly at BSC.

I don't know why I haven't thought about this over the past 4 years(!!), but we should simply add another command line option in this script:
https://github.com/dmwm/WMCore/blob/master/bin/wmagent-resource-control
such that we can add opportunistic resources with a single command line without the CRIC dependency.


Add Table Sort Behavior

Would be nice if table rows have subtle hover effect.
Also customised sorting  js may be attached in js source files because the kylefox's tablesort plugin needs heavy modifications because of classes names.

:+1: on the hover effect   @jlukic would you like me to create an issue for table hover effect?

tablesort.js has an issue when sorting classes with events. Changing the lines:
rows.each(function(i, tr) {
    fragment.append(tr);
});
table.append(fragment.html());

to
rows.each(function(i, tr) {
    table.append(tr);
});

seems to fix it. The issue came up when using ratings in a column

@kylefox just released an update that includes fixes as well as custom sorting: https://github.com/kylefox/jquery-tablesort/pull/34, hope this helps someone!
For example, using the alpha sort defined here: http://www.davekoelle.com/alphanum.html is now possible in the tablesort library within Semantic UI:
$("table").tablesort({
        compare: function(a, b) {
            function chunkify(t) {
                var tz = new Array();
                var x = 0, y = -1, n = 0, i, j;

                while (i = (j = t.charAt(x++)).charCodeAt(0)) {
                  var m = (i == 46 || (i >=48 && i <= 57));
                  if (m !== n) {
                    tz[++y] = "";
                    n = m;
                  }
                  tz[y] += j;
                }
                return tz;
              }

              var aa = chunkify(a.toLowerCase());
              var bb = chunkify(b.toLowerCase());

              for (x = 0; aa[x] && bb[x]; x++) {
                if (aa[x] !== bb[x]) {
                  var c = Number(aa[x]), d = Number(bb[x]);
                  if (c == aa[x] && d == bb[x]) {
                    return c - d;
                  } else return (aa[x] > bb[x]) ? 1 : -1;
                }
              }
              return aa.length - bb.length;
        }
    });


need to be able to run scripts in non-rowan stones

may be limited by -S topaz option with regards to which stones this can be done with, however, for gemstone versions that support -S option, we should be able to run scripts (like test scripts) in the stone itself ...
the non-rowan bit just means that we need to be able to use .gs files to bootstrap superDoit into a stone ...

being addressed by Issue #10


Excluded participants throwing error and unable to select

Describe the bug
When editing the participants of an experiment (on experiment creation or editing an existing experiment), the user cannot exclude participants. Clicking the drop down does not provide any options and the console has a Error: NG0901. The delete icon is not an icon but the text "delete icon".

Locally, the console error is ERROR Error: Cannot find a differ supporting object 'Individual' of type 'string'. NgFor only supports binding to Iterables such as Arrays.
To Reproduce
Steps to reproduce the behavior:

Go to the participants step
Select "Include Specific"
Add a member to the Excluded list
See error

You can also click on the row to return the delete icon to a trashcan, but the dropdown is still unusable.

Expected behavior
The excluded participants section behaves like the included participants section

i think i see issue that caused this and will assign to myself

Looks good in dev! This change is ready for staging.


Merge develop to master

Codecov Report

Merging #179 into master will increase coverage by 15.39%.
The diff coverage is 74.79%.

@@             Coverage Diff             @@
##           master     #179       +/-   ##
===========================================
+ Coverage   59.04%   74.44%   +15.39%
===========================================
  Files           4        9        +5
  Lines         315      759      +444
===========================================
+ Hits          186      565      +379
- Misses         89      125       +36
- Partials       40       69       +29

Impacted Files
Coverage Δ

internal/settings/settings.go
100% <100%> (ø)

pkg/dunner/list_tasks.go
100% <100%> (ø)

internal/util/util.go
60% <56.81%> (-40%)
:arrow_down:

internal/logger/logger.go
72.22% <58.33%> (ø)

pkg/docker/docker.go
65.89% <68.46%> (+12.79%)
:arrow_up:

pkg/dunner/dunner.go
75.33% <81.72%> (ø)

pkg/config/config.go
80.68% <83.17%> (+20.04%)
:arrow_up:

pkg/initialize/initialize.go
81.39% <83.78%> (+9.96%)
:arrow_up:

pkg/initialize/recipe.go
86.66% <86.66%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


RFD - Create a Nebari blog

Status
Accepted ✅

Author(s)
@pavithraes

Date Created
13-Sep-2023

Date Last updated
13-Sep-2023

Decision deadline
19-Sep-2023

Create a blog space for Nebari
Summary

Create a new Nebari blog to have a space for community articles, something we don't currently have.
It'll be open to contributions from all community members, and the scope will be everything related to Nebari usage & development.
User benefit

It'll give us a space for communication that doesn't fit in the docs, like introducing and discussing the "why" behind certain features/changes
It'll reinforce the community-first direction of Nebari, because all project-communications will happen in the Nebari domain (previously, Nebari communication was on the Quansight & Labs blogs)
It'll hopefully also attract more contributors :)

Design Proposal

We can use the Blog functionality that Docusaurus provides: https://docusaurus.io/docs/2.2.0/blog
Alternatives or approaches considered (if any)

We can use GitHub discussions, but they're not as searchable as our docs website.

Best practices

N/A
User impact

Since this will be on the documentation side, we'll have a PR that adds the blog section.
The first blog post can be about the new extension mechanism
Unresolved questions

N/A

@nebari-dev/core-team - This is open for comments and votes

Hi @pavithraes do we have a follow up issue for this implementation?

@viniciusdc - Thanks for following up. This needs to be implemented alongside the first published blog post. We might publish something over the next few weeks, will add an update here when we do.

nice, thanks @pavithraes. Let me know if I can help with anything.


⚠ ClamAV has degraded performance

In fa26098, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 71 ms

Resolved: ClamAV performance has improved in 4322580.


Can't "yarn" all dependencies

VSCode Version: commit cdd8256
OS Version: W10 10.0.14393 (LTSB)

I tried to build VSCode from source. I already had everything except yarn, which I installed. If I type Yarn in an elevated cmd in the vscode repo, I get the following error:

It says my MSBuild is not found... I tried installing it with the Windows Build Tools but with no luck. Then I tried to install the Tools with the VS Installer, here's my config from there:

Is there anything I need to change about my setup? Like a path variable? (Python is all set and fine) I think I could also use some explanation about this step:

It's not very clear what to do, and as I said, the Windows Build Tools did not work.
************* Edit:
I just tried again at another computer, here it worked. the only difference I could see was the msvs_version was "auto" instead of "2015"

Can you send me the full output of the error, not just that narrow screenshot?

@joaomoreno how can I change msvs_version?

https://www.google.com/search?q=msvs_version&rlz=1C5CHFA_enIE767IE767&oq=msvs_version&aqs=chrome.0.0l6.1181j0j7&sourceid=chrome&ie=UTF-8

@joaomoreno yarn does not respect --msvs_version=auto

I'm not sure how I can help, I can't really see what the issue is from the screenshot. Try to find out what the difference between the two machines really is.


Cannot collapse a folder

Attempting to collapse a folder in the nav tree on the left causes an error in the console:

I think this is because react-bluekit requires immutable>=3 while it appears the keyOf method was added for Lists in v3.8.1. (I'm seeing this error with immutable version 3.7.6)

Resolved. Thanks for the report.


Testing

Solution

Upgrade to 3.79

PR Status

[ ] Code finished.
[ ] Tested with Package_check.
[ ] Fix or enhancement tested.
[ ] Upgrade from last version tested.
[ ] Can be reviewed and tested.

Package_check results

Successfully upgraded from 3.45 to 3.79.
Thank you for your work!


[Snyk] Upgrade fs from 0.0.1-security to 0.0.2

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade fs from 0.0.1-security to 0.0.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 10 years ago.

Release notes

Package name: fs

0.0.2 - 2014-09-12

0.0.1-security - 2016-08-23

    from fs GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: Add nix flake build

Adds a nix flake build and does some opinionated structure changes (moves lean files to src).

Look good to me, except for what I've commented out.
Also the doc-gen ci fails. Can you figure out what's happening? I may not have enough time to fix the code this week.

I will wait for https://github.com/leanprover/lean4/pull/1121 to get merged.


BC break in ArrayHelper on 2.0.36

What steps will reproduce the problem?
$list = self::find()
    ->select(['lng', 'name' => 'name_' . Yii::$app->language])
    ->orderBy('sort, name')
    ->all();
$test = ArrayHelper::map($list, 'lng', 'name');
var_dump($test);

What is the expected result?
2.0.35 result
array(9) {
  ["Icao"]=>
  string(27) "ICAO/NATO phonetic alphabet"
  ["Lapd"]=>
  string(19) "LAPD radio alphabet"
  ["NlBe"]=>
  string(15) "Dutch (Belgium)"
  ["NlNl"]=>
  string(23) "Dutch (The Netherlands)"
  ["Fr"]=>
  string(6) "French"
  ["De"]=>
  string(6) "German"
  ["It"]=>
  string(7) "Italian"
  ["Es"]=>
  string(7) "Spanish"
  ["Useless"]=>
  string(49) "The Non-Phonetic Alphabet (Use at your own risk!)"
}

What do you get instead?
2.0.36-dev result
array(9) {
  ["Icao"]=>
  NULL
  ["Lapd"]=>
  NULL
  ["NlBe"]=>
  NULL
  ["NlNl"]=>
  NULL
  ["Fr"]=>
  NULL
  ["De"]=>
  NULL
  ["It"]=>
  NULL
  ["Es"]=>
  NULL
  ["Useless"]=>
  NULL
}

Additional info

Q
A

Yii version
2.0.36-dev

PHP version
7.4.6

Operating system
Debian 9

What results does the query return?

 The correct return is never reached.
https://github.com/yiisoft/yii2/blob/35fb9c624893855317e5fe52e6a21f6518a9a31c/framework/helpers/BaseArrayHelper.php#L214

I think it could be this line https://github.com/yiisoft/yii2/blob/35fb9c624893855317e5fe52e6a21f6518a9a31c/framework/helpers/BaseArrayHelper.php#L209
If the key doesn't exist it shouldn't return $default and should continue.

Is it possible to put together a unit test that fails with 2.0.36 but does not with 2.0.35?

I can't test it right now, but this might do the trick:
    public function testGetValueArrayObjects()
    {
        $data = [
			new \ArrayObject(['foo1' => 'bar1']),
			new \ArrayObject(['foo2' => 'bar2']),
		];
        $this->assertEquals('bar2', ArrayHelper::getValue($data, 'foo2'));
    }

First one fails. Second one pass.

Seems the first one fails with 2.0.35 as well.

It fails for any version from 2.0.0 to 2.0.36 so what's in the tests is not a regression.

public function testMapObjects()
{
    $data = [
        new DynamicModel(['name' => 'bar1', 'value' => 1]),
        new DynamicModel(['name' => 'bar2', 'value' => 2]),
    ];

    $expected = [
        'bar1' => 1,
        'bar2' => 2,
    ];

    $this->assertEquals($expected, ArrayHelper::map($data, 'name', 'value'));
}

Works as expected.

Right. I will come back with a correct test this evening then :)

The only difference I can find right now is that in my case name is not actually a column in my database.table, but declared as public $name; in the class. This causes it to be not listed under _attributes in the object.
If that did not help I will take a look this evening.

This works for me on .35
$data = [
  new class(['value' => 1]) extends \yii\base\DynamicModel {
    public $name = 'bar1';
  },
  new class(['value' => 2]) extends \yii\base\DynamicModel {
    public $name = 'bar2';
  },
];

I just did some quick local code and removing the return $default from this line https://github.com/yiisoft/yii2/blob/35fb9c624893855317e5fe52e6a21f6518a9a31c/framework/helpers/BaseArrayHelper.php#L209 made it work on .36
I think lines 199-201 & 208-210 could be combined too.
if (static::isArrayAccess($array) && static::keyExists($key, $array)) {
  return $array[$key];
}

Thank you @alex-code, that is most helpful :)
	public function testMapObjects()
	{
            $data = [
                new class(['foo' => 'val1', 'bar' => 1]) extends \yii\base\DynamicModel {
                    public $foo = 'value1';
                },
                new class(['foo' => 'val2', 'bar' => 2]) extends \yii\base\DynamicModel {},
            ];

	    $expected = [
		'val1' => 1,
		'val2' => 2,
	    ];

	    $this->assertEquals($expected, ArrayHelper::map($data, 'foo', 'bar'));
	}

1) yiiunit\framework\helpers\ArrayHelperTest::testMapObjects
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    'val1' => 1
     'val2' => 2
+    'value1' => 1
 )

@Thoulah it fails the same way in 2.0.0. There was no regression about it.
So... is there something that worked in 2.0.35 and doesn't work in master?

I seem to be unable to make this into a phpUnit test, but the following code will reproduce the difference in results.
TestController.php:
<?php

namespace app\controllers;

class TestController extends \yii\web\Controller
{
    public function actionTest()
    {
        $model = new \app\models\Test();
        $data = $model->getList();
        var_dump($data);exit;
    }
}

Test.php:
<?php

namespace app\models;

use yii\helpers\ArrayHelper;

class Test extends \yii\db\ActiveRecord
{
    public $key;

    public static function tableName()
    {
        return '{{%test}}';
    }

    public static function getList(): array
    {
        $list = self::find()
            ->select(['key' => 'name', 'value'])
            ->all();
        $data = ArrayHelper::map($list, 'key', 'value');
        return $data;
    }
}

Mysql dump:
CREATE TABLE `tbl_test` (
  `id` int(11) NOT NULL,
  `name` varchar(5) NOT NULL,
  `value` varchar(15) NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=utf8mb4;

INSERT INTO `tbl_test` (`id`, `name`, `value`) VALUES
(1, 'name1', 'Yii2 test'),
(2, 'name2', 'Another value'),
(3, 'name3', 'Last one');

And finally the results of this code:
2.0.35: array(3) { ["name1"]=> string(9) "Yii2 test" ["name2"]=> string(13) "Another value" ["name3"]=> string(8) "Last one" }
2.0.36-dev: array(1) { [""]=> string(8) "Last one" }

Ignore those commits, thought I had it but we get other issues.
The problem is if an object implements ArrayAccess you can't use ArrayHelper::getValue to access public properties anymore and still have $default return on non existent keys.

Is that related to my issue on the forum?

That's different, if you're trying to access a property that exists (as part of the object) that'll be used before deferring to magic methods.
class Test
{
  public $foo = 'bar';

  public function __get($name)
  {
    if ($name === 'foo') {
      return "I'm also bar";
    }
  }
}

(new Test)->foo; // is 'bar'

mkay, so, issue is that since you want to get property off of ActiveRecord instance, and ActiveRecord instance implements \ArrayAccess interface(meaning I can ask stuff with \ArrayAccess::offsetGet()) that means pretty much, that public properties and getters cannot be accessed using ArrayHelper::getValue()
this Is a regression yes..

hmm, it is actually more interesting, as it should not matter, as ActiveRecord::offsetGet() is https://github.com/yiisoft/yii2/blob/master/framework/base/Model.php#L1024

@samdark maybe we should fix BaseActiveRecord::__isset() to check for public properties as well, but that might be Backwards breaking for some folks out there

So BaseActiveRecord::__isset() sure does not work correctly

I don't think this issue is specific to ActiveRecord
Quick dirty example
$data = new class(['value' => 123]) implements \ArrayAccess {
  private $attributes = [];
  public $name = 'bar1';

  public function __construct($attributes)
  {
    $this->attributes = $attributes;
  }

  public function offsetExists($offset) { return isset($this->attributes[$offset]); }
  public function offsetGet($offset) { return $this->attributes[$offset]; }
  public function offsetSet($offset, $value) {}
  public function offsetUnset($offset) {}
};

var_dump(ArrayHelper::getValue($data, 'value')); // dumps as `123`
var_dump(ArrayHelper::getValue($data, 'name')); // dumps as `NULL`

If a class implements ArrayAccess you can't use ArrayHelper::getValue to get object properties.

I don't think this issue is specific to ActiveRecord
Quick dirty example
$data = new class(['value' => 123]) implements \ArrayAccess {
  private $attributes = [];
  public $name = 'bar1';

  public function __construct($attributes)
  {
    $this->attributes = $attributes;
  }

  public function offsetExists($offset) { return isset($this->attributes[$offset]); }
  public function offsetGet($offset) { return $this->attributes[$offset]; }
  public function offsetSet($offset, $value) {}
  public function offsetUnset($offset) {}
};

var_dump(ArrayHelper::getValue($data, 'value')); // dumps as `123`
var_dump(ArrayHelper::getValue($data, 'name')); // dumps as `NULL`

If a class implements ArrayAccess you can't use ArrayHelper::getValue to get object properties.

well, yes but if this PR wasn't merged you could not get ArrayAccess offset values

@Thoulah is your issue https://github.com/yiisoft/yii2/issues/18086#issuecomment-649557239?

These smart individuals came to that conclusion based on my report and code. I have never even looked at ArrayAccess to say anything useful in this matter.

I could fix it with checking first with offsetExists

@mikk150 let's do it but I'd like to see a test that works on 2.0.35 but fails on 2.0.36 first.

With my example here, https://github.com/yiisoft/yii2/issues/18086#issuecomment-649557239

_
.35
.36

1st var_dump
Causes an exception, treated as a non existent object property
Result as expected

2nd var_dump
Result as expected
Result is always NULL (or whatever $default is) even though property exists

Added a test. Looking for a way to fix it.

@alex-code, @Thoulah, @mikk150 implemented a fix. Need someone to check me. Especially on better ways achieving it.

Is it prefered to return $default from a catch?
The static function isArrayAccess is now unused.

Is it prefered to return $default from a catch?

In this case yes because we have no idea what's missing, an object property or ArrayAccess-accessible data. Since it's ArrayAccess object we assume that's data so it behaves according to this assumption.

btw., was your original issue fixed with it?

btw., was your original issue fixed with it?

I will let you know soon, I am in the middle of a server migration (Plesk now support Debian 10). It won't take very long

Looking good from my end!

@alex-code, @Thoulah, @mikk150 implemented a fix. Need someone to check me. Especially on better ways achieving it.

Looks OK to me, I can't think of another way to do it either.

Thank you all for your help in resolving this issue :+1:

After update to 2.0.36 my script crashed with memory overflow.
You can replace this:
if (static::keyExists($key, $array)) {
to
if (is_array($array) && static::keyExists($key, $array)) {
My object with getter-property - catchs this section, instead of getting into the section:
if (is_object($array)) {

How to reproduce it?

How to reproduce it?

wrote to email

Hi i found a bug in
public static function filter ($ array, $ filters)
yii2 / framework / helpers / BaseArrayHelper.php file
line 947 if ($ filter [0] === '!') {https://github.com/yiisoft/yii2/blob/763a2b04eae79ce3cb4cc448ad179605c52bdec2/framework/helpers/BaseArrayHelper.php#L947}
if $ filters is one dimensional array i see
PHP Notice - yii \ base \ ErrorException
Trying to access array offset on value of type int
1.in /var/www/test_dev/vendor/yiisoft/yii2/helpers/BaseArrayHelper.phpat line 947
 * @return array Filtered array
 * @since 2.0.9
 * /
public static function filter ($ array, $ filters)
{
    $ result = [];
    $ excludeFilters = [];

    foreach ($ filters as $ filter) {
        if ($ filter [0] === '!') {
            $ excludeFilters [] = substr ($ filter, 1);
            continue;
        }

        $ nodeValue = $ array; // set $ array as root node
        $ keys = explode ('.', $ filter);
        foreach ($ keys as $ key) {
            if (! array_key_exists ($ key, $ nodeValue)) {
                continue 2; // Jump to next filter

In older versions it worked like this
$ array:
Array
(
    [0] => #
    [1] => Action Column
    [2] => User
    [3] => Subject
    [4] => Is Active
    [5] => Created At
    [6] => Updated At
)

$ filters:
Array
(
    [0] => 0
    [1] => 1
    [2] => 3
    [3] => 2
    [4] => 6
    [5] => 5
)

$ result:
Array
(
    [0] => #
    [1] => Action Column
    [3] => Subject
    [2] => User
    [6] => Updated At
    [5] => Created At
    [4] => Is Active
)

It was possible to use ArrayHelper :: filter to display the array in the desired order.
I fixed it like this
- 947 if ($ filter [0] === '!') {
+ 947 if (is_array ($ filter) && $ filter [0] === '!') {

Maybe it should be included this in the repository?

@rikcage please create a separate issue. Thanks. It's easy to lose track of comment in a closed issue.


[small issue] dead links in README

This link from the README is dead (404):

this one as well: