Add initial FSL & Freesurfer docker methods

Initial notes on SPM12, Freesurfer, and FSL containers.

Having tested some of the neurodocker commands, new versions of some software are not available and installation of some older versions fail. We may therefore want to build and publish our own ones from scratch.


Objects not drawing unless used directly, even when added to a batch

I'm trying to create some Rectangles and draw them as part of a batch.
However, this only actually works if if "use" the Rectangle in some way (assigning it to a variable, adding it to a list, etc.) for some reason, even if that list/variable/etc. is completely unused otherwise.
Minimal reproducible example:
import pyglet

def main()
	window = pyglet.window.Window()

	batch = pyglet.graphics.Batch()

	do_work = False # Set to True to make it draw properly, False to not.

	for i in range(5):
		if do_work:
			alive_cell = pyglet.shapes.Rectangle(x=50 + i * 30, y=50 + i * 30, width=10, height=10, color=(0, 255, 0), batch=batch)
		else:
			pyglet.shapes.Rectangle(x=50 + i * 30, y=50 + i * 30, width=10, height=10, color=(0, 255, 0), batch=batch)

	@window.event
	def on_draw():
		window.clear()
		batch.draw()

	pyglet.app.run()

if __name__ == '__main__':
	main()

Expected outcome: Regardless of how do_work is set, it should draw 5 green rectangles on the screen at different positions (in a diagonal line), when the batch is drawn.
Actual outcome: This only occurs when they're added to a list (or assigned to a variable, etc.), which can be done by setting do_work to True.

I thought that might be it, but I tried calling gc.disable() to disable the garbage collector, and confirmed that it was disabled. The behaviour still happened.

It's GC. If you want to stop GC'ing objects you can by overriding the __del__ behavior.
import pyglet

class NoGCRectangle(pyglet.shapes.Rectangle):
    def __del__(self):
        print("I Was GC'd")
        #if self._vertex_list is not None:
        #    self._vertex_list.delete()

def main():
    window = pyglet.window.Window()

    batch = pyglet.graphics.Batch()

    do_work = False  # Set to True to make it draw properly, False to not.

    for i in range(5):
        if do_work:
            alive_cell = pyglet.shapes.Rectangle(x=50 + i * 30, y=50 + i * 30, width=10, height=10, color=(0, 255, 0),
                                                 batch=batch)
        else:
            NoGCRectangle(x=50 + i * 30, y=50 + i * 30, width=10, height=10, color=(0, 255, 0), batch=batch)

    @window.event
    def on_draw():
        window.clear()
        batch.draw()

    pyglet.app.run()

if __name__ == '__main__':
    main()

Do this at your own risk otherwise you will have objects you will no longer be able to delete without creating a whole new batch.

I see, thanks!
To clarify, is Pyglet doing its own "manual GC" somewhere then?

In python you can hook into when the object is deleted by the user or gc with the __del__ function. In this case we use it to remove from the batch to prevent leaking objects.


Link in documentation intro

I was just going through the documentation:

The wasm-bindgen tool is sort of half polyfill for features like the host bindings proposal and half features for empowering high-level interactions between JS and wasm-compiled code (currently mostly from Rust).

The host-bindings proposal seems to be inactive, and it links to the component model repo.
Should the documentation be updated (at least to explain why the proposal is inactive)?

The original https://github.com/WebAssembly/host-bindings link in the doc is indeed redirected to https://github.com/WebAssembly/interface-types, which is the inactive one. There the "explainer" sheet says work moved to https://github.com/WebAssembly/component-model.
It is really unclear what the book intends to link to.

Updating the documentation to point to the component model proposal would be nice indeed.


Library on godelbrot import root

Remove libgodelbrot directory and put that stuff in the root.
Move the godelbrot binary to a new 'godelbrot' package.

Done with above commit


fetch upstream done and conflicts resolved

@SauravMukherjee44  review the PR #74 . Resolved all the conflicts.

Once again ! Just now i pushed a change.


Refactor JWT handling, because of security issue in biscuit

The issue has been resolved through cleaning up the project structure and fixing the imports of the dependencies:

[x] https://github.com/truzzt/ids-basecamp-clearinghouse/security/dependabot/1


[video_player_web] Improve handling of "Infinite" videos.

Videos that report Infinity duration are breaking the initialization event of the video player on the web.
This PR modifies the function that prepares the init event, to handle the Infinity duration value gracefully by using isFinite instead of !isNaN when deciding what to do with it.
(In Dart, a number can be only one of: isFinite, isNaN, isInfinite, so double.infinity.isNaN returns false!)

