fix: labeler refactor

Do not merge! I'll do it.

Codecov Report
Patch and project coverage have no change.

Comparison is base (1c7c7df) 18.04% compared to head (96c6419) 18.04%.

:exclamation: Current head 96c6419 differs from pull request most recent head fc59b2e. Consider uploading reports for the commit fc59b2e to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #575   +/-   ##
========================================
  Coverage    18.04%   18.04%
========================================
  Files           54       54
  Lines         5082     5082
========================================
  Hits           917      917
  Misses        4138     4138
  Partials        27       27

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


docs: Fix a typo the minimum rust version required

This change is

Coverage remained the same at 91.872% when pulling 71dabba3ea0a17c88b0e2199c9d99f0acbf3bc17 on mdaffin:doc-typo-fix into 6095f239badec209ed3b3eed6103383b3c203148 on kbknapp:master.

Thanks! @homu r+

:pushpin: Commit 71dabba has been approved by kbknapp

:zap: Test exempted - status


append backslash to namespace in twig_component.yaml

This is the way the flex recipe creates this file and it throws an error without the final backslash.

Thanks Peter!


Create an abstract class that all datamodules inherit from

The datamodules contain a lot of redundant code. To make this more flexible and less duplicate, an abstract class (based on Lightning datamodule) should be implemented that all datamodules inherit from.
Also Tests should use this class to test all basic functionality and only the individual additional functionality should be in the individual test classes

The base class was introduced in this commit and all related changes fixed in the commits of this pull request


Server Crash [1.9.4] : After Long Idle Time

Issue Description:
If my server has been idle too long (possibly long enough to unload chunks from memory) the next login will cause the server to hang and eventually crash.  Restarting the server 'fixes' the problem.
What happens:
Server crash
What you expected to happen:
No crash
Steps to reproduce:

Logout from a server and wait awhile for chunks to unload (seems to be a couple hours on my setup)
Login
Get in world but unable to interact with anything for 2-3 minutes, eventually get disconnected.  Server has crashed.
...

Affected Versions (Do not use "latest"):

EnderIO:
EnderCore:
Minecraft: 1.9.4
Forge:

Your most recent log file where the issue was present:
http://pastebin.com/Xq1d5cha

I believe this may be the same error. We leave the server idling over night and when someone tries to log in the next day this occurs.
crash-2016-07-10_13.03.35-server.txt

This should be fixed with the changes in build 58


Moves angular-mocks as a development dependency

Coverage remained the same at 87.273% when pulling 8a838298ea0f92a9000a7226339d87dc383a4e06 on Enegia:devDepsFix into 87e0adb3b6a1025e55e3ebea1e5423feb1010ef6 on angular-cron-jobs:master.


Use separate export folder and config for Open Brush

First Commit
Store exports in Documents/OpenBrush not TiltBrush, and use separate Open Brush config file.
Note to devs:
I hope you did not clone the repo as C:\Users{USER}\Documents\Open Brush
or your might get Sketches in your repo.
Second Commit
Changed the version number to 0.1.0 as requested by @mikeskydev

There's a few changes to ProjectSettings that I have not added to the commit, I believe its due to me switching back from Oculus to PC build. Not sure if changing the version number in Config would have changed anything but the Main scene, so this needs checking.

There's a few changes to ProjectSettings that I have not added to the commit, I believe its due to me switching back from Oculus to PC build. Not sure if changing the version number in Config would have changed anything but the Main scene, so this needs checking.


az logic workflow update --set unrecognized arguments

Describe the bug
I'm currently using Logic App Consumption and I'm trying to update one specific value in a definition of that logic app using Azure CLI
below is the json shows when I executed az logic workflow show

{
"accessControl": null,
"accessEndpoint": "",
"changedTime": "0",
"createdTime": "",
"definition": {
"$schema": "https://schema.management.azure.com/",
"actions": {
"Initialize_variable": {
"inputs": {
"variables": [
{
"name": "set",
"type": "string",
"value": ""
}
]
}
}
}
}

I'm trying to update the value under definition > actions > Initialize_variable.inputs.variables > input > variables > array index 0 > value
here's the Azure CLI I'm using - az-logic-workflow-update

When I tried using --set command this is the response I received
unrecognized arguments: --set definition.actions.Initialize_variable.inputs.variables[0].value=MyValue
This is the command I used.
az logic workflow update --resource-group $resourceGroupName --name $logicConsumptionName --set definition.actions.Initialize_variable.inputs.variables[0].value=MyValue
Could you guide me on how to use --set command please.
Related command
az logic workflow update
Errors
unrecognized arguments: --set
Issue script & Debug output
unrecognized arguments: --set definition.actions.Initialize_variable.inputs.variables[0].value=MyValue
Expected behavior
This should update the definition schema of my initialize variable
Environment Summary
azure-cli                         2.52.0
core                              2.52.0
telemetry                          1.1.0
Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @staycool13,
2.52.0 is not the latest Azure CLI(2.59.0).
If you haven't already attempted to do so, please upgrade to the latest Azure CLI version by following https://learn.microsoft.com/en-us/cli/azure/update-azure-cli.

still occurs in 2.59.0

updates anyone?

updates?

Hi @staycool13 , az logic workflow update belongs to the logic extension. Please check and update the extension by following https://learn.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview#how-to-update-extensions.
Additionally, I was unable to replicate the issue using the latest logic extension. It may be beneficial to verify that your extension is up to date.


fix: resolve TitleProvider warnings logged in the browser console

unable to update componenets , which are using updateTitle to set
the label of the page.
Fixes #2417.
Changes proposed in this pull request:

Add updateTitle(t('label')) into effect hook(useEffect).

hi @dev-script, please sign our Contributor License Agreement before we can accept your contribution :)

hi @dev-script, please sign our Contributor License Agreement before we can accept your contribution :)

Done 🙂


Clean up Pandoc compilation with text-processing tools

Since I am one picky motherfucker, I'll need to implement a script to clean up some nitpicks with Pandoc's compilation.

[x] Footnote return link in HTML pages (Fixed in af7a3a73564e04ceaed48082c99644fcb8d51c0c)
[ ] Unnecessary footnote line breaks in plaintext pages
[ ] Possibly fix #1 by removing some extraneous HTML in fenced code blocks

Updated post with checklist of items

#1 has been fixed and removed from this list


Failed to update from 2.13.0 to 3.1.3

I'm running io.js 2.4.0. The error message is similar to #10 but happens in upgrade-npm.ps1 rather than Microsoft.PowerShell.Commands.WriteDebugCommand if I'm reading it correctly:
PS C:\> npm-windows-upgrade
npm-windows-upgrade 0.2.6
? This tool will upgrade npm. Do you want to continue? (Y/n)
? This tool will upgrade npm. Do you want to continue? Yes
? Which version do you want to install? 3.1.3
Upgrading.. |Error:  C:\Users\Rosalie\AppData\Roaming\npm\node_modules\npm-windows-upgrade\powershell\upgrade-npm.ps1 :
Impossible de

