How to use a variable within a different variable file?

I would love to be able to use variables in my variable files, but I'm not sure how to do this.
Here's a simple example.... suppose I'm building a "light" and a "dark" stylesheet. I might have colorsLight.js and colorsDark.js variable files, as well as another colorsUniversal.js variable file for colors that are the same in both light and dark.
Then, I might want to include something like this in colorsLight.js:
someFeature: [
	"match",
	["get", "someField"]
	"value1",
	"rgba(0, 0, 0, 1)",
	"value2",
	"rgba(255, 255, 255, 1)",
	context.colorsUniversal.transparent
]

In this particular case I could work around it, but it illustrates what I'd like to do in more complex situations. Is this possible already, and I just don't know the syntax? Or would it require code changes on your end?
(Hello and yes, I'm using mapbox-gl-style-build at my current job! :wave: )

Hi @kateler ! Sorry for the delayed response and glad to see this is getting use out in the world!
You should be able to do this with conventional JS rather than being a build system specific solution. So in your specific example of light, dark, and universal color variables files, that might look like:
// universal.js

module.exports = {
  red: 'rgb(255, 0, 0)',
};

// light.js

const universalColors = require('./universal');

module.exports = {
  varyingColor: [
    	"match",
	["get", "someField"]
	"value1",
	"rgb(255, 255, 255)",
         universalColors.red
  ]
};

// light.js

const universalColors = require('./universal');

module.exports = {
  varyingColor: [
    	"match",
	["get", "someField"]
	"value1",
	"rgb(0, 0, 0)",
	universalColors.red
  ]
};

Let me know if this solves this for you!

Thanks @aparlato ! That did work, and unlocked some more efficiency for us. 🚀

Great to hear! Closing this as resolved then!


Allow renaming storage item prefixes

Resolves #8964.
This adds the ability to tag #[pallet::storage_prefix = "CustomName"] on storage items. What this does is renaming the storage prefix for that storage item to "CustomName", instead of using the Rust type alias name of the item. Currently, only string literals are accepted as a valid identifier for the new storage prefix, and it also must be a valid Rust identifier. See https://doc.rust-lang.org/reference/identifiers.html for more information on what constitutes as a valid Rust identifier.

bot merge


epoch value in ModelCheckpoint is inconsistent

Hi:
Keras offer a easy way to add epoch to the model name when saving
https://github.com/fchollet/keras/blob/master/keras/callbacks.py#L400
filepath = self.filepath.format(epoch=epoch, **logs)
But with the code like below one, which is suppose to save model every 20 eopch:
checkpointer = ModelCheckpoint(filepath="{epoch:03d}"+".hdf5", period=20)

It save the model every 20 epochs like:
019.hdf5
039.hdf5
The model saving operation does happen after 20, 40 epoch were finished.
Is there a bug, or I missed something(?
thanks for your review

Don't think it's a bug. The epoch value coming into the callback is zero indexed.

fixed in https://github.com/fchollet/keras/pull/8137


Update redis to 4.0.2

Is this a request for help?:
No

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
Version of Helm and Kubernetes:
N/A
Which chart:
redis
What happened:
Redis chart currently install version 3.2.9
What you expected to happen:
Redis chart should install verison 4.0.2
How to reproduce it (as minimally and precisely as possible):
N/A
Anything else we need to know:

Strange, a PR should have automatically been submitted. /cc @sameersbn @tompizmor

@whs @prydonius The updates were temporarily disabled for internal testing. We're in the process of restoring the updates. Will keep you updated

@whs p.s. PR #2380 upgrades to redis 4.0.


Switch to Pandoc 2.19 (current 2.17)

Checklist

[x] Checked the nixpkgs master branch
[x] Checked the nixpkgs pull requests

Project name
nix search name: pandoc
current version: 2.17.1
desired version: 2.19.2
Notify maintainers

maintainers: @peti
Note for maintainers
Please tag this issue in your PR.

I'm stupid, found it as pandoc_2_19_2 in unstable


Does not sort properly

The sorting isn't accurate, here's my HTML Code : https://pastebin.com/c5YsTFUS (Sorry for the indentation,  i took it from the dev tools of Chrome and it didn't respect my indentation in my code)
for example on the second click on this column :

I just added a little piece of code to make the arrows change on click and to keep my total row the last row :
https://pastebin.com/FTcaybV8
( I know it's a little bit complicated for a simple thing but i'm a beginner developper :c )
Thanks if you find the issue,
Max

Hey there. Can you use JSFiddle or something similar so I can have the
source code to replicate the issue?
Thanks!
On Monday, October 29, 2018, Maxime Lalo<EMAIL_ADDRESS>wrote:

The sorting isn't accurate, here's my HTML Code :
https://pastebin.com/c5YsTFUS (Sorry for the indentation, i took it from
the dev tools of Chrome and it didn't respect my indentation in my code)
for example on the second click on this column :
[image: image]
https://user-images.githubusercontent.com/39947841/47654673-a598ed00-db8b-11e8-9797-67af1e5b7f88.png
I just added a little piece of code to make the arrows change on click and
to keep my total row the last row :
`$(document).ready(function($) {
var table = $("#sortable-table").stupidtable();
table.on("aftertablesort", function (event, data) {
var th = $(this).find("th");
th.children().remove()
th.append('')
var thisColumnState = th.eq(data.column).attr('class');
var test = th.eq(data.column).children().remove();
var sortState = th.find(".sort-state").remove();
var totalRow = $(".totalRow");
totalRow.remove();
table.append(totalRow);
if (thisColumnState == 'sorting-desc') {
th.eq(data.column).append('up');
}else{
th.eq(data.column).append('down');
}
$(".not-sortable").children().remove();
});
});`
( I know it's a little bit complicated for a simple thing but i'm a
beginner developper :c )
Thanks if you find the issue,
Max
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joequery/Stupid-Table-Plugin/issues/201, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAevIEIKQPwYx6yZvmDQ0-G0Kr-VwPDZks5upwrkgaJpZM4X_Pok
.

--
Joe McCullough
(512) 701-7400
http://joequery.me

Hello, here's the jsfiddle : https://jsfiddle.net/Myzal/Lhv7kf6s/
Thanks for your reactivity !

That's perfect, thank you, you're awesome!

Glad to hear it worked! Closing this issue. Thanks for using the plugin.


Persist auth sessions with Redis

Description
Testing out using Redis to persist auth sessions.
Issue
https://github.com/papercups-io/papercups/issues/90
Checklist

[ ] Everything passes when running mix test
[ ] Ran mix format
[ ] No frontend compilation warnings

closing, handled with postgres instead of redis in https://github.com/papercups-io/papercups/pull/440


not working pls fix

code doesnt work i use windows 93 often and i need this program but it doesnt work for me

Download v1.0, unzip it and import the resulting folder in Windows93 (Don't drop the folder in /a/boot/). Then open Terminal and use the command js [path of thminstall.js] (example: js /a/desktop/ThemeMaster-1.0/thminstall.js). A window will appear. Accept all the message boxes, Windows93 will reboot. Once the reboot has finished, enter the themes command in Terminal to open ThemeMaster.

ok works now


Babel env preset

babel-preset-env: a preset that configures Babel for you.
More here

Oups wrong repo


Besu is failing some tests, please dedicate a person to work on this with me

Since besu has implemented test RPC it is a part of test stats program: http://retesteth.ethdevops.io/
Besu runs fine, but many tests are still red and I am trying to reach someone to work on fixing this from the besu team.
Please contact
telegram: @wdimitry
List of failing tests
info: (stTransactionTest/HighGasPrice, fork: Berlin, TrInfo: d: 0, g: 0, v: 0, TrData: ` 0x..`)
info: (stTransactionTest/ValueOverflow, fork: Berlin, TrInfo: d: 0, g: 0, v: 0, TrData: ` 0x..`)
info: (stExample/accessListExample, fork: EIP150, TrInfo: d: 0, g: 0, v: 0, TrData: `:label declaredKeyWrite 0x00..`)
info: (stExample/basefeeExample, fork: EIP150, TrInfo: d: 0, g: 0, v: 0, TrData: `:label declaredKeyWrite 0x00..`)
info: (stExample/eip1559, fork: EIP150, TrInfo: d: 0, g: 0, v: 0, TrData: ` 0x00..`)

info: (bcBlockGasLimitTest/GasUsedHigherThanBlockGasLimitButNotWithRefundsSuicideFirst_London, fork: London, block: 0)
info: (bcStateTests/TransactionNonceCheck2_London, fork: London, block: 0)
info: (bcEIP1559/feeCap_London, fork: London, block: 0)
info: (bcBerlinToLondon/initialVal_BerlinToLondonAt5, fork: BerlinToLondonAt5, block: 5)

Thanks for these reports. @daniellehrner does this relate to any of the RPC issues you were investigating prior?

Thanks for these reports. @daniellehrner does this relate to any of the RPC issues you were investigating prior?

It seems not to be related. It seems more like EVM related problems.

Hello @winsvega could you please post the command line that you are using to start Besu and the corresponding one for retesteth to get the output in the OP? Thanks!

The besu cmd is
besu retesteth --rpc-http-port $((47710+$i)) --data-path=$tmpdir --logging=DEBUG >> ~/.retesteth/logs/besu-$file

@winsvega I'm trying to run the retesteth from Docker (as I'm not being able to compile it directly on my Mac) to connect it to my besu instance running in the host. From the guest perspective, the JSON-RPC endpoint would be host.docker.internal:8545. How can I configure the tool to run this way?
These are my steps so far for building/running the tests:
$ git clone git@github.com:ethereum/retesteth.git
$ cd retesteth
$ docker build . --platform linux/amd64 -t retesteth
$ docker run --platform linux/amd64 --rm -v /Users/diegoll/dev/upstream/eth-tests:/tests -e ETHEREUM_TEST_PATH=/tests retesteth -t GeneralStateTests/stTransactionTest -- --singletest HighGasPrice --clients besu --singlenet Berlin
Running 1 test case...
Running tests using path: /tests
Active client configurations: 'besu '
Filter: 'HighGasPrice Berlin'
Running tests for config 'Hyperledger Besu on TCP' 2
Test Case "stTransactionTest": (1 of 1)
100%
/root/.retesteth/besu/start.sh
Finishing retesteth run
java: no process found
/root/.retesteth/besu/stop.sh
*** Total Tests Run: 0

Error: Error connecting to TCP socket: <IP_ADDRESS>:47710 (stTransactionTest/, step: TestSuite::executeTest)

--------
TestOutputHelper detected 1 errors during test execution!
/retesteth/retesteth/TestOutputHelper.cpp(226): error: in "GeneralStateTests/stTransactionTest":

*** 1 failure is detected in the test module "EthereumTests"

https://ethereum-tests.readthedocs.io/en/latest/retesteth-tutorial.html
Client outside docker

I can actually make besu into docker

I need to run the retesteth command from inside Docker b/c I'm unable to compile it into my Apple M1 (I may open an issue about it later on), but most importantly I want to run Besu from the IDE so I'm able to better debug the behavior.
I'm running my Besu instance with the following arguments:
retesteth --rpc-http-host=<IP_ADDRESS> --rpc-http-port=8545 --host-allowlist=* --logging=TRACE --data-path=/var/folders/qb/0fq2wn6j1bb9xxdbyff91s6r0000gn/T/retesteth

And the retesteth runs fine from its container, seemsly resolving fine the output client, e.g.
$ docker run --rm --platform linux/amd64 --entrypoint=/usr/bin/curl retesteth -s http://host.docker.internal:47710/liveness
{
  "status" : "UP"
}%

But when I run:
$ docker run --rm --platform linux/amd64 -v /Users/diegoll/dev/upstream/eth-tests:/tests -e ETHEREUM_TEST_PATH=/tests retesteth -t GeneralStateTests/stTransactionTest -- --singletest HighGasPrice --clients besu --singlenet Berlin --nodes host.docker.internal:8545

nothing seems to happen. The prompt gets stuck there, with no output and no apparent timeout (I waited more than 5 minutes and I couldn't see anything from Besu's console either) so I end up killing the process.

There is no hostname resolution.
It actually should throw an error message but there is a bug.

@winsvega ok, thanks for clarifying. I just found a hacky way to resolve host's IP and I'm using it to fire the test and it seems to be working. I'll keep you posted

I pushed a fix to resolve the hostname.
But once again how does it work from inside the docker need to check.
You can also use dretesteth.sh script to build the docker. Then it uses docker image providing the same interface as original executable.
Open an issue about mac build

@winsvega I was waiting to finish the analysis for all the tests in this report but it's taking more than I expected. We can start talking about the first ones while I finish the others.
HighGasPrice@Berlin
Command run:
$ ./retesteth  -t GeneralStateTests/stTransactionTest -- --singletest HighGasPrice --clients besu --singlenet Berlin --nodes <IP_ADDRESS>:8545

Response:
{"jsonrpc":"2.0","id":3,"error":{"code":-32602,"message":"Invalid params"}}

Reason: gasPrice * gasLimit exceeds the max representable value.
ValueOverflow@Berlin
Command run:
./retesteth  -t GeneralStateTests/stTransactionTest -- --singletest ValueOverflow --clients besu --singlenet Berlin --nodes <IP_ADDRESS>:8545

Response:
{"jsonrpc":"2.0","id":3,"error":{"code":-32602,"message":"Invalid params"}}

Reason: tx value exceeds the max representable value.

Given that both tests are marked to expectException in their post definitions, what would be the expected output from the client?

Yes, we actually never come up with error reports. As I understand its difficult for besu to report transaction exceptions. The exception report is not required, but very nice to check that the tr rejected for expected reason.
In case of exception not an rpc request fails but a certain response comes. I can clarify the format tommorow

accessListExample@EIP150, basefeeExample@EIP150, and eip1559@EIP150 fail because the transactions are being sent with EIP-155 protection and this isn't enabled until Spurious Dragon (eip158Block in Besu config).
Also, it's worth mentioning that the response to the eth_sendRawTransaction call from the test returns success with the empty hash, which is the behavior only for use within the context of the retesteth subcommand. I.e.
{"jsonrpc":"2.0","id":3,"result":"0xc5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470"}

GasUsedHigherThanBlockGasLimitButNotWithRefundsSuicideFirst@London and TransactionNonceCheck2@London fail because the fifth transaction is being rejected from the txpool. We need to change the tx pool configuration for the retesteth subcommand here.

Here in -t GeneralStateTests/stTransactionTest -- --singletest ValueOverflow --verbosity RPC
The transaction does not pass verification, so test_minBlocks report it with rejectedTransactions info
Response test_mineBlocks {1}
{
    "result" : 1,
    "rejectedTransactions" : [
        {
            "hash" : "0xfbd91685dcbf8172f0e8c53e2ddbb4d26707840da6b51a74371f62a33868fd82",
            "error" : "insufficient funds for gas * price + value: address 0xa94f5374Fce5edBC8E2a8697C15331677e6EbF0B have<PHONE_NUMBER> want<PHONE_NUMBER>37316195423570985008687907853269984665640564039457584007913131739937"
        }
    ]
}

This is not required to report transactions like this. so if you can't track the transactions in the pool, test_mineBlocks return 1, but the rejected transactions are discarded, so that all requests like eth_getBlockByNumber  will return block without the failing transaction. however if you implement this, while generating/runing the tests the exception will be checked to the one that test expects.

We can add your suggestion to the roadmap and work towards that, but for the scope of this issue, do you think it would be possible to make the reference tests to pass against the current main@ae6c885 given the analysis provided and the changes performed?

since the method is eth_sendRawTransaction  I think it is a correct behavior to reject invalid transaction if the RLP or structure is invalid. but if transaction is semantically valid and rejected due to another reason then it should return hash.

Ok all tests resolved now.
Only the shift operations on legacy forks need a fix.
Btw besu test rpc is the fastest now on the test run.

@winsvega, Thanks for reporting back! Can we close this issue then? Would you mind opening up another issue with the shift operations on legacy fork please?

Yes, all fixed now.
Next are Merge tests


list index out of range

newspaper.build function raises 'list index out of range' error when building 'zdnet.com', and then stop running.

Please post the code that you're running and the stack trace.

Closing due to inactivity


feat(Deployments): support Parameters

Supports setting Parameters on Deployment resouces.
Related to https://github.com/PrefectHQ/terraform-provider-prefect/issues/238

Need https://github.com/PrefectHQ/terraform-provider-prefect/pull/246 to unblock the tests.


scrollable-tabs

typescript: src/components/scrollable-tabs/scrollable-tabs.ts, line: 132
Element implicitly has an 'any' type because index expression is not of type 'number'.
 L132:  ['Zone']['__symbol__']('requestAnimationFrame')]) || (window[window['Zone']['__symbol__']('webkitRequestAnim
 L133:      let nativeRaf = originalRaf !== undefined ? originalRaf['bind'](window) : window.requestAnimationFrame.bind(win

dow);

Scroll not working plz help me


UI test with JupyterLab

UI looks good to me on JuypterLab for the Hello World job and job runs. Anything more specific we need to check to prepare for the transition?


AMLII-2210 enable Java FIPS provider on Windows FIPS images

What does this PR do?
Enable Java FIPS provider on Windows FIPS images.
Motivation
Describe how you validated your changes
Configure ActiveMQ with JMX mutual TLS encryption. Check that the key store for JMXFetch is in BCFKS format.
Run Datadog Agent JMX FIPS image. JMXFetch should be able to connect to both the Agent and JMX endpoint and collect metrics.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Setting EntryCompletion for Entry

The Entry trait is currently lacking the function set_completion(). It is included in the ffi crate, but somehow doesn't seem to have made it into the gtk crate.

Well saw, thanks ! Do you mind opening a PR to fix it ?
PS: doc url

I have created one including the fix in #154


Udpate openstack_containers in periodic lines

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/319
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/approve

https://review.rdoproject.org/zuul/buildset/229ead8205de437e88fdeaa35b0fd22e testproject results ...https://logserver.rdoproject.org/59/45359/37/check/periodic-podified-edpm-deployment-ante[…]/ci-framework-data/artifacts/update_containers_patch.out shows e1ee122c409bcbb83bc782fb7abbae37 which is https://trunk.rdoproject.org/centos9-antelope/podified-ci-testing/delorean.repo.md5

Thanks @raukadah lgtm


[FEATURE REQUEST] Tabbed monocle layout

Description
In a sense, this is a duplicate of already existing feature requests (see #2050, #2822), but I want to expand on them and describe in more detail a feature that I think would be very useful and hopefully inspire developers capable of creating it.
I think one of the biggest weaknesses of hyprland's window management is that there is no proper monocle layout, and I think to create one, it would be a really good idea to use the already existing groups feature, but implement it as a layout.
Of course, I know that there is a fullscreen 0 and 1 dispatcher, which in a way works like a monocle, but its purpose is a bit different. You only want to toggle fullscreen for a moment when the window is too small to see things more clearly, and then untoggle it.
With a properly implemented monocle layout, you can spawn windows, move them to or from the workspace, arrange them in a specific order, all without leaving the layout.
Take a look at this script from #2822, which replicates this behavior to some extent, but the problem is that it's still quite limiting and a proper implementation needs to be more than a hacky bash script:
https://gist.github.com/Atrate/b08c5b67172abafa5e7286f4a952ca4d
Here is the video showcasing the script:
https://github.com/hyprwm/Hyprland/assets/103893901/cff0247a-3722-4d30-b58f-4da3631feb71
The problems with the script are that:

If there's too many windows on the screen then not every window will be toggled as group
I can't use my keybindings for master layout, I need to define specific dispatchers for groups and if the bindings will be the same as for the master then I will not be able to use groups in the normal way.

bind  = $mainMod, J, layoutmsg, cyclenext
bind  = $mainMod, K, layoutmsg, cycleprev
bind  = $mainMod SHIFT, J, layoutmsg, swapnext
bind  = $mainMod SHIFT, K, layoutmsg, swapprev

bind = $mainMod, J, changegroupactive, f
bind = $mainMod, K, changegroupactive, b
bind = $mainMod SHIFT,J, movegroupwindow, f
bind = $mainMod SHIFT,K, movegroupwindow, b

The solution would be to add specific dispatchers, such as "monocle, cyclenext", "monocle, swapprev", etc.

I cannot move a window in a group to another workspace without moving the whole group. First I have to detach the window from the group and only then I can move it. Same as when I want to move a window to an already toggled group.
The position of the windows changes randomly when I use this script, and the desired behavior would be so the placement of the windows in the monocle layout would mirror the placement of the windows in the tiled layout.

So yeah, I think tabbed layout would be a really nice feature to have

#2050
use the search feature


Add large code model information.

Hi,
This PR add description about large code model.
I was wondering if we need large+fpic model.
In general, position independant code model puts external symbol addresses into the GOT table.
Is there any case that we have to layout GOT table far away from code over +-2GB?

I think I'd prefer to define a set of relocations to materialize a 64-bit address with four instructions and let the linker to relax it to 1 to 3 instruction depending on the offset to the materialized address. That approach is easier to implement than the address pool.
I'd also think it could be faster than reading addresses from the address pool because 1) the processor could fuse 3 or 4 instructions into a single macro-op, and 2) loading an address from the address pool is just a waste of resources if the materialized address happens to be not too far from PC.

@rui314 I am not sure if we can generate any arbitrary 64 bit address within 4 instruction? did you mind share the instruction sequence?

@kito-cheng Apologies, we can't materialize a 64-bit value with four instructions in RISC-V. We actually need six instructions to, for example, load a value from an arbitrary 64-bit address as follows:
lui   t0, <highest20>
addi  t0, t0, <higher12>
slli  t0, 32
auipc t1, <hi20>
addi  t1, t1, t0
ld    t1, <lo12>(t1)

which can be relaxed to the following 5 instructions if the symbol is within ±2^44 bytes
addi    t0, zero, <higher12>
c.slli  t0, 32
auipc   t1, <hi20>
addi    t1, t1, t0
ld      t1, <lo12>(t1)

and of course to the following two instructions if it's within ±2GiB.
auipc   t1, <hi20>
ld      t1, <lo12>(t1)

It looks to me that the RISC-V psABI's design choice to allow the linker to shrink the section really shines for this use case.

Creating new ABIs that only support position-dependent code seems like a bit of a questionable thing to be doing in this day and age

I think using constant pool for large model doesn't cost so much.  Because compiler can use anchors to tag variables, and load each variable just by its offset from the anchor.
Ex:
If we want to get values of global variables A and B. We don't have to load constanct pool entries twice for A and B.
auipc t0, hi20(.LC0)
ld       t1, t0, lo12(.LC0)
lw      a4,0(a5)
lw      a0,4(a5)
.LC0:
.dword  .LANCHOR0
.bss
.set    .LANCHOR0,. + 0
a:
.zero   4
b:
.zero   4

I know there are many extremely large programs out there that might already need the large code model, but to my knowledge, most of these programs are server-side and run in datacenters. They naturally need to be built as position-independent executables, and their text segments need to be read-only (or execute-only if possible). This made me wonder about your motivation to define a position-dependent-only ABI in the first place.
So, before diving into the details, I think we need to take a step back and start by understanding the context of this change. I'd like to understand your motivation, explore potential alternative specifications, and learn why you believe this is the best way to achieve the goal.

I have some notes about large code models in aarch64/powerpc64/x86-64: https://maskray.me/blog/2023-05-14-relocation-overflow-and-code-models#aarch64-code-models
I know that certain JIT programs may use large code models, possibly just the position-dependent form.

I think using constant pool for large model doesn't cost so much. Because compiler can use anchors to tag variables, and load each variable just by its offset from the anchor.

Agree.
For server side large x86-64 applications, they can use the medium code model. This larger range makes it unlikely for AArch64 to encounter relocation overflow issues before the binary becomes excessively oversized for x86-64.

to my knowledge, most of these programs are server-side and run in datacenters. They naturally need to be built as position-independent executables, and their text segments need to be read-only

Without commenting on the merits of this particular code model, I'll remark that there is a distinct and very real use case: RV64 embedded systems, which might not consume that much memory in total but need to cope with a sparse address space.  The text/rodata might be separated by gigabytes from the absolute-addressed I/O, and there might be multiple regions of each.  There's no virtual memory, so it isn't possible to remap the relevant regions to improve virtual spatial locality.

Actually, using constant pools as the large code model can generate position-independent executables. It only needs the static linker to level dynamic relocations for the loader or the memery manager to add the offset when executables is remapped.
In my first comment, I was just wondering if there is the real case that we need large+fpic.

Yes, constant pools are equivalent to a hand-rolled GOT.

Yes, constant pools are equivalent to a hand-rolled GOT.

Yes. It's a nice description.  Thanks.

So, before diving into the details, I think we need to take a step back and start by understanding the context of this change. I'd like to understand your motivation, explore potential alternative specifications, and learn why you believe this is the best way to achieve the goal.

I think I'm still waiting for a response to this comment...

lui   t0,
addi  t0, t0,
slli  t0, 32
auipc t1,
addi  t1, t1, t0
ld    t1, (t1)

Can use lui rather than auipc? I think all using lui would be easier to shared the high-part (first 5 instruction)? that should be able let compiler share the high-part between different low-part?
Use auipc we may either enforce whole instruction sequence must together or has a relocation let last instruction point to the auipc instruction like PCREL_LO12_*.

So, before diving into the details, I think we need to take a step back and start by understanding the context of this change. I'd like to understand your motivation, explore potential alternative specifications, and learn why you believe this is the best way to achieve the goal.

I think I'm still waiting for a response to this comment...

I involved the design and implementation of this code model when I still collage with @kuanlinchentw, so I guess I can give few detail from my brain dump: that design come with several advantages: 1) simple to implement, because it can be borrow the implementation from AArch64 :P, 2) NO new relocation required.
However the disadvantage is obviously: 1) every address need load from constant pool, 2) the pool has duplicated entries.
But we think the disadvantage can be ignore in most use case of large code model, since it mostly used when MMU-less situation, and also we have ePIC proposal, that could address some special use case in embedded world.
IIRC, llong instruction sequence scheme also has discussed before in somewhere (publicly?), but it just come with more overhead to implement: new relocation and new linker relaxation, also psABI TG isn't exist in that moment, so we are trying to prevent touch psABI as possible at that moment.

