ButtonArray aligmnent issues

Seen on Godot 2.1.1
ButtonArray text alignment is always set to left and there is no option in the editor to change it.
Align modes have been exposed but have no effect.
Side note: the aligment is not initialized in the C++ constructor so it's most of the time set to BEGIN, by guess.

I worry about changing default value.

I took a look at Godot 2.1 and there is an align property set to Begin by default, indeed.
However, this property actually doesn't control alignment of text in the buttons, but the whole widget, somehow. Text is always centered correctly inside buttons.
Also I realize align is not missing, I was assuming that because I missed the part where it's listed manually in _get_property_list. But there is a clear difference between 2.1 and 2.1.1:
In 2.1, it looks normal and Align has a visible effect on the position of the whole widget:

In 2.1.1, text is tied to the left and align doesn't seem to have any effect.

Works fine in 2.1.4:


Export autograding grades

Right now, I can see the points the students scored on the dashboard. But, more useful would be able to export a CSV of the grades so that I can import them into my Blackboard gradebook.

This is a good suggestion, we've opened an issue internally to track this. The information is embedded in the check run for all of the Github Actions but it is pretty difficult to extract that. I think if we are able to add this functionality it would be in GitHub Classroom directly not in this action... so I'll close this for now.

Is there a way to check the status?

Is there a way to check the status?

Since this is the first hit I got on a Google search I thought I should answer this issue to indicate that this functionality now exists. See https://education.github.community/t/introducing-easy-grade-downloading-for-classroom/69995


Got UnicodeDecodeError during installation.

Hi. I got an error when I tried to install jupyterlab-flake8.
Here is detailed log for the error message.
Yarn configuration loaded.
Node v10.13.0

> /home/ehgud5/miniconda3/envs/mv/bin/npm pack jupyterlab-flake8
Traceback (most recent call last):

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/debuglog.py", line 47, in debug_logging
    yield

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/labextensions.py", line 93, in start
    ans = self.run_task()

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/labextensions.py", line 149, in run_task
    for i, arg in enumerate(self.extra_args)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/labextensions.py", line 149, in <listcomp>
    for i, arg in enumerate(self.extra_args)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 398, in install_extension
    return handler.install_extension(extension, pin=pin)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 611, in install_extension
    info = self._install_extension(extension, tempdir, pin=pin)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 1507, in _install_extension
    info = self._extract_package(extension, tempdir, pin=pin)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 1574, in _extract_package
    ret = self._run([which('npm'), 'pack', source], cwd=tempdir)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 1748, in _run
    return proc.wait()

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/site-packages/jupyterlab/commands.py", line 107, in wait
    out, _ = proc.communicate(timeout=.1)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/subprocess.py", line 863, in communicate
    stdout, stderr = self._communicate(input, endtime, timeout)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/subprocess.py", line 1574, in _communicate
    self.stdout.errors)

  File "/home/ehgud5/miniconda3/envs/mv/lib/python3.6/subprocess.py", line 760, in _translate_newlines
    data = data.decode(encoding, errors)

UnicodeDecodeError: 'ascii' codec can't decode byte 0xf0 in position 23: ordinal not in range(128)

Exiting application: jupyter

This is likely due to your terminal encoding. See:

https://stackoverflow.com/questions/18649512/unicodedecodeerror-ascii-codec-cant-decode-byte-0xe2-in-position-13-ordinal/49127686#49127686
https://stackoverflow.com/questions/21129020/how-to-fix-unicodedecodeerror-ascii-codec-cant-decode-byte


Fix for columns dtype changes in pandas 2.0.1

In pandas 2.0.1, empty data frames have a RangeIndex(start=0, end=0) as .columns attribute. This PR changes our behavior:

obs and var coerce the .column attribute of empty dataframes to be a string Index. I decided not to throw ImplicitModificationWarnings here since this empty data frames are so inconsequential.
read functions for IOSpec('dataframe', …) now pass columns=columns if len(columns) else None instead of effectively columns=list(columns), which makes uns entries that are data frames conform to default pandas behavior.

Questions:

[ ] Is the implicit modification in _set_dim_df OK or are there cases where an empty data frame must remain untouched?
[ ] We don’t actually need all the columns=EMPTY_IDX in anndata.py, as the _set_dim_df method takes care of it. Should we do away with it?

Fixes #991

Nice!

I can't think of a case where an empty DataFrame must be untouched.
If we don't need them I don't see why we'd keep them?


Source of camera movement

Hi!
Is there an equivalent for OnCameraMoveStartedListener.REASON_GESTURE? If not, can you please support it? I would like to trigger a specific action when users moves the map. I've searched CameraPositionState.kt and found isMoving but this works the same if the camera was moved by code or by user.

@webfrea-k Please take a moment to fill out this short survey. Thank you!
This is an automated message, feel free to ignore.

We do have the same use case. Some location with markers is being followed and when the user moves the map, the automatic "following" will stop. GoogleMap.OnCameraMoveStartedListener.REASON_GESTURE will tell us exactly that.

As a workaround I could use a detection modifier which doesn't consume the events:
.pointerInput(Unit) {                 forEachGesture {                     awaitPointerEventScope {                         awaitFirstDown(requireUnconsumed = false)                         actionOnReasonGesture()                     }                 }             },

This can be supported as another property on CameraPositionState


document part: proposed new terms (for the discussion on Nov 9, 2020)

@ddooley @zhengj2007
term label: “descriptive data section”

definition
A part of a document that lists and defines data variables, describes data characteristics (e.g. missing data information) and any assumptions and simplifications made.
definition source
https://www.equator-network.org/reporting-guidelines/prisma/
suggested parent term if possible
n/a

term label: “additional results section”

definition
A part of a document that reports analyses other than main results of the study (e.g. subgroups analyses, adjusted analyses, sensitivity analyses, etc.)
definition source
https://www.equator-network.org/reporting-guidelines/strobe/
https://www.equator-network.org/reporting-guidelines/consort/
https://www.equator-network.org/reporting-guidelines/prisma/
suggested parent term if possible
“results section” (IAO:0000318)

term label: “research participants section”

definition:
A part of a document that describes human subject(s) that participated in a study (e.g. inclusion & exclusion criteria, recruitment methods, reasons for non-participation, grouping and randomisation, methods of follow-up, etc.).
definition source
https://www.equator-network.org/reporting-guidelines/consort/
https://www.equator-network.org/reporting-guidelines/strobe-nut/
https://www.equator-network.org/reporting-guidelines/prisma/
notes:
This term could be a parent term of “patients section” (IAO: 0000635).

term label: “measurement methods section”

definition
A part of a document that describes details of data assessment methods (data measurement).
definition source
https://www.equator-network.org/reporting-guidelines/strobe/
suggested parent term if possible
“methods section” (IAO:0000317) for describing investigation method.

term label: “research settings section”

definition
A part of a document that describes the physical/social/cultural conditions around a research trial.
definition source
https://www.ncbi.nlm.nih.gov/books/NBK262175/
suggested parent term if possible
n/a

term label: “study bias section”

definition
A part of a document that describes systematic error introduced into sampling or testing by selecting or encouraging one outcome or answer over others
definition source
DOI: 10.1097/PRS.0b013e3181de24bc
notes
Disjoint with “study limitations section” (IAO:0000631)

@cyang0128 Thanks for the submission. Will add to next Monday call's agenda.

Discussed on OBI/IAO Nov. 9, 2020 call. Note that study limitations are defined as "A part of a document about biases or short comings related to the study design and execution." so would not be disjoint. Could be a subtype.

@cyang0128 I added the terms you request on a tab-delimited format with some minor edits following OBO Foundry recommendations. Could you please review the document rows 3 - 8?
https://github.com/information-artifact-ontology/IAO/blob/master/src/ontology/template/IAO_newTerms.tsv
Thanks!

Dear @zhengj2007, thanks a lot. I have checked the rows 3-8, everything is fine. I agree and thanks for your work!

Thanks @cyang0128. I will add the terms in the IAO and let you know when the new release is made.

The new terms are available in release 2020-12-09.


update yushi5058.json

Changes proposed
update infos in my linkfree profile

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

I just updated my linkfree profile, the other things are old and already merged


Create and Publish New Search Gateway Image on Dockerhub

Description
Ensure that the changes to the search provider are reflected on the Vistaar instance gateway. Create and publish a new search gateway image on Dockerhub.
Goals

[x] Reflect changes to the search provider on the Vistaar instance gateway.
[x] Create a new search gateway image incorporating these changes.
[x] Publish the new search gateway image on Dockerhub.
[x] Test the new image to ensure it functions correctly.

Expected Outcome
Successful creation and publication of a new search gateway image on Dockerhub, ensuring all changes are accurately reflected on the Vistaar instance gateway.
Acceptance Criteria

[ ] Changes to the search provider reflected on the Vistaar gateway.
[ ] New search gateway image created.
[ ] New image published on Dockerhub.
[ ] Image tested and confirmed to function correctly.

Mockups / Wireframes
Not applicable
Product Name
Beckn
Domain
Search Provider Integration
Tech Skills Needed

Docker
Image Creation
Image Publishing
Integration
Testing

Complexity
Medium
Category
Integration and Deployment
Sub Category
Search Provider

created new image of gateway and tagged it as fidedocker/gateway:search-provider

Due to de-prioritization of the search provider, closing this ticket for now.


Base64 decoding function

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
I would like to take the KubeConfig from 'azure-native:containerservice:ManagedCluster' and use it to create a kubernetes provider ('pulumi:providers:kubernetes'), using Pulumi Yaml, but for that I need an fn::base64decode function.
Right now I have to split it into two parts, and manually put in the decoded kubeconfig from the first part.
Affected area/feature
Pulumi Yaml

Thanks @BoSoeborgPetersen for reporting this, seems like a blocker for replicating the demos we've done on other clouds on Azure.
@iwahbe a base64 decode intrinsic seems useful, and some providers do force us to manipulate base64 data. Another example of this is the Linode LkeCluster which returns a base64 encoded kubeconfig.

