RFC: Do we want to support building images with unsigned rpms?

I recently discovered with @ondrejbudai that rhel nighty repos contain unsigned rpms and that the image build fails because of it. I discussed this briefly with @jikortus and we came to a conclusion that  some customers might make a use of a feature where they could disable this rpm signature checking, for example if they have their own mirror which contains some unsigned packages. It is a dnf feature as well so someone has to be using it. Maybe setting some environment variable to disable this checking would work? What do you think?

I am not sure. Some considerations:

Currently we enable signature verification for all rpms, we could disable it for all, but if we want it configurable, maybe we want it per-rpm?
If the repo metadata is trusted, then there is no need to verify the rpms at all. The reason is that the metadata contains the contant hashes for all the rpms we install, and we verify that this matches. Should we rather consider disabling the verification unconditionally?

@larskarlitski @gicmo @dvdhrm we discussed things like this before. Thoughts?

* Currently we enable signature verification for all rpms, we could disable it for all, but if we want it configurable, maybe we want it per-rpm?

How would osbuild-composer choose for which rpms to enable verification? I think having it per-repository would solve all the cases @jrusz describes, for example, using base repos of a distro (verified) + some custom repos (unverified).
(Or do you mean in osbuild's file source?)

* If the repo metadata is trusted, then there is no need to verify the rpms at all. The reason is that the metadata contains the contant hashes for all the rpms we install, and we verify that this matches. Should we rather consider disabling the verification unconditionally?

Yeah, you're right. However, I think the repo metadata is not signed in practice and cannot be trusted on that level.

FTR: This seems to be the reason for #566 (at least for me on 8.3 nightly) and also negatively affects any sort of internal testing where we've got unsigned packages.

How would osbuild-composer choose for which rpms to enable verification?

Per repo, yes. But would need to somehow pass this information down to osbuild per-rpm, as osbuild currently does not have the concept of a repo (could introduce that, of course).

Yeah, you're right. However, I think the repo metadata is not signed in practice and cannot be trusted on that level.

If the metadata is not trusted, then a roll-back attack would be fairly straight-forward. I know that fedora does not sign their metadata, and I seem to remember reading that this is ok due to the metalink being served over https. This is an odd argument in my view, but we could consider whether it is our job to "improve" on their security model?

How would this work in the situation where a customer adds a few RPMs from their internal repository? That repository might not be signed, but the customer may have some other trust setup that is sufficient for them. 🤷♂

Per repo, yes. But would need to somehow pass this information down to osbuild per-rpm, as osbuild currently does not have the concept of a repo (could introduce that, of course).

Just as a follow up for everyone else: We discussed with @gicmo how osbuild could gain support for this.

How would this work in the situation where a customer adds a few RPMs from their internal repository? That repository might not be signed, but the customer may have some other trust setup that is sufficient for them. man_shrugging

Yeah, another good point that we need this per repository. Thanks!

In osbuild/osbuild#405 the plain <sha1>: <url> is extended to support <sha1>: {url: <>, secrets: <>}. This could be extended even further to include a gpgkey (or gpgkey-id with a global list of gpg keys).
I am also still a bit unsure if we never want to do gpg verification, but I tend to think that having it at least optional with relying solely on the checksum of the package should be possible. Additionally, when the ostree source was added it was agreed that the verification should happen in the source and not the pipeline.

In https://github.com/osbuild/osbuild/pull/409 I proposed dropping signature verification entirely. Which would solve the issue at hand, if merged.

I'm now working on this again.


Electron7 compatibility

Arch Linux just upgraded their electron version to 7.
Ferdi build with my package fails of coarse again.:D I just wanted to know if I need to fix the package or Ferdi does not work with electron 7 at the moment :)

Ferdi is currently using Electron 6 so you should probably update your package to use that. Are there any specific error messages you are getting?

Buliding with electron7 finishes normally, the developer console shows this:
Uncaught Error: Module did not self-register.
at process.func (electron/js2c/asar.js:138)
at process.func [as dlopen] (electron/js2c/asar.js:138)
at Object.Module._extensions..node (internal/modules/cjs/loader.js:922)
at Object.func (electron/js2c/asar.js:138)
at Object.func [as .node] (electron/js2c/asar.js:147)
at Module.load (internal/modules/cjs/loader.js:735)
at Module._load (internal/modules/cjs/loader.js:648)
at Module._load (electron/js2c/asar.js:715)
at Function.Module._load (electron/js2c/asar.js:715)
at Module.require (internal/modules/cjs/loader.js:775)
at require (internal/modules/cjs/helpers.js:68)
at bindings (/usr/lib/ferdi/resources/app.asar/node_modules/bindings/bindings.js:112)
at Object. (/usr/lib/ferdi/resources/app.asar/node_modules/macos-notification-state/lib/index.js:1)
at Object. (/usr/lib/ferdi/resources/app.asar/node_modules/macos-notification-state/lib/index.js:81)
at Module._compile (internal/modules/cjs/loader.js:880)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:892)
at Module.load (internal/modules/cjs/loader.js:735)
at Module._load (internal/modules/cjs/loader.js:648)
at Module._load (electron/js2c/asar.js:715)
at Function.Module._load (electron/js2c/asar.js:715)
at Module.require (internal/modules/cjs/loader.js:775)
at require (internal/modules/cjs/helpers.js:68)
at Object. (/usr/lib/ferdi/resources/app.asar/node_modules/@meetfranz/electron-notification-state/lib/index.js:5)
at Object. (/usr/lib/ferdi/resources/app.asar/node_modules/@meetfranz/electron-notification-state/lib/index.js:27)
at Module._compile (internal/modules/cjs/loader.js:880)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:892)
at Module.load (internal/modules/cjs/loader.js:735)
at Module._load (internal/modules/cjs/loader.js:648)
at Module._load (electron/js2c/asar.js:715)
at Function.Module._load (electron/js2c/asar.js:715)
at Module.require (internal/modules/cjs/loader.js:775)
at require (internal/modules/cjs/helpers.js:68)
at Object. (/usr/lib/ferdi/resources/app.asar/stores/AppStore.js:1)
at Object. (/usr/lib/ferdi/resources/app.asar/stores/AppStore.js:2)
at Module._compile (internal/modules/cjs/loader.js:880)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:892)
at Module.load (internal/modules/cjs/loader.js:735)
at Module._load (internal/modules/cjs/loader.js:648)
at Module._load (electron/js2c/asar.js:715)
at Function.Module._load (electron/js2c/asar.js:715)
at Module.require (internal/modules/cjs/loader.js:775)
at require (internal/modules/cjs/helpers.js:68)
at Object. (/usr/lib/ferdi/resources/app.asar/stores/index.js:1)
at Object. (/usr/lib/ferdi/resources/app.asar/stores/index.js:2)
at Module._compile (internal/modules/cjs/loader.js:880)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:892)
at Module.load (internal/modules/cjs/loader.js:735)
at Module._load (internal/modules/cjs/loader.js:648)
at Module._load (electron/js2c/asar.js:715)
at Function.Module._load (electron/js2c/asar.js:715)
Also the console dispalys "(electron) 'getName function' is deprecated and will be removed. Please use 'name property' instead."
But this is not an acute problem...
I would change the packagebuild to electron6 but Arch does not provide a version....

Is this something that can be bumped up in priority? Older versions of electron have exploits that Imo should be fixed by upgrading electron.

Ferdi - Just Like Franz - is currently using Electron 6, which is still withing ElectronJS's support period, so there shouldn't be any exploits present that Electron 7 doesn't have.
I think we will wait until Franz is upgrading to Electron 7 - but PRs for this are still welcome.

As far as I can see, Electron 7 doesn't have any breaking change that affects us.

The file electron.asar no longer exists. Any packaging scripts that depend on its existence should be updated. #18577

@AGCaesar does the above prevent you from packaging the AUR archive?
I upgraded Electron from 6 to 7 on develop for now. I also updated all the dependencies with minor updates that were visibly not breaking anything. That includes the latest React 16.x.
Overall all I noticed is more deprecation warnings, but everything working as usual.

Available in v5.4.1-beta.1.

Builds perfectly, thanks!


Document Privacy Policy (and others)

Review access control policy / execution / application life-cycle - make sure that we have acceptable AC policy/protocol, and more importantly (and more difficult) devise a way to convey this policy to the user (mainly to the resource provider?) App developer needs to know this to understand where their application run. We should make this transparent so that user feel comfortable uploading data / running apps. Resource provider wants to know what apps are running (and who develop them) on their resource.

Per https://protect.iu.edu

Created this page > https://brain-life.github.io/docs/privacy/


A few small tweaks to the readme

Also, I removed “#!/usr/bin/python” from the top of abel/test/run.py
since it doesn’t seem necessary.