Fixes: https://github.com/flutter/flutter/issues/105649
Maybe: https://github.com/flutter/flutter/issues/100257
Maybe: https://github.com/flutter/flutter/issues/90038
Maybe: https://github.com/flutter/plugins/pull/3167

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Very good catch, I thought I remember a magic number for "infinity" length, but I ended up not using it.
-9223372036854775807 is kinda an unfortunate value in JavaScript, because it's much smaller than JS' Number.MIN_SAFE_INTEGER (-9007199254740991).
Anyway, I tested it in Safari 13 on an iPhone simulator (which is the latest modern browser without BigInt support), and it is still a ridiculously small number (even though there's some wrapping):
let magic = -9223372036854775807
->          -9223372036854776000

Close! And since we're not doing math with the value (only using it as a boundary), it's probably fine.

Son of a...
../lib/src/video_player.dart:39:18: Error: The integer literal<PHONE_NUMBER>854775807 can't be represented exactly in JavaScript.
Try changing the literal to something that can be represented in Javascript. In Javascript<PHONE_NUMBER>854775808 is the nearest value that can be
represented exactly.
  milliseconds: -9223372036854775807,
                 ^^^^^^^^^^^^^^^^^^^

It seems like the next best option is to decide on our own standard and have web be the first to adopt it, and then we can do a breaking change at some later point to move the other platforms over to it.
Your point about a threshold is a good one. Maybe we should just make -1 be our new target standard for representing this at the platform interface layer? I assume there are no legitimate cases of negative durations.

Maybe we should just make -1 be our new target standard for representing this at the platform interface layer? I assume there are no legitimate cases of negative durations.

I thought about -1 as well, but then decided to do a blame on the current value of TIME_UNSET to understand why exoplayer had settled in such a "wacky" value (other than maybe matching androidx.media2.common.SessionPlayer.UNKNOWN_TIME). From their commit message:
- Use a single constant for unset/unknown times across
  all time bases. Note also that this moves away from
  use of -1 for unset/unknown times in ms, which was a
  bad choice (it might conflict with real times, such
  as a time representing being just behind the start
  of a live window).

If we follow the same trick that ExoPlayer2 used to come up with their value, but that works fine in JavaScript:
JS_COMPATIBLE_TIME_UNSET = Number.MIN_SAFE_INTEGER + 1; // -9007199254740990
The number is several orders of magnitude larger than the current one, but if you convert it from milliseconds to years it is still over 285000 (sounds long enough for any pre-roll!)
I'll use that, and create an issue to standardize the value across the different implementations, and the flutter-provided UX of the Player (which seems to NOT render well when a video is "Live", or seek...)


Xiaomi Mi Mop 1C (dreame.vacuum.mc1808) remote function

Device model / 设备型号
dreame.vacuum.mc1808
Component version / 插件版本
0.7.5
HA core version / HA版本
2023.1.1
Integrated mode / 集成方式
Automatic (自动模式)
The problem / 问题详情
Hi. I have a Xiaomi Mi Mop (Mi 1C) Robot Vacuum.
I've added them successfully to Home Assistant using your integration. However I simply can't find any documentation on how xiaomi_miot.send_command and  xiaomi_miot.call_action works? How do I know which parameters that I can specify? Where do I find the list of supported methods?
In your example you are making a start-clean action, but is there a list of functions I can use with my device?
Apart from that what I'm actually trying to achieve is the possibility to remote control my vacuum, so that I can get it out of its dock without having to manually open the app and do so.
In the Xiaomi app I currently have this view:

But I'm not entirely sure if this is possible with your integration.
Hope you can guide me in the right direction.
Entity attributes / 实体属性
fan_speed_list: 安静, 标准, 中档, 强力
battery_level: 94
battery_icon: mdi:battery-90
fan_speed: 标准
model: dreame.vacuum.mc1808
lan_ip: <IP_ADDRESS>
mac_address: 7C:25:DA:5C:0B:FE
entity_class: MiotVacuumEntity
exclude_miot_services: consumable, annoy, remote, time
exclude_miot_properties: delete_timer, clean_info, map_view, frame_info
miot_type: urn:miot-spec-v2:device:vacuum:0000A006:dreame-mc1808:1
vacuum.fault: 0
vacuum.status: 1
audio.volume: 75
audio.voice_packets: ES
clean.work_mode: 2
clean.area:
clean.timer: 85-0-1662058057-12:00-0000010-1-1-0,21-1-1662354096-13:00-0111100-1-1-0
clean.mode: 1
clean.water_box: 1
clean.total_clean_time: 4426
clean.total_clean_times: 108
clean.total_clean_area: 2875
clean.clean_log_start_time:<PHONE_NUMBER>
clean.button_led: 0
clean.task_done: 0
brush_left_time-28-1: 73
brush_life_level-28-2: 36
filter.filter_life_level: 15
filter.filter_left_time: 23
brush_cleaner.brush_left_time: 173
brush_cleaner.brush_life_level: 57
battery.charging_state: 2
state_updater: lan
vacuum.status_desc: Sweeping
sub_entities: vacuum-3.status-2, filter-27.filter_life_level-1, filter-27.filter_left_time-2, battery-2.battery_level-1, battery-2.charging_state-2, brush_cleaner-26.brush_left_time-1, brush_cleaner-26.brush_life_level-2, brush_cleaner-28.brush_left_time-1, brush_cleaner-28.brush_life_level-2
friendly_name: Consuela Robot Cleaner
supported_features: 13048

Home Assistant Logs / 系统日志
No response

https://home.miot-spec.com/spec/dreame.vacuum.mc1808

service: xiaomi_miot.call_action
data:
  entity_id: vacuum.dreame_mc1808_entity_id
  siid: 21 # remote
  aiid: 1  # start-remote
  params:
    - '90' # piid: 1 - deg
    - '10' # piid: 2 - speed
  throw: true # throw result to HA notifications

I'm not sure what the specific params are, maybe it can be obtained through the MiHome APP modified by vevs.

Nice thanks for adding more details to it. I've tested it, and nothing really happens. How can I debug this? Unfortunately I don't have an Android phone to download the MiHome modified version

@al-one I just want to add that it worked!! I just had to provide the parameters as a string, just like stated in the documentation. Thank you very much for your help! 😃

Hey I just updated HA, and now I'm getting this error when using the remote

What does it mean?

@LukasMendez  Upgrade to v0.7.10.

Hi @LukasMendez! Did you find any frontend plugin to use with the Remote service? I mean something similar to the three arrows remote present in the Mi Home App.
Thanks!

Hi @LukasMendez! Did you find any frontend plugin to use with the Remote service? I mean something similar to the three arrows remote present in the Mi Home App.
Thanks!

Yes basically what @al-one shared worked for me. I just had to update.


UN-101: Migrate database to Turso and Libsql

Migration from Vercel/PostgreSQL to Turso/LibSQL
Overview
This PR implements a significant infrastructure change, moving our database from Vercel's PostgreSQL to Turso's LibSQL. This migration offers several advantages:

Increased storage capacity
Elimination of bandwidth restrictions
Potential for improved performance and scalability

Implementation Details

Followed the Turso documentation for integration with Drizzle ORM
Updated database configuration and connection logic
Modified queries and schema definitions to ensure SQLite compatibility

Configuration Notes

Current database configuration points to .env.local
For production deployment, this should be updated to use .env

Database Ownership

The Turso database is currently set up under a personal account
Action Item: Establish a team account or organization-level ownership for the database

Next Steps

Review and test all database interactions to ensure full compatibility
Update deployment scripts and environment variables for production
Set up a team account on Turso and migrate the database ownership
Update documentation to reflect the new database setup and any changed procedures

Additional Considerations

Verify backup and restore procedures for the new database
Assess any impacts on application performance and optimize if necessary
Consider implementing database migration scripts for smooth transitions in the future

Please review these changes carefully, particularly focusing on SQLite-specific adaptations and any performance implications.

Most of the grunt work has been done. schema.ts has been migrated to sqlite syntax. For some of the properties, there was not a direct migration such as serial for postgres, this is just an integer  and there is no timestamp / date column for drizzle + sqlite so this is just a text field.
New migrations were generated and pushed to turso with no issues but there is still work to do where there is now some type incompatibilities.

transaction files have been migrated to sqlite syntax also now, pr needs to be tested at this point, posts created / selected etc, same for comments. I am unable to login to the app at the moment so I cannot test this myself.

Bigint does not exist in sqlite but I noticed this was used for the consumed offsets, sqlite integers have a max value of 263 - 1 ==<PHONE_NUMBER>854775807 so I figure this should be fine.

Fix this: frontpage:build: Type error: Type 'string' is not assignable to type 'Date'.


TypeScript does not infer SetupContext properly

Version
3.0.0
Reproduction link
https://codesandbox.io/s/vue3-example-hooks-ts-forked-uzeyf?file=/src/components/HelloWorldThatWorks.vue
Steps to reproduce
When the instance option emits is used with arrow-function validators (or in its simple array form), TypeScript fails to infer SetupContext<Record<string, any>> and instead infers SetupContext<{"update:modelValue": () => true}> (see the attached codesandbox example).
What is expected?
That I can just typehint { emit }: SetupContext in my composable.
What is actually happening?
I need to update type definition whenever I add an emits (or use "full" functions which is ugly).
(what you're looking for in the codesandbox is a TypeScript error. If you don't see one, wait a couple of seconds, usually it becomes visible after 15-20sec on the website - the context variable is underlined in red.)

I don't know if this is the right place to report. If this is not a bug but deliberate, please let me know. Thanks

@signor-pedro moved to the correct repository

Looks like this might be related, will investigate later https://github.com/vuejs/vue-next/issues/2474

I think this behavior is correct because this update:modelValue indicates that it does not receive any parameters.
https://github.com/vuejs/rfcs/blob/master/active-rfcs/0030-emits-option.md#type-inference
  emits: {
    'update:modelValue': function works() { return true; },
  },

I think this should also infer SetupContext<{"update:modelValue": () => true}>.
Maybe #2362 is related?
  emits: {
    'update:modelValue'() { return true; },
  },

This may work for now. (though I think this also should infer SetupContext<{"update:modelValue": () => true}>)

@sapphi-red - if you are right and we should be inferring SetupContext<{"update:modelValue": () => true}>, how can I then type-hint the context properly?
All I wanted to do in my coposable is
function myFn({ emit }: SetupContext) {
   emit('sth', ...);
}

so that TypeScript (and in turn PHPStorm) infered the function signature for the emit function.
However, if I have to revisit the signature every time I add a new emits, that sounds like I am not getting something about TypeScript... sorry, quite a TS newbie here 🤷

I think you should change it to SetupContext<any> if you want to be able to emit any event.
function myFn({ emit }: SetupContext<any>) {
   emit('sth', ...);
}

If you want to limit it to 'sth' event, do it like below.
function myFn({ emit }: SetupContext<{ sth: (params: any) => any }>) {
   emit('sth', ...);
}

I think there is no way to infer SetupContext type from the usage of emit.

I think there is no way to infer SetupContext type from the usage of emit.

Yeah, I think there's not much to improve here as far as Vue's own types are concerned.
If you want to limit your function to certain event types, you will have to explicitly type them. The same is true for props, for example.
This is unrelated to #2474


Add support for building eBook with Docker

This change adds support for building the eBook with Docker. Supporting Docker means
that users who want to build the eBook do not have to install Ruby/Bundler/Calibre in their
environments.
I'm a Docker n00b, so if there's a better way to handle file permissions on the output, I'm all
ears.
Build tested on Linux. Outputs were fine.
There is also a change to the pandoc outputs. It seems the +smart extension is not supported for epub/html[0] in their latest versions.
Big thanks to @captn3m0 for creating this repo!
[0] - https://pandoc.org/MANUAL.html#extension-smart

@dentafrice Thanks for the PR. Other than a minor change in the ubuntu version, everything else looks good. :100:

Thanks! I'd have preferred 18.04, since it is LTS, but this should work as well.

I'm slightly editing the flow so as to be able to run this without having to build the image (I realized that the Docker image doesn't have any book sources, so it could be published). Something like:
docker run --rm --volume pwd:/output captn3m0/google-sre-ebook
with the bootstrap.sh script being included in the Dockerfile as the entrypoint (and slight edits to accommodate for the extra chown and mv commands).

Looks awesome!  That is a much better flow.


name and localName are empty

On Android, latest version of flutter_blue, I get advertisement and device data without any names.  Any ideas?

me too

+1

+1 I try to connect to a OBD2 device, and name is empty all time.  Any ideias ?


JavaScript error and crash when calling =../2

