Aspect Ratio Option

If possible, it would be very useful to add an aspect ratio adjustment into the display options. In particular, I work with tail radar data primarily, and the default aspect ratio produces a very "squished" display which can be difficult to interpret.

ok, that can be an easy add. Can you post one image,  so that I have an better idea on how that is showing up to you?

but before that, have you tried "adjust display limits"? I know it is not optimal, but this should be able to unsquish your image.

Here is the image with the default aspect ratio, and display limits set to incorporate the entire observable domain:

Here is the same image, with the same display limits, but with aspect ratio set to 1
(using this hard coded addition to plot_radar): self.display.set_aspect_ratio(aspect_ratio=1,ax=self.ax)

And finally, an image using the default aspect ratio, but adjusting the display limits and panning to unsquish and center the data. This method works, but is not ideal.

Ok, I see the difference, I will implement right away.

I just submitted a PR for that. The aspect radio can be found in the adjust limits.
thank you again @stechma2 for helping improving ARTView.

Great! Thank you for doing this @gamaanderson.

Tangentially, I finally figured out why the file does not come up with the default Airborne x/y ranges. The scan_type value of the radar variable is not recognizing these particular files as airborne. I have a workaround that I'll comment on in the PR. We'll have this out in the very near future with this fix.

Fixed with merge of PR #140


Failures to extract IDL

The following specs have IDL but it doesn't get extracted by parse-webidl (I haven't investigated why yet):

[ ] https://w3c.github.io/mediacapture-image/
[ ] https://w3c.github.io/mediacapture-depth/
[ ] https://w3c.github.io/screen-orientation/
[ ] https://w3c.github.io/webdriver/webdriver-spec.html
[ ] https://w3c.github.io/requestidlecallback/
[ ] https://w3c.github.io/presentation-api/
[ ] https://w3c.github.io/resource-timing/
[ ] https://w3c.github.io/navigation-timing/
[ ] http://w3c.github.io/selection-api/

One more recent runs, I now have the following ones suspiciously without IDL:

[ ] https://w3c.github.io/mediacapture-image/ has no IDL
[ ] https://w3c.github.io/requestidlecallback/ has no IDL
[ ]  https://w3c.github.io/performance-timeline/ has no IDL

FYI, all 3 specs use the "markdown" format of ReSpec, which seems to bug jsdom (at least, the postProcess event handlers are never processed) and make the parsing time out after 30 seconds.

OK, so forwarding the window's console output to the Node.js/io/js console reveals the problem: ReSpec markdown processing uses DOMParser which is not (yet?) supported by jsdom, see: https://github.com/tmpvar/jsdom/issues/1341

Follow-up to note that ReSpec only uses DOMParser in core/utils:
https://github.com/w3c/respec/blob/fce1b4aedcb4a569eb538488d9aa132567433735/js/core/utils.js#L226
It should be doable to use a custom build of ReSpec that falls back to another mechanism, e.g. the one described in:
https://developer.mozilla.org/en-US/docs/Web/API/DOMParser

For now, I'll just blacklist the said documents since their TR versions is acceptable

but there are other documents for which I don't know why they fail, e.g. https://w3c.github.io/mediacapture-fromelement/

https://w3c.github.io/encrypted-media/ has no IDL
https://w3c.github.io/mediacapture-screen-share/ has no IDL
https://w3c.github.io/mediacapture-fromelement/ has no IDL
https://w3c.github.io/mediacapture-output/ has no IDL
http://www.w3.org/TR/2008/REC-ElementTraversal-20081222/ has no IDL
http://www.w3.org/TR/2011/WD-orientation-event-20111201/ has no IDL
optionnel :
http://www.w3.org/TR/2011/WD-geolocation-API-v2-20111201/ has no IDL

Crawler should now correctly extract the IDL for all of these specs. However, please note that some of them have IDL errors, so the IDL parser fails to turn that IDL content into a tree and rather reports an error:

encrypted-media
orientation-event
geolocation-API-v2
findtext (not mentioned above)
mediaont-api (not mentioned above)
WebIDL (not mentioned above)

thanks!


[Feature][Workflow] Main package.

Purpose of the pull request
close #7176

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #7177 (65819b5) into dev (ba2b2a6) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #7177      +/-   ##
============================================
- Coverage     40.30%   40.26%   -0.04%
- Complexity     3655     3661       +6
============================================
  Files           653      653
  Lines         26917    26963      +46
  Branches       3005     3016      +11
============================================
+ Hits          10848    10856       +8
- Misses        15044    15081      +37
- Partials       1025     1026       +1

Impacted Files
Coverage Δ

...eduler/server/master/processor/CacheProcessor.java
17.17% <0.00%> (-4.35%)
:arrow_down:

.../dolphinscheduler/api/aspect/CacheEvictAspect.java
2.04% <0.00%> (-1.41%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <0.00%> (ø)

...pache/dolphinscheduler/common/enums/CacheType.java
0.00% <0.00%> (ø)

...org/apache/dolphinscheduler/alert/AlertSender.java
88.46% <0.00%> (ø)

...che/dolphinscheduler/alert/AlertPluginManager.java
4.54% <0.00%> (ø)

.../dolphinscheduler/alert/AlertRequestProcessor.java
92.85% <0.00%> (ø)

...dolphinscheduler/plugin/alert/http/HttpSender.java
58.92% <0.00%> (ø)

...olphinscheduler/plugin/alert/email/ExcelUtils.java
81.25% <0.00%> (ø)

...olphinscheduler/plugin/alert/email/MailSender.java
72.72% <0.00%> (ø)

... and 11 more

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 ba2b2a6...65819b5. Read the comment docs.


T07.05 typo in TODO (4)

The method name described in
// TODO (4) Create a new addGuest method
should be
// TODO (4) Create a addNewGuest method
also in the last quiz question

Fixed.


请求始终处于等待回复的过程中

It never stop......

Please don't open issues for problems unrelated to the frontend client code.


make fetchStatsFromGA tests deterministic

Coverage remained the same at 88.265% when pulling 88d128aa8ba9331960fa6592179a7f21c7a3c512 on sort-analytics into a0a43a18523b0a595b15b48bef9a03c4102b8d73 on master.


fix(internal/cli/build): Using forward slash with embed fs

This PR fixes #396, I changed the copyDirEntries() function implementation to use fs.WalkDir() instead of implementing it in a loop/recursion
Machines Tested:

[x] Windows
[x] Linux (on Docker)
[x] macOS - best if this is tested on an actual macOS machine before it's accepted

@itamadev Sorry, we found a flaw in the functioning of the method fs.WalkDir(). This function considers the softlink to be a file and not a folder. So when copying, it causes some files to not be copied correctly. We may need to change back to the original method to ensure consistent behavior.
// WalkDir walks the file tree rooted at root, calling fn for each file or
// directory in the tree, including root.
//
// All errors that arise visiting files and directories are filtered by fn:
// see the fs.WalkDirFunc documentation for details.
//
// The files are walked in lexical order, which makes the output deterministic
// but requires WalkDir to read an entire directory into memory before proceeding
// to walk that directory.
//
// WalkDir does not follow symbolic links found in directories,
// but if root itself is a symbolic link, its target will be walked.
func WalkDir(fsys FS, root string, fn WalkDirFunc) error {
	info, err := Stat(fsys, root)
	if err != nil {
		err = fn(root, nil, err)
	} else {
		err = walkDir(fsys, root, &statDirEntry{info}, fn)
	}
	if err == SkipDir || err == SkipAll {
		return nil
	}
	return err
}

WalkDir does not follow symbolic links found in directories
test code
	dir := "****/ui/node_modules/.pnpm/@ampproject+remapping@2.2.0/node_modules/@jridgewell/"
	fs.WalkDir(os.DirFS(dir), ".", func(path string, d fs.DirEntry, err error) error {
		fmt.Println(d.Name(), "is dir:", d.IsDir())
		return nil
	})

	stat, err := os.Stat(dir + "gen-mapping")
	if err != nil {
		fmt.Println(err)
	} else {
		fmt.Println("gen-mapping is dir:", stat.IsDir())
	}
	return

the output
gen-mapping is dir: false

gen-mapping is dir: true

os.Stat and fs.WalkDir behavior is different.
ls command output
ls -al
total 0
drwxr-xr-x  4 linkinstar  staff  128 Jul 12 15:07 .
drwxr-xr-x  4 linkinstar  staff  128 Jul 12 15:07 ..
lrwxr-xr-x  1 linkinstar  staff   75 Jul 12 15:07 gen-mapping -> ../../../@jridgewell+gen-mapping@0.1.1/node_modules/@jridgewell/gen-mapping
lrwxr-xr-x  1 linkinstar  staff   80 Jul 12 15:07 trace-mapping -> ../../../@jridgewell+trace-mapping@0.3.15/node_modules/@jridgewell/trace-mapping

@LinkinStars Well I understand the problem, sorry for not testing it thoroughly enough.
I think I can implement a fix while using the original method of walking down the directory, so the issue we fixed using the fs.WalkDir() method won't resurface,
I'll revert the changes and see if I find a fix and update on it.


Stage verification.args[] are not available in the AnalysisRun (anymore)

Checklist

[x] I've searched the issue queue to verify this is not a duplicate bug report.
[x] I've included steps to reproduce the bug.
[x] I've pasted the output of kargo version.
[x] I've pasted logs, if applicable.

Description
In Kargo 0.7.1 the handover of stage verification.args[] to the actual analysis run stopped working. Tested this in 0.3.0 successfully but now when re-evaluating for productive use, this feature seems to stopped working.
Steps to Reproduce
apiVersion: kargo.akuity.io/v1alpha1
kind: Stage
metadata:
  name: dev
  namespace: kargo-demo-helm
spec:
...
...
  verification:
    analysisTemplates:
    - name: kargo-demo-helm
    analysisRunMetadata:
      labels:
        app: kargo-demo-helm
      annotations:
        bat: baz4
    args:
    - name: exit-code
      value: "0"

apiVersion: argoproj.io/v1alpha1
kind: AnalysisTemplate
metadata:
  name: kargo-demo-helm
  namespace: kargo-demo-helm
spec:
  metrics:
  - name: fail-or-pass
    provider:
      job:
        spec:
          template:
            spec:
              containers:
              - name: sleep
                image: alpine:latest
                command: [sh, -c]
                args:
                - exit {{args.exit-code}}
              restartPolicy: Never
          backoffLimit: 1

After trigger of promotion:
apiVersion: argoproj.io/v1alpha1
kind: AnalysisRun
metadata:
  annotations:
    bat: baz4
  creationTimestamp: "2024-07-23T11:06:42Z"
  generation: 2
  labels:
    app: kargo-demo-helm
    kargo.akuity.io/freight: 37903f475b44b5d801dbffa3e35c2dac9a363d27
    kargo.akuity.io/promotion: dev.01j3fmb8dh5pwhq2pc00epcxdq.37903f4
    kargo.akuity.io/stage: dev
...
...
status:
  message: 'Unable to resolve metric arguments: failed to resolve {{args.exit-code}}'
  phase: Error
  runSummary: {}

This test passed in 0.3.0 version
Version
v0.7.1

Logs
From rollouts-controller:
time="2024-07-23T11:06:42Z" level=warning msg="Unable to resolve metric arguments: failed to resolve {{args.exit-code}}" analysisrun=dev.01j3fmb9ng4qq33a2c314em3ts.37903f4 namespace=kargo-demo-helm

I wonder if this is somehow related to #1577, which was added in 0.5.0

The solution is to ensure the AnalysisTemplate contains the definition of the argument you are attempting to use:
---
apiVersion: argoproj.io/v1alpha1
kind: AnalysisTemplate
metadata:
  name: kargo-demo-helm
spec:
  args:
  - name: exit-code
  metrics:
  - name: fail-or-pass
    provider:
      job:
        spec:
          template:
            spec:
              containers:
              - name: sleep
                image: alpine:latest
                command: [sh, -c]
                args:
                - exit {{args.exit-code}}
              restartPolicy: Never
          backoffLimit: 1

This subtle change is required because the bug fix in #1577 now uses the arguments from the AnalysisTemplate as a (default) base to resolve #1573. This implicitly means that arguments must be defined within the AnalysisTemplate for values (overrides) from the Stage to be taken into account.

This implicitly means that arguments must be defined within the AnalysisTemplate for values (overrides) from the Stage to be taken into account.

And that's how analyses work natively in Rollouts, is it not?

Yes, based on my current understanding that's correct. You can't implicitly add new variables, so to say.

Hi guys, can confirm:
By defining the parameters in the AnalysisTemplate it works. Also from overall pattern it make sense. Any chance I can contribute some documentation on this?

@WZBFLM that would be great. There is a Verifications section in docs/docs/15-concepts.md.
One thing I don't want to do is go overboard repeating things that are already covered by Rollouts docs (as long as we're linking to the relevant pages), but adding note or info blocks to call out especially important details would be great.


