Is it possible to triangulate using a sparsely used vector of points as a data source?

Hi!  I recently found your detria implementation of CDT, and am interested in trying it out in our application.  We're currently using poly2tri (or more precisely a tweaked older version of poly2tri) to triangulate approximations of NURBS surfaces in 2D, and we rely on both holes and steiner points to get representative meshes.  Our input data is a series of C arrays holding polygon and point data.
Looking at the test cases, it appears that the steiner points are added into the set of points fed to setPoints, and the implication appears to be that any point in setPoints is either active in a polygon or is a steiner point - am I correct about that?
It's not a big problem if that's the case, it just means I need to copy the active set of points to their own vector before triangulation (which also gets the data into a form suitable for easy feeding to detria).  It would be nice to be able to directly use the data from the C arrays without copying, but that may be a bit too ambitious of me (even if I'm wrong about setPoints, my C++ template skills may not be up to making all the necessary connections to feed in everything as-is...)

Hi,
You're right, currently you have to set all points at once, with a single setPoints call. Points which are not referenced by any polygon will automatically become steiner points.
I've also came to the conclusion that this might not be the most flexible way - just as you described, this only works if the points are in a contiguous memory location.
Currently, we have the PointAdapter class, which was supposed to handle conversions from user input, but I'm thinking of replacing it with a PointGetter class, which would work with your use case as well.
I'd change the setPoints function to also have another overload which takes a lambda function (or a functor), and the number of elements. The lambda function then could return the point for the given index, and you could implement a custom logic to retrieve it from a sparse vector. For example:
std::vector<detria::PointD> outlinePoints = ...;
std::vector<detria::PointD> steinerPoints = ...;

uint32_t numOutlinePoints = uint32_t(outlinePoints.size());
uint32_t totalNumPoints = uint32_t(outlinePoints.size() + steinerPoints.size());

auto pointGetter = [&](uint32_t pointIndex)
{
    // pointIndex is in the range [0, totalNumPoints)

    if (pointIndex < numOutlinePoints)
    {
        // Point belongs to an outline
        return outlinePoints[pointIndex];
    }
    else
    {
        // Point is a steiner point
        return steinerPoints[pointIndex - numOutlinePoints];
    }
};

triangulation.setPoints(pointGetter, totalNumPoints);

Note that the points would still need to be random-accessible, which means that each point must have an unique index, and from an index, you should be able to get the corresponding point in constant time.
If the points are not random accessible, then you do have to create a copy of them (or create a mapping that is random accessible, by e.g. creating a vector of pointers to the points).
Let me know if this would work for you, I'll probably implement this in the next days.

You're correct - one of the use cases is to re-triangulate a small subset of a large mesh, and in that situation what we (try) to do is define an "outer" polygon and interior steiner points for the area of interest that is a subset of the larger mesh.  In that case the thought would be to just reuse the "parent" mesh's point set rather than having to make a new points array and re-index the polygon definitions for just the subset.
With poly2tri we're just duplicating everything in terms of their types, so it's not a big deal if we need to do the same thing here - I just got intrigued by the possibility of being able to avoid extra copies of our data.
I'm still getting things set up to feed in our data to detria (not working yet, just got something to compile last evening but I've not gotten it setting up correctly yet: https://github.com/BRL-CAD/brlcad/commit/8b6905b75a32c6351a2f3a600963d873fb153df8) - one thing I may be doing wrong there is setting the points after I'm adding the outer and hole polygons.  Do I need to do the setPoints call first, and then set the loops up?  (Not a problem if that's needed, I was just assigning them as I defined them for convenience.)
Also, is there a simple way to get a count of all "inner" triangles produced by a triangulation?  Right now I'm just iterating over the result and incrementing a counter, but at least internally it looks like there's a size() value that corresponds to the triangle count.  Would it be OK to add a method (if it's not already there and I missed it) to get that info?  Something like tri.InnerTriCount() and/or tri.HoleTriCount()?

Update - got detria working in our code.  Preliminary results looking good!

Oh, one other question I had - how deeply does detria depend on C++20 features?  Would it be practical to use only C++17?  Mostly asking because it can have an impact in some environments using older compilers.

I gave a C++17 compile a shot, and got the following errors:
https://brlcad.org/~starseeker/detria_cpp17_errors.txt

I've updated the code to work with C++17, I remembered that it just worked before, I guess I was wrong :D
Also, I've added "point getters", which allow a bit more flexibility with setting the input points:
auto pointGetter = [&](Idx idx) -> const Point&
{
    // Retrieve point from an index; 0 <= idx < numPoints
    return ...;
};

struct Config : detria::DefaultTriangulationConfig<Point, Idx>
{
    using PointGetter = decltype(pointGetter);
};

detria::Triangulation<Point, Idx, Config> tri;
tri.setPointGetter(pointGetter, numPoints);
...

I couldn't implement the version with fully sparse indexing, since the topology class stores the vertices (and other primitives) in contiguous collections, which means that they are indexed from zero up to size. And since we need to be able to retrieve the topology vertex from an input vertex (and also the other way around), we'd need to use a hashmap or something to map from the original index to the topology vertex index (which would probably be slower, and use more memory).

Awesome - thank you!  Confirmed the updated version builds in C++17 mode with our code.  Thank you for adding the "point getters" - I'll have to think about how to organize things on our end to take advantage in situations where we can do so.
I'll close this out rather than tacking on more question to the thread :-)

Alright, if you have any questions, feel free to ask!
I'll also update the documentation soon.


Add a 1ms debounce by default

This PR adds a 1ms debounce period by default to inputs.

Previous to this a simple switch would show a waveform like this:

Where you can see the "thick" vertical lines showing many transitions in a short period due to hardware debounce.
After this change when repeatedly pressing to try and provoke the same you see this:


has-participant subProperty of s-depends?

From<EMAIL_ADDRESS>on July 14, 2012 14:05:18
The reference says: "s-dependence obtains between every process and its participants".
This sounds like a sub property axiom, but is not marked as such in the reference.
Original issue: http://code.google.com/p/bfo/issues/detail?id=110

From<EMAIL_ADDRESS>on July 18, 2012 19:06:47
Barry concurs.
I am mixed, disliking the idea, but wanting to expose everything so that we can all peer at it.
Status: Started

From<EMAIL_ADDRESS>on July 24, 2012 13:15:53
Asserted in the OWL file - needs to be made an axiom in the reference.


Not installing php.56

Info:

Docker version 18.09.2, build 6247962
Laradock commit ($ git rev-parse HEAD):  d0ebb32ad121fee684201617e05b119e34c7241e
System info (Mac, PC, Linux):Mac
System info disto/version: Mojave

Issue:
Can't install php.56 . Installing 7.0 version

Expected behavior:
Install php 5.6

Reproduce:
in .env change php_version to 5.6 and run docker-composer build php-fpm

Relevant Code:
// place a code sample here

docker-compose build --no-cache php-fpm workspace

docker-compose build --no-cache php-fpm workspace

Restart full docker and it works.


Fix broken links to pages under /en/docs/tasks/administer-cluster/manage-resources/

Problem
Pages under /en/docs/tasks/administer-cluster/manage-resources/ has been moved from its parent directory at some time. But it seems that all the links have not changed.
What this PR fixes
This PR fixes all the links to pages under /en/docs/tasks/administer-cluster/manage-resources/.

/assign @makoscafee

/lgtm
/approve


adding convenience function for uploading from string

@mistercrunch
Just creating a helper function to expose another useful boto feature. For small files, this allows setting keys from in memory representations instead of going to a file each time.

Perfecto, merging!


Flag for max cache size

Further to the changes here https://github.com/librespot-org/librespot/commit/9d832baf83adf653243ac6e0b6de0b12b53b0a37
I think it would be nice to have a flag for --maxcachesize or something similar where you can define in MB a maximum cache size, without entirely disabling the cache.

Same as #34. Please continue discussion there.

Same as #34. Please continue discussion there.


More efficient BertEmbeddings

🚀 Feature

Motivation
I'm using a graph where each node holds a long text (up to 60k works). The current implementation of BertEmbeddings would only create Out-of-Memory issues.

the inputs to the Bert model are not batched
the mapping matrix cant be hold in memory
the tokenization is done on the fly
only BERT can be used, as it is hardcoded with BertModel.from_pretrained(...)
BertEmbeddings is using EVERY hidden state from EVERY layer of EVERY token

Alternatives

Pitch
I'm willing to pitch in a PR for a different BertEmbeddings class:

processes raw text OR pre-processed texts (so we can split pre-processing and embedding)
accepts any huggingface transformer model (so we can use lighter ones)
split the text in chunks, and average the embeddings of the CLS token of each chunk

Additional context

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


Video Not Playing [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null)

Hello
Steps to Reproduce

The path which I have given it show in my file manager and also video gets stored there but I'm unable to play this video it gives the error of open failed: ENOENT (No such file or directory) please help me out in this issue.

2.error: E/flutter (19622): [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1600060851325.mp4: open failed: ENOENT (No such file or directory), null)
3.E/ExoPlayerImplInternal(19622): com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1600060851325.mp4: open failed: ENOENT (No such file or directory)
Logs

D:\daq-dynamic>flutter doctor -v
[√] Flutter (Channel stable, 1.20.4, on Microsoft Windows [Version 10.0.18362.1082], locale en-IN)
• Flutter version 1.20.4 at C:\flutter
• Framework revision fba99f6cf9 (10 days ago), 2020-09-14 15:32:52 -0700
• Engine revision d1bc06f032
• Dart version 2.9.2
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at C:\Users\jk\AppData\Local\Sdk
• Platform android-29, build-tools 29.0.2
• ANDROID_HOME = C:\Users\jk\AppData\Local\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Android Studio (version 4.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 49.0.2
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] VS Code, 64-bit edition (version 1.49.1)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.14.1
[√] Connected device (1 available)
• ONEPLUS A3003 (mobile) • <IP_ADDRESS>:5555 • android-arm64 • Android 9 (API 28)
• No issues found!

you can refer to my repo git repohttps://github.com/jk-hub/Custom_camera
clone and check it. if you need any thing more please tell me also please looking into this video player issue
Thanks,

@jk-hub
What are the steps to replicate the issue ? I cloned your repo but see video and camera options but can't choose any video from storage. Am I missing anything ?
Thanks.

@jk-hub
What are the steps to replicate the issue ? I cloned your repo but see video and camera options but can't choose any video from storage. Am I missing anything ?
Thanks.

@drashankawar just click on video btn and start recording after stopping the video i have passed the path to video player for viewing the video but as soon as we stop the video and video player get the path of the clicked video but the path that i have given it of storage/0/ then our custom folder so it can not read that file that the issue I hope you got the point
if not I can explain again
Thank you.

@jk-hub
I am unable to stop video recording after tapping on it. Also, I get below exception:
Unhandled Exception: FileSystemException: Creation failed, path = 'storage/emulated/0/Own_Camera_Demo' (OS Error: Permission denied, errno = 13)
Can you try the official video_player plugin example and see if you get same error which you are getting originally ?
Thanks.

@jk-hub
I am unable to stop video recording after tapping on it. Also, I get below exception:
Unhandled Exception: FileSystemException: Creation failed, path = 'storage/emulated/0/Own_Camera_Demo' (OS Error: Permission denied, errno = 13)
Can you try the official video_player plugin example and see if you get same error which you are getting originally ?
Thanks.

@darshankawar  please go in app info and give  storage permission the path creation is failed because of that so please give the permission and try again also i tried with official video player plugin example but there same error is occurred which was initial occurred  so I request to you please check this issue.
error: E/flutter ( 7738): [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1601442884367.mp4: open failed: ENOENT (No such file or directory), null)
Thank you.

Issue replicable. Tested on Android device 8.1.0. After recording video, app throws below exception:
E/flutter (16380): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1601445645494.mp4: open failed: ENOENT (No such file or directory), null, null)
Not sure if this is implementation or plugin issue. Keeping this open for further analysis.

Issue replicable. Tested on Android device 8.1.0. After recording video, app throws below exception:
E/flutter (16380): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1601445645494.mp4: open failed: ENOENT (No such file or directory), null, null)
Not sure if this is implementation or plugin issue. Keeping this open for further analysis.

