Fix highlighting, scrolling, and selecting in "new chat" section list

To highlight selected row properly in "new chat" section list, we need to do some calculations to be able to go from "selected index" to "section, index within section". For scrolling (on desktop), we need to be able to go from "selected index" to "index in flat list", where flat list contains rows, sections, and placeholders.
Pressing "enter" key also did not work correctly because to determine which item was highlighted it was using the list from before transforming it into sections. So I made it keep an additional list of flattened sections, that keeps the order within sections but all items are in one list for easy access.

ready to review cc @keybase/react-hackers


Expected expression, found: QUESTION in Shader Editor

Godot version:

3.2.3
Steps to reproduce:

Create a shader with this code

shader_type canvas_item;

void fragment() {
	COLOR = vec4(0, 0, 0, );
}

On master it says COMMA instead of QUESTION.

Still happening in 4.4 dev


Setup fails

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem. You can copy the version
information from the tool's About dialog here as well.

Follow the default setup for either the NetCore or NetFramework Exe files and it fails with : -
Runtime error at (10:171):
Out Of Range.


Hacktoberfest: Problem Solving

Everybody are welcome to add a problem with it's solution in any language
Please add the *problem, example then the solution.
File name  has to be as the problem name or function name

@moogacs can you assign this issue to me...

@himanshujain171  you are welcome to add a problem with a solution, there's no need to assigned to you


Get History of events, teams and Roles of a Member

Create an endpoint that given a member, it delivers a list containing an event, team, and role. For instance, for member Diogo Pacheco, it delivers:
[
     {26, DevTeam, Member},
     {27, DevTeam, Member},
     {28, DevTeam, Team Leader},
     {29, DevTeam, Team Leader}
]

Endpoint is in this branch. Still missing swagger though.


Intel d415 IR stream in RGB coordinates

Is it possible to obtain the depth stream from the d415 camera converted to the RGB camera's coordinates at the source? i.e. similar to what the "align" function does, but without the computing resource investment.
Thanks you

Not as far as I know.  The Vision Processor D4 hardware component inside the camera performs operations such as image rectification, described in the brochure image in the link below, and then the data travels through the USB cable to the computing device.
https://www.stemmer-imaging.com/media/uploads/cameras/13/136603-Intel-RealSense-D4-Vision-Processor-Brochure.pdf
If freeing up CPU resources is a concern for you, then you could try "offloading" CPU work onto a GPU.   For devices with an Nvidia GPU, this can be done by building Librealsense with CUDA support.
For non-Nvidia GPUs, there is alternatively "GLSL Processing Blocks" in Librealsense, though this method may be ineffective on low-power devices.
https://github.com/IntelRealSense/librealsense/issues/4905#issuecomment-533854888

This case will be closed after the next 7 days  if there are no further responses.  Thanks!


POD_NAME variable refers to wrong container instead of KubeArmor container

Bug Report
Short Description
Basically all the logs stored inside KubeArmor container whenever any activity is triggered whether it's against or validating to the  applied KuberArmorPolicy in the Kubernetes cluster. Since, all logs are stored inside /tmp/kubearmor.logs. Therefore user need to exec inside that KubeArmor pod to see those logs.
But in the documentation logs are checked inside wrong container. Where POD_NAME variable referencing to wrong pod instead of KubeArmor pod.

Expected behavior
Logs should be checked inside KubeArmor pod instead of other pods. For the same POD_NAME need to replace.
Discussion
For more, see to slack discussion.

I will love to work on this issue.
/assign me.

Lets not use /tmp/kubearmor.log directly for alerts/telemetry purpose ... We have a karmor tool (karmor log --logFilter all) that can be used to get the alerts/telemetry. Going forward we may not use /tmp/kubearmor.log to store the alerts/telemetry.

the purpose of /tmp/kubearmor.log is for debugging and quickly checking if KubeArmor works as we expected.
As Rahul and Barun mentioned, we need to use a proper way (e.g., karmor) for alerts and telemetry.

Ok, got it. Basically, we have to use standard way to see the logs and i.e. karmor log --json. And this also seems much convenient.


Unwanted feature values retrieved when no feature flags set within Azure App Configuration

I've been working with your library for some time now, and have got it working so that a list of feature flags is passed through an API of my own creation, via Azure App Configuration and the use of this package. However, is there a way to limit what feature store featureManager.GetFeatureNamesAsync() has access to? I only want to poll my AAC connection with this method, but currently, if no features are found within my instance of Azure App Configuration, it will fall back to using what appears to be environment variables of some kind?
TL:DR - Is there a way to limit the featureManager to access only one store/provider?

It is possible to pass in an instance of IConfiguration that is separate from the apps default IConfiguration instance that contains non feature related settings. This requires building a separate IConfiguration instance.
I would say if you added an empty FeatureManagement section that should work to hide the unwanted values. It seems like a bug and I'll look into fixing it. It sounds like that might be the best approach here.
Another option is to provide a setting in the feature manager to disable the functionality that falls back to the root of the IConfiguration instance. Although, I'd like to just use the empty FeatureManagement section approach.

Thanks for the reply @jimmyca15 - I do like the approach of leaving an empty section within AppSettings to fall back on, though I'm unsure currently whether it's something that I've done wrong, or something that perhaps isn't working correctly with the fallback mechanism.
Let me know if you need any more information from me regarding this, as I'm quite keen to avoid displaying unwanted configuration to callers of my app/api.
Just as a lil side note, below is the empty section of FeatureManagement in my appSettings.json file.
    "FeatureManagement": {
    },

Just to make you aware. I had passed in the whole configuration instance into the AddFeatureManagement service before, which led me to the above error. I had looked at some tutorials on how other users have integrated this functionality, and this led me to just passing in the blank FeatureManagement section within configuration as below:
services.AddFeatureManagement(Configuration.GetSection("FeatureManagement"))
After building the solution with the above line in, and removing all features within my feature manager, I had the expected result of an empty list within the response of my API call. Good stuff! Thank you for replying once again, but I'll close this as resolved.


when logging in on the lock screen or tty terminal interface : nknown error occurred

howdy version:Howdy 3.0.0 BETA
Linux distribution:debian 12.4 bookworm
de: gnome 43.9
dm:gdm
When I enter the graphical interface, Howdy works fine, but when I lock the screen or log out and log back in, an error occurs.
I tried logging in using the tty interface and encountered the same error, along with an error code.

login:
Traceback (most recent call last):
File "/lib/security/howdy/compare.py",line 262, in
hist_total = np.sum(hist)
AttributeError: module 'numpy' has no attribute 'sum'
Password：
nknown error:

I have confirmed that the numpy module is installed, and executing "python3 /lib/security/howdy/compare.py" did not result in any errors.

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable):
Howdy version (sudo howdy version):

我好像找到原因了
重新安装numpy模块解决了
sudo apt reinstall python3-numpy


Fix code sample in the .NET platform page

@tellarin Thank you

@lindexi, thank you for spotting the error! 👍


Auto close IW when another marker is clicked

Hello,
I applied a fix for a project so that when an infowindow opens, the others close when clicking on a marker.

/dist/markers-on-map-1.4.0.js

Delete var momInfoWindow; line 419
Add var momInfoWindow = null; line 99
See you soon on the web

Hi,
Thank you for the suggestion.
I have designed the info window element to show only one info window.
I will add (ASAP) an init option that will allow multiple info windows.
Thanks.
Furkan.


Giving permissions when containing page is shown breaks the app

Problem description
When a new page is created containing the scanner view and its start() method gets called, a permission dialog will be shown when the camera permission has not been given yet. When accepting the permission request, the app content will slide to the right and the user will be left with an unusable white screen.
Environment

Tabris.js version: 2.4.0
Device: Samsung Galaxy S8
OS: Android 8

Code snippet
const {Button, Page, NavigationView, ui} = require('tabris');

let navigationView;
let scannerView;
let button;

ui.contentView.append(
  navigationView = new NavigationView({left: 0, top: 0, right: 0, bottom: 0}),
  button         = new Button({centerX: 0, centerY: 0, text: 'Open scanner page'})
                    .on({tap: () => openScannerPage()})
)

function openScannerPage() {
  button.dispose();
  navigationView.append(
    new Page().append(
      scannerView = new esbarcodescanner.BarcodeScannerView({left: 0, right: 0, top: 0, bottom: 0})
    )
  )
  .appendTo(ui.contentView);
  scannerView.start();
}

We could reproduce the issue and are looking into a solution.

The issue is caused by a regression in tabris-android 2.4.0. It is fixed in 2.4.1. Closing this issue.


PR/Branch scanning support

Introduces a DSL to configure git repository pull/merge request and branch scanning for a number of git hosting providers. This mechanism will iterate over the corresponding refs and allow the user to emit pipeline configs per ref via templating.
The main moving parts are as follows:

DSL lexicon to allow user to define a means to enumerate interesting git refs (pull requests, etc.), generally via provider API (e.g., GitHub)

User provides repository and auth details, as well as an option regex filter to match interesting refs

