observe alerts: add RE_Observe_No_Successful_Updates alert

https://trello.com/c/vAmktonY
This is an alert we want to fire when we haven't seen any successful targets updates in a region for a certain period. updates are triggered through the /update-targets endpoint, so luckily we should just be able to look at the 200 responses to these requests to see successful updates.
Wrap in an outer sum() aggregation to leave open the possibility of multiple app instances.
 I've initially chosen an increase() window of 30m, a for: period of 12h, and ticket severity.
The manual link doesn't exist (yet).

I've added another commit on this which should (I think) annotate a region="ireland" label on to ireland's metrics. However I'm worried about this breaking anyones existing setups. It's the kind of thing that looks like it might not have been done for a reason, because it's so simple to copy the config from the paas-london-targets job. Having no label does fit in with everything else in paas ireland being basically unmarked as such almost everywhere else. Thoughts?

It's the kind of thing that looks like it might not have been done for a reason

I think Observe pre-dates PaaS having multiple regions (or at least pre-dates anyone in GDS using any region other than ireland), so I think it's most likely that the tags were only added to london when the deployment was added there.
It think it's worth adding the label.

Oh dear smoke test failed on prom-3 going to staging.

Re-ran & it passed :+1:


Feature: different times

Introduces multiple times (3 new scenes) whose assets, objects and order can be configured in separate config files.
Optimizes all images (using pngquant)

@kennethlarsen Merging it in order to not produce conflicts while working on the new mice things.

Just catching up - it all looks really great!


Register custom types

Description
I assume you have already thought about this, but what is the plan for custom element types and other custom GraphQL types?

The plan is here: https://github.com/craftcms/craft-gatsby#integrating-3rd-party-content :)


fix(dart_frog): Response.json() overwrites content-type header

Status
READY
Description
Closes #593
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Closing in favor of https://github.com/VeryGoodOpenSource/dart_frog/pull/596


Error integration with CLI

Hi, I'm using angular cli 12 generated project. But I'm seeing an error:
ERROR in ./node_modules/ngx-ui-tour-core/fesm2020/ngx-ui-tour-core.mjs 7:0-69
Module not found: Error: Can't resolve 'rxjs/operators' in 'C:\Users\huangs8\workspace\cde\node_modules\ngx-ui-tour-core\fesm2020'
Did you mean 'index.js'?
BREAKING CHANGE: The request 'rxjs/operators' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '*.mjs' file, or a '*.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
 @ ./node_modules/ngx-ui-tour-md-menu/fesm2020/ngx-ui-tour-md-menu.mjs 3:0-39 4:0-135 18:30-41 109:12-39 135:49-62 147:45-58 152:349-371 159:238-260 166:40-67 198:16-35 199:16-27 209:8-18 215:8-18 218:12-22 223:11-21 235:24-34 239:24-34
 @ ./modules/_app/app.module.ts 41:0-69 66:16-41 87:16-27
 @ ./modules/_app/main.ts 3:0-44 15:41-53

Any idea where this error coming from?
Thanks!

Do you have rxjs library installed? If yes, can you try to update to v7?

I can't reproduce the issue. Seems to work fine with Angular v12 and Rxjs v6 for me. See the attached project for the demo.
Delete your node_modules and package-lock.json, update your npm to the latest version, reinstall and then try again. If the issue persists, please create a demo project reproducing the issue.
ng-tour-test-angular-v12-rxjs-v6.zip

It's solved. Thanks!


GEODE-4916: Fixes compilation errors when including in empty VC++ project.

Convert to CMake GenerateExportHeader.
Remove use of sprintf which caused error on Windows.

Found some issues.


Add nav of timetable week using keyboard keys Ctrl+Shift+Left/Right

Pull Request Test Coverage Report for Build 418

52 of 52 (100.0%) changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 90.955%

Totals

Change from base Build 417:
0.1%

Covered Lines:
3238

Relevant Lines:
3560

💛  - Coveralls


Correct json path in interactive.py

Path to json in interactive.py (line 81) is hard-coded with the utah mesowest path. Needs to be changed in order to have roadmap.ipynb working.

shoot forgot about this -- not sure the best way to fix this other than having a note that the itslive catalog geojson file needs to be downloaded and then include an argument to pass its path when creating the widget?
I just tried to pass a url to the where the file is in the gh repo and it didn't work, I'll try to see if i can fix it

In my instance I just wrote the path as: "nbs/catalog_v02.json" in 'interactive.py', it seemed to work.

i just updated it to be a url so that users don't need to download the json file, will push in a minute

addressed in https://github.com/vdevauxchupin/ITS_LIVE_TOOL/pull/33


hu translations of strings beginning with a number all in lowercase

Can the hu translations of strings beginning with a number please be all in lowercase, i.e. {0} fiók, {0} mappa and {0} üzenet?

The translations seem to be from: https://github.com/devmount/third-stats/blob/master/public/_locales/hu/messages.json#L78-L82
However, Fiók and Mappa should start with a capital letter as shown in the image below.

Thank you

Thank you for reporting this. I totally agree, I'll improve the lang files to use separate keys for the locations you mentioned.

You might also like to use Localization and Plurals.
There is no difference in Hungarian when there is a number before the object; however in English it is:

1 account, 2 accounts
1 folder, 2 folders
1 message, 2 messages

Other languages have different plural forms so you add-on can also benefit from using the 19 different plural forms.
Thank you

Can you please program it just like Localization and Plurals? The Hungarian rules are just like that. Sorry, creating a pull request to do that is out of our scope as we only translating in the browser, there is no copy on our computer of your add-on.
Plural rule #1 (2 forms) is the form used by English and Hungarian, so when English is done, it can be used for Hungarian.
Thank you once again.

I see, so there is no need for an additional pluralizationRules object for Hungarian, since it works with the English one. Once I corrected the keys, you can provide the correct plural form. I let you know, when it's done.

It's implemented now. I already corrected the Hungarian translation to lowercase.


Multiple of the same sticky header appeared

No repro, I just switched back to this editor that I opened in a different session and this is what I saw:

Version: 1.70.0-insider
Commit: a7cb6851ea2fe94740fea17c9acb6f4c612e8026
Date: 2022-08-03T16:53:11.332Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0
Sandboxed: Yes

It kept happening, even after closing and reopening the file

Thanks for the bug, I'll look into it

First thing today 🤯

Hi Daniel, thank you for the communicating the bug. I had meetings today so only got down to working on it now. It seems like my changes from yesterday may have caused this. I have created a new pull request referencing this issue, which should solve this unexpected behavior.


Add new DCOS Diagnostics API support

Fixes: https://jira.mesosphere.com/browse/DCOS_OSS-5435
High-level description
Add support for new diagnostics endpoints
Corresponding DC/OS tickets (obligatory)
These JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-5435 Implement new API in dcos-test-utils

Related tickets (optional)
Other tickets related to this change:

DCOS- Foo the Bar so it stops Bazzing.

Related dcos-launch and dcos PRs
Is this change going to be propagated up into another repo? Test the change by bumping the dcos-test-utils SHA to point to these changes to test it. Link the corresponding PRs here:
Checklist for all PRs

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Include a test in dcos-integration-tests in https://github.com/dcos/dcos or explain why this is not applicable:
[ ] Include a test in https://github.com/dcos/dcos-launch or explain why this is not applicable:

Integration tests were run and

[ ] Integration Test - Enterprise (link to job: )
[ ] Integration Test - Open (link to job: )

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner.

Looks like tests aren't passing yet

@drewvanstone Fixed, I have some problems with PEP8 (my code was not pythonic). I think we should switch to https://github.com/psf/black (gofmt for Python) in all our python projects.

For some reason this PR breaks dcos buidl any idea why?

ImportError: Error importing plugin "pytest-dcos": No module named 'pytest-dcos'

https://github.com/dcos/dcos/pull/6032
https://teamcity.mesosphere.io/viewLog.html?buildId=2161438&buildTypeId=DcOs_Open_Test_IntegrationTest_AwsOnpremWStaticBackendGroup1TerraformSpotBlock

Here is passing test https://teamcity.mesosphere.io/viewLog.html?buildId=2162042&buildTypeId=DcOs_Open_Test_IntegrationTest_AwsOnpremWStaticBackendGroup1TerraformSpotBlock

@cprovencher @orsenthil Can you take a look at this?

@orsenthil Can you merge this?


LivPrb can vary across Markov states

Updated the Markov model to allow the survival probability LivPrb to
vary between discrete Markov states.  This is useful for having states
represent discrete health levels (so that someone in poor health is more
likely to die than someone in good health).
Moreover, it's a back-door route to do the Krussell-Smith model of
infinitely lived dynastic agents who occasionally change their discount
factor due to a generational change (DiscFac --> 1.0, LivPrb substitutes
for DiscFac).  Must turn off mortality in simulation for this to work.

I think eventually this needs a bequest motive added in, but otherwise looks good


Added insert rows functionality

Summary
Added convenience method to insert any rows in between existing rows.
Test plan
Added 3 new test cases for insert row.

Resolved conflict with latest master

We just did a group review - once those two points are addressed, this is good to go 👍

Added option to inherit style for addRow/s and insertRow/s without affecting existing implementations. Added test cases for same.
Please review.

@alubbe Please review the latest changes. Thanks

@alubbe Please review my recent changes. Thank you

The new changes look great - just left a comment regarding the copying of styles, let's see what @Siemienik has to say


Merge feature/authentication

This merges increases functionality. It does some simple login with msal-react.

Click on the "Login" button in the header to open the popup and log in using a business account
Buttons for checkout and checkin dialogs are disabled if you are not signed into an account

It does not do any sort of caching or refreshing. It still needs a lot of work but.
#4

Closes #4 for now.


2 Tabs change to dropdown on mobile

When there are 3 or more Tabs on top banner, they need to switch to dropdown. Pages like Accessibility have this issue:

Seems that page tabs style has its own style, rather than inheriting Carbon tabs style. We need to see whether we should start inheriting Carbon tabs style or we should add dropdown support to website tabs style.

this is a duplicate #578


Mkdocs-based documentation website

Looks great! What's holding you to publish this?

I'm setting up CI 😉
I could not make it all work locally with act, so I'll have to merge then test by pushing.


Bump Dependencies #1887998499

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1887998499

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


istioctl kube-inject doesn't support podAnnotation

why is istioctl not calling https://github.com/istio/istio/blob/master/pilot/pkg/kube/inject/inject.go#L408?
This makes the behaviour of istioctl kube-inject different then the mutatingwebook

Thanks  and I agree we should make the webhook and istioctl behave the same.

@irisdingbj What are the plans for this issue?


[ new ] Snake! Using IR!

Building on top of #27
As I said on discord:

Running it should be as simple as going into examples/snake and running make && ./build/exec/snake (you may need to explicitly say NCURSES_VERSION=X make instead of make with X either 5 or 6 depending on the version of ncurses you have installed)

I absolutely love this example.
Can you (a) merge main in now that I've merged the PR for the branch on which this was based and (b) build this example in CI a la https://github.com/mattpolzin/ncurses-idris/blob/main/.github/workflows/ci.yml#L61..L65 so future changes to the library don't break this code.
For CI, don't bother with the checks I do for the longer example at the end, just the basic build like for all the other examples.


[v2023.1.0 compatibility]: RR's AMQP driver updated

Starting from v2023.1.0, RR requires a queue name (if the consumer is used) or/and routing key for the push operations. On empty, RR will return an error.

Hi @rustatian, is it possible do not keep connection for push-only operations?
Open connection -> push task -> close connection ?

What? Why? To perform slower?

to decrease connections number, or share single connection for all push operations (by one amqp exchange for example)
Maybe it will conflict with pipeline ideology, is not it ?

to decrease connections number

Why decrease the number of connections? And why share a single connection?


xiaomi imilab ec2

hello, what is the possibility of this hack working on a xiaomi imilab ec2 (CMSXJ11AG)?

I don't know.
Please post the firmware version you read in the app.
And take some picture of the internal board.

The gateway unit is using a Mediatek Mt7628NN SOC, running a customized version of OpenWrt (version 14, if I recall).
It has 64MB of RAM and 8MB of flash (as far as I could tell).
There is a serial (UART) console connector, close to the usb power port, connect at speed 57600 bps.

This hack can't work with your cam.
It needs a new hack.

what do you need to do it?? Or can you give me some tips
where to start??

Sorry but I don't own this cam and I can't create a hack at the moment.


Should vendor directory be committed?

