Update RadioButton content to use Border instead of Frame

MAUI has a bug whereby when the visual structure of a RadioButton is redefined using a ControlTemplate, the click event isn't recognised correctly on Android. This occurs when a Frame in the ControlTemplate prevents the GestureRecognizer from firing. See https://github.com/dotnet/maui/issues/6938 for more info.
When the Frame is replaced with a Border, the RadioButton works as expected. Therefore, update the RadioButton sample and doc so that it uses a Border rather than a Frame.

Associated WorkItem - 66540

I have a PR open for this currently, was not sure how to link it to this issue: https://github.com/dotnet/maui/pull/13407


Failed to Download

Hi, since yesterday we get an 403 when trying to use checkov action (-:
    - name: Terraform security check
        id: checkov
        uses: bridgecrewio/checkov-action@master
        with:
          directory: ${{ matrix.working_directory }}
          skip_check: CKV_AWS_65,CKV_AWS_51,CKV_AWS_23,CKV_AWS_85,CKV_AWS_40,CKV_AWS_52,CKV_AWS_21,CKV_AWS_18,CKV_AWS_107
          quiet: true # optional: display only failed checks
          framework: terraform
        continue-on-error: false

Failed to download action 'https://api.github.com/repos/bridgecrewio/checkov-action/tarball/60970bd282ce39baab17ecc3f6ccb67fd8b3bda3'. Error: Response status code does not indicate success: 403 (Forbidden).

@ArneRiemann4711 can you please try again now?

(-:
Warning: Failed to download action 'https://api.github.com/repos/bridgecrewio/checkov-action/tarball/60970bd282ce39baab17ecc3f6ccb67fd8b3bda3'. Error: Response status code does not indicate success: 403 (Forbidden).
Warning: Back off 28.948 seconds before retry.
Warning: Failed to download action 'https://api.github.com/repos/bridgecrewio/checkov-action/tarball/60970bd282ce39baab17ecc3f6ccb67fd8b3bda3'. Error: Response status code does not indicate success: 403 (Forbidden).
Warning: Back off 25.936 seconds before retry.
Error: Response status code does not indicate success: 403 (Forbidden).

Looks like it is working again !!


Money, Debt, and Consumer Issues is not showing as a parent topic

Money, Debt, and Consumer Issues is not showing as a parent topic
Version [17.1]
Reported by Alex Forger 1/7/2020

Hello, @KellyeDC fixed on dev:


Backport Adding tolerations to a workload via the UI does not work

https://github.com/rancher/rancher/issues/15585

This fix is already validated by @davidnuzik here:
https://github.com/rancher/rancher/issues/15585#issuecomment-450993838


(Enhancement) Kodi Krypton support for multiple IDs in NFO

Kodi added support for multiple identifiers for IDs in Krypton https://github.com/xbmc/xbmc/pull/10000. Can the movie NFOs be updated to include TMDB? Thank you!
<uniqueid type="imdb" default="true">(the ttid)</uniqueid>
<uniqueid type="tmdb" default="false">(the id)</uniqueid>

https://kodi.wiki/view/NFO_files/Movies

should already be possible in v3

this is a new style for ids which we do not support so far
we will include that in v3 since NFO parsing/writing is much more powerful there

Sound good, thank you!


TESTS: Some tests have shebangs

Some tests have shebangs. Many don't. Is there a particular reason for this? Just convenience so you don't have to type python test_foo.py? Or just because it once was like that?
https://github.com/scipy/scipy/blob/master/scipy/linalg/tests/test_lapack.py

I think it was there for convenience, but it's used rarely and so wasn't added to newer test files. Doesn't matter too much either way, although consistency is always nice.

Answered, so closing --- not an issue that needs to be necessarily tracked.


Could not spawn [C:\phantom\bin\phantomjs.exe]

Expected behavior
[Describe expected behavior here]
Actual behavior
[Describe observed behavior here]
Steps to reproduce the behavior

[First Step]
[Second Step]
[Other Steps...]

Environment

OS:
Node version (node -v):
package version:

Working perfectly without transpiling, however after transpile throws error:
Using webpack with target node.
error: Could not spawn [C:\phantom\bin\phantomjs.exe] executable. Please make sure phantomjs is installed correctly.
error: undefined {"errno":"ENOENT","code":"ENOENT","syscall":"spawn C:\phantom\bin\phantomjs.exe","path":"C:\phantom\bin\phantomjs.exe","spawnargs":["\\shim\index.js\"]}

Does C:\phantom\bin\phantomjs.exe exist? I am afraid you are on your own if I can't reproduce this. I am on a mac.

Oh sorry. I don't think I understood your problem. There isn't anything stopping you to transpile. I can't imagine what could be the problem because if you look at the source code, it's actually all written in flow and transcompiled to es6. If it is the path, then you can override most of these settings https://github.com/amir20/phantomjs-node/blob/master/src/phantom.js#L64.
I closed this issue because you didn't follow the template. If you want more help, then provide the answers to the template so that I can reproduce it locally.

Good, thank you!


Add support for dots in rtpmap encoding name

Addresses #44

Coverage remained the same at 90.741% when pulling 40a97dac6014043cdc8bf07346e5db38c3e22200 on alexanderklintstrom:master into 88453cde6cf775fe03ab27633beff89a38269ecf on clux:master.

Awesome. Thanks for doing that. Good tests, sensible extension.

Landed in 1.6.2


Create a complete guide

Write a well-documented README.md for software and hardware installation.

Done - the write-up is finished. Check the README.d


Reconstruction Loss

❓ Questions
Hello, my question is about reconstruction loss in frequency domain, in a paragraph 3.4 it is stated that you use "mel-spectrogram using a normalized STFT", what type of normalization is mentioned here? Is it sufficient to use normalized flag of
torchaudio.transforms.MelSpectrogram  which normalizes "by magnitude after stft"?
Also in practice stft loss is sometimes computed via log mel-spectrogram for better convergence, so I want to clarify, in your implementation, S_i from formula 1 is a mel-spectrogram or log mel-spectrogram?

Clarification for the first question - because here spectrogram is being normalized by the argument I think the answer on my question will be yes.

Clarification for the second question - in the formula number (5) in the SoundStream paper the log is being taken of the mel-spec when computing L2 part of the multi-scale spectral reconstruction loss, so the question remains, did you remove it on purpose?

And one more question about the multi-scale spectral reconstruction loss, when constructing MelSpectrogram with 64 n_mels and window_size < 512 I get the following warning (4 of them)
/opt/conda/lib/python3.10/site-packages/torchaudio/functional/functional.py:539: UserWarning: At least one mel filterbank has all zero values. The value for n_mels (64) may be set too high. Or, the value for n_freqs (65) may be set too low. Is it an expected behavior and I should leave this loss as it is?


Google Sign-In and MongoDB for Email Management

Describe the Feature you want to end
I will add a home page through which user can sigin through google or signup page,and if user already sigin through google or sigup page through same email id he will redirect to failure page
and also create Accepted and rejected mongodb database with name,email id and reason for rejecting
Expected behavior

User can singin from Google and sing page
than it will take email id and user name save in mongodb database under Accepted and rejected collections
if user try to use same id again it will redirect user to failure Page

@pranjay-poddar @MohitGupta121 @TusharKesarwani please assign me this


Compatibility with iojs 2.0.0

Please, check the compatibility with io.js 2.0.0.

fixed in v2.1.5
@albertosantini : tx!

Thanks for the super fast fix!


PackageId:CPython39.Exe.x64;PackageAction:Install;ReturnCode:1;

This issue has been moved from a ticket on Developer Community.

Describe your issue here
===DO NOT EDIT BELOW THIS LINE===
PackageId:CPython39.Exe.x64; PackageAction:Install; ReturnCode:1;

Original Comments
Feedback Bot on 4/19/2023, 00:21 AM:
(private comment, text removed)

Original Solutions
(no solutions)

Hi there, from reading your logs,
[389C:1FAC][2023-04-19T14:02:34]e000: Detected Windows 7
[389C:1FAC][2023-04-19T14:02:34]e000: Windows 8.1 or later is required to continue installation

This error message indicates that the installer has detected that you are running Windows 7, but the minimum system requirements for the software you are trying to install requires Windows 8.1 or later. Could you please upgrade your operating system to meet the minimum system requirements and try again. Please reopen the ticket if the issue persists. Thanks.


disabledDaysOfweek doesn't work

when i dont set disableddaysofweek everything works fine but as soon as i set it my whole app freezes

i did it just like the example in the documentation

Hi!! sorry about that, I am a little away from this project due to my current work.
Can you provide with a little more of code?

This issue is related with #312


update docker file

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.36%. Comparing base (040bb00) to head (1e56a47).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #261   +/-   ##
=======================================
  Coverage   82.36%   82.36%
=======================================
  Files          20       20
  Lines         845      845
=======================================
  Hits          696      696
  Misses        122      122
  Partials       27       27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

latestdepth test is fixed in PR https://github.com/alibaba/opentelemetry-go-auto-instrumentation/pull/260. So you can ignore it.


⚠ SceneTime has degraded performance

In f68d4ac, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 9696 ms

Resolved: SceneTime performance has improved in c68fa72.


Yaml support

Hey, there's also snakeyaml 2 which supports higher versions of YAML.
org.snakeyaml:snakeyaml-engine:2.7

I started with a Node based implementation
public final class SnakeJsonNode extends SimpleJsonNode {

    private SnakeJsonNode(Object node, String jsonPointer) {
        super(Objects.requireNonNull(node), jsonPointer);
    }

    public SnakeJsonNode(Object node) {
        this(node, "");
    }

    @Override
    public boolean asBoolean() {
        return Boolean.parseBoolean(((ScalarNode) node).getValue());
    }

    @Override
    public String asString() {
        return ((ScalarNode) node).getValue();
    }

    @Override public BigInteger asInteger() {
        return new BigInteger(asString());
    }

    @Override public BigDecimal asNumber() {
        return new BigDecimal(asString());
    }

    @Override
    public List<JsonNode> asArray() {
        List<JsonNode> elements = new ArrayList<>();
        for (Object o : ((SequenceNode) node).getValue()) {
            elements.add(new SnakeJsonNode(o, jsonPointer + "/" + elements.size()));
        }
        return elements;
    }

    @Override
    public Map<String, JsonNode> asObject() {
        MappingNode jsonObject = (MappingNode) node;
        Map<String, JsonNode> map = MapUtil.newHashMap(jsonObject.getValue().size());
        for (NodeTuple entry : jsonObject.getValue()) {
            String key = ((ScalarNode)entry.getKeyNode()).getValue();
            map.put(key, new SnakeJsonNode(entry.getValueNode(), jsonPointer + "/" + JsonNode.encodeJsonPointer(key)));
        }
        return map;
    }

    @Override
    boolean isNull(Object node) {
        return ((Node) node).getTag() == Tag.NULL;
    }

    @Override
    boolean isArray(Object node) {
        return node instanceof SequenceNode;
    }

    @Override
    boolean isObject(Object node) {
        return node instanceof MappingNode;
    }

    @Override boolean isBoolean(Object node) {
        return node instanceof ScalarNode && ((ScalarNode) node).getTag() == Tag.BOOL;
    }

    @Override
    boolean isString(Object node) {
        return node instanceof ScalarNode && ((ScalarNode) node).getTag() == Tag.STR;
    }

    @Override
    boolean isInteger(Object node) {
        return node instanceof ScalarNode && ((ScalarNode) node).getTag() == Tag.INT;
    }

    @Override
    boolean isDecimal(Object node) {
        return node instanceof ScalarNode && ((ScalarNode) node).getTag() == Tag.FLOAT;
    }

    public static final class Factory implements JsonNodeFactory {

        private final LoadSettings load;

        public Factory(LoadSettings load) {
            this.load = load;
        }

        @Override
        public JsonNode wrap(Object node) {
            if (node instanceof SnakeJsonNode) {
                return (SnakeJsonNode) node;
            } else {
                return new SnakeJsonNode(node);
            }
        }

        @Override
        public JsonNode create(String rawJson) {
            return new SnakeJsonNode(new Composer(load, new ParserImpl(load, new StreamReader(load, rawJson))).getSingleNode().orElseThrow());
        }
    }
}

and then I realized
it can load plain Java objects
public final class SnakeJsonNode extends SimpleJsonNode {

    private SnakeJsonNode(Object node, String jsonPointer) {
        super(node, jsonPointer);
    }

    public SnakeJsonNode(Object node) {
        this(node, "");
    }

    @Override
    public boolean asBoolean() {
        return (Boolean) node;
    }

    @Override
    public String asString() {
        return node.toString();
    }

    @Override
    public List<JsonNode> asArray() {
        List<JsonNode> elements = new ArrayList<>();
        for (Object o : (List<?>) node) {
            elements.add(new SnakeJsonNode(o, jsonPointer + "/" + elements.size()));
        }
        return elements;
    }

    @Override
    public Map<String, JsonNode> asObject() {
        @SuppressWarnings("unchecked")
        Map<String, ?> jsonObject = (Map<String, ?>) node;
        Map<String, JsonNode> map = MapUtil.newHashMap(jsonObject.size());
        for (Entry<String, ?> entry : jsonObject.entrySet()) {
            String key =entry.getKey();
            map.put(key, new SnakeJsonNode(entry.getValue(), jsonPointer + "/" + JsonNode.encodeJsonPointer(key)));
        }
        return map;
    }

    @Override
    boolean isNull(Object node) {
        return node == null;
    }

    @Override
    boolean isArray(Object node) {
        return node instanceof List;
    }

    @Override
    boolean isObject(Object node) {
        return node instanceof Map;
    }

    public static final class Factory implements JsonNodeFactory {

        private final Load load;

        public Factory(Load load) {
            this.load = load;
        }

        @Override
        public JsonNode wrap(Object node) {
            if (node instanceof SnakeJsonNode) {
                return (SnakeJsonNode) node;
            } else {
                return new SnakeJsonNode(node);
            }
        }

        @Override
        public JsonNode create(String rawJson) {
            return new SnakeJsonNode(load.loadFromString(rawJson));
        }
    }
}

One important thing to note:
I'm using a similar factory for validating YAML against a schema.json file, and while writing the above 2 classes, I ran into a potential problem: the Factory is used to parse the input AND the schema. I don't think this is desired as the Yaml documents (.yaml) would still have JSON Schemas (.json). We're extremely lucky that JSON actually parses as YAML, BUT potentially only if it's well-indented?

Hi there

Hey, there's also snakeyaml 2 which supports higher versions of YAML.

Yeah, I know but I was under the impression that org.yaml:snakeyaml is more widely used so I decided to go for that one. But any feedback on that is appreciated :)

it can load plain Java objects

I believe that's the approach I went with on the PR? TBH working directly with "Nodes" from the lib seemed way better to me but then I realized that all that fancy stuff like aliases and references won't work in such setup (not 100% percent sure though).

I don't think this is desired as the Yaml documents (.yaml) would still have JSON Schemas (.json).

Interesting point, I would assume that all stuff will be written in YAML. But I believe any valid JSON is also a valid YAML? I even depend on that behavior slightly as meta-schemas are only provided in minified JSON format - and it seems to work fine.
I'm still delaying merging YAML provider - the PR seems kind of ready but I'm really not sure if I didn't overlook something. So it's good to get any feedback on that, thanks :)

So it's good to get any feedback on that, thanks :)

I guess above is some feedback :) I won't be using that class even if you release it, because I'm slightly coupled with snakeyaml 2 and it works for now. Writing the custom wrapper wasn't that complicated (amazing API btw). I was only sad about having to copy-paste computeNodeType.

I was under the impression that org.yaml:snakeyaml is more widely used so I decided to go for that one

I'm using snakeyaml 2 because it has a better object model, I want to write a YAML formatter soon, so having all the node stuff is useful. I guess there's no harm in implementing both?

aliases and references won't work

No clue, have heard that these exists, but never seen it :)
If they exist and work on GitHub Actions, I would definitely use them in some places to reduce "code" duplication.

