Fix regex expression

[Fix regex expression]
It can happen that you directly modify the storage with any program and it adds or alters the structure by adding new files that can be treated incorrectly. It happened to me when I used the draktable directly on the SD where the images of my gopro were that it had added .xml to the JPG and GRP files
And by the way, it does not allow it to be stricter in the file names so that there are no combinations such as the following:
GOPR0964.GPR.xmp
GX011014-MP4
GX011014.MP4.old
Type:

[X] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Documentation

Camera:

[X] GoPro
[X] Insta360
[X] DJI
[ ] Android
[ ] This PR adds a new camera

Component:

[ ] Core logic
[ ] Import
[ ] Merging
[ ] Firmware Update
[ ] Video Manipulation

Checklist before approval:

Linter pass
Build pass

Note that .MP4 .JPG etc... are hardcoded thru the program and those .old etc... files would be incorrectly parsed.

I didn't understand what you meant before
look at the execution of a test
~/mmt $ ./mmt import -v -c gopro -i /mnt/datos/test/ -o /mnt/datos/test.out
🎥 [HERO11 Black]:
📹 FW: H<IP_ADDRESS>.00 SN: C3471324581852
💾 485 GB/1.1 TB (44.17%)
Looking at 100GOPRO
GOPR0964.GPR: 17.44 MB / 17.44 MB [==========================================================]  ✔
GOPR0964.GPR.xmp: 7.29 KB / 7.29 KB [==========================================================]  ✔
GX011024.MP4.bak: 74.31 MB / 84.26 MB [==================================================>-------] 0s

+----------------+---------------+--------+
| FILES IMPORTED | FILES SKIPPED | ERRORS |
+----------------+---------------+--------+
|              3 |             0 |      0 |
+----------------+---------------+--------+

there are files that are processed and should not be

When dealing with chaptered video the code looks at GP\d+.MP4 vs GP\d+.MP4$.

echo "GP231120.MP4" |grep -P "GP\d+.MP4$"
run ok
What exactly is the problem?
See GoPro Camera File Naming Convention
https://community.gopro.com/s/article/GoPro-Camera-File-Naming-Convention?language=en_US

Thought the chaptered regex looked until the end, no problem then.
Thanks!

Do you want me to make any changes to this code?
something else is missing

What's missing?

Me referia si faltaba algo por añadir o corregir en este pull para fusionarlo en la rama principal. El traductor lo carga el diablo :-(

No creo que no.


Script to convert Konsole colorscheme to Alacritty color scheme

I made a script, putting it here so that maybe it's useful for someone. (I don't know where else to put it.)
It needs python 3 and pyyaml. You run it on files like python3 converter.py /share/konsole/Linux.colorscheme.
#Note this does not generate the 'cursor:' section becuase I don't know what would correspond to that.
#I also dont know if theres an equivalent light/dark for the foreground/background so those are ignored.
#Also alacritty does not have "dark" colors. (?)

import sys
import configparser as confp #ini format
import yaml
import collections

#fixes extraneous stuff when yaml-ing a defaultdict
from yaml.representer import Representer
yaml.add_representer(collections.defaultdict, Representer.represent_dict)

confpath = sys.argv[1]
c = confp.ConfigParser()
c.read(confpath)

colormap = {
	'Color0':'black',
	'Color1':'red',
	'Color2':'green',
	'Color3':'yellow',
	'Color4':'blue',
	'Color5':'magenta',
	'Color6':'cyan',
	'Color7':'white',
	'Foreground':'foreground',
	'Background':'background',
	}

typemap = {
	'Intense':'bright',
	'Faint':'dark',
	'#default#':'normal'
	}

def stype(s):
	for k,v in typemap.items():
		if s.endswith(k):
			return v
	return typemap['#default#']

def ctype(sb):
	for k,v in colormap.items():
		if s.startswith(k):
			return v
	raise NotImplementedError

def hexcolor(s):
	tohex = lambda x: "{0:02x}".format(int(x))
	return "0x%s" % "".join([tohex(x) for x in s.split(",")])

data = { 'colors' : collections.defaultdict(dict) }

for s in c.sections():
	if "color" in c[s] and stype(s) is not "dark":
		if ctype(s) in ["foreground","background"]:
			if stype(s) == "normal":
				data['colors']["primary"][ctype(s)] = hexcolor(c[s]['color'])
			else:
				pass
			continue
		data['colors'][stype(s)][ctype(s)] = hexcolor(c[s]['color'])

print(yaml.dump(data, default_flow_style=False))

Example output is:
colors:
  bright:
    black: '0x686868'
    blue: '0x5454ff'
    cyan: '0x54ffff'
    green: '0x54ff54'
    magenta: '0xff54ff'
    red: '0xff5454'
    white: '0xffffff'
    yellow: '0xffff54'
  normal:
    black: '0x000000'
    blue: '0x1818b2'
    cyan: '0x18b2b2'
    green: '0x18b218'
    magenta: '0xb218b2'
    red: '0xb21818'
    white: '0xb2b2b2'
    yellow: '0xb26818'
  primary:
    background: '0x000000'
    foreground: '0xb2b2b2'

to which you should add something like
  cursor:
    text: '0x2E3440'
    cursor: '0xD8DEE9'

You could add Konsole as an import option for https://terminal.sexy. I think this is a bit too specific to be useful for a broad audience (too many different terminals with too much different colorschemes).
But if you have a terminal theme you think that would be cool for alacritty you can add it to the wiki!

Yeah. I mainly used this to confirm that the color issues I was having compared to Konsole weren't actually color issues, it was just a different color scheme.

Well, I didn't know about https://github.com/stayradiated/termcolors , I guess if I were to add "konsole" import it would probably be able to do the conversion.


chore: update Error message for bad bigtable.env variable

At some point, the allowed settings were changed from "prod" and "emulator" to "cloud" and "emulator".
This change updates the error message associated to the illegal argument exception.

Yikes! Sorry about that and thank you for the fix!
However I was wondering if you are modifying that property directly? It's meant to be controlled via a maven profile:
https://github.com/googleapis/java-bigtable/blob/master/google-cloud-bigtable/pom.xml#L260
ie: mvn verify -P bigtable-prod-it

Hope you don't mind, I amended your commit to comply with conventional commit messages

Thanks for the work in checking this in!
We have a need to create the test tables within a VPC Service Controlled security perimeter. This requires us to customize the environment variables.

May I suggest to use a combination?
‘’’bash
mvn verify -Pbigtable-prod-it -Dbigtable.project=... -Dbigtable.instance=... -Dbigtable.table=....
‘’’
The bigtable.env property wasn’t meant to be set directly

Perfect, thank you for the advice. I will modify the invocation accordingly.


install_cortex-cpp has a few errors that block installation

On line https://github.com/janhq/cortex/blob/f5e6b2f48f61c74e8a81fca11b4ea7bfe39580fa/cortex-cpp/install.sh#L57 the script fails if /tmp/cortex-cpp.tar.gz does not already exist (probably needs to be rm -f). On line https://github.com/janhq/cortex/blob/f5e6b2f48f61c74e8a81fca11b4ea7bfe39580fa/cortex-cpp/install.sh#L66 the script fails when cp tries to copy engines from the tarball (a directory). A possible fix is to add -r, however I'm not sure that is always going to be correct. It might make more sense to run tar -C /usr/local/bin --strip=1 /tmp/cortex-cpp.tar.gz instead of using the for loop.

The install.sh file has been deprecated.


[CodeCompletion] Don't check 'InvalidAsyncContext' for imported globals

InvalidAsyncContext depends on the decl context. That may case "sticky" not-recommended If it's cached for a non-async context.
To workaround this, stop checking InvalidAsyncContext when collecting completion items for caching. Also consistently use the SourceFile as the decl context to avoid decl context specific behavior.
rdar://78315441

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 0ce1faa6c76cbe3807c3114a40097de2b2be9f72

@swift-ci Please test Linux


Cannot install osu-stable and osu-lazer at the same time

I'm not sure what's going on :(
(installed via home-manager)
{config, pkgs, inputs, ...}: {
  home.packages = [
    pkgs.osu-lazer-bin
    (inputs.nix-gaming.packages.${pkgs.stdenv.hostPlatform.system}.osu-stable.override {
      location = "${config.xdg.dataHome}/osu-stable";
    })
  ];
}

nixos-rebuild --flake . switch
error: builder for '/nix/store/qh4c8byaicwdfdx656k7kf227gjjl8cj-home-manager-path.drv' failed with exit code 2
error: 1 dependencies of derivation '/nix/store/d1c12asygbw7c2ccxrqgyd329vcazl54-home-manager-generation.drv' failed to build
error: 1 dependencies of derivation '/nix/store/8x5a9a0vc79g0wnzjhbhn5hxrbpnvvmp-user-environment.drv' failed to build
error: 1 dependencies of derivation '/nix/store/6f4g0d78ldryr0cwv3j1gs8i9nld765w-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/1h4hzk2jz625kxmgpsshkpa1hmqq4qmw-nixos-system-Aristotle-24.05.20240102.bd645e8.drv' failed to build

nix log /nix/store/8w7yz2w0fbcnxih200gi04b1bywqx5bd-home-manager-path.drv
warning: different permissions in `/nix/store/qmilf5gjzf09f65xc1nj32s9jdf514xq-osu-stable/share/icons/hicolor/16x16/apps/osu!.png' and `/nix/store/x1w93kxfizzxwjf8xfk6l26i0kh1li2y-osu-lazer-bin-2023.1229.0/share/icons/hicolor/16x16/apps/osu!.png': 0444 <-> 0555 at /nix/store/w1qzwvqwv127nblp4hjydp8c9hhmb7zk-builder.pl line 103.
error: collision between `/nix/store/x1w93kxfizzxwjf8xfk6l26i0kh1li2y-osu-lazer-bin-2023.1229.0/share/icons/hicolor/16x16/apps/osu!.png' and `/nix/store/qmilf5gjzf09f65xc1nj32s9jdf514xq-osu-stable/share/icons/hicolor/16x16/apps/osu!.png'

I think both icons should come from osu-mime. @dixslyf do you know more about this?

I think at the moment the only difference is that ours is updated daily.

You sent an entire flake file, what exactly am I looking at? and why would overlays be a cleaner solution?

Oh, I think you misunderstood. osu-lazer-bin has been packaged here long before it existed in Nixpkgs. We just never removed it. Also no point in removing it now, since it works fine.

If I understand it correctly, the collision happens between Nixpkgs' osu-lazer-bin and our osu-stable, right?
Nixpkgs' osu-lazer-bin uses the icon from the AppImage (src) whereas we generate the icons using SVG files from the osu-web repo (in our osu-mime package, which is shared by our osu-lazer-bin and osu-stable). In both Nixpkgs' osu-lazer-bin and our osu-stable, the icons are placed under share/icons/hicolor/16x16/apps/osu!.png (and the respective locations for other resolutions of course). Since the icons are different (permissions and contents), we get the collision error.

Also BTW, quick question: what is the difference between this repo's osu-lazer-bin and nixpkgs's osu-lazer-bin?

I think at the moment the only difference is that ours is updated daily.

Another difference is that the osu! packages here handle MIME types. AFAICT, the ones in Nixpkgs don't.
In the long-term, it might be worthwhile to just override Nixpkgs' package definition, though it might not be trivial since the Nixpkgs' package is conditional and depends on architecture and OS.

For a fix, we could rename the icons here so that the collision doesn't happen, but that seems more like a workaround.
Another possibility is to use the icons from the AppImage so that the collisions don't happen (technically, they'll still happen, but will be ignored as the file contents are the same). Assuming the icon in the AppImage is the same as this, we wouldn't have to go through the hassle of extracting it from the AppImage. This approach feels flimsy though — if the icon in the Nixpkgs package changes, we'll get the collision again.
I'm sure there are better ways to resolve this, but I can't think of one right now. Ideas?