ryujinx: 1.1.327 -> 1.1.373

It requires the 7.0 SDK and runtime now
Signed-off-by: Amneesh Singh<EMAIL_ADDRESS>Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Tested Pokémon Sword, seems good.

#198719 and #181887 should be closed.


Example about BLOB with a new image in data folder

@LaurentBerger, thank you for the contribution!
here are a few comments:

can you convert the image to png format to save space?
can you rename sample to detect_blobs.cpp or something like that? (i.e. avoid generic name "example" and do not use upper-case names without urgent need)
the example announces the demonstration of MSER, but it's only about SimpleBlobDetector. So, can you remove words about MSER? (or maybe add code to demonstrate MSER?)

thanks! could you please also remove example_blob.bmp from the patch?

thanks! :+1:


constructor domain

I am trying to use this in react/http where $_SERVER is not available. So, I created a way to specify domain in constructor as a workaround. I am open to better ways to solve this :D

any chance of releasing a patch version or a minor version? :D

any chance of creating a tag?

I created a library which depends on this version of the package.
So, a tag would be great.

https://github.com/paragonie/PHP-Cookie/releases/tag/v3.2.0


gd not opening

what is that?
it says it's not opening correctly, all the dll's are intalled correctly and content file is alright

probably you installed gdh incorrectly or geometry dash files are broken, try to reinstall gdh and redownload gdh again


[1.5.x] FAI-354 - Domain dashboards for DRL (cherry pick)

Cherry pick for https://github.com/kiegroup/kogito-runtimes/pull/1180
Kogito-examples PR: https://github.com/kiegroup/kogito-examples/pull/639
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Your code is properly formatted according to this configuration
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Pull Request
Please add comment: Jenkins retest this

Quarkus LTS checks
Please add comment: Jenkins run LTS

Native checks
Please add comment: Jenkins run native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.


Provider a way for user to set custom values

https://docs.gradle.com/enterprise/gradle-plugin/#adding_custom_values

@melissachang could you please clarify what feature you would like? You can always add any custom values that you want to the Gradle Enterprise Gradle plugin.


is this project dead?

Title: Is this still being developed?

This project is waiting on one of three things to happen:

Main Create team port to 1.17
@hYdos finishes porting to 1.17
Someone who knows what they’re doing fixing the rendering related errors in the current branch

ah thanks.


Cannot see General folder after adding Azure SQL Migration ext.

Issue Type: Bug
Downloaded Azure Data Studio
Connected to an Azure database
Added the extension Azure SQL Migration
Unable to view the extension under the Home panel
I figured it out.  I needed to be connected to the onprem db.
Azure Data Studio version: azuredatastudio 1.39.1 (7553f799e175f471b7590302dd65c997b838b29b, 2022-08-30T01:11:13.047Z)
OS version: Windows_NT x64 10.0.17763
Restricted Mode: No

System Info

Item
Value

CPUs
AMD EPYC 7452 32-Core Processor                 (4 x 2346)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

|Memory (System)|32.00GB (18.80GB free)|
|Process Argv||
|Screen Reader|no|
|VM|0%|
Extensions (1)

Extension
Author (truncated)
Version

sql-migration
Mic
1.0.4

@prhodesbajaj You mentioned you figured it out - do you still have an issue that should be investigated or can we close this?

You may close the ticket.  Thanks,
Patti
Upcoming PTO:
Wednesday, October 12 thru Friday October 14
Holiday Schedule:
Thursday, November 24, 2022
Friday, November 25, 2022 Friday, December 23, 2022
Monday, December 26, 2022 Monday, January 2, 2023
Patti Rhodes | Senior SQL Developer
[cid:ad4114e9-bfc1-4f3b-a0cf-3b70be9f8fd2]
m:<PHONE_NUMBER>  |  o:<PHONE_NUMBER>
<EMAIL_ADDRESS>340bplus.comhttp://340bplus.com/  |  LinkedInhttps://www.linkedin.com/company/rxstrategiesinc-?trk=tyah&trkInfo=clickedVertical%3Acompany%2Cidx%3A2-1-2%2CtarId%3A1433444475163%2Ctas%3Arxstra  |  Twitterhttps://twitter.com/340BPlus