I believe that's the approach I went with on the PR?

Yeah looking at my code above and yours in the PR, it's pretty similar.

But I believe any valid JSON is also a valid YAML?

https://news.ycombinator.com/item?id=30052128, TL;DR

I even depend on that behavior slightly as meta-schemas are only provided in minified JSON format - and it seems to work fine.

I guess these assumptions will work until you add support for, or someone tries to parse JSON5 schema as YAML. I think the best would be if there was an API to override how to parse schemas.

Interesting point, I would assume that all stuff will be written in YAML.

Nope. The data is YAML, but why would we convert schemas into YAML, perfectly fine in JSON. In the past, we even used JSON schemas to validate Kotlin/Swift data classes.

I'm using snakeyaml 2 because it has a better object model, I want to write a YAML formatter soon, so having all the node stuff is useful. I guess there's no harm in implementing both?

Yeah, I just don't want to add provider to every possible JSON/YAML lib. I'm thinking about having a separate module for additional stuff (maybe even whole YAML support could end up there).

Nope. The data is YAML, but why would we convert schemas into YAML, perfectly fine in JSON. In the past, we even used JSON schemas to validate Kotlin/Swift data classes.

Having 2 JsonNodeFactory instances responsible for different things would probably require some significant refactors. Also this would overcomplicate APIs - not necessarily a bad thing, just a price for more features and flexibility. But I agree that it may be worthwhile to implement, gonna explore it as a 2.0.0 candidate :)

@TWiStErRob Just FYI I've decided to go with the solution with 2 node factories. It turns out that having a JSON indented with tab instead of spaces causes YAML parser to fail. Honestly, I doubt this behavior is mandated by YAML 1.1 spec or anything but as snakeyaml lib behaves like this I am forced to do something with it.
And TBH this change turned out to be really small and straightforward: https://github.com/harrel56/json-schema/pull/182
Hoping to ship it in the next minor release (with YAML support as well), probably in a next few days.

Nice, thanks for the heads up! I have some feedback, will comment on the PR.


Fixed #24895 -- Fixed loading a pair of squashed migrations with a dependency.

buildbot, add to whitelist.

Looks good besides more blank lines in the test than I'd opt for and missing release notes if you plan to backport to 1.8.

Thanks for reviews! Removed a blank line in the test, converted the new test and the one right before it to use set literals (the only tests in this file that needed that conversion), and added release note for 1.8.3. Will merge when CI is green.


Collapsing the file tree

Collapsing the tree when encrypting or decrypting a file inside.
2022-07-25 10-12-04.webm

Fixed.


Worn Item's sprites are teleported to (0,0) location on unequip

if unequipping items from character's inventory, their bounding box is set in actual location of the item, while sprite gets under the character teleported to (0,0) location, making item impossible to pick up by clicking the sprite.

Fixed in https://github.com/space-wizards/space-station-14/commit/ab1dfed25cd91602f24af69d2c06c1c8c89238d8
Just need a submodule update here.


setup.py

Hi,
have you thought about setting up a setup.py? This would enable us to do something like:
from abstractionlayer.cleaning_api import install_tools
install_tools()

This would remove the necessity to copy cleaning_api.py into every new project :)
Best regards,
Felix

Hi,
That is a good idea. Yes, we need to do that. Thanks.
Kind regards,
Mohammad

As we discussed, I agree with having "setup.py". However, it remains as a TODO for later. :)


Specifying additional metrics when running HaProxy exporter doesn't seem to have any effect on the metrics reported.

