Array / Loops not working

Issue Description:
Arrays / Loops don't work and produce severe errors
What happens:
Array / Loop with items, strings or integers results in errors
What you expected to happen:
A tutorial code that does not produce errors
Script used (Please Pastebin or gist your script, posting an unpasted or ungist'd script will automatically close this issue):
Item: https://pastebin.com/dwdyaxz7
Copy & Paste code from your Array/Loop example page.
String script: https://pastebin.com/7MmeWMi0
Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue):
Item log: https://pastebin.com/PRiwhhr0
String log: https://pastebin.com/4t7ysEPT

Affected Versions (Do not use "latest"):

Minecraft: 1.12.2 / 1.12.1
Forge: <IP_ADDRESS>2 / <IP_ADDRESS>1 / <IP_ADDRESS>5 /
Crafttweaker: 4.08

Could I please gets arrays.zs in its full entirety

That is the full script in pastebin. I already removed everything else and still get these errors...

Updated the logfiles in case i may have uploaded logfiles before stripping the script file to the bare minimum.
Also updated some more Forge versions i tested, all with the same error as result.

So, any idea on this ?
Bug, old example code or some weird non reproduceable bug on my side ?

Try importing the classes and casting the types of the arrays

What classes exactly and what means "casting the types of arrays" ?
As i said, it's just the tutorial code from http://crafttweaker.readthedocs.io/en/latest/#AdvancedFunctions/Arrays_and_Loops/
and you have neither class imports nor anything else in there in the code so could you please give a working complete example script  on how to define an array correctly ?

Import crafttweaker.item.IItemStack;

val itemArray = [<minecraft:coal>,<minecraft:wood>] as IItemStack[];

for item in itemArray{
print(item.name);
}

Dunno if it works, am currently about 300km away from my computer ^^

Nope, doesn't work: Error parsing test.zs:1 -- ; expected
If i follow a similar example code from http://crafttweaker.readthedocs.io/en/latest/#Tips_Tricks/General/Loops_Readability/
that script works. So it indeed seems that arrays need an import and an "as ..." definition after the array itself.
The question is now: What do i need to import for a string array ?

String is an always imported thing, so nothing.
On Oct 18, 2017 4:09 PM, "Thyriel"<EMAIL_ADDRESS>wrote:

Nope, doesn't work: Error parsing test.zs:1 -- ; expected
If i follow a similar example code from http://crafttweaker.
readthedocs.io/en/latest/#Tips_Tricks/General/Loops_Readability/
that script works. So it indeed seems that arrays need an import and an
"as ..." definition after the array itself.
The question is now: What do i need to import for a string array ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jaredlll08/CraftTweaker/issues/350#issuecomment-337603663,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFS9xRAzGMiN5lKzzy_DpnkaxpmYmnQtks5stgaSgaJpZM4P8EAQ
.

Found it... Your example code is just garbage...
It's NOT "val stringArray = ["Hello", "World"];" as written at http://crafttweaker.readthedocs.io/en/latest/#AdvancedFunctions/Arrays_and_Loops/
It's: "val stringArray = ["Hello", "World"] as string[];"
...

Instead of bashing the docs, why not make a PR to fix them? https://github.com/jaredlll08/CraftTweaker-Documentation

Also it appears the issue has been solved, so I'm closing this.

Sorry but something i just had to bash after spending hours to solve a problem that i'm sure you could have solved within a second if you would have looked at the code ;)

Pull Request, it is you helping out essentially


fix def of secondary energy carrier

Description of the issue
The def of secondary energy carrier  is currently
A secondary energy carrier is an energy carrier that has the disposition primary energy carrier disposition.
Ideas of solution
Change def to:
A secondary energy carrier is an energy carrier that has the disposition secondary energy carrier disposition.
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

That's an obvious error and should be fixed quickly.


bugfix: use the stable image ID in test

Signed-off-by: Wei Fu<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
The busybox:1.28 has been repushed so that the imageID will be changed.
In order to make the test stable, we should use the stable image.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
NONE
Ⅳ. Describe how to verify it
NONE
Ⅴ. Special notes for reviews

LGTM

Codecov Report

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

@@             Coverage Diff             @@
##           master    #1397       +/-   ##
===========================================
+ Coverage   16.28%   38.55%   +22.27%
===========================================
  Files         198      248       +50
  Lines       13470    16516     +3046
===========================================
+ Hits         2193     6368     +4175
+ Misses      11126     9328     -1798
- Partials      151      820      +669

Impacted Files
Coverage Δ

cri/stream/remotecommand/httpstream.go
0% <0%> (ø)

storage/volume/error/errors.go
28.57% <0%> (ø)

cri/stream/portforward/portforward.go
0% <0%> (ø)

storage/volume/modules/local/local.go
68.57% <0%> (ø)

pkg/system/network.go
70% <0%> (ø)

main.go
59.85% <0%> (ø)

pkg/debug/debug.go
22.22% <0%> (ø)

internal/generator.go
100% <0%> (ø)

lxcfs/lxcfs.go
55.55% <0%> (ø)

cri/stream/remotecommand/errors.go
0% <0%> (ø)

... and 112 more


For remote deployments will there ever be a possibility of application via email attachments?

Our remote notebook users are heavy users of LTE services which are dependant on APN application.
Will it ever be possible to provide our staff provisioning packages that can be applied via an attachment?
I appreciate there may be security concerns with this, but this is just as much as concern of our remote users plugging in a random USB to apply the same package we supply to them<EMAIL_ADDRESS>
Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 130f5d75-4b2b-5d1f-3505-6a5fb9cb00b8
Version Independent ID: c594739e-bc96-720b-a667-1a62601bf49a
Content: Use Windows Configuration Designer to configure Windows 10 Mobile devices (Windows 10)
Content Source: windows/configuration/mobile-devices/provisioning-configure-mobile.md
Product: w10
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

@Nick-Shortal - thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback on how to make Windows 10 better. They keep a close tab on user feedback and it would be awesome if you could add your voice in User Voice https://support.microsoft.com/en-za/help/4021566/windows-10-send-feedback-to-microsoft-with-feedback-hub
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
Thank you for your contribution to make the docs better! Much appreciated!

@AndreaBarr - This issue can be closed. Thank you.


make type variable more flexible do declare generic functions

Suggestion

🔍 Search Terms

type parameters, generic types, type variable usage, etc.
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
I'd like the behaviour of type variables to change a bit to make it more flexible to declare generic functions. Example below.
📃 Motivating Example
It should be possible to do something like this:
const justEcho = <T>(...arg: T): typeof arg => arg;

<[string, number]>justEcho("hi", 4);

But it gives an error saying A rest parameter must be of an array type.ts(2370).
💻 Use Cases

Generic usage.

I am sorry. I just found out that it is in fact possible to achieve the behaviour I desired. I used the error message to search for the solution. Below is the respective code:
const justEcho = <T extends unknown[]>(...arg: T): typeof arg => arg;

<[string, number]>justEcho("hi", 4);


Dr-861 Domain Manager

Changes:

Add Test.
Add Service, Controller and view to add Domain Manager.

Reminder: This it's a WIP story.
Pending:

Confirm Backend API (I think that maybe i'll have to handle in a different way the response by the backend, but i'm not sure yet)
Change the fake call to the real api call in the setting's service.
Apply style/design/ux
More tests

Reminder: This it's a WIP story.

Could you detail what things are missed?

I think that maybe i'll have to handle in a different way the response by the backend, but i'm not sure yet. And also later i'll have to change the fake call to the real api call in the setting's service.

It seems that it is more mature than I though, nice! I will review it in deeper later.

The values are being translated in the UI by the "translate" method.

The values are being translated in the UI by the "translate" method.
👍

❤

❤ ❤ ❤

Lovely work @mvillaseco !


Default upload host parameter in config/admin.php

The default value provided in the config is http://localhost:8000/upload/, which may not work for everyone, if host or port number
A safer value would be url('upload')
Example snippet of the relevant section in the config file:
    'upload'  => [

        'disk' => 'admin',

        'directory'  => [
            'image'  => 'image',
            'file'   => 'file',
        ],

        'host' => url('upload'),
    ],

This issue is more for convenience, but another reported issue is similar:
https://github.com/z-song/laravel-admin/issues/992
(about the lack of helper function usage for urls)