Looks great. For the way functions parameters are formatted this will probably depend on documentation (issue #25 ) so sphynx can pick it up correctly, and generate function definitions.


Observer mod

On game start, we welcome the player and explain to them what actually "Postcolonialism" does
We give them the option to become an Observer (maybe they just want to see the AI go random)
Culture fixing events will no longer fire if you are the only person alive in your country
Added a party to Observer so that it can be converted to HoI4

Note: As the Observer nation will only hold Jan Mayen, upon conversion to HoI4 it will disappear. Nevertheless, the party is needed or the converter would crash looking for a party that never existed to begin with.


Explorer not focused if no sections expanded

VSCode Version: 1.50.1 (system setup)
OS Version: Windows 10x64 Version 10.0.19042.572

Steps to Reproduce:

Collapse all the sections in the explorer sidebar.
2.Focus another area of VS Code, such as the editor.
Try using Ctrl+Shift+E to focus the explorer.

Does this issue occur when all extensions are disabled?: Yes

@isidorn I pulled the changes and tried this and when I use the above steps, I don't see it focusing. Please let me know if I'm doing something wrong.

@meganrogge thanks a lot for trying it out.
I just tried also with newest VS Code insdiers and out of source and for me it works just fine.
Do you still see this not working with latest vscode insdiers
@SaschaCowley did you also get a chance to try this out?

I didn't even know this shortcut was a thing.
It works in the latest Insiders, thanks!

@rzhao271 thanks for verifying!


Provision "root" jwt_token on installation

I'm looking into installing zitadel with the Helm chart but immediatly bootstrapping it with the terraform provider without any human interaction: https://registry.terraform.io/providers/zitadel/zitadel/latest/docs
As by the latest docs they require a token/jwt_file to be provisioned to connect to the Zitadel instance. Is there a workaround to execute the terraform provider without the need to login manually and generating a jwt token?
Per example, similar setups can be found here:

https://github.com/bitnami/charts/blob/cc8884fdf59ed94b9c7177b34070dd6ea86c9b5b/bitnami/keycloak/values.yaml#L124

@eliobischof @stebenz can you answer this question?

@thomaspetit I believe this is what you're looking for: https://github.com/zitadel/zitadel-charts/blob/main/examples/6-machine-user/README.md
Which creates a secret named whatever you configure in .Values.zitadel.configmapConfig.FirstInstance.Org.Machine.Machine.Username
https://github.com/zitadel/zitadel-charts/blob/f21506f1f21308ef89ef047a224aa7c96a8c08c5/charts/zitadel/templates/setupjob.yaml#L108

Awesome.. looks like exactly what I was looking for. I should have looked at the source code a bit better. 😅

we fixed this by running the terraform provisioner also as a kubernetes job. it took some effort to get it running, but basically we mounted the generated secret as a volume in a Job that does "terraform apply".
i can share more details if you want. i think with some work it would be possible to integrate terraform provisioning in the helm chart (where you could just specify .Values.terraformScriptConfigmap or so)

I'm actually also doing this (using the terraform operator  from galleybytes) but it seems that there is no way to provision that 'zitadel-admin-sa.json'? You found something for that? 😃
All help or ideas are welcome.

@thomaspetit my comment here might help you: https://github.com/zitadel/terraform-provider-zitadel/issues/167#issue-2197959352
I found that the Zitadel Terraform Provider tries to use the secret before it exists, so you have to do one of a few things: terragrunt apply, terraform apply -target helm_release.zitadel, or make two separate modules for the Helm release and Zitadel resources so that Terraform will correctly wait for the dependency to be resolved.

Hi thomas!
let me lay out my plan in a bit more detail. i guess you want to work "the other way around" but i wonder if that's really needed.

First i deploy zitadel in kubernetes using this helm chart. I use a values so that it will provision an initial machine user like so (yaml redacted to remove DB creds):

zitadel:
  configmapConfig:
    FirstInstance:
      Org:
        Machine:
          Machine:
            Username: zitadel-admin-sa
            Name: Admin
          MachineKey:
            ExpirationDate: "2026-01-01T00:00:00Z"
            # Type: 1 means JSON. This is currently the only supported machine key type.
            Type: 1
    ExternalDomain: zitadel.atlas.intern.kapernikov.com
    ExternalPort: 443
    ExternalSecure: true
    TLS:
      Enabled: false
  masterkey: x123x567890123456789012f4567891y

now, i want to deploy my application that uses zitadel. In my case its logical to have the zitadel config as part of the deployment procedure of my application rather than zitadel itself. I want it to be easy to deploy (create as much test instances as i want)
this also means that i want to be able to deploy+configure it when my https cert is missing or even when the DNS for my ingress is not yet good. Here i had some difficulties to tackle:

Zitadel terraform uses GRPC api, and the nginx ingress controller needs to be configured to support it. The ingress for zitadel needs this:

ingress:
  annotations:
    nginx.ingress.kubernetes.io/backend-protocol: "GRPC"
    ## edit the nnginx ingress controller configmap to make sure snippets are allowed, after editing, kill the pod
    nginx.ingress.kubernetes.io/configuration-snippet: |
      grpc_set_header Host $host;
    cert-manager.io/cluster-issuer: selfsigned-ca-issuer

that works but is not ideal because by default nginx ingress controller doesn't allow for setting config snippets (you have to enable it when installing the controller).
Second difficulty: i now have to use the public ingress to connect to my zitadel instance. I'd rather connect using the internal service in kubernetes, because this is both more robust (works when the ingress is not fine yet for whatever reason) and more secure. But if i change the uri i also change the issuer because of https://github.com/zitadel/terraform-provider-zitadel/issues/143
Because i used a selfsigned cert i need to modify the terraform docker image, to autotrust my selfsigned cert:
openssl s_client -connect $TF_VAR_ZITADEL_DOMAIN:443 | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p' > /usr/local/share/ca-certificates/example.crt
update-ca-certificates

This would also not be a problem if i could connect using plain http (it is intra-cluster anyway) but that doesn't work due to the fact that then i also change the issuer. So i think (without more support from zitadel) the good way would be to add a sidecar to the terraform  container that acts as a proxy. This way i don't have to use grpc over the ingress, and i can modify the "Host" header so that it matches the issuer in the zitadel configuration.
I don't use the operator, i use a Job as part of the postinstall of my own helmchart. So i'm free to add a sidecar. But i don't know if the terraform operator would allow that.
Greetings,
Frank

@thomaspetit my comment here might help you: zitadel/terraform-provider-zitadel#167 (comment)
I found that the Zitadel Terraform Provider tries to use the secret before it exists, so you have to do one of a few things: terragrunt apply, terraform apply -target helm_release.zitadel, or make two separate modules for the Helm release and Zitadel resources so that Terraform will correctly wait for the dependency to be resolved.

Could this issue actually be closed if we implemented zitadel/terraform-provider-zitadel#167 (comment)


MSID: 93656 Version: 2 DOI: 10.1101/2<IP_ADDRESS>1603

MSID: 93656
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>1603
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=93656
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1603

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/93656v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>1603
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/93656/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [given-names-test-16] given-names for author is made up only of uppercase letters (and spaces) 'J'. Are they initials? Should the authors full given-names be introduced instead?
warn: [ref-person-group-type-2] person-groups within a reference should be distinct. There are numerous person-groups with the person-group-type editor within this reference (id=c27).
warn: [ref-person-group-type-2] person-groups within a reference should be distinct. There are numerous person-groups with the person-group-type editor within this reference (id=c36).
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c32) has a publication-type='other'. Is that correct?
warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c33) has a publication-type='other'. Is that correct?


All health gained from stats/relics is lost upon taking any damage

Minecraft 1,17,1
Fabric: 0.11.7
Mods: PlayerEx(3.0.6), RelicEx(3.0.1), and their requirements. Trinkets Fabric(3.0.4), Data Attributes(1.0.6)
Bug: -All- extra hearts earned through constitution/relics are lost upon a single hit when a Relic with +Health is equipped. Taing any amount of damage for instance will hit you for all of your non-default health while wearing a relic with +health/+con on it.
How to Reproduce: Give yourself about 10-20 levels, put enough points into constitution for a few extra hearts to see the difference, I'd say around 10 extra hearts. Equip -one- relic that has +health or +con on it, you will immediately take damage for all extra health gained from relics and stats combined. Heal up to full health, have 20 or so hearts combined with Relics & Stats, then take any damage.
PS: Another note entirely would be health not working well with other mods that also add health, but in this example, I only used the requirements for PlayerEx and RelicEx

I updated to the newest version of well this time. (Fabric 0.12.3) While the problem isn't -as- bad it seems there are a few items that will still break health damage. For instance, this first relic (an amulet) doesn't break the damage to health calculation.

Whereas this helmet most certainly will.

Both have +health on them but varying amounts, but the +%con items seem to be having the same bug as well! It's just more rare to find these items that break the health bar in the newer fabric I feel but it still persists.

Hi there,
Thanks for the detailed report. I'll look into it and get back to you.

Hi again,
I've spent some time testing, and I've verified the issue.
The bug is only present with armour type relics (i.e. not trinkets). I've taken an extra couple of days to come up with a solution before replying, but no dice I'm afraid.
I have some idea about what's causing the bug, but I think it's an underlying issue with Data Attributes, which makes it hard to solve.
I'll probably not manage to come up with a solution before 1.18 drops, at which point I'm aiming to rework everything after receiving some design considerations from other developers.
Thank you for taking the time out to bring this to my attention, sorry I couldn't be of more help.

A temporary fix might be to prevent all attributes that modify health on body armor and helmets.

Ah, that's true! Might be a good solution for now until there's another update to work with. Honestly, in my case, I just tell my server not to use Armor Relics with +HP/CON on them for now which is fine if people are aware of it. Though I think your best bet would be to hotfix just in case just removing HP/CON from the armor tables until more time and incentive is added by a new update or features.
It's honestly up to you and the time required! Removing the HP/CON would prevent further reports or bugs involving this minor'ish issue while leaving it in with a simple disclaimer would be easier until you get to sit down and work with a new update.
This definitely isn't game-breaking, just requires a workaround or awareness of it!

Yeah @Lorelando I think that's a good idea; I will try and release a hotfix before the end of this week.

Fixed.


Return direct3 d10

Hi, xoofx!
I returned Direct3D10 (#733), please look.
Thanks.

Wow, that's amazing, thanks a lot for this. This should fix issue #733


Device pyaudio

Simple PyAudio Node : grab and playback.

Code looks good to me. Haven't been table to get the test running, though:
Traceback (most recent call last):
  File "pyacq/devices/tests/test_audio_pyaudio.py", line 76, in <module>
    test_local_app_in_out()
  File "pyacq/devices/tests/test_audio_pyaudio.py", line 26, in test_local_app_in_out
    format = 'int16', chunksize = 1024)
  File "/home/luke/work/pyacq/pyacq/core/node.py", line 111, in configure
    self._configure(**kargs)
  File "/home/luke/work/pyacq/pyacq/devices/audio_pyaudio.py", line 75, in _configure
    rate=int(self.sampling_rate), input_device=self.input_device_index),\
  File "/home/luke/work/pyacq/venv3/lib/python3.4/site-packages/pyaudio.py", line 934, in is_format_supported
    return pa.is_format_supported(rate, **kwargs)
ValueError: ('Invalid number of channels', -9998)

I'll add this to my to-fix list.

This error come from pyaudio when you chosse a pair of input/ouput whihch is not full duplex.
Try others input_device_index/output_device_index.
Playing while grabbing is not so easy in pyaudio it depends on hostapi.

Aha! I probably don't have full duplex on any of my built-in devices.

