RuntimeError(f"{self.class.qualname} is not attached to a Trainer.")

Hello
Thanks very much for your comprehensive repository.
The latest commit and additions to src/models/semantic.py cause an error when running python src/train.py experiment=semantic/s3dis datamodule.fold=5
[2024-10-29 12:48:41,430][src.utils.utils][ERROR] -
Traceback (most recent call last):
  File "/home/appuser/miniconda3/envs/spt/lib/python3.8/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 92, in _call_target
    return _target_(*args, **kwargs)
  File "/home/appuser/superpoint_transformer/src/models/semantic.py", line 141, in __init__
    if self.trainer.check_val_every_n_epoch is not None:
  File "/home/appuser/miniconda3/envs/spt/lib/python3.8/site-packages/pytorch_lightning/core/module.py", line 218, in trainer
    raise RuntimeError(f"{self.__class__.__qualname__} is not attached to a `Trainer`.")

Without searching to understand it in detail, I figured that the access to self.trainer in init with self.trainer.check_val_every_n_epoch  causes the error.
Reverting to the last commit solved the issue for me.
I hope this helps
BR
Fiona

Hi @fclairec, thanks for pointing this out !
This is indeed an issue I recently caused by pushing some unsufficiently tested code :sweat_smile: I am working on fixing this and will push the fix as soon as possible. In the meantime, you can use commit 6cb9d93, where none of the breaking changes were added yet.
Sorry for the inconvenience !
If you ❤ or simply use this project, don't forget to give the repository a ⭐, it means a lot to us !


ActiveRecord::RecordNotUnique error occurs when save  .

I have an osm data. Both node and way ids are increasing from 1,so it is with changset. I use postgresql database to store , openstreetmap-website to show, and ID editor to edit.  All works well when edit.  However, When I try to save , It  occurs error:
ActiveRecord::RecordNotUnique: PG::UniqueViolation: ERROR: duplicate key value violates unique constraint "changsets_pkey" DETAIL: Key (id)=(1) already exists. : INSERT INTO "changesets" ("created_at", "user_id") VALUES ($1, $2, $3) RETURNING "id"
Click save button again ,it occurs error:
ActiveRecord::RecordNotUnique: PG::UniqueViolation: ERROR: duplicate key value violates unique constraint "current_nodes_pkey1" DETAIL: Key (id)=(1) already exists. : INSERT INTO "current_nodes" ("latitude", "longitude", "changeset_id", "version", "visible", "timestamp", "tile") VALUES ($1, $2, $3, $4, $5, $6, $7) RETURNING "id"
I guess id editor save node,way and changeset using ids increasing from 1. If I am right ,How to edit osm data which ids increasing from 1?

closing, moved upstream to openstreetmap/openstreetmap-website#1542

Thanks for your quick and detailed reply. it works well now.


Fix two memory leaks

See SD-950.
This fixes two leaks that showed up pretty quickly with some profiling and a very simple shell script to generate a steady request load.
The one that presumably was causing the crashes reported by the user was the waitForInput loop, which would consume 100s of MB over the course of a week even with no requests coming in.

Current coverage is 80.17%

Merging #1146 into master will decrease coverage by -0.01% as of c4ba1e0

@@            master   #1146   diff @@
======================================
  Files          163     163
  Stmts         8913    8915     +2
  Branches       282     281     -1
  Methods          0       0
======================================
+ Hit           7147    7148     +1
  Partial          0       0
- Missed        1766    1767     +1

Review entire Coverage Diff as of c4ba1e0

Powered by Codecov. Updated on successful CI builds.

I've been running a build of this branch for 21 hours, with a script running a query, downloading an HTML page, and uploading a small JSON dataset every 5 seconds. There are no new retained objects visible in the memory profiler, and the used hep size is steady at just under 200 MB.

@mossprescott Thanks for your efforts in fixing these! :+1:

Merged as 5.0.2 via the script.


Upgrade the xRegistry resource to high quality

This is the high quality upgrade of xRegistry for #160.
It includes @Amedama96 's fix for setting a Binary registry value to 0.
Unit tests, integration tests, and examples have been added.
Documentation has been updated.

This change is

Reviewed 2 of 17 files at r1.
Review status: 1 of 16 files reviewed at latest revision, 7 unresolved discussions.

README.md, line 363 at r1 (raw file):

* **[String] Key** _(Key)_: The path of the registry key to add, modify, or remove. This path must include the registry hive/drive (e.g. HKEY_LOCAL_MACHINE, HKLM:).
* **[String] ValueName** _(Key)_: The name of the registry value. To add or remove a registry key, specify this property as an empty string without specifying ValueType or ValueData. To modify or remove the default value of a registry key, specify this property as an empty string while also specifying ValueType or ValueData.
* **[String] Ensure** _(Write)_: Specifies whether or not the registry key or value should exist. To add or modify a registry key or value, set this property to Present. To remove a registry key or value, set the property to Absent. { *Present* | Absent }.

...'set this property to Absent' for consistency.

README.md, line 365 at r1 (raw file):

* **[String] Ensure** _(Write)_: Specifies whether or not the registry key or value should exist. To add or modify a registry key or value, set this property to Present. To remove a registry key or value, set the property to Absent. { *Present* | Absent }.
* **[String] ValueData** _(Write)_: The data the specified registry key value should have as an array of strings.
* **[String] ValueType** _(Write)_: The type of the specified registry key value should have. { *String* | Binary | DWord | QWord | MultiString | ExpandString }

The type the specified...'  (extra of)

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 176 at r1 (raw file):

        If an empty string ValueName has been specified and no ValueType and no ValueData
        has been specified, treat this case as if ValueName was not specified and target
        the Key itself. This is to cater the limitation that both Key and ValueName

This last sentence is worded a bit weird.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 303 at r1 (raw file):

                    Write-Verbose -Message ($script:localizedData.RemovingRegistryKeyValue -f $valueDisplayName, $Key)

                    # If the specified registry key value exists, check if it is the user specified a registry key value with a name to remove

'check if the user....'

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 555 at r1 (raw file):

<#
    .SYNOPSIS
        Converts the specigied registry drive root to its corresponding registry drive name.

'specified'

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 951 at r1 (raw file):

<#
    .SYNOPSIS
        Creates a new subkey with the specified name under the specfied registry key.

'specified'

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 1287 at r1 (raw file):

<#
    .SYNOPSIS
        Sets the specified registry key value with the specified name to the specigied value.

specified   lol

Comments from Reviewable

Review status: 0 of 16 files reviewed at latest revision, 7 unresolved discussions.

README.md, line 363 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
...'set this property to Absent' for consistency.

Done.

README.md, line 365 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
The type the specified...'  (extra of)

Done.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 176 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
This last sentence is worded a bit weird.

Moved to the ValueName parameter description

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 303 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
'check if the user....'

Done.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 555 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
'specified'

Done.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 951 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
'specified'

Done.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.psm1, line 1287 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
specified   lol

Done.

Comments from Reviewable

Reviewed 14 of 17 files at r1.
Review status: 12 of 16 files reviewed at latest revision, 14 unresolved discussions.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.schema.mof, line 4 at r1 (raw file):

class MSFT_xRegistryResource : OMI_BaseResource
{
  [Key,Description("The path of the registry key to add, modify, or remove. This path must include the registry hive/drive.")] String Key;

can you add spaces after the commas? 'Key, Description...'

DSCResources/MSFT_xRegistryResource/en-US/MSFT_xRegistryResource.strings.psd1, line 26 at r1 (raw file):

    RegistryKeyValueTypeDoesNotMatch = The type of the value {0} under the registry key at path {1} does not match the expected type. Expected {2} but was {3}.
    RegistryKeyValueDoesNotMatch = The value {0} under the registry key at path {1} does not match the expected value. Expected {2} but was {3}.

did you mean to have an extra newline here?

Tests/MSFT_xRegistryResource.TestHelper.psm1, line 346 at r1 (raw file):

<#
    .SYNOPSIS
        Tests if the registry drive of he given registry key path is mounted.

'...drive of the given....'

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 4 at r1 (raw file):

    WARNING: DO NOT RUN THESE TESTS ON A VALUABLE MACHINE!
    Running on a disposable VM or AppVeyor is strongly recommended.
    If these tests go awry, your machine's registry could be corrupted which will brick your machine!

lol brick your machine? Is that what happened to yours?

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 59 at r1 (raw file):

            $registryKey = Get-Item -Path $registryParameters.Key -ErrorAction 'SilentlyContinue'

