Miscellaneous Minor Doc Enhancements and Fixes

👾 Description of the issue
After the effort to split openjdk-build into two, we find ourselves with some documentation that is out of date, jumbled or outright wrong in some places. This issue intends to amend these irregularities by tidying up the visual features of this repo. Some ideas:

We have a lot of .md files at the root of this repo that could be moved into their own directory (called docs/ maybe?). NOTE: SECURITY.md and LICENSE would need to stay as they are special files that GitHub needs to have at the repository root.
README.md contains a few images that are invalid (i.e. they show a blank image). It would be good to have these replaced or removed.
The Metadata section of the README file above can be removed as it is more a part of the pipeline side of the project instead of the build side.
CONTRIBUTING.md could be condensed down to be less wordy. It's probably quite a daunting document to read for new contributors.
The FAQ contains an error that states that the make-adopt-build-farm.sh script is a G script, not a S script
There are probably more that I have missed as this was just from a quick look through.

📋 Step by Step
To solve this issue and contribute a fix you should check the following step-by-step list. A more detailed documentation of the workflow can be found here.

[x] Claim this issue: Comment below.
[x]  Fork the repository in github by simply clicking the 'fork' button.
[x]  Check out the forked repository
[x]  Create a feature branch for the issue. We do not have any naming definition for branches.
[x]  Commit your changes.
[x]  Start a Pull Request.
[x]  Done 👍 Ask in comments for a review :)
[x]  If the reviewer find some missing pieces or a problem they will start a discussion with you and describe the next steps how the problem can be solved.
[x]  You did it 🎉 We will merge the fix in the master branch.
[x]  Thanks, thanks, thanks for being part of this project as an open source contributor ❤

🤔❓ Questions
If you have any questions just ask us directly in this issue by adding a comment. You can join our community chat at Slack. Next to this you can find a general manual about open source contributions here.

Hi, I am Jessica and I'm new to contributing on projects. I'm currently looking for a first good issue to start contributing and this seems like it could be a good one. I would like to give it a go. Thank you

Welcome @ljessicc !  It is now assigned to you, if you have any questions, please feel free to ask them in this issue.

Very well. Thank you

Are the images mentioned in the README.md, the links referring to [slack], [DockerFileCheck]? I have tried finding valid updated links to replace these but I haven't been able to. Should I only delete the links or the whole paragraphs pointing to them?

Actually I just noticed many of the links in the FAQ.md are invalid

Are the images mentioned in the README.md, the links referring to [slack], [DockerFileCheck]? I have tried finding valid updated links to replace these but I haven't been able to. Should I only delete the links or the whole paragraphs pointing to them?

DockerFileCheck can probably be deleted, I don't think that job exists anymore after the rejig of the docker pipelines a while back. The image icon url for slack can probably be replaced by any generic one found on google. As for the actual link that the icon will take you too, it is possible to copy a link to a channel in slack by right clicking on the channel. Maybe we could replace the https://slackin-jmnmplfpdu.now.sh/ part with whatever that link may be?

Ok. Thanks

I am done with my changes, please review

PR was merged


Bug Report: Extra comma in filter command [-vf ", hflip"] causes encoding failure

Describe the bug
When using the "hflip" filter to horizontally flip a video, the current command includes an extra comma and space before the filter name, causing the encoding process to fail.
To Reproduce
Steps to reproduce the behavior:

Open WebM for Lazys v3.40.7 (or fresh install of v3.40.0 from WebMConverter-3.40.0.zip)
Attempt to encode a video with horizontal flip (or "Preview filters")
Observe the error message in the output

Expected behavior
The video should be successfully encoded with the horizontal flip applied.
Screenshots

Software version(please complete the following information):

WebM for Lazys: v3.40.7
Avisynth: 2.6

Additional information:
Claude AI informed that the current command includes -vf ", hflip", which results in an error:
WebM for Lazys v3.40.7: -vf ", hflip"
Arguments: -f nut -i pipe:0   -an -c:v libx264 -pix_fmt yuv420p -threads 8 -slices 8 -metadata title="2024-09-13 17.17.26<PHONE_NUMBER>966763356_22797478" -lag-in-frames 16 -auto-alt-ref 1 -vf ", hflip" -qmin 17 -crf 19 -qmax 21 -qcomp 1 -b:v 0 -f mp4 -y "O:\4K Stogram\annelesemilton\story\2024-09-13 17.17.26<PHONE_NUMBER>966763356_22797478-1.mp4"
--- CREATING AVISYNTH PROXY ---
Input #0, nut, from 'pipe:0':
  Metadata:
--- FFMPEG HAS EXITED ---
    encoder         : Lavf56.40.101
  Duration: N/A, start: 0.000000, bitrate: N/A
  Stream #0:0: Video: rawvideo (I420 / 0x30323449), yuv420p, 1080x1920, 30 tbr, 61440 tbn
[AVFilterGraph @ 00000159ce2666c0] No such filter: ''
[vost#0:0/libx264 @ 00000159ce265000] Error initializing a simple filtergraph
Error opening output file O:\4K Stogram\annelesemilton\story\2024-09-13 17.17.26<PHONE_NUMBER>966763356_22797478-1.mp4.
Error opening output files: Filter not found

ffmpeg.exe exited with exit code -1279870712. That's usually bad.
If you have no idea what went wrong, open an issue on Github and copy paste the output of this window there, also upload test video.

The command should be -vf "hflip" without the extra comma and space. Removing the extra comma and space from the filter command resolves the issue.

--- Final file size is 1,6 MB ---

And that file is correctly flipped.

Check latest update.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210718153436370 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210718153436370 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26721

Relevant Lines:
56896

💛  - Coveralls


devicePublicKey => devicePubKey

Trivial update aligning the extension name to the spec.

Hmm, there are a few unit tests that continue to reference the now "old" value devicePublicKey:

Can you update these tests so everything refers to this extension as devicePubKey as well?

Sorry, I overlooked your comment. Will fix them.

@MasterKale Updated the tests. Can you check?

lgtm :shipit:

There's no issue associated with this so I'll let you know here that this is now available in<EMAIL_ADDRESS>


[Question] Swift New Build System can't handle R.swift on Travis

As in title, I've turned on New Build System on my repository, but it has failed with following error:
❌  error: missing input '/Users/travis/build/drwalcore/trckr/R.generated.swift' and no rule to build it
❌  error: cannot build '/Users/travis/Library/Developer/Xcode/DerivedData/trckr-fobgqgzdhufvsjbtgcuwuruufjhf/Build/Intermediates.noindex/trckr.build/Debug-iphonesimulator/trckr.build/Objects-normal/x86_64/AppDelegate.o' due to missing input

I've run and tested it locally and R.swift was processed flawlessly. Maybe it's not R.swift related, so I am sorry for that 🤖
After turning  off New Build System it is working again!

I don't have any experience with R.swift + Travis. But like you I can build with R.swift and the New Build System just fine.
So I'm assuming that this is indeed a Travis issue.
Are you sure Travis is doing a clean build? Or is it reusing cached build results from a previous build with the Standard Build System?

Thank You for a response, I will look for issues on Travis side. I am pretty sure Travis CI makes a clean build every time the build starts. I will make some research,  and I will be really glad to help to resolve this issue, to be prepared for official New Build System usage along with R.swift and Travis duo!

I found this problem too. I believed the problem is you didn't keep R.generated.swift in your repository. You can reproduce this by deleting R.generated.swift and choosing New Build System then it will build fail on 1st time and it will build success on the next time.

This is a clever solution, thank You for the response. I will definitely give it a try.

Assuming that this is solved now, so I will close this one.

how to  add rswift generate in your build script.?

@AwaisFayyaz what do You mean? Everything is written there...
https://github.com/mac-cain13/R.swift#cocoapods-recommended

I had another "missing input" error. I was confused with this. sorry, removed my comment. you can also remove your reply. after that i will remove this message.
BTW, i just learned about R.swift. Cool

No worries :)!
Z poszanowaniem,
Michał.
On 13 Aug 2018, 14:01 +0200, M.Awais Fayyaz<EMAIL_ADDRESS>wrote:

I had another "missing input" error. I was confused with this. sorry, removed my comment. you can also remove your reply. after that i will remove this message.
BTW, i just learned about R.swift. Cool
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Adding generated source files to the repo is not a nice solution.
Better solution: in the run script phase just specify the generated file as an output file. That will satisfy the new Xcode build system which needs that information to properly parallelize and schedule build tasks.

We observe that adding only the output file is not enough. It will fix running R.swift on a clean build. But on a incremental build it will still fail to detect changes.
I don't understand why Xcode 10 makes the assumption this is okay. I would think that no inputs defined means that we should be runned always, but it seems that is not the case. So I'm affraid we need to generate input file lists.

Urgs. That sounds like a bug we should file a Radar for.
In session 408, Apple was pretty clear about when no input files were declared, that the build phase would always be run.
https://developer.apple.com/wwdc18/408

Note that only adding R.generated.swift as an Output File to the R.swift build step is not sufficient.
As this means the builds step is only run once during incremental builds. Since the output file already exists in subsequent builds.
For continued discussion on this topic, see:
https://github.com/mac-cain13/R.swift/issues/456


bug: Trouble windows should set winfixbuf

Did you check docs and existing issues?

[X] I have read all the trouble.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of trouble.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0
Operating system/version
Ubuntu 22.04
Describe the bug
Trouble's special windows do not set winfixbuf. In particular, when in the qflist window, using :cnext or :cprev results in a Cursor position outside buffer error (whereas neovim's own copen/cfile window handles this perfectly fine).
Steps To Reproduce

nvim -u repro.lua
:cfile cfile.txt (attached cfile.txt)
:copen, switch to this window
:cnext, works perfectly fine
:Trouble qflist toggle, switch to the Trouble window
:cnext, results in error

Expected Behavior
:cnext and :cprev should work as if in the native copen window.
Note: If :set winfixbuf is used inside the Trouble window, then :cnext and :cprev work as expected.
Repro
-- use vim-plug
local vim = vim
local Plug = vim.fn['plug#']

vim.call('plug#begin')
Plug('folke/trouble.nvim')
vim.call('plug#end')
require("trouble").setup()

