rsync over websocket sometimes fails

it looks like the real nc isn't much better but we shouldn't do this. I believe:
https://github.com/tych0/lxd/commit/033e9627fc8fc0d0fb922757e9999b12ebc9a928
will fix it, but I don't have a reproducer. We have someone who does, this is just a reminder to track it when he gets back.

Did we hear any other report of this? has any progress been made on this issue?

Yeah, I think we've heard other people say stuff like this. I can't reproduce it though.

I may be hitting this. I was trying to live migrate a container with criu, but it didn't get that far.
The observed behavior was it started rsyncing some stuff, then died.
 lxc move --verbose --debug nuc:es-1 es-1
# ...
 error: websocket: bad handshake

Debug output.
https://gist.github.com/ChrisLundquist/2850dba12397e78648e638f5cabfc073
One system is ext4 backed, the other is btrfs. Both have the same version:
clundquist@NUCALASCAGE:~$ lxd --version
2.0.8
clundquist@NUCALASCAGE:~$ uname -a
Linux NUCALASCAGE 4.4.0-59-generic #80-Ubuntu SMP Fri Jan 6 17:47:47 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

clundquist@Draupnir:~$ lxd --version
2.0.8
clundquist@Draupnir:~$ uname -a
Linux Draupnir 4.4.0-59-generic #80-Ubuntu SMP Fri Jan 6 17:47:47 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

Look at /var/log/lxd/lxd.log on both hosts, I bet you'll see a CRIU error in there. I'm not sure why this leads to that "bad handshake" instead of the error being surfaced back to the client though...
@tych0

/var/log/lxd/lxd.log is pretty clean, just mentions the migration failed, but log level is only at info.
Moving from this host.
t=2017-02-01T10:24:48-0800 lvl=info msg="Migrating container" ephemeral=false statedir=/tmp/lxd_checkpoint_429974479 actionscript=true stop=true name=es-1 created=2017-01-31T19:03:54-0800
t=2017-02-01T10:24:49-0800 lvl=info msg="Failed migrating container" actionscript=true stop=true name=es-1 created=2017-01-31T19:03:54-0800 ephemeral=false statedir=/tmp/lxd_checkpoint_429974479
t=2017-02-01T10:24:49-0800 lvl=info msg="Migrated container" name=es-1 created=2017-01-31T19:03:54-0800 ephemeral=false statedir=/tmp/lxd_checkpoint_429974479 actionscript=true stop=true

Moving to this host.
t=2017-02-01T10:24:49-0800 lvl=info msg="Deleting container" name=es-1 creation date=2017-02-01T10:24:05-0800 ephemeral=false
t=2017-02-01T10:24:49-0800 lvl=info msg="Deleted container" name=es-1 creation date=2017-02-01T10:24:05-0800 ephemeral=false
t=2017-02-01T10:24:49-0800 lvl=info msg="Deleting container" name=es-1 creation date=2017-02-01T10:24:49-0800 ephemeral=false
t=2017-02-01T10:24:49-0800 lvl=info msg="Deleted container" name=es-1 creation date=2017-02-01T10:24:49-0800 ephemeral=false
t=2017-02-01T10:24:50-0800 lvl=info msg="Deleting container" name=es-1 creation date=2017-02-01T10:24:49-0800 ephemeral=false
t=2017-02-01T10:24:50-0800 lvl=info msg="Deleted container" name=es-1 creation date=2017-02-01T10:24:49-0800 ephemeral=false

Not sure why it tries to delete the failed migration 3 times, but everything seems normal considering.

@stgraber @tych0 it looks like it is related to on of the LXD action_scripts / callbacks failing, the dump getting nuked, then the protocol falling over.
This is in the migration log after running lxc move packer nucalas:
(04.263599) Writing image inventory (version 1)
(04.263688) Running post-dump scripts
(04.263692) 	[/tmp/lxd_checkpoint_978884516/action.sh]
(05.664371) Error (criu/action-scripts.c:116): One of more action scripts failed
(05.664421) Post dump script passed with 1
(05.664426) Unlock network
(05.664431) Running network-unlock scripts
(05.664446) 	[/tmp/lxd_checkpoint_978884516/action.sh]
(05.674973) Unfreezing tasks into 1
(05.675000) 	Unseizing 1487 into 1
(05.675015) 	Unseizing 1717 into 1
(05.675025) 	Unseizing 1971 into 1
(05.675034) 	Unseizing 2108 into 1
(05.675085) 	Unseizing 2112 into 1
(05.675133) 	Unseizing 2113 into 1
(05.675232) 	Unseizing 2131 into 1
(05.675245) 	Unseizing 2183 into 1
(05.675254) 	Unseizing 2190 into 1
(05.675266) 	Unseizing 2196 into 1
(05.675275) 	Unseizing 2200 into 1
(05.675333) 	Unseizing 2211 into 1
(05.675345) 	Unseizing 2215 into 1
(05.675353) 	Unseizing 2217 into 1
(05.675384) 	Unseizing 2350 into 1
(05.675420) 	Unseizing 2369 into 1
(05.675431) 	Unseizing 2376 into 1
(05.675517) Writing stats
(05.675615) Dumping finished successfully

lxc snapshot --stateful packer works fine, but doesn't seem to have any callback scripts:
(03.986474) page-pipe: Killing page pipe
(03.986489) sk unix: Dumping external sockets
(03.986498) sk unix: 	Dumping extern: ino 0x2eb17 peer_ino 0 family    1 type    5 state  7 name (null)
(03.986519) sk unix: 	Dumped extern: id 0xcd ino 0x2eb17 peer 0 type 2 state 10 name 0 bytes
(03.986527) Writing image inventory (version 1)
(03.986809) Running post-dump scripts
(03.986816) Unlock network
(03.986820) Running network-unlock scripts
(03.993215) Unfreezing tasks into 1
(03.993236) 	Unseizing 1536 into 1
(03.993252) 	Unseizing 1656 into 1
(03.993263) 	Unseizing 1901 into 1
(03.993275) 	Unseizing 2048 into 1
(03.993286) 	Unseizing 2060 into 1
(03.993329) 	Unseizing 2068 into 1
(03.993407) 	Unseizing 2072 into 1
(03.993422) 	Unseizing 2074 into 1
(03.993433) 	Unseizing 2097 into 1
(03.993457) 	Unseizing 2099 into 1
(03.993506) 	Unseizing 2105 into 1
(03.993521) 	Unseizing 2126 into 1
(03.993532) 	Unseizing 2128 into 1
(03.993542) 	Unseizing 2277 into 1
(03.993552) 	Unseizing 2286 into 1
(03.993645) Writing stats
(03.993723) Dumping finished successfully

Checking what is in the failing action script is pretty hard because the folder gets cleaned up very quickly (it exists for ~2-4 seconds)
clundquist@nuc:~$ lxc --version
2.0.9
clundquist@nuc:~$ uname -a
Linux nuc 4.8.0-41-generic #44~16.04.1-Ubuntu SMP Fri Mar 3 17:11:16 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
clundquist@nuc:~$ dpkg -l | grep criu
ii  criu                                2.6-1ubuntu1~ubuntu16.04.2              amd64        checkpoint and restore in userspace

clundquist@NUCALASCAGE:~$ lxc --version
2.0.9
clundquist@NUCALASCAGE:~$ uname -a
Linux NUCALASCAGE 4.8.0-41-generic #44~16.04.1-Ubuntu SMP Fri Mar 3 17:11:16 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
clundquist@NUCALASCAGE:~$ dpkg -l | grep criu
ii  criu                                2.6-1ubuntu1~ubuntu16.04.2        amd64        checkpoint and restore in userspace

https://github.com/lxc/lxd/blob/e202b07dec34a1b93b65a2c8c8347345fc8a38b2/lxd/migrate.go#L233-L238
Appears to where the action scripts come from, I'm not sure what the parameters are when it fails, and it looks like it would take recompiling to inspect them.

@brauner do you think your recent fix solves this issue? if so, we should close it.

@stgraber, I added a few more commits. The last one implements logging for our netcat implementation. I think we can close this issue for now as I think the other failing rsyncs might be related to rsync being confused about the paths. In any case, if the fixes I pushed prove not to be sufficient we can reopen.

For future reference, we didn't have a producer in the traditional sense but one of our users provided me with access to two LXD instances and a container that would reliably fail. I was able to track the failure down to our netcat code. @tych0 was correct in assuming that our netcat or rather golang's io.Copy() did not handle EAGAIN. However, the EAGAIN was not present on the unix domain socket connection but rather on stdout. Also, we shouldn't simply return a short-write on EAGAIN but rather keep on trying. The trick is to not retry the io.Copy() call which indeed is not handling this case appropriately, rather we need to handle this in the Read() and Write() calls directly.
Since I see no harm coming from it, I've wrapped all readers and writers passed to io.Copy() with my new EAGAIN wrappers for them. Let's keep an eye open for new weirdness. :)


Limit edit

Deleted max select in pipeLimited proc because of it's incorrect logic.
proc pipeLimited was used only in httpclient.nim. It is used when length of body is known. That's why it has to read only certain number of digits and not more

I did similar changes simultaneously. I'll push them shortly when I finish debugging HTTP parser. Anyway, thank you for your work.

Can you update repo? Push your commits.
It's more comfortable to use your repo with correct code

Done.


fix(mobile): make font-size more legible

Closing to resolve components classname conflict issue https://github.com/map3xyz/components/actions/runs/4569313510/jobs/8065416316


427 ratify governance actions in the correct order

Description
Will close #427.
Checklist

[X] Commit sequence broadly makes sense and commits have useful messages
[X] Code is formatted according to CONTRIBUTING.md
[X] Self-reviewed the diff

@WhatisRT  Will a GovState contain GovActions with different GovActionIDs?  If so, should the ordering of elements in the GovState take into account GovActionIDs?

TODO: fix checks/invariants in Ledger.Ledger.Properties to account for inserting instead of appending new elements to GovState.

The GovActionIDs are irrelevant for the order here.

I removed another unused lemma (insertGovAction-∈) and made the new definition of insertGovAction more compact, so now the insertGovAction and govActionPriority functions fit inside Fig 15.


Only schedule AlterIsr thread when we have an ISR change

Rather than scheduling every 50ms to check for unsent updates, we should schedule the propagation thread only after we receive ISR updates

