add unit-test for apis/opts/portbindings.go areas/test

Ⅰ. Describe what this PR did
add test cases for portbindings #1905
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 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.

Codecov Report

Merging #1951 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1951      +/-   ##
=========================================
+ Coverage   63.31%   63.4%   +0.09%
=========================================
  Files         200     200
  Lines       15522   15522
=========================================
+ Hits         9828    9842      +14
+ Misses       4455    4437      -18
- Partials     1239    1243       +4

Flag
Coverage Δ

#criv1alpha1test
33.81% <ø> (-0.05%)
:arrow_down:

#criv1alpha2test
34.31% <ø> (+0.03%)
:arrow_up:

#integrationtest
38.39% <ø> (-0.04%)
:arrow_down:

#unittest
20.99% <ø> (+0.13%)
:arrow_up:

Impacted Files
Coverage Δ

cri/v1alpha2/cri.go
66.31% <0%> (-0.18%)
:arrow_down:

daemon/mgr/container.go
53.85% <0%> (-0.16%)
:arrow_down:

apis/opts/portbindings.go
80.95% <0%> (+80.95%)
:arrow_up:

LGTM


Documentation: Optimised replication

Is it possible to have the replication events sent as a pool periodically (or delayed on event)? So that a node will not have to receive the same amount of traffic as the primary.
The reason I ask is it would be nice to deploy instances of eureka on very small vm's, I've had a tinker with the eureka client so that it round robins between nodes to split load, but if each node is going to forward the heartbeat immediately then our traffic on the primary node will be the same anyway.

After looking through source, I have found that eureka does effectively have a delayed event so updates are grouped before being replicated.
Is it possible to have this documented, as well as what the effects of reducing the number of workers for replication will have?

@Lovett1991 thanks for the issue, will look to add documentation on batched replication.


adding Quadrafuzz to sound causes speaker icon to appear on Chrome's tab

If I start with:
    var sound = new Pizzicato.Sound({
        source: 'file',
        options: { path: app.config.get('servers/static') + `/sounds/wrong.mp3` }
    });
    var quadrafuzz = new Pizzicato.Effects.Quadrafuzz({
        lowGain: 0.6,
        midLowGain: 0.8,
        midHighGain: 0.5,
        highGain: 0.6,
    });

then everything is fine, no sound is playing, and there is no speaker icon on the tab in my browser. But as soon as I follow up with:
sound.addEffect(quadrafuzz);

the speaker icon on the tab appears, even though no sound is playing right now.
I've tested with LowPassFilter instead of Quadrafuzz and the icon does not appear.

(my Chrome Version is 73.0.3683.103 (Oficial) (64-bit),
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.103 Safari/537.36 )


exposes Unboxer's init publically

My use case is that I need to go to <-> from JSON. Unboxer get's me from JSON wonderfully. I love it, I'd marry it.
During the to JSON part, however, I have to start filling up my model with more functionality which saddens me.
Personal preference, I like to keep my models really dumb. So I said, "hey! I know, I'll just pull all the to <-> from out of the model into a service that does the work and just initialize Unboxer for all it's swanky unbox<T> functionality." "Alas" I said, the initializer on Unboxer is private =( It doesn't have to be though! So I tried it out, maybe things blow up and I just don't know it.
Turns out, nothing blows up, and it appears as though convenience and flexibility rain from on high.
Here's an example of one of my serializer classes using this change:
public struct ActivitySerializer {

    public static func serialize(model: Activity) -> AnyObject{
        var data = [String: AnyObject]()

        data["obj_id"] = model.objId
        data["label"] = model.label

        return data
    }

    public static func deserialize(value: AnyObject) -> Activity? {
        guard let json = value as? [String: AnyObject] else {
            return nil
        }

        let unboxer = Unboxer(dictionary: json, context: nil) // Weather forecast: 100% chance of AWESOME!

        let model = Activity(
            objId: unboxer.unbox("obj_id"), // awe SNAP!
            label: unboxer.unbox("label") // awe SNAP again!
        )
        return model
    }
}

It's great! I don't even have to have my models extend Unboxable. I mean I can if I want, but for this particular case, I get to keep the knowledge away from my model all together and I love Unbox even more!

Hmmm, I see what happened. I edited Unbox.swift in sublime, which I have kill whitespace only lines, hence all the changes.
If this is no good for you, but the general idea of the PR is satisfactory, I can just resubmit it, editing the file in XCode so it keeps the extra spaces on those lines.

I don't even know if I would need to use SwiftyJSON anymore if that initializer were public. Especially because you already allow string dotted access: unbox("something.other") and 99.999% of the time it's just making an assignment, which you solved with unbox<T>.

Unbox actually now has a custom unboxing API (new in 1.3) which enables this very usecase without exposing the initializer of Unboxer. The reason for not exposing this initializer was discussed here: https://github.com/JohnSundell/Unbox/issues/18. Basically it voids the guarantee of a 1:1 mapping between a model and its unboxer that Unbox can currently assume, since an API user might (accidentially) use the same unboxer for multiple models, and thus making a single failure cause multiple ones.
Here is how you can use the custom unboxing API in your ActivitySerializer:
public struct ActivitySerializer {
    public static func serialize(model: Activity) -> AnyObject {
        var data = [String: AnyObject]()

        data["obj_id"] = model.objId
        data["label"] = model.label

        return data
    }

    public static func deserialize(value: AnyObject) -> Activity? {
        guard let json = value as? UnboxableDictionary else {
            return nil
        }

        return try? Unboxer.performCustomUnboxingWithDictionary(json, closure: {
            let unboxer = $0

            return Activity(
                objId: unboxer.unbox("obj_id"),
                label: unboxer.unbox("label")
            )
        })
    }
}

Also, have you checked out Wrap (https://github.com/johnsundell/wrap)? It's like Unbox - but for encoding JSON :)

Oh that's awesome I did not even see that! Ha! Nice work on that!!

@aventurella What I usually do is I implement both Unbox and Wrap for my models in an extension. That way they don't feel "heavier" and I don't have to introduce extra serialization objects :)


DNSCrypt client proxy service hang?

Just had an issue where I was disconnected while messaging with someone and pages wouldn't load, but a download that was going was fine, which had me confused until I tried a different browser that actually gave a useful error message referencing a DNS failure. So I tried loading SimpleDNSCrypt and it would show in the tray but it wouldn't open. After trying a second time, I went into services and stopped and restarted the DNSCrypt one, after which everything went back to normal. First time this has happened and absolutely no idea why. I dug through event viewer and didn't see anything around that time except the actual DNS query failures. Nothing related to the service itself having issues, though. So I'm not sure if anything can be done since there doesn't seem to be any info to provide, but wanted to bring it up just in case.

Just happened again, though this time I already had the interface running and it opened, so I stopped and started it through that.

Just happened again, TWICE in the past half hour or so. This time the service wasn't running, despite being set to restart automatically, nor could I start it via the app's interface, as doing so just kept failing and causing SimpleDNSCrypt to crash. It says a lot about something when it fails so often, in such a repeatable way, for so long that it becomes the first thought in troubleshooting. It's been 3.5 months since I first posted about this and there's been no response or fix. Apparently I'm wasting my time.


Is it possible to check whether group message can be sent or not?

Is there any api to check if the group permissions is set to only admin or if you are removed from the group or any other way to check that we can't send messages to particular group

Update version


Errors when db set to 'NONE'

I set:
defaultExport.DB_TYPE = DB_TYPES.NONE;
and when I run 'npm run dev' I get:
Attempted to use '(de)serialize User' but DB type 'NONE' doesn't support it
Attempted to use 'passport-local' but DB type 'NONE' doesn't support it
Attempted to use 'passport-google-oauth' but DB type 'NONE' doesn't support it
module.js:327
    throw err;
    ^

Error: Cannot find module './webpack.config.dev-client'

remove passportConfig(); from /server/index.js.
@choonkending we need to remove this when DB type is NONE.

Actually I still ended up getting the Error: Cannot find module './webpack.config.dev-client'
As it turns out, line 24 of server/index.js has the incorrect path for that file. It should be:
const webpackDevConfig = require('../webpack/webpack.config.dev-client');
Once I made that change, there was no need to remove passportConfig();

@jgentes Thanks for pointing out! I have fixed this here https://github.com/reactGo/reactGo/blob/master/server/index.js#L24.


cudatoolkit-dev installed within a conda-build requires conda package dependency

Issue: see the title, otherwise the conda-build fails with

ERROR:conda.core.link:An error occurred while installing package 'conda-forge::cudatoolkit-dev-10.0-2'.
ERROR conda.core.link:_execute(696): An error occurred while installing package 'conda-forge::cudatoolkit-dev-10.0-2'.
Rolling back transaction: ...working... done
Traceback (most recent call last):
  File "/home/pearu/miniconda3/bin/conda-build", line 11, in <module>
    sys.exit(main())
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 445, in main
    execute(sys.argv[1:])
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 436, in execute
    verify=args.verify, variants=args.variants)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/api.py", line 209, in build
    notest=notest, need_source_download=need_source_download, variants=variants)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/build.py", line 2343, in build_tree
    notest=notest,
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/build.py", line 1407, in build
    create_build_envs(top_level_pkg, notest)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/build.py", line 1247, in create_build_envs
    is_conda=m.name() == 'conda')
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda_build/environ.py", line 867, in create_env
    execute_actions(actions, index)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/common/io.py", line 88, in decorated
    return f(*args, **kwds)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/plan.py", line 321, in execute_actions
    execute_instructions(plan, index, verbose)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/plan.py", line 533, in execute_instructions
    cmd(state, arg)
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/instructions.py", line 73, in UNLINKLINKTRANSACTION_CMD
    unlink_link_transaction.execute()
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/core/link.py", line 247, in execute
    self._execute(tuple(concat(interleave(itervalues(self.prefix_action_groups)))))
  File "/home/pearu/miniconda3/lib/python3.7/site-packages/conda/core/link.py", line 712, in _execute
    rollback_excs,
