EthStorage Devs Meeting #110 Agenda

Meeting Info

Dec 31, 2024, 11:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress
BD/Marketing progress

Goal:

EthStorage Mainnet Ready

Updates:

EthStorage was deployed on beta testnet

https://github.com/ethstorage/storage-contracts-v1/pull/125
Boot and mining nodes

Bridge and https ready:

Bridge: https://bridge.beta.testnet.l2.quarkchain.io/
RPC: https://rpc.beta.testnet.l2.quarkchain.io:8545
Explorer: https://explorer.beta.testnet.l2.quarkchain.io

ai16z proposal and meetup: https://docs.google.com/document/d/1JJSBW_27WQDpUlEfngzbJkB6OdCiXFupQgji6cOhVZk/edit?tab=t.0.
Bug: https://github.com/ethstorage/optimism/issues/148

TODO:

Investigate op-conductor
https://github.com/ethstorage/pm/issues/149

Goal:

Fix issues found in WSC L2 beta testnet

Updates:

Pushed a workaround for https://github.com/ethstorage/optimism/issues/148
make a op-program/v0.1 tag to unblock https://github.com/ethstorage/optimism/pull/141
Answer KePing's questions about l2 tx cost calculation.
reviews:

https://github.com/ethstorage/optimism/pull/149

TODO:

inbox e2e test.

WSC L2

Add E2E test for L2 Blob: https://github.com/ethstorage/optimism/pull/142, https://github.com/ethstorage/da-server/pull/9
Faucet: done
Integration test: https://github.com/ethstorage/es-node/pull/351, https://github.com/ethstorage/storage-contracts-v1/pull/126
Add chain ID to p2p filter
Update the dashboard to support multi-chain

Goal:

Prepare WSC L2 testnet launch

Updates:

https://github.com/ethstorage/pm/issues/157
https://github.com/ethstorage/pm/blob/main/L2/devnet_fault_proof.md
https://github.com/dajuguan/op-notes/blob/main/play-op-challenger.py

Goal:

circleci tests

Updates:

https://github.com/ethstorage/optimism/pull/143
https://github.com/ethstorage/optimism/pull/145
https://github.com/ethstorage/optimism/pull/146
https://github.com/ethstorage/optimism/pull/147
https://github.com/ethstorage/optimism/pull/149
TestRequiredProtocolVersionChangeAndHalt
https://github.com/ethstorage/web3url-gateway/pull/43
beta: es-node setup
code reviews
doc on maintaining circleci

TODO:

complete circleci doc

offline meetup with ai16z
world super computer 2 offline workshop, shanghai and hangzhou
prepare annual report

Run the ethstorage node and provide rpc.
ethfs-cli and ethstorage-sdk are upgraded to support beta test network.
Migrate related dapps.
Conduct airdrop testing on the test network.


Promo alt text

Images in our promo card links include alt text, which is the first thing read out by screen readers.
In this context though the alt text is largely irrelevant, and the user has to listen to a detailed description of the image before hearing the title of the thing being promoted (i.e. what the link is for).
It would probably be better to use empty alt attributes for promo images.

@alicerichmond I'd be interested to know if what I described above is something that users find annoying in testing, or whether they like the alt text and don't have a problem skipping over it.

I think that the best practice behind alt text is that if the image doesn't provide any extra context and is purely decorative then it is ok to not use alt text. I have seen the long alt text before the content on a couple of the promo cards around the site and I think removing them might be the best approach.
Not all screen reader users are blind and the extra unecessary words could be really problematic for other disabilities.

I think that the best practice behind alt text is that if the image doesn't provide any extra context and is purely decorative then it is ok to not use alt text. I have seen the long alt text before the content on a couple of the promo cards around the site and I think removing them might be the best approach.
Not all screen reader users are blind and the extra unecessary words could be really problematic for other disabilities.

The DAC accessibility report confirmed this was a problem


typeahead: Add type definitions for shared/js/typeahead.js

We add a .d.ts file for typeahead.js so that people.js has the types when importing it.
Because Zulip Mobile also relies on the shared module, we do not directly convert typeahead.js to TypeScript to avoid transpilation.
This change also comes with some quick refactorings:

Removed the default value to the get_item parameter of triage, since that default is incompatible with the type signature we expect.
Removed is_unicode_emoji. So we don't need a user-defined type guard to type narrow Emoji, which might have undefined emoji_code.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@N-Shar-ma @aero31aero can you help review this?

Also @gnprice @chrisbobbe thoughts on the question of whether we should avoid changing files under shared/ to be fully implemented in TypeScript to avoid transpilation? I'm not sure how to think about that potential constraint.

@PIG208 this one may take a while to answer the above questions, so please be encouraged to work on other conversions in parallel.

Also @gnprice @chrisbobbe thoughts on the question of whether we should avoid changing files under shared/ to be fully implemented in TypeScript to avoid transpilation? I'm not sure how to think about that potential constraint.

The path forward on that is #22586.

Posted one question, otherwise lgtm.

I rebased the PR to incorporate changes related to prioritise_realm_emojis. I have also reworked the type annotations so that we translate the original .js file into a typescript given that #22586 has been merged.
The test cases are updated to always ensure the presence of reaction_type on the emoji object to be passed into any of typeahead's helpers. This assumption holds because emoji_collection is the only source of emoji objects where we call the emoji matcher/sorter from.

Rebased to resolve the merge conflicts with 431cd8b10a1a020994433fdeca0ea500c183d153.

We don’t need to change the parse_unicode_emoji_code API. We can type it parse_unicode_emoji_code(emoji: UnicodeEmoji): string (with the UnicodeEmoji type factored out of Emoji), and have callers narrow the argument before the call with a .reaction_type === "unicode_emoji" check. This seems simpler than checking for undefined after the call.

Rebased to revert back to the original parse_unicode_emoji_code API that takes a string, and passes emoji.emoi_code once we type narrow Emoji to UnicodeEmoji. Factoring out UnicodeEmoji now is not necessary, but it's better for readability.
I also updated codepaths that have the is_realm_emoji property missing on the emoji object. However, I am not sure if is_realm_emoji is necessary, since it is basically reaction_type == "realm_emoji" || reaction_type == "zulip_extra_emoji"

Merged, thanks @PIG208!

However, I am not sure if is_realm_emoji is necessary, since it is basically reaction_type == "realm_emoji" || reaction_type == "zulip_extra_emoji"

Yes, well spotted. It looks like that is_realm_emoji property was added very recently. It's unnecessary, and worse than that, it is wrong: the name says it should be true only for realm emoji, but in fact it is also true for the Zulip extra emoji.
Because it changes the API of the shared typeahead code, it looks like it probably will also break updating that code in mobile, though I haven't tested that.
It'd be good to take it out and replace it with looking at reaction_type.


Search Widget issues

Did some tests on Chrome in http://demo.tout-culture.s3-website.ca-central-1.amazonaws.com/
Here are 5 issues:
Number of events on results page does not match number of results in search widget (using keyword)

In search widget field, type "danse". Stay in events tab.
'See all results' shows a count of 12. Click see all results
15 events are displayed on the page (and only 8 have the Danse event type).
###Number of ogranizations on results page does not match number of results in search widget (using keyword)
In search widget field, type "association". go to the organizations tab (you can go first or after you enter the search)
'See all results' shows a count of 4. Click see all results
5 organizations are displayed on the page.

Results don't match number in search widget when you search by date (in events)

In search widget field, enter August 23. Stay on the events tab.
'See all results' shows a count of 14. Click see all results
22 results are displayed on the results page.
Some of the results are workshops, for example Danse Créative (Parent-enfant). You can go into the CMS and see that the event type for this event is Atelier | Danse. A second example is "Danse en ligne". You can see the the event types for this one are Atelier | Danse and Cours. (cours is "course" in English)
Go back to the search widget field, keep August 23 as your date. Go to the atelier (workshop) tab.
'See all results' shows a count of 8. Click see all results
8 results are displayed, so it looks like this one is okay.

total number of organizations doesn't match result number

in the search widget show the organizations. See all results' shows a count of 48. Click see all results

95-ish results are displayed on results page

"Centre d'action culturelle de la MRC de Papineau" organization is showing an address

in the search widget show the organizations. the last one in the list "Centre d'action culturelle de la MRC de Papineau" displays an address
click "Centre d'action culturelle de la MRC de Papineau". There is no address on the results page.

Clicking a tab twice displays a no results message

click any tab in the search widget (Events, Workshops, Organizations). results are displayed
Click the same tab again. The "no events available" messages appear in the widget

@saumier I fixed the issue of "Clicking a tab twice displays a no results message" and pushed it to develop. In my local and in demo site (http://demo.staging-tout-culture.s3-website.ca-central-1.amazonaws.com/index.html) I am able to see the change. But in https://minimal-footlight-search-widget.s3.ca-central-1.amazonaws.com/v2/test-dist.html the issue still persists. Because of this, I haven't updated the production yet. Is this an issue with the bucket? if not I will look into what went wrong from my side.

@SyamBabu-M Is this still an issue?


add tooltip for dashboard learn more button

#22656

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 41.647%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30343

Relevant Lines:
67449

💛  - Coveralls


No default body tag.

Found a nice edge case for ya 🗡
Opening:
<link href="https://fonts.googleapis.com/css?family=Lilita+One" rel="stylesheet">

<style>
  html {
    background-color: #f1f1f1;
    font-family: 'Lilita One', cursive;
    animation-duration: 5000ms;
    animation-fill-mode: both;
    animation-iteration-count: infinite;
    backface-visibility: visible;
    animation-name: colors;
    animation-timing-function: linear;
  }

  @keyframes colors {
    0% {
      transform: rotate3d(0deg);
      background-color: #f10a75;
      color: #fed024;
    }
    50% {
      background-color: #fed024;
      color: #f10a75;
    }
    100% {
      transform: rotateY(360deg);
      background-color: #f10a75;
      color: #fed024;
    }
  }

  body {
    margin: 0;
    text-align: center;
    line-height: 100vh;
  }

  body:after {
    content: "Coming Soon";
    font-size: 5vh;
  }
</style>

will display a spinning loading message, while after it has been ran through staticrypt it will not due to no default body tag being generated.

Are you sure this is the correct repo? That code is not part of staticrypt

@mini-eggs Do you mean you encrypted that snippet and when you unlock it it behaves differently than if you open your snippet directly with a browser?
A full html page is expected (with html and body tags) to be decrypted then loaded correctly in browser, if you just use a fragment you might encounter different display behaviors. Most browsers try to fix HTML with missing tags on load page but you shouldn't really rely on it.

Yes.
This is pretty up in the air to fix since it's not a best practice to rely on browsers fixing broken HTML, just thought it would be interesting to throw your way. If this tool requires/expects there to be valid HTML the no issue here

If broken HTML is required for this issue to show up this feels slightly too edgy to me, people are a bit responsible for the display of what they encrypt :) Since the placeholder starts with <html><head>... it should be clear we expect all tags.
If you have a simple fix in mind feel free to open a PR and I'll look into it. Meanwhile I'm closing for now, thanks for testing and reporting !


