Username and Pasword

While running the file i'm having this particular issue i.e. username and password invalid. I'm using the default values as username and password i.e. username,password and also tried testadmin and testpass.
Altthough i am able to run the judges server but can't run the client server. both the clients failed.
please help ASAP

How did you perform setup?  If you used the automatic provisioning, via vagrant up/setup.sh, the admin username and password will be testadmin/testpass. There is also an unprivileged user, testuser/testpass. The client tests use those users, do they pass?
You can also manually create an admin user:
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py createsuperuser

i have created a dedicated machine using linux and used setup.sh to setup the server.

That should have created an admin user. Regardless, you can create your own admin user with the above command, or to install the testadmin user with manage.py:
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py loaddata fixtures/test_admin.yaml

To install all of the test data required for the client tests:
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py loaddata fixtures/test_fixture.yaml

I believe I am getting the same error. I have put the full stack trace on Pastebin. I am not sure how to fix this. What do you suggest?
http://pastebin.com/PqiPaZfx

as told by @DarkmatterVale the same error is stil recieved by me.

@sharma16aug Are you are still receiving the same error after trying the loaddata command I suggested?
@sharma16aug and/or @DarkmatterVale could you provide a dump of the database you have? Run:
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py dumpdata --format yaml -o db.yaml

And upload db.yaml. This will let us verify whether the user is in the database at all.
Do either of you happen to still have the output of setup.sh that you can upload (even just the end). Unfortunately, we don't currently log it to a file, so it will only be around if you still have that terminal, but it should not have completed successfully without installing this user.

@prattmic Sorry, but I did not save the output of setup.sh. It did appear to run correctly however, because it did not stop and quit at any point in time.
I am not able to run the above commands. I get this error:
CommandError: Unable to serialize database: No module named yaml

It might be due to puppet maybe not being run properly? Is there something I need to run other than setup.sh?

Try changing --format to json.  We may not install YAML by default.
On Fri, Nov 6, 2015, 09:04 Vale<EMAIL_ADDRESS>wrote:

@prattmic https://github.com/prattmic Sorry, but I did not save the
output of setup.sh. It did appear to run correctly however, because it
did not stop and quit at any point in time.
I am not able to run the above commands. I get this error:
CommandError: Unable to serialize database: No module named yaml
It might be due to puppet maybe not being run properly? Is there
something I need to run other than setup.sh?
—
Reply to this email directly or view it on GitHub
https://github.com/auvsi-suas/interop/issues/7#issuecomment-154473598.

Here is the link to the pastebin:
http://pastebin.com/cmQqicsz

It looks like none of the default data was loaded into your database.
Try loading the test fixture: (this is what should have been installed)
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py loaddata fixtures/test_fixture.yaml

It should print some number of fixtures installed.
No worries about not keeping setup.sh logs. I don't expect folks to think to log it, but I figured I'd ask. I'm going to submit a change soon to start logging output by default for each run so we will have it.

Here is the output from running those commands:
Installed 0 object(s) from 0 fixture(s)

Is this my problem? Do I need to install a set of fixtures?

Yes. The fixture is the file fixtures/test_fixture.yaml, and it should have objects in it, one of which is the test user.

Hm, you couldn't export YAML before, perhaps this is the same problem, but
loaddata is being quiet about it.  Try installing PyYAML, I think it is
missing from our list of dependencies.
$ cd ~/interop/server
$ source venv/bin/activate
$ pip install PyYAML

Then try the loaddata command again.
On Sat, Nov 7, 2015, 08:27 Phillip Tischler<EMAIL_ADDRESS>wrote:

Yes. The fixture is the file fixtures/test_fixture.yaml, and it should
have objects in it, one of which is the test user.
—
Reply to this email directly or view it on GitHub
https://github.com/auvsi-suas/interop/issues/7#issuecomment-154720710.

That loaded the data properly. The output was Installed 18 object(s) from 1 fixture(s)
I am testing the clients once more, hopefully it will work :)

We now install PyYAML in automated setup.
@sharma16aug this should fix your problem as well. Just git pull the latest version of the repo, and run setup.sh again (or install PyYAML like @DarkmatterVale did).

@prattmic I'm still getting the same error even after creating the data base. the test.sh script fails.
here's the link to the output. http://pastebin.com/hgXMnP4j

Hm, it looks like the data was installed correctly.
I don't think it should be necessary for the server to pick up the changes, but can you try restarting Apache and rerunning the tests?
$ sudo service apache2 restart

If that doesn't work, can you upload a database dump like @DarkmatterVale did?
$ cd ~/interop/server
$ source venv/bin/activate
$ python manage.py dumpdata --format json -o db.json

here's the link to the db.json file :-  http://pastebin.com/QbiuvV8v
also i'm getting this dependency error whenever i'm running setup.sh :- http://pastebin.com/EbMDv6hP
still getting the error even after making a clean install of the server once again.

The error says it's timing out when executing the Karma and Jasmine installation. The installation is done via NPM install command. The default is 300s, so I'm rather confused why it isn't succeeding.

basically the problem is during installation and not with the server script. If on another system it gets installed then the server and client will run smoothly, isn't it ?

Yes that should be true. I haven't seen this error before, usually it's a non-issue.

OK. Thanks a lot. I will try it on another machine,and will surely let you know about any problem if persists again

Closing for now, please reopen if the error returns and persists.


Use more standard angular formatting for factory

Not sure what the problem is, but I switched over to the more standard formatting for angular factories and modules (based on the docs) and the problem went away. sooo ¯_(ツ)_/¯

Silly Angular... Thank you!


Run in an HDInsight cluster error

For the session "Run in an HDInsight cluster", I got the error below:
The specified project file is invalid or does not exist', 'errorCode': 'Microsoft.DataPrep.MessagingErrors.InvalidProjectFile. I was able to run every step in previous session successfully. Any ideas what might go wrong?

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

ID: 6a2981a1-6887-9986-e1ec-d19003ba37e5
Version Independent ID: f04ec0f0-5b08-47c6-53a8-4b275d8b3d10
Content: Bike-share tutorial - Advanced data preparation with Azure Machine Learning Workbench | Microsoft Docs
Content Source: articles/machine-learning/preview/tutorial-bikeshare-dataprep.md
Service: machine-learning
GitHub Login: @ranvijaykumar
Microsoft Alias: ranku

@CindyBai
Thanks for the feedback! We are currently investigating and will update you shortly.

Thank you, Jason. I found the root cause. The name of my data preparation package is different than the file name in the BikeShareDataPRep.py, which led the error: 'The specified project file is invalid or does not exist'. After I modify the file name in the python file, it ran successfully.

Thank you for the update @CindyBai. If you should have any additional issues with this tutorial, you are welcome to reopen this issue tracker.


stlink_probe.read_memory_block8 returning extra bytes on un-aligned 1 byte read

It seems to me that commit 46f1ac66c26169270b6c6e3d7992f3b106a8077b introduced a bug. The function stlink_probe.read_memory_block8 will return an array with two bytes in it if the addressed is aligned on a 2-byte boundary but a read of 1 byte (size=1) is requested. This ends up causing an assertion at /memory.py(263)read_memory_block8(): 0107462 E Unhandled exception in handle_message (b'v'): result size (2) != requested size (1) [gdbserver]
I assume we need to add something along the lines of array[0] somewhere, though I am not sure about the appropriate place.
@flit

Thanks for the detailed report! (And very sorry it took a long time.) Fortunately, it looks like this was fixed in #1428 and released in v0.34.1. If you still have a problem, please reopen this issue.

v0.34.2 still have same problem. #1428  not fix this problem.  But error in same place.  Error occurs if (address & 3) >1   and read size 1 or 2.


[SPARK-8807][SparkR] Add between operator in SparkR

JIRA: https://issues.apache.org/jira/browse/SPARK-8807
Add between operator in SparkR.

Merged build triggered.

Merged build started.

Test build #37091 has started for   PR 7356 at commit c6a25c5.

Test build #37091 has finished for   PR 7356 at commit c6a25c5.

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

Merged build finished. Test PASSed.

@sun-rui could you also take a look at this ?

Merged build triggered.

Merged build triggered.

Merged build started.

Test build #37463 has started for   PR 7356 at commit 7f51b44.

Test build #37463 has finished for   PR 7356 at commit 7f51b44.

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

Merged build finished. Test PASSed.

Thanks @viirya -- LGTM.

@viirya,

Why pass the lower bound and upper bound in a vector instead of pass separately? They are passed separately in Scala API. and A vector can not hold complex types (at least a list is expected).
Instead of writing R wrapper functions individually for each method of Scala Column, currently Column.R dynamically creates wrapper function using template. Is it possible to use the same mechanism for supporting between? something like:

column_functions3 <- c("between")
createColumnFunction3 <- function(name) {
setMethod(name,
signature(x = "Column"),
function(x, arg1, arg2) {
... # some check of arg1/arg2
jc <- callJMethod(x@jc, name, arg1, arg2)
column(jc)
})
}

@sun-rui I use the API described in the JIRA ticket. Actually I used two parameters for the lower bound and upper bounds at the first. I am not sure if the between operation supports complex types?
I think it is possibly to have createColumnFunction3 to support it. If there are enough methods with 3 parameters, I think we can create it.

Hmm I think the scala API having two arguments makes an interesting case -- I did not notice that. What does the Python API do ?
One thing about the createColumnFunction things is that they are hard to document with roxygen. We should create a separate JIRA to discuss this.


[release-v1.7] Align midstream images 1.7 ubi  8minimal

In #116 we started using ubi image for midstream (like product), but ... I did not see the dockerfile with kodata template... hence some file (the actual images) were forgotten.
This aligns the rest of the images w/ the ubi image, like product

