Mailrise service fails to start

Here is the failure I'm getting:
● mailrise.service - Mailrise SMTP notification relay
Loaded: loaded (/etc/systemd/system/mailrise.service; disabled; vendor preset: disabled)
Active: failed (Result: exit-code) since Thu 2023-03-30 11:32:49 CDT; 2min 5s ago
Process: 73661 ExecStart=/usr/local/bin/mailrise /etc/mailrise.conf (code=exited, status=1/FAILURE)
Main PID: 73661 (code=exited, status=1/FAILURE)
Mar 30 11:32:49 bde1.bluedragon.local systemd[1]: Started Mailrise SMTP notification relay.
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]: Traceback (most recent call last):
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]:   File "/usr/local/bin/mailrise", line 7, in
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]:     from mailrise.skeleton import run
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]:   File "/usr/local/lib/python3.6/site-packages/mailrise/skeleton.py", line 5
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]:     from future import annotations
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]:     ^
Mar 30 11:32:49 bde1.bluedragon.local mailrise[73661]: SyntaxError: future feature annotations is not defined
Mar 30 11:32:49 bde1.bluedragon.local systemd[1]: mailrise.service: Main process exited, code=exited, status=1/FAILURE
Mar 30 11:32:49 bde1.bluedragon.local systemd[1]: mailrise.service: Failed with result 'exit-code'.

The minimum Python version for Mailrise is 3.8 or above. Python 3.6 has been out of support since the end of 2021.


Add initial changes for Composites in Siddhi level

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Add initial changes for Composites

Documentation

Add initial changes for Composites

Training

LAdd initial changes for Composites

Certification

N/A

Marketing

N/A

Automation tests

Unit tests

N/A

Integration tests

N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

N/A

Test environment

Cellery 0.4.0-SNAPSHOT local setup

Learning

N/A

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@691ebe9). Click here to learn what that means.
The diff coverage is 22.62%.

@@           Coverage Diff            @@
##             master     #66   +/-   ##
========================================
  Coverage          ?   37.2%
  Complexity        ?     451
========================================
  Files             ?     126
  Lines             ?    5196
  Branches          ?     997
========================================
  Hits              ?    1933
  Misses            ?    2814
  Partials          ?     449

Impacted Files
Coverage Δ
Complexity Δ

...ility.ui/src/components/cells/component/Details.js
3.77% <ø> (ø)
0 <0> (?)

...ui/src/components/cells/cell/CellDependencyView.js
2.85% <ø> (ø)
0 <0> (?)

...y.ui/src/components/cells/component/K8sPodsList.js
1.44% <ø> (ø)
0 <0> (?)

...ervability.ui/src/components/tracing/view/index.js
3.38% <ø> (ø)
0 <0> (?)

...lity.ui/src/components/cells/cell/ComponentList.js
1.38% <ø> (ø)
0 <0> (?)

...ervability.ui/src/components/cells/cell/Metrics.js
2.22% <ø> (ø)
0 <0> (?)

...ervability/api/siddhi/SiddhiStoreQueryManager.java
100% <ø> (ø)
3 <0> (?)

...io/cellery/observability/k8s/client/Constants.java
0% <ø> (ø)
0 <0> (?)

...ervability.ui/src/components/cells/cell/Details.js
3.63% <ø> (ø)
0 <0> (?)

...servability/api/exception/UnexpectedException.java
0% <0%> (ø)
0 <0> (?)

... and 23 more

Continue to review full report at Codecov.

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


blocking doesn't work in youtube.com

It doesn't block the ads in youtube homepage and also the built in ads when loading a video. Also there are cosmetic issues in ads on the sidebar when viewing a video.

currently, no ad blocker is technologically capable of blocking ads in the native YouTube App on mobile devices

What about the Youtube in desktop using browser? my issue is happening in Youtube desktop.
I'm using brave browser with brave shield active, but I see that bebasdns has custom filtering and adblocking feature so I turned off brave shield but the ads are visible instead.

DNS blocking of YouTube ads no longer works, because ads are injected into the stream and appear to originate from YouTube. In other words, if you want to block YouTube ads, it means blocking YouTube itself as well.
Use browser addons such as uBlock Origin instead.


awkward read_ascii behavior

I know that there are many more interesting things to do with Larch than read column ascii files.  That said, I have really been pounding on that particular function this week.
Suppose you have a file like this:
# comment
# comment
# ---------------
# weird_label  weirder_label  all  the  rest  are  ok
  1.0 2.0 3.0 4.0 5.0 6.0 7.0
  1.1 2.1 3.1 4.1 5.1 6.1 7.1
... etc

Doing
larch> a=read_ascii('file', labels="a b")

results in something like
Traceback (most recent calls last):
   File <stdin>, line 1
    a=read_ascii('file', labels="a b")
  ^^^
   File <stdin>, line 1
  File "/usr/share/larch/plugins/io/columnfile.py", line 202, in read_ascii
    _labels.append("col%" % (i+1))

ValueError: incomplete format

which seems like the wrong behavior.
My point is that it seems like specifying fewer than the total number of column labels should default to something like a b col3 col4 etc... or use existing column labels.  That is, specifying a subset of the labels does not seem like it should be user error.

Will fix.
On Jan 4, 2017 9:08 AM, "Bruce Ravel"<EMAIL_ADDRESS>wrote:

I know that there are many more interesting things to do with Larch than
read column ascii files. That said, I have really been pounding on that
particular function this week.
Suppose you have a file like this:
comment
comment
---------------
weird_label  weirder_label  all  the  rest  are  ok
1.0 2.0 3.0 4.0 5.0 6.0 7.0
1.1 2.1 3.1 4.1 5.1 6.1 7.1
... etc
Doing
larch> a=read_ascii('file', labels="a b")
results in something like
Traceback (most recent calls last):
File , line 1
a=read_ascii('file', labels="a b")
^^^
File , line 1
File "/usr/share/larch/plugins/io/columnfile.py", line 202, in read_ascii
_labels.append("col%" % (i+1))
ValueError: incomplete format
which seems like the wrong behavior.
My point is that it seems like specifying fewer than the total number of
column labels should default to something like a b col3 col4 etc... or
use existing column labels. That is, specifying a subset of the labels does
not seem like it should be user error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/xraypy/xraylarch/issues/161, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASke-yqDCX04cnDK_kQSOZqXYD-vx11ks5rO7XZgaJpZM4LatuX
.

Should be fixed in master now, so that it produces 'weird_label', ..., 'ok', 'col8', 'col9', ....

I think this is now resolved....


Bundled Growl and Sparkle frameworks don't have arm64 part

The bundled Growl framework is an x86_64/i386 universal binary and the bundled Sparkle framework is for x86_64 only. This prevents the software from building for arm64.
This was reported to MacPorts here: https://trac.macports.org/ticket/65690

Growl is gone long enough that Hermes should probably not depend on it any more anyway.

So Sparkle supports Apple silicon - https://github.com/sparkle-project/Sparkle/discussions/2598 .
Growl is long gone.
Wasn't there so other component that needed to be updated to ARM64 too?

So Sparkle supports Apple silicon - sparkle-project/Sparkle#2598 .
Growl is long gone.
Wasn't there so other component that needed to be updated to ARM64 too?
https://trac.macports.org/ticket/65690#no1
This person reported no. Ideally it should be updated to new macos notifications instead of growl but I have got it to work without growl. Right now testing a version for x86 macos 11 and 12 and then updating latest sparkle. I don't have an arm mac to test though.

Here is my version for arm and x86 to test https://github.com/dtseto/Hermes-master/releases/tag/prereleasev132

I just downloaded your 1.3.2d1 version, but after unpacking it, it indicated it was for Intel after doing a Get Info, versus a Universal build. I haven't tested it since I have nothing on my system that uses Rosetta 2.


[WIP]Envelope sns message when sending to sqs

According to the doc, message published to SNS is wrapped as json when sending it to SQS queue.
http://docs.aws.amazon.com/sns/latest/dg/SendMessageToSQS.html
Fixing: https://github.com/spulec/moto/issues/1063

Coverage decreased (-0.2%) to 93.317% when pulling 38cba371bbeb453ab65bf8a934ff000ae1b2a9f9 on kawasakitoshiya:envelope_sns2sqs_message into bd98996a19e441963773908fe2d6a4fe3ea5a7bd on spulec:master.

Coverage decreased (-0.1%) to 93.396% when pulling e8a2713c8600b06193d602032d964fdf86ae2fe2 on toshitanian:envelope_sns2sqs_message into bd98996a19e441963773908fe2d6a4fe3ea5a7bd on spulec:master.

Coverage remained the same at 93.515% when pulling 7d8e5d0193293e3dd34656c569373d5f7b89af04 on toshitanian:envelope_sns2sqs_message into bd98996a19e441963773908fe2d6a4fe3ea5a7bd on spulec:master.


#3078 Providing a version for aggregating that is too high should aggregate to null

@erdtsieck I'm not sure I agree with this one

@erdtsieck I'm not sure I agree with this one

@jeremydmiller Well, I do not expect you to. My intent is to be able to do polling whether an aggregate has arrived at an expected point, as @oskardudycz did in his example: https://github.com/oskardudycz/EventSourcing.NetCore/blob/main/Sample/Helpdesk.Wolverine/Helpdesk.Api/Incidents/ResolutionBatch/GettingIncidentResolutionBatch/GetIncidentsBatchResolution.cs
But you could of course just do the aggregate.version check yourself instead of using the API
Or another API method?

There's the FetchStreamStateAsync() API you could use for that that would be much, much lighter weight. Not sure what Oskar was up to there.

I think I'd rather have it throw an exception rather than do that, and I think I disagree w/ that too because it's filtering the events and might not even know what the current stream version is


Add ability to reset a db to execute from genesis

Description
This PR allows us to reset a db to execute from genesis. The main use case of this would be to run an instance from genesis using a db snapshot. Follow the below steps to test:

Get a db snapshot. Either generate one by running stress locally and enabling db checkpoints or download one from S3 bucket (pretty big in size though).
Download the snapshot for the epoch you want to restore to the local disk. You will find one snapshot per epoch in the S3 bucket. We need to place the snapshot in the dir where config is pointing to. If db-config in fullnode.yaml is /opt/sui/db/authorities_db and we want to restore from epoch 10, we want to copy the snapshot to /opt/sui/db/authorities_dblike this:
aws s3 cp s3://myBucket/dir /opt/sui/db/authorities_db/ --recursive —exclude “*” —include “epoch_10*”
Mark downloaded snapshot as live: mv  /opt/sui/db/authorities_db/epoch_10  /opt/sui/db/authorities_db/live
Reset the downloaded db to execute from genesis with: cargo run --package sui-tool -- db-tool --db-path /opt/sui/db/authorities_db/live reset-db
Start the sui full node: cargo run --release --bin sui-node -- --config-path ~/db_checkpoints/fullnode.yaml
A sample fullnode.yaml config would be:

---
db-path:  /opt/sui/db/authorities_db
network-address: /ip4/<IP_ADDRESS>/tcp/8080/http
json-rpc-address: "<IP_ADDRESS>:9000"
websocket-address: "<IP_ADDRESS>:9001"
metrics-address: "<IP_ADDRESS>:9184"
admin-interface-port: 1337
enable-event-processing: true
grpc-load-shed: ~
grpc-concurrency-limit: ~
p2p-config:
  listen-address: "<IP_ADDRESS>:8084"
