chore: version cleanup of TF, inclusion of version 1.8.x

Purpose

Clean up of the test matrix for the Terraform CLI versions, namely removal of outdated versions
Addition of Terraform  version 1.8.x to the test matrix

Does this introduce a breaking change?

[ ] Yes
[X] No

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

[ ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe: setup of tests

How to Test

Execution of test.yml GitHub Action

What to Check
Verify that the following are valid:

Test matrix contains the TF versions 1.5, 1.6, 1.7 and 1.8

Other Information
ATTENTION - This PR must be merged at a point when no PRs are open.
After merge, the settings for the PR checks in the GitHub repository must be updated, namely the list of the required status checks ""Terraform Provider Acceptance Tests " in the branch protection for the main branch must be updated:

Remove the required checks for Terraform version 1.3 and 1.4
Add the check for version 1.8.x

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

[X] The PR is assigned to the Terraform project and a status is set (typically "in review").
[X] The PR has the matching labels assigned to it.
[X] The PR has a milestone assigned to it.
[X] If the PR closes an issue, the issue is referenced.
[X] Possible follow-up items are created and linked.

BP Rules for 1.3&1.4 removed. 1.8 to be added after merge


remember time entries for more than n days

When time flies by, Toggl do not remember previous registrations. My solution is to register new tickets... but I thing Toggl should have them remembered/searchable. I usually have recurring activities every ex 14th day , or every month. Toggl do not have them searchable..

Could you add config option to sync for example time entries for last week, month, two months?


Regression for no attribute 'reduce_cython' in Cython 0.29

Platform: Windows 10
Python Version: 3.6.7
When running a program built with PyInstaller, cdef class types built with Cython 0.29 or later fail during extension module initialization with
AttributeError: type object '<type>' has no attribute '__reduce_cython__'.
I can confirm that the fix in Cython 0.28.3-2.8.6 still works.
The program works without issue when run from Python instead of the PyInstaller bundle.

I assume you read #1953, especially https://github.com/cython/cython/issues/1953#issuecomment-398128940
Is the Cython version the only thing you changed that made it fail? (Same CPython, same PyInstaller version, …)
Did you verify that the generated C code that fails comes from a 0.29.x version? (First line in the file.)
Did you verify that your code does not have circular imports?
I have no experience with PyInstaller and cannot say what it does internally. To debug this, you can search for Re-initialisation is not supported in the Cython generated C file and add a couple of C printf() debug statements here and there, to see what code path it takes and how often.

Cython version is the only variable.
Building a toy example does not encounter this problem, but when I include a large module with many C-extensions, I see __pyx_pymod_exec_<module> gets called twice from PyInstaller, but not Python. I've checked that all of the generated C files for that module were produced by Cython 0.29. To my knowledge, there are no circular imports, though the extension module initialization order does differ when run through PyInstaller compared to Python.
Curiously, when passing through the __pyx_pymod_exec_<module> function, the re-initialization is not detected:
static CYTHON_SMALL_CODE int __pyx_pymod_exec_<module>(PyObject *__pyx_pyinit_module)
#endif
#endif
{
  PyObject *__pyx_t_1 = NULL;
  int __pyx_t_2;
  static PyThread_type_lock __pyx_t_3[8];
  __Pyx_RefNannyDeclarations
  printf("Begin exec_peak_statistics\n");
  #if CYTHON_PEP489_MULTI_PHASE_INIT
  printf("BEGIN MULTIPHASE_INIT\n");
  if (__pyx_m) { //<<< Never True
    if (__pyx_m == __pyx_pyinit_module) return 0;
    PyErr_SetString(PyExc_RuntimeError, "Module '<module>' has already been imported. Re-initialisation is not supported.");
    printf("Module '<module>' has already been imported. Re-initialisation is not supported.\n");
    return -1;
  }
  #elif PY_MAJOR_VERSION >= 3
  if (__pyx_m) return __Pyx_NewRef(__pyx_m);
  #endif

When the program is run with PyInstaller, my print calls run like so:
Begin exec_<module>
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...
Begin exec_<module>
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...

When run with Python, instead I get:
Begin exec_<module>
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...

So my print call in the block that tests to see if the module has been re-initialized is never executed. During both passes through the exec_<module> function, __pyx_m is NULL and __pyx_pyinit_module is some arbitrary pointer. __pyx_pyinit_module does not point to the same location when PyInstaller calls exec_<module> for the second time:
Begin exec_<module>
BEGIN MULTIPHASE_INIT
__pyx_m =<PHONE_NUMBER>000000
__pyx_pyinit_module = 000002ACCC241AE8
Initializing module...
Caching module...
Begin exec_<module>
BEGIN MULTIPHASE_INIT
__pyx_m =<PHONE_NUMBER>000000
__pyx_pyinit_module = 000002ACCC2206D8
Initializing module...
Caching module...

Thanks for the investigations. Could you try a couple of more things? There is a function called __pyx_module_cleanup. Please add debug prints to that, too, to see if it is called in between the imports. Also, please add a printf() at the end of the exec function to see if the second import happens before or after terminating it. If it's within the first call, that would suggest that something in your module code triggers a re-import of the module. If it happens afterwards, that could suggest that PyInstaller is triggering it itself.
__pyx_pyinit_module is the module object created for the import, so it's normal that it refers to different objects on different imports.
Could it be that you are importing the module in different ways in your code? E.g. with absolute and relative imports, or with both import pkg.module and from pkg import module? It might be that PyInstaller is unable to detect that both refer to the same module, and then it might end up asking CPython to import the same module twice, which could make it re-initialise the same extension module.
It is weird, though, that __pyx_m is NULL in both cases. I assume it's being set where it says Caching module in your output? After that, it should have a non-NULL value…
Also, could you switch to 0.29.10 please, or even give the latest master a try? Just to be sure we're not chasing zombies.

I switched to 0.29.10:
Unfortunately, I can't find any symbol with the word "cleanup" in it.
Here's an expanded log for PyInstaller:
Begin exec_<module>
__pyx_m =<PHONE_NUMBER>000000
__pyx_pyinit_module = 000001D45C300AE8
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...
Begin populating globals...
An error occured? __pyx_lineno 1
__pyx_clineno 34110
Py_CLEAR(__pyx_m)
Returning -1
Begin exec_<module>
__pyx_m =<PHONE_NUMBER>000000
__pyx_pyinit_module = 000001D45C2E06D8
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...
Begin populating globals...
An error occured? __pyx_lineno 621
__pyx_clineno 33939
Py_CLEAR(__pyx_m)
Returning -1

and for Python:
Begin exec_<module>
__pyx_m =<PHONE_NUMBER>000000
__pyx_pyinit_module = 000001BCC3E7BF48
BEGIN MULTIPHASE_INIT
Initializing module...
Caching module...
Begin populating globals...
NumPy C-API Initialized
...Done populating globals
__pyx_m = 000001BCC3E7BF48
Returning 0

This shows the PyInstaller ran module initialization is quitting somewhere in the init code and going to the error handling bock. It looks like the failure state differs between passes through the initialization block, but the module globals might have been partially initialized.
The first error happened in __Pyx_modinit_type_import_code, on a line importing a C module which no Python code imports directly. This made it a "hidden import", which I have to tell PyInstaller about explicitly. However, the explicit notification method I used was written for Python 2, and there was a small change required for Python 3. Without it , the relevant .so/.pyd isn't included in the bundle, and attempting to import it should error out. I would have expected that to bubble up to the top immediately with an ImportError, as it does on Py2, but alas it instead just leaves the module in a partially incomplete state that the next import attempt runs into. When I properly specify the hook for Py3, the bundle works without issue.
So the problem appears to be not that the generated __reduce__ methods were missing, but that when they were queried and failed was somehow the next moment that Python errors could be raised?

I just encountered a similar problem again after a situation where an aggressive system upgrade altered the Python minor version level, which caused certain cimports to break, but issued no warning messages, only yielding a partially initialized extension module.
Under some conditions __Pyx_AddTraceback used in may fail to produce a visible traceback for the caller. Would it be acceptable to include some print statements through standard error when an error occurs? I can write the patch and PR.

I have a pretty simple pure-Python and Cython repro of this problem. Using Cython 0.29.13, create this file cytest.pyx:
import doesnotexist

cdef class Foo:
    pass

Then after compiling it, run:
try:
    import cytest
except ImportError:
    pass

import cytest

This is not just a toy example, it was simplified from a real-world Django application. When importing your INSTALLED_APPS, Django has code to catch ImportError and try to import again, treating the final component of the dotted path as an attribute instead of a submodule. We had an ImportError in unrelated code several layers deeper in the import stack from our Cython module, and Cython (in combination with Django's re-importing behavior) effectively transformed this ImportError into the __reduce_cython__ error, totally hiding the real ImportError and making it very difficult to debug.

Thanks for the reproducer. I don't know if there is something that can be done as an intermediate fix (so I'll leave the ticket open), but this will definitely be resolved with #2343.

Closing as duplicate of #3545.


Bump ts-protoc-gen version to include server interfaces

Bump the version of the typescript generator dependency to include its latest changes:

0.13.0: Added support for @grpc/grpc-js.
0.14.0: Added support for grpc and @grpc/grpc-js server interfaces.

Hi @ido-namely, any chance that this could be merged? It's a simple library dependency bump and would be really nice to have.

Hi @ido-namely, any chance that this could be merged? It's a simple library dependency bump and would be really nice to have.

@playground-julia, thank you for the contribution!


Voter choice shown "Invalid choice" in UI

Briefly describe the bug.
Choices are defined invalid in some proposals
How can we reproduce the bug?

Go to the proposal in issue
View the Votes list
See error

What is the expected behaviour?
It seems only choices in ENS name are defined as "invalid choice" and thus can not be shown in the UI
The graphql query to this vote is not influenced
Can you attach screenshots?
No response
Can you provide your device specific details below?
No response

This is only happens on older ranked-choice proposals where we didn't enforce to select all ballots. The results seem to have been not recalculated since our change to ranked-choice voting and so they are still intact.


Please invite me to the GitHub Community Organization

Name
funnyleap
Discord Username (if applicable)
fatBoy
Additional Context
i wanna to join in this community

thank you.


chore: Build all modules of sdk-platform-java in hermetic generation workflow.

Could you please add the issue link to the pr body?
I think a better solution is not downloading/installing the generator if it's not used, generating proto-only libraries, for example.
That being said, install all modules can be a quick fix and we can improve this later.


interpret undefined outputs in tx as an empty array (not an error)

See https://github.com/cardano-foundation/ledgerjs-hw-app-cardano/issues/32.
If yes, it should be done so in other possible places.

Won't do; Conway CDDL is even more strict.


Fix typos + light copyediting in ReadMe

"Instroduces" -> "Introduces", "it's" -> "its"; also added a couple of commas and colons

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


[Setup] Fix dependency

Description
This PR fixes the dependency issues. By default, users do not need to have DeepSpeed or HuggingFace Transformers to be installed.
Checklist

[x] PR's title starts with a category (e.g. [Bugfix], [Model], [Tutorial], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Thanks @chhzh123


Exposing DropZone class so that confirm can be edited

The DropZone class can have its confirm method overridden.
please see:
https://docs.dropzone.dev/configuration/basics/methods
as it stands, there doesn't seem to be a way to access the DropZone class directly via nuxt-dropzone, only an instance.
My suggestion would be to allow DropZone to be exposed via a plugin

I think a simple static property would make more sense than a plugin, but in general, I think the idea is good if the reference can be easily recovered. As covered in the readme, contributions are welcome so feel free to open a pull request with the relevant changes.


Cc/collections limited

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Migrations

[ ] Migration is backwards-compatible with current production code

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings are documented and present in app.json
[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
(Required)
What's this PR do?
(Required)
How should this be manually tested?
(Required)
Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
(Optional)
What GIF best describes this PR or how it makes you feel?
(Optional)

Codecov Report

Merging #1055 (8b1798f) into master (e2a38eb) will increase coverage by 4.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1055      +/-   ##
==========================================
+ Coverage   91.13%   95.30%   +4.17%
==========================================
  Files         201       97     -104
  Lines        7679     2641    -5038
  Branches     1332      546     -786
==========================================
- Hits         6998     2517    -4481
+ Misses        596      124     -472
+ Partials       85        0      -85

Impacted Files
Coverage Δ

static/js/types/websites.ts
100.00% <ø> (ø)

static/js/components/widgets/MarkdownEditor.tsx
81.25% <100.00%> (-0.39%)
:arrow_down:

...tic/js/components/widgets/ResourcePickerDialog.tsx
100.00% <100.00%> (ø)

static/js/lib/site_content.ts
97.56% <100.00%> (+0.01%)
:arrow_up:

ocw_import/tasks.py

content_sync/backends/github.py

content_sync/apis/github.py

content_sync/pipelines/concourse.py

websites/admin.py

ocw_import/conftest.py

... and 98 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 e2a38eb...8b1798f. Read the comment docs.


Added: wox 1.3.881

Hi, I uploaded this. I'd like to know if there are any problems with merging this


feat: retrieve deployed contract

Any thoughts on making tainting/internal optional fields for the macro with a default value? It would be nice when making these, but I admit you run the risk of forgetting to set the field.

One last complaint: I really don't love the word "tainting" for this. It feels like internal kitchen stuff that driver authors shouldn't need to be aware of? Would something like critical work?

@MicaiahReid

what's the internal flag for?

Internal stands for inputs needed internally, that we don't want to expose in the documentation

dependency_contract_ids & lazy_dependency_contract_ids

Addressed with latest commit

Is "lazy dependency" a known concept?

In this context, it means that the dependencies can be in the same block - vs committed in a prior block.

It's interesting that the api url isn't tainting for call_contract, but it is for this

Addressed with latest commit

It feels like internal kitchen stuff that driver authors shouldn't need to be aware of?

Unfortunately this work needs to be done manually, and a default value would lead to bugs. I don't see how we can infer this, but could be missing something

I really don't love the word "tainting" for this

The tainting concept / terminology is coming from Terraform where tainted resources are detected automatically but can also be done manually through a command line - txtx taint <runbook> <action> for forcing the re-execution of a subgraph. Open to another idea, but I don't see critical as an adequate alternative.

Internal stands for inputs needed internally, that we don't want to expose in the documentation

Very nice 💯

The tainting concept / terminology is coming from Terraform where tainted resources are detected automatically but can also be done manually through a command line - txtx taint <runbook> <action> for forcing the re-execution of a subgraph. Open to another idea, but I don't see critical as an adequate alternative.

Okay, understood. How about can_taint_execution or can_taint_state? The ing suffix feels off, I'd love to have it worded more like a bool. After that, I'm good!

We're currently using adjectives for qualifying inputs (internal vs is_internal , optional vs is_optional, etc).
Can you think of an alternative adjective for qualifying these tainting inputs?

Sure, but I'd argue that a bool like internal/optional is much clearer in this case - they say what this input is, while tainting describes what it does.
Since as you said tainting is a specific concept with Terraform, I don't think we should look for another adjective, but rather word it in a way that is less awkward and makes the purpose of the field clearer.


Human tolerant database manager

Fixed bug that prevents $ manage-db import from running
Added warning for $ manage-db import and $ build-test-db

(cree-intelligent-dictionary) beef:~/cree-intelligent-dictionary$ manage-db import CreeDictionary/res/dictionaries
This will DELETE database /home/matt/cree-intelligent-dictionary/CreeDictionary/db.sqlite3
 and create a new database from scratch.
Are you sure? [y/N] N
Aborted

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@54d5b1a). Click here to learn what that means.
The diff coverage is 68.75%.

@@           Coverage Diff            @@
##             master    #288   +/-   ##
========================================
  Coverage          ?   83.3%
========================================
  Files             ?      55
  Lines             ?    2109
  Branches          ?       0
========================================
  Hits              ?    1757
  Misses            ?     352
  Partials          ?       0

Impacted Files
Coverage Δ

...Dictionary/tests/DatabaseManager_tests/cmd_test.py
100% <100%> (ø)

...aseManager_tests/test_xml_importer_regression_0.py
100% <100%> (ø)

...y/tests/DatabaseManager_tests/xml_importer_test.py
100% <100%> (ø)

CreeDictionary/tests/API_tests/model_test.py
100% <100%> (ø)

CreeDictionary/DatabaseManager/__main__.py
78.72% <54.54%> (ø)

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 54d5b1a...8cdbd3c. Read the comment docs.

Looks good but...
Why did most of the files in this PR get their mode changed from 644 to 755? This is like you ran
chmod a+x *

Uh... why?

wait my fault, I was doing some really derpy debugging and chmod -R 777d everything


Remove mix firmware.push

firmware.push had a number of issues due to it using Erlang's built-in
ssh implementation and not the system's OpenSSH. The main ones were that
it didn't know about .ssh/config, ssh-agent, how to ask for a
key passphrase, or how to keep track of host keys. Work get be done to
fix all of these issues, but the easier solution for now is to shift use
over to the shell script since it just calls the system's ssh with
appropriate arguments.

This probably looks pretty extreme. I think that many of you know that I don't use firmware.push due to the cleartext passphrase issue, but additional issues have come to my attention. I don't see firmware.push being fixable without a lot of work.
For the record, it would be nice to have push integrated into mix again. I think that the route for that is by using commandline ssh rather than Erlang's ssh client. Since I'm gotten used to the quick startup time of the shell script and felt some of the pain of invoking interactive commandline apps from Erlang, I'm not feeling motivated to pursue this route myself. I'm definitely not opposed to it, though.

Resolves #4

Rebased - I'm ready to remove mix firmware.push :fire:

Yes - no happiness. I'm grumpy too, but as you know, this has been an upsetting issue since before I even posted this PR 6 months ago. Pressing the merge button is going to feel good...


Set-TargetResource fails for array based security option if not already configured

Set-TargetResource will fail to succeed if an array based security option is passed but there is no corresponding value already present in the $currentSecurityOptions dictionary object. Found whilst testing with Kerberos ETypes which isn't configured by default in the OS.
Cannot bind argument to parameter 'ReferenceObject' because it is null.

CategoryInfo          : InvalidData: (:) [], CimException

Simply adding another if clause to simply return false if the security option is missing from the $currentSecurityOptions resolves the issue.
Line 1031 of MSFT_SecurityOption.psm1
if ( $desiredSecurityOptionValue -is [array] )
            {
                If ( $currentSecurityOptions[$policy] )
                {
                    $compareResult = Compare-Array -ReferenceObject $currentSecurityOptions[$policy] -DifferenceObject $desiredSecurityOptionValue

                    if ( -not $compareResult )
                    {
                        return $false
                    }
                }
                else
                {
                    return $false
                }
            }

@russelltomkins Are you about to fix this too in the PR that you are about to send it?

I do have a fix (as above), but another having another quick look at the parent condition loop I think there maybe another bug

If you were to wrap the reference object in an empty array it would handle a null value: @($currentSecurityOptions[$policy]).
The array will be flattened by PS if there's an array held in the hashtable value.

@russelltomkins curious if you are still looking at this? This bit me today and @indented-automation helped me track it down to here.


Rainmeter.Rainmeter version <IP_ADDRESS>21

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Rainmeter cannot install silently, also the download url is not proper.

Rainmeter cannot install silently, also the download url is not proper.

but it works!!!

Hello @ayushthesmarty,
The package manager bot determined there was an issue with some of the URLs included in the manifest file. Please check the pull request for more details and make sure the url is correct.

it is

Just right click the installer and copy the link https://github.com/rainmeter/rainmeter/releases/download/v<IP_ADDRESS>21/Rainmeter-4.3.1.exe. This applies to GitHub links only

we have removed the "silent install" process from Rainmeter 4.0, and it won't be supported going forward.

Once winget supports portable apps and scripting some trickery can be done to install this application.


Change csw orm to delete records

Coverage increased (+0.0007%) to 99.471% when pulling 944e85d0da7ac6819b19f64ecc12dcaec9cf0f00 on JorgeMartinezG:csw_orm into 514886e35149f8e1a5616fd407af0f45079efb83 on boundlessgeo:master.


Bot not listening to itself

I have a bot that listens to other bots, but it wont listen to itself.
Below is the implementation. When I post "@botname test", it should
var Botkit = require('./node_modules/botkit/lib/Botkit.js');

// Configure bot
var controller = Botkit.slackbot({
    debug: false
});

var bot = controller.spawn({
    token: process.env.token,
}).startRTM();

controller.hears(['howdy'],'ambient',function(bot, message) {
bot.reply(message, "heard howdy")
console.log(message)
})

controller.hears(['test'],'ambient',function(bot, message) {
bot.say(
    text: "howdy",
    channel: "CHANNELID"
)
})

That's called a direct mention, starting a message with @username.
You are listening to ambient events, and also note thats likely only in the channels your bot is invited to. Are you able to trigger a response on simply saying 'howdy' or ''test?
try this code to get your bot to responde to the input @botName test:
controller.hears(['test'],'direct_mention',function(bot, message) {
  bot.reply(message, 'Howdy!')
})

@arawal did this answer your question?


Is there any way to do rotate in this library?

It will be good if the viewer have ability to rotate, isn't it?

It is also necessary for me

I implemented rotation outside of the lib by doing something like this:
given the ref to an image:
  const imageRef = useRef<HTMLImageElement>(null)

I implemented and exposed the following API (through a custom context):
    const rotateClockwise = useCallback(() => {
      if (!imageRef?.current) return
      setRotation((rotation) => {
        const newRotation = (rotation + 90) % 180
        imageRef.current!.style.transform = `rotate(${newRotation}deg)`
        return newRotation
      })
    }, [setRotation, imageRef]);
    const rotateCounterClockwise = useCallback(() => {
      if (!imageRef?.current) return
      setRotation((rotation) => {
        const newRotation = (rotation - 90) % 180
        imageRef.current!.style.transform = `rotate(${newRotation}deg)`
        return newRotation
      })
    }, [setRotation, imageRef]);

Then I implemented it on the image itself:
  return (
      <TransformWrapper>
        <ImageControls />
        <TransformComponent>
          <img ref={imageRef} id="img" src={uri} />
        </TransformComponent>
      </TransformWrapper>
   )

then the ImageControls is just a custom version of the default Controls that implements both useControls() for the lib and my own context.
TBH, if I got little time, I believe it can be easy to add to the API
HTH

This should be a pull request.

This solution doesnt update the trasform wrapper/component rotation thus it make the panning bugged.
Im searching for an alternative solution, did anyone found something yet?

Setting the width and height in the wrapper and content wrapper with an !important fixed the panning issue for me, and centering the content with flex if your alignment is off.
const [rotation, setRotation] = React.useState(0);

const rotateClockwise = useCallback(() => {
    setRotation(rotation => {
        const newRotation = (rotation + 90) % 360;
        return newRotation;
     });
}, [setRotation]);

const rotateAnticlockwise = useCallback(() => {
    setRotation(rotation => {
        const newRotation = (rotation - 90) % 360;
        return newRotation;
    });
}, [setRotation]);

return (
    <TransformWrapper>
       <TransformComponent
           wrapperClass="!h-screen !w-screen"
           contentClass="!h-screen !w-full flex items-center justify-center"
        >
            <picture>
                <img
                   src={src}
                   style={{
                       objectFit: "cover",
                       maxHeight: "100vh",
                       maxWidth: "100vw",
                       transform: `rotate(${roation}deg)`
                   }}
                />
            </picture>
        </TransformComponent>
     </TransformWrapper>

     <button onClick={() => rotateClockwise()} >Rotate clockwise</button>
     <button onClick={() => rotateAnticlockwise()} >Rotate Anticlockwise</button>
)


IA-VS signature mismatch should filter out system-value semantics

If a vertex shader use system-value semantics, renderdoc will incorrectly report that IA and VS signatures does not match.

Can you give some more details? I don't doubt that there is a bug, but I don't think this is a general problem any time system-value semantics are used anywhere.
If you could give me a capture that would be ideal, but at least e.g. the shader signature used to generate both the IA bytecode and VS bytecode and the input-layout descriptors.

Sure, sorry for being vague. I took a capture of the Microsoft InstancingFX11 sample as an example ( https://code.msdn.microsoft.com/Effects-11-Win32-Samples-cce82a4d ).
You can grab it here https://www.dropbox.com/s/j3cq2wdy1eakvu8/InstancingFX11_2015.10.25_23.07.40_frame227.rdc?dl=0
The problem can be found on the event 92. Both IA and VS has the same signature except for the additional "SV_VertexID" on the vertex shader side. I used renderdoc v0.26.

Thanks! I see what you mean now, trailing 'extra' system-value elements are allowed, since the rest of the signature remains the same. That commit should fix the false positive.

Closing as this was fixed for v0.27. Re-open if it's not properly resolved.


zrtp: check for RTP packet in send handler

non-RTP packets such as STUN, TURN, ICE might be shared on the
same UDP socket, and these packets should not be processed by ZRTP.
ref #262
@juha-h please review and merge to master if OK.

Alfred E. Heggestad writes:

non-RTP packets such as STUN, TURN, ICE might be shared on the
same UDP socket, and these packets should not be processed by ZRTP.
ref #262
@juha-h please review and merge to master if OK.

I'll merge after @edgeworks-vucao has tested the patch.
-- Juha

@juha-h , I tested the patch, it works well.


Add changes to support Jax 0.4

Description
Add changes to support jax>=0.4.0 and jaxlib>=0.4.0
Fixes #359
Note : There might be some typing imprecisions, I think some Any type could be remove since jax.Array is supposed to be general, but I wasn't so sure.
To add in the next major release
Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)

Screenshots
Please attach before and after screenshots of the change if applicable.

Checklist:

[X] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

@charraut There has been a change of plan and the next release with be a major release allowing us to include this PR
Could resolve the conflicts then we should be good to merge

@charraut There has been a change of plan and the next release with be a major release allowing us to include this PR Could resolve the conflicts then we should be good to merge

Done! Let me know if you need any specific changes

Done! Let me know if you need any specific changes

Amazing, thanks


How to pass videofile name to representation?

Hi, how do I pass file url to each Representation?
I generate different quality videos on file upload: video_240.mp4, video_360.mp4 and so on.
I can't set neither representation ID or Media url, so I don't get it how player supposed to find these convertions for adaptive bitrate streaming? Options are generated in mpd file but they seem to point to the same video file
$ffmpeg = FFMpeg::create();
$video = $ffmpeg->open("$fullPath$fileName" );

$fileName = pathinfo($fileName, PATHINFO_FILENAME); // removes extension
mkdir($fullPath . $fileName);
$path = $fullPath . $fileName . '/';

$codec = new Video\X264();
//hd 1080
$codec
    -> setKiloBitrate(6000)
    -> setAudioKiloBitrate(192);
$video
    ->filters()
    ->resize(new Coordinate\Dimension(1920, 1080))
    ->synchronize();
$video
    ->save($codec, $path . $fileName .'_1080.mp4');
//hd 720
$codec
    -> setKiloBitrate(3000)
    -> setAudioKiloBitrate(128);
$video
    ->filters()
    ->resize(new Coordinate\Dimension(1280, 720))
    ->synchronize();
$video
    ->save($codec, $path . $fileName .'_720.mp4');
//high
$codec
    -> setKiloBitrate(1400)
    -> setAudioKiloBitrate(96);
$video
    ->filters()
    ->resize(new Coordinate\Dimension(854, 480))
    ->synchronize();
$video
    ->save($codec, $path . $fileName .'_480.mp4');
//med
$codec
    -> setKiloBitrate(1000)
    -> setAudioKiloBitrate(96);
$video
    ->filters()
    ->resize(new Coordinate\Dimension(640, 360))
    ->synchronize();
$video
    ->save($codec, $path . $fileName .'_360.mp4');
//low res
$codec
    -> setKiloBitrate(400)
    -> setAudioKiloBitrate(64);
$video
    ->filters()
    ->resize(new Coordinate\Dimension(426, 240))
    ->synchronize();
$video
    ->save($codec, $path . $fileName .'_240.mp4');

$r_240p  = (new Representation)->setKiloBitrate(150)->setResize(426, 240);
$r_360p  = (new Representation)->setKiloBitrate(276)->setResize(640, 360);
$r_480p  = (new Representation)->setKiloBitrate(750)->setResize(854, 480);
$r_720p  = (new Representation)->setKiloBitrate(2048)->setResize(1280, 720);
$r_1080p = (new Representation)->setKiloBitrate(4096)->setResize(1920, 1080);

$video->dash()
    ->setSegDuration(10)
    ->setAdaption('id=streaming,streams=v id=1,streams=a')
    ->x264()
    ->addRepresentations([$r_240p, $r_360p, $r_480p, $r_720p, $r_1080p])
    ->save($path . '/' . $fileName . '.mpd');

I didn't get your question meaning. If you mean how DASH works, you can see its documentation and RFC.

Thanks. Will read that RFC and am still studying dash documentation. I asked how to choose multiple files I have created when I use dash() to generate mpd file. Segment template seems to use representation id to locate correct files, but I am still new to this so I might be wrong. What I have used for now is this solution you suggested: https://github.com/aminyazdanpanah/PHP-FFmpeg-video-streaming/issues/21/ , it works as desired. If anyone want's to setup their streaming like that on their website and have problems installing shaka packager I suggest using npm to install already build shaka packager binary


Add support for Air 2 and Air 2 Pro

I added the necessary code to detect and connect to the Air 2 and Air 2 Pro headsets (I didn't personally test but I had some of my users test it).
Since I don't have the hardware, I wanted to try to utilize the on-board config JSON to calculate the lens parameters. I know you said in Discord that you found them pretty inaccurate, but I did a comparison using the hand-calibrated values and what's on my device, and I found the results imperceptibly different. A few things to note:

I had to customize the projection matrix based on the K matrix included. The main issue here is the off-center principal point in the matrix, which is effectively a translation of the lens.
The on-board config data is weirdly translated, putting the right eye near x = 0. I compensate for the translation in the driver by centering them on the origin. It would probably be better to also rotate the values so that they fall on the x axis, but the difference is small.
There's also a lens distortion array in the config JSON. The format is straightforward, it's an 18x32 list of (sx, sy, dx, dy) tuples, where sx, sy is a point on the device's screen and dx, dy is the linear distortion at that point. I haven't attempted to use this, so this is just a point of interest for future work.

Yes, it replaces imu_to_display_matrix (which returns the extrinsic matrix of the camera) and display_fov, which is derived from the intrinsic matrix of the camera (yfov = atan(h / 2 / fy)). For reference, here is the code my Mac app uses to create the projection matrix from the intrinsic matrix:
struct IntrinsicMatrix {
    /// Horizontal focal length, in pixels
    var fx: CGFloat
    /// Vertical focal length, in pixels
    var fy: CGFloat
    /// Principal point X coordinate
    var cx: CGFloat
    /// Principal point Y coordinate
    var cy: CGFloat
    /// Image width
    var w: CGFloat
    /// Image height
    var h: CGFloat

    init(_ from: DisplayMatrices) {
        guard from.intrinsic_matrix.len() == 9 else { fatalError("invalid intrinsic matrix") }
        fx = from.intrinsic_matrix[0]
        fy = from.intrinsic_matrix[4]
        cx = from.intrinsic_matrix[6]
        cy = from.intrinsic_matrix[7]
        w = CGFloat(from.resolution.w)
        h = CGFloat(from.resolution.h)
    }

    func projectionTransform(zNear: CGFloat, zFar: CGFloat) -> SCNMatrix4 {
        let a11 = 2 * fx / w
        let a22 = 2 * fy / h
        let a31 = 2 * cx / w - 1.0
        let a32 = 2 * cy / h - 1.0
        let a33 = -(zFar + zNear) / (zFar - zNear)
        let a43 = -2 * zFar * zNear / (zFar - zNear)

        return SCNMatrix4(
            m11: a11, m12: 0.0, m13: 0.0, m14: 0.0,
            m21: 0.0, m22: a22, m23: 0.0, m24: 0.0,
            m31: a31, m32: a32, m33: a33, m34: -1,
            m41: 0.0, m42: 0.0, m43: a43, m44: 0.0
        )
    }
}

Note that the projection matrix is transposed from how OpenGL does this. I have no idea why SceneKit wants this, but when you implement this in another framework you'll want to transpose this.

Ok, this seems like a well thought-out change then. I'll test it out in detail when I have some time and maybe deprecate the other two functions.
I will have to think about (and experiment with) IPD differences though. Do you do anything with that at all? Or are these calibration values calibrated to a specific 'median' IPD?

The translation component is likely calibrated to the focal point of the lenses (which I read from an XREAL forum post is 65mm apart). A human with a different IPD would experience some distortion from not having their eyes in the place the lenses expect. That said, it’s a pretty minor distortion, and the translation component can be outright ignored, which I believe would only affect the scale factor between real-world and virtual-world coordinates.

Do I understand correctly that the newly introduced display_matrices function should probably replace imu_to_display_matrix?

Also, we could view display_matrices as a more detailed version of these interfaces. I could write code that derives the data for imu_to_display_matrix and display_fov from these values and submit that as a PR, if you'd like.

I was planning to do exactly that, but a PR would also be helpful :D


Infer group networking.x.k8s.io from native resource types

What would you like to be added:
The ability to infer group networking.x.k8s.io for native resource types. Group core is inferred for the Service resource. Extended this idea to infer networking.x.k8s.io for native resources such as HTTPRoute.
Why is this needed:
Simplify the UX.
/assign @bowei
/cc @Miciah

/assign

@danehans I think this came from a time when we still had RouteObjectReference. I've looked through our other ObjectRefs and haven't found any remaining ones that would reference networking.x-k8s.io types, do you think this is safe to close now?

This issue is no longer valid now that gateway<>route binding is being accomplished with selectors.


Update Hashing Algorithm to BCRYPT

Could you use password_hash ( string $password , mixed $algo [, array $options ] ) : string instead of using hash?

thank for the review but the idea is not to hash only password. it can be used for othe purpose also

I thought it was for passwords only, sorry about that!
On Wed, Jul 8, 2020, 11:38 PM ibrahim<EMAIL_ADDRESS>wrote:

thank for the review but the idea is not to hash only password. it can be
used for othe purpose also
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bim-g/library/pull/1#issuecomment-655771349, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIKSKRSMBDKL7OP6RCHXEP3R2TRLVANCNFSM4ONI4NAQ
.


HasOneOrMany wrongly uses TRelatedModel as the type for the declaring model of the relation

HasOneOrMany has only a single template parameter TRelatedModel which is bound to the model class which the relation points to. However, HasOneOrMany also uses TRelatedModel as the type for the model class which declares the relation.
https://github.com/nunomaduro/larastan/blob/ccdd599c464f9de6983a8d5e0f36daaf324b646f/stubs/HasOneOrMany.stub#L11-L20
The declaring model model (or parent or owning model) is not the same as the related model.
Originally posted by @nagmat84 in https://github.com/nunomaduro/larastan/discussions/1275#discussioncomment-2933818

@canvural, @szepeviktor, this can now be closed


Improved logging format

Description of Work
Fixes #105
Slightly improved the logging format such that it is more readable.
Testing Instructions

In the mantidpython terminal, run example_runScript.py.
Check if the output is satisfying.

sorry what is the difference in the output?

Sorry, should have been more specific. The differences are in the log file. The logs can be found in /fitbenchmarking/fitbenchmarking/logs/fitbenchmarking.log


Allow configuring the sass engine

According to this article node-sass may be behind in terms of features but is still faster by a factor of around 2.

Hey @jonathantneal, any feedback on this ?


Group chat support

We need the ability to test multiple users in a conversation. Today, I have to deploy my bot service and attach VS to that deployed instance and test from one of the channels to be able to test group conversations. This obviously then interferes with other regular users if your bot has already been published. We really need a solution here to help test a group conversation through the emulator.

In the current release, you're able to add and remove users from the conversation by sending the corresponding conversationUpdate system activity from the app menu:

In the next emulator release, we plan to add basic support for switching between users in the conversation, meaning you'll be able to select which user is 'you'.
Is the functionality described here adequate to test your scenario?

Yeah, this is not adequate.  This seems like its just a simulator for you to be able to test the activity types that come in.  But when you're actually in a group chat within, for example, Skype, behaviors are extremely different.  Right off the bat, my bot which uses LUIS dialog completely failed once I published my app and turned on Group chat capabilities from the bot framework portal.  When in a group chat, you can only interact with the bot if the bot was called with an @ mention.  Out of box, LUIS dialogs fail when a mention is in the text because that text contains the XML representing the bot name that was called.  So I had to put in code to strip out XML from text before it was sent to my luis app instance.  Also my bot uses the Bot Location Control and that didn't work in group chats for the same reason. I ended up having to replace the NuGet reference with a copy of the location control source code so I can edit the code to handle group conversations.  This part is a separate issue, but related to this issue I've created, the problem is that there is no way for me to have tested all this code prior to publishing my bot and trying it out from an actual channel because this bot emulator doesn't really emulate group chat.

XML in the text field sounds like a Skype channel bug. XML shouldn't exist in the text field of the message. Hence the name text. Would you mind opening and issue for this over at https://github.com/Microsoft/BotBuilder/issues?
If this XML issue were cleared up in the Skype channel, do you think the emulator with the planned features I described above be adequate to test your scenario?

Closing due to inactivity. If you'd like to continue the discussion please comment here and I will reopen.

@eanders-MS : when is the next emulator release scheduled? We are facing issue while debugging group conversations. Also, in group conversation lets say we send a message to bot - "@bot please use this address", in next step, the response we get is same we entered i.e. it includes the bot's name. Shouldn't it be only "please use this address" in response?

Same issue reported from @sagarpomal. Now I have fixed it removing the bot tag in every message the user send to bot, but this is a quite tricky solution. Hope this will be fixed in next updates


What's meaning of the magic numbers?

in https://github.com/simonw/til/blob/704ad30c4b7eb6d6009790261e2209331cce7917/metadata.yaml#L24
I wonder what 'b4de2a49c8' and '8c94a2ed4b' mean. Would you explain me this?
Thank you very much.

Those are the sequences that get used to wrap highlighted text - for example:
https://til.simonwillison.net/tils?sql=select
++snippet(
++++til_fts%2C
++++-1%2C
++++'b4de2a49c8'%2C
++++'8c94a2ed4b'%2C
++++'...'%2C
++++60
++)+as+snippet
from
++til
++join+til_fts+on+til.rowid+%3D+til_fts.rowid
where
++til_fts+match+case
++++%3Aq
++++when+''+then+'nomatchforthisterm'
++++else+escape_fts(%3Aq)
++end
order+by
++til_fts.rank
limit
++20&q=pyodide
This returns the following:

b4de2a49c8Pyodide8c94a2ed4b provides "Python with the scientific stack, compiled to WebAssembly" - it's an incredible project which lets you run a full working Jupyter notebook, complete with complex packages such as numpy and pandas, entirely in your browser without any server-side Python component running at all.

Then later I can turn b4de2a49c8Pyodide8c94a2ed4b into Pyodide
I could use  and  directly instead, like this:
https://til.simonwillison.net/tils?sql=select
++snippet(
++++til_fts%2C
++++-1%2C
++++''%2C
++++'<%2Fstrong>'%2C
++++'...'%2C
++++60
++)+as+snippet
from
++til
++join+til_fts+on+til.rowid+%3D+til_fts.rowid
where
++til_fts+match+case
++++%3Aq
++++when+''+then+'nomatchforthisterm'
++++else+escape_fts(%3Aq)
++end
order+by
++til_fts.rank
limit
++20&q=pyodide
But then I'd have to think really hard about HTML escaping: I want to HTML escape any tags that show up in the result, but NOT the strong tags themselves.
So it's easier to use a random token that won't show up in any of the TIL content.


bug OSError: You seem to have cloned a repository without having git-lfs installed. Please install git-lfs and run git lfs install followed by git lfs pull in the folder you cloned.

python main.py --base_model model/chatglm-6b
Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a configuration with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 8/8 [00:06<00:00,  1.30it/s]
2023-09-25 17:02:43,232 [INFO] [SentenceTransformer.py:66] Load pretrained SentenceTransformer: ./text2vec/text2vec-large-chinese/
2023-09-25 17:02:43,232 [WARNING] [SentenceTransformer.py:805] No sentence-transformers model found with name ./text2vec/text2vec-large-chinese/. Creating a new one with MEAN pooling.
Traceback (most recent call last):
File "/data/sg/lib/python3.9/site-packages/transformers/modeling_utils.py", line 415, in load_state_dict
return torch.load(checkpoint_file, map_location="cpu")
File "/data/sg/lib/python3.9/site-packages/torch/serialization.py", line 795, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/data/sg/lib/python3.9/site-packages/torch/serialization.py", line 1002, in _legacy_load
magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, 'v'.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/data/g/LexiLaw/demo/main.py", line 26, in
application = LangChainApplication(config)
File "/data/g/LexiLaw/demo/clc/langchain_application.py", line 26, in init
self.source_service = SourceService(config)
File "/data/g/LexiLaw/demo/clc/source_service.py", line 25, in init
self.embeddings = HuggingFaceEmbeddings(model_name=self.config.embedding_model_name)
File "/data/sg/lib/python3.9/site-packages/langchain/embeddings/huggingface.py", line 54, in init
self.client = sentence_transformers.SentenceTransformer(
File "/data/sg/lib/python3.9/site-packages/sentence_transformers/SentenceTransformer.py", line 97, in init
modules = self._load_auto_model(model_path)
File "/data/sg/lib/python3.9/site-packages/sentence_transformers/SentenceTransformer.py", line 806, in _load_auto_model
transformer_model = Transformer(model_name_or_path)
File "/data/sg/lib/python3.9/site-packages/sentence_transformers/models/Transformer.py", line 29, in init
self._load_model(model_name_or_path, config, cache_dir)
File "/data/sg/lib/python3.9/site-packages/sentence_transformers/models/Transformer.py", line 49, in _load_model
self.auto_model = AutoModel.from_pretrained(model_name_or_path, config=config, cache_dir=cache_dir)
File "/data/sg/lib/python3.9/site-packages/transformers/models/auto/auto_factory.py", line 464, in from_pretrained
return model_class.from_pretrained(
File "/data/sg/lib/python3.9/site-packages/transformers/modeling_utils.py", line 2301, in from_pretrained
state_dict = load_state_dict(resolved_archive_file)
File "/data/sg/lib/python3.9/site-packages/transformers/modeling_utils.py", line 420, in load_state_dict
raise OSError(
OSError: You seem to have cloned a repository without having git-lfs installed. Please install git-lfs and run git lfs install followed by git lfs pull in the folder you cloned.

模型文件咋放啊？
ls model/
chatglm-6b  LexiLaw_finetune  readme.md

python inference_ptuning.py
AttributeError: 'ChatGLMModel' object has no attribute 'prefix_encoder'


WIP: Emoticon Translation In Blockquotes (Issue #17789)

#17789
Testing plan:
GIFs or screenshots:

Thanks for working on this.  Any change like this needs tests; I also don't understand how this strategy could work as written, since it mutates the content with spurious characters?  Finally, please don't create merge commits.
Check out our GitHub guide and commit guidelines for more details.

It was not a working solution, but part of it. I apologize for the pre-mature pull request.


Simplify how Prefect tasks are defined in Dataset class

This should get us closer to resolving #113

I'm going to close this in favor of lumping it with #120, because it's a small change and would likely require a merge anyways


Memory Regression in Latest Ubuntu

Description
We are using ubuntu-latest to run a docker container. In the container, we use sbt to build and test our Scala project.
Starting ~2 days ago, our tests stop with an EOFException and exit code 137 hinting at not sufficient heap memory.
The docker container has not changed and rerunning workflows that succeeded 3-4 days ago now fail with the same error.
We suspect that this memory issue is caused by a regression in the virtual environment 20210718.1 for ubuntu-20.04. One of the last successful runs was using virtual environment 20210712.0.
First of all, is there a way to explicitly set the virtual environment such that I can test that the issues are indeed caused by the virtual environment?
Are there any memory related settings that have changed?
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019

Image version and build link
20210718.1
Failing run
Is it regression?
yes, 20210712.0 successful run
Expected behavior
sbt test runs to completion without memory issue an exception
Actual behavior
sbt test fails because of not enough memory
Repro steps
Rerun the failing run above

Hi, @ArquintL
Could you please check recommendations from the https://www.programmersought.com/article/47811098599/ article ?
-Xms2048m -Xmx2048m -Xmn512m

@al-cheb Thanks a lot for the recommendation. I've tried it several combinations of specifying these parameters. The error looks now slightly differently but the tests are still killed with error 137 as can be seen for example here. Is there anything that has changed in the environment?

@ArquintL, Could you please try to rerun failed build, because we have rolled back a recent configuration change- https://github.com/actions/virtual-environments/issues/3805#issuecomment-887884901?

@ArquintL  the tests start to pass https://github.com/akv-platform/gobra/runs/3186048348?check_suite_focus=true

@al-cheb @dsame thanks a lot for your help! The tests indeed start to work again.
In the meantime, I've added logging of the RAM consumption of Java to our CI and the results are impressive:
Before the rollback

executing tests with sbt test: 5.7GB and exit code 137 run
executing tests directly with java: 3.6GB run

After the rollback

executing tests with sbt test: 2.4GB run
executing tests directly with java: 2.39GB run

One question though: Is it possible that the problematic virtual environment and the rolled back one both have version 20210718.1?

@ArquintL, Yep, it should be the same version.
https://github.com/viperproject/gobra/runs/3181827928 - Ok

https://github.com/viperproject/gobra/runs/3182822991 - Failed

@al-cheb I've seen that but was just wondering how I can differentiate before rollback vs after rollback

@al-cheb I've seen that but was just wondering how I can differentiate before rollback vs after rollback

In that particular case we reverted internal changes. I don't think it's easy to differentiate it.

Hello @ArquintL. As @al-cheb mentioned, it was our internal issues and they were reverted. I'm closing the issue, because you should not experience this issue anymore.


Add pylint settings for vscode

Proposed Changes
Get pylint working in vscode and add the necessary settings to nautobot.code-workspace. Ideally make it work with the docker container and poetry local development environments. Enabling the linter works for doing basic linting but some extra configuration is going to be required to get the pylint-django plugin working so that it can recognize GenericForeignKeys in the django models.

Justification
Improved developer experience. See pylint errors as soon as the file is saved instead of just before a commit.

Commenting to link this to #2292 - great idea!


luci-theme-openwrt-2020: fix anchor decorations

add the decorations for when the links are included in the field descriptions

Thanks @dibdot!

@stangri I've tested the change on my current snapshot. Could you please change your PR as follows:
.cbi-map-descr a,
.cbi-value-description a {
  color: var(--main-bright-color);
  text-decoration: none;
  outline: 0;
}
.cbi-map-descr a:hover,
.cbi-value-description a:hover {
  color: var(--main-bright-color);
  text-decoration: underline;
  outline: 0;
}

The '.cbi-map-descr' is needed to fix the anchor decoration for adblock, banIP and travelmate as well ... :-)  and it seems  that both inherit lines are needless.
Thanks for your contribution!

@stokito, not sure what you suggested as a fix, but since you thumbed up @dibdot's suggestion I'm guessing you're OK with that.
Feel free to merge and then I'll cherry-pick the commit and merge it to stable.

@stokito ping?

Sure, I'm ok.
@bryanforbes you are added the theme but not mentioned as a maintaner for theme.
Could you become a maintainer for it?
If yes, then please approve the PR

@stokito I didn't add this theme. I think I submitted a fix a long time ago, but I didn't add the theme.


[NOMRG] Test skipping CI Smac

As you can see from the CI smac was skipped @vmoens

But it has the label so it shouldn't


[JAST-019] Microservice architecture

As a developer, I'd like to split the different components (syntax tree, services, gui, etc.) into different projects.

Done


Player entities should be displayed last

Makes them be on top, which is desirable

std::partition() with !entity->is_player() maybe?

Closed via https://github.com/Skorezore/Gaem/commit/ac0529dd9ca05bfb5590817c4ec78fb79e642ab6


feat: use tailwind and style

Signed-off-by: vgarciavalen<EMAIL_ADDRESS>Issue number of the reported bug or feature request: #
Describe your changes

Remove styled components
Setup tailwind
style

Testing performed

Jest passing
Manual test using GCP

Additional context

Switched to tailwind to iterate faster

@Abhilasha3112 and @dakotalillie I've addressed the PR comments and squashed merged. Can you all add your reviews again?


Add Kurdish (Sorani)

I've added Kurdish (Sorani) to the "lang" folder and "README.md" file.

No problem


port is required when set the enabled to false.

Details
What steps did you take and what happened:
app-template:
  service:
    main:
      controller: main
      ports:
        http:
          enabled: false

What did you expect to happen:

the port only required when enabled = true

Hey, thanks for raising the issue. This is not a bug, and is very much by design. As of v3, even a disabled entity (whether it's a service, port, controller, etc) requires a valid configuration, otherwise there is no need to even include it in the values YAML.

That's more reasonable for that. Thanks for you work.


Upgrade node-libxml

Supercedes #1625.

This keeps failing on Heroku - it cannot find the install apparently
       npm ERR! gyp: binding.gyp not found (cwd: /tmp/build_ef149a0c/node_modules/node-libxml) while trying to load binding.gyp

Locally everything seems to work fine so I'm not sure what's happening here.

I'm going to merge this because it seems like this is a heroku issue.


Don't ignore sql test query exception, add it as cause exception

provided
If test running with info logging level real sql exception is hidden. In my case it was "ORA-01882: timezone region not found". So "Container is started, but cannot be accessed by (JDBC URL: %s), please check container logs" is misleading in my case. Additional configuration was required on a client side. From my point of view having real sql exception can be helpful for solving issues.


Feature | Adds function to flush messages

Fixes #6
Should consider if flushing without closing would also be useful


chip-tool commissioning error: Failed to verify peer's MAC (CON-676)

The device is already commissioned and part of an Apple fabric, my understanding is that I can pair this same device using chip-tool so it becomes part of a second fabric. When doing so I get the message "Failed to verify peer's MAC" in chip-tool console.
chip_tool_commissioning_issue.txt
device_commissioning_issue.txt

@albrownwood Did you open the commissioning window using Apple Home? If yes, it would have generated a new QR code/manual code (enhanced commissioning). You can then use the following command from chip-tool:
chip-tool pairing code <new-node-id> <code-from-Apple-Home>

@dhrishi yes the commissioning window was opened with Apple home, pairing was successful with your indications, thanks!

hello @albrownwood ,
which release of esp-IDF are you using because i get error when i try to commission the device over wifi?
your help will be appreciated.


[Improvement][Test] Remove dependency of powermock

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

As stated in https://github.com/apache/dolphinscheduler/issues/10976#issuecomment-1207427898, Powermock does not and will not support jUnit 5. We need to remove the dependency on Powermock.
One possible way is to use a high version (greater than 3.4.0) of Mockito to mock static method, see: https://wttech.blog/blog/2020/mocking-static-methods-made-possible-in-mockito-3.4.0/

[ ] Furthermore, we could use Spotless to block the usage of Powermock. #11637

Sub-Tasks
I have split the task into six sub-tasks by workload:

[ ] Remove Powermock in dolphinscheduler-api
[ ] Remove Powermock in dolphinscheduler-datasource-plugin
[ ] Remove Powermock in dolphinscheduler-master #12124
[ ] Remove Powermock in dolphinscheduler-common, dolphinscheduler-alert, dolphinscheduler-log-server, dolphinscheduler-spi, dolphinscheduler-worker
[ ] Remove Powermock in dolphinscheduler-service, dolphinscheduler-server
[ ] Remove Powermock in dolphinscheduler-task-plugin

Examples
Currently we have several typical scenarios where we heavily rely on Powermock

Mock static class
Suppress Static Initialization Error
Access private method in UT
... (to be continued)

I will give some examples and corresponding solutions for these scenarios respectively as follows:

Add mockito-inline and use Mockito.mockStatic as an alternative.
Simply mocking some special static classes with Mockito.mockStatic will cause Static Initialization Error. For these special cases, we need to refactor the origin class with some tricks to decouple the usage of static method.
Change private method to protected

For examples, see: #11588
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

 However, anyone interested and willing to help are very welcome to pick those sub-tasks. To help with this issue, just create an issue for the sub-task you want to pick and comment with the issue link below.

Are you fixing the dolphinscheduler-common module @EricGao888

Are you fixing the dolphinscheduler-common module @EricGao888

Yes, I will take care of the rest ones.


Status for all used services in a failed job report to determine which service failed

Situation
When receiving a report of a failed job, I don't know which service led to the failure.
Wanted
Failed job reports could include a list of all used services with a status for each. (success or failure)

@AndreasScheumann thanks for creating the issue.


Undefined error on headless browsers

I have no idea if it is to be expected, but PhantomJS and Webkit crash with:

The pad function and the way it is called look pretty harmless and the error is Chinese to me :confetti_ball:
Happened with Capybara->Poltergeist->PhantomJS 2.0.0
  applyMiddleware(thunk, createLogger({
    predicate: (getState, action) => (process.env.NODE_ENV === 'development') // do not use with headless browsers
  }))

solved the issue for me of course.

have you tried jsdom?

No. But same thing with capybara-WebKit (QT)

OK, it looks like PhantomJS 2.0 doesn't support repeat function, because the version of WebKit used doesn't support it. According to this StackOverflow answer WebKit 538.1 used in PhantomJS 2.0 was released in November of 2013 (that's quite some time ago).

@bbenezech You could try to use polyfill (e.g., the one included in Babel) for this function and probably a few other functions as well.

Ok, thanks a lot, I ended up ditching babel-runtime and importing babel/polyfill instead.
For future ref.: http://stackoverflow.com/questions/31781756/is-there-any-practical-difference-between-using-babel-runtime-and-the-babel-poly


External ref resolution does not parse keywords at the mo!

External refs do not parse any keywords.
e.g.,
[user.id|name: {something}]
In the above example {something} will not be resolved even if it exists.
The process of parsing keywords probably needs changing altogether. We have the ability to parse keywords in strings - maybe we can just parse the entire string as it comes in and then do operations on it. Should resolve all low-level issues relating to keyword resolution.

Resolved in https://github.com/forceedge01/behat-sql-extension/pull/51


💥 [DevOps] Enforce 60% coverage (admin)

💥 DevOps ticket

Enforce 60% coverage (admin)

We approch the 70%


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hotfix/zendesk integration

Codecov Report

Merging #225 (a7bfffa) into main (286d5ac) will increase coverage by 0.30%.
The diff coverage is 88.73%.

@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
+ Coverage   75.43%   75.73%   +0.30%
==========================================
  Files         191      192       +1
  Lines        7661     7732      +71
==========================================
+ Hits         5779     5856      +77
+ Misses       1882     1876       -6

Impacted Files
Coverage Δ

core/utils/tests/test_lsd.py
88.57% <88.57%> (ø)

core/utils/lsd.py
84.61% <100.00%> (+56.61%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Affected symbol position

The symbol showing a sample is affected can be behind the allele counts.

You can replicate this on the demo data by just going to the data selection and making everyone affected.

To test, run platinum demo trio and toggle on affected for Mother and Father in Files dialog.  Click on a variant.  Now there should be room for the allele bars and the affected glyph.

@tonydisera are these changes on stage.gene?
I tested there and toggled mom and dad affected status as on

But I don't see an affected status symbol for dad

I see this as working


Re-adopt staging branch release strategy

Do this ticket near the end of April, when the product will start actually being used.
In order to have a more stable app, instead of having deployments triggered by merging feature branches into main, either

Have a production branch that we need to merge main into that manages deployments.
Merge feature branches into a dev branch first and periodically merge dev into main.

In either case, the staging branch should be tested extensively and possibly even deployed to a test environment. Regular deployments could correspond to a MINOR change in the MAJOR.MINOR.PATCH pattern of semantic versioning, important deployments could correspond to a MAJOR change, and patches correspond to PATCH.

I would also like to hear any input from @kamadorueda @racherene on this :slightly_smiling_face:

I agree you need to have a development deployment where you can manually test changes, but introducing semver, deployments that happen selectively, and the manual process for managing releases, is not the only approach to ensuring stability and quality, only covers part of the software development life cycle, and requires human intervention (that can be partially automated)
My opinion on this is around the lines of the DevOps ways. This means: having a strong CI/CD that automates testing as much as possible, always deploying ephemeral environments per every branch before main automatically so that you can manually test what cannot be tested by the CI/CD before merging the pull request, and at this point just merge to main, deploy to main automatically, and learn from the instrumentation, logs, monitoring, and alerts that your services (back and front) have included. If a bug escapes the process, just improve the process to prevent it from happening again.
But as always, it's a trade-off, and you need to evaluate each alternative's pros and cons individually without bias. In my experience, DevOps is an upfront cost, but after that, it's a very well-oiled machine that automates as much as possible with the least amount of human intervention. Manual processes do not scale well
Since I also use a SPA for the frontend and lambda for the backend, I can show you how I architected this process at the company I work for, but essentially is what I mentioned before: there is a strong CI/CD, every PR creates ephemeral environments (backend,db,frontend,everything) like $branch.app.holdings.io (e.g. https://kamadoruedaonboarding.app.holdings.io/) automatically, and then everything else is main (e.g. https://app.holdings.io/), and there are logs, monitoring, backups, alerts to catch the bugs that escape all of this

always deploying ephemeral environments per every branch before main automatically

I know that AWS Amplify frontend can also do this with pretty straightforward settings, deploying to something like pr-237-change-color-scheme.lifeline.techstartucalgary.com for all prs and branches, and admins can limit access to internal with credentials.

Regarding the DevOps, it's really ideal to have it than having production branch. But there are many works to do from us to make it to the ground, just as Kevin mentioned, the upfront costs. I think there are two big questions we need to answer: 1) whether we want to take that upfront cost, and 2) how are we making sure to have DevOps done right.
First It's fair to say that all deploy process are people centred, and lifeline team is currently composed by a bunch of students mainly focusing on studying or interning, and have limited or unstable time and effort commitment to such a side-project. There seems to be a big uncertainty in people, it's nothing like a formal job or having a structured management in a workplace. (And I may say things are all a bit messy for lifeline development, but I think it's ffineee cuz it's a side-project anyways with looser organization u know) So I wonder, with the amount of uncertainty, if DevOps is going to function properly, what is the tolerance in it. So far I have personally seen examples of DevOps done good job in workplaces with commercial products, but obviously lifeline is simply a whole different situation from that, so maybe we want to see examples of similar projects like lifeline running DevOps to give us more confidence, and possibly learn from them.
(And thinking for the future, it's likely gonna be an open-source and community-driven-ly project from what we see today)
Second, how much work would look like for upfront cost. Comparatively speaking, it's probably fine if it's about 10-20% of work compare to what we have done so far for the project. But if the work is over, say 40-50%, then that implies that we don't actually have that many work done at all and we may be risking to take time and effort to acquire a too strong deploy process than we actually demand, and I don't believe this is a wise choice. We should evaluate that definitely.
And the last, we simply barely have experience with DevOps. It's the fact that there's not a single "right" definition for DevOps, teams/companies all done it differently for fitting their demands. It's like a reading question in essay class that there's no right answer to it. What if we cannot find our way of DevOps, what if we keep trying and never succeeding, what if we never done it right for us...? I don't know how this should go...
Hope I expressed myself correctly!

For now I'm just gonna root for the idea of having a production branch, before we draw any conclusion on DevOps (or never). Because we are so unsure of DevOps, the regular "making releases" just sounds a really safe play that it can never go wrong, and it's easier to do with both deploy and our lives. :)
We can have semver, ephemeral env, CI/CD all that stuff. But, just not deploy right from main. Because we YOLO things, and I know that I am always just reviewing PRs casually and give approval as long as it's not ridiculous, and find out bugs afterwards like always. LOL.


Change DbContext to IObjectContextAdapter in InteractiveViews.SetViewCacheFactory

Hello, (not an issue - don't know where to say it)
Thank you for library.
Please change DbContext to IObjectContextAdapter in InteractiveViews.SetViewCacheFactory
public static void SetViewCacheFactory(IObjectContextAdapter context, DbMappingViewCacheFactory viewCacheFactory)
I my case I have to fork just for that, as we use custom dbContext implementation.
Thank you.

What do you mean by “custom dbContext implementation”?
On Mon, Oct 21, 2019 at 2:58 AM Alexey Boyko<EMAIL_ADDRESS>wrote:

Hello, (not an issue - don't know where to say it)
Thank you for library.
Please change DbContext to IObjectContextAdapter in
InteractiveViews.SetViewCacheFactory
public static void SetViewCacheFactory(IObjectContextAdapter context,
DbMappingViewCacheFactory viewCacheFactory)
I my case I have to fork just for that, as we use custom dbContext
implementation.
Thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/moozzyk/EFInteractiveViews/issues/12?email_source=notifications&email_token=AAK7JJB4IGRL7OPEZVQBDKDQPV4NZA5CNFSM4JC27NJ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HTD2JTA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK7JJGLAI7YIFKGKK7NLKLQPV4NZANCNFSM4JC27NJQ
.

Regardless - you can just pass ObjectContext using the other overload:
https://github.com/moozzyk/EFInteractiveViews/blob/8cda367cf0b722cc8e89a1bff917fe918b1f5728/src/InteractivePreGeneratedViews/InteractiveViews.cs#L64

What do you mean by “custom dbContext implementation”?
Our context do not inherit DbContext.

you can just pass ObjectContext using the other overload
Thank you


Occasional test timeout on Windows

[ ] //test/builtin:bestmatch_test
[ ] //test/builtin:ujoin_test
[ ] //test/builtin:waitdo_test
[ ] //test:args_test
[ ] //test:args2_test
[x] //test:familiar_test -- Disabled on Windows. WAI.
[x] //test/builtin:xpipe_nested_test -- Seems to be fixed.

The timeouts can be caused by either not closing a file descriptor in the parent process or crashing with a double-close.

//test:familiar_test is flaky too, but perhaps for a different reason (not a timeout).

https://github.com/grencez/lace/commit/b30cfabde3eeba82834d67ffa9f46087a27fee90 disabled the test on Windows as it always fails now...

I'm also disabling //test:familiar_test because grep doesn't usually exist on Windows. That's a different issue.
The real issue is: Windows is more strict about file descriptors. It crashes when one is closed twice (I suspect that the second one being an fclose() helps). Hopefully this is mostly caused by lose_sysCx() closing stdout. Good time to remove it!

https://github.com/grencez/lace/commit/a9efa43906b45aa23a8d6daacdd4e240c762ca67 added more tests, now some of those time out fairly consistently (//test/builtin:bestmatch_test￼,￼ //test/builtin:sshall_test, //test/builtin:ujoin_test, and //test/builtin:waitdo_test). At least //test/builtin:xpipe_nested_test seems to be fixed.

Oh wow that may have done it. 2 green builds in a row!

I should also fix some undefined behavior in the pthread compositional functions. They should probably move to wrapped versions in the lace_compat_lib. There's just a little too much to patch in a header:

Different calling conventions (cdecl vs stdcall).
Thread function shouldn't have different return type than expected (https://stackoverflow.com/a/559671/5039395).

In the end, it made sense to keep this in lace_posix_thread.h.


Upgrade CI to use ROCm2.8 userbits packages

This PR is part of the process to move TF-ROCM CI builds to use ROCm2.8 as the base.
Additional docker run options --cap-add=SYS_PTRACE --security-opt seccomp=unconfined is required by the THUNK library, for details, refer to the following notes:
https://github.com/RadeonOpenCompute/ROCm#docker-container-environment-variable-setting

jenkins: retest this please


Cannot run demo

Hello,
I'm pretty new to ktor and found this repository and blog post very helpful, but I can't get the authenticated routes work. I checked out the repo, added my service account json, built and ran the project without issue. However, when I GET http://<IP_ADDRESS>:8080/authenticated in Postman (using Bearer Token auth) this line FirebaseAuth.getInstance().verifyIdToken(authHeader.blob) throws an error with this stack trace.
com.google.firebase.auth.FirebaseAuthException: Unknown error while making a remote service call: Connection refused (Connection refused)
        at com.google.firebase.auth.internal.AuthErrorHandler.createException(AuthErrorHandler.java:122)
        at com.google.firebase.auth.internal.AuthErrorHandler.createException(AuthErrorHandler.java:35)
        at com.google.firebase.internal.AbstractHttpErrorHandler.handleIOException(AbstractHttpErrorHandler.java:63)
        at com.google.firebase.internal.ErrorHandlingHttpClient.send(ErrorHandlingHttpClient.java:110)
        at com.google.firebase.auth.internal.AuthHttpClient.sendRequest(AuthHttpClient.java:75)
        at com.google.firebase.auth.FirebaseUserManager.lookupUserAccount(FirebaseUserManager.java:238)
        at com.google.firebase.auth.FirebaseUserManager.getUserById(FirebaseUserManager.java:121)
        at com.google.firebase.auth.RevocationCheckDecorator.validateDisabledOrRevoked(RevocationCheckDecorator.java:59)
        at com.google.firebase.auth.RevocationCheckDecorator.verifyToken(RevocationCheckDecorator.java:54)
        at com.google.firebase.auth.AbstractFirebaseAuth$3.execute(AbstractFirebaseAuth.java:307)
        at com.google.firebase.auth.AbstractFirebaseAuth$3.execute(AbstractFirebaseAuth.java:304)
        at com.google.firebase.internal.CallableOperation.call(CallableOperation.java:36)
        at com.google.firebase.auth.AbstractFirebaseAuth.verifyIdToken(AbstractFirebaseAuth.java:269)
        at com.google.firebase.auth.AbstractFirebaseAuth.verifyIdToken(AbstractFirebaseAuth.java:241)
        at com.plusmobileapps.firebaseadminsample.firebase.FirebaseAuthKt$verifyFirebaseIdToken$token$1.invokeSuspend(FirebaseAuth.kt:80)
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
        at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
        at kotlinx.coroutines.internal.LimitedDispatcher.run(LimitedDispatcher.kt:42)
        at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:95)
        at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:570)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:749)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)
Caused by: java.net.ConnectException: Connection refused (Connection refused)
        at java.base/java.net.PlainSocketImpl.socketConnect(Native Method)
        at java.base/java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:412)
        at java.base/java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:255)
        at java.base/java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:237)
        at java.base/java.net.Socket.connect(Socket.java:609)
        at java.base/sun.net.NetworkClient.doConnect(NetworkClient.java:177)
        at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:508)
        at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:603)
        at java.base/sun.net.www.http.HttpClient.<init>(HttpClient.java:276)
        at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:375)
        at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:396)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient(HttpURLConnection.java:1253)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect0(HttpURLConnection.java:1187)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect(HttpURLConnection.java:1081)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.connect(HttpURLConnection.java:1015)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getOutputStream0(HttpURLConnection.java:1367)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getOutputStream(HttpURLConnection.java:1342)
        at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:113)
        at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:84)
        at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:1012)
        at com.google.firebase.internal.ErrorHandlingHttpClient.send(ErrorHandlingHttpClient.java:96)
        ... 19 more

