Create new componentized metadata frontend.

Created a new project under web built to have a slightly more decomposed and hopefully more maintainable frontend UI in the future. I used a JSX-based framework for basic state management and decomposition (Preact) but kept everything very minimal on purpose (no frameworks, no design systems, plain old CSS). Tested with the big three browsers and VoiceOver.
Uhh, and no tests. I kept this waiting long enough after I was asked, and maintaining tests would add a pointless burden to future non-frontend friendly devs.
To run the project locally:

Install dependencies (npm install or bun install)
Run dev server (npm run dev or bun dev)
And when you're run, build it into a static app (npm run build or bun run build)

Very nice! Thanks for your work, I'll review this in more detail when I can.

Hmm. I see a package.json but no package-lock.json. Did you forget to commit it?
npm ERR! code ENOENT
npm ERR! syscall open
npm ERR! path /path/to/attribution-metadata/web/package.json
npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, open '/tmp/emmie/attribution-metadata/web/package.json'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

@emmiegit That's odd, you can see both package.json and package-lock.json in the git diff here.

@emmiegit That's odd, you can see both package.json and package-lock.json in the git diff here.

Oh weird. I'll try it again when I'm at home.

Spent some time playing with this, this is very nice! Thank you so much smlt.


Flexible header height for expanded panels?

I noticed that the header height (edge insets)
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/expansion_panel.dart#L250
of an expanded expansion panel is fixed to 64
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/expansion_panel.dart#L13
Is there any special reason for this implementation (e.g. Material UI constraints)?
Or in other words: What about making this a dynamic property?
Thank you for your opinion!

https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/expansion_panel.dart#L250
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/expansion_panel.dart#L13

I wanted to use a different icon for ExpansionPanel but the property is not available. So I created my own using ListView and AnimatedContainer but I can't get it to expand without having a fixed height. What I want to happen is for the ExpansionPanel/AnimatedContainer to expand to fit the body/child's height.
AnimatedContainer(
	duration: Duration(milliseconds: 300),
	child: Container(
		child: Text(widget.subtitle),
		padding: EdgeInsets.only(top: 8),
	),
	curve: Curves.easeInOut,
	// height: _isExpanded ? <cannot be determined> : 0,
);

Closing issue, as pull request https://github.com/flutter/flutter/pull/47951 is merged.


Facts api changes

Facts API and UI updates
API

[x] Adding source and description fields to facts
[x] New addFacts and deleteFactsFromSource GQL resolvers that allows bulk adding/removing facts

UI

[x] Updated UI Facts tab with new source and description fields

We'll have to discuss which branch to merge this to.  Lagoon 2 and all development is currently on main.  Master may be used if we need 1.x releases in the meantime.

@tobybellwood
yea, I discussed with the SRE team that we would get this into 1.9.2 and then merge master into main, as this is a rather small change and would help the maintenance teams a lot, I think it's worth doing this.

@timclifford
sorry I think I confused myself, we need two alter tables, see example:
https://github.com/amazeeio/lagoon/blob/dcbb74ce0ee688a123d7564fcfbaa102f91009e4/services/api-db/docker-entrypoint-initdb.d/01-migrations.sql#L274-L277


chore(volar): @volar/server -> @volar/vue-language-server

See https://github.com/johnsoncodehk/volar/issues/876
See https://github.com/johnsoncodehk/volar/blob/master/packages/vue-language-server/package.json#L2
The renamed @volar/vue-language-server package has not been released yet. Just pre-emptively submitting the PR

According to the changelog, we should add languageFeatures.implementation = true.

You're right, thanks for the heads up. Added in https://github.com/neovim/nvim-lspconfig/commit/ac3f907511de300b2eac34c3c27b6e6ece34c165
For documentation's sake:
https://github.com/johnsoncodehk/volar/blob/6532141fa7b92e077f16d5a9a4e5d04ca2ba41a2/packages/vue-language-server/src/registers/registerlanguageFeatures.ts#L14
https://github.com/johnsoncodehk/volar/blob/master/CHANGELOG.md?plain=1#L8


Add logos from sponsors of the project

Aktion Mensch, Sophie Scholl Schulen, Uni Bremen sponsor this project. Please add the following logos to the footer
https://www.uni-giessen.de/ueber-uns/veranstaltungen/sonstige/AM_Foerderungs_Logo_RGB.png/image
https://www.sophie-scholl-schulen.de/fileadmin/resources/gfx/logos/sophieSchollSchulen.png
https://www.uni-bremen.de/typo3conf/ext/package/Resources/Public/Images/logo_ub_2021.png
and link to
www.aktion-mensch.de
www.sophie-scholl-schulen.de
https://www.uni-bremen.de/fb12/arbeitsbereiche/abteilung-c-inklusive-paedagogik/inklusive-paedagogik-geistige-entwicklung-und-lernen
and exchange the copyright symbol with a cc-0-Logo.
<EMAIL_ADDRESS>

@cabotine The first logo from uni-giessen.de doesn't load for me, it only shows <plone.namedfile.file.NamedBlobImage object at 0x7f853ca2cc80 oid 0x520a60 in <Connection at 7f858c0d1cc0>>
You could also paste the logo into the comments box here, and it will upload it directly into the issue.

@cabotine The first logo from uni-giessen.de doesn't load for me

https://www.uni-giessen.de/de/ueber-uns/veranstaltungen/sonstige/AM_Foerderungs_Logo_RGB.png


2020 Silverado steeractuator delay .11 Steer ratio 16.3

Experimented extensively with individual settings to eliminate placebo and figure out what works. Steer delay has a very small window of effectiveness. Any variation below .1 or above .1 beyond .015 is either too much or too little and becomes ineffective. Not sure if the math goes out of wack and is ignored or what but there is a sweet spot. I had issues with hugging inside of curves. .11 fixes the right curve and helps with left curves. If any other vehicles are having hugging issues try this setting to verify it works for all hugging issues. I am experimenting now with very little adjustments from .11 up and down to see if I can get the truck to center curves both directions with just this setting.
Also raising the steering ratio has very simular effect as raising kdv. It makes the steering very responsive to adjustments. Please put it back to 16.3. I haven't played with reducing ratio below this but have dropped kdv down to .155. It smooths out the steering greatly but can induce wandering at that level. Thanks Jason!
elif candidate == CAR.SILVERADO_NR:
ret.minEnableSpeed = -1. # engage speed is decided by pcm
ret.minSteerSpeed = -1 * CV.MPH_TO_MS
ret.mass = 2241. + STD_CARGO_KG
ret.wheelbase = 3.745
ret.steerRatio = 16.3 # Determined by skip # 16.3 # From a 2019 SILVERADO
ret.steerRateCost = 0.45
ret.centerToFront = ret.wheelbase * 0.49
ret.steerActuatorDelay = 0.11 # Determined by skip # 0.075
ret.pcmCruise = True # TODO: see if this resolves cruiseMismatch

This PR has tons of irrelevant changes due to simply being the dev branch of the fork you happen to be running. You'll need to make a clean branch based off commaai/master with only the relevant changes.


Connect to several eventbus

Hi,
I got two eventbus, with two different URLs. How could I connect and use both of them?
Thanks for this module and your work ;)
Hugo

This is currently not a feature.
This is an interesting idea, but would require additional factory service to handle multiple instances of connections.

For which version you are planning to add this feature?

This is currently not active on any schedule. I had some work for the stability, authentication and other things.
I'm very happy for actual ideas how to implement this without loosing the simplicity of today.

Closing this. Nice idea, but the effort is too much. If someone will contribute, we can talk maybe.


Inconsistency of naming .sol files

Most of the .sol files are named after smart contracts defined in them, but there are several exceptions:

ResolverInterface.sol (defines AbstractPublicResolver smart contract)
interface.sol (defines AbstractENS smart contract)
HashRegistrarSimplified.sol (defines Deed and Registrar smart contracts)

This inconsistency makes it harder to navigate through the come and makes code reviews less efficient which in turn may lead to missed bugs.

Resolved 2. in #106


Fix command to override theme dashboard

This new command fix this issue https://github.com/Laravel-Backpack/theme-tabler/issues/178

thank you @jcastroa87 🙏


Disabling HTML5 pushstate

Hey guys, how to put back the #/some-url?

If I remember correctly, in your AppModule routes, you can do:
RouterModule.forRoot(routes, {useHash: true})

Thanks!!!


Feature Request: Option to clear cache DNS Unbound

An interesting option that I missed in the web interface is the possibility of cleaning the Unbound cache. If possible add this option in the interface would be of great help.
This action is possible through the reload command (without restarting the process).
Other options are by zone and by registers:
By zone:
unbound-control flush_zone google.com
By host:
unbound-control flush www.google.com
By type:
unbound-control flush_type name www.google.com aaaa

As of today you need to specify the config location with '-c'
unbound-control -c /var/unbound/unbound.conf stats
unbound-control -c /var/unbound/unbound.conf flush_zone google.com

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


Feign - How to set up my own default configuration?

Hi,
I like the Feign Spring Cloud support, but I'd like to provide custom default configuration (basically using JAX-RS contract and add a RequestInterceptor). Eventually, I want it to be a part of my company's Spring Boot starter.
Right now I can't even override current Feign autoconfiguration.
Is it because apart from Feign autoconfiguration: https://github.com/spring-cloud/spring-cloud-netflix/blob/master/spring-cloud-netflix-core/src/main/java/org/springframework/cloud/netflix/feign/FeignAutoConfiguration.java, there is an @Import statement in @EnableFeignClients annotation:
@Import({ FeignClientsConfiguration.class, FeignClientsRegistrar.class }), which sets up SpringMvcContract and rest of the properties?

You should be able to create a bean type Contract like:
@Bean
public Contract feignContract() {
	return new JAXRSContract();
}

I tried setting up the bean in several places (within @Component, @Configuration) with no luck.
No matter what I do, SpringMVCContract is always picked and used.

Have you tried using @Primary annoatation?
@Primary
@Bean
public Contract feignContract() {
    return new JAXRSContract();
}