I also encounter this issue when I mess with the windows while Trouble is open. Specifically if I close every window except the trouble window using <c-w>o, and then press enter to jump to the selected diagnostic.

If anyone else has the same problem, I worked around this by putting the following in my ~/.config/nvim/after/ftplugin/trouble.lua:
vim.keymap.set('n', ']q', 'j', { buffer = 0, noremap = true, silent = true })
vim.keymap.set('n', '[q', 'k', { buffer = 0, noremap = true, silent = true })

This is still a problem and shouldn't be closed imo.
Doing things with your quickfix list like :cdo %s/match/replace/g to search and replace spams so many notifications of Cursor position outside buffer that I have to close neovim or stop using notify.nvim.
Workaround
What worked for me was putting this in my init.lua:
vim.api.nvim_create_autocmd({"BufWinEnter", "WinEnter"}, {
    pattern = "*",
    callback = function()
        if vim.bo.filetype == "trouble" then
            vim.wo.winfixbuf = true
        end
    end
})

Note that simply setting vim.wo.winfixbuf = true in ftplugin/trouble.lua didn't work as it seemed to be overridden after that ran.


E/launcher - Unable to create new service: InternetExplorerDriverService

I am getting the below error

protractor outputjs/conf.js

[09:17:28] I/launcher - Running 1 instances of WebDriver
[09:17:28] I/local - Starting selenium standalone server...
[09:17:30] I/local - Selenium standalone server started at http://<IP_ADDRESS>:50504/wd/hub
[09:17:31] E/launcher - Unable to create new service: InternetExplorerDriverService
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
System info: host: 'DESKTOP-S5DALSR', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '1.8.0_201'
Driver info: driver.version: unknown
[09:17:31] E/launcher - SessionNotCreatedError: Unable to create new service: InternetExplorerDriverService
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
System info: host: 'DESKTOP-S5DALSR', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '1.8.0_201'
Driver info: driver.version: unknown
at Object.checkLegacyResponse (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\lib\error.js:546:15)
at parseHttpResponse (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\lib\http.js:509:13)
at doSend.then.response (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\lib\http.js:441:30)
at process._tickCallback (internal/process/next_tick.js:68:7)
From: Task: WebDriver.createSession()
at Function.createSession (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\lib\webdriver.js:769:24)
at createDriver (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\index.js:170:33)
at Builder.build (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\selenium-webdriver\index.js:635:14)
at Local.getNewDriver (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\protractor\built\driverProviders\driverProvider.js:53:33)
at Runner.createBrowser (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\protractor\built\runner.js:195:43)
at q.then.then (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\protractor\built\runner.js:339:29)
at _fulfilled (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\q\q.js:834:54)
at C:\Users\Praveen\Desktop\Protractor e2e\node_modules\q\q.js:863:30
at Promise.promise.promiseDispatch (C:\Users\Praveen\Desktop\Protractor e2e\node_modules\q\q.js:796:13)
at C:\Users\Praveen\Desktop\Protractor e2e\node_modules\q\q.js:604:44
[09:17:31] E/launcher - Process exited with error code 199
npm ERR! code ELIFECYCLE
npm ERR! errno 199
npm ERR<EMAIL_ADDRESS>test: protractor outputjs/conf.js
npm ERR! Exit status 199
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Praveen\AppData\Roaming\npm-cache_logs\2019-03-20T16_17_31_280Z-debug.log
conf.js
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.config = {
seleniumServerJar:'../node_modules/protractor/node_modules/webdriver-manager/selenium/selenium-server-standalone-3.141.59.jar',
chromeDriver:'../node_modules/protractor/node_modules/webdriver-manager/selenium/chromedriver_2.46.exe',
IEDriverServer: '../node_modules/protractor/node_modules/webdriver-manager/selenium/IEDriverServer.exe',
//directConnect: true,
//seleniumAddress: 'http://localhost:4444/wd/hub',
framework: 'jasmine',
capabilities: {
browserName: 'internet explorer'
},
specs: ['./specs/calculator.js'],
jasmineNodeOpts: {
    defaultTimeoutInterval: 30000
}
//not modified
//seleniumAddress: 'http://localhost:4444/wd/hub',
// You could set no globals to true to avoid jQuery '$' and protractor '$'
// collisions on the global namespace.

};

Started InternetExplorerDriver server (32-bit)
<IP_ADDRESS>
Listening on port 5555
Only local connections are allowed

You'll need to start up the selenium standalone server with the iedriver. In the current configuration, you can only start Chrome and Firefox (this is defined in the driverProvider local.ts)
I would start with webdriver-manager start --ie  and use seleniumAddress: 'http://localhost:4444/wd/hub'.
Closing this issue as resolved. If you still run into this issue, please ask support questions on  StackOverflow. For more information please reference https://github.com/angular/protractor/blob/master/CONTRIBUTING.md#questions
Thank you!

i want to configure it on jenkins where after executing webdriver-manager start --ie  , we are not unable to execute  protractor test case config.js file , because both need to run in separate  windows:
please provide proper solution .

i want to configure it on jenkins where after executing webdriver-manager start --ie , we are not unable to execute protractor test case config.js file , because both need to run in separate windows:
please provide proper solution .

You can use --detach key. It will run selenium server in background.


Match the way that undefined/numeric experiment column values are shown in the table in tooltips and quick picks

This PR changes the way that we display experiment column values that are undefined in the experiments view tooltip and experiments quick pick. We now match the way that undefined is displayed in the table. I have also removed the truncation marks from truncated values as I didn't feel they added much value to the user.
Main reason for doing this is that undefined means nothing to the average Python user.
LMK what you think.
Screenshots

LGTM ... I would change it in the future though. On tooltips, ... looks like we cut the text (it was too long or something), not that it's an undefined value.

LGTM ... I would change it in the future though. On tooltips, ... looks like we cut the text (it was too long or something), not that it's an undefined value.

@shcheklein I can change all of the '...' to '-' as per the CLI (table included):

WDYT?

@mattseddon yep, that look better to me (and consistent with CLI).


⚠ 366.ru has degraded performance

In fd6fbee, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1799 ms

Resolved: 366.ru performance has improved in 05b4cbf.


Option to load firestore key from dict

Original issue by alizahidraja on 2023-02-28 20:52:22+00:00
I am trying to deploy an app & I do not want to publicly show my firebase key
For this I use st.secrets to make a dict object, but there is no option to pass it to the track function

Comment by nathanjones4323 on 2023-03-08 19:47:59+00:00
@alizahidraja This PR I just created would resolve your issue.
If you want to use this or see this merged, give it a 👍

Comment by alizahidraja on 2023-03-08 19:53:19+00:00
Thankyou so much!

@alizahidraja  @nathanhones is this still relevant?

Hello
... for me it is :)


Update continuous-delivery-faqs.md

1 FAQ added
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ X] Feature

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65a4f80dcd15ad20f7b17d7c--harness-developer.netlify.app


Airhorn Solutions bot is Offline

I can't solve my problems which require airhorns. We have tried removing and adding him multiple times but it seems he is out of gas.

Are you running this on your local computer? From what I am getting, this needs to be hosted on a remote server (for 24/7 uptime and for a routed IP).

The bot worked normally by just inviting to our discord server via: https://airhorn.solutions/
Are you saying that at least 1 person needs to be hosting the bot on a server for it to work? Just trying to understand how this works.

I'm  not entirely sure either, the description is quite vague. From my point of understanding, yes, somebody needs to host the bot on their own server.

Interesting. It seemed to work with no server setup for a time. We just invited the bot and it worked. Maybe just 1 person in the world needs to host the bot? I'm unsure. I'll go through the server setup tonight and find out.

Nah, it's hosted already.  So you just invite it, and it should work.  However, it may be down sometimes :)  I suspect it'll get fixed eventually...  However, you can host it yourself if you want.

Bot is still offline, any updates on this?

Mine is offline too?
I really need help.
It says to run a command, but I have no idea how and where?


Target has transitive dependencies that include static binaries:

Installing Atlas (1.0.11)
Installing LayerKit (0.16.0)

and then I get
[!] The 'Pods-ProjectName' target has transitive dependencies that include static binaries: (/Users/viktor/Documents/ProjectName/Pods/LayerKit/LayerKit.embeddedframework/LayerKit.framework)

Using Xcode 7.0 and use_frameworks! enabled.

same problem.  Anyone have a solution please?

Freshly cloned repo fails during pod install:
Atlas-Messenger-iOS (master<>)]$ pod install
Updating local specs repositories
Analyzing dependencies
Downloading dependencies
Installing Atlas (1.0.11)
Installing ClusterPrePermissions (0.7.0)
Installing LayerKit (0.16.0)
Installing SVProgressHUD (1.1.3)
[!] The 'Pods' target has transitive dependencies that include static binaries: (......code/Atlas-Messenger-iOS/Pods/LayerKit/LayerKit.embeddedframework/LayerKit.framework)

Bump

Have you tried the new LayerKit version - 0.17 ?

Thanks @inna215
Half a workaround. Adding pod 'LayerKit', '= 0.17.0' to the Podfile allows Cocoapods 0.38.2 to finish. However the project does not compile in XCode 7.0.1

What is your error in XCode?  I just spend some time upgrading to XCode 7, finally got my project working.

Several symbols not found for x86_64:
Undefined symbols for architecture x86_64:
  "_LYRClientObjectChangesUserInfoKey", referenced from:
  "_LYRClientObjectsDidChangeNotification", referenced from:
  "_LYRConversationDidReceiveTypingIndicatorNotification", referenced from:
  "_LYRConversationOptionsDeliveryReceiptsEnabledKey", referenced from:
  "_LYRMessageOptionsPushNotificationConfigurationKey", referenced from:
  "_LYRTypingIndicatorParticipantUserInfoKey", referenced from:
  "_LYRTypingIndicatorValueUserInfoKey", referenced from:
  "_OBJC_CLASS_$_LYRCompoundPredicate", referenced from:
  "_OBJC_CLASS_$_LYRConversation", referenced from:
  "_OBJC_CLASS_$_LYRMessage", referenced from:
  "_OBJC_CLASS_$_LYRMessagePart", referenced from:
  "_OBJC_CLASS_$_LYRPredicate", referenced from:
  "_OBJC_CLASS_$_LYRPushNotificationConfiguration", referenced from:
  "_OBJC_CLASS_$_LYRQuery", referenced from:
ld: symbol(s) not found for architecture x86_64

my errors were a little different.  Are you using the new
use_frameworks!
tag in your Podfile?

Also having this problem, on a fresh project with
platform :ios, '9.0'
use_frameworks!
pod "LayerKit", "~> 0.17.0"
pod 'Atlas', '~> 1.0.13'

Undefined symbols for architecture x86_64:
  "_LYRClientObjectChangesUserInfoKey", referenced from:
      -[ATLConversationViewController layerClientObjectsDidChange:] in ATLConversationViewController.o
  "_LYRClientObjectsDidChangeNotification", referenced from:
      -[ATLConversationViewController atl_registerForNotifications] in ATLConversationViewController.o
  "_LYRConversationDidReceiveTypingIndicatorNotification", referenced from:
      -[ATLConversationViewController atl_registerForNotifications] in ATLConversationViewController.o
  "_LYRConversationOptionsDeliveryReceiptsEnabledKey", referenced from:
      -[ATLConversationViewController conversationWithParticipants:] in ATLConversationViewController.o
  "_LYRMessageOptionsPushNotificationConfigurationKey", referenced from:
      _ATLMessageForParts in ATLMessagingUtilities.o
  "_LYRTypingIndicatorParticipantUserInfoKey", referenced from:
      -[ATLConversationViewController didReceiveTypingIndicator:] in ATLConversationViewController.o
  "_LYRTypingIndicatorValueUserInfoKey", referenced from:
      -[ATLConversationViewController didReceiveTypingIndicator:] in ATLConversationViewController.o
  "_OBJC_CLASS_$_LYRCompoundPredicate", referenced from:
      objc-class-ref in ATLConversationViewController.o
  "_OBJC_CLASS_$_LYRConversation", referenced from:
      objc-class-ref in ATLConversationListViewController.o
      objc-class-ref in ATLConversationViewController.o
  "_OBJC_CLASS_$_LYRMessage", referenced from:
      objc-class-ref in ATLConversationViewController.o
  "_OBJC_CLASS_$_LYRMessagePart", referenced from:
      objc-class-ref in ATLMessagingUtilities.o
  "_OBJC_CLASS_$_LYRPredicate", referenced from:
      objc-class-ref in ATLConversationListViewController.o
      objc-class-ref in ATLConversationViewController.o
  "_OBJC_CLASS_$_LYRPushNotificationConfiguration", referenced from:
      objc-class-ref in ATLMessagingUtilities.o
  "_OBJC_CLASS_$_LYRQuery", referenced from:
      objc-class-ref in ATLConversationListViewController.o
      objc-class-ref in ATLConversationViewController.o
ld: symbol(s) not found for architecture x86_64

use_frameworks! is now fully supported with LayerKit v0.17.0

Still facing the same issue by @seeker89

Having same issue cocoapods 0.39.0, layerkit 0.17.0, atlas 1.0.13 with tag use_frameworks!


AddEntityFrameworkStores blocks using the generic IdentityRole directly

The check should allow using the generic type directly, instead of requiring a derived type.
AddIdentity<ApplicationUser, IdentityRole<int>> won't work
while
AddIdentity<ApplicationUser, ApplicationRole> where ApplicationRole : IdentityRole<int> works
@ajcvickers @blowdart this is hitting a lot of people, something we can consider fixing in the next patch as it should be a low risk fix.

@HaoK Yes, we should try to patch this. if you get fix done as a priority and I'll see if there is still time to get it approved for 2.0.1.

@HaoK Please add risk assessment either here on directly in the OneNote.

The fix should be low risk as it will just involve and additional check to allow the base generic type to be  used instead of only derived types.

On Aug 21, 2017, at 3:43 PM, Arthur Vickers<EMAIL_ADDRESS>wrote:
@HaoK Please add risk assessment either here on directly in the OneNote.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @HaoK. I updated the OneNote.


Fix a data race in b2BlockAllocator::b2BlockAllocator()

Fix for #527
This allows multiple threads to safely use their own (not shared)
b2World instances.
The data race is avoided by taking advantage of the thread-safety
guarantees provided by static local variable initialization in C++11:
https://en.cppreference.com/w/cpp/language/storage_duration#Static_local_variables

This would fix the data race but it would cause static initialization order problems.
If a user has a static or global object that creates bodies in in its constructor, then the lookup table could be used before its initialized.

@jhoffman0x : that's a good observation, although:

The proposed fix doesn't change the initialization order (it still happens inside the b2BlockAllocator's constructor, just in a thread safe manner). So it wouldn't introduce any new issues related to the initialization order.

b2BlockAllocator is b2World data member. In order to create any bodies, you need to instantiate a b2World first, which means that the piece relevant to this fix would run.

To sum up: yes, creating box2d objects from a dynamic initializer is a bad idea in general, but nothing related to the proposed fix. (a iffy part is the existing b2BlockAllocator::s_blockSizes, but except for a compiler flaw even that should be statically initialized)

Ah, yeah I see. I was looking at the diff on github and with the parts cut off around the changes I didn't notice that the static bool was now in the constructor.

Fixed in #583


Translations for more languages

The integration currently supports English and Croatian languages.
It should support at least couple of the popular ones - German, French, Italian, Spanish, Chinese.
It should be documented which languages are supported and how others can contribute more translations.

I can help with translation to Polish language

I can do the French one if you want

@szmidtpiotr If you'd like to contribute those Polish translations, that would be awesome! Since I don't have proper guidelines, feel free to look at @alexbelgium's PRs since I just merged his French translations.
Btw, thank you @alexbelgium for the contribution ❤

I can help with Spanish language.

@bananet please do, that would be awesome.
Check out @alexbelgium's PR where he added French translations and my comments above for guidelines.

Hello if you need it I can translate in Italian. Let me know how to do it, I need to do a pull request for this?

I attach the json for italian traslation.
I have a doubt if error and about objects need to be translated.
Now, I leave the default version of english. Let me know if they need to be translated.
I changed the file extention to txt beacuse I can't upload json file.
it.txt


some tests fail with type-inference disabled

These language tests fail if we disable type-inference:
language/assert_assignable_type_test
language/method_override4_test
language/classes_static_method_clash_test
language/cyclic_default_values_test
language/many_named_arguments_test

Since type inference is an integral part of a Dart 2 implementation, this doesn't seem relevant any more.

Thanks @askeksa-google  - I should clarify however that if you see bugs in dart2js mentioning 'type-inferenece' then it likely means something else: in dart2js we have a whole program type propagation analysis that we named type-inference long before Dart2 and user-type inference was added to the language.
I'm ok closing this bug though, just for different reasons :). We plan to add a bot to test things with many dart2js passes disabled, and we'll track status using the .status files, not individual issues.

Thanks for the clarification.


Support request emails accidentally remove underscores from package links

We got an email today which I imagine should have linked to
https://www.nuget.org/packages/JQueryFileUpload_Demo_with_Backload/
but it actually linked to
http://www.nuget.org/packages/JQueryFileUploadDemowithBackload/
and
http://www.nuget.org/packages/JQueryFileUploadDemowith_Backload/<IP_ADDRESS>
which didn't work

This still happens.


How to use with CMake on MacOS

On MacBook Air M1 trying:
cmake .. -DCMAKE_TOOLCHAIN_FILE=/Users/user/vcpkg/scripts/buildsystems/vcpkg.cmake -G "Unix Makefiles"

but I get this error:
Unable to determine target architecture, continuing without vcpkg

I tried adding also:
-DVCPKG_DEFAULT_TRIPLET=arm64-osx

but with no luck

I suggest you can not use the -G option, the default generator is makefile.

not using -G option does not change anything

@strega-nil Can you please take a look?
Thanks.

hrm. @cdcseacave could you post the result of:
message(STATUS "${CMAKE_HOST_SYSTEM_PROCESSOR}")

the result is:
arm64

which is correct.
I found a way to make it work with a hack to vcpkg.cmake by adding around the if at line 341 if(VCPKG_TARGET_TRIPLET) these lines:
set(CPY_CMAKE_GENERATOR_PLATFORM ${CMAKE_GENERATOR_PLATFORM})
set(CMAKE_GENERATOR_PLATFORM "arm64")
..................
set(CMAKE_GENERATOR_PLATFORM ${CPY_CMAKE_GENERATOR_PLATFORM})

vcpkg.zip

I suddenly remembered that MacBook recently switched from x64 to arm64 platform.

Just a note: MacBook  did not switched entirely to arm64, there are still x64 MacBooks produced
Additionally both x64 and arm64 can be compiled on both versions of native platforms

vcpkg has now added confirmation for arm64 architecture by PR #18112

Thanks for posting this issue. Please reopen this issue if it still be a problem for you.

I've issue similar to this using GitHub Actions, see the log https://github.com/jayces-lang/jayces/actions/runs/7321648566/job/19942284291#step:6:15. I've the same settings on all platforms (Windows, Linux, macOS), but only the macOS failed to generate the build project. Any idea? 🤔

I've issue similar to this using GitHub Actions, see the log https://github.com/jayces-lang/jayces/actions/runs/7321648566/job/19942284291#step:6:15. I've the same settings on all platforms (Windows, Linux, macOS), but only the macOS failed to generate the build project. Any idea? 🤔

cmake -S . -B build/ninja/ -GNinja doesn't contains vcpkg toolchain.

Thank you for the response! 👍 The solution is in https://github.com/microsoft/vcpkg/discussions/35869.


Flat shading with light reflection strength

Hi, first let me say that I grew up with the PSX so this project is really awesome and I am actually using your shaders in my game in development currently. I hope more people start using these because I love the aesthetic created by the PSX's technical limitations
I noticed that you started adding some more features in the past days to make this suite even more complete and while playing Spyro recently, I noticed a cool effect that I think would be a great effect.
Watch the spinning talisman at 11:03 in this video https://www.youtube.com/watch?v=vpjrIzTXdo8
Notice that its golden surface appears highly reflective by using normals to have certain faces almost completely black while faces coming into view are almost completely white. I think adding some sort of reflection strength and whatnot could be a nice addition.

