parameterise dynatrace user

Thanks @soniah! This bug has already been fixed in commits:

Update template location: https://github.com/Dynatrace/Dynatrace-Collector-Ansible/commit/36c6b24e78addbde8ae5225c61a08ddbcd598ec5
Integrate latest init.d script: https://github.com/Dynatrace/Dynatrace-Collector-Ansible/commit/c590a06c390fae8e87f39510da9d41f993d113e1
Fix mistyped ansible_distribution == 'RedHat'. Resort to 'ansible_os_family': https://github.com/Dynatrace/Dynatrace-Collector-Ansible/commit/ace6660d28a01b1da95c2db56f2ab664bd0ed6d4
Add ability to specify owner and group of a Dynatrace installation: https://github.com/Dynatrace/Dynatrace-Collector-Ansible/commit/c415968052096464a545348e1a3800407f6ab66a


[feature] Exposing more info through mapping.json

There is still more info that could be exposed through mapping.json like field types, access flags and most importantly the corresponding line on the decompiled code.

@yotamN mapping.json supposed to be just index for quick search of needed class/method and next go to class specific json file for additional info like access flags and code. I don't want to add much data into mapping.json because it becomes very big for large apk files.

field types
Should be in class json file, but I just notice I forget to fill that field. Sorry 😞


Emitter#scalar always adds "non-specific" tag

In the following example, the "!" character is placed in front of the mapping value.
stream = StringIO.new
emitter = Psych::Emitter.new(stream)
emitter.start_stream(Psych::Parser::UTF8)
emitter.start_mapping(nil, nil, true, Psych::Nodes::Sequence::BLOCK)
emitter.scalar("foo", nil, nil, true, false, Psych::Nodes::Scalar::ANY)
emitter.scalar("bar", nil, nil, true, true, Psych::Nodes::Scalar::DOUBLE_QUOTED)
emitter.end_mapping
emitter.end_document(true)
emitter.end_stream
stream.flush
stream.string  # => foo: ! "bar"

Normally this isn't a problem, but let's say I wanted to output a stringified number 7, eg:
emitter.scalar("foo", nil, nil, true, false, Psych::Nodes::Scalar::ANY)
emitter.scalar("7", nil, nil, true, true, Psych::Nodes::Scalar::DOUBLE_QUOTED)

This results in foo: ! "7". So far so good. Unfortunately parsing this with YAML.load returns { "foo" => 7} - the literal "7" character has been converted to an integer. This can also happen with a literal "no" string, which gets converted to false on parse.
No matter what combination of options I pass to Emitter#scalar, the exclamation point is still added. Surely there must be a way to avoid emitting it, since YAML.dump behaves correctly. What am I missing here?

Ok, I was able to figure this out. The verbiage used in the YAML spec is "non-specific" while the verbiage used in Psych and her documentation is "implicit". Apparently these are equivalent terms (this patch is what clued me in).
In order to avoid the additional exclamation points, you have to start the document in non-implicit mode:
emitter.start_document([], [], false)

The third argument is the key here. true indicates implicit (non-specific tag) mode, while false means the opposite.


Port SCP's hashes into 64-bit

This is the first part of resolving https://github.com/bpfkorea/agora/issues/702

StellarHashFmt needs to be removed in SCPPrettyPrinter.

StellarHashFmt needs to be removed in SCPPrettyPrinter.

Done.

Ah I think the tests will fail now, the hashes probably changed..

Hm nope, I was wrong. :)

Fixed up, green.


enhancement-on-search page

This PR improves the search bar style and alignment of sont list and changes the twitter logo .
Fixes #50

@youtanimstar Please review the pull request . If everything  is good then merge it .

@youtanimstar Please review this PR .   I want to change your song list in a grid in the form of cards with details if you will give permission  then I can do it.


fix: reducer clusterInfo should not be used

Reducer clusterInfo is redundant and should be removed.

Yes, now it is needed in internal version.


The XML program is very slow to generate

It takes around 2.3 seconds to generate the XML of the program. That is too slow for a good user experience.
Add some cache to the URL https://participez-2016.capitoledulibre.org/planning/program/public/?format=xml and/or improve its speed.

Cachind added (with a timeout of 3h for now).

Good, it's working.


[web] Allow the usage of url strategies without conditional imports

Today, users have to do conditional imports in their apps in order to customize URL strategy. @johnpryan suggested that we make it possible to use the package without having to do conditional imports. This PR is an attempt to achieve that.
Thanks @ditman for the help!

(PR triage): Are there still plans for this PR?

@goderbauer yes, I'm planning to come back to this PR, address the review and land it.

   info • Unnecessary type check; the result is always 'true' • packages/flutter_web_plugins/lib/src/navigation/url_strategy.dart:75:12 • unnecessary_type_check


fix(chat): disallow auto-select uploaded files if can't attach files (Issue #1665)

Description:
disallow auto-select uploaded files if can't attach files
Issues:

Issue #1665

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review


[kubernetes] catch exception from kubelet init

Note: Please remember to review the Datadog Contribution Guidelines
if you have not yet done so.
What does this PR do?
Kubeutil raises an exception if it can't reach the kubelet. We should catch it.
Motivation
Ugly collector logs.
Testing Guidelines
Run the agent on a non-k8s node, see that the exception is now caught.
Additional notes
Sister PR: https://github.com/DataDog/dd-agent/pull/3310

thanks @masci, updated.


Error: Class 'App\View\Helper\ComponentCollection' not found

Hi ,
Am using cakephp3.2, trying to integrate with acllinkhelper but am getting below error,
Error: Class 'App\View\Helper\ComponentCollection' not found
Please help
Regards

hi,
please use this version https://github.com/shahonseven/CakePHP-AclLinkHelper/tree/3.x


Added a mechanism and cops to check for undesirable migration changes

This PR resolves #26

@joshpencheon this should be mergeable now.

Cool, I didn't know start_with? could take multiple candidates:
'hello'.start_with?('hello', 'world') #=> true


List of Callbacks Raises ValueError("Expected a parent")

Bug description
If the callbacks list, passing to the trainer, has more than one element, it raises the error ValueError("Expected a parent").
If you pass any of those elements alone the code runs fine.
p.s. I'm already importing like this: import lightning.pytorch as pl which is version >=2. specific.
What version are you seeing the problem on?
2.0+
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

ValueError: Expected a parent
Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
cc @awaelchli

Can you share your code?
I've seen this happen when lightning.pytorch and pytorch_lightning imports are mixed together. You need to use only one of them. This includes the callback parent
import lightning.pytorch as lp
import pytorch_lightning as pl

class MyCallback(pl.Callback):
    ...

trainer = lp.Trainer(callbacks=MyCallback)  # bad!

@carmocca I had this bug and your suggestion fixed it

@carmocca Do you think we could inspect sys.modules and detect whether there are both pytorch_lightning and lightning loaded, and warn the user? Not sure whether that would help.

@awaelchli I mean, better warnings are always great.

https://github.com/Lightning-AI/lightning/issues/17485#issuecomment-1524198677

Can you share your code?
I've seen this happen when lightning.pytorch and pytorch_lightning imports are mixed together. You need to use only one of them. This includes the callback parent
import lightning.pytorch as lp
import pytorch_lightning as pl

class MyCallback(pl.Callback):
    ...

trainer = lp.Trainer(callbacks=MyCallback())  # bad!

this also solved a similar bug for learning rate monitor callbacks

@carmocca Do you think we could inspect sys.modules and detect whether there are both pytorch_lightning and lightning loaded, and warn the user? Not sure whether that would help.

I feel like that's too strong of a check. Imagine you are in a team of data scientist who share utility python packages and two packages use lightning, each with a type of import. This check would break their code just on import even if they don't mix the classes. Fixing it might also be out of the control of the person who just tried to import it.
I think it would be simpler that we add a specific check for this in is_overridden and _maybe_unwrap_optimized

Can you share your code?
I've seen this happen when lightning.pytorch and pytorch_lightning imports are mixed together. You need to use only one of them. This includes the callback parent
import lightning.pytorch as lp
import pytorch_lightning as pl

class MyCallback(pl.Callback):
    ...

trainer = lp.Trainer(callbacks=MyCallback())  # bad!

That solved my error. Thanks!

I'm seeing this error with the Optuna PyTorchLightningPruningCallback. They extend pytorch_lightning.callbacks.callback.Callback and I can't control that, but I'm using the new lightning package and don't want to change that.
My rather hacky workaround is this:
import lightning
import optuna

class PatchedCallback(
    optuna.integration.PyTorchLightningPruningCallback, lightning.Callback
):
    pass

I think since there's going to be two lightnings floating around for years, instance checks should be updated to be if isinstance(obj, (old_pl.Callback, new_pl.Callback)), before eventually being reverted after a suitable deprecation period.
Also, will the fix https://github.com/Lightning-AI/lightning/pull/17548 just mean that I get an error if I use a package that is still on old Lightning? Or is it assuming direct use of the class and it doesn't handle the inheritance case?

could you please fix the error msg for this? The error message is useless. in my case, it raised TypeError: __init__() got an unexpected keyword argument ‘gpus’ after the ValueError("Expected a parent"), but I didn't even pass gpus args. And it took so long to find this thread.


Modified kcp version to release-0.5

Changes

Modified KCP image to release-0.5
Modified  KCP Syncer image to release-0.5
Modified ./ckcp/test.sh to look for namespace with proper label

Known Issue
When executing ./ckcp.test.sh pipelines, the PipelineRun, such like test-pipelinerun-xxxx will fail due to the SA token secret being renamed from default-token-xxxx in icp to  kcp-default-token-xxx on the workload cluster.  It couldn't find the secret.
Testing Locally

Download PipelineRun manifest using_context_variables.yaml
Remove serviceAccountName property
Seed the modified PipelineRun manifest into KCP cluster
$ KUBECONFIG=<kcp_kubeconfig> kubectl create -f using_context_variables.yaml

Hi @xinredhat. Thanks for looking at that.
There are a few more places that would need to be changed:
https://github.com/openshift-pipelines/pipelines-service/blob/main/images/kcp-registrar/register.sh#L127
https://github.com/openshift-pipelines/pipelines-service/blob/main/images/kcp-registrar/Dockerfile#L8
https://github.com/openshift-pipelines/pipelines-service/blob/main/local/kcp/start.sh#L43
Also could you please modify test.sh so that it deploy TaskRuns PipelineRuns that either don't have a serviceAccountName set or use pipeline SA.
As part of this PR I would like this to get documented, possibly in a new document "phase 1 limitations". There we should stipulate:

not specifying an SA is OK
pipeline SA can be used
default SA does not work and would point to kcp API
custom SA created in kcp workspace may have the same flaw for our phase 1 approach
Describing the behavior and the challenge:
The default SA in kcp workspace has the secret containing its token synchronised in the workload cluster but under a different name (with kcp prefix). In phase 1 we don't want Tekton pods to communicate with the kcp API. As such they should not use an SA and its token created by kcp. This is a limitation but existing SA from the workload cluster can be used instead.

@fgiloux sure will do. thanks for the clarification.

Hi @xinredhat. Thanks for looking at that. There are a few more places that would need to be changed: https://github.com/openshift-pipelines/pipelines-service/blob/main/images/kcp-registrar/register.sh#L127 https://github.com/openshift-pipelines/pipelines-service/blob/main/images/kcp-registrar/Dockerfile#L8 https://github.com/openshift-pipelines/pipelines-service/blob/main/local/kcp/start.sh#L43

Fixed

Also could you please modify test.sh so that it deploy TaskRuns PipelineRuns that either don't have a serviceAccountName set or use pipeline SA.

Fixed by removing serviceAccountName set

https://github.com/openshift-pipelines/pipelines-service/blob/main/ckcp/openshift_dev_setup.sh - cert-manager plugin
Could we add some meaningful error message either here or as a pre-requisite check before starting the rest of the script to check that all the pre-requisites are satisfied.
@ramessesii2 has created #104 for the same.

Fixed

@xinredhat It may be a good idea to jump on a quick call with Bharath and Frederic if there is just a few comments that are blocking the merge of the PR.


Use of existing Python-JavaScript tools such as Brython?

This looks like a great project. Looking forward to using it. Just one question. Have you considered using something like [Brython] (http://www.brython.info/) for the JavaScript handling instead of writing your own implementation? Having a Python3-only solutions seems fine to me. Do you want to avoid too many dependency? Or does Brython not meet your needs?

Hi Mike,
Flexx is designed to be a Python library, that really only uses JS as a "runtime" to draw its UI. The PyScript (Python code that gets converted to JS) that you can use in Flexx is really just JavaScript with a Python syntax (similar to Coffeescript and RapidScript).
We could use Brython on the JS side to allow "real Python code". But in addition to being a dependencie, Brython is quite heavy and relatively slow, because it tries so hard to be real Python (e.g. with exceptions behaving in the same way).
Further, I would like the PyScript code to be able to interface with other JS libs. When using a Python "runtime" such as Brython, you also lose that interaction (or at least make harder/awkward).
For the record: The PyScipt converter is based on/inspired by existing code from another project, which helped a lot.

Hi Almar,
Thanks for your answer. Obviously, you did a sound research for options and alternatives. Sometimes rolling your own is better than re-using existing libraries. This project seems to fall into this category. Looking forward to your talk in Cambridge.

PyScript is now a bit more Pythonic. Enough to run PyStone. Here are some results: https://github.com/zoofIO/flexx/blob/master/examples/app/benchmark.py#L10-L17  This shows that the approach of PyScript has significant advantages compared to Brython et al. in terms of performance (and its JS should also be much smaller).


changed $::operatingsystemmajrelease to $::lsbmajdistrelease for Debian

The fact operatingsystemmajrelease does not exists in facter < 1.7 wich is installed by default on debian wheezy. That comes along with puppet 2.7.23 wich should be supported. The resolution is to use lsbmajdistrelease fact.
I only adopted the code for debian, just to here what you think. If you agree, I would also update the code for redhat etc.

Yeah, I'm down with this. If you wanna PR for RH too, that would be most appreciated.

@jfryman do you want to revert this? See discussion at https://github.com/jfryman/puppet-nginx/pull/562


Refactor covalent radii, bonding rules, and overlap detection

refactored covalent radii
refactored bonding rules
added better overlap checking function
updated .travis.yml for Julia 1.6.0

Covalent radii are now easier to work with.  Augmented Cordero values w/ Manz data.  get_covalent_radii() returns Dict w/ all radii in Angstroms (no more standard deviations); get_covalent_radii(sym) returns radius of single element.
Bonding rules are easier to work with.  Removed minimum distance (was never used for anything).  Bonding rule set can be calculated from default covalent radii w/ bondingrules() or from customized covalent radii dictionary w/ bondingrules(covalent_radii=dictionary)
Overlap detection on a crystal applies simple rule from covalent radii.  overlap(crystal, apply_pbc) compares atoms' pairwise distances to the lesser of the pairs' covalent radii.  Could do better still, but it's an improvement over applying a user-supplied tolerance to every pair of coordinates.
Closes #38
Partially addresses #42 but should be augmented further

Codecov Report

Merging #65 (e3f17e9) into master (b7820f0) will decrease coverage by 3.80%.
The diff coverage is 73.80%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   92.67%   88.86%   -3.81%
==========================================
  Files          11       11
  Lines         887      952      +65
==========================================
+ Hits          822      846      +24
- Misses         65      106      +41

Impacted Files
Coverage Δ

src/covalent_radii.jl
33.33% <33.33%> (ø)

src/bonds.jl
93.49% <83.33%> (-0.78%)
:arrow_down:

src/Xtals.jl
100.00% <100.00%> (ø)

src/distance.jl
93.33% <100.00%> (+2.26%)
:arrow_up:

src/cpk_colors.jl
9.52% <0.00%> (-90.48%)
:arrow_down:

src/atomic_masses.jl
14.28% <0.00%> (-85.72%)
:arrow_down:

src/repfactors.jl
100.00% <0.00%> (ø)

src/misc.jl
98.43% <0.00%> (+0.10%)
:arrow_up:

... and 5 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 b7820f0...e3f17e9. Read the comment docs.

much improved bonding rules!
comments

if you can do function overloading, this is clear than making a single function that takes in multiple types (a Union). faster too maybe? don't know, but I find it more clear.
defined some intermediate vars to make clearer what the function was doing, took me a min to understand

questions

I think the overlap doc string is now not pertinent to the Crystal input--- since it doesn't use tol, but rather the bonding rules.
overlap computes the distance between two atoms. then it asks if the distance is less than the minimum of the two radii of the atoms involved. but shouldn't it be considered overlap if r < cr_i + cr_j? think of two atoms as spheres right against each other. I suspect this will give a lot of errors if you don't build in a tolerance, but I could be wrong. see our PR on the overlap for inspiration, don't want to give up on that PR, but I feel like this PR is making that one obsolete since that's what we were trying to do there anyway-- use the bonding rules.

* if you can do function overloading, this is clear than making a single function that takes in multiple types (a `Union`). faster too maybe? don't know, but I find it more clear.

Yes, more clear.  Also should be faster, since the individual functions are now branch-less (if is slow)

* defined some intermediate vars to make clearer what the function was doing, took me a min to understand

Also more clear, but possibly slower, due to garbage collection on the intermediate variables

* I think the `overlap` doc string is now not pertinent to the `Crystal` input--- since it doesn't use `tol`, but rather the bonding rules.

There are two signatures for overlap now: one for Coords and one for Crystal.  I wasn't sure about removing the ability to set an absolute tolerance, so I left the original one--meant to ask about this.  I can get rid of the old one, or we can have both options; either way, I will fix up the docstring.

* `overlap` computes the distance between two atoms. then it asks if the distance is less than the minimum of the two radii of the atoms involved. but shouldn't it be considered overlap if `r < cr_i + cr_j`?

r == cr_i + cr_j is the criterion for the longest bond between i and j.  r < cr_i + cr_j doesn't always mean i and j are too close, just that the bond length is less than maximum.  The centers of i and j being within each other's covalent radii, however, will tend to mean they are genuinely too close.  For example, we give carbon a covalent radius of 0.88 and hydrogen 0.38, meaning we will infer a bond if C and H are separated by no more than 1.26 Å, which is actually fairly permissive.  The average C-H bond length is around 1.1 Å.  No C-H bond will ever be as short as 0.38 Å, so that was a safe choice for asserting an overlap.  It would probably be safe to say that 0.88 Å is too short, so maybe the overlap threshold should be the max of the two radii.

due to garbage collection on the intermediate variables
not sure how smart the compiler is. might be faster b/c now u aren't slicing an array and constructing another array at an intermediate step. hard to know since the compiler is so opaque to me! probably small difference anyway.

I wasn't sure about removing the ability to set an absolute tolerance, so I left the original one
agree it's good to keep original just in case.
I think u should modify the overlap function to handle atoms named as e.g. C23, like we did here:
https://github.com/SimonEnsemble/Xtals.jl/pull/62/files
then, every time a crystal is constructed, you call this overlap function without worrying about C23 breaking it.

then in the check_overlap function we can call your new function using the radii! I think this will be a more rigorous overlap test. I think users should be aware when there is overlap in their crystals, since then the fidelity of the sim is in question.
https://github.com/SimonEnsemble/Xtals.jl/blob/master/src/crystal.jl#L453
this will make our old PR here obsolete, I think. and the reason we pursued that is for a more rigorous check of overlap when the xtal is built.
I SEE, thanks for explaining. I guess checking for overlap is not an exact science, and I'm not sure there is a rigorous number we can use to check for it. < min(r_i, r_j) seems it will have a lot of false okay's to me, but I don't know of a better solution, and this is ten times better than not accounting for the van der waals radii! so I'm super happy with this rule. and it's better than tripping false not okay's all the time, since a slightly too-short bond will not impact adsorption simulations too much.

@SimonEnsemble changes made, ready to merge

awesome. just one more thing:

I think this doesn't properly check overlap when you add two crystals? I suspect you want to assert that the overlap flag is false. https://github.com/SimonEnsemble/Xtals.jl/blob/refactor_covalent_radii/src/crystal.jl#L1130-L1132

assert that the overlap flag is false

Added non-overlap assertion to default +(::Crystal,::Crystal) and updated unit test
Now it's ready to merge :grin:


Anyone tried using this with Traefik?

First off - this is a great, simple, very slick webfile manager - thank you!
Trying to get this to work with Traefik.  I tried the traefik labels below which should have made it available via https://webfiles.xxx.biodati.com. I can access it fine via http://xxx.biodati.com:8989 (IP locked - I can set it up on an open server if needed).  I'm getting a gateway timeout - though I do see the front and backend's showing up in the Traefik dashboard for droppy.  I also see an entry in the access.log for traefik, but I don't see anything in the droppy docker logs.  I'm not sure if this is a Traefik issue or droppy, but I have several other services that work great using the same setup.
# Web file manager
droppy:
  container_name: droppy
  image: silverwind/droppy
  ports:
    - 8989:8989
  volumes:
    - ./webfiles/config:/config
    - ./webfiles/files:/files
  # labels:
  #   - traefik.enable=true
  #   - traefik.backend=droppy
  #   - traefik.frontend.rule=Host:webfiles.${BS_HOST_NAME:?err}
  #   - traefik.port=8989
  restart: unless-stopped

Never tried it, but I don't see why it should not work, it's a standard HTTP 1.1 service. Just make sure that websocket traffic is forwarded correctly, as this is often an issue with reverse proxies (Thought this does not explain your current issue).


mermaid Graph Overlap

Describe the bug
mermaid Graph Overlap
Steps to reproduce
add :
流程图
flowchart LR
    Start1 --> Stop1
    start2>This is the text in the box] --> end2{This is the text in the box} --> end3(((This is the text in the circle)))

