Using multi-GPU with accelerate is not working

accelerate launch -m lm_eval \
    --model hf \
    --model_args pretrained=mistralai/Mistral-7B-v0.3 \
    --tasks winogrande \
    --num_fewshot 0 \
    --batch_size 16 \

I tried using accelerate to speedup evaluation latency for efficient test, but it not works.
I thought my script was wrong, so I tried adding 'parallelize=True', 'num_processes=9' or using other models,
but it stops running after building contexts as below.
I couldn't find any related issue, so I think I'm doing something wrong.
Can anybody help me?

Your code in the block works for me. It will also output something to inform that I'm using two processes because my machine has two GPUs. Not sure how you add num_processes, but you can try
accelerate launch --num_processes 2 -m lm_eval \
    --model hf \
    --model_args pretrained=mistralai/Mistral-7B-v0.3 \
    --tasks winogrande \
    --num_fewshot 0 \
    --batch_size 16

Or explicitly specify the GPU by
export CUDA_VISIBLE_DEVICES=0,1

Oh, thank you for the kind and quick response. It only works when I set num_processes to 2. I'm not sure why, but it seems to be an issue with the environment I'm using. On another server, I can set num_processes to 8 and it utilizes all 8 GPUs just fine.


AttributeError: 'ProgbarLogger' object has no attribute 'params'

Hi!
I am trying to train the model on russian data, but I get AttributeError: 'ProgbarLogger' object has no attribute 'params'.
I guess it can be because of versions of the packages that I am using. First, I used the command pip install -e . That wasn't enough and I had to install modules transformers, tensorflow-gpu and torch myself.
Which version of those three modules did you use? Or do you have any idea why this problem occures?
Thank you in advance!

Please refer to https://github.com/emorynlp/iwpt-shared-task-2020/blob/master/setup.py#L37

Thank you!
All of the modules in this line are installed in required versions, but the problem remains.


Some considerations regarding security

We are going to provide the dynamic query language to our end-users. That causes us to double-check potential pitfalls in security.
Since the string presenting the query translated into the code which can be executed inside the context of our system we have to be sure the user is very restricted in their usage of types, operators, etc., etc.
After the research, we have found some issues I would like to discuss.

Enums are used without any restriction. It might seem to be safe, but I wouldn't like my end-user could potentially analyze what enums with what values are used in our code which is irrelevant to his query. Consider the example:

namespace IrrelevantNamespace
{
    public enum SecurityLevels
    {
        Weak = 1,
        Strong = 10
    }
}

[Test]
        public void TestIrrelevantEnums()
        {
            var entities = new Entity[]
            {
                new("KnownName1", 1),
                new("KnownName2", 10),
            }.AsQueryable();

           var query = entities.Where("ExperienceInYears = IrrelevantNamespace.SecurityLevels.Strong").Select(e => e.Name);
            var result = query.ToList();

            TestContext.WriteLine(string.Join(",", result));
        }

Parameterless constructors of any entity (at least of the same namespace) can be called. It might be useful but, obviously, constructor calls could have side effects so there should be an ability to turn that feature off. Even in regards to the 'it' type the possibility to use its properties doesn't cause the safety of calling its constructor in all circumstances. Consider the example:

namespace MyNamespace
{
    public class IrrelevantDataClass
    {
        public IrrelevantDataClass()
        {
            TestContext.WriteLine($"{typeof(IrrelevantDataClass)} unexpected ctor is called");
        }
    }

    public class Entity
    {
        public string Name { get; }

        public int ExperienceInYears { get; }

        public Entity()
        {
            TestContext.WriteLine($"{typeof(Entity)} unexpected ctor is called");
        }

        public Entity(string name, int experienceInYears)
        {
            Name = name;
            ExperienceInYears = experienceInYears;
        }
    }

    public class DynamicLinqTests
    {
        [Test]
        public void TestCtor()
        {
            var entities = new Entity[]
            {
                new("KnownName1", 1),
                new("KnownName2", 10),
            }.AsQueryable();

            var query = entities.Where(new ParsingConfig { AllowNewToEvaluateAnyType = false, ResolveTypesBySimpleName = true },
                "it eq object(new IrrelevantDataClass())").Select(e => e.Name);

            var result = query.ToList();

            //Even Entity ctor call might be not good in some circumstances
            query = entities.Where(new ParsingConfig { AllowNewToEvaluateAnyType = false }, "it eq new Entity()").Select(e => e.Name);
            result = query.ToList();
        }
}

And the last question is not about the security but about addressing. Why, in the following case, cannot I use a full type name (i.e. with namespaces)? It is important when several types with the same name but from different namespaces are added as custom.

namespace IrrelevantNamespace
{
    public class SameNameClass
    {
        public static int UserExperience() => 10;
    }
}

 [Test]
        public void TestAddressing()
        {
            var entities = new Entity[]
            {
                new("KnownName1", 1),
                new("KnownName2", 10),
            }.AsQueryable();

            var query = entities.Where(new ParsingConfig
            {
                ResolveTypesBySimpleName = false,
                CustomTypeProvider = new TestCustomTypesProvider()
            }, "ExperienceInYears == IrrelevantNamespace.SameNameClass.UserExperience()").Select(e => e.Name);

            var result = query.ToList();

            TestContext.WriteLine(string.Join(",", result));
        }

        public class TestCustomTypesProvider : DefaultDynamicLinqCustomTypeProvider
        {
            public override HashSet<Type> GetCustomTypes()
            {
                var customTypes = base.GetCustomTypes();
                customTypes.Add(typeof(IrrelevantNamespace.SameNameClass));
                return customTypes;
            }
        }

causes the exception:
System.Linq.Dynamic.Core.Exceptions.ParseException : Enum value 'UserExperience' is not defined in enum type 'IrrelevantNamespace.SameNameClass'

1]
If you expose it to end-users, all linq queries on all your database entities are possible, so indeed also using enums if you use enums.
Can you add a simpler example where you write the code and explain in detail what could go wrong?
2]
Full contructor support is only made possible when using linq-to-objects.
Adding a configuration setting to disallow all constructors could be added, I'll investigate this.
3]
Thanks for the question, indeed it seems that full namespace enums cannot be handled, I'll take a look.

Issue 2: see https://github.com/zzzprojects/System.Linq.Dynamic.Core/pull/732

@ademchenko
Is the answer enough for you? Or do you need more clarification?

@ademchenko
Is the answer enough for you? Or do you need more clarification?

Closing


Implement About Page

If you want to add an "about" page, feel free to do so.
If you do have one, I suggest clear wording to inform the user that this is an academic project, and not related to, or endorsed by, any similar business from the real world.

Didn't do due to time issue


文件不刷新

当修改文件名或者删除文件时，页面不自动刷新

When the file name is modified or the file is deleted, the page does not automatically refresh

Thanks for this report.

What version of Files are you using?
Does this happen on every folder or only in some locations?


[ENH] move from pip to conda

Although heavier, conda allows us to install more stuff (e.g. R) in envs and not system wide. We needed it for #45

@tomMoral I missed the place where conda is installed, however given the current tests failures I was thinking of having one conda init $SHELL right after conda installation.
Should it replace:
    displayName: Add conda to PATH
    condition: eq(variables['Agent.OS'], 'Windows_NT')

  - bash: |
      echo "##vso[task.prependpath]$CONDA/bin"
      sudo chown -R $USER $CONDA
    displayName: Add conda to PATH
    condition: and(eq(variables['PACKAGER'], 'conda'),
                   ne(variables['Agent.OS'], 'Windows_NT'))

?
Also, is it ok to remove packager=ubuntu?

The test fail locally, conda activate fail when executed in subprocess @tomMoral
Locally, error messages look like:
E               RuntimeError: Failed to conda install packages ['numpy', 'cython']
E               Error:
E               CommandNotFoundError: Your shell has not been properly configured to use 'conda activate'.
E               To initialize your shell, run
E
E                   $ conda init <SHELL_NAME>
E
E               Currently supported shells are:
E                 - bash
E                 - fish
E                 - tcsh
E                 - xonsh
E                 - zsh
E                 - powershell
E
E               See 'conda init --help' for more information and options.
E
E               IMPORTANT: You may need to close and restart your shell after running 'conda init'.

benchopt/util.py:73: RuntimeError

In the end I managed to make it work locally, with the exception that celer, lightning etc are installed from pip, which conda does not handle, so we still need pip for those.
For the install, we can filter with: install_with_pip = [req for req in cls.requirement_install if req.startswith("git+")], and pip install those (the install happens in the conda env)
For the uninstall, it's a bit heavier since names needed for the uninstall like sklearn-contrib-lightning do not start with git+.
What do you think ?

what do you mean conda does not support pip install?

sorry, meant install from github such as pip install git+https://github.com/tommoral/blitzl1.git@FIX_setup_deps; you can't do conda install git+https://github.com/tommoral/blitzl1.git@FIX_setup_deps so you need to use pip (inside the conda env) for these.

yes use pip inside the conda env for this

Tests are almost green :)
From my experience with this PR I now wonder if we should not go more often with a .sh install script for packages, even if it mostly contains conda commands.

an exemple is conda vs pip for packages installed from github. I am not sure the syntax I used with pip: is easy to understand for a beginner. With a bash script and a single conda env, the bash script would simply be conda activate benchopt_env; pip install git+https...
cvxpy needs to be downloaded from conda-forge, so we need to conda config --append channels conda-forge before, and remove it after. This is a bit complicated. With a bash script this can be done explicitely and simply

It would feel more flexible to have a bash script for every package, as soon as installation is not all vanilla conda.
WDYT @tomMoral ?

Codecov Report

Merging #46 into master will decrease coverage by 0.52%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   53.93%   53.40%   -0.53%
==========================================
  Files           8        8
  Lines         636      631       -5
  Branches       99      100       +1
==========================================
- Hits          343      337       -6
- Misses        263      268       +5
+ Partials       30       26       -4

@agramfort ready for review