From: Charles Gagnon @.>
Sent: Tuesday, September 13, 2022 6:45 PM
To: microsoft/azuredatastudio @.>
Cc: Patti Rhodes @.>; Mention @.>
Subject: Re: [microsoft/azuredatastudio] Cannot see General folder after adding Azure SQL Migration ext. (Issue #20600)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
@prhodesbajajhttps://github.com/prhodesbajaj You mentioned you figured it out - do you still have an issue that should be investigated or can we close this?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azuredatastudio/issues/20600#issuecomment-1246021868, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APFH4G7KY555WQEAEHK3AW3V6D7ZJANCNFSM6AAAAAAQLWDGJ4.
You are receiving this because you were mentioned.Message ID: @.***>


Expose user-facing documentation related to byterange DAP functionality.

I just discovered you forgot to fix docs/Makefile.am and docs/CMakeLists.txt to refer
to byterange.md instead of byterange.dox


Relative URLs in README starting with /

If I have a relative URL starting with a forward slash / in a README for example:
[Documentation](/docs/getting-started.md)

JSR will link to jsr.io and 404, e.g. https://jsr.io/docs/getting-started.md
I would expect this to link back to Github. Both GitHub and NPM allow / linking this way.
If the URL is not prefixed with / e.g. docs/getting-started.md JSR links back to GitHub correctly.

Hey @dbushell, can you please provide me with the link of the page, where the issue arises.

@Muhammad-Owais-Warsi see this package for example — the "MIT License" link at the end is /LICENSE

The bug is somewhere in this function, presumably the early return that checks for leading / is wrong at the top of this file: https://github.com/jsr-io/jsr/blob/main/api/src/docs.rs#L131.

s somewhere in this function, presumably the early return that checks for leading / is wrong at the top of this file:

Thanks @dbushell

The bug is somewhere in this function, presumably the early return that checks for leading / is wrong at the top of this file: https://github.com/jsr-io/jsr/blob/main/api/src/docs.rs#L131.

Hey @marvinhagemeister , are you talking about this code snippet ?
      if key.strip_prefix('/').unwrap_or(key) == entrypoint {
        entrypoint_url = Some(specifier.clone());
      }

If the URL is not prefixed with / e.g. docs/getting-started.md JSR links back to GitHub correctly.

Not if the package was published out of a workspace monorepo, in which case it will link relative to the workspace root and 404 https://github.com/jsr-io/jsr/issues/709

Shouldn't this ideally link to jsr.io/@namespace/repo/version/file instead of github.com/repo/file? There aren't necessarily GH repos for every package, this could properly pin the version, and it might be a more expected & beneficial behavior for relative links to stay on site.


Z-index

In tree mode fullscreen dialog displayed behind.
Can i set value z-index for it?

Hi, @raise-dev
Thanks for your issue, You can customize the style; I'll fix it.


Configure excluded line number ranges

Allow exclusion of words by their line number. It would also then make sense to optionally show line numbers in the web ui source editor.

Options for showing line number with text area
Code Mirror


onping event

Sorry for the late reply, I'll try have a look at this Tomorrow or the day after.

Adding ping/pong messages would be helpful in the examples, to show how to avoid timeouts. @eidheim are you going to approve this, or does more work need to be done? Thanks!

I never could decide what to do with this PR, and the server has to be considered too (this PR is on the client only). However, @RichLewis007 , you can send ping yourself, you just have to add the correct fin_rsv_opcode. At the moment, the client and server handles ping responses (pong) automatically, but again you still have to send the ping yourself.

OK, thanks for the info!

on_ping and on_pong handlers has finally been added!


Improve error handling

Just wanted to open up a draft PR to start a discussion because I think we should initially remove the playback-core error handling for some reasons like:

Currently it's possible to get in an endless loop
It looks like this sample code shouldn't be taken literally, it requires more work on our part to handle it and check for specific conditions before to actually try and recover. The hls.js demo shows more of this, see here: https://github.com/video-dev/hls.js/blob/2635570ca3ffc6ec2c0f8ef4be84b7da9bf39443/demo/main.js#L1041-L1084
hls.js does already retry many areas if they fail with an exponential backoff for network errors etc. and retries for appending segments see https://github.com/video-dev/hls.js/blob/master/docs/design.md#error-detection-and-handling
A fatal network error can be on the master manifest or the playlist which need different recovery; hls.startLoad(); and hls.loadSource(manifestURL) but again this is already retried in hls.js itself.
The sample code at https://github.com/video-dev/hls.js/commit/c4f6564e7dcf50da295e938a166fda48d02d3fa9#diff-b161966f7254af9c9c5382e859495c69622843e4602a6bb4208e08134e660121R139-R158 was written in 2015.

Some resources for offline / online handling:

https://github.com/video-dev/hls.js/issues/1878#issuecomment-416752563
https://github.hubspot.com/offline/docs/welcome/

General Feedback, re: i18n/l10n solution
I may be ok with this as an interim solution, but it is building out an infrastructure that moves us away from where the standards/specifications are and where they are very likely going. While they do not (yet, more below) have native browser support, Internationalized MessageFormat strings have been around for a long time and are based on ICU standards, and the Intl API is intentionally closely tied to things like the ICU CLDR (see: https://cldr.unicode.org/ & https://github.com/unicode-org/cldr-json).
Though it hasn't yet been updated on the official tc39 ECMA 402 github stage tracker, it looks like Intl.MessageFormat has recently been accepted to stage 1 (see: https://docs.google.com/presentation/d/1oThTeL_n5-HAfmJTri-i8yU2YtHUvj9AakmWiyRGPlw/edit#slide=id.p & https://github.com/dminor/proposal-intl-messageformat). There is also a suite of polyfills and build tool packages for using MessageFormat as well, found here.
While I am not recommending we go "whole hog" and use the polyfills for runtime support of Intl.MessageFormat, I do think we should be writing code/an infrastructure that "points in the direction" of assuming that will be where we are aiming. It also opens the door for other tooling, databases, 3rd party infrastructures that conform to these standards.
Again, since you've already provided a solution that will work, I could be convinced to keep it "as is" for now, but I wouldn't want us to continue to invest in more complex use cases if that's going to be moving our codebase farther away from where I think we will want to be.


metric_aggregation error

I'm trying out the new metric_aggregation feature. I'm receiving the error below:
ERROR:root:Uncaught exception running rule Apache_Thread: ElastAlerter instance has no attribute 'rule'
^C[zsssc5z@lsdnecsseot bin]$ vi ~/elastalert_rules/apachethreads.yaml
[zsssc5z@lsdnecsseot bin]$ ./elastalert
ERROR:root:Traceback (most recent call last):
File "/home/zsssc5z/.local/lib/python2.7/site-packages/elastalert-0.1.8-py2.7.egg/elastalert/elastalert.py", line 904, in run_all_rules
num_matches = self.run_rule(rule, endtime, self.starttime)
File "/home/zsssc5z/.local/lib/python2.7/site-packages/elastalert-0.1.8-py2.7.egg/elastalert/elastalert.py", line 669, in run_rule
if not self.run_query(rule, rule['starttime'], tmp_endtime):
File "/home/zsssc5z/.local/lib/python2.7/site-packages/elastalert-0.1.8-py2.7.egg/elastalert/elastalert.py", line 476, in run_query
data = self.get_hits_aggregation(rule, start, end, index, rule.get('query_key', None))
File "/home/zsssc5z/.local/lib/python2.7/site-packages/elastalert-0.1.8-py2.7.egg/elastalert/elastalert.py", line 411, in get_hits_aggregation
query = self.get_aggregation_query(base_query, rule, query_key, term_size, self.rule['timestamp_field'])
AttributeError: ElastAlerter instance has no attribute 'rule'
My rule is:
name: Apache_Thread
type: metric_aggregation
es_host: localhost
es_port: 9200
index: logstash-%Y.%m.%d
use_strftime_index: true
buffer_time:
hours: 1
metric_agg_key: apache_threads
metric_agg_type: avg
#query_key: _source
doc_type: newssi_monitoring
bucket_interval:
minutes: 5
sync_bucket_interval: true
#allow_buffer_time_overlap: true
#use_run_every_query_size: true
min_threshold: 1
max_threshold: 5
#filter:
#- term:
metricset.name: cpu
(Required)
The alert is use when a match is found
alert:

"debug"

Any ideas?

This was fixed earlier today. https://github.com/Yelp/elastalert/commit/bf631b80a3e4db3414cacec5c1823f4a8abf4be5
Please pull master and try again.


feat: adds events to handle changes in xblocks

Description: Adds events and data classes to allow consumers to react on xblock events like published, deleted and duplicated.
The idea is to fire these events in delete_item, publish_item and _duplicate_item and update skills related to the xblock in taxonomy-connector (part of course-discovery).
Answer for a question asked in previous similar MRs:
Do consumers have to make separate requests to get more information?
Generally: Additional data if required related to the event needs to be fetch separately as it can be different for different use case.
Specifically for skill tagging: Yes, for skill tagging to work we need concatenated text representation of children xblocks for units and transcript for video xblocks which can be extracted using index_dictionary of the xblock. The idea is to add a new api to fetch block metadata and use this API in taxonomy-connector.
Related MRs:
These MRs are the foundation for storing skills related on xblocks level, specifically for unit and video xblocks.

https://github.com/openedx/taxonomy-connector/pull/119
https://github.com/open-craft/taxonomy-connector/pull/1
https://github.com/openedx/edx-platform/pull/31273

ISSUE: Link to GitHub issue
Merge checklist:

[x] All reviewers approved
[ ] CI build is green
[x] Version bumped
[x] Changelog record added
[x] Documentation updated (not only docstrings)
[x] 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)

Author concerns: List any concerns about this PR - inelegant
solutions, hacks, quick-and-dirty implementations, concerns about
migrations, etc.

Thanks for the pull request, @navinkarkera!  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.
Please let us know once your PR is ready for our review and all tests are green.

@navinkarkera One thing that kind of stood out from the PR description was this question:

Do consumers have to make separate requests to get more information?

The answer is specific to skill tagging which is great. It can also be more generally answered, as the event is only passing the UsageKey to the consumer, all other metadata will need to be fetched by the consumer separately. Eg., User deleting an XBlock, diff of the content during publishing..etc.,
This is just a note, to help myself understand the scope of the change here and aid other reviewers, if needed. No action item here :)

@tecoholic Thanks for the suggestions! Updated the MR.

The answer is specific to skill tagging which is great. It can also be more generally answered, as the event is only passing the UsageKey to the consumer, all other metadata will need to be fetched by the consumer separately. Eg., User deleting an XBlock, diff of the content during publishing..etc.,

Makes absolute sense. Thanks, i'll add this to the description.

@mariajgrimaldi Sorry, when I clicked on request-a-review-again button for @tecoholic, it removed you from reviewers list and I cannot add you back.

Hello! Thanks for your contribution! I'll review this as soon as I can.

Thank you!

do you have a working example for testing? if so, can you add instructions to the PR description? thanks!

do you have a working example for testing? if so, can you add instructions to the PR description? thanks!

@mariajgrimaldi I don't have one yet as the work on the edx-platform side is pending. I do want to add one more signal and data class to send list of verified skill ids and xblock usage_key. The idea is that users will verify the tags/skills associated to an XBlock. We want to send this data via openedx-event signals to course-discovery and update the relevant tables. I was wondering whether I should add it in the same MR or create a separate MR once this is merged. Do let me know your thoughts here.

@navinkarkera: it's best to create a separate PR for those changes.
@felipemontoya: what do you think of these changes? I'd like to test a working version, for now, this looks quite good!

@mariajgrimaldi Thanks! I have created a separate MR: https://github.com/open-craft/openedx-events/pull/1 on top of this branch. The new event requires support for array types so it is also added in the MR. Please have a look.

@mariajgrimaldi Any updates on this? Let me know if something needs to be changed.

Hello @navinkarkera, thank for the contribution.
Even though I came in late to the discussion, everything seems taken care of.
You even added the avro serializer. The only question I have remaining is if there is any need to also pass the user reference to the events? In the publish, delete and duplicate actions that you linked there is always an xblock and a user. I don't necessarily think that we should pass it along, but I'l like to know if that user is important.

Hello @navinkarkera, thank for the contribution.
Even though I came in late to the discussion, everything seems taken care of. You even added the avro serializer. The only question I have remaining is if there is any need to also pass the user reference to the events? In the publish, delete and duplicate actions that you linked there is always an xblock and a user. I don't necessarily think that we should pass it along, but I'l like to know if that user is important.

@felipemontoya Thanks. We don't need user details, at least for the purpose that I am adding these events for.

@navinkarkera: can you fix the CI? I think a test for the new type is missing. Do you know if UsageKey is ever used here?

@navinkarkera: can you fix the CI? I think a test for the new type is missing. Do you know if UsageKey is ever used here?

@mariajgrimaldi Yes, it is used in https://github.com/openedx/openedx-events/blob/1934a839b6b04db1b02634cd74320045f2e6ed17/openedx_events/event_bus/avro/tests/test_avro.py#L70-L83
Not sure what is missing now.

@mariajgrimaldi Gentle reminder.

@navinkarkera: I'm out for a few days, I'll check on this when I'm back. Sorry for the delay.

can you rebase with main? thank you!

can you rebase with main? thank you!

@mariajgrimaldi It is already up to date.

I thought rebasing would help with the coverage issue. Can you check that so we can merge? Let me know if you need help!

I thought rebasing would help with the coverage issue. Can you check that so we can merge? Let me know if you need help!

