[Snyk] Upgrade webpack from 5.77.0 to 5.79.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade webpack from 5.77.0 to 5.79.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released a month ago, on 2023-04-12.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-JS-TERSER-2806366
265/1000  Why? CVSS 5.3
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: webpack

5.79.0 - 2023-04-12New Features

webpack will now support simple destructuring scenarios for treeshaking namespaced imports and DefinePlugin by @ vankop in #16941

Bugfixes

Truncate extremely long module names in DefaultStatsPrinter by @ snitin315 in #16882
Add [contenthash] template support in DllPlugin's name option by @ snitin315 in #16935
Fixed a bug where readRecords compiler hook was causing hangs in conjunction with the ReadRecordsPlugin by @ snitin315 & @ zookatron in #16944
webpack can now consume ESM bundles generated by webpack's esm output support by @ vankop in #15608
[CSS] - webpack now respects CSS's case-insensitivity with atTags like @ MEDIA by @ alexander-akait in #16915
[CSS] - Fixes a bug where crossOriginLoading anonymous would not work when loading styles by @ chenjiahan in #16925

Developer Experience

Fix broken links and typos found in examples by @ snitin315 in #16937
Export more Externals Option types by @ snitin315 in #12774

Contributor Experience

Add new test case for ModuleFederationPlugin usage with shareScope option by @ snitin315 in #16943
Bump core-js from 3.20.3 to 3.30.0 by @ dependabot in #16905
Update all applicable local dependencies and devDependencies by @ alexander-akait in #16919, #16924, #16936, #16968
Update to Jest 29 by @ alexander-akait in #16947

New Contributors

@ chenjiahan made their first contribution in #16925
@ karlhorky made their first contribution in #16419
@ zookatron made their first contribution in #16301

Full Changelog: v5.78.0...v5.79.0

5.78.0 - 2023-04-05Features

Implement amdContainer support for AMD libraries (Fixes #16561) by @ long76 in #16562

Bugfixes

[CSS] - Nested atRule's @ media or @ supports now properly are replaced with unique identifiers by @ noreiller in #15812
[CSS] - Fix bug where closing parenthesis in CSS were not properly parsed and compiled by @ janlent1 in #16864
Fix an issue where oneOf rule has been picked multiple times by @ xiaoxiaojx in #16477
Add createRequire support for node:module prefix by @ alexander-akait in #16904
Fix bug where self-referencing a package in a shared module failed by @ weareoutman in #16685

Performance

Make ErrorHelpers named functions; Add types by @ TheLarkInn in #16893
Introduce ModuleTypeConstants for plugins by @ TheLarkInn in #16896
Refactor memory footprint in string usages for multiple plugins by @ TheLarkInn in #16894
Add more module type constants, use them across codebase by @ TheLarkInn in #16898

Contributor Experience

Implement default PR Template to use GitHub Copilot for PR's integration and fix template name usage by @ geromegrignon in #16890
ci: update actions/cache to v3 by @ armujahid in #16462
webpack org Collaborators and Members now have funded GitHub Codespaces!

New Contributors

@ geromegrignon made their first contribution in #16890
@ armujahid made their first contribution in #16462
@ long76 made their first contribution in #16562
@ weareoutman made their first contribution in #16685

Full Changelog: v5.77.0...v5.78.0

5.77.0 - 2023-03-29 Read more

    from webpack GitHub release notes

Commit messages

Package name: webpack

ecdcc1a 5.79.0
c07ca8e Merge pull request #15608 from webpack/fix/consume-esm-bundled-library
3b8c9c6 Merge pull request #16419 from karlhorky/patch-1
a9ea0d9 Merge pull request #16915 from webpack/fix-case-sensitivity-in-css
69d9c40 Merge pull request #16944 from snitin315/fix/read-records-callback
66f6472 Merge pull request #12774 from snitin315/export-more-types
96c5d21 Merge pull request #16882 from snitin315/limit-identifier-length
7f08e4d Merge pull request #16925 from chenjiahan/fix/css_cross_origin_loading
ee1a267 Merge pull request #16935 from snitin315/fix/dll-plugin-contenthash
4cacd7e Merge pull request #16941 from webpack/feat/destructuring-assignment
254f8aa Merge pull request #16968 from webpack/chore-again-more-deps
a9344bf chore: update more open-cli
f2619d3 chore: update more deps
ee24969 test: update snapshots
4a07cc8 chore: bump deps
4d2bdfc Merge pull request #16947 from webpack/chore-jest
76f077b ci: fix azure
54e72f0 ci: fix azure
8b6f33d ci: refactor
d2c133f fix: avoid extra lock
4118555 ci: fix github actions
9028ef5 ci: fix
23a06f2 chore: fix installation
080fc12 chore: update yarn

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (8b479a1) 100.00% compared to head (6aa7756) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #56   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          177       177
  Branches        39        39
=========================================
  Hits           177       177

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


Paginate data entities on tool detail page

The control to select a data entity (before running an analysis) populates via the "load all entities (of type)" endpoint. It did this when it was in the Launch Analysis modal previously. We should hit the paginated endpoint instead.
This involves either a lot of duplication from the data tab, or factoring out that functionality into a component.

Gonna just fold this into current work


Hello, can you generate a wafer diagram of any shape defect? Can the bottom rectangle pattern be removed when saving the police officer map

Hello, can you generate a wafer diagram of any shape defect? Can the bottom rectangle pattern be removed when saving the police officer map

Detail refer to the README file and sample2.js

A new helper method named diePalette(function) which can customize color depending on the grade of dies. You can just draw 'failed' dies.
Default the dies grid will not be drawn.
Use helper method visibility(id, state) to change visibility of some elements.

Detail refer to the README file and sample2.js

A new helper method named diePalette(function) which can customize color depending on the grade of dies. You can just draw 'failed' dies.
Default the dies grid will not be drawn.
Use helper method visibility(id, state) to change visibility of some elements.

disable circle or rectanglevar shotmap = uia.shotmap('wafer2')
    .circleBackground(false)
    .dieRect(false)

right click to save to an image, support full map, zoom in/out


[WIP] Property behaviors

A different, experimental take on property behaviors.

@swift-ci please test source compatibility

@swift-ci please test source compatibility release

I'm abandoning this approach in favor of https://github.com/apple/swift/pull/23440


SSL Certificate for https://packages.fluentbit.io/ expired 01.05.2022

Describe the bug
Hello,
https://packages.fluentbit.io/
Websites prove their identity via certificates, which are valid for a set time period. The certificate for packages.fluentbit.io expired on 5/1/2022.
Error code: SEC_ERROR_EXPIRED_CERTIFICATE
To Reproduce
$ curl https://packages.fluentbit.io/fluentbit.key
curl: (60) SSL certificate problem: certificate has expired
Expected behavior
Please renew the certificate.
Your Environment
- Fluentd version:
- TD Agent version:
- Operating system:
- Kernel version:
.

Your Configuration
.

Your Error Log
.

Additional context
.

https://github.com/fluent/fluent-bit/issues/5389


optimization enum/struct

Change how enum declaration are done to put the possibility to set the memory size.
TODO:
-Test if everything work well. Review.
-Rename some internal struct variable ?
-Struct padding optimization with new enum declaration ?

Thank you very much!
I reviewed the changes and made a few updates. Do you agree with them?
https://github.com/littlevgl/lvgl/commit/839757e6c88f23d79f0cb9a2bc0d93a568683036

Yes, i fixed some of them too, you are faster :)
A suggestion:
If some people are interested by speed optimization and not the memory optimization, maybe it can be good to add custom lvgl variable. E.G:
typedef uint8_t  lv_u8_t
typedef uint16_t lv_u16_t
typedef int8_t   lv_i8_t
typedef int16_t  lv_i16_t

It will allow to put faster variables like "int".
No need to do it if no demand on it

Great! :)
About the int typedefs:
For simplicity, I'd suggest having only an lv_int_t typedef. It should be at least int16_t and can be used when just a not to too big and not too specific variable is required. E.g. in for loops, counting things,  etc. What do you think?

Yes, can be good. Specific variable for intensive operation on them.
See this : signed/unsigned
typedef uint_fast16_t  lv_int_t
PS: fast mean that the minimum will be 16 bit but higher if possible.

Interesting read! According to this lv_uint_t and lv_int_t seems more suitebel.


Help screen

There should be a help screen describing controls of the current scene. Maybe we should integrate it with UI system so it can pick control information directly from the scenes.

"Press 'Z' to read signs!"


JSON output is single quotes not double quotes