时序图
sequenceDiagram
    Alice->>John: Hello John, how are you?
    John-->>Alice: Great!
    Alice-)John: See you later!

类图
---
title: Bank example
---
classDiagram
    class BankAccount
    BankAccount : +String owner
    BankAccount : +Bigdecimal balance
    BankAccount : +deposit(amount)
    BankAccount : +withdrawal(amount)

状态图
stateDiagram-v2
    state if_state <<choice>>
    [*] --> IsPositive
    IsPositive --> if_state
    if_state --> False: if n < 0
    if_state --> True : if n >= 0

pie title Pets adopted by volunteers
    "Dogs" : 386
    "Cats" : 85
    "Rats" : 15

思维导图
mindmap
  root((mindmap))
    Origins
      Long history
      ::icon(fa fa-book)
      Popularisation
        British popular psychology author Tony Buzan
    Research
      On effectiveness<br/>and features
      On Automatic creation
        Uses
            Creative techniques
            Strategic planning
            Argument mapping
    Tools
      Pen and paper
      Mermaid

The version of Memos you're using.
v0.22.5
Screenshots or additional context

Not able to replicate in the latest version. The graphs are spaced under each other


@GulpWeekly twitter feed to ignore blacklisted plugins

I'm confident this isn't the correct place for this, but as @GulpWeekly twitter feed has no author or codebase links, I'll put it here.  Get it to not tweet updates to plugins that are blacklisted.

Not a gulp issue.


not found via adobe

the link https://creative.adobe.com/addons/products/12557 leads to a message "product nor found"
and the manual way of installation does not work for me. nay help i appreciated, thx!

I had the same problem

did u find any solution?

The alternative installation instructions are in the github readme.

On Jul 29, 2016, at 11:53 AM, yesmetoo<EMAIL_ADDRESS>wrote:
did u find any solution?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/bodymovin/bodymovin/issues/124#issuecomment-236151996, or mute the thread https://github.com/notifications/unsubscribe-auth/AHBgA1hTIOiapIWmPr1O61VqFQHTtl-hks5qadu4gaJpZM4JYFV3.

I'm trying to solve this issue with Adobe.
Will let you know when it's up again.

The plugin is up again in the adobe store
https://creative.adobe.com/addons/products/12557


is there any same project written in java？

There are many written in java but majority of them are for android. You can check them out on Github by filtering the search results.

Sorry, I'm not really familiar with what similar tools might exist for java.

@ExtremeYu  I'm trying to wrap up face_recognition project with Jave project, and it can work on one docker image which running two project at same time. I will try to wrap up the whole face_recognition as a RESTful API service with Java. Can check this dockerfile https://github.com/AndrosEt/face_recognition/blob/master/Dockerfile, ps. this's just demo


removes getErrorAndDoneTopicPromises() from workflow & calls from mediator instead

moves getErrorAndDoneTopicPromises() out of module & calls them from mediator instead
part of https://issues.jboss.org/browse/RAINCATCH-629 which involved refactoring raincatcher-workorder and raincatcher-workflow to remove the getTopicPromises() and getErrorAndDoneTopicPromises() functionality in them, and move them into the mediator instead
related prs:
https://github.com/feedhenry-raincatcher/raincatcher-workorder/pull/23
https://github.com/feedhenry-raincatcher/raincatcher-mediator/pull/23

@damienomurchu - thanks for contribution. We decided to use different api functions and document them as part of different PR. Going to close this PR, but feel free to reopen if needed.


fix: Transparent buttons in the header now have feedback in android

#3237

PR seems to add unnecessary commas and round braces which is not related to the change. @cyrus25 Are you sure this is ready to be merged?


minor fix in fix doc string

fix small mistake in docstring of lwrb_get_linear_block_write_address().
Came across this today when I wanted to use memset on lwrb data and it confused me for a second so I thought I'd fix it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


parser,ast: support alter table add partition partitions by hash syntax

support alter table  add partition partitions by hash syntax.
CREATE TABLE employees (
    id INT NOT NULL,
    fname VARCHAR(30),
    lname VARCHAR(30),
    hired DATE NOT NULL DEFAULT '1970-01-01',
    separated DATE NOT NULL DEFAULT '9999-12-31',
    job_code INT,
    store_id INT
)
PARTITION BY HASH(store_id)
PARTITIONS 4;

alter table employees add partition partitions 8;

support alter table coalesce partition by hash syntax.
CREATE TABLE clients (
    id INT,
    fname VARCHAR(30),
    lname VARCHAR(30),
    signed DATE
)
PARTITION BY HASH( MONTH(signed) )
PARTITIONS 12;
ALTER TABLE clients COALESCE PARTITION 4;

@tiancaiamao @kennytm PTAL

Will we support alter table coalesce partition by hash ?

@tiancaiamao Temporarily support syntax first.

@tiancaiamao @kennytm PTAL

@tiancaiamao PTAL

Is there a link reference to the TiDB PR?
We should make sure the new change doesn't break TiDB at least. @ciscoxll

LGTM


Composing stylesheets counts as failure

By using the following syntax const style = {...commonStyle, ...specificStyle}. I can combine two stylesheets into one.
However, even though the styles are used through the style object, the styles from the specificStyle sheet count as unused.

@thijssteel thanks for reporting! I guess this is a good improvement to add


#33196 Option to disable integrated security in NTLM

Checked other authentication methods. Need to give an EA to the user to test NTLM on external domain


Help with creating a PowerShell module

I'm trying to install my PowerShell module via scoop but I can't get it to work properly.
https://github.com/krokofant/lovely-powershell
This is the config and the psmodule name matches with the module name.
{
    "version": "1.0",
    "url": "https://github.com/krokofant/lovely-powershell/archive/master.zip",
    "psmodule": {
        "name": "LovelyPowershell"
    }
}

When installed it is extracted into directory scoop/modules which is in the PowerShell module path.
This is the file tree:
└───LovelyPowershell
    │   install.json
    │   manifest.json
    │
    └───lovely-powershell-master
            .gitignore
            Installer.bat
            Installer.ps1
            lovely-powershell.json
            LovelyPowershell.psd1
            LovelyPowershell.psm1
            README.md

Now, I'm not able to import the the module (Import-Module "LovelyPowershell") but if I rename the lovely-powershell-master folder to the version 1.0 then the module is able to load.
I've tried to make the foldername be 1.0 via "extract_dir": "1.0"  but the installer doesnt like that att all:
Installing 'lovely-powershell' (1.0) [64bit]
Loading master.zip from cache
WARN  Warning: No hash in manifest. SHA256 for 'master.zip' is:
    1e90a2d1e1ded901a10605395c2fd2554911463554f28564a554ced81228b9f4
Extracting master.zip ... DEBUG
-------------------------------------------------------------------------------
   ROBOCOPY     ::     Robust File Copy for Windows
-------------------------------------------------------------------------------

  Started : Monday, November 5, 2018 20:43:52
   Source : C:\Users\***\scoop\apps\lovely-powershell\1.0\_tmp\1.0\
     Dest : C:\Users\***\scoop\apps\lovely-powershell\1.0\

    Files : *.*

  Options : *.* /S /E /DCOPY:DA /COPY:DAT /MOVE /R:1000000 /W:30

------------------------------------------------------------------------------

2018/11/05 20:43:52 ERROR 2 (0x00000002) Accessing Source Directory C:\Users\***\scoop\apps\lovely-powershell\1.0\_tmp\1.0\
The system cannot find the file specified.

ERROR Could not find '1.0'! (error 16)

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop/issues/new?title=lovely-powershell%401.0%3a+extract_dir+error
True

Solved it by specifying the extract_dir as lovely-powershell-master.


Visual Studio 2017 crash when this plugin is enabled

Everytime I try to debug my Xamarin.Forms / Android project while this plugin is active, the whole Visual Studio 2017 crashes since the latest update of it. A fix would be great because I was using this plugin a lot...
Maybe this is related: My project has currently 11 languages which each over 10.000 entries. But I don't understand why the whole studio crashes when I try to debug even when the ResxManager window is not open.. the plugin needs just to be active.
Edit: I have this problem on all of my 3 Windows mashines since I updated the Visual Studio 2017 to the latest version. And it crashes already in the compiling process.

Same here. Windows 10 Pro x64 with Visual Studio 2017 15.6.1.
Compiling a c# solution with 80 projects, will make VS hang. Waiting doesnt help either (waited 30mins++)

Duplicate of #97


[BUG]: [ERROR] Code 69 (UpdateApp): Please update your Android app.

Program version
4.7.1
Operating System
Windows
Type
GUI
Service
Hidive
Command used
NIL
Show ID
NIL
Episode
NIL
Console Output
The error message appears right after selecting HiDive in the Browser GUI.

GUI server started on port 3000
[WS] Connection to public ws from '::1'
[WS] Connection from '::1'
[ERROR] Code 69 (UpdateApp): Please update your Android app.

Additional Information
No response

This will be fixed in the next version, since the old hidive API will be removed

Ok. Thank you. I'll leave this open until implementation is completed so it won't reappear.

This should be fixed in v5.0.0 which was just released earlier, feel free to re-open if this is still an issue


Reloading not working when Grape API mounted as sub project

Hi,
i had tried to mount a grape api project successfully, however i am unable to get the reload working when i change a file in the grape api project.
i tried to find some documentation on this but to no avail.
is this expected or i missed out something?