@mariajgrimaldi Running make coverage locally shows that the coverage increased from 95% to 96%. Also visiting the codecov link also shows an increase of 0.13% as opposed to what is displayed here in github below.

Not sure how to proceed here. Should we implement tests for missing lines even if they are not related to any change in this MR? Please advice.

@mariajgrimaldi Friendly ping on @navinkarkera's comment above.

I'm back on monday. I'll take a look at this then. Sorry for the delay!

hello! can you rebase with main? The codecov for the project changed, so I think it'll pass this time

@mariajgrimaldi Updated! Please approve the workflow.

@mariajgrimaldi Thanks! Updated.

@navinkarkera 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

New release: https://github.com/openedx/openedx-events/releases/tag/v4.1.0


Ctrl + S behaves like "Save and Exit" in embed mode

Preflight Checklist

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

You must agree to search and the code of conduct. You must fill in this entire template. If you delete part/all or miss parts out your issue will be closed.
If you are technical, you should reporting bugs along the lines of https://marker.io/blog/how-to-write-bug-report. If you are not technical, we will make allowances, please try to make an effort to understand the process.
Describe the bug
As per the description of JSON protocol in embed mode (https://www.drawio.com/doc/faq/embed-mode), exit value would be present for "Save and Exit" in post message. Save behaves correctly and doesn't have exit in the post message. But Ctrl + S behaves like "Save and Exit" includes exit: true in the post message.
To Reproduce
Steps to reproduce the behavior:

Open draw.io in embed mode.
Press Ctrl + S.
The post message has exit: true

Expected behavior
Ctrl + S shouldn't send exit as true in post message or provide a way to differentiate between save operations and exit operation.
Screenshots

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

draw.io version 22.0.3

Desktop (please complete the following information):

OS: Mac OS
Browser: chrome
Browser Version: 119

Smartphone (please complete the following information):

Device: NA
OS: NA
Browser NA
Version NA

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:

yes

Additional context
I am embedding the draw.io app in our application to let users draw, but when users use the keyboard shortcut, due to exit:true being present in the post message, I am closing the iframe. But in confluence, Ctrl+S doesn't close draw.io, would like to have same behaviour. It might inconvenience the user if I close it on Ctrl + S.

Any help here please?

Are you showing the Save and Exit button in addition to the Save button?

Are you showing the Save and Exit button in addition to the Save button?

Yes. But i have removed it for now and ignoring the exit: true in the postmessage.

If save and exit is shown, we're going to change the keyboard shortcut (and tooltips) for Ctrl+S and assign Ctrl+Shift+S to Save (which will deliver the save message without the exit flag). If save and exit is not shown, the exit flag will not be used. This keeps the protocol backwards compatible with the current implementations.


[Feature Request] Login/Logout

What to do:

[ ] Login
[ ] Logout

That's it :))

@Ranie179

Seem to be done in #1 by @Ranie179


creaminstaller: Add version <IP_ADDRESS>

A DLC unlocker for Steam, Epic, and Ubisoft games.

[x] I have read the Contributing Guide.

/verify

Giving the shady nature of the programs it's relying on (and dubious legality), I think it's best to refuse this.
Thanks for the contribution nonetheless :slightly_smiling_face:


Unique "key" prop warning in v7.1.0

Loving this addon, so thank for that!  I am noticing a warning in my browser console when I have this addon installed & enabled, and it disappears if I don't have this addon registered in my Storybook addons.js:

Looks like the warning is coming from Emotion, used by @storybook/theming.  I downgraded to v6 and still saw this warning.

I'm also getting this error.

Same to me

:rocket: Issue was released in v7.1.1 :rocket:


Re-authenticate on PyiCloudAPIResponseError

Sometimes when downloading many photos iCloud will
return session errors. Catch these and re-authenticate.
Fixes the "Invalid global session" Traceback.
Fixes #47

Awesome, thankyou!


Refactor panorama attitude planner for standalone testing. Port Python reference implementation to C++.

This version should have identical inspection manager behavior to the previous version, but it's easier to test the attitude planner on a standalone basis.
I also ported the latest version of the Python reference implementation to C++ so we can compare to the attitude planner that's currently used by the inspection manager.

Hi Trey,
I am confused, we already had one working version of this function, and we are adding a second one to do the same thing? Am I missing something? Thanks!

I am confused, we already had one working version of this function, and we are adding a second one to do the same thing? Am I missing something?

Stepping back for a second, part of what I'm trying to do is formalize the semantics of the "panorama coverage planning" problem so that we can agree whether a panorama plan is sound (i.e., covers the area it's supposed to, with required overlap and robustness to pointing error) and within that envelope try to optimize better (reduce execution time by requiring fewer images).
The pano_orientations2() function is a modest improvement in that it accounts for pointing error, is more careful about unnecessary overshoot at the edges of the panorama, and tries to be smarter about not taking too many images near the poles.
I don't regard either version of the function as having been validated thoroughly enough. We never completed and stitched even a single full panorama. I hope to write some tools to visualize coverage to help us get more comfortable with all of this.

Ok! It sounds like there is more work to do--- I would suggest let's leave this as a pull request and not merge it yet. It would also be better to only have one version to maintain once it's been verified. You and Marina can discuss. Thanks!

closing this one as I used it as base for https://github.com/nasa/isaac/pull/70


Ingress + Request transformer + Service do not work together

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
Kong 2.5
Current Behavior
I have noticed this inconsistency when using an Ingress and a Request Transformer.
An Ingress specifies a path and a backend (which is a Kubernetes Service). It also defines the konghq.com/plugins annotation with one of the annotations being the request transformer plugin which rewrites/replaces the URI. As such, when this exact scenario happens, when the request is forwarded from Kong to the Kubernetes Service, the Service does not use the konghq.com/path: annotation at all!
For a more concrete example, we are using Kong to handle certain paths in the Ingress, which is forwarded to a Service of type ExternalName, which is configured with the DNS of an API Gateway.
Now, API Gateway has Stages which are basically prefixes to the URI that are added. This URI prefix is added by the Service using the konghq.com/path: /qa (in our QA environment). This works fine for all paths handled by the Ingress. However, we have certain paths which need to be rewritten to the latest paths, and for these, the /qa prefix is not prefixed when the request is forwarded from the Service to the API Gateway.
This might be related to Kong aws-lambda plugin applied to an Ingress. An unrelated note is that when things go wrong like this, it is super-hard to debug Kong internals and the documentation wasn’t great in this case. I arrived on the solution after hours of debugging. :confused:
Expected Behavior
No response
Steps To Reproduce
No response
Anything else?
Cross posted at https://discuss.konghq.com/t/ingress-request-transformer-service-do-not-work-together/10796/1

cc @Kong/team-k8s

This is the expected effect of calling the PDK set path function in the plugin code, correct? The controller has no bearing on that; the PDK function just sets ngx_var.upstream_uri to the absolute value given.


Planner supports() and asserts

I see that the Solver interface has a supports() method to establish if the solver can solve the problem.
I see that tamer (up-tamer) checks for (asserts) this before starting solving the problem, while pyperplan (up-pyperplan) has the correct supports() function, but it is not used. Wouldn't it make sense to always (i.e. from the interface itself) check this, instead of demanding the check to the single implementations?

This is reasonable.
However, we need to intercept the call to the operation modes and this causes a slight change in the Engine. @alvalentini will work on this (since we already broke backward compatibility anyway 😆 )


[BUG] Extract only crashing on joins

Describe the bug
Tableau crashes on trying to extract with a join. If the join contains no data it will not crash unless you undo.
Removing extract only allows the extract to happen and does not crash Tableau.
tabprotosrv_2020_04_07_08_30_54.txt
Desktop (please complete the following information):

OS: MacOS 10.14.6
Tableau Version 2019.4.1 & 2020.1

About you:
Name: Brandon Montoya
Company: Beezwax

Hi Brandon,
We opened a story on this issue to determine the source of the bug. We will get back to you as soon as there are further updates from our engineering team.
In the meantime, can you please provide the source files for the Connector? Either a github repo or sharing directly with me is fine.
Thank you!
TS


Can't kill DSC Configuration process at the server

Hello,
I am having troruble to remove hanging DSC Configuration process from the server.
It started when I tried to run upgrade process for ArcGIS License Server and Desktop using Powershell commands, on disconnected server. I installed NuGet and ArcGIS module by copying them from connected machine:
PS D:> Get-Command -Module ArcGIS
CommandType     Name                                               Version    Source

Function        Get-FQDN                                           3.0.0      ArcGIS
Function        Invoke-ArcGISConfiguration                         3.0.0      ArcGIS
Function        Invoke-BuildArcGISAzureImage                       3.0.0      ArcGIS
Function        Invoke-PublishWebApp                               3.0.0      ArcGIS
But when I tried to run "Invoke-ArcGISConfiguration ...." process, it hangs indefinitely.
Now I am trying to stop the process, but it does not work as well.
I tried following command, but it is hanging:
PS D:> Stop-DscConfiguration –Force
After the server reboot, I tried this command, which produces some message, but also hangs:
PS D:> Remove-DscConfigurationDocument -Stage Current,Pending -Force –Verbose
VERBOSE: An LCM method call arrived from computer MSSEMPAPVWDV01 with user sid
S-1-5-21-4258890176-1816793054-651103780-16727.
VERBOSE: The -Force option was specified with the Stop operation. The current configuration has been successfully
However, this message is misleading. The command line was never returned (process hangs), and the DSC Configuration process still running:
PS D:> Get-DscLocalConfigurationManager
ActionAfterReboot              : ContinueConfiguration
AgentId                        : 3A2AC180-6959-11EA-A319-0050569753DE
AllowModuleOverWrite           : False
CertificateID                  :
ConfigurationDownloadManagers  : {}
ConfigurationID                :
ConfigurationMode              : ApplyAndMonitor
ConfigurationModeFrequencyMins : 15
Credential                     :
DebugMode                      : {NONE}
DownloadManagerCustomData      :
DownloadManagerName            :
LCMCompatibleVersions          : {1.0, 2.0}
LCMState                       : Busy
LCMStateDetail                 : LCM is performing a consistency check.
LCMVersion                     : 2.0
StatusRetentionTimeInDays      : 10
SignatureValidationPolicy      : NONE
SignatureValidations           : {}
MaximumDownloadSizeMB          : 500
PartialConfigurations          :
RebootNodeIfNeeded             : False
RefreshFrequencyMins           : 30
RefreshMode                    : PUSH
ReportManagers                 : {}
ResourceModuleManagers         : {}
PSComputerName                 :
My question now, how can I kill this process? It continues running even after server reboot. I need to clean the server from it, to be able to proceed with working upgrade solutions (like GIU-based or silent upgrade). But I am afraid that running DSC Config process may cause issues.
Can someone please help with "cleanup" issue?