It seems I over-complicated this while trying to avoid clearing the inflight in the response handler when another request needs to go out. I think what you suggest does work. If the response handler just immediately sets the inflight flag to false it doesn't matter if another submit call races with checking the queue size since one of them will set the inflight flag true and the request will get submitted.
I'll go ahead and try this out

It seems I over-complicated this while trying to avoid clearing the inflight in the response handler when another request needs to go out. I think what you suggest does work. If the response handler just immediately sets the inflight flag to false it doesn't matter if another submit call races with checking the queue size since one of them will set the inflight flag true and the request will get submitted.
I'll go ahead and try this out


Thank you Lei! Really appreciate you iterating on this :-)

Finally one success run: https://github.com/openxla/iree/actions/runs/4508013248/jobs/7936341562?pr=12725
It takes around 1 hour 10min with a cold start. Much slower than the Mac mini we have (~30min). It's the slowest in all the CI jobs now. Trying to trigger a new run with cache hits to see how much time we'd need to wait.

Actually given this is a pull request it cannot write cache so cannot verify that.

Yeah running it on presubmit was only for testing. Let's put it back to postsubmit and try it out. If you're concerned about the detection on postsubmit being too slow, we could make this a duplicate job instead of a replacement and then see where we want to go based on how fast the cached runs are. Do you remember how fast the cold-start run on our mini took?

Yeah running it on presubmit was only for testing. Let's put it back to postsubmit and try it out. If you're concerned about the detection on postsubmit being too slow, we could make this a duplicate job instead of a replacement and then see where we want to go based on how fast the cached runs are.

SGTM to have duplicated job for now. Let's see how fast the GitHub hosted runners run with cache hits. Let me update the pull request to do that.

Do you remember how fast the cold-start run on our mini took?

It takes a bout 30min to start afresh.

@GMNGeoffrey I've marked this as ready for review and give approval to it. Please feel free to double check and land if you don't have more issues.


Not suggesting snippet?

I'm using nvim-cmp + vsnip
here is my configuration:
In cmp.lua
cmp.setup {
  completion = {
    completeopt = "menu,menuone,noinsert",
  },
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body)
    end,
  },
  mapping = {
    ['<C-p>'] = cmp.mapping.select_prev_item(),
    ['<C-n>'] = cmp.mapping.select_next_item(),
    ['<C-d>'] = cmp.mapping.scroll_docs(-4),
    ['<C-f>'] = cmp.mapping.scroll_docs(4),
    ['<C-Space>'] = cmp.mapping.complete(),
    ['<C-e>'] = cmp.mapping.close(),
    ['<CR>'] = cmp.mapping.confirm({
      behavior = cmp.ConfirmBehavior.Insert,
      select = true,
    }),
    ['<S-Tab>'] = cmp.mapping(cmp.mapping.select_prev_item(), { 'i', 's' }),
    ['<Tab>'] = function(fallback)
      if vim.fn.pumvisible() == 1 then
        vim.fn.feedkeys(vim.api.nvim_replace_termcodes('<C-n>', true, true, true), 'n')
      elseif vim.fn['vsnip#available']() == 1 then
        vim.fn.feedkeys(vim.api.nvim_replace_termcodes('<Plug>(vsnip-expand-or-jump)', true, true, true), '')
      else
        fallback()
      end
    end,
  },
  sources = {
    { name = 'nvim_lsp' },
    { name = 'nvim_lua' }
    { name = 'vsnip' },
  },
}

and update capabilities
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities = require('cmp_nvim_lsp').update_capabilities(capabilities)

I created javascript snippet like this in javascipt.json file:
{
  "useState": {
    "prefix": ["useS"],
    "body": [
      "const [$1, $2] = useState($3);$0"
    ]
  },
}

in test.js file, and This is what I got.

What is wrong here? or am I missing something?

You should set g:vsnip_filetypes.
See https://github.com/hrsh7th/vim-vsnip#2-setting

You should set g:vsnip_filetypes.
See https://github.com/hrsh7th/vim-vsnip#2-setting

Oh i missing, i also created this vsnip.lua file:
local g = vim.g
local map = require('utils').map
local cmd = vim.cmd

g.vsnip_filetypes = {
  javascriptreact = { 'javascript' },
  typescriptreact = { 'typescript' },
}

map('n', '<leader>V', ':VsnipOpen<CR>')

cmd [[
  imap <expr> <Tab>   vsnip#jumpable(1)   ? '<Plug>(vsnip-jump-next)'      : '<Tab>'
  smap <expr> <Tab>   vsnip#jumpable(1)   ? '<Plug>(vsnip-jump-next)'      : '<Tab>'
  imap <expr> <S-Tab> vsnip#jumpable(-1)  ? '<Plug>(vsnip-jump-prev)'      : '<S-Tab>'
  smap <expr> <S-Tab> vsnip#jumpable(-1)  ? '<Plug>(vsnip-jump-prev)'      : '<S-Tab>'
]]

Could you check filetype? :echomsg &filetype

And Could the snippet  be expanded with manually?

Could you check filetype? :echomsg &filetype

the result is javascript

And Could the snippet be expanded with manually?

What do you mean? I can't get it

You mean i have to put all of these lines of code in my setting?: https://github.com/hrsh7th/vim-vsnip#2-setting

Can you open the tour snippet file by :VsnipOpen?

Yep, I can open it. Even when I type useS then tab, it shows the snippet that I wrote. but the cmp not suggest

Did you install htsh7th/cmp-vsnip source?

Hi everyone, i solved my problems. Because I did not install the source for vsnip.
So please don't forgot to install this one https://github.com/hrsh7th/cmp-vsnip
Thanks for the help.


Remove services for showing RA locations that are no longer needed

Their functionality has been replaced by the institution configuration options API.
This can be merged after #114 has been approved and merged.

@DRvanR

:+1:


Broken against newer prompt_toolkit

I have just installed pyvim in a fresh venv (under python 3.6). And it failed to launch:
$ pyvim
Traceback (most recent call last):
  File "/tmp/env/bin/pyvim", line 7, in <module>
    from pyvim.entry_points.run_pyvim import run
  File "/tmp/env/lib/python3.6/site-packages/pyvim/entry_points/run_pyvim.py", line 17, in <module>
    from pyvim.editor import Editor
  File "/tmp/env/lib/python3.6/site-packages/pyvim/editor.py", line 28, in <module>
    from .key_bindings import create_key_bindings
  File "/tmp/env/lib/python3.6/site-packages/pyvim/key_bindings.py", line 4, in <module>
    from prompt_toolkit.key_binding.bindings.utils import create_handle_decorator
ModuleNotFoundError: No module named 'prompt_toolkit.key_binding.bindings.utils'

Looks like latest prompt_toolkit doesn't contain module prompt_toolkit.key_binding.bindings.utils, as well as that create_handle_decorator function (tried to grep it).
My pip freeze is like this:
docopt==0.6.2
prompt-toolkit==1.0.15
pyflakes==1.6.0
Pygments==2.2.0
pyvim==0.0.20
six==1.10.0
wcwidth==0.1.7

Here is when that module was removed (17 Dec 2016): https://github.com/jonathanslenders/python-prompt-toolkit/commit/17047496500480fc2c7653e701da0fdd5e08bfd4

Hi @Kentoseth and @MarSoft,
I just pushed a new version of Pyvim (0.0.21) that contains the fix. The fix was already in the repo, but not released.
FYI: at this point I'm mostly working on prompt_toolkit v2.0. This contains many improvements. Once this is finished, you can expect an improved version of Pyvim, using prompt_toolkit 2.0 as well.


[KARAF-6450] Upgrade to jackson 2.9.10

Some features still need "old" jackson version. I will update the dependencies as well.

retest this please


filter all closed whatsaround needs from connections-overview -> issu…

fixes #1588

Does not change anything - this screenshot was taken after logout + login


Bug: 300 / M0 means "use last value" not a literal zero

Channel zero of this song - the Moog sample should continue w/ the tone portamento (3xx / Mx in volume column) instead of restarting, but the effect is unimplemented.
Broam_-_FSharp_Jump.xm.zip
Song's CC-By-SA, as noted in the sample text

Thanks for the report!
Just pushed a fix... Now you can actually hear the filter sweep in the song. Will update my site later tonight.

FWIW, 300 was implemented correctly, but Mx wasn't implemented at all.

Fixed on https://www.a1k0n.net/code/jsxm/
Thanks for the report.

However the Lxx effect isn't implemented; I believe your song uses it heavily. See #5


Feature Request: Adding observable aggregator function

Hi
Ionic cloud recently has added Database, which is very similar to firebase,
They also added a new function which is called Aggreegate which takes observables and return the result and keep returning it based on observables,
https://docs.ionic.io/services/database/#aggregate-model
Something like below
const userName = "ionitron";
this.db.aggregate({
    name: userName,
    user: this.db.collection('users').find(userName),
    activity: {
        posts: this.db.collection('posts').findAll({user: userName}),
        topComments: this.db.collection('comments').findAll({user: userName}).order('rating', 'descending').limit(10)
    }
}).watch().subscribe(subscribeFunction);
I was thinking it would be great if we could have same functionality in angularfire so we could combine couple of observable lists to an object

Hi @RezaRahmati! The Firebase Realtime Database does not have aggregation capabilities out of the box. Most people use transactions to an aggregate node or rely on Cloud Functions for Firebase for computations.


size quota & page title

Hi
Would it be possible to add a quota settings, something that would block upload if 'to directory' + 'attempted upload' > quota ?
Another proposition would be adding an option for a title. Actually mine is just /, kinda unclear if I wanna tell mom do drop / get stuff with caddy :-)
Anyway, thanks for the great job !

Regarding the title: I guess you refer to a directory listing of some sorts, which is rendered by plugin browse. You can configure templates with browse.
I won't implement any quota enforcement. That would require implementing a proper user management including annotated users. Such an annotation would be the intended allowance. Unfortunately, that is out of scope of this plugin and should go into a dedicated one. The same requirement applies to writing a file as a user, or postponed change of ownership of a file, I am afraid.


add the note press animation for the opponent

add the animation for pressing a note when the opponent sings instead of the notes just disappearing.
example: FNF Multiplayer
https://user-images.githubusercontent.com/61301511/120357424-6e56c380-c2d3-11eb-9251-39a49b8ffad2.mp4

your kinda late because there is alredy