/hold
let's see
Also there seems to be an issue w/ UBI and ls /usr/share/zoneinfo/....  /cc SRVKE-1422

/retest

/retest

/retest

@matzew we might need to add RUN microdnf install tzdata for zoneinfo?

/test 412-test-e2e-aws-ocp-412

/close
started for 1.9 and 1.10 ... that's good enough


Set Dehya as a one off after 3.6 banners are out

Since, well, she'll be a standard character.

Not going to bother with, v2 doesn't really distinguish anymore.


[Subtask] Add authorization for metalake

Describe the subtask
Add authorization for metalake
Parent issue
#2250

Duplicated.


Make Repository.findPath relative to resource

Hi
maybe it's only me, but when I program against a Repository instance, I'd expect every operation bounded to the resource path. If I don't want that, I can simply program against the Rest endpoint..
I need the findPath method, because my endpoint follows a (very common) REST structure where ids are in the path: /categories/id
If there's another clean way to GET a resource by altering the path, then discard the following.
The findPath method on Repository feels wrong to me. It simply forwards the path to the underlying Rest endpoint, without prefixing resource path:
var findQuery = this.getTransport().find(path, criteria);

but it can be changed to
var findQuery = this.getTransport().find(this.resource + path, criteria);

and change the find method accordingly:
    Repository.prototype.find = function find(criteria, raw) {
      return this.findPath('', criteria, raw);
    };

Then I can use the findPath method like repo.findPath('/id')

findPath is one of the methods that's there to allow you to change the path. The reason for this is because not all routes are relative to the resource (like find is).
It's there in case you want to use the repository for a different path, in stead of find which is relative. I think this makes perfect sense, but I'd love to hear what you think.

For what I understood in the library docs, the repository should be tied to a particular resource, like users or documents etc.
For that matter I expect that every action I do through a repository is tied to that particular resource. If I want to fiddle with the api endpoint I can program against the underlying rest interface of aurelia-api...
Having said that, imagine that I have a resource that maps some subpaths to some related entities, like users/{id}/delegates. I usually do that when the related entities involves particular server-side effort (like fetching from remote resources).
If findPath were relative, I could simply do:
this.repository.findPath(`${id}/delegates`);

instead I have to do the pretty awkward:
this.repository.findPath(this.repository.getResource() + `/${id}/delegates`);

For my use case the former is more frequent than the latter, and as I said above if I wanted to program against the endpoint I could simply use the aurelia-api rest interface.
Thx,
P.

UPDATE: missed a criteria parameter in the examples above, because if I want to do a GET to a subresource I can use the find method like
this.repository.find(`${id}/delegates`);

and that works, but I can't do
this.repository.find(`${id}/delegates`, criteria);

I have to use findPath instead, but given that's not relative to the resource I have to do:
this.repository.findPath(this.repository.getResource() + `/${id}/delegates`, criteria);

@nexbit Then I should update the docs. You shouldn't think in resources for custom repositories, but rather in worker classes. You could use api for that, but that's not the point of repositories.
We could add a method, to make it easier to find relative to the resource, or refactor find to accept that + criteria. I'd +1 that!.

I see your point, but from a first (quick) analysis of the plugin's sources, it reminds me more the hybrid approach that breezejs takes than a pure Repository pattern.. But it can make sense if a repository tied to a model spans different physical resources on the server, even if it's a rare condition in my personal experience.
Your second proposal (refactor find to accept a path AND a criteria) maybe it's the way to go, if you see fit.
P.

@nexbit I agree. I like absolute separation.
We'll close this issue, and open up a new one to keep it clean. I'll reference back here.


Netatmo thermostat

Hi, I hope someone can solve this problem.
I have broadlink mini and when I connect the air conditioning it gives me the temperature value 0ºC.
I also have the netatmo thermostat to control the heating through the boiler.
Is it possible that the broadlink read the current temperature through netatmo?
Another problem is that when I ask Siri to turn on the heating at 24ºC, he starts the air conditioning and the boiler, when the only thing I want is for the boiler to start. Is it possible that siri understands that when I want to warm up just turn on the boiler and when I want to cool just turn on the air?
Thanks to all for the help

you may can try to get the temperature using node.red and putting on a MQTT topic - readable, you know, by this plugin.


MediaPipe KNIFT knift_labelmap.txt

How to get the correct tab order？
In the knift_labelmap.txt  I thought it was
1-XLP
2-AJS
3-LHQWJN
4-ZZM

but it turned out to be
4-ZZM
3-LHQWJN
1-XLP
2-AJS

I used my own image to build template_matching.aar knift_index.pb mobile_cpu.binarypb but I don't know how to fill in knift_labelmap.txt correctly, when I run it in android studio, I don't know how to make it correct Modified knift_labelmap.txt.

@hbyjw In the template creation directory what is the sequence of image files?
bazel-bin/mediapipe/examples/desktop/template_matching/template_matching_tflite \ --calculator_graph_config_file=mediapipe/graphs/template_matching/index_building.pbtxt \ --input_side_packets="file_directory=<template image directory>,file_suffix=png,output_index_filename=<output index filename>"
That particular sequence should be taken in knift_label.txt file.

@PrinceP
template creation directory

knift_label.txt

If I put it in android studio according to the settings in knift_label.txt, the order displayed in the phone is wrong

I would like to know if the knife_label.txt is created automatically, or if I create it myself. If it was me, where should I put the knift_label.txt when I create the knife_index.pb

@PrinceP
https://github.com/google/mediapipe/issues/3133#issuecomment-1060133736
The correct name of the item in the picture is 3-LianHuaQingWenJIaoNang

@hbyjw

The knift_label file has to be given by the user only.

Step 3: Replace mediapipe/models/knift_index.pb with the index file you generated, and update mediapipe/models/knift_labelmap.txt with your own template names.

The index is made here. Can you put some debug statements in calculator and check the sequence of indexes taken. https://github.com/google/mediapipe/blob/b899d17f185f6bcbf3f5947d3e134f8ce1e69407/mediapipe/calculators/video/box_detector_calculator.cc#L51

@PrinceP
I don't know much about C++, but I'll give it a try

Is image #3 the only wrong detection? if you shoot the image #1 xianglianpian, i wonder what is detection result from the demo?

@yege

@yegenzhi

that's indeed an interesting bug.
when you run index_building.pbtxt, can you look out for the logs from this file?
https://github.com/google/mediapipe/blob/e6c19885c6d3c6f410c730952aeed2852790d306/mediapipe/calculators/util/local_file_pattern_contents_calculator.cc#L60
you can just copy paste the logs here.

Where can I see the log, I don't see it in the terminal.

ok.
i believe if you add the following line to LN54 of this file, this may fix your issue.
std::sort(filenames_.begin(), filenames_.end());
can you please try?

https://github.com/google/mediapipe/issues/3133#issuecomment-1060212351
I wrote the record to txt

you can add " --alsologtostderr" to your command to make the logs show in terminal.
yes. the bug is because the match file api at LN50 does not guarantee the matched result in exact order.
we will fix this in the next release.
Thanks for the catch.

Okay thank you.
I will add the code and try again.

https://github.com/google/mediapipe/issues/3133#issuecomment-1060220075
After adding the code, the correct order is displayed.


Error grant --sdk-auth role Static web app in Cloud Shell

Following instructions
https://docs.microsoft.com/en-us/azure/storage/blobs/storage-blobs-static-site-github-actions?tabs=userlevel
to
"Set up a GitHub Actions workflow to deploy your static website in Azure Storage"
Error running script in Azure Status Web App CLI
az ad sp create-for-rbac - ...
results in

Option '--sdk-auth' has been deprecated and will be removed in a future release.

in full:
The underlying Active Directory Graph API will be replaced by Microsoft Graph API in a future version of Azure CLI. Please carefully review all breaking changes introduced during this migration: https://docs.microsoft.com/cli/azure/microsoft-graph-migration
Option '--sdk-auth' has been deprecated and will be removed in a future release.
Creating 'contributor' role assignment under scope '/subscriptions/98c67d1e-1b85-4c28-8b54-f7a634f94ef5/resourceGroups/'
  Role assignment creation failed.

Invalid scope. Please use --help to view the valid format.

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

ID: 7aef74eb-a8c5-0251-82f6-0cba6f58d712
Version Independent ID: fa7b29e1-f341-e7ac-f4a8-d04602e2cd0f
Content: Use GitHub Actions to deploy a static site to Azure Storage
Content Source: articles/storage/blobs/storage-blobs-static-site-github-actions.md
Service: storage
Sub-service: blobs
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@OzBob I apologize for the delay. The underlying GitHub Action Login action still uses the --sdk-auth flag, which creates the warning. We can't update the article until the Login active is updated. Here's a related issue: https://github.com/Azure/login/issues/223

#please-close
Closing this because it is a product issue and not a documentation one right now.


Fixes error code 42803 when trying to query a view with a column named count

Just adds a test case and prefixes the uses of the count function with pg_catalog.

This is great, thank you!
Could you add a note to the changelog in this pull request as well please?


Support read-only computes in 'compute_ctl'

Extracted from PR #3886 for more visibility.
There are no tests included in this PR, but after PR #3886, this will be exercised by all the current python tests that we have for read-only nodes.

Superseded by PR #4136


Create option to preserve Google Sheet dynamic data in either JSON or CSV format in repo

Create option for users who wish to transform and preserve their dynamic Google Sheet data into more stable CSV files to be uploaded into the GitHub repo, to create a more stable version of the map.

1, even as JSON. If I get some spare time I'd love to help play around with loading the configuration and data from json instead of google sheets,