In case it's useful for the Authentication header I'm getting my idToken from https://identitytoolkit.googleapis.com/v1/accounts:signInWithPassword?key={{FIREBASE_API_KEY}} and passing in my test account credentials. And using the returned idToken just as the blog post decsribes.

It might help to see the whole project you have and the curl request you made to better help you out?
Not sure if you have seen this page, but seems like based off that error your app might only be connected to your local network? Hard to know though without seeing what code you have written.

Thank you for the quick response! To your first question:

It might help to see the whole project you have and the curl request you made to better help you out?

The error I'm seeing is in this project. I git cloned this project, and followed the setup section of your README, and then  ./gradlew run the project exactly as you describe. I made no other modifications to the code. The same for the curl requests. I initially tried with Postman but I did re-try with the curl request from your blog post.

your app might only be connected to your local network

Is there something that you did to configure you project so that this wasn't the case? I haven't seen anything in the docs for the admin sdk that describe this behavior or what to do about it. Are there environment vars that need to be set or something that might not have been mentioned in the set up section of your README? I certainly can't think of any reason my network would prevent Firebase making network calls.

I finally figured out what the issue was. I had previously enabled Firebase Auth Emulator by setting FIREBASE_AUTH_EMULATOR_HOST and although I had tried unsetting it with the unset command in zsh I needed to delete this line from my ~/.zshrc. Then after restarting IntelliJ it finally worked.
export FIREBASE_AUTH_EMULATOR_HOST="localhost:9099"