Remove relaunched Zen banner

This appears to be confusing people now, so we need to remove it from the login page.

This is live now, thanks @josmas


added glob-importer to node-sass build command

Description
Added the node-sass-glob-importer in the command line in package.json.
This was a bug, now the glob is able to be parsed by SCSS in src/assets/scss/app.scss
Testing
npm run build:css
To do after merging this PR (optional)
Write awesome code in SCSS & fix the import the components SCSS with a glob.

Yo noob mistake brb


x/crypto/acme/autocert: Listener should set NextProtos: []string{"h2"} to enable HTTP2.

What version of Go are you using (go version)?
go version go1.8.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN="/home/johan/gows/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/johan/gows"
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build589265406=/tmp/go
-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
I deployed a server using autocert.NewListener to acquire a certificate. I was surprised to see the server did not have HTTP2 enabled.
What did you expect to see?
I expected the server to have HTTP2 enabled.
What did you see instead?
The server did not have HTTP2 enabled.
There's a fairly simple fix for this, simply adding
NextProtos:     []string{"h2"},

after line 72 in listener.go enables HTTP2 support on the server.

If this is indeed considered a bug worth fixing, I would like to fix it.

While that would work for you, I wonder if it would break
(1) anybody not expecting HTTP/2 (people not using net/http?) or whether
(2) there are net/http paths where the http2.ConfigureServer path isn't invoked. I worry about the case where the client thinks it can do h2 but the server isn't wired up to accept h2.
For (1), we can just document it. It's new enough and I imagine everybody is using net/http for it. And if you're running, say, an SMTP server, the client won't be trying to do h2.
For (2), I'd just want to audit the various paths again. I forget how the auto-http2.ConfigureServer code works.
But yeah, no real objections.

I will work on this

Just to clarify what needs to be done for (2), would you like me to audit all net/http functions accepting net.Listener to verify that they all call to http2ConfigureServer at some point?

Yes.

Serve in net/http/fcgi accepts a net.Listener but does not appear to call to http2ConfigureServer, is this a case we should be concerned about? All other exported functions in net/http and subpackages call to http2ConfigureServer.


NG: Cast medalID inputs for medal arguments

This can cause weird issues where the medal ID is a string and therefore technically does not exist:

This also fixes the input type from STRING to NUMBER since the other block for medals uses NUMBER as well.

LET'S GO IT GOT MERGED


Use soft tabs in generated BUCK files.

The generated BUCK files currently use hard tabs for formatting instead of soft tabs. It's visually distracting when the IDE's show whitespace chars option is enabled.

These are generated files. You can turn off that option in your ide for now. Will probably handle this at a later time. Not really a high priority sorry


Chore: Update connectionStatus' doc comment

Remove true, since it doesn't return a bool.
Status
READY
Breaking Changes
NO
Description
Remove true from doc comment of connectionStatus method, since it doesn't return a bool.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[x] 📝 Documentation
[x] 🗑 Chore

@AleksandarSavic95 connectionStatus is returned by the getter connectionStatus, so you could improve the doc by specifying it and then I'll merge the changes made by you. Thanks for your contribution

@AleksandarSavic95 connectionStatus is returned by the getter connectionStatus, so you could improve the doc by specifying it and then I'll merge the changes made by you. Thanks for your contribution

Hello! :) Well isn't that already written?
in line 167-169, we have
  /// [InternetConnectionStatus.connected].
  /// [InternetConnectionStatus.disconnected] otherwise.

That specifies the return values, right?

@AleksandarSavic95 Thanks for your contribution


Having a directory called, e.g., holiday with some go files and their corresponding test files, I called the go test -cover from the parent directory. Depending on directories arguments, I have different percentage values, which wasn't occurring on go < 1.22.0.
What did you see happen?
DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./holiday/
ok  	gitlab.in.registro.br/dev/br/core/holiday	(cached)	coverage: 97.6% of statements

DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./h*
...
ok  	gitlab.in.registro.br/dev/br/core/holiday	0.005s	coverage: 47.3% of statements

DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./...
...
ok  	gitlab.in.registro.br/dev/br/core/holiday	(cached)	coverage: 23.5% of statements
...

What did you expect to see?
A consistent percentage value (in this case, 97.6%) for all commands:
DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./holiday/
ok  	gitlab.in.registro.br/dev/br/core/holiday	(cached)	coverage: 97.6% of statements

DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./h*
...
ok  	gitlab.in.registro.br/dev/br/core/holiday	0.005s	coverage: 97.6% of statements

DEV=dev go test -tags dev -cover -buildvcs=false -ldflags '-s -w' ./...
...
ok  	gitlab.in.registro.br/dev/br/core/holiday	(cached)	coverage: 97.6% of statements
...

This is almost certainly an effect of recent changes in -coverpkg:

#58770 (Go 1.22.0)
#51430 (Go 1.19)

See previously:

#23883

(attn @thanm)

@dev-gto would you kindly post a pointer to the code that you are working with, so that I can reproduce the problem on my machine? Thanks.

https://github.com/golang/go/issues/62212 might be related

GOEXPERIMENT=nocoverageredesign go test -cover ./... does appear to provide the expected coverage result, matching running go test -cover for each package separately.  So this does seem related to the coverage redesign.
I've confirmed this with both @dev-gto's example above and a large private monorepo where I also encountered the issue.

Indeed, GOEXPERIMENT=nocoverageredesign returns the expected values on my original files

Code coverage has been broken for about a year now... Please can someone make this a priority to fix.

I came here because I just ran into this as well on 1.22.2

1.23.1 and 1.22.7 still have this issue, although it was closed.

sorry about that, actually it is working

is it? Can't see the difference

https://go-review.googlesource.com/c/go/+/627316 Seems to include a fix for this, but it has now a merge conflict.

but it has now a merge conflict

This is expected behavior given the way Gerrit works. It's a 2-patch sequence, with 627316 dependent on 627315; since 627315 hasn't been submitted yet 627316 will always show "merge conflict". Yes, it's confusing; virtually every developer new to Gerrit/Go puzzles over this same thing.

Was this fixed in Go 1.23?


improved start/stop functionality

Also, can you move all the start/stop logic on its own file?

Yeah I'll move it all to its own file. I suppose I could keep the multi server functionality and maybe have the kernel write a file with the port number too? That way if we ever have more servers again, we can have them write their own respective files too.

No file with a port number, pick that up from the config for each of those.
Il giorno mar 14 apr 2015 alle ore 15:52 Darragh Hayes <
<EMAIL_ADDRESS>ha scritto:

Yeah I'll move it all to its own file. I suppose I could keep the multi
server functionality and maybe have the kernel write a file with the port
number too? That way if we ever have more servers again, we can have them
write their own respective files too.
—
Reply to this email directly or view it on GitHub
https://github.com/nearform/nscale-client/pull/46#issuecomment-92856936.

Sure. Are the api, web and kernel the only possible servers we would ever have?

yes. But API and web are not started now, only the kernel is.

I understand I was just asking so I know how I should fetch the relevant config for each 'server' passed into the start/stop functionality.

@mcollina If you can please review the latest commit

@mcollina if you can review now please

Great! :dart:


Pascal-50s

Hello,
I tried to download the pascal-50s dataset using the original link you provided, but I cannot access the link somehow.
https://vrama91.github.io/cider/

Thank you so much! :)


Force filterKeys and mapValues to return a Map.

Description
Scala 2.13 makes both of these return views rather than strict maps. We don't need view semantics in any of these places, so we're safe to use the proposed alternative code as seen in https://docs.scala-lang.org/overviews/core/collections-migration-213.html#what-are-the-breaking-changes.
Related issue and scope