I agree that the extrinsic is useful and worth adding. I recommend the name Fn::FromBase64. We want the name to make sense with the existing intrinsic Fn::ToBase64.


Not working in Linux

Hi
I just tried this tool in Linux (ubuntu 22.04) environment and not working . It is accepting the request and return 200. No errors in the logs. But no print is coming out
Just tried to print it from terminal and it is working
echo "^XA^FO50,50^ADN,36,20^FDTest Print^FS^XZ" | nc <IP_ADDRESS> 631

Any help will be appreciated

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hmm I think the problem we are encountering here is that StreamWriter behaves differently on Windows compared to Linux.
Perhaps we can try using Direct Byte Writing instead of StreamWriter
using (TcpClient client = new TcpClient(printerIpAddress, port))
using (NetworkStream stream = client.GetStream())
{
    byte[] zplBytes = Encoding.ASCII.GetBytes(template);
    await stream.WriteAsync(zplBytes, 0, zplBytes.Length);
    await stream.FlushAsync();
}

Can you perhaps clone the code, update it, run the command dotnet publish -r linux-x64 -c Release /p:PublishSingleFile=true --self-contained true and try it out? I don't have a Zebra printer at the moment.


Not enough permission to execute the command

Hello, I have installed the seedcracker mod but every time I type any command related to seedcracker it gives me this message:

this happens when u dont have the mod installed. make sure u install fabric properly, then put the seedcracker jar in mod folder


Inference without GT

is it possible without GT?
it seems there is no function of inference.

Hi, thanks for your interest in our work. You can find the inference part without GT in  evaluate_crack.py. For more details, Please refer to the code.
Hope that helps!


Infer stream parameters when not given

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #34963
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks
Just a quick fix only need to fix this properly
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

BTW, with this PR, now the following code works
public function main() {
    EvenNumberGenerator evenGen = new();
    stream evenNumberStream = new(evenGen);
   // stream<int,error?> str = <stream<int,error?>>evenNumberStream;
}

class EvenNumberGenerator {
    int i = 0;
    public isolated function next() returns record {| int value; |}|error? {
        self.i += 2;
        return { value: self.i };
    }
}

But if I uncomment the commented line, it crashes at runtime

BTW, with this PR, now the following code works
public function main() {
    EvenNumberGenerator evenGen = new();
    stream evenNumberStream = new(evenGen);
   // stream<int,error?> str = <stream<int,error?>>evenNumberStream;
}

class EvenNumberGenerator {
    int i = 0;
    public isolated function next() returns record {| int value; |}|error? {
        self.i += 2;
        return { value: self.i };
    }
}

But if I uncomment the commented line, it crashes at runtime

Somehow I cannot reproduce this. What I get is the below error which is correct.
error: {ballerina}TypeCastError {"message":"incompatible types: 'stream<(any|error),error?>' cannot be cast to 'stream<int,error?>'"}
	at t4:main(t4.bal:5)


Icons are not updated when a button is enabled or disabled

When a button is enabled or disabled the icon is not updated.
The first button is enabled, the second is disabled

After pressing "toggle buttons enabled" the icons are not updated

After pressing the second button the icon is updated

Test project
Test.zip

I tried both using Attached.Icon (the first button) or setting the icon as the content of the button (the second button), the result is the same

Thank you for filing this issue.
Your test project really did help.
With version 3.1.1 the bug is fixed.


can't scroll to the end of a large file

First of all: I think the way CodeMirror renders to DOM only the visible part of the text is absolutely fantastic.
I know I'm really pushing limits here but there seems to be a threshold at around 1.5-2 million
lines beyond which it's impossible to scroll.  Repro: go to https://codemirror.net/mode/markdown/index.html and type in the JS console:
var s = Array(2000000).join('a\n'); // executes almost instantly
editor.setValue(s); // a few seconds, depending on browser

Drag the scrollbar all the way down or press ctrl+end: the scrollbar goes to the bottom but the text is far from the last line.  In Chromium it's around line 1.7 million.  In Firefox: 1.9 million and you can't see the scrollbar (well, not unexpectedly).
Any pointers for where I should look in the code to circumvent that limit?

The problem appears to be that Chrome only uses ~25 bits for the height of an element. I assume a similar issue exists in Firefox. If I inspect the .display.sizer element after running your example code, I see an element with min-height: 38000008px, yet if I get its clientHeight, it says 33554428 (which is 2^25 - 4). The scrollbar height is based on this reported element height, but it seems that in general the browser refuses to scroll the wrapping div any further (manually adding to its scrollTop has no effect).
So you seem to be reaching a limitation in browsers, not in CodeMirror, here. We could work around it by doing a lot more scroll-related work in our own code, but I am not really interested in doing that right now (I'm relying on native scrolling to keep wheel scrolling smooth, and changing that would create all kinds of new issues).
I'm closing this as 'can't cleanly fix'. If you feel it is very important, try bugging browser vendors to lift their height limitations.

Initially I thought this would be due to some arbitrarily set constant in CodeMirror.  25 bits in the browser - that's an interesting insight.  Thank you.


🐞 Bug: Username  Disappear When reCAPTCHA Not Clicked During Sign-Up

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When users forget to click reCAPTCHA during the sign-up process, the entered username  disappear.
Steps To Reproduce

Go to sign-up
try to sign up without clicking reCAPTCHA
see the bug.

Expected Behavior
If a user forgets to click reCAPTCHA during sign-up, the entered username  should remain visible and not disappear.
Screenshots
https://github.com/CircuitVerse/CircuitVerse/assets/99977240/bf386bc2-4829-4a2b-8f4d-ac6a9b7b70fd
Are you working on this issue?
Yes

@tachyons Could you please verify if this issue is valid?

maintainers, Could you please verify if this issue is valid?

maintainers, Could you please verify if this issue is valid?

You can work on the issue.

Hey @tanmoysrt,
What about this similar issue? Was it not valid?
https://github.com/CircuitVerse/CircuitVerse/issues/4739

hey @tanmoysrt  can i work on this issue

is this issue still open

@subhamkumarr let me take this issue, i couldn't find any pending issues throughout the day

Problem still exists! let me work on it


Formating ICR projectId to registry-registryNum format

Changes the ICR projectIds to the regular registry-registryNum format.
test deployment: https://api.thegraph.com/subgraphs/id/QmSLBeufcG3XUUXbh1bAXYzNdykMCb2WHmDW56Z3DXR78Q

@cujowolf @psparacino this one is kind of high priority as it is breaking the ICR retire receipt pages https://github.com/KlimaDAO/klimadao/issues/2285


LuCI does not offer NFS mounts

I manually mounted a remote drive to my router but this does not seem to be supported from with LuCI. The mount is shown on luci/admin/system/mounts and I am invited to unmount it, but nowhere I saw that I could mount it.
The "why do I want that" is that storage may be more than the single SSD attached via USB or PCIe. A NAS already active on the local network comes to mind. In my experimental setup I am actually tunneling to one via the Internet. Works nicely. This could then be used to share that remote storage locally, to store data (like video or sensor input) or (shameless promotion) for running BOINC without the need to have physical access to the device and a USB stick.
Is the management of NFS mount via LuCI something that would be desirable to have? Where should that be implemented? Extension of the existing mount functionality? Or a separate luci-app-nfs in analogy to luci-app-samba?
Thanks!
Steffen

It is a limitation of the underlying block mount functionality in OpenWrt. It currently does not support any kind of network volumes, hence no possibility to expose that in LuCI.


Crash During Load

Describe The Bug:
Upon initially installing the plugin and basic stock config, experiencing a crash upon load. The plugin is being used to control a Govee Air Purifier, model H7121.
Logs:
[6/5/2022, 11:55:32 AM] ====================
[6/5/2022, 11:55:32 AM] ERROR LOADING PLUGIN homebridge-ultimate-govee:
[6/5/2022, 11:55:32 AM] Error: EAFNOSUPPORT, Address family not supported by protocol
    at new Hci (/homebridge/node_modules/homebridge-ultimate-govee/node_modules/@abandonware/noble/lib/hci-socket/hci.js:80:18)
    at new NobleBindings (/homebridge/node_modules/homebridge-ultimate-govee/node_modules/@abandonware/noble/lib/hci-socket/bindings.js:25:15)
    at module.exports (/homebridge/node_modules/homebridge-ultimate-govee/node_modules/@abandonware/noble/lib/resolve-bindings.js:13:12)
    at Object.<anonymous> (/homebridge/node_modules/homebridge-ultimate-govee/node_modules/@abandonware/noble/index.js:2:51)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/homebridge/node_modules/homebridge-ultimate-govee/src/data/clients/BLEClient.ts:5:1)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
[6/5/2022, 11:55:32 AM] ====================

Plugin Config:
{
    "username": "***",
    "password": "***",
    "connections": {
        "iot": true,
        "ble": true,
        "api": true
    },
    "devices": {
        "lights": [
            {
                "hideSegments": true,
                "effects": [
                    {
                        "id": 0,
                        "enabled": false
                    }
                ]
            }
        ]
    },
    "platform": "Govee Ultimate"
}

Environment:

Plugin Version: v2.56.0
Homebridge Version: v1.4.1
Node.js Version: v16.14.0
Operating System: Docker

Sorry for delay - had some family stuff - I will look into this asap

@raneses I see you're using BLE for communication - Unfortunately I have not devoted much time to the BLE integration but I'll try to make it a priority. Thanks for the report

@raneses you can try disabling BLE communication but it seems like there's a kernel module you may need to enable https://github.com/abandonware/noble/issues/152

the H7121 works perfectly over Rest and AWS


String.toFloat "." yields Ok NaN

The Result of String.toFloat "." yields Ok Nan instead of Err "..."
Probably a bug?

Yeah, that's pretty freaky!
If you are interested, seems like a nice bug to check out and turn into a PR!

There are two pull requests addressing this issue: #342 and #343. I suggest to close the ticket here and select one of the two PRs (probably #342, which appears to be more encompassing).


GeoJSON seen as just one object?

I have a GeoJSON FeaturCollection with around 5000 LineStrings. I tried adding it per the example code in the readme:
L.Map.include(L.LayerIndexMixin);