Doesn't work in Microsoft Edge

It shows a gray question mark sign in the icon. Logging out of Facebook then logging back in doesn't seem to fix the issue. In Chrome and Vivaldi works well. Please fix it.

Hello, it should be fixed in the next release (Pending review).

Related to #35.

The Edge version of the extension is now available at https://microsoftedge.microsoft.com/addons/detail/fnotifier/jkpbopolkbhegaabkoljoofcfingihlp.


test_translations not working locally?

Describe the bug
When trying to run the instruction as stated here:
selfdrive/ui/tests/create_test_translations.sh && selfdrive/ui/tests/test_translations

it will not work:

Or is it a typo and is test_translations.py meant with the second command?
OS Version
Ubuntu 20.04
openpilot version or commit
No response
Additional info
No response

You need to build with the --test flag, e.g. scons -j8 --test.


Background image gets cropped out when screen size is varied

Thank you for taking the time to file a Bug report.
Describe the bug
Background image in the header is not responsive. Also the header button overflows the header section.
To Reproduce
Steps to reproduce the behavior:

Go to journal_policy_tracker_app
Right-click the mouse button and then click on 'inspect '
Set the dimentions to 'Responisve'  and decrease the window size to 800px.
See error

Expected behavior
The background image in the header should not get cropped when changing the screen size and the button should be inside of the header section.
Screenshots

