Fixed shift in onion-bordas

In the Onion-Bordas method, we shift the image by half a pixel to align our 0,1,2,3 grid with what the transform expects (0.5, 1.5, 2.5,...). When we shifted back, we didn't include mode='nearest, which resulted in the x=0 value getting set to zero. In fact, we would rather just set this value to the nearest value.
There is still a 1.5 pixel shift needed in the code (was previously 1 pixel, but 1.5 provides better agreement) that I don't know the origin of. This requires more attention.
But, for now, we should merge this, since it at least fixes the first-pixel issue.

Transform pair profiles 1-6 (excluding profile 4), %difference curves, with MSE (mean-square-error):
old:

@DanHickstein "fix" (as above):

There is an improvement in mse.
The 1.5 pixel shift provides the main r > 0 improvement. There is still a smaller issue near r=0.
old:

New:

The transform pair results suggest the normalization for onion_bordas is correct.
For a real VMI (O2-ANU####.txt) the relative amplitude of the onion_bordas image improves with image size.
1024x1024:

cf 3072x3072:

onion_bordas appears more sensitive to the image grid size (dr).
I think the normalization for onion_bordas is correct (no doubt we checked this in the past). This PR improves the mse for the transform-pair functions. As good as it gets for onion_bordas until there is closer scrutiny of the algorithm code.

@stggh,

the relative amplitude of the onion_bordas image improves with image size.

Is it an effect of the interpolation? Visually, the integral intensity seems to be OK, but the peaks are slightly broader. And shifted by something like 0.5–1 px. Is this shift radially-dependent?

That is a tricky question, because the plot is an angular integration (code gist)

Looks pretty uniform

Looks pretty uniform

Interesting. Can you test it with a uniform comb of gaussians to have a fair comparison at different radii?

Sorry, I had other things to do today.
The uniform comb of gaussians will need to be forward Abel transformed first?

Something like this (Gaussians would be better due to their smoothness, but this is analytical):

shift_grid=True seems to work correctly, but the central pixel is apparently lost in any case.

Oh, so there is a one-pixel shift when shift_grid=False?
There was a line that I deleted that shifted the results by one pixel. Probably that line should be returned.

Good point, I wondered why the onion_bordas shift was not so obvious in Fig. 4(c) of the PyAbelPaper.

Something like this

Neat coding @MikhailRyazanov!

shift_grid=True seems to work correctly

Good enough, but not perfect:

I am not sure how to change dr, altered a few other parameters:
hw = 2  # peak half-width
step = 20  # center-to-center distance between peaks
n = 5  # number of peaks

rmax = int(3*n * step)

   :
subr = np.logical_and(comb.r >= 35, comb.r <= 45)
print(f'area Bordas = {simps(recT[subr], comb.r[subr]):5.3f},'
      f' area actual = {simps(comb.func[subr], comb.r[subr]):5.3f}')
print(f'Bordas CoM = {center_of_mass(recT[subr])[0]:5.3f},'
      f' actual CoM = {center_of_mass(comb.func[subr])[0]:5.3f}')

area Bordas = 1.988, area actual = 2.000
Bordas CoM = 5.087, actual CoM = 5.000

I modified the code to reinstate the 1 pixel shift for both True and False values of shift_grid. I think that the results are now more reasonable:

After f60f22b5e305d2f7024cab2cd3bea98663d94fad seems to work well for the experimental image also (very small shift, < 1 pixel):

But the central pixel (r = 0) is still lost somewhere:

Should we go ahead and merge this then?

Probably yes (but please do not forget to rebase).
The central pixel can be investigated later (maybe we need to open an issue as a reminder).

but please do not forget to rebase

Uh, so, I basically don't understand the motivation for rebasing, nor how to do it easily. But, I know that whenever I submit a PR for another repo, people ask me to do it. What is the deal? It basically just makes the revision history cleaner?

I actually do not know. If it will not rewrite the previous changes (“This branch is 4 commits ahead, 13 commits behind PyAbel:master.”), then it might be fine to merge as is. You can try, and if this does not work, then revert and rebase/merge again.

Ah, okay, I see what you are saying. I can easily just make a commit to merge the latest changes into this branch. This is easier than actually rebasing, which you can use to change numerous commit messages into a single commit message. True rebasing has always been a huge hassle for me.
This branch is now "5 commits ahead" of Pyabel:master.

The image shift to create an odd-image should only be applied to the columns:
    # onion-peeling uses grid rather than pixel values,
    # odd shaped whole images require shift image (0, -1/2)
    if shift_grid:
         IM = shift(IM, (0, -0.5),  mode='nearest')

See onion_bordas.py for the full version. It is necessary to call IM = np.atleast_2d(IM.copy()) before the single axis shift. Similarly, for the return to an even image.

Woah! Yeah, thanks for catching that. We should certainly only be shifting in the horizontal and not the  vertical. I'm kinda surprised that scipy.ndimage.shift even allows a single value to be entered.

Okay, should I go ahead and merge this?

only be shifting in the horizontal

This is not as obvious as it might first appear, because in principle the image centre is moved from centre-of-the-centre-pixel to the pixel-grid. However, empirically, for the O2 velocity-map image, only the column shift yields the correct photoelectron spectrum, consistent with the other PyAbel methods.
Yes, good to merge.

I think the shift(image, -0.5) should remain. The pseudo resolution change improvement for a column-only shift, appears to have been due to shift=False, i.e. the grid shift not being applied. The effect of a grid shift of (-0.5, -0.5) vs (0, -0.5) on the O2 peak resolution is miniscule. I believe that (-0.5, -0.5) is correct.
@DanHickstein please revert the (0, -0.5) shift change. Sorry, about this, to much going on at the moment.

The effect of a grid shift of (-0.5, -0.5) vs (0, -0.5) on the O2 peak resolution is miniscule. I believe that (-0.5, -0.5) is correct.

Why do you expect that shifting in the vertical is correct? Intuitively, there is not reason that the transform should be having any issue with the rows. Shifting rows should cause clear issues with the transform on a pixel level.
Here is a comparison. I've made a synthetic image using our "comb" of several sharp peaks. I've then made it into a 2D image by putting rows of zeros on the top and bottom. (Gist link)
The left side of this figure shows the projection, and the right side is the inverse Abel transform using the new ((0, -0.5)) shifted onion_bordas. Everything looks fine:

If I revert back to the ((-0.5, -0.5)) behavior, clear artifacts are seen, as a result of the shift:

So, I'm fairly sure that the vertical shift should be zero.

Hmmm, so another issue is that a mysterious half-pixel shift is required. Shifting the image back by (0, 0.5) produces fairly good results, but shifting by (0, 1.0) produces even better agreement with the gaussian test (0.81% error rather than 1.45%) and the transform of the "comb" also matches better.
In the absence of really understanding what's going on, I'm tempted to make the heuristic correction that improves agreement with our test datasets.

Why do you expect that shifting in the vertical is correct?

Because of the definition of even vs odd pixel size image centre https://github.com/PyAbel/PyAbel/issues/39#issue-119736937

I think the initial shift, for an odd-size image, should be (-1/2, -1/2) to align the centre to that for an even-size image, as required for onion_bordas. Then a post inverse Abel shift (+1/2, +1/2) to return to an odd-size image. Plus (0, #) to correct any grid misalignment of the onion_bordas implementation.  I am still testing this.

The onion_bordas transform is unaffected by any row shift, so it might as well be zero (0, dy). My own O- test spectrum gist gives a return shift of (0, 0.9) i.e. +0.4 rather than the +0.5 that you obtained, but +0.5 may be close enough.

I found that the 0.9 shift provided better agreement with a sharp peak (the "comb" example) but worse agreement with the gaussian.

So, it's clear that we still don't perfectly understand how the onion_bordas method works. At some point we need to re-write the basic code so that it operates on the 0,1,2,3 grid instead of the 0.5, 1.5, 2.5 grid so that it can stand on a more even footing with the other PyAbel methods (and not need a shift).
Still, this PR makes the method way better than it was before. I suggest that we merge this and tackle a better re-write of onion_bordas later. Sound good? If @stggh or @MikhailRyazanov agree, please hit the green button :)

As I understand, this version was used for the article figures? Then we should definitely merge it, even if it still has some unresolved questions.

Yes, this version is needed to fix the bug at r=0, so it was used for the article figures.

Yes, merge.  Thanks, @DanHickstein

Merging... :)


Check GF(2^m) support

RedHat derivatives, Fedora in particular, ship OpenSSL without EC over GF(2^m). DSTU 4145 uses this kind of EC, so we need to check if OpenSSL supports it.

Implemented in 1.3.3.


V0.8.7

