text:
Make it obvious when event has precise start location and when it doesn't

Ideally we want all events to have a google map link that points to exact location, but sometimes events can be created without this "luxury" for various reasons. We should at least make it obvious when location can be trusted and when it cannot be and when flyer must be opened.
One of the options is to show a label "precice start address" and when address is approximate show a circle like this one: https://developers.google.com/maps/documentation/javascript/examples/circle-simple

I think we need 3 labels:

Race Location (as we have now) - it should be the registration place
Precise race location (as you said)
Start Location (with start sign like this http://screencast.com/t/xNjmYpzH, if we don't have a registration place from flier, but could find the start place)

@Me4ta I think as far as racer concerned we need just one – registration location (where should I drive 5am in the morning), but also we need to reinforce if that location is "precise" meaning just copy-paste or open in Google Maps or NO.
Start Location is not important at all, since they will figure it out at the race most likely.


text:
Add a hollowsquare pattern

Hey i want to have patterns like this:
* * * * *
*       *
*       *
*       *
* * * * *

Do it in your favorite programing language, remember to ask for assign this issue.
Make sure it's dynamic meaning you have a horizontal and vertical value that changeable
If your favorit language folder doesn't exist, make it by yourself and add file, name it hollowsquarepattern
Don't forget to test and check your PR

Hey @Reynadi531, can you assign this to me?

@Reynadi531 Please assign me for this.

Please assign me this.

sure everyone, go ahead

hey! i want to contribute in this repo in c language

Hi @Reynadi531 ,
Please assign me.
Thanks

Hello,
I have a Java solution for the hollowsquare, if it's still open I'd like to be assigned.
Thanks!

Yeah, go ahead

@Reynadi531 I want to contribute in python. Please assign me for this project.

i have a solution in javascript, shall i go ahed!

Assign me

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
chore: updates and scripts

scripts for prettier and type checking
dependency updates
1 test failure for const-namespace where it turns it into a let instead of a const.

would you consider commiting the generated types and ensure they are in sync  with a script before release?
eg https://github.com/dominikg/tsconfck/blob/b436363b353d76a55947a672593344a57b10ab62/.github/workflows/release.yml#L50
otherwise it may be good to call the cli as first step for check:types to ensure they are in sync

reverting back to ~5.0.4 lets the test pass, not sure what exactly causes it, but this reads like it could affect dts-buddy https://devblogs.microsoft.com/typescript/announcing-typescript-5-1/#reduced-calls-into-scanner-for-jsdoc-parsing

closing in favour of #45

source: gharchive/pull-request

authors: Rich-Harris dominikg

repo: Rich-Harris/dts-buddy


text:
MagicString.indent is slow

According to https://github.com/rollup/rollup/issues/723#issuecomment-235984088, MagicString.indent can take up quite a bit of Rollup's bundle time. It may be worthwhile to try to optimize it.

Yeah. It's not any slower than any other operation, it's just that it needs to do a lot more of it (Rollup only needs to make small edits here and there until indent(), at which point every line is affected) – would definitely be good to make it faster, but it may be the case that we're better off just disabling indentation by default. Won't be able to take a look any time soon (am on a harsh deadline hence my recent radio silence!) but keen to improve this.

authors: Rich-Harris Victorystick

repo: Rich-Harris/magic-string


text:
Mid0011 does not work because of incorrect length calculation

in Parse() you set the size:
GetField(1, (int)DataFields.EACH_PARAMETER_SET).Size = package.Length - GetField(1, (int)DataFields.EACH_PARAMETER_SET).Index;

This is wrong, because "package" includes the Message End termination NUL.
If you add a "- 1", Mid0011 works fine.
My fix is a bit more secure:
int length = package.Length - GetField(1, (int)DataFields.EACH_PARAMETER_SET).Index;
if (package.Length > 0 && package[package.Length - 1] == '\0')
{
  length--;
}
GetField(1, (int)DataFields.EACH_PARAMETER_SET).Size = length;

Since the header length property excludes NUL termination, it's more reliable and performance will be better (just a little)
Documentation about header:

Message part
Byte
Parameter
Value

Header
1-4
Length
The length is the length of the header plus the data field excluding the NUL termination.The header always includes information about the length of the message. The length is represented by four ASCII digits (‘0’…’9’) specifying a range of 0000 to 9999.  When using the message linking functionality the length represents the length of each message part number.When having one ASCII part followed by an binary part the length is the total length of the message.

You are right for the header field "Length", but package.Length does not access the header field Length.
package ist the string, which contains the entire message and this includes the NUL termination. Therefore your calculation is wrong and Mid0011 does not work.

authors: IngeRuppio Rickedb

repo: Rickedb/OpenProtocolInterpreter

license: mit

license_source: bigquery


text:
How to change the mvc action argument type?

I have a mvc action like this:
public async Task AddBatchDevice(string[] deviceNos ,ObjectId? customerID=null)
but the ObjectId in mongodb is the struct,I write the jsonconverter to change the string to objectId,but nswag can't mapper the objectid to string when it on method argument?
how to resolve it?thk

I try to use:
1: opt.TypeMappers.Add(new PrimitiveTypeMapper(typeof(ObjectId?), (s) =>
{
s.Type = JsonObjectType.String;
s.Title = "ObjectId";
}));
2.[JsonSchema(JsonObjectType.String)]ObjectId? customerID=null
above all can't work

Did you try [JsonSchemaType(typeof(string))] on the parameter?

Did you try [JsonSchemaType(typeof(string))] on the parameter?

yes,I try it,,but no work

and I find out a way to fix it ,but it look like so.......bad

authors: RicoSuter kugarliyifan

repo: RicoSuter/NSwag


text:
"Accommodation" industry is misspelled

Fix this in utils/industry.py

Corrected for version 0.1.4

repo: RiskLens/verispy


text:
paffy, youtube-dl not found

I got error message like that, how to fix it?

@gusandy505 hi, just install youtube_dl to fix it.
run the following commands in cmd:
cd C:\Python27\Scripts (or wherever you installed it)
pip.exe install youtube_dl
Now that error message should not pop up

repo: Ritiek/Spotify-Downloader


text:
"Game" in "Game or Player?" slides in whenever you open the Report menu.

A minor glitch that I've noticed. The text "Game" will always slide in when you click Report rather than it not animating.
https://i.gyazo.com/3c069451ef7212d9d64e82a4a6d3a01b.mp4

Yes, I also noticed this. I can confirm that it shouldn't animate each time.

authors: VRService boynedmaster

repo: Roblox/Core-Scripts


text:
New internal Context API

Helps to close #4. This is a prerequisite for the new public Context API proposed in #246.
This change introduced two new methods to the Component API:

Component:__addContext(key, value) - Adds a new entry to the context table. This will be available to this component and all of its children, and is intended to be used in a Provider component.
Component:__getContext(key) - Retrieves the context value for the given key. This intended to be used by a Consumer component.

This interface allows us to avoid defensive copying, as even internal Component APIs do not get direct access to the context table. The legacy context API is still present and usable, and will not be affected by these changes.
Checklist before submitting:

Added entry to CHANGELOG.md (this should come with the changes in #246)
[x] Added/updated relevant tests
[x] Added/updated documentation (documentation here consists of code comments, public docs will be part of #246)

Coverage increased (+0.1%) to 93.667% when pulling 28cc872d561186c47345b2dd0b234f7edf4a8524 on ZoteTheMighty:new-context-api into deaa8000220252f6edc27ec15e5704266560ebe5 on Roblox:master.

authors: ZoteTheMighty coveralls

repo: Roblox/roact


text:
Cleanup various source code (mostly) formatting issues

The majority, but certainly not all, of this is automated fix up. It is on my TODO to enforce all the style checks on CI so things do not get stale again.

This change is

+@EricCousineau-TRI for review.

, with a few minor comments.

Reviewed 13 of 23 files at r1, 13 of 13 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

drake_cmake_installed/src/find_resource/find_resource.cc, line 53 at r2 (raw file):

    std::cerr << "Should have thrown" << std::endl;
    return 1;
  } catch (const std::runtime_error&) {

BTW This looks a tad odd?

drake_cmake_installed/src/particles/test/test_particle.cc, line 30 at r2 (raw file):

 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 * POSSIBILITY OF SUCH DAMAGE.
 *****************************************************************************/

Can you add a TODO for me in this file to change it to pure C++, and not use GTest?

drake_cmake_installed/src/particles/test/test_particle.cc, line 33 at r2 (raw file):

// clang-format off
#include "particle.h"  // IWYU pragma: associated

BTW Shouldn't this be allowed by default? Seems odd that the linter catches this.

Comments from Reviewable

Review status: 18 of 28 files reviewed at latest revision, 3 unresolved discussions.

drake_cmake_installed/src/particles/particle_test.cc, line 30 at r2 (raw file):
Previously, EricCousineau-TRI wrote…
Can you add a TODO for me in this file to change it to pure C++, and not use GTest?

I added it in the CMakeLists.txt. There are multiple files.

drake_cmake_installed/src/particles/particle_test.cc, line 33 at r2 (raw file):
Previously, EricCousineau-TRI wrote…
BTW Shouldn't this be allowed by default? Seems odd that the linter catches this.

Seems we were not following the Google naming convention for tests. Should be fixed by the latest commit.

drake_cmake_installed/src/find_resource/find_resource.cc, line 53 at r2 (raw file):
Previously, EricCousineau-TRI wrote…
BTW This looks a tad odd?

BTW True, but it keeps clang-format happy.

Comments from Reviewable

Reviewed 15 of 17 files at r3.
Review status: 27 of 28 files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from Reviewable

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from Reviewable

authors: EricCousineau-TRI jamiesnape

repo: RobotLocomotion/drake-shambhala


text:
Can take screenshot in Android 7.1 (API 25)

The name of the decor view class appears to have changed, so isn't matched in isDecorView. If I use the debugger for force a match, the screenshot is successfully captured.

"cannot take screenshot on android 7 " problem +1

repo: RobotiumTech/robotium


text:
It will be nice to have migration functionality.

How do you recommend setting migration with this structure?

I would probably put it in manage.py but I haven't had to implement migration in a Flask app using this template so I'm not too sure.

But you need to register the 'db' command like in this example
`
from flask import Flask
from flask_sqlalchemy import SQLAlchemy
from flask_script import Manager
from flask_migrate import Migrate, MigrateCommand
app = Flask(name)
app.config['SQLALCHEMY_DATABASE_URI'] = 'sqlite:///app.db'
db = SQLAlchemy(app)
migrate = Migrate(app, db)
manager = Manager(app)
manager.add_command('db', MigrateCommand)
class User(db.Model):
id = db.Column(db.Integer, primary_key=True)
name = db.Column(db.String(128))
if name == 'main':
manager.run()
`
And If you do this in manage.py file inside a command function. It will not work. As the command you will fire will be.
Python manage.py <command/function name> db init

How I managed to do. I made a new file called migration.py with the below code. I don't know how to do this in manage.py.
import sys
import os
from personalization_engine.application import create_app, get_config
from flask_script import Manager
from flask_migrate import Migrate, MigrateCommand
from personalization_engine.extensions import db
def parse_options():
"""Parses command line options for Flask.
Returns:
Config instance to pass into create_app().
"""
# Figure out which class will be imported.
if os.environ.get('CONFIG') == "production":
    config_class_string = 'personalization_engine.config.Production'
else:
    config_class_string = 'personalization_engine.config.Config'
config_obj = get_config(config_class_string)

return config_obj

if name == 'main':
app = create_app(parse_options())
migrate = Migrate(app, db)
manager = Manager(app)
manager.add_command('db', MigrateCommand)
manager.run()

manager.add_command('db',migrateCommand)


text:
Feature : Adding Special Effects

Hello , We Can add a hover effect like when user gets to any button on navbar then the buttons should increase in size or pop up .. It will be more good if we can apply it

@Sing-Li

@samad-yar-khan


text:
wip: inappchat  component for virtual conference

https://user-images.githubusercontent.com/73601258/153422109-ac96ef88-39c6-42b3-ab9f-30c3ff9ae13c.mp4
#38

Fetching Messages And Sending Messages
https://user-images.githubusercontent.com/73601258/153489088-371d8a78-6552-4b7a-ac13-e9084bb0b45e.mp4

rocketchatcomponent1.mp4
#38

This is terrible UX  fwiw.   When there is no chat - we want the video to be in "full size" maxed out on the frame without sacrificing aspect.   The "tab" or "button" or "arrow" to activate chat should be up at the top right - as small as reasonable -- and be transparent if placed over the playing video.  On mobile, there should be no "tab"/"button"/"arrow" at all, but just a "swipe right" gesture.
In no case should be there be "jumbo white space" and "jumbo blue button" that hogs real estate for no purpose at all.
Please confer with irfan on the "design" for next iteration.

True, I've just placed them as placeholders for now.Will be working on them soon

@sidmohanty11   does the current code include the emoji support you have shown us today?
I'd like to marge it down ASAP so @demonicirfan  can add the animations.
Let me know.   It would be good if the cookie implementation can be added as well (but not necessarily at this time).

Just give me one more day so I can try my best to figure out if we can use the cookies and clean some things up
Thank you!

@Sing-Li , I have added the cookies part here (full-validation from browser cookies)
https://user-images.githubusercontent.com/73601258/154857956-f4be68f3-b789-4616-84bc-08c8cb600dd0.mp4
But we are facing a major roadblock, i.e, with the previous approach (with CSR approach) after a certain interval the requests are being revalidated which as I have discussed with Rohan sir and he agreed that that might not be the best approach. So it is still to be figured out. :(

Cool.  @sidmohanty11    Let's go with the current way of  handling  (hard coded ENVIRONMENT VAR) for now.   Is it OK for me to merge this down now?

@Sing-Li I can work on that on a separate PR, and as UI needs a lot of work I think @demonicirfan can take it from here


text:
Uploading files can block your heartbeat.

The heartbeat setting is 20 seconds, the chat page sends the larger file, the upload time more than 20 seconds, causes the heartbeat timeout abnormal.

Not valid anymore for version 2.0.0.

authors: ainiyou rafaelks


text:
[NEW] Add day marker in chatroom

Added Day Marker for messages in chatroom
Closes #825

@pcforgeek Thank you very much for the PR! Feature is working really nice! I had one crash while scrolling thought the messages, not sure if it's related to this change or not. 👍

@rafaelks Happy to help. I didn't face any such issues on my system, but I do hope that you resolve this glitch. Let me know if you need any further help regarding this.

@pcforgeek Could you please solve the conflicts?

@leonardoaramaki Solved the conflicts.

There are a lot of indentation problems on this PR, like if(data.showDayMarker){
You should take a look on the Kotlin Coding Convetions to reformat the code.

@luciofm Changes made.

@pcforgeek I'm sorry, but there are some new conflicts on your PR... can you take a look and @luciofm can you review as soon as he fixes it, please?

@rafaelks @luciofm Fixed the conflicts.

@rafaelks Changes made. Thanks for pointing about Index 0 of 0 error.

I've tried implementing this branch into a fork of RocketChat and found a bug.
When i send a message on a different day, it doesn't show the day marker. Only after I re-enter the chatroom it shows the day marker.

@leonardoaramaki is going to fix the conflicts of this PR, make some code adjustments and merge it for the 2.5.0 release! 🎉

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: rafaelks:white_check_mark: filipedelimabrito:white_check_mark: Pitstopper:white_check_mark: shubhsherl:white_check_mark: leonardoaramaki:white_check_mark: luciofm:x: ericrosenthalYou have signed the CLA already but the status is still pending? Let us recheck it.

authors: Amirko28 CLAassistant leonardoaramaki luciofm pcforgeek rafaelks


text:
Refactoring

Improve code style and readability
Group API routes by tag as different modules
Fix pylint errors

JFYI: All integration tests passed for this.

I will have to remove W0511 from pylint as well since we will persist using #TODO for now

repo: RocketPy-Team/Infinity-API


text:
Problem with import @unform/web v2.1.2

Description of bug

After the v2.1.2 update, importing @ unform/web is no longer recognized. Because there are no components in the folder.
To Reproduce

run yarn add @unform/core @unform/web
try import import { Form } from '@unform/web';

Screenshots

Environment:

OS: Ubuntu

I have the same problem

yarn add @unform/web@2.1.0 solves while the package is not properly updated.

Fixed, install it again please.

repo: Rocketseat/unform


text:
🛑 URL Redirection (roco2.tk) is down

In 0599d2c, URL Redirection (roco2.tk) (https://roco2.tk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: URL Redirection (roco2.tk) is back up in 6c16e71.

repo: Roco-Team/stats


text:
Exporting to unreal stops 3d preview from updating when changing nodes values.

When I export to unreal the 3d view permanently stops respoding to changes in values of some nodes like Tones Range.To fix it either unplug and plug in back some node, change one of the values that forces screen refresh like blending mode in blend node. or restart Material Maker. By the way 3d view still works, rotates and all. It is just that it stops reacting to values input changes.
1.0 on win 10

same with export to godot


text:
Si un jugador tiene una energía inferior a 0, ese turno se pasa y se le acreditan 5 puntos extra

Creo que esto no esta implementado

Solucion hardcodeada
        if (this.energia.isAgotada()) {
            logger.info(this + " se encuentra sin energia para moverse, para el siguiente turno se repondran 5 puntos");
            this.energia.aumentarEnergia(new Energia(5));
            throw new MovimientoException("El gladiador no se puede mover sin energia");
        }


text:
D&D 5e

Changes

Removed inline style
Added a class

Roll20 Requests

If changes fix a bug or resolves a feature request, be sure to link to that issue.
For pull request that change multiple sheets please confirm these changes are intentional for all sheets. This will help us catch unintended submissions.
When updating existing sheets if you are changing attribute names please note what steps you have taken, if any, to assist players in keeping their data.

Close this. I need to separate out my Shadowrun stuff so its not getting pushed twice.


text:
[Malefices] Feuille de personnage pour Malefices

Changes / Comments
Bonjour
Nouvelle Feuille de personnage pour Malefices V3.
Cordialement
Le Pelican
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[X] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


text:
Multicore preprocessing solution for pytorch

I’m looking at this framework from performance perspective- I want to use it to quickly preprocess videos en masse.
I am happy to help build a wrapper - I would need it to match record to make it easier to consume. I really want to achieve this though
https://github.com/johndpope/MegaPortrait-hack/issues/38
To get best performance - I’m prepare to cut code in c++ to do this. Or is there other things that come to mind ?
Related - https://github.com/dmlc/decord/issues/283

Hi @johndpope
From your description I got some overall understanding of what you are trying to achieve.
What VALI features are you interested in WRT your project ?

I’m looking at this framework from performance perspective

It does natively support:

Multi GPU
Video seek, separate decode and demux
Zero copy memory sharing with torch cuda tensors
Simple image processing routines like resize and color conversion

i had a play with multithreading - on cpu - but to get images done - looks like can't get away from cpu.
https://github.com/johndpope/MegaPortrait-hack/issues/38
i think the problem is pilimage has never had gpu support....
i use the streamreader from torchvision - somewhat successful to cycle through frames.
            streamer = StreamReader(src=video_path)

            # - ``"rgb24"``: 8 bits * 3 channels (R, G, B)
            # - ``"bgr24"``: 8 bits * 3 channels (B, G, R)
            # - ``"yuv420p"``: 8 bits * 3 channels (Y, U, V)
            # - ``"gray"``: 8 bits * 1 channels

            streamer.add_basic_video_stream(
                frames_per_chunk=16000,
                frame_rate=25,
                width=self.width,
                height=self.height,
                format="rgb24"
            )

i appreciate your timely response - i guess i continue down this streamer path for now.
but you're saying i can do gpu + images? maybe i can extend your work.
off to karate training now. i take another look later.

@johndpope
If you just want to get decoded video frames and run your inference on them as a first step, you may follow the code from torch segmentation test:
https://github.com/RomanArzumanyan/VALI/blob/820caeae824199bd01b9649aa02095137e2df780/tests/test_TorchSegmentation.py#L178
It keeps everything on GPU. Yet there's a room for perf optimization which wasn't done because it's a unit test, not perf test. E. g. decode video frames in batches, not one by one. Anyway, it may be a good starting point.

@johndpope
Hi. Please LMK if your issue is resolved.

Hi @RomanArzumanyan  - thanks - i circle back when i have more bandwidth. closing for now.
this kinda wrapper thing - i had a crack at cupy -
https://github.com/johndpope/MegaPortrait-hack/blob/feat/38-multicore/EmoDataset.py#L251
wasn't successful - but i take another look with this.
processing a video with gpu - and then augmenting etc - it's very valuable -
please consider adding some helper wrappers to allow working with the frames.
I'll take a stab when i have bandwidth.
tensor_frame, image_frame = self.augmentation(frame, self.pixel_transform, state)

        if self.apply_crop_warping:
            transform = transforms.Compose([
                transforms.Resize((self.width, self.height)),
                transforms.ToTensor(),

Closing the issue because last response was month ago.

authors: RomanArzumanyan johndpope

repo: RomanArzumanyan/VALI


text:
Can't connect to Inav-Configurator on Linux after connection with HIL

Steps to reproduce:
1 run XPlane and connect to flight-controller (from plugin menu)
2 disconnect plugin (from plugin menu)
3 try to connect with inav-confaigurator - failed
But! If after the step 2 we will connect to the port by terminal:
screen /dev/ttyACM0 115200 and then killall screen
we will be able to connect by inav-configurator
In my mind, after we closed the serial port in plugin we still have some unflashed data
Also, it would be great to add if (isConnected()) to each methods

I have HITL installed under x-plane 12 on Ubuntu 22.04. However, HITL doesn't show up under plugin options. I have tried the same on Windows 10. HITL works perfectly.    Not sure which step I did wrong on Linux system. Or, is there anyone having the similar issue?

I used chmod for FC connection with inav-configurator on Ubuntu and it works well, but HITL doesn't work on my Ubuntu machine. Not sure what happened. It looks there is a compatibility issue with lin.xpl on my Linux machine.
sudo  chmod 0777 /dev/ttyACM0

(I am not using Linux and I am not an exprt in Linux)
One guess is that some libraries are missing.
Try running:
ldd lin.xpl

(I am not using Linux and I am not an exprt in Linux) One guess is that some libraries are missing.
Try running: ldd lin.xpl

Yes, libopenal.so.1 is missing. I installed the library using the following command and it works!
sudo apt install libopenal1
Thanks so much!!

authors: RomanLut lwan288 sevrugin

repo: RomanLut/INAV-X-Plane-HITL


text:
1.15.0 upgrade

Maybe it's nothing but when installed over an older WAU the user task just writes to the log:
Running in User context
Then nothing happens but it finds user installed apps and tries to upgrade them, but fails (the toasts pop up).
Not until an uninstall and then an install it behaves correctly, but then boy it works! 👏💕🌹
This is now my second upgrade.
On this I ran the install with (on the other  with default blacklist:
-UpdatesAtLogon -Silent -UseWhiteList -BypassListForUsers -StartMenuShortcut -DoNotUpdate

I need to check what happens when upgraded "manually"

and on your second upgrade, everything works as expected ?

Uninstall/Install - then it's perfect! 👌✌👏

Oh, I think I know why... If the log file exists, the ACL are not overwritten. I handled that for the auto-update, but not for reinstall.

authors: KnifMelti Romanitho

repo: Romanitho/Winget-AutoUpdate


text:
Units for public methods

I have added units to all the properties for data and results. I had a problem with the LineParametersy_shunt which can be None. A None can not be in S/km... Now, I assign a zeros matrix to this field and I have created a method to know if the parameters includes shunt (or not).
I changed the maintainer in the __about__ file
I modified the JsonMixin class in order to include the json methods into it. I also dispatched the assignation of results and the results' reader methods.
I uniformised the dtype=complex for numpy arrays. When the project started the behaviour of dtype=np.complex_ and dtype=complex was not the same... It is now the case. Thanks to the Numpy team!

Thanks.

I have also looked at pint pandas to have units in the resulting data frames, but it seems less finished than pint itself. We may reconsider this in the future


text:
Add Rachna-Bansal to Contributors list

This is my first pull request.

@rachnaban, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

authors: Roshanjossey rachnaban

repo: Roshanjossey/first-contributions


text:
Addon missing in Mozilla repository

Hi, I reinstalled Firefox today and now I can't find the ACF-TOOLS extension anywhere to download. It also disappeared from the official repository.
Is it possible to send the installation package?

Would like to know as well

Will get it back up on Mozilla asap.
Expect an update on this issue next week.

I managed to dig out the old XPI from the profile backup, but I would still appreciate the extension in the repository. Thank you

+1

Please!

Not in the store - hope you doin ok!

I ran into the same issue and generated the Firefox version from source (first time doing this). I'm attaching it here for other folks who may need it.
acf-tools-v3.0.2-firefox.zip

@my-pixels: is it possible that you send me your XPI file?

Hi!
This ZIP contains the following: A compiled version of both the Chrome and Firefox version of ACF Tools
ACF Tools compiled.zip
Unfortunately Mozilla never re-reviewed the package after it was deleted. As I don't work that much with WordPress anymore, I don't have the time and resources to continue maintaining this extension.
You are of course free to do whatever you with with the code, whether it's forking it, submitting a PR (I'll be happy to review, and merge it) or even uploading it to Mozilla yourself. The code was written at a time when I was just starting off with web development, and is far from ideal or even readable. I apologise for the inconvenience.
I suggest checking out Hookturn's alternative to ACF Tools' code generator: https://hookturn.io/downloads/acf-theme-code-pro/

authors: RostiMelk Sanabria adrien-robert giorgiodare goerlitzer mortensassi my-pixels txrris

repo: RostiMelk/ACF-Tools


text:
Merge 'ThreeMammals:master' into 'master'

Let's update your repo with latest top-commits from head repository.
That will allow to perform rebase-operation for feature branches without merge-commits.
⚠ Warning!
Use fast-forwarding aka Rebase and merge option to merge please! 👇

Alternatively,
Could you Sync fork please? This is preferred option!
After repo synchronization, close this PR please!

Also, I don't see develop branch in your forked repo!
Could you Sync fork please? So, new develop branch will occur with all top commits!
Don't forget to make develop branch as default one!

repo: RupenAnjaria/Ocelot


text:
No working if only sink is kept after constructing

pub fn new() -> Self {
  let (_stream, stream_handle) = OutputStream::try_default().unwrap();
  let sink = Sink::try_new(&stream_handle).unwrap();
  Self {sink}
}

If i try to use the struct only keeping sink, it is unable to play any music

Perhaps it is better to mention that?

it will be in the next release: https://github.com/RustAudio/rodio/pull/591
and I hope to remove the need for keeping the stream in the future

authors: chanmaoganda dvdsk

repo: RustAudio/rodio


text:
32 bit support

I fixed a compile error and then a few tests that depended on oom-catching.
cc @aurelien-m

I was fixing one sequence issue, and I sorta just went all out on fixing other stuff that I'd been bothered by for a while ¯\_(ツ)_/¯

Thanks for looking into this :)
Seems to work perfectly on my Raspberry Pi running on Raspbian (or called Raspberry Pi OS now) 32bit!

repo: RustPython/RustPython


text:
add markdown linting, using markdownlint

new workflow definition to run the checks
new directory to hold markdownlint specific configuration
rule configuration for the markdownlint tool
a problem-matcher to associate markdownlint output with the content
a new section in the Style Guidelines

See also https://qmacro.org/2021/05/14/notes-on-markdown-linting-part-2/.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant qmacro

repo: SAP-docs/contribution-guidelines


text:
Fixed wrong documentation about Security->Users menu access

My name is Mario Kabadiyski. I am development architect in the Datasphere Cross-Architecture team and would like to fix an inconsistency in this document that we recently found due to a customer incident.
The documentation about accessing the Security->Users menu in the Datasphere security application falsely claimed that this menu can be accessed by a user having just the USER.read and one of the permissions USER.create, USER.update or USER.delete permissions. This has never been implemented like this and also doesn't work because all four permissions are required to access this menu. However, since only USER.read and USER.manage can be manually set in the role editor, in fact, only user roles that have been derived from the DW Administrator role can have all 4 USER permissions set and hence be able to access the Security->Users menu and screen. The current change corrects the documentation to reflect the real implementation that has been in place for more than 3 years now.

Dear Mario
thank you for reporting this point! We are checking with relevant team and will come back to you with a proposal.
best regards
cindy

Hello Mario,
Thanks for your valuable feedback and for the suggested changes to the documentation.
Before I can enter them, I have two questions:
1-About the Read permission: We talk about a list of users in a dialog: the example is related to a stories and teams. These are related to SAC. Do we have other examples that are not related to SAC?
In this wiki https://wiki.one.int.sap/wiki/display/DWAAS/Introduce+Additional+Space+Privileges, it seems that in combination with the other privileges Spaces.Read and Team.Read, User.Read enables to see the list of members in a space.
Also, I can see that the Read permission (and only this permission) is included in the DW Space Administrator role.
Could you please confirm and I’ll mention it in the doc?
2-About the Manage permission: do you know what “approving role assignment requests from users” refers to?
Thanks for your help!
Best regards,
Olivia

Hello Mario,
Thanks for your valuable feedback and for the suggested changes to the documentation. Before I can enter them, I have two questions:
1-About the Read permission: We talk about a list of users in a dialog: the example is related to a stories and teams. These are related to SAC. Do we have other examples that are not related to SAC? In this wiki https://wiki.one.int.sap/wiki/display/DWAAS/Introduce+Additional+Space+Privileges, it seems that in combination with the other privileges Spaces.Read and Team.Read, User.Read enables to see the list of members in a space. Also, I can see that the Read permission (and only this permission) is included in the DW Space Administrator role. Could you please confirm and I’ll mention it in the doc?
2-About the Manage permission: do you know what “approving role assignment requests from users” refers to?
Thanks for your help! Best regards, Olivia

Hello Olivia,
regarding your 1-question:
My change only relates to the access to the security->users menu in the application shell. All other use cases in the USERS privilege explanation are still valid (including those in the mentioned Wiki). As for the DW Administrator role, it is not true that it only includes only the USERS.read permission. The DW Administrator role and only this role sets all CRUD Bits for the USERS privilege (create, read, update, delete). You can see this when creating a custom user role that derives from the DW Administrator role as a template. The role management screen will then show all four bits set, but the their checkboxes are disabled (except the one for USERS.read).
regarding your 2-question:
Users may request for obtaining additional roles in a so called security request. This is an extra menu in the SAC tenant and is only visible there. Not sure, why this is mentioned in the Datasphere USERS documentation. Having the "manage" permission a user will be able to approve such "security requests" for role assignments. As far as I know, this permission has no meaning in Datasphere roles.

Hello Mario,
Thanks for your answers.
After a discussion with Reiner, we’ve decided to keep all texts related to SAC functionality.
I’ve updated the documentation as follows. You can see the change you’ve suggested in the green frame:

You can also see the whole topic here:
https://help.sap.com/docs/SAP_DATASPHERE/9f804b8efa8043539289f42f372c4862/d7350c6823a14733a7a5727bad8371aa.html?state=DRAFT&version=DEV
Please let me know if the changes are correct.
This will be delivered in version 2023.14.
Thanks again for your feedback!
Best regards
Olivia

Thank you for your valuable content contribution, @ChilliFog! So that we can recognize your contribution in SAP Community, please tell us your SAP Community profile URL in a reply to this comment; don't include any other text, just the URL on its own, like this:
https://people.sap.com/your-user-name

Thanks!

https://people.wdf.sap.corp/profiles/D021551

authors: ChilliFog CindyVenet OliviaValeano

repo: SAP-docs/sap-datasphere


text:
Week 3, unit 2, step 2

For Week 3, unit 2, step 2
https://github.com/SAP-samples/abap-platform-rap-opensap/blob/main/week3/unit2.md#step-2-adjust-the-behavior-definition-of-the-travel-entity
The screenshot is,

and the code listed is below, instead of "1234" it should be "####"
Plus line 1 and 37 from the screenshot is missing,
looks like there is a similar issue for step 3
define behavior for ZI_RAP_Travel_1234 alias Travel
persistent table zrap_atrav_1234
lock master
//authorization master ( instance )
etag master LocalLastChangedAt
{
  create;
  update;
  delete;
  association _Booking { create; }

  field ( numbering : managed, readonly ) TravelUUID;

  mapping for zrap_atrav_1234
  {
    TravelUUID         = travel_uuid;
    TravelID           = travel_id;
    AgencyID           = agency_id;
    CustomerID         = customer_id;
    BeginDate          = begin_date;
    EndDate            = end_date;
    BookingFee         = booking_fee;
    TotalPrice         = total_price;
    CurrencyCode       = currency_code;
    Description        = description;
    TravelStatus       = overall_status;
    CreatedBy          = created_by;
    CreatedAt          = created_at;
    LastChangedBy      = last_changed_by;
    LastChangedAt      = last_changed_at;
    LocalLastChangedAt = local_last_changed_at;
  }
}

Hallo ,
Thanks for feedback.
The suffix _1234 has been replaced by the placeholder _####.
Kind regards,
Carine

thanks, also I got a syntax error because I copy pasted the full code example, which is missing line 1, and maybe also line 37,

Hello,
In Step 2, you are not supposed to replace the whole source code, but just the behavior definition of the Travel entity instead.

define behavior for ZI_RAP_Travel_#### alias Travel
...
...

Kind regards,
Carine

authors: CarineTD larshp

repo: SAP-samples/abap-platform-rap-opensap


text:
Adaptive Search Profile is not applied on Category Page

Environment Details

Spartacus:  1.2.0

Steps to Reproduce

Prepare Category Aware Search Profile (to be done in Adaptive Search) with some configuration e.g promoted products.
Make sure configuration is synchronized to online catalog version.
Navigate to category with adaptive search changes applied.
Search Profile changes for category is not applied.

Observed Results
Requests sent from spartacus does not narrow down search to category and rather treat category as facet.
Expected Results
Category Aware Search Profile changes are applied Category Page
Offer to Fix
No
Additional Information
Simple Search Profile works fine. Problem is only occurring for Category Aware Search Profile.
On old storefront it works fine.

Potentially this seems to be a fix in backend Search API, creating ticket for backend search team

@Xymmer  to followup with backend team.

@Xymmer I have just taken a look at this because of a customer query.  The short version is: the Adaptive Search functionality on the backend looks for use of the allCategories field, whereas Spartacus is currently using the category field.
You can see this in DefaultSolrAsCategoryPathResolver.resolveCategoryPath() which is responsible for working out the category for the query, and from that the category path to the root category ... and this determines which Category Aware Search profiles are activated. If you follow the code through, it is looking first for a Filter Query against allCategories, then for a Facet Value against allCategories
If you look at the query that Spartacus issues, it is currently specifying a Facet Value against category (query=:relevance:category:1360) whereas the old storefront accelerator applies a Filter Query against the allCategories field when it constructs the query for a category page.  Where the comment above says 'On old storefront it works fine' ... that isn't really true.

I would need to modify product-list-component.service.ts to try this for real from Spartacus ... but if I modify the request manually in my browser to do query=:relevance:allCategories:1360, then the result set from OCC matches what I see via the old accelerator & products boosted for that specific category rise to the top

@Xymmer @tobi-or-not-tobi I just created a pull request to fix this item.  First time I have tried this for Spartacus so hopefully I have followed the guidelines correctly.  Not sure who I should ping to review this though?

@neilhubertprice nice one! I had fixed this in a local branch so can directly confirm you went right with your implementation. What I also remember however is that the breadcrumb is not returned with the response if you use allCategories relevance. We need an investigation on that one, as breadcrumb is currently a crucial part. Can you double check?

Ok, so I have also pushed a fix to the same branch for CategoryPageMetaResolver to understand the allCategories facet.
Note: Commerce setup needs modifying for this to work correctly.  The allCategories facet by default has no Facet Value Display Name Provider set.  Without that the breadcrumb displays the code rather than the name. Set this to the same as the category facet (categoryFacetDisplayNameProvider) to have proper names in the breadcrumb.
A couple of other points:

I'm not sure the breadcrumb behaviour is ideal currently (I haven't modified the overall behaviour), as it shows breadcrumb links for any categories/brands chosen while filtering the result set.  This gives a non-intuitive list of breadcrumbs that links directly to the indivdual categories/brands .... really it probably ought to link to the search results with/without those filters.
I haven't modified the test for CategoryPageMetaResolver, as it is out of date & only references the deprecated resolve() method

authors: KateChuen Xymmer mateuszburaczek neilhubertprice tobi-or-not-tobi

repo: SAP/cloud-commerce-spartacus-storefront


text:
chore: Add PR checks

Add checks on PRs. The checks run on changes to the PR metadata and on external PRs.
It checks:

cc is used
title is upper case
there is a body and it is not the template
there is a changelog that fits the cc preamble

Closes SAP/cloud-sdk-backlog#926.

[x] I know which base branch I chose for this PR, as the default branch is v2-main now, which is not for v3 development.
[ ] If my change will be merged into the main branch (for v3), I've updated (V3-Upgrade-Guide.md)[./V3-Upgrade-Guide.md] in case my change has any implications for users updating to SDK v3

I have a question regarding .github/actions/check-pr/node_modules/. Should this directory be ignored, or is it intended to include them?

This is intended. For actions this is required unfortunately.

This is intended. For actions this is required unfortunately.

I've never worked with actions inline in the repo, but would not be building the action using ncc be a way to avoid that? this produces one big inlined js artifact which can be executed without node modules..

Do we need the test files in the changeset dir?

ncc

I think this is a nice improvement. But I would prefer to separate this from this PR, given that we have another action, that I would probably have to touch as well. I will create a ticket.

Do we need the test files in the changeset dir?

Nope, I removed them now. Good catch 👍

authors: fwilhe marikaner

repo: SAP/cloud-sdk-js


text:
fix: Fix batch read response type

In my batch v4 refactoring a while back, I think I made a wrong assumption on a type. If the type remains as is, users would have to always cast the result from .as(BusinessPartner) as BusinessPartner otherwise it can be either an error or a business partner. Instead as should only be called, when we know it is not an error. I described this behavior in https://github.com/SAP/cloud-sdk/pull/11.

The improvement makes sense, but we should consider the release notes.

Done.


text:
Can't use property name starting with "{" when using complex binding syntax

OpenUI5 version: latest (master branch)
Browser/version (+device/version): all
Any other tested browsers/devices(OK/FAIL): browser/device independent
URL (minimal example if possible): http://jsbin.com/mamoyaqizu/edit?html,output
User/password (if required and possible - do not post any confidential information here): n/a
Steps to reproduce the problem:

Open JSBin
Observe button does not render, and see error in console: Uncaught SyntaxError: no closing braces found in '{foo' after pos:0
Change "{foo" to "foo" and observe error go away

What is the expected result?
The property value should be set to "{foo"
What happens instead?
A syntax error is thrown
Any other information? (attach screenshot if possible)
Sometimes, a property value starting with a curly brace is desired, but not for binding, just for it's raw value. In our use-case this is for reporting errors with messages that sometimes start with that character.
When using the default xx-bindingsyntax value (default) the problem cannot be repped. When set to complex (as in the above referenced jsbin) I can rep the problem.
The code path difference that results when using complex is in ManagedObject.extractBindingInfo when it invokes the bindingParser. It will use BindingParser.complexParser rather than BindingParser.simpleParser.
The runtime SyntaxError is explicitly thrown by the complexParser where it is trying parse the string.

Hello sasjonbri,
you will need to escape the "{" in the binding syntax with "\" then it will work.
I updated your example with the fix:
http://output.jsbin.com/cavovawaki
Kind Regards,
Michael

authors: Michadelic sasjonbri


text:
[FEATURE] Trusted IdP configuration options for shadow users and user logon

What area do you want to see improved?
terraform provider
Is your feature request related to a problem? Please describe.
As a customer I want to have an option, to configure the settings of my trusted IdPs, such as disabling the automated creation of shadow users, or the availability of the IdP for user logon. These settings can be configured using the SAP BTP CLI as follows.
btp update security/trust sap.custom --subaccount --auto-create-shadow-users false
btp update security/trust sap.default --subaccount --available-for-user-logon false
As of today, these configuration cannot be changed using the SAP BTP Terraform provider.
Describe the solution you would like
It would be amazing to have a new resource, allowing the more granular "configuration" or "update" of a Trusted Identity Provider, or giving users the option to provide additional parameters when configuring the trust setup resources, for instance:
resource "btp_subaccount_trust_configuration" "ias_trust" {
  subaccount_id     = "9ef6b690-a1b2-c3d4-e5f6-cf96ceba3904"
  identity_provider = "sap-demo.accounts.ondemand.com"
  auto_create_shadow_users = "false/true(Default)"
  available_for_user_logon = "false/true(Default)"
}

Describe alternatives you have considered
Currently, the alternative of using the SAP BTP CLI solves the problem as kind of workaround.
Additional context
https://help.sap.com/docs/btp/btp-cli-command-reference/btp-update-security-trust
(--available-for-user-logon [BOOL], --auto-create-shadow-users [BOOL])
https://help.sap.com/docs/authorization-and-trust-management-service/authorization-and-trust-management/switch-off-automatic-creation-of-shadow-users
https://help.sap.com/docs/btp/sap-business-technology-platform/hide-logon-link-for-default-identity-provider

@martinfrick thank you for placing this as a feature request. We'll be looking into both cases and keep you updated here.

Tasks:

[ ] Check diff between CREATE and UPDATE command of BTP CLI
[ ] Add diff of available configuration options to schema of btp_subaccount_trust_configuration and btp_globalaccount_trust_configuration
[ ] Add flow to implementation of resource namely execute two commands in corresponding functions to trigger the create and update calls in sequence. Handling of state must be considered so that in case the second call fails, a consistent state is stored.  This is needed for the CREATE and UPDATE functionality.
[ ] Enhance test cases and fixtures to consider additional settings.
[ ] Update documentation

repo: SAP/terraform-provider-btp


text:
feat: Add used_in_production arg in subaccount creation and update

Purpose

Closes #273
With this PR users can now set used_for_production = UNSET | USED_FOR_PRODUCTION | NOT_USED_FOR_PRODUCTION when creating subaccounts

...

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

resource "btp_subaccount" "subaccount" {
  name = var.display_name
  subdomain = var.subdomain
  region = var.region
  parent_id = var.parent_guid
  used_for_procduction = "NOT_USED_FOR_PRODUCTION"
}

What to Check
Verify that the following are valid

Verify that the subaccount created has the attribute correctly set

Other Information

Thanks for your contribution! Can you execute a make generate and commit the updated documentation to this PR?

@Thanhphan1147 : No worries about the failed tests. I will check what is missing an eventually add new test recordings to this PR

@Thanhphan1147 there is one issue to fix: you need to adjust the facade_accounts_subaccount_test.go. Due to the addition of the usage the expected result now deviates (see e.g. https://github.com/SAP/terraform-provider-btp/actions/runs/5532790691/jobs/10095867971?pr=288#step:6:214)
The other error in the test must be fixed by us by re-recording the test fixtures for the subaccount

I updated the test file

I merged it into a temporary branch to re-record the tests and then merge it into main.


text:
fix: use hana visitor for tablefunction parsing

Fixes https://github.com/SAP/xsk/issues/1276
Fixes https://github.com/SAP/xsk/issues/1047

Added listener and model for tablefunctions to hana parser with some tests.
Used the new listener to populate the model in the hdb facade XSKHDBTableFunctionParser.
Found & Fixed issue where handleHDBTableFunctions folder recursion is started once per each local file in migration

⚠ Merge with: https://github.com/dirigiblelabs/xsk-ide-migration/pull/64

@Fluctuationqt could you please rebase the PR as there are merge conflicts?
@MarinHadzhiev could you please do the code review again, confirming that all required modifications are performed as expected?

authors: Fluctuationqt ThuF

repo: SAP/xsk


text:
Tutorial : Integrate a Custom-Developed SAPUI5 App Into Your Launchpad Site

I followed the tutorial and completed the following chapters successfully.

Set Up SAP Business Application Studio for Development
Create a Dev Space for SAP Fiori Apps
Create an SAP Fiori App Using SAP Business Application Studio

In "Build and Deploy Your SAP Fiori App to SAP Business Technology Platform", I followed all the steps.
In step 4, sub-step 5 it says we will be able to see the application. But for me it shows "Application could not be started due to technical issues."
Git : https://github.wdf.sap.corp/sli-innersource/SocialAtSAP-Admin.git
Git Branch : adminUI.
Url after deloyment : https://social-sap-dev.launchpad.cfapps.sap.hana.ondemand.com/e41c7fde-8c61-4d7b-9321-c281370be8f4.admin.comsocialadmineventevent-1.0.0/index.html
Please help me out at the earliest.

Hi
If you did all the tutorials then it can only be one of the following reasons:
You used a standalone app router when you created the MTA
You didn't add an intent to the app
If it's the first problem, you should delete the app from the space on in your subaccount and develop it again using this tutorial https://developers.sap.com/tutorials/appstudio-fioriapps-create.html.
Make sure you select 'Managed by SAP Cloud Platform' App Router' (as explained in step 2)
If it's not one of these issues, then there is really a technical problem and you should open up a support ticket.
Hope this helps.
Thanks
Lindsay

Hi Lindsay,
We have used the Managed App router option as mentioned in the tutorial.
The one major difference is, we have CAP service which has UI annotations as well. So, instead of SAP Freestyle application, we chose 'List Object Page'.
While loading the app, there are issues with metadata.
Could you please help here?
Thanks,
Viji

Hi
Thank you for your reply. It seems that your issues require investigating your code which is beyond the scope of supporting the tutorials. To have your issue inspected, please open a BCP for the EP-CPP-CF-DEV component.
Thanks
Lindsay

Thanks Lindsay, I will do it.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

authors: I321500 LindsayBert MichaelCzcz

repo: SAPDocuments/Issues


text:
add libbiblio.xsl from ALEA framework

In order to use the transformations as preview for Diwan, libbibli is required.

[x] make a package from ALEA's libliblio.xsl
[x] use that package in preview for diwan
[x] assert that it works for some examples

[x] BAlam56

[x] regression test

added 20m of time spent

mentioned in commit 52085f32b5085d6a49493c1c9e726022a0ed2c68

marked the checklist item assert that it works for some examples as completed

marked the checklist item Aqaf25 as completed

added 1h 30m of time spent

marked the checklist item use that package in preview for diwan as completed

marked the checklist item BBlam3 as completed

marked the checklist item make a package from ALEA's libliblio.xsl as completed

mentioned in commit 8a824ce85e1508ce0b02d1b5be47d30785526166

mentioned in commit d7ea7ac9a2d5730c8a8406f6a3f8884589e2fca1

added 2d 4h of time spent

marked the checklist item regression test as completed

added 30m of time spent

added 15m of time spent

mentioned in commit 2ed1c4f5257f46e2c497668f00ae6d99d5c0df55

added 1h of time spent

Unit test works as should.
But using it in diwan.xsl shows that dereferencing does not work correctly.

repo: SCDH/seed-tei-transformations


text:
Create dataset loader for Thai Toxicity Tweet Corpus

Dataloader name: thai_toxicity_tweet/thai_toxicity_tweet.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?thai_toxicity_tweet

Dataset
thai_toxicity_tweet

Description
Thai Toxicity Tweet Corpus contains 3,300 tweets (506 tweets with texts missing) annotated by humans with guidelines including a 44-word dictionary. The author acquired 2,027 toxic and 1,273 non-toxic tweets, which were manually labeled by three annotators. Toxicity is defined by the author as any message conveying harmful, damaging, or negative intent, in accordance with their defined criteria for toxicity.

Subsets
-

Languages
tha

Tasks
Text Classification

License
Creative Commons Attribution Non Commercial 4.0 (cc-by-nc-4.0)

Homepage
https://github.com/tmu-nlp/ThaiToxicityTweetCorpus/

HF URL
https://huggingface.co/datasets/thai_toxicity_tweet

Paper URL
https://www.ta-cos.org/sites/ta-cos.org/files/1_W32.pdf

#self-assign

@holylovenia Not sure if any of the current tasks suit this dataset, maybe adding something like HATE_CLASSIFICATION or TOXICITY_CLASSIFICATION would work?

authors: SamuelCahyawijaya elyanah-aco

repo: SEACrowd/seacrowd-datahub


text:
Question-tags: checkbox isn't correct (so you can't uncheck a checkbox)

When a checkbox of a questiontag get's checked (to show in select students), the card/information of the student gets changed (but not always), but if you go back to the settings of the questiontag, everything is unchecked.
That way it is impossible to uncheck a checkbos (and questiontag)

It seems that every patch to the backend fails. Editing question tags also doesn't work.

Changing visibility (showInList) of mandatory tags does not work, the patches fail.
Is it possible that this is a problem in the backend?
@victormylle @Vital7788 @QuintenVervynck

The group agreed that the showInList of mandatory tags should not change. They represent data that is mostly already visible in the student list. So we will not implement the feature to change the showInList of mandatory tags.

authors: Jarne-Andries SimonVandeVyver