Error:  trouver un param?tre positionnel acceptant l'argument ?env:ProgramFiles(x86)?.
Au caract?re Ligne:1 : 4
+ & {& 'C:\Users\Rosalie\AppData\Roaming\npm\node_modules\npm-windows-upgrade\powe ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument : (:) [upgrade-npm.ps1], ParameterBindingException
    + FullyQualifiedErrorId : PositionalParameterNotFound,upgrade-npm.ps1

You wanted to install npm 3.1.3, but the installed version is2.13.0.
Please consider reporting your trouble to http://aka.ms/npm-issues.
Here is the output from the upgrader script:
[] [ 'C:\\Users\\Rosalie\\AppData\\Roaming\\npm\\node_modules\\npm-windows-upgrade\\powershell\\upgrade-npm.ps1 : Imposs
ible de \r\n',
  'trouver un param?tre positionnel acceptant l\'argument ?env:ProgramFiles(x86)?.\r\nAu caract?re Ligne:1 : 4\r\n+ & {&
 \'C:\\Users\\Rosalie\\AppData\\Roaming\\npm\\node_modules\\npm-windows-upgrade\\powe ...\r\n+ ~~~~~~~~~~~~~~~~~~~~~~~~~
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\r\n    + CategoryInfo          : InvalidArgument : (:) [upgrade-
npm.ps1], ParameterBindingException\r\n    + FullyQualifiedErrorId : PositionalParameterNotFound,upgrade-npm.ps1\r\n \r\
n' ]

I'm running the script on a PC with Windows 8 32-bit (EDIT: previously erroneously stated it was Windows 7) so while I have the 32-bit version of io.js, I don't have a Program Files (x86) folder... which the error log mentions so it could be related?

Hey @elisee - the error here is that the upgrader is currently only for Node (I'm actually a fan of io.js, I swear). I fixed the issue by adding an error in case someone else runs into this.
Since io.js and Node will merge again, I probably won't upgrade the script to work with io.js. You can still upgrade npm manually though!

Ah ok, thanks!


Add OneElementFastSequence to be used in ConnectionPool

Codecov Report

Merging #420 (bf18fbe) into main (358fa59) will increase coverage by 0.45%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #420      +/-   ##
==========================================
+ Coverage   55.18%   55.64%   +0.45%
==========================================
  Files         110      111       +1
  Lines        7904     7985      +81
==========================================
+ Hits         4362     4443      +81
  Misses       3542     3542

Files
Coverage Δ

.../ConnectionPoolModule/OneElementFastSequence.swift
100.00% <100.00%> (ø)


[Request for External] page 510 "Blanket Purchase Order Subform" - DeltaUpdateTotals

We would like to set procedure DeltaUpdateTotals() to global like it has been done in the other purchase subpages so we can use it in page extensions:
procedure DeltaUpdateTotals()
begin
    //A108F0000.Event18.03 local=false
    DocumentTotals.PurchaseDeltaUpdateTotals(Rec, xRec, TotalPurchaseLine, VATAmount, InvoiceDiscountAmount, InvoiceDiscountPct);
    CheckSendLineInvoiceDiscountResetNotification();
end;

We have this instead
procedure CallDeltaUpdateTotals()
begin
    DeltaUpdateTotals();
end;


[V1] VLM - Support running the mm_mapper preprocessor in the frontend process

This PR adds support to run the multi-modal mapper/preprocessor (from huggingface) in the frontend process. Execution of 512 prompts with 64 output tokens results in 1.7X improvement. Command used:
VLLM_USE_V1=1 VLLM_ENABLE_V1_MULTIPROCESSING=1 python examples/offline_inference_vision_language.py -m llava --num-prompts 512 --modality image
Without frontend generate() time is: 28.91 seconds
With frontend generate() time is: 16.84

@robertgshaw2-neuralmagic I think you right about P0 always running the mm_mapper, so I will remove the disable to simplify the code.

@robertgshaw2-neuralmagic I think you right about P0 always running the mm_mapper, so I will remove the disable to simplify the code.

Does this mean we can then remove mm_data (and mm_processor_kwargs?) from EngineCoreRequest? :)

Yea if P0 is always going to run the multimodal data processor (mm_mapper), then P1 should only need to receive mm_inputs

@njhill @ywang96 removed mm_data and mm_processor_kwargs from EngineCoreRequest.

Will revert changes to offline_inference_vision_language.py and see if I can use the other script.


Automatically calculate Version

This PR adds update_version_h.sh script
The script will:

Automatically calculate WEEK_4 and WEEK_12 versions based on Version tag date and current date.
Bump CurrentVersion by increasing patch version by 1

This script should be executed at the beginning of the next development cycle, after performing the git tagging.
This script replaces versions in the following lines in stablehlo/dialect/Version.h
  static Version getCurrentVersion() { return Version(x, y, z); }
  static Version getWeek4Version() { return Version(a, b, c); }
  static Version getWeek12Version() { return Version(m, n, p); }

getCurrentVersion() will be (x, y, z + 1)
getWeek4Version() - The most recent git tag that was created at least 28 days ago.
getWeek12Version() - The most recent git tag that was created at least 84 days ago.

Usage Example:
$ build_tools/update_version_h.sh

Next Current Version: 1, 7, 9
WEEK_4 Version: 1, 7, 1
WEEK_12 Version: 1, 4, 0
Saving...
Done

$ git diff
diff --git a/stablehlo/dialect/Version.h b/stablehlo/dialect/Version.h
index 74e8df74..5828ced7 100644
--- a/stablehlo/dialect/Version.h
+++ b/stablehlo/dialect/Version.h
@@ -38,13 +38,13 @@ class Version {
   static FailureOr<Version> fromString(llvm::StringRef versionRef);

   /// Return a Version representing the current VHLO dialect version.
-  static Version getCurrentVersion() { return Version(1, 7, 8); }
+  static Version getCurrentVersion() { return Version(1, 7, 9); }

   /// Return CompatibilityRequirement::WEEK_4 VHLO dialect version.
-  static Version getWeek4Version() { return Version(1, 7, 0); }
+  static Version getWeek4Version() { return Version(1, 7, 1); }

   /// Return CompatibilityRequirement::WEEK_12 VHLO dialect version.
-  static Version getWeek12Version() { return Version(1, 3, 0); }
+  static Version getWeek12Version() { return Version(1, 4, 0); }

   /// Return a Version representing the minimum supported VHLO dialect version.
   static Version getMinimumVersion() { return Version(0, 9, 0); }

For convenience we can use the following script to get WEEK_4 and WEEK_12 tag / tag date / days ago
python3 get_compatibility_versions.py

WEEK_4: (tag: v1.7.1) 2024-09-06T15:27:44-07:00 34 days ago
WEEK_12: (tag: v1.4.0) 2024-07-15T14:21:55-07:00 87 days ago

import subprocess
import time

utc = int(time.time())

command = [
  "git",
  "log",
  "--tags",
  "--simplify-by-decoration",
  "--pretty=format:%ct,%d,%cI",
]
process = subprocess.Popen(
  command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True
)

WEEK_4_tag = ""
WEEK_12_tag = ""

for line in process.stdout:
  line = line.strip().split(",")
  tag_sec = int(line[0])
  diff_days = int((utc - tag_sec) / (60 * 60 * 24))
  if WEEK_4_tag == "" and diff_days >= 7 * 4:
    WEEK_4_tag = line[1].strip()
    print("WEEK_4:", WEEK_4_tag, line[2], diff_days, "days ago")
  if WEEK_12_tag == "" and diff_days >= 7 * 12:
    WEEK_12_tag = line[1].strip()
    print("WEEK_12:", WEEK_12_tag, line[2], diff_days, "days ago")
    break

Maybe add it to one of the documents?

Actually this is super useful! We can add this to our weekly integrate / version bump script. Tag the latest release, check if the values should be updated, bump the Version.h value.
Mind sending this PR as a .sh file in buid_tools? When we get a chance we'll integrate it into the process

Added build_tools/update_version_h.sh
See Usage Example in the PR description above

Updated the script to use the following git command to get tag_ts and tag_name
git for-each-ref --sort=taggerdate --format '%(taggerdate:unix),%(refname:short)' refs/tags

This command retrieves the actual tagger date, which represents the date the tag was created, rather than the date of the last commit in the tag.


Upload all geocoded datasets

Needs a little more review

I was going to open an issue suggesting geocoding (and volunteering to help out feeding everything through the Census geocoder) but it sounds like you might have this handled already.

Curious if there is the geocoded data available somewhere


MultiCurl improvement

As for now, after 1 curl returned response, only 1 another can be added even if there is free space in activeCurls (size of activeCurls less than concurrency). It can be usefult for example, if you added curls in queue in another curl's response callback

Thanks!
PR: #500


Website

Can anyone help me make a website for this github repo
For template you can visit the website in homepage and i want it to update automatically when someone adds a new file instead of doing it manually

@diwas7777 Hey! I want to work on this issue as I feel I can complete this task, so please assign this to me.

yes

@utkarshgoel10

@utkarshgoel10 assigned

@utkarshgoel10 can you please reply for confirmation

Yes @diwas777777 I'm working

@utkarshgoel10 is there any way to make update website automatically like in template website and new folders that is added to github and display it accordingly for users to choose to view

@diwas7777 Okay, let me check once.
I've also dm you.

@utkarshgoel10 okay

Is this issue resolved?


[Question] Is it possible to use custom binding in use.template in test step?

Describe your question
Hi there,
I'm trying to create reusable (StepTemplate) building blocks and have the following:
apiVersion: chainsaw.kyverno.io/v1alpha1
kind: Test
metadata:
  name: basic-test
spec:
  bindings:
  - name: rootPath
    value: ../..
  - name: templatesPath
    value: (join('/', [$rootPath, 'templates']))
  steps:
  - name:  Print path # Works fine
    try:
    - script:
        env:
        - name: TEMPLATES_PATH
          value: (join('/', [$templatesPath, 'example/custom_bindings.yaml']))
        content: |
          echo $TEMPLATES_PATH
  - name: Test custom bindings
    bindings:
    - name: filePath
      value: (join('/', [$templatesPath, 'example/custom_bindings.yaml']))
    use:
      # template: ../../templates/example/custom_bindings.yaml
      template: $filePath # does NOT work
      # template: (join('/', [$templatesPath, 'example/custom_bindings.yaml'])) # does NOT work

Version: 0.2.8
No configuration provided but found default file: .chainsaw.yaml
Loading config (.chainsaw.yaml)...
Loading tests...
Running tests...
=== RUN   chainsaw
=== PAUSE chainsaw
=== CONT  chainsaw
=== RUN   chainsaw/my-test
    | 18:46:04 | basic-test | Paths                            | TRY       | RUN   |
    | 18:46:04 | basic-test | Paths                            | SCRIPT    | RUN   |
        === COMMAND
        /usr/bin/sh -c echo $TEMPLATES_PATH
    | 18:46:04 | basic-test | Paths                            | SCRIPT    | LOG   |
        === STDOUT
        ../../templates/example/custom_bindings.yaml

Version: 0.2.8
No configuration provided but found default file: .chainsaw.yaml
Loading config (.chainsaw.yaml)...
Loading tests...
Error: open tests/basic-test/$filePath: no such file or directory

It seems it's not currently supported or did I miss anything?
Would it be possible to add support for this?
There are more use cases for this, the above is just a quick example.
chainsaw version Version
v0.2.8
Additional context
No response

@jingav it will be supported in file in the next version (v0.2.9).
There's no plan to support it in template at this point. Step templates are resolved at discovery time and no binding are evaluated at that time, only when the test executes. We could delay the step template loading until execution but again, there's no plan for that yet.

Yes you can change it to a feature request.

how about use can support from workdir

selector: crossplane.io/claim-name=($dnsClaimName)

selector: (join('=', ['crossplane.io/claim-name',$dnsClaimName]))

Sorry folks, i didn't follow the issue here, is there anything left to discuss ?


Config files are not properly updated after relation broken event.

Describe the bug
When the fiveg_gnb_identity  between the GNB and the NMS is removed, the GNB config file inside the container is not updated.
The relation broken event for this event is not observed for this relation and even if it was, there is a known juju issue in which the databag of a broken relation is not empty. ( https://github.com/canonical/operator/issues/888)
This https://github.com/canonical/sdcore-nms-operator/pull/24 was an attempt of a workaround to handle the issue in our charm. However, it is flawed. This situation can be properly handled once https://github.com/canonical/operator/issues/940 is addressed, which should allow us to easily identify the broken relations.
At the moment the charm is working at a best effort basis: once the relation is removed, the content inside the file will be updated over the next event that triggers the charm config method... I have seen the kubernetes patch event to be triggered from time to time, for example.
The same behavior happens with the fiveg_n4 relation between the NMS and the UPF.
To Reproduce
Deploy Charmed 5G including the NMS
Add the fiveg_gnb_identity relation between gnbsim and the NMS
Check that the NMS contains the /nms/config/gnb_config.json file or check on the GUI.
Remove the the fiveg_gnb_identity and relation
The /nms/config/gnb_config.json file still contains the gnb information or check on the GUI.
Expected behavior
Once the relation is removed, gnb_config.json should contain the up to day information.
Environment

Juju version (output from juju --version): 3.1.6
Kubernetes version (output from kubectl version --short): 1.27

This bug is still valid, but the configs get updated on juju status refresh event, so it eventually gets updated.


Use only power above the 70% cutoff (in Germany)

I just came home on this very sunny day with a car around 90% SoC and I was wondering if I should hook it up to the charger or not.
I avoid leaving the car at 100% SoC but for one night it should be fine. So I connected it to the charger because I was sure I'd be above the 70% level (You don't get paid for the power above 70% of your peak power into the grid in Germany). So before 'wasting' this I'd rather use it to charge my car.
In my opinion this could be a nice additional charging mode for evcc.
Usecase: You know your car is almost at 100% SoC or the whole week will be very sunny and you barely will use the car and the SoC is very low and you put it into this '70%-mode'. So at high noon the car will be charge with energy you'd give to the energy company for free.

Duplicate of https://github.com/andig/evcc/issues/150


[DEFECT] Title Here

Defect Description
Describe the defect
What did you expect to see happen?
If the trajectory N is removed after the trajectory N-1 is, a crash happens
What did you see instead?
a crash
( 9949.15 sec) GradientDescent          : DEBUG           -> Trajectory 1 has converged successfully 1 / 1 time(s)!
Traceback (most recent call last):
  File "/Users/alfoa/projects/raven_github/raven/framework/Driver.py", line 305, in <module>
    raven()
  File "/Users/alfoa/projects/raven_github/raven/framework/Driver.py", line 258, in raven
    simulation.run()
  File "/Users/alfoa/projects/raven_github/raven/framework/Simulation.py", line 750, in run
    stepInstance.takeAstep(stepInputDict)
  File "/Users/alfoa/projects/raven_github/raven/framework/Steps.py", line 338, in takeAstep
    self._localTakeAstepRun(inDictionary)
  File "/Users/alfoa/projects/raven_github/raven/framework/Steps.py", line 693, in _localTakeAstepRun
    sampler.finalizeActualSampling(finishedJob,model,inputs)
  File "/Users/alfoa/projects/raven_github/raven/framework/Samplers/Sampler.py", line 1033, in finalizeActualSampling
    self.localFinalizeActualSampling(jobObject,model,myInput)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/RavenSampled.py", line 265, in localFinalizeActualSampling
    self._useRealization(info, rlz)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/GradientDescent.py", line 347, in _useRealization
    self._resolveNewOptPoint(traj, rlz, optVal, info)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/RavenSampled.py", line 469, in _resolveNewOptPoint
    self._updatePersistence(traj, converged, optVal)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/GradientDescent.py", line 672, in _updatePersistence
    self._closeTrajectory(traj, 'converge', 'converged', optVal)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/RavenSampled.py", line 621, in _closeTrajectory
    Optimizer._closeTrajectory(self, traj, action, reason, value)
  File "/Users/alfoa/projects/raven_github/raven/framework/Optimizers/Optimizer.py", line 445, in _closeTrajectory
    self._activeTraj.remove(traj)
ValueError: list.remove(x): x not in list

Do you have a suggested fix for the development team?
Describe how to Reproduce
Steps to reproduce the behavior:
1.
2.
3.
4.
Screenshots and Input Files
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.
Platform (please complete the following information):

OS: [e.g. iOS]
Version: [e.g. 22]
Dependencies Installation: [CONDA or PIP]

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[ ] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

That's pretty bizarre! I don't think we use the location in the list to add, track, or remove the trajectories, so it's strange that we try to close a trajectory that isn't there.

agree, I need to investigate further...I am trying to understand why it happens


splitkey not working

I'm using the following  code to get data using a split key on pid. But unfortunately, spark is fetching all the data from MongoDB collection.Can someone say me where I'm going wrong.
val mongoConfig =  MongodbConfigBuilder(
Map(
Host -> mongoHost,
Database -> mongoDatabase,
Collection -> mongoCollection,
SamplingRatio -> 1.0,
WriteConcern -> "normal",
SplitSize -> "5",
SplitKey -> "pid",
SplitKeyMin -> "357",
SplitKeyMax -> "368",
SplitKeyType -> "int"
)
).build()
val df = spark.sqlContext.fromMongoDB(mongoConfig)

I found the reason for this problem,there is no index on "pid" field in my mongoCollection.I would be very helpful if this is included in the documentation.


Feat/icon dropdown

Closes #1373
Closes #1374
Closes #1259
Summary

Added icon and icon with label variant for dropdown

Change List (commits, features, bugs, etc)

Added Dropdown, Dropdown.story.jsx, Dropdown.test.jsx

Acceptance Test (how to verify the PR)

Two stories were created at

https://deploy-preview-1705--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-dropdown--default
https://deploy-preview-1705--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-dropdown--default-with-icons

I opened https://github.com/carbon-design-system/carbon-addons-iot-react/issues/1771 for the small visual fixes.

:tada: This PR is included in version 2.141.0-next.4 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.141.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Release cobigen-javaplugin-v2.2.3

Automatically processed.
The decisions taken by the developer and the context of the decisions throughout the script:

User confirmed that tests are running on CI and the build is not failing.
User has accepted to continue when found that some files were outside of build folder name


Satin Panties Erect doesnt play

Describe your bug here.
Crashes on load (well it restarts. whoever came up with that is a genius)

Command Prompt/Terminal logs (required)
(aren't able to copy from game | is included in screenshot)

Are you modding a build from source or with Lua?
No modding
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nothing

this is from 2.0.1 by the way. 2.1 just has a black screen


Rebase and Updated to F39 from F38

I was updating my steam deck with bazzite fedora 38 to verison 39 using the terminal, everything went fine and installed but as soon as I rebooted the system and was stuck in Emergency Mode.
(Screenshot below)

You might want to copy the files it mentions to a usb and post the log content so someone official here could better help you with it as that screenshot doesn't provide much information.

Due to the lack of activity on this issue, I am going to close this issue. Please reply back with logs to continue troubleshooting if required.


Dev portal side panels for Android SDK

Side panel docs are added https://github.com/wso2/identity-apps/pull/978


Support WM_CLOSE for graceful shutdown on Windows

Description
Currently go-spacemesh handles correctly the case when you run go-spacemesh under the attached console (cmd.exe) and press CTRL+C. However, it does not work when you run it without an attached console, for example, as a child process.
So if you send SIGINT from another application (that runs it detached or not in a console environment) — it will be ignored or forcefully killed.
Here is a topic from Node.js repo that is referencing this "common" (not only node.js) problem: https://github.com/nodejs/node/issues/35172
In our case, Smapp is killing go-spacemesh (when trying to send SIGINT or SIGTERM signals) without a chance to shut down properly. It also may cause db corruption.
Solution
Let's add support of TASKKILL without /F argument, aka WM_CLOSE.
In case of any problem with it we may consider listening for /q command in stdin, but it sounds hacky, since we don't use stdin
Steps to reproduce

Save the attached below JS code into some file
Change the path to go-spacemesh binary
Run it using node.js in cmd.exe, e.g. node script.js

In production, you can run Smapp on Windows and then click on Close and choose "QUIT" in a prompt.
Open the logs and you won't see there any log messages related to graceful shutdown.
Environment
Please complete the following information:

OS: Windows (any)
Node Version: <=1.2.8
Smapp version (if applicable): any

Additional Resources
The script:
const { spawn, exec } = require('child_process');

const cmd = spawn('C:\\Users\\username\\AppData\\Local\\Programs\\Spacemesh\\node\\go-spacemesh.exe');

setTimeout(() => {
  exec(
    'TASKKILL /PID ' + cmd.pid,
    console.log
    // Error: Command failed: TASKKILL /PID 2928
    // ERROR: The process with PID 2928 could not be terminated.
    // Reason: This process can only be terminated forcefully (with /F option).
  );
}, 5000);

setTimeout(() => {
  console.log('SIGINT');
  cmd.kill('SIGINT'); // <-- killed, not gracefully
}, 10000);

cmd.stdout.on('data', (data) => {
  console.log(`STDOUT: ${data}`);
});
cmd.stderr.on('data', (data) => {
  console.log(`STDERR: ${data}`);
});
cmd.on('exit', (code) => {
  console.log(`Child exited with code ${code}`);
});

I'm not 100% sure, but probably WM_CLOSE is for GUI applications only.
If so — then we need to pick some other solution:

listening stdin
starting some gRPC service ASAP, which implements the Shutdown endpoint
something else

I think the second option (\q) will be better choice for now, as we want to keep new API dependent on DB layer only.


Finding an advisory by its document ID

Advisories have "unique" IDs in their documents. At least, they should be unique. For SBOMs, every newly generated SBOM must have a new ID. For CSAF, documents get updated, keeping their ID.
Taking a look at our API, I can "get all advisories". That will return me a list of advisories, providing me both "hashes", a "uuid", as well as an "identifier". The "identifier" being the "document ID". However, that is not necessarily unique in our system. Either due to "bad data" (SBOMs) or due to updates (CSAF).
However, when I want to "get" information about an advisory, I must provide either the UUID, or a digest of the original document.
Coming from a perspective where I do have the "document ID", there currently doesn't seem to a way to: "get all advisories with this document id". And I think we should have that.

Doesn't /api/v1/advisory?q=DOCUMENT_ID return what you're describing?

Doesn't /api/v1/advisory?q=DOCUMENT_ID return what you're describing?

No because that would search all fields for DOCUMENT_ID, which not the same as searching for identifier = DOCUMENT_ID.

Probably amounts to the same results (and possibly more interesting to the user) but you could also call /api/v1/advisory?q=identifier%3dDOCUMENT_ID

I wasn't aware of this, good we already have it.


[chore] update dependabot pr script

This removes the need for the "Skip changelog" label and update the title to contain "[chore]". This works around the issue that opentelemetrybot doesnt have permission to edit a PR and add a label to it.
Signed-off-by: Alex Boten<EMAIL_ADDRESS>

How is the changelog CI check skipped without the "Skip Changelog" label?

How is the changelog CI check skipped without the "Skip Changelog" label?

Having [chore] in the name is another way of skipping the check https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/12f662b6c5b245a185a670e6a46f982f0805d385/.github/workflows/changelog.yml#L21


能在pytorch 1.7下训练吗

能在pytorch 1.7下训练吗？或者要怎么修改一下才能使用？

should be ok. you may have to change loss[0] to loss.item() if there are issues about the loss.

OK,thank you!


added codeforces sheet 1 questions and solutions

I have added problem statements along with their solutions from sheet 1 from codeforces

thank you!


fix(work-item-links): Display toast notification to indicate that a work item can have a single parent.

Display toast notification to indicate that a work item can have a single parent.
Fixes 1609

@nimishamukherjee fabric8/fabric8-ui:SNAPSHOT-PR-2345-7 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2345-fabric8-planner.badger.fabric8.io

@nimishamukherjee fabric8/fabric8-ui:SNAPSHOT-PR-2345-8 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2345-fabric8-planner.badger.fabric8.io


Wildcards in errorOverrides

Is there a way we can ignore any SQL errors with out existing the migration execution? I tried to explore the error override, but it requires to manually input all the sql error code and STATE and action, I am looking for something like flyway.errorOverrides=*:*:W. I have also tried to use a callback statement with "WHENEVER SQLERROR CONTINUE" , but the change doesnt stay when we move to a migration patch.( I guess that works if we have the "WHENEVER SQLERROR CONTINUE" in each migration patch. Appreciate your help here.

You can currently achieve this with Oracle, with SQL*Plus support activated, by placing WHENEVER SQLERROR CONTINUE in a beforeEachMigrate.sql callback.
I'm leaving this open as a feature request as it would however be nice to have this for all databases.

As mentioned initially, I did try to create a beforeEachMigrate.sql call back with "WHENEVER SQLERROR CONTINUE", but that did not work as this session setting did not take effect when the migration patch was run. However, if I place the "WHENEVER SQLERROR CONTINUE" inside the migration Patch, It continues the execution by throwing a warning message for the ORA error and the migration is marked as success on the FLYWAY schema table.
Logs are below:
Successfully validated 8 migrations (execution time 00:00.876s)
DEBUG: Schema "PROCESS_CICD" already exists. Skipping schema creation.
Executing SQL callback: beforeMigrate
DEBUG: Executing SQL: WHENEVER SQLERROR CONTINUE
DEBUG: Locking table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"...
DEBUG: Lock acquired for table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"
Current version of schema "PROCESS_CICD": 20.0502.201902181324
WARNING: outOfOrder mode is active. Migration of schema "PROCESS_CICD" may not be reproducible.
Migrating schema "PROCESS_CICD" to version 20.0502.201902181325 - create table test1
DEBUG: Executing SQL: delete * from sdr.test1
DEBUG: Rolling back transaction...
DEBUG: Transaction rolled back
ERROR: Migration of schema "PROCESS_CICD" to version 20.0502.201902181325 - create table test1 failed! Please restore backups and roll back database and code!
DEBUG: Locking table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"...
DEBUG: Lock acquired for table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"
DEBUG: Schema History table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE" successfully updated to reflect changes
DEBUG: Memory usage: 61 of 245M
ERROR: Unexpected error
org.flywaydb.core.internal.command.DbMigrate$FlywayMigrateException:
Migration V20.0502.201902181325__create_table_test1.sql failed
SQL State  : 42000
Error Code : 903
Message    : ORA-00903: invalid table name
Location   : C:\gireesh\Enviornment\flyway_revamped\repo\source\V20.0502.201902181325__create_table_test1.sql (C:\gireesh\Enviornment\flyway_revamped\repo\source\V20.0502.201902181325__create_table_test1.sql)
Line       : 1
Statement  : delete * from sdr.test1
    at org.flywaydb.core.internal.command.DbMigrate.doMigrateGroup(DbMigrate.java:370)
    at org.flywaydb.core.internal.command.DbMigrate.access$200(DbMigrate.java:54)
    at org.flywaydb.core.internal.command.DbMigrate$3.call(DbMigrate.java:284)
    at org.flywaydb.core.internal.jdbc.TransactionTemplate.execute(TransactionTemplate.java:74)
    at org.flywaydb.core.internal.command.DbMigrate.applyMigrations(DbMigrate.java:281)
    at org.flywaydb.core.internal.command.DbMigrate.migrateGroup(DbMigrate.java:246)
    at org.flywaydb.core.internal.command.DbMigrate.access$100(DbMigrate.java:54)
    at org.flywaydb.core.internal.command.DbMigrate$2.call(DbMigrate.java:164)
    at org.flywaydb.core.internal.command.DbMigrate$2.call(DbMigrate.java:161)
    at org.flywaydb.core.internal.database.base.Connection$1.call(Connection.java:147)
    at org.flywaydb.core.internal.jdbc.TransactionTemplate.execute(TransactionTemplate.java:74)
    at org.flywaydb.core.internal.database.base.Connection.lock(Connection.java:143)
    at org.flywaydb.core.internal.schemahistory.JdbcTableSchemaHistory.lock(JdbcTableSchemaHistory.java:155)
    at org.flywaydb.core.internal.command.DbMigrate.migrateAll(DbMigrate.java:161)
    at org.flywaydb.core.internal.command.DbMigrate.migrate(DbMigrate.java:139)
    at org.flywaydb.core.Flyway$1.execute(Flyway.java:1395)
    at org.flywaydb.core.Flyway$1.execute(Flyway.java:1356)
    at org.flywaydb.core.Flyway.execute(Flyway.java:1711)
    at org.flywaydb.core.Flyway.migrate(Flyway.java:1356)
    at org.flywaydb.commandline.Main.executeOperation(Main.java:161)
    at org.flywaydb.commandline.Main.main(Main.java:108)

Caused by: org.flywaydb.core.internal.sqlscript.FlywaySqlScriptException:
Migration V20.0502.201902181325__create_table_test1.sql failed
SQL State  : 42000
Error Code : 903
Message    : ORA-00903: invalid table name
Location   : C:\gireesh\Enviornment\flyway_revamped\repo\source\V20.0502.201902181325__create_table_test1.sql (C:\gireesh\Enviornment\flyway_revamped\repo\source\V20.0502.201902181325__create_table_test1.sql)
Line       : 1
Statement  : delete * from sdr.test1
    at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.handleException(DefaultSqlScriptExecutor.java:253)
    at org.flywaydb.core.internal.database.oracle.OracleSqlScriptExecutor.handleException(OracleSqlScriptExecutor.java:83)
    at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.executeStatement(DefaultSqlScriptExecutor.java:202)
    at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.execute(DefaultSqlScriptExecutor.java:125)
    at org.flywaydb.core.internal.resolver.sql.SqlMigrationExecutor.execute(SqlMigrationExecutor.java:77)
    at org.flywaydb.core.internal.command.DbMigrate.doMigrateGroup(DbMigrate.java:367)
    ... 20 more

Caused by: java.sql.SQLSyntaxErrorException: ORA-00903: invalid table name
    at oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:461)
    at oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:402)
    at oracle.jdbc.driver.T4C8Oall.processError(T4C8Oall.java:1065)
    at oracle.jdbc.driver.T4CTTIfun.receive(T4CTTIfun.java:681)
    at oracle.jdbc.driver.T4CTTIfun.doRPC(T4CTTIfun.java:256)
    at oracle.jdbc.driver.T4C8Oall.doOALL(T4C8Oall.java:577)
    at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:205)
    at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:41)
    at oracle.jdbc.driver.T4CStatement.executeForRows(T4CStatement.java:1222)
    at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1401)
    at oracle.jdbc.driver.OracleStatement.executeInternal(OracleStatement.java:2493)
    at oracle.jdbc.driver.OracleStatement.execute(OracleStatement.java:2423)
    at oracle.jdbc.driver.OracleStatementWrapper.execute(OracleStatementWrapper.java:1071)
    at org.flywaydb.core.internal.jdbc.JdbcTemplate.executeStatement(JdbcTemplate.java:235)
    at org.flywaydb.core.internal.sqlscript.StandardSqlStatement.execute(StandardSqlStatement.java:42)
    at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.executeStatement(DefaultSqlScriptExecutor.java:189)
    ... 23 more