try changing on your LCM
from
ConfigurationMode : ApplyAndMonitor
to
ApplyOnly
https://docs.microsoft.com/en-us/powershell/scripting/dsc/managing-nodes/metaconfig?view=powershell-7
https://docs.microsoft.com/en-us/powershell/module/psdesiredstateconfiguration/set-dsclocalconfigurationmanager?view=powershell-5.1

@ludmilabogucharska You could also try using the following. Keep in mind this will clear Everything and caution should be used.
#Remove all mof files (pending,current,backup,MetaConfig.mof,caches,etc)
rm C:\windows\system32\Configuration\*.mof*
#Kill the LCM/DSC processes
gps wmi* | ? {$_.modules.ModuleName -like "*DSC*"} | stop-process -force

@nshampur
Sorry, I don't understand how it helps to remove the DSC Configuration process from the server? Can you please explain better why we need it?
Btw I tried to run it, but there's no such parameter in Set-DscLocalConfigurationManager command.
PS H:> $Host.Version
Major  Minor  Build  Revision

5      1      14393  3471
PS H:> Set-DscLocalConfigurationManager -ConfigurationMode ApplyOnly
Set-DscLocalConfigurationManager : A parameter cannot be found that matches parameter name 'ConfigurationMode'.
At line:1 char:34

Set-DscLocalConfigurationManager -ConfigurationMode ApplyOnly

                             ~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Set-DscLocalConfigurationManager], ParameterBindingException
FullyQualifiedErrorId : NamedParameterNotFound,Microsoft.PowerShell.DesiredStateConfiguration.Commands.SetDscLoc
alConfigurationManagerCommand

@PleaseStopAsking
That helped, thank you! I run the shorter version of Get-Processes command at first, just to see what processes are there:
PS H:> gps wmi* | ? {$_.modules.ModuleName -like "DSC"}
Handles  NPM(K)    PM(K)      WS(K)     CPU(s)     Id  SI ProcessName

470      24    15500      28360      17.13   2844   0 WmiPrvSE
635      35    64124      34496       4.97   3216   0 WmiPrvSE

I found them in Event Viewer as well, by thier ID, they seem relevant.. Thanks again. I hope I did not kill anything I'm not supposed to..


SetDBOptions inside onBecomeFollowerFromOffline based on resource_con…

…figs on zk

Integration tests: checkin the code and deploy to test cluster with 2 segments:
Setup

segment "test" with the resource_configs at its zk metadata path:

/metadata> get rockstoredev-rw-beta-test/test/resource_configs
{"disable_auto_compactions":"true"}