But red... :(
https://dev.azure.com/benchopt/benchopt/_build/results?buildId=337&view=results

Enfer! I though I was done but cyanure is tripping the PR...

I guess I will skip the test for this one and add an issue as it is not central to this PR.

Its green!!!


Too many file created by yo @microsoft/sharepoint

I noticed nearly 97,049 Files, 16,863 Folders created during the helloworld-webpart development. If i try to delete i am not able to delete the folder due to long folder/file names. Is is it a normal behavior?
I am using a free tool to delete such files / folders.

Make sure you are running V3 of npm by running the following command
npm -g install npm@next

https://github.com/SharePoint/sp-dev-docs/wiki/Setup-your-machine
That should get rid of your long folder/file names issue and the size of your node_modules folder should be relatively smaller too.

Vardhaman,
I did this already but still i tried to reinstall everything again and found the following numbers which is good
30,056 Files, 4,000 Folders
Deletion of root folder is still an issue.

This is effectively tracked here - https://github.com/SharePoint/sp-dev-docs/issues/59


Code blocks are not supported

I am using remarkable in conjunction with this library to render markdown in a PDF.
Typically, an HTML code block is achieved like so:

    def sum_eq_n?(arr, n)
      return true if arr.empty? && n == 0
      arr.product(arr).reject { |a,b| a == b }.any? { |a,b| a + b == n }
    end

(Specifically, this is how remarkable handles code blocks, see https://github.com/jonschlinkert/remarkable/blob/58b6945f203ca7a0bb5a0785df90a3a6a8b9e59c/lib/rules.js#L70)
Currently, this will result in the following being rendered in the PDF:

Working around this limitation appears to be quite challenging. Obviously any custom code renderer will not be hit.

this can probably just duplicate the behavior for pre. I'll look into it

I think this is already supported

Hey I have the same issue, how was this resolved? @danomatic
It looks like your  tag considers the child as raw text even though that's not up to HTML spec, instead  should simply not ignore whitespace and newlines etc.


[KeyVault] - Update API Extractor version

What

Update KeyVault's API Extractor dependency to the latest
Regenerate API Views

Why
As part of #9410 we are upgrading API Extractor piecemeal. We do need to have everyone upgraded to a later version
in order to support import type / export type statements which will eventually enable us to re-export OTel's APIs

Closing as we did this as part of #17917


Conversion from read::File to write::Object?

I'm trying to build some binary analysis/cyber security tools using this library (ELF format, specifically) and it would be useful if I could read a shared library, modify it, and then write it back out to a new location. Is this possible currently?

Ok, that seems to work, but a cmp on the two files shows significant differences between them, and I'm not sure why.

Maybe segments are reordered compared to the original. This should be harmless. Does the library still work after running the copy example that you used on it? (of course try this only on a non-malicious library. it the file you tested it is malicious you could try for example a system library)

It works. Looks like it's partially a reordering, but some of it looks like the instructions have changed- maybe objdump is interpreting the assembly differently for the two versions, for some reason. Apparently the changes are harmless, though- it's a very simple test library built specifically for testing.

Which of the examples are you using? The elfcopy example should copy all relocations verbatim, while the objcopy example interprets the elf relocation into a generic format and then turns it into an elf relocation again. This roundtrip is lossy in some cases due to unimplemented support for some relocations I believe.

I'm working with the elfcopy example.

No clue what is going on then.

No worries. I'll keep working with the elfcopy example. Thanks a lot for the help, man.

Looks like it's partially a reordering, but some of it looks like the instructions have changed- maybe objdump is interpreting the assembly differently for the two versions, for some reason.

There shouldn't be any changes to instructions.

Changes are mostly in the .dynstr and .dynamic sections.

Changes in these sections are expected because it must recreate the .dynstr section in case your modifications require strings to be added or removed, and string table offsets in the .dynamic section may change as a result of that.
I won't be surprised if elfcopy doesn't handle everything, but please create issues if you do encounter anything.

@philipc If I see anything, I'll let you know. I know it's a long shot, but I don't suppose there are any docs explaining how to remove a single function during the copy, is there?

For an already linked library, there isn't really any way to do so other than to disassemble, remove the function and assemble it again. After linking the code section contains relative offsets to other parts of the code section and to the data section everywhere. If you want to cut a part of it away, you have to update every reference with the new addresses.


get_transcripts no longer working

I had been using this API without any issues but today I keep getting "The video is no longer available" error for videos that are available and have transcripts, code below, error output below that:
    # import the YouTube transcript api downloader
    from youtube_transcript_api import YouTubeTranscriptApi

    # get subtitles
    subtitles = YouTubeTranscriptApi.get_transcripts(["5Cdv932eoU0"])

    # to test - print the results
    print(subtitles)

$ python downloader1.py
Traceback (most recent call last):
  File "downloader1.py", line 72, in <module>
    subtitles = YouTubeTranscriptApi.get_transcripts(["1mf06IwdpVM"])
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_api.py", line 102, in get_transcripts
    raise exception
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_api.py", line 99, in get_transcripts
    data[video_id] = cls.get_transcript(video_id, languages, proxies, cookies)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_api.py", line 128, in get_transcript
    return cls.list_transcripts(video_id, proxies, cookies).find_transcript(languages).fetch()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_api.py", line 70, in list_transcripts
    return TranscriptListFetcher(http_client).fetch(video_id)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_transcripts.py", line 34, in fetch
    self._extract_captions_json(self._fetch_html(video_id), video_id)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_transcript_api/_transcripts.py", line 42, in _extract_captions_json
    raise VideoUnavailable(video_id)
youtube_transcript_api._errors.VideoUnavailable:
Could not retrieve a transcript for the video https://www.youtube.com/watch?v=1mf06IwdpVM! This is most likely caused by:

The video is no longer available

If you are sure that the described cause is not responsible for this error and that a transcript should be retrievable, please create an issue at https://github.com/jdepoix/youtube-transcript-api/issues. Please add which version of youtube_transcript_api you are using and provide the information needed to replicate the error. Also make sure that there are no open issues which already describe your problem!

Hi @Garrettlynch, I just checked the video 1mf06IwdpVM you posted about and it in fact does not have a transcript. Is this the same for other videos which actually have transcripts? Also, which version of this module are you using?

Hi @Garrettlynch, I just checked the video 1mf06IwdpVM you posted about and it in fact does not have a transcript. Is this the same for other videos which actually have transcripts? Also, which version of this module are you using?

Hi, I've tried multiple videos using both get_transcript("5Cdv932eoU0") and get_transcripts(["5Cdv932eoU0"]) - always the same VideoUnavailable error today. As I said it was working without issue until today. Nothing changed in the code.
I was using youtube-transcript-api 0.3.1. I just updated to 0.4.1 and it seems to be working now. The video without a transcript threw back a TranscriptsDisabled error and a video with a transcript has worked. Odd that the version that had been working stopped working but at least it works now - thanks :)

Yeah, that version isn't working anymore since YouTube has added a cookie setting screen which occasionally is returned when doing requests towards YouTube. Therefore, versions older than v0.4.1 are most likely not working anymore.


golang-dummy-app to latest

Promote golang-dummy-app to version latest

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add a ConfigValueFactory.newSubstitution(path) method ?

Hello,
The resolve() method doesn't resolve ${} expressions if they are parsed from map or properties. If I use the parseString method it does resolve those expressions.
So how a config object resolves depends on the source of the config? This felt not quite obvious to me. Especially since the config objects are already created when resolving. Both -from map and from string created config objects - appear to be equal but when resolving the first it does not resolve ${} but the second does resolve it.
Is there any chance to get resolvable ConfigObjects created from maps? I would really appreciate to use this feature in order to add default values by map instead of string.
Thanks,
David

Retitled to indicate the idea that we could have an API for creating an unresolved substitution node.


insert issue - parameter 'xxx' already added

hi mike
do you have any idea why this would be happening? i have a model class that throws an exception on insert saying "parameter 'xxx' has already been defined / added" - but of course it hasn't and it is a single insert of a single model
using mysql 5.7 btw
thanks for any ideas you may have

For fast fix, would you be able to share your table schema and your class model? I will try to simulate.

happy to but is it possible to email you a zip or something?

clients.zip

Thanks. I will do check this one this coming weekend or Friday. Would that be okay?

yes of course - thanks so much :)

more info:
mysql connector 8.0.20
mysql server 5.7
i replaced an "insertAll(list-of-T's)" that failed reliably, with a loop that does raw sql insert statements one at a time (could do a batch but this was quicker for a test) and it works fine
the bug seems to be that "'name' parameter is already added" (but it can be a diff column is already added on another model) - which i think might be in the query builder where is builds a field list from the model
not sure if this helps but i might have to look into writing a new querybuilder if i cannot find a solution soon
:)

Will look at this issue soon. Will revert to you ASAP

thanks so much

so more info: the reading of meta data from the information_schema has a syntax error in it: this is what is logged in mysql error log:
SELECT COLUMN_NAME AS ColumnName
	            , CASE WHEN COLUMN_KEY = 'PRI' THEN 1 ELSE 0 END AS IsPrimary
                , CASE WHEN EXTRA LIKE '%auto_increment%' THEN 1 ELSE 0 END AS IsIdentity
                , CASE WHEN IS_NULLABLE = 'YES' THEN 1 ELSE 0 END AS IsNullable
                , DATA_TYPE AS ColumnType /COLUMN_TYPE AS ColumnType/
                , CHARACTER_MAXIMUM_LENGTH AS Size
                , COALESCE(NUMERIC_PRECISION, DATETIME_PRECISION) AS `Precision`
                , NUMERIC_SCALE AS Scale
                , DATA_TYPE AS DatabaseType
            FROM INFORMATION_SCHEMA.COLUMNS
            WHERE
	            TABLE_NAME = 'syncchecksums'
            ORDER BY ORDINAL_POSITION

but this bit:
/COLUMN_TYPE AS ColumnType/
looks like a failed comment and when i comment it the query runs just fine
was this a tired copy / paste error maybe?
:)

currently looking on this one.

/COLUMN_TYPE AS ColumnType/ - it is in purpose that this line of code is commented.

This one is working on my end. The line of the code can be seen here.

Hi, this seems to be working on my side. I tried the Query, QueryAll, Insert and InsertAll operations with the models and schemas you shared.
Solution/Project: SimulateRepoDbIssue432.zip
Local Results: Issue 432 Result.txt
I am now suspecting the version of MySql in your part. As I remember, some of the syntaxes I had written is for MySql (>= 8.0.x) - it was explained it my old documentation in ReadTheDocs, whereas not in RepoDb.NET.
Is it correctly understood that you are running MySql (5.7.X)?

yes we are stuck on mysql v5.7.xx

Okay, then let me debug using the local solutions I had of RepoDb and RepoDb.MySql with updated version of Connector to 8.0.20. Maybe I can replicate it there.

or we could make a legacy 5.7.x compatible querybuilder maybe for a longer term solution when there is some time? either way thanks so much - next time i'm in denmark i owe you a good few beers :)

Thanks for the beers soon! :) I tested it using the MySql.Data (8.0.20) and it is working there. I also tried running the Integration Tests on my DEV machine and all are passing.

Would you be able to share me the result of the following?
Retrieving the model fields.
var fields = FieldCache.Get<Models.Client>().AsList();

