Add ability to load data in formats other than xarray

Slack discussion 10th Dec 2021
Scott Hosking Today at 10:53 AM
Should the default scivison format for storing images be a numpy array, rather than xarray? Xarray would remain an option, but there is a steeper learning curve to using it (edited)
3 replies
Scott Hosking  23 minutes ago
I'm not convinced, but it might be worth us discussing.
Alejandro Coca  21 minutes ago
there’s indeed a learning curve in xarray according to the preliminary experience with the DSG. In addition, some CV practitioners prefer data in raw format (png, tiff) to load them into custom data loaders…. (edited)
Scott Hosking  16 minutes ago
We could include "format=" in the data loader with options: raw, numpy, xarray,

What you have to do is specify the driver to be correct for the data type you want (see example), but currently scivision will only understand intake drivers, as oppose to other ways of loading the data, since load_dataset essentially just returns intake.open_catalog(path)
So we want load_dataset  to do something different when it sees a non-intake driver in the scivision.yml config

We could have a higher level above driver in the config called something like loader in the config, where driver is only relevant when the loader is "intake"

You can load data to numpy with intake by just using the default driver

Could we make it so scivision selects a relevant driver/data-loader when none is specified by the config?

load_dataset(path, loader='numpy') <- is something like this a more useful way of specifying the loader/datatype from a user perspective?

We want scivision to recognise when a user tries to run a model that handles a certain data-type on a dataset of a certain data type that they match and to throw an error or suggest what to do if they don't

The main thing here might just be to have clear documentation on how to select the driver or set up the intake catalog (scivision.yml config), so the user gets the data format they want


Map with FnMut(&Self::Item) -> Result<B, Self::Error>

Hello,
I am trying to integrate FallibleStreamingIterator into rusqlite.
And I am looking for a function like map function that handles error when the map function is called.
Something likes this:
    fn advance(&mut self) -> Result<(), I::Error> {
        if let Some(value) = self.it.next()? {
            self.value = Some((self.f)(value)?);
        } else {
            self.value = None;
        }
        Ok(())
    }

So that next() returns Result<Option<&B>, I::Error> (and not Result<Option<Result<...>>, ...>).
Is it feasible/relevant ?
Thanks.

Sure, a try_map method seems reasonable.


Support for nodejs

Can you add support for nodejs.

Can you elaborate on your idea?

Add support for render graphs via server.

not planned right now... but PR are always welcome ;-)

Thanks!

Looks like you can do this using https://github.com/francoislaberge/diagrams


Warn or error when update handlers dangle across CAN or workflow exit

We can recommend the await-all-handlers or drop-all-handlers methods. TBD whether error or warning.

I've merged the first PR, for Python: https://github.com/temporalio/sdk-python/pull/556
The basic spec that that PR's proposing for other SDKs is

Workflow code can wait on a method named similarly to workflow.all_handlers_finished()

By default, the worker will emit workflow.UnfinishedUpdateHandlersWarning and workflow.UnfinishedSignalHandlersWarning if any handlers of those types are unfinished when the workflow completes (cancellation, failure, success). See PR for wording of message.

These warnings are controlled by a policy whose default value is workflow.HandlerUnfinishedPolicy.WARN_AND_ABANDON

Users can silence the warning on a per-handler basis by setting unfinished_policy to workflow.HandlerUnfinishedPolicy.ABANDON

Tests should verify the above for worker-side workflow exit via cancellation, failure, and success


#80 - Moved predicate component groupIds to be derived from the resou…

…rce path
The groupId for a predicate component is now the Abs value of the resource path's hash-code (same as AEM Core Form components [1]). This does have the chance of not being unique, though highly unlikely.
Also updated the PagePredicate to always use negative group id's to avoid collisions.
[1] https://github.com/Adobe-Marketing-Cloud/aem-core-wcm-components/blob/master/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/internal/models/v1/form/AbstractFieldImpl.java
/cc @arpithaar @kaushalmall