Hi, sorry for the late reply.
I have edited the reflective shader a little to obtain something similar to the talisman surface.

The edited shader is here:
psx-unlit-simple.txt
You have to use a cubemap like this (maybe done better ;P)

Very nice! I'll have to find some uses for the cubemap shaders, could spice it up
here is a picture of my game!


Contradictory messages when deleting branch

Description
When I delete a branch on the Code -> Branch tab, a message pops up that this delete is permanent and cannot be redone.
After confirming I see the image below where I can restore the branch...

Whether or not this might work it would be better to have a more consistent wording.
By the way thanks for this great software!
Gitea Version
1.18.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Ubuntu 18.04
How are you running Gitea?
Manual server installation with systemd
Database
MySQL

It can be reverted in a very short time.

According to discussion at Crowdin (in German) for repo:branch.delete_desc there is a proposal to change it into:
"Deleting a branch is permanent after 'git cleanup task' was run (by default once per day). It CANNOT be undone. Continue?"
Is options/locale/locale_en-US.ini the right place to create a pull request?

Per https://github.com/go-gitea/gitea/pull/22696#issuecomment-1521036199


JB-589 - Cycle data discrepancy

What does this PR do and why?
Closes https://linear.app/peel/issue/JB-589/cycle-data-discrepancy
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3830

PR #3832

PR #3835   👈

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


Unattended configuration

I'm trying to create support for Plexdrive for my "Hosting as a Service" platform by turning it into a 1-click installer application.
To achieve the best UX experience it would be great if all the configuration could be handled by my service and I just supply the authorization code via a config option/file.

Would writing the auth code to config be the best way to achieve an unattended configuration?
If so would you be open to accepting a PR that implements such a system?

Well basically everything is stored in the configuration itself. The database is only for storing the state.
I would suggest to write an own authenticator that generates the config file. The next step is to just start plexdrive with the generated configuration and everything should work.


✨ Tabular classification task

Fixes #15

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

nice!

Note: original wine model won't load - https://huggingface.co/julien-c/wine-quality, had to fork and fix it


Empty English gloss rows in value table

Given that the 'English gloss' column in https://concepticon.clld.org/values is only filled with entries from GLOSS in the original concept list tables, there are a lot of 'empties' in there and original columns like ENGLISH are not used.
While not strictly speaking problematic, this in my opinion diminishes the 'look and feel' of the values table and it feels empty. Could additional columns (ENGLISH etc.?) be considered if GLOSS is not presented? Could the appearance of the English gloss column be otherwise improved?

So you know the rule: gloss is used to mark a translation if English is not there. Theoretically, we do not even need this, as we are reaching a level where we can link to concepticon without translating every entry, and it may sometimes even be easier and probably also better. We could probably

treat the English Gloss just as an elicitation gloss in any language (maybe "English translation")
not display it
use English in first column (gloss and translation) and add a column with the other languages
decide on some three to four major languages to be displayed (may also look sparse)

If we treat English gloss just as an elicitation gloss translation, this may be easiest.

Thanks! I agree. Anything to make the column useful/improve the layout. :)

I agree that the "English gloss" column at https://concepticon.clld.org/values does not provide anything of value (since its content will also be available via "Description in source"). So I'd say we drop this.
And as @LinguList says, we have now a lot more coverage of glosses in languages other than English. Thus, I'd propose to do what I planned on doing for some time: promoting individual glosses to first-class clld objects, namely to Units. This would also mean we add each language that we have glosses for - which may at some point become interesting enough for its own page.

Sounds good, and it could even allow us to move to a stage where we ignore the English translation, as it may at times only support the bias we already have to fight with with English as major glossing language. It would mean, we can map a list in Chinese to concepticon, without providing translations. And we might consider that we simply distinguish for a glossing language in Concepticon if it is our translation by the concepticon group, of if it is the translation in the source. But that's something to be discussed later. For now, it may be useful to mechanically distinguish the ENGLISH from the GLOSS by writing ENGLISH vs. ENGLISH translation ?

I was going to just use a "MetaLanguage" as the thing that relates concepts with concept sets. If we provided an english translation, this would be the gloss in the MetaLanguage. If an english gloss was provided by the source, this would become a regular "Gloss". The labels in the MetaLanguage would actually never show in the user interface, because they can be replaced by CONCEPTICON_GLOSS of the concept sets.


Automated cherry pick of #8174

Cherry pick of #8174 on release-2.19.
/cc  @enahum
NONE

/update-branch

Leaving open for Elias to help take a look at the failing tests.

Leaving open for Elias to help take a look at the failing tests.

Failing tests are calls related and not cause of this PR.


Deploy BOSH with Terraform startup-script failed

I followed the steps here and used Terraform to deploy the infrastructure, but the whole startup script failed. The following was in /var/log/startupscript.log:
Running startup script /var/run/google.startup.script
/usr/share/google/run-scripts: /tmp/tmp.SA6GU6Llif: /bin/bash^M: bad interpreter: No such file or directory
Finished running startup script /var/run/google.startup.script