so now who is going to check this error or issue or who is going to resolve this issue @darshankawar
Thank you.

Issue replicable. Tested on Android device 8.1.0. After recording video, app throws below exception:
E/flutter (16380): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1601445645494.mp4: open failed: ENOENT (No such file or directory), null, null)
Not sure if this is implementation or plugin issue. Keeping this open for further analysis.

Any update on this till when this issue will get resolved.

Issue replicable. Tested on Android device 8.1.0. After recording video, app throws below exception:
E/flutter (16380): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.FileDataSource$FileDataSourceException: java.io.FileNotFoundException: /emulated/0/Own_Camera_Demo/Media/1601445645494.mp4: open failed: ENOENT (No such file or directory), null, null)
Not sure if this is implementation or plugin issue. Keeping this open for further analysis.

Any update on this till when this issue will get resolved.

Any update on this issue

Hi @jk-hub
Change
final path = Directory("/storage/emulated/0/$folderName/$subFolderName");
To
final path = Directory("/storage/emulated/0/$folderName/$subFolderName");

Hi @Jk-hub,
Looks like there was a implementation issue in your code and not  issue with the plugin,
I have updated your two files with

globals.dart
library own_camera.globals;

import 'dart:io';

import 'package:path_provider/path_provider.dart';

String timestamp() => DateTime.now().millisecondsSinceEpoch.toString();

Future<String> getDir() async {
  var folderName = 'Own_Camera_Demo';
  var subFolderName = 'Media';
  final Directory extDir = await getApplicationDocumentsDirectory();
  final String dirPath = '${extDir.path}/$folderName/$subFolderName';
  await Directory(dirPath).create(recursive: true);
  final String path = '$dirPath/${timestamp()}.mp4';
  return path;
}

camera_screen.dart
modified startVideo recording function.
  Future<String> startVideoRecording() async {
    print('startVideoRecording');
    if (!_controller.value.isInitialized) {
      return null;
    }
    setState(() {
      _isRecording = true;
    });
    _timerKey.currentState.startTimer();

    // final Directory extDir = await getApplicationDocumentsDirectory();
    // final String dirPath = '${extDir.path}/media';
    // await Directory(dirPath).create(recursive: true);
    var filePath = await globals.getDir();
    // final String filePath = '$dir/${_timestamp()}.mp4';

    if (_controller.value.isRecordingVideo) {
      // A recording is already started, do nothing.
      return null;
    }

    try {
      videoPath = filePath;
      await _controller.startVideoRecording(filePath);
    } on CameraException catch (e) {
      _showCameraException(e);
      setState(() {
        _isRecording = false;
      });
      return null;
    }
    return filePath;
  }

With these changes I was able to record and play the recorded video.  For more advanced features  I would recommend you to look at the sample example in the camera plugin..
Since this is not an issue with flutter but with your code, I am going to close this issue if you disagree feel free to comment and I will reopen it.


Enable core module embedding by default in CMake.

It seems that release workflows were unexpectedly broken when STDLIB was renamed to CORE_MODULE.
Changed the option default because I think it just makes more sense. Removed explicit calls to enable it. Fixed docs.
Should SLANG_EMBED_CORE_MODULE_SOURCE be disabled too? Does it do anything useful?
As reported in https://github.com/shader-slang/slang/issues/5631
NOTE: I haven't really tested it at this point, but should work.

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

The only reason to enable SLANG_EMBED_CORE_MODULE_SOURCE is so that the language server can provide useful documentation in hovering hints. We could disable this for most builds, and enable it only when building for language server, but that likely needs to come as its own PR. We can change the language server code to fetch the core module source from a separately deployed file that ships with slangd in the vs extensions, and get rid of this embeding in slang.dll to reduce binary size.

I appreciate fixing the mistake I made.
I guess I didn't carefully check directories whose name starts with dot, "."


Update uk translation in README-uk.md

The site cannot build without a local copy of the theme. Use git submodule update --init --recursive --depth 1 update the theme.
use make serve to build and test website locally.

/language uk

@kubernetes/sig-docs-uk-reviews please take a look at this one

@kubernetes/sig-docs-uk-reviews please take a look at this one

/assign @MaxymVlasov

/assign @MaxymVlasov

Thank you!
/approve

Thank you!
/approve

/assign @sftim
please merge, it's OK

/assign @sftim
please merge, it's OK

Echoing https://github.com/kubernetes/website/pull/24840#issuecomment-759688033
/approve

Echoing https://github.com/kubernetes/website/pull/24840#issuecomment-759688033
/approve


[$500] Chat - Markdown is pasted only once when message with markdown copied

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.39-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4292604
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Log in to New Expensify
Navigate to any chat
Send message with markdown and copy it
Paste copied message in compose box at least twice

Expected Result:
Everytime the message is pasted, it should contain copied markdown
Actual Result:
Message's markdown is only pasted once. Second time and onwards message is pasted as plain text
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/4368652f-5be3-428b-9d05-1dbb37edb1bd
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015f143311cf0c364a
Upwork Job ID:<PHONE_NUMBER>655240704
Last Price Increase: 2024-02-09

We think that this bug might be related to #vip-vsp
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue
Message's markdown is only parsed once if pasted one right after the other, the second time and onwards the message is not being parsed, instead remains as plain text.
What is the root cause of that problem?
The issue comes from our expensify-common/ExpensiMark.js library which we use to parse the chat input text to markdown.
The problem lies in the incorrect usage of word boundaries (\b and \B) in regex patterns, which do not correctly match markdown syntax across sequences.
What changes do you think we should make in order to solve the problem?
A fix for bold and italic only would look something like this:
{
    name: 'italic',
    regex: /(?:^|\s|>)(_)(?=\S)([\s\S]*?\S)_(?=$|\s|<)/g,
    replacement: '$1$2$3',
},

{
    name: 'bold',
    regex: /(?:^|\s|>)(\*)(?=\S)([\s\S]*?\S)\*(?=$|\s|<)/g,
    replacement: '$1$2$3',
},

This modification adds a non-capturing group (?:^|\s|>) to check for a start of string, a whitespace, or a > character before the markdown symbol and (?=$|\s|<) to check for an end of string, a whitespace, or a < character after the markdown symbol. These changes help ensure that markdown styles are correctly applied throughout the text.

Proposal
Please re-state the problem that we are trying to solve in this issue
Message's markdown is only parsed once if pasted one right after the other, the second time and onwards the message is not being parsed, instead remains as plain text.
What is the root cause of that problem?
The copy to clipboard uses this function to get the text only version of the message:
https://github.com/Expensify/App/blob/c3555eddaad725625c86445b355647ccb09ca61c/src/pages/home/report/ContextMenu/ContextMenuActions.tsx#L39-L42
and since this only extracts the text, when pasted multiple times back in the composer this will be plain text instead of markdown.
What changes do you think we should make in order to solve the problem?
Change the copy to clipboard getActionText function to check if the curr?.text === curr?.html and if they are not equal this means the message contains markdown.
In this case we need to use the ExpensiMark library and return parser.replace('${acc}${curr.html}') which will parse the html message into markdown then copy it to the clipboard instead of copying the plain text message.

This bug no longer exists, adding it to the tracking issue

@thienlnam would you like to close this?

I'm holding onto them so we can track that they're fixed in the new PR and don't get recreated once we merge again

Okay, I've made that clear in the title.

Same as above

@thienlnam we should be able to close this one now


Scrape posts from Quora

Description
Use python to scrape posts from quora and save the result in a txt or pdf file
Type of issue

[x] Feature (New Script)

Checklist:

[x] I have read the project guidelines.
[x] I have checked all the existing projects, before submitting a new project issue.
[x] I have checked previous issues to avoid duplicates.
[x] This issue will be meaningful for the project.

I can work if it is still up.

I would like to work on this issue.


Add WES-NG

Add WES-NG to the UsefulRepositories list in the 'Offensive' section

https://github.com/bitsadmin/wesng
https://github.com/Daethyra/Cybersecurity-References/blob/main/UsefulRepositories/UsefulRepositories.md

Done


Fix scroll and wand trick magic item skill selection from spell tradition

Are scrolls and wands not gaining the traits of their spells?

Are scrolls and wands not gaining the traits of their spells?

Not the traditions apparently. Which kind of makes sense since that decision happens upon casting for spells with multiple traditions.


Latest projects and maintainers are not being added to the site.

Even after the latest PRs have been approved, the changes are not being reflected on the website
https://github.com/github/brasil/pull/64
https://github.com/github/brasil/pull/68