genesis:
  genesis-file-location:  <path to genesis blob for the network>
authority-store-pruning-config:
  num-latest-epoch-dbs-to-retain: 3
  epoch-db-pruning-period-secs: 3600
  num-epochs-to-retain:<PHONE_NUMBER>3709551615
  max-checkpoints-in-batch: 200
  max-transactions-in-batch: 1000
  use-range-deletion: true

Test Plan
By running it locally

Awesome! Can you move the how-to from the PR description into the code somewhere? the db tool command would be a fine place


x/build/cmd/release: silent uninstall of stable Go by release candidate

I specified c:\go17 directory for RC's installation and Go 1.6.2 from c:\go was removed without confirmation. Please add uninstallation confirmation during release candidate cycle.

/cc @broady @adg @ianlancetaylor

We really need help from someone who know how the MSI builder stuff works.

I specified c:\go17 directory for RC's installation

Go installer assumes (see GOROOT_FINAL) that you will install Go into C:\Go directory. If you install RC into c:\go17, then all programs you build with RC will have references to files under C:\Go. So your stack traces will be wrong, your debugger will be confused.
If you want to install Go anywhere but c:\go you should just build from source.
Alex

@alexbrainman I can't build yet. TestDirWindows and TestEnvOverride fails in net/http/cgi test. Looking into it.

@alexbrainman I can't build yet. TestDirWindows and TestEnvOverride fails in net/http/cgi test. Looking into it.

That was perl. Now GDB dumps core in TestGdbPython and TestGdbBacktrace. Anyway issue was about installer, not about building from sources on windows.

Anyway issue was about installer, not about building from sources on windows.

I understand. But IMHO the installer is for users how are happy with single version of Go installed into c:\go. If you want something different you should build from source.
Installer adds your %GOROOT%\bin directory to the %PATH%. If you use installer to install 2 different versions of Go into 2 different directories, the installer will add both ...\bin directories to the PATH.

TestDirWindows and TestEnvOverride fails in net/http/cgi test.

Feel free to create new issue for it. Tests shouldn't fail just because you don't have PERL installed.

Now GDB dumps core in TestGdbPython and TestGdbBacktrace.

Same.
You don't have to run all tests to "build" your Go installation from source. Just running make.bat should do.
Alex

Feel free to create new issue for it. Tests shouldn't fail just because you don't have PERL installed.
I don't want to bother the devs with secondary platform's build problems. I have ubuntu vm for experiments.

I don't want to bother the devs with secondary platform's build problems.

Are you implying that Windows is a secondary platform? That's definitely not accurate. Windows is a first-class port.

Secondary building platform, not port. Primary platform - where Go is developed in the first place. For example, problem with perl was totally my fault - there was perl.bat on my PATH, but no perl installed. Not worth filling an issue.

I can try to take a look at it or failing that, pass it on to someone in my company who knows WiX extremely well.

@alexbrainman I am pretty sure the installer sets GOROOT, so setting an install path to C:\Go17 should work fine.

It seems there's multiple things being discussed here.  So is the issue still simply that no confirmation dialog is displayed?   That's it right?

@pbennett, I think this about us wanting them to be able to be co-installed (Go 1.6 stable and Go 1.7rc*), with different GOROOTs.
I think we need to use a different Product UUID in the wxl XML manifest thing.