Add link to relocatable package in issue template

Developers would appreciate link to relocatable package when we create an issue. E.g.:
http://downloads.scylladb.com/unstable/scylla/master/relocatable/2022-03-17T14:10:37Z/scylla-debug-x86_64-unified-package-5.1.dev.0.20220317.c45050895403.tar.gz for Scylla Version: 5.1.dev-20220317.c45050895403
To achieve that we can reuse dev's solution (of course using some python library):
build_id=076f513b6143670def988c7626389b270411f8f7; curl -s -X POST http://backtrace.scylladb.com/index.html -d "build_id=$build_id&backtrace=" | sed -En -e "/^$build_id/s:^.*http:http:" -e "/^http/s:</a>::p"

I can volunteer in implementing it.

A good idea, I think for responsiveness reasons this should be done as an AJAX endpoint that is called when Issue Template is first opened


Minor adjustments on calendar print stylesheets

Week view:

[x] Can we add current week to the title? "My timetable Feb. 26 - Mar. 03"
[x] Can we expand calendar to full screen / page height?

** Month view**
Good as it is
Day view

[x] Can we expand to 100% height?
[x] Can we include the date of the day show in the title 23 February, 20145?

Per discussion we'll leave out the 100% height comments from the PR

Reassigning to @Coenego for review.

Minor issue left. Reassigning to @bp323

Reassigning to @Coenego for review.

Coverage remained the same at 100.0% when pulling 59d5f7d419b1930c40202a2890a6f088864c6825 on bp323:issue-257 into f8259e2e4e3df452b4d8f4e1d24573c3a0fe5d01 on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling f4427a33cbb0a60f2e657eedafd5b845fcfef99a on bp323:issue-257 into f8259e2e4e3df452b4d8f4e1d24573c3a0fe5d01 on CUL-DigitalServices:master.


Fix unique constraint generator for informix

Hello Nathan,
I have made some changes to the change log generator to correctly make unique constraints for informix.
Thanks.

Thanks!

Hello Nathan,
This issue seems to be resolved in 3.4.2, but has resurfaced in 3.5x.  Can you pull the same fix for informix unique constraints into 3.5x?  This is Jira ticket number: CORE-2381.
Thanks.


collapsed section failing

I'm testing the hacker theme to the limits. Not everything works but 1 thing I really want is the collapsed section to work. See my site at hacker-theme-test

The section is colapsable but the things within the section is not cssed. This is the "code" in the .md file
CLICK ME

#### We can hide anything, even code!

```python
   print("Hello World")
```

I also got this error, probably not from this colapse
GET https://azlancoding.github.io/assets/images/black-dot.png 404

I tried removing  it doesn't help

I found a workaround but it's very difficult when there is a lot of stuff
CLICK ME

We can hide anything, even code!

print("Hello World")

CLICK ME
We can hide anything, even code!
print("Hello World")


Update cert requirements to allow new roots in 1903

Update cert requirements to allow new roots in 1903

@PPacentMSFT : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.

#sign-off


wrong balance after witness transfer

Trying the usual transfer series via rgb-sandbox, using rgb-wallet from master (beta 6 + PR #211 for human-readable values), I run into an issue. After a witness transfer, the balance for the receiving wallet is incorrect.
Here are the steps to reproduce the problem, using 2 opret wallets:

issue a NIA contract on wallet 0
send some assets from wallet 0 to wallet 1 using a witness invoice

The transfer completes successfully, accept included. Here are the wallets' outpoints and contract states after the transfer:

wallet 0Height     Amount, ṩ    Outpoint
bcrt1qrjeyktuuchvks9x7vwf58tq5mjkduq5ghqzmma    &9/1
105         99997600    5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b:1

 Owned:
  assetOwner:
    value=2000, utxo=bc:opret1st:9cc679405e0aa2ad4e79559aa3123831241f429875544af7d84880134a421525:1, witness=~ -- owner unknown
    value=100, utxo=bc:opret1st:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b:0, witness=bc:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b -- owner unknown
    value=1900, utxo=bc:opret1st:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b:1, witness=bc:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b

wallet 1Height     Amount, ṩ    Outpoint
Loading descriptor from wallet wallet_1 ... success

Owned:
  assetOwner:
    value=2000, utxo=bc:opret1st:9cc679405e0aa2ad4e79559aa3123831241f429875544af7d84880134a421525:1, witness=~ -- owner unknown
    value=100, utxo=bc:opret1st:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b:0, witness=bc:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b -- owner unknown
    value=1900, utxo=bc:opret1st:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b:1, witness=bc:5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b -- owner unknown

It looks like wallet 1 doesn't see the UTXO from the transfer's tx (txid: 5f8491a78476b278bb981b2545b890e73847436558a40b69fe9957c2a2f5886b) and thus sees no owned allocations, leading to a balance of 0, instead of the expected 100.
Printing the address used by the invoice I get bcrt1qc7huwmgyuq6snsku0papx7grafde9fldhg73rz, which does not appear in the generated PSBT.
Here are the wallets' data dirs:
wallet_data.tar.gz

So basically it doesn't recognize tweak applied to the first tapret output, which is owned by wallet 1... Working on it

After patching rgb-std to master for https://github.com/RGB-WG/rgb-std/issues/220 I have re-tested for this issue as well and it gone, too.
I don't think it had to do with tweaks, as both wallets are opret in this case and I use no special construction (just the regular CLI invoice/transfer commands). Still, I fail to see how switching to rgb-std master branch should have fixed this problem.
I'll leave this open for now so we can further discuss if needed, but the original problem is solved.

Sorry, I have somehow read opret as tapret. Indeed, this is very strange...

Seems we can close this now


Wi-fi features stopped and wifi task constantly triggers the watchdog. (IDFGH-26)

Environment

Development Kit:      none
Core (if using chip or module): ESP32-PICO-D4
IDF version ESP-IDF v3.1-75-gfbcfc86
Development Env:		Eclipse
Operating System:		Ubuntu
Power Supply:         external 3.3V

Problem Description
Task watchdog being triggered for the first time after ~10h. 5s before task watchdog had triggered, it logged the following error:
E (58930960) wifi: ieee80211_scan.c 1395
Log output shows the following information:
Task watchdog got triggered. The following tasks did not reset the watchdog in time:

IDLE (CPU 0)

Tasks currently running:
CPU 0: wifi
CPU 1: uart_rx_task
All CPU 0 being executed is wifi task. CPU1 task changes among other tasks on each log output. Application runs without wifi features and task watchdog trigger every 5 seconds.
Steps to reproduce
I could not reproduce this bug yet, but I am aware it could happen other times as my application should always be powered on.

Hi @tmedicci, it's our bug, we will debug it.

there are is a case may cause this issue, that is no beacon operation channel in the beacon or probe response, if it that, station can not judge which channel to follow, so we add assert here, if you want to reporduce the phenmenon, maybe you should find the router which do not have DS information in the beacon

@tmedicci we have do a fix of this serval days before, if you still remember the commit-id of you IDF, I can verify the root cause of your problem

Hello @XinDeng11 , my IDF commit is:
commit fbcfc869e2339be33f5c7258f079aa862efef7a9
Merge: 4107361 bb48077
Author: Jiang Jiang Jian<EMAIL_ADDRESS>Date:   Thu Sep 20 18:44:18 2018 +0800

    Merge branch 'bugfix/fix_the_bug_create_socket_fail_v3.1' into 'release/v3.1'

    lwip: fix the bug that failed to create socket (backport v3.1)

    See merge request idf/esp-idf!3311

Hope this can be helpful. Thank you!

Hi @tmedicci could you help to check whether this issue still exist in latest release/v3.1?

Hi @liuzfesp , it never happened again. In fact, I have seen this bug only once. I think the issue could be closed. Thank you for your attention.


Using Struct Tags to Control Encoding

How can we convert go native types into etf.Term via reflection? Is a reflection api included?
E.g. slices of map[string]interface{} to etf.Term?

etf.Ters is just an interface type. You don't need it. What exactly are you trying to do?

I think I was looking for how go Structs can be converted to an etf.Map via annotations/tags just like the built-in json lib and aws-ion marshaller.

We use *string extensively and ergo seems to crash when any nested value is a *string.

Hmm. It's a very interesting case. Actually, the value should be dereferenced during the encoding process. I'll take a look closer.

created issue https://github.com/halturin/ergo/issues/65

I think I was looking for how go Structs can be converted to an etf.Map via annotations/tags just like the built-in json lib and aws-ion marshaller.

ETF encoder encodes any struct to the map using field name as a key.
as an example this kind of value
type test struct {
	A string
}

will be encoded like this
#{'A' => "example"}

may I ask you to check out my fix of #65? just merged into the master

Thanks for the quick response. It's amazing what this library could do if it takes off in the Elixir/Golang community.
We are using it to work with AWS services where the quality of the rust-native drivers are not yet production ready (to use with Rustler NIFs). Management (like in many startups) is comfortable with Golang, but we know how much more productive a true functional language with pattern matching can be.
Will test out your fix over the weekend.
I'm also comfortable with golang so if you are open to contributions, let's formulate some guidelines.


electrum: 4.1.5 -> 4.2.0

Description of changes

new upstream release: https://github.com/spesmilo/electrum/blob/master/RELEASE-NOTES

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build electrum

The icon is not displayed for me. Do you know why?

@linsui can you please retest with electrum 4.2.1?

I tested 4.2.1 and the icon works now. I don't know why...

@GrahamcOfBorg build electrum


Ensure PostgreSQL database 'log_checkpoints' flag is set to 'on'

Possible check to target the following resource

[ ] google_sql_database_instance

closing, tracking in spreadsheet


Questions about running the registration code

Hi, I have used the AbdomenMRCT dataset and tried to run your alignment code but did not get the desired results, is there a problem with my settings? My settings are as follows:
    result_path = './results'
        convex_adam(
            ckpt_path='../model-weights/best_val_net_G.pth',
            expname='demo',
            lambda_weight=0.75,
            grid_sp=2,
            disp_hw=1,
            selected_niter=80,
            selected_smooth=0,
            grid_sp_adam=2,
            ic=True,
            result_path=result_path,
            fixed_image=fixed_image,
            moving_image=moving_image,
            use_mask=False,
            # fixed_mask=fixed_mask,  # 提供 fixed mask
            # moving_mask=moving_mask,  # 提供 moving mask
            fixed_minclip=-450,
            fixed_maxclip=450,
        )

Thanks.

Hi @BhLee98,
Thanks for your interest!
The AbdomenMRCT experiments used the ROI masks provided by the dataset/challenge (in masksTr). Set use_mask=True and provide the mask paths. Without the masks, all the hyperparameters for the underlying convexadam solver would need to be reoptimized with another gridsearch.
Also, I just pushed an addition to the CLI to optionally warp a segmentation label map and compute Dice for convenience, please pull the latest commit.


Add add the linked data uris to the right hand side box

Itatti URI
VIAF / ULAN / etc

Also, the museum links for the Botticelli and Filippino schools

add British Museum links to right side bar

Should be complete


Removed editor note

@fawazsiddiqi, there were two broken links within the Note for Step 1. Unless you have the updated links available, I recommend just removing that part of the Note.

Agreed and approved sorry about that


NonASCII Chars preventing seeing more text since 1803 release

This bug-tracker is monitored by Windows Console development team and other technical types.  We like detail!

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.17134.1]