So, before diving into the details, I think we need to take a step back and start by understanding the context of this change. I'd like to understand your motivation, explore potential alternative specifications, and learn why you believe this is the best way to achieve the goal.

I think I'm still waiting for a response to this comment...

As @kito-cheng mentioned, It's easy to implement at the compiler veiw, and it doesn't need to modify binutils.
For compiler, each variable access can be a dependent load intruction after setting the anchor value.
This can avoid using lots of pseudo intructions that may not scheduled apart.
We might consider the way that using a set of relocations to materialize a 64-bit address before.
But there is a trade-off between the compiler scheduler and the linker relaxation.
If the compiler expands the instruction sequence to schedule, it's hard for the linker to relax.
Even if linker can recognize the sequence and relax, the delete instructions may affect the schedule result.
And the disadvantages as @kito-cheng mentioned, I think it's still an issue.
Obviously, it waste the space to save redundant entries. Maybe the compiler can generate the mergable constant sections to reduce the harm.

If no new feature is required for it, what's the point of adding a new section to the psABI document for it? Does AArch64 psABI has a section for their counterpart?

If no new feature is required for it, what's the point of adding a new section to the psABI document for it? Does AArch64 psABI has a section for their counterpart?

It need to add a new option for code model just like medany and differenct code generations.
Yes. AArch64 defines small, kernel, medium and large model, and there is a section about code model.