Some deployments are working but are not adding the latest projects and maintainers. Or did I understand it wrong? #84
@AndreaGriffiths11 is it possible for someone to help us?


Feature request template added

fixes #162
@PiyushKalyanpy Kindly review....

Hey @Tushar98644, can you please share some screenshots or the text preview of this? How is it looking?
And, instead of tagging the people in your comment just tag the people in the reviewers section. Check this doc.

I don't think the repository permissions allow for a contributor to request a review , you need to be a collaborator or have write permissions , please confirm from your end

The changes requested were not done, so we acre closing this PR, please reopen the PR once you made the changes.


Remove translation of exception on reconnect

This translation logic breaks Rails parallel tests, because that explicitly rescues the underlying ActiveRecord::NoDatabaseError. This means it fails all tests when running with parallel tests.
Based on Rails upstream, this translation is done there either so this drops it here as well. One test had to be slightly updated for this. But that test is also deleted already on upstream Rails.
WIth this change, we can successfully run tests against our Rails app with Trilogy instead of mysql2.

WIth this change, we can successfully run tests against our Rails app with Trilogy instead of mysql2.

Not entirely true, we also hit https://github.com/trilogy-libraries/trilogy/pull/97 but we're getting there :smile:.

@dbussink thank you! can you share what version of Rails you are using?

@dbussink thank you! can you share what version of Rails you are using?

This is happening on the latest release, 7.0.6. See also the code here:
https://github.com/rails/rails/blob/7-0-stable/activerecord/lib/active_record/tasks/database_tasks.rb#L405-L421
This explicitly only rescues ActiveRecord::NoDatabaseError which happens with parallel tests since it runs against  N different test databases. It then creates the database and tries again.
The wrapping code called here ends up turning the ActiveRecord::NoDatabaseError into a generic ActiveRecord::StatementInvalid error and doesn't get rescued, no database creation happens and all tests fail.
The upstream Trilogy adapter doesn't do this translation either so that's why I'm proposing to remove it here too.

@dbussink thank you for the explanation! 🙏🏻

@bensheldon With this and all the other fixes, would a new release be coming soon? I can also further test against latest git to see if there's any other issues we hit before a release?

@dbussink I have a few more PRs I'd like to get merged in before I cut a release. Please use git in the meantime (but soon!)


update readme to reflect new focus on model augmentation

@mehalter what do you think? does it capture the essence of the ModelTools component?


Inconsistent resource type information

I'm curious about how this was chosen as a resource's ResourceType: https://github.com/gohugoio/hugo/blob/2f721f8ec69c52202815cd1b543ca4bf535c0901/resources/resource_spec.go#L203-L207
This is causing issues when trying to find and reference video files as resources. The .MediaType.MainType and .MediaType.SubType are correct, but this inconsistent behavior makes using the .Resources.ByType function difficult. Take this example:
Given these files:

image.jpg
image.png
movie.mp4
movie.ogg

.Resources.ByType "image" will return image.jpg and image.png, but .Resources.ByType "video" returns nothing. Wouldn't it be desirable in this case to return the mp4 and ogg files? .Resources.ByMediaMainType would be nice to have I guess.
I'm afraid I don't understand why the ResourceType exists in the first place. Is it possible to simplify Hugo and just use MediaType alone?

@cmahnke @bep I believe this is closed with the advent of 81975f847dc19c21c2321207645807771db97fab?


Don't use setjmp/longjmp for exceptions

In C++, setjmp/longjmp are not compatible with C++ exceptions. Therefore, I want to change the berry exception to error code mechanism. I think it should be possible to use the int return value of the native-function to be compatible with the existing code as much as possible.
Simply put, we may need to provide some macros:
#define be_call(vm, argc) for (int errcode = be_call_(vm, argc); errcode;) return errcode;
#define be_raise(vm, except, msg) do { be_raise_(vm, except, msg); return BE_EXCEPT_RAISED; } while (0)

In short, when an exception occurs, the function will always return along the call stack. Due to the use of return semantics, this will be compatible with C++ exceptions.

I have a couple of changes around solidify that I will submit before 1.1.0.

Will this work without C++ exception support? Tasmota uses C++ but disables C++ exceptions completely (for code size reasons)

Will this work without C++ exception support? Tasmota uses C++ but disables C++ exceptions completely (for code size reasons)

Yes, we generally disable C++ exceptions in the MCU.  Let me give an example:
struct Test {
    char *data;
    Test() { data = new char[10](); }
    ~Test() { delete[] data; }
};

int berry_func(bvm *vm) {
    Test obj;
    // do somthing ...
    // if be_raise is called, longjmp will be called to pop the stack
    // directly instead of Temp::~Temp(). So that obj cannot be destructed.
}

I understand but I'm still nervous on this change. When running C code called by Berry, it is very handy to call be_raise() deep in the C code when something goes wrong and longjmp does its magic. With the new scheme, I need to change the entire call chain to get back the error code from call to call.
Would there be any way to still enable longjmp when Berry calls C code for which I'm certain that there are no C++ exceptions nor cleaning code needed?

This is the main problem I'm worrying about now.

Would there be any way to still enable longjmp when Berry calls C code for which I'm certain that there are no C++ exceptions nor cleaning code needed?

There's nothing that prevents you calling setjmp/longjmp from a C only chain. The only requirement is making sure there's no C++ with exceptions involved in the chain, or if they are, that the C++ code does not leak exceptions up.
So the easiest solution is to add 2 macros:
#define EnterCPPCode   try {
#define LeaveCPPCode  } catch (const std::exception & e) { be_raise(vm, CPP_CODE_EXCEPT, e.what()); } catch (...) { be_raise(vm, CPP_CODE_EXCEPT, "unknown C++ exception"); }

As long as you don't mix the C++ and C code in the same call chain, it'll work.

Thanks, I love this approach. It should be obviously added in the documentation.
As mentioned, C++ exceptions are often disabled in embedded systems, so we're safe here. The danger would be in non-embedded systems.

Well, avoiding C++ exceptions in Berry should be a very viable approach. And I found that people don't use C++ exceptions very much.


Update JFrog CLI to 2.48.0 and update dependencies

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] This pull request is on the dev branch.
[ ] I used npm run format for formatting the code before submitting the pull request.

Looks like newer versions have been released. Please update to the latest released version (2.49.1 at time of writing).
@ashokm
Yes, that's the reason we waited. We will release it soon.


Can Lilygo T-A7670E charge internal battery from vbat pin?

hi
I have a question: Can Lilygo T-A7670E charge internal batter (NCR18650B) from external battery by vbat pin?
And can the battery be measured?

battery is measured via resistor divider 100k:100k on GPIO35. Measure voltage on GPIO35 and multiply by 2.  VBAT pin is directly connected to battery, you can charge it via this pin using common li-ion chargers.

thank you @theMladyPan, it work for me


set lastGopDTS smaller then 0

As discussed in an issue, we have a DTS which is less then 0 so we set the lastGopDTS to  big negative value, so this should be a workaround for the issue

This closes issue #55


Serving models with custom op using tfgo

Hi, I trained a model with custom op and export it using saved_model API and I would like to serve it using tfgo. However, since tfgo binds to tensorflow C library(more precisely libtensorflow.so) which is very hard to modify, I'm not sure how to register the custom op in the binary in this case. Is there any chance and would anyone give some help?
Here is an example of training script in which dynamic_embedding is implemented as a custom op built in the library tensorflow-recommenders-addons, and when I try to load the model using tfgo I get the following error:
panic: Op type not registered 'TFRA>CuckooHashTableOfTensors' in binary running on CXJK8129239. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) `tf.contrib.resampler` should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.

Same issue for TensorFlow Decision Forests models, they require TF custom ops.

Custom ops can be loaded using
tf.LoadLibrary(opFilePass)
For decision forest i found ops file here.
File path: /tensorflow_decision_forests/tensorflow/ops/inference/inference.so


Better subquery rewriting

Not all subqueries get flattened properly, perhaps see this paper:
http://www.btw-2015.de/res/proceedings/Hauptband/Wiss/Neumann-Unnesting_Arbitrary_Querie.pdf

this is duplicate of #81


nodes panic when checking incoming blocks

We observed this on devnet on several API nodes. The log says:
message: "called `Option::unwrap()` on a `None` value"
panic.column: 68
panic.file: "zilliqa/src/consensus.rs"
panic.line: 1731

It was the unwrap in consensus.rs on line 1731:
        // Derive the proposer from the block's view
        let proposer = self.leader_at_block(&parent, block.view()).unwrap();

since leader_at_block_raw() returned None on line 2115-2117 because the parent block was missing:
        let Ok(state_at) = self.try_get_state_at(block.number()) else {
            return None;
        };

which is strange because we would have returned an error on line 1710 before we even attempt to get the proposer on line 1731:
        let Some(parent) = self
            .get_block(&block.parent_hash())
            .map_err(|e| (e, false))?
        else {
            warn!(
                "Missing parent block while trying to check validity of block {}",
                block.number()
            );
            return Err((MissingBlockError::from(block.parent_hash()).into(), true));
        };

Caused by #1056


mongodb 3.2 needs different repo

mongodb_version empty should take the latest stable which is 3.2 but the task always points to the 3.0 repo
https://github.com/William-Yeh/ansible-mongodb/blob/master/tasks/use-apt.yml#L23
see also this ticket about Support for Ubuntu 16.04 Xenial support for 3.0:
https://jira.mongodb.org/browse/SERVER-24634
The correct repo for 3.2 is
http://repo.mongodb.org/apt/ubuntu/dists/xenial/mongodb-org/3.2/multiverse/binary-amd64/Packages

Another option is using the stable repository which will always point to the latest stable version

Yep, 3.4 is out already. This needs to be updated.


Responsiveness doesn't seem to be working

This seems to be an issues on the homepage and request pages, at least.

Not an issues on homepage for https://data.openaccessbutton.org/

Fixed. css error


Display theme accent color in theme picker

What's the Problem?
In the settings menu there is a Color section but you can only choose Auto/Light/Dark. Several themes, including the default theme, use accent color, but it is not configurable in settings.
Solution/Idea
We could add an indicator (i) containing an explanatory message and a link to open the Windows color panel.
Alternatives
We can provide a color picker to change the accent color in the color section.
We can also, for each theme, display if it uses the accent color or not.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

