Can not delete comment

Please follow the guide below

Issues submitted without this template format will be ignored.
You will be asked some questions and requested to provide some information, please read them carefully and answer completely.
Do not post screenshots of error messages or code.
Put an x into all the boxes [ ] relevant to your issue (like so [x] no spaces).
Use the Preview tab to see how your issue will actually look like.
Issues about reverse engineering is out of scope and will be closed without response.

Before submitting an issue, make sure you have:

[x] Updated to the lastest version v1.4.0
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones
[x] Reviewed the sample code in tests and examples

Which client are you using?

[x] app (instagram_private_api/)
[x] web (instagram_web_api/)

Purpose of your issue?

[x] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[ ] Question
[x] Other

Describe your issue
I found that somtimes it is impossiable to delete media comment. I tried app and web clients. Actualy I found instagram cliend for NodeJS with same problem.  Instagram API returns
{'status': 'ok'}
But comment still on its place. It is possiable to delete it from instagram site or from device

For a bug report, you must include the Python version used, code that will reproduce the error, and the error log/traceback.

I'm not sure that this is a bug in your code. Because usually the comment is deleted normally. But if necessary, the version of Python 3.6.3
And there is no error at all, response is {'status': 'ok'}, but comment still un place.

@ping, I can send you mediaId, commentId and login/password from my test account, so, you can check it


...

@plandem: We use feature branches tied to specific issues to handle QA / PRs. Would you please create a new PR to merge these changes into feature#23-abstract-payload (instead of develop)?

ok. Must i use master as base?

Never mind, Github lets you modify the base. I've take care of it.


Keyboard events are not being emitted

Click events work beautifully, but ketboard events are displatched but never recieved by Reat

We just need to add to the list of events for keyboard events. There is an outstanding issue with ref attribute though, which I need to find a workaround for.

Could you try with v0.2.0?

Yeah, keyboard events are now emitting. But I still cant get the value of the text area, either from ev.target.value or using refs

I can get the value now using event.target.getAttribute('value') but somehow not otherwise. Although hacky but it works. Although now I have another bug where the button is always disabled, but I will add that separately


Graal js does't show error location in script

When a script is evaluated by graal.js, on error it throws exception that shows lineNumber and columnPosition where error occured as -1 and -1 respectively.
With with nashorn engine lineNumber at which error occured is displayed is shown correctly (although it also shows column number as -1).
How can I get the line number in script at which error occured?
Below is code snipppet (I am using graal jvm 22 on mac M1)
public class Main {

    private static final String code = " print(' hello from JS');\n document.write(\"Hello World\");\n var output='DONE'; 'RET';";

    public static void main(String[] args) throws IOException {

    	System.out.println("=== Script To Execute ===");
    	System.out.println(code);

        System.out.println("\n=== Graal JS ScriptEngine ===");

        ScriptEngine scriptEngine = GraalJSScriptEngine.create(null,null);//Context.newBuilder("js") .allowAllAccess(true));
        try {
            Object returnVal = scriptEngine.eval(code);
            Object outputVal = scriptEngine.get("output");
            //System.out.println("return: "+returnVal+", variable output: "+outputVal);
        } catch (ScriptException e) {
        	System.out.println(" Error:"+e.getMessage()+"\n Error At line: "+e.getLineNumber()+", column: "+e.getColumnNumber());
            //e.printStackTrace();

        }

        System.out.println("\n=== Nashorn JS ScriptEngine ===");

        //  ScriptEngine scriptEngine = GraalJSScriptEngine.create(null,null);//Context.newBuilder("js") .allowAllAccess(true));
        scriptEngine=new NashornScriptEngineFactory().getScriptEngine();
        try {
            Object returnVal = scriptEngine.eval(code);
            Object outputVal = scriptEngine.get("output");
            //System.out.println("return: "+returnVal+", variable output: "+outputVal);
        } catch (ScriptException e) {
        	System.out.println(" Error:"+e.getMessage()+"\n Error At line: "+e.getLineNumber()+", column: "+e.getColumnNumber());
           // e.printStackTrace();

        }
    }
}

One more question:
Can we access multiple errors that may occur when engine evaluates a script?
or
It hust haults at first error and throws exception, so catching exception is only way to find error?

How can I get the line number in script at which error occured?

When the cause of the ScriptException is  PolyglotException (like in your case) then you can get the information about the error's location from its sourceSection property: ((PolyglotException) e.getCause()).getSourceSection(). On the other hand, I agree that we can propagate this information directly into ScriptException (when we wrap the PolyglotException by ScriptException).

As per your suggestion, below code gives error location.
		try{
			engine.eval(script, bindings);
		}catch(ScriptException e){
			if(e.getCause() instanceof PolyglotException) {
				PolyglotException p=(PolyglotException) e.getCause();
				SourceSection s=p.getSourceLocation();
				int line=s.getStartLine();
				int col=s.getStartColumn();
				System.println("line:"+line+" col:"+col+" Err:"+e.getMessage());
			}else {
				System.println(e.getMessage());
			}
		}

Yes, It would be better if this information can be accessed directly from ScriptException.
Thanks

FYI: I have modified the wrapping of PolyglotException by ScriptException such that the source-related information (like line and column number) is propagated from PolyglotException into ScriptException, see https://github.com/oracle/graaljs/commit/67f29221c3067a30894132e428fbdc33889576f1.


Allow reading bluetoothctl data from a file

This is needed to deploy & test our code w/o the need of an actual Bluetooth adapter.

v0.1.0 release solved it


chore(main): release 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-07-11)
⚠ BREAKING CHANGES

