[HeroLayout] Add Simple mode

Closes #.
Vercel link:

https://kitten-git-…

Screenshot:
TODO:

[ ] Tests
[ ] Changelog
[ ] A11Y
[ ] Stories / Docs
[ ] New component added to assets/javascripts/kitten/index.js

🎉


(Issue #61) : Deal with PCL.

Only Kinugasa.Mvvm (Other project will deal with PCL)

Relate to #61


move constants to config file

common.h has a lot of stuff that seems like it should be configurable at launch time, not compile time.

@cw75, was this fixed by #59?

Yes


Fix checklist page

Switch to left justification
Expand the columns when the screen is large
Provide a better ordering when the screen is small
Have name links go to the show name page rather than a page of observations that ignores the user id

Coverage: 92.715% (+0.0003%) from 92.714% when pulling c987396a907790279d95983863fe5306085b3432 on fix-checklist-page into b81b333206f4690f9d613c4a0374f922038664fd on master.


[Theme Architecture] TBD

As a developer, I want a logical, understandable system defining the name and use of our theme variables.
AC:

All current use cases considered and accounted for
Reasonable effort to consider future use cases

For potential reference:

Closing as we cover most of this in #570


how to convert hd5 file to ckpt

I have model build in keras and save in hd5 file format.How to convert to tensorflow ckpt file

@fchollet is this also working the other way round?
So converting ckpt to h5?
How would that look like?

@GustavZ Nope it doesn't. You basically have to load the checkpoint file, extract the weights, save them, and then use set_weights to load them into the keras model.

You can load your Keras model (with the Keras backend), then export a TF checkpoint of the TF graph created when loading the Keras model.
In general, I would recommend posting this kind of question on stackoverflow.
Essentially something like:
# Add ops to save and restore all the variables.
saver = tf.train.Saver()
model = keras.models.load_model("model.h5")
sess = keras.backend.get_session()
save_path = saver.save(sess, "/tmp/model.ckpt")

For TensorFlow 2.x
This code worked for me
saver = tf.train.Checkpoint()
model = keras.models.load_model('Model.hdf5')
sess = tf.compat.v1.keras.backend.get_session()
save_path = saver.save('model.ckpt')


First Draft of LSQ

@Zhanzhang-2020 and I went through the files and we removed the optimization started in python. The rest of the changes have to do with restructuring the model class, which should be helpful as we move towards integrating with Julia.


583 - made sure mar id is returned as str type and not as integer

What's In This
In misc.py check_mar_for_address() we were return the address id result from mar api call as integer and using that for dedupe process for *_addre.csv. This was primarily occurring for dhcd raw data since it isn't processed with any address id metadata. Simply cast it as str type before returning the result.
Test
Run get_api_data on dhcd unique data id and verify that all mar ids are string. Example is 1354 Euclid St.
TODO
Since we're already calling mar api to get mar id for missing projects, we should go ahead and store that value in the output.csv (_project.csv & _addre.csv) instead of making another api call when we do geocode clean up in the cleaner code. I'll create a ticket to expand on this further.

@jkwening Thanks for this! I'll re-run the load code tomorrow w/ this and see how it impacts our matching.


Support parentheses in DISTINCT clause, CURRENT_TIMESTAMP, CURRENT_TIME, and CURRENT_DATE

The following query works in TrinoDB:
SELECT DISTINCT (orderstatus,orderpriority)
FROM tpch.sf1.orders
LIMIT 10

🤔  I didn't catch this prior to merge, but there appears to be a difference between (col1, col2) and col1, col2 in postgres
alamb=# select distinct (t1_id, t1_name) from t1;
  row
--------
 (11,a)
 (22,b)
 (33,c)
 (44,d)
 (77,e)
(5 rows)

alamb=# select distinct t1_id, t1_name from t1;
 t1_id | t1_name
-------+---------
    44 | d
    77 | e
    11 | a
    22 | b
    33 | c
(5 rows)

alamb=#

Turns out mysql doesn't like this syntax at all:
mysql> select distinct t1_id, t1_name from t1;
+-------+---------+
| t1_id | t1_name |
+-------+---------+
|    11 | a       |
|    22 | b       |
|    33 | c       |
|    44 | d       |
|    77 | e       |
+-------+---------+
5 rows in set (0.01 sec)

mysql> select distinct (t1_id, t1_name) from t1;
ERROR 1241 (21000): Operand should contain 1 column(s)
mysql>

@yuval-illumex  any thoughts?


Attach and detach onScroll event handler

Hi, we have a use case where we dynamically need to change handlers in a scrollView but when it changes the first handler is still used.
I have created a repo to show this behavior https://github.com/pontusab/RN-onscroll one is using Animated from React Native and that one works, but the other one using reanimated in AppReanimated.js
Dependencies used:
  "react": "16.6.3",
   "react-native": "0.58.4",
   "react-native-reanimated": "^1.0.0-alpha.12"

On iPhone X 10.1
Is this something that reanimated supposed to support?

I have noticed the same behaviour. Would be awesome to get this in, if possible.


Geofence notifications are not working in background

Notifications are works only while i am logging into app(if I'm in given radius area) after that notifications are not getting.
My android device version is 9:
Code :-
let fence = {
      id:  2,
      latitude: 14.501243,
      longitude:62.814435,
      radius: 100,
      transitionType: 3,
      notification: {
        id: 1,
        title: 'Hello..',
        text: 'You crossed a fence',
        openAppOnClick: true
      }

how you have resolve this issue...Im also stuck with this

This plugin has been deprecated. Please find the other alternative


Overlay dont work with Game Stronghold Crusader

Hello,
i just wanted to use your library but i have the problem that the overlay will never be drawn above the Stronghold Crusader HD Game from firefly. Its a old Game maybe that's why.
Here my Setup, copied the example Code, for Notepad it works:
 private void CreateWindow()
        {
            var gfx = new Graphics()
            {
                MeasureFPS = true,
                PerPrimitiveAntiAliasing = true,
                TextAntiAliasing = true,
                WindowHandle = ParentWindow,
                VSync = true,
            };

            _window = new GraphicsWindow(0, 0, 800, 600, gfx)
            {
                FPS = 60,
                IsTopmost = true,
                IsVisible = false
            };
            _window.SetupGraphics += _window_SetupGraphics;
            _window.DestroyGraphics += _window_DestroyGraphics;
            _window.DrawGraphics += _window_DrawGraphics;
            _window.VisibilityChanged += Test;
            _window.Create();
        }

I used  the WindowHandle i get from:
[DllImport("user32.dll", SetLastError = true)]
private static extern IntPtr FindWindow(string lpClassName, string lpWindowName);
public void Dummy(){
_overlay.ParentWindow = FindWindow(null, "Crusader");
}
Also i don't know how to get the Class Name for the Window initialization.
Picture not ingame:

Ingame:

You can use
https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.process.mainwindowhandle?view=netframework-4.8
or
https://michel-pi.github.io/GameOverlay.Net/api/GameOverlay.Windows.WindowHelper.html#GameOverlay_Windows_WindowHelper_FindWindow_System_String_System_String_
to find your window instead of doing the PInvoke yourself.
You do not need to specify a class name for FindWindow. You also do not need one for the overlay because it will try to register a random one by itself.
You do not need to specify a render target (window handle) when using the Graphics object with the GraphicsWindow
Using VSync with a GraphicsWindow is also not needed since you can specify the FPS yourself.
Also, setting IsTopmost to true and IsVisible to false seems to be a bit of a conflict right?
var gfx = new Graphics()
{
    MeasureFPS = true,
    PerPrimitiveAntiAliasing = true,
    TextAntiAliasing = true,
};

_window = new GraphicsWindow(0, 0, 800, 600, gfx)
{
    FPS = 60,
    IsTopmost = true,
    IsVisible = true
};

Can you tell me the Video/Graphic settings of the game? You would either use Windowed Mode or Bypass the fullscreen mode for the overlay to be visible.

The Game has only Fullscreen Mode, only with the Program DxWnd i can force it to Window Mode.
Ingame it only runs with 30-60 fps.
The IsVisible was false because i only set it if the process is running and deactivate it after it closed
i also tested it with true at the start.
Wired is if i use the Program DxWnd and drag and drop the Window than it work correctly, so after the Window Moved.

I think the Stronghold Program set itself on topmost if i click in it.
(I click outside of the "Crusader "Window and after i click inside of it again it will not show the overlay)
Do you have a idea what i can do about this behavior?

it's because the game will always force itself to top again. nothing you can do there. the game is just to old

@michel-pi
ok i got it to work, i just wrapped the old ddraw.dll from Stronghold with the one from here: https://github.com/elishacloud/dxwrapper
With the file it upgrade to directX 9 and in the config i added Borderless Window Mode. After this it just worked great :)

Thanks for the library.


Better rendering of stop filter buttons on XS

Summary of changes
Asana Ticket: Stops Page | Filters overflow for XS breakpoint

Only affects stop page filters
Bumps buttons to new line under FILTERS header
Better margin alignment for wrapped buttons

👍 lgtm!


Support for laravel 10

thank you for this great package!
unfornately when i try to install in laravel v10, i got this error:

Thanks for this report. Support for laravel 10 added to the new release: https://github.com/filipfonal/filament-log-manager/releases/tag/1.2.0


Pages Functions Compiler

This PR open-sources our functions compiler which compiles a folder of functions into a single Worker script (previously this was done with the @cloudflare/pages-functions-compiler package).
It also adds support for sourcemaps (as requested in #138) and automatically watches all dependents (fixes #49).
Still to follow in a subsequent PR is a command which performs just this compile step once (which will be used by production builds). I'm thinking of having two modes:

wrangler functions compile which builds functions will a fallback of fetch
wrangler functions compile --service=SOME_SERVICE which builds functions with a fallback of env.SOME_SERVICE.fetch.

So Pages things would use wrangler functions compile --service=ASSETS.
This small piece of extra work would allow any user to use functions folder syntax to generate a Worker ✨

I'll review in a bit, but can start by saying this should probably be under pages, i.e- wrangler pages functions ...

To clarify, I haven't added any new commands yet. This PR is entirely built into wrangler pages dev. I was just talking about additionally exposing it in a different command in the future. I'm relatively indifferent to that being wrangler pages functions ... vs. wrangler functions .... Only thing I would say is that if people are using it outwith a Pages project, it's maybe a bit unexpected to use wrangler pages.


Drop [DynamicallyAccessedMembers] from Marshaler.AbiType

Related to https://github.com/microsoft/CsWinRT/pull/1460#issuecomment-1903537019. This PR drops the [DynamicallyAccessedMembers] use from Marshaler<T>.AbiType, which was not needed and was causing a lot of unnecessary stuff to be preserved unnecessarily (for explicit T types that were not helper types). The public members for the helper types are already preserved separately, and this attribute was only used to suppress warnings within the static constructor. I've changed the logic there to just flow the annitations through a local variable instead.

Saves 51 KB on .NET 9! 🎉


Connect to Flutter web debug session, from Chrome on Android device

Problem/Question
I have a Flutter web application thats deployed and working correctly on desktop. Only when I inspect via my Chrome on my Android device (with chrome://inspect:devices) I can see an "Uncaught" error.

Im trying to connect my android device to the debug session on my macbook but no luck yet. Tried ngrok proxy but even then I cant get the app to load (and no error).
Is there a way to either connect of decompile the main.js so I can understand the error better?
Logs

Flutter doctor output

[✓] Flutter (Channel stable, 2.8.1, on macOS 12.2 21D5039d darwin-x64, locale en-GB)
• Flutter version 2.8.1 at /Users/mark/fvm/versions/stable
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (5 weeks ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/mark/Library/Android/sdk
• Platform android-31, build-tools 30.0.2
• ANDROID_HOME = /Users/mark/Library/Android/sdk
• Java binary at: /Users/mark/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7351085/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.2)
• Android Studio at /Users/mark/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7351085/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
[✓] Android Studio (version 4.1)
• Android Studio at /Users/mark/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7199119/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
[✓] VS Code (version 1.63.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.32.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-x64     • macOS 12.2 21D5039d darwin-x64
• Chrome (web)    • chrome • web-javascript • Google Chrome 97.0.4692.99
• No issues found!

Thanks to discord I got this working just now :-) Posting the answer here as well.

Run your app with flutter run -d web-server Take note of the port of where its running
Expose your local port with ngrok (ngrok http [port])
Enable USB debugging on you phone
Navigate to chrome://inspect#devices on you desktop
Open a tab with the ngrok URL on your phone and start inspecting.

Hope this helps someone struggling with the same issue later on.


Support key-value annotations on wrap

Based on @prashantv's suggestions.
In the future, we should consider adding support for errtrace.Wrap to include key-value pairs as part of wrapping.
These values can be shown next to the function call in the trace, e.g.
example.com/main.getUser name="foo"
    path/to/file:42

The API for this is undecided at this time.

cockroachdb/errors has a ton of useful functionality... but is a very heavyweight and complicated library.
In #106 I added an interface so that the main value proposition of this library is composable with external custom error types.
I hope that allows this particular project to maintain a tighter focus.

@StevenACoffman Agreed! We're hoping to keep this library lightweight and focused, and not turn it into a general "error utils" library.
Whether this feature goes into this library is still up for discussion, and it would have to be lightweight and composable (similarly to #106) if it did. An argument can be made that this is a logging-specific concern and it should go in a logerr package; nothing about this in set in stone at this time.


Needless Assignment in Collect

https://github.com/avidraghav/MyNotes/blob/624946c017a9560b51ac571178ac95eb83500847/app/src/main/java/com/raghav/mynotes/ui/AllTasksVM.kt#L25
Assigning Resource.Loading() to _task variable in the collect lambda does nothing as you are immediately assigning Success to it. Move it outside the scope.

Yes, are you willing to fix this by raising a PR?

Yes, will do it. Thanks @avidraghav.


Version undefined is now available

Hi
I installed version 0.11 (amd64.deb) in linux mint 19 and I keep getting a window popping up saying
"Version undefined is now available." with an option to download. The download link is to the github page with the current version which I just installed.

Same package on Ubuntu 18.04 and same behaviour.

on OSX 10.14 with version 0.11, and get the same behavior

Same for me on Ubuntu 18.04.1
Noticed that Tusk appears twice in my applications list.
It seems to be installed to:
/snap/tusk/16/tusk
/opt/Tusk/tusk-app
Both are 0.11.0 (x64)
I've uninstalled the one in /opt/ using Synaptic package manager. Let's see what happens...

Can confirm this failure on openSUSE Linux. Using the AppImage file.

Fedora 29 RPM package has the same problem.

The bug doesn't get reproduced if you run Tusk using the latest github sources.
Basically it seems like the bug was caused by updateURL change from 'https://klauscfhq.github.io/tusk/update.json' to 'https://klaussinani.github.io/tusk/update.json' here https://github.com/klaussinani/tusk/commit/0a7c477a0bae703e4a9cc6497151d2a623dc223c
The latest sources have the correct link and work fine but the 0.11 release packages still reference to the old update.json which is not present anymore.
I guess Klaus just needs to get the new release done with all the fixes included to make the problem go.  Can this be made into a less intrusive behavior if possible? Thanks very much!

os: ubuntu 18.04
version: tusk_0.11.0_amd64.deb
got the same issue

same on arch linux https://aur.archlinux.org/packages/tusk/
v0.11.2

0.11.2? The latest release both on https://github.com/klaussinani/tusk/releases and https://klaussinani.github.io/tusk/update.json is 0.11.0 :thinking:

Hi, same problem for me on Linux Mint 19.  Tusk was installed via snap.

Hi, Same problem here. 0.11.0 version installed with .deb on Linux Mint 19 and MX Linux 17.1. It is quite annoying some times. Hope to get it fixed soon. Thanks.

New 0.12.0 release is out. Full changelog & binaries are available on the GitHub Releases page.


UNUSED_CLASS_VARIABLE warning - except it's used and removing it crashes everything

Godot version:
v3.1.1.stable.official
OS/device including version:
iMac (Retina 5K, 27-inch, 2017)
macOS Mojave (10.14.5)
Issue description:
So I import a file that defines "mods" via class definitions. Via this line of code:
onready var Mods = load("res://Services/Mods.gd")

This line gets the warning UNUSED_CLASS_VARIABLE: The class variable 'Mods' is declared but never used in the script.
Except that it's used below in the same file:
onready var all_sentry_mods = [
	Mods.ModOption.new("Capacitor", Mods.CapacitorMod, 6, 'Damage +++   Fire rate --'),
	# ...
]

But if I was to remove the variable, the game would crash before it can start, stating:
Parse Error: Identifier not found: Mods

... pointing at that Mods.ModOption.new(...) line.
Can someone clarify what's going on, or if it's an actual issue?
Thank you!

Sounds potentially related to #29406, though that was fixed. Can you retest on a recent nightly version?

Sounds potentially related to #29406, though that was fixed. Can you retest on a recent nightly version?

Sure thing. I looked around in the GitHub Releases tab but I couldn't find any nightly release. Where could I get it?

You could try Calinou's builds.

Just tried and it would seem the warning is gone in the nightly build. Thanks!


forge(verify): OKLink support

OKLink contract verification
Motivation
Add support for OKLink contract verification

https://www.oklink.com/docs/zh/#explorer-rpc-api-contract-verifysourcecode hi @mattsse please check the link, which introduce the oklink verify.

@Pentaclezion @bixia I am not a developer on Foundry, so this is my own opinion - this is a very low quality pull request. If you want this to get merged, please fix the issues. Linking to the landing page of API documentation in chinese doesn't help.
In my opinion it would be easier for everybody for Oklink to have an Etherscan-compatible API endpoint. That way there don't need to be any changes in many many tools that you may want support for.  Then all sorts of frontend tools that grab the ABI from an etherscan-compatible API, to facilitate transaction creation (tools similar to OpenZeppelin Defender, Gnosis Safe etc etc.)
You will have an uphill battle changing this in many many places. I think it's easier if you provide an etherscan compatible API endpoint that everybody uses already. See etherscan docs here: https://docs.etherscan.io/api-endpoints/contracts#verify-source-code . Users will be able to migrate to your block explorer smoothly, without thinking about whether their tools support your API.
I am commenting here because I ran into this - My team is trying to use the new Polygon Amoy testnet, which is only supported on the oklink block explorer. We can't verify our contracts programmatically, because we rely on Foundry, which supports etherscan. It's a huge roadblock.


Restore acceptance testing from Windows

Since it's a PR from a clone, the windows tests were actually not run. It looks good to me, but we never know. might be better to test it in a branch.


Add TeamSpeak 3 Config

This is the draft of the TeamSpeak 3 config.

This template is affected by the config file issue.

Related Issue: https://github.com/CubeCoders/AMP/issues/583

what is the status of this plugin for amp at the moment is i in a working order ???

This is mostly working in Windows, but I have a few details to clean up before release. In Linux it is not ready for use. There are issues with how TS3 is packaged and a bug in AMP that I need to report.

How's the linux aspect of this? Recently changed their directory, so does AMP work with that?

I noticed they changed their structure and it's very annoying. Lol. I have a method that will work, but need time to implement it.

I'll have to verify. I know symlinking was failing. That was needed because I can't specify different working directories per OS. So I had to extract one into the folder of the other.

This is ready for testing. As of now it does not work in Docker containers due to the related bug. If changes are needed at a later time, I'll create a second PR. https://github.com/CubeCoders/AMP/issues/718

This was successfully tested for basic functionality by DocGalaxyBlock on Discord. (Thank you!)

Can verify this works as well. Good job IceofWraith

I appreciate you trying it! Please feel free to give any feedback if you come across anything at a later time too.

Error with docker
This task could not be completed: Running Update Tasks - - State: 3.

tar returned exit code 2 while attempting to extract archive.

Information:

OS
Linux

Platform
Debian GNU/Linux 11 (bullseye)

System Type
x86_64

Virtualization
Docker

Module
GenericModule

Loaded Plugins
FileManagerPlugin, EmailSenderPlugin, WebRequestPlugin, LocalFileBackupPlugin, CommonCorePlugin

Application Name
AMP

Application Version
<IP_ADDRESS>

Release Stream
Release

Build Date
26/07/2022 20:08

InstanceID
28add950-11d5-444a-a4aa-7a6cea6cf4d4

Last Executable
/usr/bin/ldd

Last Arguments
--version

Last Process ID
55

`

Thanks! I address that in my comment above. :)


Broken on nightly

ref http://pkg.julialang.org/detail/Patchwork.html - I'm not sure what to make of this, whether it's a Julia issue or something here. Likely caused by the 0.5-style comprehensions?
Testing Diffs
  > testing are_equal
  > things that should return empty patches
WARNING: symbol is deprecated, use Symbol instead.
 in depwarn(::String, ::Symbol) at ./deprecated.jl:64
 in symbol(::Symbol, ::Vararg{Symbol,N}) at ./deprecated.jl:30
 in (::##27#71)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:60
 in context(::##27#71, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:475
 in (::##5#49)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:59
 in facts(::##5#49, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:449
 in include_from_node1(::String) at ./loading.jl:426 (repeats 2 times)
 in process_options(::Base.JLOptions) at ./client.jl:266
 in _start() at ./client.jl:322
while loading /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl, in expression starting on line 42
WARNING: symbol is deprecated, use Symbol instead.
 in depwarn(::String, ::Symbol) at ./deprecated.jl:64
 in symbol(::Symbol, ::Vararg{Symbol,N}) at ./deprecated.jl:30
 in (::##27#71)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:60
 in context(::##27#71, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:475
 in (::##5#49)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:59
 in facts(::##5#49, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:449
 in include_from_node1(::String) at ./loading.jl:426 (repeats 2 times)
 in process_options(::Base.JLOptions) at ./client.jl:266
 in _start() at ./client.jl:322
while loading /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl, in expression starting on line 42
ERROR: LoadError: LoadError: MethodError: Cannot `convert` an object of type Int64 to an object of type Tuple{Vararg{Int64,N}}
This may have arisen from a call to the constructor Tuple{Vararg{Int64,N}}(...),
since type constructors fall back to convert methods.
 in Patchwork.Elem{:xhtml,:p}(::Array{Any,1}, ::Tuple{String}) at /home/vagrant/.julia/v0.5/Patchwork/src/Patchwork.jl:78
 in (::##27#71)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:60
 in context(::##27#71, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:475
 in (::##5#49)() at /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl:59
 in facts(::##5#49, ::String) at /home/vagrant/.julia/v0.5/FactCheck/src/FactCheck.jl:449
 in include_from_node1(::String) at ./loading.jl:426 (repeats 2 times)
 in process_options(::Base.JLOptions) at ./client.jl:266
 in _start() at ./client.jl:322
while loading /home/vagrant/.julia/v0.5/Patchwork/test/diff.jl, in expression starting on line 42
while loading /home/vagrant/.julia/v0.5/Patchwork/test/runtests.jl, in expression starting on line 1

This seems to be gone now.
btw, on that page all the badges say v0.3.0 for some reason

0.3.0 was your most recent tag?

Oh right, I confused that with the Julia version for some reason. Sorry! :P


Update shuttle-persist.mdx

This commit adds a step-by-step guide with detailed code examples (for rocket) as referenced by @oddgrd in this comment.

Like so? (Also please squash these commits when you merge! 🙈)


Add custom metrics to the maya api service

We don't have any information about how much request has been made and duration of request also on the endpoints like /latest/volume. We can add some custom metrics so that prometheus can scrap the metrics from that endpoint.

Merged #106


[WebSite][DOC]Where can I get more help?

Add a user guide in the community column: Where can I get more help?
The methods we recommend are StackOverflow, mailing lists, and our official Slack.
It is best to provide users with detailed questioning steps and templates.

I can do this.

done


Pypi mkdocstrings 0.15.0 has dependency constrain Jinja2 < 3.0

I'm encountering an error when updating the dependencies of one of my projects with pip-compile due to the dependency constrain Jinja2 < 3.0.
To Reproduce
Steps to reproduce the behavior:

Clone the repo: git clone https://github.com/lyz-code/mkdocs-newsletter/
cd mkdocs-newsletter
Edit the first line of docs/requirements.in from -r requirements.txt to -r ../requirements.txt
make update
See error

Could not find a version that matches jinja2<3.0,==3.0.0,>=2.10.1,>=2.11 from https://files.pythonhosted.org/packages/48/9b/dc3bbfc44d851632df958acf9d47e4de662c6bbd238e46798d555d427b27/Jinja2-3.0.0-py3-none-any.whl#sha256=2f2de5285cf37f33d33ecd4a9080b75c87cd0c1994d5a9c6df17131ea1f049c6 (from -r docs/../requirements.txt (line 56))
Tried: 2.0, 2.1, 2.1.1, 2.2, 2.2.1, 2.3, 2.3.1, 2.4, 2.4.1, 2.5, 2.5.1, 2.5.2, 2.5.3, 2.5.4, 2.5.5, 2.6, 2.7, 2.7.1, 2.7.2, 2.7.3, 2.8, 2.8, 2.8.1, 2.8.1, 2.9, 2.9, 2.9.1, 2.9.1, 2.9.2, 2.9.2, 2.9.3, 2.9.3, 2.9.4, 2.9.4, 2.9.5, 2.9.5, 2.9.6, 2.9.6, 2.10, 2.10, 2.10.1, 2.10.1, 2.10.2, 2.10.2, 2.10.3, 2.10.3, 2.11.0, 2.11.0, 2.11.1, 2.11.1, 2.11.2, 2.11.2, 2.11.3, 2.11.3, 3.0.0, 3.0.0
Skipped pre-versions: 2.0rc1, 3.0.0a1, 3.0.0a1, 3.0.0rc1, 3.0.0rc1, 3.0.0rc2, 3.0.0rc2
There are incompatible versions in the resolved dependencies:
  jinja2==3.0.0 from https://files.pythonhosted.org/packages/48/9b/dc3bbfc44d851632df958acf9d47e4de662c6bbd238e46798d555d427b27/Jinja2-3.0.0-py3-none-any.whl#sha256=2f2de5285cf37f33d33ecd4a9080b75c87cd0c1994d5a9c6df17131ea1f049c6 (from -r docs/../requirements.txt (line 56))
  Jinja2>=2.10.1 (from mkdocs==1.1.2->-r docs/requirements.in (line 3))
  Jinja2<3.0,>=2.11 (from mkdocstrings==0.15.0->-r docs/requirements.in (line 11))

Expected behavior
pip-compile correctly updates the dependencies.
Information (please complete the following information):

OS: Debian
Browser: irrelevant
mkdocstrings version: 0.15.0

Additional context
I've seen that the pyproject.toml file doesn't have that dependency, it asks for Jinja2 < 4.0. Maybe there was a packaging error.

Already fixed in https://github.com/mkdocstrings/mkdocstrings/pull/283, not released yet (which is why you see the error but found pyproject.toml to be correct)


Issue in datetime_convert.java

Description A clear and concise description of what the issue/bug is.
The java file is missing the import at the top of the file. It uses the LocalDateTime object without importing it.
Steps to Reproduce
First step to reproduce the issue/bug
Running the file will always produce this error
Expected Behavior A clear and concise description of what you expected to happen.
Expected behavior of the program is for is to run and convert date and time and format it.
Actual Behavior A clear and concise description of what actually happened.
The file will not be able to compile because it doesn't know the LocalDateTime object that is used.
Screenshots If applicable, add screenshots to help explain the issue/bug.
Additional Information
Operating System: MacOS
Browser: Chrome
Version: Monterey 12.5.1
Any other relevant information
Possible Solution If you have any suggestions for a possible solution, please include them here.
Related Issues Are there any related issues? If so, please link to them here.
Priority How critical is this issue/bug? Low, medium, or high priority?
High priority the file will not compile
Reproducibility How often can this issue/bug be reproduced? Always, sometimes, or random?
Always is an issue.
Impact What is the impact of this issue/bug? Low, medium, or high?
High - file does not run

added: import java.time.LocalDateTime


aur package is broken

==> Making package: pyprland 1.3.1-1 (Sun 14 May 2023 12:49:34 PM IST)
==> Retrieving sources...
  -> Cloning pyprland git repo...
Cloning into bare repository '/home/ultrahalf/.cache/paru/clone/pyprland/pyprland'...
remote: Enumerating objects: 268, done.
remote: Counting objects: 100% (30/30), done.
remote: Compressing objects: 100% (20/20), done.
remote: Total 268 (delta 14), reused 23 (delta 10), pack-reused 238
Receiving objects: 100% (268/268), 40.99 KiB | 545.00 KiB/s, done.
Resolving deltas: 100% (178/178), done.
==> Validating source files with md5sums...
    pyprland ... Skipped
==> Making package: pyprland 1.3.1-1 (Sun 14 May 2023 12:49:36 PM IST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Updating pyprland git repo...
==> Validating source files with md5sums...
    pyprland ... Skipped
==> Removing existing $srcdir/ directory...
==> Extracting sources...
  -> Creating working copy of pyprland git repo...
Cloning into 'pyprland'...
done.
Switched to a new branch 'makepkg'
==> Starting pkgver()...
Note: switching to '1.3.1'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Sources are ready.
pyprland-1.3.1-1: parsing pkg list...
==> Making package: pyprland 1.3.1-1 (Sun 14 May 2023 12:49:39 PM IST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Starting pkgver()...
HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Starting build()...
/usr/bin/python: No module named build
==> ERROR: A failure occurred in build().
    Aborting...
error: failed to build 'pyprland-1.3.1-1':
error: packages failed to build: pyprland-1.3.1-1

Sorry for the text dump I'm still exploring wayland

Thank you for the bug report, indeed, it looks like there was a wrong dependency, I just updated, can you confirm it's fixed ?

still failed to build
==> Making package: pyprland 1.3.1-2 (Wed 17 May 2023 01:29:22 PM IST)
==> Retrieving sources...
  -> Cloning pyprland git repo...
Cloning into bare repository '/home/cafo/.cache/paru/clone/pyprland/pyprland'...
remote: Enumerating objects: 292, done.
remote: Counting objects: 100% (54/54), done.
remote: Compressing objects: 100% (40/40), done.
remote: Total 292 (delta 27), reused 36 (delta 14), pack-reused 238
Receiving objects: 100% (292/292), 46.28 KiB | 1.13 MiB/s, done.
Resolving deltas: 100% (191/191), done.
==> Validating source files with md5sums...
    pyprland ... Skipped
==> Making package: pyprland 1.3.1-2 (Wed 17 May 2023 01:29:23 PM IST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Updating pyprland git repo...
==> Validating source files with md5sums...
    pyprland ... Skipped
==> Removing existing $srcdir/ directory...
==> Extracting sources...
  -> Creating working copy of pyprland git repo...
Cloning into 'pyprland'...
done.
Switched to a new branch 'makepkg'
==> Starting pkgver()...
Note: switching to '1.3.1'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Sources are ready.
pyprland-1.3.1-2: parsing pkg list...
==> Making package: pyprland 1.3.1-2 (Wed 17 May 2023 01:29:26 PM IST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Starting pkgver()...
HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Starting build()...
* Getting build dependencies for wheel...
* Building wheel...
Successfully built pyprland-1.3.0-py3-none-any.whl
==> Entering fakeroot environment...
==> Starting package()...
/usr/bin/python: No module named installer
==> ERROR: A failure occurred in package().
    Aborting...
error: failed to build 'pyprland-1.3.1-2':
error: packages failed to build: pyprland-1.3.1-2

Thank you for the bug report, indeed, it looks like there was a wrong dependency, I just updated, can you confirm it's fixed ?

nope
ERROR Backend 'poetry.core.masonry.api' is not available.

I'm supposed to have it fixed in 1.3.1-3 ...
I'll test on a minimal installation to understand what's still missing. I may need to add poetry too indeed, but I didn't want to depend on it directly.

HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Sources are ready.
pyprland-1.3.1-3: parsing pkg list...
==> Making package: pyprland 1.3.1-3 (Fri 19 May 2023 06:35:10 PM IST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Starting pkgver()...
HEAD is now at b117037 monitors: trigger the rules on init & reload
==> Removing existing $pkgdir/ directory...
==> Starting build()...
* Getting build dependencies for wheel...

Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/pyproject_hooks/_impl.py", line 321, in _call_hook
    raise BackendUnavailable(data.get('traceback', ''))
pyproject_hooks._impl.BackendUnavailable: Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 77, in _build_backend
    obj = import_module(mod_path)
          ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1128, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1128, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1128, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1142, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'poetry'

ERROR Backend 'poetry.core.masonry.api' is not available.
==> ERROR: A failure occurred in build().
    Aborting...
error: failed to build 'pyprland-1.3.1-3':
error: packages failed to build: pyprland-1.3.1-3

I added poetry too...  tell me if the latest version of the package still have some issues.

now it works. Thanks
I'll close this issue and fix the doc I think scratchpad doc is still using hpr... command instead of pypr in hyprland config in the readme


Fix Mythril link & add Mythril Platform fork

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks! We're still trying to figure out what to do with commercial tools. Let's fix the link to Mythril OSS and I'll think a little more about how we want to reference commercial tools, with what context, etc.

(In particular, because I don't want to get locked into a mode where we're obligated to list every security company under the sun.)

Fair enough.


fix(@angular-devkit/build-angular): fix error with inline styles when running extract-i18n

fixes: #20750
When extracting inline styles in Angular 12 there seems to be no CSS loader specified and there will be something like the following message:
angular-resource://:2:4 - Error: Module parse failed: Unexpected token (2:4)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
>     :host {
|       display: block;
|       height: 0;

This change adds the mime type to the angular-resource:// url in order to allow extract-i18n to provide a webpack.NormalModuleReplacementPlugin regex for the style URLs to handle them as the non inline styles are handled.

@alan-agius4 was just doing that right now. I think the title is now a little more accurate

@terencehonles can you add the PR description and the fixes clause as part of the commit message please?

@terencehonles can you add the PR description as commit message body and the fixes clause as part of the commit footer please?

@terencehonles can you add the PR description as commit message body and the fixes clause as part of the commit footer please?

Sorry, I had thought I had (for the description) and missed the trailer, updated.


add missing datetime import

Readme example fails if you are using the latest version of the repo due to the missing import. thought i'd add it quickly while im at it.

it seems like a duplicate to this PR

it seems like a duplicate to this PR

yep, looks like it, closing this then.


wip

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 8 (75.0%)  changed or added relevant lines in 2 files are covered.
47 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.4%) to 90.753%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

security/backends/elasticsearch/management/commands/init_elasticsearch_log.py
3
5
60.0%

Files with Coverage Reduction
New Missed Lines
%

example/apps/test_security/tests/commands.py
16
78.08%

security/backends/sql/writer.py
31
72.85%

Totals

Change from base Build<PHONE_NUMBER>:
-1.4%

Covered Lines:
3337

Relevant Lines:
3677

💛  - Coveralls


Can't run expectation against array index values

I can't seem to find a way to do ExpectJson with a path like payload[0].state or payload.[0].state, and based on the code i don't think it's supported.
Do you think it would be possible to update the way that paths are looked up from simp_json.GetPath to a loop that evaluates each of the keys and checks to see if there's is an index lookup in it?
Currently we have:
path_items := strings.Split(path, ".")
simp_json = simp_json.GetPath(path_items...)

We could look at something like (exclude the psuedo code, I haven't tested this):
for _, pathItem := range strings.Split(path, ".") {
	matches := regexp.MustCompile("^\\[([0-9]+)\\]$").FindStringSubmatch(pathItem)
	if len(matches) < 2 {
		simp_json = simp_json.Get(pathItem)
	} else {
		index, _ := strconv.Atoi(matches[1])
		simp_json = simp_json.GetIndex(index)
 	}
}

Your suggestion seems reasonable, 2 level splitting
Does something like payload.0.state work?
I would prefer to see this as a PR agains https://godoc.org/github.com/bitly/go-simplejson#Json.GetPath
(since frisby is just wrapping the simplejson.GetPath call)

payload.0.state crossed my mind but we'd need to be careful to avoid the inverse problem of not being able to address this valid structure:
{
  "payload": {
    "0": {
      "state": "test"
    }
  }
}

Using [0] felt more specific and less likely to collide, but it might be ok to just use 0 along with a check if the element is an array.
Correct on making this an issue for the upstream. Will look into it 👌

+1 for the payload.array.[0].field syntax then

Non-bracket array indexing added by #10


Support Python 3.x < 3.10

This code currently requires Python 3.10 or later, but it can work with 3.7 (3.6?) or later with only a small change. This method…
def _chunks(
    buffer: bytearray,
    size: int,
) -> collections.abc.Generator[bytearray, None, None]:

…can have return type changed to -> typing.Generator[bytearray, None, None]:. This will allow use in code that targets older versions of Python.

The most i would probably consider loosening it would be to 3.8 as 3.7 is end-of-life next month.
I guess the library as is probably compatible with 3.9 already when collections.abc got generic support, its 3.10 mostly because i was too lazy to test older versions properly.
I would consider a PR to make it 3.8 compatible - with comments to make pylint stop moaning about the depreciation of the typing classes - but thats only got a little over a year left till its EOL.
what version are you hoping for?

For my libraries I try to support the lowest possible version of Python, which currently is 3.7 as 3.6 doesn't support from __future__ import annotations. Supporting 3.8+ is reasonable.


pmtileswriter is dropping tiles

For some reason there a fewer tiles in a converted pmtiles container

nope, was the versatilesreader. I forgot a single line.


Allow Camouflaging blocks

i.e. make them look like a redstone furnace or sth else

How about something like the Conduit Facade in EnderIO.
These are pretty nifty and can be anything one wants. If you don't want an extra machine for painting you could use shapeless crafting for example...

It will be more like drawbridges with a slot for the block used as camouflage.

Or like that. Haven't used drawbridges since now so I didn't know they could do that. :)

Currently working on a 1.10.2 version, the 1.7.10 version is not being maintained anymore, sorry.
Closing all issues not relevant for the new version.


Use CircleCI 2.0 for post-build

Run post-build tasks in a separate job. Use CircleCI 2.0 configuration to only run this job on the master branch.

Why is travis blocking this PR? I haven't seen the travis check show up in quite a while.

This branch is on the main repo. Usually I push to my forked repo. Travis runs on branches here.


Can you check if you get the correct identity if you use https://guggero.github.io/cryptography-toolkit/#!/aezeed, take the xprv and use that as --rootkey for chantools?

Can you check if you get the correct identity if you use https://guggero.github.io/cryptography-toolkit/#!/aezeed, take the xprv and use that as --rootkey for chantools?
Originally posted by @guggero in https://github.com/guggero/chantools/issues/42#issuecomment-1065840533

?


Unable to load structures with /load or structure blocks

Describe the Bug
Unable to load structures with /load or structure blocks, able to do so with vanilla structures
Reproduction Steps
1.Use /load structure infernalexp:[insert any structure name here]
OR
1.Place a structure block, set to "load", then input infernalexp:[insert any structure name here]
Expected Result
Structure exists
Screenshots and Videos
No response
Operating System
Windows 10
Mod Version
2.5.1
Minecraft Version
1.19
Forge Version
1.19.2-43.1.27
Other Mods
None.
 Infernal Expansions structures are kinda picky with where they choose to spawn. They need a large enough and flat enough space to be able to spawn on. If the space you're currently trying to spawn a structure in doesn't meet these requirements it will fail spawning it.
As for the structure block, it expects a resource location to an NBT file, not a resource location to a structure. An example would be infernalexp:glowstone_canyon/heaven_4. You can see all the IE NBT files here https://github.com/infernalstudios/Infernal-Expansion/tree/1.18/src/main/resources/data/infernalexp/structures.
I'm going to close this issue now. If you want more help please just re-open it.


[Do not merge] External contributor fix

Raising this to trigger the build for Ollie.

You can preview this change here:
Built with commit 98bd31983edccd83e0b0fc1051a46be9058d0732
https://deploy-preview-257--govuk-design-system-preview.netlify.com

Weirdly does not have the commit. Closing.


Expressions: Allow escapes in quoted identifiers.

Also add tests for escaped string literals and for numeric literals.

👍

👍


repository apis to implement

[ ] list repos
[ ] create repo
[ ] delete repo
[ ] backup
[ ] dump
[ ] import
[ ] browse


Zod superRefine does not correctly revalidate the form on change

Describe the bug
The zod resolver doesn't seem to run the superRefine function correctly on form changes, it only works when changing the field defined in the path of the issue, but in this case I have errors based on the existance of data in other fields, so I expect this to work
To Reproduce
Steps to reproduce the behavior:

Go to the Codesandbox below
Write something in the "Last name" field
The "First name" field should get an error because the refine function requires first name to be defined if last name is
Writing something in the "First name" field will correctly remove the error, but removing the content in the "Last name" field will not

Codesandbox link (Required)
https://codesandbox.io/p/github/MattisAbrahamsson/rhf-zod-refine-minimal-reproduction/main
Expected behavior
I expect the resolver to revalidate my form when it changes, since reValidateMode  is set to onChange by default, and I have not opted out of it.

We have exactly the same error, we have it set that the start date must be smaller than the end date and when we set the start date later than the end date, an error appears, of course, but now if we set the end date to a later date than the start date, even though validation passes (correct behavior), the error does not disappear

Same error for us! We have a refine on a zod schema that applies an error message to the second of two form fields if the combination of both of the form fields is incorrect. We are only applying the error to the second field as that's where we would like to show the error message and autofocus on failed validation. The form does correctly pass validation if the first form field is updated, but the error on the second field never disappears. Would love to see this fixed!

Same issue here, I have an object where you need to fill in a start and end time, i have a refine on the object that checks if the start_time is before the end_time, i can throw the error on the start_time like this.
export const CreateWorkActivityRequest = object({
  start_time: date(),
  end_time: date(),
}).refine(
  (val) => {
    if (val.type !== 'travel') {
      return !isAfter(val.start_time, val.end_time)
    }
    return true
  },
  {
    message: 'Start tijd is later dan eind tijd',
    path: ['start_time']
  }
)

But i want to make use of the 'root' property in the errors so i can show the error at the end of the form like we do with server errors
setError("root.serverError", {
  type: "400",
})

but throwing this error will make the form pass but the object its sending is empty
.refine(
  (val) => {
    if (val.type !== 'travel') {
      return !isAfter(val.start_time, val.end_time)
    }
    return true
  },
  {
    message: 'Start tijd is later dan eind tijd',
    path: ['root', 'start_end_time_mismatch']
  }
)

Is there any workaround for this?

Is there any workaround for this?

I guess you can clear the errors yourself by calling form.clearErrors(fieldName); in the onChange of the element.

Up ! is there a clean way to achieve this ?

Up ! is there a clean way to achieve this ?

found a nice workaround from RHF: https://github.com/colinhacks/zod/discussions/2220#discussioncomment-5563621


compactor 0.24 shutdown on downsampling error after duplicate "mark for deletion"

Thanos, Prometheus and Golang version used:
prometheus, version 2.29.1 (branch: HEAD, revision: dcb07e8eac34b5ea37cd229545000b857f1c1637)
  build user:       root@364730518a4e
  build date:       20210811-14:48:27
  go version:       go1.16.7
  platform:         linux/amd64

thanos, version 0.24.0 (branch: HEAD, revision: 7fa248981cd98d2cd1e86b06854957d5b9a213fd)
  build user:       circleci@2fe18935fc7e
  build date:       20211222-15:58:53
  go version:       go1.16.12
  platform:         linux/amd64

Object Storage Provider:
S3 on AWS
What happened:
Compacted blocks marked for deletion and eventually deleted are processed by the downsampler  which crash with
downsampling to 5 min: download block 01FSJT5R0SZ3Z834PFCVT53M52: get file 01FSJT5R0SZ3Z834PFCVT53M52/meta.json: The specified key does not exist.
first pass of downsampling failed
main.runCompact.func7
	/home/circleci/project/cmd/thanos/compact.go:435
main.runCompact.func8.1
	/home/circleci/project/cmd/thanos/compact.go:471
github.com/thanos-io/thanos/pkg/runutil.Repeat
	/home/circleci/project/pkg/runutil/runutil.go:75
main.runCompact.func8
	/home/circleci/project/cmd/thanos/compact.go:470
github.com/oklog/run.(*Group).Run.func1
	/home/circleci/go/pkg/mod/github.com/oklog/run@v1.1.0/group.go:38
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1371
error executing compaction
main.runCompact.func8.1
	/home/circleci/project/cmd/thanos/compact.go:498
github.com/thanos-io/thanos/pkg/runutil.Repeat
	/home/circleci/project/pkg/runutil/runutil.go:75
main.runCompact.func8
	/home/circleci/project/cmd/thanos/compact.go:470
github.com/oklog/run.(*Group).Run.func1
	/home/circleci/go/pkg/mod/github.com/oklog/run@v1.1.0/group.go:38
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1371
compact command failed
main.main
	/home/circleci/project/cmd/thanos/main.go:158
runtime.main
	/usr/local/go/src/runtime/proc.go:225
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1371

and exits, despite the fact that the Thanos Compactor should halt on error.
I also see a lot of:
requested to mark for deletion, but file already exists; this should not happen; investigate

What you expected to happen:
Downsampler should skip over missing block.
Blocked marked for deletion with a default 48h delay are kept and produce this investigate message  every time AFAICT
How to reproduce it (as minimally and precisely as possible):

all retentions set to 30d
compaction set to level3
delete delay set to default of 48h

Full logs to relevant components:
{"block":"01FSJT5R0SZ3Z834PFCVT53M52","caller":"block.go:209","level":"info","msg":"block has been marked for deletion","ts":"2022-01-26T00:00:39.043050403Z"}

Followed by many of these at different times
January 26th 2022, 18:45:18.278
{"caller":"block.go:191","err":"file 01FSJT5R0SZ3Z834PFCVT53M52/deletion-mark.json already exists in bucket","level":"warn","msg":"requested to mark for deletion, but file already exists; this should not happen; investigate","ts":"2022-01-26T23:45:16.564433914Z"}

and eventually
January 27th 2022, 19:06:17.303
{"block":"01FSJT5R0SZ3Z834PFCVT53M52","caller":"blocks_cleaner.go:54","level":"info","msg":"deleted block marked for deletion","ts":"2022-01-28T00:06:17.049541254Z"}

which fatally yields to
January 27th 2022, 19:06:24.310
{"caller":"http.go:74","component":"compact","err":"error executing compaction: first pass of downsampling failed: downsampling to 5 min: download block 01FSJT5R0SZ3Z834PFCVT53M52: get file 01FSJT5R0SZ3Z834PFCVT53M52/meta.json: The specified key does not exist.","level":"info","msg":"internal server is shutting down","service":"http/server","ts":"2022-01-28T00:06:20.792128638Z"}

So the errors

first pass of downsampling failed
compact command failed
should not end with  "internal server is shutting down" because -debug.halt-on-error=false is not specified.

IIUC the doc

Hidden flag --no-debug.halt-on-error controls this behavior. If set, on halt error Compactor exits.
When not set, on halt error Compactor should keep running without doing compaction.

Anyway if should not halt in the first place if it just deleted the file it tries to downsample.

I think I understood that the is not considered a HaltError so it continues and shutdown anyway, but why not skip over missing blocks instead of error out ?


breaking change: replace output.targets with output.target

Summary

breaking change: Replace output.targets with output.target
chore: Generate rspack / webpack config based on environment instead of output.targets

Each environment has and only corresponds to one target. Multiple environments can have the same target.
one environment  = one rsbuild configuration = one rspack configuration = one build outputs.

Related Links
https://github.com/web-infra-dev/rsbuild/issues/2620

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

Please add to: https://github.com/web-infra-dev/rsbuild/discussions/2508


Feature/be more robust if db is down

Hello Rob,
I've made some changes in the connectionpool:

Connection pool will not fail hard if database is not available on initalization
DataSource alerts are only fired once in multi thread applications
Other threads are blocked, until the DataSource alert is processed completely.

Use Case:

I cannot create a default Ebean server if DB is down when my web application starts.
2+3. If DB was not available on startup, I have to run DDL scripts when DB comes up the first time (this is done in the notifyDataSourceIsUp event.

It would be great if you give me some feedback about this change.
Cheers
Roland

I think this change looks fine except that I believe / suspect some people will want "fail fast" on ensureMinimumConnections().  That is, there should probably be configuration (e.g. boolean failOnStart) such that people can set that to true.
Assuming we add a boolean failOnStart - I suspect that failOnStart should probably default to true to maintain current behavior.

Hello Rob, yes. I would prefer if you introduce a "failOnStart" - but this meant, that I also have to change datasource api.
If you plan to release a new datasource-api you may take into account to extend the DatasourceAlert interface:
void dataSourceDown(String dataSourceName);
void dataSourceUp(String dataSourceName);
void dataSourceWarning(String subject, String msg);

my suggestion:
void dataSourceDown(String dataSourceName, SQLException reason); // pass the reason why DS is down
void dataSourceUp(String dataSourceName);
void dataSourceWarning(String dataSourceName, String msg); // do not pass computed subject here

or
void dataSourceDown(DataSource dataSource, SQLException reason); // we can also pass complete datasource instead only it's name.
void dataSourceUp(DataSource dataSource);
void dataSourceWarning(DataSource dataSource, String msg);

Yes I think that all makes sense.  Yes I'm happy to update the API.
On 2 May 2017 8:51 p.m., "Roland Praml"<EMAIL_ADDRESS>wrote:

Hello Rob, yes. I would prefer if you introduce a "failOnStart" - but this
meant, that I also have to change datasource api.
If you plan to release a new datasource-api you may take into account to
extend the DatasourceAlert interface:
void dataSourceDown(String dataSourceName);void dataSourceUp(String dataSourceName);void dataSourceWarning(String subject, String msg);
my suggestion:
void dataSourceDown(String dataSourceName, SQLException reason); // pass the reason why DS is downvoid dataSourceUp(String dataSourceName);void dataSourceWarning(String dataSourceName, String msg); // do not pass computed subject here
or
void dataSourceDown(DataSource dataSource, SQLException reason); // we can also pass complete datasource instead only it's name.void dataSourceUp(DataSource dataSource);void dataSourceWarning(DataSource dataSource, String msg);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ebean-orm/avaje-datasource/pull/6#issuecomment-298562153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMwTXG1nS7WSm-AI3X2KNPtT2aF07uhks5r1u6XgaJpZM4NJb_J
.

@rbygrave Can you take look at this PR again?

This PR is no longer needed right?  The change was all contained in the other PR I believe.

PR is no longer needed. There are a few changes left. I'll create a new one.
(btw io.ebean:ebean-datasource-api is not yet at mavencentral)


reflect: Convert to unsafe.Pointer doesn't work

If you have a reflect.Value that's a pointer, and you try to call its
"Convert" method with the reflect.Type representation of the unsafe.Pointer
type, you get the error "panic: reflect.Value.Convert: value of type *int cannot be
converted to type unsafe.Pointer". That's wrong.

http://play.golang.org/p/ONG4bE_8iW

Implementing this is a security hole.  I don't think we can allow it.
With this feature, I can convert between pointer types without importing unsafe.
Example: http://play.golang.org/p/4Nz3ZoMWQ_

Note that you can always use
unsafe.Pointer(rx.Pointer())

+1 to everything Keith said.


Start cleaning up github auth routes

Move github token debug route to separate module
Use crypto.timingSafeEqual
Rename getTokenDebugInfo -> serializeDebugInfo

This picks some more of the work from #1205.

Warnings

:warning:

This PR modified the server but none of the service tests. That's okay so long as it's refactoring existing code.

:warning:

This PR modified helper functions in lib/ but not accompanying tests. That's okay so long as it's refactoring existing code.

:warning:

This PR modified services/github/admin.js but not services/github/admin.tester.js. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS


docker:24.0.4-git image is broken

We are using gitlab ci for docker in docker  and when we run this command:
git clone https://oauth2:$/{GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}
we started to get the error:
remote: HTTP Basic: Access denied. The provided password or token is incorrect or your account has 2FA enabled and you must use a personal access token instead of a password. See https://git.tiatechnology.com/help/topics/git/troubleshooting_git#error-on-git-fetch-http-basic-access-denied
It only happens in docker:git and docker:24.0.4-git, docker:24.0.5-git, docker:24.0.6-git images (reference https://hub.docker.com/_/docker/tags). So we switched to 24.0.3. After we made a switch we do not get this error anymore

There are a few details here that don't match:

neither of these images exist yet docker:24.0.5-git and docker:24.0.6-git, did you mean 23.0.5/23.0.6 and maybe 23.* in more of your testing?
did you mean this? (note the random extra slash)- https://oauth2:$/{GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}
+ https://oauth2:${GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}

If you are just doing a git clone from within the docker:git container, then I am unsure how that could be different, since the git package hasn't changed in Alpine since April: https://git.alpinelinux.org/aports/log/main/git?h=3.18-stable.
$ docker run --rm docker:24.0.3-git apk info -v | grep git
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
git-2.40.1-r0
$ docker run --rm docker:24.0.4-git apk info -v | grep git
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
git-2.40.1-r0

There is a difference in a few system package versions, but if they are the cause, then there is nothing we can change in the image. It could maybe be one of the following: a deliberate change in Alpine or upstream, or a bug in the Alpine packaging. The difference for the openssl package – openssl, libssl3, and libcrypo3 –  is just two CVE patches: https://git.alpinelinux.org/aports/log/main/openssl?h=3.18-stable.
 diff -u <(docker run --rm docker:24.0.3-git apk info -v) <(docker run --rm docker:24.0.4-git apk info -v)
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
--- /dev/fd/63  2023-07-24 10:51:22.519176530 -0700
+++ /dev/fd/62  2023-07-24 10:51:22.519176530 -0700
@@ -22,8 +22,8 @@
 libblkid-2.38.1-r8
 libc-utils-0.7.2-r5
 libcom_err-1.47.0-r2
-libcrypto3-3.1.1-r1
-libcurl-8.1.2-r0
+libcrypto3-3.1.1-r3
+libcurl-8.2.0-r0
 libedit-20221030.3.1-r1
 libexpat-2.5.0-r1
 libidn2-2.3.4-r1
@@ -31,7 +31,7 @@
 libmnl-1.0.5-r1
 libncursesw-6.4_p20230506-r0
 libnftnl-1.2.5-r1
-libssl3-3.1.1-r1
+libssl3-3.1.1-r3
 libtirpc-1.3.3-r2
 libtirpc-conf-1.3.3-r2
 libunistring-1.1-r1
@@ -46,7 +46,7 @@
 openssh-client-common-9.3_p1-r3
 openssh-client-default-9.3_p1-r3
 openssh-keygen-9.3_p1-r3
-openssl-3.1.1-r1
+openssl-3.1.1-r3
 pcre2-10.42-r1
 pigz-2.7-r3
 scanelf-1.3.7-r1

I believe I ran into the same or similar issue.
#19 ERROR: failed to prepare vzth5z938h7x0xhvyjugduq3c: invalid argument

[publish 1/1] RUN dotnet publish "Blah.Api.csproj" -c Release -o /app/publish:

WARNING: buildx: git was not found in the system. Current commit information was not captured by the build
ERROR: failed to solve: failed to solve with frontend dockerfile.v0: failed to build LLB: failed to prepare vzth5z938h7x0xhvyjugduq3c: invalid argument

There are a few details here that don't match:

neither of these images exist yet docker:24.0.5-git and docker:24.0.6-git, did you mean 23.0.5/23.0.6 and maybe 23.* in more of your testing?
did you mean this? (note the random extra slash)- https://oauth2:$/{GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}
+ https://oauth2:${GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}

If you are just doing a git clone from within the docker:git container, then I am unsure how that could be different, since the git package hasn't changed in Alpine since April: https://git.alpinelinux.org/aports/log/main/git?h=3.18-stable.
$ docker run --rm docker:24.0.3-git apk info -v | grep git
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
git-2.40.1-r0
$ docker run --rm docker:24.0.4-git apk info -v | grep git
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
git-2.40.1-r0

There is a difference in a few system package versions, but if they are the cause, then there is nothing we can change in the image. It could maybe be one of the following: a deliberate change in Alpine or upstream, or a bug in the Alpine packaging. The difference for the openssl package – openssl, libssl3, and libcrypo3 – is just two CVE patches: https://git.alpinelinux.org/aports/log/main/openssl?h=3.18-stable.
 diff -u <(docker run --rm docker:24.0.3-git apk info -v) <(docker run --rm docker:24.0.4-git apk info -v)
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory
WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/community: No such file or directory
--- /dev/fd/63  2023-07-24 10:51:22.519176530 -0700
+++ /dev/fd/62  2023-07-24 10:51:22.519176530 -0700
@@ -22,8 +22,8 @@
 libblkid-2.38.1-r8
 libc-utils-0.7.2-r5
 libcom_err-1.47.0-r2
-libcrypto3-3.1.1-r1
-libcurl-8.1.2-r0
+libcrypto3-3.1.1-r3
+libcurl-8.2.0-r0
 libedit-20221030.3.1-r1
 libexpat-2.5.0-r1
 libidn2-2.3.4-r1
@@ -31,7 +31,7 @@
 libmnl-1.0.5-r1
 libncursesw-6.4_p20230506-r0
 libnftnl-1.2.5-r1
-libssl3-3.1.1-r1
+libssl3-3.1.1-r3
 libtirpc-1.3.3-r2
 libtirpc-conf-1.3.3-r2
 libunistring-1.1-r1
@@ -46,7 +46,7 @@
 openssh-client-common-9.3_p1-r3
 openssh-client-default-9.3_p1-r3
 openssh-keygen-9.3_p1-r3
-openssl-3.1.1-r1
+openssl-3.1.1-r3
 pcre2-10.42-r1
 pigz-2.7-r3
 scanelf-1.3.7-r1

Hi, I see that 24.0.5-git does exist https://hub.docker.com/layers/library/docker/24.0.5-git/images/sha256-ed1845bc9d642f2434db56764b5a6e37b9a893bd92fdf240d1ba089fe261b898?context=explore and when I use that git fails on clone. And regarding second question this is the line which I am using in both versions docker image versions and they fail:
git clone https://oauth2:${GITLAB_TOKEN}@${CI_SERVER_HOST}/${CSW_GATEWAY_API_PROJECT_PATH}
So once again, 24.0.4-git image works for my pipeline and 24.0.5-git does not

I am still confused by the inconsistency of exactly which docker images are broken:

Title: docker:24.0.4-git image is broken

It only happens in docker:git and docker:24.0.4-git, docker:24.0.5-git, docker:24.0.6-git images [...]. So we switched to 24.0.3. After we made a switch we do not get this error anymore

So once again, 24.0.4-git image works for my pipeline and 24.0.5-git does not

You keep changing which docker versions work and which don't (with versions that didn't exist) so I can't tell if it might be a change in docker or a change in system packages included or something else entirely (like a server side/proxy change).

/ #

I don't think it is a change in the image, but for every change in the docker images, you can track what changed in the history of this directory of repo-info.  This repo scans every official image push and saves the package metadata among other things.

docker:24.0.5* images did not exist when I made the previous comment; they were merged to be built a few hours after my comment in https://github.com/docker-library/official-images/pull/15080.

Good morning, I tried to reproduce this command with docker:git latest docker image and it looks like it started to work again, so probably we will close this issue


Archival tool

Migrated from https://github.com/input-output-hk/catalyst-fund-archive-tool
Added error handling and propagation

Errors are unrelated, merging.


add is_active_in and located_in

http://noctua-dev.berkeleybop.org/editor/graph/gomodel:ZFIN_ZDB-GENE-990415-160 is not pushed out in the noctua-gpad file.
@dustine32 discovered two relations that need to be added in the SPARQL query in gpad-basic.rq: 'located_in', 'is_active_in'.
He also noticed we need to add the two relations to the PREFIX section at the top.
PREFIX located_in: http://purl.obolibrary.org/obo/RO_0001025
PREFIX is_active_in: http://purl.obolibrary.org/obo/RO_0002432

Somewhat related: #179


Add a contact_method_fields method to ALIndividual

This is another screen people may need to customize the intro text of depending on who the recipient of the form is

E.g., https://github.com/SuffolkLITLab/docassemble-RentalRepairLetter

Wait for https://github.com/SuffolkLITLab/docassemble-AssemblyLine/pull/760 to be merged


[BUG]When fine-tuning an LLM, the following error occurs after training for some time: self.optimizer.param_groups[param_group_id]['params'] = [] IndexError: list index out of range

File "/share/home/xxx/miniconda3/envs/qwen2_vl_FT/lib/python3.10/site-packages/deepspeed/runtime/zero/stage3.py", line 2095, in step
[rank2]:     self._optimizer_step(sub_group_id)
[rank2]:   File "/share/home/xxx/miniconda3/envs/qwen2_vl_FT/lib/python3.10/site-packages/deepspeed/runtime/zero/stage3.py", line 980, in _optimizer_step
[rank2]:     self.optimizer.param_groups[param_group_id]['params'] = []
[rank2]: IndexError: list index out of range

Can you provide reproduction code?

Can you provide reproduction code?

I am fine-tuning the LLM using LLaMA Factory.


question: is it still recommended to use "trigger" not "delegate" to bind event handler?

I remember I read something in old dialog doc, but could not find relevant information on current aurelia doc hub.
I tested click.delegate, submit.delegate inside a dialog, all seems working fine.
Is is still recommended to use "trigger" for event handler? Does "delegate" break dialog behaviour on certain browser, or under certain configuration?
Thanks.

So if we switch the examples from trigger to delegate does that mean that we are not recommending trigger?
As I said, I do not remember nor could find, an explicit warning about the use of delegate. All recommendations regarding Binding: Delegate vs Trigger do apply.

Use whichever works best for you there is no recommendation to use either.


include spring security  in example.

I'm about 10 billion kinds of frustrated with the documentation right now. Official and especially this guide. So let's start with this guide. It doesn't include how to make it work with spring security. I know spring security isn't required for CORS, but I think it's probably at least an 80% of the time use case. This still doesn't give me proper CORS headers.
I honestly think this is beyond a problem with just documentation though... but that's a good place to start.
// © Copyright 2024 Caleb Cushing
// SPDX-License-Identifier: AGPL-3.0-or-later

package com.xenoterracide.controller.authn;

import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.springframework.boot.SpringApplication;
import org.springframework.boot.actuate.web.exchanges.HttpExchangeRepository;
import org.springframework.boot.actuate.web.exchanges.InMemoryHttpExchangeRepository;
import org.springframework.boot.autoconfigure.SpringBootApplication;
import org.springframework.context.annotation.Bean;
import org.springframework.lang.NonNull;
import org.springframework.lang.Nullable;
import org.springframework.security.config.Customizer;
import org.springframework.security.config.annotation.web.builders.HttpSecurity;
import org.springframework.security.core.Authentication;
import org.springframework.security.web.SecurityFilterChain;
import org.springframework.web.bind.annotation.CrossOrigin;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RestController;
import org.springframework.web.cors.CorsConfiguration;
import org.springframework.web.cors.CorsConfigurationSource;
import org.springframework.web.cors.UrlBasedCorsConfigurationSource;
import org.springframework.web.servlet.config.annotation.CorsRegistry;
import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;

@SpringBootApplication
public class ResourceServer {

  ResourceServer() {}

  public static void main(String[] args) {
    SpringApplication.run(ResourceServer.class, args);
  }

  @Bean
  SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
    http
      .authorizeHttpRequests(auth -> auth.anyRequest().permitAll())
      .oauth2ResourceServer(oauth2 -> oauth2.jwt(Customizer.withDefaults()))
      .cors(Customizer.withDefaults())
      .httpBasic(c -> c.disable())
      .formLogin(f -> f.disable());
    return http.build();
  }

  @Bean
  HttpExchangeRepository httpExchangeRepository() {
    return new InMemoryHttpExchangeRepository();
  }

  @Bean
  CorsConfigurationSource corsConfigurationSource() {
    var cors = new CorsConfiguration();
    cors.addAllowedOrigin("http://localhost:3000");
    cors.addAllowedMethod("*");
    cors.addAllowedHeader("*");
    var source = new UrlBasedCorsConfigurationSource();
    source.registerCorsConfiguration("/**", cors);
    return source;
  }

  @Bean
  WebMvcConfigurer corsConfigurer() {
    return new WebMvcConfigurer() {
      @Override
      public void addCorsMappings(CorsRegistry registry) {
        registry.addMapping("/api/external").allowedOrigins("http://localhost:3000");
      }
    };
  }

  @RestController
  static class OidcTestController {

    private final Logger log = LogManager.getLogger(this.getClass());

    @CrossOrigin(originPatterns = "*")
    @GetMapping("/api/external")
    @NonNull
    String index(@Nullable Authentication details) {
      this.log.info("{}", details);
      var name = details != null ? details.getName() : "world";
      return "Hello, " + name;
    }
  }
}

* Host localhost:8080 was resolved.
* IPv6: ::1
* IPv4: <IP_ADDRESS>
*   Trying [::1]:8080...
* Connected to localhost (::1) port 8080
> GET /api/external HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/8.6.0
> Accept: */*
>
< HTTP/1.1 200
< Vary: Origin
< Vary: Access-Control-Request-Method
< Vary: Access-Control-Request-Headers
< X-Content-Type-Options: nosniff
< X-XSS-Protection: 0
< Cache-Control: no-cache, no-store, max-age=0, must-revalidate
< Pragma: no-cache
< Expires: 0
< X-Frame-Options: DENY
< Content-Type: text/plain;charset=UTF-8
< Content-Length: 12
< Date: Fri, 05 Apr 2024 10:13:21 GMT
<
{ [12 bytes data]
* Connection #0 to host localhost left intact
Hello, world

I found out that a big part of my problem is "testing" apparently there's a hidden undocumented optimization that prevents the headers from being written if the expected headers aren't in the request. So it would be good in addition to showing the expected spring security config to include how to test it within a junit test. That would show that you need headers.


Add Vulcan

Project URL
https://github.com/jinSasaki/Vulcan
Description
Multi image downloader with priority in Swift
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 1 link issue

Link issue by awesome_bot

Line
Status
Link

2084
301
https://buddybuild.com/ redirects tohttps://www.buddybuild.com/

  Generated by :no_entry_sign: danger

Thank you for contributing to awesome-ios 😄


MD003: Settings in .markdownlint.json do not match parameter values

Per the documentation, the values for MD003 are:

Parameters: style ("consistent", "atx", "atx_closed", "setext", "setext_with_atx", "setext_with_atx_closed"; default "consistent")

However, when trying to add a value for MD003 in the .markdownlist.json file, it notifies the user that it should be a boolean value.
Whether the value is marked as false or "setext_with_atx", having mixed headings still show as an error.
Expected:

The .markdownlint.json file should prompt for the correct values for MD003
The value of MD003 should be respected by the interpreter

I am guessing you are doing this in the VS Code extension? If so, it sounds like the wrong syntax for specifying a perimeter value is being used. Here is an example of how to apply the setting you want:
https://github.com/DavidAnson/markdownlint/blob/2710c375b3c9b9b6531d546b667f50f7d0c956f2/test/headings_good_setext_with_atx.json
Please let me know if that doesn’t help -  and include an example of the syntax you are using.

Correct, I'm using the VSC extension. I followed the format from the link and it works. I was just setting the key directly as "setext_with_atx" instead of as an object.
Is there a reason why the tool tip says it needs to be a boolean value and not an object?

It’s because this syntax is valid to disable (or enable) the rule without modifying its parameters:
https://github.com/DavidAnson/markdownlint/blob/2710c375b3c9b9b6531d546b667f50f7d0c956f2/test/headings_with_spaces_at_the_beginning.json

Once you create an object (instead of a Boolean), VS Code should prompt you with the valid parameters and values.

Awesome.. Thank you for the clarification.


feat(CheatCode): prank(who)

Let the dev call cheatcodes.prank(who) and replace the msg.sender of the next execution with who. Essentially, this allows you to impersonate another contract wallet.

Closed, see #1259 for more information.


Create README.md

Added a readme file.

Working some coding exercises and thought I'd add a readme.

User submitted readme


[Mobile] the button "Scroll down" doesn't disappear for Dall-E on mobile

EPAM AI DIAL chat version
0.8.0
How to reproduce

open DIAL chat on mobile
request picture on Dall-E
expand generated picture (p.1)
scroll down to see picture
collapse picture (p.2)

Actual result
p.1: wasn't scrolled down to see picture
p.2: the button "Scroll down" doesn't disappear
Expected result
p.1: should be scrolled down automatically
p.2: the button "Scroll down" should disappear
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging env successfully

Verified on sales+chrome mobile view sucecssfully


On macOS the extension defaults to system interpreter, which is not recommended

Environment data

VS Code version: 1.25.1
Extension version (available under the Extensions sidebar): 2018-7.0-beta
OS and version: macOS high sierra
Python version (& distribution if applicable, e.g. Anaconda): 2.7.10
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): NA
Relevant/affected Python packages and their versions: NA

Actual behavior
When opening a new folder on macOS the python version it selects by default is the system interpreter:

Previously we used to warn users that the system version was not supported.
Installing PyLint in this interpreter also fails.
Expected  behavior
We warn users to install python and link them to our getting started tutorial which has the homebrew instructions.
Steps to reproduce:

Open a new workspace with VS Code on macOS

This was reported previously, looks like no action was taken to address this https://github.com/Microsoft/vscode-python/issues/1187

I was able to verify that the default interpreter is /usr/bin/python2.7.

@ericsnowcurrently

are you using a Mac?
Did you install any other versions of Python in there?


Uncaught Error: Call to undefined method DedexBundle\Simplifiers\SimpleAlbum::getTracks()

Hello, When I use the following code example, I get the Uncaught Error: Call to undefined method DedexBundle\Simplifiers\SimpleAlbum::getTracks()
use DedexBundle\Controller\ErnParserController;
use DedexBundle\Simplifiers\SimpleAlbum;

$xml_path = "tests/samples/with_assets/004_complete/1199119911991.xml";
$parser = new ErnParserController();
$ern = $parser->parse($xml_path);
$album = new SimpleAlbum($ern);

// Then you can access high level property of this album
$release_date = $album->getOriginalReleaseDate();
$artists_at_album_level = $album->getArtists();
$tracks_for_cd_1 = $album->getTracks()[1];

I checked SimpleAlbum class/file, there is no getTracks() method
Thanks

Will check that soon @nleonar25. Thank you for your interest.

Hello @nleonar25. I just pushed a fix on master that updates the documentation.
The getTracks method has been replaced with getTracksPerCd but I forgot to update the documentation.
Working sample is:
use DedexBundle\Controller\ErnParserController;
use DedexBundle\Simplifiers\SimpleAlbum;

$xml_path = "tests/samples/with_assets/004_complete/1199119911991.xml";
$parser = new ErnParserController();
$ern = $parser->parse($xml_path);
$album = new SimpleAlbum($ern);

// Then you can access high level property of this album
$release_date = $album->getOriginalReleaseDate();
$artists_at_album_level = $album->getArtists();
$tracks_for_cd_1 = $album->getTracksPerCd()[1];

// You can also access exact XML mapping by getting
// ddex objects at album or track level
$ddex_release = $album->getDdexRelease();
$ddex_release->getReleaseId()[0]->getICPN()->value(); // direct access
$ddex_track = $tracks_for_cd_1[1]->getDdexSoundRecording()->getDuration(); // direct access

Thank you for the update and thank you for putting this library together.
It has saved me so much time.
On Sat, Apr 9, 2022, 4:11 AM Mickaël A. @.***> wrote:

Hello @nleonar25 https://github.com/nleonar25. I just pushed a fix on
master that updates the documentation.
The getTracks method has been replaced with getTracksPerCd but I forgot
to update the documentation.
Working sample is:
use DedexBundle\Controller\ErnParserController;use DedexBundle\Simplifiers\SimpleAlbum;
$xml_path = "tests/samples/with_assets/004_complete/1199119911991.xml";$parser = new ErnParserController();$ern = $parser->parse($xml_path);$album = new SimpleAlbum($ern);
// Then you can access high level property of this album$release_date = $album->getOriginalReleaseDate();$artists_at_album_level = $album->getArtists();$tracks_for_cd_1 = $album->getTracksPerCd()[1];
// You can also access exact XML mapping by getting // ddex objects at album or track level$ddex_release = $album->getDdexRelease();$ddex_release->getReleaseId()[0]->getICPN()->value(); // direct access$ddex_track = $tracks_for_cd_1[1]->getDdexSoundRecording()->getDuration(); // direct access
—
Reply to this email directly, view it on GitHub
https://github.com/miqwit/dedex/issues/11#issuecomment-1093834642, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLUIFOY33MFVNL6S36MOW3VEFCTZANCNFSM5RMB2S5A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Examples broken

Hey Kyle, Harley from Pachama! Seems like our time together was too early and we're only just looping around to COG integration now (we had a huge piece of work getting our datasets COG ready).
I'm just starting to look at how we can use this awesome package and I noticed the examples were broken. It seems there's a CORS issue (or possibly missing data issue) that has broken them:

👋  All these examples were running on my personal AWS account. I shut all my services down after someone scraped this and I received an $850 AWS bill: https://twitter.com/kylebarron2/status/1412494170107920389


OAIHarvest: stuck when trying to harvest

For some estrange reason after deploying 3d60603e08d0a4e95e79d70d37bf9107531be30a on CDS, specially the modification done on modules/oaiharvest/lib/oai_harvest_getter.py, OAIHarvest gets stuck on source arXiv is going to be updated.
There is no error on the logs or any kind of output.
cc @aw-bib

There was an issue just discovered and fixed by @jalavik with some greedy/nongreedy regexp in removing namespaces in https://github.com/inveniosoftware/invenio/pull/3297 and https://github.com/inveniosoftware/invenio/pull/3298. Same for CDS?

@egabancho Yeah, I believe this problem with resumptionToken is causing the stuck forever issue. I've seen on INSPIRE that #3298 fixes the problem.

Great, I will test it asap and will let you know

It works smoothly for us :+1:


how to extract the PCs and UMAP coordinates after running CHOIR? Misc slots are null

thank you for the nice package,
i managed to run it-
however i would like to extract the PCs and UMAPs coordinates which CHOIR runs to store in the reduction slots of the seurat object.
i tried Misc(data.integrated, slot=Integrated) and Misc(data.integrated, slot=RNA) which you mentioned in the tutorial but everything is NULL.
here's my code snippet before running CHOIR, i stored it in integrated data slot
DefaultAssay(data.integrated) <- "integrated"
library(CHOIR)
data.integrated = CHOIR(data.integrated , n_cores = 2)

thanks

Hi!
For Seurat objects, try extracting the 'misc' data using the code data.integrated@misc$CHOIR instead. The Misc() function doesn't work for this.
The CHOIR-computed PCA reductions will be under data.integrated@misc$CHOIR$reduction. The primary PCA reduction can also be found under data.integrated@reductions$CHOIR_P0_reduction.
CHOIR does not automatically compute UMAP reductions. To do so, please run the runCHOIRumap() function.
Thanks for trying CHOIR!

thanks! i got it


Broken URL for Aberdeen City Council

Broken URL: #https://spatialdata-accabdn.opendata.arcgis.com/api/feed/dcat-ap/2.0.1.json

Automatically closed due to URL now working.


cleanup: remove global CSIInstanceID and pass configured value instead

There has been some confusion about using different variables for the
InstanceID of the RBD-driver. By removing the global variable
CSIInstanceID, there should be no confusion anymore what variable to
use.

@mergifyio queue


Description for issues.ref_closing_from ambiguous and incorrectly localized

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

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

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

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

Log gist:

Description
After a pull request is added by a user and linked to an issue, the following reference is then described in the given format:

user referenced a pull request that will close this issue time of reference

A specific example would be:

qwertfisch referenced a pull request that will close this issue 1 week ago

Ways of reading

The meaning of it, as I understand, is that a user has referenced a pull request 1 week ago, and this pull request will (at some other time in the future) close this issue.
But the actual reading tells me that a user referenced a pull request, and this pull request will close this issue 1 week ago (which doesn’t make any sense, at least for me).

This is quite ambiguous. Even more so in German, which clearly is translated unambiguously by the second form (considering the position of the comma and the date inside the subordinate clause).

qwertfisch hat auf einen Pull Request verwiesen, welcher das Issue vor 1 Woche schließen wird

Unfortunately English does not enforce commas and also puts the date at the end of the sentence.
Screenshots

Possible solution
I am not sure if this is a definitive bug or just an ambiguous way of reading (for English). At least the German translation should be changed to not represent the second form semantically.
An umambiguous reading could be something like this:

qwertfisch has referenced a pull request 1 week ago, that will close this issue

This way only the part “has referenced a pull request” can be an href link, not the text “that will close this issue” after the date. This behaviour is already present at the incorrect German translation, where “schließen wird” (after the date) is just text, not an href.

solution: fix template to: add a check if time is in the past ... if true, use new translation strings <- has to be added

The date will always be in the past because it references the date when the pull request was added.

oh missed that ... then it's easy: fix the english translation by creating a pull and the rest is done via crowdin

Rather than changing the ordering of the words, I think a good solution would be maybe adding a hyphen before the time of reference.
i.e.

user referenced a pull request that will close this issue – time of reference
qwertfisch referenced a pull request that will close this issue – 1 week ago

But then, this will mean that many actions that have the time reference at the end would need an update, not just the reference to the pull request. Does this sound like something feasible? I can do the PR to address this. Just want to know this sounds like the correct decision (cc. @6543)

I think adding an em-dash — (U+2014) (not hyphen) would be a good idea

I've been thinking the same thing as @ivanvc. The English version is ambiguous, yes, but not a bad wording per se. Visually setting off the date string rather than incorporating it into the sentence looks to me like a good way to resolve the ambiguity and also make sure that date strings can be at the end of the message in all languages alike.
I have also suggested a correction of the faulty German translation on Crowdin, which still needs to be reviewed there. Let's see how quick they are. ;)

I'm hesitating about this change... If the localization has an em-dash, then it will be inconsistent with the text from other events on the issues and pull requests pages.
It immediately looks inconsistent with the header of the comment:

Perhaps a better solution would be to standardize the colors from the strings in the actions section with the rest of them? I think it looks fine and helps differentating the timestamp from the actual text, i.e.:

And the rest of the events even in issues could benefit from this:

I'm hesitating about this change... If the localization has an em-dash, then it will be inconsistent with the text from other events on the issues and pull requests pages.

True, if we only change it for this message.
But what if we apply this to all event texts?

That was what I initially was doing, changing it from all the event texts. However, I noticed that it looked somewhat incosistent with the text from the section headers. I guess there are two improvements here. One is to standardize the style across those texts, and another is to add the dash to the localization files.

In most cases as in „added 2 commits 22 hours ago“ the timestamp at the end of the sentence looks good and is sementically correct. It works well in English, whereas e.g. in German the timestamp always comes before the participle. The ambiguity in English for this specific case might just be kept. If one has understood the message once, one will most likely not misread on further occurences.
Also I did not intent to start a general discussion about wording and structure. My initial report was just to confirm I understood the meaning correctly, and that the German translation should be fixed.
@eneuschild Is there an accessible link on Crowdin to the proposed German translation? I did not post my translation in this issue yet and would just move the timestamp after the main verb:

qwertfisch hat vor einer Woche auf einen Pull Request verwiesen, welcher das Issue schließen wird

@qwertfisch The link is https://crowdin.com/translate/gitea/68/enus-de?filter=basic&value=0#74642, but it requires login.
My suggestion on crowdin was to move the timestamp to the end:

qwertfisch hat auf einen Pull Request verwiesen, welcher das Issue schließen wird vor einer Woche

Now that I look at it in a proper sentence, that looks terrible. Your way is clearly better for German, but there's still the timestamp-at-the-end consistency issue. I'm unsure which way to go.

As I said previously, the timestamp is in many German translations already not at the end, because of the language itself. I see no reason why to start with awkwardly sounding sentences now.
Would you change your suggestion on Crowdin or how does this work?

Would you change your suggestion on Crowdin

Yes, I agree. Done.


refactor WcfLogReceiverClientFacade, fix backwardscomp

Fix backwardscompatiblity with 3.2
The change in NLog 4.0 was unwanted breaking. We need to fix this:
The problem:

WcfLogReceiverClientFacade won't inherit from ClientBase<...>. But in 3.2.0 WcfLogReceiverClient was returned, and this class did implement  ClientBase<...> (which contains 'EndpointandClientCredentailsproperties). Thus: WcfLogReceiverClientFacade  should inherit fromClientBase<..>`. But this is difficult to fix. We have to find a good solution for this.

See: #783 and #561
Refactor or remove WcfLogReceiverClientFacade
The class WcfLogReceiverClientFacade introduced in 4.0 (#565), contains unwanted stuff:

there is a lot of code duplication in the constructors
every property is implemented with the same if-else constructions
there are two properties, m_twoWayClient and m_oneWayClient and there is always only one used, hence, it should be one property.
there are some Hungarian names. e.g. m_oneWayClient should be without the prefix m_.
ILogReceiverClient and ILogReceiverOneWayClient are almost identical
WcfLogReceiverClient and WcfLogReceiverOneWayClient are code duplication?

Solutions:

We should clean this up and create one factory method with do the   if (useOneWay) check. It should return an interface and only that interface should be used in the rest of the class.
Clean up code duplication as much as possible.

code: https://github.com/NLog/NLog/blob/master/src/NLog/LogReceiverService/WcfLogReceiverClientFacade.cs

code introduced by  #565

I just finished doing it in my fork. Can you take a look? I'm not sure how things should be named. I just used "V2" but I'll change it to something else if you want to.
I restarted from the original code and added the method to the client interface. For the server interface, I inherited the original interface and added the new method. The target kept its parameter to indicate which method to use.
There is however one possibility in which the changes could break old code, it is if someone was inheriting directly the client interface, they would have to implement the new method for the code to compile. I tried to make it work with two client interfaces, but I get errors with the .config file that I have not managed to fix yet.

Great!! Just make a PR please. That's easier to discuss.

fixed by #874


Usability testing and feedback for timeline density: resource labels

Once prototype for activity timeline rendering and packing is in place, we should run some usability tests to understand gaps and opportunities

Summary:

SCET (UTC) and SCET (PST)
Confirm no risk of units getting truncated (once implemented)
Consider e6 (vs M) for number abbreviations
Consider ability to toggle between abbreviation and full number
To differentiate between line plots rows on top of one another, consider slightly darker gray line and/or subtle alternating background color
Request left in chat: these improvements are great! next request for timeline visibility in an activity row: can we collapse decompositions to just the parent level? or to just parent and children but not grand-children/great-grand-children?


User role homescreen

@jkleinsc this is for review, not necessarily for merging. The idea here is to overcome the feature issue (and therefore usability concern) of users logging into a back “what do you want to do?” screen. Let’s discuss.

@jkleinsc can you review this PR for the purposes of getting into 1.0?

@jkleinsc I think I've satisfied the requested changes. Happy to discuss the why's Monday.

@jkleinsc All 16 acceptance tests pass for this locally (http://localhost:4200/tests?filter=Testing User Role homescreen). Travis appears to be timing out with a few of these. What shall we do?