Specifying -haproxy.server-metric-fields doesn't seem to have any effect on the list of metrics being collected by the haproxy exporter,
Based on the  help documentation, we got the list of metrics from here
So when we run the exporter with the following commands,
docker run -d -p 9101:9101 prom/haproxy-exporter -haproxy.scrape-uri="http://<User>:<Password>@wordpress.wordpress.mysite.com:8010/haproxy?stats;csv" -haproxy.server-metric-fields="1,2,3,4,7,12"
docker run -d -p 9101:9101 prom/haproxy-exporter -haproxy.scrape-uri="http://<User>:<Password>@wordpress.wordpress.mysite.com:8010/haproxy?stats;csv"
The list of metrics seems to be the same,  we still don't metrics like max connections/request queue size etc.

--haproxy.server-metric-fields is there to let you whitelist server-level metrics to avoid too many timeseries. If you want to add new metrics, that requires code changes.
Does that address your problem?

hi @brian-brazil
ty, If thats the case, when if we don't do anything, should we be seeing all the metrics listed here, we don't seem to be seeing all the metrics then.
Also, we are using the Interlock HaProxy, https://github.com/bfirsh/interlock-haproxy-swarm.
Here's the metrics that we are seeing currently
go_gc_duration_seconds{quantile="0"} 0.000380405
go_gc_duration_seconds{quantile="0.25"} 0.00039399000000000004
go_gc_duration_seconds{quantile="0.5"} 0.00040280700000000003
go_gc_duration_seconds{quantile="0.75"} 0.00041959100000000003
go_gc_duration_seconds{quantile="1"} 0.0006179180000000001
go_gc_duration_seconds_sum 0.205200134
go_gc_duration_seconds_count 496
go_goroutines 16
haproxy_exporter_csv_parse_failures 0
haproxy_exporter_total_scrapes 734
haproxy_up 0
http_request_duration_microseconds{handler="prometheus",quantile="0.5"} 21230.955
http_request_duration_microseconds{handler="prometheus",quantile="0.9"} 21886.518
http_request_duration_microseconds{handler="prometheus",quantile="0.99"} 41909.256
http_request_duration_microseconds_sum{handler="prometheus"} 1.6067188661999993e+07
http_request_duration_microseconds_count{handler="prometheus"} 733
http_request_size_bytes{handler="prometheus",quantile="0.5"} 294
http_request_size_bytes{handler="prometheus",quantile="0.9"} 294
http_request_size_bytes{handler="prometheus",quantile="0.99"} 294
http_request_size_bytes_sum{handler="prometheus"} 215866
http_request_size_bytes_count{handler="prometheus"} 733
http_requests_total{code="200",handler="prometheus",method="get"} 733
http_response_size_bytes{handler="prometheus",quantile="0.5"} 866
http_response_size_bytes{handler="prometheus",quantile="0.9"} 870
http_response_size_bytes{handler="prometheus",quantile="0.99"} 872
http_response_size_bytes_sum{handler="prometheus"} 634222
http_response_size_bytes_count{handler="prometheus"} 733
process_cpu_seconds_total 1.87
process_max_fds 524288
process_open_fds 8
process_resident_memory_bytes 1.1718656e+07
process_start_time_seconds 1.43878871601e+09
process_virtual_memory_bytes 2.6300416e+08

haproxy_up 0

I'm guessing you've got a network problem there, can you check you can get to the haproxy from the exporter container?

@brian-brazil
Thank you, that looks like the issue. Will keep you posted as to what we find.
Here's where I'm right now
msg="Error while scraping HAProxy: Get http://stats:interlock@wordpress.wordpress.mysite.com:8010/haproxy?stats;csv: dial tcp <IP_ADDRESS>:8010: connection refused" file="haproxy_exporter.go" line=244
Added DNS entry to fix connection errors, Now getting this error.
msg="Error while scraping HAProxy: Get http://stats:interlock@wordpress.wordpress.mysite.com:8010/haproxy?stats;csv: read tcp <IP_ADDRESS>:8010: i/o timeout" file="haproxy_exporter.go" line=244

@brian-brazil
We were able to resolve the issue, we had a problem with the url as well.
Thank you for your help.

Glad to help.


Corporate Console>>Request new CLA manager:Pop up title should be “Request new CLA manager” instead of  “Request CLA manager Request”

Summary
Text Request is displayed multiple times in pop up title
Expected behavior
Pop up title should be “Request new CLA manager” instead of  “Request CLA manager Request”
Screenshots

This page has been completely redesigned.


fix the Vagrantfile

the provisioning script fails with the message:
cannot create regular file '/var/lib/coreos-vagrant/
not a directory
This change provides a clean installation.

It looks like you're trying to use cloud-config while using the VirtualBox provider, which is currently unsupported. Please use Ignition instead.

I did mv user-data.sample user-data first by mistake, not understanding the difference between cloud-config and ignition. Once I read through the docs properly and figured it out, I created the ignition file as instructed with ct and ran vagrant up. I also received the error message about not being able to create the file. I wound up going into the Vagrantfile and commenting out this block entirely:
      #if File.exist?(CLOUD_CONFIG_PATH)
      #  config.vm.provision :file, :source => "#{CLOUD_CONFIG_PATH}", :destination => "/tmp/vagrantfile-user-data"
      #  config.vm.provision :shell, :inline => "mv /tmp/vagrantfile-user-data /var/lib/coreos-vagrant/", :privileged => true
      #end```

So, the part about the file not having any effect even if it does exist isn't quite correct. It causes the process to error out and even though the first vm starts up, if you've specified multiple instances the others won't.

Yeah, originally it didn't have an effect, but then we changed the way we handled some of the configuration. We plan to move VMWare over to Ignition in the near future, and then we'll completely remove that block


How do I get a terminal size bigger than 80x30

I tried to start tmux with a larger size by passing in -x and -y but the output on play still shows it as 80x30.. asciienma on it's own picks up the size of my terminal.. reaching out before spending more time on this, hoping for a quick hack.

Hello,
I am aware of the problem and unfortunately it is still eluding me at the time. I suspect subprocess.Popen is the culprit, but even trying to set the column and row sizes as environment variables didn't work. Please let me know if you have any success with this.

Thanks for quick response.. all my attempts to trigger tmux to set the width and height have not paid off.. now looking into asciienma itself.. really cool project though! Keep up the good work!
Ref: https://github.com/vmware/docker-volume-vsphere/wiki/Demos
I have forked the repo and made some changes to not backspace the line output.. I think an alternative can exist to line in the yaml where at the end a cntrl C is sent.. more on that later.

Yeah, subprocess is the problem.  There's no terminal so tmux tries to be helpful by defaulting to a smaller size.  I can't figure out how to turn that off, so this mostly works, still working out some issues
-        self.reel = Command('tmux new-session -d -s %s' % self.session_name)
+        self.reel = Command('tmux new-session -d -s %s -x 160 -y 40' % self.session_name)

-        movie = subprocess.Popen(asciinema_cmd % (self.session_name,
-                                                  self.output_file),
-                                 stdout=subprocess.PIPE,
-                                 stderr=subprocess.PIPE,
-                                 shell=True)
+        full_asciinema_cmd = asciinema_cmd % (self.session_name, self.output_file)
+        movie = pexpect.spawn(full_asciinema_cmd, dimensions=(40, 160))

fyi, I am also looking at changing the 'backspace over input' thing.  I just changed it to send
'# ' + line + enter

which works pretty well.  I may end up refactoring the whole thing to remove most of the classes and simplify things, but swapping out subprocess for pexpect fixes most of the immediate problem

Nice find! Would you mind sending a patch for review on https://softwarefactory-project.io when ready? Let me know if you need help with setting up your review.


[BLAS][EXAMPLE] Enable RT example for portBLAS backend

Description
This PR enable the RT example for portBLAS. It requires more cmake checks due to the library usage.
Update selectors in CT blas example to SYCL2020.
Checklist
All Submissions

[x] Do all unit tests pass locally? Attached example log.
portblas_cuda_example.txt
mklcpu_cublas_examples.txt
portblas_default_example.txt

[x] Have you formatted the code using clang-format?

Looks good to me and thanks for the example outputs! I will leave it to more knowledgeable people to approve however.


file transports log messages out of order

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs:
8.15.1
Operating System? (Windows, macOS, or Linux)
Windows 10/Ubuntu 16.04/macOS
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart)
ES6

What is the problem?
console messages are in correct order, but the same messages written to disk are not
Logger.js
const winston = require('winston');
const _ = require('underscore');
const { format, transports } = winston;
const { combine, colorize, timestamp, json, simple, printf } = format;

/**
 * @param {string} identifier - the identifier to use as a message prefix
 */
class Logger {
	constructor(identifier) {
		this._identifier = identifier;
		this._logger = winston.createLogger({
			transports: [
				new transports.Console({
					level: 'info',
					format: combine(colorize(), simple())
				}),
				new transports.File({
					name: 'demo-json',
					filename: 'demo.json',
					level: 'info',
					format: combine(timestamp(), json()),
					options: { flags: 'a' }
				})
			]
		});
	}

	/**
	 * Logs a message
	 * @param {(string|Object)} message - the message, Error, or JSON object
	 * @param {string} [level=info] - the level for the log message
	 */
	log(message, level = 'info') {
		if (message instanceof Error) {
			this._logger.log(level, `[${this._identifier}] ${message.stack}`);
		} else if (typeof message === 'string' || typeof message.toString === 'function') {
			this._logger.log(level, `[${this._identifier}] ${message}`);
		} else {
			this._logger.log(level, `[${this._identifier}] ${JSON.stringify(message, null, '\t')}`);
		}
	}
}

const loggers = [];

module.exports = {
	/**
	 * @param {string} identifier - the identifier to use as a message prefix
	 */
	getLogger: (identifier) => {
		const loggerInfo = _.findWhere(loggers, { identifier: identifier });
		if (loggerInfo) {
			return loggerInfo.logger;
		} else {
			const logger = new Logger(identifier);
			loggers.push({ identifier: identifier, logger: logger });
			return logger;
		}
	}
};

test.js
const Logger = require('./Logger');
const items = [
	{
		logger: Logger.getLogger('One'),
		messages: ['Hello', 'Goodbye']
	}, {
		logger: Logger.getLogger('Two'),
		messages: ['Bonjour', 'Au revoir']
	}
];

