logs: add the buffer logger to inspect logs during testing

When a test runs and the function under tests writes messages in the logs, there is no reliable way to inspect them. By adding the buffer logger at the beginning of a test and removing it at the end, all log messages written during the test are collected. It is then possible to obtain the buffer and inspect it to verify it contains the expected strings.
Although it is not a good practice to rely on log strings to assert the desired side effect of a function being tested, there are a number of functions in the Gitea code that have that behavior. In order for them to be refactored and improved to be more explicit in how they expose their side effects, the first step is to write a test for their current behavior without modifying the code. The second step is to refactor the function and run the same tests to ensure the behavior of the function was not unexpectedly modified during the refactor.
An example of such function is the extraction of the git SHA / Refs during migration.

forgefriends source

Because of the matter explained in this forum post, this PR is withdrawn.

Resumed as explained in this forum post, this PR is withdrawn.

Obsoleted by https://github.com/go-gitea/gitea/pull/18752 closing.


Undo five commits

For details, please see the commentary on each commit.
I do not planning on squashing this if it gets merged.

see also https://www.openssl.org/blog/blog/2017/06/17/code-removal/

I think pushing NULL on the error data is really a programming error, and I wanted to be able to find them not ignore them.
This fix was easy, my first attempt to run the tests had a crash in strlen where “a” was NULL.

Hmmm, and you didn't try to fix the tests that are giving it NULL parameters?
On the other hand, you're right, this is pretty much the same as printf does when given NULL parameters.

Fixing them would be a better fix.  But this PR is just about removing certain commits.

A nit, but did you consider "revert" vs. "undo" in the commit messages?

Yes we did.  Revert has a specific/common meaning in git, and this was not that.

FYI, config with enable-asan and enable-msan, and running all tests (including fuzz) came out clean.

@dot-asm, can you explain the travis failures?  Not what I expected.

@mattcaswell mentioned this a couple of days ago.  The working assumption so far is that there's a bug in clang 3.9, or perhaps locally at Travis'.

thank you, done.


Rikudo830 sony apps

Description
New Sony apps

