L1_L2 regularization implemented wrong?

Looking at the source code, the L1_L2 (also known as elastic net) regularization differs dramatically from description of this paper.
class L1L2(Regularizer):
    """Regularizer for L1 and L2 regularization.

    def __call__(self, x):
        regularization = 0.
        if self.l1:
            regularization += K.sum(self.l1 * K.abs(x))
        if self.l2:
            regularization += K.sum(self.l2 * K.square(x))
        return regularization

Is this done on purpose or do you understand something different on the term L1/L2 regularization?

This appears to me to be an accurate implementation of the regularization terms from the equation 3 in the paper.


feat: add write read split

What this PR does:
add read write split. https://dubbo-kylin.yuque.com/staff-rvuh2g/qau55u/nshtei
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

pls add ut

addressed

PTAL @AlexStocks


refactor: rework search

fix searching when selected any bucket
display connections and buckets separately as PaneItems

:tada: this issue has been resolved in version 1.0.0 :tada:
release is available on GitHub


Update PD Milestone levels for Teamwork

Background
On #31  we did a first iteration using the Communication competency as an example of how we could allocate milestones by identifying what is good and what isn't. We need to do the same work for all other competencies.
This ticket is dependent on #41
Acceptance criteria

Add the milestones for all Teamwork coursework (Emerging - Level 1, Developing - Level 2, Established)
Evaluate if changes are needed to the coursework and raise other tickets for it.

Owner: @LaraHuzjan @kfklein15
Stakeholder: @SallyMcGrath
Approver: @kfklein15

Teamwork Levels


Cookies are not forwarded

With the following configuration the forwarding of cookies does not work
zuul:
ignored-headers: Access-Control-Allow-Origin, X-Frame-Options, Access-Control-Allow-Credentials, Vary, Access-Control-Allow-Credentials
sensitive-headers: Cookie,Set-Cookie,Authorization,LocalCookie
host:
socket-timeout-millis: 600000
...
...
...
sessioncontrol:
  path: /sessioncontrol/login/**
  **sensitive-headers: Cookie,Set-Cookie,Authorization,LocalCookie**
  url: 'http://localhost:8082/sessioncontrol/login'
  stripPrefix: true

...
...
...

But it works with this
zuul:
ignored-headers: Access-Control-Allow-Origin, X-Frame-Options, Access-Control-Allow-Credentials, Vary, Access-Control-Allow-Credentials
sensitive-headers: Cookie,Set-Cookie,Authorization,LocalCookie
host:
socket-timeout-millis: 600000
...
...
...
sessioncontrol:
  path: /sessioncontrol/login/**
  **_sensitive-headers:_**
  url: 'http://localhost:8082/sessioncontrol/login'
  stripPrefix: true

...
...
...

Please direct this question to the Spring project.

What exactly to do here. I sthe issue resolved as it saysthat second part works...??


Add spawn node

Working Example:
! FIX MERGE ISSUES BEFORE MERGING!
see #149  for API references migrated to the new inheritance structure

[ ] Add enumerator in some way so one can do Exp_1, ... Exp_n
[ ] loading a spawned node by name should work, currently only iteratiing over the spawner works

from zntrack import Node, dvc, zn
from zntrack.core.spawn import SpawnNode

class Spawner(SpawnNode):
    param1 = zn.iterable([x for x in range(3)])
    param2 = zn.iterable([x*10 + 10 for x in range(3)])
    param3 = zn.iterable([x*100 + 100 for x in range(3)])
    result = zn.outs()

    def run(self):
        self.result = self.param1 + self.param2 + self.param3
        print(self.result)

class Pooling(Node):
    deps = dvc.deps([x for x in Spawner.load()])
    result = zn.outs()

    def run(self):
        maximum = 0
        for node in self.deps:
            if node.result > maximum:
                maximum = node.result

        self.result = maximum

Spawner().write_dvc()
Pooling().write_dvc()

Have a condition(**kwargs): return True where the kwargs are all zniterables to loop over. This could be the easiest way to allow for e.g. x != y

moved to #184


[Tabular][fastai] Refactoring

Description of changes:

extracted and reused model-specific feature from tabular NN into fastai tabular NN
refactoring of fastai code

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-829-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-829/1/index.html

Code looks good to me.
When benchmarking it could also be good to look at whether:

this change improves FastAI model
this change improves autogluon-ensemble (I'd have slight worry that sharing common preprocessing between MXNet and FastAI nets may reduce ensemble diversity, even if it does help FastAI nets individually)

I also feel that the ensemble score might drop if the gains are only minor for FastAI, so it will be interesting to see the results.


Signing payloads with a size greater than 64kb fails while using CloudKMS

Problem
When updating contracts, the whole contract source is converted to hex and then sent as part of a contract update transactions. The resulting payload that needs to be signed thus can grow larger than 64kb. This runs into issues when using CloudKMS because it has a 64kb plain text limit on its signing (and encryption/decryption) API endpoints.
The issue was introduced in the May 5th commit where the data hashing was moved from being done in-memory to happening in the signer.
Steps to Reproduce

Attempt to deploy a contract larger than 64kb.

Acceptance Criteria

Any sized contract should be deployable

Context
We've worked around the issue by using the older version of flow-cli, so this is not immediately blocking.

Thanks for creating the issue.
I suggest to fix the issue by keeping the HSM hashing for data of size smaller than the limit, and pre-hashing outside the HSM for data larger than the limit. This would keep the in-HSM hashing as the default method.


Integrate llvm-project and bump dependencies

llvm-project: 5ae0472694f5
mlir-hlo: f6f29b1eb37e0601b1a06a3d3b0eb50c5c2b2b18

cherry-picked ea5b21f8f9cdd7cc97965a02d27967fe3c68cec1

tensorflow: c8efbf27244b2a3ea4856654d52bd821ad3cf5a8

There are several cmake changes in the upstream, which break the build. Looking into it.

The cmake issue is fixed, but now I have errors with TF.
ERROR: /home/kbuilder/.cache/bazel/_bazel_kbuilder/1900d0fac5123d725c8d2e08b3f8c209/external/org_tensorflow/tensorflow/core/kernels/mkl/BUILD:280:22: Compiling tensorflow/core/kernels/mkl/mkl_dequantize_op.cc failed: (Exit 1): clang failed: error executing command /usr/lib/llvm-9/bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -fcolor-diagnostics -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 355 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox
external/org_tensorflow/tensorflow/core/kernels/mkl/mkl_dequantize_op.cc:59:53: error: use 'template' keyword to treat 'flat' as a dependent template name
      const float min_range = ctx->input(kMinIndex).flat<float>()(0);
                                                    ^
                                                    template
external/org_tensorflow/tensorflow/core/kernels/mkl/mkl_dequantize_op.cc:60:53: error: use 'template' keyword to treat 'flat' as a dependent template name
      const float max_range = ctx->input(kMaxIndex).flat<float>()(0);
                                                    ^
                                                    template
2 errors generated.

The code using only flat was introduced by https://github.com/milpuz01/tensorflow/commit/b601c0560402354bc3337cb51d94ca407102969c in May. I am not sure why the errors appear now.

Okay, the code was made in May and pulled into tensorflow yesterday.

This is the breaking PR. I asked the author to revert and fix it.
https://github.com/tensorflow/tensorflow/pull/56086

The issues with TF and mhlo were resolved in the upstreams. Abandoning this PR to handle them in a different PR with the latest changes.


send email on failed master builds

There is currently no obvious indication if the master builds (still) succeeds.
Please:

[ ] send emails on failed master builds
[x] (if easily possible) add badge on main README if jobs succeed #521

@fel115 I assigned you as discussed. Ideally emails only get send if master was stable and just got unstable. (And not emails again when it remains unstable).

So is the desired way to link the issue in the commit and not link it anymore by hand? or either is fine, as long as its linked once?

Better by commits, then it is also in the history. But yes, the most important is that the link exists.


Add support for Klaytn, SFI

Coverage decreased (-40.3%) to 58.75% when pulling<PHONE_NUMBER>2b69a7a62945713177860681a44ea4 on metagabbar:klaytn into 2e23da17221ed8e18911f9a54adf4df3ac53b82a on synapsecns:dev.


Improve test coverage for the keep-alive scenario based on HTTP versions within curl transport

These recently added code-paths here are currently not covered by tests:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3775597&view=codecoverage-tab

@antkmsft, I am assigning this issue to you since you have the most expertise in this space, and introduced the change.
Feel free to unassign if you don't plan to work on it.


CdsPaginationButton: Disabled button has non-disabled styling

Describe the bug
In Clarity v6, when using <CdsPaginationButton action="first" aria-label="go to first" disabled></CdsPaginationButton>, the button is correctly disabled and has the disabled cursor icon, but the regular (i.e. non-disabled) styling:

I do notice that Clarity's Storybook looks fine, so it may be a React wrapper bug only: https://clarity.design/storybook/core/?path=/story/components-pagination--page
How to reproduce
See https://codesandbox.io/s/pagination-button-styling-bug-6tzc7x?file=/src/App.tsx
Expected behavior

Versions
Clarity project:

[x] Clarity Core
[ ] Clarity Angular/UI

Clarity version:

[ ] v5.x
[x] v6.0.1

Framework:

[ ] Angular
[x] React
[ ] Vue
[ ] Other:

Framework version:
React 17
Device:

OS: MacOS
Browser Chrome

It's not a react-only bug - for some reason the storybook is serving an old version of the library.
It looks like it broke with this PR: https://github.com/vmware-clarity/core/pull/52
Looking into it.


Output as pdf or word

The image size does not allow all sentences to fit in single image. It would be great of we could get pdf output.

Thanks for pointing out this, we will look into this and let you know.

Is this issue still open? If yes, I have a solution and would like to create a PR to solve it.

Hey @Siddhesh-Agarwal, thanks for reaching out. You can surely open up a PR for this issue. Make sure to go through the Contribution Guidelines.

it still not working...


Generator is not get the complete key value

Hello,
Thank you for make this package is really useful.
I'm facing an issue, in my .env file I have the following
API_URL=http://localhost:5050/path/to/api
and the generator is only generate part of this:
// GENERATED CODE - DO NOT MODIFY BY HAND

part of 'env.dart';

// **************************************************************************
// EnvifyGenerator
// **************************************************************************

class _Env {
  static const api_url = 'http://localhost:5050';
}

Testing I was able to see that the call work with the correct path anyway. So I'm closing this issue.


Incorrect thumbnail obtained when calling GetThumbnailAsync on just-downloaded OneDrive On-Demand picture

When using FileOpenPicker.PickMultipleFIlesAsync on a not-yet-downloaded OneDrive On-Demand picture file, and then subsequently calling GetThumbnailAsync(ThumbnailMode.SingleItem), it often times just returns the generic picture icon, even though the picture is downloaded by the time that's called and IsAvailable returns true.

The repro isn't consistent, but it happens most of the time. One time, I've had it actually correctly display the thumbnail.

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

ID: 8e9b19b5-9b36-67ac-f224-722356a4ffc0
Version Independent ID: aae2ca06-7de3-76fb-2b0c-995f84736f9f
Content: StorageFile.GetThumbnailAsync Method (Windows.Storage) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.storage.storagefile.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

Can confirm, can recreate, this issue is frustrating.
Sometimes it returns the right thumbnail, most of the times won't.

If there are any significant updates you all would like to see on our documentation, please open up this thread again. Closing for now due to staleness. Many thanks.


Reassigned drawer

Here is where you can check it out: https://pr-147ui.apps.pd01i.edc1.cf.ford.com/
Lets us know what you find!

SO GOOD!
@alexwojtala You don't currently have the persons role in window. Is that a seperate story or should that be aded before accepting this one?

@jshaddac That can be done in this story, for now we just pulled the code from the old ui, which did seem to get most of the current design. We figured we would just get it up quickly for feedback, we can definitely add that!

After taking another look, you should be able to see the person's role, right under their name

Took me a second to realize that the person I was moving around didnt even have a role. Looks good!!!


⚠ Cioos National CKAN Sitemap has degraded performance

In cd73e8f, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 12972 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 092b7f0 after 7 minutes.


Implicitly create parameter when passing non artifact to decorated functions

Describe your changes and why you are making these changes
When non-artifact types are passed to decorated functions, we attempt to implicitly create a parameter with name being the function argument's name and default value being the user-specified value. We print out a warning when this happens. If there is already an operator with the same name in the dag, we throw an error.
Related issue number (if any)
Checklist before requesting a review

[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@kenxu95 addressed comments.


'updateProfile' does not update 'auth subscribe'

Version info

Angular: 2.1.0
Firebase: 3
AngularFire: 2.0.0-beta.6
How to reproduce these conditions
Failing test unit, Plunkr, or JSFiddle demonstrating the problem
http://embed.plnkr.co/QSIlt7mIlXPu39PyJm4d/
Steps to set up and reproduce

Click login
Click Update Profile

Expected behavior
That the timestamp in the name would update because of the updateProfile function updating the displayName and the af.auth.subscribe listener listening for changes to the auth object.
Actual behavior
The displayName is not updated via the af.auth.subscribe listener.

Data is updated but you subscribe is not invoked. If you update this.name in promise .then function it should work. See my plnkr: https://plnkr.co/edit/cORFFvoMB6p4nZT1OrSs

Great that works, but should the subscribe handler still be invoked?

No idea what you mean by "should the subscribe handler still be invoked."
The problem with updateProfile() is a known SDK bug that should be fixed in an upcoming release :(. Not an AngularFire issue.

Thanks @katowulf. I just mean, after updateProfile() is run, should the callback for the auth.subscribe be called with new auth info. But as you said, it's a known SDK bug so :nothingtodohere:

Ah, of course. Well glad I, err, answered it accidentally.


DOM Manipulation With Webassembly?

How would that work?
Will you also provide complete webassembly examples?

DOM manipulation is sorta working on master, in a limited form. You can follow the instructions in the README, and then you can do:
from browser import document
elem = document.query_selector("css query")
elem.set_attr("attr", "value")
elem.get_attr("attr")

It's in it's early stages now, but we're working on fleshing it out more.

@coolreader18 how could we document this features in a nice way? Maybe we can add some examples on the website for this.

#210 looks like related issue


Error using workoffline REREG

Hi,
I have an issue when executing an offline transaction with Era discovery. I ran

05a to create the pool.json file
Edited the JSON file to include my changes
05c   REREG

Then when I brought offlinetransfer to my online server, I tried to execute and got this error:
ERROR - Online(alonzo) and Offline(auto) Era mismatch!
I tried to review the code in the workoffline script and in the JSON file, and considered just manually editing the JSON to use alonzo instead of auto, but figured I'd submit this first and see if you have a BKM just in case I screw something up.

BTW, outcome when I run "execute":
Version-Info: cli 1.30.1 / node 1.30.1          Scripts-Mode: online
Checking Content of the offlineFile: offlineTransfer.json
Protocol-Parameters: present           Protocol-Era: auto
History-Entries: 45
Last-Action: extracted file 'myPool.metadata.json' (Sat, 06 Nov 2021 15:15:57 -0700)
Offline Version: cli 1.30.1

Here is my transaction in the offline JSON file (with names and addresses removed)
<   "transactions": [
<     {
<       "date": "Sat, 06 Nov 2021 15:09:14 -0700",
<       "type": "PoolReRegistration",
<       "era": "auto",
<       "fromAddr": "",
<       "sendFromAddr": "",
<       "toAddr": "",
<       "sendToAddr": "",
<       "poolMetaTicker": "",
<       "regProtectionKey": "",
<       "poolMetaUrl": "https:///pool.metadata.json",
<       "poolMetaHash": "",
<       "txJSON": {
<         "type": "Tx MaryEra",
<         "description": "",
<         "cborHex": ""
<       }
<     }
<   ]

I worked around the issue for now by removing "exit 1" from this line in 01_workoffline:
if [[ ! "${transactionEra}" == "$(get_NodeEra)" ]]; then echo -e "\e[35mERROR - Online($(get_NodeEra)) and Offline(${transactionEra}) Era mismatch!\e[0m\n"; exit 1; fi
I put the "exit 1" back after what appeared to be a successful execution of REREG

thx @robcolby yes i think we can lift the restriction here if an offline transaction was initiated via an auto-era transaction. this is the case when the offline machine does not know the era it is currently in it. its a safety feature from the past and only an additional check. i'll look into it, thx for the discovery!

hmm... it could be an issue with 1.30.1, because sometimes 1.30.1 is not reporting back queries like it should. only an empty query is reported back. in this case, the outcome would be "auto". can you check the "protocol" section in your offlineTransfer.json please. if your online node is synced when you do an "add", "new" or "clear" it updates the current protocol data in the offlineTransfer.json. so i guess you have "auto" in the .protocol.era key in there?

I actually did check the protocol section of offlineTransfer at the time, it showed "auto" as well. I wasn't sure if I should manually edit that or not.
I suspect you are correct in that 1.30.1 is not reporting as it should, because I actually did another REREG shortly after this and it worked the second time without me having to edit 01_workoffline.
I also just checked my offlineTrasfer file and can see it now has the correct Era at the top.

hi, actually i have the pool update in offline mode as a full example in the readme, you can find it here:
https://github.com/gitmachtl/scripts/tree/master/cardano/mainnet#update-stakepool-parameters-on-the-blockchain-in-offline-mode
i will close this issue for now, because its a node issue. 1.31.0 does not show the behaviour.


Add panic level logging

Hello,
First let me say that we love this library.  Being able to turn verbose logging on by tag at runtime is a huge win for us.  Now to the issue we're trying to solve.
We found that if we panic right after we log.Error, our error log was not ending up in our logs.  This was because we were not waiting sufficiently for all the log entries to be processed before panicking.  So, we've forked the library in order to add a log.Panic function, which will wait until the panic log message has been processed and written to the appenders before panicking.

Thanks for this one, very useful.
I have some ideas to stream line the logging pipeline, so i will probably be editing the implementation soon, but should keep the API and behavior exactly the same.


Cursor slip random when select a string via mouse

VSCode Version:   1.36.1
OS Version: macOS 10.14.6

Steps to Reproduce:

For example, I got a function call in my code.

some_func_call(Cursor slip random when select a string via mouse)

I select the string slip random when by mouse, and I want to paste a string not slip any more while  then the cursor might slip and I will get a result like

some_func_call(Cursor snot slip any more while select a string via mouse)

or
some_func_call(Cursor not slip any more whilen select a string via mouse)

Notice the redundant s or redundant n in these examples. That's very bad because it cause program crash and very hard to notice.
Actually I do select strings very very carefully because of this problem, but I still keep slip into this problem when coding in VSCode.

Does this issue occur when all extensions are disabled?: Yes
Here is my screen shots.
You can find the 1st and 2nd selection go well but in the 3rd selection, the cursor slips and the letter l slips from selection.
And if you check the screen shot carefully you can find that I finish each selection very carefully. Otherwise it would be something like the 2nd selection, it slips and included the suffix .

It seems like the mouse drag/move action tends to be too sensitive.
Issue https://github.com/microsoft/vscode/issues/78464 seems like to be cause by similar reason. mouse drag/move action tends to be too sensitive.

@dove-young The editor works by taking the position which is closest. So if you click in the first half of a character, the cursor will go to the left of the character, and if you click in the second half of a character, the cursor will go to the right of the character.

Is this unexpected?

@alexandrudima you know the problem is it makes the behavior not predictable.
Unpredictable is dangers in programming.  1st, it leads to errors, as what I demonstrated above. 2nd and the even worse side is unpredictable behavior would lead to errors that you don't mean, and you don't have any sense that it would be a typo which made this error.  Many dynamic programming language do not have capacity to indicate error source like this.

@dove-young
I think this behavior is predictable and aligned with other software.
Edge Dev (based on Chromium):

Firefox

Microsoft Word

We are trying to be consistent with other software and not thus predictable.


Automated Package Upgrade

:tada: This PR is included in version 12.4.60 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


dotnetfoundation.org/News navigation

The ‘https://dotnetfoundation.org/News’ navigates to the ‘https://dotnetfoundation.org/Get-Involved’ page, should probably remove this binding url especially as there is a news link in the menu that points to blog ‘https://dotnetfoundation.org/Blog’

This seems to be fixed now, closing.


Option to use vectorized versions of density callables

I found that using subject is quite easy thanks to numpy broadcasting and emcee nice code:) It significantly speeds up sampling and costs only tiny modification of callable density code (if numpy broadcasting is possible).
Thanks for nice toolkit,
Ilya

This is finally supported in the upcoming 3.0 release!


Pi5 HW PWM not working

I'm on version 0.19.0 and this includes the commit 6bbc5ef8993982e1917aa07f3715a9e97e0fe97c.
This code works on a Pi4 but on my Pi5 I'm getting errors and don't know why.
I have set dtoverlay=pwm-2chan,pin=12,func=4,pin2=13,func2=4 in the config. Also with the permission udev rules:
SUBSYSTEM=="pwm*", PROGRAM="/bin/sh -c '\
    chown -R root:gpio /sys/class/pwm && chmod -R 770 /sys/class/pwm;\
    chown -R root:gpio /sys/devices/platform/soc/*.pwm/pwm/pwmchip* &&\
    chmod -R 770 /sys/devices/platform/soc/*.pwm/pwm/pwmchip*\
'"

$ pinctrl get 12,13
12: a0    pd | lo // GPIO12 = PWM0_CHAN0
13: a0    pd | lo // GPIO13 = PWM0_CHAN1

$  ls -l /sys/class/pwm
total 0
lrwxrwxrwx 1 root gpio 0 Sep  1 08:43 pwmchip0 -> ../../devices/platform/soc/107d517a80.pwm/pwm/pwmchip0/
lrwxrwxrwx 1 root gpio 0 Sep  1 08:43 pwmchip2 -> ../../devices/platform/axi/1000120000.pcie/1f00098000.pwm/pwm/pwmchip2/
lrwxrwxrwx 1 root gpio 0 Sep  1 08:43 pwmchip6 -> ../../devices/platform/axi/1000120000.pcie/1f0009c000.pwm/pwm/pwmchip6/

WHen I wanto to initialize the PWM pin:
    let servo_1 = Pwm::with_period(
        Channel::Pwm0,
        Duration::from_millis(SERVO_PWM_PERIOD_MS),
        Duration::from_micros(SERVO_PWM_PULSE_MAX_US),
        Polarity::Normal,
        true,
    )
    .expect("Should be able to connect to PWM pin");

When check the output of DeviceInfo It does read correctly:
rppal::system::DeviceInfo::new() = Ok(
    DeviceInfo {
        model: RaspberryPi5,
        soc: Bcm2712,
        peripheral_base:<PHONE_NUMBER>,
        gpio_offset: 851968,
        gpio_lines: 28,
        gpio_interface: Rp1,
        pwm_chip: 2,
        pwm_channels: [
            2,
            3,
        ],
    },
)

I'm getting the error Io(Os { code: 13, kind: PermissionDenied, message: "Permission denied" }).
What am I'm missing here or is there still a bug somewhere?

Hi @johansmitsnl. Getting a Permission Denied error from with_period typically means RPPAL can't export the channel, which it does by creating /sys/class/pwm/pwmchip2/pwm2. The first thing I would check is make sure the user you're logged in as is part of the gpio group.

Hi @golemparts. the user is part of the group:
uid=1000(piuser) gid=1000(piuser) groups=1000(piuser),4(adm),20(dialout),24(cdrom),27(sudo),29(audio),44(video),46(plugdev),60(games),100(users),102(input),105(render),106(netdev),993(gpio),994(i2c),995(spi)

I noticed with the current udev rules the permissions are wrong:
lrwxrwxrwx 1 root root     0 Sep  4 18:41 device -> ../../../1f00098000.pwm/
--w------- 1 root root 16384 Sep  4 18:41 export
-r--r--r-- 1 root root 16384 Sep  4 18:41 npwm
drwxr-xr-x 2 root root     0 Sep  4 18:41 power/
lrwxrwxrwx 1 root root     0 Jan  1  1970 subsystem -> ../../../../../../../class/pwm/
-rw-r--r-- 1 root root 16384 Jan  1  1970 uevent
--w------- 1 root root 16384 Sep  4 18:41 unexport

But when I chown them with sudo chown root:gpio -Rf * . and I try to touch of make a dir in: /sys/class/pwm/pwmchip2/pwm2 I get the permission denied:
touch /sys/class/pwm/pwmchip2/pwm2
touch: cannot touch '/sys/class/pwm/pwmchip2/pwm2': Permission denied

But even when I use sudo to create the directory I also get permission denied.

I had some time to look into the hardware PWM code some more. My earlier mention of the creation of /sys/class/pwm/pwmchip2/pwm2 causing the Permission Denied error wasn't accurate, as the creation of pwm2 is triggered by writing a 2 (for channel 0) to /sys/class/pwm/pwmchip2/export.
I checked the permissions on my Pi 5, which are different from yours:
--w--w---- 1 root gpio 16384 Sep  4 11:41 export
-r--r--r-- 1 root gpio 16384 Sep  4 11:41 npwm
drwxrwxr-x 2 root gpio     0 Sep  4 11:41 power
lrwxrwxrwx 1 root gpio     0 Sep  4 11:41 subsystem -> ../../../../../../../class/pwm
-rw-rw-r-- 1 root gpio 16384 Sep  4 11:41 uevent
--w--w---- 1 root gpio 16384 Sep  4 11:41 unexport```

I don't actually recall having to edit `/etc/udev/rules.d/99-com.rules` anymore, so it's possible that recommendation in the documentation is no longer needed for more recent versions of Raspberry Pi OS. Could you check what permissions it defaults to after deleting those rules and rebooting?

Without the udev rules the permissions are not of the GPIO group:
$ stat /sys/class/pwm/pwmchip2/export
  File: /sys/class/pwm/pwmchip2/export
  Size: 16384     	Blocks: 0          IO Block: 16384  regular file
Device: 0,18	Inode: 23381       Links: 1
Access: (0200/--w-------)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2024-09-05 19:14:28.863921234 +0200
Modify: 2024-09-05 19:14:28.863283257 +0200
Change: 2024-09-05 19:14:28.863283257 +0200
 Birth: -

I don't run as root but as a normal user so this can't work since the permission is 200 and not even writable for the group.
I changes the udev rule to this:
SUBSYSTEM=="pwm*", PROGRAM="/bin/sh -c '\
    chown -R root:gpio /sys/class/pwm && chmod -R 770 /sys/class/pwm;\
    chown -R root:gpio /sys/devices/platform/soc/*.pwm/pwm/pwmchip* &&\
    chown root:gpio /sys/class/pwm/pwmchip*/export &&\
    chmod 220 /sys/class/pwm/pwmchip*/export &&\
    chmod -R 770 /sys/devices/platform/soc/*.pwm/pwm/pwmchip*\

This makes sure the permission is correct on the export path.
But this is also not sufficient due to permission lacking write permission on the new created pwm2 directory:
$ ls -l /sys/class/pwm/pwmchip2/pwm2
-r--r--r-- 1 root root 16384 Sep  5 19:27 capture
-rw-r--r-- 1 root root 16384 Sep  5 19:27 duty_cycle
-rw-r--r-- 1 root root 16384 Sep  5 19:27 enable
-rw-r--r-- 1 root root 16384 Sep  5 19:27 period
-rw-r--r-- 1 root root 16384 Sep  5 19:27 polarity
drwxr-xr-x 2 root root     0 Sep  5 19:27 power/
-rw-r--r-- 1 root root 16384 Sep  5 19:27 uevent

By manually adding chmod -R g+rw on the pwm2 dir it still isn't solved and I still get the permission denied error.
$ ls -l /sys/class/pwm/pwmchip2/pwm2=
-r--rw-r-- 1 root gpio 16384 Sep  5 19:27 capture
-rw-rw-r-- 1 root gpio 16384 Sep  5 19:32 duty_cycle
-rw-rw-r-- 1 root gpio 16384 Sep  5 19:32 enable
-rw-rw-r-- 1 root gpio 16384 Sep  5 19:32 period
-rw-rw-r-- 1 root gpio 16384 Sep  5 19:32 polarity
drwxrwxr-x 2 root gpio     0 Sep  5 19:27 power/
-rw-rw-r-- 1 root gpio 16384 Sep  5 19:27 uevent

If this is happening with a fresh install of a recent Raspberry Pi OS release, I think at this point it's better if you ask about this on the official Raspberry Pi forum, as this does not appear to be an RPPAL issue but a linux one, and I'm not familiar enough with how best to solve permission issues like this.
I'd appreciate it if you can let me know when you've found a solution, as it may be useful to people who run into the same problem.

Yes this is a fresh install of 1 week ago. I'll let you know the outcome but I'm on holiday for the next 3 weeks so it might take some time for me to reply.

Created a question on the pi forum

No response on the forum but I did a reinstall on the pi.
Now the permissions are the same as yours but I have an issue with the Channel::Pwm1 that fails. The Channel::Pwm0 does not panic.
Now my output of the directory is like this:
ls -l /sys/class/pwm/pwmchip2/
total 0
lrwxrwxrwx 1 root gpio     0 Oct 30 20:32 device -> ../../../1f0009c000.pwm
--w--w---- 1 root gpio 16384 Oct 30 20:42 export
-r--r--r-- 1 root gpio 16384 Oct 30 20:32 npwm
drwxrwxr-x 2 root gpio     0 Oct 30 20:32 power
lrwxrwxrwx 1 root gpio     0 Oct 30 20:32 subsystem -> ../../../../../../../class/pwm
-rw-rw-r-- 1 root gpio 16384 Oct 30 20:32 uevent
--w--w---- 1 root gpio 16384 Oct 30 20:40 unexport

Full firmware config:
# /boot/firmware/config.txt

# For more options and information see
# http://rptl.io/configtxt
# Some settings may impact device functionality. See link above for details

# Uncomment some or all of these to enable the optional hardware interfaces
dtparam=i2c_arm=on
#dtparam=i2s=on
#dtparam=spi=on

# Enable audio (loads snd_bcm2835)
dtparam=audio=on

# Additional overlays and parameters are documented
# /boot/firmware/overlays/README

# Automatically load overlays for detected cameras
camera_auto_detect=1

# Automatically load overlays for detected DSI displays
display_auto_detect=1

# Automatically load initramfs files, if found
auto_initramfs=1

# Enable DRM VC4 V3D driver
dtoverlay=vc4-kms-v3d
max_framebuffers=2

# Don't have the firmware create an initial video= setting in cmdline.txt.
# Use the kernel's default instead.
disable_fw_kms_setup=1

# Run in 64-bit mode
arm_64bit=1

# Disable compensation for displays with overscan
disable_overscan=1

# Run as fast as firmware / board allows
arm_boost=1

[cm4]
# Enable host mode on the 2711 built-in XHCI USB controller.
# This line should be removed if the legacy DWC2 controller is required
# (e.g. for USB device mode) or if USB support is not required.
otg_mode=1

[cm5]
dtoverlay=dwc2,dr_mode=host
dtoverlay=pwm-2chan,pin=12,func=4,pin2=13,func2=4

[all]

Maybe a suggestion where to look as all permissions seem correct?

Glad to hear a fresh install fixed the initial problem. I'm not sure what would cause an issue with PWM1 when PWM0 is working. Your configuration looks correct. What error are you getting?

The code I use is:
let servo_2 = Pwm::with_period(
        Channel::Pwm1,
        Duration::from_millis(SERVO_PWM_PERIOD_MS),
        Duration::from_micros(SERVO_PWM_PULSE_MAX_US),
        Polarity::Normal,
        true,
    )
    .expect("Should be able to connect to PWM 2 pin");

And the error running the code:
thread 'main' panicked at recorder/src/gpio.rs:285:6:
Should be able to connect to PWM 2 pin: Io(Os { code: 16, kind: ResourceBusy, message: "Device or resource busy" })

@golemparts do you have ideas what can cause this or where you able to test/verify it on a pi5?

I haven't had a chance to test this myself, but if the error message is accurate, I would suspect some other process is using that PWM channel. Another possibility is RPPAL is trying to access the wrong PWM channel, but I would expect a Permission denied error in that case.
On the Pi 5, RPPAL will use /sys/class/pwm/pwmchip2/pwm2 for Pwm0 and /sys/class/pwm/pwmchip2/pwm3 for Pwm1. You could try exporting pwm3 from the commandline and check what error message you get.
I should have some time this upcoming weekend to test the second PWM channel myself and see if I get the same error.

@golemparts thanks for the feedback and your time to test. How can I check it on the command line?

Check out this answer for some example command line commands. You'll want to use pwmchip2 instead of the device mentioned in the example, and echo 3 to export pwm3.

@johansmitsnl I just ran some tests on my Pi 5, and both PWM channels worked fine. However, while recreating your setup I did notice an issue in your /boot/firmware/config.txt. You have the pwm-2chan overlay listed under [cm5] instead of [all]. Try moving it there and rebooting. Hopefully that'll resolve the issue for you.

Closing this due to inactivity. If you're still encountering this issue and the above-mentioned didn't resolve it, please let me know.

An issue was discovered specifically related to PWM on GPIO12/13 on a Pi 5, which was fixed in release 0.22.0. If you're still encountering this issue, try that release.

I did some testing but could not get it to work so I didn't provide a update of nothing to report. But with the 0.22.0 it works again :+1:
Thanks for the effort!


Changelogs needs dates so I can know if I am out of date or not

Same for the config file itself, the one I used doesn't tell what version I was in.

Good and somewhat obvious idea. Wonder why I didn't already do it. Will try do add this today.

Fixed via ef6fafe67b679e88ab2f061d90472117072cf871 and dodekeract/manta-config-engine-app@2f2c85957c312b524c6290dea8937c0339e01509


Add anonymous classes

Hey, not sure if you'd prefer to get several small pull requests for various version features, but it seems like there are a number of things that can probably be extracted from these pages: https://www.php.net/manual/en/migration70.new-features.php (They all follow the format of php.net/manual/migrationVERSION.new-features.php). I'll wait before submitting any more off that, but it may even be possible to create a tool to extract the info from those pages to be manually edited.

Thanks again - keep them coming! Single-feature PRs are probably best. Otherwise any discussion is going to get complicated.
Automation has been suggested by a lot of people, but the documentation isn't in any standard format, so it's hard to achieve.


[Proposal] Add "bot account" as a type of user

Gitea version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Hello, Sometimes we will want add a new accout which only used by bot apps to call api. So I think maybe we can add it as a new user type. It maybe follow this tips.

Bot account willn't login on ui page
Bot account maybe should not have ability create it's own repos
Each boat account should belong to a user or an org, and this user or the owners of this org can change  the setting of the bot account. (important)
manager can limit the max number of the bot acounts that one user and org can have
Bot account is not necessary to request email and the system will't send email notify and action notify to them also
show bot label on frofile page
......

Maybe usefull to fix #12985.
Screenshots

Bot account should not have a password
Bot account should have the ability to create app tokens and add ssh/gpg keys

This would also be interesting for gpg key checks of mirrored repositories.
You could just add a bot for the original owner.

I would say that multiple owners could also be useful, so more than 1 person can operate the bot (useful if one of the owner is away)

My proposal would be:

Bot can be server wide (managed by server admins) - owner_id = nil
Bot can be organization owned (managed by organization owners) - owner_id = organization id
Bot can be user owners (managed by single user) - owner_id = user id

now, I found it in gh, maybe we can add an Similar mechanism.
https://docs.github.com/en/free-pro-team@latest/developers/apps/about-apps#about-github-apps

I have a bot account in my CI pipeline that adds commits to PR's (code formatting and cleanup, dependency upgrades, ..).
We trust this bot, so these commits don't require review. But these commits cause the earlier approvals of the PR to become stale.
Is that something that could be included in the requirements for the bot account implementation?

Maybe this can also be used to change the sort order in the request review dropdown:
Because we have a bot that deploys a preview and creates a comment with a preview link the bot will always appear as the first suggestion in the list.

Have there been any proposals to implement this? If one were to start implementing this, what would be a good starting point?

Have there been any proposals to implement this? If one were to start implementing this, what would be a good starting point?

looks github app is similar  with this.

So technically there is now a type of bot, but it doesn't have any logic for displaying a badge etc..

So technically there is now a type of bot, but it doesn't have any logic for displaying a badge etc..

You mean with current Gitea? How would bot accounts work with the existing version?

Well… We regocnize them since #24026.
Practically, however, the only effect this has at the moment is prohibiting login and reserving the username.
Have a look at the PR for more details.


Pagination Search bar does not show database columns in "#search_key" dropdown

Version information

Laravel: v5.6
Voyager: v1.1.0
PHP: 7.1
Database: 10.1.22-MariaDB- Debian 9.0

Description

When Server Side pagination is activated, the search functionality fails to work correctly. The Search key select drop down in the search bar shows "No Results Found". The $searchable variable returns as null.
$searchable = $dataType->server_side ? array_keys(SchemaManager::describeTable(app($dataType->model_name)->getTable())->toArray()) : '';
Steps To Reproduce
Steps to reproduce the behavior:

Create a BREAD
Activate the "Server-side Pagination" option.
Try to perform a search.
The first field in the search bar reads "No Results Found".

Expected behavior
The first field should provide a list of the database columns to make the search possible
Screenshots

Please fill in the version numbers, otherwise we are not able to help you.

I've updated the numbers..

Database: [type] [version] (e.g. MySQL 8.0) is in your case after Voyager the most important version number.

I've updated that. Thanks you.

So I traced back to VoyagerBaseController.php on this line
$searchable = $dataType->server_side ? array_keys(SchemaManager::describeTable(app($dataType->model_name)->getTable())->toArray()) : '';
I checked the describeTable(), then the listTableDetails($tableName)
I tried putting the raw table name (for instance invoices), and it worked.
The $tableName is gotten from the $dataType->model_name which returns the Eloquent model class of the bread (for instance, App\Invoice).
I think the SchemaManager doesn't work with Eloquent, but with raw table name.
Suggestion

We can add a separate column to store the raw table name, which can be referenced for this particular functionality
We can create a function to return the raw table name when using the $dataType->model_name
We can make SchemaManager work with Eloquent Models.

I might be wrong with my conclusions, but what do we think?

app($dataType->model_name) gives an instance of your model and getTable() returns the name of the table as a string.
So that's not the problem.

So I traced back to VoyagerBaseController.php on this line
$searchable = $dataType->server_side ? array_keys(SchemaManager::describeTable(app($dataType->model_name)->getTable())->toArray()) : '';
I checked the describeTable(), then the listTableDetails($tableName)
I tried putting the raw table name (for instance invoices), and it worked.
The $tableName is gotten from the $dataType->model_name which returns the Eloquent model class of the bread (for instance, App\Invoice).
I think the SchemaManager doesn't work with Eloquent, but with raw table name.
Suggestion
* We can add a separate column to store the raw table name, which can be referenced for this particular functionality

* We can create a function to return the raw table name when using the `$dataType->model_name`

* We can make `SchemaManager` work with Eloquent Models.

I might be wrong with my conclusions, but what do we think?

I was Very Wrong with my conclusion. The issue is with the prefix I added to my DB Tables.
The eloquent model returned the table name as invoice, instead of mums_invoice, where mums_ is the prefix.
To solve this, I created a new database connection with "prefix" => null, and specified my $table="mums_invoices", to make it return the required name to the SchemaManager.
The problem was solved by declaring the actual table name in my eloquent model.
Thank you.


BUG: Different results between numpy.fft.rfft and scipy.signal.freqz

Describe your issue.
I'm using the following two functions to calculate the frequency response of an impulse response: numpy.fft.rfft and scipy.signal.freqz, but I get different results. Let's simplify the problem using a small number of FFT.
import numpy as np
import scipy.signal as signal
h = np.array([1,2,3])
H1 = np.fft.rfft(h, n=4)
w, H2 = signal.freqz(h, 1, worN=len(H1), include_nyquist=True)

NFFT = 4, rfft calculates DFT at frequencies 0, pi/2, pi which is exactly equal to w returned by freqz
>>> print(w)
[0.         1.57079633 3.14159265]

However H2 is different from H1
>>> print(H1)
[ 6.+0.j -2.-2.j  2.+0.j]
>>> print(H2)
[ 6. +0.j          0.5-4.33012702j -1.5+0.8660254j ]

The DFT at pi/2 should be real-valued for a real-valued signal, so rfft gives the correct result.
From the source code of freqz in line 438-455 of scipy/signal/_filter_design.py we can see why it gives the wrong result. It doesn't give the nyquist result at all even include_nyquist is set to True.
Reproducing Code Example
import numpy as np
import scipy.signal as signal
h = np.array([1,2,3])
H1 = np.fft.rfft(h, n=4)
w, H2 = signal.freqz(h, 1, worN=len(H1), include_nyquist=True)
print(H1)
print(H2)

Error message
No error occurs.

SciPy/NumPy/Python version information
1.9.3 1.23.4 sys.version_info(major=3, minor=9, micro=0, releaselevel='final', serial=0)

Thanks for reporting the issue, @happyTonakai.  That's definitely a bug. It works in some cases (specifically the IIR case where a is not a scalar, or when worN is less than the length of b), but in examples such yours or the following, setting include_nyquist=True doesn't actually cause the Nyquist frequency to be included in the calculation of h, so the values returned in h do not correspond to the frequencies returned in w (except at w=0):
In [56]: freqz([1, 2, 3, 2], worN=8)
Out[56]:
(array([0.        , 0.39269908, 0.78539816, 1.17809725, 1.57079633,
        1.96349541, 2.35619449, 2.74889357]),
 array([ 8.        +0.j        ,  5.73444627-4.73444627j,
         1.        -5.82842712j, -2.20371254-3.20371254j,
        -2.        +0.j        , -0.03892814+1.03892814j,
         1.        +0.17157288j,  0.50819441-0.49180559j]))

In [57]: freqz([1, 2, 3, 2], worN=8, include_nyquist=True)
Out[57]:
(array([0.        , 0.44879895, 0.8975979 , 1.34639685, 1.7951958 ,
        2.24399475, 2.6927937 , 3.14159265]),
 array([ 8.        +0.j        ,  5.73444627-4.73444627j,
         1.        -5.82842712j, -2.20371254-3.20371254j,
        -2.        +0.j        , -0.03892814+1.03892814j,
         1.        +0.17157288j,  0.50819441-0.49180559j]))

The parameter include_nyquist was added in https://github.com/scipy/scipy/pull/11368; @AtsushiSakai and @larsoner, can you take a look at this?

Yikes I think we just changed the frequency returned, not the actual values.
To avoid this problem, I think we should do two things:

Don't use np.linspace to construct the frequencies, use rfftfreq instead. It makes sure they're always at least consistent.
Fix the actual bug mentioned here by changing the number of FFT points and slicing afterward correctly

@AtsushiSakai are you up for giving it a shot?

I discussed this in #15273, there is a solution proposed in one my posts there.
The frequency grid is wrong, when using rfft. To illustrate, two calls of to freqz give different results:
import numpy as np
import scipy.signal as sig
d = sig.unit_impulse(4,3)
d_ = np.pad(d, (0,126-len(d)))
w, D = sig.freqz(d,1,worN=64,include_nyquist=True)
w, Dp = sig.freqz(d,1,worN=w)
D_ = np.fft.rfft(d_)
w_ = np.fft.rfftfreq(len(d_)) * 2 * np.pi
print('error freqz[rfft] v/s freqz[poly] = ', np.sum(np.abs(D-Dp)**2))
print('error freqz[rfft] v/s np.fft.rfft = ', np.sum(np.abs(D-D_)**2))
print('error freqz[poly] v/s np.fft.rfft = ', np.sum(np.abs(D_-Dp)**2))

The output:
error freqz[rfft] v/s freqz[poly] =  0.4657960974715796
error freqz[rfft] v/s np.fft.rfft =  0.4657960974715789
error freqz[poly] v/s np.fft.rfft =  1.1300975888209927e-29

@drestebon would you be up for opening a PR to fix the accuracy / grid problem? Ultimately we should make sure the performance is correct first, i.e., in a first PR with test cases showing errant behavior on main and fixes on your PR. Then we can improve performance (assuming it's not already part of the fixes) in a separate PR. WDYT?

@larsoner I will try to squeeze a couple of hours out of next week (or the following) ;)


Switch statement crashes jslint

Describe the bug

A clear and concise description of what the bug is.

jslint test.js

[JSLint was unable to finish] Cannot read property 'disrupt' of undefined // line 1, column 1

TypeError: [JSLint was unable to finish] Cannot read property 'disrupt' of undefined
at parse_statements (file:///opt/jslint/jslint.mjs:4231:35)
at jslint_phase3_parse (file:///opt/jslint/jslint.mjs:6347:24)
at jslint (file:///opt/jslint/jslint.mjs:862:9)
at jslint_from_file (file:///opt/jslint/jslint.mjs:1075:24)
at jslint_cli (file:///opt/jslint/jslint.mjs:1275:5)
2. Expected 'switch' to be in a function. // line 1, column 1
switch(a)
3. Expected statements before 'default'. // line 4, column 3
default:

Jslint edition:

v2021.9.20

Code snippet (if applicable):

switch(a)
{
case 1:
default:
}
Expected behavior
Expect it to not get a TypeError

thx for reporting.  should be fixed in cl fce38bcc


How to disable the opacity on drag?

On Farhadi's demo there doesn't seem to be opacity when dragging items. See http://farhadi.ir/projects/html5sortable/
Is there a way to disable this opacity? I've but couldn't find a solution..

This is currently not possible, as this is the default HTML5 Drag & Drop behavior. A custom ghost creation would have to be implemented.


Replace Nextclade 0.x with Nextclade 1.x

Description of proposed changes
This makes ingest to use the new Nextclade C++ implementation. There is a few differences in this version.
Differences compared to 0.x:

It should be much faster
It does not require input sequences batching or external parallelization, so batching scripts are removed.
Nodejs is no longer required
Added time command to docker container to measure Nextclade's performance.
Nextclade 1.x requires additional external data to be downloaded. This is handled inside ./bin/run-nextclade script. This data is placed to data/gisaid/nextclade-inputs. Perhaps should be moved to tmp/ or similar?
There are now some additional outputs you cannot opt out from, such as aligned sequences, peptides and insertions. They currently go to data/gisaid/nextclade. We should probably make these optional in Nextclade itself.
If input fasta is empty, Nextclade currently fails with error (bug or feature)? So there is an external check in ./bin/ingest-gisaid to account for that and skip if there are no sequences. However, when it is the case, the script ./bin/flag-metadata fails with:AttributeError: 'DataFrame' object has no attribute 'divergence

Calculation of divergence and other metrics should probably be moved out from ./bin/join-metadata-and-clades.

Other considerations:

Full recompute on entire dataset will be needed when going live with this
The flag --in-order is used to preserve the order of outputs in TSV. I think this is needed for ./bin/join-metadata-and-clades to function properly.
The new flag --include-reference is currently not used, as we seem to not care about the ref sequence. It will also break the order.

Future improvements:

The newly detected sequences can be placed on auspice tree if needed with --output-tree. This has some performance hit: both memory consumption and and runtime.
The JSON output can be written. It contains more info compared to TSV and easier to parse. This also has some performance hit: both memory consumption and and runtime.

Files of interest:
https://github.com/nextstrain/ncov-ingest/blob/0f5716348571cbe3b77bd168e5b3c57fa5bfebd7/bin/ingest-gisaid#L96-L115
https://github.com/nextstrain/ncov-ingest/blob/0f5716348571cbe3b77bd168e5b3c57fa5bfebd7/bin/run-nextclade#L25-L54
Related issue(s)

Testing

This has passed. I updated the clades on S3 such that they contain 21A and will merge this now. It will now use the new version of nextclade which is a lot faster.


Implement support for IQueryable when reading data

Maybe some help can be found here https://iqtoolkit.codeplex.com/

https://github.com/re-motion/Relinq
http://relinq.codeplex.com
http://queue.acm.org/detail.cfm?id=2001564

Have you may any progress on this? is this on the back burner?

Hehe. This issue has been here for quite a while.
I did push on implementing some of it a long time ago, but the complexity got to me, given my limited time.
Wanted to do implement it based Relinq, but always found the documentation rather... lacking.
So, not much is happening on this issue right now, I am afraid.


Bot Registration for web channel for Production environment

Hello Team
I have created the Bot using Microsoft Bot framework in C# and I have hosted the Bot in the cloud environment (Pivotal cloud Foundry) not in Azure. I have registered the bot in the link (https://dev.botframework.com/bots/new)  to enable webchat using direct line and I have received the   appid and app password for that. I Understand that registration is free without azure subscription to enable the webchat. I have red the forum link(https://techcommunity.microsoft.com/t5/teams-developer/register-bot-without-azure-process/m-p/1490808) which clearly tells that subscription is not required for bot registration if the bot is hosted in the non-azure environment. Now my Bot is working without any issues. But my question is shall i proceed with same process for production environment. is there any limitation on this? Kindly Advise on this

Hi @NaveenKumarNN
If you create a Bot Registration with within a subscription (for example Pay-As-You-Go), you would have the option to switch to a different payment tier, up and over the free. Creating the registration via the portal, you won't have the ability to change.

Other than that, there are other options and settings that you can get to (setting up OAuth config settings, etc.)

Thanks for your response.
I m still not clear. I don't have azure subscription ,I want to go with free bot registration()  for the production environment. Actually i am using direct line\webchat  channel for my bot . I am trying to understand whether the application in prod environment will be stable or not ,since we couldn't face any issues in the development environment.

Thanks for your response. I m still not clear. I don't have azure subscription ,I want to go with free bot registration() for the production environment. Actually i am using direct line\webchat channel for my bot . I am trying to understand whether the application in prod environment will be stable or not ,since we couldn't face any issues in the development environment.


Propagate context for processes in tasks and other fixes

All changes are documented in their respective commits, please look there for more detailed explanation.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: ed1d8a6d022f7f7490092a5e711b60b5a4bcdfba

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: antonsatin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [antonsatin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Question: How to get named graph from Full Text search query

Hi
I'm able to use full text query with Lucene (LuceneIndex) but can't figure out how to get the named graph of the subject. Is it also indexed with Lucene.
I've tried to modify the query as follows, to include ?graph but it doesn't show up in the results. Only ?subj and ?text can be found in the results
 val qry = "PREFIX search: <http://www.openrdf.org/contrib/lucenesail#> " +
      "SELECT ?subj ?graph ?text " +
      "WHERE { GRAPH ?graph { ?subj search:matches [" +
      " search:query ?term ; " +
      " search:snippet ?text ] }} ";

Thanks a lot

Could you post your question on the support forum?
https://groups.google.com/forum/#!forum/rdf4j-users
Please close this issue when you have done that.

Thanks, will do


using the IncludeGraph?

I have a list of object with a complex multi child / grand child / etc
I currently do a context.AddRange(topParent) then context.SaveChanges() with EF Core 3.1 which work nicely
everything get inserted, identity column get their values and everything is fine ... except it's slow like hell
(parent, childs, grand childs, etc)
so I'm trying this nuget to see if it can solve the slowness issue.
question, based on the readme, this should replace what EF is doing right?
context.BulkInsert(topParent, b => b.IncludeGraph = true);
If I replace both addrange/savechanges with this line, I'm currently getting a stack overflow exception without any other kind of error message or indication of what is going on.
is there anything that I am missing? (i'm sure i am...)

I download the branch for 3.1 and I tried with the source code instead of nuget, more informations;
it crash in file GraphUtil.cs
line 136
the dictionary entityGrahMap contain 465 entries and the collection result contain 40 items
the loop at line 132 contain 19 items (entityNavigations)
callstack, it seem it loop between line 125 -> 145 -> 164

I downloaded this today and also enabled the Graph (i had other errors with the SetOutputIdentity: false, setting) as I need the identity key returned. It seems to work for a small number of entities, but anything large (but not massive) seems to have this exception. Ironic since it is for bulk inserts :) Did you try any previous version @Spirch ?

IncludeGraph feature was PR, @maitlandmarshall could you take a look?
Also linked issue has similar error, having second level child A -> B -> C.

I'll take a look.

@Spirch IncludeGraph was originally designed for a one-way object graph. It fails when you are attempting to include a graph which has recursive navigation properties. I use it to bulk save nested JSON schemas.
In order to get this working in its current state, you must iterate through your entity graph and ensure that the inverse navigation properties are not set.
For example, let's say we have a list of Assets:
class Asset {
    int Id;
    int? ParentId;

    Asset Parent;
    ICollection<Asset> Children;
}

Iterate through this list and ensure Asset.Parent is set but Asset.Children is not set (or vice versa).
I've created a branch here which should support recursive / circular schemas, could you please give it a go?
https://github.com/maitlandmarshall/EFCore.BulkExtensions/tree/issue-520
If it still doesn't work can you provide an example of your schema please?

@maitlandmarshall i was able to see your message before going to bed so i quickly tried it. i had to adapt the changes for EF core 3.1 (it was easy) and now it doesnt crash and work as expected
i need to do more testing when i go back to work in about 6-7 hours
i'm testing over a vpn so right now there is a lot of talk between my laptop and the server, about 800 command send and because of that these 2 conditions, it's 2x slower than the basic EF core solution
i will try without the vpn and see if it's faster or not
do you know if

all the create table (temp tables)
create output table (temp tables)
alter table column on the output temp table (why it is doing that?)

could be send in one shot and then the data could be sent for the bulk insert
then the cleanup, all the drop table sent in one shot
that would reduce the back-forward between the client and sql server, currently I see 7 commands per table

@maitlandmarshall
Hi, here is my example
class A{     int Id;     ICollection<B> Child; }
class B{     int Id;     int ParentId;     ICollection<C> GrandChild; }
class C{     int Id;     int ParentId; }
Can you help to see if this issue can be fixed?thanks.
https://github.com/borisdj/EFCore.BulkExtensions/issues/522

good morning (for me)
first, thanks for the quick response!
data inserted seem to / look to be ok now
i just retested, still over the vpn, in my case the graph is 50 tables for 350 sql command

50 create table
50 create output table
50 alter table
50 merge
50 select
50 drop table
50 drop output table

for option 1-2-3, this maybe could be done in a single command
6-7 also in one command
which would drop the total to 102 command, i don't think 4 and 5 can be reduced
if you don't have time implement it, i can take a peek at the logic behind this and see what i can do but i'm not familiar with the codes so a little bit of guidance would be appreciated

@xiafeicn try the pull request linked

for the step 5. 50 select
i assume this is only to update the identity for later insert, so instead of a select * from the output table, select primarykey should be good enough and faster, this could be a lot of data being transferred over the wire for a single id

@Spirch appreciate you checking out that branch for me. Glad to know it's working for you.
Seems like an interesting problem to tackle, optimizing the amount of calls. I haven't had much to-do with the SqlServerAdapter.cs but there's definitely room for improvement in there.
I'll take a look over the next week and see what I can do. Can you do me a favour and raise another issue for this? These potential changes will affect all features not just IncludeGraph and I would like to keep them distinct.

Reducing Db calls when doing multiple bulks that use temp tables is something that in principle could be optimized.
However implementing it could be little complex.
Also it's worth checking how much time would be gained.
@Spirch how many rows, what order of magnitude ?, do those yours 50 table have, if it's one procedure do do they all have big list or some are smaller ?
Currently am working on BulkSaveChanges functionality so will consider this optimization for that method as well.

@borisdj
sorry for the long text, i started small and i finished with maybe too much information? anyway, here it is;
the process is; i'm receiving a json file, parsing it into tables (50) and then i need to insert the file and the structure into the database. right now there is some talk about using the same process to do bulk update but that is for another year or two if it get agreed/approved.
this is for a web application, right now we are evaluating live result vs later and result by email so speed is very very important.
the json file contain a list of what we call "report", one report populate these tables, the json file could potentially have 500-600 reports in it.
one report can range from 100kb to 300kb (minified json)
if there is 1 to 5 reports, normal entity process context.addrange(reports) vs context.bulkinsert(reports) look kind of identical but when it reach a high amount of report the context.bulkinsert(reports) currently win by a lot (when not on vpn)
I finally tried without the vpn and it was day and night, the problem was the chattyness and the identity retrieval that was bottle necking context.bulkinsert(reports) (see issue #525 and #526)
latency of external, over vpn, connection is for #525 it take a lot of time to open and initiate the connection, over 75ms sometime, so if you have 7 query that is about 0.5sec alone per table so you can imagine in my case that is close to 25 seconds alone.
size of data, #526, is related to the fact that the file get uploaded in a table (kind of archive and spot check verification with xpath since sql server now support it) and I need that identity column, for that specific task right now I'm using the basic savechanges of EF since it wont retrieve the file again to get the id
each table can have between 7 and 20 columns, some table doesn't really need bulk operation (less than 100 rows) but some really need it (over easily 10000 rows)
so best case scenario, that could maybe too complex to do, is an hybrid mode where the bulk insert (in your case bulksavechanges) would be intelligent enough to detect how many rows are in the table on the client side, if it's a low amount, use EF savechanges else use sqlbulkcopy, using sqlbulkcopy for let say 5 rows might be slower than using EF savechanges
but maybe these 5 rows have big varbinary and using sqlbulkcopy could be a better choice, just to add more complexity!
some stats for fun: EF vs Bulk
1 report, EF win, no question
3 reports, identical result or i would say EF is a little bit faster because of batch queries being sent
30 reports, bulk is about 15-20x faster

So a mix of all options as suspected. Will take it into the consideration when implementing BulkSaveChanges.

@maitlandmarshall
I have tested from master branch and seems that this issue  #522  have been fixed. But the latest version from NuGet still not fix. When will this package in nuget will be updated to the latest version?thanks.

DbContext in my project is based on the HTTP request lifecycle. If I use Bulkinsert first and then use await DbContext.SaveChangesAsync()  to persiste the data  in other repository, I will get an error because some entity tracing is already being handled in Bulkinsert. DbContext saves an entity separately and returns an error as below.
The property 'xxx.Id' is part of a key and so cannot be modified or marked as modified. To change the principal of an existing entity with an identifying foreign key, first delete the dependent and invoke 'SaveChanges', and then associate the dependent with the new principal.
It will be fixed if I add DbContext.ChangeTracker.AutoDetectChangesEnabled = false; before  await DbContext.SaveChangesAsync() .Is there a better way to resolve this problem?


Only Displays a White Screen

I have been able to get everything running from DSLR. However, I cannot get the Questionnaires to load after clicking the link. I am redirected to:
http://localhost:3005/launch?launch=4b62a12d-6462-407f-bbdf-ad3c953b3eb3&iss=http://localhost:8080/test-ehr/r4
But it fails to load anything and only displays a blank screen. Any assistance is appreciated. Thanks.

When inspecting the Network traffic, it seems to be failing at this request:
localhost:8080/test-ehr/_auth/0573af38-9f37-429b-a818-cb59942a2b96?redirect_uri=http://localhost:3005/index?state=19245489&session_state=ca935c1b-c38c-4be7-a91f-95fface2ee85&code=95434d12-f21e-4e98-b7e4-cca866521e8a.ca935c1b-c38c-4be7-a91f-95fface2ee85.5dd19d73-8f92-46a1-8751-c5aa3b5b5285
I am being redirected as mentioned above, but no Questionnaire is loading after a substantial wait. Any assistance is greatly appreciated. Thanks.

Not entirely able to recreate the issue with the given information.  I assume you are generally following the guide at https://github.com/HL7-DaVinci/CRD/blob/master/SetupGuideForMacOS.md so a few things to check might be:

All required servers including Keycloak are running
A VSAC_API_KEY environment variable has been set before running the CRD server

Please reopen the issue if needed.

Hi Brandon,
We eventually got past some of the issues by modifying the Java itself (I believe we also modified the EHR, in regards to the redirect).
However, I have tried to enable my VSAC API Key for use in the CRD Service, and for some reason it does not seem to get applied. I followed the instructions for enabling it within the same instructions file you are referring to.
Any insight would be appreciated. Thanks.
Sincerely,
Kenneth M. Crawford | Sr Full Stack Developer | FHIR Engineer | MB-200, PL-400, PL-200, PL-100, AI-100
P:<PHONE_NUMBER> | newwave.iohttp://www.newwave.io/
@.https://www.facebook.com/NewWave.Telecom.and.Technologies/
@.https://www.instagram.com/newwave_tech/
@.https://twitter.com/NewWave_Tech
@.https://www.linkedin.com/company/newwave_technologies
From: Brandon Stewart @.>
Date: Tuesday, March 19, 2024 at 4:22 PM
To: HL7-DaVinci/dtr @.>
Cc: Kenneth Crawford @.>, Author @.>
Subject: [EXTERNAL]Re: [HL7-DaVinci/dtr] Only Displays a White Screen (Issue #194)
CAUTION: This email originated from outside your organization. Exercise caution when opening attachments or clicking links, especially from unknown senders.
Not entirely able to recreate the issue with the given information. I assume you are generally following the guide at https://github.com/HL7-DaVinci/CRD/blob/master/SetupGuideForMacOS.md so a few things to check might be:

All required servers including Keycloak are running
A VSAC_API_KEY environment variable has been set before running the CRD server

Please reopen the issue if needed.
—
Reply to this email directly, view it on GitHubhttps://github.com/HL7-DaVinci/dtr/issues/194#issuecomment-2008061511, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARBUGOEPNYSZXB5JLH5GFQ3YZCNAHAVCNFSM6AAAAAA5DJVCQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBYGA3DCNJRGE.
You are receiving this because you authored the thread.Message ID: @.***>
Confidentiality Notice: This message and all attachments are for the sole use of the intended recipient(s) and may contain NewWave Telecom & Technologies confidential and privileged information or otherwise be protected by law. This message is intended for use by the individual or entity to which it is addressed and any unauthorized review, use, disclosure or distribution is prohibited . If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.


Bump Dependencies #1637922200

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1637922200

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


Coprocessor: v2 row format for decode

What have you changed?
a row_slice implementation for v2 decoder.
parts of (https://github.com/pingcap/tidb/blob/master/docs/design/2018-07-19-row-format.md) codec
the encoder part of this PR is only for test
What is the type of the changes?
New feature
How is the PR tested?
Unit test
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
2018-07-19-row-format.md
https://github.com/tikv/tikv/pull/5497
Benchmark result if necessary (optional)
on MBP (2.7 GHz Quad-Core Intel Core i7)
benches::bench_from_bytes_big                             ... bench:          39 ns/iter (+/- 2)
benches::bench_search_in_non_null_ids                     ... bench:          28 ns/iter (+/- 2)
benches::bench_search_in_non_null_ids_big                 ... bench:          48 ns/iter (+/- 1)
benches::bench_search_in_non_null_ids_middle              ... bench:          32 ns/iter (+/- 1)
benches::bench_search_in_null_ids_middle                  ... bench:          31 ns/iter (+/- 7)

BTW, Thank you a lot! :smile:

/merge


Finish warning cleanup for clang LLVM

I cleaned up a lot of warnings in this pull request, but I ran out of time/steam to do them all. In the end, I set it to /W4 /Wpedantic /Wextra instead of /Wall /Wpedantic /Wextra.
This issue is to get back and do the rest of the warning cleanup to be able to use /Wall for clang/LLVM in CMakeLists.txt instead of /W4.
Primarily it's -Wold-style-cast and -Wsign-conversion warnings.

Fixed almost everything in this commit. One open issue due to some weird code that I need to investigate.


Cannot uninstall/reinstall (Error: 0x80080005)

I tried to install bash but it hung during the setup (for hours) so I closed it. When I open bash now it's unresponsive. I sought to uninstall and reinstall from an elevated command prompt using "lxrun.exe /uninstall /full" but I keep getting simply "Error: 0x80080005". I am on Windows Version 1607, Build 14393.10.
I noticed that this thread also covers error 0x80080005, but the situation seemed very specific, the person with the error seems to have a level of familiarity with BOW that I don't, and I'm wondering if there is a more generally applicable fix to this issue.

Are you using Avast or Kaspersky?

Hmm, interesting. I use neither, but I do have AVG.

I heard there may be problems with AVG as well.  First try disabling, then closing, then uninstalling (reboot after uninstalling) and see if you can get it working.  I think that the antivirus software is blocking pico processes/lxss.sys in order to prevent things like rootkits from installing with kernel mode drivers (total guess!!!).

Thank you, this helped. I had to uninstall AVG and then I could uninstall and reinstall.
However, now I have another problem. The feature downloads from the Windows store, but then is stuck on "Extracting filesystem, this will take a few minutes..." for HOURS.
Anyone encountered this before?

Someone said he got that but when he rebooted after uninstalling avast, it just went through

Thank you for the help. I've tried it a few times, and it's not going through. I'll keep poking around for solutions.

I have no 3d party software on my PC and have the same error Error: 0x80080005 during uninstalling and in Bash there is  just black screen

Facing same issue now, anyone found any good solution?

I'm experiencing the same issue. An uninstallation has become necessary (?) after a mv operation that took very very long and has been aborted by me. After that, the black screen with a blinking cursor appeared. Bash essentially seemed broken. Then I uninstalled the subsystem with LxRun.exe /uninstall /full and installed it again with LxRun.exe /install. Both operations (uninstall and install) yielded the same error 0x80080005.

@netbrothers-tr dont use 'mv xx/ yy/', use 'mv xx yy', it will be fine

Thanks @luo4neck ! I will try that. Meanwhile I have deinstalled the subsystem completely via "Remove Windows features" and then reinstalled it. It's usable again. 😉

I was all set to uninstall and reinstall the subsystem.  Then, when I saw:
C:\windows\system32>lxrun.exe /uninstall /full
This will uninstall Ubuntu on Windows.
This will remove the Ubuntu environment as well as any modifications, new applications, and user data.
Type "y" to continue: n
Accepting is required to proceed. Exiting...
C:\windows\system32>
I decided to stop and ask if there is another way to recover.  I've invested quite a few hours in this, and I would rather not  blow it all away if possible.  As it happens, all of the application software I've developed is under /mnt/c/Users/$USERNAME/ so that's not a problem.  The public keys in ~/.ssh are a bit of a problem, surmountable, of course, but still a problem.  But the applications: getting both versions of python to not fight with each other, all of the optional applications I've installed with pip and apt-get, getting the .bashrc file just right.
Is there another alternative to just blowing all that away?

@jeffsilverm - if you do not supply the /full flag none of the files in your /home or /root directories will be deleted (this includes ~/.ssh and ~/.bashrc). The applications that you've installed via apt and pip unfortunately are stored outside of those locations so those will have to be reinstalled.

Ben,
You used a double negative.  Are you telling me that if I do not use the /full switch, then none of the files in /home or /root will be deleted?
Jeff

/full deletes them. Omitting them makes it not do so.

@jeffsilverm - correct.

In my case, any mv or cp command was blocked by the windows firewall. When I cancel one of this command, the problem with the uninstall happens. Try to disable the firewall, reboot the windows and run the uninstall. That work with me.

@luo4neck I think I'm experiencing the same problem that you described: dont use 'mv xx/ yy/', use 'mv xx yy', it will be fine.  It hangs with the error message: Error: 0x80080005. How can I undo the mv command? I killed the powershell/bash window, and it did not perform the mv operation, but not I'm hosed.

@luo4neck nevermind. Uninstalling and reinstalling windows subsystem for linux fixed it. thanks.

I have the same issue with my work laptop. I have to guess it's one of the many, many security measures and the corporate network.

@benhillis
Hello there, I came here because I just had the "Error: 0x80080005" on bash, and from what I've read on many threads when I was looking for a solution you were still looking into this.
I tried to unistall "full" but it would not let me reinstall anyway, the same message would appear.
At the end of the day, all I had to do was:
"Disable Antivirus permanently & Restart (You can enable it after the installation is complete)
Open Command Prompt or Windows PowerShell as admin
Uninstall WSL by running the following command lxrun /uninstall /full
Now stop WSL service - sc stop lxssmanager
Remove App data folders - rmdir /S %LOCALAPPDATA%\lxss. Confirm the folders have been deleted. You might need to change the ownership to delete the folder if you get any access-denied errors.
Once the folders are deleted, try installing WSL again by running lxrun /install /y"
and it worked.
So there's nothing extraordinary in this solution, but at least it worked where the command line did not, I don't know if that's of any use to you, maybe one of these easy step can help pinpoint the issue.

I had an error the other day because something pinned the service.  After rebooting, I was able to restart it.

can any one help me I am trying to uninstall the linux sub-sytem on windows10
C:\Users\Nikhil>lxrun uninstall/full
Performs administrative operations on the LX subsystem
Usage:
/install - Installs the subsystem
Optional arguments:
/y - Do not prompt user to accept or create a subsystem user
/uninstall - Uninstalls the subsystem
Optional arguments:
/full - Perform a full uninstall
/y - Do not prompt user to confirm
/setdefaultuser - Configures the subsystem user that bash will be launched as. If the user does not exist it will be created.
Optional arguments:
username - Supply the username
/y - If username is supplied, do not prompt to create a password
/update - Updates the subsystem's package index
C:\Users\Nikhil>lxrun uninstall/y
Performs administrative operations on the LX subsystem
Usage:
/install - Installs the subsystem
Optional arguments:
/y - Do not prompt user to accept or create a subsystem user
/uninstall - Uninstalls the subsystem
Optional arguments:
/full - Perform a full uninstall
/y - Do not prompt user to confirm
/setdefaultuser - Configures the subsystem user that bash will be launched as. If the user does not exist it will be created.
Optional arguments:
username - Supply the username
/y - If username is supplied, do not prompt to create a password
/update - Updates the subsystem's package index
C:\Users\Nikhil>


🚀 - Add built-in dark mode support to Root

Description
I have an idea based on this stackblitz, I will add built-in support, disabled by default, and in v5 we can enable it by default:
https://stackblitz.com/edit/angular-skozuk

Linked PR added easy support, not sure if we should enable it by default, we need to think about before v5


Added the "for" param to the conditions as well

Not sure about the time.sleep in the tests, thought that it was the most correct way to test the time condition but it might slow down the tests? Is there a better way to do it?

Otherwise looks great to me - thanks for doing this!

I think this is ready to be merged if you guys don't have any other comments!

You're missing a test for an if-action with an invalid for config: https://coveralls.io/builds/5146696/source?filename=homeassistant%2Fcomponents%2Fautomation%2Fstate.py#L116


Package react-native-push-notification is no longer maintained.

This implementation is no longer working in our production app. Not sure what to recommend, but with the above package no longer being maintained, there is no longer a documented way to reliably implement cross platform react-native push notifications using Azure notification hubs.  Notifee seems to be the recommended replacement for the expired package, but it only supports notifications through FCM, and ANH doesn't seem to support iOS notifications for FCM.  If there's a way to use ANH without the dead package, I'd love to see this doc updated with that info.  Right now I'm considering replacing ANH with FCM, but that would be a ton of work both client and server side.

@ThugLifeAquatic never mind, I see that you filed another issue with the information. Thanks!


[Backport 1.5.latest] upgrade macos version (#10974)

backport https://github.com/dbt-labs/dbt-core/pull/10974
CI only, no release needed

Failing artifact check is not relevant on this branch


Minecraft version 1.8.9 fails to launch. Other versions work fine

I've been having this issue whilst trying to launch Minecraft 1.8.9 from OLauncher. I've tried version 1.12 on OLauncher for it to launch successfully yet I cannot get 1.8.9 to launch.
The game will launch for a split second then will crash showing the following error in the log file:
Exception in thread "main" com.google.gson.JsonSyntaxException: com.google.gson.stream.MalformedJsonException: Unterminated array at line 1 column 575 	at com.google.gson.internal.Streams.parse(Streams.java:56) 	at com.google.gson.TreeTypeAdapter.read(TreeTypeAdapter.java:54) 	at com.google.gson.Gson.fromJson(Gson.java:803) 	at com.google.gson.Gson.fromJson(Gson.java:768) 	at com.google.gson.Gson.fromJson(Gson.java:717) 	at com.google.gson.Gson.fromJson(Gson.java:689) 	at net.minecraft.client.main.Main.main(SourceFile:90) Caused by: com.google.gson.stream.MalformedJsonException: Unterminated array at line 1 column 575 	at com.google.gson.stream.JsonReader.syntaxError(JsonReader.java:1505) 	at com.google.gson.stream.JsonReader.doPeek(JsonReader.java:465) 	at com.google.gson.stream.JsonReader.hasNext(JsonReader.java:403) 	at com.google.gson.internal.bind.TypeAdapters$25.read(TypeAdapters.java:658) 	at com.google.gson.internal.bind.TypeAdapters$25.read(TypeAdapters.java:667) 	at com.google.gson.internal.bind.TypeAdapters$25.read(TypeAdapters.java:642) 	at com.google.gson.internal.Streams.parse(Streams.java:44) 	... 6 more
I've tried launching 1.8.9 on several different computers all with different Java versions all to come to the same conclusion.
Looking at the error log it seems that it is trying to contact a server at Google to get a JSON file and it is coming back corrupted or the server doesn't exist anymore.
Here is a video of what happens:
Video

Sorry for getting back to you so late. I believe I have seen this error before, and it relates to either Java or Windows truncating the command line for Minecraft (which is very long). It only happens on Windows from what I can tell.
I will be out of the house for today, but tonight I will try and investigate a solution.

(probably) fixed in 0965d8e6e3c03f4e1eb65adfd95df9ff4b4abcd0


JEI Bookmarks rendering underneath NEC+ GUI

presently, the Forge (1.16) build of NEC+ doesn't properly have zones set for JEI item wrapping, meaning bookmarked items will render behind the NEC+ GUI.

this functions fine with the Fabric build and the 1.12 Forge build.

#12
I really feel like this is a problem with the incredibly janky 1.16 JEI, but I'll check this as I mentioned in #12; just haven't gotten around to it.


Reply to OOC with OOC

Issue
When using the reply button on an OOC message, the console should be populated with an OOC response.
See discussion on Wolfery Forum:
https://forum.wolfery.com/t/suggestion-reply-to-ooc-with-ooc/2235

Resolved in PR #141


[PE-D][Tester A] Incorrect example for add command on both UG and product

Labels: severity.Medium type.DocumentationBug
original: Song-Mengfei/ped#3

This has been addressed.


Verilator bareMetal test VCD is incomplete in CEP v4.5 and v4.7

I was able to de-cipher (perhaps?) through looking to the sims/verilator/Makefile how to create a VCD when running Verilator in v4.5 going forward.  The documentation as far as can tell has never been updated from CEP 4.4 where one can just add -v to the command line without having to compile with "debug", but in 4.5 going forward apparently one has to compile with "debug".  The problem is the VCD appears to just stop after a short time when running a bareMetal test and is therefore not showing the entire timespan of the sim.
This is how I compiled the Verilated code:
make SUB_PROJECT=cep_verilator debug
Then I ran this:
./simulator-chipyard.harness-CEPVerilatorRocketConfig-debug -vcdfile=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd ../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.elf +loadmem=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.hex +loadmem_addr=80000000
cep_playback: playback command sequence totalCmds=212
cep_playback: playback command sequence totalCmds=212
Test Passed
[UART] UART0 is here (stdin/stdout).
In the Makefile I see +vcdfile but in my invocation I have to user -vcdfile=.. otherwise I get an invalid parameter message. Why is it -vcdfile instead of +vcdfile?
Here is the size of the VCD is this in v4.7 which is incomplete:
-rw-rw-r-- 1 jkrim jkrim 6893742 Sep 23 10:31 ./cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd
In CEP v4.4 a full simulation VCD that is working with the correct size is:
-rw-rw-r-- 1 jkrim jkrim<PHONE_NUMBER> Sep 23 10:40 ../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd
Any idea why is it being "truncated" (or stopping short of the simulation end time?) in V4.7?

It appears that the order of the plusargs makes a difference.  I don't know why this is the case.  The VCD one has to be last.
This works in both CEP v4.5 and v4.7 vs. what I showed previously:
./simulator-chipyard.harness-CEPVerilatorRocketConfig-debug  ../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.elf +loadmem=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.hex +loadmem_addr=80000000 +vcdfile=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd
But, there is still a problem where CEP v4.5 doesn't print any of the "normal" cep_playback messages but creates the proper sized VCD as this:
./simulator-chipyard.harness-CEPVerilatorRocketConfig-debug ../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.elf +loadmem=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.hex +loadmem_addr=80000000 +vcdfile=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd
[UART] UART0 is here (stdin/stdout).

/mnt/data/projects/avalon/jkrim/cep_v4_5_test_perl3/CEP/sims/verilator/generated-src/chipyard.harness.TestHarness.CEPVerilatorRocketConfig/gen-collateral/TestDriver.v:158: Verilog $finish

But in CEP v4.7 this is printed to the screen:
./simulator-chipyard.harness-CEPVerilatorRocketConfig-debug ../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.elf +loadmem=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/riscv_wrapper.hex +loadmem_addr=80000000 +vcdfile=../../sims/cep_cosim/testSuites/bareMetalTests/macro4Mix/macro4Mix.vcd
cep_playback: playback command sequence totalCmds=212
cep_playback: playback command sequence totalCmds=212
Test Passed
[UART] UART0 is here (stdin/stdout).
Any idea why CEP 4.5 prints are gone with debug (to dump a VCD)?

OK.  The prints are not gone.  They disappear if the last thing done was run the Xcelium bareMetal tests.  If one wants to back to the Verilator tests one has to re-compile all the baremetal tests for Verilator (which uses libgloss) and then the prints will work.


The CPU is running too slowly in large v3

Does it support running under GPU? The CPU is running too slowly in large v3

Yes, it supports different GPU runtimes, check:
https://github.com/sandrohanea/whisper.net/tree/main/examples/NvidiaCuda for NVidia GPU
https://github.com/sandrohanea/whisper.net/tree/main/examples/OpenVinoExample
https://github.com/sandrohanea/whisper.net/tree/main/examples/Vulkan
Make sure you install all the drivers and the runtimes for each platform if Vulkan or OpenVino is used: https://github.com/sandrohanea/whisper.net/tree/main?tab=readme-ov-file#gpu-support


Java based migrations

Does it support Java based migrations just like flyway?

No, but it might be a feature worth investigating.
Cheers
Patrick

Hi @patka,
any plans for this? :)
Thanks!

Hi @SimonPNorra,
sorry for the delayed response. Yes, I still would like to do this but currently my time is very limited and I have some other priorities first. My next step is to provide more hooks for integration so that custom code can be added more easily. That should then also make the way for something like this to be added more easily.
Cheers
Patrick


Please remove YY from Organizations using Saturn

I'm the platform architect of YY. Afaik, we are not using Saturn either in production or in testing. Actually we have a similar task scheduling system (not open sourced yet) based on Kubernetes.
So, please remove it in the list, thanks.

Hi Homer, I removed YY from the list.


Let ore & tree generators require a specific world name

Currently, all ore generators and tree generators work in all worlds. It would be nice if they would optionally require a specific world name, and only generate objects in worlds with that name.

This was implemented in KCI 12


mirror, CORS policy, and append headers rule proto support

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
adds support for new fields in the route rule proto (append_headers, cors_policy, mirror).
Special notes for your reviewer:
Release note:

NONE

@GregHanson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
2b822a4820890309191fe8b57d01bfd546dd0db6
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mattklein123 is it okay to have the cors filter always on but selectively supply the cors settings for one or more routes?

@GregHanson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/auto-merge_master.sh
18308da4134c70d82f005aa63e72bc8e808b1ac0
link
/test auto-merge_master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test auto-merge_master

@GregHanson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/auto-merge_master.sh
18308da4134c70d82f005aa63e72bc8e808b1ac0
link
/test auto-merge_master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[Fix] Fix type hinting for OrderedDict (python 3.8)

We ran into this error because we also import OrderedDict from collections?

More specifically, because in python3.8, the OrderedDict from collections cannot be subscripted for use as type-hinting (it could probably be used without subscripting).


Missing LICENSE file in npm package

Hello, is it possible to incorporate the LICENSE file inside the npm package and this repository ? Now it's missing. By definition your code is not proprietary and cannot be used. Can this be remedied please ?
Thank you

ok, it will be fixed in the next publish


Consider marking any definition that uses an image from DockerHub as not compatible with Codespaces

Docker has changed their terms of service for DockerHub to only allow 100 pulls per anonymous IP. Unfortunately, cloud services generally can't guarantee that an IP in use is going to have any pulls (along with many corporate environments for that matter). While this can be resolved with a sign-in (where the limit becomes 200 pulls per user or unlimited with paid), forcing a sign-in to DockerHub for someone getting started is not a good user experience. Unfortunately, this means that we would likely not want DockerHub-based definitions to be explicitly recommended by the product in the Codespaces context as a result. ☹
We could mark these as "No" for works in Codespaces and add a README note explaining how to get them up and running. This would filter them out of the recommender and in-product UX for Codespaces specifically, but not Remote - Containers.
(This is a bit less likely to be a problem in Remote - Containers since Docker encourages sign in as a part of things like Docker Desktop setup. So there's no reason to outright exclude these in the desktop context. Also, any image originating from mcr.microsoft.com or another public image registry like GitHub Container Registry shouldn't have this problem as it is not subject to DockerHub limits - so things starting from a pre-built image are fine.)
If DockerHub updates their terms or another solution is determined that does not force a user sign in, we could reconsider.

Closing for now as we think we might have a workaround.


DALL-E-2 Edit   demo  does not work

I am trying the  demonstration code (as below)  for DALL-E-2 Edit
as given in https://platform.openai.com/docs/guides/images/usage?context=node
I am on python3.8 windows10.
The input image and mask files are as provided, but the output file does not contain any edit.
from openai import OpenAI
client = OpenAI()
response = client.images.edit((
model="dall-e-2",
image=open("sunlit_lounge.png", "rb"),
mask=open("mask.png", "rb"),
prompt="A sunlit indoor lounge area with a pool containing a flamingo",
n=1,
size="1024x1024"
)

can show the error you got?

there is no error as such, the unmodified image is returned.
On Sun, 24 Dec 2023 at 12:12, ayush rajgor @.***> wrote:

can show the error you got?
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-cookbook/issues/951#issuecomment-1868502465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADYVMFJF4WZYZGUM7U4UTK3YLAL4VAVCNFSM6AAAAABBACKBOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGUYDENBWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Nick "Mac" McElwaine

Hello, actually there is also an issue with extra ( in function usage. client.images.edit((
https://github.com/openai/openai-openapi/issues/154#issue-2068663624

Ok, thanks for the clarification.
On Sat, 6 Jan 2024 at 14:41, Igor Udot @.***> wrote:

Hello, actually there is also an issue with extra ( in function usage.
client.images.edit((
openai/openai-openapi#154 (comment)
https://github.com/openai/openai-openapi/issues/154#issue-2068663624
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-cookbook/issues/951#issuecomment-1879716359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADYVMFJDQJKUQ7N6TTRETITYNFPBFAVCNFSM6AAAAABBACKBOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZG4YTMMZVHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Nick "Mac" McElwaine

I  still do not have a resolution  for this.

sdshmkh ha mentioned the issue which is still outstanding


Update LKG

Fixes #17500
Our services layer already depends on something (generic inference improvement related) from our latest changes. In jake, services get built with built/local because the command line compiler is built first and used to build other things. In gulp however, services get built first since we hand that off to gulp-typescript.

What is watchGuard and why is it here?

@DanielRosenwasser #14098


Issue when building / serving -  TypeError: Cannot read property 'nodes' of undefined

Hi,
Thanks for building this - it looks great! I'm trying to use it, and it works fine in development mode. However when I try and build & serve (gatsby build, gatsby serve), or host it using netlify, I get the following error:
 TypeError: Cannot read property 'nodes' of undefined
I'm using the Gallery Component in the default folder. I was wondering if this is a known issue?
Thanks

@dhruuuuuv what version of gatsby are you using? Any specific steps to reproduce?

Hi @epilande,
Thanks for getting back to me. I'm using gatsby version 2.15.20
In terms of reproduction, I'm just using the Gallery component, with the default folder location (content/gallery). When I run this in dev mode everything appears fine, but when serving the built site, I get an error when navigating to the page on which the gallery component is used in.
Hope that helps.

Hi!
I am using the theme gallery as a component to a Stackbit Vanilla site.
I created the component, but Gatsby wont build anymore and I don't know why.
Here is an image of the build log:

Hope this helps to improve the plugin ;)
Jairo


feat: add paraswap contracts to more chains

Based on: https://developers.paraswap.network/smart-contracts

Added addresses on-chain.


[Command Palette] Full text search implementation in an app

Description
I am fairly new to web development and I’m trying to add a command palette that contains all the available routes of the app inspired by the nuxt-ui palette itself.
I stumbled upon the Full-text search in the documentation and from my basic understanding, the “:fuse” prop looks for a “title” and a “description” on routes dynamically.
I used “useHead()” on some pages  adding “title” and “description” just to try it out but the palette itself is always empty.
Am I totally missing the point of the feature or am I missing something crucial?
I know I could display the routes by manually adding them to the component as mentioned in the beginning of the documentation, but at this point my interest is peaking and want to figure what  is the Full-text search actually supposed to do.
Appreciate any feedback.

Not sure to understand here, the Full-text search section on the documentation only describes how to customize Fuse behaviour. You still have to pass some groups.

I see now, appreciate the quick response