items.forEach((item) => {
	item.logger.log('First message');
	item.messages.forEach((message) => item.logger.log(message));
});

console output
c:\code\temp>node test.js
info: [One] First message
info: [One] Hello
info: [One] Goodbye
info: [Two] First message
info: [Two] Bonjour
info: [Two] Au revoir

demo.json output
{"level":"info","message":"[Two] First message","timestamp":"2019-08-22T15:09:16.233Z"}
{"level":"info","message":"[One] First message","timestamp":"2019-08-22T15:09:16.230Z"}
{"level":"info","message":"[Two] Bonjour","timestamp":"2019-08-22T15:09:16.234Z"}
{"level":"info","message":"[One] Hello","timestamp":"2019-08-22T15:09:16.231Z"}
{"level":"info","message":"[Two] Au revoir","timestamp":"2019-08-22T15:09:16.234Z"}
{"level":"info","message":"[One] Goodbye","timestamp":"2019-08-22T15:09:16.232Z"}

What do you expect to happen instead?
the messages in demo.json should show up in the same order they hit the console especially given that they each have a correct timestamp
Other information

The "issue" here was that I should have been using the same transport to write to the same file -- here's my corrected sample code (if anyone comes across this in the future):
const winston = require('winston');
const _ = require('underscore');
const { combine, colorize, timestamp, json, simple, printf } = winston.format;

const consoleTransport = new winston.transports.Console({
	format: combine(colorize(), simple())
});

const fileTransports = [];
const getFileTransport = (options) => {
	const transportInfo = _.findWhere(fileTransports, { filename: options.filename });
	if (transportInfo) {
		return transportInfo.transport;
	} else {
		const transport = new winston.transports.File(options);
		fileTransports.push({ filename: options.filename, transport });
		return transport;
	}
};

/**
 * @param {string} identifier - the identifier to use as a message prefix
 */
class Logger {
	constructor(identifier) {
		this._identifier = identifier;
		this._logger = winston.createLogger({
			transports: [
				consoleTransport,
				getFileTransport({
					name: 'demo-json',
					filename: 'demo.json',
					level: 'info',
					format: combine(timestamp(), json()),
					options: { flags: 'a' }
				})
			]
		});
	}

	/**
	 * Logs a message
	 * @param {(string|Object)} message - the message, Error, or JSON object
	 * @param {string} [level=info] - the level for the log message
	 */
	log(message, level = 'info') {
		if (message instanceof Error) {
			this._logger.log(level, `[${this._identifier}] ${message.stack}`);
		} else if (typeof message === 'string' || typeof message.toString === 'function') {
			this._logger.log(level, `[${this._identifier}] ${message}`);
		} else {
			this._logger.log(level, `[${this._identifier}] ${JSON.stringify(message, null, '\t')}`);
		}
	}
}

const loggers = [];

module.exports = {
	/**
	 * @param {string} identifier - the identifier to use as a message prefix
	 */
	getLogger: (identifier) => {
		const loggerInfo = _.findWhere(loggers, { identifier: identifier });
		if (loggerInfo) {
			return loggerInfo.logger;
		} else {
			const logger = new Logger(identifier);
			loggers.push({ identifier: identifier, logger: logger });
			return logger;
		}
	}
};

Having the same issue with winston 3.8.2 when using different loggers created by winston.createLogger using a new instance of transports.File for each. Each logger instance injected (via InversifyJS) into service-class instances that use them for logging and logging is done in async/await'ed methods on those services-classes.
While the console transport logs messages in the correct order, the file transport fumbles the ordering of events, especially for different log-level.
Example - Console:
[2022-12-02 11:15:45][info]:<enter loop>
[2022-12-02 11:15:45][info]: Fetching LDAP-Person with Matriculation-Number: 007
[2022-12-02 11:15:45][info]: Found LDAP-Person with DN: Matrikelnummer=s007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:15:45][info]: Found matching LDAP-Person with DN: Matrikelnummer=s007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:15:45][info]: Fetching Person-Relation with personId: 007
[2022-12-02 11:15:45][info]: Found Person-Relation matching given uuid with relationId: 007
[2022-12-02 11:15:45][info]: Fetching LDAP-Identity with UUID: 007
[2022-12-02 11:15:49][info]: Found LDAP-Identity with DN: UUID=007,ou=People,ou=Identities,ou=Idm,o=My University,c=DE
[2022-12-02 11:15:49][info]: Fetching 1 LDAP-Persons for given linkDNs
[2022-12-02 11:15:49][info]: Fetching LDAP-Person with linkDN: Matrikelnummer=007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:15:49][info]: Found canonical address<EMAIL_ADDRESS>[Email]; <NONE> [Phone]; <NONE>, <NONE>, <NONE> [Post]
[2022-12-02 11:15:49][info]: Fetching Electronic-Addresses for HIS-Person with personId: 007
[2022-12-02 11:15:49][info]: Fetching Post-Addresses for HIS-Person with personId: 007
[2022-12-02 11:15:49][info]: Found 1 Email-Addresses, 0 Phone-Numbers & 1 Post-Addresses
[2022-12-02 11:15:49][info]: Managing Email-Address<EMAIL_ADDRESS>for HIS-Person with personId: 007
[2022-12-02 11:15:49][info]: Working with existing Email-Address: [007]<EMAIL_ADDRESS>[STU, STUALL, RZK, DOK, RZ, DOKALL]
[2022-12-02 11:15:49][info]: Found 1 existing matching canonical Email-Address(es) having category RZ, selecting one with addressId: 007
[2022-12-02 11:15:49][info]: <exit loop>
[2022-12-02 11:15:49][info]: <enter loop>
[2022-12-02 11:15:49][info]: Fetching LDAP-Person with Matriculation-Number: 008
[2022-12-02 11:15:49][error]: Trying to manage Addressses for studentId 008, got exception: Could not find any result for DN ou=People,o=My University,c=DE (Filter: (&(objectClass=Student)(Matrikelnummer=s008)), Scope: sub)
[2022-12-02 11:15:49][info]: <exit loop>

Example - File:
[2022-12-02 11:25:11][info]: <enter loop>
[2022-12-02 11:25:11][info]: Fetching LDAP-Person with Matriculation-Number: 007
[2022-12-02 11:25:11][info]: Found LDAP-Person with DN: Matrikelnummer=s007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:25:11][info]: Found matching LDAP-Person with DN: UniMrMatrikelnummer=s007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:25:11][info]: Fetching Person-Relation with personId: 007
[2022-12-02 11:25:11][info]: Found Person-Relation matching given uuid with relationId: 007
[2022-12-02 11:25:11][info]: Fetching LDAP-Identity with UUID: 007
[2022-12-02 11:25:12][info]: Found LDAP-Identity with DN: UUID=007,ou=People,ou=Identities,ou=Idm,o=My University,c=DE
[2022-12-02 11:25:12][info]: Fetching 1 LDAP-Persons for given linkDNs
[2022-12-02 11:25:12][info]: Fetching LDAP-Person with linkDN: UniMrMatrikelnummer=s007,ou=Students,ou=People,o=My University,c=DE
[2022-12-02 11:25:12][info]: Found canonical address<EMAIL_ADDRESS>[Email]; <NONE> [Phone]; <NONE>, <NONE>, <NONE> [Post]
[2022-12-02 11:25:12][info]: Fetching Electronic-Addresses for HIS-Person with personId: 007
[2022-12-02 11:25:12][info]: Fetching Post-Addresses for HIS-Person with personId: 007
[2022-12-02 11:25:12][info]: Found 1 Email-Addresses, 0 Phone-Numbers & 1 Post-Addresses
[2022-12-02 11:25:12][info]: Managing Email-Address<EMAIL_ADDRESS>for HIS-Person with personId: 007
[2022-12-02 11:25:12][info]: <exit loop>
[2022-12-02 11:25:12][info]: Fetching LDAP-Person with Matriculation-Number: 008
[2022-12-02 11:25:12][info]: Working with existing Email-Address: [007]<EMAIL_ADDRESS>[STU, STUALL, RZK, DOK, RZ, DOKALL]
[2022-12-02 11:25:12][info]: Found 1 existing matching canonical Email-Address(es) having category RZ, selecting one with addressId: 007
[2022-12-02 11:25:12][info]: <enter loop>
[2022-12-02 11:25:12][error]: Trying to manage Addressses for studentId 008, got exception: Could not find any result for DN ou=People,o=My University,c=DE (Filter: (&(objectClass=Student)(Matrikelnummer=s008)), Scope: sub)
[2022-12-02 11:25:12][info]: <exit loop>

As can be seen the output between the second <exit loop> and <enter loop>  loop is jumbled in the file output, but is ok in the console output. So this is not caused by a missing async -> await in my code.


Error While Pushing Commits to Enterprise GitHub

https://github.company.com/pravin-pande/test-repo.git: git-upload-pack not permitted on 'https://github.company.com/pravin-pande/test-repo.git/'
I get above error while trying to push commits to GitHub.
@serge-rider  I thought this was taken care of in #5581 as you mentioned in #5666

I am using latest Egit, DBeaver 6.1.5 on Windows 10

seems to be fixed.
if it is still actual - feel free to reopen the ticket


Document the required secrets on the README

      FLOWZONE_TOKEN:
        required: true
      NPM_TOKEN:
        required: true
      DOCKER_REGISTRY_USER:
        required: true
      DOCKER_REGISTRY_PASS:
        required: true
      BALENA_API_KEY_PUSH:
        required: true
      BALENA_API_KEY_TEST:
        required: false
      COMPOSE_VARS:
        required: false

Included in https://github.com/product-os/flowzone/pull/56


kpt.dev/latest-revision label doesn't work well with controllers