[X] I opened an issue to propose and discuss this change (#4741)

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[X] I added tests to cover my changes.

Codecov Report

Merging #4833 into master will decrease coverage by 6.47%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #4833      +/-   ##
==========================================
- Coverage   82.35%   75.88%   -6.48%
==========================================
  Files         197      197
  Lines        9019     9016       -3
  Branches      370      350      -20
==========================================
- Hits         7428     6842     -586
- Misses       1591     2174     +583

Impacted Files
Coverage Δ

...rg/apache/openwhisk/core/entity/WhiskPackage.scala
96.61% <100%> (-0.17%)
:arrow_down:

...ache/openwhisk/core/database/DocumentHandler.scala
87.68% <83.33%> (ø)
:arrow_up:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0% <0%> (-100%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-96.23%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-93.9%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-92.31%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-77.78%)
:arrow_down:

... and 9 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 acf4cda...44c2563. Read the comment docs.

Codecov Report
Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 75.88%. Comparing base (acf4cda) to head (44c2563).
Report is 361 commits behind head on master.

Files with missing lines
Patch %
Lines

...ache/openwhisk/core/database/DocumentHandler.scala
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4833      +/-   ##
==========================================
- Coverage   82.35%   75.88%   -6.48%
==========================================
  Files         197      197
  Lines        9019     9016       -3
  Branches      370      350      -20
==========================================
- Hits         7428     6842     -586
- Misses       1591     2174     +583

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


sso ga

@bdecoste, are there any JIRA associated with these changes?  If so could you please amend the commit message?  If not could you please create one and amend the commit message?  Thanks in advance.


Read-only and Draggable fix.

simplify code for nullable control event assignment
make textbox readonly and dragpanel visible controlled in code behind.
Use compile binding in demo.

@heartacker Please help to review.

GOOD & I have no more


Implement mv command.

I'll do it !!
Have I to include the mv.py in the commands folder no ?

I'll do it !!
Have I to include the mv.py in the commands folder no ?

Yes. you have to

It's done take a look the #5 PR !


adding date types to meta fields

this lets you put more interesting things into metas, which in turn let you do clever things with relationships while not having to fully-inflate those relationship children.


Apply DependentUpon Convention Only to .resx Files

Description
Fix #4740, build or runtime failures that arise because non-resx resources get embedded with an unexpected name.
Customer Impact
Builds fail with CS1508 or applications fail to extract a resource at runtime. Roadblock to adopting the final 3.0 SDK and to migrating to .NET Core 3.0 for applications with embedded resources (UI applications in particular).
Reported by multiple internal partners (CoreFX, EF) and customers.
Workaround
Users can explicitly opt out of the new behavior back into preview8 behavior with a property
<EmbeddedResourceUseDependentUponConvention>false</EmbeddedResourceUseDependentUponConvention>

Regression?
Yes, builds fail in cases that passed before. Introduced with #4597, a new feature to support (resx) resources without project-file impact.
Testing
Automated tests, including a new one for this specific case. e2e validation in failing scenarios using privates.
Risk
Low. Targeted fix in a conditioned codepath, so the existing workarounds will continue to work and can mitigate any problems.

Taking the suggestion made by @nguerrera to restrict dependentupon convention to .resx files.
https://github.com/microsoft/msbuild/issues/4740#issuecomment-532962819

Restricting to resx makes sense because that is the case that would have been persisted with DependentUpon in a classic project. I'm not aware of anything that would have put DependentUpon for non-resx in your project unless you did it manually so I don't think it should have been part of the convention to do so.

Added a check on the Type metadata and file extension to prevent dependent upon convention from being used on non-resx files.
Users can manually assign the Type metadata when including embedded resources in their projects like so:
<EmbeddedResource Include="MyResource.txt" WithCulture="false" Type="Non-Resx" />
If the metadata isn't set, it should automatically be populated by the time the task runs.
In case the Type metadata is missing for whatever reason when the task runs, a check on the file's extension will be made to determine whether or not it's a resx file.

Tests for the convention are failing. I assume they are test defects at this point?

They are failing because the tests don't set the Type metadata, yes. Unfortunately, that metadata isn't documented in the CreateCSharpManifestResourceName docs, so I'm wavering in my commitment to relying on it. Are our tests representative of real-world calls, or special?
I am seeing some third-party calls to the task: https://github.com/search?l=XML&q=CreateCSharpManifestResourceName&type=Code

Two relevant offline conversations yesterday:

@nguerrera is now willing to take the double-checked extension as a belt-and-suspenders approach to direct callers of the task that don't use the usual common.targets flow to assign Type for inputs.
Bar check for 16.3.x passed, so we need a build.

I'm going to merge.

Now that the rules for manifest resource name selection have changed, refer to this dotnet docs issue for a detailed explanation on how this selection works now.


Add new mempool panels + existing mempool panels to new dashboard

Adds some new panels for the transaction pool into a new dashboard reth-mempool.
Copies existing transaction pool panels into new reth-mempool dashboard.
Makes it easier to review future changes to transaction pool panels that can now be reviewed in isolation.

As discussed with @shekhirin, updating the grafana dashboard is painful so long as it is not modular. Every new panel changes at least the grid position of existing panels placed lower than it.


manually injected url gets deleted

When I inject an url that is already in the spiderdb and not in the sitelist the page gets downloaded successful and then deleted few seconds later.

I haven't seen this behavior, but I have been mostly loading warc files for my testing from Common Crawl so I haven't played too much with the spidering yet.

sorry - I had a url-filter that deletes pages after 3 ERRORS


feat: read scrape instructions from app root dir

Now that https://github.com/dhis2/maps-app/pull/3072 has also been merged, the current PR can go in too


Improve handling of extra attributes with empty values

In the web UI, improve the handling of extra attributes with empty values so that:

Entries where both the attribute and value is empty are removed when sending data to the server.
The attribute field is mandatory when a value is entered.

Fixes #882.

LGTM.


Why does the atom.io page say this is the only working web browser?

This is not true.  It has said this for a long time.

my bad i haven't checked and yea i've practiced poor coder etiquette on this one.
at the time the iframe based browser of yours wasn't working (although the UI was excellent) and the statement is obviously no longer true so i've unpublished the package.
hope you're doing well. thanks, -gstack

I've effectively unpublished web-browser also.  I've handed the reins
over to browser-plus.
On Mon, Jun 8, 2015 at 9:34 PM, Mac Gray<EMAIL_ADDRESS>wrote:

my bad i haven't checked and yea i've practiced poor coder etiquette on
this one.
at the time the iframe based browser of yours wasn't working (although the
UI was excellent) and the statement is obviously no longer true so i've
unpublished the package.
hope you're doing well. thanks, -gstack
—
Reply to this email directly or view it on GitHub
https://github.com/gstack/atom-browser-webview/issues/15#issuecomment-110221263
.


Create native Kafka Sink and Source connectors

Create Kafka classes which can use Element generators to transport data using Kafka.

Issue #824 added an operation to add elements to Gaffer from Kafka using Flink. I will update this issue to be to export to Kafka.


Add event for token-burn

Transfers from the zero address are already interpreted as mint events, and cause an emit of token-mint. Similarly, transfers to the zero address should be interpreted as burn events, and cause an emit of a new token-burn event.

f058ac4ac3700bbf84978cc3f5994bac827d1034


JS: improve support for destructuring and default values

This PR contains three related, but independent changes, all motivated by https://jira.semmle.com/browse/ODASA-7715.
The first change is a def-use bugfix that removes the direct step from x to y in var { y } = x;, the implicit step from x.y to y is still performed in the dataflow graph.
The other changes add support for default values in parameters and destructuring assignments. The type inference already had support for default parameter values.
Please note that the introduction of default parameter values has the effect that parameters now have a predecessor in the dataflow graph, which may be surprising, as seen in the RangeAnalysis.qll change.
Implementationwise, there is a small nit: unlike default parameter values, default values in destructuring assignments can not be implemented smoothly in the def-use graph at the moment, so they are implemented in the dataflow graph where destructuring are handled already. The reason for this is that a single VarDef represents multiple sources and targets, and the relation between each source and target is lost when VarDef::getSource() is used.
A larger evaluation is on the way, but a preliminary evaluation shows plenty of result improvements at no performance cost.

All comments addressed, as a rebase, see https://github.com/Semmle/ql/compare/9b3381e0548d6887ebcb9333d22cccb638209012..5ad83360be73c21999425dfc3e8ac053817ad574 for the pretty diff.
A big-apps.slugs evaluation shows that this still scales. Do we want a full evaluation on default.slugs?

LGTM. I would prefer a full evaluation, just because I've been caught out by unexpected corner cases in this area of the language before.

(The full evaluation is just more of what we already saw in the partial evaluation.)


Accessing data_log.ndc and data_es.ndc

Hi! This is not so much an "issue", but a request for help.
I am giving Neware the benefit of the doubt and really hoping that they are not interpolating data. I wonder if the missing data is contained in the data_log.ndc and data_es.ndc files? These are currently not converted from binary in this package.
I tried to convert these by adapting the functions that you have written for the other files, but I am not sure what the numbers you have used in the conversions should be e.g. Record length, header, and the numbers used here and here.
How did you find these numbers when you were creating the existing functions?
Have you tried to convert these other ndc files in the later versions of the software that have the missing data issue talked about here?

Hi, the data_log.ndc and data_es.ndc files seem to be small, typically <100kb. I'm not sure what's in them, but it doesn't seem like enough space to store the missing data (which for some of my files should be >10MB).
The numbers you referred to come from using a hex editor to look for repeated patterns in the binary data. They are memory offsets, field lengths, starting positions, etc. If you look at data_es.ndc, it's mostly zeros...
The files in question are generally much smaller than similar experiments from other machines, which is a clue. The second clue is if you plot the output of BTSDA, you will see the expected level of noise from the instrument in the voltage and current (all these points are saved in the file). However, the capacity, which I claim is not being saved, will be perfectly linear with no noise. If I make the same plot for an older file, the file will be larger and there is noise in the capacity field.
I suspect this is related to the middle machine, and I first noticed it with version 4S_<IP_ADDRESS>_20230308_185745.

I think the capacity not being saved is ok, we could deal with that! It is the timestamps not being saved that are truly frustrating in some circumstances (mainly the beginning and end of steps).
Thanks for the reply anyway, I appreciate it!

They seem to save a few time values at the beginning and end of most steps. Maybe there is a setting somewhere that controls this?

Hi, for the record data_log.ndc seems to contain information about alarms or warnings during the test. BTSDA shows a little red light symbol in data where alarms happened:

If I unpack an .ndax file, remove data_log.ndc, re-pack it and open it, then those symbols no longer shows in BTSDA.
I do not think I have seen any data_es.ndc files, I guess those are specific to some equipment or BTS versions.


No mention of permissions needed

This page should acknowledge the permissions required to work with emails. I am D365 System Administrator. Historically this is "god mode" and allows full access. I could not edit email templates. A few weeks ago, trying to do so caused the generic "Sorry, something went wrong" error so I was granted "Survey Services Administrator" which resolved it. More recently, the ability to swap templates or save/save as was greyed out. I now need "Survey Owner" too. I found this by trial and error - please add a section on permissions! :)

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

ID: dbe130e3-5f7e-f3af-4a5b-3dab02d7ec28
Version Independent ID: 464ab2a9-0cb2-a920-b32b-a19ad9c8e8ab
Content: Send a survey by using email - Microsoft Forms Pro
Content Source: fp-docs/send-survey-email.md
Service: forms-pro
GitHub Login: @sbmjais
Microsoft Alias: shjais

We have a document for required privileges at: Privileges required. Please see if this is the information you were looking for.

Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.


Release version 0.1.2

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

Comparison is base (bafac08) 100.00% compared to head (75f4f46) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #152   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          301       301
  Branches        60        60
=========================================
  Hits           301       301

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


FFDC to collect PLDM strace log ?

strace -p `pidof pldmd`

Ideally, this needs to be collected in the event when PLDM hung, but the log collector won't have that logic to check..
so if we can collect this one? run the command and then CTRL +C after 5-10 seconds
@peterp-ibm  can u check on it.

Probably not a good idea to put this in for generic log collection. only on failure.. but lets see if we can learn something from it to improve the script

To get strace output need to use option '-o' or pipe stderr to a file
strace -p $(pidof pldmd) -o /tmp/pldmd.txt
strace -p $(pidof pldmd) 2> /tmp/pldmd.txt
Now the issue is how to interrupt/stop strace without ffdc code having to look at the command itself.

Done! https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/51705
Committed for review and merge.


Active scene will be resign

Please check and explain to me why app triggered function sceneWillResignActive(_:) when I showed image picker.
Thanks team.

I reviewed and realize that this issue might be cause of iOS present popup to request access camera and photos.

@luatvudinh I have no idea to be honest :)

@luatvudinh I have no idea to be honest :)

I think that from iOS 13 the scenes exist and when we present YP image picker, the current scene will be resigned and replacing by new scene. That is why the function sceneWillResignActive(_:) be triggered.


Question: best way to handle authorization - i.e. who can edit a file/post for routes

