crosslink: Support go.work

Why
Towards https://github.com/open-telemetry/opentelemetry-go/issues/3964
go.work is honored by gopls language server so that develoeprs get better UX in tools like VSCode, Vim, Emacs etc
What
Make crosslink to update the use in go.work file (if it exists).
Take notice that it will not harm any repositories which do not use Go workspaces. However developers could still use the functionality during development 😉
PS. The tests have a lot of repetition . However refactoring it in this PR would make it very hard to review. I can refactor all crosslink unit tests in next PR.
Testing
You can check the result on this branch and on https://github.com/open-telemetry/opentelemetry-go/pull/4083

Also, should we support the use case of creating a go.work file if it does not exist? Why should having a preexisting go.work file be a prerequisite?

Is there a scenario where you would only want to update go.mod files and not go.work files? What about the other way around?

I was thinking about it as well and I have not come up with such scenario.

Also, should we support the use case of creating a go.work file if it does not exist? Why should having a preexisting go.work file be a prerequisite?

I was thinking about it as well. For sure it should be an opt-in functionality as somebody may want NOT to use Go workspaces.  I found following reasons why I did not want to add such functionality (at least in this PR):

Adding a new go.work file is simply done by running go work init. I do not think it is worth to make crosslink more complex is creating the file is so easy.
We would probably need to add a logic for "guessing" a proper Go version. For instance in this repo I had to manually change go 1.20 to go 1.19
YAGNI 😉 This can be always added later

Is there a scenario where you would only want to update go.mod files and not go.work files? What about the other way around?

I was thinking about it as well and I have not come up with such scenario. I do not see why anyone would like to update only go.mod without updating go.work and vice-versa. Doing it separately may make the repo "out-of-sync".

I see a use case. Someone may only use go.work and not care about having replace in go.mod files 😉 Do you have any proposal how the CLI should look like? Personally, I propose to add flags to ignore processing of go.mod or/and go.work files. Then by default user can still update both type of files using one command. Also prune could still support both type of files.
Personally, I propose to add it as a separate PR to make it easier to review. Also I do not think it is a blocker as this can be always added on top of existing PR.

@bryan-aguilar After longer thoughts I am considering to change the code to have a new gowork command. Then the command could generate a go.work file if it does not exist. What do you think?

@bryan-aguilar After longer thoughts I am considering to change the code to have a new gowork command. Then the command could generate a go.work file if it does not exist. What do you think?

I think this makes sense. My suggestion would have been very similar. I would expect the cli interface to be something like crosslink gowork. I am sure there will be some flags that are shared between the root and gowork command.

Moving to draft. I will change this PR to introduce a work command. ETA Monday/Tuesday.

Closing this PR in favor of https://github.com/open-telemetry/opentelemetry-go-build-tools/pull/311
CC: @bryan-aguilar @hanyuancheung


WIP remove printers_include

do not submit yet, still testing
Commit Message: Remove the "printers_include" library in favour of "printers_lib".
Additional Description: The partial, header-only library :printers_include provided no service that wasn't already subsumed by :printers_lib.
Risk Level: low
Testing: in progress
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

In this present draft, I'm removing :printers_include from the Starlark rule envoy_cc_test_library, for the following reason: the alternative is to change :printers_include to :printers_lib, as indeed many tests are using that dependency. However, a large number of tests are already directly depending on that rule, and simple duplication would be an error. The alternative would this be to remove the dependency from a large number of targets.
The present proposal is a bit simpler, since we just need to add the missing dependency in a handful of places.
Conceivably one could put more complicated deduplication logic into the Starlark rule, but that would seem to be unhealthy.
@alyssawilk, @qiwzhang: could you please advise on the preferred direction?

I don't have a strong opinion.  @htuch and @lizan for build organization?
Alternately we could go for https://github.com/envoyproxy/envoy/issues/2205 and try to remove this library entirely, but that's a larger change.

I think the only reason to have separation would have been some cyclic dependency, so this makes sense.

@htuch: Perhaps, though in my experience you can never really work around circular dependencies like that -- try bazel build --dynamic_mode=off to make sure. Typically you'd just have to put all the TUs that use the circular dependency into a single cc_library rule.


Pattern47

Related Issue
Closes: #169
Describe the changes you've made ✍
Added the 47th pattern in JS, tried to include the prompt() library function to get user input but wasn't working so i just used 7 as an example number
Screenshots (if any) 📸

@Vishal21121 just saw this, I will start working on it tonight!


doc: Healthcare Analytics Sample

/azp run

Codecov Report

Merging #1333 (610dc28) into master (9e51ba7) will decrease coverage by 80.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1333       +/-   ##
==========================================
- Coverage   84.76%   3.85%   -80.91%
==========================================
  Files         286     257       -29
  Lines       14165   12904     -1261
  Branches      698     698
==========================================
- Hits        12007     498    -11509
- Misses       2158   12406    +10248

Impacted Files
Coverage Δ

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/scala/org/apache/spark/ml/param/UDPyFParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/ByteArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/EvaluatorParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/ParamSpaceParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/com/microsoft/azure/synapse/ml/stages/udfs.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/spark/ml/param/TransformerParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../org/apache/spark/ml/LimeNamespaceInjections.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...org/apache/spark/ml/param/ArrayParamMapParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 268 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e51ba7...610dc28.


BUG Fetch Post For v2.1.1 - v2.5.4

Because Some Blogs Have Differences In Feed Structure
If Had Some Error, Try To Change post[randNum].link[4] To post[randNum].link[2]

I Will Fix It On Next Version


docs: update topic to avoid displaying issue

Fixes: #771
Changes:
Currently, the website displays two identical articles on the first page of blog.
After some researching and testing, I find the cause of this problem. While it is completely fine to post multiple articles in the same date directory, the title of articles under the same directory must be identical and unique.
In other words, when adding two or more articles in the same date directory, you need to make sure each article's title must be different.
This problem is caused by two articles with the same title.

After updating the title of an article, then yarn start run it locally, it works out fine.

In addition, I also checked the files in apisix-website/website/i18n/zh/docusaurus-plugin-content-blog/2021/11/23/ directory and run it locally. It works fine, so there is no need to modify files in this directory.
For future changes, if you are adding two or more articles in the same date directory, please make sure they have different title. Use yarn start to run it locally before submission is also suggested.

Good catch!! This may be a bug from Docusaurus?

Good catch!! This may be a bug from Docusaurus?

Well, not necessarily.
It is most people's first instinct to look at file names when the above problem occurs.
File name is used to generate urls, thus duplicated file names is prohibited from the very beginning, i.e. creating two files with the same file names in the same directory is not allowed.
But if you take a step back and have a look: in what kind of scenario would a person create two or more articles with the same title? Basically none except done by accident.
With this being said, each article has a unique title is a very common-sense thing. I would rather call this a human-like thinking logic than a bug.

I would rather call this a human-like thinking logic than a bug.

😄


Fix chunked transfer-encoding with preload_content

When we implemented our own chunked transfer-encoding logic, we broke
the use case (that we did not anticipate) of people not specifying
preload_content=False to stream their data. This provides a fix, sans
tests, for that use case.
Closes gh-907

Note:

I'm not sure we want to fix this case (see the issue).
I didn't write tests, I just wanted to see if I could work in a fix quickly before work.
An alternative, that forces people to use preload_content and avoids this is to not set chunked to True when preload_content is also True. I'm just not sure which behaviour we want.

Hmm can you expand on the alternative? Is avoiding using chunked when not preloading allow all the necessary scenarios? (Keeping in mind that we do want to make preload_content=False by default sooner than later)

@shazow so we have logic that checks self.chunked and doesn't check self._body so if you make a request to the site that is in gh-907, and instead do r.data (instead of r.stream()) then you'll get the same error. So what that says to me, is that if preload_content is True, we should probably set self.chunked to False because we're going to preload the content and make it not chunked. That alternative is valid regardless of the default value of preload content now or in the future. It's a preventative measure for a foot-gun.

If I understand this correctly, you're saying that chunked responses should not be streamable? Or that the distinction of chunked vs not chunked is meaningless when we're streaming anyways?

It's not just chunked versus not chunked when streaming. It also requires preload_content. If you make a request with the URL that was in the original issue, with our current implicit preload_content=True to a service that returns a chunked response, then we will happily preload that content for you. The problem is that if you use r.stream() (which people shouldn't if they don't pass preload_content) you get the AttributeError. I'm not sure if we want to refuse to stream in this case or if we want to never let self.chunked be True because that branch of logic is what causes this error.

Er sorry I was using stream interchangeably with read-on-demand (as opposed to preloading).
If we can avoid the branching, all the better, but it's still not clear to me which scenario we're sacrificing (if any?).

I think the confusion is generated by my original comment. Let me clarify with what I understand right now:

We used to allow people to request a chunked URL without preload_content=False and allow them to use response.stream()
We broke this because while we still pre-load the content, we also set response.chunked = True which causes us to try to read from the original response

It seems like this is very rare behaviour, but I don't think we want to continue having an AttributeError be raised when someone calls response.stream() in some cases. This PR as it is now does not really do what we want (I don't think). The question becomes: What do we want the behaviour to be?

I agree that side-effect surprise exceptions are no good. What are the options for the behaviour?
Keeping in mind that ultimately we might want to get rid of preload_content altogether (and force False), especially once we have the contextmanager pattern which closes unconsumed responses on exit.

What are the options for the behaviour?

So we can:

re-implement the old behaviour. This is as simple as setting response.chunked = False if preload_content is True.
ignore preload content (like this PR presently does, although it does more than that) if we're dealing with a chunked response which might not be what the user actually wants
catch the AttributeError in response.stream (or read_chunked or somewhere else) and raise a new exception instead that explains what might have happened?

I honestly don't know what's best here. :) First option seems okay but they all seem okay (the third one seems saddest though).
Paging @Lukasa @jonparrott @haikuginger

Let's do the first for now. We can then adjust that behaviour later down the line when preload_context gets nixed.


