option clashed when numbered=no and thmbox

as the question mentioned: https://tex.stackexchange.com/questions/222122/problem-with-thmtools-thmbox-and-numbered-no

Analysis
\declaretheorem[numbered=no, thmbox=M, name=Theorem]{theo} will eventually call (in \thmt@newtheoremiv)
\newboxtheorem[M]*{theo}{Theorem}

But thmbox's \newboxtheorem doesn't support this starred form, hence raises error.
Discussion
I would like to prevent the error and maybe add proper warning messages, but hesitate to make thmbox={<kv-list>} compatible with  numbered=no and probably other options.
Misc - thmbox side
Although thmbox has options headstyle=<code using #1 and #2> documented, currently you have to manually double the hashes:
\documentclass{article}
\usepackage{thmbox}

\newboxtheorem[L, headstyle={\bfseries\boldmath####1 ####2}]{theo}{Theorem}
% works
\newboxtheorem[L, headstyle={\bfseries\boldmath#1 #2}]{cor}{Corollary}
% raises error
%     ! Illegal parameter number in definition of \thmbox@temp.

\begin{document}
\end{document}

A possible patch to thmbox:
\usepackage{regexpatch}

\makeatletter
% double the hashes in #1
\xpatchcmd\thmbox@newA
  {\def\thmbox@temp##1{#1}}
  {\edef\thmbox@temp##1{\unexpanded{#1}}}
  {}{\PatchFailed}

\xpatchcmd\thmbox@new
  {%
    \expandafter\def\csname#2\endcsname{%
      \setkeys{thmbox}{#1}%
      \@ifnextchar[{\thmbox@beginA{#3}{#4}}{%
        \thmbox@begin{#3}{#4}{}}}%
  }
  {%
    \expandafter\edef\csname#2\endcsname{%
      % expand #1 (\thmbox@temp) by one-step, then double the hashes
      \noexpand\setkeys{thmbox}{\unexpanded\expandafter{#1}}%
      \unexpanded{\@ifnextchar[{\thmbox@beginA{#3}{#4}}{%
        \thmbox@begin{#3}{#4}{}}}}%
  }
  {}{\PatchFailed}
\makeatother

I've mailed the above issue to thmbox's author.

Users are suggested to use thmbox={L, headstyle=...} in thmtools, or drop thmtools and use tcolorbox.

Currently, user has to double hashes four times, leading to 16 hashes:
\declaretheorem[thmbox={M, headstyle=\bfseries\boldmath ################1 ################2}, name=Theorem]{theo}

WIth the above thmbox patch, only four hashes are needed. Still some work to do on thmtools' side.

Thanks, the problem is solved temporarily. Will there be a compatibility of numbered=no and thmbox? Or will there will be a notice of not using numbered when thmbox is used?

Will there be a compatibility of numbered=no and thmbox? Or will there will be a notice of not using numbered when thmbox is used?

My plan:

when numbered=no and thmbox are both used, prevent current error and add warning to redirect user to thmbox={...}
adapt doc

Might not been implemented soon.

WIth the above thmbox patch, only four hashes are needed. Still some work to do on thmtools' side.

% patch<EMAIL_ADDRESS>\xpatchcmd\KV@thmdef@thmbox
  {\def\thmt@theoremdefiner{\newboxtheorem[#1]}}
  {\edef\thmt@theoremdefiner{\unexpanded{\newboxtheorem[#1]}}}
  {}{\PatchFailed}

\xpatchcmd\declaretheorem
  {\@ifnextchar}
  {\thmt@ifnextchar}
  {}{\PatchFailed}

% Compared to \@ifnextchar from kernal, this one uses
%     \edef\...{\unexpanded{...}}
% to prevent the number of hash characters reduced to half
\long\def\thmt@ifnextchar#1#2#3{%
  \let\reserved@d=#1%
  \edef\reserved@a{\unexpanded{#2}}%
  \edef\reserved@b{\unexpanded{#3}}%
  \futurelet\@let@token\@ifnch}

Side note: \tl_set:Nn is equivalent to \edef#1{\unexpanded{#2}}.


If teacher deletes a question with an answer that has been used by the API the API answer is not deleted

If teacher deletes a question with an answer that has been used by the API the API answer is not deleted


get rid of internal 'detector' variables

With this 7ee0e303e86ad72137a0234446c83cbfba75856e now it can fetch the detector name from the environment. Thus, we can clear the codes a little bit and ask users to put their names in the environment file once they are installed the package.
In principle, we can also prompt this to the user if the existing detector name is == 'TEST'

Maybe raise a warning if detector name is not set.


added link to ecs-run-taskdef.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.slack.com/archives/C013KJMDV0U/p1684688940277399?thread_ts=1684443297.461519&cid=C013KJMDV0U

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Out of memory when training with a P-100, 12G

sorry but I met with the OOM problem when training with 128 batch_size and  12G P-100.
is that normal?
i wonder why this happened since my card is 12G memory and i just want use larger batch_size.
thanks.

how many classes are there in your datasets? maybe you should minimize the batchsize.

since i also met with the OOM problem with 90 batch_size(default)
my device has 8G memory
finally i choose 64 batch_size to finish the training, it takes about 7G memory
so apparently the OOM problem will occur when deal with 128 batch_size using 12G memory GPU
you can use "nvidia-smi" to check your GPU status, including memory usage

thanks Itcs11.
close, when need reopen.


Messages being cropped when they are longer than what IRC allows for

Steps to reproduce

Connect to mattermost via matterircd
Have somebody send a message long enough from the mattermost side, (the RFC states 512 chars including the command etc.)

What happens
The message is cropped.
What I expect
The message should appear entirely. I know this is due to IRC limitations, but the gateway could send multiple messages until it is complete.

I would argue this is actually a bug and really requires attention. I am not really fond of losing (part of) messages

Fixed in master, release v0.12.1 soon


Revised and added to Using Sawtooth with Docker

Revised Using Sawtooth With Docker to follow new outline,
and added additional information and sub-sections.

Note: until the docker containers are updated, you will run into problems testing this document.
The container sawtooth-tp_config-default was changed to sawtooth-tp_settings-default, but the docker containers published have not been updated.

Note 2: Also, the intkey command on the docker images does not yet include the --count option, which was recently added:
intkey create_batch --count 10 --key-count 5


Cannot assign to read only property 'data' of object '#< InputEvent>

zepto.js:1002 Uncaught TypeError: Cannot assign to read only property 'data' of object '#< InputEvent>'
at HTMLInputElement.s.proxy (zepto.js:1002)
Error appears only with production build (Vue SPA).
            autocomplete('#id', {hint: false}, [{
                name: 'name',
                source: (query, callback) => {
                    const hits = []
                    ... // getting hits from custom source
                    callback(hits);
                },
                displayKey: 'name',
                templates: {
                    suggestion: suggestion => suggestion.name,
                    empty: data => `No results for ${data.query}`,
                }
            }
            ]).on('autocomplete:selected', (event, suggestion) => {
                // ...
            })

With dev mode all ok

I recognise that error, and thought we fixed it. The fix is likely the same as back porting https://github.com/madrobby/zepto/pull/1319

Did you fix this @SmelayaPanda?

I modified source repo by wrapping line e.data = data (which throw exception) with the following code:
  try {
      var dataPropDescriptor = Object.getOwnPropertyDescriptor(e, 'data')
      if (!dataPropDescriptor || dataPropDescriptor.writable) {
          e.data = data
      }
  } catch (e) {} // when using strict mode dataPropDescriptor will be undefined when e is InputEvent (even though data property exists). So we surround with try/catch

I’ll reopen, so we can do that fix in the source code itself.

Okay, thanks


ETag for read operations?

As a F/U to my previous issue on the topic. The page now contains:
"$etag:... It can also be used in HTTP headers in these ways:
with read operations to avoid fetching content that hasn't changed"
Actually as far as I can tell, IfMatch header is not available for ADT read operations (only for update and delete).

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

ID: 9e97b4d5-7efe-4f5f-0b83-c67de8c36d10
Version Independent ID: d64704f7-0e1b-eb34-46e7-0bca3ab707fd
Content: Manage digital twins - Azure Digital Twins
Content Source: articles/digital-twins/how-to-manage-twin.md
Service: digital-twins
GitHub Login: @baanders
Microsoft Alias: baanders

Hi @algattik, thanks for catching this. You're right, If-Match should be used for update and delete, not read.
I'm updating the document in PR #151729 to state that you can use If-Match to perform updates and deletes:

Since this removes the incorrect line, I'll have Git automatically close this issue when the PR is merged. However, feel free to continue the conversation and we can reopen if necessary.
Thank you!


stopped working

was working fine before, stopped working for me ...oh well ...
"You have been blocked from accessing this resource on Have I Been Pwned".

Updated to API v3
https://github.com/thewhiteh4t/pwnedOrNot/commit/44db4d4f4095f5d65e105dadc08706424b7d0b50


Add CI configuration for ROCm

This PR is for #1094 .

I fixed.

/test

Will merge after confirming https://ci.preferred.jp/cupy.py3.amd/41454/ succeeds.

Will merge after #2632 got merged.

/test

Merging as I confirmed that the test succeeds with https://ci.preferred.jp/cupy.py3.amd/41936/.

I've activated CI webhook for PRs.


unable to manually log in to seafile

maybe the user is not einstein?

Maybe I should specify https://github.com/pondersource/dev-stock/blob/e4c041e45f715c01039647dffb12bb4527e5bc7a/dev/ocm-test-suite/login/seafile.sh#L59-L60
./dev/ocm-test-suite/login/seafile.sh 11.0.5 dev einstein relativity

Although I tried this and it didn't seem to help

I'll try with:
./dev/ocm-test-suite.sh login seafile 11.0.5 dev electron

Hm, same result still

maybe the user is not einstein?

Yes, user name and password are defined here as<EMAIL_ADDRESS>and xu
https://github.com/pondersource/dev-stock/blob/e4c041e45f715c01039647dffb12bb4527e5bc7a/dev/ocm-test-suite/login/seafile.sh#L138


migrate semantics to nullsafety

Description
NNBD migration for semantics
Related Issues

dart-lang/sdk#40146

Tests
I added the following tests:
None
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Combined with the latest master, this PR is producing a dead_code analyzer warning. I am going to re-open it with an ignore for that at the right place to get it landed.

Superseded by https://github.com/flutter/flutter/pull/64055.


Added milliseconds support

This pull request addresses issue #30

Added support for milliseconds, microseconds and nanoseconds parsing in now.go
Adeed multiple unittests for the milliseconds support
Needed to change the existing unittest (line 251 in now_test.go) since now nanoseconds are supported

thank you for your PR


[Test-operator] Add support for tempest kubeconfig secret

This patch makes sure that one kubeconfig secret is created that can be consumed by both tempest and tobiko test pods.
Depends-On:  https://github.com/openstack-k8s-operators/test-operator/pull/62
As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

Testing here -> https://github.com/openstack-k8s-operators/test-operator/pull/63

No longer needed.


[Bug] Curations under 100KB show as 0.0MB

They should show as whatever amount of KB it is (for this one, 3.4KB).

https://yourbasic.org/golang/formatting-byte-size-to-human-readable-format/

fixed in https://github.com/Dri0m/flashpoint-submission-system/commit/0723e99dde33a203df4862321bdf7b0508aab94b


error: use of undeclared identifier 'SREV_TAM'

Describe the bug
I try to load a material but I get an error with the following expression in android studio:
static constexpr uint8_t DEFAULT_MATERIAL_PACKAGE [] = {
 #include "Assets/materials/aiDefaultMat.filamat"

};
The error I get is: "error: use of undeclared identifier 'SREV_TAM'"
SREV_TAM is the first string that appears in the aiDefaultMat.filamat file.
To Reproduce
I use the matc tool and the aiDefaultMat.mat material that comes in the examples to create aiDefaultMat.filamat, I try to load the material and it gives me an error. I am using c ++("-std=c++17").
Expected behavior
I think the error is due to the fact that the version of c ++ that I am using does not recognize the expression "#include" to load the material file.
Additional context
The operating system is OSX Catalina

You can't do it this way. When you invoke matc you need to use -f header to produce a file that can be included in a C++ file. This is not the preferred way of loading materials. Instead you should just read them like normal binary files.

Very grateful for the answer, I have followed examples and I have only seen that way. I will try read them like normal binary files.


Text rendering error in combination with width

The following program:
import Element exposing (..)
import Text exposing (fromString)

main =
  fromString "hello" |> centered
  |> width 200
  |> toHtml

Gives the following error
Uncaught TypeError: Cannot read property 'arity' of undefined
Commenting the width line gets rid of the error.
This error is present for Text Elements. If the Text Element is replaced by a collage, the presence of width does not produce the error anymore.
For example, this works:
import Element exposing (..)
import Collage exposing (..)
import Color exposing (..)

main =
  collage 20 20 [rect 10 10 |> filled red]
  |> width 200
  |> toHtml

👍 , ran into this myself, would like to see a fix. (I'd send a PR if history would lead me to believe that it would be merged in a timely fashion...)

I ran into this issue myself as well.

Ran into this too, but with height instead of width.

When I used width 800 etc. nothing is rendered. Removing it solves the issue. Height etc. all working fine.


What geo-database features are/will be supported?

Hi,
nice work!
I have the following two questions:

Are there plans to support also the SQL simple features specification (http://www.opengeospatial.org/standards/sfs) that is supported geospatial databases? For example, it would be useful if we could pose a spatial join with an ST_Overlaps condition, e.g., to retrieve overlapping geometries, etc.
Can we use Simba via jdbc? If not, are there plans for that in the future?

Thanks!
Konstantina

Hi Konstantina,
Thanks for your attention to our project. These are the answers to your question:

We can support sfs in Simba since it can be reduced to a parser problem or a query rewriting problem. I can see the reason why people want this feature since sfs is a wide accepted specification for spatial operations in SQL. We will consider integrating this feature in following up versions.

Currently, no. In Spark 1.x, JDBC service is bounded with hive context which uses a different parser from SQLContext. However, they merge SQLContext and HiveContext together as SparkSession in Spark 2.x. I think it will be natually supported when we migrate our project to Spark 2.x.

After answering these questions above, I would like to mention that I am a first Ph.D. student by now and I am pretty heavy loaded by my course works and other research. I will try my best to make some time in adding new features and migrating Simba to Spark 2.x. But please understand that we cannot do this in a fast and efficient industry working cycle.
Thanks again!
Dong

Hi Konstantina,
Thanks for your interest to the project!
As Dong mentioned, we are in the process of migrating to Spark 2.0.
Hopefully we will get that done in the next few months, after which we
should be be able to support JDBC as well as additional query feature
support. Stay tuned...
Thanks!
feifei
On Tue, Nov 1, 2016 at 9:38 AM, Dong Xie<EMAIL_ADDRESS>wrote:

Hi Konstantina,
Thanks for your attention to our project. These are the answers to your
question:

We can support sfs in Simba since it can be reduced to a parser
problem or a query rewriting problem. I can see the reason why people want
this feature since sfs is a wide accepted specification for spatial
operations in SQL. We will consider integrating this feature in following
up versions.
2.
Currently, no. In Spark 1.x, JDBC service is bounded with hive context
which uses a different parser from SQLContext. However, they merge
SQLContext and HiveContext together as SparkSession in Spark 2.x. I think
it will be natually supported when we migrate our project to Spark 2.x.
After answering these questions above, I would like to mention that I am a
first Ph.D. student by now and I am pretty heavy loaded by my course works
and other research. I will try my best to make some time in adding new
features and migrating Simba to Spark 2.x. But please understand that we
cannot do this in a fast and efficient industry working cycle.
Thanks again!
Dong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/InitialDLab/Simba/issues/64#issuecomment-257600278,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEX3pzl8eJQtb2AxfTyzRfpQZc6GnPUHks5q51z7gaJpZM4Kl7PG
.

Hi Dong,
thanks a lot for the reply! Of course I understand that it takes a good amount of effort to support these things, my intention was more to understand what Simba currently offers as I am about to use it than to request for the new features. So no worries at all, thanks again and keep up the good work!
Best,
Konstantina


Add linter

Ideas for linting:

[x] Powershell style linter (@Ana06 - #6)
[ ] Ensure file paths use lowercase
[ ] Verify .nuspec has only the required fields
[ ] Verify nuspec version has the correct format
[ ] Verify correct versioning update

[X] fail if nuspec is not modified (@Ana06 - #8)
[ ] fail if nuspec version is not increased (by geting both revisions)

[ ] Verify the package depends on “common.vm”
[ ] Verify .ps1 imports the “common.vm”
[ ] Verify first line is “error action stop” in install
[ ] send a comment into PR with the problem

Make the linter output easy to see, currently lines like Progress: Downloading PSScriptAnalyzer 1.20.0... 0% add a lot of noise.


Convert TSP to ATSP

Describe the bug
I used the notebook in the link below to learn about rl4co(https://github.com/ai4co/rl4co/blob/main/notebooks/tutorials/2-creating-new-env-model.ipynb). I now want to verify the ATSP method, so I import ATSPEnv instead of TSPEnv like this:
batch_size = 2
from rl4co.envs import ATSPEnv
env_atsp = ATSPEnv(num_loc=30)
reward, td, actions = rollout(env_atsp, env_atsp.reset(batch_size=[batch_size]), random_policy)
env_atsp.render(td, actions)

which run correctly but when I Rollout untrained model like below, I encounter the following bugs:
Greedy rollouts over untrained model
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
td_init_atsp = env_atsp.reset(batch_size=[3]).to(device)
model_atsp = model_atsp.to(device)
out_atsp = model_atsp(td_init_atsp.clone(), phase="test", decode_type="greedy", return_actions=True)
actions_untrained = out_atsp['actions'].cpu().detach()
rewards_untrained = out_atsp['reward'].cpu().detach()

for i in range(3):
    print(f"Problem {i+1} | Cost: {-rewards_untrained[i]:.3f}")
    env_atsp.render(td_init_atsp[i], actions_untrained[i])

bugs are:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
Cell In[5], line 5
      3 td_init_atsp = env_atsp.reset(batch_size=[3]).to(device)
      4 model_atsp = model_atsp.to(device)
----> 5 out_atsp = model_atsp(td_init_atsp.clone(), phase="test", decode_type="greedy", return_actions=True)
      6 actions_untrained = out_atsp['actions'].cpu().detach()
      7 rewards_untrained = out_atsp['reward'].cpu().detach()

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
   1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
   1517 else:
-> 1518     return self._call_impl(*args, **kwargs)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1527, in Module._call_impl(self, *args, **kwargs)
   1522 # If we don't have any hooks, we want to skip the rest of the logic in
   1523 # this function, and just call forward.
   1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1525         or _global_backward_pre_hooks or _global_backward_hooks
   1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
   1529 try:
   1530     result = None

File ~/miniconda3/lib/python3.10/site-packages/rl4co/models/rl/common/base.py:246, in RL4COLitModule.forward(self, td, **kwargs)
    244     log.info("Using env from kwargs")
    245     env = kwargs.pop("env")
--> 246 return self.policy(td, env, **kwargs)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
   1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
   1517 else:
-> 1518     return self._call_impl(*args, **kwargs)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1527, in Module._call_impl(self, *args, **kwargs)
   1522 # If we don't have any hooks, we want to skip the rest of the logic in
   1523 # this function, and just call forward.
   1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1525         or _global_backward_pre_hooks or _global_backward_hooks
   1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
   1529 try:
   1530     result = None

File ~/miniconda3/lib/python3.10/site-packages/rl4co/models/zoo/common/autoregressive/policy.py:140, in AutoregressivePolicy.forward(self, td, env, phase, return_actions, return_entropy, return_init_embeds, **decoder_kwargs)
    125 """Forward pass of the policy.
    126
    127 Args:
   (...)
    136     out: Dictionary containing the reward, log likelihood, and optionally the actions and entropy
    137 """
    139 # ENCODER: get embeddings from initial state
--> 140 embeddings, init_embeds = self.encoder(td)
    142 # Instantiate environment if needed
    143 if isinstance(env, str) or env is None:

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
   1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
   1517 else:
-> 1518     return self._call_impl(*args, **kwargs)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1527, in Module._call_impl(self, *args, **kwargs)
   1522 # If we don't have any hooks, we want to skip the rest of the logic in
   1523 # this function, and just call forward.
   1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1525         or _global_backward_pre_hooks or _global_backward_hooks
   1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
   1529 try:
   1530     result = None

File ~/miniconda3/lib/python3.10/site-packages/rl4co/models/zoo/common/autoregressive/encoder.py:74, in GraphAttentionEncoder.forward(self, td, mask)
     62 """Forward pass of the encoder.
     63 Transform the input TensorDict into a latent representation.
     64
   (...)
     71     init_h: Initial embedding of the input
     72 """
     73 # Transfer to embedding space
---> 74 init_h = self.init_embedding(td)
     76 # Process embedding
     77 h = self.net(init_h, mask)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
   1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
   1517 else:
-> 1518     return self._call_impl(*args, **kwargs)

File ~/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py:1527, in Module._call_impl(self, *args, **kwargs)
   1522 # If we don't have any hooks, we want to skip the rest of the logic in
   1523 # this function, and just call forward.
   1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1525         or _global_backward_pre_hooks or _global_backward_hooks
   1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
   1529 try:
   1530     result = None

File ~/miniconda3/lib/python3.10/site-packages/rl4co/models/nn/env_embeddings/init.py:49, in TSPInitEmbedding.forward(self, td)
     48 def forward(self, td):
---> 49     out = self.init_embed(td["locs"])
     50     return out

File ~/miniconda3/lib/python3.10/site-packages/tensordict/tensordict.py:3697, in TensorDictBase.__getitem__(self, index)
   3695     idx_unravel = _unravel_key_to_tuple(index)
   3696     if idx_unravel:
-> 3697         return self._get_tuple(idx_unravel, NO_DEFAULT)
   3698 if (istuple and not index) or (not istuple and index is Ellipsis):
   3699     # empty tuple returns self
   3700     return self

File ~/miniconda3/lib/python3.10/site-packages/tensordict/tensordict.py:4625, in TensorDict._get_tuple(self, key, default)
   4624 def _get_tuple(self, key, default):
-> 4625     first = self._get_str(key[0], default)
   4626     if len(key) == 1 or first is default:
   4627         return first

File ~/miniconda3/lib/python3.10/site-packages/tensordict/tensordict.py:4621, in TensorDict._get_str(self, key, default)
   4619 out = self._tensordict.get(first_key, None)
   4620 if out is None:
-> 4621     return self._default_get(first_key, default)
   4622 return out

File ~/miniconda3/lib/python3.10/site-packages/tensordict/tensordict.py:1455, in TensorDictBase._default_get(self, key, default)
   1452     return default
   1453 else:
   1454     # raise KeyError
-> 1455     raise KeyError(
   1456         TensorDictBase.KEY_ERROR.format(
   1457             key, self.__class__.__name__, sorted(self.keys())
   1458         )
   1459     )

KeyError: 'key "locs" not found in TensorDict with keys [\'action_mask\', \'cost_matrix\', \'current_node\', \'done\', \'first_node\', \'i\', \'terminated\']'

Reason and Possible fixes
I think the problem is the mismatch between model and ATSPEnv, but I have not found a solution. Thank you for your time and attention

By the way, how should I train an ATSP model like a TSP model

I think the problem comes from https://github.com/ai4co/rl4co/blob/1a2da37d6104c33646f74bb4b040d2a4006876c2/rl4co/models/nn/env_embeddings/init.py#L16
Basically by default, the same InitEmbedding used for TSP is used for the ATSP environment. The issue is that in TSP you can just embed the coordinates of each node ('locs' in the TSP environment) and make the encoder infer the euclidean distance, while in the ATSP I think you can't because all you have is an asymmetric distance matrix ('cost_matrix' in the ATSP environment) and giving the encoder the coordinates of each node would not help it understand why going from one to the other has a cost and going back has another.
So I think that in order to solve the ATSP problem with the AM model you need a custom InitEmbedding that encodes the nodes in such a way that you also provide information about the asymmetric distance matrix. Maybe a GNN or something like that.

Hi @Mu-Yanchen, thanks for raising this bug and sorry for our late reply. Also thanks to @Haimrich's help!
In the current version, we applied the MetNet[1] on the ATSP. Different from other environments, the initial embedding for ATSP is located at here.
We updated the MatNet implementation in b3f1446820fd6c2d9ac3399369ffc134dd86b3ab. You may want to check a minimum testing on this notebook and play with it 🚀.
[1] Kwon, Yeong-Dae, et al. "Matrix encoding networks for neural combinatorial optimization." Advances in Neural Information Processing Systems 34 (2021): 5138-5149.

Closing now. Feel free to reopen if any other issue arises! :+1:


Provider: Request for portability page and form

Create a page with a form in order to request the portability of your number.
When the user already have a phone number (or more than one) with another provider and he/she is migrating to your provider and the user wants to keep his/her phone number(s).

The Request for Portability is already included in the Provider Checkout


Azure AppConfiguration Service for .NET (January-June 2021)

This tracks the work for Azure AppConfiguration Service for .NET.

Closing;  no open issues remain and the milestone has ended.


Review dependencies for FCS requirements based on DSC changes

This is based on https://github.com/DSC-iTC/cPP/pull/346#pullrequestreview-2210084779. The idea here is to review the changes made for the DSC to some of the FCS requirements and to add proper dependencies based on those changes as well as to consider if the SFRs from the DSC should be adjusted similarly based on the actual app notes.
For the purposes of the DSC this would make sense. I don't think this is critical, but would tighten up the document overall.

Where EXT SFRs are added as new dependencies, the rationale should be expanded to include why these are covered. This is not always done at the moment, and so should be included in the review.


docker node tasks --filter desired_state=accepted shows running tasks

# Looking at `docker node tasks`
# We have one task on node self. It is running.
$ docker node tasks self
ID                         NAME   SERVICE  IMAGE    LAST STATE             DESIRED STATE  NODE
brscnd76m5eh8l3cvmpt5lkj4  top.1  top      busybox  Running About an hour  Running        bd4242800842

# As expected, it is displayed when filtering running tasks.
$ docker node tasks --filter desired_state=running self
ID                         NAME   SERVICE  IMAGE    LAST STATE             DESIRED STATE  NODE
brscnd76m5eh8l3cvmpt5lkj4  top.1  top      busybox  Running About an hour  Running        bd4242800842

# Unexpected: it is displayed when filtering accepted tasks.
$ docker node tasks --filter desired_state=accepted self
ID                         NAME   SERVICE  IMAGE    LAST STATE             DESIRED STATE  NODE
brscnd76m5eh8l3cvmpt5lkj4  top.1  top      busybox  Running About an hour  Running        bd4242800842

# Now comparing with `docker service tasks`
# We have one service with 3 running tasks
$ docker service tasks  top
ID                         NAME   SERVICE  IMAGE    LAST STATE             DESIRED STATE  NODE
brscnd76m5eh8l3cvmpt5lkj4  top.1  top      busybox  Running About an hour  Running        bd4242800842
c4id3nmv05ivk1brv7qk5ik3e  top.2  top      busybox  Running About an hour  Running        04f850c4a3db
8t2qoxgr0e95w7bs0gwunf0az  top.3  top      busybox  Running About an hour  Running        04f850c4a3db

# As expected, all tasks are displayed when filtering running tasks.
$ docker service tasks --filter desired_state=running top
ID                         NAME   SERVICE  IMAGE    LAST STATE             DESIRED STATE  NODE
brscnd76m5eh8l3cvmpt5lkj4  top.1  top      busybox  Running About an hour  Running        bd4242800842
c4id3nmv05ivk1brv7qk5ik3e  top.2  top      busybox  Running About an hour  Running        04f850c4a3db
8t2qoxgr0e95w7bs0gwunf0az  top.3  top      busybox  Running About an hour  Running        04f850c4a3db

# As expected, no tasks are displayed when filtering accepted tasks.
$ docker service tasks --filter desired_state=accepted top
ID  NAME  SERVICE  IMAGE  LAST STATE  DESIRED STATE  NODE

$  docker version
Client:
 Version:      1.12.0-dev
 API version:  1.25
 Go version:   go1.6.2
 Git commit:   83e6197-unsupported
 Built:        Tue Jun 28 20:00:19 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.0-dev
 API version:  1.25
 Go version:   go1.6.2
 Git commit:   83e6197-unsupported
 Built:        Tue Jun 28 20:00:19 2016
 OS/Arch:      linux/amd64

I'm on it :angel:
#dibs

The fix works for me. Thanks @vdemeester


Scourge Arrow Thorn Arrows Missing 0.2 Multiplier

When calculating the DPS of Scourge Arrow's Thorn Arrows part, the 750% More from 5 stages is taken into account, however the calc is missing the multiplication by 0.2 to account for charging to 5 stages that the "Release at 5 stages" skill part has in the "Skill DPS" layer of the calc.

Yes I was looking into something else and noticed it as well. It multiplies the average damage by the attack speed only.
The fix is in the skill data (act_dex.lua), adding skill part 3 to the DPS multiplier mod.
I'll make a PR for it.


Re-add multiple emails in mozillians.org

Coverage increased (+0.1%) to 93.22% when pulling 960cb88a1df0f7d6c7130a1f67aa48532ef63d74 on akatsoulas:multiple-emails into f2e4b43b5b03ca3d1828bdf077e691ac4ae7adfd on mozilla:master.

tested locally:

Logging in with primary email
Logging in with alternate email
Add alternate email
Make alternate email primary

Coverage increased (+0.1%) to 93.227% when pulling 8d82a62482e02a6fcb3570fb762547292901a420 on akatsoulas:multiple-emails into 9422ecc3e0ca59af3fa3565a9384889797f06208 on mozilla:master.


Main README converted to Markdown

Summary
As discussed on the list, here's the main README converted to Markdown. If you go to "view file" on GitHub, it should be rendered as GitHub renders Markdown files. This was a mostly mechanical conversion of the syntax. I probably missed some places where I should use backticks for commands, files, etc. mentioned in the text. I can do that in a second pass.
Have a look and feel free to point out places where I missed something.
PS: maybe you can cancel the CI when it triggers since it will be unnecessary for this PR.
Impact
Nothing, just a README
Testing
Proofread the formatting in GitHub and on own Markdown editor to confirm it looks as intended.

@btashton @adamfeuer @w8jcik Would you like to take a look?

@Ouss4 I will check it out and report back. 🙂

@Ouss4 @v01d Overall it looks great, the rendering on Github is a great help. I suggested some minor formatting changes.

@v01d This is one big readme :) I will try to help if you guys won't be done soon. What is visible for me, right now:

You should use git mv on this file, otherwise the history is lost.
You can use backticks with titles as well. They look fine and are more readable I think.

I added all suggested changes (mostly a lot of backticks).

@v01d is it ready for merging?

Yes, looks OK to me.

If you consider history important I can try to make it so it is a rename, but I'm not really sure how to do so safely.

@v01d One way to fix this branch to preserve README history would be the following:

locally rename v01d:readme to v01d:readme-old
make a new branch from a fresh copy of master calling it v01d:readme
do the git mv and commit it
git merge --squash v01d:readme-old – this will take the changes from the old branch as a change into the new branch
commit the change
force push the branch here

That should fix the history.

I tried with a rebase and with your approach but I get a conflict:

CONFLICT (rename/delete): README.txt deleted in readme and renamed to README.md in HEAD. Version HEAD of README.md left in tree.
Squash commit -- not updating HEAD
Automatic merge failed; fix conflicts and then commit the result.

and README.md is left with the original contents of the .txt. I was trying to look if using rebase I could only select the added changes to the file and not the deletion of the .txt since that is what mostly conflicts with the git mv.

@v01d I'll try it myself and see if I can come up with a better way. 😄

Ok, did it the manual way: create a branch, made the rename and then used git show readme:README.md > README.md.
Should be all set now.

@v01d Cool. That's similar to what I just worked out.

@xiaoxiang781216 you can merge now when you're able

@v01d done.


Improve DX around body parser content types

Many new users to Denali will likely attempt to use JSON-API, which requires a specific content-type header. The default body-parser middleware won't parse this default type without explicit configuration. These two taken together means a common source of confusion for new users.
We could configure the body-parser to handle JSON-API out of the box, but that's not ideal - it reduces but ultimately masks the issue at hand, and others using different content types would have the same problem.
Not sure if there's a way we can effectively warn users about this based on runtime behavior.
We may want to consider if there is a way to move body-parsing out of the middleware stack, into Denali's Parser. That would allow us far more flexibility as well when it comes to things like streaming requests. The challenge is how to handle any downstream middleware that depend on body-parser running first. Perhaps we run the parser before the middleware?

Closing in favor of #372


Unable to validate the token: should have a JSON content type for JWKS endpoint

[KRAKEND] 2022/09/22 - 18:29:31.731 ▶ ERROR [ENDPOINT: /geo/v1/countries][JWTValidator] Unable to validate the token: should have a JSON content type for JWKS endpoint
Attached below is my config file.....
  "extra_config": {
    "auth/validator": {
      "alg": "RS256",
      "cache": true,
      "issuer": "https://auth.sophware.com/realms/sophware",
      "jwk_url": "https://auth.sophware.com/realms/sophware/protocol/openid-connect/certs",
      "operation_debug": true,
      "scopes": [
        "sophware_geo_api"
      ]
    },
    "qos/ratelimit/router": {
      "client_max_rate": 10,
      "key": "Authorization",
      "strategy": "header"
    }
  }

If I replace the jwk_url with local file with the contents returned by the URL it works just fine.....
Can someone help me figure out what is wrong?
Thanks!

I am using the latest KrakenD with keycloak19 behind an AWS Load Balancer....  Content-Type appears to be set properly to me....(assuming that is what it is complaining about.

@rjsoph you're right, that error is telling you that the response your KrakenD instance is getting is not a JSON. It does not check the status code, so it could be getting a non 200 response or it could be that the cert from the ELB/ALB is not recognized by the system CA... Can you consume the JWK url from the same machine/container running KrakenD?

The URL is public (try it if you like) I see nothing wrong with it....
https://auth.sophware.com/realms/sophware/protocol/openid-connect/certs
I do have a work around that bypasses the load balancer but I would like to use the load balancer if possible.....  I am sure others are running keycloak behind a load balancer so that doesn't sound like a legit excuse....

Hi @rjsoph,
Even that your load balancer is public and accessible from the internet, it doesn't mean that your KrakenD instance within AWS has access to it. The headers transform is not a problem.
I have tested the configuration, and it works just fine (because my krakend can reach your balancer). Obviously I don't have a token signed, but the error is not what you have.
Spend some time checking the connectivity between KrakenD and the balancer. For instance you could try the following in the KrakenD machine:
$ docker run --rm -it -w /tmp --entrypoint=/bin/sh devopsfaith/krakend
/tmp $ wget https://auth.sophware.com/realms/sophware/protocol/openid-connect/certs
Connecting to auth.sophware.com (<IP_ADDRESS>:443)
saving to 'certs'
certs                100% |****************************************************************************************************************************************|  2925  0:00:00 ETA
'certs' saved

And confirm you can do it within the machine.

Thank you for checking.... yes I suspect you are correct.  Fortunately/Unfortunately the problem has now resolved itself so I can confirm it is working.  We did have some connectivity problems and that is probably what led me astray....


Should use rabbitmq/amqp091-go instead of streadway/amqp

Library github.com/streadway/amqp (used in couple places in pkg/reconciler/brokerstandalone/resources/) hasn't been updated or maintained for a while. RabbitMQ team has hard forked the repo to https://github.com/rabbitmq/amqp091-go.

Hey @ChunyiLyu
Thanks for creating the issue. Based on the listed Project Maturity, is there a milestone/planned release after which one could expect more stability in terms of breaking API changes ?

I managed to get this repo with the amqp091-go lib by just aliasing and changing the lib in the https://github.com/vaikas/wabbit repo, But im interested to see if we could add the wabbit lib or fork to the amqp091 project.

Related #328

Hey @ChunyiLyu
Thanks for creating the issue. Based on the listed Project Maturity, is there a milestone/planned release after which one could expect more stability in terms of breaking API changes ?

cc @ikvmw @gvmw do either of y'all have context regarding at which point switching is a good idea ?

Switching now is a good idea. This has been a long-time coming, and we would like to see as many users of the Go RabbitMQ client switch to a version which is officially supported and maintained.
cc @michaelklishin @GSantomaggio

Totally agree.
Even not related I was thinking to propose the same change in argo-events


Switch to older gcc version for cuda for backwards compat

What changes were proposed in this pull request?
Switch the gcc version used in cuda for backwards compatibility.
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Quick checklist
The following checklist helps ensure your PR is complete:

[ ] Eclipse Contributor Agreement signed, and signed commits - see IP Requirements page for details
[ ] Reviewed the Contributing Guidelines and followed the steps within.
[ ] Created tests for any significant new code additions.
[ ] Relevant tests for your changes are passing.

I don't think libnd4j will compile GCC 7.x. The minimum version is probably 8.0.

@saudet  builds seem to work fine:
Cuda 11.0: https://github.com/eclipse/deeplearning4j/actions/runs/785376933
Cuda 11.2: https://github.com/eclipse/deeplearning4j/runs/2437755102


Create self-contained experiences

Is your feature request related to a problem? Please describe.
I'm trying to use this in a class. The problem is that we have to setup all the pcs with the SDK and the dotnet-try tool.
Describe the solution you'd like
Create a self-contained deployment so I can share it (USB, Network, Windows/Linux) to speed up this process.
Describe alternatives you've considered
The alternative is setting up the environment with every student.

Ideally this is available via Docker image that can be pulled and run.
In #191, I showed how to do this:

In an empty directory, create a file named Dockerfile with the following content:
FROM microsoft/dotnet:3.0-sdk

ENV PATH="${PATH}:/root/.dotnet/tools"

RUN dotnet tool install -g dotnet-try \
 && curl -k https://dot.net/v1/dotnet-install.sh \
    | bash /dev/stdin --install-dir /usr/share/dotnet --version 2.1.503 \
 && git clone https://github.com/dotnet/try.git

WORKDIR try/Samples

ENTRYPOINT ["dotnet", "try"]

then build an image using:
docker build -t dotnet-try .

Once the image has built successfully, run it using (note I had to re-map the ports):
docker run -it --rm -p 5000:80 dotnet-try

It would just need to be adapted to not clone this repo into the image during building but rather mount a volume of where a clone exists on the host.

Well, I think that using docker doesn't solve the problem. I'm trying to avoid installing prerequisites and give a simple ".exe" for Windows.

One idea that we've been discussing is a one-liner script that you run on each machine that bootstraps the SDK if needed (like these), then dotnet-try if needed, then launches you into the tutorial. It's a reasonably quick experience on a fast connection.
But a USB key is obviously faster and if your internet connection is slow or nonexistent, an approach like this is the only viable option. This also has the advantage of not taking up disk space.
@KathleenDollard @LadyNaggaga

Agree with  @smartcodinghub  docker isn't the natural solution for this. It also just adds to ramp for something we designed are continuing to design to be as friction-less as possible.
@smartcodinghub and @jonsequitur  having a self contained experience on a USB stick would really move the needle. Let's investigate this

Why use local instance if we can just listen from any ip - it is just webserver. Students just need local network and browser.
set ASPNETCORE_URLS=http://*:5000/
dotnet try

@zii-dmg I think allowing the teacher to host for the classroom could be a compelling experience. It's not straightforward, though. This would allow students to run arbitrary code on the host machine, and students would effectively be sharing builds. Sandboxing students' sessions and isolating them from one another, e.g. via containers, would limit the class size based on how much compute the teacher's machine has.

@jonsequitur Agree with that. The ability to share some build files so the students can run it easily could be perfect. Or even to distribute the class in an online platform without exposing an url or having the servers to host it.
I think this can be beneficial, maybe can we use the Exe and platform options?


colsample non-deterministic behavior on a different machines fix

C++ standard rand() was changed to mt19937 fixing this problem and this issue.
Problems:

Only for C++11.
I'm not sure about calling functions and variables in the updater.cpp.

C++11 is now used by  https://github.com/dmlc/xgboost/pull/735


Blueprint: File upload in list

I have following blueprint for a page:
title: Website
'@extends':
  type: default
  context: blueprints://pages

form:
  fields:
    tabs:
      types: tabs
      active: 1

      fields:
        test:
          type: tab
          title: The Tab
          fields:
            header.test:
              type: list
              fields:
                .title:
                  type: text
                  label: Test
                .file:
                  type: file
                  label: A file
                  destination: 'assets'
                  blueprint: 'themes.theme'
                  accept:
                    - image/*

These blueprint results in a correct view:

But after uploading a file nothing happens. No file was copied in the destination directory neither any entry for member "file" is visible in the pages yaml file.
If the file form is outside of the list everything works fine.

File uploads are not functioning in lists due to the dynamic nature of the list.  We know about this but it requires a radical change in how file uploads work in order to properly support it.  Basically we need an ajax loader that loads the files into a temporary location and that is the information that would be passed on form save.  Currently the file uploader works with regular multipart form data, but this is too complex for a list scenario.
I'll close this because we do know about it already and have plans to address it.  Thanks!


Refactor Sensors Model for  v2 architecture

We must improve the model and refactore the tests.

Closed in commit


How to center the visible items afte snapping/

I have a requirement where I need to center the items shown to the user, is that applicable?

I think that would be hard with this library because the interval depends on the item count in such a case.
I would recommend you to implement your own SnapHelper to do so. 🙏


Investigate using Jenkins for GitOps orchestration

As a developer I would like to see an example of a manifest resource generation pipeline using Jenkins. Similar to #356

Closing since we have #451


Proposal for the term "Filter"

My initial read was as a filter for material things as opposed to an optical filter, but I think the term could include both. I wanted to verify that interpretation. If optical filters are included, to help document could an example be included in the comment or as example of usage?
In addition, I propose factoring this as a 'filter function' and making the artifacts a defined class.
Assuming this factoring, I'd propose the definition:

Filter function: "function of an artifact, realized in a process where entities enter it but the portion that are of a specified type or which have specified properties constitute the majority of what passes through it"

Filter: "any artifact that bears a filter function"

Rationale for changes in wording:

The reference to a system isn't essential as it isn't clear what additional inference that might bring.
As a matter of consistency of definitions, it doesn't seem that reference to being part of a system is consistently made in analogous items, for example flywheel or fuel system. In the case of fuel system there is reference to a propulsion system but not to a system that includes both the fuel system and propulsion system.
Mention of separation seems redundant and may suggest mechanism. For example in a resonance filter it's not necessary that there is a separation into parts of the original. The portion not passed might be transduced.
The reason for the "majority of" language is that most filters aren't perfect.

Rationale for factoring: Consider a filter that both filters some type of thing and limits the amount of that thing. Limiting isn't filtering in that there's no specific property that is being selected. So, the limiting is one function and the filtering another and artifacts can have one or both functions. If we represented the artifact only we would land up with multiple asserted polyhierarchy.
I am representing optical filters in my work and I would do this defining subclasses of filter function then adding additional defined classes.

bump. This seems like an easy and I would think uncontroversial change. Should I submit a PR?

@alanruttenberg Please do. Make the PR on develop, not master.

@alanruttenberg This would be a good change for 2.0.  I would like to suggest the following revised language for the definitions:
Filter = A Material Artifact which bears an Artifact Function.
Filter Function = An Artifact Function that is realized in a process where entities of a specific type or which have specified properties enter it or pass through it.

Should be Filter: A Material Artifact that bears a Filter Function
Also see That vs Which


Localization for decimal number

Actually, it seems there is no way to use decimal numbers with localization info. I would like to use the comma instead of the dot as a decimal separator but i was unable to find the way (nor the doc).
Using "editable-number" instead of "editable-text" does not work either.

I think this is more of an angular issue.  See this issue

@eugef please close, this is an angular issue, not a xeditable issue.


capitalise faction in signature stats

 Issue by Trimatix
Monday Aug 03, 2020 at 17:22 GMT
Originally opened as https://github.com/Trimatix/GOF2BountyBot/issues/59

Unnecessary - Moving to v2


need a custom implementation: BroadcastTo.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): windows 10, x86-64
TensorFlow installed from (source or binary): source
TensorFlow version (or github SHA if from source): 1.15

Provide the text output from tflite_convert

Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CONCATENATION, CONV_2D, DIV, LEAKY_RELU, MAX_POOL_2D, MUL, SUB, TANH, TRANSPOSE_CONV. Here is a list of operators for which you will need custom implementations: BroadcastTo.

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem.
If including tracebacks, please include the full traceback. Large logs and files
should be attached.
Traceback (most recent call last):
File "c:\users\anaconda3\envs\pytorch1\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\users\anaconda3\envs\pytorch1\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\Anaconda3\envs\pytorch1\Scripts\toco_from_protos.exe_main.py", line 7, in
File "c:\users\anaconda3\envs\pytorch1\lib\site-packages\tensorflow_core\lite\toco\python\toco_from_protos.py", line 89, in main
app.run(main=execute, argv=[sys.argv[0]] + unparsed)
File "c:\users\anaconda3\envs\pytorch1\lib\site-packages\tensorflow_core\python\platform\app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "C:\Users\AppData\Roaming\Python\Python37\site-packages\absl\app.py", line 303, in run
_run_main(main, args)
File "C:\Users\AppData\Roaming\Python\Python37\site-packages\absl\app.py", line 251, in _run_main
sys.exit(main(argv))
File "c:\users\anaconda3\envs\pytorch1\lib\site-packages\tensorflow_core\lite\toco\python\toco_from_protos.py", line 52, in execute
enable_mlir_converter)

@mahdaneh
looking at the error log,seems like this is similar to  issue and let us know if it helps.Thanks

Please use the recent TF 2.x version for TFLite conversion after creating your TF model in TF 1.x. The above BroadcastTo op is already supported in the recent versions.

Thanks for your hint @abattery. The issue is resolved by migrating my  code (in TF1.15) to TF2 via help of this. Then using TF2 and setting tf.compat.v1.disable_v2_behavior(), I was able to convert my model to tflite throught tf.compat.v1.lite.TFLiteConverter.from_frozen_graph.


21 make benchmarks

Made benchmarks and fixed recursion issues

The benchmarks showed that haellang is ~100 times slower than python.


Testing on Tanks and Temples

Hi, I am using the script in the MVSFormer repo to test on Tanks and Temples (training set, Ignatius, etc.) with --prob_threshold set to a scalar instead and other settings unchanged. And the resulting point cloud seems too dense, which makes the recall and precision not balanced:
evaluation result : Ignatius
distance tau : 0.003
precision : 0.6671
recall : 0.9895
f-score : 0.7969
The f-score should usually be around 0.9 for Ignatius.
Is there anything setting wrong? Thanks!

If you need to improve the precision, increase the prob_threshold is a good choice.

Hi. Could you provide the setting to reproduce T&T benchmark results (if the current prob_threshold is not what is used)? My question is more about reproducing the results instead of asking for some improvement. Thanks!


deepUnmock doesn't work well with ts-jest@24

Issue :
deepUnmock doesn't work on ts-jest@24
Expected behavior :
deepUnmock should work on ts-jest@24
Debug log:

log file content
# content of ts-jest.log :
No content

Minimal repo :
https://github.com/hojovi/jest_test_failure
The tests successes in ts-jest@23 but fails in ts-jest@24.

The test did not fail due to version change in ts-jest.  ts-jest has never supported deepUnmock like jest does through babel-jest - see my comment .
I am assuming that you have changed your configuration and that before using ts-jest for transform you was using @babel/preset-typescript ( and jest was transforming with babel-jest, which performed the hoisting ) or the project was written in javascript ( and again babel-jest was hoisting ).

prior to this commit 04ab3dee231a896db74c36f0bef330884fcb1987 , after the typescript had been transformed the transformed source was then passed to babel for an addition transform if you did not explicitly skip this step with skipBabel  This would have performed hoisting.
This has now been changed to opt in.  ts-jest does attempt to port from the old method to the new but it only works for skipBabel = false.
There is now the concept of presets in jest.ts-jest presets help configure the transform.  Although one of these presets is called jsWithBabel it is not what now controls babel hoisting.
For that you need to set the globals/ts-jest/ babelConfig property.  In the case of your example repo setting this to true would have resulted in hoisting of deepUnmock.

Babel Config option
ts-jest by default does NOT use Babel. But you may want to use it, especially if your code rely on Babel plugins to make some transformations. ts-jest can call the BabelJest processor once TypeScript has transformed the source into JavaScript.

The option is babelConfig and it works pretty much as the tsConfig option, except that it is disabled by default. Here is the possible values it can take:

false: the default, disables the use of Babel

true: enables Babel processing. ts-jest will try to find a .babelrc, .babelrc.js file or a babel section in the package.json file of your project and use it as the config to pass to babel-jest processor.

{ ... }: inline Babel options. You can also set this to an empty object ({}) so that the default Babel config file is not used.

Note that ts-jest does now do hoisting but it does not currently hoist deepUnmock.  See my comment

Closes via #1372


Filter Transfer-Encoding Header

this fixes #1 by creating a "filteredOriginHeaders" object to use for cache and response.
headers to be filtered are stored in an excludedHeaders Array.

Thanks @LukasGrebe


Preprocessing and training improvements

This is a draft for the fixes I did during this 2/3 days to the preprocessing and training steps. It achieves on nick instance:

a total time of 1h for preprocessing (compared to 4h before)
130h (5.5 days) for training (in here the reference was 16 days, but not sure if the number of epochs is the same, I need to double check)

Pending:

[x] Test that nothing else is broken
[x] Create unit test
[x] Fix legacy non-working unit test
[x] Update documentation

I converted it to draft and removed [DRAFT] from the name

Could you add a list of major improvements in the PR descriptions to summarise the main changes?

Could you add a list of major improvements in the PR descriptions to summarise the main changes?
Yep, was on my TODO list! Please check: https://docs.google.com/presentation/d/1_1A87qb8tqczqlJTaT7a1UMYZpS1GKgBDJquMA2hl-s/edit?usp=drive_web&ouid=112294026481106883293

It's A LOT, please tell me if we need a call to go over everything.


example URL is not working - https://microsoft-my.sharepoint.com/personal/user1_contoso_com

[Enter feedback here]

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

ID: 4bca1c5d-b73f-1780-b002-81c8f7ef4c0a
Version Independent ID: 8a98d773-6bb4-8182-5405-2ce80c5b54ea
Content: Change a specific user's OneDrive storage space - OneDrive
Content Source: OneDrive/change-user-storage.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@nvgni Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@nvgni How are you? The sample url is not intended to be browseable. Thanks

@nvgni  Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Unable to get real-time pings from code ...

Using a minimal working example: I'm using ping as an example for streaming real-time outputstream.  I must be missing something critical.  Does anyone have an idea on how to fix this code to produce real-time results? In RESTeasy, I used StreamingOutput without a problem.
I'm using Javalin 6.1.6 with Temurin 21 (latest).
I'm using curl http://localhost:7070/stream (curl 8.8.0) for my client.
public static void main(String[] args) {
        final Javalin app = Javalin.create(config -> {
        }).start(7070);

        /* non-blocking */
        app.get("/stream", ctx -> {
            ExecutorService executor = null;
            try {
                executor = Executors.newSingleThreadExecutor();
                final OutputStream outputStream = ctx.res().getOutputStream();
                final ProcessBuilder processBuilder = new ProcessBuilder("/bin/bash", "-c", "ping -c 10 www.google.com");
                final Process process = processBuilder.start();

                final InputStream inputStream = process.getInputStream();
                executor.execute(() -> {
                    try (final BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream))) {
                        reader.lines().forEach(s -> {
                            try {
                                Logger.info(s);
                                //outputStream.write(s.getBytes(StandardCharsets.UTF_8));
                                //outputStream.flush();
                                ctx.result(s);
                                ctx.res().flushBuffer();
                            } catch (IOException e) {
                                Logger.error(e);
                            }
                        });
                    } catch (IOException e) {
                        Logger.error(e);
                    }
                });
            } catch (IOException e) {
                Logger.error(e);
            } finally {
                if (executor != null) {
                    executor.close();
                }
            }
        });
    }
}

Ok, I figured it out.  I'd like to help someone out there in the cyber world that knocking their head against the wall trying to solve this problem.  The following is the simple short solution that worked for me.
    public Context streamCommand(final Context ctx) {
        processCommandThread(line -> {
            try {
                final ServletOutputStream servletOutputStream = ctx.res().getOutputStream();
                servletOutputStream.println(line);
                servletOutputStream.flush();
            } catch (IOException e) {
                Logger.error(e);
            }
        });
        return ctx;
    }

Solved


Add a global bucket run to the buildkite pipeline, on GPU

Purpose
Adds a simple global land model run on GPU. This currently runs, but failed on plotting CuArrays. I am thinking that error is fixed now (TBD once buildkite runs) since I use ClimaCore Remapping functions to create the interpolated (regular) array for plotting.
To-do
Future PR: add a version of this with land/sea mask, time and spatially varying input
Content

adds CairoMakie to experiments' Project.toml
uses analytic TimeVaryingInput0D
does not employ a land/sea mask
the test is mainly that it runs and looks reasonable. We could also in principle test performance here, since this is a very simple system with no file IO

I also fixed a bug in that the drivers in the bucket tutorial were not being updated (no driver callback). There is now a driver callback.
Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

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

Overall it looks good. Two comments for future directions:
* Maybe the next PR could be add a "everything" test, where we turn on all the components in their most complex iteration (e.g., reading data from file) and we can keep expanding as we build new components

* For plotting and diagnostics, it would be nice to use nice to use the same Atmos interface in the future (to avoid reinventig the wheel)

For this PR, I'd suggest adding some minimal timing information so that we can have a sense of how fast/slow things are. At this point, just adding @time might be enough.

Thank you! I like the idea for the next PR, agree on the 2nd part, and added in @time.


stuck on "downloading content"

When exporting all of my notes with the script, evernote gets stuck on the "downloading content" window. worked as i tried it without the script - strange... does anyone have the same issue?
file size: 2.3GB
what i did: reboot, timeout of over 5hours, tried it around 15 times
system: newest sierra and bet evernote.

I don't see a "Downloading content" screen when I run the script (I forced it to run just now). Instead, I see an "Exporting Notes" screen, like this:

Mine takes a little while to export now as I've got almost 15,000 notes, but it still completed successfully. The script itself is set to time out after 30 minutes, but you can control this by changing line 7. (E.g., changing the 30 to 60 will set the timeout to 60 minutes.)

hey jamie
thanks for your help! ok, in this case i need to search the bug in evernote.
and thanks again for your great work! recently read an old blogpost where you searched for a software that counts the words written. take a look at http://wordcounterapp.com. yet very simple, but works great.


Add the Weboob project to scraping frameworks

The Weboob project is a mature scraping project with capabilities for banking sites, video viewing sites, public transport info, etc. It comes with a number of plugins for actual sites and usable client apps for the aforementioned capabilities.

Thank you!

@Mistress-Anna It would be great if you can add yourself in the contributors list at the bottom of web-development.md file


C_CPP : C++14 vs C++17

Description
I would like to provide the documentation of a comparison between C++14 and C++17.
Domain of contribution: C/CPP
Format: Documentation
Language: C++
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Please assign me this issue.

/assign

Closing as the PR got merged


CLIP(CNN)

Excuse me, the existing code only gives the code about ALBEF, TCL, CLIP(VIT), I did not find CLIP(CNN), what should I do if I want to use it?

Hello, if you want to use CLIP(CNN), directly change the "image_encoder" from "ViT-B/16" to "RN101" since we use ResNet-101 as the base architecture for the image encoder (Sec. 5.1).

Thanks for your reply, it does work. By the way, I have one more question. How can I reproduce the results of attack methods other than SGA in the table? Only the SGA code seems to have been given

Sorry to bother you, it seems that the code from CLIP to TCL or ALBEF is missing. I would appreciate it if you could provide it.

Thanks for your reply, it does work. By the way, I have one more question. How can I reproduce the results of attack methods other than SGA in the table? Only the SGA code seems to have been given

We will update the related codes soon.
Here is the official repo. of Co-Attack, https://github.com/adversarial-for-goodness/Co-Attack. PGD and BERT-Attack are modified based on 'Towards Deep Learning Models Resistant to Adversarial Attacks (https://arxiv.org/abs/1706.06083)' and 'BERT-ATTACK: Adversarial Attack Against BERT Using BERT' (https://arxiv.org/abs/2004.09984), respectively. And Sep-Attack denotes the combination of PGD and BERT-Attack. All the official repositories are accessible.

Sorry to bother you, it seems that the code from CLIP to TCL or ALBEF is missing. I would appreciate it if you could provide it.

Thanks for your interest in our work. The code towards clip2albef is available now; see in eval_clip-vit2albef_flickr.py

Sorry to bother you, it seems that the code from CLIP to TCL or ALBEF is missing. I would appreciate it if you could provide it.

Thanks for your interest in our work. The code towards clip2albef is available now; see in eval_clip-vit2albef_flickr.py

Thank you very much for your efforts to update the code, looking forward to your subsequent code updates. In addition, I would appreciate it if you could update TCL2ALBEF and ALBEF2TCL in SGA first when you are free. Thanks again for your reply.


[Fix-11314] [Doc] add tools/libs to the init database steps

Purpose of the pull request
Fix #11314

there is a deadlink found, could you please also fix the dead link in this PR, it related to the doc migration PR #11313 . You should change /development/ to docs/contribute

there is a deadlink found, could you please also fix the dead link in this PR, it related to the doc migration PR #11313 . You should change /development/ to docs/contribute

I think we might have some doc checks in our pre-commit hook https://github.com/apache/dolphinscheduler/issues/11347 :thinking:

there is a deadlink found, could you please also fix the dead link in this PR, it related to the doc migration PR #11313 . You should change /development/ to docs/contribute

I think we might have some doc checks in our pre-commit hook #11347 🤔

I haven't used this new feature yet. 🤣 Can we add this to the docs too?

there is a deadlink found, could you please also fix the dead link in this PR, it related to the doc migration PR #11313 . You should change /development/ to docs/contribute

I think we might have some doc checks in our pre-commit hook #11347 🤔

I haven't used this new feature yet. 🤣 Can we add this to the docs too?

We have already had it in our docs. But later this week I will find a way to install the hook automatically for our developers in case they miss it in docs : ) https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/development-environment-setup.md#code-style

there is a deadlink found, could you please also fix the dead link in this PR, it is related to the doc migration PR #11313 . You should change /development/ to docs/contribute

I think we might have some doc checks in our pre-commit hook #11347 🤔

It is not related to our CI like github action or pre-hook, this dead link because we separate our docs into multiple repos, but they have to connect to each other. In this case, we have to use the whole URL in our document, but when we change the place of source docs(PR1), the link will be changed, and a dead link error throw.
The reason why we can pass CI in PR1 is that before the PR1 merged, the whole URL link still work, but it become deadlink when PR1 is be merged

there is a deadlink found, could you please also fix the dead link in this PR, it is related to the doc migration PR #11313 . You should change /development/ to docs/contribute

I think we might have some doc checks in our pre-commit hook #11347 🤔

It is not related to our CI like github action or pre-hook, this dead link because we separate our docs into multiple repos, but they have to connect to each other. In this case, we have to use the whole URL in our document, but when we change the place of source docs(PR1), the link will be changed, and a dead link error throw. The reason why we can pass CI in PR1 is that before the PR1 merged, the whole URL link still work, but it become deadlink when PR1 is be merged

Got it, thx @zhongjiajie for the explanations!


Restructure the Core for consistency, clarity and new functionality

Changes in this humongous PR:
Creation of class hierarchy:
New classes:
EraSegWits --> EraTx --> EraTxBody --> EraTxOut --> Era
                     \              `--> EraPParams --> Era
                       `--> EraWitnesses --> EraScript --> Era
                        `--> EraAuxiliaryData --> Era

Note that all of the Era* classes have a few properties:

depend either directly or transitively on the Era type class.
define an associated type family (which was previously standalone).
add all constraints that the associated type must implement.

This means that any function that uses an associated type family can safely use
the corresponding type class. For example a function that accepts TxBody as
argument can always set EraTx as a constraint, which will bring all of the
superclasses as constraints as well.
Loose corespondance of new classes to the old removed concepts:

Era class is the same as before with the addition of type level protocol
version and deprecation of all of the functions
EraTxOut has  all superclasses as in UsesTxOut type  synonym constraint as
well  as defines  lenses that  supercede all  functions previously  defined in
Era type class
EraTxBody has all superclasses as in UsesTxBody type synonym constraint
EraAuxiliaryData has all superclasses as in UsesAuxiliaryData type synonym
constraint and consolidates ValidateAuxiliaryData into it
EraPParams replaces UsesPParams type class now with an associated type family
EraScript has replaced UsesScript type synonym constraint and the
ValidateScripts type class, with an exception of validateScript function,
which was moved into EraTx type class, since it depends on the Tx

Lenses
This PR expands usage of
microlens package.
Each class defines lenses for associated types, all of which are getters and
potentially setter. They also define a way to construct a minimal value of an
asscoiated type. For example if we want to construct a function that will
construct a Tx with a new TxBody, which uses AuxiliaryData from another
Tx:
newTxWithOtherAuxData :: EraTx era => TxBody era -> Tx era -> Tx era
newTxWithOtherAuxData txBody tx =
  mkBasicTx txBody & auxDataTxL .~ (tx ^. auxDataTxL)

Above function is guaranteed to work with transactions of all eras.
Some are just getters, which simply relay some information from one or more
fields of a concrete type, altohugh they might do some minimal computation. For
example allInputsTxBodyF will return all of the inputs present in the
TxBody.
Naming convention
All lenses are suffixed with the type name they operate on and either Lens or Getter.
Migration guide
Accessor fields
Names suffered some very minor adjustments that don't match the spec (tagged with *)
TxOut

makeTxOut (Proxy era) -> mkBasicTxOut

getField @"value" txOut -> txOut ^. valueTxOutL

_ -> txOut ^. compactValueTxOutL

getCoin -> txOut ^. coinTxOutL

getTxOutAddr txOut -> txOut ^. addrTxOutL

getTxOutEitherAddr txOut -> txOut ^. addrEitherTxOutL

getTxOutCompactAddr txOut -> txOut ^. compactAddrTxOutL

getTxOutBootstrapAddress txOut -> txOut ^. bootAddrTxOutF

getField @"datahash" txOut -> txOut ^. dataHashTxOutL (*)

getField @"referenceScript" txOut -> txOut ^. referenceScriptTxOutL

_ -> txOut ^. dataTxOutL

_ -> txOut ^. datumTxOutL

TxBody

getField @"inputs" txBody ->  txBody ^. inputsTxBodyL

getField @"outputs" txBody ->  txBody ^. outputsTxBodyL

getField @"txfee" txBody ->  txBody ^. feeTxBodyL (*)

getField @"auxiliaryData" txBody ->  txBody ^. auxDataHashTxBodyL (*)

getAllInputs txBody ->  txBody ^. allInputsTxBodyF

getField @"minted" txBody ->  txBody ^. mintedTxBodyF

getField @"wdrls" txBody -> txBody ^. wdrlsTxBodyL

getField @"ttl" txBody -> txBody ^. ttlTxBodyL

getField @"update" txBody -> txBody ^. updateTxBodyL

getField @"certs" txBody -> txBody ^. certsTxBodyL

getField @"vldt" txBody -> txBody ^. vldtTxBodyL

getField @"mint" txBody -> txBody ^. mintTxBodyL

getField @"collateral" txBody -> txBody ^. collateralInputsTxBodyL (*)

getField @"reqSignerHashes" txBody -> txBody ^. reqSignerHashesTxBodyL

getField @"scriptIntegrityHash" txBody -> txBody ^. scriptIntegrityHashTxBodyL

getField @"txnetworkid" txBody -> txBody ^. networkIdTxBodyL (*)

getField @"sizedOutputs" txBody -> txBody ^. sizedOutputsTxBodyL

getField @"referenceInputs" txBody -> txBody ^. referenceInputsTxBodyL

getField @"totalCollateral" txBody -> txBody ^. totalCollateralTxBodyL

getField @"collateralReturn" txBody -> txBody ^. collateralReturnTxBodyL

getField @"sizedCollateralReturn" txBody -> txBody ^. sizedCollateralReturnTxBodyL

Tx

getField @"body" tx -> tx ^. bodyTxL
getField @"wits" tx -> tx ^. witsTxL
getField @"auxData" tx -> tx ^. auxDataTxL
getField @"size" tx -> tx ^. sizeTxF
getField @"isValid" tx -> tx ^. isValidTxL

Nested instances are no longer needed, since composition of lenses works much better:

getField @"addrWits" tx -> tx ^. witsTxL . addrWitsL
getField @"scriptWits" tx -> tx ^. witsTxL . scriptsWitsL
getField @"bootWits" tx -> tx ^. witsTxL . bootAddrWitsL
getField @"txdatahash" tx -> tx ^. witsTxL . datsWitsL . to unTxDats

Witnesses

getField @"addr" txWits -> txWits ^. addrWitsL
getField @"bootAddr" txWits -> txWits ^. bootAddrWitsL
getField @"script" txWits -> txWits ^. scriptWitsL
getField @"dats" txWits -> txWits ^. datsWitsL
getField @"rdmrs" txWits -> txWits ^. rdmrsWitsL

Removal of constraint type synonyms
All of the Trans*, Uses* constraints have been removed as they are fully
replaced by the above hierarchy
Names removed and their closest replacements:

BlockAnn -> EraTx
ChainData - no replacemnt, this was just an adhoc collection of classes
SerialisableData,
AnnotatedData -> FromCBOR (Annotator t) + ToCBOR t
SerialisableData -> FromCBOR t + ToCBOR t
WellFormed -> EraTx

Val class
Addition of two class methods that allow efficient manipulation of compact coin
representation: injectCompact and modifyCompactCoin, without uncompacting
the actual type t for which the Val instance is being defined
Data type renaming
All concrete types for every era where either inconsistent or clashing with the
core type families:

Before this PR:

Core
Shelley
ShelleyMA
Alonzo
Babbage

Tx
Tx
--
ValidatedTx
--

TxBody
TxBody
MATxBody
TxBody
TxBody

TxOut
TxOut
--
TxOut
TxOut

AuxiliaryData
AuxiliaryData
MAAuxiliaryData
AuxiliaryData
--

Witnesses
WitnessSetHKD
--
TxWitness
--

Script
MultiSig
Timelock
Script
--

Value
Coin
Value
--
--

PParams
PParams
--
PParams
PParams

PParamsDelta
PParamsUpdate
--
PParamsUpdate
PParamsUpdate

After this PR

Core
Shelley
ShelleyMA
Alonzo
Babbage

Tx
ShelleyTx
--
AlonzoTx
--

TxBody
ShelleyTxBody
MATxBody
AlonzoTxBody
BabbageTxBody

TxOut
ShelleyTxOut
--
AlonzoTxOut
BabbageTxOut

AuxiliaryData
ShelleyAuxiliaryData
MAAuxiliaryData
AlonzoAuxiliaryData
--

Witnesses
WitnessSetHKD
--
TxWitness
--

Script
MultiSig
Timelock
AlonzoScript
--

Value
Coin
MaryValue
--
--

PParams
ShelleyPParams
--
AlonzoPParams
BabbagePParams

PParamsUpdate
ShelleyPParamsUpdate
--
AlonzoPParamsUpdate
BabbagePParamsUpdate

Module restructure:
Concrete era data type (eg. BabageEra) is no longer defined at the top level
module (eg. Cardano.Ledger.Babbage), but rather in a new private .Era module
(eg. Cardano.Ledger.Babbage.Era). It is later re-exported from the top level
module together with all of the orphan instances. This allows us to define all
of the relevant instances in their respective modules (eg. EraTxBody is defined
in the TxBody module)
Other minor changes

Defined NFData for Sized
Move module Cardano.Ledger.Shelley.Address.Bootstrap
(cardano-ledger-shelley) -> Cardano.Ledger.Keys.Bootstrap (cardano-ledger-core)
Moved langsUsed into tests, since that was the only use site.

@lehins thank you also for the amazing PR notes. Not only is it very helpful for reviewers, it leaves a great trail for others later down the line. (and for myself too, later, when I might have forgotten some details). :muscle:


Changes to this have caused isses

The page in AAD has changed, the button to resend invites is no longer there, the option is now behind the Manage option. There's no way to give users the ability to use this however and the page is vague about what permissions are needed, it just says "... an Azure AD administrator."
What role can I assign that will allow this? "Guest Inviter" allows invites, but not access to resend.

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

ID: 751889da-f001-117f-d525-3b27d467edf2
Version Independent ID: ce46adec-de86-7c34-45ba-d735cba76d2d
Content: Add B2B collaboration users in the Azure portal - Azure AD
Content Source: articles/active-directory/external-identities/add-users-administrator.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@brian-h1 , Thank you for your query . We will investigate and update the thread.

@brian-h1 , This issue has been escalated to our engineering team and they have been investigating this issue meanwhile you could use "User Admin" or "Global Admin" role to resend invite for guest users. I will keep you posted with an update. Appreciate your patience. Thanks.

@brian-h1 , This issue has been fixed and now Guest Inviter can Resend invite. Kindly check and update. Thanks !


write a function to copy a model and strip out the data

Added  in https://github.com/mattjj/pylds/commit/12fe0d4a4b2dc30926e8b263ea1a4de034f65186


"Drop Items Instantly" clashes with Menu Entry Swapper drop

After setting an item from "use" to "drop" left click, Instant Inventory causes there to be problems while spam clicking to drop items. As shown in the .gif above, double-clicking causes the item to be clicked again as a "use" action once Instant Inventory considers the item to be dropped. This behavior doesn't happen when Drop Items Instantly is disabled as shown on the left.

Thanks for the report, It seems to be an edge case for when the item is hidden fully (Hidden item opacity is 0%)

I tried this again randomly today and cannot replicate this anymore. Could you verify whether this also is fixed for you, or if its something on my end

I still have the issue when I set Hidden Items Opacity to 0%. You also need to have used menu entry swapper. Shift clicking behaves as you would expect. Setting it to 1% is a workaround.


Update content view height

I want to update content view height after performing some operations (adding some text or images). But height still stays fixed.
But calling foldingCell.contentView.requestLayout() does not do anything.
Where is the trick?
Please, help me.

I am also facing the same problem.. did you find any solution?

@ykorshev did you find any solution?

@ykorshev did you tried using
fc.addOnLayoutChangeListener(new View.OnLayoutChangeListener() {             @Override             public void onLayoutChange(View v, int left, int top, int right, int bottom, int oldLeft, int oldTop, int oldRight, int oldBottom) {                 Toast.makeText(MainActivity.this, "Changed", Toast.LENGTH_SHORT).show();             }         });

Hello, guys. To apply changes to height of your list items with folding cells, you should let your list (whether it recyclerview or listview) to know that there are changes in data of your items. It may be notifyDataSetChanged() for RecyclerView for example. Also be sure that you don't use hardcoded height for your list item in layout.xml.


Unable to extract video data, connecting cookies file

Please tick all applicable boxes.

[X] I am using Python 3.5.3 or higher (run python --version on the command line)
[X] I have followed the official guides to install the bot for my system
[X] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[ ] Bug
[ ] Feature request
[X] Question

Description of issue
Hello!
Is there a way to connect youtube-dl config file to bot, or somehow let the bot use --cookies command from youtube-dl?
I wanted to get rid of "Unable to extract video data" error, i managed to get rid of it when i used youtube-dl manually, but the error is still there when i'm trying to download the same video with MusicBot
Steps to reproduce

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
ERROR: gnIO6ve4_uk: YouTube said: Unable to extract video data

I've solved it, after @AutumnClove message, if you want to use cookies file in order to bypass age-restriction on videos you need to download you cookies file from youtube in netscape format (i used "Get cookies.txt" extension to a browser).
In order to use the cookies in MusicBot you need to add 'cookiefile': 'Cookies location', in downloader.py file located in MusicBot/musicbot/downloader.py in ytdl_format_options.
Im not getting Unable to extract video data error anymore, so if you have that problem try doing that.

I've tried this but I get this syntaxError for some reason when trying to run the bot any advice?

I've tried this but I get this syntaxError for some reason when trying to run the bot any advice?

I've tried this but I get this syntaxError for some reason when trying to run the bot any advice?

Make sure you're using tabs instead of spaces, and make sure it's in line with everything else.

I've tried this but I get this syntaxError for some reason when trying to run the bot any advice?

Make sure you're using tabs instead of spaces, and make sure it's in line with everything else.

That was part of my issue, another thing was a silly mistake I did that was I missed adding a comma to the line above (ops). Thanks for your help!

That was part of my issue, another thing was a silly mistake I did that was I missed adding a comma to the line above (ops). Thanks for your help!


v0.1.83 sst shell not injecting resources

Good evening to all, suddenly on every resource inside sst shell I got error: {Prop} is not linked.
I tried to remove complete dev stage and set new, but again same error.
Only Resource.App... works

sst shell as well as sst dev need --stage to inject proper Resources


fixed database tests

Fixed the database tests that were occasionally failing and made it so that each database that was created for testing is properly deleted after each test case.

This still doesn't seem to fix the problem of testing data somehow being created in main database (projectDatabase.db) during testing. After some debugging I think that the issue is that while testing other classes, the DataAccessConfig is accessed (ex. use case factory classes), causing the main database to initialize without closing after the test is finished. Then, the first repository test after it would use the unclosed connection to do tests, which depending on which test it is could cause dummy data to be created on the main database. From my testing it seems that disconnecting the DatabaseConnection before initializing during the setup of each test would fix this (adding DatabaseConnection.disconnect() to the beginning of each setup()).

I don't think disconnecting before each setup is the right solution. The issue is that in the use case files, we use DataAccessConfig directly instead of injecting it. If you look at DataAccessConfig, it is by default set to use the database implementation. If you set it to use local by default and edit the static block to only initialize the database if we are using the database implementation, you'll see that the main database will not be affected.

@Raqkkkkk I have fixed the changes. Please review again.


Public Repo - Reuse Registration

Register to reuse tool according to internal wiki page

Done - Compliance badge is green


Please rebase this and continue.


License File

Any chance I can convince you to put a license on this thing so I know exactly what I should do to feel guilt-free about stealing from it for my own slides?

Absolutely!

@Trott I have included the license information in the Readme and package.json. The copy for the licenses are also now included in the repo.
The software behind the talk is licensed MIT.
The content is licensed Creative Commons Attribution-ShareAlike 4.0 International Public License

Hooray! Thanks!


Forecast both units and price of products

Hi,
When forecasting units of many forecast dimensions, is it possible to also forecast price (rather than treating price as related series)?
Is it possible to illustrate it in the python notebook?
thank you and kind regards
Aga

For now, you can only forecast 1 variable, and use others as Related time series.
Have you tried our Related time series notebook :
https://github.com/aws-samples/amazon-forecast-samples/blob/master/notebooks/6.Incorporating_Related_Time_Series_dataset_to_your_Predictor.ipynb


gov: add prune wasm codes proposal

In order to release storage space for unused contracts a prune proposal makes sense.
When executed wasm codes without an instantiation would be removed from CodeInfo and wasmvm.
The proposal may include a relative height (or fix one in the past?) as a threshold to not delete new uploads.

Should exclude pinned code ids

Follow up on:
https://github.com/CosmWasm/wasmd/issues/840#issuecomment-1118525916

You can shedule this for 0.31 and start the work if you want to. The cosmwasm/wasmvm code will be ready as part of CosmWasm 1.2.

#1455 was closed after doing some benchmarks because of poor performance. Most likely a different approach is needed e.g. a migration script. Since this task is going to take more time than expected, it is descoped from v0.41 milestone.


Proper line highlighting

The caret is great, but when I am jumping across panes it can be difficult to see what line you're on at first glance. I'm seeing this issue with the primer.dark.tmTheme.

Hey, thanks for reporting this @tnypxl. This was fixed in v1.1.3. :smile:


Update to Gradle 8.1.1

Description of changes: Update to Gradle 8.1.1
To update Gradle I did:
./gradlew wrapper --gradle-version 8.1.1
./gradlew wrapper --version

All the code in this PR was auto generated, with except to the fixes to the Jacoco plugin configuration.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #405 (aea0b0c) into main (1ca11a4) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##               main     #405   +/-   ##
=========================================
  Coverage     85.71%   85.71%
  Complexity       19       19
=========================================
  Files             3        3
  Lines            49       49
  Branches          5        5
=========================================
  Hits             42       42
  Misses            3        3
  Partials          4        4

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


Preview Pane stopped working

Microsoft PowerToys version
0.74.1
Installation method
GitHub, Microsoft Store
Running as admin
Yes
Area(s) with issue?
File Explorer: Preview Pane
Steps to reproduce

✔ Expected Behavior

Show code like in Peek
Show PDF file

❌ Actual Behavior
Shows empty dark screen
when I click on one of these files, it doesn't load like it used to be
Other Software
No response

/bugreport