fix in motion controllers docs

@geyang The example script for the quest motion controllers has a typo in the import from vuer.schemas . The class definition is MotionControllers not MotionController, just fixed that in docs.

updated live version of docs, thanks for your contribution
https://docs.vuer.ai/en/latest/examples/20_motion_controllers.html


Conditionally disable initializer

I'm proposing the ability to conditionally disable the auto injection of store, as well as renaming the initializer.  This will allow ember-cli-simple-store to live along side ember-data, which is a circumstance we're dealing with while we need the two to live in harmony as we port the application over to ember-cli-simple-store.
Hopefully this will help others.  Thanks again for your hard work :+1:

Nice work! I'll look at this tonight before we merge it /publish v3.6.0 assuming that's cool w/ you

Sounds good :+1:

I'm setting this up quick in my client app to see if all is good 2 go :) reporting back in just a few

Sadly in production builds I'm getting this blow up :(
Have you used this exact setup (ie- copy/pasted the changes into your node_modules) ?

Only in production builds?

If we instead ask that this config is put on the APP key/value setup we could grab that from the original initializer
APP: {
      disableAutoInject: true
    }

then in the addon ...
    var timeout = application.disableAutoInject || false;

thoughts?

@jasonmit yes -seems my test build was fine but in production no dice

Does your client use ember-cli-babel?

@jasonmit
    "ember-cli-babel": "^5.1.5",

We are running the usual stack w/ ember-cli 1.13.13