your kinda late because there is alredy

a development release or the 1.4.2 release?

github kade engine master

dev release

1.4.2 is really old

yes at this point its just like kade engine 1.0


Plugin GitHub checks

Description
Plugin hosting request: https://issues.jenkins-ci.org/browse/HOSTING-994
Repository link: https://github.com/jenkinsci/github-checks-plugin

Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

cc @timja @uhafner

Have you logged into Artifactory?   User name not known to Artifactory: xiongkezhi

Have you logged into Artifactory?  User name not known to Artifactory: xiongkezhi

Yes, I have logged

@slide build is green


Error: connect ECONNREFUSED <IP_ADDRESS>:9222

I tried to debug chrome, but got an error. Here is a repro step:

open chrome: chrome.exe --remote-debugging-port=9222 --no-sandbox
debug: chrome-remote-interface new 'https://github.com'
error: Error: connect ECONNREFUSED <IP_ADDRESS>:9222

Component
Version

Operating system
Windows 10

Node.js
12.18.1

Chrome/Chromium/...
88.0.4324.150 (Official Build) (64-bit)

chrome-remote-interface
0.29.0

Is Chrome running in a container? YES / NO

Never mind. I should have used '--headless" option, too.


Add license

Would it be possible to add a license to the project? It's required in some business contexts. I would suggest MIT or WTFPL but any of the github default licenses should be fine.
Thanks

Done.


Collect new inspection button still appears even if feature service doesn't participate in 1:M relationship

This came up while testing with a custom web map which contains only a single feature layer - 59ad551de0f042f088ee99ea26321b96.
Running the app against that web map, you'll notice that identifying a tree point still includes the + button to collect a new 1:Many related record, despite the fact that the layer does not participate in a relationship. We should conditionally remove the button if no such relationship exists.
The good news is that we have pretty good error messaging for this case:

@mikewilburn changes have been merged to master. Can you confirm the app works as expected?

Verified on master that the + button no longer shows up if the identified feature does not participate in a 1-to-Many relationship.


Update for SE-0025 ('fileprivate' and 'private')

SE-0025 makes private restrict access to a lexical scope; the Swift 2 behavior is now named fileprivate. At the time of this pull request, the compiler treats both private and fileprivate as having fileprivate semantics, but that will soon change.

Thanks! These bindings aren't currently built in CI, so I'm just merging this.

Actually last I checked the bindings were not building at all because of the changes in compiler...

Yeah, I wasn't able to test this properly.


Fix trace validation error follow up fix

Issue #, if available:
Follow up on this https://github.com/aws-observability/aws-otel-java-instrumentation/pull/617.
The previous PR applied two separate filter expressions while searching for xray traces, which returned any traces matching either conditions. Need to change the logic so that traces matching both conditions are returned.
Description of changes:
Combined the two filter expressions into one.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Attention: 102 lines in your changes are missing coverage. Please review.

Comparison is base (09e6487) 85.71% compared to head (ddcd125) 35.97%.
Report is 163 commits behind head on main.

Files
Patch %
Lines

...ent/providers/AwsAppSignalsCustomizerProvider.java
24.00%
35 Missing and 3 partials :warning:

...gent/providers/AwsSpanMetricsProcessorBuilder.java
0.00%
20 Missing :warning:

...ders/AttributePropagatingSpanProcessorBuilder.java
0.00%
16 Missing :warning:

...viders/AwsMetricAttributesSpanExporterBuilder.java
0.00%
11 Missing :warning:

...try/javaagent/providers/AwsSpanProcessingUtil.java
90.16%
1 Missing and 5 partials :warning:

...vaagent/providers/AwsMetricAttributeGenerator.java
96.89%
2 Missing and 3 partials :warning:

...y/javaagent/providers/AwsSpanMetricsProcessor.java
91.48%
0 Missing and 4 partials :warning:

...t/providers/AttributePropagatingSpanProcessor.java
94.59%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##               main     #626       +/-   ##
=============================================
- Coverage     85.71%   35.97%   -49.75%
- Complexity       19      205      +186
=============================================
  Files             3       39       +36
  Lines            49     1301     +1252
  Branches          5      141      +136
=============================================
+ Hits             42      468      +426
- Misses            3      811      +808
- Partials          4       22       +18

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


docs: Introduce an official Code of Conduct

This is merely my proposal, but I strongly encourage some form
of a posted code-of-conduct.

Thanks @MySolace. We've updated the CONTRIBUTING.md to point to our "living" Code of Conduct.

Looks great! A central living document seems like a fantastic way to handle it. :)
CasKer Joel
https://mysolace.github.io/CrisisTrends
On Dec 10, 2017, 8:41 AM -0800, David Lemayian<EMAIL_ADDRESS>wrote:

Thanks @MySolace. We've updated the CONTRIBUTING.md to point to our "living" Code of Conduct.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Map hasEntry combined with collection matcher seems to fail incorrrectly

Here is some example code showing what I think is a bug
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.empty;
import static org.hamcrest.Matchers.hasEntry;
import static org.hamcrest.Matchers.not;

import java.util.HashMap;
import java.util.List;
import java.util.Map;

class Scratch {

	public static void main(String[] args) {
		Map<String, Object> map = new HashMap<>();
		List<String> value = List.of("value1", "value2");
		map.put("key", value);
		assertThat(map, hasEntry("key", not(empty())));

	}
}

The assert fails with the output
Exception in thread "main" java.lang.AssertionError:
Expected: map containing ["key"-><not an empty collection>]
     but: map was [<key=[value1, value2]>]
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)

But key is mapped to a "not empty collection"? So I think it should pass.

Note on Java 11 with Hamcrest 2.1

Can you test with hamcrest v2.2

Your saying Java 11, are you aware if it works on another Java version?

Its exactly the same issue on Java 11 with Hamcrest v2.2 and on Java 8 with Hamcrest v2.2

Can you try the following and see if it works for you hasEntry(equalTo("key"), not(empty())).
The method hasEntry(K, V) assumes your giving literal Key and Value parameters. So it takes your not(empty()) and assumes it's a value, and instantly wraps it as equalTo(not(empty())).
The other method hasEntry(Matcher<K>, Matcher<V>) works for me as it takes matchers for both Key and Value. We don't have a method that takes the combination for hasEntry(K, Matcher<V>) currently which we would need for your original test.

Yes that works. Ok so what would be needed for this is 2 extra signatures for hasEntry

hasEntry(K key, Matcher<? super V> valueMatcher)
hasEntry(Matcher<? super K> keyMatcher, V value)
If you think that would be an acceptable solution I can look at making a PR? I think it would be a nice feature.

Yes that works. Ok so what would be needed for this is 2 extra signatures for hasEntry

hasEntry(K key, Matcher<? super V> valueMatcher)
hasEntry(Matcher<? super K> keyMatcher, V value)

If you think that would be an acceptable solution I can look at making a PR? I think it would be a nice feature.

It would make sense to overload org.hamcrest.collection.IsMapContaining.hasEntry to have those signatures you have listed.

Please be careful with this. Things get hard to predict with this level of overloading. It's easy to end up trying to do equals() on a matcher object. Perhaps a better option would be both matchers?


Fix broken link in issue templates

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/890/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/890/


Test issue for training

Test with no real purpose

[x] seen it
@StMSchl This is a test only ...

Test is hereby finished.

closed


[SPARK-30321][ML] Log weightSum in Algo that has weights support

What changes were proposed in this pull request?
add instr.logSumOfWeights in the Algo that has weightCol support
Why are the changes needed?
Many algorithms support weightCol now. I think weightsum is useful info to add to the log.
Does this PR introduce any user-facing change?
no
How was this patch tested?
manually tested

Test build #4972 has started for PR 26972 at commit e86a2d3.

Test build #115629 has started for PR 26972 at commit e86a2d3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20429/
Test PASSed.

Test build #4972 has finished for PR 26972 at commit e86a2d3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #115629 has finished for PR 26972 at commit e86a2d3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115629/
Test PASSed.

cc @zhengruifeng

Test build #115652 has started for PR 26972 at commit 93cf0be.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20451/
Test PASSed.

Test build #115652 has finished for PR 26972 at commit 93cf0be.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115652/
Test FAILed.

I notice that other statistics for tree models are logged in RandomForest.run:
    instr match {
      case Some(instrumentation) =>
        instrumentation.logNumFeatures(metadata.numFeatures)
        instrumentation.logNumClasses(metadata.numClasses)
        instrumentation.logNumExamples(metadata.numExamples)
      case None =>
        logInfo("numFeatures: " + metadata.numFeatures)
        logInfo("numClasses: " + metadata.numClasses)
        logInfo("numExamples: " + metadata.numExamples)
    }

Can we just log weightSum here?

We can log weightSum in RandomForest.run, but for GBTClassifier and GBTRegresor, I think this is logged for each iteration, so it's logged multiple times.

Yes, current GBT impls will log these metrics for each iteration. But I guess we can just log weightSum there, if there is not an easy way to log all of them only once (at iteration 0)

@huaxingao Can we pass the instr of GBTClassifier/GBTRegressor into the internal impl, and use it to log the metrics in the first tree building?

Test build #115918 has started for PR 26972 at commit 6f9a9c8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20709/
Test PASSed.

Test build #115918 has finished for PR 26972 at commit 6f9a9c8.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115918/
Test FAILed.

Test build #115919 has started for PR 26972 at commit 92754ba.

Test build #115919 has finished for PR 26972 at commit 92754ba.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20710/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115919/
Test FAILed.

Test build #115920 has started for PR 26972 at commit 9d3821e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20711/
Test PASSed.

Test build #115920 has finished for PR 26972 at commit 9d3821e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115920/
Test PASSed.

Test build #115965 has started for PR 26972 at commit 7120e19.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20758/
Test PASSed.

Test build #115965 has finished for PR 26972 at commit 7120e19.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115965/
Test PASSed.

retest this please

Test build #115982 has started for PR 26972 at commit 7120e19.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20774/
Test PASSed.

Test build #115982 has finished for PR 26972 at commit 7120e19.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115982/
Test PASSed.

Merged to master, thanks all!
Happy new year!

Thank you for your help! Wish you all a very Happy New Year!!


AMBARI-23575 - Download mpacks during blueprint luster provisioning (benyoka)