Running a fairly basic script to output all the current alerts and the JSON is outputting in single quotes. I'm trying to import the data from the script into ElasticSearch and from what I've found, Elastic only accepts JSON in double-quotes.
My script:
from noaa_sdk import noaa
n = noaa.NOAA()
alerts = n.alerts()
print(alerts)
A Sample of the output:
{'@context': ['https://raw.githubusercontent.com/geojson/geojson-ld/master/contexts/geojson-base.jsonld', {'wx': 'https://api.weather.gov/ontology#', '@vocab': 'https://api.weather.gov/ontology#'}], 'type': 'FeatureCollection', 'features': [{'id': 'https://api.weather
Any help would be appreciated.

Than k you for the help.


loadProgramWOptions does not expose compile errors

loadProgramWOptions raises an exception on compile failure, but there isn't enough information in the exception to call clGetProgramBuildLog (just CL_BUILD_PROGRAM_FAILURE).  It would be nice to print the build log, or return the program p in the exception, or something, to make this easier to handle.  I have one quick-and-dirty approach at https://github.com/bergey/CLUtil/commit/8986f72fd1f4cd9b39563b2fb56bc6d8363517a3

Your quick and dirty fix looks like more or less the right thing. Is it worth doing anything more complicated?


Internal Server Error

For some days now, I get the following error when starting tabula:

What causes this? I didn't change anything, as far as I remember.

org.jruby.exceptions.RaiseException: (Encoding::CompatibilityError) incompatible character encodings: (Windows): Your Windows computer expects a type of encoding other than Unicode or Windows's English encoding. You can fix this by entering a few simple commands in the Command Prompt. (The commands won't affect anything besides Tabula.)
Open a Command Prompt
type `cd` and then the path to the directory that contains tabula.exe, e.g. `cd C:\Users\Username\Downloads`
Change that terminal's codepage to Unicode by typing: `chcp 65001`
Run Tabula by typing `tabula.exe`


models: tolerate empty field names when unpacking binary points

Prior to #5697 influx crashed with a panic if a binary point contained an empty field name. After #5697, influx entered an infinite loop in the same circumstance. With this commit, we silently elide any field with an empty field name when unmarshaling a binary point. A related PR (#5714) prevents SELECT INTO trying to create binary points with empty field names.

/cc @jwilder @e-dard

This isn't quite the right way to fix this.  A point should never contain fields without a name.

NewPoint should ensure that fields passed in do not have any empty field names and return an error if they do.
AddField should be removed since points should be created fully via NewPoint.
select queries with math that produces columns without a name should not happen.  It needs to generate a column name if an alias is not used.

Ok, np. I'll fix accordingly.

@jwilder - should I trim the field name of leading and trailing whitespace before testing for equality with the empty string?

@jwilder - I have implemented the first two suggestions with this series. I'll investigate what's required to fix the queries and may submit that as a separate series. I am going to leave 930fc42 in the series because the existing code didn't handle the case gracefully (it should have scanned to the end of the value buffer before skipping to the next field).
I'll check the result of the CI build later in the night.

@jonseymour thanks for this. I'm down with the following changes: (1) check for empty fields in NewPoint; (2) removing  AddFields.
Can you explain why 930fc42 needs to be accepted though?  newFieldsFromBinary is only called when retrieving the fields of a Point created via NewPoint, and NewPoint will now check for empty fields.

@e-dard - the algorithm is just wrong - if the data is corrupt, the existing algorithm an infinite loop is guaranteed, as illustrated by the test cases that begin the series.. With the version I propose, it is guaranteed that the loop will terminate.

@jonseymour which test case are you referring to? Could you throw a link my way? Cheers!

This commit 23be938. A transcript of executing this test, merged into both #5697 (88937ab0f)  and the commit immediately preceding (b8bb32e) is as follows:
greywedge2:influxdb jonseymour (1)$ cd models
greywedge2:models jonseymour (1)$ git checkout 88937ab0f
HEAD is now at 88937ab... Fixes #5664
greywedge2:models jonseymour (1)$ git merge --no-commit 23be938
Automatic merge went well; stopped before committing as requested
greywedge2:models jonseymour (1)$ go test ./...
--- FAIL: TestAddFieldWithEmptyName (1.00s)
	points_test.go:1790: failed: probable infite loop
FAIL
FAIL	github.com/influxdb/influxdb/models	1.021s
greywedge2:models jonseymour (1)$ git reset --hard HEAD
HEAD is now at 88937ab Fixes #5664
greywedge2:models jonseymour (1)$ git checkout 88937ab0f^1
Previous HEAD position was 88937ab... Fixes #5664
HEAD is now at b8bb32e... Merge pull request #5470 from influxdata/ross-packaging-updates
greywedge2:models jonseymour (1)$ git merge --no-commit 23be938
Updating b8bb32e..23be938
Fast-forward
 models/points_test.go | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
greywedge2:models jonseymour (1)$ go test ./...
panic: runtime error: index out of range

goroutine 77 [running]:
github.com/influxdb/influxdb/models.scanTo(0x82049e930, 0x8, 0x10, 0x0, 0x82049663d, 0x0, 0x0, 0x82049e920, 0x1)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points.go:892 +0xa7
github.com/influxdb/influxdb/models.newFieldsFromBinary(0x82049e930, 0x8, 0x10, 0x0)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points.go:1396 +0x108
github.com/influxdb/influxdb/models.(*point).unmarshalBinary(0x82049cb40, 0xffffffffffffffff)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points.go:1298 +0x40
github.com/influxdb/influxdb/models.(*point).Fields(0x82049cb40, 0x82049e930)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points.go:1211 +0x3a
github.com/influxdb/influxdb/models.(*point).AddField(0x82049cb40, 0x1c7d60, 0x1, 0x15a540, 0x82049e940)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points.go:1217 +0x25
github.com/influxdb/influxdb/models_test.TestAddFieldWithEmptyName.func1(0x82049cab0, 0x82049a3c0)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points_test.go:1784 +0x35b
created by github.com/influxdb/influxdb/models_test.TestAddFieldWithEmptyName
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points_test.go:1786 +0x6c

goroutine 1 [chan receive]:
testing.RunTests(0x216df8, 0x2b3980, 0x48, 0x48, 0x101)
	/usr/local/Cellar/go/1.5.3/libexec/src/testing/testing.go:562 +0x8ad
testing.(*M).Run(0x820355ef8, 0x820392280)
	/usr/local/Cellar/go/1.5.3/libexec/src/testing/testing.go:494 +0x70
main.main()
	github.com/influxdb/influxdb/models/_test/_testmain.go:218 +0x116

goroutine 76 [select]:
github.com/influxdb/influxdb/models_test.TestAddFieldWithEmptyName(0x82049cab0)
	/Users/jonseymour/ninja/go/src/github.com/influxdb/influxdb/models/points_test.go:1787 +0x16a
testing.tRunner(0x82049cab0, 0x2b4028)
	/usr/local/Cellar/go/1.5.3/libexec/src/testing/testing.go:456 +0x98
created by testing.RunTests
	/usr/local/Cellar/go/1.5.3/libexec/src/testing/testing.go:561 +0x86d
FAIL	github.com/influxdb/influxdb/models	0.017s

OK cool, I see. But that test is now irrelevant because you're removing AddFields and also doing the check for empty field names inside of NewPoint.
With the changes you have made to NewPoint it should no longer be possible to get to newFieldsFromBinary with an empty field name.
I think 930fc42 can be dropped from the PR. Hope that makes sense?

As I said, i can create a test case with points.NewPointFromBytes (which does not involve any calls to the now deleted points.AddField) method which will force influxd into an infinite loop if you really want me to prove that the algorithm prior to 930fc42 is vulnerable.

@e-dard I've created a test case which shows that points.NewPointFromBytes is vulnerable to an infinite loop if corrupt data is read from the binary encoding of the point.
Given that it costs to little to protect the system from an infinite loop caused by a data corruption, there seems to be no sound reason not to accept (8490c6e) (which replaces 930fc42).

To see the vulnerablity, checkout 9b56025 which does not include the fix and then run the tests in the model package.

@jonseymour Now I see. It wasn't clear to me that this issue relied on a successful call to NewPointFromBytes and then a call to p.Fields...
OK, just a small nit on the test case then squash all these down?

Thanks for the review, @e-dard.
With this series, I've removed the test that used AddField and fixed the failure messages per the standard.

Actually, I should check that NewPointFromBytes returns an error if there are no fields in the parsed point. I'll push a new series with those tests and changes.

Ok, @e-dard  added two additional commits which require that points.NewFieldFromBytes() errors if a corrupt binary point does not yield at least one field.
It is quite late where I am (03:45), so I'll look for responses later in my morning.

@e-dard, @jwilder - I have now extended this series of commits to ensure that the fields that result from a SELECT INTO query are unique and non-empty. The test case in 3518ae8 shows the behaviour.
Since this series now touches the query executor it would be worth getting @benbjohnson to have a look at it.
The RHS of the final merge could (in principle) be merged with the 0.10.0 maintenance branch (not suggesting that you do this, only that it is possible). The conflicts between the two branches were resolved in the merge commit.

@jwilder I see that is an overlap with #5730. I am not sure if #5730 is guarantees uniqueness of column names. Does it need to?
Do you want me to extend the changes to points.NewPoint()/points.NewPointFromBytes() to check that field names are unique?

@jonseymour #5730 doesn't do anything with column uniqueness, it just gives a name to columns with a binary expression that is missing in the new query engine.

Thanks for the info @jsternberg.
I can see one argument for pushing up the column uniqueness logic from cluster/ to influxdb/ - it comes to the predictability of column names to be used by a SELECT INTO statement. As implemented above, the generated column names will be different to the names that appear in the influx UI, since for example, select max(power), max(temperature) from "device" group by time(1h) will currently get two max columns in the UI. With the change above, the columns will be max, and max_2, however without running the INTO query and inspecting the target, the user won't necessarily be able to predict that.
If the column uniqueness functionality is pushed up into influxql, then no downstream client of influxql (including the SELECT INTO logic) will have to worry about having to use their own algorithm to ensure unique column names which, I think, would be a very good thing.
I'll await @ed-dard/@jwilder's comments about this before I make any further changes in this area.

Whatever the naming scheme should be, the logic should probably go in (*ast.Field).Name() or (*ast.SelectStatement).ColumnNames(). That way a third party could use it to determine the name of the column directly from the select statement.

@jwilder @e-dard looking at the code, it may be easier to just throw an error when a select statement is used that has conflicting output names and advise that the user uses AS to disambiguate rather than come up with some default collision detection that will work for everyone. We can also gradually improve the automated naming. A call to max(value) should probably be named max_value instead.

@jwilder should make the final call on this.
Personally I would vote for generated column names that match the expression, e.g., max(value) would generate a column name "max(value)", and for arbitrary math some placeholder, e.g., col, col (1), col (2) or whatever.

@e-dard I mentioned us changing how the column is generated, but we also need a way to resolve conflicts. When resolving conflicts we have three options.

Automatically rename columns that are in conflicts
Always name columns in a way that they can never possibly conflict
Send an error back to the user and tell them to use an alias
Do nothing (causes hidden errors)

The first one I think may be unintuitive, especially when it comes to SELECT INTO calls. Someone may do SELECT value, value INTO mem FROM cpu and be confused that suddenly it has implicitly renamed value to value_0. Maybe we can only rename later entries instead so the above call would be value and value_1 and that would be more intuitive? That may be my favorite if we're going to do any implicit renaming.
Always naming columns different would cause confusion with a single value so I don't like it. It would cause SELECT value FROM cpu to output a column of value_0 or value_1 depending on the name.
The third is the safest, but causes existing queries to break and requires us to do a better job of choosing a better implicit name. It may not be feasible at the moment.

@jsternberg I don't think this:

Someone may do SELECT value, value INTO mem FROM cpu and be confused that suddenly it has implicitly renamed value to value_0.

is necessarily a big problem if the UI produces the same result - at least the UI and SELECT INTO would produce the same result.
Personally my vote is with option 1 (deterministically rename), followed by option 3 (error out).
jon.

@jonseymour I did a hangout with @jwilder and @e-dard to work through some of the use cases that have been discussed in this thread.
The implementation should probably be done in ColumnNames(). We didn't talk about that, but I think that would be a better place. It would allow someone to use the influxql package and call ColumnNames() to get the real column names without having to worry that they get implicitly renamed later. I don't yet know if this causes an issue for other places in the code though that may rely on ColumnNames() not doing that, so that's tentative assuming the implementation works.
We also discussed heavily how renames should be done and how aliases should be treated. We agreed on a two phase method of assigning names. First, aliases would get assigned to columns. Conflicts would not be implicitly renamed.
For other names, they would be resolved in the order they are present in the query with internal state being kept for which values had been encountered. For example, with SELECT value, value_1, value + 2 FROM cpu, you would get the columns value, value_1, value_1_1. This simple implementation would prevent names from conflicting with each other accidentally and also prevent us from having to do a very complicated set of recursions to determine how to resolve the names. Users could use aliases if they don't like the output.
I'm going to have a PR up later today with this implementation. Thank you for your contribution and for helping us to hash this out.

Thanks @jsternberg that all makes sense. I've removed the uniqueness logic from this series.
@e-dard: I think I am about done this PR so I'll leave it with you for final review.

@e-dard I have squashed most of the commits in this series. There is one commit for the tests showing the problems that needed to be fixed and another for the fixes. The RHS of the merge commit merges cleanly with the 0.10.0 branch (which is in the form I need it for my production system). I have added a CHANGELOG update.

I've removed the CHANGELOG.md update until I get a +1.

One small thing, but :+1: otherwise.

@jwilder - done. Thanks for the review!

LGTM. I'll merge this Monday morning @jonseymour

Thank you, @e-dard


Add a mol2 reporter for OpenMM

Often, I want to save structures of molecules from a simulation (i.e. gas phase) to a format which retains the charges, bond orders, and coordinates. None of the current OpenMM reporters can retain all of this info since OpenMM doesn't retain bond orders.
I should make a mol2 reporter which is initialized with an OEMol or OEMols for the components of the system, and then writes coordinates from a simulation/minimization to .mol2 format when done.
For my own reference, an easy entry point would likely be to consult the OpenMM PDB reporter:

class structure for reporter: https://github.com/pandegroup/openmm/blob/master/wrappers/python/simtk/openmm/app/pdbreporter.py
file definition: https://github.com/pandegroup/openmm/blob/master/wrappers/python/simtk/openmm/app/pdbfile.py

See also #167


feat: use lazy import

Coverage increased (+0.8%) to 42.466% when pulling 35832e594bfbe6cc056d98aa3bcd457890238f64 on next into 2826e07718ef16c8ac022b13f606250f0574c341 on master.

Coverage increased (+0.8%) to 42.466% when pulling 35832e594bfbe6cc056d98aa3bcd457890238f64 on next into 2826e07718ef16c8ac022b13f606250f0574c341 on master.


SA1028 Fix All in Solution appears to throw an exception

I'm working with Newtonsoft.Json.Portable.sln from the recently added churn tests, and ran StyleCopTester with /id:SA1028 /fixall. I haven't yet tracked down the cause.
The fix 'SA1028CodeFixProvider' threw an exception after 19ms:
System.ArgumentException: The changes must be ordered and not overlapping.
Parameter name: changes
   at Microsoft.CodeAnalysis.Text.SourceText.WithChanges(IEnumerable`1 changes)
   at StyleCop.Analyzers.SpacingRules.SA1028CodeFixProvider.FixAll.<FixAllInDocumentAsync>d__5.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCop.Analyzers.CodeFixes\SpacingRules\SA1028CodeFixProvider.cs:line 92
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at StyleCop.Analyzers.Helpers.DocumentBasedFixAllProvider.<GetSolutionFixesAsync>d__5.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCop.Analyzers.CodeFixes\Helpers\DocumentBasedFixAllProvider.cs:line 107
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<ComputeOperationsAsync>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<GetOperationsCoreAsync>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<GetOperationsAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at StyleCopTester.CodeFixEquivalenceGroup.<GetOperationsAsync>d__23.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCopTester\CodeFixEquivalenceGroup.cs:line 141
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at StyleCopTester.Program.<TestFixAllAsync>d__3.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCopTester\Program.cs:line 198

This was caused by a bug in #1983, and was corrected in 5fa8d695.


Charm crash on server startup. 0.18.1 Beta 20

I'm aware that Charm was fixed as of 0.18.1 Beta 20 of Quilt. Whilst it now works on my client it does not launch on the server.
crash-2022-12-02_<IP_ADDRESS>27-quilt_loader.txt

Thanks for reporting - it looks like something has gone wrong while quilt-loader performed dependency resolution, rather than a specific problem with charm.

Okay beta 21 should fix this - can you retry?

Just tried it and it seems to work now thanks


Add support for macOS menu commands

The macOS app should have support for menu commands, for instance:

[ ] Styles - cmd+b, cmd+I, cmd+u
[ ] Actions - cmd+c, cmd+v, cmd+z, cmd+shift+z

There's a new menu command for rich text formatting. We can add more menu commands like this in the future.
This has been merged to the main branch and will be available in 0.4


Improve selection of colors for quick colors for concepts.

also, increase font size step when augmenting for diminshing size.


Mark http.nonProxyHosts as not a typo in the docs

@bot-gradle test and merge

OK, I've already triggered a build for you.


Animated.Value resets if component was re-mounted

Description:
If Animated.Value is used for a natively driven animation and then the component is unmounted, when the component is mounted back, the rendered value resets to the initial state.
React Native version:
0.61.5
Steps To Reproduce

Create Animated.Value av.
Use av in styles for component C (e.g. translateX). Don't attach any listeners to the value.
Run an animation (useNativeDriver: true) with av.
Unmount C (and all other components that utilize av in their styles).
Mount C.
C won't be positioned correctly since av's internal value resets to its state on step 2.

Expected Results
Animated.Value should hold the last value it had before the component was unmounted, and the component should return to the same state on being re-mounted as a result.
Code Example
https://github.com/djonin/react-native-animated-value-bug
This is a duplicate of issues #23712 and #23621, which were incorrectly closed as fixed, due to using a listener in the example, which avoids this defect. The stated fix #24571 only fixed the problem with the listener not correctly re-binding to the new value, however the broader case issue mentioned in #23621 is still there.
This issue is due to the fact the native node, which has the up to date value, is destroyed after a component is dismounted. The JS animated value object doesn't receive any native value updates unless there is a listener attached explicitly by the user. Therefore the next time the component is mounted, it uses the value as it was before the native animation.
Currently the only way to preserve the value is to attach a listener, meaning the value updates will be sent over the bridge often during animations. In addition, it's not obvious to users that attaching a listener is required for correct re-mounting behavior.
Proposed solution
One possible fix would be to make a new event to call back to JS with the value just once when the native node is being detached/destroyed, so that we can preserve whatever the final native value was in our JS object, and for the JS animated value object to always listen to this new native event, even with no listeners attached explicitly. We need some place to store our value, and since the native side is quite correctly being cleaned up after unmount, JS side seems like the best place to do it.

Hi, I've found two possible solutions for this problem.

Save last value when animation has finished but it is required that every native driver with callback has to be invoked with additional argument which is last animation value and then we save value on JS side though It wouldn't work when animated component was unmounted during animation.

We have to invoke newly created function from native side i.e getValue in nodes managers,(which just takes current value from given AnimatedValue node), when we are detaching node and then we save returned value as a internal value of Animated.Value node on JS side. Something similar is has been already implemented in reanimated.

From the way the components are written, it seems the intention was for the JS node to hold the correct value while it is not attached to any native nodes. I definitely agree with your second suggestion, I had a look through reanimated, and calling getValue  explicitly is a good fix as well. I had in mind a listener similar to onAnimatedValue , however provided there are no situations where the native value gets lost without going through JS __detach, asking for the value there directly might be better.

cc @JoshuaGross Can you take a look at our proposed solution? We could really use any feedback. Thanks


Got the errror when build on AppStore

I got the below errors. I'm using version 1.17.0.
Dear Developer,
We identified one or more issues with a recent delivery for your app.  Please correct the following issues, then upload again.
ITMS-90562: Invalid Bundle - One or more dynamic libraries that are referenced by your app are not present in the dylib search path.
ITMS-90562: Invalid Bundle - There's an issue with the app's bitcode compilation. Rebuild the app using the current public (GM) version of Xcode and upload it again.
ITMS-90562: Invalid Bundle - The app submission can not be successfully recompiled from bitcode due to missing symbols during linking. You can try to reproduce and diagnose such issues locally by following the instructions from: https://developer.apple.com/library/archive/technotes/tn2432/_index.html
I also found out the same issue here (https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/702)
But could you help tell me that did you fixed this issue? And how can I deploy my application to AppStore?

@duydtdev Thanks for the report. Which architectures you are targeting in your app ? Speech SDK supports arm64, so in case you have armv7 in the list of supported architectures, please remove it.

@jhakulin Thanks for your reply. I turn off Bitcode when deploy, then it works fine. I will try your way once. Thank you


Handle negative content-length header

Currently a negative content-length header results in the following error:

11:11:13.109 [info] Running MyPlug with Bandit 0.6.0 at <IP_ADDRESS>:4000 (http)

11:11:16.385 [error] GenServer #PID<0.383.0> terminating
** (FunctionClauseError) no function clause matching in :prim_inet.recv0/3
    :prim_inet.recv0(#Port<0.11>, -328, 15000)
    (thousand_island 0.5.11) lib/thousand_island/socket.ex:78: ThousandIsland.Socket.recv/3
    (bandit 0.6.0) lib/bandit/http1/adapter.ex:218: Bandit.HTTP1.Adapter.grow_buffer/4
    (bandit 0.6.0) lib/bandit/http1/adapter.ex:160: Bandit.HTTP1.Adapter.read_req_body/2
    (plug 1.14.0) lib/plug/conn.ex:1158: Plug.Conn.read_body/2
    #cell:wxlzqfftjgta7e6vzfyvswxlroajtju3:7: MyPlug.call/2
    (bandit 0.6.0) lib/bandit/http1/handler.ex:73: Bandit.HTTP1.Handler.call_plug/2
    (bandit 0.6.0) lib/bandit/http1/handler.ex:15: Bandit.HTTP1.Handler.handle_data/3
Last message: {:continue, :handle_connection}
State: {%ThousandIsland.Socket{socket: #Port<0.11>, transport_module: ThousandIsland.Transports.TCP, connection_id: "E7BD30C0B1D5", acceptor_id: "0C74D2FB7D59", read_timeout: 15000}, %{handler_module: Bandit.InitialHandler, plug: {MyPlug, []}}}

Because the negative body_remaining is used as a param for Socket.recv/3.
This will terminate the reading of the body, when the body_remaining is less or equal to 0.

After thoroughly reading RFC9112§6.3 I realized this is quite complex.
I converted this to a draft, since it needs some more work to handle all case defined here.

Heads up that I just merged https://github.com/mtrudel/bandit/pull/57 which caused a bunch of diff noise in tests. You'll probably want to rebase on top of that before you get too far along on tests.

A couple of suggested changes. Sorry it took me so long to get to this. Life &c.
I think the only other thing missing here would be to handle the differing response codes as you point out. I think you could handle those in the get_content_length function (assuming you provide it with the response code). That would keep all the logic contained.

If I understand that correctly, the status code are only relevant in the response to the client, not the request to the server.
So we only need to look at 6.3.{3-7}, which should be covered by this PR.

Merged main in, fixed a minor typo and a racy test. Once CI is green we can merge!
A M A Z I N G


Max: TyCache enhancement cached material

Changelog Description
Related to https://github.com/ynput/ayon-core/pull/56
This PR is to allow AYON to publish the material(s) assigned to tyCache along with all cached data.
Additional info
Simple Remark on the enhancement in the future, we can support instanced material override in the loader after the implementation of look family (i.e. the publish of the shader in Max)
Testing notes:

Launch Max

Open the scene with tyCache

Assign some materials to tyCache by adding instance material nodes

Use any scene materials for the tyCache node

Create tyCache instance with the tyCache node

Make sure tyCacheChanMaterials being selected

Publish

Open new scene

Load the tyCache instance, there should be instanced materials lib found in current file info, which means the material should be also loaded along with tyCache.

Will close this issue as this is also resolved in https://github.com/ynput/ayon-core/pull/56

closed the issue as https://github.com/ynput/ayon-core/pull/56 resolves this.


Visibility of test controller label

Testing #129471
I'm not sure where I can see which test controller is being used

If you have exactly one controller, it won't be shown in many places (you can see it in the output peek of test failures in the description on items that show their "path"). But if there are multiple controllers that have items, they'll appear as separate nodes in the Test Explorer view, for example.


Request for Database Permissions

Hello,
I am currently working on Lab 7 StayWell and have encountered an issue that requires additional database permissions. The specific permission I need is the ability to use the GRANT command.
Having this permission will allow me to run my code and complete my task.
Could you please assist in granting me this permission or guide me on how to obtain it?
Thank you for your time and assistance.
Best,
Mansi
Version: 1.91.1
Commit: f1e16e1e6214d7c44d078b1f0607b2388f29d729
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

Please use the course support channel.

Please use the course support channel.

^ please do as mentioned here


NBody setParameters bugfix

This prevents NperBatch from being uninitialized when setting parameters of the NBody solver by checking if the solver has been initialized before using the number of particles as the default. Happy to implement a different solution if you have an alternative!
One thought- is there ever a use case where you don't want NBatch*NperBatch = numberParticles?  If not, maybe we should force NBody to be initialized before setting parameters so we can check and make sure the conditional holds, especially since I noticed that NBody.Mdot() was unstable and would crash if the parameters didn't satisfy the above.

I updated the changes to make it illegal to set parameters for NBody before initialization, which works just fine. However, I learned that the opposite is true for PSE and it is illegal to initialize PSE before setting parameters. This is due to PSE initializing a uammd solver with the box size that is given in setParameters. It looks like the same is true for DPStokes (i.e. setParameters before initialize).
In either case, it seems like I should stick to (and enforce) a specific convention for the initialization and setting of parameters for each solver. In my head, it makes sense to initialize a solver first and then set it's parameters, although I'd have to investigate further to see if that's feasible since it'd involve changing PSE and DPStokes. @RaulPPelaez do you have a convention you'd like me to try and implement to remain consistent?

Acording to the README setParameters is supposed to be called before initialize


Support scarthgap (the latest yocto LTS) / cut dependency for meta-oe / upgrade nnstreamer version

Support recent yocto LTS scarthgap
Delete deprecated tensorflow-lite v1
Add recipe for googletest and remove dep for meta-oe
Upgrade nnstreamer version as 2.4.2

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #89. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Fix updating 'FormatMessage' elements prop

Fixes #86, updates previous test fix for beforeFormat handling to exercise afterFormat handling too

Thank you


Require plan to be positive

Is this okay? Using the _internalFail, explain, message, etc

@davidmarkclements should we setup that brittle org so we can collab there instead?


create reusable directive for a number input

This directive can be used in the recordedit and faceting.

integer
float

While writing this directive, I realized that float and integer can't be handled in the same directive. Float will require a separate directive for showing properly.

Moving this issue to round 2.

An input-switch has been recently implemented that will create an input for any type of columns. This includes numbers too.


Do not attempt to write source server data with PdbStr.exe if on Unix OS

Fixes https://github.com/gluck/il-repack/issues/173

Look good to me.


Added java.time.Period assertions

Added assertions:

hasDays()
hasMonths()
hasYears()
isPositive()
isNegative()

Check List:

Fixes #1853
Unit tests : yes
Javadoc with a code example (on API only) : yes

Thanks @joel-costigliola 😄

@HaydenMeloche thanks, I realize there are missing tests for ShouldHavePeriod and ShouldBePeriod, use ShouldHaveDuration_create_test as a reference for implementing them

@joel-costigliola Sorry about that, I added the missing the tests.

Integrated thanks @HaydenMeloche!


Add setAttrWithMinSize to feature API

Add a new features.getMinZoomForPixelSize(zoom) method to feature API that returns the minimum zoom level at which a feature is a certain size in pixels, and a convenience feature.setAttrWithMinSize(key, value, minPixelSize) method that uses it to set the min zoom based on feature size.
For more flexibility you can do math on the result of features.getMinZoomForPixelSize(zoom) or create a ZoomFunction that calculates feature.getPixelSizeAtZoom(zoom) and applies a custom threshold based on the zoom level.

Nice. Do you have an example use case?

This openmaptiles PR does something similar for the brunnel tag on road segments at lower zooms to  help avoid breaking them up when they get merged https://github.com/openmaptiles/planetiler-openmaptiles/pull/126 - I figure that's a common use case so makes sense to be a reusable utility in planetiler


Sharing: You need pulse audio 1.0 and above to support audio share

pulseaudio-utils-32bit-16.1-1.2.x86_64 libpulse0-32bit-16.1-1.2.x86_64 alsa-plugins-pulse-<IP_ADDRESS>-2.2.x86_64 pulseaudio-setup-16.1-1.2.x86_64 alsa-plugins-pulse-32bit-<IP_ADDRESS>-2.2.x86_64 libpulse-mainloop-glib0-16.1-1.2.x86_64 libpulse0-16.1-1.2.x86_64 pipewire-pulseaudio-0.3.58-1.1.x86_64
pipewire-spa-tools-0.3.58-1.1.x86_64 pipewire-0.3.58-1.1.x86_64 libpipewire-0_3-0-0.3.58-1.1.x86_64 pipewire-tools-0.3.58-1.1.x86_64 pipewire-alsa-0.3.58-1.1.x86_64 pipewire-spa-plugins-0_2-0.3.58-1.1.x86_64 pipewire-modules-0_3-0.3.58-1.1.x86_64 pipewire-pulseaudio-0.3.58-1.1.x86_64
My system is openSUSE Tumbleweed
And it uses pipewire-pulseaudio

Can you try on a system without Wayland and without PipeWire?

It works correctly

Thanks for confirmin @kendal-pioneer. I am not interested in investing any time in Wayland but if someone cares enough to send a pull request I will consider it, as long as it doesn't break Xorg.

PLEASE NOTE: In general, you should download AppImages only from the original authors of the software (here: https://zoom.us/), but since they are not yet providing one I am offering this here. Please contact them if you would like to see an official one.

@kendal-pioneer
The issue you experienced isn't with Wayland, it's just with Pipewire's PulseAudio plugin. Wayland doesn't have anything to do with audio. Try it with Pipewire again and if the issue still exists then bring if up on Pipewire's repository so they can work on it.
Probo doesn't really know what Wayland and Pipewire are or how they relate to each other. He saw "Pipewire" and since he's only heard of Pipewire when talking about Wayland, he assumed you're using Wayland. Since he thinks Wayland breaks everything, he saying it's the issue here even though it can't be.
Now he's using this issue you opened about an audio problem as an example in his thread to try to push a boycott of something that's part of Linux's graphics stack.

Well. I'd suggest to try on a system without Wayland and without Pipewire and see whether it works correctly there. ;-)

@probonopd yes because they would be using PulseAudio directly instead of Pipewire's Pulse plugin. There was never a chance it had anything to do with Wayland because Wayland has nothing to do with audio.
You could have suggested they try Zoom on Windows and it would have worked, too. That way Pulse, Linux, and AppImage would be removed from the equation, too. It wouldn't have meant that you figured out the problem though.
Looking up the issue shows that most people fixed it by installing pulseaudio-utils.

Actually if I just ignore the zoom warning/notification and although I don't get the option to share audio with video check box
I know it does actually work, regardless

Oh! I didnt realize it was working regardless of the error message lol Glad it worked!


BUG: Update twitter icon

Fixes #1222 Update old twitter icon:)

Pull Request Checklist

[x] I have added screenshots and videos to show before and after the working of my code.
[x] I have ensured that the screen size is set to 100% while making the video.
[x] I have synced the latest fork with my local repository and resolved any conflicts.
[x] I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
[x] I understand that if any the above conditions are not met , my PR will not be MERGED .

@ANSHIKA-26 Please review my PR and merge with labels:)

is it fixed in footer of all pages ? kindly show video

Ok i will edit and provide video to you
On Sun, Oct 20, 2024, 11:04 ANSHIKA-26 @.***> wrote:

is it fixed in footer of all pages ? kindly show video
—
Reply to this email directly, view it on GitHub
https://github.com/ANSHIKA-26/WordWise/pull/1224#issuecomment-2424608743,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHONQVARAZIUVTBNO5XLNTDZ4M6GFAVCNFSM6AAAAABQIDIKHGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRUGYYDQNZUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@ANSHIKA-26 Please review and merge it:)
Video:)
https://github.com/user-attachments/assets/0adf61a2-1abb-489e-9aa7-05d064dc6c31

I do erase any code in my vscode when we use ctrl+c and ctrl+v then it format whole page thats why these changes show here:)

sync fork once , ig your fork is old

@ANSHIKA-26 Everything is fine now please merge it:)

@harshbhar0629 everything is not fixed , while solving merge conflicts you removed webp images and added png ones , we need webp format only , kindly refork , resync , and then make new PR , in this PR you have changes things which were not even necessary , kindly restrict your code to only changing an icon!

i would honestly recommend you to either make a new branch from the latest fork , then redo the changes that you have done and make a PR !

Ok i will
On Sun, Oct 20, 2024, 15:55 ANSHIKA-26 @.***> wrote:

i would honestly recommend you to make a new branch from the latest fork ,
then redo the changes that you have done and make a PR !
—
Reply to this email directly, view it on GitHub
https://github.com/ANSHIKA-26/WordWise/pull/1224#issuecomment-2424825734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHONQVHOA7YY6PH5WSOE2YLZ4OAJPAVCNFSM6AAAAABQIDIKHGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRUHAZDKNZTGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


[QUESTION] Conflicting docs recommend View Blade Engine Mode TASK or SYNC?

This doc page seems to recommend Task mode and shows Blade engine Task mode config https://github.com/hyperf/hyperf/blob/master/docs/en/view.md#sync-mode

Configuration:
... Hyperf\View\Engine\BladeEngine::class ... Mode::TASK ...
... 'engine' => BladeEngine::class, ... 'mode' => Mode::TASK, ...
...please ensure that the relevant engine is coroutine safe, otherwise there will be data consistency problems. It is recommended to use the more data-safe Task mode.

But This doc page seems to recommend SYNC mode and shows Blade engine SYNC mode config https://github.com/hyperf/hyperf/blob/master/docs/en/view-engine.md

This component recommends using the SYNC rendering mode, which can effectively reduce the loss of inter-process communication
... 'mode' => Hyperf\View\Mode::SYNC, ...

Please advise - Is Blade engine coroutine safe? Is it better to use it with Task mode or Sync mode? Any difference between configuring the View vs. the View Engine? Should View be in Task mode, but the View Engine be in SYNC mode?

hyperf/view-engine is a coroutine safe component, it's different with duncan3dc/blade, but their syntax and usage are similar.

@huangzhhui it doesn't seem the question was answered... I was asking about Mode: Task mode vs. Sync mode, and how to use both configurations from both docs pages together. Please see the above initial question in detail. The answer seems to only mention about coroutine safeness, and about duncan3dc/blade but not about the issue of Mode and how to understand the docs

Sync mode
If you use the Sync mode to render the view, please ensure that the relevant engine is coroutine safe, otherwise there will be data consistency problems. It is recommended to use the more data-safe Task mode.
And I just tell you that hyperf/view-engine is a coroutine safe component, so you could use Mode::SYNC, notice that hyperf/view-engine syntax and usage is similar with blade, but it is not the original blade.


[Event Change Request] Table 37 Sales Line - procedure InitQty - event OnBeforeInitQty

We would like the Parameter/-s ShouldInitQty added to the following event.
Original:
    local procedure InitQty()
    var
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeInitQty(Rec, xRec, IsAsmToOrderAllowed(), IsAsmToOrderRequired(), IsHandled);
        if IsHandled then
            exit;

        if (xRec.Quantity <> Quantity) or (xRec."Quantity (Base)" <> "Quantity (Base)") then begin
            InitOutstanding();
            if IsCreditDocType() then
                InitQtyToReceive()
            else
                InitQtyToShip();
            InitQtyToAsm();
            SetDefaultQuantity();
        end;
    end;

...

    [IntegrationEvent(false, false)]
    local procedure OnBeforeInitQty(var SalesLine: Record "Sales Line"; xSalesLine: Record "Sales Line"; IsAsmToOrderAlwd: Boolean; IsAsmToOrderRqd: Boolean; var IsHandled: Boolean)
    begin
    end;

Change:
    local procedure InitQty()
    var
        IsHandled: Boolean;
        ShouldInitQty : Boolean;
   begin
        IsHandled := false;
        ShouldInitQty := (xRec.Quantity <> Quantity) or (xRec."Quantity (Base)" <> "Quantity (Base)");
        OnBeforeInitQty(Rec, xRec, IsAsmToOrderAllowed(), IsAsmToOrderRequired(), IsHandled, ShouldInitQty);
        if IsHandled then
            exit;

        if (ShouldInitQty) then begin
            InitOutstanding();
            if IsCreditDocType() then
                InitQtyToReceive()
            else
                InitQtyToShip();
            InitQtyToAsm();
            SetDefaultQuantity();
        end;
    end;

...

    [IntegrationEvent(false, false)]
    local procedure OnBeforeInitQty(var SalesLine: Record "Sales Line"; xSalesLine: Record "Sales Line"; IsAsmToOrderAlwd: Boolean; IsAsmToOrderRqd: Boolean; var IsHandled: Boolean; var ShouldInitQty : Boolean)
    begin
    end;

This request does not have sense as You can set IsHandle parameter of OnBeforeInitQty to be TRUE and skip further code in procedure.

@AleksandricMarko We don't want to skip the procedure, we want to add a additional parameter which forces the execution of procedure (not only Quantity or Quantity (Base) is different) and we want to avoid that we have to copy the whole function.
There are also a lot of more examples where such Should-parameter exists in Eventhandling. Please check f.e.  the OnCheckSalesDocumentOnAfterCalcShouldCheckItemCharge in COD80.

Hi @SKeiler
Now I understand purpose.
Will be implemented.


The data img_align_celeba.zip is not zip file.

The data img_align_celeba.zip is not zip file. Instead, it's a HTML file.
When I run python download.py mnist celebA, it successfully downloaded the  img_align_celeba.zip but the file format is incorrect.

If you have some trouble to get the data, you can just use the original data here!  :)

Use this :https://drive.google.com/drive/folders/0B7EVK8r0v71pTUZsaXdaSnZBZzg?resourcekey=0-rJlzl934LzC-Xp28GeIBzQ
download and put it under the /data/.
finally run "python download.py celebA" again.

您好：       来信我已收到，我会尽快回复的。


More info / Feature request

Is it possible to change the size or colour of the icon I found this https://github.com/FortAwesome/vue-fontawesome#why-use-the-concept-of-a-library can we do it in our example. If yes then please add it in readme.
If not then its a feature request

Hi @prashantnirgun yes, we can change color and size, the readme has already that info here https://github.com/emiliogrv/nativescript-vue-fonticon#using


Add support for using featureKey with entitlements

Overview

adds support for using feature key when interacting with entitlements
stores UsagePeriod

Fixes #(issue)
Notes for reviewer

implemented in another PR


Unveiling the Hidden Economy of Open Source Software

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

/kind event

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area community

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

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nigel-falco
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/close
as obsolete

@LucaGuerra: Closed this PR.

In response to this:

/close
as obsolete

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


add an optional key parameter

what if the user wants to use a specific url key? this makes that possible. i'm still pretty new to golang, so this might be ugly.

Thanks for your participation. I'll have a look at your PR and provide feedback as soon as possible.

i see what you're saying. how full-fledged do you want this app to be? will there be authentication? oauth2? i've been playing around with other little example apps, and i'd like to do something along those lines, but i understand if you want to keep this simple.
since there is no user handling right now, i think we should always return the same key when asked to shorten a url without a pre-supplied key. so your desire to mark pre-supplied keys is a good one. have you been working on that? i wouldn't mind trying my hand at it today, but i wasn't going to add user auth.

One simple solution to our problem is that we could just omit storing user-supplied keys in the "invshorty" Bolt bucket (https://github.com/makkes/shorty/blob/master/db.go#L59). This way the check whether the URL already exists in our DB would be negative in this case. I haven't thought this till the end, though but it should work. You're free to elaborate on this.


Cant Use in WatchOS 10

Hey @dickysuryos, the library hasn't been considered for or tested on watchOS. If you would like to try adding watchOS support I would be glad to consider it, but admittedly it is currently out of scope for my work.

Opened a PR with watchOS support here https://github.com/EmergeTools/Pow/pull/75


Added xSQLAlias resource

Have added the xSqlAlias resource which I developed for xSqlPs but am
merging my changes into xSqlServer as it seems to be more actively
maintained.

Hi @Matticusau, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Matt Lavery). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

It is required that you provide adequate test coverage for the code you change. Contribution Guidelines -Adding test coverage for DSC resources  Please work with us to come up with a plan to cover this code.

Hey @TravisEz13 i am confused because in my other contribution you mentioned i needed an empty line at the end of the file yet Appveyor fails the test when that exists?

This failed because the code cannot merge, not the new line at the end of the file.

@Matticusau Is this PR duplicate?

No this is the same resource that I contributed to xSqlPs but as there is no functionality for this in xSqlServer either I thought I would share the code between them (also seeing as it seemed like I would never get it merged into xSqlPs). There is also some community discussions around the fact that we have two SQL resources, what are your thoughts?

As both resources already exist, they should be maintained in parity, or one should be deprecated, the other taking over the functionality (with a warning added to  start using the other.)  If we maintain both, I would suggest creating a 3rd common PS module which both package and use to implement the functionality.

Please resolve all merge conflicts.  For more information see GitHowTo - 30. Resolving Conflicts
Instructions to setup BC4, my favorite merge tool
If there is no update of the pull request within 7 days, we will close the pull request.  After that, if you have an update, feel free to reopen the pull request or a new pull request.  Please consider closing and re-opening when ready for review.

Hi @TravisEz13 thanks for the message, I will work to have this resolved today/tomorrow. As you know we have had other internal discussions around all of this so I will review the current pull request in line with those discussions to make sure we are heading in the right direction.

Hi @TravisEz13 thanks for the message, I will work to have this resolved today/tomorrow. As you know we have had other internal discussions around all of this so I will review the current pull request in line with those discussions to make sure we are heading in the right direction.

A couple of blocking issues in the comments.  I didn't see any other major issues.

If there is no update of the pull request within 7 days, we will close the pull request.  After that, if you have an update, feel free to reopen the pull request or a new pull request.  Please consider closing and re-opening when ready for review.

@travisEz13 I am going through these conflicts you have mentioned this weekend

Reviewed 1 of 4 files at r1, 2 of 2 files at r2, 1 of 1 files at r4, 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 8 unresolved discussions.

Comments from Reviewable

Reviewed 4 of 4 files at r6.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@Matticusau: Any updates on this PR? If not, I'm going to close it - if there are any new updates it can be reopened

These changes are now included in #89


[Question] A code question about getting yarn task state

Recently, I looked at the code of the job status on YARN after the job was submitted and got the job status on the YARN.When I saw incubator-dolphinscheduler/dolphinscheduler-server/src/main/java/org/apache/dolphinscheduler/server/worker/task/AbstractCommandExecutor.java , I found a method named isSuccessOfYarnState(List appIds).Why here is a list of appid?I think through log file to get state is one-to-one with yarn app.I really hope to get the answer, thx!
最近我在看作业提交到yarn之后,获取yarn上作业状态的代码.当我看到incubator-dolphinscheduler/dolphinscheduler-server/src/main/java/org/apache/dolphinscheduler/server/worker/task/AbstractCommandExecutor.java这个类时,有个方法叫isSuccessOfYarnState(List appIds).为什么这里会是一个appid的列表?我理解的通过log文件获取当前app状态应该是一对一的关系.非常希望得到解答,谢谢!

Which version of DolphinScheduler:
-[dev-branch]

because shell task can submit many yarn task，the task runtime log will return many application ids

因为 shell 任务类型可以写多个 MR 任务，这样的话，这个shell任务的运行日志里会返回多个 application id


Add OpenAPI definition for the API

Requested in https://twitter.com/PermittedSoc/status/1646389772577824768

Happy to help review / collaborate. Thanks.

metoo!

Thanks for the feature request. Can I ask about the intended use cases? Is it so that API client code can be auto-generated?

Partly, mainly for ease of consuming / validating the documentation. I run the Apis.guru OpenAPI directory. We could also convert from a Google discovery format file.

OAS can be used for various purposes, including:

interactive documentation purposes: it can be rendered by UIs using a standard presentation format (e.g. se editor.swagger.io)
client code generation
data model validation
catalogs.

Thanks for asking, R.

Please allow me to provide some justification for this request.
I just spent an afternoon manually converting https://docs.deps.dev/api/ to an OpenAPI definition.  I was typing the whole thing by hand using Swagger Editor.  This was very tedious, especially coding all the JSON schema type definitions.  The document was a ~625 lines when done, also including the descriptions for each operation, parameter, and JSON Schema type.  I am also not sure I got the "format" right on everything.  Things that are a "number" I assume are probably "type: integer" and "format: int32" but this is just my guess.  Additionally, some strings are URLs which where I initially used "format: uri" but I found that the "links" returned in GetVersion sometimes fail to parse as such.
In my case, I was doing this for my own client code generation.  This also simplifies stubbing out or mocking response objects when writing unit tests.  There is a bit of tooling already available for OpenAPI and JSON Schema.  So, I do not really need a special client library for an API when all I really need is an OpenAPI definition that describes the API.

I just spent an afternoon manually converting https://docs.deps.dev/api/ to an OpenAPI definition.

Mind sharing that work somewhere? Thanks in advance!

Mind sharing that work somewhere?

deps_dev.yaml.txt

A long overdue update on this issue!
Having a formal API definition is certainly a good thing. For us, as a Google team, the gRPC service definition is it. While our API is accessible via gRPC and HTTP, it's actually a single implementation under the hood, with the latter transcoded to and from the former. The API documentation is also generated from the same definition. This is how we ensure it all stays in sync.
Additionally providing an OpenAPI definition for the HTTP API would be nice, but since OpenAPI is not part of our setup, we'd need to have a reliable way to generate it from the gRPC service definition, including regenerating it when needed, and be confident that it's correct and won’t drift over time. We did experiment with this, using protoc-gen-openapi, but unfortunately found that it did not generate correct output.
Fwiw, we have just updated the published gRPC service definition to include HTTP annotations, which define how gRPC methods are mapped to URLs. This means that it's now possible for anyone to generate HTTP client code from the gRPC service definition alone.


Automatic Platform Assigning

Be able to select multiple platforms at the same station for a train line and have the train pick between these platforms which one to use (pick an available one). So if you have a station with 4 platforms and you select platforms 1,2,3 the train will arrive in one of those, lets say platform 1.  Then another train arrives before the previous one leaves now it can pick between platforms 2,3 due to platform 1 already having a train.

Suggested way to many times

Duplicate of #537


Saving Settings Issue

I use version v1.1.8 of miniflux and everytime i want to save the settings i have to enter the second password so that they match. I think this is a bug or is it by design? I think it should save the settings without entering the password. Otherwise the settings will not be saved.
I reistered to hosted miniflux to test it there and there i have not to enter the password if i save the settings. So this seems to be a problem.

You can update the settings without entering each time the password. Leave both fields empty. Make sure that your browser doesn't fill automatically the form with the saved password.

i tried it and yes it works. t's a little bit bad that the browser automatically fills in the password each time.


UI for Azure China AKS Integration Support

UI ticket for https://github.com/rancher/rancher/issues/20881

For AKS clusters, add two options in the region list: China East 2(chinaeast2) and China North 2(chinanorth2) which are the current supported two in Azure China Cloud. When they are selected, pass the following fields in azureKubernetesServiceConfig:

baseUrl: https://management.chinacloudapi.cn/
authBaseUrl: https://login.chinacloudapi.cn/

UI should disable HTTP Application Routing configuration for these regions as it is not supported there.
UI should talks to the above endpoint when validating credentials for Azure China Cloud.

For reference: https://github.com/Azure/azure-cli/blob/24119559d134d9e6bd2317c3af4c28847b20de9f/src/azure-cli-core/azure/cli/core/cloud.py#L177

References for testing:
https://docs.azure.cn/zh-cn/aks/kubernetes-walkthrough
https://rancher.com/docs/rancher/v2.x/en/cluster-provisioning/hosted-kubernetes-clusters/aks/

I'm sorry @gitlawr @loganhz for the late notice. I tried and we aren't allowed to create AKS in these regions. Is this something your team might be able to validate?

sure. thanks

Version: master master-head (8/29/19)
Passed the following test cases:

Ensure that the cluster can be successfully created in China North 2 and China East 2region
Create a workload in the cluster and create a nodeport to ensure access to the workload via nodeport
Ensure the L4 Balancer and storage class are available
Modify the cluster


Create palette from list of Hex codes

It would awesome if there was a command for "Create Palette". You could provide an input dialog and ask the user to enter in the hex colors (comma separated) and then load those into the document colors. You can manually do this by modifying the exported .sketchpalette file but having this built-in would take it to the next level.

I think this could be handy, but I'd really like to keep the plugin as simple and straightforward as possible. And since I have other features planned (like supporting other color formats, and global colors), I want to make sure the plugin and its menu don't get too complex.
It seems like you're savvy enough to just edit the file, but I could whip up a separate script for you to just do that if you're interested.

In case anyone is interested, I added this functionality in a forked version for the meantime. It only has the basic functionality, see this commit for details.


connection impossible after compilation

Hello,
I've compiled with success the c# API and put the given dll in the debug directory of the c# sample.
Now the program launch without error but it is impossible to connect to my X-IM3 with "L. UsbConnection"
The COM port is listed by the sample "E. GetPortNames"
The IMU is well connected, I can use it with the GUI. If I force the COM to the value given by the GUI I don't receive any data.
Is there a tip to make the detection possible ?
Thanks

I just tested the UsbConnection.cs without issue.  I had to first download the Windows MSVC files as instructed by this README.  I am using Visual Studio 2022.
Please can you tell me what x-IMU3 software and firmware versions you are using.  Are you able to share a video demonstrating the x-IMU3 working with the GUI but then not working with the C# example using the same COM port?

I've downloaded the MSVC file to compile the dll.
here is a link to view the video

Thanks

Please can you tell me what x-IMU3 software and firmware versions you are using.

All last version
firmware 2.4.0 updated today
GUI 1.5.4

Thank you.  It is not clear why the x-IMU3 GUI would work but the C# example would fail, especially given that the C# example is able to see COM19.
Are you able to test the equivalent C, C++, Rust, or Python example?  Please share your findings as a video again.

tested with the usb_connection script in python and it works fine the data are updated correctly

Here is my c# project, maybe it is corrupted or the dll is not compiled correctly.
Test it like this and with your dll
x-IMU3-Examples.zip
Thanks

I thought it could also come from ID of the COM port over 10 sometime it make some trouble.
I've tested with a serial port number at 3 on another USB port . The result is the same

I ran your x-IMU3-Examples.exe and I get the same issue.  I do not know what the problem is with your compilation.  I suggest you clean your local repository and start from scratch.  There are only two steps:

Download the missing library file as instructed here.  That is, download ximu3.lib (Windows MSVC) and put these files in the local directory, x-IMU3-Software/x-IMU3-API/Rust/target/release
Open x-IMU3-Examples.sln and click run.

Now it works don't understand what I've missed last time.
Thanks for your help.


Add CNP

This is essentially an attempt to translate CNP in PyTorch in Jax.
Currently a draft.

Codecov Report

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

@@          Coverage Diff           @@
##             main     #14   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?       6
  Lines           ?     119
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?     119
  Partials        ?       0

Continue to review full report at Codecov.

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

Thanks @Yash-10. I'll review it over the weekend


Fix failing navmesh for static meshes

PR Details
There seems to be a common error that I think comes from GameStudio. This error makes references not available when processors attempt to reference them in code.
This is a band-aid fix that should resolve issues with reading static mesh data that's not available.
A bit of added info, this seems to only affect procedural meshes.. not sure why that would be.
Related Issue
https://github.com/stride3d/stride/issues/2326
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have built and run the editor to try this change out.

Some more research may be needed here. This got rid of the first hurdle but now it seems to crash when a second static mesh collider is added.

Cool, I'll mark this as wip then


Initialize Module

what
Terraform module to setup CloudWatch Alarms and autoscaling policies for ECS Service.
why
Automatically scale up/down based on performance metrics.

@osterman ready to get a final review on this one - added link to the example


Fix regression regarding certified-conformance mode

When the mode is set as certified conformance mode, the skip value
does not get properly reset and as a result skips the disruptive
tests.
Fixes: #1388
Signed-off-by: John Schnake<EMAIL_ADDRESS>

Woohoo :100: Thanks for the changes!
@johnSchnake


specifications: Architecture overview clarification

Fixes #80
Fixes #81

cc @ ozkoyuncu

Lgtm with minor fixes

@rsahita Let me know if the changes look good to you.


main.ArtistController.java class is missing

implements ArtistControler

Please check your createArtist routine (and for this matter, all routines in your controller). You are not setting the right HTTP status response code. See
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes
for reference.
Cheers,
Naza

@nmaguirre I have a question. HTTP status code must be in bootstrap or SongController?

Wherever you handle the construction of the response object.
Cheers,
N


Add check to make sure all output suffixes are present for all input words

This applies a check when learning IO systems to ensure that all input words have the necessary output suffixes in the DT. This fix applies to both SLDT and SLLambda.

Shall we merge this PR, or is there any reason to wait?

I see no reason why it shouldn't be merged


PlatformClean script in package.json uses rm -f

the clean script in package.json refers to rm -f which will not work on Windows based machines not using bash/csh

Good catch @erwinvanhunen. We could replace it with a simple x-plat node script that would call the right command depending on the os. Just to check, on Windows, should it be calling rd /s node_modules?

Here is some more information how this could be solved: https://stackoverflow.com/questions/45082648/npm-package-json-os-specific-script

Let me take a stab at it


ENH: Added Proj.get_factors & made Proj subclass Transformer

[x] Closes #533
[x] Closes #503
[x] Tests added
[x] Fully documented, including history.rst for all changes and api/*.rst for new API

The factors only work on a Proj - like object. So, should keep Proj around.

Need to do more investigation to ensure this is done properly.


Unable to authorize client

Despite making a file containing my token as ~/.config/slack-term/config/.slack-term, I get this error:
2022/01/24 18:24:59 not able to authorize client, check your connection and if your slack-token is set correctly
This is my file:
{
    "slack_token": "mytokenwashere"
}

What is the issue?

Version: 0.5.0
Installation method: sudo snap install slack-term

Same issue, the Docker container works and can connect to my Slack account,  but the binary does not. My config file is located at the following per the README:
~/.config/slack-term/config\

I experience the same issue but can get it to work if I specify the config location with the -config flag: slack-term -config ~/.slack-term or slack-term -config ~/.config/slack-term/config.

Update: I apparently had comments in my config file (JSON doesn't support comments and neither does the app).
I got this to work:
sudo docker run -it -v ~/.config/slack-term/config/.slack-term:/config erroneousboat/slack-term
But this still does not work:
slack-term -config ~/.config/slack-term/config/.slack-term
I will happily use the Docker container though.

I wanted to test slack-term and I get this error too.
First attempt with method 4 (window.promt() trick, very clever :) ), then method 2 https://github.com/erroneousboat/slack-term/wiki#method-2 . I got two different tokens, and neither of them worked :(

$ docker run  -sh -v /home/[username]/.config/slack-term/config:/config erroneousboat/slack-term
2022/04/27 08:45:26 not able to authorize client, check your connection and if your slack-token is set correctly
$ ./slack-term-linux-amd64
2022/04/27 10:46:44 not able to authorize client, check your connection and if your slack-token is set correctly

(I also ran docker run -sh -v /home/[username]/.config/slack-term/config:/config --entrypoint /bin/sh erroneousboat/slack-term to check that the config file was mounted correctly in /config with the token, and that I could ping servers on the 'net from inside the container - just in case. All seemed fine.)

First, the default config file is in ./config/slack-term/config (its a file, not a directory).
When installing it via snap, that config file is stored in:
~/snap/slack-term/current/.config/slack-term/config
(you can confirm as it is created automatically).
You need to edit that file and add your token there.  You can also add other config options as specified.
When updating slack-term that file is very likely to be removed, so keep a copy somewhere.

Facing the same issue when using docker and from my local. My config is set correctly,
{
  "slack_token": "mytoken"
}

➜ slack-term -config ~/.config/slack-term/config
2022/06/22 08:28:25 not able to authorize client, check your connection and if your slack-token is set correctly

Anyone still facing this problem?

Yes, I did some digging around in the code, and it turns out this is correct.
The library that slack-term uses to interface with the official API is slack-go, but that (as far as I can tell) does not support this particular authentication method (normal token + xoxd token in Cookie header).
After cloning this repo, and cloning slack-go, and then forcing slack-term to use my local version of slack-go, and then modifying slack-go to shove my own personal xoxd token into the Cookie parameter in the headers of the API request, I was able to get slack-term connected.
So, in summary, it's an issue with the API library that this project uses. Not this project itself, necessarily. I'm going to try and work on an MR to submit.
I'm not sure why this doesn't seem to be a bigger issue for more people. Some comments in various threads on this problem mention that it's only newer workspaces. That seems like the most likely explanation.

I'm affected by the same issue.

I was able to bodge together a working proof of concept together by copying the chttp implementation from slackdump, which was referenced in the last comment of the above linked issue 1092
If it ends up meeting my needs, I'll try to put together a more proper implementation that takes the cookie as a config value rather than hardcoding it, though I don't really know Go so we'll see.

I have this problem too.

@thomas-scrace sorry for the slow reply. I have a working version here in my fork. It's still messy but you can specify the additional token from the config value slack_xoxd_token and it's been working for me all summer.
I had to do some funny stuff with dependencies to make it work that I don't really recall the details of any more, probably not all up to snuff at all, but if someone who knows how Go works wants to clean it up be my guest.

^ ceandreasen's fork works.
Instructions on how to get xoxd token on this other project's readme:  https://github.com/NoF0rte/slack-slurp


[FR] -  Prevent block-producer node to start with Invalid KES (InvalidKesSignatureOCERT)

Internal/External
Area
cardano-node
Describe the feature you'd like
Curently a block-producer node can start with an invalid KES signature. The only way to know if the KES signature is valid is to make a block.
Describe alternatives you've considered
Cradano-node simply should not start with invalid KES signature.
This will avoid missing block caused by OcertFailure (InvalidKesSignatureOCERT 192 189 3 "Reject"))

+1 on this issue. It causes small pool operators to miss blocks. If it can be detected on startup it will be great to add this validation.

This would need your node to be in sync (atleast to a point where previous block created by your node is in DB), not something you can assume when you start the node. Instead, the alternate solutions could be using this information from public / private hosted dbsync when you actually rotate the key.

@rdlrt Ya I read that thread and commented. It looks like what happened to us is we missed updated the kes file but updated the node.cert. Its not just us it looks like this was also documented here https://www.tiredpixel.com/2020/10/06/crab-cardano-stake-pool-botched-kes-key-rotation-incident-report/
Do we know how we can decode the Op Cert column in the Blocks table? I see that our block has this Op Cert value and I would like to decode it to see the counter to validate if this is the issue. I got this data from caradno-sycn-db.
5721125	d2eb97f83fed36a42a4893992bcb57533fd13891815a106873b24a3a699572eb	265	29,512,883	396,083	5,720,948	5721124	Null	5721125	7,884	5/15/2021 11:46:14 AM	18	4	0	vrf_vk12553zj6px5lklma89sxatl9w82a28u6e84zgnufckpy5a9jvrqvsudg0f9	3724b02d0e5775c65e199eb33f278b95fb23f03616e60f9cad368f1022b42788```

Do we know how we can decode the Op Cert column in the Blocks table?

It is only available in dbsync post this commit, and available as an integer


Problem with latest version(ES6 not work in browser)

Sided lib pulling punycode.js. In webpack setting for babel-loader - all external lib ignore for babel. And I have problem with punycode lib: he return error don't parse this string:
const ucs2encode = array =>String.fromCodePoint(...array).
I resolve problem: add to my project old version punycode("punycode": "~1.4.1"), but it's hack, because for my project this dependency not needed. How fix it?

[…] but it's hack […]

It’s not a hack.
From the README:

The current version supports recent versions of Node.js only. For the old version that offers the same functionality with broader support, including Rhino, Ringo, Narwhal, and web browsers, see v1.4.1.


first spike of fabric8:setup

looks good ! (except some minor things, see comments). Really nice for a quickstart ....

@rhuss you happy with this PR? would love a release soon...

@rhuss gonna merge this one for now - we can always keep polishing later - its damn useful as is!
LTGM [merge]

Will have a look this morning
James Strachan<EMAIL_ADDRESS>schrieb am Do., 8. Sep. 2016,
09:58:

@rhuss https://github.com/rhuss gonna merge this one for now - we can
always keep polishing later - its damn useful as is!
LTGM [merge]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8io/fabric8-maven-plugin/pull/302#issuecomment-245522318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGDCHmhbecfD4u5s68tkMpWBspHPzKpks5qn8AygaJpZM4Jz7g1
.

no problem, will have an eye on it later ;-)


Fix typo in VaultKeyValueBackendProperties

Fixed the typo

@saikatsgupta Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@saikatsgupta Thank you for signing the Contributor License Agreement!

Thank you for your contribution. That's merged and backported now.


Nightly builds, GitHub "releases"?

https://github.com/readium/readium-js-viewer/releases
Related issue:
https://github.com/readium/readium-js-viewer/issues/89

Releases are produced at every checkin:
https://github.com/readium/readium-js-viewer/releases
See:
https://github.com/readium/readium-js-viewer/blob/develop/build-config/deployCrx.js#L22


fixed fedora collection presentations without proxy order

Coverage decreased (-0.7%) to 88.836% when pulling 8628ed4ce0b3d851794f68e02fc5e879c83bcc20 on fedora-collection-presentaiton into 88308ad68458a5bd06625428e16a67e200e534ce on master.


docs: Add missing space to compound verbs “back up”, “log in”, etc.

Noun: backup, login, logout, lookup, setup.
Verb: back up, log in, log out, look up, set up.
(Followup to #16528, #18265.)

This lgtm; feel free to merge after resolving the merge conflicts with your other recent wording PR :).


annotation property: EUPATH_0000274 display order

The label of term in the clinEpi terminologies is 'displayOrder'.
Proposed to add the alternative label: displayOrder for the term.
Other annotation associated with the term are:
definition: The order variables will appear in under a given parent term. Left blank if alphabetical order is accepted
comment: annotation used for specifing the order of terms displayed on the EuPathDB website
Proposed to delete comment and update the definition as:
The annotation is used to specify the order of terms displayed on the website.

"An annotation property used to specify the order of terms displayed on a website."

I'm removing the comment and updating to Chris's definition in the release candidate.


Missing --dd option in Rust rewrite

The --dd option to extract everything to a directory shown in many online tutorials is now missing in the Rust rewrite.

The --dd option in Binwalkv2 is now largely replaced with the --carve option in Binwalkv3. They are not exactly the same, but the --carve option in v3 does effectively the same thing that --dd='*' did in v2, namely, carve out all identified file types to their own separate files on disk.


Add OIDC components to the provider

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Setting up OIDC with Pulumi Cloud can be difficult. It would very helpful if we included resources such as DeploymentsAwsOidcConfig, DeploymentsAzureOidcConfig(different name/scope because of how Azure does OIDC).
Off the top of my head, it could even be something minimal like:
const awsDeploymentsConfig = new pulumiservice.DeploymentsAwsOidcConfig("deployments-aws", {
  stacks: Stacks.All, // Would create the proper wildcard for all stacks and all projects in the org
  operations: [PulumiOperation.Refresh], // e.g. if doing drift detection
  sessionName: "pulumi-deployments" // This would be the default value
});

const deploymentsRole = new aws.iam.Role("deployments-role", {
  assumeRolePolicy: awsDeploymentsConfig.assumeRolePolicy, // That is, the correct subject claims, etc.
  managedPolicyArns: ["arn:aws:iam::aws:policy/IAMReadOnlyAccess"]
});

const awsEscConfig = new pulumiservice.EscAwsOidcConfig("esc-aws", {
  environments: ["env1", "env2"],
});

const escRole = new aws.iam.Role("esc-role", {
  assumeRolePolicy: awsEscConfig.assumeRolePolicy,
  managedPolicyArns: ["arn:aws:iam::aws:policy/AdministratorAccess"]
});

By only outputting the assume role policy, we avoid having to mix in the AWS provider as a dependency.
Also keeping a constant in the Pulumi Cloud provider SDKs with the value of PulumiCloudOidcFingerprint could be helpful since getting the fingerprint is annoying if you follow AWS' instructions.
I can help with the component design or feedback since I've set this up a few times. Please loop me in if there's serious consideration for implementing this.

Affected area/feature

Would it be ok to just have an API endpoint that's hard-coded with the certificate thumbprint?

Yeah we could do that.

Looks like I would just manually edit schema.json?

Yup! I experimented with some Invokes for Deployments earlier this year, if you want some prior art to work off of. https://github.com/pulumi/pulumi-pulumiservice/pull/164


Move headers to their own package.

We aren't really into deprecation yet, but in light of the growing user base and magnitude of this change, I did so here.
Closes #173.

This was heroic work. I don't see any problems. I'm +1 to merge.

That's a casual use of the word heroic. I think If go with merely unenviable. :-)


BUG: Few Navbar icons are unnecessary placed here.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
https://github.com/Anushkabh/krishiconnect/assets/119615342/99deb2c2-4308-45be-ac80-a060343b6b6f
Overlapping each other and re-appear once scrolled up
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hi there!
Thank you for opening this issue! 😊
I just wanted to check in and see if you need any help or guidance with this issue. If you have any questions or run into any challenges, please feel free to ask. I'm here to assist you!
Happy coding!


URA Error: the "filename" option is required to use includes and extends with "relative" paths

extends layouts/default
include components/main-page/main-page

URA Error: the "filename" option is required to use includes and extends with "relative" paths

What should I do?

My configuration is
    {
        test: /\.pug/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'file-loader',
            options: {
              name: '[name].html'
            }
          },
          {
            loader: 'pug-loader',
            options: {
              pretty: true
            }
          },
        ]
      }

I did add the option
root: path.resolve(__dirname, 'src')
and did change the paths to
extends /layouts/default
include /components/main-page/main-page

Now the error is

URA Error: the "basedir" option is required to use includes and extends with "absolute" paths

Oh, god! I used the fork. It has the different behavior.

Ok, so there is another error with paths:
ERROR in ./src/index.pug
Module build failed: Error: Cannot find module '/Users/studentIvan/projects/frontend/src/node_modules/pug-runtime/index.js'
Why it searching node_modules not in webpack config directory?

I was trying to use the branch #runtime-url and it works. Can you do it stable and push to npm?