Type of change
:x: Bug fix (non-breaking change which fixes an issue)
:white_check_mark: Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:x: General change (non-breaking change that doesn't fit the above categories, such as copyediting)
Icons added
Music(com.sonyericsson.music) - added from icon request form
Photo Pro (com.sonymobile.photopro`)
Cinema Pro (com.sonymobile.cinemapro)

Can you add mocopi?


Accessing extensions from tasks

It seems like there's no mechanism to access extensions from tasks. I have come across the need in this scenario: I may want to have a build task that does something (in my case add revision info to the code) before the extension does its thing (arduino:verify, cordova:build, etc)
Now, in the case of Cordova I can make a task that does get the revision info, and chain to another that calls the command line "by hand", circumventing the extension's API. While that's okay (but confusing) for Cordova, it isn't always trivial (as in the Arduino case).
It feels like it would be a valid use case to be able to call the command palette from the task.
As a secondary wish it would also help be to able to either hide the extension's entry in the command palette OR be able to 'hook' to it in order to avoid ambiguity. It would enable us to setup VSCode for people so they can focus on solving problems and forget about the steps and toolset involved :)
thanks!

VSCode Version:
1.18.0
OS Version:
MacOS 10.12.6

Reproduces without extensions: No

I think this would be a useful feature and would avoid having to replicate the extension's functionality through external commands.
For example, in the case of arduino.verify, it already does what is needed but if you want to run the verify command, you'd have to create a bat/bash script that runs the arduino tools from command line. That feels like a waste of time and prone to error when being able to call extension commands from a task would be a better course of action.

I added this last month 😊 Added as part of #4758


Setting disk size to 80gb

Summary | Résumé
Setting disk size of EKS nodes to 80gb to remove disk pressure issues. To ensure no downtime, manual steps must be taken in production:

Manually resize disks in AWS Console:
a. Navigate to node in AWS
b. Select the volume
c. Choose "Modify Volume"
d. Change size from 20gb to 80gb
e. Save
kubectl cordon the node
kubectl drain the node
Restart the node in AWS Console
Uncordon the node when it is in ready, scheduling disabled state
Repeat for all nodes
Terragrunt refresh-only to update terragrunt

Closing this PR for now since TFState will not recognize the disk size changes. When we migrate to new nodes, we can set this then.


Ensures that SecondaryButtonOrientation is set on starting up in CompactInline DisplayMode.

Problem:
If an app starts in CompactInline DisplayMode, it fails to update SecondaryButtonOrientation because the SecondaryButtonContainer StackPanel is not yet loaded and it [gracefully] misses out on first chance UpdateSecondaryButtonOrientation() method within SplitView_IsPaneOpenChanged event handler.
Solution:
Easy. Just put the missed out UpdateSecondaryButtonOrientation() method call in the SecondaryButtonContainer StackPanel Loaded event handler to catch up and all will be fine. Note that the problem would be self-correcting for other display modes, because menu pane opening provides the needed trigger for [a second-time chance] UpdateSecondaryButtonOrientation() method call (the closed pane with other modes just concealed away the problem that CompactInline revealed -- interesting how bugs hide themselves).
Gotchas!

It's your responsibility to collapse menu items under HamburgerMenu.SecondaryButtons to MaxWidth=48 so that the icons line up horizontally, else some of them disappear sideways since the label strings also take up space. (This can be done easily by binding the MaxWidth of all secondary menu items to a single Property).

Furthermore, make sure that the secondary icons can fit into the HamburgerMenu.PaneWidth which means the max number of icons = Int(HamburgerMenu.PaneWidth/48).

It's also your responsibility to provide a spacer so that the most-right (or most-left for rtl system) icon aligns nicely with the margin; this is for aesthetics but not something to be ignored lightly. The Dynamic Themes sample project shows how the spacing and alignment is done.

Oops! Somehow overlooked a second file. An updated PR will be replacing this.


WiFi connection on Mesh WiFi system

Hi everyone, I am experiencing inconvenient behavior where the device that is connected with a high signal, for some reason, reset the wifi connection and it switches on a station that has a very low signal. I do not understand why it prefers to connect to a station of the mesh with a low signal, I think the reason is that there is no order to which station connects.
I have no time or experience in modifying the code, but hypothetically speaking using a library like this https://randomnerdtutorials.com/esp32-wifimulti/ could the problem be solved?
Thanks in advance for your time.

I experience this too. The esp32 seems to randomly select the AP.
In my case both AP broadcast the same SSID so wifimulti would not help.
However, it seems that the problem is identified and is being fixed in esp-idf: https://github.com/espressif/esp-idf/issues/8269
(BTW it seems it's more an authentication failure (it can happen) on the best AP that make it end up on the worst).
Hopefully, it will solve this issue soon.
Nothing related to ESPAltherma but I'll keep an eye on this to see if some code change is needed.

I just pushed a commit that should fix this... 🤞


Relative Angles function in Ensembler Removed

In the update log to ensembler on Nov 30th 2015, the relative phase angles function was removed.
Is it possible to have it re-integrated?

Access to relative phase has been disabled as the code is currently being revised for accuracy and better functionality.


Update Sound.js

Fixed the removeEffect bug, that caused a crash if the effect removed was not the first in stack.

Wow good catch. Thanks for this.
It would be good to add a test case in tests > Sound.test.js, perhaps under the describe('effects', ... suite.
Would you be interested in adding it? Else I could add it after merging your commit.

Related to #51

Hi,
I could probably add it, but I cannot do it now. I still have to build a
lot stuff upon this before my speech on Web Audio Api and I only have today
to do it all. =) So if you have time, just add the test, otherwise it will
have to wait for a while  so I can properly concetrate on it.
Do you mind if I promote your library in the speech? It is only for small
group of JS enthusiast in my area (Tampere, Finland).
Best Regards,
Henrik Collin
On 25 March 2017 at 13:23, Alejandro Mantecon Guillen <
<EMAIL_ADDRESS>wrote:

Wow good catch. Thanks for this.
It would be good to add a test case in tests > Sound.test.js, perhaps
under the describe('effects', ... suite.
Would you be interested in adding it? Else I could add it after merging
your commit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alemangui/pizzicato/pull/52#issuecomment-289205862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATtIrqt35UjV9rN2c6QFqd44U1mQeaaGks5rpPkUgaJpZM4MpCi-
.

OK, I will merge your fix and make a bug-fix release.
Sure no problem! Feel free to talk about the library, and good luck in your presentation :)


Create a system with methionine rich protein and methanol

Create a system with:

a methionine-rich protein: 2MAT
water (TIP3)
ions (Na+, Cl-, K+, Mg2+, OH-)
methanol, ethanol and urea (why not)

I can add 2MAT + methanol, ethanol and urea with PACKMOL. But I can't add ions and water with it

Done ! And it work, Grodecoder find the correct composition :tada:


MSID: 98081 Version: 1 DOI: 10.1101/2<IP_ADDRESS>1131

MSID: 98081
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>1131
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=98081
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1131

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/98081v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>1131
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/98081/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S1. Clinical information and genetic analysis of CTRL and WS donors.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S2. Characterization of primary cells and the generated iPSCs.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S3. Bulk RNA sequencing (RNA-seq) reveals the altered expression of neurodevelopmental genes.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S4. scRNA-seq reveals the abnormal neuronal differentiation in WS forebrain organoids.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S5. The level of TTR decreases in WS organoids.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S6. Recombinant TTR rescues neurodevelopmental defects in WS organoids.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S7. Gtf2ird1 deficiency induces the abnormal proliferation and differentiation of embryonic neuronal progenitor cells in vitro.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'Figure S8. Schematic diagram illustrating the roles of GTF2IRD1-TTR-ERK pathway in regulating neurogenesis and neurodevelopment.'. Is this correct?

Removed the above section.


Add operator

Seeing "operator" over and over again caused me to have to search for it's meaning.  And I'd like to see it on the terminology page.  Feel free to add/refactor definition which was obtained here.

Added in https://github.com/cloudfoundry/docs-bosh/commit/e840a1f5d6ae7a3ea942ab1ba8f14f6a5da659f7.
Not sure why CLA thing is popping up for you...


Correction for raw affiliation Professor, Department of Medicine, Schulich School of Medicine and Dentistry, Western University, Medical Sciences Building, Suit 102A, London

Correction needed for raw affiliation Professor, Department of Medicine, Schulich School of Medicine and Dentistry, Western University, Medical Sciences Building, Suit 102A, London
raw_affiliation_name: Professor, Department of Medicine, Schulich School of Medicine and Dentistry, Western University, Medical Sciences Building, Suit 102A, London
new_rors: 02grkyz14
previous_rors: 05cgtjz78
works_examples: W4379535445
contact: f0e2d2221048c9297015018cb696c0da:d7fb87ebac0fdc40 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Debug/check if cherry pick works

Can be deleted after tests pass, just to make sure everything is correct before merging it to master

Codecov Report

Merging #2701 into master will decrease coverage by 6.34%.
The diff coverage is 14.38%.

@@            Coverage Diff            @@
##           master   #2701      +/-   ##
=========================================
- Coverage   14.36%   8.01%   -6.35%
=========================================
  Files         257     259       +2
  Lines        9455   10480    +1025
  Branches     1092    1332     +240
=========================================
- Hits         1358     840     -518
- Misses       7970    9585    +1615
+ Partials      127      55      -72

Impacted Files
Coverage Δ

src/util/alias/get-deployment-from-alias.ts
0% <ø> (ø)
:arrow_up:

src/util/dev/builder.ts
12.23% <ø> (-60.91%)
:arrow_down:

src/util/domains/move-out-domain.ts
0% <ø> (ø)
:arrow_up:

src/util/certs/finish-cert-order.ts
0% <ø> (ø)
:arrow_up:

src/util/dns/delete-dns-record-by-id.ts
0% <ø> (ø)
:arrow_up:

src/util/dev/static-builder.ts
31.25% <ø> (-62.5%)
:arrow_down:

src/util/input/text.ts
0% <ø> (ø)
:arrow_up:

src/util/alias/get-domain-aliases.ts
0% <ø> (ø)
:arrow_up:

src/util/output/create-output.ts
27.08% <ø> (-10.76%)
:arrow_down:

src/util/deploy/get-deployments-by-appname.ts
0% <ø> (ø)
:arrow_up:

... and 147 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 f6552c4...8da0534. Read the comment docs.


Race condition between notification and account fetch.

In rare cases after HelixAccountService receives notification for account update, it still cannot fetch the updated account from remote. This makes the HelixAccountService to possibly miss update and not keep in sync with the actual account data.

Fixed by #742


Update Sidekiq 3 to 4

After update I see in log infinite [Sidetiq] Sidetiq::Actor::Handler id: 6906860 initialize :

Sidetiq 0.7.0, ice_cube ∽> 0.13.2, sidekiq 4.1.1

Please help

Guys, my log grows to hundreds of megabytes per day.

solved: https://github.com/tobiassvn/sidetiq/pull/154/commits


Add developer console

An overlay containing two main components: display pane (90% top) and input pane (10% bot). Potentially with autocomplete features, see one of FX util projects

WIP in Console.kt

Basic functionality complete


turn_costs handling in /isochrone

Recently we fixed the handling of turn_costs for the /isochrone endpoint on the 1.0-pre32-fixes, but these changes also need to be ported to current master.
The other thing we are missing is the turn_costs<->edge_based conversion we are already doing for /route (at least for GET requests, see #1966). The same holds for /spt as well!

We should probably do this parameter handling in ProfileResolver, then there should no longer be need to do this in every Resource.

So far we only support node-based for isochrones. So we either have to make sure there is a profile without turn costs or we make sure the isochrone algorithm (which is node-based) ignores turn costs or we use the __disable_turn_costs_for_lm_preparation in GraphHopper#createWeighting (and probably rename back it to __disable_turn_costs). See also #1965.

Assuming isochrone cannot be run edge-based (as is currently the case), whats better (or less ugly):

it does not matter whether or not a profile has turn costs and the turn costs are ignored for /isochrone even when turn_costs: true is set in the profile

/isochrone does not allow profiles with turn_costs: true, because its not implemented. So to make use of /isochrone we always have to add a profile with turn_costs: false and potentially we have to add a copy with turn_costs: false for every profile we want to use to make it also work for /isochrone.


typo -- "toekn"

In 2 places within the ReadMe the word 'token' is misspelled.

I submit a pull request  to fix this. #11

Fixed #11


Spotlightlight search results icon

The icon does not display correctly in spotlight results

This is due to electron-icon-maker not adding the correct sizes into the icns file. It should contain only 16x16, 32x32, 128x128, 256x256, 512x512 and 1024x1024 pixels
You could use https://iconverticons.com/ in the mean time, or icnsutils

Too bad! I have the same problem. I liked electron-icon-maker...

same problem

I tried to change this package's source to only include the sizes mentioned by @Kilian but it still came out corrupted. Not sure why.
For anyone struggling with this, here's how I managed to generate my icons properly. I used the iconutil tool which should be preinstalled on your mac.

Make a directory with the .iconset extension and put your pngs in it. They should be named like so: icon_16x16.png
run iconutil -c icns yourdirectory.iconset
It should create yourdirectory.icns

If you're not on mac, I think I saw iconutil on NPM. Not sure if it's the same tool exactly but worth a try.

This should be fixed in the latest patch v0.0.5. Let me know if you still have issues with it. Sorry it took so long.


All logs appear as INFO in stackdriver

Hi,
I've used the tips in springboot-appengine-standard. I had log statements for various levels, but everything comes across as INFO in stackdriver. In other projects where we have used java.util logging directly we managed to see the log level icon correctly in stackdriver, filter by level, etc.
As another note I can customise the log pattern to some extend in spring boot's standard property logging.pattern.console in springs application properties/yaml however I cannot get rid of the prefix that appears on the line: [f~my-app-id/1.407102846225810748].<stdout>:  that is part of the message. Once again on non spring apps using java util logger I did not see this. I tried various customisations of the util logging logging.properties too (e.g. https://github.com/GoogleCloudPlatform/appengine-pipelines/blob/master/java/example/war/WEB-INF/logging.properties) but these did not remove it. This pattern thing is not a massive deal, but would be nice to know how to remove it.

Using Logback is the preferred log formatter for Spring Boot. Checkout their Logging docs for more configuration info. Though focused on Cloud Run, Java Logging on Cloud Run with Stackdriver can provide more background.


Group the price related contribution items in a sub-menu

Problem

In My Contributions, Group the price related item in a Prices sub-menu (gala pointed out that there were too many items)

@monsieurtanuki @raphael0202 @raphodn @g123k I created a mockup above.
Having this section means the News system can potentially link to it instead of an external page.
This in-app page could have the deeplink prices.openfoodfacts.org, and become the "landing page" of Open Prices within the app.


[Feature Request] Enable Hyperlink for "About Product" section and

At the moment, added information about the product in its "About Product" section simply shows up as text.
Sample:

It would be better if this link was enabled into a hyperlink by default for easier user interaction

@digracesion Thanks for your feedback, we gonna work on that next time.


Fix typo in CSS

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

Fixes a small typo in the CSS code.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.006%) to 68.45% when pulling 3f9ebb588e5cf83c29fb4651be59461702fee518 on bonanitech:patch-1 into 3ace7eeafd6308543edb1135920dabf93ec60e94 on node-red:master.


Bug when intersection without pedestrian crossing

Hi,
I have a save where I have multiple crossings where the lights for the pedestrians never go green. I want them to use the pedestrian bridge. But they still seem to be queuing for it. Is this a bug?
Windows 10 64-bit, CS: 1.4
And I also have Traffic++ V2 installed
all my subs from workshop: http://steamcommunity.com/id/thibaultmol/myworkshopfiles/?browsesort=mysubscriptions&browsefilter=mysubscriptions&p=1
Link to save file: https://drive.google.com/file/d/0B0rhFYAkXDVJUHd6b01TLVdrM1k/view?usp=sharing
Thanks

Btw, the region where the problem is occurring is in the 'industrial' district at the bottom right corner.

Will be implemented in version 1.7.0!


Maximum flow

Should I merge your pull request now to work further on it?
And has it to do something with the version of the Pluto package? Because I see that line changed.
Do you mean the documentation html or the reference section in the notebook for the link to the wikipedia page?
Thanks for the suggestions and revision!

Should I merge your pull request now to work further on it?

Ik denk dat dit het eenvoudigst is. Kleine dingen heb ik zelf aangepast, grotere suggesties staan als comments.

And has it to do something with the version of the Pluto package? Because I see that line changed.
Do you mean the documentation html or the reference section in the notebook for the link to the wikipedia page?

Ik heb je notebook getest en mijn versie van Pluto is ouder. Die lijn wordt automatisch aangepast.
M

Thanks for the suggestions and revision!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/DouweDeVestele/STMOZOO/pull/1#issuecomment-751284086, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABN5PI3XIRWNUXCQYYTZSI3SWTRRTANCNFSM4VHGAU2A.


Allow enabling anonymous login

This commit would allow to enable anonymous login for the S3FileSystem.
FYI: I saw that dvc already has an option  allow_anonymous_login for azure. Should the name be aligned? A pull request to add this option to dvc/config_schema.py would then be set up accordingly.

FYI: I saw that dvc already has an option  allow_anonymous_login for azure. Should the name be aligned? A pull request to add this option to dvc/config_schema.py would then be set up accordingly.

good point, @2maz . It seems better to align them since allow_anonymous_login is not Azure specific, it is already a DVC-specific name for the option, so why not use the same across all remotes? @dberenbaum what are your thoughts on this?


[BUG]Cannot install deepspeed when cuda is installed by a non-root user

Describe the bug
I have already installed the nvcc here (without root), but I get an error when pip install deepspeed .
$  which nvcc
/mnt/nlp/gaoqiang/cuda/cuda-12.0/bin/nvcc
$ nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Fri_Jan__6_16:45:21_PST_2023
Cuda compilation tools, release 12.0, V12.0.140
Build cuda_12.0.r12.0/compiler.32267302_0

$ echo $CUDA_HOME
/mnt/nlp/gaoqiang/cuda/cuda-12.0/

$echo $PATH
/home/gaoqiang/.vscode-server/bin/903b1e9d8990623e3d7da1df3d33db3e42d80eda/bin/remote-cli:/mnt/nlp/gaoqiang/miniconda3/envs/deepseekMoE1/bin:/mnt/nlp/gaoqiang/miniconda3/condabin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/mnt/nlp/gaoqiang/cuda/cuda-12.0/bin:/mnt/nlp/gaoqiang/cuda/cuda-12.0/bin

OS
Linux AuroraStation 6.2.0-32-generic #32~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Fri Aug 18 10:40:13 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
output
pip install deepspeed
Defaulting to user installation because normal site-packages is not writeable
Collecting deepspeed
  Using cached deepspeed-0.13.3.tar.gz (1.3 MB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [16 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-46kb5itk/deepspeed_83bd7c100d41430c9a3b2422c7d61dfd/setup.py", line 100, in <module>
          cuda_major_ver, cuda_minor_ver = installed_cuda_version()
        File "/tmp/pip-install-46kb5itk/deepspeed_83bd7c100d41430c9a3b2422c7d61dfd/op_builder/builder.py", line 52, in installed_cuda_version
          output = subprocess.check_output([cuda_home + "/bin/nvcc", "-V"], universal_newlines=True)
        File "/usr/lib/python3.10/subprocess.py", line 421, in check_output
          return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
        File "/usr/lib/python3.10/subprocess.py", line 503, in run
          with Popen(*popenargs, **kwargs) as process:
        File "/usr/lib/python3.10/subprocess.py", line 971, in __init__
          self._execute_child(args, executable, preexec_fn, close_fds,
        File "/usr/lib/python3.10/subprocess.py", line 1863, in _execute_child
          raise child_exception_type(errno_num, err_msg, err_filename)
      FileNotFoundError: [Errno 2] No such file or directory: ':/mnt/nlp/gaoqiang/cuda/cuda-12.0:/mnt/nlp/gaoqiang/cuda/cuda-12.0/bin/nvcc'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

I don't understand why the paths are nested when looking for nvcc
No such file or directory: ':/mnt/nlp/gaoqiang/cuda/cuda-12.0:/mnt/nlp/gaoqiang/cuda/cuda-12.0/bin/nvcc'

Same issue!

Same issue!
And when I use  python -c "import torch.utils.cpp_extension; print(torch.utils.cpp_extension.CUDA_HOME)", it outputs "none"


Support for Tokens Restricted with Attributes

Summary
Support for restricting tokens created by a marker which require accounts to possess certain attributes before allowing the tokens to move.
Problem Definition
Currently the restricted marker system has a common use case of requiring attributes such as KYC records to exist on accounts before tokens can be sent.  This is implemented with extra steps using smart contracts to broker transfers.  While this is a workable process it prevents more complex workflows such as exchange functions from using these tokens without adding in auth grant steps.  These auth grant steps allow the checks to be bypassed which while workable is less than ideal.
Proposal
With the bank transfer authorization hooks being added to the bank module it will be possible to support new authorization flows during any transfer of tokens between accounts.  With these hooks we should be able to look up a token marker registration and determine if there are restrictions in place.  If we added support for defining a set of attributes that must be present on the destination account then these hooks could allow the constraints to be easily enforced in a universal way.  This would mean that exchange functions and other smart contracts could start accepting these tokens while the marker admin can ensure that their attribute assertions are uniformly and consistently applied.

Update the marker module with a method for defining bank transfer restriction rules
Update the bank module send authorization hooks with a method that checks the marker module to ensure that any require checks are performed before approving a transfer of tokens between accounts.

Requires: https://github.com/cosmos/cosmos-sdk/pull/14224 and associated development of the hooks that are registered.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

The ideas here overlap a bit with this Ethereum proposal: https://eips.ethereum.org/EIPS/eip-3643

More detail:

Restrictions can be added to the existing Restricted Coin type.
A repeated list of name/value elements can be added to the Marker Account proto to specify the restrictions
Values for the name should support wildcards (e.g. - *.kyc.passport.pb)
The bank hooks for restrictions on transfer can be used so that normal bank transfers can occur and deprecate the Transfer method on the marker.
Transfer access-grant can be checked as an override to the additional restrictions
The Transfer access-grant can be used for controlling the transfer process ... so by extension this role should be required for requests that add/remove/modify restrictions on a marker

One more related transfer concept that is less restriction and more control... it should be possible to specify that a transfer authority has "clawback" capability ... this is a somewhat sensitive issue in the blockchain world as it indicates that the owning account doesn't have full control over their assets and they could be removed without their signature.  However, when a token is modeling a real-world asset such as a bank deposit there can be cases where funds are transferred without the account holder's permission.  In order to model these use cases digitally a corresponding capability should be possible on chain as well.
Proposed: an additional flag on the marker for "restricted" tokens that indicates if clawback is allowed.  If set to true then the check for the source account signature on the transfer request should be skipped.  This would provide a lasting use for the Transfer method and prevent it from being deprecated as mentioned above.

See #1368 for the clawback piece.


Bump npm version to 2.1.0?

I noticed this new version wasn't published to npm yet.

Same issue for homebrew-bats-assert. It's still on version 2.0.0. The new version 2.1.0 would be very welcome as it includes new features like assert_regex/refute_regex.

So far, only the bats npm package is maintained by this org. You could post your request oer at @jasonkarns' repo.
In the long run we might might offer npm packages under this org.

?!? I don't understand. Isn't https://github.com/bats-core/homebrew-bats-core part of the same org as the one owning https://github.com/bats-core/bats-assert/?
Updating bats-assert.rb for the latest version should be as easy as updating the url and the sha256. The following patch should do the trick: patch-upgrade-to-latest-version.txt.
If you do that, then anyone using bats-assert via Homebrew can simply run "brew update; brew upgrade bats-assert" to get the latest version, which is much easier than manually installing it.
I wonder whether updating the npm package would also be as easy. The npm package is still pointing to version 2.0.0, even though version 2.1.0 was released many months ago. I'm sure npm users too would be very happy to enjoy the latest version.

@paltherr I am sorry, I did not notice you were talking about homebrew. Please open a separate issue in that repository, since the maintainer likely won't see it here. I was only talking about the NPM releases which will require some additional work to automatically generate them during releases. I just wanted to let @mdwagner know that there are currently no official NPM packages besides bats(-core).

OK, I created https://github.com/bats-core/homebrew-bats-core/pull/7.


LiteDB 5.0.10 - Insert method does not return the ID [BUG]

Hi!
I'm using LiteDB 5.0.10
I've noticed that the Insert method became a void type. So it doesn't return the ID of a new record anymore (like it used to do in 4.x). Meanwhile the method description claims that it still does return the new ID (which is of course not true because method type is void).
Can anybody do something about that? I have to say that returning the ID on insert was a very useful feature.
Thanks a lot!

@AdamVavaR Just to confirm, are you talking about the LiteRepository?


Add heuristic and exact damping to KFACInverseLinearOperator

Resolves #92.
Please review #91 first, since this PR is based on that branch.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

94 of 103 (91.26%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 89.429%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

curvlinops/inverse.py
70
74
94.59%

curvlinops/kfac.py
24
29
82.76%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
1049

Relevant Lines:
1173

💛  - Coveralls


Feature Request: Add Gogs support

@kubiko
Gogs integration is now available as separate project:
https://github.com/wekan/wekan/commit/674d45a81c94e0f862c9624b0f89743a5c675d89
It could be addition to wekan-snap like caddy is.
In general, I think that for both Docker Compose files and Snap plan is to have many additional integrations available, like Huginn, Rocket.Chat, GitLab, etc. like there is currently for wekan-mongodb and wekan-postgresql docker compose files.
For Sandstorm, integrations need to be run outside of Sandstorm using REST API, because grains are only running when someone is using them.

Related https://github.com/wekan/wekan-gogs


IteratorMatcher

In order to test Iterator implementations more easily as a developer I want an IteratorMatcher.
The problem is that an Iterator is not immutable and testing with Matchers would be subject to temporal coupling. The proposed solution is to test the result of a Single which provides an Iterator instead. Technically this it not really different from an IterableMatcher, but semantically it would emphasize that the Iterator is tested.

Here is how it would look like:
assertThat(() -> new Seq<>("a", "b"), iterates("a", "b"));
assertThat(() -> new Seq<>(), iteratesNothing());

The IteratorMatcher would take care of testing the iterator contract (i.e. hasNext is idempotent and next throws if no more elements are present).


Really poor performance with streaming parser

Parsing an HL7 file with really long lines results in unusable performance. In most of the ORU_R01 messages I deal with, we have some OBX segments with embedded PDF files (base64 encoded). This results in lines in the PDF being millions of bytes long. So the problem is that in StreamTextCursor.ParseText it does a full parse after every time it loads a new chunk because it reaches the end of the data before it reaches the end of the line.
Here is an HL7 attachment that can help illustrate the issue. It's parsed like this:
string filePath = ...;
using var stream = File.OpenRead(filePath);
StreamText text = await new StreamTextReader(stream).Text;
var parse = await parser.ParseStream(text, new TextSpan(0, text.Length));

HR7Message.txt

Can you please add your query.

The issue is actually separate from any query. The problem is in parser.ParseStream. Here's a full main function that can reproduce the issue with the file I attached.
private static async Task Main(string[] args) {
    var filePath = "HR7Message.txt";

    ISchema<HL7V26Entity> schema = Schema.Factory.CreateHL7<HL7V26Entity>(cfg => cfg.AddFromNamespaceContaining<HL7Version26>());
    IEntityParser<HL7V26Entity> parser = Parser.Factory.CreateHL7(schema);

    using var stream = File.OpenRead(filePath);
    StreamText text = await new StreamTextReader(stream).GetText();
    var parse = await parser.ParseStream(text, new TextSpan(0, text.Length)); // <-- problem is here
}

Hi @bryanmatteson
I created a branch with your message and added a performance benchmark
See this branch
I am running 2.2 Ghz I7 2016 MacBookPro and here are the performance benchmarks I got
`// * Detailed results *
StreamingParserBenchmarks.StreamingParserBenchmark: Job-KKEHVM(Runtime=Core, InvocationCount=5, LaunchCount=1, RunStrategy=Throughput, TargetCount=5, UnrollFactor=1, WarmupCount=5)
Runtime = .NET Core 2.1.13 (CoreCLR 4.6.28008.01, CoreFX 4.6.28008.01), 64bit RyuJIT; GC = Concurrent Workstation
Mean = 284.0624 ms, StdErr = 0.6272 ms (0.22%); N = 5, StdDev = 1.4024 ms
Min = 282.4000 ms, Q1 = 282.9314 ms, Median = 284.0312 ms, Q3 = 285.2089 ms, Max = 286.2371 ms
IQR = 2.2775 ms, LowerFence = 279.5152 ms, UpperFence = 288.6251 ms
ConfidenceInterval = [278.6631 ms; 289.4617 ms] (CI 99.9%), Margin = 5.3993 ms (1.90% of Mean)
Skewness = 0.4, Kurtosis = 1.56, MValue = 2
-------------------- Histogram --------------------
[281.682 ms ; 286.955 ms) | @@@@@
Total time: 00:00:26 (26.77 sec)
// * Summary *
BenchmarkDotNet=v0.10.14, OS=macOS 10.14.2 (18C54) [Darwin 18.2.0]
Intel Core i7-4770HQ CPU 2.20GHz (Haswell), 1 CPU, 8 logical and 4 physical cores
.NET Core SDK=3.1.201
[Host]     : .NET Core 2.1.13 (CoreCLR 4.6.28008.01, CoreFX 4.6.28008.01), 64bit RyuJIT
Job-KKEHVM : .NET Core 2.1.13 (CoreCLR 4.6.28008.01, CoreFX 4.6.28008.01), 64bit RyuJIT
Runtime=Core  InvocationCount=5  LaunchCount=1
RunStrategy=Throughput  TargetCount=5  UnrollFactor=1
WarmupCount=5
               Method |     Mean |    Error |   StdDev |      Min |      Max |     Gen 0 |    Gen 1 |    Gen 2 | Allocated |

------------------------- |---------:|---------:|---------:|---------:|---------:|----------:|---------:|---------:|----------:|
StreamingParserBenchmark | 284.1 ms | 5.399 ms | 1.402 ms | 282.4 ms | 286.2 ms | 1200.0000 | 800.0000 | 400.0000 |  15.46 MB |
// * Legends *
Mean      : Arithmetic mean of all measurements
Error     : Half of 99.9% confidence interval
StdDev    : Standard deviation of all measurements
Min       : Minimum
Max       : Maximum
Gen 0     : GC Generation 0 collects per 1k Operations
Gen 1     : GC Generation 1 collects per 1k Operations
Gen 2     : GC Generation 2 collects per 1k Operations
Allocated : Allocated memory per single operation (managed only, inclusive, 1KB = 1024B)
1 ms      : 1 Millisecond (0.001 sec)
// * Diagnostic Output - MemoryDiagnoser *
// ***** BenchmarkRunner: End *****
// * Artifacts cleanup *
`

Right, this is the workaround. In your benchmark, you load the entire stream into memory first, and then use a StringReader and finally a TextReaderStreamTextReader. This will work because it never needs to load another chunk from the stream as it has already been loaded completely into memory by your call to File.ReadAllText(_largeFilePath).
If you change the benchmark to use the StreamTextReader directly, like this:
[Benchmark]
public async Task StreamingParserBenchmarkLargeFileDemo() {
    Console.WriteLine("Starting streaming parser benchmark run");

    using (var stream = File.OpenRead(_largeFilePath)) {
        //Console.WriteLine("About to parse stream");
        StreamText text = await new StreamTextReader(stream).Text;
        ParseResult<HL7Entity> result = await _hl7Parser.ParseStream(text, new TextSpan(0, text.Length));
        ...

then I think you'll run into the bug I'm seeing. So while not using the StreamTextReader is a potential solution, there still exists a bug in the stream parsing logic for really long lines.


remove "show certificate info" on certificate list from pending CSR

In certificates list, "show certificate info" button for pending CSR file shows empty dialog, because certificate file is not available yet.
Therefore, it is better to remove this button if no certificate is available.

@nhirokinet thanks, good addition


Navigating via buttons is slow

Which OS ?
Android
Version
Which versions are you using:

react-native-swiper v^1.5.6
react-native v0.55.4

Expected behaviour
Pushing Next/Previous buttons should navigate to the next/previous page without delay.
Actual behaviour
There's a short delay of a second or so before scrolling after clicking the button (Button goes grey for a second and then page switches). Swiping navigates without delay.
Steps to reproduce

Have a swiper component with showsButtons={true}
Push the 'next' button
Observe short delay before page switches to the next one

https://github.com/leecade/react-native-swiper/issues/713

same problem +


Install catalystcoop.pudl

hey all - any reason this one hasn't been merged?

@choldgraf it's part of https://github.com/2i2c-org/pilot-hubs/issues/105


security/acme-client: Add Simply.com (used to be UnoEuro)

… DEV branch #3978, that fixes dns for simply, since they removed support for the old UnoEuro REST API (V1) in favour of V2 that uses Basic Auth.
The current dns_simply.sh does not work, neither does dns_unoeuro.sh in acme.sh, when the PR in acme.sh is merged it will work.
There is no danger in merging this, since it will just not work, just like the UnoEuro doesn't work. But when acme.sh merges PR #3978, it will start working.

This will be merged when the following conditions are met:

https://github.com/acmesh-official/acme.sh/pull/3978 is merged upstream
https://github.com/acmesh-official/acme.sh/pull/3978 is part of an official release of acme.sh

In the past I've merged similar PRs too soon, which caused confusion and frustration for users.

The simply v2 plugins from upstream works, except it doesn't delete the txt-records after use.

Merged, thanks!


Use importable settings

Instead of passing around universal configuration as arguments, configs should be importable as necessary.  That way tools that need access to them can use them without requiring devs to trace through the entire process flow any time the necessary configuration details change.
import pudl.settings as settings is better than the various things going on in the codebase.

stale. we need a full rethink of settings in both local and distributed context....


bug: patient context state is not initiated properly by definitions base

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When operation for PatientContext is initiated a Operation Target state container is created but
this initiation method is not handling context states correctly.
Cannot access the context state via regular api methods like
mgr.get_context_state(state)

Expected Behavior
PatientContext state is correctly created and available in the mdib.states index
Minimal Reproducible Example
No response
Solution proposal
No response
Python Version
3.9
Operating system
Windows
Sdc11073 Version
2.0.0a5
Link to sdc11073 Logs
No response
Further Information
No response
Participation

[X] I am willing to submit a pull request to fix this bug.

Already fixed in version 2.0.0a6


Minimal changes to support Windows / Visual Studio

const keyword additions and one new file in the platform/windows directory called unistd.h.

@karpathy - this should be the minimal set of changes. Thanks.

Do we not have to include it?
Do we need to change the .cu as well?

@karpathy - We'll need to include the fake unistd.h somewhere - I can change the makefile to create it "on the fly". We can add the fake one now and I can make the makefile change and delete it once those makefile changes are in?
The const keyword adds are (mostly/all?) the same as in the train_gpt2.c from a commit 2 days ago. We'll need these const in Windows because of a problem with OpenMP support. Adding these fixes it.

@azret @rosslwheeler can either of you try just this single change around gelu_backward in train_gpt2.c?
#pragma float_control( precise, on, push ) // Save current setting and turn on /fp:precise

// code that needs more precision

#pragma float_control(pop) // Restore file's default settings

Seems like it might be the only required change.

In fact, first try to wrap only the tanhf call

Sure - will report back. Thanks.

This #pragma is ignored by gcc, but breaks clang. I can add it, wrapped in yet another ifdef, if it makes /fp:fast /Ox work, without losing the performance gains in the other math calls.
But check the performance of just wrapping tanhf vs wrapping the whole gelu_backward, and make sure to compile with /fp:fast /Ox - on gcc all the optimizations fell away when trying to wrap only tanhf. Not sure why, maybe msvc is more fine grained.

Test run 1
cl.exe /Ox /fp:fast /I. /I .\dev\win train_gpt2.c
D:\Repos\llm.c>train_gpt2.exe
[GPT-2]
max_seq_len: 1024
vocab_size: 50257
num_layers: 12
num_heads: 12
channels: 768
num_parameters: 124439808
train dataset num_batches: 1192
val dataset num_batches: 128
num_activations: 73323776
val loss 5.252007
step 0: train loss 5.356174 (took 10266.000000 ms)
Error: must forward with targets before backward
Test run 2 with
#pragma float_control( precise, on, push ) // Save current setting and turn on /fp:precise
void gelu_backward(float* dinp, float* inp, float* dout, int N) {
    for (int i = 0; i < N; i++) {
        float x = inp[i];
        float cube = 0.044715f * x * x * x;
        float tanh_arg = GELU_SCALING_FACTOR * (x + cube);
        float tanh_out = tanhf(tanh_arg);
        float coshf_out = coshf(tanh_arg);
        float sech_out = 1.0f / (coshf_out * coshf_out);
        float local_grad = 0.5f * (1.0f + tanh_out) + x * 0.5f * sech_out * GELU_SCALING_FACTOR * (1.0f + 3.0f * 0.044715f * x * x);
        dinp[i] += local_grad * dout[i];
    }
}
#pragma float_control(pop) // Restore file's default settings

cl.exe /Ox /fp:fast /I. /I .\dev\win train_gpt2.c
D:\Repos\llm.c>train_gpt2.exe
[GPT-2]
max_seq_len: 1024
vocab_size: 50257
num_layers: 12
num_heads: 12
channels: 768
num_parameters: 124439808
train dataset num_batches: 1192
val dataset num_batches: 128
num_activations: 73323776
val loss 5.252007
step 0: train loss 5.356174 (took 10156.000000 ms)
step 1: train loss 4.300983 (took 9938.000000 ms)
step 2: train loss 4.623276 (took 10109.000000 ms)
step 3: train loss 4.600286 (took 10172.000000 ms)
...
Wow! 10 seconds without OpenMP 👍
Thanks @dagelf! Now convince Andrej to merge it in :)

Now re: /openmp:
D:\Repos\llm.c>cl.exe /Ox /fp:fast /openmp /I. /I .\dev\win train_gpt2.c
Microsoft (R) C/C++ Optimizing Compiler Version 19.39.33523 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

train_gpt2.c
train_gpt2.c(163): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(164): error C3015: initialization in OpenMP 'for' statement has improper form
train_gpt2.c(188): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(189): error C3015: initialization in OpenMP 'for' statement has improper form
train_gpt2.c(204): error C3015: initialization in OpenMP 'for' statement has improper form
train_gpt2.c(234): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(235): error C3015: initialization in OpenMP 'for' statement has improper form
train_gpt2.c(397): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(398): error C3015: initialization in OpenMP 'for' statement has improper form

(https://learn.microsoft.com/en-us/cpp/error-messages/compiler-errors-2/compiler-error-c3015?view=msvc-170)
Microsoft is always special. It wants the loops to be in this format. Notice the inline b and t were moved out.
void matmul_forward(float* out,
                    float* inp, float* weight, float* bias,
                    int B, int T, int C, int OC) {
    // most of the running time is spent here and in matmul_backward
    // OC is short for "output channels"
    // inp is (B,T,C), weight is (OC, C), bias is (OC)
    // out will be (B,T,OC)
    int b;
    int t;
    #pragma omp parallel for collapse(2)
    for (b = 0; b < B; b++) {
        for (t = 0; t < T; t++) {
            float* out_bt = out + b * T * OC + t * OC;
            float* inp_bt = inp + b * T * C + t * C;
            for (int o = 0; o < OC; o++) {
                float val = (bias != NULL) ? bias[o] : 0.0f;
                float* wrow = weight + o*C;
                for (int i = 0; i < C; i++) {
                    val += inp_bt[i] * wrow[i];
                }
                out_bt[o] = val;
            }
        }
    }
}

After fixing all those
D:\Repos\llm.c>cl.exe /Ox /fp:fast /openmp /I. /I .\dev\win train_gpt2.c
Microsoft (R) C/C++ Optimizing Compiler Version 19.39.33523 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

train_gpt2.c
train_gpt2.c(165): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(194): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(242): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(407): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
Microsoft (R) Incremental Linker Version 14.39.33523.0
Copyright (C) Microsoft Corporation.  All rights reserved.

/out:train_gpt2.exe
train_gpt2.obj

Kaboom!
D:\Repos\llm.c>train_gpt2.exe
[GPT-2]
max_seq_len: 1024
vocab_size: 50257
num_layers: 12
num_heads: 12
channels: 768
num_parameters: 124439808
train dataset num_batches: 1192
val dataset num_batches: 128
num_activations: 73323776
val loss inf
I even removed the /Ox and /fp:fast
D:\Repos\llm.c>cl.exe /openmp /I. /I .\dev\win train_gpt2.c
Microsoft (R) C/C++ Optimizing Compiler Version 19.39.33523 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

train_gpt2.c
train_gpt2.c(165): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(194): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(242): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(407): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
Microsoft (R) Incremental Linker Version 14.39.33523.0
Copyright (C) Microsoft Corporation.  All rights reserved.

/out:train_gpt2.exe
train_gpt2.obj

D:\Repos\llm.c>train_gpt2.exe
[GPT-2]
max_seq_len: 1024
vocab_size: 50257
num_layers: 12
num_heads: 12
channels: 768
num_parameters: 124439808
train dataset num_batches: 1192
val dataset num_batches: 128
num_activations: 73323776
val loss inf

We're still not there for windows runs...

I removed parallelization on every layer except matmul_forward  and 7 seconds.
D:\Repos\llm.c>cl.exe /openmp:llvm /fp:fast /Ox /I. /I .\dev\win train_gpt2.c
Microsoft (R) C/C++ Optimizing Compiler Version 19.39.33523 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

train_gpt2.c
Microsoft (R) Incremental Linker Version 14.39.33523.0
Copyright (C) Microsoft Corporation.  All rights reserved.

/out:train_gpt2.exe
train_gpt2.obj

D:\Repos\llm.c>train_gpt2.exe
[GPT-2]
max_seq_len: 1024
vocab_size: 50257
num_layers: 12
num_heads: 12
channels: 768
num_parameters: 124439808
train dataset num_batches: 1192
val dataset num_batches: 128
num_activations: 73323776
val loss 5.252007
step 0: train loss 5.356173 (took 7235.000000 ms)
step 1: train loss 4.300982 (took 7000.000000 ms)
step 2: train loss 4.623275 (took 7015.000000 ms)
step 3: train loss 4.600286 (took 8797.000000 ms)
step 4: train loss 4.616758 (took 10031.000000 ms)
step 5: train loss 4.231482 (took 9579.000000 ms)

Unfortunately enabling parallelization in attention and softmax errors out with inf. loss.

@azret  when you say "errors out with inf. loss" what value metric are you talking about or is it crashing? Also, your compiler flags are still off a bit. Just an FYI. Not sure how it's building for you.

@azret - adding /fp:fast definitely hangs it. But it runs fine about 3 seconds per step with /0x and the top of the trunk for me.

@rosslwheeler thank you very much. I have tried you project files. Very nice setup. They are very useful though when debugging. Unfortunately it's not about the projects files. There is only one file to build, really. You can just command line it.
It is however about the compiler flags.
We want to match the performance on Windows in the same ballpark that they have on Mac and Linux.  Turning all debug flags off. Turning all optimizations on. Turning fast floating points on. In addition, we want to parallelize as many layers as possible. So, when you finetune all these compiler flags on MSVC you get #19.
This is what we are trying to solve. And thanks to @dagelf we can now use /Ox +/fp:fast can just exclude the gelu layer see https://github.com/karpathy/llm.c/pull/134#issuecomment-2065415311

@azret got it! Thanks for the clarity!

@azret - adding /fp:fast with the pragma wrapper around gelu_backward took the OpenMP steps from ~3.5 seconds down to ~2.5 seconds. This is with zero code changes - about what you would expect based on your data above? Also, running WSL with GCC - would you expect it to be roughly the same speed as Windows (I know the compilers are different - just curious what you've seen)?

I don't think the ifdefs are too much... maybe we can even do a custom tanhf function to include.
I have two questions about this pull request,

Why do you want to create a header file in a Makefile rather than just adding it to the repo and including it where needed?
Does this pull actually fix OpenMP for msvc or is that still a work in progress?

@dagelf

Andrej was asking for minimal changes - discussion in https://github.com/karpathy/llm.c/pull/118 and check the top of this thread. I'm fine with a header in the repo or an auto-generated one.
OpenMP has been working in MSVC for about a week in my forks. I would not call it a fix per se - it's a work-around. I filed a bug report with Microsoft's compiler team today on it. That's why we need the const source fixes in the PR. Otherwise, all we would need is the Makefile changes at a minimum.

wow the diff on this PR. This is what is needed to compile C on Windows?

@karpathy - we're running some compiler limitations with Windows.

OpenMP 5.2 - requires either the const changes OR changes in the actual C code to move some var initializers around. No other work-around that I'm aware of. I've filed a bug report with the MSVC team. So our choice is either fix the const's or fix the initializers in the OpenMP functions.
unistd.h - Windows doesn't have this file. We can put unistd.h in dev? Would that work? The alternative is to auto-generate it in the makefile (that's what you see here in the PR).

Thoughts?

The const changes look good even as a general thing independent of Windows, I'd merge that regardless of Windows considerations.
The makefile echo I'm not a huge fan of.
Ideally we wouldn't introduce more files in the root of the repo. If there's any ways to hide them that could be a good idea.
As for where, I'm thinking of dev as a scratch space for dev. Maybe it makes sense to create a platforms/win, directory for any platform-specific code we might want and put things there.

@karpathy - Not a fan of the Makefile header echo either. I'd rather move the header into dev or platform/win. Either works for me. Just let me know and I can generate a PR for this.

Closing this - creating a document to cover building with different methods for Windows/MSVC.


[Bug] Discord Buttons & Components

Hey. today discord released the buttons but somehow i find 2 codeexamples in the docs one use buttons and one use components. if i use the components exaple i get a json not seziable error and if i use buttons i get this:
Ignoring exception in on_message
Traceback (most recent call last):
  File "D:\Downloads\Python\python 3.8.7\lib\site-packages\discord\client.py", line 343, in _run_event
    await coro(*args, **kwargs)
  File "D:/my_python_projects/MongoDB Test/mongosetup.py", line 31, in on_message
    m = await msg.channel.send(
  File "D:\Downloads\Python\python 3.8.7\lib\site-packages\discord_buttons\client.py", line 57, in send_button_msg_prop
    return await self.send_button_msg(ctxorchannel, *args, **kwargs)
  File "D:\Downloads\Python\python 3.8.7\lib\site-packages\discord_buttons\client.py", line 150, in send_button_msg
    data = await self.bot.http.request(
  File "D:\Downloads\Python\python 3.8.7\lib\site-packages\discord\http.py", line 192, in request
    async with self.__session.request(method, url, **kwargs) as r:
AttributeError: 'NoneType' object has no attribute 'request'

anyone knows a solution?

same error
AttributeError: 'NoneType' object has no attribute 'from_json'```

I have a fix. Just gimmie a moment and I'll set up a PR

Nvm. I guess i was a little to quick to read this issue, my PR fixes the error that @helish88 gets.
But I think your error stems from using discord-buttons and not the updated discord-components @FeelsBadMan1

Nvm. I guess i was a little to quick to read this issue, my PR fixes the error that @helish88 gets.
But I think your error stems from using discord-buttons and not the updated discord-components @FeelsBadMan1

Okey it is good that you solved to error from the other one that had almost the same. I defently tried both modules buttons aswell as components. while using the components i get this errorlog:
 json not seziable


support wj poc

变更内容

添加：getBucketLocation接口；
添加：putBucketStorageCapacity和getBucketStorageCapacity接口；
添加：getBucketMeta接口；

Coverage increased (+0.4%) to 91.698% when pulling 914dfef0fee024607980dc8dd73a662614cb5af6 on dev-head-bucket into f7c787b19aa0435fd19eee0cee50076dea600803 on master.

Coverage increased (+0.4%) to 91.698% when pulling 4e53c89ac8e871711c3a7a69d163b589e683fe6f on dev-head-bucket into f7c787b19aa0435fd19eee0cee50076dea600803 on master.


Plugin icon isn't removed from sidebar when disabled

Apologies for a late reply, I have been pretty busy with other things.
I'll continue investigating this.

Fix merged with the plugin-hub repository.  You should get an update on restart.


feat: add proxy pallet

Fixes: #286

@jak-pan What extrinsics do we want to allow for proxy accounts?

not sure if we also want to include NonTransfer proxy group or some other use case specific groups.

add transfer group

I have decided to not include cross-chain transfers in the transfer proxy group.


Bugfix: Single-select - option selected directly after the component has rendered now also di…

…splayed correctly if selected
By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
The option selected directly after the component has rendered now displays correctly as selected with the checkmark

📦 Published PR as canary version: 20.12.1--canary.430.bb854649eaeda1fcf222539aa963e2ffd0dc7647.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

When I set the "selected: true" on the first option, what is displayed on the field is "undefined", and the first option does not have the checkmark on.
When an option has been selected, then you select 'disabled', the selected option becomes 'undefined'.

If all of the options are "selected: false", then the select list is empty.
The displayed option is always the previously selected option, not the current one. Eg. if I select option A, then option B, the field shows option A. If I then select option C, the field shows option B.
When search is disabled (search-enabled: false), then two options get selected (marked) at the same time.


faker 17.0.0

Recipe directory diff between current master and this update:
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
index 29fe4e4..ccedf91 100644
--- a/recipe/meta.yaml
+++ b/recipe/meta.yaml
@@ -1,4 +1,4 @@
-{% set version = "8.8.1" %}
+{% set version = "17.0.0" %}

 package:
   name: faker
@@ -6,7 +6,7 @@ package:

 source:
   url: https://pypi.org/packages/source/F/Faker/Faker-{{ version }}.tar.gz
-  sha256: ccd76cd86a49f1042811faaa3a7d1b094fcf8e60a1ec286190417bbb5a3f2f76
+  sha256: 17cf85aeb0363a3384ccd4c1f52b52ec8f414c7afaab74ae1f4c3e09a06e14de

 build:
   number: 1

Jira ticket:  (-)
The upstream data:
Github releases:  https://github.com/joke2k/faker/releases
Diff between the latest and previous upstream releases
License: https://github.com/joke2k/faker/blob/master/LICENSE.txt
Changelog: https://github.com/joke2k/faker/blob/master/CHANGELOG.md
Requirements:

setup.cfg:  https://github.com/joke2k/faker/blob/v17.0.0/setup.cfg
setup.py:  https://github.com/joke2k/faker/blob/v17.0.0/setup.py

Actions:

Notes:
*
Additional info:

Package's statistics

Priority D | effort: easy | Category: other | subcategory:  | pkg_type: python
Outdated platfroms: 'linux-64', 'linux-aarch64', 'linux-ppc64le', 'osx-64', 'osx-arm64', 'win-64'
Latest version: 17.0.0
Release on PyPi:

version: 17.0.0
date: 2023-02-13T17:20:08

PyPi history
Popularity:

3 months downloads: 50277.0
All time:  718646 downloads -  faker  17.0.0  Faker is a Python package that generates fake data for you

Other checks:

[ ] Check the pinnings

[ ] Verify that the build_number is correct

[x] has setuptools

[x] has wheel

[x] pip in test

[ ] Verify the test section

[ ] Verify if the package is architecture specific

[ ] Verify that private modules are not mentioned in the recipe For example: (_private_module)

[x] license_file: LICENSE.txt is present

[x] license_family MIT is present

[x] License: MIT
[x] License is spdx compliant

Check if the license identifier has correct name from the SPDX License List

Identifier

MIT

MIT-0

MIT-advertising

MIT-CMU

MIT-enna

MIT-feh

MIT-Modern-Variant

MIT-open-group

MITNFA

Check dependency issues:

../aggregate/faker-feedstock/recipe/meta.yaml
Dependencies: ['wheel', 'python >=3.6', 'python-dateutil >=2.4', 'text-unidecode ==1.3', 'pip', 'setuptools']
noarch:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-ppc64le:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-s390x:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-arm64:

py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

win-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

Package Build Score: 16

Links:

Anaconda Recipes feedstock
Update branch
conda-forge recipe
PyPI
Diff between upstream and feature branch
Diff between origin and feature branch
The last merged PRs

Updating the recipe:
If the recipe needs additional modification the update branch can be modified. Note that the PR diffs are not updated with these changes.
git clone -b 17.0.0 git@github.com:AnacondaRecipes/faker-feedstock.git

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2b9itkwzc7/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_2b9itkwzc7/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_2b9itkwzc7/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1b7n7ec4xx/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_1b7n7ec4xx/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_1b7n7ec4xx/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_3ek086b09c/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_3ek086b09c/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_3ek086b09c/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b2skx8_uhz/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_b2skx8_uhz/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_b2skx8_uhz/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_22tll1h6a4/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_22tll1h6a4/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_22tll1h6a4/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_676ymc9pon/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_676ymc9pon/clone/recipe/meta.yaml:37: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_676ymc9pon/clone/recipe/meta.yaml:52: missing_description: The recipe is missing a description
Errors were found


linux_auditd_detection

Details
What does this PR have in it?
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.
[ ] Confirm updates to lookups are handled properly.

Notes For Submitters and Reviewers

If you're submitting a PR from a fork, ensuring the box to allow updates from maintainers is checked will help speed up the process of getting it merged.
Checking the output of the build CI job when it fails will likely show an error about what is failing. You may have a very descriptive error of the specific field(s) in the specific file(s) that is causing an issue. In some cases, its also possible there is an issue with the YAML. Many of these can be caught with the pre-commit hooks if you set them up. These errors will be less descriptive as to what exactly is wrong, but will give you a column and row position in a specific file where the YAML processing breaks. If you're having trouble with this, feel free to add a comment to your PR tagging one of the maintainers and we'll be happy to help troubleshoot it.
Updates to existing lookup files can be tricky, because of how Splunk handles application updates and the differences between existing lookup files being updated vs new lookups. You can read more here but the short version is that any changes to lookup files need to bump the datestamp in the lookup CSV filename, and the reference to it in the YAML needs to be updated.

@tccontre :awesome work here! made some updates in here and we are good to go!


Remove: Remove env-logger from navigator-rs library

Due some issues on Jupyter and navigator-assistant,
Noticed that Logger call panick if there is another instances or logging services.
https://docs.rs/env_logger/latest/env_logger/fn.init.html
UPDATE
We removed env-logger from navigator-rs library, loggers should be initiallized from the application side.

Actually, I'd rather not use env-logger, but only the log one, and let the log part always be done on the application side, as mentioned here.

Ok, I agree with the idea,
So we can just remove the entire logger.

Will wait @joaoantoniocardoso to approve and merge


FEATURE: 表单什么时候发布API呢

Version
Vben Admin V5
Description
表单什么时候发布API呢
Proposed Solution
表单什么时候发布API呢
Alternatives Considered
No response
Additional Context
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.

duplicate #4682


Add saptune support for SLES

This PR adds saptune support for SLES based systems in the sap_hana_preconfigure and sap_netweaver_preconfigure roles.  As discussed, this replaces the current Ansible SAP note implementation with saptune on SLES systems.
The documentation has been updated to reflect these changes.
Assert mode has also been added for SLES for the above roles.

Added new commit fixing highlighted issues.

@mr-stringer We have some ansible-lint errors. Please have a look at the results of the GitHub workflows for sap_hana_preconfigure and sap_netweaver_preconfigure. I am going to remove the ansible-lint errors for the defaults/main.yml files in a separate PR.

I just checked and found that the trailing spaces in defaults/main.yml are part of your PR. So I'll leave it to you to remove them.


Album Art

Hi,
Is there anywhere in the interface that we would like to show artwork (if existing)?
It could be as simple as displaying cover.jpg in the same folder, or if not existing, the first image file found in the folder.
Cheers

Hello Carlos I'm a great fan of your app, it is probably the one on my phone I most enjoy! Many try to do too much in their functionality and interface and so this project is very refreshing. But I also miss album art. Sometimes when I look for a particular song or album I like to see a familiar cover while I listen. Maybe there could be a visualizer that displayed the embedded cover under the song? I don't know if that's feasible though.
OK thanks 4 your hard work

Hi, there!!!
Thanks for the words! I'm super happy to know you like the player :)
Yes, that would be really cool. Suggestion noted!
I'm about to upload a new version to the Play Store, with improvements for
the Particles visualizer. As your suggestion is not very complex to
implement, expect to see it soon ;)
Best regards!
On Sun, Feb 22, 2015 at 6:17 PM, hasturhub<EMAIL_ADDRESS>wrote:

Hello Carlos I'm a great fan of your app, it is probably the one on my
phone I most enjoy! Many try to do too much in their functionality and
interface and so this project is very refreshing. But I also miss album
art. Sometimes when I look for a particular song or album I like to see a
familiar cover while I listen. Maybe there could be a visualizer that
displayed the embedded cover under the song? I don't know if that's
feasible though.
OK thanks 4 your hard work
—
Reply to this email directly or view it on GitHub
https://github.com/carlosrafaelgn/FPlayAndroid/issues/13#issuecomment-75460128
.


Can't download the PolypGen dataset

Hi, thanks for the great work. However,  the link (https://www.synapse.org/#!Synapse:syn26376620.1) provided in the paper [1] is invalid. Can you help me?
[1] Sharib Ali etl. PolypGen: A multi-center polyp detection and segmentation dataset for generalisability assessment

Me too ,ask for help.


AB#6877 Any code cleanup for task related to MIP label updates for odsp driver.

Description
As mentioned in the comment here: https://github.com/microsoft/FluidFramework/pull/19142#discussion_r1445266736
Make the sensitivityLabelsInfo on join session response non optional once new service code which provides this in the response saturates.
This task also tracks any other code cleanup related to this feature like always listening on doc service once we have the driver version saturation.

There are some failing tests caused by the changes, you can see which file and which line have the errors:
deltaConnectionUpdateTests.spec.ts

134

joinSessionCacheTests.spec.ts

110
176

joinSessionPeriodicCall.spec.ts

139
177

socketTests.spec.ts

127
158
192
224
255
290
322
359
391


Add option for embedded resources.

Expected
There be an option for embedded resources.
Actual
There isn't one.

Generated scripts are being embedded into a .dll by default.
http://bridge.net/kb/global-configuration/#injectScriptToAssembly
Please clarify what else would you like to embed and what options could we provide? Please elaborate.

In .Net, you can embed any type of file (.wav, .txt, .csv, .mp3, etc...) into code. In Bridge, I don't think there is a way to do that.

Thank you for clarifying.
This functionality relates to Reflection - #15. Once Reflection is implemented, it opens a gate for this feature.

Embedding Resources is now supported in Bridge as of version 15.0.
We will have to provide some more user friendly samples. See #1793.
The related Unit Tests can provide some insight into how this functionality works:
https://github.com/bridgedotnet/Bridge/blob/master/Tests/Batch1/Reflection/AssemblyTests.cs#L248-L278


Table footnotes

Why
Implementation of Feature 'Table Footnotes'.
Related:

#556

What

[x] Extend kitchen-sink (O)
[x] Extend TextureArticleSchema to allow for <table-wrap-foot>
[x] Extend TableFigure to allow for 0-n footnotes (-> children)
[x] Implement Importer and create footnotes and populate tableFig.footnotes
[x] Implement a label generator for table footnotes (O)
[x] Refactor internal xref collection API for XrefEditDialog (O)
[x] Display footnotes and footnote references in TableFigure
[x] Implement dialog for table-footnotes so that it allows only to choose from table footnotes
[x] InsertFootnoteCrossReferenceCommand for both regular and table footnotes according to detected scope
[x] Determine xpath for table selections (CustomSelection)
[x] Implement an InsertFootnoteCommand that creates a footnote in the according scope
[x] Add InsertFootnoteTool to both views (currently only metadata)
[x] After creating the first footnote xref, table- footnotes do not update anymore (O)
[x] Fix TableFootnoteManager issues (e.g. no label for xrefs in caption) (O)
[x] Add tests for table-footnote label generator (O)
[x] Order footnotes in order of cross-refs (similar to how references are ordered) (O)
[x] Implement exporter that stores created footnotes into <table-wrap-foot> (O)
[x] Add a command and tool to remove a footnote (enabled when cursor inside fn)
[x] Add tests for table-footnote interaction (create, delete, x-reference)
[x] Fix table editor in Metadata mode
[x] Add labels for table figures ('Label:', 'Title:', 'Caption'; 'Footnotes:') and regular figures styled in a same way as we have it in manuscript

Note: please introduce a TableFigureModel (extends FigureModel) and a TableFigureComponent (extends FigureComponent)

@Integral I have started a first iteration on the label generator (WIP)

If you need the label for rendering, there is tableFigure._tableFootnoteManager which has getLabel(pos)

For the EditXrefDialog I will have to do some refactorings. The current approach is not sufficient.

@oliver---- do we want to use collections for managing table footnotes? E.g. one collection for each figure? I see that with footnote manager we are already going this way, and I think that it we should do that the same with collections...

Nice. Also it would be easier for us to manage it.

UI needs improvement: #1069

Quick-fixed the Xref target resolver. At some point this needs a refactor.

We should do an iteration on metadata table. There are no fn renderers and wrong css classes now. Also tools should be available there. My suggestion would be to do it separately closer to test phase.

@michael we need your action regarding a potential requirement for re-ordering of (table) footnotes.

@Integral very nice! Thanks for the tests!

Perfect! 👏

Let's get this merged! 🍾 🎉 🎆

Wonderful!
On Fri, 7 Dec 2018, 09:05 Oliver Buchtala<EMAIL_ADDRESS>wrote:

Merged #1057 https://github.com/substance/texture/pull/1057 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/texture/pull/1057#event-2011652616, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALG-v3KSP402aNqrESo9qYl3ndztjBzks5u2gUogaJpZM4Y-bFr
.


Typo in background(color: UIColor) -> Attributes

This line should say NSBackgroundColorAttributeName.

@andreyz Thanks so much for finding this!
Ill quickly get around to fixing it but if you or someone is up for it Id gladly accept a Pull Request!

Just such a tiny edit, thought might be simpler for you :)


Feature Request: Store nested dictionaries as new tables

This is both a feature request and to get an idea if people would be interested. Essentially, if you have a sub-dictionary that is not at the top level, the entire thing also has to be encoded.
I propose that when a user explicitly says to, make a subdictionary a new table as well. The idea is still very rough, but essentially modify the main class to be able to have a parent class (and therefore share the connection). Then, add a function that creates a new SqliteDict child instance. The table name should either something like tablename + uuid4(). Then this new object will behave just like its own SqliteDict. The only difference will be when you __setitem__ (or update) a child instance, it will instead make a pointer to that table and when you __getitem__ it will recreate it and when you __delitem__, it will drop it.
I think autocommit should still work and carry through.
A final idea along the same vein (though certainly harder to do) is to make a list-like object that behaves the same way. Use a sub SqliteDict under the hood but allow for storage. THis may be farther fetched.
Any ideas?

You can try this:
from collections import defaultdict
from sqlitedict import SqliteDict

class nested_dict_sqlite(defaultdict):
    'like defaultdict but default_factory receives the key'

    def __missing__(self, key):
        self[key] = value = self.default_factory(key)
        return value

dicts = nested_dict_sqlite(lambda table: SqliteDict('dicts.sqlite3', table, 'c', True))
dicts['dict1']['x'] = 1
dicts['dict2']['y'] = 2

print(dicts['dict1']['x'])
print(dicts['dict2']['y'])
print(dicts['dict2']['yy'])

not best solution but let you work 2D


norm sir weights in logspace.

to avoid overflow when moving to exp space.
keep the normalisation in exp space below because the norm could change in case there are still some nans.

Codecov Report

Merging #499 (2949005) into main (0ba6c31) will decrease coverage by 1.35%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #499      +/-   ##
==========================================
- Coverage   69.05%   67.69%   -1.36%
==========================================
  Files          55       55
  Lines        3832     3968     +136
==========================================
+ Hits         2646     2686      +40
- Misses       1186     1282      +96

Flag
Coverage Δ

unittests
67.69% <0.00%> (-1.36%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sbi/mcmc/init_strategy.py
34.37% <0.00%> (-1.11%)
:arrow_down:

sbi/utils/sbiutils.py
71.09% <0.00%> (-11.63%)
:arrow_down:

sbi/inference/posteriors/direct_posterior.py
77.77% <0.00%> (-11.34%)
:arrow_down:

...inference/posteriors/likelihood_based_posterior.py
72.97% <0.00%> (-9.00%)
:arrow_down:

sbi/inference/posteriors/ratio_based_posterior.py
77.27% <0.00%> (-8.07%)
:arrow_down:

sbi/inference/posteriors/base_posterior.py
66.00% <0.00%> (-5.72%)
:arrow_down:

sbi/utils/__init__.py
100.00% <0.00%> (ø)

sbi/inference/snle/snle_base.py
97.59% <0.00%> (ø)

sbi/inference/snre/snre_base.py
96.80% <0.00%> (ø)

sbi/inference/snpe/snpe_a.py
66.50% <0.00%> (+0.15%)
:arrow_up:

... and 1 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 0ba6c31...2949005. Read the comment docs.


Improve reporting of Tidal geographic restrictions

While looking at errors in the server logs, I noticed that there are some albums for which the Tidal API's /albums/<id> endpoint returns a 404 when passed a country where the album is unavailable:
20:44:18 Fetching https://api.tidal.com/v1/albums/251633624/tracks?countryCode=US
20:44:18 Fetching https://api.tidal.com/v1/albums/251633624/credits?countryCode=US
20:44:18 Fetching https://api.tidal.com/v1/albums/251633624?countryCode=US
20:44:18 Sending 500 to XXX: album: not found

% curl --silent "https://api.tidal.com/v1/albums/251633624?countryCode=US" -H x-tidal-token:gsFXkJqGrUNoYMQPZe4k3WKwijnrp8iGSwn3bApe
{"status":404,"subStatus":2001,"userMessage":"Album [251633624] not found"}

I'd previously just seen albums that are still partially available in the queried country (as reflected by /tracks returning a shorter tracklist). /albums still works in that case.
As usual (?), this album is fully available in Norway:
% curl --silent "https://api.tidal.com/v1/albums/251633624?countryCode=NO" -H x-tidal-token:gsFXkJqGrUNoYMQPZe4k3WKwijnrp8iGSwn3bApe
{"id":251633624,"title":"Like Drawing Blood (Deluxe Version)","duration":4347,"streamReady":true,"streamStartDate":"2012-04-01T00:00:00.000+0000","allowStreaming":true,"premiumStreamingOnly":false,"numberOfTracks":16,"numberOfVideos":0,"numberOfVolumes":1,"releaseDate":"2006-01-01","copyright":"Lucky Number Music Limited","type":"ALBUM","version":null,"url":"http://www.tidal.com/album/251633624","cover":"32aaa3ef-9ae6-4f3b-ad8e-a68562087836","vibrantColor":"#ebe5c4","videoCover":null,"explicit":false,"upc":"5025425173233","popularity":0,"audioQuality":"LOSSLESS","audioModes":["STEREO"],"artist":{"id":3544059,"name":"Gotye","type":"MAIN","picture":"c91486ee-a3ca-451c-944c-704e5853586e"},"artists":[{"id":3544059,"name":"Gotye","type":"MAIN","picture":"c91486ee-a3ca-451c-944c-704e5853586e"}]}

It'd be nice to find some way to detect which countries an album is available in without querying them all, but I'm not sure if that's possible.
At the very least, I think that I should change the user-visible error message for album lookup failures to indicate that there may be geographic restrictions.

Another annoying thing is that there's no way to specify the country to use for the album and credits lookups when passing -country XW -- the code just defaults to US. I had thought that was safe to do, but it isn't.
I could let the user additionally specify the country to use for these lookups, but another possibility would be to do the tracklist lookups first and then use the results to choose the country for getting the album and credits. The downsides are more-complicated code and two network round trips instead of one when -country XW is passed, but those seem pretty minor.


feat(28): Add write-stream output support

Context
There are situations where users may not want to save the thumbnails to disk, instead, they may want to pipe the image stream to a write stream (eg. S3, HTTP response, etc.).
Users will now be able to pass Stream.Writable as the output argument, eg.
// "app" refers to an Express application
app.get('/give/me/a/thumbnail/please', (req, res) => {
  // cool, huh?
  genThumbnail('path/to/video.webm', res, '150x100')
    .then(() => console.log('Done! :D'))
    .catch(() => console.error('Error! D:'))
})

Objective

Allow users to path a writable stream instead of a file path

References

https://github.com/ScottyFillups/simple-thumbnail/issues/28

Lessons learned

pnpm does not play nicely with ALE and Standard :sob:

@cmd430 Is it alright if I add you to the contributors section? :smiley:

Yeah no worries :)

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Installing via pip does not make the package available

I want to use this package as a PIP install in a Docker container, but am unable to get it to work.
I have tried simply installing the package in my local environment using python3 -m pip install markov-word-generator, which seems to go fine.
Trying to use the package, however, does not work:
from markov_word_generator import MarkovWordGenerator, WordType
results in ModuleNotFoundError: No module named 'markov_word_generator'
I could include the project by adding it to my code base, but I would very much like to avoid that and simply install it using pip and a requirements file.

Thanks for the report, the package was malformed.
Published a new revision
pip3 install markov-word-generator==0.6
cd /tmp
python3
Python 3.10.12 (main, Sep 11 2024, 15:47:36) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from markov_word_generator import MarkovWordGenerator, WordType
>>> generator = MarkovWordGenerator(
...     markov_length=4,
...     dictionary_filename="/tmp/EN-words.dic"
... )
>>> print(generator.generate_word())
diapas
>>> print(generator.generate_word())
punition
>>> print(generator.generate_word())
constoned

That works! Thank you very much!


[Core] Add experimental tag for extensions

An extension can now be marked as experimental by adding "azext.isExperimental": true to azext_metadata.json:
{
    "azext.minCliCoreVersion": "2.0.46",
    "azext.isPreview": true,
    "azext.isExperimental": true   // Add here
}

When it appears
Same as preview, experimental appears when
Installing the extension from index
> az extension add -n webapp
The installed extension 'webapp' is experimental and not covered by customer support. Please use with discretion.

Installing the extension from wheel
This is newly added, also applies to preview
> az extension add -s "webapp-0.2.24-py2.py3-none-any.whl"
The installed extension 'webapp' is experimental and not covered by customer support. Please use with discretion.

Command help
> az webapp container up -h
This command is from the following extension: webapp
The extension is experimental and not covered by customer support. Please use with discretion.

ℹ If true, azext.isExperimental will override azext.isPreview
ℹ Experimental/Preview warning doesn't appear when invoking a command within an experimental/preview extension
Change of existing commands
For extension list-available, extension property is added to each entry.
[
  {
    "experimental": false,   // New
    "installed": false,
    "name": "vm-repair",
    "preview": false,
    "summary": "Auto repair commands to fix VMs.",
    "version": "0.2.6"
  },
  {
    "experimental": false,   // New
    "installed": true,
    "name": "webapp",
    "preview": true,
    "summary": "Additional commands for Azure AppService.",
    "version": "0.2.24"
  },
  ...
]

For extension list, preview and extension properties are added to each entry.
> az extension list
[
  {
    "experimental": false,   // New
    "extensionType": "whl",
    "name": "support",
    "preview": true,         // New
    "version": "0.1.1"
  },
  {
    "experimental": true,    // New
    "extensionType": "whl",
    "name": "webapp",
    "preview": true,         // New
    "version": "0.2.24"
  },
  ...
]

add to S167

LGTM. The overiding feature seems reasonable. It's interesting to think that if more tags need to be added, how we change the design and existing code.

This is the tricky part because we need to keep azext.isPreview so that old extensions can still be installed while adding more tags. But luckily, it is not very likely to require more tags (hopefully).


how do we define a virtual function in typegoose

is there a way to integrate a virtual function into the typegoose schema definition?

this was precipitated by mania surrounding _id and id within mongoose
and it seems these modeling issues are now buried for good
see
https://github.com/szokodiakos/typegoose/issues/121

I am reopening this because mongoose insists on storing models using _id
and thats whats landing in the mongodb
my exercising of @prop id was a side show it had no affect as it does not sync with the mongoose objectId as denoted by _id
this needs to be resolved with virtual function support in typegoose

this can be closed, because i cant find any problem with .id or ._id (with the latest mongoose & typegoose)
@Ben305


GitLab reviews may not have the updated_at field set

On behalf of Loïc Dachary

Fallback to created_at if that the case and to time.Now() if it is
also missing.
Fixes: #18434

source

Codecov Report

Merging #18450 (0a4d285) into main (3349fd8) will increase coverage by 0.00%.
The diff coverage is 42.19%.

@@           Coverage Diff            @@
##             main   #18450    +/-   ##
========================================
  Coverage   46.03%   46.04%
========================================
  Files         840      842     +2
  Lines       92856    93199   +343
========================================
+ Hits        42746    42912   +166
- Misses      43323    43479   +156
- Partials     6787     6808    +21

Impacted Files
Coverage Δ

cmd/restore_repo.go
0.00% <0.00%> (ø)

models/auth/twofactor.go
20.89% <0.00%> (-0.65%)
:arrow_down:

models/db/engine.go
36.13% <ø> (ø)

models/issue_milestone.go
72.11% <ø> (+1.76%)
:arrow_up:

modules/generate/generate.go
0.00% <0.00%> (ø)

modules/indexer/code/elastic_search.go
1.36% <0.00%> (-0.25%)
:arrow_down:

modules/migration/issue.go
100.00% <ø> (ø)

modules/private/restore_repo.go
0.00% <0.00%> (ø)

modules/queue/queue.go
37.09% <0.00%> (ø)

modules/queue/setting.go
24.32% <0.00%> (ø)

... and 69 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 726715f...0a4d285. Read the comment docs.

please send a backport :)