conda.CondaMultiError: post-link script failed for package conda-forge::cudatoolkit-dev-10.0-2
location of failed script: /home/pearu/miniconda3/conda-bld/omniscidb-cuda_1566553013867/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/.cudatoolkit-dev-post-link.sh
==> script messages <==
<None>
==> script output <==
stdout:
stderr: Traceback (most recent call last):
  File "/home/pearu/miniconda3/conda-bld/omniscidb-cuda_1566553013867/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/cudatoolkit-dev-post-install.py", line 44, in <module>
    from conda.exports import download, hashsum_file
ModuleNotFoundError: No module named 'conda'

Details about  conda and system ( conda info ):

$ conda info
     active environment : base
    active env location : /home/pearu/miniconda3
            shell level : 1
       user config file : /home/pearu/.condarc
 populated config files : /home/pearu/.condarc
          conda version : 4.7.11
    conda-build version : 3.18.9
         python version : 3.7.3.final.0
       virtual packages : __cuda=10.1
       base environment : /home/pearu/miniconda3  (writable)
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/pearu/miniconda3/pkgs
                          /home/pearu/.conda/pkgs
       envs directories : /home/pearu/miniconda3/envs
                          /home/pearu/.conda/envs
               platform : linux-64
             user-agent : conda/4.7.11 requests/2.22.0 CPython/3.7.3 Linux/4.15.0-51-generic ubuntu/18.04.2 glibc/2.27
                UID:GID : 1001:1001
             netrc file : None
           offline mode : False

Fixed in #16


Make named parameter parsing use byte slices

As discussed in blackbeam/mysql_async#194, MySQL actually accepts
arbitrary byte slices, not just strings, as queries. This commit is the
first in a cross-repo, likely ongoing effort to update both mysql_common
and mysql_async to accept either byte slices or strings - that
distinction is planned on the mysql_async side, but for now this updates
the main entrypoint for queries in this crate, which is
pares_named_params, to take &[u8] instead of &str.
This PR is intended to go along with a similar PR in mysql_async to update it
to work with these new APIs

note that this is a breaking change.