I couldn't find a section in https://github.com/ARM-software/abi-aa/blob/844a79fd4c77252a11342709e3b27b2c9f590cf1/aaelf64/aaelf64.rst to how to use a constant pool to load an object's address from memory. Could you share the URL?

I couldn't find a section in https://github.com/ARM-software/abi-aa/blob/844a79fd4c77252a11342709e3b27b2c9f590cf1/aaelf64/aaelf64.rst about how to use a constant pool to load an object's address from memory. Could you share the URL?

https://github.com/ARM-software/abi-aa/blob/2982a9f3b512a5bfdc9e3fea5d3b298f9165c36b/sysvabi64/sysvabi64.rst#code-models

And which code model? It looks like the "large" code model in the AArch64 psABI is different from this proposal because the AArch64's large code model requires that GOT is within 2 GiB from the text segment and seems like addresses are read from GOT.

And which code model? It looks like the "large" code model in the AArch64 psABI is different from this proposal because the AArch64's large code model requires that GOT is within 2 GiB from the text segment and seems like addresses are read from GOT.

I think you can find example at https://github.com/ARM-software/abi-aa/blob/2982a9f3b512a5bfdc9e3fea5d3b298f9165c36b/sysvabi64/sysvabi64.rst#get-the-address-of-a-symbol-defined-in-the-same-elf-file
I think the distance of GOT means the literal pool not normal GOT. Because it doesn't support PIC.

If "GOT" in the documentation doesn't mean the .got section, that's super confusing, but if that's the case, that's their problem and not ours. Thank you for pointing that out.

Large code models usually deal with both code and data. It's fine not to deal with large amounts of code, but it should be called out. Are we going to utilize range extension thunks to implement the large code model? I have some notes on https://maskray.me/blog/2023-05-14-relocation-overflow-and-code-models

I've added linker support for almost all Unix psABIs, except for MIPS and PA-RISC, to mold. This experience has given me insights into the topic. I've observed that the approach used in the PPC64 ELFv2 psABI appears to be the most effective for supporting a large code model, although it was designed not for supporting large code models but to compensate for the lack of PC-relative addressing in the ISA.
In the PPC64 ELFv2 psABI, the r2 register always holds the address of the GOT section. Consequently, obtaining an address stored in the GOT can be done in an r2-relative manner, even when the GOT is out of range. This effectively resolves the problem of large code models. However, this approach introduces a challenge: how to set r2 to the correct address. Since the GOT exists in each ELF file, r2 may have different values for different functions unless the ELF file is a statically-linked executable.
An important observation is that most function calls occur within the same ELF file. Therefore, recalculating r2 for every function call is a wasteful use of machine resources. Recomputing r2 is only necessary for inter-ELF-file function calls, meaning it is needed for calls via PLT entries or function pointers.
In the PPC64 ELFv2 psABI, each function has two entry points: global and local. The global entry point typically precedes the local entry point by a few instructions, and the instructions in between recalculate r2. When the linker resolves a function call relocation to a local function, it is directed to the caller's local entry point. Otherwise, it resolves to the global entry point. Given that most function calls are module-local, the overhead of r2 recomputation is minimal.
In cases where a function may call an external function, $sp+24 is reserved for the r2 save slot. A PLT entry saves the caller's r2 to the r2 save slot. After the function call, the caller restores its own r2 from the stack.
I believe an approach based on it could resolve the problem for large code models efficiently and work not only for position-dependent code but for dynamically-linked executables.