Desktop (please complete the following information):

OS: 20.04
Browser Chrome
Version 99

I have already sent a pr for this issue. Hope you can take some new issue.

HI, please wait for the maintainers to assign you an issue before you make a PR for it. Sometimes, we might not want the a particular change proposed in an issue. In such a case, we don't want the contributor's work to go in vain.

Hi @padmajabhol , thanks for your comments. I'll take care of this next time :)


Get rid of registration0 job

Search service should point to the same registration base URL as index.json. Today registration1 and registration0 are both in the same storage account so it does not provide anything to the disaster recovery story,
index.json points to:

https://api.nuget.org/v3/registration1/
https://api.nuget.org/v3/registration1-gz/

search services points to:

https://api.nuget.org/v3/registration0/

Wait for https://github.com/NuGet/NuGetGallery/issues/3748 to be completed.
Verify blob access for registration0 is limited to the catalog2registration job.

Note that callers may have hard-coded registration0. We should wait 3 days after all links to it are removed before moving on.

Stop the catalog2registration job for registration0 on the prod VM.
Disable deployment of catalog2registration job for registration0.
Stop catalog2registration job for registration0 on the prod VM.
Remove the job cmd file from NuGet.Services.Metadata for this job.
Remove the unused configuration from Octopus.

/cc @scottbommarito @skofman1 @cristinamanum

