Using [SKIPPED] for un-supported component upgrades

I feel we should be using [SKIPPED] instead of [ERROR] when KET is upgrading components we don't support, see below ...
ingress1 [ingress]                                                              [ERROR]
- Performing an online upgrade of ingress nodes is not supported.
storage1 [storage]                                                              [ERROR]
- Performing an online upgrade of storage nodes is not supported.
storage2 [storage]                                                              [ERROR]
- Performing an online upgrade of storage nodes is not supported.

The user should only be seeing the word [ERROR] when something has gone wrong with the process being executed.

I'd agree with @swade1987. Skipped seems more appropriate.

I do agree that seeing [ERROR] is unexpected here but I think [SKIPPED] might be just as confusing, as it might indicated that that node will be skipped and not upgraded.
What do you think of [WARNING] printed in orange color?

Agreed I feel that [INFO] would be better than [WARNING] but the orange colour makes sense.

The important thing here is to properly communicate to the user that KET cannot guarantee that ingress and storage will remain available during the upgrade.

@alexbrand are we not just stating that Ingress and Storage nodes aren't going to be upgraded? If so, we just need to make the user aware of this.

No, that is not what is being communicated. Ingress and Storage nodes are still upgraded if you pass the --ignore-safety-checks flag.

I think the resolution here should also include improving the messages that are being printed. I can see where the confusion is coming from.

@alexbrand you read my mind, it's not obvious when you read the message what is going on under the covers.


Why using offset in calculating eigenvector?

Hi,
Thanks for sharing your code.
I just wonder why you use offset in the following code.
 if 'variation' in method:

        if level == 1:
            if (Uk is not None) and (lk is not None) and (len(lk)>= K):
                mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
                B = Uk[:,:K] @ np.diag(lsinv[:K])
            else:
                offset = 2*max(G.dw)
                T = offset*sp.sparse.eye(G.N, format='csc') - G.L
                lk, Uk = sp.sparse.linalg.eigsh(T, k=K, which='LM', tol=1e-5)
                lk = (offset-lk)[::-1]
                Uk = Uk[:,::-1]
                mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
                B = Uk @ np.diag(lsinv)

According to the experiment results, a larger offset value will result in a smaller $\epsilon$ error bound and vice versa. Is the offset value a hyper-parameter? If yes, how should I choose it?

Hello,
The offset trick allows one to use the Lanczos method, which finds the
eigenvalues of largest magnitude, in order to find the eigenvalues of
smallest magnitude. Normally, the Lanczos implementation should do the same
internally (if the correct flag is set), but in my tests my method was
always faster.
Actually, the trick should work when the offset is close to and larger than
the norm of L. To avoid actually computing the norm, I use the fact that
||L||_2 <= 2 max_degree, which is a well known upper bound.
In summary, changing the offset is not advised.
On Tue, Jul 2, 2019, 2:41 PM zachary<EMAIL_ADDRESS>wrote:

Hi,
Thanks for sharing your code.
I just wonder why you use offset in the following code.
if 'variation' in method:
    if level == 1:
        if (Uk is not None) and (lk is not None) and (len(lk)>= K):
            mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
            B = Uk[:,:K] @ np.diag(lsinv[:K])
        else:
            offset = 2*max(G.dw)
            T = offset*sp.sparse.eye(G.N, format='csc') - G.L
            lk, Uk = sp.sparse.linalg.eigsh(T, k=K, which='LM', tol=1e-5)
            lk = (offset-lk)[::-1]
            Uk = Uk[:,::-1]
            mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
            B = Uk @ np.diag(lsinv)

According to the experiment results, a larger offset value will result in
a smaller $\epsilon$ error bound and vice versa. Is the offset value a
hyper-parameter? If yes, how should I choose it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/loukasa/graph-coarsening/issues/1?email_source=notifications&email_token=ACQ7ZWR4PQ6KXDWF36PMSS3P5NEGZA5CNFSM4H43MLZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G437CZA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQ7ZWQY7UCYXSNN4RIKFLTP5NEGZANCNFSM4H43MLZA
.

Thanks. It's much more clear to me now.


How can I infer state from the component? (TS)

Hi, everyone!
I have this question, it's about reducing boilerplate amount of code required to write one test. Originally posted in react issues, but felt like you might know the answer as well because it is tightly coupled with the enzyme.
Original thread
Thanks in advance :heart:

This project isn’t authored in TS; i think the typescript repo is a better place to ask general TS questions, for react or otherwise.

Thanks


Introduce writer to models store interface