Hi there,
I don't if this is the offending commit, but after upgrading to mysql_common to 0.29, all code which uses FromValue and ToValue for "chrono::NaiveDate" type fails miserably as below.  Please note that the "exact code" builds fine with mysql_common 0.28.2
CASE 1
error[E0277]: the trait bound chrono::NaiveDate: FromValue is not satisfied
--> src\eni_db.rs:210:27
|
210 |             ENI_WORKPACK_DATE: row.take("ENI_WORKPACK_DATE").unwrap(),
|                                    ^^^^ the trait FromValue is not implemented for chrono::NaiveDate
|
= note: required because of the requirements on the impl of FromValue for std::option::Option<chrono::NaiveDate>
note: required by a bound in mysql::Row::take
--> C:\Users\turkerali.cargo\registry\src\github.com-1ecc6299db9ec823\mysql_common-0.28.2\src\row\mod.rs:130:12
|
130 |         T: FromValue,
|            ^^^^^^^^^ required by this bound in mysql::Row::take
CASE 2
error[E0599]: the method to_value exists for struct chrono::NaiveDate, but its trait bounds were not satisfied
--> src\eni_db.rs:472:148
|
472 | ...E,"%Y-%m-%d").unwrap().to_value()});
|                           ^^^^^^^^ method cannot be called on chrono::NaiveDate due to unsatisfied trait bounds
|
::: C:\Users\turkerali.cargo\registry\src\github.com-1ecc6299db9ec823\chrono-0.4.19\src\naive\date.rs:99:1
|
99  | pub struct NaiveDate {
| --------------------
| |
| doesn't satisfy chrono::NaiveDate: Into<mysql::Value>
| doesn't satisfy chrono::NaiveDate: mysql::prelude::ToValue
|
= note: the following trait bounds were not satisfied:
chrono::NaiveDate: Into<mysql::Value>
which is required by chrono::NaiveDate: mysql::prelude::ToValue

@turkerali

note: required by a bound in mysql::Row::take
--> C:\Users\turkerali.cargo\registry\src\github.com-1ecc6299db9ec823\mysql_common-0.28.2\src\row\mod.rs:130:12

This seem to refer to 0.28.2 even though your version is updated to 0.29.0. It it somehow possible that you are using multiple versions of mysql_common (direclty or via some dependency)?

Hi again.
You are 100% right. Here is the cargo tree output:
|->mysql v22.1.0
|-------->mysql_common v0.28.2
|->mysql_common v0.29.0
As you can see above, there are 2 mysql_common crates involved. Mysql crate pulls "mysql_common v0.28.2" automatically, which does not provide the chrono feature. The second mysql_common is coming from my config.toml:
[dependencies]
mysql = ""
mysql_common = {version = "0.28.2", features = ["chrono"]}  -----> WORKS
mysql_common = {version = "", features = ["chrono"]}   -----> FAILS
What is the proper solution here? Should I continue using the old mysql_common?
Thanks in advance.

Yeah. You should continue to use the old one because the mysql crate is not yet updated.


Release Google.Cloud.Asset.V1 version 2.10.0

Changes in this release:

Commit b71303f: feat: Release of relationships in v1, Add content type Relationship to support relationship search
Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.Asset.V1-2.10.0


poweroff: nordic: streamline RAM retention strategy

retained_mem should be used for RAM retention management on nRF devices.
sys_poweroff should disable RAM retention for unused RAM blocks to achieve lowest power consumption in System OFF state.
Align system_off sample accordingly.

Adding DNM for now until downstream tests finalize

@nika-nordic needs a rebase

@nika-nordic needs a rebase

done


Update clobber to match ARM ARM

I just learned about this asm! feature, which prompted me to look at whether we were using it.
Not sure if some of the others were options when #381 was written, but strictly speaking arm cores are defined to follow the AAPCS on exception entry/exit. In practice, I think C and aapcs (and the others here) are just aliases that do the same thing. Nonetheless, just in case, we should probably put the 'most correct' name here.
Relevant doc: https://doc.rust-lang.org/nightly/reference/inline-assembly.html#abi-clobbers

Looking at that it seems like "C" is the correct name to use, as we can use it on all platforms. Why do you think aapcs is the correct name?

Strictly speaking, ARM cores are defined to adhere to the AAPCS on exception entry and exit. There also exists a separate specification, the C Library ABI. In practice (and for obvious reasons), these both resolve to the same base ARM ABI.
Will this ever matter? Not in this century I can't imagine, the C and AAPCS ABIs will likely always be the same. But it struck me as odd to see C there when I happened to look at this earlier, so I figured we should update to match the letter of the law. Not a critical thing (see that branch name 😆).
[Plus, what a shame to have a lingering C hanging around in the Rust libtock on the Rust tock -- especially if it shouldn't be there anyway!]

There is more to this than just exception entry and exit. Yield may invoke an upcall, which has its own ABI. Currently, libtock-rs is hardcoded to use the C ABI for upcalls.
That ABI should match the ABI used by Yield's asm! -- except it is in libtock_platform, which is platform-independent code! I'm not sure we want to switch the ABI specification for different architectures.
Also IDK if we want to touch libtock_unittest. Ostensibly libtock_unittest should be cross platform too and should use AAPCS on ARM, but it's not an easy change to make and we don't have an easy way to test it.

Strictly speaking, ARM cores are defined to adhere to the AAPCS on exception entry and exit.

What exactly does this mean? Does the hardware store callee-saved registers for us? I thought the Tock kernel saves them in the grant region.

This looks like it touches more things than I realized, and given that the two are almost certainly identical for the forseeable future, leaving this as C makes sense.

What exactly does this mean? Does the hardware store callee-saved registers for us?

The hardware stores the caller-saved registers for us. Conceptually, ARM exception entry is a function call (adhering to the AAPCS) injected by the hardware at the point of the exception. As the hardware is the thing 'calling' the function, it has to do all the caller-saving responsibilities.

I thought the Tock kernel saves them in the grant region.

Yes, callee-saved registers the kernel stores in the CortexMStoredState.regs, which is part of the ProcessStandard struct.


Update dependabot.yml

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

approve

LGTM

LGTM


Fix Cognito configuration for OpenSearch

Currently, our OpenSearch configuration with respect to Cognito is not correct.  The master user isn't working.  That's because I was having trouble mapping users to roles by any means other than just dividing between authenticated & unauthenticated.
I wonder if this may be a bug in the terraform AWS provider aws_cognito_identity_pool_roles_attachment resource.  It doesn't appear role_mapping type = Token is effective.
In the AWS Console, here's how it would be configured:

Console -> Cognito -> Identity Pool -> Edit
expand Authentication providers
select the Cognito tab
scroll down to Authenticated role selection
select Choose role from token from the drop-down
make a selection for If no roles are specified in the token... in drop-down labeled Role resolution

I have that set in Terraform but it doesn't show up that way in the UI.  Instead, it says User default role.  Seems buggy.
Related: Spurious User Pool Clients
There are three User Pool Clients presently configured (tf resource aws_cognito_user_pool_client) but only one of them is in use.  The others appear to be the result of OpenSearch upgrade operations; AWS had to create a new UP-Client to change the login URL(s) to /_dashboards instead of kibana 😵 but it didn't clean up the old ones 😮💨.  The unused UP-Clients should be deleted:

1a30pmm active in-use!
2rqq72 delete it
4518b delete it

User Pool Clients don't have tags so it's hard to detect when they've been created unintentionally.  😠

This may be easier with https://github.com/hashicorp/terraform-provider-aws/pull/30140 creating the new resource-type aws_cognito_managed_user_pool_client.

Some of this was related to the user_pool_client problem.  Re-check the above problems before troubleshooting further.  Might still need some permission/role adjustments.

May still be some outstanding work to do.  Didn't intend to close this issue.


Document changing DB plans

This covers what @brianhelba showed @zachmullen.

I meant he showed Zach and me, this week.


Suppressed vulnerabilities of RelatedDependencies missing in report

Describe the bug
When dependencies are bundled up as they concern the same set of libraries (e.g. Spring) and vulnerabilities are suppressed for any of the 'related vulnerabilities' these suppressions do not show up at the Suppressed Vulnerabilities section of the report.
Version of dependency-check used
The problem occurs using version 7.3.0 of the maven plugin, but due to their nature are expected to be reproducable on any of the integrations.
To Reproduce
pom.xml:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
    <modelVersion>4.0.0</modelVersion>
    <groupId>net.aikebah</groupId>
    <artifactId>suppressions-test</artifactId>
    <version>1.0-SNAPSHOT</version>
    <dependencies>
        <dependency>
            <groupId>org.springframework</groupId>
            <artifactId>spring-web</artifactId>
            <version>5.3.23</version>
        </dependency>
    </dependencies>
    <build>
        <plugins>
            <plugin>
                <groupId>org.owasp</groupId>
                <artifactId>dependency-check-maven</artifactId>
                <version>7.3.0</version>
                <configuration>
                    <suppressionFile>${project.basedir}/filter.xml</suppressionFile>
                </configuration>
            </plugin>
        </plugins>
    </build>
</project>

filter.xml:
<?xml version="1.0" encoding="UTF-8"?>
<suppressions xmlns="https://jeremylong.github.io/DependencyCheck/dependency-suppression.1.3.xsd">
    <suppress>
        <notes><![CDATA[
      file name: spring-web-5.3.23.jar
      ]]></notes>
        <packageUrl<EMAIL_ADDRESS>        <cve>CVE-2016-1000027</cve>
    </suppress>
</suppressions>

Run mvn dependency-check:check
Observed behavior
Spring-web is reported as a related dependency of Spring-core and the suppression is not accounted for in the suppressed vulnerabilities count and the suppressed vulnerabilities section.
Expected behavior
Spring-web is reported as a related dependency of Spring-core and the suppression is accounted for in the suppressed vulnerabilities count and the suppressed vulnerabilities section.
Additional context
Already checked with mvndebug that the suppressions are available, the reporting needs to be adapted to take suppressed vulnerabilities of related dependencies into account.

An alternative idea - could we generate a suppression rule in the HTML report that includes all of the related dependencies?

Might be a good way to achieve the same result: prevent not reporting on suppressed vulnerabilities.
Recently ran into this issue on a project where we were suppressing the issue as Spring 6 is not yet available, just completed the digging into the root-cause and creation of a MWE reproducer.


disableAutoUpdateWhenUpdating not working

I am on nova v2.8.0 and trying out your package.
When I use the disableAutoUpdateWhenUpdating() method, the slug field isn't updating at all. When I remove it, it is behaving like expected (updating the slug field also on update view)

@bernhardh I'd say that this is expected behaviour and does exactly what the method says it does, because when you're calling the disableAutoUpdateWhenUpdating() method for the slug field, you are telling it to not update its value if you are editing/updating the Resource in Nova.

I am not sure, what I have changed since then or if somehing changed in nova, but it is already working as expected.
When I created this issue, the problem was, that when I used the method disableAutoUpdateWhenUpdating(), that it didn't update the field even on creation. But thats not the case anymore. idk why.


Failed to build vcpkg.exe

I deleted the old version vcpkg folder and got the latest version from github. The script could not build vcpkg.exe.
I'm sure I have installed the vs2017 and english language package.
The error message is as follows:
PS D:\vcpkg> .\bootstrap-vcpkg.bat

D:\vcpkg>powershell.exe -NoProfile -ExecutionPolicy Bypass "& {& 'D:\vcpkg\scripts\bootstrap.ps1' }"
PS D:\vcpkg> .\scripts\bootstrap.ps1

Building vcpkg.exe ...

Building vcpkg.exe... done.

Telemetry
---------
vcpkg collects usage data in order to help us improve your experience.
The data collected by Microsoft is anonymous.
You can opt-out of telemetry by re-running the bootstrap-vcpkg script with -disableMetrics,
passing --disable-metrics to vcpkg on the command line,
or by setting the VCPKG_DISABLE_METRICS environment variable.

Read more about vcpkg telemetry at docs/about/privacy.md
Copy-Item : 找不到路径“D:\vcpkg\toolsrc\msbuild.x86.release\vcpkg.exe”，因为该路径不存在。
所在位置 D:\vcpkg\scripts\bootstrap.ps1:433 字符: 1
+ Copy-Item "$vcpkgReleaseDir\vcpkg.exe" "$vcpkgRootDir\vcpkg.exe"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (D:\vcpkg\toolsr...lease\vcpkg.exe:String) [Copy-Item], ItemNotFoundExce
   ption
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand

Copy-Item : 找不到路径“D:\vcpkg\toolsrc\msbuild.x86.release\vcpkgmetricsuploader.exe”，因为该路径不存在。
所在位置 D:\vcpkg\scripts\bootstrap.ps1:437 字符: 5
+     Copy-Item "$vcpkgReleaseDir\vcpkgmetricsuploader.exe" "$vcpkgRoot ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (D:\vcpkg\toolsr...icsuploader.exe:String) [Copy-Item], ItemNotFoundExce
   ption
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand

@lambdald, it might due to antivirus software, could you try to build with '.\bootstrap-vcpkg.bat -disableMetrics " and see if this issue still occur?

Related to https://github.com/microsoft/vcpkg/issues/10059

This operation did not work, and then I reset the windows10 system. The script can compile vcpkg.exe normally now.

Great to hear that, thanks for your updates!


[Event Request] Codeunit 9520 "Mail Management" – DownloadPdfAttachment

Hi,
Can you add event OnBeforeDownloadPdfAttachment in function DownloadPdfAttachment?
procedure DownloadPdfAttachment(TempEmailItem: Record "Email Item" temporary)
    var
        FileManagement: Codeunit "File Management";
        //>>>>>>>>>>>>>>>>>>>
        IsHandled: Boolean;
        //<<<<<<<<<<<<<<<<<<<
    begin
        //>>>>>>>>>>>>>>>>>>>
        IsHandled := false;
        OnBeforeDownloadPdfAttachment(TempEmailItem, IsHandled);
        if IsHandled then
            exit;
        //<<<<<<<<<<<<<<<<<<<
        with TempEmailItem do
            if "Attachment File Path" <> '' then
                FileManagement.DownloadHandler("Attachment File Path", SaveFileDialogTitleMsg, '', SaveFileDialogFilterMsg, "Attachment Name")
            else
                if "Body File Path" <> '' then
                    FileManagement.DownloadHandler("Body File Path", SaveFileDialogTitleMsg, '', SaveFileDialogFilterMsg, EmailAttachmentTxt);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeDownloadPdfAttachment(var TempEmailItem: Record "Email Item" temporary; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Rather than a pop up, consider a HUD of all the Processes for Monitoring

maybe http://masonry.desandro.com/ like grid for the amount of processes ?

Thanks a lot @chrisamoore
At the beginning, I created this rep for monitoring my PM2 processes during the PM2 dev, as we all known it's not easy to monitor remoting processes by PM2 CLI, and I certainly not anticipated that so many people are interesting about this project, they sent Emails and Notifications to provide helps and suggestions (just like you ;) ), I really appreciate that.
I'm currently working on a productive & professional one, it bases on InfluxDB (time series metrics, events and analytics), could do all the jobs that pm2-gui does and including:

Timeline of PM2 events
Status of triggers (restart / stop warning - SMS / Email)
I/O frequency
Network latency (response time, Apdex stuffs)

And I'll consider your suggestion, it's really pretty cool.
Powered by bouncy's dashboard UI:


Can not Auto Dupe/Loss protection between server crashes

In plugin page _https://torchapi.com/plugins/view/24fc7724-0740-4a54-8bb3-1191fd3c8db4_
I see that

plugin can Auto Dupe/Loss protection between server crashes.

And then I make a test.
First, I paste a Grid for test.
Second,  I run command "!save" to save game.
Third I use "!h save" to  save  the grid into hanger.
Fourth ,I  make the server crash with a command I write.
        [Command("stackOverFlow", "test")]
        [Permission(MyPromoteLevel.Admin)]
        public void stackOverFlow()
        {
            stackOverFlow();
        }

and then I kill the server.
This option is mock when a error happen and server can't autosave.
Fifth, I restart the server,and then I had copy a Grid(the one in the hanger,other one in game).

By test seems that Hangar can't promise prevent Clone and Loss of grid.
In the multiplayer online environment with many mods, server crashes are frequent. Players can clone spaceships through this way, or players will lose spaceships.

Old information; dupe protection is not in the plugin. Didn't pan out. it was in for awhile under trial but just didn't pan out

I have a well-run implementation that can recover automatically and avoid most problems.
My server has a plug-in written by myself, which provides functions similar to QuantumHangar. Among them, the functions of ship synchronization and crash recovery between multiple servers are realized. Their approximate implementation is as follows:
Add two new subfolders to each player's hangar folder: loadTemp and saveTemp.
When the player loads a grid from the hangar, he cuts the grid file into loadTemp and loads the grid in the game.
When the player saves a grid from the game, the serialized grid file will go into saveTemp instead of directly into the hangar folder. This means that players can load grids from the saveTemp folder and the hangar folder.
I call the two key steps hangar synchronization operation and hangar recovery operation.
Hangar synchronization operation: When the game is saved, delete the files in loadTemp and cut the files in saveTemp into the hangar folder.
Hangar recovery operation: every time the server is started, the files in loadTemp are cut into the dock folder, and the files in saveTemp are deleted.
In order to avoid the problem of copying the grid in a multi-server environment, each server should have its own saveTemp and loadTemp folders.
When the player runs in a server:! H list, he can only see the grid in the saveTemp of his server and the grid in the hangar.
I call the grids in saveTemp "unsynchronized grids", and they can't be loaded in other servers for the time being.
Because the synchronization is carried out when the game is saved, the more frequently the game is saved, the more timely the synchronization between docks is. My setting is to automatically save the game once every 15 minutes. This will not bring many bad experiences, because players only need to play in one server in most cases.
However, in order to ensure non-invasion and higher success rate, it is necessary to ensure that the synchronization operation of the hangar should be carried out every time the server is saved, and when the game is saved, the synchronization operation of the hangar cannot be carried out.
In order to realize it, it needs to be repaired by patch. After many failed experiments, the method of "MySessionSnapshot.Save" has the best repair effect.

I have a well-run implementation that can recover automatically and avoid most problems.
My server has a plugin written by myself, which provides functions similar to QuantumHangar. Among them, the functions of ship synchronization and crash recovery between multiple servers are realized. Their approximate implementation is as follows:
Add two new subfolders to each player's hangar folder: loadTemp and saveTemp.
When the player loads a grid from the hangar, he cuts the grid file into loadTemp and loads the grid in the game.
When the player saves a grid from the game, the serialized grid file will go into saveTemp instead of directly into the hangar folder. This means that players can load grids from the saveTemp folder and the hangar folder.
I call the two key steps hangar synchronization operation and hangar recovery operation.
Hangar synchronization operation: When the game is saved, delete the files in loadTemp and cut the files in saveTemp into the hangar folder.
Hangar recovery operation: every time the server is started, the files in loadTemp are cut into the dock folder, and the files in saveTemp are deleted.
In order to avoid the problem of copying the grid in a multi-server environment, each server should have its own saveTemp and loadTemp folders.
When the player runs in a server:! H list, he can only see the grid in the saveTemp of his server and the grid in the hangar.
I call the grids in saveTemp "unsynchronized grids", and they can't be loaded in other servers for the time being.
Because the synchronization is carried out when the game is saved, the more frequently the game is saved, the more timely the synchronization between docks is. My setting is to automatically save the game once every 15 minutes. This will not bring many bad experiences, because players only need to play in one server in most cases.
However, in order to ensure non-invasion and higher success rate, it is necessary to ensure that the synchronization operation of the hangar should be carried out every time the server is saved, and when the game is saved, the synchronization operation of the hangar cannot be carried out.
In order to realize it, it needs to be repaired by patch. After many failed experiments, the method of "MySessionSnapshot.Save" has the best repair effect.
some of the code may like this:
    internal static readonly MethodInfo Save =
        typeof(MySessionSnapshot).GetMethod(nameof(MySessionSnapshot.Save),
            BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public) ??
        throw new Exception("Failed to find patch method");

 public static void SaveSuffixes(ref bool __result)
    {
        if (__result)
        {
           DoHangerSync();
        }

    }

After my test, there is still a low probability that this scheme can't solve the problem (such as an error during synchronization). But it works well in most cases.
In the early stage of writing my own plug-in, I referred to many codes of quantum hangars. Originally, I wanted to submit a pull request. But I found that some stateful saving files were used in the quantum hangar.
(The quantum hangar saves some meta-information through a PlayerInfo.json file. If I want to manually modify the player's hangar, it will be very inconvenient. In fact, this is why I chose to implement a stateless library plug-in myself)
Moreover, because Quantum hangar provides some third-party interfaces for other plug-ins, such as "NexusApi". I'm not familiar with it, and I'm afraid it will destroy its function. So I'm sorry I can't submit a PR. I'm just here to offer a suggestion on how to realize it.

This is exactly what was in Hangar, but was removed for the simple fact that there is no way for you to know what happened to the grid ingame after unloading reliably. Did it separate? Did it transfer to a different server? Did players grind everything down? Did someone else grab it and hangar it?
It was decided that its safer not to do anything as anything similar to what you described and what I had would leave holes for players to abuse and dupe grids in any server environment. In nexus, or not. (many servers share their hangars). Sure, decreasing the save interval is better, its not something that should be considered or required when running the plugin.
There are fixes, but none are easy and would require a lot of effort to implement effectively.
However, if you feel like your solution works, great! Just be aware that there are possibilities where duplication can occur.
I should go ahead and remove that line from the description.


(BUG) QnA Maker - "Unable to get endpoint from State"

Created a new QnA Maker Bot using the template.

Went to "QnA" and added in a Key Value Pair

Added an QnA Maker Key ("Choose from existing"). Got "Added" message (see check box on rocket menu).

Ran Bot. Asked question. "Unable to get endpoint from State".

Version
Version:  1.4.0-nightly.237101.f01cb52
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0

Closing as duplicate


How to customize error message sent to an end-user when bot's endpoint is unavailable?

If a user sends message to a bot while it's endpoint is unavailable or failed processing the message, the user receives an error message from Bot Connector, which seems like that:

POST to {SomeArbitraryBot} failed: POST to the bot's endpoint failed with HTTP status 502

or

POST to {SomeArbitraryBot} failed: A task was canceled.

etc.
Is there a way to customize logic of sending error message or to suppress it at all in such a case?

Hi @Deilan, is this a v1 bot or a v3 bot?
The behavior was changed in v3 so that the Bot Connector returns an error to the channel but does not generate error text.

It's all about the latest version: v3.
As a matter of fact this behaviour (to send error or not; if yes, then what message text to send) should be customizable. Exposing the technical issues and details to the end-users is usually undesirable (unless it's debugging or testing).

Which channel are you using? There may be a bug in the way that channel is handling errors.

@dandriscoll I am using Telegram.

Yep, looks like a bug. Thanks @Deilan.

@nwhitmont And... what is the answer? :) Having this one added to the internal backlog?


Add aryanishan1001 to approvers in README.md

Please add aryanishan1001 in file README.md, in the approvers section.

Closing, fixed by #479


dynamic quant for llava 1.5 / 1.6 models

is it possible to have the dynamic quant versions of  unsloth/llava-v1.6-mistral-7b-hf-bnb-4bit  and "unsloth/llava-1.5-7b-hf-bnb-4bit ?
seems like 4bit quant create some hallucination on these models aswell

is it possible to have the dynamic quant versions of unsloth/llava-v1.6-mistral-7b-hf-bnb-4bit and "unsloth/llava-1.5-7b-hf-bnb-4bit ?
seems like 4bit quant create some hallucination on these models aswell

Great suggestion - I thought we did it but did not. Thanks for letting us know - we plan to release it along with textbased models soon. You can sign up to our newsletter if you'd like to be updated: https://unsloth.ai/waitlist


[📚] [Cloud Firestore] Change all the cloud_firestore example with null_safety

flutter has announce flutter 2.2 since the last google i/o, and Sound null safety is now the default for new projects. Most of our docs example still used legacy version of dart code.
Instead of two seperated example (https://firebase.flutter.dev/docs/firestore/2.0.0_migration/) and (https://firebase.flutter.dev/docs/firestore/usage#one-time-read)
why don't we use null safety as the default docs and code snippet example?

Related but for auth; #6308


sleep when resource in not satisfy. fix #1522

Codecov Report

Merging #1523 into dev will increase coverage by 0.09%.
The diff coverage is 5.71%.

@@           Coverage Diff            @@
##             dev   #1523      +/-   ##
========================================
+ Coverage   6.75%   6.85%   +0.09%
========================================
  Files        272     272
  Lines      13612   13619       +7
  Branches    2165    2168       +3
========================================
+ Hits         920     934      +14
+ Misses     12628   12622       -6
+ Partials      64      63       -1

Impacted Files
Coverage Δ

...che/dolphinscheduler/api/service/QueueService.java
0% <0%> (ø)
:arrow_up:

...er/server/master/runner/MasterSchedulerThread.java
0% <0%> (ø)
:arrow_up:

...apache/dolphinscheduler/alert/utils/FuncUtils.java
100% <100%> (+100%)
:arrow_up:

...apache/dolphinscheduler/alert/utils/JSONUtils.java
80% <0%> (+30%)
:arrow_up:

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 947d804...836bf48. Read the comment docs.

this operation is very heavy, first will get zk lock, second will scan database.
when cpu and memory is enough, not sleep will give a lot of pressure on ZK and database
What do you think？

yes， when no command , this will increase pressure for zk and db.
I have updates the code logic

this operation is very heavy, first will get zk lock, second will scan database.
when cpu and memory is enough, not sleep will give a lot of pressure on ZK and database
What do you think？

yes， when no command , this will increase pressure for zk and db.
I have updated the code logic

oh， I didn't see the last sleep logic

this operation is very heavy, first will get zk lock, second will scan database.
when cpu and memory is enough, not sleep will give a lot of pressure on ZK and database
What do you think？

yes， when no command , this will increase pressure for zk and db.
I have updated the code logic

oh， I didn't see the last sleep logic

Haha, added just after your first comments.

good job!


Optimized implementations of auc_parity and log_loss_parity do not match sklearn.metrics

auc_parity and log_loss_parity give a different result than the sklearn.metrics implementation if the number of unique y_prob is below a certain threshold. This is especially an issue for tree-based models that tend to output from a discrete set of possible prob values.
following test cases added to vertiastool/metrics/tests/test_fairnessmetrics.py will fail
def test_compute_auc():
    # rounding y_prob to simulate fewer distinct y_prob output
    y_prob_ =  np.round(y_prob, 4)  # only passes at 6 decimal places
    container = ModelContainer(
        y_true = y_true, y_train = y_train, p_var = p_var, p_grp = p_grp, x_train = x_train,  x_test = x_test,
        model_object = model_object, model_type  = model_type,model_name =  model_name, y_pred= y_pred,
        y_prob= y_prob_)
    cre_sco_obj= CreditScoring(model_params = [container], fair_threshold = 0.43, fair_concern = "eligible",
                            fair_priority = "benefit", fair_impact = "significant",
                            num_applicants =num_applicants,  base_default_rate=base_default_rate)

    cre_sco_obj.curr_p_var = 'SEX'
    cre_sco_obj.y_true = [container.y_true]
    cre_sco_obj.y_prob = [container.y_prob]
    cre_sco_obj.sample_weight = [None]

    # adding 'y_pred_new' will use sklearn.metrics implementation
    result = FairnessMetrics._compute_auc_parity(cre_sco_obj, y_pred_new=cre_sco_obj.y_prob)
    cre_sco_obj.evaluate()
    result2 = cre_sco_obj.get_fair_metrics_results()["SEX"]["auc_parity"][:2]
    assert result == pytest.approx(result2)

# repeated for log_loss
def test_compute_log_loss():
    y_prob_ =  np.round(y_prob, 4)  # passes at 6 decimal places
    container = ModelContainer(
        y_true = y_true, y_train = y_train, p_var = p_var, p_grp = p_grp, x_train = x_train,  x_test = x_test,
        model_object = model_object, model_type  = model_type,model_name =  model_name, y_pred= y_pred,
        y_prob= y_prob_)
    cre_sco_obj= CreditScoring(model_params = [container], fair_threshold = 0.43, fair_concern = "eligible",
                            fair_priority = "benefit", fair_impact = "significant",
                            num_applicants =num_applicants,  base_default_rate=base_default_rate)

    cre_sco_obj.curr_p_var = 'SEX'
    cre_sco_obj.y_true = [container.y_true]
    cre_sco_obj.y_prob = [container.y_prob]
    cre_sco_obj.sample_weight = [None]

    # adding 'y_pred_new' will use sklearn.metrics implementation
    result = FairnessMetrics._compute_log_loss_parity(cre_sco_obj, y_pred_new=cre_sco_obj.y_prob)
    cre_sco_obj.evaluate()
    result2 = cre_sco_obj.get_fair_metrics_results()["SEX"]["log_loss_parity"][:2]
    assert result == pytest.approx(result2)

Hi Micheal,
Thanks for your question.
In recent release we have added two test cases in test_fairnessmetrics.py

test_compute_log_loss_parity and
test_compute_auc_parity()
These shoud ensure that the results are similar to sklearn.metrics implementation.

Hello Nisith, does it cover the specific scenario highlighted above regarding unique y_prob values? And does the new code pass the test I wrote above? I am not suggesting you add my exact test to your test suite but just want to know if it addresses that specific issue.

okay i tested it and it seems to be fixed. thanks.

Hi Micheal,
Thanks for verifying.


"Affordable"

We should perhaps compare search strings for targets that include "affordable". Add targets where affordable is included under:

3.8 (also added to 3.7 but more as an element of universal access, "affordable" not in the target)
4.3
7.1
11.1
11.2

I've added some terms for this now in the vocab list

Affordable is included in the targets:
3.8
3.b
3.b.3
4.3
6.1
7
7.1
9.1
9.3
9.c
11.1
11.2


Add Xyce simulation flow for the temperature sensor

@saicharan0112

I think we need to be able to install Xyce as a conda package? @proppy

As discussed early this week. @alibillalhammoud can you work with @saicharan0112 on this

Update: issues with the current spice template for Xyce. Related to the Xyce simulator - https://groups.google.com/u/1/g/xyce-users/c/PD5I1OElQSE

I am not sure if we are supporting Xyce anymore.


Refactoring to use Identities instead of tax tags

Mexico now validating using Identity Keys instead of the previous Use tax tags.
Identity objects will now be validated against the regime's list of acceptable "Identity Key Definitions", with the list of valid codes. This is somewhat basic validation at this time, just compares the list of codes, but could be expanded.

Codes now supported:

mx-cfdi-fiscal-regime - Fiscal Regime Code, set in the Invoice Supplier and Customer
mx-cfdi-use - Set as part of the customer's identities.
mx-cfdi-prod-serv - Aligned with "ClaveProdServ" in the CFDI XML.

I'm going to go ahead and merge this to create a new release. Please do comment regardless!

@cavalle Thanks for looking at this in detail and for the comments! Here's my take:

UsoCFDI at the document level. This was the first implementation that aligned with the previous tax tags, but then while reading the CFDI specs, it became clear this is something that should be associated with the customer. While its true that in theory the value could change per invoice, this is very unlikely. A more likely scenario is that a single customer will have multiple very-similar invoices that should be issued under the same label. In my opinion, I think we should stick with it at the customer level reflecting that as a supplier, you're only likely to ask your customer once what their "Use" is, and every invoice after that initial decision will be the same. The alternative is asking for the "use" on each invoice, and that doesn't sound quite right.

"Identities" name isn't accurate enough. I'm inclined to agree. Also, in #184, to be able to achieve the same with the tax rates, a new name has been used: codes which is simply a map of a key to a code. It would be better to have a uniform implementation. We still have time to change this while we don't have many customers in MX.

I'll think about this in the background... something along the lines of exts referring to "extensions" or something similar could work. You want to infer that the data is not really part of GOBL, its something "extra" that is required to work locally. Pagero took this approach: https://pagero.github.io/puf-billing/#_country_specifics along the same lines.

Good points! Regarding UsoCFDI, it's always hard for me to anticipate what would be more common. I imagine that the same customer, from the same supplier, buying the same type of product or service, will most of the time set the same UsoCFDI, yes. But I can see how the same customer is more likely to need to use a different UsoCFDI if the supplier (or the type of product/service they buy) varies.
Also, one could argue that whether the same customer will use the same UseCFDI value or not should be a concern of the system providing a UI to the customer, not of the underlying struct that transports the data, which should be more concerned with modelling the domain in an intuitive way for developers. In this respect, a developer may be misled into thinking that the UsoCFDI is an attribute of the supplier, independent of the invoice (like the tax ID, the name or the RegimenFiscal are), when this isn't the case.
On the other hand, I have to admit that in the CFDI XML, the UsoCFDI attribute is part of the Receptor node, which does not support my point of view! 🤷
As for using extensions in GOBL, the same idea crossed my mind. That's a typical way to solve this problem in XML formats (e.g. UBL) which allow us to have structs more complex that key/values. The risk is that extensions could be abused and, as it happens with UBL and many others, everything ends up being an extension, and you end up with no standard. Also, you'll need additional JSON schemas, etc… But I'm not against the idea, in any case.

Hehe one option would be to support both. So effectively developers have the option of overriding the Customer's UsoCFDI at the invoice level, if they need to. I think what I worry about though, imagining the implementation here, is that if you don't have the "use" in the customer, you'll need to store it somewhere else. I'm not sure where that "somewhere else" is, unless you ask the user every time you build an invoice. In that scenario, it is just as easy to update the customer each time.
This morning I woke with the idea of a simple ext field that operates like a simple map. Kind of like the meta, but stricter. Examples in context:
{
  "$schema": "https://gobl.org/draft-0/bill/invoice",
  ....
  "tax": {
    "ext": {
      "mx-cfdi-use": "G01"
    }
  },
  ....
  "customer": {
    "tax_id": {
      "ext": {
        "mx-cfdi-fiscal-regime": "601"
      }
    }
  },
  .....
  "lines": [
    {
      "item": {
        "ext": {
          "mx-cfdi-prod-serv": "12345678"
        },
     },
     "tax": [
       {
          "cat": "VAT",
          "rate": "exempt",
          "ext": {
             "mx-cfdi-exempt": "XXX"
          },
      }
    ]
  }
}

I made up the mx-cfdi-exempt key, there are better examples for this in PT and IT. Basically, though, we're talking about a very simple extension key-value map, with validated keys and values. This would avoid as you say the complex extensions which are allowed in UBL/XML.

Hehe one option would be to support both

I've checked a couple of Mexican invoice apps (Facturama and Alegra) and they both have the "Uso CFDI" field as part of the client form (and not the invoice) (see for example here). So, as much as I still stand by my previous arguments, I'm now happy to leave this field as part of the supplier, and not at invoice level, if only to mirror the CFDI model and the way real-world clients seems to be implemented.

Kind of like the meta, but stricter.

Sounds like a good option to me! It may be accompanied by something in the regime data that allow us to define the valid keys, their labels and descriptions… similar to what you did with Identities. I wonder if these should completely replace org.Identities or not…

Thanks for investigating the Uso thing! So the customer seems okay for the time being, but we'll be mindful of feedback to see what could be useful.

Sounds like a good option to me! It may be accompanied by something in the regime data that allow us to define the valid keys, their labels and descriptions… similar to what you did with Identities. I wonder if these should completely replace org.Identities or not…

I have a PR in progress to do precisely that. Its no longer using org.Identities for anything official. I'll upload as soon as I've had chance to iron out all the kinks.


Update ujson to 1.3.12

Updates com.lihaoyi:ujson from 1.1.0 to 1.3.12.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.1.0).
You might want to review and update them manually.
CONTRIBUTING.md
algebras/algebra/src/main/scala/endpoints4s/algebra/StatusCodes.scala
project/plugins.sbt
xhr/build.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "ujson" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #815.


[gui] sway: Window contents are not rendered on Wayland backend

Describe the bug
When run in Sway, psst only displays the background of the window, but none of the contents. This means when first run, it display7s the log in screen as an empty white square. This is Wayland-specific as it doesn't happen when run in XWayland.
To Reproduce

Build dev version
cargo run --bin psst-gui

Expected behavior
The contents of the windows should be displayed when using Wayland as the backend.
Screenshots
Log in screen, not working:

Log in screen, XWayland:

After setting up a profile via XWayland and launching via Wayland. Notice it seems to skip log-in, and remembers the user's theme setting:

Environment

OS: nixos unstable
Version: master / de5c12f8b27385c609e73a9354b1a04a6e0953d2

Additional context

Working version via XWayland: WAYLAND_DISPLAY= cargo run --bin psst-gui
Repros on the release build
I haven't had a chance to try other Wayland-based desktop environments

Hi @oslerw, Druid does not yet have a native wayland API. There is a Draft PR pending: https://github.com/linebender/druid/pull/1498

For me it seems to be working just fine on the current master on Wayland (also running sway), no idea how if druid is not there, yet :)

@filmor -- out of curiosity, what version of Wayland are you running, and on what distro? I can do more testing to try to narrow down why it's reproducible on my system but not yours.

This is Gentoo Linux. The relevant(?) versions are:

Package
Version

sway
1.6.1

wayland
1.19.0

@oslerw @filmor Psst works perfectly fine for me on GNOME wayland. (I can verify it's running on wayland by running xeyes)
I'm on arch using GNOME on wayland, and this is my wayland and mutter version
wayland 1.20.0-1
mutter 41.2-1

isn't this fixed now ?
https://github.com/linebender/druid/pull/2079
and psst is already running with an app_id which means it runs on wayland in my experience.

It worked fine on Gnome Wayland, but doesn't for me on KDE Wayland now that I've switched. I'm also running NixOS.
Since druid uses some GTK stuff I was able to force XWayland with GDK_BACKEND=x11.

Strange, it's working on Wayland for me.
Can those affected please provide the output of:
echo $GDK_BACKEND:$WAYLAND_DISPLAY:$DISPLAY$XDG_RUNTIME_DIR
And does the following command make it work how it should?
GDK_BACKEND=wayland psst-gui

I'm also running into this issue on Hyprland using psst version unstable-2023-05-13.
System: Linux 6.5.2, NixOS, 23.11 (Tapir), 23.11.20230911.3a2786e
$ echo "$GDK_BACKEND:$WAYLAND_DISPLAY:$XDG_RUNTIME_DIR"
wayland:wayland-1:/run/user/1000


[Draft] Support using the public mpas-model release 8.2

This PR contains changes to switch from using the internal JCSDA mpas-model to the public mpas-model release 8.2.
It includes

Change stream names from "static" to "invariant".
Change lsm scheme name from "noah" to "sf_noah"

It is based on changes in https://github.com/NCAR/MPAS-Workflow/pull/313, authored by @mos3r3n.
Tests completed
Tier 1:

[ ] 3dvar_OIE120km_WarmStart
[ ] 3denvar_OIE120km_IAU_WarmStart
[ ] 3dvar_OIE120km_ColdStart
[ ] 3dvar_O30kmIE60km_ColdStart
[ ] 3denvar_O30kmIE60km_WarmStart
[ ] eda_OIE120km_WarmStart
[ ] getkf_OIE120km_WarmStart
[ ] ForecastFromGFSAnalysesMPT

Tier 2 (optional):

[ ] GenerateGFSAnalyses
[ ] GenerateObs

Hi @jim-p-w, @liujake, @amstokely, @mos3r3n, and @byoung-joo, I ran the reference tests with this feature branch and got some interesting results:

Dual resolution 30km-60km tests failed with same issue as we have seen for the variable mesh, i.e., gets stuck after minimization and QCs after analysis but cannot overwrite the background files with updated analysis variables

Tests for ColdStart initialization and ForecastFromGFSAnalysesMPT failed in the creation of the external analysis with the following error message in files such as log.init_atmosphere.0013.err:

ERROR: At least one fields to be read from the 'input' stream is dimensioned
ERROR: by 'nSoilComps', but the 'nSoilComps' dimension is not defined
ERROR: in the file x1.40962.static.nc
CRITICAL ERROR: Please check the input file(s) to be read by the 'input' input stream.

and in the log.init_atmosphere.0000.out
 Reading dimensions from input streams ...

 ----- reading dimensions from stream 'input' using file x1.40962.static.nc
        nCells = 40962
        nEdges = 122880
        nVertices = 81920
        TWO = 2
        maxEdges = 10
        maxEdges2 = 20
        vertexDegree = 3
        R3 = 3
        nMonths = 12
        nSoilComps *** not found in stream ***

ERROR: At least one fields to be read from the 'input' stream is dimensioned

Do we need to update the static file to one created with this new version of the model, or make any additional change?
Tests with single resolution for DA, such as 120km and with WarmStart for 3denvar, getkf and eda, passed successfully. In case you want to take a look, please find the tests here: /glade/derecho/scratch/ivette/pandac-PR314

I wonder if this is related to https://github.com/MPAS-Dev/MPAS-Model/pull/1194.  From that PR, nSoilComps is only related to sf_noahmp, while in our workflow we use sf_noah.

Ah, https://github.com/MPAS-Dev/MPAS-Model/pull/1194 is rekated ti core_atmosphere.
I realized that the error message Ivette see is related to core_init_atmosphere.
Yes, I think we need to re-generate our existing MPAS static files, such as it can be also used for sf_noahmp option.
Can you confirm this, @mgduda ?

The error message is a bit confusing because from our existing static files, there is no variable with a dimension nSoilComps...

The error message is a bit confusing because from our existing static files, there is no variable with a dimension nSoilComps...

Exactly! That's why I wondered if we needed to update the static file.

@mos3r3n when you run 60km-3km test with public model 8.2.x code, did you generate 'invariant' file, or still using 'static' file (linked to a init.nc file)?

@mos3r3n when you run 60km-3km test with public model 8.2.x code, did you generate 'invariant' file, or still using 'static' file (linked to a init.nc file)?

I have not tested v8.2.x yet. In my v8.1.x run, the invariant file is linked from the init.nc file, which is generated by myself.

In that case, we need a cleaner test by using a real 'invariant' file produced using 8.2.x code. There is a chapter 8.3 (if I remember correctly) for generating invariant file, from the latest model user guide. @jim-p-w I guess you also did not generate an 'invariant' file for your 120km test. So I consider our 'public' cycling tests so far are not rigorous. We need to re-do the 'public' tests with an 'invariant' file for all scenarios.

https://www2.mmm.ucar.edu/projects/mpas/mpas_atmosphere_users_guide_8.2.0.pdf

@jim-p-w It seems to me that we should need to modify somewhere in the workflow script to link a real 'invariant' file, instead of the old 'static.nc' linked to an 'init.nc' file. But I only see 'static' stream is changed to 'invariant' stream in configuration files, but looks like still using 'static.nc' file for the 'invariant' stream. I think we'd better make it logically consistent to use an 'invariant.nc' file for the 'invariant' stream. 'static.nc' and 'invariant.nc' are different things in 8.1/8.2 notation. We need to be rigorous and remove the use of 'static.nc' file in the workflow.

@liujake I am looking into the static/invariant issue. Michael's initial thought was using the static file as the invariant file should be OK. Apparently the static stream was never officially supported and there's no documentation for it.

We can assume our use of 'static' file produces the same results as using 'invariant' file. But we need to confirm it by using 'invariant' file. Take your time to learn first to generate necessary 'invariant' file(s), which is something new to us. I did not make this very clear from the beginning of testing 'public' mpas-bundle code.

Hi all, an update on my recent tests using Jim's build code with MPAS-Model v8.2.1:
30km-60km:

Created static, invariant, init and 1st background (following the user's guide). Ran test and DA worked successfully!!!
Experiment is almost finished!
Please check: /glade/derecho/scratch/ivette/pandac/ivette_3dhybrid_O30kmIE60km_public_MPAS-modelv8.2.1/CyclingDA/2018041500/run/mem001

Two things here for the success of this test:

For our DA, we use the init file as our static/invariant file, therefore, here we have to regenerate init files for this experiment with new public model version to work.

I tried using new init file with old 1st background file and did NOT work, therefore, the 1st background files also need to be regenerated.

60-3km-60km:

Created static, invariant, init and 1st background. Ran test and DA did NOT work. Differently than before, it now shows an error message in jedi.log.all/jedi.log:

 --> write_fields: MPAS_stream_mgr_write failed ierr=          -3

MPICH ERROR [Rank 3] [job id 31e52196-bc35-4d1c-93ae-eec8a32ab078] [Sat Jul 20 23:34:11 2024] [dec1985] - Abort(1) (rank 3 in comm 0): application called MPI_Abort(MPI_COMM_WORLD, 1) - process 3

It also shows this message in log.atmosphere.0255.d0002.err
ERROR: MPAS IO Error: Undefined field

Please check: /glade/derecho/scratch/ivette/pandac/ivette_3dhybrid_O60-3kmIE60km_TEST_Modelv8.2.1/CyclingDA/2018041500/run/mem001
Important

While using Jim's build code, I noted that the new INIT files created do not contain diagnostics for 3D pressure or winds at cell centers (uReconstructZonal and uReconstructMeridional), which are used for our model space verification. Please check these lines: https://github.com/amstokely/MPAS-Model/blob/c8f4fe737b80990db3143eecf01b264f3db1ebe8/src/core_atmosphere/Registry.xml#L526-L572
I tried to make a new compilation adding these variables in that section for 'input' but didn't work, I may have missed something. May need some help adding these. In the meantime, I will try using old init files for model space verification.

I forgot to mention, I created a new branch changing all references to old static files and renaming then as invariant, which I have been using for my experiments: https://github.com/NCAR/MPAS-Workflow/tree/feature/changes4publicMPAS-8.2.1

@ibanos90 I want to make sure I understand you correctly: during the cycling of 30km-60km run, you used the new 'invariant.nc' files (one for 30km, one for 60km) in place of the old 'static.nc' files (which is linked to an 'init.nc' file)? i.e., during the cycling, you have mpasout file along with invariant file.

@ibanos90 I want to make sure I understand you correctly: during the cycling of 30km-60km run, you used the new 'invariant.nc' files (one for 30km, one for 60km) in place of the old 'static.nc' files (which is linked to an 'init.nc' file)? i.e., during the cycling, you have mpasout file along with invariant file.

That's correct, but I created new init file only for 30km in this case and worked

Hi all, my last test using same physical parameterizations as the ensemble members and 1st background (i.e., mesoscale reference suite) succeeded for the DA dual resolution @60-3km-60km!
If I use a different configuration in the parameterizations the test hangs until the walltime is exceeded. This wasn't noted before because for 30km-60km we use mesoscale reference for everything (ensemble forecast members, 1st background, analysis, subsequent forecasts).

@ibanos90 Good catch! So that was an issue of physics inconsistency between ensemble input and high-res. background/analysis.

@ibanos90 Good catch! So that was an issue of physics inconsistency between ensemble input and high-res. background/analysis.

Exactly!

Or inconsistency between 2 template files at different resolution could also played a role in the overwriting issue: one at 60km-3km (convection-permitting suite), one at 60km (mesoscale suite).

@ibanos90 Good catch! So that was an issue of physics inconsistency between ensemble input and high-res. background/analysis.

Exactly!

Hi @ibanos90. I noticed that in DA and forecasting, the init.nc was still linked as invariant.nc. Have you ever tried just using the invariant.nc? I think we need to do that eventually.

@ibanos90 Good catch! So that was an issue of physics inconsistency between ensemble input and high-res. background/analysis.

Exactly!

Hi @ibanos90. I noticed that in DA and forecasting, the init.nc was still linked as invariant.nc. Have you ever tried just using the invariant.nc? I think we need to do that eventually.

Hi @mos3r3n, I am not sure that's what we want to do. We link the init file as we do the dual resolution approach in the cycling DA. In the past, I have tried linking the static file and it doesn't work for the DA. @byoung-joo can you provide more guidance on this?

There might be misunderstanding here. The right way with 8.2.1, as I mentioned in earlier conversation, is to link the 'invariant.nc' file to 'invariant' stream file. The 'static.nc' file or 'init.nc' file should no longer be involved in the cycling. We should only need mpasout file and invariant file in the cycling. That is the main purpose we switch to the new 8.2.1 code. The old way of pretending init.nc file as static.nc file is not a clean way to do and is confusing.

If you link 'init.nc' to 'invariant.nc' in the cycling workflow, it is like you pretend init.nc as invariant.nc. But we should use actual invariant.nc file you generated, not init.nc file.

That's a good point. I tried the old way. I didn't realize this change, sorry about that.


one fdescribe fails causes rest of the tests to skip in Protractor Jasmine in Windows 10

We are trying to run the Protactor jasmine Testcase in Windows 10 machine.
In our suite i have 6 Testcase. If 3rd test case got failed rest of testcases are skipped automatically and executions stops.
We have Tried modifying the configuration file. But still rest of the cases are getting skipped.

In Config file we have added the below list of configuration keywords
stopSpecOnExpectationFailure: false

onPrepare: function() {
var failFast = require('jasmine-fail-fast');
jasmine.getEnv().addReporter(failFast.init());
}
afterLaunch: function() {
failFast.clean(); // Cleans up the "fail file"
}
jasmineNodeOpts: {
...
realtimeFailure: true
},

And also we have installed the following npm modules and included the below sentences. Still we are getting the same skip error.

Jasmine-bail-fast
Protractor-fail-fast

require('jasmine-bail-fast');
jasmine.getEnv().bailFast();
Please help us to resolve this issue.
-Prem

This sounds like a general support question, not directly related to the tutorial. Please, use one of the appropriate support channels for these types of questions.
GitHub issues are reserved for bug reports and feature requests.
Thx!


Vuex integration method

Hi there,
this plugin is comfy (thank you, mr. Alfhen) , but if you want to incapsulate smth in vuex store (for example the choice between cookie and local storage), you may have an issue: from-the-box vuex doesn't know about this.$cookie or vm.$cookie.
All u need is just reinitialize that. Look on the pictures.

I had the same problem

@noguilty4you @MauricioHernanCabrera Another option is to just use vue.prototype.$cookie directly.
import vue from 'vue'

async someAction ({ commit }) {
  // Do something
  vue.prototype.$cookie.set(key, value)
}

thanks, vue.prototype is interesting syntax


Fix attachment clipboard copy on insecure origin

Fixes: https://github.com/go-gitea/gitea/issues/26202
Actually later I found out the code did not use clippie, so I fixed it. The bug was never in the clippie module like I initially suspected. Also, I added a tooltip for feedback.

I was unable to create a backport for 1.20. @silverwind, please send one manually. :tea:
go run ./contrib/backport 26224
...  // fix git conflicts if any
go run ./contrib/backport --continue


AND query does not work correctly with drafts: true

Link to reproduction
https://github.com/diesieben07/payload/tree/where-and-drafts-reproduction
To Reproduce

Create a collection and enable versions on it.
Attempt to use an AND query combined with drafts: true via GraphQL. For example given the above reproduction case:query {
    Posts(
        draft: true,
        where: {
            AND: [
                {
                    text: { equals: "exampsle post" }
                }
            ]
        }
    ) {
        docs {
            text
        }
    }
}

Note the intentional misspelling of "example", so no posts should be returned.

Describe the Bug
AND queries combined with drafts: true are completely ignored by Payload. The above query returns all posts.
The reason for this is as follows:
The internal API uses and and or (defined here). The GraphQL API however uses AND and OR (defined here). Payload attempts to rectify this unfortunate discrepancy, but only for AND and in an insufficient manner. This is handled not in the GraphQL resolver like I would expect, instead the internal API has a special case for the AND key, which is not documented.

Internal operation find: https://github.com/payloadcms/payload/blob/2d0441a72e19297a5ce76b636d55141a15c43321/src/collections/operations/find.ts#L87-L92
Internal operation findVersions: https://github.com/payloadcms/payload/blob/2d0441a72e19297a5ce76b636d55141a15c43321/src/collections/operations/findVersions.ts#L58-L59

So far this is just code that isn't very pretty, but doesn't cause any problems (as far as I can tell, except potential collisions with fields named AND or OR, but that will cause problems anyways).
The bug now occurs when setting drafts: true, because appendVersionToQueryKey is called with the unaltered Where parameter from the internal operation find (without the and / AND normalization applied) and thus fails to properly prefix the filters with version.. This causes a query such as version.AND: [{text: {equals: "example post"}}] to be generated, which is obviously nonsense.
Payload Version
1.7.0

Daaanng good find! Fix incoming. 👍


Add vim mode V2

Noticed your update, and thought I'd rebase and add again. It was a lot cleaner to add this time I think. Didn't add docs yet yours have been updated hastily, and are in fact, broken for setting controls. Copied exactly, they fail to parse properly.
EDIT: Seems that my rebase introduced some bugs.

Update. Bugfixed. Seems to work fine now. Had to do an additional check against the search box if in vim mode to allow navigation to work properly, but block actual keys in the search box.

Hi @kdb424, If I understand the changes correctly, what this does is basically add insert/normal mode to the search input. Without other vim-like actions (motions come to mind) I guess I don't really see how this is so useful. Especially since searches will generally be quite short (a few words at most) and not be left half-finished in the search box to be edited later.
To me, the more useful vim-like features would be something like G/gg for going to the top and bottom of the search/queue lists, ctrl-u/ctrl-d for moving half-pages up and down, and other things like that.

@cfangmeier I definitely agree this can be expanded upon, and would like to do so. My main goal, at least for initial pass, was to have binds that were single key, while not losing the ability to type in the search box, which this pass has done. For example, if you set your "pause/play" button to "p" every time you type that letter in the search box, your music will start/stop. The initial PR was basically to see if you would be interested, and as a foundation for more features while being minimally invasive.

ok. That makes sense.
I checked out the branch and tried it out. I have a few suggestions to make it even better:

Add some visual indicator for normal/insert mode. This could be as simple as changing the cursor style in the search field. Not sure how easy urwid makes that...
Switching to insert mode should automatically move the focus to the search input.
Hitting tab/shift-tab should automatically exit insert mode and move the focus as usual.

How does that sound?

1: That's a fantastic idea. I have been dealing with not knowing what mode I was in, and missed this obvious idea.
2: I must have missed this when I re-based, but I originally had it where if you jumped to search with a bind (example /), it would put you in insert mode as that was the clear intention to search, where as just tabbing through may mess with other binds if a user bound "tab" to a single alphanumeric.
3: With the last comment in mind, maybe hitting enter when searching should exit insert mode instead? This type of action would mimic the command mode more closely.


updated dataset source for Twitter sentiment analysis template

Looks like you need to restart kernel and clear all cells. What environment are you using?

Deal check the new commit. I'm editing and running the notebook in Google Colab.

Its still pretty hard to see your changes. Can you run it locally?

Its still pretty hard to see your changes. Can you run it locally?

Yes, it seems the changes are clear now.


Empty error box shown on the node drawer for SLURM executor workflows

Example file
`from covalent_slurm_plugin import SlurmExecutor
import covalent as ct
executor = SlurmExecutor(
ssh_key_file="~/.ssh/beehive",
username="sankalp",
poll_freq=3,
address="beehive.agnostiq.ai",
remote_workdir="/federation/sankalp/workdir",
options={"partition": "debug", "cpus-per-task": 2, "nodelist": "beehive-debug-st-t2medium-1"},
)
@ct.electron(executor=executor)
def join_words(a, b):
return ", ".join([a, b])
@ct.electron(executor=executor)
def excitement(a):
return f"{a}!"
@ct.lattice
def simple_workflow(a, b):
phrase = join_words(a, b)
return excitement(phrase)
dispatch_id = ct.dispatch(simple_workflow)("Hello", "World")
print(dispatch_id)`

@Aravind-psiog Can you take a look at this script and run them in your local and see the output once.

I believe this can be closed.

Thanks @arosen93 just getting back to closing all old issues !


auto conversion of string to int8 arrays in buffer_from_vector makes debugging quite hard

I noted that buffer_from_vector converts every value to an array. This may be fine for actual data but makes debugging very hard if something goes wrong in a data loading pipeline.
Take the example form the docs:
import mlx.data as dx

def list_files(root: Path):
    files = list(root.rglob("*.jpg"))
    classes = sorted(set(f.parent.name for f in files))
    classes = dict((v, i) for i, v in enumerate(classes))
    return [
      {"file": str(f.relative_to(root)).encode(), "label": classes[f.parent.name]}
      for f in files
    ]

root = Path("/path/to/image/dataset")
dset = (
  dx.buffer_from_vector(list_files(root))
  .load_image("file", prefix=str(root), output_key="image")
)

The resulting dset will have entries where the dset[0]["file"] values look like:
array([ 73,  77,  71,  95,  54,  55,  51,  50,  46, 106, 112, 101, 103],
      dtype=int8)

When running into an error, it is very hard to tell which actual file causes problems.
Is there a good way to at least transform the int8 array back to a readable string? How do you handle string encoding when converting back and forth?

just figured it out: .tobytes().decode('utf-8')


PR status is not propagating correctly with closed draft PRs

We use a 3rd party merge queueing system that will create a draft PR (which is never upgraded into an open PR) that batches multiple change sets together for CI evaluation. This draft PR contains the commit with the Issue-ID prefix and is linked to the PR.  The PR creation and merge commit actions are performed via an internal automation user and when that users closes the PR the state doesn't transition from Open in Jira to Closed or Declined.  The PR never enters the "open" PR state.
Am I correct in assuming that if a PR is closed it would show up as Declined in Jira?
This is blocking automation around closing issues as the development[pullrequests].open = 0 evaluates to false.

Is this a bug that draft PR state doesn't propagate correctly into Jira?

Is this a case of https://github.com/atlassian/github-for-jira/blob/main/SUPPORT.md#workflow-transitions-are-not-running. I briefly looked into this but the patch files for the merge commits are on behalf of the user (who has Jira permissions) and not the automation user.  Pr creation is done by the automation user though.  It would be strange to me that the creation event would get picked up but then the transition from Open to Declined.. I tested this theory and disproved it.

Is there a way to bypass this bug and target my automation around a single closed Pull Request? development[pullrequests].merged = 1 (this is invalid jql)... I am having trouble finding documentation around the schema for this field.

Found info related to #3 here https://support.atlassian.com/jira-software-cloud/docs/advanced-search-reference-jql-developer-status/. Doesn't seem like there is any way to target a single closed PR with jql syntax.

For clarity, the PR is always in a draft state and never upgraded to an open PR status?

@mboudreau Yes

@mboudreau do you need something else from me or can we remove the "awaiting-response" label?

I confirmed that #2 is not an issue.

If I can get some help triaging this issue I'd be happy to fix it

I would also be willing to contribute to this, if we could get a nudge in the right direction.

Hey @sholsapp and @michaellarocca90. We'd be happy for one of you to tackle this one. What do you mean by a nudge in the right direction?

@rachellerathbone what I meant was maybe a pointer to where in the code the maintainers would start. I've never worked on this codebase before so don't know what important objects, architecture patterns, gotchas, etc., exist. If there's an obvious starting point, that'd be great. 😄

Ping!

Important fix for us @rachellerathbone could we get a little help?

Hey all. We had someone on our team do some investigatory work on this one. Unfortunately, it looks like it would require changes in several locations (so not solely the GitHub for Jira integration) so this would need to be picked up by an Atlassian employee. I have created a ticket in our backlog.

@rachellerathbone was something done here? I believe I'm seeing a change on our end that PR's are getting marked as Declined

Hi @rachellerathbone , just curious where this issue sits on the roadmap. We're still needing to manually close all our tickets due to this issue

Hi @rachellerathbone, we recently saw ticket begin to autoclose again and wanted to check and see if you fixed something on your end.


Handle bad input / errors for Web local cluster creation

In the web UI it is possible to send negative integers or an empty string for the numWorker argument in local cluster creation. It seems like this could be handled on the React side but given there is already a mechanism to signal errors from Python concerning cluster creation/connection, I've added the necessary try/except and exceptions to notify about the bad inputs. As a bonus if we have other errors during creation of DaskJobExecutor this should at least propagate the exception to the web interface.
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[x] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed
[x] No import of GPL code from MIT code

/azp run libertem.libertem-data

/azp run libertem.libertem-data


Synology Active Backup for Business Agent 2.2.0-2070

[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?

Microsoft Reviewers: Open in CodeFlow


Custom width for each sidebar item.

When I'm on the "Explorer" tab, the required Width to browse and see items easily is not too wide, however, If I'm on another tab like, maybe source control, or AI chat assistant, I need something wider to see the content fully.
With the current VScode setup, when I change the width of one tab, read, and go to another tab, the width stays the same. So, now I need to resize the new tab again.
workflow example:
I'm on the Explorer tab and its 10rem width
I navigated to the AI assistant tab, and it's 10rem.
It's too narrow, so I made it wider, now it's 30rem.
My task is finished, I navigated back to the Explorer tab,
now my explorer tab is 30rem wide.
I resized it to 10rem back.
Now, I need to go back to AI assistant
make it 30rem again.
you get the point.

/duplicate of #158603


Support for html widgets

It would be nice to add support to the api so its possible to have a library of widgets / templates so that the user can drag an item from a widget gallery and drop it inside a content area. Today you can only drag & drop between existing content areas but it would be nice to add support to drag items from an library of predefined html and then drop it directly onto the content area.

Hi @andtii - this is something that is in the pipeline (though it's a long pipeline right now), it's a feature request that's popular and has been discussed in more detailed previously here https://github.com/GetmeUK/ContentTools/issues/13 (I think perhaps we should change the title of the issue to be more appropriate).
The idea is to create a ContentFlow library that allows for dynamic creation/management of the editable regions within a page from templates.
I'm going to mark this as a duplicate for now an close it but only because I just want the one issue regarding it open - and I realise that the existing issue wouldn't be very apparent currently so that needs to change.
Thanks for your interest :+1:  in CT
Duplucate of #13


Refactor input gatherer

still ingest all the series it needs to ingest from the series provider
but then has a second step that checks for post processing steps and instantiates each class in the right order from the dspec to post process the data collected

Notes

remove dspec parser into it's own class
three separate processes (noted in dspec task)

Notes/Improvements:

Pass only the series that postprocess call needs to compute the out keys when making the post process call rather than the entire input vector


Preventing Color Changes in drawio SVGs When Switching to Dark Mode

I am very grateful for your efforts. Excalidraw has always been a tool I am very satisfied with. However, I have recently encountered a small problem.
When I import a drawio SVG in excalidraw, switching to dark mode completely changes the color of the drawio SVG. Is there any way to prevent this from happening?

Dark theme is definitely something we aim to improve upon. Thanks for the feedback!


Improve documemntation

Is your feature request related to a problem?
I have been unable to find a clear explanation of how to deploy the espp component in my idf development environment.
Has this library been added to the Espressif Compnent register? I have been unable to find it.
It will be nice if the documentation starts with a little chapter explaining how to deploy the library.
Thanks!
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Hey @JaviVilarroig thanks for opening this issue - updating the documentation has def. been on my todo-list, so thanks for poking me about it 😅
I'll make some updates to the documentation shortly, but really quick to answer your question and provide some help until then:
You have a couple of options:

You can use the esp-cppp/template repository as a starting point, which is very similar to the esp-idf template, but is geared towards c++ development and already has espp as a submodule, along with the appropriate configuration in the top-level CMakeLists.txt.
If you have an existing project with a components directory, then you can clone espp as a submodule within that directory e.g. git submodule add https://github.com/esp-cpp/espp components/espp, then make sure to run git submodule update --init --recursive. Afterwards, simply update your CMakeLists.txt to add

# add the component directories that we want to use
set(EXTRA_COMPONENT_DIRS
  "components/espp/components"
)

You can clone espp somewhere on your computer and then point your project to its components directory to use any of the components it contains, similar to the step above.

The mention about the idf component registry is a good one - I don't personally use it much (preferring to use git submodules instead), but I'll add it as a todo to publish this as an idf-component so that its easier for others who use the registry to integrate into their projects.

Hi @finger563
Thanks for the fast answer :)
Any of the three answers looks OK, thanks :) I will try when I have a minute.
I really think is good to try the component registry, because it has a good integration with the idf environment and most newbies in the ESP32 world, like me, will go there first.
I my case I was already starting to develop my own library because I didn't found anything similar in the registry. I found this one when I was checking if the name espp was already taken in a search engine. :smile:
Thanks again!


setting height for image views has no effect

When getting network views there is an option to select the height of the image to be retrieved (http://idekerlab.github.io/cyREST/#2100447251). I have not been able to get this to have any effect.
Below are two responses I get when trying to set the image to be two different heights:
Try with h=2000
Response [http://localhost:1234/v1/networks/4290/views/first.svg?h=2000]
  Date: 2016-08-08 14:04
  Status: 200
  Content-Type: image/svg+xml
  Size: 1.48 MB
<BINARY BODY>

Try with h=4000
Response [http://localhost:1234/v1/networks/4290/views/first.svg?h=4000]
  Date: 2016-08-08 14:08
  Status: 200
  Content-Type: image/svg+xml
  Size: 1.48 MB
<BINARY BODY>

The images are exactly the same size and dimensions as if I just used the default:
Response [http://localhost:1234/v1/networks/4290/views/first.svg]
  Date: 2016-08-08 14:17
  Status: 200
  Content-Type: image/svg+xml
  Size: 1.48 MB
<BINARY BODY>

actually have figured out that "h" works for png but not for the svg and pdf.


🚀 Feature: Table redesign

🔖 Feature description
https://www.figma.com/design/OXLtrl1EAy885to6S69554/DocsGPT?node-id=0-1&t=Sav41zuHC2ts3pxh-1

🎤 Why is this feature needed ?
Better UI/UX and functionality
✌ How do you aim to achieve this?
Better UI/UX and functionality
🔄 Additional Information
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Are you willing to submit PR?
None

i would love to work on this issue please assign me

Hello, @AkashJana18.
Please how's it going with the issue?

Hello, @AkashJana18.
Please how's it going with the issue?

Working on it


Testing: added macros to expand functionality

Added false assertion, checks for !=, <, <=, >, >=
Added close checks for vectors and arrays

the ci is saying Testing.hpp:167:88: error: call of overloaded ‘abs(float)’ is ambiguous which I dont know how of course thats ambiguous.

I'm not sure why this only did the Appveyor check, since Travis is what complained.

I'm not sure why this only did the Appveyor check, since Travis is what complained.

Not sure, i was having trouble loading their website the other day, perhaps they are having issues.


add a version field to view configs

User story
Users try to input an older view config into a newer version of vitessce. We want to understand when this occurs to enable:

better warnings/errors
potentially translate automatically from an older schema to a newer schema by filling in missing fields with sensible defaults

Fixed by #728


Dev 1.7.x fixing validator issues

primary change
added php-cs-fixer as a tool, other files mainly changed space line ending etcs form php-cs-fixer