I've observed that the approach used in the PPC64 ELFv2 psABI appears to be the most effective for supporting a large code model, although it was designed not for supporting large code models but to compensate for the lack of PC-relative addressing in the ISA.

I will be nervous if RISC-V adopts PPC64 ELFv2-style TOC pointer. A large code model may be used by JIT when the address is known, and be used when the address is unknown (e.g. clang -mcmodel=large). For JIT use cases, there are usually very few calls from small code model code to the jitted code, so a lot of problems are not severe. However, for clang -mcmodel=large, it's easy to mix object files with different code models, and we should think about the result.
PPC64 ELFv2 supports both TOC and non-TOC, and the interaction is quite complex. For the curious, they can refer to "PPC64 ELFv2 TOC caller" and "PPC64 ELFv2 non-TOC caller" on https://maskray.me/blog/2023-02-26-linker-notes-on-power-isa.

Large code models usually deal with both code and data. It's fine not to deal with large amounts of code, but it should be called out. Are we going to utilize range extension thunks to implement the large code model? I have some notes on https://maskray.me/blog/2023-05-14-relocation-overflow-and-code-models

I thinks range extension thunk is worth to implement because it can reduce unnecessary loads for non-large function calls. But it need the linker support. Maybe we can list in TODO list?

In the PPC64 ELFv2 psABI, the r2 register always holds the address of the GOT section. Consequently, obtaining an address stored in the GOT can be done in an r2-relative manner, even when the GOT is out of range. This effectively resolves the problem of large code models. However, this approach introduces a challenge: how to set r2 to the correct address. Since the GOT exists in each ELF file, r2 may have different values for different functions unless the ELF file is a statically-linked executable.

I think this approch is too complicated to implement because it have to change current ABI.  Therefore, it may cause compatible issues.

I think so far we have 3 different approaches to implement the large code model, and one optimization for the function call.
Let me summarize the pros and cons of those 3 approaches:
| Size | Performance | Relaxable | Compatibility with other code model | Implementation effort | PIC support
-- | -- | -- | -- | -- | -- | --
Constant pool | Medium | Bad 7 | Yes3 | Yes6 | Low4 | No
Long instruction sequence | Large1 | Bad2 | Yes | Yes | Medium | Yes
TOC pointer | Small | Good | Yes | No | Medium*5 | Yes
*1 Most cases should be relaxable, then the code size is not really so huge.
*2 Most cases should be relaxable, then the performance impact is not really so huge.
*3 Harder than other options, we may did something like #397, but maybe need to add a few new relocations.
*4 Andes folk already offered the implementation for GCC and LLVM, and verified for years.
*5 I am not sure the exact implementation effort, but I guess that should be similar to GP-relaxation.
*6 It is also compatible with older toolchain releases since the current proposal didn't introduce new relocation types.
*7  Could be improved by linker relaxation, but not included in this proposal yet.
The capability of PIC support may not be a must requirement, one reference from AArch64 is they don't support PIC with the large code model.
And compatibility is important, otherwise it can only be enabled on those systems which have full control like Android or embedded system projects, or build a multi-lib for the large code model, which is not ideal.
So my thought is that a constant pool or long instruction sequence are preferred over TOC pointer schemes. I don't have a strong opinion on either constant pool or long instruction sequence scheme, or one thought in my mind: let we have both, and use command line options to decide which favor, one reason is I could imagine constant pool may faster than long instruction sequence on low end core, but get opposite situation on high end core, so having both maybe an good idea.

Range extension thunks is an add-on, it can combine with all 3 approaches, so we can implement that no matter which approach, one decision point here is should we make it mandatory as part of the large code model, this will add few implementation effort, but let us get better code gen for long jump.

FWIW, the long instruction sequence is likely a preferred method for calling a function when using CFI (https://github.com/riscv/riscv-cfi). With CFI, it’s necessary to have a "landing pad" instruction at the beginning of any function that might be called indirectly through a function pointer. Consequently, all functions would require a landing pad in the constant pool scheme, which could significantly increase the number of gadgets an attacker might exploit.

@rui314 I think that would be a problem in either scheme, since in either case the call sequence will end in a JALR instruction.  (Even though the long instruction sequence isn't loading the pointer from memory, the use of a JALR still engages the CFI ISA mechanism that requires a landing pad.)
However, the CFI mechanism offers an escape hatch in the form of a "software-guarded jump".  If rs1=x7 (for tail calls) or x1/x5 (for regular calls), then no landing pad is expected.  The software contract is that such a jump is only used when the pointer is known to be safe, which it should be in either case here.  (The assumption is that the constant pool is in an rodata section so can't be corrupted.)
So, I don't think CFI is a problem here, aside from constraining register allocation.

@rui314 that's good point, I think for this case we can use software-guarded jump, then we are free from that

Implementing range extension thunks shouldn't be too hard, but that would work only for code. If compiled machine code assumes both code and data are PC ± 2GiB, range extension thunks can solve only half of the problem.

Some comment from the last LLVM sync meeting:
Constant pool and long instruction sequence are both has it own use case, so we may allow both scheme and let user to choose which scheme should be used by some option, also same for function call.
Also some other comment from the last psABI call:
We didn't (officially) reserve intra-procedure-call scratch register, AArch64 has listed r16 and r17 ad IP0 and IP1, and explicitly say they may clobber during procedure call, that might be an issue when we implement range extension thunks .
However we actually already use t0, t1, t2 and t3 at PLT stuffs, so we could use same set of register to implement that, then we should specify that explicitly in the psABI, the only concern is it will seem like an incompatible ABI change, but this is less risky since it's kind of de facto behavior.

Some comment from the last LLVM sync meeting:
Constant pool and long instruction sequence are both has it own use case, so we may allow both scheme and let user to choose which scheme should be used by some option, also same for function call.
Also some other comment from the last psABI call:
We didn't (officially) reserve intra-procedure-call scratch register, AArch64 has listed r16 and r17 ad IP0 and IP1, and explicitly say they may clobber during procedure call, that might be an issue when we implement range extension thunks .
However we actually already use t0, t1, t2 and t3 at PLT stuffs, so we could use same set of register to implement that, then we should specify that explicitly in the psABI, the only concern is it will seem like an incompatible ABI change, but this is less risky since it's kind of de facto behavior.

No; using custom calling conventions within an object has always been allowed (and that’s a thing that’s done across architectures), but range extension thunks clobbering registers that weren’t previously reserved for it would break that. It’s only safe to do in the PLT case because people know PLTs exist and they need to be careful.

No; using custom calling conventions within an object has always been allowed (and that’s a thing that’s done across architectures), but range extension thunks clobbering registers that weren’t previously reserved for it would break that. It’s only safe to do in the PLT case because people know PLTs exist and they need to be careful.

Yeah, fair enough, so I think let moving forward without range extension thunks, then extend that later with necessary changes (e.g. adding new tag) if needed

ping @MaskRay @rui314 , would you like to give some blessing to moving this forward?

My biggest concern here is that we're allocating the name "large" and creating a compatibility promise for a short-term code model.  If in the future we have a fully designed large model, gcc won't be able to switch to it for -mcmodel=large because that will regress functionality for anyone with an old binutils, so the new, better code model will be stuck with a worse name.
@kito-cheng There is a fourth option - use a real GOT.  RISC-V does not have a meaningful concept of a GOT base, so there's nothing forcing the GOT to be contiguous; interleave text and GOT in 4 GiB chunks to support GOTPCREL_HI20 relocations in the large model.  Obviously this won't work if you're generating a.out and need RX and RW memory to be a single contiguous range each, but it should work for ELF.
I'm a strong supporter of range extension thunks and implemented them for the riscv Go linker a while ago.  Ideally we would support them with both 4-byte and 8-byte call sites, which means we need a new relocation type JAL_THUNK anyway, so adding CALL_THUNK might not be so bad.

I think I can post this here for some brief: Am running Ubuntu 20.04 L.T.S on AMD 64-bit processor and I got a compiler error when executing my linked embedded python into caffe framework. The compiler tells me to recompile with -fPIC. This causes memory relocation and I don't know why the linker and the compiler are failing to use a linked static library. My caffe build is a large code-base of more than 35GB. The program is built from source as per the manual. I have tried to look for solutions on Stack-overflow with no success. I would really appreciate for your assistance. Kivoi Musa

This is the specification for the RISC-V instruction set's ABI, and your 64-bit AMD processor is not a RISC-V processor; unless you're cross-compiling for RISC-V (doubtful?) you seem quite lost and this is not the place for this kind of question since it's for a completely different processor instruction set.

@sorear
I incline to accept current proposal with optional range extension thunk*1 support, we already have note say we may have other code generation strategies, so it let us have room to add more large code model variant in future, I am not really comfortable with the multiple GOT design, that's complicate and it would be challenge on the customized linker script to specify that.
*1 Add note to mention function call may use auipc+jalr sequence if linker support range extension thunk.

Will moving forward/merge this PR after next psABI meeting, GCC already merged for a while, and LLVM also provided PoC.


filter unicode fix

Fixes #2951
Hi! Initially I thought this should be easy to fix but struggled due to wrong approach. Later I found that pyparsing  included several unicode character ranges that can help to set non-ascii filters.
Would be Glad to add if I missed something.

Thanks! 🎉 👍


Authentication system

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Please give infomation about the issue your having so we can help you out

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add Beginners-friendly repositories list

I was looking for a list of beginners-friendly repositories (that have appropriate labels for issues) to share with my junior-level colleagues.
Unfortunately such a list did not exist, so I've compiled it myself using information from @yourfirstpr Twitter account, various blog posts and GitHub search.

Please read the instructions for creating a new list, contributing guidelines and the awesome manifest.
Especially this part:

Wait at least a couple of weeks after creating a list before submitting to give it a chance to mature.

Ping me here in a few weeks and I'll reopen ;)

Awesome idea :)

Thanks! Yeah, read the guidelines, but missed the "maturity" part. Will change the title case as well.


Add mentor mentee guides

Purpose

The purpose of this PR is to fix #139
Goals

Add mentor mentee guides
Approach

Added mentor mentee guides
Screenshots

Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Thanks @anjula-sack


Update numerics_and_downstream_ml.ipynb

readability additions to downstream ML PyCaret notebook - instructions, cleaned up deprecated blueprint

@hhpang 👋🏾 are these changes required for Console release?

@hhpang 👋🏾 are these changes required for Console release?

@duggiemitchell Yes, we do want it as part of the Console release, although we don't have to block on it.

thanks all


Dual Stack support

Type
New feature or Enhancement of existing feature
Description
Is there any plan for supporting Dual stack in near future ?
As we all know that Kubernetes v1.20+ started supporting Dual stack and it is a stable feature now.
Also, at present,  is there any method of assigning multiple IP addresses from different ranges ? (I couldn't find one)
If it exists, we can start developing Dual stack feature on top of it.

Thank you for your response @maiqueb :)
Sure, that will be a good start. Thank you for inviting me to the bi-monthly call.
Can you please let me know the schedule of the same ?
I could only find the info. about bi-weekly call here.

Thank you for your response @maiqueb :) Sure, that will be a good start. Thank you for inviting me to the bi-monthly call. Can you please let me know the schedule of the same ? I could only find the info. about bi-weekly call here.

Oh I just had my wording word and confused bi-monthly with bi-weekly. That's the one - once every 2 weeks.

Sounds cool, and hope to see the dual stack support soon.

@dougbtv can you please assign this issue to me based on #250 ? :)


PullRequest

i just changed lots code for use my own program. and i request pull that want you update nuget :)