Here is the log, when I added "WHENEVER SQLERROR CONTINUE" inside the migration script.
Successfully validated 8 migrations (execution time 00:00.943s)
DEBUG: Schema "PROCESS_CICD" already exists. Skipping schema creation.
Executing SQL callback: beforeMigrate
DEBUG: Executing SQL: WHENEVER SQLERROR CONTINUE
DEBUG: Locking table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"...
DEBUG: Lock acquired for table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"
Current version of schema "PROCESS_CICD": 20.0502.201902181324
WARNING: outOfOrder mode is active. Migration of schema "PROCESS_CICD" may not be reproducible.
Migrating schema "PROCESS_CICD" to version 20.0502.201902181325 - create table test1
DEBUG: Executing SQL: WHENEVER SQLERROR CONTINUE
DEBUG: Executing SQL: delete * from sdr.test1
WARNING: DB error ignored (Code: 903, State: 42000): ORA-00903: invalid table name
DEBUG: Executing SQL: commit
DEBUG: Successfully completed migration of schema "PROCESS_CICD" to version 20.0502.201902181325 - create table test1
DEBUG: Locking table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"...
DEBUG: Lock acquired for table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"
DEBUG: Schema History table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE" successfully updated to reflect changes
DEBUG: Locking table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"...
DEBUG: Lock acquired for table "PROCESS_CICD"."FLYWAY_SCHEMA_TABLE"
Successfully applied 1 migration to schema "PROCESS_CICD" (execution time 00:05.082s)
DEBUG: Memory usage: 61 of 245M