Grumbly segfaulty. (I'm back!)

I spend lot of time on this. Any help would be appreciate.
With PyQT5 , there is no segfault.
the main problem is that in the same test module I app.exec_() and app.quit() several times for different test. And it is hard to known when object are garbage collected and remove by qt4 itself. I guesss you have more experiment than me. I never before start/stop the same app several app.
Note that if there is only one test then no seg fault.

I culled it down to two tests required to cause the fault:
pyacq/test_segfault.py::test_visual_node_both_in_main_qapp_and_remote_qapp PASSED
pyacq/test_segfault.py::test_ThreadPollInput PASSED

===================================================================================================== 2 passed in 6.91 seconds ======================================================================================================
Segmentation fault

Removing app.exec_() / quit() from the first test did not help.


Can we just update this repo

Is there a reason why this repo doesn't get updated or does it live on in another project. It seems pretty awesome but I'm afraid of using it due to the lack of maintenance around it. There's also the flashlight work that's done by Facebook as well but it's not really setup to be used in pytorch.

I'm also looking for substitutes, some compiling problems are really annoying


--realpath and --fullpath have no effect on make:migration

Laravel Version: 8.51.0
PHP Version: 7.4.21
Database Driver & Version: MySQL 8.0.21

Description:
--realpath is declared as a switch and does not accept values.
As a result, returned value is always false, caused by $this->option('realpath') always false.
https://github.com/laravel/framework/blob/277c2fbd0cebd2cb194807654d870f4040e288c0/src/Illuminate/Database/Console/Migrations/BaseCommand.php#L37-L40
--fullpath also faces the same issues and make:migration currently will never display the fullpath
https://github.com/laravel/framework/blob/277c2fbd0cebd2cb194807654d870f4040e288c0/src/Illuminate/Database/Console/Migrations/MigrateMakeCommand.php#L113-L115
Steps To Reproduce:
All 3 commands below write to the same folder, which I think is incorrect, --fullpath also has no effect.
php artisan make:migration testmigration --path=/abcd
php artisan make:migration testmigration --path=/abcd --fullpath
php artisan make:migration testmigration --path=/abcd --fullpath --realpath

I can file a PR. Thank you.

@ajuchacko91 yes?

Ok, but doesn't work for me. I'm on a CentOS 8 guest running off Windows 10.

Appreciating PR's, thanks.

I realized my issue, sharing here in case it helps someone
# doesn't work
$this->call('make:migration', [
    'name' => "table_name",
    '--fullpath' => '',
    '--realpath' => '',
    '--path' => $path,
]);

# works
$this->call('make:migration', [
    'name' => "table_name",
    '--fullpath' => true,
    '--realpath' => true,
    '--path' => $path,
]);


Proxy configuration update without 404

Changes to service, ingresses, or CRDs can cause 404 for live traffic due xDS protocol not staging updates between RDS/CDS/EDS.
Proxy should not apply partial xDS configuration update.
Also:

https://github.com/istio/pilot/issues/926
https://github.com/istio/istio/issues/1038
https://github.com/istio/pilot/issues/1325

isn't this a duplicate of v2 API issue ( #1257 ) ? Because as part of the move, ADS is a natural outcome.

Yes, it's a dependency, but we need to verify that it solves the issue and add regression tests.

Issue moved to istio/istio #1471 via ZenHub


Vulnerability CentOS 7 : openssl (CESA-2018:0998)(Medium) was found with NESSUS

A vulnerability: CentOS 7 : openssl (CESA-2018:0998)(Medium) was found.
Scanner: NESSUS
Asset Information:
Asset name: ct1
IP: <IP_ADDRESS>
Scan was run on: 2018-09-05 09:15:06
Notification Rules Triggered: External Assets - New Critical Vulnerabilities to Github
Please see Nucleus for more information on these vulnerabilities

I need help from somebody!


spawn issue in tauri

Hi j-tai!
girlboss is working well for me in my web server.  But, I also use that web server as a library for a tauri app.
The issue I'm experiencing there is that the task never starts.  I checked out girlboss and put in a bunch of printlns.
It gets to the point where it calls tokio::spawn, but the task itself never printlns.
I tried a plain tauri::spawn and it was able to kick off a task - "I'm in the spawn" below.
      info!("PrivateRequests::SyncRemote");
      let dbpath: PathBuf = state.config.orgauth_config.db.to_path_buf();
      let file_path: PathBuf = state.config.file_path.to_path_buf();
      let uid: i64 = uid;

      let jid = new_jobid(state, uid);
      info!("SyncRemote jobid: {:?}", jid);

      println!("right here");

      let jh = tokio::spawn(async {
        println!("I'm in the spawn");
      });

      println!("await: {:?} ", jh.await);

      let job = state
        .girlboss
        .start(jid, move |mon| async move {
          println!("gb thread");
          let gbm = GirlbossMonitor { monitor: mon };
          // spawn thread, local runtime.  success.
          std::thread::spawn(move || {
            println!("sync thread");
            let rt = Runtime::new().unwrap();
            let local = LocalSet::new();
            let mut callbacks = &mut zknotes_callbacks();
            write!(gbm, "starting sync");
            let r = local.block_on(
              &rt,
              sync::sync(&dbpath, &file_path, uid, &mut callbacks, &gbm),
            );
            match r {
              Ok(_) => write!(gbm, "sync completed"),
              Err(e) => write!(gbm, "sync err: {:?}", e),
            }
          });
        })
        .await?;

      info!("job: {:?}", job);

the sequence of printlns below.  (info! goes to a separate file right now)
right here
I'm in the spawn
await: Ok(())
girlboss-manager::start
girlboss-manager::start2
girlboss-manager::start3
girlboss-job::start1
girlboss-job::start2
girlboss-job::start3
girlboss-job::start7
girlboss-manager::start4
sr: JobStatus
zimsg

So, there's some unwanted interaction between girlboss and tauri, it seems.  Got any ideas what might be going on?  TY!

Whoa, that worked!?  What does it mean?
right here
I'm in the spawn
await: Ok(())
girlboss-manager::start
girlboss-manager::start2
girlboss-manager::start3
girlboss-job::start1
girlboss-job::start2
girlboss-job::start3
girlboss-job::start7
girlboss-manager::start4
girlboss-job::start4
gb thread
girlboss-job::start5
girlboss-job::start6
sync thread
sr: JobStatus
zimsg
sr: JobStatus

I'm pretty sure I found the problem - going back up the call chain it turns out I had created an actix_rt in which to run my web handler in tauri, since it's !Send and actix_rt is ok with that.  Its been long enough that I forgot all about it!  So yeah, single threaded runtime.
I probably won't have time to work on it the next few days, but seems solvable now, will post back when its working.

Both the trouble runtime, and the one where it was working (actix_web handler) report num_workers of 1.  But in one case it works and the other not so much.
tokio metrics in both:
num_workers 1
num_alive_tasks 0
global_queue_depth 0

Sort-of solved the problem by using a non-async handler in tauri.  GB can start a job in that and it works without needing the sleep.  Don't really get it, but I think I'm going to leave it there for now, so I'm closing this issue.

Hi @bburdette, I've published a new version (1.0.0-alpha.1) which supports both tokio and actix-rt. If you have time, would you be able to check it out & provide feedback? :)
Hopefully, since girlboss now supports actix-rt directly, you won't need the std::thread::spawn workaround anymore.

cool!  I'll give it a try when I get a chance!

Finally circling back to this.  I'm having trouble using the new girlboss in my actix_web state:
https://github.com/bburdette/zknotes/blob/1dbfec7dd30844ad870902a4562cbecfffdd5bde/server-lib/src/state.rs#L6
The compile error is:
error[E0277]: `RefCell<std::option::Option<tokio::task::JoinHandle<()>>>` cannot be shared between threads safely
   --> server-lib/src/lib.rs:864:32
    |
864 |     let server = HttpServer::new(move || {
    |  ________________---------------_^
    | |                |
    | |                required by a bound introduced by this call
865 | |     let staticpath = c.static_path.clone().unwrap_or(PathBuf::from("static/"));
866 | |     let d = c.clone();
867 | |     let cors = Cors::default()
...   |
913 | |       .service(web::resource("/{tail:.*}").route(web::get().to(mainpage)))
914 | |   })
    | |___^ `RefCell<std::option::Option<tokio::task::JoinHandle<()>>>` cannot be shared between threads safely

For actix-rt, you need to create a separate Girlboss instance per thread. That's because each actix-rt task (and hence, each girlboss Job) is confined to its own runtime.

Ok done!  That works, I can kick off the sync task that way in my actix_web server.
But I'm hazy on how to do job tracking under this regime.  When a request comes in to the web server to inquire about a job, I can't return any information since the gb manager cannot be stored in the web server state.  Plus each thread requires its own Girlboss manager, so there's no one-stop-shop for job status.

Hmmm, that's a tricky problem. Right now, girlboss is designed such that a Job stores the job's current status and its JoinHandle. However, in actix-rt, JoinHandle is not Send, but in theory, the job's current status could be.
I'll have to think about how to solve this in girlboss's API -- I somehow have to make "Job without the join handle" implement Send...
I'll look into it!

@bburdette I've released girlboss 1.0.0-alpha.4, with a few changes:

Monitor no longer needs a Runtime generic argument
Girlboss can now store monitors rather than jobs: girlboss::Girlboss<K, Monitor> should implement Send + Sync, allowing you to query the job status from any thread.

Let me know how it works out! And thank you for the feedback!

Ok job tracking is go!
But its a mixed bag as far as my apps.  In the actix_web server the sync process is kicked off and runs.  But in tauri the task never starts.
I recently discovered that actix_web doesn't actually use actix_rt in its handlers anymore, and hasn't for a few years.  News to me.
On the tauri side, I'm spawning a thread to handle a message, and in that thread I make a new actix_rt System, and it calls the same gb start that works in actix_web mode.

Okay, I took a look at your code (on the gb-actix-rt branch), and I think I see the issue.
You're creating an actix-rt System in the thread. Then, you spawn the girlboss Job within that system. Next, you block_on the future returned by zk_interface_loggedin, but this does not wait for the job to complete -- the future merely starts the job and then resolves. Then, the thread terminates, destroying the System along with it, leaving no chance for the System to start polling (i.e., running) the Job.
Here are some options:

You could make the thread wait for the job to complete rather than just "spawning and forgetting" it.
After the rt.block_on call, use rt.run() to allow the System to run its event loop, thereby running the Job. At the end of the Job's code, you might want to run actix_rt::System::current().stop() to make it exit the event loop, thereby making rt.run() return (not 100% sure if this is necessary).

Both of these methods will keep the thread (and hence the System) alive until the job completes. Hence, if you still want to run the job asynchronously, you might not want to join() the thread right after spawning it.


Dump when pulling repo with objects of type BDEF

Due to latest code changes in SAP development environment, we are getting dumbs when pulling a repo with objects of type BDEF. The reason for this is a type change in our latest development systems. As a consequence this assignment leads to dumbs.
This problem can only be reproduced in SAP systems with this code change.
To solve this problem, we suggest to change this type assignment
https://github.com/abapGit/abapGit/blob/8555d1a0da6400fc04dbb0e3851e69bf76e5b720/src/objects/zcl_abapgit_object_bdef.clas.abap#L232
to
CREATE DATA lr_metadata  TYPE ('CL_BLUE_SOURCE_OBJECT_DATA=>TY_OBJECT_DATA-METADATA').
I would like to create a pull request that does that change. In order not to have to create a fork, I would like to have rights to contribute. Can you assist please, @larshp?

I dont have access to do abapGit development in any systems that supports BDEFs
@ThomasPloski can you help?

I can do the changes but would prefer having rights to contribute to this repository without having to fork.

@wurzka try again

I can do the changes but would prefer having rights to contribute to this repository without having to fork.

What's the reason for not forking the repo?

@larshp works, thanks!
@christianguenter2  For this single issue, there is no reason why forking is worse. For further issues it would be better to be able to open pull requests so others can also contribute to them easily


Add prometheus rules verification in CI

This pull request does the following:

Adds a new GitHub Actions workflow that verifies the latest Prometheus rules YAML files when changes occur in the metrics/* directory.
Modifies the Makefile to include a new target 'verify-latest-prometheus-rules-yamls', which triggers the verification script.
Creates a new script 'verify-latest-prometheus-rules-yamls.sh' in the hack directory. This script checks for uncommitted changes in the Prometheus rules files and prints a success message when no out-of-tree changes are found.

cc @umangachapagain


Cannot load because 'pkg' is not a package

Hi, I'm trying to use pkg_tar and pkg_mkdirs but the load's fail and the rules_pkg directory in the cache looks only like the pkg/ subdirectory rather than the top-level. I've verified the contents of the .tar.gz's referenced by the workspace urls so I'm not sure where it's going wrong.
❯ bazel version
Build label: 5.2.0

WORKSPACE:
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")

http_archive(
    name = "rules_pkg",
    sha256 = "8a298e832762eda1830597d64fe7db58178aa84cd5926d76d5b744d6558941c2",
    urls = [
        "https://github.com/bazelbuild/rules_pkg/releases/download/0.7.0/rules_pkg-0.7.0.tar.gz",
        "https://mirror.bazel.build/github.com/bazelbuild/rules_pkg/releases/download/0.7.0/rules_pkg-0.7.0.tar.gz",
    ],
)

load("@rules_pkg//:deps.bzl", "rules_pkg_dependencies")

rules_pkg_dependencies()

Relevant BUILD load statements:
load("@rules_pkg//pkg:tar.bzl", "pkg_tar")
load("@rules_pkg//pkg:mappings.bzl", "pkg_mkdirs")

The error:
❯ bazel build //services/api:dev-latest
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
ERROR: Skipping '//services/api:dev-latest': error loading package 'services/api': Label '@rules_pkg//pkg:tar.bzl' is invalid because 'pkg' is not a package; perhaps you meant to put the colon here: '@rules_pkg//:pkg/tar.bzl'?
WARNING: Target pattern parsing failed.
ERROR: error loading package 'services/api': Label '@rules_pkg//pkg:tar.bzl' is invalid because 'pkg' is not a package; perhaps you meant to put the colon here: '@rules_pkg//:pkg/tar.bzl'?
INFO: Elapsed time: 17.897s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)

Looking at the bazel cache:
❯ ls ~/.cache/bazel/_bazel_kmaris/f591fac78c330cff025c1124cfd2e525/external/rules_pkg
archive.py  build_tar.py  deps.bzl      helpers.py   make_deb.py  path.bzl  README.md  rpm.bzl      WORKSPACE
BUILD       build_zip.py  experimental  __init__.py  make_rpm.py  pkg.bzl   releasing  version.bzl

It makes sense there's no pkg package, but something in the unpackaging is going wrong. Any pointers why?

And to add, manually downloading the tarball and extracting in the cached folder does result in a successful build.

I had this issue too. The fix for me was to move the rules_pkg higher in my WORKSPACE file.
Maybe something else is in the WORKSPACE file is loading rules_pkg at an older version, when //pkg/... didn't exist.

@sashamor Thank you for taking the time to comment with that advice, your fix worked for me! 😄


couldn't use ng commands in angular2?

$ ng init
Cannot find module './models/config'
Error: Cannot find module './models/config'
at Function.Module._resolveFilename (module.js:455:15)
at Function.Module._load (module.js:403:25)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (C:\cygwin64\home\ishu.mishra97\frontpage\frontpage-angular2-app\node_modules\angular-cli\addon\ng2\index.js:4:16)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)

4200 port is already in use so i have changed it and rewrite it by changing to 3130 then again it is saying that 3130 is already in use...whats the f*** is happening plz chalk it out.?


Don't nest compute calls

In some cases Iris is nesting a compute call within a compute call. This has the undesirable behaviour of generating a loop that increments the total number of threads being used by the realisation operation, which can cause various problems, particularly when running on a cluster:

in the less serious, running in parallel on the cluster takes a lot longer than running synchronously on the cluster (if you interrupt the job it looks like the majority of the time is taken up switching threads)
in the more serious, the maximum number of allowed threads is hit and the job fails.

This is happening in a case where a cube loaded from a PP file with lbpack=2 (packed on masked land/sea points) is realised. In this case compute is called twice, one call nested inside the other, for a single PP field - the original call to realise the field's data and the nested call to realise the attached land/sea mask that comes from a separate PP field.
Here's the full stacktrace:
cube.data

Traceback (most recent call last):
  File "/.../create_jules_ic_v2.py", line 279, in <module>
    main()
  File "/.../create_jules_ic_v2.py", line 274, in main
    nameoffile=create_jules_ic(Ddate,INDIR,UM_FILE_IN,OUTDIR)
  File "/.../create_jules_ic_v2.py", line 152, in create_jules_ic
    print(type(cube_vegfrac.data <=0.0001))
  File "/.../python2.7/site-packages/iris/cube.py", line 1644, in data
    return self._data_manager.data
  File "/.../python2.7/site-packages/iris/_data_manager.py", line 216, in data
    result = as_concrete_data(self._lazy_array)
  File "/.../python2.7/site-packages/iris/_lazy_data.py", line 128, in as_concrete_data
    data = np.asanyarray(data.compute())
  File "/.../python2.7/site-packages/dask/base.py", line 143, in compute
    (result,) = compute(self, traverse=False, **kwargs)
  File "/.../python2.7/site-packages/dask/base.py", line 392, in compute
    results = get(dsk, keys, **kwargs)
  File "/.../python2.7/site-packages/dask/threaded.py", line 75, in get
    pack_exception=pack_exception, **kwargs)
  File "/.../python2.7/site-packages/dask/local.py", line 521, in get_async
    raise_exception(exc, tb)
  File "/.../python2.7/site-packages/dask/local.py", line 290, in execute_task
    result = _execute_task(task, data)
  File "/.../python2.7/site-packages/dask/local.py", line 270, in _execute_task
    args2 = [_execute_task(a, cache) for a in args]
  File "/.../python2.7/site-packages/dask/local.py", line 271, in _execute_task
    return func(*args2)
  File "/.../python2.7/site-packages/dask/array/core.py", line 74, in getter
    c = a[b]
  File "/.../python2.7/site-packages/iris/fileformats/pp.py", line 647, in __getitem__
    self.mdi, self.mask)
  File "/.../python2.7/site-packages/iris/fileformats/pp.py", line 770, in _data_bytes_to_shaped_array
    land_mask = mask.data.astype(np.bool)
  File "/.../python2.7/site-packages/iris/fileformats/pp.py", line 1033, in data
    self._data = as_concrete_data(self._data)
  File "/.../python2.7/site-packages/iris/_lazy_data.py", line 128, in as_concrete_data
    data = np.asanyarray(data.compute())
  File "/.../python2.7/site-packages/dask/base.py", line 143, in compute
    (result,) = compute(self, traverse=False, **kwargs)
  File "/.../python2.7/site-packages/dask/base.py", line 392, in compute
    results = get(dsk, keys, **kwargs)
  File "/.../python2.7/site-packages/dask/threaded.py", line 70, in get
    pool = ThreadPool(num_workers)
  File "/.../python2.7/multiprocessing/pool.py", line 727, in __init__
    Pool.__init__(self, processes, initializer, initargs)
  File "/.../python2.7/multiprocessing/pool.py", line 159, in __init__
    self._repopulate_pool()
  File "/.../python2.7/multiprocessing/pool.py", line 223, in _repopulate_pool
    w.start()
  File "/.../python2.7/multiprocessing/dummy/__init__.py", line 75, in start
    threading.Thread.start(self)
  File "/.../python2.7/threading.py", line 736, in start
    _start_new_thread(self.__bootstrap, ())
thread.error: can't start new thread

To fix this we can make sure that the original field and the land/sea mask are realised together and not separately. Note that there might be other cases where such nested compute calls might be possible within Iris.
Originally raised as https://github.com/dask/dask/issues/4240, but re-raised here as it's now clear that the origin of the problem is in Iris.

Closed by https://github.com/SciTools/iris/pull/3255


New version: RAFF v0.6.1

Registering package: RAFF
Repository: https://github.com/fsobral/RAFF.jl
Created by: @fsobral
Version: v0.6.1
Commit: fded7baa0be0f689ddb4025b4c5a3cec0f0f8500
Reviewed by: @fsobral
Reference: https://github.com/fsobral/RAFF.jl/commit/fded7baa0be0f689ddb4025b4c5a3cec0f0f8500#commitcomment-33828126
Release notes:

Changed the signature of model functions model and gmodel!. Now we are using a more statistical notation to functions and comments.

What happened to 0.6.0?

Ok, you are right. I thought it was not necessary, since the PR for 6.0 had been closed. I'll send another request for v0.6.0. Sorry for the mess, I'm getting used to Registrator and the way Julia organizes its versions.


Clean Contributors Doc

By splitting original huge single page into nice chapters, and various related doc MD format fixes.

@jingtang10 as requested and discussed! LGTM?


lyxal: Watch [Pp][Ww][Cc][Hh]ousing

lyxal requests the watch of the watch_keyword [Pp][Ww][Cc][Hh]ousing. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

[Pp][Ww][Cc][Hh]ousing has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'no matches; presumed typo; see also https://chat.stackexchange.com/transcript/message/64891205#64891205'


How to know target options trigger crashes?

I found crashes with yuan-fuzz, but how to know which target argv trigger the crash?

https://github.com/zodf0055980/Yuan-fuzz#interpreting-output


Use value_for_platform to specify different package name for differen…

…t platform_version.
Signed-off-by: Ali Yahya<EMAIL_ADDRESS>Description
Revert previous change that removed checking platform_version.
Issues Resolved

Package name for RHEL 6-based platforms should use iptables instead of iptables-services

Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Closing in favor of #94


Toggle button State is not detecting

Inside SwipeFlingAdapterView when I used Toggle button it is not detecting its state and shows null pointer reference.ASAP give Solution/Answer
(Even i texted code in separate layout, and in Same Layout(by using tow controls toggle and swipeFlingAdapterView)).
Thank You,
Anant Shah

Could you post your code?

I Found Solution of my problem, I have to register toggle button inside the adapter and set its property focusable="false" and then create interface and listener to handle its events.


[Feature Request]: Want to Fix UI of Profile Page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

Use Case
It will enhance user Experience
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Hey @dinxsh can you review this issue and assign it to me

assigned @nishant0708 looking forward to this

For sure @dinxsh . Can you review my other issues as well


OneDrive authentication failure

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

You must agree to search and the code of conduct. You must fill in this entire template. If you delete part/all or miss parts out your issue will be closed.
If you are technical, you should reporting bugs along the lines of https://marker.io/blog/how-to-write-bug-report. If you are not technical, we will make allowances, please try to make an effort to understand the process.
Describe the bug
Attempting to open a drawing stored on Microsoft OneDrive, the authorization fails.
To Reproduce
Steps to reproduce the behavior:

Go to draw.io
In the 'Save diagrams to:' dialog, select OneDrive
In the 'Authorization required' dialog, select the Authorize button
A new popup windows opens to login.microsoft.com asking me to Sign In to my Microsoft account and authorize draw.io access to my account. select to allow.
The popup window now redirects back to https://app.diagrams.net/microsoft?code=code&state=state
The popup window shows HTTP ERROR 401
I am unable to load the drawing from my OneDrive account

Expected behavior
Authorization for OneDrive succeeds and I can use draw.io to work on my diagram
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 22.1.21

Desktop (please complete the following information):

OS: Windows
Browser: MS Edge
Browser Version:  120.0.2210.133

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:
Yes
And with different MS Accounts
Additional context
Loading a file stored locally does work. Work-around is to use the local file.

I'm getting this issue too - thought it was just my machine so I've tried the following:

restarted browser
restarted machine
tried incognito
enterprise and personal MS accounts

and still getting access denied

I can confirm. Few days ago it was working

Can confirm I'm seeing the same behaviour, have also tried the things mentioned above.

Please stop with the "me too"s, only report if you have technical information to add.

seeing this error in the Azure Logs

This should now be fixed, thanks for the report. We've implemented steps to avoid this happening again.

can confirm that it's working!

Thank you for the swift resolution. Awesome 🎉


Fixes #642 - Allow specify all deploy preferences on deploy

Since we display the dialog on each deploy, the setting to prompt for project ID on deploy has been removed.

stop previous version only applies to flex projects where 'promote' is true.

@patflynn stop-previous-version works with Standard as well if the scaling is set to basic or manual. I just did a quick test with basic scaling to verify it.

Interesting. @etanshall, something to think about.
Andras, can you add a screenshot of what this UI looks like?
On Thu, Sep 8, 2016 at 1:32 PM, Andras Kerekes<EMAIL_ADDRESS>wrote:

@patflynn https://github.com/patflynn stop-previous-version works with
Standard as well if the scaling is set to basic or manual. I just did a
quick test with basic scaling to verify it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/cloud-tools-for-eclipse/pull/674#issuecomment-245675552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5Hdy-tGN3-1e44WK4YQn22T7IpP5cks5qoEaYgaJpZM4J4OoC
.

Screenshot added to description

I really like the wording here. I think that you've really improved clarity of what these flags do.
One thing we did is made the 'stop previous version' checkbox a child of 'promote' and only enabled if promote is set.  To have 'promote' unchecked and 'stop previous version' checked apparently does nothing.
@etanshaul maybe we can borrow some of this wording for our UI.

:+1: yes I like the wording too. "promote" alone never really meant much to me. I'll incorporate this into IntelliJ as well.

I can press the OK button while the project ID validation fails.

Also, my project doesn't set any default project ID (i.e., created via the wizard while leaving the project ID field blank. I noticed that when the preference page first pops up, the project ID validation isn't executed.

Lastly, dialog margins don't seem to be set properly.

Latest screenshot:

The dialog should be retitled as it is opened on the actual "Deploy" operation.  Deployment Configuration?

@elharo @chanseokoh @briandealwis PTAL

LGTM


Add icon for noelle.dev

Created using Noelle's Animated Pixel Gradients Maker if you're curious.

pretty :3


Buildwrapper in panic

When I open this module I get panic message:

I have source distributable to share. But here it did not gave me rights to upload it, so I have uploaded it to GHC ticket.
As message states I posted this issue on GHC bug tracking system here:
https://ghc.haskell.org/trac/ghc/ticket/9877
I have posted it here, too, because it is not reproducable from cabal build, but only in EclipseFP.

Do I have to report something more to GHC than this:
https://ghc.haskell.org/trac/ghc/ticket/9877


outdated screenshot  on renaming a repository page

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
renaming a repo
What part(s) of the article would you like to see updated?

it shows the default branch setting below repository name instead of the toggles for tempalte and sign off
it shows the default branch as "master"

Additional information
No response

opened #19605 instead


"Use hierarchical imports" - still needed?

There are hints suggesting hierarchical imports, such as
import List -- import Data.List

Are they still needed? I can't find a module named "List" or "Monad" in base (or anywhere). Maybe they existed a long time ago?

Good spot - these can be deleted

Is it fine to ask what the original motivation is? (curiosity)

In the before times Haskell didn't support module names with . in them, eg. its an addition after-the-fact to Haskell98. Therefore base shipped with List, IO etc. Time went by, people realised that people only like long module names if they can put a dot in them (e.g. Data_List everyone frowns at, Data.List is just fine, humans are weird). Then the modules moved and List was there for compat, so HLint advised you to move over. And then, sometime later, List got deleted, so its no longer an interesting hint.

Good to know! That must be ancient history - even base 3.0.3 doesn't seem to have List.

Yeah, I think this code is at least a decade old - HLint as a whole is 14 years old.


Inconsistent author names --> author pages

Some author names that are automatically pulled from the PDFs are not properly recorded/formatted, so the website is not linking the names to the appropriate author pages. Need to fix this by ensuring that all variations of an authors name will be properly mapped to their page.

I added another mapping for Todd. Both Todd A. Pascal and Todd Pascal should refer to him now.


Added rubber-info checker for tex

rubber-info is mainly a log parser.  This is not a problem by itself, but the checker doesn't do anything to make sure the logs are up to date after you make changes to the sources.  This is not apropriate for syntastic.  There are also other problems, f.i. the checker looks for rubber instead of rubber-info for activation, and lines such as the following are ignored by your errorformat:
vuldmexp.tex:170-172: Overfull \hbox (2.01886pt too wide) (page 3)

Sorry, but no.


AWS Deployments: close redis external port

What do these changes do?
Do not expose self-hosted Redis. There is no need since we only need it for AWS Master which now uses AWS-hosted Redis
Related issue/s

https://github.com/ITISFoundation/osparc-ops-environments/issues/241

Related PR/s
Checklist

[ ] I tested and it works

Is the AWS memoryDB already in aws-master active? Nice!

And used. We use Redis OSS (not MemoryDB) because it supports multiple databases while MemoryDB does not support multiple Redis Databases.
The reason is MemoryDB is implemented as Redis Cluster and Redis Cluster does not support SELECT statement (required for multiple databases) by definition of it.
it is documented here: https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/blob/configs/deployments/osparc-master-zmt.click/terraform/ElasticCache.tf?ref_type=heads#L44-46


Content API Discrepency - update() is not an available method

It seems that the documentation references a method not available on the products resource. Our team is really hoping there's a way to partially update resources even with this method missing, as insert() seems to always overwrite the entire product representation.
Environment details

OS type and version:
Python version: 3.6.12
pip version: 20.3.3
google-api-python-client version:Name: google-api-python-client
Version: 2.3.0
Summary: Google API Client Library for Python
Home-page: https://github.com/googleapis/google-api-python-client/
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /usr/local/lib/python3.6/site-packages
Requires: httplib2, google-auth-httplib2, google-auth, six, uritemplate, google-api-core
Required-by: pygsheets

Steps to reproduce

Instantiate a content API client with the most recent version
Attempt to fetch the products().update attribute as defined in the documentation

Code example
     >>> client = discovery.build('content', 'v2.1', credentials=<credentials>)
     >>> client.products().update
     Traceback (most recent call last):
       File "<console>", line 1, in <module>
     AttributeError: 'Resource' object has no attribute 'update'

Stack trace
     Traceback (most recent call last):
       File "<console>", line 1, in <module>
     AttributeError: 'Resource' object has no attribute 'update'

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

Hi @Syntaf,
I'm able to see the <client>.products().update() instance method for this client. Please note that update() is an instance method not an attribute.
>>> from googleapiclient.discovery import build
>>> client = build('content', 'v2.1')
>>> client.products().update()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/google/home/partheniou/git/google-api-python-client-upstream/googleapiclient/discovery.py", line 1012, in method
    raise TypeError('Missing required parameter "%s"' % name)
TypeError: Missing required parameter "merchantId"

@parthea Interesting, thanks for replying and confirming it does indeed exist. Do you have a different package version than what I posted above?
>>> client = discovery.build('content', 'v2.1', credentials=credentials)
>>> client.products().insert
<bound method createMethod.<locals>.method of <googleapiclient.discovery.Resource object at 0x7f6c1d08d3c8>>
>>> client.products().update
Traceback (most recent call last):
  File "<console>", line 1, in <module>
AttributeError: 'Resource' object has no attribute 'update'

I'm going to close this issue as I'm not able to reproduce the issue but please feel free to re-open it.


Not compatible with textual 0.9.1

If I try a poetry add rich-pixels I get the following dependency conflict:
❯ poetry add rich-pixels
Using version ^2.0.0 for rich-pixels

Updating dependencies
Resolving dependencies... (0.1s)

Because no versions of rich-pixels match >2.0.0,<3.0.0
 and rich-pixels (2.0.0) depends on rich (>=12.0.0,<13.0.0), rich-pixels (>=2.0.0,<3.0.0) requires rich (>=12.0.0,<13.0.0).
And because no versions of rich match >12.6.0,<13.0.0, rich-pixels (>=2.0.0,<3.0.0) requires rich (>=12.0.0,<=12.6.0).
And because textual (0.9.1) depends on rich (>12.6.0)
 and no versions of textual match >0.9.1,<0.10.0, rich-pixels (>=2.0.0,<3.0.0) is incompatible with textual (>=0.9.1,<0.10.0).
So, because transitions depends on both textual (^0.9.1) and rich-pixels (^2.0.0), version solving failed.

I've relaxed the Rich dependency so this should be fixed now


feat: improve transaction signing

Try out this version of the Stacks Wallet - download extension builds.

Apologies on the larger PR.
This PR fixes these issues:

https://github.com/blockstack/stacks-wallet-web/issues/1172
https://github.com/blockstack/stacks-wallet-web/issues/1165
https://github.com/blockstack/stacks-wallet-web/issues/1146
https://github.com/blockstack/stacks-wallet-web/issues/1115
https://github.com/blockstack/stacks-wallet-web/issues/1147

Function call

Post Conditions
Function args
Function args cont.

STX transfer

Overview
Details

Contract Deploy

No post conditions
Details
Code

@MarvinJanssen if you could give this build a test with the issues you were seeing, that'd be great!

Still failing for me.
https://explorer.stacks.co/txid/34dcd4c4d1e40d20780633cb94df873c9e54cbfbc599df712d5586fe0d3667c9?chain=testnet
https://stacks-node-api.testnet.stacks.co/extended/v1/tx/0x34dcd4c4d1e40d20780633cb94df873c9e54cbfbc599df712d5586fe0d3667c9
Also the post condition looks a bit odd on the confirmation screen.


Add CHANGELOG

It would be great for this project to have a changelog file where we keep track of versions and features/fixes we added.
Here is a template that we might use: https://gist.github.com/juampynr/4c18214a8eb554084e21d6e288a18a2c
Furthermore, it might be interesting to investigate methods to generate and update the changelog automatically.

Hi @samuelpcabral ,
after some investigation, I would like to settle on changelog-ci as a changelog generator.
Here is why:

it is lightweight and easy to use
it runs as github action, so there is no need to add dev-dependencies for this project
since it is using github actions, there is no such thing as forgetting to update the changelog
you are not required to have your git commits follow a certain convention

How it works:

Changelogs can generated based on commits or pull request titles
When choosing to work with pull request titles, the CI will generate a changelog if the PR title matches a certain regex. By default this is ^(?i:release). So PRs like "Release 1.0.0", "release 1.2.2" will generate it, others won't.

Personally, I prefer the generation based on pull request titles, because there are generally less PRs than commits, so the CHANGELOG won't bloat up. Second, PR titles are usually a bit more informative than commit messages.
There are many ways the generation can be configured. As next steps, I can prepare a PR and we can discuss the specifics there.


Apollo Client does not use the pollingInterval option.

I have tried to set Apollo to use a pollingInterval to update, but I cannot seem to get it to function. How should I go about this?

Did you follow this? http://dev.apollodata.com/react/receiving-updates.html#polling

I figured it out, I'm not used to decorators.
Now that I have that figured out, could you possibly help me figure out when I keep getting errors in decorated components? As far as I can tell, it's trying to render before the props have been filled.

Please post to stackoverflow, using the reactql tag if it's something specific to this starter kit.


When click "DELETE" from the left bar, it crash Godot

https://github.com/feagi/brain-visualizer/assets/65916520/561a5206-ae05-40b4-bed9-eaba1cb955e1

fixed


Create function returning bad Base64 values

First, the documentation should be updated to indicate that both Create() and Combine() return TWO parameters: data, error.
Second, the Create() function is returning an array of shards, but as it turns out each of the shards/secrets has the Base64 string twice, thus, attempts to decode the Base64 result in an error. Below a sample output of the first "secret" in the list:

Here is the result of the following statement prior to attempting Base64 decode:
fmt.Printf("\nShard #%d %T\n\t%s\n", i, shard, shard, shard)
Shard #0 string
	CxhWcKn2w3_TzcIbQZr0KQe7IBqLiL_9uZsT-_bWMkM=EkW1Z-EsCm3u061sXnK3_5N60hSP_bGmz7L4pX-eRww=
%!(EXTRA string=CxhWcKn2w3_TzcIbQZr0KQe7IBqLiL_9uZsT-_bWMkM=EkW1Z-EsCm3u061sXnK3_5N60hSP_bGmz7L4pX-eRww=)

As you can observe, each of the returned secrets has the Base64 value followed by what it appears to be the same value (repeated) as a result of a badly formatted SPrintf() statement.

Hi @lordofscripts -- if you're looking for a mature and tested Shamir's library, I'd suggest checking out<EMAIL_ADDRESS>

Hi @lordofscripts -- if you're looking for a mature and tested Shamir's library, I'd suggest checking out<EMAIL_ADDRESS>
Thanks for the lead. I looked into it but unfortunately it drags a lot of other stuff with it. I settled for RSA Shamir Secret Sharing which is bare ones and has the necessary building blocks.

If the disclaimer in the RSA one is an issue, I'd probably take a PR to add a separate go module for Shamir's upstream in OpenBao.


Any way to resolve container ip addresses from host ?

I'm experimenting with LXD. I have created many containers and now I'd like to access them trough certain ports. I adapted some iptables rules found here:
iptables -t nat -A PREROUTING -p tcp -i eth0 -m tcp --dport 80 -j DNAT --to-destination <IP_ADDRESS>:80

Where <IP_ADDRESS> is the dynamically allocated ip for the srv-web1 container.
It works well, but I had to hard-code the dynamic ip (found with ifconfig). I think there's a great risk that the thing break if I restart the host or move containers around.
I could use static ips, but I didn't found any documentation to do that, and I'd prefer to keep DHCP and manage containers by name and not mess with a lot of ips. Is there a way to query the hostnames from the host machine, like dig lxd.srv-web1 or a bash command / environment variable that stores this data ?
Edit: I've found I can do dig @<IP_ADDRESS> mycontainer. If I use that can I be sure that the NAT router/dns will always be found at this address ?

The dig trick is probably about as reliable as it gets without poking the LXD API.
If you want to know for sure what the container's IP address is, using "lxc info container_name" or the REST API would be the way to go.

Using the REST API and a trusted certificate, you can do something like (admittedly not a super simple filter).
stgraber@castiana:~$ curl -k -s --cert .config/lxc/client.crt --key .config/lxc/client.key https://localhost:8443/1.0/containers/abc | jq '.metadata.status.ips | map(select(.interface=="eth0")) | map(select(.protocol=="IPV4")) | .[0].address'
"<IP_ADDRESS>"

Closing since this was a question which I believe has been answered. Feel free to let us know if you think there's a bug somewhere in LXD or some way to make it easier for your use case.

Is this broken?
lxc remote list | grep lxc
| lxc             | https://my-lxd-server.my-domain.com:10443               | lxd           | NO     | NO     |

$ lxc list lxc: | grep dummy
| dummy               | RUNNING | <IP_ADDRESS> (eth0)              |      | PERSISTENT | 0         |

When I attempt to get the IP via API, I get:
$ curl -s -k --cert ~/.config/lxc/client.crt --key ~/.config/lxc/client.key https://my-lxd-server.my-domain.com:10443/1.0/containers/dummy | jq '.metadata.status.ips | map(select(.interface=="eth0")) | map(select(.protocol=="IPV4")) | .[0].address'
jq: error (at <stdin>:1): Cannot index string with string "ips"

Here's evidence that the machine really exists:
curl -s -k --cert ~/.config/lxc/client.crt --key ~/.config/lxc/client.key https://my-lxd-server.my-domain.com:10443/1.0/containers/dummy | jq .metadata.config
{
  "image.architecture": "amd64",
  "image.description": "ubuntu 16.04 LTS amd64 (release) (20170307)",
  "image.label": "release",
  "image.os": "ubuntu",
  "image.release": "xenial",
  "image.serial": "20170307",
  "image.version": "16.04",
  "volatile.base_image": "x",
  "volatile.eth0.hwaddr": "00:16:3e:e4:b3:2a",
  "volatile.idmap.base": "0",
  "volatile.idmap.next": "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
  "volatile.last_state.idmap": "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
  "volatile.last_state.power": "RUNNING"
}

Found it - Rest API:
$ curl -s -k --cert ~/.config/lxc/client.crt --key ~/.config/lxc/client.key https://my-lxd-server.my-domain.com:10443/1.0/containers/dummy/state | jq .{
  "type": "sync",
  "status": "Success",
  "status_code": 200,
  "operation": "",
  "error_code": 0,
  "error": "",
  "metadata": {
    "status": "Running",
    "status_code": 103,
    "disk": {},
    "memory": {
      "usage": 143683584,
      "usage_peak": 769069056,
      "swap_usage": 0,
      "swap_usage_peak": 0
    },
    "network": {
      "eth0": {
        "addresses": [
          {
            "family": "inet",
            "address": "<IP_ADDRESS>",
            "netmask": "24",
            "scope": "global"
          },
      ...


Different code between codelabs and github

Data is not fetched at the first time because in the codelabs code, at searches variable,
it is not calling method .onStart { ... }, but in github actually we need to call it.

Codelabs (page 7)
Github

Codelabs
val searches = actionStateFlow
            .filterIsInstance<UiAction.Search>()
            .distinctUntilChanged()

Github
val searches = actionStateFlow
            .filterIsInstance<UiAction.Search>()
            .distinctUntilChanged()
            .onStart { emit(UiAction.Search(query = initialQuery)) }

also in codelabs (page 9) said We can now remove binding.list.scrollToPosition(0) from updateRepoListFromInput()., but in codelabs and github the code isn't removed

Oh wow, thanks for catching this! I'll fix it as soon as I'm able to.

Fixed.


Is it possible to use the api without MessageEmbed ? I say using the api with only plain replies, and not Embeds.
And can we edit the embed, like edit some words in the description or the title ?

Hello! Due to the simplistic design of the package, this will not be a feature I will be adding.
The only way you can do this is to create a private version of the package for only yourself, and then edit the code that way. 👍


access to $index?

Is there a way to access the current index in the repeated HTML?

Please answer the question.

We are facing the same issue. Please answer.


tortoise: add on malfeasence handler

closes: https://github.com/spacemeshos/go-spacemesh/issues/4538
main change is to handle equivocating atxs in tortoise, so that they can't skew global threshold by making it unreachable.
for this purpose there is OnMalfeasance method that is expected to be called when equivocating ballot or atx is found, hare equivocation doesn't matter for tortoise, so it can be added later at its own timing.
tortoise will store mapping with all malicious identities without pruning them, even considering some unrealistic number of such identities (1_000_000) that will only be about ~30MB, which is clearly not a ddos vector.
field for ballot.Malicious can be dropped in future pr.

bors try

it's enough to report malfeasance to tortoise on the gossip codepath, as every proof, internally or externally, will go through it.

if node is not synced then it won't be called. beside that OnMalfesance has to be called before equivocating OnBallot/OnAtx

if node is not synced then it won't be called.

HandleSyncedMalfeasanceProof will take care of the sync path when a node is not synced.

beside that OnMalfesance has to be called before the call to OnBallot/OnAtx with object that equivocated

but malfeasance proof isn't valid if an identity doesn't exist locally. in syncer, it syncs ATXs first before syncing malfeasance proofs.

HandleSyncedMalfeasanceProof will take care of the sync path when a node is not synced.

this absolutely should not depend on what data is synced

but malfeasance proof isn't valid if an identity doesn't exist locally. in syncer, it syncs ATXs first before syncing malfeasance proofs.

it should not pass 2 atxs in the same epoch from identity that equivocated

bors merge


Typescript error highlighting in for loop

Hi everyone,
I found a wrong highlighting in for loop, please have a look below screenshot.
This error can make all below lines go wrong syntax highlight too.

This is a duplicate of https://github.com/Microsoft/TypeScript-TmLanguage/issues/11


Possibly wrongly implemented Sound panning on 5.1+ surround setup on LWJGL (asking for test)

LWJGL Backend: It seems when played on surround speaker setup, Sound panning sends audio to rear speakers instead of fronts. I was able to correct this behaviour by modifying
FloatBuffer orientation = (FloatBuffer)BufferUtils.createFloatBuffer(6)
                                                          .put(new float[] {0.0f, 0.0f, 1.0f, 0.0f, -1.0f, 0.0f}).flip();

on com.badlogic.gdx.backends.lwjgl.audio.OpenALAudio (I flipped signs).
However, my real question is: is this behaviour driver/soundcard-dependent? Is it just me? Can you recreate the same issue? (audio plays on rear speakers when panning is centred) I was about to make a pull request, but I couldn't test it on the various environment.
I'm including the test code. (It's written in Kotlin) You will need to put your own MONO sample as assets/loopey.wav
package net.torvald.terrarum.audio.surroundpanner

import com.badlogic.gdx.Game
import com.badlogic.gdx.Gdx
import com.badlogic.gdx.audio.Sound
import com.badlogic.gdx.backends.lwjgl.LwjglApplication
import com.badlogic.gdx.backends.lwjgl.LwjglApplicationConfiguration
import com.badlogic.gdx.files.FileHandle
import java.awt.BorderLayout
import javax.swing.*

/**
 * Created by minjaesong on 2018-05-18.
 */
class SurroundPannerTest : JFrame() {

    val mixerPanel = JPanel(BorderLayout()) // LR slider

    val mixerPanSlider = JSlider(JSlider.HORIZONTAL, -32768, 32767, 0)

    init {
        val sliderPanel = JPanel(); sliderPanel.add(mixerPanSlider)
        mixerPanel.add(sliderPanel, BorderLayout.CENTER)

        this.add(mixerPanel, BorderLayout.CENTER)
        this.defaultCloseOperation = EXIT_ON_CLOSE
        this.isVisible = true
        this.setSize(400, 600)
    }

}

class AudioPlayerSlave : Game() {

    lateinit var audioSample: FileHandle
    lateinit var gdxSound: Sound
    var soundID = 0L
    lateinit var surroundPanner: SurroundPannerTest

    override fun create() {
        audioSample = Gdx.files.internal("assets/loopey.wav")
        gdxSound = Gdx.audio.newSound(audioSample)
        surroundPanner = SurroundPannerTest()
        soundID = gdxSound.loop()
    }

    override fun render() {
        gdxSound.setPan(soundID, surroundPanner.mixerPanSlider.value.toFloat() / 32768f, 1f)
    }

    override fun dispose() {
        super.dispose()
        gdxSound.dispose()
    }

}

fun main(args: Array<String>) {
    val appConfig = LwjglApplicationConfiguration()
    appConfig.vSyncEnabled = false
    appConfig.resizable = true
    appConfig.width = 256
    appConfig.height = 256
    appConfig.backgroundFPS = 20
    appConfig.foregroundFPS = 20

    LwjglApplication(AudioPlayerSlave(), appConfig)
}

TL;DR — Mono pans to back speakers as opposed to the front. Is it just specific to certain device or global issue?

I don't have a surround speak setup at hand, but testing it on all the audio devices I have here I couldn't reproduce your issue. I think it seems rather unlikely that only surround speakers would be affected and judging by the lack of comments no-one else seems to have this issue.
I think closing this for now is the best idea, we can always reopen if someone can reproduce the issue. What do the others think?

Yep, we should close it.


[3.x] GDNative stringification of uint64_t

This reflects some additional unimplemented functionality for stringification of uint64_t to the GDNative.

Sorry for the late review. Exposing those two makes sense, but you would also need to add them to modules/gdnative/gdnative_api.json, no?
Note that to avoid breaking compatibility, they need to be added to an extension API and not to the base one. We just added Core API 1.3 in #55826 for Godot 3.5, so you could add those two new methods there together with PoolStringArray::join.

Superseded by #63136.


async/await in nuxt.config.js

I'm starting to do pretty advanced stuff in config.generate.routes and miss having async/await for control flow. Can I run the config through babel somehow?

@jroxendal If you want to have a chance doing some stuff before nuxt build, you may use modules.
Modules are not documented yet, but you may browse modules repo to get idea how they basically work.
modules/test.js
module.exports = function myModule(moduleOptions) {
  // You can access and modifiy options (nuxt.config.js) using this.options
  // You can even access nuxt instance using this.nuxt

  // If you want to do async things here, simply return a promise
  // or use second callback argument of function
}

nuxt.config.js
modules : [
  '~/modules/test.js'
]

Hi @jroxendal
You can use node 8 to use async/await in your nuxt.config.js file.


Proposed new strategy for handling saving and loading

The current approach is straining at the seams because it doesn't fully reconstitute the state. The current HDF5 state file is really only useful for the current version of the kana UI, as it only holds the statistics needed by the UI; this is not generalizable to other applications, nor will it work for future versions of the UI that need more information.
Here, I propose a shift to how we think about saving and loading. The first component of this shift involves reducing the scope of the save/load to just the analysis parameters. This allows users to restore the analysis state by just repeating the analysis itself. Such an approach is slower but is more comprehensive with respect to obtaining a fully functional state. It is easier to reason about as we don't have to worry about mixing up results from different versions (#45), and we avoid problems with back-compatibility when migrating old result formats into more recent versions of the State objects.
The second component of this shift involves saving the results into something that can be reloaded into the "explore-only" version of the application. This will allow users to explore pre-computed results without the strict obligation of reproducing the analysis UI. Importantly. the analysis UI does not need to be tied down by back-compatibility with old results, because those results are only relevant to the explore-only mode; and given that the explore mode doesn't do much, its contracts with its inputs are inherently looser, so back-compatibility is less of an issue there.
In particular, I am thinking of saving the results into an alabaster-based SCE that contains many (but not all) results from various steps. This can be downloaded as a Zip/tarball or saved to an ArtifactDB backend. On reload, we open the "explore-only" mode and just treat it like any other SCE, populating the reddims and clusters and such. Now we don't have to care about changes to colData names, how the % variance per PC is encoded, etc., because the explore mode doesn't care either.

This change has been implemented, though the existing loadAnalysis() and related functionality has been left in so that we can softly deprecate the old paradigm. @jkanche We should probably add banners on kana to indicate deprecation if someone tries to load a kana file, and to redirect them to re-save configuration/results in the new world.

Or we continue to host the current version of kana as a legacy option for old datasets


did the banner image get added?

i just updated the npm package and i saw the enemies data, thank you btw for adding that. Tho did u add the image when u rolled the character? example: https://external-preview.redd.it/GIiM20bfwIimwB0zxndUtzCVsczmECefK5cPqeTsRmM.png?format=pjpg&auto=webp&s=84ec1301828bfe748b9c6aea91056e1d0f922170

please see #45


Remove files_dropped signal from SceneTree

Fixes #57442
In Godot 4 the files_dropped signal was moved to Window, but in SceneTree the signal was still declared, but never emitted.

Updated

Thanks!


Provide better error: specify modified extension

Issue Type: Feature Request
This error is popping up too often:

Extensions have been modified on disk. Please reload the window.

But I have ~100 extensions and want to know which one is modified to find out the cause.
VS Code version: Code - Insiders 1.33.0-insider (5a88160c7dc5db820a564e3872e0de087ff3563d, 2019-03-15T15:54:09.237Z)
OS version: Windows_NT x64 10.0.17763

@usernamehw do you have reproducible steps? This should not happen. It means something is changing the contents of package.json on disk and the cache we build is invalidated because of these changes.

I do not have the steps because there's no indication which extension is doing it.
Can't even reproduce it now.


Implement PSF file format

These files only contain topology information and seem to be widely used in bio-molecules simulations.
See here for a description: https://www.ks.uiuc.edu/Training/Tutorials/namd/namd-tutorial-unix-html/node23.html
The format is a simple text-based format that looks relatively free-form, but I could not find a specification of the format. Maybe looking at what VMD does would help.

It's also used by NAMD quite frequently. Which begs the question, should we also consider GROMACS top files?  These are also topology only.

Yes, every format can be supported =) Would you mind opening an issue for top file, with some kind of reference for the format ?
For topology only format, I think we can just set all positions to 0 in the frame, and only parse the topology.