Retrieving the database fields.
Make sure to change the connectionString.
using (var connection = new MySqlConnection("Server=localhost;Database=RepoDb;Uid=user;Pwd=Password123;"))
{
	var dbFields = DbFieldCache.Get(connection, "clients", null).AsList();
}

Lastly, for the fun of it, would you execute the code below and see if the generated SQL Statement is correct?
using (var connection = new MySqlConnection("Server=localhost;Database=RepoDb;Uid=user;Pwd=Password123;"))
{
	var builder = connection.GetStatementBuilder();
	var dbFields = DbFieldCache.Get(connection, "clients", null).AsList();
	var fields = FieldCache.Get<Models.Client>().AsList();
	var insertStatement = builder.CreateInsert(new QueryBuilder(),
		ClassMappedNameCache.Get<Models.Client>(),
		fields,
		dbFields.FirstOrDefault(df => df.IsPrimary),
		dbFields.FirstOrDefault(df => df.IsIdentity),
		null);
}

The expected output would be.
INSERT INTO `clients` ( `guid`, `notes`, `servicenotes`, `medicalhistory`, `espsitecode`, `site`, `referredby`, `active`, `cardcode`, `organisation`, `name`, `title`, `firstname`, `lastname`, `classificationcode`, `categorycode`, `listclientloyaltycode`, `description`, `origin`, `activesince`, `firstvisit`, `retainer`, `representative`, `subscriptionvalue`, `subscriptionends`, `applysalestax`, `balance`, `contact`, `accounts`, `operator1`, `operator2`, `owner1`, `owner2`, `mailinglist`, `status`, `physicaladdress`, `physicalstreet`, `physicalarea`, `physicalcity`, `physicalcountry`, `physicalcode`, `postaladdress`, `postaldetails`, `postalarea`, `postalcity`, `postalcountry`, `postalcode`, `telephonehome`, `telephonework`, `telephone1`, `telephone2`, `home`, `callbox`, `office`, `cellphone`, `fax`, `modem`, `email`, `web`, `calldate`, `called`, `lastvisit`, `frequency`, `lastcharge`, `totalspent`, `promomiles`, `account`, `complimentary`, `picturepath`, `tracking`, `nextvisit`, `appointmentgroupstr`, `missedappointments`, `staff`, `staff1`, `staff2`, `staff3`, `sex`, `birthdate`, `occupation`, `spouse`, `children`, `salestax`, `salestaxnumber`, `enableearnmiles`, `settings`, `listswversionguid`, `clientimagepath`, `emergencycontactperson`, `emergencycontactnumber`, `id`, `sync_action` ) VALUES ( @guid, @notes, @servicenotes, @medicalhistory, @espsitecode, @site, @referredby, @active, @cardcode, @organisation, @name, @title, @firstname, @lastname, @classificationcode, @categorycode, @listclientloyaltycode, @description, @origin, @activesince, @firstvisit, @retainer, @representative, @subscriptionvalue, @subscriptionends, @applysalestax, @balance, @contact, @accounts, @operator1, @operator2, @owner1, @owner2, @mailinglist, @status, @physicaladdress, @physicalstreet, @physicalarea, @physicalcity, @physicalcountry, @physicalcode, @postaladdress, @postaldetails, @postalarea, @postalcity, @postalcountry, @postalcode, @telephonehome, @telephonework, @telephone1, @telephone2, @home, @callbox, @office, @cellphone, @fax, @modem, @email, @web, @calldate, @called, @lastvisit, @frequency, @lastcharge, @totalspent, @promomiles, @account, @complimentary, @picturepath, @tracking, @nextvisit, @appointmentgroupstr, @missedappointments, @staff, @staff1, @staff2, @staff3, @sex, @birthdate, @occupation, @spouse, @children, @salestax, @salestaxnumber, @enableearnmiles, @settings, @listswversionguid, @clientimagepath, @emergencycontactperson, @emergencycontactnumber, @id, @sync_action ) ; SELECT LAST_INSERT_ID() AS `Result` ;

will do now

ok all the things worked as expected, except the insert query should not have @id and @sync_action - those are not in the table and are why i asked for an [Ignore] attribute

The library is intelligent enough to ignore the fields that are not aligned with the database. The codes I shared you above is just generating the Insert SQL statement based on the list of fields you passed (without the additional layer of checking).
To help you understand, the CommandTextCache is the cacher of the Command Texts and does the trick here. In which it calls the GetActualFields method here.

i thought so when i saw the list of columns you pull from the information_schema but i thought i would just check ;)

I will conclude this issue as "Working as Expected". Can I close now? Thanks

well it doesn't work for me so how would i proceed now?

Have you tried this in you local machine?
https://github.com/mikependon/RepoDb/files/4599232/SimulateRepoDbIssue432.zip

let me try that now

That contains the models and schemas you shared with a very thin code implementations. Make sure to restore the Nuget Package before running (and you need to be in VS 2019).

ahhhhhh you have mysql data 8.0.19 - i cannot install that - i have 8.0.20 which is where the error starts showing - i am unable to downgrade that package according to vs2019

Okay then your last resort is to debug/trace it yourself. To help you with, you have to implement the tracer.
public class CustomTrace : ITrace
{
    public void AfterQuery(TraceLog log)
    {
        // Some implementations here
    }

    public void BeforeQuery(CancellableTraceLog log)
    {
        // Some implementations here
    }

    ...
}

Then inject the CustomTrace class when you call the Insert operation (inside the saveToDb method your Client class).
recordno = Convert.ToInt32((ulong)conn.Insert<Client>(this), trace: new CustomTrace());

Then add a breakpoint in the BeforeInsert method.
Lastly, please share me the content of the log.Statement and a screenshot of the log.Parameter.

ok will do - thanks again so much for your time and help - really appreciated

Btw, if you install the MySql.Data 8.0.20, it will automatically be resolved by VS.

yes but the 8.0.20 is what causes the problem to appear and it is not possible to downgrade to 8.0.19 so i am stuck with this problem

Just do implement the Tracer and that will give you the full picture of what really happens.

ok

How about manually copying the MySqlHelper class into your solution, remove the syntax error in the SQL text and reinject in your code?
Use the code below to inject it.
DbHelperMapper.Add(typeof(MySqlConnection), new ModifiedMySqlDbHelper());

so do i have to implement all 6 million interface methods to write the trace class??

Implementing a trace is very direct and is very easy. Just create a class that inherits the ITrace interface. Implement the methods and simply comments the defaultly generated NotImplementedException codes.
Then inject your custom trace class when you call the operation (ie: connection.QueryAll<T>(trace: new CustomTrace());).
And I suggest I think you need both :)

will do thanks - sorry for my lack of C# skills

i think i found the bug - you do not mention table_schema in the query to get columns, this one:
SELECT COLUMN_NAME AS ColumnName
	            , CASE WHEN COLUMN_KEY = 'PRI' THEN 1 ELSE 0 END AS IsPrimary
                , CASE WHEN EXTRA LIKE '%auto_increment%' THEN 1 ELSE 0 END AS IsIdentity
                , CASE WHEN IS_NULLABLE = 'YES' THEN 1 ELSE 0 END AS IsNullable
                , DATA_TYPE AS ColumnType /COLUMN_TYPE AS ColumnType/
                , CHARACTER_MAXIMUM_LENGTH AS Size
                , COALESCE(NUMERIC_PRECISION, DATETIME_PRECISION) AS `Precision`
                , NUMERIC_SCALE AS Scale
                , DATA_TYPE AS DatabaseType
            FROM INFORMATION_SCHEMA.COLUMNS
            WHERE
	            TABLE_NAME = 'syncchecksums'
            ORDER BY ORDINAL_POSITION

so when we have 30 of the same databases called something different, we get multiple versions of the columns which is the exact error - you have to use "and table_schema='database-name'"
then it works

Hmmm. That make sense! I will do update soon or feel free to issue a PR. For now, keep your custom DB helper in your solution. Cheers

i don't seem to have permissions to add a pull request?

I had added you as collaborator.

Fixed by the user itself via PR #433 - will be a part of RepoDb.MySql > v1.9.0.
Please do let me know if you need a Beta for this, otherwise I will release this together with other enhancements.

i would love a beta so i can get the app deployed to our testers - thanks so much for all your help and guidance :)

Will issue you a beta later for this fix.

The fix has been deployed via RepoDb.MySql (v1.0.10-beta1). The package can be found here.


Model#call() -> Error: "no method 'reduce' on 'localRoot.set(...).reduce'"

lib/response/CallResponse.js:229
Looks like the Rx = require("rx/dist/rx") (line 0) is pulling in a rx library without the reduce method.
Other files do Rx = require("rx/dist/rx") && require("rx/dist/rx.aggregates") which will pulls in the aggregate methods (reduce being one of them), and is probably why you are not seeing this error in your tests.
If you never get around to invoking those files that do the double-require, the rx.aggregate functionality is not there.
You might also want to update the README to add alias's for both "rx$" and "rx/dist/rx" path(s) to point to the consumer's preferred rx library.

Ooops... this belongs in the falcor repo -> https://github.com/Netflix/falcor/issues/533


Adding a Quiz test feature for the course "Python For Everybody"

I want to add a Quiz test for  the course "Python For Everybody" . So  which will tell a student based upon their performance in a this topic that he/she is at beginner , Intermediate or advance level and on that basis the Student can decide that this course is preferable for them. It will make the site more Interactive for users.
Please assign me this issue under gssoc-ext label.

For now we are not looking for this.
So I am closing this issue.
Please understand.


Add a couple more, mostly text, reference tests for non-embedded symbolic fonts without included encoding information

I've started to look into how we can fix issue #7580, but quickly became worried that fixing it could easily mean that we'd trade one fixed PDF file for a multitude of broken ones.
Hence I started going through the history of the code that choose the fallback encoding, and noticed that it has been changed a number of times over the years to deal with various cases of weirdness/errors in non-embedded fonts.
To my relief it turned out that almost all the PRs, please see a possibly incomplete list here, that changed this code actually included eq test-cases.
However, in one case it appears that a PR missed to add a test-case. Furthermore since the fallback encoding may also be the only source for creating a toUnicode map, changing the encoding could possibly regress only the text-selection despite a PDF file still rendering correctly.
Therefore, this PR adds one new eq test, and also a number of additional text tests for PDF files already present in the test-suite.
Note that it's obviously possible that there's a certain overlap between the added tests, but I'd be a whole lot more concerned with causing regressions.

/botio test