var map = L.map(...);
...
var layer = L.GeoJSON(data).addTo(map);
map.indexLayer(layer);

// Search visible features for example
map.on('moveend', function () {
    var shown = map.search(map.getBounds());
    console.log(shown.length + ' objects shown.');
});

But the console.log only ever outputs 1.
Am I supposed to add each feature as its own layer? (as is done with the circle markers in the live demo)

Yes, exactly. The specified layer is indexed by its bounds. If you want
individual object you'll have to index each sub-layer...
Don't hesitate to contribute some improvements if you think we can do
better ;)
Thanks!
Le 20 sept. 2017 11:47, "Johnnie Hård"<EMAIL_ADDRESS>a écrit :

I have a GeoJSON FeaturCollection with around 5000 LineStrings. I tried
adding it per the example code in the readme:
L.Map.include(L.LayerIndexMixin);
var map = L.map(...);
...
var layer = L.GeoJSON(data).addTo(map);
map.indexLayer(layer);
// Search visible features for example
map.on('moveend', function () {
var shown = map.search(map.getBounds());
console.log(shown.length + ' objects shown.');
});
But the console.log only ever outputs 1.
Am I supposed to add each feature as its own layer? (as is done with the
circle markers in the live demo)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/makinacorpus/Leaflet.LayerIndex/issues/4, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAhXhJpDUXq0tuzUvKtivbMb7k7erhnLks5skN84gaJpZM4PdnOE
.


Homebrewery DNS Address Not Found?

I love the Homebrewery a lot and use it for a ton of stuff, but I got on to do some editing and I couldn't even get on the page. The message that came up was DNS Address Not Found. Don't know what it means exactly XD
Is it an issue on my end or is it some kind of maintenance? Thanks very much!
Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

http://downforeveryoneorjustme.com/homebrewery.naturalcrit.com
Seems like it's an error on your side. There are sooo many possible reasons for that to happen, there's not much we can do here to help you. If you are at work, ask your IT guy. If you are at home, check if it happens with every device or just one, to locate the error.

Closing since there hasn't been a reply from the user for many months and it is assumed the problem was solved.
@TheRealNeal  If you are still experiencing this issue, feel free to reopen and provide information.


Changed college textfield to dropdown with dropdown values from firebase

Description
Changed the college textfield in sign up page to a dropdown button which gets it's dropdown values from firebase's 'colleges' collection.
Fixes #95
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

I have to fetch course and stream from Firebase too?

I have to fetch course and stream from Firebase too?

Yes, the issue was for college name, course and branch

How did these conflicts rise? Did some other PR related to this file got merged recently? I see a lot of stupid changes that make absolutely no sense like missing parenthesis, semi-colons etc.

How did these conflicts rise? Did some other PR related to this file got merged recently? I see a lot of stupid changes that make absolutely no sense like missing parenthesis, semi-colons etc.

I can see conflict in only file and that quite reasonable. Are you facing any problem?

I can see conflict in only file and thats quite reasonable. Are you facing any problem?

When I pushed the changes yesterday there were no parenthesis or square brackets or semi-colons missing but now there are. How did this happen? How does this usually happen in any open source project that when you push changes it's fine but after some time conflicts pop up, conflicts like missing parenthesis which aren't actually conflicts between code but errors?

Hahahah I've been trying to solve the conflicts but there just are so many parenthesis missing it's getting difficult to solve error.

Found the issue, somehow someone removed a sizedbox, a parenthesis and a square bracket in the code. Guessing removing the sizedbox wasn't a mistake since it got merged by one of the maintainers, I have resolved the error due to the missing parenthesis and square bracket.

I might need a lil assistance here. My forked repo is 2 commits behind and there also seems to be some changes in the file this PR is about. Now that I need to make more changes to this file, should I first do
git pull upstream master
to update my local repo and then start working?

I might need a lil assistance here. My forked repo is 2 commits behind and there also seems to be some changes in the file this PR is about. Now that I need to make more changes to this file, should I first do
git pull upstream master
to update my local repo and then start working?

To even your repo with the master repo
git remote update
git rebase upstream/master
git push

To even your repo with the master repo
git remote update
git rebase upstream/master
git push

Also, lets have a discussion with Samarpan and figure out this issue

Won't that remove all the changes that I've made on my local repo that haven't gotten merged with the upstream master yet?

Or maybe merge these changes on the college dropdown because this is absolutely final and don't offer any points for this issue to me. Let's discuss over converting the remaining fields to dropdowns too and then open another issue with the new ideas.

The fields are repeating?

Not repeating anymore

Can you also create an issue to discuss further ideas with @SamarpanCoder2002 and me?

Can you also create an issue to discuss further ideas with @SamarpanCoder2002 and me?

Yah....Required a strong discussion here @D-Ajay-Kumar @HimeshNayak ...... Waiting for the reponse @HimeshNayak

Can you also create an issue to discuss further ideas with @SamarpanCoder2002 and me?

Yah....Required a strong discussion here @D-Ajay-Kumar @HimeshNayak ...... Waiting for the reponse @HimeshNayak

Lets continue on discord.

Can you also create an issue to discuss further ideas with @SamarpanCoder2002 and me?

Yah....Required a strong discussion here @D-Ajay-Kumar @HimeshNayak ...... Waiting for the reponse @HimeshNayak

Lets continue on discord.

ok

Lets continue on discord.

Can't send more than 3 texts in 900 seconds

Lets continue on discord.

Can't send more than 3 texts in 900 seconds

we'll use the voice channel. Please check the text channel.

Lets continue on discord.

Can't send more than 3 texts in 900 seconds

we'll use the voice channel. Please check the text channel.

Ok Sure


Add a -f option to let a user pass a filename for creating secrets

#1589 This would read the secret from the file instead of STDIN.
This PR also servers as a platform for continuing the discussion from https://github.com/docker/swarmkit/pull/1567#discussion_r80588914 and doing a design discussion of what docker secret create <secret-name> should look like.
A summary of that discussion and some IRL discussions:
@aluzzardi suggests the following UI:

Pass as an argument:  docker secret create foo bar
Pass to stdin: echo bar | docker secret create foo -
Pass as a file:  echo bar > value.txt && docker secret create foo -f value.txt

The reason the current design does not to accept the secret itself as an argument is that we do not want to encourage users to type the secret into a command.  If we did accept a secret as a direct argument, then it would be available in the shell history of any user that created one, unless they explicitly removed it.
With STDIN and file, we can't prevent them from echoing to STDIN or a file, but it doesn't legitimize that as a way to add a secret.
cc @dmp42 on UI
cc @diogomonica

Current coverage is 53.89% (diff: 100%)

Merging #1595 into master will increase coverage by 0.06%

@@             master      #1595   diff @@
==========================================
  Files            84         84
  Lines         13942      13937     -5
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7504       7511     +7
+ Misses         5429       5420     -9
+ Partials       1009       1006     -3

Powered by Codecov. Last update 670003b...fa9fbb3

Looks fine to me. I'm sure the real debate will play out when the Docker PR is reviewed, though.


feature: template extraEnvs with helm tpl function

The allows camunda to get dynamic envVars like a Release Name from parent Helm-Charts

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

@FabiusE Thanks for your PR, nice addition :clap:


Implement dependabot

Automate dependabot updates by auto-merging PRs to a separate branch and
then, once a week, creating a PR from this branch pointing to master.
Closes #8
The workflows are similar to those implemented in oteapi-core.
Requirements before this PR can be merged:

[X] Creating the permanent branch ci/dependabot-updates.
[ ] Create the RELEASE_PAT secret.

Codecov Report

Merging #16 (d6885ec) into master (38972f3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   69.49%   69.49%
=======================================
  Files           8        8
  Lines         295      295
=======================================
  Hits          205      205
  Misses         90       90

Flag
Coverage Δ

pytest
69.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 38972f3...d6885ec. Read the comment docs.


[node][opengl-2] Update workflow for node 21 + allow publish from opengl-2 branch

This PR allows a node release to be published from the 'opengl-2' branch.  This would be run by selecting the 'opengl-2' branch when running then workflow

Changes

added back macos building
allow publish steps to work outside main
put 'fallback-to-build' back false. I had changed this to true in another PR long ago, but it should be false since the gyp build isn't set up to work.
I realized for node 20 support that node-pre-gyp needed the abi list updated. since we are still using my version of that package, I went and updated it to add support for the latest node versions.

Cherry Picks