Small detail, the disks in the config/filesystems.php file, can use a url attribute.
Maybe the disk defined in config/admin.php could have also the url attribute instead of defining host. Then you can get the image url with Storage::('admin')::url('image/avatar-filename.jpg )`

I'm sorry, I just discovered I was wrong about something.
The url() helper method (and some other) cannot be used in the config files. The philosophy behind it being config must be stateless or static. Some discussion about this:
https://github.com/laravel/framework/issues/7671
In this discussion, a proposed way is to use the url() helper in a provider register() method. I used it like so in my AppServiceProvider :
public function register()
{
    config([
        'admin.upload.host' => url(config('admin.upload.host')),
        'filesystems.disks.public.url' =>  url(config('filesystems.disks.public.url')),
        'filesystems.disks.admin.url' =>  url(config('filesystems.disks.admin.url')),
    ]);
}

and left only the path portion in the config that I want to use in the url() helper function.

I try your register() method on AppServiceProvider, but it seem that at that time the route() helper still has not loaded yet

I was using the url() helper, so maybe it's different with route().
What you can try is call route() in a new provider, declared after the RouteServiceProvider in config/app.php.


Controller name returned on macOS does not match other platforms

Tested versions

Reproducible in 4.3. Introduced with commit 07313a08f41146e30005acfa784bdf005d23750b

System information
Godot v4.3.stable - macOS 14.6.1 - Vulkan (Forward+) - integrated Apple M1 Max - Apple M1 Max (10 Threads)
Issue description
With the switch to using GCController, the controller name returned no longer matches other platforms. For instance, "PS5 Controller" becomes "DualSense Wireless Controller". This causes a problem when using this value in GDScript to determine controller icon sets across platforms: checking for an additional value on Mac doesn't seem to be a robust solution.
Unfortunately, there doesn't seem to be a way to associate an IOHIDDevice with GCController: so a robust code solution doesn't appear to be possible. SDL works around this by doing string compares and translating the GCController vendorId to the original ids (eg. 0x0ce6 (PS5 controller) 0x054c (Sony)). What I wonder is - should the same thing be done in Godot, or should the differences be handled in the game code?
Some notes:
This is an entry point to seeing how SDL handles this:
https://github.com/libsdl-org/SDL/blob/6e885d96193a4b0096fe7fed6d4e6c3e5f247283/src/joystick/apple/SDL_mfijoystick.m#L387
Here's a stack overflow detailing how you could tell if a IOHIDDevice would be handled by GCController, but I was unable to get the method mentioned to get a valid vendor or product id. I suspect the underlying API has changed, and this is not part of the exposed interface anyway - so even it worked, I suspect it might be risky from an app store review perspective.
https://stackoverflow.com/questions/33509296/supporting-both-gccontroller-and-iohiddeviceref
Steps to reproduce
Use var controller_name := Input.get_joy_name(device), and compare the results from other platforms with that on macOS.
Minimal reproduction project (MRP)
It's really just the returned value from Input.get_joy_name(device).

I suspect this has to do with Apple being the middleman in controller support, whereas on Windows you have 3rd party drivers that are supposedly consistent. I made a workaround for this in my own input manager:
func _deduce_controller_type_from_name(controller_name: String) -> ControllerType:
	if controller_name.contains("XInput") or\
			controller_name.contains("Xbox"):
		return ControllerType.XBOX
	elif controller_name.contains("Sony") or\
			controller_name.contains("Playstation") or\
			controller_name.contains("PS4") or\
			controller_name.contains("PS5") or \
			controller_name.contains("DualSense"):
		return ControllerType.PLAYSTATION
	elif controller_name.contains("Joy-Con"):
		if controller_name.contains("(L)"):
			return ControllerType.SWITCH_JOYCON_LEFT
		else:
			return ControllerType.SWITCH_JOYCON_RIGHT
	elif controller_name.contains("Switch"):
		return ControllerType.SWITCH
	elif SteamManager.is_on_steam_deck or controller_name.contains("Steam"):
		return ControllerType.STEAM_DECK

	return ControllerType.GENERIC

Yes - that's the workable approach at the moment. The issue is more a question of if that should be done in the engine code instead (like in SDL2). I'm happy to open a PR to do it at the engine level if that's deemed to be the correct approach, but also totally understand if it should be considered non-portable.

The workaround I posted above somehow no longer works?! The names I'm getting now are "Controller" and "Wireless Controller", making it impossible to determine the type of controller! Uggghhh. Will make a new issue.

Which version macOS and Godot are you using? The method used in the link above (https://github.com/rsubtil/controller_icons/issues/83) works for me with macOS 15.0.1, and Godot 4.3 stable.

That's the weird thing: I am using Godot v4.3.stable on macOS 15.0.1.


Too big names

If programming language is too big, it would cause that bug

I meet the same problem. It seems to be a trouble for fixing.

Closed as fixed in #81.

Closed as fixed in #81.


Activating Open Collective

This pull request adds backers and sponsors from your Open Collective https://opencollective.com/oni ❤
It adds two badges at the top to show the latest number of backers and sponsors. It also adds placeholders so that the avatar/logo of new backers/sponsors can automatically be shown without having to update your README.md. [more info]. See how it looks on this repo.
We have also added a postinstall script to let people know after npm|yarn install that you are welcoming donations (optional). [More info]
You can also add a "Donate" button to your website and automatically show your backers and sponsors there with our widgets. Have a look here: https://opencollective.com/widgets
P.S: As with any pull request, feel free to comment or suggest changes. The only thing "required" are the placeholders on the README because we believe it's important to acknowledge the people in your community that are contributing (financially or with code!).
Thank you for your great contribution to the open source community. You are awesome! 🙌
And welcome to the open collective community! 😊
Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other open source sustainers!

Awesome, thank you @asood123 for this! Looks great.
Two questions:

Unfortunately I'm hitting https://github.com/opencollective/opencollective/issues/606 - I'm unable to admin or set up my collective. It seems my e-mail<EMAIL_ADDRESS>is associated with bryphe1, which isn't a core contributor to the oni collective.
Is it possible to change the URL for our collective to be https://github.com/onivim/oni? I migrated to an org account after signing up. Might be an admin option that I can change once I have the core contributor permission.

Thanks again! I'm excited to get going on open collective.

Done on both counts. Try logging in with that email now and it should let you edit your collective.


Updating develop with release branch

Codecov Report

Merging #871 into develop will decrease coverage by 0.04%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop     #871      +/-   ##
=============================================
- Coverage      62.37%   62.32%   -0.05%
  Complexity       869      869
=============================================
  Files            444      444
  Lines          12828    12828
  Branches        1480     1480
=============================================
- Hits            8001     7995       -6
- Misses          4238     4242       +4
- Partials         589      591       +2

Impacted Files
Coverage Δ
Complexity Δ

.../gov/gchq/gaffer/store/schema/SchemaOptimiser.java
75.26% <0%> (-6.46%)
0% <0%> (ø)

Continue to review full report at Codecov.

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


JMS: Support passing a ConnectionFactory factory

We currently only support the JEE way: pass an InitialContextFactory class, create an InitialContext and fetch the ConnectionFactory with a JNDI lookup.
Some JMS client, such as AWS SQS don't have such InitialContextFactory and only support creating a ConnectionFactory manually.

I don't mind looking into this. But if possible I'll need some up front guidance if OK?

We need to be able to support both strategies:

the existing JNDI based one
a new one where one would pass a () => ConnectionFactory function

Then, we need to be able to pass this from the DSL, ie JmsProtocolBuilder.
It looks like the current DSL is not very right, as the ContextFactory and the ConnectionFactory JNDI name are passed in different places while they are actually completely related.
If one can manage to implement this without breaking the current DSL (probably in an ugly, but temporary way), we could ship it in Gatling 2.2. Otherwise, we'll have to wait for Gatling 3.

What about moving ConnectionFactory initialization out of the JmsClient? Let's keep a reference to a factory in JmsProtocol, and let user create that factory instance.
  // create a factory explicitly
  val cf = new ActiveMQConnectionFactory()

  // or use DSL
  val cf = jmsConnectionFactory
    .name("FFMQConstants.JNDI_CONNECTION_FACTORY_NAME")
    .url("tcp://localhost:10002")
    .credentials("user", "secret")
    .contextFactory("FFMQConstants.JNDI_CONTEXT_FACTORY")

  // and the protocol builder just uses a reference to our factory
  val jmsConfig = jms
      .connectionFactory(cf)
      .listenerCount(1)
      .usePersistentDeliveryMode
      .receiveTimeout(1000)
      .messageMatcher(IdentificationMatcher)

@Mishail Yes, I suspect that's the way to go. Then, I'd like to know if we can completely drop the JNDI lookup way (meaning that we can indeed create a ConnectionFactory for all well know JMS implementations).


Spanish guide for CCS811 Firmware Update

Procedimiento para la actualización de firmwareCSS811_1.1.pdf
Spanish guide for CSS811 Firmware Update thanks Maarten for your great job!

Hi Ernesto,
Thanks, I have added the spanish doc.
I did rename it slightly (underscores), and put it in a subdir, but I have
also added a link from the main readme.
regards,
Maarten
On Tue, Oct 20, 2020 at 1:38 PM Ernesto<EMAIL_ADDRESS>wrote:

Procedimiento para la actualización de firmwareCSS811_1.1.pdf
https://github.com/erguro1973/CCS811/files/5407858/Procedimiento.para.la.actualizacion.de.firmwareCSS811_1.1.pdf
Spanish guide for CSS811 Firmware Update thanks Maarten for your great job!
You can view, comment on, or merge this pull request online at:
https://github.com/maarten-pennings/CCS811/pull/58
Commit Summary

Add files via upload

File Changes

A Procedimiento para la actualización de firmwareCSS811_1.1.pdf
https://github.com/maarten-pennings/CCS811/pull/58/files#diff-2ed3e88e5795cb2d2848bbd5d356e7f0108a43e07f1cb3cc0119897d1cab33a1
(0)

Patch Links:

https://github.com/maarten-pennings/CCS811/pull/58.patch
https://github.com/maarten-pennings/CCS811/pull/58.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/maarten-pennings/CCS811/pull/58, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHH3FQ3CLLQS75MUCZ32VWDSLVZD3ANCNFSM4SYGFYDA
.

Hi Maarten
Thank you so much for your great Job, I’m currently trying to fix baseline this sensor seems to be more complex to run it properly

El 20 oct 2020, a las 15:32, Maarten Pennings<EMAIL_ADDRESS>escribió:

Hi Ernesto,
Thanks, I have added the spanish doc.
I did rename it slightly (underscores), and put it in a subdir, but I have
also added a link from the main readme.
regards,
Maarten
On Tue, Oct 20, 2020 at 1:38 PM Ernesto<EMAIL_ADDRESS>wrote:

Procedimiento para la actualización de firmwareCSS811_1.1.pdf
https://github.com/erguro1973/CCS811/files/5407858/Procedimiento.para.la.actualizacion.de.firmwareCSS811_1.1.pdf
Spanish guide for CSS811 Firmware Update thanks Maarten for your great job!
You can view, comment on, or merge this pull request online at:
https://github.com/maarten-pennings/CCS811/pull/58
Commit Summary

Add files via upload

File Changes

A Procedimiento para la actualización de firmwareCSS811_1.1.pdf
https://github.com/maarten-pennings/CCS811/pull/58/files#diff-2ed3e88e5795cb2d2848bbd5d356e7f0108a43e07f1cb3cc0119897d1cab33a1
(0)

Patch Links:

https://github.com/maarten-pennings/CCS811/pull/58.patch
https://github.com/maarten-pennings/CCS811/pull/58.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/maarten-pennings/CCS811/pull/58, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHH3FQ3CLLQS75MUCZ32VWDSLVZD3ANCNFSM4SYGFYDA
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[ja] Translate tasks/debug-application-cluster/troubleshooting into Japanese

This is a Feature Request
What would you like to be added
Translate tasks/debug-application-cluster/troubleshooting  into Japanese.
Why is this needed
This page has not been translated.
Comments
/language ja
/assign

PR:
https://github.com/kubernetes/website/pull/30997

/triage accepted


capital T on Thorium

On the Mac version at least, the app name in the top navigation bar is "thorium". Same for the app name in the tray.
A capital T on Thorium would be cool.

https://github.com/readium/readium-desktop/blob/f1251186bd99cd7f9c6bca5fe9f3563f548a9d31/package.json#L75
https://github.com/readium/readium-desktop/blob/738d908d51cc103d0b845a33fb75a247227f6fd9/webpack.config-preprocessor-directives.js#L39
https://github.com/readium/readium-desktop/blob/8b83bb22171cf8dc26597b0df25ee8515998b9bf/src/preprocessor-directives.ts#L41-L42
https://github.com/readium/readium-desktop/search?q=_APP_NAME&unscoped_q=_APP_NAME

Thank you Pierre ;)
const capitalizedAppName = _APP_NAME.charAt(0).toUpperCase() + _APP_NAME.substring(1);

productName:
https://www.electron.build/configuration/configuration#configuration

I don't think we can override productName for the Mac DMG: https://www.electron.build/configuration/dmg
...so maybe the only solution is to change productName to Thorium, but we need to check for regressions with the Linux and Windows builds, and the CLI executable.

little problem :
https://github.com/readium/readium-desktop/blob/f1251186bd99cd7f9c6bca5fe9f3563f548a9d31/package.json#L2
I don't think you can replace the project name without changing the product ID of the appx needed to update on the Microsoft store.
I tried a while ago.


previous prompt bleeds through in GitHub Issue Notebooks

Steps:

Clone VS Code Repo or open with GitHub Repos
Install GHIN (make sure you haven't logged in to it)
Open a notebook like the my-endgame.github-issues notebook
Run
See "log in" prompt and go through login flow
Run notebook again

See this:

Possibly related... further up in the file, I notice some bad formatting of issues:

Not sure if this is GHIN or Notebooks issue.

another example 😅

It looks much worse. It seems we are not even picking the right mime type.

Verifier, with the next Insiders

Open endgame notebook
Run all cells from the top toolbar
On the first cell which has outputs, click the ... icon -> Change Presentation -> Choose Markdown
Run all cells again
Scroll the notebook to the end, there should be no outputs overlapped on each other (or output overlapping with markdown cell )


Bump framework and governance versions

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6418600520

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6418600520
Status: success


Fixes Issue for DV when collect field is Polyymorphic.

Fixes https://github.com/microsoft/Power-Fx-Dataverse/issues/165

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


Release <IP_ADDRESS>

OpenSearch 2.2.1 has been released, see release notes.
Official OpenSearch release ticket: https://github.com/opensearch-project/opensearch-build/issues/2481
Checklist:

[x] Compatibility Matrix is updated
[x] Release <IP_ADDRESS> tag is present in the repo
[x] Release artifacts are available and downloadable
[x] Release notes published
[ ] Forum announcement published

Release <IP_ADDRESS> is published: https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/releases/tag/<IP_ADDRESS>


Adds test-time only build tag for assertions in main code

Signed-off-by: Takeshi Yoneda<EMAIL_ADDRESS>

talked offline, I will follow the path of having a feature flag for testing rather than build flag when another necessity comes up

I think I will close this :D


Use the template literal syntax in ES6

You can use the template literal syntax in ES6 to get rid of escaping quote. For example:
// Instead of
const message = 'It\'s a message';

// We can do this which is more convenient
const message = `It's a message`;

Done.


Feature: image credits/attribution

We have a requirement to attach credits to images (not to particular placements of the image). The attribution info would be available through Image fields, but not in RichTextField image placements (seems ok given the move toward more structured content editing).
Upstream interest?

I did this yesterday, modeling closely on the RCA implementation, and it went through without problems. I'm starting a doc section "Using a custom image model" under "Images/Advanced topics".
I have a guess at what the Rendition model is for, but have treated it as a black box and just followed the example from RCA. Are there any additional warnings that should be in the docs about it?
Also, I was able to preserve existing data by the following procedure:

Deploy the new model, with no field additions, and run the migration to create it.
Dump a fixture from wagtailimages and manually edit it to:

Change the model names for wagtailimages.image and wagtailimages.rendition to our custom model names
Remove the filter records.

Restore the edited fixture.
Rename any ForeignKey fields into the image model to use the new model and deploy those migrations.

Thanks for sharing your migration experience!
I also migrated image tags:

Dump the fixture: python manage.py dumpdata taggit | python -m json.tool > taggit.json
Connect to the database and get your custom image model's content type ID by querying the django_content_type table. Also note the wagtailimages.image model's content type ID.
Edit taggit.json from step 1, removing taggit.tag entries and anything else that's not taggit.taggeditem model entries mapping images to tags.
Update all content_type properties for the taggit.taggeditem model to use the new ID.
Load the updated fixture with python manage.py loaddata taggit.json.

Finally, a Postgres table belonging to a model referencing the new image model failed to have its foreign key constraint updated (not sure why, two other tables had their constraints properly updated), which meant I got a "key ... is not present in table" IntegrityError on trying to update that table's model with a new image using the new image model. I then manually dropped the stale foreign key constraint, fixed the references and readded it.

This is quite old, but not handled in any conclusive way that I'm aware of.
I was thinking that it might be easier (better?) to add an ImageAttribution table that has an FK to the current wagtail image model. This doesn't mess with wagtail, and attributions would be optional, or could be added as needed.
I'd be curious if anyone had thought there? @gasman?

Hi @edrex,
This can be done by subclassing wagtail.wagtailimages.models.AbstractImage and wagtail.wagtailimages.models.AbstractRendition within your app (see https://github.com/torchbox/verdant-rca/blob/develop/django-verdant/rca/models.py for an example of this) and specifying WAGTAILIMAGES_IMAGE_MODEL in your settings. (Swapping the model on a site that already has existing content is quite tricky, unfortunately.)
This isn't really documented right now because I originally regarded it as a bit of a hack and didn't want to encourage it - however, it's used on multiple sites now and the feature is clearly here to stay, so it probably ought to get some proper documentation!

The link from above has changed to:
https://github.com/torchbox/verdant-rca/blob/d9ede994dbd1ef68eaa159ec930fd89a351c1329/django-verdant/rca/image_formats.py#L4-L25 Which might help the next person that comes along and reads this thread.


Optimize faq end & about fa-en & header fa-en for Seo

I really appreciate your work and effort,
thank you so much, keep up the good work.


Merge branch 'main' of https://github.com/nunocoracao/blowfish

Describe your changes
Added spelucin.me to the users.md list :D

My bad!! I tried to add me to users.md but messed up. Closing this.

Added you no problem


Text input

Hello
Is there a way to accept text input from the user
I want to drive a speech chip via MIDI and would need the user to add some text
Thanks!

yes,
from lua, from text controls, from text files…

Ah thanks ! I am digging into Lua


The download page cannot be clicked on the footer.

There is no response when you click the footer on the download page. When you double-click the footer, the focus is in the previous <div> tag. I checked the code, but couldn't find the problem.

thanks @wc7086 it was already reported:
duplicates https://github.com/dutchcoders/transfer.sh/issues/358
Still had no time to check what is the problem.
Actually the whole footer needs a refactoring (removing bitcoin donation link etc, see the other issue)


fix(behavior_path_planner): pull out on lane boundary

Description

Fixed a problem that prevented pull_out on lane boundaries.

Fixed back_finished decision
Fixed previous lane of getExtendedCurrentLanes

Use this function also for pull_out  and pull_over

Related links

TIERIV INTERNAL LINK ticket
Tests performed

psim
pull_over -> pull_out works well
https://user-images.githubusercontent.com/39142679/193543633-08cc9214-016a-454d-a768-05778f0dfa0c.mp4
shift pull out works well on the lane boundary

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@rej55
Could you approve this PR as conde owner of route_handler?


Implement sticky page header

In order to facilitate orientation on the site we could implement a sticky header that sticks to the top of the viewport and provides some minimal information on what page we are, e.g. the play title, the corpus title or simply the page heading. This could also indicate the corpus information requested in #84.

Implemented for play info pages and merged to the design branch.


[wip] Cli

Dropped in @troydai's branch on the main repo

@david-driscoll the change on the branch is way too big to be reviewed. can we get the first 2 dozens of commits in the master branch and then integrate the CLI project system gradually?

I don't want to merge any of the beta 8 stuff because it never worked, I'm fine with squashing all the commits to remove some noise though.  I was trying to get it building on my machine last night but I ran into an exception thrown by csc.

Ok. I will fork a new branch from the master and rebuild my changes over there. I believe it will be faster and easier.

Sounds good, let me know when it's ready, and I'll work on the travis build
stuff.
On Wed, Dec 16, 2015 at 1:25 PM, Troy Dai<EMAIL_ADDRESS>wrote:

Ok. I will fork a new branch from the master and rebuild my changes over
there. I believe it will be faster and easier.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-roslyn/pull/364#issuecomment-165199722
.

Troy has a different branch, I'll close this and open a PR for tracking the other branch.


First line not rendering until event happens

Describe the bug
Just updated to 0.26.1 and noticed that the first line of my UI does not render until an event takes place(keypress, window resize etc). When downgrading to 0.25, everything works correctly, just wondering if something new was introduced/fixed that I need to account for or if this is maybe a regression?
Setup
Please complete the following information along with version numbers, if applicable.

OS = macOS
Shell = zsh
Terminal Emulator = warp
Terminal Multiplexer = N/A

To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/mistakenelf/fm
Run the app by running make
Noticed that the first line is not rendered until a keypress is made or window is resized
Downgrade to 0.25 of bubbletea, and noticed that it works as expected

Source Code
https://github.com/mistakenelf/fm
Expected behavior
The first line of the UI should render without any actions
Screenshots
Notice that the first line of the filetree and the help screen on the right are not rendering until an action happens

Noted, thanks for the report, Tyler. I'm able to reproduce on various terminals on macOS. Just so we can diagnose properly, would you mind posting a screenshot of the actual expected outputs together?

Sure thing, here is what the expected output would look like:

Thanks again for reporting this, @mistakenelf — extremely appreciated. @aymanbagabas fixed this like a pro and we're releasing Bubble Tea v0.26.2 with the fix in a few minutes.


Tf recommenders update

Added a step to setup a specific kernel for TF Recommenders lab.
Followed #288 format, but this PR creates a kernel for a single notebook, instead of creating a kernel in directory level leke #288.

#284


Added The Play not found feature

Description
Added The play not found feature in the search functionality, Now whenever the user will search for a play which is not there it will throw the play not found messsage
Fixes # 43
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
npm start
Checklist:

[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

@atapas sir i've made the changes you told me


SOA util

corresponding to issues #37  and #40
I still have this problem
c305-005.ls6(1003)$ make
black src/
make: execvp: black: Permission denied
make: *** [Makefile:2: format_code] Error 127

black src/
make: execvp: black: Permission denied
make: *** [Makefile:2: format_code] Error 127

Try chmodding the makefile:
chmod 777 -R Makefile
Also I am working on integrating diplomacy_research into CI currently.

black src/
make: execvp: black: Permission denied
make: *** [Makefile:2: format_code] Error 127

Try chmodding the makefile:
chmod 777 -R Makefile
Also I am working on integrating diplomacy_research into CI currently.

@wwongkamjan If this works for you, or if you find another solution, please deal with #43.

c302-006.ls6(999)$ make
black src/
make: execvp: black: Permission denied
make: *** [Makefile:2: format_code] Error 127

Still unable to run make

c302-006.ls6(999)$ make
black src/
make: execvp: black: Permission denied
make: *** [Makefile:2: format_code] Error 127

Still unable to run make

Please look into/ask for a solution. I will do the same.

@wwongkamjan I do not experience this problem on frontera.

I tested on Frontera and still faced this issue :(


Refine GitHub Profile Documentation

Improved clarity and consistency in the 'About Your Profile' section.
Enhanced formatting and structure for better readability.
Updated links and descriptions for accuracy and relevance.

Why:
Closes:
This pull request aims to enhance the documentation for the 'About Your Profile' section on GitHub. By improving clarity and consistency, users will have a better understanding of how to utilize their profiles effectively.

What's being changed (if available, include any code snippets, screenshots, or gifs):
The changes include refined text to improve clarity and consistency, better formatting for enhanced readability, and updated links and descriptions to ensure accuracy. This will provide users with a more informative and navigable profile page experience.

Check off the following:

I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

For content changes, I have completed the self-review checklist.

@5YearsAgain Thanks for opening a PR! I'll get this triaged for review ✨

Hi @brotherrabbit91—thanks for your submission and interest in the GitHub docs, but unfortunately we can't accept the content changes here. I'll outline our reasons below:

We use AUTOTITLE deliberately: https://docs.github.com/en/contributing/style-guide-and-content-model/style-guide#formatting-links

The double ' instances in the intro frontmatter are also intentional because the same character is used to open the frontmatter text itself

We use sentence case for headers: https://docs.github.com/en/contributing/style-guide-and-content-model/style-guide#headers

What remains is really personal preference

We'd encourage you to open an issue in future if you're not sure about a submission, so we can discuss changes beforehand. You can also take a look at issues marked "help wanted" under https://github.com/github/docs/issues
Thanks again for your interest in the GitHub docs!


Update docs for resign.rb

Fixes #5978
Correct docs to say that CFBundleVersion (not CFBundleIdentifier) was changed AND clarify docs when CFBundleIdentifier is updated

Thanks @samskiter 🚀


Development: Decouple VCS and CI and migrate external settings

DO NOT DEPLOY, CONTAINS MIGRATION

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context
With this PR we decouple our VCS and CI systems. Up until now Bamboo used Linked Repositories from Bitbucket, with this change we can clone repositories without the special Application Link. Also, until now the builds were triggered from the VCS to the CI system, now Artemis is the only source of the build triggers for Bamboo and Jenkins, so we remove the webhooks in GitLab and the Build triggers in Bamboo.
For this PR, new requirements for Bamboo are mandatory: To be able to clone repositories, a shared credential with the name of artemis.version-control.user and its username and password needs to be created in Bamboo before being able to run Artemis. See
Other things are now unnecessary, such as the ci-token for the GitLab setup and the empty-commit-necessary and vcs-application-link-name.
Description
Bamboo:
This Migration removes the linked repositories in all BuildPlans and replaces them with a 'normal' git repository. Furthermore, the trigger on changes in Bitbucket is removed and the notification URL is replaced with a new one.
Jenkins/GitLab:
The webhook from GitLab to Jenkins is removed and the build trigger in the Jenkins pipeline is removed.
The webhooks that are removed are not needed as Artemis triggers the build by directly calling the CI system instead of going through the VCS. As a result, we can remove ci-token and empty-commit-necessary from the configuration.
Steps for Testing

Prerequisites:

1 Instructor
1 Student
Programming Exercises with and without auxiliary Repositories

Check the updated documentation and set the internal-urls for bamboo:
https://artemis-platform--6682.org.readthedocs.build/en/6682/dev/setup.html#bamboo-bitbucket-and-jira-setup
Migration

Log in to Artemis on develop
Create some new programming exercises with/without auxiliary repositories
Generate some student submissions
If you do not have a shared credential for the git user in bamboo, create one before running the migration, without it, Artemis will not start.
Checkout this branch, run the migration
Verify that upon submitting the builds still get executed as before, the workflow should not be changed.
Check that the build plans in Jenkins/Bamboo and the webhooks in GitLab have been updated, see the screenshots at the end of this description for references of before and after the migration.

Programming Exercise
After checking out this branch, make sure to test all programming exercise related scenarios. Creating new ones, working on existing ones, deleting exercises, importing, exporting and reimporting, pushing to exercises, in exams and in normal circumstances etc. Whatever cases you can think of, please test thoroughly so that we catch issues before releasing.
Review Progress

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Before
After

@dfuchss, thanks for the review! I changed the variable names to be more readable. What exactly do you mean by refactoring the migration steps? You are right, line 269 should not be attempted if we don't have a valid repository URL, I adapted the code.

@dfuchss, thanks for the review! I changed the variable names to be more readable. What exactly do you mean by refactoring the migration steps? You are right, line 269 should not be attempted if we don't have a valid repository URL, I adapted the code.

See https://github.com/ls1intum/Artemis/pull/6682#discussion_r1321952131

@Everyone let's focus on the important aspects here: testing, testing, testing the programming exercises
We cannot question and then answer every little detail in the migration code which will be deleted afterwards anyway!
So while we appreciate your comments, please try to focus on the important ones, otherwise we will be blocked by answering all questions (feels a bit like a DOS attack 😅)

The documentation still talks about application links, but these are now no longer needed, right? Can this section be removed?

@tobias-lippert Yes this looks like it could not download the dependencies, we didn't change anything with the templates or the content of a build plan for that matter.

I tested with the local Bamboo, Bitbucket, and JiraSetup
In general, working on old and new exercises worked fine for me, however, I have faced issues with exams
❗❗❗ NOTE: If the exercises of an exam are prepared before the migration, the repositories will not be unlocked automatically
I was able to reproduce it reliably on my local setup via:
setup exam -> prepare exercises -> migrate -> ❌ setup exam -> prepare exercises -> migrate -> prepare exercises -> prepare start -> ❌ setup exam -> prepare exercises -> migrate -> import exam to new exam -> prepare exercises -> prepare start -> ✔ setup exam -> migrate -> prepare exercises -> prepare start -> ✔
However, there are two ways to resolve the issue that are probably more efficient than investigating the issue further - assuming that there won't be a way to miss it in the upcoming changelog:

Create a new Exam by importing the old exam -> Repositories will be unlocked automatically (at least this worked for me)
Unlock the Repositories manually (locking manually probably needed aswell)

❗❗❗
Successfull testing steps

Tested the migration of normal course exercises:

Repository Url & Trigger was migrated as expected for Gradle exercise with aux repos
Repository Url & Trigger was migrated as expected for Maven exercises with and without aux repos

Started local setup with an empty database

no errors; working on maven exercises from code editor within courses worked fine
creating new exercise and submitting from code editor as instructor triggered build
creating programming exercise in exam and submitting from code editor as student triggered build

Creating a new exam with an old exercise, submitting via online code editor worked and cloning repository as well, builds triggered

Additionally, I think that the documentation should be adjusted further (see #6682 (comment) and #6682 (comment)), but this can of course be done in a follow up PR

Thanks for testing this scenario.
However, I cannot imagine how the migration could influence the unlock operation. Unlock is triggered by a schedule, which is completely independent of the migration.
The unlock task is (re)scheduled at the following times:

when editing a programming exercise in the user interface
when restarting the Artemis server instance with the scheduling profile
when editing an exam, if the visible date or start date has been changed

If this is actually a problem (which I doubt), there are easier workarounds to reschedule the unlock operation than importing exams.
It might be the case, that you run into an edge case with too short exam duration or too close changes during or shortly before the exam becoming visible.


Screenshot feature and  fetching of top 10 upcoming events from your google calendar Added

#3  say take screenshot and it will take screenshot and save it on desktop.

Please resolve the conflicting files

Conflicts resolved


Website: change styles and formatting to adhere to US Web Design standards

Use web design standards alpha was created during the summer of 2015 by designers and developers at the U.S. Digital Service and 18F.
The website uses jekyll. Folks working on this can install jekyll for their personal machines by following instructions of https://github.com/codeforamerica/howto/blob/master/Jekyll.md .
To learn more about how to get started, see https://standards.usa.gov/getting-started/. For the actual code base with instructions on how to bring it into the project, go to https://github.com/18F/web-design-standards .
For an overall view, see https://standards.usa.gov/

@ErnieAtLYD Re: installing Jekyll - it might work better to use github page's instructions here, the main difference is to use bundler instead of jekyll directly, I believe it does some stuff specific to github pages that makes it work better.

Here is a condensed set of commands you might use (this might get adapted into the main readme):

Open command line, terminal, git bash, or whatever is appropriate for your OS.
ruby --version - make sure you have at least 2.0.0 or higher, if not then need to update / install latest version
gem install bundler - installs the bundler gem
Clone the promise-zones/promise-zones.github.io repo somewhere, then cd into the repo folder.
bundler install - install stuff needed
bundle exec jekyll serve - now you can access on http://localhost:4000

Note: skip to the last step once you've gone through the steps once to start up the "local server".  Now when you make changes in your local copy of the files, you can test them out at localhost:4000


Mapping Consequences to FAIR forms of loss

Does anyone see value in mapping the consequences on Risk Consequence page to FAIR's six forms of loss? Additional column?
If so, I'd be willing to take a first pass at it.

Yes, that sounds useful.
I'd be tempted to add it as an appendix or a separate document in the repository that we link to as we are trying to be analysis method independent in the standard itself.
Phil


[Enhancement] Add util to simplify the ExampleCoding configuration

What's changed:

Add an util class ExampleCodingConfigUtil.java which provide methods to configure example coding and methods to convert DataTypes and TypeInformation.
Add a test of ExampleCoding for table using above util class.

Related to #19


feat: Publish handleLinkNavigation to App Props

@zsviczian there is already a PR open for the same https://github.com/excalidraw/excalidraw/pull/4694 :)


KeyGesture not having empty ctor on 0.9

Hey,
Trying out 0.9 preview 4 and noticed KeyGesture doesn't have empty ctor, is there any reason for it?
Since I can no longer serialize it
Thanks

Calling ToString() and Parse(string s) should work for you.

I know, the thing is that when I serialize it to json I prefer that the JsonSerializer will deserialize it automatically.
Anyway just wondered why the constructor was removed. (I will use the Parse and ToString to serialize it)

KeyGesture is now immutable, all setters have been removed so only constructors that allow to construct a valid object were kept. This is in line with WPF API as well.


pythonPackages.pybase64: init at 0.2.1

Motivation for this change
Adds the pybase64 package, a simple python-based libbase64 wrapper.
Please have a look at the commit messages for further reference.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

currently running nox-review pr 34589...

ok, the package flexget fails for me, however this seems unrelated (I also tested this on the latest nixpkgs master without this change), I'd submit a separate PR to fix this.

@FRidh done. The flexget package is still broken, however this is an issue on master and I already started investigating, however I need some after-FOSDEM sleep first  :-)

done.

This causes a Hydra build to fail: https://hydra.nixos.org/build/68773666

@dotlambda so, yielding warnings (in this case because of a deprecation by the package author) causes our builds to fail? oO
For me it seems weird to me to break a build because of warnings (in this case because of a deprecation warning due to a deprecated package which is something that happens from time to time), but @FRidh might know more about this.
However if there's no simple workaround to keep the deprecation warning without breaking the build I'd rather drop the warning for now as I think that patching the parameterized package in order to look like nose-parameterized is IMO not a better solution since we just patch around to satisfy some minor warnings.

I think we might be able to fix the flexget package by pinning pytest to a certain version (probably 3.2.3 or 3.2.4). I'll try this and report whether it succeeds.

👍 I think we could also create a package set using ‘pypi2nix’ as this would solve the dependency version problem

I think the best solution would be to just disable tests completely. Even when up-/downgrading pytest, I have to disable about half of the tests.

In this case we’d have to test the resulting binary manually right?
As I mentioned we could try if pypi2nix actually fixes these issues, however I can’t have a look at this before the next weekend %)

On 8. Feb 2018, at 9:36 AM, Robert Schütz<EMAIL_ADDRESS>wrote:
I think the best solution would be to just disable tests completely. Even when up-/downgrading pytest, I have to disable about half of the tests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I don't think derivations generated by pypi2nix are fit for inclusion in Nixpkgs yet.

can you elaborate, please?
We use tools like hackage2nix and node2nix in nixpkgs to generate derivations. The thing is: such applicatiojs pin their dependencies explicitly, so we’d either have to patch and test such applications whenever we bump a python library or we work around this issue by generating package sets for bigger applications (which actually happens with several nodejs and ruby applications)

On 8. Feb 2018, at 9:36 AM, Robert Schütz<EMAIL_ADDRESS>wrote:
I think the best solution would be to just disable tests completely. Even when up-/downgrading pytest, I have to disable about half of the tests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

can you elaborate, please?

Security and testing. Give it some thought how these are affected by the use of pypi2nix.

such applicatiojs pin their dependencies explicitly

Which is bad practice. They should not pin on patch level, but unfortunately it is often done.

have to patch and test such applications whenever we bump a python library

That's why most of our packages have tests, so we can see this happen, and we typically see it happen! It just requires maintainers of these packages to actively maintain their package.

Security and testing. Give it some thought how these are affected by the use of pypi2nix.

ACK, you're right.

That's why most of our packages have tests, so we can see this happen, and we typically see it happen! It just requires maintainers of these packages to actively maintain their package.

Okay then, I already started chasing and fixing some build/test issues of flexget, however I lack the inside knowledge in how Python packaging works in Nix, so I had some broken tests where I didn't know how to fix them.
Will push the branch, then we can see what we can do next ok? :-)


What about access from non-Windows devices such as Azure AD registered MacOS and mobile devices?

The documentation stated the requirements for access to Azure AD joined Windows 10 PCs via RDP from other Windows devices. But it does not mentioned access from non-Windows devices. For example, MacOS/iOS etc have official Microsoft RDP client apps, and they can be Azure AD registered as well. Is access from these devices supported?

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

ID: 72e1cc18-3fc2-d7fa-7ff3-20fc9a3596d4
Version Independent ID: 8ed24664-ce8c-7c08-654a-d3ef267f274f
Content: Connect to remote Azure Active Directory-joined PC (Windows 10) - Windows Client Management
Content Source: windows/client-management/connect-to-remote-aadj-pc.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Hello @felinepc ,
in the article, it states:
Your local PC (where you are connecting from) must be either Azure AD joined or Hybrid Azure AD joined if using Windows 10 version 1607 and above, or Azure AD registered if using Windows 10 version 2004 and above. Remote connections to an Azure AD joined PC from an unjoined device or a non-Windows 10 device are not supported.
So for AAD-Joined - must be Win 10 1607 or newer, for AAD-registered - Win 10 2004 or newer, and non-windows 10 devices are not supported.
Thank you


Wrong Data for IFSC code PUNB0428100

Json data from API (https://ifsc.razorpay.com/PUNB0428100)
{"CITY":"KUTHUPARAMBA","MICR":"","CONTACT":"04902361321","STATE":"KERALA","ADDRESS":"17/306(47), MALABAR PLAZA KANNUR ROAD, KUTHUPARAMBA, KANNUR DISTT, KERALA 670643","CENTRE":"KUTHUPARAMBA","UPI":true,"RTGS":true,"DISTRICT":"CANNUR","IMPS":true,"NEFT":true,"BRANCH":"KUTHUPARAMBA","BANK":"Punjab National Bank","BANKCODE":"PUNB","IFSC":"PUNB0428100"}
Expected Data:
Address: WANDOOR ROAD KALIKAVU KERALA 676525
State: KERALA
District: MALAPURAM (Click here for all the branches of "PUNJAB NATIONAL BANK" in "MALAPURAM " District)
Branch: KALIKAVU

The details come from the RBI website itself: https://rbidocs.rbi.org.in/rdocs/RTGS/DOCs/RTGEB0815.xlsx (Linked at https://www.rbi.org.in/scripts/FAQView.aspx?Id=65)
Can't do anything here, perhaps reach out to RBI or the banks to get this corrected.


Populate location table flags

Follow-on to https://github.com/pulibrary/marc_liberation/issues/964
https://github.com/pulibrary/marc_liberation/blob/alma/app/services/location_data_service.rb uses the Alma API to pull the new libraries and locations into the locations database tables. However no flags have been set.
If these flags can be set via the Alma API, we need specifications for doing so. If they can't, they must be set directly, either by mapping existing locations to alma locations and reading the values from current production, or by hand in the bibdata-alma-staging UI.
Question: Will there be new locations added to Alma? Previously we were told no, those were locked, but recently there was talk of new Marquand locations. If there are new locations that is relevant here because the code that syncs the locations from Alma deletes / repopulates the table, so it would wipe out any flags that had been updated if it were re-run.

If we add a new code in Voyager we will add it to Alma at the same time, so it should be in the data pulled out from Alma. We have been asked to add one location for Marquand due to a late breaking change for their closure next month. That should be the only one.

We need to do an audit of the flags that one can set it Alma and see we can set these flags in Alma rather than here. It appears Alma has some extensible features in this space.

@hackartisan what's the best way to manage the bulk updating of the location flags mapped to their new alma location codes in the bibdata alma environment?  I will also add the two locations need for processing scsb records, scsbnypl and scsbcul, to the data.

@kevinreiss probably a rake task that reads the mappings directly into the database, that we then run on the server. What do you think @eliotjordan ?

For anyone who will work on this issue when checking how locations/flags display in orangelight, remember to clear the cache in the catalog-alma-orangelight box. https://github.com/pulibrary/orangelight/blob/6f35623fb747934e4ef2bd428ed725787390c0f8/app/services/bibdata.rb#L40 There is a rake task to clear the cache in orangelight.https://github.com/pulibrary/orangelight/blob/6f35623fb747934e4ef2bd428ed725787390c0f8/lib/tasks/cache.rake#L3


ISSUE-1003 - Support for event-time windows in Join processor

@shahsank3t can you test UI changes using this PR ?

Sample windowConfig  json, passed by UI to backend for EvenTime :

"window" : {"windowLength" : {"class":".Window$Duration", "durationMs":1000},
            "slidingInterval" : {"class":".Window$Duration", "durationMs":1000},
            "tsFields": ["s1:tsField1" , "s2:tsField2", "s3:field3.innerTsField"], //new
            "lagMs": 10,
            "lateStream": "optionalLateStream"  //new
 }

Note: Need to ensure there is one timestamp field per input stream when using event time joins

@roshannaik
What is lateStream in the JSON? Currently, we have nothing like that on UI.

@roshannaik , are these changes backward compatible? It will be better if it works seamlessly for users who already have aggregate and join processor in their SAM app and migrate to HDF 3.1.0. If you can accept the tsField as a string itself and handle both single field plus and new format it may work. Otherwise we will have to provide migration scripts which will get tricky.
For the new field (lateStream) we should handle the cases where late stream is missing in the stored config so that existing saved configs in DB will continue to work with HDF 3.1 without any migration.

@arunmahadevan
Yes these enhancements are backward compatible and limited to Join processor. Does not interfere with any other components.
@shahsank3t
lateStream is a feature of Storm's windowing subsystem, specific to event-time windows. In short .... if an event arrives too late (after its window is already processed) then such late events are simply emitted to a lateStream if it is configured. If it is not configured, then the event is simply logged and dropped. Such late events don't go to the regular output stream.  In principle any component dealing with event-time windows should support this.
The late stream would be an "special" but optional 2nd output stream for Join processor (only available for event-time joins).

@shahsank3t,  you may want to tackle the other stuff before exposing lateStream, given that it is an optional setting, things will still work without it being exposed.

@roshannaik , can you rebase this and update when its ready to test? Will try out the changes.

@shahsank3t , can you try it out from the UI side?
@roshannaik , can you rebase this with master ?

rebased it.

@roshannaik  can you address https://github.com/hortonworks/streamline/pull/1004/files#r150451212 ?

@roshannaik , can you try out if the end-end runtime flow after applying https://github.com/hortonworks/streamline/pull/991 ?
@shahsank3t has already verified the UI-backend interaction.
Once you confirm I will merge it.

@roshannaik , I am also seeing the following findbugs warnings when I do a "mvn clean install". Can you address that as well?
[INFO] BugInstance size is 3
[INFO] Error size is 0
[INFO] Total bugs: 3
[INFO] Invocation of toString on Window.tsFields in com.hortonworks.streamline.streams.layout.component.rule.expression.Window.toString() [com.hortonworks.streamline.streams.layout.component.rule.expression.Window] At Window.java:[line 209] DMI_INVOKING_TOSTRING_ON_ARRAY
[INFO] com.hortonworks.streamline.streams.layout.component.rule.expression.Window.getTsFields() may expose internal representation by returning Window.tsFields [com.hortonworks.streamline.streams.layout.component.rule.expression.Window] At Window.java:[line 159] EI_EXPOSE_REP
[INFO] com.hortonworks.streamline.streams.layout.component.rule.expression.Window.setTsFields(String[]) may expose internal representation by storing an externally mutable object into Window.tsFields [com.hortonworks.streamline.streams.layout.component.rule.expression.Window] At Window.java:[line 163] EI_EXPOSE_REP2
[INFO]

To see bug detail using the Findbugs GUI, use the following command "mvn findbugs:gui"

@arunmahadevan ... updated the PR with the fix for that.

@roshannaik ,
I built a sample topology and on deployment I am getting the following exception.
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.apache.storm.flux.FluxBuilder.buildObject(FluxBuilder.java:310)
	at org.apache.storm.flux.FluxBuilder.buildComponents(FluxBuilder.java:358)
	at org.apache.storm.flux.FluxBuilder.buildTopology(FluxBuilder.java:74)
	at org.apache.storm.flux.Flux.runCli(Flux.java:153)
	at org.apache.storm.flux.Flux.main(Flux.java:98)
Caused by: com.fasterxml.jackson.streamline.databind.exc.UnrecognizedPropertyException: Unrecognized field "tsFields" (class com.hortonworks.streamline.streams.layout.component.rule.expression.Window), not marked as ignorable (4 known properties: "lagMs", "slidingInterval", "windowLength", "tsField"])
 at [Source: {"windowLength":{"class":".Window$Duration","durationMs":5000},"slidingInterval":{"class":".Window$Duration","durationMs":5000},"tsFields":["kafka_stream_1:ts","kafka_stream_2:ts"],"lagMs":0}; line: 1, column: 141] (through reference chain: com.hortonworks.streamline.streams.layout.component.rule.expression.Window["tsFields"])
	at com.fasterxml.jackson.streamline.databind.exc.UnrecognizedPropertyException.from(UnrecognizedPropertyException.java:62)
	at com.fasterxml.jackson.streamline.databind.DeserializationContext.reportUnknownProperty(DeserializationContext.java:855)
	at com.fasterxml.jackson.streamline.databind.deser.std.StdDeserializer.handleUnknownProperty(StdDeserializer.java:1083)
	at com.fasterxml.jackson.streamline.databind.deser.BeanDeserializerBase.handleUnknownProperty(BeanDeserializerBase.java:1389)
	at com.fasterxml.jackson.streamline.databind.deser.BeanDeserializerBase.handleUnknownVanilla(BeanDeserializerBase.java:1367)
	at com.fasterxml.jackson.streamline.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:266)
	at com.fasterxml.jackson.streamline.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:125)
	at com.fasterxml.jackson.streamline.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:3789)
	at com.fasterxml.jackson.streamline.databind.ObjectMapper.readValue(ObjectMapper.java:2779)
	at com.hortonworks.streamline.streams.layout.component.rule.expression.Window.<init>(Window.java:119)
	... 9 more