@kaushalmall @arpithaar -  this can have collisions. An alternative is to use this as a fallback, and introduce a PostProcessor that saves a guaranteed to be unique id (since the PostProcessor would walk resources and collect all used id's)


Enstrophy for 2D Navier-Stokes

This PR extends functionalities existing in 3D to 2D.

Hm, the CI MPI test for mac fails when run with coverage, the non-covered version seems to be fine.

Could be related to https://github.com/trixi-framework/Trixi.jl/issues/1590


ML-507 Gaussian Process Regression

Random Fourier Features accelerated Gaussian Process Regressor

@RogerDev  changes are made based on the comments. Please review.

@ghalliday I fixed the typos. Please review.


drv: qm: Add support of user space (v)SVA task resetting

While app run on ACC PF and VF in user space including
VM (vSVA) user space, they can be notified as RAS resetting.
Afer that, app can request queue end release the old queue, and
do the task on the new queue again.
Signed-off-by: Kai Ye<EMAIL_ADDRESS>

abandon it

abandon it


build: replace codecov with python-coverage-comment-action

Description:
This PR replaces the codecov action for  python-coverage-comment-action, continuing with the codecov deprecation efforts and avoiding the coverage upload issues altogether.
These changes were 1stly presented in this PR: https://github.com/openedx/openedx-events/pull/300
ISSUE:
https://github.com/edx/edx-arch-experiments/issues/528
Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPI after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Thanks for the pull request, @mariajgrimaldi! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

Hello @mariajgrimaldi. I'm not seeing how this addresses Ned's concern detailed here: https://github.com/edx/edx-arch-experiments/issues/528#issuecomment-1889740696. I never went very deep with this, so I'm not sure if this just requires testing from a personal fork, or other?

@robrap: Sorry, I totally forgot. I'll research further :). Thanks!

@feanil: This PR should have had a waiting-on-author label, which I’ve added. I’m guessing you may have gone through and approved blocked PRs? I wonder if we should double check which of those were actually blocked on the author?

Ah, thanks for the heads up, yea I missed the details of the comment about external PR coverage checks.  I think it's accurate that this needs more work to be able to handle external contributions though it may not be possible. I wonder if we need to just run coverage as a check and fail on violations rather than attempt to post a comment.

Hi folks. I think I did it: https://github.com/mariajgrimaldi/openedx-events/pull/1

It seems to be working: https://github.com/mariajgrimaldi/openedx-events/pull/2. I already added those commits to this PR for you to review.

Hi @mariajgrimaldi! Just checking in on this to see if it's still in progress?

@mphilbrick211: thanks for the ping! I don't know the current status of this, so I'll convert it to draft for now.


Please clarify which Octopus Energy this can be used with

Hi - Since it appears that Octopus Energy operates in multiple countries, each with different web infrastructure/reporting capabilities, it might be helpful in the documentation to indicate which locals are supported.  I presume you are focused on the UK location, "https://octopus.energy/", and not (at least currently) on the US location, "https://octopusenergy.com/".  Thanks!

Hi @dfludwig. You are correct that this integration has been tested against the UK version of Octopus Energy. I couldn't find any documentation for a US version of the API, so I'm not sure if it uses the same one or a different one.


Different characteristics handles in other versions

Great work!, but I had to modify it to get it to work with my Calima fan:
ModelNumber: 10
HardwareRevision: 01.00
FirmwareRevision: 01.02
SoftwareRevision: 01.05
It has a different set of handles:
[0x18] 4cad343a-209a-40b7-b911-4d9b3df569b2 (WRITE )
[0x1a] d1ae6b70-ee12-4f6d-b166-d2063dcaffe1 (READ ) = (1) 01
[0x1c] b85fa07a-9382-4838-871c-81d045dcc2ff (READ WRITE ) = (20)<PHONE_NUMBER>2070c3a52062616465740000000000
[0x1e] 638ff62c-3823-4e0f-8179-1695c46ee8af (READ ) = (6) 582bdb0111dd
[0x21] 25a824ad-3021-4de9-9f2f-60cf8d17bded (READ ) = (1) 00
[0x23] 528b80e8-c47a-4c0a-bdf1-916a7748f412 (READ ) = (12) 02006e002f00000000000000
[0x26] 63b04af9-24c0-4e5d-a69c-94eb9c5707b4 (READ ) = (1) 01
[0x28] 90cabcd1-bcda-4167-85d8-16dcd8ab6a6b (READ WRITE ) = (1) 01
[0x2a] 1488a757-35bc-4ec8-9a6b-9ecf1502778e (READ WRITE ) = (6) 68088b060000
[0x2c] e782e131-6ce1-4191-a8db-f4304d7610f1 (READ WRITE ) = (4) 01030103
[0x2e] 49c616de-02b1-4b67-b237-90f66793a6f2 (READ WRITE ) = (2) 0a0a
[0x30] a22eae12-dba8-49f3-9c69-1721dcff1d96 (READ WRITE ) = (5) 19ca081400
[0x32] 118c949c-28c8-4139-b0b3-36657fd055a9 (READ WRITE ) = (5)<PHONE_NUMBER>
[0x34] 8b850c04-dc18-44d2-9501-7662d65ba36e (READ WRITE ) = (2) 0000
[0x36] f508408a-508b-41c6-aa57-61d1fd0d5c39 (READ WRITE ) = (1) 01
[0x38] 6dec478e-ae0b-4186-9d82-13dda03c0682 (READ WRITE ) = (4) 06160c2a
[0x3a] b5836b55-57bd-433e-8480-46e4993c5ac0 (READ WRITE ) = (5)<PHONE_NUMBER>
[0x3c] faa49e09-a79c-4725-b197-bdc57c67dc32 (READ WRITE ) = (2) 0101
[0x3e] ff5f7c4f-2606-4c69-b360-15aaea58ad5f (READ WRITE ) = (1) 00
When changing some handles in the code everything works fine.

I had similar issue, I had to change from 0x21 to 0x23 or temp/humidity. I did not test the uuid branch as it was several commits behind. One other thing that confuses me is that the reported humidity is between 14-18 which does not sound reasonable. Do you know how this should be interpreted? I have another sensor reporting almost 70%

I believe the uuid branch should work for all versions since the UUID seems to be static over version. For example the uuid which begins with 528b80e8 matches handle 0x23 for you and in the uuid branch it's read out as state. Which seems correct. I've updated the uuid branch but since I'm not near my Calima fan I can't test it out at the moment. Feel free to try it.
Regarding the humidity, the value is currently just read out an assumed to be a percentage. It's possible that it's something other. If you have another humidity sensor close by, you can try to create two or three values far apart and their matching readout from this script. That should simplify the deciphering. Take a shower and measure before and after!

Extracted characteristics from the Android app and used the UUID's instead of handles. This should eliminate problems with different versions.


Add support for speaker labels

Technically, it is possible to generate speaker labels using VOSK - the necessary API and small speaker recognition model exists.
However, the current implementation doesn't support this - I don't yet know how complicated implementing this support would be.

Forgot to mention this issue in the pull request (again), so I'm closing this manually.


Move cron jobs

Codecov Report

Merging #344 (0a20e51) into master (315c8a2) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #344      +/-   ##
==========================================
+ Coverage   65.27%   65.40%   +0.13%
==========================================
  Files          95       94       -1
  Lines        6465     6400      -65
==========================================
- Hits         4220     4186      -34
+ Misses       1648     1620      -28
+ Partials      597      594       -3

Impacted Files
Coverage Δ

plugin/link/httplink/httplink.go
71.42% <0.00%> (+3.57%)
:arrow_up:

Continue to review full report at Codecov.

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


URL toshibamobileservice.azurewebsites.net is not valid

Toshiba change API.
Old address toshibamobileservice.azurewebsites.net has no longer valid DNS record
New address is mobileapi.toshibahomeaccontrols.com
Please update this change in Your code.

this was fixed in #34


Which OpSim

#22 chose the location of the pointing to be one of the deep drilling fields, but not all realizations of the OpSim have the same defined deep drilling fields.  We need to choose one with the field chosen for Twinkles.

I think enigma_1189 would be a good run to use. It is close to the next baseline simulation
https://confluence.lsstcorp.org/display/SIM/OpSim+Datasets+for+Cadence+Workshop+LSST2015

Expanding on what @connolly said...
According to this page, the newest version of OpSim is enigma_1189.  See here for the results of MAF on that run.  It looks like it does indeed contain the chosen DD field:


Create a proxy class to decouple sentry integration validation

Creates a proxy class to decouple sentry integration validation and dramatiq middleware, this is necessary when i need to use middleware without sentry integration to avoid errors when use stub broker and pytest, monkeypatches are not friendly to mock.

Closing this as I did not hear back if my suggestion resolved the issue for you.


Pages are created twice when SSR is false and splitChunks layouts is true

Versions

nuxt: v2.14.5, v2.14.6, v2.14.7
node: v12.x.x

Reproduction
The bug can be reproduced at the following codesandbox:
https://codesandbox.io/s/cool-water-vpncu?file=/nuxt.config.js

Additional Details

nuxt.config.js:
export default {
  ssr: false,

  build: {
    splitChunks: {
      layouts: true
    }
  }
};

Steps to reproduce

Configure nuxt as shown in the details above
Log something in the created lifecycle hook of any page.
Open the console and reload the page

What is Expected?
The page component is created only once and only one message should be logged in the console.
What is actually happening?
The page is created twice and 2 logs are printed
Observations
Not sure if it's of any help but the first time the page is created this.$nuxt.layoutName is an empty string.

same problem,but i don't know the reason,and you let me know,thanks

I have same problem, does anyone know how to fix it?


Cron expression Calculator

creating a calculator which helps you to build cron expressions.

Please tag these issues under the hacktoberfest and assign it to me


fix(build): renamed option key to _option

Description
This PR updates the NgRspackModuleFederationPlugin to standardize the attribute name from options to _options, aligning it with the naming conventions used in other Module Federation plugins. This change resolves the inconsistency and improves compatibility across plugins.
Changes
Renamed options to _options throughout the plugin configuration.
Updated all references to use _options instead of options.
This update should improve interoperability for users who rely on a consistent API structure across Module Federation plugins.
Closes #25

this awesome, should we dm Column?

If we're doing this for consistency can we also update the other Plugins in the build package?
https://github.com/Coly010/ng-rspack-build/blob/main/packages/build/src/lib/plugins/ng-rspack.ts
https://github.com/Coly010/ng-rspack-build/blob/main/packages/build/src/lib/plugins/module-federation/ng-rspack-module-federation-dev-server-plugin.ts


Icons: added magnet and torrent

I chose to add this since there wasn't really an icon for torrent and magnet files and figured that  would be a good fit.
Feel free to merge it or not, I thought it might be nice but if not just close it

TODO

[ ] Use cargo fmt
[ ] Add necessary tests
[ ] Add changelog entry

Hi @Kara-Zor-El thanks for the PR, it would be great for us to add more icons, feel free to open PRs like this.
as for the icon, I would like to discuss some more, you used is

but it seems not that fit the torrent and magnet file usage, how about one of below

especially the nf-mdi-download-network,  it looks like downloading in a link which could mean part of the P2P network.
this also seems to be a good choice

How about this for magnet? (f076)

Hi,
thank you @zwpaper . I have made some changes based on your and @meain 's request

Codecov Report

Merging #629 (2f7d828) into master (93af717) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #629   +/-   ##
=======================================
  Coverage   86.11%   86.12%
=======================================
  Files          37       37
  Lines        3955     3957    +2
=======================================
+ Hits         3406     3408    +2
  Misses        549      549

Impacted Files
Coverage Δ

src/icon.rs
97.68% <100.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


[Cherry-pick] Add "Solve Unmounted Database Problem" doc

Coverage remained the same at 34.07% when pulling 032163235e8c96c098ef237efece55fd04fb5b44 on zhiyuhe/cp/fix_umounted_doc into 53bdcbe82d4b326a6804bb73be5f20a32673e12e on pai-1.6.y.


[Bug]: Memory over flow when add Atem source.

What happened?
Tally arbiter takes all memory and processor resources. If I add source Atem. It doesn't matter if the device is online or not!
Try on my windows machine the same issue.

Version
3.0.4
Distribution
Source
OS
MacOS
What browsers are you seeing the problem on?
Safari
If applicable, What Listener Clients are You Using?
GPI
TallyArbiter configuration
11/9/22 8:47 PM] Tally Arbiter Sources loaded.
[11/9/22 8:47 PM] 0 Sources configured.
[11/9/22 8:47 PM] Tally Arbiter Devices loaded.
[11/9/22 8:47 PM] 1 Devices configured.
[11/9/22 8:47 PM] Tally Arbiter Device Sources loaded.
[11/9/22 8:47 PM] 0 Device Sources configured.
[11/9/22 8:47 PM] Tally Arbiter Device Actions loaded.
[11/9/22 8:47 PM] 0 Device Sources configured.
[11/9/22 8:47 PM] TSL Clients 1 Second Updates are turned off.
[11/9/22 8:47 PM] Tally Arbiter Cloud Destinations loaded.
[11/9/22 8:47 PM] Tally Arbiter Cloud Keys loaded.
[11/9/22 8:47 PM] Initiating 1 TSL Client Connections.
[11/9/22 8:47 PM] Finished TSL Client Connections.
[11/9/22 8:47 PM] Tally Arbiter running on port 4455
[11/9/22 8:48 PM] An error occurred with the connection to <IP_ADDRESS>:80 Error: connect ETIMEDOUT <IP_ADDRESS>:80
[11/9/22 8:56 PM] Source Added: 2ME

Relevant log output
11/9/22 8:47 PM] Tally Arbiter Sources loaded.
[11/9/22 8:47 PM] 0 Sources configured.
[11/9/22 8:47 PM] Tally Arbiter Devices loaded.
[11/9/22 8:47 PM] 1 Devices configured.
[11/9/22 8:47 PM] Tally Arbiter Device Sources loaded.
[11/9/22 8:47 PM] 0 Device Sources configured.
[11/9/22 8:47 PM] Tally Arbiter Device Actions loaded.
[11/9/22 8:47 PM] 0 Device Sources configured.
[11/9/22 8:47 PM] TSL Clients 1 Second Updates are turned off.
[11/9/22 8:47 PM] Tally Arbiter Cloud Destinations loaded.
[11/9/22 8:47 PM] Tally Arbiter Cloud Keys loaded.
[11/9/22 8:47 PM] Initiating 1 TSL Client Connections.
[11/9/22 8:47 PM] Finished TSL Client Connections.
[11/9/22 8:47 PM] Tally Arbiter running on port 4455
[11/9/22 8:48 PM] An error occurred with the connection to <IP_ADDRESS>:80 Error: connect ETIMEDOUT <IP_ADDRESS>:80
[11/9/22 8:56 PM] Source Added: 2ME

Error stacktrace (if applicable)
No response

Was it user error? What was your resolution

Looks like I had a version installed prior to this fix https://github.com/josephdadams/TallyArbiter/pull/501 (a2061ef). Removed everything and reinstalled from source. now everything is stable. I continue testing.


Remove old elasticsearch images & give flexibility with dedicated version images

I'm not sure about this one tbh but since Magento doesn't support only 1 version accross, I think it would be better to at least have a the latest supported versions.
System prerequisites: https://experienceleague.adobe.com/docs/commerce-operations/installation-guide/system-requirements.html
Magento support lifecycle: https://experienceleague.adobe.com/docs/commerce-operations/release/lifecycle-policy.html

Hello! 👋
You also have to update two more things: the documentation for these images and the mapping in the application.