update for node 21 (#1794)
Fix flaky Node.js CI with cache (#1443)

I think we should just merge the two Node workflows. Here the the frequency of PRs is so low that we can afford also running Linux and Windows.

@ntadej I installed it.

I tried to re-run that macos arm64 job that failed to see if it would pick up the update, but it still failed. I'm not sure it would pick up the new changes yet though.

I'll merge this. We can fix the missing fonts in parallel as I think this has already been failing earlier.


Search box clear button does not work

Tested in safari and chrome MacOS

Should be fixed now, thanks.


New release date??

Question
Hi there, when will you be releasing a new version of this repo? I've noticed many brilliant works are done and many PRs are merged but the latest version is from July 6th. Would you please advise?

Hey @ozgurgul
I apologize for the long time between releases, we have been swamped with other projects. I'm finally able to start shifting focus back to this project. I'm starting work on the RHEL8 CIS and I hope to possibly, maybe, hopefully have a release by Wednesday next week. I'll keep dumping updates into devel before the full release into main.
-George

hi @ozgurgul
Apologies for the delay on this, we have had other client priorities which means unfortunately, we didn't get around to release this one.
However we have added alot of new enhancements, issue fixes, workflow piepline updates which will help going forward, these have been in devel for the last 3-4weeks and all looks good so we hope to get a release out the next couple of days.
thanks
uk-bolly

hi @ozgurgul
The new release is now avaible. will be synced with galaxy later today. Once again apologies for the delay.
Many thanks
uk-bolly

ansible-galaxy install mindpointgroup.rhel8_cis

Thanks for sharing.


Help build a basic example

Hi I'm trying to build a basic working example and I've found some difficulties (using CLion IDE), and now I need help to get started.
Following the suggested steps did not make me produce a working project, basically what I've done is:

Set up a new project in CLion
Copied the SQLiteCpp directory into it

My main.cpp basic example is
#include <iostream>
#include <cstdio>
#include <cstdlib>
#include <string>

#include <SQLiteCpp/SQLiteCpp.h>

#ifdef SQLITECPP_ENABLE_ASSERT_HANDLER
namespace SQLite
{
/// definition of the assertion handler enabled when SQLITECPP_ENABLE_ASSERT_HANDLER is defined in the project (CMakeList.txt)
void assertion_failed(const char* apFile, const long apLine, const char* apFunc, const char* apExpr, const char* apMsg)
{
    // Print a message to the standard error output stream, and abort the program.
    std::cerr << apFile << ":" << apLine << ":" << " error: assertion failed (" << apExpr << ") in " << apFunc << "() with message \"" << apMsg << "\"\n";
    std::abort();
}
}
#endif

/// Get example path
static inline std::string getExamplePath()
{
    std::string filePath(__FILE__);
    return filePath.substr( 0, filePath.length() - std::string("main.cpp").length());
}

/// Example Database
static const std::string filename_example_db3 = getExamplePath() + "/example.db3";
/// Image
static const std::string filename_logo_png    = getExamplePath() + "/logo.png";

int main() {
    try
    {
        // Open a database file
        SQLite::Database    db("example.db3");

        // Compile a SQL query, containing one parameter (index 1)
        SQLite::Statement   query(db, "SELECT * FROM test WHERE size > ?");

        // Bind the integer value 6 to the first parameter of the SQL query
        query.bind(1, 6);

        // Loop to execute the query step by step, to get rows of result
        while (query.executeStep())
        {
            // Demonstrate how to get some typed column value
            int         id      = query.getColumn(0);
            const char* value   = query.getColumn(1);
            int         size    = query.getColumn(2);

            std::cout << "row: " << id << ", " << value << ", " << size << std::endl;
        }
    }
    catch (std::exception& e)
    {
        std::cout << "exception: " << e.what() << std::endl;
    }
}

My CmakeLists.txt is the following
cmake_minimum_required(VERSION 3.6)
project(dbextractor)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")

include_directories(SQLiteCpp/include)
add_subdirectory(SQLiteCpp)

set(SOURCE_FILES src/main.cpp)
add_executable(dbextractor ${SOURCE_FILES})
target_link_libraries(SQLiteCpp)

Now trying to compile a basic example gives errors such as undefined reference to SQLite::Database::Database(char const*, int, int, char const*) and lots of similar.
I'm sure it has something to do with the Cmake file, but I did not find any ready-to-use example other than the code snippet, which is ok but does not give an idea about how to build a program from scratch.
Thanks for your help!

Hi @beraldofilippo, what you have done is already quite good, I'll try to create a new side-repository with an example of "from scratch project" so that I can discover what can possibly go wrong.
Cheers!

Thanks! This will be for sure  a useful resource for everyone!

Hi @beraldofilippo
There is a simple mistake in your CMakeLists.txt, target_link_libraries() should target your executable, the the sqlite3 C library (and also, link pthread and dl on Linux, but not on Windows) :
cmake_minimum_required(VERSION 3.6)
project(dbextractor)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")

include_directories(SQLiteCpp/include)
add_subdirectory(SQLiteCpp)

set(SOURCE_FILES src/main.cpp)
add_executable(dbextractor ${SOURCE_FILES})
target_link_libraries(dbextractor SQLiteCpp sqlite3 pthread dl)

Please have a look at the new SQLiteCpp_Example side repository for a standalone "from scratch" example.
(for now only tested under Linux)
Thanks again for reporting this common problem !
Cheers!

Great! Thanks!


[3.0] Improve compressed URL param performance

What is the purpose of the change
assume key is in constant pool, improve performance
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #10125 (46a90b9) into 3.0 (4689fef) will decrease coverage by 0.05%.
The diff coverage is 92.10%.

@@             Coverage Diff              @@
##                3.0   #10125      +/-   ##
============================================
- Coverage     65.74%   65.68%   -0.06%
+ Complexity      320      294      -26
============================================
  Files          1220     1220
  Lines         53144    53169      +25
  Branches       8057     8059       +2
============================================
- Hits          34937    34926      -11
- Misses        14393    14422      +29
- Partials       3814     3821       +7

Impacted Files
Coverage Δ

.../common/url/component/param/DynamicParamTable.java
88.33% <92.10%> (-0.24%)
:arrow_down:

...nt/metadata/ServiceInstanceHostPortCustomizer.java
65.78% <0.00%> (-21.06%)
:arrow_down:

...che/dubbo/registry/client/NopServiceDiscovery.java
44.44% <0.00%> (-11.12%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
58.13% <0.00%> (-9.31%)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
51.13% <0.00%> (-9.10%)
:arrow_down:

...pache/dubbo/registry/support/AbstractRegistry.java
73.80% <0.00%> (-2.05%)
:arrow_down:

...exchange/support/header/HeaderExchangeHandler.java
61.06% <0.00%> (-1.77%)
:arrow_down:

...ubbo/registry/client/AbstractServiceDiscovery.java
85.12% <0.00%> (-1.66%)
:arrow_down:

...ting/zookeeper/curator/CuratorZookeeperClient.java
69.23% <0.00%> (-0.97%)
:arrow_down:

...dubbo/remoting/exchange/support/DefaultFuture.java
87.17% <0.00%> (-0.86%)
:arrow_down:

... and 5 more

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 4689fef...46a90b9. Read the comment docs.


Code Assist AI for devs building with Fluent UI in Flutter

Dear maintainers, greetings from CommandDash!
We are a tool to turn docs and examples of your library into a code generation AI agent which helps devs directly generate code for your library as per their requirements.
Our team came across Flutter Fluent UI and figured it could be helpful for new developers building on it to have one such agent. It's ready and is accessible on: Web (Try here!) | VSCode Extension
You can link this in your readme as a badge:

Please feel free to mention it if you believe this could be helpful for new users. Or, close the issue if not. 🙏🏻
The agent is free to use, fully safe (no data retained) and auto-refreshes to stay upto date.
Best wishes!

I will let @bdlukaa give an answer to you. But I tried and honestly I'm impressed.
I give it that prompt:

How to create a button with a background red (while keeping the hover and pressing effects with a darker red) and a blue border. The text inside the button should be "Confirm".

And here is the result:

To create a button with a red background, a darker red hover and pressing effect, a blue border, and the text "Confirm" using the fluent_ui package, you can customize the ButtonStyle accordingly. Here's how you can achieve this:
import 'package:fluent_ui/fluent_ui.dart';

class CustomButtonExample extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return ScaffoldPage(
      header: PageHeader(title: Text('Custom Button Example')),
      content: Center(
        child: Button(
          onPressed: () {
            // Handle button press
          },
          style: ButtonStyle(
            backgroundColor: WidgetStateProperty.resolveWith((states) {
              if (states.isPressed) {
                return Colors.red.darker;
              } else if (states.isHovered) {
                return Colors.red.dark;
              } else {
                return Colors.red;
              }
            }),
            foregroundColor: WidgetStateProperty.all(Colors.white),
            shape: WidgetStateProperty.all(
              RoundedRectangleBorder(
                borderRadius: BorderRadius.circular(4.0),
                side: BorderSide(color: Colors.blue),
              ),
            ),
            padding: WidgetStateProperty.all(EdgeInsets.symmetric(horizontal: 16.0, vertical: 8.0)),
          ),
          child: Text('Confirm'),
        ),
      ),
    );
  }
}

void main() {
  runApp(FluentApp(
    home: CustomButtonExample(),
  ));
}

In this example:
backgroundColor is set using WidgetStateProperty.resolveWith to provide different colors based on the button's state (pressed, hovered, or default).
foregroundColor is set to white to ensure the text is visible against the red background.
shape is set to a RoundedRectangleBorder with a blue border.
padding is added to give some space around the text inside the button.

This will create a button with the specified styles and the text "Confirm". The button will have a red background, a darker red color when hovered or pressed, and a blue border.
Source
/...-master-lib-src-controls-inputs-buttons-theme.dart
/...master-lib-src-controls-inputs-buttons-button.dart


Parsing error when using extrapolated variable as extend name

Getting parsing errors when I try to use an extrapolated variable in a silent extend.
Warning: Parsing error at test/fixtures/test1.scss: Please check validity of the block starting from line #3 Use --force to continue.

$var: 'foo';

%#{$var} {
  font-size: 18px;
}

Hey @micahgodbolt - thanks for the issue. It's currently being caused by the parser so I've added an issue. Here's a reference https://github.com/tonyganch/gonzales-pe/issues/110

Confirmed as working in gonzales-pe 3.2.1

As far as I can see gonzales-pe >= 3.0.0 fixed the issue. When I look at the PR it seems you are preparing to upgrade to version 3.2 soon, right? Do you guys have any eta when this will be finished? Or maybe upgrade to 3.0.x before 3.2 if there are no so many breaking changes in it?
We would love to use sass-lint, but this bug keeps us from doing that.

3.0 contains as many breaking changes as 3.2.. there are still regressions in 3.2 that are going to need to be fixed before this is complete but fingers crossed! As for an eta, whenever we get the time to work on it, i'm afraid i can't be any more specific than that.
Might be worth keeping an eye on this issue though, https://github.com/sasstools/sass-lint/issues/495

@micahgodbolt The parse error has been fixed in 1.6.

ping @danez too ^^


Install on OSX with OSX Server running?