How will callers of registration0 know that the container has been deprecated? It could be very confusing for those who have hardcoded it if they start finding discrepancies between what they see in registration and what exists on nuget.org.
Do we want to consider putting a redirect on the CDN to a "registration0 has been deprecated" page? Do we want it to redirect to registration1?

Do we have the same situation on dev&int?

@scottbommarito:

How will callers of registration0 know that the container has been deprecated? It could be very confusing for those who have hardcoded it if they start finding discrepancies between what they see in registration and what exists on nuget.org.

This is breaking contract. v3/index.json is the source of truth on registration base URLs. That being said our client does cache the index.json for 40 minutes. We will wait 3 days for callers to pick up on the new index.json. If someone hard-coded the registration0 location, they will encounter stale data after we stop the registration0 job and encounter 404's when we remove registration0 (not planned).

Do we want to consider putting a redirect on the CDN to a "registration0 has been deprecated" page? Do we want it to redirect to registration1?

I don't think this is worth the effort. Clients should be following index.json. We can re-evaluate this if we observe a high level of traffic on registration0 after we switch the search service over.
@skofman1:

Do we have the same situation on dev&int?

Yes. We will prove the process there first.

Access to registration0 on PROD is limited to jobs and around external 6735 hits over a course of 24 hours (May 1, 2017, UTC). registration1 have over 16,000,000 hits over the same 24 hours.
The set of external hits to registration0 have the following characteristics:

None have the NuGet client user agent.
6730 have user agents that suggest 3rd party scripting:

3397 - no user agent
3261 - python-requests
82 - RestSharp

The 5 hits are browser or bingbot hits.
There are only 8 distinct IPs with pattern X.X.X.0, suggesting a low number of organizations involved.

For this reason, I think we can safely deprecate registration0. That is, we can stop updating it with new packages.

/cc @skofman1, @dtivel, @chenriksson, @cristinamanum: thoughts?

Agreed.

I have disabled the registration0 job. The remaining work is:

[ ] Remove the registration0 service from the PROD jobs machine.
[ ] Remove the job cmd file from NuGet.Services.Metadata for this job.
[ ] Remove the unused configuration from Octopus.


Base Dataset and Dataset Registry

This PR provides an implementation of BaseDataset as described in https://github.com/luxonis/luxonis-ml/issues/69.
It also includes some of the following changes:

The dataset.py file is now broken into its own datasets module for better organization
A change to LuxonisFileSystem.get_dir is made so it can either be passed a list of remote paths or a path to a directory. (Similar to the way LuxonisFileSystem.put_dir is implemented)
Removal of self.online attribute from LuxonisDataset
Support for LuxonisParser to use a custom dataset plugin

Python 3.8 and 3.9 tests are failing. I only tested 3.10 locally - will figure out what the issue is.