Can you fix this?

@roshannaik , I am also seeing errors and not able to configure an "Aggregate" processor. Heres the response received -
(was java.lang.NullPointerException) (through reference chain: com.hortonworks.streamline.streams.layout.component.rule.Rule["window"]->com.hortonworks.streamline.streams.layout.component.rule.expression.Window["tsFields"])

@roshannaik , I am able to deploy the Join topology.
Only while creating an Aggregate processor I am not able to save it. @shahsank3t , can you also take a look ? I applied this PR + PR #991 to test.
@roshannaik , confirm once you have tried sending some data to your test topology and processed it with event time.

@arunmahadevan I am not able to start streamline server, getting Exception: https://gist.github.com/shahsank3t/1b0faef3c4cdef38ffa6066dc8e3bdfa
Once that resolves, I'll check Aggregate processor.

@shahsank3t , can you rebuild latest registry locally and then rebuild streamline. A related issue was fixed in registry recently. You may also need to cleanup local registry maven artifacts.

@arunmahadevan @roshannaik
Getting error:
(was java.lang.NullPointerException) (through reference chain: com.hortonworks.streamline.streams.layout.component.rule.Rule["window"]->com.hortonworks.streamline.streams.layout.component.rule.expression.Window["tsFields"])
on PUT request to API: api/v1/catalog/topologies/:topologyID/windows/:windowID