Just tried and still no luck :(

I wanted to suggest last final sollution which basically would be based on excluding the FeignAutoConfiguration.class, something like:
@SpringBootApplication(exclude = FeignAutoConfiguration.class)

or
@EnableAutoConfiguration(exclude = FeignAutoConfiguration.class)

but this is not going to work since @EnableFeignClients does also include FeignClientsConfiguration

So, it seems that my initial assumption was right. Is that a bug ?

Please see the Overriding Feign Defaults section that was recently added.


Upgrade dependencies (mainly to pick up bugfix in apibuilder-validation)

[info] ScalaTest
[info] Run completed in 21 seconds, 222 milliseconds.
[info] Total number of tests run: 56
[info] Suites: completed 15, aborted 0
[info] Tests: succeeded 56, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[info] Passed: Total 56, Failed 0, Errors 0, Passed 56


python310Packages.dvc: update checksum

Description of changes
Once again is the checksum the cause of the build failure.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 264798 run on x86_64-linux 1

10 packages built:

dvc (python311Packages.dvc)
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist (python311Packages.dvc.dist)
python310Packages.dvc
python310Packages.dvc.dist
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvclive
python311Packages.dvclive.dist


inbuilt emmet extension.

Type: Feature Request
hey there ,
In my vs code there is not emmet visible in my vs code extension tab of setting.
Somehow it is also not showing in extension tab, and one thing more that whenever i open vs code it takes 7-9 seconds to launch but earlier it launch within a milisecond but now its not behaving like before.
VS Code version: Code 1.78.2 (b3e4e68a0bc097f0ae7907b217c1119af9e03435, 2023-05-10T14:39:26.248Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

In the extensions viewlet, try searching @builtin emmet.
Let me know if that helps find the extension.
As for the startup time, please follow the wiki section on slow startup issues and share the performance details you're seeing.

Got it. Thank you very much❤
On Fri, May 26, 2023, 10:34 PM Raymond Zhao @.***>
wrote:

In the extensions viewlet, try searching @builtin emmet.
Let me know if that helps find the extension.
As for the startup time, please follow the wiki section on slow startup
issues
https://github.com/microsoft/vscode/wiki/Performance-Issues#slow-startup
and share the performance details you're seeing.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/183500#issuecomment-1564679732,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6CXWPLMYBLN6AECJW3YDRLXIDPDJANCNFSM6AAAAAAYP3KS7M
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey @rzhao271, this issue might need further attention.
@Govind1326, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Yupp


Add overwrite protection to parent directory

This PR ensures that if using the --overwrite option and target is a parent directory of the project, the process will exit(1).
This is inline with ember-cli.

hmm, tests failing but not locally

@rwjblue do you know of any reason to allow [target] to be a direct parent directory to the build file?

I think I found the problem.
Was talking with Toby, but it seems that when mocking process.exit(), statements after the exit line are run in unit tests because the process does NOT exit and continues to run. With no return statement this means a build can run when an error is produced. I will whip up a quick test of this theory but this is what I’m seeing in existing tests.

@rwjblue @chrmod updated, tests passing.

Good work, thank you!!


RFE Support returning function/module documentation

I don't know what the underlying LSP service is, but several other language servers support returning documentation for functions and variables I assume based on the presence of comments in the corresponding source. It would be fantastic if you could support that as well.
The difficulty I can see is that there isn't really a standard for documenting openscad modules (although if you were to implement support for some documentation method that may very well help to establish a standard). Most libraries include module documentation in the comment immediately preceding the module definition, but the actual format can vary e.g.:
BOSL: https://github.com/revarbat/BOSL2/blob/master/beziers.scad#L178-L209
MCAD uses JavaDoc style: https://github.com/SolidCode/MCAD/blob/master/triangles.scad#L27-L34
I tend to use Javadoc style myself. I have seen a couple of libraries that include module documentation immediately after the module declaration, but this seems quite rare.

This is a useful feature, I will try in the near future.

Already added a simple support for that.


[0.30.4] Update JDT.LS

Closing this, merging this PR with https://github.com/kiegroup/kie-tools/pull/2051


Bump Plutus to GHC-8.10.7

Why?
Note that this will break the workaround for the questionable haskell.nix patch, so I don't want to merge it unless that's fixed.
Also you need to update the materialized files, have a read of CONTRIBUTING.

See thread here: https://input-output-rnd.slack.com/archives/C21UF2WVC/p1641799667005900
Long thread, so tldr:

cardano-wallet wants to use some plutus libraries in it's tests.
cardano-wallet then needs to depend on plutus/plutus-apps.
cardano-wallet uses GHC 8.10.7 (and needs to for some M1 fixes), build of Plutus wasn't working with GHC 8.10.7 due to something around that questionable haskell.nix patch and GHC.Tickish.
Tried some workarounds, no success.
So then tried to upgrade Plutus to GHC 8.10.7 and solve the underlying issue/fix the questionable haskell.nix patch.
In the meantime Moritz submitted a workaround for my issue, which works: https://github.com/input-output-hk/plutus/pull/4325

So the pull request isn't needed and can be closed as soon as https://github.com/input-output-hk/plutus/pull/4325 is merged.

cardano-wallet uses GHC 8.10.7 (and needs to for some M1 fixes), build of Plutus wasn't working with GHC 8.10.7 due to something around that questionable haskell.nix patch and GHC.Tickish.

That doesn't seem like it should be true. The GHC 8.10.7 in haskell.nix also has that patch applied: https://github.com/input-output-hk/haskell.nix/blob/master/overlays/bootstrap.nix#L175
What actually went wrong?

Oh I see - perhaps the issue is that you didn't get the change for the patch because of the workaround, hence the need for a flag you can turn on. How irritating.

I think it'll be best to approach upgrading the GHC version as part of it's own task. At the moment this is three layers deep on a rabbit hole for me, so I'll close this for now.


LinkedList to PyList conversion

Resolves #2405
Has been mentioned on Discord that converting a LinkedList to a PyList could be surprising for users.

CI failure is because impl<T, const N: usize> From<[T; N]> for LinkedList<T> is only available for Rust 1.56+. vec![1,2.3].into_iter().collect() is a decent way to do it for a test, I think.

What would the alternative be? As a data point pybind11 converts std::list to Python lists. https://github.com/pybind/pybind11/blob/0e956a2e4f7838fc26f42a6399bc358ae7248134/include/pybind11/stl.h#L200

I mean, they're very different data structures. I'm not convinced that linkedlist -> pylist is the obvious thing to do.


DDC error

 (master) $ bazelify serve
Building server via bazel...

WARNING: Sandboxed execution is not supported on your system and thus hermeticity of actions cannot be guaranteed. See http://bazel.io/docs/bazel-user-manual.html#sandboxing for more information. You can turn off this warning via --ignore_unsupported_sandboxing.
____Loading package:
____Loading package: @dart_style//
____Found 1 target...
____Building...
____[4 / 12] Compiling @build//:build with ddc
____[8 / 27] Writing file bwu_ng_mdl_worker_args
____[13 / 28] Writing file bwu_ng_mdl_ddc_args
ERROR: /home/zoechi/.cache/bazel/_bazel_zoechi/3e443aaa6a6cf1824cc4980f306f0a94/external/barback/BUILD:19:1: Compiling @barback//:barback with ddc failed: Worker process sent response with exit code: 1..
[error] Invalid override. The type of 'CancelableFuture.then' ('<S>((T) → S, {onError: Function}) → Future<S>') isn't a subtype of 'Future<T>.then' ('<S>((T) → dynamic, {onError: Function}) → Future<S>'). (package:barback/src/utils/cancelable_future.dart, line 32, col 3)

Please fix all errors before compiling (warnings are okay).
Target //:main_ddc_serve failed to build
Use --verbose_failures to see the command lines of failed build steps.
____Elapsed time: 2.173s, Critical Path: 1.59s

barback-0.15.2+9

Dart VM version: 1.21.0-edge.a850b4c977a5b177abe17864006657111d835840 (Mon Oct 31 18:04:44 2016) on "linux_x64"

/cc @jakemac53

This isn't really a barback issue, barback shouldn't be compiled with ddc (and thus doesn't need to be strong mode compliant).
The core problem here is any library that implements a transformer ends up with a barback dependency, but we need some way of stripping that out for web projects.
Opened https://github.com/dart-lang/bazel/issues/65 to track this.

At a high level, anything that uses a transformer doesn't work yet with bazel. We will make it clear in the dart-lang/bazel repository when we have it working with angular2


Design cost and scaleability measures

As a project developer, I want clear specifications for cost and scalability measures, so that expenses related to the collection and storage of telemetry data are minimized

[ ] Design doc created that defines the sampling percentage that should be used
[ ] Log retention timeframe documented
[ ] Log levels documented for all environments
[ ] Processes that require log rotation identified and the rotation process documented

Mentioned in #417


FormData polyfill for IE / Edge

This restores the ability to save changes in IE11 and Edge.

If you refer to Mozilla docs on FormData, cross-browser support is pretty shaky:
https://developer.mozilla.org/en-US/docs/Web/API/FormData#Browser_compatibility

Thanks @hooch
So, currently the form submission is not working in IE 11 and edge is that why the Polyfill is needed?
Just want to clarify.
Thanks.

Moving to 1.x as it requires rebuilding and republishing JS.


[sw,mask_rom] Configure Debug ROM in ePMP so debug can be enabled in RMA lifecycle

Low level initialization enables Debug ROM in ePMP, then configuration is updated/locked when lifecycle is read.
Signed-off-by: Daniel Beitel<EMAIL_ADDRESS>

Thanks for the PR! Can you add 'Fixes #10131' to the commit message. That way that issue will be closed when this PR is submitted.
Issue #10131 is on the list for a security opinion so it should be discussed in the next security meeting. I'll hold off reviewing this PR until we've had that discussion.
One thing to bear in mind is that the reset values for the ePMP registers are going to change so that eflash access is restricted when coming out of reset. So we'll need to add this change to the reset values as well if we want the RV_DM ROM to be unlocked immediately.

I totally understand.  Commit message updated.
I think I've got the DM ROM sequence set up properly so it won't interfere with the eFLASH setup.  I set up DM ROM for access early in the mask_rom boot (mask_rom_epmp_init), then I update and lock the configuration based on lifecycle state as soon as the lifecycle value is read from OTP.  I chose a free ePMP entry so, I don't think it conflicts with eFLASH.

@mundaym I was thinking about this change a bit more and I had a question.  Is JTAG enabled in other LCS steps (eg TEST and DEV) as well as RMA?  I may need to augment this to include the other LC states.

I need to tweak the PR a little to match HW LCS.  @mundaym JTAG is just disabled in Prod and Prod-End, correct?

I need to tweak the PR a little to match HW LCS. @mundaym JTAG is just disabled in Prod and Prod-End, correct?

I'm not certain. @moidx or @alphan might know.

Have a look here https://docs.opentitan.org/doc/security/specs/device_life_cycle/ - I don't know how to directlink to the figure but search for "HW_DEBUG_EN" which includes JTAG - it's also disabled in RAW / TEST_LOCKED / SCRAP / INVALID.

Ok, I have updated the PR to encompass all debug-enabled LC states.

I think I've got the DM ROM sequence set up properly so it won't interfere with the eFLASH setup. I set up DM ROM for access early in the mask_rom boot (mask_rom_epmp_init), then I update and lock the configuration based on lifecycle state as soon as the lifecycle value is read from OTP. I chose a free ePMP entry so, I don't think it conflicts with eFLASH.

I think @mundaym's comment here is about what happens when the chip comes out of reset before this code executes.

My apologies for the slow review @dbeitel-opentitan. The code looks good to me modulo some minor nits.

@mundaym Looks like there is was a recent change to mask_rom_epmp.S that removes enabling of whitelisting. That change likely removes the requirement to configure debug rom into ePMP in this file. Should I rework this PR?

Whitelisting is now enabled in the reset value for the MSECCFG register, so it is enabled without the need for the mask ROM to configure it explicitly. Is it OK that RV_DM_ROM acces is initially disabled and then enabled by the mask ROM code or does that break things? If it breaks things we will need to modify the reset values (
https://github.com/lowRISC/opentitan/blob/1e16f2cc2c78182d93c3276731f67110223247d4/hw/ip/rv_core_ibex/rtl/ibex_pmp_reset.svh#L26
).

I think this will be ok.  I will double check.

@dbeitel-opentitan are you happy for this to be merged?

Yes.  :-)
On Sat, Mar 5, 2022 at 5:01 AM Michael Munday @.***>
wrote:

@dbeitel-opentitan https://github.com/dbeitel-opentitan are you happy
for this to be merged?
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/10426#issuecomment-1059562500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKSQ5FRQ4VE5NIGTQ3DRLTU6KB3BANCNFSM5M7F2UCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Daniel Beitel
Firmware Engineer

Shoot, now I am running into a problem now when the debugger connects and attempts a reset of the core.  looks like the DV_ROM code is getting a second debug exception while executing and gets stuck in a loop.  I am investigating. (FYI, if I just attach the debugger with no reset the debug session works as expected.

@mundaym Ok, I updated opentitan/hw/ip/rv_core_ibex/rtl/ibex_pmp_reset.svh to include DV_ROM and now its working as expected.  Please check out the change in the PR.  Also, looks like the ROM entry isn't the entire 32kB, is this expected?  And MMIO is not locked, also expected?

Also, looks like the ROM entry isn't the entire 32kB, is this expected? And MMIO is not locked, also expected?

No that was not intentional, thanks for pointing those mistakes out. I've sent #11486 to address them (still draft as I want to double check the encodings are correct).

I updated this as well.

The PR should be ready to merge now.


Document update of all find functions in the search module with ontology annotation support

Document changes from SimPhoNy PR #819.

Closing (branch find_annotations was created from wrong branch).


[Feature request]: Improve pause functionality

First of all, thanks for making this plugin - works very well!
Using /manhunt pause I noticed that the day/night cycle did continue while manhunt was paused. I think it would be nice if the day/night cycle was paused during a paused manhunt game.
You could probably go as far as to set the game tick speed to 0 in newer versions that support this - just so that mobs are unable to move during a pause. Currently, hostile mobs can run up to you and surround you, leaving you in a tricky situation once you unpause (haven't tried it, but that's my theory).

Hi there!
Thank you so much for using and enjoying my plugin! I genuinely appreciate it. Also, I'm grateful for your advice.
Regarding the day/night cycle issue, I'll fix that right away. As for setting the tick speed to 0, I'll see if I'll make it, because I want it to be compatible for all 1.16+ versions. Your theory about hostile mobs is indeed true, so I'll try to find potential solutions.

Ok, so I fixed the day/night issue, but idk how to set the game tick speed to 0, idk if this is even possible. If mobs surrounded the player during the pause, they can kill them while still being paused, and also after resuming the game, everyone has 10 seconds of invulnerability. I hope that this is enough, and I will close the issue.

Thank you for taking care!


Thanks and question about multiple editors :)

First a big thanks for Xonomy, it is probably a thankless job and what should be the norm in every editing (schema / semantically guided editing)  keeps staying the exception.
But Xonomy is an incredible tool to maybe change that and at least enable very interesting applications.
(Substance is another approach to the problem for non XML document)
You mention right away in the presentation page of Xonomy that "There Can Only Be One" in a page.
Is that a hardcore intractable limitation or just a by-product of the "current state of the API" that could be changed relatively easily ?
I am thinking about multiple tabs or on demand editions of snippets of document outside the "main  editing zone" but in the same window, or multiple views on the same document as frequent use cases for such a thing.
It is probably possible to instantiate editor in particular div on demand but I suspect it is much more difficult than it seems in practice given the state of the editor at the time of the "div/context" switching.

Thank you for your kind words!
The "one only" limitation is just a by-product of my sloppy design and lazy programming. If I ever get to do a substantial rewrite of Xonomy I will make sure to avoid that shortcut.
The usual workaroud is to embed each Xonomy instance in an IFRAME. Your suggestion to create and destroy Xonomy instances on demand would of course work too but, as you rightly say yourself, this would not persist the state of the editor (if by "state" we mean which nodes are expanded/collapsed and so on).


Broken syntax in generated script

By default, the generated source JavaScript has syntax errors.
yo jquery vanilla
// ...
grunt

And I get this output:
Running "jshint:gruntfile" (jshint) task

✔ No problems

Running "jshint:src" (jshint) task

src/yo-jquery-vanilla.js
  line 10  col 9   Expected an identifier and instead saw '='.
  line 18  col 6   Expected an identifier and instead saw '='.
  line 20  col 30  Expected an identifier and instead saw '.'.
  line 26  col 5   Expected an identifier and instead saw '.'.
  line 31  col 16  Expected an identifier and instead saw '='.

  ✖  5 errors  ⚠  0 warnings

Warning: Task "jshint:src" failed. Use --force to continue.

Aborted due to warnings.

because
cat src/yo-jquery-vanilla.js

/*
 *
 *
 *
 * Copyright (c) 2015
 * Licensed under the MIT license.
 */
(function ($) {
  // Collection method.
  $.fn. = function () {
    return this.each(function (i) {
      // Do something to each selected element.
      $(this).html('' + i);
    });
  };

  // Static method.
  $. = function (options) {
    // Override default options with passed-in options.
    options = $.extend({}, $..options, options);
    // Return the name of your plugin plus a punctuation character.
    return '' + options.punctuation;
  };

  // Static method default options.
  $..options = {
    punctuation: '.'
  };

  // Custom selector.
  $.expr[':']. = function (el) {
    return $(el).text() === 'awesome test markup';
  };
}(jQuery));

This seems to be due to <%= camelname %> resolving to '.' everywhere.

I was able to confirm this.

Yay!

Thanks! :D:D


v8(services): consistently handle whitespaces

Solves #1794
#168509681

A couple of thoughts:

this fix is in the CLI, did you consider fixes in the CC?

The API ignores whitespaces when creating/updating, but doesn't handle them correctly when querying. Might be good to fix that on the API as well 🤔

I would slightly prefer an approach that involves implementing a custom unmarshal like this: https://github.com/cloudfoundry/cli/blob/0d68f88697b720575fa0bad866b24d2f9354c0aa/command/flag/credentials_or_json.go#L18

func (s *ServiceInstance) UnmarshalFlag(input string) error {
  *s = strings.TrimSpace(input)
  return nil
}

Nice. I'll take a look at that.


Updated terminology, the task assign response and the volume definition

The task assign example response was quite different from what the server was actually returning, so I changed that to reflect what the server responds with at the current time. Some of the values in the response aren't exactly known to me so those still have to be filled in.
The terminology had some incorrect parts in it and others which needed more description so I changed what I saw was wrong.

Changed it so it doesn't reference omni-only things and also changed call name from save to submit

Should be fine now


Animate is not a function

Hi,
First of all, thank you for this component. It is very good. However, I can not change the animation with
refs, and animate() function.. React native says that animate is not a function. I am on Expo.. Do you know about this issue?

Had to dig through the source code to find it. The function has been changed to 'performLinearAnimation (toValue, duration)'.
As of it right now, it doesnt accept an easing funtion, instead its hardcoded to linear, as indicated by the function name.

This happened because version 1.0.0 wasn't published to npm yet. It has now been published, please update 👍


Exception on frames without objects

Describe the bug
The following error is raised when a frame without object is given:
Traceback (most recent call last):
  File "neuralet-distancing.py", line 22, in <module>
    DistanceApp(args)
  File "neuralet-distancing.py", line 16, in __init__
    self.ui.start()
  File "/repo/applications/smart-distancing/ui/web_gui.py", line 191, in start
    self.__ENGINE_INSTANCE.process_video(video_path)
  File "/repo/applications/smart-distancing/libs/core.py", line 94, in process_video
    self.logger.update(objects, distancings)
  File "/repo/applications/smart-distancing/libs/loggers/loggers.py", line 48, in update
    self.logger.update(objects_list, distances)
  File "/repo/applications/smart-distancing/libs/loggers/csv_processed_logger.py", line 43, in update
    self.log_objects(objects_list, distances, objects_log_file_path)
  File "/repo/applications/smart-distancing/libs/loggers/csv_processed_logger.py", line 57, in log_objects
    violating_objects = extract_violating_objects(distances, self.dist_threshold)
  File "/repo/applications/smart-distancing/tools/objects_post_process.py", line 18, in extract_violating_objects
    triu_distances = np.triu(distances) + np.tril((float(dist_threshold) + 1) * np.ones(distances.shape))
  File "<__array_function__ internals>", line 6, in triu
  File "/usr/local/lib/python3.6/dist-packages/numpy/lib/twodim_base.py", line 465, in triu
    mask = tri(*m.shape[-2:], k=k-1, dtype=bool)
TypeError: tri() missing 1 required positional argument: 'N'

To Reproduce
Use dummy detector

It turns out the problem was not with zero objects, but with None in https://github.com/neuralet/neuralet/blob/8b1d6fb811225dcdf4485074c1897741445e6d5c/applications/smart-distancing/libs/core.py#L52


[Experimental Dependencies] Add -enable-experimental-dependencies and push it through.

Adds -enable-experimental-dependencies option and pushing it through to the places where it will be needed in the frontend and driver. Getting this in now will reduce future rebasing work.

@swift-ci please smoke test os x platform

@swift-ci please smoke test

@swift-ci please smoke test linux platform


Temporarily fix for using multisites  advanced

There are 2 way to setup multisites in Grav CMS.
The first way: with different plugins set for every sites - it was supported already with earlier temporarily fix
The second way: COMMON plugins set for all domains in multisite was unsupported.
With this modification you can use a stream definition in your setup.php like this
'pluginupdates' => [
'type' => 'ReadOnlyStream',
'paths' => [
'user/plugins',
],
'prefixes' => [
'' => ['user/plugins'],
]
],
And if this stream definition exists, the plugin install path will not be modified and will be suitable for the common use in 'user/plugins' folder.

You can already do this by defining plugins:// and plugin:// stream not to use user:// stream.

@mahagr I am sorry, you completely misunderstood. The point of this modification was not to create a new stream, but to make the hardcoded plugin install folder variable depending on the multisite settings. Do you see the suggested code?

If this part is not handled, from line 57, the line 61 will overwrite the plugin install folder and you cannot set it up form the setup.php
foreach (json_decode($this->raw, true) as $slug => $data) {
            // Temporarily fix for using multisites
            if (isset($data['install_path'])) {
                $path = preg_replace('~^user/~i', 'user://', $data['install_path']);
                $data['install_path'] = Grav::instance()['locator']->findResource($path, false, true);
            }
            $this->items[$slug] = new Package($data, $this->type);
        }


Update quota-memory-cpu-namespace.md

fix bad url

This change is

Also, please resolve the merge conflict. Thx


[feature] allow test notifications to use title and body in settings

Version and OS
0.44 linux/docker
Is your feature request related to a problem? Please describe.
It is not possible to test a notification for a JSON endpoint
Describe the solution you'd like
It would be great to have a dummy notification that has values for all of the variables that can be used in the template. Then, when sending a test notification it could use the title and body set below.
Describe the use-case and give concrete real-world examples
I am sending notifications to a self-hosted ntfy server with authentication. I need to send raw JSON. When I get a real notification it works, but the test notification only sends some text. That test text is not accepted by ntfy.

0.45.3
Is it true that the test notifications completely ignore the template settings? I have customized my template but when I click "send test" it always uses the default format.

I think this needs storage of the HTML of the current and previous fetches, so it can recalculate and resend it as a notification

Well for my needs, it can just be some static test page that gets sent with the notification. I am more concerned with the test actually using the template setup in the settings, and having at least the basic fields available.


letsencrypt.sh on CentOS 5 doesn't see curl

I've pulled a copy of the letsencrypt.sh script from github earlier today on my CentOS 5-based system (SME Server 8.2; see contribs.org).  I've created a config.sh and a domains.txt file.  But when I run letsencrypt.sh, I get this:
[root@sme8-test letsencrypt.sh]# ./letsencrypt.sh -c
ERROR: This script requires curl.
curl is installed and is in $PATH:
[root@sme8-test ~]# curl -V
curl 7.15.5 (x86_64-redhat-linux-gnu) libcurl/7.15.5 OpenSSL/0.9.8b zlib/1.2.3 libidn/0.6.5
Protocols: tftp ftp telnet dict ldap http file https ftps
Features: GSS-Negotiate IDN IPv6 Largefile NTLM SSL libz
[root@sme8-test ~]# which curl
/usr/bin/curl
So I figured I'd comment out the line in letsencrypt.sh that checks for curl and run it again.  That didn't work at all:
[root@sme8-test bin]# letsencrypt.sh -c
INFO: Using config file /etc/letsencrypt.sh/config.sh

Generating account key...
sed: invalid option -- E
Usage: sed [OPTION]... {script-only-if-no-other-script} [input-file]...
-n, --quiet, --silent
suppress automatic printing of pattern space
-e script, --expression=script
add the script to the commands to be executed
-f script-file, --file=script-file
add the contents of script-file to the commands to be executed
-i[SUFFIX], --in-place[=SUFFIX]
edit files in place (makes backup if extension supplied)
-c, --copy
use copy instead of rename when shuffling files in -i mode
(avoids change of input file ownership)
-l N, --line-length=N
specify the desired line-wrap length for the `l' command
--posix
disable all GNU extensions.
-r, --regexp-extended
use extended regular expressions in the script.
-s, --separate
consider files as separate rather than as a single continuous
long stream.
-u, --unbuffered
load minimal amounts of data from the input files and flush
the output buffers more often
--help     display this help and exit
--version  output version information and exit

If no -e, --expression, -f, or --file option is given, then the first
non-option argument is taken as the sed script to interpret.  All
remaining arguments are names of input files; if no input files are
specified, then the standard input is read.
E-mail bug reports to<EMAIL_ADDRESS>.
Be sure to include the word sed'' somewhere in the Subject:'' field.
sed: invalid option -- E
Usage: sed [OPTION]... {script-only-if-no-other-script} [input-file]...
-n, --quiet, --silent
suppress automatic printing of pattern space
-e script, --expression=script
add the script to the commands to be executed
-f script-file, --file=script-file
add the contents of script-file to the commands to be executed
-i[SUFFIX], --in-place[=SUFFIX]
edit files in place (makes backup if extension supplied)
-c, --copy
use copy instead of rename when shuffling files in -i mode
(avoids change of input file ownership)
-l N, --line-length=N
specify the desired line-wrap length for the `l' command
--posix
disable all GNU extensions.
-r, --regexp-extended
use extended regular expressions in the script.
-s, --separate
consider files as separate rather than as a single continuous
long stream.
-u, --unbuffered
load minimal amounts of data from the input files and flush
the output buffers more often
--help     display this help and exit
--version  output version information and exit
If no -e, --expression, -f, or --file option is given, then the first
non-option argument is taken as the sed script to interpret.  All
remaining arguments are names of input files; if no input files are
specified, then the standard input is read.
E-mail bug reports to<EMAIL_ADDRESS>.
Be sure to include the word sed'' somewhere in the Subject:'' field.

Registering account key with letsencrypt...

ERROR: An error occurred while sending post-request to https://acme-staging.api.letsencrypt.org/acme/new-reg (Status 500)

Details:
{"type":"urn:acme:error:serverInternal","status":500}[root@sme8-test bin]#
I'm not sure what to try from here--thoughts?

Mh, looks like your sed version is too old to support the -E (Interpret regular expressions as extended (modern) regular expressions rather than basic regular expressions (BRE's).) option.
I added that option to the scripts dependency checks in 63ab72a5217c51bb0b04607626b5f6f7284db38b.
Unfortunately I kinda need to use that option to get support for BSD and GNU versions of sed at the same time without duplicate code all over the place...
I'd suggest getting a newer version of sed. About the curl problem... I don't really understand why the check fails... Does curl -V return with an exit-code != 0 on your system? That seems a bit odd.

[root@sme8-test ~]# curl -V
curl 7.15.5 (x86_64-redhat-linux-gnu) libcurl/7.15.5 OpenSSL/0.9.8b zlib/1.2.3 libidn/0.6.5
Protocols: tftp ftp telnet dict ldap http file https ftps
Features: GSS-Negotiate IDN IPv6 Largefile NTLM SSL libz
[root@sme8-test ~]# echo $?
2
[root@sme8-test ~]#
I'll see if there's a well-supported way to update sed.  This system has GNU sed version 4.1.5; a CentOS 6 system (which does work) has GNU sed 4.2.1.

Exit code 2 should be given if the initialization of libcurl failed, which should never happen on just asking for the version... i could probably build a workaround for this, but this seems weird.
You could just build sed from source, place it anywhere you like and set the PATH environment variable for letsencrypt.sh to use this location.

Re sed please see my comment https://github.com/lukas2511/letsencrypt.sh/commit/63ab72a5217c51bb0b04607626b5f6f7284db38b#commitcomment-15485814
sed should be -E OR -r
-E is not a normal switch - http://blog.dmitryleskov.com/small-hacks/mysterious-gnu-sed-option-e/
If I swap to -r it works perfectly well.
No idea on the curl issue but if you hash out the check it works.

sed -r doesn't work on BSD versions of sed while sed -E works on both GNU and BSD versions, at least on newer versions of both.

Not on the one we are using, and we can't change that ! RHEL/CentOS 5 is still there and supported.

I was trying to figure if I could rewrite this a bit but this is way beyond my primitive bash skills.
I believe the issue lies with the way that set -e handles error codes.
On the version of sed being used 4.1.5 I get the following results
sed -r "" < /dev/null
echo $?
0
sed -E "" < /dev/null
echo $?
4
But quite clearly this version of sed does support extended regular expressions (as shown if we look at sed -r) so the error checking is at fault.
My guess there is a similar issue with curl - on CentOS 5 it gives a result of 2 but on CentOS 6 it results in 0. If the checks are removed the script functions as expected.

If the issue is that some versions of sed require the -r flag, and others require the -E flag, wouldn't it be possible (perhaps in the compatibility check section) to determine which (if any) is the appropriate flag for the version installed on the system on which the script is running, set a variable to contain that flag, and then call sed using "sed $SED_OPTS" rather than "sed -E"?

or enable a hook via config.sh?

Since SME8 is customized in a number of ways, I installed a clean CentOS 5 system to test curl.  Same thing:
[dan@pc-00226 ~]$ curl -V
curl 7.15.5 (x86_64-redhat-linux-gnu) libcurl/7.15.5 OpenSSL/0.9.8b zlib/1.2.3 libidn/0.6.5
Protocols: tftp ftp telnet dict ldap http file https ftps
Features: GSS-Negotiate IDN IPv6 Largefile NTLM SSL libz
[dan@pc-00226 ~]$ echo $?
2
Since RHEL5/CentOS5 is still an actively supported OS, it would be good to deal with this.
On sed, it looks like sed -E is called twice in the script.  Surely there's a way of dealing with -r instead?

I note that what I thought was one issue (le.sh doesn't see curl) is actually two--le.sh doesn't see curl in CentOS 5, and le.sh uses sed -E, which isn't available in CentOS 5.  Would it be beneficial to open a second issue for sed?

I'm working on the sed issue, about the curl issue i'll see later...

The sed issue should be fixed f7c7d8c5991d78e5666de3b69ee63844266eaf21

The curl issue should also be fixed 0af7f388d6e63e74920e5f0cc9ae21f717d223d1

I'm afraid that didn't fix it:
[root@sme8-test letsencrypt.sh]# ./letsencrypt.sh
ERROR: This script requires sed with support for extended (modern) regular expressions.
[root@sme8-test ~]# uname
Linux

Moved OSTYPE check (was executed after dependency-checks) in c3c9ff4c75643e0b878aa0f71ea39b6020b06ef6, try again.

It now runs without giving an error--no time to test extensively at the moment though.

Thanks Lukas !


minor improvement to cheapOneOf0

We were potentially hiding a OneOf by not carefully checking if each argument might be a Parser.

Codecov Report

Merging #400 (1b7c7fe) into main (fe8adc5) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #400      +/-   ##
==========================================
- Coverage   96.72%   96.66%   -0.07%
==========================================
  Files           9        9
  Lines        1284     1288       +4
  Branches      124      133       +9
==========================================
+ Hits         1242     1245       +3
- Misses         42       43       +1

Impacted Files
Coverage Δ

core/shared/src/main/scala/cats/parse/Parser.scala
96.44% <100.00%> (+0.01%)
:arrow_up:

...shared/src/main/scala/cats/parse/Accumulator.scala
97.56% <0.00%> (-2.44%)
:arrow_down:

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 fe8adc5...1b7c7fe. Read the comment docs.


API Exception 404

Hi,
I want to install Stethoscope on a Debian 8 server. I basically followed those steps:

Install node.js and python3-virtualenv
Clone the repo
Create a virtualenv and install backend using make develop
Compile frontend using make react-build
Adapt the nginx configuration file to the correct path
Run backend using stethoscope-login runserver -p 5002 and twistd -n web -p 5001 --class=stethoscope.api.resource.resource

Now I can see the UI interface, with the message "Error loading notifications" and a redirection to "/user%40example.com". I've got this error on the API stdout:
2017-04-13T14:08:14+0200 [-] Site starting on 5001
2017-04-13T14:08:14+0200 [twisted.web.server.Site#info] Starting factory <twisted.web.server.Site object at 0x7f11ec378780>
2017-04-13T14:08:28+0200 [stderr#error] [2017-04-13 12:08:28.673352] ERROR: stethoscope.api.factory: Exception: 404: Not Found
2017-04-13T14:08:28+0200 [stderr#error] Traceback (most recent call last):
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/twisted/web/http.py", line 766, in requestReceived
2017-04-13T14:08:28+0200 [stderr#error]     self.process()
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/twisted/web/server.py", line 190, in process
2017-04-13T14:08:28+0200 [stderr#error]     self.render(resrc)
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/twisted/web/server.py", line 241, in render
2017-04-13T14:08:28+0200 [stderr#error]     body = resrc.render(self)
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/klein/resource.py", line 201, in render
2017-04-13T14:08:28+0200 [stderr#error]     d = defer.maybeDeferred(_execute)
2017-04-13T14:08:28+0200 [stderr#error] --- <exception caught here> ---
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/twisted/internet/defer.py", line 150, in maybeDeferred
2017-04-13T14:08:28+0200 [stderr#error]     result = f(*args, **kw)
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/klein/resource.py", line 179, in _execute
2017-04-13T14:08:28+0200 [stderr#error]     (rule, kwargs) = mapper.match(return_rule=True)
2017-04-13T14:08:28+0200 [stderr#error]   File "/data/services/web/stethoscope/stethoscope-venv/lib/python3.4/site-packages/werkzeug/routing.py", line 1563, in match
2017-04-13T14:08:28+0200 [stderr#error]     raise NotFound()
2017-04-13T14:08:28+0200 [stderr#error] werkzeug.exceptions.NotFound: 404: Not Found

Did I missed some configuration or dependency?
Thanks, you're project seems great and I can't wait to use it ;)
Kind regards,

Thanks for the report. I'm looking into this, but I'll note off the bat that we haven't done much testing on Python 3 nor on stock Debian.

I'm so far unable to replicate this issue. I'll keep digging into it but it may be next week before I have much time to do so.

I still haven't been able to replicate this exact issue. However, #23 does fix a Python 3 issue which may be related. If you could give the current master a try an let me know if it fixes your issue, that would be great.

Hi,
I pulled the repo and retried but the result is still the same. I will try to reinstall from scratch and let you know if it persists.
Kind regards,

I'll should have some time early next week to attempt to reproduce this on an actual Debian image. I'll report back once I've done so.

I was finally able to replicate this. The issue, I suspect, is that you haven't configured any notification plugins. I'll work on improving the log messaging to indicate when this is the case.

Closing since it's been 7 days without a response.


lack of validation on returned case citations

I am doing some experiments on some of the "Statutes at Large" search-able PDFs on FDsys. The text layer presumably contains raw OCR output, since it contains a lot of errors. I am extracting the text layer and sending it to cite-server running locally.
The following code snippets return false positives:
Citation.find("pursuant to 5 use 552(a)(1)(E) and") // "use" instead of "usc"
Citation.find("pursuant to 5 GARBAGE 552(a)(1)(E) and")
Citation.find("The sum of 27 and 42 is a number between 68 and 70.") // two citations found!

I am seeing the first case ("use") often where US Code citations in historical documents often omit periods in the abbreviation "USC" (see https://www.gpo.gov/fdsys/pkg/STATUTE-70/content-detail.html, open the PDF, search for the string "use", and see it highlighted often in the margins). I think the OCR engine that generated the text guessed "use", a word more common in everyday English than "USC". (Just in case, I'm NOT suggesting that it is the responsibility of the citation finder to anticipate and fix things like OCR errors.)
The last case has been popping up every once in a while, where you have a single word in between two numbers (see https://github.com/unitedstates/citation/issues/100).
Generally, the issue seems to be that citations of the reporter type are not being properly validated before being returned to the caller of Citation.find.

This is a great writeup of the problem, thank you! I'll take a look into this, though I don't have an ETA for it. If you're using this in something where time is of the essence, let me know -- and I'd welcome a pull request with a fix, if you have one.

Weird. Seems like the regex here would only allow USC or U.S.C.:
https://github.com/unitedstates/citation/blob/master/citations/usc.js#L51
Is it being picked up as a U.S.C. citation?

@konklone not time sensitive for me. I'm happy to collaborate on this issue though. I wouldn't be able to take it on entirely myself (not too knowledgeable about law and legal citations) but I am quite good with regular expressions.
@mlissner each of the examples I mentioned are being interpreted as citations of type reporter


New env variable FORCE_RESET_PERMISSIONS to control resetting nginx webroot permissions

Hi,
Copy paste from the other PR docker-nginx ownership adjustsments are also done here.
Some times it is not a good idea to always force webroot ownertship at container boot, for example when you have a very big webroot that is over NFS, and several containers will be doing that at the same time over the same files.
This PR adds a new env variable called FORCE_RESET_PERMISSIONS to control if ownership of files should be done.
Would it make sense to leave chown's  on the nginx image ? only run them when it is a new wordpress install, or better yet, make 09-nginx run last ?

I think it would be good if I started integrating this into all of my images. I have it all over the place due to some quirks related to installing and also as a santiy check as sometimes I have people heading into our exported folders and making changes to files/adding files under the wrong uid which drives me up the wall. It should be quite easy to rework this as it usually sits in every single one of my images as the last statement, except in event of doing things like install where its all over the place. At some point I'm moving things towards doing installs more so under the ${NGINX_USER} and moving away from executing things as root. That's going to take a wee bit longer. I'll start thinking about this over the holidays and start work in the NY.

Excellent, have a nice holidays.

New images are just in testing on my private servers now. Couple more days and I'll be merging them to Github.

Excellent :+1:

I've thrown Docker Hub on a loop for building - The images that are there are indeed working, but there are new ones coming within the next 24 hours. Might want to let it settle down before you proceed for usage.

Excellent, for now I'm using my own images with this change so no worries for now.


I can't export my project

Tested versions
Godot 4.2.2 stable
System information
Android 12 - Godot 4.2.2 stable - mobile
Issue description
I can't reduce the export window. That's why

I can't export the game
Steps to reproduce
Export project
Minimal reproduction project (MRP)
🥶

Duplicate of #73130, this will be fixed in 4.3.


How to choose? Local setup on VM vs Remote AKS cluster?

Topic
FarmVibes.AI setup, Local cluster, Remote (AKS) cluster
Ask away!
Hi Team,
I would like to get clarity purely from cost perspective what and how to choose! What are cost implications of setting an Azure VM and then do the installation vs setting up a remote cluster?

I initially thought that AKS would be cheaper but seeing the cost in resource group made me believe that probably they would cost almost same.
On an Azure VM with installation I can run notebooks and connect via my local machine via port forwarding. I believe I can’t do the same on Remote AKS setup! Please guide.
What’s the situation where you would choose Azure AKS setup over VM setup?

Thanks,
Chetan

Hello @chetan2309, thanks for your question!

If memory serves, an empty VM using the default configuration on Azure should be cheaper than an AKS cluster.
When you setup an AKS cluster, FarmVibes.AI gives you a public endpoint you can connect to. That endpoint is open by default, so you point your notebooks to it and run your workflows directly. (If you have security requirements, we encourage you to restrict access from the Azure portal itself.)

In general, you'd want to use a VM for development and experimentation and once you get into "steady state" of execution, you should probably migrate to AKS, as that'll allow for faster workflow execution and automatic storage scaling.
AKS has two other advantages as well: it is inherently multi-user (and multi-admin), and storage costs scale much slower than those of the VMs, because on AKS the asset storage is backed by Azure Blobs, whereas the VM uses HDDs or SSDs.

@renatolfc - thanks for your detailed explanation on this.


Response.data() silently failing when parsing invalid JSON

When getting a response back from an API and calling the ´.data()´ function on the response object, it will swallow any error from trying to parse the JSON. If it fails it will just return the string data. What is the reasoning behind this decision? as it seems odd to not just throw an error here as if you're expecting an object back and get something invalid then it should throw an error.
The function in question: https://github.com/tulios/mappersmith/blob/master/src/response.js#L92

At the surface, There are 4 basic cases to consider here:

Invalid JSON in rawData and headers are NOT content-type json => returns rawData - This is OK!
Valid JSON in rawData and headers are NOT content-type json => still returns rawData - This is OK!
Valid JSON in rawData and headers are content-type json => returns JSON.parse(rawData) - This is OK!
Invalid JSON in rawData and headers are content-type json => returns rawData - "This seems odd, it should throw" <-- this is the case we want to discuss

So before answering why (4) behaves like this, let's think about why would a case like (4) even happen? Here is my I guess:
Reason A: A content-type json header was incorrect set by the remote while sending a plain text body.
Analysis A: In this case you as the caller didn't expect JSON back anyway, you expected text so it is actually GOOD that it doesn't throw in this case.
Reason B: A content-type json header was correctly set by the remote, but it has problems serializing json and also didn't validate it.
Analysis B: In this case you are expecting JSON, I agree that in this case you would expect mappersmith to throw actually.
But in the end, I don't think A is about a million times more likely to happen than B, so I would not agree to change the code with my current understanding. But maybe I am missing something?
If this really is a problem, I can offer a workaround that you can use rawData() instead and do your own JSON parsing just to be sure.


java.lang.IllegalAccessError

I'm getting this error but not sure why. Locally everything is fine but in Firebase logs I can see it:

[✓] Flutter (Channel stable, 2.5.3, on macOS 11.6.1 20G224 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.62.0)
[✓] Connected device (4 available)

• No issues found!

The stack trace here doesn't seem to have anything to do with Firebase so not sure this is the right place for this issue?

The stack trace here doesn't seem to have anything to do with Firebase so not sure this is the right place for this issue?

No, not sure at all, but I just get this in Firebase and not sure what it complains about. Any idea maybe?
I can close this if it sounds unrelated.

@erperejildo
Although it's not super clear from the screenshots, but it seems to be pointing to native backend stack and may have to do with trying to access the db but unable to do so. You might want to search for it a bit more to get more ideas on it.
Since it doesn't look like related to flutterfire plugins, I'll close it. If you get concrete steps leading to any of the flutterfire plugins, write in comments and I'll reopen it.
Thanks.

@erperejildo Although it's not super clear from the screenshots, but it seems to be pointing to native backend stack and may have to do with trying to access the db but unable to do so. You might want to search for it a bit more to get more ideas on it. Since it doesn't look like related to flutterfire plugins, I'll close it. If you get concrete steps leading to any of the flutterfire plugins, write in comments and I'll reopen it. Thanks.

I know. I tried to get more logs but that's what I see from Play console app. I'll try to gather more info


[codemod] Add accordion props deprecation

part of #40417
transformation:
 <Accordion
-    TransitionComponent={CustomTransition}
-    TransitionProps={{ unmountOnExit: true }}
+    slots={{ transition: CustomTransition }}
+    slotProps={{ transition: { unmountOnExit: true } }}
 />

existing slots and slotProps are also handled.
Note
Adjusting the codemod structure a bit. It was really painful in v5 to navigate between codemod (the implementation and test cases are separated in different folder.
Moving forward, I think the implementation and test cases should be in the same folder. This is just a DX improvement for our team, there is no impact on consumers.
So from:
/src
  /version
    /transformation.test
      actual.js
      expected.js
    …a lot of folder
    transformation.js
    transformation.test.js

to:
/src
  /version
    /transformation
      /test-cases
        actual.js
        expected.js
      transformation
      transformation.test.js
      index.js
    …a lot of folder

[x] I have followed (at least) the PR section of the contributing guide.

Makes sene! 👍

Another question regarding codemods, the <path> provided is traversed recursively? I mean given
my-library/
├── my-components/
│   ├── my-accordion.tsx

If I run the codemod on the my-library path, would my-accordion.tsx be modified?

Another question regarding codemods, the <path> provided is traversed recursively? I mean given
my-library/
├── my-components/
│   ├── my-accordion.tsx

If I run the codemod on the my-library path, would my-accordion.tsx be modified?

Yes, that's right.

I tried running the codemod locally to modify the test cases (just to try it out), and it modifies tests-cases/actual.js as expected, but it doesn't modify test-cases/theme.actual.js. You know why that might be?

I figured it out: the difference lies in the AST parsing. For some reason, when I run the codemod locally the TransitionComponent is considered an ObjectProperty instead of a Property (related issue). For some reason, it must be running with a different parser, babel, maybe. I think the correct type is Property, as far as I understand, jscodeshift should be using recast under the hood, and recast users Property, not ObjectProperty. Maybe it's a limitation of running the codemod locally, so we should merge as is (after fixing this) and see if it works when bundled.

I tried running the codemod locally to modify the test cases (just to try it out), and it modifies tests-cases/actual.js as expected, but it doesn't modify test-cases/theme.actual.js. You know why that might be?

I figured it out: the difference lies in the AST parsing. For some reason, when I run the codemod locally the TransitionComponent is considered an ObjectProperty instead of a Property (related issue). For some reason, it must be running with a different parser, babel, maybe. I think the correct type is Property, as far as I understand, jscodeshift should be using recast under the hood, and recast users Property, not ObjectProperty. Maybe it's a limitation of running the codemod locally, so we should merge as is (after fixing this) and see if it works when bundled.

Yes, you are right. It happens to me locally as well.

@DiegoAndai I think it's good to go. I added a CONTRIBUTING.md in this PR as well because I always forget about the steps and script when testing locally.


feat(subtract): added Subtract function to wire

Fixes #8

cc @jayzhuang, please give some comments or suggestions here. <3

I'm really in need of this PR. Is there any progress ?

I also want this. But there are no review from the core team. So...


The request addess is not valid in this context

Added the IP address that shows up in IPconfig in both the sender and the receiver but I get the above message in the console when running in the Editor

Having a similar problem with above "Unable to get hand data, System.Threading.ThreadAbortException
".
-Same network
-Opened port 27000
-Disabled firewall
-Same ip adress on both sender and receiver
Unable to get hand data, System.Threading.ThreadAbortException
at (wrapper managed-to-native) System.Net.Sockets.Socket.ReceiveFrom_internal(intptr,byte*,int,System.Net.Sockets.SocketFlags,System.Net.SocketAddress&,int&,bool)
at System.Net.Sockets.Socket.ReceiveFrom_internal (System.Net.Sockets.SafeSocketHandle safeHandle, System.Byte* buffer, System.Int32 count, System.Net.Sockets.SocketFlags flags, System.Net.SocketAddress& sockaddr, System.Int32& error, System.Boolean blocking) [0x0000c] in :0
at System.Net.Sockets.Socket.ReceiveFrom (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags, System.Net.EndPoint& remoteEP, System.Net.Sockets.SocketError& errorCode) [0x00024] in :0
at System.Net.Sockets.Socket.ReceiveFrom (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags, System.Net.EndPoint& remoteEP) [0x00026] in :0
at System.Net.Sockets.Socket.ReceiveFrom (System.Byte[] buffer, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags, System.Net.EndPoint& remoteEP) [0x00000] in :0
at System.Net.Sockets.UdpClient.Receive (System.Net.IPEndPoint& remoteEP) [0x00036] in :0
at Assets.RemoteHandsTracking.HandsDataReceiver.ListenForUdpData () [0x00005] in C:\Users\Stefano\Documents\Repos\KineticXr-Wedge\ProjectWedge\Assets\RemoteHandsTracking\HandsDataReceiver.cs:68

Hi @StefanoDef ,
There are 2 issues that I was able to identify

first, error in editor, as you see

for me that was down to invalid IP, I was using loopback <IP_ADDRESS>, what is exact IP you're using?

second was quest app freezing on startup

this was again IP not reachable via quest, it's freezing as Quest is making blocking call to IP address, if all goes well you don't see that but if IP is not reachable it's freezing.

Generally I don't think it'll be code issue but rather network. Are you familiar with wireshark? https://www.wireshark.org/ - it may be easiest to run it on PC and see if any packets are coming in from Quest.

Hi @handzlikchris
Thks for your reply,
I tried with my pc ip on sender/receiver and headset ip on sender/receiver
also tried these ips on both, in short, any combination, but could not get it working.Tbh havnt noticed any freezing, just errors. A couple of times, no errors were displayed but could not see my hands nonetheless. I will investigate with wireshark and try with the loopback ip.


Tokenomics consulting request

form to submit such a request.

Update it with the content from the consulting description
How to design tokens for new protocols;
Tokenomic simulations;
Think tanks for new projects;
Peer Review of white paper or token papers.

https://forms.gle/h5hQeTvysB5xpC1K8


[C-API] TessBaseAPIRecognize() segfaults when no language data files is available

Hello,
When working on https://github.com/jflesch/pyocr/issues/51 , someone reported to me the following issue:
When no language data file is available at all (not even English), TessBaseAPIRecognize() segfaults.
Also happens when TESSDATA_PREFIX is set to an invalid directory.
Tested with libtesseract 3.04.01 (Debian Sid).
Example code:
#include <stdio.h>
#include <stdlib.h>

#include <leptonica/allheaders.h>

#include <tesseract/capi.h>

/*
 * gcc -Wall -Werror -g -ggdb -ltesseract -llept test.c -o test.out
 * ./test.out ~/git/pyocr/tests/input/real/basic_doc.jpg
 */

int main(int argc, char **argv)
{
	TessBaseAPI *handle;
	PIX *image;

	setenv("TESSDATA_PREFIX", "/opt/tulipe", 1 /* overwrite */);

	handle = TessBaseAPICreate();
	TessBaseAPIInit3(handle, NULL, "fra");
	TessBaseAPISetVariable(handle, "tessedit_zero_rejection", "F");
	image = pixRead(argv[1]);
	TessBaseAPISetImage2(handle, image);

	TessBaseAPIRecognize(handle, NULL);

	TessBaseAPIDelete(handle);

	return 0;
}

Stacktrace:
(gdb) bt
#0  0x00007ffff76f2597 in tesseract::Tesseract::recog_all_words(PAGE_RES*, ETEXT_DESC*, TBOX const*, char const*, int) () from /usr/lib/x86_64-linux-gnu/libtesseract.so.3
#1  0x00007ffff76dcb7f in tesseract::TessBaseAPI::Recognize(ETEXT_DESC*) ()
   from /usr/lib/x86_64-linux-gnu/libtesseract.so.3
#2  0x0000555555554aa2 in main (argc=2, argv=0x7fffffffe3d8) at test.c:27

PS: This is a very low-priority issue for me. I can work around it really easily on Python side.

@jflesch : Problem is that you are not using c-api as designed ;-) : you forget to check return value. So instead of
TessBaseAPIInit3(handle, NULL, "fra");`

you use something like this:
if(TessBaseAPIInit3(handle, NULL, "eng") != 0)
		die("Error initializing tesseract\n");

Oops :-)
Thanks for the reply, and sorry for wasting your time. I'll make sure I didn't miss any other return values in Pyocr.

no problem. With testing you issue thanks for test case) I found and fix another problem ;-)
BTW: maybe it would be good if you join your effort with @jbarlow83 : he is wrapping leptonica in python in his project OCRmyPDF. leptonica provides several great function for ocr preprocesing like descew and dewarp removing background)

Interesting. I did write my own image manipulation library based on unpaper's algorithms (the idea at the time was mainly to reduce dependency on stuff like OpenCV as much as possible because they made installing my project a lot more complicated). But unskewing is the only algo I haven't re-implemented yet. I'll have a look later at what leptonic and OCRmyPDF provide exactly.

I spoke with some developer (not python) and he was quite surprised why tesseract users are using/looking for other libraries for OCR image preprocessing if all really needed is in leptonica ;-)
Maybe easy to use python wrapper could help to promote it.
@jbarlow83 make a good job in this area, but to have leptonica as separate python module could help gain more attention...

Actually, to be honest, in Paperwork, I don't really worry much about accuracy. OCR is used for indexing documents only. Fuzzy-searching takes care of non-entirely-accurate OCR results. So at the moment I don't even bother pre-processing images at all before passing them to Tesseract.
The image manipulations I do are to fix user perception of the document or try to simplify their content before exporting to make the files smaller (Unpaper's algorithms / SWT).
But I'm wondering: Is there an official recommendation for set of pre-processing algorithms to apply before passing an image to Tesseract ? Are some algorithms that would make documents match more closely what was used for training Tesseract for instance ? (grayscale ? pure b&w ? unskewing ? etc)

It depends on which ocr engine in 4.0.0 you want to use (lstm/legacy).

Some recommendations is on ImproveQuality - not sure how many of them are must for 4.0.
Deskew, dewarp, noise removing and crop should also applicable general also for your project.
Passing only text region (or graphics removing including lines), maybe trying other binarization method (tesseract use Otsu) could improve result too.

Thanks :)


npm run dev not working

I run the command npm run dev it's show me error
To investigate the issue, examine this stack trace:                                          │ │    at pollTunnelStatus [as _onTimeout] (@shopify/app/src/cli/services/dev/urls.ts:107)        │ │      if (result.status === 'error') return reject(new BugError(result.message,                │ │      result.tryMessage))                                                                      │ │    at listOnTimeout (node:internal/timers:569)                                                │ │    at processTimers (node:internal/timers:512)
After i search google, the issue is cloudflare, Then i install ngrok, I connect my project to tunnel ngraok, the issue is same.

Haven’t seen this one before unfortunately. Make sure you’re specifying the correct port on ngrok/cloudflare.
Are you on Mac or windows?

Yes, i am using the right command, i use this prevouse my remix application its work fine.
i am using windows


Rename "Optional" interface for lazy 1:1 relationships

Before the release of Java 8, NEBA introduced the concept of lazy-loading (#15). The expectation was that, once java 8 was well established, NEBA could transition to using the Java 8 Optional type. However, the final implementation of Optional in Java 8 is not extensible and has no means to support delegating to a factory, which lazy-loading essentially requires.
Now, NEBA's optional potentially conflicts with Java's Optional type and is sub optimal naming.
Thus, it should be renamed - perhaps to "Lazy" . The original Optional Interface should be deprecated, but kept until the next major release (5.x.).

Design decision:

The name of the new interface is "Lazy"
Lazy shall no longer mangle characteristics of java.util.Optional with lazy-loading semantics. Instead, first-class support for java.util.Optional field values shall be introduced (see #201). Then, one may, for instance, write:

@ResourceModel(types = "some/type")
public class MyModel {
    private Lazy<Optional<OtherModel>> otherModel;
}

And thus benefit from the well-supported java.util.Optional and use lazy loading at the same time.

Implemented, done!


Add notes and slides for May 20, 2020 oneMKL TAB meeting.

@gslavova Do you plan to review/merge these notes? Do we need to add more committers?

@rscohn2 I can do a quick review but likely will merge as submitted.  Can you add @sknepper as a committer?


refactor folders into 1a and 1b, adjust file reading script accordingly

@yuen-francis

Just an update: there is currently an issue with pull requests where if the folders are refactored, the changes are considered too large to view and I cannot make a pull request. Workaround in progress


Fix AutoMockable for closure with multiple parameters

Resolves https://github.com/krzysztofzablocki/Sourcery/issues/1356

Linux tests failed:

SourceryRuntime/MethodParameter_Linux.swift:25: Fatal error: unable to lookup: isOptional in MethodParameter: argumentLabel = Optional("x"), name = x, typeName = (any StubProtocol?) -> Void, inout = false, isVariadic = false, typeAttributes = [:], defaultValue = nil, annotations = [:], asSource = x: (any StubProtocol?) -> Void, index = 0

It looks like unrelated issue, probably introduced in https://github.com/krzysztofzablocki/Sourcery/pull/1358


Implement RecordBatch <--> FlightData encode/decode + tests

Which issue does this PR close?
Closes https://github.com/apache/arrow-rs/issues/3371
(note a large amount of this PR is test code)
Rationale for this change
The details of encoding / sending / receiving / reconstructiing RecordBatches over flight is common and somewhat duplicated across every implementation of flight. The mid level flight client aims to automate much of the common piece of this
What changes are included in this PR?

Add encode.rs logic in FlightDataEncoderBuilder (based on the code from  IOx in https://github.com/influxdata/influxdb_iox/pull/6460)
Move decode logic out of client and into decode.rs
Make the decoders more generic
Tests: round trip tests of encoding/decoding
Tests for FlightClient::do_get

Are there any user-facing changes?
yes, encoders and decoders
Next planned PRs:

[ ] Rewrite FlightSQL client in terms of the mid level client.

cc @avantgardnerio  @nevi-me  and @tustvold

@tustvold  if you have a chance, I would appreciate a review of this PR.
I am particularly interested in your opinion of how we should handle sending RecordBatch'es and retain the dictionary encoding.
One way might be to have the encode stream match / assign dictionary ids prior to sending. I realize this would be less efficient than if the creators of the dictionaries were to set the ids, but it would likely be more efficient than hydrating the dictionaries into actual data.

I am particularly interested in your opinion of how we should handle sending RecordBatch'es and retain the dictionary encoding.

I think we could just assign a unique ID to each dictionary encoded field, and then send a new DictionaryBatch for each field for each RecordBatch. We could then potentially retain the previous dictionary and use ArrayData::ptr_eq to elide sending the exact same dictionary again.

Mostly just some minor nits, I personally would avoid upstreaming the hydration of dictionaries so that we aren't committed to supporting it long term, I think it should be relatively straightforward to properly support dictionaries in a somewhat sane manner

Thank you -- I will investigate how much effort it will take to do so.  I am almost out of time today so I will likely not get a chance to work on this until later in the week

Mostly just some minor nits, I personally would avoid upstreaming the hydration of dictionaries so that we aren't committed to supporting it long term

I have pondered this for a while and my plan is to proceed with merging this PR (with the temporary dictionary hydration) and work on proper dictionary support as a follow on PR for the following reasons:

I want to make sure that the rust dictionary handling is correct and for that I would like to use the existing flight integration tests but they need work to update them to the new client too
I have one PR (and about to be more than one) queued up waiting on this PR

In order to minimize the API churn however, I plan to hold this PR until after we have released 30.0.0 (#3336 ) so we can minimize the chance of releasing the dictionary hydration code

The follow on PR is ready: https://github.com/apache/arrow-rs/pull/3402


avg_runner.py stops after few iterations

I'm trying to use your code but I get a strange error with the avg_runner.
it seems like it has some issue with the size of some images.
this is however very unlickly as I exported all the images from a video with ffmpeg.
any idea about why it breaks?
gino:Code Lorenzo$ python2.7 avg_runner.py -n Test -s 1000 --model_save_freq=1000 --test_freq=1000
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
Init discriminator...
Init generator...
Init variables...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
DiscriminatorModel: step 10 | global loss: 0.603209
Training generator...
GeneratorModel : Step  10
                 Global Loss    :  540.297
                 PSNR Error     :  5.45025
                 Sharpdiff Error:  4.7847
Training discriminator...
Training generator...
Training discriminator...
Training generator...
Training discriminator...
Traceback (most recent call last):
  File "avg_runner.py", line 185, in <module>
    main()
  File "avg_runner.py", line 181, in main
    runner.train()
  File "avg_runner.py", line 70, in train
    self.d_model.train_step(batch, self.g_model)
  File "/Users/Lorenzo/development/1-frame-in-the-future/Code/d_model.py", line 171, in train_step
    feed_dict = self.build_feed_dict(input_frames, gt_output_frames, generator)
  File "/Users/Lorenzo/development/1-frame-in-the-future/Code/d_model.py", line 132, in build_feed_dict
    resized_frame = resize(sknorm_img, [scale_net.height, scale_net.width, 3])
  File "/usr/local/lib/python2.7/site-packages/skimage/transform/_warps.py", line 119, in resize
    preserve_range=preserve_range)
  File "/usr/local/lib/python2.7/site-packages/skimage/transform/_geometric.py", line 1296, in warp
    image = _convert_warp_input(image, preserve_range)
  File "/usr/local/lib/python2.7/site-packages/skimage/transform/_geometric.py", line 1108, in _convert_warp_input
    image = img_as_float(image)
  File "/usr/local/lib/python2.7/site-packages/skimage/util/dtype.py", line 301, in img_as_float
    return convert(image, np.float64, force_copy)
  File "/usr/local/lib/python2.7/site-packages/skimage/util/dtype.py", line 205, in convert
    raise ValueError("Images of type float must be between -1 and 1.")

Hi Lorenzo,
Does this happen consistently on a specific frame? Have you tried looking at the frame values when it breaks?

It actually seems to happen on multiple frames.
probably there are some issues with the image normalization procedure.
I temporarily solved it by getting rid of the +0.5, -0.5 transposition in the d_model.py
# sknorm_img = (img / 2) + 0.5
# resized_frame = resize(sknorm_img, [scale_net.height, scale_net.width, 3])
# scaled_gt_output_frames[i] = (resized_frame - 0.5) * 2

sknorm_img = (img / 2)
resized_frame = resize(sknorm_img, [scale_net.height, scale_net.width, 3])
scaled_gt_output_frames[i] = (resized_frame) * 2

This seems to have solved the issue and I managed to train and test the network.
However I'm not sure if my dataset has been prepared correctly.
I trained the network with a video of me biking trough my neighbourhood;
from the video-clips I take 3 images per second and use them for the training.
this is the result after just 1000 steps.

It seems to me that the generated frames are far closer to the 4th input frame that to the target expected image.
Do you think the frames of the input sequence are too far apart? or I should just let the machine train for way longer?
really appreciate your feedback on this.


Can i use 1 Tag instead of 3?

Can i use 1 Tag instead of 3?

If you want to use 1 Tag you'll need to modify the code. You can start by looking at the do_predock() function, and change the line centre_tf = self.get_centre_of_side_markers() to something that uses only 1 marker.
Preempting the question of why 3 markers are used instead of 1, I am not the author of the code, but I have hypothesised some reasons why the 3 Tags are used here.

Yes the above comments are right.
When looking at the state machine diagram on the readme, the sides markers are used during predock, and steerdock states. You can try to edit the way both do_single_side_marker_rotate() and get_centre_of_side_markers() are currently implemented. PR is always welcomed!


Add section for CD conditions in troubleshooting

In the doc for troubleshooting, add a section that talks about looking at the cluster deployment conditions
xref: https://issues.redhat.com/browse/HIVE-2251
/assign @2uasimojo

/lgtm


Settings problem

In the settings tab, at the change password section, the app crash. Please resolve this issue

Thank you for your report!
We'll consider this issue


feat(gio-save-bar): add possibility to override default css

📚  View the storybook of this branch here

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


qa: Fix layout.

Most of the tests had a strange layout up until now. See qa > uiButton for example, with the button either stretched (windows, unix) or weirdly in the center (darwin).
This was due to the controls being placed in a stretchy vbox. These two patches introduce another non stretchy sub-vbox and remove some code duplication while at it.
Unix before:

Unix after:

Darwin before:

Darwin after:


Constructor override is out-of-sync with Ruby 3.3.0's Logger class

After https://github.com/ruby/logger/pull/85, Logger#initialize creates a @level_override = {} and access it every time level is called .
Since MonoLogger overrides the constructor but doesn't initialises the same @level_override variable, it now raises NoMethodError whenever level is called.
irb(main):001:0> require "mono_logger"
=> true
irb(main):002:0> MonoLogger.new(nil).level
/opt/rubies/3.3.0-dev/lib/ruby/3.3.0+0/logger.rb:385:in `level': undefined method `[]' for nil (NoMethodError)

    @level_override[Fiber.current] || @level
                   ^^^^^^^^^^^^^^^
        from (irb):2:in `<main>'
        from <internal:kernel>:187:in `loop'

Should be fixed by #12

Closing as #12 has been merged


Download attempts should print the URL and destination path

I'm seeing this error:
 Building package zlib[core]:x86-windows...
  A suitable version of git was not found (required v2.17.1). Downloading portable git v2.17.1...
  Downloading git...
  WinHttpSendRequest() failed: 12002

I'm not sure how to debug this WinInet error - at least googling this message suggests that WinInet is what's behind this error.  A simple (one-line) tweak to vcpkg can print out the URL and path we're trying to download.  I put the URL in my browser, download the file to the location vcpkg wants to store the tool at, and move on with my life.
Ideally, the problem wouldn't occur in the first place, but downloading stuff from a scripted environment in Windows isn't an easy problem to solve.  I've seen many different approaches to this problem, which tells me there isn't an easy answer.

Half-fixed by your PR #4640, though we should also improve downloading.
I'll leave this open for now, but we may close it in the future in favor of some unified "download issues" issue.

Looks fixed now.


Github Readme Stat error

I have a problem with my Wakapi instance and GitHub-readme stats.
URL:
https://github-stats.agentbot.xyz/api/wakatime?username=phamleduy04&api_domain=wakatime.agentbot.xyz&bg_color=2D3748&title_color=2F855A&icon_color=2F855A&text_color=ffffff&custom_title=Wakapi%20Week%20Stats&layout=compact)

Error:

I also enabled Public Data:

Hi @phamleduy04, you set your maximum data range to 10 days, while the default range requested by Readme Stats is all_time. Either set you time range setting to -1 or add &range=last_7_days to the image URL. That should help.

I just set my time range to -1 and the error still popping up

At least the error is a different one now ("user not found"). API domain and username seem to be correct, though. The only thing I could imagine as a potential issue is the fact that you seem to have CloudFlare protection in front of your Wakapi instance. Maybe that causes problems with Readme Stats. Could you try to temporarily disable it and try again?
Also, do you see the request from Readme Stats coming through to your Wakapi instance? There you be an incoming request in the logs.

Found the issue. After checking events on cloudflare I see github stats being set as "bot" and got blocked.
Turn off Bot Fight Mode will fix the issue since there is no way to bypass this using firewall rule.

Very interesting! Thanks for reporting back.


Update Store.cs

There's no evidence about a wrong deserialization of the stored informations, due to a potentially missing external libraries or a field which cannot be deserialized properly.
In case of an exception, the navigation stack  is cleared without any clue about it.
I created a Dialog, and I put an unserializable private field into it... I spent 2 hours in order to discover why the dialog were lost. I have to  download the botbuilder source code, and use it in order to undesrtand what's wrong.

@Magic73,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Add Boolean logic notation preference to My account page

Backend change here.
This adds an option on the "My account" page to change how Boolean logic is displayed - either using engineering, maths, or exam board specific notation.
The choice is stored as a map from each option to a Boolean value. In the case of multiple true values, the option occurring last (in the order they are listed above) is selected. I also added a userPreferences selector to neaten things a bit (removed in the latest commit for name clash and relative uselessness compared to other ways of accessing preferences).
The user needs to have the BOOLEAN_NOTATION beta feature enabled to see the "My account" option, and for their selection to affect which katex macros are used.
I'm wondering if line 162 in UserDetails.tsx (value = ["ENG", ...) gets run more times than needed, and if so whether it matters/could be changed?

Still need to add beta feature flag

Merged into september will get merged into master in time


LAB_05_PIM.md

English:
Global Reader
Current Translation:
全域讀者
Suggested Translation:
全域讀取者
Description:
Inconsistent translation. Global check

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run bugfix tool and waiting for result.


[Workflow] Make sure all capabilities are documented (purge, etc)

What content needs to be created or modified?
Need more documentation about purge capability for workflow in the features/concepts doc. May also need to add other capabilities that haven't been documented.
Describe the solution you'd like
Section/paragraph, bullet list, cross linking
Where should the new material be placed?
Workflow features and concepts
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
As mentioned in a comment here: #3969

Specifically I would add the following

Before this section https://docs.dapr.io/developing-applications/building-blocks/workflow/workflow-features-concepts/#limitations add a title called "Purging" with text something like this
"Workflow state can be purged from a state store for which purges all its history and removes all metadata related to a specific workflow instance. This is typically used for workflows that have run to a COMPLETED or TERMINATED state. Purging is also performed when a workflow moves to a CONTINUED_AS_NEW state. There are of course other times that you may choose to purge a workflow history and reuse a workflow instance with new inputs, for example with a FAILED workflow instance"

In the https://docs.dapr.io/developing-applications/building-blocks/workflow/workflow-architecture/#state-store-usage section add that "Workflow state can be purged from a state store for which purges all its history. Purging removes all metadata related to a specific workflow instance"

I would get @cgillum to review this text.


Dark mode

It is possible to have a dark mode to your extension? That would be awesome!

Twitter already has a dark mode:

Thanks! Love it!


Extension issue

Issue Type: Bug
Extension Name: csharp
Extension Version: 1.21.14
OS Version: Darwin x64 16.4.0
VSCode version: 1.43.0

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Closing this as there is no information in the report. If you have a more detailed problem, please ping back here and we can re-open it.


Allman style in ForEach-loop

Summary of the new feature / enhancement
As a C# developer using Allman style, I find that Powershell ForEach-loops always trips me up where curly braces needs to be on the same line.
For instance:
//... some other code

I think this is because this script block is the value for the Process parameter, and the parameters are specified on one line. For example, the same in Where-Object (parameter FilterScript)

The desire is understandable, but it is at odds with PowerShell's hybrid nature of being both a shell and a scripting language.
Commands such as ForEach-Object (as opposed to language statements such as foreach) are parsed  in argument mode, i.e, like a shell command line, which means:

The command and its arguments must be specified on the same line, ...
unless:

line continuation is explicitly requested via a ` at the (very) end of a line.
the line is syntactically incomplete from a language perspective.

ForEach-Object on its own is syntactically complete from a language perspective, so the script-block  must at least start on the same line in order to be recognized as an argument (as shown in your first statement).
By contrast, the script block itself, due to being parsed in expression mode, may span any number of lines, given that where it ends is unambiguously implied by the closing }. The same applies to language statements , such as if  in your example: there, { ... } isn't a script-block argument, but a part of the statement syntax itself.

In addition to what has already been said - that the foreach-object cmdlet,  needs the script blocks which form its parameters to begin on the same line  like all other parameters so
foreach-object  -process
{ script block }

has exactly the same problem as
Get-ChildItem -Path
$env:temp

This does not apply to language statements , while, switch, if , for , foreach
SO this is valid
foreach
 (
$x
 in
 @(
 1..
 3
 )
 )
 {
 $x *
 $x
 }

In the example in the first post
$Variable | foreach-Object
may not be the best way to write it.
foreach($v in $variable)
{
$v *$v
}

may be preferred IF the output is not piped into another cmdlet
and if piping is required the foreach method can be used - it unclosed ( tells PowerShell the line is not complete -  similar to the operators with the second operand on a new line
$variable.foreach(
{
$_ * $_
}
)  | sort-object -desc

Although it probably isn't what's required here you can also write
$variable |
    foreach-object {
   ....

Because the lack of anything between the | and the line break means the line automatically continues.

I see, thanks for all the answers.
I think we can close the issue. This...
foreach ($thing in $things)
{
// do stuff
}
... was appropriate for the task I was doing.
However, ForEach-Object seems to have some overhead due to pipline vs the other.
So in summary if I got it right, use ForEach-Object for large collections and foreach for smaller if memory is important since foreach has space complexity of O(n).

ForEach-Object seems to have some overhead due to pipline vs the other. So in summary if I got it right, use ForEach-Object for large collections and foreach for smaller if memory is important since foreach has space complexity of O(n).

There IS an overhead to pass objects down the pipeline but usually time running the process block is much greater so
$x = 1..1000000
$timer.Start()
$x | foreach-object {do-nothing}
$timer.Start()

Takes many more microseconds than
$x = 1..1000000
$timer.Start()
foreach ($item in $x) {do-nothing}
$timer.Start()

when do-nothing is replaced with something to real work, the extra microseconds are usually insignificant.
depending on that is being fetched
$x = get-Stuff
foreach($item in $x) ...

May need a lot more memory - because all the items go into X  - than
get-stuff  | foreach-object ...
and the latter may display the first output sooner.


fix NumpyConversion for strided numpy arrays

This PR is like https://github.com/apple/swift/pull/18936, but without the deduplication changes.

@swift-ci please test tensorflow Linux CPU


Handle associations with Gson

As used in https://github.com/konifar/droidkaigi2016

Handled by https://github.com/gfx/Android-Orma/pull/155