Add support for preload meta tags

Great work on this library! Seems like an elegant solution to a very hairy problem space. I'd like to add support for rendering preload meta tags on the server to shorten the request waterfall.
Something like chunk-manifest-webpack-plugincould probably be used to achieve this, but I thought I'd ask if this is something you have already considered before to trying to build something myself.
I get that this might be outside the scope of this module, buy having a recommended/suggested way of doing this would be good.
Cheers

Hi @mrlundis,
First thank you for the compliment.
Yes I considered to add support for this, I think it's a very good feature. The philosophy of this library is to be as simple as possible for the user. We have to get Webpack stats, yes. But the biggest problem to solve is linking a component to a webpack chunk, and it's not easy.
I don't want to couple this library to Webpack. If you have an idea to do it without adding complexity it could be interesting. I am thinking about it but I didn't find a good solution yet.

I have just given another look, to clarify, server-side we do not have any idea of the corresponding Webpack chunk. The key to achieve it is to be able to know server-side the chunk id corresponding to the import. I don't know the best way to do it.

Maybe we can solve this by adding a companion webpack plugin? I don't think this package would need any significant changes, maybe just an option to output the preload tags given a manifest.
I know that Vue projects has prefetch tags working out-of-the-box. Their webpack setup is documented here:
https://github.com/vuejs/vue-ssr-docs/blob/master/en/build-config.md and their plugin source is available here: https://github.com/vuejs/vue/tree/dev/src/server/webpack-plugin

@mrlundis a Webpack plugin is not sufficient because we have to know it server-side, and server-side Webpack is not present. Do not hesitate to try it, if it is OK I will merge it!

What I meant was that a companion webpack plugin could be used to generate a manifest mapping the module ids to the correct bundles/chunks. That manifest could then be used by the server to generate the correct preload tags.
Will send a PR as soon as I get around to implementing this, will probably take some time though. Have a few other big tasks to tackle first.

Hey there. I have another problem, but related. So I did prerendering, and I have code splitting and I have <link rel="preload"> to speed up download. But the problem is that async component loading triggers redraw of pre-rendered content. Do you have solution for this?
P.S. Full code here https://github.com/stereobooster/an-almost-static-stack, I use react-loadable, not loadable-components, but I suppose it will be the same for any "loadable" solution.

@stereobooster no I am sorry loadable-components does not support preloading yet.

I poked a bit more. You are actually doing excellent work with getLoadableState. This is not a solution for my case because I do snapshots, not a SSR, but anyway this is totally brilliant.

@stereobooster thank you! This is inspired from Apollo server-side rendering, I found the idea brillant and I copied it 🤗.

This feature requires a strong dependency with Webpack and increase the setup complexity. It is actually possible to prefetch in JavaScript. Until a simplification in the Webpack ecosystem I don't plan to support preload meta tags.

I think preloading not so important if:

You are using manifest chunk and preload it
You are using vendors (biggest) chunk and preload it
Setup correct chunk size with https://webpack.js.org/plugins/min-chunk-size-plugin/
And also limit their amount with https://webpack.js.org/plugins/limit-chunk-count-plugin/


Integrated OC_SORT Algorithm into X-AnyLabeling for Enhanced Annotation

Hello,
I've integrated the latest OC_SORT algorithm into X-AnyLabeling, a high-level automatic annotation tool with multiple state-of-the-art model integrations. I wanted to bring this to your attention so that others can explore and experience the integration.
Great work on OC_SORT!
Best regards,
CVHub

Thanks for your work and contribution with OC-SORT! @CVHub520


Namespace filter is broken

The namespace filter doesn't show all the namespaces in the cluster and doesn't filter pod names.
Below screenshot shows jsearch-elk-kibana namespace typed manually with metrics showing, but is not in the dropdown.

Grafana seems to have issues with updating templates. If you go to manage dashboard, click on templates, edit namespace templates and click on update the new namespace will pop in. I will take a look on how to workaround this problem.
As of pods not being filtered - there is no show tag values filtering possible in influxdb. But we can create an additional tag for time serieses that will contain namespace and pod name concatenated together.

+1 for pod filtering

Is this related to this issue?  https://github.com/kubernetes/heapster/issues/1120

Having same issue - just default and kube-system displayed. @mwielgus - where to find "templates" - could find out location of namespace templates.
By adding query-parameter var-namespace e.g.
?var-namespace=dev
we are able to drill in the namespace.

This issue appears to be resolved now.


Script upload for protocol-mappers deprecated in Keycloak 7.0.1+

Describe the bug

Until now, administrators were allowed to upload scripts to the server through the Keycloak Administration Console as well as through the RESTful Admin API.
For now on, this capability is disabled by default and users should prefer to deploy scripts directly to the server. For more details, please take a look at JavaScript Providers.

