chore: merge upstream as of 2022-10-28

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Merges upstream changes as of 2022-10-28. This is needed to pick up the snapshot client API usage fix.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 79.807%

Files with Coverage Reduction
New Missed Lines
%

pkg/batch/batch.go
2
81.37%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
17327

Relevant Lines:
21711

💛  - Coveralls


jumpToDate does not meet minDate / maxDate requirements

I resolved it in https://github.com/flatpickr/flatpickr/pull/2825
Copied from PR:
We have two date inputs:
#dateInput1, who's minDate is equal to today
#dateInput2, who's minDate is equal to the date of #dateInput1 (or today, when #dateInput1 is empty)
We now set a date in both these inputs:

Then we set the date of #dateInput1 to a date with a month greater #dateInput2's month

This automatically clears #dateInput2 because it's value is lower than the minDate (correct behaviour)
But now, when we open #dateInput2, it opens November:

This is because the jumpTo behaviour works like this:

Is a date given? Set it to that date
Set it to latestSelectedDateObj (if not undefined)
Set it to minDate (if not undefined and greater than today)
set it to maxDate (if not undefined and lower than today)

The problem here is that latestSelectedDateObj of #dateInput2 was 2022-11-26 (November), which lies before the new minDate 2022-12-10 (December).
The jumpTo function does not check if latestSelectedDateObj meets the minDate & maxDate "requirements" and the user lands on a month with disabled dates.
The solution:
Only jump to latestSelectedDateObj if it meets the minDate and maxDate requirements (if these are set ofcourse)

@chmln could you maybe merge this please?


Unable to decode blueprint strings

It looks like the code should be able to take in a blueprint string generated from Factorio and output JSON, however when I try to do that I get a stack trace:
Traceback (most recent call last):
  File "scripts/to_exchange.py", line 19, in <module>
    blob = EncodedBlob.from_exchange_file(input)
  File "/home/me/src/factorio-blueprint-compendium/scripts/python_factorio/blueprints.py", line 43, in from_exchange_file
    return cls.from_exchange_string(open(filename).read().strip())
  File "/home/me/src/factorio-blueprint-compendium/scripts/python_factorio/blueprints.py", line 38, in from_exchange_string
    data = json.loads(json_str, object_pairs_hook=collections.OrderedDict)
  File "/usr/lib64/python3.5/json/__init__.py", line 312, in loads
    s.__class__.__name__))
TypeError: the JSON object must be str, not 'bytes'

I fixed this by adding .decode('ascii') to line 37 (which I can gladly submit as a pull request), however even with this change, the library chokes when I give it a blueprint book.

I've fixed it in Pull #9.  The problem is the code was written for Python2, and you're attempting to run it with Python3.  The pull request makes the code Python3 compatible.


Add ChoiceParameter to restrict parameter options to those specified.

Description
As in description.
Motivation and Context
This allows the user to define the permissible values for a parameter and the value type, and reject any parameter that does not meet these criteria.
Have you tested this? If so, how?
I've been using the class for several weeks without issue.

lgtm although would be cool if this works with --help

@erikbern, can you give me an example of what you mean?  I'd be happy to make the change(s).  When I create a Task using a ChoiceParameter, the parameter and its description show up when I run luigi --module <module> <Task> --help.

would be cool if the --help also shows what options are available. but no biggie :)

Ah, yeah.  I'm trying to figure out a way to do that - updating self.description in init does not appear to be sufficient.

Ok, got it working.  My system has a luigi install I was importing instead of the one I was editing.

awesome! flake8 is complaining: https://travis-ci.org/spotify/luigi/jobs/149050059

Can you also add test cases please?

This class is somewhat similar to EnumParameter. Would you mind interlinking those two in the docs? Maybe add something like "If you want a more typed and structured version of a ChoiceParameter, you should consider using an EnumParameter instead." could guide our users to pick the right choice?

You could be entirely right here.  I am a pretty new user of luigi and may have overlooked this class before coming up with my own version.  I will clean it up if requested, but maybe at this point it just makes more sense to not include this.

I still think this makes sense. Not everyone likes to create a new Enum type for every input-restriction and add all the conversion overhead if they want to eventually go back to a string type. I would be happy to include this, as long as the docs comes with recommended good practices. Like we've previously  done here for example. :)

I changed the docstring to I believe the proper format and added a test suite to the test/ directory.  Please let me know if any further changes would be welcome.

Apart from the small comment about assert/convert. This looks very good. I must say I'm very impressed how you managed to fix all the shortcomings in a single review turnaround! :)

Made a couple changes to assert the choices are the proper type as you suggest.  And sure, glad it's helpful.

I rebased, updated docstrings, and added unit tests for the class.

Awesomeness! :)

A post-merge note. Some users might want to use the "Functional API" of enums. Instead of
ChoiceParameter(choices=['a', 'b', 'c'])

Some users might prefer
EnumParameter(enum=Enum('Character', 'a b c'))


No matching extensions for 'authV2'

In attempting to run this command locally, it tries to install extension authV2. However it appears that the extension doesn't exist.
az webapp auth microsoft update         --resource-group $rgName
--name $name         --client-id $appId
--issuer $tokenIssuerUrl         --allowed-audiences $aadTokenAudiences
--yes
ProcessLastExitCode $operation
I also attempted to run az extension list-available --output table but in the list of extensions available, authV2 isn't part of the list.
I then attempted to run the classic version, but it also attempted to install authV2. No matching extensions for 'authV2'. Use --debug for more information.
az webapp auth-classic update         --resource-group $rgName
--name $name         --enabled true
--action LoginWithAzureActiveDirectory         --aad-allowed-token-audiences $aadTokenAudiences
--aad-client-id $appId         --aad-token-issuer-url $tokenIssuerUrl
--output none

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

ID: f5323dc0-8973-d579-328f-5608a8e667f7
Version Independent ID: 93c445df-3a56-4674-691e-030be4187368
Content: az webapp auth-classic
Content Source: latest/docs-ref-autogen/webapp/auth-classic.yml
Service: app-service-web
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to service team

any update on this?

@mkarmark FYI.

Just wanna follow up on this and find out if there's an update

@Chibuzor27 are you still seeing this issue?

@Hamster-Huey FYI

closing as the service team has confirmed functionality. please reopen if you're still experiencing the issue.


can change color with speed!

Link: https://worldbiomusic.github.io/star-field/.
My first pull request. Thank you!

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Looks great! Thanks for sharing.


'About' does not work

Navigating to 'about' via
a) (i) icon at bottom left
b) 'About' on top navigation bar
results in a page not loading
Additionally, there is no way out - except back. No timeout is observed if a remote request is being issued.

original report: https://github.com/odpi/egeria-ui/issues/480

@planetf1 it would be better if you can add the developer tools in the print screen to get a better understanding of what is happening(interested in the network response and console output of developer tools)


0 prefix for octal

IMO there is only one significant difference from Val(): since the lone leading zero is the prefix of an octal string, decimal strings cannot start with a zero.

It should not be like this- it must try octal notation only after & prefix (supported by FPC).
and binary notation - only after % prefix.

Why?

no documentation for FPC says that 0 is octal prefix.

So this is an extension. Other languages don't seem to care about FPC documentation?

Ok, you are a boss.

However, the problem can be solved using conditional compilation, but this will also affect the tests. Can you take over the editing of tests?
By the way, the current documentation of Val() is rather poor and does not contain an adequate description of its behavior.

sorry, I cannot. I just wanted to disable 0 as octal prefix, that's it.

Just try replacing the current MainCaseMacro with this one:
  case p^ of
    '$': Result := ConvFunMacro(p+1, aCount-1, TypeMacro rxHex, OutMacro);
    '%': Result := ConvFunMacro(p+1, aCount-1, TypeMacro rxBin, OutMacro);
    '&': Result := ConvFunMacro(p+1, aCount-1, TypeMacro rxOct, OutMacro);
    '0':
      begin
        if aCount = 1 then
          begin
            aValue := 0;
            exit(True);
          end;
        case p[1] of
          '0'..'9':
            begin
              SetDecTrueMacro
              Result := ConvFunMacro(p+1, aCount-1, TypeMacro rxDec, OutMacro);
            end;
          'X', 'x': Result := ConvFunMacro(p+2, aCount-2, TypeMacro rxHex, OutMacro);
        end;
      end;
    '1'..'9':
      begin
        SetDecTrueMacro
        Result := ConvFunMacro(p, aCount, TypeMacro rxDec, OutMacro);
      end;
    'X', 'x': Result := ConvFunMacro(p+1, aCount-1, TypeMacro rxHex, OutMacro)
  end;

It seems like it should work, but the tests are likely to fail.

See b750fea8

Thanks, ok now.


Operation is created twice [8]

Describe the Bug:
When a user creates and operation and clicks submit, the operation is created twice
Probability (how likely the bug is to happen, scored from 1-5): 4
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 4
(For example, effect of 5 is "the entire app crashes and makes it unusable for all users" or "the bug causes the wrong data to be saved, with critical information (e.g. payment) being affected."
Effect of 1 is "It makes some styling look a little bit weird.")
Steps to reproduce the behaviour:
https://www.loom.com/share/878875480f92489cb8d1ea3caacc7467
Screenshots:
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context

Closing, this occured because my db was in a funny state


fix: Fixes the typesVersions map to fix incorrect auto-import paths

Fixes the same auto-import path issue as mentioned in Melt UI: https://github.com/melt-ui/melt-ui/pull/56

Thank you!!


row_groups argument in ak.from_parquet seems to have no effect

Writing multiple row groups seems to work
>>> import awkward as ak
>>> ak.__version__
'1.2.0rc1'
>>> array = ak.Array([1, 2, 3, 4, 5, 6, 7, 8])
>>> ak.to_parquet(ak.repartition(array, 2), "array.parquet")
>>> import pyarrow.parquet as pq
>>> f = pq.ParquetFile("array.parquet")
>>> f.num_row_groups
4

but then restricting via the row_groups argument in ak.from_parquet seems to have no effect
>>> ak.from_parquet("array.parquet", row_groups=0)
<Array [1, 2, 3, 4, 5, 6, 7, 8] type='8 * int64'>
>>> ak.from_parquet("array.parquet", row_groups=[0, 1])
<Array [1, 2, 3, 4, 5, 6, 7, 8] type='8 * int64'>