Hi,
I've just installed CodeJS and itunes-api and it doesn't seem t be working.
When I hit localhost:8181 from a browser (to check it's working)  I get nothing...  Should I be getting anything.
I'm running Mavericks & OSX Server with Web sites enabled in the server console.
Also, where might I find the logs?
Thx
JP

try running with just script/server to get logs. Also if you script/install, you should find logs in the logs directory.

Thanks for that - when I run script/server (with and without sudo) I get the following;
jjpeet:iTunes-API james$ sudo script/server
Password:
warn:    --minUptime not set. Defaulting to: 1000ms
warn:    --spinSleepTime not set. Your script will exit if it does not stay up for at least 1000ms
error: Could not read .foreverignore file.
error: ENOENT: no such file or directory, open '/Users/james/iTunes-API/.foreverignore'
module.js:327
    throw err;
    ^

Error: Cannot find module 'express'
    at Function.Module._resolveFilename (module.js:325:15)
    at Function.Module._load (module.js:276:25)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/Users/james/iTunes-API/app.js:4:15)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Function.Module.runMain (module.js:441:10)
error: Forever detected script exited with code: 1

Also, I found the log file in ~/.forever/itunes-api.log and it has this in it;
jjpeet:.forever james$ tail itunes-api.log
      ^

Error: ENOENT: no such file or directory, open 'log/logs.log'
    at Error (native)
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: ENOENT: no such file or directory, open 'log/logs.log'
    at Error (native)

Any thoughts?
JP

Is there a logs dir in the itunes-api dir?
If not, try making one first and then starting the server.
On August 19, 2016 at 10:05:16 PM, James Peet<EMAIL_ADDRESS>wrote:

Thanks for that - when I run script/server (with and without sudo) I get
the following;
jjpeet:iTunes-API james$ sudo script/server
Password:
warn:    --minUptime not set. Defaulting to: 1000ms
warn:    --spinSleepTime not set. Your script will exit if it does not stay up for at least 1000ms
error: Could not read .foreverignore file.
error: ENOENT: no such file or directory, open '/Users/james/iTunes-API/.foreverignore'
module.js:327
throw err;
^
Error: Cannot find module 'express'
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (/Users/james/iTunes-API/app.js:4:15)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)
at Function.Module.runMain (module.js:441:10)
error: Forever detected script exited with code: 1
Also, I found the log file in ~/.forever/itunes-api.log and it has this in
it;
jjpeet:.forever james$ tail itunes-api.log
^
Error: ENOENT: no such file or directory, open 'log/logs.log'
at Error (native)
events.js:141
throw er; // Unhandled 'error' event
^
Error: ENOENT: no such file or directory, open 'log/logs.log'
at Error (native)
Any thoughts?
JP
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/maddox/itunes-api/issues/21#issuecomment-241172277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABBOP_NRJskzd_Vd1gmm5Bg1_BdTUrks5qhmDcgaJpZM4Jjkz_
.

hi,
Is there an itunes api to use on windows ??

Sorry no, iTunes API uses AppleScript, which isn't available on Windows.


Add multiple specialization constants to Forward+ and Mobile.

Since https://github.com/godotengine/godot/pull/90400 is now merged, it is now possible for us to further specialize shaders without adding stutters when the configuration of the scene changes.
I turned the following dynamic attributes of the shader into specialization constants and got some measurable improvements in frame time already. Most of these specializations are focused on the mobile renderer.

Omni light count
Spot light count
Directional light count
Reflection probe count
Multimesh format
Ambient cubemap toggle
Reflection cubemap toggle
Scene roughness limiter

TPS Demo from here
Average GPU Time in Mali-G78

master: 31.2ms
mobile-scs: 28.6ms

Further improvements could be found as I dig deeper into the shader for more optimization opportunities.

For future reference for others, these are the low hanging fruit optimizations we can do with spec constants. In the future, we can continue to investigate what other areas would benefit from specializations.

Thanks!


Session service

-User can create a Session using the post request
-Token gets generated and assigned to Session

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: dkars2s:x: OthmanMoukhliYou have signed the CLA already but the status is still pending? Let us recheck it.


Changes in timezone are not handled in parsing dates

Describe the bug
dateTimeProperty is not being parsed correctly and in accordance with timezone changes.
To Reproduce
Steps to reproduce the behavior:

Run the full test suite of cicero

Expected behavior
The following tests should have passed
1) FormattedDates
       #draft - formatted-dates-DD_MM_YYYY
         should be able to roundtrip formatted-dates natural language text:

      AssertionError: expected 'dateTimeProperty: 30/11/2018' to equal 'dateTimeProperty: 01/12/2018'
      + expected - actual

      -dateTimeProperty: 30/11/2018
      +dateTimeProperty: 01/12/2018

      at Context.<anonymous> (test/formattedDates.js:68:27)

  2) FormattedDates
       #draft - formatted-dates-MM_DD_YYYY
         should be able to roundtrip formatted-dates natural language text:

      AssertionError: expected 'dateTimeProperty: 12/30/2019' to equal 'dateTimeProperty: 12/31/2019'
      + expected - actual

      -dateTimeProperty: 12/30/2019
      +dateTimeProperty: 12/31/2019

      at Context.<anonymous> (test/formattedDates.js:68:27)

Desktop (please complete the following information):

OS: Ubuntu 18.04

Additional Context
https://github.com/accordproject/cicero/pull/525#issuecomment-605024212

runVMScriptCall(utcOffset,now,options,context,script,call) {
        const vm = new VM({
            timeout: 1000,
            sandbox: {
                moment: moment,
                logger: Logger,
                utcOffset: utcOffset,
                now: now,
                options: options
            }
        });
        vm.freeze(context, 'context');
        vm.run(script);
        return vm.run(call);
    }

I think this part of the code isn't able to handle timezone changes. It is not converting the time parsed correctly.

@jeromesimeon Please look into this if you have time. :sweat_smile:

@jeromesimeon Please look into this if you have time. 😅

I have a lot on my list for the coming release, but it's on my radar. Please bear with me.

@jeromesimeon Sure! :)

@algomaster99 Shall I work upon this issue.

@algomaster99 This issue doen't exist any more. So it is safe to close this issue.

@Cronus1007 thanks! I think https://github.com/accordproject/cicero/issues/547 takes care of it.


AppDesigner/Editors code style clean up

Applied the rules applied by https://github.com/dotnet/roslyn-project-system/pull/1002, to editors/appdesigner.
This fixes (what appears to be - it's hard to tell because of https://github.com/dotnet/roslyn/issues/15993) the last "real" code style issues with AppDesigner/Editors.
The ones that left are due to https://github.com/dotnet/roslyn/issues/15762.

tag @dotnet/project-system

I've got a follow up PR coming that basically cleans up all the resource classes.


docker version output is not consistent when there are downgrades or incompatibilities #30994

[1.13] docker version output is not consistent when there are downgrades or incompatibilities #30994
Signed-off-by: Daniel Zhang<EMAIL_ADDRESS>

@thaJeztah,
What do we want to improve? By the filer, seems to avoid error arise? Could you please describe more about the requirement?

@thaJeztah, any new advice?

@thaJeztah, already done, please let me know if there is any problem

LGTM! thanks @jmzwcn


can not set default text background and frame

Hello again,
Project 'starred'!
I've tested the version you sent above and thank you for it! For some reason I don't understand, the text feature always offers me the second option, when it comes to writing on the screen : by default the version you sent me is set to 'frame + transparent background', the second option. Could I ask you to activate the 'transparent background' feature by default?
For me, it's quite a handicap to have to disable the frame by switching to option 1 (only transparent background) that you suggest, every time I write with the text tool. Thanks for your wonderfull app and thank you for your understanding.
All the best,
Ventilo
ps: since i understand you are living in france, i assume you speak french. So let me try a translation. If i'm wrong then do not bother with this!
ReBonjour à toi,
j'ai testé la version que tu as envoyé plus haut et t'en remercie sincèrement! Pour une raison que je ne comprends pas, avec cette version, la fonctionnalité text (T) me propose toujours la seconde option, quand il s'agit d'écrire sur l'écran (par défaut  la version que tu m'a envoyés est mise sur 'un cadre + un background transparent'). Pourrai-je te demander que  la fonctionnalité 'transparent background' soit activé par défaut, comme cela semble être le cas sur la version que tu avais proposée dans un thread antérieur, version qui était nommée 'ppink_shift_control_hand'.
Avoir  à désactiver le cadre en passant a l'option 1 que tu proposes à chaque fois qu'on écrit avec l'outil texte est assez contraignant.
Un grand merci pour ton travail pour cette magnifique app.
et
Merci d'avance de ta compréhension,
Très bonne journée,
Ventilo
Originally posted by @Ventilo23 in https://github.com/pubpub-zz/ppInk/discussions/315#discussioncomment-10598654

Here is a very first beta :

miss a way to change the setup through the options dialog - currently you need to modify config.ini),
miss the option in the rest API

ppInk Text BackGround.zip

Thanks so much !!
Have a nice week-end!

Meanwhile, i see that you have implemented another pplink version that fix the scale bug concerning the numbers icons. The version you sent on the thread i mention is this one:  'ppInkScaleTag.zip'
Unfortunately the version you sent me priorly do not integer this fix.
May I ask if you could send me a new version including both fixes?
Thanks a lot, in advance,
Best Regards,
Ventilo

here is the full version allowing changing the value in the options and with the parameters in REST API. I've also added SetTagNumber to complete #308
ppInk Text BackGround 2.zip

@Ventilo23
Have you been able to test the beta version?

Sorry for the delay, was not up to my screen. Thanks a lot for this. I'll test it and get back to you. Many thanks,
Regards.

" allowing changing the value in the options and with the parameters in REST API"
What do you mean by REST API? Do you mean, the option are now accessible from the option dialog?
Thanks in advance,

" allowing changing the value in the options and with the parameters in REST API" What do you mean by REST API? Do you mean, the option are now accessible from the option dialog?
Thanks in advance,