Thanks for the PR! Bit busy this week but will take a look over next few days. At a glance, I see options to set a personal access token. Need to closer but still possible to use anonymous github calls too?


AOT Rollup : 'TextMaskModule' is not exported by node_modules/angular2-text-mask/dist/angular2TextMask.js

Hi,
I'm trying to get AOT working on my project but am facing the following error with Rollup :
Export 'TextMaskModule' is not defined by '/path-to-project/aot/app/app.module.ngfactory.js'
'TextMaskModule' is not exported by node_modules/angular2-text-mask/dist/angular2TextMask.js (imported by app/app.module.js).
The script I'm doing is : ngc -p tsconfig-aot.json && rollup -c rollup-config.js
The ngc part works but when rollup doesn't like something...
angular2-text-mask version : 2.0.0
text-mask-core : 0.16.4
Note that the plugin is perfectly working without AOT.
My rollup config :
import rollup      from 'rollup'
import nodeResolve from 'rollup-plugin-node-resolve'
import commonjs    from 'rollup-plugin-commonjs';
import uglify      from 'rollup-plugin-uglify'
//paths are relative to the execution path
export default {
    entry: 'app/main-aot.js',
    dest: 'aot/dist/build.js', // output a single application bundle
    sourceMap: true,
    sourceMapFile: 'aot/dist/build.js.map',
    format: 'iife',
    plugins: [
        nodeResolve({jsnext: true, module: true}),
        commonjs({
            include: ['node_modules/rxjs/**']
        }),
        uglify()
    ]
}

Thanks for your help !

I had to add the text-mask directories to the commonjs module.  I think if you add that, rollup should work.  Did for me...  Add this to the include configuration (where we have rxjs):
'node_modules/angular2-text-mask/',
'node_modules/text-mask-core/',
'node_modules/text-mask-addons/**',

Thank you !
Solved this issue by adding to my commonjs include array :
'node_modules/text-mask-core/dist/textMaskCore.js', 'node_modules/angular2-text-mask/dist/angular2TextMask.js'


Legal Disclaimer

Add to landing page:
Please refer to Sections 4.3 and 4.4 of the RTP for details regarding ownership of responses and collection and use of information.

Update to content for #56


Correctly represent pseudonyms and alternative names

We integrate person records of different data sources that follow different policies with respect to pseudonyms.
For example, at KBR there is usually one record for both the real person and all the pseudonyms, whereas other data sources have one record per pseudonym. For ISNI the policy is that each pseudonym gets its own ISNI identifier.
However, there might be different cases at KBR:

One person record about the real entity plus information about pseudonyms
One person record about a pseudonym plus information about the real entity
Different person records for the real entity and the pseudonym(s)

A general data quality remark: For books written via a pseudonym that link to a person record (cases 1 and 2), it is not clear from the structured data if the book was written by the real entity or the pseudonym. However, there might be additional information in a free text field to clarify this at least for human users.
We have to come up with a proper way to encode pseudonyms and alternative names in our RDF data model and map data accordingly.
KBR
For all cases: indications of pseudonyms via subfield $c may be p, pseudonyme, pseud., ... or any combination with other information, e.g. p, scénariste de bande dessinée.
Case 1
A Person record may link to different alternate names via field 400. Multiple such fields may exist and if it is a pseudonym name, there is an indication in the subfield 400$c. We can create a separate authority identifier and a link to it, based on the person record identifier and the sequence number in subfield $#.
For example
btid:authority_14150228
    a schema:Person ;
    label "Vandersteen, Willy" ;
    hasPseudonym btid:authority_14150228_0 ;
    hasPseudonym btid:authority_14150228_1 .

btid:authority_14150228_0
    a schema:Person, bdm:Pseudonym ;
    label "Wil" .

btid:authority_14150228_1
    a schema:Person, bdm:Pseudonym ;
    label "Wirel" .

(for this example with the collective pseudonym "Wirel", please also check #185)

[x] Extract KBR pseudonym and alternate name information for person records from field 400 info
[x] Create RDF pseudonym and alternate name information for person records
[x] Load pseudonym and alternate name information for person records to our database
[x] Add a pre-processing script that adds sequence numbers in case none are provided

Case 2
A pseudonym record may link to different alternate names via field 400. Multiple such fields may exist and if it is another pseudonym name, there is an indication in the subfield 400$c. We can create a separate authority identifier and a link to it, based on the person record identifier and the sequence number in subfield $#.

[x] Extract KBR pseudonym and alternate name information for pseudonym records from field 400 info
[x] Create RDF pseudonym and alternate name information for pseudonym records
[x] Load pseudonym and alternate name information for pseudonym records to our database

Case 3
A person or a pseudonym record might link via field 500 to a separate (other) pseudonym. In such a case, the subfield 500$c contains an indication for pseudonym. We may have cases where we only have pseudonyms (both the source pseudonym record and the target 500-link to another pseudonym) but no link to a real identity. We can create a link between the records.

[x] Extract KBR pseudonym information from field 500
[x] Create RDF pseudonym relationships between KBR IDN and linked 500 IDN
[x] Load pseudonym information from 500 fields and links to our database

BnF
At BnF, different identities have different authority records. However, there seem to be no property connecting identities, such as real persons with their pseudonyms.
KB
Different pseudonyms seem to have different authority records. However, it is not explicitly indicated via type what the pseudonym and what the real identity is (if any). For example:
John Flanders (p069202990) of type schema:Person links via schema:sameAs to Jean Ray (p06902684X) of type schema:Person and vice versa (this is not wrong as a schema:Person can be real or fictional, however, in this situation it does not express the difference between the two)

Both have a schema:description (free text) Pseud. van De Kremer, Jean Raymond Marie
the first has a schema:description (free text) Zie ook Jean Ray
the second has a schema:description (free text) Zie ook John Flanders

It seems that there should be a third record with name Jean Raymond Marie De Kremer which is the real identity. Yet, there is no link to such an authority.
However, in some cases there is also an authority record for the real identity, but still without meaningful links.
Joseph Lous Baronian (p152741429) links via schema:sameAs to Jean-Baptiste Baronian (p068872445) and to Alexandre Lous (p073881198)

the first (real identity) has two schema:description values with "see also" (zie ook) similar to Jean Ray above
the second and third indicate with a schema:description value Pseud. van Joseph Lous Baronian

Many KBR records wrongly list even alternate names as pseudonym. This would make the list of person contributors unnecessarily long.
We thus decided to keep the person contributor list as is for now. We only indicate pseudonyms manually in the correlation list.


Can't see map in iOS

Hi, i'm using TK.CustomMap in a cross-platform project.
In Android works fine but in iOS i can't see the map.
In y appDelegate i have this code:
global::Xamarin.Forms.Forms.Init();
Forms.Init();
FormsMaps.Init();
Xamarin.FormsMaps.Init();
TKCustomMapRenderer.InitMapRenderer();
NativePlacesApi.Init();
LoadApplication(new App());
return base.FinishedLaunching(app, options);
Missing something?
Itryed also this without succes:
var renderer = new TKCustomMapRenderer();
LoadApplication(new App());
return base.FinishedLaunching(app, options);

We had the same issue, please change the map layout options and see.
myMapView.HorizontalOptions = LayoutOptions.FillAndExpand;
myMapView.VerticalOptions = LayoutOptions.FillAndExpand;

I have same problem. I'm also wondering where iOS is google map key. Does it not use google map infrastructure? He didn't write a book for appdelegete.


fix(Embed): do not show control when not appropriate

This PR resolves issue #1767, which states that:

the pause control button is visible, even if we are using an iframe in Embed component.

Changes

control button is not shown when not appropriate
background set to grey colorScheme.brand.background
added YouTube example
changed icon from Play to Search, on the example with map

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

These changed will be integrated in #1758 instead.


Fix for LNK2019 error when compiling tf1.9 with CMake on Windows due to missing definition.

This is the suggested fix for the missing NumHyperthreadsPerCore definition #19761 which is still present in release 1.9.

I signed it!
On 12 July 2018 at 12:45, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/20733#issuecomment-404483888,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjCqnxxW_SfWl7LPDFkJSrnE00qaAAGWks5uFzbzgaJpZM4VMsYK
.

To clarify, this is only an issue only appears when building with MKL on Windows, which is maybe how it didn't show up in testing.

Hi, is there any update on when this will be reviewed? Thanks.

Sorry for the delay. Adding @andrehentz as reviewer.

Nagging Reviewer @andrehentz: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Ping  @andrehentz again for review.

Adding @gunan since this is push to R1.9.

1.10 is out and we are preparing to cut 1.11.
We wont be creating a new tag, and publishing new binaries with this fix.
I would consider 1.9 branch closed at this point.


AppVeyor broken

Appears the repo got created on AppVeyor. However, there is no pending build and trying to start a build fails. Any ideas, @pelson?

Looks to be queued in #2

Well that's good. Had no luck with master earlier. Will retry.

Was able to retrigger through the webhooks. Also reshuffled the builds so that master will build before that PR does FYI.


Difficulty connecting to Google IoT Core

I'm trying to connect to Google IoT Core which uses TLS 1.2 and requires it's root certificates. I'm setting those by filename with:
client->set_ca_cert_file(roots);
where roots is "roots.pem" and is the filename for a certificate file in pem format downloaded from Google. The file is in the current directory when the code is run.
Has anyone had luck connecting to Google IoT with mqtt_cpp?

Have you called
set_default_verify_paths()
or
set_verify_mode()
or the other functions,
to set the SSL behavior?

No, I haven't seen those. I followed the "mqtt_cpp/example/tls_client.cpp" example code and it only calls:
set_ca_cert_file()
I will have a look at those two methods. Thank you.
The roots.pem file Google expects contains a number of certificates, it's in the current directory, and I pass simply the filename to set_ca_cert_file().
I'm very excited about this library and I'm looking forward to completely replacing paho.mqtt.cpp with it. Thank you for your help and patience.

I suspect what you might be running into is that the Google IOT service is using TLS 1.2, and the Boost ASIO defaults to a different version of TLS, so it's getting grumpy.
You can also set callbacks to the ssl context, and the on_error functions for mqtt_cpp, which can give you more information about why it's failing.

That's a good point. Google does require TLS 1.2. I'll try again with some callbacks set.

Actually, no... But I got a little time to work on this and decided to take just the tls example code and add the jwt token generation and other Google IoT Core idiosyncrasies. It connected immediately and I saw all the callbacks. So, I'll close this issue since clearly it's not related to Google IoT. If I get clean working code, maybe I'll open a pull request to add a connect option for Google IoT Core if anyone is interested.
Weirdly, my project code that fails uses almost exactly the same functional code as the tls example, but with the tls client as a member of the class. It doesn't so much fail, as I never see a single callback. The only difference in the setting of the callbacks is that I'm capturing "this" in the lambda for some of them, including connack_handler. Maybe I'll try dropping that and see if it's called next.


Add alt, width and height to page image

Currently we are not able to retrieve page image alt width and height in twig, these values are important for SEO if you are implementing schemas that are used by crawlers to get details about your page.

fixed with 3.0.0 release


Is this project being maintained?

I've noticed that the last commit was some time ago. and that there are quite a number of open issues with no reply.

That's a important question...

@joyent would you find another maintainer ?

We've been maintaining an internal fork up to this point. This project hasn't seen any activity in some time.

Duplicate of #488 (well, 488 is the dupe, but it has more discussion).

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


elasticsearch: 1.7.2 -> 2.1.0

Tested the build and the module.
Did not get it to work with the security manager.

Elasticsearch 2.x is backwards-incompatible with 1.x. It must not be replaced like that.

@lethalman, ok. Should I put them into a different package and module (elasticsearch2)?

@markWot it can be a different .nix in the same folder, and attribute elasticsearch2.

@lethalman ok thank you very much. I will update it later.


Adding prototype.evm_decreaseTime

It's currently hard to perform tests that are timestamp dependent since the timestamp changes between tests are shared (E.g. see #2, Ganache-cli-issue-390 and here) . Using evm_increaseTime leads to non-reversible timestamp changes that persist through all the tests.
Adding evm_decreaseTime allow developers to revert the timestamp change after each test, using a logic such as
beforeEach(async function () {
  await web3.currentProvider.send({
      jsonrpc: "2.0",
      method: "evm_increaseTime",
      params: [timeToIncrease],
      id: 0
  });
});

afterEach(async function () {
  await web3.currentProvider.send({
      jsonrpc: "2.0",
      method: "evm_decreaseTime",
      params: [timeToIncrease],
      id: 0
  });
});

@roderik could you elaborate, please? By any chance is your code open? A link to a repo or a gist would help my feeble mind tremendously! 😄

@benjamincburns Thank you for the response!
I believe trufflesuite/ganache-cli#390 is fixed in this Pull Request (see my comments on that pull request).
I'm in favor of either method.

Sure,
look at this repo: https://github.com/DataBrokerDAO/dtx-crowdsale-contracts
I worked around it in this case though.
I like my source files small, so it is more easy to get an overview of what is happening. Which means I would make lots of test files with specific things I wanted to test.
But multiple tests run at a random order. So if the first test that runs is one that jumps in time, all the other tests would run at that time in the future.
So If i wanted to test "cannot contribute before the sale" and "can contribute during the sale", it would be total guess work if they would work, because this depends if the "during saler" test runs first because it jumps ahead in time.
While the entire idea of the snapshots is that you start "clean" (but still faster than redeploy every time) at the beginning of each file.

While the entire idea of the snapshots is that you start "clean" (but still faster than redeploy every time) at the beginning of each file.

@roderik as an aside, I've suggested internally that a cleaner approach to this might be if our tests were to use the forking feature of ganache-core. That would also enable tests to be run in parallel against multiple chains which were forked from an initial chain post-setup.
I'll raise it again during our next internal planning meeting.

No, seems fine, do not have anything to test it on right now :)