Thanks for the guidance, I updated the PR and created one for the application: https://github.com/origamiphp/source/pull/400


Unwind doesn't remove notifications from run_context

This becomes a problem if you unwind a resource and redefine it. I wrote the original unwind (https://github.com/bryanwb/chef-rewind/pull/12) and didn't realize that notifications weren't removed (I use Chef 11.18.6). I will try to replicate this problem with some tests tomorrow and then submit a pull request.

@thommay: how do you run tests locally? I figured that I need to create the gem and put it under pkg/ before running test-kitchen, but it took me a while to figure this out. Can you please shed some lights or add the steps to the README?


Git uri handler clone should preserve branch

Invoke vscode-insiders://git.clone?https://github.com/microsoft/vscode/tree/release/1.70
Select a directory to clone your repo into
Open cloned repo
:bug: branch context is not preserved

This impacts Continue Working On--branch context is preserved for Clone in Container and Create New Codespace.
I propose adding a new optional query parameter to the uri handler here:
https://github.com/microsoft/vscode/blob/6cc52565f2ca557a3b0919aee29ac259be02c02f/extensions/git/src/protocolHandler.ts#L26-L33
and passing that to the git.clone command as an optional argument here
https://github.com/microsoft/vscode/blob/6cc52565f2ca557a3b0919aee29ac259be02c02f/extensions/git/src/protocolHandler.ts#L59
Then after cloning here:
https://github.com/microsoft/vscode/blob/6cc52565f2ca557a3b0919aee29ac259be02c02f/extensions/git/src/commands.ts#L517-L521
and before opening the repo in a new window, we would perform the ref checkout:
if (ref) {
    const repository = this.model.getRepository(Uri.file(repositoryPath));
    await repository?.checkout(ref);
}

@joyceerhl, will you create a test plan item/verification steps for this feature?

@lszomoru yes I will!


[Discuss] FlexBox Vs Grid

Flex box layout :
Was designed for layout in one dimension - either a row or a column
browser support

pros

Flex can be arranged in any direction
Flex can have its visual order reversed or rearranged.
Items can be aligned in your container or between them.
Support all browsers.

Grid grid layout :
Similar to flex being divided into flex containers and flex items, grid is also divided into grid containers and grid items.
browser support

pros

Grid tracks are created within your stylesheet.
Reduced file sizes.
Prototyping with CSS Grid is fast and efficient.

Given our userbase, I wouldnt worry too much about browser compatibility. We dont even actively test on older browsers and we already do use Grid in the application. Both Grid and FlexBox layout styles coexist together because they serve similar but different use cases. I think a grid vs flex discussion without context is useless because the answer will always be "it depends". Is there a specific problem we are trying to solve for here that needs a discussion between the two layout styles?

This actually leads me to think that the conversation we should be having is "What are the browsers we will choose to support"? and more importantly, "What are the browsers that we explicitly choose not to support" -- once we establish that, then we can choose the right implementation strategy for individual components. What that means is that some compositions may be better suited for a grid layout, and some might be better off with flex

@kgcreative should we re-propose this issue to discuss CSS modernization?

@kgcreative @kavilla  Yeah, thats a better question. But in either case,  Flex box and Grid solve different layout problems, so choosing one over the other is being overly simplistic. Also even if we decide to be compatible with very old browsers, there are still very good reasons to use modern CSS features with useable (but not perfect) fallbacks for older browsers and not worry too much about browser compatibility as much like this.

@ashwin-pc -- My philosophy is "Let's use modern features, and accept graceful fall-backs" -- I personally would rather not rely too much on shims or polyfills, since those end up oftentimes introducing, rather than eliminating technical debt. Perhaps that's the primary agreement we should have here, and one that the community could help weighing in.

@kaddy645 status of this?

@kavilla - I think we can close this. This was a good discussion, but I think the outcome is to evaluate which layout scheme makes the most sense for each component individually. Ultimately this will be abstracted away from both users and developers


Update airparrot to 2.7.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 4817bb834598a24dee75fa253c38f745248d0bee. Your contribution is still credited to you.


Modify viewing of incidents

I am wanting to integrate this into a website widget a bit better.
I would like to be able to click on a Service that is disrupted, and for the incident to show in a popup modal.
Any suggestions?

I expect you are using the API currently? I could add additional parameter to the incident api which would be the service id :slightly_smiling_face:

Yes please!

with this, Can you please also add the ability so that I can click on an affected service and it will popup with the incident linked?

@mattfleaydaly I'm not really fan of popups, but I think that would be a possibility. Please create a new issue for the GUI and leave this one for the API part. Thanks. API will definitely be sooner.


[ceskatelevize] ČT currently uses WebVTT subtitles

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
So the extractor should save the subtitle file with correct extension and omit now unnecessary call of _fix_subtitles method.

probably no longer applicable.


Linux download not packaged correctly

The Linux binaries should include all the files in the OUT directory, and not just the binary, because trying to run that doesn't work:
$ ./FootballDock
./FootballDock: error while loading shared libraries: libnode.so: cannot open shared object file: No such file or directory

Thanks for pointing this out, I will try to fix it up in the next release which I am working on.


Cannot read property 'split' of undefined

What version of WebTorrent Desktop? (See the 'About WebTorrent' menu)
0.17.2
What operating system and version?
Yosemite
What did you do?
Tried to open the app.
What did you expect to happen?
App should start.
What actually happened?
Black screen. Contents of devtools:

torrent-summary.js:55 Uncaught TypeError: Cannot read property 'split' of undefined

This isn't good. @zeke Does this happen every time you open the app?
Maybe there was a bug in a schema migration and your data file isn't in good shape now? To get things working again, you can run: rm ~/Library/Application\ Support/WebTorrent/config.json

Does this happen every time you open the app?

Yep, but removing config.json seems to have done the trick. Thanks!

..or should this issue remain open?

I'll re-open if there are other reports like this :)


Support a clustered redis

works on #1183 -- probably need a developer to vet this a bit...

OK, testing this in a dev environment, and things seem to be working mostly as expected.
Given the time crunch, I don't know if this is a now thing, or a later thing... but right now this config ONLY supports a clustered redis with a failover sentinel. It would be nice for us to maybe handle the use case where someone can easily drop in a single redis node instead. Could also lob that into the project to fix how we handle not having a redis in #1208


Automated cherry pick of #3589: split brands capabilities

Cherry pick of #3589 on release/2.13.
#3589: split brands capabilities

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@5dcb8fb). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3627   +/-   ##
==============================================
  Coverage                ?   6.97%
==============================================
  Files                   ?     499
  Lines                   ?   83667
  Branches                ?       0
==============================================
  Hits                    ?    5832
  Misses                  ?   77337
  Partials                ?     498

Flag
Coverage Δ

#aFlag
6.97% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudprovider/cloudprovider.go
1.68% <0%> (ø)

pkg/compute/models/capabilities.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Getting 401 using the library

I hope you still maintain the library but a simple implementation is not working for me
var req = new digestAuthRequest('GET', url, 'tester', 'tester');
is giving back 401 while the same in cURL is working
i dont know if its related but in cURL im passing the credencials like this 'tester:tester'

I used this on a client project that had a server using digest - that was over a year ago and I don't have access to test with it anymore. But it worked great then and it's worked for lots of people since.
You can at least do some basic debugging. Did you turn logging on? Did you get the digest headers back? If so, what were they?

I just updated the script to log how it parses the returned headers, and as well I fixed a parsing issue. So please pull the latest version, and try again. Logging is turned on out of the box, so you shouldn't have to type any console.logs. You can turn it off by setting loggingOn to false.


Poor coverage of hotspot by openjdk tests

On jdk11+ many categories of hotspot tests are either not ran at all, or just in very limited subset. (runtime, gc, servicability, ...) Current set does not even cover tier1 used in upstream GHA. (Situation on 8 is better, but even there tier1 is :jdk hotspot group, not just :jre)
Hotspot tests ran in different levels:
extended.openjdk:

target
jdk
tests

hotspot_compiler
8, 9
:hotspot_compiler

hotspot_gc
8, 9
:hotspot_gc

hotspot_runtime
8, 9
:hotspot_runtime

hotspot_serviceability
8, 9
:hotspot_serviceability

hotspot_jre
8, 9
:jre

hotspot_serviceability_jvmti
19+
serviceability/jvmti

serviceability_jvmti_j9
19+ (ibm/j9)
serviceability/jvmti

jvm_compiler
11+
:hotspot_compiler

jvm_native_sanity
11+
:hotspot_native_sanity

runtime_nestmate
11+
runtime/Nestmates

ASGCTBaseTest
16+ (ibm/j9)
serviceability/AsyncGetCallTrace/MyPackage/ASGCTBaseTest.java

dev.openjdk:

target
jdk
tests

hotspot_container
11+
:hotspot_containers

hotspot_container_jdk8
8
runtime/containers

special.openjdk:

target
jdk
tests

hotspot_custom
all
only runs default gc/stress/gclocker/TestExcessGCLockerCollections.java

@zzambers as mentioned in slack I have assigned the issue to you.  Please feel free to update the project status according to your work plan. Initially I've set as Q2 iteration 6 (June 11 - June 30).


Chrome Custom Scrollbar

Thanks for the PR @adalenv ! We are currently working up slowly a migration of our infrastructure and maintainership so I'd kindly ask for your patience until we are able to review this.


Value of statementcat in Econtent statements

Currently in the econtent recipe statements (accessed service and viewed econtent)  the spec and JSON templates has the value of optional property context.extensions.statementCat set to the string "Library". This may be because I had always seen the econtent statements as part of the jisc library recipe. However, in the documentation it looks like the econtent statements are part of a seperate recipe.
The context.extensions.statementcat was put in by Jisc as an option context extension for all statements for querying purposes. So the question is:
What should the value of statementcat in Econtent statements be?
Are they part of the library category, or should there be a seperate elearning category?
But I think it raises a wider question I was trying to get at on tuesdays call, I had tried to raise at the meeting a conversation around 'whats included in a recipe'. The general feeling was that reading list was different from library. Is this because they come from a seperate datastream? Is that what defines a recipe?
Another question: is a category different to a recipe?