REST API allow you to send commands to ppink to 'remotely' control it, for example you can control ppink from a streamdesk device (cf https://github.com/pubpub-zz/ppInk#rest-api)
else, yes you can modify the "default" text background/frame.

So, just to keep you up to date, I haven't beta tested yet. I'm running out of precious time to do so, but I'll try tomorrow, count me in.
I have a secondary question,
Would it be possible to constrain the creation of lines (line tool) to a 90° angle, using one of these commands: shift ,Alt or Ctl? This would be very useful for tracing lines on a diagram or chart. It's just a question of comfort. But it would be very useful. Ideally, to be able to configure this angle in the options, would be great!
Thanks again for all your hard work, it's great!
Best regards,
Ventilo
Translated with DeepL.com (free version)

So, just to keep you up to date, I haven't beta tested yet. I'm running out of precious time to do so, but I'll try tomorrow, count me in.

thx. It just to have a second way of doing thing.

I have a secondary question, Would it be possible to constrain the creation of lines (line tool) to a 90° angle,

ppInk has lots and lots of features including this one : You should have a look at the help on ppInk page (https://github.com/pubpub-zz/ppInk) and look at the magnetic effect
using one of these commands: shift ,Alt or Ctl? (imho, ctl+linetool= 90° straight lines would be the most appropriate.) And hence, this would be very useful for tracing lines on a diagram or chart and made those line, 100% straight. It's just a question of comfort.
Remember that the default value for the magnetic effect in "angle" is 15°
I experienced sometimes when i want to redraw a line which is 90°, I lack the possibility to match those 90°. I hope I am clear.. ..But it would be very useful. Ideally, Also, to be able to configure this angle in the options, would be great! I understand, i made a lots of ask, so feel free to consider this one or not. Thanks again for all your hard work, it's great! Best regards, Ventilo
RTFM! 😳😜😀

So i did test your beta, and thank you a lot for this !!
Meanwhile, I tested the magnet effect. If i may this is not what I asked in my prior message.
Magnet will magnet two lines to eachother with a desired radius.
What i'd like to be implement is the ability to constraint a line draw to be at 90% angle might it be vertical or horizontal. I don't need to magnet lines. I just need to create perfect straight lines. And i asked this function to be activated with the Ctl+ Line tool.
Hope i'm clear. Though you're right, my first intuition is to ask on the forum. Not the proper way, i agree.
NB: i did not get what was those very first numbers ( 0.625) I understand the degree number next to it, but can't figure out those numbers. :

if you press Shift will will only use the "angle" magnetic effect that will allow you to draw lines aligned on 15° (default value adjustable in the options). it will be then be very easy to get on  0°/90°/180°/270° (you will just get intermediate values which should not be an issue)
Ctrl is to ignore the angle alignement but catch points

update:
"if you press Shift will will only use the "angle" magnetic effect that will allow you to draw lines aligned on 15° (default value adjustable in the options). it will be then be very easy to get on 0°/90°/180°/270° (you will just get intermediate values which should not be an issue)"
Sorry, but shift option + line tool does not constrain anything (on the version of the soft you sent me). Might it be the angle to 15° or whatever degree you choose (90° e.g).
Am i missing something? Could the shift button already be assigned to another task and hence, not working when adding it to the line tool?
Regards and thanks,
Ventilo.

this is an example of jumps to align on angles:
2024-09-22 09-36-02.zip
if you have different result show me

ok, so first of all, thanks a million for this.
Let me switch to french for the ease of understanding, i'll tempt a traduction after.
1- Merci mille fois pour ta vidéo et qui semble assez claire! Merci encore pour le temps que tu m'accordes!
2- Apres visionnage de ta video:
-Quand tu souhaites tracer une ligne avec l'outil ligne tu sembles te servir du texte que tu as écris (et surtout du cadre de ce texte) comme 'snap'. Ce n'est pas, si je peux me permettre, ce que je recherchais.
-Je me demandais s'il était possible, quand on est dans l'outil ligne, de contraindre à 90°, la création de cette ligne (verticalement et/ou horizontalement), sans avoir a utiliser un quelconque snap sur une forme autour. Juste simplement faire que la ligne, au lieu qu'elle ne se crée ou l'on veut, comme nous le faisons lorsque le snap n'est pas actif, qu'on puisse la crée que sur un degré choisi:
SI je choisi 90° comme angle de contrainte, et bien j'aurai aimé que la ligne se forme a 90° (horizontalement ou verticalement, en fonction de mon choix). Chez moi, et c'est peut être dû à la mauvaise configuration de mes options, si:
1- je pars d'une feuille blanche,
2- j'invoque l'outil ligne
3- j'invoque le snap (que j'ai préalablement réglé à 90° comme suit, dans les options:

4- Je m'attendais a ce que, lors de l'utilisation de l'outil Ligne, la ligne que je souhaite crée soit crée directement avec un angle de 90°. Sans avoir a passer par un snap sur une autre forme.
5- J'ajoute que chez moi, invoquer l'outil aimant tel que je l'ai configuré (cf mon image plus haut) n'a aucune incidence sur le degré choisi, qui contraindrait la création de ma ligne à l'angle que je désirerai (ici pour moi 90%). (Désolé pour cette phrase compliquée mais j'ai du mal a la formuler autrement :)).
Je suis bien d'accord, c'est difficile a comprendre sans l'appui d'une video ou explication orale.. J'avoue que je ne serai trop comment l'expliquer même avec la video, et c'est pour cela que je ne t'en envoie pas.
La partie de ta video (à la seconde 00:58) ou tu montres que tu bouges ta ligne par ce qui semble être des incréments de 15 degré est très intéressante.
Mais je crains que ce soit en fait des snaps sur le cadre du texte que tu as écris qui donne l'impression que la ligne que tu souhaites crée est bien contrainte à 15°
Puis je te demander une copie ecran de ton paramétrage de snap? Mais comprend bien que pour l'instant l'outil aimant, aimante à une forme, ou a une arrête (une edge) d'une forme crée,  alors que j'aurai aimé qu'il aimante à un degré choisi, sans avoir a s'appuyer sur une forme dessiner a coté.
Merci encore,
Ventilo

English traduction:
1- Thanks a million for your video, which seems pretty clear! Thanks again for your time!
2- After watching your video:
-When you want to draw a line with the line tool, you seem to use the text you've written (and especially the frame of this text) as a snap. This is not, if I may say so, what I was looking for.
-I was wondering if it was possible, when using the line tool, to constrain the creation of this line (vertically and/or horizontally) at 90°, without having to use any snap on a surrounding shape. Just make sure that, instead of creating the line wherever you want, as we do when the snap is not active, you can only create it to a chosen degree:
IF I choose 90° as the constraint angle, then I'd like the line to form at 90° (horizontally or vertically, depending on my choice). In my case, and this may be due to the poor configuration of my options, it's not:
1- I start with a blank sheet of paper,
2- I invoke the line tool
3- I invoke the snap (which I've previously set to 90° as follows, in the options:

4- I expected that, when using the Line tool, the line I wanted to create would be created directly at a 90° angle. Without having to snap to another shape.
5- I'd like to add that for me, invoking the magnet tool as I've configured it (see my image above) has no effect on the degree chosen, which would constrain the creation of my line to the angle I want (in this case 90%).
I agree, it's hard to understand without video support... I confess I wouldn't know how to explain it even with the video, and that's why I'm not sending you one.
The part of your video (at second 00:58) where you show that you move your line in what appear to be 15-degree increments is very interesting. But I'm afraid they're actually snapshots on the frame of the text you've written that give the impression that the line you want to create is constrained to 15°.
Can I ask you for a screen copy of your snap settings?
Thanks again,
Ventilo

Tiens, en fait, si tu pars du principe que l'outil rectangle te crée un rectangle  donc des lignes droites. Et bien je souhaiterai pouvoir faire la même chose avec l'outil ligne, que l'outil ligne te crée directement des lignes parfaitement droite (verticale ou horizontale).

if you press Shift will will only use the "angle" magnetic effect that will allow you to draw lines aligned on 15° (default value adjustable in the options). it will be then be very easy to get on 0°/90°/180°/270° (you will just get intermediate values which should not be an issue)
Ctrl is to ignore the angle alignement but catch points

ha.. so this, if it works, with my option settings and my software version doesn't work: i feel no constraint. The constraint is only active when it is next to a shape.  Or at least i don't know how to set it properly.
As said in my previous messages, feel free to send me yours!

Here is a small video on how my pplink behave at time of drawing a line.
Second line tracement, i did press the shift key. But no 90° constraint where active. Neither for the first exemple, by the way.
https://www.swisstransfer.com/d/3c502223-bb06-4ad0-8421-a8a44d1af4bd
Take care,
Ventilo.

I think I've got the trick!
The issue is coming from the fact that you are using the "standard" french regional setup (using the comma as decimal separator)
This version should fix this:
ppInk French magnetic.zip
PS: you should not need to activate the magnet to use the shift to stick to directions
PPS : I came to this conclusion only because of the french message you wrote:😜

Simply brilliant! What insight!
Thanks a million!
Pss:  ( erased but Long one!.. ;), thanks again for your time and patience!)

In french s'il vous plait... 😄😄😄😄