Add an Example for InlineKeyboardMarkup Constructor (InlineKeyboardButton[][])

Please add an example for InlineKeyboardMarkup Constructor (InlineKeyboardButton[][]).
TODO (optional): Describe a specific scenario you would like to see addressed.
Help Topic: https://mrroundrobin.github.io/telegram.bot/html/M_Telegram_Bot_Types_ReplyMarkups_InlineKeyboardMarkup__ctor_2.htm

have you tried this ?
 var keyboard = new InlineKeyboardMarkup(new[]
                {
                    new[] // first row
                    {
                        new InlineKeyboardButton("1.1"),
                        new InlineKeyboardButton("1.2"),
                    },
                    new[] // second row
                    {
                        new InlineKeyboardButton("2.1"),
                        new InlineKeyboardButton("2.2"),
                    }
                });


Login with Mobileclient to G-Apps Account with 2FA and app-password not working

Hi,
for a while now, my application is not able to login to gmusic using my g-apps username and a generated app-password. The login to gmail using the account password ist working.
I tested with some python code:
from gmusicapi import Mobileclient
api = Mobileclient(debug_logging=True)
print<EMAIL_ADDRESS>'APP-PASSWORD', Mobileclient.FROM_MAC_ADDRESS)
print<EMAIL_ADDRESS>'ACCOUNT-PASSWORD',Mobileclient.FROM_MAC_ADDRESS)
Output:
root@sbs:~# python test_gmusic.py
/root/.cache/gmusicapi/log/gmusicapi.log
/usr/local/lib/python2.7/dist-packages/Cryptodome/Math/_Numbers_gmp.py:230: UserWarning: implicit cast to 'char *' from a different pointer type: will be forbidden in the future (check that the types are as you expect; use an explicit ffi.cast() if they are correct)   _gmp.gmp_snprintf(buf, c_size_t(buf_len), b("%Zd"), self._mpz_p)
False
True
I tested the app-password with some other app (calender): it works!
The logfile says:
2017-02-10 13:43:57,993 - gmusicapi.Mobileclient1 (shared:74) [INFO]: initialized
2017-02-10 13:43:57,993 - gmusicapi.Mobileclient1 (shared:98) [INFO]: logged out
2017-02-10 13:43:58,848 - gmusicapi.Mobileclient1 (mobileclient:113) [INFO]: failed to authenticate
2017-02-10 13:43:59,605 - gmusicapi.Mobileclient1 (mobileclient:117) [INFO]: authenticated
Is there something wrong with gmusicapi and app-passwords?

Hm, I'm not aware of any issues affecting app passwords.
I noticed that your example uses two different email addresses. Is that intentional?

The example is actually two examples:
one uses gmail (@gmail.com), the other one uses the Google-Apps Account with the app password.
Besides the warning, the example outputs "False" for G-Apps and "True" for gmail.
So, basically gmusicapi works.
And I verified with another app (calender), that the app-password works.
So why gmusic does not?
You say, that you're not aware. Have you verfied it?

Okay, I did a little debugging:
gpsoauth.perform_oauth
returns
{u'Error': u'DeviceManagementRequiredOrSyncDisabled'}
The reason for this is, that for this google-apps account, the enforcement of policies on mobil clients is enabled in admin console (enforce_android_policy).
If I disable this in admin-console, the authentication works.

Ah, interesting. I'll make a ticket to surface those kinds of errors better.

enforce_android_policy

The setting is available here: https://admin.google.com/AdminHome?hl=no#MobileSettings:section=advanced&flyout=security (took me a while to find it..)

Er...any idea where this setting lives in the Google Admin console nowadays? I can't find it.

Hi @drinkspiller,
Google shut down Google Play Music at the end of 2020. This project interfaced with that product, and therefore is no longer really relevant.
This probably isn't the best place to get support for GApps administration.
Best of luck!


Netstandard

Continuing on from #64

There seems to be intermittent failing tests that I can't reproduce locally :(
Going to merge and deal with CI issue after.

@BennieCopeland Sincerest thanks for this contribution :)


README repeatedly uses "regex_result", without the "s"

In 90335706, a number of examples were added to the README file, but the s at the end of regex_results was forgotten.

Fixed in db3bcf5bd42a1c3c3ed5a98d94729ee26e2ccd42. Thanks!


fix(@angular/cli): add $schema as a schema prop

Fixes "Property $schema is not allowed" error on editors.

LGTM


[SP-4204][PDI-16942]-When using the Excel Input Step an org.apache.poi.POIXMLException error can result if the compression ratio of the file is too high

fixed.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/excelinput/ExcelInput.java
  engine/src/test/java/org/pentaho/di/trans/steps/excelinput/ExcelInputContentParsingTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/fileinput/BaseParsingTest.java
  engine/src/test/resources/org/pentaho/di/trans/steps/excelinput/files/Balance_Type_Codes.xlsx

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.excelinput.ExcelInput

Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.excelinput.jxl.XLSWorkbook

Instruction Change: -3.3% :small_red_triangle_down:
Line Change: -2.6% :small_red_triangle_down:

org.pentaho.di.trans.steps.excelinput.poi.PoiWorkbook

Branch Change: -6.3% :small_red_triangle_down:
Instruction Change: -17.4% :small_red_triangle_down:
Line Change: -18% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:


2.1 release notes

Signed-off-by: Kavitha Conjeevaram Mohan<EMAIL_ADDRESS>Description
Add 2.1.0 release notes
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #676 (22356ea) into main (8751994) will decrease coverage by 31.92%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #676       +/-   ##
=============================================
- Coverage     94.69%   62.76%   -31.93%
=============================================
  Files           281       10      -271
  Lines          7592      658     -6934
  Branches        558      119      -439
=============================================
- Hits           7189      413     -6776
+ Misses          349      192      -157
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

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

Impacted Files
Coverage Δ

...ensearch/data/value/OpenSearchExprBinaryValue.java

...rg/opensearch/sql/sql/config/SQLServiceConfig.java

...rch/planner/physical/MLCommonsOperatorActions.java

...ensearch/storage/script/sort/SortQueryBuilder.java

...expression/function/BuiltinFunctionRepository.java

...earch/storage/script/filter/lucene/RangeQuery.java

...opensearch/sql/expression/ReferenceExpression.java

...rotocol/response/format/FlatResponseFormatter.java

...arch/sql/expression/aggregation/AvgAggregator.java

...ava/org/opensearch/sql/utils/SystemIndexUtils.java

... and 261 more

Continue to review full report at Codecov.

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


initReactI18next error

Describe the bug

react-i18next:: You will need to pass in an i18next instance by using initReactI18next
Your Example Website or App
gitlab repo
Steps to Reproduce the Bug or Issue
on gitlab
Expected behavior
translated components, im seing the error above and no translation
Screenshots or Videos
No response
Platform

OS: windows
Browser: [Chrome, Firefox]
Version: [23.7.8]

Additional context
No response

What error exactly are you getting here? The screenshots don't show anything aside from the yellow warning lines but I'm not sure what the content of the warning is.
At first glance, you are probably not initialising the I18NextProvider context component because you are wrapping it in the then block but not passing the new instance of i18next that you got after your promise was resolved. You are using await already so the then block is unnecessary.
You can just do:
await i18next
    .use(initReactI18next)
	...

startTransition(() => {
    hydrateRoot(
      document,
      <I18nextProvider i18n={i18next}>
		...
      </I18nextProvider>,
    );
  });
}

thanks for the help, i removed the promise, but this is the warning im getting even though i am using .use(itnitReactI18next)
this is the warning:
react-i18next:: You will need to pass in an i18next instance by using initReactI18next

It's a little confusing but if you're using the i18next that you exported from .i18n.server then that's the wrong i18next. If you check the instructions, you need to use the i18next exported by the i18next package.
import i18next from "i18next"