Mpacks which are referenced by either the blueprint or the cluster template but not yet installed (based on name and version) are downloaded automatically during cluster provisioning. Such mpacks must have their uri defined.
How was this patch tested?

tested manually
wrote new unit tests
unit test run result:
Tests run: 4771, Failures: 31, Errors: 223, Skipped: 91

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2181/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2185/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2187/
Test FAILed.
Test FAILured.


Implement Match APIs.

Implement 2 methods, which have different arguments as follows.
namespace LibAoE2net
{
    public static class AoE2net
    {
        public static async Task<Match> GetMatchAsync(int id)
        {
        }
    }
}

The new APIs do not support This feature.


Changed the deep learning framework from tensorflow to pytorch

#18

@shravya312 , interesting, but can we be benefited from it ?

can we add more features to our model ?

A dynamic computation graph is a key feature of PyTorch that allows the framework to create a map of calculations while the program is running.
Key Points:
Built While Running: The graph is formed as you run your code, not before. This means it can change every time you execute it.
Flexible and Adaptive: You can easily change how your model behaves based on the data you provide. For example, if the input data is different, the graph adjusts accordingly.
Easy to Debug: Since the graph is created as the code runs, you can use regular Python tools to check for errors right where they happen, making it easier to fix problems. these are the features of pytorch which is beneficial

can we add more features to our model ?

Educational Resources
Learning Modules: Offer tutorials or modules that educate users on space mining, including best practices and case studies.
Resource Library: Create a library of articles, research papers, and case studies related to space mining and exploration.

can we add more features to our model ?

Educational Resources Learning Modules: Offer tutorials or modules that educate users on space mining, including best practices and case studies. Resource Library: Create a library of articles, research papers, and case studies related to space mining and exploration.

this can be done using deep-learning which can use the mining data and results from this model to educate people

I have done this in different file @Devanik21

Can u review @Devanik21 if any changes needed I will do


what is the difference between other one ( logseq-wide-eyed)

what is the difference between other one ( logseq-wide-eyed)

The other one tries to keep track of everything which gets loaded to the page via DataScript queries.  It's cumbersome because doing this adds overhead.  And I can't possibly guess every block which will get loaded to or updated on a page.  It makes it a little more brittle.  I have to poll the page to.
Take Journals.  That page lazily loads content.  I have no idea how far down a person will scroll and how many days of entries they will load so I query 90 days out.  But if they scroll 91 days or further I'm not going to be hiding/manipulating any other blocks.  It's the reason I requested certain features (event hooks to let me know when blocks are loaded/updated on a page) because then I could just tap into what the main app is already doing.
The other huge difference is Wide Eyed is matching regular expressions against content.  It can match anything at all (e.g. full text search) in the content, where as Style Carousel can only match against the kinds of metadata the core chooses to make visible in the DOM.  It is much more restrictive it what can be matched because CSS selectors can't do what a regular expression can.  So it is a compromise.  Yes, both do the same thing initially, by default, but both are configured in very different ways.  And neither was meant exclusively as means of toggling to-do visibility.  They just showcase what they can be made to do.  Both could be made to do dozens of other things.
Style Carousel is constantly matching the actual DOM and Wide Eyed is constantly presuming what's being loaded to the page and matching the actual content.  Style Carousel works better with trivial use cases like to-dos.  Fortunately, there is enough being added to the DOM to make this use case work fine.  That's because all the statuses (DONE, CANCELED, WAITING, etc.) are written to selectable attributes on elements.  This make Style Carousel light weight by comparison.  Until certain event hooks are added Wided Eyed is a cumbersome beast.
Both are configurable, but the average user is going to have more success with Style Carousel.  They're going to simply look at the DOM and design custom stylesheets that are conditionally toggled on/off.  Wide Eyed can match any content whatsoever, even if it's not exposed in the DOM.  Additionally, because Style Carousel is so light weight, it affords as many buttons as you like, whereas Wided Eyed presents a single button.
So to summarize, it depends on which capabilities you need: can you get away with just inspecting the DOM or do you want to inspect and match actual database content and metadata not presented in the DOM?
If I had my event hooks Wide Eyed could probably be made to do the same thing equally well and also be light weight.  But I do not have my event hooks.  So Wide Eyed is a kludge.  It works reasonably well given its constraints.  But as one user asked, "Why are some of my to-dos still showing?"  And that's because I have to presume what might get loaded on a page (in Wide Eyed) and I have no foolproof way of doing that.
For example, I considered reading in the embedded queries and running them to get and match the content but at that point I felt like I was taking something which was already doing too much work and giving it even more work.  What's really needed to simplify are the event hooks.  If a page was simply announcing (via hooks) any block it rendered (e.g. the actual object with id, metadata, and content) then I wouldn't have to simply guess what might be loaded on a page.
If I might put it another way, and sorry to have drawn this out... consider how you might have designed a configurable plugin for conditionally styling content.  My intent was never to make it just about to-dos.  I wanted the user to be able to match on tags, phrases, schedules, deadlines, etc.  Wide Eyed can be made to do that.  Style Carousel cannot, unless all the metadata bubbles up to the DOM in a way conducive to selectors.  But that is an implementation specific dependency.  If the nature of the rendering changes, selectors will break.  Ideally, what you want are event hooks that tell you what's being added to (or updated on) the page and what area of the page (main content, side bar, etc.).  Simply announcing that with hooks would be powerful and allow a plugin like Wide Eyed to match on nearly every property.

thanks for a very very detailed answer. it seems style carousel is better for me. in my experience wide eye could not perform well and was slow relative to carousel

Agreed.  I went back to the drawing board.  If the added/updated hooks I requested are ever added Wide Eyed would become the superior option as it could do much, much more without the performance bottleneck.  Right now, both serve a purpose because Wide Eyed can still do more but, as you said, it is not snappy.

@hodanli — Unfortunately, even the approach used by Style Carousel has problems when installed from the marketplace.  It fails to detect matches on the page, the feature which underlines the eye when completed/canceled to-dos are found on a page.  Everything else works fine.
Plus, if you install it locally (e.g. unpack the plugin) locally it works completely, all features.

Per the latest release, Wide Eyed is now obsoleted by Style Carousel.  Wide Eyed works far too hard to track what is on the current page, unfortunately, because the Logseq plugin api doesn't help the situation.
Style Carousel ignores the current page and, when queries are optionally used, queries across the entire database and builds style rules that can be applied to any page to achieve the desired effect.
Style Carousel's one deficit is the need to install it locally for the reasons described in the README.


Sprockets choose wrong manifest with asset_path option used

When using the asset_path option, Kamal copies all files from the assets folder to the brand-new deployed container. If the app inside the container uses Sprockets gem with default settings, then in the assets folder it has two manifest files. The app chooses random one, unless it is specified explicitly.
From the end-user perspective it looks like no new assets were deployed.
The fix is simple:
# config/environments/production.rb

config.assets.manifest = Rails.root.join('config', 'manifest.json')

I am not sure if the issue is worth mentioning in config/deploy.yml comments. At least I've posted it here.

I encountered the same problem. Thanks for the solution.


Get rid of two "+" buttons from the wallet tab

@ChintanRathod
We need a quick change.
Currently, we have two "plus" icons in the Wallet tab. This is very confusing, but we can fix this quite easy and quickly.

So we need to change what follows:

Take out the very top + button and replace it by QR icon https://zpl.io/VYvX6Ge
This QR code icon will open "My Wallet Address". We already have this screen in Settings tab. So now, we will have two ways to access the same screen. First is QR Code icon in the Wallet tab, second is "My wallet Address" item in the Settings tab.
Currently, to add a Custom Token you can click very top "+" button. Now, you can open "Add /Hide Tokens" and there you will find + icon.
Change text from "Show / Hide Tokens" to "Add / Hide Tokens"

@ChintanRathod I second this, this fix should be quickly done and released!

@colourful-land Generated PR, could you please check.


Account model

This PR contains the whole Account model changes.
This PR at the moment is marked as draft and supposed to be merged to dev_evm_dest branch as is later.
All the bug fixes and improvements expected to be done in the different PRs on top of this branches.
The goal is to make a comprehensive review of the whole codebase of the Address model.

PR closed, all PR comments were addressed.


glyph name list

I am looking for a full list of glyph names for Kannada font.
I am confused between AGL and AGLFN.
say for kannada letter U+0C95, in AGL it is named as uni0C95 and AGLFN it is named as knKa
What is the advantage or need for AGLFN naming?
Is it primarily because of name field length limit?

The glyph names uni0C95 and * knKa* are in neither of the lists. You must be talking about the GlyphOrderAndAliasDB file in the soure-kannada-sans repository. The first column of that file lists the glyphs' final names, i.e. the glyph names that the OTF file will have. The second column lists the glyphs' friendly names, i.e. the glyph names used in the UFO source files.

The advantage of using “friendly” glyph names is the level of abstraction – in feature code, it is easier to read
sub x by y;
than
sub uni0078 by uni 0079;
The GlyphOrderAndAliasDB file just provides a handy means of translating between your custom easy-to-read glyph names, to the final, required names.
For a list of code points, you can consult the Kannada Unicode range:
http://unicode.org/charts/PDF/U0C80.pdf
Hope this helps!

I understand that, is there is advantage in the applications? Besides easing the font development while coding, is there any particular reason to name the glyph in particular way?

It is purely for font development. Most developers prefer names like knKa (Kannada letter Ka) to uni0C95, because the level of abstraction is not as high.
In the final OTF, all friendly glyph names are gone.

I am reading and also testing the font with abstract glyph names vs AGLFN.
With abstract glyph names "knKa", I am not able to retrieve the unicode text thru copy/paste from PDF.
But with AGLFN, copy/paste retrieve the unicode text successfully.
Is this the main reason, it is recommended to use AGLFN vs abstract glyph names?
Even with AGLFN, some text such as "reph" "ಧರ್ಮ". When copied and pasted to text pad, it is retrieved as "ಧಮರ್"
Please assist.

@bengulurumanjunatha: Have you read the AGL Specification?

@kenlunde Although, my understanding could be wrong. I have read the specification.
Please check this font https://github.com/erinmclaughlin/Hubballi
This is built with AGLFN naming scheme.
Here is an example:
Original text with Hubbali font: ಬಗ್ಗೆ
Retrieved by copying from PDF created with above text: ಬಗೆ್ಗ

Original: ದರ್ಪಣ
retrieved: ದಪರ್ಣ