Update:  i get this error message at open (i

and if i press 'no' then
I get this:

(language section (in french: langue) on the left side, as no language loaded.
NB: when i first installed your version, it was set to 'english' for the language. And i think it worked perfectly fine.
Then i switch to french to test the interface in french, and i got this bug.
I can of course (i think) reinstall the version you sent me and keep the english language (not a problem for me) but wanted to mention those 2 minor bugs.
Regards
Ventilo

these are related. Can you tell me the path where you are using ppInk?

from my windows download folder.

small update, i cannot reuse the .exe you sent me, it will reproduce the bug. Now i did not test yet , to redownload the .zip file you sent me priorly. I guess this should make the program work but in english language, which is fine to me.

For me the issue is coming from the fact that the program did not found the language files.
Find attached a new  version. I've added some debug traces
ppInk anapb1.zip
To get the traces, goto the options, and click on top right button "Debug Window"
Can you retry what you have done and look for "loading ..." in the debug window in this case
Thanks

any update?

@Ventilo23.
Any update ?

hey pubpub,
Sorry for the late reply! I was very very busy..
Working perfectly! Thanks a lot.


Update readme with wait info

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   96.85%   96.85%
=======================================
  Files          11       11
  Lines         127      127
=======================================
  Hits          123      123
  Misses          4        4

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 0786ec9...05e26b1. Read the comment docs.


[Discussion / ~Idea] .pdf files that track users

Hey there,
Today the following article came up:
https://www.heise.de/meinung/Verlag-Elsevier-Unser-Tracking-ist-Ransomware-Schutz-6345492.html
It's in german, but there are probably english-related websites that explain what is going on. A quick
summar is that elsevier (as publisher) embeds some meta-code into a .pdf file to identify "bad
users", aka you purchase something, a .pdf file, and get tracked. My concern is less so with elsevier
but more generally when .pdf files are used for tracking, identifying and fingerprinting users. I myself
have largely transitioned into .pdf files - while I still love oldschool books, I also like that I don't have
books that take up physical space (I still own and retain many important books, but a .pdf file is
super-practical really - efficient to store the information).
So what does this have to do with qpdf? Well, nothing directly, but it would be nice if there could be
some way to query a .pdf file and give some summary or something as to whether it may track
a user or not. Or, if that is too hard to implement, whether any summary that is used for qpdf to
yield information to the user, that tracking is kept in mind. I think many users would like to know
whether they have .pdf files that can be used to fingerprint them.
I don't necessarily mean that qpdf should change anything as-is, but I wanted to mention it as a
thing to keep in mind for the future. Please do feel free to just close this issue request - it really
is meant more as a discussion / idea.

This is an interesting problem. I'm not sure what mechanism a PDF file could use to track someone, though it is possible to embed javascript in PDF under certain conditions. The viewer would have to support it. I imagine it would be possible to systematically remove javascript from a PDF file, but I'll admit to not being familiar with that part of the spec. I also have no idea whether that's the issue -- there could be other things, like using external URLs. I'll leave this open until I have a chance to investigate. Thanks for raising the issue.

FYI it seems to be embedded in the XMP meta data, so removing would be quite easy. However, detecting and/or removing tracking would be quite complicated, see e.g. https://twitter.com/json_dirs/status/1486274161655771136/photo/1

Initial tweet: https://twitter.com/json_dirs/status/1486120144141123584
Gist with hashes and Python extract script: https://gist.github.com/sneakers-the-rat/6d158eb4c8836880cf03191cb5419c8f

Btw. qpdf was mentioned several times on those Twitter threads ;)


Handle OAuth errors gracefully

Recognize them, and handle them correctly, in particular when the error is just the user cancelling the configuration of a device.

Pull Request Test Coverage Report for Build 97

3 of 20 (15.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 66.202%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/helpers/oauth2.js
1
18
5.56%

Totals

Change from base Build 92:
-1.3%

Covered Lines:
301

Relevant Lines:
429

💛  - Coveralls


Compilation error with Phoenix 1.7

Describe the bug
In Phoenix 1.7 project compilation fails with this error:
== Compilation error in file lib/my_app_web/router.ex ==
** (RuntimeError) attempting to redefine live_session :default.
live_session routes must be declared in a single named block.

    (phoenix_live_view 0.18.3) lib/phoenix_live_view/router.ex:237: Phoenix.LiveView.Router.__live_session__/3
    lib/my_app_web/router.ex:37: (module)
    (elixir 1.14.2) lib/kernel/parallel_compiler.ex:346: anonymous fn/5 in Kernel.ParallelCompiler.spawn_workers/7

To Reproduce
Add ash_admin macro to the router file of Phoenix 1.7-rc0 application.
Expected behavior
No errors
Runtime

Elixir version 1.14.2
Erlang version 25
Ash version 2.4.24
AshAdmin 0.7.0
Phoenix 1.7-rc0
Phoenix LiveView 0.18.3

Same error on phoenix-1.7 branch too.

Thanks for the report! can you try out the phoenix-1.7 branch again?

Error message is slightly different now (redefine live_session :default redefine live_session :ash_admin):
== Compilation error in file lib/table_hero_web/router.ex ==
** (RuntimeError) attempting to redefine live_session :ash_admin.
live_session routes must be declared in a single named block.

    (phoenix_live_view 0.18.3) lib/phoenix_live_view/router.ex:237: Phoenix.LiveView.Router.__live_session__/3

🤔 are you calling the function twice?

oh yes, sorry

all good now, closing the issue.


ZooKeeper sometimes fails to start

When performing a appscale down --clean && appscale up, ZooKeeper failed to start for me. The following was in the logs:
java.io.IOException: Unable to create data directory /opt/appscale/zookeeper/version-2
        at org.apache.zookeeper.server.persistence.FileTxnSnapLog.<init>(FileTxnSnapLog.java:85)
        at org.apache.zookeeper.server.ZooKeeperServerMain.runFromConfig(ZooKeeperServerMain.java:110)
        at org.apache.zookeeper.server.ZooKeeperServerMain.initializeAndRun(ZooKeeperServerMain.java:87)
        at org.apache.zookeeper.server.ZooKeeperServerMain.main(ZooKeeperServerMain.java:53)
        at org.apache.zookeeper.server.quorum.QuorumPeerMain.initializeAndRun(QuorumPeerMain.java:116)
        at org.apache.zookeeper.server.quorum.QuorumPeerMain.main(QuorumPeerMain.java:78)

Looks like there is a timing issue to reproducing this but it happens frequently if not every time.
On appscale down --clean we should perform a systemctl daemon-reload so that any runtime systemd configuration that was deleted is applied. I think this will prevent zookeeper attempting to start too soon on appscale up.
The zookeeper drop-in unit (10-appscale-zookeeper.conf) should probably be updated to include:
AssertPathExists=/opt/appscale/zookeeper
as we delete this directory on appscale down --clean and zookeeper cannot run without it.


Support setting nproc limit

This PR introduces the ability to set the nproc limit on containers. There are at least two use cases for this. The first being compatibility with Heroku's buildpacks, which use ulimit -u (example 1, example 2) to detect dyno size and adjust parameters accordingly. The second being that nproc control is great for security purposes.
It is meant to be an optional part of the constraints spec, and not enabled by default for anything but the fixed dyno sizes. Note that this PR sets nproc for 1X and 2X dynos, but not PX. This is because buildpack-jvm-common sets too high of a heap size limit for the legacy dyno spec that Empire is using, and I fear that might lead to bad things. A custom nproc can be set by appending a third value to the constraints spec.

This looks fantastic. Any comments @phobologic?

I think this is great - but I have one concern, and that's largely the syntax of the command.  It's getting a little unwieldy to keep appending new values on the end of the scale command, and this one is a bit arbitrary - there are 15 other Ulimits that ECS accepts, and I could see folks following this as an example when they add their favorite ulimit.
Not sure what the best way to fix this is - I think in the long run we just have to start getting away from the heroku CLI and come up with something that handles all the features available to us.  For now, I'm cool with accepting this (because having nproc would be great), but I'd be curious if we can come up with a more generic, extensible syntax for modifying these things. @protomouse ?

@phobologic Having a separate command for this would be great. Something like:
$ emp container web ulimits set nproc=256 nofile=65536
$ emp container web ulimits
  nproc: 256
  nofile: 65536
$ emp container web ulimits unset nofile
$ emp container worker set readonlyRootFilesystem=true
$ emp container web security-opts add type:docker_apache_t

What's up with the tests timing out by the way?

Alright, so apparently there are some subtle differences in how the SQL package treats int64 compared to ordinary int types. This caused a bunch of reflect-related errors when we put this into our staging environment. This is most likely what happened with the tests as well. Nproc is now an uint.

Cool! This looks great to me. @phobologic brought up that it might be nice the make the third component of the scale (cpu:mem:xxx) a comma separated list of key=value pairs, so you could do:
emp scale web=1:256:30mb:nproc=100

It's a little more readable and opens it up for adding additional flags that you can set. In the long run, a separate command is better imo, so up to you if you want to change it in this PR, otherwise we'll merge!

The key-value thing definitely seems a bit nicer, so added.

sorry - commented in the commit itself, rather than here - other than that, I think this looks great.  Thanks for following through @protomouse !

@phobologic Saw your comment. I've added a check for unsupported keys.

Awesome - this is great, thanks a ton @protomouse


MODFQMMGR-134 Fix a bug with the material type config

This adds the source property back to item_material_type in the Items entity type, so that the query builder plugin knows how to ask for the available values.
This commit also adds "inventory-storage.material-types.collection.get" to the GET /entity-types sub-permissions, so that users who can access the entity types can also access the available values for that entity type

CI is always mad


Post pagination incorrect

Pagination on page 2 of the public application works incorrectly if several posts have the same published on date and time.

Fixed on release <IP_ADDRESS>30304


is DDoS part of this starter kit?

Found a new Configurable Denial-Of-Service prevention for http services library. Does this starter has something already for DDoS protection? if not, is it possible to integrate with node-ddos? https://github.com/rook2pawn/node-ddos.git

Instead of application-level DDoS protection in production, I would recommend using a 3rd party service such as Cloudflare.  A service like Cloudflare does a much more comprehensive job of protecting the application from DDoS attacks.

That's fair enough, I will go ahead and close this one.


[Feature]: Import application via API

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
We are deploying the application on top of the on-premise Appsmith container once in few day days. As of now we are importing the application from the development environment.
We want to automate this as don't want to manually handle this operation when we wanted to deploy in multiple test environments very soon.
So, we are looking for API to import the application programmatically.
Why should this be worked on?
Programmatically updating the application is a must have feature to avoid the manual intervention of the application and avoiding mistakes.

@dhananjayank you can connect the app to git and import it on both instances. You can then simply pull the master branch to update / deploy the app on the new instance following the normal development workflow you use.
We're also building CI/CD integration here #11522 So you can track that ticket

Reopening this GitHub issue as per user's request.

Few more points to add,

Git connection or CD pipeline wont work for production environments where internet is not allowed.
We wanted to protect the staging environment with controlled changes via the import function

It would be nice if import allows options like "import as new app" to import as new application or "import and override" to override the old app.

@dhananjayank for an environment where internet is not allowed, even the API method would not work since you need internet to trigger it externally right?
I think the real solution here seems to be that we need to ensure that our git feature works for large applications without any delay which seems to be the core problem.

We downloaded the json to a airgap environment and copy the json file to the target machine and can trigger an API. This doesnt call for an internet enabled target machine right?
And you are right about the large applications not working with git. thats also the reason why we wanted to use the import feature.


Add ability to handle import of .ts or .js files

This PR adds the ability to import both .ts files and .js files allowing for a smoother transition when moving a serverless repo to TypeScript or when someone wants to only use .ts when they use the .ts superset.
Tests are also updated.
Basically we just check if the .js or .ts exists.
Because the watcher will throw an error if it can't find a file to watch, we handle it here by checking if the file exists first and if it doesn't throwing an error letting the user know.
I also wasn't sure why dist is included in the repo, but ignored by .gitignore. Therefore the files in dist haven't changed in this PR.

👍 We'd like this functionality as well.

Please, can you merge this PR? :-D

Any idea when this might get merged in? Only thing stopping us from using the plugin, since we need it to work together with the lambda warmup plugin, which is js.

@divyenduz @JackCuthbert

Hey @ro-savage thanks for helping out! I'll take a look at this one (repro and test fix) on the weekend as it looks pretty important :smile:

+1 for this merging please. this is causing us big time headaches with our serverless build.

would love to see this merged, this is blocking me right now

Sorry for the delay on this one friends! I've just been able to repro this for myself and the fix appears to be good :tada: Thanks, @ro-savage!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Work in progress

Let's merge this now to cap off the work done during the away day, then deal with comments in separate issues/PRs


is not working pls help bro

the output is an .py, i want .exe
is not stealing nothing

Not supported anymore, and just learn to use python


chore: migrate to owlbot

Merge after storage team reviews.

Something is still missing the move of gapic_metadata.json from src/main/java to src/main/resources.
I'm not sure what exactly is causing this.
Steps to repro:
git clone https://github.com/googleapis/java-storage.git
cd java-storage
rm -rf {gapic,grpc,proto}-google-cloud-storage-v2/src
docker run --rm --user $(id -u):$(id -g) -v $(pwd):/repo -w /repo gcr.io/cloud-devrel-public-resources/owlbot-cli:latest copy-code
docker run --rm --user $(id -u):$(id -g) -v $(pwd):/repo -w /repo gcr.io/cloud-devrel-public-resources/owlbot-java:latest

git status | grep "gapic_metadata.json"
        deleted:    gapic-google-cloud-storage-v2/src/main/resources/com/google/storage/v2/gapic_metadata.json


Implementing JSON Web Token Library for User Authentication

POC - Working In Progress

Application is NOT break
Build is successful

What I did

Implement JWT Util and Filter classes
General Random JWT Token and send the token to the client

TO DO

[ ] TOO MANY, the functionality is not working

JWT now works but we need to restructure how we handle the authorization token.
Only API calls should require authentication which means tinkering with the AJAX calls to ensure they are pulling from the newly created JWT token cookie.
Lots more work to be done but getting there...

@WAftring, Great job on the session!!!

I see what you mean by there is still a lot needs to be done...

This PR is an absolute monster. @hanwenzhang123 @to-whitney let's schedule some time to go over this.

Need to update the unit tests.


grpc_tools.protoc generates python3 incompatible imports in *_pb2_grpc.py files

What version of gRPC and what language are you using?
libprotoc 3.19.4 for compiling
What did you do?
Folder structure:
/project
    test.proto
    /generated
        __init__.py

# test.proto
syntax = "proto3";

message DataFrame {
  uint32 value = 1;
}

message Empty {
}

service API {
  rpc APISingleData (Empty) returns (DataFrame) {}
}

run in project root $ python -m grpc_tools.protoc --proto_path=. --python_out=generated --grpc_python_out=generated test.proto
What did you expect to see?
In generated file test_pb2_grpc.py I expect there is an import from . import test_pb2 as test__pb2
What did you see instead?
There is an implicit relative import import test_pb2 as test__pb2, which does not work when trying to use generated file. This results of having to manually edit generated files or use additional import fixing script

Here is workaround script that replaces imports if anyone is interested
project strucutre:
/project
    test.proto
    /generated
        test_pb2_grpc.py
        test_pb2.py
        __init__.py
    /fix_imports
        fix_imports.py

# fix_imports.py
import glob

def fix_imports():
    my_path = '..'
    filenames = glob.glob(my_path + '/**/*_pb2_grpc.py', recursive=True)

    print(filenames)

    for filename in filenames:
        data = ''
        with open(filename, 'r') as file:
            for line in file:
                if 'import grpc' not in line and 'from . ' not in line:
                    line = line.replace('import', 'from . import')
                data += line

        with open(filename, 'w') as file:
            file.write(data)

if __name__ == '__main__':
    fix_imports()

run it using python ./fix_imports/fix_imports

Related to https://github.com/grpc/grpc/issues/9575


[SPARK-40876][SQL] Widening type promotion for decimals with larger scale in Parquet readers

What changes were proposed in this pull request?
This is a follow-up from https://github.com/apache/spark/pull/44368 implementing an additional type promotion to decimals with larger precision and scale.
As long as the precision increases by at least as much as the scale, the decimal values can be promoted without loss of precision: Decimal(6, 2) -> Decimal(8, 4):  1234.56 -> 1234.5600.
The non-vectorized reader (parquet-mr) is already able to do this type promotion, this PR implements it for the vectorized reader.
Why are the changes needed?
This allows reading multiple parquet files that contain decimal with different precision/scales
Does this PR introduce any user-facing change?
Yes, the following now succeeds when using the vectorized Parquet reader:
  Seq(20).toDF($"a".cast(DecimalType(4, 2))).write.parquet(path)
  spark.read.schema("a decimal(6, 4)").parquet(path).collect()

It failed before with the vectorized reader and succeeded with the non-vectorized reader.
How was this patch tested?

Tests added to ParquetWideningTypeSuite to cover decimal promotion between decimals with different physical types: INT32, INT64, FIXED_LEN_BYTE_ARRAY.

Was this patch authored or co-authored using generative AI tooling?
No

 - SPARK-34212 Parquet should read decimals correctly *** FAILED *** (364 milliseconds)
[info]   Expected exception org.apache.spark.SparkException to be thrown, but no exception was thrown (ParquetQuerySuite.scala:1055)
[info]   org.scalatest.exceptions.TestFailedException:
[info]   at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:472)
[info]   at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:471)
[info]   at org.scalatest.funsuite.AnyFunSuite.newAssertionFailedException(AnyFunSuite.scala:1564)
[info]   at org.scalatest.Assertions.intercept(Assertions.scala:766)
[info]   at org.scalatest.Assertions.intercept$(Assertions.scala:746)
[info]   at org.scalatest.funsuite.AnyFunSuite.intercept(AnyFunSuite.scala:1564)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.$anonfun$new$213(ParquetQuerySuite.scala:1055)
[info]   at scala.collection.immutable.List.foreach(List.scala:333)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.$anonfun$new$211(ParquetQuerySuite.scala:1054)
[info]   at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
[info]   at org.apache.spark.sql.catalyst.SQLConfHelper.withSQLConf(SQLConfHelper.scala:56)
[info]   at org.apache.spark.sql.catalyst.SQLConfHelper.withSQLConf$(SQLConfHelper.scala:38)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.org$apache$spark$sql$test$SQLTestUtilsBase$$super$withSQLConf(ParquetQuerySuite.scala:47)

