Installation errors -- Cython and Questeval

When installing GEM-metrics and running this command:
pip install -r requirements.txt -r requirements-heavy.txt
I run into the following error:
Collecting moverscore@ git+https://github.com/AIPHES/emnlp19-moverscore.git@b9d4144 (from -r requirements-heavy.txt (line 3))
  Cloning https://github.com/AIPHES/emnlp19-moverscore.git (to revision b9d4144) to /tmp/pip-install-bd7eqreb/moverscore_3a90ae24901c4f03a4588ab7fbcf6988
  Running command git clone --filter=blob:none --quiet https://github.com/AIPHES/emnlp19-moverscore.git /tmp/pip-install-bd7eqreb/moverscore_3a90ae24901c4f03a4588ab7fbcf6988
  WARNING: Did not find branch or tag 'b9d4144', assuming revision or ref.
  Running command git checkout -q b9d4144
  Resolved https://github.com/AIPHES/emnlp19-moverscore.git to commit b9d4144
  Preparing metadata (setup.py) ...done
Collecting datasets>=1.2.1 (from -r requirements.txt (line 1))
  Obtaining dependency information for datasets>=1.2.1 from https://files.pythonhosted.org/packages/09/7e/fd4d6441a541dba61d0acb3c1fd5df53214c2e9033854e837a99dd9e0793/datasets-2.14.5-py3-none-any.whl.metadata
  Using cached datasets-2.14.5-py3-none-any.whl.metadata (19 kB)
Collecting diskcache>=5.2.1 (from -r requirements.txt (line 2))
  Obtaining dependency information for diskcache>=5.2.1 from https://files.pythonhosted.org/packages/3f/27/4570e78fc0bf5ea0ca45eb1de3818a23787af9b390c0b0a0033a1b8236f9/diskcache-5.6.3-py3-none-any.whl.metadata
  Using cached diskcache-5.6.3-py3-none-any.whl.metadata (20 kB)
Collecting nltk (from -r requirements.txt (line 3))
  Using cached nltk-3.8.1-py3-none-any.whl (1.5 MB)
Collecting numpy (from -r requirements.txt (line 4))
  Obtaining dependency information for numpy from https://files.pythonhosted.org/packages/c4/36/161e2f8110f8c49e59f6107bd6da4257d30aff9f06373d0471811f73dcc5/numpy-1.26.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata
  Using cached numpy-1.26.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (58 kB)
Collecting rouge-score (from -r requirements.txt (line 5))
  Using cached rouge_score-0.1.2-py3-none-any.whl
Collecting logzero (from -r requirements.txt (line 6))
  Using cached logzero-1.7.0-py2.py3-none-any.whl (16 kB)
Collecting pycountry (from -r requirements.txt (line 7))
  Using cached pycountry-22.3.5-py2.py3-none-any.whl
Collecting repro==0.1.4 (from -r requirements.txt (line 8))
  Using cached repro-0.1.4-py3-none-any.whl (639 kB)
Collecting sacrebleu>=2.0.0 (from -r requirements.txt (line 9))
  Using cached sacrebleu-2.3.1-py3-none-any.whl (118 kB)
Collecting sacremoses (from -r requirements.txt (line 10))
  Using cached sacremoses-0.0.53-py3-none-any.whl
Collecting bert_score (from -r requirements-heavy.txt (line 1))
  Using cached bert_score-0.3.13-py3-none-any.whl (61 kB)
Collecting questeval==0.2.4 (from -r requirements-heavy.txt (line 2))
  Using cached questeval-0.2.4.tar.gz (25 kB)
  Preparing metadata (setup.py) ...done
Collecting pyemd (from -r requirements-heavy.txt (line 4))
  Using cached pyemd-1.0.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (666 kB)
Collecting docker==5.0.0 (from repro==0.1.4->-r requirements.txt (line 8))
  Using cached docker-5.0.0-py2.py3-none-any.whl (146 kB)
Collecting overrides==3.1.0 (from repro==0.1.4->-r requirements.txt (line 8))
  Using cached overrides-3.1.0-py3-none-any.whl
Collecting pytest==6.2.4 (from repro==0.1.4->-r requirements.txt (line 8))
  Using cached pytest-6.2.4-py3-none-any.whl (280 kB)
Collecting six==1.16.0 (from repro==0.1.4->-r requirements.txt (line 8))
  Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Collecting transformers==4.8.1 (from questeval==0.2.4->-r requirements-heavy.txt (line 2))
  Using cached transformers-4.8.1-py3-none-any.whl (2.5 MB)
Collecting sentencepiece==0.1.95 (from questeval==0.2.4->-r requirements-heavy.txt (line 2))
  Using cached sentencepiece-0.1.95.tar.gz (508 kB)
  Preparing metadata (setup.py) ...done
Collecting datasets>=1.2.1 (from -r requirements.txt (line 1))
  Using cached datasets-1.7.0-py3-none-any.whl (234 kB)
Collecting bert_score (from -r requirements-heavy.txt (line 1))
  Using cached bert_score-0.3.9-py3-none-any.whl (59 kB)
Collecting Unidecode==1.2.0 (from questeval==0.2.4->-r requirements-heavy.txt (line 2))
  Using cached Unidecode-1.2.0-py2.py3-none-any.whl (241 kB)
Collecting spacy==3.0.6 (from questeval==0.2.4->-r requirements-heavy.txt (line 2))
  Using cached spacy-3.0.6.tar.gz (7.1 MB)
  Installing build dependencies ...done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [167 lines of output]

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
          int length

      cdef class Vocab:
          cdef Pool mem
          cpdef readonly StringStore strings
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:28:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...

      cdef class Vocab:
          cdef Pool mem
          cpdef readonly StringStore strings
          cpdef public Morphology morphology
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:29:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...

      cdef class Vocab:
          cdef Pool mem
          cpdef readonly StringStore strings
          cpdef public Morphology morphology
          cpdef public object vectors
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:30:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
      cdef class Vocab:
          cdef Pool mem
          cpdef readonly StringStore strings
          cpdef public Morphology morphology
          cpdef public object vectors
          cpdef public object _lookups
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:31:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
          cdef Pool mem
          cpdef readonly StringStore strings
          cpdef public Morphology morphology
          cpdef public object vectors
          cpdef public object _lookups
          cpdef public object writing_system
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:32:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
          cpdef readonly StringStore strings
          cpdef public Morphology morphology
          cpdef public object vectors
          cpdef public object _lookups
          cpdef public object writing_system
          cpdef public object get_noun_chunks
                ^
      ------------------------------------------------------------

      spacy/vocab.pxd:33:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
          cdef float prior_prob

      cdef class KnowledgeBase:
          cdef Pool mem
          cpdef readonly Vocab vocab
                ^
      ------------------------------------------------------------

      spacy/kb.pxd:31:10: Variables cannot be declared with 'cpdef'. Use 'cdef' instead.
      Copied /tmp/pip-install-bd7eqreb/spacy_931479f3684d4722903c62eb45cd7226/setup.cfg -> /tmp/pip-install-bd7eqreb/spacy_931479f3684d4722903c62eb45cd7226/spacy/tests/package
      Copied /tmp/pip-install-bd7eqreb/spacy_931479f3684d4722903c62eb45cd7226/pyproject.toml -> /tmp/pip-install-bd7eqreb/spacy_931479f3684d4722903c62eb45cd7226/spacy/tests/package
      Cythonizing sources
      Compiling spacy/training/example.pyx because it changed.
      Compiling spacy/parts_of_speech.pyx because it changed.
      Compiling spacy/strings.pyx because it changed.
      Compiling spacy/lexeme.pyx because it changed.
      Compiling spacy/vocab.pyx because it changed.
      Compiling spacy/attrs.pyx because it changed.
      Compiling spacy/kb.pyx because it changed.
      Compiling spacy/ml/parser_model.pyx because it changed.
      Compiling spacy/morphology.pyx because it changed.
      Compiling spacy/pipeline/dep_parser.pyx because it changed.
      Compiling spacy/pipeline/morphologizer.pyx because it changed.
      Compiling spacy/pipeline/multitask.pyx because it changed.
      Compiling spacy/pipeline/ner.pyx because it changed.
      Compiling spacy/pipeline/pipe.pyx because it changed.
      Compiling spacy/pipeline/trainable_pipe.pyx because it changed.
      Compiling spacy/pipeline/sentencizer.pyx because it changed.
      Compiling spacy/pipeline/senter.pyx because it changed.
      Compiling spacy/pipeline/tagger.pyx because it changed.
      Compiling spacy/pipeline/transition_parser.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/arc_eager.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/ner.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/nonproj.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/_state.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/stateclass.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/transition_system.pyx because it changed.
      Compiling spacy/pipeline/_parser_internals/_beam_utils.pyx because it changed.
      Compiling spacy/tokenizer.pyx because it changed.
      Compiling spacy/training/align.pyx because it changed.
      Compiling spacy/training/gold_io.pyx because it changed.
      Compiling spacy/tokens/doc.pyx because it changed.
      Compiling spacy/tokens/span.pyx because it changed.
      Compiling spacy/tokens/token.pyx because it changed.
      Compiling spacy/tokens/span_group.pyx because it changed.
      Compiling spacy/tokens/graph.pyx because it changed.
      Compiling spacy/tokens/morphanalysis.pyx because it changed.
      Compiling spacy/tokens/_retokenize.pyx because it changed.
      Compiling spacy/matcher/matcher.pyx because it changed.
      Compiling spacy/matcher/phrasematcher.pyx because it changed.
      Compiling spacy/matcher/dependencymatcher.pyx because it changed.
      Compiling spacy/symbols.pyx because it changed.
      Compiling spacy/vectors.pyx because it changed.
      [ 1/41] Cythonizing spacy/attrs.pyx
      [ 2/41] Cythonizing spacy/kb.pyx
      Traceback (most recent call last):
        File "/users/PAS1647/ashleylewis/.conda/envs/GEM/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/users/PAS1647/ashleylewis/.conda/envs/GEM/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/users/PAS1647/ashleylewis/.conda/envs/GEM/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-6h6ptrev/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 355, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-6h6ptrev/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 325, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-6h6ptrev/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 341, in run_setup
          exec(code, locals())
        File "<string>", line 224, in <module>
        File "<string>", line 211, in setup_package
        File "/tmp/pip-build-env-6h6ptrev/overlay/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1154, in cythonize
          cythonize_one(*args)
        File "/tmp/pip-build-env-6h6ptrev/overlay/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1321, in cythonize_one
          raise CompileError(None, pyx_file)
      Cython.Compiler.Errors.CompileError: spacy/kb.pyx
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess,