sitting on the sofa of pontification....
I think "library" is not a helpful category, and that we should consider recipes to be aligned to online services; "library" feels like a term for an organisational unit which may operate several services, including (historically, but not necessarily) reading lists. I feel a recipe should be a good match to "whats in the database for {system X}".
So: "library loans" might be a service, so might "econtent", and "reading list", and all have their own recipes, but re-using several statement templates.

While on the softa:

In terms of the layout of the repository:

I think thats where we are heading currently, we are getting to a point where the statements in each seem like they are repeats of the same statement templates (viewed econtent, viewed vle, viewed video). Perhaps at this stage the repetition doesn't matter because we do have a link on each to the  'generic template' and the examples in the recipes are relevent to that recipe.

In terms of what is a helpful category:

Jisc requested the statementcat element, is that the same as the recipe? Or do the analysts have a different purpose for it?
Off the softa on to the treadmill of procedure:

Are the current statementcats/recipe headings ok
Adam has raised that library as a statementcat is actually not very useful, I don't really know what they are used for and what the implications of changing them now are, but expect we need to change econtent statements from library.

I don't believe this is being used by us anywhere but will need to investigate further before I can confirm.  I think it could be a helpful way of categorising groups of recipes from a documentation perspective but also for us to more easily group activities when looking at the data (without having to map all the elements to determine a grouping).
Somewhat related, I'm not sure why the e-content stuff was separated from library borrowed as these exist in the same category in my mind (most digital resources that we would want to track activity from will be maintained by library systems teams).

so for us to more easily group activities when looking at the data (without having to map all the elements to determine a grouping).

I think this was why michael added it.

Somewhat related, I'm not sure why the e-content stuff was separated from library borrowed as these exist in the same category in my mind (most digital resources that we would want to track activity from will be maintained by library systems teams).

They are only seperated by different headings in the readme, sugguesting they are different recipes. They currently have the same statementcat, and are in the library subdirectory

I think this is the important question (just for clarity): "Jisc requested the statementcat element, is that the same as the recipe? Or do the analysts have a different purpose for it?"
I suggest that we should all be clear that a recipe is a bunch of statement templates within some form of cognate group; for example, the "VLE" recipe has a bunch of statement templates that are pertinent to VLE activity data; "video" refers to statement templates for video activity data. If needed, we could arrange a constant within each statement that states the recipe that the statement template is in.
On the other hand, if statementCat is an attempt to cut the statements along a different axis, then the different name ("statementCat" rather than "recipe") makes sense. Again, if needed, we could implement that in a similar way to the previous paragraph.

statementCat is not being used in any way currently but the intention was/is for it to be used to group the statements in line with how they are presented in the documentation e.g. VLE, Library etc..
Does it really matter that we call this statementCat as opposed to recipe?
The original question was:
What should the value of statementcat in Econtent statements be?
As most digital resources such as e-journal access, ebooks and other content are maintained by library system teams it makes sense in my mind to keep them here, either as 'Library' or better as 'Library Services' perhaps, as per Adam's comment.

For call:
statementcat was added by Michael in issue #225 to "to make querying easier." Originally it fell into one of three valid values:  Attendance, VLE, Library
Econtent statements are the only one with a different value in there other than then the recipe name.  I think my thinking was that library systems house the content:

VLE Recipes have cat: "VLE"
Econtent have cat "Library"
Library has cat "Library"
Readinglist is "Readinglist"
Intervention is "Note" (We did decide that we wanted the first stab of intervention statements to have a more 'generic' name, but keep intervention as the heading in the documentation)
Attendance is "Attendance"
Video is "Video"
Setting amd reaching targets don't have JSON templates

So:

should econtent stay as library,
Are the other others ok, if so Readinglist, Note and Video need adding to the vocab

From call:

depricate this and replace with something along the lines of recipeCat:
Add what we mean by 'a recipe' to documentation

For call April 1st:

statementCat has been deprecated in 1.0.3
recipeCat has been minted, updated documentation, templates & examples in 1.0.3


TypeError: load() got an unexpected keyword argument 'setting_kwargs'

Hi @nicklashansen, when I ran the following command
python3 src/train.py \
  --algorithm svea \
  --seed 0

I got the following error -
 File "src/train.py", line 150, in <module>
    main(args)
  File "src/train.py", line 50, in main
    mode='train'
  File "/home/tejas/github/dmcontrol-generalization-benchmark/src/env/wrappers.py", line 48, in make_env
    background_dataset_paths=paths
  File "/home/tejas/github/dmcontrol-generalization-benchmark/src/env/dmc2gym/dmc2gym/__init__.py", line 64, in make
    return gym.make(env_id)
  File "/home/tejas/anaconda3/envs/dmcgb/lib/python3.7/site-packages/gym/envs/registration.py", line 235, in make
    return registry.make(id, **kwargs)
  File "/home/tejas/anaconda3/envs/dmcgb/lib/python3.7/site-packages/gym/envs/registration.py", line 129, in make
    env = spec.make(**kwargs)
  File "/home/tejas/anaconda3/envs/dmcgb/lib/python3.7/site-packages/gym/envs/registration.py", line 90, in make
    env = cls(**_kwargs)
  File "/home/tejas/github/dmcontrol-generalization-benchmark/src/env/dmc2gym/dmc2gym/wrappers.py", line 90, in __init__
    setting_kwargs=setting_kwargs
TypeError: load() got an unexpected keyword argument 'setting_kwargs'

I'm using latest version of Mujoco i.e. 2.1.0. Seems to me like this error is in dmc2gym but I'm unable to resolve it.
Thanks!

Nevermind, I didn't install dm_control properly.

I meet the same error,can you tell me how to install dm_control properly?THANKS!!!


Doc patch 1

Summary
These are some clean up of the docs so that it is easier for new users to copy and paste the example to run the code.
P.s. It would be great if there was more information on how to add custom mappings

Merged. Thanks for the PR!
For the custom mappings question, do you mean custom values in a valueset? There is an example of that here, although I'm open to creating something different or more clear if that would help.

Are you looking to create your own ValueSets with the codes you're looking for, or your own FHIR resources with custom codes or fields?
If it's the former, this tutorial I used at the most recent FHIR DevDays might be helpful. The getting started notebook has an example of loading ValueSets from a bundle of valuesets.

@rbrush This is awesome! Thank you


Fix max zoom level

Overview
Fix max zoom level on map to 30
Checklist
~- [ ] Styleguide updated, if necessary~
~- [ ] Swagger specification updated, if necessary~
~- [ ] Symlinks from new migrations present or corrected for any new migrations~
Notes
I'm not sure why we were able to zoom past zoom level 19 before, since this fixed it.
Testing Instructions

Verify that you can zoom past zoom level 19 to zoom level 30

Closes #1856

This actually doesn't fix the problem -- the problem isn't with the basemap, but project tile layers.
This manifests itself when a small, but high resolution area is uploaded and used -- the loading icon on the page stays forever because no tiles are requested. Then, when zoomed out to zoom level 18 tiles are requested.
Zoom Level 18 -- tiles show up:

Zoom Level 19 - no tiles:


Automatic ARM docker image builds

Test m1 builds here
macstadium.com
### Tasks
- [ ] https://github.com/gofractally/psibase/issues/471
- [x] Write images that work for arm builds
- [ ] https://github.com/github/roadmap/issues/528
- [ ] Add arm builds to CI/CD

We are waiting on github to add mac runners in order to automatically build images and store them in ghcr, because qemu emulation is too slow. The github runners kill the build after 6 hours.

Update
The m1 runners were added to github, however, they do not have Docker installed on them.
Furthermore, it is not possible, or at least not trivial, to add Docker to the image when I only have CLI access (since Docker Engine on Mac can only be installed through Docker Desktop which requires GUI interaction).
Therefore, this task is now roadblocked until linux arm64 runners are added, which is scheduled for January 2024.


MLFlow model deplyoments error when deploying PyTorch model from GCS bucket - ModuleNotFoundError: No module named 'models'

Willingness to contribute
The MLflow Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the MLflow code base?

[ ] Yes. I can contribute a fix for this bug independently.
[x] Yes. I would be willing to contribute a fix for this bug with guidance from the MLflow community.
[ ] No. I cannot contribute a bug fix at this time.

System information

Have I written custom code (as opposed to using a stock example script provided in MLflow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 18.04): Debian 4.19.194-3 (2021-07-18) x86_64 GNU/Linux
MLflow installed from (source or binary):  binary
MLflow version (run mlflow --version): 1.19.0
MLflow TorchServe Deployment plugin installed from (source or binary): binary
MLflow TorchServe Deployment plugin version (run mlflow deployments--version): 0.1.0
TorchServe installed from (source or binary): binary
TorchServe version (run torchserve --version): 0.4.2
Python version: 3.9.6
Exact command to reproduce:  mlflow deployments create -t torchserve -m gs://<model_bucket>/models/classnet/48d548cc841d4c2b9a06e975dec88c8e/artifacts/classnet_model --name classnet -C 'MODEL_FILE=models/classnet.py' -C 'HANDLER=model_handler.py' -C 'EXTRA_FILES=transforms.py,artifacts/models/desnse_depth.pt,models/dense_depth.py'