Issue is very well described in the below blog.
http://aravindavk.in/blog/glyph-names/
Please review.

@bengulurumanjunatha When you wrote "copying from PDF," please be sure to indicate which PDF client you are using, because the results can differ in terms of extracting "content" from a PDF.
Looking at the Hubballi font, the glyph names include lots of sequences, whose content should extract without issues, but the glyph names include several odd suffixes, and it seems that not all final names conform to the AGL Specification.
Also, when you provide example, in addition to the actual text, it would be useful to know the final glyph names that correspond to the character sequence.

@kenlunde Thank you for responding.
Client: Acrobat Reader DC 2017.012.20093 on Mac OS Sierra 10.12.6, Build 16G29
Example 1:
Original text: ಬಗ್ಗೆ
Unicode sequence: u+0C97 u+0CCD u+0C97 u+0CC6
Font glyph sequence: uni0C970CC6, uni0CCD0C97
copied text from PDF: ಬಗೆ್ಗ
Unicode sequence: u+0C97 u+0CC6 u+0CCD u+0C97
Example 2:
Original text: ದರ್ಪಣ
Unicode sequence: u+0ca6 u+0cb0 u+0ccd u+0caa u+0ca3
Font glyph sequence: uni0CA6 uni0CAA  uni0CB00CCD uni0CA3
copied text from PDF: ದಪರ್ಣ
Unicode sequence: u+0ca6 u+0caa u+0cb0 u+0ccd u+0ca3
I think the problem is acrobat see only the glyph names and not the feature associated with it.
I don't know if something is wrong in the glyph name that is not letting Acrobat reader know the feature (say uni0CB00CCD.reph) or is it the problem in the AGLFN standard to recognize this reordering sequence via shaping engines as mentioned in the blog in the previous post.
I am sorry, if I have still not provided all the information you have asked me. If I have not provided all necessary, please let me know.

@bengulurumanjunatha Can you also supply a PDF that includes these sequences? Another factor in this is the PDF producer. If you didn't use Adobe InDesign, it would be useful to repeat this exercise using that app, because it produces a PDF with separate presentation and content layers, which does a much better job at preserving the original text sequence for the purpose of searching and copy&paste.

PDF created from Pages with "Export To PDF" feature.
Original Text
ನಡುಗನ್ನಡ ಸಾಹಿತ್ಯದಲ್ಲಿ ಅನೇಕ ಹೊಸ ಸಾಹಿತ್ಯ ಪ್ರಕಾರಗಳು ಬೆಳಕಿಗೆ ಬಂದವು. ಇವುಗಳಲ್ಲಿ ಮುಖ್ಯವಾದವು ರಗಳೆ, ಸಾಂಗತ್ಯ ಮತ್ತು ದೇಸಿ. ಈ ಕಾಲದ ಸಾಹಿತ್ಯ ಜೈನ, ಹಿಂದೂ ಹಾಗೂ ಜಾತ್ಯತೀತ ಬೋಧನೆಗಳ ಮೇಲೆ ಆಧಾರಿತವಾಗಿದೆ. ಈ ಘಟ್ಟದ ಪ್ರಮುಖ ಲೇಖಕರಲ್ಲಿ ಇಬ್ಬರೆಂದರೆ ಹರಿಹರ ಮತ್ತು ರಾಘವಾಂಕ. ಇಬ್ಬರೂ ತಮ್ಮದೇ ಶೈಲಿಯಲ್ಲಿ ಕನ್ನಡ ಸಾಹಿತ್ಯದ ದಾರಿಯನ್ನು ಬೆಳಗಿದವರು.
ಹರಿಹರ ರಗಳೆ ಸಾಹಿತ್ಯವನ್ನು ಬಳಕೆಗೆ ತಂದನು, ತನ್ನ ಶೈವ ಮತ್ತು ವೀರಶೈವ ಕೃತಿಗಳ ಮೂಲಕ. ರಾಘವಾಂಕ ತನ್ನ ಆರು ಕೃತಿಗಳ ಮೂಲಕ ಷಟ್ಪದಿ ಛಂದಸ್ಸನ್ನು ಜನಪ್ರಿಯಗೊಳಿಸಿದನು. ಅವನ ಮುಖ್ಯ ಕೃತಿ ಹರಿಶ್ಚಂದ್ರ ಕಾವ್ಯ, ಪೌರಾಣಿಕ ಪಾತ್ರವಾದ ಹರಿಶ್ಚಂದ್ರನ ಜೀವನವನ್ನು ಕುರಿತದ್ದು. ಈ ಕೃತಿ ಸಹ ತನ್ನ ತೀವ್ರವಾದ ಮಾನವತಾವಾದಕ್ಕೆ ಪ್ರಸಿದ್ಧವಾಗಿದೆ.
ಇದೇ ಕಾಲದ ಇನ್ನೊಬ್ಬ ಪ್ರಸಿದ್ಧ ಜೈನ ಕವಿ ಜನ್ನ. ತನ್ನ ಕೃತಿಗಳಾದ ಯಶೋಧರ ಚರಿತೆ ಮತ್ತು ಅನಂಥನಾಥ ಪುರಾಣಗಳ ಮೂಲಕ ಜೈನ ಸಂಪ್ರದಾಯದ ಬಗ್ಗೆ ಬರೆದನು. ಇದೇ ಕಾಲದ ಕನ್ನಡ ವ್ಯಾಕರಣದ ಬಗೆಗಿನ ಮುಖ್ಯ ಕೃತಿ ಕೇಶಿರಾಜನ ಶಬ್ದಮಣಿ ದರ್ಪಣ.
‘ಜ಼ಾಕಿರ್' 'ಜಾ಼ಕಿರ್'
AGLFN.pdf

@bengulurumanjunatha I recommend that you download and install the fully-functional thirty-day trial version of Adobe InDesign, and use it to repeat this test, being sure to specify the "Adobe World-Ready Composer" (Single-line or Paragraph) for the paragraphs. You use File→Export… to export a PDF file.

BTW, when I open your PDF using Adobe Acrobat Pro DC (2017 Release), select all of the text, copy, then paste into the TextEdit app on macOS (Version 10.12.6), I get the following:

@kenlunde you are displaying this text in "Kannada Sangam MN" which is default font in mac for Kannada. The last two words in single quote is issue of font.
But I see the line above the last word ದರ್ಪಣ. It is has changed the order of reph (arkavattu). This is still an ISSUE.
Please paste the text here. We can compare the text.
I will post the PDF from indesign, shortly.

Here is the text that I copied from the PDF you supplied:
ನಡುಗನ್ನಡ ಸಾಹಿತ್ಯದಲಿ್ಲ ಅನೇಕ ಹೊಸ ಸಾಹಿತ್ಯ ಪ್ರಕಾರಗಳು ಬೆಳಕಿಗೆ ಬಂದವು. ಇವುಗಳಲಿ್ಲ
ಮುಖ್ಯವಾದವು ರಗಳೆ, ಸಾಂಗತ್ಯ ಮತು್ತ ದೇಸಿ. ಈ ಕಾಲದ ಸಾಹಿತ್ಯ ಜೈನ, ಹಿಂದೂ ಹಾಗೂ
ಜಾತ್ಯತೀತ ಬೋಧನೆಗಳ ಮೇಲೆ ಆಧಾರಿತವಾಗಿದೆ. ಈ ಘಟ್ಟದ ಪ್ರಮುಖ ಲೇಖಕರಲಿ್ಲ ಇಬ್ಬರೆಂದರೆ
ಹರಿಹರ ಮತು್ತ ರಾಘವಾಂಕ. ಇಬ್ಬರೂ ತಮ್ಮದೇ ಶೈಲಿಯಲಿ್ಲ ಕನ್ನಡ ಸಾಹಿತ್ಯದ ದಾರಿಯನು್ನ
ಬೆಳಗಿದವರು.
ಹರಿಹರ ರಗಳೆ ಸಾಹಿತ್ಯವನು್ನ ಬಳಕೆಗೆ ತಂದನು, ತನ್ನ ಶೈವ ಮತು್ತ ವೀರಶೈವ ಕೃತಿಗಳ ಮೂಲಕ.
ರಾಘವಾಂಕ ತನ್ನ ಆರು ಕೃತಿಗಳ ಮೂಲಕ ಷಟ್ಪದಿ ಛಂದಸ್ಸನು್ನ ಜನಪಿ್ರಯಗೊಳಿಸಿದನು. ಅವನ
ಮುಖ್ಯ ಕೃತಿ ಹರಿಶ್ಚಂದ್ರ ಕಾವ್ಯ, ಪೌರಾಣಿಕ ಪಾತ್ರವಾದ ಹರಿಶ್ಚಂದ್ರನ ಜೀವನವನು್ನ ಕುರಿತದು್ದ. ಈ
ಕೃತಿ ಸಹ ತನ್ನ ತೀವ್ರವಾದ ಮಾನವತಾವಾದಕೆ್ಕ ಪ್ರಸಿದ್ಧವಾಗಿದೆ.
ಇದೇ ಕಾಲದ ಇನೊ್ನಬ್ಬ ಪ್ರಸಿದ್ಧ ಜೈನ ಕವಿ ಜನ್ನ. ತನ್ನ ಕೃತಿಗಳಾದ ಯಶೋಧರ ಚರಿತೆ ಮತು್ತ
ಅನಂಥನಾಥ ಪುರಾಣಗಳ ಮೂಲಕ ಜೈನ ಸಂಪ್ರದಾಯದ ಬಗೆ್ಗ ಬರೆದನು. ಇದೇ ಕಾಲದ ಕನ್ನಡ
ವಾ್ಯಕರಣದ ಬಗೆಗಿನ ಮುಖ್ಯ ಕೃತಿ ಕೇಶಿರಾಜನ ಶಬ್ದಮಣಿ ದಪರ್ಣ.
‘ಜ಼ಾಕಿರ್' 'ಜಾ಼ಕಿರ್'