Some themes change the accent color in order to ensure proper contrast, I think we can add the accent color to the little preview in the theme section.

Custom themes are currently revamped so this is no longer applicable.


After the android R8 packaging is completed, there is a problem with libgltfio-jni.so

Describe the bug
I am using APG7.2 for packaging Android project. In the debug environment, I can use filament smoothly, but in the release environment, I can't use it, and an error is reported.
To Reproduce
Steps to reproduce the behavior:
    release{
        buildConfigField("boolean", "IS_DEBUG", "false")
        isMinifyEnabled = true
        isShrinkResources = false
        isDebuggable = true
        proguardFiles(getDefaultProguardFile("proguard-android-optimize.txt"), "proguard-rules.pro")
        signingConfig = signingConfigs.getByName("release")
        manifestPlaceholders["NETWORK_SECURITY_CONFIG"] = "@xml/network_security_config"
    }

Expected behavior
I expected it to work
Screenshots
If applicable, add screenshots to help explain your problem.
Logs
2022-08-18 11:00:40.095 A/x: runtime.cc:642]   native: #13 pc<PHONE_NUMBER>1cd8ac  /data/app/x.x.x-ogXgD09McunoKUsWjxwRIA==/base.apk!libgltfio-jni.so (offset 8b39000) (???)
2022-08-18 11:00:40.101 A/x: runtime.cc:642]   native: #33 pc<PHONE_NUMBER>4c2fc4  [anon:dalvik-classes28.dex extracted in memory from /data/app/x.x.x-ogXgD09McunoKUsWjxwRIA==/base.apk!classes28.dex] (okhttp3.internal.http2.Http2Connection$ReaderRunnable.invoke+28)
2022-08-18 11:00:40.104 A/x: runtime.cc:642]   native: #24 pc<PHONE_NUMBER>0c7870  [anon:dalvik-classes9.dex extracted in memory from /data/app/x.x.x-ogXgD09McunoKUsWjxwRIA==/base.apk!classes9.dex] (com.tuya.smart.mqttclient.mqttv3.internal.wire.MqttInputStream.readMqttWireMessage+28)
2022-08-18 11:00:40.106 A/x: runtime.cc:642]   native: #11 pc<PHONE_NUMBER>1cd8ac  /data/app/x.x.x-ogXgD09McunoKUsWjxwRIA==/base.apk!libgltfio-jni.so (offset 8b39000) (???)
2022-08-18 11:00:40.107 A/libc: Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 10639 (x), pid 10639 (x)
2022-08-18 11:00:40.369 A/DEBUG: pid: 10639, tid: 10639, name: x  >>> x.x.x <<<
2022-08-18 11:00:40.373 A/DEBUG:       #09 pc<PHONE_NUMBER>1cd8ac  /data/app/x.x.x-ogXgD09McunoKUsWjxwRIA==/base.apk!libgltfio-jni.so (offset 0x8b39000) (BuildId: 7c1475039d0c8e2871bf4edb5b52944b24045663)

Desktop (please complete the following information):

OS: Ubuntu
GPU: NVIDIA 470
Backend: [OpenGL/Vulkan]

Smartphone (please complete the following information):

Device: SM-G9650
OS: Android 10

Additional context
filament version is 1.25.5

Please show the full logs.

logs.txt

The actual issue is:
2022-08-18 11:33:39.741 A/plusminusrhyth: java_vm_ext.cc:578] JNI DETECTED ERROR IN APPLICATION: JNI GetFieldID called with pending exception java.lang.NoSuchFieldError: no "Z" field "doubleSided" in class "Lcom/google/android/filament/gltfio/MaterialProvider$MaterialKey;" or its superclasses

@prideout Looks like MaterialKey is missing the annotations to preserve the field names.

I have everything marked to keep Filament,
-keep class com.google.filament.** { *; }

So, can you do R8 obfuscation directly in ARR? If you can, then there is no need to obfuscate again on the Android side.

No you don't obfuscate the aar. But the aar can provide rules to avoid obfuscating some of its APIs.

I just ran into this exact issue with version 1.31.3 - the same stacktrace as in the other comment:
2022-08-18 11:33:39.741 A/plusminusrhyth: java_vm_ext.cc:578] JNI DETECTED ERROR IN APPLICATION: JNI GetFieldID called with pending exception java.lang.NoSuchFieldError: no "Z" field "doubleSided" in class "Lcom/google/android/filament/gltfio/MaterialProvider$MaterialKey;" or its superclasses

Looking into the fix - #5954 - it appears that the existing proguard rules only keep constructor for the annotated classes. Members that must be kept need to be annotated on their own in addition to the class.
-keep @com.google.android.filament.proguard.UsedBy* class * {
  <init>();
}
-keepclassmembers class * {
  @com.google.android.filament.proguard.UsedBy* *;
}

That explains why the crash is still happening - MaterialKey members are not annotated with @UsedByNative.

Thanks! I'll fix this asap.

Done!


⚡ LibBit Optimizations

Description
⚡
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[x] Ran forge snapshot?
[x] Ran npm run lint?
[x] Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

Close first. May not be always better if code is inlined.


[DOCS]

Issue Description
I was scrolling through the Examples of Valid Commit Messages subsection of the Contributing Guideand noticed that some of the examples are missing backticks(see screenshot for reference). This causes contributors, especially those who are new to open source, to be confused.
Screenshots or Examples (if applicable)

Proposed Solution (optional)
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[ ] I am willing to work on this issue (optional)
[ ] I am participating in Hacktoberfest 2023

Can I work on this? @Anmol-Baranwal @CBID2

Can I work on this? @Anmol-Baranwal @CBID2

I would have assigned you, but 0scvr has already created a PR for this. I've asked him to comment to get assigned.
If he doesn't reply within next six hours, then I will assign you.

alright

Please assign me 😀

Please assign me 😀

You're assigned @0scvr

I want to work on it assign me!

I want to work on it assign me!

Sorry @shivam200446. It's already assigned


User management action redirect fails in Keycloak 26

I have problems after upgrading from Keycloak 25 to Keycloak 26.
When I click on the cancel or submit buttons of a user management action like UPDATE_PASSWORD or UPDATE_PROFILE I get redirected to my application but I see a white page.
This redirect has some parameters. It looks like in Keycloak 26 a new parameter was added. This is the URL I get redirected to:
https://my-application.com?oidc-spa_config_hash=some-value&oidc-spa_result_omit=some-value&oidc-spa_intent=some-value&state=some-value&session_state=some-value&iss=some-value&kc_action=UPDATE_PROFILE&kc_action_status=success&code=some-value
If I compare this with an old redirect URL I can see that the parameter kc_action=UPDATE_PROFILE was added. When I remove this parameter manually the page works as expected.

Fixed in<EMAIL_ADDRESS>Thanks again for reporting

I can confirm: v5.4.4 fixes the problem.
Thanks for the fast fix!


ERROR: Communication socket unexpectedly closed (libssh)