segment "test2" w/out resource_configs at its zk metadata path (all previous existing mutable segment don't have resource_configs at the zk path).

Test
deploy the cluster to trigger the Offline->follower ST. Following logs prove it works as expected:
# this prove: segment with new `resource_configs` on zk, will actually setupDBOptions.
[HelixTaskExecutor-message_handle_thread] ERROR .pinterest.rocksplicator.Utils - setDBOptions for partition: test00000, host: <IP_ADDRESS>, port: 9090

# this prove: existing segment w/out the `resource_configs` still proceed offline->follower transition, and will skip setDBOptions
[HelixTaskExecutor-message_handle_thread] ERROR .pinterest.rocksplicator.Utils - ResourceInfoPath (/metadata/<cluster>/test2/resource_configs) not visible or doesn't exist yet:


clean-fid build_resizer Import error

I am working on vid2vid GAN model from Nvidia-Imaginaire library. It uses clean-fid library. While running the model on google colab. I encounter this error related to the clean-fid.
from imaginaire.evaluation import compute_fid
File "/content/imaginaire/imaginaire/evaluation/init.py", line 5, in
from .fid import compute_fid, compute_fid_data
File "/content/imaginaire/imaginaire/evaluation/fid.py", line 10, in
from imaginaire.evaluation.common import load_or_compute_activations
File "/content/imaginaire/imaginaire/evaluation/common.py", line 14, in
from cleanfid.resize import build_resizer
File "/usr/local/lib/python3.7/dist-packages/cleanfid/resize.py", line 10, in
from cleanfid.utils import *
File "/usr/local/lib/python3.7/dist-packages/cleanfid/utils.py", line 5, in
from cleanfid.resize import build_resizer
ImportError: cannot import name 'build_resizer' from 'cleanfid.resize' (/usr/local/lib/python3.7/dist-packages/cleanfid/resize.py)
The Issue is it is not able import the build_resizer from cleanfid.resize.
I don't have this issue like two days back now I am having this issue. Is it due to the new version.
Thanks in advance

Hi,
Thank you for pointing out this issue.
Yes it looks like the clean-fid v0.1.16 and v0.1.17 have some import functions that cause this issue.
The new library version v0.1.18 should fix this.
Please let me know if the issue persists in this new version.
Regards,
Gaurav

Hi @GaParmar Thanks for your response. It solved my issue.


Enhance the autoupdates macro

This PR reimplements the @autoupdates macro by

Allowing for more detailed expressions within the autoupdate specification. For example, the specification can include complex functions with constants and multiple latent states.
Enabling the use of full function definitions with the macro, allowing for dynamic construction of autoupdates.
Adding a thorough set of tests.
Improving the documentation.
Adding a possibility to broadcast a function over a collection of variables

This PR fixes https://github.com/ReactiveBayes/RxInfer.jl/issues/61

@bvdmitri what if we make the autoupdates macro recursive such as constraints, meta and init? Might be nice to also update prior distributions for variables hidden in some submodel.

I like the idea @wouterwln , I think we can accomplish this in a separate PR, since we really need a generic code for those
for _something_ in submodel

end

pattern. As far as I understand now its mostly copy-pasted for all macros, lets not do that again. Also, this would require to extend the VarDict, which now looks only at the top-level.

I've also come to realize that to support this feature in submodels, we'll need a more integrated approach with GraphPPL. This is because the autoupdates requires that certain arguments remain unspecified. However, I don't believe we can create a submodel with certain arguments underspecified.


Serenity/JS 3.0 - Support for easier debugging

Serenity/JS supports standard Node.js Promises, so in order to debug an actor flow, one can create an interaction to SetABreakpoint and run the test through a debugger:
import { Interaction } from '@serenity-js/core';

export const SetABreakpoint = () =>
  Interaction.where(`#actor sets a breakpoint`, actor => {
    debugger;
  });

Then:
import { actorCalled } from '@serenity-js/core';
import { Navigate } from '@serenity-js/web';

describe('using a debugger', () => {

  it('stops at a breakpoint', async () => {
    await actorCalled('Debbie').attemptsTo(
      Navigate.to('https://serenity-js.org'),
      SetABreakpoint(),
      // other activities
    );
  });
});

However, this doesn't seem like the most convenient way to do it.
So, Serenity/JS should:

provide built-in interactions to help with debugging
look into building on top of capabilities provided by the integration tools

I don't know if this is in scope, but I really like the debugging offered by the Playwright test runner, in particular through the VS Code extension. That and the trace capturing are the most useful parts when working out what is going on/wrong with a test, for me.


calculate_logical_size: no longer use spawn_blocking

Calculation of logical size is no longer blocking, so we shouldn't use spawn_blocking for it. Use of spawn_blocking exhausted resources which are needed by tokio::io::copy when copying from a stream to a file.
Fixes: #3657

That's still a valid concern, right? I guess it's a more general issue that we use blocking I/O on files, and we don't need to specifically care about it here.

This is an issue still yes, more like #2975, but we created a deadlock with the block_on from within spawn_blocking, so yeah, not entirely resolved issue. I don't think it's worthwhile to retain this one comment.

Yay, first non-transient pass of regress-tests of the evening!


Add OWNERS, SECURITY_CONTACTS and code-of-conduct.md

Adding necessary meta information while referring to
https://github.com/kubernetes/org/issues/1875
The contribution guidelines have been updated as well and now follow
https://github.com/kubernetes/kubernetes-template-project/blob/master/CONTRIBUTING.md

@saschagrunert while we are here anyway, can you also add the "Community, discussion..." and "Code of conduct" sections to the README as per https://github.com/kubernetes/kubernetes-template-project#community-discussion-contribution-and-support? :pray:

@saschagrunert while we are here anyway, can you also add the "Community, discussion..." and "Code of conduct" sections to the README as per https://github.com/kubernetes/kubernetes-template-project#community-discussion-contribution-and-support?

Done as well. Ready for another review round :-1:

Lgtm from me :+1:


Embed EKS-A version in image-builder with support for overrides

This PR adds the logic to embed the EKS-A version in production image-builder CLI binaries during build time to make it easy for customers to understand what version of EKS-A the built images will correspond to. An eksa-release flag is also added for customers to be able to override the EKS-A version for which images are to be built. Based on the version passed or embedded, the corresponding bundle will be used to get the artifacts and git commit to checkout.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/approve


Fixing execution plan not finding provider

Issue: #22237
Was a wrong syntax in the for loop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 41.965%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
28810

Relevant Lines:
63747

💛  - Coveralls


Fluvio bug reproduction

This is not meant to be merged
While putting together an end-to-end test for CI, I discovered this problem which I believe to be a bug in the Fluvio client.
To reproduce, check out this branch and run the following:
$ cd cdc-mysql/
cdc-mysql $ ./redo.sh    # This will uninstall and reinstall fluvio and mysql CDC containers
cdc-mysql $ mysql -h <IP_ADDRESS> -P 3080 -ufluvio -pfluvio4cdc! -v < test_files/producer_script.sql
cdc-mysql $ cargo run --bin cdc-producer -- ./producer_profile.toml  # In one terminal window
cdc-mysql $ fluvio consume rust-mysql-cdc  # In another terminal window, to see that all events are produced
cdc-mysql $ RUST_LOG=debug cargo run --bin cdc-consumer -- ./consumer_profile.toml  # In another terminal window

In the consumer window, you should see debug output that looks like this:
Oct 13 13:39:09.971 DEBUG cdc_consumer: Received record: offet=0
Oct 13 13:39:09.972 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.972 DEBUG cdc_consumer: Received record: offet=1
Oct 13 13:39:09.972 DEBUG cdc_consumer: Received message: msg="{\"uri\":\"flv://mysql-srv1/flvDb\",\"sequence\":0,\"bn_file\":{\"fileName\":\"binlog.000003\",\"offset\":233},\"operation\":{\"Query\":\"CREATE DATABASE flvDb\"}}"
Oct 13 13:39:09.972 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.972 DEBUG cdc_consumer: Received record: offet=2
Oct 13 13:39:09.972 DEBUG cdc_consumer: Got record bytes
FluvioMessage { uri: "flv://mysql-srv1/flvDb", sequence: 0, bn_file: BnFile { file_name: "binlog.000003", offset: Some(233) }, columns: None, operation: Query("CREATE DATABASE flvDb") }
UriProps { db_name: Some("flvDb"), table_name: None }
Oct 13 13:39:09.972 DEBUG cdc_consumer: Received record: offet=3
[query] "CREATE DATABASE flvDb"
Oct 13 13:39:09.974 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.975 DEBUG cdc_consumer: Received record: offet=4
Oct 13 13:39:09.975 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.975 DEBUG cdc_consumer: Received record: offet=5
Oct 13 13:39:09.975 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.975 DEBUG cdc_consumer: Received record: offet=6
Oct 13 13:39:09.975 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.975 DEBUG cdc_consumer: Received record: offet=7
Oct 13 13:39:09.975 DEBUG cdc_consumer: Got record bytes
Oct 13 13:39:09.988 DEBUG cdc_consumer: Received message: msg="{\"uri\":\"flv://mysql-srv1/flvdb\",\"sequence\":1,\"bn_file\":{\"fileName\":\"binlog.000003\",\"offset\":423},\"operation\":{\"Query\":\"CREATE TABLE pet (name VARCHAR(20), owner VARCHAR(20), species VARCHAR(20), sex CHAR(1), birth DATE)\"}}"
FluvioMessage { uri: "flv://mysql-srv1/flvdb", sequence: 1, bn_file: BnFile { file_name: "binlog.000003", offset: Some(423) }, columns: None, operation: Query("CREATE TABLE pet (name VARCHAR(20), owner VARCHAR(20), species VARCHAR(20), sex CHAR(1), birth DATE)") }
UriProps { db_name: Some("flvdb"), table_name: None }
[query] "CREATE TABLE pet (name VARCHAR(20), owner VARCHAR(20), species VARCHAR(20), sex CHAR(1), birth DATE)"
Oct 13 13:39:10.027 DEBUG cdc_consumer: Received message: msg="{\"uri\":\"flv://mysql-srv1/flvdb/pet\",\"sequence\":2,\"bn_file\":{\"fileName\":\"binlog.000003\",\"offset\":832},\"columns\":[\"name\",\"owner\",\"species\",\"sex\",\"birth\",\"color\"],\"operation\":{\"Add\":{\"rows\":[{\"cols\":[{\"String\":\"Puffball\"},{\"String\":\"Diane\"},{\"String\":\"hamster\"},{\"String\":\"f\"},{\"Date\":{\"year\":1999,\"month\":3,\"day\":30}}]}]}}}"
FluvioMessage { uri: "flv://mysql-srv1/flvdb/pet", sequence: 2, bn_file: BnFile { file_name: "binlog.000003", offset: Some(832) }, columns: Some(["name", "owner", "species", "sex", "birth", "color"]), operation: Add(WriteRows { rows: [Cols { cols: [String("Puffball"), String("Diane"), String("hamster"), String("f"), Date { year: 1999, month: 3, day: 30 }] }] }) }
UriProps { db_name: Some("flvdb"), table_name: Some("pet") }
[query] "INSERT INTO pet (name, owner, species, sex, birth, color) VALUES (\"Puffball\", \"Diane\", \"hamster\", \"f\", \"1999-3-30\")"
Error: Query: MySqlError { ERROR 1054 (42S22): Unknown column 'color' in 'field list' }

The mysql error at the end is occurring because the record carrying the ALTER TABLE was dropped. You can see exactly where in the consumer code this is happening:
// cdc-mysql/src/bin/consumer.rs

// read read from producer and print to terminal
while let Some(Ok(record)) = stream.next().await {
    debug!(offet = record.offset(), "Received record:");
    if let Some(bytes) = record.try_into_bytes() {
        debug!("Got record bytes");
        let msg = String::from_utf8(bytes).expect("error vec => string");
        sender.send(msg).expect("error sending message");
    }
}

problem running producer
cargo run --bin cdc-producer -- mysql-cdc/producer_profile.toml
    Finished dev [unoptimized + debuginfo] target(s) in 0.11s
     Running `/home/ubuntu/fluvio-demo-apps-rust/target/debug/cdc-producer mysql-cdc/producer_profile.toml`
Error:
   0: CDC config error
   1: No such file or directory (os error 2)

Location:
   cdc-mysql/src/bin/producer.rs:89

$ cd cdc-mysql/

@sehz you need to be in the cdc-mysql directory if you're running the commands exactly. Otherwise you need to make sure to use the right path to point to the producer_profile.toml file.

there is no mysql-cdc directory

I am in  cdc-mysql directory

ls
Cargo.toml  MYSQL_COMMANDS.md  README.md  consumer_profile.toml  docker  producer_profile.toml  redo.sh  src  test_files

there is no mysql-cdc directory

My bad. What I meant was that the filepath you're giving for the producer profile is wrong. You ran
cargo run --bin cdc-producer -- mysql-cdc/producer_profile.toml

but if you're already in cdc-mysql, you should run
cargo run --bin cdc-producer -- ./producer_profile.toml

that command was in the CI.yaml


Review Ticket - Week 2

Lists, Dictionaries, Iterations
Attempt at Theme Change
Table
Quiz Using Lists
Blog + Proof of Dark Mode Working

I believe Jishnu deserves above an 2.7 because he has followed and completed all of the requirements that were needed to be completed for this week.


Emacs tree-sitter

I'm missing the robot-ts-mode major mode and for now stay with robot-mode.
While playing with the treesit-explore-mode minor mode, I found some strange behavior:
*** Settings ***
Documentation       A simple web scraper robot.
...                 Opens a website.
...                 Stores the web page text in a file in the output directory.
...                 Saves a screenshot of an element in the output directory.

 The first one would be the Library on line 8 but I have no idea it isn't resulting in a setting_statement like the Library on line 7.
Is it me and my limited Emacs and Tree-Sitter skills or did I find a bug?

I think I've forgotten to add *** Tasks *** as a valid section header. Could you try replacing it with *** Test Cases *** to make sure?
Strange that it would invalidate the second "Library" setting though :thinking:

Nice catch. I forgot that *** Test Case *** is the normal and *** Tasks *** was added later by https://github.com/robocorp, if I got it right, for RPA.


iOS support?

Hello, just found this package after searching for a long time for such functionality. Looks promising, but I was curious whether it supports iOS? There is an unchecked TODO in the readme for 'iOS: Get a list of available ringtones dynamically'. I was not sure what it meant so wanted to clear up. I haven't got a Mac/iOS device right now to test it but want to support iOS in the future.

Hello, the same question. I get an error on IOS:
flutter: type 'String' is not a subtype of type 'List?' in type cast
[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: type 'String' is not a subtype of type 'List?' in type cast
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:296:41)

#1      MethodChannelFlutterSystemRingtones.getNotifications (package:flutter_system_ringtones/flutter_system_ringtones_method_channel.dart:35:25)

#2      FlutterSystemRingtones.getNotificationSounds (package:flutter_system_ringtones/flutter_system_ringtones.dart:25:12)


Know when the NonSelected list box is empty

Is there a way to know when the NonSelected list box is empty? I did a $scope.$watch to know when a filter is being applied but I need to know what is left in the list afterwards. More specifically when the list is empty.
        $scope.$watch(
            "vm.settings.filterNonSelected",
            function handleFooChange( newValue, oldValue ) {
                console.log( "vm.fooCount:", newValue );
            }
        );

You need to use the ng-model to do that, and you can evaluate if there's anything selected by checking the array size.

I am probably doing something wrong.The list on the left is the one I need to know when it is empty.Basically if the item I am filtering for is not in the list. The ng-model contains the list of selected items on the right.

I'm sorry but I can't reply via personal email. Please open an issue on the
repo.
Best,
On Fri, Jun 24, 2016 at 1:51 PM jmorrisIII<EMAIL_ADDRESS>wrote:

I am probably doing something wrong.The
https://drive.google.com/open?id=0B7qRPBu8FCiaX3ZRaWJPeFpRSTQ list on
the left is the one I need to know when it is empty.Basically if the item I
am filtering for is not in the list. The ng-model contains the list of
selected items on the right.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/frapontillo/angular-bootstrap-duallistbox/issues/19#issuecomment-228337370,
or mute the thread
https://github.com/notifications/unsubscribe/ABFmDozvqg07vE2ebixCkfX6QI8mpWnQks5qO9LSgaJpZM4IeA9o
.

--
Francesco Pontillo

Open up a new issue? Not sure what the previous comment meant about personal email.

I need to look at a live plunker to help you.
Sorry for the previous email, too.


Fix spelling in January month name

Hi @jawn, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@jawn - Thanks - some other contributors also fixed this, so I'm going to close.


few links don't working

Hi! I tried build an orangepi-radio, but have small problem, need few info and clarification:
-I can't download the h3 consumption tool, because link doesn't work.

I can't download the spotify key, because link doesn't work too.
I can't see, where need connect power switch another wire. I tried GND, but doesn't work. I didn't dare connect to 3,3 or 5 V.
But my biggest problem, I configure anything (except the 2 core and GPU switch off, and certainly the spotify key), wired the display, powered the orangepi, but nothing happened, I tried two working display, the result is same: nothing.
Please help me, if you can.
Sincerely: Istvan from Hungary

Hi Istvan,
it really looks like that the h3 consumption tool is no longer available. But to get this to work this tool is not really needed.
The problem with spotify is because they do no longer support the way spotify-connect-web is talking to spotify. It is still working if you do have an existing key but they do not issue new keys anymore.
Instead of that you can now use the following software https://github.com/librespot-org/librespot
If I have time I will update my project to that.
The power switch is connected to GND (as far as I remenber - I will doublecheck).
Now the display problem. Can you please send the output of i2cdetect -y 0
It should look like this. At 3C is the display.
Thomas

Thanks for help! On week-end I will try it!


UI not loading as part of Hugo module

Hi,
Can you help to troubleshoot what's wrong with using of Sveltia cms as hugo module?
I created an issue https://github.com/privatemaker/headless-cms/issues/2 but I'm not sure if that's hugo module issue or sveltia issue

Please discuss in https://github.com/privatemaker/headless-cms/issues/2, and I’m happy to help if it’s a Sveltia CMS issue.

Yes, privatemaker doesn't seem to be responding on how to fix this. at least from august to October :-). Looking at his partial, I'm no go or hugo expect, but besides a lot of duplicated code in the partial, it looks like it is loading the correct sveltia.js file
https://github.com/privatemaker/headless-cms/blob/main/layouts/_default/headless-cms.html
I confess i don't understand the yaml file that is also there https://github.com/privatemaker/headless-cms/blob/main/layouts/_default/index.headlesscmsconfig.yaml


add imagepullpolicy support.

ref: https://github.com/open-cluster-management/backlog/issues/18192
Signed-off-by: morvencao<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

/assign @clyang82

/lgtm


Mongo driver shouldn't be dependency

I use vagrant for development so don't want to be forced to install the mongo driver on my local machine.
Problem 1
    - Installation request for doctrine/mongodb dev-master -> satisfiable by doctrine/mongodb dev-    master.
    - doctrine/mongodb dev-master requires ext-mongo * -> the requested PHP extension mongo is missing from your system.
  Problem 2
    - Installation request for doctrine/mongodb 1.0.x-dev -> satisfiable by doctrine/mongodb 1.0.x-dev.
    - doctrine/mongodb 1.0.x-dev requires ext-mongo * -> the requested PHP extension mongo is missing from your system.
  Problem 3
    - doctrine/mongodb 1.0.x-dev requires ext-mongo * -> the requested PHP extension mongo is missing from your system.
    - iampersistent/mongodb-acl-bundle dev-master requires doctrine/mongodb 1.0.x-dev -> satisfiable by doctrine/mongodb 1.0.x-dev.
    - Installation request for iampersistent/mongodb-acl-bundle dev-master -> satisfiable by iampersistent/mongodb-acl-bundle dev-master

@jmikola Wondering if any consideration has been given to this in light of hhvm, which includes the classes that doctrine/mongodb uses but does not seem to pass the extension requisite in composer.

@stanlemon We would consider it. What is your suggestion? Make the dependency require-dev? It sounds like this should be something that composer solves.

Personally, I'd rather see this solved in composer. If any other library (and be it hhvm itself) provides the same functionality as ext-mongo, the composer requirement for it should pass, period. As it is now, doctrine/mongodb requires ext-mongo to run, so it should still fail if ext-mongo (or an equivalent) is not present.

For context, Composer can ignore platform requirements as of October 2014 (this comment):

Latest composer now has --ignore-platform-reqs in update and install to ignore php, hhvm and ext-* requirements.

@stanlemon: Regarding "in light of hhvm, which includes the classes that doctrine/mongodb uses", are you referring to: https://github.com/facebook/hhvm/tree/master/hphp/runtime/ext_zend_compat/mongo. I wasn't aware that any code from the extension was imported into HHVM, but that source appears to be about 2 years old.
I initially thought you were referring to mongofill, which I assume most HHVM users may be using at present. That library, which is implemented completely in PHP, happens to provide ext-mongo and satisfy Doctrine MongoDB's Composer dependency.

@jmikola Yes, that's what I had in mind. It's possible it's not usable - I confess that Travis was sort of my avenue to explore it as I have a project that uses ODM. This makes me wonder if there would be an easy enough way to add mongofill for hhvm runs in travis just to see how ODM holds up.

This makes me wonder if there would be an easy enough way to add mongofill for hhvm runs in travis just to see how ODM holds up.

I did just that in https://github.com/bernardphp/bernard/pull/133. The mongofill installation is now handled by tests/travis.sh.


Appstore submission issue - Invalid privacy manifest

We have received an Appstore email mentioning "Invalid privacy manifest" related to the recent build upload with Connectivity version 8.0. Hopefully its due to the empty array in the PrivacyInfo
Email Info -
Although submission for App Store review was successful, you may want to correct the following issues in your next submission for App Store review. Once you've corrected the issues, upload a new binary to App Store Connect.
ITMS-91108: Invalid privacy manifest - The PrivacyInfo.xcprivacy file from the following path is invalid: “Frameworks/Connectivity.framework/Connectivity_Privacy.bundle/PrivacyInfo.xcprivacy”. In addition to the privacy manifest files in the locations outlined in the documentation, starting November 12, 2024, all privacy manifests you submit must have valid content. Keys and values in any privacy manifest must be in a valid format. For more details about privacy manifest files, visit: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files and https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/adding_a_privacy_manifest_to_your_app_or_third-party_sdk.

We received this issue and Apple has rejected the app. We got below response:
ITMS-91056: Invalid privacy manifest - The PrivacyInfo.xcprivacy file from the following path is invalid: “Frameworks/Connectivity.framework/Connectivity_Privacy.bundle/PrivacyInfo.xcprivacy”. Keys and values in your app’s privacy manifests must be valid. For more details about privacy manifest files, visit: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files.
Please fix the issue.

Now we are unable to submit new releases to Appstore due to this issue
"Please correct the following issues and upload a new binary to App Store Connect."
Please provide a fix for this :)