I was also wondering if you guys have any plans / ideas for how to handle authorization? I'm curious if there's any best practices you guys operate with.
I was wondering if I should just implement jsonapi-server and write a seperate front-end utilizing jsonapi-client? Am I asking for too much customization out of an already very solid code-base?
Thanks,
Paul

looks like I just should just use the server and client - sorry for the spam!


[Clean-up] Client simplifications after server-side ?q= redirect

Description
Simplify the routing and telemetry logic on the client following the new server-side redirection of full text queries introduced in https://github.com/Qwant/erdapfel/pull/1066, following great advice by @amatissart

Remove the need for a client side redirection of /?q= queries as start url (as it's now done exclusively on the server)
Remove extra telemetry fields related to tracking this special case
Use 'direct-search' as a new client param to track this case using the generic fields

Note: I removed the integration test case that checked the client-side redirection on app startup, because it couldn't work anymore. Instead we should now test the server-side redirect with the 3 cases (no result, single, multi). We could either unit test the middleware in isolation, or find a way to test a full HTTP request like other tests in server.js… but this implies mocking the underlying call to the autocomplete API.

I'm reverting back to a draft until I add the tests for the server-side redirection.

I'm reverting back to a draft until I add the tests for the server-side redirection.

ok :+1: Note that some server-side mocked requests are already defined in https://github.com/Qwant/erdapfel/blob/master/tests/integration/server_start.js using "nock". Maybe the same mechanism can be used here.

@amatissart

Note that some server-side mocked requests are already defined

Yup I've seen that :+1: I started adding stuff there, it works fine, things should go smoothly :)


Explore web3 and solidity

https://medium.com/coinmonks/solidity-and-web3-js-141115b0f8c5
my friend, Mehul told me about this.  Want to check it out.

also see https://ide.zilliqa.com/#/


Nuxt 3 + Vanilla Extract Vite nuxt build Error

Describe the bug
Running yarn build (nuxt build is the actual command) in a project that uses the Vanilla Extract Vite plugin produces this error:
▶ yarn build
yarn run v1.22.15
$ nuxt build
Nuxt CLI v3.0.0-27255771.f78ec93

 ERROR  [vite:css] /Users/timothymorris/Projects/vanilla-extract-ssr-error/styles.css.ts$$$nuxt3-vanilla-extract-ssr-error.vanilla.css:1:10: Unknown word
file: styles.css.ts$$$nuxt3-vanilla-extract-ssr-error.vanilla.css

 ERROR  /Users/timothymorris/Projects/vanilla-extract-ssr-error/styles.css.ts$$$nuxt3-vanilla-extract-ssr-error.vanilla.css:1:10: Unknown word

  at Input.error (node_modules/postcss/lib/input.js:123:16)
  at Parser.unknownWord (node_modules/postcss/lib/parser.js:518:22)
  at Parser.other (node_modules/postcss/lib/parser.js:149:12)
  at Parser.parse (node_modules/postcss/lib/parser.js:59:16)
  at parse (node_modules/postcss/lib/parse.js:11:12)
  at new LazyResult (node_modules/postcss/lib/lazy-result.js:133:16)
  at Processor.process (node_modules/postcss/lib/processor.js:36:12)
  at compileCSS (node_modules/vite/dist/node/chunks/dep-85dbaaa7.js:20288:10)
  at processTicksAndRejections (internal/process/task_queues.js:95:5)
  at async Object.transform (node_modules/vite/dist/node/chunks/dep-85dbaaa7.js:19876:50)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Link to reproduction
Repo (Reused from #445, though I don't believe these issues to be related)
System Info
Output of npx envinfo --system --npmPackages @vanilla-extract/css,@vanilla-extract/webpack-plugin,@vanilla-extract/esbuild-plugin,@vanilla-extract/vite-plugin,@vanilla-extract/sprinkles,webpack,esbuild,vite --binaries --browsers:
  System:
    OS: macOS 12.0.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 49.77 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    Yarn: 1.22.15 - /usr/local/bin/yarn
    npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
  Browsers:
    Chrome: 95.0.4638.54
    Firefox: 89.0.1
    Safari: 15.1
  npmPackages:
    @vanilla-extract/css: ^1.6.3 => 1.6.3
    @vanilla-extract/vite-plugin: ^3.0.0 => 3.0.0

I've updated the example repo with a static css file for comparison in behavior, and upgraded all the deps. Still seeing the above error when trying to build the project.

same error

After a bit more investigation, it appears that during build, this code is being fed into postcss:
import { injectStyles } from '@vanilla-extract/css/injectStyles';                                                                                15:33:58

const inject = (css) => injectStyles({
  fileScope: {"filePath":"styles/reset.css.ts","packageName":"my-app"},
  css
});

inject("/* my css here, omitted for brevity */");

import.meta.hot.on('vanilla-extract-style-update:styles/reset.css.ts$$$my-app', (css) => {
  inject(css);
});

which appears to match the output of @vanilla-extract/vite-plugin/src/index.ts lines 90-103.
Ultimately its become clear that this is a Nuxt 3 issue not a vanilla-extract issue (as this works perfectly with vite out of the box), but does anybody have any ideas on what configuration might cause postcss to receive the above code (rather than what I assume is the output of running that code)?


Question about exporting an integer-only MobileBERT to TF-Lite format.

Hi, I'm trying to export a mobilebert model to tflite format.
Environment
Docker (tensorflow/tensorflow:1.15.0-gpu-py3) image
V100 16GB
As guided in README.md., I followed "Run Quantization-aware-training with Squad" then "Export an integer-only MobileBERT to TF-Lite format." However, I got an error while converting to quantized tflite model.

2020-07-15 10:26:10.934857: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:786] Optimization results for grappler item: graph_to_optimize
2020-07-15 10:26:10.934903: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:788]   constant_folding: Graph size after: 4461 nodes (-1120), 4701 edges (-1124), time = 779.203ms.
2020-07-15 10:26:10.934931: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:788]   constant_folding: Graph size after: 4461 nodes (0), 4701 edges (0), time = 374.792ms.
Traceback (most recent call last):
File "run_squad.py", line 1517, in
tf.app.run()
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/platform/app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 299, in run
_run_main(main, args)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 250, in _run_main
sys.exit(main(argv))
File "run_squad.py", line 1508, in main
tflite_model = converter.convert()
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/python/lite.py", line 993, in convert
inference_output_type)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/python/lite.py", line 239, in _calibrate_quantize_model
inference_output_type, allow_float)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/python/optimize/calibrator.py", line 78, in calibrate_and_quantize
np.dtype(output_type.as_numpy_dtype()).num, allow_float)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/python/optimize/tensorflow_lite_wrap_calibration_wrapper.py", line 115, in QuantizeModel
return _tensorflow_lite_wrap_calibration_wrapper.CalibrationWrapper_QuantizeModel(self, input_py_type, output_py_type, allow_float)
RuntimeError: Invalid quantization params for op GATHER at index 2 in subgraph 0

I used pre-trained weights (uncased_L-24_H-128_B-512_A-4_F-4_OPT) that mentioned in README.md.
Is it required to distillation process before quantization-aware-training?
Regards,
Dongjin.

@renjie-liu

That's interesting, can you update to the latest tf-nightly and try again?
We're counting on the new quantizer.
thanks

Hi @renjie-liu,
thank you for your response.
I've already tried to export using tf-nightly version on docker image. (tensorflow/tensorflow:nightly-gpu)
As far as I know, mobilebert uses tensorflow 1.15 version right? so It's not working.

Traceback (most recent call last):
File "run_squad.py", line 31, in
from mobilebert import modeling
File "/home/google-research/mobilebert/modeling.py", line 32, in
from tensorflow.contrib import layers as contrib_layers
ModuleNotFoundError: No module named 'tensorflow.contrib'

BTW, does it mean "counting on the new quantizer" to use tensorflow 2.x converter?
Regards,
Dongjin.

@liufengdb
Can you help take a look?
Thanks

I think the real issue is the model trained in 1.x world but the quantization needs 2.x. (it's easier for us to do internally)
@saberkun we probably need to migrate mobilebert to 2.x asap.
wdyt?

I think we already removed all tf.contrib usage. The code could probably run with TF1 compatible mode with tf.comat.v1.disable_v2_behavior. https://www.tensorflow.org/api_docs/python/tf/compat/v1/disable_v2_behavior
We just did not get a change to test in open source.

Hi @saberkun,
I tried to run a command "python run_squad.py ..." in tf 2.4.0-dev20200712 version with tf.compat.v1.disable_v2_behavior() in main and also in modeling.py.
The error is as follows.

WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/compat/v2_compat.py:96: disable_resource_variables (from tensorflow.python.ops.variable_scope) is deprecated and will be removed in a future version.
Instructions for updating:
non-resource variables are not supported in the long term
Traceback (most recent call last):
File "run_squad.py", line 32, in
from mobilebert import modeling
File "/home/google-research/mobilebert/modeling.py", line 32, in
from tensorflow.contrib import layers as contrib_layers
ModuleNotFoundError: No module named 'tensorflow.contrib'

Do I remove code that uses the contrib package? (contrib_layers and quantize method only)
Also, you provide saved model files for float and quantized type so I tested quantized saved model file with toco in tf 1.15 but I got an error message during conversion.

2020-07-22 13:05:32.200417: F ./tensorflow/lite/toco/toco_tooling.h:38] Check failed: s.ok() Unimplemented: this graph contains an operator of type Cast for which the quantized form is not yet implemented. Sorry, and patches welcome (that's a relatively fun patch to write, mostly providing the actual quantized arithmetic code for this op).
Fatal Python error: Aborted
Current thread 0x00007f06cdae8740 (most recent call first):
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 52 in execute
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 250 in _run_main
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 299 in run
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/platform/app.py", line 40 in run
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 89 in main
File "/usr/local/bin/toco_from_protos", line 8 in
Aborted (core dumped)

Regards,
Dongjin.

I modified the run_squad.py script to remove the contrib module in order to export a quantized tflite model file in tf 2.4.0-dev20200712 version.
I used two saved_model; one is provided (https://storage.googleapis.com/cloud-tpu-checkpoints/mobilebert/mobilebert_squad_savedmodels.tar.gz), the other one is trained as guided.
I got the same error during convert.

loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): error: 'tfl.minimum' op quantization parameters violate the same scale constraint: !quant.uniform<i8:f32, 0.32020080089569092:-67> vs. !quant.uniform<i8:f32, 0.18242761492729187:-77>
Traceback (most recent call last):
File "export_tflite.py", line 642, in
tf.app.run()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 299, in run
_run_main(main, args)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 250, in _run_main
sys.exit(main(argv))
File "export_tflite.py", line 632, in main
tflite_model = converter.convert()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 1972, in convert
return super(TFLiteConverter, self).convert()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 1341, in convert
result = self._calibrate_quantize_model(result, **flags)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 444, in _calibrate_quantize_model
return _mlir_quantize(calibrated)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/convert.py", line 147, in mlir_quantize
inference_type)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/wrap_toco.py", line 52, in wrapped_experimental_mlir_quantize
inference_type)
RuntimeError: Failed to quantize: :0: error: loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): 'tfl.minimum' op quantization parameters violate the same scale constraint: !quant.uniform<i8:f32, 0.32020080089569092:-67> vs. !quant.uniform<i8:f32, 0.18242761492729187:-77>
:0: note: loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): see current operation: %3377 = "tfl.minimum"(%3373, %36) : (tensor<1x4x384x384x!quant.uniform<i8:f32, 0.32020080089569092:-67>>, tensor<!quant.uniform<i8:f32, 0.18242761492729187:-77>>) -> tensor<1x4x384x384x!quant.uniform<i8:f32, 0.32020080089569092:-67>>