Hi,
I'm using Netopeer2 server with my onos client. When i connected to the client I'm getting error below.
VERBOSE: Accepted a connection on <IP_ADDRESS>:830.
[2018/07/25 09:29:31.190078, 1] ssh_server_connection_callback:  SSH client banner: SSH-2.0-SSHD-UNKNOWN
[2018/07/25 09:29:31.190100, 1] ssh_analyze_banner:  Analyzing banner: SSH-2.0-SSHD-UNKNOWN
[2018/07/25 09:29:31.196641, 2] ssh_server_ecdh_init:  SSH_MSG_KEXDH_REPLY sent
[2018/07/25 09:29:31.196702, 2] ssh_server_ecdh_init:  SSH_MSG_NEWKEYS sent
[2018/07/25 09:29:31.204195, 2] ssh_packet_newkeys:  Received SSH_MSG_NEWKEYS
VERBOSE: Received an SSH message "request-service" of subtype "ssh-userauth".
VERBOSE: Received an SSH message "request-auth" of subtype "none".
VERBOSE: Received an SSH message "request-auth" of subtype "publickey".
VERBOSE: User "sdn" tried to use an unknown (unauthorized) public key.
VERBOSE: Failed user "sdn" authentication attempt (#1).
VERBOSE: Received an SSH message "request-auth" of subtype "interactive".
[2018/07/25 09:29:31.311094, 2] ssh_message_auth_interactive_request:  Warning: Got a keyboard-interactive response but it seems we didn't send the request.
VERBOSE: Failed user "sdn" authentication attempt (#2).
VERBOSE: Received an SSH message "request-auth" of subtype "interactive".
VERBOSE: User "sdn" authenticated.
VERBOSE: Received an SSH message "request-auth" of subtype "interactive".
ERROR: User "sdn" authenticated, but requested another authentication.
VERBOSE: Received an SSH message "request-channel-open" of subtype "session".
[2018/07/25 09:29:31.316603, 1] ssh_socket_exception_callback:  Socket exception callback: 1 (0)
[2018/07/25 09:29:31.316628, 1] ssh_socket_exception_callback:  Socket error: disconnected
ERROR: Communication socket unexpectedly closed (libssh).

Hi,
most likely a duplicate of #299.
Regards,
Michal


fix(bun): Rewrite TextEncoderStream polyfill implementation for Bun middleware

Overview

Closes #5929
The implementation of bun adapter implemented in #5129 has also introduced a polyfill for TextEncoderStream. While the adapter itself works, this polyfill introduced a bug, caused on attempt to close a response stream (see #5929 for more info). This PR fixes this issue by rewriting the polyfill based on this comment: https://github.com/QwikDev/qwik/issues/5929#issuecomment-2108951045
What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

The PR itself is ready, but it need to be tested anyway. I will do it manually, since I am not much familiar with the codebase. Please, let me know if I can add some automated tests.

this should be fine as is. bun should include TextEncoderStream anyways

Alright then.
Prettier fails on index.ts in bun middleware.

@octet-stream I'll fix the pr don't worry about it

Thanks @octet-stream for your help :rocket:


Multiple WriteValueAsync() results in stuttering data transmission

I use WriteValueAsync() to write data without response to an BLE device.
Therefor
gattCharacteristic.WriteValueAsync(dataWriter.DetachBuffer(), GattWriteOption.WriteWithoutResponse))
is called ~200 times in ~1 seconds to write data as fast as possible.
On receiving BLE device I can see now ~2-5 write-without-response messages arriving in a "block" with nearly no time delay between them, then I see a break without new receives with a length of  ~1-2 seconds before next "message-block" is received.
So data, which was written in ~1s, takes up to 90 seconds to arrive completely on BLE device (probably buffered from Windows).
Does anyone has an idea, why breaks are there and how to get rid of them?
Using a smartphone app, which does the same data transfer with write-without-response messages to same BLE device, transfers data much more quickly than Windows does.

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er wird für die Verknüpfung von learn.microsoft.com zum GitHub-Issue benötigt.

ID: 90524148-a1cb-13ad-922b-7557f64abc00
Version Independent ID: 05adef7b-05c2-ad83-f476-ae7f4970b57b
Content: GattCharacteristic.WriteValueAsync Methode (Windows.Devices.Bluetooth.GenericAttributeProfile) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Devices.Bluetooth.GenericAttributeProfile/GattCharacteristic.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @ChriDit, and thanks for filing this issue. The feedback channel you're using here (Submit and view feedback for > This page) is for feedback about the documentation in this topic itself, as published. For example, if the content is incorrect or incomplete.
But this particular issue doesn't feel like it's saying that something needs changing in the docs.
For questions (or requests for help) regarding a specific scenario, like this, one option is to click Submit and view feedback for > This product, and you'll be taken to Microsoft Q&A (more info about that below).
Here are all of the developer support resources, for completeness.

Microsoft Q&A is the right place to lobby for a change or a new feature, to make a suggestion, or to ask a product question.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

I'll go ahead and close this issue now, since there's no action to take in the documentation. You can always reply to a closed issue, or even re-open it if you feel it's been closed in error.
Thanks!
-Steve


Implement click selection of cities

This is '1st visit - idea A,' where all cities display as gray dots. In this mode, selecting a city will highlight that city as well as its family (if applicable). Should also make highlights in the GDP and pop histograms.

We've decided to not implement this.


Support zero-dim results in split_axis

We can support that split_axis method returns zero-size array.

LGTM!


java.lang.IndexOutOfBoundsException

I tried to run this:
bds atac.bds -species Vc -nth 16 -fastq1_1 "Rep1_R1.fastq.gz" -fastq1_2 "Rep1_R2.fastq.gz" -fastq2_1 "Rep2_R1.fastq.gz" -fastq2_2 "Rep2_R2.fastq.gz" -fastq3_1 "Rep3_R1.fastq.gz" -fastq3_2 "Rep3_R2.fastq.gz" -enable_idr
I got this error:
Picked up _JAVA_OPTIONS: -Xms256M -Xmx728M -XX:ParallelGCThreads=1
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 18, Size: 18
at java.util.ArrayList.rangeCheck(ArrayList.java:653)
at java.util.ArrayList.get(ArrayList.java:429)
at org.bds.BdsParseArgs.setVarInit(BdsParseArgs.java:245)
at org.bds.BdsParseArgs.initializeArgs(BdsParseArgs.java:138)
at org.bds.BdsParseArgs.initializeArgs(BdsParseArgs.java:106)
at org.bds.BdsParseArgs.parse(BdsParseArgs.java:172)
at org.bds.Bds.runCompile(Bds.java:872)
at org.bds.Bds.run(Bds.java:815)
at org.bds.Bds.main(Bds.java:182)
if I remove the -nth 16 and add -no_par the pipeline would run, but at the at certain points would use the maximum number of threads, which is problematic, because there are other people using the same server.  In addition, I noticed from the report.html that Bowtie2 is using only one thread, which I guess is the reason is taking forever to run (bowtie2    -X2000 --mm --local --threads 1), is there a way to change this? As you can see I am not versatile in the cluster/parallelization, I would welcome any recommendation from you.
Our lab is using a server with CentOS Linux 7.4.1708 (Core) , rhel fedora.
Thanks

Please post a full log.
-nth 16 means total number of threads for the pipeline. You have 3 replicates so 16 threads are distributed to these threads according to the size of fastqs. You can check number of threads for other replicates they will sum up to 16.


Copy vim syntax support from sourceforge

Copied from https://svn.code.sf.net/p/quattor/svn/trunk/util/vim/
Needs updates to support latest pan features, but this is just a base to work on (see quattor/release#112).

Have I misread something or is this adding the files at the top level? (Including a README which applies only to the syntax plugin). Shouldn't this go under a new directory called utils or syntax?

I'm attempting to preserve the original history and forgot to commit the move to a subdirectory :tired_face:

LGTM


bugfix: remove deconstruction for pageProps

getInitialProps in Page will stop working if you deconstruction pageProps.

Good catch, thanks!


fixing the 'unknown.host' test in ClientRequestExecutorPoolTest

it turns out that somebody has registered 'unknown.host':
$ host unknown.host
unknown.host has address <IP_ADDRESS>
unknown.host mail is handled by 10 serpens.uberspace.de.

which is kinda of annoying. Have switched to using unknown.invalid as
this is reserved and should never resolve.

I pulled your branch and manually merged it, to keep the version history clean. Thanks for the contribution.


extVisionSample limited covariance

PX4 accepts ROS nav_msgs/Odometry messages and distributes them as vehicle_visual_odometry. The Odometry messages provide covariance for position, velocity, angular velocity and rotation. EKF uses most of the provided information with one exception. Out of the rotation covariance only yaw variance is used. What is the rationale behind this? Also extVisionSample seems to disregard position-rotation covariance as well, why is that?

We only use the yaw information from the attitude. We use position-velocity covariance, but yaw is used as an independent observation in a different part of the code due to historical reasons.


Reduced Planck constant should be italic

In CairoMakie, when I do
ax.xlabel = L"\hbar\omega"

I get this

which is not correct as \hbar should be italic like this:

Is there any fix or workaround for this?

Thanks for the fix!!

Planck's constant is a fixed numerical quantity since 2019. Shouldn't it be upright as variables are italic and constant upright, according to the ISO norms?

I'm trying to follow TeX convention as they are the most used. So the upright hbar should be available through \mathrm{\hbar}.

yeah, for quantum theory papers upright looks awful. Good to have it available as upright, but italic is used universally


gltfpack: floating point attributes for normals

Currently, positions can use floating point attributes with:
-vpf: use floating point attributes for positions

But normals are always quantized. I'm seeing little artifacts in the model with normal attributes which are not present in uncompressed model. When I reduced the quantization bits from default (8) to 4, the artifacts became severe => Artifacts are due to quantization.
I would love a flag to use floating point attributes for normals. Like below:
-vnf: use floating point attributes for normals

I would recommend -vn 12 instead.

I have tried with -vn 16 and still seeing artifacts.

If you're still seeing them with -vn 16 then I don't think they are normal-related, so I doubt floating-point normals would improve this -- or maybe there's some issue with -vn. Do you have a model you can share?

Do you have a model you can share?

Yes. Models are quite large: https://drive.google.com/drive/folders/1lkT9YFSFEx-MgRC-G8YAaGlb3GDlz9Ow?usp=drive_link I also added a imgur post link so you can directly see the models without having to download them. I used https://gltf-viewer.donmccurdy.com to view models.
Let me know if you have viewed the models and I'll make the folder private,

Thanks - feel free to delete the files, I'll need to debug this later.

This is a similar problem to #515 -- the model uses morph target based animation by just serializing keyframes as morph targets, and currently the range of normals is always [-1, 1] when quantized, so significant normal deformations see visible artifacts regardless of the number of bits used. On that model the animations are less aggressive so the issue is mitigated by clamping, but here clamping isn't sufficient. Using floating point normals would fix this; there may or may not be better alternatives though, we'll see...


项目要求最低版本是16，插件要求最低是19，我改了插件版本放本地，还是报错。求楼主解答，感谢感谢

/Users/dongfang/Documents/isoftstone/gwadar-app/gwadarpro/android/app/src/main/AndroidManifest.xml Error:
uses-sdk:minSdkVersion 16 cannot be smaller than version 19 declared in library [com.github.shaoshuai904:RecordWav:1.0.2] /Users/dongfang/.gradle/caches/transforms-2/files-2.1/a5c9de0c28bd84fc04e0951d313ffc99/RecordWav-1.0.2/AndroidManifest.xml as the library might be using APIs not available in 16
Suggestion: use a compatible library with a minSdk of at most 16,
or increase this project's minSdk version to at least 19,
or use tools:overrideLibrary="com.maple.recorder" to force usage (may lead to runtime failures)
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugManifest'.

Manifest merger failed : uses-sdk:minSdkVersion 16 cannot be smaller than version 19 declared in library [com.github.shaoshuai904:RecordWav:1.0.2] /Users/dongfang/.gradle/caches/transforms-2/files-2.1/a5c9de0c28bd84fc04e0951d313ffc99/RecordWav-1.0.2/AndroidManifest.xml as the library might be using APIs not available in 16
Suggestion: use a compatible library with a minSdk of at most 16,
or increase this project's minSdk version to at least 19,
or use tools:overrideLibrary="com.maple.recorder" to force usage (may lead to runtime failures)

项目中Android目录下添加


Remove duplicate random queries check

Addresses #216

I had to adapt the Merkle batch proving code in order to account for the possibility of duplicate indices. I spent some time thinking about the best way to do so without changing the current algorithm for batch proving and I couldn't think of anything better than just deduplicate at the outset.
Let me know if you see a better way to do so.

Superseded by #224.


Route consts vs magic strings

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1664
Originally created by @derek-knox on 2020-02-03 22:23:56

Create SSOT for route strings. Initial concern https://gitlab.com/meltano/meltano/-/merge_requests/1410#note_281646601
Possibly consts to obj vs. just string and possibly both

View 1 previous comment from the original issue on GitLab


Temporary small fix on gftools-fix-weightclass.py

I need this script to also fix weight class in OTF italic instances, so I made a raw and simple fix. Although it is probably not the cleanest way to do it.
A nice update would be:

[ ] work for style name build as: weight + any other style (italic, condensed…)
[ ] specify these values are for otf
[ ] add accurate values for ttf and variable

Thanks for this. I've made the changes in #223 so I'll close this,.


Archive civil-rights-complaints repo past issues

Description
@UchMok made this repo public and wants to ensure the past issues we had in this repo are archived and exported out of the repo, in case there was anything sensitive..
Definition of done

[x] @alexsoble to add exported, closed issues to this Drive folder
[ ] @UchMok to delete closed issues from repo
[ ]

Attachments
title

Exported issues JSON added to Drive under Archive folder as civil-rights-complaints-issues-export.json.

@UchMok Upon closer inspection, the export data I prepared earlier was incorrect — it was an export of open issues, not closed ones.
I fixed the export query and wrote a script to handle GitHub's API pagination. The new, correct data export is uploaded in the Archive folder as closed-civil-rights-complaints-issues-export.json.

@UchMok That said, when you have a minute, please double-check the data export to make sure it's what we want. The best way to double check is to look for the "title" and "body" fields in the JSON file.


Remove old / stale remote branches

In step 1,
rosbookibm-8028:carbon-tutorial mattrosno$ git fetch upstream
From github.com:carbon-design-system/carbon-tutorial
 * [new branch]      hallOfFame                               -> upstream/hallOfFame
 * [new branch]      master                                   -> upstream/master
 * [new branch]      react-step-1                             -> upstream/react-step-1
 * [new branch]      react-step-2                             -> upstream/react-step-2
 * [new branch]      react-step-3                             -> upstream/react-step-3
 * [new branch]      react-step-4                             -> upstream/react-step-4
 * [new branch]      react-step-5                             -> upstream/react-step-5
 * [new branch]      react-step-6                             -> upstream/react-step-6
 * [new branch]      revert-4597-add-tests-step-3             -> upstream/revert-4597-add-tests-step-3
 * [new branch]      revert-4600-revert-4597-add-tests-step-3 -> upstream/revert-4600-revert-4597-add-tests-step-3
 * [new branch]      revert-4685-tw15egan-patch-2             -> upstream/revert-4685-tw15egan-patch-2
 * [new branch]      tw15egan-patch-1                         -> upstream/tw15egan-patch-1
 * [new branch]      tw15egan-patch-2                         -> upstream/tw15egan-patch-2

After you merge in this round of tutorial fixes, can you make sure that there's only master and react-step-1 through -6 remote branches? The rest are unnecessary noise for tutorial takers.

CC @alisonjoseph @tw15egan

Updated


Path Support

Is it possible to support path for proxy?
for instance a.com/s1 and a.com/s2 route to different backends?
PS. Path header in h2 and h3 is :path header

Thanks for bringing this up.
Routing based on path is certainly doable. We haven't had a need for it yet. That's why we haven't added it.
Is this something you would use? Could you describe your use case a little bit more?

Let's describe our goal and our use case.
Our objective is to develop an easy-to-use, secure, and open-source proxy server and client that supports a wide range of protocols while safeguarding them from active probing.
Imagine our proxy server is hosted at a.com. Under typical circumstances, if the Great Firewall of China (GFW) makes a request to a.com, it could potentially conduct active probing on our proxy. To counter this, our strategy involves concealing the proxy behind a secret path, /proxy, on the website. As all requests are encrypted using HTTPS, the GFW is unable to detect this hidden path, rendering its active probing attempts ineffective. This approach ensures that our proxy remains secure and inaccessible to such probing attempts.
in addition, since we have several protocols, like grpc, h2, http1.1, we need to be able to sperate them by path
for example
a.com/proxy/grpc -->backend1
a.com/proxy/h2 -->backend2
a.com/proxy/h1 -->backend3

I see. Thanks for the details.
Adding support for path routing isn't the main issue we'd need to solve to support your use case. We'd need to add support for proxying http/2 requests first (and perhaps http/3 as well based https://github.com/c2FmZQ/tlsproxy/issues/29). GRPC uses http/2 for transport. So, it should just work if http/2 was supported.
So far, we only have support for http/1 for http reverse proxy. Path routing can be added to http/1 relatively easily.
Adding support for http/2 is a larger project. Is this something you'd be interested in contributing to?

It is my pleasure to contribute to this great project. I am in hard period now but in one month i can manage for that

Adding support for HTTP/2 was easier than I expected. I think https://github.com/c2FmZQ/tlsproxy/pull/31 should take care of it.

That is amazing 😍

In exploring potential optimizations for this system, I've considered a new approach. Currently, you handle h2 requests in tlsproxy, followed by initiating another h2 request to the backend. My proposition is to streamline this process by directly routing these requests to the backend (after SSL termination). This method would eliminate the redundancy of decoding and encoding h2/http1.1 twice. We could achieve this more efficiently by simply overlaying the proxy protocol on the existing packets. Do you think whether it is possible or not?

If I understand correctly, your suggestion is to parse the incoming HTTP request, decide where it should be rerouted, and the REPLAY it somehow?
This is more or less what mode: TLS or mode: TCP do, without actually parsing anything other than the TLS ClientHello.
With mode: HTTPS and mode: HTTP, tlsproxy acts as a http reverse proxy. It needs to modify the requests to add the x-forwarded-for and via headers, perhaps send redirects and set cookies when user authentication is enabled, etc.

You are right. That is exactly what i mean.
Thanks

Right. So, I don't think the REPLAY idea will work in the context of an http reverse proxy.

I think if we do not alter any packets it should work. I mean similar to sni proxy that we read some packets and based on the sni we forward it to a backend, similar thing should be possible in http, read first bytes of a packet and decides the backend. but i agree that it is not the priority now.

Path support is available in v0.2.0


Add resource for provisioning SSO

Similar to #72, it would be nice to be able to configure Integration > SSO Authentication via Terraform.

Would love to see this happen as well. For both  SAML and OIDC. ( But OIDC prefered )

Any news?


Remove use of ruby2_keywords

Remove the use of ruby2_keywords to create an initialize method that is better behaved with explicit keyword arguments

Any chance we can approve the actions workflows here?

I fixed the Rubocop linting issues. The jruby tests are failing but at a bundle install step which must be independent of this change.

I don't follow. What does "better behave" mean?

@flash-gordon the use of ruby2_keywords is something of a hack to help legacy code continue to work in the old way where the last positional arg could be used to pass keyword args. As of Ruby 3 keywords args must be passed explicitly.
The specific problem is that before this fix if you introspect the constructor you get a misleading answer:
require 'dry-initializer'

class User
  extend Dry::Initializer

  param  :name,  proc(&:to_s)
  param  :role,  default: proc { 'customer' }
  option :admin, default: proc { false }
  option :phone, optional: true
end

User.instance_method(:initialize).parameters #=> [[:rest, :args]]

After this change
User.instance_method(:initialize).parameters #=> [[:rest, :args], [:keyrest, :kwargs]]

I'll add a test that makes that clear.
I also realize that for consistency we should have a better signature if the dry class doesn't accept positional args

It would be great if the Initializer signature represented the state of the params. Ie if the dry-initialize doesn't include positional params the create an initialize method with just **kwargs similarly only *args for only positional args.

Any chance we can get 👀 on this?

A release would be awesome

3.1.0 is out


Cannot update the storage in tests

I update a value in storage in tests, and then when I read it, the entry in storage is not changed.
let code: Vec<u8> = vec![1, 2, 3];
println!("> Storage before:\n{:?}", storage::unhashed::get_raw(well_known_keys::CODE));
println!("wasm code: {:?}", code);
storage::unhashed::put_raw(well_known_keys::CODE, &code);
println!("< Storage after:\n{:?}", storage::unhashed::get_raw(well_known_keys::CODE));

A full code to reproduce this issue:
https://gist.github.com/siman/a86efbdc91aae04fe736abe0ba2595e9

Hey @siman, after looking into the issue it is not a bug.
Looking here you see that for the key CODE there is a special treatment in tests. You can test by replacing the key with codesiman or something like that and your test will work :)

@bkchr I think I need CODE key because I'm trying to test a logic that updates runtime code in storage. Then what is the best way to test that code runtime code has been updated based after some condition met?

@siman I will prepare a pr to make it possible to alter the CODE. I will mention you in the pr.


Custom domain problems

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

Go to https://gov.wagumi.xyz
It doesn't to Wagumi dao snapshot page (https://snapshot.org/#/wagumi.eth/)

Expected behavior
A clear and concise description of what you expected to happen.

Redirect to wagumi dao snapshot page (https://snapshot.org/#/wagumi.eth/)

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Followed all the steps in https://docs.snapshot.org/spaces/add-custom-domain

There is one more PR that needs to be merged, which was merged - should work fine now


addListener() support

would be nie to add that.
described here: http://dbaron.org/log/20110422-matchMedia
could be done with a timer loop and leaving the elems in the page..

I just add a pull request to response.js - If you combine this with response.js you get listener support for about an extra 200 bytes min+gzipped.


Labels().values as numpy arrays should wrap ExternalCPUArrays

Just as TensorBlock values are, when the Labels(...).values are a numpy array, they should wrap an external CPU array. This would prevent the following behaviour attributed to random memory access when the labels object is not stored in a variable and therefore has no reference:
import numpy as np
import equistore
from equistore import Labels

vals = [0, 1, 2, 3, 4, 5]
Labels(names=["_"], values=np.array(vals).reshape(-1, 1)).values

>>> array([[          0],
       [          0],
       [-2085138416],
       [          2],
       [          2]], dtype=int32)

where instead the correct values are only returned if instead the Labels object is assigned to a variable first:
labels = Labels(names=["_"], values=np.array(vals).reshape(-1, 1))
labels.values
>>>array([[0],
       [1],
       [2],
       [3],
       [4]], dtype=int32)

Just to clarify: this is not a random memory access but a use after free: the Labels allocate some memory, and the values use a pointer to this memory. When doing values = Labels(...).values, the Labels are immediately freed by Python, and the memory released, but the values do not know that, and contain a pointer to freed memory.
ExternalCpuArray, defined here
https://github.com/lab-cosmo/equistore/blob/master/python/equistore-core/equistore/core/data/extract.py#L120
solves this by keeping a reference to the parent (here the Labels)  alive, preventing Python from freeing the corresponding memory.


feat(eco-content-menu): retrieve configurable deep attributes

Add a better way to configure attribute path from component props using lodash

Or instead of adding individual changes, another alternative could be to add the official lodash-babel plugin https://github.com/lodash/babel-plugin-lodash

Thanks @ChristianMurphy that's a great purpose, I pushed a commit for a such change. :angel:


feature: Add deployer options as an argument

Draft PR for an initial proposal for fixing #85.
I've only implemented from argument since other options doesn't seem so relevant as a custom deploy address.

Thanks for giving this a shot @pedrobranco!
Let's see, there's several transactions for each deployProxy or upgradeProxy function call:

Implementation contract deployment
Admin contract deployment
Proxy contract deployment
Upgrade function call

In your PR you seem to only have used the from parameter for the first two. Why not also for the proxy?
For the function call it doesn't make as much sense because we should infer the sender from the admin contract (though we don't do this yet).

In your PR you seem to only have used the from parameter for the first two. Why not also for the proxy?

True, didn't noticed it. The main goal here is to replace the default account when deploying, so for all of those transactions still seems fine.

For the function call it doesn't make as much sense because we should infer the sender from the admin contract (though we don't do this yet).