I think this is a bug, but maybe it just needs more testing on different browsers and computers.
Problem: How do I know if the map contents are accurately pulled from Google Sheets or CSV? I ran a test and labeled them differently (with either Sheets or CSV in the map title).
Chrome did not correctly read the CSV tables, and continued to display old map data (from the browser cache?), even after I did several hard refreshes (on Mac: command-shift-R)
By contrast, my FFox correctly displayed the CSV table contents. However, I have set up my FFox browser to store no history, cache, etc. between sessions.

Another problem: with this new code, Chrome displays this error:
jquery-3.1.0.min.js:4
Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.

Another problem: with this new code, when I erase the CSV folder and tables inside it, Chrome (but not FFox) displays this error:
Failed to load resource: the server responded with a status of 404 (Not Found)
https://jackdougherty.github.io/leaflet-maps-with-google-sheets/csv/Options.csv

Jack, first and third are connected. Google Chrome does not let you work with local .geojson or .csv files for security reasons (hence XMLHttpRequest error). Thus, even when you have an Options csv in the folder, it fails to load it and hence reads the Tabletop version. It won't happen if you load it from GitHub.
"Failed to load resource" will always be an error in console if Options.csv does not exist. With JavaScript, there is not really a nice way to check if file exists – so you have to load it anyway and see if it returns failure or success.
As for jquery not supporting synchronous requests, I will try to change it to asynchronous.

It may be worth emphasizing that once csv files are cached, users may not get updated csv from the server unless they clear cache (I was changing the contents of the csv file on my local machine and did not see any changes in Safari when refreshing the page, only Private Mode – and also clearing cache – helped).

It may be worth emphasizing that once csv files are cached, users may not get updated csv from the server unless they clear cache (I was changing the contents of the csv file on my local machine and did not see any changes in Safari when refreshing the page, only Private Mode – and also clearing cache – helped).


Error 13

Whenever I point Legilimens.exe to my save game folder
C:\Users-user-\AppData\Local\Hogwarts Legacy\Saved\SaveGames
It comes back with this error
.sav file path: C:\Users\-user-\AppData\Local\Hogwarts Legacy\Saved\SaveGames\241704685
Legilimens encountered an unexpected error
Traceback (most recent call last):
  File "legilimens.py", line 280, in main
  File "legilimens.py", line 259, in run
  File "legilimens.py", line 185, in _read_save
  File "legilimens.py", line 124, in __enter__
PermissionError: [Errno 13] Permission denied: 'C:\\Users\\-user-\\AppData\\Local\\Hogwarts Legacy\\Saved\\SaveGames\\241704685'
If the problem persists, you can create an issue on GitHub with your save file attached:
https://github.com/Malin001/Legilimens-Hogwarts-Legacy-Collectible-Finder/issues

Press enter to close this window...

EDIT: edited username out

Hi RandomGuy908,
Use file path NOT folder path
you have to put in the path to the file (you just gave the path to the folder):
.sav file path: C:\Users\-user-\AppData\Local\Hogwarts Legacy\Saved\SaveGames\241704685\HL-00-XY.sav
Decompress before execution
Then you will probably encounter this error message:
SQLite was unable to read parts of the database
You need to first decompress the save game file using this tool: https://www.nexusmods.com/hogwartslegacy/mods/1983. Follow the usage guide until step 6, from then on do instead:
step 7: Decompress your save file, e.g: .\hlsaves.exe -d HL-03-05.orig HL-03-05.sav
step 8: execute Legilimens
Use a backup
I suggest you copy the SaveGame folder to a convenient location. Use that copy as a target for HL Save Tool and Legilimens. If things go south you keep your save game, also you can skip step 9 an 10 of the usage guide.
Happy achievement farming :)
Johannes


[Lua]Add Lua DG pressure and docking subsystems + associated API

This PR adds the following OAPI functions to Lua :
oapiGetObjectType
oapiGetGbodyParent
oapiGetGbodyByName/Index
oapiAcceptDelayKey
VESSEL::GetFlightStatus
VESSEL::AddParticleStream
VESSEL::DelExhaustStream
The Lua DG example has been updated to use these.

Ready for merge


perf: remove unnecessary allocation in check_next_block

Motivation
It seems that rust wasn't able to resolve the correct .zip(...) because rayon has it under IndexedParallelIterator which luckily .par_values(...) implements. This removes the unnecessary reallocation of transactions.


React roles don't embed when discord option is flagged.

Description:
When a user has Settings -> Text & Images -> Show embeds and preview website links pasted into chat - disabled. The end user is not able to see the notification from the bot in a private message. This can cause reaction responses from a user to not show up in something such as an embedded raid sheet.
To Reproduce
Steps to reproduce the behavior:

Go to discord settings, select Text & Images, ensure the slider for Show embeds and preview website links pasted into chat - is disabled.
React to event sign up
Expected result: bot sends a DM with a requested digit response, that is not visible to the end user.
End user is not able to see the embedded event sign up sheet, nor are other users.

Expected behavior
The DM from the bot should provide context to ensure that a response is requested.
Additional context
Suggest adding a non-embedded text-based message in DM when a response is requested to confirm the event sign up.

Hi,
i'm a little bit confused. This bot does not send any dms on raid sign up. Are you sure that you opened your issue in the correct repository?
I will definetly look into you issue, because users with disabled embeds is a situation i've never thought of before.


Upgrade issue:  Old versions of JDBC libraries are not removed on upgrade

Describe the bug
When running a CMS upgrade, new versions of JDBC drivers are correctly copied into /jetty/base/lib/jdbc.  The problem is that the older version of the JDBC driver jars are not being removed by the upgrade process.  For example, if you look at the below directory listing, there are two versions of the Microsoft SQL Server jdbc driver, and because they both use the same driver class name, there is no guarantee that Jetty will pick the newer driver version.
``
Directory of C:\Rhythmyx\jetty\base\lib\jdbc
09/06/2023  01:19 PM              .
09/06/2023  01:20 PM              ..
07/19/2023  10:47 AM         3,853,921 db2jcc4.jar
07/19/2023  10:47 AM             1,013 db2jcc_license_cu.jar
09/06/2023  01:19 PM         3,232,770 derby-<IP_ADDRESS>.jar
09/06/2023  01:19 PM           588,394 derbyclient-<IP_ADDRESS>.jar
09/06/2023  01:19 PM           272,626 derbynet-<IP_ADDRESS>.jar
09/06/2023  01:19 PM           317,816 jtds-1.3.1.jar
07/19/2023  10:47 AM           294,726 jtds.jar
07/19/2023  10:47 AM           960,558 mssql-jdbc-7.1.3.jre8-preview.jar
09/06/2023  01:19 PM         1,327,642 mssql-jdbc-9.2.1.jre8.jar
09/06/2023  01:19 PM         2,739,670 ojdbc6-<IP_ADDRESS>.jar
``
To Reproduce
Steps to reproduce the behavior:

Install a 7.3.2 version of Rhythmyx
Upgrade it to the latest with java -jar perc-distribution-tree.jar
Check the \jetty\base\lib\jdbc folder
You will see duplicate drivers new and old for mssql.

Expected behavior
Only the current version of each JDBC driver should be in the jdbc lib folder.
Versions Affected

8.0.3
8.1.2
8.1.3

Additional context
I looked at the installation code.  This can't be done as part of the current ant installation process as all of the jars are on the classpath that is passed to ant.  I think this needs to be done from the Java process that controls the installation.  We can't just blow away the folder because a customer may have added their own drivers there.

Related to #1035

@natechadwick Following are the old jars or I am missing some?
"mssql-jdbc-7.1.3.jre8-preview.jar",
"jtds.jar",
"db2jcc_license_cu.jar",
"db2jcc4.jar"


Cirrus task failed: Error getting access token for service account, Invalid JWT Signature

Error getting access token for service account: 400 Bad Request POST https://oauth2.googleapis.com/token {"error":"invalid_grant","error_description":"Invalid JWT Signature."}

https://cirrus-ci.com/task/6024081108631552
The task steps succeeded.

Unfortunately this does not seem actionable from flutter side. @fkorotkov do you know what happened here? I could not find anything in the flutter-cirrus project logs.

Do you frequently see this error? I've never seen it before and trying to google it also didn't provide much of results.
Under the hood it throws when a access token is trying to being refreshed in the underlying google-auth-library-oauth2-http library which is used by Kubernetes.

This is the first time I've seen this issue. @jmagman is this happening frequently?

I've only seen it once.

Lowering priority, it passed on rerun.

It just happened three times on the same PR:
https://cirrus-ci.com/task/4845811306594304
https://cirrus-ci.com/task/5408761260015616
https://cirrus-ci.com/task/6534661166858240

And again for me:
https://cirrus-ci.com/task/4663663723282432

@fkorotkov do you get any logs about this error in the scheduler side? I can't find any logs related to the problem in the GCP kubernetes side.

@fkorotkov this is happening a lot more frequently: e.g. https://github.com/flutter/flutter/pull/84777

I've been investigating it and just deployed a retry logic. But I still not quite understand why it suddenly started happening on your account (gcp related code hasn't been in months). Could you please check you API quotas on the GCP project just to check if there are any abnormalities. Also just to confirm you didn't change anything about your cluster or service account, right?

Just an update: I do see from the logs that a retry helps. On the seconds try getting a token to GKE cluster always succeeds.

The entire GCP project permissions were updated but that was a couple of months ago.
Kubernetes quota is below 1% usage:

I see some kubernetes container errors like the following:
"2021/06/17 18:41:25 Error retrieving gcp metadata: rpc error: code = PermissionDenied desc = Request had insufficient authentication scopes."