@rwbutler, I'm facing the same issue. Do you have any update on this?

Thanks for tagging me I'll take a look this evening (also f you're happy to raise a PR that's usually fastest)

We are stuck unable to update our app because of this :(

@itsthisjustin I don't believe I can add a NSPrivacyAccessedAPIType key for any of the APIs covered in https://developer.apple.com/documentation/bundleresources/describing-use-of-required-reason-api since the framework doesn't use any of these. It seems that the options available are:

Remove the privacy manifest since the framework doesn't actually use file timestamp APIs, system boot time APIs, desk space APIs, active keyboard APIs or User Defaults.
Remove Reachability

I'll remove the privacy manifest since I can't see a reason why it should be required currently.
I'll also seek to remove Reachability framework likely this weekend.

Have pushed 8.0.1 now, hope this helps. Please could you let me know how you get on?

In case it helps, we ended up temporarily switching to this fork which worked. https://github.com/Cosmos-Entity/Connectivity

Thanks @itsthisjustin it's certainly interesting to hear - your app has been accepted using that manifest? I've noted that it's still essentially an empty manifest without any reason strings present so I think the solution for now is to remove it entirely. I've spoken to engineers who are using the version prior to the manifest being added and they haven't experienced any issues submitting to the store.

I downgraded the version to 6.1.1 and it worked. The is submitted and released. Now will check with the updated version.

Thanks for the feedback @AkshayUdupa, looking forward to hearing how it goes


Labelsin form

Addding new labels to metadata of entities "sometimes"  make the form invalid

Fixed with update in metadata schema


[Doc Improvement][Update the SBS guide for SPFx]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-spfx.yml
94
26
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-spfx.yml
94
26
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit f45b3b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-spfx.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-spfx.yml
94
26
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit c6ed0c4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-spfx.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Utilize dedicated logger for non pipe-run activities

Currently we are not using a logger that persists information across Simple Data Pipe deployments.

Delivered by #40


Create the About Us section

Changes are to be made in index.html, styles.html and index.js files.
About Us Section should contain:

Image: Click here for the image link
Title: About Us.
Content: GirlScript Organization is a non-profit company registered under the government of India on 2nd June 2017. GirlScript aims to empower people and especially underrepresented groups in the field of technology.
GirlScript Asansol's mission is to create opportunity, learning, growth and diversity in technology academia and industry.
Social Media Button : link

Title 2: Our Programs
Content: Encourage people to learn to program and make them capable of writing a simple computer program to building a complex system. Empower people by providing an opportunity to showcase their technical skills.
Card Section:
- Career Guidance
- Open Source
- Workshops and Sessions

Design
Inspo: (You can refer to it and can make your own interpretations too, Creativity is welcomed.)

Please join the Discord to discuss design of the components.

Please assign this to me

I want to work on this issue under hacktoberfest 2022
THANKS!!!

@souravaniket If you would rather like to consider a less cluttered solution, you can add your html/js components in separate folders (with appropriate feature naming) inside components/[feature] instead of the index.html in the root. We'll have it merged later into the main HTML file (we'll put up a separate issue). The contents of [feature] can be: index.html, styles.css, index.js with the body of index.html wrapping the component.

@souravaniket any update about the work?

I would be willing to work on this issue if there are no updates soon! @girlscript-asansol Could you maybe assign it to me if needed? Thanks!

I would be willing to work on this issue if there are no updates soon! @girlscript-asansol Could you maybe assign it to me if needed? Thanks!

Thanks for letting us know.

@souravaniket any updates kindly let us know by today else we have to reassign it.

Please assign this to me

Please look for any other issues as I am working on this. Thanks!

Please assign this to me

Please look for any other issues as I am working on this. Thanks!

I can work on this, kindly assign me this issue.

Yes, @girlscript-asansol. Could you please assign this issue to him as I have completed a similar issue in a different repository, as a result l, I am unassigning this issue from me. Hope you understand. Thanks!

Please assign this to me

Please look for any other issues as I am working on this. Thanks!

I can work on this, kindly assign me this issue.

Yes, @girlscript-asansol. Could you please assign this issue to him as I have completed a similar issue in a different repository, as a result l, I am unassigning this issue from me. Hope you understand. Thanks!

@nub-rin will be carrying forward the rest of my work, that's why I am asking to assign this issue to him.

@RathoreAbhiii Okay, sure.

hey @JASKIRAT11011  Please assign this to me

@nub-rin Any updates? Please respond, or we'll have to reassign. We have other contributors willing to work on this portion.