@roderik I think you were the only one still advocating for this issue now that #2 is merged. Can you please reach out to me at<EMAIL_ADDRESS>if this is still important to you?
Otherwise in the mean time I'm closing this to clean up our backlog some.

I would find this useful for gas-profiling contracts where a timestamp is passed as a parameter, because the zeroeness of the bytes of the timestamp affect the gas consumption. I can then profile as if it was way in the past without reconfiguring ganache for just this test.
Instead of evm_decreasetime, I think a better interface would be passing a negative value to evm_increasetime.


Add build tags for non-Linux or with CGO disabled

This allows for easier embedding in other projects.

Closing as switched to the external binary rather than embedding.


How can I determine the pyswagger version programatically?

Most packages have a version attribute, but pyswagger does not.  I have perused the source, and I haven't found any place where the pyswagger version is stored.
Am I missing something?
If not, can you please add a version attribute?
Thanks.

@MartinDelVecchio  do you mean something like this? It indicates a __version__ property instead of version one.

Ah, my double underscores were eaten by the markup.
Yes, most of the packages I use have a version attribute.  Usually a string such as '0.8.33', and I check minimum required versions like this
min_pytest_version = (3, 0, 7)
pytest_version = tuple (map (int, pytest.__version__.split ('.')))
if pytest_version < min_pytest_version:
    print >>sys.stderr, "Error:  requires py.test version %s; running %s" % (".".join (str (digit) for digit in min_pytest_version), ".".join (str (digit) for digit in pytest_version))
    sys.exit (1)

@MartinDelVecchio  no problem, I'll add it in next release.

Thank you!
I wish I could star this project 100 times; it is exactly what I needed, and you have been super responsive to my pestering.

I just want to make this project a reliable component when using OpenAPI in python. Wish you could use it more and find more bugs before me :)

This issue should be fixed in v0.8.35, please feel free to open it if it's not working.

Verified.  Thanks!


MIC-748 Remove /prisoners/:prisonNumber route