More of a question, and less an issue.
I was trying to follow the guide here https://github.com/kubernetes-incubator/external-storage/tree/master/docs/demo/hostpath-provisioner but got stuck because the k8.io/api was included both in 2 vendors folders (the parent of my project and the one under this repo)
So I had to create a fork ( https://github.com/YiannisGkoufas/external-storage/tree/v3.2 ) from which I deleted the vendor directory and replaced the references to github.com/kubernetes-incubator/external-storage and I got it working

the hostpath provisioner has its own glide.yaml, if you run glide install -v in that directory then the generated vendor should take precedence over any other. The guide might be outdated is the main problem, what versions specifically are in conflict? I don't think it is needed to replace references, I believe you should be able to just delete the problematic dependencies like k8s.io/api from the vendor directory and things should build fine.
for external-storage in general, we check it in because...installing vendor can take a long time even with glide.yaml & glide.lock provided, and figuring out kube dependencies has been historically hard in case there is some conflict during install, so we just provide it and let you decide what to do with it at the cost of some MB (checking now, it seems the last time we updated dependencies nobody ran glide-vc so it's much bigger than expected...). Note there are two conflicting goals of this repo, 1 is to provide the library in which case the vendor dir is not needed, 2 is to house provisioners for which the vendor dir ensures reproducible builds


All competition pages show number of active league runs in top line - should only be league and latest decks and homepage

This is no longer true. Woo.


docs for directive ng-container

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ x ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question

Current behavior

No docs for ng-container. It is however used in the API documentation for:

NgPluralCase
NgPlural
NgSwitch - where it is mentioned "use a ng-container to group multiple root nodes", as referenced in this stackoverflow answer.

Expected behavior

There should be a separate entry in the angular docs where it is listed as a directive along with whether it is stable or experimental. Would have guessed in the section @angular/common but could not find it there after a quick search of the source code so not sure where exactly.
Minimal reproduction of the problem with instructions

Search or look through the docs site.
What is the motivation / use case for changing the behavior?

Given that it is presumably a public directive it would be good to know its usage as well as its stability.
Please tell us about your environment:

Angular version: 2.0.1

Browser: [all ]

Language: [all ]

Node (for AoT issues): node --version =

Let's continue at #2303 where we have an ongoing discussion.


Stop maintaining the Arch Linux package

I haven't been running Arch Linux for a while now. Therefor, maintaining the Arch package has been a bit of a hassle to the extent of it falling behind a version.
If someone wants to to take over maintaining the package I would appreciate it a lot and transfer ownership of the AUR package. If not, I'll disown the package on the AUR site and someone else can pick it up in the future.
Let's let this PR sit for at least a week before merging.

It's been two months and no one has volunteered to take over maintaining the package. I'll merge this PR and disown the package.


TLS import error on some systems

This issue has been happening to me several times when trying to install on some systems (after a quite basic mamba install -c conda-forge xdem:

In [1]: import xdem
ImportError                               Traceback (most recent call last)
 in
----> 1 import xdem
~/miniconda3/envs/xdem/lib/python3.9/site-packages/xdem/init.py in
----> 1 from . import coreg, dem, examples, spatial_tools, spstats, volume, filters
2 from .ddem import dDEM
3 from .dem import DEM
4 from .demcollection import DEMCollection
~/miniconda3/envs/xdem/lib/python3.9/site-packages/xdem/spstats.py in
20 with warnings.catch_warnings():
21     warnings.filterwarnings("ignore", category=DeprecationWarning)
---> 22     import skgstat as skg
23     from skgstat import models
24
~/miniconda3/envs/xdem/lib/python3.9/site-packages/skgstat/init.py in
----> 1 from .Variogram import Variogram
2 from .DirectionalVariogram import DirectionalVariogram
3 from .SpaceTimeVariogram import SpaceTimeVariogram
4 from .Kriging import OrdinaryKriging
5 from .MetricSpace import MetricSpace, MetricSpacePair, ProbabalisticMetricSpace, RasterEquidistantMetricSpace
~/miniconda3/envs/xdem/lib/python3.9/site-packages/skgstat/Variogram.py in
11 from scipy.spatial.distance import pdist, squareform
12 from scipy import stats, sparse
---> 13 from sklearn.isotonic import IsotonicRegression
14
15 from skgstat import estimators, models, binning
~/miniconda3/envs/xdem/lib/python3.9/site-packages/sklearn/init.py in
81     from . import __check_build  # noqa: F401
82     from .base import clone
---> 83     from .utils._show_versions import show_versions
84
85     all = ['calibration', 'cluster', 'covariance', 'cross_decomposition',
~/miniconda3/envs/xdem/lib/python3.9/site-packages/sklearn/utils/_show_versions.py in
10 import importlib
11
---> 12 from ._openmp_helpers import _openmp_parallelism_enabled
13
14
ImportError: dlopen: cannot load any more object with static TLS

Similar as: https://github.com/pytorch/pytorch/issues/2575#issuecomment-523657178
It can be quite problematic if it happens on systems where we don't control the C libraries (e.g., HPC).
A temporary fix is to add import sklearn before import xdem in the first called script, not very satisfactory...
Otherwise, downgrading sklearn but this has not worked for me with python 3.9

Now fixed by #200


Improved doc string for hyp2f1

I have changed the doc string of hyp2f1 function and included all the necessary details. Issue #6966

Changes in _ufuncs_cxx.pxd  etc. shouldn't be here. Guessing it's because of the merge commits; instead of merging in the master branch you should rebase on it.

@person142  I am not able to understand why my code is not able to pass the checks , can you please help me out here?

You can click through the CI status to see the test log: Docstring contains a non-printable character '\x0c'! Maybe forgot r"""?

I did include r at the starting , after defining the function in add_newdocs.py but still the same result.

There are at least a couple more errors, e.g. not all of the math is in math mode.

Is that the reason of not passing the checks or the math code is incorrect according to the hyp2f1?

Let's try to get this sorted out. The erroneous +13,737 LOC and large number of commits is making review a bit of a pain, so we'll start there. Some things for the future:

It looks like you did your development on master; it's better to do it on a topic branch.
Don't merge the master branch to resolve conflicts--instead rebase on master (git rebase).

Probably the easiest way to fix this now is to:

Rename this branch to something else (git branch -m <oldbranch> <newbranch>). Let's call it hyp2f1-doc for concreteness.
Grab a new copy of the master branch from the main SciPy repo so that you have a clean master again.
 Also modify the commit message to be formatted according to https://docs.scipy.org/doc/numpy/dev/gitwash/development_workflow.html#writing-the-commit-message.
There should now be one commit that only modifies add_newdocs.py.
Push the changes, force pushing (git push -f) is fine.

When i did rebasing of the branch , in the interactive mode, there was noop present rather then the commits, so I was not able to squash the commits.

Please don't push more changes until this is sorted out; it is making the situation worse.

When i did rebasing of the branch , in the interactive mode, there was noop present rather then the commits

I am not sure what you mean here. When you run git rebase -i it opens a text editor which shows the commits you have added since your topic branch diverged from master. Did you create a new topic branch as instructed?

yes i followed the same steps.... can i make a new pull request instead?

can i make a new pull request instead?

Sure. Just be careful of the points mentioned above.

I am really sorry for the last commits , please ignore them


Failed to parse nested Github Connection type ?

Hello, I found a confused situation, when I try Github GraphQL API and use the script below:
query TestQuery($first: Int, $query: String!, $type: SearchType!) {
  search(first: $first, query: $query, type: $type) {
    edges {
      node {
        ... on Repository {
          name
          owner {
            login
          }
        }
      }
    }
  }
}

Then it works fine. But, if now I also want to query repositoryTopics, and use below:
query TestQuery($first: Int, $query: String!, $type: SearchType!, $count: Int) {
  search(first: $first, query: $query, type: $type) {
    edges {
      node {
        ... on Repository {
          name
          owner {
            login
          }
          repositoryTopics(first: $count) {
            edges {
              node {
                ... on RepositoryTopic {
                  topic {
                    name
                  }
                }
              }
            }
          }
        }
      }
    }
  }
}

After that, I will get Failed to parse http response, but it is correct and works in Github GraphQL Explorer. Can Apollo support this kind of nested query script ? Thanks.

Hi, is there any update ?  🤔

Sorry was busy with the other stuff, could you please attach the full stack of the error?

Did you follow github instructions to generate token and use it to make any requests?

Could you please provide what version of Apollo are you using.
Just tried on my sample your query works for me, the only assumption it could be some permission issues?

I found it works fine now, strange.
Thanks, and sorry for wasting your time.  😮


On keeping binaries_path in ~/.config/airwave/airwave.conf…

Hey,
NixOS-related issue once more. So after initial launch, Airwave generates and keeps binaries_path in airwave.conf. My file looks like that:
{
   "binaries_path" : "/nix/store/5lzjpc90rzi8gcf196hvzklws8s8r72m-airwave-1.3.2/libexec",
   […]
}

Now, each update to the Airwave itself will result in a different package hash being generated in the /nix/store where all programs are stored (we have no /usr, /bin and the like) — to keep things immutable and easily rollback-able. This is the essence of NixOS.
Now, you might see the problem already. Scenario:

I’m launching Airwave 1.3.2 and creating a link.
System update. Airwave gets updated to 1.3.3.
Until I run GC on the /nix/store—i.e. sudo nix-collect-garbage -d—the old versions of programs still reside in the /nix/store.
If I now run a DAW, my links will still be using Airwave 1.3.2 binaries and that’s fine and expected, I guess.
However, if I now run airwave-manager and create a new link, the old binaries will still be used for it! That’s bad. Moreover, I’d say, it should automagically repair (regenerate) all existing links to use the  new binaries.

This kind of an error will be pretty hard to debug for the user.  But, from my experience, that happens not so often.
Thank you!

One solution would be to remember binaries_path only if it was changed by the user. If not, generate it every time airwave_manager is launched.
The question of regenerating stays open. Hmm. airwave_manager could detect that existing links are using binaries’ paths that are not equal to the generated path. And if that is the case, ask user if they want to regenerate all such links.


Add convert commands

New commands to be added for the new release. It should cover most simple usage of data extraction from Kraken reports to standard tree formats and CSV files.
For this release I focus on the Newick and Nexus format. Further work may involve supporting other formats. My main goal is to be compatible with R's ape read.tree.

name
# input
# output
format
misc

convert-phylo
1
1
newick / nexus

convert-abundance
1
1
CSV

Options:

--has-headers

Output kind:

stdout
one file

closed by #21


auto-populate broken since config rework

This issue was originally submitted by @PorPit in SpongePowered/SpongeForge#2393.

I am currently running

SpongeForge version: 1.12.2-2705-7.1.0-BETA-3409

Forge version: forge-1.12.2-<IP_ADDRESS>05

Java version: 1.8

Operating System: windows server 2012

Plugins/Mods:
Plugins (27): Minecraft, Minecraft Coder Pack, SpongeAPI, SpongeForge, ActionBroadcaster, CMD Item, CarrotShop, CrazyFeetSponge, EconomyLite, EpicBanItem, GriefPrevention, LinkMe, LuckPerms, MCRMB, McrmbCommand, Nucleus, PixelVip, PlaceholderAPI, PlotSquared, ProjectCore, SpongyCord, StackBan, StopUnloadWorld, Twenty Ticks, VirtualChest, VirtualTool, World Polisher
Mods (44): Minecraft, Minecraft Coder Pack, Forge Mod Loader, Minecraft Forge, SpongeAPI, SpongeForge, Advanced Solar Panels, Applied Energistics 2, Baubles, Better Records, Botania, CraftTweaker JEI Support, CraftTweaker2, CraftTweakerSync, CreativeCore, CreativeCoreDummy, CustomNPCs, Extra Planets, FTB Utilities, FTBLib, FoamFix, FoamFixCore, Galacticraft Core, Galacticraft Planets, Immersive Engineering, Immersive Tech, IndustrialCraft 2, Iron Chest, Just Enough Items, MJRLegendsLib, MTLib, Mekanism, MekanismGenerators, Micdoodle8 Core, MineCamera, MineCameraDummy, Mod Tweaker, OnlinePictureFrame, Pam's HarvestCraft, Shadowfacts' Forgelin, Thaumcraft, Wireless Crafting Terminal 2, WorldEdit, p455w0rd's Library

Issue Description
when i use the new sponge version（3361 or 3409）
set the  global config  "tileentity-activation=true" and "auto-populate=true" and restart server
config does not add any tileentities to default settings
but when i use sponge 7.1.0-BETA-3206
the default settings can be generated normally

after discussion and some testing, the issue is the config system rework which got rid of config enabled
getActiveConfig will now always return the world specific config, due to how the fallback works in the new system
sadly it is currently impossible to see which config has actually set auto-populate set explicitly, as a temporary fix we should for now always force auto-populate to go to the global config

q: maybe we just move the activation stuff to another config file?
a: then we lose the ability to overwrite things on a per world/dimension basis


erroneous ArrayBuffer polyfill when using node environment with node 7+

Do you want to request a feature or report a bug?
bug
What is the current behavior?
When working with ArrayBuffers, it appears that when jest is run using the node environment (--env node) a polyfill is being used for ArrayBuffer incorrectly only in node v7+.  It is unclear to me if this really is a jest-specific problem, but it does seem to be related to the jest environment.
If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can yarn install and yarn test.
https://github.com/mbroadst/jest-arraybuffer-issue

switch to node v6.2.2, yarn install && yarn test and the test will pass
switch to node v7.74 (latest) yarn install && yarn test and the test will fail

What is the expected behavior?
the test passes
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
see: github repo

i haven't looked deep into the issue, but what i found interesting is:
~/tmp/jest-arraybuffer-issue > yarn test
 FAIL  ./test.js
  ● fetch › should fetch a document

    expect(value).toBeInstanceOf(constructor)

    Expected value to be an instance of:
      "ArrayBuffer"
    Received:
      []
    Constructor:
      "ArrayBuffer"

      at test.js:8:26
      at process._tickCallback (internal/process/next_tick.js:109:7)

  fetch
    ✕ should fetch a document (372ms)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   0 total
Time:        0.531s, estimated 1s
Ran all test suites.

and right after that
dabramov-mbp ~/tmp/jest-arraybuffer-issue master* ./node_modules/.bin/jest
 PASS  ./test.js
  fetch
    ✓ should fetch a document (398ms)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        1.136s
Ran all test suites.

ok. sorry, i missed --env node, now i can reproduce

it consistently fails on all node versions that i tried, even without Jest.
dabramov-mbp ~/tmp/jest-arraybuffer-issue master* nvm use 6.2.2
Now using node v6.2.2 (npm v3.9.5)
dabramov-mbp ~/tmp/jest-arraybuffer-issue master* node
> require('isomorphic-fetch'); fetch('https://google.com').then(response => response.buffer()).then(buf => console.log(buf.constructor.name));
Promise { <pending> }
> Buffer

This is probably related to this change:
https://github.com/bitinn/node-fetch/commit/f0d0675d00845c59ac4bac58d8c9b89d2a39d38e

@dmitriiabramov hm you're right that all of a sudden I can't repro the success case with earlier version of node - I'll work on updating that. In the meantime, your test is incorrect above, you need to check the constructor name of the buf.buffer (internal buffer) of the response:
mbroadst@gorgor:~/Development/jest-arraybuffer-issue (master %)$ node --version
v6.2.2
mbroadst@gorgor:~/Development/jest-arraybuffer-issue (master %)$ node
> require('isomorphic-fetch');
[Function]
>  fetch('https://google.com').then(r => r.buffer()).then(b => console.log('ctor: ', b.buffer.constructor.name, ', instanceOf: ', b.buffer instanceof ArrayBuffer));
Promise { <pending> }
> ctor:  ArrayBuffer , instanceOf:  true

mbroadst@gorgor:~/Development/jest-arraybuffer-issue (master %)$ node --version
v7.7.4
mbroadst@gorgor:~/Development/jest-arraybuffer-issue (master %)$ node
> require('isomorphic-fetch');
[Function]
>  fetch('https://google.com').then(r => r.buffer()).then(b => console.log('ctor: ', b.buffer.constructor.name, ', instanceOf: ', b.buffer instanceof ArrayBuffer));
Promise { <pending> }
> ctor:  ArrayBuffer , instanceOf:  true

So what we are seeing here is that in both versions of node the returned buffer has an internal ArrayBuffer which is an instanceof ArrayBuffer. What the broken jest test is showing you is that for some reason, when --env node is supplied to jest, that same object is not an instanceof ArrayBuffer, and so my guess is that ArrayBuffer is being polyfilled somewhere - but only when --env node is specified.

@dmitriiabramov the problem seems to be with the setup of your node environment here: https://github.com/facebook/jest/blob/master/packages/jest-environment-node/src/index.js#L29-L38
Specifically, I think this line:
const global = (this.global = vm.runInContext('this', this.context));

thinks its making a copy of the global node context, but it isn't at all (if you run that locally you'll see the result is {}). So if below that I put: global.ArrayBuffer = ArrayBuffer then this bug goes away. I'm not sure what the intent is here, so I'm refraining from submitting a PR, if you can fill me in I have no problem doing the work.
For instance, why not just say const global = Object.assign({}, global) ? Then you wouldn't have to manually assign the global properties.

ah. i know what's going on there.
Because we run tests in a separate isolated context (inner scope) we end up with two instances of ArrayBuffer (one from the inner scope, and one from the outer scope, where the Jest runner runs itself).
Both are exactly the same, both have constructor.name === 'ArrayBuffer', but they come from different node environments so they are two different objects and instanceof will return false.
The change that you suggested would fix the issue, since the it will share the same object between two environments.
We don't share all properties between outer and inner context, because if a test modifies something shared like Array.prototype.toString = () => 'lol' that will affect other test files (and that's really bad).
one thing i'm not sure is why fetch returns an instance of ArrayBuffer from the outer scope, rather than from the inner scope of the test.

ok. i'm not sure what exactly is going on, but it seems like the Buffer that's created here
https://github.com/bitinn/node-fetch/blob/f0d0675d00845c59ac4bac58d8c9b89d2a39d38e/src/body.js#L269
is an instance of an outer scope Buffer and not inner scope. that's why the check fails

@dmitriiabramov are you sure? That should mean I could explicitly set global.Buffer = Buffer in the env setup and it should work, but it doesn't. Only setting global.ArrayBuffer = ArrayBuffer seems to work.

I countered this issue when writing some integration tests using the ws library in the node environment.
I set socket.binaryType = 'arraybuffer' and received an ArrayBuffer in the callback but it was failing instanceof ArrayBuffer checks.
Creating and using a custom environment solved it, code below:
const NodeEnvironment = require('jest-environment-node')

module.exports = class ArrayBufferEnvironment extends NodeEnvironment {
  constructor(config) {
    super(config)
    this.global.ArrayBuffer = ArrayBuffer
  }
}


Possible article (Azure Batch Account - For User Subscription Mode) update

In think that it should be added the following note : The user who can deploy Azure Batch Service in user subscription mode should be Directory Service Admin, and not just a user with owner privileges in this directory.

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

ID: 2146b9a6-94f1-b2a8-972e-63602992d092
Version Independent ID: ec164b99-5fc9-f5a1-6eb7-1d5fb1cad568
Content: Create a Batch account in the Azure portal
Content Source: articles/batch/batch-account-create-portal.md
Service: batch
GitHub Login: @laurenhughes
Microsoft Alias: lahugh

Thanks for the feedback! We are currently investigating and will update you shortly.

@gxgrammatikos can you elaborate on why you believe this change in the doc is needed? Looking through the doc I am not finding any issues with how it is setup

Hello Micah,
Allow me to elaborate, the problem is that I couldn’t deploy an Azure Batch (with user subscription Pool Allocation Mode) if  the user was not service admin if you try so, you get the following error.
<EMAIL_ADDRESS>If you want to reproduce the error, you should try to deploy a new Azure Batch Service with Use subscription pool allocation mode, with not Service Admin user.
Regards,
George Chrysovalantis Grammatikos
From: Micah<EMAIL_ADDRESS>Sent: Friday, February 8, 2019 7:50 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: gxgrammatikos<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Possible article (Azure Batch Account - For User Subscription Mode) update (#24561)
@gxgrammatikoshttps://github.com/gxgrammatikos can you elaborate on why you believe this change in the doc is needed? Looking through the doc I am not finding any issues with how it is setup
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/24561#issuecomment-461887787, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKWajIfig5Dv1oVqvHpinFbaepDFU0Ouks5vLbjJgaJpZM4au7_a.

@gxgrammatikos got it. Thank you. I will attempt a repro and get back to you

@gxgrammatikos also, did you see the note at the beginning of the doc?

Yes, of course I read it. But, it doesn’t says that you must be a service administrator if you want to deploy  a “user subscription”  pool allocation mode.
Regards,
George Grammatikos
From: Micah<EMAIL_ADDRESS>Sent: Friday, February 8, 2019 10:27 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: gxgrammatikos<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Possible article (Azure Batch Account - For User Subscription Mode) update (#24561)
@gxgrammatikoshttps://github.com/gxgrammatikos also, did you see the note at the beginning of the doc?
[image]https://user-images.githubusercontent.com/32313503/52504265-c0242e80-2b9c-11e9-9885-e76c824796ca.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/24561#issuecomment-461935872, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKWajE2oJ2OPnGbEArb2AO22tWvi_p_6ks5vLd1-gaJpZM4au7_a.

Thanks @gxgrammatikos
I will assign to the author to review to see if we can make this more clear.
@laurenhughes can you take a look?

@gxgrammatikos In a previous reply you added some screen shots. Unfortunately the screenshots do not show up if you reply via email. They only show up if you reply from a PC directly in GitHub. Could you possibly go back and provide us with the screenshots again but do it from a PC so we can see?

Hello @MicahMcKittrick-MSFT
Allow me to elaborate, the problem is that I couldn’t deploy an Azure Batch (with user subscription Pool Allocation Mode) if  the user was not service admin if you try so, you get the following error.

If you want to reproduce the error, you should try to deploy a new Azure Batch Service with Use subscription pool allocation mode, with not Service Admin user.

Thanks for sending the screenshot again, @gxgrammatikos!
Did you try following the Additional steps for user subscription mode section? I believe the steps in that section will fix the error you're seeing.

I also followed these steps. But the problem solved when I logged in the Portal with the Service Admin User. I couldn't find this instruction. To reproduce the problem try to deploy the Azure Batch Service with not Service Admin user, login with a contributor role user.

The only criteria for creating an account are:

The Batch service must have an "Owner" or "Contributor" role.
The user creating the account must be an "Owner" or "Contributor" to the subscription.

I attempted to repro the error with a "Contributor" role account, but the error is only present when the Batch service itself is not a "Contributor" or "Owner".
I've passed this information on to the engineering team, so please let me know if you've followed the above criteria and are still having trouble creating an account, in which case we can investigate further.

I checked again and it worked. Probably something changed in the Portal because I follow the same steps since the time I send the feedback and now the deploy was successful without any error.

I have a new clue for the feedback, I reproduced it under an Enterprise Agreement Subscription, but not under my MS Azure Sponsorship subscription.

Thanks for the additional feedback, @gxgrammatikos. Since you were able to successfully create an account, I'm going to go ahead and close the issue. I've passed the Enterprise Agreement Subscription on to the engineering team to investigate.
#please-close


IMOD SIRT

Change log

IMOD align

Binning images at newstack
Added implementation of auto-fiducial-finding [WIP, not working]

IMOD recon

Outputting reconstruction algorithm in metadata

RLDeconv

Outputting metadata after operation

Codecov Report
Base: 13.94% // Head: 39.12% // Increases project coverage by +25.18% :tada:

Coverage data is based on head (c5c370b) compared to base (2f9d016).
Patch coverage: 27.27% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##              dev      #75       +/-   ##
===========================================
+ Coverage   13.94%   39.12%   +25.18%
===========================================
  Files          15       15
  Lines        1850     2267      +417
===========================================
+ Hits          258      887      +629
+ Misses       1592     1380      -212

Impacted Files
Coverage Δ

src/Ot2Rec/magicgui.py
69.76% <ø> (+1.19%)
:arrow_up:

src/Ot2Rec/params.py
84.21% <ø> (+59.64%)
:arrow_up:

src/Ot2Rec/rlf_deconv.py
0.00% <0.00%> (ø)

src/Ot2Rec/recon.py
56.93% <25.00%> (+43.14%)
:arrow_up:

src/Ot2Rec/align.py
58.26% <83.33%> (+45.13%)
:arrow_up:

src/Ot2Rec/metadata.py
30.23% <0.00%> (+10.07%)
:arrow_up:

src/Ot2Rec/motioncorr.py
24.08% <0.00%> (+10.10%)
:arrow_up:

src/Ot2Rec/aretomo.py
56.61% <0.00%> (+40.13%)
:arrow_up:

src/Ot2Rec/logger.py
100.00% <0.00%> (+41.66%)
:arrow_up:

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


[FLINK-28709] Implement dynamic filtering operators

What is the purpose of the change
This pull request introduces operators to support dynamic filtering.
Brief change log

Introduce coordinatorListeningID in SourceCoordinator to listen to events from other coordinators.
Introduce DynamicFilteringData and the DynamicFilteringDataCollectorOperator to build and distribute the data.
Introduce ExecutionOrderEnforcerOperator to ensure the source with dynamic filtering is executed after the DynamicFilteringData is collected.

Verifying this change
This change added new tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Hi @gaoyunhaii ,
Thanks for the reviewing! All comments so far should have been addressed and resolved.

@flinkbot run azure

Hi @godfreyhe ,
Thanks for the suggestions!
I suppose I've resolved most of the comments, and the failover case is coming soon.


Add more icon sizes to prevent pixelated icons on Windows

This fork modifies assets/icon-scaled.ico to contain icons in these 5 sizes:

16x16
32x32
48x48
96x96
256x256

The existing version of this file contains only a 96x96 icon, which appears pixelated when scaled to different sizes.
This fork does not currently modify assets/icon-scaled.icns, as I don't have a Mac and therefore don't know if the issue exists on macOS or not.

Issue doesn't exist on macOS the last time I checked

@CocoTheMii Is the PR finished or did you forget to mark it as draft?

@CocoTheMii Is the PR finished or did you forget to mark it as draft?

It's finished as far as Windows is concerned. If the issue doesn't exist on macOS, then this should be everything.

I built Catalyst and installed it with this pr checked out and it appears the window titlebar icon is still messed up as it was

Ah, right. I forgot to check the titlebar icon.
Where would that icon be located? If my patch didn't fix it, then it's using a different file.

Ah, right. I forgot to check the titlebar icon.
Where would that icon be located? If my patch didn't fix it, then it's using a different file.

The one who made the icon is named @VelocityDesign, the person who applied the icons to the software is @webdev03

I can scale the icons directly from my design software. I will also make sure to export a set specifically for Mac (a family member has one I can test on). The Mac set will be compliant with Apple's HIGs.

ok


v0.5.8 - ImportError: Could not load whisper.dll.

[2022-12-01 17:42:11,680] whispr.:31 ERROR ->
Traceback (most recent call last):
File "whispr.py", line 28, in
File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
File "whisper_cpp.py", line 863, in
File "whisper_cpp.py", line 551, in call
ImportError: Could not load whisper.dll.

Could you try the latest version https://github.com/chidiwilliams/buzz/releases/tag/v0.6.4?

for 0.6.4
[2022-12-15 01:23:01,568] transcriber.:36 ERROR ->
Traceback (most recent call last):
File "buzz\transcriber.py", line 33, in
File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
File "buzz\whisper_cpp.py", line 863, in
File "buzz\whisper_cpp.py", line 551, in call
ImportError: Could not load whisper.dll.
[2022-12-15 01:23:01,601] gui.init:556 DEBUG -> Loaded settings from path = \HKEY_CURRENT_USER\Software\Buzz\Buzz
[2022-12-15 01:23:01,602] gui.init:556 DEBUG -> Loaded settings from path = \HKEY_CURRENT_USER\Software\Buzz\Buzz
[2022-12-15 01:27:31,038] transcriber.:36 ERROR ->
Traceback (most recent call last):
File "buzz\transcriber.py", line 33, in
File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
File "buzz\whisper_cpp.py", line 863, in
File "buzz\whisper_cpp.py", line 551, in call
ImportError: Could not load whisper.dll.
[2022-12-15 01:27:31,065] gui.init:556 DEBUG -> Loaded settings from path = \HKEY_CURRENT_USER\Software\Buzz\Buzz
[2022-12-15 01:27:31,066] gui.init:556 DEBUG -> Loaded settings from path = \HKEY_CURRENT_USER\Software\Buzz\Buzz
[2022-12-15 01:27:58,946] connectionpool._new_conn:1003 DEBUG -> Starting new HTTPS connection (1): api.github.com:443
[2022-12-15 01:27:59,453] connectionpool._make_request:456 DEBUG -> https://api.github.com:443 "GET /repos/chidiwilliams/buzz/releases/latest HTTP/1.1" 200 1294

The app itself still work though, right? You're just not able to use the Whisper.cpp models?

The app itself still works though, right? You're just not able to use the Whisper.cpp models?

Yes, the app is working fine, but the "Enable GGML Inference" option is gray and cannot be selected.


Trie (Medium) java soln

Solved issue #550
In Language- Java

[x] Updated README Link
[x] Added code with comments

Thanks for contributing!

Mention issue number

Done


tmuxPlugins.tilish: init at 2020-08-12

Motivation for this change
Adding tmux plugin tilish. It provides useful keybindings similar to i3wm/sway to tmux.
Things done

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

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

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

Result of nixpkgs-review pr 105381 1

1 package built:

tmuxPlugins.tilish


The readme file update

This update only states the obvious (year 2023). Should be distributed across branches.

All branches should see the same README as in main, including 1st-branch.


Ctrl + F4 closes wrong tab

Description
I expected to close tab with Search-tab but Sql-tab was closed.
Same issue - for tab Error log (seems like all non-sql tabs affected?).
This hotkey is very common in many applications where tabs are used.
I think, it should behave the same in spite of the kind of the tab.
Please also check it for all other kinds of tabs.
https://github.com/user-attachments/assets/55fd541a-2a6f-4051-86fa-a10aebe2c29e
DBeaver Version
Community Edition 24.1.5-202408122322
Operating System
Windows 10
Database and driver
PostgreSQL 16
PostgreSQL JDBC Driver 42.7.4
Steps to reproduce

search some text in tables
set focus to Search-tab by mouse click within the content of Search-tab
press Ctrl + F4

Additional context
No response

Thank you for the report

It is common in many applications where tabs are used, that there are "document tabs" and "tool tabs".
It is also common that Ctrl+F4 (Ctrl+Tab, and other tab-related commands) operates with "document tabs" even when the "tool tab" is in focus.
"Search tab" is a "tool tab", so it operates exactly the way it is expected of the "tool tab".
So I fail to see any problems here.

The tool tab has an item "Close" in its context menu but there is no way to use any hotkey to use it from keyboard.

All of this is not a DBeaver-specific thingy, it is a defaults coming from the eclipse platform, as far as i can see, but still.
@ask9 what you want is a different command, note the description of the command you are complaining about:

("Tool tabs" are not editors in the eclipse platform)
Here i changed the binding of Ctrl+W to execute another command, the one you wanted:

(All top-level view tabs of the perspective are "workbench parts" in the eclipse platform)

Thank you!
After setting to Ctrl + F4 it works as expected.
I believe this command should have a hotkey Ctrl + F4 or Ctrl + W by default.

Setting the hotkey gave me what I need. I'll leave this request open for now as It might still be good to implement this suggestion, but do feel free to close it if you feel it doesn't add enough value or not enough benefit to other users.


⚠ Bofrost.at has degraded performance

In 4253f5a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1588 ms

Resolved: Bofrost.at performance has improved in 523652b after 8 minutes.


Typescript freezes and never being initialized

Typescript freezes and never being initialized at the random situation. Restarting doesn't help. The only thing that can fix that issue is close/open VSCode. But sometimes after opening a random typescript file the issue repeats.
I already tried to disable/enable built in typescript plugin but nothing helps to get rid of this issue. LMK if I can provide more information
Thank you in advance

Please fill out the issue template
/needsMoreInfo
Also, does this reproduce in the latest VS Code insiders build with all extensions disabled?


Validation fails to trigger when next field is already filled out

To reproduce:
On the index demo, edit the markup to fill in the second field with a valid email address. Load that demo, type "abc" into the first field, tab to the next field, tab back, delete the value.
Expected: "This field is required" error.
Actual: No error
Originally reported as #244, but its really not specific to select elements. Reverted the fix from #396 in 5934f4d4509b900594b40eb4399446dd46debf36, as the caused a regression (#521).
This seems pretty tough to fix, as it scratches on the very basics of how the plugin figures out if a field should be validated or not. Haven't yet been able to come up with a better solution.

Replaced by #1508.


Color scheme mismatch

Color Palette defined at https://github.com/horizon-ui/horizon-ui-chakra/blob/main/src/theme/styles.js differs from the documentation at https://horizon-ui.com/documentation/docs/theming/theme#gray
For example gray.100 in code is "#FAFCFE" whereas in documentation, it is "#EDF2F7". This leads to designs going off sync. I believe there are other examples of mismatch too.

Hello, @misrasaurabh1 ! Thanks for using our products! I changed the gray.100, I'm going to let this issue open if there are anymore theming problems. Regards, Vlad.


Fixed expandable content does not work when using enableVirtualization in Table

Changes

When enabling virtualization using the virtualizer from @tanstack/react-virtual, expandable content was reported broken since it was originally rendered with each original row, which was completely ignored by the virtualizer. To fix this problem, the expandable content in this PR is moved to a separate component and regenerated its index. The virtualizer now is made aware of the changes in the page size when sub-components are defined. These components are rendered accordingly to their indices associated with which row they are expanded from.
Testing

Confirmed that sub-components are rendered correctly when enableVirtualization is set to true.

An e2e test is added for expandable contents in virtualized table.
Docs

N/A

@smmr-dn Can you please explain the new changes? I'm a bit lost, because this PR looks completely different from when I first reviewed it and I don't see any context anywhere around why these large changes were made.

For the new approach, instead of making the sub-component a sub-row, the sub-component now is a component with index. The virtualizer is also notified about this change in the page size when there are any expanded content/row on the screen.

The listOfRowsAndSubComponents is an array of strings indicating the rows and currently expanded rows in the page. The length of this array is fed to the virtualizer so it will be notified every time the page changes its size due to expansion and collapse.

The getRowIndices is an array which calculates the index of the subcomponent, whose data is associated with the expanded row. We use this index calculation to determine if the current row is the main row or subcomponent to render accordingly. (The list of virtual items only appends more rows and keeps the main rows in the original order, so that's why we need to map subcomponents with the correct indices).

For example, if the row0 (index 0) is expanded then the index of the subcomponent0 is 1, row1 (index 2) is not expanded, row2 (index 3) is associated with subcomponent2 with the index of 4. Therefore, subcomponent2 should take the data from virtual item of index 2 and rendered as item of index 4.

As long as we assign the right index to the correct subcomponent, the table should be rendered correctly.


How do I close a connection?

does connect just timeout?
or is there some sort of close command?

looks like I just call .end.
Thanks!


Helm charts for snapshot-warmer and snapshot-scheduler & snapshot_maker container files

This PR creates 2 helm charts

charts/snapshot-warmer README
charts/snapshot-scheduler README

This used to be deployed manually from https://github.com/oxheadalpha/tezos-tarball-maker with kubectl.
Now its deployed via helm chart and targets 1 rolling tezos node and 1 archive tezos node in each namespace and kicks off the snapshot workflow.

Please rename the PR to reflect what it is actually accomplishing

Question on the kubectl commands we use in regards to the sleeping we do: Why not just use kubectl wait cmd?
https://mrkaran.dev/posts/kubectl-wait/
https://kb.novaordis.com/index.php/Kubectl_wait
There is also a --wait flag you can pass to cmds. But it may be that the kubectl wait is the better way to do it. Could be because you can pass multiple resources to it.
kubectl wait --help

Never seemed to work right unfortunately.

Question on the kubectl commands we use in regards to the sleeping we do: Why not just use kubectl wait cmd?

Never seemed to work right unfortunately.

It does seem actually to be considered an experimental command:
❯ k wait --help
Experimental: Wait for a specific condition on one or many resources.

Cool you tried though. Now we know :)


AMBARI-25615 Add force_metrics_fetch flag to get the live metrics data from the HDFS (ihorlukianov)

What changes were proposed in this pull request?
Added the option force_metrics_fetch to immediately fetched from the source,  bypassing the cache
How was this patch tested?
Manually + UT

retest this please


fix(TaskProxy): Bluebird usage

promisify was using the wrong argument syntax.
Fixing this also fixes the cache.clear() method - previously it
returned undefined errors.
:LINK: http://bluebirdjs.com/docs/api/promise.promisify.html

Thanks, @jiaweihli. Published as v0.4.2.


Documentation/tests

Document how to create a new test.

Maybe we can add how to view tests? For example in the browser it's pretty straightforward but in cohtml you have to load http://localhost:9876/debug.html to view the tests


Tensors must be contiguous when deepspeed broadcasting

When I train the model with one GPU. It works. But When I set the --num_gpus 8, it returns "Tensors must be contiguous".
The command:
deepspeed --num_gpus 8  --module training.trainer --deepspeed /root/github/dolly/config/ds_z3_bf16_config.json  --epochs 1 --local-output-dir /root/models/dolly/dolly --per-device-train-batch-size 8 --per-device-eval-batch-size 8 --lr 1e-5
The output:
Traceback (most recent call last):   File "/root/miniconda3/envs/dolly/lib/python3.8/runpy.py", line 194, in _run_module_as_main     return _run_code(code, main_globals, None,   File "/root/miniconda3/envs/dolly/lib/python3.8/runpy.py", line 87, in _run_code     exec(code, run_globals)   File "/root/github/dolly/training/trainer.py", line 265, in <module>     main()   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/click/core.py", line 1128, in __call__     return self.main(*args, **kwargs)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/click/core.py", line 1053, in main     rv = self.invoke(ctx)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/click/core.py", line 1395, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/click/core.py", line 754, in invoke     return __callback(*args, **kwargs)   File "/root/github/dolly/training/trainer.py", line 257, in main     train(**kwargs)   File "/root/github/dolly/training/trainer.py", line 220, in train     trainer.train()   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/transformers/trainer.py", line 1543, in train     return inner_training_loop(   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/transformers/trainer.py", line 1612, in _inner_training_loop     deepspeed_engine, optimizer, lr_scheduler = deepspeed_init(   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/transformers/deepspeed.py", line 344, in deepspeed_init     deepspeed_engine, optimizer, _, lr_scheduler = deepspeed.initialize(**kwargs)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/__init__.py", line 125, in initialize     engine = DeepSpeedEngine(args=args,   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 301, in __init__     self._configure_distributed_model(model)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 1187, in _configure_distributed_model     self._broadcast_model()   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 1102, in _broadcast_model     dist.broadcast(p,   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/comm/comm.py", line 127, in log_wrapper     return func(*args, **kwargs)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/comm/comm.py", line 232, in broadcast     return cdb.broadcast(tensor=tensor, src=src, group=group, async_op=async_op)   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/deepspeed/comm/torch.py", line 70, in broadcast     return torch.distributed.broadcast(tensor=tensor,   File "/root/miniconda3/envs/dolly/lib/python3.8/site-packages/torch/distributed/distributed_c10d.py", line 1404, in broadcast     work = group.broadcast([tensor], opts) RuntimeError: Tensors must be contiguous

What are you running on? does it have 8 GPUs?

I am running it on my own server and it has 8 GPUs. This problem also occurs when the num_gpus set to 1 or 2, or 4.

You're on CUDA 12. I wonder if that makes a difference, or if you're following this repo, if you're installing CUDA 11.3 libs. I would think that would result in a different error. No idea why it wouldn't work otherwise

Thanks, I'll try it. Hope it works.


image flipping

Hello, I have a question to ask. I saw that image flipping was used for data enhancement, but the instance label did not change. For example, the left label is 1, 2, 3, and the right label is 4, 5, 6, According to the label definition, when the slope is negative, the label is 1, 2, 3. After flipping, the labels 4, 5, and 6 don't need to become 1, 2, and 3? I don't know if there is a problem with my understanding. I look forward to receiving your reply, thank you!

I think you might be right. The exist label should also be flipped.
Could you try it and see whether the performance will be improved ?

Thank you for your reply, I tried to remove the flipping directly, which has little effect on the results. I will try it you said and see whether the performance is improving.The probability of flipping in the code is 0.5, why does it not have a great impact on the result ? I don’t know how to understand this. Can I give your opinion? Thanks!

We don't have instance label, only exist label, 1 stand for lane is exised, 0 is not. For example, [1,1,1,0] means the image has two left lanes, one right lane. We don't number it as 1,2,3,4,5, ...
If filp, [1,1,1,0] should be [0,1,1,1], are you using this method?

Thanks! The acc in my custom data improves 2% after changing the exist label when I used image flipping.

Do you try the CULane or Tusimple?


chore(examples): updated contributing & docs with SAM

Description of your changes
Now that the project offers SAM-based examples, there are some changes needed to the documentation, contributing guidelines, and readme files needed to reflect this change. This PR introduces these changes and it should be the last one related to SAM.
List of changes:
Readme files
This includes:

Project's README.md
packages/tracer/README.md
packages/metrics/README.md
packages/logger/README.md

The "Examples" link above used to bring you to the examples/cdk folder, it now brings you to examples so you can choose your favourite flavour to use.

Added the link to the SAM samples under the "Examples" section.
Documentation

In the main page (docs/index.md) there is a list of items under the "Examples" section, I have added an item for SAM
Contributing

I have added AWS SAM CLI as requirement for the project, as it's now needed in order to build the SAM sample as part of the pre-push hook. (This is similar to CDK, but since SAM CLI is not vended via NPM we must install it manually).

Finally I also added the Open in GitPod button that will automatically create the setup to start contributing in a few seconds.
Gitpod / Codespaces
With the latest changes I have made changes also to the configurations needed to use GitHub Codespaces & Gitpod. These include the addition of SAM CLI, as well as the initialisation of both example folders. This last step is needed to successfully execute the pre-push hooks.
How to verify this change
Check content of the CONTRIBUTING readme (link to rendered version in this PR), and optionally check README files + docs links to examples.
If you want to try the Gitpod setup click here (it will open a new Gitpod workspace in your account using this branch).
Related issues, RFCs
#849
#674
PR status
Is this ready for review?: YES
Is it a breaking change?: NO
Checklist

[x] My changes meet the tenets criteria
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have made corresponding changes to the examples
[x] My changes generate no new warnings
[x] The code coverage hasn't decreased
[x] New and existing unit tests pass locally and in Github Actions
[x] The PR title follows the conventional commit semantics

Breaking change checklist
N/A

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Wondering if we might want to wait until #877 is merged (Node16) and just do this in one go?

Thought about that, but at the same time I wanted to follow the CONTRIBUTING guidelines and keep the PR focused & w/ small diff (both are already quite big).


Add a "timed" mode to command

I made changes I talked about in Issue #5
This will alter the temperature every 10 minutes. The temp adjusts in a linear fashion based on the time of day.

I made adjustments based on your feedback.
Thanks.

ping!
Not sure if you got a notification when I made latest commit. No rush, just checking.

Yeah, sorry about that. Classes are finishing up for me and I got a bit distracted.
Looks good to me!


Issues on a video loader for future hand prediction

Hi,
I've tried to implement baseline code for future hand prediction using this command,
 python tools/run_net.py --cfg /path/to/ego4d-hand_ant/configs/Ego4D/I3D_8x8_R50.yaml OUTPUT_DIR /path/to/ego4d-hand_ant/output/

Before running, I renamed 'fho_hands_train.json' to 'train.json', because of these lines.
https://github.com/EGO4D/forecasting/blob/c232d5f2e36dde029b37f6a3e17b992166fa39b7/Ego4D-Future-Hand-Prediction/slowfast/datasets/ego4dhand.py#L82-L84
However, it faces errors, and I think it is because the structure of annotations files doesn't match with the baseline code.
For example,
https://github.com/EGO4D/forecasting/blob/c232d5f2e36dde029b37f6a3e17b992166fa39b7/Ego4D-Future-Hand-Prediction/slowfast/datasets/ego4dhand.py#L97 clip_id and hand_dicts returns 'version' and '1' respectively.
If I am missing something, please let me know.
Thanks.

I have the same problem and I did the same procedure. Have you solved somehow? thanks!

I haven't solved my issues. I'm just waiting for help.

Thank you both for opening and commenting on this issue and apologies if there has been no reply yet. Maybe @VJWQ can help?

Hi @sunilhoho @Marchetz we've fixed our dataloader, no need to rename the test file anymore, please update both the code base and the fho_hand_*.json files to run the code!


Use std::move instead of scoped_ptr::Pass().

Follow-up to 44b2e8a ("Add a popup window to exit full screen mode"). In
the upcoming Chromium M49 it is not possible to use
scoped_ptr::Pass(). Prepare beforehand by using std::move() instead.

lgtm


:arrow_up: CodeQL now uses clang++-13

Description
It seems that clang++-11 is deprecated, leaving the CodeQL analyse cpp CI to fail.
Checklist:

[x] The pull request only contains commits that are related to it.
[x] I have added appropriate tests and documentation.
[ ] I have added a changelog entry.
[x] I have created/adjusted the Python bindings for any new or updated functionality.
[ ] I have made sure that all CI jobs on GitHub pass.
[x] The pull request introduces no new warnings and follows the project's style guidelines.

@wlambooy, you can find all the information about the Github runners here: https://github.com/actions/runner-images. There you can see that Clang: 16.0.6, 17.0.6, 18.1.3 is supported for ubuntu-latest.


Updating the versions for operator 0.0.3 release

Signed-off-by: Vivek Lohiya<EMAIL_ADDRESS>Description: Updating the versions for operator 0.0.3 release

/azp run

/azp run


Updated ClassIcons for Destro Warlock

Hey haste,
I made some mods to classicons.lua so the same soul-shards icon system works for destruction lock burning embers.
You can see the edits here:
https://github.com/PProvost/ouf_quaiche/commit/624d52223309bee06397a756e48060c8da214b7d
If you'd prefer I can do a clone and pull request, but given the elegance of your lua code I suspect you'll have a neater way to do it than I did.
Regards,
PProvost

No longer needed in Legion, see #260.

On one side Legion won't be out until Q2/Q3 next year, so closing the issue is kinda stupid, but on the other hand... We're still not supporting it after so many years. :P

Was adding the note mostly to link the two issues together so it won't be forgotten once Legion came :)


Definie a common test suite of supported documents

Define a test suite of documents with a specification of how they should be interpreted.

Test Suite is now in /test_suite


Playing MIDI but no sound

Hi all,
I tried playing MIDI on an Apache Cordova app. I used to be able to play, but now it's just plain silent. The player keeps listening, but no sound at all, even if I already set the volume to 127.
MIDI.loadPlugin({
    soundfontUrl: 'midi/soundfont/',
    instruments: 'acoustic_grand_piano',
    onsuccess: function () {
        pluginLoaded = true;
        var player = MIDI.Player;
        player.loadFile("midi/files/h" + number + ".mid", function () {
            MIDI.AudioTag.setVolume(0, 127);
            MIDI.AudioTag.setVolume(1, 127);
            for (var i = 0; i < 2; i++) {
                MIDI.channels[i].mute = !document.getElementById('chkChannel' + (i + 1)).checked;
            }
            player.start();
            $('#midiPlayPause > i').toggleClass('mdi-av-play-arrow mdi-av-pause');
            loaded = true;
        }, null, function (error) {
            createToast('midi file does not exist.');
        });
        player.addListener(function (data) {
            var now = data.now;
            var end = data.end;
            var channel = data.channel;
            var note = data.note;
            var velocity = data.velocity;
            if (now + 1 > end) {
                loaded = false;
                $('#midiPlayPause > i').toggleClass('mdi-av-play-arrow mdi-av-pause');
            }
        });
    }
});

What am I missing here? Thanks

Try different channel number

Hi!, same problem...
anyone solved it?

I'm encountering the same problem. Any follow up?

In my case, I fix the issue by "require" (I'm using webpack) the acoustic_grand_piano-mp3.js from soundfont

@shrekshao what do you mean by "require"?

include the script acoustic_grand_piano-mp3.js. If you are using <script> tag in html, I think what you will do is to add <script src='acoustic_grand_piano-mp3.js'></script> in your html

Yes, I just encounter simular issue today and finally figure out another way instead of pre-compile the soundfont into code.
When you use webpack, the MIDI object is not registered to global window (browser case). So the solution is very simple:
import * as MIDI from "midicube";
...
window.MIDI = MIDI; // Register to global first

MIDI.loadPlugin({
...
});

perfect @jeffcaiz, it works!
guys don't forget to declare the player like this:
const player = new MIDI.Player();


Multiselect validation fixes

Description
Fixes validation issues found in the new multiselect widget.
Fixes #6126
Fixes: https://github.com/appsmithorg/appsmith/pull/5228#issuecomment-886544183
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Unit tests and manual
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/multiselect-validations

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
51.99 (0.16)
33.83 (0.14)
29.89 (0.15)
52.64 (0.17)

:sparkles: :new:
app/client/src/components/designSystems/appsmith/MultiSelectComponent/index.styled.tsx
92.31
100
0
100

:sparkles: :new:
app/client/src/components/designSystems/appsmith/MultiSelectComponent/index.tsx
14.63
0
0
15.79

:green_circle:
app/client/src/components/designSystems/blueprint/DropdownComponent.tsx
45.83 (13.73)
26.67 (4.03)
5.56 (2.33)
45.65 (13.6)

:green_circle:
app/client/src/components/designSystems/blueprint/ModalComponent.tsx
33.33 (2.3)
15.38 (0)
0 (0)
34.48 (2.34)

:red_circle:
app/client/src/icons/WidgetIcons.tsx
63.79 (-0.5)
100 (0)
25 (-0.93)
63.79 (-0.5)

:green_circle:
app/client/src/utils/WidgetPropsUtils.tsx
66.43 (2.73)
58.8 (1.51)
57.95 (4.46)
65.74 (2.81)

:red_circle:
app/client/src/utils/WidgetRegistry.tsx
74.51 (-0.24)
100 (0)
23.53 (-0.71)
74.51 (-0.24)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
50.83 (0.24)
40.25 (0.84)
36.21 (0)
55.04 (0.27)

:green_circle:
app/client/src/utils/migrations/TableWidget.ts
70.45 (4.54)
61.67 (6.67)
81.25 (12.5)
73.95 (5.04)

:green_circle:
app/client/src/widgets/DropdownWidget.tsx
55.81 (23.76)
0 (0)
55.56 (22.23)
60.53 (27.65)

:sparkles: :new:
app/client/src/widgets/MultiSelectWidget.tsx
55.56
0
50
57.14

:green_circle:
app/client/src/workers/validations.ts
81.48 (1.15)
68.48 (0.42)
83.33 (0.98)
82.49 (1.2)

/ok-to-test sha=6092b31


fix: Export useExperiment hook from index

Make useExperiment hook available as an export from the top level package module.

can this be released too ? please


Output explanation?

I just started testing DLIO and the output has me confused. Is there an explanation of the output somewhere?
Thanks!

@laytonjbgmail We don't have it yet. Could you explain which part made you confused?

I'll start with a few questions and then add others if that's OK.

In the output is the primary measure of performance Samples/s? If we know the sample size, we can then convert this to bytes/s (or other units) - correct?

Does the output sum the results from all MPI ranks?

Is the checkpoint I/O included with the epoch output of Samples/s? Or is it separate?

Thank you!
Jeff

Hi Jeff,
We recently updated the report in the mlperf_storage_v0.5.1 branch (will also be merged to the main branch soon). We reported samples per second; and the bytes / second is also estimated by samples/second * record_size. The output in the log sum up the results from all the MPI ranks.
The checkpoint I/O is not included the output of Samples / second.


New version: StateSpaceRoutines v0.3.3

Registering package: StateSpaceRoutines
Repository: https://github.com/FRBNY-DSGE/StateSpaceRoutines.jl
Created by: @chenwilliam77
Version: v0.3.3
Commit: 9715923b08b9398fb978b9d4f7e2927b67058b19
Reviewed by: @chenwilliam77
Reference: https://github.com/FRBNY-DSGE/StateSpaceRoutines.jl/commit/9715923b08b9398fb978b9d4f7e2927b67058b19#commitcomment-40786067

Please delete this pull request. I will fix the problem preventing the auto-merge in another commit (namely, I had tried to register at v0.3.2 earlier, but that commit was unable to auto-merge because I had listed two dependencies in the Project.toml without upper compat bounds)

@chenwilliam77 for the record, after fixing the issues reported in #18158, you had to retrigger registration without bumping the version number


Enforce correct number of data points is returned from query_range in CotsDataModelRange/Aggregation

Testing during open-sourcing showed occasional vector lengths being off-by-1. Eventually we will want to rewrite these in terms of query instead of query_range, but in the meantime it would be good to see if we can guarantee correct number of data, or normalize by expected number.
        | params      | cpuAllocAvg |    cpuCost | len(cpuCoV) | avg(cpuCoV) |   totalCost |
--------+-------------+-------------+------------+-------------+-------------+-------------|
 5m exp | 1m, 0.01667 |  0.05000000 | 0.00009220 |           5 |  0.00110638 | 0.000206120 |
    new | 1m, 0.01667 |  0.05000000 | 0.00009220 |           5 |  0.00110638 | 0.000184503 |
        |             |             |            |             |             |             |
10m exp | 1m, 0.01667 |  0.05000000 | 0.00018440 |          10 |  0.00110638 | 0.000412241 |
    new | 1m, 0.01667 |  0.05000000 | 0.00018440 |          10 |  0.00110638 | 0.000369007 |
        |             |             |            |             |             |             |
30m exp | 1m, 0.01667 |  0.05000000 | 0.00055319 |          30 |  0.00110638 | 0.001236722 |
    new | 1m, 0.01667 |  0.05000000 | 0.00053475 |          29 |  0.00110638 | 0.001070121 |
        |             |             |            |             |             |             |
 1h exp | 1m, 0.01667 |  0.05000000 | 0.00110638 |          60 |  0.00110638 | 0.002473445 |
    new | 1m, 0.01667 |  0.05000000 | 0.00108794 |          59 |  0.00110638 | 0.002177142 |
        |             |             |            |             |             |             |
 2h exp | 1m, 0.01667 |  0.05000000 | 0.00221276 |         120 |  0.00110638 | 0.004946890 |
    new | 1m, 0.01667 |  0.05000000 | 0.00217589 |         118 |  0.00110638 | 0.004354285 |
        |             |             |            |             |             |             |
 4h exp | 1h, 1.00000 |  0.05000000 | 0.00442552 |           4 |  0.00110638 | 0.009893780 |
    new | 1h, 1.00000 |  0.04933333 | 0.00545817 |           5 |  0.00110638 | 0.010922614 |
        |             |             |            |             |             |             |
 1d exp | 1h, 1.00000 |  0.05000000 | 0.02655312 |          24 |  0.00110638 | 0.059362680 |
    new | 1h, 1.00000 |  0.04987500 | 0.02759048 |          25 |  0.00110638 | 0.055199922 |
        |             |             |            |             |             |             |
 2d exp | 3h, 3.00000 |  0.05000000 | 0.05310624 |          16 |  0.00110638 | 0.118725360 |
    new | 3h, 3.00000 |  0.04990285 | 0.05631600 |          17 |  0.00110500 | 0.112718531 |

See https://github.com/kubecost/cost-model/pull/618/files#r543727699

Resolved by https://github.com/kubecost/cost-model/pull/719


v<IP_ADDRESS> - Spurius Lartius

Add out of date version check when running upload or download commands #75.
Allow a prefix for top level directories to be specified for all commands. Useful to store frameworks built with different Swift versions #83 #82. See Supporting multiple Swift Versions for reference.

          4 Warnings

:warning:
Big PR

:warning:
src/Lib.hs#L533 - Found Reduce duplication

1frameworkNameWithFrameworkExtension = appendFrameworkExtensionTo f
2platformBuildDirectory = carthageBuildDirectoryForPlatform platform
3frameworkDirectory
4  = platformBuildDirectory </> frameworkNameWithFrameworkExtension
5

Why Not

1Combine with src/Lib.hs:962:5

:warning:
src/Lib.hs#L533 - Found Reduce duplication

 1frameworkNameWithFrameworkExtension = appendFrameworkExtensionTo f
 2platformBuildDirectory = carthageBuildDirectoryForPlatform platform
 3frameworkDirectory
 4  = platformBuildDirectory </> frameworkNameWithFrameworkExtension
 5dSYMNameWithDSYMExtension
 6  = frameworkNameWithFrameworkExtension ++ ".dSYM"
 7dSYMdirectory
 8  = platformBuildDirectory </> dSYMNameWithDSYMExtension
 9

Why Not

1Combine with src/Lib.hs:580:5

:warning:
src/Utils.hs#L256 - Found Use &&&

1zip
2  (map (repoNameForFrameworkName reverseRomeMap . _frameworkName)
3     frameworkNames)
4  (map _frameworkVersion frameworkNames)

Why Not

1map
2  ((repoNameForFrameworkName reverseRomeMap . _frameworkName)
3     Control.Arrow.&&& _frameworkVersion)
4  frameworkNames

  Generated by :no_entry_sign: Danger


Link to research / researches regarding human computer interaction

Thanks for rolling out concurrent mode docs!
While going through the docs, in particular putting research into production section. There is a mention of research / researches around human-computer interaction. Would it be possible to link to these? Can send a PR if I've the links to research(es).
Thanks!

Nothing that we can link to at the moment since it’s mostly internal to fb. Maybe in the future!


Combined patches - Make GrowthBookSDK gb property optional

This was a late property before which was causing issues because the
UI would load before gb was initialized. This caused the screen to
flash red with a LateInitializationError. Once gb was initialized the
red screen would be replaced with the example app main screen.
Check gb != null before loading UI.
Change properties from being late initialized to empty maps
When the device has no internet connection we are still trying to use
these properties, but they have not been initialized. We can fix this
by initializing them as empty maps to prevent a LateInitializationError.
Then we check .isNotEmpty when setting the model.
Add “features” to GBSDKBuilderApp constructor
Add test that makes sure we get features even when the network
connection fails or we get a bad response.
Change late properties to be optional. When they are late and we get
a failed network response it results in a LateInitializationError and
freezes the app.
If model is null, return out of fetchFeature so we don't reset the
seeded features we added before the request. This way we can still rely
on them within the project logic.

This SO has some good discussion around using late variables:
https://stackoverflow.com/questions/67197902/how-to-check-late-variable-is-initialized-in-dart
I don’t understand this change. Is there a test that proves that it’s fixed? Where are you seeing a red flash?
Could use some help thinking about how to test this. Reproducing requires a DEV_KEY and HOST_URL, but we could fake the response/result in the test.
Here is a screen recording showing the red flash prior to the changes made in this PR:
https://user-images.githubusercontent.com/13595016/232622753-5c3b3248-a605-42c7-ba3d-bc447c7111b5.mp4


Fix spelling/grammar

Thanks - the spelling has been corrected in recently published version. The contributing guideline for the tutorial wasn't clear - now I've written the guidelines here. Also this repository is now obsolete - everything is moved to https://github.com/theimowski/SuaveMusicStore


NATS notification support

Adds support for NATS for publishing of patient records.  Supports NATS clusters with and without TLS. Provides a simple integration test using a NATS cluster without TLS.  Tested with 4.1.1.

Changes made in a new PR.  Closing this one.


[5.5] New Preset option for Bulma css library

I've notice over the last few months that many Laravel developers like the Bulma CSS library. I wanted to include a preset to quickly remove Bootstrap and get Bulma up and running.
Thanks as always guys.

I made this command Macroable so the community could easily distribute packages to do this kind of thing. :+1:

I came up with unofficial way to use the preset instead. Included Foundation and Bulma for now.. Please give it a try.
https://github.com/Laraveldeep/Laravel-Scaffolding-Presets
Feedback welcome.


Production Environment Setup

Overview
Set up a production environment that deploys from a tag.
Acceptance Criteria

[x] create production firebase project
[ ] set up production mongodb
[ ] github action deploys to firebase hosting / functions on creation of a new tag

need client billing details to enable functions

updated client on the setup for production environment. they are considering hosting using their internal infrastructure.


How can I work with VCards.Message?

Afternoon!
Someone could help? What is the purpose of VCards and how can I get work with this?

Apparently sending vCard is no longer working, since Web Whatsapp has changed. See my issue tracking the problem:
[(https://github.com/mukulhase/WebWhatsapp-Wrapper/issues/649)]

Thanks!


Towards using a metagraph

Fixes https://github.com/Deltares/Ribasim/issues/784.

Locallly a test from Expand logic_mapping fails, but not when run in the REPL 🤔

Ha, I messed up here, and reviewed #807 instead. I was wondering why it was so big haha. Though given that, maybe we should just merge #807 when the comments are addressed and close this.

Closing in favor of #807.


Calories

Article paused due to lack of time (and interest).
Next step is force myself to monitor my meals calories for a month when I'll be in France (July). Will use that data to continue the article later.


bug: AlertButton has no "icon" property defined

Bug Report
Ionic version:
5.2.3
Current behavior:
Getting warnings in IDE because "icon" is not a known property.
Please see:
core/src/components/alert/alert-interface.ts
Mentioned in the docs:
https://ionicframework.com/docs/api/action-sheet
Expected behavior:
IDE should be aware of the property so it can be used in an action sheet.
Steps to reproduce:

Attempt to create an action alert with a number of buttons that defined an "icon".

Related code:
Example:
const as = await this.alertCtrl.create({
        header: `Create Item`,
        buttons: [
            { text: `Video`, icon: 'youtube' },  // <-- Problem here!
            { text: `Gallery` },
            { text: `Action` },
            { text: `Cancel`, role: `destructive` },
        ],
      })
      await as.present()

Ionic info:

Ionic:

   Ionic CLI                     : 5.2.3 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.7.0
   @angular-devkit/build-angular : 0.802.0
   @angular-devkit/schematics    : 8.2.0
   @angular/cli                  : 8.2.0
   @ionic/angular-toolkit        : 2.0.0

Capacitor:

   Capacitor CLI   : 1.1.1
   @capacitor/core : 1.1.1

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   NodeJS : v10.16.0<EMAIL_ADDRESS>   npm    : 6.9.0
   OS     : macOS Mojave

Nevermind, closing this issue. I mixed up AlertController and ActionSheetController. 🙄


ERA001 skipping some lines

I have the following code:

def backfill(apps: Any, schema_editor: Any) -> None:
    """
    Create TimelineEvents for everything
    """
    db_alias = schema_editor.connection.alias

    Recipe = apps.get_model("recipeyak", "Recipe")
    TimelineEvent = apps.get_model("recipeyak", "TimelineEvent")

    # user_content_type, _ = TimelineEvent.objects.using(db_alias).get_or_create(
    #     app_label="core", model="user"
    # )
    timeline_events = []
    for recipe in Recipe.objects.using(db_alias).all():
        timeline_events.append(
            TimelineEvent(
                action="created", created_by=None, recipe=recipe, created=recipe.created
            )
        )
        if recipe.archived_at:
            timeline_events.append(
                TimelineEvent(
                    action="archived",
                    created_by=None,
                    recipe=recipe,
                    created=recipe.archived_at,
                )
            )
    TimelineEvent.objects.using(db_alias).bulk_create(timeline_events)

and the auto fix generates the following diff (which is missing a line):
diff --git a/backend/recipeyak/migrations/0089_backfill_timeline.py b/backend/recipeyak/migrations/0089_backfill_timeline.py
index 7c7bf670..4d99ae38 100644
--- a/backend/recipeyak/migrations/0089_backfill_timeline.py
+++ b/backend/recipeyak/migrations/0089_backfill_timeline.py
@@ -15,8 +15,6 @@ def backfill(apps: Any, schema_editor: Any) -> None:
     TimelineEvent = apps.get_model("recipeyak", "TimelineEvent")

     # user_content_type, _ = TimelineEvent.objects.using(db_alias).get_or_create(
-    #     app_label="core", model="user"
-    # )
     timeline_events = []
     for recipe in Recipe.objects.using(db_alias).all():
         timeline_events.append(

Edit: it's not the auto fix, I think the issue is it's not detecting the first line s commented out code

Does look like a bug. I have to look at the rules for detection to understand what’s going on. It might also be worth checking if the originating eradicate library catches those lines, but I can do that when I debug.

eradicate doesn't catch this either.


Zooming out shows map multiple times

Verify latest release

[X] I verified that the issue exists in the latest release

I know how to solve the bug
None
Link to the code that reproduces this issue
https://www.npmjs.com/package/google-maps-react-markers
To Reproduce
Zoom out of the map, you will see map repeating multiple times
Describe the Bug
When we zoom out of the map, it gets repeated
Expected Behavior
Map should not be zoomed out enough to show it multiple times
Which browser are you using? (if relevant)
Google Chrome

@awahab3109 I'm sorry, but I can't reproduce the issue on my end.
You didn't insert a reproduction link, but the link of the repository.


Support for address based array initialization under opaque values mode

part of rdar://problem/30080769
We have to keep (some) address operations on existentials unless we change the way we initialize arrays:
Consider the following code-patern:
%3 = apply %2<Any>(%1) : $@convention(thin) <τ_0_0> (Builtin.Word) -> (@owned Array<τ_0_0>, Builtin.RawPointer)
%5 = tuple_extract %3 : $(Array<Any>, Builtin.RawPointer)
%6 = pointer_to_address %5 : $Builtin.RawPointer to [strict] $*Any
%7 = init_existential_addr %6 : $*Any, $Int
%10 = integer_literal $Builtin.Int2048, 3,
%11 = apply %8(%10, %9) : $@convention(method) (Builtin.Int2048, @thin Int.Type) -> Int
store %11 to [trivial] %7 : $*Int
%13 = apply %0(%4) : $@convention(thin) (@owned Array<Any>) -> ()

Above we are initializing an array of any, we have a pointer to the array’s storage (which is actually a memory address), the init_existential_addr instruction initializes the array’s storage before we store the integer literal ‘3’ into it.
Adding a new init_existential instruction that takes an opaque value does not make any sense in here, we have to keep using the _addr instruction and deal with addresses.
This patch enables the pattern described above in opaque values mode.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


Replace notifications para

Replaced with "Choose all the locations..." for clarifications questions and answers themes edits.

Agreed at standup that we're keeping expenses for labs, so I'm going to merge these changes now.


Tag v0.9.0-RC2 contains v0.9.0-RC1 in version.sbt file.

When build marathon from tag v0.9.0-RC2 result package has v0.9.0-RC1 version.
version.sbt file contains v0.9.0-RC1 https://github.com/mesosphere/marathon/blob/v0.9.0-RC2/version.sbt#L1

Hey @quamilek thanks for reporting this. I moved the tag to the correct commit.


Add coverage for subset operators.

On Main -
>>> set1 = {1, 2, 3, 4, 5}
>>> set2 = {1, 2, 3}
>>> set2 < set1
Traceback (most recent call last):
  File "<stdin>", line 1
    set2 < set1
TypeError: unsupported operand type(s) for <: 'set' and 'set'
>>> set3 = {2, 3, 4, 5}
>>> set1 > set3
Traceback (most recent call last):
  File "<stdin>", line 1
    set1 > set3
TypeError: unsupported operand type(s) for >: 'set' and 'set'

On Branch -
>>> set1 = {1, 2, 3, 4, 5}
>>> set2 = {1, 2, 3}
>>> set2 < set1
True
>>> set3 = {2, 3, 4, 5}
>>> set1 > set3
True

Accordingly issubset(), issuperset() have been modified and all tests are passing.

Seems a bit weird.. I will re-check this later.

Okay. I observed it while running a data structure benchmark, so I'm sure of it's validity.

I don't think the current modification is necessary. However, I think _set.py needs a refactor, instead of a small change like this.

However, I think _set.py needs a refactor, instead of a small change like this.

No problems. If you have anything particular issue/refactor in mind, feel free to let me know.


Add Ethereum address to FUNDING.yml

I'd like to see an Ethereum address added to the custom property of the FUNDING.yml file for GitHub Sponsors. This would allow a canonical address to send funds to. For example, I'm building a project that allows developers to setup one recurring donation that is split between all of their dependencies with an Ethereum address: https://sustainus.io

@lastmjs feel free to create a PR that adds 0x6e013395109d1b7Bd941D9E5F6BAE3ee624E63EE to the FUNDING.yml


Add support for xAuth ...

Services including Twitter & Instapaper have either added support or are using it as their primary authorisation service  and whilst it's not strictly designed for browser based apps it is a decent alternative for obtaining the access token and can also be used by web apps that need access to users accounts when the user is offline etc

Anyone know if this gem: https://github.com/aereal/omniauth-xauth will work with Twitter xAuth in iOS5?


Check request status code for target state

Only transitionate when request state is success. Now request status in response context. Response list removed and always response.data used

Coverage decreased (-0.05%) to 93.684% when pulling 5dd9e4f6c8251c4aa40825c343c4e09fbc091648 on enhacement/request_status into 6dcc0c7ab87f17be788db1c28b8af8c4f61bacbe on master.


Fix typo in convert-group-messages.rst

Summary
I've fixed a typo replacing particpants by participants in source/collaborate/convert-group-messages.rst

Hello @komodin,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.
This is a standard procedure for many open source projects.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

/check-cla


densify does not open

Hey, I would love to use densify and I installed it on my customized Manjaro via the software center. The installation was successful, however everytime I want to open it, the mouse button turns into loading for a few seconds and nothing happens.
When I use the terminal to open densify I get this:

[aaron@aaron-pc ~]$ densify
Traceback (most recent call last):
File "densify", line 462, in
win = MyWindow()
File "densify", line 43, in init
p = subprocess.Popen(disp, stdout=subprocess.PIPE)
File "/usr/lib/python3.8/subprocess.py", line 854, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/lib/python3.8/subprocess.py", line 1702, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'xrandr'

So I am not sure what to do. I can not even run the AppImage that I downloaded from here: https://github.com/hkdb/Densify/releases/download/v0.3.1/Densify-v0.3.1-x86_64.AppImage
And I was also searching about this 'xrandr' thing in the software center. What I found out was that libxrandr 1.5.2-3 is installed. I have no clue about programming, so maybe these things have nothing to do with each other.
Do you have an idea what I could try?

Hi @aarontrom ,
Did a quick search and found this: https://github.com/yancharkin/games_nebula/issues/15
Based on that, maybe try installing xorg-xrandr?
I don't have a machine with Manjaro but I do have an Arch box that runs Densify and it looks like xorg-xandr is installed on my system:
pacman -Ss xorg-xrandr
extra/xorg-xrandr 1.5.1-2 (xorg-apps xorg) [installed]
    Primitive command line interface to RandR extension

In case you ask even though I assume that you know how to since you got this far, the command to install xorg-xrandr is:
sudo pacman -S xorg-xrandr

That's just my best guess not knowing if you are running xorg, wayland, or other details about your system.
Let me know if that helps...

Hi @hkdb
that was it! I just installed xorg-xrandr and now it works :)
Thank you so much for getting back that fast and providing help!

Awesome! Glad it helped! Cheers!


Add missing items to package

Adds missing code for manual rolls.
Fix path for translations so they are added to releases.

I hope this is correct. I couldn't test 100% since I don't understand how I could build a proper release yet. I created a release in my fork and modules.zip contain the previously missing folders (apps and translations).