How and from where (i.e,. where are the files) are you mounting your grape
api?
On Wed, Mar 25, 2015 at 8:54 AM roychoo<EMAIL_ADDRESS>wrote:

Hi,
i had tried to mount a grape api project successfully, however i am unable
to get the reload working when i change a file in the grape api project.
i tried to find some documentation on this but to no avail.
is this expected or i missed our something?
—
Reply to this email directly or view it on GitHub
https://github.com/padrino/padrino-framework/issues/1900.

you can take a look at https://github.com/dblock/grape-on-padrino my project is very similar

Confirmed no reloading on two levels: reloader middleware is not called and grape app does not provide reloading routines. I don't think it could be easily fixed from Padrino side.
You could look at this solution https://github.com/AlexYankee/grape-reload, it's based on Padrino Reloader code and as far as I can see, adds reloading routines to grape apps.


Great works! how about use transform hook to avoid generate *.mjs in development?

As the title says, I think it is more clean, furthermore build on demand is possible? What a interesting question 😂

After research of rescript document, rescript build and rescript convert will generate files directly, so that's not possible at the moment. Otherwise, rescript build -with-deps -w is enough and clean after I add vscode config like this:
{
  "files.exclude": {
    "src/**/*.mjs": true,
    "lib": false,
    "node_modules": false
  }
}


hierarchical sorting broken?

Release number or git hash: taggle branch
Web browser version and OS: chrome

Steps to reproduce

sorted hierarchically by:

Continent
Human dev index
Ppl knowing they...

Observed behavior

Ppl knowing they... should be sorted on level 3. This is not the case - see screenshot.
Expected behavior
Proper hierarchical sorting.

part of https://github.com/Caleydo/lineup/commit/acd37a2add00e4c2381ab9e3784542afda754e39


option proposal: programs.defaultBrowser = "${pkgs.firefox}/bin/firefox";

Issue description
It should be possible to define a default system browser (Debian has sensible-browser for that). This setting can then be used by other modules.
Example interface:
 programs.defaultBrowser = "${pkgs.firefox}/bin/firefox"; # or perhaps just plain "firefox" or both.

It depends on the use case when "firefox" is better or "${pkgs.firefox}/bin/firefox". Perhaps we should just have both options.

The standard solution for default browser is associating x-scheme-handler/http. Perhaps we need a NixOS module for MIME apps spec.

Since the use case in this case was for Eclipse, I think we should do something which Eclipse supports. Standards are great, but if the application of interest doesn't support it, it doesn't seem to solve anything.
You could argue that someone should open a bug in Eclipse (and I would agree). However, having such an option would still be useful for the large amount of software which doesn't support such standards.
I have not checked whether Eclipse supports this particular standard, but I expect it does not.


Make sure the refererURL is set when pointing to developer sandbox URLs

Make sure the refererURL is set when pointing to developer sandbox URLs
That will be the way that allows us to measure usage and adoption of the tool

Extension is not handling clicks to open links. @benoitf Is this already implemented on podman-desktop side?

Yes it is implemented


Allow basic math operations in number fields of the inspector

Godot version:
3.1
Issue description:
Something I find extremely useful in the GIMP is that in number fields it is possible to do basic maths that update the field when it loses focus.
I find myself really missing this feature in Godot.
Example:
Inspect a Control in the Inspector and navigate to Min Size, the values are x: 380, y: 100
If I want to half the Min Size I would go to x and update it to read 380/2 and once the field loses focus it evaluates to 190. (So the new value is 190).

I can confirm it no longer works in 3.1.

On 2.1 was also possible to do things like pow(some_number), that cannot be done on 3.0.

Closing as duplicate of #18932


Infinite loop in 0.10.1

Caused by the document.location setter in history.js:
html.HTMLDocument.prototype.__defineSetter__("location", function(value) {
  return this.parentWindow.location = value;
});

in conjunction with the parentWindow getter in index.js of jsdom:
dom.Document.prototype.__defineGetter__('parentWindow', function() {
  if (!this._parentWindow) {
     var window = exports.windowAugmentation(dom, {document: this, url: this.URL});
     this._parentWindow = window.getGlobal();
   }
  return this._parentWindow;
});

which eventually ends up calling createWindow, which creates a new DOMWindow() and in its constructor it tries to set the document.location:
if (options && options.document) {
  options.document.location = this.location;
}

which returns us back to the location setter where we originally came from

:+1+: (+1)

I'm also having the same problem.

Also, zombie 0.9.x doesn't have this problem.

Resolved.


dask.config.write API

Sometimes third party libraries want to re-use the dask config system (Coiled for instance).
What we currently do is to write some global state into the ~/.config/dask/ directory to store some user specific global state (e.g. an account name).
Since the config system is a bit elaborate and there are actually many places where this config may live, it would be nice for dask to offer an API to do just this.
# dask.config
def write(basename: str, config_dict: dict):
    """
    basename: str
         The filename for the new config file
    config_dict: str
        The configuration that should be written to this file
    """

The path of that file would then be determined by dask in precedence order. Trying to write to the location with highest precedence (accounting for environment vars and everything) and walking backwards in case the directory is protected or something similar prohibits us from creating this file (e.g. https://github.com/dask/dask/pull/10536)
This way, whoever calls dask.config.write can be certain that the configuration will also be read by dask

Might be worth thinking about whether this (1) always overwrites, (2) always merged, or (3) has merge kwarg that determines.
Probably simplest is overwrite and make caller responsible for merging in advance as needed. Haven't thought much about whether that's a problem in some cases.


Duplicate port spec results in unstartable container

When creating a container that address-binds the same port more than once, that (correctly) fails.
docker run -t -i -p <IP_ADDRESS>:9090:9091 -p <IP_ADDRESS>:9090:9090 ubuntu bash
2014/12/11 19:44:54 Error response from daemon: Cannot start container cdcc8ce8aa7539db230847e9df931fa38fa9b8405359d6b31dd9b2196c52d655: Bind for <IP_ADDRESS>:9090 failed: port is already allocated

However docker willingly creates this bricked container that's unstartable due to binding errors. Note the misleading error message.
docker start cdc
Error response from daemon: Cannot start container cdc: Bind for <IP_ADDRESS>:9090 failed: port is already allocated
2014/12/11 19:52:15 Error: failed to start one or more containers

I guess the best solution would be to detect dupe bindings and error out sooner, and be more accurate with the error message. Save the next guy 10 minutes of port debugging.

dupliicate https://github.com/docker/docker/issues/9797


pre-trained bit_models for EfficientNet

Do you have any plans for releasing pre-trained EfficientNet models (in PyTorch) ?

+1, pls!

+1, pls!

Hi, thanks for your interest, but we do not currently plan to do so.


Question : Multiple IR Commands

Hi Michael
As your suggestion, I'm using the "issues" tab to to ask a couple of questions, even if they're not issues!
First one : Is it possible to send multiple IR signals from a single Alex command?
As example, I'd like to create a device called "Cinema" - Then when I say "Alexa turn the cinema on", I'd like to issue several IR signals (turn amp on, turn projector on, turn tv off, switch Amp to HDMI 1 etc etc) ?
Many Thanks
James

You have a few options here
You can use Alexa's new built in "Routines" feature and just tie multiple device commands together, this works alright for simple power but if you want to get more complicated you can use the JSON commands to send multiple signals via the IRController
For example, I have my TV power on command also switch my surround sound input to TV and turn the roku to Plex
Here is what my URL for that looks like as an example
json?plain=[{type:'NEC',data:'40BE629D',length:32},{type:'NEC',data:'FF13EC',length:32},{type:'roku',data:'keypress/home',ip:'<IP_ADDRESS>'},{type:delay,rdelay:1500},{type:'roku',data:'launch/13535',ip:'<IP_ADDRESS>'}]
the readme goes over the syntax more, but essentially each item does the following

Turn on TV
Change surround sound input
Go home on the roku (closes anything running)
Wait 1500ms
Open plex on roku

All of this gets execute by Alexa when the turn on command is issued
The other option is to create a 'scene' using the scenecontroller with the command above just depending on your preference

Perfect, this is exactly what I was looking for. Thanks a lot - off to go and experiment now :-)


Message extension is not displayed in pop out window

If user clicks "pop out chat" , our app's message extensions is NOT displayed in pop out window (and it seems all 3rd party message extension are not displayed in pop out window).
Is this expected behavior or there is someway to enable message extension in pop out window as well ?

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

ID: 5319949f-7a29-1d17-0090-5f423ae8174e
Version Independent ID: 2d45f040-a391-ac0b-17e8-fa99e880fcf1
Content: Initiate actions with messaging extensions - Teams
Content Source: msteams-platform/resources/messaging-extension-v3/create-extensions.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Any update ?

@simiczhang , Could you please provide your app manifest?

@Trinetra-MSFT  you could search Jabber in "apps" and install this app and choose to "pop out chat" and click phone icon in message extensions in any chat

This is by design. Please feel free to reopen if you have further query on this


feat: Add support for aws_db_subnet_group resources

The feature should be listed in https://github.com/mineiros-io/terraform-aws-vpc#module-features also


make test harness importable

Make the e2e test harness importable so it can be reused in kpt pipeline e2e test.
@natasha41575

I expect the end states are:

The e2e test harness in the kpt-functions-catalog repo is merged with the test harness in the kpt repo.
The test harness in the kpt repo is stable enough. IOW, it is not updated very often.
kpt-functions-catalog repo depends on the test harness in the kpt repo for e2e tests.

Making it exportable can be the 1st step to transition to the end states mentioned above.


[DOC] Rolling upgrade from OpenSearch 1.0.0 to OpenSearch 2.x is broken

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
Our documentation explains how rolling upgrade works, usually OpenSearch 1.x should be directly upgradable to OpenSearch 2.x.
Unfortunately with a bug in OpenSearch 1.0.0 it does not talk to OpenSearch 2.x nodes.
We have to update our documentation to help users upgrade from 1.0.0 to 2.x.
The recommended path is to upgrade to a new minor in 1.x and then do an upgrade to 2.x.
See issue[1] for more details
[1]https://github.com/opensearch-project/OpenSearch/issues/4130

Moving this to in-progress since I'm beginning work on major/minor version upgrades.

@saratvemulapalli can you provide a link to the details for the bug you referenced? What specifically is the issue and is there an issue against the OS repo?