Sure, I'll do that.  That's what InChI does, so sounds good.

Hi, I wanted to ask if you are planning to implement the psf reader using the psfplugin from VMD, or is anyone currently working on a parser for chemfiles?
I am asking because I need to read weights from psf files in the same C++ code I am developing. I was planning on writing my own code to read the psf myself, but if it is being implemented in chemfiles, then I would rather not "reinvent the wheel". (If no one is working on a parser, then I am happy to share my code with you)

I don't think anyone is working on a PSF parser right now. If you want to try implementing PSF parsing within chemfiles I can help you get started. Looking at the example in the link in the first message, it should not be very hard to implement. By "weights" you mean the atomic mass, right?
If you prefer writing it inside your own code, sharing the result will be very helpful our implementation effort once we get there!

@Luthaf Yes I meant the atomic masses.
There are actually several "flavours" of PSF files, (see here). The molfile plugin contains a psf reader which is supposed to read all types of psf files. Since it is shipped with chemfiles already and it is tested by VMD so using it would be a stable solution. However, I don't know enough C or C++ to be able to write the interface between that and chemfiles.

Right, enabling the molfiles based reader should be relatively easy, but I would prefer to add tests files with it. I would like to move away from the molfiles code for a variety of reason (RAM usage, direct logging to stderr/stdout, …); and this will only be possible if we can validate that the new code works. Do you have a few PSF files, ideally one of each flavour, that you could share to that purpose? Once we have a couple of test files, I can enable the reader relatively easily.