Describe the problem
I have trained a custom PyTorch models for an image classification problem. The model is logged to a Google Cloud Storage bucket. When I try to deploy the model to torchserve I get: ModuleNotFoundError: No module named 'models' error. From what I understand  mlflow.pytorch.log_model() calls torch.save(model) internally. This creates a dependency on the directory structure 18325 .
Code to reproduce issue
I have saved the MLFlow model on GCS bucket using the below script:
mlflow.pytorch.log_model(model, "{}_model".format('livenet'))
The model deployment is using the command below:
 mlflow deployments create -t torchserve -m gs://<model_bucket>/models/classnet/48d548cc841d4c2b9a06e975dec88c8e/artifacts/classnet_model --name classnet -C 'MODEL_FILE=models/classnet.py' -C 'HANDLER=model_handler.py' -C 'EXTRA_FILES=transforms.py,artifacts/models/desnse_depth.pt,models/dense_depth.py'
Other info / logs
2021-08-30 10:13:37,521 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG - /tmp/models/555ed568ad5f4fb4a4ebe1b231e298fb/model.pth
2021-08-30 10:13:37,523 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG - <class 'livenet.LiveNet'>
2021-08-30 10:13:38,338 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG - Backend worker process died.
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG - Traceback (most recent call last):
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/model_service_worker.py", line 183, in <module>
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     worker.run_server()
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/model_service_worker.py", line 155, in run_server
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     self.handle_connection(cl_socket)
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/model_service_worker.py", line 117, in handle_connection
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     service, result, code = self.load_model(msg)
2021-08-30 10:13:38,339 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/model_service_worker.py", line 90, in load_model
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     service = model_loader.load(model_name, model_dir, handler, gpu, batch_size, envelope)
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/model_loader.py", line 110, in load
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     initialize_fn(service.context)
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/torch_handler/vision_handler.py", line 20, in initialize
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     super().initialize(context)
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/torch_handler/base_handler.py", line 69, in initialize
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     self.model = self._load_pickled_model(model_dir, model_file, model_pt_path)
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/ts/torch_handler/base_handler.py", line 133, in _load_pickled_model
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     state_dict = torch.load(model_pt_path, map_location=self.device)
2021-08-30 10:13:38,340 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/torch/serialization.py", line 607, in load
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/torch/serialization.py", line 882, in _load
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     result = unpickler.load()
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -   File "/opt/conda/envs/vkyc/lib/python3.9/site-packages/torch/serialization.py", line 875, in find_class
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG -     return super().find_class(mod_name, name)
2021-08-30 10:13:38,341 [INFO ] W-9000-spoofnet_1.0-stdout MODEL_LOG - ModuleNotFoundError: No module named 'models'

What component(s) does this bug affect?
Components

[x] area/deploy: Main deployment plugin logic
[ ] area/build: Build and test infrastructure for MLflow TorchServe Deployment Plugin
[ ] area/docs: MLflow TorchServe Deployment Plugin documentation pages
[ ] area/examples: Example code

@akasantony
From the information you have shared, the model is saved using mlflow.pytorch.log_model and while loading the model, base handler is trying to load it as state dict.
mlflow.pytorch.log_model uses cloudpickle and saves the entire model structure using torch.save.  To load the model, you can refer this IrisClassification example. In this example, the model is saved using mlflow and loaded using torch.load.
We are about to make another release. there are lot of changes went in after 0.1.0 release. Can you please install the mlflow-torchserve plugin from source -  Reference - https://github.com/mlflow/mlflow-torchserve/blob/master/README.md#installation


test: test with configure-aliyun-credentials-action action

Codecov Report

Merging #813 (3349669) into master (d1ad71c) will not change coverage.
 Consider uploading reports for the commit 37b894d to get more accurate results

@@           Coverage Diff           @@
##           master     #813   +/-   ##
=======================================
  Coverage   83.10%   83.10%
=======================================
  Files          47       47
  Lines        3261     3261
=======================================
  Hits         2710     2710
  Misses        418      418
  Partials      133      133


IIS Deployment Error - Call to Node module failed with error: Error: no such file or directory at MemoryFileSystem.readFileSync

I can run the project (React & Redux template) fine using dotnet run but after running "dotnet publish -c Release" and then hosting the build in IIS I hit the below exception. Anyone had success with this? I've got the latest code, set the IIS AppPool to unmanaged code and given the app pool user full control over the entire directory.
Exception: Call to Node module failed with error: Error: no such file or directory
at MemoryFileSystem.readFileSync (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\memory-fs\lib\MemoryFileSystem.js:114:10)
at C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\aspnet-webpack\LoadViaWebpack.js:70:65
at Compiler.<anonymous> (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:194:14)
at Compiler.emitRecords (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:282:37)
at Compiler.<anonymous> (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:187:11)
at C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:275:11
at Compiler.applyPluginsAsync (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\tapable\lib\Tapable.js:60:69)
at Compiler.afterEmit (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:272:8)
at Compiler.<anonymous> (C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\webpack\lib\Compiler.js:267:14)
at C:\Development\Test Projects\CoreReactTest\bin\Release\netcoreapp1.0\publish\node_modules\async\lib\async.js:52:16