The following call to the =../2 predicate is expected to throw a domain_error(non_empty_list,[]) exception but we get a JavaScript error and crash instead:
?- Term =.. [].
/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:5388
				while( list.indicator === "./2" ) {
				            ^

TypeError: Cannot read property 'indicator' of undefined
    at =../2 (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:5388:17)
    at Thread.step (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2823:13)
    at Thread.again (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2921:13)
    at Thread.answer (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2881:8)
    at Thread.answers (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2896:8)
    at Session.answers (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2886:22)
    at Object.success (/opt/local/share/logtalk-3.40.0-b01/integration/logtalk_tau.js:42:17)
    at parseQuery (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:1219:15)
    at Thread.query (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2592:10)
    at Session.query (/Users/pmoura/Documents/Logtalk/tp/node_modules/tau-prolog/modules/core.js:2586:22)

Fixed by @pmoura in #181 (164947b).


This PR updates the Experimental Features doc

Moves Customizable Keyboard Shortcuts experiment (since it's on by default in Edge 86) and cleans up the dual-screen emulation experiment docs.

Docs Build status updates of commit 5cdf209:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-surface-duo-emulation.msft.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit cbac31f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-surface-duo-emulation.msft.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit d9a1505:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-surface-duo-emulation.msft.png
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2019/12/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/01/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/03/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/05/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/06/devtools.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b136f37:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/inspect-styles/animations.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-surface-duo-emulation.msft.png
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2019/12/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/01/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/03/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/05/devtools.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/whats-new/2020/06/devtools.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Fixed disabled cross-validation due to cache system

The new cache system has disabled any cross validation, and only the first fold is performed as a result.
Currently, the cache system caches the first fold's score, and applies that score to every remaining fold of the same individual.  This effectively disables cross-validation.
I've altered the logic so that the cache system caches the overall cross-validation score, and applies the overall score to the same individual in future.

Cool! looks great. Thanks for the pull request


The legend color was wrong when switching mulitple chart instances in IE11 and Edge

Version
4.2.1
Steps to reproduce

There are 6  charts, and we can switch  those charts arbitray in a selection
Each chart have the same legend and color config
Switch thos charts more than 8 times

What is expected?
Every time the legend color should be the same
What is actually happening?
The legend  color will be wrong after switch those charts many y times.

This bug can reproduce definitely in IE11  and Edge, but can't reproduce in Chrome(Version 74.0.3729.131)
I loged the option, found that the  option  was the same but the render of legend was different

Can you provide a minimum demo to reproduce this problem? https://jsfiddle.net/ovilia/yh76w9Lt/

OK, I will provide a demo in the later time.

You  can refer this link https://codesandbox.io/s/wonderful-mayer-16jlx

https://codesandbox.io/s/great-shannon-vk48p?fontsize=14
Sorry, the  previous link was wrong, you can refer to this new link.

We have this problem too and bumping to latest echarts version 4.4.0 does't make a difference. I wonder if you are polyfilling collection functions to ensure IE11 compatibility?

What happens when switching charts? What functions are you calling? The demo has error and pleas provide a simpler one.

@Ovilia I have simplified the demo, please head to this demo. here is a gif about this bug.

@Ovilia I have simplified the demo, please head to this demo. here is a gif about this bug.

so，what is the result?


Availability on Package Manager

Wondering if we could get these wonderful workflows in PM?

It is in our "to do" list but and it will be made available as soon as possible, most probably not earlier than a few more months.

I am very interested in this too: it will greatly improve my workflow.

So ... 1+ years later - is this going to be released via package manager at some point?

Work is underway to publish this code as a package https://github.com/Unity-Technologies/NavMeshComponents/issues/172#issuecomment-741714518

It is now an experimental package for Unity 2019.4 or newer. Please follow this other discussion https://github.com/Unity-Technologies/NavMeshComponents/issues/172#issuecomment-808105998


Fix list -disabled for blake2 alg

[x] CLA is signed

487d3a7 in master and 87f7c47 in 1.1.0; thanks!


Fix heading syntax

There must be a space between the # and heading text so that the GitHub Markdown parser correctly renders a heading.

LGTM


The "install" or "ini save" writes garbage options to app.ini

Description
Users should a have a clear and simple app.ini by default.
APP_NAME=Gitea: Git with a cup of tea
RUN_USER=xiaoguang
RUN_MODE=prod

[database]
DB_TYPE=mysql
HOST=<IP_ADDRESS>:3306
NAME=gitea
USER=gitea
PASSWD=gitea
SCHEMA=
SSL_MODE=disable
CHARSET=utf8
PATH=/Users/xiaoguang/work/gitea/data/gitea.db
LOG_SQL=false

[repository]
ROOT=/Users/xiaoguang/work/gitea/data/gitea-repositories

[server]
SSH_DOMAIN=localhost
DOMAIN=localhost
HTTP_PORT=3000
ROOT_URL=http://localhost:3000/
APP_DATA_PATH=/Users/xiaoguang/work/gitea/data
DISABLE_SSH=false
SSH_PORT=22
LFS_START_SERVER=true
LFS_JWT_SECRET=............
OFFLINE_MODE=false
HTTP_ADDR=<IP_ADDRESS>
PROTOCOL=
USE_PROXY_PROTOCOL=false
PROXY_PROTOCOL_TLS_BRIDGING=false
PROXY_PROTOCOL_HEADER_TIMEOUT=5s
PROXY_PROTOCOL_ACCEPT_UNKNOWN=false
ALLOW_GRACEFUL_RESTARTS=true
GRACEFUL_HAMMER_TIME=1m0s
STARTUP_TIMEOUT=0s
PER_WRITE_TIMEOUT=30s
PER_WRITE_PER_KB_TIMEOUT=10s
STATIC_URL_PREFIX=
LOCAL_ROOT_URL=http://localhost:3000/
LOCAL_USE_PROXY_PROTOCOL=false
REDIRECT_OTHER_PORT=false
PORT_TO_REDIRECT=80
REDIRECTOR_USE_PROXY_PROTOCOL=false
STATIC_ROOT_PATH=/Users/xiaoguang/work/gitea
STATIC_CACHE_TIME=6h0m0s
ENABLE_GZIP=
ENABLE_PPROF=false
PPROF_DATA_PATH=/Users/xiaoguang/work/gitea/data/tmp/pprof
LANDING_PAGE=home
SSH_SERVER_CIPHERS=
SSH_SERVER_KEY_EXCHANGES=
SSH_SERVER_MACS=
SSH_KEYGEN_PATH=
SSH_LISTEN_PORT=22
SSH_SERVER_USE_PROXY_PROTOCOL=false
SSH_TRUSTED_USER_CA_KEYS_FILENAME=/Users/xiaoguang/.ssh/gitea-trusted-user-ca-keys.pem
SSH_AUTHORIZED_PRINCIPALS_ALLOW=off
MINIMUM_KEY_SIZE_CHECK=true
SSH_AUTHORIZED_KEYS_BACKUP=true
SSH_CREATE_AUTHORIZED_KEYS_FILE=true
SSH_EXPOSE_ANONYMOUS=false
SSH_AUTHORIZED_KEYS_COMMAND_TEMPLATE={{.AppPath}} --config={{.CustomConf}} serv key-{{.Key.ID}}
SSH_PER_WRITE_TIMEOUT=30s
SSH_PER_WRITE_PER_KB_TIMEOUT=10s
BUILTIN_SSH_SERVER_USER=xiaoguang
SSH_USER=xiaoguang

[lfs]
PATH=/Users/xiaoguang/work/gitea/data/lfs

[mailer]
ENABLED=false

[service]
REGISTER_EMAIL_CONFIRM=false
ENABLE_NOTIFY_MAIL=false
DISABLE_REGISTRATION=false
ALLOW_ONLY_EXTERNAL_REGISTRATION=false
ENABLE_CAPTCHA=false
REQUIRE_SIGNIN_VIEW=false
DEFAULT_KEEP_EMAIL_PRIVATE=false
DEFAULT_ALLOW_CREATE_ORGANIZATION=true
DEFAULT_ENABLE_TIMETRACKING=true
NO_REPLY_ADDRESS=noreply.localhost

[openid]
ENABLE_OPENID_SIGNIN=true
ENABLE_OPENID_SIGNUP=true

[cron.update_checker]
ENABLED=false

[session]
PROVIDER=file

[log]
MODE=console
LEVEL=info
ROOT_PATH=/Users/xiaoguang/work/gitea/log
STACKTRACE_LEVEL=none
BUFFER_LEN=10000
ENABLE_SSH_LOG=false
ACCESS_LOG_TEMPLATE={{.Ctx.RemoteHost}} - {{.Identity}} {{.Start.Format "[02/Jan/2006:15:04:05 -0700]" }} "{{.Ctx.Req.Method}} {{.Ctx.Req.URL.RequestURI}} {{.Ctx.Req.Proto}}" {{.ResponseWriter.Status}} {{.ResponseWriter.Size}} "{{.Ctx.Req.Referer}}" "{{.Ctx.Req.UserAgent}}"
REQUEST_ID_HEADERS=

[repository.pull-request]
DEFAULT_MERGE_STYLE=merge

[repository.signing]
DEFAULT_TRUST_MODEL=committer

[security]
INSTALL_LOCK=true
INTERNAL_TOKEN=.........
PASSWORD_HASH_ALGO=pbkdf2

[ssh.minimum_key_sizes]

[oauth2]
JWT_SECRET=........

Is this issue addressed? I've starting testing gitea v1.20.0-rc2 (which includes 8302b95d6bfdbc40083a72e5d1b727fcb1e00940) locally and have gotten these errors on startup:
2023/06/26 18:44:53 ...es/setting/oauth2.go:140:loadOAuth2From() [F] save oauth2.JWT_SECRET failed: failed to save "/custom/conf/app.ini": open /custom/conf/app.ini: permission denied
In our config we set oauth2.ENABLE = false. I think there are two problems here. The first is that gitea shouldn't attempt to modify the oauth2 configuration if not enabled. Second, ideally gitea would never write config back out to disk. As a user using configuration management and container images I don't want our config to by modified outside of these processes.

I do not think "/custom/conf/app.ini" is a right path for config file.
How do you run Gitea? Can you provide reproducible steps?

I do not think "/custom/conf/app.ini" is a right path for config file.

 In our case we build our docker image with this Dockerfile: https://opendev.org/opendev/system-config/src/branch/master/docker/gitea/Dockerfile (modified to update the git tag checkout to v1.20.0-rc2 in this case) and deploy them with this docker-compose file: https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/gitea/templates/docker-compose.yaml.j2

How do you run Gitea? Can you provide reproducible steps?

I believe that there are two things necessary to reproduce this problem. The first is the app.ini file must have permissions set such that the gitea user cannot write to the file. Second gitea must see some file difference it wants to write back to that file.
To achieve the first item you can chown root:root app.ini and chmod 0600 app.ini then run gitea as a user other than root. The file will be owned by root with permissions only allowing root to modify the file. Then to trigger the write create an oauth2 section with this content:
[oauth2]
ENABLE = false

This seems to force gitea into thinking it needs to add JWT_SECRET to that config and write it back to the file. As mentioned above I think there are two bugs here. The first is that if oauth2 is disabled gitea shouldn't do anything with secrets for oauth2. The second is that gitea should not write back to its config file. If gitea generates new config like this unexpectedly then configuration can be broken and/or mismatch what configuration management believes is in place.
It is possible that this is separate issue and I'm happy to file one for that. I just saw this issue and it looks very similar (but this is happening outside of the gitea runtime and the issue I have is during runtime?)

The first is that if oauth2 is disabled gitea shouldn't do anything with secrets for oauth2.

Yes, it is a bug, it doesn't make sense to generate the secret if the oauth2 is disabled.

The second is that gitea should not write back to its config file.

Gitea requires that the "app.ini" should be writable by itself, it's not well documented but many functions depends this fact.


Remove IPv6 related stuff (fix #268)

Tomo I'm good if this is merged, if you don't mind, resolve the conflicts and go ahead and merge. Thanks!

@dougbtv sure. after resolve-conflicts, I will merge it.


Custom ranges for views

Hey,
Would it be possible to pass custom ranges to agenda and some kind of week view, to not display a range of days that follow each another, but to have total control about which days exactly, possibly with gaps?

Hi! I'm not sure I understand what you're trying to achieve, so this might be totally off, but if you want to hide certain days in the agenda view, it'd probably be easiest not to pass events from those days in the first place.
In the week view however, I'm not really sure what you mean by "not displaying" some days? Do you want the whole day (i.e. the column) to be removed entirely from the calendar? Or should they just be styled differently?

Yes, something like the week view but instead of displaying say

['2015-01-01', '2015-01-02', '2015-01-03', '2015-01-04', '2015-01-05', '2015-01-06', '2015-01-07']
I'd like to show
['2015-01-01', '2015-01-05', '2015-01-14', '2015-01-20', '2015-02-01', '2015-02-02', '2015-02-03']

So a view that just gets an array of dates that it shall show.

you can take a look at how the Week and Day components are built on TimeGrid and provide a custom View component to the calendar. I believe the api is general enough to allow your situation I've never tried it with non contiguous ranges but I don't think that should be a problem. You'll need to provide custom navigate static methods to your view.

I looked into it, but the TimeGrid would only accept a start/end date in its current state. Why not calculate the array of displayed dates  from within the view instead of inside the TimeGrid. So instead of providing start/end to let TimeGrid calculate the range, you'd always provide an array of dates.

I'd be fine with that change. feel tree to send a PR over :)

Closing to keep the tracker neat. Please feel free to PR anything back if you implement this :)


after_validation :report_validation_errors_to_rollbar not working in 2.11.0 and up

Is there a new way to use it?

@rassom can you give us details about the problem? In which class are you trying to use it? Which version of active record, rails, etc.. are you using?
Thanks

@jondeandres Newest versions and custom classes, i.e.
class Risk < ActiveRecord::Base
  has_paper_trail

  belongs_to :user

  validates :user_id, :presence => true
  validates :single_trade_max_percent_to_risk_between_entry_and_stop_price, :presence => true, :format => {:with => PRICE_REGEX, :message => PRICE_ERROR_MESSAGE }

  after_validation :report_validation_errors_to_rollbar

end

Thanks :)

@rassom can you try something? can you execute require 'active_model/validations' in your rollbar initializer just before Rollbar.configure?
Thanks

@jondeandres Just tried and still get undefined method 'report_validation_errors_to_rollbar' for #<Risk:0x007fd2d190fb90>

mmm, I didn't know you were having an error, just that it wasn't working. Ok, let me reproduce your problem and I'll reach you here.
Thanks.

Ah, sorry - my bad for not being explicit!
And thanks - looking forward to the solution :-)

