Gif images are not showing in the Wiki Pages

@Jon007 Any idea how to solve this, I can not figure it out ? is this a Github issue or the problem is in the images?

It seems there was a bad images commit:
https://github.com/hyyan/woo-poly-integration/commit/c6677908d57d273a3633e3e03bdb793dfadb1b32?diff=unified
On further testing it looks like my Github Desktop on OSX is not handling the images properly at all.
I've just fixed manually by checking in the uncorrupted file versions again.
It's now working again on the wiki eg:
https://github.com/hyyan/woo-poly-integration/wiki/Product-Attributes

@Jon007 Great news , Thanks  :)


WebUI stops working for Qbit, Sonarr...etc

Describe the Bug
After running docker compose all containers are up but after 20-30 mins, all web ui fail to load (Sonarr ,Radarr, Qbit) mainly these three stop responding as in they dont load.
using protonVPN and Configured Correctly, tried the logs but nothing there to show me what could be, it's like the containers connected to the protonVPN network service are crashing/timing out, it just happened
Expected Behavior
they shouldn't crash
Steps to Reproduce
for me i just run the docker compose and it happens after some time
Screenshots

Logs of proton VPN Container

all containers are still running

this is what is showing
Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:
Linux Kubuntu
Docker version 20.10.23, build 7155243
Additional Context
the only way for the web ui to come back online is to docker compose down the containers and up them again while keeping ProtonVPN Running

Yes I'm using the free version of proton VPN, and during the timeout the connection, not that I saw it was getting dropped at the same time, as I can't even track what's making it go down, I'm suspecting that the issue is with VPN container, but the logs doesn't show anything out of the usual, I'm gonna try checking the connection before and during the timeout/unresponsiveness

The might be the culprit, if the VPN container is not stable the other containers connections' won't as well.
Try changing the server and/or purchasing a paid plan, that might help.


Run tests on BrowserStack

Hi
We just integrated Materialize test suite to run on BrowserStack. It will run across latest browsers of Chrome, Firefox, IE and Safari. The exposed command for this is: npm test:ci. It is using browerstack-runner for integration.
To enable it on your Travis build, you just have to expose BROWSERSTACK_USERNAME and BROWSERSTACK_ACCESS_KEY. You can set it at repository level. More information - here.
Let us know if you face any issues with this integration. We would love to get your feedback on your experience of using BrowserStack to run your test suite. :)
Thanks
Ankur Goel
- BrowserStack Automate

What tests does it run exactly?

It automatically runs the Jasmine tests against the 4 browsers from the settings file. Just a way to automate this step for CI. I think you have to contact BrowserStack for a free opensource plan.

Hi @acburst if you can share your email address with me, I can set you up with a Free open source plan from BrowserStack. My email is<EMAIL_ADDRESS>and I lead the Product team for the Selenium Automated testing product.

@acburst anything more you need from us? It'd be great if you could give our test suite a spin. Feel free to email me -<EMAIL_ADDRESS>so that I can set you up with a sponsored account.
thanks!


Add note on concurrency limit config to concurrency.md

Description

This PR adds information on concurrency.md about the existing concurrency limit config that needs to be changed if one wants to enable concurrency.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[x] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cd6fded). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #5103   +/-   ##
=========================================
  Coverage          ?   69.37%
=========================================
  Files             ?      211
  Lines             ?    10435
  Branches          ?      447
=========================================
  Hits              ?     7239
  Misses            ?     3196
  Partials          ?        0

Continue to review full report at Codecov.

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

LGTM

@bdoyle0182 thanks for merging, haven't been checking on GitHub in a while and missed the approval


chore: remove unsupported configurations ScopeWhenExpressionsToTask

This feature flag has been completely removed in v0.35.
Ref:

https://github.com/tektoncd/pipeline/pull/4715
https://github.com/tektoncd/operator/pull/2223

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[ ] pre-commit Passed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind misc

/retest

/ok-to-test

/retest

/retest

/retest


increase tolerance for testing

xref  - https://github.com/uscuni/sgeop/pull/67#issuecomment-2453552847

Since we are erroring out in the last test from test_simplify_network_full_fua(), we can be virtually certain that small changes in precision are the cause of the failures in dev starting several days ago.
I try increasing the tolerance incrementally by +0.1 to see where that gets us.

Seems to not be a tolerance thing, but a reversed coordinates things... https://github.com/uscuni/sgeop/pull/73#issuecomment-2453602330

resolved by #73


force: package: version: create fails to reference Case metadata on linux

Summary
Running sfdx force:package:version:create command fails to find references to Case object metadata when run on linux.
Steps To Reproduce:
I've opened a case regarding this advised to add an issue here and link the case to it.  The case contains the failing DX Project code.
Expected result
Successful package creation.
Actual result
Package creation fails.  Error message appears to indicate that Case metadata (standard or custom) does not exist.
{
  "status": 1,
  "name": "",
   "message": "Multiple errors occurred:\n(1) ClassA: Variable does not exist: CaseFieldA__c\n(2) ClassATest: Field does not exist: RecordtypeId on Case\n(3) ClassATest: Variable does not exist: CaseFieldB__c\n(4) ClassATest: Variable does not exist: CaseFieldC__c\n
   ...
   (490) ClassZTest: Variable does not exist: CaseFieldY__c\n(491) ClassZTest: Variable does not exist: CaseFieldZ__c\n",
   "exitCode": 1,
  "context": "PackageVersionCreateCommand",
  "stack": "Multiple errors occurred:\n(1) ClassA: Variable does not exist: CaseFieldA__c\n(2) ClassATest: Field does not exist: RecordtypeId on Case\n(3) ClassATest: Variable does not exist: CaseFieldB__c\n(4) ClassATest: Variable does not exist: CaseFieldC__c\n
   ...
   (490) ClassZTest: Variable does not exist: CaseFieldY__c\n(491) ClassZTest: Variable does not exist: CaseFieldZ__c\n    at Function.wrap (/usr/local/lib/node_modules/sfdx-cli/node_modules/@salesforce/core/lib/sfError.js:61:25)\n    at PackageVersionCreateCommand.catch (/usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/ToolbeltCommand.js:255:44)\n    at async PackageVersionCreateCommand._run (/usr/local/lib/node_modules/sfdx-cli/node_modules/@salesforce/command/lib/sfdxCommand.js:106:13)\n    at async Config.runCommand (/usr/local/lib/node_modules/sfdx-cli/node_modules/@oclif/core/lib/config/config.js:272:25)\n    at async run (/usr/local/lib/node_modules/sfdx-cli/node_modules/@oclif/core/lib/main.js:76:5)\nCaused by: Error: Multiple errors occurred:\n(1) ClassA: Variable does not exist: CaseFieldA__c\n(2) ClassATest: Field does not exist: RecordtypeId on Case\n(3) ClassATest: Variable does not exist: CaseFieldB__c\n(4) ClassATest: Variable does not exist: CaseFieldC__c\n
   ...
   (490) ClassZTest: Variable does not exist: CaseFieldY__c\n(491) ClassZTest: Variable does not exist: CaseFieldZ__c\n    at /usr/local/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/package/packageUtils.js:712:27\n
at processTicksAndRejections (node:internal/process/task_queues:96:5)",
  "warnings": [],
  "commandName": "PackageVersionCreateCommand"
}

System Information

Which shell/terminal are you using? bash

{
  "cliVersion": "sfdx-cli/7.172.0",
  "architecture": "linux-x64",
  "nodeVersion": "node-v16.18.0",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 1.3.1 (core)",
    "@oclif/plugin-commands 2.2.0 (core)",
    "@oclif/plugin-help 5.1.14 (core)",
    "@oclif/plugin-not-found 2.3.3 (core)",
    "@oclif/plugin-plugins 2.1.1 (core)",
    "@oclif/plugin-update 3.0.2 (core)",
    "@oclif/plugin-version 1.1.2 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.6 (core)",
    "@oclif/plugin-which 2.1.0 (core)",
    "alias 2.1.5 (core)",
    "apex 1.2.0 (core)",
    "auth 2.2.13 (core)",
    "community 2.0.3 (core)",
    "config 1.4.21 (core)",
    "custom-metadata 2.0.1 (core)",
    "data 2.1.4 (core)",
    "generator 2.0.5 (core)",
    "info 2.1.9 (core)",
    "limits 2.0.5 (core)",
    "org 2.2.7 (core)",
    "packaging 1.9.4 (core)",
    "schema 2.1.10 (core)",
    "signups 1.2.7 (core)",
    "source 2.0.17 (core)",
    "telemetry 2.0.2 (core)",
    "templates 55.1.0 (core)",
    "trust 2.0.4 (core)",
    "user 2.1.10 (core)",
    "@salesforce/sfdx-plugin-lwc-test 1.0.1 (core)",
    "salesforce-alm 54.8.1 (core)"
  ],
  "osVersion": "Linux 5.15.0-1021-azure",
  "shell": "unknown",
  "rootPath": "/usr/local/lib/node_modules/sfdx-cli"
}

Additional information
Using same package files to create a package version...
Works on cli version 7.170.0 on linux or win32-x64
Works on cli version 7.171.0 on win32-x64 only

hi @6ithu8u53r sorry for the delayed response, have you tried using the force:package:beta commands? It might be fixed in there

hi @6ithu8u53r sorry for the delayed response, have you tried using the force:package:beta commands? It might be fixed in there

Yes, but it doesn't work due to issue #1750.  We are also running the commands in an automated pipeline and it's easier for us to use <IP_ADDRESS> until it's fixed so we don't need to make modifications to our codebase.

yes, I'll close this to keep #1750 as the original

Hi @6ithu8u53r what is the exact command you're executing?

Hi @6ithu8u53r what is the exact command you're executing?