@Luthaf Thank you! I have two example PSF files, one generated by VMD (XPLOR standard format(?)) and other generated by CHARMM-gui (but is in EXTended XPLOR format). They are attached: ligandrm.psf.txt, and 776wat_1Ca.psf.txt. I do not currently have CHARMM formatted parameter files, but the main difference between them seems to be that CHARMM one uses numbers instead of letters to identify atom types. However, CHARMM-gui seems to have adopted the XPLOR format in favour of their original format.

@Luthaf I have wrote a C++ psf reader for use in my own code (for example here). The psf reader reads the atoms, bonds, angles and dihedrals/impropers right now. The full code is here. You mentioned that you wanted to mode away from VMD molfiles based code, feel free to use my code in chemfiles. I have tested it on three psf files (XPLOR, EXT and VMD formats), and it seems to work well so far. There are some other information in the psf file as well which are not read by the code, I would be happy to implement them if required.

Thanks for sharing! It will be useful when we try to write our own PSF parser in chemfiles


Depict null values in array type ECProperty output

itwinjs-core: https://github.com/iTwin/itwinjs-core/pull/6948
Matched ECSqlRowAdaptor and JsonECSqlSelectAdapter to work as ECInstanceECSqlSelectAdapter and display null values as nulls in the JSON outputs.
Closes iTwin/itwinjs-backlog#1162