Can you please share the timestamp from one of the Cirrus scheduler logs to try to correlate?

Here are a few timestamps when it failed to get the token in EDT time:
2021-06-17 13:16:00.532 EDT
2021-06-17 13:35:37.614 EDT
2021-06-17 14:17:09.673
2021-06-17 14:48:46.892

Not sure about your logs timezone but seems it's not relates (no logs on cirrus side at 41 minutes)

Raising priority again, this is being seen on lots of PRs, blocking rollers, etc.

Can you add links to new tasks? The added retry should've helped. At least I see so from the logs/stats.

https://cirrus-ci.com/task/6165407628263424

I've added more retries but it will just wait until the requests succeeds. So it's just mitigating the issue rather then a proper solution...
@godofredoc could you please provide your clusters configuration both for test-cluster and build-32-cluster? You can email me details of them if you don't want to post publicly.

Sounds good, let me collect them and will send them in a few mins.

@fkorotkov I sent you the configs by mail, please let me know if you need some other parts of the configs.

New bunch of failures on multiple repos:
engine: https://cirrus-ci.com/task/5601630021222400
framework: https://github.com/flutter/flutter/pull/84997/checks?check_run_id=2878318668

Yeah, I’m waiting for @godofredoc response in our email thread. Since these tasks are running on your infrastructure I have limited options to debug on my own. My guess is that there are issues with the credentials / or rate limits on token creation.

The service account key being used expired on 06/13 we'll need to create a new one and update the secret in the different flutter repositories.

Also occurring here: https://github.com/flutter/flutter/pull/84298/checks?check_run_id=2878475301

and here https://github.com/flutter/engine/runs/2878211362 (on dart->engine roll)

Could we get some kind of comprehensive audit of our Flutter/Cirrus configuration? This is the second time in as many weeks that the plugins tree has been closed for a day (or longer) due to a hard-to-debug failure in our Cirrus tasks due to the way they are running on our infrastructure.
If we can't get to a high degree of confidence that this kind of thing isn't going to keep happening, it's not clear to me that the hybrid configuration is actually an improvement over the previous setup.

Should we start planning on migrating plugins to LUCI?

Sounds good, let's create a bug to track the requirements for a potential migration.


More information when reading a book

Develop the "More Information" feature, i.e. the timeline feature at the bottom of the reading screen.
Be careful about the layout of the tooltip (chapter and page), the current display is not conform to the design (the original HTML is not perfect either)

Done in March 2019


Figure out a way to handle whitespaces more strictly

Currently, the following statements are equivalent:
let x = a[0]

let x = a   [0]

and
let x = a
[0]

Ideally, only the first one should be valid.
Originally proposed by @bobbinth in https://github.com/0xPolygonMiden/air-script/issues/123#issuecomment-1428693626

Maybe we can just add semi-colons to address this.

Largely closed by #353. The only outstanding thing is adding commas to separate arms of the match statement.
sydhds will open a PR for this soon.


Closes: #1

Updated validations.py python script.
Fixed the behavior of the validate_user function in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!


Deal with truncated responses

If the response exceeds the token limit, which is set in constants.py, the response is simply truncated and cannot be parsed as a JSON. Currently there is no mechanism for identifying and dealing with truncated responses. I have the token limit set to its maximum (8192), but this doesn't guarantee that this problem will be avoided. Users may wish to reduce this value to limit costs, which will make truncated responses more likely.
To do:

[ ] Modify the system prompt to request an unambiguous symbol to be added at the beginning and end of the response. These can be removed before parsing the JSON. But their absence would signal that the response is truncated
[ ] If the response is truncated, automatically send a message back to the API stating that the response was truncated, requesting another, more concise response, and including the current token limit for reference.
[ ] Post a message to the chat saying something like "I'm sorry, my first response didn't fit into the current token limit of {token_limit}. I'm going to try being more concise. You might also consider increasing the token limit in the settings".
[ ] Limit the number of retries using the MAX_RETRIES variable in constants. Come up with a message for the chat that explains that we've reached the maximum number of retries and couldn't come up with a response that fit within the token limit. The user could then ask a new question.

This has been addressed. Closing


Make slick and shapeless a runtime dependency

The project build depends on slick and shapeless, naturally. But should they be run-time dependencies to allow the end user to make minor version changes?
Not sure what the best practice is here.  For example, as of right now while I'm writing this issue, we've published slickless for Slick 3.2.0-M2. If I want to use this with 3.2.0 final I see:
[warn] There may be incompatibilities among your library dependencies.
[warn] Here are some of the libraries that were evicted:
[warn] 	* com.typesafe.slick:slick_2.12:3.2.0-M2 -> 3.2.0
[warn] Run 'evicted' to see detailed eviction warnings

I think we can mark these lines as % runtime (or something like that):
https://github.com/underscoreio/slickless/blob/master/build.sbt#L21-L22
I'll do that, unless anyone wants to point to th error of my ways :-)

Eviction isn't problem, you can make changes to version as long API does not change with version change. Adding them as Runtime would mean that everybody must declare dependencies and their version, now upgrading is optional.
For most Scala projects dependencies aren't declared as runtime.
I like to declare every dependency separately so I can use https://github.com/rtimush/sbt-updates plugin to see updates (and then check agains security update notices). I have 30 evection notices, best example I can give is this: org.slf4j:slf4j-api:(1.7.21, 1.6.1, 1.7.7, 1.7.22, 1.7.10, 1.7.16, 1.7.12) -> 1.7.24 over 20 packages depend on this with different versions 😄


[Removal]: Welcome to buy bobcat any cheating will not be joining blacklist!

Hotspot Name
112121
Hotspot b58 Address
123456
Discord Handle
No response
Hotspot Manufacturer
bobocat
Removal Reason
111
Modifications
Yes
Extra forwarders
Yes
Extra antennas
Yes
Additional Information
Welcome to buy bobcat any cheating will not be joining blacklist!
Take the antenna can perfect witness! That's great!
Blacklist experts are received money, not to report the bobcats! Someone reported that doesn't matter, they turn a blind eye!
Other brands do not buy, buy the bobcats is the right choice!

FUD. People contributing receive nothing.


Modal freezes when swiping down or up at an angle.

My environment:
-iPhone 8
-react-native-cli: 2.0.1
-react-native: 0.52.0
-react-native-image-view: 2.0.10
When I slide down or up at an angle it sometimes freezes the modal. I think it get's confused as if you are trying to close out of the modal or if you are trying to swipe to the next image. It freezes until you hit the 'X' in the upper right-hand corner. I replicated the issue once on the android expo example online, but it took a while to replicate.
My Usage:
<ImageView
images={this.state.images}
imageIndex={this.state.idx}
isVisible={this.state.visible}
animationType='slide'
onClose={()=>this.imageView()}
renderFooter={(currentImage) => (My footer)}
/>

Faced with the same, try to fix in several days

Having the same problem as well. Is there anyway to disabled the slide up/down to close functionality?

Partially fixed in 2.1.0 Its probably need to rethink all logic to make it work correct. Going to do it in a week.

Hey guys and sorry for long time not responding.
This repo has been abandoned a little bit but I have a good news: in my current company we needed the same functionality that this module provides so we completely rewrote it and made a lot of improvements. So there should be no more jumpy scrolling and finally zoom on Android works as expected! We also have optimised module it terms of performance for large lists of photos.
Please check it: https://github.com/jobtoday/react-native-image-viewing
The API is very close to this one so there should be a lot of problems to replace.
There are still some features missing and we are planning to add it very soon.
Thanks a lot for using this module.


Job should fail if Collect yields no files/file items

I'll leave #29 open for longer term quality assurance.
Presently, if Collect yields only errors, then Process never finishes.
I'll have a look at Process to see if it can finish despite receiving no workable data (might be in "completable" function).
That said, once all tasks are complete, the job status should be failed.

Can test with Buenos Aires, currently.

Kingfisher Collect increments kingfisher_process_expected_files_count for both File and FileError. Let's say only FileError's are sent.
Kingfisher Process creates a CollectionFile for FileError's. It creates a note, but otherwise it doesn't create any further rows.
Kingfisher Process's compilable function would return True if expected_files_count were 0. As it's not, it then checks the collection's data_type. Since only FileError's were received, the data_type isn't set, so it returns False.

Okay, I've made changes to Collect and Process such that errors aren't counted.
We can now see our first failed empty collection: https://data.open-contracting.org/admin/data_registry/job/2190/change/?_changelist_filters=status__exact%3DRUNNING