I am working on Oracle Database 12c Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production

Wildcards have now been implemented.


Figure out if using URLFetch if running on AppEngine is necessary

If we can always use NetHttpTransport, we can remove the logic in DefaultHttpTransportFactory.create (in ServiceOptions) to check if we're running on App Engine.

I think it should be safe to remove it.

I run a quick storage example and requests seem to work fine in AppEngine with NetHttpTransport. However from here I read:

com.google.api.client.javanet.NetHttpTransport is discouraged due to a bug in the App Engine SDK itself in how it parses HTTP headers in the response

Is this a known problem in AE? Or has it been already fixed?

Is this a known problem in AE? Or has it been already fixed?

Yes, we saw that before (and we should have mentioned that in the issue).
This was actually the reason we didn't do anything about it yet.
I am %99.9 confident that this should be fine however I could not trace the source of this comment or anyone that knows more about it. We did test it and it seems to be working fine.

This has become irrelevant with recent developments. We are not going to ensure google-cloud-java is compatible with Java 7 on App Engine; Java 8 on App Engine doesn't have the same restrictions with network fetching. Java 8 support on App Engine is quite close, and is being tracked in https://github.com/GoogleCloudPlatform/google-cloud-java/issues/1918 . Closing out this issue.


Move (or add) the "Show viewer list" from the context menu to the title bar

What should be added?
Move (or add a button) the "Show viewer list" from the context menu to the title bar more or less like this https://i.imgur.com/RZMozlU.png
Why should it be added?
Makes it easier to see the viewer list

Related: #1273 #1590


model is loaded as Loader._recreate_base_user_object.._UserObject

python == 3.8
tensorflow (gpu) == 'v2.8.0-rc1-32-g3f878cff5b6'
keras == 2.8
windows 10
I am saving the model as both an h5 and keras format with
model.save('model.h5')
model.save(model_location)
type(model)

keras.engine.functional.Functional

Then, I am loading the model with
model = keras.models.load_model(model_location)
# or
model = tf.saved_model.load(model_location) # results in the same
type(model)

<class 'tensorflow.python.saved_model.load.Loader._recreate_base_user_object.<locals>._UserObject'>
There is something up with the loading process that results in the following error when calling model.predict(X)
AttributeError: '_UserObject' object has no attribute 'predict'
tf.saved_model.contains_saved_model(model_location)

true

However, loading the same model saved as an h5 file
model = keras.models.load_model('model.h5')
type(model)

keras.engine.functional.Functional

works.
Thanks in advance!

@darrahts ,
Model.save() will save  keras type model. tf.keras.models.load_model will load it back as keras model.
tf.saved_model.save will save  generic model. tf.saved_model.load will load it back either type, as generic _UserObject model.
Also can you please look at this comment from the issue with the similar error.Thanks

@tilakrayal,
Thanks for the comment. The point of this is, why is keras.models.load_model not returning the model? It is returning the same thing as tf.saved_model.load.

@darrahts ,
In order to reproduce the issue reported here, could you please provide the complete code you are using. Thanks!

@darrahts This issue doesn't exist on my Linux machine. Please try with the latest version of tensorflow gpu on windows and let me know if the issue still persists . Thanks!

I am having this issue using tensorflow 2.8.0
from tensorflow.keras.layers import Input, Dense
from tensorflow.keras import Model
from tensorflow import saved_model
from pathlib import Path
from numpy.random import rand

inputs = Input(shape=(200,))
x = Dense(64, activation="relu")(inputs)
outputs = Dense(1, activation="sigmoid")(x)

model = Model(inputs=inputs, outputs=outputs, name="DummyTF")
model.compile()

data = rand(1, 200).tolist()
model.predict(data)

path = str(Path(__file__).parent.resolve() / "ai_model" / "models" / "dummy_tf")

model.save(path)

model_loaded = saved_model.load(path)
model_loaded.predict(data)

Error message:
Traceback (most recent call last):
  File "creating_dummy_tf_model.py", line 22, in <module>
    model_loaded.predict(data)
