Question: Working with child components

Is it possible to pass ngrx-forms state change from a child component to a parent component?
I currently have a shared form that is populated by a bunch of different ngrx-forms, I currently pass the ngrxFormState to the child and populate the form through a loop over the control.
I am able to pass the state to the child component for all the forms but I am not able to detect the changes in state, would I have to manually detect changes and pass them back through an emitted event and update the state back in the parent?
Example:
The shared component
<form class="form" [ngrxFormState]="form$ | async">

    <ng-container *ngFor="let control of ((form$ | async).controls  | keyvalue)">

        <ng-container [ngSwitch]="control.value.userDefinedProperties.controlOptions.controlType">

            <mat-form-field class="form__field">

                <!-- Label -->
                <mat-label [attr.for]="control.key">{{control.key | titlecase}}</mat-label>

                <!-- Input -->
                <input *ngSwitchCase="'input'" matInput [ngrxFormControlState]="control.value" required>

                <!-- The rest of the types -->
            </mat-form-field>

        </ng-container>

    </ng-container>
</form>

export class SharedFormComponent implements OnInit {
  @Input('form') form$: Observable<FormGroupState<any>>;
  ...
}

The parent component
<app-shared-form [form]="formState$"></app-shared-form>

export class Component implements OnInit {
  formState$: Observable<FormGroupState<FormState>> = this.store.select(selectForm);
  ...
}

I've just spotted the support for local form state which seems to be a solution although not tested it out yet.
https://ngrx-forms.readthedocs.io/en/master/user-guide/local-form-state/


Add inline R code to Rmd snippet

It would be great if we could add inline R code in R Markdown snippet like inserting R chunk.

There are some other useful snippets that I would like to add to Rmd snippet:

[ ] ^[Inline footnotes text].

[ ] Speaker notes (useful for presentations).

[ ] Side-by-side two column layout.

[ ] Reference list (this is useful when inserting a reference list fence into a custom location).

I will make a PR shortly.


[Snap] Error: failed to download modules: exec: "go": executable file not found in $PATH

What version of Hugo are you using (hugo version)?
hugo v0.88.1+extended linux/amd64 BuildDate=2021-09-04T13:52:13Z
Does this issue reproduce with the latest release?
Yes
Description
I just installed hugo via snap on my Ubuntu 21.10. When I try to run, it says Error: failed to download modules: exec: "go": executable file not found in $PATH. Then I installed go via apt  install golang-go and checked if my go path shows in my $PATH variable, and it does:
whereis go
go: /usr/bin/go /usr/lib/go /usr/share/go /usr/share/man/man1/go.1.gz

echo $PATH
/home/gca/.local/bin:/home/gca/bin:/home/gca/.local/bin:/home/gca/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/snap/bin:/home/gca/bin:/home/gca/.local/bin:/home/gca/.cargo/bin:/usr/bin

Any sugestions?

/cc @anthonyfok isn't Go part of the Snap deps?

@bep Good catch!
No, the /usr/bin/go executable is not yet part of the Snap.  This is an oversight due to my lack of understanding of how Hugo modules work and how it requires the go executable to work.  I might have had a vague idea about it, but not enough to remind myself to explicitly install the go executable inside the Hugo Snap.
And, if I understand correctly, Hugo Snap cannot access the go executable from the system.
So, yes, this issue needs be fixed in snap/snapcraft.yaml.  I hope to find time this weekend or early next week to do so.

Hi @jmooring, thank you for your reminder!  I really did forget about that conversation.
While I maintain some parts of Hugo, I am still very much a Hugo newbie as both Hugo contributor and Hugo end user.  For starters, I still haven't used the Hugo Modules feature and still don't quite understand how it works.
I have justed posted to ask them to re-open the request for classic confinement at https://forum.snapcraft.io/t/request-for-classic-confinement-for-hugo-snap/19892/8.
That said, I am unable to answer whatever doubts or questions due to my lack of understanding, and lack of time to study the in detail, and lack of motivation to fight the fight (due to still being overwhelmed by too many things in personal/real life).
So, if you wouldn't mind, please feel free to continue the conversations there and put for the arguments to convince them to change the Hugo Snap to classic confinement?  Thank you for your understanding.
Meanwhile, for this bug, I'll try and see if including go executable in the Snap would do any good.
@george-gca, thank you for reporting.  Until we resolve this situation, please try alias hugo=/snap/hugo/current/bin/hugo as suggested by @nobuto-m at https://github.com/gohugoio/hugoDocs/issues/1222#issuecomment-750725904 and see if it resolves your situation?  There is also the option of sudo apt install hugo though I understand that it is still the old version 0.80.0-6.  Debian now has hugo 0.82.1-1 though that unfortunately didn't make it to Ubuntu 21.10.  The Debian package will eventually catch up to the latest version, hopefully before the end of the year.


No longer works with latest version of angular material

In the latest versions angular material MaterialModule is no longer present. Including the package fails with
ERROR in project/node_modules/ngc-float-button/index.ts (3,10): Module '"project/node_modules/@angular/material/material"' has no exported member 'MaterialModule'.

I will fix this problem in this weekend, thank you.

Any news ?