Switch embed to llama_get_embeddings_seq

Due to updates in https://github.com/ggerganov/llama.cpp/pull/5796, sequence level embeddings are now output through a separate channel from token level embeddings, and they are accessed with llama_get_embeddings_seq.

@iamlemec thank you! Just so I understand, the sequence level embeddings are the ones that are pooled up to the end of the from the last processed batch?
Also, I think the new function in llama_cpp.py is duplicated by accident.

Oh yeah, I didn't see you added it already! Yup, it's the pooled embeddings by sequence for the last batch. So for both mean pooling and cls (first token) it works, and for non-pooling it returns null.


PowerRename does not appear in Windows 11's default context menu

Microsoft PowerToys version
0.82.1
Installation method
WinGet
Running as admin
No
Area(s) with issue?
PowerRename
Steps to reproduce

I right clicked a folder
Apps that relies on the new context menu on Windows 11 don't show up (which is apparently a normal behavior)
Right click the folder again and apps like Windows Terminal, Winrar show up, except for PowerRename

✔ Expected Behavior
PowerRename should appear in default context menu on Windows 11.
❌ Actual Behavior
PowerRename doesn't show up in default context menu at all. I had to use the extended one.

Other Software
No response
Edit: Added the bug report
PowerToysReport_2024-07-27-00-08-13.zip