/azp run imodel-native


Store the skip status in the database instead of the local storage

Right now we are using LocalStorage to save the skip status. A better idea would be to save that information in the backend.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


No mention of how to exit the psql command line interface.

setup.md mentions how to get into the psql command line interface but doesn't mention how to exit it...
\q
\quit

Thanks for the issue, does #18 make it clear enough how to exit psql? ctrl + D is a standard for exiting pretty much any Terminal process (other than vim).

(PR auto closed this issue) Just want to check that this is resolved and no need to re-open 🙂.


Slow navigation between pages | as seen on QR Scan screen

Expected Behavior
After the QR scanning is complete in the QR Code Scan Screen, it take a second or two to transition to the next screen. This gives an appearance that QR Scanning is slow but instead the processes after the code scan take most time.
The transition between screens should be faster and/or the user should be shown a loading icon as soon as the data from code is scanned.

Closing, duplicate of resolved https://github.com/celo-org/celo-monorepo/issues/898


solr looks for configsets in wrong directory

this problem may be isolated to docker deployments; see: https://issues.apache.org/jira/browse/SOLR-7323
when creating a core, solr seems to look to /var/solr/configsets, but the _default configset is actually stored at /opt/solr/server/solr/configsets. this interacts with SOLR_HOME; if the variable is set to point at opt/solr/server/solr the core will instead fail because the directory isn't writable by solr (guesswork).
@kmcelwee and I were using
docker run -d -p 8983:8983 -e "SOLR_HOME=/opt/solr/server/solr" solr:latest