I did obfuscate a few values, but this the the command with all the switches that is failing.
sfdx force:package:version:create --json -c -v<EMAIL_ADDRESS>-x -b refs/pull/28/merge -d source/sfdx/force-app -f ./config/project-scratch-def.json -t example/sfdx-project__20221109.1 -w 320

This appears to be fixed in the latest (7.179.0) release.

Thanks for following up here @6ithu8u53r! Glad you're unblocked!


root.json expired

I let the root.json file expire and now my app is erroring out on load. Will I have to publish a new build or can I update the root.json on the back-end to keep things going?

@abenzick It should be sufficient to re-sign the root.json file.
(instructions will follow)

On the command line (also see output of tufup sign --help):
tufup sign -e <number of days until expiration> root <path to root key(s)>

This will update the expiration date and re-sign the root.json file. You'll end up with an updated root.json and a new version, e.g. 2.root.json (which is a copy of the updated root.json).
All versions of root.json should be made available on the server.

On the command line (also see output of tufup sign --help):
tufup sign -e <number of days until new expiration> root <path to root key(s)>

This will update the expiration date and re-sign the root.json file. You'll end up with an updated root.json and a new version, e.g. 2.root.json (which is a copy of the updated root.json).
Note that all versions of root.json should be made available on your update-server.

Back in the green! Simple and painless. THANK YOU FOR THE PROMPT RESPONSE. You saved me a lot of headaches today. Do you have a donation page?

Back in the green! ...

That's good to hear. Thanks for the feedback!

Do you have a donation page?

No, I don't, but thanks for the kind gesture. I'm happy to help. :)


CI: vcpkg verification PR should include all packages at all times, and unconditionally run daily

Note: I am talking about the verification PR and not the release process.
Issue 1: https://github.com/microsoft/vcpkg/pull/17119 has different packages depending on the time of day.
It should push all packages at all times.
Issue 2: It won't update the package if daily live test run fails.
It should push packages regardless whether live test is failing or not.
Live tests could be affected by timeouts, CI issues. Could be valid bugs.
But if the change was good enough for the CI that ran non-live tests, it is good for vcpkg validation as well.
It may discover useful issues that can be fixed independently of live test issues - for example, if sonething does not compile on ARM or UWP.
Also, given that we only update vcpkg PR once daily, we are missing the window to validate changes too often. We should be able to open that PR at any time and see whether all packages do build as of the last <=24 hours.

For Issue 1... Does this mean every merge to main updates the packages?

I was going to answer with a question if we could make it to run on a schedule, once or several times daily, but I actually like the idea of running every time we push to main - I hope it is not too often for the vcpkg's CI to handle. We don't seem to have that many daily commits, so I hope it is ok.

Resolved: Let's do 1x/day validation and then if we need more frequent, we can do that.


add travis-ci status into readme

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


blueprint plugin polyhedral mesh support

Description

Adds polyhedral mesh support to the blueprint plugin. Now, if you ask visit to read a polyhedral mesh, it will automatically convert the mesh to a tetrahedral mesh, as well as convert the fields as needed. It will also keep track of original element ids so that you don't see the new lines from the resulting tetrahedra, and instead see the original topology.
Type of change

[ ] Bug fix
[x] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?

I added new tests using the 3D and 2D polytessellation I worked on last summer. I tested on rztopaz.
Checklist:

[x] I have followed the style guidelines of this project.~~
[x] I have performed a self-review of my own code.~~
[x] I have commented my code where applicable.~~
[ ] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.~~
[ ] I have added debugging support to my changes.~~
[x] I have added tests that prove my fix is effective or that my feature works.~~
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[x] I have added new baselines for any new tests to the repo.~~
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~
[x] I have assigned reviewers (see VisIt's PR procedures for more information).~~

updating release notes now... should I be doing either of these?

[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.

updating release notes now... should I be doing either of these?

[ ]  I have made corresponding changes to the documentation.

Are there any user-level impacts for this changes? Are there new read options for example. Are existing plugin limitations already documented (e.g. works only for multi-block structured data -- that isn't true but if it was and it was documented, then I'd say we should update that verbiage to indicate it now works with arbitrary polyhedral meshes).
If none of the above (or anything similar in spirit) applies, strikethrough that one by adding leading ~~ just before the [ at beginning of line.

[ ]  I have added debugging support to my changes.

Did you add any debug5 <<  kinda statements to the plugin or any other coding aimed at helping developers debug issues when/if they occur? If so, check that box too, otherwise strikethrough it too by adding leading ~~

merged w/ develop in #17494


il2cpp: libil2cpp.so has been loaded, but such event hasn't been detected

hello im running a code from snippet but why i got this error
npm run spawn com.mobile.legends

<EMAIL_ADDRESS>spawn
run() { frida -U -f "$1" -l _.js --runtime=v8; }; run com.mobile.legends

 ____
/ _  |   Frida 16.1.7 - A world-class dynamic instrumentation toolkit

| (| |
> _  |   Commands:
// |_|       help      -> Displays the help system
. . . .       object?   -> Display information about 'object'
. . . .       exit/quit -> Exit
. . . .
. . . .      Connected to 22071212AG (id=C6HISK9LB67LFQKB)
Spawned com.mobile.legends. Resuming main thread!
[22071212AG::com.mobile.legends ]-> il2cpp: libil2cpp.so has been loaded, but such event hasn't been detected - please file a bug
Error: access violation accessing 0x0
at NativeFunction. ()
at Object.initialize (il2cpp/module.ts:19:24)
at Object.perform (il2cpp/perform.ts:5:36)

Hi, thanks for reporting. Are you using an emulator? If so, you need to run Frida with --realm=emulated. Please follow https://github.com/vfsfitvnm/frida-il2cpp-bridge/issues/437 and let me know!

im using android device not a emulator
my device: xiaomi 12T
and im using USB to connect with adb and frida

Thanks.
What happens if you eval:
Process.getModuleByName("libil2cpp.so")

throught the Frida REPL?

hello
i Got this response
`[22071212AG::com.mobile.legends ]-> Process.getModuleByName("libil2cpp.so")
{
"base": "0x7695243000",
"name": "libil2cpp.so",
"path": "/data/app/~~2aUqt56EA83TXpBt_YIxpg==/com.mobile.legends-xXkqYG7V3IWVEDF9rdnVXA==/split_config.arm64_v8a.apk!/lib/arm64-v8a/libil2cpp.so",
"size": 237568
}`

Thanks, it looks like this game (or your device) is using a strategy I'm not aware of to load libil2cpp.so

@dziunincode69 have you solved this problem?

I guess it's due to the private customized version of Unity

I guess it's due to the private customized version of Unity

I am learning to try to solve this problem on my own


Could not able to install local visix extenion

I have created a basic helloworld extension in my location machine ( Windows machine)
Then I have packaged it.
And now I am trying to install it in running code-server but giving me the following error

Unable to install extension 'xxx.xxx' as it is not compatible with VS Code '1.43.2'

I am using the docker container of the code-server

You might need to change the value of engines.vscode in the extension's package.json.
From https://code.visualstudio.com/api/get-started/extension-anatomy#extension-manifest:

engines.vscode: This specifies the minimum version of VS Code API that the extension depends on.

Please post the VSIX.

Please post the VSIX.

How to post VSIX to code-server marketplace?

@chocoford You can't at the moment. See #1299
When I asked him to post, I meant upload it to GitHub we so can investigate it.

Closing due to inactivity


Update all_plants_ferc1 opex_nonfuel column to feed from all tables

Discussion
Right now, the opex_nonfuel column only comes from the steam table. This is the column that the RMI Optimus formatted table relies on for the Total O&M Cost column. In order for this column to represent values from all the FERC tables we need to do some simple calculations.
Discussed in # 216 in the rmi_ferc1_eiarepo
The stated calculations are:

small gens: opex_nonfuel = opex_total - opex_fuel
hydro: opex_nonfuel = opex_total
pumped storage: opex_nonfuel = opex_total

Questions

This is technically a transformation, should this live in the transform step for each of the individual tables or in the output layer where the ferc1 tables get combined? My vote is for the transform tables so that the column gets added to those individual tables as well. This will also entail updating all the metadata to include the new column in those output tables.

I like this idea! The calculations sound pretty straightforward and clear to me. I think doing these calculations in the output layer is more in line with the overall pudl conventions of doing most of the (even minor) calculations post the normalized db tables.
(It'll also be easier to integrate/update w/o reloading the full db 😎 )

I think doing these calculations in the output layer is more in line with the overall pudl conventions of doing most of the (even minor) calculations post the normalized db tables.

That makes sense -- it is kind of like a teeny tiny analysis layer

Idk! I think we do a ton of these little calculations in the output layer right now (so much calculating of capacity_factor in pudl/output). We've always said the db tables were the tidy, cleaned tables and the outputs were where we do calculations. This may change slightly if we move our denomalized output tables into db views or otherwise publish them as db tables.... but even in that design configuration we has talked about keeping the normalized/"raw"-ish pudl tables separate from the de-normalized tables w/ simple calculations in them

We've always said the db tables were the tidy, cleaned tables and the outputs were where we do calculations.

That's fair! I can get behind that. I just couldn't remember if we added things like report_year or something in transform or output.


Self-nominate mimowo for approver in impl packages

What type of PR is this?
/kind documentation

What this PR does / why we need it:
My reviewing contributions in Kueue is 111 merged PRs by now: https://github.com/kubernetes-sigs/kueue/pulls?q=is%3Apr+is%3Amerged+reviewed-by%3Amimowo+-author%3Amimowo.
I was an active reviewer in areas such as: MultiKueue, Provisioning controller, tests, as well as smaller PRs across the codebase.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
I would like to make my nomination gradual, thus only implementation-related PRs for now. In particular, I would still like to leave approval for API packages to the current approvers.
Does this PR introduce a user-facing change?

NONE

/assign @tenzen-y @alculquicondor

/lgtm
/approve
/hold
for @tenzen-y


in our integration testSet we'd like to use a unit test's class

there is a common a test class which we'd like to use in both normal unit test and integration test. the strange thing is that the generated eclipse classpath contains it since it build and do not gives error. but in our integration testSet we can't use it from command line ./gradlew integration because of the missing class from the unit test. unfortunately i can't add
testSets {
     integration { extendsFrom test }
}

since it gives error.
of course we can put the class into the main sourceSet but we wouldn't like to do so.
is there any solution to this?

Hi, this is pretty much what the "test libraries" were introduced for. They're intended for supporting code shared between multiple test sets. Please check the readme for a description and usage.
Of course, you could try the "quick and dirty" way and just add a dependency on the test source set's output:
dependencies {
    integrationTestImplementation sourceSets.test.output
}

ok but AFAIS the "nornal" java buildin test configurtation is not a testSet so i can neither write:
testSets {
    integration {
        imports test
    }
}

nor
testSets {
        test
        integration {
        imports test
    }
}


Shortcut to launch notice messages in custom actions?

Hi! Thanks for the awesome gem!
Sometimes I need to create a custom actions to do specific purposes, and of course tell to user what happened, but I can't find a good way to do that, the way that I know is something about it:
def inactive
  @object.update(...)

  respond_with @object, location: my_path,
                              notice: I18n.t('flash.actions.update.notice', resource_name: Model.model_name.human)
end

It will be nice if I can do something like that:
respond_with @object, location: my_path, notice: :update_notice)

Thanks!

Thanks for the report, but considering how this might not a recurrent case I believe it's better to deal it on the application code by manually setting an flash.actions.inactive.notice translation String or force a more RESTful approach when designing controllers, and not add any more logic to the FlashResponder for dealing with this - even overriding interpolation_options on the controller class might be enough to deal with some edge cases of customizing the i18n lookup.


Where is the web app code

Description says this is web app example for servo driver, but it seems to be missing.
Only Arduino sketch files and servo driver is here.
Is the web app code (both client side and server side) available?

All of it runs on the ESP, the "front" is in https://github.com/waveshare/Servo-Driver-with-ESP32/blob/main/ServoDriver/WEBPAGE.h
The "back" is mainly at https://github.com/waveshare/Servo-Driver-with-ESP32/blob/main/ServoDriver/CONNECT.h


hooking up observers and clicking for ui node

Makes the newly merged picking usable for UI elements.
currently it both triggers the events, as well as sends them as throught commands.trigger_targets. We should probably figure out if this is needed for them all.
Objective
Hooks up obserers and picking for a very simple example
Solution
upstreamed the UI picking backend from bevy_mod_picking
Testing
tested with the new example picking/simple_picking.rs

If there's an obvious use case for using events, I would keep them in, otherwise I would remove them for now. It's easier to add them later than removing a feature.

Did not know observesrs did not use events internally, but for now, removed the events that was not needed for further processing and changed them to Triggers instead.
I agree that we can add them if needed.

I'd like to improve the docs further (and unify this with the rest of bevy_ui), but this is a good step forward. Happy enough to merge this now.


Moving graphql-ws to a dynamic import and display a nice message when not found

Fixes #2405

Updated 👍 @thomasheyenbrock

Derp. Should be good now!


Changed in Search Bar title name

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Description
[Please include a brief description of the changes or features added]
Type of PR

[x] Bug fix
[x] Feature enhancement
[x] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

There have some conflict issue check it once and make a new PR. ok

There have some conflict issue check it once and then make a new PR ok.


ci: Move CFFormat to separate, speedy workflow

Brings the CFFormat task into 2022 :calendar:

Drops CFFormat runtime from 1:15 to under 30s :racing_car:
Declutters the PR workflow :wastebasket:
Intros a new, simplified standalone CFFormat workflow :package:

Example run: https://github.com/michaelborn/totp/runs/5185020861?check_suite_focus=true

Thanks!


docs: add inputs for difyChatbotConfig

Add the description about new options for difyChatbotConfig. This is related to the merged PR langgenius/dify#6376.

Failed to generate code suggestions for PR


How is the target predictive?

Hi, I stumbled across this repo looking for an LSTM implementations for FreqAI and the PyTorch regressor you've created is great, so many thanks for sharing your work on that.
I'm just confused about the implementation in the example strategy really... the set_freqai_targets method uses a lot of indicators to create the score to be used as a target variable but these all appear to be based on rolling windows of past data, essentially an aggregation of standard indicators. Where is it looking ahead so it can make predictions about the future score? e.g. All of the other example FreqAI strategies I've seen use some variation of shift(-n) for target variables.

Hi @JunkMyFunk,
Please find my response on #7
All the best.


Documentation: Zero downtime

(tbd)
Write a document with advice on how to ensure that the Gateway stays up and always handle traffic.
Example:
zero downtime upgrades
scaling without downtime
things that could go wrong and some advice how to prevent them
example: invalid NGINX configuration
example: OOM
more examples.

closed because this work cannot be defined yet because the underlying features have not been implemented

I would rather delete situations these rather than falsely close them as complete.
We have others that should meet this fate as well.


Make JS varaible "current" local to the event handler

Save a tiny amount of memory by not keeping the value around. Only used by the event handler. The value does not need to be remembered across different invocations of the event handler.

Thank you


James darkmode

added darkmode should hopefully not cause issues when merging

This looks great, one thing, is it our intention to change the card color on the login page? It is a bit hard to see the text on the login.


[WFLY-5151] Reject the new resource mdb-delivery-group when tranforming the model, plus some minor fixes for the xsd version bump

Linux with security manager Build 2951 outcome was FAILURE using a merge of da9bf4312651374aa100021e3126ff32ee4aa652
Summary: Tests passed: 611, ignored: 195; exit code 1 (new) Build time: 00:03:09

Linux Build 8112 outcome was FAILURE using a merge of da9bf4312651374aa100021e3126ff32ee4aa652
Summary: Tests passed: 611, ignored: 195; exit code 1 (new) Build time: 00:03:32

Windows Build 3146 outcome was FAILURE using a merge of da9bf4312651374aa100021e3126ff32ee4aa652
Summary: Tests passed: 611, ignored: 195; exit code 1 (new) Build time: 00:05:30

Retest this please

I'm closing the PR... somehow, team city is testing with an old commit, as the error has been fixed in my branch.


String dragModel treated as an array

When the ngModel of the draggable is just a String, it is treated as an array. So instead of adding a single entry to the droppable array model, an entry is added for each character of the String.

@amahfouz any working demo would be helpful to understand the real problem.


Publish new version with exported Recorder

Hi there, thanks for the great work!
I'm looking into using this for Common Voice. Atm we're using MediaRecorder which (as you well know) doesn't let us support Safari & Edge. I investigated using a lame encoder written in JS but that didn't seem to work for iOS. So my next thought was to go full WASM and fortunately I found that somebody already did the heavy lifting. Awesome! Also can confirm that it works with iOS 11.3 (discussion in #3 was very helpful) & Edge.
I was wondering if you could publish the most recent version to NPM, so that we can use the recently extracted Recorder?

Published v0.3.0.

Wow! That was lightning-fast. Thank you! 🙂


New World template - search icon

MW 1.3.1
New World template 5.5
How to disable search icon since it does not work in a site without blog/shop?

Hello, there are Template settins from where you can stop the search or other settings if needed.
Please look at the screenshot below -

@Radanovn I think the screenshot you posted if from a previous issue you replied to.

Yes, in Template Settings we have:

Show search bar
Show shopping cart
We can set to "no".
Thank you


Update PULL_REQUEST_TEMPLATE.md

Pull Request (PR) description

Improving the CHANGELOG.md entry
Adding that entry in CHANGELOG.md is mandatory.

This Pull Request (PR) fixes the following issues
None
Task list

[ ] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Label this temporary as invalid so it should not count towards my personal hacktoberfest count.

Codecov Report

Merging #522 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #522   +/-   ##
=======================================
  Coverage   91.39%   91.39%
=======================================
  Files          17       17
  Lines        2661     2661
=======================================
  Hits         2432     2432
  Misses        229      229

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 22802e7...59bed1e. Read the comment docs.


Fixing up more classes for #88

adds (average) heating/cooling demand sensor
standardize tags to heat/cool rather than heating/cooling

@shreyasnagare let me know if there are any more outstanding classes left for #88 and I can add them in this PR


Integrate[Sqrt[x^3 + x], x] doesn't work

The following antiderivative cannot be found by Rubi. However, it doesn't say so but rather brings many messages:
$LoadElementaryFunctionRules = True;
<< Rubi`

Integrate[Sqrt[x^3 + x], x]

This results in the following. Why is that?

The Rubi 4.15.2 available on the UWO website gets it in 6 steps.  What version are you running.  Try single stepping to see where it goes haywire.

I'm using version <IP_ADDRESS>. The error comes from evaluating this Int along the way
Int[1/Sqrt[1 + x^4] - x^2/Sqrt[1 + x^4], x]

For expression, the very early 13th DownValue matches (please ignore the Hold
HoldPattern[Int[u_, x_Symbol]] :> Simp[IntSum[u, x], x] /; SumQ[Rubi`Private`u]

Therefore, this call goes haywire on my machine
Rubi`Private`IntSum[1/Sqrt[1 + x^4] - x^2/Sqrt[1 + x^4], x]


Feat/keploy ci

Ticket Link

Related Links

Description

Steps to Reproduce / Test

Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are [changed or added]
[ ] Relevant documentation is changed or added (and PR referenced)

GIF's

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


in the logs, misreported ID

https://curation.pombase.org/dumps/latest_build/logs/identifier_problems.txt
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: FAM155A
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: FAM155B
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: SGD:S000029663
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: SGD:S000029664
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: SGD:S000029665
CDS SPAC750.08c.1 from gene: SPAC750.08c: can't find feature for: SGD:S000029662
the first 2 aren't SPAC750.08c, they are SPAC1F5.08c ?

I think I've fixed this, but not in time for tonight's load.  It should be OK from Saturday morning onwards.

This is still buggy.  I'll look at it again.

This is still buggy. I'll look at it again.

Fixed again.  I'll check after Friday night's load.

There are still log messages but they make sense now: https://curation.pombase.org/dumps/latest_build/logs/identifier_problems.txt


Reenable git extension support

Fixes #1078
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Note: The nbdime and git versions listed are not yet released, if we decide to release using pre-release versions then they will need to be updated. This also means the tests/build will fail until they are released or we update the versions to pre-release ones

Note: The nbdime and git versions listed are not yet released, if we decide to release using pre-release versions then they will need to be updated. This also means the tests/build will fail until they are released or we update the versions to pre-release ones


Tool language version determination may be incorrect for release branches

Currently the flutter tool reads the value of bin/cache/dart-sdk/version to determine what the maximum language version is. This strategy may not work with local engines rolled through release branches.
See also https://github.com/flutter/engine/pull/25595
We either need a way to configure this for our CI, or to take the local engine path into consideration

I think this should be fixed instead by having the engine test scripts clone the appropriate RC branch of the framework, which should have a matching dart sdk version.


Fix/fixed measures

Given the policy wizard
When I am creating a cube
And I leave fixed measure name and value empty
Then cube is saved successfully
Given the policy wizard
When I am creating a cube
And I leave fixed measure name empty
And I introduce value to fixed measure value
Then error is shown below fixed measure name
Given the policy wizard
When I am creating a cube
And I leave fixed measure value empty
And I introduce value to fixed measure name
Then error is shown below fixed measure value
Given the policy wizard
When I am editing a cube
And I remove fixed measure name and value
Then fixed measure is not added to policy json

:shipit:

Comment accepted. Will treat it ASAP


postgres: add cancel_token

This PR adds the ability to cancel postgres queries from AsyncPgConnection.
https://github.com/weiznich/diesel_async/discussions/57

Thanks :+1:


Rail Departure Velocity and time

From what I've known for so long using RocketPy (you can correct me if necessary), the rail departure time is assumed to be the time when the Rocket CM crosses the top of the Launch Rail. However, the main international rocket competition (SACup) defines rail departure time as "the first instant in which the launch vehicle becomes free to move about the pitch, yaw, or roll axis."[DTEG - SACUP]
I believe that we must make some changes to the Flight class so that the railDepartureTime variable be considered as the time that the upper button crosses the top of the rail.

I agree with you completely in this one, @Gui-FernandesBR. Initially, I wanted to implement a set of new equations of motion which described the flight of the rocket with a single rail button still sliding along the rail.
However, the equations of motion have proven quite challenging and this will take a while to correctly derive, implement and test.
For the meantime, do you have any suggestions on how to mitigate this issue?

I made a pull request a few minutes ago and I believe the problem was already solved, at least until we implement a set of new equations of motion.
I created the 'effective1RL' attribute to be used as the new Rail length in all simulations. It discounts the distance between the nozzle exit and the upper button. However, the original rail length inserted by the engineer is still saved in another attribute.
Here's a copy of the script created:
        upperRButton = max(self.rocket.railButtons[0])
        lowerRButton = min(self.rocket.railButtons[0])
        nozzle = self.rocket.distanceRocketNozzle
        self.effective1RL = self.env.rL - abs(nozzle - upperRButton)
        self.effective2RL = self.env.rL - abs(nozzle - lowerRButton)

Just for information purposes: the pull request being cited is #36.

@Gui-FernandesBR, did you replace all occurrences of self.env.rL with self.effective2RL in the Flight class? Or just the ones used to compute rail departure conditions?
By the way, perhaps outOfRailXxxxxx  should be renamed to railDepartureXxxxxx.

I will close this issue since there is already another one about implementing the rest of code needed: #28.


Adding fix to Sonoff DUAL power status

The Dual returns the message { POWER1 : ON } or { POWER2 : ON } whereas the regular Sonoff switch is { POWER : ON } This patch makes it look at the correct Power only.
Also, @Arduingo your fix was throwing an error, where the no_callback was not defined.  I set that to null.

This should fix #8

Hi guys,
Am I wrong or the fix for the status was merged in the code?
I still have the problem on my Sonoff Dual!

Me too, same problem :-(


CI: Publish VS Code extension to OpenVSX Registry

Description
This will make the extension more discoverable and more easily accessible to users.
Currently, This VS Code extension is only available at Microsoft's Visual Studio Marketplace. But, the proposed change would be super helpful, if we can release it to Eclipse Foundation's Open VSX Registry. This would help products like Gitpod, IDX, VSCodium, etc. to use this VS Code extension directly.
ℹ Actions Items required from Admins

[ ]  Create an access token
[ ]  Login and sign the Publisher Agreement

[ ]  Create an Eclipse account
[ ]  Create a namespace/publisher.

[ ]  Add access token to GitHub repository's Secrets. Name it OPEN_VSX_TOKEN.

Thanks @Siddhant-K-code! I'm going to the process of setting up the account and claiming the OpenFGA namespace https://github.com/EclipseFdn/open-vsx.org/issues/2057

I completed the process and added the token

Thanks for your contribution @Siddhant-K-code!

@aaguiarz / @rhamzeh, I think, we need to manually run Publish workflow once. Because I can't yet see the Extension on OpenVSX registry. It should be up here: https://open-vsx.org/extension/openfga/vscode-ext
(And, Also, we need to update this list in README)

Thanks @Siddhant-K-code , @aaguiarz
The extension is published here: https://open-vsx.org/extension/OpenFGA/openfga-vscode


Selecting login on a small screen device, directs to home in redbasic.

From @Gadget-Guru on October 21, 2014 7:56
Login and Home are in the same position on small screens, preventing the selection of login.
Copied from original issue: friendica/red#654

From @Gadget-Guru on October 21, 2014 16:57
It has been fixed.


Pinpoint collector performance when receiving more traffic from more than 100 JVM servers

Quick question here - currently we have on-boarded around 50 JVM servers to my collector box
we are planning to add another 50 more JVM's to the box
Is there any specific parameters in collector properties which would limit us on adding more JVM's ?
Like whether we need to increase any of the parameters in collector.properties to ensure good performance when more than 100 JVM send data to Collector box
great if could you share some context on this ?

Hi dinesh4747
Thank you for using Pinpoint more actively.
Unfortunately, there is no option to limit agent count.
It's hard to know exactly because it differs from server spec or data transfer from agent. (In our case, we have on-boarded around thousands of agents to our's collector box.)
If the collector can not accept data, a RejectedException will be thrown in the collector log.
In this case, use the following options to change the settings according to the environment of the collector.
pinpoint-collector.properteis
# number of tcp worker threads
collector.tcpWorkerThread=128

# number of udp statworker threads
collector.udpStatWorkerThread=256

# number of udp spanworker threads
collector.udpSpanWorkerThread=256

In addition, you can put bulk the data from the collector to hbase using the following options:
It's a better performance for us. However, please be sure to test it because it may be different from situation.
hbase.properties
# enable hbase async operation. default: false
hbase.client.async.enable=true
# the max number of the buffered asyncPut ops for each region. default:10000
hbase.client.async.in.queuesize=10000
# periodic asyncPut ops flush time. default:100
hbase.client.async.flush.period.ms=100
# the max number of the retry attempts before dropping the request. default:10
hbase.client.async.max.retries.in.queue=3

thanks.

Hi @koo-taejin
Thanks for the quick response
I am seeing the below rejected exception on UDP Span in my collector logs
2017-06-06 22:46:47 WARN  RejectedExecutionCount=1113200
2017-06-06 22:46:47 WARN  RejectedExecutionCount=1113300
2017-06-06 22:46:48 WARN  RejectedExecutionCount=1113400
2017-06-06 22:46:48 WARN  RejectedExecutionCount=1113500
2017-06-06 22:46:48 WARN  RejectedExecutionCount=1113600
2017-06-06 22:46:48 WARN  RejectedExecutionCount=1113700
2017-06-06 22:46:53 WARN  RejectedExecutionCount=1113800
Kindly let me know whether this would be fixed on increasing the collector.udpSpanWorkerThread=256

Try increasing the collector.udpSpanWorkerThread option,
but if the exception still occurs, then it seems that the time to store in hbase has reached the limit.
In this case, I suggest to change the configuration to the bulk put type,
and If the exception still occurs, then I recommend to increase the hbase.
thanks.

Increase the Hbase in the sense - do you want me to increase the Heap size of Hbase
In other terms I am also seeking for better options to get more write performance in Hbase
Do you follow any best practices to on specific Hbase configuration to get good write performance?

It means a way to increase the performance of Hbase. (like adding server, upgrading spec, tuning Hbase)
Hbase operate in the operation team, so I do not know best practice exactly.
So, Hbase best practice would be more helpful to ask the Hbase community

Hi @dinesh4747 ,  please help me in hbase tuning


Feat: Add/Retrieve Codable objects in a session

This PR adds two function to add or retrieve codable objects from a session as a fix for issue #53.
The two function are:
public func add<T: Encodable>(_ value: T, forKey key: String) throws

public func read<T: Decodable>(as type: T.Type, forKey key: String) throws -> T {

Tests, Jazzy docs and the README have been updated to cover the new API.

We have changed this to use subscript with Codable instead of now functions. the API from aboe would now be:
session["key"] = codable
let retrieved: Type? = session["key"]


onHardwareBackPress behaviour

Hello guys,
I used onHardwareBackPress so that when the user presses the back button the dialog gets dismissed like this :
onHardwareBackPress={() => {                     onDismiss();                     return true                 }}
But the problem i'm facing is that after i press the back button the dialog gets dismissed, but the back button does not work properly anymore as long as the dialog component is present.
Am i not using this property properly or is this a bug ?
Thanks.

Hi @Boubaker93 Thanks for report. This issue has been fixed in v0.17.3

@jacklam718  Hi I think this problem is back again(0.18.2)


Tag Projection

Hi there - I'm working with complex documents feature densely tagged sentences (sometimes up to 4-40+ tags) and was wondering if the tags-projection method is 1. Language specific or 2. whether it can be tuned/trained somehow.  Is there any information about this feature before I dive in and study the code?  Many thanks!

Hi @nonsequitir
the tag projection feature is a 2-steps process:

First the system computes the alignment between the source and the target sentence with FastAlign model.
Then the tags are projected with some hard-coded heuristics using the computed alignment.

The first step uses the aligner model, so this step is a classic machine-learning task. The second part is hard-coded into this class: XMLTagProjector.java

Awsome, I see that now.  Thanks @davidecaroselli !

I'm closing this issue for now.
If you have any other question or something you want to discuss, please don't hesitate to re-open it!


At present, how do you handle errors in JS? When the program crashes, do you give up submitting the currently started span or do you have additional processing?

At present, how do you handle errors in JS? When the program crashes, do you give up submitting the currently started span or do you have additional processing?

There is no handling in the span processors included in the OTel SDK. Only ended spans are exported. Also no dedicated handling for ended spans somewhere in transit over the wire.

Yes, at crash time nothing special is done.
I'm not sure about anything planned regarding this. I guess a generic solution should start in OTel spec and not in OTel js. It requires quite some thought because e.g. backends need to be prepared to receive partial spans. Also a crashed application is in an undefined state so accessing any data results in undefined behavior. It's not that unlikely that during the crash processing again something crashy happens.

ok, thank you


Record pages rework

Personally I'm ready to approve but I pulled up the v4 versions of these pages and I wanted to flag some stuff from there that we could either a) drop for v5, b) try to get done now, or c) make separate issues for
for genes, I think it's be good to put the concept ID up in the header next to the gene name (maybe greyed out/unbolded, and either spaced away or flexed to the right side). This is not actually a field correctly exposed by graphql right now (lol) so I will make a ticket in a sec

I agree that it would be good to provide the concept id next to the name of the drug at the top (in grey text or some kind of subtext format for sure). The improvements to normalization are intended to be one of the update focuses for v5 so providing that information in a visible location seems prudent.
For the big category pills, those are present and represented in the categories box on gene record pages, but they definitely don't have the same 'pop' as the v4 implementation. I don't think it is necessary for this immediate PR, but maybe we can think about ways to make those clinically relevant categories pop out more (or maybe we just copy paste from v4)
For the drugs pages, I don't think our drugs will have associated categories. That is mostly a gene record associated data point (outside of immunotherapy, antineoplastic flags) so I think we can probably drop that from the drugs pages. For the 'Active' box, my original intent was to show some representation of market-active products that could be referenced easily for clinical use cases using the fields below:

Originally that was intended to just be application numbers linked out to label information, but in its current form I am totally open to representing approval information there instead and revisiting labeling in a later issue/update.

 We should brainstorm this going forward


Plugin not working with Redmine 4.1.1

@koledas patch looks overall to be working with 4.1.1 apart from listing issues. with the normal user view, to list issues only assigned to them, those that are assigned with custom field, do not show in list. if we switch user group to show all issues, then they will show (with filters, it works perfectly, but is not ideal, since users see more than they might need).
To Reproduce
just create a task in a project that is assigned to a role that can see only its own assigned tasks, and make sure this is under new custom fields box.
Expected behavior
filtering should take into account additional assignes in custom field
Additional context
it is probably ommited with 4.x upgrade done by @koledas, as it only happens if user role can only see tasks assigned to them.

Thank you for report. I'm fixed this bug. Try those changes.

thanks, this is fixed now. :)


[WIP]Added cleveland_museum_of_art.py with respect to issue #255

Fixes #255
Description
@mathemancer I am sending you this PR just to show what work I have done till now. There's still long way to go in fixing this issue. The script is working in my local system, I have used DelayedRequester class and also ImageStore wherever it was necessary. Please review the code and let me know what changes I have to make(if any) to the script. I haven't written any test suite for this, so that is still left. I have picked up most of the code and ideas from wikimedia_commons.py. I will write proper comments and description for the newly written script, once you confirm that the code is correct.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@mathemancer  Any updates ?

@kgodey , @mathemancer Seems busy with other tasks. Can you please review this PR for me. I am not asking to merge this yet. I just want to confirm that I am on the right track.
Thanks

@kgodey , @mathemancer Seems busy with other tasks. Can you please review this PR for me. I am not asking to merge this yet. I just want to confirm that I am on the right track.
Thanks

Sorry to take so long, I had a big deadline last Friday, and couldn't get done with this review till now.  Let me know if you need any more info.

@mathemancer  I have made the required changes, please review once again.Thanks


Making NormalizedDateSegmentNameGenerator use joda time to keep consistency other time handling

Context
For a Kafka ingestion use case, the time column was configured with a time format that was working as expected. But when configuring a minion task leveraging the SegmentProcessorFramework, a date format exception was thrown with the same exact time column/format:
Caught exception while parsing simple date format: yyyy-MM-dd'T'HH:mm:ss.SSSSSSSSSZ with value: 2023-03-07T21:18:05.072784135Z at org.apache.pinot.segment.spi.creator.name.NormalizedDateSegmentNameGenerator.getNormalizedDate(NormalizedDateSegmentNameGenerator.java:142)

Specifically the realtime ingestion was using SegmentColumnarIndexCreator, which uses joda time DateFormatter for parsing time format into epoch millis (See SegmentColumnarIndexCreator).
But the minion task was configured to use the NormalizedDateSegmentNameGenerator which failed since it uses another date time format library, java.text SimpleDateFormat. This library seems to be stricter that the joda time library and if a time column values has literal ‘Z’ in it, the time format must have the Z in single quotes. E.g. "2023-01-01T12:00:00.111111111Z" needs a time format of "yyyy-MM-dd'T'HH:mm:ss.SSSSSSSSS’Z’" otherwise it will fail.
Changes
This PR makes the NormalizedDateSegmentNameGenerator also use the joda date time formatter. Joda is no longer maintained and java.text seems to be outdated. This PR aims to unify the time library until we are ready to make the switch to use the newer java.time library.
Instructions:

The PR has to be tagged with at least one of the following labels (*):

feature
bugfix
performance
ui
backward-incompat
release-notes (**)

Remove these instructions before publishing the PR.

(*) Other labels to consider:

testing
dependencies
docker
kubernetes
observability
security
code-style
extension-point
refactor
cleanup

(**) Use release-notes label for scenarios like:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

Codecov Report

Merging #10721 (f7de6fb) into master (b190599) will decrease coverage by 54.63%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10721       +/-   ##
=============================================
- Coverage     68.47%   13.85%   -54.63%
+ Complexity     6450      439     -6011
=============================================
  Files          2119     2065       -54
  Lines        114203   111718     -2485
  Branches      17244    16947      -297
=============================================
- Hits          78202    15473    -62729
- Misses        30441    94978    +64537
+ Partials       5560     1267     -4293

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
13.85% <0.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...eator/name/NormalizedDateSegmentNameGenerator.java
0.00% <0.00%> (-83.34%)
:arrow_down:

.../org/apache/pinot/spi/data/DateTimeFormatSpec.java
0.00% <0.00%> (-82.93%)
:arrow_down:

... and 1629 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thank you for quickly addressing this!
Can you add the testing case where the format works in joda while it's not working in java to cover this case?

Will do. Also I checked out the other references and it doesn't seem like they will be impacted by this case since they are formatting a constant string.


Added some CLI scripts

I added some scripts to facilitate the usage of Mutate++ from a CLI.
For projects with lots of files, it can be quite annoying to manually add them using the web interface and then generate the patches for each one.
I also added scripts for adding projects to the DB and to control the queue, in case someone would want to use it in an semi-automated environment (the output still needs to be observed in the web interface).
Originally I wanted to put the scripts in the app directory, but I ran into some issues with importing from sibling/parent directories, python kept yelling at me that the root directory is not a package. In the end, I found the workaround using sys.path.append(".") which allows the scripts to import the app package but only when run from the parent directory.

Thanks!


Autoplay property not reflecting changes in property

When I change the autoplay property from outside the component
<Carousel
     autoplay={!this.state.modalVisible}
....

then this does not change the autoplay state in the component.
I can do a PR but the change is pretty straight forward:
This is because the componentWillUpdateProps call needs to consider the autoplay and delay properties and start and stop the timer appropriately
  componentWillReceiveProps(nextProps) {
    let childrenLength = 0;
    if (nextProps.children) {
      childrenLength = nextProps.children.length ? nextProps.children.length : 1;
    }
    this.setState({ childrenLength });

    // START update autoplay or delay if changed

    if (nextProps.autoplay !== this.props.autoplay || nextProps.delay !== this.props.delay) {
        if (childrenLength && nextProps.autoplay && nextProps.delay > 0) {
            this.timer = setTimeout(this._animateNextPage, nextProps.delay);
        } else {
            clearTimeout(this.timer);
        }
    }

    // END
  }

Hello @Paul-Todd PRs are welcome!


Feature Request: Action title Conventional Commit prefix

Is your feature request related to a problem?
Recently switched to using helm auto discovery for repos.
One thing I've lost in doing this is a Conventional Commit title for the GitHub PRs, which means when I use Squash merges with title as commit message, I loose build(deps): Bump dependency...
Solution you'd like
I'm not sure whether a titleprefix or a bool addprefix makes more sense.
I think it could be more complicated to gate it by bool , but probably more usable.
actions:
  default:
    kind: github/pullrequest
    titleprefix: 'build(deps): '
    # addprefix: true
    spec:
      mergemethod: squash
      labels:
        - dependencies
    scmid: default

Alternatives you've considered
I was thinking about whether target format made sense for auto discovery configuration, which would then be used for the PR title.
autodiscovery:
  scmid: default
  actionid: default
  targetformat: build(deps): bump depedency {target} to {version}
  crawlers:
    helm: {}

Anything else?
No response

I agree with you, it bothers me on the Golang plugin as well.
I was considering fixing the target name such as https://github.com/updatecli/updatecli/blob/c6150b3663dcd144da43f09ea351623f9a90b73b/pkg/plugins/autodiscovery/helm/container.go#L203
And improve the default action title https://github.com/updatecli/updatecli/blob/c6150b3663dcd144da43f09ea351623f9a90b73b/pkg/core/engine/main.go#L514C21-L514C21
using an information from the generated manifest such as the target name using the key "default" or reusing the manifest title


Add support for FPM

The Fortran Package Manager is a package manager and build system for Fortran developed by us. Modern Fortran should natively support it, allow for easy project setup, etc.
TODO List (to be expanded)
This issue is still in its planning stages, so the TODO list might be expanded

[ ] Typescript API design with fpm
[ ] Spawn fpm in vscode tasks
[ ] Ways for users to interact with FPM e.g. .toml, launch.json, settings.json?
[ ] autodetection of fpm.toml

CC @everythingfunctional

I'm really interested in this feature, while setting up tasks to interact with fpm ain't that hard, a full integration with vscode will surely be a good thing to ease the process. Some kind of integration with debugging would be nice, too. Right now I only know of three ways of debugging:

Having a separate Makefile (with it's specific task) to compile the executable to a specific directory and keep a fixed launch.json file.
Building with fpm and moving the executable to a specific dir (to keep the launch.json file intact).
Build with fpm and edit the launch.json file to point to the new executable.

Maybe there is a better way that I'm not aware of...
I think that good enough interaction with fpm to both run, debug and test would be awesome.

Hi @fedebenelli thanks for the interest in this issue, I haven't had a lot of time to think about how we will best implement this. My original thoughts were to offer some high level integration for building targets (similar to CMake), you wouldn't need to explicitly set build tasks in launch.json.
For debugging I would suspect an out of source debug installation would be what you are looking for, so your debug targets would not affect your optimised targets. My workflow in general is me using debug mode and if I need to run with optimisations enabled I will reconfigure and recompile.
If you have any ideas or suggestions feel free to post them on this issue. Later this summer I will be opening a thread on Fortran Discourse to get some ideas about how the UI should look like.

You may want to keep an eye on the discussion at https://github.com/fortran-lang/fpm/discussions/479
I've shown a basic example of fpm in Sublime (https://github.com/fortran-lang/fpm/discussions/479#discussioncomment-1778860) and @plevold has demonstrated it for VS Code (https://github.com/fortran-lang/fpm/discussions/479#discussioncomment-1238142). It works best if you have your code open as a "Project". In this case, you can easily run fpm in the top folder containing the manifest.
I think it's good to adopt a co-design strategy, meaning if there is something VS Code needs, we can see to add it in fpm.

Awesome to see this here. I have stumbled on the issue related to fpm + this extension: https://github.com/fortran-lang/fpm/issues/732.
It would be great if the Modern Fortran extension picked up the build/ directory as the mod files directory when fpm.toml file is found in the project's root.
A question, in the meantime, is there a way to configure this vsc extension to always store mod files in build/, and pick them up from the same directory?

You can set your global vscode setting call modOutputDir (I think) to be something like ${workspaceFolder}/build that should both store your .mod files in build/ and include the path to the linter (gfortran should do that by default)

You can set your global vscode setting call modOutputDir (I think) to be something like ${workspaceFolder}/build that should both store your .mod files in build/ and include the path to the linter (gfortran should do that by default)

Mmmm, this is not going to be so easy. Apparently, I need to hard-code that gfortran_... directory, too.

Fpm can run additional commands to the shell by passing them after the fpm specific commands e.g. fpm build -- find build -name '*.mod' -exec cp {} ./build/include ;
You could define in a build task or a series of them that will accomplish what you want. I think I posted somewhere in the fpm discussions a series of tasks.json and launch.json files that get vscode to build and debug fpm built codes.
What you are effectively trying to do is what this issue is about. Working out that hash is part of the problem. There was a discussion in fpm about making the paths somewhat deterministic so tools would be able to access builds.
Basically this issue is still on brainstorming stage about how to implement the vscode integration.

Again, the additional command I posted will not work, since you might be copying .mod files from outdated or unrelated builds. The easiest way we found was to install the package and it's mod files into some temporary location for vscode to access.

Thank you, I will look into those when time allows. I’m learning Fortran for fun, pretty cool so far.

I am glad to hear that, if you want any help don't hesitate to reach out here or in our Fortran Discourse https://fortran-lang.discourse.group/


Bug: Clear Input field after adding the To-do

When the Todo was added, the previous text was still present in Text field. So this will solve this issue and whenever the To-do is added successfully the text field of Todo form will be cleared.

Can I work on this issue?

@anand-harsh sure


LDM Outage in Development Environment

The Unidata Local Data Manager (LDM) service running in Development environment was stopped or hung last week. This was determined when a user reported the https://github.com/USACE/cumulus product "MBRFC KRF QPE precip 1hr" was missing recent data files.

LDM on EC2 working this afternoon.  Plan to let it run and check again Tue morning (13DEC).
@brettpalmberg , @adamscarberry

LDM on EC2 refactored to better handle discovered issues.

updated permissions allowing uploads to S3 and product notifications to Cumulus
Dockerfile and docker-compose.yml updated

base image unidata/ldm-docker instead of building from source
removed many lines of configuration
overall cleaner setup
seems to have solved quirky user:group and permissions

new logs shell script with list file controlling log rotation
updates to Python scripts, appkey and grid_upload_castle

logging in scripts using logging.handlers.WatchedFileHandler to sync with log rotation

new changes merged into ldm-config develop


[Bug]: Explorer pane shows a datasource connection error even when error is rectified

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I have a datasource connectivity error, and I fix the error, I don't expect to see the error on the explorer pane after it has been rectified.

Steps To Reproduce

Go to the datasources page and enter an invalid datasource configuration.
Hit "Test".
The connection test should fail and the connection error should be shown on the explorer pane on the left. You should see something like :

Now fix the connection error.
Although the connection error is fixed, you will still see the error on the explorer pane.

Public Sample App
No response
Version
Cloud

The issue looks good on the Dp and hence we can merge it to release


Add a helper script for localization

The script generates a report of outdated contents in content/<l10n-lang> directory by comparing two l10n team milestone branches.

At the beginning of every team milestone, it’s helpful to open an issue comparing upstream changes between the previous development branch and the current development branch.
-- from Localizing Kubernetes Documentation

/assign @chenopis @tengqm

This script is used for #16242, #16068 and #15745. Please take a look.

/assign @zacharysarah @jimangel

@gochist 👋 Thanks for adding this script--it's a handy tool for @kubernetes/sig-docs-l10n-admins. ✨
Please update /scripts/README.md with a description, otherwise LGTM.

@zacharysarah Thank you! I have addressed your comment.

/lgtm
/approve


Masking Capability for Input Component

Leading and trailing texts (like currencies or units) would be possible to put our Input component. Details will come later.

Design changes are ready: https://www.figma.com/file/RrcLH0mWpIUy4vwuTlDeKN/Baklava-Design-Guide?node-id=4%3A5586&t=dpos9bPGKoCkqo5m-0

This task had originally title Leading/Trailing text for Input field.
We talked about this and decided to implement this feature with usage of input's placeholder functionality and usage of input masking.

Our latest decision is not to implement input masking as a feature but provide detailed examples with compatible libraries. Leading/Trailing text for Input field should be achieved by placeholder usage and input masking integration with 3rd party libraries such as IMask and Maskito.


Adding build time in build stats

Webpack outputs build time when building, e.g. Time: 4716ms in following output :
Hash: 374e0bb6b1cd7f3b7bb7
Version: webpack 1.14.0
Time: 4716ms
    Asset  Size  Chunks             Chunk Names
bundle.js  5 kB       0  [emitted]  main
    + 5 hidden modules

Build time is a very useful data and I couldn't find a way to add it in grunt-webpack stats.
Is it possible to add build time in grunt-webpack stats ?

Oh actually just saw a timings item in the stats part of the options. Can't test it right now but it probably works. Feel free to close my issue then :)
But maybe adding it to the example configuration file would be a good idea ?


Update doc references

This removes the ?checked reference in the docs in favor of ?disabled, since this is a cleaner representation of a boolean attribute since there are no user interactions that can set the property.

I would love to have additional notes about why a dev should use .checked rather than ?checked but I'm unsure where that could go.
Would love to have a discussion about this :)

Thanks @jordanaustin!

I would love to have additional notes about why a dev should use .checked rather than ?checked but I'm unsure where that could go.
Would love to have a discussion about this :)

We can have a discussion about this in an issue with @arthurevans


[FEATURE]: ADD LORA+ ALGORITHM

What does this PR do?
Feature: add lora+ algorithm (see:https://arxiv.org/pdf/2402.12354.pdf)
we got result on spider experiment:
{
"rouge-1": 79.56683916827852,
"rouge-2": 63.493279206963244,
"rouge-l": 73.69117195357835,
"bleu-4": 72.11999584139265,
}
Before submitting

[Y] Did you read the contributor guideline?

LGTM! Thanks for adding this feature.


Make ActivityExtensions public

Describe the feature
Hi,
could you please make ActivityExtensions public? We'd like to use GetTraceId() in our code.
We could copy it to our code, but as the package is not that big overall then if we have to copy a part of it, there is not much point in using a package at all and we could just copy everything.
Regards
Mathais

Would accept a small PR.

Great, thanks!


Update onevnet

Description

Branches to which this PR applies

[ ] master
[ ] one-X.X

[ ] Check this if this PR should not be squashed

THANKS, merged as per 532ca2f88f8056a84aa6717b3f29339077ae556a


CheckExistenceByID and CheckExistence always returning false with Get or GetByID confirming the resources exists

Bug Report

Import path of package: github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resources/armresources
SDK version: v63.4.0
go version: go1.18.1 windows/amd64

What happened? I created a code shown below.

// Set the package
package main

import (
	"context"
	"log"
	"testing"

	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resources/armresources"
	"github.com/stretchr/testify/assert"
)

// Set the required variables
var subscriptionID = "82d66c37-8d61-4db4-b138-xxxxxxxxxxxx"
var apiVersion = "2022-02-01-preview"
var resourceID = "/subscriptions/82d66c37-8d61-4db4-b138-xxxxxxxxxxxx/resourceGroups/DEXT-EUN-RGRP01/providers/Microsoft.KeyVault/vaults/dexteunautomationkv01"

// Set the function to test resource existsence
func TestAzureKeyVaultExistence(t *testing.T) {
	// Create a context
	ctx := context.Background()

	// Create a new credential
	cred, err := azidentity.NewDefaultAzureCredential(nil)
	if err != nil {
		log.Fatal(err)
	}

	// Create a new resource client
	resourceClient, _ := armresources.NewClient(subscriptionID, cred, nil)

	// Check if the resource exists
	exists, err := resourceClient.CheckExistenceByID(ctx, resourceID, apiVersion, nil)
	if err != nil {
		log.Fatal(err)
	}
	assert.True(t, exists.Success)
}

When I executed the command go test -v, I got the error shown below.

=== RUN   TestAzureKeyVaultExistence
    integration_test.go:31:
                Error Trace:    integration_test.go:31
                Error:          Should be true
                Test:           TestAzureKeyVaultExistence
--- FAIL: TestAzureKeyVaultExistence (2.83s)
FAIL
exit status 1
FAIL    test.com        4.764s

What did you expect or want to happen? I should have a successful output as shown in the snip below.

=== RUN   TestAzureKeyVaultExistence
--- PASS: TestAzureKeyVaultExistence (1.98s)
PASS
ok      test.com        3.437s

How can we reproduce it?

Create a similar go code as above.
Ensure an Azure resource exists.
Execute the code to check the resource from the go code.

Anything we should know about your environment. The Get-AzKeyVault command shows the resource exists as shown below

Vault Name          : dexteunautomationkv01
Resource Group Name : DEXT-EUN-RGRP01
Location            : northeurope
Resource ID         : /subscriptions/82d66c37-8d61-4db4-b138-xxxxxxxxxxxx/resourceGroups/DEXT-EUN-RGRP01/providers/Microsoft.KeyVault/vaults/dexteunautomationkv01
Tags                :

@kabal2010 Apologies for the late reply. I see that this PR has been filed Azure/autorest.go#819 to address this. As per this comment the Fix is in upcoming preview.41 release. If you have any follow-up questions please feel free to reopen this thread. We would be happy to address it.

@navba-MSFT The PR is to fix the wrong handling with 405 response from service. Actually, I think the root cause is this method has not implemented by service. Could you help to triage this issue to service team to have a look.

@tadelesh Thanks for clarifying this. I am adding Service Team to have a look into this further.

@navba-MSFT
Can I please ask if there has been any update regarding this issue?
Best Regards

@josephkwchan @jennyhunter-msft Could you please review this ask and help us loop in the right owners for this ? Thanks in advance.

@kabal2010 This is pending with Service team now.
@josephkwchan @jennyhunter-msft Could you please provide an update on this ?

@navba-MSFT
Thank you

@navba-MSFT
I think the issue here has to do with using the HEAD method for querying for the resource as seen in https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/resourcemanager/resources/armresources/zz_generated_client.go#L139. I've tested again with another resource and got the error in the snip below.
=== RUN   TestResourceExistence
2022/06/28 21:02:19 HEAD https://management.azure.com/subscriptions/82d66c37-8d61-4db4-b138-0c2789b05709/resourceGroups/DEXT-EUN-RGRP01/providers/Microsoft.Network/privateDnsZones/konga.lab/virtualNetworkLinks/dext-eun-pvnet-03
--------------------------------------------------------------------------------
RESPONSE 405: 405 Method Not Allowed
ERROR CODE UNAVAILABLE
--------------------------------------------------------------------------------
Response contained no body
--------------------------------------------------------------------------------
FAIL    test.com        2.439s
FAIL

The test code as as below.
// Set the package
package inegration_test

import (
	"context"
	"log"
	"strings"
	"testing"

	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resources/armresources"
)

// Set function to test for resources
func TestResourceExistence(t *testing.T) {
	// Set the required variables
	resourceId := "/subscriptions/82d66c37-8d61-4db4-b138-0c2789b05709/resourceGroups/DEXT-EUN-RGRP01/providers/Microsoft.Network/privateDnsZones/konga.lab/virtualNetworkLinks/dext-eun-pvnet-03"
	apiVersion := "2021-04-01"

	// Create a new context
	ctx := context.Background()

	// Create a new credential
	cred, err := azidentity.NewAzureCLICredential(nil)
	if err != nil {
		log.Fatal(err)
	}

	// Split the resource ID
	resource := strings.Split(resourceId, "/")

	// Create a new client
	client, err := armresources.NewClient(resource[2], cred, nil)
	if err != nil {
		log.Fatal(err)
	}

	// Check existence by ID
	checkExistenceByID, err := client.CheckExistenceByID(ctx, resourceId, apiVersion, nil)
	if err != nil {
		log.Fatal(err)
	}
	log.Println(checkExistenceByID.Success)
}

Changing the method to http.MethodGet should resolve he issue. I've tested this using the Azure Rest API and can only get the resource when the method is GET and not HEAD.


jsx-curly-brace-presence fails with typeError

In the newest version (7.15.0) we get unexpected TypeError. Does not happen with previous version. Our .eslintrc.json config.
Either disabling react/jsx-curly-brace-presence rule on pinning down 7.14.3 version is quick workaround if anyone else sees this.
$ cat foo.jsx
import React from "react";
import { OverlayTrigger, Tooltip } from "patternfly-react";

export class Foo extends React.Component {
    render() {
        return (
            <OverlayTrigger overlay={ <Tooltip id="tip-service">Foo</Tooltip> } />
        );
    }
}

$ eslint foo.jsx
TypeError: Cannot read property 'substring' of undefined
Occurred while linting /home/mmarusak/cockpit/cockpit1/foo.jsx:7
    at Object.fix (/home/mmarusak/cockpit/cockpit1/node_modules/eslint-plugin-react/lib/rules/jsx-curly-brace-presence.js:129:30)
    at normalizeFixes (/usr/lib/node_modules/eslint/lib/linter/report-translator.js:176:28)
    at args (/usr/lib/node_modules/eslint/lib/linter/report-translator.js:278:49)
    at Object.report (/usr/lib/node_modules/eslint/lib/linter/linter.js:904:41)
    at reportUnnecessaryCurly (/home/mmarusak/cockpit/cockpit1/node_modules/eslint-plugin-react/lib/rules/jsx-curly-brace-presence.js:117:15)
    at lintUnnecessaryCurly (/home/mmarusak/cockpit/cockpit1/node_modules/eslint-plugin-react/lib/rules/jsx-curly-brace-presence.js:198:9)
    at JSXExpressionContainer (/home/mmarusak/cockpit/cockpit1/node_modules/eslint-plugin-react/lib/rules/jsx-curly-brace-presence.js:278:11)
    at listeners.(anonymous function).forEach.listener (/usr/lib/node_modules/eslint/lib/linter/safe-emitter.js:45:58)
    at Array.forEach (<anonymous>)
    at Object.emit (/usr/lib/node_modules/eslint/lib/linter/safe-emitter.js:45:38)

Got same error when upgrade from 7.14.3 to 7.15.0 at https://github.com/Hongbo-Miao/hongbomiao.com/pull/240
Travis log is at https://travis-ci.com/Hongbo-Miao/hongbomiao.com/jobs/240621518
Maybe the info will help find the issue.

This was introduced in #2409 by @vedadeepta. Thanks for the great test case.

@marusak  not able to reproduce this issue - can you provide a sandbox link ?

@vedadeepta see the test case i added in https://github.com/yannickcr/eslint-plugin-react/commit/feba50716273b24753e83440393eccffa80bbf59, but without my workaround/fix.

we'll want to follow up with a proper fix that actually reports on jsx elements inside props.

I think curly braces for jsx elements inside prop should always be required.
<App elm={<CustomElement>.....</CustomElement>} elm1={<Custom />} />

so maybe the test should detect and skip curly braces check if the element inside it is a jsx-element.
Let me know if you think otherwise or if any special cases.


Add option to include "My Flight" jobs in route finder calculations

Sometimes you've already reserved a few jobs, but still want to compute routes. I can see where you wouldn't want this to be the default, so it'd be nice if we had the option to include those jobs too.

Not sure how this would work. Should the route starts with these jobs (in which order?)? Should the route contain these jobs, but can start anywhere and end anywhere?

I was thinking just have the jobs be part of the available jobs again. Just
so you can reserve them but midway through you decide, hey are there better
routes I could take at this point.
On course it’s up to the user to update their jobs, and airplanes first.
On Tue, Apr 6, 2021 at 5:16 PM piero-la-lune @.***>
wrote:

Not sure how this would work. Should the route starts with these jobs (in
which order?)? Should the route contain these jobs, but can start anywhere
and end anywhere?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/piero-la-lune/FSE-Planner/issues/44#issuecomment-814445399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2BPIV34UA5FIKL6BX3CSDTHN24LANCNFSM4ZNXZC7A
.

Added to v1.5.0 now live


Univalue: Do not build JSON escape list at runtime

No need to waste startup time building something that can be done
at compile time.
This also resolves a clang++ warning originally reported in #4714,
univalue/univalue_write.cpp:33:12: warning: array subscript is of type 'char
escapes['"'] = "\"";
^~~~
etc.

utACK


Corrected num of cuts removed from time per person

@hiroto-udagawa
Made minor adjustment so further scripts can run: the number of cuts removed from the beginning wasn't standardized.

will this fix the issue that we were seeing in the glm?

thanks for this fix. i'm going to merge because this is crucial for us to keep going forward


monitor children and write exit paths

This set of patches encompasses child handling in conmon-rs. specifically it:

adds state to keep track of the grandchildren container processes
sets self as subreaper to catch grandchildren exiting
catches the sigchld from the children/grandchildren and writes their exit codes down
kills the children its tracking upon exiting

you know, like a normal parent does :slightly_smiling_face:

Codecov Report

Merging #80 (2b75a43) into main (80b5595) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #80   +/-   ##
=====================================
  Coverage   6.89%   6.89%
=====================================
  Files          2       2
  Lines         29      29
  Branches      17      17
=====================================
  Hits           2       2
  Misses        27      27

test failures were similar to https://github.com/kata-containers/kata-containers/issues/1826 -- thanks kata folks for an inspiration for the fix


housekeeping: update version dependencies of xamarin samples

I am updating as per noted by the user in https://github.com/reactiveui/website/issues/320

Give me a few mins to check this please, I have tried this before and wasn't compiling on Mac.
It may be something in my environment (most likely) or something related to the toolchain.

Go on, merge it. It's all working :)


Figure out missing source location for vtable initializers.

https://github.com/vecchiot-aws/rmc/tree/vtable-initialize-location

Currently blocked; the source locations correctly appear in the XML output, but do not appear in the HTML output.

Closing as there isn't enough information to see where we still might be missing locations (#3253 does add a locations to some vtable-related expressions).


Fix vkb::screenshot()

Description
Addresses the incorrect use of VK_IMAGE_USAGE_SAMPLED_BIT as described in issue #259.
Changes vkb::screenshot() to use VkCopyImageToBuffer  rather than VkCopyImage.
This simplifies the code, and I believe is a better practice that trying to read image data directly.
The previous code also didn't work for me. It either produced empty images, or images that started with a horizontal offset for me.
Tested only on Linux devices.
Fixes #259
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

LGTM and works as expected on an NVIDIA RTX 2060.
One thing I noticed, which is not related to this PR at all, is that the screenshot feature only seems to work in samples using the ARM framework, but not in the samples using the API sample base class. That may be worth a separate issue.


Find out if quil is going to provide binary wheels

Description of the issue
pip install cirq attempts to compile quil-0.13.0 source-only release,
which fails if host machine does not have rust installed, e.g.,
https://github.com/quantumlib/Cirq/actions/runs/11226159459/job/31206161707#step:7:1329
and the internal docs-from-notebooks building tool.
Related: https://github.com/rigetti/quil-rs/issues/291
Cirq version
1.5.0.dev at 769ad5015087af511be1b11dcac89f585b6d28f2

cc @mhucka

As a temporary workaround, the development release of cirq has a pin to the previous version of quil and can be installed using
pip install cirq~=1.0.dev

Find out if quil is going to provide binary wheels

Yes, the updated release quil-0.13.1 has them - https://pypi.org/project/quil/0.13.1/#files.


Added GetPageIDByTitle(pagename, spacename)

Useful if you don't want to have to go to confluence and find the page, click on page information, pull out the pageid.. etc

Help method to solve the use case of I don't know what my page id is.  Not that it's hard to get but why bother.


New plugin system doesn't work on macos (Darwin)

from the example:
go build -buildmode=plugin -o hello.so test.go
-buildmode=plugin not supported on darwin/amd64

Hi, @srctxt
Plugin is introduced in Go1.8, https://golang.org/doc/go1.8#plugin

Go now provides early support for plugins with a “plugin” build mode for generating plugins written in Go, and a new plugin package for loading such plugins at run time. Plugin support is currently only available on Linux.

Maybe the situation will be improved in the future, the Go team is great, I think they will do it one day.

I will be adding OS X to the CI tests. Even though the feature may be unavailable, at least it should be possible to compile and use the rest of conscheme.
And will keep the bug open to show that the feature is currently missing on OS X.


keep the grid overlay on as you navigate from page to page and/or ref…

#61
Use cases

User enables the grid (and/or horizontal lines) overlay. User refreshes the page. After the page refresh the grid (and/or horizontal lines) overlay is rendering automatically;
User enables the grid (and/or horizontal lines) overlay. User opens a new URL (by link or directly input) in current tab. After the page refresh the grid (and/or horizontal lines) overlay is rendering automatically.

It don't work

When user follows the targer="_blank" link (which opens in a new tab);
When user use extension in local file (the file:// protocol). It's necessary to use web server. (Chrome web server will work fine).

Issues

Add keypress listener in content_script and let user use keyboard shortcuts without opening the popup (enhancement);
After a page refresh, when the grid overlay is rendered with non-default options, if user opens the popup, options set to default and the grid re-renders with default options.

Also fixed double grid overlaying


Game Instructions Modal

Game Instructions Modal
When the game first opens it would be good to have an html modal popup with some instructions on how to play the game, like the following.
How to play the game

Complete each level by avoiding the enemy characters
You can kill the enemy by jumping on them
Killing them also adds a bonus to your score

Controls

On screen buttons
Keyboard arrows

If you need help creating a modal this is a good reference
https://www.w3schools.com/bootstrap/bootstrap_modal.asp

I will work on this one :)

@JorgeAndd Great, let me know if you need any help.

@EdwardDunn I created the modal and some logic to show it only at the first start of the game.
I don't have images for the title or the button, should I push it as it is and maybe open another issue for new images?

@JorgeAndd Looks good! Yeah go ahead and create a new pull request and i will create new issues for the images.

@JorgeAndd Completed


Speed up TargetEncoder with vectorization.

Roughly 300X faster and uses less memory.
fit() computes category smoothed means with vector ops, stores mapping as (dict of) Series.
transform() then just calls Series.map(mapping) to do vectorized lookup with the mapping Series.
Also fixes IndexError when passing DataFrames with non-contiguous indexes.
Also fixes not being able to re-use TargetEncoder by always setting mapping=None in fit() (you don't want fit to remember anything from previous calls).
rows, cats = 1000000, 1000
X = pd.DataFrame({'x': np.random.randint(0, cats, rows).astype(str)})
y = pd.Series(np.random.rand(rows))

# old
%time TargetEncoder().fit_transform(X, y)
CPU times: user 1min 24s, sys: 71.2 ms, total: 1min 24s
Wall time: 1min 24s

# new
%time TargetEncoder().fit_transform(X, y)
CPU times: user 235 ms, sys: 43.6 ms, total: 279 ms
Wall time: 276 ms

Hi @jkleint, couldn't you look at LeaveOneOutEncoder and possibly vectorize it as well? I failed in the attempt.

Sure, I'm happy to take a look at it.


Analysis of Algorithms : Space Complexity

Description
Analysis of space complexity of algorithms.
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@lvi131
Pls, refrain from getting the issue assigned yourself.
You can go ahead with this issue. But change the name according to the guidelines.

@goelesha
Sorry for self assigning.
Can you please help me with the name, I am not sure about the numbering in it -
DSA 1.2.??? Algorithms: Space Complexity

Can you pls elaborate what exactly you will be adding in space complexity ?

@goelesha
Sorry for self assigning.
Can you please help me with the name, I am not sure about the numbering in it -
DSA 1.2.??? Algorithms: Space Complexity

Can you pls elaborate what exactly you will be adding in space complexity ?

@goelesha
Definition, 1 or 2 simple examples how to calculate, one example of recursive algo, and auxiliary space vs space complexity

Okay. You can go ahead with the issue. I will rename it myself

Okay Thanks.


Thumbnails not appearing on seek bar

There is an issue while hovering over the seek bar to view video thumbnails.
When you start seeking from a point where the video does not have a captured thumbnail, then for the duration that the user is still hovering over the seek bar from that position, he won’t be able to view the thumbnail even if there is one at a later point in time.
For example, there is no thumbnail at 00.01 and the first thumbnail capture is at 00:04, then if the user starts seeking at 00:01 and hovers over the seek bar to say around 00:07, he still won’t be able to view any thumbnail, until the point that he moves his/her cursor away from the seek bar and hovers over it again.

Thumbnails are not appearing at all (Video.js 7.6.)
Any chance to provide url with working example?

For me the thumbnail holder had no size, so I had to manually set it:
    <style type="text/css">
       .vjs-vtt-thumbnail-display {
            width: 160px;
            height: 90px;
        }
    </style>

For me, the thumbnails were not appearing in the right place, so I had to use margin-top: -89px to get them in the proper place..


Deprecated modules (dependencies?) when install powerbi-visuals-tools globally

Hey team,
I'm trying to build a custom visual with a CI/CD approach in Jenkins.
Once I run my pipeline (Jenkins runs locally on my macOS), I got the following warning during installing the powerbi-visuals-tools:

Console output:
...

npm install -g powerbi-visuals-tools
npm warn deprecated<EMAIL_ADDRESS>This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/config-array instead
npm warn deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/object-schema instead
npm warn deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported

changed 554 packages in 4s
91 packages are looking for funding
...

Can you please provide feedback on that? Are inflight, humanwhocodes, rimraf and glob are indirect dependencies in the powerbi-visuals-tools npm package? I can't find those here: https://www.npmjs.com/package/powerbi-visuals-tools?activeTab=dependencies
When I run it under Windows these kind of npm warn deprecated part doesn't appear.
Thanks a lot!
Best Regards,
Gellert

Hi @ggintli,
Could you, please, take a look if you install any other dependencies globally (ex. eslint)?