@kenlunde
I. In the above text you have copied from PDF. all the blwf, reph features have failed to copy.
II. I have attached the PDF created from indesign.
Hubballi experiment page1 .pdf
Original text:
‘ಜ಼ಾಕಿರ್' 'ಜಾ಼ಕಿರ್’
ಬಗ್ಗೆ ದರ್ಪಣ
ದಟ್ಸ್, ಎಕ್ಸ್ , ಮಾರ್ಚ್, ಟೆಕ್ಸ್ಟ್, ಬುಕ್ಸ್, ಸಾಫ್ಟ್, ಜಸ್ಟ್, ಪೋಸ್ಟ್‌ಪೇಯ್ಡ್
ಎಕ್ಸ್‌ಪ್ರೆಸ್
ರ್ಪೊ
ಆ್ಯಕ್ಷಿಸ್
Version: Adobe InDesign CS 6
OS: Windows 10
Issue 1: Most of the text is successfully copied from PDF created from Adobe Indesign. Expect "zero width non-joiner" has copied to copy over.
Words "ಇನ್‌ಡಿಸೈನ್" "ಪೋಸ್ಟ್‌ಪೇಯ್ಡ್" "ಎಕ್ಸ್‌ಪ್ರೆಸ್"
Hubballi font has u+200c has labeled the glyph as "uni200c".
Text is copied has space.
Issue 2: Not related to AGLFN.
Adobe World-Ready Composer has issue rendering the words  ‘ಜ಼ಾಕಿರ್' 'ಜಾ಼ಕಿರ್’ 'ಆ್ಯಕ್ಷಿಸ್'
If you know how to report this issue to right team, it will be great help.
Kannada script uses "u+200c" and "u+200d" frequently. I will check some words with zero width joiner, u+200d.
Also, what is different between PDF generate in adobe indesign and else where.
Thanks for looking into this.

@bengulurumanjunatha I don't read the script that is being discussed, so you'll need to be a lot more explicit about things.
First, does the PDF that was exported from InDesign exhibit improved behavior? What you wrote suggests so, but please confirm.
About Issue 2, I will need excruciating, such as screenshots that show the expected behavior versus the actual behavior, and any suggestions as to how to address it. I can then pass along something meaningful (and hopefully actionable) to the InDesign development team.

@kenlunde
"Review of PDF created from Indesign"
Yes, situation has improved to near perfection.
Issue:
Version: Adobe InDesign CS 6
OS: Windows 10
Font: Hubballi
Words
"ಇನ್‌ಡಿಸೈನ್" glyph names-> ( uni0c87 uni0ca80ccd uni200c uni0ca10cbf uni0cb80cc8 uni0ca80ccd ) without spaces
"ಪೋಸ್ಟ್‌ಪೇಯ್ಡ್" glyph names-> ( uni0caa0ccb uni0cb80ccd uni0ccd0c9f uni200c uni0caa0cc7 uni0caf0ccd uni0ccd0ca1 ) without spaces
"ಎಕ್ಸ್‌ಪ್ರೆಸ್" glyph names-> ( uni0c8e uni0c950ccd uni0ccd0cb8 uni200c uni0caa0cc6 uni0ccd0cb0 uni0cb80ccd ) without spaces
All the above glyphs were copied successfully.
Expect "zero width non-joiner", U+200C, glyph name "uni200c". This is non printing character "zero width non-joiner" was copied either as space or nothing. (https://en.wikipedia.org/wiki/Zero-width_non-joiner).
Result:
"ಇನ್ಡಿಸೈನ್"
"ಪೋಸ್ಟ್ಪೇಯ್ಡ್"
"ಎಕ್ಸ್ಪ್ರೆಸ್"

Resolution needed:
u+200c (zwnj) and u+200d (zwj)needs to be written in the PDF.
Question:
Should the glyph name of u+200c be uni200c or u200c and glyph name of u+200d be uni200d or u200d?

@bengulurumanjunatha: Thank you for confirming that the PDF that was exported from InDesign was nearly perfect, and for the additional details and suggestions that I will pass along to the InDesign team.
About your question, glyphs that correspond to characters in the BMP should use the "uni" prefix in their names, meaning that you should use uni200C and uni200D as the glyph names for U+200C and U+200D, respectively.

@kenlunde Thank you.
Follow up question.

What does InDesign writes additionally in PDF, which other PDF writes are not writing?
I suppose Adobe Acrobat Pro/DC also writes this information or is it specific to text editor not to PDF writer.
If this is a something PDF writer dependent. We would like to this to be available in all PDF writers. We want to start reporting this issues in correct channel.

Also, I will write a detailed report about the issue in "Adobe World Ready Composer" with Kannada language.
Thanks,
Manjunatha

@bengulurumanjunatha: Answers to your questions:

InDesign writes to the PDF file a "content" layer that is a more accurate representation of the text that corresponds to what is displayed in the PDF, which is the "presentation" layer.

Adobe Acrobat Pro/DC is merely copying the text from the "content" layer. If a PDF file does not have a "content" layer, the glyph information in the "presentation" layer is used to derive content, and that is what is copied.

The reason why InDesign-exported PDFs have better "content" is because its PDF Library has direct access to the original text that is used in the source InDesign document. For virtually all other PDF producers, what ends up getting converted to a PDF file is a PostScript file that is fed into a separate PDF-producing client, such as PDFWriter.

@kenlunde
Adobe World-Ready Composer issue is reported here. Please forward it to the right team.
https://indesign.uservoice.com/forums/601180-adobe-indesign-bugs/suggestions/31005514-bug-in-adobe-world-ready-composer-for-kannada-lang

I reported both sets of issues to the InDesign team.

@kenlunde Thank you very much for your support.


Allow project developer to edit repository description

UT works are tracked in #3691.

Coverage decreased (-23.6%) to 33.88% when pulling 3ebff9ae951b4e5186e7e44b141591432bd83330 on ywk253100:171220_desc into 068d6a35dfaed51a752d87df1858e0427ddbe160 on vmware:master.

Coverage increased (+0.02%) to 57.539% when pulling 42c9c439f9f5102593988f1bfd7d5bf4b1d86649 on ywk253100:171220_desc into 068d6a35dfaed51a752d87df1858e0427ddbe160 on vmware:master.


config.properties

Hello,
I really like what you did with AgrarMonitor.
when running the Agrarmonitor.exe on PC, where can I find the "config.properties" ?

my savegames are located in E:\Documents\My Games\FarmingSimulator2022
I edited config.properties as follows:
savegameDir=E:/Documents/My\ Games/FarmingSimulator2022
but it does nor show the data.

Try
savegameDir=E:\\Documents\\My\ Games\\FarmingSimulator2022
then it should work

OK, I'll try that.
but: you can use '/' (as in Linux) in paths since Windows XP, so why should I use \ which has to be escaped (\) ?
You can use \ or / but / is better because it is OS-independent. Internally, Java will convert it to the file seperator of the OS

Yes, normally you could use a path with '/' instead of '\'. Both work fine in Java / with AgrarMonitor, but sometimes '/' seems to be a bit buggy.
Did you check if the path is correct? On my computer the "documents"-folder is inside my user-home. So the path should look more like "E:\Users\NAME\Documents..."
If the path is correct, you can verify that AgrarMonitor is trying to load the correct savegame. By default, the application tries to load the first savegame slot. If there is no savegame in slot 1, no data is shown. To switch to the correct savegame you should edit the last part of the URL in your browser: "?savegame=X" where X is the number of the slot you want to load. (In the next version there will be a button to switch savegames, so no URL-editing will be necessary anymore)
If it still doesn't work, could you send the error messages that appear on the command line?
There should be some sort of "java.io.FileNotFoundException"-Exceptions.

commandline? I run it by double-clicking the AgrarMonitor.jar file in the Windows Explorer.

You could open a command line and start it with "java -jar AgrarMonitor.jar". Then you can see the (error) log

Thanks: now it works with: savegameDir=E:\Documents\My\ Games\FarmingSimulator2022
and I made a batch file with :
if not "%minimized%"=="" goto :minimized set minimized=true start /min cmd /C "%~dpnx0" goto :EOF :minimized java -jar AgrarMonitor.jar /MIN
Furthermore can you make it so that in config.properties I can pass language and standard savegame to load ?


crawl: error: running 'scrapy crawl' with more than one spider is no longer supported

crawl: error: running 'scrapy crawl' with more than one spider is no longer supported

Hi @amirktc
I didn't understand your issue.
Are you trying to run with more than one spider?

Hi @amirktc
I didn't understand your issue.
Are you trying to run with more than one spider?

As I've got no response for this issue I'll close it.


qvina installed

qvina is installed, i can see it in my conda environment, but i dont have commandline access to qvina? how can that be? i installed qvina via conda-forge

Although conda distribution of QuickVina is not done by me, here is a general answer. It's unusual for software installed through conda-forge to lack command-line access. Here are some potential explanations and solutions for why you might not be able to access qvina in your terminal:
1. Activation Issue:

Check Environment Activation: Make sure you have activated the conda environment where qvina is installed. Run conda list or conda env list to see your available environments. If you haven't activated the environment, use conda activate environment_name (replace environment_name with your actual environment name).

2. Path Issue:

Check Path: The executable might not be added to your system's path. You can verify this by running which qvina (or where qvina on Windows).

Add to Path (Optional): If the output is empty, you can add the path to your environment manually. This process varies depending on your operating system. Refer to conda documentation or search online for specific instructions.

3. Installation Issue:

Reinstall qvina: In rare cases, the installation might be incomplete. Try reinstalling qvina using conda install -c conda-forge qvina.

4. Permissions Issue:

Check Permissions: Less likely, but an issue with file permissions might prevent execution. You can try searching online for specific instructions to adjust permissions if needed.

Additional Tip:

Consult conda documentation: Finally, refer to the conda documentation for troubleshooting installation and environment management issues: https://conda.io/activation

Please let us know whether any of these helped you.
(This answer was prepared with the help of Google gemini)

qvina is installed, i can see it in my conda environment, but i dont have commandline access to qvina? how can that be? i installed qvina via conda-forge

The program name is qvina2, so I think you can try qvina2 --help to see whether it is correctly installed.


HUE Ledstrip LCL001 not supported yet?

Hi Bram,
I thought all HUE light are supported (for auto discovery), but my HUE led strip LCL001 isn't

Thanks,
Rien

Did you check the supported models list? Only lights in this list are supported.
The library of lights is build by volunteers which measure the lights using a tool I developed. For each light to support this takes a lot of effort, and someone need to have physical access to this exact Hue light model. The process is described here: https://github.com/bramstroker/homeassistant-powercalc/wiki/Contributing-new-lights
Maybe someone has this light and will measure it somewhere in the future. You can also request measurement for light models here.
Or you can have a look into contributing to the project yourself.


Ensure remotedialer kubelet connections use kubelet bind address

Proposed Changes
Ensure that connections to the kubelet through the egress-selector (either local or via remotedialer) use the correct kubelet bind address.
We have long overridden the kubelet address to the loopback address when dialing via the remotedialer tunnel, and ensured that the kubelet always binds to an ipv4 or ipv6 wildcard address. We broke this in https://github.com/k3s-io/k3s/pull/10019 when we wired the --bind-address flag up to the kubelet --address flag by way of the ListenAddress field.
This is blocked on:

[x] https://github.com/rancher/remotedialer/pull/80

Types of Changes
bugfix
Verification

Set --bind-address on a server or agent node
Note that kubectl logs works properly to retrieve logs from pods on that node

Testing
Linked Issues

https://github.com/k3s-io/k3s/issues/10444

User-Facing Change

Fixed an issue where setting the `--bind-address` flag to a non-loopback or wildcard address would prevent `kubectl logs` from working properly.

Further Comments

Has this bugfix been merged ?
Using --bind-address PRIVVATE_IP_ADDR still see errors in kube-system namespace


update lastname

Docs Build status updates of commit 8f32a73:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/map.yml
:warning:Warning
View
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/toc.yml
:warning:Warning
View
Details

advocates/media/profiles/sandra-ahlgrimm.png
:white_check_mark:Succeeded
View

advocates/sandra-ahlgrimm.yml
:white_check_mark:Succeeded
View

advocates/map.yml

[Warning] Unable to find uid "advocates.sandra-kriemann".
Line 0: [Warning] Invalid file link:(~/media/profiles/sandra-kriemann.png).

advocates/index.html.yml

[Warning] Unable to find uid "advocates.sandra-kriemann".
Line 0: [Warning] Invalid file link:(~/tweets).
Line 0: [Warning] Invalid file link:(~/map).
Line 0: [Warning] Invalid file link:(~/media/profiles/sandra-kriemann.png).

advocates/toc.yml

[Warning] Unable to find file with uid "advocates.sandra-kriemann" referenced by TOC file "toc.yml"

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit e92b029:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/toc.yml
:warning:Warning
View
Details

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/sandra-ahlgrimm.png
:white_check_mark:Succeeded
View

advocates/sandra-ahlgrimm.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 0: [Warning] Invalid file link:(~/tweets).
Line 0: [Warning] Invalid file link:(~/map).

advocates/toc.yml

[Warning] Unable to find file with uid "advocates.sandra-kriemann" referenced by TOC file "toc.yml"

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 5fe6c63:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/sandra-ahlgrimm.png
:white_check_mark:Succeeded
View

advocates/sandra-ahlgrimm.yml
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 0: [Warning] Invalid file link:(~/tweets).
Line 0: [Warning] Invalid file link:(~/map).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow user to fix depsolve errors on Edit page

Before this update, if a blueprint had a package with a version that resulted in a depsolve failure, then an error message would display in the Edit page, and the only workaround for fixing the error is by editing the TOML file through the cli.
This update still displays the error message, but also includes the list of selected components so that the user can find the component that is mentioned in the error and resolve the issue by either removing the component or selecting a different version.
~Still to be included is the last statement in issue #718 (i.e. preventing the user from creating an image when an error exists).~ Going to defer this to a future PR.
Some ways to test this:

Add the package "soletta" to a blueprint. In this case, dependencies required by soletta are missing and will return an error. But the list of packages are still displayed. In this case, you will also see the Undo action display in the alert.
Edit the toml file of a blueprint to have bad version numbers for packages (or navigate to the Edit page for the atlas blueprint, which has old version numbers defined).

This is ready for review. After chatting with @larskarlitski, I'm going to create a separate PR to add the ability to block image creation when a depsolve failure occurs.
Please let me know if any updates are needed.
Thanks!

This looks good to me. I have two things which I think are fine to add in separate PRs.

I think we should display a similar error on the packages tab of the blueprint details page. Maybe it could direct the user to navigate to the edit blueprint page to fix the issue.

When I drill down to view the details of a component some of the information is missing and there is console error: Error in fetchInputDeps TypeError: "response[responseIndex] is undefined"

Rebased to include Jacob's eslint fixes and fix conflicts.

@larskarlitski, feel free to merge it. All green now.


feature: Enable app Podfile

Sometimes a project just needs to add a Podfile without any JS/TS, but a full plugin must be created:

https://github.com/tjvantoll/nativescript-IQKeyboardManager (Until recently was only a podfile)
https://github.com/Essent/nativescript-disable-bitcode (just a postinstall to strip all bitcode from a project).

Additionally, this is where a project should be able to set its platform version level, as well. (and plugin Podfiles should not).
In many ways this is similar to many of the settings and package management actions that can be taken in app.gradle (dependencies not requiring JS, package overrides, and min platform).
Would likely just come from App_Resources like similar files.

+1 From slack conversation with a client

This issue was moved to NativeScript/nativescript-cli#3807


Improvement Suggestion-Add Comment Block!

Hello, Ling!
I have a suggestion now!
What about adding a comment block at the bottom of the blog with the help of GitHub issues, inspired by the project gitment and this issue.

Hi Ming,
I don't think I'll add comments for now because I wanted the theme to be quite minimal and personally I don't think a comments section is essential to a blog. Furthermore I think there are many commenting options out there (disqus / gitment / isso / talkyard etc) so I think it is better for people to add it if they want to. Thanks anyway!


Add TruthfulQA as QA examples [Eval Branch]

Data is in csv here: https://github.com/sylinrl/TruthfulQA/blob/main/TruthfulQA.csv

TruthfulQA tests model truthfulness through questions spanning law, medicine, finance, and politics, and other areas
these were adversarially generated using GPT-3 davinci v1 (175B) as the target model.


Static allocation bones copy for SkeletonModifier

Co-authored-by: @SlugFiller
The original bone pose is copied/restored before and after the Skeleton modification process. This PR make it use static allocation ~and memcpy~.

Bone does not seem to be trivially_copyable, so I changed the approach to static allocating the LocalVector of the structure for backup.

Thanks!


add debug testing

i.e., set up a local server, have bots play the game, extract a full data set

This issue was moved to Dallinger/Dallinger#123


C++ constraints shall use double instead of int64_t when type is number

For example, if we  have this in the schema:
"test": {
  "title": "Test",
  "type": "number",
  "minimum": -1.7e+308,
  "maximum": 1.7e+308
}

It generates this C++ code:
test_constraint(-1.7e+308, 1.7e+308, std::nullopt, std::nullopt, std::nullopt)

But ClassMemberConstraints is using int64_t for min_value/max_value:
class ClassMemberConstraints {
    private:
    std::optional<int64_t> min_value;
    std::optional<int64_t> max_value;

So, we should have 2 different classes for constraints, one for int64_t, and another for double.

@dvdsgl I think this issue can be closed due to the merged PR. Thanks for your work!

Closing as resolved in #2257


WFLY-17330 Remove unused nu.xom, org.dom4j, org.jaxen modules and remove Hibernate ORM 6 dependency on dom4j

Signed-off-by: Scott Marlow<EMAIL_ADDRESS>https://issues.redhat.com/browse/WFLY-17330

Pending test results passing, will make one more change to also remove modules nu.xom + org.jaxen

Looks like a legit failure in org.jboss.as.test.layers.LayersTestCase:

Running org.jboss.as.test.layers.LayersTestCase
[ERROR] Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 223.362 s <<< FAILURE! - in org.jboss.as.test.layers.LayersTestCase
[ERROR] test(org.jboss.as.test.layers.LayersTestCase)  Time elapsed: 216.515 s  <<< ERROR!
java.lang.Exception: ERROR, some modules have not been provisioned: nu.xom,org.dom4j,org.jaxen,
  at org.jboss.as.test.layers.LayersTest.test(LayersTest.java:133)
  at org.jboss.as.test.layers.LayersTestCase.test(LayersTestCase.java:317)
  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  at java.base/java.lang.reflect.Method.invoke(Method.java:566)
  at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)

@scottmarlow The solution to the test failure is to remove the nu.xom, org.dom4j and org.jaxen modules. That test is written to detect cases where we provide unused modules, so it has correctly detected that removing the org.dom4j dep from org.hibernate has resulted in unused modules. :)