We use the kpt.dev/latest-revision label to tell clients which PackageRevision is the latest for a package. This works well when users use the LIST command so we can always provide a snapshot of all PackageRevisions from the same point in time. But for controllers that rely on watches to build a local cache of PackageRevisions, this doesn't work, since publishing a new PackageRevision that becomes the latest one doesn't notify the clients that the kpt.dev/latest-revision label on other revisions are no longer correct.
One possible solution would be that we send watch updates for any PackageRevisions that are no longer the latest and thereby make sure that controller caches get updated correctly. However, this will lead to a race that can lead to controllers for short period of times can either have two PackageRevisions marked as latest, or have no PackageRevision marked as latest. This depends on whether we chose to send the watch events for the new or previous latest PackageRevision first. Also, this approach is difficult to manage with CRDs, since it doesn't give us a good way to do cross-PackageRevision logic.
So I think our best alternative is to drop the kpt.dev/latest-revision label and rely on clients to use the revision field to determine which PackageRevision is the latest. After https://github.com/GoogleContainerTools/kpt/pull/3593, this should be pretty straightforward to handle in the client.

I think asking the client to use the revision field to determine the latest revision is reasonable. Perhaps we could provide a library function in the kpt repo that does the bulk of the logic, for clients to call?


Performance tracking using Event Timing API

Context
TBD

Summary

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[ ] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #9328

NB: we use snake_case everywehre for telemetry, not camelCase.

snake_case for name sounds right, though the documentation examples use English sentences e.g. "JS Dependencies". I don't really mind either way.


no scrolling with revealjs output

Hello,
I looked at your example and in the html format you can scroll on the code if it is too long to be show on the screen. However, it isn't possible for the revealjs format.
Is it a bug?

Can you provide me with a reproducible example?

When I look at the same demo, I can see the scrollbar but only if I don't activate the fullscreen. If I activate it I don't see the scrollbar and only the first 45 lines and a part of the line 46 of the code are show. I also can't scroll down.
I'm on windows 11 and I'm using firefox 110.0.1 to view the demo.

Okay, now I am facing the same issue you mentioned while viewing the demo with firefox. But it is working well with google chrome. Thanks for reporting. Let me investigate the issue.

@PierreSolomon, install the extension again. The raised issue should be fixed for Firefox now 😊. Let me know if there are still problems.

It is fixed on my side and there doesn't seem to be any problem.
Thank you for the extension


Publish to local Docker

While we want things to work if you don't have Docker installed, as an inner dev loop scenario folks will surely want "make it super easy to run the thing I just built", so we should have a "publish to local Docker" option.
The only way I currently see to do this is via docker load, which takes as input a tarball in the format produced by docker export, which doesn't appear to be documented but is basically hash-named objects in a tarball.
Jib has this and mentions that it requires installed Docker, so I suspect they're doing the same thing.
Questions:

What happens if the tarball doesn't have the the base layer tarballs inside it? Do we have to download them so we can produce the tarball?
Is there any other way to do this?
Should we expose this tarball as an artifact, or create it transiently in memory and stream it to docker load?

We'll need this in order for folks to have any kind of success with our initial previews, so adding it to the preview7 milestone.


zero length array in C99

Any particular reason for using vtable[0] instead of vtable[] here, since std is chosen to be gnu11 in Makefile?

Just habit :)


Improve reconcilation performance for external dashboards

Improves reconcilation time on druid based dashboard having 45 dims and 30 measures from 50 sec to 4 sec.

@nishantmonu51 main change is its doing all dims and measures in one go and if there are any errors then go one by one on them to find actual failing one ?

Yes, its trying to improve reconcilation time by doing the check in one shot rather than one by one. Prepares one query instead of 70.


Fixing Attached File Matcher

@wbigal can you please fix the conflicts? I notice you you upgrade the paperclip dependency. Are you sure is that fine?

I am going to decline this pull request as I don't think this change belongs here.
If I am wrong please let me know in the comments.


Transition back to CSS styles

I can create a transition that will correctly get applied from a CSS style to an in-line style. For example say I create a circle:
var c1 = svg.append("circle")
            .attr("class", "red")
            .attr("r", 25)
            .attr("cx", 50)
            .attr("cy", 50);

with some CSS
.red {
   fill: red;
}

I can change the style of this circle to blue using a transition and this works fine.
svg.selectAll("circle")
   .transition()
   .delay(2000)
   .duration(2000)
   .style("fill", "blue");

What I really want to be able to do, is then revert this style by simply removing it:
c1.transition()
  .delay(5000)
  .duration(2000)
  .style("fill", null);

This will cause a sudden switch back to red, because I've not defined the colour. What I really would like to be able to do is transition back to the default style from the CSS without having to record the colour red so I can apply it inline to revert back nicely.
Question + Sample on StackOverflow: http://stackoverflow.com/questions/29701689/transitions-styles-back-to-css-defaults#29702008

The nearest I've got to making this work is to go re-acquire the style of the element. So to go back to red:
c1.transition()
  .delay(5000)
  .duration(2000)
  .style("fill", function(d) { return d3.select(this).style("fill"); });

This is better than trying to save the colour before making it blue (although I'm kinda surprised this code works). I wonder if this suggests a change to style may be useful? Something like (pseudo-ish code):
style = function(name, value) {
   if(value === null || value === undefined) {
     value = d3.select(this).style("name");
  }
  // apply value
}

This would make it much nicer to use - although I'm not sure on the performance hit of selecting this for each instance changing?

You could also capture the value immediately before starting the transition, like this:
c1.transition()
    .style("fill", function() {
      var that = d3.select(this),
          currentFill = that.style("fill"),
          defaultFill = that.style("fill", null).style("fill"); // retrieve default fill
      that.style("fill", currentFill); // restore current fill
      return defaultFill;
    });

Perhaps it would be nice if transition.style(name, null) did this for you by default: i.e., computed the style property that you’d get from removing the inline style, and then transitioned to it automatically before removing the inline style on transition end. That seems reasonable.

@mbostock yeah, I've played around with solutions like that (although not quite so generic) which made my code a little verbose. I think I considered re-defining style at some point to add an approach to do this, but it does feel like it'd be a nice addition to core D3.

This has been implemented in d3-transition for 4.0.

:+1: Just wanted to say thank you @mbostock for looking at this - it helps eliminate more styling information in our JavaScript code, putting it back to its rightful place in CSS land and just makes D3 an even better tool to be working with :)


Killed Instances not Cleaned up

$ qstat -f
<EMAIL_ADDRESS>BIP   0/1/1          -NA-     lx-amd64      auo
    103 0.55500 bip_runner ec2-user     dt    07/22/2015 00:56:13     1
---------------------------------------------------------------------------------
<EMAIL_ADDRESS>BIP   0/1/1          -NA-     lx-amd64      auo
    105 0.55500 bip_runner ec2-user     dt    07/22/2015 00:58:28     1

Please provide more details when opening an issue. By default sqswatcher should cleanup any instances terminated. If this is not happening, it suggests something is wrong.

What about if the instance is killed manually in the console?
What about if it is a spot instance that is killed?

All instances that are terminated, either manually, due to spot conditions or via nodewatcher, should be handled by sqswatcher. If this is not the case, please test with the latest version and report an issue with exact reproduction steps.

@dougalb I am experiencing the same issue.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.9 to before v5.4.135",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a02b9dc90d844cc7df7b63264e7920cc425052d9",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=41f45e91c92c8480242ea448d54e28c753b13902"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "a02b9dc90d844cc7df7b63264e7920cc425052d9",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "41f45e91c92c8480242ea448d54e28c753b13902",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "dma-buf/sync_file: Don't leak fences on merge failure\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.135 by commit 41f45e91c92c8480242ea448d54e28c753b13902, it was introduced in version v4.9 by commit a02b9dc90d844cc7df7b63264e7920cc425052d9. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001318


Add Player Vars to High Score Mode

This updates the high_score.py file to pass through both the player_num and the category of the award.  This is pushed through a new unique event (to prevent breaking existing games). The high_score.yaml file is updated as well to show how to use these new features to create a new specific slide for the awards where player and machine variables should be shown.

I don't know if we care about those two fails.  I didn't touch that file, and it's odd it failed both Windows and Mac for 3.8 but passed the rest.  I will leave that up to someone wiser than myself if it needs addressed, and how to address it.

Nice!


Error: application.js:39 Cannot read properties of undefined (reading 'url')

Steps to reproduce;