Encountered the same issues - no action @GustavoCostaW so we probably need to remove this fab :(

Thanks @GustavoCostaW for this fantastic package. Please find our PR #7 which resolves this.
To @all waiting for the merge you can use upgraded package by installing it this way npm install -S ngc-float-button/settings

Hey @fennekit @adrian23  @hblanken @mordka , unfortunately, I was pretty sick because of this I took so long to fix that, sorry and I just published the new version and it's compatible with angular >= 4.4.4 and >= angular material beta 12, enjoy.
just run npm install --save ngc-float-button


Update self hosted runner for an org steps

Why:

Closes #6160
What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (eg. a "before and after video")

Summary
Closes https://github.com/github/docs/issues/6160
Changes in this PR

Article in staging
Changes

“Adding a self-hosted runner to an organization”
Updated steps in procedure.

Versioning
dotcom
Tests
All tests are passing! :green_apple:

Thanks for the review @lecoursen ⚡ I've added in the fixes now

Thanks

Non interacterable UI elements should be in "quotes" not italics.
IMO, we should track adding GHES versioning for these new conditions somewhere (e.g. a new issue). I can see us forgetting about this when it comes time for the next release 🙂

Thanks @lucascosti - I've added the the UI element fixes, and raised an internal issue that will check if these changes apply to a future product release.


Replace tempdir with tempfile

The tempdir crate has been deprecated in favour of tempfile.
This PR switches elan to use tempfile instead of tempdir.

Thanks!


Error removeAllLayers is not a function when pulse Clear All buton

[X] I'm reporting a bug, not asking for help
[X] I've looked at the documentation to make sure the behaviour is documented and expected
[X] I'm sure this is a Leaflet Draw code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[X] I've searched through the issues to make sure it's not yet reported

How to reproduce

Leaflet version I'm using: 1.03
Leaflet Draw version I'm using: 0.4.9
Browser (with version) I'm using: Chrome 56
OS/Platform (with version) I'm using: Windows 10
step 1 Drawing a polygon
step 2 Edit it
step 3 Pulse Clear All button and it return this error: this._activeMode.handler.removeAllLayers is not a function

Seems like this option was added in commit 9eded8696db26d1b405564c4906a1bfc690134ac only for the Edit Toolbar in src/edit/EditToolbar.js .
However, the referenced removeAllLayers function only exists in EditToolbar.Delete.js . This should also be implemented in EditToolbar.Edit.js for the button to work.


read counts

Dear Chris,
just to be sure about the meaning of counts I see in my results.
If I set the --pair option, in the last iteration subdir, I got two read pool files: I suppose that the one with -readpool-itxx-se.fastq  contains the exact baited reads, while the -readpool-itxx.fastq contains both of the copies.
The MITObim run log says "readpool contains 107683 reads", and I got this number if I count the reads in the file xxx_info_contigreadlist.txt file; I was thinking that this number was the count of the reads in the file -readpool-itxx.fastq, but those counts are not coincident.
What am i missing? I'm interested in the exact read list used by MITObim in order to remove them from the dataset I will use to assemble the genome.
Cheers,
Stefano

I've found my answers..
sorry


ML 8.14 : Single Shot Detector (A)

Description
📌  Issues for Week 8
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
Single Shot Detector
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the `Machine Learning/Machine_Learning/ML_for_Sequential_Data_&_Visual Data' Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Audio

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Audio
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Name- Pratyaksha Gupta
Batch-ML batch 3
Github link-https://github.com/Pratyaksha-047
Type- Audio

What's the progress here @Pratyaksha-047


Component browser does not react to symbol input

Screencast from 2023-06-22 12-38-09.webm
I would expect CB starts filtering out suggestions, the same way as it does for other inputs.
On enso-linux-2023.2.1-nightly.2023.6.21.AppImage.

I believe this is by design--see part 1 here: #6854. > and = are complete operators, so we are showing suggestions for a following operand.

Marking as duplicate of #6854

If the user types <, the Component Browser should indeed hint something starting or containing <. No reaction from Component Browser is indeed a bad behavior. Just a small explanation.


HttpResponse not consume responseEntity if objectMapper throw exception

HttpResponse.java
line 101, 103,
} else if (objectMapper != null) {
this.body = objectMapper.readValue(new String(rawBody, charset), responseClass);
} else {
throw new Exception("Only String, JsonNode and InputStream are supported, or an ObjectMapper implementation is required.");
}
if the objectmapper throw exception or new Exception is throw,  the exception is catcher and a new RuntimeException  will be throw out.
then the line 110, EntityUtils.consume(responseEntity);  will never be executed.
i believe, it should be clean up, before throw the RuntimeException

Hello, this should be resolved or addressed in Unirest 2. Please see the upgrade guide for important changes.


Invalid argument propertyAccessor.setValue on IE 11

Hi,
I just tried my Aurelia application on IE11 and discovered it is broken. I get a SCRIPT87: Invalid argument. error here. It works fine in all other browsers (Firefox and Chrome).
I tried to look at the problem, by adding this line before the setting of the value:  console.log('nrsauternet', value, obj, propertyName, Object.keys(obj), typeof obj, typeof value);. Here is what I get:

Which looks good to me. I don't know what is causing this.

when the error occurs, what is the value of obj and propertyName at the line of code you mentioned

As shown in the capture:

propertyName is textContent (string)
obj is some king of object called EmptyTextNode although I have also seen the error with a Text object.

Do you need more information? If so, can you be a little more specific on what you need?

any chance you could reproduce this issue using plunker:  http://plnkr.co/edit/1cBskxDsHcMP1hLmedyw?p=preview

I managed to dug into it today and I found the view that caused problem in my app. It came from this code:

        C'est le tour de

        ${playerName}

The injected translation by aurelia-i18n is "It is the turn of
{{playerName}}". If I remove t-params.bind it works (but the message is not correctly translated).
To make it work without errors, I removed ${playerName} (legacy from when I had not yet translations). I guess that this was confusing IE.
I tried to reproduce in this commit on a small app based on the skeleton. The result can be tested here: http://testaot.jujens.eu/ However, it works fine on IE11 even if it looks very similar to what my code was. Can see something different that might explain that this small example works but not my app?
Anyway, I don't know if it is a real bug in aurelia-binding or just bad luck from IE.

@zewa666 Can you look into this?

Issue moved to aurelia/i18n #138 via ZenHub


Problems with /ipmi & /metrics : Could not find inband device

Hello, I want to use the ipmi exporter as a container so here are my configs :
ipmi_remote.yml
modules:
        default:
                # These settings are used if no module is specified, the
                # specified module doesn't exist, or of course if
                # module=default is specified.
                user: "root"
                pass: "myPass"
                # The below settings correspond to driver-type, privilege-level, and
                # session-timeout respectively, see `man 5 freeipmi.conf` (and e.g.
                # `man 8 ipmi-sensors` for a list of driver types).
                driver: "LAN_2_0"
                privilege: "admin"
                # The session timeout is in milliseconds. Note that a scrape can take up
                # to (session-timeout * #-of-collectors) milliseconds, so set the scrape
                # timeout in Prometheus accordingly.
                # Must be larger than the retransmission timeout, which defaults to 1000.
                timeout: 10000
                # Available collectors are bmc, ipmi, chassis, dcmi, sel, and sm-lan-mode
                # If _not_ specified, bmc, ipmi, chassis, and dcmi are used
                collectors:
                #- bmc
                #- ipmi
                #- chassis
                # Got any sensors you don't care about? Add them here.
                exclude_sensor_ids:
                #- 2
                collector_cmd:
                        ipmi: sudo
                        bmc: sudo
                        chassis: sudo
                custom_args:
                        ipmi:
                        - "ipmimonitoring"
                        bmc:
                        - "ipmi-bmc"
                        chassis:
                        - "ipmi-chassis"

prometheus.yml
- job_name: 'ipmi'
    params:
      module: ['default']
    scrape_interval: 1m
    scrape_timeout: 30s
    metrics_path: /metrics
    scheme: http
    file_sd_configs:
      - files:
        - '/prometheus/ARC0_targets.yml'

    relabel_configs:
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: ipmi-exporter:9290

docker-compose.yml
ipmi-exporter:
    build:
      context: .
    image: ipmi_exporter2.0
    networks:
      - monitoring
    volumes:
      - /home/osadmin/monitaur/ipmi_exporter/ipmi_remote.yml:/config.yml:ro
        #- /home/osadmin/monitaur/refish_exporter/redfish_exporter.yml:/etc/prometheus/redfish_exporter.yml
    ports:
      - "9290:9290"

ARC0_targets.yml
- targets: [ '<IP_ADDRESS>' ]
  labels:
    hostname: ARC0CPU005
    pop: ARC0
    role: Compute
- targets: [ '<IP_ADDRESS>' ]
  labels:
    hostname: ARC0CPU006
    pop: ARC0
    role: Compute

Now the problem is about the prometheus.yml configuration. When i use /metrics, prometheus isn't returning any error but the exporter returns :
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:17:02.281Z caller=collector_ipmi.go:151 level=error msg="Failed to collect sensor data" target=[local] error="error running ipmimonitoring: exit status 1: could not find inband device\n"
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:17:02.284Z caller=collector_dcmi.go:53 level=error msg="Failed to collect DCMI data" target=[local] error="error running ipmi-dcmi: exit status 1: could not find inband device\n"
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:17:02.287Z caller=collector_bmc.go:53 level=error msg="Failed to collect BMC data" target=[local] error="error running bmc-info: exit status 1: could not find inband device\n"
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:17:02.289Z caller=collector_chassis.go:53 level=error msg="Failed to collect chassis data" target=[local] error="error running ipmi-chassis: exit status 1: could not find inband device\n"

First problem i've noticed is "target=[local]" which means the exporter isn't using the target list I gave him (I guess). But also the error "could not find inband device" is weird because from the ipmi-exporter container, I'm completly able to use a typical command like "ipmi-chassis -D lanplus -h <IP_ADDRESS> -u root -p 'myPass' --get-chassis-status.
(If i'm correct LAN_2_0 is equivalent to lanplus)
Then I tryed to change /metrics to /ipmi but i'm getting an error 400 from prometheus when scraping and the exporter isn't returning anything except the default launching logs:
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:34:04.483Z caller=main.go:103 level=info msg="Starting ipmi_exporter" version="(version=1.6.1, branch=master, revision=8fdc078f6c7ccd4ce443e8e5711d34149c81f3fe)"
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:34:04.483Z caller=tls_config.go:232 level=info msg="Listening on" address=[::]:9290
monitaur-ipmi-exporter-1     | ts=2023-03-13T14:34:04.483Z caller=tls_config.go:235 level=info msg="TLS is disabled." http2=false address=[::]:9290

When I'm executing the ipmi container, I'm also able to use FreeIPMI commands...
By using  "./ipmi_exporter --config.file=ipmi_remote.yml --log.level=info" on the host
And pointing to a specific target as "http://localhost:9290/ipmi?target=<IP_ADDRESS>", It ask on the CLI the password of the host then returns the good values:
# HELP ipmi_chassis_power_state Current power state (1=on, 0=off).
# TYPE ipmi_chassis_power_state gauge
ipmi_chassis_power_state 1
# HELP ipmi_current_amperes Current reading in Amperes.
# TYPE ipmi_current_amperes gauge
ipmi_current_amperes{id="57",name="Current 1"} 0.8
ipmi_current_amperes{id="58",name="Current 2"} 1
# HELP ipmi_current_state Reported state of a current sensor (0=nominal, 1=warning, 2=critical).
# TYPE ipmi_current_state gauge
ipmi_current_state{id="57",name="Current 1"} 0
ipmi_current_state{id="58",name="Current 2"} 0
# HELP ipmi_fan_speed_rpm Fan speed in rotations per minute.
# TYPE ipmi_fan_speed_rpm gauge
ipmi_fan_speed_rpm{id="158",name="Fan4A"} 8040
ipmi_fan_speed_rpm{id="159",name="Fan4B"} 6120
ipmi_fan_speed_rpm{id="160",name="Fan5A"} 7200
ipmi_fan_speed_rpm{id="161",name="Fan5B"} 5160
ipmi_fan_speed_rpm{id="162",name="Fan6A"} 9720
ipmi_fan_speed_rpm{id="163",name="Fan6B"} 7920
ipmi_fan_speed_rpm{id="164",name="Fan7A"} 9840
ipmi_fan_speed_rpm{id="165",name="Fan7B"} 7560
ipmi_fan_speed_rpm{id="166",name="Fan8A"} 9840
ipmi_fan_speed_rpm{id="167",name="Fan8B"} 7680
ipmi_fan_speed_rpm{id="35",name="Fan1A"} 8040
ipmi_fan_speed_rpm{id="36",name="Fan1B"} 6120
...

If someone could explain what's wrong in my configuration and how should I correct it would be very nice :)

You want the /ipmi endpoint. I think you've essentially gotten it right, except that it's asking for what I suspect to be the sudo password? Since you're using a container, I'd say you could just run the exporter as root and get rid of the sudo stuff? If not, you'll need to setup passwordless sudo in the container.

Thank you for your answer @bitfehler !
I'm already running the container as root. Something weird i've noticed is that I can use the targets URL like: http://localhost:9290/ipmi?target=<IP_ADDRESS> after running the exporters. But it only returns :
Unknown module "default"

Indeed. So it seems that the Dockerfile and the docker-compose.yml got out of sync. The container itself no longer specifies a config file, so this has to be done in the compose file. Can you add something like this:
    command: /bin/ipmi_exporter --config.file /config.yml

See also the commit I just pushed to fix this.

I agree with you, it's must be a sync issue.
After modifying the docker-compose.yml with the commit you just pushed, it returns:
monitaur-ipmi-exporter-1     | ipmi_exporter: error: unexpected /bin/ipmi_exporter, try --help

My bad, sorry. The command arguments actually get appended to the containers entrypoint, so no need to put it in there (fix):
    command: --config.file /config.yml

I think it works now, but I'm getting another error without link with the previous one :
monitaur-ipmi-exporter-1     | ts=2023-03-14T11:06:56.438Z caller=collector_chassis.go:53 level=error msg="Failed to collect chassis data" target=<IP_ADDRESS> error="error running sudo: exec: \"sudo\": executable file not found in $PATH: "
I'm working on it, thank you @bitfehler !


swcrc module type

Describe the bug
I can't use es6 imports in migration files in production (via kubernetes). But locally it works well.
I wrapped the dynamic import (const migrationFunctions = (await import(migrationFilePath)) as IMigrationModule) in try-catch and this is the error in the logs:
/opt/nest-api/migrations/1699935100948-drop-orders.ts:1
import * as mongoose from 'mongoose';
^^^^^^
SyntaxError: Cannot use import statement outside a module
at Object.compileFunction (node:vm:360:18)
at wrapSafe (node:internal/modules/cjs/loader:1088:15)
at Module._compile (node:internal/modules/cjs/loader:1123:27)
at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
at Object.newLoader [as .ts] (/opt/nest-api/node_modules/pirates/lib/index.js:104:7)
at Module.load (node:internal/modules/cjs/loader:1037:32)
at Function.Module._load (node:internal/modules/cjs/loader:878:12)
at Module.require (node:internal/modules/cjs/loader:1061:19)
at require (node:internal/modules/cjs/helpers:103:18)
at /opt/nest-api/src/ts-migrate-mongoose/migrator.ts:519:28
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at Migrator.runMigrations (/opt/nest-api/src/ts-migrate-mongoose/migrator.ts:518:31)
at Migrator.run (/opt/nest-api/src/ts-migrate-mongoose/migrator.ts:209:27)
at runMigrations (/opt/nest-api/src/migrator.ts:45:3)
at AppBootstrap.bootstrap (/opt/nest-api/src/main.ts:118:5)

Why are you using type: 'commonjs' and not type: 'es6' in swcrc.ts?

Hm, looks like the same topic again, provide some example that I can run and test, but previously es6 aka ESM were not supported I added it in latest release, so swcrc was defaulted to common js. We can patch that during build time for esm good point.

If that's the case this should help https://github.com/ilovepixelart/ts-migrate-mongoose/pull/239

Update to https://github.com/ilovepixelart/ts-migrate-mongoose/releases/tag/v3.5.4 and tell me if it works for you now

Update to https://github.com/ilovepixelart/ts-migrate-mongoose/releases/tag/v3.5.4 and tell me if it works for you now

Still doesn't work. But I haven’t yet wrapped it in a try-catch block and looked at what error occurs.

If I want to use your sources directly in my project, do I just need to copy your src folder or do I need other files too?

Not sure what is your goal, but it's MIT licence you can do whatever

I have to use the sources directly to print the error that occurs during dynamic import. So I’m wondering if I need to copy additional files to myself besides those located in the src folder.

I encourage you to revisit this example: https://github.com/ilovepixelart/ts-express-nest, since I have no clue what is current setup on your side, this example contains docker, and all files you can follow, I tested it with latest release today works as expected. Regarding folders and copy of files I have no clue what you trying to do, read about npm link if you want to test you code against local package


Non-default format for binary, library, test

Features

Implement a non-default formatting option for scala_binary, scala_library, and scala_test.
Only format .scala and source files (exclude generated files).
Triggered by :<target_name>.format.

Issues

Not sure how to test formatting with non-default option.

@pauldraper @andyscott Would you get a chance to take a look if the concept is what we want? I will fix the CI when we are ready to merge it.

This looks pretty good to me. I'd like to see it implemented as an optional phase using the newly refactored rules.

I'd just add a new phase, since it's this is logically separate from things resources jar bundling, compilation, or singlejar'ing.

I'd just add a new phase, since it's this is logically separate from things resources jar bundling, compilation, or singlejar

Done, I hope this is what you expect. Also, great work for the newly refactored rules.

@andyscott We added a way to do testing, it's basically another non-default output that runs a shell script.
All tests have passed, hopefully we can get it to merge soon. Thanks.


Improve renaming to allow inline renaming, fix same name conflict

Changes

RenameDialog replaced with inline renaming in GenericFileBrowser which is the behaviour on Windows Explorer (also more user-friendly in my opinion)
Files and folders can now be renamed to the same name but in different case
File extensions can now be changed when renaming files
F2 to rename shortcut unchanged

Screenshot

Known Issues

~Editing only works when name cell is focused~
Since Windows' file path format is case-insensitive, changing the case of a file/folder does not trigger an automatic update in the UI. ~As such, the relevant cache of the directory will have to be updated programmatically to reflect this change~ Seems like this is a bug in the Windows Storage API; files renamed to the same name are not updated (even reopening the app doesn't update renamed files)
~RenameDialog still used in PhotoAlbum~

Related Issues
Closes #189

Amazing work, there are no words to describe how helpful all your contributions are! Does this include pressing F2 to rename?

Thanks!
Yes, it retains the F2 keybindings. Pressing F2/Rename (whether from the ribbon or from right clicking) triggers DataGrid.beginEdit() which puts the name cell into edit mode.
However it doesn't work when selecting the row by clicking on other cells of that row (Date modified, Size, etc) because the DataGrid tries to edit the current cell that is chosen (which will be ReadOnly). A possible workaround may have to be subclassing DataGrid.

Fixed the UI problem; now renaming in GenericFileBrowser can be done regardless of how the selected row is brought into focus.
I'll get to doing this for PhotoAlbum
Now, the remaining problem is with renaming files to the same name, but of different case. The Windows Storage API does not reflect the change in file name when the current directory is updated. Even though the change in folder contents is detected, the StorageFile has the same FolderRelativeId and retains its original name in StorageFile.DisplayName. Navigating to a different folder then returning to the original folder causes the renamed file to appear as the original file, even though the case is actually changed.
Steps to reproduce:

Create file a.txt
Rename it to A.txt
Navigate to parent folder
Navigate back to original folder
File name shows as a.txt (looking at this file with Windows Explorer shows it as A.txt)

Possible solutions:

Renaming the file to another name, then renaming it back.

Is this a limitation, or is it possible to create a work around?

@jeffsieu That sounds like a limitation of the Windows Storage API

@duke7553 make sure to bring this up on your call next month...

@yaichenbaum @duke7553 A possible workaround until the API is updated would be to rename it to a different name, then rename it back. Like a.txt -> b.txt -> A.txt. I've tried doing this and it works (albeit a bit hacky)

@jeffsieu I am all for temporary hacks while we wait for a update so long as it is clearly commented so other developers can work off the changes and understand why its done that way.

Just added support for PhotoGrid as well.
With regards to the hack, somehow it doesn't seem to work... I'll have to figure out something else.

Maybe the best option for now is to not allow such renaming.

@jeffsieu is this ready for review?

@yaichenbaum I'll need to change a few lines to prevent the "same name but different case" renaming, then the PR should be ready.


Add support for track-changes

Builds on https://github.com/XLSForm/pyxform/pull/304.
Implemented this PR per the spec at https://opendatakit.github.io/xforms-spec/#audit-attributes.

Added a test for blank parameters because it seemed like a good idea.
Added a test for expected and unexpected values for track-changes
Added test to show track-changes and location work together
Instead of checking for the number of params for location, we now check if you have the correct params.
Reformatted code to match current style

Rebased off master, then added changes from #304, then reformatted with black.


Missing return(0x00...) from trace?

For the test stCreateTest/CREATE_ContractRETURNBigOffset.json we generate the following trace:
0:PUSH3, gas=0x5500000, stack=[]
4:PUSH3, gas=0x54ffffd, stack=[0x10000]
8:RETURN, gas=0x54ffffa, stack=[0x10000, 0x74ac2]
return(0x00...)

(where there are a lot of zeros in the return data)
However, the expected trace is missing the return(0x00...) for reasons unknown.

The contract code is just 0x6201000062074ac2f3.  Running this through Geth seems to do the right thing.  So it feels like the problem is that the large string is being clipped somehow ?


Update fr.yaml

Add missing translations

Awesome thanks for adding the missing items. Not sure on a new release, but changes should be available to pull from master.
Currently a bit behind on issues, PRs and releases so any help is appreciated.


Increase proxy buffer size to avoid nginx "upstream sent too big header"

Redmine gets nginx "502 bad gateway" error when using long issue filter and
using "right click assign to" on issue.  Increasing nginx proxy buffer size fixes this problem.
From nginx log
2017/01/17 09:54:55 [error] 261#261: *5514 upstream sent too big header while reading response header from upstream, client: <IP_ADDRESS>, server: , request: "POST
/redmine/issues/bulk_update?back_url=%2Fredmine%2Fprojects%2Fquality%2Fissues%3Fc%255B%255D%3Dtracker%26c%255B%255D%3Dstatus%26c%255B%255D%3Dpriority%26c%255B%255D%3Dsubject%26c%255B%255D%3Dassigned_to%26c%255B%255D%3Dcategory%26c%255B%255D%3Ddue_date%26c%255B%255D%3Ddone_ratio%26f%255B%255D%3Dstatus_id%26f%255B%255D%3Dassigned_to_id%26f%255B%255D%3D%26group_by%3D%26op%255Bassigned_to_id%255D%3D%2521%26op%255Bstatus_id%255D%3Do%26set_filter%3D1%26t%255B%255D%3D%26utf8%3D%25E2%259C%2593%26v%255Bassigned_to_id%255D%255B%255D%3D150%26v%255Bassigned_to_id%255D%255B%255D%3D149%26v%255Bassigned_to_id%255D%255B%255D%3D99%26v%255Bassigned_to_id%255D%255B%255D%3D1420%26v%255Bassigned_to_id%255D%255B%255D%3D1466%26v%255Bassigned_to_id%255D%255B%255D%3D713%26v%255Bassigned_to_id%255D%255B%255D%3D379%26v%255Bassigned_to_id%255D%255B%255D%3D514%26v%255Bassigned_to_id%255D%255B%255D%3D760%26v%255Bassigned_to_id%255D%255B%255D%3D1490%26v%255Bassigned_to_id%255D%255B%255D%3D1465%26v%255Bassigned_to_id%255D%255B%255D%3D1461%26v%255Bassigned_to_id%255D%255B%255D%3D200%26v%255Bassigned_to_id%255D%255B%255D%3D106%26v%255Bassigned_to_id%255D%255B%255D%3D15%26v%255Bassigned_to_id%255D%255B%255D%3D1488%26v%255Bassigned_to_id%255D%255B%255D%3D20%26v%255Bassigned_to_id%255D%255B%255D%3D148%26v%255Bassigned_to_id%255D%255B%255D%3D64%26v%255Bassigned_to_id%255D%255B%255D%3D191%26v%255Bassigned_to_id%255D%255B%255D%3D849%26v%255Bassigned_to_id%255D%255B%255D%3D90%26v%255Bassigned_to_id%255D%255B%255D%3D1494%26v%255Bassigned_to_id%255D%255B%255D%3D199%26v%255Bassigned_to_id%255D%255B%255D%3D682%26v%255Bassigned_to_id%255D%255B%255D%3D170%26v%255Bassigned_to_id%255D%255B%255D%3D17%26v%255Bassigned_to_id%255D%255B%255D%3D360%26v%255Bassigned_to_id%255D%255B%255D%3D1493%26v%255Bassigned_to_id%255D%255B%255D%3D3%26v%255Bassigned_to_id%255D%255B%255D%3D1464%26v%255Bassigned_to_id%255D%25"

LGTM 👍


[0.12.1.x] update splash screen

raze created some new splash screens for the traditional/drk theme with nicer URL font :)

hmmm.. isn't that light theme actually?

updated to correct theme now

Nice! What about updating light theme with new images you had here before?

I'll discuss updating light theme with raze at a later time, not sure what our designers are up to :)


Wagtail 5.1

This includes Wagtail 5.1 support in the testing.
NB: Modeladmin will soon be deprecated, so while this still works as things stand, it might be worth looking into either the use of wagtail-modeladmin or Snippets

thanks @lparsons396, I've merged your changes together with others in https://github.com/wunderweiss/wagtail-rangefilter/pull/6


MP3 Materials erroneously marked as Additive

Some materials are being erroneously marked as additive, as a result materials are being improperly blended.
An example of this is the Leviathan Infant:
Metroid6/06_Phaaze_#SERIAL#/07 Genesis Chamber/00a Default/CMDL_5F49A5452B1491BC.blend

Fixed


Store Shoot cluster CA bundle as ConfigMap

How to categorize this issue?

/area usability
/area security
/kind enhancement
What would you like to be added:
The Shoot cluster CA bundle should be stored as ConfigMap so that it can be read by users with viewer Project role. The CA bundle is not sensitive data, hence it doesn't need to be stored as a Secret.
Why is this needed:
Gardener stores a Secret with the name <shoot-name>.ca-cluster in the project namespace in the garden cluster. This Secret contains the CA bundle of the cluster. The dashboard and gardenctl generate a gardenlogin kubeconfig by reading the CA data from the Secret and mapping it to the certificate-authority-data field.
Gardenlogin will soon also support the shoot/viewerkubeconfig subresource (#78) which enables users with viewer role to have read-only access to a cluster. However, users with viewer role using gardenctl or the dashboard do not have the permission to read this Secret.
A potential workaround for generating the gardenlogin kubeconfig could involve calling the subresource shoot/adminkubeconfig or shoot/viewerkubeconfig solely for the certificate-authority-data field and discarding the rest. However, this approach might lead to unnecessary audit events.

/assign


Bar charts - Uncaught TypeError: Cannot read property '0' of undefined

I'm trying to update the values on a bar chart dynamically.  I followed the instructions and get an error "Uncaught TypeError: Cannot read property '0' of undefined"
I've created an example here: http://jsbin.com/zikuzusaqe/edit?html,js,output
It seems that "datasets" is undefined on a bar chart, but the example at http://www.chartjs.org/docs/#bar-chart-prototype-methods shows to use it to update existing data.

Is this still open? The JSBin references Chart.js 1.0.2 so the v2.0 milestone should be removed.

I'm not sure why I originally assigned this to version 2. This looks to be an error in the test case. The datasets array does exist in the bar chart, however the array to look for the element in is called bars not points. The latter is for line charts only.

How was this resolved? I have this weird error that only happens when the data is computed. For example:
// adding another dataset..
data.datasets.push({
  label: "Total",
  data: [30, 20, 40, 15]
});

is fine, but
// adding another dataset..
data.datasets.push({
  label: "Total",
  data: transactions.map(t => t.qty)   // =>  [30, 20, 40, 15]
});

fails. This is dumbfounding.

I'm running into an issue at the exact same location as @yanickrochon but slightly different case:
I have a doughnut chart that is instantiated with several datasets and can be updated by selecting a time span from a dropdown. With one setting the amount of datasets changes from 3 to 4 (i.e. the maximum datasetIndex should be 2 and 3 respectively) but after the change it tries to read databaseIndex 4 and crashes with Cannot read property '98' of undefined. After going back to the previous setting (which has 3 datasets) and then switching back to the problematic case (i.e. 4 datasets) everything works as expected. I'm using the React library react-chartjs-2.

@Toxyl do you have a working example because I am not able to reproduce the issue you or @yanickrochon have, with the current master (v3 rc 6, https://jsfiddle.net/Leelenaleee/q9tnLs8y/7/) and latest stable v2 release (2.9.4, https://jsfiddle.net/Leelenaleee/19v8qenz/2/)

@LeeLenaleee Unfortunately nothing I can share nor a reliable test case, the only thing that seems to work rather reliable is a doughnut chart that has multiple datasets (in my case 3 but I assume the exact number does not matter). In my React application the chart is part of a component that is refreshed once the data changes. As long as it stays at 3 datasets all is fine, but when a dataset is added it crashes. However, if I start out with a setting that has 4 datasets I can go back to a setting with 3 datasets without crash. But going back to a setting with 4 sets will crash it again. Stepping through the code @yanickrochon's screenshot shows I noticed that the databaseIndex is off-by-one.
Be aware that I'm using a React library for ChartJS, so the bug might be hiding in there as well.


more dependencies are needed in quickstart

Proposed Changes

Added tslib and @walletconnect/encoding as dependencies. The process in quickstart was failing without them

Why tslib is needed? It should be only needed in a typescript project, but then, it's no relevant to a general setup. Algo Builder should work with JS and TS.
I think we should add them to the dependencies of packages/web/package.json.
Could you update? The PR?


Switch allows case undefined: even when parameter cannot be undefined

🔎 Search Terms
switch case undefined, switch "case undefined:", "case undefined:"
Note, all of these result in a lot (100+ pages) of hits, so I haven’t gone through them all in detail. None of them appeared to match based on their titles, but it’s possible I’ve missed something.
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about switch (there are none)

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.7.0-dev.20241029#code/BQMw9mBCCGBOBcACAROMzEB8UCM7IEpEBeAPkQG8AoRWxAZwHcBLAFwGMALRUCGWItTrDE7aPQCmKNMng0RC2BNYBXWADtErWCokBueQrGTc+OQsXK1mkNAA2kgxcQB6F4gACregFoJADwAHCXZWP1hYMFhEZnV2KKVQuwBPRE5xRHUwRAkIqIAaBk4wFTsAExy86JwQ6BUTFXUyiRBYiQrmekywVkRxemYAc3VoHDspVmzUCAxsZDxYZEMRYylG5tb1dvNnOlZOSMZMiSOAUSrgZABJAFtAsHoBsbWmlraywic6AF8qX6peFA4EhpugsKZFkQyJRlkw2FweGh+IJlsJVtIZjtdoglKoNFodPpUXR0fMzMTLHibPZHBTXO4vL4AsFQuFItF4hEQqwUpV2V0MmSAF4YTrdXr9IYjZ5aKYycFkxZ00l4EVY3b7Q7HM4Xa53B5PcaIVWfZa-b5AA
💻 Code
(fooBar: "foo" | "bar") => {
    switch (fooBar) {
        case "foo":
            return true;
        case "bar":
            return false;
        // @ts-expect-error incorrectly has no error, should error because undefined is not assignable to "foo" | "bar"
        case undefined:
            throw new Error("Impossible undefined");
    }
}

(fooBar: "foo" | "bar") => {
    switch (fooBar) {
        case "foo":
            return true;
        case "bar":
            return false;
        // @ts-expect-error correctly errors as "baz" is not assignable to "foo" | "bar"
        case "baz":
            throw new Error("Impossible baz");
    }
}

🙁 Actual behavior
Typescript allows case undefined: even if undefined is not assignable to the switch parameter and strictNullChecks: true. This despite the fact that it does not allow other cases that can’t be assigned to the switch parameter.
🙂 Expected behavior
Typescript errors on case undefined: when

strictNullChecks: true, and
undefined is not assignable to the switch parameter.

Additional information about the issue
null has exactly the same issues as undefined in this situation.

Duplicate of #11920.


Fix names of smeshing-opts-verifying-* CLI flags

Motivation
The name of CLI flags must be the same as mapstructure tags.

bors merge


Various APOD issues/suggestions

These APODs are returning 'media_type' as 'other' but without any image/video content.   The json response does not include a 'url' field, although some return a 'digg_url' field.  I tested these through the https://apod.nasa.gov/ site as well and it seems these links may have broken content.  Here is the complete list:
5/29/2006 - https://api.nasa.gov/planetary/apod?date=2006-5-29&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap060529.html - No digg_url - No image/video content at apod.nasa.gov
5/22/2007 - https://api.nasa.gov/planetary/apod?date=2007-5-22&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap070522.html - No digg_url - Flash content at apod.nasa.gov not working
12/31/2008 - https://api.nasa.gov/planetary/apod?date=2008-12-31&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap081231.html - Has digg_url - No image/video content at apod.nasa.gov
4/5/2009 - https://api.nasa.gov/planetary/apod?date=2009-4-5&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap090405.html - Has digg_url - Flash content at apod.nasa.gov not working
6/29/2009 - https://api.nasa.gov/planetary/apod?date=2009-6-29&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap090629.html - No digg_url - Youtube embedded content not working
5/10/2010 - https://api.nasa.gov/planetary/apod?date=2010-5-10&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100510.html - No digg_url - Youtube embedded content not working
6/8/2010 - https://api.nasa.gov/planetary/apod?date=2010-6-8&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100608.html - No digg_url - Youtube embedded content not working
8/25/2010 - https://api.nasa.gov/planetary/apod?date=2010-8-25&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100825.html - No digg_url
12/15/2010 - https://api.nasa.gov/planetary/apod?date=2010-12-15&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap101215.html - No digg_url
1/23/2011 - https://api.nasa.gov/planetary/apod?date=2011-1-23&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap110123.html - No digg_url
2/1/2011 - https://api.nasa.gov/planetary/apod?date=2011-2-1&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap110201.html - No digg_url
2/22/2011 - https://api.nasa.gov/planetary/apod?date=2011-2-22&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap110222.html - No digg_url
3/7/2011 - https://api.nasa.gov/planetary/apod?date=2011-3-7&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap110307.html - No digg_url - Youtube embedded content at apod.nasa.gov not working
Also, these APODs return 'media_type' as 'other' but the content at apod.nasa.gov is to Youtube embedded videos.  In consideration to consistency, is it possible to change these to 'media_type' 'video' and then include a 'url' field to the youtube url?
For example, APOD 7/1/2020 - https://api.nasa.gov/planetary/apod?date=2020-7-1&api_key=DEMO_KEY returns video and the URL to the Youtube content.  While these do not:
8/10/2009 - API: https://api.nasa.gov/planetary/apod?date=2009-8-10&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap090810.html - Has digg_url
1/20/2010 - API: https://api.nasa.gov/planetary/apod?date=2010-1-20&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100120.html - Has digg_url
1/24/2010 - API: https://api.nasa.gov/planetary/apod?date=2010-1-24&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100124.html - Has digg_url
5/26/2010 - https://api.nasa.gov/planetary/apod?date=2010-5-26&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100526.html - No digg_url
7/25/2010 - https://api.nasa.gov/planetary/apod?date=2010-7-25&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100725.html No digg_url
8/25/2010 - https://api.nasa.gov/planetary/apod?date=2010-8-25&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap100825.html - No digg_url
12/15/2010 - https://api.nasa.gov/planetary/apod?date=2010-12-15&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap101215.html - No digg_url
Lastly, these are not issues but they feature Flash content.  I wonder if it's possible to get only the flash embedded video?
4/13/2009 - API: https://api.nasa.gov/planetary/apod?date=2009-4-13&api_key=DEMO_KEY | https://apod.nasa.gov/apod/ap090413.html - Has digg_url
Thank you!!

Also, this date: 4/19/2020 (A Vimeo Video) https://api.nasa.gov/planetary/apod?api_key=hhLOEStXTcfad68lAeqOkEoVqhzNhTCyaPp5kaaO&date=2020-04-19&thumbs=true
Should show the thumbnail.
Instead shows error.
Something wrong with endpoint.
But works perfectly for YouTube videos.

Looks like the older ones have the youtube video inside of an object where as the newer one that "works" have the youtube video inside of an iframe.
The APOD API works by parsing the APOD website HTML. It doesn't have any special access.
Unfortunately the admin of this repository doesn't have time to investigate these edge cases right now. However, we would be happy to accept any pull request that investigates all the links above to confirm the patterns in the HTML and suggests code changes to the parser that return better data for those examples.
Thanks for your work in highlighting these issues.


Add more documentation for users of Callisto

I'll look into how to get Documenter to add line numbers. In the meantime I agree it's confusing as it stands. I renamed the base_freq variable to be different from the option name. Hopefully that adds a little clarity.


Fix issue #28

@tisnik Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-29

@tisnik Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-29


feat(stats): add stats to v1 rpc module

bors merge

bors try

bors merge

bors merge

bors merge


Repeatable CMS page access performance benchmark tools dev environments

What happened (or feature request):
There are probably three key areas of performance interest for users of ddev:

How long does it take me to set up in the beginning
How long does it take to start and set up a site (including importing)
How does accessing the site feel? Can I get my job done without waiting for pages to load?

This ticket is about the third: Measuring the time it takes to access key pages in a CMS
We need access stats on a number of axes:

Authenticated and unauthenticated (Especially admin auth, as many devs will use admin privs)
Drupal and Wordpress generated pages, and static files

The tools that we use to do this should be able to create a site and populate it (by import), then provide page access times in a comparable format. They should work identically with ddev and other competing environments.
There are many tools available for benchmarking web apps, but it would be wonderful if we could also integrate this process with our Circleci builds (or probably a special circleci build dedicated to performance). That means that it needs to be package-able on linux.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else do we need to know:
Related source links or issues:

@rfay Fair to say this is handled in https://github.com/drud/ddev/pull/264 and https://github.com/drud/ddev/issues/238?

When #264 goes in it will automatically close this one.


Unable to pair Knock/FingerKey/MacID using CAT 2.0.0 and BrcmPatchRAM (but handoff/continuity works)

On iMac Early 2009 using an Asus BT 400 dongle. Was surprised to get continuity working with just a dongle on this machine as it was. However even though BLE is now operating on this machine, no apps appear to be able to see/pair over it from my iPhone 5s.
CAT installed with no errors. Tried BrcmPatchRAM 1.2 as per issue #122 and is installed correctly (firmware version up to c5563).

Hi, thanks for reporting this. It is a known issue (similar to #129), no workarounds have been identified so far.
My technical guess: this is possibly related to encryption and/or OS X identity services, which are also causing problems with Instant Hotspot when using USB BT4.0 dongles.
This does not happen on 2011 MBAs / Mac minis and when using Continuity-compatible AirPort Extreme hardware, as far as I know.

so, any clue to solve this?

I think it's something related to the kext patch. i've tried to uninstall CAT and restore the kext backup, but knock to unlock and other apps that uses the BT 4.0 dongle before, no longer works. now, I can't use knock anymore, i've reinstalled CAT, at last I'm glad to be able to enjoy continuity features.

I installed BrchPatchRAM but it doesnt work. How do I uninstall it?


can ytt auto-explain why overlay matching did not match expected number of nodes

Describe the problem/challenge you have
Users may run into a challenge determining why their overlay does not match expected number of nodes. (e.g. if overlay.subset({"apiVersion": "cert-manager.io/v1"}) but the actual node has cert-manager.io/v1alpha1). it would be interesting to somehow let the user know potential nodes that were "almost" matched.
no clue how to implement this.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Perhaps a fuzzy match?  Like, "did you mean ?"


Linux non-root install failure: Create BOND_ENABLE_BASHCOMPLETION

Hi,
I would like to install Bond in an environment where I do not have root access. It fails because it tries to create /etc/bash_completion.d/gbc. The library archive happens to be installed afterwards so it is important to install the remaining files. An easy fix would be to set BOND_ENABLE_BASHCOMPLETION=FALSE in cmake, similar to gRPC.
CMake Error at compiler/cmake_install.cmake:53 (file):
file INSTALL cannot copy file
"/home/me/bond_src/build/compiler/gbc.comp" to
"/etc/bash_completion.d/gbc".
Call Stack (most recent call first):
cmake_install.cmake:71 (include)

Sounds like you have some knowledge, care to submit a PR?
Also before working on a PR, play around with this cmake var https://cmake.org/cmake/help/v3.0/variable/CMAKE_INSTALL_PREFIX.html
On UNIX one can use the DESTDIR mechanism in order to relocate the whole installation. DESTDIR means DESTination DIRectory. It is commonly used by makefile users in order to install software at non-default location. It is usually invoked like this: make DESTDIR=/home/john install
It might help if you try installing to a folder were you have write permissions.

Happy to submit a PR, just following contribution guidelines that request that I post here first.
The destination is hard coded in compiler/CMakeLists.txt. Presumably because the author wanted it to be picked up automatically by bash. As a result, using CMAKE_INSTALL_PREFIX isn't going to give equivalent behavior. I can try checking write permissions but there isn't a native way in cmake. I can try using execute_process, I would have to play around with it.
set (completion_dir /etc/bash_completion.d)
set (completion_dir /etc/bash_completion.d)
...
if(EXISTS "${completion_dir}" AND IS_DIRECTORY "${completion_dir}")
    install (FILES ${completion}
        RENAME gbc
        DESTINATION ${completion_dir})
endif()

Closed with f78a418


vercel/ai provider integration

https://github.com/vercel/ai is the best lib to build amazing AI experiences with tools, good user feedback, etc.
they have a concept of provider which is way to integrate ai framework / api in their lib https://sdk.vercel.ai/providers/ai-sdk-providers
i did a dirty provider once, cant find it anymore but this is some hook i did before:
https://github.com/llm-edge/hal-051224/blob/main/lib/hooks/use-web-llm-chat-engine.tsx
would be amazing to have a clean web-llm provider for vercel/ai

Thanks for providing this idea! We will have a look and check whether we could integrate web-llm with it.


Add UITextView+BlocksKit.h to Copy Files

If you include blockskit as a dependency target, UITextView+BlocksKit.h was missing -> compile error.

+1 exactly what I need to get the unit test to compile again.


Performance Issue with complex SVGs

Huge edit:
When applying fill changes of shapes in a large numbers of shapes, hence the title complex SVGs, the performance is way too slow.
GET ALL SHAPES:
extension Node {
    var allShapes: [Shape] {
        get {
            var shapes = [Shape]()

            if let shape = self as? Shape {
                if shape.tag.count > 0 {
                    shapes.append(shape)
                }
            } else if let group = self as? Group {
                group.contents.forEach { (node) in
                    node.allShapes.forEach { (shape) in
                        shapes.append(shape)
                    }
                }
            }

            return shapes
        }
    }
}

ADDING TAPS TO EACH SHAPE
self.svgScrollView.svgView.node.allShapes.forEach { shape in
    shape.fill = Color.clear
    shape.onTap { [weak self] (tapEvent) in
        self?.changeColor(color: "\(Constants.selectColor)", shape: shape)
    }
}

CHANGE FILL/COLOR
func changeColor(color: String, shape: Shape) {
    let fill = Int(color.replacingOccurrences(of: "#", with: "") , radix: 16)
    let color = Color(val: fill!)
    shape.fill = color
}

Is there something wrong I'm doing here? Help please! Thank you!

Hi Yuri @ystrot! Shamelessly tagging you, sorry! But I think I made the question more straightforward, maybe you can help me here? And to other Macaw team. Thank you so much!

Hi @glennposadas,
I don't see explicit issues in your code. Probably you don't need to call color.replacingOccurrences(of: "#", with: "") for every node, just calculate Color once and use it.
Most probably your issue relates to performance of complex SVG rendering. You may try to profile your app to find the bottleneck. One of the most common way to optimize your rendering is to draw complex parts to images and then draw images (which is usually much faster).


operator must be surrounded by single space

This warning is not consistent between operators. It is given for *= but not for =. Also, it prevents alignment of code so I suggest the rule should be "at least one space".

The checking for multiple spaces is configurable and was added in #1654. Perhaps this feature is obsolete now that Style/ExtraSpacing is enabled by default.

Hm, Style/ExtraSpacing does not seem to be enabled here. But I guess I can configure my way out of this as you suggested. Ty.

Looks like this can be closed.


Update Mle and Joiner implementations

This PR contains two different (related) commits:
[joiner] update Joiner implementations
This commit contains the following changes to Joiner class
implementations:

Notifier events are signaled on all Joiner state changes
and are logged (through use of Joiner::SetState()).
Discover scan during join uses filtering (removes the need
to perform filtering by the Joiner class itself).
Joiner::Stop() is changed to allow it to be called at any
stage of join process and it correctly cleans all states.
Priority assignment to discovered networks is updated/fixed
(prioritizing networks with an exact match of Joiner ID in their
Steering Data compared to ones that allow all joiners, and
sub-prioritize based on signal strength).
Logs are updated.

[mle] update discover scan filtering
This commit makes improvements to MLE Discover Scan implementation,
mainly the steering data filtering logic:

The SHA-256 of EUI64 (Joiner ID) and then the related bloom filter
CRC-CCIT and CRC-ANSI indexes are calculated and saved at start of
scan operation (removing the need to re-calculate them on every scan
result).
When filtering is enabled, a Discover Scan result with no Steering
Data is now rejected.
Mle class member variables related to discover scan operation
are renamed to start with mDisocver prefix.

@librasungirl, you are correct, however, the new design of not trying next joiner routers after we pass  OT_JOINER_STATE_CONNECTED state (i.e. we successfully connect to one over DTLS) is intentional. Basically if we can successfully establish a session through a JoinerRouter with commissioner but fail at the next steps (say the JoinerFinalize response) we would consider the whole Join attempt a failure. This seems to align with the spec, but there are also other reasons why we went for this.
We want to remove the requirement for the vendor related info to be static pointers (without the need to have buffers/memory space to store them). Note that these parameters are only used in "Joiner Finalize req" message. The idea is to prepare the the "Joiner Finalize" message from Start() call, keep track of message pointer, and once we get to CONNECTED state we send the message (which basically consume and free the message). With this model, we cannot try out another Joiner Router since the message is already consumed. I plan to submit follow-up PRs related to this after this.
Thoughts? Concerns?

@abtink , if only speaking of the spec, the joiner hunting algorithm in <IP_ADDRESS>.2 seems in favor of keep trying next available joiner router until success.

b. If the DTLS handshake fails or there is a timeout, move on to the next network.

For the requirement you described, maybe clone the FIN.req message for one Joiner Router try - which would consume more message buffer space instead. Sorry that I don't have any better idea.

Thanks @librasungirl
With new model, we would still try the next joiner if DTLS connection itself fails/timeout (from the HandleSecureCoapClientConnect() with aConnected being false). Note that we don't start the mTimer in Joiner class till after that. So we rely on Dtls class for timeout of DTLS connection.
Only if DTLS is successfully established (we reached our intended commissioner and established a secure connection to it) and then something fails afterwards we don't try other routers/networks.  In spec, section 8.4.5 we seem to have this statement for when JOIN_FIN.rsp contains a reject status (or I assume we timeout and never get JOIN_FIN.res):

The Joiner reattempts joining the same network anticipating that the Commissioner is now running the correct application.

If I understand this correctly, we need to try to connect to same router (same network) again so I guess we don't need to try other routers (other networks) after this?
I think this is all rather special case (considering we currently have 2 joiner-router entries)... :)


ftplugin has no effect

I have the following inside ~/.config/nvim/after/ftplugin/dapui_stacks.lua:
vim.cmd "setlocal winbar=\\ Stacks"
vim.cmd "setlocal statuscolumn=\\ "

However, this does not seem to do anything.  When I put the following inside ~/.config/nvim/after/ftplugin/dap-repl.lua:
vim.cmd "setlocal winbar=\\ REPL"
vim.cmd "setlocal statuscolumn=\\ "

It works.  See the attached image.

Expected behaviour:

Off topic but how did you get the dap debug ui buttons,  continue, setup over etc... in the top right. Nice them too which one is that?

The theme is nordic.nvim :)
Check out these lines for the controls.


No Code Completion For Script Instanced Nodes

Operating system or device - Godot version:
Linux, Godot 2.0.3
Issue description (what happened, and what was expected):
I followed the instancing tutorial for scripting. I expected code completion for the node/instance, but there was none.
Steps to reproduce:
Instance a node in code and type the node on a new line and try putting a period after it.
Link to minimal example project (optional but very welcome):

@toliaferrode I closed your other issue as duplicate and you opened another one? Please do not do that. You can discuss your problem in the still open issue about the subject, no need not make new tickets for the same thing. It won't make be solved faster nor better. If you don't think it's a duplicate, then explain why not in the same issue (it's just closed, not locked) and I'll reopen it.
So I'll also close this as duplicate of #1039 & #4835.

Hey, sorry about that truly. I honestly thought I didn't post the original
issue. I apologize.
On May 27, 2016 7:43 PM, "George Marques"<EMAIL_ADDRESS>wrote:

Closed #4849 https://github.com/godotengine/godot/issues/4849.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/4849#event-675053152, or mute
the thread
https://github.com/notifications/unsubscribe/ARM3vO3Ide8EGHUfpmzecUhqSCit7YiYks5qF4GMgaJpZM4Io5OU
.

@toliaferrode ok, I can understand that. BTW, you can see the issues you created if you click the "Issues" link on the top bar of GitHub. So next you can be sure you didn't make one yet 😉

Thanks a lot. :)
On Sat, May 28, 2016 at 9:19 AM, George Marques<EMAIL_ADDRESS>wrote:

@toliaferrode https://github.com/toliaferrode ok, I can understand
that. BTW, you can see the issues you created if you click the "Issues"
link on the top bar of GitHub. So next you can be sure you didn't make one
yet 😉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/4849#issuecomment-222308209,
or mute the thread
https://github.com/notifications/unsubscribe/ARM3vDLhkOlNVTxYMe9A5mrunbzwGMkuks5qGEDngaJpZM4Io5OU
.


npm start not working

I tried to do "npm start" from command prompt at the ClientApp level like this page says, but I always get the error "error missing script: start"

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

ID: 32a18270-91cd-2a02-2eb8-2c8b1ed03737
Version Independent ID: c58d57e5-a0c2-201d-f6dd-076ad5971fcb
Content: Use the Angular project template with ASP.NET Core
Content Source: aspnetcore/client-side/spa/angular.md
Product: aspnet-core
GitHub Login: @SteveSandersonMS
Microsoft Alias: scaddie

@wcdeich4 Does your package.json file define a start script inside of the scripts property?


Process markdown for summary

In the Post struct, the summary field should become summary_markdown. And then we can process it correctly and get some good looking text for our atom feed and open graph summaries.

fixed


Feature request: selectable in diff result

I do not really like to re-type once I deleted but I still need something. I just like to copy and paste, can I change it like Submit Text that can be modified?

Do you mean the diff view under source control? I'm able to copy and paste there.

Or is this a different view?

By click git diff field. I can select the things which I have added, but cannot select the deleted things.

/duplicate #8226


rise time calculation broken for umux pulses

Original report by Dan Becker (Bitbucket: danbek, GitHub: danbek).

In the latest master branch (559c211), rise time are being calculated incorrectly, at least for pulses from the umux system.
To reproduce this, you can grab umux data from here:
$ scp pcuser@687kpac:/data/umux/2015/20150828_85mk/*163641* .
$ scp pcuser@687kpac:/data/umux/2015/20150828_85mk/*110048* .

Then the following:
#!python

import numpy as np
import pylab as pl
from pylab import clf, plot, hist
pl.ion()
import mass
import mass.gui

import mass.calibration.gaussian_lines

pl.close()
dir_p = "/home/beckerd/data/20150828_85mk"
prefix_p, suffix_p = "20150828_163641","ljh"
dir_n = "/home/beckerd/data/20150828_85mk"
prefix_n, suffix_n = "20150829_110048","noi"
channels=(1,3,5,7)
pulse_files=["%s/%s_chan%d.%s"%(dir_p, prefix_p, c, suffix_p) for c in channels]
noise_files=["%s/%s_chan%d.%s"%(dir_n, prefix_n, c, suffix_n) for c in channels]
data = mass.TESGroup(pulse_files, noise_files)

data.summarize_data(forceNew=True)
ds = data.channel[7]
ds.plot_summaries()

If you look at the rise_time plot, you'll see that things are screwed up;  there are certain times when the rise times have clearly wrong values. Sorry to be so vague, but come talk to me if the problem is unclear!

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Dan, sorry that this is only fixed in branch develop, but it is what it is. Young made an honest mistake.

Joe

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Duplicate of #29.


Candlefactory firstrec fix

force use of includeFirst in generated requests, see #100

Changes Unknown when pulling f4008d2d279379de6876707b432afd0355047696 on candlefactory-firstrecfix into ** on master**.


Check that latest change to native is persisted correctly to the model

Enrique, I made a change yesterday that allows native annotations to have a variadic list of backend names (just like by for example). Nothing seems to break but could you please check that this information gets persisted correctly to your model? (This to prevent having to make backward incompatible changes in Ceylon 1.2.1 or 1.3 or whatever)

no it won't get persisted correctly to the model because native is one of the annotations that is packed into a bitmap, so when building the model from JSON, only the JS backend is set for the annotated declaration, but the annotation itself doesn't have any args.

Remember that supposedly this was enough because the JS model will only be used to compile to JS?

I suppose I can remove native from the bitmap annotations to store it as a regular one with its args.

You asked if this was important and I think it is, otherwise we release this week and the next day we'll have a version that's not backward compatible anymore with what we released. So hoping that the change is small enough I'd prefer to do it now.

ok then

Err, I added it to the metamodel yesterday man ;)

$mod-nat and stuff.

Err, I added it to the metamodel yesterday man ;)

For all declarations?

can't find any occurrences of $mod-nat anywhere. But the native annotation is packed in a bitmap along with shared, formal, actual, etc so it doesn't store any arguments. I'm working on this (just unpacking it and checking that it gets restored properly)

can't find any occurrences of $mod-nat anywhere

You can't? It's here: https://github.com/ceylon/ceylon-js/commit/8bb46f110a5592319398ef1800fe3b1b9dda2586

ah, I was looking in the js code.

So, I unpackaged the native annotation and now it gets stored, without any args, because the one that gets stored is the one from the header, not the implementations.

Ok, the idea is that when we get a working Dart backend it could be possible that we'd have code that both for JS and for Dart because they happen to have the same implementation. So you'd perhaps see things like:
native("js", "dart") shared String fooBar() { ... }

We'd have to be able to tell from the model that this method can be used by both backends, while a js-only method would given an error for Dart and vice versa a dart-only one would give an error for JS.
That might not actually work yet today, but I just want to make sure that when we make it work we only have to make changes to code not to any file formats. So if you think that's okay now we can close this.

OK let me push this stuff and close.

I also left the code that unpacks the native if it's in the bitmap, for the modules that still use that.

Just don't remove what I out there because it's used by herd ;)

s/out/put/

yeah I won't touch that


Issue/474 refactor time

Pull Request
Description
refactor unique and monotonically increasing times stamps for nwp and satellite
Fixes #474
How Has This Been Tested?
Normal unittest tests

TODO add unittests for specific functions

[ ] No

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #569 (7bc1602) into main (03641c8) will increase coverage by 0.62%.
The diff coverage is 61.29%.

@@            Coverage Diff             @@
##             main     #569      +/-   ##
==========================================
+ Coverage   92.57%   93.20%   +0.62%
==========================================
  Files          42       42
  Lines        2776     2768       -8
==========================================
+ Hits         2570     2580      +10
+ Misses        206      188      -18

Impacted Files
Coverage Δ

nowcasting_dataset/utils.py
68.51% <42.85%> (-6.20%)
:arrow_down:

...asting_dataset/data_sources/nwp/nwp_data_source.py
95.00% <100.00%> (+13.94%)
:arrow_up:

...et/data_sources/satellite/satellite_data_source.py
94.73% <100.00%> (+11.92%)
:arrow_up:

Continue to review full report at Codecov.

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


cup abandoned - PEP 541 - Ownership transfer

Hi dear admins,
I'd like to request an ownership transfer of pypi project cup to me (username: mythmgn)
The project has not been maintained since 2014.  I tried to contact the owner (email addr pavanmishra@gmail.com), but got no response. I searched him on facebook and linkdin, and found no one.
The opensource project I lead is here:
repo of github: https://github.com/baidu/CUP
docs of cup: http://docs.iobusy.com/docs/cup/
Thank you very much.
-- Gallon Ma
Original cup project info:
pypi: https://pypi.org/project/cup/
repo of github: https://github.com/pavanmishra/cup
author email<EMAIL_ADDRESS>

@pavanmishra doesn't seem to have any recent activity on GitHub, but it might be worth opening an issues on the repository.

hi @jamadden
six weeks has been passed away.   We still cannot reach Pavan.  No response.
Is it OK to go ahead with the transfer?   Thanks.
--mythmgn

@jamadden  anything else that I can do to get this issue moving?
Thx.

There’s not much more I can do. We can try pinging the admins (@di) directly, but please remember that PyPI is mostly a volunteer project. Patience is appreciated, sincerely.

The original owner Paven showed up and helped with the procedure.
Ownership transfer has been done.  This issue is closed.
Thank you all for the help.  @jamadden  @di


"Undo" action becomes unavailable when all have been deleted

"Undo" action becomes unavailable when all have been deleted and will fail to render.

I've submitted a PR with the fix - thanks for reporting this @gotoeasy!


Callback errors with v5.6.23 (Juce 6.0.8)

I did compile the latest build (v5.6.23) on Mac, but when trying to use existing panels I get always the same error when using callBack methods:

What could be the origin of these errors?

I can't even find requestDataChanged anywhere in the repo

OK, doing some more searching, the error windows seem to get created here.
https://github.com/RomanKubiak/ctrlr/blob/7fab6623d3a2b26b1bcc583568028552936fd487/Source/Lua/Methods/CtrlrLuaMethodManagerCalls.cpp
This might be a good place to start to  narrow down this error.  I'll keep searching for other potential places.
Out of curiosity, does this only happen with panels that are imported from other versions, or does it still happen if panels are created from scratch in this version?

Indeed this link gives me (or someone else...) a good start!
I did a panel from scratch with a function sending only one Midi message and got the same kind of error(s). All imported panels from older versions also give the error on every function which is called when clicking on a button. At least that is what my experience is. I did not test it with other kind of functions which involve calling "something".

I think I found the reason for the errors. In older versions assigning a variable to and formatting a sysex message like this: syxProgram = {0xF0, 0x07, 0x00, 0x78, 0x14, 0x09, 0x40, 0x08, 0xF7} was no problem. In this latest version (5.6.23) this is apparently a problem and causes the overload mismatch. The solution is to format the message explicitly as a string, like this: "F0 07 00 78 14 09 40 08 F7". Other ways also are OK, as long as (I think) the outcome is a string. I hope I'm right, but at least my panel does now run without issues so far in this version.

I think the bug must he in this file, though the only change has been to remove the luabind overload.  As far as I can see it should still accept an array, but I'm definately no C++ guru.
It's weird because it seems to work fine in Windows.
https://github.com/RomanKubiak/ctrlr/blob/master/Source/MIDI/CtrlrMidiMessage.cpp

Looking at this, I'm wondering is line 118 should be
MemoryBlock possibleMidiData = luaArrayToMemoryBlock(luaArray.getObject());

and not
MemoryBlock possibleMidiData = luaArrayTomemoryBlock(luaArray.getObject());

Normally all the ctrlr stuff is strictly lowerCamelCase.

We've confirmed that it's just midi messages formatted as a table that cause the issue.  MIDI messages in a memory block or string work fine.  MemoryBlocks are probably the easiest conversion to make by using this code snippet.
requestMessage = {0xf0, 0x00, 0x00, 0x2f, 0x03, 0x01, 0x02, 0x00, 0xf7}
requestMessageMB = MemoryBlock()
requestMessageMB:createFromTable(requestMessage)
midi = CtrlrMidiMessage(requestMessageMB)
panel: sendMidiMessageNow (midi)

can you show me what causes an error ? a line in Lua ?

Sorry Roman, but I only found a kind of workaround and Godlike-Production (aka the Puppeteer) did elaborate on this. See his comment above. My guess is in this version (5.6.23) Lua does not accept a sysex message formatted as a table.

can you show me what causes an error ? a line in Lua ?

Hi Roman.  Basically  this works.
requestMessageMB = MemoryBlock()
requestMessageMB:createFromTable(requestMessage)
midi = CtrlrMidiMessage(requestMessageMB)
panel: sendMidiMessageNow (midi)

This also works
requestMessage = {0xf0, 0x00, 0x00, 0x2f, 0x03, 0x01, 0x02, 0x00, 0xf7}
requestMessageMB = MemoryBlock()
requestMessageMB:createFromTable(requestMessage)
midi = CtrlrMidiMessage(requestMessageMB:toHexString(1))
panel: sendMidiMessageNow (midi)

This does NOT work
midi = CtrlrMidiMessage(requestMessage)
panel: sendMidiMessageNow (midi)

It is only an issue with Ctrlr v5.6.23 for Mac. Older versions, in particular the 5.3.xxx versions, work OK.

@Godlike-Productions  You don't need all those lines to create a memoryblock:
just replace :
panel:sendMidiMessageNow(CtrlrMidiMessage({x, y, z }))
with
panel:sendMidiMessageNow(CtrlrMidiMessage(MemoryBlock({x, y, z})))
where {x, y, z } is already a table. since you can create a MB from a table you can get a volatile MB with just : MemoryBlock({x, y, z})

Thank you, this is a great solution: "panel:sendMidiMessageNow(CtrlrMidiMessage(MemoryBlock({x, y, z})))"
Greeting, Fred


Animation

Hi! Thanks for great and easy to use library!
Could you please open API to change animation duration? Visually it's too slow

Thank you. I did it check the new version or see this commit:
https://github.com/pedromassango/IBackdrop/commit/ef79cfc77055fd4625cdf1717d1bb4125e7ff7f4


Frontend tooling: introduce Prettier and ESLint

This PR introduces Prettier and ESLint inside our development environment. This is propedeutical towards the adoption of a new frontend library to architect things in a more modern way.
You can actually see the steps I took looking at the commits, but I'll summarize that here too:

I installed the new packages using NPM;
Added some sensible default configuration just to ensure everything is on track from day one. I also added the eslint React plugin since we're moving towards React;
Formatted all the frontend source code using Prettier;
Corrected all the eslint errors either adjusting the configuration or disabling noisy rules for legacy code;
Added a make step in the CI so we're covered from now on.

Hope you like it!

@arbulu89 well this can be changed in the future, the cool thing about prettier is that we have one config to rule them all :D


architecture is "x86' on arm - kestrel will try loading x86 libuv.dll

https://github.com/aspnet/KestrelHttpServer/blob/dev/src/Microsoft.AspNet.Server.Kestrel/KestrelEngine.cs#L36
Related:
https://github.com/aspnet/dnx/issues/2063

@DamianEdwards is supporting Kestrel on ARM is out of scope for V1?
Also please see: https://github.com/aspnet/dnx/issues/2063

Actually this should resolve itself semi-automatically when dnx starts loading native dlls automatically. - then we will be able to remove the code where we have the check.

Uhm, current Kestrel works fine on Linux on ARM given a proper libuv. Making it run on Windows on ARM should be really straight forward even now.

Uhm, current Kestrel works fine on Linux on ARM given a proper libuv. Making it run on Windows on ARM should be really straight forward even now

Could you check latest version on Linux arm? Unfortunatelly in my RP2 doesn't work (Windows IOT)

This was actually fixed in d4853f9b7 where we started using the built-in support for loading native libraries instead of custom code we had before where we had this check.
@matizk144 - I believe what you are seeing is a different issue. You did not publish for win10 arm but used the default runtime id for which libuv is not defined. This is bug #340.
I am closing this bug and will follow up on this in the other issue - I did hit the same thing when I first tried Kestrel on R-Pi2.


Fixed installation on other than Ubuntu GNU/Linux distributions

Fixed installation on other than Ubuntu GNU/Linux distributions.
Fedora and other GNU/Linux distributions use different $libdir prefixes. Now it can be installed on any GNU/Linux distributions.

Coverage remained the same at 96.586% when pulling 81f04d30a9a001120a9eeee6c11655b2c7bd61f5 on xvitaly:fix-installation into 5b5ca1471b8aa564b664ba41a7de0dd8f5a9c6d8 on SRombauts:master.

Thanks for your contribution!


feat: traverse CSS stylesheets looking for @import statements

Problem Statement
As of today, we only compute the SRI hashes for stylesheets that are either:

directly inlined via <style>
directly included via <link rel="stylesheet">

This does not include, however, the possibility of indirect imports via the @import statement.
Proposed Solution
We should look for (recursively) import statements (static & dynamic ones) in the CSS code that is either inlined or directly included in our HTML code.
Progress Tracking

[ ] Inline CSS
[ ] Included CSS

We might come back to this idea later, but this particular issue does not accurately describe the current state of things nor a good-enough solution.


[Messaging] Update message size on link creation

Summary
The focus of these changes is to enable the client to refresh the maximum message size each time a new AMQP link is opened; this is necessary for large message support, where the maximum message size for entities can be reconfigred adjusted on the fly.  Because the client had cached the value, it would not be aware of the change and would enforce the wrong size for batch creation.
References and related

[Feature Req] ServiceBusMessageSender should refresh maximum message size when service settings change (#43983)

/azp run net - eventhub - tests

/azp run net - servicebus - tests

API change check
API changes are not detected in this pull request.


Feature request: showing moved lines differently

One thing I (and others) find sad about GitHub's diff is that it doesn't show succintly moved lines (it shows one line removed, one line added...). I was wondering if this project could improve that when using git diff locally ? WDYT ?

Yes this is something Phabricator does much better. I think modifying the diff algorithm itself requires more information than we have in this tool (since it's just parsing the output of git diff), so it'll be hard to incorporate something like that here. I know there is git diff --color-moved, but it doesn't format the diff differently for moved lines, so that information gets lost.

Could we perhaps interpret the color bytes somehow?


Add ARM64 versions of docker-compose files for nexus release

See also https://github.com/edgexfoundry/developer-scripts/issues/104


feature/4944-4965-Added XRay and AWS Secret Support

feature/4944-4965-Added XRay and AWS Secret Support
📲 What
Added the following features:

AWS XRay
AWS Secrets

🤔 Why
Progress the Stacks AWS roadmap
🛠 How
See ADR
👀 Evidence
N/A
🕵 How to test
mvn clean test
✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

/azp run


Broken libgit2-sys 0.3 compatibility (bump to 0.4?)

Commit 287be9ccb404668cf8f56643d83068ec8536031a, now part of libgit2-sys 0.3.8, broke compatibility with previous 0.3.7 API by moving around some pub constants.
You noticed yourself the effect of this when touching cargo.
According to vanilla semver, 0.3.7 and 0.3.8 are not forced to be compatible (rule 4, aka v0 major).
According to cargo semver, 0.3.7 and 0.3.8 are considered compatible.
It is my understanding that ligbit2-sys should be bumped to 0.4.

Yeah this is true, although it's possible to be compatible with both by updating to using the constants from the intended locations. Along those lines I'd prefer to not bump this just yet (e.g. "minor changes" can be compatible with both) so I'm gonna leave it at 0.3 for now.
Thanks for the report though!


Relicense to Apache.

Use a more permissive license than LGPL.
All contributors must agree to this change.
@Creeperface01 @phase

I agree

I agree to relicense my commit (https://github.com/NukkitX/Network/commit/48ed4e7086ee9b078df73dfe1bd2a08e65900899) from LGPL to Apache.


Updating NuGet.config to adhere to standardized schema

Issue:

Currently <settings><repositoryPath> is ignored with NuGet 3.3 (VS 2015) and results in the packages being restored at the solution level (default)

Fix/Reasoning:

Switched to the standardized format
Some discussion from StackOverflow specifically under VS 2015

Testing:

Tested for backwards compatibility - against NuGet 2.8 (VS 2013)

Inquiry for reviewers:

I realize the packages are checked into the repository.  What is the rational for this?

Thank you :smile: This problem has bothered me ever since I shifted to Visual Studio 2015, but I honestly didn't know that there was an easy way to address it. :+1:


table里面的信息能否渲染成按钮功能

ElementUI version

OS/Browers version

Vue version

Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

#169


Instructions for applying Viya ServiceMonitors

I have recently deployed the monitoring components into a new Viya deployment on EKS. However, prometheus and grafana are not displaying any information and tech support directed me to ask this project for further instructions or resources. I do see the documentation regarding ServiceMonitors here.
However, there is not a specific set of instructions for deployment. Is it as simple as applying the manifest, or are there additional steps? For example:
kubectl apply -f viya/serviceMonitor-sas-cas-operator.yaml -n monitoring

Hi there,
The SAS Viya ServiceMonitors are deployed when you enable the Viya namespace for monitoring. This is a specific step that needs to be done for each Viya namespace in the cluster.
See https://github.com/sassoftware/viya4-monitoring-kubernetes/tree/stable/monitoring#deploy-sas-viya-monitoring-components for details.

Hi Bryan,
We have deployed the components to monitor the cluster (deploy_monitoring_cluster.sh) and Viya (deploy_monitoring_viya.sh), but Grafana does not show any data. Here is what the SAS Cas Overview dashboard looks like:

For additional context, I only configured the user-values-prom-path.yaml file in the samples/ingress location of this project prior to deployment. Are there additional files that we need to pull from one of the other sample directories?

Changing any files in samples won't have an effect. You can read here about setting a USER_DIR and customization.
The first thing to check is if Grafana can talk to Prometheus and if cluster monitoring is working. Look at the 'Kubernetes Cluster' dashboard in Grafana. If that has data, then it's just an issue of properly enabling monitoring of the SAS Viya deployment.
To troubleshoot monitoring a SAS Viya Deployment, you can check Prometheus - Go to the Status menu, then click Service Discovery. Look for various PodMonitor/[namespace]/sas-* or ServiceMonitor/[namespace]/sas-* entries and see if any targets are found. If so, that means the deployment is being properly watched. Assuming there are targets, you can check Status -> Targets to see if there are any errors monitoring the SAS components. If there are no targets or you don't see the 'sas-*' entries under Service Discovery, then you probably need to run monitoring/bin/deploy_monitoring_viya.sh again and make sure you set VIYA_NS to the namespace containing your Viya deployment. For example:
VIYA_NS=my-viya-namespace monitoring/bin/deploy_monitoring_viya.sh
I hope this helps. Let me know how it goes.

Assuming no news is good news here.  Closing this issue.


Allow name to be edited

Allow users to edit their name using https://github.com/impactasaurus/server/issues/198

 User Management


try to fix issue #251

I think that the following modification (seen in quick-error) could work here, to fix issue #251.
I splitted the problematic part in two parts:

ignore unknown/renamed/removed lints
disable warnings about unused doc comments

I'm trying to push for a release for some minor changes that have serious impact on users.  If you don't mind, I would like to hold off on this one.

This fixes an issue with rustc 1.27.0 (and probably 1.28.0, too), which doesn't apply the (#allow) unknown_lints, renamed_and_removed_lints to unused_doc_comment, unused_doc_comments, when these lints-to-ignore are in the same line

Thanks, that was news to me!


Optimize usage of test-runner

Introduces the usage of test-runner with fully qualified method names, allowing for optimal usage, i.e. calling test-runner once per test framework.
Mirror of #46.
Ready to merge now @martinezmatias

Great, sorry for the mistake!

Great, sorry for the mistake!

No problem. My bad too, I should have only one PR marked as ready at a time, so this type of mistakes doesn't occur.


Cosmetics

correct the wrongly formatted links i inserted, sorry

sorry. didn't realize that it didn't merge before i delted the branch...

So please feel free to re-create it again, and I'd like to merge it.


Annotate controller jobs to prevent node eviction

Changes
This change annotates pods running controller components (controller and webhook) to prevent the node they're on from being evicted when autoscaling scales nodes down.
Without this change, when nodes are autoscaled up and then back down, the node that gets selected to kill might be the one that runs Tekton components, causing a brief outage until the pod can be scheduled on another node (which itself might be able to be scaled down! :cold_sweat: )
See also https://github.com/knative/serving/pull/4329
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ nope ] Includes tests (if functionality changed/added)
[ nope ] Includes docs (if user facing)
[ yep ] Commit messages follow commit message best practices

See the contribution guide
for more details.
Release Notes

release-note

/test pull-tekton-pipeline-integration-tests


Snapshot/Backup/IOPs workflow related UI issues.

https://github.com/rancher/rancher/issues/5059
https://github.com/rancher/rancher/issues/5060
https://github.com/rancher/rancher/issues/5061
https://github.com/rancher/rancher/issues/5062
https://github.com/rancher/rancher/issues/5071
https://github.com/rancher/rancher/issues/5072
https://github.com/rancher/rancher/issues/5073
https://github.com/rancher/rancher/issues/5076
https://github.com/rancher/rancher/issues/3208
https://github.com/rancher/rancher/issues/5077
https://github.com/rancher/rancher/issues/5094
https://github.com/rancher/rancher/issues/5099
https://github.com/rancher/rancher/issues/5102
https://github.com/rancher/rancher/issues/5077
https://github.com/rancher/rancher/issues/5125
https://github.com/rancher/rancher/issues/5126
https://github.com/rancher/rancher/issues/5139
https://github.com/rancher/rancher/issues/4329

Following UI issue need to to be still addressed (rest from the above list have already been addressed and validated for Beta release):
rancher/rancher#3208
rancher/rancher#5126
rancher/rancher#5139
rancher/rancher#4329


Return Transaction hash from Aggregator

Resolved in https://github.com/web3well/bls-wallet/pull/186


analysis server updateContent response includes bogus result?

https://htmlpreview.github.io/?https://github.com/dart-lang/sdk/blob/master/pkg/analysis_server/doc/api.html#request_analysis.updateContent
Shows:
response: {
  "id": String
  "error": optional RequestError
  "result": {
  }
}

but does not explain what that result field is or looks like.

It's actually correct. That means that it doesn't return any data. The response (assuming there's no error) basically tells the client that the request was received.

This is different from the structure of, say, setSubscriptions:
response: {
  "id": String
  "error": optional RequestError
}

Which has no result field.


Custom Controller

Hi,
Can you please give us an insight on how to use a custom controller that would override MRS-MPC controller, for a given period of time?
Would it also be possible to override without crashing the drone, mid-flight? in the simulations

Hey, with the new version of the system, you have an option to insert a controller plugin from and external package. The example for that is here: https://github.com/ctu-mrs/mrs_core_examples/tree/master/controller_plugin


Remove angular1

Known bugs

Links in modals doesn't work (need angular2-modal upgrade)
Speak indicator icon works erratically
Missing indicator for "sticky" feed
Push to talk works is weird

Missing features

Hotkeys cheat sheet
Gridster suppport
Embeded options in chat

Not this PR's fault, but again I'm having problems to test this because I cannot install typings from most networks. Depending on the network I get different errors but all of them seem to be network related. Typings feel like a very weak thing to me... and according to the number of open issues at Typings github, I'm not the only one. :-(
Anyway, I will comment on the code now and will try to test during the afternoon.

I tried again deleting my virtual machine and performing a vagrant up from scratch. In that clean environment I'm still unable to install the typings. It fails in various ways. :-(

I've tried this PR and it works for me (apart from the issues mentioned in the description).


[WIP] Cleanup OWNERS in k/website

For https://github.com/kubernetes/test-infra/issues/11687
What does this PR do?

Adds a new team.
This PR adds website-milestone-maintainers, which specifies who can use /milestone in this repository. Currently, it gives /milestone permission to  aliases for various language owners (sig-docs-en-owners, etc.).

Cleans up existing teams.
Most of the info defined in OWNERS_ALIASES is severely outdated, and also not used for automation in this repo. The exception is sig-cluster-lifecycle-*, who actively curate their membership. (#9497)

Dependencies
This PR has cross-team dependencies:
k/org: https://github.com/kubernetes/org/pull/628
k/test-infra: TBD

/assign @Bradamant3 @zparnold

/lgtm
/approve


Feature/modules

Added modules and handlers for Summarization, NGrams, TopicModeling. Also, added the mockserver.

well done


Use tabpage handle rather than tabpage number as keys in tabline_tab_data

The plugin is using tabpage number as keys in tabline_tab_data.
The tabpage number of a tab A will be changed if the tab B whose tabpage number is less than A's is removed. Thus the tabline_tab_data key-value mapping won't be correct after this kind of tabpage deletion.
We should use tabpage handle which is a unique identifier for the tabpage
Neovim provides nvim_set_current_tabpage(), nvim_get_current_tabpage(), nvim_list_tabpages()  and api-tabpage APIs that uses tabpage handle. We can also utilize the tab-scoped variables to store data specific to a tabpage to avoid global tabline_tab_data

Thanks for reporting that issue! I didn’t know about that. I’ll update the plugin accordingly.

Well, I find a situation that we should pay attention to. The sessions in neovim don't store tabpage handles. As a consequence, when reopening a session, all tabpage handles are reassigned.
One solution I find is that we can store tabpage number in tab data so that we can remap the tabline_tab_data with tabpage number. And this should be done when SessionLoadPost autocmd event is triggered.

Maybe you can have a look at my attempt to fix this?
It works for me but probably not with sessions as tab local data doesn't seem to be stored. Maybe that can be fixed somehow?

Thanks for making a PR! I unfortunately won’t have time to look into this until this weekend, I can look into it then.

Thank you for writing the plugin!
Don't, stress. I think it might be possible to store the tab-data in a global variable that is saved by :mksession and then also register an autocmd to read the data back on session load. Looking at other plugins this seems to be how you do it. So I might update with such a fix as well.
So there is no immediate rush to merge the patch. I was rather looking for input on improvements or issues.


New UI Example Page

Fixes #
Describe the changes you have made in this PR -I have changed the design and the colour scheme in accordance with the new UI/UX design of the Circuitverse also i have added a search bar to search the projects.

Screenshots of the changes (If any) -

Coverage remained the same at 84.266% when pulling aa298814a5c0690d2e6c17bfa22c2708f0390fdf on harshgujrati:New-UI-Example-Page into 0499c088b162e52aa8bbe2dbaf68a002beef5c0c on CircuitVerse:master.


[Backport 2.x] Expand Dls Tests for easier verification of functionality

Description
Adds an integration test class for testing Dls specific workflows. This PR bases off of the Dls tests that were part of FlsDlsAndFieldMaskingTest.java.
Backports #2585
Check List

[x] New functionality includes testing
[x] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2634 (979bbb2) into 2.x (7356d9e) will decrease coverage by 0.07%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.x    #2634      +/-   ##
============================================
- Coverage     61.25%   61.19%   -0.07%
+ Complexity     3310     3304       -6
============================================
  Files           258      258
  Lines         18272    18272
  Branches       3251     3251
============================================
- Hits          11193    11182      -11
- Misses         5503     5508       +5
- Partials       1576     1582       +6

see 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


V0.0.3

Codecov Report

Merging #7 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   81.81%   81.81%
=======================================
  Files           1        1
  Lines          22       22
=======================================
  Hits           18       18
  Misses          4        4

Continue to review full report at Codecov.

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


Lenis and GSAP

Hi, I need advide. Lenis working excellent both on my dekstop and mobile. I use it with GSAP. On mobile it is very smooth. The only thing on mobile is that it is to sensitive, one touch scroll and its scrolling too fast and long. I tried to adjust the lerp between 1-2 but that is not helped. Is it any way to control the sensitivity?

Please provide a minimal reproduction link.


Locale keep switching to default locale when visit non ignore route

How can I prevent locale switch to default when visit ignore route?
Example, my default locale is th my current page is /en/view-blablabla. I go some page that in ignore /settings. I would like it to stay in en but it just switching to th.
Every localized route now is also switched to th too.

Hi @Azeranex !
Thanks for reporting this, I made a small change to nuxt-i18n's middleware to prevent it from switching the locale when navigating to an ignored route, this should fix your issue.