It seems that the Linux bot wasn't able to correctly download the new file :-(
Downloading http://web.archive.org/web/20160910104607/http://www6.inra.fr/productions-animales_eng/content/download/4803/45520/version/1/file/Prod_Anim_1997_10_2_04.pdf to pdfs/issue2017.pdf...
Error during downloading of http://web.archive.org/web/20160910104607/http://www6.inra.fr/productions-animales_eng/content/download/4803/45520/version/1/file/Prod_Anim_1997_10_2_04.pdf: HTTP 503
Downloading http://web.archive.org/web/20160910104607/http://www6.inra.fr/productions-animales_eng/content/download/4803/45520/version/1/file/Prod_Anim_1997_10_2_04.pdf to pdfs/issue2017.pdf...
WARNING: File was not downloaded. See "pdfs/issue2017.pdf.error" file.
WARNING: File was not downloaded. See "pdfs/issue2017.pdf.error" file.
Unable to verify the checksum for the files that are used for testing.
Please re-download the files, or adjust the MD5 checksum in the manifest for the files listed above.

I'll trigger a retry after the reference images are created in the other PR. The Windows bot did succeed and I noticed previously that the Web Archive intermittently returns 503 responses, so this can happen.

/botio-linux test

It does not retry the download, which is not particularly useful... @yurydelendik Could you download this file and put it on the Linux bot manually?

It's probably because of https://github.com/mozilla/pdf.js/blob/122d4732398ca67381cc469dc2f49696b824c98d/test/downloadutils.js#L96, which creates an empty file so that we do not retry. Why is that even there?

Instead of waiting for someone to have time to manually fix the PDF file that failed to download, I just went ahead and replaced it with a reduced test-case instead :-)
/botio test

/botio makeref

Awesome! Good to have more test coverage and to even have a reduced test case.


Support deno in addition to node?

Since lit-html is written in TypeScript and has no production dependencies, what would it take to get it working on deno?

I should have been more clear: my concern isn't with running lit-html inside deno, but serving a lit-html-powered front-end (static assets as well as developer templates) from a deno server. Since deno apps have no package.json, and there's no equivalent to something like Webpack, I wonder how much work would be needed to make lit-html apps servable from deno.

Curious about any new potential interests in reviving idea. Since deno's compiler supports the dom library, I'm considering using deno as a high performance just-in-time compiler (not a runtime) for typescript in an Electron app. It can also simultaneously serve as a complementary runtime to the node runtime inside Electron.
The idea is to ship lit-html elements in *.ts files. When user first launches the app, deno will compile and cache the *.js output. The goal is to completely eliminate the build/bundling step. The source code can be pushed to CDNs like pika and be directly consumed by end users.
It's a very experimental idea but if it works, it would make lit-html so much easier to develop with.

I would strongly recommend against bundling the typescript compiler in either applications or tools like Deno, ts-node, etc. For applications, why would you have each user compile the typescript rather than doing it once before publishing? That's just wasted work. For tools you end up typing the major version to the latest version of TypeScript, which can and does have breaking changes on nearly every release.
As for lit-html and Deno: we publish standard ES2017 and depend on browser APIs. If your environment supports those, you should be good to go.


Remove extra "preview" word

The preview version of VS for Mac now is 8.7, and the current release is 8.6.
So, 8.4 is no longer preview.

Once @gewarren approves, we can merge. Thanks @Youssef1313 !

Once @gewarren approves, we can merge. Thanks @Youssef1313 !

I think we need to update 3.0 to 3.1 and remove the TODO comment, right?

Thinking about it some more, I think the todo can be removed. This article will be evaluated and updated when .NET 5 releases or the desktop guide is released.

I think we need to update 3.0 to 3.1

@gewarren, Wouldn't 3.x be more accurate?


[feature request] ORDER BY (or ) ASC

Hi.
Is there any roadmap for being able to sort data with custom fields (not only time) ?

This is a duplicate for #3954 (order by tags) and #4794 (order by aggregate fields). It doesn't look like either have been slated for a release yet, though.


[$500] mWeb - Chat - Long display name is copied along with ellipsis and "New messages" text in chat header

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/30476
Action Performed:

Go to staging.new.expensify.com
Open chat with user with a long display name.
Highlight the name on the chat header and copy the name.
Long press on the composer and paste it.
Note that ellipsis and 'new message' is copied over.

Expected Result:
Only the display name, which is the email, is copied.
Actual Result:
The display name is copied along with the ellipsis and 'New messages'.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/Expensify/App/assets/93399543/86fc949e-dc92-4401-872d-37605ab39f1e

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018617982cf69dd0f6
Upwork Job ID:<PHONE_NUMBER>436833280
Last Price Increase: 2023-11-01

Proposal
Please re-state the problem that we are trying to solve in this issue.
On mWeb, when copying long chat name, ellipsis and New messages text get's copied as well.
What is the root cause of that problem?
In components/DisplayNames/DisplayNamesWithTooltip.js, ellipsis text is selectable enabling use the native copy and paste functionality, even though according to API is should be false by default.
What changes do you think we should make in order to solve the problem?
 Instead, I suggest making a fix in Expensify/react-native-web.
https://github.com/Expensify/react-native-web/blob/master/packages/react-native-web/src/exports/Text/index.js#L144
We should handle case when selectable is null the same way as when selectable === false
  (selectable == null || selectable === false) && styles.notSelectable,

What alternative solutions did you explore? (Optional)
N/A
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

@kbecciv I just tested on chrome mweb and the copy/paste didn't include the ..., but it also didn't include the whole name. Can you point me to the test this is coming from?

@garrettmknight This issue is associated with the following PR: https://github.com/Expensify/App/pull/30476

Issue is reproducible
https://github.com/Expensify/App/assets/93399543/5bc8ec1c-fe64-443e-a1ad-f15bf60a42bb

Ok, got it - so you have to manually select the ellipsis? Since you're manually attempting to select the ellipsis, that doesn't seem unintended.  I suppose the 'New Message' part is but that's an extreme edge case.
If you force tap/long press to select the name and copy/paste, it only pastes the name without an ellipsis and "New Message" so we're good there.
@kbecciv can you point me to the test for this one so we can clarify that this isn't a relevant bug?

Maybe I'm missing the reproducible step there, but it looks like you're doing the same thing and getting different results - can you explain how the third attempt differed from the first two?

Got it - I think that's unexpected, but an edge case.


Make a release?

Hi,
Would it be possible to make a release?
Like a similar project which I am using:
https://github.com/prudhvitella/terraform-provider-infoblox/releases
Binaries there are for linux/darwin/windows...

That's a good idea. We can probably add a release step in the CI configuration.
@zoobab feel free the send a PR if you want to tackle that :)

I made a release here:
https://github.com/zoobab/terraform-provider-ldap/releases
Right now it is built on Docker hub, and extract the binary manually and upload it to Github. If there would be an automated way to do that...


Convert ALL DateTime values to UTCDateTime in Query\Builder::compileWheres

$builder->where('timestamp', '>=', $carbonDateObject1)->where('timestamp', '<=', $carbonDateObject2)

was working, but
$builder->whereBetween('timestamp', [$carbonDateObject1, $carbonDateObject2])

wasn't.
This was because in the compileWheres function only converted $where['value'] to UTCDateTime, but not $where['values'], where the former was filled in easy queries and the latter one in between queries. MongoDB then couldn't handle the given Carbon objects (possibly autoconverted to strings)

oh, I see it's a duplicate of https://github.com/jenssegers/laravel-mongodb/pull/953/files , but that one has merge conflicts


Fix: Prefix issue