Most companies these days release 'major upgrades' for their product (eg: an new MSI that upgrades a previous MSI installation).  When running the upgrade it will silently remove the previous version of the product behind-the-scenes before installing the new product (or it could install the new product first and remove anything that's orphaned from the previous version at the very end of the upgrade).
https://msdn.microsoft.com/en-us/library/windows/desktop/aa369786(v=vs.85).aspx?f=255&MSPPError=-2147217396
I downloaded 1.6.2 and 1.7 and confirmed the UpgradeCode UUIDs are the same so any upgrade will remove the previous version from the system.
To @bradfitz, the ProductCodes have changed between 1.6.2 and 1.7 (also another factor in developing an install as a major upgrade to a previous product release).  The ProductCode changes, the ProductVersion changes, and the UpgradeCode stays the same.
There's ways to hack this, but you'll break any future upgrade path possibilites.  Not something I'd advise.

@bradfitz Does go really support co-installs?  The OSX install forces a single install, so it at least, implies only one is allowed.  Is GOROOT just set implicitly from the go executables run path if not set explicitly?
Welcome @chadmiles !  (He's one of the install guys where I work - so we're in good hands now for WiX).

I am pretty sure the installer sets GOROOT, so setting an install path to C:\Go17 should work fine.

I am not sure about GOROOT.
But I was concerned about what happens to the PATH. The installer is updating global user PATH variable to include %GOROOT%\bin directory in it. For example, if you tell installer to install your Go into c:\go, you PATH will have:
set PATH=....;c:\go\bin

Do you suggest we install different version of Go into, for example, c:\go17 and add c:\go17\bin to the global PATH? Like:
set PATH=....;c:\go\bin;c:\go17\bin

But then Windows will find go.exe in c:\go\bin, not in c:\go17\bin. How are you going to control which version you want to use?
Same for GOROOT.
Also do not forget that GOROOT_FINAL is set to "c:\go" for a reason here https://github.com/golang/build/blob/master/cmd/release/release.go#L315. All binary *.a files under %GOROOT%\pkg contain full path to the package source files. *.a files will be copied by installer to the user disk as is. When user links these into her executable it will still contain c:\go... regardless of where these are on the disk. So your stack traces and debugging file paths will be different to reality (unless you install Go into c:\go).
Alex

This is something we actually struggle quite a bit with ourselves. We do have to support various versions of our software, not all using the same go version, and basically had to "vendor" go itself, making go a versioned dependency of our projects and wrapping all uses under tools which deal with setting the various environment variables as needed. This goes a bit beyond the initial scope of the discussion, but being able to actually attach projects to a go version (and then map this to an actual go installation) is something I could see a usage for.

@pbennett @chadmiles we're having another unrelated issue related to wix that has some urgency - could you e-mail me at<EMAIL_ADDRESS>please?

@broady what's the status here?

@broady what's the status here?

@broady what's the status here?
Also, @alexbrainman, I think we have fixed the toolchain so that the right paths end up in the final binaries even for alternate install locations. What you say was true for a long time but I think (at least hope) it's no longer true.

Sorry about dropping this. I made contact with someone who knows more about the installer recently. I'll follow up with him to figure out what we should be doing here.

MajorUpgrade looks promising.
It sounds like we should consider changing the Product GUID for each major version. But there's also some separate GUID for upgrades that appears to be used to associate packages.
RemoveExistingProducts will uninstall those associated packages, but it's unclear whether we can prompt the user to say Yes/No to it.
I've requested a Windows machine, will provide more updates once it arrives.

I think we have fixed the toolchain so that the right paths end up in the final binaries even for alternate install locations. What you say was true for a long time but I think (at least hope) it's no longer true.

You are right, I was wrong. Final binaries refer to installed location, not c:\go.
Alex

If you can provide a dialog with a checkbox that is tied to a PROPERTY, I believe you can override the stock conditioning of the "RemoveExistingProducts" CustomAction like below:
<InstallExecuteSequence>
  <RemoveExistingProducts Before="InstallInitialize" Overridable="yes"><![CDATA[USER_CHOSE_YES_IN_DIALOG_CHECKBOX_TO_REP=1]]></RemoveExistingProducts>
</InstallExecuteSequence>

/cc @johnsonj, if this is up your alley.

@broady, this just came up again (dup bug #20679).
Here's a proposal: we stop releasing Windows MSIs for betas. Those users can use the zip files or go get golang.org/x/build/version/go1.9beta1. That also simplifies your code signing step.

@bradfitz but the same problem will then happen for stable as well. I really don't like that I should compile the or download an archive. On Windows / OS X it should be really just what users are used to do, download the MSI, install them and that's it.

I see, I make no promise but I'll try and have a look to see if I can work something out. The MSI stuff is open as well, yes?

See golang.org/x/build/cmd/release for how the MSI is built.

Anybody have time for this for Go 1.9?

There's clearly a path to to disabling the uninstall (as @chadmiles pointed out) but if you install 2 Go's (C:\go17, C:\go19) what should happen when you type go.exe in a new cmd window? Is it enough to prepend the latest install to the PATH?
I propose we prompt the user before uninstalling (example) and if they say no, we exit the installer. No user confusion on why things are uninstalled, no official management of multiple versions. If that sounds good I can do it for Go1.9.

Sounds like an improvement over the current situation.

Is it enough to prepend the latest install to the PATH?

I wouldn't like that done to my PATH. Especially, if I did not clearly understand what you 're doing.
I think that people, who are interested in beta versions, are clever enough to use zip file instead of MSI. Maybe we should push install from zip file more. It is really easy to install Go from zip file - just PATH needs to be changed.

I propose we prompt the user before uninstalling (example) and if they say no, we exit the installer.

SGTM
Alex


Remove Duplicate IP's

Hi (apologies for yet another issue ticket):
I modified my version of spunky bot to do two things and wanted to share them:
The first I sorted out storing ip bans in the bots-banlist in a easier to read form.
!permban section:
# add IP address to bot-banlist.txt
                                ip_address = victim.get_ip_address() + ':-1'
                                with open(os.path.join(HOME, 'bot-banlist.txt'), 'a') as banlist:
                                    banlist.write("%s // %s banned: %s  reason: %s\n" % (ip_address.ljust(20), victim.get_name().ljust(20), time.strftime("%d/%m/%Y (%H:%M)", time.localtime(time.time())), reason))

and then in the !unban section in added a method for removing duplicates since they where never removed from the bots-banlist.txt file after !unban.
# check bot-banlist.txt duplicate IP's
                            symlink = os.path.join(HOME, 'bot-banlist.txt')
                            realfile = os.path.realpath(symlink)
                            duplicate = False
                            count = 0
                            for line in fileinput.input([realfile], inplace=True):
                                if line.strip().startswith(ip_addr):
                                    duplicate = True
                                    count += 1
                                    continue
                                sys.stdout.write(line)
                            if duplicate:
                                self.game.rcon_tell(sar['player_num'], "^2Success!^7 Removed ^3%s^7 duplicate%s." % (count, 's' if count > 1 else ''))
                            else:
                                self.game.rcon_tell(sar['player_num'], "^3No duplicates where found")
                        else:

p.s. i symlink all my bots to one ban list, so needed to use os.path.realpath()
the only way i found to do this was using fileinput module and sys module
perhaps you know a better way of doing this, either way I wanted to try and help anyway I could.

Hi,
all issues are welcome.
The bot-banlist.txt has no function at all for SpunkyBot.
The reason for this list was at a time, when the UAA exists, so that you could share it.
From my point of view, the file and the corresponding code can be deleted. Except there is any reason to have an additional file, since SpunkyBot stores all data in the database and e.g. PRISM is reading the database and nothing else.

Ok I understand - its not needed from spunkybot perspective however it is still good in my view to keep around for purposes like you describe but perhaps not worth adding extra modules for.
I like it because if something goes wrong with spunkybot or rcon there is still a backup measure of ip blocking.

Nothing can go wrong with spunkybot :-)
Well, when you are using rcon to ban/unban IPs, then it will anyway not work.
The bot-banlist file is only used for permban, so why would you anyway want to unban a perm-banned player?
I do not even see, why your code should work... looks more like an extreme risky code with potential endless loop, no unlink of file and manipulating stdout.
The original code is just appending to the end and closing the file.  Hopefully you are not running the bot with sudo or as root...


Suppress errors in .ts files using ts-ignore

Fixes https://github.com/microsoft/kiota/issues/4397
Added // @ts-ignore comment line as per discussion to suppress typecheck errors.
Currently, generated TypeScript .ts files contain imports that are never referenced in the code.
This causes TypeScript typecheck to fail in projects where tsconfig.json compiler option noUnusedLocals: true is set.
Example: Vitejs default tsconfig.json

@microsoft-github-policy-service agree company="Cybozu"

updated unit test
updated CHANGELOG


Scheme to reduce the memory of npy saving files

Hi, your current implementation for saving npy file looks fine but the memory store is too big due to using 3D array. (400mb/file). So we can reduce it by two steps:

Create 2D array. For each mask, it will have 1 integer value from 0 to number of mask. Using this scheme, we not only save all masks in a 2D matrix and make memory storing smaller, but also can seperate each mask based on the integer number.
Modify the load mask to convert from 2D to 3D matrix by a loop from 0 to number of mask
Hope it can help saving store npy file

Yea, this is a good solution. I'd be glad if you'd implement this and provide a separate dataset to decode this format.


Indication whether a stream of composed observables has action/progress/activity in it

Feature Request
I am trying to determine if an observable stream has some activity going on in it.
I have asked on stackoverflow but I don't think this is a usage question but a feature, unless I'm mistaken of course.
Is your feature request related to a problem? Please describe.
For an example use case - An entity (ex. customer) stream that get updated from several actions like loading, updating, and socket update listening. Each time the stream is doing something like loading or updating for example, I would like to show a spinner.
class CustomerService {
  customerStreamInProgress$ = /* how to calculate it */

  customer$ = this.streamCustomer();
  // ...
  streamCustomer() {
    return rx.pipe(
      () => merge(
        this.streamCustomerFromLoad(),
        this.streamCustomerFromUpdate(),
        this.streamCustomerFromSocket(),
      ),
      shareReplay(/* refcount 1*/),
    )(null):
  }

  streamCustomerFromLoad() {
    return rx.pipe(
      () => this.loadCustomerAction$,
      switchMap(cusotmerId => this.customerRestApi.fetchCustomer(customerId)),
    )(null);
  }

  // ... other methods the same design pattern.

  loadCustomer(id: number) {
    this.loadCustomerAction$.next(id);
  }
}

Describe the solution you'd like
It would be a lot more easier if an observable could expose on it an observable property inProgress$ that one can subscribe on. The library itself will take care while combining observables to notify upstream the in-progress state.
Other way could be to expose another type of signal on top of next, error, and complete. inProgress signal.
I am not saying this is a good idea, just what I'm feeling is missing in current architecture. Some way to observe a structure like a river and tell if water started/stopped flowing in at least one of it sections/branches. Some way to know if it is alive or not doing anything at the moment.
Describe alternatives you've considered

Currently, the way to solve this is by using manual behavior subject isMyModelInProgress$ that get set as true/false in each sub streams before and after actions start and end.

This is very verbose and cumbersome. I am trying to find a better aspect oriented way of solving this.

I have tried to hack some operators to include and provide a custom property inProgress$ on the observables they return, but it is very hacky.

An augmented mergeMap operator for example:
function ipMergeMap(project) {
  const inProcesSelf$ = new BehaviorSubject(false);
  const inProcessSelfSources$ = new Subject();

  return (source$) => {
    const resultOperatorObs$ = mergeMap((...args) => {
      inProcesSelf$.next(true);
      const result$ = project(...args);
      inProcesSelf$.next(false);
      inProcessSelfSources$.next(result$);
      return result$;
    })(source$);

    const inProcess$ = combineLatest(
      inProcesSelf$,
      source$.inProcess$ || new BehaviorSubject(false),
      inProcessSelfSources$.pipe(
        mergeMap((src$) => {
          return src$.inProcess$ || new BehaviorSubject(false);
        })
      )
    ).pipe(map((list) => list.some((inProcess) => !!inProcess)));

    resultOperatorObs$.inProcess$ = inProcess$;

    return resultOperatorObs$;
  };
}

I have tried looking into the scheduler functionally, maybe constructing a custom scheduler that can be given to the specific streams and use it to capture whether there is something going on at all or none. I do not know if a custome scheduler can be given to all desired piped operators for instance. I have tried looking into overriding the schedule method and augmenting the work with something that will notify start/end of the work but I'm limited at this time by my knowledge of schedulers mechanism and architecture.

(If this is new operator request) describe reason it should be core operator
This is not a new operator request.
Additional context
A more full expample of what I tried as the second solution and I do not think is the right way but is interesting:
https://codesandbox.io/s/github/elpdcore/rxjs-experiments-in-proces-observables/tree/master/?file=/src/1.1_create_an_observable.js

I'm closing this because I don't see how a feature like this would fit within the RxJS core.
Observables already have a mechanism for indicating whether or not a stream has activity in it: the complete notification. If a subscriber has not received a complete notification, then there is still 'something happening'. If the feature needs information more specific than that, I don't think it would be a good fit for the core.
In general, we encourage proposals to be built as user-land libraries - to provide some validation that the feature is generally useful.

I have asked on stackoverflow ...

You might have better luck opening a discussion in this repo, to seek input from others.


Use JPA in Custom Logic

Tutorial URL: https://developers.sap.com/tutorials/cp-apm-05-using-jpa.html
Step 1:
The pom.xml file after finisching tutorial 4 does not even look remotely like the one shown in the tutorial and is much simpler. So it is extremely hard to find the places where to put the code in.

Hi @MichaelWegelin ,
sorry for getting back to you that late.
Have you opened the pom.xml in your srv folder or in the root of your project? Please use the one in your srv folder and then you should be ready to go.


use karma-mocha

karma-mocha dropped the Mocha peer dep in v2.0.0.


build: Remove dependency on SplitMix in ChainSync demo

This PR removes the dependency on SplitMix in ChainSync demo and uses system build-in PRNG instead...

bors merge


How to create correct index fields in Riak objects

Calling indexfold with the 'wrong' parameters returns an empty set of results:
leveled_eqc:init_backend(o_rkv, [{root_path, Dir}]) -> <0.669.0>
leveled_eqc:put(<0.669.0>, <<98, 117, 99, 107, 101, 116, 49>>,
    <<0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0>>,
    <<53, 1, 0, 0, 0, 44, 131, 108, 0, 0, 0, 3, 104, 2, 100, 0, 6,
      97, 108, 98, 101, 114, 116, 97, 1, 104, 2, 100, 0, 5, 108, 101,
      105, 108, 97, 97, 2, 104, 2, 100, 0, 5, 99, 108, 97, 114, 97, 97,
      3, 106, 0, 0, 0, 1, 0, 0, 0, 33, 1, 188, 209, 79, 102, 144, 93,
      252, 224, 238, 140, 79, 222, 113, 86, 179, 177, 91, 91, 22, 85,
      135, 2, 125, 198, 183, 239, 39, 219, 227, 111, 160, 28, 0, 0, 0,
      15, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 101, 0>>,
    [], o_rkv) ->
  ok
leveled_eqc:indexfold(<0.669.0>,
    {<<98, 117, 99, 107, 101, 116, 51>>,
     <<150, 102, 157, 36, 213, 187, 216, 240, 27, 221, 28, 243, 131,
       76, 124, 45>>},
    fold_collect, {range, 1, 10}, {false, undefined}, 0) ->
  #Fun<leveled_runner.4.80344816>
leveled_eqc:fold_run(0, #Fun<leveled_runner.4.80344816>) -> []

But where can I find the correct way of IndexFields to be provided to Riak objects?

At this point
https://github.com/Quviq/leveled/blob/eqc-model/test/leveled_eqc.erl#L190
you need to provide some "index-specs" in the list you give to
generate-object.
https://github.com/Quviq/leveled/blob/eqc-model/src/leveled_codec.erl#L105
defines index specs. But it's a bit vague.
Here is an example of some indexes being generated in the existing code.

Oops, https://github.com/Quviq/leveled/blob/master/test/end_to_end/testutil.erl#L576

Got that!


Analytics on Opendatabutton.org

Do we have them

In place on the new static site - see odb_static. So once Joe is happy with that and it goes live, analytics will start working


Fix: Compiler warning about unused arguments

Can we try to get this change upstream?

Yes, but there is another issue with upstream, where two other packages don't build. Those are temporarily removed in this fork


Problems logging in

Hey there :)
I'm trying to write a basic script that will just accept all tradeoffers, I'm using you storehouse example...
How do I trigger a Steam e-mail with my auth code so that I can put it in my code?
Thanks and much appreciated,
James

#121


Remove sources

Closes #5473
Closes #5807
Closes #5803
Closes #5650
Closes #5608
Closes #5645
Closes #5813
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[ ] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[ ] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Seems like Jobside was moved to domain mentioned in #5805
Up to you if you want to address this in this PR.


Reset staging task

Goal
Facilitate testing by combining the :back_up and :delete_real_users into one task.
Approach

Add :reset_staging to combine the two tasks necessary for staging.
Document 1.

Pull Request Test Coverage Report for Build 709712578

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

Totals

Change from base Build 707489314:
0.0%

Covered Lines:
113

Relevant Lines:
117

💛  - Coveralls

@wraasch merging since this isn't user-facing.


"Iclod" is listed in the drop-down menu

Description:
When user writes "Iclod" the location appears in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Iclod"
Expected results
The location appears in the drop-down menu as "Comuna Iclod, CLUJ"


Add kubeConfig.QPS option for kwok controller

What would you like to be added?
An option just like --kube-api-qps=50 should be added in KwokConfigurationOptions (or somewhere else? idk) to configure the QPS for kwok controller.
I noticed there is an --disable-qps-limits option for kwokctl but I think it just configures the controller manager,  rather than the kwok controller itself.
Why is this needed?
For mocking the cluster under a huge burst pod creating and deleting.

The kwok doesn't have a QPS limit, I thought you might need this #1122 to make a huge burst pod creating and deleting.

The kwok doesn't have a QPS limit, I thought you might need this #1122 to make a huge burst pod creating and deleting.

But when I wrote my custom controller using kubebuilder, I usually needed to set a large kubeConfig.QPS before I connect my controller to the API server, just like:

import	ctrl "sigs.k8s.io/controller-runtime"

...

	kubeConfig := ctrl.GetConfigOrDie()
	kubeConfig.QPS = 10000
	kubeConfig.Burst = 10000
	ctrl.NewControllerManagedBy(mgr).For(&corev1.Pod{}).Complete(&UnscheduledPodReconciler{
		Client:      mgr.GetClient(),
	})

If I don't do that I cannot send too many messages to the API server. So shouldn't we add an option for the Kwok controller too.

https://github.com/kubernetes-sigs/kwok/blob/4f5fb135109ce234d8ccdb185ed041647bc4dd90/pkg/utils/client/clientset.go#L101
In this repo, use fake flow control to make it unlimited.

https://github.com/kubernetes-sigs/kwok/blob/4f5fb135109ce234d8ccdb185ed041647bc4dd90/pkg/utils/client/clientset.go#L101
In this repo, use fake flow control to make it unlimited.

Oh, that helps. Thank you.


How can I run the example project?

Describe the bug
No instruction for running the example project
To Reproduce
Steps to reproduce the behavior:

Try to run the example project :)
Expected behavior
Make it run
Screenshots
I wish
Desktop (please complete the following information):