Dug into this a little and it seems in the aspnet-webpack module in LoadViaWebpack.js in the following code the compiler.run() call doesn't return an err object but doesn't raise pass an error via stats and it results in webpack.output.js being missing.
        compiler.run(function (err, stats) {
            reject(stats);
            return;
            if (err) {
                reject(err);
            }
            else {
                // We're in a callback, so need an explicit try/catch to propagate any errors up the promise chain
                try {
                    var fileContent = compiler.outputFileSystem.readFileSync('/webpack-output.js', 'utf8');

The error I'm seeing in the stats variable is below. So far I can't see why it thinks there is a syntax error there.
ERROR in ./ClientApp/boot-server.tsx
Module build failed: SyntaxError: C:/Development/Test Projects/CoreReactTest/bin/Release/netcoreapp1.0/publish/ClientApp/boot-server.tsx: Unexpected token (20:24)
18 | // Build an instance of the application
19 | const store = configureStore();
> 20 | const app = <Provider store={store}>
| ^
21 | </Provider>;

Thanks for reporting this!
I've just published an update to the aspnet-webpack NPM package (new version: 1.0.5) which does a better job of catching Webpack compilation errors and reporting them, so if you update to that, you should see the actual syntax error message in the browser instead of the useless "no such file or directory" message.
As for why you actually have a syntax error, are you saying it works fine when you use dotnet run, but it's an error when you run inside IIS?

Thanks for the update.
Yes works fine using dotnet run but I then ran publish to get a build to deploy to IIS and from within IIS I'm getting the issues. I'm using VS code but I did also try running from within VS2015 and it worked there as well.
I was originally trying to publish to Azure using a webhook to pull from a gitlab repo but was getting issues there so thought I'd simplify it and do a basic deploy to IIS to see what the problem is.

Thanks for the info. It turns out this was because .babelrc was not included in the publishOptions/include config in project.json. After adding this, publishing works fine.
You can fix your own site either by:

Manually copying .babelrc from your source root dir to your publish root dir
Or by modifying project.json like this and republishing (remember to delete your old publish output first, or even better, publish to a separate directory not a child directly as is the default, because having a second copy of your app in a subdirectory will interfere with TypeScript module resolution)
Or by updating your copy of  generator-aspnetcore-spa to the latest (0.2.1) and generating a new project.

Awesome, nice work! I checked the babel node modules and webpack config but overlooked the .babelrc. I'll fix it up manually in my build.
Yeah I saw the Typescript issues with using a sub dir but this was mainly just a quick test to find out the issue so I didn't bother changing the publish target folder. I'm using ES6 instead of typescript in my main project.

@SteveSandersonMS WooOoow, i just used npm install -g aspnet-webpack
and Now everything is working fine !
Thanks


Perkeep's gerrit instance should have CI

Right now, CI runs on master in the form of travis-ci, but only a subset of tests are run (the default devcam test set).
Changes proposed on gerrit do not have tests run automatically, and it's up to the developer and any reviewers to do due diligence.
To reduce the burden on developers and remove some chances of human error, Perkeep should include automated test-running for CLs under review.
For security reasons, they may need to be kicked off by a human (e.g. as the golang trybot works).
If we also include tests that require credentials (e.g. s3 / azure / etc) we'll almost certainly want to make them human-triggered.

Since this issue was filed, perkeep no longer uses Gerrit.  All development is on GitHub, and CI is moving over to GitHub Actions (#1317).
The point about needing a system for manually kicking off a test that requires credentials is still a good point though.  That is possible with GitHub Actions's workflow_dispatch event, which might be an option.  Renaming issue to continue tracking that.


ThinEdge can't connect to tenant

We are using the example from https://github.com/thin-edge/thin-edge.io_examples/tree/main/opcua-solution.

c8y.url is configured without any prefix (http/https)
The certificate is created.
When trying to upload the certificate the following error is shown:

Two weeks before we could connect via ThinEdge to our tenant. At the moment it is not possible.
Thanks in advance for your support.

The versiob of the thin-edge is 0.6.1

I just double checked with the same version on adamos.com and adamos-dev.com with no issues. I have the feeling that there is something within the network not properly configured in terms of DNS. Can you describe your setup a bit more?

We modified the dockerfile to transfer the certificate, the key and the baseUrl via the .env file. Then via a shell script the certificate and key are copied in the respective folder. The script launches tedge disconnect c8y to disconnect any possible bridges to the c8y cloud, set the baseUrl via tedge config set c8y.url ${baseUrl} and finally connects to the cloud via tedge connect c8y. Everything else is the same except some additions to make running the shell script work via openRC in the dockerfile.

Could you do an nslookup ${baseUrl} from inside that container to rule out any networking misconfiguration?

Did you rebuild the docker container after the 21st of May? I just saw that there was a jump in the base image version from 3.15 -> 3.16 about 10 days ago. The Dockerfile uses :latest tag.

@albinsuresh: the nslookup was ok. The IP was the same I got over a Windows ping.
@mbay-ODW: Yes, I already rolled the alpine version back because it was causing the /sbin/init Entrypoint to fail
I did some research myself and apparently we had a certificate problem, which caused this kind of behaviour. After generating new certificates the error message is gone.

@smeus A corrupt certificate "appearing" as DNS lookup error is confusing and we'd like to improve that error reporting from thin-edge if possible. So, could you please share some details of that certificate issue?

@albinsuresh: At first we tried to get the connection working via our own certificate and private key file. This would lead to a not authorized error message. The key file was an encrypted private key, but the header and footer stated just -----BEGIN PRIVATE KEY----- and -----END PRIVATE KEY-----. That was probably the issue. To check if the connection was working normally, we removed the certificate inside the container via tedge cert remove and then generated ( tedge cert create ) a new one. When trying to upload the new test certificate the above mentioned error message was displayed.

@albinsuresh Customer found out that this was based on the fact that he encrypted the certificates via -sha256.
I guess this should be considered somewhere in the documentation that the certificate should not be encrypted. And of course some log message that indicates that something is wrong with the certificate.


CI: Bypass the CI column if there's no integrations for a repo / pull

https://github.com/iFixit/pulldasher/blob/master/views/standard/spec/columns.js#L55
The selector function for the CI Blocked column is a good place to start.

We can't automatically know if there are any integrations, we just know that CI hasn't run for this pull. The commit status is something that's stored per-commit, and its absence doesn't tell you anything about the future (maybe CI hasn't run, maybe CI will never run, ...)
I'd lean toward making this configurable per-repo in the config: ci_required: false or something.

This might be a good reason to have the repos list in config.js be an object instead of a org/repo string.

This was addressed in the short term with https://github.com/iFixit/pulldasher/issues/130 and is probably going to be long term fixed by #102


Metadata and TOC

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

titleMapping.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Acrolinx Scorecards
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard

docs-ref-conceptual/authenticate-azure-cli.md
94
1
link

docs-ref-conceptual/create-an-azure-service-principal-azure-cli.md
89
6
link

docs-ref-conceptual/format-output-azure-cli.md
91
2
link

docs-ref-conceptual/get-started-with-azure-cli.md
88
15
link

docs-ref-conceptual/install-azure-cli.md
88
4
link

docs-ref-conceptual/manage-azure-subscriptions-azure-cli.md
89
1
link

docs-ref-conceptual/overview.md
87
2
link

docs-ref-conceptual/query-azure-cli.md
79
8
link

docs-ref-conceptual/release-notes-azure-cli.md
85
3
link

More info about Acrolinx

The scorecard links are good for one hour. If you're prompted to authenticate, you can close and reopen the PR to get new links. Or, use these credentials:
Account: report-viewer
Password: kermit$
Report false positive results or other issues

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

docs-ref-conceptual/authenticate-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/create-an-azure-service-principal-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/format-output-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/get-started-with-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/install-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/manage-azure-subscriptions-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/overview.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/query-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View

titleMapping.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Completion shows global variable where hover shows a local field of the same name

When a class contains a field that also matches a global, the code completion shows the global version whereas the hover shows the local version:

I think hover is correct and completion may be doing the wrong thing (@bwilkerson?) - however I'm not completely sure because I encountered this while testing what seemed like surprising behaviour in extension classes and unqualified access (see https://twitter.com/anaisbetts/status/1675961549339467776) where the global/top level field is being used over the instance one.

then we look in the enclosing library (including imported names), and finally we look in the supertypes of the class

Ah! I thought this behaviour was specific to extensions so had been looking there for an explanation. The spec clearly describes it where you noted (and gives some good reasoning). Thanks!

Cannot reproduce this issue, it appears to have fixed with the refactor of the completion code.

@keertip my original repro still seems to still show the issue for me (on a bleeding-edge build):

Although what wasn't visible in my screenshot above was that dart:io is already imported (I should've included a full copy/pastable repro!). Here's my complete file:
import 'dart:io';

class Foo {
  /// Local exitCode
  int get exitCode => 1;

  foo() {
    exit
  }
}

@DanTup , thanks! Yes, can reproduce the issue with the above example.

Something more on this. Noticed this was happening only with exitCode and not with other globals from dart:io. Looked at the source code and saw that exitCode has both a getter and a setter defined. If we do the same with the example,
import 'dart:io';

class Foo {
  /// Local exitCode
  int get exitCode => 1;
  void set exitCode(int i) {}

  foo() {
    exitC // complete here
  }
}

then it completes with the local. In the previous example, the global getter was not suggested as it was shadowed by the local, it was the setter for exitCode that was being suggested in the completion.

Yes, that is right, we are using the name of the setter rather than display name.


Added policy to deny resources in specified window.

Description
This policy denies the creation and updating of resources specifically for Deployment and Pod kinds during a specified time window. The policy is designed to enhance control over resource modifications during critical periods, ensuring stability and consistency within the Kubernetes environment.
Checklist

[] I have read the policy contribution guidelines.
[] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

@chipzoller , updated with new schema

Sorry, but Chainsaw tests are failing. Need to investigate and solve.

Resolved the issues in chainsaw test


fix: add specific rewrites for all SSR routes

Summary
Currently (with some exceptions) we only add rewrites for routes that need to use an ODB, and rely on a catch-all to rewrite the rest to the regular handler. However this causes problems when an ODB dynamic route overrides a static SSR route, because it never falls-back to the catch-all. This leads to bugs due to it unexpectedly using an ODB, so having the wrong cache behaviour and missing query data etc.
This PR changes the rewrite handling to generate rewrites for every route, including SSR. These are done starting with the most specific, static routes and then the dynamic routes as defined in the routes manifest. We look these up in the prerender manifest so we can see which need ODB and which SSR handlers.
This does lead to a lot of rewrites, particularly for sites with i18n, because we need to generate separate rewrites for all of them. There may be a way to simplify these rewrites, collapsing some into single rules. However I don't want to attempt this until we have better tests.
I have been working on tests, but have had to abandon my original plan because it's really hard to isolate them and just test the rewrites. It looks like the best bet will be to do a new integration test suite, where we do a full build, then use the plugin to generate a toml and run ntl dev, then use fetch and check the x-render-mode for each page. We could in theory also do this with cypress. I will investigate further and do a follow-up PR
Test plan

Load all the pages in the demo site, checking in dev tools to ensure the JSON works properly and then reload each page to ensure the HTML does too. Then switch to the French locale and try again.

Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal

Fixes #1077, fixes #1068
Standard checks:

[ ] Check the Deploy Preview's Demo site for your PR's functionality
[ ] Add docs when necessary

🧪 Once merged, make sure to update the version if needed and that it was published correctly.

👈😎👈


Fix black screen on Krom

The used Kha version didn't match with the Krom version in armsdk which led to a black screen in Krom, so now the same Kha version as used by the armsdk is used for Armory2D.

Argh, thanks.


Update registration

@noahbass let me know what you think. If we can get this in soon then we can finish the MLH form that Shy sent us and we're almost done


Godot editor 3.0.3 on opensuse tumbleweed linux, crashes after some clicking

Godot version: v3.0.3-stable_x11.64

OS/device including version: openSUSE Tumbleweed Linux

Issue description: Editor starts but after some clicking just crashes

Steps to reproduce: start editor via console (running: ./Godot_v3.0.3-stable_x11.64 in the terminal) one the dialog showing that there are no projects existing select the second option (but clicking abort also works) chose the samples views and click on next page.
Minimal reproduction project: i have none, it is an editor problem

here is some console output:
OpenGL ES 3.0 Renderer: Mesa DRI Intel(R) HD Graphics 630 (Kaby Lake GT2)  ERROR: _png_error_function: Not a PNG file    At: drivers/png/image_loader_png.cpp:62. libpng error: Not a PNG file ERROR: _load_image: PNG Corrupted    At: drivers/png/image_loader_png.cpp:94. ERROR: _load_mem_png: Condition ' err ' is true. returned: Ref<Image>()    At: drivers/png/image_loader_png.cpp:252. ERROR: copy_internals_from: Condition ' p_image.is_null() ' is true.    At: core/image.h:327. ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.    At: modules/jpg/image_loader_jpegd.cpp:125. ERROR: _png_error_function: Not a PNG file    At: drivers/png/image_loader_png.cpp:62. libpng error: Not a PNG file ERROR: _load_image: PNG Corrupted    At: drivers/png/image_loader_png.cpp:94. ERROR: _load_mem_png: Condition ' err ' is true. returned: Ref<Image>()    At: drivers/png/image_loader_png.cpp:252. ERROR: copy_internals_from: Condition ' p_image.is_null() ' is true.    At: core/image.h:327. ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.    At: modules/jpg/image_loader_jpegd.cpp:125. ERROR: _png_error_function: Not a PNG file    At: drivers/png/image_loader_png.cpp:62. libpng error: Not a PNG file ERROR: _load_image: PNG Corrupted    At: drivers/png/image_loader_png.cpp:94. ERROR: _load_mem_png: Condition ' err ' is true. returned: Ref<Image>()    At: drivers/png/image_loader_png.cpp:252. ERROR: copy_internals_from: Condition ' p_image.is_null() ' is true.    At: core/image.h:327. ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.    At: modules/jpg/image_loader_jpegd.cpp:125. handle_crash: Program crashed with signal 11 Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues [1] /lib64/libc.so.6(+0x382d0) [0x7fbdca68f2d0] () [2] ./Godot_v3.0.3-stable_x11.64() [0x9d5231] () [3] ./Godot_v3.0.3-stable_x11.64() [0x9d5c92] () [4] ./Godot_v3.0.3-stable_x11.64() [0xa0287c] () [5] ./Godot_v3.0.3-stable_x11.64() [0xa030e0] () [6] ./Godot_v3.0.3-stable_x11.64() [0xa05648] () [7] ./Godot_v3.0.3-stable_x11.64() [0x9fef1d] () [8] ./Godot_v3.0.3-stable_x11.64() [0x8cc0eb] () [9] ./Godot_v3.0.3-stable_x11.64() [0x198cf54] () [10] ./Godot_v3.0.3-stable_x11.64() [0x1012bc5] () [11] ./Godot_v3.0.3-stable_x11.64() [0x101478d] () [12] ./Godot_v3.0.3-stable_x11.64() [0xbe73a4] () [13] /lib64/libpthread.so.0(+0x7554) [0x7fbdc90f6554] () [14] /lib64/libc.so.6(clone+0x3f) [0x7fbdca751fdf] () -- END OF BACKTRACE -- Abgebrochen (Speicherabzug geschrieben)

Duplicate of #19336


Can't disable the intro skip

Is there a way to not skip the intros? I tried setting it to 0 in the P3RFix.ini file but it still skips to the opening movie

If you set "SkipLogos = false" in P3RFix.ini that should disable the intro skip.

Ohh "false" got it, thanks


move project page to github

github pages is an awesome feature and it will make it easier to update the file in case someone else forks the project and decide to add new animation types, fix errors, improve documentation... (issue #1 and #3 are a good example)

Do these pages have to be static HTML? I'd like to use PHP to allow users to create custom builds etc easily.

gh-pages should be static HTML or jekyll, no php. You would need to call the custom build script from another domain.
I don't think it is that important but was just giving a heads up that other people could fork the page and fix things. You can close the issue if you think it isn't necessary. Cheers.

Thanks for clarifying, I'm pretty new to Github still!
I'll close it for now, but I think it's a good idea, particularly if things like a JS plugin etc are developed soon.


Fix Anchor for installing Gateway API CRD manually on Getting Started Guide

Fixes #1403

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

/lgtm
/approve


investigate what is throttling api/v1 endpoints

loadtest revealed
that api/v1 endpoints are being throttled - perhaps there is some default behaviour of actix-web
 Name                     |                                        Status codes
 ------------------------------------------------------------------------------
 GET /advisories          |                                        33,963 [200]
 GET /api/v1/advisory     |                                           292 [200]
 GET /api/v1/importer     |                                           292 [200]
 GET /api/v1/organization |                                           292 [200]
 GET /api/v1/product      |                                           292 [200]
 GET /api/v1/purl         |                                           292 [200]
 GET /api/v1/purl?q=curl  |                                           292 [200]
 GET /api/v1/sbom         |                                           292 [200]
 GET /api/v1/vulnerabil.. |                                           295 [200]
 GET /importers           |                                        33,964 [200]
 GET /index               |                                        33,968 [200]
 GET /openapi             |                                        33,968 [200]
 GET /packages            |                                        33,963 [200]
 GET /sboms               |                                        33,967 [200]
 POST query advisory wi.. |                                        79,520 [200]
 -------------------------+----------------------------------------------------
 Aggregated               |                                       285,652 [200]

also open question if such throttling should be done at the app layer eg. this could be done by nginx, etc etc.

I guess this might not be throttling but lack of default pagination in rest endpoints ...

I think this turned out to be how goose determines groups of http invokes, closing


Certificate for https://artifacts.duckdb.org/ is outdated

Example, visit https://artifacts.duckdb.org/latest/duckdb-binaries-osx.zip from a Web browser (tried on Chrome)
Raised by users on https://discord.duckdb.org general channel (https://discord.com/channels/909674491309850675/921073327009853451/1287366791810519091).
Opening for visibility, should anyone else bump into this.

This is back working normally, go 🦆.


RTSP Feed instead of using Webcam?

Hello Torin,
Awesome work on this repo. I'm trying to figure out how it would be possible to modify the code to enable RTSP feed input instead of a webcam.
The reason, I'm trying to figure this out is because, I realize that when needing multiple  types of data, faces, landmarks, and so on, we get better results by running multiple MediaPipe bases for marginal performance costs. I'm using NDI webcams instead of Spout right now, because it allows me to share the same feed over 4 MediaPipe bases on the same machine, but I'm limited by the 4 NDI Cam sources.
I believe that with a multicast RTSP it would be possible to use the same source across multiple instances of MediaPipe bases.

Hey @Ganesh-Baron-Aloir glad you're having fun with the plugin! We looked into the RTSP option, but it's a bit of a pain to get working, and requires you to have the licensed version of TouchDesigner to run. I haven't played with it to test, but I also suspect the latency might be quite high.
I'm currently (well, I will be when I next get some time to tinker) looking at capturing a window as the video source, which would (I believe) allow us to have unlimited number of MediaPipe instances all working from the same source material with 0 delay... you can keep track of it here: https://github.com/torinmb/mediapipe-touchdesigner/issues/64

Hey @domisjustanumber I can understand the challenge of integrating RTSP. Good point on the latency, that is indeed a possiblity.
If I get some time I might look at this external python library: pyvirtualcam it might allow to get the data from a TOP of some GLSL wizardry and virtualize a camera.

Thanks for the kink, thats an interesting tool! Dotsimulate has also done some cool stuff with pulling GPU memory directly into Pyhton, but alas MediaPipe is Javascript running in embedded Chromium. As it's a web browesr, it's very heavily walled off from memory and network access.


Request: Update to latest Tabler-Icons (v1.x branch)

@codecalm has released a crazy amount of icons to tabler-icons in the last couple of weeks.
I'm stuck using a Vue2 app in which I would love to use some of those icons too.
Could you do an up-to-date release in the v1.x branch?

done


Fix UG bugs

Bugs fixed: #168 and #192

Codecov Report

Merging #219 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #219   +/-   ##
=========================================
  Coverage     68.67%   68.67%
  Complexity      542      542
=========================================
  Files            91       91
  Lines          1839     1839
  Branches        197      197
=========================================
  Hits           1263     1263
  Misses          502      502
  Partials         74       74

Continue to review full report at Codecov.

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


How to compile scss files

Loving the pattern lab you've created!
How are the scss files meant to be compiling?
I'm used to using a config.rb file in the root of my projects.
I'm running pattern lab locally on my macbook, using the following command to watch the project for changes:
php builder/builder.php -w
Please let me know if I'm missing something - I've read through the docs and couldn't see anything about compiling.

Yes, mightily confused about why my sass/scss didn't get compiled. Using node it shouldn't be hard to add a gulp/grunt task for this by default.


bug(text-field): When the mobile keyboard hides the text-field example, the text is written backwards, cursor positioning problem.

📚 Docs or angular.io bug report
Description
When trying the input fields examples in mobile horizontal direction and the mobile keyboard hides de input that is being tested, the text appears written backwards. This happens because the cursor stays in the start of the string instead of the end.
🔬 Minimal Reproduction
What's the affected URL?**
https://material.angular.io/cdk/text-field/examples
Reproduction Steps**
1 - Open https://material.angular.io/cdk/text-field/examples on mobile
2 - The keyboard must be big enough to hide the input
3 - Write in any of the examples "1234"
Expected vs Actual Behavior**
Expected: The text written is "1234"
Actual: The text written is "4321"
📷Screenshot
The keyboard hiding the input:

The result after typing "1234":

The result after typing "12348" in case the keyboard is not hiding the input:

The result after typing "123" in case the keyboard is not hiding the input:

🌍  Your Environment
Browser info
Chrome for Android: 88.0.4324.93

This seems like a combination of the styling on our docs site which somehow prevents the input from being shown while typing (I'm not exactly sure how this happens, I tried repro-ing with a fixed position div and the input just pushed it out of the way). Combined with what seems like a Chrome bug that causes the input to be interpreted incorrectly when the input isn't shown.
It doesn't seem like a bug with the mat-form-field itself, so I'll move this issue to our docs repo.


Feature: Odd design for large screens

Type of feature
🎨 Style
Current behavior
When you are on large screen, the add to insights button and the table collabse.

a
Suggested solution
It would be great to keep the button visible
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

@a0m0rajab I think the button is not visible because the table on lg screen of the container becomes a bad resolution
I think this issue can have a good-first-issuehttps://github.com/open-sauced/insights/blob/ede678b5624ec3a2406789e0a8dfec0880fd8080/components/organisms/Repositories/repositories.tsx#L97

Can I attend to this issue?

.take


Redis overcomit memory

Noticed this error in my logs.. might be something to add.
redis_1           | 1:M 02 Feb 2023 15:06:09.237 # WARNING Memory overcommit must be enabled! Without it, a background save or replication may fail under low memory condition. Being disabled, it can can also cause failures without low memory condition, see https://github.com/jemalloc/jemalloc/issues/1328. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.

Since these errors seem to be coming from the Redis container, this isn't a core issue per se.
I would be interested to know if this is only present on the Docker appliance. If so, we could probably patch the command to run: sysctl vm.overcommit_memory=1 before starting redis-server.

Seems that since the container wouldn't have the appropriate privileges to set this from within, it must be set on the host:
https://github.com/docker-library/redis/issues/298#issuecomment-972275132

Adding fix to Troubleshooting section in documentation as this is something that's to be performed by the user, not really something we can configure Assemblyline automatically on our side:
https://cybercentrecanada.github.io/assemblyline4_docs/administration/troubleshooting/


Docs/readme update

Add the mention to run asdf install to install the right versions specified in the bundled .tool-versions file.
Remove the mention of JetBrains’ AppCode as the product has been discontinued in December 2022.

🦀 Requesting reviewers for this pull request:

@marcantoinefortier (contributor with 4 commits in the last 90 days and 29 commits overall)


Update to upstream

#156

[ ] Android Studio Works

[ ] Works on Windows

[ ] Works On MACOS

Works on android Studio
Android Studio 3.3.2
Build #AI-182.5<IP_ADDRESS>14842, built on February 15, 2019
JRE: 1.8.0_152-release-1248-b01 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

Windows worky
IntelliJ IDEA 2019.1 (Ultimate Edition)
Build #IU-191.6183.87, built on March 27, 2019
Licensed to Alex Simons
Subscription is active until February 21, 2020
JRE: 1.8.0_181-b13 amd64
JVM: Java HotSpot(TM) 64-Bit Server VM by Oracle Corporation
Windows 10 10.0

and
WebStorm 2019.1
Build #WS-191.6183.63, built on March 21, 2019
Licensed to Alex Simons
Subscription is active until February 21, 2020
JRE: 1.8.0_202-release-1483-b39 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0


Fix post to slack bug when no timestamp

Quick bugfix in #1900. When there isn’t a timestamp in the message, we were trying to find relevant CloudWatch logs and throwing a TypeError. This is a quick patch which means it won’t be thrown again.
In practice this only occurs just after we’ve created new alarms, so it’s quite rare – otherwise I’d write a more sophisticated test.

When an alarm is created with “missing data = breaching”, it immediately fires off an alarm in CloudWatch and triggers our Lambda.
That message doesn’t have a timestamp, but the Lambda tries to work out when it should be getting CloudWatch Logs for… and then it throws a TypeError because None isn't a valid timestamp.
Subsequent alarm notifications usually include values, so this is much less likely to recur after the initial creation.


convert basic statistics pp to use the new data object

Pull Request Description
Currently, four tests of basic statistics are failed.
framework/PostProcessors/BasicStatistics.timeDepOutCsv FAILED (CWD /Users/wangc/projects/wangc/raven METHOD ? Expected files not created /Users/wangc/projects/wangc/raven/tests/framework/PostProcessors/BasicStatistics/timeDepOutCsv/fileOut.csv /Users/wangc/projects/wangc/raven/tests/framework/PostProcessors/BasicStatistics/timeDepOutCsv/1-PlotStatsT2_scatter-scatter-scatter.png)
framework/PostProcessors/BasicStatistics.timeDependentAsyncHistories FAILED (CWD /Users/wangc/projects/wangc/raven METHOD ? Expected files not created /Users/wangc/projects/wangc/raven/tests/framework/PostProcessors/BasicStatistics/basicStatisticsTimeDependentAsynchronousHistories/output_basicStatistics_timedep_PP1.csv)
framework/PostProcessors/BasicStatistics.variationCoefficient FAILED (
Could not find match for row "2" in Gold:
vc_x0         inf
vc_y0    0.011313
vc_z0    0.150734
vc_x     0.176795
vc_y     0.011246
vc_z     0.156545
dtype: float64)
framework/PostProcessors/BasicStatistics.general FAILED (
Could not find match for row "2" in Gold:
skew_time                      NaN
skew_x0              -7.411787e-01

The first one BasicStatistics.timeDepOutCsv is related to IOStep,
The second one is related to postprocessor InterfacedPostProcessor|HistorySetSync
The third and forth are related to the csv diff, which can not handle NAN and inf.
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

I checked the regolding...As far as I can say, they look fine... => approved

Comments are fixed, but the test time_dep.xml failed, because of the check alignment. Maybe I did not use it correctly.

We have fixed the checkIndexAlignment. @alfoa Do you have any comments?

test time_dep.xml is passing now.

@PaulTalbot-INL can you do the merge?

Done.


google-authenticator: 1.03 -> 1.05

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/bin/google-authenticator -h got 0 exit code
ran /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/bin/google-authenticator --help got 0 exit code
ran /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/bin/google-authenticator help got 0 exit code
directory tree listing: https://gist.github.com/a10c019e5dc879bc33b83175a42ed947

cc @aneeshusa for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: google-authenticator
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: google-authenticator
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05
shrinking /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/bin/google-authenticator
shrinking /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/lib/security/pam_google_authenticator.so
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/lib  /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05/bin
patching script interpreter paths in /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05
checking for references to /build in /nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05...
/nix/store/f3718pm1ndwa2bj7q3pdbx01qiday4sj-google-authenticator-libpam-1.05

Success on aarch64-linux (full log)
Attempted: google-authenticator
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05
shrinking /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05/lib/security/pam_google_authenticator.so
shrinking /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05/bin/google-authenticator
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05/lib  /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05/bin
patching script interpreter paths in /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05
checking for references to /build in /nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05...
/nix/store/2x3i4159r9qh4fp7zgyys71b5ajy0i4l-google-authenticator-libpam-1.05


trillian-db

Image needs to be tested by rebooting nodes after DB has been running. If passes we should bring the registry.redhat.io image into the operator

the updated image passes the reboot test


Release rules_release as well

Released rules_task and rules_release successfully! https://github.com/vgijssel/setup/releases


Added new file for the take operator micro perf test

Added a new file for the take operator micro perf test.  This is my first attempt at contribution just to learn the architecture.  I'm really interested in helping out, so please let me know if I did this incorrectly

Thank you so much for the PR! Can you add the test for the immediate thread scheduler for take as well? Then call them in the perf/micro/index.js file?
Thanks again for your help!

Updated.  Thanks for the input / guidance!

@smaye81 @blesh looks good to me!

This has been merged as of 2239313


Create Service to listen to Tweet Stream.

[ ] Launch the service off of the TweetFeedActivity

[ ] Set up the twitter stream off the service

[ ] Figure out the filtering for the location

[ ] Initial setup of data structure to hold tweet counts for locations

In case you might need it:
https://gist.github.com/srowhani/49ff5f76eb5c1046e8ff1821243df18d


Regarding Dialogflow's V2 API

Hi,
Any idea when version 2 of the Dialogflow API would be implemented since it's now generally available and out of beta.
Migration guide
Official Dialogflow v2 SDK for Node.js
— H

Implemented Dialogflow V2 API for my own needs.
It's available in a GitHub repository in case someone else needs it.
Two major changes to take into account when porting:

Dialogflow V2 authentication now requires Google Cloud Credentials instead of a token.
The raw nlpResponse object structure has changed and is completely different since V2.

QueryResult | Dialogflow API
V1 to V2 mapping | Dialogflow API

v2.0.2 now supports both Dialogflow API versions v1 and v2.  Complete refactor, updated README and examples using V2.  Thanks @ehrhart - I was able to patch in some of your work.


Incorrect atom:link in RSS feed

While scraping the RSS, I noticed that the <atom:link> tag specifies the feed's URL as http://os.phil-opp.com/post/atom.xml, while the actual feed is at http://os.phil-opp.com/atom.xml.

Thanks a lot for reporting!

Fixed in #212


Update on QR Scan Page

What I want to implement?
Hello everyone! I want to change the QR scan UI design. The issue is that when we come to the "Scanning QR" page, there is no necessary color palette used by Alby except for white and its shadows. I want to make changes by mainly adjusting four capture elements in the scan camera. Specifically, I want to change the color to "orange" and increase the corner radius to 7 to make it more rounded.
Why we need to implement it?
There are mainly two reasons for these changes:

There are no "Alby" colors used on this page, which makes it look out of design for me.
Changing the corner radius is intended to make it more consistent with Alby's design, since we use corner radius values from 5 to 9 in many elements.

How to implement it?
Simply go to this figma link I created and see all necessary details.
Screenshot of what I did

Hey, this is something I would like to work on.Is it possible to assign this to me?

@reneaaron please guide on how can I start customizing the QR code scanner?

Apply custom CSS styles to the scanner container element.
or
Design a custom SVG icon using the desired color.

@AdarshRawat1 I don't think the library in use here has any options to customize the look & feel: (not even hide the existing box)
https://scanapp.org/html5-qrcode-docs/docs/apis/classes/Html5Qrcode
I don't think we would want to overlay the existing box with a custom one, not sure if this "feature" is that important to justify that much of customization.

@AdarshRawat1 I don't think the library in use here has any options to customize the look & feel: (not even hide the existing box)
https://scanapp.org/html5-qrcode-docs/docs/apis/classes/Html5Qrcode
I don't think we would want to overlay the existing box with a custom one, not sure if this "feature" is that important to justify that much of customization.

Thank you @reneaaron , noted!


Install Suricata from source in base Docker image

The latest release from Suricata could suffer from a bug, impacting Antrea features. When installing from PPA, we don't have flexibility when it comes to the Suricata version we ship with Antrea.
Instead, we can install Suricata from source as part of our Docker build.
The advantages are:

full control over the version we install.
a smaller Antrea image, as we do not need to install all of Suricata's dependencies / enable all its features.

The disadvantages of building from source are:

CVEs in Suricata itself won't be detected by scanners (CVEs in Suricata's dependencies will).
While we have more customization options, we also have to manually keep track of Suricata's dependencies (and use correct build options).
We have to remember to update the Suricata version we build & install.

For #4921
This can also be treated as a temporary fix until Suricata 6.0.12 is released, at which point we could revert this change.

@tnqn @hongliangl this is a draft to get your feedback. It's not ready for merging (I haven't updated the UBI build yet).

One more thing: is the script suricatasc  installed automatically after building Suricata from source code? In Antrea, suricatasc is called to add/delete/update/reload Suricata tenants.

One more thing: is the script suricatasc installed automatically after building Suricata from source code? In Antrea, suricatasc is called to add/delete/update/reload Suricata tenants.

yes, it is there:
$ ls -l bin/suricata*
-rwxr-xr-x 1 root root 6322424 Apr 14 07:14 bin/suricata
-rwxr-xr-x 1 root root    1286 Apr 14 07:14 bin/suricata-update
-rwxr-xr-x 1 root root    1397 Apr 14 07:14 bin/suricatactl
-rwxr-xr-x 1 root root    3156 Apr 14 07:14 bin/suricatasc

It is built and installed automatically as long as Python is present at build time

Closing as we don't need this at the moment