OPRO, DPO don't work with Mixtral-8x22B. FSDP + QLORA & bigstral-ds-zero3

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
DPO/ORPO training should run successfully
Current behaviour
Models tested (don't have any issues with inference)

mistral-community/Mixtral-8x22B-v0.1
0-hero/Matter-0.2-8x22B (FSDP + QLORA finetune of the above)

Machines tested (tried each type from multiple providers)

8xA100 (80GB)
8xH100 (SXM)

Images tested

winglian/axolotl-runpod:main-latest (as of 18 Apr 2024)
winglian/axolotl-runpod:main-py3.11-cu121-2.1.2
winglian/axolotl-runpod:main-py3.10-cu118-2.1.2
runpod/pytorch:2.0.1-py3.10-cuda11.8.0-devel-ubuntu22.04

Ran tests with all the combinations mentioned above
Both issues mentioned below happen for both ORPO & DPO
Issue 1 - FSDP + QLORA
https://github.com/OpenAccess-AI-Collective/axolotl/issues/1494
Issue 2 - bigstral-ds-zero3
Happened anytime before the first 20 steps. Tried reducing the below to 1 but the issue persists.
gradient_accumulation_steps: 1
micro_batch_size: 1

Training hangs and eventually stops with a NCCL timeout https://github.com/huggingface/accelerate/issues/314
GPU util also falls once it hangs, example below

Steps to reproduce
Start training with any of the configs below
FSDP + QLORA config
base_model: mistral-community/Mixtral-8x22B-v0.1
model_type: AutoModelForCausalLM
tokenizer_type: LlamaTokenizer

load_in_8bit: false
load_in_4bit: true
strict: false

rl: dpo
datasets:
  - path: argilla/ultrafeedback-binarized-preferences-cleaned
    split: train
    type: chatml.ultra

dpo_beta: 0.1

chat_template: chatml
default_system_message: You are a helpful assistant

dataset_prepared_path: data
val_set_size: 0
output_dir: output

sequence_len: 8192
sample_packing: false
pad_to_sequence_len: false

adapter: qlora
lora_model_dir:
lora_r: 32
lora_alpha: 16
lora_dropout: 0.05
lora_target_linear: true
lora_fan_in_fan_out:
lora_modules_to_save:
- embed_tokens
- lm_head

gradient_accumulation_steps: 8
micro_batch_size: 4
num_epochs: 1
optimizer: adamw_bnb_8bit
lr_scheduler: cosine
learning_rate: 0.0002

train_on_inputs: false
group_by_length: false
bf16: auto
fp16:
tf32: false

gradient_checkpointing: true
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: true

warmup_steps: 10
evals_per_epoch: 4
eval_table_size:
eval_max_new_tokens: 128
saves_per_epoch: 1
debug:
weight_decay: 0.0
fsdp:
   - full_shard
   - auto_wrap
fsdp_config:
 fsdp_limit_all_gathers: true
 fsdp_sync_module_states: true
 fsdp_offload_params: true
 fsdp_use_orig_params: false
 fsdp_cpu_ram_efficient_loading: true
 fsdp_transformer_layer_cls_to_wrap: MixtralSparseMoeBlock
 fsdp_state_dict_type: FULL_STATE_DICT
 fsdp_auto_wrap_policy: TRANSFORMER_BASED_WRAP

special_tokens:
  bos_token: "<s>"
  eos_token: "<|im_end|>"
  unk_token: "<unk>"
tokens:
  - "<|begin_func|>"
  - "<|end_func|>"
  - "<|begin_func_response|>"
  - "<|end_func_response|>"
  - "<|im_start|>"
  - "<|im_end|>"

bigstral-ds-zero3 config
base_model: 0-hero/Matter-0.2-8x22B
model_type: AutoModelForCausalLM
tokenizer_type: LlamaTokenizer

load_in_8bit: false
load_in_4bit: false
strict: false

unfrozen_parameters:
  - ^lm_head.weight$
  - ^model.embed_tokens.weight$
  - model.layers.4[4-9]+.block_sparse_moe.gate
  - model.layers.4[4-9]+.block_sparse_moe.experts
  - model.layers.5[0-5]+.block_sparse_moe.gate
  - model.layers.5[0-5]+.block_sparse_moe.experts

model_config:
  output_router_logits: true

rl: orpo
datasets:
  - path: mlabonne/orpo-mix-40k
    split: train
    type: orpo.chat_template

chat_template: chatml
default_system_message: You are a helpful assistant

dataset_prepared_path: data
val_set_size: 0
output_dir: output

sequence_len: 8192
sample_packing: false
pad_to_sequence_len: false

gradient_accumulation_steps: 8
micro_batch_size: 4
num_epochs: 1
optimizer: adamw_bnb_8bit
lr_scheduler: cosine
learning_rate: 0.0002

train_on_inputs: false
group_by_length: false
bf16: auto
fp16:
tf32: false

gradient_checkpointing: true
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: true

warmup_steps: 10
save_total_limit: 1
save_steps:
debug:
deepspeed: deepspeed_configs/zero3_bf16_cpuoffload_params.json
weight_decay: 0.0
fsdp:
fsdp_config:
special_tokens:
  bos_token: "<s>"
  eos_token: "<|im_end|>"
  unk_token: "<unk>"
tokens:
  - "<|begin_func|>"
  - "<|end_func|>"
  - "<|begin_func_response|>"
  - "<|end_func_response|>"
  - "<|im_start|>"
  - "<|im_end|>"

Config yaml
No response
Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10,3.11
axolotl branch-commit
main/0eadfc8
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

@winglian raised a new issue as mentioned in the other discussion


sluggish/invisible keyframes animation when both translateX transforms are being applied

Fixes #2
Stacking transforms wasn't the issue.  I thought it had something to do with that as the issue happened when -100% was "done".  But, TIL: "The + and - operators must be surrounded by whitespace." (The reason i was stacking transforms was the calc() i originally used did not include whitespace and failed.)
The "fix" provided by StackOverflow: backface-visibility: hidden;  IDK why it works or if it is the correct or best fix, but it works.

👍


Error in grepcmd

I defined:
nn <silent> <leader>fg <scriptcmd>vim9cmd scope#fuzzy#Grep('rg --vimgrep --smart-case', 1, '', '/path/to/foo')<cr>

Typed aaa  And got these errors:
rg --vimgrep --smart-case a /path/to/foo

rg --vimgrep --smart-case aa /path/to/foo

rg --vimgrep --smart-case aaa /path/to/foo

rg --vimgrep --smart-case aaa\  /path/to/foo
Error detected while processing function <lambda>22:
line    2:
error: rg: regex parse error: from rg --vimgrep --smart-case aaa\  /path/to/foo
error:     (?:aaa\) from rg --vimgrep --smart-case aaa\  /path/to/foo
error:     ^ from rg --vimgrep --smart-case aaa\  /path/to/foo
error: error: unclosed group from rg --vimgrep --smart-case aaa\  /path/to/foo

I could not reproduce. You may have older version. But I found issue with escaping ) that shows up in rg only, and it is fixed. Pl upgrade.

I was on the latest commit. For reference, &shellxescape on windows is set to "&|<>()@^ by standard.

I do not have windows. It looks like escape character should be ^ but I am using \. Can you set grep_echo_cmd option and check what it is sending to the shell? It prints the command string in the commandline.
Can you run your rg ... command on the shell, using proper escape char for all the characters listed in :set shellxescape? and see if it works? And then I can modify this plugin to use appropriate escape for windows.
						*'shellxescape'* *'sxe'*
'shellxescape' 'sxe'	string	(default: "";
				 for MS-Windows: "\"&|<>()@^")
			global
	When 'shellxquote' is set to "(" then the characters listed in this
	option will be escaped with a '^' character.  This makes it possible
	to execute most external commands with cmd.exe.
	This option cannot be set from a |modeline| or in the |sandbox|, for
	security reasons.

						*'shellxquote'* *'sxq'*
'shellxquote' 'sxq'	string	(default: "";
					for Win32, when 'shell' is cmd.exe: "("
					for Win32, when 'shell' is
					powershell.exe: "\""
					for Win32, when 'shell' contains "sh"
					somewhere: "\""
					for Unix, when using system(): "\"")
			global
	Quoting character(s), put around the command passed to the shell, for
	the "!" and ":!" commands.  Includes the redirection.  See
	'shellquote' to exclude the redirection.  It's probably not useful
	to set both options.
	When the value is '(' then ')' is appended. When the value is '"('
	then ')"' is appended.
	When the value is '(' then also see 'shellxescape'.
	This is an empty string by default on most systems, but is known to be
	useful for on Win32 version, either for cmd.exe, powershell.exe, or
	pwsh.exe which automatically strips off the first and last quote on a
	command, or 3rd-party shells such as the MKS Korn Shell or bash, where
	it should be "\"".  The default is adjusted according the value of
	'shell', to reduce the need to set this option by the user.  See
	|dos-shell|.
	This option cannot be set from a |modeline| or in the |sandbox|, for
	security reasons.

rg --vimgrep --smart-case a /path/to/foo
rg --vimgrep --smart-case aa /path/to/foo
rg --vimgrep --smart-case aaa /path/to/foo

I believe redir was done with grep_echo_cmd on.

FYI, I'm not sure I understood your request, by when I call
:shellscape('rg --vimgrep --smart-case a /path/to/foo \ " & | < > ( ) @ ^')

I get
"rg --vimgrep --smart-case a /path/to/foo \ "" & | < > ( ) @ ^"

Try :
C:\Users\pedro> rg --vimgrep --smart-case a\  /path/to/foo