The issue with kafka was env config issue. probably we should add some checks in Ambari to prevent invalid configs. Will raise an issue for that
I think now you should be able to see the events flowing in runtime into the Join processor. Will wait for your test results.

@shahsank3t There is UI bug.  When i use aliasing to disambiguate output fields in the join processor, the downstream KafkaSink's input schema does not reflect the aliased names. it shows the unaliased names. here screen shots of the schema mismatch.

@sameer79 could you please fix the bug @roshannaik mentioned above, in your existing PR https://github.com/hortonworks/streamline/pull/991

@roshannaik , i have a patch for addressing the aggregate processor issue I will merge it along with this once you have tested the Join runtime.

@roshannaik address the issue and fixed in PR https://github.com/hortonworks/streamline/pull/991

@roshannaik , can you comment if you have tested the runtime part and the UI issues you encountered in https://github.com/hortonworks/streamline/pull/1004#issuecomment-346198546 have been addressed?

Was able to test both backend and front end. Backend is looking good. One minor issue with validation. Have rebased this PR.

The UI issues are fixed in the PR. Verified again that both UI pr & this backend PR are now working fine.


⚠ Bofrost.at has degraded performance

In dedcc6b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1410 ms

Resolved: Bofrost.at performance has improved in b6e916b after 21 minutes.