I think this may be happening because we add the module ti ActiveModel::Validations after your models are already loaded.

Ok, so what would the solution be? Something on my side or something you will fix in Rollbar?

It's something we would need to fix. I'll try to find the reason for this and explain you here ok?
Thanks!

Of course! Thanks! :-))

same here... I'm using Rails 5

Hey @danieldocki, give me one day and I will be able to check this.
Sorry for this, we'll fix this soon.

Any news on this?

Hey @rassom, I've been out this week and couldn't take a look at this. I hope I can do it during today or tomorrow ok? We'll contacto you here.
Thanks again.

Hey there, I've been trying to reproduce this. I'm using Ruby 2.2.2 and Rails 5.0.0.beta3.
My model:
class User < ApplicationRecord
  validates :username, presence: true

  after_validation :report_validation_errors_to_rollbar
end

And in a rails console:
irb(main):005:0> u = User.new
=> #<User id: nil, username: nil, created_at: nil, updated_at: nil>
irb(main):006:0> u.valid?
[Rollbar] Reporting form validation error: Username can't be blank for #<User:0x00559521086720>
[Rollbar] Scheduling item
[Rollbar] Sending item
[Rollbar] Success
[Rollbar] Details: https://rollbar.com/instance/uuid?uuid=c9864062-e12b-4f70-9f0e-28ca97965897 (only available if report was successful)
=> false

I'm using the last rollbar version, 2.11.3. Have you tried with it?

@jondeandres
Rails 5.0.0rc1 and Ruby 2.3.1
irb(main):001:0> u = User.new
=> #<User id: nil, username: nil, created_at: nil, updated_at: nil>
irb(main):002:0> u.save
   (0.3ms)  BEGIN
  User Exists (28.3ms)  SELECT  1 AS one FROM "users" WHERE "users"."nickname" IS NULL LIMIT $1  [["LIMIT", 1]]
   (0.7ms)  ROLLBACK
NoMethodError: undefined method `report_validation_errors_to_rollbar' for #<User:0x007f99f88393b8>
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activemodel-5.0.0.rc1/lib/active_model/attribute_methods.rb:433:in `method_missing'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:382:in `block in make_lambda'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:218:in `block in halting'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:456:in `block in call'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:456:in `each'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:456:in `call'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:101:in `__run_callbacks__'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activesupport-5.0.0.rc1/lib/active_support/callbacks.rb:750:in `_run_validation_callbacks'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activemodel-5.0.0.rc1/lib/active_model/validations/callbacks.rb:113:in `run_validations!'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activemodel-5.0.0.rc1/lib/active_model/validations.rb:338:in `valid?'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/validations.rb:65:in `valid?'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/validations.rb:82:in `perform_validations'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/validations.rb:44:in `save'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/attribute_methods/dirty.rb:22:in `save'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:319:in `block (2 levels) in save'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:395:in `block in with_transaction_returning_status'
... 1 levels...
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/connection_adapters/abstract/transaction.rb:189:in `within_new_transaction'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/connection_adapters/abstract/database_statements.rb:232:in `transaction'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:211:in `transaction'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:392:in `with_transaction_returning_status'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:319:in `block in save'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:334:in `rollback_active_record_state!'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/transactions.rb:318:in `save'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/activerecord-5.0.0.rc1/lib/active_record/suppressor.rb:41:in `save'
	from (irb):2
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-5.0.0.rc1/lib/rails/commands/console.rb:65:in `start'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-5.0.0.rc1/lib/rails/commands/console_helper.rb:9:in `start'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-5.0.0.rc1/lib/rails/commands/commands_tasks.rb:78:in `console'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-5.0.0.rc1/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
	from /Users/danieldocki/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-5.0.0.rc1/lib/rails/commands.rb:18:in `<top (required)>'
	from bin/rails:4:in `require'
	from bin/rails:4:in `<main>'

Hi @danieldocki, thanks for these details.
I could reproduce the bug. This was happening if, for example User constantis used before Rollbar monkey patches are loaded, soUser` class doesn't have the modules hierarchy it should.
This bug could exist or not depending on how the models are being used during the boot up.
Could you try this branch and confirm us this is resolved for you?
https://github.com/rollbar/rollbar-gem/tree/immediate-validations-monkey-patch.
Thanks!

@jondeandres The fix didn't solve the errors for me unfortunately (tried Rollbar 2.11.5). Do you want repo access so you can try to reproduce with my code?

Hey @rassom, before that. Can you try something on lolcalhost? Put a debugger in https://github.com/rollbar/rollbar-gem/blob/master/lib/rollbar/plugins/validations.rb#L30 and another one before your class Risk definition. Bootup the rails server and tell us the order of those debugger breaks.
If this doesn't work, having access to the repo will help, but it'd be better not need that.
Thanks!

I'm also running into this issue on a Rails 4.2.4 / Ruby 2.2.3 project.
I ran a bisect, and found that the first bad commit is:
847e47be501332fb36391249749f67135f30505f
The one that starts trying to inject the Rollbar::ActiveRecordExtension into ActiveModel::Validations.
Solution
My solution was to include the Rollbar::ActiveRecordExtension wherever I need it.
class User
  include Rollbar::ActiveRecordExtension

  after_validation :report_validation_errors_to_rollbar
end

Proposal
I checked out how we are monkey patching the ActiveModel::Validations module, and the problem is here:
ActiveModel::Validations.module_eval do
  include Rollbar::ActiveRecordExtension
end

Since ActiveModel::Validations is module, we cannot include Rollbar::ActiveRecordExtension into it, instead we would need to include it, once ActiveModel::Validations is included.
I'm not sure how to do this, I don't think it is possible to extend what is already being included by  ActiveModel::Validations.
IMO, it would be cleaner if we just renamed and provided Rollbar::ActiveRecordExtension as a mixin, and updated the docs.
Does anyone have a better idea?

Hi @herenow, thanks for writing in this issue, analyze it and suggest a solution.
I tried to reproduce this problem, and I succeded on some scenarios when the project models were loaded before rollbar gem. I think including the module in ActiveModel::Validations should work, but it seems we are having issues in some scenarios, like yours.
Have you tried debugging if your models are being loaded before the include we do in ActiveModel::Validations?
Thanks

@jondeandres I don't think the problem is the loading order.
Even if I add this:
module Rollbar
  # Module that defines methods to be used by instances using
  # ActiveModel::Validations
  # The name is ActiveRecordExtension in order to not break backwards
  # compatibility, although probably it should be named
  # Rollbar::ValidationsExtension or similar
  module ActiveRecordExtension
    def report_validation_errors_to_rollbar
      errors.full_messages.each do |error|
        Rollbar.log_info "[Rollbar] Reporting form validation error: #{error} for #{self}"
        Rollbar.warning("Form Validation Error: #{error} for #{self}")
      end
    end
  end
end

ActiveModel::Validations.module_eval do
  include Rollbar::ActiveRecordExtension
end

Just before my model class, it still won't include the report_validation_errors_to_rollbar method to my class.
We could do something like:
ActiveModel::Validations.module_eval do
  def report_validation_errors_to_rollbar
    errors.full_messages.each do |error|
      Rollbar.log_info "[Rollbar] Reporting form validation error: #{error} for #{self}"
      Rollbar.warning("Form Validation Error: #{error} for #{self}")
    end
  end
end

It does work, but it is way to dirty IMO, we should not add methods directly to ActiveModel::Validations :)
Ideally, we would have a way to define that Rollbar::ActiveRecordExtensions should be included once ActiveModel::Validations is included.
https://github.com/rails/rails/blob/master/activemodel/lib/active_model/validations.rb#L41
As I said before, I don't know how we could accomplish this monkey patch (and I'm not even sure we should do this).

@herenow thanks for sharing your thoughts. We'll figure out what exactly to do here.
Nobody likes monkey patching, but we are trying to build a gem that you just add it in your Gemfile and everything works without too many intervention of the user.

Any news on this?

@rassom no yet, sorry. we've been working on some other issues,  I'm adding this to our priority queue. Sorry for the delay.

Hi there,
we have probably two solutions for this, but we'd like be sure they are working for you. We'd like to still redefine a method in ActiveModel::Validations since our docs say you just need to use after_validation.
Could you try this branch and check it works as expected? https://github.com/rollbar/rollbar-gem/tree/fix-active_model-validations.
The other solution is defining the method in ActiveModel::Validations as @herenow commented.
Thanks!

Thanks.
Just tried the fix-active_model-validations branch and still get this error NoMethodError at /users undefined method report_validation_errors_to_rollbar' for #Risk:0x007feef6d4bee8`, so it didn't help unfortunately.
PS. Can see what you are referring to as the other solution also seems to be mentioned here https://github.com/rollbar/rollbar-gem/issues/248

@rassom we "added" the method to ActiveModel::Validations so all the objects using ActiveModel will have it. #248 should be really closed, except for this issue.

@rassom in case you are using Rails, can you run this in a Rails console and share the output?
Rollbar.plugins.collection.detect { |plugin| plugin.name == 'active_model' }
Thanks

Sure thing :)
Here's the output:
Rollbar 2.10.0:
NoMethodError: undefined method `plugins' for Rollbar:Module
	from (irb):1
	from /Users/rasmus/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-4.2.6/lib/rails/commands/console.rb:110:in `start'