MIC-748
Remove /prisoners/:prisonNumber route,
and Fix pagination component

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.10%. Comparing base (6122e7b) to head (de6787a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
- Coverage   92.15%   84.10%   -8.05%
==========================================
  Files         345      343       -2
  Lines        5137     5109      -28
  Branches      787      786       -1
==========================================
- Hits         4734     4297     -437
- Misses        399      792     +393
- Partials        4       20      +16

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


create launch strategy for SpeckJS

Draft the post that will be used for Hacker News, Reddit, and various social media outlets

List I have so far of places we should post.
List of websites for exposure:
Hacker News - https://news.ycombinator.com/
Echo JS - http://www.echojs.com/
Subreddits:
Node - https://www.reddit.com/r/node
JavaScript - https://www.reddit.com/r/javascript


how to open ipynb file in atom ?

hi Sir :+1:
I have installed jupyter-notebook  but I can not find .ipynb file (which is like your example in setting .)
would you show this details in your setting descriptions?
Thanks.

This repo is not being actively maintained 🙁 In the meantime, I suggest that you look at nteract, which is a Jupyter notebook client built using Electron (the same native app shell that is used by Atom), React, Redux, RxJS, etc. and it has a lot of momentum. It goes without saying that you also have Jupyter Notebook and its WIP successor JupyterLab.
Having said that, nteract is currently extracting its many components into separate packages which will allow them to be reused and assembled to create new apps. When I have the time, I would like to use them to completely refactor this project.


Codeunit 5763 new event on procedure PostSourceDocument

Hi,
We need a new event on codeunit 5763 procedure PostSourceDocument :
....
    case WhseSetup."Shipment Posting Policy" of
                            WhseSetup."Shipment Posting Policy"::"Posting errors are not processed":
                                TryPostSourceTransferDocument(TransferPostShipment);

                            WhseSetup."Shipment Posting Policy"::"Stop and show the first posting error":
                                PostSourceTransferDocument(TransferPostShipment);
                        end;
                        // > New Event
                        OnAfterTransferPostShipment(TransHeader, WhseSetup, CounterSourceDocOK);
                        // < New Event
                        if Print then begin
                            IsHandled := false;
                            OnPostSourceDocumentOnBeforePrintTransferShipment(TransShptHeader, IsHandled, TransHeader);
                            if not IsHandled then begin
                                TransShptHeader.Get(TransHeader."Last Shipment No.");
                                TransShptHeader.Mark(true);
                            end;
....

   [IntegrationEvent(false, false)]
    local procedure OnAfterTransferPostShipment(var TransferHeader: Record "Transfer Header"; WhseSetup: Record "Warehouse Setup"; var CounterSourceDocOk: integer)
    begin
    end;

There are already existing events OnAfterTryPostSourceTransferDocument and OnAfterPostSourceTransferDocument, please use those instead.
If it's not possible to use those create a new issue and specify a scenario why those can't be used.


SVG-YAML Mapping issue

Dear Andrew,
I'm trying to setup my first svg from drawio, but looking at the Mapping log, it seems the svg items cannot be mapped correctly by the yaml file.
In a test, I took the default SVG example that pops up creating a new Flow panel and I edited it with drawio to change the svg cells IDs. I uploaded the new file and also updated the yaml mapping accordingly and everything works fine.

If, however, I change the position of a cell in the SVG, the mapping is completely lost, even though the cell IDs have remained the same.

For the same reason, I cannot successfully create a new svg and map it to test data
Could you please help me understand what am I missing?
Thank you so much
Regards

All I can think is you are somehow removing the first cell and creating a new one. Or somehow hiding the one you are driving behind a second.
The Debugging.mappings button will show you what cells are mapped and what aren't. The debugging.svg will print the svg after all annotation is applied.
Copy the original and updated and post annotation svgs into a prettier (just Google svg prettier or formatter) and you should be able to see what's going wrong.
If you still can't see it take the working svg and reduce it down to a single cell. Check it still works, then apply your breaking change and see whats happening in your svg
Hopefully those steps will get you going

Update: it seems like when saving the SVG in draw.io, the cell IDs get messed up, draw io saves the IDs in another format than that indicated in the guide, so it cannot correctly match the IDs thorugh the YAML.
Please not that it does this with both the desktop version with the svgdata plugin installed and also with the web version as well.

What desktop version and os are you using?
And if you create a cell id  of 'abc' how does it appear in the svg?

If after trying the above you are still having problems, you'll need to post the svg source xml from before the transformation and after (you get the after in the console via Debugging->svg). With those I can at least see what's happening and maybe able to help. Your posts above appear to just be images which don't allow me to look deeper. If there's a way for me to convert the above images to the svg xml, do just say and I'll have a look.

After some debugging I found the issue, it's in draw.io not in the plugin :)
Draw.io, both in the latest desktop or web version, saves the cell IDs in a format slightly different than that expected, so the yaml file cannot map the cells correctly.
We can close the issue since the problem is in draw.io saving method and not in in the plugin.
Thank you very much for the prompt support


Use the strict UNIX_LINES mode to do regular expression parsing

Recently we added RegexMessageParser to parse the text log file, occasionally I saw the error message about cannot parsing some messages.
After investigation, I realized that the problem is some text log contain non-standard unicode (e.g. 0x85) in the url field, these 0x85 char are being treated as newline in java.text.Pattern.
Fix this by using strict UNIX line parsing (i.e. only recognize 0x0a as the line terminator)

r+


Update gokan-ozcifci.yml

@xGokan
Your change in this PR has been published. Please check it: https://rd.microsoft.com/en-us/


questions about continuous data generation

Hi, I have read your impressive paper about ehr data generation. I tried to generate continuous data using this code, both github issues and paper mentioned that HALO can generate continuous data like lab values. I also noticed there is a variable "tsDf" in "genDatasetContinuous.py" reading timeseries data and I assumed "tdDf" contains lab values. But there is no csv files in MIMIC with format like "episode{visit_count}_timeseries.csv", so I am wondering how to process lab values dataset. I would be appreciated if you are so kind to provide the script of processing labevents and d_labitems in MIMIC, or the detailed format of "episode{visit_count}_timeseries.csv". Thanks!

Hello, my apologies for the limited documentation regarding the usage of the continuous data generation module. I've had a number of questions about that code, and we have found a number of errors stemming from subsequent changes since the experiments. I have also realized that given the interest, this section requires its own specific README guide, and I have added one to that directory. However, the answer to your specific question is that the time series .csv's come from preprocessing performed in the https://github.com/YerevaNN/mimic3-benchmarks benchmark repository (this is also the source of phenotype mappings used in the main experiments). That repository has good documentation, but you just need to get through the extract_episodes_from_subjects and then point the timeseries_dir variable to the correct spot. Hope this helps, and please let me know if you have any further questions.

auto reply


Could not find com.github.yalantis:ucrop:2.2.2 again

Could not find com.github.yalantis:ucrop:2.2.2.
Searched in the following locations:

https://dl.google.com/dl/android/maven2/com/github/yalantis/ucrop/2.2.2/ucrop-2.2.2.pom
https://dl.google.com/dl/android/maven2/com/github/yalantis/ucrop/2.2.2/ucrop-2.2.2.jar
https://jcenter.bintray.com/com/github/yalantis/ucrop/2.2.2/ucrop-2.2.2.pom
https://jcenter.bintray.com/com/github/yalantis/ucrop/2.2.2/ucrop-2.2.2.jar
Required by:
project :app > com.github.dhaval2404:imagepicker:1.3

my gradle
repositories {
google()
jcenter()
maven { url "https://jitpack.io" }
}

i am url   - https://jcenter.bintray.com/com/github/yalantis/ucrop/2.2.2/ucrop-2.2.2.jar
{
"errors" : [ {
"status" : 404,
"message" : "Could not find resource"
} ]
}
please update

@harloom Strange! Looking into it now.

iam sory ,because i dizzy today


missing step in tutorial

Following the instructions, this error is reported:
1>C:\Program Files\dotnet\sdk\7.0.203\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(266,5): error NETSDK1004: Assets file 'xxxxx\obj\project.assets.json' not found. Run a NuGet package restore to generate this file.

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

ID: ab408035-6313-a183-9940-32de00e0e977
Version Independent ID: ab408035-6313-a183-9940-32de00e0e977
Content: Tutorial: Create a minimal API with ASP.NET Core
Content Source: aspnetcore/tutorials/min-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @wadepickett
Microsoft Alias: wpickett

@ehardebeck thanks for letting us now.  I will test through this topic as soon as I have a chance.

@ehardebeck, I tested through the entire tutorial with the lastest VS 2023 +.NET 7.0 and the latest packages (not previews):
Microsoft.EntityFrameworkCore.InMemory  (v. 7.0.5)
Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore v. 7.0.5)
I could not recreate the issue you are running into.  It ran as expected.
You might take a look and see what package versions you currently have in the project.  Verify you have both I listed above and both are the same version 7.0.5.
In Visual Studio, right-click on your TodoAPI project, select Open in terminal, and and run the following command to get details on the installed packages:
dotnet list TodoApi.csproj package --verbosity d

The expected output should be the following:
Project 'TodoApi' has the following package references
   [net7.0]:
   Top-level Package                                           Requested   Resolved
   > Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore      7.0.5       7.0.5
   > Microsoft.EntityFrameworkCore.InMemory                    7.0.5       7.0.5

Since it is not clear what state your app is in, you might have to restart with the tutorial and a new project.
There were some minor changes in the UI for the recent Postman app update, which I have an issue for already which I will address in a doc update, but nothing related to your issue here.  Just that the JSON setting is under
I am going to close this issue, but feel free to reopen if you have a new insight on a change that is needed in the tutorial.  Thanks again for letting us know your experience.


implemented dynamic metadata

Current dependencies on/for this PR:

main

PR #30

PR #31
PR #25

PR #26

PR #33

PR #34   👈

This comment was auto-generated by Graphite.


KAS-4130 Added signflow status codelist

Provisions for status concepts to be linked to sign flows.
Includes 1 migration (uuids were generated using SPARQL on ACCEPT) and 1 model addition.
Related PR: https://github.com/kanselarij-vlaanderen/frontend-kaleidos/pull/1770
Related service: https://github.com/kanselarij-vlaanderen/signflow-status-sync-service

For some reason this PR is not syncing with the actual branch... It's 3 commits behind


NativeScript actionbar add Icon

Hi Guys, How to add something icon beside title Bar in Native script, would you like to help me, please
Thank in advanced

Hi @culunvb,
You could review the both examples in nativescript-sdk-examples-ng, where has been shown, how to use systemIcon  or icon from the resources in ActionBar component.
For further How to questions, please use our StackOverflow thread here.


Cleaned Up Wawawa

I cleaned up the code for Wawawa to make it easier to modify in future. This mainly involved removing repetitive chunks of code that appended slightly different lines to the char under different conditions. I also placed Wawawa's option inside a .pcstats table to make it look neater.

Nothing to see here. whistles


JAMES-2586 Implement BlobReferenceSource(s) for postgres-app

Please write integration tests for bob deletion using the deduplicating blob store.

The only integration test I found for blob GC is RabbitMQWebAdminServerBlobGCIntegrationTest in distributed-webadmin-integration-test. Should we have integration test for this ticket covered in the ticket Webadmin integration tests for postgres-app (https://github.com/linagora/james-project/issues/5005) ?

I would be ok with it

Then a demo video so that I am sure such an important topic received the attention it requires:
GIVEN I start with generation 1
AND I create 2 messages with attachent A
AND I delete one message
WHEN I reboot the server on generation 2 and run GC
THEN I expect no deletion

GIVEN I start with generation 1
AND I create 2 messages with attachent A
AND I delete two message
WHEN I reboot the server on generation 2 and run GC
THEN I expect deletion of attachent A content

BTW it is not too late to write such an integration test ;-)

RabbitMQWebAdminServerBlobGCIntegrationTest in distributed-webadmin-integration-test

Would do of course.
But I need to see a github ticket about it so that is is not forgotten.
(and the mantra : what is not tested do not work - might be relevant here)
Time to fasten that webadmin integration test for PG app?

integration test ticket created https://github.com/linagora/james-project/issues/5031

integration test ticket created https://github.com/linagora/james-project/issues/5031

https://github.com/linagora/james-project/issues/5005
Why another ticket?

integration test ticket created linagora#5031

linagora#5005 Why another ticket?

https://github.com/orgs/linagora/projects/2/views/2?pane=issue&itemId=48767936 just implements webadmin-integration-test-common. Other tests wuould be added in another ticket.

Note: I see the todo comment in the code, can you resolve it in this pr?

// should remove this after https://github.com/linagora/james-project/issues/4998

https://github.com/apache/james-project/blob/3e86c78b58f53e50606aea06adeb17ab2a778861/server/apps/postgres-app/src/main/java/org/apache/james/PostgresJamesServerMain.java#L142

CI fail
shouldPreserveDsnParameters – org.apache.james.mailrepository.postgres.PostgresMailRepositoryTest

javax.mail.internet.AddressException: Out of data at position 1 in 'dsn-ret' in string ``dsn-ret'' at position 1
Stacktrace
com.github.fge.lambdas.ThrownByLambdaException: javax.mail.internet.AddressException: Out of data at position 1 in 'dsn-ret' in string ``dsn-ret'' at position 1
	at com.github.fge.lambdas.supplier.ThrowingSupplier.get(ThrowingSupplier.java:27)
	at org.apache.james.mailrepository.postgres.PostgresMailRepositoryContentDAO.asMailAddress(PostgresMailRepositoryContentDAO.java:297)
	at org.apache.james.mailrepository.postgres.PostgresMailRepositoryContentDAO.lambda$recipientToHeaderStream$16(PostgresMailRepositoryContentDAO.java:291)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1655)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)

Don't forget update the document for new postgres properties