async function hydrate() {
  await i18next
    .use(initReactI18next)
	...


add BSD socket layer

The new BSD socket layer depends on hermitcore/libhermit-rs#633 and a revised version of std. std is part of the rust fork. Note: If you use this version, please modify Cargo.toml.  hermit-abi must point to the version of this PR.

We tested it, but it is actually 4x slower than it was before. :pensive:
Seems to be reliable though…

bors r+

borr r-

bors r+


sensor not managed

Questo errore ha avuto origine da un'integrazione personalizzata.
Logger: custom_components.ecowitt
Source: custom_components/ecowitt/init.py:274
Integration: Ecowitt Weather Station
First occurred: 11 maggio 2022, 22:20:03 (1427 occurrences)
Last logged: 22:30:02
Unhandled sensor type runtime value 274594, file a PR.
Unhandled sensor type runtime value 274655, file a PR.
Unhandled sensor type runtime value 274716, file a PR.
Unhandled sensor type runtime value 274777, file a PR.
Unhandled sensor type runtime value 274838, file a PR.

Hello.
I have exactly the same warning. have you been able to solve it?
Greetings and thank you.


docs: refer to support table for deletionPolicy support

:book: refer to support table for deletionPolicy support.

/approve


luci-base: fix a typo in Russian translation

Signed-off-by: Anton Kikin<EMAIL_ADDRESS>

ACK


How to Resume Training From checkpoint

Hi,
I see the code in train.py

But I am not sure whether the training would resume from the latest checkpoint.
Does it resume from the latest checkpoint?

Hi,
I see the code in train.py

But I am not sure whether the training would resume from the latest checkpoint.
Does it resume from the latest checkpoint?

@syue-fen Yes, the latest one.


Save Cinema Frontend Project with Payment Gateway Integration

Title and Issue number
Save Cinema Frontend Project with Payment Gateway Integration and #1636
Close #1636
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

inactive


v3.2 SAML Docs

Added a WIP label to remind me to figure out where to place a script which helps users generate service provider public/private certificates.
Otherwise it's completed and ready for a review

+1 let me know if you want me to setup a repo for the script to generate self-signed certificates

Tweaked the index.rst, merging this in. Propose using includes as noted above^ (but this edit can be made after it's on master).


Correction for raw affiliation Schulich School of Medicine and Dentistry, Western University

Correction needed for raw affiliation Schulich School of Medicine and Dentistry, Western University
raw_affiliation_name: Schulich School of Medicine and Dentistry, Western University
new_rors: 02grkyz14
previous_rors: 02agqkc58
works_examples: W4360620733;W4390574503;W4386414933;W4380989175
contact: 9bd4d11dd815789836f42c6a3c7eba30:9a061ba1f456218a @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Add support for PROTO-1751

Motivation
See JIRA ticket
Solution
Was able to do what's described in the ticket
Compare the Allow's from /network/options and the asserter configuration file
Open questions
Probably nothing. Using reflect.DeepEqual instead of JSON equality since I don't think the latter is feasible for nested / unsorted types

Pull Request Test Coverage Report for Build 6922

110 of 148 (74.32%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.2%) to 42.013%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/validate_asserter_config.go
98
108
90.74%

cmd/check_construction.go
0
13
0.0%

cmd/check_data.go
0
15
0.0%

Totals

Change from base Build 6853:
3.2%

Covered Lines:
697

Relevant Lines:
1659

💛  - Coveralls

ping on this


refactor(cli): prefix zod schemas and fix comment

fixes https://github.com/latticexyz/mud/pull/507#discussion_r1148944776
and adds z prefix to remaining zod schemas (I'd already started this with internal store config stuff)
the prefix is good for clarity and to avoid having types and schemas having the same name, as they're not referring to the same thing

this needs a rebase now after recent merge


Feature/438 update alert layout

👋 Thanks for creating a pull request! 🚀 Checkout the storybook we've created for it: chlorophyll


scroll in jupytext

@itcarroll Did you figure out how to add the scroll-output at the notebook level?

Yes, I added the tags to the jupytext. You still need both the metadata scrolled=true and tags=["scroll-output"]. That's something that should be raised on MyST-NB. They should read the existing metadata rather than define a new tag, methinks.

Did you have to scroll through the src and add # + scrolled=true tags=["scroll-output"] language="bash" where needed or what there a way to apply that to all bash cells at the notebook level?

Had to do it for each cell. But I could search/replace in the text files, didn't have to use the notebook interface.


PDF Created In Acrobat omits lines

Creating a PDF via Acrobat does not persist drawn lines in a pdf that is created from a Word Doc. It used to work. This does not happen when creating a PDF via Word or other applications, but only with Acrobat.

@flipcents
Are you using Aspose.PDF at your end? Can you please share a bit more details of the issue that you are facing while sharing the code snippet and the sample files. Please note that we recommend posting issues related to the API in our official support forum where you can easily share files and track the progress appropriately against the issue investigation. You can please create a topic there in case you are facing some issues while using Aspose.PDF and we will further assist you accordingly.

@asadalikhan90 it was specific to the last upgrade. Same standard code we always used which always worked fine, all we did was update the library. And it only happened with PDF's created in Acrobat. Real easy to test, Create a document with drawn lines in Word, then create a PDF with drawn lines in Acrobat. One will display the lines one will not in Aspose.PDF
With that said, I posted this issue SIX months ago, hahaha... We wrapped that project up back in February. The business didn't want to roll back the library as rolling it back had some issues with thumbnails as they decided this was less of an impact and we were not getting any feedback from this issue. I don't have access to the code anymore.

Since there's no access to original code, issue isn't described as Aspose PDF related and there's no updates for year long period issue considered as closed


generate does not support Podman

Due to detecting running under Docker via looking for the file /.dockerenv.
Podman doesn't create this file, instead creating /run/.containerenv. See e.g. https://github.com/containers/podman/issues/648
The fix should be simple enough, also checking for the presence of /run/.containerenv in the isDocker() function.

I should add I worked around it via the following:
touch empty
docker run --rm -v $(pwd):/output -v empty:/.dockerenv  livekit/generate --local

thanks for the report! that's a good insight. We'll modify this in the generator.

Yes, I can confirm that fixes it @davidzhao . Thanks!

awesome! thanks for the report and your insights!


Add Authenticate with OAuth sample

The team is looking into the OAuth settings issue you brought up. Either the sample design will be updated (in which case this sample will be updated on each platform), the other samples will be updated, or neither. In any case, I don't know that it is worth holding up this PR.

@njarecha Thank you!

verified the sample pr and works as expected


[FEATURE] Design responsive layout to the Explorer app

Feature Description
Design different layouts and breakpoints to make the app more responsive (although the intent it to use this on a MD or larger viewport)

[ ] stackql logo doesn't resize properly for mobile or tablet


Make partial rendering a dedicated role of the renderer (and support partials nested within subdirectories)

Do this by adding a Renderer#partial method and centralizing the logic around handling partials into Renderer. This means partial handling is no longer spread around Part and Scope.
At the same time, allow partials nested within subdirectories by translating partials named "foo/bar" into "foo/_bar" before doing the template lookup and rendering. This makes it easier for 3rd party gems shipping their own dry-view partials to prevent naming collisions with partials shipped as part of an app. n.b. partial files are still found using the same lookup path, this just allows some nesting within each of those directories.
I need this latter change particularly because I'm building up icy-form, which will ship with its own dry-view partials. Having the partials collected under the same subdirectory (e.g. icy_form/_form, icy_form/_text_field, etc.) seemed much nicer than having to organise them within a single flat namespace.
I wanted to get this initial change up as a PR so you could take a look at it (:wave: @solnic, @GustavoCaso, @flash-gordon). If there are no objections, I'll fix the specs I broke and add a few more in, and then we can probably make a new release.

My one thought about the changes here is if we have Renderer#call (for rendering templates) and Renderer#partial (for rendering partials), perhaps we can improve the symmetry of these methods and rename them to #template/#partial, or maybe even #render_template/#render_partial.

👏 👏 Looks good to me, also very fun of renaming the methods to render_template and render_partial

Thanks for checking this stuff out, @solnic and @GustavoCaso. I'll probably go ahead and try release it on Friday (here in Aus).


Add autoload support

I needed to include the file manually for now in UserAgentParser
// include 'vendor/whichbrowser/whichbrowser/libraries/whichbrowser.php';
With this line it should work with the composer autoloader

I've released a new composer package with this change. When I install UserAgentParser from source and run composer install -o I can confirm the WhichBrowser class is now picked up by the autoloader.
However if I install UserAgentParserMatrix from source, I get an error from composer:
[RuntimeException]
  Could not scan for classes inside "/Users/niels/Projects/UserAgentParserMatrix/vendor/thadafinser/user-agent-parser/vendor/whichbrowser/whichbrowser/libraries/whichbrowser.php" which does not appear to be a file nor a folder

It's because i included the composer.lock file their (and so the old version is installed)
Reason in simply, that everybody gets the same result. It need to be updated from time to time.
I will do it their soon :-)
Thank you

Release 1.0.1039 now supports namespaces.
The old WhichBrowser object is now only available when you manually include vendor/whichbrowser/whichbrowser/libraries/whichbrowser.php
The new WhichBrowser\Parser object is available using autoloading.
Apart from the naming the object is still the same as before. The API is still unsupported for now and will be replaced soon by a proper PHP api.

Great, thank you.
I raised the minimum version to 1.0.1039 and released a new version
https://github.com/ThaDafinser/UserAgentParser/releases


Add progress middleware with onProgress callback

Added a middleware to make it possible to receive progress callbacks while request is downloading.
To enable progress events the middleware needs access to the raw response body, so this PR also splits the middleware stack into two as a simple proof of concept.
Also extracted the current formatting of the response body into a format middleware between the middleware stacks.
The progress middleware takes two options:

sizeHeader: (optional, default: Content-Length) which header to look for total size of response, can be useful when server sends the body in chunks and Content-Length is not available.
onProgress: (required, function(bytesCurrent: number, totalSize: number | null) => void) function to call on each progress event, if no header with total size is available totalSize is null.

Inspiration for progress with fetch from Umar Hansa on twitter
This feature only supports modern browsers for now.
I have never used flow before (usually work in Typescript) so the types are not correct.

Sorry icannot review it right now. Midnight in my timezone. Will do it tomorrow.
To tired right now. Sorry.

@AnotherHermit I made some changes in the code.
The main goal of changes to keep RelayNetworkLayer.constructor the similar with a previous version. And do not break anybody's apps.
Please check that it works in your app and progress middleware does not break.
Also please describe options in README.md for new middleware.

Of course, I wasn't sure if you wanted to solve it differently so I just implemented a rough bare minimum. Changes look great to me and everything works on my end. I updated the README with descriptions of the new functionality. Let me know if there's anything else you need.

Merged! Thanks a lot for great middleware 🍻


Incorrect reproduction results

I want to reproduce the results of the method in the paper using CLIP without fine-tuning，as shown in the figure below

I use the following command
"""
python eval_refcoco/main.py --input_file /REC/Zeroshot_REC/data/reclip_data/input_file/refcocog_val.jsonl --image_root /REC/Zeroshot_REC/data/train2014 --method matching --clip_model ViT-B/32 --triplets_file /REC/Zeroshot_REC/data/reclip_data/triplets_file/gpt_refcocog_val.jsonl --detector_file /REC/Zeroshot_REC/data/reclip_data/detector_file/refcocog_dets_dict.json
"""
The results obtained are different from those in the paper

Is there something wrong with the command I ran?

I think you should add --rule_filter command.


Replace print(..) statement with logger callbacks

Such as here

The only print found is in DictUtils:
print(DictUtils.get_pretty_json(dictionnary))


fix typo

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.00%. Comparing base (11f5a30) to head (7be4ff8).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   94.00%   94.00%
=======================================
  Files          37       37
  Lines         801      801
  Branches       76       76
=======================================
  Hits          753      753
  Misses         48       48

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


Using multiple instances on one page

Hello,
first, thanks for your work on building a wrapper for quill.js, it works like a charm.
Unfortunately, there is an issue with multiple editors on one page (or within one controller). Quill.js attaches the default styles multiple times to the HEAD of the HTML file. I fixed it by adding quill.js "styles"[1] attribute to the directives scope [2], passing "true" or "false" to "editor-styles" (defaults to "false").
I can open a PR if you think it helps others as well.
[1] http://quilljs.com/docs/configuration/
[2] https://github.com/Cordobo/ngQuill/commit/b0af6f63a2d5a936eeaf141ba6cd8726aa2196d6

sounds good. you can open a pull request. but please to the develop-branch (i am using git-flow) and do not forget to add the key to the readme.md :).
I will release a new verison afterwards.
Thanks for contributing :+1:

Hi Bengt,
I added the description to the readme and opened the PR.


Remove library.cfg

A configuration file based on a directive [exclude] is a anti-pattern, so is proposed to remove library.cfg.
See: http://trac.osgeo.org/openlayers/ticket/3498
Documentation affected:

http://trac.osgeo.org/openlayers/wiki/Profiles
http://trac.osgeo.org/openlayers/wiki/HowTo%3ASingleFileJsLibrary
http://docs.openlayers.org/library/deploying.html#custom-build-profiles

@jorix please update the doc pages if you get a chance.

@elemoine: Ok, you have gone faster, I was going to report the link ;-) The wiki documentation is also updated.