Rollbar  2.15.0 (standard, not the branch you specified earlier in this thread):
=> #<Rollbar::Plugin:0x007f88dfd22520 @name="active_model", @dependencies=[#<Proc:0x007f88dfd22340@/Users/rasmus/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/rollbar-2.12.0/lib/rollbar/plugins/validations.rb:2>, #<Proc:0x007f88dfd222c8@/Users/rasmus/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/rollbar-2.12.0/lib/rollbar/plugins/validations.rb:3>, #<Proc:0x007f88dfd22250@/Users/rasmus/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/rollbar-2.12.0/lib/rollbar/plugins/validations.rb:4>], @callables=[], @loaded=true>
````

@rassom we need to find what is happening in your side. This is working for me with all the different branches I've tested.
Can you open a Rails console a execute ActiveModel::Validations.instance_methods.grep /rollbar/
This is my output:
Loading development environment (Rails 5.0.0.rc1)
irb(main):001:0> ActiveModel::Validations.instance_methods.grep /rollbar/
=> [:report_validation_errors_to_rollbar]

With 2.11.5, I get this:
SyntaxError: (irb):1: syntax error, unexpected =>
=> [:report_validation_errors_to_rollbar]
  ^
	from /Users/rasmus/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-4.2.6/lib/rails/commands/console.rb:110:in `start'

With 2.12.0, I get what you get:
=> [:report_validation_errors_to_rollbar]
BUT even with 2.12.0, I still get errors the NoMethodError at /users undefined method report_validation_errors_to_rollbar' for #Risk:0x007fb64c83a220`error when trying to sign up a new user.
Let me know if you want access to my github repo for the app.

@jondeandres I've added you to the repo. Let me know if you encounter problems - documentation and code is not top class unfortunately. You'll have to remove the version restriction of the Rollbar gem in the Gemfile, update to latest Rollbar and then try to sign up as a new user from the front page to see the error.

Nice, thanks @rassom, I'll try to reproduce this in next days. I'll reach you here

Any updates on this? I am also having the same issues. Using rails 4.1.5 & Ruby 2.1.2 with rollbar 2.11.5.
I got it to work adding include Rollbar::ActiveRecordExtension for now.
Suggestions?

@ESouza have you follow the suggestions I asked here? Can you add a debugger in https://github.com/rollbar/rollbar-gem/blob/master/lib/rollbar/plugins/validations.rb#L30 and before one of your models and tell us which one stop the first?

Hey @rassom, I've detected the problem exists since ActiveRecord::Base is loaded before Rollbar validations plugin. There are several causes for this in your code, delayed_job_activerecord, devise and few more.
I've commented/disabled some features in your code related to ActiveRecord::Base in config/initializers/ and now it's working. I'll need to figure out what we can do here.
But thanks for sharing with us your code, it's being very useful!

Glad you found the problem.
Please let me know when it's solved and please remember to delete my code once you're done with it.
Thanks :)

@rassom can you try something? Move 'rollbar' gem in Gemfile to the top, before any other gem statement.
Or in config/application.rb do require 'rollbar' before Bundler.require(*Rails.groups). The goal is that our module is included in ActiveModel::Validations before ActiveRecord::Base is used by any class inheriting from it.

@rassom I think I have a solution that can work for anyone. Can you use this in your Gemfile?
gem 'rollbar', git: 'https://github.com/rollbar/rollbar-gem/', branch: 'include-validations-plugin-in-activerecord-base'
Confirm me it's working fine and I can do a new release of the gem with it.

Can you try it with my code?
I'm offline until next week
Den 3. august 2016 kl. 16.30.23, Jon<EMAIL_ADDRESS>skrev:

@rassom https://github.com/rassom can you try something? Move 'rollbar'
gem in Gemfile to the top, before any other gem statement.
Or in config/application.rb do require 'rollbar' before
Bundler.require(*Rails.groups). The goal is that our module is included
in ActiveModel::Validations before ActiveRecord::Base is used by any
class inheriting from it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rollbar/rollbar-gem/issues/464#issuecomment-237244779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkRyjTU278RvzUHcyIHJ3WnsEapwKkPks5qcKX_gaJpZM4IZrnB
.

--
Sent from my mobile device

@rassom I tried and it worked, but I'd like you to confirm it. Maybe your use case is different or something weird happens for you.

Hey @rassom, did you have a moment to try our branch? Thanks!

I don't have time this week. Sorry.
Den mandag den 8. august 2016 skrev Jon<EMAIL_ADDRESS>
Hey @rassom https://github.com/rassom, did you have a moment to try our
branch? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rollbar/rollbar-gem/issues/464#issuecomment-238285361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkRygD0xjCzD4--mVOfck0bqC25Yacbks5qd1PcgaJpZM4IZrnB
.

@jondeandres your branch works for me 👍 thanks

awesome @longkt90, we'll try to merge and release in next days.

Hi @jondeandres  Please let me know when you have merged and I will test on my end too :)

@rassom It's merged to master 1 week ago, not sure when will it be released. I think it's an important issue. https://github.com/rollbar/rollbar-gem/pull/503
Btw deleted your branch broke my builds @jondeandres

Hi,
we'll try to release a new version today. We were waiting to close few other issues so we had a release with more bugfixes. @longkt90 sorry to hear your builds broke :disappointed:, you can use master branch if you prefer now.

@jondeandres Just tested and master (2.13.0) works for me 👍

awesome @rassom, thank you very much for your help!

@jondeandres You too! :)


There is no Recipients > Shared.

[Enter feedback here]
"In the Exchange admin center, go to Recipients > Shared" There is no Shared link under Recipients.

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

ID: db2dbb29-38c4-afe4-bc0d-3bbc195f6288
Version Independent ID: bcfee113-ecfe-e358-974e-6f9cd76b8d35
Content: Create a shared mailbox - Microsoft 365 admin
Content Source: microsoft-365/admin/email/create-a-shared-mailbox.md
Service: o365-administration
GitHub Login: @SKjerland
Microsoft Alias: sharik

@1024-Kibibytes Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/8329
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @1024-Kibibytes Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Added Typescript support in README docs

See #270

import BigNumber from "./bignumber.mjs";

just demonstrates that the library can be imported by specifying the path to the single file, "bignumber.mjs", which works in browsers and node.js.

The existing line in the README:
import BigNumber from "./bignumber.mjs";

was just to just demonstrate that the library can be imported in browsers and node.js by specifying the path to the single file, bignumber.mjs.  It was not intended to demonstrate how to load the library after using npm install bignumber.js as shown in the separate section above it.
I have updated the README to make it explicit that npm users can import the library using
import BigNumber from "bignumber.js";

The existing line in the README:
import BigNumber from "./bignumber.mjs";

was just to just demonstrate that the library can be imported in browsers and node.js by specifying the path to the single file, bignumber.mjs.  It was not intended to demonstrate how to load the library after the npm install bignumber.js as shown in the separate section above it.
I have updated the README to make it explicit that npm users can import the library using
import BigNumber from "bignumber.js";

I see no need for a typescript section, as there is no difference.


Senec changed from HTTP to HTTPS

Describe the bug
Senec Webinterface and API is no longer available over HTTP. They switched over to HTTPS.
Screenshots & Logfiles
Screenshot is only in german:

fully covered since 1.4.3 - please update your adapter if you still experience this issue.

#164


Support running on IBM JDK 8

Description:
Created as a followup item from #9308

Are we planning to support running ballerina in IBM JDK since we are now shipping the JDK with the installation itself?

Are we planning to support running ballerina in IBM JDK since we are now shipping the JDK with the installation itself?

@mohanvive I think we don't need to support as we already pack inbuilt JRE along with the installation.
@Kishanthan
We have also included to fetch JRE dependencies along with ballerina update tool as well. Hence i am closing this issue as as all the installations will use inbuilt JRE. Please feel free to reopen if there are any concerns


Unhandled 'error' event when password authentication fails

When I try to connect to my postgresql instance from the command line, I get the following error message, even though I can connect directly using psql. I've tried using the -c option when running postgraphql, but that doesn't seem to work either. Any help would be appreciated!
PostGraphQL server listening on port 5000 🚀
‣ Connected to Postgres instance postgres://localhost:5432
‣ Introspected Postgres schema(s) public
‣ GraphQL endpoint served at http://localhost:5000/graphql
‣ GraphiQL endpoint served at http://localhost:5000/graphiql

events.js:160
throw er; // Unhandled 'error' event
^
error: password authentication failed for user "root"
at Connection.parseE (/usr/local/lib/node_modules/postgraphql/node_modules/pg/lib/connection.js:554:11)
at Connection.parseMessage (/usr/local/lib/node_modules/postgraphql/node_modules/pg/lib/connection.js:381:17)
at Socket. (/usr/local/lib/node_modules/postgraphql/node_modules/pg/lib/connection.js:117:22)
at emitOne (events.js:96:13)
at Socket.emit (events.js:188:7)
at readableAddChunk (_stream_readable.js:176:18)
at Socket.Readable.push (_stream_readable.js:134:10)
at TCP.onread (net.js:548:20)

What are your environment variables that start with PG*?

There aren't any PG* environment variables set. Are there any that should be?

It's possible that psql locally is using the peer authentication mode of postgres, whereas postgraphql is trying to use a network connection. See: https://www.postgresql.org/docs/9.5/static/auth-methods.html
Try using the same address for psql; for example:
psql "postgres://localhost:5432"
postgraphql -c  "postgres://localhost:5432"

You might also try explicit authentication (replace pg_user, pg_password and db_name with valid details):
psql "postgres://pg_user:pg_password@localhost:5432/db_name"
postgraphql -c  "postgres://pg_user:pg_password@localhost:5432/db_name"

Thanks @benjie, I tried again with the explicit authentication, and that seemed to work.


NEXT-00000 - add active from to tax rule

1. Why is this change necessary?
We currently have no way to manage the tax activity date. We cannot schedule a specific day for tax implementation. Sometimes the tax rate changes. In addition, such a date will allow us to "recalculate" the tax backwards (if a tax change was introduced and we did not know about it). This change is useful for large stores selling in multiple countries, so they need more tax management.
2. What does this change do, exactly?
My change adds a new date time field to the tax rule. We can set the date of its activation. It is stored in UTC format. The admin can set this date in the panel when creating a tax rule. In SalesChannelContext, the latest tax activity date is always selected from all the rules that meet the criteria.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at 256b18a
This pull request adds a new feature to the tax system that allows defining tax rules that are only valid from a certain date onwards. It introduces a new activeFrom property to the TaxRuleEntity and TaxRuleDefinition classes, and a new active_from column to the tax_rule table. It also modifies the tax rule matching logic in the SalesChannelContextFactory and the tax rule type filters to check the activation date of the tax rules. Additionally, it updates the administration UI and the unit tests to support the new feature.