https://www.keycloak.org/docs/latest/release_notes/index.html#keycloak-7-0-1-final
Our current entrypoint script relies on this functionality in services/keycloak/start.sh, in the configure_opendistro_security_client function.
#1358 overrides the new default to allow script upload functionality for now, but we should use the new recommended method which involves rolling .jar files.
To Reproduce
Steps to reproduce the behavior (without the changes in #1358):

build lagoon locally completely from scratch (docker system prune -af first)
run make up
observe errors on keycloak start

Expected behavior
Keycloak should start successfully.
Screenshots
n/a
Additional context
See the link above for details about what we should be doing instead of uploading scripts.

resolved in v2.11.0


[SPARK-5064][GraphX] Add numEdges upperbound validation for R-MAT graph generator to prevent infinite loop

I looked into GraphGenerators#chooseCell, and found that chooseCell can't generate more edges than pow(2, (2 * (log2(numVertices)-1))) to make a Power-law graph. (Ex. numVertices:4 upperbound:4, numVertices:8 upperbound:16, numVertices:16 upperbound:64)
If we request more edges over the upperbound, rmatGraph fall into infinite loop. So, how about adding an argument validation?

Can one of the admins verify this patch?

ok to test

Test build #25240 has started for   PR 3950 at commit d760bc7.

This patch merges cleanly.

Test build #25240 has finished for   PR 3950 at commit d760bc7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25240/
Test PASSed.

Thanks! I also added a unit test. Merged into master & branch-1.2.


[Schema] Fix publish schema

Description
This PR fixes #
Removed comma after Kubernetes in technology enum, which was causing error
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Merging this PR as this is small change

Thanks for this, @Yashsharma1911 👍


Broom icon

🎨 Visual Design Request
Can we add a broom icon?
📌 Request Background
We need a icon that suggest "clean".
🎯 Core Requirements
To look like a broom.
🍆 Non-requirements
It should be made out of wood not plastic.
🥅 Acceptance Criteria

[ ] Visual Design Spec submitted (link)
[ ] Visual Design Spec reviewed by design team
[ ] Visual Design Spec reviewed by dev team

Hi @MileaRobertStefan
We don't use a broom similar to LabVIEW for "clean-up" in our most modern iconography, the team has been using two-four-point-stars to indicate clean.  In "Fuse" (our older design system, but still mostly modern metaphors) a broom has been used for "clear" in certain context, but we've since moved to better more standard metaphors around clear, delete, remove.
I'm planning for clean or clean up to remain these sparkly stars vs a broom.
Here's the revised version I've created to include in Nimble:

Let me know this will cover your needs and I'll get it implemented as a token for you.
Thanks,
Brandon
CC: @jattasNI  @fredvisser

Looks good!

@fredvisser @jattasNI Please add to token library.

Tags: Clean, Clean up, Stars, Sparkle

@MileaRobertStefan the icon above is now available: nimble-icon-sparkles.


ENOENT, no such file or directory 'coverage/'

I really don't know why this happens.
I use a shellscript to run the tests:
"$PWD"/node_modules/.bin/mocha \
  -b \
  --compilers coffee:coffee-script/register \
  --require coffee-coverage/register-istanbul \
  --require should \
  --reporter spec \
  --timeout 120000 \
  --slow 300 \
  "$@"

You can see all the code here
The problem is that   --require coffee-coverage/register-istanbul line throw an Error related with the coverage folder:
✓ release a new version and hook pre releases plugins in order (163ms)

  16 passing (300ms)

Failed to write coverage data Error: ENOENT, no such file or directory 'coverage/'
  at Error (native)
  at Object.fs.mkdirSync (fs.js:747:18)
  at exports.mkdirs (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/coffee-coverage/lib/utils/helpers.js:43:14)
  at process.<anonymous> (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/coffee-coverage/lib/register.js:116:11)
  at process.emit (events.js:107:17)
  at process.exit (node.js:600:17)
  at done (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/mocha/bin/_mocha:406:32)
  at afterWrite (_stream_writable.js:361:3)
  at _stream_writable.js:349:9
  at process._tickCallback (node.js:355:11)

the strange thing is that the folder doesn't exist before I run the script (good) and after the exceptions exists.
Any idea of what happens?

No.  I'm doing (almost) exactly this, and I'm not seeing this issue.  The error is in mkdirs, too, which is supposed to be creating folders.  Is $PWD in fact, the current working directory here?  Or is this part of a script that's setting $PWD and then changing the directory?
You can add some debug logging in ./node_modules/coffee-coverage/lib/helpers.js, inside exports.mkdirs(), and see if you can figure out the full path of the folder it's trying to create.  Feels like some kind of relative path problem...

You're right. Some debug logs before call mkdirs method:
if (options.writeOnExit) {
      return process.on('exit', function() {
        var dirName, ref1;
        try {
          dirName = path.dirname(options.writeOnExit);
          console.log('DEBUG ::');
          console.log(dirName);
          console.log(process.cwd());
          console.log(__dirname);
          mkdirs(dirName);
          return fs.writeFileSync(options.writeOnExit, JSON.stringify(global[options.coverageVar]));

output:
DEBUG ::
coverage
/Users/josefranciscoverdugambin/Projects/bumped/bumped/test/plugin_directory
/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/coffee-coverage/lib
Failed to write coverage data Error: ENOENT, no such file or directory 'coverage/'
  at Object.fs.mkdirSync (fs.js:654:18)
  at exports.mkdirs (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/coffee-coverage/lib/utils/helpers.js:48:14)
  at process.<anonymous> (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/coffee-coverage/lib/register.js:154:11)
  at process.emit (events.js:95:17)
  at process.exit (node.js:740:17)
  at done (/Users/josefranciscoverdugambin/Projects/bumped/bumped/node_modules/mocha/bin/_mocha:406:32)
  at afterWrite (_stream_writable.js:278:3)
  at _stream_writable.js:267:9
  at process._tickCallback (node.js:448:13)


Support query string encoding

I'm doing this here in case it's of any use: https://github.com/the-grid/Disc/blob/0.2.1/Disc/Networking/Helpers/Request.swift#L6-L19

Looks like iOS actually provides a nice way to do this now https://littlebitesofcocoa.com/128-nsurlqueryitem-nsurlcomponents?utm_source=Little+Bites+of+Cocoa&utm_campaign=2598619d3c-005_watch_screenshot_magic5_21_2015&utm_medium=email&utm_term=0_2b6df83b4b-2598619d3c-111012021

Yep, that's what I used.
I just mapped from [String: String] and then sorted by key because unit tests were failing (only when running from the command line) due to different ordering.
I don't remember why I checked for empty query items but it was probably to prevent a redundant trailing ?.

hahaha I didn't actually get around to reading that snippet yet, my bad. I was just brain dumping that link here since I just read about it.

Given that, I'm going to close this.


Feat/evs support

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature: Adds support for EVS content type
Other information:
Status

[X] Code documentation for the relevant parts in the code have been added/updated by the PR author
[X] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #563 (50b0ae3) into release37 (27f73aa) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release37     #563      +/-   ##
=============================================
- Coverage      67.56%   67.46%   -0.11%
=============================================
  Files            258      261       +3
  Lines          17678    17846     +168
  Branches        3999     4035      +36
=============================================
+ Hits           11944    12039      +95
- Misses          5695     5766      +71
- Partials          39       41       +2

Impacted Files
Coverage Δ

meteor/server/migration/databaseMigration.ts
60.27% <0.00%> (-6.50%)
:arrow_down:

meteor/server/security/showStyle.ts
46.57% <0.00%> (-5.10%)
:arrow_down:

meteor/server/migration/1_35_0.ts
62.50% <0.00%> (-1.50%)
:arrow_down:

meteor/server/api/blueprints/api.ts
78.74% <0.00%> (-0.63%)
:arrow_down:

meteor/lib/lib.ts
79.76% <0.00%> (-0.44%)
:arrow_down:

meteor/server/api/peripheralDevice.ts
71.00% <0.00%> (-0.31%)
:arrow_down:

meteor/server/api/blueprints/migrationContext.ts
99.32% <0.00%> (-0.29%)
:arrow_down:

meteor/lib/api/migration.ts
100.00% <0.00%> (ø)

meteor/server/migration/0_19_0.ts
57.54% <0.00%> (ø)

meteor/server/migration/1_36_0.ts
100.00% <0.00%> (ø)

... and 15 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 27f73aa...50b0ae3. Read the comment docs.


Support autonaming in the Environment resource

It would be nifty if the Environment resource supported autonaming and if the name attribute was optional.

I think this is already captured in https://github.com/pulumi/pulumi-pulumiservice/issues/110 so I'm gonna close this out as a dup


Fix Shellcheck warnings

Description
After change travis to github actions, Shellcheck are warning some issues.
https://github.com/jasperes/bash-yaml/runs/1626353860?check_suite_focus=true

I will take care of this!
Before I begin, I have a question though. Is there a reason that you use a specific Shellcheck version in lint.yaml?
We could do the following:
sudo apt-get -y update && sudo apt-get -y install shellcheck
find $GITHUB_WORKSPACE -type f -and \( -name "*.sh" \) | xargs shellcheck

This grabs the Shellcheck version from the repo and runs it against every *.sh file in the project.
What do you think?

I will take care of this!
Before I begin, I have a question though. Is there a reason that you use a specific Shellcheck version in lint.yaml?
We could do the following:
sudo apt-get -y update && sudo apt-get -y install shellcheck
find $GITHUB_WORKSPACE -type f -and \( -name "*.sh" \) | xargs shellcheck

This grabs the Shellcheck version from the repo and runs it against every *.sh file in the project.
What do you think?


Make SSplotComparisons subplot 14 the same as others

It would be nice to make the x-axis of this plot match the others. It is the survey index one (subplot 14).
See bridging plot - Fig 17, and sensitivity plots 44 and 50. I know the survey was in 2019 but it would be nice to make the plot go to 2020.

I think that we can pass "endyrvec" to SSplotComparisons to get it to work. Should we pass the same endyr to all plots so that they are more comparable as you scroll through pages?

Sure, except the survey index should probably start in 1995 or the plot will be crazy small. endyrvec could end with assess.yr

I can do this if you're busy Kelli, it's just cosmetic anyway

r4ss doesn't currently change the x-axis of the index plot. I changed r4ss, but I am worried about the result

because if you remove the padding (i.e., xaxs = "i", ) then the points are hard to see if they are at the extreme ends of the axis. What do you think @cgrandin?

I couldn't get it to change easily by passing 2020 to endyrvec and since you can't make it happen easily I would say we can drop it for now.  Thanks for checking but I'll close it.


Misclassification: Right intent but wrong entity

Hi,
I have some training examples for entity1 in form of [word]+[Mix of numbers and letters]
and for another entity2 just numbers like "12.4".
Now if I parse just "12.6" eg. it is recognized as entity1, though it should be entity2 of course!
How can I prevent these misclassifications for entities?

You probably don't have enough training examples. Furthermore, do they occur in the same sentence structure? If so, this might be a reason why they get confused.


watch of *v1.EndpointSlice ended with: an error on the server

During a run of Scale test upstreams NFR, NGF generated the following error entries:
W0522 16:32:33.918224       7 reflector.go:470] pkg/mod/k8s.io/client-go@v0.30.1/tools/cache/reflector.go:232: watch of *v1.EndpointSlice ended with: an error on the server ("unable to decode an event from the watch stream: got short buffer with n=0, base=4092, cap=20480") has prevented the request from succeeding
. . .
W0522 16:32:55.524513       7 reflector.go:470] pkg/mod/k8s.io/client-go@v0.30.1/tools/cache/reflector.go:232: watch of *v1.EndpointSlice ended with: an error on the server ("unable to decode an event from the watch stream: got short buffer with n=0, base=4092, cap=81920") has prevented the request from succeeding

Full log https://github.com/nginxinc/nginx-gateway-fabric/blob/d443d6a55988eae8091d230dbbd90ceef1de2c8f/tests/results/scale/edge/TestScale_UpstreamServers/ngf.log
Acceptance criteria:

Figure out if this causes or can cause any issues (like improper configuration, outages).

How often does this occur? Have we seen this elsewhere?


chore: fix typescript error in nextjs-template

This PR aims to revert a change from https://github.com/dendronhq/dendron/pull/3533 which is downgrading @types/react.
The cause of if it basically a missmatch between antd still depending on an older version.
This commit adds also missing yarn.lock changes that came in through upgrading next version in the above mentioned PR.
Pull Request Checklist
If you are a community contributor, copy and paste the PR template from Dendron Community PR Checklist and add it to the body of the pull request.
If you are a team member, copy and paste the template from Dendron Extended PR Checklist.
To copy the template, click on the Raw button on the gist to copy the plaintext version of the template to this PR.

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

@namjul  - I added a few more fixes to the playwright test pass in this PR: https://github.com/dendronhq/dendron/pull/3573


Feature requests on searchable reference selector

Hi Benjamin,
Great job on this widget, it's exactly what I was looking for in my project! Thanks!🙏
I have two suggestions. Firstly, the dropdown doesn't update its position when you scroll the page. There should probably be a scrollListener that calls updatePosition so the dropdown doesn't stick to its x and y but rather adjusts to the position of the input field. Secondly, it'd be great if we could configure an 'On Click' action for the badge, so we could let the user open the corresponding edit page for the referenced object.
Cheers,
Marius

Hi Marius,
Thank you for the feedback! I haven't tried the scrollListener before, but it'll be worth a shot. For the second, this is a great idea! I'll do some experiments to see if the pluggable widget library will let me do this.
Thank you!
Ben

Hi Benjamin, in case you'd like to be able to support the horizontal rendering of radio buttons, I did it this way:
.mx-name-searchableReferenceSelectorMx92.radio-button-horizontal .srs-list {
display:flex;
flex-direction: row;
}
and added the class radio-button-horizontal to the widget properties. Works nicely and had been a missing piece to finally kick the legacy widget RadioButtionAssociations out.
Cheers, Boris

@Maismaus I have a fix for the scroll behavior and on-click badge coming soon. I've got the code ready ~ only needs just a bit more testing.
@Boris-Scharinger Nice find! Do you have a picture of what the horizontal radio buttons look like (so I can better understand the use case)?

v1.4

changed text settings to textTemplates to support different languages
the dropdown popup now updates its position when scrolling
added the ability to set an action when a reference set's badge is selected


Update new site content

Hey @lorforlinux
This content was removed while updating the documentation site from docusaurus to sphinx.
Please have a look.

@bhawnasehgal Thanks for doing this, I will merge it now!


fatal git errors when updating toolkit

The following is the result of running Maintenance -> Update ToolKit after a clean install on ubuntu 16.04:
AtoMiC ToolKit Updater

This will overwrite any manual changes you made to the script.
Type y/Y and press [ENTER] to continue with update or any other key to exit: y

--->Refreshing packages list...
Hit:1 http://us.archive.ubuntu.com/ubuntu xenial InRelease
Get:2 http://us.archive.ubuntu.com/ubuntu xenial-updates InRelease [102 kB]
Get:3 http://us.archive.ubuntu.com/ubuntu xenial-backports InRelease [102 kB]
Get:4 http://security.ubuntu.com/ubuntu xenial-security InRelease [102 kB]
Ign:5 http://download.opensuse.org/repositories/home:/emby/xUbuntu_16.04  InRele                                                                                ase
Hit:6 http://download.opensuse.org/repositories/home:/emby/xUbuntu_16.04  Releas                                                                                e
Fetched 306 kB in 0s (469 kB/s)
Reading package lists... Done

--->Installing prerequisites...
Reading package lists... Done
Building dependency tree
Reading state information... Done
git-core is already the newest version (1:2.7.4-0ubuntu1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

--->Stashing any local changes...

--->Git stashing any local changes made to ...
fatal: not in a git directory
fatal: not in a git directory
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

--->Updating AtoMiC ToolKit...
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
AtoMiC ToolKit updated successfully.

Press [Enter] key to continue or Ctrl+C to exit...

This has now been fixed. You might have to do a manual 'sudo git pull' in the AtoMiC-ToolKit the first time you update to get this fix.


update confirmations for different collection tabs are consistent (bug 594232)

Different tabs in collections.edit would show different confirmation message on submitting their respective forms. Confirmation message for every tab is the same and consistent. Update confirmation for every tab is the same as the current update confirmation message for the "Descriptions" tab.
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=594232

Needs review.

Needs review.

Commits squashed. Please review.

64c5df6


Manually specifying startup hook can be error prone

Take the following minimal deployment that enables the experimental exceptions feature (ignore the fact that exception collection doesn't work on the samples image due to trimming of startup hooks):
apiVersion: apps/v1
kind: Deployment
metadata:
  name: deploy-exampleapp
spec:
  replicas: 1
  selector:
    matchLabels:
      app: exampleapp
  template:
    metadata:
      labels:
        app: exampleapp
    spec:
      restartPolicy: Always
      containers:
      - name: app
        image: mcr.microsoft.com/dotnet/samples:aspnetapp
        imagePullPolicy: Always
        env:
        - name: ASPNETCORE_URLS
          value: http://+:80
        - name: DOTNET_DiagnosticPorts
          value: /diag/dotnet-monitor.sock
        - name: DOTNET_STARTUP_HOOKS
          value: /diag/libs/8.0.0-preview.3.23180.9/any/net6.0/Microsoft.Diagnostics.Monitoring.StartupHook.dll
        volumeMounts:
        - mountPath: /diag
          name: diagvol
      - name: monitor
        image: mcr.microsoft.com/dotnet/nightly/monitor:8.0.0-preview.3
        args: [ "collect", "--urls", "http://localhost:52323", "--metricUrls", "http://+:52325", "--no-auth" ]
        imagePullPolicy: Always
        env:
        - name: DOTNETMONITOR_DiagnosticPort__ConnectionMode
          value: Listen
        - name: DOTNETMONITOR_Storage__DefaultSharedPath
          value: /diag
        - name: DOTNETMONITOR_InProcessFeatures__Enabled
          value: 'true'
        - name: DotnetMonitor_Experimental_Feature_Exceptions
          value: 'true'
        volumeMounts:
        - mountPath: /diag
          name: diagvol
      volumes:
      - name: diagvol
        emptyDir: {}

Note that the path of the startup hook is the following and contains the exact version of .NET Monitor:
/diag/libs/8.0.0-preview.3.23180.9/any/net6.0/Microsoft.Diagnostics.Monitoring.StartupHook.dll

Needing to specify a path that contains the .NET Monitor version is problematic; for any update to .NET Monitor, this version number will change. The vast majority of users will be using a floating tag (e.g. mcr.microsoft.com/dotnet/nightly/monitor:8-preview) whose version changes for every release. The target application will crash if any of the startup hook paths do not exist:
Unhandled exception. System.ArgumentException: Startup hook assembly '/diag/libs/8.0.0/any/net6.0/Microsoft.Diagnostics.Monitoring.StartupHook.dll' failed to load. See inner exception for details.
 ---> System.IO.FileNotFoundException: Could not load file or assembly '/diag/libs/8.0.0/any/net6.0/Microsoft.Diagnostics.Monitoring.StartupHook.dll'. The system cannot find the file specified.

File name: '/diag/libs/8.0.0/any/net6.0/Microsoft.Diagnostics.Monitoring.StartupHook.dll'
   at System.Runtime.Loader.AssemblyLoadContext.<LoadFromPath>g____PInvoke|5_0(IntPtr ptrNativeAssemblyBinder, UInt16* ilPath, UInt16* niPath, ObjectHandleOnStack retAssembly)
   at System.Runtime.Loader.AssemblyLoadContext.LoadFromAssemblyPath(String assemblyPath)
   at System.StartupHookProvider.CallStartupHook(StartupHookNameOrPath startupHook)
   --- End of inner exception stack trace ---
   at System.StartupHookProvider.CallStartupHook(StartupHookNameOrPath startupHook)
   at System.StartupHookProvider.ProcessStartupHooks()

We need determine a way of creating a stable path (independent of the .NET Monitor version) so that it is predictable, more easily understood by a user, and doesn't change for minor and patch releases.

This feature was completed with the release of .NET 8 and .NET Monitor 8.


Cannot find any information on using the Build.rakumod module

I know it's in the docs somewhere, but searching for 'Build' doesn't find it for me.

Search for build, scroll down to "Build.pm6". That'll at least get you the docs right now.

Thanks!
On Sun, Jun 23, 2024 at 16:51 Will Coleda @.***> wrote:

Search for build, scroll down to "Build.pm6". That'll at least get you the
docs right now.
—
Reply to this email directly, view it on GitHub
https://github.com/Raku/doc-website/issues/400#issuecomment-2185333727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH5VVFFPF7DVOBNTYJUIVDZI47ODAVCNFSM6AAAAABJYWVO32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBVGMZTGNZSG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Tweak needed to Schematron controlling periods at the end of titles

We've had a case where a monograph title genuinely ends with a period because its last component is the abbreviation ff. This needs to be added as a special case to our rule about periods at the end of titles so it's not made invalid.

This was fixed a couple of weeks ago. Closing.


Clean the trace after creating a layer?

Is it possible to clean the trace, once a new item is created?
map.on('pm:create', function(e) {
{my logic}
map.pm.clearDraw('Line');
});

@josejaner you mean deleting the drawn layer?
Just like you would with regular leaflet:
map.on('pm:create', function(e) {
  {my logic}
  var myLayer = e.layer;
  map.removeLayer(myLayer);
});

Thanks for your interest and help, it's an excellent project.

did my comment solve your problem? Let me know if there is an issue with leaflet.pm! 👍


Are there plans to (attempt) to support Elden Ring after it releases?

I ask because after viewing the previews I worry it has the potential to end up like DS3 where trolls endlessly invade your co-op sessions making it nearly impossible for a couple of newbs to enjoy co-op together.

Maybe at some point in the distance future, but definitely not on release.


Add Changelog for 1.1.0

@msau42, PTAL
Note: not bumped to 2.0, IMO "action needed" is enough

/hold
waiting for cluster registrar deprecation in  https://github.com/kubernetes-csi/cluster-driver-registrar/issues/44 to get resolved

/approve
/lgtm

/LGTM

/approve


Update QT version to 5.14.2

In preparation for re-enabling pyside2 support.
Updated bv_qt.sh
Update  a couple of the config-site files.
Fixed a few warnings regarding deprecated QT methods.
Updated release notes.
Update regresstion test script for pascal.
I compiled and ran test suite on pascal ( a couple of test may need rebaselining due to pixel diffs.)
I compiled on Windows.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have updated the release notes
[X] New and existing unit tests pass locally with my changes

CI is failing due to the source code changes to remove QT deprecation warnings. Should I revert those changes until qt is updated for CI?

How much work is it? My preference would be to update the CI. I will talk to Cyrus about exactly what is involved. I can create a new container pretty quickly.

@cyrush can you take a look at this?

I don't want other people's PR's to fail CI. What does it take to update the docker image?

there are scripts to build the docker container in scripts/ci/docker, I am trying it now from your branch -- I'll double check they pick up the new qt.

@cyrush, is this good to go yet? Is there a formal process for updating the docker image when we need to add new TP libs?

@griffin28 I appreciate your diligence in re-approving after merges from develop into this branch, but I think you can wait for @cyrush to finish the changes to CI that will make this PR ready for merge.

So it isn't running any tests on circleci -- but there is an error b/c of the lack of config.
I would ignore the results from circle. We should still be able to merge when the time is right.
After we are sure things are in order to fully to transition, I will turn off circle. (I assume after this PR is done)

We had a pass just before Kathleen's merge a few minutes ago of develop into this branch. I think we are ready to go.

We should probably decide on the strategy for merging this in and turning off CircleCI or we will continue to get hit with changes to develop and never get over the finish line.

@brugger1 I agree.

Shall I merge now?

I think so, let's check with @cyrush and see if he can you turn off CircleCI after you commit.

@cyrush is it okay to merge?


[Feature]: Handle GraphQL errors

Feature scope
Taps (catalog, state, stream maps, etc.)
Description
As documented in https://www.apollographql.com/docs/react/data/error-handling, GraphQL errors are included as an array in the response:
{
  "errors": [
    {
      "message": "Cannot query field \"nonexistentField\" on type \"Query\".",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "extensions": {
        "code": "GRAPHQL_VALIDATION_FAILED",
        "exception": {
          "stacktrace": [
            "GraphQLError: Cannot query field \"nonexistentField\" on type \"Query\".",
            "...additional lines..."
          ]
        }
      }
    }
  ],
  "data": null
}

(getting JSON-RPC flashbacks 💀)
It'd be good to handle these as part of the GraphQL.valildate_response implementation.
See https://github.com/MeltanoLabs/tap-github/pull/181

To replicate what I wrote in the issue linked above, graphql seems to often (because the spec isn't clear about it) return HTTP 200 status codes even if an error is found.
So the checks in the RESTClient aren't enough to catch errors.
As an example, I found the issue with tap-github where the API was responding with "errors": "Your token has insufficient scope for this request" but this was ignored by the tap, leading to a hard to track problem, with no error and no data.
The spec clarified that a key errors should be present at the root of the JSON response (next to data if the latter is present), and each item in the array of errors should at least contain a message key. The fact that both keys data and errors can be present in parallel tells me the sdk should probably proceed with the normal handling of data even if an error is detected, and maybe raise a fatal error if there is an error but no data.
What do you think?
Obligatory  meme:

To replicate what I wrote in the issue linked above, graphql seems to often (because the spec isn't clear about it) return HTTP 200 status codes even if an error is found.

@laurentS Even some allegedly REST APIs do that 🤦 and it was part of the motivation for adding RESTStream.validate_response (even had to add the same meme to the docs 😅)

The spec clarified that a key errors should be present at the root of the JSON response (next to data if the latter is present), and each item in the array of errors should at least contain a message key. The fact that both keys data and errors can be present in parallel tells me the sdk should probably proceed with the normal handling of data even if an error is detected, and maybe raise a fatal error if there is an error but no data.

Yeah, that seems relatively easy to implement in a new GraphQLStream.validate_response. It should still probably call super().validate_response to catch HTTP status errors.
I added the https://github.com/meltano/sdk/labels/Accepting Pull Requests in case you or someone else is interested in contributing.

Still relevant


Format codebase

I saved all the files and ran the eslint on the codebase.
Depends on https://github.com/fenjalien/obsidian-typst/pull/62 being merged first to prevent merge conflicts

Its fair that I've not been formatting the code but I can't stand tabs, 2 spaces only please.

As requested, I've changed everything to use two spaces.


feat(create-contentful-studio-experiences): add more validation to CLI tool [ALT-991]

Purpose

Checks to see if the Experience Type already exists before creating one. (ALT-994)
Checks to see if there is already an Experience Entry for the homePage slug before creating one. (ALT-999)
Fixes a bug when choosing not to connect to a contentful space. When this step is skipped the project is bootstrapped and the CLI tool informs the user to update their .env.local file. (ALT-998)

Screenshots
Bootstrapping a project only (now mentions to update the .env.local file at the end)

When selecting an existing space, there are additional validation checks before creating a content type and entry

One inconsequential bug I found is that I can move onto the next step with an invalid directory name and then hitting ctrl+c. SO rather than killing the entire process it let me go to the next step with the invalid name. Eventually it fails and nothing gets created but just something for you to note in my testing

@ryunsong-contentful ~ I noticed this to it's kind of annoying when you want to cancel the script. It looks like the library we're using provides a method for handling cancellation more gracefully so I'm going to make a small update to incorporate this in a follow-up PR: https://www.npmjs.com/package/@clack/prompts#cancellation


Add debug option to retrieve PDU from another server

Would it be feasible to update the !admin debug get-pdu command to accept an optional server on the end to trigger retrieving that PDU from another server?
It'd really help while testing the odd missing PDU I have, but would also help fill gaps in the database when odd things happen.
For example, currently if I run this:
!admin debug get-pdu $$LGX4aNj2JJq1g324w-VMQQzhPge99aE95klwMc6BiIA

I get "PDU not found." as the response, but if I could run an alternative like this:
!admin debug get-pdu $$LGX4aNj2JJq1g324w-VMQQzhPge99aE95klwMc6BiIA example.com

Conduwuit would hopefully retrieve that missing event from the example.com server, so I could not only see the content but also it'd fix the gap in my room history.

This should now be implemented.
!admin debug get-remote-pdu <PDU_ID> <SERVER>
It will treat the remote PDU as an incoming backfilled PDU. Standard event auth checks and if we have the event already is already done.

Amazing, thank you!


train.py gives tensorflow ValueError: No variables provided

I try to train the KittiSeg net on my own data, so I have generated Kitti-like groundtruth masks, exported my DATA and RUN directories and created train/val txt files (and put those in KittiSeg.json. When I then run train.py, it starts and initialises the net, but it fails on training with the tensorflow ValueError 'no variables provided'. That does not sound data-set related or so. Do you know what should I update/provide/change to fix that?
(...)
2017-04-06 15:42:59,665 INFO Creating Summary for: score_pool3/biases
2017-04-06 15:42:59,704 INFO Creating Summary for: upscore32/up_filter
Traceback (most recent call last):
  File "train.py", line 131, in <module>
    tf.app.run()
  File "/home/shah/tensorflow/tensorflow_v1.0.0_py3.5_gpu/lib/python3.5/site-packages/tensorflow/python/platform/app.py", line 44, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "train.py", line 127, in main
    train.do_training(hypes)
  File "incl/tensorvision/train.py", line 377, in do_training
    tv_graph = core.build_training_graph(hypes, queue, modules)
  File "incl/tensorvision/core.py", line 100, in build_training_graph
    global_step, learning_rate)
  File "/home/shah/Coding/KittiSeg/hypes/../optimizer/generic_optimizer.py", line 91, in training
    global_step=global_step)
  File "/home/shah/tensorflow/tensorflow_v1.0.0_py3.5_gpu/lib/python3.5/site-packages/tensorflow/python/training/optimizer.py", line 380, in apply_gradients
    raise ValueError("No variables provided.")
ValueError: No variables provided.

possibly related things:

I commented out line 123 (train.maybe_download_and_extract(hypes) in train.py because otherwise it would still download kitti;
I got the same commentjson import error as issue #28, so I had to import json instead of commentjson in train.py

Could that last one cause some values to be empty or is it something else?

Hi WPSanberg,
It seems to me that we are meeting the similar problem, i posted my question in the previous thread.
Wenouyang

Hello , I don't know where the problem is in what you did ,
but I've trained the model on my own dataset and it did work , I'll tell what I did , maybe it will help ,
I mainly changed the input file , and set it up so that it loads the images directly from train and validation folders , after loading it resizes them to the same resolution as Kitti images , and then it gets the ground truth mask from the hypes , and use it to read the 'gt_images'  and separate it into the different classes, the rest I didn't touch ... hope it helps

To me this looks like the tensor loss/total_loss is not well defined. Can you check, whether the loss function actually gets computed correctly?

Hi Marvin,
It indeed seems to be about the loss, the error rises in tensorvisions def build_training_graph(hypes, queue, modules): at this segment:
with tf.name_scope("Loss"):         losses = objective.loss(hypes, decoded_logits,                                 labels)
but I am not sure how to trace it beyond that.

It seems to be python-related, I changed from python 3.5 to python 2.7 and the KITTI training works out of the box now.

Yeah, if I find some spare time, I will look into python 3.5. In theory it should work out of the box.

It because the return format of zip() is different between python2 and 3.
change the line grads_and_vars = zip(clipped_grads, tvars) in optimizer/generic_optimizer.py to grads_and_vars = list(zip(clipped_grads, tvars)), it works for me.

I already had the data and did not want the script to download it again, so I
commented it out fully, without replacing it/changing it to something else.
2018-05-14 16:40 GMT+02:00<PHONE_NUMBER><EMAIL_ADDRESS>
@WPSanberg https://github.com/WPSanberg how to comment out line
123,train.maybe_download_and_extract(hypes),it should be what
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarvinTeichmann/KittiSeg/issues/35#issuecomment-388841303,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZuCwbYVZlWlmz3zbC6lW5HSCBe4_AcJks5tyZdggaJpZM4M1s6E
.

@WPSanberg how to comment out line 123,train.maybe_download_and_extract(hypes),it should be what

Nothing. Just go to Format and Comment out that region.


[Question] Is High Availibility an open?

Is it an option to activate High Availability?
We would like to make sure that gatus is never down on our cluster, at this moment we would deploy 2 charts with 2 chart names that monitor eachother but also each need their own configuration.
It would be nice to use a headless method or something like that with multiple working pods!

Duplicate of #176 - feel free to comment on that issue (and react with a 👍 to show your interest in the feature)


Start implementation of Screen9 widget

Waiting for Screen9 access

@fernandolucchesi to add proper description to the task so that it is possible to know when it is done

Meeting scheduled with Screen9

When creating a new video asset document, I choose import from media bank - BrandMaster loads fine, I successfully import the asset but then get a validation issue on the "Video" field:

Similarly on a video asset I created earlier (before Screen9 was added), I import the asset from BrandMaster successfully, but the web prioritizes the existing video field (which was uploaded manually as file).


Manually add libnl3 for CentOS 7

PR ( https://github.com/conda-forge/cdt-builds/pull/26 ) added libnl3. However it appears CentOS CDTs for x86_64 and aarch64 packages. This manually adds them so they get packaged.
cc @beckermr @mike-wendt

We should figure out why these did not get generated. Manually generated cdts need to go in the custom cdts and legacy customs cdts folders. Also, all platforms for a given cdt need to be moved there even if one is manaual.


Add function for weighted annual resampling

This addresses https://github.com/bradyrx/esm_analysis/issues/49. This adds a module temporal with the function to_annual() which resamples from finer-scale temporal output (e.g., monthly) to annual with weighting.
E.g., if it's monthly, it computes weights so that February counts for 28/365 of the annual mean (depending on the calendar).
Future efforts could be made to turn this into an accessor, and also to have to_seasonal() that does a seasonal time series (if you use the .groupby() method you just get the full season means. You need to use .resample() to get individual seasons time series).
In the end, the difference is pretty small between the weighted mean and arithmetic mean, but it's good to have.

what about a weighted sum? is that needed also?
thinking about co2_flux that we sum up over the months. Its actually more useful to leave co2_flx in per second and then only convert to PgC/yr in the final step.
therefore, i'd say no.

what about a weighted sum? is that needed also?

Yeah I think so. I need to think about how the command/math would work out, unless you have an immediate suggestion. But similarly, for monthly averages, January + ... + December for the yearly sum would give extra weight to February.
Also, wouldn't monthly averages just give you the average for that whole month? So in reality you should be doing something like Jan31 + Feb28 + ... + Dec*31 to integrate the monthly averages into a yearly sum. Also this gets complicated for fluxes with unit time.... or are you only summing for things with units of per time?

thinking about co2_flux that we sum up over the months. Its actually more useful to leave co2_flx in per second and then only convert to PgC/yr in the final step.
therefore, i'd say no.

Maybe you were getting at that here. I'm using this for CO2 flux right now. So in CMIP6 it's kg/m2/s. So I multiply by areacello first to get it to kg/s. Then use this function to go to kg/s for each year. Then convert to Pg/yr.

future can bring to_seasonal, to_weekly for daily output.

Yep and could add how='mean' and how='sum' like xarray language if we decide to do this sum thing.

so which is the calendar to use when the output is on the real world calendar? standard? I think our model uses that.

'standard'/'gregorian' is the real-world calendar. I decided to do None as the default so people have to think about which calendar is being used.
CF-compliant should have an attribute under time called calendar. The reason I asked about attribute maintaining is that i wanted to try checking the ds for ds.time.calendar, but open_dataset() was deleting that attribute for some reason.

@aaronspring , let me know your thoughts on weighted sum before I merge.

I added a feature to try to auto-infer the calendar type. So if xarray converts your calendar to cftime, this will detect what type that is.

I like the how='mean', that would be great.

I like the how='mean', that would be great.

Okay, I added this. The infrastructure is there to add how='sum' and to_monthly() etc. when we need it!


How to use the DebugMode

Hi, I am new user of DAT and have some questions that I hope some one will guide me through.
When I run the command ".......\Invoke-CMApplyDriverPackage.ps1 -DebugMode -Endpoint "sccm-SITE.domain.local" -UserName "UserName" -Password "******" -TargetOSVersion 20H2"
There is no answer on the screen.

Am I doing something wrong or should I look in a log file somewhere?
Most I use the DAT servrice account (SVC_CM_AS) credentials ?

I guess the debugmode should return whether the driver exist in ConfigMgr or not ?
As I understand the script, it should in debugmode create a log file "C:\Temp\ApplyDriverPackage.log" which I cannot find.
Any help will be appreciated

The log file can be found in C:\Windows\Temp\ApplyDriverPackage.log
You can use any user account that has read access to ConfigMgr


Allows [lazyLoad] to have condition coding

I want to set [lazyLoad]="Condition? url : null", but if  Condition == false then the div will transform into:

.

HI,
Thanks for your PR. Is there any use case where you can't use *ngIf? Like:

In your use case, can the condition be false and then, at any time, switch to true or is the condition always true/false?
It may be a better approach to remove the event listener if the url is falsy. Something like:
     }

     setImage(image) {
+        if (!image) {
+            return;
+        }
         const element = this.elementRef.nativeElement;
         const isImgNode = this.elementRef.nativeElement.nodeName.toLowerCase() === 'img';
         if (isImgNode) {

Otherwise will isVisible be unnecessary called for every scroll event.

Thank you for your response
The first approach is not what I want.
I want to keep the div container and using Condition to display image
The second approach .
Originally, I would like to change in the setImage() method.
If the judge in the setImage() method, the element was still added the className: ng2-lazyloaded
Provide another approach? Something like:
ngAfterContentInit() {
         this.scrollSubscription = getScrollListener(this._scrollTarget)
             .filter(() => !!this.lazyImage) // remove the event listener if the lazyImage is falsy.**
             .filter(() => this.isVisible())
             .take(1)
             .switchMap(() => [this.loadImage(this.lazyImage))]

Can you teach me how to add diff code in comment area? Like in your comment, the diff code had light green background color.

I'm buying what you're saying and I will accept the PR.
Regarding showing a diff. I often make some changes in my editor and copy the result from git diff which looks something like:
      ngAfterContentInit() {
         this.scrollSubscription = getScrollListener(this._scrollTarget)
+            .filter(() => !!this.lazyImage)
             .filter(() => this.isVisible())
             .take(1)
             .switchMap(() => this.loadImage(this.lazyImage))

and then set the code type to diff (the same way you setting the type to javascript above).
      ngAfterContentInit() {
         this.scrollSubscription = getScrollListener(this._scrollTarget)
+            .filter(() => !!this.lazyImage)
             .filter(() => this.isVisible())
             .take(1)
             .switchMap(() => this.loadImage(this.lazyImage))

I saw that you added a new commit in your master branch but you are making a PR from your patch-1 branch. Are you planing to create a new PR?

You can ignore. The commit I want to try diff code format, which is operational mistakes

@natqiu I will close this PR now. Let me know if you are still interested in this feature.


Fix Dependencies

Right now (on a fresh install), plotting won't happen. See https://pkgdocs.julialang.org/v1/creating-packages/#Test-specific-dependencies-in-Julia-1.2-and-above
I think this means that GLMakie would need to be a weakdep / extension. See the conditional loading of packages section

Here's another example: https://github.com/pebeto/julia_extensions_example/blob/main/Project.toml


Ensure golangci-lint runs on all files

removed unused func params and import
ignore revive dot-imports rule on ginkgo and gomega as dot import
there is the recommended practice
various small style fixes found by the linter

/approve /lgtm
We will be looking at the issue in a separate bug.

/lgtm


[discuss] mvndw, alias and more support?

Hi,

I wonder if a mvndw is planned with a correct option parsing (maven one is not yet) which would enable to replace mvnw in script to benefit mvnd.
also wonder if alias command support can be added ('mvnd my-alias" which would run "mvnd install -pl foo -pl bar" etc)
lastly i wonder if mvnd can cache most of the cache for releases in .m2 to just reload it when started - and benefit the no daemon mode

I wonder if a mvndw is planned with a correct option parsing (maven one is not yet) which would enable to replace mvnw in script to benefit mvnd.

Could you please explain what do you mean with "correct option parsing"?

also wonder if alias command support can be added ('mvnd my-alias" which would run "mvnd install -pl foo -pl bar" etc)

This functionality is provided by shell. What added value would balance the design and maintenance overhead in mvnd?

lastly i wonder if mvnd can cache most of the cache for releases in .m2 to just reload it when started - and benefit the no daemon mode

Interesting idea.
P.S.: One question per issue would make it easier to track the individual questions.

This is related to https://github.com/mvndaemon/mvnd/issues/143
The main problem is that currently mvnd is tied to a specific maven version, because the maven cli is not extensible enough for mvnd and I did not want to rewrite it all.

Maybe patching a bit more mvn script and replacing this part - even with plain java code since mvnd can do it (yeah!) would make sense?
Another common use case is to be able to force a java version before launching the maven command (either with sdk use java xxx or a plain script). Would be great to be able to put it in $rootProject/.mvnd/script.jshell or so (only point to take care is this should be optional - maybe an option --skip-mvnd-script - to enable to bypass it if we set up the env manually)
wdyt?


ERROR  1  DifferentialDrive... Output not updated often enough.

ERROR  1  DifferentialDrive... Output not updated often enough.  edu.wpi.first.wpilibj.MotorSafety.check(MotorSafety.java:101)
Warning at edu.wpi.first.wpilibj.IterativeRobotBase.printLoopOverrunMessage(IterativeRobotBase.java:276): Loop time of 0.02s overrun

This is a known issue with the way the CTRE Phoenix Framework interacts with WPIlib.  It's mentioned in the Phoenix Framework docs here.  CTRE has been attempting to optimize their framework to work around the issues with WPIlib for the last two seasons but, we're bumping up against the limitations of the hardware.


[Typo/Mistranslation] 09-perform-data-analysis-in-power-bi-desktop.md

On Practice 3, task 1 step 3 through 6, the column and table names need to stay in English so it will align with the lab environment. It is very confusing for the learners to see the column/table names in Korean on the instruction and then have to match it with the corresponding English names on the lab environment.
Current:
날짜
날짜
판매
판매
날짜
연도
제품
범주
자전거
To be:
Date
Date
Sales
Sales
Date
Year
Product
Category
Bikes

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team


Add docker support

Have already the image for 1.0.2 at @consenSys docker hub account.

Regarding the point above: It'd be nice to add somewhere an automated task to build the image on new versions or pushes to master.

A new directory docker had to be made, as two files were needed: A Dockerfile and a entrypoint.sh file.

This solves #114

Hey @hermanjunge, can you take a look at what the String team is doing here? https://github.com/dominicwilliams/instant-dapp-ide
It's a full dapp development docker image with TestRPC and Truffle. Would love to get your opinion. I'd like to evaluate supporting their effort as that pushes maintenance of docker images over to them.

Done the evaluation
Cons first, as I don't want to give the idea that I don't want other people to do the docker stuff. Actually I love the ole' Give a man a fish / Teach him to fish saying. That way, we have more bandwith to learn more things and solve other problems. So, here is the thing:
Cons:

Running more than one process inside a container is an anti-pattern.
They need help to make the image lighter.
Forces the dev to develop using Cloud9. What if the dev just wants truffle without anything (they should not be using testrpc, but a real node)? What if they use vim (or, god forbid me, emacs), atom, sublime, etc?
Regarding the above: A much better pattern is an array of containers orchestrated using docker-compose.
This is nitpicking, but they need help in the design of the Dockerfile, every line is a layer, and layers must be compact.
Finally, they should rely more on external files (and directories), they could be easily updated at build time, adding the right instruction to the build script, in order to keep a smaller Dockerfile.

Pros:

We won't have, in the future, to maintain anything about docker. (HUGE PRO)
We can support their effort to modularize their dev environment, should they want our help, of course. (See above what I think of fishes and code).

TLDR: Cool. If they let us help them, we can have this excellent dev env: powerful slim, fit, adaptable. And we can focus on solving other problems.

Which would you want to do? I just spent a few days with them, and I'm sure
they'd be very open to us helping. I think if we do it right we can make it
a team effort.
Totally understand both your pros and cons. Good points all around.
On Apr 12, 2016 9:36 PM, "Herman Junge"<EMAIL_ADDRESS>wrote:

Done the evaluation
Cons first, as I don't want to give the idea that I don't want other
people to do the docker stuff. Actually I love the ole' Give a man a fish
/ Teach him to fish saying. That way, we have more bandwith to learn more
things and solve other problems. So, here is the thing:
Cons:

Running more than one process inside a container is an anti-pattern
.
They need help to make the image lighter.
Forces the dev to develop using Cloud9. What if the dev just wants
truffle without anything (they should not be using testrpc, but a real
node)? What if they use vim (or, god forbid me, emacs), atom,
sublime, etc?
Regarding the above: A much better pattern is an array of containers
orchestrated using docker-compose.
This is nitpicking, but they need help in the design of the
Dockerfile, every line is a layer, and layers must be compact.
Finally, they should rely more on external files (and directories),
they could be easily updated at build time, adding the right instruction to
the build script, in order to keep a smaller Dockerfile.

Pros:

We won't have, in the future, to maintain anything about docker. (HUGE
PRO)
We can support their effort to modularize their dev environment,
should they want our help, of course. (See above what I think of
fishes and code).

TLDR: Cool. If they let us help them, we can have this excellent dev
env: powerful slim, fit, adaptable. And we can focus on solving other
problems.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/ConsenSys/truffle/pull/134#issuecomment-209223698

I can work with them, my biggest complain is that the development environment must be modularised into several containers orchestrated using docker compose.

I'm sure they'd love your input. Let me reach out to them to see if they're
interested and we'll go from there.
On Apr 12, 2016 9:43 PM, "Herman Junge"<EMAIL_ADDRESS>wrote:

I can work with them, my biggest complain is that the development
environment must be modularised into several containers orchestrated using docker
compose.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/ConsenSys/truffle/pull/134#issuecomment-209225727

Good. @tcoulter, is your call to keep this PR open or closed. I got this very implementation in my dockerfiles collection :-)

Herman, I am using this here https://github.com/combient/truffle
I guess you havent progresed with this somewhere else?

@oluies Here https://github.com/hermanjunge/dockerfiles/blob/3d6b00279f044505b576560bede0b7836e4e0133/truffle/Dockerfile


License

Please add a license, preferably Apache 2.0, LGPLv3 or GPLv3.

Scala-gopher is open source (license is Apache2); binaries are available from the maven-central repository.

Please keep it ASL 2.0

Added copy of Apache-2.0  license to root project dir.

lol, you added a modified version of the Apache 2.0 LICENSE so even github can't recognise it. You're supposed to put your name and copyright information in a NOTICE file and leave the LICENSE file untouched.

thanks, fixed ;)


Customize CacheKey according to request parameters

Is there an existing issue that is already proposing this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe it
When I have a method like this, I can't set the cache key using id:

Describe the solution you'd like
Similarly, customize the CacheKey:

Or so:

Teachability, documentation, adoption, migration strategy
No response
What is the motivation / use case for changing the behavior?

https://docs.nestjs.com/techniques/caching#adjust-tracking


Issue with StockTradingEnvStopLoss

If I replace at the papertrading.py the import: from finrl.neo_finrl.env_stock_trading.env_stocktrading_np import StockTradingEnv to the stopp loss env: from finrl.neo_finrl.env_stock_trading.env_stocktrading_stoploss import StockTradingEnvStopLoss and define env = StockTradingEnvStopLoss at the API section in the papertrading.py file.
I get at the train section train(start_date = '2021-10-11'...) the following error.
Dont understand what I did wrong
How to fix this error to make the stop loss env work?
(note: neo_finrl was renamed at the imports)
TypeError                                 Traceback (most recent call last)
 in ()
13       erl_params=ERL_PARAMS,
14       cwd='./papertrading_erl', #current_working_dir
---> 15       break_step=1e5)
/usr/local/lib/python3.7/dist-packages/finrl/train.py in train(start_date, end_date, ticker_list, data_source, time_interval, technical_indicator_list, drl_lib, env, model_name, if_vix, **kwargs)
34         "if_train": True,
35     }
---> 36     env_instance = env(config=env_config)
37
38     # read parameters
TypeError: init() got an unexpected keyword argument 'config'

Refer here. You can find the init config of Stop loss environment


Update batch dataplane for version 2021-01-01-13.0 - Data Plane

New features in this release are:

Managed identity support for pools
Virtual machine extension support for nodes
Availability zone support for pools

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet. Service has already begun to roll out.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet. ASAP (delivery due 01-01-2021)
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[x] PowerShell
[x] CLI
[x] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

This PR was split off from #12393 (which contained both data and management planes previously). Added @anuchandy as the reviewer since he has already reviewed these changes in the other PR.
We're in a bit of a holding pattern here while we sort through an issue on the service side with VM extensions, so marked this as not ready for review. Will update when we have things sorted on our end.

This PR was split off from #12393 (which contained both data and management planes previously). Added @anuchandy as the reviewer since he has already reviewed these changes in the other PR.
We're in a bit of a holding pattern here while we sort through an issue on the service side with VM extensions, so marked this as not ready for review. Will update when we have things sorted on our end.

/azp run

/azp run

Marked as DoNotMerge until the issue with virtual machine extensions is resolved on the service side. (Even though this likely won't affect the data plane swagger)

Marked as DoNotMerge until the issue with virtual machine extensions is resolved on the service side. (Even though this likely won't affect the data plane swagger)

Hi, @paterasMSFT. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

Hi, @paterasMSFT. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

Hi, @paterasMSFT. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.


add pprof for agent and gateway

Signed-off-by: jiawei<EMAIL_ADDRESS>
What type of PR is this?
/kind feature

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

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@JiaweiGithub Does pprof occupy too many resources during running? Can you make a function that can be switched on and off?

@JiaweiGithub Does pprof occupy too many resources during running? Can you make a function that can be switched on and off? example:
in edgemesh-agent.yaml:

commonConfig:
    pprof: enable

At present, this function is disabled by default, and it is enabled by the flags parameter.The parameters are profiling-bind-address and profiling-port.
It is also possible to change it to the enable switch of commonConfig, can consider which one is more appropriate.

can compare which one is better.

So, what's your advice?

can compare which one is better.

So, what's your advice?

I prefer the current way, can choose the ip and port to listen to.

commonConfig

can compare which one is better.

So, what's your advice?

It will be more convenient to use the config configuration file, I changed the code, please help to review.

@JiaweiGithub  e2e test is failed

@JiaweiGithub e2e test is failed

@JiaweiGithub e2e test is failed

Fixed

/lgtm
/approve


Background task for calculating rankings

Takes too long, especially for pro-am couples

Closed with pull request #82


Convert http_backend to an interface

By converting the http_backend into a implementation of a common
interface, it allows flexibility of using other HTTP clients or
remote browsers to drive HTTP requests. This commit adds the HTTPDriver
interface which provides the same functionality as the http_backend and
updates the Collector to communicate the the http_backend through the
interface methods.
Also attempted to convert the AppEngine call to use the HTTPDriver interface
This is in support of issue #4

@asciimoo I assume that chromedp can trigger javascript (and thus is able to provide the raw html as you would see it in the browser).
An easy way to check if this is correct (and make sure that chromedp is the right solution) is to try to scrape this toy website. It requires javascript rendering in order to set up the dom the same way you see it in the browser.

@manugarri I assume that too, but at some point we have to ask the response body from the browser. Then we have no option to interact with the browser.
What if I have to click a button to trigger an AJAX request (e.g. login) and I'm interested in the DOM after the AJAX request?

hmm good point, good point. @asciimoo , that would be equally important. Im a bit lost with chromedp to be honest, but it seems like colly would need to yield to the library in order to use chromedp api to interact with the page (because colly does not support page interaction)

Hey @asciimoo I've been playing around with chromdp and the HTTPDriver, and you're right, there would need to be some more modifications in order to have a meaningful relationship.
In order to get a copy of the HTML page, you can use chromedp.OuterHTML("html", &somestring).
To answer the bigger question of interacting with the javascript, there would need to be a way were the colly.Collector can pass some information to the backend.  The cleanest solution I've been able to think of is through the colly.Content.  Right now the context is passed to fetch but not into http_backend.Cache().  If we could do this then I could imagine passing simple actions to the chromedp, or any other driver.

Another way to interract with the browser is extending the collector with an OnBrowser callback. Its' signature could be func (c *Collector) OnBrowser(browserEvent string, func(*Browser)), where Browser type would be the interface to the browsers, like HTMLElement in OnHTML callbacks.

@asciimoo I like the idea of having a Browser as a first class citizen.  I'm trying to think through what a Browser interface might look like.  I'll put some ideas up when I get some time to digest it.

I'm gonna have to move this to another PR.


Multiple runs on same data with early_stopping not working in GPU training with dask_cudf

Hi,
currently I am working on a multi-gpu workflow using dask_cudf together with xgboost 1.7.1 and early stopping.
The basic workflow works fine. However, if I want to iterate multiple times over the same data, e.g. with different parameters, it appears that the early stopping is only working for the first iteration. In subsequent iterations training is not starting and it only prints the validation loss for node 0 and then is finished with training without any error:

The basic workflow looks like this:
cluster = LocalCUDACluster(n_workers=4)
client = Client(cluster)

df = dask_cudf.read_parquet("path_to_parquet")

X_train = df[df["CalDay"] < validation_start_date][features]
y_train = df[df["CalDay"] < validation_start_date][label]

X_val = df[df["CalDay"] >= validation_start_date][features]
y_val = df[df["CalDay"] >= validation_start_date][label]

def fit_xgboost(params, X_train, y_train, X_val, y_val):
  es = xgb.callback.EarlyStopping(rounds=early_stopping_rounds, save_best=False)

  Xy = dxgb.DaskQuantileDMatrix(client, X_train, y_train)
  Xy_valid = dxgb.DaskDMatrix(client, X_val, y_val)

  booster = xgb.dask.train(
    client,
    params,
    Xy,
    evals=[(Xy_valid, "Valid")],
    num_boost_round=1000,
    callbacks=[es],
    verbose_eval=True
  )
  return booster

for params in params_list:
  fit_xgboost(params, X_train, y_train, X_val, y_val)

Thanks for any help that you can provide!

Could you please share the size of your dataset and the number of GPUs you are using? It seems dask is pulling all subsequent data into one worker.

@trivialfis Of course, currently I am on 4 V100 GPUs. I could try out machines with 2 or 1 GPU as well but havent so far.
The dataset that I am currently looking at has 3 152 629  rows and 131 columns.
I noticed this behavior with larger and smaller datasets as well.
For me it seems that there is some random element as well if it works or not, e.g. if I iterate over different subsets/groups of the same dataset and train a model per group then it works for some groups while it does not for others.

Have you tried to use dask_ml for splitting the data? https://github.com/dmlc/xgboost/blob/15a88ceef0f60d461925bb5133716ab67de6ca8e/demo/dask/dask_callbacks.py#L8

NVM, it seems you are doing some sort of time series task. Spiting data on a distributed system can be difficult sometime. I will take a deeper look later.

I think the issue is in data split, XGBoost doesn't move data and runs on the workers where data is resided on. If there's only one worker being used, then all the data is on that worker.

Yes it is a timeseries use-case.
I have around 30 datasets and on these I want to train around 200 models in total (1 per group). Therefore I do a lot of iterating over groups and/or parametersets and would like to use early stopping for each of these.

Could you please try to repartition the dataset after each iteration of tuning?

@trivialfis same issue unfortunately


Template Ref watcher is not triggered with flush: post option in Vue 2.7.

Version
2.7.6
Reproduction link
codesandbox.io
Steps to reproduce
Open the CodeSandbox and watch the console output.
There seems to be a problem when Template Ref watcher components are nested.
What is expected?
There should be a log entry for App.vue.
What is actually happening?
There's no log entry because the watcher is not triggered.

This works in Vue 3.
https://codesandbox.io/embed/modest-bash-u84i9q?fontsize=14&hidenavigation=1&theme=dark

Yeah, had the same issues while used useResizeObserver from @vueuse/core.


Update Vault allocation to use a mapping of one stablecoin to one strategy

We want to move to a system where the Vault has a mapping of one asset to one Strategy for allocations
E.g. mapping(address => address) defaultStrategies which maps from asset addresses to strategy addresses.
The allocate logic then should be updated to just look at this mapping and transfer the asset to only the strategy in the mapping. For redeems that are larger than the Vault buffer, the same mapping will be used to choose where to withdraw from.

Closed by #368


fix: cms page test

Description :sparkles:
Issues :bug:
Closes :no_good_woman:
DEV-XXX:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

kultus-admin-ui branch is deployed to platta: https://kultus-admin-ui-pr355.dev.hel.ninja :rocket::rocket::rocket:


add the --sign operation of TemurinGenSBOM.java

add the --sign operation of TemurinGenSBOM.java
Ref:  https://github.com/adoptium/temurin-build/issues/3158

@julian55455 The various builds failed.

@julian55455 For testing this create your own RSA private key, eg.
openssl genrsa -des3 -out RSAPrivate.pem 2048

Then pass the path to the RSAPrivate.pem to --signSBOM

/thaw

Requires jdk-19 setup PR to run the openkeystore code

@julian55455 Can you do the following to resolve the jdk17+ and Solaris build issue please. We will then have a new Jenkins job SignSBOM called as part of the post-build tasks:

Move all of the --signSBOM and openkeystore code out of TemurinGenSBOM.java into a new class TemurinSignSBOM.java
Add a new build.xml target "buildSignSBOM", which builds TemurinSignSBOM.java
Add a new build.xml target "runSignSBOM", which performs a unit test of this new class:

Create a dummy pem file: openssl ...
Invoke --signSBOM
Verify signature

fixed in this pr https://github.com/adoptium/temurin-build/pull/3243


Broken links to pdfs of specs

This link from the project readme currently 404s:
https://input-output-hk.github.io/formal-ledger-specifications/pdfs/cardano-ledger.pdf
The link to the html works though:
https://input-output-hk.github.io/formal-ledger-specifications/html/ledger/Ledger.PDF.html

A previous commit of mine has caused this, the relevant changes being the different structure of the Haskell modules.
Upon further inspection of the CI logs:

In step "Build static file", the following command runs:

for f in pdfs/*.pdf; do
    echo "
$(basename $f)
" >> index.html
done

Since we can see the .pdf files in the deployed website (in index.html), it means that pdfs/ contains both PDFs after the Nix builds.
In step "Add files", the command git add pdfs runs but there are no corresponding Git lines that indicate this, i.e. the following lines are missing / not printed:
 create mode 100644 pdfs/cardano-ledger.pdf
 create mode 100644 pdfs/midnight-example.pdf

Thanks Orestis, if this gets broken then it shouldn't pass CI.
A interim step might be to add "check that the PDFs build locally" to the PR checklist.

Thanks Orestis, if this gets broken then it shouldn't pass CI.
A interim step might be to add "check that the PDFs build locally" to the PR checklist.

Actually the PDFs build both locally and in the CI machine, but somehow they do not get pushed to the gh-pages branch at the end of the CI run.


use onsets and offsets in seconds, not Hz, for make_notmat function

since .not.mat files have onsets and offsets in seconds
this makes it possible to "round trip" from .not.mat to Annotation and back without anything besides the .not.mat file (e.g. without getting sampling frequency from the associated audio file)

fixed by #45


MG and MIS ready for review: Sam

Hello @samm82,
Thanks for your help. My MG and MIS are ready to review. I am open to your suggestions. :)

Since #37 is a non-trivial issue that will affect a lot of your modules (and I think I've given you quite a lot to do!) I think I'm going to stop my review here. Great document! It was quite clear what the intent between each module/function was.
Feel free to close this issue for now! (I'm not a contributor yet, so I can't do it myself lol.) I might take a second pass later to see if I missed anything, but based on my workload, this is unlikely. 😅

Thank you so much @samm82 for your valuable feedback. :)


Add copier function

Codecov Report

Merging #3 (838f38c) into master (7d63f73) will decrease coverage by 1.47%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   73.41%   71.94%   -1.48%
==========================================
  Files           5        6       +1
  Lines         568      638      +70
==========================================
+ Hits          417      459      +42
- Misses        127      142      +15
- Partials       24       37      +13

Impacted Files
Coverage Δ

common/copier.go
60.00% <60.00%> (ø)

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 7d63f73...838f38c. Read the comment docs.


First documentation of the light API

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (daf4402) 81.68% compared to head (354f23a) 81.64%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
- Coverage   81.68%   81.64%   -0.04%
==========================================
  Files          44       45       +1
  Lines        5974     6032      +58
==========================================
+ Hits         4880     4925      +45
- Misses       1094     1107      +13

Files
Coverage Δ

onnx_array_api/validation/docs.py
100.00% <100.00%> (ø)

onnx_array_api/light_api/var.py
91.22% <50.00%> (-0.74%)
:arrow_down:

onnx_array_api/light_api/model.py
81.87% <72.72%> (-2.08%)
:arrow_down:

onnx_array_api/light_api/_op_var.py
51.42% <55.00%> (+0.84%)
:arrow_up:

... and 1 file with indirect coverage changes

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


refactor(checks) Stronger typing for exists

This causes exists to function as a type guard, preserving the original type of the input.
I have been using this version locally, and I don't think it breaks any builds.  (There is no change to runtime behavior).

Thanks very much @gavinpc-mindgrub & sorry for the delay (roadtrip)!


Work done or not

Check if the work is done or not

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: biswajitdeb11 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add support for url in url

Hmm, the failed tests do not seem to be related to our changes? But I'm in not completely shure...

23 09 2016 15:52:19.298:WARN [preprocess]: Can not load "webpack"!


how to give nested child path?

i am working on get the value at
.user
-user1
-id_Card_number=1235667
how to get the value 1235667

Did you try:
FirebaseArduino.getInt('/user/user1/id_Card_number') ?

Same thing for me.
Firebase.getInt('/user/user1/id_Card_number');
or
FirebaseArduino.getInt('/user/user1/id_Card_number') ?
Not working!

@Kamoba can you confirm how your data is stored in the database (as an int, or as a string)?
I wonder if we should convert string value to int directly if the user use getInt(), let's discuss this in #345

Im just trying to push to a nested array. Have tried to use @ViewChildren decorator to import the HTML  data into the typescript file. I get a bunch of html that Im not sure how to work with over there though, I just want objects and arrays and the same as I put in to work with the same way! Is this not a thing?


How to delete pose landmarks or add new pose landmarks?

The reason im writing is that Im trying to add new landmarks in torso, in between shoulders and in the middle of hips to connect them. At the same time I want to delete the unneccessary joints from face and thumbs.
I started my search in the pose.py and deleted 2 joins and added 3 joins:
class PoseLandmark(enum.IntEnum):
  NOSE = 0
  LEFT_EYE_INNER = 1
  LEFT_EYE = 2
  LEFT_EYE_OUTER = 3
  RIGHT_EYE_INNER = 4
  RIGHT_EYE = 5
  RIGHT_EYE_OUTER = 6
  LEFT_EAR = 7
  RIGHT_EAR = 8
  MOUTH_LEFT = 9
  MOUTH_RIGHT = 10
  LEFT_SHOULDER = 11
  RIGHT_SHOULDER = 12
  LEFT_ELBOW = 13
  RIGHT_ELBOW = 14
  LEFT_WRIST = 15
  RIGHT_WRIST = 16
  LEFT_PINKY = 17
  RIGHT_PINKY = 18
  LEFT_INDEX = 19
  RIGHT_INDEX = 20
  # LEFT_THUMB = 21 # these were deleted (not to draw nor save to csv)
  # RIGHT_THUMB = 22
  LEFT_HIP = 23
  RIGHT_HIP = 24
  LEFT_KNEE = 25
  RIGHT_KNEE = 26
  LEFT_ANKLE = 27
  RIGHT_ANKLE = 28
  LEFT_HEEL = 29
  RIGHT_HEEL = 30
  LEFT_FOOT_INDEX = 31
  RIGHT_FOOT_INDEX = 32
  # SHOULDERS_INDEX = 33 # these joints were added
  # TORSO_INDEX = 34
  # HIPS_INDEX = 35

in the pose_connections.py changed this part:
POSE_CONNECTIONS = frozenset([(0, 1), (1, 2), (2, 3), (3, 7), (0, 4), (4, 5),
                              (5, 6), (6, 8), (9, 10), (11, 12), (11, 13),
                              (13, 15), (15, 17), (15, 19), (15, 21), (17, 19),
                              (12, 14), (14, 16), (16, 18), (16, 20), (16, 22),
                              (18, 20), (11, 23), (12, 24), (23, 24), (23, 25),
                              (24, 26), (25, 27), (26, 28), (27, 29), (28, 30),
                              (29, 31), (30, 32), (27, 31), (28, 32), **(33, 34),
                              (34, 35), (0, 33)**])

and in drawing_styles.py changed the code in lines 111-132 to (deleted left and right thumb):
Pose
_THICKNESS_POSE_LANDMARKS = 2
_POSE_LANDMARKS_LEFT = frozenset([
    PoseLandmark.LEFT_EYE_INNER, PoseLandmark.LEFT_EYE,
    PoseLandmark.LEFT_EYE_OUTER, PoseLandmark.LEFT_EAR, PoseLandmark.MOUTH_LEFT,
    PoseLandmark.LEFT_SHOULDER, PoseLandmark.LEFT_ELBOW,
    PoseLandmark.LEFT_WRIST, PoseLandmark.LEFT_PINKY, PoseLandmark.LEFT_INDEX,
    PoseLandmark.LEFT_HIP, PoseLandmark.LEFT_KNEE,
    PoseLandmark.LEFT_ANKLE, PoseLandmark.LEFT_HEEL,
    PoseLandmark.LEFT_FOOT_INDEX
])

_POSE_LANDMARKS_RIGHT = frozenset([
    PoseLandmark.RIGHT_EYE_INNER, PoseLandmark.RIGHT_EYE,
    PoseLandmark.RIGHT_EYE_OUTER, PoseLandmark.RIGHT_EAR,
    PoseLandmark.MOUTH_RIGHT, PoseLandmark.RIGHT_SHOULDER,
    PoseLandmark.RIGHT_ELBOW, PoseLandmark.RIGHT_WRIST,
    PoseLandmark.RIGHT_PINKY, PoseLandmark.RIGHT_INDEX,
    PoseLandmark.RIGHT_HIP, PoseLandmark.RIGHT_KNEE,
    PoseLandmark.RIGHT_ANKLE, PoseLandmark.RIGHT_HEEL,
    PoseLandmark.RIGHT_FOOT_INDEX
])

and between 209-225:
pose_landmark_style = {}
 left_spec = DrawingSpec(
     color=(0, 138, 255), thickness=_THICKNESS_POSE_LANDMARKS)
 right_spec = DrawingSpec(
     color=(231, 217, 0), thickness=_THICKNESS_POSE_LANDMARKS)
 for landmark in _POSE_LANDMARKS_LEFT:
   pose_landmark_style[landmark] = left_spec
 for landmark in _POSE_LANDMARKS_RIGHT:
   pose_landmark_style[landmark] = right_spec
 pose_landmark_style[PoseLandmark.NOSE] = DrawingSpec(
     color=_WHITE, thickness=_THICKNESS_POSE_LANDMARKS)
 pose_landmark_style[PoseLandmark.SHOULDERS_INDEX] = DrawingSpec( # want to add these (shoulders, torso, hips):
     color=_WHITE, thickness=_THICKNESS_POSE_LANDMARKS)
 pose_landmark_style[PoseLandmark.TORSO_INDEX] = DrawingSpec(
     color=_WHITE, thickness=_THICKNESS_POSE_LANDMARKS)
 pose_landmark_style[PoseLandmark.HIPS_INDEX] = DrawingSpec(
     color=_WHITE, thickness=_THICKNESS_POSE_LANDMARKS)

and after all these changes, this part - starting in bootstrap():
# Bootstrap all images.
      2 # Set limit to some small number for debug.
----> 3 bootstrap_helper.bootstrap(per_pose_class_limit=None)

<ipython-input-8-b13058f9160c> in bootstrap(self, per_pose_class_limit)
     99                  for lmk in pose_landmarks.landmark],
    100                 dtype=np.float32)
--> 101             assert pose_landmarks.shape == (31, 3),

is throwing and error, and then from there it is going to drawing_utils.py and throwing:
      if not (0 <= start_idx < num_landmarks and 0 <= end_idx < num_landmarks):
        raise ValueError(f'Landmark index is out of range. Invalid connection '
                         f'from landmark #{start_idx} to landmark #{end_idx}.')

so, I was looking for some hard coded value of 33 joints inside these files, yet didnt find any:
drawing_utils.py
pose_connections.py
pose.py
pose_test.py
drawing_styles.py
solution_base.py
landmark_pb2.py
To sum up, my question is: How to not to draw some of the "default" joints and draw some additional ones.
Best,
J

Instead of tweaking those data structures, you'd better to implement your own version of
def draw_landmarks(image: np.ndarray, landmark_list: landmark_pb2.NormalizedLandmarkList)
by following https://github.com/google/mediapipe/blob/master/mediapipe/python/solutions/drawing_utils.py#L120 and having the customizations.
I think the number 33 or 31 you mentioned is the length of landmark_list. Even it's not hardcoded, it's a fixed number returned by the solution output.

@jiuqiant  is the landmark number fixed to 33 joints? We would like to add torso joint which is not possible in this model. Is there any way we can solve it?
Your help would be highly appreciated.

@jiuqiant is the landmark number fixed to 33 joints? We would like to add torso joint which is not possible in this model. Is there any way we can solve it?
Your help would be highly appreciated.

The solution API only outputs the landmarks that generated by the model. If you need additional joint, please calculate the joints based on the landmark output.

@Jakub112998 were you succesful in adding additional landmarks to the torso? This is exactly what I would like to do as well.

I'm also looking to do the same thing. Is this documented?

I don't think it's possible to add new landmarks unless Mediapipe train a new model with new joints.


JVM metrics are not exposed

MetricsModule.Initializer was implemented as Lifecycle, but after #190 is not counted as application root

Resolved #196


Remove blocked and filtered reducers/selectors/actions.

Fixes
Issue Number: https://github.com/lbryio/lbry-desktop/issues/7681

What is the current behavior?
What is the new behavior?
blocked & filtered actions/selector/reducers were removed.
Also, if there is a censor reason, we will display it.

If no message is present, the default one will be used
("Your hub has blocked this content because it subscribes to the following blocking channel:")
Other information

PR Checklist

Toggle...
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[x] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I added a line describing my change to CHANGELOG.md
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

Thank you @jessopb, sadly I couldn't fix it without using // $FlowFixMe. Not ideal but it gets rid of the warning.
I tried typecasting but it kept complaining about not being compatible with mixed

This will be merged when infrastructure is ready.


Fbsql fixarama

This PR was created to address issues brought up by @travisturner on Slack.
Some things also to note:

I'm coming at this from the POV of a user who's never used the system, so I'm attempting to make the docs as unambiguous as possible.
The approach is the same as that taken with the Community Ingest docs, where instead of one big bucket of flags and whatnot, they're divided into high level groupings based on the assumption users will want to do specific things (e.g., connect to a db, run queries, output results and whatnot).

The thread
URL: https://moleculacorp.slack.com/archives/D04QW9A8TM1/p1693234779116399
Message annotations
I've annotated the copied message below as follows:

FIXED = yep, agree, no wukkas, etc
TODO = to be fixed, no discussion needed (will change to FIXED)
DISCUSSION = need to discuss your feedback and perceptions vs the approach

Hi Lisa,
I just reviewed the latest fbsql docs and I feel like there have been some regressions.

For context, we try to follow the general usage of PostgreSQL’s interactive terminal, the docs for which can be found [here](https://www.postgresql.org/docs/current/app-psql.html).

Speaking as an engineer, the psql docs are far easier to navigate and understand than what we have for fbsql (which, for the most part, works exactly the same way). The psql docs also contain a section describing “Meta-Commands”, the spirit of which I think is useful to understand. (DISCUSSION)

Some general things that I stumbled over when reviewing our docs:

*    The section titled “CLI flag Prefix” is confusing to me; I don’t understand why we’re conflating bash commands and fbsql meta-commands into something called a “flag prefix”? (DISCUSSION)
*   The section titled “Meta-command flag prefix” is also confusing to me; again we seem to be conflating meta-commands with bash, command line arguments. Also, meta-commands aren’t separated by \, rather, they are denoted by \ at the beginning of the command. (DISCUSSION)
*   This format -[c|-command] is something I have not seen before. Why not [-c|--command]? (FIXED)
*    fbsql should always be lowercase. I see it both FBSQL and FBSql, neither of which is correct. (FIXED)
*    The use of “pset” next to and/or in addition to “output” is confusing. “pset” stands for “print set”; it denotes how the output should be printed (i.e. its format). “output” generally refers to where the output should be printed (i.e. to the terminal, to a file, etc).(DISCUSSION)
*    The TOML configuration file for fbsql contains more than connection options, so calling it a “TOML connection file” is confusing. (FIXED)
*    While I appreciate the usage docs being split into various sections, it’s not clear how to navigate these pages when the information is not in an obvious place. For example, the “Connect to cloud/community db” pages have a section called “Optional arguments” which describe the -f flag for executing commands located in a file. That functionality seems to align more with “Running SQL queries” than connecting to a database (but note, commands run from a file can contain meta-commands, so we should be careful where we imply something is always a “SQL query”). (DISCUSSION)

Also, it looks like more instances of “setup” being used as a verb have snuck in. See [this PR](https://github.com/FeatureBaseDB/featurebase-docs/pull/65) for previous fixes. (TODO)

DISCUSSION
"...https://www.postgresql.org/docs/current/app-psql.html..."

The method I've put together gives users:

a generic starting point (connection, running queries, output, etc)
a set of flags/commands that can be run in this context,
structured the same as the SQL they'd be running (the point of FBSQL),
that isn't an endless scroll
The approach taken to FeatureBase docs has been to pull related information together into the contexts in which it'll be relevant for new users who may or may not be engineers.

DISCUSSION
"...While I appreciate the usage docs being split into various sections, it’s not clear how to navigate these pages when the information is not in an obvious place. For example, the “Connect to cloud/community db” pages have a section called “Optional arguments” which describe the -f flag for executing commands located in a file..."
I wrote it this way because the -f flag works on the CLI and if it doesn't contain connection details, those details need to be provided before the flag.
"...That functionality seems to align more with “Running SQL queries” than connecting to a database..."
There's documentation for running SQL from a flag in the running sql page as the flag is totally different -- \i or \include -- but does the same thing -- runs SQL contained in the file.
"... (but note, commands run from a file can contain meta-commands, so we should be careful where we imply something is always a “SQL query”)..."
Ok, so this is something I'll add to the connection and running SQL files, that -f, \i, \include files can contain meta-commands.

I'll look shortly, but is it OK with @travisturner?


Improve finding Device from Identifier

Thanks for this library, was the only one I could find that would work for my use case. I'm using this server side to convert device identifiers into human readable model/generation names.
If I could suggestion an improvement, it would be nice to have a more efficient way to find the model for an identifier. Currently I have to search through all the models of iPhone, iPod touch, iPad, iPadAir, iPadMini and iPadPro just to convert an identifier to a generation name. Should probably be searching through all the Mac devices as well since M1/M2 Macs can run iOS apps. Ideally there would be a dictionary lookup to easily and efficiently do this.
Also by having separate models for each type of iPad and Mac, despite the models being exactly the same, will make things more difficult to compare models etc.

Hey @grahamburgsma, thanks for reaching out. To confirm, are you after something like the following being generated?
protocol DeviceType {
    var identifier: String { get }
    var generation: String { get }
    // anything else that is common
}

let allDevices: [String: DeviceType] = ...
let generation = allDevices?["iPhone14,5"]?.generation

@ptrkstr Thanks for the quick response. Yes something like that would be great!

@grahamburgsma I've just realised that because multiple devices can have the same identifier it would actually be like the following:
let allDevices: [String: [DeviceType]] = ...
let generation = allDevices?["iPhone14,5"]?.first?.generation

They all have the same generation so it wouldn't matter which one you pull out.

@ptrkstr Sorry I missed that last message. Thanks for the explanation, makes total sense.
And yes, that should work great. Thank you!


add label syncer

examples:
syncer.syncLabel(1)
imgWithOneSegThatCorrespondsToTraMarker1 = syncer.fuse(inputSegs,inputTRA)

or
syncer.syncAll()
imgWithAllSegThatCorrespondToTheirTRAMarkers = syncer.fuser(inputSegs, inputTRA)

this seems to have been added in commit 2e7f916c0e46e5ac79996690bd2dec4749417489