Does it still missing after rebooting?
/needinfo

I did reboot my computer and restarted PowerToys, PowerRename still doesn't show up in the new context menu.

Note that the attached report is still showing 0.81.0 version of MSIX for context menu packages even if user is on 0.82.1
Display name: PowerToys ImageResizer Context Menu
Full name: Microsoft.PowerToys.ImageResizerContextMenu_<IP_ADDRESS>_neutral__8wekyb3d8bbwe
Version: <IP_ADDRESS>.
Publisher: CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Status: OK

Display name: PowerToys PowerRename Context Menu
Full name: Microsoft.PowerToys.PowerRenameContextMenu_<IP_ADDRESS>_neutral__8wekyb3d8bbwe
Version: <IP_ADDRESS>.
Publisher: CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Status: OK

Oops, I forgot about that since i haven't used the laptop in a while and PowerToys hasn't been updated for a while.
There you go:
PowerToysReport_2024-07-27-12-59-10.zip

That's weird. The new bug report show 0.75.1 version for that packages
Display name: PowerToys ImageResizer Context Menu
Full name: Microsoft.PowerToys.ImageResizerContextMenu_<IP_ADDRESS>_neutral__8wekyb3d8bbwe
Version: <IP_ADDRESS>.
Publisher: CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Status: OK