Manga mode do not work when converted to cbz format

When I converted zip to cbz format(checked the manga mode option),I found that the page turn is still left to right.How could I fix this?
device:kobo libra colour

Have you tried the epub format? @nodscrim

Yes, it worked on epub format. But cbz would be better because of the high loading speed and zoom function @axu2

isnt that just a limitation of your reader? cbz is just a zip archive, you dont define any reading direction there

duplicate #351

Ok,thanks a lot @axu2


Relevant

Are you still using this module? I know old doesn't mean bad, but I thought I'd ask because in this case old is pretty old :p

@RWOverdijk IDK if it's still in use at my previous company, but as for myself, I'm not using it anymore.

@felixge Thanks :)


Add binary_poly1_loss

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

Project coverage is 100.00%. Comparing base (7d9a3e8) to head (695ab15).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #374   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          106       106
  Lines         3250      3255    +5
  Branches       268       268
=========================================
+ Hits          3250      3255    +5

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


Work in progress: Externalized page metadata

This issue describes a way to add or modify Page metadata in batches from external file(s). This is the information you currently only set in Page front matter.
Some use cases:

The default page sort is ByWeight (with fall back to date and then title). Setting weight in front matter to get a particular order can be cumbersome if you have thousands of articles. With this implemented, you can re-arrange the pages by assigning weight in batches by Glob pattern matching.
Tag all pages with both funny and cat in the path with the tag Funny Cats
Add category Other to pages with no category set.