What this PR does / why we need it:
The idea has remained from PR #3144 but I change the implementation based on the comment https://github.com/pipe-cd/pipecd/pull/3144#discussion_r794164550
This PR introduces a new field named writer to each models' store, the store.writer will be used by Update-related functions or whenever we need to know who is the caller of that datastore operation.
Currently, we have 4 known writers which are: web, pipectl, ops, piped. In this PR, I added 2 more writers which are: TestWriter and UnknownWriter. The first is dedicated to testing, the latter is used in case there is no way to figure out who will be the caller for that datastore operation (there is one like that, it's used by HTTP API server in cmd/pipecd/server.go).
Note for the UnknownWriter, as the next work after this PR is merged, I planned to make logic like UnkownWriter is unable to order write operation to the datastore or such in datastore layer, please lets me know if you guys have a better idea to handle that kind of thing.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

I think it's better to change into Introduce commander to models store interface in the title of this PR.
The other is pretty good!
/lgtm

Nice. Can't wait to have this feature.
/approve


PUR: Feedback Link Test

As a follow-on to #8854, add test coverage for the feedback link once the test file is created in #8830

While we're adding test coverage for this link, let's go ahead and fix it and the other link outs on the Protocol Upload empty state. Right now, they seem to be opening inside a captive webview in electron, but we want them to open in the user's default browser


Disable Reopen Last File menu action when no last tab

This pull request introduces the possibility to disable the "Reopen Last File" menu action. Updated it in the client accordingly.
Closes #1173

@barmac I took the review over.

Ready for the review! Updated it according to the things we discussed. Inside the tests, I made sure only the lastTab flag set is tested; with sinon.match; and get rid of other properties (e.g. tabsCount) into the calledWith matcher. Just to make sure the test work regardless of changes inside the public API and only tests the flag.

Merged via https://github.com/camunda/camunda-modeler/commit/d1350f1a4414cd5d00d6fa478b66213296d6907c.


Cannot create a template ending in a variable

For example:
[{participant}] agrees to spend [{amount}] conga coins on [{swag}]

Because the template grammar requires a LastChunk piece of text this template parses as ambiguous (incomplete).
Need to make the LastChunk optional in the template grammar.

Fixed


Re-importing loop when using a plugin

Linux x86, Godot 2.1 beta
After using Tileset Importer by Nullifier https://github.com/Nulifier/GodotTileSetImporter and importing created tileset into the scene, my godot is stuck in an infinite loop of reimporting the tileset.res. Deactivating plugin doesn't help. Only if resource is deleted from the project it stops trying to reimport it.

Has this plugin worked with previous versions of Godot (such as 2.0.x)? I also note it's not been updated in two years, and various things (such as the plugin/module interface) have changed somewhat in that time.

Not sure, I've added it only to 2.1. Still, I don't think it's plugin's fault.

Related/duplicate of #4919?

I'd say yes

Is this still valid in 2.1.x and in master/3.0-alpha?

I'll check it in a couple of days.

Can't reproduce


Update docs for vue-identicon

Includes:

An Intro
Commands for installing with yarn or npm
3 Usage examples

Related to vue-polkadot/vue-ui#23

LGTM!


fix: correct two misspellings

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14530/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14530/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14530/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14530/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14530/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14530/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14530/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14530/merge#BCU1XR#0


Protect asset create endpoint

Dev Summary
only a user that is part of an org should be able to create an asset/post
to do this i added a new auth guard and used it on the endpoint
still to do -> fix the test that's breaking locally
(later to come, checking that updating/deleting a post can only be done by user within poster's org)

Test Plan

repro steps:
for these tests you can see that i have the seeded users and orgs, but i also inserted into user_orgs userId, organizationsId values 1, 1 to create a relationship between my user 1 and the first org

in browser login with user 1 and get cookie from dev tools
make curl request with user 1's cookie. expect success (because they have an org related)

curl -X POST localhost:3001/api/assets \
--cookie "NEH_is_cool=s%3AeyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MSwiZmlyc3ROYW1lIjoidXNlcjFGaXJzdCIsImxhc3RfbmFtZSI6InVzZXIxTGFzdCIsImVtYWlsIjoidXNlcjFGaXJzdEBleGFtcGxlLmNvbSIsImVtYWlsX25vdGlmaWNhdGlvbl9vcHRfb3V0IjpmYWxzZSwiaWF0IjoxNjcyODg2NTk4LCJleHAiOjE2NzI4OTAxOTh9._QCbE3mJWcKa72l6vnCEO1hCrtgR2cwzm8CdyzuOjTE.FdOBENq0wBAtBcXENuEwGX7X4Wm2ctqlajkL1enxP64" \
-H "Content-Type: application/json" \
-d '{ "title": "title", "description": "description", "quantity": 1 }'

repeat 1 and 2 with other user (with no org). expect fail

--cookie "NEH_is_cool=s%3AeyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MiwiZmlyc3ROYW1lIjoidXNlcjJGaXJzdCIsImxhc3RfbmFtZSI6InVzZXIyTGFzdCIsImVtYWlsIjoidXNlcjJGaXJzdEBleGFtcGxlLmNvbSIsImVtYWlsX25vdGlmaWNhdGlvbl9vcHRfb3V0IjpmYWxzZSwiaWF0IjoxNjcyODg4NjM5LCJleHAiOjE2NzI4OTIyMzl9.bqtpcaXXf3Eg1plPHBsBtjOmpdpDY-nvV7jHZKwe-7Y.MVgCtcVqxDUkxSpBhGlUQVtQEWRte%2FQjpa8ijLGxZ6k" \
-H "Content-Type: application/json" \
-d '{ "title": "title", "description": "description", "quantity": 1 }'

Resources

find where
https://docs.nestjs.com/custom-decorators
https://docs.nestjs.com/guards

My changes in https://github.com/Nonprofit-Exchange-Hub/web-app/pull/264 throws a wrench in your work. Happy to pair on merge conflicts

Aren't assets with that are "offers"/ "donations" created by users without orgs? It's only the "request" asset type that needs this guard.

This sounds right.


[misc] Remove maxAnswers=1 from *_department reference questions in prems surveys

These reference questions copy the value of Visit information/provider, which is multi-valued. Placing more than one value in Visit information/provider now causes the CPESIC, OAIP, and Rehab forms to be flagged as INVALID.
While in production we will always have a single value there and shouldn't have any issues, it is best practice for the referencing question to match the settings of its source.

tested


open-webui: import missing googleapis-common-protos dep

Description of changes
Import missing dep for protobuf/grpc code missing upstream in pyproject.toml, but does exist in uv.lock https://github.com/open-webui/open-webui/blob/b64c9d966a6498d4f2ffe12ab1498fab298afb37/uv.lock#L1145
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This should be reported upstream, can you confirm @tjbck ?

Even with this PR, building open-webui is currently broken. We need to wait the next merge of staging-next branch, hopefully this is going to fix the issue.
That said, I don't mind merging this PR.

Thanks @drupol 🙏


Fix publish chart versioning

Fix version generation by scripts/helm/publish-chart-yaml.sh
Description
Replace hard coded path and minor values of 0
with values derived from current values defined in
chart/Chart.yaml
Motivation and Context
This change is required to build with version numbers that monotonically increase
This makes is possible to test upgrades with setting override flags.
Regression
Is this PR fixing a regression?  No
How Has This Been Tested?
Tested by manually executing the script using the command line
nix-shell --pure --run './scripts/helm/publish-chart-yaml.sh  --helm-testing helm-testing/release/2.7 --date-time 2024-12-11-10-21-41' ./scripts/helm/shell.nix

Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

merge for 2.7 not develop


Autogenerate Pojos?

Currently the schema and object creation is two separate workflows.
For example, I had an object type of Foo I wanted exposed I would include it in both my .graphqls file as well as implementing the pojo
type Foo {
    bar: String
}

@Data
public class Foo {
     private final String bar;
}

Since the graphql tools are already inspecting the .graphqls file, it would be great to autogenerate this Foo pojo to avoid duplication

This isn't something we'll be implementing, since most use cases will have some actual logic to handle. They're two separate concepts.


[Wallet] Adjust integration

Description
This PR adds the Adjust SDK via Segment for tracking install attributions from ad campaigns.
Other changes

Upgraded all analytics dependencies.
Clean-up the secrets.json file to avoid duplicating configs between networks.

Specifically all alfajores builds now use a different SEGMENT_API_KEY than mainnet. Up until now that wasn't the case. All analytics were going to the same Segment source.

Tested
Couldn't test the full attribution flow with the Adjust dashboard, but checked the SDK is configured correctly and events are sent.
Related issues

Fixes https://github.com/celo-org/celo-monorepo/issues/6635

Backwards compatibility
Yes

Hey @jeanregisser , @gnardini Could you please let us know if anything need to test here from backend as well as front end from our side ?

Hey @jeanregisser Let me please know if any updates for above comment.

Hi @gnardini thanks for the confirmation.


Update Expr::CompoundIdentifier to have more accurate data format

To resolve #768

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

138 of 141 (97.87%)  changed or added relevant lines in 12 files are covered.
29 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.08%) to 97.812%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/executor/aggregate/state.rs
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

test-suite/src/tester.rs
1
88.89%

core/src/plan/planner.rs
28
78.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
30762

Relevant Lines:
31450

💛  - Coveralls


Vote - Rank: Incorrect results are displayed

The results for ranking are thus:

With the following votes having been cast:

"Red" was once 1, once 2 and once 3
"Green" was twice 1, and once 2
"Blue" was once 2 and twice 3

The results don't seem to correspond to the explainer:

Results of rank question corresponds to the percentage of the score a candidate has. Each voter gives candidates points by ranking them from 1 to N (lower is better). The score corresponds to the sum of the points a candidate got and is divided by the total number of points attributed across all ballots, and subtracted to one.

(I interpreted "substracted to one" as "substracted from one", as this corresponds to "the points that this candidate did not receive" and would thus lead to it being ranked higher in a "lower is better" ranking)

18 points have been awarded in total
"Red" has 6 points => 6/18  => 1 - 6/18 ~= 0.66 => ~ 66% OK
"Green" has 4 points => 4/18 => 1 - 4/18 ~= 0.77 => ~ 77% =/= 88.89%
"Blue" has 8 points => 8/18 => 1 - 8/18 ~= 0.55 => ~ 55% =/= 44.44%

The exported JSON contains the correct number of "points" for each candidate:
{
  "MainTitle": "Colours",
  "TitleFr": "",
  "TitleDe": "",
  "NumberOfVotes": 3,
  "Ballots": [
    {
      "BallotNumber": 1,
      "Results": [
        {
          "Title": "Colours"
        },
        {
          "Title": "{\"en\":\"RGB\",\"fr\":\"\",\"de\":\"\"}",
          "Results": [
            {
              "Rank": "1",
              "Choice": "{\"en\":\"Red\"}"
            },
            {
              "Rank": "2",
              "Choice": "{\"en\":\"Green\"}"
            },
            {
              "Rank": "3",
              "Choice": "{\"en\":\"Blue\"}"
            }
          ]
        }
      ]
    },
    {
      "BallotNumber": 2,
      "Results": [
        {
          "Title": "Colours"
        },
        {
          "Title": "{\"en\":\"RGB\",\"fr\":\"\",\"de\":\"\"}",
          "Results": [
            {
              "Rank": "1",
              "Choice": "{\"en\":\"Green\"}"
            },
            {
              "Rank": "2",
              "Choice": "{\"en\":\"Red\"}"
            },
            {
              "Rank": "3",
              "Choice": "{\"en\":\"Blue\"}"
            }
          ]
        }
      ]
    },
    {
      "BallotNumber": 3,
      "Results": [
        {
          "Title": "Colours"
        },
        {
          "Title": "{\"en\":\"RGB\",\"fr\":\"\",\"de\":\"\"}",
          "Results": [
            {
              "Rank": "1",
              "Choice": "{\"en\":\"Green\"}"
            },
            {
              "Rank": "2",
              "Choice": "{\"en\":\"Blue\"}"
            },
            {
              "Rank": "3",
              "Choice": "{\"en\":\"Red\"}"
            }
          ]
        }
      ]
    }
  ]

This can either be seen as a documentation or an implementation problem.
As I understand it, they're implementing a variation of Borda count, although typically Borda count would typically rank higher-is-better (from 0 to N-1).
The numbers in your demonstration add up if instead of giving "1, 2, 3, ...", voters give "0, 1, 2, ..." points.
Basically, we're dealing with an off-by-one, it's probably not worth disabling for such a small issue.

OK, in that case, we need to fix the docstring in the interface


Add status of block to flow blocks get

Closes #576
Description

For contributor use:

[x] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[x] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report

Merging #946 (94548e1) into master (23c85cb) will not change coverage.
The diff coverage is n/a.

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

@@          Coverage Diff           @@
##           master    #946   +/-   ##
======================================
  Coverage    2.53%   2.53%
======================================
  Files           5       5
  Lines         434     434
======================================
  Hits           11      11
  Misses        422     422
  Partials        1       1

Flag
Coverage Δ

unittests
2.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Abyssal amulet boosts aren't working properly

🐛 Bug Report
Noticed that the boosts on the abyssal amulets are inaccurate for the tiers pictured below. Tiers have 70/75/85 % boosts but all are only receiving 50% boost. The elemental and elder tiers are accurate at 65%. I was not able to test the combo runes.

This isn't a bug or issue. The rates were nerfed back in august of 2022 here https://github.com/oldschoolgg/oldschoolbot/pull/4393 and the wiki was never updated to reflect the nerf. This should be reported in the #wiki channel on the main discord.

I mentioned it to them here: https://discord.com/channels/342983479501389826/847902471271874580/1197913082953990204
just letting you know so we don't double post the same issue.


Very Important Slash

When I clicked the link it said page not found because this slash was not there.
Description

Fixes

Thanks for the fix!


Improve Reactions

The reactions leave "anybonds" in the SMILES. Should be improved with better reactions (and a broader set). See notes here: https://github.com/ur-whitelab/exmol/issues/129

DINGOS reactions https://codeocean.com/capsule/8271554/tree/v1


Automatic imports

I've looked through the issues and haven't yet found one for automatic imports. One of my favorite thing from ElmJutsu (Atom plugin) is that I can by typing, and it'll suggest symbols from the packages listed in my elm-package.json. I can select one of the functions in that autocomplete list and it'll automatically add that import to the top of the file, which turns out to be an incredible time saver.
please let me know if I've missed the boat and there's already another issue for this somewhere.
Thanks!

I though my installation of the plug-in was incomplete, because even though the auto-complete could find the type constructor, the import was not altered. It's good to see the enhancement ticket is submitted :)


fix: redirect without appending extra chars

Closes #1204
We might simplify it when https://github.com/unjs/ufo/pull/24 will be merged.
@rchl what do you think?

https://github.com/unjs/ufo/pull/24 has been merged.
Should be now better.
Thanks!

Good to go, let me know if anything is left @rchl
Thanks!


GH-37799: [C++] Compute: CommonTemporal support time32 and time64 casting

Rationale for this change

The original problem in mentioned in https://github.com/apache/arrow/issues/37799

SECOND in Parquet would always cast to MILLS
eq not support eq(time32[s], time32[ms)

This patch is as advice in https://github.com/apache/arrow/issues/37799#issuecomment-1737747352 . We tent to add time32 and time64 in CommonTemporal.
What changes are included in this PR?

Support time32 and time64 with different time unit in arrow::compute::internal::CommonTemporal.
Are these changes tested?

Yes
Are there any user-facing changes?
bugfix

Closes: #37799

@bkietz I've do a basic impl, would you mind take a look?

@bkietz I've resolve the comments now


help me please!

hello! i want to ask a question here. i have a log wrritten like this:
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)
[textdraw.streamer] DestroyDynamicPlayerTextDraw: No such id was found. (playerid: 0, textId: 0) (UI\ui_notif-2.inc:21)

how can i fix that? they said no such id was found. what does that mean not found?

nvm, i just fixed it


Adding version to the composer.lock displayFileName

Description of Change
Added the version to the displayFileName
Added basic test for composer parsing, including the new version
Changed output to debug
Have test cases been added to cover the new functionality?
yes

Can one of the admins verify this patch?

About the most recent changes... in subsequent testing I discovered that there was almost always an extra, empty dependency left around for the composer.lock file. Since this never gets evidence added to it, it was pointless to leave in the report. I made sure this only gets removed when there are dependencies added from parsing the lock file, so nothing is accidentally removed.

The build simply ran past the allocated 15 minutes?

After looking at some of the other analyzers, I'm going to take a standardized approach. Pulling this pull for now.


Scrolling on iOS

Hi There!
I was struggling with a modal of my own, were the div.modal-content has more content then what fit's into in iPad screen. I noticed that when scrolling it was the background that scrolled and not, as one would expect, the content inside the div.modal-content. So I started debugging...
I noticed that the problem actually is also present in the Photo Gallery demo on http://drublic.github.io/css-modal/#modal-pictures . Little screenshot below: the content of the modal doesn't fit (that's not a big deal) but is also not scrollable. The background is crolling instead.
Tested on iOS 7 (latest release) and iOS 8 (latest Beta).
Although I saw some commits that were intended to deal with this or a similar problem, the issue is present. Any thoughts on the problem would be highly valued!

Thanks for the report and sorry for handling this issue this late.
The latest master has some commits that might fix this issue. My tests with the visual test (index.html) indicate this although the background still scrolls if touch outside the modal.
Please let me know of your test results.

Please also see 3c66b9f027520e37fe78d6ae90e9eb190e18ec13. This should fix this issue mostly.


Reimplement computation of entry selection

Current implementation have few major flaws:

it does not take into consideration margins of ScrollView items
it does not take into consideration gaps created between ScrollView items by ItemDecoration
in certain cases, when selectionRatio (value [0.0; 1.0] representing relative selection of an entry) selects an "empty space" (such as aforementioned margin), the entry to be selected in inferred incorrectly


Some addition/fix to profunctors and bifunctors

This PR fix some typos, relax some constraints and add some utilitaries modules (mainly for dealing with Kleisli and Cokleisli Arrow. Because having Profunctor.From_functor_via_clown (etc) seems anoying.

Localisation of kind expansion into modules (Kleisli and Cokleisli) seems a good idea.
Thanks!


Covid Glimpse ideas

This is a new page that I have working on lately. The goal of this page is to display information on the current COVID-19 situation.
Some of current delivery customers choose to use our service because they are worried about safety due to COVID-19. I realized it might be good idea to include some relevant information on COVID-19 in the shop page as well. Right now, I have been following this awesome Youtube video on building up some related functionalities. This is what I have built so far.

There's certainly a lot more we can do to make this page do better. Any ideas or suggestions would be appreciated. If you want to modify this page or add new features, just leave a comment down below and open a new issue describing what you will be doing.
If this is something people are really interested in working on, I will potentially categorize this as one of the milestones! ♥

Hi Hans, my name is Hayder and this is the first time for me to contribute. I would start with this issue Altho there is not much to do except maybe change the color of the numbers of each. For example, the number of Recoveries in green, and Infected in red. Also, I would remove the Deaths section just my opinion.

Also, I am thinking maybe show the data as chart or graph.

Hey Hans, I'm praveen basically im a photo editor & simple animator so i have the Ideas for your website designs. I would start with the background design and color. Use the contrast colors or a GIF background with a colour of "Blue Shades, White, and Red-Violet" because ,It uses pale blue as the background color to provide a stark contrast. The contrasting colors add more depth to the isometric web design. and use more animation like mouse animations like click sounds etc,.. and  graphs for covid  cases increase decrease while clicking each classification like infected , recoveries and death .use background aminations like TESLA - "https://www.tesla.com/".... I think this will helps u to make a contrast websites.
use these images for references!

https://www.windy.com/ like this show the live ,past, and future with time line option.


Can build android-studio on windows ?

Hello.Sorry to bother you,but I have to say that you just did a very amazing work!!! That's very awsome!!!
I want to know if you have any idea about how to build android studio on windows. I've been busy with this for a few weeks,but  nothing worked out. It is a great luck to see your amazing work！！ So I see a glimmer of hope. If you have any idea about this, hope you can give some Precious suggestions.
And I have a question, if I follow your steps，deleting some module fileurl ,then I can get right build result on windows?  Because When I build on windows, always meet error about bazel ......
Thanks a lot and very sorry to bother you.

Hi @ligenvin,
I received your e-mail but suggest to keep our discussion here publicly available.
Yes, my colleague was able to use this instruction and successfully build Android Studio on Windows. I'm not using Windows so I can't help you with the proper Docker setup, especially when WSL is used. If you have problems with the bazel - please post error message here. Maybe I can help you.
In most cases you don't need Android Studio. You will not be able to get full Android Studio installation using my scripts anyway. It works only up to the point where you can load sources into IDEA and built them or debug them. Using IDEA Community sources is much easier, it also has the same Android Plugin as Android Studio. In IDEA's master branch current version of Android Plugin is EE Canary 9. I always recommend using IDEA sources whenever possible.
If you still sure that you need Android Studio - what is the bazel error you are facing?

Oh,I'm happy to receive your reply.I used a installed Community Edition IEDA 2022.2.1(not build from source) and download android studio source code (studio-4.1-dev) from their website. When I begin to build this android studio source code,there are some errors about bazel as below shows:

Do you know how to solve this?

I google it and find some solution，such as delete bazel dependency.But there are still some errors as below shows:

@ligenvin, it does not work this way. Bazel needs native toolkits (gcc, ndk, android-sdk and so on), and it needs some concrete versions of that toolkits. Also AOSP team does not publish some parts of their repository (e.g. /tools/vendor/google/android-ndk or /prebuilts/studio/sdk). After deleting private parts (I assume you probably did something like this: https://github.com/andrei-kuznetsov/android-studio-builder/blob/c34cceaca09bd528cf3f10da3c5982f17bd463a1/patches/tools.base.bazel.patch) you also need to recover missing parts. I don't have a list of patches required for 4.1 (Why do you need that old Android Studio?). The earliest I have is for 4.2 (https://github.com/andrei-kuznetsov/android-studio-builder/blob/c34cceaca09bd528cf3f10da3c5982f17bd463a1/patches/runpatches.sh). Besides the patches you'll also need toolkits like gcc-6, g++-6, python3 and others. See https://github.com/andrei-kuznetsov/android-studio-builder/blob/c34cceaca09bd528cf3f10da3c5982f17bd463a1/Dockerfile.
I recommend two things:

Forget about official guide. It is not updated for years. Follow my guide, and use Docker to build everything as explained in the readme: https://github.com/andrei-kuznetsov/android-studio-builder/blob/main/README.md .
Don't use studio-4.1. Use something more up-to-date. If EE Canary 8 is too new for you, maybe 2021.2.1 will work? https://github.com/andrei-kuznetsov/android-studio-builder/commits/main

Thanks!
I just follow Official website tutorials ,then the studio-master-dev I download is 4.1.But the newer version of Android Studio is ok for me.I will try what you said.Thanks a lot for your help.

Sure. studio-master-dev is dead. They renamed it to studio-main a year ago o so.

Oh, I see.And I think they may update their document.I just follow their tutorials but met a lot of mistakes. It is so lucky to get your   help. Have a good day!

Thanks to your outstanding work.We followed your guide and worked it out! With the help of your work,we finally built Android Studio on Windows as below show:

Oh, we have solve this problem,the reaseon is that we lack some packages,so we copy from a preinstalled version of Android Stuido.Thanks again ! Have a good day


created Simple Options Handler

added tests for correct Allow header
added private methods to add headers to allowed methods list

I could add in GET to all of the methods that you could potentially GET from but I'm not sure if that is the right approach?

I'd like to hear why YOU think this may or may not be a good approach


Adding a new rebus

Summary of Changes

Added a new rebus.

Pull Request Test Coverage Report for Build 403

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.617%

Totals

Change from base Build 400:
0.0%

Covered Lines:
154

Relevant Lines:
175

💛  - Coveralls

Nice, thanks 👍


Error when using whitespace tab style options with HTML formatter

(Original issue 1028 created by pmoura on 2014-09-11T13:35:42.839982+00:00)
Using the Pygments 1.6 release with Python 2.7, using the option "-F whitespace:tabsize=4" with the HTML formatter results in the following error:
*** Error while highlighting:
TypeError: expected a character buffer object
(file "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pygments/formatters/html.py", line 709, in _format_lines)

(Original comment by tshatch on 2014-10-07T05:02:08.661077+00:00)
Not quite enough to reproduce, but this appears to be a str.translate call where the values of the dict are all unicode; I can trivially get this TypeError by doing this on a regular string object.

(Original comment by pmoura on 2014-10-07T13:41:30.749038+00:00)
It should be easily reproduced with any file with a recognized extension. For example:
#!bash

$ touch foo.pl
$ /opt/local/bin/pygmentize-2.7 -F whitespace:tabsize=4 -o foo.html foo.pl

*** Error while highlighting:
TypeError: expected a character buffer object
   (file "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pygments/formatters/html.py", line 709, in _format_lines)

(Original comment by birkenfeld on 2014-10-07T14:29:51.541414+00:00)
Closes #1028: fix filters to return Unicode strings
→ <<cset 685abf49e997>>

(Original comment by pmoura on 2014-10-07T14:33:12.584547+00:00)
Thanks for the quick fix :-) Looking forward to the next Pygments release.

(Original comment by tshatch on 2015-11-01T00:22:20.444787+00:00)
Issue #1136 was marked as a duplicate of this issue.


Fixed links in header

Which is the relevant ticket ?
Closes #9
What does this PR do ?
Fixes the link in header logo
URL for Live instance:
https://arbisoft545.zendesk.com/hc/en-us

@Wassaf-Shahzad
Awesome. Looks good to me! 👍


Bytes

Added a module for basic handling of bytes at the lib level, a structure analogue to a FlatBuffer is proposed here to handle single-byte modification and appending.

nitunits are missing :)

Added nitunits, and addressed all the concerns of @privat

+1

Reroll with import of standard hoping tests will pass this time

+1


Too much vertical spacing

used to be...

@c-bik which browser are you using? and some steps in particular? I was unable to reproduce filtering on firefox, chrome and IE 11.

@acautin
Browser: Chrome (Version 59.0.3071.29 (Official Build) beta (64-bit)) in Windows 10
No special steps (just start DDerl and filter... window on any table)
Clearing browser cache or Incognito Window didn't help!

@c-bik, that browser says beta, maybe something messed up there, or it could be Windows 10 sadly I don't have access to it.

@acautin
seems like its ok with Edge:

If you can confirm with ZHH Mgmt, I will then close this issue.

@c-bik confirmed in Zhh


Avoid setAttribute for CSP compliance

We're using materialize-sass with Rails with a strict CSP config. Using setAttribute('style', style) causes issues the style policy. element.style = ... is CSP compliant and could be used instead. jQuery offers .css(), which uses the aforementioned .style.
This PR changes the problematic setAttribute calls to using a setStyle() function which checks if jQuery is loaded and uses .css() if possible or falls back to setAttribute.
It might be possible to enhance the function to fall back to looping through the styles and directly applying them with .style = , but I didn't look into that yet.
Using this PR with appropriate settings in Rails' content_security_policy.rb at least fixed all the CSP conflicts we had with materialize-sass.

This... might have been the wrong repo, sorry…


Some inventory items do not appear in search

Whilst looking at https://datalab.odbx.science/edit/jmas-1-2, I noticed that the Na2O2 precursor does not appear in the search when adding it as an ingredient, yet it appears in the xlsx dump from cheminventory.

Pretty sure this is closable, can't reproduce.


Implement generic inlining with trait bounds

This should be the last PR related to inlining generics (unless a new bug shows up)
Now, when dealing with trait bounds, instead of the unit type (), a unit struct defined by the user that implements all the given traits must be passed into #[ts(unit_type = "path::to::Struct")]
Example
#[derive(TS)]
struct Unit;
impl ToString for Unit {
    fn to_string(&self) -> String {
        "".to_owned()
    }
}

#[derive(TS)]
#[ts(unit_type = "Unit", export)] // exporting now works!
struct Generic<T: ToString> {
    foo: T
}

#[derive(TS)]
struct Container {
    g: Generic<String>,

    #[ts(inline)]
    gi: Generic<String>,

    #[ts(flatten)]
    t: Generic<u32>,
}

Fixes #214

@NyxCode what do you think of this approach?

I like that this fixed the issue, but to be honest, I kinda dislike that we have to fix it.
If it's okay with you, I'd like to hold off on merging this for a bit - Specifically, I'm hoping that maybe improving the way we export types could have a trickle-down effect, maybe making this change unnecessary. Really not sure about that though, and if it doesn't, I'd be happy to merge this.

Don't worry about it, this one can wait

In that case I'm gonna convert this to a draft to signify that it's on hold

Rest looks good!
I'm still trying to come up with a better, more descriptive name, but I havent yet.

I'm still trying to come up with a better, more descriptive name, but I havent yet.

Yeah, I also struggled with that, still have no idea what would be a good name for this attribute

Also, I just realized what would probably fix this, if that feature ever comes to stable - !, the never type! It's the perfect default, since it's a subtype of every trait!
#![feature(never_type)]

struct X<T: ToString>(T);

type XX = X<!>;

(there's no point in waiting for it, the Rust PR is from 2016)

(there's no point in waiting for it, the Rust PR is from 2016)

That's a huge shame, I'm sure ! would help with some other stuff too

Maybe std;:convert::Infallible works?

Maybe std;:convert::Infallible works?

Nope, just tried it

WOW, wait a minute!

Okay, never mind 😅
I just realized that ! is already kind of a thing, but only in specific places as far as i can tell - for example in function return types.
So I tried to use some traits to "extract" the "!" from there. But unfortunately, it seems like it currently does not have the property that it's a subtype of everything: playground

Yeah, I think there is currently no way to do this properly

So I've spent some time experimenting, but I haven't been able to come up with a nicer way to handle generics.
If you believe there's a need for this feature[^1], I think we can merge this. I'll leave it up to you if adding the feature is worth the added complexity.
There's no docs so far - maybe a sentence at the very bottom of the docs for the TS trait might be usefull.
Or maybe this is something which would be better placed in a wiki, not sure.
[^1]: In general, I believe having trait bounds on structs is a mistake in almost all cases. Even std::collections::HashMap doesn't have any trait bounds on it. This StackOverflow answer describes that nicely.

In general, I believe having trait bounds on structs is a mistake in almost all cases. Even std::collections::HashMap doesn't have any trait bounds on it. This StackOverflow answer describes that nicely.

Interesting! I haven't been writing Rust for very long (just under a year now) so I never realized trait bounds on structs were such a bad idea.

I'm fine with closing this PR in light of that! It really adds a lot of complexity with the totally useless unit type that must implement all traits required by the struct and the attribute name is not helpful at all.
If people really end up needing it, someone will open an issue

Or maybe this is something which would be better placed in a wiki, not sure.

I love the idea of creating a wiki! It's usually dificult to find docs for derive macro helper attributes through LSP definitions (which don't autocomplete, only telling you the mispelled word isn't accepted) or docs.rs, which only allows you to document the trait itself, making something like #[ts(...)] hard to find (although documented under the TS trait, docs.rs doesn't generate a separate page or links for it, we have to manually do that, which is easy to forget)

@NyxCode I have started writing the wiki! Check it out :D


0.67.0b1 - BT mac addresses get prefixed with double underscore in known_devices.yaml

Home Assistant release with the issue:

0.67.0b1
Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):

Docker
Component/platform:

Bluetooth device tracker
Description of problem:
BT devices are getting a double underscore in the mac address in known_devices.yaml
mac: BT__00:11:22:AA:BB:CC
This causes an error as it thinks that the second underscore is part of the mac address
Correct mac address...
mac: BT_00:11:22:AA:BB:CC
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:
When removing the rogue underscore, or making any other changes in known_devices.yaml the device is rediscovered again, with a double underscore again

Confirming the same behaviour running 0.67b1 in hassio. When a device is added as a new one, it shows up in known_devices.yaml as:
my_iphone:
  hide_if_away: false
  icon:
  mac: BT__40:xx:xx:xx:xx:DB
  name: my iPhone
  picture:
  track: true

If I change my existing device but suffix BT with an extra _ the bluetooth device tracker gives the following error:
[homeassistant.components.device_tracker.bluetooth_tracker] Error looking up Bluetooth device
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/homeassistant/components/device_tracker/bluetooth_tracker.py", line 92, in update_bluetooth
    result = bluetooth.lookup_name(mac, timeout=5)
  File "/usr/lib/python3.6/site-packages/bluetooth/bluez.py", line 59, in lookup_name
    raise BluetoothError ("%s is not a valid Bluetooth address" % address)
bluetooth.btcommon.BluetoothError: _40:xx:xx:xx:xx:DB is not a valid Bluetooth address

Can, however confirm, that bluetooth tracker works if you delete your previously added BT-devices and let it recreate them in known_devices.yaml.

This issue is directly related to this PR: https://github.com/home-assistant/home-assistant/pull/12458

Appears to be working correctly again in 0.67.0


Removed unused code that issues a GL_INVALID_OPERATION

In order to use the SDL_GL_GetAttribute, the OpenGL context has to have been created and make current. Which at this point has not been. But the code is never used so it should be removed.

I believe it is related to https://github.com/OpenSAGE/OpenSAGE/issues/388

I believe it is related to https://github.com/OpenSAGE/OpenSAGE/issues/388

SDL_GL_CreateContext is called on line 260, and that function should both create the context and make it current. So I'm not sure that the description here is accurate. However, the code is indeed unused, so there's no reason to keep it around. I'd like to understand whether this is actually fixing the underlying issue that OpenSAGE is facing, though.

SDL_GL_CreateContext is called on line 260, and that function should both create the context and make it current. So I'm not sure that the description here is accurate. However, the code is indeed unused, so there's no reason to keep it around. I'd like to understand whether this is actually fixing the underlying issue that OpenSAGE is facing, though.

Now that you mention, you're right, it should have been made current, I'll do some investigation if this is a MESA issue or something else.

Now that you mention, you're right, it should have been made current, I'll do some investigation if this is a MESA issue or something else.


apiwrappers.py refactor

this opens the bot up to eventually get rid of the apiproxy dependency

TODO for manual testing:

[ ] twitch global emotes refresh correctly
[ ] twitch channel emotes refresh correctly
[ ] ffz global emotes refresh correctly
[ ] ffz channel emotes refresh correctly
[ ] bttv global emotes refresh correctly
[ ] bttv channel emotes refresh correctly
[ ] User-Agent is sent correctly
[ ] safe browsing api can filter - https://testsafebrowsing.appspot.com/
[ ] Stream data is refreshed correctly (live status, title, game, etc.)
[ ] VOD links, thumbnails, etc are refreshed/fetched correctly
[ ] set game and title work
[ ] followage and followsince work, including the 2 minute cache
[ ] download logo in web interface works correctly

oof

[ ] migration to move bot token
[ ] migration to move webtwitchapi to twitchapi in config


Remove Babel as a dependency

Before a commit is made, the dist files should be compiled with Babel.
That will remove any non-dev dependencies.

Done!


[HW] BruceW2099

Name: Alan Kwok
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/alan-k-0593a0109/
Attach the homework step I screenshots below:

-- image removed by team --

Attach the homework step II screenshots below:

Hey Alan Kwok, great job! Congrats!  Please delete your token from the Astra UI so there are no security issues.

Good morning David,
Thank you very much. I will go in and delete it right now.
Thanks... Alan
------ Original Message ------
From: "David Jones-Gilardi" @.>
To: "datastaxdevs/workshop-spring-stargate"
@.>
Cc: "Alan Kwok" @.>; "Author"
@.>
Sent: 9/7/2021 9:48:09 AM
Subject: Re: [datastaxdevs/workshop-spring-stargate] [HW] BruceW2099
(#131)

Also, I had to remove one of your images because they contained your
Astra token. Please delete your token from the Astra UI so there are no
security issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datastaxdevs/workshop-spring-stargate/issues/131#issuecomment-914321482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADQ3YSHDCK5RZIIPOECZD3TUAYJZTANCNFSM5DPQNPNQ.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Force repository and Vfs update after current branch reset

... as original IntelliJ dialog implementation do

Whooops...

(WTF since it doesn't fail on fix/opened-machete-file-update branch)


Rating, RatingStar

Introducing the Rating component, a cheeky lil extra for dessert.
🙏

@jxom the theming is incomplete -- would like your feedback and guidance before implementing as per existing standards. Cheers.


mwc713 taking too long message appears on login

It seemed to appear very soon after clicking login. Times should be adjusted upwards.

Change mwc713 stopping timeout from 3 to 8 seconds. Reason: stopping tor listener might take more time

Successfully verified with the latest build - https://ftp.mwc.mw/nightly_builds/06-19-2020/ , hence closing this ticket.


Update index.md

Grammatical Errors

Thank you for helping to improve the documentation. We're unable to approve all these chages because:
Our style guide stipluates that "the" must not come before "Common Data Model".
Our style guide stipulates that spaces must not surround dashes.
Thanks,
Deon


Analytic path reverse

This seems to work well enough in my mind. I didn't make it very clean (I'm outta time, ATM).  I also didn't test this vertex by vertex to the old method, so it may not be the same. But the patterns (fractal tree 1/2, and penrose) look the same in the preview.
I am pretty sure it isn't reversing an enormous amount of the path, because I am only keeping one reversePath open at a time.

I am pretty sure it isn't reversing an enormous amount of the path, because I am only keeping one reversePath open at a time.
This is fantastic. Less code and much faster. I'm going to up the limits and add a few more shapes to this PR if you don't mind (ones I had to remove due to poor performance!). Or I can add to a new PR...

You can add it to this PR. I have a few minutes, I am going to clean up the code in common/lindenmayer.

I am pretty sure it isn't reversing an enormous amount of the path, because I am only keeping one reversePath open at a time.

I've been playing with this more, and at higher iterations, and its adds a 3X multiple to distance.
Penrose (5 iterations)
Dijsktra's: 98K
Reversing path: 283K
Penrose (6 iterations)
Dijsktra's: 270K
Reversing path: 778K
I like the way to more complex patterns look, but they will take forever to draw on the table. Thoughts?

I've been playing with this more, and at higher iterations, and its adds a 3X multiple to distance.

Is that just with the penrose? I was focusing more on the fractals, but the penrose has those loops (it's a bit hard to wrap my head around). I am not sure which way dijkstra is finding around the loop.

I like the way to more complex patterns look, but they will take forever to draw on the table. Thoughts?

IDK. I like a snappy UI, but I also spend more time in developing sandify than I do watching patterns play out. Maybe I should play the pattern each way and see which one looks better.


Update microsoft-365-mailbox-utilization-service-alerts.md

I've changed all reference of 'alert' to advisory (or advisories) and updated the duration from four days to seven days.  I originally created this article with Mark Johnson.

None


customized monthviewDisplayEvent.html template and slide action

customizing the month cell with

{{view.dates[row*7+col].label}}

. . .

when I slide the month to the next (or prev), the cells shows only the day number and not my custom template

@bartsimp Could you provide the snippet that how you specify the template option?

actually I checkout the branch 0.3.0 and used the demo code.
I have not done changes except edit dist/js/calendar-tpls.js as follow:
`+++ dist/js/calendar-tpls.js
@@ -1285,7 +1285,7 @@ angular.module("templates/rcalendar/month.html", []).run(["$templateCache", func
angular.module("templates/rcalendar/monthviewDisplayEvent.html", []).run(["$templateCache", function($templateCache) {
$templateCache.put("templates/rcalendar/monthviewDisplayEvent.html",

"{{view.dates[row*7+col].label}}");

"{{view.dates[row*7+col].label}}. . .");
}]);
`
and open in browser demo/index.html

@bartsimp I tested and can see the extra div under the day number.
Could you use browser dev tool to check if it's not loading template file from some where else?
Also check the extra div in the template is not hidden?
Actually you don't need to modify the template directly, version 0.3.0 allows you to directly pass an external template url.

I try to explain with this screenshot

as you can see, while I slide from september to october the template is not used (instead is applied when the slide action is finished)
In addition there is a problem with heights . the events overlap to calendar.
It is just my problem?

@bartsimp
This is because I don't apply the same template to the previous/next month.
It simply just renders the day number. Because there's no event associated, there's no need to apply a template on it.
You can apply some customised css to make the height of the cell are the same in previous/current/next month.
If you want to also display the same content in previous/next month, you can modify the below code snippet in the month.html.

                    {{view.dates[row*7+col].label}}

Regarding your second question, this is there's a css class "event-detail-container" applied to event detail, it has the margin-top set to 262px, which is the height of the day number section. Since you add more contents in the day number sections, the height is changed. You can adjust the margin-top of event-detail-container to accommodate the new height.

thanks!


Unable to type in receive codes

All of the text input fields in the app seem to break when I try to input a string matching [0-9]+-[a-z]+. For instance, if i start typing 1-a, the a never appears in the text box, all I can type is 1-. This is obviously especially problematic for the recv code field, but the send text field has the same issue.
I'm using version 1.0.8 on a Pixel 3a running LineageOS 18.1.

(The app isn't completely unusable, I can still paste in recv codes from the clipboard, but I need to type them into a text field on a different app to copy them.)

We made a change in 1.0.8 to automatically convert spaces to dashes (-) to make it easier to input codes. I wonder if that is causing an issue for you here. Could you try testing with 1.0.6 to see if that version also has the issue? https://github.com/psanford/wormhole-william-mobile/releases/tag/v1.0.6

Just checked, 1.0.6 works fine. I'm puzzled why fields other than the recv code field would also be affected if the space conversion were the cause of the bug, but it's definitely due to some change between 1.0.6 and 1.0.8 :/

1.0.7 also works for me.

That makes it very likely this change is somehow responsible: https://github.com/psanford/wormhole-william-mobile/commit/07a623f353b1ed4c4c6ff593ec863eb03a3fc466
Not sure why this would behave differently on lineageOS vs stock Android 12. I'll see if if can dig in a bit more when I have some time. Worst case, we can revert that change.

LineageOS 18.1 is based off Android 11 fwiw. Given that text fields other than the recv code field have the same bug, I'm not sure reverting the change to that one field will actually make a difference, but maybe if I have time at some point I'll try and set up a build environment and do some testing of my own. I appreciate you looking into this!

I get the same behavior on my Pixel 6 Pro (running GrapheneOS, which is based on Android 13).
I tried downgrading to 1.0.7, but that only lead to crashes. 1.0.6 allows me to type in the code easily, and works.


eq and ne Not working

I have created a pipeline build variable "svctag" and assigned its value to blank (empty string). Now trying to execute below script inside a job in yaml file. Every time it prints "svctag is not blank". Not sure what I am doing wrong.
Please help.
- ${{ if eq('$(svctag)', '') }}:
   - script: echo 'svctag is blank'
- ${{ if ne('$(svctag)', '') }}:
   - script: echo 'svctag is not blank'

Try with variables['svctag'] or variables.svctag instead of '$(svctag)'

If the pipeline is defined as the following, then it will show 'svctag is blank' as @guibirow said.
trigger: none

variables:
  svctag: ''

steps:

- ${{ if eq(variables.svctag, '') }}:
   - script: echo 'svctag is blank'

- ${{ if ne(variables.svctag, '') }}:
   - script: echo 'svctag is not blank'


Add method and prompt to fix a failing test

Is your feature request related to a problem? Please describe.
As a key building block for fixing failing tests we need to pass a test failure message to the LLM and have it generate a solution.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Example from manual process:
Generated test code =
def test_link_tests():
    session = create_autospec(Session, instance=True)
    mock_function = create_autospec(CodeFunction, instance=True)
    mock_function.function_name = "mock_func"
    mock_test = create_autospec(CodeTest, instance=True)
    mock_test.test_name = "test_mock_func"
    session.query(CodeTest).all.return_value = [mock_test]
    session.query(CodeFunction).filter_by(
        function_name="mock_func"
    ).first.return_value = mock_function
    link_tests(session)
    session.query(CodeTest).all.assert_called_once()
    session.query(CodeFunction).filter_by.assert_called_with(function_name="mock_func")
    assert mock_test.tested_function is mock_function
    session.commit.assert_called_once()

Original function code =
def link_tests(session: Session):
    """Link tests to the functions they test."""
    tests = session.query(CodeTest).all()
    for test in tests:
        function_name = test.test_name.replace("test_", "")
        function = (
            session.query(CodeFunction).filter_by(function_name=function_name).first()
        )
        if function is not None:
            test.function_id = function.id
            session.commit()

Failing test message =
FAILED                      [  5%]
tests/test_code_database.py:62 (test_link_tests)
<MagicMock name='mock.tested_function' id='140314999669520'> != <NonCallableMagicMock name='mock.query().filter_by().first()' spec='CodeFunction' id='140315001491840'>

Expected :<NonCallableMagicMock name='mock.query().filter_by().first()' spec='CodeFunction' id='140315001491840'>
Actual   :<MagicMock name='mock.tested_function' id='140314999669520'>
<Click to see difference>

def test_link_tests():
        session = create_autospec(Session, instance=True)
        mock_function = create_autospec(CodeFunction, instance=True)
        mock_function.function_name = "mock_func"
        mock_test = create_autospec(CodeTest, instance=True)
        mock_test.test_name = "test_mock_func"
        session.query(CodeTest).all.return_value = [mock_test]
        session.query(CodeFunction).filter_by(
            function_name="mock_func"
        ).first.return_value = mock_function
        link_tests(session)
        session.query(CodeTest).all.assert_called_once()
        session.query(CodeFunction).filter_by.assert_called_with(function_name="mock_func")
>       assert mock_test.tested_function is mock_function
E       AssertionError: assert <MagicMock name='mock.tested_function' id='140314999669520'> is <NonCallableMagicMock name='mock.query().filter_by().first()' spec='CodeFunction' id='140315001491840'>
E        +  where <MagicMock name='mock.tested_function' id='140314999669520'> = <NonCallableMagicMock spec='CodeTest' id='140315001501920'>.tested_function

tests/test_code_database.py:76: AssertionError

Example answer:
The test is failing because it's trying to assert that mock_test.tested_function is equal to mock_function, but in the actual function link_tests, what is happening is that the function_id attribute of the test object is being set to function.id.
So, the test is not aligning with what the function is actually doing. To fix the test, you instead want to assert that mock_test.function_id is equal to mock_function.id.
Here is the corrected test:
def test_link_tests():
    session = create_autospec(Session, instance=True)
    mock_function = create_autospec(CodeFunction, instance=True)
    mock_function.function_name = "mock_func"

    # Give a unique 'id' attribute to the mock_function object
    mock_function.id = 1

    mock_test = create_autospec(CodeTest, instance=True)
    mock_test.test_name = "test_mock_func"
    session.query(CodeTest).all.return_value = [mock_test]
    session.query(CodeFunction).filter_by(
        function_name="mock_func"
    ).first.return_value = mock_function
    link_tests(session)
    session.query(CodeTest).all.assert_called_once()
    session.query(CodeFunction).filter_by.assert_called_with(function_name="mock_func")

    # Assert mock_test.function_id equals the id of the returned function object
    assert mock_test.function_id == mock_function.id
    session.commit.assert_called_once()

In the real function implementation, you're linking a function to a test by setting the function_id attribute of the test object to the id of the function object. Therefore, in the test, you should also be checking the function_id attribute of the mock test object.

Process:

Run pytest on just the test function first - revise until passes. Then run the whole tests (after commit?).

Prompt:
I have an LLM that has generated the test code for the function below. The test is currently failing with the failure message as below. Can you suggest changes to the test (preferably) or the function (less preferable) to get the test passing?
* Please do not use fixtures at this stage, so that the test code functions independently of other definitions.

Generated test code =
```python
{test_code}

Original function code =
{function_code}

Failing test message =
{failure_msg}

For classes:
I have an LLM that has generated the test code for the function below. The test is currently failing with the failure message as below. Can you suggest changes to the test (preferably) or the function (less preferable) to get the test passing?

Please do not use fixtures at this stage, so that the test code functions independently of other definitions.

Generated test code =
def test_CodeFunction___repr__(mocker):
    """
    Test the __repr__ method of the CodeFunction class.
    """
    mock_id = 1
    mock_function_name = "test_function"
    mock_code_function = mocker.MagicMock()
    mock_code_function.id = mock_id
    mock_code_function.function_name = mock_function_name
    result = mock_code_function.__repr__()
    expected_result = f"<CodeFunction({mock_id}, {mock_function_name})>"
    assert result == expected_result

Original function code =
class CodeFunction(Base):
    """Model for a function in a Python file."""

    __tablename__ = "code_function"
    id: Mapped[int] = mapped_column(primary_key=True)
    function_string: Mapped[str] = mapped_column()
    function_name: Mapped[str] = mapped_column()
    file_path: Mapped[str] = mapped_column(nullable=True)
    doc_string: Mapped[str] = mapped_column(nullable=True)
    vector: Mapped[str] = mapped_column(nullable=True)
    test_status: Mapped[str] = mapped_column(nullable=True)
    imports: Mapped[str] = mapped_column(nullable=True)
    # Can be "function" or "test"
    code_type: Mapped[str] = mapped_column(nullable=True)
    is_test: Mapped[bool] = mapped_column(nullable=True)
    is_function: Mapped[bool] = mapped_column(nullable=True)

    # Foreign Key to class
    class_id: Mapped[int] = mapped_column(ForeignKey("code_class.id"), nullable=True)
    code_class = relationship("CodeClass", back_populates="functions")

    def __repr__(self):
        return f"<CodeFunction({self.id}, {self.function_name})>"

Failing test message =
test_code_database.py::test_CodeFunction___repr__ FAILED                 [100%]
test_code_database.py:127 (test_CodeFunction___repr__)
"<MagicMock id='139870366645408'>" != '<CodeFunction(1, test_function)>'

Expected :'<CodeFunction(1, test_function)>'
Actual   :"<MagicMock id='139870366645408'>"
<Click to see difference>

Functions required:

Get test code

Use the code database
CodeTest.test_string

Get associated function code

CodeTest.tested_function

Run individual test

See code below

Check test status
Get test failure code

Returned as a string from the code below

Send off prompt and extract revised test code
Replace previous test code with revised test code

Use the existing test writing code
Maybe needs a function to parse with AST and make a substitution when writing back

Repeat

import subprocess
import sys

def run_specific_test(test_identifier):
    """
    Runs a specific pytest test, returns the stdout as a string, and indicates pass/fail.
    :param test_identifier: String identifier of the test (e.g., 'test_module.py::test_function')
    :return: Tuple containing stdout from the test run as a string and a boolean indicating pass (True) or fail (False)
    """
    result = subprocess.run(['pytest', test_identifier], capture_output=True, text=True)
    test_passed = result.returncode == 0
    return result.stdout, test_passed

if __name__ == "__main__":
    # Example usage: python run_test.py 'test_module.py::test_function'
    if len(sys.argv) < 2:
        print("Please provide a test identifier.")
    else:
        test_identifier = sys.argv[1]
        output, passed = run_specific_test(test_identifier)
        print(output)
        if passed:
            print("Test passed! 🎉")
        else:
            print("Test failed. 😢")

Commit after each individual passing test with the test name in the commit message.

For classes - copy in Class definition, init, then ... then class method being tested.

Main functions are all in code_management.test_writer.

General agent function method:

populate/update/clean code DB - agent.core.populate_db with with_reset=True for fresh DB build
generate missing tests - agent.core.generate_tests_from_db
run tests to update test status - code_management.test_writer.run_all_tests_and_update_status

Continued in #21 - this issue is addressed and the branch has been merged with main.


[Enhancement] Add a "New" section in the home page

Enhancement
Add a "New" section in the home page, like the "Open Source" section.
New softwares are being displayed in this section, such as Datalya for instance.
Other informations
No response

Feature was added.


Only include some files in the cargo crate

@homu: r+ f38462d

:hourglass: Testing commit f38462d with merge e80b40c...

:sunny: Test successful - status


Typo in Note at bottom of page

Note
The daily period that can be also seen in the chart wasn't found becasue the sampling is too coarse (12h bin size), so a daily period of 2 bins is below the minimum period size of 4 points, required by the algorithm.

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

ID: f4393ed0-5f93-4715-1b12-a7c6885c3a29
Version Independent ID: 6ff5d7db-4a52-54a4-b99b-afacda4e1e67
Content: series_periods_detect() - Azure Data Explorer
Content Source: data-explorer/kusto/query/series-periods-detectfunction.md
Service: data-explorer
GitHub Login: @shsagir
Microsoft Alias: shsagir

@hvoerman Thanks for pointing this out. The document is updated, and changes will go live soon.


Documentation setup

Setting up documentation on the new system.

Current plan is to merge this asap, and then for future updates create smaller PRs that can be handled individually.


Support for AWS::Serverless::LayerVersion

There exists support for AWS::Lambda::LayerVersion, but not AWS::Serverless::LayerVersion (though it's almost the same, save for Content vs ContentUri. A Layers property would also have to be added to the Function resource.
I can create the pull request myself once I get around to setting up a local dev repo.

Added it into serverless.py. Thanks for the issue.


[Good First Issues]: Review Elasticsearch Sink

Documentation Review
Motivation
The best way to learn about an open-source project starts with reading its documentation. In order to better know about Vanus and improve the quality of Vanus documentation, we're looking for OSS enthusiasts to review our docs.
Details
Review and test connectors documentation
Requirements
When you test:

Use the playground to test GitHub Source
Run the connector according to the documentation
Test every link in the documentation

When you review the documentation:

Grammar check
Wrong capitalization
All image are working
You can understand every step.

If you have any questions or comments, please feel free to leave a comment in this issue.
How to change it:
Comment to Claim this issue.

Fork the docs repository.
Create a branch Review GitHub Source .
Review document [Elasticsearch Sink(https://github.com/linkall-labs/vanus-connect/tree/main/connectors/sink-elasticsearch/README.md)
Change: Use the playground terminal to test the connector, Review document.
Make a PR.

I'll take ownership of this issue. Thanks for bringing it to our attention. I'll keep you posted on my progress.


Default values for Optional fields in WebSocketURI

Hello!
Due to the nature of dataclasses in Python, for a field to be truly optional, it must be provided with a default value, not only Optional annotation. Otherwise, when creating WebSocketURI сlass instance without login and password, we will get an error like:
TypeError: WebSocketURI.init() missing 2 required positional arguments: 'username' and 'password'

Judging by the code, this was an accidental oversight, because checking for None value of the name and password already exists and the rest of the project code has default values for dataclasses.
Thanks in advance.

Thank you!


emit_pointers_for_null_types does not work on default 'database/sql' go package

Version
1.20.0
What happened?
While generating the go queries using the emit_pointers_for_null_types flag in a configuration file v2 nothing append and database/sql nullable types are used instead.
I think I've found where in the code that might happen and will propose a fix alongside this issue.
Thanks for reading me.
Relevant log output
No response
Database schema
No response
SQL queries
No response
Configuration
version: "2"
sql:
  - engine: "postgresql"
    queries: "query.sql"
    schema: "../database/schema/1.2.9.sql"
    gen:
      go:
        emit_interface: true
        emit_enum_valid_method: true
        emit_all_enum_values: true
        emit_pointers_for_null_types: true
        package: "database_sqlc"
        out: "."

Playground URL
No response
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

#2590 To fix this issue

As discussed in the linked pull request, this is surprising but working as intended. By default, code that uses database/sql and lib/pq can't support scanning into pointer fields. In the future we can add a warning when this parameter is set to true and lib/pq is being used.


refactor(kit)!: rename TuiFilterComponent to TuiFilter

Closes #

identifiers-to-replace?

it's already listed there as "TuiFilter"


RAI dashboard page won't jump out.

On Pycharm it shows these two lines after calling ResponsibleAIDashboard(rai_insights), no other error shown. When clicking on the link it says "localhost refused to connect".
       **ResponsibleAI started at http://localhost:5000
       Process finished with exit code 0**

Then I try to run the code from "responsibleaidashboard-housing-classification-model-debugging.ipynb" both on Pycharm and Jupyter notebook and saw these two lines too:
        ResponsibleAI started at http://localhost:5001/
       <raiwidgets.responsibleai_dashboard.ResponsibleAIDashboard at 0x7f4a9569bd90>

What could be the reason that the dashboard page won't jump out?

Hi @DinaChen! The dashboard requires a running Flask server in the background. As soon as the execution ends the server shuts down and you can't connect to it anymore. In Jupyter notebooks you have the kernel running which keeps it alive. In PyCharm you can also do that, you just need to make sure the execution keeps going. For example, you might prompt for user input so that it doesn't end. I hope that helps!

I came up with the same issue that both Pycharm and Jupyter notebook won't render RAI dashboard. In jupyter notebook, when i click the link http://localhost:5000, i got HTTP ERROR 403

Again, in the code you posted initially it seems like the script just finishes. The dashboard is interactive so you need the Python process to keep running otherwise it won't work.
If it's a notebook then make sure the kernel keeps running. I don't know if Pycharm renders it in the IDE itself because I don't use Pycharm. You can certainly open a browser and enter the URL there. Does that work?

@chendingyan, did the workaround that @romanlutz suggested work for you?

Closing this issue since there's no response. Feel free to reopen if you've tried the suggestion. Please also provide details on your OS, Python version, and relevant Python package versions if you reopen.


Optimism Network Support

OP Contract

contract address: 0xdE44d3fB118E0f007f2C0D8fFFE98b994383949A
RPC: https://mainnet.optimism.io OR https://chainlist.org
Ark network key: OPTIMISM-MAINNET

About the ABI
In the current ark-ui repo, there is only one ABI (Vyper contract ABI) used to create the contract instance. However, the OP contract has been coded in Solidity instead of Vyper. Therefore, the UI has to switch to the Solidity ABI only when the selected network is Optimism.

Both of the ABIs: https://github.com/decentldotland/ark-network/blob/main/src/utils/evm/abis.js
Where to switch ABIs based on the network https://github.com/decentldotland/ark-ui/blob/main/src/utils/eth.ts#L114
Where ark-ui store the current ABI: https://github.com/decentldotland/ark-ui/blob/main/src/assets/ArkNetwork.json
example of how Ark Node does it: https://github.com/decentldotland/ark-network/blob/main/src/utils/evm/ethers.js#L40

integrated


OS image installation

We should consider to not advise to use client on the OS image but install it after OS deployment.
https://docs.microsoft.com/en-us/configmgr/core/clients/deploy/deploy-clients-to-windows-computers#BKMK_ClientImage

Hi @mvagafonov, can you provide more detail as to why? It's technically a valid method. The current "important" note makes it somewhat invalid for an end-to-end scenario with ConfigMgr. But if you use ConfigMgr to create the image and then use that image for another purpose, then installing the client in the image might be applicable.

Thanks again for the feedback. We recently moved this content to a new GitHub repository, so this issue is no longer tracked with the original article. As part of that process, we reviewed our backlog of feedback, and made some tough decisions on items that we don't think we'll realistically ever address. If you're passionate about this specific feedback, please refile it on the current article, now under docs.microsoft.com/mem/configmgr.


MINOR: fix failing Streams system tests

In #5850 the output that prints the current Kafka version was reformatted (space before : was removed):
Old format:
Kafka version : 2.2.0-SNAPSHOT

New format:
Kafka version: 2.2.0-SNAPSHOT

This breaks some Streams system tests that grep the logs for this line.

Triggered system test: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2086/
Call for review @guozhangwang @bbejeck @vvcephei

StreamsBrokerBounceTest system test failed -- seems to be unrelated to this fix though. All StreamsUpgradeTest passed. Are you ok if I merge this and we tackle the failing test separately?

Fine with me

@bbejeck Merge to trunk. You can rebase your PR to pick up the fix.

Thanks @mjsax!


Derive folder structure from properties

Thank you for this plugin.
I have an idea, but i'm not sure if it is compatible with your vision.
When implemented, the folder structure would be derived from YAML properties.
The user would define multiple derived paths such as [genre]/[decade]/[year] and [genre]/[author] and would be able to quickly switch between them.
If the YAML property is an array, there would be N virtual folders

Hi. This is an interesting idea, but I don't understand yet how it is better. The option you proposed has a right to exist, but it is a different plugin structure. Now each virtual folder should be a real existing note. When searching for a note via the command, the full path is already displayed. I was thinking about including the full path in the search, and not just the note name.

I don't see how this is an improvement, because you can already do this by setting the parents. Make the parent a year or author note. Then you can change the structure by changing the categorization note.
It also changes the nature of the plugin. The plugin is meant to display a hierarchy of linked notes. I love this functionality because Obsidian has no great method of visualization of linked notes. I find the graph view to be lacking.
If you want to display a hierarchy, there's likely other add-ons which handle this.


Consider using autograd.make_vjp() instead of ceviche.jacobian()

Hey there, cool package! I wanted to suggest an alternative implementation that may be more efficient.
It looks like javiche is currently using ceviche's jacobian() method, which (I think) may be less efficient if @javiche.jaxit is ever applied to a function whose output is not scalar-valued. The reason is that ceviche's jacobian() method loops over the output basis vectors to construct the Jacobian, and you typically don't need the explicit Jacobian when calculating VJPs. An example of where this would matter is if you just applied @jaxit to a function that returns the field distribution, but performed the loss function calculation in terms of the field distribution in JAX, calling jax.grad or jax.value_and_grad on the combination.
Below is a sketch of a more direct approach that maps autograd's make_vjp() function to JAX's VJP mechanism.
Given an autograd function, f_ag(*args) -> np.ndarray we can wrap it into a function, f(*args) -> jnp.ndarray. This is not ceviche-specific; it can be used for any autograd function with multiple inputs (*args) and a single array output, though it could be generalized to support multiple array outputs as well.
import jax
import jax.numpy as jnp
import numpy as np
import autograd

def as_numpy(x):
  def as_numpy_map(a):
    if isinstance(a, jnp.ndarray):
      return np.asarray(a)
    else:
      return a
  return jax.tree_util.tree_map(as_numpy_map, x)

def as_jax(x):
  def as_jax_map(a):
    if isinstance(a, np.ndarray):
      return jnp.asarray(a)
    else:
      return a
  return jax.tree_util.tree_map(as_jax_map, x)

@jax.custom_vjp
def f(*args):
  return as_jax(f_ag(*as_numpy(args)))

def f_fwd(*args):
  args = as_numpy(args)
  argnums = tuple(i for i, _ in enumerate(args))

  def f_ag_tupled(*args):
    ans = f_ag(*args)
    if isinstance(ans, tuple):
      return autograd.builtins.tuple(ans)
    else:
      return ans

  vjp_f, ans = autograd.make_vjp(f_ag_tupled, argnums)(*args)
  return as_jax(ans), jax.tree_util.Partial(vjp_f)

def f_rev(vjp_f, g):
  g = as_numpy(g)
  return as_jax(vjp_f(g))

f.defvjp(f_fwd, f_rev)

I was thinking about adding something like this to ceviche_challenges, but had not gotten around to it.

Thanks for the suggestion!!
I'll have a deeper look soon.
Do you want to open a pull request for proper attribution?
Or would you rather want me to integrate it with jaxit directly?
Regards JD

I do not really have time to implement this and open a pull request right now. You're welcome to adapt the sketch above if you have time.

Top. I'll do that!

A the proposed change has been merged to master I'll close the issue.


To include the new APIs to few legacy forwardings

In the CONFIG file , the following operation-server shall be included as an input fc-port of the forwarding
ServiceRequestCausesLoggingRequest,
ro-2-1-0-op-s-bm-013 - /v1/dispose-remainders-of-deregistered-application
ro-2-1-0-op-s-bm-014 - /v1/inform-about-preceding-release
ro-2-1-0-op-s-bm-015 - /v1/update-client-of-subsequent-release
ServiceRequestCausesLtpUpdateRequest,
ro-2-1-0-op-s-bm-015 - /v1/update-client-of-subsequent-release
ro-2-1-0-op-s-is-013 - /v1/notify-embedding-status-changes
ServiceRequestCausesLtpDeletionRequest,
ro-2-1-0-op-s-is-002 - /v1/deregister-application

Services added as input in CONFIGfile and ServiceList.


[dv] enable/disable tlul_assert correctly

Relates to #6597.
cip_base_vseq__tl_errors.svh should smartly enable/disable set_tl_assert_en() with the proper path argument such that only the tlul_assert_device bound to the corresponding TL interface is disabled.
Right now we blindly disable/enable ALL tlul_assert devices that have been created, which may not be as clean.

closing this via https://github.com/lowRISC/opentitan/issues/16742


Loosen tuple definition in utilities.epoch

In order to facilitate usage of functions via JuliaCall from R, we could loosen the \tau = Tuple{Numeric,Numeric} condition to ??

Can you give an example of what's going wrong here / what the desired behavior is?

Can you give an example of what's going wrong here / what the desired behavior is?

yes, still using issues as a note to myself ...utilities.epoch
function epoch(data::Array{T,2},tbl::DataFrame,τ::Tuple{Number,Number},sfreq;eventtime::Symbol=:latency) where T <:Union{Missing,Number}

Here we enforce a tuple, but in R's JuliaCall it is not straightforward to define tuples (because they dont exist in R afaik). Thus I want to allow lists here too.

yes, still using issues as a note to myself ...utilities.epoch
function epoch(data::Array{T,2},tbl::DataFrame,τ::Tuple{Number,Number},sfreq;eventtime::Symbol=:latency) where T <:Union{Missing,Number}

Here we enforce a tuple, but in R's JuliaCall it is not straightforward to define tuples (because they dont exist in R afaik). Thus I want to allow lists here too.

So in that case I would just add another method that has tau as an array which then immediately dispatches to this variant:
epoch(data::Matrix tbl::DataFrame, τ::Vector,sfreq; eventtime::String="latency") = epoch(data, tbl, Tuple(τ...), sfreq; eventtime=Symbol(eventtime))

(not that I also converted Symbol to String for that indirection as well)

So in that case I would just add another method that has tau as an array which then immediately dispatches to this variant:
epoch(data::Matrix tbl::DataFrame, τ::Vector,sfreq; eventtime::String="latency") = epoch(data, tbl, Tuple(τ...), sfreq; eventtime=Symbol(eventtime))

(not that I also converted Symbol to String for that indirection as well)

I like it! Makes lot of sense and I will implement it later, thanks

I like it! Makes lot of sense and I will implement it later, thanks


add activesupport-json_encoder

This PR includes the fix proposed by @Stex.
Although for me the tests just run fine.

Thanks for the idea, but I think the issue was fixed.


Generalized position

How about support @position for all rule?

The current string char and enum cannot mark positions.
For enum, just call position directly on all variants, the rust compiler will prompt which variants do not mark the position and report an error.
For string/char, position is also necessary, I want to report which escaped char fails and which identifier is undefined with position.
The hacking I am currently using is
@position
Identifier = identifier:IDENTIFIER;

@string
@no_skip_ws
IDENTIFIER = (XID_START | '_') {XID_CONTINUE};

But I think I can generate the Identifier directly
@position
@string
@no_skip_ws
Identifier  = (XID_START | '_') {XID_CONTINUE};

Leads to
pub struct Identifier {
    pub string: String,
    pub position: Range<usize>
}

Sounds like a good idea, I'll see what I can do.
(Except for @char, those rules are meant to be special and very lightweight)


[gardening] Avoid unnecessary list comprehensions

Rewrite [x for x in y] as list(y)

Tests passed on Ubuntu 14.04.


Add disclaimer about lowercased whitelist

After spending way too much troubleshooting this, a disclaimer stating that the whitelist must contain lowercased versions of usernames should be added.
The source converts all usernames to lowercase before comparing them with the whitelist, meaning that non-lowercase usernames in it are always ignored.

Rather than a disclaimer, might be better to just patch the code to be case-insensitive. I'm looking into updating some things anyway, I'll add this to the list :)

Ah, I assumed there was a reason for lowercasing the nicknames.
However, wouldnt case-insensitivity allow unauthorized users to log in by creating an account with a differently cased name ("whitelisted_name" vs "wHiTeLiStEd_NaMe")?
Not sure how screeps handles that.

The server uses usernameLower for the name checks during registration and during login, so is case-insenstive anyway.
See: https://github.com/screeps/backend-local/blob/master/lib/game/api/register.js#L36

Ah, I see. Yeah, making the whitelist case-insensitive is probably the best option then.


Support and test type serialization

Prior to this PR, Cirq supported serialization of instances of Cirq types, but not the types themselves. This PR adds serialization support for Cirq types, with the format:
{
    'cirq_type': 'type',
    'typename': $NAME
}

where $NAME is the cirq_type of the object in its JSON representation. For type T, $NAME is usually T.__name__, but some types (mostly in cirq_google) do not follow this rule. The json_cirq_type protocol and _json_cirq_type_ magic method are provided to handle this.
It is worth noting that this PR explicitly does not support serialization of non-Cirq types (e.g. python builtins, sympy and numpy objects) despite instances of these objects being serializable in Cirq. This support can be added to json_cirq_type and _cirq_object_hook in json_serialization.py if we decide it is necessary; I left it out of this PR as it is not required by the motivating changes behind this PR (#4640 and sub-PRs).

cc: @mpharrigan

Objects of type type aren’t special in Python. They’re just objects. Why should they be special here?
We can’t support this the “normal” way because we can’t put a _json_dict_ method onto the definition of the Type class, because this is in the Python standard library.
There are some strong analogies:

We special case support for some sympy, numpy, and pandas objects. Their ‘cirq_type’ is namespaced with sympy, numpy, and pandas.
We don’t provide support for some Python objects. Most notably dictionaries with non-string keys. The escape hatch is for cirq classes encapsulating such an object to marshall to list(d.items()) and back. But there’s a variety of Python things we don’t support serializing: e.g. datetime is one I’ve run into.

We should make this work like (1) or like (2). Would love to hear your thoughts, though.

I chatted with @95-martin-orion to explain myself. Some conclusions:
(1) might not actually give us the serialization we want for the use case in mind. A full serialization of python objects of type type might be big, have a whole list of methods, or some other can of worms. That seems to imply that (2) might be more natural for this use case
(2) might be more natural, but in some sense benefits enormously from being able to hijack the existing resolver infrastructure.
@95-martin-orion to investigate whether (1) could be feasible (we aren't familiar with what objects of type type are) and whether (2) could be done by leveraging the resolver infrastructure without hijacking it.

Further investigation following @mpharrigan's comment:
From the Python type object documentation: "each type object stores a large number of values, mostly C function pointers". Given that the contents of a pointer do not persist across (de-)serialization, unless we were to serialize the functions themselves (1) is probably not feasible.
On the other hand, (2) should be possible by using cirq.DEFAULT_RESOLVERS, which is updated by each imported sub-module to include serializable types from that module. However, we would still need the json_cirq_type protocol for types with non-standard cirq_type values.

Thanks for investigating! I think I agree.
I think we'd need to architect the json_cirq_type protocol more carefully in this case, see my comment.

I have demoted type-serialization to a pair of methods which can optionally be used in [_from]_json_dict to serialize types.
@mpharrigan, what comment are you referring to regarding json_cirq_type?

https://github.com/quantumlib/Cirq/pull/4693#discussion_r752428586

Did you consider a class attribute instead of a class method? It could look a little cleaner. I wasn't sure exactly how things work in Python, so I did the following test. All give the desired namespace string:
class MyThing:
    namespace = 'cirq.google'

    def __init__(self, x: int):
        self.x = x

    @classmethod
    def _namespace_(cls):
        return 'cirq.google'

print(MyThing.namespace)
print(MyThing(5).namespace)
print(MyThing._namespace_())
print(MyThing(5)._namespace_())

Did you consider a class attribute instead of a class method?

I did, but (a) protocol methods are the more common pattern in Cirq, and (b) the syntax for class attributes differs between "normal" classes and @dataclasses, since @dataclass uses the class attribute syntax for instance attributes. Taking those into consideration, I felt @classmethod was preferable.


Refresh Not Working

Hi, the refresh functions does not seem to be working, can you please help.
Thank you.

maybe explain why it isn't working in your case, because in my case it is working perfectly, e.g.:
afterSelect: function(values){
                $('#select3,#select5,#select7,#select9,#select11').children('option[value="'+ values +'"]').attr('disabled', true);
                $('#select1,#select3,#select5,#select7,#select9,#select11').multiSelect('refresh');

        }


Module '"sveltekit-sitemap"' has no exported member 'sitemapPlugin'

Hey I am trying to use your lib, but it seems that I can't get started..
This is my vite.config.ts:
import { sveltekit } from '@sveltejs/kit/vite';
import { defineConfig } from 'vitest/config';
import { sitemapPlugin } from 'sveltekit-sitemap';
/** @type {import('vite').UserConfig} /
export default defineConfig({
plugins: [sveltekit(), sitemapPlugin()],
test: {
include: ['src/**/.{test,spec}.{js,ts}']
}
});
but i am getting Module '"sveltekit-sitemap"' has no exported member 'sitemapPlugin'.

You can use import { sitemapPlugin } from 'sveltekit-sitemap/dist/plugin.js'; as a workaround.

Looks like sveltekit-sitemap/dist/index.(m)js does not export sitemapPlugin in the published npm package, even when the dist folder in the repo it does
https://github.com/beynar/sveltekit-sitemap/blob/d35ca45502a3c4cc353a01e1dd888cc4c36f7d34/sveltekit-sitemap/dist/index.mjs#L41


WIP Translate: Design Principles

#4

@carburo, thanks for your review. I applied all of the suggested changes, it was a really long and complex article and even though I read it 3 times it was hard for me to spot any errors.