🤖 Generated by Copilot at 256b18a

Add the activeFrom property to the TaxRuleEntity class and the tax_rule table to store the activation date of each tax rule (link,link,link,link,link,link)
Add the isTaxActive method to the AbstractTaxRuleTypeFilter class to check if a tax rule is active based on the current date and the activeFrom property (link,link)
Modify the match methods of the EntireCountryRuleTypeFilter, IndividualStatesRuleTypeFilter, ZipCodeRangeRuleTypeFilter, and ZipCodeRuleTypeFilter classes to use the isTaxActive method if the activeFrom property is not null (link,link,link,link,link,link,link,link)
Modify the getTaxRules method of the SalesChannelContextFactory class to filter the tax rules by the highest type position and the latest activation date (link)
Modify the testGetTaxRuleCollectionCustomerShippingCountry and testGetTaxRuleCollectionCustomerShippingTypeZipCodeRangeOverridesIndividualStates methods of the SalesChannelContextTest class to create and assert tax rules with different activeFrom dates (link,link,link,link)
Add the sw-datepicker component to the sw-settings-tax-rule-modal template and the sw-settings-tax-rule-modal unit test to allow the user to select a date and time for the activeFrom property (link,link)
Add the activeFrom property to the list of mapped errors and columns for the sw-settings-tax-rule-modal and sw-tax-rule-card components to display validation errors and activation dates for the tax rules (link,link)
Add the sw_settings_tax_rule_grid_column_tax_rule_active_from block to the sw-tax-rule-card template to format the activeFrom property as a date string in the grid column (link)
Add the labelActiveFrom snippet to the de-DE.json and en-GB.json files for the sw-settings-tax module to provide the German and English translations for the label of the activeFrom property (link,link)

Something's wrong with "Lint Admin" because it's getting errors in files I haven't changed....

Something's wrong with "Lint Admin" because it's getting errors in files I haven't changed....

@shyim Can you check that job?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-29559
Please use this issue to track the state of your pull request.

Hey @CodeproSpace, thank you for your contribution.
We've imported the PR to our internal gitlab instance, to be able to show this to our designers and other non-developer team members.
They want to have a good look to it, before we can merge this.
I'll keep you updated.

Hey @CodeproSpace, sorry for the delay in this. We did not forget you, no worries.
We've had a lot in the pipeline in the past few weeks.
I will ensure, that somebody from quality assurance will have a look at this in the next days. Form a dev perspective, your PR looks got to me.

The PR was just merged on our side and will be part of the next release.
Thank you for your contribution :+1:  @CodeproSpace :tada: :tada: :tada:


[Feature] Auto Update Plugins!

Is your feature request related to a problem? Please describe.

Describe the feature you'd like
Make a toggleable setting which disables that banner and instead automatically updates the plugins without asking.

Duplicate of #1224

BD has no updater at all yet, any notices you get for updating plugins are added by other plugins (probably BDFDB or ZeresPluginLibrary).

BD has no updater at all yet, any notices you get for updating plugins are added by other plugins (probably BDFDB or ZeresPluginLibrary).

Ah okay, makes sense. Thank you!

Does BetterDiscord 1.8.0's new plugin/theme update system allow for client-level updating rather than plugin-level? This would be very convenient if so.


feat: DMS endpoint credential from secret manager