The argument was completely ignored. I must have intended this, but forgot. PR #748 fixes that.


autogen: guile fails to compile (in procedure load-thunk-from-memory: Invalid argument)

Originally reported by @LigH-de on https://github.com/msys2/msys2/issues/124.
Last week I set up a completely new instance of MABS, which downloads an up-to-date MSYS2 environment for building various audio and video tools. Among those, sndlib requires autogen, which requires guile. Issue #116 was just closed, but it seems to work only in MSYS2 environments which were installed a lot earlier.
In this new MSYS2 installation, autogen does not work because it seems to require a lot of guile modules which cannot be loaded from a cache, but cannot be compiled either. Attached is the stderr output of autogen --version just as it is used to check for the availability of autogen in configure scripts. Head snippet:
;;; WARNING: loading compiled file /usr/lib/guile/2.2/ccache/ice-9/eval.go failed:
;;; Throw to key system-error with args ("load-thunk-from-memory" "~A" ("Invalid argument") (22))
;;; WARNING: loading compiled file /home/LigH/.cache/guile/ccache/2.2-LE-4-3.A/usr/share/guile/2.2/ice-9/common-list.scm.go failed:
;;; In procedure load-thunk-from-memory: Invalid argument
;;; note: auto-compilation is enabled, set GUILE_AUTO_COMPILE=0
;;;       or pass the --no-auto-compile argument to disable.
;;; compiling /usr/share/guile/2.2/ice-9/common-list.scm
;;; compiling /usr/share/guile/2.2/system/base/compile.scm
;;; it seems /usr/share/guile/2.2/system/base/compile.scm
;;; is part of the compiler; skipping auto-compilation
;;; compiling /usr/share/guile/2.2/system/base/syntax.scm
;;; WARNING: compilation of /usr/share/guile/2.2/system/base/syntax.scm failed:
;;; In procedure variable-ref: variable is unbound: #<variable 204a9688 value: #<undefined>>
;;; compiling /usr/share/guile/2.2/system/base/language.scm
;;; WARNING: compilation of /usr/share/guile/2.2/system/base/language.scm failed:
;;; In procedure variable-ref: variable is unbound: #<variable 204a9688 value: #<undefined>>
...

autogen-version.txt

This issue appears to be specific for Win32 MSYS2 installations. In a Win64 installation it works well.

Did you solve the problem ?


Scale returns some containers in initialized state

Seeing weird container states after a scale operation.  I'd expect everything to be up before we return.
$ deis scale web=10
Scaling processes... but first, coffee!
done in 124s
=== zircon-addendum Processes
--- web:
zircon-addendum-v3-web-1pchs up (v3)
zircon-addendum-v3-web-23c64 up (v3)
zircon-addendum-v3-web-duj6q up (v3)
zircon-addendum-v3-web-j00na up (v3)
zircon-addendum-v3-web-sjljt initialized (v3)
zircon-addendum-v3-web-vus0o up (v3)
zircon-addendum-v3-web-wmp6k initialized (v3)
zircon-addendum-v3-web-xx3qd up (v3)
zircon-addendum-v3-web-ze4yo initialized (v3)
zircon-addendum-v3-web-vkrt8 up (v3)

I should note that this occurred on a "cold" cluster, where the deis/slugrunner image was not yet present on all the worker nodes -- hence the 124 seconds.   Tho one out of many of them are 120s, so technically your scale up could take 10+ minutes if every single timeout loop is touched.
Problem here is very likely like you said the slugrunner image is being copied in for the first time and is huge, this should however keep a pod in a Pending state. Right now the scale operations may not be super friendly to that first download to be totally honest.  Should we? How long should we let an image download from a k8s perspective before our scaling cope bails out?
When the image is downloaded from a kubernetes perspective everything looks quite good because there are no healthchecks by default from outside - It thinks the image is there and everything is ready to start booiting up. Basically k8s sees it as started pods so any downloads of the slug itself and untar'ing is going to be extra "wait" time.
To me this all spells out some of the weaknesses of the slug builder / slug runner concepts and how we are somewhat breaking the k8s model with it.


Merge master -> refactor

This merges master -> refactor since a rebase is not as useful or easy due to https://github.com/foundriesio/docs/commit/a8927e432e1c0ad41527c1a05c7848d846843311 and the manual adding of files thereafter. Once merged we should complete the refactor as soon as possible, this gives us a window to merge.

Docs for 9ff85a3e427565ffbab94bbe9549de1ac209e9eb are browesable at: https://ci.foundries.io/projects/fio-docs/builds/558/docs/artifacts/html/index.html

Merging to continue and finish getting-started draft in the next hours, can always later revert. But I feel something is going to go wrong with team communication if I rebase my outstanding PRs on this #97 #88 #92


GPU need

Can I use 4 NVIDIA 4090 GPUs to train this model？

Can I use 4 NVIDIA 4090 GPUs to train this model？

Yes, just make sure your environment is configured correctly. For specific parameter settings, please refer to the Experiment chapter of our paper.

Can I use 4 NVIDIA 4090 GPUs to train this model？

Yes, just make sure your environment is configured correctly. For specific parameter settings, please refer to the Experiment chapter of our paper.

Thanks for your reply.  When will the training data, test data, and test code be released? I would like to follow up on this wonderful work in the future.

@AsukaCamellia Permission to republish the source data has not yet been granted, but a script for the data remastering will be released this week. Some test data and evaluation code will be released later, but their priority is not high compared to other current tasks, so please wait patiently~

@AsukaCamellia scripts Now, the data production instructions and scripts have been released. If you have any questions, please feel free to submit ISSUE and PR.

@AsukaCamellia scripts Now, the data production instructions and scripts have been released. If you have any questions, please feel free to submit ISSUE and PR.

Thank you so much for your open-source spirit. I believe your work will become very impactful!


Feature request: webinterface like rpki-validator-3

Hello, i love the stability and easiness of routinator as RP and RTR-server, but i also like the GUI of ripencc's rpki-validator-3 very much. By chance, do you have any plans for adding a similiar webinterface to routinator as well? Thanks

We don’t have any plans. This isn’t because we are opposed to the idea but rather we are a little uncertain what use cases people have for using the UI.
Could you perhaps outline which parts of the RIPE NCC Validator’s UI you are using and would like to see offered by Routinator?

Not knowing the reason for 'shadom' for this feature I would like to mention our usecase for this request and that is editting the 'exceptions.json'. It's much easier for 1st/2nd line to whitelist prefixes from GUI perspective than editting a json file. If this could be added, that would be really great.

The one thing I would kind of like to avoid is the web interface actually changing configuration or state of a running Routinator because that suddenly means we need to supply some kind of protection against tempering with it.
Things like a HTML/JS frontend for the validity check should totally be doable.

changing configuration or state of a running Routinator

Does that include 'editting' the whitelist to you? That would the most import reason to have one for us. I agree that the webinterface must then not be left open to anyone (nginx with authorization in front). Could it be a configuration option which is off by default? Or will this make the interface part too complicated.

I’m tempted to do a SLURM editor as a separate, stand-alone thing. The extra effort to then set it up would serve as a bit of protection against someone just setting an option and opening themselves up for trouble. It’ll also keep Routinator smaller, which is probably a good thing.

We often have requests from people that are learning RPKI to "check their work..".  The "Preview" feature in the NCC's validator is generally where I point them to...

Routinator 0.8.3 now includes a UI for checking the validity of prefixes. I’ll close this issue.
If you feel there are features missing from the UI, please either reopen or create a new issue.


chore: update the test-transformer command to support passing arguments

#3286  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite


Feature request: Logging changelog when deploying

It would be great if Shipit could log the commits that are being deployed.
It would be helpful to see history of the deployment.

👍  This would be a feature in https://github.com/shipitjs/shipit-deploy
Feel free to make an issue there and/or send a PR!


ERC20.transaction doesn't assigns to WriteOperation.transaction property fix

This one fixing the ERC20 implementation bug, where ERC20.transaction never assigns to Contract.transaction for any given method that writes on chain.
Obviously it's a bug spreading all over the Token implementation. So this fix should be applied to all ERC*.swift methods that returns WriteOperation.
This is the special case of #712.
Closes #711

I'd completely remove transaction property from Contract. It's a perfect place for side-effects and other uncatchable errors.
I'll actually strongly advocate for removing this property.
If you could give me some pros about having it I'd gladly read that.
@yaroslavyaroslav @janndriessen

I'd completely remove transaction property from Contract. It's a perfect place for side-effects and other uncatchable errors. I'll actually strongly advocate for removing this property. If you could give me some pros about having it I'd gladly read that.
@yaroslavyaroslav @janndriessen

For the long term I agree (and I would actually completely rewrite the ERC20 class) but as I understood Yaroslav this was only supposed to be a quick fix. Also about your other comments, he articulated that this will have to be done for all classes in #712.

@janndriessen
As a quick fix, it's completely acceptable. But I'll open a separate issue with links to this PR and the comment from above about removing that transaction object from contract class.
It will be open for a discussion.

@yaroslavyaroslav  But I still think we should update the rest of the functions across this and other ERCs implementations we have. Because it's basically the same bug that is repeating from one function to the other.

@yaroslavyaroslav I think this PR is sort of ready to be merged but a have a few concerns that were reflected in my comment above.
To rephrase my comment https://github.com/web3swift-team/web3swift/pull/713#discussion_r1052511329: I think we should either update all other functions in this PR or in another PR that is forked out of this branch and then we are good to merge.

@yaroslavyaroslav I think this PR is sort of ready to be merged but a have a few concerns that were reflected in my comment above. To rephrase my comment #713 (comment): I think we should either update all other functions in this PR or in another PR that is forked out of this branch and then we are good to merge.

I'll look into adding a PR on top for that.

I'd completely remove transaction property from Contract. It's a perfect place for side-effects and other uncatchable errors.

The designed pipeline of contract iteration flow was:
ERC+*** -> Contract -> [Read|Write]Operation.
The intention that I've got in minds while refactoring this bit was:

The main intent of this bit of refactoring was to provide unified pipeline to both create, setup and send transaction when it comes to the contract interaction. Therefore there was 3 major classes has been kept: [Read|Write]Operation, Contract, ERC+***.

The ERC+*** classes are treated by me as just a convenient wrappers around the bare bone EthereumContract one. So the Contract type which is inherits of it treated by me the same: as a some convenient yet general wrapper around the EthereumContract.
In the given pipeline *Operation awaits that there would be some *Contract instance provided that is incapsulates all Contract related setup and that should be encoded and sent to a chain to made a success contract iteration.
So to sum up: tx property of a Contract type is provided there to make things a bit more consistent. Actually I had an idea to drop all ERC+*** tx and contract implementation detail and made it belongs to Contract one.

As a quick fix, it's completely acceptable. But I'll open a separate issue with links to this PR and the comment from above about removing that transaction object from contract class.

A contrary question here is, how the user would be manage an arbitrary (Non ERC+* listed contract) without the tx property. Like do you assume that they will be setup it in WriteOperation.transaction directly?

@JeneaVranceanu I guess we've paused this topic for a holiday, but it's still required to be done, to present appropriate bug-fix.

@JeneaVranceanu I guess we've paused this topic for a holiday, but it's still required to be done, to present appropriate bug-fix.

Do you see anything else to be done besides the clean up in https://github.com/web3swift-team/web3swift/pull/715?

@JeneaVranceanu I guess we've paused this topic for a holiday, but it's still required to be done, to present appropriate bug-fix.

@yaroslavyaroslav No, it looks good now. I've left a review in #715 but this particular PR is ready to be merged.

@janndriessen please approve the PR and merge it within


Question 8 Part 1

Do you guys think the row names and column names functions are actually needed? They are already mentioned in the previous questions.

No issue I have been able to remove the code


fixed backlog setting

@lodejard to look at the change.
@chopdown can you check out https://github.com/aspnet/Home/blob/master/CONTRIBUTING.md and fill out a contributor license agreement (CLA)? It's required for all PRs to the repos. Should only take a few minutes to go through.

@Eilon CLA done.

Thanks, waiting to see it show up on our side.

CLA confirmed.

@lodejard can you take a look?


Increased flexibility