API clients for common git repo hosting providers to perform the work of fetching and digesting pull request information
DSL mechanism for securely retrieving (read: encrypted), arbitrary, user-configured data; this is intended to facilitate configuring authentication secrets used by the internat API clients
The ability to interpolate and emit configurations from templates; in Groovy, this comes for free by virtue of the fact that Groovy is a general-purpose programming language

Merging so we can dogfood the workflow on build.gocd.org.


Make bodyparser limit configurable to allow for posts larger than 100kB

Added support for a config option (under server) that sets the bodyparser limit. Currently this limit is the default of express: 100kB.
grunt validate has some trouble with csscomb and shell:ember:test but this seems to be a problem with my dev environment. Will try to get that fixed and run tests locally again.

This makes sense to me. Is there a sensible default that we could use to make the experience better for users without introducing a new setting?

Looking at word count, the 100kB default can be called sensible as it gives your roughly 15k words. #5998 on the other hand tells it's not enough for everyone.
If we up it to 1mb, you could post a short novel, so that seems like a solid upper limit to me. What do you think @sebgie?
A configurable limit value might be a thing to put on the wish list if someone really needs it.

I think this solution would be spot on! Increase the limit to 1mb and see if there is demand for a config option :+1: (cc: @ErisDS?). Would you mind updating this PR to increase the limit?

Oops, thought I'd get away with a minor update without running grunt validate. Will update again with correct quote marks.

Ok, updated and passing checks.

Sorry for nit-picking, but the variable looks really out of place, inlining the string twice would be better. Again, I'm so sorry. :wink:

No problem, was not sure about it myself ;).

@reinoudk yes, please squash the tow commits, then it's good to go :+1:.


Cannot submit API Projects (Submitting refreshes the page)

Challenge Name
Timestamp Microservice
Issue Description
I cannot submit any API Project after completion. When I enter the glitch and Github URL and press submit, it just refreshes the page.
I have not used the boilerplate given by FCC and created my project from scratch. Please let me know if that's a problem.
Glitch URL App
https://timestamp-fcc-.glitch.me/
Glitch URL Project
https://glitch.com/edit/#!/timestamp-fcc-
Github
https://github.com/anku255/FreeCodeCamp/tree/master/API%20Projects/Timestamp%20Microservice
Browser Information

Browser Name, Version:  Chrome 62.0.3202.94
Operating System: Windows 10
Mobile, Desktop, or Tablet: Asus Laptop

We need help in assessment. Can any one check and confirm the issue in submissions?
@anku255 do you have any screenshots? Could you also check your browser's Dev Tools console, for logs if any?

@anku255 were you able to get any support in the Chat rooms and the forum?

@raisedadead No, I posted in forum 2 days ago but nobody replied. Also, I asked in chat but no body was able to help me. Then, I had to create this jssue.

do you have any screenshots?
Could you also check your browser's Dev Tools console, for logs if any?

@raisedadead I didn't think that screenshots will be helpful. Anyway, I am posting screenshots that also contain logs.
Before Clicking Submit:

After Clicking Submit:

Is there anything else that I can help with?

@raisedadead Hey! I found the bug. The problem is in my app URL https://timestamp-fcc-.glitch.me/. Notice, before glitch, there is a dash and dot. This is making the URL invalid. I came to know about this when I tried to submit the same project in beta (beta wasn't working for me before else this would have been solved days ago). In beta, I get an error message saying URL is invalid.
So, I think you need to fix the algorithm for validating URL or maybe not if that's how you want it.  Anyways, thanks for your help.


Some suggestions for the generator

Hey I just started using volto and i found some confusing stuff during the setup for a new volto project. I have some suggestions for a better setup experience.
If you want to create a new  volto-project by using the command
npm init yo @plone/volto
You will automatically have to answer a few questions.

? Project name
-> maybe a hint would be helpful about the restriction (no blanks, etc.)

? Project description (A volto -powered Plone frontend)
-> im not sure if this is really needed but here you simply continue by pressing enter.

? Would you like to add addons? (true)
-> should be false by default (should say: would you like to add existing Volto add-ons to your project?)