I'm able to run the regular metrics by installing the missing dependencies first, but when I try to run heavy metrics I run into:
ModuleNotFoundError: No module named 'questeval'
When I try to install questeval, I get the same error as above (Cython.Compiler.Errors.CompileError). I've been googling around and haven't been able to find any solutions.

Was able to fix this problem by downgrading my python version to 3.8


Accessibility for search

Some things need to be verified to ensure that the search is as accessible as possible.

Exactly how much of a bad idea is it to set autofocus in the search box when you open the search overlay
Reach tab component set focus on the tab itself before focusing on the individual hits. Can we improve this somehow, without sacrificing a11y (aka find out why they do this)
General test with screen reader. How does the live search work and how easy/hard is navigating the search results. This would be a great discussion topic with an a11y expert. We can only assume that live regions will be too much, but it might be that we need it?

@wenche to update this card after talking to expert

We need an expert opinion on this one

Exactly how much of a bad idea is it to set autofocus in the search box when you open the search overlay-> This is not recommended, so let's just keep it as it is until somebody can verify if this is OK for screen readers
2)Reach tab component set focus on the tab itself before focusing on the individual hits. Can we improve this somehow, without sacrificing a11y (aka find out why they do this) -> Let's trust Reach on this one. It seems like it will provide some useful summary information for all the tabs when I test with a screen reader
3)General test with screen reader. How does the live search work and how easy/hard is navigating the search results. This would be a great discussion topic with an a11y expert. We can only assume that live regions will be too much, but it might be that we need it? -> I have done some testing with aria live region for both the whole search result component and just the tab header. In theory, especially the latter approach should be useful. But it seems like it has some issues with the time out for the search input field since the screen reader will read out what you type in the search input field and not the content of the live region most times. We should look more into this with an expert. It you type a search and tab to the tab component, the information is pretty useful. The worst part is probably to understand that this is a live search.


Replace instrumentation with opencensus in the releasing instructions.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #432   +/-   ##
=========================================
  Coverage     90.18%   90.18%
  Complexity      582      582
=========================================
  Files           100      100
  Lines          2058     2058
  Branches        209      209
=========================================
  Hits           1856     1856
  Misses          148      148
  Partials         54       54

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 26a3ff3...d161450. Read the comment docs.


flow seems to be confused by function declaration

Below are two examples, first one works as expected while the second (I believe incorrectly) does not:
/* @flow */

export const run = (task:?Function) => {
  if (task != null)  {
    task()
  } else {
    throw TypeError("task must be a function")
  }
}

The following example fails with "function call Function cannot be called on possibly null value" error.
/* @flow */

const run = (task:?Function) => {
  if (task != null)  {
    const run = () => {
      return task()
    }

    run()
  } else {
    throw TypeError("task must be a function")
  }
}