Can someone please clarify :
To reproduce the quantized int8 savedmodel as provided here (https://storage.googleapis.com/cloud-tpu-checkpoints/mobilebert/mobilebert_squad_savedmodels.tar.gz),

Distillation on Pre-trained mobilebert checkpoint using Pre-trained data (https://github.com/google-research/google-research/tree/master/mobilebert#distillation)
Running Quantization-aware-training with Squad (https://github.com/google-research/google-research/tree/master/mobilebert#run-quantization-aware-training-with-squad)

Are both the above steps required to get the quantized int8 savedmodel or just the step 2 with pre-trained model will give the quantized int8 model?
@saberkun @renjie-liu @liufengdb @nadongguri

Hi all,
the quantized int8 SavedModel can be converted in tensorflow v2.3.0.
I got the result as
{"exact_match": 81.06906338694418, "f1": 88.54016833795568}.
Thank you.

I think this (88.54) matches the expectation.

Thanks, I'll close this issue.

@nadongguri Hi, Could you share us how to convert? Thanks a lot

Hi @nadongguri
What is you replace contrib module with?
Can you provide your modification code?
Thanks

I modified the run_squad.py script to remove the contrib module in order to export a quantized tflite model file in tf 2.4.0-dev20200712 version. I used two saved_model; one is provided (https://storage.googleapis.com/cloud-tpu-checkpoints/mobilebert/mobilebert_squad_savedmodels.tar.gz), the other one is trained as guided. I got the same error during convert.

loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): error: 'tfl.minimum' op quantization parameters violate the same scale constraint: !quant.uniform<i8:f32, 0.32020080089569092:-67> vs. !quant.uniform<i8:f32, 0.18242761492729187:-77>
Traceback (most recent call last):
File "export_tflite.py", line 642, in
tf.app.run()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 299, in run
_run_main(main, args)
File "/usr/local/lib/python3.6/dist-packages/absl/app.py", line 250, in _run_main
sys.exit(main(argv))
File "export_tflite.py", line 632, in main
tflite_model = converter.convert()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 1972, in convert
return super(TFLiteConverter, self).convert()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 1341, in convert
result = self._calibrate_quantize_model(result, **flags)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/lite.py", line 444, in _calibrate_quantize_model
return _mlir_quantize(calibrated)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/convert.py", line 147, in mlir_quantize
inference_type)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/wrap_toco.py", line 52, in wrapped_experimental_mlir_quantize
inference_type)
RuntimeError: Failed to quantize: :0: error: loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): 'tfl.minimum' op quantization parameters violate the same scale constraint: !quant.uniform<i8:f32, 0.32020080089569092:-67> vs. !quant.uniform<i8:f32, 0.18242761492729187:-77>
:0: note: loc("bert/encoder/layer_2/attention/self/clip_by_value/Minimum"): see current operation: %3377 = "tfl.minimum"(%3373, %36) : (tensor<1x4x384x384x!quant.uniform<i8:f32, 0.32020080089569092:-67>>, tensor<!quant.uniform<i8:f32, 0.18242761492729187:-77>>) -> tensor<1x4x384x384x!quant.uniform<i8:f32, 0.32020080089569092:-67>>

Hi, @nadongguri , I'm having the exact issue you had as the following:
Traceback (most recent call last):
  File "mobilebert/run_squad.py", line 32, in <module>
    from mobilebert import modeling
  File "/home/khsu4/google-research/mobilebert/modeling.py", line 32, in <module>
    from tensorflow.contrib import layers as contrib_layers
ModuleNotFoundError: No module named 'tensorflow.contrib'

What did you do exactly on modifying run_squad.py? thanks.


Flixtor not a valid Win32 application

Hi! I have downloaded the windows version of Flixtor installer. Installation completes smoothly, but after that it won't work. I get the warning:
C:\Program Files\Flixtor\Flixtor.exe is not a valid Win32 application
My stystem: Microsoft Windows XP - Professional, Service Pack 3
Thanks in advance

@boniak3
i don't think flixtor support Win XP OS. I think Flixtor supports only Win7 and up. I could be wrong but i'm alnost sure this is the case.
Wait for some official respond about your issue to be sure .

Thanks Draxler! I'll wait for the official respond also :)

@boniak3
you could try the compatibility mode settings if you right click on the flixtor launcher, maybe this help.
But all in all Draxler is right, we didn't give additional support for win xp sorry

@boniak3
you could try the compatibility mode settings if you right click on the flixtor launcher, maybe this help.
But all in all Draxler is right, we didn't give additional support for win xp sorry


baremetal schedule net bonding check

这个 PR 实现什么功能/修复什么问题:
调度 baremetal server 时添加对 bonding 的检查
是否需要 backport 到之前的 release 分支:

release/2.9.0
release/2.8.0

/area scheduler
/cc @swordqiu

/cherrypick release/2.9.0

/cherrypick release/2.8.0

Codecov Report

Merging #611 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #611      +/-   ##
=========================================
+ Coverage    7.39%   7.42%   +0.02%
=========================================
  Files         346     346
  Lines       54802   54802
=========================================
+ Hits         4055    4067      +12
+ Misses      50367   50356      -11
+ Partials      380     379       -1

Flag
Coverage Δ

#aFlag
7.42% <ø> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
66.15% <0%> (-3.08%)
:arrow_down:

pkg/util/rbacutils/rabc.go
73.86% <0%> (+5.3%)
:arrow_up:

Continue to review full report at Codecov.

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

/lgtm
/approve


Andres arango solution

Comfort score: 2
Completeness: 4
It was very challenging to me. I don't think I would have been able to do it without collaboration.

👍


Using caret

Coverage remained the same at 100.0% when pulling dda2675c7e7a334572417f339523665a67252d99 on jans510:using_caret into 159b926c5657f7af20a4e762dd4e4249b11f76e1 on zalando:master.

Coverage remained the same at 100.0% when pulling dda2675c7e7a334572417f339523665a67252d99 on jans510:using_caret into 159b926c5657f7af20a4e762dd4e4249b11f76e1 on zalando:master.


fix: fix command not update in UI after eject

Description
fix command not update in UI after eject
Task Item
close #5832
Screenshots

Coverage remained the same at 53.906% when pulling 991ffb45fe2b984bc6d45b17970099842d8211c0 on wenyluo/fixEject into 267ee0f2f587e847140247825658d6fa9c8b434e on main.


devlopment branch: incompatibility between HID host and repeating timer

Code works on V2.0.0 but was advised to try development branch to fix an issue with a repeating timer occasionally stopping after an extended period
    add_repeating_timer_us(-1000, timer_callback, NULL, &timer);
The development branch appear to correct this except if tuh_task(); is called in which case, if a hid device is plugged in, the timer stops.
In the example output below I call tuh_task() for the first time 10 seconds after the program starts. The time is updated by the timer callback. You can see in the output after the call to tuh_host() the timer stops updating. Then after an extended period (c1min) the hid device spontaneously disconnects and reconnects and the timer is running again. During all of this the main program is working as expected.

ping @hathach in case he has any ideas?

"The time is updated by the timer callback." Sorry, what do you mean by this exactly?

The time is maintained in the timer callback using the usual simple code. The fact this it isn't updating shows the callback isn't executing. This is just a convenient way to demonstrate it.  I've also shown it with more direct monitoring by comparing in the main code a counter incremented in the callback with time_us_64()
`    if(++SecondsTimer >= 1000) {
SecondsTimer -= 1000;
#ifndef PICOMITEWEB
if(ExtCurrentConfig[PinDef[HEARTBEATpin].pin]==EXT_HEARTBEAT)gpio_xor_mask64(1<<PinDef[HEARTBEATpin].GPno);
#endif
// keep track of the time and date
if(++second >= 60) {
second = 0 ;
if(++minute >= 60) {
minute = 0;
if(++hour >= 24) {
hour = 0;
if(++day > DaysInMonth[month] + ((month == 2 && (year % 4) == 0)?1:0)) {
day = 1;
if(++month > 12) {
month = 1;
year++;
}
}
}
}
}
}
`

Is your code in github somewhere?

Closing this. I tried to reproduce the problem without luck. Reopen if you can point me towards some code that reproduces the problem.


[WIP] Enabling the 'snapshot' feature through OCS Operator

Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

@Madhu-1  , I have enabled CSI_ENABLE_SNAPSHOTTER to true in OCS Operator generated CSV, in order to enable snap shot feature. Currently labelled the PR as WIP to see if we need additional changes in OCS Operator front.

/ok-to-test

According to the above comments, this PR is no longer needed...
Closing this PR...


Font too big only on --gui

I have the latest version installed on Manjaro linux, when running amdgpu_top --gui I get a very big font, I think about 12x or more, and the readability it's not good. Same on the appimage version. No issue with tui and smi parameters. I wonder why. Is there some parameter or configuration to set? Thank you.

Please try remove ~/.local/share/amdgpu_top or press Ctrl + 0 .

Or try adjusting it with Ctrl + - .


[SMALLFIX] Fix the FileInfo construction to be 1.0.1 compatible

@jsimsa

LGTM, thanks for the fix

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9730/
Test PASSed.

Merged build finished. Test PASSed.


Endorsement policy: multi-endorser

Requires resolution of #274,  #183 and #365.

See #273 and #274 for "sibling" features

btw: With externalized validation we have to be quite careful when going to support more than one enclave:  Our __invoke query also has from a fabric perspective a valid R/W set and hence must not be allowed to succeed when run as an invoke (as nobody would have verified the enclave signatures)!  Not that with the (necessary) derandomized encryption, R/W sets would match!

btw: With externalized validation we have to be quite careful when going to support more than one enclave:  Our __invoke query also has from a fabric perspective a valid R/W set and hence must not be allowed to succeed when run as an invoke (as nobody would have verified the enclave signatures)!  Not that with the (necessary) derandomized encryption, R/W sets would match!


tdd/EntityStateController