Display name: PowerToys PowerRename Context Menu
Full name: Microsoft.PowerToys.PowerRenameContextMenu_<IP_ADDRESS>_neutral__8wekyb3d8bbwe
Version: <IP_ADDRESS>.
Publisher: CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Status: OK


Throw Errors

Error objects have a stack trace property which greatly helps tools. This will let developer debug nested errors more easily both in NodeJS and in the client.

👍


Footer Improvement

I can make this type of productive footer in website. If yo like this format you can assign this function to me.

@akshattiwarii i have assigned this issue before, if he does that then i will be closing this issue.
if he doesn't complete it then i will be assigning this issue to you.

we will look for this later, so i am closing this issue.
Please Understand.


Example issue Compilation error: 'mySerial' was not declared in this scope

Hello, first of all thanks for your work.
I am currently trying to get communication between two E220 working and seem to be struggeling a bit. In your examples mySerial is used in the loop however it is out of scope for me.
Also as a sidenote is there any known issues working with softwareserial as i am really struggeling to get messages to send/recieve.
Thanks for your help!

Hi there, sorry for the late response.
Would you be able to tell me which file the error is occurring in, Or possibly show me your code?
I've had a quick look through the examples and they should be fine in the release 1.1.2
A possible error I think you may be running into is my use of Serial3, unique to Due and Mega board, if so you need to use the commented-out software serial coded instead.
If you can provide me some more info we should be able to solve this pretty easily