update Documentation

⭐ Summary
I want to add more practice questions for searching and sorting.
💻 Location
This lesson is present under the
Section: 5
Lesson: E
Location Order: 5E
File-Name: practise-question.md
❌ Additional Info
This lesson must consist of at least the following topic:
Approach
Program
program explanation
Time Complexity

@Utkarsh1504 Please assign me this issue, only 1 PR is left for me to complete the challenge.

ok let me know what I need to do

@Utkarsh1504 tell me any change in documentation or anything you want to update.I will do it asap


Documentation update to advertize timedDrag

Description
This documentation PR adds several 'see also' sections in order to make WidgetController.timedDrag more discoverable.
Related Issue
Fixes https://github.com/flutter/flutter/issues/96893
Tests
No Test, documentation update only.

@loic-sharma Can you do a secondary review?


Any conditional-generation method available??

Hello, would it be possible to share your conditional-generation code
test_x = preprocess(user_input_midi),  model.predict(test_x) ??
I'm very curious to test this too,
Thank you very much for sharing your work.

Hello ikarus, sorry for the late response.
unfortunately, I don't have any code available for conditional generating code, but you can do it by encoding an arbitrary MIDI sequence and then generating the latent codes that stems for that.


Zone parameters always empty

Title
When using Zone.current.get("any param") the result is always null.
Functional impact
Zone.current.get does not work properly.
Minimal repro steps

Install default/latest JavaScriptServices; yo aspnetcore-spa
Add asp-prerender-data='new { }' to the <app/> within index.cshtml
For example add cookies as a data object; cookies = ViewContext.HttpContext.Request.Cookies
Within any part of the module, add Zone.current.get('cookies') and print this somehow
The result is now undefined