Change the API endpoint to use staging API or local API (most like you won't get this error on production, figure out why later 😉)
Just start the frontend server and you should see this error in console

Hey Swaraj, I want to take this up.


Add qtcharts for pyside6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

We are currently using QtCharts in our applications, but it is not supported in the existing setup. Therefore, we need to include the QtCharts DLLs to ensure that our applications continue to function properly with charting capabilities.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

It should work already. Just install qtcharts in your environment

We were able to successfully download the qt6-charts package separately, so I will close this PR as it is no longer required. Thanks for the support!


Decouple ids from record content

The current procedure uses IDs to filter out existing records but a variety of problems have arisen from this approach including:

duplication of IDs.
new provider coding (JH_HIT "_masks" to "_mask") which changes ID values.
Changes in provider IDs which break this system (CDC, ACAPS).

The purpose of the ID should be to uniquely identify a record (and the records preceding and following a certain record) but I believe should be decoupled from the task of identifying the content of a record.
Other methods should be developed for identifying duplicate records and for identifying new additions to the dataset (those that have not yet been cleaned), which are then merged in the master dataset.
Curious to hear thoughts on this.

A case where it is valuable to have the ID refer to record content is where important information has been changed in a cleaned record that makes it look different from a previous version of itself.
i.e. changing a record country from Sudan to South Sudan
In this case, an unchanged version of the record should be retained in order to identify incoming records as duplicates of the original.

Proposed workflow for a new update dataset:

Generate combination strings for update data and update data -1.
Identify new records between update data and update data - 1.
Return subset containing only new records.
Append this subset to the manually cleaned data.

Done in part - for OXCGRT data. other datasets will require more time to handle coding discrepancies and misaligned ids. Closing for now.


V1.17

Description
Updates the to the latest SNAPSHOT of the 1.17 API
Checklist
The following items should be checked before the pull request can be merged.

[X] Version number updated in settings.gradle.
[X] Unit tests added. - NONE REQUIRED
[X] Code follows existing code format.

I would like to adjust the title to V1.17.1 because it maybe looks like a duplicate due to the project is already on 1.17.


CKO: Changes added to handle cko-calico flavor configuration

Added a new task file which run cko calico side changes via acc-provision
Added the new net-operator-controller deployment alongwith the CRD and CR in templates folder.
Dockerfile acc-provision branch changes

It's a First Draft. Kindly review with all possible comments


arduino-ci: init at 0.1.0

arduino-ci allows you to quickly add continuous integration (CI) tests
to Arduino libraries. It uses arduino-cli to install dependencies and
then it compiles every example with every board.
I am the primary author of the arduino-ci project.

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.


Link pointing to wrong page

In the second paragraph of the "Working with CSV and JSON data in Azure" section of the web page found at the link https://docs.microsoft.com/en-us/azure/architecture/data-guide/scenarios/csv-and-json, there is an "import CSV files" link.  Instead of taking you to a page about importing CSV files it takes you to the "Import JSON documents into SQL Server" page.(https://docs.microsoft.com/en-us/sql/relational-databases/json/import-json-documents-into-sql-server?view=sql-server-ver15)

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

ID: c5ac957c-9568-b3f0-d79e-72a36439bee9
Version Independent ID: e89413dd-8815-3d38-d7f0-6a6d0268db1e
Content: Processing CSV and JSON files - Azure Architecture Center
Content Source: docs/data-guide/scenarios/csv-and-json.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @ZoinerTejada
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 403563

Removed link.


Tellform.com is not working.

Hi,
I tried to open tellform.com to learn more about it and try its demo templates but seems like its not working.
Can you please look into it and resolve the issue.
Thanks

fixed


Track statistics of components of RJMC acceptance probability

It would be useful for us to store the components of the RJMC acceptance probability to help us figure out how to optimize these moves later.
If we have the statistics for all moves stored in a log, we can later analyze the data to compute things like the log acceptance probability of every component,
log <A> = log <min{1, exp[-w]}>

If we also store the identities or indices of the proposed switch (e.g. from species 3 -> 7), we can later use BAR to reconstruct pair free energies, or an approximate Bayesian pairwise version of BAR to get a better estimate of relative free energies.

I agree completely. I was actually planning on implementing a Sampler class of some sort that would automate all of these things, including saving to a file (I was thinking NetCDF), and allowing a resume feature.

If we also store the identities or indices of the proposed switch (e.g. from species 3 -> 7), we can later use BAR to reconstruct pair free energies

Yes, this sounds really useful!

an approximate Bayesian pairwise version of BAR to get a better estimate of relative free energies.

Sounds really interesting!

I like the idea of a driver Sampler class which can automate much of the machinery.
Ideally, we could have a hierarchy, like
Sampler (base class)
   + MCMCSampler
      + AdaptiveWeightSampler
      + FixedWeightSampler
   + BayesianBanditsSampler
      + ThompsonSampler
      + (other strategies)

We could, of course, simply have a flag that turns weight adaptation on/off in MCMCSampler instead of having two derived classes.

I'm also still wondering if there isn't a way to have an MCMCSampler that couples together several parallel chains and uses some sort of weight adaptation algorithm that couples the chains together and provably converges to more complex target distributions, like a marginal distribution proportional to the resistance index Z_{P_k L_ATP} / Z_{P_k L_I}.

I've also emailed Zhiqiang Tan (CCing you) about my thoughts on extending the recursion scheme for handling the more complex target sampling case where we want pi_k \propto Z_{P_k L_ATP} / Z_{P_k L_I}.

Samplers are implemented.


feat: support OTel 0.27

~Waiting for https://github.com/tokio-rs/tracing-opentelemetry/pull/184~ now released

Looks like it's failing to build with that feature enabled 🤔
Can you take a look?

Pushed!

Thanks! Can you push a tag to crates.io?


Crash when fetching prices for 1inch

/Users/adedade/poly-flashloan-bot/dist/inchPrice.js:30
        amount.toString() +
               ^

TypeError: Cannot read properties of undefined (reading 'toString')
    at get1inchQuoteCallUrl (/Users/adedade/poly-flashloan-bot/dist/inchPrice.js:30:16)
    at /Users/adedade/poly-flashloan-bot/dist/inchPrice.js:53:31
    at Generator.next (<anonymous>)
    at fulfilled (/Users/adedade/poly-flashloan-bot/dist/inchPrice.js:5:58)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

I am on the latest master

Should we also enable better stacktraces since they reference compiled js instead of ts sources? e.g. https://www.npmjs.com/package/source-map-support

My bad, I did forget to run yarn build once pulled the latest changes from master :) will keep an eye out if I encounter any other issue


How to use real world latitude and longitude for or-tools BuildOrder in C#

private void BuildFleet(int number_of_orders,
                        int number_of_vehicles,
                        int x_max, int y_max,
                        int end_time,
                        int capacity,
                        int cost_coefficient_max) {
    Console.WriteLine("Building fleet.");
    vehicle_capacity_ = capacity;
    vehicle_starts_ = new int[number_of_vehicles];
    vehicle_ends_ = new int[number_of_vehicles];
    vehicle_end_time_ = new int[number_of_vehicles];
    vehicle_cost_coefficients_ = new int[number_of_vehicles];
    for (int vehicle = 0; vehicle < number_of_vehicles; ++vehicle) {
      int index = 2 * vehicle + number_of_orders;
      vehicle_starts_[vehicle] = index;
            int latVehicle = (int)(13.0033656 * 10000000);
            int lngVehicle = (int)(80.2597263 * 10000000);
     locations_[index] =
          //new Position(random_generator.Next(x_max + 1),
          //             random_generator.Next(y_max + 1));
            new Position(latVehicle,
                       lngVehicle);//todo:hardcoded
            vehicle_ends_[vehicle] = index + 1;
      locations_[index + 1] =
            //new Position(random_generator.Next(x_max + 1),
            //             random_generator.Next(y_max + 1));
            new Position(latVehicle,
                       lngVehicle);
            vehicle_end_time_[vehicle] = end_time;
      vehicle_cost_coefficients_[vehicle] =
          random_generator.Next(cost_coefficient_max) + 1;
    }
  }

here position is int but real world position will be latitude and longitude.How to use real world latitude,longitude for BuildOrders and BuildFleets.
FileName:cscvrptw.cs

You can use Fixed point arithmetic to transform your decimal latitude/longitude into integer (e.g. multiplie your values by 10000000).
You can see it, in your example...
int latVehicle = (int)(13.0033656 * 10000000);
int lngVehicle = (int)(80.2597263 * 10000000);


Installation fails if node-gyp does (the fallback busy-wait isn't available)

I just tried this out as a dependency for a project, but found that installation of this module fails if the C++ compiler is insufficient. I found a different backup approach that isn't quite as good, but I thought I'd report it here in case you wanted to remove the fallback code (or could tell me how to utilize it). Here's some output that explains the install error:
https://travis-ci.org/nfischer/shelljs-plugin-sleep/jobs/155675245
This only occurs for the version of gcc installed on travis. Clang works fine.

The fallback uses 100% CPU and shouldn't really be used. If you don't have an insufficient C++ compiler you can't compile any nodejs modules or nodejs itself.
You can tell travis to use a more recent version of g++ using something like:
env:
  - CXX=g++-4.8
addons:
  apt:
    packages:
      - g++-4.8

Sounds like you could get away with trimming out the fallback code in that case :smile: Either way, I just wanted to make sure I wasn't making an error with the fallback code, so I can close this now.

I agree so I just remove the fallback code.


babel dependencies

Hello @jrichardlai,
Do you think it's possible to get rid of all babel dependencies ? Why are they needed ?

Hey @FaridSafi! Yeah I can do that, they are probably not needed :)

Let me know if v0.0.15 works

@FaridSafi is this good for you now?

@FaridSafi did you had time to take a look?

Sorry for late reply, yes it works great! Thanks @jrichardlai

Next step will be to remove of all of them :D
Or maybe there is a preset for whats needed

babel-preset-react-native-stage-0 does that cover everything?


DRAFT pull request for discussion of new carbon cycle implementation

I added the new structure for the handling of atmospheric CO2 and connected it to the atmospheric CO2 concentration that is used in the rest of the model. The first results are promising, the fit to calibration data is as good as with the FAIR implementation. At the end of the century, the atmospheric CO2 concentration is a bit lower with the new implementation.
I kept the old structure in for reference. We can remove it in the next commit. The new structure accounts for the selected climate case, just as the old structure, via the initial value of the C reservoir.
TODO: The initial ocean C reservoirs, as well as the initial pH values for pCO2 calculation probably have to have different values depending on the selected climate case. Otherwise there will be initialisation shocks at the start of the integration. However, this would require an offline analysis using FAIR time series to find the best initial values for each climate case.

@lnnrtrmm FYI -- a few more emissions sources will be modified inside of Temperature.itmx I'll let you know when we're done touching things in there.

@lnnrtrmm If you need me to get these changes in sync with the mainline. model let me know.  I am happy to help.

Not a problem, I'll rebase once Murali has added his stuff


Want to add solution to Problem Statement of Leetcode 827. Making A Large Island

https://leetcode.com/problems/making-a-large-island/

please assign me this issue

please assign me this issue

You can work on this issue


Set up ktlint and swift-format

Aim
To avoid wasting time during PRs for mundane things like indentation and other stuff that don't matter.

ktlint is enforced for all kotlin code in PR github actions
swift-format is enforced for all swift code in PR github actions

Scope

Apply ktlint plugin to every project

perhaps create a convention plugin to not copy code too much

Run both ktlint and swift-format on the whole codebase

add that commit to .git-blame-ignore-revs

This will ensure our codebase is consistent (desirable for such a big open-source project)

detekt, + SwiftLint


Breakpoints in MXML files