AttributeError: '_UserObject' object has no attribute 'predict'

Ok, I solved it using tf.keras.models.load_model as suggested here.
However, was the error I got the expected behavior?


1.18 - PowerShell clear command results in non-rendered prompt using Atlas Engine

Windows Terminal version
1.18.1421.0
Windows build number
10.0.22000.0
Other Software
Oh My Posh v16.9.0
Steps to reproduce
Oh My Posh recently added support for FTCS marks which seems to work really well with the latest preview release of Terminal, VSCode and iTerm. However, when typing clear, the entire prompt disappears and you can see the multiple cursors when typing (seems that way). Waiting a few seconds, or removing focus from Windows Terminal, it renders the prompt again. This behaviour wasn't there in 1.17.
https://github.com/microsoft/terminal/assets/2492783/00c657d5-42b7-4113-a1ff-07837c4b4937
This seems to be an issue with the new Atlas Engine as I can't reproduce it when it's not enabled.
EDIT: even without printing the marks, the behaviour is there.
Expected Behavior
It displays the prompt correctly.
Actual Behavior
There's an invisible prompt at the top (and consequent prompts as well for a few seconds).

I have the same problem, but when I recorded my screen with ScreenToGif to show the error, the error is not captured but in my screen I am seeing the error,
Here the result with ScreenToGif:

Here the error recorded my phone because as I mentioned is record my screen with software the error is not showed
https://github.com/microsoft/terminal/assets/3514681/313b7128-3e2d-4bfc-8e03-b801b6b635f6
and also we have this error sometimes:

What font is that? this might be #15416 or one of the other few rendering glitches we hotfixed

@zadjii-msft i have the same problem and I'm using the following font CaskaydiaCove Nerd Font

What font is that?

In my case that's Victor Mono Nerd Font.

What font is that? this might be #15416 or one of the other few rendering glitches we hotfixed

In my case it is Fira Nerd Font.

I see the issue. I also cannot capture a screenshot of the problem since that causes the problem to go away. And I'm also using CaskaydiaCove Nerd Font.

@rkeithhill-keysight true, I noticed the video I posted above also doesn't show the issue clearly and you can't screenshot either because I'm too slow 😂

Once again, I'm sorry for causing this issue! It was fixed by #15403 which has shipped in v1.18.1462.0. It should also already be available via the app store. I hope it works now!

@lhecker no problem at all, in fact thanks to you for solving the issue, you are the best. I will do feedback if I find something new regarding this.

Yeah it is working right now, thanks a lot for your help @lhecker


[Bug] SystemProgram.AccountDataSize  not found

Describe the bug
var blockHash = rpcClient.GetRecentBlockHash();
var minBalanceForExemptionAcc =
rpcClient.GetMinimumBalanceForRentExemption(SystemProgram.AccountDataSize).Result;
.AccountDataSize_  not found...

This is related to #267 and #277, in this concrete example it should be TokenProgram.TokenAccountDataSize

Closed by #280


[Question] whisper-tiny and return_timestamps

I am new to both transformers.js and whisper, so I am sorry for a lame question in advance.
I am trying to make return_timestamps parameter work...
I managed to customize script.js from transformer.js demo locally and added data.generation.return_timestamps = "char"; around line ~447 inside GENERATE_BUTTON click handler in order to pass the parameter. With that change in place I am seeing timestamp appears as chunks (result var in worker.js):
{
    "text": " And so my fellow Americans ask not what your country can do for you ask what you can do for your country.",
    "chunks": [
        {
            "timestamp": [0,8],
            "text": " And so my fellow Americans ask not what your country can do for you"
        },
        {
            "timestamp": [8,11],
            "text": " ask what you can do for your country."
        }
    ]
}

however the chunks are not "char level" granular as expected following the return_timestamps doc.
I am looking for ideas how to achieve char/word level timestamp granularity with transform.js and whisper. Do some models/tools need to be updated and/or rebuild?

Hi there 👋  Yes, you are correct, Transformers.js doesn't yet support char-level granularity.
For whisper (which is the only speech-to-text model currently supported), the model itself doesn't produce char-level timestamps. Instead, they output "timestamp tokens", which are placed between tokens . This means they are essentially treated as token boundaries (which act as word-level timestamps).
I will try add this functionality once more speech-to-text models are supported! Do you have any suggestions for what we should prioritize? By far, the most popular speech-to-text model seems to be wav2vec2 (34.7M downloads this month)! So, we can look into adding that one soon.

wav2vec2 seems popular, worth to consider adding for sure, I hope it is possible to provide base/small versions of its model suitable for browser runtime
I have also came across https://github.com/ggerganov/whisper.cpp which provides karaoke style / word level output. But I am not sure this is compatible tech.
Also it would be nice to extend callback_function with progress information or an extra progress_callback so one can track the progress of transcription

I have also came across https://github.com/ggerganov/whisper.cpp and https://github.com/jianfch/stable-ts which provides karaoke style / word level output. But I am not sure this is compatible tech.

There are indeed various implementations which are able to produce word-level (or lower) timestamps for whisper, but they all are essentially "hacks" to add this functionality (since whisper was not trained to do that). As stated here:

An alternative approach, that does not require an additional model, is to look at the probabilities of timestamp tokens estimated by the Whisper model after each (sub)word token is predicted. It was implemented for instance in whisper.cpp and stable-ts. But this approach lacks of robustness, because Whisper models do not have been trained to output meaningful timestamps after each word. Whisper models tend to predict timestamps only after a certain number of words have been predicted (typically at the end of a sentence), and the probability distribution of timestamps outside this condition may be inaccurate. In practice, these methods can produce results that are totally out-of-sync on some periods of time (we observed that especially when there is jingle music). Also the timestamp precision of Whisper models tend to be rounded to 1 second (as in many video subtitles), which is too inaccurate for words, and reaching a better accuracy is tricky.

Moreover, this functionality isn't currently present in the HF python library, so, it would probably be quite strange for the JavaScript implementation to have it before the original python version haha.
Maybe one day though! 😇

Is https://github.com/linto-ai/whisper-timestamped something you are looking into? Seems doing exactly what I am looknig for

Is https://github.com/linto-ai/whisper-timestamped something you are looking into? Seems doing exactly what I am looknig for