to start a local instance.

this is an issue with solr8 docker image, not parasolr, as indicated by the jira link; work around implemented and working in github actions build


Link Ceramic Stream IDs to Tiles

Closes #26

Verified the preview build with CORS disabled. Good to merge.


Add GDPR Language to Stand Alone Demo Pages

Context:
GDPR will go into effect on May 25th; we need the demos to display the following language in order to make them compliant with the law:
This system is for demonstration purposes only and is not intended to process Personal Data. No Personal Data is to be entered into this system as it may not have the necessary controls in place to meet the requirements of the General Data Protection Regulation (EU) 2016/679.
For more information see: https://github.ibm.com/Watson/developer-experience/issues/4342
How to update:

Add disclaimer as text.

@stevenpkg Can we prioritize this issue? Please let me know if you need any help from our team on any of the demos.

Hi @mamoonraja I maintain about 3+ Watson Assistant and Discovery demo's, I am not sure if any of then use watson-react-components, can I just look in the package.json file, or is it more complete then that? These are the main demos I look after:
https://github.com/watson-developer-cloud/assistant-simple
https://github.com/watson-developer-cloud/car-dashboard
https://github.com/watson-developer-cloud/assistant-with-discovery

@germanattanasio
Hi @mamoonraja can we have a talk on my Tuesday morning, or a time that your free. I need to fully understand what you need me to do.