Entity always has id
EntityStateController provides a CRUD on models in redux state
It also adds indexing capabilities
Indexes are the separate store part { [indexValue]: EntityId[} }
So, resulting state looks like
entityState: {
  data: EntityState<DomainType>
  [index1.name]: {[indexValue]: EntityId[] }
  [index2.name]: {[indexValue]: EntityId[] }
}

Indexes provide fast access and order.

Closing this issue is not supposed to happen.
We have a lot of work there to do


ui: improve some customization aspects [+]

remove specialized header templates {DEPOSITS,SEARCH_UI}_HEADER_TEMPLATE
fix header.html's html

Longer term things to look into:

move all frontpage html + less + images into cookiecutter: made an issue for it: https://github.com/inveniosoftware/invenio-app-rdm/issues/407
we can reduce html nesting (I've collapsed a few layers in my instance)
invenio-cli watch can be slow
Had to invenio-cli assets update and/or restart even if running invenio-cli watch
(don't know what we can do, but a better update experience would be nice)

@fenekku just to fully understand, for:

Had to invenio-cli assets update and/or restart even if running invenio-cli watch
(don't know what we can do, but a better update experience would be nice)

Did you use --development when installing, and/or when updating assets?

@ppanero

Had to invenio-cli assets update and/or restart even if running invenio-cli watch

Here it's really just that the experience feels a little clunky even though we know why (and I don't really have a good solution). We have to run assets update to symlink/copy; we have to restart to get invenio.cfg changes and we can run watch to get updates to already symlinked files. Perhaps ideally invenio-cli run can be improved to account for all that, but it's complicated.

different headers

We are now using the same header across all pages, so removing those is fine. Issue for being able to override the header template of specific files is here: https://github.com/inveniosoftware/invenio-app-rdm/issues/410

Merging in!


Preprocess Data: ModuleNotFoundError: No module named 'general_utils'

I'm trying to run the Preprocess Process notebook on Google Colab. I was wondering where general_utils comes from. Currently, I get the following error:
ModuleNotFoundError: No module named 'general_utils'

I found them in the same repository!

I have the same problem :(


How to delete the account's public key

My account AAA has two public keys like this:
Account name : AAA
Owner EOSXXXXXXXXXa
Active EOSXXXXXXXXXa
Owner EOSXXXXXXXXXb
Active EOSXXXXXXXXXb
How to delete the account's public key EOSXXXXXXXXXa,and keep the EOSXXXXXXXXXb
Positionals:authority TEXT - [delete] NULL, [create/update] public key, JSON string, or filename defining the authority
cleos set account authority ???
I looked up the developer documentation, but didn't see how to do it,maybe my question is stupid,I really need help.

anybody can help me ?

https://www.bcskill.com/index.php/archives/51.html

I assume your account looks like this now (when you call cleos get account AAA)
permissions:
     owner     1:    1 EOSXXXXXXXXXa, 1 EOSXXXXXXXXXb
        active     1:    1 EOSXXXXXXXXXa, 1 EOSXXXXXXXXXb

In that case, you can just use cleos set account permission.
The following command is used to update active:
cleos set account permission AAA active EOSXXXXXXXXXb owner -p AAA@owner

The following command is used to update owner:
cleos set account permission AAA owner EOSXXXXXXXXXb -p AAA@owner


Fix bug in optional when field is present but doesn't decode

This fixes the following scenario.
Given
Using optional or optionalAt to decode an object where the requested field is a string, and the requested field is present but is not a string...
Expected behavior
...the decoding operation should fail, because the field was present but did not decode.
Actual behavior
...the decoding operation succeeds with the provided fallback. (This is because optional and optionalAt have been using Json.Decode.maybe under the hood, which translates any failed decoding - not just "field missing" - as equivalent.)

@rtfeldman what should happen if the field is present and null?

Up to the caller; you can use e.g. Json.Decode.Extra.maybeNull or (by default) let it be treated as a failure.

okay, makes sense.
This all looks good 👍


Crash on adding tap on microphone on iOS 13

I systematically get this crash when adding a tap on a microphone when earphones are plugged in:
[avae] AVAEInternal.h:76    required condition is false: [AVAudioEngineGraph.mm:2026:InstallTapOnNode: (format.sampleRate == inputHWFormat.sampleRate)]
*** Terminating app due to uncaught exception 'com.apple.coreaudio.avfaudio', reason: 'required condition is false: format.sampleRate == inputHWFormat.sampleRate'

If you've got an idea, I'll take it!

I fixed that for now by adding the tap before starting the AudioKit. Seems to work for now :)


About Enter bug

When I press Enter to open a Local Disk or Folder, nothing appears. It shows Empty. But when I double-click mouse left button, it works properly.
When Double-Clicking mouse left button-

When pressing Enter-

Yes I could confirm this issue.

Thank you


Health Check - Not able to start new containers when unhealthy instances are detected when service has exposed ports/scehduled with anti-affinity rules.

Server version - V0.43.1
Steps to reproduce the problem:
scenario 1:
Set up has 3 hosts.
Create a service with health check enabled and having 1 exposed port with scale 3.
Make one of the instances of the service unhealthy .
Service gets marked as "unhealthy".
There is an attempt made to start another instance , but it fails to find a placement.
scenario 2:
Set up has 3 hosts.
Create a service with health check enabled and having scheduling rules set to anti-affinty service to itself.
Make one of the instances of the service unhealthy .
Service gets marked as "unhealthy".
There is an attempt made to start another instance , but it fails to find a placement.

thats a good one. We should relax port constraint for containers in unhealthy state.

Shouldn't be an issue anymore as we delete unhealthy instances prior to creating their replicas now.

Tested with latest build from master.
Both the scenarios mentioned in the issues works as expected.


Thanks! The latest KiCAD ESP8266 with additional resistors works!

I have purchased two units from Paul but wanted to build one. I did 2-3 prototypes and breadboards on the 3 10k resistor configuration and two 2N7002 mosfet. I then tried this as I read that the voltages might need to be tuned down.
kicad_files/D1 Mini - ESP8266/RatGDO-OpenSource-D1Mini-ESP8266.pdf
Note I still used two 2N7002 mosfet plus 5 10k resistors and 1 1k resistor as per the diagram.
This build worked like a charm and has been running for a bit.
So this isn't an issue but I think this schematic is probably better.  At least it was for me.

Thanks for posting that. I was able to use an old esp8266 nodemcu 'Lolin' and it works!
I followed the schematic you mentioned,
https://github.com/Kaldek/rat-ratgdo/blob/main/kicad_files/D1 Mini - ESP8266/RatGDO-OpenSource-D1Mini-ESP8266.pdf
The latest esphome firmware worked well, but I'm using this with homebridge, so prefer the mqtt apis... I loaded the latest ratgdo firmware on it and also works. Now I can control the garage door from an iphone or apple carplay.

but I'm using this with homebridge, so prefer the mqtt apis

Note that the https://github.com/ratgdo/homekit-ratgdo firmware looks to be in a useable state now, in case you didn't see it.


Setup Home Assistant & Home Server

Tasks

[x] Network setup #14
[ ] Get Linux server going and Ansibled
[ ] Clone repository on Linux
[ ] Document...

Steps

[ ] Create Ansible host on Synology VM
[ ] Setup VM
[ ] sudo apt update
[ ] Install ansible
[ ] Install git
[ ] clone home-server-configuration repository


Add examples

This is a dummy issue to remember me to add more examples.
Blocked by https://github.com/robertpainsi/filtering.js/issues/26

The homepage has a hand full of examples now. More examples will be added on demand. Closing!


Update Shrewsbury Colleges Groups domains

Follow up to #5233, it turns out that there are indeed students/staff
with emails using the legacy @ssfc.ac.uk domain.

@jamierocks Pull request merged. Thank you for the update!


Thunderbird -> 52.2.1

Issue description
https://www.mozilla.org/en-US/thunderbird/52.2.1/releasenotes/ contains some fixes.

Should this issue be closed now that #26859 is merged?

Well, thunderbird-bin is still 52.2.0...


Missing escape HTML in release view

When I go to release view, you can see this :

I suppose the data isn't escaping 😄

Thanks for reporting! This would need to be solved here: https://github.com/electron/releases/issues/17

Can I work on this?

Hey @TruptiM18 thanks for chiming in, and apologies for the delay getting back to you.
@pierreneter started working on this, but a solution hasn't been shipped yet. Please take a look at https://github.com/electron/releases/pull/19 -- I think the best way to solve this is with a small patch to the hubdown module. I've added a comment about it on that PR.

This issue was resolved by changing the release notes to wrap the <input> tag in backticks (`) so it is treated as code instead of an input element.


Service name mis-matches

The service names in the README.md do not match the app.js keys and cause the app to fail launch.
2015-05-30T14:31:59.42-0400 [App/0]      OUT service SpeechToText not bound to this application
2015-05-30T14:31:59.42-0400 [App/0]      ERR
2015-05-30T14:31:59.42-0400 [App/0]      ERR /home/vcap/app/app.js:44
2015-05-30T14:31:59.42-0400 [App/0]      ERR speechToTextCreds.version = "v1";
2015-05-30T14:31:59.42-0400 [App/0]      ERR                           ^
2015-05-30T14:31:59.42-0400 [App/0]      ERR TypeError: Cannot set property 'version' of null
Update README to match service names in apps.js

Thanks for the heads up. I just pushed a big update which included updating the README, as well as a few other files that had these old service names.


favor type usize for variables that are primarily an array index

Type usize is required for the index into an array or slice. It is generally preferable to go ahead and use this type from the outset for variables for which this is the primary purpose, to reduce the need for casting later. Unless there is a specific reason to do otherwise, of course.
Examples of places that might benefit from this change...
geodesic: GEODESIC_ORDER
geomath::_A1m1f: geodesic_order
geomath::_C1f: geodesic_order, m, o
geomath::_A2m1f: m
geomath::_C2f: o, m

SGTM

Fixed in https://github.com/georust/geographiclib-rs/pull/58

In #57


Bagel Small Fixe

small air alarm/firelock fixes

Does this include the air alarm that's in Atmos, near the mixing chamber? Seemed to be stuck on alert mode last time I was on

that air alarm is linked to the mixer and that starts in vacuum


added use cases for smart contracts

added use cases for smart contracts

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Looks good. Congrats on your first contribution @r3t51w 🎉


fix: Allow omit of final_snapshot_identifier

Resolves #39
Defaulting final_snapshot_identifier to null if skip_final_snapshot is true
Ready to be merged
Description

Motivation and Context

Breaking Changes

How Has This Been Tested?

duplicate, trying to make semantic PR happy, but failing


Add NonExecutable attribute

Fixes #2933
Scope:
Estimate:

Review checklist
The author performs the actions on the checklist. The reviewer evaluates the work and checks the boxes as they are completed.

[ ] Summary. Write a summary of the changes. Explain what you did to fix the issue, and why you did it. Present the changes in a logical order. Instead of writing a summary in the pull request, you may push a clean Git history.
[ ] Documentation. Write documentation for new functions. Update documentation for functions that changed, or complete documentation where it is missing.
[ ] Tests. Write unit tests for every change. Write the unit tests that were missing before the changes. Include any examples from the reported issue as integration tests.
[ ] Clean up. The changes are already clean. Clean up anything near the changes that you noticed while working. This does not mean only spatially near the changes, but logically near: any code that interacts with the changes!

@ana-pantilie Is it possible to have the attribute apply to rewrite rules as well as functions in this change? For example, given these rules:
rule g(X, Y) => X +Int Y [non-executable, priority(1)]
rule g(X, Y) => X requires X >=Int Y
rule g(_, _) => 4567 [owise]

The LLVM backend will rewrite g(3, 100) to 4567 as expected, while the Haskell backend as tested at this PR will rewrite to 103 using the non-executable rule.

@Baltoli We discussed the attribute being available to only equation (function) rules, AFAIK. Has someone requested this feature for rewrite rules as well?

@Baltoli We discussed the attribute being available to only equation (function) rules, AFAIK. Has someone requested this feature for rewrite rules as well?

Apologies if so; I may be misremembering - let me try and track down the original conversation again.

@ana-pantilie Let me know if there is anything else I can do to help you with this issue - it would be good to get it merged, as the frontend support tests passing will depend on this PR.
In particular, you note above that it's necessary to test what's being sent to Z3 - is this from the perspective of the frontend, or the haskell backend?
Thanks so much for your help on this!

@Baltoli Thank you for the help!

In particular, you note above that it's necessary to test what's being sent to Z3 - is this from the perspective of the frontend, or the haskell backend?

This is just from the perspective of the backend. I will add some tests for this in the backend in a follow-up PR, since I need the updated frontend to do that.


NaTType does not support .time()

I am currently writing a script to take data from an API and save it to an Excel file for non-technical users who need to access the data. One of the requirements is to have separate date and time column, rather than a single datetime column.
I am using pandas 0.17.0.
import pandas as pd

# creating datetime series
dates = pd.date_range('2015-01-01 00:00', '2015-01-01 03:00', freq='H')
dates = pd.Series(dates)
# setting one record to NaT
dates.iloc[0] = pd.NaT
# converting to date
dates.apply(lambda x: x.date()) # ignores the NaT
# converting to time
dates.apply(lambda x: x.time()) # fails with error "NaTType does not support time"

I expected NaTType to behave in the same way for converting into a time as it did for a date i.e. returning NaT.
The full error is:

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-98-b34fccfc4c06> in <module>()
      9 dates.apply(lambda x: x.date()) # ignores the NaT
     10 # converting to time
---> 11 dates.apply(lambda x: x.time()) # fails with error "NaTType does not support time"

C:\Users\Nathan\Anaconda\lib\site-packages\pandas\core\series.pyc in apply(self, func, convert_dtype, args, **kwds)
   2158             values = lib.map_infer(values, lib.Timestamp)
   2159
-> 2160         mapped = lib.map_infer(values, f, convert=convert_dtype)
   2161         if len(mapped) and isinstance(mapped[0], Series):
   2162             from pandas.core.frame import DataFrame

pandas\src\inference.pyx in pandas.lib.map_infer (pandas\lib.c:62187)()

We had this debate here: https://github.com/pydata/pandas/pull/10372
time is a special animal and should be an obvious error and not glossed over if it returned np.nan (as does .date) on a NaTType.
Further using .apply is very inefficient. You can catch the exception and return np.nan if you'd like.
In [55]: dates
Out[55]:
0                   NaT
1   2015-01-01 01:00:00
2   2015-01-01 02:00:00
3   2015-01-01 03:00:00
dtype: datetime64[ns]

In [56]: dates.dt.date
Out[56]:
0           NaN
1    2015-01-01
2    2015-01-01
3    2015-01-01
dtype: object

In [57]: dates.dt.time
Out[57]:
0         NaN
1    01:00:00
2    02:00:00
3    03:00:00
dtype: object

Thanks @jreback this makes sense.


Validate kindConfigFile while configuring Kind cluster

What type of PR is this?
/kind bug

What this PR does / why we need it:
This PR fix the situation, when someone would like to create Kind cluster with options, but without kindConfigFile defined, i.e.
envfuncs.CreateClusterWithConfig(kind.NewProvider(), kindClusterName, "", kind.WithImage("kindest/node:v1.22.2")),

In this case, create procedure will fails with unpredictable behavior:
15:14 $ make e2e
go test ./test/e2e/... --v 4
F0906 15:17:25.100154   88839 env.go:375] Setup failure: failed to create kind cluster: exit status 1 : exit status 1: ERROR: unknown command "kindest/node:v1.27.3" for "kind create cluster"

Which issue(s) this PR fixes:

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

Additional documentation e.g., Usage docs, etc.:

/ok-to-test

/retest-required

@dmvolod Are you still working on this ?

No worries @dmvolod  I was just checking to see if you were still looking into this or not. No hurry! Take your time.

@harshanarayana added configFile validation logic for kwok

/lgtm


simple memory leak

hi,
missing call to EVP_MD_CTX_free() here.
https://github.com/laurencelundblade/t_cose/blob/d5ff4e282d8af34e5756627cf877ab399e7e51af/crypto_adapters/t_cose_openssl_crypto.c#L624

Address by #94

Fixed by #94


Get a modal with command description and parameters on button click

The puzzle 9-34c43099 from #9 has to be resolved:
https://github.com/Decision-Driven-Development/logic-checker/blob/6615bc14217058d1d2d7662a564461f81f9c6edc/src/main/java/ru/ewc/checklogic/server/StatePage.java#L64-L64
The puzzle was created by Евгений Терехов on 18-May-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

No description, no form with parameters, just the modal with dynamic command identifier

@0pdd the puzzle #22 is still not solved.

The puzzle 9-34c43099 has disappeared from the source code, that's why I closed this issue.

@0pdd the only puzzle #22 is solved here.


Fix typo

Thanks.
It's done the wrong way as README is autogenerated though, only realised it after merging

this is the right way


Line chart: 2+ incomplete datasets = broken tooltip "index" mode

Expected Behavior
When line charts are given incomplete datasets, "index" mode tooltips should appear at the mouse cursor position, as normal.
Current Behavior
When given incomplete datasets (e.g., data property lists ten labels, but each dataset only contains data points for four or five), "index" mode tooltips are all over the place.
Possible Solution
A possible workaround is to explicitly specify all missing datapoints with { x: ..., y: 0 }
Steps to Reproduce
https://codepen.io/rlue/pen/WNGrKaq
new Chart(ctx, {
    type: 'line',
    data: {
        labels: [
          'red',
          'orange',
          'yellow',
          'green',
          'blue',
          'indigo',
          'violet',
        ],
        datasets: [
          {
            label: 'Set 1',
            data: [
              { x: 'red', y: 12 },
              { x: 'orange', y: 6 },
              { x: 'yellow', y: 19 },
            ]
          },
          {
            label: 'Set 2',
            data: [
              { x: 'indigo', y: 12 },
              { x: 'violet', y: 6 },
            ]
          },
        ]
    },
    options: {
        scales: {
            yAxes: [{
                ticks: {
                    beginAtZero:true
                }
            }]
        },
        tooltips: {
          enabled: true,
          mode: 'index',
          position: 'average',
          intersect: false,
        },
    }
})

Context
Just trying to use "index" mode tooltips.
Environment

Chart.js version: 2.9.4
Browser name and version: Firefox Developer Edition 84.0b7 (latest), Chromium 83.0.4103.116 (Developer Build)
Link to your project: N/A (sorry, closed source / no demo)

Not sure but it kind of looks like intended behaviour.
If it takes the index of the data and matches that to the labels even though you say match this point to label x its the first datapoint so index 0. This makes the index at which the tooltip will look also 0.
There are 2 possible solutions, first is for every value that is not occupied add a null or undefined or switch the tooltip mode to point then it works as you want

The name for the mode is intentionally index and not for example nearest as it seems you think it should work like.
For fun, try mode: 'nearest', axis: 'x'

Filed in error indeed! Thanks for the speedy responses.


iOS safari support indexedDB

When loading my app in under iOS 11.1.2 / safari localForage always uses webSQLStorage driver instead of indexedDB. Even the attempt to force the driver either in config or by using setDriver fails and the driver() function always provides webSQLStorage. Actually indexedDB should be supported since iOS 10.2. Looking into the localForage issues there is no clear statement whether indexedDB is now supported or not. I must save quite big data on the client side and think the alternatives like local Storage and webSQLStorage will not work for my application.
Best regards
Matthias

Any update on this? My app needs the storage capacity of IndexDB to function for some users. What's involved in updating angular-forage to use localForage ^1.7?

@itjustwerks
You can already insist on using IndexedDB if you have to have it for your application: https://localforage.github.io/localForage/#settings-api-setdriver. You just need to call setDriver(localforage.INDEXEDDB). Does that work for your case?

I tried setting the drive to "asyncStorage" and am getting "Error: No available storage method found" in Safari on Mac OS. Without the specific setdriver, I get a non-specified WebSQL error.

I've not had to do this, but can you use localforage.INDEXEDDB instead of asyncStorage?


Unsupported for  Intel(R) Xeon(R) Silver 4410Y

ppm
Processor Counter Monitor  (202201-1)
Linux arch_perfmon flag  : yes
Hybrid processor         : no
IBRS and IBPB supported  : yes
STIBP supported          : yes
Spec arch caps supported : yes
Error: unsupported processor. Only Intel(R) processors are supported (Atom(R) and microarchitecture codename Nehalem/Nehalem-EP, Atom(tm), Westmere/Clarkdale, Sandy Bridge, Westmere-EP, Sandy Bridge-EP/Jaketown, Nehalem-EX, Westmere-EX, unknown, Centerton, Baytrail, Ivy Bridge, Haswell, Broadwell, Ivy Bridge-EP/EN/EX/Ivytown, Haswell-EP/EN/EX, Cherrytrail, Avoton, Skylake U/Y, Broadwell-EP/EX, Skylake-SP, Cascade Lake-SP, Broadwell-DE, Knights Landing, Apollo Lake, Skylake, Denverton, Icelake-SP, Icelake, Snowridge, Tiger Lake, Kabylake/Whiskey Lake, Alder Lake, Kabylake, Comet Lake, Rocket Lake). CPU model number: 143 Brand: "Intel(R) Xeon(R) Silver 4410Y"
Access to Processor Counter Monitor has denied (no MSR or PCI CFG space access).

How to solve it?

please use the latest version. 202201 is too old.


Equivalent to -ss (Seeking)

Hi, great repository!
I was wondering if there is a way to retrieve a frame from a specific timestamp, like this:
ffmpeg -ss 00:15:00 -i full_movie.mp4 -vframes 1 frame00001.jpg
It searches for the timestamp and exports that exact frame from the mp4. It's important to put the -ss before the input file, because that speeds up the process by a lot (see Seeking).
Do you think it's possible to do such thing in ffmpeg-python?

I solved it like this:
timestamp = "00:15:00"
output_file = "frame00001.jpg"
input_file = "full_movie.mp4"

stream = ffmpeg.input(input_file)
ffmpeg.nodes.get_stream_spec_nodes(stream)[0].kwargs['ss'] = str(timestamp)

stream = stream.output(output_file, frames=1)
stream.run(overwrite_output=True)

Looks awful but it works. If there is any better method I'd like to here it!

 I tested and the speed is fine!

This works! Thanks


Replaces installation links for .deb/Ubuntu

The provided curl command does not work; replaced filenames to match those of the current .deb release.

Thanks for the fix!


[layout] add option to keep layout centered

Fixes #27 via an optional property to ensure backward compatibility.

Maybe align: center?

Haha, that would definitely be better. Since the current (which will be default) implementation is align: right and this would be align: center, then I guess I should be adding an align: left, too. What do you think? In 3D, left and right don't make much sense actually, but I guess they are used widely. Do you have an alternative suggestion?

How about align start and align end

Makes sense, and is inline with the text-align CSS property. I'll get cracking.

Ok, I've updated the PR to make it into an align property. I've also updated the README to document the new variable and the Line Layout example to show how it works.
Would you like me to squash my commits or will you do that as you are merging?
Thanks for the guidance, for these components and for A-Frame in general. Great work!

Thanks! This will be useful

👍


add translations to all models

:+1:

:+1:

This was a horrible PR. Because we were figuring things out and working on experimental features, it ended up being a mammoth PR that just grew organically.
We take note of this horrible break in protocol and we'll endeavour to not make the same mistake next time.


Fix server side rendering

Prevent calling ReactDOM.createPortal if the DOM is not yet available by setting shouldRender state in componentDidMount.

Please cleanup this PR. It should not contain any changes in  /docs


Public API for adding to SourceSetOutput.classesDirs

Expected Behavior
JVM language plugins need to be able to modify SourceSets similar to core Gradle plugins. Currently, there is no way to add a classesDir to the SourceSetOutput.

Current Behavior
Currently, you need to use internals:
Either:
SourceSetUtil.configureOutputDirectoryForSourceSet(sourceSet, clojurescriptSourceSet.getClojureScript(), compile, project);

But the signature is changing in 4.8.
Or you have to cast to DefaultSourceSetOutput to access addClassesDirs.

Context
Third-party JVM language plugins have nearly identical requirements to the core ones, and need stable public APIs for core features like adding and configuring SourceSets.
Similar issues #727, #3191, #831.
As a side note here, the deprecated model-space approach for this was on a much better path. I just had to declare I wanted a ClojureSourceSet and register it as a language. Gradle dealt with providing default directories for everything for me. Ultimately, there's no reason that I (as a language plugin author) need to care what directory you put those classes in or where the sources are (as long as it's unique and consistent with other plugins).
It would be great to cut down the boilerplate for language plugins (particularly JVM ones).

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

@ajoberstar Stutter is awesome. Considering using it on some of my plugins. Good catch on this one.


Can't use config-path flag

sorry I don't know if im stupid or what but I can't add a local config.go file.
I am trying:
gitleaks -r https://somerepo.com/ -q --config-path="/root/githubsearch/"
gitleaks -r https://somerepo.com/ -q --config-path="/root/githubsearch/default.go"
gitleaks -r https://somerepo.com/ -q --config-path=/root/githubsearch/default.go
gitleaks -r https://somerepo.com/ -q --config-path=default.go

I tried all the ways It is common to select a path.. but none work
Can somebody please help me?

The usual that most golang tools use is -c config.go or -c /path/to/file.yaml and that's it, none of them working here

Hello @marcelo321 !
I'd try without the -q mode, and use -v or --verbose instead, to have a better look of what's the error.
However, the config path  it refers to, the path to the gitleaks config file, not the config file of your application.
If you want to scan that file, use it as it's shown in this example -> https://github.com/zricethezav/gitleaks#scan-a-file
Here you have more info about the --config-path option you are using -> https://github.com/zricethezav/gitleaks#configuration
Cheers!

@w0rmr1d3r thanks for helping out!

@zricethezav @w0rmr1d3r,
I am trying to load a gitleaks configuration using my own regexs, but I am unable to pass it as an argument using the mentioned flag.
--config-path loads a local gitleaks configuration
I am trying the following:
gitleaks -r https://github.com/marcelo321/repo -q

Works as expected, scans perfectly. Now I am trying to pass my own set of rules/regexs so gitleaks uses them instead of the default one here: https://github.com/zricethezav/gitleaks/blob/master/config/default.go
So I created my own config.go with my own regexs and I am trying to pass it like:
gitleaks -r https://github.com/marcelo321/repo -q --config-path= config.go

But I get no error there and the file full of regexs is not even proccessed. How do I pass my own default.go? does it need to be .go?

Ok so the config file had to be .toml.. damn.

Also I noticed that regex matching more than 1 line aren't working. Is there a way to do it?
Trying:
regex = '''((?i)someword)((\S|\s*)?){0,75}anotherword'''

I used that regex a lot, trying to switch from shhgit to gitleaks but can't manage how to change the regexs

Hello @marcelo321 !
About this:

Ok so the config file had to be .toml.. damn. Is there any public .toml file we can take a look? I just have it with rules and Idk If I can add blacklist strings and others.

Yes, there're examples here and you can add blacklist strings ->
https://github.com/zricethezav/gitleaks#rules-summary
Regarding the comment about regex, you have an example here -> https://github.com/zricethezav/gitleaks#example-1
However, can you provide us the full toml file? Maybe there's something else we are missing.
Thank you!

I missed your comment. Basically I want to generate a rule/regex that matches the following words:
word1
word2

In the config.yaml of shhgit I was using a regex like this one: ((?i)someword)((\S|\s*)?){0,75}anotherword to match spaces, but it doesn't work with gitleaks.
If you find an example to match words from 2 or more different lines like with my example, let me know

@marcelo321
You probably want to put your regex in square brackets. This is a toml syntax.
regex = ['''((?i)someword)((\S|\s*)?){0,75}anotherword''']

Gitleaks has been updated to v8.0.0 and this issue will not be fixed moving forward. Sorry for the inconvenience, feel free to fork older versions introduce a fix in your fork.
This issues are likely resolved with the latest version of gitleaks


Receiving "No methods for platform 'windows'!" on Windows 11 in script

Describe the bug
I receive a RuntimeWarning error when executing function through Python, but it's fine when running through the terminal.
Expected behavior
I am on a Windows platform, so Python should use the windows module, like it does from the terminal.
System info
(please complete the following information):

OS name (e.g. Windows 10 x64): Windows 11 x64
OS Version (e.g. 1804 build 17134): 22H2 22631.1835
Python version (e.g. CPython 3.6.5 x64): Python 3.9.13
getmac version (e.g. getmac 0.8.3): getmac 0.9.4

Additional context
When running get_mac_address(interface="WiFi") in Python, I get the following output:
C:\Users\User\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\getmac\getmac.py:1441: RuntimeWarning: No methods for platform 'windows'! Your system may not be supported. Falling back to platform 'other'.
warnings.warn(warn_msg, RuntimeWarning)
04:6c:59:0f:10:40
However, when running via the terminal I get the following, confirming platform is windows:
python -m getmac -v -d
DEBUG    Initializing 'ip4' method cache (platform: 'windows')
DEBUG    Current method cache: {'ip4': 'CtypesHost', 'ip6': 'None', 'iface': 'None', 'default_iface': 'None'}
DEBUG    Current fallback cache: {'ip4': '[<getmac.getmac.ArpExe object at 0x0000029C0C921D90>]', 'ip6': '[]', 'iface': '[]', 'default_iface': '[]'}
DEBUG    Finished initializing 'ip4' method cache
DEBUG    Attempting get() (method='CtypesHost', method_type='ip4', arg='<IP_ADDRESS>')
DEBUG    Raw MAC found: 046c590f1040
DEBUG    getmac took 0.0028 seconds
DEBUG    Adding colons to MAC 046c590f1040
(This is my first ever issue, so apologies if there is anything missed)

Hey @Jackio007, thanks for reporting this issue (and you got everything, thanks for following the template 😉  ). This is definitely unusual behavior, and somewhat surprising considering I primarily develop on Windows 11 machines, so I thought I'd have noticed it by now. I'll take a look at it after work tomorrow and see if I can reproduce.
In the immediate short term, you should be able to force the platform by setting getmac.getmac.platform = "windows" before calling the function.

Apologies for the delay. This is definitely a bug I was able to reproduce. Just to confirm, do you also get 04:6c:59:0f:10:40 when running getmac -v -d -i 'Wi-Fi'?
What seems to be happening is the default interface isn't being determined properly. On my Windows 11 laptop, if I run getmac -v -d (no interface specified), it returns the MAC for "Microsoft Wi-Fi Direct Virtual Adapter", when it should be returning the MAC for "Wi-Fi", as that's the default interface.
Not sure why this is happening, I'll look into it further and get working on a fix. The silver lining is the code for finding the default interface on Windows has needed work for a while and this is a good excuse to finally clean it up properly 😅


Image not found

I'm trying to evaluate if I can use BeaconCMS, so I tried to run this demo.
Clone, mix deps.get, mix ecto.setup, mix phx.server .
The issue that I'm facing is the it can't found the narwin.webp file, and also getting the same error when trying to upload new media. The error is:
KeyError at GET /admin/demo/media_library/upload
key :agent not found in: %{
  socket: #Phoenix.LiveView.Socket< ....

The weird thing is that down the page, it really shows that narwin.webp was loaded indeed:
streams: %{
    assets: %Phoenix.LiveView.LiveStream{
      name: :assets,
      dom_id: #Function<1.120097161/1 in Beacon.LiveAdmin.MediaLibraryLive.Index.mount/3>,
      ref: "0",
      inserts: [
        {"4765ae81-e575-4c24-8813-7a60b093e922-e905343b-ec93-4080-90dd-be0834f40511",
         -1,
         %Beacon.MediaLibrary.Asset{
           __meta__: #Ecto.Schema.Metadata<:loaded, "beacon_assets">,
           id: "e905343b-ec93-4080-90dd-be0834f40511",
           site: :demo,
           file_name: "narwin.webp",
           file_body: <<82, 73, 70, 70, 24, 180, 0, 0, 87, 69, 66, 80, 86, 80,

Also, the file is not displayed on the entry page, just a placeholder with error.
What am I doing wrong?
Thanks!

Hey @dantodor nothing wrong on your side :) Both issues are fixed now on the main branch.
The first one by https://github.com/BeaconCMS/beacon_live_admin/pull/200 and the other by https://github.com/BeaconCMS/beacon_demo/commit/22212414a989d43738ede06d2831f080fc1a0bb2
Thanks!