AWS DMS endpoint resource now(https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/dms_endpoint#secrets_manager_access_role_arn) supports getting credentials from secret manager.
Description
AWS DMS endpoint resource now(https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/dms_endpoint#secrets_manager_access_role_arn) supports getting credentials from secret manager. This two line change allows users of the module to configure the DMS credentials from the secret manager.
  secrets_manager_arn             = lookup(each.value, "secrets_manager_arn", null)   secrets_manager_access_role_arn = lookup(each.value, "secrets_manager_access_role_arn", null)
Motivation and Context
Getting credentials from secret manager for DMS  is particularly useful when the database credentials has special characters which RDS supports but the DMS does not support.

Breaking Changes

How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects. I have tested the changes from my local branch.

Thanks for the PR @vigneshwar-rajkumar-vineti  - we'll need to update the minimum supported version of the AWS provider for this change

@bryantbiggs Thank you for the review. Is that something that can be done at this time ? Also Can I update this PR with the minimum supported version.


NaN

Redirect: https://github.com/louislam/uptime-kuma/issues/1340

@louislam delete this too please


drivers: flash: remove W25QXXDV driver

The spi_flash_w25qxxdv driver has been superseded by the generic
spi_nor driver for over a year.  The only non-refactoring change to
the W25Q driver in the last 18 months was done to support a backport
to 1.14.
All devices supported by spi_flash_w25qxxdv driver are expected to be
supported by the spi_nor driver, using the standard jedec,spi-nor
devicetree compatible.  No in-tree devicetree files make use of this
driver.
Remove the confusion about which driver to select by removing the
unmaintained redundant driver.
Signed-off-by: Peter Bigot<EMAIL_ADDRESS>

Inspired by #27791.  I don't think we have a policy for this, but as deprecation isn't really feasible I propose to email the user and devel lists requesting feedback on this PR, with merge in one week if it's approved.

:+1:  Just wasted an hour last week to find the differences

I have some general questions about this, I'm not against removing the old driver:

The first thing that comes to mind is “if it ain’t broke, don’t fix it” - does the w25qxxdv driver work fine without issues? If so, then I don’t really see a lack of code updates as the main reason to remove it.
"All devices supported by spi_flash_w25qxxdv driver are expected to be supported by the spi_nor driver, using the standard jedec,spi-nor devicetree compatible" this reads like it hasn't been fully tested or there might be a device whereby it doesn't work. Can you guarantee 100% of devices using the w25qxxdv driver will work with the spi_nor driver?

The first thing that comes to mind is “if it ain’t broke, don’t fix it” - does the w25qxxdv driver work fine without issues?

No, per #27791 and other informal communications.  At least it could only work if you used a compatible that incorrectly identified the device actually present.

Can you guarantee 100% of devices using the w25qxxdv driver will work with the spi_nor driver?

No, because I have no idea what devices are using that driver.
I can guarantee[*] that any JEDEC SPI NOR serial flash memory using standard 4-signal (SDI, SDO, SCK, CS) SPI that does not work with the spi-nor driver would be considered a bug in that driver and given pretty high priority for a fix.  That is not true for the winbond driver, which nobody maintains.
[*] not a guarantee, but certainly an intent.

I think the file dts/bindings/mtd/winbond,w25q16.yaml must be removed, too.

Thanks, done, also the all-device build devicetree node.  The other references are still valid (e.g. when there's an actual winbond device being identified).

I propose to email the user and devel lists requesting feedback on this PR, with merge in one week if it's approved.

I suppose I should have marked this DNM, or maybe we need another label that indicates "Delayed awaiting community feedback".

I suppose I should have marked this DNM, or maybe we need another label that indicates "Delayed awaiting community feedback".

I think DNM is sufficiently general for this. It's always possible to leave a comment explaining why it's DNM as done here: https://github.com/zephyrproject-rtos/zephyr/pull/27688#issuecomment-680157951


Support the Layout data type so the content variable can be passed

Making good progress on this, hopefully it'll be pushed by tomorrow.


Update dart, flutter_web, and appengine

It seems like https://storage.googleapis.com/compilation_artifacts/2.4.1/dart_sdk.js isn't working.

I'll publish the compilation artifacts as soon as my laptop updates to Dart 2.4.1. I'll re-run the travis build when I've published.

TBR: @johnpryan =)


make linux start script relative

Thanks for the effort, but this is not how the Linux start script is intended to be used.
It's being copied to the app folder just after the compilation and therefore the directories and file modes are correct the way they are. You don't start the application from the build/resources/linux folder.
If you want to launch the app directly from the repo, follow the steps described in CONTRIBUTING.md.


Modmail implementation

Currently working on the modmail implementation for snoots. This one was a touch trickier because of the quantity of new interfaces needed to make the whole thing work. Typedoc complained about nested interfaces, so the documentation now seems overwhelmingly polluted with new interfaces.
Features

New modmail controller that exposes the following

modmail.getConversations( subreddit, after?, limit?, sort?, state? ), to fetch an array of recent conversations. These are minimally detailed.
modmail.getConversation( conversationID, markRead), to fetch a modmail conversation in detail. This makes use of an extended class to add these extra details
modmail.approveParticipant( conversationID )
modmail.archiveConversation( conversationID )
modmail.disapproveParticipant( conversationID )
modmail.highlightConversation( conversationID )
modmail.muteParticipant( conversationID, duration )
modmail.tempBanParticipant( conversationID, duration)
modmail.unarchiveConversation( conversationID )
modmail.unbanParticipant( conversationID )
modmail.unmuteParticipant( conversationID )
modmail.getSubreddits( )

A ModmailConversation class to handle these new functions, along with data sent back by reddit
A ModmailConversationDetailed class to handle conversations where extra details have been fetched, including messages, moderator actions, etc. This class extends ModmailConversation

Not yet implemented

modmail.createNewConversation. I just haven't got round to it yet
modmail.replyToConversation. Similarly.
modmail.unhighlightConversation. This requires the DELETE method on the gateway, which as yet is not implemented. It has been implemented in #83, and don't see a reason to re-implement it until a decision is made on the PR.
modmail.markAsRead
modmail.markAsUnread
modmail.getUnreadCount

Weird stuff
For the life of me, I can't seem to get the api/mod/bulk_read API endpoint to return anything other than Error 404. It remains implemented in modmail.bulkMarkRead, but as far as I can tell, this endpoint no longer exists.
Hope to get some feedback on this. Seems to be taking ages to get this pull requests looked at though.

Codecov Report

Merging #94 (0792cb1) into master (4e61001) will decrease coverage by 0.73%.
The diff coverage is 0.00%.

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


Syntax highlighting javascript fails in markup mode if it highlights each line

Hi, if you use prism.highlight to highlight each line instead of all at once, the javascript mode in markup fails to highlight anything within

Same for css too.

Yep, confirmed that by them adding .top in the value, it fixed the problem in highlight.js (that's if you do)
state.baseContext = result.top; after it.

cc @Golmote

https://github.com/isagalaev/highlight.js/commit/bc42f1b6955793fb98c6a4e92a024fa9e0f54375

Please use the "Edit" option on your comments instead of commenting multiple times in a row.
I understand your issue now, and I see how highlight.js implemented a solution for it. That being said, Prism does not really work the same way, so I'm not sure it would be doable.
Knowing what was "parsed" in the lines before the one we want to highlight is one thing, but Prism might need to know what comes after the current line. And I don't see how we could circumvent this.

oh, i will use the edit button next time (sorry about that).
And oh.

Seems prism.js is better at highlighting then highlight.js :).
I really like the colours + that it highlights functions and params better.
/me just wishes there was a way for it to look ahead like highlight.js


Add DBIX Support

Sorry. We do not allow default nodes without EIP-155. Anything sent via this node could be replayed on other Ethereum networks, like ETH, ETC, EXP, UBIQ, etc.
We strongly encourage you to implement EIP-155 on your network before you have cases of people losing valuable ETH when they only meant to send DIBX.
You can read more about EIP-155 and replay protection here: https://github.com/ethereum/EIPs/blob/master/EIPS/eip-155.md


Added a FAQ Section

@kunal-kushwaha, @samjn08, I added a FAQ section and wrote 4 questions. We can edit it and can change the question, answers. This is just an idea if you like it, then we can improve this section.
Thank you !!

Ok will do


Opens Chrome stable if Chrome Canary is running

Description
npm start opens wrong browser. I have both Chrome and Chrome Canary on my system, but I only use Canary (which is set as my default browser system-wide).
Expected behavior
npm start should open my default browser.
Actual behavior
Ignores my default browser setting, and opens Chrome (stable).
Reproducible Demo

Start Google Chrome Canary.app
run npm start

Cause of the bug
node_modules/react-dev-utils/openBrowser.js:
execSync('ps cax | grep "Google Chrome"');

grep incorrectly matches "Google Chrome Canary". Probably the same error would happen with Beta.
execSync('ps cax | grep "Google Chrome$"');

seems to fix the problem (tested on OS X 10.11). Alternatively, pgrep -x "Google Chrome" may give more precise result.

@n3tr Could this be a regression after https://github.com/facebookincubator/create-react-app/pull/1165? Can you check before and after?

created PR #1215

@pornel Can you help me verify by change "Google Chrome" to "Chrome" in node_modules/react-dev-utils/openChrome.applescript line 17 and 61, does it works as expected?

Yes, changing  "Google Chrome" to "Chrome" in the applescript makes Canary reload the page as expected.

Should be fixed in<EMAIL_ADDRESS>Please verify.
https://github.com/facebookincubator/create-react-app/releases/tag/v0.8.4


Multiple apks builded but not actually different

Hi!
I'm experiencing a strange behavior: until last week when I builded apks those was recognized respectively as x86 and armv7 when uploaded to the Play Store.
Now, even if I do the same tasks and I get the same two apks, those apks are recognized as compatible with all the native architecture available.
Is anything that happens just to me? Any ideas?
Thanks!

Do you mean you built two different architecture apk , but can't uploaded to the Play Store?

When I build the app I get two flavors: x86 and arm. I can upload both the apks successfully on PlayStore but it says that both apks support all the native platform even if each one is actually build for a single architecture.
For example this is what I see on PlayStore for android-x86-release.apk

the same happens for android-armv7-release.apk

@enricodeleo did you already solve that issue? We are experiencing the same issue!

I have filed a bug to track it in JIRA
https://crosswalk-project.org/jira/browse/XWALK-4970

hi,
I verfiy this issue with these test steps:

$ cordova create foo com.example.foo foo
$cd foo
$cordova platform add android
$ cordova plugin add ../cordova-plugin-crosswalk-webview   --->>download form https://github.com   and crosswalk version:14+
$ cordova build android
then $aapt d --values badging android-armv7-debug.apk
{code}
supports-screens: 'small' 'normal' 'large' 'xlarge'
supports-any-density: 'true'
locales: '----' 'ca' 'da' 'fa' 'ja' 'nb' 'de' 'bg' 'th' 'fi' 'hi' 'vi' 'sk' 'uk' 'el' 'nl' 'pl' 'sl' 'tl' 'am' 'in' 'ko' 'ro' 'ar' 'fr' 'hr' 'sr' 'tr' 'cs' 'es' 'it' 'lt' 'hu' 'ru' 'lv' 'sv' 'iw' 'sw' 'en-GB' 'zh-CN' 'pt-BR' 'es-US' 'pt-PT' 'zh-TW'
densities: '120' '160' '240' '320' '480' '640'
native-code: 'armeabi-v7a'
{code}
then $aapt d --values badging android-x86-debug.apk
{code}
supports-screens: 'small' 'normal' 'large' 'xlarge'
supports-any-density: 'true'
locales: '----' 'ca' 'da' 'fa' 'ja' 'nb' 'de' 'bg' 'th' 'fi' 'hi' 'vi' 'sk' 'uk' 'el' 'nl' 'pl' 'sl' 'tl' 'am' 'in' 'ko' 'ro' 'ar' 'fr' 'hr' 'sr' 'tr' 'cs' 'es' 'it' 'lt' 'hu' 'ru' 'lv' 'sv' 'iw' 'sw' 'en-GB' 'zh-CN' 'pt-BR' 'es-US' 'pt-PT' 'zh-TW'
densities: '120' '160' '240' '320' '480' '640'
native-code: 'x86'
{code}

so,can you provide crosswalk version and crodova-plugin-crosswalk-webview download form github or npm?Then i will verfiy this issue again.

hi,
I verfiy this issue with these test steps:

$ cordova create foo com.example.foo foo
$cd foo
$cordova platform add android
$ cordova plugin add ../cordova-plugin-crosswalk-webview   --->>download form https://github.com   and crosswalk version:14+
$ cordova build android
then $aapt d --values badging android-armv7-debug.apk

supports-screens: 'small' 'normal' 'large' 'xlarge'
supports-any-density: 'true'
locales: '--_--' 'ca' 'da' 'fa' 'ja' 'nb' 'de' 'bg' 'th' 'fi' 'hi' 'vi' 'sk' 'uk' 'el' 'nl' 'pl' 'sl' 'tl' 'am' 'in' 'ko' 'ro' 'ar' 'fr' 'hr' 'sr' 'tr' 'cs' 'es' 'it' 'lt' 'hu' 'ru' 'lv' 'sv' 'iw' 'sw' 'en-GB' 'zh-CN' 'pt-BR' 'es-US' 'pt-PT' 'zh-TW'
densities: '120' '160' '240' '320' '480' '640'
native-code: 'armeabi-v7a'

then $aapt d --values badging android-x86-debug.apk
supports-screens: 'small' 'normal' 'large' 'xlarge'
supports-any-density: 'true'
locales: '--_--' 'ca' 'da' 'fa' 'ja' 'nb' 'de' 'bg' 'th' 'fi' 'hi' 'vi' 'sk' 'uk' 'el' 'nl' 'pl' 'sl' 'tl' 'am' 'in' 'ko' 'ro' 'ar' 'fr' 'hr' 'sr' 'tr' 'cs' 'es' 'it' 'lt' 'hu' 'ru' 'lv' 'sv' 'iw' 'sw' 'en-GB' 'zh-CN' 'pt-BR' 'es-US' 'pt-PT' 'zh-TW'
densities: '120' '160' '240' '320' '480' '640'
native-code: 'x86'

@enricodeleo can you provide the version of crosswalk and crodova-plugin-crosswalk-webview download from NPM or master when you used to builded apk?
Then we can help to reproduce  this issue.

@jingjingxli I use the plugin at version 1.2.0, crosswalk is at version 14.43.343.24.
I also tried to checkout the plugin from master branch and re-build this morning with --release and what I got was native-code: 'arm64-v8a' 'armeabi' 'armeabi-v7a' 'x86' 'x86_64'

Hi @jingjingxli and @enricodeleo,
I've found the reason for this issue in our app. In previous versions we had included the following plugin cordova-sqlite-storage but somehow sneaked back in. This plugin adds native libraries for each specific Android architectures. Removing this plugin solved the issue!
@enricodeleo it could be possible different plugins or frameworks can cause this issue. I've detected it by unzipping a APK and look in the ./lib folder. If there are multiple folders, named by architecture, see which libraries are added in the other architecture folders. There should only be one folder named the architecture the build is for.
Cheers!

@ChristiaanScheermeijer thank you for the hint! Checking the ./lib folder I was able to find a .so library added by SQLite Plugin. Appartently the latest versions of this plugin prevent the build phase to generate multiple apks. The only solution for now is to switching back com.brodysoft.sqlitePlugin to version r1.0.4.
Hope this feedback helps others that are experiencing the same issue.

I'm also seeing this compatibility issue with the Cordova-sqlite-storage plugin. The apk lib directory looks like this:
lib
|-- arm64-v8a
|   +-- libsqlc-native-driver.so
|-- armeabi
|   +-- libsqlc-native-driver.so
|-- armeabi-v7a
|   |-- libsqlc-native-driver.so
|   +-- libxwalkcore.so
|-- x86
|   +-- libsqlc-native-driver.so
+-- x86_64
    +-- libsqlc-native-driver.so

This issue is also mentioned on this stackoverflow question.
Is there any acceptable way to use these two cordova plugins simultaneously?


fix: allow 0h as permanent duration option

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5924

Relevant Lines:
8027

💛  - Coveralls


feature: Make .meltano path configurable

Feature scope
Configuration (settings parsing, validation, etc.)
Description
Making creation of .meltano  folder configurable will make it easy when running Meltano projects in read-only file systems. This will be similar to how Meltano accepts db path

Thanks @edgarrmondragon . I got confused and created this issue on wrong repo. Thanks for fixing this

@rawwar - Love this idea. Thanks for raising.
Is this a feature you may be interested in contributing?

Yes, I will be working on this. As of now, I am doing some analysis on how meltano supports other configurations. Once I am done with that, i'll post my approach

@aaronsteers .. Can you please check the draft pr.

@edgarrmondragon assigning to you since you're on the PR


Import of DLR OPS files

For PHILEAS the DLR tool for flight preparation can export the flight path as submitted to ATC as a kml and csv file. It would be beneficial if one of these files could be imported into MSS again to further work on the flight plan.  The csv file is attached directly, the kml as zip. Maybe someone here is faster than I am with writing a plugin. Thanks, Daniel
EDMO - EDMO - 20230725 - 0915 [DL4] - DADLR.csv
EDMO - EDMO - 20230725 - 0915 [DL4] - DADLR_kml.zip

On windows, the utf-8 encoding does not work with xml-parsing. Switching to "latin1" seems to have fixed the problem.

Dnce if I should guess over more than 5 years ago Windows got completly utf-8 aware
https://learn.microsoft.com/de-de/windows/apps/design/globalizing/use-utf8-code-page
recently python3 was added to excel. This would be impossible without utf-8

Both Wolfgang and Sören had a crash on Windows with xml not being able to parse the file. With latin1 decoding, it works.
I haven't looked into why the xml decoding crashes, could be other errors on our side, but with latin1, it works like a charm on Sörens Windows computer.

We maybe can use sys.getdefaultencoding()  for decoding


fix: unique name for di graph singletons

closes: https://github.com/customerio/issues/issues/10311
Changes

Uses fully qualified name for DiGraph keys instead of simple name to avoid duplication

@Shahroz16 Agreed. I too initially tried with similar options first, but eventually decided to go with short-term fix to prevent any potential bugs first. We can discuss this more internally and see how to improve this in future.


DragSelectionUpdateCallback not found - BREAKING AFTER FLUTTER UPGRADE

Package fails to compile after flutter upgrade:
: Error: Type 'DragSelectionUpdateCallback' not found.
text_selection.dart:771
final DragSelectionUpdateCallback? onDragSelectionUpdate;
^^^^^^^^^^^^^^^^^^^^^^^^^^^
: Error: The non-abstract class 'RawEditorState' is missing implementations for these members:
raw_editor.dart:240

TextInputClient.insertContent

The package analytics on pub.dev also shows failure :(
https://pub.dev/packages/flutter_quill/score
This is breaking my app :(

Getting same error here as well.
`Launching lib/main_dev.dart on sdk gphone64 arm64 in debug mode...
Running Gradle task 'assembleDebug'...
../../.pub-cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart:716:9: Error: Type 'DragSelectionUpdateCallback' not found.
final DragSelectionUpdateCallback? onDragSelectionUpdate;
^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart:716:9: Error: 'DragSelectionUpdateCallback' isn't a type.
final DragSelectionUpdateCallback? onDragSelectionUpdate;
^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/raw_editor.dart:277:7: Error: The non-abstract class 'RawEditorState' is missing implementations for these members:

TextInputClient.insertContent
Try to either
provide an implementation,
inherit an implementation from a superclass or mixin,
mark the class as abstract, or
provide a 'noSuchMethod' implementation.
`
Running on latest version of flutter.

Flutter 3.10.0 • channel stable • https://github.com/flutter/flutter.git Framework • revision 84a1e904f4 (2 days ago) • 2023-05-09 07:41:44 -0700 Engine • revision d44b5a94c9 Tools • Dart 3.0.0 • DevTools 2.23.1

I fixed in: https://github.com/Mmisiek/flutter-quill.git

same issue. Will try this PR and see if this helps

@singerdmx I'm still experiencing this issue after flutter_quill: 7.1.17+1:
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart(716,9): error G5FE39F1E: Type 'DragSelectionUpdateCallback' not found. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/raw_editor.dart(277,7): error G76B49859: The non-abstract class 'RawEditorState' is missing implementations for these members: [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/screenshot-1.3.0/lib/screenshot.dart(160,7): error GC6690633: No named parameter with the name 'window'. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart(716,9): error G9B3D8C68: 'DragSelectionUpdateCallback' isn't a type. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\adilh\Documents\GitHub\saber\build\windows\CMakeFiles\fb8fe2dbd8e12b
0697a675fb01bf473c\flutter_windows.dll.rule;C:\Users\adilh\Documents\GitHub\saber\build\windows\CMakeFiles\d9012f90891ce70ee21846a3c69bf2b8\flutter_assemble.rule;C:\Users\adilh\Documents\GitHub\saber\windows\flutter\CMakeLists.txt' exited with code 1. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
Building Windows application...                                   118.2s
Exception: Build process failed.

The static analysis fails with the same error on https://pub.dev/packages/flutter_quill/score

@singerdmx I'm still experiencing this issue after flutter_quill: 7.1.17+1:
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart(716,9): error G5FE39F1E: Type 'DragSelectionUpdateCallback' not found. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/raw_editor.dart(277,7): error G76B49859: The non-abstract class 'RawEditorState' is missing implementations for these members: [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/screenshot-1.3.0/lib/screenshot.dart(160,7): error GC6690633: No named parameter with the name 'window'. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_quill-7.1.17+1/lib/src/widgets/text_selection.dart(716,9): error G9B3D8C68: 'DragSelectionUpdateCallback' isn't a type. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\adilh\Documents\GitHub\saber\build\windows\CMakeFiles\fb8fe2dbd8e12b
0697a675fb01bf473c\flutter_windows.dll.rule;C:\Users\adilh\Documents\GitHub\saber\build\windows\CMakeFiles\d9012f90891ce70ee21846a3c69bf2b8\flutter_assemble.rule;C:\Users\adilh\Documents\GitHub\saber\windows\flutter\CMakeLists.txt' exited with code 1. [C:\Users\adilh\Documents\GitHub\saber\build\windows\flutter\flutter_assemble.vcxproj]
Building Windows application...                                   118.2s
Exception: Build process failed.

The static analysis fails with the same error on https://pub.dev/packages/flutter_quill/score

There is no new build, you need to use version from GIT.

You can raise a PR to bump the version

Somehow I thought that version update was part of PR merging.  When creating PR one does not know when it will be merged. Shall I create another PR with version bump ?

Yes please update changelog as well


setuptools CVE

Upgrade setuptools to 70.0.0
https://github.com/ministryofjustice/operations-engineering-reports/security/code-scanning/11

setuptools upgraded to patched version 70.0.0


AzureBlobFile._fetch_range fetches too many bytes?

Right now _fetch_range is implemented as https://github.com/dask/adlfs/blob/96958331854c5a67917d4c9a802a62720d091374/adlfs/spec.py#L1730-L1748. I think that's mixing up the meaning of end from fsspec and length from azure.storage.blob. If I fetch a with a non-zero start, I read past the end:
In [32]: fs = adlfs.AzureBlobFileSystem("daymeteuwest")

In [33]: blob = fs.open("daymet-zarr/annual/hi.zarr/.zmetadata")

In [34]: len(blob._fetch_range(5, 10))
Out[34]: 10

IIUC, the intent is to read from bytes 5 to 10, so the length should be 5.

Thanks @TomAugspurger .  You are correct.  Submitting fix now.

closed with #247


zabbix agent: do not cache the server's InetAddress internally, allow…

… re-resolve from time to time
In our environment we noticed that microservices using the Zabbix active Agent would lose track
of Zabbix Proxy when we replaced the proxy instance (which makes IP address to change).
The reason is InetAddress caches the resolved IP internally. The appropriate way is let
agent call InetAddress.getByName() often (because internally they cache the results,
and maintain appropriate TTL) - so that environment DNS changes are properly reflected.
It seems by letting ActiveThread use the Socket(String host, int port) constructor, the Socket class will do the right thing (resolve as appropriate, etc).

Looks like a nice change to me. Thanks for the contribution...


Error: type instantiation is excessively deep and possibly infinite

Describe the bug
Hello,
I am using the 3.33.0 version of @ts-rest/core, @ts-rest/next and @ts-rest/react-query with Next.js 14.2.4. When I set the project up, support for the app directory wasn't available so I configured it using the pages directory.
I would like the API to be accessible from any origin. I setup CORS at the Next.js level as described here: https://nextjs.org/docs/pages/api-reference/next-config-js/headers#cors
However, this didn't seem sufficient; I got CORS errors when making a request. I created a test API route outside of ts-rest (inside app/api), and that worked fine, which suggested to me that the issue had something to do with ts-rest. I eventually came across this in the documentation: https://ts-rest.com/docs/serverless/options – it suggests CORS needs to be explcity enabled, however this option is only listed for the ts-rest/serverless package which I am not at present using.
So my first question is; is there a way to enable CORS without having to use this?
I then tried upgrading my project to use ts-rest/serverless (as part of which I moved everything over to the app directory along with the other required changes, and upgraded every package to the latest version (3.50.0).
After completion, I noticed my project was failing to build as I was getting the following errors in the files that use the @ts-rest/react-query client:
Type instantiation is excessively deep and possibly infinite.

An example of the code that is being flagged as erroneous:
const useSearch = ({ query }: { query: string }) => {
  const { workspaceSlug } = useWorkspaceContext();

  return client.workspaces.workspace.search.search.useQuery(
    [workspaceSlug, "search", query],
    { params: { workspaceSlug: workspaceSlug as string }, query: { query } },
    {
      enabled: query.length > 0,
    },
  );
}

Interestingly, these errors don't show up in VS Code – they only appear when I try building / run ts:watch.
I spent a while searching for a solution, and came across this thread: https://github.com/ts-rest/ts-rest/issues/389 – I tried to follow the suggestions made here, but no luck.
Switching back to my main branch, I then tried only installing ts-rest/serverless without making any other changes. It turns out that the simple act of having it installed is enough to trigger the errors I mentioned above - with no other changes required.
Unsure whether or not this is a bug, but flagging it here in case – and if anyone has any suggestions as to what I could attempt next, I would be most grateful.
How to reproduce
See above – if necessary I will attempt to create a repro.
Expected behavior
No response
Code reproduction
See above.
ts-rest version
"@ts-rest/core": "^3.33.0",
"@ts-rest/next": "^3.33.0",
"@ts-rest/react-query": "^3.33.0",
"@ts-rest/serverless": "3.50.0",

I realised after posting this what the problem was; whilst testing things I'd placed my contracts into two routers, like so:
export const testContract = c.router({
  health: healthContract,
  utils: utilsContract,
  user: userContract,
  workspaces: workspacesContract,
});

export const apiContract = c.router(testContract);

It should have just been:
export const apiContract = c.router({
  health: healthContract,
  utils: utilsContract,
  user: userContract,
  workspaces: workspacesContract
});

Leaving this here in case it helps someone else.