            It 'Should have created registry key' {

'..the registry key'

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 87 at r1 (raw file):

            $registryKeyValue = Get-ItemProperty -Path $registryParameters.Key -Name $registryParameters.ValueName -ErrorAction 'SilentlyContinue'

            It 'Should have created registry key value' {

the registry

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 214 at r1 (raw file):

            $registryKeyValue = Get-ItemProperty -Path $registryParameters.Key -Name $registryParameters.ValueName -ErrorAction 'SilentlyContinue'

            It 'Should be able to retrieve registry key value' {

the registry

Comments from Reviewable

Review status: 12 of 16 files reviewed at latest revision, 14 unresolved discussions.

DSCResources/MSFT_xRegistryResource/MSFT_xRegistryResource.schema.mof, line 4 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
can you add spaces after the commas? 'Key, Description...'

Done.

DSCResources/MSFT_xRegistryResource/en-US/MSFT_xRegistryResource.strings.psd1, line 26 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
did you mean to have an extra newline here?

Yup. They separate the Get/Test/Set-TargetResource-specific messages and the error messages

Tests/MSFT_xRegistryResource.TestHelper.psm1, line 346 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
'...drive of the given....'

Done.

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 4 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
lol brick your machine? Is that what happened to yours?

Pretty much. I couldn't open ANY programs... I could get to the desktop and that was it.

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 59 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
'..the registry key'

Done.

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 87 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
the registry

Done.

Tests/Integration/MSFT_xRegistryResource.EndToEnd.Tests.ps1, line 214 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
the registry

Done.

Comments from Reviewable

Reviewed 1 of 17 files at r1.
Review status: 11 of 16 files reviewed at latest revision, 14 unresolved discussions.

Comments from Reviewable

Reviewed 1 of 17 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

LGTM


An implementation of the {% comment %} tag including a test

Should address issue #31
Please let me know if you'd like any changes.

You can already use {# comment goes here #}. Why would you need an extra tag for the same thing? Comment docs

Because the {# comment #} syntax doesn't allow for multi-line comments
If the {# comment #} syntax was extended to allow for multi-line comments, it breaks django template syntax highlighting.
It's an incredibly simple tag/test.

Yep, you should be able to get the fix for gh-30 that includes multi-line comments in<EMAIL_ADDRESS>which went up about an hour ago.


How does lightgbm calculate the initial score for multiclass models?

For each tree in the multiclass model, I assumed that lightgbm would start boosting from logit(% of total classes). It appears to work this way for a binary model. However it doesn't look like this is the case for multiclass models:
import lightgbm as lgb
import numpy as np

features = np.random.uniform(0,1,(1000,10))
target = np.random.binomial(1,0.1,1000)
dtrain = lgb.Dataset(features, target)
pars = {"objective": "binary", "verbose": 2}
model = lgb.train(params=pars, train_set=dtrain, num_boost_round=1)

def logit(x):
    return np.log(x/(1-x))

# Initial score is the logit of the average target:
print(logit(target.mean()))

Prints:
[LightGBM] [Info] Number of positive: 102, number of negative: 898
[LightGBM] [Debug] Dataset::GetMultiBinFromAllFeatures: sparse rate 0.000000
[LightGBM] [Debug] init for col-wise cost 0.000018 seconds, init for row-wise cost 0.000295 seconds
[LightGBM] [Warning] Auto-choosing col-wise multi-threading, the overhead of testing was 0.000672 seconds.
You can set `force_col_wise=true` to remove the overhead.
[LightGBM] [Info] Total Bins 2550
[LightGBM] [Info] Number of data points in the train set: 1000, number of used features: 10
[LightGBM] [Info] [binary:BoostFromScore]: pavg=0.102000 -> initscore=-2.175197
[LightGBM] [Info] Start training from score -2.175197
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 10

-2.1751972550179284

As you can see, the initial score is the logit of the average target. However, in the multiclass case:
target = np.random.randint(0,5,1000)
dtrain = lgb.Dataset(features, target)
pars = {"objective": "multiclass", "verbose": 2, "num_classes": 5}
model = lgb.train(params=pars, train_set=dtrain, num_boost_round=1)
class_percent = (np.arange(5).reshape(-1,1) == target).astype("int8").mean(1)

# Not equal to the initial scores printed above:
print(logit(class_percent))

This prints:
[LightGBM] [Debug] Dataset::GetMultiBinFromAllFeatures: sparse rate 0.000000
[LightGBM] [Debug] init for col-wise cost 0.000009 seconds, init for row-wise cost 0.000235 seconds
[LightGBM] [Warning] Auto-choosing col-wise multi-threading, the overhead of testing was 0.000532 seconds.
You can set `force_col_wise=true` to remove the overhead.
[LightGBM] [Info] Total Bins 2550
[LightGBM] [Info] Number of data points in the train set: 1000, number of used features: 10
[LightGBM] [Info] Start training from score -1.682009
[LightGBM] [Info] Start training from score -1.514128
[LightGBM] [Info] Start training from score -1.650260
[LightGBM] [Info] Start training from score -1.551169
[LightGBM] [Info] Start training from score -1.660731
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 8
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 10
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 9
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 12
[LightGBM] [Debug] Trained a tree with leaves = 31 and depth = 11

[-1.47621369 -1.26566637 -1.43706669 -1.31291182 -1.45001018]

In more imbalanced cases, these initial scores can be very significantly different in the multiclass model. How are these calculated?

I initially thought, since multiclass uses the softmax, that the initial scores might be the result of using softmax on the logit(% of total classes) for each class. But that doesn't seem to be the case either. The initial scores according to the lightgbm printout are always lower than the theoretical calculated one, at least so far as I've seen.

Hi @AnotherSamWilson, thank you for your interest in LightGBM. For multi-class objective the init score is the log of the class proportions https://github.com/microsoft/LightGBM/blob/c7102e56b246cc5cd73d9787b2c837c0bc384d1e/src/objective/multiclass_objective.hpp#L155-L157 so in your example these should match the output of np.log(class_percent).
Please let us know if you have further doubts.

Interesting, from my understanding the multiclass procedure creates N binary trees and then takes the softmax of their output. Therefore, this behavior seems strange. For larger class sizes, this will cause the initial score to be very different from the desired final average output:
import numpy as np
def logit(probability):
    odds_ratio = probability/(1-probability)
    log_odds = np.log(odds_ratio)
    return log_odds

class_percent = np.array([0.01, 0.1, 0.89])

np.log(class_percent)
logit(class_percent)

In this case, a class with 89% of the data would have an initial score of -0.1165, when that class should end with an average score around 2.0907 after some boosting rounds. Is this a bug, or desired behavior because of some property I am not familiar with?

The init score is defined as the raw score (before any transformation like softmax) so the actual predictions start on the class proportions, i.e.
softmax(init_score)
  = softmax(log(class_proportions))
  = exp(log(class_proportions)) / sum(exp(log(class_proportions))
  = class_proportions / sum(class_proportions)
  = class_proportions


Multiselect

To enter data in a cell there are several options as checkbox, text, select.... is there a "multi-select" one? The select one allows me to choose just one option from several ones. I need to be able to select some of those options.
Thx

I've no idea about what you said, you mean in cell editing?

yes... At this page:  http://allenfang.github.io/react-bootstrap-table/example.html#celledit
at the Custom Editing Cell Class  part there are tree options: textarea, select, checkbox. Do you have, instead of just  select  a multiselect option?
Thx

With the use of React Select (https://jedwatson.github.io/react-select/) I am trying to incorporate a multiselect option to enter data in the table. The code I am using is:

class MultiSelectEd extends React.Component {
constructor(props) {
super(props);
this.updateData = this.updateData.bind(this);
this.state = { selValues: props.defaultValue.selValues };

this.state = { allValues: props.defaultValue.allValues};

console.log('@@@@@@@@@@@@@@@@  prop allValues = ' + this.props.allValues);
console.log('@@@@@@@@@@@@@@@@ state allValues = ' + this.state.allValues);

}
focus() {
this.refs.inputRef.focus();
}
updateData() {
this.props.onUpdate({ selValues: this.state.selValues });
}
render() {
return (

<Select name = "multiSelect-name-intable" options = {this.props.allValues}  value = {this.state.selValues} multi={true} onChange={(ev) => { console.log('+++++++++++++' + ev);}}/>
{ /* <select
value={ this.state.selValues }
onKeyDown={ this.props.onKeyDown }
onChange={ (ev) => { this.setState({selValues: ev.currentTarget.value }); } } >
{  }
 /
/<button
className='btn btn-info btn-xs textarea-save-btn'
onClick={ this.updateData }>
save
 */}

);
}
}
const options = {
onAddRow: handleOnAddRow
};
const multiSelectEditor = (onUpdate, props) => (<MultiSelectEd onUpdate={onUpdate } {...props}/>);

With this code I've been able to show the control:

but whenever I try to see the content of the list, its options appear behind the table's rows as in:

I am wonder if there is a bug here with the table. I tried z-index but no effects. On the other hand, can you please tell how can I place the selected values in the table at the respective col,row? Your examples are somehow hard to follow to understand the process.

at the Custom Editing Cell Class part there are tree options: textarea, select, checkbox. Do you have, instead of just select a multiselect option?

Currently not yet, PR is welcome

I am wonder if there is a bug here with the table. I tried z-index but no effects

I think so and it was report at before https://github.com/AllenFang/react-bootstrap-table/issues/530

Your examples are somehow hard to follow to understand the process.

which part?

let me know if you still have any questions

Hi Allen,
I m using react-bootstrap-table as I m writing my app in React-TypeScript.
I tried to implement checkbox selecton/multiselection.
Need your help and advice on the following error encountered:
/Users/Darryl/Desktop/project/react/costa/src/components/screens/AnnouncementList.tsx
TypeScript error in /Users/Darryl/Desktop/project/react/costa/src/components/screens/AnnouncementList.tsx(357,102):
No overload matches this call.
Overload 1 of 2, '(props: Readonly): BootstrapTable', gave the following error.
Type '{ mode: string; clickToSelect: boolean; onSelect: (row: { [x: string]: string; }, isSelected: any, e: any) => void; onSelectAll: (isSelected: any, rows: string | any[]) => void; }' is not assignable to type 'SelectRow'.
Types of property 'mode' are incompatible.
Type 'string' is not assignable to type 'SelectRowMode'.
Overload 2 of 2, '(props: BootstrapTableProps, context?: any): BootstrapTable', gave the following error.
Type '{ mode: string; clickToSelect: boolean; onSelect: (row: { [x: string]: string; }, isSelected: any, e: any) => void; onSelectAll: (isSelected: any, rows: string | any[]) => void; }' is not assignable to type 'SelectRow'.  TS2769
355 |                                         </style>
356 |                                         <BootstrapTable data={dataNotBroadcast} striped hover pagination={true} options={options} search={true} multiColumnSearch={true} exportCSV={true} version='4'

357 |                                             headerStyle={{ background: '#9898AD' }} bordered={false} selectRow={ selectRowProp } >
|                                                                                ^
358 |
359 |                                             S/No.
360 |                                             Announcements

Thank you so much!


dependency-check 11.1.0 returns 0 dependencies on a local bitbucket repo

Trying to scan a local repository with the following command:
dependency-check.bat --project "EPG" --nvdApiKey xxx  --scan "C:\Users\xxx\Documents\WORK\Repos\xxx-sps_epg-body-616e81a81488\xxx-sps_epg-body-616e81a81488**" -o C:\Users\xxx\Documents\WORK\Repos\epg.html
OUTPUT:
[INFO] Analysis Started
[INFO] Finished File Name Analyzer (0 seconds)
[INFO] Finished Dependency Merging Analyzer (0 seconds)
[INFO] Finished Hint Analyzer (0 seconds)
[INFO] Finished Version Filter Analyzer (0 seconds)
Nov 20, 2024 10:49:02 A.M. org.apache.lucene.store.MemorySegmentIndexInputProvider
INFO: Using MemorySegmentIndexInput with Java 21 or later; to disable start with -Dorg.apache.lucene.store.MMapDirectory.enableMemorySegments=false
Nov 20, 2024 10:49:02 A.M. org.apache.lucene.internal.vectorization.VectorizationProvider lookup
WARNING: Java vector incubator module is not readable. For optimal vector performance, pass '--add-modules jdk.incubator.vector' to enable Vector API.
[INFO] Created CPE Index (1 seconds)
[INFO] Finished CPE Analyzer (1 seconds)
[INFO] Finished False Positive Analyzer (0 seconds)
[INFO] Finished NVD CVE Analyzer (0 seconds)
[INFO] Finished Sonatype OSS Index Analyzer (0 seconds)
[INFO] Finished Vulnerability Suppression Analyzer (0 seconds)
[INFO] Finished Known Exploited Vulnerability Analyzer (0 seconds)
[INFO] Finished Dependency Bundling Analyzer (0 seconds)
[INFO] Finished Unused Suppression Rule Analyzer (0 seconds)
[INFO] Analysis Complete (1 seconds)
The created document contains no dependencies:
dependency-check version: 11.1.0
Report Generated On: Tue, 19 Nov 2024 15:41:14 GMT
Dependencies Scanned: 0 (0 unique)
Vulnerable Dependencies: 0
Vulnerabilities Found: 0
Vulnerabilities Suppressed: 0

It simply means your folder contains no supported filetypes for identifying dependencies.
Given your folder structure with Repos in it it looks like you're scanning the raw bitbucket repositories storage location rather than a checked-out revision of your projects.
If that's the case I'm not surprised at all, as I can easily guess that bitbucket's backend storage will not use the actual filenames/files, but use more optimal storage files for storing your revisions.

Closing because of no answer from the author in more than 3 weeks.


Update harbor default log level to info

To set the release default log level of adminserver and ui to info instead of debug, this will help to reduce log size and the redundant message.

Coverage remained the same at 64.353% when pulling e4255b7a5641c27e09d0df529101b3dcb2049728 on wy65701436:update-log-level into 04bad55134c8a38f0ba08f92498b0cf659c7783f on vmware:master.


pipenv install error: bad interpreter: No such file or directory

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Issue description
Pipenv suddenly nor working today. When I try to search Stackoverflow, it seems not only me having the problem. I saw two questions:
https://stackoverflow.com/questions/64008062/pipenv-error-bad-interpreter-no-such-file-or-directory
https://stackoverflow.com/questions/64005389/pipenv-install-error-bad-interpreter-no-such-file-or-directory
Expected result
No error
Actual result
Installing pandas…
⠋ Installing...Failed to load paths: /bin/sh: /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93/bin/python: No such file or directory

Output:
⠙ Installing pandas...Failed to load paths: /bin/sh: /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93/bin/python: No such file or directory

Output:
Failed to load paths: /bin/sh: /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93/bin/python: No such file or directory

Output:
Error:  An error occurred while installing pandas!
Error text:
/bin/sh: /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93/bin/pip: /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93/bin/python: bad interpreter: No such file or directory

✘ Installation Failed

Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
If you're on macOS, run the following:
$ pipenv --support | pbcopy

If you're on Windows, run the following:
> pipenv --support | clip

If you're on Linux, run the following:
$ pipenv --support | xclip

Do you use Homebrew? This is a common problem with Python installed from Homebrew. The only way to fix this is to pipenv --rm and rebuild the virtual environment.

@uranusjr Yes I am using Homebrew. But I have also tried pipenv --rm and rebuild by pipenv install. The same problem exists.

Try reinstalling pipenv (from Homebrew if that’s how you installed it initially).

@uranusjr I tried to reinstall by brew reinstall pipenv but the problem still exists.

which -a pipenv: /usr/local/bin/pipenv
which -a python python3:
/usr/local/opt/python/libexec/bin/python
/usr/bin/python
/usr/local/opt/python/libexec/bin/python
/usr/local/bin/python3
/usr/bin/python3
which -a pip pip3
/usr/local/opt/python/libexec/bin/pip
/usr/local/bin/pip
/usr/local/opt/python/libexec/bin/pip
/usr/local/bin/pip3
/usr/bin/pip3

I tried to uninstall python by brew uninstall python and brew uninstall pipenv and pip uninstall pipenv, I still get the same error.

but when I create a brand new folder. It works..but not the old folder.

I found the solution. I need to remove the  /Users/johnchan/.local/share/virtualenvs/src-_ygy3i93 file.
In fact all file starting with src-*

What helped me was using the "system" option.
pipenv install --system

This did it for me:
apt-get -y autoremove --purge pipenv
pip uninstall pipenv # If you have it installed with pip
pip install pipenv

pipenv install --system

Did this and then restarted my system and it worked.


[Question] Is there a way to exclude an API from logging requests?

I tried looking through the code to find where this may be possible but I couldn't find anything. There are also no examples of this.

Forgot to explain it more before hitting send:
I have API endpoints that are used in kubernetes health checks - logging those requests would not be necessary as they are sent roughly 10x per minute. Is there any way to disable request logging for specific endpoints?

there is not a way to do this but definitely doable, PR is welcome btw. current to busy to handle it, I need to take care of my kids when they are at home due to Covid-19 🤣

fixed in version 1.2.5

Thank you!


Defer indexed accesses on generic potential discriminants

The issue was that {id: ID} & ({id: "yes", ...} | {id: "no", ...}) didn't register as an object type indexed accesses needed to be deferred on. Since instantiating ID could discriminate the type upon instantiation, and thus change what types are indexed into, the indexing operation must be deferred, as with other more traditional generic object types.
Fixes #45560

@typescript-bot pack this
@typescript-bot user test this inline
@typescript-bot test this
@typescript-bot run dt
@typescript-bot perf test this

@typescript-bot perf test this

Haven't looked at the code yet, but boy that's a pretty hefty perf hit in material-ui.

Awk, well, I just fixed half of it, hopefully the other half is mostly fixable, too, and just indicative of a lack of caching somewhere.

@weswigham Do you want to come back to fix the slower performance, or should we close this PR?


operations shall be updated to /v1/approve-basic-auth-request for RequestForInquiringOamApprovals and CreateLinkForApprovingOamRequests

In service, /v1/regard-application, we have callbacks RegardApplicationCausesSequenceForInquiringOamRequestApprovals.RequestForInquiringOamApprovals and RegardApplicationCausesSequenceForInquiringOamRequestApprovals.CreateLinkForApprovingOamRequests. In which the attribute "oam-approval-operation" for former and "operation-name" for later is specified to point to aa-2-1-0-op-s-is-004 (/v1/approve-oam-request) .
As per example of that callback and as per the diagram (https://github.com/openBackhaul/AdministratorAdministration/blob/v2.1.1_spec/spec/diagrams/is002_regardApplicationCausesOamSequence.plantuml : line 66) , it shall be pointing to "aa-2-1-0-op-s-is-005" (/v1/approve-basic-auth-request).

Uuids have been corrected as proposed.


Compilation fail

There is a small fix by Brian, but we have to think about long-term fix
https://dockyard.slack.com/archives/C02E1GA5THB/p1702234776493379
bcardarella
1:59 PM
I discovered a strange elixir quirk when I renamed live_view_native_swift_ui to live_view_native_swiftui
2:02
Compilation was failing within live_view_native because LiveViewNative.SwiftUI was uncompiled. It seems changing that app name changed the compilation order of the dependencies. Because live_view_native and live_view_native_swiftui don't depend on one another there is no way to inform the compiler that one must be compiled before the other. The way we'd typically do that is to set live_view_native_swiftui to be a dependency of live_view_native so it will compile live_view_native_swiftui first and LiveViewNative.SwiftUI would be available at the compile-time for live_view_native. But we cannot do that because the live_view_native library really shouldn't be away of clients.
2:02
this block of code: https://github.com/liveview-native/live_view_native/blob/main/lib/live_view_native/platforms.ex#L6-L17
platforms.ex
@env_platforms :live_view_native
|> Application.compile_env(:plugins, [])
|> Enum.flat_map(fn plugin_mod -> apply(plugin_mod, :platforms, []) end)
|> Enum.concat(@default_platforms)
|> Enum.uniq()
Show more
https://github.com/[liveview-native/live_view_native](https://github.com/liveview-native/live_view_native)|liveview-native/live_view_nativeliveview-native/live_view_native | Added by GitHub
bcardarella
2:04 PM
in LVN its doing a compile-time collection of all plugins. I assume this was decided for performance reasons so the plugins don't need to be re-discovered constantly at runtime. However, because of the compilation order change due to the name change this was failing because live_view_native_swiftui was now compiling after live_view_native. I suspect it worked with the previous name due to simple luck of however Elixir decides compilation order of dependencies that are not bound to one another (edited)
15 replies
Last reply 20 hours agoView thread
bcardarella
2:04 PM
The simple fix for now is to move that into run-time and cache the result in ets
2:05
however we'll probably need to re-think this long-term. I hope that a bunch of the code can be refactored out as we get Phoenix and LiveView upstream to work with formats and such. We should be able to simplify the lib quite a bit at that point
2:06
and the plugin system could also benefit from this
2:06
thanks to
@Paulo Valente
for the pairing to figure this out today
2:07
exactly how Elixir is determining order and if that is a bug is unknown, kind of strange the compilation order would change like that due to a removal of a single character
2:07
the alphabetic sort order doesn't change, so order must be determined in some other fashion
bcardarella
2:23 PM
I'm working on a patch that will store and fetch from an ets cache so we don't incur the runtime overhead

According to @carson-katri this has been fixed


Rename CMake project to gz

See https://github.com/ignition-tooling/release-tools/issues/718

may need to replace ignition- with gz- in the following script to fix some tests:

https://github.com/ignitionrobotics/ign-cmake/blob/gz_cmake_project/examples/test_core_child_requires_core_no_deps.bash

may need to replace ignition- with gz- in the following script to fix some tests:

https://github.com/ignitionrobotics/ign-cmake/blob/gz_cmake_project/examples/test_core_child_requires_core_no_deps.bash

done in https://github.com/ignitionrobotics/ign-cmake/pull/244/commits/e54db0df90d24453a9a202798e7931be70ec45c0 and added an extra fix to the pkg-config file name in https://github.com/ignitionrobotics/ign-cmake/pull/244/commits/12be647e6e428af7af5eeee4573f21c7a40e01f5

I just noticed there is an ignition- that should change to gz- on the following line, but there's also a super weird 1.1 that makes no sense to me

https://github.com/ignitionrobotics/ign-cmake/blob/gz_cmake_project/cmake/pkgconfig/ignition-component.pc.in#L7

the 1.1 is also in cmake/pkgconfig/ignition.pc.in

I just noticed there is an ignition- that should change to gz- on the following line, but there's also a super weird 1.1 that makes no sense to me

https://github.com/ignitionrobotics/ign-cmake/blob/gz_cmake_project/cmake/pkgconfig/ignition-component.pc.in#L7

the 1.1 is also in cmake/pkgconfig/ignition.pc.in

some explanation for 1.1 in https://osrf-migration.github.io/ignition-gh-pages/#!/ignitionrobotics/ign-cmake/pull-requests/104/page/1
I think we can remove it

regex replaces removing ignition to no-op

Good point, removed in a477d3b since this PR will go along all downstream project name changes anyway.

Good point, removed in a477d3b since this PR will go along all downstream project name changes anyway.

Ah that broke some test cases. Reverted in 73ef90e. We can address that in follow-up PRs.


add punnycode to encode the slug

try to the punnyCode to encode the slug of url ,but it will not take effect

This fixes https://github.com/gothinkster/aspnetcore-realworld-example-app/issues/66 ?


Kindly help me

I want to write my own randomforest.p4 file for classification on this dataset. I am using three features (Protocol, ipv4.src and ipv4.dst). How should I proceed, I already have trained the model in python.

Hello,
I think to perhaps first try to test and understand switchtree.p4
After that you will have to add these features (feature 13, 14, 15) and modify action CheckFeature and design the rules to be uploaded to the tables.
Note that these 3 features dont need to be stored in registers as they are already present in the header or metadata. Your randomforest.p4 will be much simpler.

Thank you so much for the reply. I am sorry if you find my questions silly. I am very new to p4 programming and Networking in general. Could you please tell me in what sense I have to modify the Checkfeature?
I get your point that these features might not be reliable enough, but right now my main focus is to get a very basic .p4 program. I can always do manipulations in the machine learning side as I am comfortable with that.
Thank you for the quick response sir, I am again very sorry if my questions seem very naïve to you

Could you tell me what kind of values do I need to add in the rules. Like in your rftop4_v2.py file you have created previous feature, previous_id, current_id and classname for SetClass and threshold for Checkfeature.
I think I just need the class name and the threshold. Am I right?

Hello,
You should check the paper: https://hal.archives-ouvertes.fr/hal-02968593/document.
For example section 4.1 describes node id, previous feature id etc.
A rule looks like the following.
Key(node id, dpkts feature id, 1) -> CheckFeature(ct state ttl, 1)
You may just create a new feature id for your features. think of adding rules for them and then add the check in the end of the checkfeature.

Thank you so much @ksingh25. This helped me a lot.


odan/phinx-migrations-generator[5.0.0, ..., 5.3.2] require php ^7.2 -> your php version (8.0.2) does not satisfy

When trying to install latest slim4-skeleton with php 8.0.2 getting following error for dependent composer package. I am also using composer 2.0 to manually install all composer dependencies.
Problem 1
- odan/phinx-migrations-generator[5.0.0, ..., 5.3.2] require php ^7.2 -> your php version (8.0.2) does not satisfy that requirement.
- Root composer.json requires odan/phinx-migrations-generator ^5 -> satisfiable by odan/phinx-migrations-generator[5.0.0, ..., 5.3.2].

This odan/phinx-migrations-generator package has been updated for PHP 8 now. Please try it again.


Allow platform (end) user to save multiple personalized action plans to profile and have access to refer back to these plans as needed

Allow platform (end) user to save multiple personalized action plans to profile and have access to refer back to these plans as needed

Ready for PO review.  However, it appears that the GA is not loaded properly.  Currently displaying "additional reading" resources only

@KellyeDC I was able to save multiple PAPs, but when I save a plan, I get no confirmation or indication it was save. I click on save an nothing a user can see happens. If I click on Share or Print, something happens. We should have some sort of confirmation that the plan was saved or the user is likely to keep clicking on it thinking nothing has happened.

There is a small pop up notification in top upper right, with green back ground that informs the user that the PAP was saved. @grawdon   I doesn't work all of the time.  I will return to DEV.

Done in Sprint 11. Small pop up notification appears.
Move to Ready fore Review

Yes, I was able to save multiple PAPs.


Feature: blit + sd data bundles

This is a proposal to extend the blit format to handle extra game data files which should be installed on the SD card.
Blit files can have extra data appended after #785. The questions then are how should the system handle locating the data and how should it work for users?
Ideally installing a bundle blit should not be any different to a flash-only blit. Installing with either the tools or copying to SD card and running it from the launcher should both work. The tools will need to be extended to refuse to install a bundle if no SD card is available in the target.
The game will need to locate the extra data when running from flash. I can see three ways of doing that:

Store the path to the bundle dynamically in the metadata during install.
Require the user to always launch the blit from the SD card and hide it from the internal flash menu.
Unpack the extra data into the SDK game data path (currently only used for saves).

Of these options, only 3 requires no extra metadata. It also allows compression and directory trees to be unpacked during installation. Option 2 has the benefit that bundles with unavailable data won't show up anywhere, but requires the user to hunt on the SD card for the bundle every time they want to run it.
I propose the following:

Install the blit to flash as normal.
If there is extra data, extract it to the SDK save path eg /<author>/<blitname>/gamedata/.
Add a function to the sdk to open a filename from the gamedata directory without needing to know its path.
Show an error if the gamedata is not available when attempting to launch the blit from the flash menu.

I'm assuming the SDL-side handling of this is basically "copy files to build/install dir". The function to open these could possibly just be another open flag like the "cache" one.
I guess the big part is what format to use for the "bundle" and a tool to make those. Possibly the install command would need to handle unpacking itself if the dest is flash as stream-unpacking does not sound fun... or it could just not allow that and force installs through the SD card as you need it anyway.

:thinking: Some of this could probably be reused to help with the lua/interpreted games packaging thing.

Hmm... I didn't consider SDL. I suppose there the data path would just be alongside the executable in the output zip, as you say.

I love this as a concept and totally see where it would be useful - ignoring the implementation details your proposal as laid out seems pretty reasonable.
Over on the RP2040/Pico side I accomplished... something?... like this by packing up a manifest of files into a LittleFS filesystem that gets appended onto the firmware .uf2 (https://github.com/Gadgetoid/dir2uf2/). Yeah it's not the same but... files... in places.

I wonder if there are any existing, simple filesystems that would be amenable to this kind of tomfoolery without being zero-padded to oblivion.

What about EMScripten/Web Assembly?
Is compression even really required?, I guess it could be optional
RP2040 ports technically kinda sorta have (4MB iirc) storage, even if it isn't an SD card
However we accomplish this it should probably also work pretty much exactly the same way for files appended to .blits that are small enough to just get wholesale flashed to XiP
Something something updates

Oh, meant to look into this more, but uh guess what happened :laughing:

Emscripten has it's own embedded file thing, probably reusable in some way for this.
If we're also allowing just appending files that don't get extracted, then I'd want those to not be compressed (as this is what I'm doing with my messy scripts for daftboy/doom)
Some of the RP2040 ports might get SD card support if I ever finish the code... though there isn't a launcher over there anyway (also on the list...)

Regarding the format, MicroPython have a simple, linear filesystem that can be memory mapped in the works here - https://github.com/micropython/micropython/pull/8381
They also have a mpremote deploy-mapfs <directory> which packs up a local directory into this format.
Short of potentially adding optional zlib compression on top of this, this might be more or less what we want?

Definitely a simple format. Looks like
MF

uint16 name len + dir flag
uint32 data len
name
data

... more files

null terminator

vs my
APPFILES
uint32 num files

uint16 name len
uint32 data len
[... for each file]

name
data
[... for each file]

... except mine doesn't represent dirs.
The only problem with compression is that it would be the firmware doing the decompression... so not much space.


Bug: defineExpose does not work with functions

Describe the bug

When using script setup + defineExpose, exposed functions are not found within VTU .vm
To Reproduce

Modify ScriptSetup_Expose.vue component as follows:
<script setup lang="ts">
// imported components are also directly usable in template
import { ref } from 'vue'
import Hello from './Hello.vue'

const count = ref(0)
const inc = () => {
  count.value++
}
const resetCount = () => {
  count.value = 0
}

defineExpose({
  count,
  resetCount
})
</script>

<template>
  <button @click="inc">{{ count }}</button>
  <Hello />
</template>

Within expose.spec.ts append to the test:
it('access vm with <script setup> even without defineExpose()', async () => {
    const wrapper = mount(ScriptSetup)

    await wrapper.find('button').trigger('click')
    expect(wrapper.html()).toContain('1')
    // can access `count` even if it is _not_ exposed
    // @ts-ignore we need better types here, see https://github.com/vuejs/test-utils/issues/972
    expect(wrapper.vm.count).toBe(1)

    wrapper.vm.resetCount()

    expect(wrapper.vm.count).toBe(0)
  })

Expected behavior

The test should pass, instead we get
 FAIL  tests/expose.spec.ts > expose > access vm with <script setup> even without defineExpose()
TypeError: wrapper.vm.resetCount is not a function
 ❯ tests/expose.spec.ts:54:15
     52|     expect(wrapper.vm.count).toBe(1)
     53|
     54|     wrapper.vm.resetCount()
       |               ^
     55|
     56|     expect(wrapper.vm.count).toBe(0)

Test Files  1 failed | 51 passed (52)
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

     Tests  1 failed | 403 passed | 1 todo (405)
  Start at  14:13:35
  Duration  7.76s

Related information:

@vue/test-utils version: 2.x.x
Vue version: 3.x.x
node version:
npm (or yarn) version: Ran with pnpm on top of main branch

Additional context

SFC of component shows correct binding of function to expose call https://sfc.vuejs.org/#eNptUcFuwjAM/RUrF0BAwxkVtGmatI/IpQSXhaVOlKRsU9V/n9PQscNufvbzs/08iGfvq1uPYi/qqIPxCSKm3oNt6HJQIkUljoqkBNN5FxKeQTuOCClFaAJCY6ODswmok/2GPjYni2AIEnbeNgkVlU4YIGALI7TBdbDgmYvf0hta6+6FSk4oL8UERdpRTDy0pwSHLLHcreasIc255QoORxgUQaFVt8b2uF4rGmdiQL7q5a7xP58Lu6lD0RlbQ/j65V3E5YO3ydFDicm8SC2LbWwSg/loRgD1qU/JETxpa/QHm8nrspvDcL9mHGtZKIVeXJAMavlHSGxEsWnbNb66Rkf8rGkrVpwK/KN9uSfn2LiMlXhPyce9lLHV2c1rrFy4SI6qwONNhxXGbnsK7jNiYGElsgQ7MIrxBzIhu4w=

Hey@marina-mosti
Thanks for the repro. I'm off work right now, so I'll take a deeper look in a few weeks, but that does not surprise me: we "cheat" to have access to the functions if they are not exposed, by using the internal proxy of what is available in the template. As resetCount is not used in the template, it is not available on the VM object.
Can you check that's what I suspect? If resetCount is used in the template, the test should succeed.

Hey @cexbrayat thanks for the reply, sadly no - it still fails the same way.
<script setup lang="ts">
// imported components are also directly usable in template
import { ref } from 'vue'
import Hello from './Hello.vue'

const count = ref(0)
const inc = () => {
  count.value++
}
const resetCount = () => {
  count.value = 0
}

defineExpose({
  count,
  resetCount
})
</script>

<template>
  <button @click="inc" @hover="resetCount">{{ count }}</button>
  <Hello />
</template>

Hmm interesting, I'll need to take a deeper look then (probably not until one or two weeks, so if someone else is up to it, feel free)

I verified this does work in production - Test Utils only bug, I guess.

@marina-mosti I have added your reproduction code to the tests and run it successfully #1735. I think you updated ScriptSetup_Expose.vue but used ScriptSetup.vue within your test 😉

🤦🏻♀  Uff. Terribly sorry

Awesome, thanks @freakzlike for looking into it ❤

Turns out after tons of trial and error that the actual problem was that the exposed function was named do
defineExpose({
  count,
  do: _do
})

Changing the name makes it be correctly exposed again


Migration to decompress node data

:construction: WIP, don't merge yet.
@marcparadise does this look plausible as a starting point for decompressing unnecessarily compressed data in postgres in chef-server? I tested it locally via
$ dvm load chef-mover --force && dvm start chef-mover
erlang> mover_manager:migrate(all, 1, mover_uncompress_serialized_objects_callback).

and it seems to work fine, and work idempotently at that, since chef_db_compression:uncompress checks for the gzip magic number and does nothing if it's not present. Moreover because of that the frontend works fine without any further changes, independently of whether the underlying database items are compressed or not. e.g. I can run this migration, and chef-manage still happily lists node attributes. The questions I want to resolve before expecting substantive review are:

Is it important to hunt down all the other database columns that are compressed just like serialized_object in table node? There aren't that many of them, are there?
Is this compression noted in some schema file (formal or informal) that I should change? Or does the schema only say bytea and leave it at that?
Should I be batching this change org-by-org? I probably should, yeah? I could use some more information about how that ought to be done maximally idiomatically inside chef-mover

For the sake of not being redundant, the node SELECT might itself want to check for gzip magic number via a WHERE NOT (get_byte(serialized_object, 0) = 31 AND get_byte(serialized_object, 1) = 139) in case the migration gets interrupted, but as noted above, I think this is only a matter of performance, not correctness.
Also, it's worth noting that although I am putting up this PR first, it seems to me that the logically prior thing to do is to disable, in chef_db_compression itself, any compression of data going forward. I don't know of any downsides to having a database in a mixed state where old rows are gzipped and new rows aren't.

Is this compression noted in some schema file (formal or informal) that I should change? Or does the schema only say bytea and leave it at that?

I don't think we have any schema to change, but we might want to reach out to CSE and OPS about any maintenance or reporting scripts that might need to be changed either for Hosted or our largest customers.

Should I be batching this change org-by-org? I probably should, yeah? I could use some more information about how that ought to be done maximally idiomatically inside chef-mover

If we change erchef to disable inbound compression first, then we can get away with not doing it org by org I think.
If we don't change erchef first, we'll end up in a situation where we have a mix of compressed and uncompressed nodes even after the migration because of in-flight chef-client runs.  Mover theoretically already knows how to do this using the "mover_org_migrator_sup" rather than the transient worker.  However, I think that our per-org tracking is broken since the couchdb->sql migration for orgs, so we might need to fix it up in mover.

@stevendanna does https://github.com/chef/chef-server/pull/427 seem reasonable as a "disable inbound compression" step? Can this PR be merged as-is if I put some kind of "danger danger don't actually run this migration unless you know what you're doing" comment in the header?

Sorry @jcreedcmu , I didn't see this PR - I will take a look more detailed through on Monday morning.

@jcreedcmu I did some more research on the schema and left a comment on #427.  Namely, we might need to change the storage types.

@jcreedcmu Thanks for doing these spikes.  I'm sorry thy never moved foward.  I'm going to close the PRs but keep the branches around in case someone wants to pick them up.  I still think both of these are pretty worthwhile.


[Suggested Feature] Tab groups

Grouping tabs, as in Vivaldi or before Firefox 45, was a great way of organizing tabs for those with a lot of tabs. Integrating tab grouping with Overview offers better tab management for users. The option to automatically group tabs based on parent site (say, through a right-click option) would also be a welcome addition.
Related issue: #954, #955, #956, #957

Thanks for this suggestion. Our telemetry for tab groups in Firefox showed that it was used (and loved) by many users (including me) but the majority of people didn't get it. And it encouraged people to forget about tabs and end up with a huge working set.
We're thinking about a way to archive tabs and create tab collections in Tofino, and hope that will answer the tab-management problem, but I don't think this will end up looking like Tab Groups from old Firefox. So I'm going to close this bug.
When we have some mock-ups to share I'll try to remember to make them public somewhere.


Apply swizzle to user supplied mipmaps

The recent changes to allow user supplied mipmaps is very useful. However, I noticed that the swizzle settings were not being applied to the custom mipmap data.

Thanks!


LWC integration

Pull request checklist

[x] Did you bump the project version?
[x] Did you add a description to the Pull Request?
[ ] Did you add new tests?
[x] Did you add reviewers?

@artememelin, quite dirty


Configuration Issue

Hello.
I generated a WebKit binding, but I had to do manual edits on the generated code, which should not be the case.
I uploaded the configuration file that I used. Here they are:
https://github.com/antoyo/webkit2-rs/blob/master/gir/Gir_WebKit.toml
https://github.com/antoyo/webkit2-sys-rs/blob/master/conf/gir-webkit2.toml
https://github.com/antoyo/webkit2-sys-rs/tree/master/gir-files
Is there anything wrong in the configuration that cause this code generation issue?
Thanks for your help.

The generator ignores classes by default so if there are external super-classes you need to include them in the config. For instance this hierarchy
    GObject
    ╰── GInitiallyUnowned
        ╰── GtkWidget
            ╰── GtkContainer
                ╰── WebKitWebViewBase
                    ╰── WebKitWebView

means adding
manual = [
    Gtk.Container,
    Gtk.Widget,
    GObject.Object,
]

(InitiallyUnowned we ignore). I'd also add GLib.Error used for error handling.

A GTK+ widget crate should have generate_safety_asserts = true option set, which requires a bit of manual work. Perhaps we should export these macros.

Editing generated files may make regenerating unnecessarily difficult and error-prone. What we do here is take advantage of multiple inherent impl sections being allowed (like this) or in the trait case making a second trait. Thus the files in mod auto are just a function of the config, gir-files, and gir revision; they can always be deleted and recreated safely.

Thanks for your answer.
I still have some manual edits to do, though.
Do you know why there are these use u64 (that triggers an unresolved import)?
There are also some missing feature attributes.
So, what do I need to do with the safety asserts since the macro are not exported?
Will you update the code in the gtk crate to export them?
Thanks.

Do you know why there are these use u64 (that triggers an unresolved import)?

Looks like a bug in gir, perhaps it doesn't know u64 doesn't need to be imported.

There are also some missing feature attributes.

Looks like the signals which reference NavigationAction don't carry appropriate version annotations and need that version supplied in the config. Something like
[[object]]
name = "WebKit.WebView"
status = "generate"
    [[object.signal]]
    name = "create"
    version = "2.6"

The import should become conditional then.

So, what do I need to do with the safety asserts since the macro are not exported? Will you update the code in the gtk crate to export them?

Let's start by copying the macros (I believe all functions they use are public) to your crate and see how it goes.

Thanks for your quick answer!
I added the following (and remove WebKit2.WebView in the generate array):
[[object]]
name = "WebKit2.WebView"
status = "generate"
    [[object.signal]]
    name = "create"
    version = "2.6"

(with the missing 2 after WebKit)
But, the feature attribute is still missing.
Am I using the right syntax?
Moreover, the rt module is private, so I cannot use it for the assert_initialized_main_thread macro.

Seems no min_cfg_version = "2.0" (or other sane value) in options, see https://github.com/gtk-rs/gtk/blob/master/Gir.toml

Moreover, the rt module is private, so I cannot use it for the assert_initialized_main_thread macro.

Public items from rt get reexported at the top level.

About version: seems min_cfg_version don't help, as config info unused :cry:
Compare https://github.com/gtk-rs/gir/blob/master/src/analysis/signals.rs#L48 and
https://github.com/gtk-rs/gir/blob/master/src/analysis/functions.rs#L70-L71

Thanks to both of you!
I have macros working and the attributes are not generated.
Now, with this pull request, I believe the code generation for webkit will not require manual edits.

With the previously mentionned pull request, I do not need any manual edits for webkit.
I added the macros here.
Could you export them so that I don't need to copy them (that will probably needs some updates to the macros themselves so that they work in another crate)?
Besides that, I think this issue is fixed.
Thanks.

I think I found another bug.
I added the missing enums to the config file and I had to do these manual edits.
Thanks.

Maybe something wrong with you WebKit2-4.0.gir xml.
This string go almost directly from glib:error-domain
    <enumeration name="FileChooserError"
                 glib:type-name="GtkFileChooserError"
                 glib:get-type="gtk_file_chooser_error_get_type"
                 c:type="GtkFileChooserError"
                 glib:error-domain="gtk-file-chooser-error-quark">

Thanks @EPashkin!
I've fixed the strings in the gir file.

@antoyo, now you can remove manual changes from WebKit2-4.0.gir


always crash

New Issue Checklist

[ ] I have read and understood the CONTRIBUTING guide
[ ] I have read the Documentation
[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios / tvos

Platform Version
e.g. 8.0

SDWebImage Version
e.g. 3.7.6

Integration Method
e.g. carthage / cocoapods / manually

Xcode Version
e.g. Xcode 7.3

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.
I'm showing images on UICollectionView. I'm getting memory warning and app crashes after several run.
i use leaks , find  decodeImageWithImage -> 238M .
help , thanks!

@bianzhifeng looks like this is the same issue with #586 and #538. Please take a look at those. Bottom line: decoding images is memory consuming, so if you are dealing with large images, you can use SDWebImageScaleDownLargeImage option. If your images are not that big, one option (https://github.com/rs/SDWebImage/issues/586#issuecomment-215059553) is to stop decompressing in the background
[[SDImageCache sharedImageCache] setShouldDecompressImages:NO];
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];

Please try using our latest version.
PS: Please fill in the info in the table above - so we know which version are you using and so on.

now, i update latest version 4.1.2.
i can't find setShouldDecompressImages in SDImageCache.

i will try this:
SDWebImageDownloader.shared().shouldDecompressImages = false
SDImageCache.shared().config.shouldDecompressImages = false
Wish me luck.

@bianzhifeng sorry, for this version the configs from SDImageCache were moved to a separate object.
So [SDImageCache sharedImageCache].config. shouldDecompressImages = NO;

Yeap, your code is correct:
SDWebImageDownloader.shared().shouldDecompressImages = false
SDImageCache.shared().config.shouldDecompressImages = false

yes  i find , thank you very mach.
now  i will use :
SDWebImageDownloader.shared().downloadImage(with: URL(string: ""), options:        SDWebImageDownloaderOptions.scaleDownLargeImages, progress: nil) { [weak self] (downImg,
data, error, finished) in
self?.CollectionView?.reloadItems(at: [indexPath])
}
and give up
sd_setImage(with: URL(string: ""), placeholderImage: UIImage(named: ""))


Documented setup of of OpenTelemetryTracingProvider breaks apps that write to those same URL's directly via https

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
This is a bit of a long one, buckle in. We are currently on Istio 1.19.7 and trying to migrate to 1.22.5. We also use Dynatrace, which as of Istio 1.22, uses OpenTelemetry instead of OpenTracing for its traces/spans, due to OpenTracing being removed in Envoy 1.30, which Istio 1.22 uses.
Dynatrace docs mimic the Istio docs pretty much, and say to use the below block in the IstioOperator spec to configure the sidecars to send OTEL traces to their platform.
meshConfig:
      extensionProviders:
      - name: dynatrace-otel
        opentelemetry:
          port: 443
          service: "istio-config/${dynatraceTenantId}.live.dynatrace.com"
          http:
            path: "/api/v2/otlp/v1/traces"
            timeout: 10s
            headers:
            - name: "Authorization"
              value: "Api-Token ${apiToken}"
          resource_detectors:
            dynatrace: {}
          dynatrace_sampler:
            tenant: ${dynatraceTenantId}
            cluster_id: ${dynatraceClusterId}

In addition, we are advised to deploy this DestinationRule:
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dynatrace-dr-otel
  namespace: istio-config
spec:
  host: "akpxxxxx.live.dynatrace.com"
  trafficPolicy:
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE

These steps seem to mimic the docs here. Presumably, this is because the OTEL piece of Envoy only sends to HTTP, and we need a DestinationRule to upgrade it to TLS/HTTPS. Otherwise, it wouldn't work since the Dynatrace SaaS URL doesn't accept HTTP traffic obviously.
Now, this is all fine and dandy except after rolling this out to a nonprod cluster as a test, it broke all the apps that, in their own code, write to the same URL custom metrics but over https already. Presumably, this is because Istio is trying to originate an already HTTPS connection. This is what we get when trying to do a curl -v https://apxxxx.live.dynatrace.com
[2024-10-01T18:26:32.694Z] "GET / HTTP/1.1" 503 URX,UF "TLS_error:|268435703:SSL_routines:OPENSSL_internal:WRONG_VERSION_NUMBER:TLS_error_end:TLS_error_end" 0 248 118 - "-" "curl/8.5.0" "8eab814f-fade-964b-9a1d-589c94107e55" "akp68187.live.dynatrace.com" "<IP_ADDRESS>:80" outbound|80||akp68187.live.dynatrace.com - <IP_ADDRESS>:80 <IP_ADDRESS>:33358 - default -
So basically, it seems like the documented setup for configuring an OpenTelemetryTracingProvider inherently breaks any apps that are ALSO trying to write to that same URL over HTTPS. If we remove the DestinationRule above, it goes back to working.
So my question is; why can't the OpenTelemetryTracingProvider section be configured to write to an HTTPS endpoint directly, which is the same thing all our apps do and wouldn't require upgrading of the connection at all? Or, is there a way to configure Istio to allow both HTTPS passthrough AND http>https upgrades, which would allow both the app to work (writing custom metrics) AND the sidecar to work (sending OTEL traces configured via the IstioOperator)? We've tried everything and can't get it to work.
Tagging @joaopgrassi since he was involved in the other issue over on the Envoy GitHub.
Version
$ istioctl version
client version: 1.23.0
istiod version: 1.22.4
data plane version: 1.22.4 (88 proxies)

$ kubectl version
Client Version: v1.30.3
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.12-eks-a18cd3a

Additional Information
No response

Hi @jbilliau-rcd
Did you get the ServiceEntry snippets from Dynatrace or from the Istio docs? From our Istio deployment page, we have since Dynatrace version 1.299 changed the ServiceEntry and DestinationRule snippets. Please take a look there, but the content should be this:
MeshConfig:
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  meshConfig:
    extensionProviders:
    - name: dynatrace-otel
      opentelemetry:
        port: 80
        service: "istio-system/akpxxxxx.live.dynatrace.com"
        http:
          path: "/api/v2/otlp/v1/traces"
          timeout: 10s
          headers:
            - name: "Authorization"
              value: "Api-Token <API_TOKEN>"
        resource_detectors:
          dynatrace: {}
        dynatrace_sampler:
          tenant: "<TENANT>"
          cluster_id: <ID>

ServiceEntry/DestinationRule:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: dynatrace-se-otel
spec:
  hosts:
  - akpxxxxx.live.dynatrace.com
  ports:
  - number: 80
    name: http-port
    protocol: HTTP
    targetPort: 443
  - number: 443
    name: https-port
    protocol: HTTPS
  resolution: DNS
  location: MESH_EXTERNAL
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: dynatrace-dr-otel
spec:
  host: akpxxxxx.live.dynatrace.com
  trafficPolicy:
    portLevelSettings:
    - port:
        number: 80
      tls:
        mode: SIMPLE

Trying that in a pod and trying to invoke the custom metrics endpoint works for me:
curl -X 'POST' \
  'https://akpxxxxx.live.dynatrace.com/api/v2/metrics/ingest' \
  -H 'accept: application/json; charset=utf-8' \
  -H 'Content-Type: text/plain; charset=utf-8' \
  -d 'server.cpu.temperature,cpu.id=0 42'

Returns {"error":{"code":401,"message":"Missing authorization parameter."}} because the request is not authenticated.
Envoy needs the TLS configurations in order to be able to use HTTPS. If you provide the HTTPS URL without the TLS config, the export will fail (you can try it yourself). This new config then configures envoy to export to HTTP, but then Istio upgrades the request to HTTPS.

@joaopgrassi to the rescue again, I used your objects and it works now....it looks like we were. missing location and targetPort on our ServiceEntry as well as using port: 443 on the istiooperator object. Outdated docs :(
This Istio 1.22 migration to OTEL is going to be the end of me, lol.....it's been nothing but issues and frustration. We are going to look into filtering the OTLP HTTP exporter received a non-success status code: 503 while exporting the OTLP message from going to Splunk for now and hope that Istio releases a patch release soon with the back ported Envoy fixes.
Thanks! Closing issue.

We are going to look into filtering the OTLP HTTP exporter received a non-success status code: 503 while exporting the OTLP message from going to Splunk for now and hope that Istio releases a patch release soon with the back ported Envoy fixes.

I guess you figured it out, but just for completion. If you increase the export interval as I mentioned here, then the requests will be less often, preventing Dynatrace from throttling the requests and then no more logs in Envoy with 503s.

create that configmap and that annotation on our thousands of deployments

Hum I'm not a k8s expert, but maybe there's a way to do this by default like "to all pods in a namesmace"? Then you wouldn't need to change the manifest of all deployments.

@jbilliau-rcd  Actually I think I found another way out for you 😅. We can configure a set of default runtime values via the MeshConfig https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/#ProxyConfig.
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  meshConfig:
    defaultConfig:
      runtimeValues:
        tracing.opentelemetry.flush_interval_ms: "300000" <<<<< ADD HERE

Those land in the same place, as a static layered config in Envoy:
      - name: global config
        static_layer:
          envoy.deprecated_features:envoy.config.listener.v3.Listener.hidden_envoy_deprecated_use_original_dst: true
          envoy.reloadable_features.http_reject_path_with_fragment: false
          overload.global_downstream_max_connections: "2147483647"
          re2.max_program_size.error_level: "32768"
          tracing.opentelemetry.flush_interval_ms: "300000"

So you get the same result, without having to run a collector. Just tried and it worked.

NOW WE TALKING! We trying this right now....at 30000ms, our 503's still exist but have dropped by around half (note the green bars showing log volume)

We can try 60000ms and see what that does. Is there any downside to increasing this interval? Would the log spans just pile up for 60s instead of 5s? Would that have any negative impact on the performance or memory footprint of envoy, holding on to that many spans?

With a longer timeout export, more spans will be buffered and that of course has a memory impact. About how much is hard to tell, it will depend on the load of actual requests that general actual spans in Envoy.
I guess you can try bumping to 5 min interval and monitor the proxy health. I think it should be okay though. This then should reduce even further the errors for you.

Sounds good, we going to try this and see what the impact will be. It will end up being this solution or a separate OTEL collector, haven't decided yet. Thanks again for all the help!


Authentication Microservice POC

This PR creates a new Spring Microservice focused on authentication via Okta and proxying requests to the right service.
Test Steps:

Run unit tests in auth project to ensure all is working locally
Ask me for staging secret for okta and set to proper environment variable locally (I'll help)
Ask Arnej for script to retrieve valid auth token, use it to get auth token
Startup this service and the submissions service
Call http://localhost:9000/submissions/health
Ensure you get a 200 response with the body "up"

Changes

New microservice
Additional code added in submissions for future handling of authorization

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x] Added tests?

Linked Issues

Fixes #15623

all in all this is awesome work! nice job
couple of quick things

whatever the directory structure for the project is should probably be the directory structure for all of the services (or at least close to it). Might want to think now about where you are going to want to put your repositories, whether or not you want "model" to be the package name for both json entities and db DAOs. Not a big deal rn. Something to think on given this is the first and therefore the reference implementation.

if we're going to be using webflux we should document the pattern and ensure we don't mix reactive and non-reactive code.

one more thing!
there's a plugin we can use to auto-generate swagger doc
https://springdoc.org/#gradle-plugin
It auto generates both the openapi file and creates an endpoint to view the API contract

whatever the directory structure for the project is should probably be the directory structure for all of the services (or at least close to it). Might want to think now about where you are going to want to put your repositories, whether or not you want "model" to be the package name for both json entities and db DAOs. Not a big deal rn. Something to think on given this is the first and therefore the reference implementation.

I think this is good enough for now. The current idea is that this will not be hooked up to any db or datastore so thats not something we'd have to worry about.


Where it creates the log file in the Pcl Project for android and ios?(path/folder)

I just called in the app.cs
CrossSimpleLogger.Current.Info("Application Started...");
Will this create a  log file in the project?
Any help @jirimatejka
Thanks in advance!

Yes, it creates a new log file but not in the VS project of course but on the device storage.
On IOS and Android it is a folder returned by
System.Environment.GetFolderPath(System.Environment.SpecialFolder.ApplicationData);
and on UWP it is:
Windows.Storage.ApplicationData.Current.LocalFolder;
You can get the log content by calling
CrossSimpleLogger.Current.GetAllLogContent()
method.
regards
Jiri


Regenerate package-lock.json

I'm gonna keep this on hold until GitHub/npm is updated to include the node-https-proxy-agent 2.2.3 fix

added 1376 packages from 1058 contributors and audited 13766 packages in 86.026s
found 0 vulnerabilities


Captcha is removed and also 20 searches per login will this new change effect the search currently using your method ?

No. There are close to 1000 bookings since the removal of captcha.
I'm monitoring it to see if there are any issues and there have been none so far. @smartsubbi


[Feature Request] Request support for esp32

⚡ Feature Request
Please Describe The Problem To Be Solved
Hello, I'm unable to compile this for Esp32 due to missing libs like span in esp32 g++. Could you please add the support for Esp32.
If there are multiple solutions, please present each one separately. Save comparisons for the very end.)

No. You may do it yourself or you can book a private consulting engagement.


PEP 621: Migrate from setup.py to pyproject.toml

How
Migrate setup.py to setup.cfg using setuptools-py2cfg plus manual modifications.  Use setup-cfg-fmt to format the results.
Migrate settings from setup.cfg into pyproject.toml using ini2toml to do the file conversion and running pyproject-fmt and then validate-pyproject to validate the results.
Why
https://setuptools.pypa.io/en/latest/userguide/quickstart.html#basic-use

We recommend users expose as much as possible configuration in a more declarative way via the pyproject.toml or setup.cfg, and keep the setup.py minimal with only the dynamic parts (or even omit it completely if applicable).

From validate-pyproject README:

With the approval of PEP 517 and PEP 518, the Python community shifted towards a strong focus on standardization for packaging software, which allows more freedom when choosing tools during development and make sure packages created using different technologies can interoperate without the need for custom installation procedures.

This shift became even more clear when PEP 621 was also approved, as a standardized way of specifying project metadata and dependencies.

validate-pyproject was born in this context, with the mission of validating pyproject.toml files, and making sure they are compliant with the standards and PEPs.

@frankwiles Ill let you take a peek at this one as you're likely using the setup.py to push to pypi. This is the first i've seen of this new config stuff.


Switch Mac GHA runner to latest available Intel system

Open PR to update the Mac GHA runner. I'll try to collapse the commits into something more reasonable once I get everything working, I've previously made messes in the Git logs in trying to debug GHA scripts...
Status

[ ] Ready for merge

Eventually, the official MOPAC distribution on Mac will switch from Intel to ARM processors. Before that, I will probably switch all MOPAC distributions from Intel compilers to GNU or LLVM compilers. However, a major reason that I am holding off on this change for as long as possible is that the MKL symmetric eigensolver has an appreciable performance advantage (nearly 2x) over the reference LAPACK implementation used by OpenBLAS. The symmetric eigensolver is one of the main bottlenecks of MOPAC, so moving away from MKL will have an immediate performance impact if future changes to reference LAPACK, OpenBLAS, or some other vendor-specific version of LAPACK do not replicate the (still-proprietary) performance gains of MKL's symmetric eigensolver. Unfortunately, I'm not optimistic about this, and MOPAC will probably suffer a permanent performance drop when it is eventually forced away from using MKL.


chore: release 1.8.2

Manually constructed as I couldn't get release-please to appear.
I kept the release-please format. See https://github.com/googleapis/google-api-python-client/pull/868/files

Release is at https://github.com/googleapis/google-api-python-client/releases/tag/v1.8.2

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


fix(tests): We change IntegrationTest annotation target from TYPE to …

…ANNOTATION_TYPE
Closes #1949
What this PR changes/adds
This pr change the target of IntegrationTest annotation to block its direct use on test classes.
Also add a MinikubeIntegrationTest annotaton for End to End integration testing based on Minikube.
Then change the minikube-test.sh file to support the MinikubeIntegrationTest annotation.
Why it does that
Further notes
Linked Issue(s)
Closes #1949
Checklist

[ ] added appropriate tests?
[ ] performed Checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] assigned appropriate label? (exclude from changelog with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Close as it was addressed on upstream


[Bug] Random fuel values

many items have the fuel value of a lava bucket

identified and fixed in Blue Skies


SYSTEM ERROR : No server states have been detected. Server responses are likely empty

Hello, the below situation happened when I follow the Tutorial - Fuzzing Live555 media streaming server. Should I  how to solve this?
[*] No -t option specified, so I'll use exec timeout of 1000 ms.
[+] All set and ready to roll!

[-]  SYSTEM ERROR : No server states have been detected. Server responses are likely empty!
    Stop location : main(), afl-fuzz.c:9160
       OS message : Operation already in progress

Succeed after deleting parameter -E.

I meet the  same problem.And I try to use your approach that delete -E and it seems work.Thank you very much.


How to import an existing site?

I began yesterday to use wocker, and I'd to use an existing site on it.
How can I import the sql code to the database, and btw where is the db located. Is phpmyAdmin installed too?
Thanks for the answer
Hubert

Thanks for using Wocker.
The simplest way to import databases is using WP-CLI.
Here's an example:
1. Connect to the guest machine via SSH
$ vagrant ssh

2. Run a new container named yourproject
core@wocker ~ $ wocker run --name yourporject

4 Copy or move your MySQL file to yourproject directory in your local machine
The MySQL file will be synced into the virtual machine and yourproject container.
3. Enter the running container
core@wocker ~ $ wocker exec -it yourproject bash

4. Import your database (e.g. yourproject.sql ) using WP-CLI command
root@****:/var/www/wordpress# wp --allow-root db import yourproject.sql

Note: the --allow-root option is required.
More information about WP-CLI: http://wp-cli.org/
These issues may also help you:

Accessing the database from host OS #6
Sharing DBs across machines #7

Thank you, I'll give it a try


amazonec2 login to centos ami

Not obvious to me, but I'm trying to login into a centos based Amazon AMI.   The username is "root" and not "ubuntu"
Maybe add this flag "--amazonec2-username=root"
script example
docker-machine create --driver amazonec2  --amazonec2-access-key=AXXY --amazonec2-secret-key=Uxdfsc --amazonec2-vpc-id=vpc-123 --amazonec2-subnet-id=subnet-123 --amazonec2-security-group=vpc-wide-open-for-testing-only --amazonec2-ami=ami-centosbased testcentos

I'd expect --ssh-user as a flag I guess.

If we had that flag it maybe would have worked with previous versions of machine, but with the current master branch of docker-machine we only support Ubuntu and boot2docker as OSes.  We will gradually be expanding our reach into other OSes, though (just did a big refactor to support this), so stay tuned.
cc @ehazlett probably want to close this.

@joshuacalloway yeah as @nathanleclaire mentioned we currently only support Ubuntu so this would not have worked even with the login.  We will be expanding support for more base operating systems soon.  Thanks for the feedback!


pkg:instrumentation-graphql ignoreTrivialResolveSpans doesn't work

What version of OpenTelemetry are you using?
     "@opentelemetry/api": "^1.4.1"

What version of Node are you using?
v18.17.1

What did you do?
I enabled the auto instrumentation with ignoreTrivialResolveSpans option enabled
new GraphQLInstrumentation({
  ignoreTrivialResolveSpans: true
})

Expected:
What did you expect to see?
The resolves that don't have any custom implementation don't generate any span
What did you see instead?
All resolves generate a corresponding span
Additional context
Digging into Apollo's implementation, I found that when it loads its internal plugins, it always wraps the field.resolve with a function that executes the default or the user-defined resolver, as you can see here:
https://github.com/apollographql/apollo-server/blob/f6e3ae021417c3b54200f8d3fcf4366dc3518998/packages/server/src/utils/schemaInstrumentation.ts#L57-L101
For that reason, this condition:
https://github.com/open-telemetry/opentelemetry-js-contrib/blob/deb9aa441dc7d2b0fd5ec11b41c934a1e93134fd/plugins/node/opentelemetry-instrumentation-graphql/src/utils.ts#L327-L329
evaluates true for all resolvers (including the default ones) and always ends up instrumenting all field resolvers

Hey @blumamir
Apologies for poking you directly from here, since you're the feature implementor, would you be able to help me?
I think this issue isn't receiving attention because I was unable to tag it properly.
I'm working on a solution but I have nothing concrete yet. At least I have some sort of clue of what's happening. I can provide more details if want. Thanks

Thanks @pichlermarc for confirming my assumptions. I've been trying to come up with a solution and I found no other than dead ends.
Currently we're using a workaround that consists of filtering all the spans corresponding to scalar field types, which is a bit inconvenient as you might guess but it's the only feasible way we found.
I guess the effort of developing a specific Apollo instrumentation is not trivial, isn't it?
Even though, in which way do you suggest detecting that the resolver is the default one? I don't see that Apollos leaves any record when the resolver is replaced by their default.
Thank you again for looking into this.

Thanks @pichlermarc for confirming my assumptions. I've been trying to come up with a solution and I found no other than dead ends.
Currently we're using a workaround that consists of filtering all the spans corresponding to scalar field types, which is a bit inconvenient as you might guess but it's the only feasible way we found.
I guess the effort of developing a specific Apollo instrumentation is not trivial, isn't it?

Yes, the effort of implementing a specific Apollo instrumentation would not be trivial and would likely require maintaining it indefinitely.

Even though, in which way do you suggest detecting that the resolver is the default one? I don't see that Apollos leaves any record when the resolver is replaced by their default.

Often times there are multiple angles to approach this from; I'm not familiar with Apollo, but in some cases, it's also helpful/necessary to make changes to the upstream repo to get it to work (i.e. make Apollo leave some information that it modified it). Maybe @blumamir knows of an angle to approach this.

Maybe remove all the resolver spans?
If they are trivial they are of no interest.
If they are not trivial they tend do e.g. some HTTP call, DB request,... which likely creates spans via the corresponding HTTP/DB instrumentation.

The problem is that Apollo's plugins system overrides them to hook additions behavior, but given that there are many built in plugins enabled by default that can't be turned off (metrics, cache), all resolvers end up being defined. Kind of they abused their own plugins system to attach core modules.
I tried to register in the field whether the resolver function is user defined or not but the field type comes from graphql lib.

For our app, I applied these two patches to fix it for now:
@apollo/server
diff --git a/dist/cjs/utils/schemaInstrumentation.js b/dist/cjs/utils/schemaInstrumentation.js
index 3401ddcb3275222733678a9ec6b4c6d3b64ea3c8..c86846d6985d27ccd27c2ced3808f47e66e48f91 100644
--- a/dist/cjs/utils/schemaInstrumentation.js
+++ b/dist/cjs/utils/schemaInstrumentation.js
@@ -31,6 +31,7 @@ function pluginsEnabledForSchemaResolvers(schema) {
 exports.pluginsEnabledForSchemaResolvers = pluginsEnabledForSchemaResolvers;
 function wrapField(field) {
     const originalFieldResolve = field.resolve;
+    field.isTrivialResolver = !originalFieldResolve;
     field.resolve = (source, args, contextValue, info) => {
         const willResolveField = contextValue?.[exports.symbolExecutionDispatcherWillResolveField];
         const userFieldResolver = contextValue?.[exports.symbolUserFieldResolver];

@opentelemetry/instrumentation-grapqhl
diff --git a/build/src/utils.js b/build/src/utils.js
index 85eb3433d332f4e5d07bb206a029d7e45e45988c..f5059456697fd39ccfe9505eaea6ba96552573a0 100644
--- a/build/src/utils.js
+++ b/build/src/utils.js
@@ -225,7 +225,7 @@ function wrapFields(type, tracer, getConfig) {
             return;
         }
         if (field.resolve) {
-            field.resolve = wrapFieldResolver(tracer, getConfig, field.resolve);
+            field.resolve = wrapFieldResolver(tracer, getConfig, field.resolve, field.isTrivialResolver);
         }
         if (field.type) {
             let unwrappedType = field.type;


dart fix flag to specify a single lint rule

Let's say I contribute to a medium or big package is using lints 1.0.0, or even pedantic, and I want to upgrade to lints 2.0.0. After changing the include directive to use lints 2.0.0, I see 2000 new diagnostics in my IDE. Yikes! I'd like to fix those, but I don't want to mail out a change to my team with 2000 fixes, each of which fixes one of eight-ish new lint rules.
It would be much easier for my team to review a 500-fix change that only fixes one thing, then a 200-fix change that only fixes one other thing. Single-purpose CLs.
What is my workflow to produces these changes? Today, its pretty manual: find the name of one lint rule I wish to fix, comment out the include directive in analysis options, then add the lint rule at the bottom. Then run dart fix. Mail change (not including analysis_options.yaml!), remove the one lint rule, un-comment the include directive, re-analyze. Repeat.
It would be very nice if instead, I could specify a diagnostic to fix. Something like --rule=foo. It doesn't even have to be a list, or support inversions (maybe someday we'll get to dart fix --rules=include:lints/recommended.yaml,-this_rule,-that_rule,+oh_but_add_this_one, but I'm not asking for that 😁 ).

cc @bwilkerson @pq @goderbauer @stereotype441


Investigate workarounds for build systems that change JAR files, but don't close their classloaders

Some build systems (Maven sometimes, but also Gradle) opened URLClassLoaders on a JAR file they created. They just release the ClassLoader, but don't close it (as only Java 7+ allows this, and most build system are Java 5 or even below). You can check for Closeable interface and close if it is, but not all users do this correctly.
The problem with forbiddenapis is: It also uses a URLClassLoader (but closes correctly, so no issue here), but if the build system previously opened a URLClassLoader, then rebuilt the Jar file for some reason, the internal cache gets invalidated and breaks. Loading classes fails with FileNotFoundException (on older pre-Java 7 JVMs it sometimes crushed completely with mmap SIGSEGV).
The workarounds are:

Add a setting to the Checker (one more enum Option), so it disables the URLConnection cache when fetching resources. This may slowdown forbiddenapis, but only users affected by this must use it.
Use a hack I investigated: If opening resource fails with IOException (FNFE only), do the following: If JarURLConnection, call getJarFile(), close it, and get a new URLConnection(). Closing the JAR file removes the open file from Cache. A new URLConnection will then reopen it. The problem with that approach is: It breaks other threads accessing the same jar files possibly in other ClassLoaders. So we should carefully only apply that hack on URLs we know that they are from our URLClassLoader.

I created a PR: #76

I am not sure if we really need this workarounds. @rjernst has only seen the problem in Gradle builds, but was not able to reproduce. It looked like this was caused by a cranky Gradle Daemon running in background and changing JARs.
I will keep this issue/PR open for a while, but will not merge the stuff. At least we have a solution, but it is far from good to add workarounds for broken stuff anywhere else.

FWIW, I got hit by this in my build (Gradle with daemon), at least I suppose this is this issue; I experienced FileNotFoundException for classes that are present in my JARs:
* What went wrong:
Execution failed for task ':oasis-webapp:forbiddenApisMain'.
> de.thetaphi.forbiddenapis.ForbiddenApiException: Check for forbidden API calls failed: java.io.FileNotFoundException: JAR entry oasis/model/applications/v2/SimpleCatalogEntry.class not found in …/oasis-model/build/libs/oasis-model.jar

It is known that this happens with the daemon. The reason is some resource leak in the daemon, that does not close its classloaders.

Has this been reported upstream? Do you know what in Gradle keeps the classloaders alive?


fix matrix add row method

Fixes #

Hi @KupperXD, could you please tell me how to reproduce the bug you are fixing?

Hi @KupperXD, could you please tell me how to reproduce the bug you are fixing?

Hi
In order to recreate the bug described, it's needed  to add a column into matrix, which has a template.
for example, for me it's like this
I'm creating a Matrix and add an area   Upload::make (standard Orchid)
It consists of  template -
And that's what matrix_controller.js  is looking for, but it should find a template for a current line coping.
So, in case we add a field to matrix, which has an attached template. so the matrix_controller.js is not going to work properly while row adding.

Maybe use stimulus targets instead of querySelector?
https://stimulus.hotwired.dev/reference/targets

@tabuna Could you please merge that?


Implementaion of Send Notification using Email and Sms

Implementation of Send Notification using Email and SMS
Fetch the email from db to send a notification to a particular user by using grivience_id
Implementing RabitMq to listen to grievance service when it updates the Message Queue.

Facing Isuue while connecting to RabbitMq Getting time out connection error tried to change the port to 5672 from 15762 didn't resolved

Done


Support for colors?

There are applications that can change the color scheme like Gradience, Possible support for that?

Color theme support was implemented in #106 and is available in the latest release.
There are a few presets and you can make your own themes.
This should also make creating things like a Gradience plugin much easier because now it only needs to generate a color scheme and run a script to recolor graphics assets.


i18n(ko-KR): update customization.mdx

Description

update customization.mdx
#2083

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/customization.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Datasource always change state, if there is a number in it.

I've made a wildfly datasource, where some config keys have number in.
'min-pool-size'                  => 1,
'max-pool-size'                 => 10,
It configure it right, but every puppet run it tells me that state changed.
/state: state changed '{"max-pool-size"=>10, "min-pool-size"=>1}' to '{"min-pool-size"=>"1", "max-pool-size"=>"10"}'

Try alphabetizing your keys.

It makes no difference.
My ruby skills is very limited,but in wildfly/lib/puppet/type/wildfly_resource.rb there is a to_s method which would "" quote numbers.
Could this be why the state changes?

Here is some debug info:
Debug: /Stage[main]/Profile::Wildfly/Wildfly::Datasources::Datasource[dupDS]/Wildfly::Util::Resource[/subsystem=datasources/data-source=dupDS]/Wildfly_resource[/subsystem=datasources/data-source=dupDS]/state: Should: {"connection-url"=>"jdbc:oracle:thin:@ks-oracle02:1521:PREPR", "min-pool-size"=>"1", "driver-name"=>"oracle", "password"=>"pwd", "max-pool-size"=>"10", "jndi-name"=>"java:/jdbc/dupDS", "user-name"=>"DUP"} Is: {"connection-url"=>"jdbc:oracle:thin:@ks-oracle02:1521:PREPR", "driver-name"=>"oracle", "jndi-name"=>"java:/jdbc/dupDS", "max-pool-size"=>10, "min-pool-size"=>1, "password"=>"pqd", "user-name"=>"DUP"}
I have currently "fixed" this by adding this, to the "insync?" method.
       is.each do |k,v|
        if ( v.is_a? Integer )
          is[k]=v.to_s
        end
      end

I have encountered the same problem, INT/LONG values in "should" are quoted but the "is" values are not, resulting in insync always returning false.
This is not a datasource only problem, it happens on any resource where you try to and enforce state that includes INT/LONG values using the wildfly_resource type.

I'll investigate this tonight. Could you (@icetrain @lasse-kristensen) provide more information about your environment? (Ruby version, Puppet version, module version and etc)

Hey
ruby --version: ruby 2.0.0p598 (2014-11-13) [x86_64-linux]
puppet --version: 3.8.4
The module version is: 0.4.0
Running on: CentOS Linux release 7.1.1503 (Core)

ruby --version: ruby 2.0.0p598 (2014-11-13) [x86_64-linux]
puppet --version: 3.7.5
The module version is: 0.5.1
Running on: Red Hat Enterprise Linux Server release 7.1 (Maipo)

Fact: Puppet custom types checks current state (Is) against desirable state (Should).
Wildfly HTTP API returns a JSON representation of a resource (Is) and if one its properties is a number, a number is returned.
User provides a Hash representation (Should) of a resource in any form, but generally a string representation.
If the user provided a string representation of a number property, this issue will happen.
e.g.
wildfly::datasources::datasource { 'DemoDS':
  config         => {
    'driver-name' => 'postgresql',
    'connection-url' => 'jdbc:postgresql://localhost/postgres',
    'jndi-name' => 'java:jboss/datasources/DemoDS',
    'user-name' => 'postgres',
    'password' => 'postgres',
    'min-pool-size' => "10",
    'max-pool-size' => "30",
  }
}

instead of:
wildfly::datasources::datasource { 'DemoDS':
  config         => {
    'driver-name' => 'postgresql',
    'connection-url' => 'jdbc:postgresql://localhost/postgres',
    'jndi-name' => 'java:jboss/datasources/DemoDS',
    'user-name' => 'postgres',
    'password' => 'postgres',
    'min-pool-size' => 10,
    'max-pool-size' => 30,
  }
}

I changed custom type to ignore types (convert everything to strings), but if you change number properties to number literals you'll might get the same outcome.

Please re-open this issue. This 'fix' breaks recursive resources!!!
I will provide an alternative pull-request, that combines this issue with #108

It's back, at least for booleans: state changed '{"value"=>"false"}' to '{"value"=>false}'