drop beforeEach and afterEach support for each assert (#539)
drop assertPromise helper (#538)
change listFiles helper from sync to async (#537)
drop deprecated methods and options (#536)
drop support for Node.js 6 and 7 (#534)

Features

support for strict assertions (#541) (68dae82)

Miscellaneous Chores

change listFiles helper from sync to async (#537) (945c4fb)
drop assertPromise helper (#538) (954ca1b)
drop beforeEach and afterEach support for each assert (#539) (a8455a7)
drop deprecated methods and options (#536) (ae6ff4b)
drop support for Node.js 6 and 7 (#534) (a11b1b4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wellwelwel/poku/releases/tag/v2.0.0 :sunflower:


Cursor not appearing

I am on Fedora silverblue Kiniote wayland. I observed the cursor not appearing in any of the examples.
https://user-images.githubusercontent.com/62714538/225092005-1253e40e-9be9-40b5-b67d-c96ef49bf600.mp4

Hi,
Thanks for the report. I wonder if it could be related to #273

I looked at the both previously opened issues about wayland, and they seem to get some errors/warnings. I get none. It's just the cursor that's missing.

I see. The thing is I haven't added any of the Wayland support myself and don't know much about it. I guess I will have to set up Wayland so I can test this out myself and see how it works.

Oh I have also never poked into Wayland myself, All the best !!!


rss response header tweak

feedgen update is recommended

Thanks! Really appreciate!

it's not necessary to apply this tweak if you update feedgenerator.py from https://code.djangoproject.com/svn/django/trunk/django/utils/feedgenerator.py :D


MM-10402: Adds view for unstarted and in-progress permissions phase 2…

… migrations.
Summary
Applies an "on hold" or "in-progress" view to the Team Override Schemes widget on the Permissions Schemes page.
Ticket Link
MM-10402
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[x] Has server changes (https://github.com/mattermost/mattermost-server/pull/8846)
[x] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json) updates

@grundleborg Fixed typo and changed from FormattedHTMLMessage to FormattedMessage.


Add tailscale Oauth support

This allows us to not have to worry about api key expiring after 90 days (solves  #18)
This modification works on my server but i did not test it with an api key for any regression
Readme might need to be improved a bit since i did not do a great job when modifying it
this could be improve by making tailnet an optional config since tailscale's doc mentions that when using oauth, tailnet isn't required (link)

When you use an access token to make an API call, you can optionally use the value - as the tailnet name—you do not need to specify the tailnet name.

Works well for me!

Thank you very much for your contribution :)

I tried your new image after the merge and it doesn’t seem to work, giving an error that parameters are missing (works fine in the fork…)

You should verify the parameters names, they changed after that pr was merged (they had a dash added)

also @marc1307 you missed the ts-clientid & ts-clientsecret in the Run using docker section (using secrets & env var)
https://github.com/marc1307/tailscale-cloudflare-dnssync?tab=readme-ov-file#run-using-docker-using-env-var

Ahh I didn’t see that change, will fix it and see how it goes :)


Update deploy-dacpac.md

@arunguru-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rothja
Can you review the proposed changes? Note that, unless they're procedural steps, lines 39 and 48 should start with bullet points instead of numbers.
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
@arunguru-msft Arun, for these type of larger changes, it is better if we do them in the private repository because we get previews, build validation, and you get the ability to sign off on your own changes. Please reach out to me internally if you have questions. I've signed off on your changes here and they should publish. Thank you!

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Wrong parameter Implementation for "ZW ESJ 300"

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?
NO, my problem is NOT within Z-Wave JS UI
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Wrong config file, parameter 0 is wrongly implemented.
What do you observe?
The permissible values are too small, the configuration of the running time for the device is done differently than described.
What did you expect to happen?
Allow to enter the required value for configuration but with current config file not possible.
To solve, either allow bigger values to be entered and let the user "calculate" the required number or implement four bitmask options which will set all four values. I used the device extensively on an old Vera Lite where setting the parameter in dec or hex works fine. When migrating one device to HA in zwaveJS, I could even see that parameter 0 is configured with value 30125 (decimal) which is "illegal" as per current device file.
I am not an expert, I try to explain how it should be. The parameter 0 is configuring the runtime up and down as a bitmask with 16bits (?) like<PHONE_NUMBER>110011.  In the device manual it is explained using hex (0x) values but I figured it is just a fancier way of saying bitmask. I tried to write a corrected version for the parameters but not sure if this is correct as I never coded anything.
It is attached.
0x0064.txt
Otherwise I am happy to explain how it should work and provide the original manual.
Thanks
Device information
Manufacturer: Reitz-Group.de
Model name: ZW ESJ 300
"manufacturer": "Reitz-Group.de",
"manufacturerId": "0x0064",
"label": "ZW ESJ 300",
"description": "Duwi ZW ESJ 300 Blind Control",
here
How are you using node-zwave-js?

[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[X] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version:
node-zwave-js branch:
zwave-js-ui branch:
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
No, it never worked anywhere
If yes, where did it work?
No response
Attach Driver Logfile
I don't think this would serve any purpose as it is a configuration file problem.

A link to the manual would be good

Manual here https://opensmarthouse.org/zwavedatabase/563/reference/DUW-blind.pdf

Can't help myself other than to upload as picture, wouldn't know how to type this in here.
This is the best I can do to explain it.

“Short mode” allows values from 0 (see below) to 127s. With “long mode” you can get values between 130s to 1390s (23m) in 10s steps.
There are two «special values»:

If runtime value set to 0, it will mean endless mode for that direction (Dauerbetrieb).
Setting 0xFFFF / 1 1111111 1 1111111 / 65535 is default, means 120s up & down, despite above rules.

If one would like to have 120s, one would rather programm it as 0x78 / 0b01111000 / 120 .
Alternatively 0xFF / 0b11111111 / 255 apparently also means 120s, yes.
If up is set to 0xFF and down to 0x00, would they go 120s up and endless down? Good question.
I don't know. I could test this with the ESJ300 I still have paired to the Vera Lite hub and see how they behave.
Can do in a couple of days, on the road currently.

Hello,
so I made a couple of tests over the Easter holidays. I can report back the following (and provide screenshots if required). The tests were done on a Micasaverde (now Ezlo) Vera Lite hub to set the desired parameter values. Setting of decimal values in the hub´s UI is easier and more reliable than hex values. Binary not possible. I always entered the decimal value as five digits, if required with leading zeros. Below I will also put the hex values in brackets but separate by an - for better readability. First pair is up movement, second pair is down movement. "The device" is the ZW ESJ 300. When I say "was accepted but re-wrote to" it means that I could set the parameter and the parameter was transmitted but the device did not save it as the given number but changed it to a different number, see below in the examples. This happened with the value xFF which was saved as x78 (120) as per manual FF is 120s.

setting 00000 (x00-00) was accepted by the device and resulted in endless mode for up and down
setting 00255 (x00-FF) was accepted by the device but re-wrote to 00120 (x00-78); up & down movement was endless though!
setting 65280 (xFF-00) was accepted by device but re-wrote to 30720 (x78-00); movement was 120s up and half second down [start and then immediately stop]
setting 65535 (xFF-FF) was accepted by device but re-wrote to 30840 (x78-78); movement was 120s up and 120s down (as expected)
setting 00025 (x00-19) was accepted but will make it run endlessly in both directions despite "down" set to x19 (25s)
setting 08448 (x21-00) was accepted but is 33s up and only half second down, with start/stop immediately
setting 65296 (xFF-10) was accepted but re-wrote to 30736 (x78-10) and is 120s up and 16s down (as expected)
setting 05119 (x13-FF) was accepted but re-wrote to 04984 (x13-78) and is 19s up and 120s down  (as expected)
setting 04112 (x10-10) was accepted, 16s up und down as expected
setting 32382 (x7e-7e) was accepted, 126s both directions as expected
setting 34698 (x87-89) was accepted and it showed the 200s up and 220s down as expected / described by manual

To summarize, any xFF given/sent to device was converted in the device to x78 and saved as such.
Any value starting with (x00-__) will make both directions go endlessly, despite whatever is set as second pair
xFF-00 (which will be saved as x78-00) and any regular value like x21-00 will make up movement as expected/set but down-movement will be very very short (start, then stop immediately).
If hope this will help you to create the required config file.
Two final notes:
You can also set the movement times manually on the device once it is connected to a network. If you want to add this information as well to the file, let me know. I can share the procedure for manually setting the movement times as well.
For the parameter 1, there is also some slight improvement in the description which could be done. But from the functionality it is already implemented correctly.
Thanks

Any more data I can provide to help progress this?

Push

@gruntpartystyle sorry for the delay... Lots of work to do, not much time with 2 toddlers :)
Anyways, I've opened a PR with an updated config file. If you like to test, use this file and follow these instructions (assuming you use zwave-js UI).
If everything works, I'll have this in the next release.

Great. Thank you. I will test the file once I am back at the property.

Can’t figure out how to load the device file. Might need a bit longer to figure that out.

Anything unclear about the documentation I linked?

Nothing you can help me with. I need to migrate from JS to JS UI first.

I see. I can also just merge this, so you'll get it with the next update.
Then I  can make a revision if necessary.

Works for me.

Just to report it works fine, change finally made it into HA zwaveJS. Thanks.


Set up access token utilization by yt client

Pull Request
Description

Updated playlist service to use access token extracted on controller level during requessts

@GetMapping("/list")
    public ResponseEntity<List<Playlist>> getPlayLists(
        @RegisteredOAuth2AuthorizedClient OAuth2AuthorizedClient authentication) throws IOException {

        String accessToken = authentication.getAccessToken().getTokenValue();
        return ResponseEntity.ok(youTubeService.getMyPlayLists(accessToken));
    }

public List<VideoSnippet> getPlayListVideos(String accessToken, String playListId) throws IOException {
        var request = youTubeClient.playlistItems().list(List.of("snippet"));
        var response = request
            .setAccessToken(accessToken)
            .setPlaylistId(playListId)
            .execute();

    // bla - bla - bla

Updated security config, it now includes success and failure urls

return http
            .authorizeHttpRequests(auth -> auth.anyRequest().authenticated())
            .oauth2Login(config -> {
                config.defaultSuccessUrl("/auth/success", true);
                config.failureUrl("/auth/error");
            })
            .build();

Added html templates for some aesthetics :)))))

Updated project's dependencies

removed docker as it's currently not necessary (compose.yaml is gone too)

I removed some files or code that I believed to be extra for now, please let me know if it's still needed in your opinion, we'll bring it back

Related Issue
#2
Checklist
Please review and check all applicable items:

[x] I have tested my changes thoroughly
[x] My code follows the project's coding standards and guidelines
[x] I have reviewed my changes to ensure there are no unnecessary changes
[x] My pull request has a descriptive title and includes a detailed description of the changes

Additional Notes
I just now noticed that I may have affected #1 , I'm very sorry
please let me know if something should be reverted or fixed
Screenshots (if applicable)

Awesome job!
Just a suggestion, maybe we could redirect users to the login URL unless they are authorized?

thank you! ❤


The readthedocs rendering of the spec should advertise the version

https://bids-specification.readthedocs.io/en/stable/ does not advertise its version on the main landing page, we should probably change this.
Perhaps display it somewhere in this corner?

@franklin-feingold since #137 has been merged, this issue can be closed - correct? If I understand it correctly, we will have the version advertised when making a new release?

yup! this issue can be closed and yes as we go through the Release Protocol this should have the version at the top for the stable and latest versions of the specification

great, thanks again!

it's working as expected and looking good :-)


[PKG] SQLite

Package name
SQLite
Package URL
https://www.sqlite.org/index.html
Package source location
https://www.sqlite.org/download.html

@Zingam In the meantime there is feature that was created microsoft/vcpkg#10271. You can build the archive yourself. Its easy
apply patch to enable android sqlite
diff --git a/ports/sqlite3/CMakeLists.txt b/ports/sqlite3/CMakeLists.txt
index 64ff00262..3e5ac4daf 100644
--- a/ports/sqlite3/CMakeLists.txt
+++ b/ports/sqlite3/CMakeLists.txt
@@ -3,7 +3,7 @@ project(sqlite3 C)

 include_directories(.)
 if(BUILD_SHARED_LIBS)
-    if(CMAKE_SYSTEM_NAME MATCHES "Linux" OR CMAKE_SYSTEM_NAME MATCHES "Darwin")
+    if(CMAKE_SYSTEM_NAME MATCHES "Linux" OR CMAKE_SYSTEM_NAME MATCHES "Darwin" OR CMAKE_SYSTEM_NAME MATCHES "Android")
         set(API "-DSQLITE_API=__attribute__((visibility(\"default\")))")
     elseif(CMAKE_SYSTEM_NAME MATCHES "Windows")
         set(API "-DSQLITE_API=__declspec(dllexport)")
diff --git a/ports/sqlite3/portfile.cmake b/ports/sqlite3/portfile.cmake
index e46036667..bc930311b 100644
--- a/ports/sqlite3/portfile.cmake
+++ b/ports/sqlite3/portfile.cmake
@@ -25,6 +25,7 @@ vcpkg_configure_cmake(
     SOURCE_PATH ${SOURCE_PATH}
     PREFER_NINJA
     OPTIONS ${FEATURE_OPTIONS}
+    -DCMAKE_SYSTEM_NAME=${VCPKG_CMAKE_SYSTEM_NAME}
     OPTIONS_DEBUG
         -DSQLITE3_SKIP_TOOLS=ON
 )

export ANDROID_NDK_HOME="/Library/Android/sdk/ndk/<version>"
./vcpkg install sqlite3  --triplet x86-android
./vcpkg install sqlite3  --triplet x64-android
./vcpkg install sqlite3  --triplet arm-android
./vcpkg install sqlite3  --triplet arm64-android
./vcpkg export sqlite3  --triplet x86-android --prefab

The resulting pom
    <groupId>com.vcpkg.ndk.support</groupId>
    <artifactId>sqlite3</artifactId>
    <version>3.30.1-2</version>
    <packaging>aar</packaging>

or Gradle
  implementation "com.vcpkg.ndk.support:sqlite3:3.30.1-2"

The vcpkg export as aar had broken for at least half a year.
This package had been publish to my maven repository hosted on github https://github.com/leleliu008/ndk-pkg-prefab-aar-maven-repo
You’re welcome to give it a try.


rewrite fetchos function to use pipeline

I have been trying to make the jetson-tx2 device type work consistantly with the os test suite.
The final issue I encountered was the HUP suite https://github.com/balena-os/meta-balena/tree/master/tests/suites/hup failing every single time. The problem was that the jetson would never flash its internal storage, when we try to provision it with the latest balena OS prod release.

the os and cloud suites worked fine when used with the same OS image version
after some examination, the only difference I could see, was that in the os and cloud suites, we provided this image (I downloaded it via the browser), but in the hup suite, we use the leviathan core fetchOS() function to get this image.
This threw the fetchOS() function into a suspicious light. So what I did, was tried the hup suite, instead using the CLI to download the image - and it worked all 3 times I tried it!
I then went into fetchOS() and experimented with rewriting parts of it. I removed the retry as it was never being used anyway, as far as I could see. I also used the pipeline utility to ensure correct handling of the read and write streams - as I suspect that something strange was happening with the previous imlpementation
I'm not exactly sure why this worked - but it did seem to fix my problem. As to why this issue never appeared for other device types, I can only theorise that its because the jetson image is much larger (2.4G compared to the ~950M or the pi's, and 1.6G for NUC and rockpi)

Change-type: patch
Signed-off-by: Ryan Cooke<EMAIL_ADDRESS>

@balena-ci rebase

@balena-ci rebase

@balena-ci rebase

This is not available now that we disabled VersionBot.
Instead select the Update Branch dropdown and choose Update with Rebase.

@vipulgupta2048 I'm going to close this, as I don't think that it was actually the problem - further investigation led me to this fix instead: https://github.com/balena-os/meta-balena/pull/3022


OnClose event trigger

Hello,
I'm trying to implement auto reconnect with your angular 4 example but I can't find a way to plug the on close event to this block:
this.messages = websocketConnect(
'ws://<IP_ADDRESS>:4201/ws',
this.inputStream = new QueueingSubject()
).messages.share()
Have you an example of implementation for autoreconnect or close detection?
Thank you for your time

Read the README before opening an issue? And don't use issues for support?


Give the ability to remove scan jobs from view in bulk via the GUI

User Story
As a user of Automate, I will use this work when I want to remove any scan jobs in bulk.
Definition of Done
A user can remove multiple scan jobs by selecting which scan jobs, and initiating a bulk remove.
Demo Script / Repro Steps

goto /compliance/scan-jobs/jobs
User can select multiple scan jobs via checkboxes
user can then bulk remove by a button.
system prompts confirmation dialogue
system removes scan jobs.
System displays success/fail message.

Mock Ups

Additional changes
1.) converting the buttons on each row to the standard ellipses button. This will be dynamic based on if the scan job failed or was successful. showing the error or the report button.
2.) Updates to the table header names
3.) removing the interstitial row with the 'Add' button and moving the button above the table header as defined in the mock up.

this work has been deprioritized, so closing.


feat: add accountId

Motivation
On-chain identifier accounts is useful for being able to identify and differentiate between different account implementations easily.
Solution
Implement accountId, inspired by ERC-7579. Note I used a different delimiter /. I think this makes for slightly easier parsing than what the 7579 spec prescribes (they use a . delimiter), since periods always exist in the version number token. I could be convinced to just conform to 7579's spec for alignment.

It seems like in both “.” and “/“ cases, we would have the requirement that the first two occurrences are the actual separators.  A slash does seem less likely to be used in tokens than a dot, but I wonder if it is worth it to have this super slight divergence from 7579 🤔

Aligning with 7579 sounds good to me. @kopy-kat @leekt216 heads up!


Server crash when using terrasteel axe from botania

Server crashes every time my friend uses his terrasteel axe to cut trees down when he has this mod enabled. If he deactivates it with the hotkey there are no problems. Only noticed this problem since we updated the MC Eternal modpack from 1.4.4 to 1.5. The mods of the MCEternal told me to post here as well.

Thanks for reporting. I’ll look into this soon

Sorry for the wait. A quick fix is to open the config/treechop-common.toml file and add "botania:terra_axe" to the list under the [compatibility.general.blacklist] section, like this:
		[compatibility.general.blacklist]
			#Whether the listed items should be blacklisted or whitelisted
			#Allowed Values: BLACKLIST, WHITELIST
			blacklistOrWhitelist = "BLACKLIST"
			#List of item registry names (mod:item), tags (#mod:tag), and namespaces (@mod) for items that should not chop when used to break a log
			#- Items in this list that have special support for TreeChop will not be blacklisted; see https://github.com/hammertater/treechop/blob/main/docs/compatibility.md#blacklist
			items = ["botania:terra_axe", "mekanism:atomic_disassembler", "@lumberjack", "practicaltools:iron_greataxe", "practicaltools:golden_greataxe", "practicaltools:diamond_greataxe", "practicaltools:netherite_greataxe"]

I'll update the mod soon and add the terra axe to the blacklist by default

The blacklist is updated as of 0.14.6. If you do update the mod, be sure to delete the treechop-common.toml file so the fix gets applied


Failed to extract any player response;

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
USA
Provide a description that is worded well enough to be understood
Using 9COnvert (https://9convert.com/en423), Trying to download this video (https://www.youtube.com/watch?v=LAGA9VYfnsM). Received the following error: "Failed to extract any player response; please report this issue on https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using yt-dlp -U"
Was working earlier, but has gave this error with a few different videos in  a row
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
Failed to extract any player response; please report this issue on https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using yt-dlp -U

This bug tracker is for yt-dlp, a command line application. If you have an issue running yt-dlp yourself you can open an issue with a verbose log.
If some other site or program uses yt-dlp the issue should be reported to them.

I have to run for this command LC_ALL=en_US.UTF-8 yt-dlp -vU --no-mtime --extractor-args youtube:player_client=android --proxy https://<IP_ADDRESS>:8800 --concurrent-fragments 2 --no-cache-dir -f “bestvideo[ext=mp4][vcodec*=avc]+bestaudio[ext=m4a]/best[ext=mp4]/best” -o “./upload/Windy_Dessert_video_1710504361838.mp4" https://www.youtube.com/watch?v=6nxWbOUAPSQ >> “./upload/Windy_Dessert_video_1710504361838.mp4.console”
I get like this error
WARNING: [youtube] Unable to download webpage: ('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)')))
WARNING: [youtube] ('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)'))). Retrying (1/3)...
WARNING: [youtube] ('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)'))). Retrying (2/3)...
WARNING: [youtube] ('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)'))). Retrying (3/3)...
WARNING: [youtube] Unable to download API page: ('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)'))) (caused by ProxyError("('Unable to connect to proxy. Your proxy appears to only use HTTP and not HTTPS, try changing your proxy URL to be HTTP. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#https-proxy-error-http-proxy', SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1131)')))")); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [youtube] 6nxWbOUAPSQ: Failed to extract any player response; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