Cannot set a breakpoint anywhere in an MXML file (not even in the Script section).

Yeah, the compiler doesn't generate source maps for MXML yet.
https://issues.apache.org/jira/browse/FLEX-35186

The FlexJS compiler? What about mxmlc for Flex 4.6 (because that's what I'm using on this project)?

Yes, source maps are not generated for MXML in FlexJS compiler. Only ActionScript right now.
SWF debugging is completely unsupported in the extension right now. That's my next big TODO item.

Closing because breakpoints are now supported for MXML for both SWF projects and Apache FlexJS 0.8 projects.


Nice framework. I need just an information.

What is the difference between cache and disk?
in short, image which are cached and saved to disk?

UIImage is stored to memory cache. Image data (Data) is stored to disk cache.
Is it what you need?

Yes, Thanks


convert_zero123_to_diffusers error

After run  diffusers/scripts/convert_zero123_to_diffusers.py , i got a dir like this.

But when i want to use it in diffusers like this, it is report an Error.
pipe = StableDiffusionControlNetImg2ImgPipeline.from_pretrained( 	"./model/zero1to3",  	controlnet=controlnet, torch_dtype=torch.float16, 	local_files_only = True )

ValueError: Pipeline <class 'diffusers.pipelines.controlnet.pipeline_controlnet_img2img.StableDiffusionControlNetImg2ImgPipeline'> expected {'text_encoder', 'scheduler', 'feature_extractor', 'safety_checker', 'vae', 'controlnet', 'unet', 'tokenizer'}, but only {'scheduler', 'feature_extractor', 'controlnet', 'vae', 'unet'} were passed.

Any help would be greatly appreciated!

Can this be loaded only by pipeline_zero1to3?

Hi, yes it should be loaded by pipeline_zero1to3

Thank you!


Qformer layer impl.

Summary: as title
Reviewed By: ankitade
Differential Revision: D46674420

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.01 :warning:

Comparison is base (ea3830b) 68.62% compared to head (2ba54de) 68.61%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #432      +/-   ##
==========================================
- Coverage   68.62%   68.61%   -0.01%
==========================================
  Files         169      169
  Lines       11339    11338       -1
==========================================
- Hits         7781     7780       -1
  Misses       3558     3558

Impacted Files
Coverage Δ

tests/modules/layers/test_multi_head_attention.py
100.00% <100.00%> (ø)

...hmultimodal/modules/layers/multi_head_attention.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


UX: spurious output for uv tool upgrade -p 3... --all

I just tried upgrading all my tools with the following command (which worked great !), and was curious to see what happened if I ran it twice in a row. I was pleasantly surprised that it correctly figured out nothing needed to be done instead of re-doing it, but also (very) mildly disappointed by the second message in the output:
❯ uv tool upgrade -p 3.13 --all
... # output not relevant here
❯ uv tool upgrade -p 3.13 --all
Nothing to upgrade
Upgraded tool environment for  to Python 3.13

This is an extremely minor annoyance which I hope is not blocking to anyone (and certainly isn't to me !), though when the user experience is consistently excellent, this level of detail still gets noticed. I also thought it might also be relatively easy to fix.

Thank you, definitely!

25 seconds:
$ curl -s -H "Authorization: token $TOKEN" "https://api.github.com/repos/astral-sh/uv/issues/9869" |  jq -r '"ID\tUsername\tCreated_At", "\(.id)\t\(.user.login)\t\(.created_at)"'
ID      Username        Created_At
2738499379      neutrinoceros   2024-12-13T14:15:00Z

$ curl -s -H "Authorization: token $TOKEN" "https://api.github.com/repos/astral-sh/uv/issues/9869/comments" |  jq -r '.[] | "\(.id)\t\(.user.login)\t\(.created_at)"'
2541555890      charliermarsh   2024-12-13T14:15:25Z
2541564001      neutrinoceros   2024-12-13T14:19:39Z

hahaha, well it felt even quicker


Add C++ starter kit

PR for #5

Good point, I'll add a readme later today. Also I thought about some smaller changes to avoid unnecessary copies, which I'll also add.

Added/modified everything as discussed above. It'd be great if you could proof read the readme. Also let me know if there is anything missing / not clear / wrong (i made some assumptions about kaggle). This  is just what I thought would be helpful to have in the readme.

All looks good to me for now. I think we will probably change the readme a little bit for clarity on bot submission as well in addition to building it.


Forecast in App Not Matching Darksky

The forecast in the app is not matching the data displayed on Darksky's site.  It appears to be averaging the temp between the high/low ((44F + 55F)/2 = 49.5, app displays 49F as the temp).  If I check the actual values on Darksky both through the UI and the API (by plugging in the appropriate Longitude/Latitude coordinates) everything looks correct.  But, again, through the app it appears as the average of the High and Low temps.

I found in the "ForecastIOService.java" file where it's doing this:
    int intTemp = (f.getTemperatureMin().intValue() + f.getTemperatureMax().intValue()) / 2;

The average temp doesn't provide any value if you swing from a high of 79F and a low of 50F (avg of 64.5F) which isn't an uncommon occurrence.  I removed the calculation to simply display the high temp.  I might reconfigure the app to display both the high and the low.


Incorrect archive URL

Ran into this when making the dune release:
https://github.com/ocaml/opam-repository/pull/12622#issuecomment-421535403

Fixed by #84


Send transaction confirmation dialog

Targets #567

add ability to run dialogs (I hope I did not reinvent a wheel)
I wasn't able to add proper component tests. I cannot pass anything to store via configureAppStore due to injected reducers pattern (probably). This throws warnings in other places as well, but due to --silent flag it's not visible. Dispatching updateValidators did not resolve that issue. I am missing something probably 🤷♂

Thanks!
I squashed some commits and merging


Support $count system query option

When adding $count=true to URL parameters for entity set collection, response should include<EMAIL_ADDRESS>element (for json), stating number on elements in the collection.
OData V4 spec for $count.
Important note: returned count MUST NOT be affected by $top, $skip, $orderby, or $expand.
Example request: http://services.odata.org/V4/Northwind/Northwind.svc/Categories(1)/Products?$count=true&$top=3

Comrades!
I'm working on this enhancement, and there is problem I'm trying to solve:
QueryOperationStrategy returns result of query. For collection, it's a list of entities.
To fix it, I might just use List#size to add @odata.count value in renderer, but according to specification I can't do that, because $top and $skip should not affect value of the $count. Entities collection already trimed by this time.
Which leads me to next thought: result of query execution should pass both: collection and value of the count for it.
So after considering different options, I'm leaning towards following approach:
QueryOperationStrategy#execute Instead of returning untyped Object, should return typed model object which might say if it's simple value, an entity, a collection or a collection with count value (set of case classes I guess).
So any places which work with query result will be updated to use that typed object, instead of doing instanceof. (Processor and renderer projects so far, as I can see.)
Does it make sense to you? How would you solve it within a framework?
/cc @renarj @vinayknl

I think the solution in the framework should be that the QueryOperationStrategy returns a result object with some metadata about the actual result. This could be a simple metadata object containing some 'key/value' kind of metadata. Or perhaps in some cases like count we could add strongly typed metadata properties there.
So QueryOperationStrategy would return a 'Result' object that contains a Map of String/Object metadata and a list of result row Objects.
Also keep in mind that you do not want to do a count(rows) DB query by default as this could be a costly operation. So the DataSource itself needs to be able to support and detect that a $count operation was part of the query and then add this to the metadata result.

Sounds good!

Resolved in #14


Remove TIMESTAMP from dayofweek Spark function

In Spark dayofweek doesn't directly accept TIMESTAMP input. For TIMESTAMP input, Spark will add a cast expression to convert it to date type, so only DATE type can be considered in Velox.
Also removing alias dow.
Spark function doc - https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.functions.dayofweek.html
Addresses #8736

@mbasmanova can you please review? Thanks!


Some errors I found when I try use it in my project

Thanks for this nice library! I really want to use it in my project. However I have few errors when I try to use it, since it's alpha version, it's okay I guess. Let me share issues with you:

I can't use "0dip" anywhere.
I can't use "dp" size in any text view (instead of "sp"), I know I should use "sp" but in some cases I need to use "dp", e.g. when I don't want text size change depended user settings in Android.

Regards.

@barisahmet Is 0dip working when you replace 0dip with dp. Im not sure, but is dip the same like dp?

Yes, sorry about that. It was same, my bad.

Im working currently on a way to support all different layout attributes. Im not sure if this will be possible, but it would be very nice.

Yes, this project needs support from community, the idea is amazing.

To Init the object with the AttributeSet it's not possible, becuase it's a xml parserobject, so the pull Parser would parse it at runtime.

But I add in the next Release Support for Dip, because when it's possible to declare it in the xml it should be supported. I will also take a look into the dp Support for textView for example.


Do we want to support procfiles?

We have discussed app manifests but not procfiles.

@poy, in your experience how important are these? And/or what features are most important?
If I remember correctly, they can be used to "just" point to the app that should be running but they can also be used to spin up multiple processes in a single container. I'm not convinced we want to support the latter because it seems like an anti-pattern but maybe the former?

I agree we don't want to support multiple processes.
The other use case is to denote if the app is a "web" or "worker". I don't know that knative serving supports a "worker" type app yet.

I'm pretty sure they do not, when I looked into this earlier I was thinking we'd have to map them to Kubernetes Jobs.

Cool, I'll close this for now then!


Binding issue with _@ character combination

I am getting a binding error when attempting to bind a razor template that has an email address in it using the following structure:
<EMAIL_ADDRESS>The email address itself is a valid one, but the RazorEngine is throwing an exception saying "somewhere" does not exist in the model. It appears the underscore character just before the @ symbol in the email address is causing it to think @somewhere is an attempt to bind to a model property. Does anyone know of a way around this?

Can you use @Raw like documented here?

Try putting two @@. At least it works within string variables.

I am unable to escape the sequence because the email address is piped into a razor template prior to me getting it for binding.  While the suggestions above would fix it if I had control over the value, I do not.  Is there a reason _@ triggers the binding and is treated as a model property?

@spotvader That's just how the razor parser works/is defined. You need to provide a template with valid syntax. You either need to fix the part where you pipe the value into the template or post-process the template to get a valid one...

I ended up post-processing the template to replace _@ with _@@, then do the razor biding, then set _@@ back to<EMAIL_ADDRESS>A little wonky in my opinion, but it does work.

Glad you were able to solve it :)