What you're doing and what's happening: (Copy & paste specific commands and their output, or include screen shots)

Setup:

Using Windows command prompt (not windows powershell)
ssh to VM running kafka and run the command kafka-console-consumer where there is AVRO binary encoded data on the topic.  So the initial data coming back will have binary encoded information then text then binary.

What's wrong / what should be happening instead:

In Pre-1803 I would get text back that looked like this:
▒▒▒▒XBusiness▒B0167BDC260D94E927C0C3BF6A88110819DE5555A2182249EE26BA1FEBF8187B9DFCD53019293B9A50Vm_KeyRotation_APP_CLT_LEG_ENT_EMAIL@6BDC0D3295D86D66B1CC2CAD8836A1CD▒fa76d7a3c1d88d0ede64cd484a971e0386421dab088ca36b1ca1ca559e94286eeccfd71373f8b9d54fb45b087855569e68c59ccf5db5805453321e6c821ec1eb▒▒▒▒▒X

In 1803 release I get nothing back.

I expect to see the same output as I saw in the Pre-1803 release as that is the common thing I see in every other tool I utilize when viewing binary data like that.

@iguyking Sorry this got lost in the backlog. Is this still happening for you? Just printing that text to the Terminal seems to work fine in newer builds of the Console and Terminal:

It might be an issue with ssh - if it's not working, which version of ssh are you using?


Jupitermoons units problem

Reported by email:

I have found something that is bound to create an error in the positions of all moons. The computation of the latitudes, B, uses coefficients ( 0.0006393 etc) that are radians, not degrees. Therefore the factor p must be omitted.

Absolutely correct, thank you to the reporter.  Fixed with ci caac145083e91dbf123957a24b0098f445ce3d38.
The issue points out another thing that was done wrong though.  This bug was causing incorrect results my reaction was to put the incorrect results in the output comment of the example function but add other inline comments indicating I noticed a problem.  An issue filed here would have been appropriate, in addition to the inline comments.  Anytime I can't reproduce Meeus' results, that should justify an issue filed here.  Not sure if there are more of these lurking.  It's been too long since I wrote the code and I don't remember.


Added support for generating ipk package instead of simple tarballs.

Use create_ipk instead of create_tgz in main to switch between modes.

Can you add a command line flag to choose between tar and ipk output? Default to tar.gz.
As discussed, should also populate the config file, but apart from that the produced IPK looks ok to me


authorizedScope can be/is an array of arrays here

Not sure if this is a bug higher up?

@ikb42 can you confirm that this is fixed by #307?

Can't confirm 100% but I am not having issues here using #307 code, will close.


Initial refactor of Household Naming - introduce Domain classes

Draft PR to be used for review and initial feedback of Household Naming refactoring work.
Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Thanks for all of the feedback @vmartinov-sforg and @force2b !  I'll try to get another round of updates in tonight/tomorrow morning.

Replaced by PR that is a child of the feature branch here: https://github.com/SalesforceFoundation/NPSP/pull/6223


test pr do not merge

test pr do not merge

Coverage increased (+0.01%) to 86.804% when pulling b3edd40a45111fc725a47bc311eb02699415ac9d on test-build into 2823f5c993dc10fe8188e47fc0691b26631a6739 on master.


limit the iCal output to the departments visible to the user who generated the link, see #1748

It should not be possible for a user to generate an iCal link that shows absences that she is not normally able to see in the system.
This patch adds a permission check to the iCal export function, so that the "whole company" view is only available to OFFICE, for all other users, the output is limited to members of departments that are accessible according to the normal permission checks.
With this, the function can be safely turned on for non-OFFICE users.
It is still possible to share a "whole company" view by having an OFFICE user generate the link and distributed out-of-band to the intended recipients.
closes #1748

Thanks for your pr, but this is intended and therefore the company calender must be activated for all other users by someone with e.g. the office role, because it shows more than they would see in the application.


Feature request: Avatar support

Phasedragon pointed me to this project and it's incredible, thank you for it!
I've been picking up animator-as-code recently, and also want to script building avatar resizes for multiple variants of my avatar. Ideally, I would kick off a build for these and manually upload as needed (for example, 8 variants x 8 sizes would probably be too many uploads, but having them ready to upload and be available in short order is sufficient).
Is this something possible within the scope of VRBuildHelper?

Once i get a full stable release of the current (world) version out there it will be more of a priority for sure, but currently i'm not working on it. I might spend some time before that on setting up an avatar uploader in VRChatApiTools though, as 95% of the work for that is already done. You would still have to set up some basic code to export the assetbundles, but that should be pretty easy (look at BuildHelperBuilder.cs, for worlds its very similar).

@yewnyx
Avatar support in the API is being tracked here https://github.com/BocuD/VRChatApiTools/issues/6

Hell yeah! Thank you very much for the heads-up.


Tracking of clicks on external links (in a generic way)

Options

Any click on body/app → all <a>'s → check external? → log: url+href+parent-element?

on each Markdown-element?


test: multiple slashes on font-name property

fixes #439

I don't think those tests are right.
https://mothereff.in/font-family#0\5FAE\8F6F\96C5\9ED1
https://mothereff.in/font-family#0\\5B8B\\4F53

@andyjansson why? we just drop quotes because their two and put one \

@andyjansson oh, i see, will fix it in near future

Close in favor, i just add invalid tests, all works fine :+1:


Fix agent build command

The origin one will emit fatal: destination path '/deepflow' already exists and is not an empty directory.

This PR is for:

Documents

Fix agent build command
The origin one will emit fatal: destination path '/deepflow' already exists and is not an empty directory.

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


#DRAFT/WIP: Update readme instructions

Provide a more granular description of setup steps need to run Opticks in esi-shell with docker.

@buddhasystem Maxim, do you have anything to add?

Not at that point. I do agree with your prior point that most users should be able to understand how to run a real GPU test, -- thanks.

@buddhasystem @plexoos I am merging this because this only deals with installation with docker. I will open a separate PR for examples.


Using Redux with Wechaty overview

I have added Using Redux with Wechaty under Tutorials. The page is divided into some subsections as shown below:

The PR contains the updated submenu and Overview page. In later PR's we''ll include tutorials for respective subsections.
The overview page looks like this:

@proudofsimin I have made the suggested changes.

Tag #891

@lijiarui I have made the requested changes

Please fix image issue here.

Actually, this image credit section was originally there on the page. So while updating it, I got confused with the image of ducks used above. I thought it was referring to that one.
I think this image doesn't make sense here. Should I remove it?

I think this image doesn't make sense here. Should I remove it?

Please fix image issue here.

Actually, this image credit section was originally there on the page. So while updating it, I got confused with the image of ducks used above. I thought it was referring to that one.
I think this image doesn't make sense here. Should I remove it?

I agree with you, remove the credit is a better solution.

@lijiarui I have removed the image credit.

@shwetalsoni Please resolve the conflict and then I will merge this PR.


fix json.Unmarshal missing precision

fix json.Unmarshal missing precision

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.徐文平 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Could you provider a test case?

I pr it in commando_test.go, func Test_sortJSON.

Codecov Report

Merging #405 (cf01210) into master (1c81252) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##           master     #405   +/-   ##
=======================================
  Coverage   84.41%   84.42%