Yea, this is a KP (see http://flowtype.org/docs/dynamic-type-tests.html#caveats)
It's because, at the moment, having a function call inside a refinement will invalidate Flow's certainty about that refinement (Flow isn't currently able to follow the function call and confirm that it doesn't change the value of task back to null by way of some other closure).
For now you can work around this for now by saving the refined variable to a different local var first inside the toplevel of the environment like this: https://gist.github.com/jeffmo/ca3323170edb7bcbd0a7

Yea, this is a KP (see http://flowtype.org/docs/dynamic-type-tests.html#caveats)

It's because, at the moment, having a function call inside a refinement will invalidate Flow's certainty about that refinement (Flow isn't currently able to follow the function call and confirm that it doesn't change the value of task back to null by way of some other closure).

Well in the mentioned caveat it makes sense because passed object can be mutated by an outer function. But the provided example outer function can't possibly change argument that is function to null or undefined so I think flow could handle that case without necessarily follow function calls.
@jeffmo Presume flow assumes that any function call may have unpredictable side effect and type can can change from anything to anything which is not great but in that case I would argue that following example should work fine (which is closer to my actual code in fact):
/* @flow */

const run = (task:?Function) => {
  const safeTask = task;
  if (safeTask != null)  {
    const run = () => {
      return safeTask()
    }

    run()
  } else {
    throw TypeError("task must be a function")
  }
}

In this case const safeTask  = task  should be refined to Function and invoking a function should have no effect given that const is used and there for it should not be possible to change safeTask value.

@Gozala The caveats section calls this out specifically.

Flow does not currently perform mutation analysis, so in some cases, Flow will throw away a refinement that is always safe to keep. As more analyses are implemented, Flow will hopefully come to understand the following examples are sound.

You're also right that const is more low-hanging fruit when it comes to keeping refinements. Investigating whether it is worthwhile to special-case const is on my radar already.


support BDM

hello @ln-nicolas, do you have any plans to support DBM within this library? I'm looking to get series data from INSEE. thanks

Hello @mxdev88 what do you mean by DBM ?

The lib supports siret and sirene API endpoints from the Sirene API. INSEE offers 3 other APIs in their catalog.
I'm interested to interact with the BDM one.
(Sorry I mispelled in my original post, I meant BDM. I've corrected).

The lib supports siret and sirene API endpoints from the Sirene API. INSEE offers 3 other APIs in their catalog.
I'm interested to interact with the BDM one.
(Sorry I mispelled in my original post, I meant BDM. I've corrected).

It's a great idea, but I don't plan to support this endpoint for the while. Fell free to create a pull-request :)

After looking into it further, I may be able to use one of these alternatives: sdmx1 or pandasdmx.


support combining function matchers with other options

fixes #211 @shepmaster

Changes Unknown when pulling 9703fdbca59d8b414fab97f3c3e9435b385921ab on rhys/function-matcher-versatility into ** on master**.

Thanks!


Update solr dependency.

Note $subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7929507761


Typo in example page name Introducion and Custom Bridges

Expected behaviour
Should be Introduction and Custom Bridges.
Screenshots

🇪🇸 🇪🇸 🇪🇸 🇪🇸


Update package.json

The postinstall script has been rewritten for cross-platform compatibility. Linting and formatting commands in the lint-staged section have been simplified. Versions for husky, lint-staged, and pinst have been added to the devDependencies section for better manageability. Dependencies have been updated to the latest versions. React dependencies have been added to the dependencies section, as it is used in the project.
What changed? Why?
Notes to reviewers
How has it been tested?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Reply doesn't seem to really be a reply

Hey Fabio thanks for such an awesome workflow! I'm not sure if this is possible, but I'd really like for my quick reply from alfred to be treated as a reply instead of a new message (which is how it appears to be now) Please see the attached screenshot

Why did you close this one?


Error while deploying Electric Sync Service to DigitalOcean

I followed the instructions in the docs for deploying to DigitialOcean and continue to receive this error:
Failed to start child Electric.Replication.PostgresConnectorMng:
[2024-07-01 19:03:14] ▓   {{:case_clause, :undefined}, [{Electric.Replication.PostgresConnectorMng, :get_verify_peer_opts, 0, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 358]}, {Electric.Replication.PostgresConnectorMng, :maybe_verify_peer, 1, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 350]}, {Keyword, :update!, 4, [file: ~c"lib/keyword.ex", line: 1144]}, {Keyword, :update!, 4, [file: ~c"lib/keyword.ex", line: 1148]}, {Electric.Replication.PostgresConnectorMng, :update_connector_config, 2, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 119]}, {Electri

same issue here

Hi @mokerson1108, @mikkimichaelis.
Which version of the sync service are you using?

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/
You can read more about why we made the decision at https://next.electric-sql.com/about
We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.
We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and report this issue!


Update CommonAdminBase64.txt

The passwords should be base64 encoded, and not plaintext keypairs like those

Thanks for the suggestion @Chequel197
however, they don't appear to be base64 encoded like the filename/others?

$ echo -n guest | base64
Z3Vlc3Q=

This value is already in there.
As a result, closing


Document WLM settings for Zowe

One customer using a commercial IBM product that prereqs Zowe have asked me about recommended WLM settings.
I suggest this is documented in the installation guide.
See this thread in #zowe-user: https://openmainframeproject.slack.com/archives/CC08782AG/p1586942924019200

From the thread by Onno Van den Troost:

For the main Zowe server, ZWESVSTC, the work here is running in USS child processes, so this is a OMVS workload in WLM.
A lot of this work can be compared to users working in TSO, so here too you should specify a high-performance, one-period velocity goal.


feat(pisa-proxy, protocol): add get_inner for local_stream to get underlying io

Signed-off-by: xuanyuan300<EMAIL_ADDRESS>
What is changed and how it works?
What's Changed:
Add get_inner for local_stream to get underlying io. Also, this pr is optimized for the client's LocalStream.

I think stream.rs in different directory can be merged into the same file.The type of LocaStream wrapping is TcpStream.The different of them is that the Server and Client is build in different method.Server is build by connect(), Client is build by accept().And they all implements AsyncWrite and AsyncRead trait.

The difference is upgrade tls connect, so split to Server and Client. But the same code can be generated by the macro in the future.


After I run the xcodesnippet install xxx command,the file ~/Library/Developer/Xcode/UserData/CodeSnippets/xxx does not exist.

Like this.

Duplicate of #1.


Iox #653 fix roudi crash when replying to dead app

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Branch follows the naming format (iox-#123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit messages are signed (git commit -s)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[x] Assign PR to reviewer

Notes for Reviewer

RouDi should never crash by wrong user input
Combine monitored and un-monitored case and do not reply to dead app
I've tested it manually as described in #653, as there is currently no framework available for such tests. I did intentionally not add any SWE.6 tests for it (@dkroenke FYI)

Please also test it manually

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #653

Please add some kind of test!
Or how do you make sure that the crash is fixed?

Please have a look at the "Notes for Reviewer"

Please add some kind of test!
Or how do you make sure that the crash is fixed?

I think currently it can only be done manually by

starting iox-roudi
starting ./iox-ex-subscriber
executing kill -9 [iox-ex-subscriber pid]& ./iox-ex-subscriber


How to insert new network architecture?

First of all thank you for your great work.
I have experimented net architecture using your facenet.
I investigated whether it converges using resnet50 provided by slim.
However, while the training error continues to decrease, the lfw validation error does not decrease at 0.65.
Could you give me advice?

Did you solve this issue?
If so, what results did you get by using ResNet50 as backbone network?


Fix wrongly marking my emulator as having pirate apps installed

Thanks for this! I've learning something new here. In return, I thought I'd share some fixes I had to do.
Shows my app not having root installed but having pirated apps installed
anyway. In the pirated apps list, it showed com.android.vending.billing.InAppBillingService
twice. This is because the contains slice check was incorrect.
Example: I had some app with package name as just android and another
with package name com.android.vending so check previously was if com.android.vending.billing.InAppBillingService
contained android. This should be reversed.

Of course, lucky patcher's package name(s) begin with com.android.vending.billing.InAppBillingService and then they have something else like LOCK, COIN, etc.
This can be efficiently fixed by adding to the package definitions a dot at the end of the original package so that also that condition will need to be matched.
Also, thanks for the fixes and improvements. I'll merge your pull request and create a new one to close this issue.


add differentiable eigenvalue calculation for small matrices

Codecov Report

Merging #952 (814d7ed) into develop (afaefcc) will increase coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Current head 814d7ed differs from pull request most recent head 04f2efa. Consider uploading reports for the commit 04f2efa to get more accurate results

@@             Coverage Diff             @@
##           develop     #952      +/-   ##
===========================================
+ Coverage    89.19%   89.23%   +0.03%
===========================================
  Files          141      141
  Lines        10517    10554      +37
===========================================
+ Hits          9381     9418      +37
  Misses        1136     1136

Impacted Files
Coverage Δ

...serac/numerics/functional/tests/test_tensor_ad.cpp
100.00% <100.00%> (ø)

...umerics/functional/tuple_tensor_dual_functions.hpp
97.53% <100.00%> (+0.18%)
:arrow_up:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

lgtm

@jamiebramwell
thanks for doing the make style for me, I don't have a machine with the right version of clang-format any more, and apparently clang-format-11 gives slightly different results

@jamiebramwell thanks for doing the make style for me, I don't have a machine with the right version of clang-format any more, and apparently clang-format-11 gives slightly different results

Yeah, the style thing is annoying. We should probably just do a batch clang update on all of our platforms.


Update UML Diagrams

Need to update at least 1 UML Diagram

[ ] @ys112
[x] @lihaoquan
[ ] @ChrisHo1341
[ ] @YuvBindal
[ ] @xinen26

Pending with PR #62


Store refreshed token

Summary
Store oauth refreshed token
Ticket Link
None

Codecov Report

Merging #147 (a0dc305) into master (0aa20ab) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a0dc305 differs from pull request most recent head d1ef1f9. Consider uploading reports for the commit d1ef1f9 to get more accurate results

@@           Coverage Diff           @@
##           master     #147   +/-   ##
=======================================
  Coverage   20.60%   20.60%
=======================================
  Files          28       28
  Lines        1291     1291
=======================================
  Hits          266      266
  Misses       1007     1007
  Partials       18       18

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 0aa20ab...d1ef1f9. Read the comment docs.


[BUG]: Copy-PnpFile not working from Team Site Library to O365 Group Library - Returns nothing, but doesn't copy.

Reporting an Issue or Missing Feature
When I use Copy-PnpFile to copy within a Doc library it works fine, but when trying to copy from the Root Team site to a Doc library on another site the file is not copied and no error is returned.
I've tried various combinations of including the leading "/" in the sourceUrl and specifying or omitting the -connection parameter in the command.
Not using the -connection parameter or specifying the target site connection returns no error, but the file is not copied.
Omitting the leading "/" in the SourceUrl causes the error "The system cannot find the file specified"
Specifying the source site connection causes a return value with the message: The system cannot find the path specified. (Exception from HRESULT: 0x80070003)
Expected behavior
Expect the file to copy
Actual behavior
Nothing
Steps to reproduce behavior
$AConnection = Connect-PnPOnline -Url "https://example.sharepoint.com/sites/a" -Credentials $Credential -ReturnConnection
$AContext = Get-PnPContext

$BConnection = Connect-PnPOnline -Url "https://example.sharepoint.com/" -Credentials $Credential -ReturnConnection
$BContext = Get-PnPContext

Copy-PnPFile -SourceUrl "/Resources/XXX/foo.docx" -TargetUrl "/sites/a/YYY/bar/targetfolder" -Force

What is the version of the Cmdlet module you are running?
Copy-PnpFile version 1.8.0
Which operating system/environment are you running PnP PowerShell on?

[ X] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

I was able to do this with 1.8.0. Here are the steps I used:
Connect-PnPOnline -Url https://m365x995492.sharepoint.com/
# Copy the file to another doclib in the root, just as a test
Copy-PnPFile -SourceUrl "/Shared Documents/Does this work.docx" -TargetUrl "/TestLib"
# Copy to a different site
Copy-PnPFile -SourceUrl "/Shared Documents/Does this work.docx" -TargetUrl "/sites/ContosoWorks/Shared Documents"

Here's how it looks:

Hi,
If I try like that (without the second connect statement to the target site) I get Access Denied. I can use the same account to copy documents within the library in sites/a

@jeremyhagan

trying to copy from the Root Team site to a Doc library on another site

Are you just trying to copy a document from a document library in Site A to a document library in Site B?

Yes, except that the source site is the default SP Team Site. I will try some more different combinations to see if I can isolate this further.

I was able to do this with 1.8.0. Here are the steps I used:
Connect-PnPOnline -Url https://m365x995492.sharepoint.com/
# Copy the file to another doclib in the root, just as a test
Copy-PnPFile -SourceUrl "/Shared Documents/Does this work.docx" -TargetUrl "/TestLib"
# Copy to a different site
Copy-PnPFile -SourceUrl "/Shared Documents/Does this work.docx" -TargetUrl "/sites/ContosoWorks/Shared Documents"

Here's how it looks:

@ToddKlindt , this is what happens to me:
Connect-PnPOnline -url http://mycompany.sharepoint.com/
Copy-PnPFile -SourceUrl "/Shared Documents/Jeremy Test.docx" -TargetUrl "/Resources"

Shared Documents/Jeremy Test.docx Microsoft.SharePoint.SPException: This operation is not supported. ---> System.Runtime.InteropServices.COMException: This operation is not supported

However, when I do the following it works:
Connect-PnPOnline -url http://mycompany.sharepoint.com/sites/myo365group/
Copy-PnPFile -SourceUrl "/sites/myo365group/MyLibrary/SubfolderA/Test.docx"  -TargetUrl "/sites/myo365group/MyLibrary/SubfolderB"


Travis-CI build failures

https://travis-ci.org/ethereumproject/etherwallet/builds/153020134
Is this an acceptable failure?  Should we get the build back to working status?

Aside from PR #6, we should look into getting builds to complete

I believe this is fixed now. Can someone verify?


Thorium lag/freeze issue

System Details

OS: Arch Linux
Kernel and Environment: 6.5.7-arch1-1, i3wm
Hardware: Intel i5-12500H, Intel Alder Lake-P GT2 [Iris Xe Graphics]
Driver: intel-media-driver
Thorium Version 117.0.5938.157-1

Problem
After 1-2 hours of continuous use, the Thorium browser begins to lag significantly. When I try to close or terminate the browser, the entire system freezes, with only the cursor remaining responsive. This requires a manual system restart using the power button.
Additional Notes

The issue appears to primarily occur during video playback on YouTube/Twitch; haven't experienced it while only browsing without any video playback.
Haven't observed this behaviour in other browsers like Brave. Thorium is the only browser where hardware accelerated decoding works, so that could be the potential issue If I had to guess.

can confirm such a behavior although sometimes waiting for a while can undone the hanging (info: arch | xfce | intel cpu | nvidia discrete gpu disabled via udev rules )

System Details

Version	23H2

OS build	22631.2506

Problem
Browser is randomly freezing, have to close the browser and start it again, but it again freezes randomly.

Unfortunately same issue on Debian 12 with Thorium version M117.0.5938.157. The longer the browser is opened, the longer the freeze (up to the point that it's better to restart whole system than to wait)


Enhance Garment Shipping

Codecov Report

Merging #12 (f2a4cf8) into dev (74015e4) will decrease coverage by 0.01%.
The diff coverage is 89.28%.

@@            Coverage Diff             @@
##              dev      #12      +/-   ##
==========================================
- Coverage   92.75%   92.73%   -0.02%
==========================================
  Files         809      809
  Lines       78666    78735      +69
  Branches     4438     4440       +2
==========================================
+ Hits        72965    73015      +50
- Misses       5043     5061      +18
- Partials      658      659       +1

Impacted Files
Coverage Δ

...hipping/CoverLetter/GarmentCoverLetterViewModel.cs
96.93% <ø> (+0.86%)
:arrow_up:

...ructure/GarmentShippingCostStructurePdfTemplate.cs
0.00% <0.00%> (ø)

...calCoverLetter/GarmentLocalCoverLetterViewModel.cs
97.61% <ø> (+1.61%)
:arrow_up:

...ion/ToBeRefactored/Utilities/Base64ImageStrings.cs
100.00% <ø> (ø)

...tshipping/ShippingNote/GarmentShippingNoteModel.cs
92.92% <30.00%> (-7.08%)
:arrow_down:

...ping/GarmentPackingList/GarmentPackingListModel.cs
96.55% <47.36%> (-2.25%)
:arrow_down:

...ingList/GarmentPackingListWithHeaderPdfTemplate.cs
80.56% <50.00%> (+0.10%)
:arrow_up:

...mentShippingInvoice/GarmentShippingInvoiceModel.cs
98.56% <58.33%> (-1.44%)
:arrow_down:

...pping/CoverLetter/GarmentCoverLetterPdfTemplate.cs
99.07% <100.00%> (-0.01%)
:arrow_down:

...SalesDO/GarmentShippingExportSalesDOPdfTemplate.cs
97.40% <100.00%> (-0.06%)
:arrow_down:

... and 30 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Convert HiFi3D++ UV texture to FLAME UV texture

Thanks for the script to modify a FLAME mesh to enable reading from a UV texture made for HiFi3D++.
I'm trying to convert a UV texture made for HiFi3D++ into a UV texture for FLAME, such that I can use it on the original/unmodified FLAME mesh. I need to basically copy the pixels from one UV texture into the corresponding location in the new UV texture. Any tips on how to get this correspondence?

I figured it out and I'm documenting it here. The process is as follows:

Take a FLAME mesh and convert it to use a HiFi3D UV map using the script provided in this repo.
Sample a large number of points on the surface of the mesh.
For each point, get the coordinates on the HiFi3D UV map that it corresponds to (obtained from the modified .obj).
For each point, get the coordinates on the FLAME UV map that it corresponds to (obtained from the original .obj).
Copy the pixel values from source to destination UV coordinates.

Code
def sample_points_from_meshes(
    meshes,
    num_samples: int = 10000,
):
    """
    Adapted from https://github.com/facebookresearch/pytorch3d/blob/v0.6.0/pytorch3d/ops/sample_points_from_meshes.py
    """
    verts = meshes.verts_packed()
    if not torch.isfinite(verts).all():
        raise ValueError("Meshes contain nan or inf.")

    faces = meshes.faces_packed()
    mesh_to_face = meshes.mesh_to_faces_packed_first_idx()
    num_meshes = len(meshes)
    num_valid_meshes = torch.sum(meshes.valid)  # Non empty meshes.

    # Initialize samples tensor with fill value 0 for empty meshes.
    samples = torch.zeros((num_meshes, num_samples, 3), device=meshes.device)

    # Only compute samples for non empty meshes
    with torch.no_grad():
        areas, _ = mesh_face_areas_normals(verts, faces)  # Face areas can be zero.
        max_faces = meshes.num_faces_per_mesh().max().item()
        areas_padded = packed_to_padded(
            areas, mesh_to_face[meshes.valid], max_faces
        )  # (N, F)

        sample_face_idxs = areas_padded.multinomial(
            num_samples, replacement=True
        )  # (N, num_samples)
        sample_face_idxs += mesh_to_face[meshes.valid].view(num_valid_meshes, 1)

    # Get the vertex coordinates of the sampled faces.
    face_verts = verts[faces]
    v0, v1, v2 = face_verts[:, 0], face_verts[:, 1], face_verts[:, 2]

    # Randomly generate barycentric coords.
    w0, w1, w2 = _rand_barycentric_coords(
        num_valid_meshes, num_samples, verts.dtype, verts.device
    )

    # Use the barycentric coords to get a point on each sampled face.
    a = v0[sample_face_idxs]  # (N, num_samples, 3)
    b = v1[sample_face_idxs]
    c = v2[sample_face_idxs]
    samples[meshes.valid] = w0[:, :, None] * a + w1[:, :, None] * b + w2[:, :, None] * c

    # fragment data are of shape NxHxWxK. Here H=S, W=1 & K=1.
    pix_to_face = sample_face_idxs.view(len(meshes), num_samples, 1, 1)  # NxSx1x1
    bary = torch.stack((w0, w1, w2), dim=2).unsqueeze(2).unsqueeze(2)  # NxSx1x1x3
    # zbuf and dists are not used in `sample_textures` so we initialize them with dummy
    dummy = torch.zeros(
        (len(meshes), num_samples, 1, 1), device=meshes.device, dtype=torch.float32
    )  # NxSx1x1
    fragments = MeshFragments(
        pix_to_face=pix_to_face, zbuf=dummy, bary_coords=bary, dists=dummy
    )

    # return
    return samples, fragments

def _rand_barycentric_coords(
    size1, size2, dtype: torch.dtype, device: torch.device
):
    uv = torch.rand(2, size1, size2, dtype=dtype, device=device)
    u, v = uv[0], uv[1]
    u_sqrt = u.sqrt()
    w0 = 1.0 - u_sqrt
    w1 = u_sqrt * (1.0 - v)
    w2 = u_sqrt * v
    return w0, w1, w2

device = 'cpu'
flame_mesh = pytorch3d.io.load_objs_as_meshes(['flame_mesh.obj'], load_textures=True, device=device)
hifi3d_mesh = pytorch3d.io.load_objs_as_meshes(['flame_mesh_w_hifi3d_uv.obj'], load_textures=True, device=device)
N,H,W,C = flame_mesh.textures._maps_padded.shape
u_extra = torch.linspace(0, 1, W)[:, None].expand(1, H, W, 1).to(device)
v_extra = torch.linspace(1, 0, H).flip(0)[:, None, None].expand(1, H, W, 1).to(device)
flame_mesh.textures._maps_padded = torch.cat([flame_mesh.textures._maps_padded, u_extra, v_extra], dim=3)
hifi3d_mesh.textures._maps_padded = torch.cat([hifi3d_mesh.textures._maps_padded, u_extra, v_extra], dim=3)

n = 500_000_000
pts, fragments = sample_points_from_meshes(flame_mesh, n)

flame_tex_uv = flame_mesh.sample_textures(fragments)
hifi3d_tex_uv = hifi3d_mesh.sample_textures(fragments)

flame_sampled_uv_coords = flame_tex_uv[..., 3:].squeeze()
flame_u_coords = (torch.clamp(flame_sampled_uv_coords[:, 0], 0, 1) * (W-1)).to(int)
flame_v_coords = (torch.clamp(flame_sampled_uv_coords[:, 1], 0, 1) * (H-1)).to(int)

hifi3d_sampled_uv_coords = hifi3d_tex_uv[..., 3:].squeeze()
hifi3d_u_coords = (torch.clamp(hifi3d_sampled_uv_coords[:, 0], 0, 1) * (W-1)).to(int)
hifi3d_v_coords = (torch.clamp(hifi3d_sampled_uv_coords[:, 1], 0, 1) * (H-1)).to(int)
hifi3d_texture_image=hifi3d_mesh.textures.maps_padded()[..., :3].squeeze()
hifi3d_values = hifi3d_texture_image[hifi3d_v_coords, hifi3d_u_coords]

flame_texture_image = torch.zeros_like(hifi3d_texture_image)
flame_texture_image[flame_v_coords, flame_u_coords] = hifi3d_values

plt.imshow(flame_texture_image)
plt.axis('off');

plt.imshow(hifi3d_texture_image)
plt.axis('off');

Result
This UV map can be applied onto a HiFi3D mesh or the modified FLAME mesh:

The code produces this UV map which is the same texture but can be applied directly onto an unmodified FLAME mesh (only the eye region is missing):

@akasharidas Thank you so much!
May I ask, should I change the size of uv of "flame_mesh_w_hifi3d_uv.obj" to 256?
Actually, I change the uv size, but got a wrong result. Could you please help me?

@akasharidas Thank you so much! May I ask, should I change the size of uv of "flame_mesh_w_hifi3d_uv.obj" to 256? Actually, I change the uv size, but got a wrong result. Could you please help me?

Hi @daladulu , @akasharidas , @csbhr ,
can you share how to get the texture for FLAME ?
Thanks a lot !

Have you solved it? Can you share the code？


Who Maestro counters:

He destroys Hibana's shock pellets and Thermite's, I'd say this is a hard counter.
He destroys Twitch's drone if he can shoot it (never really done it, but it works on normal drones and I assume it works on her's too). I'd say hard counter.
He can shoot stationary operators, like say Maverick as he's blowtorching, preventing him from really using his gadget. Soft counter i guess. Perhaps even for operators who pull out a gadget weapon (like zofia, capitao, nomad) and cant destroy the Evil Eye while using those gadgets. Thermite even, since he has to pull out and plant his gadget.
Minor counter against shield ops, forcing them to turn around, or expose themselves to shoot the cam, or Maestro aims for the parts not covered by the shield front-side.
Minor counter to glaz since his Evil Eye can see through smoke which glaz relies on to use his gadget

Actually, i have destroyed twitch's drones.


pull the kapp-controller continer image with tag equal to release

Is there some way i can pull the kapp-controller container image for a specific release say 0.48.1 or 0.48.2 etc. I see images only with some rand tags and in our company we ahve policy which allows images to be pulled if only tagged with release versions. are the container images for releases hosted somewhere from which it can be pulled.

@vivekskrishna We are using ghcr to host the images (can be found under kapp-controller packages). You should be able to use the sha instead of tags.


LB config should be able to specify accept-proxy and send-proxy

currently the source IP gets lost. Source ip is forwarded for http(s) using X-Forwarded-For (option forwardfor)
For http(s) bind *:<port> accept-proxy and sever [..] send-proxy should be possible
In case of TCP this should be more configurable, separately for bind and server.

Fixed in https://github.com/kontena/kontena-loadbalancer/pull/38


Configuration of Slanger for Cloud Load Balancing

The docs state:

Multiple Slanger nodes can sit behind a load balancer with no special configuration. In essence it was designed to be very easy to scale.

However it is not clear how this should be configured on cloud load balancers such as those provided by Digital Ocean or Rackspace. Are there are any settings that need to be applied to the individual slanger nodes so that they are aware of each other? Does redis need to be configured in a cluster, or can two independent installations (as executed per the docs) be performed with some additional config? Or is it expected that a separate Redis instance is used altogether - where all slanger nodes connect to that Redis store?

I have the same question about the Redis sharing in the cluster. Do we need different Redis instance for each server, or just one Redis cluster(or one Redis instance) for all Slanger instance?

@terryvangard  I found there is a related question about Redis sharing in cluster maybe helpful. #173

@terryvangard the post referenced by @wahyd4 looks relevant.

Are there are any settings that need to be applied to the individual slanger nodes so that they are aware of each other?

Effectively you can load balance connections to your Slanger servers, but your Slanger servers will need to be configured to point to a particular Redis instance. You can do this with the -r switch when you start Slanger.
So each server that you set up with Slanger, you will need to provide it with the server that you consider to be master for Redis (this could also be one of your other boxes running Slanger).
Your master Redis server will need to be configured to allow external connections - you can do this by editing your redis.conf file and modifying this value:

Change bind <IP_ADDRESS> to either be #bind <IP_ADDRESS> or specify the network interface that your other server will connect over

Does redis need to be configured in a cluster, or can two independent installations (as executed per the docs) be performed with some additional config?

If you're load balancing, you may want to define what type of load balance mode you are operating in. If you are setting up each machine to have both Slanger and Redis (as per the docs) but do not use Redis clustering - you can not load balance in active/active mode. Note that this limitation applies even if you can get Redis running in master/slave mode (using the Redis slaveof directive) - as you will need every instance of Slanger to connect to your master redis instance in this type of load balanced set-up. This is because you can not write to any of your Redis slaves and expect other Redis nodes to know about it. For a full active/active load balanced set up of Slanger with Redis you will need to either separate out Redis as a separate service altogether, or if you are bundling redis and slanger on the same machine - you will need to follow the guidelines in Redis Clustering on the minimum number of nodes required.
As a side note, in Slanger version 0.4.4 there seems to be an issue that if your master Redis instance goes down, that all Slanger instances will continue to accept and maintain connections but no messages are communicated. Will investigate if newer versions of Slanger have the same issue, and if so raise an issue to be addressed. But this type of behaviour may have an impact on your load balancing strategy.


Update README.md

add Edge to the Readme note that "Open links in new tab" should be disabled.

Thank you for your PR, but now I think it is better to get a URL from the green text below when "Open links in new tab" is turned on although it may be truncated.

Closed in favor of ea9238886d208d355953e9fc14b223a28badaa9b


Tour 9 Update - new course

They've removed Juniper Point from Tour 9 and replaced it with Acacia Reserve
I don't see anything on that one, it looks to be a new course.

On T4 and T11 also when the time comes, nothing from PD yet.

Thanks. Closed in 9c034a60a9e1eafcc876bae701a3ca03626ae6b4.


Support Pimple 3.x without Silex

@breenie What do you think of supporting Pimple-only installation when Silex is not present? I think this is the recommended way as it's more portable and could be used in projects using Doctrine and Pimple without using Silex. That's why the service provider was moved from Silex to Pimple and also why Silex now publish their providers as a separate Composer package.
This means not using the BootableProviderInterface. We could move everything to register() and make things more lazy. Getting the configuration object would be lazy through Pimple. The only obstacle is adding the commands - if we add them in the register method then the configuration would be retrieved from the container before any option overriding from the user has been applied. So we could still add the commands in the boot method, but if you are not using Silex you'd just need to add the commands yourself or even better - call the boot method on the provider yourself and it'd add the commands for you.
How do you feel about that? Do you want me to make a PR with the described suggestions so we could discuss a concrete implementation?

@hkdobrev I've never used Pimple on it's own before so I'd have to believe what you say. Is there any documentation anywhere or examples where providers are built to support Pimple with and without Silex?
Wouldn't registering the commands be a performance problem? Especially as it would be done every request and you're only going to want the migrations helper for console commands.
Shout me down if I've wrong but wouldn't it be easier and possibly better to have two service providers, one for Pimple and one for Silex?

I've never used Pimple on it's own before so I'd have to believe what you say. Is there any documentation anywhere or examples where providers are built to support Pimple with and without Silex?

Most of the Silex 2.x providers work without Silex.

DoctrineServiceProvider
TwigServiceProvider

Here's a simple provider I've made recently which works with Pimple, but also offers a trait for Silex: ParsedownServiceProvider.

Pimple documentation about providers
Silex documentation about providers which mentions to use only the API of Pimple so your provider could be used outside of Silex:

The Pimple\ServiceProviderInterface belongs to the Pimple package, so take care to only use the API of Pimple\Container within your register method. Not only is this a good practice due to the way Pimple and Silex work, but may allow your provider to be used outside of Silex.

Wouldn't registering the commands be a performance problem? Especially as it would be done every request and you're only going to want the migrations helper for console commands.

This provider would most likely be registered only in CLI when you have a Console application instance which it requires. So in most cases performance doesn't matter so much and requests won't be executed.
However we would still not register commands in the register() method.  We could still have a boot method to add the commands. Then the user could choose to either add the commands themselves or call the boot method. If using Silex, everything would be the same.

Shout me down if I've wrong but wouldn't it be easier and possibly better to have two service providers, one for Pimple and one for Silex?

I don't think that's a practice given the state of the Silex providers from 2.x which I've seen. And I don't think that would help users of the provider who would be more confused than we are.

I'd try to put a quick PR up for you to review.

@breenie If you want we could continue the discussion over a concrete implementation over at #12


Coord check looks good, but μTransfer is not working as expected

Hello, μP team! Very excited to see you open source your excellent work! I was looking to apply μP on our work, and on Megatron-DeepSpeed I modified the training script as suggested in the tutorial, set the infshape, reset parameters initialization, put on MuAdam, and got a coord_check that looked successful. But when we transfer the learning rate that performed well on the 350M GPT model to the large model 1.3B, we found that the 1.3B could not withstand such a large learning rate and eventually produced NaN.
I was wondering what details might not have been taken into account, or the conditions were not met, causing μTransfer to fail. How should I debug, or μTransfer just won't work under this condition?
The following is the experimental information.

350M -> 1.3B GPT model μTransfer training loss( tensorborad link ):

I think it may be a bit redundant, but if you are interested, the transformation of μP is here:

Replace output layer with MuReadout, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/model/gpt_model.py#L250
Make sure to use 1/d instead of 1/sqrt(d) attention scaling, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/model/transformer.py#L175
Set infshape and do mup parameter initiliaze, https://github.com/shjwudp/Megatron-LM/blob/mup/pretrain_gpt.py#L110
Put on MuAdam, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/optimizer/init.py#L65
Implement the equivalent MuReadout._rescale_parameters operation, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/mpu/layers.py#L191
Modify lr scheduler to update lr according to width, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/learning_rates.py#L127
Coord check, https://github.com/shjwudp/Megatron-LM/blob/mup/megatron/mup_utils.py#L16

Hi shjwudp,
Thanks for your interest in our work!
Your coordinate check plots seem identical across time steps, which is a sign that the learning rate is too small for the function to change. Can you try rerunning with a larger learning rate? It's possible that with a moderately larger learning rate, the muP run might blow up after a couple steps, in which case we can look into it further.

Hi Edward, thank you very much, your advice saved me. A larger learning rate exposed the problem, the plot showed jitter, I debugged and fixed the problem, and now the plot is smooth and shows good increase in value. I'm going to try the big learning rate μTransfer on this.
Although it looks good now, the working principle is too hard for me, and the muP is really amazing.

Hi, @edwardjhu I've recently done some experiments, an extension of the previous discussion. I found that transferring the same hyperparameters from a 350M model to 1.3B scale works fine, but transferring to a larger model 2.7B blowup, does that mean my parameters are too aggressive? how should i avoid this?
My coord:

The same hyperparameters, 1.3B model and 2.7B model comparison: https://tensorboard.dev/experiment/RirdggEZS8O2rRU9clEy0g/#scalars

Another question is that the transformer example and mutransformers use different initialization methods, (init_std / d_model) ** 0.5 vs init_std * width_mult ** -0.5, are these two formulas equivalent in some sense? Will there be pros and cons?

Thanks for your patience, Jianbin.
There are many considerations when training a very large model. In some sense, mup is a necessary but insufficient condition for successful training of large models. Other factors include the use of weight decay and floating point precision. Hope this can help with your investigation!

Another question is that the transformer example and mutransformers use different initialization methods, (init_std / d_model) ** 0.5 vs init_std * width_mult ** -0.5, are these two formulas equivalent in some sense? Will there be pros and cons?

They are equivalent up to a constant.

Hi Edward, thank you very much, your advice saved me. A larger learning rate exposed the problem, the plot showed jitter, I debugged and fixed the problem, and now the plot is smooth and shows good increase in value. I'm going to try the big learning rate μTransfer on this.
Although it looks good now, the working principle is too hard for me, and the muP is really amazing.

Hi Jiabin, Could you share info on what caused the jitter in your coord check plots? Its possible that I have a similar issue (#58).
Thanks!


IPROTO-273 Elements in collections are not properly limited

https://issues.redhat.com/browse/IPROTO-273
Trying this for the second time. There is an option to enable the fix since it changes the binary format. It is disabled by default to keep backward compatibility.

replaced by https://github.com/infinispan/protostream/pull/265


New solution doesn't pass tests, and forking gives a massive error message

https://www.codewars.com/kata/reviews/5e92b8ebd867810001cb545e/groups/5e949103955e44000113a95f
This solution does not pass the tests, in Lean, sorry is just a placeholder. It does nothing.
And when I fork it, I got the biggest error message I've seen on CW. x)

I don't know why that solution was accepted, but the cause of this error when forking was because it resulted in invalid Kumite (solution forks have kata's initial code as root and all Kumite must be different from its parent to be valid, but this solution had the initial code as is).
I removed the solution and the solution group.


Have BOA test assert on key statistics

This will make it easier to check if things are working without looking
at the GUI.
Move the old race-detector test to a separate file so we can run it only when
race detector is enabled. The new test is too slow with the race detector enabled,
and the old test is worthless with it disabled.
And some minor clean-up in boa example:

Get rid of global TheSim. It is not needed and confusing.
Rename a couple of functions for clarity
Lint violations

Looks like the tests got stuck downloading some packages. I re-ran them and hopefully will work this time.

Looks like the tests got stuck downloading some packages. I re-ran them and hopefully will work this time.

No actually the tests are timing out. Maybe try extending the timeout? Not sure if we want to run these in CI since they are a bit long.

Codecov Report
Patch coverage: 14.94% and project coverage change: -3.13 :warning:

Comparison is base (7e9561c) 40.34% compared to head (673fb36) 37.21%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
- Coverage   40.34%   37.21%   -3.13%
==========================================
  Files          53       54       +1
  Lines       10222    11030     +808
==========================================
- Hits         4124     4105      -19
- Misses       5865     6689     +824
- Partials      233      236       +3

Impacted Files
Coverage Δ

axon/deep_layers.go
13.97% <0.00%> (-6.25%)
:arrow_down:

axon/deep_net.go
0.00% <0.00%> (ø)

axon/deep_prjns.go
0.00% <0.00%> (ø)

axon/layer_compute.go
42.57% <0.00%> (-0.96%)
:arrow_down:

axon/layertypes.go
41.66% <ø> (ø)

axon/layertypes_string.go
16.66% <ø> (ø)

axon/networkbase.go
50.52% <0.00%> (-0.08%)
:arrow_down:

axon/neuron.go
81.25% <ø> (ø)

axon/pcore_net.go
0.00% <0.00%> (ø)

axon/prjntypes.go
0.00% <ø> (ø)

... and 25 more

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

I split the test into race and non-race. Overall CI is a bit slower now but it checks BOA is working at a basic level so I think it's worth it. We can adjust if it's problematic.


Press page improvements

[x] Make it easy to add new articles in a data-file, the selection should be done by the core-team and discussed in a pull-request.
[x] Add the existing /press page to the navigation so it can be reached.
[x] Extend the existing links, to not only show  articles about the OWA itselfe, but other interesting links that have been referred to in the past.
[x] Show links ordered by year of publication.
[ ] Show a little preview-image, timestamp, tags and description instead of a simple link (if I find the time)
[ ] Add filtering by tags (if I find the time)

@sbesh91 Thanks for merging! :rocket:


chore: use jenkins infra maven cd reusable workflow

🤖 Beep boop!
This is an automatic pull request that updates your GitHub action used for continous delivery workflow for this repository.
Switching to using a reusable workflow created to simplify maintance of this file in your repository.
In case of questions, please ping @jetersen.
Additional informations:
You can find the workflow here

Yes! Though we did fix a race condition by using action output from release drafter into interesting category action 👏


link problem in tcmalloc_minimal when using Debug-override configuration using Visual Studio on windows.

Downloaded the latest gperftools 2.12 from github, built it configured it with Debug-Override x86 on windows using VisualStudio 2022。
The Debug-Override configuration is newly created by me. It is copied from Release-Override, except for the following configuration items:

c/c++ - Optimization - Optimization:           Use “Disabled (/Od)”  instead of “Maximum Optimization (Favor Speed) (/O2)”
c/c++ - Preprocessor - Preprocessor Definitions:            Use “_DEBUG” instead of “NDEBUG”
c/c++ - Code Generation - Runtime Library:            Use “Multi-threaded Debug (/MTd)” instead of “Multi-threaded (/MT)”

Link Debug-Override libtcmalloc_minimal.lib in statically with a Console APP for test purpose.  Build the test app, the output is:
Build started...
1>------ Build started: Project: tcm_s, Configuration: Debug Win32 ------
1>G:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\VC\v150\Platforms\Win32\PlatformToolsets\v141_xp\Toolset.targets(39,5): warning MSB8051: Support for targeting Windows XP is deprecated and will not be present in future releases of Visual Studio. Please see https://go.microsoft.com/fwlink/?linkid=2023588 for more information.
1>tcm_s.cpp
1>libucrtd.lib(debug_heap.obj) : error LNK2005: __calloc_dbg already defined in libtcmalloc_minimal.lib(override_functions.obj)
1>libucrtd.lib(debug_heap.obj) : error LNK2005: __free_dbg already defined in libtcmalloc_minimal.lib(override_functions.obj)
1>libucrtd.lib(debug_heap.obj) : error LNK2005: __malloc_dbg already defined in libtcmalloc_minimal.lib(override_functions.obj)
1>C:\Users\muliu\Desktop\hub\tcm_s\Debug\tcm_s.exe : fatal error LNK1169: one or more multiply defined symbols found
1>Done building project "tcm_s.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Build started at 11:01 and took 03.951 seconds ==========
My guess is that the malloc, free and calloc of the Debug version have not been hooked, right?

So I am able to link debug statically linked C runtime with latest MSVC 2022. But it crashes on some internal debugging stuff in stl map and iterators business. They do some diagnostic stuff that runs too soon and touches locks that are not yet initialized. So we'll need to implement our own scalable std::map equivalent if we want to run in this configuration on MSVC.
Other than that, I found linker error trying to link non-debug C runtime. And I fixed that in comment referenced above.
Your original issue, well, perhaps it was fixed since it was reported.


Allow video.js 8

Description
Allow video.js version 8 in the dependencies.
Specific Changes proposed
Requirements Checklist

[x] Feature implemented / Bug fixed
[ ] If necessary, more likely in a feature request than a bug fix

[ ] Unit Tests updated or fixed
[ ] Docs/guides updated

[ ] Reviewed by Two Core Contributors

cc @gkatsev

Thanks for the PR. Are you able to revert the package-lock update actually? I'd like it to be tied to the version of npm/node that's set in the nvmrc.
Though, all of it is needed to be updated.

@gkatsev done. Is that how you want the PR?

@boris-petrov perfect, thanks!

Just tested it locally and it works, however, https://github.com/videojs/videojs-contrib-dash/blob/main/src/js/ttml-text-track-display.js is broken with Video.js 8.
I don't have an issue with merging this PR, but won't be able to release unless that's fixed.
I think it might be "as easy" as updating videojs-generate-rollup-config to latest release. (probably not that simple since it'll be going from 2.x to 7.x).
Would likely need a major version update release of this package afterward.


fastnlo-toolkit,python311Packages.fastnlo-toolkit: rename from fastnl…

…o_toolkit
Description of changes
part of #245383
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 278569 run on x86_64-linux 1

3 packages built:

fastnlo-toolkit
python310Packages.fastnlo-toolkit
python311Packages.fastnlo-toolkit


Typo in the in the image download.

Docker pull for the Recognize Text container
docker pull containerpreview.azurecr.io/microsoft/cognitive-services-rocognize-text:latest
should be
docker pull containerpreview.azurecr.io/microsoft/cognitive-services-recognize-text:latest

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

ID: 439c70ec-4484-25de-4942-939112188d58
Version Independent ID: 08f0bb9d-c6a0-d35b-8b3e-e46cca613487
Content: How to install and run containers - Computer Vision
Content Source: articles/cognitive-services/Computer-vision/computer-vision-how-to-install-containers.md
Service: cognitive-services
GitHub Login: @diberry
Microsoft Alias: diberry

@KevinTupper Thanks for pointing out the issue, I've addressed this issue in below PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/66997
The changes should be live EOD or tomorrow. We will now close this issue. In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. we love to take community contributions.


Update GitLab references to GitLab CI to GitLab CI/CD

Problem description
References to GitLab CI should be updated to GitLab CI/CD.
Affected product version(s)
docs issue
Suggestions for a fix
Replace instances of GitLab CI with GitLab CI/CD.

it appears that GitLab's own site has not made these changes yet...


feat: upgrades dependencies

Description

feat: upgrades dependencies

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[x] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Time LoadKeyV2

Get time= on LoadKeyV2. Useful for analyzing team loads.
2018-07-09T16:24:41.437587-04:00 ▶ [DEBU keybase util.go:534] 180a {VDL:0} - LoadKeyV2 uid:23260c2ce19420f97b58d7d95b68ca00,kid:012000afe3dbbb6bb55e4386a61c110fe803a0b0d82cf9ad3ae63072e530f7a6c7190a -> <nil> [time=2.98469613s] [tags:LT=q9C8qP2Sf3wy,TM=vGCrE2LOysIb,LK=Sm1XM0nz1g-5]

That CI failure is somewhat concerning, but unrelated.


[Question] Is there a programmatic way to deobfuscate other than using flutter symbolize?

Hello,
I am currently using the flutter symbolize tool to deobfuscate obfuscated stack traces. This is very helpful for debugging and troubleshooting. However, I would like to know if there are any other programmatic methods to deobfuscate stack traces?
For example, are there any APIs or libraries that can be called in code to achieve deobfuscation? This would help automate the deobfuscation process on the server side without relying on the command-line tool.
Thank you very much!

@shengzhou1216
This seems similar to https://github.com/flutter/flutter/issues/114669 in which you can follow-up for further updates.
Also check below documentation for further reference:
https://github.com/flutter/flutter/blob/master/docs/engine/Crashes.md#symbolicating-stack-traces-for-engine-crashes
https://github.com/dart-lang/dart_ci/blob/main/github-label-notifier/symbolizer/README.md#using-this-locally
If above don't help much, then it is better to ask this question in flutter's discord to get quick response, as this is more of a question than a bug report or feature request.
Closing from here. If you disagree, write in comments and I'll reopen it.


Shows substeps in junit

What type of PR is this?
Adds measurements/phases as separate steps for reporting - Each measurement/phase will have separate row in junit.xml, which will result in separate row in testgrid.
Depends on PR https://github.com/kubernetes/perf-tests/pull/2012
/kind feature

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

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

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

Two questions:

Do we have a guarantee about the order of substeps? I remember @tosi3k was adding "[step: X]" to get some order guarantees.
Is it a good idea to split phases into separate substeps? My concern is that without a human readable name it will be hard to figure out what it does:

/retest

/lgtm

/retest

/retest


Bmcjob controller implementation

Description
The BMCJob controller is responsible for running the Tasks (BMC actions like power and bootdev) listed on a BMCJob CR.
The controller,

Fetches the BMCJob object
Resolves the BaseboardManagementRef and establishes a connection (BMCCLient) to the baseboard management.
Pauses the BaseboardManagement object reconcile. (Defers unpause)
Reconciles the individual tasks in the BMCJob spec.
For each Task the controller creates a BMCTask object, with an OwnerReference.
Runs the action mentioned in the Task.
Updated the Task conditions and status.
Updates the Job condition and status.

Why is this needed
BMCJob is useful for performing one time actions that we want on the servers.
How Has This Been Tested?
Created a KinD cluster and tested creating and deleting BMCJob objects.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

This PR is a WIP 🏗
Also need to rebase based on changes discussed in #5

Codecov Report

Merging #6 (83dd8d2) into main (4a1c506) will decrease coverage by 27.23%.
The diff coverage is 2.43%.

:exclamation: Current head 83dd8d2 differs from pull request most recent head b823fe2. Consider uploading reports for the commit b823fe2 to get more accurate results

@@             Coverage Diff             @@
##             main       #6       +/-   ##
===========================================
- Coverage   54.46%   27.23%   -27.24%
===========================================
  Files           4        4
  Lines         112      224      +112
===========================================
  Hits           61       61
- Misses         42      154      +112
  Partials        9        9

Impacted Files
Coverage Δ

controllers/bmcjob_controller.go
0.00% <0.00%> (ø)

controllers/bmctaskhandler.go
0.00% <0.00%> (ø)

controllers/baseboardmanagement_controller.go
67.03% <100.00%> (ø)

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 4a1c506...b823fe2. Read the comment docs.

Please could you add to the description commentary on condition semantics.

Based on discussions I have refactored how BMCJob and BMCTask resources are controlled.
bmcjob_controller

Runs a BMCJob and also watches BMCTask based on owner reference.
Initially sets the Job to Running and creates the first BMCTask
The reconcile loop gets called again once the BMCTask updated and based on Task Conditions, the next BMCTask object is created.
Once all the BMCJob tasks have condition completed, BMCJob is marked Completed.

bmctask_controller

Runs the actions on a BMCTask
Establishes a connection to the BMC and executes the specifies action.
Updates the status


Fix link on #57

Click Close on #267?

I'm not sure what the comment means:
Click Close on #267?


Unable to access function arguments and local variables for linkshared binary

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

$ go version
1.15

Does this issue reproduce with the latest release?
Yes with every version of go
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="/usr/bin/gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build531678036=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Using dyninst, I tried to access local variables and arguments of a function which is part of a go binary compiled with linkshared option. To get local variables, dyninst uses libdw library to find all the modules. It uses dwarf_nextcu function of libdw to find all the modules of binary.
When I compile a go binary using "go build filename.go" then dwarf_nextcu finds all the modules (almost 118) and dyninst is able to access local variables and arguments of any function.
But when I compile a go binary with linkshared option  "go build -linkshared filename.go" then dwarf_nextcu finds only 5 modules. Due to that, dyninst is not able to access any variable of a function.
When I contact libdw team, they checked the symbols of both the binaries and gave a response.
//go binary without linkshared
$ eu-readelf --debug-dump=info wrapecho.txt | grep producer | sort | uniq -c
1            producer            (string) "GNU AS 2.25"
147            producer            (string) "Go cmd/compile go1.15.1"
2            producer            (strp) "GNU C11 6.5.0 20181026"
16            producer            (strp) "GNU C 4.9.2"
//go binry with linkshared
$ eu-readelf --debug-dump=info wrapecho_linkshared.txt | grep producer | sort | uniq -c
1            producer            (string) "GNU AS 2.26.1"
18            producer            (strp) "GNU C11 6.5.0 20181026"
Response from libdw team:
"So the information you want to extract from them is simply not there for the Go compiler generated functions. This is probably a bug you should produce to the Go language team or Ubuntu (if it is specific to their packaging)."
Is this a bug in go or there is any other way to read the symbols.
What did you expect to see?
I should be able to access variables of a function of go linkshared binary
What did you see instead?
Could not access variable of a function.

I believe currently debug info generation is not enabled for shared libraries. See #38378. I think we can close this and combine it into that issue. Thanks.


283 log based events

Closes # .
Tasks

[ ] ...
[ ] Documentation updated

ok to test

ok to test

ok to test

ok to test


Hide temp update in logs

Hi there,
Quick question, is there a way to hide the continuous temperature update from the homebridge logs? Cheers

No

In v1.1.0.


Test page doesn't work

looks like the cdn location doesn't work anymore, and I don't know how that particular one is set up

Yes, that was my own github based cdn which I no longer need since jsdelivr does it as well

:+1: i'll mark this one fixed by the current story as well, since i saw you got it working


Build phlow-init command

Create the phlow init command

git wrapping
configuration

default branch

We'll split up in more individual features

If we are splitting it up, then this is not a "won't have"! I assume we are splitting it up into more workable chunks because we do want it.
I suggest that we handle these by leaving them in "needs grooming" with a comment about split up work in progress, and then close the issue when we are ready to reference with

Will be implemented by issues #xx #yy #zz


Use FLASK_APP syntax for app config variable

Right now, the config syntax uses the format path/to/some.module.app, which tells serverless-wsgi to:

Set path/to as the PYTHONPATH.
Import some.module from this path.
Get the app variable from this module.

Instead, FLASK_APP uses path/to/some.module to represent the above, searching for an app value or a create_app function to call. Alternatively, you can do path/to/some.module:app or path/to/some.module:create_app(list, of, arguments) to choose a different variable or function name.
I think that using this syntax would make things a lot more flexible, allowing passing of arguments based upon stage, etc. in an easier way.

Thanks for the suggestion. Since serverless-wsgi is not a Flask-specific library, but targets multiple WSGI-enabled frameworks, I'm hesitant to adopt the Flask convention in this case. If you look at the implementation (https://github.com/pallets/flask/blob/master/src/flask/cli.py#L52), it's very specific to Flask, and will not have any use to other WSGI-enabled frameworks.
Functionally, I don't see any use cases that you could only accomplish with the current implementation, but the Flask-specific implementation would allow. If you have a specific case that proves otherwise, please let me know.
I'm going to close this issue for the time being, but please reopen if you disagree with my comments.


Using for react / react-native (JSX)

Description
What is the best setup for using atom-beautify with react or react-native? It seems to be tricky to get it to work well with JSX, espeicially when using the *.js file extension like react-native uses.
Related Issues:
https://github.com/Glavin001/atom-beautify/issues/742 <-- most related
https://github.com/Glavin001/atom-beautify/issues/774
https://github.com/Glavin001/atom-beautify/issues/1454
Input Before Beautification
This is what the code looked like before:
import { Component } from 'react';
import { View, Text, Image } from 'react-native';

export default class extends Component {
    render() {
        return (
            <View style={styles.test}>
              <Text style={{color:'red'}}>Test</Text>
            </View>
        )
    }
}

const styles = StyleSheet.create({
    test: {
        backgroundColor: 'red'
    }
});

Expected Output
The beautified code should have looked like this: (same as above just to test)
import { Component } from 'react';
import { View, Text, Image } from 'react-native';

export default class extends Component {
    render() {
        return (
            <View style={styles.test}>
              <Text style={{color:'red'}}>Test</Text>
            </View>
        )
    }
}

const styles = StyleSheet.create({
    test: {
        backgroundColor: 'red'
    }
});

Actual Output
The beautified code actually looked like this:
import {
    Component
} from 'react';
import {
    View,
    Text,
    Image
} from 'react-native';

export default class extends Component {
    render() {
        return (
            <View style={styles.test}>
                <Text style={{
                    color: 'red'
                }}>Test</Text>
            </View>
        )
    }
}

const styles = StyleSheet.create({
    test: {
        backgroundColor: 'red'
    }
});

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/indivisable/b11dd57c529e7c258d720b9fe1a654db
Checklist
I have:

[X ] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[X ] Reloaded (or restarted) Atom to ensure it is not a caching issue
[ X] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[X ] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[X ] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

To solve this problem you need to submit a pull request adding support for Pretty Diff's neverflatten option.  It seems there exists a pull request for this that you could pick up: https://github.com/Glavin001/atom-beautify/pull/1289

@prettydiff this does not solve, I integrated this and get same results.

I tested at http://prettydiff.com/?m=beautify&neverflatten=true and it worked there.  It could be that the last version of Pretty Diff published to NPM is 1.16.37, but current version 2.1.16.  I aggressively working to close that gap, see #881.

@prettydiff Thanks, prettydiff is a great tool, lucky to have found it today :)
I tried on the website too with neverflatten and got same results:

Sorry, I provided bad instructions.  The neverflatten option needs to be set to false. and your import lists will be flattened.

Using for React JSX it sometimes simply deletes parts of the code and breaks it.
This part at the end of my every React Redux component is deleted when atom-beautify finds an error (and sometimes randomly) in the html markup part of the jsx. I tested with all beautifiers (JSDiff etc..) provided for JS and JSX.
I will make a gif of me recreating the bug and update this comment.
};
export default connect()(Component);

@Babazon If you could open a new issue please.  That sounds like a different issue.

Closing as this issue appears to be resolved.  If I am closing this issue incorrectly please reopen it.


File deletion doesn't work

File deletion doesn't work in dev mode.
Scenario - add some file and then follow the link delete. Page reloaded but file is still exist on the page.

it's a duplicate of #241


Provide ability to enable/disable markets

This would be useful for two reasons.

Listing a market at a predetermined time. Currently as soon as one lists a markets, anyone can trade on it, which is a problem since there needs to be coordination around GUIs and market makers getting setup on the market.

Closing markets. Disabling markets will be a pre-requisite to give people a chance to exit the orderbook before closing the entire market.

Labeling as p1 since it's a pre-req for closing markets.

This can be done via permissioned markets. Closing.


fix: selectedFields on getRows not working correctly

Fixes #702
Data from Back-end is retrieving correctly, but mapping of the data with the schema fields has an issue.
On below line schema fields and data mapping is based on index so, it will map always first property even if user doesn't selected that property.
https://github.com/googleapis/nodejs-bigquery/blob/93945dd30985143ed45472e8744f30b2f2a17793/src/bigquery.ts#L387
Solution:
Back-end data is retrieving same order as schema fields order, solution for this is to filter schema based on selected fields when selectedFields is provided by user.

Looks good to me. Pending second review from @stephenplusplus

@googlebot I fixed it.


lib/codereview: cannot find CL in local repository

Cannot undo existing CL, but should be able to:

# hg id
87dcbcc9371b tip
# hg st
# hg log -r 4eda5e4001fd -v
changeset:   21122:4eda5e4001fd
user:        Brad Fitzpatrick<EMAIL_ADDRESS>date:        Mon Sep 08 17:47:12 2014 -0700
files:       src/syscall/mksyscall_windows.go src/syscall/zsyscall_windows.go
description:
syscall: keep Windows syscall pointers live too

Like https://golang.org/cl/139360044

LGTM=rsc, alex.brainman
R=alex.brainman, rsc
CC=golang-codereviews
https://golang.org/cl/138250043

# hg undo 138250043
abort: cannot find CL 138250043 in local repository
#

I can see where the problem is, but not confident enough to change codereview.

Alex

No more codereview.py!


Add component table template with legend

Signed-off-by: Peter Nied<EMAIL_ADDRESS>Description

See it in action in https://github.com/opensearch-project/opensearch-build/issues/567
Check List

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

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

Codecov Report

Merging #868 (e18bf56) into main (fa2ebcd) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #868      +/-   ##
==========================================
- Coverage   91.18%   90.97%   -0.22%
==========================================
  Files          87       87
  Lines        2370     2370
==========================================
- Hits         2161     2156       -5
- Misses        209      214       +5

Impacted Files
Coverage Δ

src/system/temporary_directory.py
82.75% <0.00%> (-13.80%)
:arrow_down:

src/system/os.py
92.30% <0.00%> (-7.70%)
:arrow_down:

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 fa2ebcd...e18bf56. Read the comment docs.