CI fail
shouldPreserveDsnParameters – org.apache.james.mailrepository.postgres.PostgresMailRepositoryTest

javax.mail.internet.AddressException: Out of data at position 1 in 'dsn-ret' in string ``dsn-ret'' at position 1
Stacktrace
com.github.fge.lambdas.ThrownByLambdaException: javax.mail.internet.AddressException: Out of data at position 1 in 'dsn-ret' in string ``dsn-ret'' at position 1
	at com.github.fge.lambdas.supplier.ThrowingSupplier.get(ThrowingSupplier.java:27)
	at org.apache.james.mailrepository.postgres.PostgresMailRepositoryContentDAO.asMailAddress(PostgresMailRepositoryContentDAO.java:297)
	at org.apache.james.mailrepository.postgres.PostgresMailRepositoryContentDAO.lambda$recipientToHeaderStream$16(PostgresMailRepositoryContentDAO.java:291)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1655)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)

Don't forget update the document for new postgres properties

where is the document?

where is the document?

sorry, I rechecked it again, now we haven't documented for it, just add a new property and comment in the sample configuration file https://github.com/apache/james-project/blob/c580f1db2284f380b1171633820512cd10e70fb3/server/apps/postgres-app/sample-configuration/postgres.properties

added

Please rebase


Crash on login and other misc bugs

First off, I've noticed that logging in occasionally causes the server to crash, and this is what the console says when the server crashes.

Second, I've noticed that other players receive the error "Unable to retrieve ponies from database" in the character creator when they try to log in after I apply an update to my server.  The only workaround to this problem I could find is to delete everybody's ponies from the database.
Third, I've noticed that the database won't save a pony if it has the same name as another pony, even if both have different designs.  All the error message says is "Error while saving pony".
And last, on rare occasions, the portals don't work.

I've noticed that using patched.dlls does reduce the amount of times the server crashes when someone connects to it, but doesn't completely eliminate the problem.


Add kernel release check to environment_check.sh

longhorn/longhorn#6854

@mergify backport v1.5.x v1.4.x

ping @derekbit @innobead


Meet.html css updated,card size fixed,ui improved

Issue #3 resolved!
the following screen shot shows the change!

Cool, merged! 🎉


EC-Earth3-Veg e2e DTR run fails at validate-biascorrected

Run : https://argo.cildc6.org/archived-workflows/default/ae3327f2-e46a-4908-a661-d049677a69cf
Log :
Validating gs://biascorrected-492e989a/stage/CMIP/EC-Earth-Consortium/EC-Earth3-Veg/historical/r1i1p1f1/day/dtr/gr/v20211221191100.zarr
/opt/conda/lib/python3.9/site-packages/dask/utils.py:35: RuntimeWarning: All-NaN slice encountered
  return func(*args, **kwargs)
/opt/conda/lib/python3.9/site-packages/dask/core.py:119: RuntimeWarning: All-NaN slice encountered
  return func(*(_execute_task(a, cache) for a in args))
/opt/conda/lib/python3.9/site-packages/dask/utils.py:35: RuntimeWarning: All-NaN slice encountered
  return func(*args, **kwargs)

and so on until :
Traceback (most recent call last):
  File "/argo/staging/script", line 47, in <module>
    tasks = dask.compute(*tasks)
  File "/opt/conda/lib/python3.9/site-packages/dask/base.py", line 570, in compute
    results = schedule(dsk, keys, **kwargs)
  File "/opt/conda/lib/python3.9/site-packages/dask/threaded.py", line 79, in get
    results = get_async(
  File "/opt/conda/lib/python3.9/site-packages/dask/local.py", line 507, in get_async
    raise_exception(exc, tb)
  File "/opt/conda/lib/python3.9/site-packages/dask/local.py", line 315, in reraise
    raise exc
  File "/opt/conda/lib/python3.9/site-packages/dask/local.py", line 220, in execute_task
    result = _execute_task(task, data)
  File "/opt/conda/lib/python3.9/site-packages/dask/core.py", line 119, in _execute_task
    return func(*(_execute_task(a, cache) for a in args))
  File "/argo/staging/script", line 34, in clear_memory_intensive_tests
    _test_dtr_range(d, v, data_type)
  File "/opt/dodola/dodola/core.py", line 758, in _test_dtr_range
    assert (
AssertionError: diurnal temperature range max is <xarray.DataArray 'dtr' ()>
dask.array<nanmax-aggregate, shape=(), dtype=float32, chunksize=(), chunktype=numpy.ndarray> for non-polar regions

Blocks progress on #265

@dgergel from this log and a quick check in the data :
(1) The maximum non-polar DTR is 74.6025, above the current threshold in the validation.
(2) I think we need to add a .values.item() after the calculation of unique values that might be printed.
(3) I am wondering where this bunch of NaN warning is coming from. It's not from the dtr_range checks. Looking at it.

I failed figuring out where (3) comes from. Dask is warning us about all nan slices, but I don't find any nan in the data, and the nan tests passed. I moved on since it's not blocking the run...
Regarding (2), I figured there are only 3 data points where the <70 test does not pass, and the three values are all within 70-80 dtr. The three data points are all at latitude -59.5, so that's above the edge of our southern polar tests.
I suggest we slightly shift up that edge ?
high = ds['dtr'].where((ds.lat > -60) & (ds.lat < 60)).where(cond=ds.dtr>70, drop=True).compute()
np.argwhere(high.values>70)
array([[ 4,  5, 12],
       [ 4,  5, 13],
       [ 4,  5, 14]])
high.lat.values[5]
-59.5

Testing the fix from this PR in this workflow : https://argo.cildc6.org/workflows/default/biascorrectdownscale-dtr-dev-79vgj?tab=workflow.

closed with switch to additive QDM/QPLAD approach


Fine-tuning pretrained model for different class

How can i fine tune the "model.pth.tar" model for a dataset that has different output class?
lets say the pretrained model("model.pth.tar") has 17 output class but i want to fine-tune it for a dataset which has 4 class.
Originally posted by @skfaysal in https://github.com/SilvioGiancola/SoccerNetv2-DevKit/issues/37#issuecomment-1186591705

Hi @skfaysal, I am not sure for CALF, but I can give you some insights for the pooling approaches. All you need to do is reset the self.fc layers in models.py and fine-tune with your new data.
However, note that:

most of the transfer learning happens through the pre-extracted features, that are not fine-tuned in this code. Those features (e.g. ResNet) are generic or fine-tuned on soccer image (see baidu embedding)
for MaxPool, AvgPool and the temporally-aware equivalents (MaxPool++, AvgPool++), there are no weights to fine-tune, as the features are directly pooled and classified, unless you want to fine-tune the feature dimension reduction here (PCA-like).
for NetVLAD, NetRVLAD, and the temporally-aware equivalents (NetVLAD++, NetRVLAD++), you can simply load a pre-trained model, re-initialize the self.fc layer with 4 classes only, and fine-tune the whole model.

I hope that helps!

Hi @SilvioGiancola ,Your quick response is much appreciated.
for NetVLAD++ pooling in self.fc layer:
self.fc = nn.Linear(input_size*self.vlad_k, self.num_classes+1)
I've changed the self. num_classes to my desired class.
Then i pass the trained model "model.pth.tar" as --load_weights parameter. I get below error:

We can see this error occurs because of the mismatch between fc layer of pretrained model which is (18,32768) and the custom model which is (3,32768). here 18 and 3 are output class.
I'm not shure how to change the fc layer of pretrained model and start training the custom model using the weights of the pretrained model.
Please help!

This issue has been resolved by doing the following steps:

Keep the model architecture as it is
Load the pretrained model weight and "state_dict" to match with the architecture.
      checkpoint = torch.load(os.path.join("models", args.load_weights, "model.pth.tar"))      model.load_state_dict(checkpoint['state_dict'])
Get the input features from fc layer and change the output node as per dataset
     num_ftrs = model.fc.in_features     model.fc = nn.Linear(num_ftrs, num_classes).cuda()

@SilvioGiancola  Thanks for your respose.


error in adding routename to the group route

Illuminate\Routing\Router::loadRoutes(): Failed opening required 'Array' (include_path='.:/usr/share/php')

Laravel Version: 5.6.*
PHP Version: 7.1
Database Driver & Version: mysql

Description:
if i add route name for group route i have got this error
Illuminate\Routing\Router::loadRoutes(): Failed opening required 'Array' (include_path='.:/usr/share/php')
Steps To Reproduce:
getting link via helper route('profile.show');
Route::namespace('profile')->name('profile.')->
group(['prefix' => 'profile', 'middleware' => ['auth'], ], function () {
Route::get('/', ['action'=>'ProfileController@index', ])->name('show');
...

The problem you have there is caused by your Route::get() call, as you provide an Array as second Parameter, though you should provide a String or a Closure.
Can you try it like this and tell me if that works for you?
Route::namespace('profile')
  ->name('profile.')
  ->group(['prefix' => 'profile', 'middleware' => ['auth']], function () {
    Route::get('/', 'ProfileController@index')->name('show');
});

Best wishes
Tom

Read over the docs on route groups, this isn't right. Also, use 'as' on a group, to prefix the route names.

Ok I'm sorry, the solution from above is wrong. You can not add an array with options for the group when specifying namespace or name beforehand.
This works in my local dev environment:
Route::namespace('profile')->name('profile.')->group(function() {
	Route::group(['prefix' => 'profile', 'middleware' => ['auth']], function() {
		Route::get('/', 'ProfileController@index')->name('show');
	});
});


Can not update when using Query

Is it by design, my wrong somewhere or a bug? If I use:
       this.addresses = _af.database.list('/addresses', {             query:{limitToFirst:5}         })
then doing:
this.addresses.update('-KKhJiJTPUqcjTjCYTvI', {code:'something'})
will throw:
ORIGINAL EXCEPTION: TypeError: Cannot read property 'code' of undefined
However when I use:
this.addresses = _af.database.list('/addresses')
Everything works as expected.
I can use a separate af.database.object to update the data and that works but shouldn't I be able to do an update with af.database.list directly here? Again without a Query I can, without a Query I'll get an error and so I have to use the separate database.object.
Thnx

By the way, the same goes for push and remove. I can work around for remove the same way (using database.object) but I don't see a way to work around for push other then setting up a separate database.list without a Query...

Follow #162 for updates on this bug. We have a fix in the works right now.