@nilesh0747 are you sure that's an https proxy? try --proxy http://<IP_ADDRESS>:8800
btw, why are you replying to this issue? open a new one with a complete verbose log if you need support


Loosen restrictions on hasUsefulInfo

Background
The babel-plugin-c-3po loader fails when called with an aspect ratio string like:
t`16:9`

This produces a fatal build time error:

BabelLoaderError: SyntaxError: Can not translate '16:9'
Error: Can not translate '16:9'

In our application we want to be able to translate aspect ratio strings. For example, in German we may prefer to translate this to "16 zu 9".
In this PR

Update the regexp to allow : as a useful character.

Thanks, for your PR! Will publish a new version soon.

your fix is already available in v0.8.1.
FYI. Currently, we are going to rename this project from c-3po to ttag (https://github.com/ttag-org/ttag/issues/105). We are going to finish the transition in a few days. No API will be changed, just name of the library and the plugin.


This is stolen.

In lines 6, 7 and 8 of the license and I quote: "

You may not reproduce or distribute any code inside this file without the licenser's permission.
You may not copy, modify, steal, skid, or recreate any of the code inside this file.
You may not under any circumstance republish any code from this file as your own."

This means that you may not copy or modify the code, as such you did.

Hey PrimateTerrorVr, can I see the "paid op" gui you had in your deleted repo: https://github.com/PrimateTerrorVr/blooket-hacks

Hey PrimateTerrorVr, can I see the "paid op" gui you had in your deleted repo: https://github.com/PrimateTerrorVr/blooket-hacks

No, due to legal reason I had to take it down.

@DannyDan0167 btw ur gimkit cheats are cool.

@DannyDan0167 btw ur gimkit cheats are cool.

There not really mine, they belong to lazysquid but thank you


[stable/mariadb] Add JSON Schema

Signed-off-by: juan131<EMAIL_ADDRESS>Is this a new chart
No
Description of the change
This PR includes a basic JSON schema for the MariaDB chart. It doesn't cover all the possible values of the chart but just the ones that, in my opinion, are more likely to be modified.
This is valid for Helm 3 to validate some values and, at the same time, for Kubeapps to render a simple form so the chart containing it is easier to configure and deploy. See https://github.com/kubeapps/kubeapps/blob/master/docs/developer/basic-form-support.md for more information.
This is an example of the file rendered by Kubeapps:

Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @carrodher @andresmgot

/ok-to-test
/lgtm


Track specific exercise metadata location.

In https://github.com/exercism/xcoq/issues/7#issuecomment-300024536 @kytrinyx suggested we start a discussion about the location of the metadata files.
Background
A track can implement problems that are not in x-common by including metadata.yml and description.md files in a problem_slug directory under /exercises/
Currently, Rust is the only track that uses these.
Coq will soon be adding some.
tracks/rust/exercises/luhn-from
tracks/rust/exercises/luhn-trait

Proposal
Put these files in problem_slug/.meta instead of in the root exercise directory.
Rationale
In .meta files won't need to be explicitly ignored when serving files to students.
They are metadata so .meta is a more appropriate location.
Caveats
Will require (easy) trackler changes.
cc: @exercism/rust , @exercism/coq

Wow, I never really checked and therefore only just noticed that indeed, http://x.exercism.io/v2/exercises/rust/luhn-from and http://x.exercism.io/v2/exercises/rust/luhn-trait serve the description.md and metadata.yml to the students. probably shouldn't do that.
any proposal that achieves that will do

You should add them to your ignore_pattern in config.json until trackler gets updated.

@petertseng would you be able to copy the 2*2 metadata files into .meta?
This will enable trackler changes without having to worry about backwards compatibility.

grumble grumble but they're not my exercises, go do it yourself or ask Ian
Just kidding Ian's currently limited, easy enough to do anyway


[Feature] support create/replace branch based on iceberg

Why I'm doing:
What I'm doing:
support creating/replacing branch syntax. Currently only implementing iceberg table
Fixes #issue
https://github.com/StarRocks/starrocks/issues/47846
syntax:
createReplaceBranchClause
    : (CREATE OR)? REPLACE BRANCH identifier branchOptions
    | CREATE BRANCH (IF NOT EXISTS)? identifier branchOptions
    ;

the simple demo based on latest snapshot with default properties
ALTER TABLE simple_demo CREATE BRANCH

use all branch options demo:
ALTER TABLE test_table CREATE BRANCH test_branch
AS OF VERSION<PHONE_NUMBER>211265306   // based on which snapshot id
RETAIN 7 DAYS   // branch retention times
WITH SNAPSHOT RETENTION 5   // min retention snapshots
SNAPSHOTS 2 DAYS;  // snapshot max age on the branch

the refs contents in the iceberg metadata file of json.
"refs" : {
    "simple_demo" : {
      "snapshot-id" :<PHONE_NUMBER>390156990,
      "type" : "branch"
    },
    "test_branch" : {
      "snapshot-id" :<PHONE_NUMBER>211265306,
      "type" : "branch",
      "min-snapshots-to-keep" : 5,
      "max-snapshot-age-ms" : 172800000,
      "max-ref-age-ms" : 604800000
    },
    "main" : {
      "snapshot-id" :<PHONE_NUMBER>390156990,
      "type" : "branch"
    }
  }

What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

I think there need a CreateBranchAnayzer to check the statement is legal and the value is valid, such as snapshot id, min-snapshots-to-keep, branch retention times value can used to execute

I think there need a CreateBranchAnayzer to check the statement is legal and the value is valid, such as snapshot id, min-snapshots-to-keep, branch retention times value can used to execute

This is indeed my initial idea. However, this part has no illegal values and is relatively independent, put it here is easier to read.


Fixed your stuff

Have fun with realising that you missed out one line! also cleaned up some stuff and added a config file because i like using them.

Yeah thanks for all that, everything was made in like 30 minutes so it was rushed a lot.


[Form] Validation not working for number inputs

I'm using semantic ui form validation in one of my projects in ember. The validation does not work for number type input.
Code Snippet:
level: {
          identifier : 'level',
          optional   : true,
          rules      : [
            {
              type   : 'integer',
              prompt : this.l10n.t('Please enter a number')
            }
          ]
        }

Template
{{input type='number' value=sponsor.level name='level' min=0}}

Note: Validation works if I remove optional: true
Please guide. Thanks

Hi @geekyd, I’ve just tested it, it works. Can you please elaborate, what did you mean when you said that it didn’t work, what happened exactly? Any console errors?

There are no console errors.
I'm able to enter e as input and its getting validated as an integer. It should prompt an error if e is present in the input. It seems to work if we remove optional.

@geekyd Alright, unfortunately, we can’t investigate it further if we can’t reproduce it in an isolated test case without Ember. I might’ve missed something when tested it, so please fork this JSFiddle and see if you can reproduce it there.

here's the fiddle guys http://jsfiddle.net/cewxfom9/ :)

@Banandrew The Validation works for

  Level
  {{input type='number' value=sponsor.level name='level'}}

However the validation does not work for the nested fields.

  Level
  {{input type='number' value=sponsor.level name='level'}}

Is it a known issue?

@geekyd Seems to be working fine: https://jsfiddle.net/Banandrew/3fwy7na6/

@Banandrew Please check https://jsfiddle.net/geekyd/7kwehhk7
Try non integer inputs eg. e
Thanks

@geekyd It’s the same JSFiddle that I posted.

@Banandrew Sorry about that, please check https://jsfiddle.net/geekyd/a2ktp4ap/

@geekyd Thank you, confirming the issue. It seems that you either have to set optional to false, or use the text type instead of the number.


Feature/jperket/externals+doc update

Update doc for 17.8.0 release, remove Develop.cfg (fixing https://github.com/GEOS-ESM/GEOSldas/issues/73)

I tried something new today... I edited README.GEOSldas_history directly on your feature branch in an attempt to further clarify where the new Git tag v17.8.0 fits into the history of CVS tags.
I approve the pull request assuming that my edits (i) make sense and (ii) actually make it into the merge.
Let me know if anything looks suspicious.

Perfect. I actually had a similar description I cut out, because it was as sussinct as
cceba97.
We'll wait for baseline to change, then merge Monday

Merging this now. @mathomp4 is ready to switch Nightly test to Externals.cfg instead of Develop.cfg.  night tests are also being named at the same time. We can do maually 0-diff checks to make sure there's no snafus

making https://github.com/GEOS-ESM/GEOSldas/issues/21 resolved


Ability to support params and params_from_vars together

Right now it appears the applier clears out the params_from_vars and just takes the param file
fatal: [projects-and-policies]: FAILED! => {"changed": true, "cmd": "oc process  --local   -f  https://raw.githubusercontent.com/redhat-cop/cluster-lifecycle/v3.9.0/files/projectrequest/template.yml  --param-file=/home/pcarney/github/rht-labs/labs-ci-cd/params/projectrequests/ci-cd | oc create  -f - ", "delta": "0:00:00.196392", "end": "2018-09-07 10:42:57.140014", "failed_when_result": true, "msg": "non-zero return code", "rc": 1, "start": "2018-09-07 10:42:56.943622", "stderr": "The Template \"${NAMESPACE}\" is invalid: template.parameters[0]: Required value: template.parameters[0]: parameter NAMESPACE is required and must be specified\nerror: no objects passed to create", "stderr_lines": ["The Template \"${NAMESPACE}\" is invalid: template.parameters[0]: Required value: template.parameters[0]: parameter NAMESPACE is required and must be specified", "error: no objects passed to create"], "stdout": "", "stdout_lines": []}
        to retry, use: --limit @/home/pcarney/github/rht-labs/labs-ci-cd/apply.retry

PLAY RECAP *****************************************************************************************************************************************************************************************************
projects-and-policies      : ok=11   changed=0    unreachable=0    failed=1

Testing with oc shows it takes -p first it appears:
[pcarney@localhost labs-ci-cd]$ oc process  --local   -f  https://raw.githubusercontent.com/redhat-cop/cluster-lifecycle/v3.9.0/files/projectrequest/template.yml -p=NAMESPACE=patrick-test-ci-cd -p=NAMESPACE_DISPLAY_NAME=patrick-test-ci-cd --param-file=/home/pcarney/github/rht-labs/labs-ci-cd/params/projectrequests/ci-cd | oc create  -f -
warning: Template parameter "NAMESPACE" already defined, ignoring value from file "/home/pcarney/github/rht-labs/labs-ci-cd/params/projectrequests/ci-cd"warning: Template parameter "NAMESPACE_DISPLAY_NAME" already defined, ignoring value from file "/home/pcarney/github/rht-labs/labs-ci-cd/params/projectrequests/ci-cd"project.project.openshift.io "patrick-test-ci-cd" created

@pcarney8 yes, and even better... it looks like the order of the arguments makes no difference:
$ oc process -f tests/files/secrets/template1.yml --local --param-file tests/files/secrets/secret1.params -p USERNAME=me
warning: Template parameter "USERNAME" already defined, ignoring value from file "tests/files/secrets/secret1.params"{
    "kind": "List",
    "apiVersion": "v1",
    "metadata": {},
    "items": [
        {
            "apiVersion": "v1",
            "data": {
                "password": "testPassword",
                "username": "me"
            },
            "kind": "Secret",
            "metadata": {
                "creationTimestamp": null,
                "labels": {
                    "template": "secret-template"
                },
                "name": "test-secret"
            },
            "type": "Opaque"
        }
    ]
}

$ oc process -f tests/files/secrets/template1.yml --local -p USERNAME=me --param-file tests/files/secrets/secret1.params
warning: Template parameter "USERNAME" already defined, ignoring value from file "tests/files/secrets/secret1.params"{
    "kind": "List",
    "apiVersion": "v1",
    "metadata": {},
    "items": [
        {
            "apiVersion": "v1",
            "data": {
                "password": "testPassword",
                "username": "me"
            },
            "kind": "Secret",
            "metadata": {
                "creationTimestamp": null,
                "labels": {
                    "template": "secret-template"
                },
                "name": "test-secret"
            },
            "type": "Opaque"
        }
    ]
}

Looks like this would not require any sort of pre-processing to make it work.

@pcarney8 I just tested myself, and it looks to have worked. I'm opening a PR with the test case I created.

One thing that cannot do today, is set up an inventory such that we can optionally pass arguments to our playbook run... i.e. I would like to be able to run either of the following without making an inventory change:
ansible-playbook playbooks/openshift-cluster-seed.yml -i tests/inventories/params-from-both/

ansible-playbook playbooks/openshift-cluster-seed.yml -i tests/inventories/params-from-both/ -e secret_username=ericTestUser

@oybed @pcarney8 any thoughts on the above?


Commerce Manager access (permissions to functions) on Identity users

Created a user which is part of the Administrators and WebAdmins groups but in Commerce manager it only states Access denied. The admin user is the only one which is working in CM. How to manage Commerce manager access to new users with Identity provider?

Is there any guidance to find in the responses to Issue 7 and Issue 8?

No not really. I have this setup and can create users based on the Identity provider. But when creating a user that has the same groups setup as the admin user (WebAdmins and Administrators) I still get the access denied in Commerce manager. The permissions to functions is already setup allowing WebAdmins and Administrators access to most of the functions but it seems to some level of access setting that I cannot find or set.

This will be fixed by COM-1553 (see http://world.episerver.com/support/Bug-list/bug/COM-1553 )

We now have a major update for Quicksilver (supporting new promotion system - currently on develop branch, shortly on master) but the fix for COM-1553 is unfortunately not in there. We hope to be able to fix it in the near future.

Since COM-1553 has been closed I'm closing this issue now.


Fixes the services definitions for commands

With Symfony 4,  commands should be registered, and non-injected services, set to public=true explicitly.

Hi @floriansemm, this fixes is ready for review. Thank you!


Fix: Remove plugin configuration for non-existent plugin

This PR

[x] removes configuration that registers a non-existent controller plugin

Thanks!

Thank you, @Ocramius!


Provider produced an unexpected new value: inconsistent values for sensitive attribute.

Your environment

Terraform Provider Version: v2.1.0

Connect Server Version: n/a

CLI Version: 2.29.0

OS: linux

Terraform Version: v1.5.7
What happened?
Provider is not able to configure a field with an empty value.
What did you expect to happen?
Provider configures a field with an empty value.
Steps to reproduce
main.tf:
provider "onepassword" {}

data "onepassword_vault" "deploy" {
  name = "deploy"
}

resource "onepassword_item" "environment_variables" {
  vault = data.onepassword_vault.deploy.uuid

  title      = "var.env_name"
  category   = "secure_note"
  note_value = "all the values for the var.env_name environment"

  section {
    label = "configuration"
    field {
      label = "TF_VAR_target_account"
      type  = "STRING"
      value = ""
    }
  }
}

vscode ➜ /workspace (dev) $ terraform apply -auto-approve
data.onepassword_vault.deploy: Reading...
data.onepassword_vault.deploy: Read complete after 2s [id=vaults/ajchpspn4wzdjdh7udhfmmr43i]
data.onepassword_item.test: Reading...
onepassword_item.environment_variables: Refreshing state... [id=vaults/ajchpspn4wzdjdh7udhfmmr43i/items/c5r33544xjzseeiz5x6czbzzni]
data.onepassword_item.test: Read complete after 2s [id=vaults/ajchpspn4wzdjdh7udhfmmr43i/items/exvjimxwtcienaiufieuvnhp5y]

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # onepassword_item.environment_variables is tainted, so must be replaced
-/+ resource "onepassword_item" "environment_variables" {
      ~ id         = "vaults/ajchpspn4wzdjdh7udhfmmr43i/items/c5r33544xjzseeiz5x6czbzzni" -> (known after apply)
      + password   = (sensitive value)
      ~ uuid       = "c5r33544xjzseeiz5x6czbzzni" -> (known after apply)
        # (4 unchanged attributes hidden)

      ~ section {
          ~ id    = "c9769e35-9298-a59e-1fac-02fcddc76b45" -> (known after apply)
            # (1 unchanged attribute hidden)

          ~ field {
              ~ id    = "7f6osx64xqvo27rysbals5zs7e" -> (known after apply)
                # (2 unchanged attributes hidden)
            }
        }
    }

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

onepassword_item.environment_variables: Destroying... [id=vaults/ajchpspn4wzdjdh7udhfmmr43i/items/c5r33544xjzseeiz5x6czbzzni]
onepassword_item.environment_variables: Destruction complete after 2s
onepassword_item.environment_variables: Creating...
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to onepassword_item.environment_variables, provider "provider[\"registry.terraform.io/1password/onepassword\"]" produced an unexpected new value: .section[0].field[0].value: inconsistent values for sensitive attribute.
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.
╵
Releasing state lock. This may take a few moments...
vscode ➜ /workspace (dev) $

I do have the same issue. Setting a username to a non-empty string did not fix the issue on my end.


revertedWith succeeds despite only partial match

Empty message string would pass too

Empty message string would pass too

Hello, maintainers, this Error still persist in 3.4.0
cc @marekkirejczyk
False-negative scenario if I remove the beginning or ending part of reveredWith parameter!
True-negative case (original string):

False-negative scenario (it pass, but should fail)

Please see https://github.com/TrueFiEng/Waffle/issues/558#issuecomment-1198266089


[dev-modernize] Cannot compile with option -Werror

Compiling SelfTest with option -Werror stops with:
In file included from .../Catch/include/internal/catch_debugger.cpp:115:0:
.../Catch/include/internal/catch_windows_h_proxy.h:12:0: warning: "NOMINMAX" redefined
 #  define NOMINMAX

The following options used with -Werror also end compilation:
-pedantic
-pedantic-errors
-Waggregate-return
-Wconversion
-Wctor-dtor-privacy
-Weffc++
-Wmissing-declarations
-Wnon-virtual-dtor
-Wredundant-decls
-Wshadow
-Wsign-conversion
-Wswitch-default
-Wswitch-enum
-Wuseless-cast
-Wzero-as-null-pointer-constant

I want to say that -Werror together with full warnings is unsupported, but at least -Wall -Wextra should be ok.

What surprised me, is that -Werror on its own breaks the compilation of SelfTest with gcc 5.2.0 due to "NOMINMAX" redefined.
In file included from .../Catch/include/internal/catch_debugger.cpp:115:0:
.../Catch/include/internal/catch_windows_h_proxy.h:12:0: error: "NOMINMAX" redefined [-Werror]
 #  define NOMINMAX
 ^
In file included from c:\mingw\include\c++\5.2.0\x86_64-w64-mingw32\bits\c++config.h:482:0,
                 from c:\mingw\include\c++\5.2.0\string:38,
                 from .../Catch/include/internal/catch_debugger.h:14,
                 from .../Catch/include/internal/catch_debugger.cpp:10:
c:\mingw\include\c++\5.2.0\x86_64-w64-mingw32\bits\os_defines.h:45:0: note: this is the location of the previous definition
 #define NOMINMAX 1
 ^
cc1plus.exe: all warnings being treated as errors
make.exe[2]: *** [CMakeFiles/SelfTest.dir/include/internal/catch_debugger.cpp.obj] Error 1
make.exe[1]: *** [CMakeFiles/SelfTest.dir/all] Error 2
make.exe: *** [all] Error 2

Catch\include\internal\catch_platform.h contains:
#elif defined(WIN32) || defined(__WIN32__) || defined(_WIN32) || defined(_MSC_VER)
#  define CATCH_PLATFORM_WINDOWS
#  if !defined(NOMINMAX) && !defined(CATCH_CONFIG_NO_NOMINMAX)
#    define CATCH_DEFINES_NOMINMAX
#  endif
#  if !defined(WIN32_LEAN_AND_MEAN) && !defined(CATCH_CONFIG_NO_WIN32_LEAN_AND_MEAN)
#    define CATCH_DEFINES_WIN32_LEAN_AND_MEAN
#  endif
#endif

Apparently CATCH_DEFINES_NOMINMAX gets defined. I was not able to unravel the sequence of events that makes this happen.

Quick guess: catch_platform.h gets processed before system header that defines NOMINMAX, and then our windows.h proxy defines NOMINMAX as well, because catch_platform.h says so.

ok

I am fairly sure that the redefinition problem is fixed now. Specific warnings levels and Werror should go in a separate issue.

Thanks @horenmar


noUncheckedIndexedAccess does not work correctly with Array

Bug Report

🔎 Search Terms
noUncheckedIndexedAccess with arrays

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about noUncheckedIndexedAccess

⏯ Playground Link

Playground link with relevant code
💻 Code

// @strict: true
// @noUncheckedIndexedAccess: true

const arr: string[] = [];

const test = () => {
    if (arr.length > 1) {
        // As arr.length > 1, we expect that length >= 2. That means arr[0] should be a `string`
        const str: string = arr[arr.length - 2];
        console.log(str);
    }

    // Same here, we check arr.length == 1, so arr[0] should be a `string` and cannot be `undefined` according to declaration `const arr: string[] = [];`
    if (arr.length === 1 && arr[0].match(/test/)) {
        console.log("First index matches 'test'");
    }
}

🙁 Actual behavior
Get following errors:
error TS2322: Type 'string | undefined' is not assignable to type 'string'.
  Type 'undefined' is not assignable to type 'string'.
    const str: string = this.arr[this.arr.length - 2];

error TS2532: Object is possibly 'undefined'.
    if (arr.length === 1 && arr[0].match(/test/))

🙂 Expected behavior
The code should not generate any error since the array is type string[] and not (string|undefined)[]

This was one of the outlined caveats of the flag; see #13778


Compile the version 7.0.0

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Just noticed this is not a stable release
Closing..


How about a section on game engines/frameworks

As a newcomer into gamedev I found very helpful the list of game engines by language compiled by the github/itch.io game off (e.g.  Help - I've never created a game before!)

Except in rare cases, I am uncomfortable sending new developers off with engines other than Unity and Unreal (and potentially Godot), as these are the engines that recruiters look for, and the only third-party engines that studios typically use. Switching game engines is a major interruption in the learning process, so it is best to start with tools that stay useful in the long term. Additionally, this list avoids lists of tools where possible, as described in the About section. Glad that this helped you, but will not include.
However, this does highlight another real issue, which is that this list does not clearly explain what a game engine is or why it is useful, and does not explain that Unity and Unreal are game engines. Will include a resource such as this and revise item descriptions to explain this clearly.

Added How Game Engines Work, revised engine course descriptions appropriately.


Serving locally with uvicorn fails

$ python3 -m uvicorn main:app --reload
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [19039] using statreload
ERROR:    Error loading ASGI app. Could not import module "main".
CINFO:     Stopping reloader process [19039]
$ python3 -m uvicorn --version
Running uvicorn 0.14.0 with CPython 3.8.10 on Linux

Closing this issue as it's solved already


repo for ubuntu 17.10

Hi,
I have a fresh install of Ubuntu 17.10 and when adding the realsense repo, I've got a 403 on apt-get update...
Here are the tasks I've followed, according to doc :
echo 'deb http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo xenial main' | sudo tee /etc/apt/sources.list.d/realsense-public.list
sudo apt-key adv --keyserver keys.gnupg.net --recv-key 7DCC6F0F
sudo apt-get update

And I've got a 403 forbidden on http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo
Thanks in advance for your help !

Hello! Same exact issue here. I get a 403 when using https as well.

Hi, I'm having the same issue, I'm getting the error:
E: Failed to fetch http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/InRelease  403  Forbidden
https://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/InRelease seems to return a 403 error, while the following all work:

https://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/Release
https://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/main/binary-amd64/Packages.gz
https://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/main/binary-amd64/Packages
https://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/main/binary-amd64/Release

Normally apt looks for an InRelease file, and if it gets a HTTP 404 file not found error, it should move to looking for a Release file. So @dorodnic either you need to change your server so it returns a HTTP 404 when looking for the InRelease file, instead of a HTTP 403 (which may be difficult depending on who controls your server), or create an InRelease file.
For all of you guys that still want to install this, try entering the following commands into your terminal, it manually downloads the files that apt requires, basically manually doing sudo apt update so you can do sudo apt-get install librealsense2-utils.
cd /var/lib/apt/lists
sudo wget http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/main/binary-amd64/Packages -O realsense-hw-public.s3.amazonaws.com_Debian_apt-repo_dists_xenial_main_binary-amd64_Packages
sudo wget http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/xenial/Release -O realsense-hw-public.s3.amazonaws.com_Debian_apt-repo_dists_xenial_Release

Hi,
Wanted to update that we are looking into this problem.
@aloisklink thank you very much for your comment, I'll try to modify the permissions as you suggested.
For now, the alternative is building the library from source (instructions).

same issue here, I had to follow @aloisklink instructions

Is this because with s3 permissions when you request a file that doesn't exists and you don't have read-access to the bucket, you get a 403 rather than the expected 404 for the file actually not being there?
If this is the case, I expect one could solve it be opening up the bucket permissions to allow directory reading.
Can one create a file with some permission set on s3 that would force a 404? Perhaps if InRelease is a directory rather than a file? I'm guessing not.

We don't plan to add Ubuntu 17 PPA in the upcoming Sprints, but will keep it in the backlog and revisit once more people start using it.

@The-Funk, in Ubuntu 16.04, when apt gets any HTTP error code for the InRelease file, apt would then look for a Release file. However, apt was updated (see https://github.com/Debian/apt/commit/cbbf185c3c55effe47f218a07e7b1f324973a8a6 for the specific commit) so that it would only happen for HTTP 404 errors. So even though they are both apt, one is newer than the other :smile:.
It might be worth making a commit for apt that fixes this, because I can imagine some other repos might still not be using InRelease files and are configured to give 403 errors instead of 404 Errors for security reasons (like this one).
@dorodnic, it might just be easier to make an InRelease file for the repo, (see here for the guide), especially since most Ubuntu 16.04 LTS users would start switching over to 18.04 LTS at the end of this month, which has an updated version of apt.
You should just need to run one command, gpg --clearsign -o InRelease Release and that should make the InRelease file (assuming the instructions in the above link are accurate). And that should fix it for everyone that has an apt that's been updated since 2017.

I know this is closed but I cannot find a solution - what is the current recommended means of installing  intel-realsense-dfu*  on ubuntu 16 or 18?

https://github.com/IntelRealSense/librealsense/blob/development/doc/distribution_linux.md seems to suggest that the repo has been updated for Ubuntu 18.04 bionic.
If that doesn't have what you want, you'd probably need to build everything manually, which might take some time, but there are some pretty good instructions for Ubuntu so you just need to copy and paste bash commands: https://github.com/IntelRealSense/librealsense/blob/development/doc/installation.md


[R] Fix memory safety issues

fixes https://github.com/dmlc/xgboost/issues/7246
ref https://github.com/dmlc/xgboost/issues/9810
ref https://github.com/dmlc/xgboost/pull/9817
This PR fixes some remaining memory safety issues:

Allocations from R interleaved with allocations from C++ that could leave leaked objects in case of R failures.
Lengths using a type of insufficient width that might overflow.

Regarding your other comment:

Is it possible to return errors in a more linear fashion?
It would require modifying the macros R_API_BEGIN, R_API_END, CHECK_CALL; and then modifying functions to be mindful of whether they put objects before/after those. I guess that's more work than the current code here, but an example of how it could work otherwise can be found in lightgbm: https://github.com/microsoft/LightGBM/blob/master/R-package/src/lightgbm_R.cpp

If you want an easier and safer way, there's also Rcpp::exception, Rcpp::unwindProtect, etc. but that would involve having Rcpp as a dependency.

Then, there's also  the possibility of passing R allocator functions to C++ functions, but this is perhaps a lot more involved and harder to remember:
https://cran.r-project.org/doc/manuals/r-release/R-exts.html#Memory-allocation

@trivialfis It looks like the CUDA job uses R 3.3, which is very old so doesn't have the unwind protection that was introduced in R 3.5, for example. Could you update those to R >= 4.3 for this and the following PRs?

Could you update those to R >= 4.3 for this and the following PRs?

I have merged the update into this branch and the test build has passed.


DONT MERGE!!!! chore(ci): fix double npm release by separating jobs

[!CAUTION]
we must coordinate it with next time we want to release, otherwise we'll try to publish 0.3.0 to npm again


GEODE-9193: Use static object for PING messages.

Prevents a leak at shutdown, ensures object is properly destroyed.

Converting to draft for the moment, trying to nail down what's going on with CI failures.


Ensure PostgresAdapter::useIdentity is always initialized

Wrapper package would call setConection() without connect() causing useIdentity to not be set.

This was the wrong change.


Keep seen_on for cached events

That way we can actually show where stuff was loaded from everywhere.

This now looks more like caching our global tracker


Defining sets of valid option values

 Issue by bergbria
Monday Sep 28, 2015 at 17:27 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/245

I'd like to create an option where there is a known, restricted set of valid values which cannot be expressed in C# as the name of an enum. For example, the set might look like {8.0, 8.1, 9.0}. Is there any way to express this using this library?
If not, what would be the best way to add support for this? A custom attribute for enum fields, another parameter for the Option attribute?

 Comment by nemec
Saturday Apr 23, 2016 at 21:34 GMT

Hi @bergbria, there is no way to support that today.
Since floating point numbers will never be valid enum values, I think it would make sense to add the ability to restrict valid values of any arbitrary option (like double Version {get;set;})
One option (like Powershell):
class Option
{
    [Option('v', HelpText="Version Selector")]
    [RestrictedValues("8.0", "8.1", "9.0")]
    public double Version { get; set; }
}

It has a number of negatives, like case sensitivity and being "stringly typed", but attributes only allow compile time constants and arrays, so you're very limited in what you can do.
Another option that offers a bit more flexibility could be a companion property:
class Option
{
    [ValueRestrictions("Version")]
    public static VersionRestrictions[] = { 9.0, 8.1 }

    [Option('v', HelpText="Version Selector")]
    public double Version { get; set; }
}

But it leads to clutter in your option class.

I like option 1. I think it's quite common to have arguments that only support a limited set of fixed values. As it stands I can't see an easy way to simply display usage text if I detect a parameter value isn't valid - HelpText.AutoBuild seems to want a not parsed instance. Ideally I'd be able to define restricted values and it'd do it for me but if I need to further validate values manually it'd be nice to easily display help text for valid parsing but invalid values, just like it does when I pass an invalid command line argument.

I guess something like this works, using a different overload of AutoBuild
parseResult
    .WithParsed(options => {
        var mode = options.Mode.Substring(0, 1).ToLowerInvariant();

        if (modeMap.ContainsKey(mode))
        {
            modeMap[mode](serviceProvider);
        }
        else
        {
            Console.Error.WriteLine(HelpText.AutoBuild(parseResult, null, null));
        }
    })
    .WithNotParsed(errors => {
        Console.Error.WriteLine("Error parsing command line");
    });


Unable to uninstall

OS:
Linux Fedora (VM)
Command used:
sudo apt-get purge --remove youtube-dl-2021.06.06.tar.gz
Result:
E: couldn't find any package by glob 'youtube-dl-2021.06.06.tar.gz'
E: couldn't find any package by regex 'youtube-dl-2021.06.06.tar.gz'
Yet the program still exists in /home/user1 as a write protected file.
I'm new to Linux, this may be a problem all on me being inexperienced but any help is greatly appreciated!
Edit:
I entered:
pip uninstall youtube-dl
and got the following:
WARNING: Skipping youtube-dl as it is not installed
How can it not be installed when I was using the program yesterday?

What did you use to install it?

Ya, how did you install it ?
You don't use the APT commands to [un]install things installed outside the repos.
In the context of a Python software such as ytdl on GitHub or something you always do 1 of in this order:

install from PyPi using pip or from Anaconda repos using conda. Easy to update and to keep the dependencies up-to-date. ytdl is officially auto-released on PyPi.
take the released executable binary.  It can be updated in place with youtube-dl --update, but it can still fail to find an update, which seems to be more likely with GitHub than with PyPi.
take the source code, either straight from the repo (or the 'Source code (zip)' or Source code (gz)) or the released package like that tgz. Being a Python app (and explicitly coded for running without being packaged) it can be started by python __main__.py.

if you installed using Fedora builtin package manager(dnf) then search for how uninstall fedora packages.
if you installed the package directly then just remove the file.

use  sudo pip uninstall youtube-dl
you might have pip installed as root user


Cluster-Level JWT Configuration

Problem Statement
Currently, CrateDB allows JWT configuration on a per-user basis. Each user must be set up individually with specific JWT settings, such as the issuer (iss), audience (aud), and username (username).
CREATE USER john WITH (jwt = {"iss" = 'https://example.com', "username" =<EMAIL_ADDRESS>"aud" = 'test_aud'})

This approach becomes cumbersome when managing multiple users, especially in larger clusters. Additionally, it complicates cluster restoration, as each user’s JWT configuration must be manually reapplied, increasing the risk of errors. Also this prevents from using SQL commands like CREATE USER without knowledge about the complete JWT/JWK setup.
Possible Solutions
A more efficient solution would be to allow JWT settings to be configured globally at the cluster level. By adding the relevant JWT configuration to the cluster’s configuration file crate.yml, it would eliminate the need for user-specific setups. For example, the configuration could look like:
external_auth:
  jwt:
    iss: https://example.com
    aud: test_aud

With this setup, any user can authenticate using JWT, provided the JWK supplies a valid token where the "username" matches the CrateDB user. This approach also simplifies the process of restoring clusters, as it avoids the need to reconfigure individual user settings. If a user has a custom JWT setup  (CREATE USER john WITH (jwt =...))., it could override the global settings, ensuring that the change remains non-disruptive.
This approach also facilitates user mapping scenarios, where CrateDB Cloud users (or users from other external authentication providers) can be mapped to database users. By enabling this route, it reduces the risk of errors associated with the per-user level setup and removes the need for users to know specific JWT setup requirements when creating users with the CREATE USER command.
Considered Alternatives
The only alternative is setting up JWT configurations individually for each user. This is error-prone, makes cluster restoration and automated user management more complex.

Implementation Details

Per user configs (CREATE USER xyz WITH ( jwt = ...)) remains supported
Per user configs override global configuration
The CrateDB username is extracted from the username field in the provided JWT token.

also see PR: https://github.com/crate/crate/pull/16763/files

One question here is how to deal with per user configs if there is a global jwt configuration. Options I see:

User config overrides global config
User config is ignored
User config is an error and not allowed. (Might break snapshot restore - would either have to ignore or remove the config?)

Behavior could also be an additional setting (but I'd defer that to add it once we've a requirement for a different behavior. Whatever behavior we choose now can become the default once a setting would be added)

User config overrides global config – I think this is the preferred and least invasive solution. If a user has a custom JWT setup (e.g., CREATE USER john WITH (jwt = ...)), it would override the global settings, ensuring that the change remains non-disruptive.
User config is ignored – This could work as well, as the cluster administrator can decide which authentication method to use. However, it would lead to a user experience where you can set a configuration on the user, but it is ignored, which might be confusing.
User config is an error and not allowed – Since this would break many managed clusters and require fixes, I would strongly vote against this option.

@tomach wdyt?

I would prefer option 1 as well. Most important thing is, that the changes do not break the setup for existing users which were already created WITH (jwt = ...)  after an upgrade to a version which supports the cluster-level config then.

I have a question from a security point of view:
On the face of it, this looks like a shared user, which is bad security. However, what we have here is:

Each person still has their own username and password in the external system (issuer) example.com. Individual persons can change their password or be disabled by admin. So this is good.
How will it work on the CrateDB side? Even if having one common user with necessary permissions etc is convenient, from an auditing perspective we would still want to know who specifically logged in. I assume this is satisfied with logging the username from the JWT json object? Will the JWT username ever be visible somewhere else? Should it?

It's not a shared user, it's delegating authentication to an external service. (Yes, you're delegating trust - but that's not different to SSO). We're still doing the authorization.
All it means is that all users implicitly gain the JWT option - using the globally configured server.

By default, old (user) jwt config is enabled. No change in functionality.
When global setting is enabled, disable the old functionality.
Long term, deprecate user jwt config

To be defined: Errors: CREATE / ALTER USER jwt=... when global config is enabled or warning


Final talk and never will bothering you after that

My dear, I worked hard for your repository and as you can see in my Video PoC in previous issue, There are several CSRF vulnerabilities in your very nice application and you should make CSRF token for each endpoint but only for login.
Can I ask you validate my reports on huntr.dev please?
they will remain hidden and unpublished until you submit a fix for them and also you can give White hackers like me A good motivation and help the open source community, be free and make a free decisions my dear.
Best regard,Amammad.

Hello,
Thank You for message but I’m not implementing CSRF tokens for each end points.
Reasons / Explanation.
Overall the CSRF token protection makes in general no sense in my project – that’s because the project should NEVER be accessible outside. If someone by any chance is putting his data outside of local network then this is not what the project was created for.
External data exposition should be handled by my other project created for just that purpose:
https://github.com/Volmarg/personal-management-system-io
Then again the project above is for personal usage only thus giving it to other ppl is a no.
Above mentioned tool, already has a fine set, of data protection.
I’m working on the other project where CSRF check for every end point makes sense because it might / most likely will be used for commercial purposes so naturally I want to avoid cases when backend route gets called from unauthorized source.
I’m open for safety matters but such as the one above as this has no right to exist in first place.
This is what has to be remembered here: The project goal is to having it in local network.
I know from other ppl that if they expose the project externally they DO filter the connections in local network so that is secure enough if they follow the: “Deny all”.

OK my dear I hope always you have a secure programming days :)
Best regards, Amammad.


Add a gallery JupyterHub service

This unauthenticated hub service can render our
gallery page easily, and launch a new user + server
per gallery launch. This solves a number of problems:

Letting users view multiple galleries at once, without
having to do complex things with named servers
Back button works correctly, since it no longer sends you
to the spawn page (Fixes #3)
Much easier to iterate and work on, since this is just a
tornado application
We have full access to the hub API, which can be very useful

Putting this in /hub/spawn was a hack, and IMO making this
into a service is the 'right' thing to do
Todo:

[x] Integrate this into the TLJH plugin
[x] Figure out what to do about binderhub dependency
[x] Make this the 'home page' when people visit the domain

/cc @jtpio (and Theo, whose GitHub handle I know not) who were working on something very similar to this at the hackathon, but for a very different use case.

@pbugnion try this out at http://<IP_ADDRESS>/services/gallery/

Ok, now available at http://<IP_ADDRESS> directly! This is ready for review :)

This will create problems with idle culling, see https://github.com/jupyterhub/binderhub/issues/831. We already don't do idle culling though...

Really nice @yuvipanda!
cc @tam203 (Theo)

We just got access to OVH resources so we can start looking into getting this to run on K8s as well.

@jtpio awesome. Although I am wondering how image start performance is gonna be. It'll need an image registry and pushing to everywhere...

@jtpio Do you think I can get access to the OVH account? :)

/cc @danielballan who is also interested in the work @jtpio and @tam203 are doing

@jtpio Do you think I can get access to the OVH account? :)

Of course! Will come back to you with the details.

Currently testing on http://<IP_ADDRESS>, seems to be working fine including the back button.
country-indicators and render-stl work fine, but the gaussian-density example is giving 500s.

but the gaussian-density example is giving 500s.

Seems like it's fixed after updating to the latest version of voila. See https://github.com/voila-gallery/gallery/pull/19

@jtpio I rebuilt the images with #19 and it seems to work.

Perfect, thanks @yuvipanda!


Dependency version conflicts preventing package publishing.

Both of the HTTPoison and Poison dependencies of this project conflict with versions specified as dependencies of Dialyze. To get around this for development I added the override: true flag. The problem is that this prevents publishing to Hex.

This was fixed by forking coverex temporarily with updated versions. 1.0.0 has been published to Hex.


[Feature Request] Implement postcss-sort-media-queries-like auto media query sort

Describe the feature request
This could be implemented as an option of stylexBabelPlugin.processStylexRules.
See also: https://www.npmjs.com/package/postcss-sort-media-queries

We won't be doing this, but a feature to automatically detect and resolve conflicts between media queries is planned. It should result in the same experience in practice.


Should we add "tracepoint:syscalls:sys_enter_clock_nanosleep" in to bpftrace tool "naptime.bt" ?

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

Describe the solution you'd like

Describe alternative solutions or features you've considered

I guess we should. @brendangregg wdyt?


the "Sign out" button on the not-allowed page doesn't do anything

There's a "Sign out" button on the page that loads when you're not authorized to sign in to a site (/not-allowed). It's linked to an empty anchor. What should it do, exactly?

I would imagine it would allow the user to sign out of Persona?
We had this problem come up with one of the onboardings but once someone signed into Persona, and they hit the not authorized screen, there was no way for them to switch to a different Persona account so they had to use a different browser.

+1 - It should be to sign out of a persona account that's already logged in (and not allowed).

Is there another way to do this at the moment? Can you go to login.persona.org and log-out there?

Users can definitely do that but there's no way they'd know to do that from the Chime app right now. Not sure whats possible, but I see three potential paths:

The button simply redirects you to login.persona.org and tells you you should log out of Persona in order to log in as a different user.
The button actually triggers a persona signout
The button somehow disassociates the current Persona user from the current Chime user and effectively logs them out of Chime. (And gives them the option to sign in again as a different Persona user.)

:hand: for option 2. There’s no concept of a current Chime user distinct from a current Persona user—they’re both the person’s email address.

I'll look into this, there's probably a way to do option 2 (maybe already written but not hooked up!)

This looks like it could be it?
https://developer.mozilla.org/en-US/docs/Web/API/IdentityManager/logout

probably related, from @wpietri in Slack:

By the way, it looks like our small red "sign out" button doesn't work if you happen to sign in to an account without access. The big blue one in the upper right corner does, though.


Set default to multi_processing

@yuvipanda Made as a PR so that you can take this change or leave it, but it may help us to get the backlog clear

This should be fine if unit tests pass b/c of this: https://github.com/pangeo-forge/pangeo-forge-runner/blob/main/pangeo_forge_runner/bakery/local.py#L43

looking into the test fix now 👀

Test failure here is just due to a bad import as we're installing from a branch that has moved forward... I'd suggest a rebase after this PR goes into yuvi's PR

I'm going to be a bad boy and go for this merge. @yuvipanda I'm sorry!


Gap near top of left-hand sidebar

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
6.0.8
Describe the bug
With the latest version of Obsidian on iOS/iPadOS (1.4.0) and the most recent version of Minimal Theme (6.0.8), there's a weird gap between the quick controls on the left side of the left-hand sidebar and the top of the sidebar. See screenshot below.

I've answered that this bug "isn't" present in the default theme, but I think this may be because of how the quick-controls are styled in the default theme as opposed to Minimal - the same gap is visible in the default theme, but because the quick controls aren't styled differently than the rest of the left-hand sidebar it isn't obvious. (The screenshot below shows the same region with the default theme.)

Debug info
SYSTEM INFO:
Operating system: ios 15.7 (Apple iPad)
Obsidian version: 1.4.0 (80)
API version: v1.0.0
Login status: logged in
Catalyst license: vip
Live preview: on
Legacy editor: off
Base theme: dark
Community theme: Minimal
Snippets enabled: 0
Restricted mode: off
Plugins installed: 5
Plugins enabled: 0
RECOMMENDATIONS:
Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.

Closed in 6.0.13
https://github.com/kepano/obsidian-minimal/releases/tag/6.0.13


Can this library work with RFdigital's Simblee nrf51822?

Please advice if this library can be compiled for Simblee. What are the changes necessary for test? I have the Simblee module here.

Hi @danlsk,
Probably because it's based on nRF51, but it will require some work. To start find the references to __RFduino__ in this library, and see what the equivalent define for Simblee is, then add a || defined(<simblee define>).
Then you'll have to somehow figure out what soft device version RFdigitial has flashed on it, and get the appropriate Nordic headers. For RFduino it was flashed with S110 v5 which is quite old, the headers for those are in src/utility/RFduino.

Hi  @sandeepmistry ,
I have made some modifications to || defined(Simblee) . The board.txt seems to use Simblee.  I believe Simblee is using S130 soft device since it is able to perform both central and peripheral roles. There is no published soft device header for Simblee and I intend to ask RFdigital for this. But, meanwhile, how should I point to any current S130 headers? How should I copy Redbearlab one?

I believe Simblee is using S130 soft device since it is able to perform both central and peripheral roles.

That sounds right.

There is no published soft device header for Simblee and I intend to ask RFdigital for this.

Ok, let's see what they say. I've heard a lot of people being upset about them not open sourcing the RFduino code that was mentioned in the Kickstarter campaign.

In fact, I was surprised how you sorted out one for RFduino.

A bit of luck, just tried out a few versions of different S110 headers 😃

But, meanwhile, how should I point to any current S130 headers? How should I copy Redbearlab one?

RedBearLab might be too old. If you click on the "Downloads" tab on this page: https://www.nordicsemi.com/eng/Products/Bluetooth-low-energy/nRF51822
There is an option to download S130 v1 and v2. Try both of those starting with v2. There are a few #ifdef's in the nRF51822.cpp/h for RFduino that tie it to S110, you'll have to switch those to use S130 instead.
Also, we'll need to figure out if it has an onboard crystal or not, because the RFduino didn't, but all RedBearLab's board do.
Btw, is the Simble Arduino core available on Github somewhere?

The Simblee Arduino core is not available on Github. However, if you have installed the board support package by entering https://www.simblee.com/package_simblee166_index.json into the board support URL and installing it via board manager , you might be able to find the library headers in this installed folder below.
C:\Users\<your user folder>\AppData\Local\Arduino15\packages\Simblee\hardware\Simblee\1.1.0
Is the library affected by the onboard crystal? RFduino and Simblee are System In Package that typically integrated the 32KHz crystals on package rather than  onboard. I am surprised you brought this up that the Raytac doesn't include this. Simblee is compiling and running at F_CPU 16MHz and not in overclocked mode. Is this the info you need on CPU clock speed?
I will get those nrf SDK library and dump into a library folder for compilation then. Will update you again.

I copied the S130 v2 header file into local \utility\Simblee
Copied nrf.h and compiler_abstraction.h into the folder too.
In nrf51822.h
#elif defined(__Simblee__) #include <utility/Simblee/ble_gatts.h> #include <utility/Simblee/ble_gattc.h> #include <utility/Simblee/nrf_soc.h>

I have this error during compilation.
error "Device family must be defined. See nrf.h."

Is the library affected by the onboard crystal?

Just need to use the right option when calling sd_softdevice_enable in nRF51822.cpp.

I have this error during compilation.

Try adding a #define nRF51 before the includes Simblee include statements. I think the RFduino core has a -DNRF51 C compiler flag in the core.

There is a build.bat at C:\Users\<user folder>\AppData\Local\Arduino15\packages\Simblee\hardware\Simblee\1.1.0\system\Simblee\source that already has this -DNRF51. Is this batch file ran during compilation? If so, it should be there somehow. But, it doesn't seem to get triggered.

Is this batch file ran during compilation?

Probably not. It probably builds a static library.
Did you try my suggestion for adding a #define NRF51? You can also change the nrf.h include in the headers to nrf51.h like I did for RFduino.

Adding the #define NRF51 in sketch. ```
C:\Users......\Documents\Arduino\libraries\BLEPeripheral\src/utility/Simblee/nrf.h:51:38: fatal error: nrf51_deprecated.h: No such file or directory
#include "nrf51_deprecated.h"
Need to look for those library files.

Add what about "You can also change the nrf.h includes in the headers to nrf51.h like I did for RFduino."

In which file did you do that for RFduino? I don't see that in nrf51822.h?

See: https://github.com/sandeepmistry/arduino-BLEPeripheral/blob/master/src/utility/RFduino/nrf_sdm.h#L49-L53
It's including nrf_sdm.h not nrf_soc.h like you have in https://github.com/sandeepmistry/arduino-BLEPeripheral/issues/97#issuecomment-257501703.

Made those changes in both nrf_sdm.h and nrf_soc.h like you did for RFduino in /utility/RFduino.
#ifdef __Simblee__ #include "nrf51.h" #else #include "nrf.h" #endif
New error.
`C:\Users\danie_000\Documents\Arduino\libraries\BLEPeripheral\src\nRF51822.cpp: In member function 'virtual bool nRF51822::updateCharacteristicValue(BLECharacteristic&)':
C:\Users\danie_000\Documents\Arduino\libraries\BLEPeripheral\src\nRF51822.cpp:1047:116: error: cannot convert 'uint16_t* {aka short unsigned int*}' to 'ble_gatts_value_t*' for argument '3' to 'uint32_t sd_ble_gatts_value_set(uint16_t, uint16_t, ble_gatts_value_t*)'
sd_ble_gatts_value_set(localCharacteristicInfo->handles.value_handle, 0, &valueLength, characteristic.value());
`

Take a look at the NRF51_S130 define.
Can you push your work to a Github fork as well?

Do you know of a low level call that would definitely determine if it is S130 or S110 that is in the Simblee?

@sandeepmistry I got the answer from RFdigital that Simblee is using S110 stack. I almost uploaded my files but this seems to be totally wrong since I was working on S130.

@sandeepmistry I got it compiled successfully using the RFduino S110 but the BLE stack call doesn't seem to work yet.

The RFduino S110 can still allow the Simblee module to boot with serial output. The V8 S110 has problem with that. I see that you work quite a lot on the S110 for RFduino because you put up with all those preprocessor. I believe you know better on integrating the S110 for Simblee.

I believe you know better on integrating the S110 for Simblee.

Well I don't have a Simblee board ... closing for now then. Feel free to update your progress on the issue though.

I have a Simblee board I'm hoping to turn into a BT HID device, so I'm very interested in this issue. Don't really have any experience to contribute, though.


Release training graph before validation

The observation dictionary currently holds the reported Variable object as is. If a forward computation code reports a Variable object, it holds the whole computational graph during the iteration. If an evaluation runs at that iteration, the computational graph is left on the memory during the evaluation, so that the maximum memory consumption becomes high. This problem can be fixed by not preserving the Variable object but instead holding the array value.
We have to think about how to retrieve the computational graph from the dump_graph extension. The current implementation requires an access to the whole computational graph via the observation dictionary. We need some workaround to give dump_graph extension an access to the computational graph.

One idea is using a global flag to configure if the reporter should clone reported variables and purge computational graphs from them. dump_graph is then modified to disable the purge at the first iteration, dumps the graph using the graph left at a reported variable at the second iteration, and then enable the purge again. Since an evaluator does not run in the second iteration in most cases, it should work.
I will work on this after #2145 is merged so that I can use the configuration tool to implement the global flag.

Fixed via #2640


⚠ Bofrost.lu has degraded performance

In e43bdf3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2357 ms

Resolved: Bofrost.lu performance has improved in d7b33a1 after 32 minutes.


FIX rm deps of tests on celer

Update in celer broke the CI on benchopt. (cannot import ConvergenceWarning from celer.homotopy, see log here )
I think it would be better to not depend on master branch of any repo for our CI so I propose to remove it.

thx @tomMoral

thx @tomMoral


Create logo for HPLAY token

Our logo for Harmony Play token.
HPLAY
Contract address: 0x66cf8375fd083e7d103d69378e9af20f1b46dc59
Site: harmonyplay.one
Discord: https://discord.gg/R4hwev7A
Telegram: https://t.me/harmonyplayone
Twitter: https://twitter.com/harmony_play

Added in another commit


ChatOllama won't use with_fallbacks when I use astream_events.

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
import asyncio
from enum import Enum

from dotenv import load_dotenv
from langchain_core.output_parsers import PydanticToolsParser
from langchain_ollama import ChatOllama
from langchain_openai import ChatOpenAI
from pydantic.v1 import BaseModel, Field

load_dotenv()

class DateEnum(str, Enum):
    first_day = "2024-10-10 10:00:00"
    second_day = "2024-10-11 14:00:00"
    third_day = "2024-10-12 14:00:00"

class SelectItem(BaseModel):
    """Confirm the user's choice based on the user's answer."""

    item: DateEnum = Field(..., description="Select a date based on user responses")

tools = [SelectItem]
ollama_llm = ChatOllama(model="llama3.1:8b").bind_tools(tools)
openai_llm = ChatOpenAI(model="gpt-4o-mini").bind_tools(tools)
parser = PydanticToolsParser(tools=tools)

chain = ollama_llm | parser
fall_back_chain = openai_llm | parser
with_fallback_chain = chain.with_fallbacks([fall_back_chain])

messages = [
    ("ai", f"Which day is most convenient for you in {list(DateEnum)}?"),
    ("human", "30"),
]

async def main():
    async for event in with_fallback_chain.astream_events(messages, version="v2"):
        print(event)  # It will not call fall_back
    print("-" * 20)
    print(await with_fallback_chain.ainvoke(messages))  # It will call fall_back

asyncio.run(main())

Error Message and Stack Trace (if applicable)

Description
ChatOllama won't use with_fallbacks when I use astream_events.
But it will use with_fallbacks when I use ainvoke.
My goal is to know which model produced this output.
When I connect PydanticToolsParser behind the model output, I can't seem to know who generated it. (it is hidden in the AIMessage of the intermediate model output).
So I wanted to take out the intermediate result from astream_events to determine who generated it.
Later I found that ChatOllama seems to be unable to call fall_back under astream_events? Is there a better solution?
System Info
langchain==0.2.11
langchain-core==0.2.24
langchain-ollama==0.1.0
langchain-openai==0.1.19
platform linux
python version = 3.10.12

Hi, I am ji, 4th year student at UofT CS. I am working with @Ismail-Bashir @Ser0n-ath and @yashankxy who are also 4th year student at UofT CS. Can we take initiative to work on this issue and contribute to langchain?
Thanks 😊


Add option to nightly build to not deploy artifacts

Describe the bug
it would be nice to have an option on the nightly build to not deploy. This can be used when testing updates to the scripts.  If we deploy it we could potentially cause a mismatch in versions used.

Close the issue per PR#4269 fixed it