Currently, the bot is set with the prefix of !. This is set in the constants, and is subject to change. The bot checks every message to see if it starts with the prefix. If it does, it checks what command was executed. The bug here is that on the switch-case of the command checking, the ! prefix is hardcoded.
...
switch strings.ToLower(strings.Split(m.Content, " ")[0]) {
    // if the command is !greet
    case "!greet":
        s.ChannelMessageSend(m.ChannelID, randomGreeting(s, m))
    ...
...

If the prefix in the constants is changed, the bot breaks, and none of the commands are accessible by users. Change the switch-case conditions to make commands flexible to changes in prefix.

Hi, I would like to work on this.

Sure! Assigned.

hey can i work on this issue?

hi @ankan792, I'm sorry, but this issue doesn't need multiple people working on it, and I've already assigned @techsnap to this issue. There are other open issues that you can try though. Thank you for showing interest!


Add a runner for Lucene's graph kNN prototype.

This branch gives a way to run benchmarks against Lucene's kNN prototypes. It uses py4j to connect to Lucene from the benchmarking framework.
To run benchmarks, first set up Lucene:

Check out this Lucene branch. It builds off the kNN prototype branch, and adds a class PythonEntryPoint to expose index + search functions.
Run PythonEntryPoint, either from your IDE or from the commandline. Note that the jar lucene/sandbox/py4j0.10.9.jar must be on the classpath, as well as lucene sandbox and lucene core.

Then to run ann-benchmarks:

Install the ann-benchmarks library through these instructions. You can skip step 3, installing the Docker images.
Make sure py4j is installed: pip3 install py4j.
Run a benchmark in 'local' mode. Example command, which runs against a dataset of GloVe word vectors:python3 -u run.py --dataset glove-100-angular --algorithm lucene-graph \
       --definitions algos.yaml --force --runs 1 --local

Print the results (this also creates plots under results/):python3 plot.py --dataset glove-100-angular

Note: converting from Python to Java through py4j likely adds non-trivial overhead. This benchmarking strategy is probably not best for measuring raw QPS, but can be useful (1) to examine recall numbers, and (2) to compare different Lucene kNN approaches against each other.

Current benchmarks for Lucene's HNSW prototype vs. FAISS HNSW for k=10:
METHOD                                            RECALL
LuceneGraph(M=6, ef_const=50, ef=10)              0.644
LuceneGraph(M=6, ef_const=50, ef=20)              0.757
LuceneGraph(M=6, ef_const=50, ef=40)              0.843
LuceneGraph(M=6, ef_const=50, ef=80)              0.885
LuceneGraph(M=6, ef_const=50, ef=120)             0.902
LuceneGraph(M=6, ef_const=50, ef=200)             0.919
LuceneGraph(M=6, ef_const=50, ef=400)             0.938
LuceneGraph(M=6, ef_const=50, ef=600)             0.945
LuceneGraph(M=6, ef_const=50, ef=800)             0.950

faiss ({'M': 6, 'efConstruction': 50}, ef: 10)    0.717
faiss ({'M': 6, 'efConstruction': 50}, ef: 20)    0.813
faiss ({'M': 6, 'efConstruction': 50}, ef: 40)    0.885
faiss ({'M': 6, 'efConstruction': 50}, ef: 80)    0.934
faiss ({'M': 6, 'efConstruction': 50}, ef: 120)   0.960
faiss ({'M': 6, 'efConstruction': 50}, ef: 200)   0.984
faiss ({'M': 6, 'efConstruction': 50}, ef: 400)   0.997
faiss ({'M': 6, 'efConstruction': 50}, ef: 600)   0.999
faiss ({'M': 6, 'efConstruction': 50}, ef: 800)   0.999


2686: Add new ExportPanel to handle advanced export options

... and honor these when generating the CSV export. Also include the dateRecorded in the CourseGrade so we can pull this out for the export (as per Gradebook1) and also generate a filename that includes the site name (as per Gradebook1).
I've only extended upon the CSV file export and have omitted PDF and Excel format types for the moment - I'd like to discuss if they are really useful 😕
Delivers #2686.

Ah.. I just noticed some extra requirements in the git issue I missed e.g. being able to toggle assignments grades and comments.  I'll resubmit when I've added those features!

Okidokes! We now offer options to toggle item scores and comments, plus only offer the include points option if the categories are not weighted.  Winning.

Nice one 🍓


Make installapplications pyhthon3 compatible

Oh yea. That. 😆

That #52 but not finished yet. Could use some help.

Closing as I'm merging python3 branch into master


[C2][GPU]LengthsMax CUDA version

lengthsmax CUDA version.
This should land after test_sparse_lengths_positional_weighted_sum

Thank you for your contribution! PyTorch and Caffe2 are now officially merged with the PyTorch 1.0 prerelease. Please submit on https://github.com/pytorch/pytorch if this is still an issue over there.


Fix use case t4rec

Remove incorrect line.

@radekosmulski there is an error in this unit test tests/unit/tf/examples/test_usecase_transformers_next_item_prediction.py. we need to figure out why. something might have changed, I could repro error at my end with different dataset.

rerun tests

rerun tests


Fix for #874, change text on Stackdriver Logs date picker.

Change text from "Get log" to "Get logs"

Fixes #874


Check if loaded then set key bindings

I've split up my neovim config file for different systems and sometimes I want to have some plugins disabled. How can I check if vim-tmux-navigator has not loaded and then set key bindings. The following code always prints the echomsg, indicating that TmuxNavigateDown command does not exist. It does I'm using it.
if exists(':TmuxNavigateDown')
  " config for vim-tmux-navigator plugin
  " Do not write all buffers before navigating from Vim to tmux pane
  let g:tmux_navigator_save_on_switch = 0

  " Disable tmux navigator when zooming the Vim pane
  let g:tmux_navigator_disable_when_zoomed = 1
else
  " select window splits
  " the following key bindings are the default for the vim-tmux-navigator plugin
  "
  echomsg 'tmux-navigator not here'
  echomsg 'setting window split navigation key bindings'
  nmap <C-h> <C-w>h
  nmap <C-j> <C-w>j
  nmap <C-k> <C-w>k
  nmap <C-l> <C-w>l
endif

Assuming you're using VimPlug or pathogen or similar, they typically don't load plugin code until after they process most of the Vimrc, so that explains why the condition you're using is always falling into the echomsg branch.
Given the above limitation, I'm not sure this is something that could be solved on the plugin side. Instead, I'd wonder if you might be able to move this code to live alongside where you activate or don't activate the plugin. That way, the activation and configuration can vary together.
Hope that helps. I'm going to close this now as I don't think there is anything specific to this plugin at play here.

I'm using the native package manager.
My setup is like this:
if has($TMUX)
	packadd vim-tmux-navigator
else
    " Setup non tmux-navigator key bindings
endif


SCSS not compiling successfully on watch

Problem
When running yarn watch, changes to src/assets/styles/scss/* are detected, and the command line informs me that compilation was successful. However the compiled theme.css.liquid file does not reflect my changes. Only when I run yarn deploy does this actually work.
And of course, these changes are not reflected on the site itself.
Editing template files seems to work as expected. It's just SCSS.
Replication steps

Run yarn watch
Edit src/assets/styles/components/responsive-images.scss
Once the the command success message is shown, check the output in dist/assets/layout.theme.css.liquid

More Information

MacOS High Sierra
Slate 1.0.0-beta.5

@simonmclean styles are only outputted to the file system when you run yarn deploy. With yarn watch they are included in the dev JS bundle and injected into the page to enable features like Hot Module Reloading.
The behaviour you are describing is intended. If you want to update your theme.css.liquid file without deploying, you can run yarn build


Create targets in CMake project for packaging

There is a lot that goes on here densly packed into a few lines so here is the summary of what is going on:

CouchbaseLiteConfig.cmake.in will be used to generate a simple CouchbaseLiteConfig.cmake dist file that can be understood by the CMake find_package() operation.  For unix-like systems this will just require having the distribution put somewhere in the normal system path, however if desired it could also use the Windows solution of putting it anywhere and both adding the path to the CMAKE_PREFIX_PATH, and adding the folder with the shared library into the system path.

Using generators to choose interface header paths that are unique to build time, and install time.  The build time header paths are absolute paths to the headers as they exist on the build system, while the install time paths will be relative to the base of the distribution package and the absolute paths will be worked out at build time of the consuming project

Install CouchbaseLiteC with the EXPORT keyword, meaning it will also write a CMake style targets file that is used by the aforementioned Config file to validate and expose targets to link with

GLOB the public headers and install them to the dist folder

Just for kicks, provide a pkgconfig file so that Linux users will be happier

Write out a package version file (CMake basically does the work for us in one call) so that versions can be used with find_package().  Currently, if you call find_package(3.0.0) it will allow any version up to but not including 4.0.0.

Make sure we are consistent about include paths for external things (for now, CouchbaseLiteCStatic will continue to use non-cbl prefixed includes, except in public headers).  This means a lot of adding "cbl/" to includes

The failures seem unrelated to the changes made

All the errors in Jenkins are coming from downstream, and need to be fixed outside this PR.


Add Index on JSON Field

Is your feature request related to a problem? Please describe.
You can't add an index on json field like you would do in sql
CREATE INDEX "test_on_json_index" ON test((data->>'somefield'), "userId");

Describe the solution you'd like
A small update to @Index() to be able to do something like this
import { BaseEntity, Entity, Index, JsonType, PrimaryKey, Property } from '@mikro-orm/core'

@Index({ properties: ["data->>'somefield'", 'userId'] })
@Entity()
export class Test extends BaseEntity<Test, 'id'> {
    @PrimaryKey({ columnType: 'uuid', defaultRaw: 'uuid_generate_v4()' })
    id!: string

    @Property({ columnType: 'timestamptz', defaultRaw: 'CURRENT_TIMESTAMP' })
    createdAt = new Date()

    @Property({ columnType: 'timestamptz', defaultRaw: 'CURRENT_TIMESTAMP', onUpdate: () => new Date() })
    updatedAt = new Date()

    // Or here
    @Index("data->>'somefield'")
    @Property({ type: JsonType })
    data: PossibleData

    @Property()
    @Index()
    userId: UserId
}

Or something like this
import { BaseEntity, Entity, Index, JsonType, PrimaryKey, Property } from '@mikro-orm/core'

@Index({ properties: [{data: { somefield: true }}, 'userId'] })
@Entity()
export class Test extends BaseEntity<Test, 'id'> {
    @PrimaryKey({ columnType: 'uuid', defaultRaw: 'uuid_generate_v4()' })
    id!: string

    @Property({ columnType: 'timestamptz', defaultRaw: 'CURRENT_TIMESTAMP' })
    createdAt = new Date()

    @Property({ columnType: 'timestamptz', defaultRaw: 'CURRENT_TIMESTAMP', onUpdate: () => new Date() })
    updatedAt = new Date()

     // Or here
    @Index({ properties: {data: { somefield: true }} })
    @Property({ type: JsonType })
    data: PossibleData

    @Property()
    @Index()
    userId: UserId
}

Which when you try to create a migration return
MetadataError: Entity Test has wrong index definition: 'data->>'somefield'' does not exist. You need to use property name, not column name.
Describe alternatives you've considered
Manual migration but mikro-orm try to delete my index with every new migration created, but maybe I missed something

Manual migration but mikro-orm try to delete my index with every new migration created, but maybe I missed something

A workaround is to define any index with some name, index diffing works just based on index names, so it is enough to have some index with the name and it will be kept intact.

Manual migration but mikro-orm try to delete my index with every new migration created, but maybe I missed something

A workaround is to define any index with some name, index diffing works just based on index names, so it is enough to have some index with the name and it will be kept intact.

In v5 you can custom index expressions:
@Index({ name: 'test_on_json_index', expression: `CREATE INDEX "test_on_json_index" ON test((data->>'somefield'), "userId");` })

This works both on entity level and property level and you have absolute control over the index definition.
The diffing works again based on index name (that can be either explicitly defined or inferred from the property name). The previous workaround probably won't work anymore as indexes are now properly diffed not just based on name, but also based on fields and their order.
I still see a room for the proposed solution, probably with dot notation instead of objects or platform specific syntax, like in #2129:
@Index({ properties: ['data.somefield', 'userId'] })
@Entity()
export class Test extends BaseEntity<Test, 'id'> {

    @Index({ properties: 'data.somefield' }) // can be a string too
    @Property({ type: JsonType })
    data: PossibleData

}

Didn't know about the upcoming v5 change, thanks for the heads up
The idea with object syntax was to provide autocompletion and more importantly TS type error if something change in the future

The idea with object syntax was to provide autocompletion and more importantly TS type error if something change in the future

v5 supports type safe dot notation paths, but in general you can't have anything type safe inside decorators unless you explicitly provide the generic argument (it can't be inferred from the class where the decorator is used)

Yeah, I was speaking more with using something like
@Index({ properties: { data: { field: true | 'asc' | 'desc' } } } as IndexType<JSONType>)

But if dot notation in paths is typesafe I'm all in 💯
My only downside from using mikro-orm despite all the amazing work you and contributor put in is type safe usage compared to prisma (still lacking micro-orm flexibility)

What would be the current workaround in v5 for nested Index using the mongo driver?

What would be the current workaround in v5 for nested Index using the mongo driver?

Found the solution which does not allow for extra options to be passed as far as I understand though
@Index({ options: { "address.location": "2dsphere" } })

What would be the current workaround in v5 for nested Index using the mongo driver?

Found the solution which does not allow for extra options to be passed as far as I understand though
@Index({ options: { "address.location": "2dsphere" } })

It works, on entity level! Thanks!

Closing as implemented in v6 via #4735


Question about Fine-tune

If I directly use  python detection_train.py --config config/model_config.py  and experiments/model/model_0018.params exists ,will it load by the model?Or i need to change begin_epoch to 18. and my class_num is 1 + 10 , i need to delete the head in the params or is there another method to use.

Your understanding is correct.
https://github.com/TuSimple/simpledet/blob/6f826134cf699223caee258f91d824706eab6bb9/detection_train.py#L127-L128
You should change the begin_epoch, num_class and delete the head params from arg_params.
Maybe we can add a more detailed guide for finetuning models from existing checkpoint.

You should change the begin_epoch, num_class and delete the head params from arg_params.
Thanks for your reply, it helps a lot. And I observe the params ,you mean i need to delete "bbox_cls_logit_bias", "bbox_cls_logit_weight"？

Your understanding is correct.
https://github.com/TuSimple/simpledet/blob/6f826134cf699223caee258f91d824706eab6bb9/detection_train.py#L127-L128
You should change the begin_epoch, num_class and delete the head params from arg_params.
Maybe we can add a more detailed guide for finetuning models from existing checkpoint.

Thanks for your reply, it helps a lot. And I observe the params ,you mean i need to delete "bbox_cls_logit_bias", "bbox_cls_logit_weight"？

Hi, @XuShoweR.

Yes, delete the weight and bias of bbox_cls_logit.
It should be fine for declaring more classes than there actually is. Since we obtain the final classification result from a softmax activation, I think the unseen classes should have large negative biases as they are always panelized by the cross-entropy loss.

Hi, @XuShoweR.

Yes, delete the weight and bias of bbox_cls_logit.
It should be fine for declaring more classes than there actually is. Since we obtain the final classification result from a softmax activation, I think the unseen classes should have large negative biases as they are always panelized by the cross-entropy loss.

According to your guidance, training has begun .Thank you very much for your timely and patient reply.Thanks！


Feature Request : Allow the ability to change languages in the admin dashboard

adding the ability to change the Ui language in the admin dashboard will open pocket base to a whole new audience, as of me, I have many friends that really loved the simplicity pocket base dashboard but had problems with the language as they speak Arabic and they're really weak with English. Our communities Can help you translate the Ui for You

Sorry, for now there are no plans for this as it will be really hard to maintain.
See also https://github.com/pocketbase/pocketbase/issues/37 and https://github.com/pocketbase/pocketbase/pull/85.


Log reason when file key store is not accessible

I have the problem that I often get exceptions when loading the key store, but I can't tell the reason from the logs.
This pull requests adds the reason to the logs.

Thanks very much, merged!


DetectRTC causes window to be defined in nodejs environments

https://github.com/muaz-khan/DetectRTC/blob/master/DetectRTC.js#L33 defines window as a property of global, which causes checks like typeof window === 'undefined' to run browser code in node environments.

WontFix. Because grunt fails if we ignore window object. If you've a solution, please make a pull request.

Can you explain what you mean by "grunt fails"? I just commented out the offending line (which now seems to be 39) and grunt still successfully built DetectRTC.js.
I also tried running npm test according to the README, but got
$ npm test

><EMAIL_ADDRESS>test /Users/ian/projects/DetectRTC
> echo 'Error: no test specified'

Error: no test specified

Finally, I modified index.html to point at ./DetectRTC.js instead of the cdn-hosted one and it still appears to work.

This one fails:

https://github.com/muaz-khan/DetectRTC/blob/master/npm-test.js

node npm-test.js

i.e. DetectRTC fails to run inside the node.js environment.


Delete lasted Ruby references

Description
Looks like recently closed PR#1102 missed some Ruby references.
This PR cleaned the rest.

To be sure, let's wait to merge this until after #1106 and we rerun CI.


compose: Prototype upload-file UI

Fixes: #57

I found Dart's new Record feature helpful in a few places!

Thanks for the review! Revision pushed, and please see my response here: https://github.com/zulip/zulip-flutter/pull/63#discussion_r1167213308

Revision pushed, fixing https://github.com/zulip/zulip-flutter/pull/63#discussion_r1167313863 and some indentation things.
Also in this revision, I've made it so the content input gets focused when you select files, if it wasn't focused before.

Thanks for the review! Revision pushed.


feat: remove raw string prefix in the hover docs

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix #1555

feat: remove raw string prefix in the hover docs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 70.549%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/sema/src/resolver/doc.rs
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

kclvm/sema/src/resolver/doc.rs
1
95.63%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.01%

Covered Lines:
50510

Relevant Lines:
71596

💛  - Coveralls


Update documentation - spelling

achive --> achieve

Thanc yu four yer contribtion.

Thanc yu four yer contribtion.


Updates for Haxe 4.0.5

Mostly fixed all the warning and errors I had on during the usage of the Library with latest Haxe
PATTERN_NON_PRINTABLE and PATTERN_NON_ASCII_LINE_BREAKS are now using a if else construct and will not complaining about Regexp not being utf-8 compliant. Difficult to test as this is only used for Yaml 1.1 documents and depends on the underlying RE engine.
Use UnicodeString methods instead of Utf8.hx
Added interp and lua  to the list of tested platforms

@octplane hey thanks for all the changes here. Just getting a little time to look at this now.
I don't actively maintain this repository anymore, and have not used haxe for many years, so can I ask how much testing you did of this work specifically across targets? I'm a little concerned given the scope of changes it could cause regressions on some platforms, but I don't have capacity to fully test these, and test coverage isn't amazing unfortunately.
That said I'd like to get upgrades out as I can see it's causing people some problems.

Not a lot of test unfortunately, I merely hacked until it built correctly....


SdFatSoftSpi usage with GSLC_SD_EN 2

Describe the bug
I had a brief discussion about this issue over email with @ImpulseAdventure, this is regarding usage of SD library with GSLC_SD_EN 2
Current State:
The project does not utilize any SD function through guislice so GSLC SD EN is also set to 0, and in this state I can declare things like:
File timerfile;
File runstatus;

also these are declared in the beginning:
#include <SdFat.h>
SdFatSoftSpi<12, 11, 13> SD;

In this way everything works fine and I can access txt files in SD and do stuff in the main loop.
The Problem:
I want to show images from SD through the guislice library/builder, so I enable GSLC_SD_EN 2 and also make this change in the SdFat library as mentioned in GUIslice_drv_adagfx.cpp   (// -     #define ENABLE_SOFTWARE_SPI_CLASS 1 // Change default from 0 to 1)
After doing these and as adviced by Calvin I remove the following from the beginning since they are being declared by the guislice library itself:
#include <SdFat.h>
SdFatSoftSpi<12, 11, 13> SD;

and this from void setup
if (!SD.begin(10)) {
    Serial.println("initialization failed!");
 return;
  }
  Serial.println("initialization done.");

Now if I compile, I get errors like these
sequence_controller_2_SD:24: error: 'File' does not name a type

 File timerFile;

 ^~~~

sequence_controller_2_SD:25: error: 'File' does not name a type

 File runstatus;

 ^~~~

sequence_controller_2_SD:26: error: 'File' does not name a type

 File relaynumber;

and many more errors in other places where SD is used in the code previously.
Device hardware

MCU: ATmega2560
*Display: using mcufriend kbv 4 wire for a 2.4inch display

Hi @rvxfahim --
Could you first try downgrading SdFat to version 1.1.4 via the Arduino Library Manager (and again set ENABLE_SOFTWARE_SPI_CLASS 1 in the SdFatConfig.h)?
Some changes were made in SdFat v2 that may not be compatible at the moment.
If that still doesn't work, could you attach your sketch for me to test?
thanks

I did a couple of testings with the new Guislice repo

Can't use SdFat v2 because SdFat v2 no longer has the "File" class which was code compatible with the "File" class of builtin arduino SD.h. It is now replaced with "File32". Therefore, I am not sure if replacing my existing code keywords from "File" to "File32" will work. I need some time to learn how to use the SdFat v2 library before making any changes to existing code.

I was already using SdFat 1.1.4, and just to avoid any doubt I reinstalled the SdFat library and set ENABLE_SOFTWARE_SPI_CLASS 1.
With the new guislice library there is one problem in GUIslice_drv_adagfx.cpp.
if (!SD.begin(SD_CONFIG)) this is in line 748 and SD_CONFIG is defined in line 199 which is for SdFat v2. So compiling it gives error about SD_CONFIG not defined. So I replaced SD_CONFIG with 10 (SD CS Pin) in line 748 : if (!SD.begin(10)

At this point my main goal is to use GSLC SD EN 2 and show images from SD and also use SdFat library to read and write into some text files.
So here is the change that made everything work
I had to include <SdFat.h> again in the main .ino file; not redeclaring = compile error
Also this line:
SdFatSoftSpi<12, 11, 13> SD1; Guislice uses SD and I am using SD1.
I also had to put this in void setup
if (!SD1.begin(10)) {     //Serial.println("initialization failed!");  return;   }
and then I replaced all keywords in my code where "SD" was previously used to "SD1".
Now it compiles fine and I can see images pulled from SD by guislice and also I can read/write to text files.
project file.zip

Thank you @rvxfahim for the detailed testing -- it's much appreciated!
As noted, I have not spent much time testing the software SdFat mode... and the recent breaking changes to the SdFat API have added to the complexity.
Given your feedback with SdFat v2, it sounds like others users may also hold off on updating to v2. Therefore, it would be wise to ensure GUIslice still runs well with SdFat v1.
I'll take a look at your suggested changes to v1 and will also ensure that I have a test setup locally configured so that I can confirm everything works properly. Please note that I may not have an opportunity to do this for the next couple weeks due to other work. In the interim, it sounds like you have identified a viable workaround, which is good to hear.
thanks!

Thank you @rvxfahim for the detailed testing -- it's much appreciated!
As noted, I have not spent much time testing the software SdFat mode... and the recent breaking changes to the SdFat API have added to the complexity.
Given your feedback with SdFat v2, it sounds like others users may also hold off on updating to v2. Therefore, it would be wise to ensure GUIslice still runs well with SdFat v1.
I'll take a look at your suggested changes to v1 and will also ensure that I have a test setup locally configured so that I can confirm everything works properly. Please note that I may not have an opportunity to do this for the next couple weeks due to other work. In the interim, it sounds like you have identified a viable workaround, which is good to hear.
thanks!


Could this package be compatible with Laravel Telescope?

Could this package be compatible with Laravel Telescope?
I tried install Telescope and on dashboard, when trying to read requests, in console I get "Call  to a member function prepare() on null".

exception: "Symfony\Component\Debug\Exception\FatalThrowableError"
file: "/var/www/html/broc/vendor/laravel/framework/src/Illuminate/Database/Connection.php"
line: 326
message: "Call to a member function prepare() on null"

you mean log the mongodb queries? or store the telescope logs in mongodb?
The first is easy you need to enable the query log:
DB::connection('mongodb')->enableQueryLog();
the second is harder. Its probably more than writing a "storage adaptor" since the package might use some raw sql queries :s
Worst case scenario you might have to fork the package & dive into the code

Bump

Has anyone been able to get this to work? It is unfortunate to not have this when switching over to mongo. It was great with mysql. I would love to see some examples of what needed to be changed to get it working. It's been really hard to find anything informative online so far.


How to use correctly state.dismissDragProgress

I´m trying to use state.dismissDragProgress to lower the opacity of the zoomable when is dismissed, but after a rotation change or even sometimes just swiping to the next image the progress is not working anymore.
As example, took the example and added this line to the zoomable modifier :
modifier = Modifier.alpha(1 - state.dismissDragProgress)
Is that the correct way of using it?
Thanks!

Yeah, generally it's the correct way, but using Modifier.graphicsLayer { alpha = 1 - state.dismissDragProgress) } should be more efficient.
I've updated the sample to include the usage of dismissDragProgress.


Per notetype scroll speed

Hello! In a mod I worm on we are usuing this. But we originally had a notetype that scrooled faster usuing the multspeed variable. But with this active it doesn't seem to work anymore. Anyone know a fix for it. Thanks in advance

good luck and happy coding

Thank you for the feedback! know any good spots to look into to start?

well you can make that your custom notetype run this code
startMod('speed', 'speedModifier', 'lane', -1) --makes a modifier
and once the note spawn you set this modifier only to that note, im not sure if it will fully work but you can test it
if you have another question i'll be glad to help

oh i didnt even think of it like that! would it be posible to call these functions through source?

yeah you can do
Modcharting.modchartFuncs (import it)
then call the functions there (was made for lua but works with source too)
so you do the same
startMod(blah blah blah)
but you make that one note have this modifier applied


Adding 'due someday' when task is over 10 years in the future

Resolves #22

Looks good! Thanks for the contribution 👍


[BUG] Cannot deserialize Microsoft.Azure.Search.Models.SearchResult

Describe the bug
The latest version of the the Microsoft.Azure.Search.Models.SearchResult class has Document with no public setter, and no JsonProperty attribute for deserialization.  As a result, the Document property cannot be deserialized.  Score and Highlights deserialize correctly.
We return a list of Microsoft.Azure.Search.Models.SearchResult from an API, and our integration tests deserialize the returned JSON to the C# type to validate equivalence.
Expected behavior
Microsoft.Azure.Search.Models.SearchResult should be deserializable from JSON, as well as serializable.
Actual behavior (include Exception or Stack Trace)
The Document property is null after deserialization.
To Reproduce
    public class TestClass
    {
        public TestClass(string testString)
        {
            this.TestString = testString;
        }

        public TestClass()
        {
        }

        public string TestString { get; private set; }
    }

    public class Program
    {
        public static void Main(string[] args)
        {
            var test = new List<SearchResult<TestClass>>()
            {
                new SearchResult<TestClass>(new TestClass("Foo"), 45.0),
                new SearchResult<TestClass>(new TestClass("Bar"), 22.7)
            };

            var serialized = JsonConvert.SerializeObject(test, new JsonSerializerSettings() { ContractResolver = new DefaultContractResolver() });
            var deserialized = JsonConvert.DeserializeObject<List<SearchResult<TestClass>>>(serialized, new JsonSerializerSettings() { ContractResolver = new DefaultContractResolver() });

            Console.WriteLine(test.ToString());
            Console.WriteLine(serialized);
            Console.WriteLine(deserialized);
        }
    }

Environment:
Microsoft.Azure.Search 10.1.0

Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows 10 .Net Core 3.1 in ASP.NET Core 3.1 web project
IDE and version : Visual Studio 2019 16.4.5

//cc: @tg-msft, @AlexGhiondea

@rcbevans What you're trying to do is not a supported scenario, but you might be able to achieve it if you use the DeserializationSettings property of SearchIndexClient when calling JsonConvert. Let me know if this workaround unblocks you.
That said, I would advise you not to take a dependency on the SDK's model classes in your own API surface. That's a degree of coupling that will come back to haunt you later, because these model classes will change.

Yeah, I understand.  My preference would be us use Automapper and have DTOs on the API surface, but this code predates me; I'm just doing the work to upgrade the platform my team is using.  I was able to achieve the same end result and get the tests passing my defining my own TestSearchResult with properties accessible properties mirroring the Azure type, and use FluentAssertions Should().BeEquivalentTo() to verify equivalence.

@rcbevans Glad to hear you're unblocked. I'm closing this issue since it's not a bug per se. If external serialization of models is something you think Azure .NET SDKs should support, please open a new issue to track it as a feature request.

@brjohnstmsft Why is this not considered a bug? We have implemented a Redis cache layer to avoid throttling issues, but above issue causes the document to always be null. The proposed workaround doesn't work for me, I guess the only way around it is to depend on the SDK models?

@dauv The model classes for Microsoft.Azure.Search were designed in tandem with custom serialization logic specific to that library; There was never an intention to support serialization/deserialization of these types by 3rd parties. That's what I meant earlier when I said this was an unsupported scenario. We made no promise that this would work, so I view it more as an intentionally missing feature than a bug.

I guess the only way around it is to depend on the SDK models?

It's the opposite -- write your own models instead of depending on the SDK models. That approach worked for @rcbevans.
That said, I'm curious why you need to represent the search result payload as strongly-typed objects in a caching layer. Do you have code that manipulates the results before returning them to the client?
Also please note that Microsoft.Azure.Search has been deprecated; Please consider upgrading to Azure.Search.Documents if possible. There's a migration guide here.


Add template live reloading

For now you need to restart server after template change or use some other tools for prototyping. It will be really great if bootique-mvc can reload template in runtime. This may be configurable, to disable on production.

Does this describe Fremarker templates behavior? As Mustache default behavior is complete opposite - templates are reloaded on every render (see #3)

Actually there is a MustacheFactory.compile(..) flavor that doesn't take a Reader that does permanent caching of templates. We may have used it in the past. So closing this task, and will need to open one for Freemarker


Problems with '||' patterns in dependency versions?

From here:
{
    "private": true,
    "dependencies": {
        "styled-components": ">=2.0 || >=3.0"
    }
}

Trying to run DefinitelyTyped locally, I get the following:
Error: In C:\Users\Daniel\DefinitelyTyped/types/grid-styled/package.json: Dependency styled-components not in whitelist.
If you are depending on another `@types` package, do *not* add it to a `package.json`. Path mapping should make the import work.
If this is an external library that provides typings,  please make a pull request to types-publisher adding it to `dependenciesWhitelist.txt`.

On the other hand, I don't know how this would work with a not-dumb version specifier (that's really just the same as >= 2.x

Shakes fist at air: "yarn.lock!!!!"


My two cents

Copolot suggested this repository while adding additional tokens (James) to my tokenizer.
Here's my two cents:
I'm afraid to say that this is basically character-level encoding or the same as one hot encoding with every single Korean character in the vocabulary because embedding is doing the same thing already.
and three hot encodings is what exactly the "Unicode" Korean table does, too.

Hi, thanks for the comments.

Copolot suggested this repository while adding additional tokens (James) to my tokenizer.
Interesting, this is my first time having that happen. I am flattered.

I'm afraid to say that this is basically character-level encoding or the same as one hot encoding with every single Korean character in the vocabulary because embedding is doing the same thing already.

I think you are slightly misunderstanding what our work does. We are doing character (= syllable/음절) level modeling, but we are doing it in a way that reduces parameter counts by only using subcharacter/자모 features. You can read about it here: https://aclanthology.org/2023.eacl-main.172/.
On the encoding side there are roughly 3 options:
One-hot syllable: requires 11k embedding vectors
One-hot jamo: requires ~70 embedding vectors, but 3x sequence length
Three-hot syllable: requires 70 embedding vectors but syllable-level sequence length
Our's is three-hot syllable, so we do produce a single syllable-level encoding for each syllable in the text, but its made from a combination of the component jamo parts.
However, our work mainly focused on the output side, where there is a fourth option: independent three-hot syllable (https://koreascience.kr/article/CFKO201832073079068.pdf). We show that this one doesn't properly model syllables, and propose conditional three-hot syllable decoding which also only requires ~70 embedding vectors and outputs full syllables in one timestep.
So, to summarize, we are doing character-level encoding but with a reduced parameter count (11k -> 70 embedding vectors but no sequence length increase).


[FEATURE] sglang integration

Description
Integrate sglang

@gabrielmbmb: Would it be possible for me to work on this issue? I'd appreciate the opportunity to learn and contribute to the project.

sure @bikash119!


Cookie with WKWebView are not persistent

Bug Report
Hi, I recently migrated to WKWebView since UIWebView has been deprecated.
With the new view I am having a lot of problems, especially regarding cookies during ajax calls, which are not maintained and consequently I cannot login to my app.
Can anyone tell me if they can somehow be saved so that the session_id is always the same and does not change with every single call?
Problem
to log in to my app I need cookies to be saved, without those the call to the server fails because the session is not set, given that with each call the PHPSESSID is different
What is expected to happen?
I expect cookies to be saved or that there is still a way to make this happen
What does actually happen?
Noe cookie are not saved or stored, they are not persistent
Information
Command or Code
Environment, Platform, Device

iPhone 6s -> 13.4
iPhone 11 -> 13.5
xcode 11.4.1
Version information

I'm using:;
"cordova-ios": "^5.1.1",
"cordova-plugin-wkwebview-engine": "^1.2.1"
"cordova-plugin-wkwebview-inject-cookie": "1.0.6"
Checklist

[x] I searched for existing GitHub issues
[x] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

when I by removing the "cordova-plugin-wkwebview-engine" then build successfully but IOS App rejected
**Guideline 2.1 - Performance - App Completeness
We discovered one or more bugs in your app when reviewed on iPad and iPhone running iOS 13.5.1 on Wi-Fi.
Specifically, the buttons in the app were unresponsive.**
I have been looking at the forums for a long time but there is no solution. Cordova crash !!!!!!!
I think it's time to cross over to Xamarin. Nobody is interested.
cordova cli 9.0
cordova platform add<EMAIL_ADDRESS>

I was facing a similar problem earlier and then found out that the cordova-plugin-inappbrowser was causing the trouble..
Do you use this plugin as well?
By adding the option clearcache=no the problem was solved for me.
See https://cordova.apache.org/docs/en/latest/reference/cordova-plugin-inappbrowser/#cordovainappbrowseropen for more information.

Same

We are archiving this repository following Apache Cordova's Deprecation Policy. We will not continue to work on this repository. Therefore all issues and pull requests are being closed. Thanks for your contribution.


[ Feature ] Display group's dropdown list in multiple columns

The group's dropdown list does not fit in the screen.
Could it be possible to display them in multiple columns ?

You have crazy workflows, you know that?

It will be better once we have multi row headers ( https://github.com/chrisgoringe/cg-controller/issues/303 )
Now I have to use abbreviations for groups, but it's all "normal" stuff.

5 image batch
Image2Image
ControlNet Depth + Canny
InstantID + IPAdapter FaceID + IPAdapter Embeddings
Model merge
Loras
2 Prompt Conditioning ( Combine / Concat / average )
4 Sampler stages

KSampler
KSampler ( Refiner / Upscaler )
Face Detailer
Ultimate Upscaler ( tiled upscale )

For the sake of simplicity, I think this one can be closed too.
I was adding to many extra groups, to just create more tabs in the GUI.
But I learned to optimize the Controller layout, so I need less groups, for the same features.
And now we also have features, like the "favorites" tab, and the "stacked" tabs, that help with that.


"Guide" crumb for all language guides

Regardless of the @ngdocs link specified at index, for all languages, the initial breadcrumb for "Guide" is "Guide" (instead of "руководство", "guía", etc). It just happens to be the same term for German, French and English, but not for the rest.
It sounds like an easy fix, but I don't know how to do it. :)

Hello,
in part of a "Spring Cleaning 2017", we are closing this issue as part of several others. The meta discussion is available here for full transparency.
 Said this, we must keep the focus and scope on the project.
We are grateful and appreciate to everyone for each made contribution (regardless being code, idea, documentation, and support).
We are deprecating/closing all issues which fulfills a specific set of criteria: mainly outdated, nice-to-have, or just out-of-scope.

Although the list has been curated to the best of our's knowledge, there will be some lack of understanding.
Okay, there a different stories behind all open issues, but we do not have the time for everything:

Support: Because we have only limited resources, we cannot handle support questions any more. Please use appropriate boards like Stackoverflow.
Feature requests or ehancements: Altough it is probalbly a great idea (otherwise it would be closed already), we can not giving more resources to things out of scope or which are useful for too specific scenarios only.
Bugs: Filed a long time ago, this bug either has a reasonable workaround, no effective solution without harming all others, or is lacking additional information/use-case.
Docs: All documentation efforts not matching English will be abonded.

Thank you.
angular-translate team

FAQ
"But this is really important!"
Well, we are all humans. Maybe we have overseen an important issue. If we missed something, we hope you are reaching as again filing a new issue with the reference to the old one. But please be aware: A bug without further details and a demo included will be rejected again.
"But this would a great feature!"
Reconsider: Are we really required to add something in order to make this feature happen? Or could you also create a project on your own utilizing this project?


pull request for e2e test at 01/15/2017 02:45:09

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error when running demo_track.py - does not work on CPU

I have created a VM with Linux Ubuntu and installed all dependencies.
I guess the following error is due to having no GPU. It seems the code does not work correctly for device='cpu'.
Do you know how to fix your code to support running pretrained models with tracking part on CPU only?
Thanks for your support!
(MyEnv)root@mv:/home/mv/ByteTrack# python3 tools/demo_track.py video -f exps/example/mot/yolox_x_mix_det.py -c pretrained/bytetrack_x_mot17.pth.tar --fp16 --fuse --save_result --device=cpu

Matplotlib is building the font cache; this may take a moment.

2021-11-04 | INFO     | __main__:main:298 - Args: Namespace(camid=0, ckpt='pretrained/bytetrack_x_mot17.pth.tar', conf=None, demo='video', device='cpu', exp_file='exps/example/mot/yolox_x_mix_det.py', experiment_name='yolox_x_mix_det', fp16=True, fuse=True, match_thresh=0.8, min_box_area=10, mot20=False, name=None, nms=None, path='./videos/palace.mp4', save_result=True, track_buffer=30, track_thresh=0.5, trt=False, tsize=None)

[W NNPACK.cpp:79] Could not initialize NNPACK!  Please do not use them for anything important until they are released as stable. (Triggered internally at  /opt/conda/conda-bld/pytorch_1623448265233/work/c10/core/TensorImpl.h:1156.)
  return torch.max_pool2d(input, kernel_size, stride, padding, dilation, ceil_mode)

2021-11-04 | INFO     | __main__:main:308 - Model Summary: Params: 99.00M, Gflops: 791.73
2021-11-04 | INFO     | __main__:main:319 - loading checkpoint
2021-11-04 | INFO     | __main__:main:323 - loaded checkpoint done.
2021-11-04 | INFO     | __main__:main:326 - 	Fusing model...

/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py:561: UserWarning: The .grad attribute of a Tensor that is not a leaf Tensor is being accessed. Its .grad attribute won't be populated during autograd.backward(). If you indeed want the gradient for a non-leaf Tensor, use .retain_grad() on the non-leaf Tensor. If you access the non-leaf Tensor by mistake, make sure you access the leaf Tensor instead. See github.com/pytorch/pytorch/pull/30531 for more information.
  if param.grad is not None:

2021-11-04 | INFO     | __main__:imageflow_demo:240 - video save_path is ./YOLOX_outputs/yolox_x_mix_det/track_vis/2021_11_04/palace.mp4
2021-11-04 | INFO     | __main__:imageflow_demo:250 - Processing frame 0 (100000.00 fps)

Traceback (most recent call last):
  File "tools/demo_track.py", line 357, in <module>
    main(exp, args)
  File "tools/demo_track.py", line 350, in main
    imageflow_demo(predictor, vis_folder, current_time, args)
  File "tools/demo_track.py", line 253, in imageflow_demo
    outputs, img_info = predictor.inference(frame, timer)
  File "tools/demo_track.py", line 166, in inference
    outputs = self.model(img)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/mv/ByteTrack/yolox/models/yolox.py", line 30, in forward
    fpn_outs = self.backbone(x)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/mv/ByteTrack/yolox/models/yolo_pafpn.py", line 93, in forward
    out_features = self.backbone(input)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/mv/ByteTrack/yolox/models/darknet.py", line 169, in forward
    x = self.stem(x)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/mv/ByteTrack/yolox/models/network_blocks.py", line 210, in forward
    return self.conv(x)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/mv/ByteTrack/yolox/models/network_blocks.py", line 54, in fuseforward
    return self.act(self.conv(x))
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 443, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/root/anaconda3/envs/MyEnv/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 440, in _conv_forward
    self.padding, self.dilation, self.groups)

RuntimeError: Input type (torch.FloatTensor) and weight type (torch.HalfTensor) should be the same or input should be a MKLDNN tensor and weight is a dense tensor

You can try running the command by deleting '--fp16'.

@ifzhang
When I run, demo_track.py, the error occurr "cannot import name 'plot_tracking' from 'yolox.utils.visualize' (/mnt/d/PycharmProjects/yolox_deepsort_devs/yolox/utils/visualize.py)".

@ImSuMyatNoe  did you find the reason?

You can try running the command by deleting '--fp16'.

It works well with 'fp16=False'.
Thanks for your kind guide.

@ifzhang When I run, demo_track.py, the error occurr "cannot import name 'plot_tracking' from 'yolox.utils.visualize' (/mnt/d/PycharmProjects/yolox_deepsort_devs/yolox/utils/visualize.py)".

visualize.py
line 8 notes or delete it
all = ["vis"]

@ifzhang When I run, demo_track.py, the error occurr "cannot import name 'plot_tracking' from 'yolox.utils.visualize' (/mnt/d/PycharmProjects/yolox_deepsort_devs/yolox/utils/visualize.py)".

visualize.py line 8 note or delete it all = ["vis"]

it seems not work

@ifzhang When I run, demo_track.py, the error occurr "cannot import name 'plot_tracking' from 'yolox.utils.visualize' (/mnt/d/PycharmProjects/yolox_deepsort_devs/yolox/utils/visualize.py)".

visualize.py line 8 note or delete it all = ["vis"]

it seems not work

maybe use python run demo_track.py instead of python3

same problem, and I have tried the above methods, but don't work.
(ByteTrack) hw@iadc:projects/ByteTrack ‹main*›$ python3 tools/demo_track.py video -f exps/example/mot/yolox_x_mix_det.py -c pretrained/bytetrack_x_mot17.pth.tar --fuse --save_result
Traceback (most recent call last):
  File "tools/demo_track.py", line 17, in <module>
    from yolox.utils.visualize import plot_tracking
ImportError: cannot import name 'plot_tracking' from 'yolox.utils.visualize' (/home/hw/projects/ByteTrack/ByteTrack/lib/python3.8/site-packages/yolox/utils/visualize.py)


Note: The repository name has changed from 'bot2014' to 'bot'

Just to save anyone the trouble of wondering why they can't pull.
You can fix your problems just by using:
git remote set-url origin git@github.com:IEEERobotics/bot.git

Do you think we should also update the Readme.md file to reflect the change in repo name? (It's a broken link if you click on the icon right now)

Do you think we should also update the Readme.md file to reflect the
change in repo name? (It's a broken link if you click on the icon right now)

Yes, please do so.
Generally, situations where you find broken docs for open source projects,
it's the right call to fix them. :)
On Tue, Jun 30, 2015 at 10:12 PM Vijay Thiagarajan<EMAIL_ADDRESS>wrote:

Do you think we should also update the Readme.md file to reflect the
change in repo name? (It's a broken link if you click on the icon right now)
—
Reply to this email directly or view it on GitHub
https://github.com/IEEERobotics/bot/issues/373#issuecomment-117399093.


Use unix sockets, not IPv4, for the dummy DB configuration.

By default, we trust local unix socket connections[1], but not
<IP_ADDRESS> or ::1.   Therefore, we should use unix sockets for the rake task
that ensures we don't try to connect using a badly configured
database URL when we try to load rails environment.
[1] https://github.com/ManageIQ/guides/blob/master/developer_setup.md
Fixes #6915

@agrare thanks for the help!

Thanks for the fix @jrafanie !

Out of curiosity, why do we have the instructions have that weird sed line, but it isn't necessary on Mac?

Must be a difference in the way the pg_hba.conf is configured in homebrew vs. rpm.


Generic environments for opened archetypes

Give opened archetypes generic environments, so that they can be used more uniformly compared to primary archetypes, and so they can eventually support nested archetypes.

@swift-ci Please test

@swift-ci Please test source compatibility