Description
This release fixes a quoting error with barcode reader
0.8.7 (https://github.com/united-manufacturing-hub/united-manufacturing-hub/compare/v0.8.6...v0.8.7) (2022-03-09)
Bug Fixes
* changed names of workflows. (141df6d (https://github.com/united-manufacturing-hub/united-manufacturing-hub/commit/141df6d1a6672cfea66ba78f7bb6eff01dbac075))
* quoted broker port (aef739c (https://github.com/united-manufacturing-hub/united-manufacturing-hub/commit/aef739cfdb9ff9b1df37fe880ce56aed2ee7ba44))

Features
* Builds and pushes docker container based on branch name. (370636f (https://github.com/united-manufacturing-hub/united-manufacturing-hub/commit/370636fe863491e243fb15a2ad924af5f6866858))
* dependabot.yml update (e3cb2cd (https://github.com/united-manufacturing-hub/united-manufacturing-hub/commit/e3cb2cd05d7c2b5384e37deb3116c27c9e379d33))

It also includes a bit of cleanup for dependabot


Add reference to svelte-ackee

Now Svelte also has a nice and easy way to use Ackee :)

Thanks! 🎉


tests: put back alembic test

Fixed in https://github.com/inveniosoftware/invenio-communities/pull/418. Thanks!


variable size at division

FYPO:0007474
variable cell size at division
Defined
A cell population phenotype in which cells in a population do not divide at a uniform size. Cells in one population may be larger or smaller than normal (i.e. compared to wild-type cells under the same conditions).
Jacky suggested
variable cell length at entry into mitosis followed by equal cell division
A cell population phenotype in which cells in a population do not divide at a uniform size. Cells undergo equal cell division but the cell length at which they initiate mitosis within one population may be longer or shorter than normal (i.e. compared to wild-type cells under the same conditions).
because a mixed population of cells could also  be caused by
abnormal cell division, large and small daughter cells (FYPO:0002029)
@mah11 I will let you figure the solution because  one is a cell phenotype and one is a population phenotype.  We may want to
be more explicit about FYPO:0007474 rather than creating a descendant term?

@JackyVH  I see you also requested another descendant of FYPO:0007474
reduced cell size homeostasis
Cellular phenotype where cells that divide at a longer or shorter size than wildtype cells fail to correct this in the following generation. Cells born shorter will fail to extend more and longer cells will fail to extend less compared to wildtype cells
maybe
variable cell size at division with reduced cell size homeostasis
(although isn't  "variable size at division
already indicating a cell size homeostasis" problem?

so
FYPO:0007474
variable cell size at division
is 'less accurate monitoring of cell size"
but
inability to correct cell-size deviations, is  "loss of homeostasis"

@JackyVH  should we have some mutants annotated as "variable cell size at division" from Navarro and Nurse?

Let me check  for tomorrow

Add to FYPO:0007474 text def to specify that all cells in the population divide "equally", i.e. daughter cells are the same size as each other.

cell size homeostasis (for pombe) = a cell that has come into existence at a wild-type size itelf divides at wild-type size

some mutants that would be annotated to FYPO:0007474 do, and others do not, subsequently divide at normal size
It's a bit of a cheat to represent them as subclasses of FYPO:0007474, because it's a little awkward putting them in as population phenotypes, but we'll bumble along with it.

OK, I futzed with the FYPO:0007474 definition, and added these:
variable cell size at division with subsequent division at normal size FYPO:0007659
variable cell size at division with subsequent division at abnormal size FYPO:0007660
Let me know if I've got anything wrong in the definitions, or if the names, defs, synonyms, etc. can be improved at all. (My brain is melting ...)

PMID:33419777


Support send fund for sub-token on some chain

Description

[x] Astar: DOT, LDOT, ACA, aUSD
[x] Bifrost: KSM, KAR, ZLK, PHA, RMRK, MOVR, USDT, aUSD
[x] ...

Resolved this issue in https://github.com/Koniverse/SubWallet-Extension/issues/1449


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
47fe351ebef76193bea7ac8a0c53df1ccb869c12
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[SSH Connection Manager] ...

Extension – SSH Connection Manager
Author:

Please add the option to use the Warp terminal as the default for ssh connection. warp.dev
Describe the feature and the current behavior/state.
Add the option in the extension settings dropdown.

Who will benefit from this feature?
The Warp Terminal user base.
Any Other info.

Duplicate of #3165


Create Dockerfile

Add Dockerfile to make it easier to run certbot with the plugin installed

@dlapiduz not sure if you use Docker, but I'm still using this plugin (thank you!) and I find using Docker to install all dependencies in a container helps.
 Also, I added your message to the Readme.
Please let me know if it is ok. Thanks for the contribution!


Option to generate as single file

This is the start of a nice utility!  I was wondering if it would be possible to have an option to generate all of the individual generated files as one file?  The reason is that on github you can use it to generate a single README.md.

(I'm probably going to write this privately in either case)

Did you ever add this? I was planning on writing something for this too, but if it's already been done, it'd be great to share :)

@alne4294 No, I didn't. It would be great if you could!


Merge failed

On merge, checkout tries to apply different stats on a removed file.

@vtemian: Does this happen every time? Who removed the file, us or them? Do we have any logs regarding this issue?


why is the last commit released?

I had to compile the source for my nginx service now, can you draft a new release?

I had an issue with my dev. environment. I've finally managed to get a working MVS instance on my new laptop, so hopefully there won't be such delays now. 1.18 has been released.

Oh, sorry to hear that.


Document S3 attributes drift detection behavior

Description
A setting change in S3 outside of Terraform will be detected as a drift by driftctl if and only if the current terraform state is not updated:
$ driftctl scan --filter "Type=='aws_s3_bucket'"
Scanning AWS on region: us-east-1
Found drifted resources:
  - ou71yo-demo (aws_s3_bucket):
    ~ Versioning.0.Enabled: false => true
Found 1 resource(s)
 - 100% coverage
 - 1 covered by IaC
 - 0 not covered by IaC
 - 0 deleted on cloud provider
 - 1/1 drifted from IaC

In the state:
      "mode": "managed",
      "type": "aws_s3_bucket",
[...]
            "versioning": [
              {
                "enabled": false,
                "mfa_delete": false
              }
            ],
[...]

After a terraform apply, that triggered no change, the state is updated by the refresh:
$ terraform apply
random_string.prefix: Refreshing state... [id=ou71yo]
[...]
aws_s3_bucket.demo: Refreshing state... [id=ou71yo-demo]
[...]

Apply complete! Resources: 0 added, 0 changed, 0 destroyed.

But now driftctl doesn't detect the drift anymore:
$ driftctl scan --filter "Type=='aws_s3_bucket'"
Scanning AWS on region: us-east-1
Found 1 resource(s)
 - 100% coverage
Congrats! Your infrastructure is fully in sync.

That's because the state has been modified by terraform refresh during apply to include the manual change:
      "mode": "managed",
      "type": "aws_s3_bucket",
[...]
            "versioning": [
              {
                "enabled": true,
                "mfa_delete": false
              }
            ],
[...]

This is by design on Terraform's side.
We should document this (waiting for us to use a two-pass system or select a source of truth that would not be the state as it's instable)

Can be viewed there https://docs.driftctl.com/next/limitations/#integration-in-terraform-workflow

very clear!


fix ground primitive specs failing intermittently

On my system Scene/GroundPrimitive renders in 2D when scene3DOnly is false would fail unless run independently and Scene/GroundPrimitive does not render when show is false would fail when run independently, sometimes with different results. This PR fixes the problem by calling an extra scene.render() before each spec to clear out anything like afterRender events.

@likangning93, thanks for the pull request! Maintainers, we have a signed CLA from @likangning93, so you can review this at any time.
:warning: I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

👍


MarkerList slow redraw when switching projects

MarkerList lags when switching between projects with large region/marker count, especially when it scrolled to the bottom. It should redraw only after rebuilding whole list, not with every single list update as it's now.

Unfortunately it doesn't fix the problem: you can see how rows in the table are being deleted one by one


imagemagick_light fails to build on x86_64-darwin

       > In file included from MagickCore/animate.c:44:
       > In file included from ./MagickCore/animate-private.h:22:
       > ./MagickCore/xwindow-private.h:34:10: fatal error: 'X11/Xos.h' file not found
       > #include <X11/Xos.h>
       >          ^~~~~~~~~~~
       > In file included from MagickCore/annotate.c:85:
       > ./MagickCore/xwindow-private.h:34:10: fatal error: 'X11/Xos.h' file not found
       > #include <X11/Xos.h>
       >          ^~~~~~~~~~~
       > 1 error generated.
       > 1 error generated.
       > make[1]: *** [Makefile:8751: MagickCore/libMagickCore_7_Q16HDRI_la-annotate.lo] Error 1
       > make[1]: *** Waiting for unfinished jobs....
       > make[1]: *** [Makefile:8744: MagickCore/libMagickCore_7_Q16HDRI_la-animate.lo] Error 1
       > make[1]: Leaving directory '/private/tmp/nix-build-imagemagick-7.1.1-29.drv-0/source'
       > make: *** [Makefile:6242: all] Error 2
       For full logs, run 'nix log /nix/store/903nn2h8vvb4yndarwpclsz79m9idjjf-imagemagick-7.1.1-29.drv'.

Most likely a sandboxing issue.

yeah that was it.

yeah that was it.

What did you end up having to do to resolve this? I started getting this all of a sudden...


Twitch API - redirect_uri "Invalid scope requested"

It appears either the twitch api has changed or passport-twitch is not doing what is intended(?).
When I try to authenticate with the twitch api, the callback redirect_uri is url encoded (which makes sense), but this appears to make twitch's api not able to pickup the scopes you request access to.
Example: scope=user_read%2Cchannel_read%2Cchannel_subscriptions%2Cchannel_check_subscription
When I change the %2C to + manually it works. The official documentation for the twitch api uses + as the delimiter instead of , which could also cause problems.
Or maybe I just missed something completely in the config or like?
Edit: Also, forgot to mention, thanks for the work you have put into this! Saved me a lot of time so appreciate it! :+1:

Hi @MrEliasen,
Thank you, will definetly look into this. I did a test very quick on our beta but I wasn't able to reproduce it, I will try with the latest version this weekend and will also try with different browsers. I'll do my best to reproduce this error, but I will need some informations, like your Operating System, Node.js (or io.js) version and twitch-irc version. Also, can you post the configuration of your oauth here (without the client id/secret of course) ?
Thank you, glad you like it ;-)

Hi @Schmoopiie,
Thanks for the quick reply!
I won't rule out that I might be the cause but in any case, I'll supply what details I can:
OS: Linux/CentOS 6.5
Browser: Chrome (Latest at time of writing)
Platform: NodeJS v0.10.35
Config:
oauth: {
    port: 6511,
    clientID: "<client-id>",
    clientSecret: "<secret>",
    scopes: "user_read,channel_read"
}```

If you wish to try it out I've enabled the bot for now: http://bot.sirmre.com:6511/auth/twitch/callback no need to actually authenticate of cause :).

Hope it helps, else if you need more info or if you would like to see the  code, just let me know. The bot will be open sourced in the end anyway :+1: .

Ahhh, I get it.. you have to visit http://bot.sirmre.com:6511/ instead of http://bot.sirmre.com:6511/auth/twitch/callback which is the callback URL. I will make sure to provide this information in the next wiki :smile:

Ahh haha! of cause! For crying out loud! :smile:  haha!
For whatever reason I never realised it had "callback" in the url"..
That is it, I need sleep!
Thank you very much for the help :smile:

Also, if it redirects to <IP_ADDRESS> after authentication, we fixed the redirect url after oauth in the beta version. You have to specify your hostname in the configuration.
Example:
oauth: {
        port: 6511,
        hostname: 'bot.sirmre.com',
        clientID: 'xxxxx',
        clientSecret: 'xxxxx',
        scopes: 'user_read,channel_read'
    }

Hopefully we will be able to release a new stable version soon.. I am waiting for Twitch to release IRCv3 tags before releasing it.

No worries, I figured I would need to code that bit in myself (I did wonder why it was hard coded).
Makes sense, no worries at all I can just make the amends manually in the meantime :+1:

Alright, have fun! :smile:
:+1:


Add a trie_root_hash variant for chain specs genesis

Right now, chain specs can look like this:
"genesis": {
  "runtime": { }
}

Or like this:
"genesis": {
  "raw": { }
}

This PR adds a third variant:
"genesis": {
  "trie_root_hash": "0x0000000000000000000000000000000000000000000000000000000000000000"
}

Substrate isn't capable of loading chain specs with this trie_root_hash variant and will probably not be capable in the near future, but I'm adding this variant anyway because the chain_spec.rs file of Substrate is kind of the reference of the format of the chain specs.
I would like to add support for this variant in smoldot (https://github.com/paritytech/smoldot/issues/1566), and this PR will provide a nice error message for people trying to load "smoldot chain specs" into Substrate.

One open question is whether the name trie_root_hash is explicit enough.
The trie can, in theory, use multiple different formats and hash functions depending on the chain. I think it's fine to leave the format of the trie implicit in this context.


MINOR:Metric clear when ReplicaManager shutdown

The following metrics also need to be closed when replicaManager shutdown
"IsrExpandsPerSec"
"IsrShrinksPerSec"
"FailedIsrUpdatesPerSec"

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@dajac Could you help review this PR? Thanks.


feat:  checking cmd dependency while running setup.sh

This commit is append a routine that checks if required commands are
installed or not.
Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>

After this PR:

RESEND V2.
@moyix , Could you review this PR?

Looks good! Sorry to take a bit to get to this, I have a bit less free time now that the school year has started :)


bug: this.clearCookies is not a function...

It throws this error when loading the app:

Documented in README

Documented in README


next button on numeric keypad

Thanks for this great library, which we have incorporated into our app(s).
Dumb question I guess - but how do we get a "NEXT" button or arrow on the keyboard when we're using a ReactiveTextField ?

actually this seems to do the trick: https://pub.dev/packages/keyboard_actions

Hi @SogoGolf,
Thanks for using Reactive Forms.
For showing the NEXT button in the keyboard, you can set the textInputAction of the text field to TextInputAction.next. In Android it works perfect. In IOS I don't remember right now if textInputAction is working, but many developers are using the package that you mentioned keyboard_actions for adding extra buttons to the keyboard.
Best Regards.

Hi @joanpablo is it plan to have this feature natively in flutter reactive_forms? 😁
I've integrated keyboard_actions on multiline fields, but it add additional boilerplate and add complexity in our forms 👎


feat(showWhen) - added platform and mode selectors

This adds the possibility to specify "platform-" or "mode-" at the beginning of a showWhen test to select a specific platform or mode, as referenced in https://github.com/driftyco/ionic/issues/9011.
What's been added
showWhen="mode-md,mode-ios,mode-wp"

showWhen="platform-android,platform-ios,platform-windows"
// equivalent to showWhen="android,ios,windows"

Example

I'll show on Material Design only

Ionic Version: 2.1.1

Nice! Fixes #6525 ;)

Yup, it does :)
showWhen="platform-android,platform-ios,platform-windows"
// equivalent to showWhen="android,ios,windows"

Hi there! So we're going to close this in favor of https://github.com/driftyco/ionic/pull/11465
Thanks for the PR though 😄


Get Logo

Need a logo to replace my face in the header and for the favicon

Apparently favicons are more complicated than they seem.
All fixed: http://realfavicongenerator.net/favicon_checker?site=markmonteiro.info


Details: Allowing copy paste using TextInput on both devices

Thanks @jwallet, testing this on an ios simulator with the existing code there already seems to be copy/paste functionality. Does setting it to editable fix it on android or is there something else I'm missing?

Setting it to editable seems to allow pasting and briefly typing into the text input before it gets reset, so wonder if it's best to keep it as editable:false on ios assuming the fix is for android only?

Yes my bad I opened this too early. But on android editable blocks the copy paste functionality. Give me some times to work on it...

my bad, ''editable:false'' actually works on android

Just opened #82 which should cover it


Dialog component

🗒 General

Would love if someone could help us create a dialog component, modeled after Material's dialog component. We're leveraging our primary + secondary buttons in this dialog component.

💻 Usage
This component is used for whenever confirmation of some sort is needed, such as:
- consent to sensitive matters like GDPR/access/etc.
- save/submit/cancel after a form is completed
- alert message
- optional action buttons
📐 Specs
Specs can be found in our Figma file, under the 'Form Validation' page.

We've included various sizes based on the width of the viewport.

🔮 Future

We'll be adding an example of inline error messaging to this component too.

Is this component meant to have any differences to the builtin dialog?
This sounds like it should be added only in the form of a storybook entry, where we explain the conventions of how to use the builtin one properly (what to wrap in Typography, what variant to use for buttons, general JSX structure, etc), rather than making an actual wrapper component in core.

ooh good point @freben - you are exactly right! if we could just add it to the storybook with the conventions explained above and in Figma, that would be great!

Yep the need is definitely there to have it explained and laid out clearly to the user in storybook!

working on it!

Heeey! Can I look into this?
@freben sorry but I didn't understand about this point, I thought to create a component into /core/components and make it visible under the Layout section in the storybook. Does it sound good @freben @katz95?
Please, feel free to give me some other examples if needed.

No, I meant that we should leverage raw material-ui as much as is possible instead of inventing wrappers. And that some storybook examples of appropriate usage of raw material-ui might be best.

ooh ok, I got the point @freben! Could you please assign it for me?

Sure!


missing content for actual testing...

no icons for mac, so I made them using svg2pdf, shall I email them to you?
I added
OSX = Sketchup.platform == :platform_osx
module TutorialController

  def self.create_step(index, title, &block)
    cmd = UI::Command.new(title, &block)
    cmd.tooltip = title
    ext = OSX ? 'pdf' : 'svg'
    # https://www.flaticon.com/free-icons/numbers_931
    icon = File.join(__dir__, 'images', "#{index}.#{ext}")

example 6:
missing semantic.min.js and semantic.min.css runtime error...
no material images shown or loaded...
john

Ah! Doh! I forgot about that.
Email works fine, pull request is better ;)

missing semantic.min.js and semantic.min.css runtime error...
Oh - that's strange. It looks like it didn't upload the file. Almost as if it thought it was a sub-repo... Let me look into that one.

Yea, Semantic UI was a sub-repo. So unless you recursively checked it out you'd not get the files. I changed it to just be regular bundled files.

here are the pdf's and .svg's that work with the code above...
images copy.zip
after many years I still haven't looked into making 'pull requests' :-o


Revert "Update DeveloperGuide for EditOrderCommand"

Reverts AY2122S2-CS2103-F09-4/tp#137

Codecov Report

Merging #139 (d5206b0) into master (775ca2b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #139   +/-   ##
=========================================
  Coverage     65.44%   65.44%
  Complexity      537      537
=========================================
  Files           101      101
  Lines          1910     1910
  Branches        215      215
=========================================
  Hits           1250     1250
  Misses          591      591
  Partials         69       69

Continue to review full report at Codecov.

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


question?

What did you do?
ℹ Please replace this with what you did.
What happened instead?
ℹ Actual results. Please replace this with of what happened instead.
What did you expect?
ℹ Expected results. Please replace this with what you expected.
OHMySQL Environment

OHMySQL version:
Xcode version:
Swift version: (if applicable, if not remove)
Platform(s) running OHMySQL:

Demo Project
ℹ Please link to or upload a project we can download that reproduces the issue.

wondering if there is a way to append results of select(mysql) to array in xcode..

Hi,
I don't understand what you mean? Please, check documentation and existing API. It is a possibility to get an array of SELECT.

I close the issue, but if it will be needed I will re-open :)


overshoot handling Update fbopt_alternate.py

Codecov Report

Merging #513 (0fab589) into fbopt (6b594fc) will decrease coverage by 0.02%.
Report is 1 commits behind head on fbopt.
The diff coverage is 75.00%.

:exclamation: Current head 0fab589 differs from pull request most recent head 75e8132. Consider uploading reports for the commit 75e8132 to get more accurate results

@@            Coverage Diff             @@
##            fbopt     #513      +/-   ##
==========================================
- Coverage   90.39%   90.38%   -0.02%
==========================================
  Files         139      139
  Lines        5247     5251       +4
==========================================
+ Hits         4743     4746       +3
- Misses        504      505       +1

Flag
Coverage Δ

unittests
90.38% <75.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

domainlab/algos/trainers/fbopt_alternate.py
88.34% <75.00%> (-0.54%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Allow proxy outbound

same as for consul ... except this time for vault nodes

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

0 out of 2 committers have signed the CLA.

[ ] bvanosten
[ ] jjinno

Have you signed the CLA already but the status is still pending? Recheck it.


Added localization to Inventory Module

Fixes #276.
Changes proposed in this pull request:
Adds i18n to all pages in inventory module
Things to keep working on

The buttons on the inventory/adjust page are pulling text from the model, which will display English no matter what
The inventory reports also generate English content in inventory/reports/controller.js

cc @HospitalRun/core-maintainers

I'm confused as to why it just failed to build. It seemed to error out when install PhantomJS...

@nerdkid93 it looks like app/inventory/edit/template.hbs has not been translated.

@nerdkid93 also, what do you mean by the reports generate english content?  What is it that can't be localized?

@jkleinsc Thanks for pointing out edit's template.hbs. That was one of the files I lost when I my github index was corrupted.

@jkleinsc As for the english content in the reports, there are a bunch of statements like this: this._addTotalsRow(Total for ${parentLocation}: , locationCost, parentCount);. I don't know if you want to try to translate each line or find another way to show translated labels for those rows. translations.js already has around 150 lines for the inventory module.

@nerdkid93 Can you add the translation of the report text?  It looks like there are only a couple that are needed: Total for {substitute here}: and Subtotal for {substitute text here}:, and Total:


Update Readme.md - add description of raw type

Readme missing information about raw type, so I have added raw into list of correct types and added following short description based on specs:

raw type means you can store Ruby Array, Hash, String and numbers.

Coverage remained the same at 95.784% when pulling 3514b2d770f934a574539020fe6a9ec556d8baf6 on andrykonchin:add-raw-type-description into 5796c139515d8c9b012e2aa3efead96dcc4c6dff on Dynamoid:master.

Thank you!


bug(gui-client): Flaky smoke test

Windows 2019 - https://github.com/firezone/firezone/actions/runs/8104640577/job/22151626943#step:11:36
Windows 2022 - https://github.com/firezone/firezone/actions/runs/8159147312/job/22302766244?pr=3950#step:11:24
Ubuntu 22.04 - https://github.com/firezone/firezone/actions/runs/8365676251/job/22904084776

Looks like it did not write the device ID to disk, whereas the other version of Windows did. https://github.com/firezone/firezone/actions/runs/8159147312/job/22302765664?pr=3950#step:11:23
Weird... Nothing comes to mind yet.

Looks like it could have hung up somewhere in here: https://github.com/firezone/firezone/blob/f69e645a20c388d65bd583c5fd9293a777b7a276/rust/gui-client/src-tauri/src/client/gui.rs#L158-L187
Or when setting up the device ID: https://github.com/firezone/firezone/blob/f69e645a20c388d65bd583c5fd9293a777b7a276/rust/gui-client/src-tauri/src/client/gui.rs#L785
I'll add more tracing::debug! in that block. The device ID is pretty simple, I think it's the deep link server.

Looks similar to #3699

I narrowed it down to app.build: https://docs.rs/tauri/1.5.4/src/tauri/app.rs.html#1554-1710
This is a huge chunk of Tauri code that builds the app instance and gets it ready to run. We call that function, and at the end it's supposed to call our callback to start run_controller, but it never does.

I think it's getting stuck on creating the GTK event loop for the window: https://docs.rs/tauri/1.6.1/src/tauri/app.rs.html#1594
https://github.com/firezone/firezone/actions/runs/8366214531/job/22905919159#step:10:614
I'll try moving some initialization after that happens, maybe it's racing something I'm doing in a Tokio task.

#4232 may have fixed it. Let's re-open this if it recurs.


Inserting boolean NULL values in BigQuery table — needs a test added

For Boolean datatype, Big Query supports True,False and NULL values. However I am not able to insert NULL values using the dataFrame.to_gbq() method.
I tried with Boolean DataFrame column containing True, False, None values and with True, False, NaN values. Nothing seems to work.
Is there any method to do this or atleast a work around.
Thanks in advance!

Possibly fixed by https://github.com/googleapis/python-bigquery-pandas/pull/445 which adds support for the "new" boolean dtype, which does support NULL values.

Re-opening, because we need a test case here: https://github.com/googleapis/python-bigquery-pandas/blob/main/tests/system/test_to_gbq.py

Test added:
https://github.com/googleapis/python-bigquery-pandas/blob/78aa01e3f039500c9fabbcdd8d8dcfa3e5c42b73/tests/system/test_to_gbq.py#L141-L144
as part of https://github.com/googleapis/python-bigquery-pandas/blob/78aa01e3f039500c9fabbcdd8d8dcfa3e5c42b73/tests/system/test_to_gbq.py#L141-L144


Migrating content packages from Nuget to bower

I am the author of a Bootstrap DateTime picker plugin. (LESS, CSS). I have (combined) 70,000 downloads with 6,000 of those being in the last month alone.
Many of my users are not aware of the upcoming changes. With Nuget/MVC/Asp.Net moving away from delivering frontend packages like this and expecting people to use bower, it will be a shocking revelation to them when this happens. As such, I'm afraid that I will lose part of my user base--those who may not realize that there's a new/different way to get frontend packages like mine. I think that it is important for Microsoft to make this transition smoother, not only for package managers like myself, but especially for my users.
I propose that an XML node be added to the spec that would allow me as an author to point Nuget to my package on bower. This should also cause Nuget to pull down a bower package instead, for new projects (MVC 6). Perhaps some indication to the user that this happened for them in the background: "Hey you downloaded this fontend package but you should really get it from bower".
This wouldn't need to be a permanent addition; just one that would allow the bandage to fall off rather than being ripped off.
Thanks to @csharpfritz for suggesting that I make this issue.
(Cross posting)

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Initial draft of a project maintainer process

Trying to keep this lightweight for now, but we still need to document it.
#6

I would recommend we use the term "committers" vs "maintainers" as we are referring to the people in the CODEOWNERS file with write access.

This update looks fine to me

@dthaler Please review changes to verify all comments addressed


Handle v prefix in versions in install and update command

Is your feature request related to a problem? Please describe.
install and update commands offer the possibility to update to a specific verison with the -v flag. The version entered must have the v prefix, or otherwise the package will not be found in the package repo. That's not the greatest user experience.
Example:
glasskube install keptn -v v2.0.0-rc.1+1 works, because the version number has the v prefix.
glasskube install keptn -v 2.0.0-rc.1+1 does not work, because the version number is missing the v.
Describe the solution you'd like
The user should be able to enter their version without the v prefix. This should work for the install command and also for the update command.
Describe alternatives you've considered
Additional context
#412 is still in progress.

Hello @christophenne. I would like to work on this issue.

@christophenne this isssue is still unassigned, right? Can I start working on this?

@sumeshmurali sure - If you encounter any questions feel free to contact us on Discord

Hey @sumeshmurali are you still working on this? If not I'm up for this issue.

Hey @sumeshmurali are you still working on this? If not I'm up for this issue.

I opened the PR now.


More friendly error message if an invalid user ID is entered

Ashish suggested displaying an error instead of 404-ing, which is a good idea because it's easy to enter an invalid user ID.
We probably should come up with a more friendly interface for selecting a user anyway, but I'm not sure what it should be  Maybe a dropdown like Metasmoke's site picker?

Fancy dropdowns would be good. I think there's a gem involved, or at least some more includes.


Ignores stuck pods rather than deleting them to avoid stateful set edge cases

1. Issue, if available:
2. Description of changes:
@anguslees pointed out that we must allow the kubelet to delete pods that are terminating to avoid violating stateful set guarantees if a kubelet is partitioned. Instead, we simply ignore pods that are past their grace window and delete the node. This ensures the guarantee is met, since the pod will be deleted by KCM once the node no longer exists.
Testing

Modified my provisioner to have a bad endpoint.
Created pod
Node came online, didn't connect
Pod tolerations ran out after 5 minute, pod evicted (w/ grace period 30 sec default)
Node deleted by liveness controller (hanging)
Node waits for pod to terminate (grace period)
Node deleted by termination controller
Pod cleaned up by KCM (2 minutes later)

3. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[X] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Nice work and nice niche catches!

niche caches!


Revert "Merge pull request #396"

Reverted using
git revert 9771dc1 -m 1

related to #396
This reverts commit 9771dc19e4ba032257a522ad3c18adde2672b2ee, reversing
changes made to c4e5328bfd7b4d636f3692e0e9c4fb05e9c2583d.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (link to Jira or GitHub issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the CHANGELOG
[ ] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

I'll go ahead and merge this in, since it's a result of a single git command with no manual intervention.

Tests are passing
I've verified that bugs identified by @igor-kamil  in https://github.com/SlovakNationalGallery/webumenia.sk/pull/398#pullrequestreview-539153873 are fixed

@martin-havala  you can still find your work in feature/WEBUMENIA-1440 branch


User adjustable TX fee

:tada: This PR is included in version 1.3.8 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


fix typo in Ch04 data manipulation lesson

Misspelled 'tensor' as 'tesnor' in two places.

Thanks! LGTM :)


Final Pre-Seed/Seed Investor Debt UBQ

So what's next steps to watch out for? Do we need to fix the amounts again when the stakes are withdrawn?

Finally!
The next 'final' batch should be done after bonds expiry. Only then the remaining exact payout amounts will be available.
The algorithm will be:

Update network block number in https://github.com/gitcoindev/uad-contracts/blob/staking-mutliplier-fix/tasks/simulateBondingDebt.ts and execute the script again using the following command:

$ npx hardhat simulateBondingDebt
This will fork the blockchain, get the current inflation rate and output missing stake values for the same set of bonds. What was paid / disbursed today, should be subtracted from the amount that will be shown.

The BondingDebtV2 / BondingDebtFinal contract will be deployed in the same way BondingDebt, but with remaining UBQ values for the bond holders.

Originally posted by @gitcoindev in https://github.com/ubiquity/ubiquity-dollar/issues/752#issuecomment-2098994982

@gitcoindev can you let me know when the final stake is expected to expire so I can set this on my calendar?

@gitcoindev can you let me know when the final stake is expected to expire so I can set this on my calendar?

Yes, I will provide the date, actually this will be a block number.
I also thought about emitting a blockchain event from the BondingDebt contract when the final payout occurs, it will then be a trace that it happened.
Just noting this to myself in order not to forget:
event LogSeedUBQDebtPayout(string msg);

emit LogSeedUBQDebtPayout('Seed investor UBQ debt payout for bond holder x ');

I collected the data in one table from the previous comments:
https://github.com/ubiquity/ubiquity-dollar/issues/752#issuecomment-1976675340

Bond ID
Bond holder
Bond expiry block
Estimated expiry date

1
0x89eae71B865A2A39cBa62060aB1b40bbFFaE5b0D
21124941
Mon Nov 04 2024 09:37:03 UTC

2
0x4007CE2083c7F3E18097aeB3A39bb8eC149a341d
22618922
Fri May 30 2025 20:33:51 UTC

3
0x7c76f4DB70b7E2177de10DE3e2f668daDcd11108
22624203
Sat May 31 2025 14:10:03 UTC

5
0xa53A6fE2d8Ad977aD926C485343Ba39f32D3A3F6
22647574
Tue Jun 03 2025 20:03:27 UTC

6
0xCEFD0E73cC48B0b9d4C8683E52B7d7396600AbB2
22676978
Sat Jun 07 2025 22:04:39 UTC

The dates are estimates provided by Etherscan taking into account average block time.
I think it is safe to say that the next final UBQ debt payout for all affected bond holders should happen in the first half of June 2025.

/start

/wallet 0x191c51715c68ae8f2d366b3294a82ac28c00cafde7aa4a2a080bd49ab622b100

/start

/wallet 0xc6dcc5424ae41dec50e9444742f83cd489d5527a

/status

Pre Seed  - seed investor
From:
"0xFBb1b73C4f0BDa4f67dcA266ce6Ef42f520fBB98 "  Bittrex,
To:  ENS Name. ubq.eth  also Ubiquity DAO: Deployer.
"0xefc0e701a824943b469a694ac564aa1eff7ab7dd"
Value:   0.498 ETH.
$1,239.19 usd value
Transaction Fee:   0.00042 ETH.
$1.05. Usd value
Gas Price:   20 Gwei
0.00000002 ETH
Ether Price:
$434.85 / ETH. Value OF ETH AT THE TIME
Gas Limit. &     Usage by Txn:
121,000.    |      21,000 (17.36%)
Other Attributes:
Nonce: 2831681
Position In Block: 26
INPUT  Data:   0x

{
"hash": "0xcd86711de4889951b410dfc6bd834a57a43c97091dc780a036e12dc3adf8d820",
"blockHash": "0xb62876f81bf0ace3e54ea747860e707d6280e7f9c3ad16fa12f5e7cd8c5e6a52",
"blockNumber": "12596436",
"to": "0x4e38d89362f7e5db0096ce44ebd021c3962aa9a0",
"from": "0xefc0e701a824943b469a694ac564aa1eff7ab7dd",
"value": "0",
"nonce": "135",
"gasPrice": "35000000000",
"gasLimit": "79244",
"gasUsed": "79244",
"data": "40c10f19000000000000000000000000efc0e701a824943b469a694ac564aa1eff7ab7dd00000000000000000000000000000000000000000000003635c9adc5dea00000",
"transactionIndex": "117",
"success": true,
"state": "CONFIRMED",
"timestamp": "1623189472",
"internalTransactions": []
}

/done

/help

/allow solved

/allow

/query

Pre Seed - seed investor From: "0xFBb1b73C4f0BDa4f67dcA266ce6Ef42f520fBB98 " Bittrex, To: ENS Name. ubq.eth also Ubiquity DAO: Deployer. "0xefc0e701a824943b469a694ac564aa1eff7ab7dd"
Value: 0.498 ETH. $1,239.19 usd value Transaction Fee: 0.00042 ETH. $1.05. Usd value Gas Price: 20 Gwei 0.00000002 ETH Ether Price: $434.85 / ETH. Value OF ETH AT THE TIME Gas Limit. & Usage by Txn: 121,000. | 21,000 (17.36%) Other Attributes: Nonce: 2831681 Position In Block: 26 INPUT Data: 0x

#831  Pre Seed - seed investor From: "0xFBb1b73C4f0BDa4f67dcA266ce6Ef42f520fBB98 " Bittrex, To: ENS Name. ubq.eth also Ubiquity DAO: Deployer. "0xefc0e701a824943b469a694ac564aa1eff7ab7dd"
Value: 0.498 ETH. $1,239.19 usd value Transaction Fee: 0.00042 ETH. $1.05. Usd value Gas Price: 20 Gwei 0.00000002 ETH Ether Price: $434.85 / ETH. Value OF ETH AT THE TIME Gas Limit. & Usage by Txn: 121,000. | 21,000 (17.36%) Other Attributes: Nonce: 2831681 Position In Block: 26 INPUT Data: 0x
"hash": "0xcd86711de4889951b410dfc6bd834a57a43c97091dc780a036e12dc3adf8d820",
"blockHash": "0xb62876f81bf0ace3e54ea747860e707d6280e7f9c3ad16fa12f5e7cd8c5e6a52",
"blockNumber":
"12596436",
"to": "0x4e38d89362f7e5db0096ce44ebd021c3962aa9a0",  "from": "0xefc0e701a824943b469a694ac564aa1eff7ab7dd",   "value": "0",
"nonce": "135",
"gasPrice": "35000000000", "gasLimit": "79244",
"gasUsed": "79244",
"data":
"40c10f19
000000000000000000000000
efc0e701a824943b469a694ac
564aa1eff7ab7dd00000000000
0000000000000000000000000
00000000003635c9adc5dea
00000",  "transactionIndex": "117",
"success": true,
"state": "CONFIRMED",
"timestamp": "1623189472",
"internalTransactions":
[
]}
/close
@Apetree100122

/help

/start


logIn redirect to localhost

Hello guys. i have deployed site on netlify https://nuxt-ts.netlify.app/login. when i`m trying logIn with GitHub, it redirect me to localhost with access and refresh token. How to solve this problem?

Now it just refresh the page and not giving authenticated user information
`
const supabase = useSupabaseAuthClient();
const user = useSupabaseUser();
const location = useRuntimeConfig().public.siteUrl;
console.log(`${location}${query.redirectTo}`);
onMounted(() => {
	watchEffect(async () => {
		if (user.value) {
			await navigateTo(query.redirectTo as string, {
				replace: true,
			});
		}
	});
});
const login = async (provider: 'github') => {
	const redirectTo: string = `${location}${query.redirectTo}`;
	const { error } = await supabase.auth.signInWithOAuth({
		provider,
		options: { redirectTo },
	});
	if (error) {
		console.error(error);
	}
};`

supabase
Site URL: http://localhost:3001/
Redirect URLs
https://nuxt-ts.netlify.app/
https://nuxt-ts.netlify.app/**

I'm facing same problem, did you came with some solution ?

I resolved. For property "redirectTo" you need to write URL something like
"http://localhost:3000/${location}${query.redirectTo} "

Go to the supabase dashboard, and check out Authentication > URL configuration, the default site URL is localhost:3000, and you need to change it to your domain.


About page sans brigade captain info

This tackles #16

I'm thinking let's maybe wait until we have an FAQ to merge this? / Maybe the FAQ could be part of the About page? I can write the FAQ.


How do I change the theme on the fly?

Hello, how can I change the theme on the fly? When I change the dark mode on the site, I would like to choose a black theme, and vice versa.

Theming is all just what CSS you have loaded, so you'd have to write JS to load/switch out different CSS on the fly if that's the desire.
You could look at the Highlight.js demo for just one way to do this:
https://highlightjs.org/static/demo/

Thanks, solved


allow django 3.2 installations (crispy forms 2.1 was blocking)

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

Comparison is base (085b940) 48.02% compared to head (481435e) 48.02%.
Report is 1 commits behind head on prep-release.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           prep-release     #211   +/-   ##
=============================================
  Coverage         48.02%   48.02%
=============================================
  Files               140      140
  Lines              5181     5181
=============================================
  Hits               2488     2488
  Misses             2693     2693

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


chore(Dockerfile): move to ubuntu-slim

Summary of Changes
This patch moves the base image in the Dockerfile to [ubuntu-slim](https://github.com/kubernetes/contrib/tree/master/images/ubuntu-slim
Issue(s) that this PR Closes
Fixes https://github.com/deis/builder/issues/327
Associated End To End Test PR(s)
All existing git push and related end-to-end tests will test this change
Pull Request Hygiene TODOs
Please make sure the below checklist is complete.


serve fails with https requests

with version 0.4.0
proxybroker --log DEBUG serve --host  <IP_ADDRESS> --port 9443 --types SOCKS4 --countries KR
 curl https://ipinfo.io/json --proxy https://<IP_ADDRESS>:9443
curl: (35) gnutls_handshake() failed: The TLS connection was non-properly terminated.

but it works fine with http
[megamind@ssdnodes-89316]:(33.8kb)~$ curl http://ipinfo.io/json --proxy http://<IP_ADDRESS>:9443
{
  "ip": "<IP_ADDRESS>",
  "city": "Bucheon-si",
  "region": "Gyeonggi-do",
  "country": "KR",
  "loc": "37.4989,126.7831",
  "org": "AS3786 LG DACOM Corporation",
  "postal": "14550",
  "timezone": "Asia/Seoul",
  "readme": "https://ipinfo.io/missingauth"

with version 0.3.2
proxybroker --log DEBUG serve --host localhost --port 8855  --types SOCKS4 --countries KR
curl https://ipinfo.io/ip --proxy https://<IP_ADDRESS>:8855
<IP_ADDRESS>

@0-MegaMind-0 can you try with other proxy?
https://github.com/bluet/proxybroker2#remove-specific-proxy-from-queue


Add PV slicing to paraview branch

Resolves VisIVOLab/VisIVOVisualAnalytics#3. If the ParaviewSliceFilter is loaded on the server, PV slice can be created and will be shown in a new window. Behaviour copied from legacy branch as far as possible.

@Pietersielie please address the following point:

Remove debug information when drawing the line.
It's unclear what the user should do after drawing the line; perhaps a popup could appear, asking the user if they want to generate the plot (which takes some time) or cancel.
In the plot window, modify the "save" menu to follow the standard notation "File > Save."
When closing the plot window, the arrow drawn on the slice should be removed.

Drawing the line shows the instructions in the status bar below the images, as it is done in the legacy application. But I will add a popup asking for confirmation.

Also, I assume you refer to the debug information shown on the server side?  The plot window now reflects the File->Save as... structure. And the arrow now correctly disappears when the user cancels the PV slice or closes the plot window. I've also disabled the debug info on the PVSliceFilter plugin.

Remove debug information when drawing the line.
i am talking about the list of frame rate shown in the right panel

Ah, okay, misunderstood what you referred to. I'll get right on that.

I've fixed the multiple frame rate reports through the expedient method of just switching off the view annotation (see here) while the arrow is being drawn. The first commit on this changed the interactor to call pqRenderView::render() instead of the more generic vtkRenderer::render() call, but I think this is a better option.

PV slice generation cannot be asynchronous at the moment, since proxies fail in "interesting" ways. Need to add user warning that disappears once generation is complete. Need to also fix the FitsReader paraview plugin to read PV slices.


task_name=None issue when loading local dataset.

I was evaluating a model on a local dataset, which have worked properly until recently, but I don't know if I changed or updated something in my enviroment, that now I get the following error:
024-08-28:15:30:50,370 INFO     [evaluator.py:469] Running loglikelihood requests
Running loglikelihood requests: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 500/500 [00:01<00:00, 448.05it/s]
Traceback (most recent call last):
  File "/home/gurbizu/LLM/venv-lm-eval/bin/lm_eval", line 33, in <module>
    sys.exit(load_entry_point('lm-eval', 'console_scripts', 'lm_eval')())
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/__main__.py", line 389, in cli_evaluate
    results = evaluator.simple_evaluate(
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/utils.py", line 397, in _wrapper
    return fn(*args, **kwargs)
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/evaluator.py", line 305, in simple_evaluate
    results = evaluate(
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/utils.py", line 397, in _wrapper
    return fn(*args, **kwargs)
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/evaluator.py", line 595, in evaluate
    results_agg, group_agg = prepare_print_tasks(task_dict, results)
  File "/home/gurbizu/LLM/lm-evaluation-harness/lm_eval/evaluator_utils.py", line 276, in prepare_print_tasks
    task_agg[name]["alias"] = tab_string + alias
TypeError: can only concatenate str (not "NoneType") to str

it seems that alias is None, which comes from task_name being None:
{'winogrande_en': ConfigurableTask(task_name=None,output_type=multiple_choice,num_fewshot=None,num_samples=250)}
I have updated the enviroment and start from a clean harness instalation, but I have no idea what's failing.
The code works properly with the online version of winogrande.

Hi! Would you be able to share the config + how the local dataset is saved with us so we can better debug this?

Sure.
This is the yaml config file:
task: winogrande_en
dataset_path: json
dataset_name: null
dataset_kwargs:
  data_files: /home/gurbizu/LLM/winogrande/winogrande_en_250.json
training_split: null
validation_split: null
test_split: train
output_type: multiple_choice
doc_to_text: !function preprocess_winogrande.doc_to_text
doc_to_target: !function preprocess_winogrande.doc_to_target
doc_to_choice: !function preprocess_winogrande.doc_to_choice
metric_list:
  - metric: acc
    aggregation: mean
    higher_is_better: true
metadata:
  version: 1.0

and the json file looks like this:
{"sentence": "The woman kept the bikini but returned the top, because the _ was the right size.", "option1": "bikini", "option2": "top", "answer": "1"}
{"sentence": "The flour spills out when we use an ancient family recipe and pour three teacups of flour into our bowl. The _ is too small.", "option1": "teacup", "option2": "bowl", "answer": "2"}
{"sentence": "The wages at the factory were lower than the wages at the plant since it was more safe to work at the _ .", "option1": "plant", "option2": "factory", "answer": "2"}
{"sentence": "Betty had more burns on their hands than Amy because _ worked as a welder.", "option1": "Betty", "option2": "Amy", "answer": "1"}
{"sentence": "A background check was performed on Natalie by Lindsey due to _ being a nanny.", "option1": "Natalie", "option2": "Lindsey", "answer": "1"}
{"sentence": "Mary felt a great deal of guilt for stealing various items from Rachel so _ forgave her.", "option1": "Mary", "option2": "Rachel", "answer": "2"}
{"sentence": "The toddler kicked the hard furniture and soft toy, but only the _ caused pain.", "option1": "furniture", "option2": "toy", "answer": "1"}

the local dataset is working fine for my coworkers, and it was working fine for me a few weeks too, but I updated the enviroment, and it stopped working. not sure why. I'm using Python 3.8.10 btw.

Hi! Can't seem to reproduce the error (on python 3.12). I copied the example json samples you provided and it seems to run fine. Maybe a duplicate yaml somewhere in lm_eval/tasks folder? You could also try running with --verbosity DEBUG to check if any errors show up.
task: winogrande_en
dataset_path: json
dataset_name: null
dataset_kwargs:
  data_files: /test.jsonl
training_split: null
validation_split: null
test_split: train
output_type: multiple_choice
doc_to_text: "{{ sentence }}"
doc_to_target: "{{answer|int - 1 }}"
doc_to_choice: |
  {% set options = [option1, option2] %}
  {{ options }}
metric_list:
  - metric: acc
    aggregation: mean
    higher_is_better: true
metadata:
  version: 1.0

Thanks for your taking the time.
I'll keep digging into that direction of folder structure and path names.

I found the reason I had that issue. I was calling to the dataset from another coworker home directory where the datasets were. I was passing --tasks /home/ander/LLM/lm-evaluation-harness/lm_eval/tasks/mmlu_eu but from my home directory and my own cloned lm-harness. I was testing with task test123 but I placed that outside lm-evaluation-harness/lm_eval/tasks/ directory, and I kept getting the same error. But when I moved the task inside that folder it was solved.
I know I did not follow the guidelines properly, and I did not use --include_path but the weird thing is that I did not get any error like "task not found", and the dataset config was read properly:
2024-09-03:12:48:04,029 INFO     [__main__.py:383] Selected Tasks: [{'task': 'test444', 'dataset_path': 'orai-nlp/orai_multi_mmlu_eu', 'training_split': None, 'validation_split': None, 'test_split': 'test', 'output_type': 'multiple_choice', 'doc_to_text': '{{question.strip()}}\nA. {{choices[0]}}\nB. {{choices[1]}}\nC. {{choices[2]}}\nD. {{choices[3]}}\nErantzuna:', 'doc_to_choice': ['A', 'B', 'C', 'D'], 'doc_to_target': 'answer', 'metric_list': [{'metric': 'acc', 'aggregation': 'mean', 'higher_is_better': True}], 'metadata': {'version': 0.0}}] 2
and it was running requests:
2024-09-03:12:48:10,256 INFO     [evaluator.py:471] Running loglikelihood requests Running loglikelihood requests (or at least knows how many examples were in the : 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1080/1080 [00:08<00:00, 132.92it/s]
but then somewhere in between or while calculating/printing results it breaks.
It would be great to make the tool work properly once it was able to reads the config file, or make it that it raise an error earlier while reading the config file and print a more human-readable error.
But since the error I caused is solved, I'm closing this issue. 😊


fix: course cross 3 time ranges

Test file: UIC MIS.html.txt

The test file attached is invalid; please prepare another one.

Sorry for uploading the wrong file. This is the correct one
My Timetable UIC MIS.html.txt


Remove version: StrawberryMusicPlayer.Strawberry version 0.8.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/strawberrymusicplayer/strawberry/releases/download/0.8.5/StrawberrySetup-0.8.5-Qt6-x64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


A lot of stuff

unban/kick reason, audit logs, autoreconnect default true, lastmessageid for channel

Thanks for this. Your effort is highly appreciated!

I would not recommend using this branch since it hasn't been updating and I'm not planning on updating it. Neither does the main dev of this library.


Unhandled Promise Rejection on first run

I tried Tinkerun.0.2.1.exe from the release page.

Unhandled Promise Rejection
TypeError: Cannot read property 'setMenu' of undefined
    at App.<anonymous> (C:/Users/ENZOIN~1/AppData/Local/Temp/1stNgUsKDUMMZJJ9hU5AWocDTMC/resources/app.asar/main/index.js:53589:12)

I'm running Windows 10 Enterprise:
Edition	Windows 10 Enterprise
Version	20H2
OS build	19042.985
Experience	Windows Feature Experience Pack 120.2212.2020.0

Same here

I don’t have a PC, 🤣 so I often don’t test on windows.
it fixed in v0.2.2


add licenses

I haven't worked with Astro before, so I hope I've done this right. This is to fix an issue I just raised about a broken link.

Feel free to change anything.

This is cool.

Hiya, the project is "dual" MIT/Apache 2.0 licensed
https://github.com/crablang/crab/blob/master/LICENSE-MIT
https://github.com/crablang/crab/blob/master/LICENSE-APACHE
There are also other accessory license notices we need to retain and link to as notice - best to link here as well to retain notice requirement:
https://github.com/crablang/crab/tree/master/LICENSES

Good pick up. I just grabbed it from the README. We should get a craft going and we can update it.  My main concern is that something comes up at the URL.


Remove high level of debug logging which is not required

High levels of debug messages are not required for ceph-deploy tests, also its causing remoto to hang during get of clent.admin key
Signed-off-by: Vasu Kulkarni<EMAIL_ADDRESS>

just to add more info, without this we are going to hit timeout's in remoto as seen here http://qa-proxy.ceph.com/teuthology/vasu-2017-01-10_03:37:51-ceph-deploy-kraken---basic-vps/703474/teuthology.log  and also the other part being the logging is just way to much to meaningfully debug ceph-deploy outputs which is what we are more interested in the ceph-deploy test.


Accept unspecified src address if dst is broadcast

To be able to accept DNS offer/request incoming packets the ip stack
neeeds to accept unspecified src addr if the destination is broadcast
address.
Signed-off-by: Olof Winge<EMAIL_ADDRESS>

Related issue: https://github.com/zephyrproject-rtos/zephyr/issues/45862

So is it about DNS or DHCP? In the bug report your mention DHCP, and from the context I assume that's the case, but here DNS is mentioned.
Can you also elaborate a bit on the usecase? Zephyr only supports DHCP client at the moment, and in such case the soruce IP address shouldn't be empty - the OFFER/ACK sent by the server should containt the server IP address as a source. The scenario you describe (empty source IP sent to broadcast) is valid for packets sent by the client (DISCOVER/REQUEST), but since Zephyr does not support DHCP server functionality it's a bit unclear to me how does that affect you? Do you run some custom DHCP server implementation on top of Zephyr stack?

So is it about DNS or DHCP? In the bug report your mention DHCP, and from the context I assume that's the case, but here DNS is mentioned.
Can you also elaborate a bit on the usecase? Zephyr only supports DHCP client at the moment, and in such case the soruce IP address shouldn't be empty - the OFFER/ACK sent by the server should containt the server IP address as a source. The scenario you describe (empty source IP sent to broadcast) is valid for packets sent by the client (DISCOVER/REQUEST), but since Zephyr does not support DHCP server functionality it's a bit unclear to me how does that affect you? Do you run some custom DHCP server implementation on top of Zephyr stack?

This is about DHCP. I've built a DHCP server on top of Zephyr. I've also built a DNS server which is why I was messing them up in the commit message. I will adjust according to your comment.s

Just please squash the commits and were good as for me.


Don't ignore exit codes when using setuptools entry points

Setuptools does not require console_scripts entry points to call sys.exit() (or similar) to exit non-zero:

The functions you specify are called with no arguments, and their return value is passed to sys.exit(), so you can return an errorlevel or message to print to stderr

This patch updates pex to follow the same behavior.
Fixes #137.

thanks for the PR @rouge8 !


Removing Visual Studio Project files and switch to Visual Studio code

Hey guys,
I'm contemplating removing the Visual Studio Project files (*.sln, *.psproj) as they are only there to facilitate us all using Visual Studio as the editor when working on xSharePoint, and adding some files that will enable support for Visual Studio Code instead (making use of the new PowerShell support for VS code)- with the good part of that being that you can use VS code, or you can not - those files won't get in the way or need updating as we add more files to the project (right now you need to keep saving the project file in Visual Studio and commiting that, then dealing with merges and conflicts of those files).
So the pros - less dealing with merges of the project files that we'll remove, and really letting people use whatever they want to edit things
Cons - harder to use Visual Studio because there isn't a folder view in it like there is in Visual Studio code
Anyone have any major objections to that change?

B

Hey. I quite like the idea, however, many people are not yet using Visual Studio Code, wouldn't that become a road blocker ? I just don't want to lose the traction and have even fewer people contributing to this project.

Well in theory you don't need to use VS code - you could use whatever you wanted. The VS Code inclusion makes a folder called .vscode which we put a file launch.json in there, this tells VS code how to run the project (which at this point I think would just call the unit tests). So if you aren't using VS code then this doesn't hurt anyone. But outside of that you could just use PowerShell ISE or anything else at all really, because we won't be using those project files anymore.

I kinda like project files, but yes, using visual studio actually causes me more problems than solutions. I always end up using ISE to finish and test files, and have lost changes because of a not to "do you want to reload your files"?
Go for it. if anyone wants they can have their own Visual studio project files

Actually that's a good point - I might add the project files to the ignore file for the project to make sure they don't sneak their way back in so people can still use them themselves if they like while not committing them.

The changes for this are in #173 - I'll leave this open for a bit in case anyone else has feedback before I force it in to dev

I'm undecided on this and (I think) for good reasons

I mainly use ISE
I'm starting to use VS for other projects
I'm undecided on how useful VSCode is in PS world
Does removal (or non-removal) make much difference in getting started with this?

Regards the last point the OfficeDevPnP PowerShell repo still has them included (I only use this as a reference as I'm about to start some work on that repo)
So my personal suggestion would be to keep them until v1.0 stage and possibly remove then

So i dont think removing it makes a problem - most scripters are used to using the ISE anyway. I've had more feedback about the fact that people have to install Visual Studio and the PowerShell extensions for it and that doing so is a pain in the butt, so if anything taking it out removes that ramp up requirement here.
I've also just had a bit of a play around with the ISE and I may even up using that myself as well. I put a file explorer extension in and it's actually working really great, so that may be the new home for this sort of stuff for me instead of VS code. Either way - no Visual Studio though.

@BrianFarnhill - Ok now that makes things clearer to me and it looks like @KarolKaczmarek has merged #173 so this issue could now be closed


BZ 1286912 - Process Designer does not manage Intermediate Compensate Event

Conflicts:
jbpm-designer-backend/src/test/java/org/jbpm/designer/test/bpmn2/Bpmn2UnmarshallingTestCase.java

@jlindop 6.3.x PR for this BZ

Tested and looks good
+1

6.3.x: a1180cd979a25ef44f1fadf0605200a9fb053adc


Archboard feedback for CLU 1.0 GA SDK

[x] Just show differences in asynchronous samples.
[x] Don't use continue to skip and instead nest the foreach in an if.
[x] See if the service is removing the "status" and "jobId" from the final output. (#29347)
[x] Show a Console.WriteLine between getting an LRO and waiting on it.
[x] Put sample parameters on separate parameters for WaitUntil.Completed calls.
[x] Use WaitUntil.Completed and just use separate LROs for tests.
[x] Update autorest for the feature branch (GA blocker).
[x] ConversationAuthoringClient
[x] Open an issue with comments here: https://apiview.dev/Assemblies/Review/20a58d7611e845939ebfd25707b16b88?revisionId=b218fb7529c54356a11d99e7e1daa4f5&doc=False#06f51472b6894935a0d0c153094ad53e (#29332)
[x] Mention Language Studio in README with URL
[x] Open issue on LRO stuff and include a breakdown of what LROs we have and which should return payloads (inference and authoring). (#29332)
[x] Remaining feedback in https://apiview.dev/Assemblies/Review/20a58d7611e845939ebfd25707b16b88

Service:

[x] How big do we expect export sizes? Might need another pattern.
[x] Talk to service about updating their readme.md with CLU transforms, and about the 200/202 pattern for LROs.

APIView:

[x] Need horizontal slider (Azure/azure-sdk-tools#3452)

See if the service is removing the "status" and "jobId" from the final output.

This is odd. For _Sample5_ImportProject.cs, the ImportProject and Train LROs are working as expected based on the session file, but the DeployProject has an extra GET and I can't tell where it's coming from. Specifically, the initial LRO request indeed returns a 202 with an Operation-Location to a status monitor:
      "StatusCode": 202,
      "ResponseHeaders": {
        "Access-Control-Expose-Headers": "*",
        "apim-request-id": "3d75422d-f5fe-4bcd-8cdc-c1b240e63a78",
        "Content-Length": "0",
        "Date": "Wed, 15 Jun 2022 01:02:17 GMT",
        "operation-location": "https://languagesdkresource.cognitiveservices.azure.com/language/authoring/analyze-conversations/projects/net-conv-367666463/deployments/production/jobs/ff2585da-967b-41dc-8f05-0a854689e260_637908480000000000?api-version=2022-05-01",
        "request-id": "3d75422d-f5fe-4bcd-8cdc-c1b240e63a78",
        "Strict-Transport-Security": "max-age=31536000; includeSubDomains; preload",
        "X-Content-Type-Options": "nosniff",
        "x-envoy-upstream-service-time": "101"
      },
      "ResponseBody": null

The next several requests are what you'd expect - status is running - until the final status update:
    {
      "RequestUri": "https://languagesdkresource.cognitiveservices.azure.com/language/authoring/analyze-conversations/projects/net-conv-367666463/deployments/production/jobs/ff2585da-967b-41dc-8f05-0a854689e260_637908480000000000?api-version=2022-05-01",
      "RequestMethod": "GET",
      "RequestHeaders": {
        "Ocp-Apim-Subscription-Key": "Sanitized",
        "User-Agent": "azsdk-net-AI.Language.Conversations/1.0.0-alpha.20220614.1 (.NET 6.0.5; Microsoft Windows 10.0.22000)",
        "x-ms-client-request-id": "ccc3e9c28c3c019344e0e2ad20829ff8",
        "x-ms-return-client-request-id": "true"
      },
      "RequestBody": null,
      "StatusCode": 200,
      "ResponseHeaders": {
        "Access-Control-Expose-Headers": "*",
        "apim-request-id": "eb1b0550-ca5f-4a97-a177-169f5fdaaad4",
        "Content-Length": "218",
        "Content-Type": "application/json; charset=utf-8",
        "Date": "Wed, 15 Jun 2022 01:02:22 GMT",
        "request-id": "eb1b0550-ca5f-4a97-a177-169f5fdaaad4",
        "Strict-Transport-Security": "max-age=31536000; includeSubDomains; preload",
        "X-Content-Type-Options": "nosniff",
        "x-envoy-upstream-service-time": "24"
      },
      "ResponseBody": {
        "jobId": "ff2585da-967b-41dc-8f05-0a854689e260_637908480000000000",
        "createdDateTime": "2022-06-15T01:02:17Z",
        "lastUpdatedDateTime": "2022-06-15T01:02:22Z",
        "expirationDateTime": "2022-06-22T01:02:17Z",
        "status": "succeeded"
      }
    },

But then immediately afterward, there's this extra GET which is what the Operation<BinaryData>.Value ultimately contains:
    {
      "RequestUri": "https://languagesdkresource.cognitiveservices.azure.com/language/authoring/analyze-conversations/projects/net-conv-367666463/deployments/production?api-version=2022-05-01",
      "RequestMethod": "GET",
      "RequestHeaders": {
        "Ocp-Apim-Subscription-Key": "Sanitized",
        "User-Agent": "azsdk-net-AI.Language.Conversations/1.0.0-alpha.20220614.1 (.NET 6.0.5; Microsoft Windows 10.0.22000)",
        "x-ms-client-request-id": "8c11e8cce5e682c8638a22cdb3087be4",
        "x-ms-return-client-request-id": "true"
      },
      "RequestBody": null,
      "StatusCode": 200,
      "ResponseHeaders": {
        "Access-Control-Expose-Headers": "*",
        "apim-request-id": "35de4d51-fcec-4df0-82ca-c06206d6f36f",
        "Content-Length": "281",
        "Content-Type": "application/json; charset=utf-8",
        "Date": "Wed, 15 Jun 2022 01:02:23 GMT",
        "request-id": "35de4d51-fcec-4df0-82ca-c06206d6f36f",
        "Strict-Transport-Security": "max-age=31536000; includeSubDomains; preload",
        "X-Content-Type-Options": "nosniff",
        "x-envoy-upstream-service-time": "22"
      },
      "ResponseBody": {
        "deploymentName": "production",
        "modelId": "Sample5-20220615T010216-aeac9f0032944941b024163bc3ebff89",
        "lastTrainedDateTime": "2022-06-15T01:02:16.6869195Z",
        "lastDeployedDateTime": "2022-06-15T01:02:22Z",
        "deploymentExpirationDate": "2023-10-28",
        "modelTrainingConfigVersion": "2022-05-01"
      }
    }

I can't tell from where this comes? Why is the LRO querying the actual resource? The generated LRO implementation uses OperationFinalStateVia.Location just like the other aforementioned LROs:
using HttpMessage message = CreateDeployProjectRequest(projectName, deploymentName, content, context);
return LowLevelOperationHelpers.ProcessMessage(_pipeline, message, ClientDiagnostics, "ConversationAnalysisProjectsClient.DeployProject", OperationFinalStateVia.Location, context, waitUntil);

@annelo-msft @AlexanderSher @chunyu3 ?
/cc @kristapratico @mshaban-msft @tg-msft @johanste

Had issues with transforming away explicit pageable parameters. Opened Azure/autorest#4565 and Azure/autorest#4566 to track and ended up with a single transform:
directive:
- from: swagger-document
  where: $.paths.*.*
  transform: |
    var paramRefs = [
        "common.json#/parameters/TopParameter",
        "common.json#/parameters/SkipParameter",
        "common.json#/parameters/MaxPageSizeParameter"
    ];
    if (/ConversationalAnalysisAuthoring_((List(Projects|Deployments|TrainedModels|TrainingJobs|TrainingConfigVersions))|Get(ModelEvaluationResults|SupportedLanguages|SupportedPrebuiltEntities))/.test($.operationId)) {
        $.parameters = $.parameters.filter(param => !paramRefs.includes(param["$ref"]));
    }

Not pretty, but it works. Also had to use $.paths.*.* instead of $.paths.. as shown in the Azure/autorest repo in several places because I got an error without the asterisks:
info    |    Loading AutoRest extension '@autorest/modelerfour' (4.23.5->4.23.5)
error   | directive/error | Error occured when running directive: Error: Parse error on line 1:
$.paths..
---------^
Expecting 'STAR', 'IDENTIFIER', 'SCRIPT_EXPRESSION', 'INTEGER', 'END', '[', got '1'


Pull in article command from v3.5.3 tag

Coverage decreased (-2.2%) to 95.798% when pulling 1548d60a89a515f63359b1d2e17bf699dd57885e on dinomite:features/new-article-command into 09b38a138c3b3946a69107fc57db9965a9ed0d80 on middleman:master.

Woo!! Thank you for adding this!!


Add README to examples with images

Great idea to add images to the examples! Could you just resolve the new conflict and I'm going to merge it?

All done @bkolobara


Why need EntityFramework.SqlServer.dll?

The project is greate and after clone with .net 4.5 and vs2019 I can run all the tests and the console sample successfully.
But I don't understand why EntityFramework.SqlServer.dll is always in the output directory. The assemly is not in the 'references' list, and I removed the sqlserver related sections in app.config. After build, the dll is there again!
The I removed the dll in the output folder, and run the exe directly, no error and the database created successfully.
So why need it?

This is due to a reference of the EntityFramework NuGet package.


Allow custom "outputPath" directory for each locale

🚀 Feature request
Command (mark with an x)

[x] build

Description
New version of Angular CLI allows us to use the new --localize option to generate different locales. This is great as it significantly reduces the build time for apps with many languages. However, as far as I can tell, the name of the output directory for each language is set to the name of the locale.
So if I have this locale defined:
        "locales": {           "foo-bar": {             "translation": "src/i18n/messages.en-US.xlf"           }         }
the output directory for this locale would be /foo-bar.
Describe the solution you'd like
I would like to be able to specify outputPath per locale, ideally by just defining "outputPath" in each locale which would then be appended to existing build options outputPath.
Describe alternatives you've considered
Alternative is to just change name of the locale to the one I want. But this then creates a problem where LOCALE_ID is also changed. And I want my LOCALE_ID to be, for example en-US and my output directory of build to be /en.

I've run into the same issue and wonder if it wouldn't make more sense to simply follow the baseHref property if present?
My settings in angular.json:
      "i18n": {
        "sourceLocale": {
          "code": "nl-BE",
          "baseHref": "nl"
        },
        "locales": {
          "fr-BE": {
            "baseHref": "fr",
            "translation": "src/i18n/messages.fr-BE.xlf"
          },
          "de-BE": {
            "baseHref": "de",
            "translation": "src/i18n/messages.de-BE.xlf"
          },
          "en-GB": {
            "baseHref": "en",
            "translation": "src/i18n/messages.en-GB.xlf"
          }
        }
      },

I've resolved into adding a postbuild script to rename the output directories,
but this seems hacky at best.
const fs = require('fs');

const currPath = './dist/public/';
const newPath = './dist/public/';
const directories = fs.readdirSync(currPath, {withFileTypes: true})
  .filter(v => v.isDirectory())
  .map(dir => dir.name);

directories.forEach(dir => {
  try {
    fs.renameSync(currPath + dir, newPath + dir.substr(0, 2));
    console.log('\x1b[32m%s\x1b[0m', 'Successfully renamed the output directory ' + dir + ' to ' + dir.substr(0, 2));
  }
  catch (err) {
    console.log('\x1b[31m%s\x1b[0m', err);
  }
});

So a year have past and still you cannot specify your output directory per locale? I've ran into same problem, I don't need my directory to be en-US but us.

So a year have past and still you cannot specify your output directory per locale? I've ran into same problem, I don't need my directory to be en-US but us.

Can someone look into this? Solution provided by @delrueba does not works in every environment. Kubernetes mounted drives etc.
Im guessing this does not require a lot of effort but it costing developers hours and hours

My problem: in my application the app locale is driven by browser cookie, because I don't want to expose the locale in app business URL.
I need either this issue or https://github.com/angular/angular-cli/issues/17416 to be solved.
But both are still open. So sad!

Our application uses a similar approach as @tkt028 mentioned: we only set a cookie and let the server decide on the index template to serve. This means that per language, we need to generate a separate bundle.
i've setup 2 configurations using localize at first until i noticed that the options "deployUrl" and "outputPath" are not available for the localized bundles. i tried again with a different approach:
// prod
"deployUrl": "/static/en/",
"outputPath": "build/en",
"i18nFile": "src/locale/messages.en.xlf",
"i18nLocale": "en",

// prod-de
"deployUrl": "/static/de/",
"outputPath": "build/de",
"i18nFile": "src/locale/messages.de.xlf",
"i18nLocale": "de",

but when i run these configurations using ng build --configuration=prod,prod-de, all js files end up in the same folder (=> build/de) and i only get the "de"  index.html.

I hope the new feature request process is also for angular-cli https://blog.angular.io/new-feature-request-process-a9f69d106fc8.
am I right @mgechev ?

We'll initially enable it for the framework repository and if it is successful consider it for CLI and components.

Still not fixed? I want my default language setting in the root /dist folder and all the other locales in their seperate fodlers e.g. /dist/fr/... It builds wrongly, i can copy the files manually but when i use the ag-github-cli module i cant do it that way...

This is also needed for firebase deploys.  To benefit from the full functionality of firebase, the directories need to be specific as in en_ALL, en_us, en_au etc.  But this directory structure doesn't match the locale id required by Angular.

+1

Is there any update on this issue?

Still not fixed? I want my default language setting in the root /dist folder and all the other locales in their seperate fodlers e.g. /dist/fr/... It builds wrongly, i can copy the files manually but when i use the ag-github-cli module i cant do it that way...

I acheived this same requirement using separate builds. One build for default language, and another for all locales.
package.json:
"build": "npm run build.en && npm run build.localize",
"build.en": "ng build --configuration production",
"build.localize": "ng build --configuration production-localize",

For the locale build configuration, preserve output files.
angular.json:
"production-localize": {
          "localize": [
            "es"
          ],
          "deleteOutputPath": false,
          ...
}

Any update on this feature? We need this

So what is the recommended way to do this at the moment? I figured since Firebase is a Google product, the Angular team would have at least been consulted on how to make this seemless for the two....
What should the standard be for localized folders? en-US or en_us?
Should the language be in the URL? (I think no, it looks messy and it's a technically a different app).

Had a quick chat with @clydin about this and our initial reaction is that we would like to keep the directory output of @angular-devkit/build-angular as stable and consistent as possible, a new outputPath option per-locale would complicate that. It would also be difficult for the Angular CLI to evolve its directory structure over time if we support options to place content anywhere.
Separately, we've been exploring making Angular builders more powerful and composable. We think this might be a better solution if we can can make them easy and accessible enough, @angular-devkit/build-angular could be left alone, and instead users could write a custom builder (or leverage an existing one built for this use case) to move files around to the structure they really want. This could be done to rename all the locales from replace - characters with _ (en-US -> en_US) or move the source locale to the root of dist/ with everything else under their own subdirectories. With a custom builder you could create basically any structure you wanted without being limited to the semantics of outputPath.
One other thing to keep in mind is that servers don't have to serve your Angular application according to the exact structure of dist/. It is quite common to have a server parse a user's cookie or some query parameter to choose a locale and then serve the Angular app under that locale, without exposing the locale as a URL path (https://foo.test/en/).
I'll leave this issue open until we have a concrete recommended solution to point to. You can do this with custom builders now, but composing an @angular-devkit/build-angular target is quite tricky and I understand why it is not an ideal solution right now. Hopefully improvements to this system will give more flexibility with output structure in the future.

If the user specifies an outputPath on that particular local, use that instead of the localecode. Is that really such a big change?

@worthy7 I'm late to this party, but here's the solution I have for using i18n with Angular and Firebase Hosting. Firebase Hosting requires the directories to be named in lowercase but Angular outputs to mixed case directories for the locales. Tweak the script for other types of locales (like en_ALL for example.) My example shows a default locale of en-CA and optional fr-CA.
In angular.json
      "i18n": {
        "sourceLocale": "en-CA",
        "locales": {
          "fr-CA": {
            "baseHref": "/fr-ca/"
            ...
          }
        }
      }
...
        "build": {
          "configurations": {
          ...
            "fr-ca": {
              "localize": [
                "fr-CA"
              ],
              "deleteOutputPath": false
            }
        }

In package.json
  "scripts": {
    "build": "npm run build.en-ca && npm run build.fr-ca && scripts/rename-locales.sh",
    "build.en-ca": "ng build --configuration production",
    "build.fr-ca": "ng build --configuration production,fr-ca",
    "predeploy": "npm run build",
    "deploy": "firebase deploy --only hosting"
  },

In scripts/rename-locales.sh (make sure the script is executable)
#!/bin/bash

DIRECTORY="./dist/browser" # dist is outputPath in angular.json

if [ -d "$DIRECTORY" ]; then
  for dir in "$DIRECTORY"/*/; do
    dirname=$(basename "$dir")
    lowercase=$(echo "$dirname" | tr '[:upper:]' '[:lower:]')
    if [ "$dirname" != "$lowercase" ]; then
      mv "$DIRECTORY/$dirname" "$DIRECTORY/$lowercase"
    fi
  done
else
  echo "Directory $DIRECTORY does not exist."
fi

In firebase.json
"hosting": {
    "public": "app/dist/browser",
    "ignore": [
      "firebase.json",
      "**/.*",
      "**/node_modules/**"
    ],
    "rewrites": [
      {
        "source": "/fr-ca{,/**}",
        "destination": "/fr-ca/index.html"
      },
      {
        "source": "**",
        "destination": "/index.html"
      }
    ],
    "i18n": {
      "root": "/"
    }
  },

@tzappia that is precisely what everyone is doing for that approach. the problem with that is that it requires separate compiles. if you have like 10 languages, you will have rather long compile times for larger applications. this ticket is about the "new" way of compiling multiple languages in a single compile step, which is way faster and scales better for many languages. however, that new approach currently doesn't allow to specify multiple output directories. hence the ticket.


(maint) fact keys as strings

puppet device --apply does not support fact keys as symbols.
This was broken as part of a rubocop cleanup
This was originally fixed by #3

👍


feat(receive-pack) read-only

This is a feature request to support receive-pack and basically a git server in read-only mode. The implementation could be used to "mirror" an existing repository(and perhaps apply a different ACL)  or create and serve in-memory repositories on the fly.

We have work in progress for this.

The low level work is done (https://github.com/src-d/go-git/pull/190), including a clone of the "git-receive-pack" command, see: https://github.com/src-d/go-git/blob/master/cli/go-git/receive_pack.go
Further work will include full git:// and ssh:// servers.


Support for Sinopia Stage and Production

Currently we only been using Sinopia Development environment. We need to support Sinopia Stage first and then Production environments.

One specific thing we know we need to figure out: how to specify the application environment (e.g. dev, stage, or prod) to the code.  At present, this is mostly hardcoded, though at least one part of the code expects an operating system environment variable.  However, in #83, @jermnelson says his "preference would be the app environment would be set per DAG or in some other fashion so that we only have to run one Airflow instance for all three of Sinopia's environments, development, stage, and production."  I believe that'd preclude the use of an OS env var, as an OS env var would be shared across all three application environments, if they're all being run by one airflow instance.

Per discussion with @jermnelson we may just need to allow the dev-airflow to monitor stage/prod SQS queues through terraform or through specific aws airflow connections

post-standup discussion:  one difficulty with using a single airflow instance is that there's not a clear path for how we go about testing DAG changes in stage, before deploying to prod.
e.g. if one DAG on one airflow instance is monitoring stage and prod, and we want to test a new task in just stage, before deploying to prod, how might we do that?
we'll try to enumerate the benefits and downsides of a single airflow instance with a single DAG monitoring multiple queues, vs a single airflow instance with multiple DAGs (one per each environment's queue?), vs multiple airflow instance (likely the setup most analagous to our other environments -- for example, our existing home grown workflow management system has distinct stage and prod instances, with distinct worker deployments and backing redis instances).

@jermnelson and I met to discuss this today, and neither of us could come up with a good answer to the pre-prod testing problem when running a single instance of Airflow to handle both stage and prod environments.  My worry was that we'd have to do something like feature flags for which tasks are enabled in which environments, for which version of a function implements an operator in a given env, etc.  Or, that we'd have to dynamically instantiate different DAG definitions for each environment.  In all those scenarios, even if the change is very small and configuration-like, we'd still be deploying a commit that hadn't been tested in stage when going to prod, which feels like an extra opportunity to introduce human error.  And versioning functionality within the code that might be better versioned in Git.
The downside of multiple Airflow instances is of course that it's more complexity to maintain, and more resourcing to pay for.  (this random person on the internet claims "Running Airflow on AWS with the default configurations (also considering cluster is not scaling up) should cost from 5 to 7 US dollars per day" -- YMMV)
Since we hope the complexity is manageable via Helm (we're deploying to Kubernetes on AWS), and since we hope the cost of supporting two environments is not terrible, we're going to go ahead and try separate stage and prod Airflow instances.  This should give the most consistency with our other environments, and the most similarity between deploying a DAG for testing on stage and then deploying that same logic on prod.
Additionally, after meeting, I did a bit of searching to see if there was an idiomatic way to do this among Airflow users, and finally found something of an official (if slightly implied) recommendation in this Airflow best practices doc:

If possible, keep a staging environment to test the complete DAG run before deploying in the production. Make sure your DAG is parameterized to change the variables, e.g., the output path of S3 operation or the database used to read the configuration. Do not hard code values inside the DAG and then change them manually according to the environment.
You can use environment variables to parameterize the DAG.
import os
dest = os.environ.get("MY_DAG_DEST_PATH", "s3://default-target/path/")

Which seems like support for our somewhat tentative conclusion.
One other worry:  neither @jermnelson nor I really had any concrete idea how to go about provisioning a second Airflow instance.  We were hoping @aaron-collier could give some advice on that 🙂 (as well as any other feedback on this discussion so far)

the docs do point out that DAGs can be configured dynamically, but they also advise minimizing DAG complexity.


Make prerender work for non html

Describe the problem
I wish i was able to prerender while keeping headers like content-type 'application/xml'.
I have a lot of urls and i have to have a complete sitemap refering to all of them.
Adding them manually would be insanity, i solve this by having sitemapIndex.xml/+server.ts as a sitemap index, refering to 30 different sitemaps categorizing all our links.
I can prerender this dynamic route by adding this to svelte.config.ts
       prerender: {             entries: [                 '*',                 '/sitemap.xml/slug', // And so on             ],
Voila! my entire one million website has a complete sitemap, it's prerendered so only have to NUKE the database at build time 👯 👯
But then the issue arrives, prerendering doesn't keep the headers and content type 😢
This also happens for any other .xml file you try to prerender.
This is a normal .xml file before prerender:

After prerender:

I took a look into my .svelte-kit/output/prerendered/pages/sitemap.xml and there's no header info in it, therefore the browser has no idea it's an xml.
Describe the proposed solution
Allow you to spesify what headers to have on prerendered stuff 🤔
I don't know, i'm not that good at this, mabye it already exists? If it's possible it's not documented as far as i could see in the docs.
Alternatives considered
No response
Importance
i cannot use SvelteKit without it
Additional Information
No response

Essentially a duplicate of #9408.


Display a countdown timer on a website

Possible resource: https://www.w3schools.com/howto/howto_js_countdown.asp


Error on iOS Chrome

Facebook does not support popup authentication on ios chrome.
On chrome, the library should use 'redirection' method for authentication.
This is a soultion for the problem:
http://www.youtube.com/watch?v=i0zdxVaK59g&feature
https://github.com/PeggApp/pegg/commit/a95404e8e42eefa1e52adc4632ae0e9fb4753a55
Thanks

+1

+1

+1