? Would you like to add workspaces? (true)
-> should be false by default (and again I'm not sure if this is even needed during the setup - we discussed that in a team and tought this could be probably removed in general.)

Fixed.


Improve support for enum variants

I noticed a few issues with enum variants. For instance:
#[derive(Serialize, TS)]
#[serde(tag = "kind", content = "data")]
enum ComplexEnum {
    A,
    B { foo: String, bar: f64 },
    W(SimpleEnum),
    F { nested: SimpleEnum },
    T(i32, SimpleEnum),
    V(Vec<Series>),
}

generates the following type:
// issue 1: there is no import for the `SimpleEnum` type

export type ComplexEnum =
  | { kind: "A"; data: null } // issue 2: IMO it would be better to omit the `data` property
  | {
    kind: "B";
    data: {
      foo: string;
      bar: number;
    };
  }
  | { kind: "W"; data: SimpleEnum }
  | {
    kind: "F";
    data: {
      nested: SimpleEnum;
    };
  }
  | { kind: "T"; data: [number, SimpleEnum] }
  | { kind: "V"; data: Array }; // issue 3: there is no type argument passed to `Array`

I have fixed this issues in my own fork with the following commit: https://github.com/arendjr/ts-rs/commit/e7a87b1b49bfa0263460a63ab995fc133a4ba362
Unfortunately, it would be a bit unwieldy to create a PR for this as this is built on top of both #15 and #18. This is just to let you know these issues are addressed and I can create a PR for it when the other PRs are merged.

As #15 has now been merged, I have included the fixes for these issues in #25, since they partially affected generics support anyway.

Fixed as part of #25.


Select: fix case when select send executed before select receive, fixed #3862

fix for #3862
spec for bug is last, other specs i just added because seems select not tested nice.

Would be great to get this merged, as it's a bugfix.

Thanks @kostya ! 💚  for the extra specs


install via pip

refactor like in this post

falstart now available in pypi https://pypi.python.org/pypi/falstart


what the difference is between websocket_message_count and api_message_count

Hi
what the difference is between websocket_message_count and api_message_count
when broadcast a message to users which of the columns will be updated ?

For questions there is discussions section, this section is for issues
https://github.com/beyondcode/laravel-websockets/discussions


Add collect method to Collection class

I constantly find myself needing to “re-collect” nested arrays after getting them. I propose adding a collect method to the Collection class. This provides a simple syntax to achieve this:
Ex.
$items = collect([
  'taco' => [
      'cheese',
      'tortilla' => [
        'flour',
        'salt',
      ],
      'beef',
  ],
]);

//-- before (works with all methods, count is just a simple example)
$tortillaCount = collect($items->get('taco'))->count();

//-- after
$tortillaCount = $items->collect('taco')->count();

I already include this using the macro method, however, I thought I'd share.

Why not just using a macro for that?

I thought this might be helpful to others as well. ¯\_(ツ)_/¯

If this is rejected try sending a PR for this package https://github.com/spatie/laravel-collection-macros I think it'll be a good addition there 😄

@fernandobandeira cool, thanks!

As nice as it looks, I do not think that this:
$items = collect([
    'taco' => [
        'cheese',
        'tortilla' => [
            'flour',
            'salt',
        ],
        'beef',
],
]);

is a standard collection. Therefore, it might not be useful to others. I'd suggest using your own macro.
(For future reference: it always helps your PR, when you add some tests)

Will hold for macro for now... see how it is received there.

@joshwhatk I'd accept a fully tested PR for this at https://github.com/spatie/laravel-collection-macros

@freekmurze Awesome, thanks! I'll put one in with added tests!


python311Packages.ipyvuetify: 1.9.1 -> 1.9.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.ipyvuetify is: Jupyter widgets based on Vuetify UI Components.
meta.homepage for python311Packages.ipyvuetify is: https://github.com/mariobuikhuizen/ipyvuetify
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.9.3 with grep in /nix/store/1lg4dlml0bgvrb43qxaj893ahm2mvmdx-python3.11-ipyvuetify-1.9.3
found 1.9.3 in filename of file in /nix/store/1lg4dlml0bgvrb43qxaj893ahm2mvmdx-python3.11-ipyvuetify-1.9.3

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.ipyvuetify
python312Packages.ipyvuetify

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1lg4dlml0bgvrb43qxaj893ahm2mvmdx-python3.11-ipyvuetify-1.9.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.ipyvuetify https://github.com/r-ryantm/nixpkgs/archive/7eea2e475a1477fa7cc84759d1c4f87d6998c299.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7eea2e475a1477fa7cc84759d1c4f87d6998c299#python311Packages.ipyvuetify

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1lg4dlml0bgvrb43qxaj893ahm2mvmdx-python3.11-ipyvuetify-1.9.3
ls -la /nix/store/1lg4dlml0bgvrb43qxaj893ahm2mvmdx-python3.11-ipyvuetify-1.9.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

4 packages built:

python311Packages.ipyvuetify
python311Packages.ipyvuetify.dist
python312Packages.ipyvuetify
python312Packages.ipyvuetify.dist

Maintainer pings
cc @drewrisinger for testing.

Merge conflict, please open a new PR.


why is checkperiod set to 600 by default?

Just wondering why zero isn't the default, since by default if the cached value is expired it returns undefined anyway.
Is it used to stop the cache from filling up with expired values?  I would expect an expired value to be removed from the cache on the 'get' call - or is this not the case?

Hi,
expired values are internally removed on a .get() and after every checkperiod.
It's set to default 600 to remove deprecated keys from the cache every 10 Minutes.
Otherwise unused keys (never called with .get()) would stay within the node-cache instance forever.
So the default 600 will prevent you from a possible memory leak if you use it in the simplest way new NodeCache();

Ok good to know.
From my point of view, if I add a value to cache then I am responsible for removing it.  It's nice there's an mechanism to automatically remove expired values but I would of expected this to be off by default and turned on as and when required.
No big deal either way though.
Thanks for your quick response.


Updates AhrProvider to fetch from new graphql source

Description and Motivation

[x] updates DatasetId to graphql versions
[x] updates getDatasetId()
[x] updates metadata
[x] updates tests
[x] updates MetricConfigs to use count metrics

Has this been tested? How?

[x] renders national and state level data
[x] render data that makes sense (not weird ALL way higher and pct_share not making sense
[x] calculates shares for all race groups, not just split between white nh and hispanic
[ ] voter and preventable working
[ ] rerun pipelines on dev
[ ] rerun pipelines on prod

Screenshots (if appropriate)
Types of changes
(leave all that apply)

New content or feature
Refactor / chore

New frontend preview link is below in the Netlify comment 😎

Need to fix issue; pct_share of sub pop. only calculating values for White NH and Hispanic

Need to fix issue; pct_share of sub pop. only calculating values for White NH and Hispanic

Fixed


cmake enableing

Hi,
I think it's better to have the changes in your fork, too. I don't think I will maintain the lib itself on a regular basis, but maybe I'll add the tests here, too. You'll get anyother PR when that is done. I saw you created a branch for emule and if you don't intend to send this stuff to me, it's better to have it all here.

I merged this to a new github_contrib branch to avoid any future conflicts when merging from the origin on sourceforge.

Also: Thank you very much for your contribution!


console message: esm.sh cannot find requested imports

I'm making a comparison of module CDNs.
I have a series of test pages for module CDNs:

https://install.guide/test/esm-pinned.html
https://install.guide/test/esm-unpinned.html
https://install.guide/test/jsdelivr-pinned.html
https://install.guide/test/jsdelivr-unpinned.html
https://install.guide/test/jspm-importmaps.html
https://install.guide/test/jspm-pinned.html
https://install.guide/test/jspm-unpinned.html
https://install.guide/test/skypack-pinned.html
https://install.guide/test/skypack-unpinned.html

When I try esm.sh, I get a console error:
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'LitElement'

I'm using (for a pinned version):
import { LitElement, html } from<EMAIL_ADDRESS>and for an unpinned version:
import { LitElement, html } from 'https://esm.sh/lit';
I'm following the import syntax I saw at https://esm.sh/.
The page loads correctly with jspm.dev or Skypack module CDNs.
How do I make esm.sh work correctly? Am I making an error in my import syntax or is this a bug with esm.sh or esbuild?

i didn't test all packages since i built this for alephjs, in http://esm.sh/?test you can see some packages I focused on. i will look into it, thanks!
btw install.guide looks awesome, nice work


Avoid multiple reload operations when configure multiple interfaces

In the event that we're configuring several interfaces (e.g. an instance with several attached ENIs (re)boots), we would previously issue multiple approximately simultaneous networkctl reload commands. With this PR, we ensure that when we have multiple concurrent setup-policy-routes processes running, only one of them ends up issuing the reload command.
At a high level, we accomplish this by creating a directory in a well-known location, and creating a file in it. Then, before exiting, we remove the file we created and attempt to remove the directory. If the rmdir call succeeds, then we've determined that there were no other running setup-policy-routes processes and reload networkd if necessary. If rmdir fails, then we've determined that another setup-policy-routes process is running and exit, letting the other process handle the reload.
Naturally, there's subtlety involved in any sort of interprocess coordination like this. A SIGKILL to a setup-policy-routes process would prevent state cleanup, essentially leading to a deadlock in which nobody can reload networkd without manual cleanup, but otherwise I believe I've covered the failure cases. Careful review and plenty of questions would be appreciated, though.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Rebased onto the latest main branch after merging the other open PRs. @halfdime-code are you good with this?


non deterministic behaviour using graphsage and pool

I'm currently facing issues som while using graphsage as gnn and graphsage_aggreagte_type equal to 'pool': despite tsetting seeds and using a single worker to achieve a deteministic behaviour i cannot succeed in that. On the other hand, when using other aggregation types (mean, lstm and gcn) I am able reproduce the same exact results. So I was wondering if I am missing some additional steps or if that is to be considered a bug.
For reference I used the code for trec text classification with graphsage_bi_fuse_dependency.yaml config. Basically I just did some changes to  the config file to use only a single worker and adapted the code slightly to run in colab.

Thank you for pointing out the problem!
We are currently looking for possible bugs, please stay tuned!

Close this issue due to inactivity. Please reopen it when necessary.


Modern logging

Brief feature description
Stream based logging is quite slow and not the recommended modern way.
Detailed information
There are fast and modern alternatives, like fmt which uses the new C++20 format API.
We should have a look at fmt and decide if it's possible to make it stack based or maybe implement the C++20 format API based on the cxx::string.

Closing, as superseded by #1345


Unable to load SDL2.dll when moving the output dir of Veldrid

So, I have a bit of a weird use case.
I have Veldrid in a "Renderer" module that exists it it's own folder that I am changing the output dir in the project file into a Renderers/ folder.
I have traced the bug down to the NativeLibraryLoader dependency searching for the native sdl2.dll file in the Application's base directory, rather than the location of the veldrid renderer.  This leads to Veldrid throwing an exception on startup unable to find the sdl library.

I think I understand your scenario, but I'm not sure how to fix it in general or automatically. What you could do is force SDL2 to load from a particular place in your code, before you call into Veldrid.SDL2 (or use Veldrid.StartupUtilities, etc.). Something like this should work:
// Force SDL2 native library to load from a non-standard location.
new NativeLibrary("my/custom/sdl2/path");

// Regular startup code. Veldrid.SDL2 will use the pre-loaded SDL2 binary.


produce json for error conditions

We should have @Produces(MediaType.APPLICATION_JSON) for all our endpoints even if they produce no content because all errors are reported as json so if we don't have annotation, then jetty tries to generate text/plain output and fails with "MessageBodyWriter not found for media type=text/plain, type=class" error message. So for all errors we get back 500 instead of the actual error code.

👍


Concurrency Error in Call to do_previous

We got a report of this:
2017-12-24T13:46:12,638][ERROR][logstash.inputs.udp      ] Exception in inputworker {"exception"=>#<ConcurrencyError: Detected invalid array contents due to unsynchronized modifications with concurrent users>, "backtrace"=>[
"org/jruby/RubyArray.java:1147:in `<<'", "org/jruby/RubyArray.java:1168:in `push'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:211:in `buffer'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:269:in `do_previous'",
"org/jruby/RubyMethod.java:124:in `call'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:205:in `decode'",
 "org/jruby/RubyArray.java:1613:in `each'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:198:in `decode'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:118:in `inputworker'",
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:89:in `udp_listener'"
]}

It seems like this is coming from the (as far as I can see) asynchronous flush functionality somehow accessing the @buffer field from 2 threads (I verified that the udp input does correctly clone the codec, so it must be coming from the codec itself here).
Maybe related to the random failures here #46

This is impacting our production environment, causing periodic data loss from our application error logs (nothing critical yet), and could also impact our SQL activity data stream collection (also using UDP input) and our compliance status.
We're presently being alerted when data stops being ingested for that index pattern, and restarting Logstash - not a solution - and will be considering alternate options (filebeat or custom UDP proxy creation).

@jordansissel @andrewvc fyi (in case you want to schedule this in some form :))

Yes, we should fix this bug, would you have the spare cycles to tackle this next week @original-brownbear ?

@andrewvc sure thing, top of the pile then :)

@jkelastic I think this is an unrelated PQ bug, I've opened a new issue for it here: https://github.com/elastic/logstash/issues/9104

@jkelastic Could you please provide complete environment information in elastic/logstash#9104  Exact LS version and OS version?

@colinsurprenant
The error below just happened last night on one of our production Logstash servers and stopped accepting traffic on one UDP port - only restarting Logstash allowed it to recover (logstash 5.6.5).
Version:
logstash 5.6.5
jruby 1.7.27 (1.9.3p551) 2017-05-11 8cdb01a on Java HotSpot(TM) 64-Bit Server VM 1.8.0_131-b11 +jit [Windows Server 2012 R2-amd64]
java 1.8.0_131 (Oracle Corporation)
jvm Java HotSpot(TM) 64-Bit Server VM / 25.131-b11
logstash-codec-multiline-3.0.9
Error:

[2018-03-07T18:57:14,513][ERROR][logstash.inputs.udp   ] Exception in inputworker {"exception"=>#<ConcurrencyError: Detected invalid array contents due to unsynchronized modifications with concurrent users>, "backtrace"=>["org/jruby/RubyArray.java:1147:in <<'", "org/jruby/RubyArray.java:1168:in push'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:211:in buffer'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:269:in do_previous'", "org/jruby/RubyMethod.java:124:in call'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:205:in decode'", "org/jruby/RubyArray.java:1613:in each'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:198:in decode'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:118:in inputworker'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:89:in udp_listener'"]}
[2018-03-07T18:57:14,513][ERROR][logstash.inputs.udp   ] Exception in inputworker {"exception"=>#<ConcurrencyError: Detected invalid array contents due to unsynchronized modifications with concurrent users>, "backtrace"=>["org/jruby/RubyArray.java:1147:in <<'", "org/jruby/RubyArray.java:1168:in push'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:211:in buffer'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:269:in do_previous'", "org/jruby/RubyMethod.java:124:in call'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:205:in decode'", "org/jruby/RubyArray.java:1613:in each'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:198:in decode'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:118:in inputworker'", "E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-input-udp-3.1.1/lib/logstash/inputs/udp.rb:89:in udp_listener'"]}

@jkelastic you are reporting using the v3.0.9 of the multiline codec which should have fixed this issue but your logs tells differently:
"E:/Logstash/vendor/bundle/jruby/1.9/gems/logstash-codec-multiline-3.0.5/lib/logstash/codecs/multiline.rb:211:in buffer'",

It seems you are in fact running v3.0.5. Can you upgrade and reassess?

@akatakatoo can you try uninstalling and re-installing logstash-codec-multiline to see if it will get it to v3.0.9?

Just removed and installed it (with logstash stopped) and although it indicated it could not remove it as it had dependencies, the correct version of files appear to be in place.  Now to wait and see if it happens again.  Thanks.
E:\elkbinaries\logstash\bin>logstash-plugin remove  logstash-codec-multiline
Failed to remove "logstash-codec-multiline" because the following plugins or libraries depend on it:

logstash-input-beats
logstash-input-file
logstash-input-lumberjack
logstash-input-s3

E:\elkbinaries\logstash\bin>logstash-plugin install logstash-codec-multiline
Validating logstash-codec-multiline
Installing logstash-codec-multiline
Installation successful
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\cache
03/08/2018  02:33 PM            22,016 logstash-codec-multiline-3.0.9.gem
1 File(s)         22,016 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems
12/11/2017  02:16 PM              logstash-codec-multiline-3.0.8
03/08/2018  02:33 PM              logstash-codec-multiline-3.0.9
0 File(s)              0 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems\logstash-codec-multiline-3.0.8
12/11/2017  02:16 PM             1,303 logstash-codec-multiline.gemspec
1 File(s)          1,303 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems\logstash-codec-multiline-3.0.8\lib\logstash\codecs
12/11/2017  02:16 PM             9,166 multiline.rb
1 File(s)          9,166 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems\logstash-codec-multiline-3.0.9
03/08/2018  02:33 PM             1,303 logstash-codec-multiline.gemspec
1 File(s)          1,303 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems\logstash-codec-multiline-3.0.9\lib\logstash\codecs
03/08/2018  02:33 PM             9,226 multiline.rb
1 File(s)          9,226 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\gems\logstash-codec-multiline-3.0.9\spec\codecs
03/08/2018  02:33 PM            11,500 multiline_spec.rb
1 File(s)         11,500 bytes
Directory of E:\elkbinaries\logstash\vendor\bundle\jruby\1.9\specifications
12/11/2017  02:17 PM             1,923 logstash-codec-multiline-3.0.8.gemspec
03/08/2018  02:33 PM             1,965 logstash-codec-multiline-3.0.9.gemspec
2 File(s)          3,888 bytes

@akatakatoo did that solve your problem?

The problem has not occurred since the 3.0.9 update, which is a great start.  There can be weeks or months between the error occurrences, so I'll continue to monitor for a bit.  Thanks.

@akatakatoo thanks for the followup. I will go ahead and close this issue. Please feel free to re-open if you think it is still not solved at some point.


v2.8.0 Breaks with ansi-to-html-umd

2.7.1 works great.
After npm installing with 2.8.0:
(webpack)-hot-middleware/client.js?path=http://localhost:3001/__webpack_hmr
[0] Module not found: Error: Cannot resolve module 'ansi-to-html-umd' in /project/node_modules/webpack-hot-middleware
[0] resolve module ansi-to-html-umd in /project/node_modules/webpack-hot-middleware
[0]   looking for modules in /project/src
[0]     /project/src/ansi-to-html-umd doesn't exist (module as directory)
[0]     resolve 'file' ansi-to-html-umd in /project/src
[0]       resolve file
[0]         /project/src/ansi-to-html-umd doesn't exist
[0]         /project/src/ansi-to-html-umd.json doesn't exist
[0]         /project/src/ansi-to-html-umd.js doesn't exist
[0]         /project/src/ansi-to-html-umd.jsx doesn't exist
[0]   looking for modules in /project/node_modules
[0]     /project/node_modules/ansi-to-html-umd doesn't exist (module as directory)
[0]     resolve 'file' ansi-to-html-umd in /project/node_modules
[0]       resolve file
[0]         /project/node_modules/ansi-to-html-umd doesn't exist
[0]         /project/node_modules/ansi-to-html-umd.json doesn't exist
[0]         /project/node_modules/ansi-to-html-umd.js doesn't exist
[0]         /project/node_modules/ansi-to-html-umd.jsx doesn't exist
[0]   looking for modules in /project/node_modules/webpack-hot-middleware/node_modules
[0]     /project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd doesn't exist (module as directory)
[0]     resolve 'file' ansi-to-html-umd in /project/node_modules/webpack-hot-middleware/node_modules
[0]       resolve file
[0]         /project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd doesn't exist
[0]         /project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.json doesn't exist
[0]         /project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.js doesn't exist
[0]         /project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.jsx doesn't exist
[0] [/project/src/ansi-to-html-umd]
[0] [/project/src/ansi-to-html-umd]
[0] [/project/src/ansi-to-html-umd.json]
[0] [/project/src/ansi-to-html-umd.js]
[0] [/project/src/ansi-to-html-umd.jsx]
[0] [/project/node_modules/ansi-to-html-umd]
[0] [/project/node_modules/ansi-to-html-umd]
[0] [/project/node_modules/ansi-to-html-umd.json]
[0] [/project/node_modules/ansi-to-html-umd.js]
[0] [/project/node_modules/ansi-to-html-umd.jsx]
[0] [/project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd]
[0] [/project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd]
[0] [/project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.json]
[0] [/project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.js]
[0] [/project/node_modules/webpack-hot-middleware/node_modules/ansi-to-html-umd.jsx]
[0]  @ (webpack)-hot-middleware/client.js?path=http://localhost:3001/__webpack_hmr 82:11-38
[0] ./src/helpers/ApiClient.js

Oh whoops, looks like we added this as a devDependency by mistake.
I'm not at a computer at the moment, will release a fix when I'm next able to.
@rossipedia - would you be able to send a quick PR in?

sure thing

Thanks for identifying/fixing this so quickly!

Out of interest, how did you find this so quickly? Are you using greenkeeper or something similar?

@glenjamin can you bump and publish to npm. this is breaking my app as well

If it's causing an immediate problem I suggest pegging to 2.7.1 - as soon as I'm near a computer I'll deprecate 2.8.0 and publish 2.8.1 the fixed version

Ok, that's resolved now - sorry for the problems everyone!
Anyone know a good way to make sure this doesn't happen again? The tests will always pass if a dep is in devDependencies instead of dependencies

No idea off the top of my head. Need to be able to determine whether a package is required at runtime or not.j
Honestly, it was just my bad, I should've thought it through just a bit longer.

It's easily done, I read the diff and checked it locally, but didn't realise.
https://github.com/BohdanTkachenko/eslint-plugin-require-path-exists is the closest thing I can find, but it doesn't that any logic to treat devDependencies differently.
I've suggested such an extension in https://github.com/BohdanTkachenko/eslint-plugin-require-path-exists/issues/15


Recommendation: Default path for publishing to blog

Hello,
I think it makes perfect sense to have the default path set to the file.name.lowerCase().spaceToHyphen(). Then the user could edit that name to the desired path.
What do you guys think?
Jason

Dear @ycmjason,
Thank you for your suggestion!
Could you please give an example of how this would change from the current to the desired state?
What would have to change where and how would it affect the user?
Thankfully,
-Robert

Sure. But apparently to me now the BlogPost controller doesn't have the file in the scope? I couldn't figure the code myself.
In here,
https://github.com/classeur/classeur/blob/master/src/blogs/github/githubBlogPostForm.html#L14
<md-input-container>
  <label>File path</label>
  <input autocomplete="off" ng-model="form.subForm.filePath">
</md-input-container>

If we could pre-define form.subForm.filePath to be the file.name, this would be done.
This should not be hard. However, in
https://github.com/classeur/classeur/blob/master/src/blogs/blogs.js#L47
I think the file is not in scope when I inspect it with the debugger. So I guess we need to change something in its parent to make this happen. However I couldn't figure out where the parent is. HAHA.
Sorry for my limited knowledge.
I would recommend the following to manipulate the file.name to make it a valid file name.
file.name
  .toLowerCase()
  .replace(/ /g, '-') // replace spaces with dash
  .replace(/[!@#$%^&*()_+=~`\|\[\]\{\};:'",./<>?\\]/g, '') // remove punctuations

This is all I could do for now. I am sure it would not be hard at all if you are familiar with the infrastructure of this project.
I hope this suggestion would be accepted and implemented.

Just realize I didn't quite answer @orschiro 's question. Let me make this clearer.

Could you please give an example of how this would change from the current to the desired state?

The current implementation have an empty filePath, which is fine. But it makes perfect sense to have the file.name as the default value.

What would have to change where and how would it affect the user?

This would not have much effect on the user experience. It's just an extra option that people could use. If that user wants a custom name, nothing will stop them from doing so. But I bet 70% of the time, people would just use the file name as the filePath.

Hi @ycmjason
Fair enough, we can have that for the next release.


Support of Vue3?

Hi there,
vuex-typex is a great tool to provide typings to store methods, but it seem to be not adapted to Vue 3. In Vue 3, Vue root is not global, so main Vue root object is created via createApp in main.ts and in store/index.ts one can't just do Vue.use(Vuex) (use is not present in that). Instead, main.ts reads:
...
import store from './store'

createApp(App)
    .use(store)
    .use(router)
    .mount('#app')

But without using Vue.use(Vuex) in store/index.ts going like
import { Store } from 'vuex'
import { getStoreBuilder } from 'vuex-typex'
import { UserState } from './modules/user'

export interface RootState {
    user: UserState
}

const store: Store<RootState> = getStoreBuilder<RootState>().vuexStore();
export default store

gives a runtime error "[vuex] must call Vue.use(Vuex) before creating a store instance"; moreover, TS gives a compile-time error:

const store: Store
Property 'install' is missing in type 'import("/node_modules/vuex-typex/node_modules/vuex/types/index").Store<import("/src/store/index").RootState>' but required in type 'import("/node_modules/vuex/types/index").Store<import("/src/store/index").RootState>'.ts(2741)
index.d.ts(18, 3): 'install' is declared here.

How can I fix this? Is support of Vue3 available or it requires updating Vuex-typex itself?

I suspect that it will require a new version of vuex-typex which targets
@.*** to make this work.
If you are interested in making this work please  start a PR, and I’ll work
through it with you.
On Thu, 19 May 2022 at 8:16 PM, Yakov Litvin @.***>
wrote:

Hi there,
vuex-typex is a great tool to provide typings to store methods, but it
seem to be not adapted to Vue 3. In Vue 3, Vue root is not global, so main
Vue root object is created via createApp in main.ts and in store/index.ts
one can't just do Vue.use(Vuex) (use is not present in that). Instead,
main.ts reads:
...
import store from './store'
createApp(App)
.use(store)
.use(router)
.mount('#app')
But without using Vue.use(Vuex) in store/index.ts going like
import { Store } from 'vuex'
import { getStoreBuilder } from 'vuex-typex'
import { UserState } from './modules/user'
export interface RootState {
user: UserState
}
const store: Store = getStoreBuilder().vuexStore();
export default store
gives a runtime error "[vuex] must call Vue.use(Vuex) before creating a
store instance"; moreover, TS gives a compile-time error:
const store: Store
Property 'install' is missing in type
'import("/node_modules/vuex-typex/node_modules/vuex/types/index").Store<import("/src/store/index").RootState>'
but required in type
'import("/node_modules/vuex/types/index").Store<import("/src/store/index").RootState>'.ts(2741)
index.d.ts(18, 3): 'install' is declared here.
How can I fix this? Is support of Vue3 available or it requires updating
Vuex-typex itself?
—
Reply to this email directly, view it on GitHub
https://github.com/mrcrowl/vuex-typex/issues/46, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANWRIHO57EX4KOHF3JOKHDVKX2HDANCNFSM5WLLHCOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Well, I am interested indeed, but I'm not sure where to start from, what changes to put so that I can create a PR. Any suggestions?

Start by forking the repo, clone locally, and run yarn to install
dependencies.
Then upgrade the dependencies in package.json to the ones that you want.
Presumably vue and vuex will need the 3.x versions.
Then see what breaks! :D
Since this will be a breaking change, it will need a new major version in
the package.json too. It might be worth matching this with Vue and making
it v3.x.x
Tbh, I'm a bit out of the loop with Vue and Vuex these days... so I haven't
kept a close eye of what's been going on.
On Sat, 21 May 2022 at 22:39, Yakov Litvin @.***> wrote:

Well, I am interested indeed, but I'm not sure where to start from, what
changes to put so that I can create a PR. Any suggestions?
—
Reply to this email directly, view it on GitHub
https://github.com/mrcrowl/vuex-typex/issues/46#issuecomment-1133627461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANWRIBJUOXKJO5EMCBJZCLVLDKPTANCNFSM5WLLHCOA
.
You are receiving this because you commented.Message ID:
@.***>

Has anyone found an alternative to this? I am migrating my team's Vue 2 app and it uses this library.

Vuex has been deprecated in favour of Pinia by the Vue core team.
Pinia has built-in TypeScript support and a similar-enough API to Vuex to make migration manageable, with a few notable differences.
Vuex-typed was only created to work around the fact that early versions of Vuex gave little consideration to TypeScript.


Add ranges to investment commitment

For fields in the investment block, in addition to exact numbers, ranges are required.
I suggest:
project/
Total investment commitment in project currency, Exact
Total investment commitment in USD, Exact
Total investment commitment in project currency, Minimum
Total investment commitment in USD, Minimum
Total investment commitment in project currency, Maximum
Total investment commitment in USD,  Maximum
investments/
Investment commitment in investment currency, Exact
Investment commitment in USD, Exact
Investment commitment in investment currency, Minimum
etc

Done

Oh wait @ScatteredInk should "Total outcome payments made" have this too?

And "Total outcome payments made"?
The "etc" there is confusing me basically

No - ranges are just intended for the investment fields so the way you have done it is right. Sorry for being too lazy to type the fields out explicitly.


Rework events archive and archive search page

Its ugly. We need to fix it.
Archive search results only shows each events date, which is confusing because the event might be from 2014. It's also sorted wrong, from oldest to newest and should be from newest to oldest (maybe?).
The events table at the archive view should be styled a bit
So yea, the whole thing needs a bit of love

Event search breaks when javascript is disabled. Added a quickfix for part of the problem #95
Agree that it should be fixed.

Agreed that the page needs some love, though it's quite clear when an event is from (which year) and it's sorted correctly.

Closing because it seems the bugs have been fixed. Will create a new issue for UI related work.


Bugfix

@koijima thx. release after fixed schema-lib
https://github.com/yahoojapan/dataplatform-schema-lib/pull/14


⚠ Minecraft has degraded performance

In 2cf9bc0, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1418 ms

Resolved: Minecraft performance has improved in c7eab68.


Did you steal my source code?

are you just clone my repo as base without credit me?
the license is still under me
original: https://pub.dev/packages/sunmi_printer

What u mean with steal?
I sent you some messages about your git to send a lot of PR ( if you see. I made a lot of changes) and you didn't reply. The credit is in the license and still the last version it was a disclaimer that it was a fork of your project. But i removed because i made a lot of changes. The whole story it was you don't have the github in your project

@zigang93
https://pub.dev/documentation/sunmi_printer_plus/2.0.12/
i can put your repository again there is no problem for me i just removed because we change a lot of things. but its not a big issue for me to do that. i never said that the code was mine because the credit was there till .12


refactor(ast): use scope_id etc methods

Utilize the methods added in #7127 in oxc_ast.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7130  👈
#7129
#7128
#7127
#7126
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Merge activity

Nov 4, 9:24 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Crash

Hi! im new to this submonds thing and I assume this directory is where I should put the mod. But when I start the game it says this:
[code]
I'm sorry, but errors were detected in your script. Please correct the
errors listed below, and try again.
File "game/submods/mas-saysomething-master/mod/generation.rpy", line 27: unexpected indent
illegal_chars = ['<', '>', ':', '"', '/', '\', '|', '?', '*']
^
File "game/submods/mas-saysomething-master/mod/screen.rpy", line 1121: u'released' is not a keyword argument or valid child for the bar statement.
released Return(_fom_saysomething.RETURN_RENDER)
^
File "game/submods/mas-saysomething-master/mod/screen.rpy", line 203: invalid syntax
raise ValueError(f"Unknown code in sprite_code: {sprite_code}")
^
Ren'Py Version: Ren'Py <IP_ADDRESS>.2187
[/code]
Thank you in advance!

Hi, unsure if you solved it yourself already — you installed the wrong version of the submod, you downloaded the source code and not the release version. Go here and download it from there.


when ssh hangs, gardenctl ssh command unable to exit

What happened:
during the trouble shooting of a cluster, when gardenctl ssh command hangs after adding keys (probably due to there's sth wrong with OS), gardenctl unable to exit (by ctrl+c), the only way is to close the terminal which causing resource leakage
What you expected to happen:
gardenctl ssh is able to exit at any phase
How to reproduce it (as minimally and precisely as possible):
gardenctl ssh a problem node

Anything else we need to know:
Environment:

similar to https://github.com/gardener/gardenctl-v2/issues/139
you have to specify your internal ip using --cidr flag. The auto-detection mechanism is only a best guess
@neo-liang-sap how do you come to the conclusion that resources leaked? gardenctl creates a bastion resource and heartbeats it as long as gardenctl is running. If there is no heartbeat anymore for a certain amount of time the controller acting upon the bastion resource cleans it up.

you have to specify your internal ip using --cidr flag.

gardenctl creates a bastion resource and heartbeats it as long as gardenctl is running. If there is no heartbeat anymore for a certain amount of time the controller acting upon the bastion resource cleans it up.

Thanks! never know these two cool features!
/close

Hi @petersutter
today i met this issue again, the cidr is correct from gardenctl-v2 but gardenctl ssh just hang after adding host to known hosts
/reopen

The bastion host is available,  you can open additional ssh sessions using the command below,  any error coce after your using ssh command?

same error in new window, hang

can't reproduce the issue.  it looks like you are accessing one of the nodes from the AWS cloud provider, let me know if not. and can you provide the following information when the issue occurred again next time.

Target Cluster
telnet bastion IP 22 , the IP is shown after the message Bastion host becomes available at xxx.xxx.xxx.xxx", and check the IP whether reachable as well?

check the IP whether reachable as well

As the key has exchanged, definitely the bastion is reachable but unable to finish ssh login

accessing one of the nodes from the AWS cloud provider

Yes
Will provide cluster details in private slack channel

we should properly handle the signals (e.g. SIGINT) in gardenctl so that the user is not forced to terminated the shell

from the man page of the ssh command:
ESCAPE CHARACTERS
     When a pseudo-terminal has been requested, ssh supports a number of functions through the use of an escape character.

     A single tilde character can be sent as ~~ or by following the tilde by a character other than those described below.  The escape character must always follow a newline to be
     interpreted as special.  The escape character can be changed in configuration files using the EscapeChar configuration directive or on the command line by the -e option.

     The supported escapes (assuming the default ‘~’) are:

     ~.      Disconnect.

     ~^Z     Background ssh.

     ~#      List forwarded connections.

     ~&      Background ssh at logout when waiting for forwarded connection / X11 sessions to terminate.

     ~?      Display a list of escape characters.

     ~B      Send a BREAK to the remote system (only useful if the peer supports it).

     ~C      Open command line.  Currently this allows the addition of port forwardings using the -L, -R and -D options (see above).  It also allows the cancellation of existing
             port-forwardings with -KL[bind_address:]port for local, -KR[bind_address:]port for remote and -KD[bind_address:]port for dynamic port-forwardings.  !command allows the
             user to execute a local command if the PermitLocalCommand option is enabled in ssh_config(5).  Basic help is available, using the -h option.

     ~R      Request rekeying of the connection (only useful if the peer supports it).

     ~V      Decrease the verbosity (LogLevel) when errors are being written to stderr.

     ~v      Increase the verbosity (LogLevel) when errors are being written to stderr.

so with Enter ~ . you should be able to disconnect
/close


Add Travis check for trailing whitespace

We should add a Travis check to make sure that files in this repo do not have trailing whitespace (at the very least .md and .yml files).
This can either a simple grep for "\s+$" or can be a more thorough check using a Markdown linter (bonus points if we can smoothly integrate a markdown linter into Travis/git hooks, especially one that reflows lines to break at 80 columns)

@shredtechular can you take this on after tackling the style issues?

@mtlynch Absolutely, I can see what I can figure out that may help with that respect after the styling fixes are complete.


ValueError: Unable to create item in table 'priority_table' since the provided trajectory is inconsistent

Hi again, thanks for helping out with my previous issue. I'm still trying to get a custom parallel pettingzoo environment to work with but I currently get the error below. The step method of our custom environment seems to be called but I'm not sure if training is actually occurring as I can't get any logs for tensorboard.
[executor/0] Traceback (most recent call last):
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/launchpad/nodes/python/process_entry.py", line 110, in <module>
[executor/0]     app.run(main)
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/absl/app.py", line 312, in run
[executor/0]     _run_main(main, args)
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/absl/app.py", line 258, in _run_main
[executor/0]     sys.exit(main(argv))
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/launchpad/nodes/python/process_entry.py", line 105, in main
[executor/0]     functions[task_id]()
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/launchpad/nodes/python/node.py", line 72, in _construct_function
[executor/0]     return functools.partial(self._function, *args, **kwargs)()
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/launchpad/nodes/courier/node.py", line 124, in run
[executor/0]     instance.run()
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/environment_loop.py", line 538, in run
[executor/0]     result = self.run_episode()
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/environment_loop.py", line 435, in run_episode
[executor/0]     self._executor.observe(
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/systems/tf/madqn/execution.py", line 267, in observe
[executor/0]     self._adder.add(actions, next_timestep, next_extras)
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/acme/adders/reverb/transition.py", line 134, in add
[executor/0]     super().add(*args, **kwargs)
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/adders/reverb/base.py", line 426, in add
[executor/0]     self._write()
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/adders/reverb/transition.py", line 207, in _write
[executor/0]     self.write_experience_to_tables(transition, table_priorities)
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/mava/adders/reverb/base.py", line 356, in write_experience_to_tables
[executor/0]     self._writer.create_item(
[executor/0]   File "/home/{name}/.cache/pypoetry/virtualenvs/ai-market-contest-RFU3MKa2-py3.9/lib/python3.9/site-packages/reverb/trajectory_writer.py", line 352, in create_item
[executor/0]     self._writer.CreateItem(table, priority,
[executor/0] ValueError: Unable to create item in table 'priority_table' since the provided trajectory is inconsistent with the table signature.  The table signature says it is a float, but you are adding doubles. Please check that the observation types are consistent between your spec and what you get from the env/add to replay.

Hi @vanpra , please see proposed solution here - https://github.com/instadeepai/Mava/issues/335 . If that doesn't work, please re-open this issue.


FPDF error: The document is closed

Hi,
I have the issue when I try to merge multiple PDF's that it gives me the error:
[2019-04-08 12:21:50] dev.ERROR: FPDF error: The document is closed {"exception":"[object] (Exception(code: 0): FPDF error: The document is closed at /project/vendor/setasign/fpdf/fpdf.php:271)
[stacktrace]

#0 /project/vendor/setasign/fpdf/fpdf.php(295): FPDF->Error('The document is...')
#1 /project/vendor/setasign/fpdi/src/FpdfTplTrait.php(329): FPDF->AddPage('P', Array, 0)
#2/project/vendor/grofgraf/laravel-pdf-merger/src/PDFMerger.php(177): setasign\\Fpdi\\FpdfTpl->AddPage('P', Array)
#3 /project/app/Jobs/ProcessCoupons.php(49): GrofGraf\\LaravelPDFMerger\\PDFMerger->merge()
#4 [internal function]: App\\Jobs\\ProcessCoupons->handle(

$merger = \PDFMerger::init();

$registration->coupons->each(function($coupon) use ($merger) {
$pdf = Storage::disk('coupons')->path($coupon->file_path);
    $merger->addPDFString($pdf);
});

$merger->merge();
$merger->save(sprintf('%s_%s_%s.%s', 'coupon_merged', time(), mt_rand(1000, 9999), 'pdf'));

My Setup:
"grofgraf/laravel-pdf-merger": "^1.0",
"laravel/framework": "5.5.*",
"php": ">=7.1.0",

What causes the issue that it's returning that error? It creates the tmp folder with temporary PDFs but it's not merging.

same issue here, identical setup but php7.2

I have a similar problem. As long as I run it in a webbrowser it seems to work fine, but when using a background worker the problem occurs. Probably because the internatl Fpdi instance isn't refreshed correctly by the INIT function...

explicity resetting all attributes seem to fix the issue, but wouldn't recommend ;-).
    $this->fpdi =null;
    $this->fpdi = new Fpdi();
    $this->tmpFiles = null;
    $this->tmpFiles = collect([]);
    $this->files = null;
    $this->files = collect([]);


convert position to v2 for Superset load_examples

update position_json for dashboards from load_examples
@mistercrunch @williaster @michellethomas

Minor comments, otherwise LGTM

Codecov Report

Merging #5515 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5515      +/-   ##
==========================================
+ Coverage   63.31%   63.32%   +<.01%
==========================================
  Files         349      349
  Lines       22181    22185       +4
  Branches     2463     2463
==========================================
+ Hits        14044    14048       +4
  Misses       8123     8123
  Partials       14       14

Impacted Files
Coverage Δ

superset/cli.py
46.59% <100%> (ø)
:arrow_up:

superset/data/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


I've tried the extension with my own Chrome Dev build and Chrome Canary, and it doesn't appear there.
I've even tried with this ancient Netscape-era "browser" and it looks fine.

this is invalid, mate

It only occurs when using the dark theme. Still there.


Use influxdb_v2 plugin in telegraf getting-started.md

Updated the Telegraf getting started documentation to recommend using the newer influxdb_v2 plugin when generating the telegraf configuration. I also felt this part could use more explanation, so I added a more simple and complicated version of the command and a link to the other possible flags. Also it seems --sample-config and -sample-config are both valid but the Telegraf docs only mention --sample-config, so updated the examples to have the extra dash.
This came from a discussion with @appletreeisyellow, what do you think of these changes?

[x] Signed the InfluxData CLA
(if necessary)
[x] Rebased/mergeable

!signed-cla


[core]: Storybook for core, composables and dates packages

General storybook setup that allows for stories for core, composables and dates packages
Changed $lib to $clib for composables and to $dlib for dates so that it is possible to have the same storybook setup for all packages
Added prettier-ignore so that the prism component is not formatted since it breaks the prism output

Note: nevermind the warning when running storybook that mention an id in stories, there is a bug with that I will investigate, but it works fine without it for now
https://user-images.githubusercontent.com/25725586/185986651-a04645bd-fc9f-45ca-9402-7fbb4b830e60.mov
Before submitting the PR, please make sure you do the following

[X] Read the Contributing guide
[X] Prefix your PR title with [@svelteui/core], [@svelteui/actions], [@svelteui/motion], [@svelteui/core], [core], or [docs].
[X] This message body should clearly illustrate what problems it solves.

Tests

[X] Run the tests with npm test and lint the project with yarn lint or just run yarn repo:prepush and check to see if it's passing.

Investigating the failing tests


Clean up transport package

This PR cleans up naming after refactor in #213. The secondary MTProto transport protocol (e.g. intermediate, abridged) may be selected for different transports (e.g. TCP, WebSocket). Note that TCP-based transport currently corresponds to dcs.PlainResolver.

Not sure about general benefits of this refactor.
Why it is needed?
The naming consistency was lost in some places, e.g. Transport vs Protocol, in other place interface got replaced with specific pointer.

The current naming is ambiguous about MTProto transport envelope protocol (abridged, intermediate, etc) vs the “socket” transport (TCP, WebSocket, etc) distinction. The later is implemented in third-party libraries (including std net).
I didn’t notice any major inconsistencies except for testAllTransports in telegram/client_e2e_test.go. The trp variable name is also a good mnemonic for telegram.Protocol so I’ve left that as is.
Ideally, telegram client shouldn’t care about the envelope protocol being used (and it already doesn’t, see dcs.Resolver, but we currently don’t have transports/resolvers other than TCP and MTProxy). That is, there are still some rough edges around the dcs package that was introduced recently.
Also I’m still working (slowly) on the DNS-based resolver. It kinda works, and this PR is one of the first steps towards integrating it with current architecture.

Personally I prefer to replace transport.Transport struct to interface, like that:
type Transport interface {
	Codec() Codec
	Handshake(conn net.Conn) (Conn, error)
	Pipe() (a, b Conn)
}


Cannot stitch multiple schemas hosted on the same gateway server

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Accessing the gateway GraphQL endpoint via Banana Cake Pop (BCP) does not yield any schema definition. BCP keeps spinning with no luck.
Steps to reproduce

Create an empty C# Web API project using .NET 6
Replace program.cs with the following source code:

var builder = WebApplication.CreateBuilder(args);

builder.Services
    .AddGraphQLServer("A")
    .AddDocumentFromString(
        @"
        type Query {
            a1: String
        }
        ".Trim()
    )
    .UseField(next => async ctx =>
        {
            ctx.Result = "Ok";
            await next(ctx);
        }
    );

builder.Services
    .AddGraphQLServer("B")
    .AddDocumentFromString(
        @"
        type Query {
            b1: String
        }
        ".Trim()
    )
    .UseField(next => async ctx =>
        {
            ctx.Result = "Ok";
            await next(ctx);
        }
    );

builder.Services.AddHttpClient("A", config =>
    {
        config.BaseAddress = new Uri("https://localhost:7000/A/");
    });

builder.Services.AddHttpClient("B", config =>
    {
        config.BaseAddress = new Uri("https://localhost:7000/B/");
    });

builder.Services
    .AddGraphQLServer()
    .AddQueryType()
    .AddRemoteSchema("A")
    .AddRemoteSchema("B");

var app = builder.Build();

app
    .UseRouting()
    .UseEndpoints(config =>
    {
        config.MapGraphQL("/A", "A");
        config.MapGraphQL("/B", "B");
        config.MapGraphQL();
    });

app.Run();

Relevant log output
/usr/local/share/dotnet/dotnet /Users/geekox86/Documents/Projects/gql/bin/Debug/net6.0/gql.dll
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: https://localhost:7000
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: http://localhost:5126
info: Microsoft.Hosting.Lifetime[0]
      Application started.
I am trying to use HotChocolate v12 stitching in .Net 6 Web API. The GraphQL gateway and downstream services are all placed in the same application server. The purpose is to keep them in one physical deployment unit for simplicity while maintaining logical separation for future scaling. However, I am not being able to connect to the gateway and stitched schemas from Banana Cake Pop. It always appear as offline and BCP keeps spinning.
Product
Hot Chocolate
Version
12.7.0-perview.6

AddLocalSchema is what you are looking for.


Snippets & Examples

Improve documentation
Link
Describe the problem
cant run most of them with Godot 4.0.2
Describe the improvement
A clear and concise description of the improvement.
Additional context
Add any other context or screenshots that help clarify your question.

Thanks for pointing out. Currently those snippets and examples are for Godot 3.x
Will make sure to port them to godot 4.

can i contribute to wiki? and how ?

Sure you can.
You may clone the repository/wiki and work on your own, pushing the changes to your fork/clone.
Then open an issue here giving me the link of your version of the wiki, so I can take your changes.

ok,
btw is can i do Realtimechannel like "from pepega_table where emote_type = 1" ?

For those who are waiting for wiki to be updated, you can take this example meantime:
Instead of
Supabase.auth.connect("signed_in", self, "_on_signed_in")
use this for Godot 4.x:
Supabase.auth.signed_in.connect(_on_signed_in)
This change is valid for every snippet in the wiki, just change names accordingly.
Basically you will call "connect" method on supabase signal, while you pass as parameter your own function to run when the signal is received. There's no need to reference to self anymore.


Fix call to missing method typo

Fixes a call to a missing method in the CakePHP ORM provider.
$this->populator->addEntity('Articles', 50, $formatter, $modifier);
$this->populator->execute();  // Fatal error

Thanks!


add host option to config

Closes #815

Looks good, thanks for the PR!

you're amazingtastic thank you!!


feat: resource resource reference

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

增加了资源引用计数的能力，将Image接入了，其他插件也需要接入资源的引用计数。


Deduplicate imports

Hello!
I searched the issues and could not find a similar issue, if there is already one on that topic I missed feel free to close this one.
I find myself solving some merge conflicts lately that happen for use statements, sometimes ending up with more than one import of the same name e.g.:
use std::vec::{Vec, Vec};

And running rustfmt on that does not deduplicate the import.
In practice the use statements causing issues are much bigger and it can be a bit of pain to deduplicate imports manually. In addition, the duplicate import prevents compilation because of the name Vec being redefined
error[E0252]: the name Vec is defined multiple times
Rust playground:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=0cd8ef07ca6cde1a2892c9b8e1697ff4
It feels like it should be possible as I've already seen rustfmt merging multiple imports from the same module into one use statement, so there is a mechanism to group names together and sort them, I'm guessing that switching the underlying container for the grouped names to some Set-like object should do the trick.
I haven't had time to check the source for that particular case, but I thought I might as well open the issue here if someone knew where the code for this is and how it could be changed to manage that case 🙂
Cheers!

Thanks for reaching out.
If you're able to use a nightly version of rustfmt you can set the imports_granularity option to anything other than the default of Preserve and the import should be deduplicated.
Let me know if that helps.
@calebcartwright when set to Preserve should we deduplicate?
We recently merged #5380, which focused on deduplicating imports for the Item case so I imagine a similar change would be needed to resolve this in the Preserve case.
Linking the tracking issue #4991

looking into this a little bit more, I don't think we can take the exact same approach as #5380, however I think a good place to perform the deduplication would be normalize_use_trees_with_granularity for the Preserve case.
https://github.com/rust-lang/rustfmt/blob/c4416f20dcaec5d93077f72470e83e150fb923b1/src/imports.rs#L213-L223

Although I have a proof of concept for how we could go from use std::vec::{Vec, Vec}; -> use std::vec::Vec; We currently have a test case in tests/target/multiple.rs that goes against this kind of normalization:
https://github.com/rust-lang/rustfmt/blob/c4416f20dcaec5d93077f72470e83e150fb923b1/tests/target/multiple.rs#L19-L23
Because of that I'm not sure if we should try to change this for the Preserve case.

What is the rationale behind keeping duplicates? (as it's not valid rust code it seems?)

Thanks a lot for the insights 🙏
From afar apart the comment one which seems a bit unavoidable and pathological, the others can be deduplicated with the normalization/flattening part I think I saw in the code? and then recompose them with the proper import granularity

Thank you for sharing @IceTDrinker and @ytmimi for providing context and details.
There's been some prior discussions on this topic which I can't find at the moment so will attempt to recap.
The historical, and thus the continued, behavior of rustfmt was to only attempt to deduplicate with the user gave rustfmt the permission to merge/modify imports, originally via the merge_imports option which was rebranded and extended as imports_granularity. So long as imports_granularity is disabled (the default), rustfmt isn't going to attempt in-place deduplication.
As such my inclination is that we should close this. The only thing we could do would be to introduce yet another imports related config option that allowed for dedupes but banned any form of granularity normalization, and while I acknowledge there's probably a non-zero set of users that would want that, I don't think there's anywhere near enough volume to justify the added complexity of the config surface/mental model around import formatting and associated code maintenance.

At the moment Module also does not dedupe: https://github.com/rust-lang/rustfmt/issues/6243
I agree with leaving Preserve as is. Semantically, Preserve is essentially leaving things as they are. Per the docs, preserve the original structure written by the developer. It is also the default option. If there were ever bugs/edge cases in the other options, there must be some "disabled" state for this part of format.


Print plan before apply

Closes #1296
Example output after testing:
kubectl logs -n flux-system sensitive-output-test-tf-runner
2024/06/04 04:56:51 Starting the runner... version  sha
{"level":"info","ts":"2024-06-04T04:56:52.618Z","logger":"runner.terraform","msg":"preparing for Upload and Extraction","instance-id":""}
{"level":"info","ts":"2024-06-04T04:56:52.618Z","logger":"runner.terraform","msg":"write backend config","instance-id":"","path":"/tmp/flux-system-sensitive-output-test/sensitive-output-test","config":"backend_override.tf"}
{"level":"info","ts":"2024-06-04T04:56:52.618Z","logger":"runner.terraform","msg":"write config to file","instance-id":"","filePath":"/tmp/flux-system-sensitive-output-test/sensitive-output-test/backend_override.tf"}
{"level":"info","ts":"2024-06-04T04:56:52.619Z","logger":"runner.terraform","msg":"looking for path","instance-id":"","file":"terraform"}
{"level":"info","ts":"2024-06-04T04:56:52.619Z","logger":"runner.terraform","msg":"creating new terraform","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0","workingDir":"/tmp/flux-system-sensitive-output-test/sensitive-output-test","execPath":"/usr/local/bin/terraform"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"setting envvars","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"getting envvars from os environments","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"setting up the input variables","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"mapping the Spec.Values","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"mapping the Spec.Vars","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.621Z","logger":"runner.terraform","msg":"mapping the Spec.VarsFrom","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.622Z","logger":"runner.terraform","msg":"generating the template founds"}
{"level":"info","ts":"2024-06-04T04:56:52.622Z","logger":"runner.terraform","msg":"main.tf.tpl not found, skipping"}
{"level":"info","ts":"2024-06-04T04:56:52.623Z","logger":"runner.terraform","msg":"initializing","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:52.623Z","logger":"runner.terraform","msg":"mapping the Spec.BackendConfigsFrom","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.235Z","logger":"runner.terraform","msg":"workspace select"}
{"level":"info","ts":"2024-06-04T04:56:54.236Z","logger":"runner.terraform","msg":"creating a plan","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.399Z","logger":"runner.terraform","msg":"show the raw plan file","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.488Z","logger":"runner.terraform","msg":"creating a plan","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.704Z","logger":"runner.terraform","msg":"save the plan","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.723Z","logger":"runner.terraform","msg":"loading plan from secret","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.729Z","logger":"runner.terraform","msg":"running apply","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # tls_private_key.example will be created
  + resource "tls_private_key" "example" {
      + algorithm                     = "ECDSA"
      + ecdsa_curve                   = "P224"
      + id                            = (known after apply)
      + private_key_openssh           = (sensitive value)
      + private_key_pem               = (sensitive value)
      + private_key_pem_pkcs8         = (sensitive value)
      + public_key_fingerprint_md5    = (known after apply)
      + public_key_fingerprint_sha256 = (known after apply)
      + public_key_openssh            = (known after apply)
      + public_key_pem                = (known after apply)
      + rsa_bits                      = 2048
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Changes to Outputs:
  + hello_safe   = "Hello, World! I'm safe!"
  + hello_unsafe = (sensitive value)
  + private_key  = (sensitive value)

tls_private_key.example: Creating...
tls_private_key.example: Creation complete after 0s [id=4c819ec87937cfb17dc71c1dafde152c12c6bfd7]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

Outputs:

hello_safe = "Hello, World! I'm safe!"
hello_unsafe = <sensitive>
private_key = <sensitive>
{"level":"info","ts":"2024-06-04T04:56:54.919Z","logger":"runner.terraform","msg":"creating outputs","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0"}
{"level":"info","ts":"2024-06-04T04:56:54.953Z","logger":"runner.terraform","msg":"cleanup TmpDir","instance-id":"e2f92e7e-3c1a-4a9a-9496-c0e2691e79f0","tmpDir":"/tmp/flux-system-sensitive-output-test"}

The implementation looks good, but we should add a test case to ensure it works as intended even when we do changes later down the road

Added a test case for it @ilithanos :)

@ilithanos can you please give this a review again?


[LLVM] Add target feature string to function attributes

Target features actually apply to subtargets, and each LLVM IR function can have a separate subtarget.

Thanks @kparzysz-quic


Update vunit.js

Proposing an optional callback (onChange) that gets called when vunit detects viewport changes.
the callback gets a parameter, the object of viewport dims. ex: {width: 1174, height: 993}.
if you're doing expensive calls here, may want to use https://remysharp.com/2010/07/21/throttling-function-calls

This is an useful addition, thank you @JosePedroDias!
Regarding throttling, I took a different route, here. Instead of listening to a window.resize event, the ìnitfunction sets up awindow.setInverval` that checks if a resize happened. It is something that can be improved for sure, but these are the key points of interest:
An observer running every 100ms checks if the viewport has been resized and regenerates the CSS rules accordingly. It's a cross-device, event-less solution to keep track of everything that would trigger a resize on the viewport, namely:

Window resizing on desktop;
Orientation change on mobile;
Scrollbars appearing/disappearing on desktop;
Navigation bars appearing/disappearing on mobile;
Zooming on mobile and desktop;
Download bar on desktop;
Password saving prompt on desktop;
Etc.

If you don't mind, I'll put this one on hold for a while because I want to properly tag the release. Your change will be the first one :)

Great. Don't rush the pull request at all - I just thought this might be useful for other people using the lib.
I wasn't proposing applying throttling to your code, but to the callback function application - it was kind of a hint to the library user, something you might want to write in the README.md file.

:+1:


Mobile like controls for the outputs page

Testing:

Please check it on different platforms.
Try it with mining wallet


Not Working in HMR Watch Mode

I'm using HMR watch mode, and I've made changes to routes in multiple controllers, but the changes aren't taking effect. I have to stop (Ctrl+C) and then restart the HMR watch command for it to work.
Expectation
When a route is changed in the controller, the routes should automatically update as well.

Error:
You must use dynamic import to make it reloadable (HMR) with hot-hook.
    at DynamicImportChecker.ensureFileIsImportedDynamicallyFromParent

Update your provider to fix it:
/*
  |--------------------------------------------------------------------------
  | Service providers
  |--------------------------------------------------------------------------
  |
  | List of service providers to import and register when booting the
  | application
  |
  */
  providers: [
    {
      file: () => import('@softwarecitadel/girouette/girouette_provider'),
      environment: [],
    },
  ],

I'm using HMR watch mode, and I've made changes to routes in multiple controllers, but the changes aren't taking effect. I have to stop (Ctrl+C) and then restart the HMR watch command for it to work.
Expectation
When a route is changed in the controller, the routes should automatically update as well.

Error:
You must use dynamic import to make it reloadable (HMR) with hot-hook.
    at DynamicImportChecker.ensureFileIsImportedDynamicallyFromParent

Update your provider to fix it:
/*
  |--------------------------------------------------------------------------
  | Service providers
  |--------------------------------------------------------------------------
  |
  | List of service providers to import and register when booting the
  | application
  |
  */
  providers: [
    {
      file: () => import('@softwarecitadel/girouette/girouette_provider'),
      environment: [],
    },
  ],


Demo model v2

Making a second model, which is meant to be an improvement to our model from the previous sprint.
Focusing on making a more stable model, that would be able to carry their parts.

Same parts as before, but increasing the durabilty of the material or improving the design, for stability.

Issue has been moved to issue "Project".
All equivalent issues will be moved there.


Spelling: wrong "its" in FAQ

⚠ Open Source Workshop ⚠
Look for "and its not well designed"

I got this!

Thanks @whoffmann!!