@nub-rin Any updates? Please respond, or we'll have to reassign. We have other contributors willing to work on this portion.

Hey, sorry for the delay. I will be creating a pr in like 3 hrs from now. Will post an update screenshot for revision in around an hour.

@nub-rin Thank you. We're actually rooting for this one to complete our first milestone. I hope you understand.

@jay-io should the about us have a separate page or integrated in index.html?

@jay-io should the about us have a separate page or integrated in index.html?

Nevermind found it.

@jay-io This is the upper part, the background can be anything (currently its white cause i dont find any image that dose not conflict with the svg above it. The card is work in progress (~15-30 mins). Lmk you opinion.

Cards feel like an overdoing. You may just arrange the paragraphs alternately one after the other on either side of the centre. Like in this sample below. The next paragraph could be towards the left and illustration to the right. For now, put the GSA logo in place of the illustrations.

PS. I don't know why the weird phantom text overlay is here

Cards feel like an overdoing. You may just arrange the paragraphs alternately one after the other on either side of the centre. Like in this sample below. The next paragraph could be towards the left and illustration to the right. For now, put the GSA logo in place of the illustrations.

PS. I don't know why the weird phantom text overlay is here

How about this?

Cards feel like an overdoing. You may just arrange the paragraphs alternately one after the other on either side of the centre. Like in this sample below. The next paragraph could be towards the left and illustration to the right. For now, put the GSA logo in place of the illustrations.

PS. I don't know why the weird phantom text overlay is here

How about this?

Instead of our services it will be our programs and remove the box shawdow (you can give that in the pics tho) and change the images. Ill share some illustration.

Alright!

Looks good! Can you add a bit more padding between two horizontal sections? Also, bring the heading and the content a tad bit closer. You could also increase the font-weight for the section heading text.

For now, you may just add two subsequent sections so that someone can easily append along any new content with your original work. And we'll raise a no-code issue for the content-writing. You may use Lorem Ipsum.

Alright!!


Update shadow_mark.Rmd

Made the point colors match so it looks better. There is also an issue in the .md version where the one that is meant to  have black points has colors .

I pottered around with other files while you were tweaking this and I'm useless with merge conflicts. I'll work on this :-)


Where can I download older versions of OpenDistro ODBC driver & see SQL Plugin & ODBC Driver compatibility matrix

Based on my ES version, I need to install <IP_ADDRESS> ODBC driver. But I'm finding only latest version available here. Where can I download previous versions of ODBC drivers for Mac?
Also, do these ODBC drivers are backward compatible with older versions of SQL Plugins installed on ES node?
Sorry for spamming it here, unfortunately I couldn't find any other way to connect and https://github.com/opendistro-for-elasticsearch/community was redirecting to sample-code.

Hi @sachinadiga, thanks for reaching out. We changed the naming scheme for our various artifacts at least twice, but the team neglected to rename older artifacts/add redirects so that you could just change the version string. I believe the file you want is here: d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/mac/ODFE+SQL+ODBC+Driver+64-bit-<IP_ADDRESS>-Darwin.pkg
No real idea about compatibility, unfortunately. I'd try to use the same version across the board for the plugin and the driver.

Thanks @aetter for the quick help. Appreciate it.
No worries, I'll test the backward compatibility and update here. Unfortunately in my case I've some other restrictions in upgrading the ES version and hence the OpenDistro SQL Plugin on server side.

Update: Installed <IP_ADDRESS> ODBC driver and configured DSN to ES cluster which was having <IP_ADDRESS> plugin installed, I was able to test the connectivity successfully through iODBC Admin UI unlike failed in 1.13. But tableau was not able to connect may be due to incompatibility.
@aetter do you mind giving me URLs to other versions of drivers >1.9 & <1.13. That would be really helpful to try out and overcome any bugs with very first version of ODBC went with tableau support - 1.9.

Sure, try these:

https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/mac/Open Distro for Elasticsearch SQL ODBC Driver 64-bit-<IP_ADDRESS>-Darwin.pkg
https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/mac/Open Distro for Elasticsearch SQL ODBC Driver 64-bit-<IP_ADDRESS>-Darwin.pkg
https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/mac/Open Distro for Elasticsearch SQL ODBC Driver 64-bit-<IP_ADDRESS>-Darwin.pkg

If those don't work, I think troubleshooting is going to be a struggle. My suspicion is that the team's recommendation will be "try it on the latest ODFE version with the matching SQL plugin." Still, best of luck!

Thanks a lot @aetter, it worked. I'm able to create connection with <IP_ADDRESS> driver & <IP_ADDRESS> Plugin and tableau is compatible with these configs.

@aetter Do you know the windows url for <IP_ADDRESS> version?

@aetter requesting help with windows url for <IP_ADDRESS> version?
I am using AWS hosted Opensearch 1.0 however, the latest version 1.13 is not working with FGAC enabled domains.

Hi all, I actually can't track these download links down, but @ashwinkumar12345 and @keithhc2 might be able to help.

Hey all, apologies for the delay in getting back to you. Here are the links to the <IP_ADDRESS> drivers:
https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/windows/old.artifacts/Open+Distro+for+Elasticsearch+SQL+ODBC+Driver+64-bit-<IP_ADDRESS>-Windows.msi
https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-clients/opendistro-sql-odbc/mac/Open+Distro+for+Elasticsearch+SQL+ODBC+Driver+64-bit-<IP_ADDRESS>-Darwin.pkg


User Story 11: Select a customer from the previously retrieved list

As a User, I want to select a customer from the previously retrieved list so that I can view more details about the particular customer.

Feature already implemented in initial implementation.
Issue closed.


Redis Plugin

https://github.com/amplication/amplication/issues/1962

Taking care of this one in #347.


Location.identifier for VTrckS

https://github.com/inferno-community/fsh-smart-scheduling-links/blob/main/input/fsh/vaccine_location.fsh#L12 should ideally enforce the inclusion of at least one identifier with VTrckS PIN

I attempted to add an appropriate slice for this in #4. Let me know if you think it needs more!


Add Heroku Instance to Build Pushes

Setup heroku web hooks

Author(s): Developer
Status: Awaiting review

Introduction
Heroku is a highly popular, free starter, and easy to use server solution that we've committed to supporting. This proposes that we setup an instance to build master and if possible PRs so that we can be confident deploys are working.
Ideally in future, we'd automate this behavior on multiple server platforms.
Motivation
Allows us to quantify current Heroku status. Will hopefully encourage server automations so we can make it as easy to check status.
Proposed solution
Set it up for master and investigate PRs
Code snippets
n/a
Impact
It will be able to be deployed to Heroku more confidently.
Alternatives considered
--
Decision (For Moderator Use)
On [Date], the community decided to (TBD) this proposal. When the community makes a decision regarding this proposal, their rationale for the decision will be written here.

Does this mean adding Heroku to the travis file or is this a separate service?

It's a separate tool, building master is easy, to build prs we need to commit a config file, that's why I held off temporarily.

This is now included in master. PRs will build on generated urls, master will be visible here: https://vapor-build.herokuapp.com


Rename urlParams -> queryParams

This does remind me that we should probably write some tests for this class...


Bump deps and remove remark and strip-markdown

Fixes #3360. Fixes #3615.

Changelog Entry
Fixed

Fixes #3360 and #3615. Use channelData['webchat:fallback-text'] field for screen reader text, before stripping Markdown from activity.text field, by @compulim, in PR #3917

Changed

Bumped all dependencies to the latest versions and sample bumps, by @compulim in PR #3831, PR #3846, and PR #3917

Production dependencies

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Description
Bump dependencies with npm audit, and a few more:

<EMAIL_ADDRESS><EMAIL_ADDRESS>
We removed remark and strip-markdown. They are used to extract screen reader text from activity:

Yesterday, we stripped Markdown syntax from activity.text field using remark and strip-markdown packages
Tomorrow

If activity.channelData['webchat:fallback-text'] field is available, we will use it as-is
Otherwise, we will strip Markdown from activity.text field with best effort

Render text using useRenderMarkdown hook into HTML string
Use new DOMParser().parseFromString() to parse HTML string as HTMLDocument
Walk the HTMLDocument and concatenates all text nodes

Please refer to ACCESSIBILITY.md for details.
Design
Considerations on removing remark
Previously, we use remark to implement the "extract screen reader text from Markdown" feature. But the feature itself is causing a few issues: #3360 and #3615.
Although remark@10 has vulnerability in their dependencies and they fixed it in remark@12, they no longer support IE11.
We have a newer logic for stripping Markdown syntax. It will work with HTML tags with best effort. However, bot developers are strongly recommended to provide speak field for precise narration. See ACCESSIBILITY.md for details.
Specific Changes

Bumped dependencies with npm audit and more:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Removed remark and strip-markdown to extract screen reader text from text field

Instead, we will extract screen reader text from channelData['webchat:fallback-text'] field, fallback to remove Markdown syntax from text field with best effort
Strip Markdown logic will use new DOMParser().parseFromString() and walk the HTMLDocument manually

[x] I have added tests and executed them locally
[x] I have updated CHANGELOG.md
[x] I have updated documentation

Review Checklist

This section is for contributors to review your work.

[x] Accessibility reviewed (tab order, content readability, alt text, color contrast)
[x] Browser and platform compatibilities reviewed
[x] ~CSS styles reviewed (minimal rules, no z-index)~
[ ] Documents reviewed (docs, samples, live demo)
[x] ~Internationalization reviewed (strings, unit formatting)~
[x] package.json and package-lock.json reviewed
[x] ~Security reviewed (no data URIs, check for nonce leak)~
[x] Tests reviewed (coverage, legitimacy)

Ready for next pass.


WIP: test: [ui-automation] credential details screen

closes #1115
Signed-off-by: Rolson Quadras<EMAIL_ADDRESS>

The tests are failing due to https://github.com/trustbloc/wallet/issues/1124


Add optional extensions to RPZ file format

In addition to the DATA and METADATA, we need a place to have other payloads to be stored in the RPZ. An example application is the ReproZip News App project, which needs to store a WARC file.

I found (Adobe's) Universal Container Format (UCF), used by Research Objects. Is also the basis of EPUB.
It looks overly complicated though (and gone fr.om Adobe's website?) with XML, JSON-LD, and multiple layers of manifests.