The failed test case seems to be related to this PR, could you check it? @johanl-db

 - SPARK-34212 Parquet should read decimals correctly *** FAILED *** (364 milliseconds)
[info]   Expected exception org.apache.spark.SparkException to be thrown, but no exception was thrown (ParquetQuerySuite.scala:1055)
[info]   org.scalatest.exceptions.TestFailedException:
[info]   at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:472)
[info]   at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:471)
[info]   at org.scalatest.funsuite.AnyFunSuite.newAssertionFailedException(AnyFunSuite.scala:1564)
[info]   at org.scalatest.Assertions.intercept(Assertions.scala:766)
[info]   at org.scalatest.Assertions.intercept$(Assertions.scala:746)
[info]   at org.scalatest.funsuite.AnyFunSuite.intercept(AnyFunSuite.scala:1564)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.$anonfun$new$213(ParquetQuerySuite.scala:1055)
[info]   at scala.collection.immutable.List.foreach(List.scala:333)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.$anonfun$new$211(ParquetQuerySuite.scala:1054)
[info]   at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
[info]   at org.apache.spark.sql.catalyst.SQLConfHelper.withSQLConf(SQLConfHelper.scala:56)
[info]   at org.apache.spark.sql.catalyst.SQLConfHelper.withSQLConf$(SQLConfHelper.scala:38)
[info]   at org.apache.spark.sql.execution.datasources.parquet.ParquetQuerySuite.org$apache$spark$sql$test$SQLTestUtilsBase$$super$withSQLConf(ParquetQuerySuite.scala:47)

The failed test case seems to be related to this PR, could you check it? @johanl-db

I fixed it, the last check timed out on another unrelated test though.

@LuciferYang or @cloud-fan since you approved the previous similar change https://github.com/apache/spark/pull/44368, could you take a look at this PR?

thanks, merging to master!


Fix formatting

Autoformat is failing on master builds.

@jamieparkinson have cherry picked that commit

The issue is that changes were pushed straight to master, and the autoformat build step does't have permission to push to master so fails.

PS I don't believe all of these changes were covered by the autofix capabilities


Allow recycling of "converted" items (bronze, etc)

Original request by IncubuzzMC on NexusMods
Awesome mod.
Was not able to recycle bronze ingot even with?
RecyclingRate = 1
Is there a way to do it?

Reworded:
Which mod are you proposing a feature for?
Simple Recycling
Is your feature request related to a problem? Please describe.
Allow recycling (uncrafting) of items created by "conversions" (bronze, etc).
Describe the solution you'd like
It should appear in the recipe list.

Problems: bronze does not have a recipe. The way the mod is currently implemented requires there to be a recipe for it to work.

Wrong. Bronze does have a recipe. Needs further digging


update version for next release

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


add step to create huginn database

not creating the database first leads to error: ERROR 1049 (42000): Unknown database 'huginn_production'

Hi @wasafiri,
we create the database with the first command in the initialize database step, did that not work for you?