OS: iOS
Browser chrome
Version 104.0.5112.79

Smartphone (please complete the following information):

Device: iPhone 13 Simulator
OS: iOS15.5
Browser N/A
Version N/A

Additional context

we have a guide on this in the contributing.md doc :) https://github.com/stripe/stripe-react-native/blob/master/CONTRIBUTING.md#running-the-example-app


Bad predictions

Raport:

telescope,
time/date,
sequence,
duration prediction (wrong)

jk15, 2024-11-26T14:38 UTC, ZERO test seq=1/u/0.0, 574s
--Mikolaj

Fixed! Now DARK, ZERO, SNAP should be ok. Ctc module is already updated on all computers: zenith, nadir, aux .
Please @majkelx give me some feedback if is ok in TOI now.
FYI: @astromg

Solved by use object parameters in other not trained well commands


feat(Standardize S3 Constructs): Match To Current Design Guidelines

Over the past few months, upgrades to some S3 constructs have led to a richer S3 interface in the AWS Solutions Constructs design guidelines. This Issue is to drive a task to upgrade all S3 constructs to make them consistent and in agreement with the new design guidelines.
The updated design guidelines enable the construct to be totally cleaned up upon deletion - let's update all the unit tests to take advantage of that.
As each construct is completed, mark -done next to it below.
Constructs Affected:
aws-cloudfront-s3/ - done
aws-eventbridge-kinesisfirehose-s3/ - done
aws-events-rule-kinesisfirehose-s3/ - done
aws-iot-kinesisfirehose-s3/ - done
aws-kinesisfirehose-s3/ - done
aws-kinesisfirehose-s3-and-kinesisanalytics/ - done
aws-kinesisstreams-kinesisfirehose-s3/ - done
aws-iot-s3/
aws-lambda-s3/ - done
aws-s3-lambda/ - done
aws-s3-sqs/ - done
aws-s3-step-function/ - done
aws-s3-stepfunctions/ - done
Use Case

Proposed Solution

Other

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

aws-iot-s3 is not published yet - checking this issue will be part of the approval process for that construct.


How to turn on caching headers

I see in https://github.com/piprate/json-gold/issues/6 we now have caching headers.
This is fantastic, but can you tell me how I invoke this.  I am not sure where to start to turn this option on.
Am I correct that this would remove the act of pulling the context from the net every time a process is called?

@Koshroy @kazarena
I just wanted to follow up on this..   I am trying to dig through the commit and see how to set the expireTime and/or neverExpires params.  I'm not quite sure how to connect to that though.
given
	proc := ld.NewJsonLdProcessor()
	options := ld.NewJsonLdOptions("")
	options.Format = "application/nquads"

Is there a struct or var I can call to set to turn on caching?   I'm hitting the schema.org context 10's of thousands of time and I'd like to stop.  :)
Thanks!
Doug

just to maintain the thread I have gotten to this point and it looks to be working but it's possibly not the best example.   Please excuse the mapping, we had some providers not using correct prefix/context strings.
func JLDProc() (*ld.JsonLdProcessor, *ld.JsonLdOptions) {
	proc := ld.NewJsonLdProcessor()
	options := ld.NewJsonLdOptions("")

	client := &http.Client{}
	nl := ld.NewDefaultDocumentLoader(client)

	// testing caching
	cdl := ld.NewCachingDocumentLoader(nl)
	cdl.PreloadWithMapping(map[string]string{"https://schema.org/": "/home/fils/Project418/gleaner/docs/jsonldcontext.json",
		"http://schema.org/": "/home/fils/Project418/gleaner/docs/jsonldcontext.json",
		"https://schema.org": "/home/fils/Project418/gleaner/docs/jsonldcontext.json",
		"http://schema.org":  "/home/fils/Project418/gleaner/docs/jsonldcontext.json"})

	options.DocumentLoader = cdl
	options.Format = "application/nquads"

	return proc, options
}


Create a Basic Flow with a SOAP Endpoint

Tutorials: https://developers.sap.com/tutorials/cp-integration-create-basic-flow.html
In the prerequisites, there is no step 6 in the Integration Suite onboarding Tutorial.  How do we assign the ESBMessaging.send role to the calling user?

Hello,
This tutorial has been deprecated. We will replace it with an updated tutorial in the coming weeks. Please watch out for "Integration Suite tutorials" in the SAP Tutorial Navigator for new updates.
Thank you.


Configuration from SQLite

General idea
OpenGL settings being stored in SQLite database.
Code example
clearColor = config["CLEAR_COLOR"] ?: Color.BLACK
config["BLEND"] = checkBox.checked

Details
Values in the configuration should be both read and written.
There should be easy to use Anko components for editing the configuration.

Descoped – there are multiple graphics quality settings that aren't related to basic configuration


[ffmpeg] Fix tools dependency dll file

Describe the pull request

What does your PR fix? Fixes #12008
Due to missing dll file, ffmpeg.exe open failed. Adding vcpkg_copy_tool_dependencies to fix this issue.
Which triplets are supported/not supported? Have you updated the CI baseline?
No
Does your PR follow the maintainer guide?
Yes

/azp run

So what is this PR waiting for? Why not merge it?

Thanks Lily!


Feature Request: filter by project root

Let's say I have a couple of projects:
project_a
project_b
project_c

If I navigate inside a particular root directory, I would like frecency results only from within that project.
$ cd project_a
$ nvim

:Telescope frecency
project_a/main.py
project_a/i_love_opening_this_file.py

Project root can be determined by the presence of .git or .svn.

you can filter by cwd since https://github.com/nvim-telescope/telescope-frecency.nvim/issues/14 but to achieve what you want (this is what I want too), we should be able to default to :CWD: as a filter. I am not sure yet if that's possible with telescope (have a default query).

There is a default_text option (sunjon asked me about that) but that one is currently broken because of missing missing prompt buffer patches in neovim. I have a workaround for that in https://github.com/nvim-telescope/telescope.nvim/pull/521 that will be merge soon. After that :Telescope frecency frecency default_text=:CWD: will do what you are looking for :)