[TACHYON-1649]Add the Chinese version of documentation page 'Running-Spark-on-Tachyon'

https://tachyon.atlassian.net/browse/TACHYON-1649

Can one of the admins verify this patch?

Jenkins, add to whitelist.

Jenkins, ok to test.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8348/
Test PASSed.

LGTM, thanks for the translation!

LGTM. Thanks!


Go beyond the limit (100) of returned folders

[x] I have checked that the CLI documentation and API documentation doesn't solve my issue

Description of the Issue
I did a simple "box folders:get xxx" where xxx is some Folder ID.
It happens that I have one of this folder with many subfolders (more than 100). The problem is that the box folders:get command only returns the first 100 subfolders. To be more precise, in the Json response, inside the "item_collection" hash-table, I have an attribute 'limit' which equals 100 and another one 'total_count' which equals 170. If I count the number of element in the 'entries' array, I have indeed 100 of them.
I cannot figure out how I can get the 70 subfolders missing. I saw nothing that might set the limit to a higher value and I did not saw anything neither that could retrieve the remaining entries in a second run. I need the full list (at least the folder ids), not just the first 100.
Versions Used
@box/cli/2.3.0 win32-x64 node-v11.13.0
Windows-Server 2016 version 1607
Steps to Reproduce
Put more than 100 subfolders in a single folder and try to get them all by "box folders:get"
Error Message, Including Stack Trace
No error message