Expected result
Instead of get returning undefined, return the actual result.
Actual result
get returns undefined instead.
Further technical details
When printing the entire zone, the properties are also empty:
Microsoft.AspNetCore.NodeServices:Information: Zone {
  _properties: { isAngularZone: true },
  _parent:
   Zone {
     _properties: {},
....
}

This also causes issues with angular2-cookie, as it's requesting Zone.current.get('req'), which is undefined too;
This console.log(cookieService.getAll()); causes:
Exception: Call to Node module failed with error: Error: Uncaught (in promise): TypeError: Cannot read property 'headers' of undefined
TypeError: Cannot read property 'headers' of undefined

Am I missing anything or is this the expected result due to a bug?

asp-prerender-data has never automatically populated Zone, so I'm not sure how to make sense of your repro steps. If you're having issues with using the Zone APIs, can you close report that to the Zone/Angular project?

@SteveSandersonMS In the previous versions before Angular 4 you could at least set the values in the zone properties in the boot-server.ts. Now however all of that's gone.
How do we push values from .net mvc through to our apps now?

Hi @SteveSandersonMS, sorry for missing a bit of information.
As @JohnGalt1717 already commented, it's hard to figure out how Zone is getting populated, as the documentation is meant for an earlier version of this build and I'm using the latest version with Angular 4.
First thing I've tried is using the providers constant and adding content into there:
    const providers = [
    ...
        { provide: "cookies", useValue: params.data.cookies }
    ]

Next thing I've tried is running the zone within the platformDynamicServer return:
    return platformDynamicServer(providers).bootstrapModule(AppModule).then(moduleRef => {
        const appRef = moduleRef.injector.get(ApplicationRef);
        const state = moduleRef.injector.get(PlatformState);
        const zone = moduleRef.injector.get(NgZone);

        // This is what I've adjusted
        zone.run(() => this.cookies = params.data.cookies);

Neither of these adjustment make it solve the issue.
Could you either tell me if this is a bug or how to correctly populate the Zone with the latest code?
Thanks!

as the documentation is meant for an earlier version of this build and I'm using the latest version with Angular 4.

Can you let me know what documentation you're referring to? There shouldn't be any documentation about using Zone APIs here, because that's framework-specific - all docs about that should be maintained by the Zone/Angular project owners.

Could you either tell me if this is a bug or how to correctly populate the Zone with the latest code?

Zone is a third-party framework API. In general this repo isn't a good place to raise framework-specific questions - those need to be addressed to the relevant third-party framework community, as we can't scale otherwise. I hope that seems reasonable!
However I have pushed a framework-specific example here: https://github.com/aspnet/JavaScriptServices/commit/678e2300211ad6f0eec9e84103e7173cf0890a66. This uses Angular 4's DI APIs to get the asp-prerender-data value to Angular components. Hope that helps.

Window is unassigned during server side pre-render and throws an unhelpful error.


Add more analytics events

Examples:

Picture taken
Picture shared
Game score (metadata: valid/invalid answers count)
Change category in the middle of play
...

Events added


markdown extra does not work as documented

the old setting markdown_extra seems to override markdown.extra: true

fixed!

Hi,
this fix seems not to work, in fact, using the latest release, I'm not able to activate markdown extra at all. it seems that the first condition isset($this->markdown_extra) cannot never be false, due to default configuration setting, so the full condition always returns false.
Restoring the code before the fix, works for me.

you're setting it in system.yaml, or in a page header?

p.s. system.pages.markdown_extra is deprecated, use system.pages.markdown.extra

Ok. That works. Thank you


Commands are wrong

azure should just be az

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e56b7281-9ab8-982c-aaef-700d6dcd6477
Version Independent ID: 326cf08d-96e1-e8d1-8ae2-a1f57e111e7d
Content: Manage Azure Key Vault using CLI
Content Source: articles/key-vault/key-vault-manage-with-cli.md
Service: key-vault
GitHub Login: @barclayn
Microsoft Alias: barclayn

@bitcoinbrisbane Thanks for reporting! I have assigned the issue to the author to investigate and update as appropriate.

Thank you! The changes were made in the private repo.

@MohitGargMSFT I already fixed this through the private repo. How do we close this?


[PUI] Tree side panel does not adhere to theme

Needs to use the selected theme / style

Theming this seems to be more involved than thought, marking for help for now


Changing error comparision in testing

When running tests, the errors are not equal.
When using []*errors.QueryError{errors.Errorf("%s",msg)}, all things being equal
the errors are returned as unequal. Comparing just the messages will be adequate
for application development environments.
The errors that are returned by the internal package cannot be duplicated (easily) when testing. This will allow the messages to be compared as it is the only required component that will be returned in the QueryError struct.

You have to use proper mock for predictable results.


Sensitivity network informations 2

This is an "intermediate" PR in order to fully-finish CopingWithCollapse Review
We have in this PR :

A one-liner access to -DFIELS content
A better description of models in get_available_models
The system of Equation for CopingWithCollapse (not reviewed, values not checked) in NotIncluded (yes, this is a paradox ! Just if you want to see it before i correct it)
Toolbox ( equivalent to [] ) containing minor operations
A deeper description of the equations in a model file
A sensitivity toolbox for easy simulations / plot

get_dfields_overview

Files modified : _models/__init__.py (contains the function) pygemmes/__init__ to call it easily
How to use it ? Just call `pygemmes.get_dfields_overview()
What does it looks like ? A lot of text
How could it be improved ? Adding filters ( on groups, on units, having alphabetical order... )
Another upgrade : Scanning all models files and say in which existing models are they used

118 fields in the library

Field                  definition                                        group                value                 units
---------------------  ------------------------------------------------  -------------------  --------------------  ------------------------
Final                  Final value                                       Temporary            0                     undefined
Omega                  Individual purchasing power                       Household            1                     Units.y^{-1}.Humans^{-1}
beta                                                                     GK-Improvements      0.1
gamma                  Input-output production recipy                    Recipies             0.1
m                      Mass doublependulum                               Temporary            1                     undefined
l                      Length doublependulum                             Temporary            1                     undefined
gravity                gravity acceleration                              Temporary            9.8                   undefined
Tmax                   Total simulated time                              Numerical            100                   y
dt                     time between two steps                            Numerical            0.01                  y
nt                     Number of timestep                                Numerical            None
nx                     NUMBER OF PARRALLEL SYSTEMS                       Numerical            1                     y
time                   Time vector                                       Numerical            None                  y
N                      Population                                        Household            1.0                   Humans
Nmax                   Saturating population                             Household            12                    Humans
L                      Workers                                           Household            None                  Humans
a                      Productivity                                      Household            1                     Units.Humans^{-1}.y^{-1}
w                      Wage value                                        Household            0.85                  $.Humans^{-1}.y^{-1}
lambda                 employement rate                                  Household            0.97
omega                  wage share                                        Household            0.85
n                      Rate of population growth                         Household            0.025                 y^{-1}
alpha                  Rate of productivity increase                     Household            0.02                  y^{-1}
Xi                     Input-output capital recipy                       Recipies             0.1
rho                    Capital consumption recipy                        Recipies             0.1                   y^{-1}
K                      Capital                                           Production           2.7                   Units
Y                      GDP in output quantity                            Production           None                  Units.y^{-1}
GDP                    GDP in output quantity                            Production           None                  $.y^{-1}
V                      Inventory of Goods                                Production           1                     Units
u                      Use intensity of capital                          Production           0.85                  Units
g                      Relative growth of GDP                            Production           None                  y^{-1}
pi                     relative profit                                   Production           None
Pi                     Absolute profit                                   Production           None                  $.y^{-1}
c                      production price                                  Production           None                  $.Units^{-1}
delta                  Rate of capital depletion                         Production           0.005                 y^{-1}
nu                     Kapital to output ratio                           Production           3
gammai                 inflation awareness                               Production           1
sigma                  rate of use adjustment                            Production           1                     y^{-1}
phillips               Wage inflation rate                               Salary Negociation   None                  y^{-1}
phinull                Unemployment rate with no salary increase         Salary Negociation   0.04
phi0                   Parameter1 for diverving squared                  Salary Negociation   None
phi1                   Parameter1 for diverving squared                  Salary Negociation   None
philinConst            wage rate when full unemployement                 Salary Negociation   -0.292                y^{-1}
philinSlope            wage rate dependance to unemployement             Salary Negociation   0.469                 y^{-1}
zphi                   nonlinearity on profit in negociation             Salary Negociation   0.1
Sh                     Shareholding dividends                            Shareholding         0                     $.y^{-1}
divlinSlope            Shareholding dependency to profits (affine)       Shareholding         0.473                 $.y^{-1}
divlinconst            Shareholding dividends when no profits (affine)   Shareholding         0.138                 $.y^{-1}
I                      Investment in money                               Investment           None                  $.y^{-1}
Ir                     Number of real unit from investment               Investment           None                  Units.y^{-1}
kappa                  Part of GDP in investment                         Investment           None
k0                     share of GDP invested when profit is zero (expo)  Investment           -0.0065
k1                     Investment slope (expo)                           Investment           0.006737946999085467
k2                     Investment power in kappa (expo)                  Investment           20
kappalinSlope          Investment slope in kappa (affine)                Investment           0.0318
kappalinConst          Investment without profit (affine)                Investment           0.575
zsolv                                                                    GK-Improvements      3.6
r                      Interest on debt                                  Debt                 0.03                  y^{-1}
D                      Debt of private sector                            Debt                 0.1                   $
Dh                     Debt of household                                 Debt                 0.1                   $
d                      relative debt                                     Debt                 0.1
solvability            capital compared to debt                          Debt                 None
inflation              inflation rate                                    Prices               None                  y^{-1}
p                      price of goods                                    Prices               1                     $.Units^{-1}
mu                     Markup on prices                                  Prices               1.3
eta                    timerate of price adjustment                      Prices               0.5                   y^{-1}
chi                    inflation rate on inventory                       Prices               1                     y^{-1}
G                      Purchased good flow                               Consumption          None                  Units.y^{-1}
H                      Household possessions                             Consumption          0                     Units
Hid                    Household optimal possessions                     Consumption          None                  Units
deltah                 possessions deterioration rate                    Consumption          0.1                   y^{-1}
fC                     Typical rate for consumption optimisation         Consumption          1                     y^{-1}
Omega0                 Purchasing power of inflexion                     Consumption          1                     Units.Humans^{-1}.y^{-1}
x                      Inflexion effectiveness                           Consumption          1                     undefined
h                      saturated p per person                            Consumption          1                     undefined
Dy                     Damage on production                              Coping-Damages       0
DK                     intermediary damage calculation capital           Coping-Damages       0
Damage                 Damage function                                   Coping-Damages       0
deltaD                 Climate induced destruction rate                  Coping-Damages       0                     Units.y^{-1}
pi1                    Linear damage parameter                           Coping-Damages       0                     Tc^{-1}
pi2                    quadratic damage parameter                        Coping-Damages       0.00236               Tc^{-2}
pi3                    Weitzman damage parameter                         Coping-Damages       5.07e-06              undefined
zeta3                  Weitzmann dmg temp exponent                       Coping-Damages       6.754
fk                     Fraction of damage allocated to capital           Coping-Damages       0.3333333333333333
Eind                   Emission from the society                         Emissions            38.85                 C.y^{-1}
Eland                  Natural Emission                                  Emissions            2.6                   C.y^{-1}
deltaEland             timerate of natural emission reduction            Emissions            0                     y^{-1}
sigmaEm                current emission intensity of the economy         Emissions            0                     C.Units^{-1}.y^{-1}
gsigmaEm               Growth rate economic emission intensity           Emissions            -0.0152               y^{-1}
deltagsigmaEm          growth rate of the emission growth rate           Emissions            -0.001                y^{-1}
pbackstop              Magic backstop technology price                   Coping-Technologies  547
pcarbon                aggregated price of carbon                        Coping-Technologies  100
deltapbackstop         growth rate of backstop price                     Coping-Technologies  -0.005
deltapcarbon           Growth rate of carbon price                       Coping-Technologies  0
emissionreductionrate  Emission reduction rate                           Coping-Technologies  0.03                  y^{-1}
Abattement             Redirection of production                         Coping-Technologies  0
convexitycost          Convexity of cost function for reduction          Coping-Technologies  1
Emission0              CO2 Emission per year (Gt) at t=0                 3Layer-Climate       38.85                 C.y^{-1}
Emission               CO2 Emission per year (Gt)                        3Layer-Climate       38                    C.y^{-1}
deltaEmission          Diminution rate of carbon emission                3Layer-Climate       0.01                  y^{-1}
F2CO2                  Forcing when doubling CO2                         3Layer-Climate       3.681                 undefined
CO2AT                  CO2 in atmosphere                                 3Layer-Climate       851                   C
CO2UP                  CO2 in upper ocean                                3Layer-Climate       460                   C
CO2LO                  CO2 in lower ocean                                3Layer-Climate       1740                  C
CUP                    Historical CO2 in upper ocean                     3Layer-Climate       460                   C
CAT                    Historical CO2 in atmosphere                      3Layer-Climate       588                   C
CLO                    Historical CO2 in lower ocean                     3Layer-Climate       1720                  C
phi12                  Transfer rate atmosphere-ocean                    3Layer-Climate       0.024                 y^{-1}
phi23                  Transfer rate upper-lower ocean                   3Layer-Climate       0.001                 y^{-1}
Capacity               Heat capacity atmosphere+upper ocean              3Layer-Climate       10.204081632653061    undefined
Capacity0              Heat capacity lower ocean                         3Layer-Climate       3.52                  undefined
rhoAtmo                radiative feedback parameter                      3Layer-Climate       1.1874193548387098    undefined
gammaAtmo              Heat exchange between layers                      3Layer-Climate       0.0176                undefined
T                      temperature anomaly of atmosphere                 3Layer-Climate       1                     Tc
T0                     temperature anomaly of ocean                      3Layer-Climate       1                     Tc
F                                                                        3Layer-Climate       3.6                   undefined
B                                                                        GK-Improvements      0                     Units
lambperc                                                                 GK-Improvements      0.95
flamb                                                                    GK-Improvements      3.6                   y^{-1}
zpi                                                                      GK-Improvements      1

Get_available_model

modification of _models\__init__.py
Now the description give variables with their definition (from fields)
Should gives the comment from the model, however I can't access to it yet...
A compact form should be great ( or a filter )
pgm.get_available_models(details=True, verb=True)

########################
########## GK ##########

DESCRIPTION : This is a Goodwin model based on extensive variables.
Inflation not integrated to the process
TYPICAL BEHAVIOR : Convergence toward solow point ( good equilibrium) or debt crisis
LINKTOARTICLE:

Created on Wed Jul 21 15:11:15 2021
@author: Paul Valcke

####Variables ####:
# Ordinary differential equations (6):
	-  a :         Productivity
	-  N :         Population
	-  K :         Capital
	-  D :         Debt of private sector
	-  w :         Wage value
	-  p :         price of goods
# State Variables (13):
	-  Y :         GDP in output quantity
	-  GDP :       GDP in output quantity
	-  inflation : inflation rate
	-  L :         Workers
	-  Pi :        Absolute profit
	-  lambda :    employement rate
	-  omega :     wage share
	-  phillips :  Wage inflation rate
	-  kappa :     Part of GDP in investment
	-  I :         Investment in money
	-  d :         relative debt
	-  pi :        relative profit
	-  g :         Relative growth of GDP

####presets:
 'default' : This is a run that should give simple convergent oscillations
 'crisis' : This is a run that should give simple convergent oscillations

Model description

Modification in _core.py

The system of Equation for CopingWithCollapse (not reviewed, values not checked) in NotIncluded (yes, this is a paradox ! Just if you want to see it before i correct it)

Toolbox ( equivalent to [] ) containing minor operations

A deeper description of the equations in a model file

A sensitivity toolbox for easy simulations / plot

gives
hub.get_equations_description()
############# DIFFERENTIAL EQUATIONS ###########
###  a  ###########
Units        : Units.Humans^{-1}.y^{-1}
Equation     : da/dt= alpha * a,
definition   : Productivity
units        : Units.Humans^{-1}.y^{-1}
Comment      : Exogenous technical progress as an exponential
Dependencies :
     alpha     y^{-1}    Rate of productivity increase

###  N  ###########
Units        : Humans
Equation     : dN/dt= n * N,
definition   : Population
units        : Humans
Comment      : Exogenous population as an exponential
Dependencies :
     n         y^{-1}    Rate of population growth

###  D  ###########
Units        : $
Equation     : dD/dt= I - Pi,
definition   : Debt of private sector
units        : $
Comment      : Debt as Investment-Profit difference
Dependencies :
     I         $.y^{-1}  Investment in money
     Pi        $.y^{-1}  Absolute profit

###  w  ###########
Units        : $.Humans^{-1}.y^{-1}
Equation     : dw/dt= w * (phillips + gammai*inflation),
definition   : Wage value
units        : $.Humans^{-1}.y^{-1}
Comment      : salary through negociation
Dependencies :
     phillips  y^{-1}    Wage inflation rate
     gammai              inflation awareness
     inflation  y^{-1}    inflation rate

###  p  ###########
Units        : $.Units^{-1}
Equation     : dp/dt= p * inflation,
definition   : price of goods
units        : $.Units^{-1}
Comment      : markup inflation
Dependencies :
     inflation  y^{-1}    inflation rate

###  time  ###########
Units        : y
Equation     : dtime/dt= 1.,
definition   : Time vector
units        : y
Comment      : dt/dt=1, time as ODE
Dependencies :
     dt        y         time between two steps

###  K  ###########
Units        : Units
Equation     : dK/dt= I/p - K * delta,
definition   : Capital
units        : Units
Comment      : Capital evolution from investment and depreciation
Dependencies :
     I         $.y^{-1}  Investment in money
     delta     y^{-1}    Rate of capital depletion
     p         $.Units^{-1}  price of goods

######### STATE VARIABLES EQUATIONS ###########
###  Y  ###########
Units        : Units.y^{-1}
Equation     : Y= K / nu,
definition   : GDP in output quantity
Comment      : Leontiev optimized production function
Dependencies :
     K         Units     Capital
     nu                  Kapital to output ratio

###  GDP  ###########
Units        : $.y^{-1}
Equation     : GDP= Y * p,
definition   : GDP in output quantity
Comment      : Output with selling price
Dependencies :
     Y         Units.y^{-1}  GDP in output quantity
     p         $.Units^{-1}  price of goods

###  L  ###########
Units        : Humans
Equation     : L= K / (a * nu),
definition   : Workers
Comment      : Full instant employement based on capital
Dependencies :
     K         Units     Capital
     a         Units.Humans^{-1}.y^{-1}  Productivity
     nu                  Kapital to output ratio

###  Pi  ###########
Units        : $.y^{-1}
Equation     : Pi= GDP - w * L - r * D,
definition   : Absolute profit
Comment      : Profit for production-Salary-debt func
Dependencies :
     GDP       $.y^{-1}  GDP in output quantity
     w         $.Humans^{-1}.y^{-1}  Wage value
     L         Humans    Workers
     r         y^{-1}    Interest on debt
     D         $         Debt of private sector

###  lambda  ###########
Units        :
Equation     : lambda= L / N,
definition   : employement rate
Comment      : employement rate
Dependencies :
     L         Humans    Workers
     N         Humans    Population

###  omega  ###########
Units        :
Equation     : omega= (w * L) / (Y*p),
definition   : wage share
Comment      : wage share
Dependencies :
     w         $.Humans^{-1}.y^{-1}  Wage value
     L         Humans    Workers
     Y         Units.y^{-1}  GDP in output quantity
     p         $.Units^{-1}  price of goods

###  phillips  ###########
Units        : y^{-1}
Equation     : phillips= -phi0 + phi1 / (1 - lambda)**2,
definition   : Wage inflation rate
Comment      : Wage increase rate through employement and profit
Dependencies :
     phi0                Parameter1 for diverving squared
     phi1                Parameter1 for diverving squared
     lambda              employement rate

###  d  ###########
Units        :
Equation     : d= D / GDP,
definition   : relative debt
Comment      : private debt ratio
Dependencies :
     D         $         Debt of private sector
     GDP       $.y^{-1}  GDP in output quantity

###  pi  ###########
Units        :
Equation     : pi= Pi/GDP,
definition   : relative profit
Comment      : relative profit
Dependencies :
     Pi        $.y^{-1}  Absolute profit
     GDP       $.y^{-1}  GDP in output quantity

###  inflation  ###########
Units        : y^{-1}
Equation     : inflation= eta*(mu*omega-1),
definition   : inflation rate
Comment      : markup dynamics
Dependencies :
     mu                  Markup on prices
     eta       y^{-1}    timerate of price adjustment
     omega               wage share

###  kappa  ###########
Units        :
Equation     : kappa= k0 + k1 * np.exp(k2 * pi),
definition   : Part of GDP in investment
Comment      : Relative GDP investment through relative profit
Dependencies :
     k0                  share of GDP invested when profit is zero (expo)
     k1                  Investment slope (expo)
     k2                  Investment power in kappa (expo)
     pi                  relative profit

###  I  ###########
Units        : $.y^{-1}
Equation     : I= GDP * kappa,
definition   : Investment in money
Comment      : Investment value
Dependencies :
     GDP       $.y^{-1}  GDP in output quantity
     kappa               Part of GDP in investment

###  g  ###########
Units        : y^{-1}
Equation     : g= (I/p - K * delta)/K,
definition   : Relative growth of GDP
Comment      : relative growth rate
Dependencies :
     I         $.y^{-1}  Investment in money
     K         Units     Capital
     delta     y^{-1}    Rate of capital depletion
     p         $.Units^{-1}  price of goods

Noise
Just for fun, you can add noise in your models ! the model Noise is just the formula y'=y*noise noise being a gaussian noise. And hop, you appear in the posh world of stochastic models.

Important : If one simulate N system in parallel, the noise will be the same for each system
Sensitivity studies

Toolbox (pygemmes init related) file modified. Core has a new method CalculateStatSensitivity and a badly-coded reinterpolation

The way it works :

pgm.GenerateIndividualSensitivity and pgm.GenerateCoupledSensitivity will generate a preset with values taken in distribution for each given field. One run the preset then ask for the calculations of statistic between trajectories, then can plot all variables with these relevant characteristics (by adding SENSITIVITY=True in hub.plot() )

SensitivityDic = {
    'alpha': {'mu': .02,
              'sigma': .12,
              'type': 'log'},
    'k2': {'mu': 20,
           'sigma': .12,
           'type': 'log'},
    'mu': {'mu': 1.3,
           'sigma': .12,
           'type': 'log'},
}

presetSimple = pgm.GenerateIndividualSensitivity(
    'alpha', 0.02, .2, disttype='log', N=10)
presetCoupled = pgm.GenerateCoupledSensitivity(SensitivityDic, N=10, grid=False)

_DPRESETS = {'SensitivitySimple': {'fields': presetSimple, 'com': ''},
             'SensitivityCoupled': {'fields': presetCoupled, 'com': ''},
             }

hub = pgm.Hub(_MODEL, preset='SensitivityCoupled', dpresets=_DPRESETS)
hub.run(verb=1.1)

hub.reinterpolate_dparam(1000)

hub.CalculateStatSensitivity()

#dax = hub.plot(color='k', lw=0.1)

dax = _plot_timetraces.plot_timetraces(hub, SENSITIVITY=True)

The type of plots one obtain :

Reinterpolate
If you need a small timestep but do not want to calculate/plot everything you can reinterpolate the data using hub.reinterpolate_dparam(Npts), practical for statistical measurement / big plots.
Network
Now one can visualize all the information of the logic file in a friendly way !

The information on the logic itself is shown in the description of the node

Hello @DaluS! Thanks for opening this PR.

In the file Examples/BASIN-ATTRACTION-GK.py:

Line 89:22: W605 invalid escape sequence '\l'
Line 90:22: W605 invalid escape sequence '\o'

In the file Examples/Cockpit-GK-Climate.py:

Line 22:101: E501 line too long (106 > 100 characters)
Line 24:101: E501 line too long (106 > 100 characters)
Line 35:101: E501 line too long (113 > 100 characters)

In the file pygemmes/_models/_def_fields.py:

Line 559:21: E241 multiple spaces after ':'
Line 564:21: E241 multiple spaces after ':'

In the file pygemmes/_models/_model_3Layerclimate.py:

Line 3:101: E501 line too long (127 > 100 characters)
Line 3:128: W291 trailing whitespace
Line 5:101: E501 line too long (1202 > 100 characters)
Line 35:9: E266 too many leading '#' for block comment
Line 35:27: W291 trailing whitespace
Line 37:101: E501 line too long (129 > 100 characters)
Line 41:101: E501 line too long (159 > 100 characters)
Line 45:101: E501 line too long (106 > 100 characters)
Line 49:101: E501 line too long (138 > 100 characters)
Line 53:101: E501 line too long (101 > 100 characters)
Line 61:1: W293 blank line contains whitespace
Line 62:9: E266 too many leading '#' for block comment
Line 62:25: W291 trailing whitespace
Line 63:1: W293 blank line contains whitespace
Line 64:1: W293 blank line contains whitespace
Line 65:1: W293 blank line contains whitespace
Line 73:101: E501 line too long (109 > 100 characters)

In the file pygemmes/_models/_model_CopingWithCollapse.py:

Line 3:101: E501 line too long (127 > 100 characters)
Line 5:101: E501 line too long (1202 > 100 characters)
Line 33:101: E501 line too long (129 > 100 characters)
Line 37:101: E501 line too long (159 > 100 characters)
Line 41:101: E501 line too long (106 > 100 characters)
Line 45:101: E501 line too long (138 > 100 characters)
Line 49:101: E501 line too long (101 > 100 characters)
Line 91:101: E501 line too long (111 > 100 characters)
Line 135:101: E501 line too long (109 > 100 characters)
Line 178:101: E501 line too long (102 > 100 characters)
Line 182:101: E501 line too long (103 > 100 characters)
Line 208:101: E501 line too long (164 > 100 characters)
Line 212:101: E501 line too long (124 > 100 characters)
Line 254:15: E241 multiple spaces after ':'
Line 255:15: E241 multiple spaces after ':'
Line 256:17: E241 multiple spaces after ':'

In the file pygemmes/_models/_model_GK.py:

Line 43:101: E501 line too long (111 > 100 characters)

In the file pygemmes/_plots/_plot_timetraces.py:

Line 156:53: W605 invalid escape sequence '$'
Line 314:78: W605 invalid escape sequence '\m'
Line 314:82: W605 invalid escape sequence '\p'
Line 314:88: W605 invalid escape sequence '\s'
Line 320:87: W605 invalid escape sequence '\m'
Line 320:91: W605 invalid escape sequence '\p'
Line 320:95: W605 invalid escape sequence '\s'
Line 320:101: E501 line too long (103 > 100 characters)

In the file pygemmes/_toolbox.py:

Line 285:101: E501 line too long (103 > 100 characters)

Do see the Hitchhiker's guide to code style

Non-regression test has been moved : multiple fields definitions /comments has been changed and create errors


Forcing an even dimension for dbSBox?

Is there a reason that the following lines require a dbSBox to have an even dimension (in DB units)?
https://github.com/The-OpenROAD-Project/OpenDB/blob/39463710c066e4da13e700caf8ac66d24d8d25b6/src/db/dbSBox.cpp#L200-L211

Probable reason was explained by @maliberty; this is to keep the center line on-grid.
Gonna close this.


Control digitalPin status using bits of a uint32_t

Hello,
I use this changes to track the state of each pins, so I can test it or print on terminal.
This was helpful to me.
Thanks for the logic correction and tips
Obter o Outlook para Androidhttps://aka.ms/AAb9ysg

From: Brian Park @.>
Sent: Wednesday, March 2, 2022 3:30:26 PM
To: bxparks/EpoxyDuino @.>
Cc: CaioPellicani @.>; Author @.>
Subject: Re: [bxparks/EpoxyDuino] Control digitalPin status using bits of a uint32_t (PR #61)
@bxparks commented on this pull request.
Hi, I left you some minor logic error fixes and improvements.
But the more important question is: Why do you want this functionality? Because in a real microcontroller, a digitalWrite() to a specific pin does not affect the value of the digitalRead() on that pin. They are completely decoupled from each other. I'm not sure that EpoxyDuino should add a semantic coupling between those 2 functions that doesn't exist in real life.

In cores/epoxy/Arduino.cpphttps://github.com/bxparks/EpoxyDuino/pull/61#discussion_r817977060:

void yield() {
usleep(1000); // prevents program from consuming 100% CPU
}
void pinMode(uint8_t /pin/, uint8_t /mode/) {}
-void digitalWrite(uint8_t /pin/, uint8_t /val/) {}
+void digitalWrite(uint8_t pin, uint8_t val) {

if( val == HIGH ){
arduinoPins |= ( 1LU << ( pin - 1 ) );

pin == 0 is a valid pin, so all the pin - 1 should be just pin

In cores/epoxy/Arduino.cpphttps://github.com/bxparks/EpoxyDuino/pull/61#discussion_r817977787:

void yield() {
usleep(1000); // prevents program from consuming 100% CPU
}
void pinMode(uint8_t /pin/, uint8_t /mode/) {}
-void digitalWrite(uint8_t /pin/, uint8_t /val/) {}
+void digitalWrite(uint8_t pin, uint8_t val) {

if( val == HIGH ){
arduinoPins |= ( 1LU << ( pin - 1 ) );
}
if( ( val == LOW ) && ( digitalRead( pin ) ) ){
arduinoPins ^= ( 1LU << ( pin - 1 ) );

this should be
arduinoPins &= ~(1LU << pin);

In cores/epoxy/Arduino.cpphttps://github.com/bxparks/EpoxyDuino/pull/61#discussion_r817979667:

void yield() {
usleep(1000); // prevents program from consuming 100% CPU
}
void pinMode(uint8_t /pin/, uint8_t /mode/) {}
-void digitalWrite(uint8_t /pin/, uint8_t /val/) {}
+void digitalWrite(uint8_t pin, uint8_t val) {

if( val == HIGH ){
arduinoPins |= ( 1LU << ( pin - 1 ) );
}
if( ( val == LOW ) && ( digitalRead( pin ) ) ){
arduinoPins ^= ( 1LU << ( pin - 1 ) );
}
+}

+int digitalRead(uint8_t pin) {

return ( arduinoPins & ( 1LU << ( pin - 1 ) ) ) == ( 1LU << ( pin - 1 ) );

Since we are testing for only a single bit, we can simplify this to:
return (arduinoPins & (1LU << pin) != 0;
This generated fewer machine instructions because it doesn't have to test against an extra argument.
—
Reply to this email directly, view it on GitHubhttps://github.com/bxparks/EpoxyDuino/pull/61#pullrequestreview-897984507, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP7MMOAQLSGKQ4ZFUQ3AUWLU56XUFANCNFSM5PYFSQUQ.
You are receiving this because you authored the thread.Message ID: @.***>

Hello,
I use this changes to track the state of each pins, so I can test it or print on terminal.
This was helpful to me.
Thanks for the logic correction and tips

I can see the convenience of being able to set the value returned by digitalRead() for testing. But I don't think we should use digitalWrite() for that purpose.
How about we create a new method, call it something like digitalReadValue(uint8_t pin, uint8_t value)? The client code will need to use an #ifdef conditional, otherwise the code will not work in a real Arduino environment:
#if defined(EPOXY_DUINO)
   digitalReadValue(xxx, yyy);
#endif

But if the code is meant to run only on EpoxyDuino, then the conditional guard can be omitted.
If this is agreeable to you, then I would like to make some commits to your branch to polish the PR for inclusion: fix bugs, add error checks, apply consistent coding style, add documentation, etc.

Sure, I think this is a great idea!
This is my first pull request, so maybe I will need some help in the process

Normally the PR author (you) would make changes requested by the reviewer (me), and then do a git push of the new changes so that the reviewer can see the changes. But it's far easier and faster to do the changes myself than to describe those changes in lots of words. GitHub has a feature which allows the reviewer to make changes directly on the PR author's branch. That also allows me to preserve the authorship of the PR. That's the theory, I think I used this feature once or twice before, but I can't remember. I'll give it a shot tomorrow.

I pushed 3 commits into your branch. Please take a quick look. If this is acceptable for you, I can merge this in this PR.

that's great, is more than acceptable


updating STARTCOMMAND to array with += syntax and allowing spaces in …

Updating to using array for STARTCOMMAND due to issues that su steam -c "${STARTCOMMAND}" can have with escaped quotes and spaces, also this will allow for spaces in server names and passwords
Context
Allow users to use spaces in certain environment variables
Choices

Better handling of spaces in program arguments that should allow them, such as server name and passwords

Test instructions

Run the server with this change and have spaces in the passwords or in the server name

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I've added documentation about this change to the README.
there is no mention spaces aren't supported, so this will just be helpful overall
[x] I've not introduced breaking changes.

Brilliant!

@clarkent86 Thank you for creating this PR, however I did find an issue. Changing this to an array while not changing the way the startcommand gets expanded, does not work. As shellcheck gives the following error on su steam -c "${STARTCOMMAND}":

Expanding an array without an index only gives the first element.shellcheckSC2128

Which I could confirm when running it myself:
palworld-server  | *****CHECKING FOR EXISTING CONFIG*****
palworld-server  | RCON_ENABLED=true
palworld-server  | RCON_PORT=25575
palworld-server  | ./PalServer.sh
palworld-server  | Shutdown handler: initalize.
palworld-server  | Increasing per-process limit of core file size to infinity.
palworld-server  | [S_API] SteamAPI_Init(): Loaded local 'steamclient.so' OK.
palworld-server  | CAppInfoCacheReadFromDiskThread took 1 milliseconds to initialize
palworld-server  | Setting breakpad minidump AppID = 2394010
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamUser021 before SteamAPI_Init succeeded.
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamFriends017 before SteamAPI_Init succeeded.
palworld-server  | [S_API FAIL] Tried to access Steam interface STEAMAPPS_INTERFACE_VERSION008 before SteamAPI_Init succeeded.
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamNetworkingUtils004 before SteamAPI_Init succeeded.

Only ./PalServer.sh gets called with no parameters

@thijsvanloef Apologies! I was testing with a local script and forgot to expand the array correctly. I should've noticed with the other comment!
Should be fixed now, can you try again?

@Dashboy1998 thanks! It looks like I missed the echo command as well, updated. Also, expanding with the @ instead of the * preserves the spaces as we want for the command.

Needed to revert the changes, I've done some more testing and it seems like it won't start:
palworld-server  | Connecting anonymously to Steam Public...OK
palworld-server  | Waiting for client config...OK
palworld-server  | Waiting for user info...OK
palworld-server  |  Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
palworld-server  |  Update state (0x5) verifying install, progress: 3.45 (77589729 /<PHONE_NUMBER>)
palworld-server  |  Update state (0x5) verifying install, progress: 16.11 (362079510 /<PHONE_NUMBER>)
palworld-server  |  Update state (0x5) verifying install, progress: 34.05 (765465170 /<PHONE_NUMBER>)
palworld-server  |  Update state (0x5) verifying install, progress: 51.43<PHONE_NUMBER> /<PHONE_NUMBER>)
palworld-server  |  Update state (0x5) verifying install, progress: 69.41<PHONE_NUMBER> /<PHONE_NUMBER>)
palworld-server  |  Update state (0x5) verifying install, progress: 88.65<PHONE_NUMBER> /<PHONE_NUMBER>)
palworld-server  | Success! App '2394010' fully installed.
palworld-server  | *****CHECKING FOR EXISTING CONFIG*****
palworld-server  | RCON_ENABLED=true
palworld-server  | RCON_PORT=25575
palworld-server  | *****STARTING SERVER*****
palworld-server  | ./PalServer.sh -port=8211 -players=16 -servername="WorldofPals" -serverpassword="worldofpals" -adminpassword="adminPasswordHere" -queryport=27015 -useperfthreads -NoAsyncLoadingThread -UseMultithreadForDS
palworld-server  | su: invalid option -- 'o'
palworld-server  | Try 'su --help' for more information.

@clarkent86 ☝

Got it. Seems like there's something unexpected happening, I'll need to figure out how to source a custom image on my NAS.


Merge Msg::Proposal into Msg::Apply to reduce apply worker messages

apply::Msg::Proposal contains callbacks of proposals, which will be invoked when the proposals is applied. So for now, there are at least two messages are sent to apply workers Msg::Proposal and Msg::Apply, in fact they can be merged into one message, by this way we can simplify the apply workflow and may improve performance.
apply::Msg:
https://github.com/tikv/tikv/blob/7165e0f3c10104810b7533985f48ea78e4eab07b/src/raftstore/store/fsm/apply.rs#L2200-L2212

Good catch.


Watermark removal

I fine-tuned the model via single video fine tuning, but the watermark is still there. Would like to know fine-tune detail that can remove the watermark
Many thanks

And also what dataset do you use for the fine-tuning. thx

Hey @azsunlight.
Usually training the temporal attention layers ('attn1', 'attn2')  at a learning rate of roughly 1e-5 and weight_decaty 0 should be enough to get rid of the watermark.

Hi, thanks for the information. Does the single-video dataset work as well?

Hi @ExponentialML, thanks for the information. Does single-video fine-tuning work as well?

@ExponentialML Could this technique potentially be applied to remove the watermark?

@green-s & @azsunlight There was a PR that got merged with this feature in the inference script. Does this resolve the issue for you two?

@green-s & @azsunlight There was a PR that got merged with this feature in the inference script. Does this resolve the issue for you two?

Yes. It works. Thank you


[Quantum] [DRAFT - Do not merge] Update version and history info for az quantum extension 0.15.0

Release PR for az quantum extension, version 0.15.0.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Quantum


Port - Restrict which sites out webview iframe can frame (#18495)

protocol handler - normalize paths

use extUri for normalizing paths

:lipstick;

Add content security policy to top level webview

This change hardens our webviews by adding a fairly restrictive csp to them. This CSP should only apply to the outer webview iframe, not to the inner iframe which is controlled by extensions
Co-authored-by: Benjamin Pasero<EMAIL_ADDRESS>Co-authored-by: Matt Bierner<EMAIL_ADDRESS>
This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 48.237%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
14799

Relevant Lines:
29111

💛  - Coveralls


Skip overwriting existing service definition

Hello,
this PR disables overwriting of existing services definition and just sets autowire option to true.
This is very usefull when you want to pass e.g. scalar parametrs from parameters.yml into controller because in SF 2.8.3 you can do:
controller_service:
    class: SomeController
    arguments:
        - ''
        - %scalar_parametr%

And in upcomming SF 2.8.4:
controller_service:
    class: SomeController
    arguments:
        1: %scalar_parametr%

So the first parametr is autowired and the second is not

Hi @podhy,
thanks for the improvement and adding tests. They really nicely cover the feature.


is this still supported? needs python 3 upgrade

is this still supported? needs python 3 upgrade
e.g. print "test" now should be print("test")

Hi @hannesdelbeke !
Thanks for your interest in the tools! As you can see from the commit history, there has been no development for quite some time on the tools. Since I moved from modelling to pipeline development, my need for the tools has steadily declined. I have no intention of maintaining the code for the time being.
That said, it should not be hard to convert the scripts to be python 3 compliant and I will happily accept pull requests for it.


merge package template from Kronos-Tools/npm-package-template

.travis.yml

chore(travis): merge from template .travis.yml

Codecov Report

Merging #96 into master will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           master   #96   +/-   ##
====================================
  Coverage      92%   92%
====================================
  Files           1     1
  Lines          25    25
====================================
  Hits           23    23
  Misses          2     2

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 ee9d7d1...9ccefc5. Read the comment docs.


[FancyZones] Outlook new message restore placement bug

Summary of the Pull Request
Some application like outlook uses multiple windows. If application was zoned "Move newly created window to zone" all other windows were moved to the zone also.
After changes only one window of same application will be zoned on creation, rest will not be moved by fancy zones.
When one window of same application window is zoned, zoning other window can be zoned, but it will not be saved in zone-settings.json.
Edit: Now multiple instances of same application will be zoned based on process id. First window of aplication instance will be zoned, rest will be unzoned while starting app

PR Checklist

[x] Applies to #1192

Validation Steps Performed

Manual testing
Unit tests passed

I'm wondering if we should allow new processes of the same application to be zoned when launched.
Since this PR changes the previous behavior, we should assume it may break some users' workflow.
@PrzemyslawTusinski how complex would it be to check if the new window belongs to a new process?

Please test this and review again as there are major changes since first version

The new version resolves the splash screen issue. Also, I like the idea of using process IDs.

tested and it works correctly, great job!

is this a 0.18 item?  been open for 5 days

@crutkas
I want to do some extra testing since it's a high impact change.

Any reason to not merge in?

@crutkas
we want to merge https://github.com/microsoft/PowerToys/pull/3132 first.

Rebased, tested on MS Word, the splash screen is not moved and the main window is
placed into the correct zones after being shown.


Remove redundant docblock types

This will make it easier to read and refactor the code going forward.
Also added a couple missing property types and removed a redundant null check.

Coverage remained the same at 99.892% when pulling ebc08142351c6366ec56aa1faa0f9537657e75a5 on theodorejb:type-cleanup into 187d99d528ed6a176f09cec089d738ad89f5b401 on slimphp:4.x.


Resolving several issues while installing LLTFI

Problems

For LLTFI, there is an extra folder which only contains the LLTFI repo, therefore removal is required for the extra LLTFI folder
Replacement of path folder for LLTFI build is needed due to 1)
Build for LLVM is needed in release mode to reduce memory space taken by the docker image
After docker build, there were no env variables configured which causes extra steps for developer to do in order to run LLTFI
Installing of json-c library needs sudo power in regular Linux desktop and does not need sudo power while running in docker, therefore need to have a new path for json-c where both of the cases above doesn't need sudo power
wget command isn't installed after the docker build has been done

Solutions

Extra folder for LLTFI is removed and now the new path is /home/LLTFI
New path for LLTFI build is /home/LLTFI/build
LLVM build is now done in release mode
In dockerfile, the script ensures to set the env variables as well to have a smooth experience for developers to run LLTFI
Pathway to install json-c library is now set to be ~/local since the path is in the user's home directory, the script would not require sudo power for both manual installation and dockerfile build
Installation of wget is added within the Dockerfile

These changes LGTM! Also, they have been tested.


Breakable light

About the PR
Fixes #3219
Wall light will break on one hit.
Also drops metal and broken bulb after destruction

Was right about to do this, you should leave a comment underneath an issue if you plan on tackling it.


crypto/x509: ParseCertificate and ParseCertificateRequest should return an error when there are duplicate x509 extensions

What version of Go are you using (go version)?

go version: 1.17.6

Does this issue reproduce with the latest release? Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
go version go1.17.6 windows/amd64
PS C:\Users\private\GolandProjects\awesomeProject> go env
set GO111MODULE=auto
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\private\AppData\Local\go-build
set GOENV=C:\Users\private\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\private\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\private\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.17.6
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\private\GolandProjects\awesomeProject\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\private\AppData\Local\Temp\go-build554477491=/tmp/go-build -gno-record-gcc-switches

What did you do?
Create a malformed CSR with multiple SAN extensions manually. Then I tried to parse the malformed CSR.
Proof of Concepts:

https://go.dev/play/p/t_jcGJ9esqc
https://go.dev/play/p/a1QmUMfkq4C

What did you expect to see?
Per RFC5280, Section 4.2 I expected to see a failure in parsing CSRs and Certificates that contain any specific extension more than once.

A certificate MUST NOT include more than one instance of a particular extension.

What did you see instead?
Go did not create any errors parsing malformed CSRs and Certificates.

Hello,
I've encountered the result of this and just wanted to double check on the expected behavior.
The RFC linked indicates there are no duplicate extensions allowed in certificates, this also seems reasonable to imply that there can be no duplicate extensions requested in a CSR.
The changes made also consider it an error to have duplicate attributes in a CSR, not extensions which I can't find anything in PKCS #10 RFC which would indicate that this should be the case but I could be reading that wrong.
Aside from that, the duplicate attribute case is also not covered in the unit tests that were included.
Can I get a confirmation/correction on the interpretation of the RFC?

Duplicate attributes in CSRs were re-allowed in https://go.dev/cl/428636.


Test adding a new orderer

This would add orderer6 to the current network environment

Tested with latest peer version. Gateway works fine.


[#493] Fix wrapping issues in live preview and Side By Side

See issue #493.
While fixing the above issue, I also noticed that absurdly long lines do not wrap in live preview and instead create an horizontal scrolling (they wrap properly in the editor, though).
See this Fiddle https://jsfiddle.net/ub6v51Lw/2/
Hope this helps!
Edit: Click on Side By Side in the above fiddle to replicate the issue.

/jsfiddle

/jsfiddle

Here's a JSFiddle that demonstrates the updated behavior: https://jsfiddle.net/ctgp3tnw/1/


What port is expected for proxy login?

If you create a new account the default is 3456, but you can choose whatever port you want.


Remove all Cargo.toml files from spec.extensions

All */Cargo.(toml|lock) files should be removed from spec.extensions.

🙇🏻


FlutterOutline is truncated in Flutter 3.0/Dart 2.17

The analyzer is not reporting the full outline for the flutter.outline notification with the 3.0 version of Flutter. The outline does not contain the content of methods, only the method name. Here are screen shots from the same project using two different SDKs. This project was created with the Flutter 2.10.5 SDK. I only included one class but all classes have the same shortened outline.
From Flutter 2.10.5:

From Flutter 3.0.0:

More detail, and a different manifestation of the problem, are available in this issue:
https://github.com/flutter/flutter-intellij/issues/6253

This problem also occurs in the Flutter beta channel, which currently has Dart 2.18.0-165.1.beta.

No issues on 3.1.0-9.0.pre-beta.

One of the plugin users reports that it does NOT work with master:
https://github.com/flutter/flutter-intellij/issues/6253#issuecomment-1178780594