and see if it gives the correct results. Then try other escaped characters like & | ( )  etc. instead of \.
If you are not getting correct results running rg directly in shell, try escaping \ with ^ or ( and `)' around. Read the doc I posted above.
Maybe:
C:\Users\pedro> rg --vimgrep --smart-case a^\  /path/to/foo

or:
C:\Users\pedro> rg --vimgrep --smart-case a\\\  /path/to/foo

you have to experiment what works.

rg --vimgrep --smart-case "a "  /path/to/foo
is the only one that worked.

What do you get in Vim when you do :set sxe? and :set sxq?

Try now, and see if it improves.

Try now, and see if it improves.

It works now, thanks!

What do you get in Vim when you do :set sxe? and :set sxq?

Honestly, I think it would be better to just check 'has('win64')'

Will about powershell?

Precisely because of powershell lol, it also only works using quotes.


OrientDB SQLs for compatibility test

Thank you for your helps!

@wuxianliang Do you know, are there any available OrientDB SQL tests?

I have write a post in OrientDB Google Group. This issue is for community's helps. The post is about OrientDB SQL tests and need to be approved to appear.

In my understanding, OrientDB SQL tests mean concrete OrientDB SQL statements. Right? @lvca @luigidellaquila

Yes. The real samples make the development easier, because you know will be in result
Отправлено с iPhone

17 мая 2015 г., в 12:21, Xianliang Wu<EMAIL_ADDRESS>написал(а):
In my understanding, OrientDB SQL tests mean concrete OrientDB SQL statements. Right? @lvca @luigidellaquila
—
Reply to this email directly or view it on GitHub.

Hi @wuxianliang
I don't know exactly what kind of tests you are thinking about, if you need practical examples you can start from something like this
http://pizzaconnections.net/2015/03/27/148/
if you need real test cases, there are a bunch of them here ;-)
https://github.com/orientechnologies/orientdb/tree/master/core/src/test/java/com/orientechnologies/orient/core/sql
Luigi

@luigidellaquila Thank you very much, this is exactly what we need!
I would be happy to talk to you sometime on the next week, when we will have first results.

@agershun I'll be happy to talk to you, let me know when you have something
Luigi

Did @luigidellaquila and @agershun ever had a chat?


Fix build errors due to missing RPM packages

When a clean build is executed, it fails with 404 errors because RPM
packages for qemu* listed in WORKSPACE are gone. This patch bumps
versions to those present in fedora repos.
Also, SHA256 sum for virtio-win was incorrect, so fixing it too.
NONE

/cherrypick release-0.17


Build failure on file without any declarations only scss import

Describe the bug
A tsx file that only imports css will fail to build.
This does not build.
import './color-pallette.scss';

This does build:
import './color-pallette.scss';

export const HAY = '2';

Reproduction
url
Steps to reproduce
See comments
System Info
[vite:dts] Start generate declaration files...
computing gzip size (1)...[vite:dts] Start rollup declaration files...
Analysis will use the bundled TypeScript version 5.0.4
*** The target project appears to use TypeScript 5.1.3 which is newer than the bundled compiler engine; consider upgrading API Extractor.
[vite:dts] Internal Error: Unable to determine module for: /usr/src/app/services/design-system/dist/src/ColorPallete/index.d.ts

You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
✓ built in 24.13s
error during build:
Error: Internal Error: Unable to determine module for: /usr/src/app/services/design-system/dist/src/ColorPallete/index.d.ts

You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
    at ExportAnalyzer._getModuleSymbolFromSourceFile (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/analyzer/ExportAnalyzer.js:168:15)
    at ExportAnalyzer.fetchAstModuleFromSourceFile (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/analyzer/ExportAnalyzer.js:69:35)
    at ExportAnalyzer._fetchSpecifierAstModule (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/analyzer/ExportAnalyzer.js:693:41)
    at ExportAnalyzer.fetchAstModuleFromSourceFile (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/analyzer/ExportAnalyzer.js:112:65)
    at AstSymbolTable.fetchAstModuleFromWorkingPackage (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/analyzer/AstSymbolTable.js:83:37)
    at Collector.analyze (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/collector/Collector.js:155:51)
    at Extractor.invoke (/usr/src/app/common/temp/node_modules/.pnpm/@microsoft+api-extractor@7.38.5_@types+node@20.9.0/node_modules/@microsoft/api-extractor/lib/api/Extractor.js:143:19)
    at rollupDeclarationFiles (/usr/src/app/common/temp/node_modules/.pnpm/vite-plugin-dts@3.6.4_@types+node@20.9.0_typescript@5.1.3_vite@5.0.5/node_modules/vite-plugin-dts/dist/index.cjs:272:41)
    at Object.writeBundle (/usr/src/app/common/temp/node_modules/.pnpm/vite-plugin-dts@3.6.4_@types+node@20.9.0_typescript@5.1.3_vite@5.0.5/node_modules/vite-plugin-dts/dist/index.cjs:933:13)
    at async Promise.all (index 0)
The script failed with exit code 1

Validations

[X] Read the FAQ.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Although without export {}, it does not occur any error when building.
It you thing this issue still exists, you should provide a reproduction.


Daemon should have been opt in, not opt out with --no-daemon, add config opt out in turbo.json

Which project is this feature idea for?
Turborepo
Describe the feature you'd like to request
Add config opt out in turbo.json
{
"$schema": "https://turbo.build/schema.json",
"daemon": false
}

Describe the solution you'd like
A config option to disable by default.
Describe alternatives you've considered
I run with --no-daemon as an alt. This is due to the daemon not really working correctly. It constantly spits out erroneous errors, specifically error Unknown. It clutters the output with confusing messages and doesn't improve performance. I also don't need remote caching. As mentioned the daemon should be opt in and configure, not opt out.

Absolutely needed !
Currently ugly like this :

We're about to rework the messaging around daemon so it will no longer be as visible. cc @anthonyshew
Is the noise of the daemon errors the primary thing you're trying to avoid?

@chris-olszewski Not completely the noise. I'd like to run turbo with a default --no-daemon. I am sure I can work up a Powershell or nix alias to do it, but IMHO it should be default opt-in not opt-out. Especially when it offers no benefit that I can see or tell. For large teams that may not be the case, but they can opt-in.
I keep trying the daemon and it does not work on my mono-repo. It doesn't cache as it should and I don't have the time troubleshoot it. I don't have the time to give an example mono-repo so you can see it. Running with --no-daemon is the only way to keep sanity and have accurate meaningful console messages. The daemon just adds confusion for no real benefit.

Also, on Windows it leaves me with directories I can't delete without killing turbo.exe.  I put --no-daemon everywhere but somehow if I run npm run build with --no-daemon in my package.json instead of running turbo --no-daemon run build it creates these immutable directories again. The only reliably solution I've found is to reboot (or maybe logout) which is silly.

I can confirm - I also experience systematic problems with turbo.exe hanging in memory and blocking node_modules from being wiped out whenever I need it, not to mention redundant console messages about "standalone daemon not being found" or alike.
In my view, as a consumer you don't have to know or note these internal hiccups in order to use the tool "as is", and "as is" should better be configured to work frictionlessly from the get go. This is not to discourage or insult anybody working on the tool, nothing like that. I am just wondering why it is the way it is, and if we could do anything about it.

I started seeing the failed to contact turbod. Continuing in standalone mode: connection to turbo daemon process failed.
warning after upgrading turbo from 1.10.14 to 1.10.16.
turbo daemon {clean,start,restart} doesn't help. I also tried:

removing all .turbo directories in the repository
removing the referenced temporary file under /var/folders

Nothing helped. I also note that the temporary files referenced in the output don't seem to change even though I deleted the directory. A different directory name than a28b2d553bc195a3 exists under /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod:
$ ls /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod
b1f2d416695e7a92

Output from turbo -vvv ... command:
2023-10-21T22:44:35.716+0200 [DEBUG] turborepo_lib::shim: Global turbo version: 1.10.16
2023-10-21T22:44:35.718+0200 [DEBUG] turborepo_lib::shim: Repository Root: <repo_root>
2023-10-21T22:44:35.718+0200 [DEBUG] turborepo_lib::shim: Local turbo path: <repo_root>/node_modules/turbo-darwin-arm64/bin/turbo
2023-10-21T22:44:35.718+0200 [DEBUG] turborepo_lib::shim: Local turbo version: 1.10.16
2023-10-21T22:44:35.718+0200 [DEBUG] turborepo_lib::shim: Currently running turbo is local turbo.
2023-10-21T22:44:35.719+0200 [TRACE] turborepo_lib::execution_state: Found npm as package manager
2023-10-21T22:44:35.719+0200 [DEBUG] turbo: Found go binary at "<repo_root>/node_modules/turbo-darwin-arm64/bin/go-turbo"
2023-10-21T22:44:35.719+0200 [TRACE] turbo: Invoking go binary with {"config":{"apiUrl":null,"loginUrl":null,"teamSlug":null,"teamId":null,"token":null,"signature":null,"preflight":null,"timeout":null,"enabled":null},"global_hash":null,"task_hash_tracker":null,"api_client_config":{"token":null,"team_id":null,"team_slug":null,"api_url":"https://vercel.com/api","use_preflight":false,"timeout":20},"spaces_api_client_config":{"token":null,"team_id":null,"team_slug":null,"api_url":"https://vercel.com/api","use_preflight":false,"timeout":20},"package_manager":"npm","cli_args":{"api":null,"color":false,"cpu_profile":null,"cwd":"<repo_root>","heap":null,"login":null,"no_color":false,"preflight":false,"remote_cache_timeout":null,"team":null,"token":null,"trace":null,"verbosity":3,"test_run":false,"command":{"Run":{"cache_dir":null,"cache_workers":10,"concurrency":null,"continue_execution":false,"dry_run":null,"single_package":false,"filter":["<app>"],"force":null,"framework_inference":true,"global_deps":[],"graph":null,"env_mode":"Infer","ignore":[],"include_dependencies":false,"no_cache":false,"no_daemon":false,"no_deps":false,"output_logs":null,"log_order":"auto","only":false,"parallel":false,"pkg_inference_root":null,"profile":null,"remote_only":false,"scope":[],"since":null,"summarize":null,"log_prefix":"auto","tasks":["dev"],"pass_through_args":[],"experimental_space_id":null,"experimental_rust_codepath":false}}}}
2023-10-21T22:44:35.834+0200 [DEBUG] turbo: build tag: rust
2023-10-21T22:44:35.848+0200 [DEBUG] turbo.TurbodClient: starting turbod binary <repo_root>/node_modules/turbo-darwin-arm64/bin/turbo
2023-10-21T22:44:36.849+0200 [DEBUG] turbo.TurbodClient: starting turbod binary <repo_root>/node_modules/turbo-darwin-arm64/bin/turbo
2023-10-21T22:44:37.850+0200 [DEBUG] turbo.TurbodClient: starting turbod binary <repo_root>/node_modules/turbo-darwin-arm64/bin/turbo
2023-10-21T22:44:38.851+0200 [WARN]  turbo: :
  warning=
  | failed to contact turbod. Continuing in standalone mode: connection to turbo daemon process failed.
  | 	To quickly resolve the issue, try running:
  | 	- $ turbo daemon clean
  |
  | 	To debug further - please ensure the following:
  | 	- the process identified by the pid in the file at /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.pid is not running, and remove /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.pid
  | 	- check the logs at /Users/mpereira/Library/Application Support/turborepo/logs/a28b2d553bc195a3-frontend.log
  | 	- the unix domain socket at /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.sock has been removed
  |
  | 	You can also run without the daemon process by passing --no-daemon

 WARNING  failed to contact turbod. Continuing in standalone mode: connection to turbo daemon process failed.
	To quickly resolve the issue, try running:
	- $ turbo daemon clean

	To debug further - please ensure the following:
	- the process identified by the pid in the file at /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.pid is not running, and remove /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.pid
	- check the logs at /Users/mpereira/Library/Application Support/turborepo/logs/a28b2d553bc195a3-frontend.log
	- the unix domain socket at /var/folders/nn/120cjkpx75b8z_jcvjm04_940000gn/T/turbod/a28b2d553bc195a3/turbod.sock has been removed

	You can also run without the daemon process by passing --no-daemon

<rest of the output clipped>

There are likely many daemons running on your development machine. The only way Turborepo's is different is that you know about it.
We're working our towards our daemon becoming properly invisible like most other daemons. Please open an issue if your daemon is being problematic.

@aristofun, my explanation was moreso context for folks who aren't aware of the various background operations that are running on their machines. You're correct that the goal is ensure that the daemon is stable so that folks aren't bothered by errors coming from it, similar to the many other daemons that help developers every day.

Realworld case — once we've spent the whole working day trying to resolve weird critical bug in our system. Until we found that some background daemon (hidden from us) silently messed up our files.

When you say "real world case", do you mean that this is something that happened with the Turborepo daemon? If so, can you open an issue to describe the problem so we can address it? We definitely had some instability in the past around the daemon and are now under the impression that we've resolved those issues.
Also, we're working towards auditing the flags, environment variables, and turbo.json configuration options that exist in Turborepo and this sounds like one that we can put on the list to help out the folks interested here. In light of that, I'll open this issue back up so we have better visibility for this configuration. 👍

I've been experiencing some sort of leak in the daemon causing it to peg a CPU core at 100% after some builds. I haven't been able to track down why this may be happening so not sure if it's worth opening an issue for that, but in the meantime I came up with this to always run without the daemon:
function turbo(){ command turbo "$@" --no-daemon; }

(add to your ~/.bashrc or ~/.zshrc)

I've been experiencing some sort of leak in the daemon causing it to peg a CPU core at 100% after some builds. I haven't been able to track down why this may be happening so not sure if it's worth opening an issue for that, but in the meantime I came up with this to always run without the daemon:
function turbo() { command turbo "$@" --no-daemon; }

(add to your ~/.bashrc or ~/.zshrc)

Just hit this as well. Looks like it's looping on calling yarn --version.

@jjhuff interesting... I'm using turbo with pnpm

Hi @jjhuff, could you send over the logs from your daemon? You should be able to see them by running turbo daemon logs or by going to the file listed in turbo daemon status. If you want, you can also send it to me at<EMAIL_ADDRESS>

I've seen it several times, I killed a 100% CPU turbo daemon just an hour ago. That's with pnpm. Will check the logs and the status next time it happens (seems to happen at least once a week).

@NicholasLYang
This happens when I start it a second time. The first time, everything works great.
2024-04-29T16:29:53.223423Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:53.624224Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:54.034158Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:54.426060Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:54.808607Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:55.176551Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:55.542629Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:55.900112Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:56.266501Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:56.630643Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:57.000904Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:57.358331Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events
2024-04-29T16:29:57.731628Z  WARN turborepo_filewatch::package_watcher: lagged behind 1 processing file watching events

I can confirm - I also experience systematic problems with turbo.exe hanging in memory and blocking node_modules from being wiped out whenever I need it, not to mention redundant console messages about "standalone daemon not being found" or alike.

I have experienced significant speed up using the Daemon. With the daemon - under 1s - without the daemon - around 10 seconds.
So I wanted to get things set up seamlessly.. the problem I have experienced is with yarn v4 being unable to switch to a different version of turbo:
➤ YN0001: │ Error: EPERM: operation not permitted, unlink 'C:\git\SaxoTrader\node_modules\turbo-windows-64\bin\turbo.exe'
➤ YN0000: └ Completed in 0s 721ms
➤ YN0000: · Failed with errors in 7s 883ms

I couldn't find any documentation for turbo daemon but it seems turbo daemon clean stops the daemon. So I thought we should have a pre-install script that stops the daemon.
The trouble with yarn v4 is that pre-install runs after linking and there is no pre-link lifecycle hook (https://github.com/yarnpkg/berry/issues/5437). I even tried writing a plugin, but it seems yarn doesn't expose a way to make a plugin that works on install, pre linking and causes yarn to wait (I got validate workspace to kick off the command before linking, but then yarn doesn't wait :/). Maybe someone has a solution for this?
Another problem I got was that sometimes the turbo daemon clean would error - in my case the daemon was not running and this directory was already deleted, but I got this error:
Failed to remove log files: failed to remove directory `C:\git\xxx\.turbo\daemon`
Please remove manually: C:\git\xxx\.turbo\daemon
  x unable to complete daemon clean

so I also had to make the clean script ignore errors.
I also thought - lets start the daemon in postinstall.. but with both concurrently and npm-run-all it seems it would wait on yarn turbo daemon start and caused yarn to hang :/
Maybe this problem with locks on running processes only affects windows, but I really don't see how daemon is usable by people :/

I'd also like a way to opt out easily via config.
I do think it's true that many people run many daemons without being aware of it, but I think it's paternalistic to use that as a justification. I think many folks run background processes as a side effect of running an IDE, but the IDE can shut those down when it exits.
Take TypeScript as an example. Running tsc does not start a daemon. Running typescript-language-server does. An IDE user might be unaware of the latter, but it's still possible for the IDE to shut down the daemon when it exits. Something is managing the lifecycle.
What's missing and intrusive about the Turbo daemon is that it starts as a side effect of simply running turbo from the command line and never stops. It would absolutely be appropriate for the Turbo language server to act as a daemon for other Turbo commands to communicate with, but I think it's a questionable decision to have turbo run start it implicitly.
That said, it's actually fine if this is really what you think is the best user experience. It's certainly possible to pass --no-daemon. However, it's not possible today to run turbo watch --no-daemon.
So while I agree with voices above that think implicitly running the daemon by default is not a great choice, I can empathize with it and work around it. But my actual feature request would be to enable a turbo watch --no-daemon mode that runs the daemon code and the user interface code in the same process.
I'm a library maintainer and I'd like to use Turbo in my libraries, but I don't want to start daemons on contributor machines without their knowledge. If they choose to engage with Turbo via a language server for integration in their IDE that's great. But I'd like them to be able to run npm run test:watch and not end up with a daemon running without their knowledge.

Addressed in https://github.com/vercel/turbo/issues/3455.

Actually addressed in https://github.com/vercel/turbo/pull/8728

Oops, thank you! Edited mine.


Visual Studio 2022 C# file icons

The Visual Studio icons for a C# source file and a C# project file should be interchanged. It is quite easy to choose the wrong file looking at the icons alone.

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

ID: 8067c580-66bc-f2a2-49c8-e9990547db48
Version Independent ID: 8067c580-66bc-f2a2-49c8-e9990547db48
Content: Tutorial: Create a simple C# console app  - Visual Studio (Windows)
Content Source: docs/get-started/csharp/tutorial-console.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @anandmeg
Microsoft Alias: meghaanand

@MRahilly -- thank you for your feedback.  This repo is solely for documentation issues, and this is product-related.  Please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


correctly sending textDocument/didSave notifications

//cc @rcjsuen
The confusion started from https://github.com/rcjsuen/dockerfile-language-server-nodejs/issues/165.
I'm implementing the LSP server for vim at https://github.com/prabirshrestha/vim-lsp. And I would like to know when should it send textDocument/didSave changes to the client.
Here are the possibilities. (I would like to support both v2 and v3 porotocols).Is the table correct?

serverCapabilities
Send textDocument/didSave notification
includeText

{}
:x:
:x:

{ 'textDocumentSync: 0/1/2 }
✔
:x:

{ 'textDocumentSync: {} }
:x:
:x:

{ 'textDocumentSync: {save: {} } }
✔
:x:

{ 'textDocumentSync: { save: { includeText: true } } }
✔
✔

{ 'textDocumentSync: { save: { includeText: false} } }
✔
:x:

@prabirshrestha your table is correct.
Please note that v1 didn't have a didSave notification. But you are not targeting this.


Performance impact due to msgpack

While msgpack produces smaller serializations in comparison to JSON, the process of encoding/decoding seems to be a lot slower:
https://jsperf.com/msgpack-vs-json-perf/12
Sample run in Chrome:
MsgPack: 7,911 ops/second
JSON: 212,227 ops/second

Edit/Update
When using the correct msgpack-package at https://jsperf.com/msgpack-vs-json-perf/13 the results are
MsgPack: 38,092 ops/second
JSON: 208.359 ops/second

A benchmark test might be nice to get a better idea about the tradeoffs involved (e.g. msgpack might be better for larger serializations when the duration of transfer exceeds the duration of encoding/decoding).

The msgpack library used in that benchmark isn't the same one used by JellySocket. The one we use is known to be much faster and encoding and decoding.
After redoing this test with the correct package, we get better results:
http://jsperf.com/msgpack-vs-json-perf/13
Instead of only being 3% as fast, we are now 80% as fast at numbers, and 20% as fast at strings.
However, where it REALLY matters is in Node, not in the browser. Although Chrome and Node both use the v8 engine, they can have very different performance characteristics due to certain optimizations that Node takes advantage of.
Implementing the same test as a Node.js script, we get even better results:
#!/usr/bin/env node
var msgpack = require('msgpack-lite');
var stringData = {"glossary":{"title":"example glossary","GlossDiv":{"title":"S","GlossList":{"GlossEntry":{"ID":"SGML","SortAs":"SGML","GlossTerm":"Standard Generalized Markup Language","Acronym":"SGML","Abbrev":"ISO 8879:1986","GlossDef":{"para":"A meta-markup language, used to create markup languages such as DocBook.","GlossSeeAlso":["GML","XML"]},"GlossSee":"markup"}}}}};
var numberData = [
	[1, 2, 3, 4, -1, true, null],
	[3, 6, 5, 4,  1, false, 7],
	[3, 2, 8, 1,  0, true, 0],
	[10, 11, 12, 13,  14, false, true],
	[15, 16, 17, 18, 19, true, null],
	[20, 21, 22, 23,  24, false, 7],
	[25, 26, 27, 28,  29, true, 0],
	100, 200, 300,
	[
		[1, 0, 0, 0, 0],
		[0, 1, 0, 0, 0],
		[0, 0, 1, 0, 0],
		[0, 0, 0, 1, 0],
		[0, 0, 0, 0, 1]
	]
];

function test(fn) {
	var times = [];
	var endTime = Date.now() + 4000;
	do {
		var t1 = process.hrtime();
		fn();
		var td = process.hrtime(t1);
		times.push(td[0] * 1e9 + td[1]);
	} while (Date.now() < endTime)
	var totalTime = times.reduce(function (a, b) {return a + b;}, 0);
	var averageTime = totalTime / times.length;
	return Math.round(averageTime) + ' nanoseconds (' + times.length + ' tests)';
}

function JSON_strings() {
	return JSON.parse(JSON.stringify(stringData));
}
function JSON_numbers() {
	return JSON.parse(JSON.stringify(numberData));
}
function msgpack_strings() {
	return msgpack.decode(msgpack.encode(stringData));
}
function msgpack_numbers() {
	return msgpack.decode(msgpack.encode(numberData));
}

console.log('JSON (strings): ' + test(JSON_strings));
console.log('JSON (numbers): ' + test(JSON_numbers));
console.log('msgpack (strings): ' + test(msgpack_strings));
console.log('msgpack (numbers): ' + test(msgpack_numbers));

On my machine, msgpack performs equally to JSON when dealing with numbers, and 30% as fast when dealing with strings.
You might think that 30% is a big performance loss, but encoding an decoding is actually a relatively small aspect of the total performance in a websocket transport layer. You likely wouldn't notice it very much, if at all. However, I do plan on setting up a real-world benchmark for JellySocket soon to address issues like this.

Yes, I think it's a good idea to keep it open until I create a solid benchmark.
Thanks for being interested in JellySocket!

Closing this issue, since JellySocket 1.0 is very very different from previous versions. See #4 for details and/or complaints.


Actions

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

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

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
30

Relevant Lines:
30

💛  - Coveralls


Support relationProvenance keyword with possible "n/a" value

Implemented, blocked on #52. Once that's cleared, check the front end.


external checks: add new network connection check

network-connection-check allows to test if a connection to a network target
could be done (via tcp or udp) and a failed connection is expected.

 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.

Some deadline documentation was touched in #452 and #454

This one is currently waiting for the feedback about the check pod timeout ENV var to be integrated before merging.

This one is currently waiting for the feedback about the check pod timeout ENV var to be integrated before merging.

was AFK the last days, will test tomorrow morning and give feedback here

Will Switch back to the previous implementation with a separate TIMEOUT variable...
... the current version sometimes fail if connection target is expected to be unreachable. time until the check pod will start does take a while ( image has to be pulled, ... ) and the current hard coded -5s isn't flexible for this and a separate parameter make sense (IMHO)

@jonnydawg / @integrii
could you please take a look again ;-)


Is HSL color allowed to use?

Tried to use HSL color for coloring lineSeries but I fault. Is HSL color allowed to set as a color?
It's just a standart CSS color like RGB color scheme, why not? :)
That's what I'm talking about:
var candleSeries = chart.addCandlestickSeries({
  upColor: 'hsl(255, 50, 50)', // 'rgba(255, 144, 0, 1)',
  downColor: '#000',

It's not supported yet in the library. I guess there is some easy conversation between these representations so I guess the workaround is convert hsl to rgba before passing to the charts.

Let's leave it open and wait for more feedback.

Actually that's sad that browsers doesn't have API to manipulating with any color they support. That's why we have hard-coded list of predefined colors - seems there is no way to convert them to the any of string representation (or any other, which could be used in the code to modify it).

Would be useful if this chart supported CSS in general, rather than implementing its own colour parsing.. Canvas (e.g. the fill method) natively supports CSS, so in theory this library could too..
my web apps are all themed using CSS variables but I can't use them with this lib (i.e. i can't specify var(--background) as a backgroundColor)..

my web apps are all themed using CSS variables but I can't use them with this lib (i.e. i can't specify var(--background) as a backgroundColor)..

Can you use CSS variables in canvas' fillStyle? Afaik you can't and if it's so you still have to convert/get all colors from CSS variables in JS code to pass them to the library (or canvas).

Would be useful if this chart supported CSS in general, rather than implementing its own colour parsing.. Canvas (e.g. the fill method) natively supports CSS, so in theory this library could too..

The only reason why we're trying to parse passed colors is sometimes we need to generate a contrast color for a given one (e.g. bar color is red and price label will be with this color but what color a text should be? for red it should be white, but for white it should be black and so on). We happy to use native browser API if there would be one. Also we can remove the code to detect a contrast color for a given one and move it up to a user (let's say provide color via API) but it might affect UX for a solution out of the box.

// adopted from https://stackoverflow.com/a/9493060/466363
        function hsla(h, s, l, a){
            h /= 360, s /= 100, l /= 100;

            var r, g, b;

            if(s == 0){
                r = g = b = l; // achromatic
            }else{
                var hue2rgb = function hue2rgb(p, q, t){
                    if(t < 0) t += 1;
                    if(t > 1) t -= 1;
                    if(t < 1/6) return p + (q - p) * 6 * t;
                    if(t < 1/2) return q;
                    if(t < 2/3) return p + (q - p) * (2/3 - t) * 6;
                    return p;
                }

                var q = l < 0.5 ? l * (1 + s) : l + s - l * s;
                var p = 2 * l - q;
                r = hue2rgb(p, q, h + 1/3);
                g = hue2rgb(p, q, h);
                b = hue2rgb(p, q, h - 1/3);
            }

            //  color: 'rgba(171, 71, 188, 1)'
            // return [Math.round(r * 255), Math.round(g * 255), Math.round(b * 255)];
            return 'rgba('+Math.round(r * 255)+', '+Math.round(g * 255)+', '+Math.round(b * 255)+', '+a+')'
        }

         const a_lineSeries = prices_chart.addLineSeries({
                    lineType: 1,
                    color: hsla((n*360/count), 100, 50,1),
                    lineWidth: 2,
                    });
``


Transforming a tab into a window gesture is broken

System Details

OS: Windows 11 BUILD 22631.2271
Thorium Version 116.0.5845.169

Problem
When tring to pull a tab from the tittle bar to transform it into a new window the browser crash, it's possible to do this action properly by right clicking into the tab and selecting Turn the tab into a new Window option, but the gesture is broken.

Do you have the experimental feature flag turned on?
@khevencolino


How to pass LatLng from Main Activity to the MyMapFragment?

How would you go about sending a custom LatLng from your mainactivity to your MyMapFragment so you could customize the initialization of the map? Thanks!

Well, IMHO, the custom XML attributes might be the more "elegant" way I guess.
Another way would be to implement an Interface on the Activity, and then check if the Activity including the Fragment implements that Interface, when instantiating the Fragment, and the get the properties by calling the function of this defined in this Interface.
It's quite heavy handed but it's the old "Java way" of doing things :
https://developer.android.com/guide/components/fragments#EventCallbacks
Now in the Android documentation, they throw an exception if the Interface is not implemented, but you can just fallback to "default" values instead.
There might be tons of other ways, I haven't done Android development for a long period so I'm not wary of new mechanisms that could make this easier.

`mapFragment.getMapAsync {
val ll = LatLng(latitude, longitude)
it.addMarker(MarkerOptions()
.position(ll)
.title(title)
.snippet(url))
var color = Color.RED
if (magnitude.absoluteValue < 4) {
color = Color.BLUE
} else if (magnitude.absoluteValue < 2.5) {
color = Color.YELLOW
}
it.addCircle(
CircleOptions()
.center(ll)
.strokeColor(color)
.fillColor(color)
.radius(magnitude.absoluteValue * 100000)
.visible(true)
)
it.moveCamera(CameraUpdateFactory.newLatLng(ll))
it.setOnInfoWindowClickListener(this)
    }`

Interesting !
I didn't know that you could define another class method like this with Kotlin.