@larhape The box folders:get command is intended to get information about a single folder; the item_collection property of the folder is only a "preview" of the items in that folder.  To retrieve the full set of items in the folder, use the box folders:items command instead.

Aouchh.  Yes, of course !
Sorry for the disturbance, and many thanks for the prompt answer : it definitely solves my problem

@larhape No worries — there are a lot of different commands in the CLI!  I'm glad it's working for you now 🙂


Attach Security Group to Launch Config

This is needed for a common bastion pattern. The bastion needs a specific security group attached to it such that autoscaling groups that are created after the bastion can reference the security group on the bastion instance (asg). Alternatively we could have exported the security group from the ASG. I chose this pattern because I thought it was the more flexible solution, but also avoided a more complex export function. However, if there is a reason we should pivot I'll be happy to change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

@eladb - fixed up the comments. Is the pattern on conventional commits, that if I put in the commit correctly to start the PR and the comment doesn't need updating you will squash the commits on merge and only keep my conventional commit?

Addresses item 2. in https://github.com/awslabs/aws-cdk/issues/623#issue-353686931


Expose pipes_as_concat in the database URL

The option to set pipes_as_concat is currently not exposed in the MySQL database URL parsing function.
Because some sqlx features like cargo build support and cargo sqlx can only be configured through a database URL this makes it impossible to work with some databases that don't support PIPES_AS_CONCAT.
This adds the ability to set the flag through the database URL, which makes it possible to work with those databases.

I'm still of the same persuasion as #2135: I don't think the database URL is the right place for this setting to live, as it's less a property of a given connection and more of a property of the code that's going to be using the connection and whether it assumes pipes mean concat or not.
We really need to be able to pass this configuration directly to the macros somehow. I'm not going to make any promises I can't keep, but I'd certainly like to work on a solution for this for the next release.
There's some existing discussion in #121.


Update RDS sync to include Authenticating Proxy PG

Relies on https://github.com/alphagov/authenticating-proxy/pull/350 and related infrastructure PRs.
Once Authenticating proxy is moved to RDS, this sets up the RDS sync for the new authenticating proxy DB.

You're absolutely right, we did need a full name there. Good catch! Fixed now.


Packet::Disconnect is not implemented

rumqtt 67d1bb465b34bf8ccf2a058fbcb4cbb5ff919ceb
thread '<unnamed>' panicked at 'not yet implemented', /home/bbigras2/.cargo/git/checkouts/rumqtt-97ac1717aa98c207/67d1bb4/src/client/mqttstate.rs:71:18
note: Run with `RUST_BACKTRACE=1` for a backtrace.

The packet is Packet::Disconnect. Because I call client.disconnect().

Implemented in #135

Thanks @denizs. Appreciate your help. Keep contributing


deal with issue 200

issue 200 is described here: https://github.com/geekcomputers/Python/issues/200

After modification, check_file.py can deal with non-exist files and non-readable files properly.


Support for Velocity Set-Points with Crazyflie Flow Deck

I was wondering, since bitcraze has now released their optical flow expansion deck for Crazyflie 2.0, it would be nice to do higher level control (with body-frame velocity setpoints, as opposed to attitude setpoints) in ROS, and to let the Crazyflie handle the velocity request to attitude loop internally. For example, the cmd_vel topic which is used by the crazyflie_server could be reinterpreted to be a velocity request, whereas currently it is an attitude request.
Does this functionality currently exist in crazyflie_ros? If not, I think it would be an excellent addition. For many people, it would be a more natural starting point for autonomy and off-board control of the Crazyflie.

This is not yet supported, but I have a student working on adding support and perhaps a python scripting layer to allow scripted behaviors without mocap.

Hello,
I just wanted to revisit this idea, now that I have some more time to hack on my crazyflie. Looking at crazyflie-lib-python, it appears that there two velocity set-point types which can be transmitted to the commander module via the CRTP protocol.
I'm speaking in particular about 'velocity_world_setpoint'(s) and 'hover_setpoint'(s). Both of these packets contain x and y velocity set-points, but not z velocity. Z velocity may not be controllable without some z distance feedback, but bodyframe velocity requests may be implementable by just sending the corresponding packet to the commander module.
I will try some basic scripted behavior through crazyflie_driver using the SendPacket service.

This is now available with the cmd_hover and cmd_position topics.

Hi Whoenig,
I am wondering if crazyflie_ros package support the Flow Deck now? If so, which demon can be launched with the Flow Deck equipped?
Thanks.

Yes, the flow deck is fully supported. There is nothing special to take care of from the ROS side - the flow deck will simply improve the onboard state estimation. You can, for example, execute waypoint following or trajectories without a motion capture this way.

Yes, that's possible. There is an example script here: https://github.com/whoenig/crazyflie_ros/blob/master/crazyflie_demo/scripts/Hover.py.
Another option is to use the high-level mode, which allows you to follow waypoints or execute trajectories.


Logging in astra-cli should use org.apache.log4j.Logger

Currently, the logging in astra-cli uses System.out.println. This should be updated to match the approach in astra-core, where we use org.apache.log4j.Logger.
An example of our logging in astra-core can be seen in the TypeReferenceRefactor
The logging that needs to be updated to match is in:

AstraChangeType
AstraMethodInvocation

i would like to work on this

Hi, thanks for your interest in Astra!
That would be great! Please take a look at our contribution policy, and
raise a pull request:
https://github.com/alfasoftware/astra/blob/main/CONTRIBUTING.md
On Mon, 15 Mar 2021 at 15:44, Alex @.***> wrote:

i would like to work on this
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alfasoftware/astra/issues/1#issuecomment-799524223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACP64E62MPWGHRVCGDADLO3TDYTPVANCNFSM4Y3LIQPQ
.

This has now been fixed by pull request #6. Thank you @AlexS968!
Closing the issue now.


Screenshot cannot render emoji

Relevant code is in automation/site-fetcher
https://github.com/wonderfulsoftware/webring-site-screenshots/blob/main/heyfirst.co_375w%402x.png

I guess many people face this problem. Let's see more information in this link. https://github.com/puppeteer/puppeteer/issues/4086

I guess many people face this problem. Let's see more information in this link. https://github.com/puppeteer/puppeteer/issues/4086

Thanks @aofdev for fixing this in #54!

Thanks @aofdev for fixing this in #54!

Awesome P'Aoffffffffffffffffffffffff @aofdev

Awesome P'Aoffffffffffffffffffffffff @aofdev