Thanks @JeffH-AWS for taking a look. The bug is documented in the issue and linked as well.
Here is the issue on OpenSearch[1]. We have a bug in OpenSearch 1.0 which does not talk to OpenSearch 2.x nodes in a mixed cluster state. This was fixed[2] in OpenSearch 1.0.1 (min) but was released in OpenSearch 1.1.0 Distribution.
So for customers who would like to upgrade from OpenSearch 1.0 to OpenSearch 2.x via rolling upgrade it will be broken.
The recommendation is to upgrade to any minor > OpenSearch 1.1.0.
Let me know if you have more questions.
[1]https://github.com/opensearch-project/OpenSearch/issues/4130
[2] https://github.com/opensearch-project/OpenSearch/pull/1191

Ok perfect! Thank you so much @saratvemulapalli

Putting on backlog since I was assigned to opensearch benchmark through the end of May.


REBAR_DEPS_DIR doesn't get expanded

I tried porting https://github.com/tonyrog/bt/blob/master/rebar.config to rebar3+pc but the build fails due to $REBAR_DEPS_DIR not being expanded.

I have the same problem.
Would you accept a pull request, if I would prepare it?
Or should I solve it in some other way?

We are being hit by this too.
When using {port_env, [{"CFLAGS", "$CFLAGS -I$REBAR_DEPS_DIR"}]}.
===> /home/cjk/git/campanja/csv/c_src/csv_parser_nif.c:5:24: fatal error: libcsv/csv.h: No such file or directory
So no expansion.


Fixing preferences in simulation + altitude must be double + updateFr…

Fixed preferences in simulation
Altitude must be double
Aircraft simulator state push frequency in Hz != altitude

Vlad I rewrote it at the same time you did.  It's a bit different now. Made a funtion of it. And found the same thing ypu did. I also mad the default map location the same. However the map does not quite work yet.On Dec 19, 2020 15:46, Vlad-Andrei Dorobantu<EMAIL_ADDRESS>wrote:Fixed preferences in simulation
Altitude must be double
Aircraft simulator state push frequency in Hz != altitude
You can view, comment on, or merge this pull request online at:
  https://github.com/The1only/rosettadrone/pull/43
Commit Summary
Fixing preferences in simulation + altitude must be double + updateFrequency != altitude
File Changes
M
app/src/main/java/sq/rogue/rosettadrone/DroneModel.java
(20)

M
app/src/main/java/sq/rogue/rosettadrone/MainActivity.java
(2)

Patch Links:
https://github.com/The1only/rosettadrone/pull/43.patchhttps://github.com/The1only/rosettadrone/pull/43.diff
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

No Problem Terje, I will close-it! :wink:


Support Meta Quest 3 native resolution

Meta Quest 3 has LCD panels with per eye resolution of 2064 x 2208. Setting this resolution is currently impossible as it gets rounded down to 2048 x 2208. This pull request relaxes resolution alignment from 32 to 16 pixels.

The Nvidia encoder does not support resolutions that are not a multiple of 32, with the exception of a few standard resolutions like HD, FHD, QHD etc. if we request a resolution which is not a multiple of 32 the image will be stretched to fill the next up multiple of 32, which will result in a blurry image.

We are talking about per-eye resolution, encoding is done on width * 2 so it is divisible by 32. Maybe align to 16 only for width with height kept at 32. With the changes and nvidia gpu 2064 x 2208 works for me nicely.

True, for width we can align to 16 per eye. I personally don't believe this has any practical benefit, as I believe that "native screen resolution" as magical numbers don't make sense. The reason is the image is distorted because of lens correction, which even causes inconsistent pixel densities across the eye view, and only a portion of the screen is often used to render the frame.

The Nvidia encoder does not support resolutions that are not a multiple of 32

huh, h264 macroblocks are 16x16

CI failure is unrelated I'm guessing a cargo apk update or something, is now enforcing a specific platform version 32

huh, h264 macroblocks are 16x16

@The-personified-devil HEVC macroblock size is 64x64. I think the macroblock side doesn't matter, it's a limitation of the API itself

@The-personified-devil HEVC macroblock size is 64x64. I think the macroblock side doesn't matter, it's a limitation of the API itself

Yes it does, 1080p and co are only supported because the encoder just adds a black bar that it then cuts out during decoding again. So if the encoder wants stuff to be aligned by macroblocks (which it should, dunno why it wouldn't, except for special resolutions) and we would have to conform to that for all codecs that we have. So with the 64 for hevc we couldn't lower the per eye to below 32.

Hm. Welp, so should we increase the alignment to 64?

Hm. Welp, so should we increase the alignment to 64

No because as we noticed per eye gets multiplied to 2, so 32 alignment per eye would be enough, and that's what it already is. Only thing I'm not sure about is how foveated encoding handles all of that/how the encoders have been dealing with it or are we actually aligning that properly?

No because as we noticed per eye gets multiplied to 2, so 32 alignment per eye would be enough, and that's what it already is.

Nope, vertical resolution is still wrong.

Only thing I'm not sure about is how foveated encoding handles all of that/how the encoders have been dealing with it or are we actually aligning that properly?

We assign a new resolution that is a multiple of 32 horizontally and vertically.

@nowak-pl ping?


Greek phone validation for landline and mobile phones.

Hello,
as I said earlier I have made a regex that validates greek phones, both landlines and mobiles. I hope I did the pull request thing correctly.

Coverage remained the same at 97.43% when pulling e9f5a045c77448eb5881a9f296d9f347995ce860 on gkaimakas:master into 86767b88ac16ba06f9c59473fd7fd0a76da51c8c on chriso:master.

Hey, thanks for the PR. Can you swap out the tabs for four spaces to keep the same style as the library?

yeah of course! I saw it too that it didn't align correctly but I didn't
know what was wrong
2015-02-20 3:05 GMT+02:00 chriso<EMAIL_ADDRESS>
Hey, thanks for the PR. Can you swap out the tabs for four spaces to keep
the same style as the library?
—
Reply to this email directly or view it on GitHub
https://github.com/chriso/validator.js/pull/364#issuecomment-75172252.

--
Γιώργος

Coverage remained the same at 97.43% when pulling 4d11a77d55b059a96dad33b356b52dd734bb2374 on gkaimakas:master into 86767b88ac16ba06f9c59473fd7fd0a76da51c8c on chriso:master.

I think i fixed the formating on my fork

If you have a look at the diff though you'll see that you changed a whole bunch of other formatting: https://github.com/chriso/validator.js/pull/364/files?diff=unified :wink: I can fix it up and merge.
Thanks again!

This is available in version 3.31.0.


Miscellaneous changes

Sort projects, issues, and clients by their id.
Add UI for help window

Codecov Report

Merging #132 (8e7408f) into master (2a0c7c9) will decrease coverage by 0.99%.
The diff coverage is 2.29%.

@@             Coverage Diff              @@
##             master     #132      +/-   ##
============================================
- Coverage     37.43%   36.44%   -1.00%
  Complexity      425      425
============================================
  Files           136      136
  Lines          2703     2782      +79
  Branches        321      333      +12
============================================
+ Hits           1012     1014       +2
- Misses         1545     1622      +77
  Partials        146      146

Impacted Files
Coverage Δ

...dress/logic/commands/client/SortClientCommand.java
0.00% <0.00%> (ø)

...address/logic/commands/issue/SortIssueCommand.java
0.00% <0.00%> (ø)

...ess/logic/commands/project/SortProjectCommand.java
0.00% <0.00%> (ø)

...eedu/address/logic/parser/ClientCommandParser.java
29.76% <0.00%> (-1.11%)
:arrow_down:

...ava/seedu/address/logic/parser/IssueCliSyntax.java
0.00% <0.00%> (ø)

...seedu/address/logic/parser/IssueCommandParser.java
0.00% <0.00%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
39.47% <0.00%> (-5.99%)
:arrow_down:

...edu/address/logic/parser/ProjectCommandParser.java
25.53% <0.00%> (-0.85%)
:arrow_down:

src/main/java/seedu/address/model/AddressBook.java
31.34% <0.00%> (-5.83%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Program admins get emails when one of their program forms gets filled out

Emails contain links to pages where they can download PDFs/CSVs (#191 or eventually #192)

OK this clearly works in staging. I have no way of knowing if it works in prod, but we'll have to wait until the program admins start getting applications (hopefully that's soon!). At that point we can ask them if they're getting emails and, if necessary, revisit this.


Footer does not display correctly on smaller width screens

Text overflows from the footer, it should be contained within the footer at all times and there should not be a need to scroll

Deleted some unnecessary text, problem solved (for now)


Minimal harness for running CIFAR10 experiments

Hello! Just in case it might be useful for others who'd like to test the optimizer out as well on the CIFAR10 dataset -- just wanted to add my training harness.
Should be easy to swap out out with any other model, or something like CIFAR100 as well.
Its more or less in the same spirit as the MNIST example, and should be easily runnable :)

Thanks! I will look over this Monday.

Thanks for the contribution. We are going to decided on the exact ResNet-18 variant and default training parameters over the next few days.

Sounds good :) Thanks for looking into it -- happy to modify if required!

@nelaturuharsha I'm currently working on this, and will have this out soon


Release 0.0.2

### v0.0.2

* Upgrade versioneer to 0.17. #15
* Extend license copyright into 2017. #16
* Update copyright years in docs. #17
* Recut with cookiecutter. #18

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Merging without Travis.


Update crossplane-runtime to get rate limiter

Description of your changes
This will let provider-helm cap the wait time in error cases to 1 minute.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested
Manually.

@bobh66 Oh sorry, I didn't notice the other PR. I'd be happy to merge yours if you can incorporate the few additional things this PR does.

@muvaf - Sure, that's fine - I'll compare them tonight and add the deltas into 131.  Thanks!

@bobh66 Actually, I'll merge your PR and then rebase this on top of your changes. Does that sound good?

@muvaf - that's sounds good - I compared the two PRs and other than the workflow updates I think they are essentially the same.  Thanks!

@turkenh ready to go.

Thanks @muvaf - I thought that was strange but that's how provider-template was done so I assumed there was a reason for it.
So concurrency is max number of running go routines, and reconciliation rate is max number of outstanding API calls to the server allowed by the RateLimiter?  Is it possible for a goroutine to have more than one API call outstanding at any one time?  I'm just wondering why they were tied together initially.  If the concurrency value is a de-facto limit on the number of outstanding API calls then it makes sense to use the same number for both.  But if it's possible to have more outstanding API calls than goroutines, it would be better to separate them.  Thanks!