hi, is there any update on this issue? I saw the PR is having merge conflicts. :(

The PR is merged, this does now work :Telescope frecency frecency default_text=:CWD: but only the first time, the second time @sunjon s persistent_filter option kicks in https://github.com/nvim-telescope/telescope-frecency.nvim/blob/721300e3d6f4a7157a781014d3d69bb1c7b702a3/lua/telescope/_extensions/frecency.lua#L254
You would need to disable it or sunjon needs to check that default_text isn't already a tag.
To disable
telescope.setup {
  extensions = {
    frecency = {
      persistent_filter = false, ------ its this one
      show_scores = false,
      show_unindexed = true,
      ignore_patterns = {"*.git/*", "*/tmp/*"},
      workspaces = {
        ["conf"]    = "/home/my_username/.config",
        ["data"]    = "/home/my_username/.local/share",
        ["project"] = "/home/my_username/projects",
        ["wiki"]    = "/home/my_username/wiki"
      }
    }
  },
}

I should probably also mention that you can do this with any other tag
:Telescope frecency frecency default_text=:project:

I've deprecated the persistent_filter option, as it was undocumented anyway.
The functionality can be reproduced with the above mentioned default_text and/or the upcoming
cache/resume PR.

I should probably also mention that you can do this with any other tag
:Telescope frecency frecency default_text=:project:

:Telescope frecency default_text=:project: also works. So how is this different from :Telescope frecency frecency default_text=:project: internally as it only one frecency


Create hashes from context menu.

What's the Problem?
We can't create checksum/hash files in locu when using Files v2
Solution/Idea
I see that checksum on file properties  has already been requested and implemented.
My suggestion is that we should be able to create checksums from the context menu. For example, I use the HashCheck shell extension a lot. Like the HashTab extension mentioned in the feature request linked above, hashcheck shows a checksum/hash  tab under file properties.
It also adds this context menu, though:

Once clicked, it lets me select between some algorithms:

Then a hash file is created in the same folder, with the same filename.
So suppose I have a file named foo.txt, and select md5 algorithm, it will create a foo.txt.md5 in the folder. Hashcheck also allows you to perform this operation in batches.
Very useful feature, would be nice if it was implemented.
Alternatives
So far, only going back to File Explorer to perform the operation, or using shell extensions on Files v2.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.194
Windows Version
Windows 11 21H2 22000.194
Additional comment
No response

Hi thanks for the suggestion. Shell extensions are supposed to appear in Files context menu. Please check if you just need to scroll down the menu to see them.
If they still not appear please upload the app's debug logs.

No problem, thanks for confirming. Having to scroll down is not intuitive at all and it's something we're looking to fix.


[#1615] create citation bug

+1. @gantian127 Merge after Jenkins checks are successful.

@gantian127 - Have restored the branch in hopes of just needing to update in with respect to the current develop branch, and then submit a new PR.


Simplify RPM support to Python 3 only

Now that the VNCDoTool no longer officially supports Python 2 we can simplify the RPM spec file and restore compatibility with most recent RPM packager distros.

Hi @sibson, have you considered a later update of the python packaging via a toml file as described in the latest packaging guidelines from https://packaging.python.org/en/latest/tutorials/packaging-projects/#creating-pyproject-toml? I am just wondering. Otherwise I will be glad if you consider this pull request restricts RPM packaging to python 3 only, updates, and simplifies it.

thanks, ✨ 🍰 ✨
I've not thought about toml.  Assuming there are no regressions I wouldn't object to upgrading.


Conservation firing when going out of gas

Steps to Reproduce Issue
run a publishing transaction with not enough gas.
Expected Result
InsufficientGas error
Actual Result
Possible assert

still investigating

we have a story for why that is happening and I am verifying it in testing but it requires a bit of reworking in the conservation


Token Action HUD does not grab modifiers.

With weapon equipped it rolls correctly from sheet.

Not sure I understand the issue.  Is their a problem with another module?  Or with what.  More information is needed.  I'm not sure how Token Action HUD works.

I saw this post and tested it myself. I assume it is from this (https://github.com/espositos/fvtt-tokenactionhud/pull/151).
I wonder if there is something else they can pull from so it can catch extra boost die(etc) that should come with weapon rolls.

I'll leave this here if anyone wants to fix it, but there is too much to do for me to worry about modules that are not part of the system to make this any sort of priority.

As a addendum to this - rolling as part of a macro (even just dragging a weapon onto the macro bar) causes it to lose connection to the weapon's Item Qualities. I.e. a weapon has Accurate 1, and when rolled from the character sheet, it adds the Boost die as expected. However, dropping the weapon on the macro bar and rolling causes it to ONLY apply the character's dice in the applicable skill to be rolled, and ignores any item quality modifiers. In the chat output, the qualities appear, but when you click them, their link to the main compendium item quality is broken.

I understand @cstadther I just wanted to bring it to attention. I know there is a lot going on.
Thanks for the response!

with the latest dev branch I may have fixed the issue, can of of you try this with 1.4-alpha-4

Seems to work correctly on 1.4-alpha-5!

Awesome... it was the rollItem method I updated.


Add an option to disable auto-format

Sometimes, I just want to change a little bit of other's project and don't want to touch their styles. If there is an option to disable auto-formatting in workspace, it will be awesome!

What's more, I think auto-formatting should be disabled by default.

this plugin do not do anything when file is saved.. It does not do anything to run auto-format.. and I try it on my code-oss, the auto-format did not happened.. maybe there is anyother plugin do the lsp format when file is changed?

Sorry, I didn't realize I enable editor.formatOnSave


Add docstrings to extensions

Adding docs to extensions
For now chaining to #398
close https://github.com/iterative/mlem/issues/412 as well

resolved merge conflict, but your tests fail @mike0sv

Will resolve and merge after #417


[vcpkg] Terminates when getting Visual Studio environment variables

Describe the bug
vcpkg terminates when getting Visual Studio environment variables:
PS D:\vcpkg\vcpkg> .\vcpkg.exe install catch2:x64-windows
Computing installation plan...
The following packages will be built and installed:
    catch2[core]:x64-windows -> 2.13.1
Detecting compiler hash for triplet x64-windows...
PS D:\vcpkg\vcpkg>

Environment

OS: Windows 10
Compiler: Visual Studio 2017 preview 17

To Reproduce
Steps to reproduce the behavior:

./vcpkg install zlib --debug
See error:

PS D:\vcpkg\vcpkg> .\vcpkg.exe install catch2:x64-windows --debug
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] BuiltinRegistry initialized with: ""
[DEBUG] Using vcpkg-root: D:\vcpkg\vcpkg
[DEBUG] Using installed-root: D:\vcpkg\vcpkg\installed
[DEBUG] BuiltinRegistry initialized with: ""
[DEBUG] Using buildtrees-root: D:\vcpkg\vcpkg\buildtrees
[DEBUG] Using downloads-root: D:\vcpkg\vcpkg\downloads
[DEBUG] Using packages-root: D:\vcpkg\vcpkg\packages
[DEBUG] Using scripts-root: D:\vcpkg\vcpkg\scripts
[DEBUG] Using ports-root: D:\vcpkg\vcpkg\ports
[DEBUG] Using versions-root: D:\vcpkg\vcpkg\versions
[DEBUG] Default binary cache path is: C:\Users\usr\AppData\Local\vcpkg\archives
Computing installation plan...
[DEBUG] CreateProcessW("D:\vcpkg\vcpkg\downloads\tools\cmake-3.19.2-windows\cmake-3.19.2-win32-x86\bin\cmake.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after    98416 us
[DEBUG] CreateProcessW("D:\vcpkg\vcpkg\downloads\tools\cmake-3.19.2-windows\cmake-3.19.2-win32-x86\bin\cmake.exe" -DVCPKG_ROOT_DIR=D:/vcpkg/vcpkg -DPACKAGES_DIR=D:/vcpkg/vcpkg/packages -DBUILDTREES_DIR=D:/vcpkg/vcpkg/buildtrees -D_VCPKG_INSTALLED_DIR=D:/vcpkg/vcpkg/installed -DDOWNLOADS=D:/vcpkg/vcpkg/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "D:\vcpkg\vcpkg\buildtrees\0.vcpkg_tags.cmake")
[DEBUG] cmd_execute_and_stream_data() returned 0 after    87737 us
The following packages will be built and installed:
    catch2[core]:x64-windows -> 2.13.1
[DEBUG] CreateProcessW("C:\Program Files (x86)\Microsoft Visual Studio\Installer\vswhere.exe" -all -prerelease -legacy -products * -format xml)
[DEBUG] cmd_execute_and_stream_data() returned 0 after    65313 us
Detecting compiler hash for triplet x64-windows...
[DEBUG] Found path: C:\Program Files\Git\cmd\git.exe
[DEBUG] CreateProcessW("C:\Program Files\Git\cmd\git.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after   127146 us
[DEBUG] CreateProcessW("D:\vcpkg\vcpkg\downloads\tools\powershell-core-7.1.0-windows\pwsh.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after   351022 us
[DEBUG] CreateProcessW(cmd /c "C:\Program Files\Microsoft Visual Studio\17\Preview\VC\Auxiliary\Build\vcvarsall.bat" -vcvars_ver=14.2 amd64   2>&1 <NUL  & echo cdARN4xjKueKScMy9C6H & set)
[DEBUG] cmd_execute_and_stream_data() returned 255 after  3175847 us
[DEBUG] C:\A\1\70\s\src\vcpkg\base\system.process.cpp(592)
[DEBUG] Exiting after 3976360 us (3931944 us)

https://github.com/microsoft/vcpkg-tool/blob/96ec678f8e924a3faf6392d58260e3b35bd81938/src/vcpkg/base/system.process.cpp#L593:
    Environment System::cmd_execute_modify_env(const Command& cmd_line, const Environment& env)
    {
        static StringLiteral magic_string = "cdARN4xjKueKScMy9C6H";
...
        auto rc_output = cmd_execute_and_capture_output(actual_cmd_line, env);
        Checks::check_exit(VCPKG_LINE_INFO, rc_output.exit_code == 0);
...

The rc_output.output only contains:
**********************************************************************
** Visual Studio 17 Developer Command Prompt v17.0.0-partner.preview.1.0
** Copyright (c) 2021 Microsoft Corporation
**********************************************************************
: was unexpected at this time.

And the rc_output.exit_code is 255.
Here is my output by manually runing the command:
C:\Users\usr>cmd /c "C:\Program Files\Microsoft Visual Studio\17\Preview\VC\Auxiliary\Build\vcvarsall.bat" -vcvars_ver=14.2 amd64_x86   2>&1 <NUL  & echo cdARN4xjKueKScMy9C6H & set
**********************************************************************
** Visual Studio 17 Developer Command Prompt v17.0.0-partner.preview.1.0
** Copyright (c) 2021 Microsoft Corporation
**********************************************************************
: was unexpected at this time.
cdARN4xjKueKScMy9C6H
ALLUSERSPROFILE=C:\ProgramData
APPDATA=C:\Users\usr\AppData\Roaming
CLIENTNAME=USR
CommonProgramFiles=C:\Program Files\Common Files
CommonProgramFiles(x86)=C:\Program Files (x86)\Common Files
CommonProgramW6432=C:\Program Files\Common Files
COMPUTERNAME=***
ComSpec=C:\Windows\system32\cmd.exe
DriverData=C:\Windows\System32\Drivers\DriverData
HOMEDRIVE=C:
HOMEPATH=\Users\usr
LOCALAPPDATA=C:\Users\usr\AppData\Local
LOGONSERVER=\\CY1-FE-DC-04
NUMBER_OF_PROCESSORS=2
OneDrive=C:\Users\usr\OneDrive
OneDriveConsumer=C:\Users\usr\OneDrive
OS=Windows_NT
Path=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Program Files\Microsoft SQL Server\130\Tools\Binn\;C:\Program Files\Git\cmd;C:\Users\usr\AppData\Local\Microsoft\WindowsApps;
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC
PROCESSOR_ARCHITECTURE=AMD64
PROCESSOR_IDENTIFIER=AMD64 Family 23 Model 49 Stepping 0, AuthenticAMD
PROCESSOR_LEVEL=23
PROCESSOR_REVISION=3100
ProgramData=C:\ProgramData
ProgramFiles=C:\Program Files
ProgramFiles(x86)=C:\Program Files (x86)
ProgramW6432=C:\Program Files
PROMPT=$P$G
PSModulePath=C:\Program Files\WindowsPowerShell\Modules;C:\Windows\system32\WindowsPowerShell\v1.0\Modules;C:\Program Files\Microsoft Monitoring Agent\Agent\PowerShell\;C:\Program Files (x86)\Microsoft Azure Information Protection\Powershell
PUBLIC=C:\Users\Public
SESSIONNAME=RDP-Tcp#2
SystemDrive=C:
SystemRoot=C:\Windows
TEMP=C:\Users\usr\AppData\Local\Temp
TMP=C:\Users\usr\AppData\Local\Temp
UATDATA=C:\Windows\CCM\UATData\D9F8C395-CAB8-491d-B8AC-179A1FE1BE77
USERDNSDOMAIN=DOMAIN
USERDOMAIN=DOMAIN
USERDOMAIN_ROAMINGPROFILE=DOMAIN
USERNAME=usr
USERPROFILE=C:\Users\usr
windir=C:\Windows

C:\Users\usr>

same result with;
Microsoft Visual Studio Community 2019 Preview
Version 16.10.0 Preview 2.0

Apparently it's due to vcvarsall.bat from 16.10.0 Preview 2.0 printing some junk - : was unexpected at this time.. Went looking and it turns out it's due to unescaped parentheses in Common7\Tools\vsdevcmd\ext\vcvars.bat, lines 62 and 66. After escaping it  goes through fine:
if "%VSCMD_DEBUG%" GEQ "1" @echo [DEBUG:ext\%~nx0] Looking up MSVC SxS Toolset version ^(<IP_ADDRESS>^) : 'VC\Auxiliary\Build\14.28'
@echo [WARN:%~nx0] SxS version 14.28 was not present/installed ^(-vcvars_ver=<IP_ADDRESS> was specified^). Attempting fallback selection of toolset 'VC\Tools\MSVC\14.28.*'

cc @strega-nil.
The reason why vcpkg exit is because the result code is non-zero.

This should be reported to VS devs since it seems vcvars.bat in preview 2.0 is to blame, not vcpkg.

We have reported a bug to the developer team to track the error of vcvars.bat in VS2019 16.10 preview 2.0.

This issue has been tracked by https://developercommunity.visualstudio.com/t/vcvarsallbat-now-fails-if--vcvars_ver-i/1400395.

Some message from vcpkg that it's VS's fault might be handy.

@OlafvdSpek You can use extra debug command --debug

Most users aren't aware of that and will just see vcpkg exiting without any message.

May be we can add some failure message here? @strega-nil @JackBoosY

I don’t think we need to change this message from DEBUG mode to normal mode, because this situation is not common and users can’t do much.

What'd be the downside of changing it to normal mode?

There absolutely should be some kind of error message. Seeing vcpkg just stop execution, for no obvious reason, with nothing useful in the logs, is a really terrible UX. It should at the very least say "there was an unexpected error" instead of silently stopping

@DethRaid @OlafvdSpek Added.


Factor out fetcher of Maven dependencies

Currently there are 4 slowly-diverging copies. Also added limit on number of retries.
Tested gephi, exhibitor & global-platform-pro on my machine, hadoop is still building.
cc @volth @Radvendii @Ekleog @joelthompson
Motivation for this change
Was considering using the same approach for building jitsi-videobridge and jicofo, which would increase the number of copies to 6. The retry limit was added after staring at infinitely looping failing build for too long.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@volth

I'm aware, unfortunately it's the only method I could use to build the application, had no luck with mvn2nix nor mavenix. In fact the dependencies change daily so it doesn't make sense to submit these packages nixpkgs. Except for hadoop all sha256s match btw.
Removed.

Agreed that this method is bad, however there's lack of alternatives. Perhaps we can name the function something like impureFetchMavenDeps to indicate that? I won't mind dropping this PR though.

I think it is good, but too early: first we have to have the immutable maven mirror running or invent an alternative

Could we not have a script that takes a snapshot of the maven repository and all projects we need to fetch the dependencies from and generates fixed input derivations for all of them?

Gonna close this as I don't have the resources to improve the current state. Feel free to reuse.


support controller tests without application in context

WithApplication tests are much slower (starting a new app for every controller test) this pull request remove the assumption there's a play application in context allowing faster tests

Wow!! I'm sorry for losing this PR.
And thanks, it is usefull.

thanks. Actually with the recent release of Play 2.5 and the deprecation of Play.current this solution is useless.
Migrating to 2.5 will mean any static access to Play and Crypto will have to be rewritten.

Thanks, you are right.
I'll publish a new release for Play2.4.
And now,  I'm migrating to 2.5.


AttributeError: module 'torch' has no attribute 'special'

I have torch==1.7.0a0 installed, but when I ran out = gau(x), there is an error:

Traceback (most recent call last):
File "", line 1, in
File "MYPATH/work/venv/pytorch-rocm/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "MYPATH/work/flash-pytorch/git-FLASH-pytorch/flash_pytorch/flash_pytorch.py", line 201, in forward
attn = self.attn_fn(sim / seq_len)
File "MYPATH/work/venv/pytorch-rocm/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "MYPATH/work/flash-pytorch/git-FLASH-pytorch/flash_pytorch/flash_pytorch.py", line 131, in forward
return (1 + torch.special.erf((x - mu) / (std * math.sqrt(2)))) * 0.5
AttributeError: module 'torch' has no attribute 'special'

It seems that torch.special is introduced in Pytorch 1.9.

@bibo-msft ah ok, i put in a quick fix, thanks!


Add json lanuguage server(vscode-json-languageserver)

test on this repository.
Should fix https://github.com/vscode-langservers/vscode-json-languageserver-bin/issues/3

if this is working (are you using it?) let's merge it rather than have it sit around. can always improve it later.

I wonder why docgen didn't create a doc commit?

@justinmk see #110


Yubikey and token generator do nothing

Expected Behavior
On the "Enable Two Factor" wizard screen I am presented with two methods:

Token generator
YubiKey

When I select either option, I am taken to the setup screen for that particular option--token generator shows a QR code and Yubikey has a box where you can touch your YubiKey and fill in the token.
When the information is filled in and "Next" is pushed, I expect to see some sort of "You did it!" screen.
Current Behavior
After hitting "Next" when setting up either option, I am redirected back to /account/two_factor/setup/ and the wizard starts over.
Steps to Reproduce (for bugs)

I followed the instructions from https://django-two-factor-auth.readthedocs.io/en/stable/installation.html
The only item I skipped was the LOGIN_REDIRECT_URL.
I created a ValidationService object using info I received by going here: https://upgrade.yubico.com/getapikey/

Your Environment

Browser and version: Chrome 79.0.3945.130 (Official Build) (64-bit)
Python version: 3.6.7
Django version: 3.0.4
django-otp version: 0.8.1
django-two-factor-auth version: 1.11.0

Are any devices being created? You can check this via the shell if using django_otp.devices_for_user
Also, what happens if you set LOGIN_REDIRECT_URL? (it's meant to be optional by the way, I just want to rule it out as causing this issue)

I just tested with LOGIN_REDIRECT_URL = '/'.
I just had a thought that it could be related to having a custom user object.
Ran:
u = DashboardUser.objects.get(pk=1) # my test account
>>> for d in devices_for_user(u):
...   print(d)
...
>>>

I just looked at the sample site (http://example-two-factor-auth.herokuapp.com/account/two_factor/setup/) and it does the exact same thing.

Weird, the example site works fine for me when I select token generator. Do you have any Chrome extensions installed that might be interfering?

Only about 11 billion. ;)
Let me see if I can track it down...

It was an extension called "Web Server Notifier" that I've had installed for years.  It would give you info on the remote web server (nginx, apache, IIS, etc...) and the tech stack on the remote end (Django, ASP.NET, PHP, etc...)
The link is no longer valid in the web store.  Maybe it got pulled for abuse...  I removed it and it's working now.  Mystery solved.  Thanks!


base-64 encode block ID

refactor credentials to use environment credential
build endpoints specific to the examples
added dummy block blob content
added // Output clauses when running examples

Can one of the admins verify this patch?


Add github actions

I wrote 2 files:

one for lint and type checking
one for the test and coverage. Tests are failing right now.

@DerouineauNicolas can you also review this PR please?
@Diaoul didn't add you as maintainer yet so I cannot assign you automatically :(

Looks good to me ! Good job !

Thanks!
For the tests, I would like remove the cassettes and data subfolders before running the tests, run one test to generate the cassettes and then run the tests for the matrix of OS and python versions with the cached cassettes (so the websites are not called 15 times).
Any idea how to do that?
On 8 June 2024 10:20:31 GMT+01:00, DerouineauNicolas @.***> wrote:

Looks good to me ! Good job !
--
Reply to this email directly or view it on GitHub:
https://github.com/Diaoul/subliminal/pull/1118#issuecomment-2155888782
You are receiving this because you authored the thread.
Message ID: @.***>

I'm not sure this is a good idea to call the external API in the CI. I would leaves the cassettes to catch any requests, but maybe I didn't understand the problematic correctly.

My idea was to run automatic test every month, without empty cassettes, so we make sure that the API are still working.
After a PR or push maybe we don't need to call the API and we could used cached cassettes.
But anyway I want the tests for the different OS and python-version to use cached cassettes also.
So maybe writing another test to run every month on a single OS-python-version and with empty cassettes would be OK.

For python versions <= 3.9, it seems that the cassettes with a gzipped content cannot be decompressed.
Any idea how to force a non-compression, so we can re-run the faulty cassettes?

for python-version 3.10, the tests pass after applying #1127
for python-version 3.12, we need to solve #1126 first.

From my perspective, the simpliest way to disable gzip compression would be to regenerate the cassetes with the option decode_compressed_response=True (https://vcrpy.readthedocs.io/en/latest/advanced.html#decode-compressed-response)

Thanks I'll try that
On 14 June 2024 11:07:25 GMT+01:00, DerouineauNicolas @.***> wrote:

From my perspective, the simpliest way to disable gzip compression would be to regenerate the cassetes with the option decode_compressed_response=True (https://vcrpy.readthedocs.io/en/latest/advanced.html#decode-compressed-response)
--
Reply to this email directly or view it on GitHub:
https://github.com/Diaoul/subliminal/pull/1118#issuecomment-2167697173
You are receiving this because you authored the thread.
Message ID: @.***>

all the tests are passing except for one in version py312 which is affected by #1126 .


feat: upgrade to typescript 5.5

#195  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @pevisscher and the rest of your teammates on  Graphite


Split Views standard available

Hi Vlad,
I know you have a split view app in demos, done with custom css, but split views are really an integral part of IOS design, so maybe you can make them standard available, like the left and right panels and popups you already have.
It will be nice to be able to make them with ready to go css classes and also have them responsive, so for example the left panel gets hidden when there is no room and the panels become single cascading panels, like the settings on IOS.
Thanks,
George

Was split-view functionality already implemented? As we are now at 1.0.6
Will be also nice if it can be combined with a drop down list. For example if there is room left panel is displayed but if not it becomes available under drop down button

It would seem this feature never really made it but is still a valid request

Ping @nolimits4web

@ZanderBrown was trying to implement it but without much success as it relies heavily on custom app layout. So delayed until have more time to dig into it

Closing as it is done in v4


Invalid glob pattern crashes chat

When trying to add files using an invalid glob pattern, the chat crashes.
Aider version: v0.15.0
OS: Ubuntu 22.04
Python version: 3.10
Example Chat:
Aider v0.15.0
VSCode terminal detected, pretty output has been disabled.
Model: gpt-4
Git repo: .git
Repo-map: universal-ctags using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args
───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
> /add **.txt

Traceback (most recent call last):
  File "/usr/local/py-utils/bin/aider", line 8, in <module>
    sys.exit(main())
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/main.py", line 564, in main
    coder.run()
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/coders/base_coder.py", line 326, in run
    new_user_message = self.run_loop()
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/coders/base_coder.py", line 405, in run_loop
    return self.commands.run(inp)
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/commands.py", line 73, in run
    return self.do_run(matching_commands[0][1:], rest_inp)
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/commands.py", line 51, in do_run
    return cmd_method(args)
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/commands.py", line 293, in cmd_add
    matched_files = self.glob_filtered_to_repo(word)
  File "/usr/local/py-utils/venvs/aider-chat/lib/python3.10/site-packages/aider/commands.py", line 256, in glob_filtered_to_repo
    raw_matched_files = list(Path(self.coder.root).glob(pattern))
  File "/usr/lib/python3.10/pathlib.py", line 1033, in glob
    selector = _make_selector(tuple(pattern_parts), self._flavour)
  File "/usr/lib/python3.10/pathlib.py", line 374, in _make_selector
    raise ValueError("Invalid pattern: '**' can only be an entire path component")
ValueError: Invalid pattern: '**' can only be an entire path component

This exits aider.

Thanks for trying aider and filing this issue. I just pushed a fix.  The change is available in the main branch. You can get it by installing the latest version from github:
python -m pip install git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.

I'm going to close this issue for now, but feel free to re-open or file a new issue any time.


implement bidirectional lstm

I need bidirectional lstm in CRNN model.
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

I think the current implementation of bidirectional lstm can work during inference, but the implementation cannot update the parameters of the model in backward propagation due to #1098. Using Tensor::cat instead of Tensor::slice_assign can solve this problem, but it seems to have some negative performance impact, so I think it's best to wait for the Tensor::slice_assign bug to be resolved for now.

Closing this ticket and linking to this ticket: https://github.com/tracel-ai/burn/issues/1537. So someone else can pick up.

@antimora Could you reopen this pull request? The issue regarding Autodiff has been resolved in #1575, and I think it's time to proceed with implementing the Bidirectional LSTM.

Wgpu test failed, but I don't know why...

What was the error in wgpu?

Hm ok well on ubuntu-22.04 everything seems fine when I run the tests locally. Given that ubuntu and windows work, maybe it hints at possible compatibility issues or differences in how Metal handles resource management compared to Vulkan (ie Linux) or DirectX (Windows)?

The outputs of both Lstm and BiLstm are now aligned with PyTorch.

I changed the batch size to 2, and the test passed. This seems to be a bug with the Wgpu backend, so I've opened an issue #1656.

We will merge it, once we have an approval from @laggui


HTCondor starts up too fast in 8.9.9 and we miss the schedd startup

core.get_stat returns None if the file doesn't exist and condor.wait_for_daemon handles None stats by looking for the relevant sentinel starting at the beginning of the file. Things look pretty good here: https://osg-sw-submit.chtc.wisc.edu/tests/20201028-0817/results.html

How does this behave with multiple osg-test runs, when htcondor-ce has already been started and stopped once?

If the CE service was started outside of osg-test, we don't end up using the stat. I could move the stat after this check so we don't bother with it in that case
If the CE service is not running, we grab the current stat and check for the schedd reporting back to the collector starting from the stat


Feat: support dynamically set client config

support update client config. e.g. auth token, org id ...

Hey, thank you for the PR! Could you please elaborate on the motivation for this dynamic config? Why doesn't re-creating client objects with new configs doesn't work for you?

Yup, my bad for not carefully reviewing the source code. Re-create client object with new config is  fine.


Use project name as container name

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@2fea693). Click here to learn what that means.
The diff coverage is 0.00%.

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

@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage        ?   38.42%
=======================================
  Files           ?       11
  Lines           ?      203
  Branches        ?        0
=======================================
  Hits            ?       78
  Misses          ?      125
  Partials        ?        0

Impacted Files
Coverage Δ

lib/src/docker/create_image.dart
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


TypeError: Can not set property 'color' of undefined

Hello,
I am getting "TypeError: Cannot set property 'color' of undefined" error when reading XSLX file with colors.
Below is an the FULL ERROR LOG, Solution but not sure if correct approach, and image of excel sheets reading.
NOTE - Removing colors/format from XSLX then reading it works.. but users will not always do this.. Also this is error is uncaught crashing server.
Error Log
ERROR - uncaughtException STACK -  TypeError: Can
not set property 'color' of undefined
at utils.inherits.parseClose (C:\Alex\AlexProjects\topSecret\node_modu
les\exceljs\lib\xlsx\xform\style\border-xform.js:99:28)
at utils.inherits.parseClose (C:\Alex\AlexProjects\topSecret\node_modu
les\exceljs\lib\xlsx\xform\style\border-xform.js:196:24)
at utils.inherits.parseClose (C:\Alex\AlexProjects\topSecret\node_modu
les\exceljs\lib\xlsx\xform\list-xform.js:87:24)
at utils.inherits.parseClose (C:\Alex\AlexProjects\topSecret\node_modu
les\exceljs\lib\xlsx\xform\style\styles-xform.js:223:24)
at SAXStream. (C:\Alex\AlexProjects\topSecret\node_modules
exceljs\lib\xlsx\xform\base-xform.js:71:19)
at emitOne (events.js:77:13)
at SAXStream.emit (events.js:169:7)
at Object.me.parser.(anonymous function) [as onclosetag](C:VZAlexAlexPro
jectstapsiumportalnode_modulesexceljsnode_modulessaxlibsax.js:245:15)
at emit (C:\Alex\AlexProjects\topSecret\node_modules\exceljs\node_modu
les\sax\lib\sax.js:615:33)
at emitNode (C:\Alex\AlexProjects\topSecret\node_modules\exceljs\node
modules\sax\lib\sax.js:620:3)
at closeTag (C:\Alex\AlexProjects\topSecret\node_modules\exceljs\node_
modules\sax\lib\sax.js:861:5)
at Object.write (C:\Alex\AlexProjects\topSecret\node_modules\exceljs\n
ode_modules\sax\lib\sax.js:1182:11)
at SAXStream.write (C:\Alex\AlexProjects\topSecret\node_modules\excelj
s\node_modules\sax\lib\sax.js:227:16)
at Entry.ondata (_stream_readable.js:528:20)
at emitOne (events.js:77:13)
at Entry.emit (events.js:169:7)
Solution but not sure if correct approach
Insides file \node_modules\exceljs\lib\xlsx\xform\style\border-xform.js:99:28
Before
if (this.map.color.model) {
this.model.color = this.map.color.model;
}
After Fix - Then it worked fine. Not sure if I'm breaking any other major things.
if (this.model && this.map.color && this.map.color.model) {
this.model.color = this.map.color.model;
}
XSLX - Image

This same problem is happening to me -- if I edit the spreadsheet to remove colors, code works fine, but otherwise I get the exception.

I have the same problem too!

@fkerekiglobant @lassmann Did you guys tried the solution part(just adding to IF Condition)?

Hi! No on my part -- the actual code depends on npm getting the right
version, and it wouldn't work.
Best regards,
F.Kereki
Ing. Federico Kereki
Tech Master - Big Data Studio
[image: Facebook] https://www.facebook.com/Globant [image: Twitter]
http://www.twitter.com/globant [image: Youtube]
http://www.youtube.com/Globant [image: Linkedin]
http://www.linkedin.com/company/globant [image: Pinterest]
http://pinterest.com/globant/ [image: Globant] http://www.globant.com/
On Fri, Jul 15, 2016 at 1:48 PM, abartolo<EMAIL_ADDRESS>wrote:

@fkerekiglobant https://github.com/fkerekiglobant @lassmann
https://github.com/lassmann Did you guys tried the solution part(just
adding to IF Condition)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/guyonroche/exceljs/issues/146#issuecomment-233005113,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATZlX2qhXgP7lCy6njZHA-FYdH01fdd5ks5qV7nWgaJpZM4JNgen
.

--
The information contained in this e-mail may be confidential. It has been
sent for the sole use of the intended recipient(s). If the reader of this
message is not an intended recipient, you are hereby notified that any
unauthorized review, use, disclosure, dissemination, distribution or
copying of this communication, or any of its contents,
is strictly prohibited. If you have received it by mistake please let us
know by e-mail immediately and delete it from your system. Many thanks.
La información contenida en este mensaje puede ser confidencial. Ha sido
enviada para el uso exclusivo del destinatario(s) previsto. Si el lector de
este mensaje no fuera el destinatario previsto, por el presente queda Ud.
notificado que cualquier lectura, uso, publicación, diseminación,
distribución o copiado de esta comunicación o su contenido está
estrictamente prohibido. En caso de que Ud. hubiera recibido este mensaje
por error le agradeceremos notificarnos por e-mail inmediatamente y
eliminarlo de su sistema. Muchas gracias.

Same Problem here. @abartolo 's workaround fixed the problem for my case.

fix it please! thanks!

:+1:

I know I am super late but I looked into this thinking it still wasn't fixed but it looks like it was addressed.
As you can see below it is now addressed if this.model is NOT defined. I will close the issue, please comment if reopen needed.
Code Before
      if (name === this.name) {
        if (this.model && this.map.color && this.map.color.model) {
          this.model.color = this.map.color.model;
        }
      }
      return false;

Code Now
 if (name === this.name) {
        if (this.map.color.model) {
          if(!this.model)
            this.model={};
          this.model.color = this.map.color.model;
        }
      }
      return false;


setup for crm app

Blocked this branch from being merged

Is this still relevant or can we close this?


fixing idioms and mental models

It still has been bothering me that there is no consistency in how we are handling errors, my mental models are off and bolierplate code was missing from the web package.
In not being able to sleep, I think I figured it out. Along the way, found another naming issue.

This change is good. I still think we could do better on the names.
If I see an identifier starting with Err like web.ErrShutdown I am going to first think it is a sentinel error like product.ErrNotFound or sql.ErrNoRows. So the fact that web.ErrRequestFailed and web.ErrShutdown are actually functions which return errors of some type is confusing.
These functions are factory functions so what about this?
package web

type AppError{
 Err    error
 Status int
 Fields []FieldError
}
func NewAppError(error, int) error

type ShutdownError{
 Message string
}
func NewShutdownError(string) error
func IsShutdown(error) bool

Another alternative would be to unexport these types and make the functions named web.AppError and web.ShutdownError. Since the error response behavior is back in the web package, no other packages need to know the concrete types outside of where we use IsShutdown.

I like the Err naming convention for returning error values. I know these are not variables but I figured it wouild convey that same message of return an error value. If you look at the handler code that is using it, the readability I think will be better using Err vs New.
prod, err := product.Retrieve(ctx, dbConn, params["id"])
	if err != nil {
		switch err {
		case product.ErrInvalidID:
			return web.ErrRequestFailed(err, http.StatusBadRequest)
		case product.ErrNotFound:
			return web.ErrRequestFailed(err, http.StatusNotFound)
		default:
			return errors.Wrapf(err, "ID: %s", params["id"])
		}
	}

vs
prod, err := product.Retrieve(ctx, dbConn, params["id"])
	if err != nil {
		switch err {
		case product.ErrInvalidID:
			return web.NewRequestFailed(err, http.StatusBadRequest)
		case product.ErrNotFound:
			return web.NewRequestFailed(err, http.StatusNotFound)
		default:
			return errors.Wrapf(err, "ID: %s", params["id"])
		}
	}

This is complicated because I think we have something new here. Using New on the return I think is wrong. Let's focus this decision on what makes this code example more readable.

I agree that NewRequestFailed is hard to read but that's not what I suggested. I suggested return web.NewAppError. Which is true! We are returning a new error.
Really though it's  "Wrapped" error. I would feel better about something like return web.WrapError
On the Shutdown side of things it absolutely is a new error. You give it a string and it gives you a shutdown error. That one definitely should be return web.NewShutdownError

prod, err := product.Retrieve(ctx, dbConn, params["id"])
	if err != nil {
		switch err {
		case product.ErrInvalidID:
			return web.ErrRequestFailed(err, http.StatusBadRequest)
		case product.ErrNotFound:
			return web.NewRequestFailed(err, http.StatusNotFound)
		default:
			return web.WrapRequestError(err, http.StatusWhatever)
		}
	}

Reading this code, which is best describing the business behavior? Forget about which one is more technically correct.

if we are returning an error for this request then, in MY mind the most clear thing hasn't been mentioned yet which is return web.NewRequestError(err, 404).
package web

type RequestError struct{}
func NewRequestError(error, int) error

Try looking at it my way
prod, err := product.Retrieve(ctx, dbConn, params["id"])
if err != nil {
	switch err {
	case product.ErrInvalidID:
		return web.NewRequestError(err, http.StatusBadRequest)
	case product.ErrNotFound:
		return web.NewRequestError(err, http.StatusNotFound)
	case product.ErrForbidden:
		return web.NewRequestError(err, http.StatusForbidden)
	default:
		return errors.Wrapf(err, "ID: %s", params["id"])
	}
}

Ok, let’s use New. I’ll make the change later.

@jcbwlkr you were right. Your suggestion works well and is more idiomatic.

Looks great. Thanks, man!


Always get bad device token when send IOS notification

Hello anyone. I download your project with test. I fill all settings of my apn from firebase for IOS.
Then i try to send push notification and i always get BadDeviceToken.
In the firebase console with that token i got push notification on my phone successfully.
Please, help me, what i do wrong ?

Hi , are you fix it this issue? Or you still get BadDeviceToken ?
because i am received the same error
thanks

Guys, I need some more details on that issue... would you possibly be able to test it with the source code of the lib and create a PR?

The problem is when i am using the fcm token from client side and send the push notification from firebase console directly it was send successfully but when i am using the fcm which i received from the client side i got BadDeviceToken my code is below                 var apnSettings = new ApnSettings();
apnSettings.ServerType = ApnServerType.Production;
apnSettings.AppBundleIdentifier = _notificationSettings.AppBundleIdentifier;
apnSettings.P8PrivateKey = _notificationSettings.P8PrivateKey;
apnSettings.P8PrivateKeyId = _notificationSettings.P8PrivateKeyId;
apnSettings.TeamId = _notificationSettings.TeamId;

@andrei-m-code  Please can you tell us how can we solve this issue?
everything i set it correctly as you mentioned in the readme file but only on the iOS device i got BadDeviceToken thanks advance

@Hamza-Hasan-Almass  you don't use the FCMToken with ApnSettings because the ApnToken and the FcmToken are different.
Check this link

The device token in APNs is a 64-byte hex string, and the Device ID in FCM is a 183-byte string, containing [a-zA-Z0-9_-] (all numbers, lowercase, and uppercase alphabets, and symbols like , _ and –).

Examples:
FCM Token: aaa12Om83EL9r57in1cJpy:APA91bFv1ydna_52WPl34jNYAFCs-_MHcgrTjjYlg2NetJRPI71111jf4449md-WsSDHcKLWS_n6mGpTGtOZZZhQQQO7NiI65LZMmxqpFUbAivn_YRYR-cIceQH4AhFmg0muUGdlAQYt
APNs Token: 2f3035b209d30980e462a0921f4853021d7ee030e200bc0000487b9008a80005
If you use FCM token with ApnSender then you'll receive BadDeviceToken. Use APNs Token with ApnSender or FCM Token with FCMSender


AudioStreamPlayer causes unpredictable behaviour in idle process

Godot version:

Godot 3.1-stable
OS/device including version:

Issue description:

So, this is a weird one!
First I thought it was the timer node (https://github.com/godotengine/godot/issues/27437).
Then @Tugsav managed to find a possible fix to this issue, but @reduz isn't sure it actually makes sense (https://github.com/godotengine/godot/pull/27458).
Now I managed to create a reproduction project, but it isn't pretty, but it shows that there is an error!
I am pretty sure that the AudioStreamPlayer is the problem, and it causes the idle process across the game to behave very weird. It stops other things running in the idle process such as Particles and Animations. In some cases I also experienced particles and animations being sped up.
Steps to reproduce:
Run the project I included and keep your eyes on two things!

See how the four animations and the particle system stops one at a time.
Check the log output and notice a few things.
2.1 ERROR: partitioner: bad comparison function; sorting will be broken At: ./core/sort_array.h:189.
2.2 The outputs should come every 0.1 seconds, but it will start to speed up significant and print way faster.

There are a few things to do in the project that will result in different behaviours.

The script "Main.gd" does "audio.play()" and "audio.stop()" right after eachother in a while loop with some other code, if one of these lines or both are commented out, this whole error doesn't occur.

The node with the "Main.gd" script attached (Node) can be placed a few different places in the scene tree that will cause different behaviours. If it is placed as the top node, just below "Main" the while loop runs forever, but eventually all the animations and the particles will stop, and the idle process runs a lot faster (see log output being printed way faster than it should)

If Node is placed as the last node in the tree the animations and the particles will continue working but the while loop will at some point straight up stop because the timer doesn't get the timeout signal correct and therefore hang forever.

If Node is placed after the "ExtraScenes" node the behaviour is a mix of the two above. First the animations and the particles will stop, and when they all are stopped the while loop will stop as a result of the timer failing.

So, to the solution provided by @Tugsav
Our thought about this is that the AudioStreamPlayer runs in the idle process and somehow is able to mess it up! To further prove that this is happening, go ahead and set one or more of the animation players in the scene to process mode physics and play the game again.
This will result in the animation players working as intended and never stop.
So the PR I linked to in the beginning makes the change to the AudioStreamPlayer, and that PR fixes this problem! But it might not be the correct solution as @reduz isn't completely convinced about the PR.
The solution is to make the AudioStreamPlayer run in physics process instead, just like the AudioStreamPlayer2D and AudioStreamPlayer3D does.
I really hope that this time around, the issue is way more clear, and that the reproduction project is enough to both prove that there is an issue and to help find a fix as this is quite a bad bug!
Minimal reproduction project:

test.tar.gz

This was caused by #25974, which was reverted.


Use setup to customize project with new name

Is your feature request related to a problem? Please describe.
The amazing bento-starter name is used in specific files like package.json and others. It would be great if the setup from #140 also allows to customize files not specific to git.
Describe the solution you'd like
Add a function to the setup command to replace technical occurrences to bento-starter with a choosen project name

This will be fix after that #140 will be merged !
Thanks for your issue !

It should replace 'bento starter' label in :

[ ] package.json name value
[ ] package-lock.json name value
[ ] /public/manifest.json name value
[ ] Vue head complement value
[ ] Vue head meta descriptions
[ ] Navbar title
[ ] VUE_APP_TITLE env variable in .env.example or .env.local
[ ] in public/index.html
[ ] inAppleAddToHomeScreenModal.vue file

We should use env variable to handle this easily.

@kefranabg in package.json there is also the "_id" which has to change if you want to change the service-worker.json cache name prefix


Merge release into master

@bot-gradle test and merge

I've triggered a build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Pre-tested commit build cancelled.

I've triggered a build for you.

Your PR is queued. See the queue page for details.

I've triggered a build for you.


showLevel does not work with File Transport

Hello,
I'm using winston for two days.
I think the new option showLevel is not available for File Transport.
(It seems like this option is not transmitted when calling common.log in File.prototype.log function)
Thanks !

Hi @mll75,
Can you send us a small snippet showing that?
Thanks in advance,

Yes pose, Here is a small example:
var winston = require ('winston');
var logger = new (winston.Logger)({
transports: [
new (winston.transports.File)({ filename: 'test.log',
showLevel: false,
json: false,
timestamp: false
})
]
});
logger.info('without level');
Without correction, it writes in the log :
"info: without level"
If I insert a line at file.js (line139) :
showLevel:   this.showLevel,
it seems correct in the log:
"without level"
Be careful, if I remove "timestamp: false", my correction is not enough, I had a remaining "-" in log file:
"2015-02-05T12:46:57.341Z - : without level"

@mll75 yes this is a bug, we need to pass this.showLevel to common.log. Will definitely take a PR! I know there is a test for the console transport that can be used for the test case as well.

I'll add to this, that I'm using a custom formatter, and I can't see to get rid of this. The following, should not display the level, even when I have not included showLevel = false, I believe. Everything else seems to work as requested.
var statsLog = new (winston.Logger)({
transports: [
new (winston.transports.File)({
filename: log_time + "_load_test_stats.log",
json: false,
showLevel: false,
maxsize:<PHONE_NUMBER>0,
timestamp: function() {
var m = moment();
var d = m.format('YYYY/MM/DD HH:mm:ss');
var d2 = m.format('x');
var d3 = d + '|' + d2;
return d3;
},
formatter: function(options) {
// Return string will be passed to logger.
return options.timestamp() +'|'+ options.message;
}
})
]
});

Ah, hadn't upgraded to the latest version. Custom formatter works.

This is fixed by #560


Provide public unsafe EntityRef::get_change_ticks_by_id that takes ComponentId

What problem does this solve or what need does it fill?
I need to get the change ticks on the type-erased component for some entities. Basically EnitityRef::get_change_ticks but with ComponentId instead of T. I currently use the modified get_ticks function. My use-case is identifying all entities with changed components, and serializing these components, hence I cannot know the component type in advance.
What solution would you like?
Solution is to define (unsafe?) public get_chage_ticks_by_id() that takes ComponentId instead of T. It should ideally be implemented for both EntityRef and EntityMut for consistency.
What alternative(s) have you considered?
There is already a get_ticks_with_type function that is pub(crate). One solution would be to make it public.
Another solution would be to make get_ticks public.

You are right it shouldn't. I hastily concluded it should be unsafe when saw that the internal functions are unsafe e.g. get_ticks_with_type
I will start working on a PR


'npm install' not installing dependency

Just a quick note that after following the docs to do "vue-init", then "npm install", running "npm run dev" had an error:
 ERROR  Failed to compile with 1 errors                                                                       6:16:37 PM

This dependency was not found:

* element-ui/lib/theme-chalk/index.css in ./src/js/main.js

To install it, you can run: npm install --save element-ui/lib/theme-chalk/index.css

...and...

ERROR in ./src/js/main.js

  ✘  https://google.com/#q=import%2Fno-unresolved  Unable to resolve path to module 'element-ui/lib/theme-chalk/index.css'
  src/js/main.js:4:8
  import 'element-ui/lib/theme-chalk/index.css';
          ^

✘ 1 problem (1 error, 0 warnings)

Errors:
  1  https://google.com/#q=import%2Fno-unresolved
 @ multi (webpack)-dev-server/client?http://localhost:8080 webpack/hot/dev-server ./src/js/main.js

ERROR in ./src/js/main.js
Module not found: Error: Can't resolve 'element-ui/lib/theme-chalk/index.css' in '/Users/mason/workspace/creator-hub/src/js'
 @ ./src/js/main.js 4:0-46
 @ multi (webpack)-dev-server/client?http://localhost:8080 webpack/hot/dev-server ./src/js/main.js
Child html-webpack-plugin for "index.html":
     1 asset
       [0] ./node_modules/html-webpack-plugin/lib/loader.js!./index.html 667 bytes {0} [built]
       [1] ./node_modules/lodash/lodash.js 540 kB {0} [built]
       [2] (webpack)/buildin/global.js 488 bytes {0} [built]
       [3] (webpack)/buildin/module.js 495 bytes {0} [built]

Just to keep moving along I tried doing the npm install of that css, but it reports:
npm ERR! code ENOLOCAL
npm ERR! Could not install from "element-ui/lib/theme-chalk/index.css" as it does not contain a package.json file.

I'll comment again if I have a chance to dig in and see what's up, but I wanted to let you know. Thanks.

Got it, I changed the element-ui spec in package.json to "^2.0.4" and it seems to be working. Looks like there was an issue in an earlier 1.x that got pulled in? Not sure really, as I don't know anything about Element (yet), but it did compile and load; so far, so good. Thanks!

I have fixed the issue now and its working. Please check PR

Thanks! Interesting though, after updating to use "^2.0.4" it works with chalk, but not with the default that's in your PR. I'll look into it a bit more.


Echarts and Google Chrome compatibility

Hello,
There is something I have noted with echarts and my Google Chrome Version 54.0.2840.98 (64-bit) browser on a MacBook.. The moment I open a page I created that has echart graphs my computer fans kick into action and have way increased activity.. wondering if there's a compatibility issue..
Thanks in advance.. great charts..

问题简述 (One-line summary)
版本及环境 (Version & Environment)

ECharts 版本 (ECharts version):
浏览器类型和版本 (Browser version):
操作系统类型和版本 (OS Version):

重现步骤 (Steps to reproduce)

期望结果 (Expected behaviour)
可能哪里有问题 (What went wrong)
ECharts配置项 (ECharts option)

option = {

}

其他信息 (Other comments)

What's your option and graph look like ?