It seems like this is to do with line endings, but I can't seem to change the file format using sudo with vim or dos2unix.
Would adding the following to .gitattributes make a difference?
docs/*.tf eol=lf

Changed docs\bosh\main.tf to Unix line endings and then used Terraform to recreate the VM and it worked.


Proxy not fast enough?

As the title! I have like 1k of socks5 proxies with less than 1 second timeout and still the first bot joining is fast but then it gets slow! I've been using another bot called deathbot and they are just connecting instant and not slower after 3-4 connections, is there a way on how to fix this?

I don't think your proxy/connection speed is an MCProtocolLib issue.

I can use a slow proxy with the normal mc client why cant i use it with MCProtocolLib

You can. If it's a slow proxy though, the connection will obviously also be slow.


adding CORS support

to allow support for cross origin requests

There is some work ongoing by @djalova  to bake in CORS support in the MAX Python framework, which should eliminate the need for this PR.

#2 should address the requirement for CORS support via the base image and maxfw


[std.algorithm] Split find docs into two

It's much clearer to have separate docs for the overload not taking needle, because its pred has a different signature and its behaviour is simpler. It was necessary to move that overload (including unittests) from between the other two. No changes were made to the overload body, or unittests except the bool pred( line which was tweaked.
Also improve the docs for the needle overloads:
Remove duplicate BIGOH sentence already present under Complexity.
Fix wrong input name.
Move "a != b" predicate tip to a note so it stands out more.
Use e, n for predicate parameter names in examples, short for element, needle.

Ping


Problem: the CI integration tests take too long

We need to either speed up the tests, or make a tiered testing system, or both. Should target 5m duration.

https://buildkite.com/granola/mina-indexer/builds/530 6min 28sec we're getting close. The build times might take too long to get under 5min.

Close enough for now!


Add error handling for passing an invalid type in serversetup

TypeError: path must be a string or Buffer
at TypeError (native)
at Object.fs.readdirSync (fs.js:953:18)
at setup (/home/rof/src/github.com/ansble/monument/routes/serverSetup.js:9:565)
at Proxy.assertThrows (/home/rof/src/github.com/ansble/monument/node_modules/chai/lib/chai/core/assertions.js:2554:7)
at Proxy.methodWrapper (/home/rof/src/github.com/ansble/monument/node_modules/chai/lib/chai/utils/addMethod.js:57:25)
at Function.assert.throws (/home/rof/src/github.com/ansble/monument/node_modules/chai/lib/chai/interface/assert.js:1973:16)
at Context.it (/home/rof/src/github.com/ansble/monument/routes/serverSetup_test.js:9:661)
at callFn (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runnable.js:354:21)
at Test.Runnable.run (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runnable.js:346:7)
at Runner.runTest (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:442:10)
at /home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:560:12
at next (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:356:14)
at /home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:366:7
at next (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:290:14)
at Immediate.<anonymous> (/home/rof/src/github.com/ansble/monument/node_modules/mocha/lib/runner.js:334:5)
at runCallback (timers.js:672:20)
at tryOnImmediate (timers.js:645:5)
at processImmediate [as _immediateCallback] (timers.js:617:5) undefined

Replace the generic node error handling with a type check and a thrown error with a better description of what is going on and what needs to be done to correct it.

Hi.  I would like to take a crack at fixing this issue.  Is that okay?

Absolutely

On Oct 4, 2017, at 8:23 PM, ignitionSoldier<EMAIL_ADDRESS>wrote:
Hi. I would like to take a crack at fixing this issue. Is that okay?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Looks like #443 corrected this error as well


Corrected invalid links to bookmarks

Docs Build status updates of commit 17312e9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mapping/az-groupMapping-7.5.0.json
:white_check_mark:Succeeded

mapping/az-groupMapping-8.1.0.json
:white_check_mark:Succeeded

mapping/az-groupMapping-8.2.0.json
:white_check_mark:Succeeded

mapping/rm-groupMapping-2.5.0.json
:white_check_mark:Succeeded

mapping/sm-groupMapping-4.0.0.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


backtick ` to escape Latex not working

The use of backtick to escape Latex code is not working in markdown
Code to replicate
-   enclosed in `$ $`  such as  `$\sigma$` get $\sigma$

Current output

Expected output

This is not a bug, it's an feature ;-)
Have a look here: https://stackoverflow.com/questions/62111699/mathjax-strangely-render-back-tick-in-code-blocks

I read it but I still cannot figure out how to comment out $\sigma$

Its not possible since backticks are reserved for math operations what you could do is to escape the dollar symbol
inline \\$\sigma\\$ latex with ==\\$\sigma\\$==

Thanks @MSzturc !


Explore the use of the 'official' custom element polyfill

https://github.com/webcomponents/custom-elements
This should remove the need to bundle the polyfil

This is likely redundant, now that modern browsers largely support custom elements. Note that Safari does support custom elements in the way they are used in this clapper component.


Update formatting in puzzle descriptions

Puzzle descriptions - would be nice if italics/etc. showed up analogously to the website version (example - see last line in attached). Tested on 1.4.1 on iPhone 8.

On the website we run the descriptions through v-dompurify-html to ensure it's sanitized while still maintaining formatting


Fix bug in numpy broadcasting

Broadcasting with dim 1 matching dim None previously resulted in the output shape having None inside of it. This PR fixes that.

Looks good, waiting for build to finish.

Could take a while again :firetravis:


Dockerfile and docker compose

Provide a Dockerfile and Compose file which lets you run codecv as a container.

Will be completed by #28


To do JOSS

Other

[ ] Add the joss draft to the repository.
[ ] Apply the steps in PyOpenSci  to biosspheres.
[ ] Check this link.

Review checklist
Conflict of interest

[x] I confirm that I have read the JOSS conflict of interest (COI) policy and that: I have no COIs with reviewing this work or that any perceived COIs have been waived by JOSS for the purpose of this review.

Code of Conduct

[x] I confirm that I read and will adhere to the JOSS code of conduct.

General checks

[x] Repository: Is the source code for this software available at the https://github.com//iamartineza/biosspheres?
[x] License: Does the repository contain a plain-text LICENSE or COPYING file with the contents of an OSI approved software license?
[x] Contribution and authorship: Has the submitting author (@iamartineza) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?
[x] Substantial scholarly effort: Does this submission meet the scope eligibility described in the JOSS guidelines
[x] Data sharing: If the paper contains original data, data are accessible to the reviewers. If the paper contains no original data, please check this item.
[ ] Reproducibility: If the paper contains original results, results are entirely reproducible by reviewers. If the paper contains no original results, please check this item.
[x] Human and animal research: If the paper contains original data research on humans subjects or animals, does it comply with JOSS's human participants research policy and/or animal research policy? If the paper contains no such data, please check this item.

Functionality

[ ] Installation: Does installation proceed as outlined in the documentation?
[ ] Functionality: Have the functional claims of the software been confirmed?
[ ] Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

[ ] A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
[ ] Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
[ ] Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
[ ] Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
[ ] Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
[ ] Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Software paper

[ ] Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
[ ] A statement of need: Does the paper have a section titled 'Statement of need' that clearly states what problems the software is designed to solve, who the target audience is, and its relation to other work?
[ ] State of the field: Do the authors describe how this software compares to other commonly-used packages?
[ ] Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
[ ] References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?

Final step: FAIR principles

[ ] Follow the fair-software.eu recommendations such as in [this repository]. (https://github.com/GrainLearning/grainLearning). You can also check this link.
[ ] Checklist
[ ] Reproducible docker environment.
[ ] Documented reproducibility

I added the JOSS draft in this folder. At this moment it is mostly empty, but the basics of the how to are there.

I added:

A description of the project.
a Code of conduct
the Contributing guide lines

I updated the README a little too.
I am working on having the package installed via pip. It is still work in progress. I am also working on the necessary tests.


Desktop run doesnt work?

I can run the app on my android device and on my desktop if i run from the command line using gradlew desktop:run but when i set up the run configuration i get the error message: " Could not create service of type BuildSessionScopeServices$CrossBuildFileHashCacheWrapper using BuildSessionScopeServices.createCrossBuildChecksumCache()."

Hi macleodiain, how can you solved the problem?

Hi macleodiain, how can you solved the problem?


Fix a typo in createNavigator

removed X (or was it there on purpose?)

hi, thanks for the PR! Yes, the X is there on purpose - createXNavigator refers to these: createDrawerNavigator, createBottomTabNavigator and etc. Would you find create*Navigator more understandable?

Yes, I think that would be more understandable. I actually realized after creating the PR the X was there to refer to the type of Navigator, but I think an asterisk would me more suitable (since it actually is a wildcard in a way). The X made me read it in a way as if it was a specific Navigator. I think the asterisk would be a nice alteration!

@Luukschoen could you change createXNavigator to create*Navigator? I'll merge the change then. Thanks.

Did the update. Thank you @vonovak !

thanks!


[NEW]: Radiant hack

Date
2024-01-02
Description
Radiant (RDNT) hacked and drained! 1,900 ETH gone.
Links
https://fxtwitter.com/officer_cia/status/1742317178341228814
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
Artemia
Personal link
No response

Thank you! https://web3isgoinggreat.com/?id=radiant-capital-hack


refactor: move app.conf handling to a separate module

Codecov Report

Merging #290 (833b071) into main (91f4652) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #290      +/-   ##
==========================================
+ Coverage   58.81%   58.83%   +0.01%
==========================================
  Files          37       38       +1
  Lines        2472     2463       -9
==========================================
- Hits         1454     1449       -5
+ Misses       1018     1014       -4

Impacted Files
Coverage Δ

splunk_add_on_ucc_framework/__init__.py
70.23% <100.00%> (-1.61%)
:arrow_down:

splunk_add_on_ucc_framework/app_conf.py
100.00% <100.00%> (ø)

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 91f4652...833b071. Read the comment docs.

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

v5.5.8
GitHub release

Your semantic-release bot :package::rocket:


Fixing a bug in upload of binaries to the release

Fixing a Go build error when creating multi-arch binaries for upload into a release.


[T6A3][T13-C3] Annabel Eng Jing Wen

Attempted:
[LO-DIP]
[LO-DI]

@zhongwei-z Thanks for the comments! Had read through them and will take note (:


Evaluation of arrow functions in expressions in v6

Description
When working with arrow functions, the new expressions in v6 seem to work substantially differently than the previous expressionProperties of v5. The problem occurs when a variable is changed by code that does not directly affect the form but should indirectly affect it as an expression would evaluate differently.
This change is a problem to my project as we heavily made use of dynamic expressions using arrow functions.
BehaviourSubjects seem to work fine, but that would mean a lot of refactoring effort. At this state we cannot upgrade to v6.
Minimal Reproduction
'props.disabled': () => this.inputIsDisabled does not seem to register changes made to the variable from outside the form,
This used to work in v5.
I prepared two stackblitz demos using the expressions examples found for Formly v5 and v6.
Formly v5: https://angular-yhetv7.stackblitz.io
Formly v6 (buggy): https://angular-yobrug-6fniuc.stackblitz.io
Clicking the 'switch' button causes the disabled state to always correctly switch in v5.
In v6 it only always correctly works when using the observable. Using arrow functions it seems to only work once and afterwards only reevaluates when changes are made to the model in another field.
Also, using the in v6 marked as @deprecated expressionProperties and templateOptions produces the same outcome.
Your Environment
My project:

Angular version: ^17.0.0
Formly version: ^6.2.2

Demo v5:

Angular version: 7.2.16
Formly version: 5.12.7

Demo v6:

Angular version: ^14.0.0
Formly version: ^6.0.0

Additional context
None.

I think it related to https://formly.dev/docs/guide/migration#4-checkexpressionon-is-set-to-modelchange-by-default , the better option is to pass an observable instead.

Thank you for the quick response.
Definitely my bad, I failed to notice this in the migration guide!
This fixed it in the provided demos, but in my project, adding a call of changeDetectorRef.detectChanges(); after updating the respective property was also necessary to have the arrow functions working again.
I agree, observables would be the better solution, but the time it takes to refactor legacy code can be too much at some point.


Update the facades to provide functions only

We currently have 3 mixed ways to provide features on out facade services:

custom function, i.e. getXyz
getter, i.e. get xyz()
pubic exposed property

To provided consistency to our Api, we could only expose functions through our facade API.
The facade services are all the services in a .../facade/ folder.
List of pros:

More consistent api between features of a same service facade and between different service facades.
Seems that functions are easier to spy on and apply mock behaviours.

List of cons:
This could become a spike if we need to try things out to verify assumptions.

need final decision from Tobias, as go was given but then Stan made some new points.

Tickets to be created for facade refactor:

refactor public accessors into functions
(optionally) rename functions to be short and into context (i.e. productService.get() rather then productService.getProduct()). Avoid redundancy. Additionally we are allowed to not mirror the commerce platform.

Separate tickets

we've refactors productGuards, but that's a side topic.

Stan to finish and merge, probably by Friday
Stan to create a checklist for gladiator devs to follow when doing the tickets

How to add generic loading flags to an existing chunk of state:

Wrap default reducer with loaderReducer, for example:
loaderReducer('UNIQIUE_ENTITY_ID')
Use LoaderState to get type safety with wrapped state
Extend from LoaderActions to set required metadata for actions
Use selectors functions to get wrapped state (loaderValueSelector) or specific flags
(loaderLoadingSelector, loaderLoadingSelector, loaderSuccessSelector)

Checklist for entityLoaderReducers would be exactly the same,
the difference is that it wraps LoadReducers (above) with EntityReducer,
to support loading flags for multiple entities (like products or components)

Testing of this work will be done with the facade regression test ticket, #6


update the main dxos.org website

Motivation
The website needs to be a more welcoming, encompassing introduction to individuals finding out about DXOS. With Strange Loop incoming, we expect an increase in visitors.
Finish Line
The website needs to answer the following questions:

What is it, in meaningful conceptual anchors/buzz words/memes? (relate it to other things they know)
Does it actually work? (show it working)
What's the code look like? (show code snippets)
How can I play with it? (Interactive code sample, embedded or codesandbox)
How does it really work? (brief, technical explanation of the features, link to docs for more detail)

Scope
We'll scope this to one week (4 days) to get a draft of the website.
Out of scope:

altering the brand

Re https://deploy-preview-213--qa-dxos-web.netlify.app first draft:

The site as presented makes us look like Composer is the main product -- not ECHO. I think the first section should be more like https://redis.io showing a code snippet and how simple it is to create a collaborative app. This could be embedded or video + code.

I don't think we should call out the name Composer in a heading. I think structurally having a prominent section for examples, that showcase the different plugins is right. I think it would be better to flush out this section instead of the separate "Example Apps" section at the bottom (which should just be in the API/docs).

visually i don't think the video "insert" looks good on the geometry field. it might be better in a full width solid color section.

@richburdon and I just spoke about website priorities and boiled it down:
Website Revamp

Big Top Header can just be text for now (no video, no image). We'll bump the font size up.
Next section: 3 side-by-side collaborative demos in (roughly) the style of the videos I've already recorded.

Stacks: live documents being edited on both sides, re-ordering the docs, and multiple media types
Tables: not sure what the demo is yet.
Kanban: not sure what the demo is yet.

Existing "Platform + 6-up" section.

Done! For now, we'll cut the code example, but if we have time we'll bring back a single larger code sample showing off ECHO reading and writing data.
Plugins Quick Start Docs page #3938 (copying the below to this issue)

Prose description of how to quickly get started building a plugin
Short description of Composer
Video of composer in action
Link to composer / labs

It's getting close to ready.
A few known things I'd like to fix:

"Check out these nifty examples of things you can build on top of the DXOS Platform." -> rewrite this copy
Include the CTA in the header, top right
Add two more demo recordings


Remove jest-mock-extended unused reference

Describe this PR
It removes the a usage reference for jest-mock-extended on this page
Changes
Only removes the reference.
What issue does this fix?
Fixes #3135
Any other relevant information
--

Thanks @isaacbatst . When you say that the unit testing guide) doesn't use jest-mock-extended I am not sure what you mean. Do you mean that one can skip the "Mocking the Prisma client"section, and connect to a real DB, and thus not use jest?

Hey @tanberry! Actually, the unit testing guide mentions and uses the jest-mock-extended as it is meant to be.
For the other side, the integration testing guide does not uses the mock, since it is a test that is meant to reach a testing database.
This integration guide has a warning suggesting the lib is been used, but it is not. So, I think this warning should be removed.

Hey @tanberry  just a ping here 🏓

👋
Thanks for the contribution. The change makes sense. I'll go ahead an merge it. 🙂


Where are Credentials Stored?

Hello,
I'm curious. I'd like to know where are login and password stored on MacOS? I visited folder ~/.config/rclone, but it's not there. I wonder where it is then…
Also, how can I no longer input login nor password; I just want to have free access.
Thanks for your gracious help.

Hey, if you don't provide a username and password while running the gui, the default username is "gui" and the password is randomly generated each time.
You can see the password with the -vv (debug logging) mode.
Also, you could provide your own username and password using the --rc-user and --rc-pass parameters to the command line.


Put tests into folders in src/client | server

We should couple context ("users" / "todos") not function ("showing" / "testing" / "controlling" / "storing"). Considering that remove /src/test and create tests folders in matching folders in client / server directory.
Eg.: https://github.com/este/este/blob/master/src/test/client/todos/actions.spec.js
Should be in
https://github.com/este/este/tree/feature/translations/src/client/todos /__test__/actions.spec.js
Exact name of __test__ dir is up to discuss. This is the way, that facebook name it (check https://github.com/graphql/graphql-js)

Worth taking a look at my current i18n approach were I addressed similar problem.

I decided to go with tests (plural), since it was already ommited in gulp file in bulding tasks.
https://github.com/mirimCZ/este/commit/3d5a81722eda072a0a3b263a4227b0fe10aef188
There is two problems:

I still dont like that I need to have src/test. When I tried to define specs in karma file, I get errors of using reserved word on first line, that would be "import". Gues its problem with ES6 and was expected, that's why src/test/index.js "hack".

My bambi solution for this may be building specs into own folder, and running karma there. It migh work, but IDK, since I'm still bambi to gulp and building. (Bambi = noob)

There was .eslint file in each dir (src/test/client | server), wouldn't we need to copy it into every tests folder? I would realy hate that to happen :(

Can you submit a PR to the repo and we can then move discussion over there?

https://github.com/este/este/pull/334
Like that? :)

@mirimCZ Can you pls send a new PR, where tests are side by side to files? Like getrandomstring.test.js or something like that?

@grabbou WDYT?

I am using the above convention in my native app and it works well. (maybe because I am using jest, but all in all, I like to keep my tests a part of the code).


Feature/food review sentiment multilabel

sadedegel/prebuilt/README.md:

Added how to use alternative prebuilt pipeline
Added how to use evaluation
Added holdout test set F-1 macro score

sadedegel/prebuilt/model/food_sentiment_multilabel.joblib:

Dumped pretrained/optimized model weights for multilabel version

sadedegel/prebuilt/food_reviews_multilabel.py:

Created functions according to CONTRIBUTING.md
Implemented build, load and evaluate functions
Added optional argument to check roc_auc score for predict_proba method
Best model and vectorizer parameters are selected using optuna

tests/prebuilt/context.py:

Added import line for the base model

tests/prebuilt/test_food_reviews_multilabel.py:

Added assertions for testing purposes.

sadedegel/dataset/food_review/*

Created a dataset imports by @dafajon based on the data I mentioned on Issue #290
Fixed minor dataset test case error

@ertugrul-dmr Can you please update datasets/README.md with information on data source and the way you discretized the continious labels into class values.

Done @dafajon

Thanks. Merging and closing.


[Feature Request] Position of bones?

I wonder if it possible to adjust position like how we can do so in Ktisis/Anam position of bones and to be persist in Custmoize+

No. Not possible sadly.

Would be nice to have some sort of relative-to-waist/leg/etc position changes for the hip/cloth bones.
I use an extreme example here, but the poofy bits on outfits don't like it when you scale the waist too far since they won't follow the new scaling.
In anamnesis I can just shift bones over manually to get a good pic, but with an all-the-time scaling it can look a bit iffy.

Any kind of position scaling/movement is impossible for customize+ as positioning changes will automatically break all animations. There is no way to implement this.


update swift version to 3.x

modify copyright to "jspatch". (I'm the swift demo's original author, whom is called ArlenKou).
update swift version to 3.x.

commit message 要用英文哈

是 commit message 而不是PR的title，重提一个？


watchmate: 0.4.5 -> 0.5.2-unstable-2024-08-13

Description of changes
Update watchmate, includes fix for Rust 1.80.0 breakage
Related #332957
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

hmm, I should've split the formatting in a different commit I guess

Yeah, please do. In the same commit it's quite hard to review.


Option to reverse comment replies' order

Currently when you click to view replies to a comment, the last one is the first reply (chronologically). Would it be possible to make an option to reverse the order, so that when you view the replies, the first one you see is the first one that was published? I find this app much better for browsing comments than youtube itself, so I thought it wouldn't hurt to ask 😀

Fixed now. i also added a check in case the API folks reverse the order again; it should still display in time-ascending order. 🤞
Technical details for those interested:
Receiving replies from the API in correct order now reveals a possible optimization. Previously, we had to fetch all of a comment's replies just to get to the first one, since they arrived in reverse order. For a comment with the maximum 500 replies, this took 5 API requests (about 2.5 seconds). But now that the first batch of 100 replies includes the first reply, we could just immediately display the first 100 (cutting the delay down to ~0.6 seconds). Sounds great, but this also necessitates a bunch of extra HTML and buttons to paginate the replies, and maybe I would've attempted it back in 2020 when I worked on this all day but now a 2.5 second delay is nothing to fret about. Plus, displaying all 500 replies in one click is something you can't do on the watch page 🙃


Runtime Comparison

Introduction

Test: 100 random 50-CZ gate on 20 qubits.
Qibo SABRE is 10x slower than Qiskit Python SABRE.

# output
qibo Trivial + SABRE   avg: 0.40797 std: 0.09014
qiskit Trivial + SABRE avg: 0.04401 std: 0.00765

Updated some logics and improved the performance.

Implementation
1. Qubit Mapping
Previous Approach

Used three mapping data structures.

self._graph_qubits_names = [int(key[1:]) for key in self.initial_layout.keys()]
self._circuit_logical = list(range(len(self.initial_layout)))
self._physical_logical = list(self.initial_layout.values())

self._circuit_logical: Represents the virtual circuit layout starting from [0, 1, 2, ... ].
self._physical_logical: Represents the logical qubit numbers, where _physical_logical[i] is mapped to the physical qubit i.
_graph_qubits_names: Stores the physical qubit name for physical qubit i.

Problem

Two data structures are used to handle only physical to logical (p2l) mapping.
The values of self._physical_logical are not changed within the code.
There is no direct logical to physical (l2p) mapping.
The current code performs l2p mapping using list.index(i), which increases runtime due to list iteration every time.

Updated Approach

Combine circuit_logical and physical_logical into a single structure _p2l.
Create _l2p to efficiently handle l2p mapping.
Save the qubit names of the ith physical qubit as _p_qubit_names.

self._p_qubit_names = list(self.initial_layout.keys())
self._p2l = list(self.initial_layout.values())
self._l2p = [0] * len(self._p2l)
for i, l in enumerate(self._p2l):
    self._l2p[l] = i

This bidirectional mapping enables both l2p and p2l conversion in $O(1)$.

Example
let
initial_layout = {"q0": 3, "q1":1, "q2":0, "q3":2}

then
_p2l = [3, 1, 0, 2]    # physical qubit number i matches logical qubit number _p2l[i]
_l2p = [2, 1, 3, 0]    # logical qubit number i matches physical qubit number _l2p[i]
_p_qubit_names = ["q0", "q1", "q2", "q3"]  # physical qubit number i has name _p_qubit_names[i]

2. Temporary Circuit
Previous Approach

When computing the cost of a candidate swap, the current code creates a temporary CircuitMap which includes all the information of the circuit.

temporary_circuit = CircuitMap(
            initial_layout=self.circuit.initial_layout,
            circuit=Circuit(len(self.circuit.initial_layout)),
            blocks=self.circuit.circuit_blocks,
        )

Problem

Creating a temporary CircuitMap involves unnecessary operations like block decomposition, circuit object creation, and gate copying, which are not required for calculating the cost of a candidate swap.

Updated Approach

Introduced a _temporary flag.

        self._temporary = temp
        if self._temporary:     # if temporary circuit, no need to store the blocks
            return

        ...
        ### add the real SWAP gate, not a temporary circuit
        if not self._temporary:
            self._routed_blocks.add_block(
                Block(qubits=physical_swap, gates=[gates.SWAP(*physical_swap)])
            )
            self._swaps += 1

For temporary CircuitMap instances used only for calculating swap costs, return immediately from init() after initializing the necessary data structures.
For these temporary CircuitMap instances, prevent gate additions or removals in the update() method.

3. Qubits of a Block in the front_layer
Previous Approach

In the front_layer, only gate (block) numbers are stored.

def _get_dag_layer(self, n_layer):
        """Return the :math:`n`-topological layer of the dag."""
        # node[0] is the gate number
        return [node[0] for node in self._dag.nodes(data="layer") if node[1] == n_layer]

Problem

To find the qubit numbers associated with a gate, the code must iterate through the list of gates using search_by_index(block), which increases runtime.

Updated Approach

Store both the gate number and the qubits on which the gates are acting in the front_layer.
This allows for retrieving the qubits in $O(1)$ time.

### additionally save the qubit pairs that the gates are acting on
    for i in range(len(gates_qubits_pairs)):
        dag.nodes[i]["qubits"] = gates_qubits_pairs[i]

4. Additional

Use shallow copy when copying an integer list.

Evaluation

Test: 100 random 50-CZ gate on 20 qubits.
Qibo SABRE becomes 6x faster.
Qibo SABRE is 1.5x slower than Qiskit Python SABRE

qibo Trivial + SABRE   avg: 0.06699 std: 0.0163
qiskit Trivial + SABRE avg: 0.04346 std: 0.00588


add deobfuscator

please add deobfusactor (allatori,zelix,proguard)

Won't work because the Obfuscators change their methods with each update. :)


fix(sshd): Validation error

The validation command fails when no user is specified. With the following error message:
'Match Group' in configuration but 'user' not in connection test specification.

This is similar to the issue described here:
https://github.com/dev-sec/ansible-ssh-hardening/issues/188
This was tested on Debian Buster (Stable) with OpenSSH 7.9p1

I also corrected the variables in the README. I spent waaay to much trying trying to figure out why the variables I passed weren't being respected. Now I know why :P

@hurricanehrndz Just ran into this issue again. Could you kindly consider merging the changes?

@MiniCodeMonkey
Thank you so much for the pull request. Do not remember seeing this back in April, probably due to lockdown.


Support for additional_special_tokens

Description
This lets users specify "additional_special_tokens" of key of the add_special_tokens method.
Motivation and Context
Special tokens are treated different by the tokenizer, ensuring that they are never broken down. This is important for some use cases, like ChatML.
Consider Yi-200K, which has <|im_start|> and <|im_end|> in its vocabulary already. They are, however not marked as special in the base variant. This means that:
from transformers import AutoTokenizer, AutoModelForCausalLM

tokenizer = AutoTokenizer.from_pretrained(
    "01-ai/Yi-34B-200K"
)

tokens = tokenizer("<|im_start|>system\nX<|im_end|>")

Results in:
{'input_ids': [59666, 59705, 622, 59593, 5858, 46826, 10707, 144, 59733,

However, adding these tokens as special:
tokenizer.add_special_tokens(
    {"additional_special_tokens": ["<|im_start|>", "<|im_end|>"]}
)

Fixes the issue:
{'input_ids': [6, 10707, 144, 59733, 7], 'attention_mask': [1, 1, 1, 1, 1]}

This change will let users handle these cases correctly.
How has this been tested?
I ran a training for 1 step after adding this to my config:
special_tokens:
  additional_special_tokens: ["<|im_start|>", "<|im_end|>"]

Without this, it's likely that many ChatML models are in-fact semi broken, because it's common practice to add <|im_start|> and <|im_end|> like this:
special_tokens:
  eos_token: "<|im_end|>"
tokens:
  - "<|im_start|>"

This means that the tokenizer might incorrectly tokenize <|im_start|> strings.
Social Handles (Optional)
dreamgen on discord
https://twitter.com/DreamGenAI

Slight clarification:
tokenizer.add_tokens(...)

Also ensures that the tokens are treated special by the tokenizer, even if the token is already in the vocabulary!

Add a list of new tokens to the tokenizer class. If the new tokens are not in the vocabulary, they are added to it with indices starting from length of the current vocabulary and and will be isolated before the tokenization algorithm is applied. Added tokens and tokens from the vocabulary of the tokenization algorithm are therefore not treated in the same way.

However, special_tokens_map.json file will not contain such added tokens as special, which will potentially confuse downstream libraries / inference engines trying to load these models, and lead to mistokenization.

Thank you for the PR. I ran into this issue today and it also seems to me that <|im_start|> should be marked as special.
Running your code on the mistral base model with this config:
special_tokens:
  eos_token: "<|im_end|>"
  unk_token: "<unk>"
  additional_special_tokens: ["<|im_start|>"]

Did yield an error:
Traceback (most recent call last):
  File "/root/miniconda3/envs/py3.10/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/root/miniconda3/envs/py3.10/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/workspace/axolotl/src/axolotl/cli/train.py", line 58, in <module>
    fire.Fire(do_cli)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/fire/core.py", line 475, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/workspace/axolotl/src/axolotl/cli/train.py", line 34, in do_cli
    return do_train(parsed_cfg, parsed_cli_args)
  File "/workspace/axolotl/src/axolotl/cli/train.py", line 54, in do_train
    return train(cfg=cfg, cli_args=cli_args, dataset_meta=dataset_meta)
  File "/workspace/axolotl/src/axolotl/train.py", line 156, in train
    trainer.train(resume_from_checkpoint=resume_from_checkpoint)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/transformers/trainer.py", line 1539, in train
    return inner_training_loop(

Changing the config to this worked:
special_tokens:
  eos_token: "<|im_end|>"
  unk_token: "<unk>"
  additional_special_tokens: ["<|im_start|>"]
tokens:
  - "<|im_start|>"

I believe it may have to do with the fact that Mistral doesn't have <|im_start|> in its vocabulary, unlike Yi.

Thank you for testing! Do you actually see tokenization differences? I did run over the whole training set with these 2 options:
(A) Only use add_tokens on <|im_start|> and <|im_end|>
(B) Use  add_tokens  + additional_special_tokens
And I did not find a single example where the token ids would differ. If you did, can you share an example so I can test with it?
On the issue you mention, I think additional_special_tokens should work even if the tokens were not in the vocab yet:
# %%
from transformers import AutoTokenizer, AddedToken

tokens = ["<|im_start|>", "<|im_end|>"]

tok1 = AutoTokenizer.from_pretrained("mistralai/Mistral-7B-v0.1")
tok2 = AutoTokenizer.from_pretrained("mistralai/Mistral-7B-v0.1")

tok1.add_tokens(
    [
        AddedToken(token, rstrip=False, lstrip=False, normalized=False)
        for token in tokens
    ]
)

tok2.add_special_tokens({"additional_special_tokens": tokens})

# %%

inputs = [
    "<|im_start|>user\nhello<|im_end|>",
    "<|im_start|>user\nhello<|im_end|>\n<|im_start|> assistant\nworld<|im_end|>",
]

for input in inputs:
    print(tok1.tokenize(input))
    print(tok2.tokenize(input))
    print()

# %%

And it does not raise an exception and the outputs are the same (in this case the ids are swapped, but that's fine):
['<|im_start|>', '▁user', '<0x0A>', 'hello', '<|im_end|>']
[1, 32000, 2188, 13, 21558, 32001]
['<|im_start|>', '▁user', '<0x0A>', 'hello', '<|im_end|>']
[1, 32001, 2188, 13, 21558, 32000]

['<|im_start|>', '▁user', '<0x0A>', 'hello', '<|im_end|>', '▁', '<0x0A>', '<|im_start|>', '▁', '▁assistant', '<0x0A>', 'world', '<|im_end|>']
[1, 32000, 2188, 13, 21558, 32001, 28705, 13, 32000, 28705, 13892, 13, 9471, 32001]
['<|im_start|>', '▁user', '<0x0A>', 'hello', '<|im_end|>', '▁', '<0x0A>', '<|im_start|>', '▁', '▁assistant', '<0x0A>', 'world', '<|im_end|>']
[1, 32001, 2188, 13, 21558, 32000, 28705, 13, 32001, 28705, 13892, 13, 9471, 32000]

So I am not sure what's the issue. The stack trace does not have much information unfortunately :-/

@DreamGenX , I think the difference in the actual implementation and your test code above is that you are adding both the tokens to the vocabulary, and then setting them as special. setting <|im_start|> as a special token without adding it to the vocabulary is what is happening in @komninoschatzipapas first version of the config.

I also get this error if the additional_special_tokens are not already in the vocabulary or included in tokens:
  File "/home/wing/micromamba/envs/dev/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/wing/micromamba/envs/dev/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 1031, in forward
    attention_mask = attention_mask if (attention_mask is not None and 0 in attention_mask) else None
  File "/home/wing/micromamba/envs/dev/lib/python3.10/site-packages/torch/_tensor.py", line 997, in __contains__
    return (element == self).any().item()  # type: ignore[union-attr]
RuntimeError: CUDA error: device-side assert triggered

alright, found the bug and added a test for it.

Thank you for the fix! Just to clarify, you can see in the example that tok2 uses only add_special_tokens, and not add_tokens, and it works. It seems that normally add_special_tokens also adds the special tokens to the vocab if they aren't there yet.

rebasing from main to pick up fixes for ci failures b/c of torch-2.2.0 release

what if the "additional_special_tokens" is already present in the tokenizer.config for the model that is being fine tuned? Is it still necessary to put them in the axolotl yaml manifest? Or they'll just be picked up as expected from tokenizer.config?

@winglian @DreamGenX Based on all the discussion and changes above, it is now very confusing knowing how to properly set up your config.yml for training a sharegpt dataset with the chatml chat template. Would the following stripped example work as intended? It mirrors those found in other issues but it's hard to know what's current at this point. Note this is for a model with a tokenizer that does not already contain <im_start> and <im_end>.
base_model: /hf_downloads/mistralai/Mistral-7B-v0.1
model_type: MistralForCausalLM
tokenizer_type: LlamaTokenizer

#etc

datasets:
  - path: 64bits/lima_vicuna_format
    type: sharegpt
chat_template: chatml

#etc

tokens: # added for chatml
 - "<|im_start|>"
 - '<|im_end|>'
special_tokens:
  eos_token: "<|im_end|>" # added for chatml


cadl-csharp emitter should create actual Types from CADL models

Describe the issue or request
For the CADL:
model Deployment { [fields definitions] };
A corresponding Deployment type should be created in the emitted C#. (Currently only the CADL routes are transformed into C# code).
Example Deployment CADL definition: https://github.com/Azure/azure-rest-api-specs/blob/feature/cognitiveservices/openai-cadl/specification/cognitiveservices/OpenAI.Authoring/models/deployment.models.cadl

@glharper, you need to define client.cadl file to list all convenience APIs, then models used in those APIs will be transformed into C# code.
Here is one example: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/cognitiveservices/AnomalyDetector/client.cadl
Note, if you create client.cadl file, you need to specify that in npx cadl compile command.


Can we specify the rowspan dynamically?

I have a requirement where i have to display image on right hand side of table and actual data on left hand side. The data is dynamic hence i cant hardcode the rowspan for given image. e.g
In most of the examples we are specifying hard coded row span value and add the exact rows mentioned in rowspan.

I don't really understand the problem. Isn't it possible to do one intermediary step and just see how many rows will be generated and in a second step generate the table with rowSpan being set correctly? This whole problem seems more a general programming question to me than a specific question or issue with easytable.
From a user's point of view that could be a convenient add-on. Sure. But I refrain from implementing it for now because I think it will add too much complexity for what it brings value.

Thanks @vandeseer .  I was trying to check it whether we can simply it in user table. I have made changes and able to resolve the issue but after implementation i am getting error at Table.java for below code. What i found that even though I have 3 column table the columIndex became 3 resulting in ArrayIndexOutOfBoundException.
Column column = table.getColumns().get(columnIndex);


Expand/Collapse all nodes

Hello,
I ve found out, that you provide a reset functionality to operate on openNodes. I am just curious about the data structure of the openNodes. Basically its a "itemA/itemB" etc. Do I have to write my own algorithm, which transform my data, which are being passed to TreeMenu into this form "key/key" etc. I am just wondering if there is a more smooth way to do that :).
BR,
Ales.

Yes, and it's intentional.
If the default expand/close behaviour meets your requirement, you only need to provide data. But if you have specific requirements, then you have to implement your own logic, this TreeMenu component allows you to manipulate almost everything.
openNodes is used when you want to have full control over every node, and you don't want the default expand/close behaviour AT ALL. You can see the storybook example here. I guess this API is rarely used.
If you only want to set the tree to a specific shape but still allow users to use the tree menu, you can use initialOpenNodes, see example here. This API is more useful I guess.


add a new extensions that allows you to run python code

#1136

Already on the way.

Already on the way.

@FurryR This is so cool! 🎉

@FurryR Did you make a new Scratch.ArgumentType.CODE?

@FurryR Did you make a new Scratch.ArgumentType.CODE (or something like that?)

It looks like a mockup. I'd be surprised if it were real!

@FurryR Did you make a new Scratch.ArgumentType.CODE (or something like that?)

It looks like a mockup. I'd be surprised if it were real!

It is real, he used patches to make it real.

@FurryR Did you make a new Scratch.ArgumentType.CODE (or something like that?)

It looks like a mockup. I'd be surprised if it were real!

It is real, he used patches to make it real.

That's so cool!


Beta - Strapi v4: Mailgun causes the link tool to fail

When @strapi/provider-email-mailgun is installed in Strapi, it causes the link tool to fail. The link controller is using open-graph-scraper, which uses got, and this is where the failure occurs. This is a known issue with got when mailgun is installed. See https://github.com/panva/node-openid-client/issues/320#issuecomment-764416863. Not sure if we should leave this as is and give users the warning, or maybe try using a different tool other than open-graph-scraper to scrape the data.
When printing out the response from server/controllers/editorjs.js link, I get this response:
{
  success: false,
  requestUrl: 'http://strapi.io',
  error: 'connect ECONNREFUSED <IP_ADDRESS>:443',
  errorDetails: RequestError: connect ECONNREFUSED <IP_ADDRESS>:443
      at ClientRequest.<anonymous> (/Volumes/Work/Web/strapi-v4/src/plugins/strapi-plugin-react-editorjs/node_modules/got/dist/source/core/index.js:962:111)
      at Object.onceWrapper (events.js:482:26)
      at ClientRequest.emit (events.js:387:35)
      at ClientRequest.emit (domain.js:470:12)
      at ClientRequest.origin.emit (/Volumes/Work/Web/strapi-v4/src/plugins/strapi-plugin-react-editorjs/node_modules/@szmarczak/http-timer/dist/source/index.js:43:20)
      at TLSSocket.socketErrorListener (_http_client.js:475:9)
      at TLSSocket.emit (events.js:375:28)
      at TLSSocket.emit (domain.js:470:12)
      at emitErrorNT (internal/streams/destroy.js:106:8)
      at emitErrorCloseNT (internal/streams/destroy.js:74:3)
      at processTicksAndRejections (internal/process/task_queues.js:82:21)
      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1148:16) {
    code: 'ECONNREFUSED',
    timings: {
      start:<PHONE_NUMBER>824,
      socket:<PHONE_NUMBER>825,
      lookup:<PHONE_NUMBER>825,
      connect: undefined,
      secureConnect: undefined,
      upload: undefined,
      response: undefined,
      end: undefined,
      error:<PHONE_NUMBER>825,
      abort: undefined,
      phases: [Object]
    }
  }
}

@jaskipper - I think that since this problem is related to MailGun, it would be nice to get a response from the developers first

@melishev - It absolutely is a problem related to MailGun. MailgunJS is deprecated. Unfortunately the plugin seems to take everything over and it breaks the link tool. I've tried to find a workaround, but had no luck up to this point.


fix: handle parser error

Fixes: https://github.com/mcollina/undici/issues/62

I though instanceof Error would have significant performance implications but doesn't seem so based on the benchmark.


Help in SignAndSendTransaction

Hi everyone,
I have been facing an issue with near-api-js for sending a transaction.
Context:
We are building a brower-based wallet and we have been trying to send a transaction but getting the following error
{
 "jsonrpc": "2.0",
 "error": {
  "name": "HANDLER_ERROR",
  "cause": {
   "info": {},
   "name": "INVALID_TRANSACTION"
  },
  "code": -32000,
  "message": "Server error",
  "data": {
   "TxExecutionError": {
    "InvalidTxError": "InvalidSignature"
   }
  }
 },
 "id": 125
}

Code snippet:
         const privateKey = '....';
        const accountId = '6162316635333866613030646163383761306634393335363433316636373061';
         const keyPair = KeyPair.fromString(privateKey);
         const myKeyStore = new keyStores.InMemoryKeyStore();
          await myKeyStore.setKey('testnet', '', keyPair);
          console.log({myKeyStore});
          const near = await connect({ networkId: 'testnet', nodeUrl: NODE_URLS['testnet'], keyStore: myKeyStore });
          const account = near.account(accountId);
          await signer.addKey(keyPair.getPublicKey().toString());
      // await signer.sendMoney('samvel-test.testnet', new BN(0.01))

Any idea about it?
Am I missing anything very basic here?
Thanks In Advance.

This is resolved by using inMemorySigner()


click() with a xpath ending the script without error

I am finding a strange problem where upon calling click() with a xpath, the click event is being sent and the download expected starts, but immediately the script dies for no apparent reason, with the tagui's console window staying up.
There are no errors nor exception to capture, so I'm stumped here.
Am I missing something? Is there any info I can provide?

Hi @carlos-negrao-tlp , can you run with r.debug(True), and attach the rpa_python.log here?
Also, there is a tagui_chrome.log which you can find in the folder r.tagui_location() --> tagui folder --> src folder
I have not come across this issue or hear other users report something like this before. Any info would help troubleshooting.
Also, can you share a simple replication script so that I can try to see if it also crashes for me.
What about other websites, does it crash for you?

Thank you responding.
Unfortunately, I had to give up on the approach due to time constraints.
I didn't bother trying with other sites, since it would have been pointless (for me, at least), and neither have the code anymore, since I throw away experiments that go nowhere.
What I can say is the website I had been trying to scrap seems to have been made with GWT, which you may know is a pain to scrap.


Allow transfers to account via alias

Allow account alias to be used in transactions. ATM only address can be used as a target of transactions. Both namespace and address should be allowed because namespaces can be used as aliases for accounts. Thomas got this working by representing namespace id as 25 bytes.
I should probably introduce some kind of abstraction that would allow use of both address and namespace id (assuming it is aliased to account). Need to come up with name for it. Either Destination or Target. it would have 2 static "from" constructors where one would take address, the other namespaceid, perhaps even string representing namespace fully qualified name as a convenience.

Recipient is terminology used for the address, so perhaps that could be the name of the abstraction

decided to create recipient but provided also convenience methods to create transaction directly with address or namespace id

pull request #67 created


Audio iso unicast client fix

Fixes a few issues that was introduced by the audio_iso refactor work, which unfortunately broke the BAP unicast client completely.
fixes https://github.com/zephyrproject-rtos/zephyr/issues/52464

https://github.com/zephyrproject-rtos/zephyr/pull/51699 unfortunately introduced more issues for the unicast client, so added another commit to fix that.


Coverity 1356971: Unchecked return value

Problem:
CID 1356971 (#1 of 1): Unchecked return value (CHECKED_RETURN)
13. check_return: Calling TSUrlCreate without checking return value
Fix:
Added the result check and also fixed a resource leak.

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/482/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/491/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2174/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/480/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/605/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/2070/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/734/


Feature: Hover and highlight contribution graph

Type of feature
🍕 Feature
Current behavior
Today the contribution graph is static. It would be nice to hover over a point and find out how many commits or lines contribution(s) are at a single point.
Suggested solution
Here is a version of what I am looking for with tabler.

Additional context
note: this should update for appearances of the contribution graph.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

I like how Soundcloud displays comments along the song where folks contributed (commented) at the exact second. If we did something similar to this, like you mentioned, and show avatars along the graph (like the comments along the song), I think that would be a great feature.

Are these the contribution graphs that is the target of this issue?

Yes. Also the contributions on the user profiles.

Will this work ?

I like. I think instead of just the number, 3. It should be "3 days ago"

I agree with "x days ago"

Aside from the "x days ago" I like it.

Another potential way is 1-7 days ago. Then the actual date, "Jul 17," which is how GitHub does it for their graph.

Thank you for the feedback and suggestions!  I believe "x days ago" would be the better option. It provides a more precise and concise representation of the timeline, making it easier for users to understand their contributions at a glance.
Shall i go ahead and implement ?

Shall i go ahead and implement ?

Yes. That date format is used in the highlights card. You can most likely reuse some of that code.

.take