Should we do it in this PR?

BTW, tests are failing with:
Error: while migrating AdminUpgradeabilityProxy: Invalid number of parameters for "undefined". Got 4 expected 3!

And by reverting the corresponding this change also fails in upgradeProxy:
Error: Invalid number of parameters for "upgrade". Got 3 expected 2!
    at upgradeProxy (<REDACTED>/openzeppelin-upgrades/packages/plugin-truffle/src/upgrade-proxy.ts:84:15)

Seems that isn't allowing to pass the Web3 options. Correct me if I'm wrong but it seems like an issue already reported here: https://github.com/trufflesuite/truffle/issues/3506

@frangio can you help clarify these issues?

@pedrobranco Sorry about the delay!
There does seem to be an underlying issue with Truffle. It's closer to https://github.com/trufflesuite/truffle/issues/2843 than the one you shared. But our situation is also a little bit different, see my comment https://github.com/trufflesuite/truffle/issues/2843#issuecomment-739083982.
The issue is we are sending an empty object and Truffle doesn't recognize that it's meant to be a params object and so sends it as part of the constructor arguments.
We will have to avoid sending the params argument when we don't have any actual params to send.

I'm not sure when it will be released (later this week?) but Truffle has now fixed the problem you ran into in this PR.
https://github.com/trufflesuite/truffle/pull/3586