I suggest updating WFLY-17330 to also note it will result in the removal of the 3 modules.

I suggest updating WFLY-17330 to also note it will result in the removal of the 3 modules.

Thanks, will remove the other unused modules also update the issue.  :)

Pushed too soon as testsuite still uses xom

@scottmarlow Please restore the dependencyManagement entries that were in boms/common-ee but put them in the root pom. For dom4j and jaxen make them 'test' scope. (The xom one is used by testsuite/shared, which is not a test module itself; it's a utility lib, so its compile deps can't be test scope.) Then we don't specify the versions in the various child poms as they are controlled by dependency management.
See the comments before various other external libs that are declared in the root pom for the kind of comment that should precede these 3 as well. Basically an explanation for why these are in the root pom to make it easier to do something better in the future.
When we do https://issues.redhat.com/browse/WFLY-17331 we'll move these kinds of things out of the root pom.

@scottmarlow Please restore the dependencyManagement entries that were in boms/common-ee but put them in the root pom. For dom4j and jaxen make them 'test' scope. (The xom one is used by testsuite/shared, which is not a test module itself; it's a utility lib, so its compile deps can't be test scope.) Then we don't specify the versions in the various child poms as they are controlled by dependency management.
See the comments before various other external libs that are declared in the root pom for the kind of comment that should precede these 3 as well. Basically an explanation for why these are in the root pom to make it easier to do something better in the future.
When we do https://issues.redhat.com/browse/WFLY-17331 we'll move these kinds of things out of the root pom.

Thanks for the detailed comment!  I made the requested change.

Thanks @scottmarlow!


Spritesheet / symbol

@adamwathan: I want to inline the same (quite complex) SVG multiple times on a page so
I can style it differently using page CSS.
Despite gzip I think using a symbol for referencing to the same inlined SVG would be the best approach.
Can I do this with blade-svg? I want to prevent duplicate SVG markup.

Is the @sprite directive from the next version what you need? You can pass different classes/style attributes to each directive: https://github.com/adamwathan/blade-svg/tree/next-version#sprite-sheets