@stevenpkg We need to add the disclaimer, see https://natural-language-classifier-demo.ng.bluemix.net/ for example.
you don't even need to update package-lock.json.

@mamoonraja I do not have edit permission for the conversation demo github pages @germanattanasio can you give me permission to make the changes

@stevenpkg which demo? I just checked and you have write access to https://github.com/watson-developer-cloud/assistant-simple, https://github.com/watson-developer-cloud/car-dashboard and https://github.com/watson-developer-cloud/assistant-with-discovery

You can also fork the repo and do a PR.

Are we adding the disclaimer to the GitHub description for each demo, or are we adding the disclaimer to the readme? If it is the readme then no problem, but I see no edit button next to GitHub page description to edit.

@stevenpkg Please take a look at https://github.com/watson-developer-cloud/natural-language-classifier-nodejs/pull/56. You can do something like that.

@mamoonraja If you are happy with this I will do the same to car dashboard and assistant with discovery

@desmarchris have created a pull request for all 3 demos, assistant-simple, car dashboard, and assistant-with-discovery

@mamoonraja please assist @stevenpkg reviewing and merging this PR if is ready

All the PR's were merged earlier today.
@stevenpkg Can you deploy the latest version for https://github.com/watson-developer-cloud/assistant-with-discovery because I remember you use to deploy it manually.

@mamoonraja I have created and tested assistant-with-discovery-new and all okay. Will you change URLs and rename etc?

assistant-with-discovery is updated.


Doesn't work with multiline history records

Doesn't work with multiline history records
echo """1
2
3
"""

after looking up in hstr it finds and inserts only echo """1\ which only first line.

I admit that this behavior is intentional as I didn't know how to fit multi-line history records in commands list nicely...
@zdmytriv @max-lobur would you have any suggestion/preference how can I improve HSTR? Like making multi-line command single line and indicating new lines using a character... any other idea?

@dvorka thanks for takin a look. "\n" will be totally ok, usually other lines are just args, I primarily search for a command name not args, so this will work. Kind of
command --arg 1 --arg 2 \n...

How about simply indenting the multilines by 1 or 2 whitespaces? As of now all history entries start on the very left, so indenting could be used to hint at a multiline entry.

You would lose to much useful space

Am I also fine if it displays first line only as long as long as full version is used.

This would be super useful! Currently having a difficult time with ROS commands like:
rostopic pub -r 10 /husky_velocity_controller/cmd_vel geometry_msgs/Twist "linear:
  x: 0.0
  y: 0.0
  z: 0.0
angular:
  x: 0.0
  y: 0.0
  z: 0.0"

Is there any workaround for this?

Is there any workaround for this?

I am also looking for this

Same here!

I am also looking for this

same here


Blank page is shown on “Checkout.sandbox.xanppol” when user tap on “Xanpool” on cash out page

Frequency: 100%
Repro on build version: Android internal build V1.14.0<PHONE_NUMBER>) & Test Flight build V1.14.0(58)
Repro on devices: One plus 6T (10),Samsung Galaxy Note 5(7),iPhone XR (14.2), iPhone 6+(12.4.5) ,
Login credential:<PHONE_NUMBER> / Backup Key
Repro Steps:

Launch the application & login with test account
Tap on Hamburger menu
Tap on Add and withdraw
Tap on cashout
Select Celo dollars in select a digital currency & Bank account in Receive funds
enter valid amount & tap on next button
Tap on Xanpool
Observe

Current Behavior: Blank page is shown.
Expected Behavior:  Related data should show
Impact: User get blocked to cashout in bank account.
Attachment: BlankPage.mp4

Hi @jeanregisser We have verified this issue on latest builds internal  build V1.14.1 & Test Flight build V1.14.0(59) and found issue is fixed so, closing this ticket.


Use new context invocation class to cache environment variables.

resolves #
Problem
Solution
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[x] This PR includes type annotations for new and modified functions

Changes look good overall. I guess we'll need to release + update our dependency on dbt-common with the corresponding changes in https://github.com/dbt-labs/dbt-common/pull/52


release ops-agent module to puppet forge please

agreed, much easier to pull from puppet forge than build another CI/CD pipeline to build from source.

@qingling128 @rambleraptor for visibility

Is this something that you are planning to do? instead of copying the whole module static in puppet code

We actually were planning on using this in Puppet Bolt. So we define the modules there instead of actually downloading them. Probably not a huge deal to include the static module in the codebase but seems silly when many of google's other modules are in forge. I agree if we were using traditional Puppet this would likely be a non issue(except when there are updates which means re-compiling)

Hey @arjunasokan-bc we are using r10k ourselves here, I agree defining their module in puppet will make things a bit more tedious. one issue i noticed is that the module is not called properly, I think puppet 6 has some specific naming convention which this module does not follow. had to change name of module for it to work did this occur to you as well?

We haven't started using this yet. Right now we're just triggering a script that installs the ops module. I just saw that this was available and started looking into switching. Not being available in forge might be a blocker, still evaluating.


imageviewer doesn't work with react-native-swipeout

Hi,
When I use this component together with the other project mentioned in the subject:
<Swipeout right={swipeoutBtns} autoClose={true} >
<ImageViewer

                                    disabled={false} // by default

                                    source={{ uri: rowData.commentImage }}

                                    //source={require('../resources/time.png')}

                                    doubleTapEnabled={true} // by default double tap will zoom image

                                    onMove={(e, gestureState) => null}

                                    //onPress={(opening) => //console.log(opening)}

                                    mainImageStyle={styles.commentImage}

                                    zoomedImageStyle={styles.zoomedImageStyle}

/>

</Swipeout>

It doesn't minimize after user clicks on it and maximizes. It works fine on ios Emulator, but not after it is deployed on the phone.
It's because onMove is not called on the phone, whereas it is on emulator..
Can you please help?
Thanks.

Ignore this, it happens only on one device. Not sure what's wrong with it


templates: indirect more values

Continuation of #963, but with a few more ones that were missed.
This should be all, I checked through all the code in templates/ that might be affected and there's nothing else that might pose an issue as far as I see.
Works as expected on selfhost.

Friendly bump @nitinr-bst :) would be nice to get this in.


log: notes and tests re: concurrent access

Addresses #24.
@ChrisHines anything else you'd put in here?

LGTM.