This branch is outdated now and would likely need to be rewritten.
This issue is being tracked in https://github.com/OpenZeppelin/openzeppelin-upgrades/issues/85.


Added os module for keys and "movie not found" message_text

Added import os for token and api_key. Removed import json as it was unused. Also added a special message_text for when a movie is not found. Removed print() statements.

Minor Updates


feat: Make OffTargetDetector.filter() generic

Closes #18

@tfenne not yet but PEP 718 is tracking that language feature:
https://peps.python.org/pep-0718/
If we do get this syntax, it could arrive as soon as Py3.13 (but likely Py3.14).
https://devguide.python.org/versions/


Unable to import into child tables correctly

1. Describe the bug
Using the D365BC Connector inside Power Automate, users are unable to import into a "child" table while referencing the "parent" table, because the parent table name is a primary key inside the child table. Primary Keys cannot be imported into, but that is a requirement for basic functionality of child tables.
2. To Reproduce
Steps to reproduce the behavior:

Create a custom API for a "child" table. Examples: Price List Lines, Production BOM Lines, or Routing Lines.
Go to Power Automate
Create a Flow using the D365BC Connector
Select "Create a Record"
Click Advanced Parameters and scroll through the list of available fields.

AL Code:
![price list AL](https://github.com/microsoft/AL/assets/108375368/839da567-ccbf-49e8-b732-2c6031bf7bf3)

Power Automate:
![power automate code](https://github.com/microsoft/AL/assets/108375368/45d2f198-2009-4321-ba61-c15ff8c3ff81)

D365 Instance:
![price list code missing](https://github.com/microsoft/AL/assets/108375368/c577bcb5-a844-46d0-b739-606666309861)

Note: Because the developers need to copy and paste the code snippet, including a code snippet as a media file (i.e. .gif) is not sufficient.
--> I was instructed to submit an issue in this repository by Microsoft Support (David Oyebode). This issue is less of an issue of AL Code and more an issue of the D365 Connector, which I don't have the code for. If there's another GitHub repository to state this issue, please advise.
3. Expected behavior

Create a custom API for a child table.
Power Automate Flows would allow a primary key through the Advanced Parameters menu (or, at least one main primary key in order to associate it with a parent record),
If the Primary Key is not imported into correctly, the flow would fail with an error message something like "Unknown Parent Record"

4. Actual behavior

Create a custom API for a child table
Power Automate, connect to API
Primary Key fields unavailable in advanced parameters menu
Power Automate successfully imports into a child table, with no relationship to a parent table.

5. Versions:

AL Language: 2024 release wave 1, Version 13.0.

Visual Studio Code: Version: Version: 1.86.2 (Universal)
Commit: 903b1e9d8990623e3d7da1df3d33db3e42d80eda
Date: 2024-02-13T19:42:13.651Z (1 mo ago)
Electron: 27.2.3
ElectronBuildId: 26908389
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.2.0

Business Central:
Version: US Business Central 23.5 (Platform 23.0.17191.0 + Application 23.5.16502.16589)

List of Visual Studio Code extensions that you have installed: AL Language Extension, AL Toolbox, AL Object Designer, AZ AL Dev Tools

Final Checklist
Please remember to do the following:

[x] Search the issue repository to ensure you are reporting a new issue
[x] Reproduce the issue after disabling all extensions except the AL Language extension
[x] Simplify your code around the issue to better isolate the problem

Hi,
Unfortunately CSS was wrong in this case. We cannot handle this issue through this repository since it is not an issue with the AL language or the compiler. I've raised the issue of CSS wrongly referring to this repository internally so it doesn't happen again.
For this issue, it needs to go to the team that handles the Power Automate connector. Usually the correct way for this to happen is either through an IcM if it's time sensitive and impacting production, or a RFH (request for hotfix) if it's behaviour that needs to be patched. CSS should be able to help you with either of this.
Sorry again that we can't help you through this channel even though you were referred here.
Thanks,
Thaddeus


Error: lookup hi-amenu-sh.lcl.host: no such host

Are there any additional details you would like to share?

Command: anchor lcl
Executable: /opt/homebrew/bin/anchor
Version: 0.0.37 (darwin/arm64) Commit: 1cd0aa11dbb338760b0ff26abf1f469d6ae5b1a9 BuildDate: 2024-05-28T21:30:13Z
Arguments: []
Flags: []
Timestamp: 2024-06-23T02:33:27.098574Z
Stdout:
| Let's set up lcl.host HTTPS in your local development environment!
|
| lcl.host (made by the team at Anchor) adds HTTPS in a fast and totally free way
| to local applications & services.
# Setup lcl.host HTTPS Local Development Environment `anchor lcl`
    | Once setup finishes, you'll have a secure context in your browsers and local
    | system so you can use HTTPS locally.
# Audit lcl.host HTTPS Local Development Environment `anchor lcl audit`
    | We'll begin by checking your system to determine what you need for your setup.
    - Checked resources on Anchor.dev: need to provision resources.
    - Compared local and expected CA certificates: need to install 2 missing certificates.
# Configure System for lcl.host HTTPS Local Development `anchor lcl config`
    | Before issuing HTTPS certificates for your local applications, we need to
    | configure your browsers and OS to trust your personal certificates.
    |
    | We'll start a local diagnostic web server to guide you through the process.
    - Entered hi-amenu-sh.lcl.host domain for lcl.host diagnostic certificate.
    - Resolved hi-amenu-sh.lcl.host domain: failed!
    | The entered domain name is either invalid or cannot be resolved from
    | your machine, possibly due to rebinding protection on your DNS server.
    | Learn more here: https://lcl.host/dns-rebinding

I can see from using dig hi-amenu-sh.lcl.host that this appears to resolve to <IP_ADDRESS> as I would expect. So it seems likely that there is something in your local context that is preventing this from working (dns rebinding is a likely culprit as the message mentions near the end). There are some further details at https://lcl.host/dns-rebinding, but I'll repeat them here:

To bypass DNS rebinding protection, try using a public DNS server like <IP_ADDRESS> (Google) or <IP_ADDRESS> (CloudFlare) instead. Or add an entry for lcl.host subdomain to your /etc/hosts file (e.g. subdomain.lcl.host <IP_ADDRESS>).

Let us know if that doesn't help or you have trouble with it though and we will work through it with you.

I'm closing this since we haven't heard back, but just let us know if you still need help.


Server Crashing on Stop/Restart

Removed all my mods one by one, and found that Player Roles is causing a hard crash on my server after saving in TACS.
1.16.4, on latest release. I changed my roles.json to be what it used to be before the crashes, to no avail:
  "admin": {
    "level": 100,
    "overrides": {
      "commands": {
        ".*": "allow"
      }
    }
  },
  "staff": {
    "level": 20,
    "overrides": {
      "commands": {
        "staffchat": "allow",
        "clearchat": "allow"
      }
    }
  },
  "everyone": {
    "overrides": {
      "commands": {
        "help": "deny",
        "staffchat": "deny",
        "clearchat": "deny"
      }
    }
  }
}

Crash reports don't seem to show anything, but I can try and attach them if needed. Not super familiar with development, so just give me instructions to grab any info you need. :)

I think it was actually Spark causing the crash. I replaced your mod, and it came back up (even though it crashed before), and then Spark, and it crashed on stop. I downloaded the latest Spark build instead, and it seems to be okay for now. I couldn't really get any behaviour to repeat consistently, but had more luck with removing/readding Spark.
After multiple restarts with Player Roles after resetting everything, it seems good.

I think it was actually Spark causing the crash. I replaced your mod, and it came back up (even though it crashed before), and then Spark, and it crashed on stop. I downloaded the latest Spark build instead, and it seems to be okay for now. I couldn't really get any behaviour to repeat consistently, but had more luck with removing/readding Spark.
After multiple restarts with Player Roles after resetting everything, it seems good.


add documentation what Close and Reset are supposed to do

Reset was recently (be2992bb959db6a29d3783af04e47893cdffd3c6) added to the Stream interface, but it's not clear what the expected behavior is.
We should describe that in the interface. Furthermore, Close is also an ambiguous term, and needs an explanation as well.
For example in QUIC, Close closes the stream in the write-direction, and Reset closes the stream in read-direction.

For example in QUIC, Close closes the stream in the write-direction, and Reset closes the stream in read-direction.

This. @whyrusleeping added this method because yamux has this same behavior and it turns out we've been half-closing our streams. This often didn't make a difference but was the ultimate cause of a rather nasty bitswap bug (https://github.com/ipfs/go-ipfs/issues/3651#issuecomment-315580747).

That's great news! Having matching interfaces here will reduce a lot of friction with the QUIC integration I've been working on. I'll send you a PR later adding some documentation here.


Fix travis builds

After building, thus setting up conda. Rerun Julia with the Conda installation as its dynamic library path. This circumvents #42

Great, thanks. At least this lets us run the tests while we work on a fix for the builds.


Produce non-form pdf

I'd be interested in using pdf-fill-form for filling out a form and then saving the result as a regular, non-form pdf. The result would be the same as filling out the form by hand and then printing to a file. Converting to an image is much too heavyweight.

Any update on this?
thanks

Thank you for your interest for this library! The pdf-fill-form is based on the popular poppler pdf -library. For saving the form we use the poppler PDFConverter (https://people.freedesktop.org/~aacid/docs/qt5/classPoppler_1_1PDFConverter.html). Currently i'm not aware of it being able to convert forms to normal text. To have this feature we have 2 options:

We can contribute to poppler library this feature for the converter. I think this would be best, but hardest solution cause usually those libraries want then support all the possible cases.

We could try to utilize poppler to remove all the form fields from the pdf and replace then with TextBox elements. Input fields -> TextBox with the input field text. Multiselect -> TextBox with the selected text. Checkbox -> TextBox with "X" / "O" symbol.

For the actual implementation of any of these solutions I would need help from the community.

If I understand correctly, cairo already supports PDF surfaces, so the existing page rendering code might mostly work for rendering to "plain" PDF already.
(In createImgPdf, I see that the page size is hardcoded to A4. That seems wrong; the output page size should probably be based on the input page size so that different page sizes and orientations will automatically work.)

Closing this issue - this is now discussed more at #74


[ACR] Opt-in verifytypes

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-containerregistry

/azp run python - containerregistry - ci

/check-enforcer override


fix broken link:https://git.k8s.io/minikube/docs/drivers.md

Can you show us the related Issue link in your PR message?

@CriaHu ping

duplicate of #22121
/close


OCM-3313 | refactor: extract autoscaler code to new modules

In order to make code less repetitive when we add day2 support of cluster-autoscaler (and also to reduce the amount of code in cmd/create/cluster/cmd.go), this refactors the code to be placed in new modules that are also more easily imported by new commands.
In the step 2 of this we want to add a rosa create autoscaler command that will use the new location of the code

/test all

/hold
waiting for reviews

/cc @Gregory-Pereira @ciaranRoche @gdbranco

@osherdp Do you have a ticket and/or DDR for this change? I have some reservations about introducing a create command for something that's not a resource in our API. Would rosa edit cluster be a better command to tackle this?

Mostly moving code around, SGTM

@osherdp but considering that this is also a day-1 option, it seems more fitting for rosa create|edit cluster. Sounds like a mistake to have put it in a separate API endpoint and should be part of Cluster.AutoScaler{}?
This code change means users can no longer set it during cluster creation, so it seems we're changing/removing functionality

but considering that this is also a day-1 option, it seems more fitting for rosa create|edit cluster. Sounds like a mistake to have put it in a separate API endpoint and should be part of Cluster.AutoScaler{}?

Maybe we can incorporate that into rosa edit cluster rather than rosa create|edit autoscaler, but either way it's out of scope for this specific PR. Whatever the command we choose we prefer to have the code re-usable across the different commands (instead of repeating a lot of code)
The reason I think rosa create|edit autoscaler is more clear is that it's more consistent with the existing behavior of default machine-pool | oidc-providers. I don't think we can justify those being separate resources and autoscaler being a cluster property

This code change means users can no longer set it during cluster creation, so it seems we're changing/removing functionality

after this code change users are still able to set it during cluster creation. it's only a preparation for handling it in a day2 scenario

okay, I see your point, thanks for clarifying. So basically customers can set it during rosa create cluster as well as rosa create autoscaler if I understand correctly? If that's the case, then I'm fine with it. Just want to make sure it doesn't cause confusion to end-users. Since day-1 create cluster resources are perceived as part of the cluster spec, users will likely expect them to also be editable as part of the cluster spec (which implies edit cluster). Does that make sense?

@vkareh yeah, I get what you mean. Actually originally I also wanted to make it all part of the cluster object, but we've got into a situation where the lack of support for PUT requests in OCM made it impossible to partially override autoscaler configuration and still preserve the PATCH contract. Because requirements were to allow that for day1 and day2, we didn't have much choice and had to have it this way
In this case we partly rely on the docs making this kind of behavior clearer to users. Either way configuring cluster-autoscaler is something I consider a bit more advance, so I assume people will first turn to the docs and won't expect it to be intuitive on first sight. Hope it makes some sense :smile:

@osherdp yes! Our API unfortunately doesn't have a way to unset attributes, apart from issuing a DELETE request. Sigh...
Okay then, this all makes more sense to me. Thanks for all this context! :+1:

@ciaranRoche @gdbranco please let me know if anything else is needed
Tests are added to the validators and this should only refactor the code (we still have the next step in order to introduce the new commands)

/hold
sorry, doing one more manual test

/lgtm

/unhold
thanks everyone!


Only native libs are shown in fragment

Showing libs list is not possible when my app is with proguard enabled. When my app is in debug, all libs are shown.
What I do is:
@Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        LibsBuilder libsFragment = new LibsBuilder()
                .withFields(R.string.class.getFields());

        LibsFragment fragment = LibsCompat.fragment(libsFragment);

        if (savedInstanceState == null) {
            getFragmentManager().beginTransaction().add(android.R.id.content, fragment).commit();
        }
    }

Also I declared in proguard this:
#about libs
-keep class .R
-keep class **.R$* {
    <fields>;
}

I don't know what could be wrong.

https://github.com/mikepenz/AboutLibraries/issues/270#issuecomment-260693576

Thanks @mikepenz however I even reproduced strings like they were in R.string.class.getFields() and then loaded again them into withFields(String.. fields). But without success.
I had to do a hack.
First I found out libs defined names (working in debug).
/*Libs libs = new Libs(getActivity());
                ArrayList<Library> externLibraries = libs.prepareLibraries(getActivity(), GenericsUtil.getFields(getActivity()), null, true, true);
                for (Library lib : externLibraries) {
                    Log.d(TAG, lib.getDefinedName());
                }*/

Then I added that names into withLibraries method.
new LibsBuilder()
                        .withActivityStyle(Libs.ActivityStyle.LIGHT_DARK_TOOLBAR)
                        .withActivityTitle("Licencje")
                        .withAutoDetect(false)
                        .withLibraries(
                                "AboutLibraries",
                                "AndroidIconics",
                                "fastadapter",
                                "gson",
                                "LeakCanary",
                                "OkHttp",
                                "okio",
                                "appcompat_v7",
                                "design",
                                "recyclerview_v7",
                                "support_v4"
                        )
                        .start(getActivity());

@mmokrzycki oh yes I missed this part (as most people just have proguard without obfuscating) the problem is auto detection either works via the xml or via a class file which it tries to find. As those class files have a different name after proguard there is no solution to automatically detect those libs, rather than just defining their identifiers.
In general the whole list can be found here on github.


base toss

WHAT IS THIS ?

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


Refactor/kotlinx.serialization

Removed Gson and instead added kotlinx.serialization

Does kotlinx.serialization have a better performance than Gson?
Are there some benchmarks?
Or why do you want to switch at all?

In general I think if kotlin provides a standard way of serialisation and deserialisation, it should be used over a third party library like Gson.
According to benchmarks linked in this issue the scores are at least similar if not better than Gson.

Improvement: Add an example for custom (de-)serialization (https://github.com/Kotlin/kotlinx.serialization/blob/master/docs/custom_serializers.md) i.e. Dates


Please invite me to the community

Name
smart dev
Additional Context
invite me

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for developing solutions and enhancing creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn about development, DevOps, tech, and contribute together and with us 👉💜👈


CDAP-15821 Whole-file-ingest plugin validation

JIRA: https://issues.cask.co/browse/CDAP-15821
WIKI: https://wiki.cask.co/display/CE/Guidelines+on+Plugin+Validation
In scope of this PR:

integrated with new validation API

@albertshau could you please take a look


Skeleton for vendor HCI commands

We've been asking for doc updates when we update flashing support.


fix: OPTIC-79: Add queueTotal to lsfProperties from project

Hi @Travis1282!
This PR was created in response to a PRs in upstream repo:

https://github.com/HumanSignal/dm2/pull/234
https://github.com/HumanSignal/label-studio-frontend/pull/1554

/git merge develop

/frontend build

/frontend build

/frontend build lsf


Always get a 404 in the browser.

Hey,
I have setup a VPS with the 2 docker images, both are running.. Now, when I connect with localtunnel, I get a link (Your url is: my-tunnel.domain.com) but when I try to access it, I get a blank page with just 404 in it.. Even if I try to access it directly over port 3000.
Does anybody know how to fix this?

Fixed it by disabling the cloudflare DNS proxy...


The Plugin LABEL and TAINT does not work

sealer version
{"gitVersion":"v0.8.5","gitCommit":"f9c3d99","buildDate":"2022-04-28 14:16:58","goVersion":"go1.16.15","compiler":"gc","platform":"linux/amd64"}

My Clusterfile:
---
apiVersion: sealer.aliyun.com/v1alpha1
kind: Plugin
metadata:
  name: MyLabel
spec:
  type: LABEL
  action: PostInstall
  data: |
    <IP_ADDRESS> node-role.kubernetes.io/worker=

---
apiVersion: sealer.aliyun.com/v1alpha1
kind: Plugin
metadata:
  name: taint
spec:
  type: TAINT
  action: PreGuest
  data: |
    <IP_ADDRESS> node-role.kubernetes.io/master:NoSchedule-
    <IP_ADDRESS> node-role.kubernetes.io/master:NoSchedule-
    <IP_ADDRESS> node-role.kubernetes.io/master:NoSchedule-

Could you paste your installation log here? @jackk01 There would be some plugin applying information.

@bxy4543 Shall we release the next version soon? @VinceCui @bxy4543

@jackk01 , pls try the latest version v0.8.6 to see if it fixed.