Closed due to lack of information


chore(master): release 0.4.25

:robot: I have created a release beep boop
0.4.25 (2024-08-30)
🐞 Bug Fixes

handle properly small or zero tasksStealingRatio (ab6e570)

✨ Polish

move dynamic pool only getters to its own class (74970f2)

🧪 Tests

fix variable casing (a2727d7)
refine pool event tests expectation (5bdf99a)

📚 Documentation

cleanup code examples (28b119c)
refine pool ready event emission condition (e7d0673)
switch examples to top level await usage (ESM) (30781c3)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.4.25
:sunflower:


Upgrade formatter to clang-format-11

CI in #733 is failing due to an upgrade of Github Actions CI from Ubuntu Focal to Ubuntu Jammy. clang-format-9 is no longer available in Jammy. (https://packages.ubuntu.com/focal/clang-format-9 vs https://packages.ubuntu.com/jammy/clang-format-9) It seems the lowest version of clang-format in Jammy is clang-format-11, which was released about a year after clang-format-9, in October 2022. I propose we upgrade our standard formatter from 9 to 11.
It turns out I have already been formatting code with clang-format-14 so I don't think there will be any formatting change.

Coverage increased (+0.0007%) to 98.641% when pulling e11b84c41ecaae37a7c891f83463b4aad4ce4f32 on isaacbrodsky:clang-format-11 into ab09177189516318c6c2b78bb17f4929ec27354e on uber:master.