[Service Connector] az webapp/continerapp/spring connection create/update containerapp: Add ACA as target service

Related command
az webapp/continerapp/spring connection create/update containerapp --source-id  --target-resource-group  --target-app-name  (--secret --client-type )
Description
This PR supports ACA as target service for POC.
Testing Guide
az webapp connection create containerapp --source-id  --target-resource-group  --target-app-name  --secret --client-type dotnet
az webapp connection update containerapp --id  --client-type python
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Service Connector


Update active-directory-accessmanagement-self-service-group-managemen…

Added text to clarify that self service is only available for SGs and Office groups, not for DLs and Mail enabled SGs

Hi @RobdeJong, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Merge prTesting_201908201003 branch to master branch

Docs Build status updates of commit ad8b020:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix(lyaml) allow the lyaml version to be modified

@hishamhm pointed out that kong-build-tools isn't consuming the environment variable necessary to change the lyaml version
I think there's better ways we should be instally lyaml but that refactor is a little out of scope for the time I have available / not a high enough priority
ref: https://github.com/Kong/kong/pull/4926

@hutchic lyaml is a regular Kong dependency (just like the bunch of other rocks listed in our rockspec). libyaml is a system library dependency that needs to be installed by kong-build-tools in advance. It's a similar relationship to luaossl and openssl. I think we should not single out lyaml in the tooling, and just let the rockspec take care of its version.

agreed we should but for reasons that's been lost to history (and my forgetful brain) I think we found we had to isolate it out. I'll try to PR the removal of installing lyaml in kong-build-tools and will trust the test suite to refresh my memory if it is in fact still needed

closed in lieu of https://github.com/Kong/kong-build-tools/pull/124


Add a slug type field partial?

Sometimes models need to have a user-settable url slug, and it might be nice to have this built in to BT.
For instance, if the Team model had a slug you might be able to view a team at /teams/my-team instead of at /teams/a39bc12.
It's kind of just a text_field with some additional requirements and validations (like uniqueness, for example), but there's also the matter of looking up models by slug instead of by id/obfuscated-id, which can be solved by changing to_param.
Maybe this is more application specific than generic and wouldn't be a good addition to BT. Just wanted to throw it out there and see what other people think.
Here's an article that talks about lots of the specific details that we'd need to address if we wanted to do something: https://nts.strzibny.name/activerecord-model-slugs/

I feel this should be packaged into the obfuscated_id functionality directly, defaulting to the :slug column, but configurable.
A slug is also a way to obfuscate the id, and to_param and find could be made to prefer the slug if defined, and fall back to the hashId.

Ooooh, that's a good connection, @pascallaliberte! I think you're absolutely right on that.


Bug 2124582: Fix filter by OS - VM list

Signed-off-by: Matan Schatzman<EMAIL_ADDRESS>
📝 Description
Fix filter by OS that was broken- VM list.
🎥 Demo

Please add a video or an image of the behavior/changes

@pcbailey @hstastna @upalatucci  - PTAL :)

/lgtm

/retest


 But I was thinking about taking the chat from my app and have a "desktop messenger". Is it possible to take the JSQMVC for that? It says iOS all the time, so I was wondering.
And if not. Why? Just out of curiosity

It uses iOS specific UI APIs everywhere. UICollectionView, UIView, etc. Another similar library with macOS support would be great though.

didn't even realized, that UICV and UIView are iOS specific. haven't really thought about macOS stuff before. just thought.  same company. all apple. same IDE. same language. so should work... but mmmkayyy. well. then I'll have a look at the alternatives... but would be nice tho

There is no UIKit in mac OS (yet?)

MacOS uses Cocoa.    UIkit for iOS is simillar but not the same.    To implement UIKit on MacOS, take a look at Chameleon, from http://chameleonproject.org
Based on the objective of the project, in theory, it should be an easy port.
This (https://itunes.apple.com/app/apple-store/id922793622?pt=814382&mt=8&ct=how_i_email)  is how I Email now

On Nov 30, 2016 at 1:53 AM,  <Julien Chaumond (mailto:notifications@github.com)>  wrote:
There is no UIKit in mac OS (yet?)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,   view it on GitHub (https://github.com/jessesquires/JSQMessagesViewController/issues/1929#issuecomment-263830513), or   mute the thread (https://github.com/notifications/unsubscribe-auth/AOaU65qfpdhYoOBBaVvoxo6iTTF00z32ks5rDUelgaJpZM4K_du3).


fix(caa upload): polyfill array.at

I though MB polyfilled this one for us, but apparently it doesn't.
I'm not sure how I feel about including a whole bunch of polyfill code just for that single method (it's not actually polyfilling anything but array.at, but just happens to include a bunch of other stuff for core-js internals I guess?). At the same time, I want the source code to be modern and not have to be limited to old ES versions.
For this method though, I'm not sure whether it's worth the bloat... But on the other hand, the script also runs on pages other than MB, where polyfills might not be available.
Possibly closes #214

I'm wondering whether it would make sense to separate all the third-party stuff into a vendor file and @require it instead of embedding it into the script itself...


feat(docker): docker support

https://hub.docker.com/r/fgribreau/psi/

I don't see the point. The official way to install PSI is with npm or another compatible client. You're free to create a Docker build, but not something that should be added here.

@sindresorhus I wanted this because we use it inside some jobs in our CI process that should not depends on nodejs/npm for it thus making it agnostic through docker.


Update and rename helpfiul-links to helpful-links

add additional sites to help you
(btw welcome to github :) )

thank you for your help, I have plenty to get started on now.


Proposal and feedback required

We have introduced an unexpected change for some of our users with the namespace resolution, for example see here. I added that change.
The issue is that to properly read config maps and secrets getting the namespace is mandatory, as we can not default or imply it. The situation is slightly more complicated because kubernetes-native client requires a namespace to be present, otherwise we currently have this issue.
To put it very simple: if we want to read a secret or a config map, we must have a namespace.
This opens an interesting discussion. A namespace makes sense only when you are in an actual cluster, of course.
What if you are not in a cluster (see the first link), but you still want to be able to start your application (or dependent applications). I can think of another reason - what if you have spring-cloud-kubernetes-XXX-config on the classpath, but start (without a cluster) an un-related feature?
Before we introduce namespace resolution concept, we used to log an error, but never fail. After introduction of that concept, we now fail. Of course users are surprised, I can't blame them.
We have @ConditionalOnKubernetesSecretsEnabled and @ConditionalOnKubernetesConfigEnabled or even @ConditionalOnKubernetesEnabled, but this adds more work for users. Also this brakes their expectations from a previous version. Again, this sort of makes sense, imo.
What if we could disable reading secrets and config maps entirely (not dependent on those conditionals), unless we are in a cluster? This would not be complicated at all, as a matter of fact.
We already have this post processor (same idea exists for fabric8 client):
public class KubernetesClientProfileEnvironmentPostProcessor extends AbstractKubernetesProfileEnvironmentPostProcessor {

	@Override
	protected boolean isInsideKubernetes(Environment environment) {
		CoreV1Api api = new CoreV1Api();
		KubernetesClientPodUtils utils = new KubernetesClientPodUtils(api, environment.getProperty(NAMESPACE_PROPERTY));
		return environment.containsProperty(ENV_SERVICE_HOST) || utils.isInsideKubernetes();
	}
}

Reusing this is not an option, because KubernetesClientPodUtils::isInsideKubernetes requires a namespace for it to work and it's exactly what we are trying to not do.
We could start implementing this by simply looking if KUBERNETES_SERVICE_HOST is present (that is what environment.containsProperty(ENV_SERVICE_HOST) does).
We are basically going to say that : if that env property is present, try to read config maps and/or secrets; do nothing otherwise. This is not bullet proof, but should be good enough to narrow users expectations.
If that is not going to be enough and we will get reports of not being so, we can always go a little deeper and replicate/extract the idea used here in KubernetesClientPodUtils:
	private V1Pod internalGetPod() {
		try {
			if (isServiceHostEnvVarPresent() && isHostNameEnvVarPresent() && isServiceAccountFound()) {
				return client.readNamespacedPod(hostName, namespace, null, null, null);
			}
		}
		catch (Throwable t) {
			LOG.warn("Failed to get pod with name:[" + hostName + "]. You should look into this if things aren't"
					+ " working as you expect. Are you missing serviceaccount permissions?", t);
		}
		return null;
	}

specifically I am talking about isServiceHostEnvVarPresent and isHostNameEnvVarPresent and isServiceAccountFound.
For the time being simply checking for KUBERNETES_SERVICE_HOST should work for the vast majority of users.
wdyt?

Spring Boot also auto-detects Kubernetes by checking for *_SERVICE_HOST and *_SERVICE_PORT environment variables.


Release Google.Cloud.Language.V1 version 3.4.0

Changes in this release:
New features

Support API keys directly in the Language library (commit 33aad2c)

Created release for Google.Cloud.Language.V1-3.4.0


[physx] Add support for checked and profile builds.

This adds the ability to build the physx port in "checked" and "profile" mode using a custom triplet, enabling reports on API misuse and allowing scene visualization with the NVIDIA PhysX Visual Debugger in Release builds.

What does your PR fix?
See above
Which triplets are supported/not supported? Have you updated the CI baseline?
No change
Does your PR follow the maintainer guide?
Yes

Hey @Hoikas! Thanks for the pull request; however, for cases like this, we recommend using custom triplets. You can, for example, write your own x64-windows-checked.cmake file which contains something like this:
# copy from x64-windows
if(PORT STREQUAL "physx")
    set(RELEASE_CONFIGURATION "checked")
endif()

We're looking at improving this experience in the future, but for now this is the "correct" way to do something like this.
Again, thanks so much :heart: :heart:
If you'd like you can modify the port file to not always set RELEASE_CONFIGURATION by doing something like:
if(NOT DEFINED RELEASE_CONFIGURATION)
  set(RELEASE_CONFIGURATION "release")
endif()
if(NOT DEFINED DEBUG_CONFIGURATION)
  set(DEBUG_CONFIGURATION "debug")
endif()

Thanks for the feedback! I look forward to the improved handling for this kind of situation that you mention. Features are not the best way to expose mutually exclusive options like build type changes, I suppose. I'll change this pull request to match your suggestion soon.

Thanks so much @Hoikas :heart:

Updated. I only added the if(NOT DEFINED ...) check for the release configuration--debug builds of PhysX already have the instrumentation included in both the checked and profile builds.