Well, with HTTP2 (and Let's Encrypt) there isn't really a need for spriting anymore.

Okay. Gonna close this one already then. Thanks for replying 👍


Add --prereleases support

Before this patch, Copier didn't ignore prereleases when detecting latest template tag.
This is mostly a bug because there's no way to safely upgrade a template to the latest non-prerelease tag automatically.
This a behavioral change that probably didn't hit anybody out there, but enough to make a new big release.

Closed by mistake, continues in #254


cmd/ctrd-protobuild: create proper command for building protos

After trying to explain the complexities of developing with protobuf, I
have now created a command that correctly calculates the import paths
for each package and runs the protobuf command.
The Makefile has been updated accordingly, expect we now no longer use
go generate. A new target protos has been defined. We alias the two,
for the lazy. We leave go generate in place for cases where we will
actually use go generate.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #521   +/-   ##
=======================================
  Coverage   62.12%   62.12%
=======================================
  Files           3        3
  Lines         404      404
=======================================
  Hits          251      251
  Misses         93       93
  Partials       60       60

Continue to review full report at Codecov.

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

LGTM


get() returns undefined for value zero

If the value of an objects property is zero, get(object,proxy) will return undefined. I guess the problem is the usage of truthy in get:
o[key] || defaultValue

if o[key] is zero, it will be truthified to false, so the defaultValue will be returned, which is by default undefined.
Simple example to reproduce:
export class Person {
  age:number
}

    const person = new Person()
    person.age = 0

    const personProxy = createProxy<Person>()
    const age = get(person, personProxy.age)
    console.log(age) // -> undefined

Instead of using truth some real check for null/undefined would appropriate, like e.g.
export function isDefined<T>(value: T) {
  return value !== null && value !== undefined
}

Ah, this just bit me.
Looks like the issue is fixed in code, but new release has not been released on npm, so the issue still exists in practice.


[wallet] Race Condition in Virtual Funding in the Wallet

It looks like if player A submits their transaction first and gets far enough ahead of player b in the funding protocol. It will send player B a message before player B is ready to receive the message, which causes the message to get discarded.
Both player A and player B will get stuck waiting for a post fund setup (since B discarded the message and A is waiting on B)

@lalexgap was this fixed with https://github.com/statechannels/monorepo/pull/753?

@lalexgap was this fixed with #753?

Yup, closing


Multiple monitors

When using multiple monitors the dialog doen't open on my primary monitor.

try setting "desktop" in the config file to be the number of your desktop (you may have to play around with this)
desktop=1

I updated the previous comment to correct what I had written, try screen instead of desktop

Sadly this does not seem to work. I have 2 monitors, tried setting it to 0, 1 and 2, but it still shows up on my second monitor.

are you using randr or xinerama? you can check what lighthouse thinks you are using by compiling with make debug and looking at the printed statements when you run it.  I can't seem to reproduce the issue

Fwiw, I use xinerama and had to set screen=3 to get lighthouse to show up on my primary monitor.

Is it possible for lighthouse to show on the screen that currently has focus?


Search_after support.

It says that the test case it_set_a_geoShape_query failed,but I change nothing about it.
Could someone help me to fix this please?

@airycanon I fix the failed test in my pr.
it doesn't a big problem, I think it could be ignored.


ReflectionException in App\Modules\NetworkData\Listeners\AtcSessionRecordedSuccessNotification@handle

Error in VATSIM-UK/Core
ReflectionException in App\Modules\NetworkData\Listeners\AtcSessionRecordedSuccessNotification@handle
Class App\Modules\NetworkData\Listeners\AtcSessionRecordedSuccessNotification does not exist
View on Bugsnag
Stacktrace
bootstrap/cache/compiled.php:1375 - ReflectionClass::__construct

View full stacktrace

An error linked to this issue has been reopened in Bugsnag
ReflectionException in App\Modules\NetworkData\Listeners\AtcSessionRecordedSuccessNotification@handle


Deployment Script Resource with Identity Property Errors with Request Too Long

Bicep version
0.4.1272
Describe the bug
When using a deployment script with an identity component (user assigned identity), I receive an error that is difficult to decipher what is going wrong.
"error": {
    "code": "BadRequest",
    "message": "<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01//EN\"\"http://www.w3.org/TR/html4/strict.dtd\">\r\n<TITLE>Bad Request</TITLE>\r\n<META HTTP-EQUIV=\"Content-Type\" Content=\"text/html; charset=us-ascii\">\r\n
Bad Request - Request Too Long
\r\n

HTTP Error 400. The size of the request headers is too long.
\r\n\r\n"
  }

When I remove the identity component, the deployment script deploys successfully.
To Reproduce
Steps to reproduce the behavior:

Create a simple bicep file that has a user assigned identity, role assignment and deployment script

    var userAssignedIdentityName = 'testuserassignedidentity'
    var roleAssignmentName = guid(userAssignedIdentity.id, 'contributor')
    var contributorRoleDefinitionId = resourceId('Microsoft.Authorization/roleDefinitions', 'b24988ac-6180-42a0-ab88-20f7382dd24c')
    var deploymentScriptName = 'CopyConfigScript'

    resource userAssignedIdentity 'Microsoft.ManagedIdentity/userAssignedIdentities@2018-11-30' = {
      name: userAssignedIdentityName
      location: resourceGroup().location
    }

    resource roleAssignment 'Microsoft.Authorization/roleAssignments@2020-04-01-preview' = {
      name: roleAssignmentName
      properties: {
        roleDefinitionId: contributorRoleDefinitionId
        principalId: userAssignedIdentity.properties.principalId
        principalType: 'ServicePrincipal'
      }
    }

    resource deploymentScript 'Microsoft.Resources/deploymentScripts@2020-10-01' = {
      name: deploymentScriptName
      location: resourceGroup().location
      kind: 'AzurePowerShell'
      identity: {
        type: 'UserAssigned'
        userAssignedIdentities: {
          '${userAssignedIdentity.id}': {}
        }
      }
      properties: {
        azPowerShellVersion: '3.0'
        scriptContent: '''
          Write-Output "This is a test"
        '''
        retentionInterval: 'P1D'
      }
      dependsOn: [
        roleAssignment
      ]
    }

Deploy the bicep file to a resource group: az deployment group create --resource-group testgroup --name deploymentscript01 --template-file main.bicep
Receive error message specified above
Remove identity property from deploymentScript resource and deploy bicep file again. Deployment succeeds.

Additional context

This bicep file was pulled from an example template here: https://docs.microsoft.com/en-us/learn/modules/extend-resource-manager-template-deployment-scripts/3-exercise-add-deployment-script?pivots=bicepcli

Removed the need for blob storage and deployment script just logs a "this is a test" for simplicity purposes (running the original example also results in this issue)

Having Azure CLI as the kind also results in this issue

We would recommend opening a support ticket for this one. Do you mind doing that?
The error looks like it is occurring prior to reaching the ARM deployments service.

Sure, I will open a support ticket for this now.

@msha1026 , did you already resolved this issue?
@alex-frankel: I am also receiving the error message “Bad Request - Request Too Long” when deploying a ARM template with an inline deploymentscript. Although I don’t use a managed identity. It’s happening a in one of my Azure subscriptions, but I use another subscription (and another account to login) it’s working fine. I do notice: the user account which fails is part of many groups in Azure AD. Could that be related?

I'm getting the same error.
Using Azure CLI 2.34.1

In my case it’s cause by the fact that the account I am using had many “Assigned roles”. When a reduces the roles from 61 to 12 I didn’t had the issue anymore.

@JohanKlijn this has not been resolved yet. Still working through a support ticket. The managed identity I am creating will only have one assigned roles.

@msha1026 : I am not talking about the assigned roles on de managed identity, I am talking about the identity (account) which you use to deploy the ARM template.
Probably many assigned roles is not the only reason the deployment could fail with the message “Bad Request - Request Too Long”, because the message just means the size of HTTP request made is to big which could be caused by many factors. In my case the access token used for the request was quite big, because my account had many roles assigned.

I'm getting the same error when Azure CLI v2.34.1 locally through WSL and Pwsh.

Just to follow up on this.
It will fail regardless of the azure CLI version, and is likely due to factors connected to my AAD login such as the number of groups i'm a member of.
When i run the same deployment script resource by a ServicePrincipal - it works great regardless of AZ version.

FYI - I heard back from the ticket I opened and they have released a fix late last week. Taken from the ticket:
"They are making a change that updates the deploymentScripts service to use the same http request size limit as ARM. Deployment of this change will start next week. Changes in Azure are usually rolled out by region so please allow some time for this change to be fully pushed to Azure and take effect"
I have not had time to test yet but will test out later this week. If it is resolved, I will close this issue.

I have tested this again and the bug seems to be fixed. Closing out the issue :)


How can I make CAPSLOCK a modifier key instead of a toggle?

I'm trying to make it so some keys are different when I hold the Capslock key but go back to normal when I let go of the Capslock key. Right now, it seems to only act as a toggle that I have to press again to turn it off. Is it possible to make Capslock not a toggle but only change keys when I'm holding it down and turn off when I let go?

Please follow this procedure:

Install Seil: https://pqrs.org/osx/karabiner/seil.html.en
Change caps lock key code to 80 in Seil. (Do not forget "No Action" configuration.)
Enable new setting by this private.xml: https://github.com/tekezo/Files/blob/master/Karabiner/private.sync_capslock_keydownup/private.xml

How to use private.xml:
https://pqrs.org/osx/karabiner/document.html#privatexml

Thanks!


sap.m.Table event selectionChange is not called when method removeSelections is executed

OpenUI5 version:
1.60.1
Browser/version (+device/version):
Chrome
Any other tested browsers/devices(OK/FAIL):
Firefox: FAIL
URL (minimal example if possible):
https://jsbin.com/muzopuv/edit?html,output
Steps to reproduce the problem:

Run sample with JS
Select with check first row of table - alert appears (table selectionChange event)
Press button "clear selection" (executes table method removeSelections) - alert do not appear

What is the expected result?
alert appears in both cases: selectionChange is called when removeSelections is executed
What happens instead?
alert do not appears when removeSelections is executed: selectionChange is not called

Hello @endrx ,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,

this is also similar with (most of the) DOM APIs; setting the value property of the HTMLInputElement  does not fire the input event. The caller of the API should know what is changed.