=======================================
  Files          48       48
  Lines        3254     3256    +2
=======================================
+ Hits         2747     2749    +2
  Misses        379      379
  Partials      128      128

Impacted Files
Coverage Δ

openapi/commando.go
80.33% <100.00%> (+0.16%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c81252...0b090d5. Read the comment docs.


Clean up POM

This PR adds several improvements to the project's pom.xml:

Set encoding to UTF-8
Update outdated Maven plugins
Use Java compiler --release option

I used nexus-staging-maven-plugin for the first time yesterday and my experience wasn't the best to say the least.
I started with the latest at the time, 1.16.10, and later downgraded to 1.16.9 which seemed to work.
Let's try 1.16.11. Hopefully it's more stable. I'm not sure what's going with the churn there over the last few days.

I feel you … I can also revert that change if you want to? Optimizing release-related components can be discussed separately in #7.
Also, let me know if you want me to rebase this PR. ✌

I'll merge and reevaluate on next release.


Answer with the error message when sending an audio to the bot

Currently, if you send a voice message to the bot, it will answer with its info message instead of the error (as it does when it receives an invalid file to convert):

Probably the issue is due to the current implementation of the method TelegramRequest.hasFile():
public boolean hasFile() {
    return message.hasPhoto() || message.hasDocument() || message.hasSticker() || message.hasVideo() || message.hasAudio();
}

EDIT: the same happens sending a video message to the bot, it should be handled as part of this ticket as well.

Hi @marcofranzoni, would you like to look into this issue?

I couldn't say no to issue #42!

Hi @marcofranzoni, do you have any update on this regard?
Let me know if I can help

Hello @marcofranzoni!
Recently the project was upgraded to handle video conversion via FFmpeg (#68) and I think it would be great to allow video message conversion within this issue.
To wrap up, this requirements are:

sending a voice message to the bot should result in its error answer
sending a video message to the bot should result in a successful conversion of said video


Make CLI in line with intended default for increment_micro

@carstencodes I think this would fix #326,
but I'm not certain about any side effects.
A different solution would be to use store_const
and have different None handling, see e.g.
https://docs.python.org/3/library/argparse.html.
Again, not sure of side effects.

Hmm the fact that the tests still pass also suggest there's a missing test case somewhere..

Yeah, tests are not fully covered yet.
Currently, changing the command-line argument would introduce a breaking change.
Sorry for being a little cautious about this, but I might find a workaround using meta-vars...
Give me a day or two

No problem, take your time

As far as I could tell, the CLI is setting the increment_micro option so the defaults deeper within the codebase are just overwritten

Ah, as the smallest supported python version is 3.9, the argparse.BooleanOptionalAction is available (cf manual):
import argparse

parser = argparse.ArgumentParser()
parser.add_argument('--foo', action=argparse.BooleanOptionalAction)  # Added in version 3.9.
parser.parse_args(['--no-foo'])
Namespace(foo=False)

This should be combinable with default values.
So, a possible solution would be
sub_parser.add_argument(
            "--micro",
            action=argparse.BooleanOptionalAction,
            dest="increment_micro",
            help="When setting pre-release, specifies "
            + "whether micro version shall "
            + "be incremented as well.",
           default=True,
        )

I could give it a try, when I'm home tomorrow.

Nice. You might need to get rid of default=True too, time will tell

Unfortunately, the BooleanOptionalAction is still a bit buggy, but I managed it to setup a working implementation with a mutual exclusive argument group.

Seems to work in the quick test I did, thanks!


What better way to install on jelastic

I am having difficulties to install kong on a jelastic cloud with ngix, can someone who has already installed on this platform pass a step by step of the best way to use it?
I know that this is not the right place for questions, I thank you for your attention.

Hi @edson-nascimento! We don't have experience using that service, you might get a luckier shot heading to Kong Nation. There are several active community members there, maybe someone has already done that.


auth.AuthWampCra does not implement on_welcome

AuthWampCra in autobahn==18.4.1 does not implement on_welcome causing Session.onWelcome to fail with AttributeError. IMHO an implementation as in AuthAnonymous returning None should be sufficient.

just verified: of the existing authenticators:

AuthScram
AuthCryptoSign
AuthWampCra
AuthAnonymous
AuthTicket

only AuthWampCra misses on_welcome.


[Popup]: Improve async rendering API

[Popup]: Improve async rendering API
Problem description
TODO
Proposed solution
Replace renderContent with positioningDependencies
BREAKING CHANGE
/**
 * Function to render popup content.
 * @param {Function} updatePosition - function to request popup position update.
 */
renderContent?: (updatePosition: Function) => ShorthandValue

will be replaced by
/**
 * Array of conditions to be met in order to trigger a subsequent render
 * to reposition the elements.
 */
positioningDependencies?: any[]

this can be done using the new children callback


IPv6 Hostname encoding as used within Browser addresses or build by .NET UriBuilder are not accepted (parsed host name is null)

If in a IPv6 environment it happened the gtcp Uri might look like that:
gtcp://[fec0:0:0:ffff::1%251]:1235
Uri parts are:
Scheme: gtcp
Host: fec0:0:0:ffff::1%1
Port: 1235
As described in RFC 6874 (https://tools.ietf.org/html/rfc6874)
If used within a URI, a literal IPv6 address is always embedded between "[" and "]" and
Zone IDs have to be URL encoded ("%" ==> "%25").

I implemented the fix within my github repo (see commit link above), but somehow cannot create a PR. Can you do it by your own @yallie ? Thanks i.A.

Thanks a lot!
Looks like your own repository is a bit outdated.
Let me try to rebase this commit.


Migrate to Junit5

Description
Migrate to Junit5
Issue reference
Fixes #875
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

Thanks for working on this! 🎉 Are those integration tests typically failing? Looks like it's SdkResiliencytIT. If so, I'd like to create an issue to track that.

@cicoyle you welcome :+1: Unfortunatly yes, SdkResiliencytIT is a flaky test.

@artursouza thank you for your kind words.

It seems that the SDk resiliency IT test is flakier than usual. I will need to spend some time debugging this before merging the PR.

@artursouza wait 5 seconds before start toxiproxy seems to solve the problem.

Amazing work @ejba !


Add verbose s3 logging 1.11

Add deploy option to enable verbose logging in S3 client
This is the 1.11 version of #5341

Moving to JD as Bryce is OOO


Update python-cmethods to v2.2.3

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

See release notes: https://github.com/btschwertfeger/python-cmethods/releases/tag/v2.2.3

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


[enhancement] Change the speed of ball on hitting the bricks

@jajodiaraghav sir, as different difficulty levels have different and fixed ball speeds, then according to me, it's meaningless for ball to change the speed on collisions. So I think the issue should be closed.


Uniformize names of variables between the standard generator scripts and rest of codebase

For example, in TEC.calc_contact_potential_STANDARD.py, there's a variable named "Emitter_barrier." It would be better to reconfigure the data structure into a dict so that it more resembles something that could instantiate a tec object.

Irrelevant with the new API.


impossible to compile reth in WSL2

Describe the bug
Latest rust version compiler (stable & nightly both doesn't compile)
Steps to reproduce
git clone https://github.com/paradigmxyz/reth/
cd reth
cargo build
Node logs
No response
Platform(s)
No response
What version/commit are you on?
No response
What database version are you on?
No response
What type of node are you running?
None
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

Could you please paste the compilation error?


Adagio Bid Adapter: add Prebid Server Adapter support

See: https://github.com/prebid/prebid-server/pull/1907

I see that you have some optional parameters for your Prebid Server adapter that are not included in this list. It might be useful to add those and somehow indicate that they are only applicable for Prebid Server. Also, the list of required parameters currently include adUnitElementId which is not present for Prebid Server. Can you please also indicate that this is only for Prebid JS? Thanks!

Hi @mansinahar, adUnitElementId is required but will be used client side. It is used in the bidder adapter transformBidParams callback which is run before the call to Prebid Server, but is not used in our SSP.
Regarding the optional parameters, you probably speak about features in the params_test.go file. This object is also computed during the transformBidParams callback and added by ourself to the params object to be part of the OpenRTB request.
Let me know if you need more info.


[Feature Request] Seamless video looping in an environment where running server/python is not an option?

When videos loop, there's a momentary drop to black before the video repeats.
If it's not just my setup that's causing this issue, are there any plans to implement an approach that would result in seamless loops?
In a scenario where image sevrer/python can't be used, is there any way to mitigate this second of blackness?
Thanks.

Hi.
That behavior occurred in my environment too and there was another person who reported this issue earlier, so I think this is an Unity issue, caused by Unity's VideoPlayer object limitation.
I guess it can be bypassed by saving the last frame (whose framenumber can be known) as Texture2D and use it until the video reloads.
I'll try to implement it after say 1-2 weeks (exam period😿)

v0.9.1-beta.3
Turns out using the VideoPlayer's Pause() not Stop() makes no flickering.

I will test over the weekend and report back.
Thank you.

My apologies for the delayed update. On my Windows 11 setup the loop now appears seamless for mp4 format videos I have tested with so far. Thanks for this fix.
There is still a very slight dip to black as the video is manually switched to the next video in the directory. I'm not sure if this is to be expected as it does not dip to black when switching between images. However perhaps this could be due to the amount of new data being loaded for every video MiDaS has to process?
I mention this as I'm also searching for a way of getting the video/image content of a directory to play seamlessly, either as linear or random sequences.
AFAIK the image server has this covered for image sequences, but I'm interested to know for video especially, are there any methods for playing through the contents of a selected directory that you have already implemented, or are exploring for this project?
My thinking is that I've previously seen VideoPlayer handling seamless transitions between multiple videos played in sequence in other Unity projects, so if the dip to black is not caused by MiDaS, possibly VideoPlayer can cache the initial frames of an array of videos in a way that might allow DepthViewer to render these video sequences seamlessly.
I'm thinking something like this could be a solution https://stackoverflow.com/questions/47401759/play-different-videos-back-to-back-seamlessly-using-unity-video-player It's an old post but checking the 2021.3 documentation, videoPlayer.Prepare() still appears to be the way this is handled. Any thoughts on whether this approach could work or would the MiDaS model still be a botttleneck when transitioning between videos?

Removing the dip-to-black between the videos (using the native VideoPlayer) would be difficult because the code frees the VideoPlayer resources when a new input is requested. Curiously enough commenting out the cleanup code (_vp.Stop(), _vp.url = null) would not remove this behavior, so other methods would be needed, like the one you linked. The one in the link looks promising (Prepare() then wait for the isPrepared -- or imo prepareCompleted event can be used), but it wouldn't be able to applied to this repo.
Side note: MiDaS would not be a bottleneck when transitioning between videos but disk reading and processing the file would be, since the input shape does not change while using the Unity-side inference engines (Barracuda or C#-OnnxRuntime)

Thank you.


Is the name HAL safe to use?

Hi, professional hardware designer here.
I followed the talk at CCC with great interest, but once I heard the name of the tool I immidiately
thought about work. Cadence Design Systems uses HAL as the Trademark name for their HDL Analyzer Tool as part of Incisive and Xcelium RTL/netlist simulator suites. It is a Linter which is used to uncover structural problems in the design like clock domain crossing, combinational logic in reset paths etc. and to enforce certain coding rules, including FSM coding styles etc.
As this is in a very similar domain I was wondering if it is safe for you to use this project name without hurting this trademark?
I'm not affiliated to cadence in any way, this jsut popped into my mind and I wanted to bring this to your attention. Might be totally irrelevant but I felt notifying you cannot hurt.

Thank you for pointing out! We will take a look at this with our team

Thank you for the notification. As @devhoffmann said, we will discuss that with our team and try to resolve this issue.

I checked the german and european trademark register and could not find an entry for HAL that targets any kind of software.
From my understanding, it should therefore be safe to use the name HAL.

@devhoffmann I agree, I couldn't find anything either. I suggest we close the issue for now and reopen it if we run into any trouble.


Liquibase should have an option/flag to generate an empty changeLog

Description
As a user I would like to have an option to have Liquibase generate an empty changeLog even if my database has some existing Objects.
For example: liquibase generateChangelog --template
Where the flag --template will ensure that the changeLog will not have any changeSets.
Here are some suggestions about how the generated changeLog should look like after the generation:

It should have a general template of a changeLog for the formats XML,JSON,YAML and Formatted SQL
Don't generate any changeSets but instead output a pre-defined string with example placeholder changeSets commented out.

➤ Mario Champion commented:
Adding this to the an epix which starts next sprint -- unsure when this exact ticket will be prioritized.


About Image Sensors

Hello, thank you very much for sharing your work. I would like to ask if the car is equipped with a depth camera that can detect the following depth images. If there is a depth camera, where should I obtain it? If not, how can I improve the car model?
I hope you can give me some advice. Thank you

Hi,
Yes, there is depth camera available as a sensor that is available on the robot. You can use rviz to see available sensors by topic. If it can be used for semantic segmentation, I do not know. You will have to test it for yourself.

Hello, I checked the topic on rviz and found the depth camera sensor, but it doesn't seem to be working. I checked the screen and it kept showing completely black

Then my terminal has the following information, it seems that the depth camera is not properly configured，When I used the command image_saver to save the topic camera/depth/imageraw, I got the following image, which doesn't seem to be the depth image I wanted.

Sorry, I am a beginner in ROS. I am not sure if it is a file configuration issue with the robot, so I would like to ask you about it

Hello, I checked the topic on rviz and found the depth camera sensor, but it doesn't seem to be working. I checked the screen and it kept showing completely black

Then my terminal has the following information, it seems that the depth camera is not properly configured，

When I used the command image_saver to save the topic camera/depth/imageraw, I got the following image, which doesn't seem to be the depth image I wanted.

Sorry, I am a beginner in ROS. I am not sure if it is a file configuration issue with the robot, so I would like to ask you about it

Hello, I checked the topic on rviz and found the depth camera sensor,

but it doesn't seem to be working. I checked the screen and it kept showing completely black

Then my terminal has the following information, it seems that the depth camera is not properly configured，

When I used the command image_saver to save the topic camera/depth/imageraw, I got the following image, which doesn't seem to be the depth image I wanted.

Sorry, I am a beginner in ROS. I am not sure if it is a file configuration issue with the robot, so I would like to ask you about it

The debug information looks fine. That is just an information output, not a warning or an error.
You should check what the raw information is from the camera topics with rostopic echo. I cannot look into this as I have no way of testing this, but you can try to see some information in ros tutorials: https://classic.gazebosim.org/tutorials/?tut=ros_depth_camera


Make path optional in get_booking

Fixes

#873

Goals :soccer:
Fixing issues with paths in the get_booking function.
Testing Details :mag:
@radekosmulski and I have validated the new code works as expected and fixes the bugs reported.

rerun tests


Invoke-API : The remote server returned an error: (400) Bad Request.

I am trying to Create a release pipeline which will deploy the powerBI report on the powerBi Web services. For this I am using PowerBI Action task and specifying the relevant details like Client ID, Secret, tenant ID and report and group id as well.
I am getting an error while executing the release, The remote server returned an error: (400) Bad Request.
tasklog_5 (1).log

Could you also supply screenshots
Groetjes,
Maik van der Gaag

From: NimNagpal<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 5:31:42 PM
To: maikvandergaag/msft-extensions<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [maikvandergaag/msft-extensions] Invoke-API : The remote server returned an error: (400) Bad Request. (#133)
I am trying to Create a release pipeline which will deploy the powerBI report on the powerBi Web services. For this I am using PowerBI Action task and specifying the relevant details like Client ID, Secret, tenant ID and report and group id as well.
I am getting an error while executing the release, The remote server returned an error: (400) Bad Request.
tasklog_5 (1).loghttps://github.com/maikvandergaag/msft-extensions/files/3832260/tasklog_5.1.log
[Capture]https://user-images.githubusercontent.com/31670186/68603665-a784d400-04a0-11ea-8365-155b97cb7adf.PNG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/maikvandergaag/msft-extensions/issues/133?email_source=notifications&email_token=ACIXAE6QIAUTAJUKN6AFKADQTGCG5A5CNFSM4JLX3TV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HYOZD7Q, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACIXAE62T7EKNXRXOSMBRO3QTGCG5ANCNFSM4JLX3TVQ.

I did found the problem, I was using a dummy files which didn't had any dataset. thus it was failing to publish the report.


iOS 9 Collection view cell width wrong at appearance

I ran into this problem after updating to Xcode 7 & iOS9 SDK:
Immediately after the VC is pushed, the collection view cell width is smaller than the screen width. A reloadData on the collectionView is needed in order to restore the cell width to match screen width. It's not happening all the time but occasionally with chat dialogs that have a lot of messages/data to display.

I'm having similar issue like this. Only on iOS 9 and if it's iPhone 6. on iPhone 5 everything is all right.
Here is my screenshot.

how to replace it?

@fanly @zigdanis: This is happening because the loaded xib file (with chat collection view) has wrong frame (in this case frame.size.width) in viewDidLoad or viewWillAppear because it's not properly laid out yet. You can get proper collectionView size in viewDidLayoutSubviews method.
Now, the problem is connected to JSQMessagesBubblesSizeCalculator - cell sizes are calculated right after viewWillAppear. When calculating cell size they are using this wrong width value and the result is stored in cache and reused afterwards.
A quick fix is to place:
- (void)viewDidLayoutSubviews {

    [super viewDidLayoutSubviews];

    // invalidate layout
    [self.collectionView.collectionViewLayout invalidateLayoutWithContext:[JSQMessagesCollectionViewFlowLayoutInvalidationContext context]];
}

cc: @jessesquires

@jessesquires
how to use "previousCollectionViewFrameWidth"?

@fanly It is just a custom CGFloat value which is set in viewDidLoad like this:
self.previousCollectionViewFrameWidth = CGRectGetWidth(self.collectionView.frame);

@jardamach Thanks! This saved me a lot of time.
Adding this -viewDidLayoutSubviews did the trick for me, with one additional change: I had to set invalidateFlowMessagesCache to YES on the invalidation context. You mentioned this in your comment, but your sample code didn't have it.
// invalidate layout
JSQMessagesCollectionViewFlowLayoutInvalidationContext *context = [JSQMessagesCollectionViewFlowLayoutInvalidationContext context];
context.invalidateFlowLayoutMessagesCache = YES;
[self.collectionView.collectionViewLayout invalidateLayoutWithContext:context];

Hello all -- I believe this is a dupe of #1212 and was fixed by #1238.
Can anyone confirm?

@jessesquires I believe that #1238 is not a fix of this issue (But can't really say 100% now - I did just a really quick test on my version). The thing is that this is not a rotation related issue - happens right after chat presentation. In #1238 there are two methods invalidating bubble size cache:
traitCollectionDidChange - called after viewWillAppear but chat.view.frame is not yet correct and recalculating bubble sizes gives incorrect values.
viewWillTransitionToSize - not called after viewWillAppear
Maybe @harlanhaskins could comment on this one as well.

@jardamach Your solution's still not working in my case though. Any thought?

Hi, does any figure out anything? I've tried all the quick fix, but non of them worked.

hey all -- I can repro this with the demo project on an iPhone 6 device.
can anyone provide further steps?

@jessesquires Thanks for following up! I can verify that #1238 fixes it for the demo project, but not for my app.
For some reason in the demo app the width of the view has already changed from 320 (width in the .XIB) to 375 (iPhone 6 width) when -traitCollectionDidChange: is called, and so the call to jsq_resetLayoutAndCaches invalidates and recalculates the layout before the view has appeared.
But in my app (and others, it looks like), the view is still 320 wide when traitCollectionDidChange: is called, and so the layout needs to be invalidated again when viewDidLayoutSubviews: is called. I suppose it's because it's loading the chat entries from Core Data.
I'll submit a pull request based on @jardamach 's fix.

Hm... @erichedstrom can you provide more details about your setup? Xcode version, iOS versions, devices, etc.

@jessesquires Xcode 7.3.1, iPhone 6 with 9.3.2, and iPhone 6 Simulator 9.3. Base deployment target for app is iOS 8.1.
Aha! The demo app shows the problem as well if I change the deployment target from iOS 7 to 8.1.

Specific steps:
Set deployment target to iOS 8.1.
Run the demo app on an iPhone 6/6s or 6/6s simulator.
Tap Push via Storyboard or Push programmatically.
View will appear with bubbles sized appropriately for an iPhone 5.
Start to pull down Notification Center.
Bubbles will suddenly resize to fit the iPhone 6.

Thanks @erichedstrom !
So I followed those steps but still couldn't repro. Did you pull the latest from master ?

Ach, yes, I did have the latest from master, but now I can't repro with the demo app either.
The bottom line is the fix in #1238 expects the view from the xib to have been resized before traitCollectionDidChange: is called, while the view is not guaranteed to have been resized until viewDidLayoutSubviews is called. (I don't see viewWillTransitionToSize:withTransitionCoordinator: being called at all.)

Hm... weird.
viewWillTransitionToSize:withTransitionCoordinator: would only be called on rotations.
Have you tried invalidating on viewWillAppear: ?

Thanks for following up.
I did try invalidating in viewWillAppear, and the base class also invalidates in a block called from its viewWillAppear.
I also have in viewWillAppear a [self scrollToBottomAnimated:YES] after calling [super viewWillAppear]. If I comment out my call to scrollToBottomAnimated:, the cells are sized correctly right off the bat, but you can see it scrolling to the bottom a second time as the view is sliding in from the right. So I'm going to stick with @jardamach 's fix above.
If I am ever able to track down and reproduce the problem consistently in the demo app I will let you know. It seems like you might as well close this again because no one else is speaking up about it.

not sure if it's the same issue, but I have layout issues with the message cells in iOS 9 on ipad after rotation. I have found turning on 'Requires full screen' fixes the issue (but prevents the app using Slide Over and Split View)

I'm having the same problem on iphone 6s simulator. Iphone 5s is fine.
The issue on iphone 6 simulator is:
If I load message from internet or from cache, the bubble size is ok.
Once I store that message in a local variable and go go back to the chat page again using navigation controller and load that chat massage from memory, the bubble width become about 30px smaller.
I tracked down the calculation process and found the issue is in the JSQMessagesBubblesSizeCalculator ,  in

(CGSize)messageBubbleSizeForMessageData:(id)messageData
atIndexPath:(NSIndexPath *)indexPath
withLayout:(JSQMessagesCollectionViewFlowLayout *)layout

The "stringRect" gave me different results for message load from cache and from memory. There is a 30px difference although their [messageData text] are the same.  I used subclass of jsqmessage.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq

Right, the problem is when you load from nib.
I fixed it with this:
`@implementation ChatVC

(void)viewDidLoad {
[super viewDidLoad];
CGRect frame = self.collectionView.frame;
frame.size.width = CGRectGetWidth(self.view.frame);
self.collectionView.frame = frame;
}
`


Support f16 and f128

As of https://github.com/rust-lang/rust/pull/121728, the backends now have stubs for f16 and f128: https://github.com/rust-lang/rust/blob/6cbf0926d54c80ea6d15df333be9281f65bbeb36/compiler/rustc_codegen_gcc/src/type_.rs#L123-L137.
It will be a while before these codepaths can even be exercised since the frontend isn't hooked up yet, (frontend is done now) but at some point this should be changed to either gracefully fail or do the right thing.

hello @tgross35 can I work on this?

Hey @zedar I saw you did https://github.com/rust-lang/gcc/pull/47 and https://github.com/rust-lang/gccjit.rs/pull/33 for this. Is anything else needed in the GCC backend, or do the unimplemented!s at https://github.com/rust-lang/rustc_codegen_gcc/blob/a63b83eb4e46433216dac0558b978619c6746ea8/src/type_.rs#L123-L137 just need to be updated?

Oh I see they were done in https://github.com/antoyo/rustc_codegen_gcc/pull/17, awesome!
I don't have any need for it, just been chasing down the f16_f128 FIXMEs and unimplemented!s.

(chances are you know this already, but Miri is using https://github.com/josh-project/josh for supposedly easier syncs, I think Clippy is looking to switch to it too)

I think this has been synced in by now right? A simple example seems to work https://godbolt.org/z/GqWvrza5e.
Many thanks for making this happen!


Interactive but non-Input Widgets

The RichText widget and also the upcoming TreeView widget are both interactive. One can scroll the view, there is the concept of selectable rows and the TreeView entries can be opened / closed. By default they are not necessarily Inputs in the common sense.
The LogViewer is another such case. I'd say it makes sense for it to have focus to allow it to be scrolled.
Maybe interesting: It's not yet implemented, but I'll give the RichText widget an optional cursor. Building a RichText widget subclass that allows editing the content is then trivial.
Not inheriting from Input has these consequences:

Don't take part in tab order.
Have no concept of being focused.

Inheriting from Input pulls in quite some functionality I'm not necessarily interested in:

hints
&process-input
error

Is this enough reason to introduce an intermediate role called Interactive or something like that?

Ooh yeah, this is a great idea.  I hadn't decided what I was going to do with the mismatch you're talking about, but Interactive seems like a good middle ground for now.
Feel free to give it a go if you like, or I will next time I get some hacking tuits.

I can give it a try. What do you think about Focusable as the role name?


Adds BMW art guide to All events sections

Adds BMW art guide to All events sections
Does not use correct bucket data, only adds view component
Needs href to BMW Art Guide

Fails

:no_entry_sign:

[danger-plugin-jest] Could not read test results. Danger cannot pass or fail the build.

  Generated by :no_entry_sign: dangerJS


Foregroundservice: MissingPluginException

When trying to execute
flutterBeacon.initializeScanning
from a foreground service function, an exception is thrown:
_Unhandled Exception: MissingPluginException(No implementation found for method initialize on channel flutter_beacon)
#0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:154:7)
#1      FlutterBeacon.initializeScanning (package:flutter_beacon/flutter_beacon.dart:60:20)
Is this an issue on my side, or is this package not suitable for foreground services, @alann-maulana ?
The foreground service package I'm using, is android_long_task (0.4.7) and flutter_beacon (0.5.1) with flutter version 2.5.1
The code snippet is the following:
@pragma('vm:entry-point')
serviceMain() async {

  ServiceClient.setExecutionCallback((initialData) async {
    if (Platform.isAndroid) {
        await flutterBeacon.initializeScanning;
    }
}

@alann-maulana can you help us?

@typexy Did you get a solution?

facing the same issue when trying to initialize in isolate.

Do you find any hit to do this?

I tried and faced same error. I guess this plugin doesn't support for foreground services.

Did anyone find a solution/alternative?


[MDETR] Directly use module dict for VQA heads

Stack from ghstack (oldest at bottom):

-> #232
#231
#230

Summary:
Remove the MultiHead class used for VQA heads, which is just a wrapper around ModuleDict anyways.
Test plan:
CUBLAS_WORKSPACE_CONFIG=:4096:8 python -m torch.distributed.launch --nproc_per_node=4 --use_env vqa_eval.py
--resume https://pytorch.s3.amazonaws.com/models/multimodal/mdetr/gqa_resnet101_checkpoint.pth --ema --eval
--dataset_config vqa.json

Gives total accuracy 61.6% as expected

Codecov Report

Merging #232 (65a6532) into gh/ebsmothers/21/base (9b3d1a1) will increase coverage by 0.14%.
The diff coverage is 14.28%.

@@                    Coverage Diff                    @@
##           gh/ebsmothers/21/base     #232      +/-   ##
=========================================================
+ Coverage                  91.20%   91.35%   +0.14%
=========================================================
  Files                         48       48
  Lines                       2877     2869       -8
=========================================================
- Hits                        2624     2621       -3
+ Misses                       253      248       -5

Impacted Files
Coverage Δ

torchmultimodal/models/mdetr/model.py
66.46% <14.28%> (+1.34%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b3d1a1...65a6532. Read the comment docs.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


bars dont refresh

i had ENV running in a shell, and spun up 20 new pods/nodes and the CLI app updated to include the new nodes but not the stats about them

after I restarted the app all the expected details where there

+1 same behavior on AWS Cloud9 instance the bars do not refresh properly and I have quite and start again to see the new status.

I haven't been able to reproduce this, I even ran it on AWS CloudShell and it updated when pod/nodes changed.

Mac OS ventura M1
Latest iterm2
There's a top up saying this is an interactive shell and if I want to let it refresh, I say yes, usually.

Here is some screeenshot when new node is added and even after 2mins it does not show any pods on it only after quit and start again.

Workernode added ^^

Workernode up for 2mins, so should be in ready state by now but timer still going ^^

After quit and start again it all looks fine ^^
Here is an example when running 2 session one is up before the node comes online fully:

And this is another session in new shell started about 30 seconds after the new node is provisioned.

AWS Cloud9 using ami "amazon/Cloud9AmazonLinux2-2022-08-01T09-18".  Happy to supply more info.

Same here on Linux Fedora 37. Starts refreshing and stops after a short period


auto-swipe stopped after swipe-left/right by touch at v2.0.4

Hi voronoanski,
line426    delay = options.auto || 0;
was removed at v2.0.4, cause auto-swipe stopped after swipe-left/right.
@voronianski Please release new version to npm and update react-swipe dependency with this version of swipe-js-iso.

3q

the problem still  exists in v2.1.4


#675: Chrome browser gets maximized

Proposed changes
Fix for #675 issue.
Checklist

[x] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

@user667 What should we do with this PR?
As I understand, it will not help because driver.manage().window().maximize() doesn't maximize the Chrome, and doesn't throw any exceptions. Thus method maximizeBrowser(WebDriver driver) will never maximize chrome window.
Should we close this PR?

@asolntsev @tbgdn yes we should.
See my comment in https://github.com/codeborne/selenide/pull/680#issuecomment-375850395


Transaction history including detailed info screen

Scope:

HistoryListItem in the "transaction history" list of the wallet.
When clicking on one item details for the transaction should be shown.

This depends on Depends on https://github.com/get10101/10101/issues/66 - For now I would suggest to just use an ExpandablePanel to represent this with some additional details unfolding when the item is clicked; see: https://github.com/get10101/10101/blob/main/mobile/lib/features/trade/position_list_item.dart#L29)

The wallet's history consists of three different items:

Lightning payments (send/receive)
Bitcoin transactions (send/receive)
Trade related items; i.e. Filled orders - when we trade we basically move funds out of the wallet and lock it in a channel

Order that moves funds into the channel (i.e. opens / updates a position)
Order that moves funds back into the wallet (i.e. closes the position); we might want to add some metadata here, because these entries could serve as trade history for the time being :)

The model that is exposed on the API should reflect this; I think it would be best to expose one complete history that is assembled internally.
This should include an extra screen/layover for the transaction details.

It would be great if we can achieve a similar look and feel throughout all the list items - and have similar behavior for them when e.g. clicking on them.
I propose three steps of information:

The list tile shows the most important information
When taping on the list tile, the tile expands (downwards) to show additional information
(optional) we show a button More details when expanded that would jump to a child-screen with even more information.

For reference, the PositionListItem already has an ExpandablePanel, see https://github.com/get10101/10101/blob/main/mobile/lib/features/trade/position_list_item.dart

Look of the OrderListItem:

Ticket that tracks adding order information in ExpandablePanel: https://github.com/get10101/10101/issues/163

Please refer to the detail mocks and explanation here:
https://github.com/get10101/10101/issues/66#issuecomment-1442859824
If there are any questions please ping me here!

basic version of wallet history is implemented.


FirebaseListAdapter, onChildMoved bug

Hey,
I've used your implementation as basic for new RecyclerView. And noticed, that your method onChildMoved updates mModels with new object, but does not update mModelKeys. So if the same item is later being used, mModelKeys and mModel are not object consistent.

any resolution in this issue?

Not yet. This isn't a terribly popular sample, so revisiting it was further down my to do list. But since you poked it, I'll give it some attention later today.

I tried to implement the recent chats like whatsapp by setpriority with  respect to inverse of time ,
but when the child is moved then it throws arrayoutofbound exception

I found that the modelKeys map does not get updated and that the models list now has a different version of the object. Since onChildChanged is called directly after onChildMoved, and it uses a Java indexOf to find the model, getting the oldModel from modelKeys doesn't work.

I believe it's just a one-line fix, and I've created a pull request -
https://github.com/firebase/AndroidChat/pull/15

Hey @achuinard, sorry for missing your PR. I just committed a fix (https://github.com/firebase/AndroidChat/commit/daeb2232dc58156197680a552c7198a62ba1301e) for another issue, which probably also fixes this one. Feel free to reopen if this problem persists.


Expand table

📥 Proposed changes
This expands the current table component.
In addition to ordinary rows and rows that act as links, the component now also supports rows that may be expanded to show more content (accordions).
☑ Submission checklist

[x] I have read the CONTRIBUTING doc
[x] yarn build works locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

I'm closing this now, as it has been available for conversation for a while now without any activity :)


Revert "implemented yahoo search"

Reverts fossasia/query-server#39

Coverage remained the same at 13.251% when pulling 99291488c3b825e02b7c92ab567e19818945d782 on abhishek1995s:revert-39-master into ec936ba6a2a5b6ea88240c4a66d5bce496215516 on fossasia:master.

@abhishek1995s why are you reverting this now ?

@niranjan94  yahoo made some changes in search engine due to security reasons few days back hence code has to be changed I will update the correct one soon

That's alright ... But don't revert .. Make a change and commit ..


Unresolved specs during Gem::Specification.reset:

Hi, I have run bundle install, and still cannot run the site
$ jekyll serve
WARN: Unresolved specs during Gem::Specification.reset:
      posix-spawn (~> 0.3.6)
      redcarpet (~> 3.1)
      jekyll-watch (~> 1.1)
      fast-stemmer (~> 1.0)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
Configuration file: /path/to/autm/_config.yml
/Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:29:in `require': cannot load such file -- jekyll-sitemap (LoadError)
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:29:in `block in require_gems'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:26:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:26:in `require_gems'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:19:in `conscientious_require'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/site.rb:79:in `setup'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/site.rb:41:in `initialize'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/commands/build.rb:29:in `new'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/commands/build.rb:29:in `process'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/lib/jekyll/commands/serve.rb:26:in `block (2 levels) in init_with_program'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `block in execute'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `execute'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary/program.rb:42:in `go'
	from /Library/Ruby/Gems/2.0.0/gems/mercenary-0.3.5/lib/mercenary.rb:19:in `program'
	from /Library/Ruby/Gems/2.0.0/gems/jekyll-2.5.3/bin/jekyll:20:in `<top (required)>'
	from /usr/local/bin/jekyll:23:in `load'
	from /usr/local/bin/jekyll:23:in `<main>'

P/S: I'm not a ruby guy

Did you try to google this? This is not this template problem.
Have a look at this thread: https://github.com/jekyll/jekyll/issues/3103

Oh ya, actually I googled for some time, couldn't get the answer. But now bundle exec jekyll serve just works. Thanks

glad it worked :)


roadmap: update the roadmap to be consistent with the existing milestones

Hello @dbentley, @yuindustries,
Please review the following commits I made in branch nicks/roadmap:
11df3aeafe039e2e9f81c2193278b6019f0c9f1f (2019-06-03 11:59:53 -0400)
roadmap: update the roadmap to be consistent with the existing milestones

(mostly just trying to make this consistent with what's already in clubhouse, rather than set new priorities. if you have disagreements on the priorities in clubhouse, we should discuss that offline)


[ISSUE #192]Fix empty check for the server address in method ClientProps#get_server_list is always false

close #192

Fix empty check for the server address in method ClientProps#get_server_list is always false

run cargo fmt before PR please.

Codecov Report
Patch and project coverage have no change.

Comparison is base (2d94382) 15.16% compared to head (e5c425c) 15.16%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   15.16%   15.16%
=======================================
  Files          49       49
  Lines        2803     2802    -1
=======================================
  Hits          425      425
+ Misses       2378     2377    -1

Files Changed
Coverage Δ

src/api/props.rs
0.00% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Well RequestObjectSelectionCapability return has malformed text body.

In the ObjectSelectionConstants.java the string for WellObjectSelection has the space inside the quotes for schema location instead of after.  Causing a parsing error in SOAP UI.  Move the space outside the escaped quote.
The response comes back as
<wells xmlns="http://www.witsml.org/schemas/1series "version="<IP_ADDRESS>">
And it should be

The current code:
public static final String WELL_OBJ_SELECTION =
"<wells xmlns="http://www.witsml.org/schemas/1series "" +
"version="<IP_ADDRESS>">" +
Should actually be
public static final String WELL_OBJ_SELECTION =
"<wells xmlns="http://www.witsml.org/schemas/1series\" " +
"version="<IP_ADDRESS>">" +
You can look at the Wellbore string to see how it should be.
public class ObjectSelectionConstants {
    //=========================================================================
    // WELL_OBJ_SELECTION variable
    //=========================================================================
    public static final String WELL_OBJ_SELECTION =
            "<wells xmlns=\"http://www.witsml.org/schemas/1series \"" +
                    "version=\"<IP_ADDRESS>\">" +
                    "<well uid=\"uid_dfTestWell1\">" +
                    "<name>dfTestWell1</name>" +
                    "<numGovt>12345</numGovt>" +
                    "<numAPI>12345</numAPI>" +
                    "<commonData>" +
                    "<dTimLastChange>2019-03-08T19:47:27.545Z</dTimLastChange>" +
                    "</commonData>" +
                    "</well>" +
                    "</wells>";
    //=========================================================================

    //=========================================================================
    // WELLBORE_OBJ_SELECTION variable
    //=========================================================================
    public static final String WELLBORE_OBJ_SELECTION =
            "<wellbores xmlns=\"http://www.witsml.org/schemas/1series\" " +
                    "version=\"<IP_ADDRESS>\">" +
                    "<wellbore uidWell=\"uid_dfTestWell1\" uid=\"uid_dfTestWellbore1\">" +
                    "<name>dfTestWell1</name>" +
                    "<nameWellbore>dfTestWellbore1</nameWellbore>" +
                    "<commonData>" +
                    "<dTimLastChange>2019-03-08T19:47:27.545Z</dTimLastChange>" +
                    "</commonData>" +
                    "</wellbore>" +
                    "</wellsbores>";

The issue fixed and QA Tested.  Getting result as expected without spaces in the XML Out as:
xmlns="http://www.witsml.org/schemas/1series" version="<IP_ADDRESS>"


Tokens - use parent::offset*() methods when moving items arround in insertAt()

since we only move already existing items arround, we directly call into SplFixedArray::offset* methods.
that way we get arround additional overhead Tokens class adds with overridden offset* methods.
blackfire shows a 35-40% runtime improvement in overall walltime.
in my testbenchmark this renders a ~20 second improvement.
https://blackfire.io/profiles/compare/4a0c1058-5910-446d-9641-8bcf2823cc63/graph

in my testbenchmark this renders a ~20 second improvement.

20 seconds improvement for 1minute / 1hour / etc run ?

From initially 1min10 secs

Thank you @staabm.

@keradus just a question: did you intentionally mix all these unrelated changes into my PR?

Maybe recent GitHub issues are somehow related...
https://status.github.com/messages

I have no problem with the current state. just curious whether this was intenionally or not.
thx


Correcting documentation

Summary

[x] Colocar exemplos completos, que possam ser copiados e colados 100% funcionais
[x] No readme, ao invés de colocar o APP ID de sandbox, colocar algo como YOUR_APP_ID por algo como "Replace YOUR_APP_ID with your public app ID that you can find at Application > Setup on your account." Acho que vale ressaltar essa parte, talvez usando um callout.
[x] Typo na sessão “Plugging in” <CroctProvider/ > para  (double check se temos outros casos assim)
[x] No exemplo com Next usar o componente Script (next/script) ao invés da tag script

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas There is no hard-to-understand areas
[ ] I have made corresponding changes to the documentation Still have to change the plug.js as well
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works No need for tests
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Correspondent PR in plug.js

Funny enough, the  inside the  in Next wasn't loading the custom.js and, consequently, wasn't loading the croct.fetch. So the example was wrong, so I said "I'll put outside the  and see what it does" and then the script worked!
I never used Next.js so I do not know the details about putting the  inside the  tag or not. That made me think "If the script is not inside the head, where is it?" and I take a look in the HTML of the example and the script was right in the middle of the  tag... Next is mysterious


chore(main): release ctrl-q 4.3.1

:robot: I have created a release beep boop
4.3.1 (2024-11-28)
Bug Fixes

qseow: Fix broken task visualisation on Windows (5d9c9b4), closes #551

This PR was generated with Release Please. See documentation.

:robot: Created releases:

ctrl-q-v4.3.1
:sunflower:


Service list refreshes every time 'load more services' is clicked

Every time when the 'load more services' button is clicked, the whole service listing refreshes. Users would lose their scrolling position and have to scroll down again in order to explore more services. The experience gets progressively worse since the list will only grow longer every time the button is clicked, thus consuming more of users' time to scroll. In addition, since all the services are listed in text format without any visual cue, it became incredibly difficult to rediscover the service the users were last on.
The experience is recorded here in the link below:
https://drive.google.com/file/d/1-14TLSFv10B_BO08w3BcDi4nZEa-9GEv/view?usp=sharing
To address the bug, the ideal experience would be to retain the scrolling position upon clicking the 'load more services' button. That way users can simply continue their search without having to spend time to scroll down the list again.
Acceptance Criteria

[ ] After clicking onto the 'load more services' button, the scrolling position is retained.

This issue is related to issue #892. Wrapping the Flatlist and loadmore button in a scrollview sometimes mitigate the issue but is ill advised as it causes warning.

This will be an Acceptance Criteria as part if Issue #1058


Fix - Multiple instances creation

Related to #1009
Description :
When a new instance of axios is created yet, the defaults object containing the default configuration is merged with the provided instanceConfig. Sadly, this merge is not deeply performed which results in a reference copy of all the values that are plain objects in defaults. This sometimes results in a definitely not expected behavior (See #1009 for demonstration).
Proposal :
Instead of using the original object, a shallow copy of defaults should be created before merging on instance creation, which avoids any collisions between instances.

Coverage increased (+0.02%) to 93.75% when pulling 0ed04a62afdd57d444e8e8f3ee1f1c27b49d9e15 on emaincourt:master into f1fb3de38fc96287763aeb7c5fee23858c851955 on mzabriskie:master.

Thanks for your contribution, but we hope to solve this in a PR with a broader scope (see #812). Also we aren't likely to add additional dependencies to the library.

I had same issue of multi axios instance with different headers.
Is there anyway to fixed this issue currently?

I found a small deepcopy lib/util - https://github.com/sasaplus1/deepcopy.js
Can put this deepcopy to dependency?


Add Demo Server & Contributing Guidelines

This addresses issue #6 by adding a demo server using Gulp

@petebrowne Thanks for the PR! Sorry it has taken me a so long to get too.
Great stuff. The only issues I hit was package.json is missing gulp-rename as a dev dep. Apart from that I got the demo up and running. Edited the scss and the changes were reflected in the demo.

Hi @petebrowne, did you get my message on this request?

Thanks, I missed the update to the other PR. I've merged everything in!


What is the correct cubature rule for BatchMomentMatching?

Commonly, the number of sigma points depends on the dimension. For example, spherical cubature rules create a pointset with shape (N(d), d) with N=N(d)=2*d; and equally many scalar weights, i.e. a weight vector with shape (N(d),).
Call the number of points N=N(d).
There are a few options now for candidate points:

Points with shape (d, N(1)) and weights with shape (d, N(1))
Points with shape (d, N(d)) and weights with shape (d, N(d))

Only the first option scales with complexity O(d). but currently, the second option works much better. Is this a mistake in the code?
For context: the TaylorFirstOrder equivalent (using only the diagonal of the Jacobian) works pretty well: https://arxiv.org/pdf/2110.11812.pdf.

This is a research question.


Disable command logs (with flag)

When a CLI command is called from the pipeline, the docker-entrypoint.sh script verifies the user, connects credentials and prints logs (Successfully logged in etc.). After that, the preview command that requires a JSON output is called and those logs get into the stdout. As a workaround we take the tail of the output like this:
OUTPUT=$(/root/docker-entrypoint.sh preview update $UFFIZZI_ENVIRONMENT_ID $UFFIZZI_COMPOSE_FILE --output=json --set-labels project=$CI_PROJECT_NAME-$CI_PROJECT_ID-$CI_MERGE_REQUEST_IID | tail -n 1)

Some possible solutions:

Add a --verbose flag with different values (0 - disable all stdout, 1 - disable only command logs, 2 - enable all stdout)
Add some key to the JSON output (for example json: ) so it can be easily found using grep

@axisofentropy We do disable stdout if the --output-format=json, but in this case, logs come from commands executed before the command we pass that flag into (like uffizzi login or uffizzi connect).

This issue is not clear to me. What problem is this solving?
@axisofentropy @moklidia

Not relevant anymore


How to setState befor render view?

I can setState in componentDidMount or componentWillMount from asycnStorage, how to initial before render?

Please give us a sample mock up code to better understand what you're trying to achieve.
This doesn't seemed like a react-navigation related issue though.

As jcaspian said, I don't believe this is a react navigation issue. Please refer to StackOverflow or join the Reactiflux community on Discord.
Here is my attempt to understand your question:
class MyComponent extends React.Component {
  constructor(props) {
    this.state = { loaded: false };
  }
  componentDidMount() {
    AsyncStorage.getItem('myKey', (result) => this.setState({ loaded: true }));
  }

  render() {
    return this.state.loaded ? <WhatINormallyRender> : <LoadingScreen>;
  }
}

I'm going to close this issue since I do not believe this is related to React Navigation. Feel free to check out the resources I mentioned in my comment above. If you are continuing to experience an issue related to React Navigation, please open another issue :)


Think through categories

Do we need categories at all?
Tags vs categories
Categories for values? Categories for activities?
How to handle initial state?

Do we provide default set of categories?
Are categories unchangeable?

I vote we ditch the idea of categories right now, get things more set up, and then see how things feel. It's hard to know how they'll be most useful or if they're useful at all until actually using it.
Your thoughts, @alexburner ?

As I'm working on this, I'm having trouble finding the clearest way to differentiate values and activities. As we've got it now, both are formatted as -ing gerunds—one's just a little broader and one's a little more specific.
That's all well and good, but having trouble figuring out how we'd communicate how to set the things up to users. Like, how do you differentiate Experiencing Nature and Going Camping other than by specificity?
The intention is to communicate that there are many different ways you can accomplish Experiencing Nature, with Going Camping being one of them. I don't know, still feel like I'm missing a critical distinction that'll make it clear what's going on.
For now, I'm going to continue on and assume we'll figure out a clear way to communicate this and/or a some kind of elegant reworking that make it all clear.


Update json5 to json5-lib to remove security issue 	 CVE-2021-44906

Which @angular/* package(s) are relevant/related to the feature request?
No response
Description
Angular/localize currently uses the "json5" package which itself uses the "minimist" package with a CVE on it (https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-44906).
Proposed solution
The "json5" alternative package "json5-lib" seems to no more uses "minimist" as pointed out here : https://github.com/json5/json5/issues/205#issuecomment-782608322.
Moving to this package can fix the security issue.
Alternatives considered
No one known

Moving to the CLI repo as Angular FW packages does no have a dependency on json5. Unfortunately it appears that json5 is pulled in by Babel deps.

We don't expect the Angular CLI to be used on production server were this vulnerability can be exploited. As @JoostK mentioned this is not actionable from our end since JSON5 is being used by third party packages.

Although it is not supposed to be run as a prod dependency, as @alan-agius4 mentioned, I believe we should fix the security issue for all the people misusing it as a "prod" dependency. I am sure more users of this library were misusing it as @jonathanantoine. Even more, if you take into account that this library is a "prod" dependency in @nestjs/cli
Now, json5 has released a new version fixing the security issue CVE-2021-44906. I think we should start using this new version 2.2.1


Interpret JSON schema validator errors

As part of #820, I implemented an attempt to clean up the schema validation error output. It was pretty messy and involved a bunch of regex matches so we removed it.
Currently missing attributes each are a separate message, and extra attributes get combined into one. There is an actual object involved that lets you identify if it's an error about attributes being extra vs missing, but what attribute is extra or missing is only available as part of the error text itself. It would be nice to have it be consistent, one way or the other.

we don't use json schemas directly anymore and we do a pretty good job of rendering these error messages anymore :)


Client: Intrusive Requests

Allows for language clients to manage their own Packet memory, removing the need for tb_client to do so and thus removing the concepts of acquire/release_packet and concurrency_max.
Also properly defines deinit semantics to cancel any pending packets before returning, even if inflight. Additionally requires that submit (or any other client call) must happen-before deinit and may not race with it.

Just some minor comments and we are good to go! Also, the CI is failing due to a zig fmt check.