Alright, this looks good to me! Thanks so much @Hoikas :)

It was no trouble, @strega-nil, but I know I am going to have a question soon about an upcoming PR I am planning :smile:


Add Spanish translation

@danielweck  : I have added the spanish translation as commented on email list.
Hope this helps. Thanks for your attention and work.

thank you very much for your PR.
Do you know how to connect the new language in the code ?
So that your PR can be integrated into thorium.
check this file : https://github.com/readium/readium-desktop/blob/develop/src/common/services/translator.ts
Otherwise we will integrate it

I modified the mentioned file.  Please let me know if something needs to be added or changed. I suppose that locale-names files are automatically generated. Hope a procedure for contributing can be documented, maybe helpful :)

With my last commit, checks failed. Seems that src\resources\locale-names/esLang.json needs to be created to translate language names. In this case, I will push another commit for this, based on
http://utils.mucattu.com/iso_639-1.html
Please, let me know if this is not needed.

yes, you are right, @Scarsniik used this script to convert csv from wikipedia to json.
If that can help you. I guess these ISO codes are not easy to get.

I have followed the following criteria:

Capitalize initial character of language names, as done in other languages, though in Spanish is only required in certain situations.
Use the first or most wnown or easy name when various names can be used for the same language.
Please, review and merge when appropiate. Thanks

perfect, thank-you. I tested it works well.
@danielweck will be able to merge if he agrees.

Thanks, if/when @danielweck  merges this, I will make a post to share this with spanish users.

Thank you! Merging this PR into a staging branch in order to check against the latest updates (keyboard shortcut editor new labels, etc.)

Follow-up PR: https://github.com/readium/readium-desktop/pull/971


#include <Crc16.h> is missing?

I have connected my esp8266 but when I compile the code, it shows this library is missing.

@rgingingon CRC16 is an external library used to detect Software Serial transmission errors.  The library can be found here. This is also mentioned in the README.md file.
Since meshquitto uses a number of external libraries, I would advise you to have a read through the README.md (or the repository homepage), where all information regarding the necessary libraries can be found.
Hope this helps!

@rgingingon Did the above solve your problem? Should we close the issue?


Rendering without  app.router()