Some examples in TOML:
# Re-arrange the blog posts so dog articles come first.
# Note that the `:counter` will restart at each `meta` element.
[[meta]]
match = "/*/blog/**"
matchFirst = """
**dog*
**cat*
**
"""
counterStart = 10 # default is 1
counterIncrement =  10 # default is 1
replace = true # default is false
[[meta.data]]
weight = ":counter"

# Add a category and a title to blog posts with none set.
[[meta]]
match = "/*/blog/**"
[[meta.data]]
title = "Blog article #:counter"
categories = ["Other"]

There can be many meta definitions, and all pages will be tested against all of them.
A page that matches both the Glob pattern in match and matchFirst will be assigned the given metadata. if replace = true any existing value will be replaced.
The match pattern is primarily "motivated by speed", but it also allows for simple and coarse-grained patterns in matchFirst.
The values in meta.data map directly to what you would normally set in page front matter.
The patterns in matchFirst sets the order of the 1-based :counter placeholder. In the example above, the articles about dog will get weight 10, 20, 30, and then come the cats and finally all the other.
In the example, matchFirst is a multiline string, but we will accept string arrays, too.
We will match against the canonical path* (always prefixed by the current language, so one can do /{en,jp}/blog/** etc).

*: It would be tempting to match against .RelPermalink, but that will typically change across languages. We need to define the term canonical path as part of this issue.
Some references:

This proposal is inspired by how we do "page resources" metadata, see https://gohugo.io/content-management/page-resources/#the-counter-placeholder-in-name-and-title
Having this configuration in the site configuration will make more sense once #5422 is ready (should be next very soon)
This proposal will also make more sense/add more value if you also consider #5255 and #5074

Great vision for the future here! :)

This is cool.
My brain wants to do
weight = “{{ mul .Counter 10 }}”

Or is that a bridge too far?

 And we will eventually get there, but I'm not sure "Go templates" is the correct medicine -- as they are basically text templates (e.g. presentational), and also not very fast.
See https://discourse.gohugo.io/t/tools-libraries-that-can-drive-new-hugo-features/7891/48?u=bep

title = "{{ .Data.name }}"
subtitle = “{{ .Data.fields.sub_title }}”

OK, so I suspect the above would be highly ineffective at scale. But, it got me thinking a little.
[[meta]]
match = "/*/blog/**"
script = """
.Scratch.Add "counter" 1
$data := index .Site.Data .Page.Section
.Set "name" $data.name
.Set "weight" (.Scratch.Get "counter")
"""

The above would probably only work for TOML? I.e. the multiline stuff. But it would be pretty flexible, a.
Note that the field names above are very temporary (meta? maybe frontmatters).

The script value could ideally be stored in a separate file to improve coder's experience.
Or better yet one of those new custom function being discussed.(https://github.com/gohugoio/hugo/issues/5222). The func could even return a set of meta built from the "page/data/source" context allows setting multiple key/value pair to be consumed by meta.data.
# Add a category and a title to blog posts with none set.
[[meta]]
match = "/*/blog/**"
[[meta.data]]
script = "func getBlogMeta"

I'm really bad at writing toml.

Dood. I missed to golua library. Hugo+Lua would be very interesting. I’ll have to ponder the implications.

But looking at the above attempts of "scripting in config" looks ... ugly. We risk ending up ...
https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.gitlab-ci.yml


Blazor compiler reports misleading error

If a tag in a Razor page does not match with a Razor Component, but component binding syntax is added, the compiler reports a binding error. I
To Reproduce
Steps to reproduce the behavior:

Create new Blazor client-side app (.NET Core 3.0 Preview 4) in VS2019 v16.1.0 preview 2
In Index.razor add <BadComponent /> in the HTML (this should colour blue as if a HTML tag)
Compile and run app will work, <BadComponent /> is treated as HTML and rendered normally.
Change to <BadComponent bind-Value="@Test" />
The tag now changes colour as though it were a Razor component, and there is a compile error:

Error: The attribute names could not be inferred from bind attribute 'bind-Value'. Bind attributes should be of the form'bind', 'bind-value' or 'bind-value-change'

This leads the user to believe they have a binding issue, when in fact it's the component that does not exist.
Expected behavior
Would expect the compiler to report that it was not able to resolve BadComponent as a Razor component
dotnet --info
Microsoft Windows [Version 10.0.17763.437]
(c) 2018 Microsoft Corporation. All rights reserved.

. Let's consider making the PascalCaseComponentName convention into in actual enforced rule, with a compiler error if you use that casing on something that resolves as an element not a component.
I'm only saying we should consider this, as we may decide not to be so heavy-handed. If we did it, we would most likely also want some directive that suppresses the rule for the current file (or everywhere, if in _Imports.razor), because it's perfectly legal to write HTML as
...
 or
...
 and some people may have obscure reasons for doing so (e.g., they have to copy-paste in massive blocks of such markup from some other system).

Can confirm for preview 3.0.100-preview4-011223, was stuck for 2 days because of a missing import.

I think it will be a welcome addition for most people.
People also think they can create a Button component and it will be something different than <button> HTML element. Currently I don't have computer with Blazor at hand and I cannot check but probably this problem still exists. Maybe it is a good idea to force PascalCase convention for components and at the same time make some changes to allow component's names which are similar to HTML elements but with different casing.

This is fine by me. This seems more appropriate as a warning rather than an error though. @NTaylorMullen do you have any cool thoughts about how users would suppress this?
I'd imagine something like:
@pragma suppress RZ1337

#YOLO

Agree with both @SteveSandersonMS and @rynowak 's suggestions. I'm not exactly new to Blazor and I made these errors very easily, as have others, so helping spot when components are unresolved makes sense.
In reply to @Andrzej-W yes it still exists, I created a
 component with no problems, so it might be a good idea to either warn or raise an error if a component has the same name as a HTML tag?

The list if html tags is know. So if we create a component with same name regardless of case this could be a warning.
If a non html tag is found in the razor that doesn't resolve to a component this to could be a warning.

@NTaylorMullen do you have any cool thoughts about how users would suppress this?

In regards to suppressing, I'd stay away from the single line @pragma suppress ... suggestion because it overloads the C# meaning of pragma which adds to the confusion of which portions of a Razor document are impacted when providing those pragmas and the level of support is in regards to C# error/warning codes? i.e:
@* What happens here? *@
@pragma suppress 4507

@pragma suppress RZ1337
...

@functions {
    // Do things in here have the suppression?
}

I see three options:

Add a Razor specific directive that functionally acts like a pragma: @razor suppress RZ1337. It wouldn't correspond to any sort of C# pragma and it would be smart about applying the pragma logic to underlying directives such as @functions
Make pragma directives encompass blocks and have clearer identifiers, i.e. @pragma suppress ComponentNameEnforcement { ... } which could disallow specific concepts inside of them (such as @functions).
Take a JavaScript approach and add a "strict" equivalent (@strict) where it's opt-in vs. opt-out. It could be added to every _Imports.razor template (we'd make it work in imports) and would still allow other frameworks that operate on pascal cased elements at JS time.

I don't understand why we can't just do 1 and call it @pragma. That name is well known unlikely to collide with code in a user's project.

My concern is just with its C# equivalent and any confusion it might introduce. If you feel like those concerns aren't valid then sure we can go with 1 with pragma

Closing this in favor of #9860 - which contains the solution we're going to use for this.


Update Issue #906

Description
#906 - Added H1 Tag as required.
Additional Context
At a glance:
    -snip-
     <title>Fork, Commit, Merge - HTML - Easy</title>

Contact Information:

        <a<EMAIL_ADDRESS>me!
    -snip-

    -snip-
     <title>Fork, Commit, Merge - HTML - Easy</title>

 John Doe

Contact Information:

        <a<EMAIL_ADDRESS>me!
    -snip-

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Verify function return values are semantically equivalent

ok, err := jwt.Verify(idToken)
if err != nil {
	return false, fmt.Errorf("failed to verify ID Token: %w", err)
}

if !ok {
	return false, fmt.Errorf("integrity compromised")
}

If ok is false then err will always be nil
Which raises the question, do we want this? Because as it is now requires one extra if statement for the same semantic situation, which is boilerplate

super good point @KendallWeihe. which makes me wonder whether we should even return ok? @alecthomas thoughts on all this from a go idioms perspective?

If you're always returning an wire in the false case then it makes sense for the function to just return the error. It would make sense if you need to distinguish between invalid for some reason and any other error.

This is fixed for the jws.Verify() and jwt.Verify() which now return (Decoded, err) in this PR
Issue still present in the Verify() functions within the crypto/ directory

I had started down this path last week but didn't get all the way https://github.com/TBD54566975/web5-go/tree/kendallw/verify-return-types


support last_affected

We've been putting this off since no producers have been using this yet, but GitHub will be doing this soon so we need to add support for it: https://github.com/github/advisory-database/issues/470

Remaining work: support last_affected for git ranges.


Delay when rendering the next component in useTransition

🐛 Bug Report
I want to use useTransition for multi step forms. First off, the initial form does not translate properly into place and the value of the translate always seems to vary on each refresh. Also when transitioning between pages, the new form that is being presented is delayed before showing and it just pops up with no real animation
To Reproduce
https://codesandbox.io/embed/1y3yyqpq7q
I followed this tutorial closely in order to implement the transitions
Expected behavior
I expect the initial page to be placed correctly with a transform value of translate(0%) and for the new forms that are being transitioned into to be already present and not have to wait for it to load
Link to repro (highly encouraged)
https://codesandbox.io/s/dazzling-hill-1kl2o
Environment

"react-spring": "^8.0.27",
"react": "^16.8.6",

Have you tried with v9? npm install react-spring@next

@aleclarson I just tried it with v9 and unfortunately it seems that it doesn't render properly still. I switched my implementation a little bit, which seems to have fixed the transform problem. This is what the current state of the code looks like right now:
const SignupContainer = () => {
  const [pageCount, setPageCount] = useState(0)
  const [userInfo, setUserInfo] = useState({})

  const goToPage = direction => setPageCount(pageCount + direction)

  const SignupProcess = [
    <InitialGreeting goToPage={goToPage}/>,
    <Typeform continueText={'Continue'} goToPage={goToPage}/>,
  ]

  const transitions = useTransition(pageCount, page => page, {
    from:  {  transform: 'translateX(100%)' },
    enter: {  transform: 'translateX(0)' },
    leave: {  transform: 'translateX(-100%)' },
  })

  return (
    <div>
      {transitions.map(({ item, props, key }) => (
        <animated.div style={props} key={key}>
          {SignupProcess[item]}
        </animated.div>
      ))}
    </div>
  )
}

here's a recording of how the bug looks like right now
https://www.loom.com/share/558e066b0cd34c1187597fb3fa60f363

I just looked at your sandbox, and there's no bug here. Here's a couple of tips:

To avoid the 2nd page being pushed down by the 1st page, your pages should use position: absolute
If your 2nd page is rendering too slow, you'll need to find a way to pre-render it. I don't know how hard this will be with useTransition (as I've never tried that)

You could try searching the Github issues (and/or the Spectrum community) for routing-related questions. Maybe you'll find some help. Anyway, I'll keep this open in case others have the same question.

@aleclarson So I looked more into it and found out that it was rendering perfectly fine, but the new components that get transitioned in are pushed to the bottom as they transition in, then they pop back up once the transition finishes. So just as you mentioned, I set the pages to position absolute, which seems to have only fixed it after the first transition.
Also, there seems to be all this empty white space that appears to the bottom and to the right of the components as they transition and when the first page initially renders. Do you know why this might be the case?

I ended up figuring out the solution to this problem. Setting the components to position: absolute was crucial and also for its parents to be position relative and overflow: hidden was also of the same importance.
After doing that, I was stuck getting a white screen only because I forgot to set the dimensions of the parent, making everything seem like it was being overflowed from the parents perspective.
Since this problem is fixed, I'm going to close this issue


refactor(cli): remove data proxy preview feature checks and env vars

These commits were already approved and were pushed to a staging branch. Since I am ditching the staging branch, I am merging this into main without further approval.

Done via another PR.


Jetbrains Pycharm Community version mismatch

Describe the bug
Package jetbrains.pycharm-community claims to be version 2024.1, but actually is 2023.3 in branches master and nixos-unstable (other branches not checked)
Steps To Reproduce
Steps to reproduce the behavior:

just run the package's binary, for example with

nix run nixpkgs/nixos-unstable#jetbrains.pycharm-community
nix run nixpkgs/master#jetbrains.pycharm-community

Expected behavior
expecting 2024.1 version of pycharm community to run, but actually 2023.3 version starts
Notify maintainers
@GenericNerdyUsername
@tymscar
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.84, NixOS, 23.11 (Tapir), 23.11.20240405.72da83d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `""`
 - nixpkgs: `/nix/store/8mk40j3mfrvndrnh89azngcgdgr1fidi-source`

Add a :+1: reaction to issues you find important.

are you looking for jetbrains.pycharm-community-bin?

In nixos stable branches there used to be only jetbrains.pycharm-community,
jetbrains.pycharm-community-bin is something new for me.
I see that now first one now claims to be built from source, second one claims to be patched binaries.
So then jetbrains.pycharm-community-bin is usable for me, yes,
But it doesn't change the fact that jetbrains.pycharm-community (and jetbrains.pycharm-community-src) package claims to be 2024.1, but actually is 2023.3.2

The same issue with idea-community, -bin package is also OK.

It seems I am also affected.
symlink of idea-community points to a directory named with the latest version from nixos-unstable.
But what actually launches is the version from nixos-stable.

After looking into this... no surprise the wrong version starts. The idea-community-src package is built from source/default.nix, which hasn't been updated in 6 months! The pycharm-community-src (with an pycharm-community alias on Linux) package has the same problem.
The reason it's tagged with the wrong version is because the version is taken from versions.json, with no checks or guarantees that the version in sources matches.
(I think this is related to symptoms from #281648. While the entry point is different (nix-shell/nix-env), the times at which people mention the symptoms changing or working workarounds kinda lines up with when the sources file was updated, lol.)

Sorry, I've had some life stuff going on and haven't been/won't be able to contribute for a while. I will continue at some point, but not in the near future

I think one of the core issues here is that mkJetBrainsProduct blindly inherits the version from bin/versions.json.
This means we don't notice when the src-package is out of date; it claims to be the same version as the bin-package.
source/build.nix does actually set version = buildVer (which may not be perfect, since that looks like 243.22562.145 instead of 2024.3.1), but this is then overridden by mkJetBrainsProduct.
If no one beats me to it, I'll try and come up with a PR to fix this.


favicon added in service page

I have added favicon in the service page.
My issue no: #749

Please add gssoc and level.

the path is not good.

Not Working in 😎 Deploy Preview see your changes!


n-data-table slot for row or component for column

This function solves the problem (这个功能解决的问题)
Hi, i feel a bit frustrating with working on datatable.
My issue is i often need custom render for columns, and doing layout via h function is a bit pain.
Expected API (期望的 API)
I see two ways there:

We can create new component called n-data-table-column for usage in vue <template>. In that case it should take column name from property for example called column="name".
If we not create n-data-table-column for other columns, they use default render from :columns property of n-data-table

Also this issue can be done via fully custom row customization, so we have component n-data-table-row, but in that case we should write all columns render here, which is might be overhead.

It seems that the official team is not planning to do this work. 😭
I created a project @skit/x.naive-ui based on Naive-UI, enables to use DataTable in template style. Welcome to improve it.

duplicate https://github.com/tusen-ai/naive-ui/issues/4273

@fudiwei thank! Appreciate translate to english.

@jahnli didn't know abotu that issues, since i'm not chinese speaker. Thanks.

There is a community that has created a Naive UI based project @skit/x.naive-uiDataTable that can be used in template styles.


JS: Phi edge barrier guards

Sanitizer guards now sanitize specific inputs to phi nodes when the corresponding control-flow predecessor is dominated by the guard.
The test case explains the situation:
function phi() {
  let x = source();

  if (something(x) && isSafe(x)) { // isSafe(x) is sanitizer guard
    // this input to the phi node for 'x' should be sanitized
  } else {
    x = null;
  }
  sink(x); // OK
}

Evaluation of security suite on nightly looks ok

Evaluation on big-apps was cut short for unexplained reasons, but here's what I've salvaged.
The run on gecko-dev ended mysteriously at the 4th iteration of Paths::PathExpr::getValue_dispred which doesn't seem like something that this PR would cause to blow up:
[2019-12-08 22:00:22] (183s) Empty delta for NodeJS::JoinedPath#class#f_delta (order for disjuncts: delta=100).
[2019-12-08 22:00:22] (183s) Empty delta for Paths::ConcatPath#class#f_delta (order for disjuncts: delta=1000).
[2019-12-08 22:00:22] (183s) Empty delta for Paths::PathExpr#class#f_delta (order for disjuncts: delta=100).
[2019-12-08 22:00:22] (183s) Starting to evaluate predicate Paths::PathExpr::getValue_dispred#ff#cur_delta/2[4]@95adc9 (iteration 4)

Could you redo the eval anyway? I know it's frustrating, but these changes can be subtle, so I'd rather err on the side of caution.

big-apps evaluation done. There's an 8% slowdown on one project - I'll retry that project with DPMs.

Re-evaluation of A2Z with DPMs show a ~1% increase in tuple counts and ~2% increase in DPM. The wall clock time was +6% but it's not clear if that's another fluke or poorly calibrated DPM weights.

OK, thanks, that's good enough for me.

Needs a merge/rebase due to conflict on change note.

@esbena any more comments? You're still listed as requesting changes.

Nope. My two requests have been addressed.


Create simlink to latest log

https://github.com/THI-Drone/deployment/blob/d96b5bc8812bee00634af02c11979e98ff7b71be/docker_run.sh#L2

Implemented in launch file

See: https://github.com/THI-Drone/mission_software_launch/pull/12


Where else have you seen this application level scale to zero pattern in use ? Is there a catchy name for it?

Hi Simon!
I discovered this scale-to-zero repo when doing a bit of nosing around the datasette ecosystem to see if there as a nice equivalent in datasette for django's nifty runserver command, that automatically reloads when a code change is detected, and I really like the implementation - thanks for publishing it open source 👍
I was doing some research into this for some work on this [Google summer of code project I'm mentoring on][gsoc], for building sustainability principles into Wagtail (and ideally upstreaming these to any other interested communities), and I was looking into how you would do scale to zero behaviour at an application level, like fly.io supports.
It looks like this datasette plugin implements a similar pattern to the one I was experimenting with and outlined in this gist, but goes quite a bit further.
One thing we'd like to do on the GSOC project is get together an example setup of application level scale to zero for Wagtail (and by extension, any django project using something like Gunicorn), and I'd like to refer to existing work, if possible.
Are you aware of a name beyond scale-to-zero used for this, that might help emphasis that this is a pattern that can be used in lots of places, and specified at an application level, instead of being something that happens at the container level?

Oh, hang on, Fly.io's docs seem to use a different term from scale-to-zero, that might be useful.

Machines are also the spawning ground for new platform features like wake-on-request (also known as scale-to-zero). You can stop a running machine to save on compute costs. It then may be started automatically when a request arrives at the Fly proxy.

Source: Fly Machines by @flydotio
This is from a blog post about doing this trick with an Pheonix app:

When a Fly Machine VM's main process exits, that VM enters a stopped state, but it isn't destroyed. The Fly.io platform proxy will try to wake up a stopped machine automatically in response to incoming connection requests, so with this recipe we can build a scale-on-demand Phoenix app!
Since HTTPS for our application on Fly.io is terminated at the load balancer, we don't need to worry about HTTPS connections.

Source: Shutting down a Phoenix app when idle by @flydotio
Anyway - this doesn't seem like it needs to be tied to Fly.io, even though they make it comparatively easy.

I've seen scale-to-zero implemented by Heroku, Vercel, Glitch and Cloud Run. I can't remember other specific terminology for it, I don't know where I first heard the "scale to zero" term but it seems to be the most common way this is described.


Fix parsing of T::Enum

The parser was returning a Class node for subclasses of T::Enum when we have a more precise type of node with TEnum.
This also showed that we were not visiting the TEnum properly as we called the visit on it as if a Class.
Maybe T::Enum shouldn't descend from Class?

I'm going to merge the parsing fix, the API can be changed in another PR as this is mostly going to be a breaking change 👍


Add support for batch_size > 1 in pgen_esm_from_fasta.py

Currently pgen_esm_from_fasta.py can only handle a batch size of 1, but it would be nice to allow larger batch sizes. There is already support in ESM_sampler for handling seed_seq arguments that are lists of sequences instead of single strings.

Just to note that when we tested this, it doesn't seem to speed up sampling. Increasing to a batch size of 2 just makes each iteration take twice as long, So for ESM models, at least with the GPUs we've been testing on, support for batching seems to increase complexity of the code without giving any speed benefits.


Need someone to continue the project

Hello,
We would like hire someone to continue this awesome project. Is there anyone can continue the project with the following features implemented?

The rest IPP Versions.
Auth feature
AirPrint on iOS Supported (capable to receive image/urf)
Convert to PDF for the job received.
Specify this is printer or proxy and forward to the specified printer

I also try to convert to PDF for the job received.