I'm working on running multiple phoenix applications on nerves on top of vultr cloud hosting (using https://github.com/nerves-project/nerves_system_vultr). With this setup it is difficult to run a reverse proxy such as nginx, so I'd like to use master_proxy as a proxy to the different applications but am running into some difficulties. I wrote a little more background in https://github.com/sasa1977/site_encrypt/issues/26
I have made some local changes to master_proxy that allows me to use it in this setup, along with some other enhancements that (I feel) make it more usable and flexible (you can preview the changes in my repo if you'd like: https://github.com/axelson/master_proxy/tree/flexiblity-1 although I need to clean it up still)
Feature requests:

Ability to inject configuration into master_proxy

This is important for me to set SiteEncrypt.https_keys/1 as part of the configuration so that I can serve https traffic from master_proxy
In my implementation this is the merge_config/2 callback function

Ability to start master_proxy as part of your supervision tree

This is important to me because master proxy need's configuration from site_encrypt, and the site_encrypt configuration SiteEncrypt.https_keys/1 is not available until after site_encrypt is running so I need better control over when master_proxy starts

Ability to control which endpoint is chosen by passing in domain

This isn't really that important but I think it makes it more easy to use master_proxy in the common case of serving two separate websites/domains (it is my perception that two or more separate domains is the most common use-case for master_proxy)

Allow the user to use a function to decide the endpoint to call at runtime

This isn't necessarily that important either, but I think it is a nice API

Before I clean up my changes to submit them as a formal PR, I wanted to let you know and solicit feedback. 1 and 2 are solved by creating a behaviour and having the user create a proxy module that has use MasterProxy and pass in the arguments from there, when gives user's the flexibility if they want to use the application environment, which I think is in the spirit of the official library guidelines: https://hexdocs.pm/elixir/library-guidelines.html#avoid-application-configuration

These are great suggestions! Starting master_proxy as part of the supervision tree has also been proposed and implemented here, but exists as a separate repo.
https://github.com/jesseshieh/master_proxy/pull/9
To be honest, this was just a quick and dirty implementation that solved the problem, and I welcome all these changes, but I'm not sure if I have too much time to really dig into this deeply. Would you like to become the maintainer of this project and take it in whatever direction you feel is right?

These are great suggestions! Starting master_proxy as part of the supervision tree has also been proposed and implemented here, but exists as a separate repo.
https://github.com/jesseshieh/master_proxy/pull/9
To be honest, this was just a quick and dirty implementation that solved the problem, and I welcome all these changes, but I'm not sure if I have too much time to really dig into this deeply. Would you like to become the maintainer of this project and take it in whatever direction you feel is right?

Hi @jesseshieh, after thinking about it a bit, I would be glad to become the maintainer. Although my plan is to form a group of elixir developers that would be willing to collectively maintain a few libraries. And thanks for the link to that other PR.

Hi @jesseshieh, after thinking about it a bit, I would be glad to become the maintainer. Although my plan is to form a group of elixir developers that would be willing to collectively maintain a few libraries. And thanks for the link to that other PR.


Missing description

Missing description in some places on the left bar

Is this still an issue Moran?

No description in the data from these places.


Refactor features.xml

We want to hand-code our single feature and add it as an attachment to the main artifact. We still do not want to lose the test, hence we need to create an integration test module.

Features are a flat namespace. Using pt-triemap plays nice with other, hence we will not do this.


Exclude routes

#21 is closed, but its fix only half-works:

routes excluded with @doc.route(exclude=True) do not appear in the swagger UI, but,
they are listed (although empty) in the /openapi/spec.json

My expectation is that an "excluded" route would not appear in either the openapi/spec.json or the swagger UI.

looks like it just needs a one liner to fix (line 144) ..
        if methods:
            paths[uri_parsed] = methods

(ie the if line)


Passenger 5 support.

For your consideration.  In addition to Passenger 5 support, this PR also includes some (opinionated) refactoring of the plugin with the goal of making it a little more flexible.  I'm absolutely open to discussion and/or revisions on implementation here.

Added support for Passenger 5's new restart mechanism and made it the default.
Added new configuration options.
Changed namespace from :deploy to :passenger since :deploy is commonly defined/used in users' deploy.rb files.
Removed automatic placement of passenger:restart in the after :publishing callback. This results in the plugin being less plug-and-play, but more flexible i.e. users can call passenger:restart wherever and whenever they want in their deployment flow.
Updated README based on all the above changes.

This PR addresses #2.

Why not use deploy_to instead of deploy_path, which just wraps the deploy_to string in a Pathname object?

I think it's also worthwhile enabling Passenger's --ignore-app-not-running option by default, because Capistrano halts the deploy if the app isn't running already, which doesn't make a lot of sense (especially on the first deploy, for example).

@pjkelly I have a few concerns with this PR, though I certainly appreciate the effort.

Thank you for bringing to my attention that the URL in the README was out-of-date.  I've corrected it in another commit.

Why does the user need to tell capistrano what version of passenger is running?  We should be detecting that automatically on a server-by-server basis.  For your reference, here's the output of passenger -v on a machine that has v4 installed:
[myUser@myServer]$ passenger -v
Phusion Passenger version 4.0.57
"Phusion Passenger" is a trademark of Hongli Lai & Ninh Bui.

Can you use passenger -v to check the version instead of requiring the user to specify it?

Changing the namespace is a breaking change.  While I understand your reasons for it, if you want to make the change, we need to support both and log a deprecation warning for now.  If you would like to do this, please create a separate PR for it.

@betesh & @jalada - awesome feedback - thanks for getting back to me so quickly.
I'm with you on all this.  I think the best course of action is to adjust this PR so that it only focuses on the Passenger 5 support, and then I'll open a separate PR with the namespace stuff - in retrospect, I should've done that in the first place.
I'll get these revisions going soon.

@pjkelly Thanks, looking forward to it!

@betesh - I just pushed the changes we discussed for you to review.  Let me know if you have any questions/feedback.

I still have 1 strong objection: You're making the assumption that all servers are running the same version of passenger.  This is what I was referring to in my previous comment about "detecting that automatically on a server-by-server basis"
If instead of 3 separate tasks, you run all the logic in a single on/with block, it will reduce code repetition (making the code more maintainable--I have very strong opinions about unnecessary duplication and its impact on maintainability), and it will also allow each server to run a different version of passenger.  Lastly, supporting 2 separate restart tasks that can be called in isolation gives the impression that it would make sense to call either one.
Also, wouldn't it be easier to use passenger-config --version than passenger -v, since it only outputs the version number?  Does passenger-config --version work in Passenger 5?

I can confirm it works with 5:
$ passenger-config --version
Phusion Passenger 5.0.4

@betesh - thanks for the feedback - the duplication was bugging me too.  I thought the way it was setup would support multiple versions of passenger at the same time, but was mistaken.  I went ahead boiled everything down into a single task now (passenger:restart) that works the way we’ve discussed.  I ran the plugin against two servers with different versions of Passenger and they were both restarted with the appropriate commands.
Version Checking
The reason I elected to go with passenger -v is because the output format of passenger-config --version changed at some point along the way:
# Passenger 3
$ passenger-config --version
3.0.9
$ passenger -v
Phusion Passenger version 3.0.9

“Phusion Passenger” is a trademark of Hongli Lai & Ninh Bui.

# Passenger 5
$ passenger-config --version
Phusion Passenger 5.0.4
$ passenger -v
Phusion Passenger version 5.0.4

“Phusion Passenger” is a trademark of Hongli Lai & Ninh Bui.

I don’t have a server running Passenger 4, so I couldn’t verify that at the moment, but this discrepancy was enough to suggest that going with passenger -v was the simpler option.

Looks good.  I will test it against passenger 4 when I get a chance and if all goes well, I will merge it.

@betesh - :+1:

@pjkelly I added a task to add passenger and passenger-config to the list of commands that need to be prefixed by rvm.  This made passenger -v work in passenger 4 with no additional configuration.
Can you remove whatever overridden values you are using for passenger_environment_variables and test that this works in passenger 3 & 5 (assuming you are using RVM)?

@betesh - we don't use RVM, so I don't have the ability to test that - apologies.

@pjkelly In that case can you take a quick look at the commits I just added to make sure they look sane enough and let me know if you spot any obvious mistakes?  https://github.com/crushlovely/passenger/compare/passenger-5-support-and-other-adjustments...capistrano:master

It looks good to me!

I'll confirm on Monday but Passenger 5 generally installs outside of RVM, even if you have RVM:
$ which rvm
/usr/local/rvm/bin/rvm
$ which passenger
/usr/bin/passenger

This is because Passenger 5 is C++ and language agnostic now. It might not cause a problem, but like I say, I'll double check :)

Thanks @jalada I was unaware.  Please let me know once you confirm.

@jalada I installed passenger 5 on one of our servers this morning, and it is thoroughly installed through RVM
[my_user@my_host ~]$ which rvm
~/.rvm/bin/rvm
[my_user@my_host ~]$ which passenger
~/.rvm/gems/ruby-2.1.3/bin/passenger
[my_user@my_host ~]$ passenger -v
Phusion Passenger version 5.0.5

"Phusion Passenger" is a trademark of Hongli Lai & Ninh Bui.
[my_user@my_host ~]$ passenger-config --version
Phusion Passenger 5.0.5

In your case, you'll probably need to override the passenger:rvm:hook task so that passenger does not end up appended to rvm_map_bins.

@betesh yes but that isn't necessarily required nor recommended. Passenger is now language agnostic and the version of Ruby you run Passenger as is not dependent on the version of Ruby you run your app with. In fact you can use multiple versions of Ruby on the same installation of Passenger. Therefore automatically tying Passenger to the RVM you are deploying your app with doesn't make sense; that should be an opt in, not an opt out.
More information about Passenger 5 and the relation with Ruby

@jalada Thank you for the link.  I took a look and I think the best approach would be to see if there's a way we could automatically detect where passenger is installed and decide based on that whether to add it to the command map.  No opting (in or out) should be necessary.
Suppose we run which passenger and see what the result is.  If it's not found in any $PATH directories, and RVM is being used, than I think it's reasonable to assume that it's installed through RVM.  If it is found, of course, than we don't add it to the command map.
Thoughts?  Opinions?

Yeah, if that's possible it sounds like a plan. It's a shame that Capistrano RVM works like that and doesn't just source in the environment in the shell it's running, because then passenger would be resolved based on the PATH including RVM paths if they are included. Is there no way of doing that? (I'm unfamiliar with the workings of capistrano-rvm)

@jalada Try it with the lastest commit (68b86f1) and let me know if this works for you out-of-the-box.


A few weird variable names in TractableBufferStock

The variable names in TractableBufferStock.py have been cleaned up as of 5/11/2016, but there are a few that still need fixing.  All of these seem to be temporary algebraic conveniences, and I don't know how to describe them in words off the top of my head:

natural
Beth
Pi
zeta

This seems to have been fixed in #184, right?

I thought I closed this?
On Tue, Feb 5, 2019, 5:48 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

This seems to have been fixed in #184
https://github.com/econ-ark/HARK/pull/184, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/36#issuecomment-460592834, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFYwf1HgG-OD2tsc-5EvzXlJ_pGshks5vKWF6gaJpZM4Icc8B
.

No, I think you just closed the duplicate issue in #183


Log Tasklist at beginning and end

Write the tasklist to a file when starting and stopping the trace

Done


nwipe: init at 0.24

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: nwipe
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24
shrinking /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24/bin/nwipe
gzipping man pages under /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24/share/man/
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24/bin
patching script interpreter paths in /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24
checking for references to /build in /nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24...
/nix/store/jqlh3g2ppxfi8mm2js6ya0xwf96m378c-nwipe-0.24

Success on x86_64-linux (full log)
Attempted: nwipe
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24
shrinking /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24/bin/nwipe
gzipping man pages under /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24/share/man/
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24/bin
patching script interpreter paths in /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24
checking for references to /build in /nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24...
/nix/store/s2jy115b95ddz1j1ibnncclw93v9ikrn-nwipe-0.24

Success on x86_64-linux (full log)
Attempted: nwipe
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24
shrinking /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24/bin/nwipe
gzipping man pages under /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24/share/man/
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24/bin
patching script interpreter paths in /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24
checking for references to /build in /nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24...
/nix/store/c20yyqc4jhngcjza2262gy1fpsng19f3-nwipe-0.24

Success on aarch64-linux (full log)
Attempted: nwipe
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24
shrinking /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24/bin/nwipe
gzipping man pages under /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24/share/man/
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24/bin
patching script interpreter paths in /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24
checking for references to /build in /nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24...
/nix/store/iylg1iz4q3yyh4j84gfm3irlhp4q21q6-nwipe-0.24


On/off

Hi.
Is it possible to invert the I/O status(?) for ”Shelly-Minimal_water_heating.js”? So when price is high, shelly turns on.
With current script it’s the other way around: Low price = shelly on.
Thanks

You should do small mofification to the code. Basically just find the rows where relay is turned on or OFF and switch the controls opposite.
I mean lines that contain this:  "Shelly.call("Switch.Set", "{ id:0, on:true}"
Just change on:true to on:false   and then on:false to on:true


fix(docs): improve wording of docs for global Number rules

This is a small change that fixes the back-to-front wording on the docs for the rules noGlobalIsFinite, noGlobalIsNan and useIsNan.

Thank you!


[Bug]: Headers get squished

Bug Description
Not sure how long this has been (I think only starting today) but headers seem to be getting squished (see screenshot).
This does not occur in the default theme. But it does occur with the default header font (the one that's the default in Style Settings; in the screen shot I use "GloucesterW01-Bold"). I also installed the theme and Style Settings in the Help vault, and the same thing happens.
I think this only happens in Live Preview. It does NOT happen in Reading or Source mode.
Relevant Screenshot

To Reproduce

Use theme
have header
see header with multiple lines is squished.

Obsidian Version
0.14.5
Installer Version
0.13.30
Which editor are you using?

[ ] New Editor
[ ] Legacy Editor

In which mode(s) does the problem occur? (if applicable)

[ ] Source (Edit)
[ ] Reading (Preview)
[X] Live Preview

Which operating system?

[ ] macOS
[X] Linux
[ ] Windows
[ ] iOS
[ ] Android

Plugin-Specific Problems

[X] I took note of the specific instructions regarding those plugins.

Checklist

[X] I updated to the latest version of the theme (Settings → Appearance → Themes - Manage → Update).
[X] I made sure that the issue is related to this theme by checking that the problem does not occur with the default theme.
[X] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.
[X] If the bug occured after updating the theme, I read the last couple of entries in the changelog to make sure the bug is not an intentional change. (In that case, there will most likely be a style setting to turn it off again.)

ah thanks, didn't notice since I rarely have long headings. Fixed with latest update

Thanks :)

Has that latest update gone out yet? Because the issue still seems to occur.

Has that latest update gone out yet? Because the issue still seems to occur.

yeah, should be? Maybe GitHub has some hickups, try again in a few minutes?

working now?

Hm, no, it's still happening.

strange, I tried something else. try again?

Yes, now it's gone, thanks!


⚠ Bofrost.de has degraded performance

In e61e231, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1631 ms

Resolved: Bofrost.de performance has improved in 5d6cf5d.


[BUG] Not installable on debian bookworm.

Describe the bug
The latest released packages has dependencies to previous versions of a number of libraries:
apt install ./MusicPlayerPlus_3.0.1-3.deb
(...)
De volgende pakketten hebben niet-voldane vereisten:
 musicplayerplus : Vereisten: libavformat58 (>= 7:4.1) maar het is niet installeerbaar
                   Vereisten: libavfilter7 (>= 7:4.0) maar het is niet installeerbaar
                   Vereisten: libswresample3 (>= 7:4.0) maar het is niet installeerbaar
                   Vereisten: libavcodec58 (>= 7:4.2) maar het is niet installeerbaar
                   Vereisten: libswscale5 (>= 7:4.0) maar het is niet installeerbaar
                   Vereisten: libavdevice58 (>= 7:4.0) maar het is niet installeerbaar
                   Vereisten: libavutil56 (>= 7:4.0) maar het is niet installeerbaar
E: Kan problemen niet verhelpen, u houdt defecte pakketten vast.

To Reproduce

Download the latest release: MusicPlayerPlus_3.0.1-3.deb.
Install with apt.
See above.

Expected behavior
Installed MusicPlayerPlus.
Installed libraries on debian 12:
# aptitude search libavformat libaffilter libswresample libavcodec libswscale libavdevice libavutil
p   libavcodec-dev                                                                                - FFmpeg library with de/encoders for audio/video codecs - development files
p   libavcodec-extra                                                                               - FFmpeg library with extra codecs (metapackage)
p   libavcodec-extra59                                                                             - FFmpeg library with additional de/encoders for audio/video codecs
i A libavcodec59                                                                                - FFmpeg library with de/encoders for audio/video codecs - runtime files
p   libavdevice-dev                                                                                - FFmpeg library for handling input and output devices - development files
i A libavdevice59                                                                                - FFmpeg library for handling input and output devices - runtime files
p   libavformat-dev                                                                                - FFmpeg library with (de)muxers for multimedia containers - development files
p   libavformat-extra                                                                              - FFmpeg library with extra (de)muxers (metapackage)
p   libavformat-extra59                                                                            - FFmpeg library with additional (de)muxers for multimedia containers
i A libavformat59                                                                                - FFmpeg library with (de)muxers for multimedia containers - runtime files
p   libavutil-dev                                                                                - FFmpeg library with functions for simplifying programming - development files
i A libavutil57                                                                                - FFmpeg library with functions for simplifying programming - runtime files
p   libswresample-dev                                                                              - FFmpeg library for audio resampling, rematrixing etc. - development files
i A libswresample4                                                                                - FFmpeg library for audio resampling, rematrixing etc. - runtime files
p   libswscale-dev                                                                                - FFmpeg library for image scaling and various conversions - development files
i A libswscale6                                                                                - FFmpeg library for image scaling and various conversions - runtime files

Desktop (please complete the following information):

OS:  Debian bookworm
Version 12.5
Terminal: Putty (ssh)

Additional context

@hdijkema version 3.0.2 release 1 of MusicPlayerPlus may address this issue as I have modified dependencies to rely less on specific versions. A debian format installation package can be downloaded at https://github.com/doctorfree/MusicPlayerPlus/releases
This release of MusicPlayerPlus uses a Python virtual environment for compatibility with recent releases of several OS distributions. Unfortunately, this seems to have broken album cover art display in mpcplus. I'm working on addressing that but wanted to get this release out as it includes many bug fixes and enhancements. Album cover art display is disabled by default.
Please let me know if this fixes installation on Debian Bookworm as I do not have that distribution in my test matrix. Thanks for reporting this!

@hdijkema I somehow missed libavfilter, thanks for catching that. A new Debian format package with the dependency now on libavfilter-dev is available at https://github.com/doctorfree/MusicPlayerPlus/releases/download/v3.0.2r2/MusicPlayerPlus_3.0.2-2.deb
Hopefully this fixes installation on Debian. If not, let me know.

Thanks. I've tried to install it today. Looks like it works now (debian 12.7).


JavaFX WebView no longer works with Google Maps JS sdk

It looks like something has changed recently with the Maps JS SDK and it doesn't work properly with JavaFX WebView.
The map is empty and every tile shows the message "Sorry, we have no imagery here.". Tested with various versions of JavaFX from v8 to v21.
Tested on Windows.
It happens with this implementation as well as online sites based on the maps sdk.

seeing the same, quite unfortunate.

Hi.  I have reported the bug, see link. I wrote Kevin Rushford (no answer) but for whatever reason, the door to the internet, the webview is not getting any attention for improvements. Its just not working working correctly for many pages. Maybe one here with more influence can change that somehow. My 50 cents, its cause of interconnection to jxbrowser behind the scenes. Dont see any other explanation that webviev is that bad.
https://bugs.openjdk.org/browse/JDK-8331640?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel

Map not loading for my project as well, waiting for an update from Java team:

I found a solution - source
"My intuition was right, version 3.56 should have been specified and not 3.exp which uses jGoogleMaps"
I corrected the HTML for myself and it finally worked

version 3.56 should have been specified

Yes but 3.56 works fine for me to as a short term workaround, but will not be available forever .
This picture about version availablity is from https://developers.google.com/maps/documentation/javascript/versions


Handle (active) states for touch devices

Currently :active styles do not work on mobile because they are based on mousedown and mouseup events.
If on mobile use touchstart and touchend instead.
Fixes https://github.com/FormidableLabs/radium/issues/63

Thanks for the contribution! Going to close this one out. Please let me know if you want to reopen and keep moving forward on it. I think there's a decent chance that we'll want to solve this problem by using addCSS for real pseudo-selectors rather than emulated ones as in https://github.com/FormidableLabs/radium/issues/510


Scrolling to default stickers slow

When i switch to the sticker tab, it is very slow and can sometimes crash Bunny.

Likely just a Discord issue, come back once it's clear that this only happen specifically with Bunny

Likely just a Discord issue, come back once it's clear that this only happen specifically with Bunny

Ah Ok, thank you!


Routing on click of notificaitons

Hi Raix,
This is not really an issue but I wondering if its possible to do so or not. So by default, the user always lands in home page in my app when he opens the app. But is it possible to route him to different location in my app when he clicks the notification?

Check #107

Push.on("startup", msg => {})
Add data in payload eg. Route or item id

So this replaces Push.addlistener?


Torch.save() for large training Dataset

Hello,
I know this is not directly related to awesome-align, but I have a large training set of 10M source/target pairs and it takes 4 hours to process them before the actual model training even starts. So, I used the --cache_data option provided, but when the code reaches torch.save(self.examples, cache_fn) it just takes forever and it uses all my RAM (60GB is not enough on a VM - it swaps into HDD on my 32gb RAM PC and then takes forever to complete). Would you know any alternative way to save the Dataset and then reuse it before training to skip the loading overhead time?
Many thanks

Hello, thanks! I think you can subsample the data as it wouldn't require a lot of parallel data to finetune BERT for word alignment.


[Bug] - Plot members are allowed to review their own plots

Description
As a plot member you can review plots, you have permissions to build.

Haven't you fixed that issue already with these lines?
https://github.com/AlpsBTE/Plot-System/blob/88f16fc4eb9aa1f8ee945b8bcca49d90361f000b/src/main/java/com/alpsbte/plotsystem/commands/review/CMD_Review.java#L112-L120
Or where does this need changes?


cDPE Storage - Firestore Requests

Samples:
-Array membership
-Start at snapshot query
-Updating sample data
Tests
README
Composer updates

@billyjacobson actually, one more thing to add.
In addition to Array Contains samples, we need samples for updating elements in an array (arrayUnion and arrayRemove). Can you add these?
https://cloud.google.com/firestore/docs/manage-data/add-data#update_elements_in_an_array
Thanks!

Added the samples requested @ryanmats

fantastic - gonna merge.


Errors in docker swarm cluster with custom overlay network

I'm trying to get a cluster running on my swarm cluster where I've a custom defined overlay network.
here is my compose file
services:
  vmq0:
    image: vernemq/vernemq
    environment:
      - "DOCKER_VERNEMQ_SWARM=1"
      - "DOCKER_VERNEMQ_ACCEPT_EULA=yes"
      - "DOCKER_VERNEMQ_LISTENER__VMQ__CLUSTERING=<IP_ADDRESS>:44053"
    networks:
        - default

  vmq:
    image: vernemq/vernemq
    depends_on:
      - vmq0
    ports:
      - "1883:1883"
      - "8083:8083"
      - "18888:18888"
    environment:
      - "DOCKER_VERNEMQ_SWARM=1"
      - "DOCKER_VERNEMQ_DISCOVERY_NODE=vmq0"
      - "DOCKER_VERNEMQ_ALLOW_ANONYMOUS=on"
      - "DOCKER_VERNEMQ_MAX_INFLIGHT_MESSAGES=0"
      - "DOCKER_VERNEMQ_MAX_ONLINE_MESSAGES=-1"
      - "DOCKER_VERNEMQ_MAX_OFFLINE_MESSAGES=-1"
      - "DOCKER_VERNEMQ_LISTENER__TCP__DEFAULT=<IP_ADDRESS>:1883"
      - "DOCKER_VERNEMQ_LISTENER__WS__DEFAULT=<IP_ADDRESS>:8083"
      - "DOCKER_VERNEMQ_LISTENER__HTTP__DEFAULT=<IP_ADDRESS>:18888"
      - "DOCKER_VERNEMQ_LOG__CONSOLE=console"
      - "DOCKER_VERNEMQ_SHARED_SUBSCRIPTION_POLICY=random"
      - "DOCKER_VERNEMQ_ACCEPT_EULA=yes"
      - "DOCKER_VERNEMQ_LISTENER__VMQ__CLUSTERING=<IP_ADDRESS>:44053"
    deploy:
      mode: replicated
      replicas: 1
    networks:
        - default

networks:
  default:
    external: true
    name: overlaynet

Nodes were not able to discover each other unless I set DOCKER_VERNEMQ_LISTENER__VMQ__CLUSTERING.
I can see following interfaces in vmq0. 172.20.0.x is the custom overlaynet I created for all our services.
eth0: inet <IP_ADDRESS>
eth1: inet <IP_ADDRESS>
lo: inet <IP_ADDRESS>

and cluster query shows the following
+----------------------+---------+
| Node                 | Running |
+----------------------+---------+
|<EMAIL_ADDRESS>| true    |
+----------------------+---------+
|<EMAIL_ADDRESS>| true    |
+----------------------+---------+

Following interfaces are available on vmq
eth0: inet <IP_ADDRESS>
eth1: inet <IP_ADDRESS>
eth2: inet <IP_ADDRESS>
lo: inet <IP_ADDRESS>

and cluster query shows:
+----------------------+---------+
| Node                 | Running |
+----------------------+---------+
|<EMAIL_ADDRESS>| true    |
+----------------------+---------+
|<EMAIL_ADDRESS>| true    |
+----------------------+---------+

So, it all looks okay.  However, both nodes keeps printing the following warning every second
vmq0:
18:23:40.367 [info] successfully connected to cluster node<EMAIL_ADDRESS>18:23:40.367 [warning] terminate due to remote_node_not_available
18:23:40.367 [warning] connection to node<EMAIL_ADDRESS>has been closed, reconnect in 1000ms

vmq:
18:24:20.448 [info] successfully connected to cluster node<EMAIL_ADDRESS>18:24:20.448 [warning] terminate due to remote_node_not_available
18:24:20.448 [warning] connection to node<EMAIL_ADDRESS>has been closed, reconnect in 1000ms

Why am I seeing above warning?  How do I address this?  How do I validate the cluster is joined correctly?

@jjaimon vmq-admin cluster show will tell you that your cluster is OK, on the level of cluster awareness and metadata synchronisation.
Now, confusingly, those VerneMQ nodes will also try to open additional connections to distribute the data (the MQTT messages). This is like a second and separate cluster layer. The vmq-clustering listener is configured for that second layer. The vmq-admin cluster show command currently doesn't take into account that second layer at all.
The info log in your snippet is from the first layer. The warning logs are from the second layer.
What you describe seems a common issue, but I still have to look into it. The log comes from here:
https://github.com/vernemq/vernemq/blob/982724b99c70116f05373ff2acbb0a527cbe73d4/apps/vmq_server/src/vmq_cluster_node.erl#L232
Seems to indicate that the other node sends a tcp_disconnect, so maybe we shoot ourself in the foot here by some TCP config or something. Or it is a connectivity/network/overlay issue but then why would the connect attempt even arrive at the other node?
We need to wireshark this somehow.

How do I track the second layer?  which IP and port are it trying to access?  I can verify connectivity across vmq and vmq0.
I have been using pre-built docker image from docker hub.  I'll rebuild the image with wireshark/tcpdump and take a trace.  Which port or activity do you want to monitor?
BTW, vmq is launched in vip mode (default deployment mode). Does it make any difference if we use dnsrr?


Fix parsing of std::vector and add test, example and documentation

This change is

The intent for vector was that you could write an option multiple times and keep all the values. But this seems like reasonable behaviour as well.


Better error handling over throw new Exception

Codecov Report

Merging #957 into develop will decrease coverage by 0.17%.
The diff coverage is 42.6%.

@@              Coverage Diff              @@
##             develop     #957      +/-   ##
=============================================
- Coverage      41.91%   41.74%   -0.18%
  Complexity       774      774
=============================================
  Files            153      153
  Lines           5487     5534      +47
  Branches         781      781
=============================================
+ Hits            2300     2310      +10
- Misses          3018     3055      +37
  Partials         169      169

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/microsoft/azure/maven/FTPUploader.java
33.33% <ø> (ø)
0 <0> (ø)
:arrow_down:

...dlers/artifact/RunFromBlobArtifactHandlerImpl.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...soft/azure/maven/function/utils/FunctionUtils.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...handlers/runtime/DockerFunctionRuntimeHandler.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...a/com/microsoft/azure/maven/AbstractAzureMojo.java
43.87% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ndlers/artifact/RunFromZipArtifactHandlerImpl.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...handlers/artifact/MSDeployArtifactHandlerImpl.java
87.17% <ø> (ø)
6 <0> (ø)
:arrow_down:

...va/com/microsoft/azure/maven/function/RunMojo.java
80% <ø> (ø)
10 <0> (ø)
:arrow_down:

...a/com/microsoft/azure/maven/webapp/ConfigMojo.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...bapp/handlers/artifact/JarArtifactHandlerImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

... and 13 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 0d926ab...b395d05. Read the comment docs.


added note re: symlinks

@rneugeba I updated the docs in this new comprehensive repo per https://github.com/docker/for-win/issues/109#issuecomment-251307391. Please have a look.
(We already had this info in OSFX topic on Mac, but didn't add it to Windows. Let me know if the way that is written is better than what I wrote here for Windows.)
We should be serving docs.docker.com from this repo any minute now (maybe today). I'll make an announcement to the team, when that happens. In the meantime, after the PR is merged, the changes will show up on this preview site: https://docker.github.io/
Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

LGTM, merging, we can update further if needed.


Library refactor

This is a lot of work on library design.
Based on comments: https://github.com/home-assistant/core/pull/75727#discussion_r937572800
I have also added a basic types annotations.

@bieniu Please take a look when you have time.


Move utilitarian function from WMWorkloadTools to Utils

Fixes #7633
Moving any generic python functions that don't depend on WMCore code.
I guess we don't want to move functions that are too specific to WMCore (e.g. parsePileupConfig), do we?

Did you check if there are any calls expecting the old exception classes in the non-test code?

I just search the code. I couldn't find any place catching old exception. merging the code now.

I was going to check it this morning :( Anyways, thanks for double-checking Seangchan.
I've just noticed I forgot to update the unit test class name ..


Failed build when running with coverage on stable Rust

Both wayland-backend and wayland-client have a line like the following: https://github.com/Smithay/wayland-rs/blob/1fc7814bee5d90f83bab428c8167af43201f8a9b/wayland-backend/src/lib.rs#L46
This is a problem because if building a project with coverage will always try to enable this feature, which will fail if using a stable rustc. This should be changed so the coverage_attribute feature and coverage(...) attributes to go with it are not enabled unless compiled with nightly.
I think the easiest fix is probably to add a Cargo feature gate for nightly features and use it in place of #[cfg_attr(coverage, coverage(...))], as well as updating the spots where the feature is enabled.

I would also like to see this implemented.


@conda-forge-admin, please add user @eacharles

Adding @eacharles as maintainer.
@conda-forge-admin, please add user @eacharles

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @eacharles in conda-forge/tables-io-feedstock#4.


fix: Account.session params to be mandatory

:tada: This PR is included in version 0.18.0-next.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improving screen reader support for tooltip with sr-only

Fixes: https://github.com/primer/view_components/issues/1296
Context
This is take 2 of improving screen reader support. For more on the current issue, read the description of the issue linked above.
Recently we updated our tooltips to use visibility: hidden; to improve tooltip screen reader support because there appears to be a bug in accessibility API and custom elements and use of hidden attribute (https://github.com/primer/view_components/pull/1256). We couldn't use display: none; either because it seemed to result in the same bug. Using visibility: hidden; let us achieve what we wanted but it also introduced layout bugs on some pages which lead to a revert. This is due to visibility: hidden taking up space.
This PR aims to improve screen reader support and avoid layout issues by using sr-only to hide or show the tooltip visually.
Upon discussion with @jscholes we landed on the following:

When tooltip is label, it will always have aria-hidden to avoid duplicate announcements for screen reader users.
When tooltip is description, we will not have aria-hidden because duplicate announcement is acceptable for accessible descriptions to allow screen reader users to access the description text at any time.

Changes include
Other changes include:

greater system test coverage to test the JS behavior. the JS originally had unit tests in dotcom but we didn't import them here because PVC doesn't seem to have JS unit test setup. I added tests in system tests since I noticed we don't have much JS coverage at all.
ensure that aria-labelledby set by JS respects existing aria-labelledby on the button.

Reviewer
Please review the changes carefully. The thing to watch out for most are layout issues but I don't believe we should encounter it given sr-only should not take up space.

I pointed dotcom to these changes and verified the previously reported regression in a PR page for mobile is not present.
Previously reported regression in mobile:

This PR in mobile:

GitHub Staff can also test these changes by checking out kh-test-view-changes (this change) or kh-bad-changes (for the regressed view).

Hey @langermank, @primer/rails-reviewers, this is ready for review. Thank you!!


Update strings for push notifications

Connected with #20
It's safe to use it with PushKit only at that moment.
Signed-off-by: Denis Morozov<EMAIL_ADDRESS>

@morozkin Why do think unsafe to merge these updated strings on develop now?
IMAGE_FROM_USER and IMAGE_FROM_USER_IN_ROOM will be used only when https://github.com/matrix-org/sygnal/pull/20 will be merged, no?

@giomfo I'm worrying about clients with previous app versions. I don't know how old version of the app will react on unknown string

@morozkin: that is why the sooner the better. Previous app version will just display the raw string sent by the server. IMAGE_FROM_USER and IMAGE_FROM_USER_IN_ROOM are not received FTM

@giomfo it's ok to merge it right now in this case


Derive 404 page from base template

Currently it seems like it's completely separate HTML page. Same for license.html.
@mdrillin @indraraj @uidoyen

Gasp, nevermind, I looked in the wrong place. Probably should take a rest ;)


Merge pull request #1 from assisrafael/master

feat(uiDateMask): enable custom date masks and support for non-Date m…

Coverage remained the same at 97.328% when pulling 6b4f1f171b70e8df63961444accb75c00bbcef25 on 1fabiopereira:master into 3f51cad0c05e8549984cd63a02760136006bc15b on assisrafael:master.


ILP exact solver segfaults on Gurobi exception

ilp_exact and ilp_improve terminate with segmentation fault when Gurobi throws exception and do not print the exception message properly.
How to reproduce:
Setup Gurobi in a way that it is not possible to create a GRBEnv() without exception. E.g. install Gurobi, but do not install the Gurobi license. In this case when trying to create a Gurobi environment a license exception will be thrown which will make the tools to segfault.
Solution:
The reason behind this issue is the way the Gurobi exceptions are handled in the try-catch block of lib/ilp_improve/ilp_exact.h and lib/ilp_improve/ilp_improve.h. There are the following lines in the catch block
catch(GRBException e) {
            GRBEnv* env = new GRBEnv();
            GRBModel model = GRBModel(*env);
            std::cout << e.getErrorCode() << " Message: " << e.getMessage() << std::endl;
            return model;
}

Before printing the error message in the catch block, the code tries to initialize an empty Gurobi environment, which again throws the same exception as it did in the try block. This results into an uncontroled termination of the job, before reaching the line where the exception message is printed. As far as I understand the only reason for doing this is to be able to return something before the function execution ends, however the return value is never used in app/ilp_exact.cpp and app/ilp_improve.cpp. My suggestion is just to make the function computeIlp a void function and not return anything, so that the catch block just prints the exception message and exits the job gracefully. I have done this in my local copy of the source code, and it works fine, let me know if you approve of this solution and I will submit a pull request.

Hi Hayk,
Thank you. Pull rq would be, also please add yourself to the bottom of the
readme file then.
Thanks
Christian
On Sat, Dec 26, 2020, 12:32 Hayk Sargsyan<EMAIL_ADDRESS>wrote:

ilp_exact and ilp_improve terminate with segmentation fault when Gurobi
throws exception and do not print the exception message properly.
How to reproduce:
Setup Gurobi in a way that it is not possible to create a GRBEnv()
without exception. E.g. install Gurobi, but do not install the Gurobi
license. In this case when trying to create a Gurobi environment a license
exception will be thrown which will make the tools to segfault.
Solution:
The reason behind this issue is the way the Gurobi exceptions are handled
in the try-catch block of lib/ilp_improve/ilp_exact.h and
lib/ilp_improve/ilp_improve.h. There are the following lines in the catch
block
catch(GRBException e) {
GRBEnv* env = new GRBEnv();
GRBModel model = GRBModel(*env);
std::cout << e.getErrorCode() << " Message: " << e.getMessage() << std::endl;
return model;
}
Before printing the error message in the catch block, the code tries to
initialize an empty Gurobi environment, which again throws the same
exception as it did in the try block. This results into an uncontroled
termination of the job, before reaching the line where the exception
message is printed. As far as I understand the only reason for doing this
is to be able to return something before the function execution ends,
however the return value is never used in app/ilp_exact.cpp and
app/ilp_improve.cpp. My suggestion is just to make the function computeIlp
a void function and not return anything, so that the catch block just
prints the exception message and exits the job gracefully. I have done this
in my local copy of the source code, and it works fine, let me know if you
approve of this solution and I will submit a pull request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KaHIP/KaHIP/issues/70, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLP7SXOKMUMQJFIHLVERADSWXCWBANCNFSM4VJ3ASKQ
.


ELD: highcharts v8.0

Type: Missing
Summary:
ELD: highcharts v8.0
Details:
Proj: Core Platform
Comp: highcharts v8.0
File: /highcharts-8.0.0/vendor/jspdf.src.js
Line: 9690
Issue: 5
Resolution:
Provide license attribution for comment in SRC.  The code following this comment is based on sample code from the cited stackoverflow.com webpage, as found at https://stackoverflow.com/questions/10135697/regex-to-parse-any-css-font/ (see specifically https://stackoverflow.com/a/10136041). Material o
Affected definitions:

highcharts<PHONE_NUMBER>45a074e465c34bdadf97eeb54c9ce9

@capfei Hi, can you resolve this conflict. Thanks.


Filter expression is not a boolean

Stack trace: https://sentry.io/share/issue/7b2b8a330c4548b6b8071c150454a262/
Sentry reference: FHIR-SERVER-R

We no longer have the supporting information to reproduce this bug.


Subject contains all caps or numbers and symbols

Yep. Keeping this in draft until requirements are set.


Seems newest list from landing is gone

Uncaught (in promise) TypeError: Cannot read properties of null (reading 'id')
    at 41d068c.js:1:10873
    at Array.map (<anonymous>)
    at f.<anonymous> (41d068c.js:1:10838)
    at v (152af89.js:2:16194)
    at Generator._invoke (152af89.js:2:15982)
    at Generator.next (152af89.js:2:16623)
    at r (152af89.js:2:2747)
    at l (152af89.js:2:2950)
    at 152af89.js:2:3009
    at new Promise (<anonymous>)

I will check it

strange, seems fixed for now? Does something change from the server-side?

It seems work well now. I think that maybe back-end interface just now is not stable.

Some nft from that list wasn't properly handled I bet https://kodadot.xyz/rmrk/gallery?search=&sort=UPDATED_AT_DESC&page=1
naturally people are listing new nft so hard to see what was making issue :)

Oh I see. I will try to reproduce it.

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

Don't worry. I have caught the error. Give me some time to fix it  🚶

woaah, the meta = null

Yeah hence it isn't occurring lowering priority, if we don't solve it happy to close it and snipping if it happen again lol 😅

Don't worry. I have caught the error. Give me some time to fix it  🚶

Seems you're a new superior bug hunter 🤟🤘 engineering

Hey sers, what's the deal with this issue? Seems fixed?


Small fix in the steps to install

The setup process in the README:
$ git clone https://github.com/olivere/es
$ go get
$ go build
$ ./es help
is missing a
$ cd es-master
between the "git" and the "go get" lines

@branchnetconsulting Hi there. Are you using it against ES 1.x or 2.x. This tool is only supported for 1.x.

@branchnetconsulting Oops. Sorry, wrong ticket...

I fixed the installation steps. But as I said in #5: This tool only works for Elasticsearch 1.x and I currently don't have the time to port it to 2.x.

Ah, thanks for clarifying.  Yes I am using the latest stable ES so this
project isn't for me.  Thanks for pointing that out.
On Fri, Apr 8, 2016 at 10:07 AM, Oliver Eilhard<EMAIL_ADDRESS>wrote:

I fixed the installation steps. But as I said in #5
https://github.com/olivere/es/pull/5: This tool only works for
Elasticsearch 1.x and I currently don't have the time to port it to 2.x.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/olivere/es/issues/6#issuecomment-207445617


Filtering of examples per runtime is broken

Parent issue: https://github.com/fabric8-launcher/launcher-planning/issues/214
Select Deploy from Example -> Click on Istio Routing -> Choose Thorntail
There should be only one version. Two are shown

See https://github.com/fabric8-launcher/launcher-booster-catalog/tree/master/thorntail/redhat


chinese locale

Changed the Chinese locale to make it closer to the Chinese custom and follow the localeplanet rules.
Added its corresponding test.

nice +1

Thanks. I’ll port this to d3-time-format for 4.0.


add v1.73 ansible role to the operator

This PR is ready for review and merge. It needs to be cherry-picked to the 1.73 branch also


avoid tiny numbers passed to scikit-learn  Scaler that detects otherw…

…ise feature as constant in version >= 1
avoid skip from https://github.com/scikit-learn/scikit-learn/issues/19726

Currently -1 on this solution, let's see if I can convince @agramfort in the sklearn thread...


Vercel e2e test

What

NP fails on vercel deployments

Why

something to do with @prisma/client import not working

How

write an e2e test

We have a bunch of these at https://github.com/prisma/e2e-tests/tree/dev/platforms-serverless


[Improve][dist]add shade check rule

Purpose of this pull request
Globally check that shade related classes are used correctly in project.   Log format：
org.opentest4j.AssertionFailedError: guava shade is not up to code, need add prefix [org.apache.seatunnel.shade.].
E:\work\seatunnel\seatunnel-api\src\main\java\org\apache\seatunnel\api\table\type\MapType.java
com.google.common.base.Preconditions.checkArgument  [27]
com.google.common.base.Preconditions.checkNotNull  [28]
E:\work\seatunnel\seatunnel-core\seatunnel-spark-starter\seatunnel-spark-starter-common\src\main\java\org\apache\seatunnel\core\starter\spark\execution\SourceExecuteProcessor.java
com.google.common.collect.Lists  [20]
 ==>
Expected :0
Actual   :2
<Click to see difference>

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

Please resolve conflicts.


The lack of a closing tag

The lack of a closing tag

Good catch. This actually needs to get changed in src, though, not the rendered out markup in docs.
For what it's worth, since it's only in the docs, I plan on leaving this bug in place for a little while. I'm making some changes to how I implement SVGs that will render that whole section moot anyways.
Refs:

https://github.com/cferdinandi/kraken/issues/202
https://github.com/cferdinandi/kraken/issues/204
https://github.com/cferdinandi/kraken/issues/205


fix: print correct argocd url after cluster bringup

Also, use idpbuilder to fetch argo admin secret for initial login
Fixes #213

Approved !


Update docs pending deployment of new namespace subdomain

We will be making a change to Runtime to support better segregation of web actions. These docs accompany those changes. Deployment date TBD so please do not push to production just yet.

@MattLusher this PR looks good to me but I don't specifically work on runtime and it would make more sense for someone on Dan's team to give this change the 👍

@macdonst You mean Dan McWeeny? He submitted this. Would you recommend Nick Halbakken, then? Or Mihai Corlan?

This is all good! I talked with Dan. We just need to wait for the release and then we can publish this. thank you!

Thanks, guys! BTW, my contract terminates tomorrow, so I'll no longer be able to officially work on the docs. However, I don't mind keeping an eye on them, and my GitHub account will still have perms. Ping me whenever you like, either through GitHub or directly at<EMAIL_ADDRESS>

Thank you Matt for your support! It was a pleasure working with you. Wish you all the best.

@mcorlan - This can get merged now, thanks.

Done @mcdan ! Thank you!


[pick2.0]conf opt mysql network timeout to 600s #32545

Proposed changes
cherry-pick #32545
issue:
https://github.com/apache/doris/issues/28807
https://github.com/apache/doris/issues/32539
related pr:
https://github.com/apache/doris/pull/28808
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


loses focus when ng-model references ng-repeat array directly

I have a simple <input> inside a

As soon as I type a character the <input> focus is lost. To continue typing I first have to click inside the input area for each character I want to enter.
The essence of the ng-repeat code that shows the problem is:

    <input ng-model="items[$index]">

However, an <input> inside the ng-repeat scope but which does not directly reference the ng-repeat array, works fine - I can enter data all day long. For example:

    <input ng-model="george">

Note: the problem is clearly related to the the ng-model referring directly to the $index-th element in the items array. My guess is that a $watch on the collection is triggered and that causes the loss of focus.
Here's a codepen that shows the problem:
http://codepen.io/djmarcus/pen/bVZGvj
Here's a distillation:

This fails

            <input ng-model="items[$index]"  label="">

This works

            <input ng-model="george"  label="">

This works (its outside of ng-repeat)

        <input ng-model="harvey" label="">

I think there's a related issue floating around somewhere, but that was because the repeated elements were moved, which caused them to lose the focus. Since in this example, there should happen no DOM reordering, this needs an investigation.

I think this is expected behaviour.
As @Narretz mentioned, your are repeating over an array and you are changing the items of the array (note that your items are strings, which are primitives in JS and thus compared "by value"). Since new items are detected, old elements are removed from the DOM and new ones are created (which obviously don't get focus)).
There are several ways to achieve the desired behaviour by:

Use the local item variable created in ngRepeat's child scope:

  <input type="text" ng-model="item" />

Use track by $index (if it's appropriate for your usecase, i.e. you know the items won't be reordered):

  <input type="text" ng-model="items[$index]" />

Wrap your strings into objects. E.g. items = [{value: 'string1'}, {value: string2}, ...]:

  <input type="text" ng-model="items[$index].value" />

You can see all 3 solutions in action in this demo pen.

Thanks @gkalpak. You are right, the behavior is expected.

I ended up using option #3. Thanks for clarifying the issue (in hindsight its obvious).
It would help if there was some documentation on Angular 'gotcha' issues.. this one is a prime candidate for that list.

TBH, I would expect you'd follow option #1. It's quite typical to use it like this - I wonder why can't you use it (if there's a specific reason).

Option #1 (as coded in the codepen) is what I was using originally. It does not work (at least when I try it in the codepen) for the reasons you gave above... perhaps you meant something different in the codepen?
Nevertheless, option #3 is most closely related to my code. The array that I ng-repeat on actually is an array of objects (not simple strings) where my interest is in one of the properties.. so option #3 is a good match.

[Option #1] does not work (at least when I try it in the codepen)

Ooops, you are right. Option #1 is the not suitable for 2-way binding via ngModel.
(I updated my previous comment to avoid confusion for future readers.)

approach #2 by @gkalpak fixed the problem. I needed this in Angular 2 though. Thanks for the solution.

If you stumbled upon this thread and are looking for a solution, the solution is quite simple.
Instead of two way binding with the array, only bind the property and handle the change event using 'change' event instead of 'input'
Example:
// Instead of writing

    <input type="text" placeholder="0" [(ngModel)]="params.value[id]">
</div

// Write

    <input type="text" placeholder="0" [ngModel]="params.value[id]" (change)="onValueUpdate($event, id)">

and update the value in onValueUpdate function

thanks @gkalpak
it worked for me.  I ended up with option3. it worked for angular 4 also


API request requirements not clear

When I execute the https://graph.microsoft.com/v1.0/applications/app-id call, I get the following error:
{
"error": {
"code": "Authorization_RequestDenied",
"message": "Insufficient privileges to complete the operation.",...
After re-reading the page, I cannot find what privilege I need to activate to be able to do this request.

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

ID: df11cb03-70da-7df3-4941-3c2f8ea022f8
Version Independent ID: 96cd8eec-ed94-05bf-1288-86b6ef3db105
Content: Troubleshoot publisher verification - Microsoft identity platform
Content Source: articles/active-directory/develop/troubleshoot-publisher-verification.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@eurog33k  Thanks for reaching out. We will investigate and update the thread accordingly.

@eurog33k , Apologies for delayed response.
Could you check, if your account has consent with Application.ReadWrite.All API permission as mentioned in this document. In case If you are using Graph explorer, you can Consent from Modify permission option as shown below. Hope this helps.

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.
#please-close


sensu-install update

Hi, just installed my plugins with sensu-install instead of gem install and now i am wondering how do i keep my plugins updated. sensu-install doesen't seem to have an update option.

Hi @stephan13360, you can update sensu plugin gems with sensu-install, by specifying a newer or the latest release, e.g. sensu-install -v -p mysql:0.0.7
Documentation can be found at https://sensuapp.org/docs/latest/plugins#sensu-install-example
We highly recommend installing specific versions at all times.

Please close this issue if this helps :+1:


Address buildkite memory issues

Purpose
Previously, we saw memory issues when running our MPI AMIP FINE test on buildkite (see #192). These issues have been resolved by memory usage changes addressed in ClimaCore PR #1113. Here, we remove the memory specifications we had added to work around this problem and replace them with the same memory limits enforced by ClimaAtmos. In general, we should ensure that our buildkite driver is similar to that of ClimaAtmos.
Closes #192

[x] I have read and checked the items on the review checklist.

bors r+


[FLINK-6341]Don't let JM fall into infinite loop

When TaskManager register to JobManager, JM will send a "NotifyResourceStarted" message to kick off Resource Manager, then trigger a reconnection to resource manager through sending a "TriggerRegistrationAtJobManager".
When the ref of resource manager in JobManager is not None and the reconnection is to same resource manager, JobManager will go to a infinite message sending loop which will always sending himself a "ReconnectResourceManager" every 2 seconds.
We have already observed that phonomenon. More details, check how JobManager handles ReconnectResourceManager.

Thanks for spotting the problem @WangTaoTheTonic. This is important to fix.
I think we cannot solve the problem as you've done it, because you access actor state from outside of the actor's main thread. I think it would be better to introduce a resource manager connection id which we can use to distinguish whether we have to reconnect from a resource manager or whether it is an outdated ReconnectResourceManager message. We can do the check then in the handler for the ReconnectResourceManager message.

@tillrohrmann What problem it will bring if we access currentResourceManager from another thread? It is a variable in JobManager and can be shared across multi threads, right? The new added code just read it and there's no cocurrency problem coming, i think.

It is a problem because, the currentResourceManager is not volatile. Thus, you might miss some state changes in the future callback because of a cached value. Moreover, accessing mutable actor state from a different thread breaks the actor encapsulation and can lead to subtle race conditions and other synchronization bugs.

I see. Connection ID is added, please check if it's ok :)

Thanks for your contribution @WangTaoTheTonic. Changes look good to me. Merging your PR.


Loot Tracker - Gem Bags

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 11
Bug description
Loot Tracker plugin doesn't track the Gem Bags purchased from the mining guild with stardust.
Screenshots or videos
No response
RuneLite version
1.10.34
Logs
No response

Restart your client, the version of RuneLite that came out 9 days ago supports Gem Bags.


Why does notebook.cellToolbarVisibility setting does not have hide option?

Testing #125013
cell tool bar visibility can be configured using notebook.cellToolbarLocation setting but there is another setting notebook.cellToolbarVisibility that can be confused with visibility of cell toolbar

Should we move "hidden" to the "visibility" setting? cc @rebornix @misolori

Ah i do think the name here is confusing, this is essentially what controls HOW the toolbar appears, so maybe another name would be better? notebook.cellToolbarInteraction? I also don't know whether we shouldn't broadcast this settings and just go with a default and remove the setting, so far we haven't seen any complaints since the default was changed.

I think the jupyter team might not like it, since they don't want to have to click twice to get the run above/below actions

and our run button is always visible on hover (actually that difference feels weird too)

The output action is always visible, no matter if the cell has focus or not, or if it has hover. It's weird that the cell toolbar, run button and output button have different interaction behavior.


Reduce v2int8 bundle size

Reduce bundle size by ~100kb.
After an investigation into our bundle size telemetry, it was determined that these two commits in main had reduced the bundle size by about ~100kb.
These changes occurred only after 2.0.0-internal.8.0.0 was released.

Please add links to the PRs that are being ported to the PR description.


Add donation links (Bitcoin & PayPal) to README.md

Thanks for the comments @Irio @fgrehm — all fixed in d8b97e1


Add --build option for bundle package

[x] Add failing spec
[x] Add baseline implementation
[x] Pass spec
[ ] Code review

@RochesterinNYC
Closes #4635

@homu delegate=RochesterinNYC

:v: @RochesterinNYC can now approve this pull request

Closing this since this feature won't be used. See #4635


Donkey Kong Country: Tropical Freeze [ARK#01]

Improper Water Rendering:
1080p:

720p (native):

360p (half):

Fixed as of 2e540dd

Latest shader edits completely get rid of the water effect. Images from Xalphenos on Discord.
Before:

After:

https://github.com/slashiee/cemu_graphic_packs/commit/175568dd28a945fb6a7da6b094f191e46f23f4bb Fixes:

Half transparent water
In game and boss cut scenes
Water during night/Dark location

Hopefully this is the last shader fix needed, I cheated around most of the game. Storm location is broken due to emulation/texture corruption. If nothing else pops up for a couple of days, i suggest closing this report.

Hey uh, @getdls is there anyway to contact you outside of GitHub? Maybe on Discord, perhaps?

Drop me a pm on reddit /getDls?
I can sign up for Discord, but I'd rather avoid another time sink, esp if it's a fun one.

https://github.com/slashiee/cemu_graphic_packs/commit/518b74c0f9ac3c2c4e2b5e496f3d05ba3ccb901a Fixes:
Edge based AA that caused artifacts when up-resing.. been buggnin' me since I noticed it yesterday. Edges should now be clean everywhere.

The design team should get a medal, my next vacation will be in the Seychelles.

I've tried to used both AA removal and the 2160p pack, this is the result:
http://imgur.com/a/OTGXs
Any idea why? I've tried different version of Cemu, always the same problem.

Thnx, checking. Reverted commit meanwhile

It happens too with a fresh cemu install. As soon as i use the graphic pack there's the bug.

All shader issues taken care of as of 1.7.6, closing issue when 1.7.6 releases publicly.

257984d  Commit, fixes AA for 1.8 release, it replaces 518b74c.   Slashie if there's a separate 1.8 fork, feel free to move it as you see fit.
All other water rendering is now works without separate shader edits, so this issue report can be closed.

I will move those commits to a 1.8.0 branch. @getdls I'm actually gonna go against what I said earlier and just close this


SpanName always be default-route instead of Api Path in zipkin in Istio0.7

Hello, I use Istio 7.0 and install zipkin by zipkin.yaml in istio-0.7.1\install\kubernetes\addons. I use spring-cloud-sleuth in my springboot application and request A service which linked to another service.And here is my result.

as you can see, All spanName are default-route instead of API Path
Is it normal or correct? How can I let these spanName be the real API Path.
For now , the only way for me to know its API Path is to find http.url in detail.

Yes this is the correct behaviour currently.
The problem with using the path as an operation/span name is that it could have path parameters - see https://github.com/opentracing/specification/blob/master/specification.md#start-a-new-span for a discussion on selecting an appropriate operation (span) name.
So the problem is how to identify a meaningful name from the data available to the proxy. The route, although not ideal, can provide useful information when multiple routes are defined.
More useful would be to derive an operation (span) name from some form of API definition - unfortunately this is not available at the moment but possibly in the future.

There was a recent PR that effectively set the default operation to the service host name as the default operation (along with any path/prefix configured by the virtual service).

@rshriram do you know the current situation with using openapi (or some subset) to define the service API, so that we could leverage it to define simple operation names?

Found this https://github.com/istio/api/blob/master/mixer/v1/config/client/api_spec.proto discussed in https://github.com/istio/api/issues/344 - but would need to investigate further and see what is possible.

We are facing the same issue with Istio 0.6 version.
From local, we can see the operation-name in Jaeger but when we hit Istio-ingress. it shows default-route.
It is difficult to find traces with default-route as it's coming for every api call.

Desired: "Istio-Ingress --> Service A --> MongDB query"

Received: Istio-Ingress --> default-route

cc @ayj as he did something recently with HTTPAPISpec.

api_spec.proto allows you to define method/uri_template-to-attribute mappings in the proxy for API attribute generation. The same mappings information could possibly be used by the mixerclient to augment trace span data. cc @qiwzhang @ostromart


trainval.txt

Where do you get the training.txt file?

I save the name of image files without suffix in the *.txt file for easy access. But you may change the access in dataset_dota.py and read data in your way.

Is it right to add the contents of the training.txt file like this?
Can you show me the sample?

Right.
P0000__0.5__0___0
P0000__0.5__0___1000
P0000__0.5__0___1500
P0000__0.5__0___2000
P0000__0.5__0___2151
P0000__0.5__0___500
P0000__0.5__1000___0
...

"container-crane." What's the error?

Hi, can you tell me more about how to create the file trainval.txt? Thanks!

您好，读了您的文章感觉受益匪浅，我这边想向您确认下，你的label格式是怎样的呢？是class t,r,b,l, w, ℎ，a的形式么？

您好，读了您的文章感觉受益匪浅，我这边想向您确认下，你的label格式是怎样的呢？是class t,r,b,l, w, ℎ，a的形式么？
The label is from DOTA or HRSC. Generally, it should be point coordinates.