Application Stuff
First of all, I really appreciate this idea and movement for a slimmer, faster JS; really impressed with the roots that you have laid out here. I have a question (I am not assuming that I am correct here, that's up to you), does a router need to be required? I understand that the goal is to create a really small, powerful framework, but I believe I am experiencing a small issue because of the router. I am implementing  in my application using electron, and I believe the router may be confusing events when the anchor tag is clicked and the router re-renders the entire page.
export default (params, state, send) => choo.view`

  ${sidebar(params, state, send)}
  ${simulator(params, state, send)}

`;

const app = choo();
app.router((route) => [
  route('/', require('./src/components/main'))
]);
document.addEventListener('DOMContentLoaded', () => {
  document.body.appendChild(tree);
});

@wenkesj hey yeah, a router is required. If you'd like to create elements without a router consider using bel and appending the result to the DOM as usual.
If you want to disable routing, try running app.start({ href: false }). If you want to enable hash routing instead try running app.start({ hash: true }).
I hope this gets you on the right path; if not I'd be happy to dig in further if you can provide me with a repro. Cheers!

@yoshuawuyts Okay, cool. Makes sense, thanks

FYI I have choo working in electron with hash routing here if that helps.


Allow recursive aliasing

Summary
Enable recursive aliasing (alias containing other aliases)
POST /registries/my-registry/aliases/group-A
{
     "type":"[string]",
     “value”:[
"{{/registries/my-registry/aliases/JPM}}",
"{{/registries/my-registry/aliases/GS}}"
]

Acceptance criteria

[ ] Allow storing a recursive aliases
[ ] Allow resolving recursive aliases
[ ] E2E tests

I don't see value in this feature and I consider it as over-engineered to be honest.


Add static pegasus seed drop in Moosh Nuun

Reason for PR:
We talked about this softlock potential of the nuun highlands cave check on discord before and the topic of using a static pegasus drop from one of the bushes came up. I tried my hand at implementing that functionality while keeping as much vanilla code as possible.
Change Description:
Inserted a obj_Pointer into group0Map27ObjectData, with disassembly macros for easier readability:
obj_DoubleValue $9ad4 $68 $48  # vanilla data that got replaced for pointer
obj_Conditional $04            # conditional for moosh nuun
obj_ItemDrop $00 $07 $58       # Drop Pegasus Seed ($07) at coordinates 5,8
obj_EndPointer

Considerations:

I included the surrounding vanilla code to make it easier to grasp the changes. Probably better to remove those when merging.

I don't know if misc.yaml is the right place for those changes. I put them there because I don't know where else they would really fit.

Works for me, but can you PR to patch instead of dev?

The branch is based on jangler/dev, so changing the PR target would mean merging all current dev changes to patch.
But I can just make a branch based on patch, port the change and change source branch.
Gimme a minute.


Docs needed for new marketing campaign APIs

I'm trying to add a recipient(s) to a list using the PHP SDK but it doesn't seem to work. I get hit with the unauthorized access error (403). Funny thing is I'm even on a paid plan, changed API keys but all to no avail. Sending emails works fine but I can't get a subscription to work.
I contacted support with my issue and I was told they don't do that kind of diagnosis and will have to leave a message here on git. smh...

We have the same issue here. The other APIs are working fine.
We get this response:
{"errors":[{"field":null,"message":"access forbidden"}]}

Hi @dswmedia apparently, using the SDK's subscribe method doesn't work right out the box for version 2. You may have to use the rest API. I, however, had to pay to get the legacy contacts activated on my account then subscription started working. Contact support to activate legacy marketing

@interwap
I have solved the issue in another way.
I have contacted support they told me to use the new api.
I have used something like this:
$sg = new \SendGrid("SG.XXXX");
$request_body = json_decode('[
    {
        "age": 25,
        "email"<EMAIL_ADDRESS>
        "first_name": "",
        "last_name": "User"
    }
]');

try {
    $response = $sg->client->contactdb()->recipients()->post($request_body);
    print $response->statusCode() . "\n";
    print_r($response->headers());
    print $response->body() . "\n";
} catch (Exception $e) {
    echo 'Caught exception: ',  $e->getMessage(), "\n";
}

@interwap
Sorry, this should be it.
https://sendgrid.api-docs.io/v3.0/contacts/add-or-update-a-contact

@dswmedia yes, this is via the API... but not using the Sendgrid SDK. I'd imagine that the point of even using the SDK at all is so we don't get to write curl requests ourselves.

@interwap I just encountered the same issue, but was able to resolve it using the SDK with the following:
$sg = new \SendGrid("SG.XXXX");
$request_body = json_decode('{"contacts":[{"email":"test@example.com"}]}');
try {
$response = $sg->client->marketing()->contacts()->put($request_body);
print $response->statusCode() . "\n";
print_r($response->headers());
print $response->body() . "\n";
} catch (Exception $e) {
echo 'Caught exception: ',  $e->getMessage(), "\n";
}
It seems that the SDK examples needs to be updated with the new marketing resources vs. the legacy ones.

@interwap I just encountered the same issue, but was able to resolve it using the SDK with the following:
$sg = new \SendGrid("SG.XXXX");
$request_body = json_decode('{"contacts":[{"email":"test@example.com"}]}');
try {
$response = $sg->client->marketing()->contacts()->put($request_body);
print $response->statusCode() . "\n";
print_r($response->headers());
print $response->body() . "\n";
} catch (Exception $e) {
echo 'Caught exception: ', $e->getMessage(), "\n";
}
It seems that the SDK examples needs to be updated with the new marketing resources vs. the legacy ones.

Thanks, will test that out

I'll review a PR to update the docs if anyone wants to create it.

bah.. just spent an hour to fix the legacy error...

Yeah, new docs need to be created. I just lost an hour trying to figure out why postman works and the library does not. I would suggest SendGrid add a disclaimer line and a link to this issue in the docs. That would save a lot of time!

I figured out adding pretty easily. However deleting was much more of a pain than it needs to be. Add a little defensive programing to this and you should be golden.
    public function removeContacts(array $emails): void
    {
        $request = ['emails' => $emails];

        /** @var string $response */
        $response = $this->emailProvider->client->marketing()
            ->contacts()
            ->search()
            ->emails()
            ->post($request)
            ->body();

        $results = json_decode($response, true);

        $emailIds = [];

        foreach ($results['result'] as $contact) {
            if (isset($contact['contact'])) {
                $emailIds[] = $contact['contact']['id'];
            }
        }

        $this->emailProvider->client->marketing()
            ->contacts()
            ->delete(null, [
                'ids' => implode(',', $emailIds)
            ]);
    }


Avoid pinning CPU to the last core

This PR changes run_benchmarks.rb to avoid pinning CPU to the last core.
My CPU, Intel Core i7-12700KF, has 8 "P-Cores" and 4 "E-Cores", 12 in total. In the CPU core list, 8 (or 16 with HT) P-Cores are placed first and then 4 E-Cores follow. Therefore, the current run_benchmarks.rb uses an E-Core. E-Cores are much slower than P-Cores, so it makes benchmarking slow and stressful for me.
To choose a P-Core, I changed the CPU core index from 100% of nproc to 50% of that. In my environment, this change speeds up Optcarrot by 70%.

I'm personally fine with that, but I thought we didn't do that with the assumption that OS would pick the first core for other tasks when every core is unused (which should be the case in a benchmarking environment) and the first core may have more noise than later cores in that case. If it's not really the case in reality, I guess it's fine to just pick the first core.

It seems very likely that some operating systems may default to the first core for either the OS itself or as a default core to assign tasks to.
imo we should either use the logic from this PR or give up on core pinning, but it did seem like core pinning did increase stability of the results in the past. I would default to sticking to this solution.


Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1

Hi all,
I am using "Sonos Control" but it seems I am getting the following errors for the respective commands:
"Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1#Pause""
"Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1#Next""
"Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1#Previous""

Play
Stop
Mute
Unmute
Volume Up
Volume Down

seem to all work. Just the others throw error messages:
22 Sep 20:49:46 - [error] [better-sonos-control:7997c9b0.f7e378] Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1#Next"
Error: HTTP response code 500 for "urn:schemas-upnp-org:service:AVTransport:1#Next"
    at Request._callback (/home/pi/.node-red/node_modules/sonos/lib/sonos.js:216:23)
    at Request.self.callback (/home/pi/.node-red/node_modules/request/request.js:186:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (/home/pi/.node-red/node_modules/request/request.js:1163:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (/home/pi/.node-red/node_modules/request/request.js:1085:12)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:111:20)

Do you have any ideas what the problem could be and how I can solve it?
Thanks!
Christoph

Same here.

Same error here.


Harmonize material names and class names

Description
For materials.resolveMaterialClassByName() to function, material.name and the class name should be the same. This PR ensures that is true for all materials.
No testing was added as the name field on the material isn't really something to be tested.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in setup.py.

If we're gonna do this, I welcome you to fix all the broken blueprints files in our benchmark models :-)

If we're gonna do this, I welcome you to fix all the broken blueprints files in our benchmark models :-)

I don't think this needed. We ran into this in terrapower-twr a few months ago with T92Depleted vs T92-Depleted-Boron. The blueprints use the class name not the name field.

Well I'll be darned, you're right. Forget my snarky comment!

Two unit tests failing:
Traceback (most recent call last):
  File "/home/runner/work/armi/armi/armi/tests/test_lwrInputs.py", line 67, in test_loadC5G7
    self.assertIn("Uranium Oxide", streamVal, msg=streamVal)
  File "/opt/hostedtoolcache/Python/3.8.16/x64/lib/python3.8/unittest/case.py", line 1179, in assertIn
    self.fail(self._formatMessage(msg, standardMsg))
  File "/opt/hostedtoolcache/Python/3.8.16/x64/lib/python3.8/unittest/case.py", line 753, in fail
    raise self.failureException(msg)
AssertionError: 'Uranium Oxide' not found in '[warn] Ignoring invalid settings: fakeBad\n=========== Settings Validation Checks ===========\n[warn] Burn chain already imposed. Skipping

@john-science  Two unit tests are updated! Sorry I didn't catch those locally

@alexhjames / @john-science  This broke a few of my internal unittests because of name changes to Uranium Oxide and Zirconium. No big deal, but I think we need to do a better job communicating these kinds of changes ... or maybe on my end listening for these kinds of changes.


Recoloured XP drops when praying doesn't work properly

When one gets more than the combat skill's XP drop in the same game tick, the XP drop isn't recoloured.
I.E. Melee XP drop when casting a mage spell (e.g. venge), Melee XP drop when cannon fires.

https://github.com/l2-/template-plugin/issues/33
Basically because of this. Because of long range attacks giving defence xp i opted to to assume xp drops containing multiple combat skills will not count as melee hits. I could hoist attack and strength xp drops and then it will only be wrong for defense xp drops but i'm not sure what would be best.

Is there a method to retrieve the currently selected attack style? Can match that to the prayer instead of xp drop mayhapsingly, but this might cause a problem with manually casted spells.
https://github.com/runelite/runelite/blob/master/runelite-client/src/main/java/net/runelite/client/plugins/attackstyles/AttackStylesPlugin.java

https://github.com/runelite/plugin-hub/pull/3036

Also while I'm here, is it too much effort to prebake in certain predicted hit drops for raids/bosses.
This is what https://github.com/l2-/template-plugin/blob/master/src/main/resources/com/xpdrops/npcs.min.json is for. I scraped it from the wiki and added a few bosses manually to the end such as hunleff. Content such as cox and tob is too much of a hassle to correctly predict hits as xp scales with team size. For regular bosses/npcs if the hit is not correctly predicted then its likely not in the list i just linked. In that case you can best calculate the correct xp bonus and add it to the wiki. I can update the file by scraping from the wiki or you can give me missing xpbonus multipliers directly. I rather not update every time for a single npc though.

Could help come up with formulas to compute XP multipilier for Chambers/Tob if needed, would need to keep track of party size and highest combat level in the party thought for that, and guardians would be a bit inaccurate if everyone isn't 99 mining. Something like a toggle for chambers might work though, if people want prebaked predicted hits for maxed accounts.

Im not willing to add such specific logic for now.


fix(autoware_auto_common): modify build error in rolling

Signed-off-by: wep21<EMAIL_ADDRESS>Description

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

move to #718


If a retryable workflow returns a non-retryable error that was originally returned by an activity, the workflow still gets retried

Expected Behavior
I expect the workflow to not be retried if the type of the error that is returned from the workflow (although originating from an activity) matches one of the non-retryable types specified in the workflow's retry policy.
Actual Behavior
The workflow keeps getting retried despite the error being a non-retryable type.
Steps to Reproduce the Problem

Define an activity that results in a non-retryable error:

func MyActivity(ctx context.Context) (string, error) {
	return "", temporal.NewApplicationError("this is a rejection", "Rejection")
}

Define a workflow that returns the error as is from the activity:

func MyWorkflow(ctx workflow.Context) (string, error) {
	ctx = workflow.WithActivityOptions(ctx, workflow.ActivityOptions{
		ScheduleToCloseTimeout: time.Second * 60,
		RetryPolicy: &temporal.RetryPolicy{
			InitialInterval:        time.Second * 1,
			NonRetryableErrorTypes: []string{"Rejection"},
		},
	})

	var result string
	err := workflow.ExecuteActivity(ctx, MyActivity).Get(ctx, &result)
	if err != nil {
		return "", err
	}

	return result, nil
}

Call the workflow with a retry policy:

func main() {
	c, err := client.NewClient(client.Options{})
	if err != nil {
		log.Fatalln("unable to create Temporal client", err)
	}

	workflowOptions := client.StartWorkflowOptions{
		TaskQueue: "someTaskQueue",
		RetryPolicy: &temporal.RetryPolicy{
			InitialInterval:        time.Second * 1,
			MaximumAttempts:        30,
			NonRetryableErrorTypes: []string{"Rejection"},
		},
	}

	var result string
	err = c.ExecuteWorkflow(context.Background(), workflowOptions, MyWorkflow).Get(context.Background(), &result)
	if err != nil {
		log.Fatalln(err)
	}

	fmt.Println(result)
}

I would expect the workflow to quit on the first attempt, but it keeps getting retried until MaxAttempts is reached.
Note that if I omit the retry policy from workflow options, I get back the error and am able to cast it as temporal.ApplicationError (as expected) and if I call its Type() method, it returns "Rejection".
Specifications

Version: v1.1.0
Platform: mac

Closing as I think this thread sufficiently clears up the issue


feat: Use since/until-build instead of listing out all releases

Why do we need this improvement?
We're manually curating a laundry-list of every IDE release version.
How will this change help?
Simpler configuration file -- that doesn't need to be updated with every IDE release version
Screenshots
No response
How could it be implemented/designed?
Per https://github.com/JetBrains/intellij-platform-gradle-plugin/issues/1510

If no specific IDEs/versions are listed, it will verify against given since/until-build by default.

🚧 Breaking changes
No
👀 Have you checked for similar open issues?

[X] I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue?
Yes I am willing to submit a PR!

Hold up, the syntax proposed in  doesn't work

Hold up, the syntax proposed in  doesn't work

Finally heard back from JetBrains people.  Now need to go back and look at this to see how to implement...


--single-transaction does not work with mysqldump anymore

The Option --single-transaction will not be appended to the mysqldump command anymore in Version
<IP_ADDRESS>001191557 on MacOS.
Mysqldump is from Homebrew<EMAIL_ADDRESS>Server Version is 5.6 connected via SSH Tunnel.

Thanks for the bug report.

verified


[Done] BilinearResize2D optimized

Description

align_corners param in resize op is first introduced by ten*low, then py*ch also bring it to its interpolate op. Our mx.nd.contrib.BilinearResize2D op only support align_corners=True before. Now, We bring align_corners=False(same as OpenCV resize in LINEAR mode) support as sometimes it will get more accuracy.
Optimize the performance of foreward and backward of mx.nd.contrib.BilinearResize2D.
Optimize the performance of mx.nd.contrib.BilinearResize2D in fp16 as it is too slow before. As I've tested, If we explictly set dtype to float16, we can speed up our training when compared to float32. So we can speed up the training of gluoncv segmentation models later. (Note: there's no effect on amp from nvidia )

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Test? :)

Yes, I've tested compared to interpolate of py*ch and It works good. There are no enough test cases of this op, should we add?

Yeah, it would be great if you could add testcases

@wkcn @zhanghang1989 @marcoabreu for review

@wkcn  It's all resolved, can we merge this?

Merged. Thanks for your great contribution!


Adding NuGet package target frameworks to component detection data.

Per a conversation about adding target framework information to the component detection data, this is a PR that shows what that could look like.
Things get slightly tricky here since packages declare target frameworks through the dependency declarations in the nuspec, as well as the folder structure of the package. This hasn't been through much testing beyond the unit tests, but they've been augmented to test some potential patterns.
I mainly want to get some feedback on this before I spend much more time on testing. I can potentially test this against all NuGet.org packages pretty easily, which also gives us a test oracle since they calculate target frameworks as well.
Things that might need consideration:

Addition of NuGet.Packaging as a dependency
how the data is plumbed through the NuGetComponent (I followed the Authors example as an array of additional strings)
Removal of async from ProcessFile (nothing in there was async when I finished)
Shifting more responsibility to NuGetNuspecUtilities even through it's not strictly nuspec-related anymore.
Using the nuspec reader rather than bespoke XML parsing

I'm open to any and all feedback here.

Hi @marklio,
Can I get a bit more context on who you spoke to about this work? And why it's being made i.e. is this a bug fix or a feature?

Yeah, no problem. @chsalgado is who I've been working with, and this is a feature to help the .NET team get a handle on target framework usage for packages across the scope of Component Detection.

@marklio If you are using Visual Studio the C# tests (what's failing) should be automatically populated in the test explorer. Otherwise you can use dotnet test .\ComponentDetection.sln

@marklio If you are using Visual Studio the C# tests (what's failing) should be automatically populated in the test explorer. Otherwise you can use dotnet test .\ComponentDetection.sln

Yeah, I can't repro these failures in VS or the command-line (I also didn't get build failures initially when I was using new C# features. I do have some local failures, but they are all in the DockerServiceTests project, which I haven't touched.

@marklio I've deconflicted this PR and rebased it on top of current master. See #365. We are keen to get this merged.


Add command and command autocompletion support for TextChatService

I will test this

Wait... Why does it need the regular prefix as well

](https://github.com/Epix-Incorporated/Adonis/assets/64012878/64d2c3be-34e4-4c59-9649-15d80ee75039">)

what

Wait... Why does it need the regular prefix as well
￼

Roblox limitation