Strange, can you try and change initialize() { to initialize: function() { inside of app/initializers/ember-cli-simple-store` just to humor me..

If we instead ask that this config is put on the APP key/value setup we could grab that from the original initializer

I'd be fine with this, but what I have in place should work.  I've also tested it in a product environment against a brand new dummy app (1.13.13) and cannot reproduce this.

@jasonmit let me give the function approach a run this morning to see if that works ... strange that only my app broke :) user error likely

@jasonmit no dice on the function -what version of ember are you using in the apps that didn't have a problem w/ this setup?

Also, does your app use i18n or any other addon with a custom initializer ?

1.13.13 and I implemented the initalizer in the README as part of the test.  Want to pair on this on hangout?

Or is the app online anymore or something I can run locally? (The disted app assets are fine)

@jasonmit I may have some time tomorrow afternoon - you available at all this weekend?

Not tomorrow, every other day including Monday.  Did you also try pointing to 3.5.0 to make sure it is the ember-cli-simple-store initializer that is blowing up.  Do you have anything importing and then trying to access ember-cli-simple-store/initializers/ember-cli-simple-store?

No worries- I found the issue that caused it to fall down (not the function or any of the code inside that). Instead using the name of simple-store (instead of store) seems to be the regression
export default {
    name: 'simple-store',
    initialize() {}
}

I'm having a terrible time trying to pause on the exception /etc so I'm not yet sure the issue. I had a few initializers that referenced "store" in the after but I've changed those to simple-store thinking that would solve my issue. I'll keep looking today and see if I can help provide a fix for those who might run into this same issue.

@jasonmit and just like that I found the issue :) another initializer w/ that old "store" name. I updated it to be simple-store and all is good now. I'll pull this in and do a v3.6.0 bump today

@jasonmit I rewrote the top 3 commits to make my rebase a lot easier/cleaner. I then rebased the v4.0 changes and pushed up a new branch so I can publish both a v3.6.0 and a v4.0.0-beta.6 today
https://github.com/toranb/ember-cli-simple-store/tree/branches/v4
Going forward can only work on the new branch shown above? If the v3.6.0 series is solid we should be able to move forward on the v4 series only making the merge back to master a lot simpler in a few weeks.

@jasonmit I just released both v3.6.0 and v4.0.0-beta.6
I'm running a full build of my 4.x app using the latest. Can you verify v3.6.0 works as you expected it would?

@toranb awesome, glad you were able to resolve this!  3.6.0 looks like it works as expected.
I'll start implementing 4.x into a project Tuesday so I can provide some feedback and chip in with any feature/bug work needed.


Membership number: user import by referencing data didn't work

Description: The import of existing users by referencing the membership number does not work. It is recognized that the membership number refers to an existing account, but the import is still not possible. Instead, a warning appears in the import preview with the message “Found more users with name and email”.
Apart from the fact that the import is blocked, the content of this message is also incorrect. The message indicates that there is more than one user with the same name and email. This is not true. During testing, an account was used that only exists once in the account database.
Furthermore, only the user name and membership number are displayed in the preview, although more attributes, such as name and e-mail, were used.
example image for blocked import:

Reproduction:

Account > Create a new Account with a first und last name, E-Mail, membership number
Create a committee and a meeting in it
open the meeting
open Participants > Three-dot-menu >  Import. Download the example import file, delete the pre-entered users and add the new created account. Use only data for membership number. Do import.
--> The preview will now show a blocker. The data can't be imported. Additionally the wrong message appears and the preview won't show all available data for the referenced account.

What shoud happen:

[ ] Importing existing Account by membership number must work

[ ] The preview should include all existing user data

Error message: I think that the problem with the incorrect error message will be solved once the other two problems have been resolved. If the error message should occur again somewhere, I think it will have to be adjusted on a case-specific basis.

Additional Info:
File of account I created: Accounts (3).csv
File I tried to import the user into the meeting:
participants_only_mem_nr.csv
payload:
{
  "action": "participant.json_upload",
  "data": [
    {
      "data": [
        {
          "member_number": "blaueshaus"
        }
      ],
      "meeting_id": 150
    }
  ]
}

Preview:

message: "Actions handled successfully"
results: [[{id: 142, headers: [{property: "title", type: "string", is_object: true},…],…}]]
status_code: 200
success: true

Response:
{
    "status_code": 200,
    "success": true,
    "message": "Actions handled successfully",
    "results": [
        [
            {
                "id": 142,
                "headers": [
                    {
                        "property": "title",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "first_name",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "last_name",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "is_active",
                        "type": "boolean",
                        "is_object": true
                    },
                    {
                        "property": "is_physical_person",
                        "type": "boolean",
                        "is_object": true
                    },
                    {
                        "property": "default_password",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "email",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "username",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "gender",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "pronoun",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "saml_id",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "member_number",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "structure_level",
                        "type": "string",
                        "is_object": true,
                        "is_list": true
                    },
                    {
                        "property": "number",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "vote_weight",
                        "type": "decimal",
                        "is_object": true
                    },
                    {
                        "property": "comment",
                        "type": "string",
                        "is_object": true
                    },
                    {
                        "property": "is_present",
                        "type": "boolean",
                        "is_object": true
                    },
                    {
                        "property": "groups",
                        "type": "string",
                        "is_object": true,
                        "is_list": true
                    }
                ],
                "rows": [
                    {
                        "state": "error",
                        "messages": [
                            "Found more users with name and email"
                        ],
                        "data": {
                            "member_number": {
                                "id": 7163,
                                "value": "blaueshaus",
                                "info": "done"
                            },
                            "id": 7163,
                            "username": {
                                "id": 7163,
                                "value": "BaluBaer",
                                "info": "new"
                            },
                            "groups": [
                                {
                                    "value": "Standard",
                                    "info": "generated",
                                    "id": 922
                                }
                            ]
                        }
                    }
                ],
                "statistics": [
                    {
                        "name": "total",
                        "value": 1
                    },
                    {
                        "name": "created",
                        "value": 0
                    },
                    {
                        "name": "updated",
                        "value": 0
                    },
                    {
                        "name": "error",
                        "value": 1
                    },
                    {
                        "name": "warning",
                        "value": 0
                    },
                    {
                        "name": "structure levels created",
                        "value": 0
                    },
                    {
                        "name": "groups created",
                        "value": 0
                    }
                ],
                "state": "error"
            }
        ]
    ]
}


Code vs Site comparison

I am trying to download pmids for certain keywords but the number of entries for a keyword is different when I search using the code and when I search PubMed Manually

I am trying to download pmids for certain keywords but the number of entries for a keyword is different when I search using the code and when I search PubMed Manually

Can you provide me the keywords you're using so i know exactly what you're looking at?

I also have seen inconsistencies in this. Here is an example, manually this gives 3 publications.
Searching for this returns nothing with the code:
keyword = "(alzheimer) AND (bisoprolol)"
fetch = PubMedFetcher()
pmids = fetch.pmids_for_query(keyword, retmax=50)

@einonm I can only guess that the NCBI API index may lag against what the front-end is accessing.  We'd have to talk to someone at NCBI to see what that's about.  My guess is they have a network of database nodes possibly with a hierarchical pattern that gets pushed to the edges over time.  The website may be able to access the top node while the API branches may get pushed the re-indexing a little later.
Or it could have just been a glitch in that moment.
Closing this for now since I don't think it's an issue with metapub itself.


Permanent URL and Citation button on the Entry Information page

[x] To create a permalink in the table after the transcriber information and add a copy button.
[ ] To create a citation button for the Entry information page at the top of the page.

Review other sites, develop approach that mirrors them and implement.

A brief glance at FreeCen and FreeReg shows that a row of buttons is used both for navigation from the Entry Information page and to carry out various operations/reports from it.
FreeCen:

FreeReg:

This is significantly different from the BMD2 Entry Information page:

Suggestion:

implement the existing navigation controls using the same button style as Cen and Reg
add buttons to support record export in various formats + printable format
add Generate Citation button (include option to generate just this record's URL, for use as a browser bookmark, etc.)

just for notice #276

Initial effort, with the URL being presented as a link:

Same approach, but with the addition of a Copy button (code shamelessly borrowed by FreeReg citation code!):

With help from @Vino-S I have made a first attempt at the citation generator for BMD2. Here's a screenshot:

As it stands, you can select more than one citation format and see them both/all on screen. I think this might actually be useful, but if not we can go back to the one-at-a time strategy that the other projects use.

OK - all supported citation formats now implemented and on test3:

Bug fixed, ready to test on Test 3.
Is it possible to have the link on the left of the cell and the copy button on the right of the cell, to separate them and improve visual alignment?

I hesitate to say this but "Copy URL" is set up as a link — it is misleading to make it look like a button. We can either make it actually a button or change its appearance so that it looks like a link.
I dare say some of the other "buttons" on the page are also links. It matters because, for someone using only a keyboard, a link is not activated in the same way as a button. This is not only a BMD issue — bith REG and CEN have links that look like buttons.

In what sense is it 'set up as' a link? The relevant source is:
`
<button class = "btn  btn--small ", onclick = copy("wikitree_ref_citation_container") style = "cursor: pointer">Copy
<button class = "btn  btn--small ", onclick = this.parentElement.parentElement.style.display='none' style = "cursor: pointer">Close
`
It's a button, with an associated onclick event handler calling a javascript function.

The "Copy URL" button has an href which makes it a link; real buttons do not have an href.
Also, I could not get it to copy the URL. I tried the Enter key, the Space key and clicking. A keyboard-only user expects a button to be activated by Enter or Space and a link to be activated by Enter.

I'm sorry, @AlOneill - I was thinking of (and quoting the source code of) the "Copy" buttons for citations. (That's what I implemented most recently.) I'll have a look at changing "Copy URL" so that it uses a similar strategy to "Copy".

The Copy URL button now acts like a button (!), and is separated more from the URL itself. Results are on test3: please check.

@richardofsussex
Thank you. That is much better. Copy worked and the button was activated as expected.
Could the "Copied" message remain on-screen for longer, please? I don't know if any particular time is recommended by accessibility experts, just that I had become distracted and almost almost missed it.
I tried activating the "Generate Citation" button (which is also a link as it has an href, albeit an invalid one) using Space and was sent first to the bottom of the page: Tab then sent me to the first of the citation buttons.
I note that "Back to results" and "Suggest a correction …" also each have an href and that Space does not work as expected for a button.
After selecting a citation format, I think I expect the focus to move such that the next press of Tab takes me to "Copy citation". It currently takes me to the next citation format. What do you think?

I should be able to fix "Generate citation" so that it acts as a proper button.
However, once you can see the buttons, I would expect Tab to cycle through them all:

As you can see, they all appear in document order before the code for individual citations, so I would expect Tab to operate in the way you describe. The only way I can see to achieve the result you are after would be to hide all the citation buttons (or all but the selected one) when one was pressed. I think this would be visually disturbing for users.

@richardofsussex
Fair enough. It remains clear which format has been selected which is important. Thank goodness there are not 20 formats to tab through before getting to the Copy button!

I've now changed Generate citation from a meaningless link to a button - see test3. Good catch - it behaves much better now while tabbing. (Also increased the timeout on the Copy button so it says "Copied" for long enough to notice. This is the same as the earlier change to Copy URL.)

Deployed to Beta, @richardofsussex can it be closed?

Yes it can.


[FEATURE] Need more info on how to change the taiga ui language

🚀 Feature request
I would like more clear documentation on how to change taiga ui language on the fly. The current readme file at https://github.com/TinkoffCreditSystems/taiga-ui/tree/main/projects/i18n just mention 1 line:
You can also switch languages on the fly. Use useFactory or useClass with a service to make a stream of dictionaries.
Can it be supplemented with example code please.

https://github.com/TinkoffCreditSystems/taiga-ui/pull/846

It was recently resolved and added to the docs:
https://taiga-ui.dev/i18n/Dynamic_loader


feat(chart): add servicemonitor and dashboard

This PR allows to collect the metrics via ServiceMonitor instead of scrape annotations. Also the example dashboard can be deployed via helm-chart. Also podLabels can be specified. Last but not least this PR allows adding RoleBinding or ClusterRoleBinding to the ServiceAccount which could be used with e.g. Vault secret injection to add the system:auth-delegator role to the ServiceAccount.

Thank you for contributing


Update MD file on new branch 2017-20-30_11-11-28

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

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.


OMERO 5.4.9 release

Production server changes associated with the OMERO 5.4.9 release

deploy the 2018.10.09 tag of the website containing the announcement and downloads page
bump the version of the production OMERO server in Dundee to 5.4.9

As discussed with @kennethgillen, I force pushed away the two commits pertaining to the OMERO servers. Kenny will open a separate PR related to the upgrade of the prod servers including a few refactoring of the SSL tasks.


[test]update eviction test

What this PR does / why we need it:
[test]update eviction test
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Codecov Report

Merging #479 (6a7f2da) into main (9a007b7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #479   +/-   ##
=======================================
  Coverage   25.48%   25.48%
=======================================
  Files          19       19
  Lines         973      973
=======================================
  Hits          248      248
  Misses        695      695
  Partials       30       30

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


covn3d mapping + operator implem

Proposed changes
Please include a description of the problem or feature this PR is addressing. If there is a corresponding issue, include the issue #.
Work in progress implementation of Conv3d
Checklist

[x] python API operator implementation
[x] python API module mapping implementation
[x] C++ api operator implementation
[x] C++ operator interface
[ ] Naive Conv3D METAL kernel
[ ] necessary tests

Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Might want to check in with @jagrit06 as he is working on an update to conv #651

What's the status of this PR? Should we move it forward or close it?

What's the status of this PR? Should we move it forward or close it?

I will continue on it in a couple of days. I was waiting initially for @jagrit06 PR to be merged. Now that it's merged I will work based on those changes to add the frontend implementations.

@AndreSlavescu are you coming back to this or should we close?

Closing as inactive.


Arduino Integer Offside

In #182 we introduced two integer parameters to the voltage sensor. But these integers are likely to offside.

A fix has been released. Waiting for the cache to refresh.


After upgrade to podman 1.9.0, ~/.local/share/containers being symlink causes problem

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
A bit of history:
For my rootless podman usage, I wanted to have the space consumption under control. After I saw the content of ~/.local/share/containers/storage grow, I created separate partition, moved ~/.local/share/containers to /mnt/containers-storage/user and created symlink ~/.local/share/containers pointing to this new directory.
Then at one point I ditched then whole ~/.local/share/containers to start afresh ... and the ~/.local/share/containers was created again, consuming space under my home. So I moved it again to /mnt/containers-storage/user, created the symlink again, and edited ~/.config/containers/storage.conf value graphroot to point to /mnt/containers-storage/user, so that should I purge my rootless containre storage in the future, it does not creep back under my home.
That was with podman-1.8.2-2.fc31.x86_64 and things were working fine.
Today I upgrade to podman-1.9.0-1.fc31.x86_64 and when I tried to run podman run ..., I got error message
Error: could not get runtime: database libpod root directory (staticdir) "/home/user/.local/share/containers/storage/libpod" does not match our libpod root directory (staticdir) "/mnt/containers-storage/user/storage/libpod": database configuration mismatch

Steps to reproduce the issue:

Move ~/.local/share/containers to different directory and edit ~/.config/containers/storage.conf to point to this directory.

Describe the results you received:
Error
Error: could not get runtime: database libpod root directory (staticdir) "/home/user/.local/share/containers/storage/libpod" does not match our libpod root directory (staticdir) "/mnt/containers-storage/user/storage/libpod": database configuration mismatch

Describe the results you expected:
If I have symlink in place so both paths resolve to the same location, podman should not complain at all, like it did in 1.8.2.
If podman needs to have these extra checks, the error message should provide some help about where that "database libpod root directory" can be configured. Or it should provide information how changing locations should be handled.
Additional information you deem important (e.g. issue happens only occasionally):
I am able to workaround the issue by editing my ~/.config/containers/storage.conf and setting graphroot to point back to /home/user/.local/share/containers/storage. But I'd rather do the proper rename and if it needs to be done in multiple places (~/.config/containers/storage.conf and somewhere else), podman should be a bit more helpful with that.
Googling aroung I found similar reports about run and tmp but no real information about updating the "database libpod root directory".
Output of podman version:
Version:            1.9.0
RemoteAPI Version:  1
Go Version:         go1.13.9
OS/Arch:            linux/amd64

Output of podman info --debug:
Error: could not get runtime: database libpod root directory (staticdir) "/home/user/.local/share/containers/storage/libpod" does not match our libpod root directory (staticdir) "/mnt/containers-storage/user/storage/libpod": database configuration mismatch

Note how this check effectively prevents proper debugging of the problem.
Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.9.0-1.fc31.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical machine.

@rhatdan I think we might be missing a symlink resolve somewhere after the containers.conf work, maybe?

@adelton Is this fixed now?  I think it is, reopen if I am mistaken.


Transition from instance.dfs.{uri,dir} to instance.volumes on upgrade

In 1.7.0, instance.dfs.uri and instance.dfs.dir were deprecated in favor of instance.volumes. These worked by resolving relative paths in the metadata, with a number of fallback mechanisms (including falling back to HDFS config files to resolve the defaultFS for the URI).
These should be replaced with an explicit resolution, similar to instance.volumes.replacements, but using a property specific to this upgrade process (instance.upgrade.volume.relative.prefix?)
All occurrences of relative paths in the metadata should be converted to absolute paths during upgrade.
In summary:

Explicit property for resolving relative paths
Mandatory upgrade of metadata to remove all relative paths
Remove old properties (this requires MiniAccumuloCluster use instance.volumes)

This issue supersedes: ACCUMULO-3535, ACCUMULO-3023, ACCUMULO-3537, and ACCUMULO-3573 from JIRA.

In addition to above, we should only require the new property if relative paths exist and check the existence of all files before renaming relative paths.  We need to make sure the value in the new property is absolutely correct before renaming metadata paths.
Also need to check delete markers for relative paths.

Unassigned myself as I am currently not working on this.  The replacement of relative paths on upgrade and removal of relative path code is done.  Removal of the old properties still needs to be done.


Should I be worried about warnings on npm install?

sudo npm install
npm WARN lifecycle<EMAIL_ADDRESS>cannot run in wd %s %s (wd=%s<EMAIL_ADDRESS>npm run npmcheckversion /home/sillas/Downloads/tinder/tinder
npm WARN lifecycle<EMAIL_ADDRESS>cannot run in wd %s %s (wd=%s<EMAIL_ADDRESS>npm run build:dll /home/sillas/Downloads/tinder/tinder
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^2.0.0 but none was installed.

Is the app going to work with these warnings or should I install those things?

I just ran the app and seems to be working fine... But I didn't used it too much... I will test later and can reply here :)
(I've got the same warnings)

those warnings are fine. the app should still work.


editable citation keys

The new citation keys are nice - much better than using "bibtex: blah" in the extra field. I notice all of my old extra field citation keys got imported too, which is nice. But now it seems that I can't edit the citation key. Is there a good reason for that? It'd be nice to have keys that were a bit more meaningful (the first 4 words aren't always that descriptive). Is there a way I can edit the citation keys of items?
Originally posted this on the zotero forums: https://forums.zotero.org/discussion/51071/editable-citation-keys/

I can't currently open pages on forums.zotero.org, but with "edit the citation key", do you mean the citation key displayed in the reference details pane? If so, then unfortunately, no. BBT knows of two kinds of keys, "soft" keys which are generated from details of the reference and which will change as those details do, and "pinned" keys which don't. The pinned keys are stored in the "extra" field, Zotero doesn't offer any other way. So I am afraid the answer to your questions is "no"; the generated keys aren't editable, and the pinned keys are only editable through the "extra" field. A future version of Zotero is expected to have fields available for extensions to use, but that is still a way off.

You can change the pattern according to which they're generated btw - it's
in the preferences.

Ah, ok, I guess I can just continue using the extra field for now. Thanks!


Remove two warnings

Should be pretty obvious. As you are using -Wall it seems like a good idea to get rid of the warnings where possible.

Merged to develop; thanks!


Add an example of how to attach labels to nodes and use nodeSelectors so...

... that pods can be scheduled on specific nodes.

Issue #2406

Thanks for the review! I've fixed up everything you mentioned.

LGTM, personally, I would soften the language around "discouraged" to be something like "generally this is unnecessary, as the scheduler will take care of it for you, but in certain circumstances like ..."

Sounds good, done.

LGTM  one minor comment


asset based atoms generation

which scripts to generate can now be added to assets.
I've just added it ontop of the current system, so there are some redundancies.
also it has to be tested if this would regenerate assets in packages.
it also contains an custom inspector for Folders, if a folder contains "Atom" it shows a button to generate a new generation file.
not sure if we should keep this, but it could be a nice user workflow improvement. e.g. for typical used atoms (IntVariable / FloatVariable)

It could be nice to be able to specify an Editor folder
We use asmdef and we keep all editor related code in a single folder, currently I manually drag the generated editor files in the folder

Finally got around checking this out.
Just as a test, I wanted to create a generator for https://docs.unity3d.com/ScriptReference/Rect.html but the type finder can't find it, these were the only three types found from UnityEngine:

If I type it in manually, the generator refuses to generate since it can't find the type:

This is on a fresh project, I have restarted Unity meanwhile but the problem keeps appearing. I added the Core and BaseAtoms packages only.
Also I can't seem to find this button that should appear if a directory name has "Atom" in it, where is it located?

@miikalo thanks for checking it out!
I fixed the Rect issue, the button should appear when selecting a folder in the inspector.

Functionality-wise it's great, however I'd like all of the Atom types to generate be checked by default when creating a generator, I'd more often want to have all of them than none of them and it saves a lot of clicks. Is it also possible to have the editor jump to the newly created generator after pushing the "Add Generator" button?
I have some nits about whitespace and commented out code in the files, after those are addressed it looks good.


"C.A. Rosetti" is listed in the drop-down menu

Description:
"C.A. Rosetti" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "C.A. Rosetti" in the search bar.

Expected results
The location is listed in the drop-down menu as "C.A. Rosetti, Buzău".


status of UTF-8 support

Hi, just wanted to ask, if there is already UTF-8 supprt in VBA-Web.
After Roadmap in here UTF-8 support is planned.
I'm getting weird chars in an response string (utf-8 encoded), just wanted to make sure, that there is no fault with the usage on my site
best regards

Hi @SLiX69 UTF-8 is not currently supported by VBA-Web, so any char issues you're experiencing are most likely due to that (and not your usage).
UTF-8 support is planned and is definitely important, but there have been many issues with implementing it for Mac. I've recently started planning for Windows-only features that could be ported to Mac at a later date (see Feature Flag RFC) that should allow for a lot more progress in this area instead of waiting until a Mac-supported solution is discovered.
Closing this for more specific issues: https://github.com/VBA-tools/VBA-Web/labels/utf-8-issues


add a replace_nan option to the to_napari convenience function

Currently, any dummy objects inserted by the tracker do not have properties. This means that tracks containing dummies will have NaN for certain properties, which causes them to render incorrectly in napari.
We should provide a flag to replace these NaN values to aid in visualization. We could perhaps interpolate the values for the missing ones.
https://github.com/quantumjot/BayesianTracker/blob/7d80399db64af4723c1da2381a22d9910e1ba7c8/btrack/utils.py#L294
would become:
 def tracks_to_napari(tracks: list, ndim: int = 3, replace_nan: bool = True):

Done - now replaces NaN with an interpolated property value


Issues with ETH* Markets

"Love it, really useful to have. Just wondering why sometimes it does not work on ETH markets? Pretty sure it worked and now it stopped."

Should be working as of 1.9.0.


[  +0.018718] usb usb5-port1: cannot disable (err = -32)
[Mar27 20:44] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.019053] usb usb5-port1: cannot disable (err = -32)
[ +40.660799] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.018685] usb usb5-port1: cannot disable (err = -32)
[Mar27 20:46] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.019019] usb usb5-port1: cannot disable (err = -32)
[Mar27 20:47] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.006564] usb usb5-port1: cannot disable (err = -32)
[Mar27 20:48] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.018923] usb usb5-port1: cannot disable (err = -32)
[Mar27 20:49] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.019101] usb usb5-port1: cannot disable (err = -32)
[ +16.277967] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.006342] usb usb5-port1: cannot disable (err = -32)
[Mar27 21:30] xhci-hcd xhci-hcd.8.auto: port 0 resume PLC timeout
[  +5.742867] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +5.232322] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.218864] usb 5-1: device not accepting address 2, error -62
[Mar27 21:31] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +5.232182] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.207003] usb 5-1: device not accepting address 3, error -62
[  +5.409325] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.207016] usb 5-1: Device not responding to setup address.
[  +0.206919] usb 5-1: device not accepting address 4, error -71
[  +0.006232] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.006210] usb usb5-port1: cannot disable (err = -32)
[  +5.365865] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +5.216288] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.207000] usb 5-1: device not accepting address 5, error -62
[  +0.027109] usb usb5-port1: unable to enumerate USB device
[ +18.534735] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +5.216250] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.206943] usb 5-1: device not accepting address 6, error -62
[  +5.395727] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +5.229836] xhci-hcd xhci-hcd.8.auto: Timeout while waiting for setup device command
[  +0.218966] usb 5-1: device not accepting address 7, error -62
[  +0.018411] xhci-hcd xhci-hcd.8.auto: Cannot set link state.
[  +0.018858] usb usb5-port1: cannot disable (err = -32)
[Mar27 21:32] usb 5-1: new SuperSpeed USB device number 8 using xhci-hcd
[  +0.029930] usb 5-1: New USB device found, idVendor=174c, idProduct=55aa
[  +0.018513] usb 5-1: New USB device strings: Mfr=2, Product=3, SerialNumber=1
[  +0.013358] usb 5-1: Product: External HDD
[  +0.006015] usb 5-1: Manufacturer:<PHONE_NUMBER>0123456789012345678901234567890123
[  +0.006308] usb 5-1: SerialNumber:<PHONE_NUMBER>13
[  +0.059266] usb-storage 5-1:1.0: USB Mass Storage device detected
[  +0.007795] usb-storage 5-1:1.0: Quirks match for vid 174c pid 55aa: 400000
[  +0.006389] scsi host0: usb-storage 5-1:1.0
[  +0.006746] usbcore: registered new interface driver usb-storage
[  +0.010794] usbcore: registered new interface driver uas
[  +6.364552] scsi 0:0:0:0: Direct-Access     WDC WD32 01ABYS-01B9A0    13.0 PQ: 0 ANSI: 5
[  +0.007974] sd 0:0:0:0: [sda] 625142448 512-byte logical blocks: (320 GB/298 GiB)
[  +0.007000] sd 0:0:0:0: [sda] Write Protect is off
[  +0.006170] sd 0:0:0:0: [sda] Mode Sense: 23 00 00 00
[  +0.000435] sd 0:0:0:0: [sda] No Caching mode page found
[  +0.006158] sd 0:0:0:0: [sda] Assuming drive cache: write through
[  +0.080574]  sda: sda1
[  +0.007636] sd 0:0:0:0: [sda] Attached SCSI disk
[  +1.791350] EXT4-fs (sda1): mounted filesystem with ordered data mode. Opts: user_xattr,usrjquota=aquota.user,grpjquota=aquota.group,jqfmt=vfsv0,acl
[Mar27 21:33] ip6_tables: (C) 2000-2006 Netfilter Core Team

I bought another external HD (WD Elements Desktop) but there seems the be the same problem.
So it isnt a problem with this specific external HD.
Any help?

Any ideas how to fix this...?

It would appear that I'm affected by this same issue. Using USB memory sticks. I'm running the image released as the april 4th build:
Linux firefly 4.4.114 #62 SMP Wed Apr 4 11:10:34 HKT 2018 aarch64 aarch64 aarch64 GNU/Linux

same issue for me. impossible to add external drive by usb 2.0 usb 3.0 hard disk penflash or other


@mzikherman => explicitly request tall images for artwork columns on artwork page

We noticed that on the artworks view in mobile web, we were requesting images of all sizes to display in the columns below the main artwork view:

We only need tall images (max) for these small views, so this PR updates the metaphysics query that requests the artwork image to explicitly fetch the tall version. (Otherwise it just seems to grab the latest image version in the list, which is anything from large to tall and mostly, normalized 😱 .
With this update, we're fetching much less data. (the largest image used to be 4.7 MB for an image that was way down in the related section, vs. 87.4 KB after the change).

this one is so intense

Nice, normalized is totally not resized from the original!


Update repository after transfer

See https://github.com/senzing-garage/garage-roadmap/issues/3

Pull requests:

https://github.com/senzing-garage/go-cmdhelping/pull/46


Add new matchStyle property

A lot of React components use inline styles instead of classes. Some of them probably do this because of idealogical reasons but others do it to simplify things for other components.
For example, a Browserify user recently filed a bug against my devtools monitor because Browserify was not able to import the stylesheet (see bvaughn/redux-devtools-filterable-log-monitor/issues/13). To resolve this I decided to inline styles. Unfortunately I could no longer customize the appearance of highlighted tags.
This PR adds that ability along with a unit test. :)
PS Your package.json should list react as a peer dependency because npm i && npm test fails without it. I didn't include that change in this PR though because I wanted to keep it focused. :)

Note that npm test passes locally. Not sure what's going on with your Travis instance.

Looks good. Thanks @bvaughn!


refactor: build step cleanup

pulled out of #2923

changed configPath to rootDir everywhere, because we almost always want the project root dir and want to avoid using path.dirname in all the downstream tooling
moved store/world build steps into a single build script so they can reuse vars etc. and map more closely to the mud build command
renamed all-codegen to all-build because that's actually what it's doing

this doesn't change anything about the CLI commands, just the arg name/intent of the internal functions, and shouldn't impact users unless someone imported tablegen from @latticexyz/store/codegen or worldgen from @latticexyz/world/node (which feels less intentional)


Batch operations for witness generation

We could defer computing inverses in the witness (for computing EC ops) and batch compute them at the end.

An explanation from @kevaundray that I found really useful:
So if you imagine that you need to compute the following circuit:
x = 1/y
t + x = 0
I could generate the witness values as:
y = 2
x= 1/2
t = -1/2
So at the end we have three field elements [2, 1/2, -1/2]
and I had to do 2 inversions to compute these witness values.
The alternative way is to have the following values at the end [2, (1,2), (-1,2)]
So no inversions are done yet, I can then do a batch inversion of the tuples/rationals. In that example ‘t’ relied on ‘x’ but we were still able to represent it without using inversions.
After doing a batch inversion and multiple of [2, (1,2), (-1,2)] i should end up with [2, 1/2, -1/2] which is what I would have gotten if I just did the inversions directly. I guess the trick is to allow witness values to be temporarily stored as fractions, which then allows you to defer inversions.
https://github.com/zcash/halo2/blob/d111807798d4719df36dc2b05b0104b69efdceb3/halo2_proofs/src/plonk/prover.rs#L286

to add more colors, this would be for the circuit-writer


Deprecate annotations that conflict with reserved words in PHP7

See:

https://github.com/doctrine/phpcr-odm/issues/641
https://github.com/doctrine/phpcr-odm/pull/643

This is not terribly urgent, since the existing mongo extension will not be updated for PHP7 compatibility (mongodb will); however, we should probably get this change in before 1.0.0 stable.

@jmikola What is your suggestion about how to solve it?

BTW I have a question: What is the difference between Bool and Boolean annotations?

@Soullivaneuh they're the same thing, we had inconsistency with Boolean being full name and Int being short so we introduced full and short versions of both

@Soullivaneuh just use the @Field(type="bool") annotation instead of @Bool

@Soullivaneuh they're the same thing, we had inconsistency with Boolean being full name and Int being short so we introduced full and short versions of both

@malarzm In this case, why not just deprecate Bool and use Boolean instead?

@Soullivaneuh just use the @Field(type="bool") annotation instead of @Bool

@stanlemon I just get this error directly on app/console:
$ php app/console
PHP Fatal error:  Cannot use 'Bool' as class name as it is reserved in vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Mapping/Annotations/Bool.php on line 23

Fatal error: Cannot use 'Bool' as class name as it is reserved in vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/Mapping/Annotations/Bool.php on line 23

I don't use Bool annotation on my documents.


download_all removes the products contained by the passed SearchResult

The items of a SearchResult (it could also just be a list of EOProduct) passed to download_all can get cleared by the method. I observed this behaviour with downloading products from peps. If it is the intended behaviour, it should be well documented because it is quite surprising.
If not, the problem comes certainly from this line (PEPS uses a HTTPPlugin which itself uses the base implementation of search_all):
https://github.com/CS-SI/eodag/blob/fec965f791a74dbf658e0833e1507b7ac950d09d/eodag/plugins/download/base.py#L308
Run this snippet to reproduce the bug:
from eodag import EODataAccessGateway
from eodag.api.search_result import SearchResult

dag = EODataAccessGateway('path/to/peps_config.yml')  # Adapt the path
dag.set_preferred_provider("peps")

search_criteria = dict(
    productType='S2_MSI_L1C',
    start='2021-03-01',
    end='2021-03-31',
    geom={"lonmin": 1, "latmin": 42, "lonmax": 5, "latmax": 46},
)
search_results = dag.search_all(**search_criteria)
print(f'Got {len(search_results)} products.')

# Download the smallest files
sorted_prods_per_size = SearchResult(
    sorted(search_results, key=lambda p: p.properties["resourceSize"])
)
prods_to_download = sorted_prods_per_size[:2]

# Just to make sure not to order products
assert all([p.properties["storageStatus"] == "ONLINE" for p in prods_to_download])

paths = dag.download_all(prods_to_download)

if not prods_to_download:
    print("Oh oh I'm empty now")

If this is a bug, we should check the behaviour of the other download plugins.

For the record, I discovered this behaviour while I was trying to inspect the value of the attribute location of the downloaded products. I have the feeling but need to have a closer look at it that it is not being updated to reflect the local path after the download has succeeded.

This is not the expected behavior. We may do a copy of the products list to preserve it duing download_all() but also update products location when each individual download is done

This was nicely spotted by the ORFEO toolbox:
https://gitlab.orfeo-toolbox.org/s1-tiling/s1tiling/-/blob/develop/s1tiling/libs/S1FileManager.py#L380


no web feeds found, no kudos given....

Hi,
I just installed your script and could finally run it...
But It gives the following response:

python3 give_kudos.py
---Logged in!!---
id xxxxx
web feeds found: 0

Kudos given: 0

Even the feed has open activities to give kudos....
Am I doing something wrong?

Hi @zuselli84,
my latest run 21 minutes ago gave back the following:
Run python give_kudos.py
---Logged in!!---
id xxxxxxxx
web feeds found: 100
==c====c====c==c====c===========================================================
Kudos given: 75

My experience was that if you could log in, but no feeds was found, the browser could be the culprit, but it could also be something else
Would you be able to share some more details on your setup?

hi @isaac-chung Thank you very much for your reply.
I didn't change your script, so it should youse firefox.
Its version:
Mozilla Firefox 124.0.1
my setup is a ubuntu server, what kind of information is required for debugging? And how can i check if the browser is the culprit?

Mine runs via Github Actions on Ubuntu 20.04.6.
Previously, I have done some debugging by getting screenshots to check if the feed page was loaded in time. This is done by calling self.page.screenshot("/out/path.png").
Separately, I wonder if you're able to set up this run in Github Actions for your fork?

Thanks for the debugging hint.
my Ubuntu is:
Description:    Ubuntu 22.04.4 LTS
Do you think that could be the reason?
Unfortunately I don't know how to run it in Github Actions....

i could manage to run it with Github Actions, but I'm not sure, if I set the environment variables correct:
python give_kudos.py
shell: /usr/bin/bash -e {0}
env:
STRAVA_EMAIL:
STRAVA_PASSWORD:
pythonLocation: /opt/hostedtoolcache/Python/3.9.10/x64
PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.9.10/x64/lib/pkgconfig
Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.9.10/x64
Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.9.10/x64
Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.9.10/x64
LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.9.10/x64/lib
---Logged in!!---
can't find own profile ID
web feeds found: 0
Kudos given: 0
Now it responds with "can't find own profile ID"

The difference in versions can be a potential cause. I'm unfamiliar with that unfortunately.
Looking at your print out, the env vars should have *** printed beside them. Did you add them to "repository secrets"? (Settings -> Security -> Secrets and Variables -> Actions)

That was it, I habe entered the secrets in the wrong place.
Now this worked....
Thanks for your support.
I'm wondering, why this is not working on my own Ubuntu Server. That would be very cool, if I could run this on my server....
Do you think this is tricky to debug?

@isaac-chung I changed versions in Github workflow, so that it's the same as my server:
ubuntu 22.04
python: 3.10.12
in Github action, it works fine...
Are there other things to consider e.g. Firefox? Or does playwright includes correct firefox version?

Great! Glad you got that sorted out. As for Firefox version, the playwright install command should install this version:
Downloading Firefox 108.0.2 (playwright build v1372) from https://playwright.azureedge.net/builds/firefox/1372/firefox-ubuntu-20.04.zip

Do I understand it correct, that the Firefox on the host system does not influence your skript.
I use same Playwright version 1.30.0, so it should use correct browser..(?)

It works, there was an issue with playwright, I uninstalled it and re-installed it according GH workflow.. Now it works.
Thank you for your help


Np 45083 upsert nvi candidate

Splitting Jira task (https://unit.atlassian.net/browse/NP-45083) into several PRs to keep them small. This is nr 2
UpsertNviCandidateHandler consumes messages on sqs queue "NewCandidateQueue"

If the candidate does not already exist in DB, create new candidate with pending approval statuses and save.
Faking NviCandidateRepository util DB Tech is chosen and implemented

@truhacevkir NviCandidateRepository will contain the DB transactions, and NviService will contain the business logic. That's why I'm faking NviCandidateRepository atm


ghcjs/remote verification: line numbers are off

When building with ghcjs (verify) reports line numbers relative to the module not the input file. When building with ghc, line numbers are relative to the input file as expected.

The solution I have in mind is something like this:

Extract line number information for each module at the client side.
Transfer that as well
Initialize the line counter with that offset before evaluation of modules.

Problem, for 1 and 3: I have no idea yet, how to actually do that.
For 2, it is not totally clear where to put that additional information.
Comment by bts:  one possible quick/hacky approach for 3 is just to add offset-many newlines before the module definition on the server side


Fix DocOpt deserialization type bounds

This is wrt https://github.com/docopt/docopt.rs/pull/222
DocOpt does not support deserializing borrowed types.
This change was reverted in
https://github.com/docopt/docopt.rs/commit/7292a374e69afb192bb7aaa00f9d9f4afebc200d
because it broke crates like Cargo etc.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 7e92513 has been approved by alexcrichton

:hourglass: Testing commit 7e92513debcf0c62999206e3675493470c4b4ead with merge d0ca5bcd6b1cfacf5f92a2fd7b99f0807b4b3407...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing d0ca5bcd6b1cfacf5f92a2fd7b99f0807b4b3407 to master...


Fix incorrect event ID

Under section "Something went wrong. Error Code 80070774", Section 2, reference is made to Event ID 30312. It is actually 30132

Docs Build status updates of commit d5db7cb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

support/mem/intune/troubleshoot-windows-enrollment-errors.md
:white_check_mark:Succeeded

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


sameo/issues/67

This is a PR to close #67.
We now parse the client certificate, build a role bitmask out of the OIDs found there and force the client into using this new role.
That will allow us to no longer need to explicitely set a role from e.g. a SSNTP client command line.

Coverage decreased (-0.2%) to 53.701% when pulling abef1b6dadcb52284252f43ef7f5c2d389aee799 on sameo/issues/67 into 7a1a8252f02bf989dda3ca7011a26a7cd78607be on master.

Coverage decreased (-0.2%) to 53.716% when pulling df8a7e635412ed8f8ca3b6399ecd7263adff0f6c on sameo/issues/67 into 7a1a8252f02bf989dda3ca7011a26a7cd78607be on master.

Coverage decreased (-0.2%) to 53.724% when pulling df8a7e635412ed8f8ca3b6399ecd7263adff0f6c on sameo/issues/67 into 7a1a8252f02bf989dda3ca7011a26a7cd78607be on master.

Coverage decreased (-0.3%) to 54.21% when pulling 44c800f64f48deb0fb67d2d9d4e969266394aa44 on sameo/issues/67 into 2293e2e8d62307896108db50f76243756a3d0e97 on master.

Coverage decreased (-0.3%) to 54.247% when pulling 8502826b3441e151af6f1bc0f8ef91580c187074 on sameo/issues/67 into ded247ebf556d2f85148a4a69430626618d7c1a6 on master.


Fix coverity-errors assign/copy/uninit

As pointed out in http://dronecode01.sh.intel.com/gazebo/coverity/0978a10/
Signed-off-by: Guilherme Campos Camargo<EMAIL_ADDRESS>

Please @dakerfp and @rchiossi take a look to see if the changes are enough to avoid those coverity errors. I was not able to test it.

Merged


Add NuFi wallet service and metadata

Follow up on https://github.com/onflow/fcl-discovery/pull/108 as the NuFi extension v2.3.3 with user signing support has just been released
Mainnet wallet is available on the Chrome Web Store.
Testnet build is available here.
cc @chasefleming

Awesome, thanks! We'll test out the wallet. Thanks for adding user signing!

Tested out the wallet and it's great! Ready to merge, but a merge conflict that need fixing. Mind fixing that?
One thing I noticed is that entering your password is required on every authorization. Assuming that's desired behavior, right?

Testnet build working smoothly for transaction and user-signing (with verification)!
Also seeing password prompt on each.
A great new addition to Wallet Discovery!

Tested out the wallet and it's great! Ready to merge, but a merge conflict that need fixing. Mind fixing that?

Done :+1:

One thing I noticed is that entering your password is required on every authorization. Assuming that's desired behavior, right?

The "connection" confirmation prompt on each authorization (with the connect/reject option) is expected, entering the password should be needed only for the first one for the given browser session, unless the "remember login" option in the password prompt was unchecked or if the user chooses to switch to a different profile - are you sure it was asking for the password on each authorization (unexpected, unless "remember login" was unchecked) and it wasn't just the connection prompt (expected)?


add upgrade/downgrade check to k2 post commit job

we currently only run e2e tests against k2 created clusters.  we should add an upgrade/downgrade check as part of the node management project.

no longer blocked by CI but node-management work isn't done yet to be tested.


changes to support configurations per hostname

Changes to support chaise configuration per hostname.

Apart from the comment, I feel like we need a proper documentation for this, which you can include in this PR. Most probably you can just copy and paste the issue description in that document.

@RFSH  I updated the code with your suggestions. I included documentation for this in chaise-config.md. I also included an example in chaise-config-sample.js. The attached issue has a comment on it further explaining the nuances of this feature


[Feature]: Support specifying quant_config details in the LLM or Server entrypoints

🚀 The feature, motivation and pitch
Background:
With the recent support for deepspeedfp quantization introduced in https://github.com/vllm-project/vllm/pull/4652 and https://github.com/vllm-project/vllm/pull/4690, a new issue has emerged due to the nature of the runtime quantization implementation. This implementation allows users to load an unquantized model and enable the quantization argument to reduce the memory footprint required for loading the model. However, the main challenge lies in the fact that the deepspeedfp implementation has a parameter num_bits that supports quantizing the weights down to either 8 or 6 bits, with the default value set to 8.
Problem Statement:
Currently, if a user wants to apply quantization="deepspeedfp", vLLM will only be able to quantize the model to num_bits=8 since that is the default value. The only way to change this behavior is by providing a quant_config.json file that explicitly defines the desired value for num_bits. This limitation restricts users from easily customizing the quantization settings without modifying the configuration file.
Proposed Solution:
To address this issue, we propose adding a new argument quant_kwargs=Union[str, Dict] to the common LLM() and OpenAI server interfaces in vLLM. This argument would accept either a dictionary of keyword arguments or a string that can be converted to a dictionary. The purpose of quant_kwargs is to allow users to override the default values or loaded config values for the quantization configuration.
By introducing this new argument, users gain the flexibility to specify custom quantization settings directly through the API, without the need to modify the quant_config.json file. This enhancement improves the usability and convenience of applying quantization in vLLM, enabling users to easily experiment with different quantization settings based on their specific requirements.
Alternatives
No response
Additional context
No response

Can you tag with RFC?


Deprecate nunit package

Fixes https://github.com/Roemer/Chocolatey-Packages/issues/7.
I think this is everything required - I've followed the steps listed here, and the work one of the chocolatey core team did to a similar NUnit package here.
The final steps, that I believe will need to be done by hand are...

[Publish] a new version of the deprecated Chocolatey Package.
Unlist all versions from the package gallery, except the final deprecated version. The final deprecated version is required so that there is an update path to the new package.
By following this process, any existing users who try to update the old package will automatically get the new package, as it will be installed as a dependency.

Thanks for putting this package online in the first place - it did take us at NUnit a while to get round to building choco into our build process - but we got there in the end. Hopefully this change should give automatic, up-to-the-minute updates for everyone from now on! 😄

I've finally done that.

Awesome, thanks!


Create NEAR WalletConnect Provider

Description
The goal of this story is to build a WalletConnect provider package that’s specific to the NEAR integration similar to Ethereum and Cosmos.
I believe the point of these providers (though not terribly clear from documentation) is to give some structure via TypeScript to the generic APIs of the Signer Client such as request to avoid an integration that isn’t actually compliant with the RPC Reference defined in SQC-157: Create NEAR WalletConnect Provider
Damir Pjetlovic  August 21, 2023 at 1:06 AM
Pull request and draft PR are raised. We’re waiting for the feedback.

Updates about Universal Provider:

NEAR has been added to Universal Providers: https://github.com/WalletConnect/walletconnect-monorepo/pull/3465

Updates about WalletConnect web-examples:

We Pinged the WalletConnect team again about the example to showcase NEAR with Universal Providers in WalletConnect: https://github.com/WalletConnect/web-examples/pull/270#issuecomment-1744512536
After some recent refactoring WalletConnect broke the implementation we made a new PR that would fix it: https://github.com/WalletConnect/web-examples/pull/301

WalletConnect has published a new version that includes NEAR in the Universal Providers: https://github.com/WalletConnect/walletconnect-monorepo/releases/tag/2.10.2

As mentioned in the above comment https://github.com/near/wallet-selector/issues/924#issuecomment-1752462595 WalletConnect has published the Universal Providers package which includes NEAR provider.
We have also made an extra PR to web-examples of WalletConnect to showcase NEAR with the universal-provider, they have started to refactor their examples but then stopped/postponed it for some reason and we never got any review there: https://github.com/WalletConnect/web-examples/pull/270
I will be closing this issue as this was originally created to add NEAR to universal-providers, we will see to re-create the example once the refactoring is done on their side.


Fix getting hostname for HOST networking

For host networked container it uses localhost as a default
fixes #24

Released as 0.3.4

This is actually breaks tests on Windows then boot2docker VM is used:
test.environment.get('WEB_HOST') == 'localhost'
|    |           |               |
|    |           <IP_ADDRESS>    false
|    |                           12 differences (0% similarity)
|    |                           (<IP_ADDRESS>)
|    |                           (localhost---)

@pzbitskiy ha! thanks, I'll fix that...


FIx few typos and improve grammar

Description of the change:
Changed reciept to receipt , snippits to snippets.
Also improved grammar.
Reason for the change:
It was wrong spelling and wrong grammar.
Link to original source: https://sendgrid.com/docs/ui/sending-email/using-handlebars/

Hello @geekysrm,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @geekysrm,Thanks again for the PR!                                         You have earned 6 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


fix(region): valid qcloud hssd for system disk

这个 PR 实现什么功能/修复什么问题:
避免腾讯云使用hssd系统盘

[x] 功能、bugfix描述
[x] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.4

/area region
/cc @zexi

Codecov Report

Merging #8961 (c5c77b4) into master (4a8eafb) will decrease coverage by 1.65%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #8961       +/-   ##
==========================================
- Coverage    5.88%   4.23%    -1.66%
==========================================
  Files         825     165      -660
  Lines      128808   24866   -103942
==========================================
- Hits         7575    1052     -6523
+ Misses     120554   23718    -96836
+ Partials      679      96      -583

Flag
Coverage Δ

aFlag
4.23% <ø> (-1.66%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/baremetal/agent.go

pkg/apigateway/handler/proxy.go

pkg/compute/models/scaling_activity.go

pkg/compute/models/scheduledtask_label.go

pkg/baremetal/tasks/deploy.go

pkg/multicloud/aws/iam_group.go

pkg/keystone/models/idp_remote_ids.go

pkg/util/stringutils2/stringutils.go

pkg/compute/models/hosts.go

pkg/compute/models/dns_zones.go

... and 649 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a8eafb...c5c77b4. Read the comment docs.


Use compasstools functions to read data

This draft PR moves the sapflow-, TEROS-, and Aqua TROLL-reading code out of this repository and into a new compasstools package. This has several advantages I think:

Less code here 🎉
Dedicated test code for the file-reading functions
The data workflows, and your planned synoptic dashboard, don't have to duplicate this code—they can just use compasstools

Dashboard still runs fine on my system.
Before I go any farther, does this make sense to you? Any concerns?

@stephpenn1 Ready for review

Apologies that I am just now getting to this - looks AWESOME and runs on my local as well. Merging!

Can this branch be deleted?

Sure. This branch doesn't include using the min_timestamp functionality of compasstools, but that's probably not crucial for now.
Very glad it ran locally without any problem!


Change sdp to only have h264 when h264 flag is on

Submitting PR again from the same repo. See https://github.com/opentok/opentok-meet/pull/51

+1 tested and it's working in Chrome. It works in Firefox but not Firefox through Mantis for some reason. Investigating why that might be.


Udemy courses details scraper

Aim
To scrape details of Udemy courses.
Details
Important details about courses like Course title, description, instructor, price, rating and number of lectures will be scrapped and stored in a csv file.

@kaustubhgupta I would like to work on this, Can you please assign this to me?

Instead of CSV, can you store it a database of your choice and then create a fetcher.py file to fetch the information stored?

@kaustubhgupta sure will do that.... so 2 scripts - one to scrape data from udemy and store in database , and other one to fetch data from database and display?

Yes

@kaustubhgupta  Sure I can do it


New name for Objectclass table

As you may know, we gave new ID numbers for every new or inherited object classes in the new "objectclass" table.
To do that, i have created a table to show and map the relations between old "objectclass" table rows and new ones. (see link)
Additionally, i have created a diagram to visualize the sub-super class hierarchy in the new objectclasses. (seelink)
However, we have serious problem about "objectclass" table, which will be exposed after using old v4 tools with the new database.
In a particular moment after running upgrade scripts users can re-populate them and ignore new tables to read.
Only exception to read old data is "objectclass" table. (We used "t_mapping_objectclass" table to re-populate the data of views)
That's why i want to offer to change the name of new objectclass table
or we can add new objectclass rows by specifying sequential ID numbers starting from 1000 (or more bigger value).
Note: I don't have any suggestion for a new name.

Hm, This is not a problem in the current v5 schema. The Upgrade-Skripts should be able to solve such kinds of conflict problem by itself.

Usually schemas does not include the problems, but the data does.

The Appearance table name is also used in both v4 and v5, and will also cause the similar issue? If yes, you will also want to change the Appearance table name vor v5?

Ok, i fixed this problem with a temporary solution, but also planning to implement another solution in following weeks.


Update README.md

ansible-galaxy must be run from the root directory, otherwise ansible.cfg ist not respected.

:+1: Thanks for the contribution!


Very simple pancakes

Step 4 is written as step 3, so we have two step 3...

My bad. Just updated it.


Error received when including an extra space in a padding or margin prop hard to diagnose

Small thing!
when I pass a padding prop like padding="0 lg 0 0 " with an extra space at the end I get this error: TypeError: Cannot read property '0' of undefined.
it'd be nice to have this typechecked (maybe with template literal types? https://www.typescriptlang.org/docs/handbook/2/template-literal-types.html), enforced in the code, or just have it show a more friendly error message!
can probably repro with:
<Box padding="0 lg 0 0 ">test</Box>

@mtblanton will evaluate our options here and get back to you on this, thanks @mtblanton

 Minor bugs

Hey @mtblanton I've been exploring this and ran into 2 main issues with the template literal types. Although if there is a better way to do this I'd appreciate your feedback.
The approach I took was to create a generic CssShorthand<T extends string> type that can then be used to create specific types that follow the format of 1, 2, 3, or 4 single space separated values.
For example, padding and margin would use type BaseSpacing = CssShorthand<SpacingSize>
This did the job quite well in terms of type checking string values in the component like this:
// TS error
<Box margin="md       md" />

// TS error
<Box margin="md randomText" />

// NO TS error
<Box margin="md md" />

However it failed in two places:
FIrst off, it is not working properly with template strings even if the variables share the type that is passed to the generic
const verticalMargin = 'md' as SpacingSize;

// Type 'string' is not assignable to type '"base" | "0" | "5xl" | etc...
<Box margin={`${verticalMargin} lg`} />

secondly, it seems the number of permutations is just too large (around 40,000), so TS complains with this all over the place:
Expression produces a union type that is too complex to represent.
Unless there is a more performant alternative, I think I will simply add an error message for devs to warn them in the console.

Error message in the console sounds good!