Yep! (That's one of the things I linked above). But as I said, it would probably be best if you open up a feature request to the Python library (https://github.com/huggingface/transformers) before we add it here 😊

I have requested a feature https://github.com/huggingface/transformers/issues/22590 ... I hope I am doing things properly, these are my very first interaction with HF and transformers

Adding more tts suggestions:

https://github.com/elevenlabs/elevenlabs-python
https://github.com/suno-ai/bark
https://github.com/coqui-ai/TTS

Not sure these are suitable for FE/transformers.js considering architecture or eventually performance.

word level timestamps has been merged into transformers https://github.com/huggingface/transformers/pull/23205
please consider adding support for this in transformers.js

I have modified title/tag of this issue to covert it into a feature request

Just an update: I've got most of it working on this branch: https://github.com/xenova/transformers.js/tree/whisper-word-level-timestamps 🔥🔥🔥  I think the last thing for me to do is test, but should be ready soon! Also expect to see a demo with https://github.com/xenova/whisper-web 😄
Python (left) vs. JavaScript (right)

surprise me

This is now available in v2.4.0! 🚀
Release notes: https://github.com/xenova/transformers.js/releases/tag/2.4.0
Example usage:
import { pipeline } from '@xenova/transformers';

let url = 'https://huggingface.co/datasets/Xenova/transformers.js-docs/resolve/main/jfk.wav';
let transcriber = await pipeline('automatic-speech-recognition', 'Xenova/whisper-tiny.en', {
    revision: 'output_attentions',
});
let output = await transcriber(url, { return_timestamps: 'word' });
// {
//   "text": " And so my fellow Americans ask not what your country can do for you ask what you can do for your country.",
//   "chunks": [
//     { "text": " And", "timestamp": [0, 0.78] },
//     { "text": " so", "timestamp": [0.78, 1.06] },
//     { "text": " my", "timestamp": [1.06, 1.46] },
//     ...
//     { "text": " for", "timestamp": [9.72, 9.92] },
//     { "text": " your", "timestamp": [9.92, 10.22] },
//     { "text": " country.", "timestamp": [10.22, 13.5] }
//   ]
// }


Update about and sources pages

Fixes #34.
Description
This PR remove the sources table in /about page and expand the one in the /sources page, as presented in @panchovm's mockups for transitional styles. Also updates the links to the Make WordPress team and to suggest new sources to the Catalog.
Preview
https://ov-fe-preview.herokuapp.com/sources

It looks excellent. I have some minor comments:

The title "We appreciate suggestions…" is in h5, it should be h3.
h3 font size is 23px, it should be 19px.
This style update is no longer using a body-big style. We should use the default paragraph style.
Related to the prior point, default paragraph looks slightly tight compared to mockup. I'd suggest a 1.8 for line-height to have more space between lines.
The "Suggest a new source" button is all capitals and text bigger than the mockup. I think this is part of button class. We should update all buttons to match the mockup.
Table should have a 4px as border-radius.

Additional comments
The :hover style of default link should not change its color but add the underline style

Thanks for pointing these details @panchovm. I updated the styles as requested, except for titles. Setting line-height: 1.9 gives exactly the value shown in mockups (30.4px). The button and table should have more accurate styles too.
The "We appreciate suggestions…" title is now an h3 but as you can see, it's now bigger than the previous titles which are semantically h3 but showing instead styles of an h4 because the title is-4 classes.
Currently, the title sizes are:
$font-size-h1: 3.56rem;
$font-size-h2: 2.25rem;
$font-size-h3: 1.75rem;   // ==> 28px
$font-size-h4: 1.43rem;   // ==> 22.88px
$font-size-h5: 1.25rem;   // ==> 20px
$font-size-h6: 1.12rem;   // ==> 17.92px

So should we update all title sizes? or just make
...
? is-5 is the closest to 19px for now.
The preview URL is the same.

We should update the title styles. The audio integration version might change it again, but I will make sure of pointing what changed and what's new. If the change impacts too much the transitional version, then we can skip it.

If we can update the title styles without a significant problem, then let's go with that.

We can, so I did it! I added specific changes to the
 titles and the button of the Sources page, both have the correct font size now. We can change global styles when the design with the audio integration is ready.
This is ready for another review.


Unable to run inference

i am getting below error when i try to test pretrained model on my image
=============End=============
Loading from ./save/pretrained/textsnake_vgg_180.pth
Start testing TextSnake.
Traceback (most recent call last):
File "demo.py", line 110, in
main()
File "demo.py", line 92, in main
inference(detector, test_loader, output_dir)
File "demo.py", line 46, in inference
contours, output = detector.detect(image)
File "/home/ec2-user/vinayak/TextSnake.pytorch/util/detection.py", line 239, in detect
tr_pred = output[0, 0:2].softmax(dim=0).data.cpu().numpy()
AttributeError: 'Tensor' object has no attribute 'softmax'

I think my pytorch version was the problem. I was using 0.4.0.
I have upgraded to 0.4.1 and now it works. Closing the issue
Regards


Code generation for schema refinements and queries

This PR contains the glue code which lets our kotlin generator build handles that can query and refine types using the type information contained in the associated Schema.
This PR contains fixes for the Refinement Kotlin generator to match Kotlin style and also changes for use of RawEntitys. It also updates golden tests to make use of the query generation.

Landing planning to address any further syntactic / style issues in future.


'Resource not found' for standalone running

When starting the standalone version I expected to be able to access the realms and other URIs.
Startup shows the following logging (including accessing a realm):
$ java -jar standalone-0.10.0.jar
[main] INFO com.tngtech.keycloakmock.standalone.Main - Server is running on http://localhost:8000
[vert.x-eventloop-thread-0] INFO com.tngtech.keycloakmock.impl.handler.CommonHandler - 404: GET /auth/realms/master

Is my expectation correct to be able to access these URLs? Or is this a bug?

At the moment, only the bare minimum endpoints are implemented to make OpenID Connect work. Stuff like /auth/realms/master is just not implemented, so you'll get a 404.

Do you need this endpoint? It does not seem to be part of the OpenID spec but a Keycloak-specific endpoint. Also, I did not find any documentation of this feature in the Keycloak documentation.

The validation that already exists in my application uses the information to check validity of the KeyCloak instance. So I would prefer to include this.
The Keycloak documentation is https://www.keycloak.org/docs-api/5.0/rest-api/index.html#_realmrepresentation and https://www.keycloak.org/docs-api/5.0/rest-api/index.html#_realms_admin_resource I think.

I think the standard way would be to use the /auth/realms/master/.well-known/openid-configuration to discover the URL to the public key, and the realm name is already part of the URL. But we can put this on the queue with low prio.


Add Deno.serveHttp adapter for webhookCallback

Example usage:
import {
  Bot,
  webhookCallback,
} from "https://raw.githubusercontent.com/rojserbest/grammY/main/src/mod.ts";

const bot = new Bot('') // <-- your bot token here;

bot.command("start", (ctx) => ctx.reply("Hi!"));

const server = Deno.listen({ port: 8000 });

for await (const conn of server) {
  serveHttp(conn);
}

async function serveHttp(conn: Deno.Conn) {
  const httpConn = Deno.serveHttp(conn);

  for await (const requestEvent of httpConn) {
    if (requestEvent.request.method == "POST") {
      await webhookCallback(bot, "serveHttp")(requestEvent);
    }

    await requestEvent.respondWith(new Response(undefined, { status: 200 }));
  }
}

This will have to wait until #61 is merged, which will largely refactor how adapters are organised.

Yeah, nice PR. But what will be done to std/http? 🤔

Also, I guess that PR lets defining custom adapters too.

The API I desire is to have a bunch of framework adapters supported out of the box and accessible via simple string identifiers. If you're using some esoteric stuff that grammY doesn't support, you can still pass your own definition. I am still waiting for feedback on this.

what will be done to std/http?

What do you mean?

Feedback on how adapters should work?

What do you mean?

I mean: will we support both serveHttp and std/http, separately?

Feedback on my suggestion that it's superfluous to expose 20 framework adapters when all you can do with them is to pass them to the same function. They shouldn't be accessible as objects.
Continue this here: https://github.com/grammyjs/grammY/pull/61#discussion_r749303091

What do you mean?

I mean: will we support both serveHttp and std/http, separately?

They're different abstractions. Why not?

Please resolve the conflicts.

Done!

Sorry, I don't understand you. What the "this" in "this can be used"?

Sorry for being ambiguous. Does this work?
import {
  Bot,
  webhookCallback,
} from "https://raw.githubusercontent.com/rojserbest/grammY/main/src/mod.ts";

const bot = new Bot('') // <-- your bot token here;
const handleUpdate = webhookCallback(bot, "serveHttp");

bot.command("start", (ctx) => ctx.reply("Hi!"));

const server = Deno.listen({ port: 8000 });

for await (const conn of server) {
  serveHttp(conn);
}

async function serveHttp(conn: Deno.Conn) {
  const httpConn = Deno.serveHttp(conn);

  for await (const requestEvent of httpConn) {
    if (requestEvent.request.method == "POST") {
      await handleUpdate(requestEvent);
    }
  }
}

Sure it does. You can even test it fast using Deno deploy.

So there's no reason you called await requestEvent.respondWith(new Response(undefined, { status: 200 }));?

@all-contributors add @rojserbest for code

So there's no reason you called await requestEvent.respondWith(new Response(undefined, { status: 200 }));?

There was a reason, but when I edited my code wrongly some days ago, I removed the lines where you would see the reason. Fixed now. See it again to see what I meant by that.

But now you're still responding to the request twice, aren't your? Once inside the adapter, and once afterwards in the try block

That line would make sense if someone sent a get request, so they would know that the server is running. After all, it's just an example and you can remove that line and it will still work.

It will also run on POST requests

Miss it's good

My bad, I'm lazy.

No worries, just wanted to know if I understood it correctly


Send response from inside a promise

I am trying to simulate a delay in my response for testing purposes. I have tried following:
Approach 1:
setTimeout(()=>{
  context.string("Hello World", 200);
}, 2000);

Approach 2:
I created a function that returns a promise:
function getResponse(){
    return new Promise((resolve) => {
    setTimeout(() => {
      resolve("Hello World");
    }, 2000);
  });
}

Then in my handler,
getResponse().then(response=>{
  context.string(response, 200);
})

Both of these approaches don't seem to work. I am getting a response immediately after invoke the request and the response body is undefined.
Apologies if I have missed something very obvious, I just started exploring this library.

Fixed it by using await instead of .then():
Previous Approach:
getResponse().then(response=>{
  context.string(response, 200);
})

Correct Approach
const response = await parser.getResponse();
context.string(response, 200);


Fix weird comic toggle behaviour

Fixes https://github.com/BookMyComics-Developers/BookMyComics/issues/94.

On principle and based on the diff, LGTM
Waiting CI to approve


Failed to load url history (resolved id: history)

Describe the bug
Describe the bug
I receive an import error locally on my machine and on stackblitz. I am using the latest version of vitest ^0.31.0"
Here is the error:
FAIL  src/components/common/Widgets/Allflex/Health/index.test.tsx [ src/components/common/Widgets/Allflex/Health/index.test.tsx ]
Error: Failed to load url history (resolved id: history) in C:/VAS/Morpheus-FE/src/testHelper.jsx. Does the file exist?
 ❯ loadAndTransform node_modules/vite/dist/node/chunks/dep-934dbc7c.js:53340:2

Reproduction
https://stackblitz.com
Run yarn install and and then yarn test
System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz
    Memory: 16.19 GB / 31.50 GB
  Binaries:
    Node: 16.19.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.19.3 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (113.0.1774.42)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-react: ^3.1.0 => 3.1.0
    @vitest/ui: latest => 0.31.0
    vite: ^4.2.1 => 4.3.5
    vitest: ^0.31.0 => 0.31.0

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The src/testHelper.tsx is importing history:
6 | import { createMemoryHistory } from 'history';

However you have not installed history as dependency and it is not even present in yarn.lock through possible transitive dependencies.
Did you forgot to install it, e.g. yarn add --dev history?

@AriPerkkio thanks for your support, have a nice day


dev: Add option to enable file polling on local system for file syncs 📁 💻

Is your feature request related to a problem?
Yes, currently devspace dev local syncs rely on your local directory to emit MODIFY events when a file is updated.
This can cause problems when running devspace on top of a vagrant file-share, for example, in which file changes effectively trigger ATTRIB events in place of MODIFY events on the "local" directory via something like https://github.com/adrienkohlbecker/vagrant-fsnotify (events are forwarded and "downgraded" via remote executions of touch, basically).
Note that file polling (in place of inotify watches) is supported as an option on the container end of the file sync, but not on the local side. Note that this is not 100% clear in the v6 docs.
Which solution do you suggest?
Add support for enabling file polling (in place of notify watches) on local system for file syncs
Which alternative solutions exist?
Refactor the "local" inotify watchers to react to ATTRIB events in addition to MODIFY events. This may be noisy and non-ideal, and most likely would not work for all use cases. A file polling fallback could be a catch all for local issues with file syncs.
Additional context
slack conversation xref

/kind feature

@sgreene570 thanks for creating this issue! Yes we should be able to use something similar to our backend solution.


Winget failing to install silently with --silent

Brief description of your issue
When installing application Krisp using the --silent command it actually launches the instllaer GUI. Interesting when I remove the --silent command it installs silently. I think I noticed this happen with another application too.

Steps to reproduce
winget install krisp.krisp --silent
Expected behavior
Application should install silently.
Actual behavior
Application shows GUI to install.
Environment
Windows Package Manager v1.6.3482
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22631.2861
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3482.0

This is likely due to the switches in the manifest or the installer itself.

It's an inno based installer. When I pass "-?" to the installer I see:

WinGet was passing in:
2024-01-09 16:35:59.604 [CLI ] Installer args: /VERYSILENT /NORESTART /LOG="C:\Users\denelon\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Krisp.Krisp.2.24.5-2024-01-09-16-35-59.604.log"
2024-01-09 16:35:59.605 [CLI ] Starting: 'C:\Users\denelon\AppData\Local\Temp\WinGet\Krisp.Krisp.2.24.5\Krisp_2.24.5.exe' with arguments '/VERYSILENT /NORESTART /LOG="C:\Users\denelon\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Krisp.Krisp.2.24.5-2024-01-09-16-35-59.604.log"'

I'm guessing /VERYSILENT is the default for INNO, but this package is expecting /quiet.
The manifest might be able to be updated so silent passes in "/quiet" rather than the default value.

There's a new version waiting for approval to be merged. I think we can update the manifest there.

Related #133500

Thanks for the help everyone!

No problem!


Image automatically rotates after picking if crop options are there.

Hi there,
given below is my code for picking image.if crop options are not given , it would work fine.otherwise the picked image is rotated 90 degrees in the image view if I haven't done with any cropping.Please help me on this.
ImagesPicker.ImagesPicker.pick(         count: 1,         ImagesPicker.PickType.image             : ImagesPicker.PickType.video,         quality: 0.8, // only for android         maxSize: 500,         cropOpt: CropOption(                 aspectRatio: CropAspectRatio.custom,                 cropType: CropType.rect, // currently for android               )            ,       );
ImagesPicker.ImagesPicker.openCamera(         count: 1,         ImagesPicker.PickType.image             : ImagesPicker.PickType.video,         quality: 0.8, // only for android         maxSize: 500,         cropOpt: CropOption(                 aspectRatio: CropAspectRatio.custom,                 cropType: CropType.rect, // currently for android               )            ,       );

Android?

Can you record a gif and show me the effect?

I think you can search action in PictureSelector


Wanted Function getTransactions

Wanted function.

std::vector<AM_val> getTransactions(const size_t& index, bool uncommitted = false);

It provide that getting all transactions after index -th in ametsuchi db.
( But its size is very large. Actually, we should get alike streaming )
It will be used by new peer when that wants to synchronize transaction.

I think we can give you some sort of iterator.
auto it = ametsuchi.iterator(N);
++it;
--it;


Add cucim.skimage.feature.match_descriptors

closes #193
This PR adds cucim.skimage.feature.match_descriptors. It is a very straightforward adaptation of the scikit-image code, only substituting numpy->cupy. The only differences of note are:

when new enough CuPy and pylibraft are not available it warns and falls back to cdist on the CPU
test cases involving BRIEF temporarily involve round trip to the host until we implement BRIEF here (looks not too hard)

This PR also removes the deprecated masked_register_translation and register_translation (these were moved to cucim.skimage.registration.phase_cross_correlation) from the feature` module. Removing those was missed when previously updating the API here to match scikit-image 0.19.

The acceleration provided here will depend on the number and size of the descriptors. Running scikit-image's SIFT on the small astronaut image results in 1234 descriptors each of size 128. For that size of input, the relative acceleration on the GPU is quite good.
When cross_check=False, max_distance=np.inf, max_ratio = 1.0
Acceleration =<PHONE_NUMBER>8425943
When cross_check=True, max_distance=np.inf, max_ratio = 1.0
Acceleration =<PHONE_NUMBER>0822292
When cross_check=False, max_distance=100.0, max_ratio < 1.0
Acceleration = 58.34578386444441
In the latter two cases there are progressively more logical comparisons and indexing operations which end up reducing the overall acceleration factor as compared to the first case without any additional checks.

Moved to 22.10
There's not a project board for this yet. So moved to feature planning for now. Can update that later

Removing ops-codeowners from the required reviews since it doesn't seem there are any file changes that we're responsible for. Feel free to add us back if necessary.

Yeah I accidentally got 21.10 the first time 🤦♂ Sorry for the noise 😞
Should be fixed now 🍀

This PR now has that same issue we saw previously where the distance_transform_edt import fails even though the files are in this branch.
src/cucim/core/operations/morphology/tests/test_distance_transform.py:8: in <module>
    from cucim.core.operations.morphology import distance_transform_edt
E   ModuleNotFoundError: No module named 'cucim.core.operations.morphology'

I need to look into how we are doing the install when testing on CI. Perhaps the files are there during build but are getting omitted somehow during the package install stage?

rerun tests

@gpucibot merge


Error utf8 while exporting in .csv

As you can see in the screenshot, i have errors while exporting in .csv my json.
This issue comes out just when I open the .csv with Excel, instead of Notepad++ that format the characters correctly.
I don't know if you can do something on your own about this..

Hi there, I think you need to create a new excel document and import the csv data using the "from text" option and selecting utf-8 encoding.
Look here  for more explanation https://www.itg.ias.edu/content/how-import-csv-file-uses-utf-8-character-encoding-0

But we can add the byte order mark on the file.
https://en.wikipedia.org/wiki/Byte_order_mark


Non CocoaPods Version

Can you please provide a non CocoaPods Version? Not able to launch even the Example while Framework is missing ...

I've just released a version with carthage support and example without cocopods integration, checkout!


Helm Release job

This adds a workflow, using chart releaser, to release the helm chart to the gh-pages branch. So others can us it as a repo.
helm repo add redis-commander https://joeferner.github.io/redis-commander/
helm install redis-commander/redis-commander

@Umaaz Many thanks for this. One short question - the github doc of chart-releaser-action [1] has an extra step before the release to install helm binary. What is that needed for as your PR does not include it? Do you know it?
[1] https://github.com/helm/chart-releaser-action?tab=readme-ov-file#example-workflow

And Readme should be updated to mention the new possibility...


Add example of training PyTorch model with Horovod using OpenMPI

Issue #, if available:
Description of changes:
Added sample notebook that demonstrates how to run Horovod directly with OpenMPI.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

👍 lgtm

Getting this notebook to work will require changes to the PyTorch DLC.


Performance Issues

Rendering a "large" number of events (see screenshot) seems to slow things down to the point where there's noticeable lag and even "A webpage is slowing down your browser" at the Quarter and Year levels.
I haven't looked too far into it yet, but I'm fairly sure it's not an issue with my code. Figured I'd raise it here and we can discuss further and/or look into it if required.
I'm using repeating events if that's of any relevance.

+1
I am facing the same issue when the number of events goes past 70-80 events

I just started using this library because it has great layout and functionalities out of the box.
I think that the performance issue should be due to the fact that it mutates schedulerData for every action, and causes a re-render of the whole scheduler component.
Any suggestions for making the state management more 'react style'?

A fairly quick/dirty hack might be to just pass in the "relevant" events in the prevClick, nextClick, onSelectDate and onViewChange functions/callbacks. RBS looks through the entire event list to find what is actually relevant to render. In my situation, it's a year's worth of repeating events that's being passed, filtered, etc. every single time. For most situations, it doesn't need all that data.

@StephenChou1017 - Any ideas or is this going to be something quite involved to fix up?

I've tried to optimize the performance issue in 0.2.7, but I don't think it's enough.

@StephenChou1017 Can we optimize _createRenderData() function for improve performance. I think this method is calling every time change on the scheduler.

@StephenChou1017 - I think it's a bit better on 0.2.7, it's still taking 6 - 7 seconds of unresponsiveness to switch to Month view though (if it's that slow for Month, it's not worth trying Quarter or Year).

@FizzBuzz791
Hi, can you send me your testing data(resources and events) to me and I can try it myself?:-)
My email<EMAIL_ADDRESS>

yup, 1k events took me around minute to load...
did you guys @FizzBuzz791 @hanchiang @desilvaNSP found any workaround to this?

50+ events is loaded over  5+ seconds in my browser, about 20 resources, Help! Is it refer to the quantity of resource?

Sadly, no. I was using this library for writing a quick demo at work. Now that the focus of my work has shifted and have been real busy ever since.

I found the moment took a lot of time,  with using of chrome performance devtools.

Most of the time is wasted on time format conversion.

@hphpwj do you have a solution to reduce the loading time?

@jeffreybos change localeMoment(time) to 'localeMoment(new Date(time))can effectively reduce the loading time in the fileSchedulerData.js'. But the function Date() is time zone related, you can't replace all the localeMoment() to avoid unexpected mistakes.

@FizzBuzz791 Do we have solution to optimize for rendering large number of events on scheduler.?

Any update on this?

I always come back to see if it's improved, guess not yet 😉

I always come back to see if it's improved, guess not yet 😉

I always come back to see if it's improved, guess not yet

same here :/

I always come back to see if it's improved, guess not yet

same here :/

I always come back to see if it's improved, guess not yet

Me too.

I always come back to see if it's improved, guess not yet

Me too.

Are there any news?

Are there any news?

Hi, has anyone found a solution to the performance issues?

Alguma solução relacionada ao desempenho ?

Hi. I started using Syncfusion Scheduler (it has a community license)

@matteomessmer does it have a year view?

@madzadev yes, it has also good support and documentation

@matteomessmer thanks a lot! 👍😉

what is optimised  solution  for performance

https://github.com/hbatalhaStch/react-big-scheduler
I have improved its performance a lot compared to how it was but there's still a lot of room for improvement and I am welcoming all the help I can get.
Changes (for now)

Reduce loading time on view change
Update antd (v5.1.6)
Update react(v18.2.0)
Update react-dom(v18.2.0)

I am using day view to show some events for some resources.
How can I disabled the passed time and change its background color to greyish to indicate it as disabled and past.

I am using day view to show some events for some resources.
How can I disabled the passed time and change its background color to greyish to indicate it as disabled and past.

here is an example:
let schedulerData = new SchedulerData('2017-12-17', ViewType.Week, false, false,
            {
                // SchedulerDataConfig
            },
            {
                getNonAgendaViewBodyCellBgColorFunc: (schedulerData, slotId, header) => {
                    if(dayjs(header.time).hour() < 10 ){
                        return '#87e8de';
                    }
                    return undefined;
                }
            }
            );

Beware, I tested on the fork react-big-scheduler-stch but it should should for react-big-scheduler with no problems so you probably should use moment instead of dayjs if you feel like it.


Website defaults to master API documentation

Website shows API documentation for master version by default. That's not useful, as most users will run the stable MXNet version and will be confused by breaking changes in MXNet 2 website.
Reference:
https://github.com/apache/incubator-mxnet/issues/18428

cc: @connorgoggins can we default to the latest stable branch?

Please note that the traffic to 1.6 version website only account for 0.54% of total traffic according to Google Analytics.( from 01/2020 to present ).
The problem seems to be the API docs website failed to clearly showing the users what version of docs they are seeing and what is the stable version.
@sandeep-krishnamurthy

This issue can be closed.


Update node-mapnik to 3.5.10

Pull in @flippmoke's fixes to segfault in node-mapnik 3.5.9.
cc/ @springmeyer

Seeing green on the branch test in appveyor, but still seeing red for the PR test. @mapsam were you seeing this before?

Green on travis. Merging.


Ensure all pending SSL data is written before closing channel during …

…handshake error.
Motivation:
We need to ensure we call ctx.flush() before closing the actual channel when an handshake failure took place. If we miss to do so we may not send all pending data to the remote peer which also include SSL alerts.
Modifications:
Ensure we call ctx.flush() before ctx.close() on a handshake error.
Result:
All pending data (including SSL alerts) are written to the remote peer on a handshake error.

@trustin PTAL

Cherry-picked into 4.0 (d9d0e0c973d05b59192d4f4d87a76af30d214839) and 4.1<PHONE_NUMBER>2cb36c8cb04ac0bb4e3b9ef58c410d)


Jetpack mini-thread

Issue 1: Mini-jetpack does not shut down after running out of propellant.
To replicate, just fill the jetpack up with any amount of air (I did it from max fill and 100kpa), toggle thrust on, let it drain to 0kpa remaining. Still able to move around in space, no loss in functionality other than not being able to double it as air supply.
Issue 2: Mini-jetpack/jetpack do not fit in exoslots.
Honestly, not as big of an issue but we have the exoslot whitelisted for O2 tanks, which this is presumably an extension of. That said, Sloth said he forgor exoslot was a thing and I can go either way on this one. Could be too powerful being an exoslot item, would merit testing though. We're already barring salvage from having this so it'd offset some of the power gain there, engineering does still need the ability to carry materials around for repairs so ditching the bag kind of hinders that. Open to discussion on this one.
Issue 3: Jetpack speeds affected by thirst/hunger slowdown, presumably other kinds of slowdown as well.
As I said in the discord, kind of funky interaction, not really opposed to this one either, mostly just reporting this so it doesn't get overlooked if not intentional.

With #9206, I'm fixing issue 1.
Issue 3 is certainly an unintended interaction, at least in my view. It doesn't make sense for hunger to affect it.
I'll make another issue to separate it and make it easier to resolve.
Issue 2 is up to the maintainers. Due to that, I'll be adding the "Needs discussion" tag


Dropdown: qa-data attribut

Dropdown option <label /> & <input/> elements are not possible to identify without checking the inner HTML, which is not convenient when trying to identify an option with test-scripts
Solution: add qa-data attributes

Closing this issue, since both the problem and solution are likely different with the new Core dropdown.
In the new dropdown, the options are represented by <gds-option> elements. So if the consumer wants to add qa-hooks, or other ways to target these, they can. Since these elements are slotted, they can be queried and selected like regular DOM nodes.
So, an application that have special QA/testing needs might do something like:
<gds-dropdown>
  <gds-option data-qa='qa-option-1'>Option 1</gds-option>
  <gds-option>Option 2</gds-option>
  <gds-option>Option 3</gds-option>
</gds-dropdown>

They can then easily single out that option by:
const opt = document.querySelector('[data-qa=qa-option-1]')


clean up things that don't work in SL4

Not sure what travis is complaining about, I guess it check with the config from master. That won't work. But the PR is green.


bug: Request is unparsable, syntactically incorrect, or violates schema.

Describe the bug
due to limitations in the AWS SSO SCIM API, specifically the patchgroup operation when some groups have more than 100 members
...
* A maximum of 100 membership changes are allowed in a single request.
...

the ACIM API response with:
{"schema":["urn:ietf:params:scim:api:messages:2.0:Error"],"schemas":["urn:ietf:params:scim:api:messages:2.0:Error"],"detail":"Request is unparsable, syntactically incorrect, or violates schema.","status":"400","exceptionRequestId":"f32ad558-5b05-45ea-8135-552a2f254d4f","timeStamp":"2022-04-15 14:09:03.693"}

To Reproduce
Have a group in Google Workspace with more than 100 members
Expected behavior
do the sync without any error

Hi there!
This bug appeared again for us; we're using last version (0.0.13).
We're syncing three groups, and one of them have more than 100 users (137 right now).
I deleted the state to force a clean run of the sync app, but it still returns that error.
Thanks a lot!

hi @snavarro-factorial, could you provide some logs to understand your case?

Sure!
First it starts syncing as it normally does, typical loop of "creating user -> it already exists":
{
    "email"<EMAIL_ADDRESS>    "level": "warning",
    "msg": "creating user",
    "time": "2022-07-18T09:31:41Z",
    "user": "xxxxxx"
}
{
    "level": "warning",
    "msg": "aws CreateOrGetUser: user already exists, trying to get the user information",
    "time": "2022-07-18T09:31:41Z",
    "user"<EMAIL_ADDRESS>}

After that loop finishes, it continues with this:
{
    "level": "info",
    "msg": "no users to be updated",
    "time": "2022-07-18T09:31:41Z"
}
{
    "level": "info",
    "msg": "no users to be removed",
    "time": "2022-07-18T09:31:41Z"
}
{
    "level": "info",
    "msg": "getting SCIM Groups Members",
    "time": "2022-07-18T09:31:41Z"
}

And then the error:
Error: cannot sync groups and their members: error doing the first sync: error getting groups members from the SCIM service: scim: error listing groups: statusCode: 400,  errCode: 400 Bad Request, errMsg: {
    "schema": [
        "urn:ietf:params:scim:api:messages:2.0:Error"
    ],
    "schemas": [
        "urn:ietf:params:scim:api:messages:2.0:Error"
    ],
    "detail": "Request is unparsable, syntactically incorrect, or violates schema.",
    "status": "400",
    "exceptionRequestId": "02a53bb5-7400-4383-ab74-70a5a5d6dcaf",
    "timeStamp": "2022-07-18 09:31:44.223"
}

After that, it just outputs the help usage.
Thanks a lot!

This is the error if I try to use the command line version:
INFO[0000] starting sync groups                          codeVersion=v0.0.13
INFO[0000] getting identity provider data                group_filter="[name=SysAdmin name=Developers name=Data]"
INFO[0051] getting state data
Error: cannot sync groups and their members: error getting state data from the repository: s3: error getting S3 object: bucket: xxxxxx, error: operation error S3: GetObject, https response error StatusCode: 301, RequestID: MJMZXDTHGK3AGCC1, HostID: Hf23B8NVmlbJWFLsiv4N7a0/Mt/BhOr28/FEV3gH0VW3KL5gOAUrn7ddCpVoRmeqwwzzIxiq9jI=, api error PermanentRedirect: The bucket you are attempting to access must be addressed using the specified endpoint. Please send all future requests to this endpoint.

What does it mean with "specified endpoint"? S3 VPC endpoints? Previously it worked just with the s3 bucket name, but I tried with ARN just in case:
INFO[0000] starting sync groups                          codeVersion=v0.0.13
INFO[0000] getting identity provider data                group_filter="[name=SysAdmin name=Developers name=Data]"
INFO[0050] getting state data
Error: cannot sync groups and their members: error getting state data from the repository: s3: error getting S3 object: bucket: arn:aws:s3:::xxxx, error: operation error S3: GetObject, invalid Amazon s3 ARN, unknown resource type, arn:aws:s3:::xxxx

Thanks!

More info, I actually deleted some users on SSO, rerun the lambda/CLI, and it recreates everything, but in the end throws that error and doesn't create a state file in S3.

@snavarro-factorial this is a new issue, this is not the same issue #64
I opened a new issue with your comment with the number #75, let's continue the traceability there.


Fixture load Alternatives

can i load fixtures using other methods other than the ones stated on the read me?
ie on a file or anywhere
module.exports = {
//fixture stuff here

}

instead of having a file, this will allow for greater flexibility as to where the assosiations are stored and can even be stored across multiple files per waterline collection,also opens up usage to other areas waterline is used outside sails

Hi,
In principe the fixtures are only defined for sails' usage via local.js. module.exports.fixtures defines the fixtures to be loaded, so you are free to add your own loader as export definition:
/i.e. sails.config.fixtures.js
module.exports = {
 loadFixtures: function () {
    return {
      //define your fixtures
    };
  };
}

//in sails.config.local.js
var fixtures = require('./fixtures');
module.exports = {
  fixtures: fixtures.loadFixtures();
};

Second, if you want to use the module outside of sails, unfortunately this is not supported. If you look at lib/index.js, the file is explicitly modeled to act as a Sails hook (according to http://sailsjs.org/documentation/concepts/extending-sails/hooks/installable-hooks). The easiest way to decouple the package from Sails would be to create a new NPM package solely for the waterline functionality, and include it as dependency in this module. It would require a major rewrite of index.js as you need to interface with the Waterline models in a different way. Right now I don't have time or incentive to implement this, so your best bet would be to fork the repo yourself.
Is this helpful in any way?

Since I've not gotten a response back, I'll assume the answer was sufficient, therefore closing this issue.


Add support for NetCDF data files

Datacubes are starting to be shared in NetCDF format.
ADRIA currently expects datacubes to be in MAT format.
Small adjustments need to be made to support the former.

No longer relevant.


Multi-root settings missing, root is set to OS root.

I can't find the setting appearing in #30.
In a multi-root workspace, It also does not open the first workspace folder as root (which is what I read was the behavior before multi-root support), but instead uses the full path to the OS root:

(first workspace folder is _site/)

Needless to say, relative links do not work either.
Is there something I'm missing?

@anghelos, apologies for not getting back to you on this until now. While pull request #30 contains the initial implementation of multi-root support, commit 442f12519f8104622ccfdfa7478afb136e42b4ed has refactors the multi-root support so that rather than picking the default workspace for the server, each workspace is being served on its own endpoint so that you can preview files from multiple workspaces. Could you please elaborate further on the issue with relative paths that you are experiencing? Thanks!

I'm a little confused by your use of the word "workspace" here. Are you talking of multiple workspaces opened in separate windows, or multiple folders in the same workspace (which I believe is what multi-root support was supposed to cover)?
As for the relative path issue, This is the behavior I'm witnessing:

In a Workspace with a single root folder, relative paths work as expected.
After adding a folder to the workspace, the server's root is set to Windows' root.
If the added folder is a subfolder of the first, the server's root will remain tied to the first folder until the workspace is re-opened, then the same problem as 2 occurs.
If I remove the second folder, everything works as intended after I close and reopen the Studio Code window.

Agreed that the folder/workspace/multi-root workspace terminology is confusing. I have to say that I a little confused on step No3 that you have described above. What do you mean that the folder that you are adding is a subfolder of the first? In what scenario would you add a folder that is already present in the Explorer? Could you share the folder layout that you have an exactly which folders are you adding to the multi-root workspace? Thanks!

@anghelos, I have tried to reproduce your steps using the test-workspace folder from this repository. Here were my steps:

Open the test-workspace folder in VS Code. Open subfolder_page.html and preview it. Image using relative path works.
Add test-workspace\subfolder to the workspace. This is extremely weird as now you have this folder appear in "both" workspaces. Notice in the screenshot below that the image is still being loaded successfully.

What am I missing? Thanks!

Actually, you can see the bug right in your screenshot! The URL on the browser should be "http://<IP_ADDRESS>:3000/subfolder_page.html" (or even just "http://<IP_ADDRESS>:3000/")
To be honest, the only reason I'm adding this subfolder as a root folder is only because I want it to be the root for Visual Preview.
I'm using a static site generator (Eleventy, but this applies to all/most of them), which generates the web pages in a subfolder ("_site"). I want to preview those pages while still being able to use src="/folder/image.jpg" (instead of your "../").

Thanks @anghelos! Interestingly enough, that was implemented originally in this extension but then the functionality was removed and as you have stated, as soon as you have a multi-root workspace, all files are being served as an out-of-workspace file. I will try to get some context on why the functionality from #30 has been refactored and will circle back to this issue.

@lszomoru: Any news about this?

Hi! The current version seems even more broken. When I start live preview in a Workspace I get the following error message on the HTML preview:
File not found
The file "\c:\Workspace\myproject\function (old_index, new_index) { \ Shortcut helper to move item to end of array if (-1 === new_index) { new_index = this.length - 1; } if (new_index >= this.length) { var k = new_index - this.length; while (k-- + 1) { this.push(undefined); } } this.splice(new_index, 0, this.splice(old_index, 1)[0]); return this; \ for testing purposes }\index.html" cannot be found. It may have been moved, edited, or deleted.
It seems there is some Javascript code, which is not evaluated, but gets put directly into the URL.

version seems even more broken. When I start live preview in a Workspace I get the following error message on the HTML preview:
File not found The file "\c:\Workspace\myproject\function (old_index, new_index) { \ Shortcut helper to move item to end of array if (-1 === new_index) { new_index = this.length - 1; } if (new_index >= this.length) { var k = new_index - this.length; while (k-- + 1) { this.push(undefined); } } this.splice(new_index, 0, this.splice(old_index, 1)[0]); return this; \ for testing purposes }\index.html" cannot be found. It may have been moved, edited, or deleted.
It seems there is some Javascript code, which is not evaluated, but gets put directly into the URL.

@falco467 does this still happen?