Create basic implementation of Properties Manager

Unsure if you'd like implementation of something similar to last year's listProperties() or not.
No implementation of any getting has been actually made yet, as that would break the code (there is no file to find) and that would be no fun.
You had mentioned a double HashMap implementation, not sure if that was for storing values. Also not sure if you want value storage in general, but the Properties class extends HashTable extends HashMap, so that should be fine.
The rationale behind the current form of PropertiesManager.java is that all calls just send a single String, and all property names are initialized as said Strings in PropertyNames and used from there, exactly as TelemetryNames does so.
Referencing issue #30.

I won't be able to make it for much of robotics today, but I'll be there at the end in time to discuss this with you for implementation on Sunday. Sorry for the late notice - everything got pushed back for rehearsal with the snow day. I think I understand what you're saying about iterating them into maps based on subsystem/sensor and returning some sort of wrapper. I'll try to get some changes to you this morning for more feedback.

Sadly, all of the next two weeks are tech for the shows next weekend and the weekend after, so I'll be doing all my work from home - rehearsal is 3-9 every day.


[Bug]: Using Manager to create repo's creates uneditable repos.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When you create a repo using PipelineManager and OperatorManager, you are unable to authenticate push requests as the repo is tied to the temporary access token that was used to create it. The resulting error is a 403 when pushing.
Expected Behavior
Create a repo with manager, push to that repo with hub.towhee.io credentials
Steps To Reproduce
1. Create a repo with PipelineManager
2. Clone the repo
3. Change files
4. Add, Commit, Push
5. Receive 403 error.

Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS):
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

@Chiiizzzy


The following 2 cases can't be detected

AAC 64kbps encoding.
OBS recording audio output, audio enhancement is enabled by default on windows 11 system.

Hi @codelive , could you elaborate more on this ?

Hi @antoine-tran,
Here's my test code:
from audioseal import AudioSeal

import torch
import torchaudio
def watermark_embed():
    model = AudioSeal.load_generator("audioseal_wm_16bits")
    audio, sample_rate = torchaudio.load("input.wav")
    audios = audio.unsqueeze(0)
    bits = [[1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0]]
    secret_mesage = torch.tensor(bits, dtype=torch.int32)
    print(f"bits: {secret_mesage}")
    watermarked = model(audios, sample_rate=sample_rate, message=secret_mesage, alpha=1)
    watermarked_audio = watermarked.detach()
    torchaudio.save("output_seal.wav", src=watermarked_audio[0], sample_rate=sample_rate)

def watermark_detect():
    audio, sample_rate = torchaudio.load("output_seal.wav")
    audios = audio.unsqueeze(0)
    detector = AudioSeal.load_detector(("audioseal_detector_16bits"))
    result, message = detector.detect_watermark(audios, sample_rate=sample_rate, message_threshold=0.5)
    print(f"bits: {message}, score: {result}")

# watermark_embed()
watermark_detect()

First call watermark_embed() to save a watermarked audio file "output_seal.wav"
bits: tensor([[1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0]], dtype=torch.int32)
The second step calls watermark_detect() to detect a watermark
bits: tensor([[1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0]], dtype=torch.int32), score: 1.0
Call ffmpeg to transcode the watermarked file to aac at 64kbps, and then convert the aac to wav.

ffmpeg -y -i output_seal.wav -b:a 64k output_seal.aac
ffmpeg -y -i output_seal.aac output_seal.wav

bits: tensor([[0, 1, 1, 1, 0, 0, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0]], dtype=torch.int32), score: 0.6068740487098694
The watermark detection score is about 0.6, which does not match the original watermark.

Record the system's sound output using OBS and then detect it.
bits: tensor([[1, 0, 1, 0, 0, 0, 1, 1, 0, 1, 1, 1, 1, 0, 0, 1]], dtype=torch.int32), score: 0.202103391289711
Screenshots for obs recording and system audio settings:

I turned off auto enhancement and then recorded again with obs. The detection score is about 0.9, but it still doesn't match my original watermark.
bits: tensor([[1, 0, 1, 0, 1, 0, 0, 1, 1, 1, 1, 1, 1, 0, 0, 1]], dtype=torch.int32), score: 0.9000480771064758

The wav file I used:
input.zip
Thank you very much for your reply.

Thanks for raising this up!
I don't see any way of solving this without fine-tuning the model with these augmentations... Tell us if you've found anything.


[MB-8856] refactor prime api demo script

Description
This PR refactors the prime-api-demo script into the pricing-acceptance script that works in the same way. The big difference is that this script creates a new shipment so instead of starting with customer onboarding, you can start with an existing move that has been approved (you'll need either the move id or move locator code)
Reviewer Notes
I discovered that the create-upload endpoint is broken while working on this. The CLI works without uploading a document when creating a payment request. However, I will create a ticket to update the script once the create-upload endpoint is fixed
Setup
pricing-acceptance RDY4PY local

You will need to create a payload for the service item that you want to create and price and will be prompted by the CLI to input the path to this payload json file
When prompted you will need to sign in to the office view as a TIO to approve the payment request
Before the script exits, you should see an EDI generated for the service item you created
Acceptance
Run this CLI against staging (first find an approved move in staging and pass the move code or ID to the cli)
pricing-acceptance RDY4PY api.stg.move.mil

References

Jira story for this change
documentation for this change

Messages

:book:
:link: MB-8856

  Generated by :no_entry_sign: dangerJS against ee672ea9b597f8595974962908714e150241ca92

Found a couple of very minor things, but otherwise this LGTM. I was able to follow the instructions and create a service item and see EDI in the end -- much quicker than before! 🚀
Note that there are still a few prime-api-demo references (first, second) in the Wiki that we should probably clean up since that script is being deleted here.

Thanks, I made the code and documentation changes


Dpys

*Simplify DSN
See most recent discussion:
https://github.com/nipy/dipy/pull/1900
1 additional tweak may still ultimately be needed if this doesn't raise discriminability, so let's keep an eye...
@dPys

you merged when travis didn't pass my dude, it shows native_dsn broken in this branch


[community][cohere] Handle when documents are not provided in the Cohere response

Description: This handles the cohere response when documents aren't included in the response
Issue: N/A
Dependencies: N/A
Twitter handle: N/A

thanks @BeatrixCohere!


can.List.replace events typo

The first events example claims this will be the output...
// 0, set, Adam, Alexis
// 1, set, Ben, Bill
// 0, remove, undefined, ['Alexis', 'Bill']
// 0, add, undefined, ['Adam', 'Ben'] <- Typo

In reality it should be this...
// 0, set, Adam, Alexis
// 1, set, Ben, Bill
// 0, remove, undefined, ["Alexis", "Bill"]
// 0, add, ["Adam", "Ben"], ["Alexis", "Bill"] <- Correction

Per this example: http://jsfiddle.net/akagomez/jfebph4e/

Interesting: while that is what the current code does, does it make sense for the current code to do that in the first place?
If you have a sequence of an add and a remove event where the new value of the remove event is undefined, then shouldn't the old value of the add event that logically happens afterwards also be undefined?

@rjgotten That seems logical to me also.
@justinbmeyer @daffl Is there a specific reason why oldVal isn't undefined in can.list.prototype.splice when the add event is fired?
if (howMany > 0) {
	// tears down bubbling
	bubble.removeMany(this, removed);
	this._triggerChange("" + index, "remove", undefined, removed);
}
if (args.length > 2) {
	// make added items bubble to this list
	for (i = 0, len = added.length; i < len; i++) {
		bubble.set(this, i, added[i]);
	}
	this._triggerChange("" + index, "add", added, removed);
}

https://github.com/bitovi/canjs/blob/master/list/list.js#L340

@rjgotten I'm going to rephrase your thought a bit, but correct me if I'm wrong.
A confusing note is that this example is listening to the "change" event, which has new and old values passed as arguments. However, "remove" only passes elements that have been removed in the event, in the place of the old values argument.
var list = new can.List(['a']);

list.bind('change', function(o, ev, type, val, old) {
  console.log(type, { new: val, old: old });
});

list.replace(['b']);

//console output
//"remove" does not have any new elements added to this collection, as such is undefined
//remove { new: undefined, old: ['a'] }
//"add" does have new elements adding to this collection
//add { new: ['b'], old: ['a'] }

Seems like the docs have it right, but the code has it wrong.  The oldVal argument with an "add" event should probably be undefined.


[Feature] Reduce app size using Android R8

Purpose

Reduce application size using Android R8 code shrinking and obfuscation tool

Does this introduce a breaking change?

[ ] Yes
[ X] No

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

[ ] Bugfix
[ X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

build and run like normal

What to Check
Verify that the following are valid

can join a call

@phileo99 In my opinion, we do not need to apply size reduction to our demo application. Our goal was to write document to share how the size can be reduced.
Applying size reductions will cause any issue in future? (like, skipping few files on SDK update)

@phileo99 In my opinion, we do not need to apply size reduction to our demo application. Our goal was to write document to share how the size can be reduced.
Applying size reductions will cause any issue in future? (like, skipping few files on SDK update)

@iaulakh most, if not all of our customers that publishes to Google Play store are doing so with code obfuscation enabled (either Proguard or R8).  Therefore we need to support code obfuscation as part of the app size reduction process.
Most of the heavy lifting to setup support for R8 has been done in this PR, any issues in future will be surfaced in bug bashes.
acs-ui-library.pro is inter-operable with both Proguard or R8

@ahammer done, I've added -dontobfuscate to acs-ui-library.pro

P4a Without Proguard

P4a With Proguard