SAFETY RULES for Gemini.

I've been trying so many times, even tried to add my own, but I am still getting errors about safety, tf

Huh, that's weird. Gemini is a very recent addition, have you tried with OpenAI or local version yet?

I found one more, can you try:
[
      {
        category: HarmCategory.HARM_CATEGORY_DANGEROUS_CONTENT,
        threshold: HarmBlockThreshold.BLOCK_NONE
      },
      {
        category: HarmCategory.HARM_CATEGORY_HARASSMENT,
        threshold: HarmBlockThreshold.BLOCK_NONE
      },
      {
        category: HarmCategory.HARM_CATEGORY_HATE_SPEECH,
        threshold: HarmBlockThreshold.BLOCK_NONE
      },
      {
        category: HarmCategory.HARM_CATEGORY_SEXUALLY_EXPLICIT,
        threshold: HarmBlockThreshold.BLOCK_NONE
      },
      {
        category: HarmCategory.HARM_CATEGORY_UNSPECIFIED,
        threshold: HarmBlockThreshold.BLOCK_NONE
      }
    ]

If you have some example code snippets that would result in safety errors, I'd be happy to test against them. It would help to resolve the issue.

@jehna @Manouchehri
asd.txt
I put here whole file, if it's possible put output. I've only 3 requests on ChatGPT, and then syntax and safety on Gemini all the time, so if u have maybe output of this thing I'm trying to deobfuscate since I remember, and only your script is making me 60% max, feel free to put output, keep me updated if there's something wrong ;)

Bump! 😊

Thanks for the file! I'll check on it later – unfortunately something came up, so I don't have time to work on humanify for a little while. I'll get back to you a bit later on this 👍

Thannks for respond now, good luck 😊

Sorry for another interrupt, but have to ask, any ideas? 😄

Maybe killing children processes?

Maybe killing children processes?

Things that makes sense in programming but could be badly interpreted by non-programmers


Tic tac toe game using React js

I have created an amazing tic tac toe game using React js with some cool animation and designing. I want to contribute in this repo and add my work in this repository.

Do i have to change something? I dont know you closed this issue.

Try to be more innovative with your project idea as we already have many projects like this, also we already have a tic tac toe game on the repo.


dispatch_io_net: fall back to fork() if posix_spawnp() is not available

Android only started supporting posix_spawnp() in API level 28 (Android 9), so
it is not going to be available on most devices. If the system doesn't provide
it, then fall back to using fork() and execve().

cc @MadCoder @compnerd

@swift-ci please test

@swift-ci please test

@MadCoder  - okay to merge?

I updated this to also handle the posix_spawnp() call in bsdtestharness and to error-check execve(). Please re-test.
cc @MadCoder @compnerd @ktopley-apple

@swift-ci please test

Rebased

@swift-ci please test

Thanks for the explanation. I'll merge this when the build has completed.


Ensure no missing values in coordinates

I know this should really be done in xarray, but I've been using the xbitinfo .to_compressed_netcdf() and .to_compressed_zarr() convenience functions even when I'm not using bitrounding because... they are convenient!  :)
CF conventions don't allow missing values in coordinates but xarray produces them nonetheless.  I usually remove them via this little code snippet:
for index in ds.indexes:
    ds[index].encoding['_FillValue'] = None

Would there be any downside to adding this to to_compressed_netcdf  so users don't have to?

Hi @rsignell-usgs,
Sorry for the slow response! Glad you like the convenience functions. Part of why I haven't responded yet, is that I'm not a big fan in implementing functions that should be done at a different place. However, the convenience functions could be an exception as they wrap stuff anyway. Just to be clear here, your suggestion does not get rid of missing coordinates, but rather gets rid of the _FillValue attribute so it looks like no coordinate is missing, am I right?


Full specification for all ASC alerts is missing

Hi
I am implementing a solution requiring details for many of the alerts generated by Security Centre defined here:
https://docs.microsoft.com/en-us/azure/security-center/alerts-reference
I only find one example schema in JSON format on this page - where can I find the specification for the other alerts please?

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

ID: 91ab6818-9668-8b2f-2c70-409f62292a9b
Version Independent ID: 7ea06d89-fe9b-2619-527f-1864ad78a86b
Content: Schemas for the Azure Security Center alerts
Content Source: articles/security-center/alerts-schemas.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@colindomoney
Thanks for your feedback! We will investigate and update as appropriate.

Hi @colindomoney, thanks for the question.
We have the schemas on their own page which you can find by looking for "Schema" in the search box:

Use the tabs on that page for the relevant schema for your scenario. If you want more information on consuming Azure Defender's alerts in a different tool, please comment here or open another ticket.
I hope this helped.
#please-close

Hi - I'm aware of that page with a sample schema that gives me the format of the alert; however what I am unable to find is the actual values of fields within this alert for specific events here: https://docs.microsoft.com/en-us/azure/security-center/alerts-reference#alerts-windows. For example for "A logon from a malicious IP has been detected" what are the values in the schema for "AlertType", "AlertDisplayName", "Description"? Without this, it is difficult to implement a response handler for automated ticketing and response.


Added codeline.rihal.om as domain

https://codeline.om/
https://www.linkedin.com/company/codeline-om/
https://www.linkedin.com/groups/9501436/

Hello @CodelineGitHub,
I am sorry, I have to close the request without merging, because this organization is a training center, not a college, nor a university. Please ensure that you are familiar with this repository rules located at the bottom of this page.
Please apply for an appropriate licensing option (which may also be free) at  https://www.jetbrains.com/community/education/#courses

Hi @nataershova, We offer free of cost Bootcamps for students who are job seekers and help them placed on jobs. We are non-commercial and are doing this with the support of many software companies. It would have been great if you have given us a chance of clarification on this matter before closing out PR.

Hi @CodelineGitHub,
Thanks for getting back to us. Usually, we add to the swot domains of the official educational institutions like high schools and universities with courses more than 1 year long. Unfortunately, we can't add schools with short-term courses.
For such schools and training centers, we have a special program. Within this program, we provide coupons for courses up to 6  months. You may apply for the coupons via this form
Please let me know if I'm mistaken and if you have 1-year courses. Please send us a link to such a course.

def show_menu():
print("--- Welcome to the Menu-Based Program! ---")
print("Please select an option:")
print("1. Print Pattern")
print("2. Rotate Array")
print("3. Help")
print("4. Exit")
def print_pattern(n):
for i in range(n, 0, -1):
print('*' * i)
def rotate_array(arr, k):
return arr[-k:] + arr[:-k]
def display_help():
print("--- Help ---")
print("Option 1: Print a pattern with 'n' rows of decreasing asterisks.")
print("Option 2: Rotate an array of 'n' elements to the right by 'k' steps.")
print("Option 3: Display this help message.")
print("Option 4: Exit the program.")
Main program
while True:
show_menu()
choice = int(input("Option: "))
if choice == 1:
    n = int(input("Enter the number of rows for the pattern: "))
    print("Output:")
    print_pattern(n)
elif choice == 2:
    n = int(input("Enter the number of elements (n): "))
    k = int(input("Enter the number of steps (k): "))
    elements = input("Enter the array elements separated by spaces: ").split()
    arr = [int(num) for num in elements]
    rotated_array = rotate_array(arr, k)
    print("Output:", rotated_array)
elif choice == 3:
    display_help()
elif choice == 4:
    print("Exiting the program. Goodbye!")
    break
else:
    print("Invalid option. Please try again.\n")


Rename clearUpdateTag to updateRelations

Hello!
The name clearUpdateTag=True/False parameter that can be given to Skeleton.toDB() is misleading. Renaming this to updateRelations would be more understandable, so that updateRelations=True is the default, updateRelations=False can be set if not relation update should be invoked.

Fixed decades ago by #688


Chapter 6 curl request issue

In Chapter 6 the curl request after listing 6-15 doesn't work. It returns the error
{"items":["This field is required."]}

Never mind, I had the wrong code in serializers.py


[eas-cli][ENG-6325] provide suggestions for developers when archive size is large

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
https://linear.app/expo/issue/ENG-6325/provide-suggestionsguidance-in-eas-cli-when-developers-have-very-large
How
When archive size is over 100 MB, show a warning containing a link to the documentation about .easignore
Test Plan
Test locally

/changelog-entry new-feature Provide suggestions for developers when archive size is large


Support ROS2 control

In trying to support ROS2 control, we found the following problems.

The ign_ros2_control plugin does not support different node names because the name robot_state_publisher is hard-coded.

I will try another plug-in to see if I can work around this problem.
Bridge the control commands to the JointTrajectoryController or JointVelocityController and JointPositionController plug-ins to control the joint angles.

Currently, namespaces are used to distinguish between the information on the ignition gazebo and the robot.
When we tried to apply ros2_control to this environment, we found that the controller_manager's spawner node does not support the namespace and we could not make it work.
This will be changed to a namespace setting that allows ros2_control to be activated.
After the configuration change, we will re-open this issue if ros2_control may be supported.


add js frontend

@starpav we are trying to keep this project lightweight. It should not have any complex projects as NFTs, etc. Also, it will be replaced with more comprehensive examples in the future.


Add a post-link script to fetch the raw data files

Add a post-link script to fetch the raw data files from a GitHub repository and download them to a specific location on the host machine.
These changes are to experiment the feasibility of removing the dependency on the conda package tudat-resources for fetching the input data files.
The existing dependency of tudat and tudatpy on tudat-resources is not removed. A different target location than .tudat/resource is specified to verify the successful execution of the post-link script.
Closes #9

Let´s see what it does :)

Huh, looks like we didn't even need to merge it. Just openeing the pull request started a run on azure. I think we have @geoffreygarrett and/or @gaffarelj tot thank for this :) I know I have some catching up to do on the automation that has been set up here.


Juay Hee 1.4 DG Update

Updated DG with implementation details and diagrams for:

Delete Person Tag Feature
Add Task Tag Feature
Delete Task Tag Feature
Sort Date Feature

Moved SortTaskByDateCommand class into .../logic/commands/task from .../logic/commands/person

Codecov Report

Merging #238 (7fdc5dd) into master (d30f807) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #238   +/-   ##
=========================================
  Coverage     71.06%   71.06%
  Complexity      920      920
=========================================
  Files           145      145
  Lines          3048     3048
  Branches        359      359
=========================================
  Hits           2166     2166
  Misses          768      768
  Partials        114      114

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
69.56% <ø> (ø)

...ain/java/seedu/address/logic/commands/Command.java
100.00% <ø> (ø)

...ddress/logic/commands/person/AddPersonCommand.java
100.00% <ø> (ø)

...ess/logic/commands/person/AddPersonTagCommand.java
78.78% <ø> (ø)

...ess/logic/commands/person/DeletePersonCommand.java
93.54% <ø> (ø)

.../logic/commands/person/DeletePersonTagCommand.java
76.66% <ø> (ø)

...dress/logic/commands/person/EditPersonCommand.java
94.73% <ø> (ø)

...ess/logic/commands/person/SortStrengthCommand.java
100.00% <ø> (ø)

...ess/logic/commands/person/SortWeaknessCommand.java
100.00% <ø> (ø)

...c/commands/person/notecommands/AddMiscCommand.java
100.00% <ø> (ø)

... and 21 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Gallery loading time

I have the following problems with the image picker plugin in Android:
Very low performance while opening the gallery. Took a very long time to open the gallery images.
The gallery is sorted from the oldest to the newest pictures. But I need to list the image from newest to oldest.
When a picture is chosen, it is not marked and not able to get image details.
The screen turns into white when we scroll the pictures in the gallery.
The plugin is working fine in the newest phones. I'm facing these issues in Samsung s6 edge with Android version 7.0
How can I solve these issues? Is there any workaround to fix those issues.
Thanks in advance.

My customers are also facing this problem on Android. When scrolling, the page turns white because loading takes a long time. Also after selecting a few images the screen suddenly jumps to halfway the list...


webui job detail broken

When loading some job detail on a local webui, such as http://<IP_ADDRESS>:2021/JobDetail/f81ffa06-8e97-4ce8-8c66-f12cd3378b26 the page fails to load, and the console shows it is attempting to load http://<IP_ADDRESS>:2021/JobDetail/assets/index-DtYsxs3q.js.  Removing JobDetail from that URL loads the asset.

See also #3170.


Query for random row(s)

Problem
In Prisma 1 there was conversation around potentially including a way to query for a random row(s) in the database.  See https://github.com/prisma/prisma1/issues/114.  I think this should still be considered in Prisma 2 (and don't want the feature request to get lost in the transition to this new repo).
Without this feature, in order to do random lookup of the table you would first need make a call for the count of the number of rows in a table.  Then create a random number bounded by that count.  And make another call to find that row.  I think prisma has the potential to make this feature usable and performant.
Suggested solution
In the linked issue (https://github.com/prisma/prisma1/issues/114) the discussion lead to utilizing the orderBy with a random option seems to make the most sense.  But, maybe that random option should only be available to id fields as to not make it confusing.
Also, how this is solved in the Engine I'm less sure.  As each database can solve this differently and probably some queries are more/less performant than others.
Alternatives
Would love some more conversation around this.
Additional context
Again think prisma is well poised to make this a usable performant feature.  Because doing it in sql is very database dependent and very non-performant if you aren't careful.  See this discussion: https://stackoverflow.com/questions/19412/how-to-request-a-random-row-in-sql

Would really benefit of this feature in Prisma 2. currently have a use case and the only way to achieve that is through a raw query.

I would love to see this feature aswell.
Going to have to use a raw query for now probably.

+1 for this :)

+1

+1

+1

Any update?
Please consider my suggestion: https://github.com/prisma/prisma/issues/25257


open har file error with charles

Steps to reproduce the problem:

I use mitmdump and har_dump.py to convert the file to har,but when Iopen it with charles,it,s error,
MalformedJson:Use JsonReader.setLenient(true) to accept malformed JSON

Mitmproxy: 3.0.3
Python:    3.5.2
OpenSSL:   OpenSSL 1.1.0g  2 Nov 2017
Platform:  Linux-2.6.32-431.el6.x86_64-x86_64-with-centos-6.5-Final

the har_dump.py is
https://github.com/mitmproxy/mitmproxy/blob/master/examples/complex/har_dump.py
Any other comments? What have you tried so far?
System information

the dump file is as follow link
https://send.firefox.com/download/3f8be846da/#UzMW099RgcYaG-J2uDbUkg

Hi,
I created a HAR file using har_dump.py and loaded it successfully by Charles Proxy.
Also the dump file link is expired, if you provide a new link I will test with that as well and revert.

the dump file can be download as follow:
http://pan.zhangwei.wiki/dump

dump.zip

@kira0204 the file has upload in 4th floor! please help me!

Hi @coderzhw, sorry for the slow response. I converted your flows to a har file using har_dump.py, loaded it successfully with Charles. The HAR file can be found here https://files.fm/u/6592asxe. If your HAR file differ from this, please share the file with us so that we might understand what is the problem.

@kira0204  your har file can be open with charles ,can you tell me whice version of your mitproxy and what command you use tou convert?
mitmdump -ns har_dump.py -r srcfile -w dstfile.har
is this command right?

I have solved it ,thank you !


Collect all errors during transaction construction

 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.i.a.finochenko seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

ISSUE https://github.com/zio/zio-dynamodb/issues/105


Discord Webhook for Personal Server

Configure repository webhook for personal server.

There will be some spamming here...

OK, it seems this now works.


fix(searchbar): better rtl suport

Short description of what this resolves:
fix position of icons and other ones in searchbar when the direction is rtl
Ionic Version: 1.x / 2.x / 3.x
Fixes: a bit part of #5035

Looks great, except for no default behavior.
If a user does not have dir, it will break his app's searchbars

Thanks for the PR! This is looking good, but I have some questions / possible issues:

Could you remove [dir="ltr"] from the rules? This should just be the default so that it will still work if no direction is specified.
The animation goes a bit crazy on iOS:

Shouldn't Search just switch sides with the icon and stay centered when it isn't focused? It is aligning all of the way right. Also, shouldn't the icon be on the right side when it is aligned to the right?

LTR:

RTL:

You all know more than me about this so please let me know if I am wrong on any of this. The above were a result of me running gulp e2e.watch --f=searchbar.


scripts: gen_relocate_app: use its own cmake property name...

...instead of piggyback onto COMPILE_DEFINITIONS. This fixes an issue where an add_compile_definitions() would result in the compiler definition being appended to the relocation file list. This is because COMPILE_DEFINITIONS is populated by cmake to contain all compiler definitions of a target (think -D being passed to compiler). So by piggyback onto COMPILE_DEFINITIONS for the relocation file list, any of those definitions would be in the file list. To avoid this, we need to have a separate named property.
Fixes #60220

Doesn't work...


Can't compile base.h with memory leak detection turned on.

To find memory leaks from the 'new' operator, Microsoft suggests to redefine it through the preprocessor using this:
`
#ifdef _DEBUG
#ifndef DBG_NEW
  #define DBG_NEW new ( _NORMAL_BLOCK , __FILE__ , __LINE__ )

  #define new DBG_NEW

#endif
#endif  // _DEBUG
`
However, that brakes the compiling of base.h on line 2310:
	void * operator new(size_t) = delete;
with the errors: C2059 (syntax error 'constant'), C2091 (function returns function) and C2802 (static member operator 'new' has no formal parameters).
I surrounded that line on base.h with #ifndef _DEBUG, to remove it from compilation and everything seems to be working fine.
However I know I'll get into trouble. Does anyone know what dangers lie ahead? Is there any safer workaround on this?

That is a terrible recommendation and I’ll get the MSDN folks to remove it. It is non-standard and ill-advised to redefine C++ keywords with macros.
The new keyword is used in a host of expressions, particularly in modern C++ that would not be compatible with such preprocessor hacking.

Thank you Kenny.
I'll disable the MSDN hack and see what they suggest instead.

MSDN has been updated.

Nice!
Thank you Kenny.


Can't find the generated pdf file

Short and descriptive title

I download the wkhtmltopdf and install successfully at my laptop
I add the bin file to my PATH
install the plug-in package for sublime
write an html file and use wkhtmltopdf convert to pdf
nothing happened and I can't find the generated pdf

Expected behavior
no idea
Actual behavior
no idea
Environment
wkhtmltopdf:
    version: 0.12.5-1

wkhtmltopdf package:
    version: ?

Sublime Text:
    channel: stable
    version: 3176 (portable)
    platform: Windows
    architecture: x64

The generated PDF files can be found in the same folder as the original HTML files.


requirement: west support

in twister v1 we can pass the west settings like below
--west-runner=pyocd --west-flash="--board-id=022900000582988000000000000000000000000097969905"
we need add this support in twister west plugin

Thanks for pointing this out. We are aware that we still miss multiple flags/functionality and already made an issue collecting this https://github.com/zephyrproject-rtos/twister/issues/135. The flags you are reporting are at 21. Since many of those are small tasks, we decided to keep them in one issue for now. Creating 21 new issues would also likely flood the backlog and possibly be distracting. In addition #135 puts in one place all the missing flags, so it is easier to see how much is already ported and what is left.
I will close this issue as duplicate of #135. We plan to create required issues based on the list whenever a task became active, to track assignments. This one can be reopened as well.
Next time please check if there is no opened issue already, or if the feature is not in #135.  If you'd like to work on something from the list, give a comment there and a task will be made. If you see something is missing and not on the list, please give a comment under #135 and we will update the list

duplicate of https://github.com/zephyrproject-rtos/twister/issues/135


Added brotli_enc and brotli_dec library targets to third party bazel build

This doesn't build because of some warnings, and we're pretty much in super-strict mode.
This change LGTM, I'll look into

See if we can apply --copt -Wno-implicit-fallthrough to the build of just this depedency
Add the bazel build to travis CI


Minor API tweaks for the next PyPI release

Our core API is now solid (otherwise we wouldn't claim to be version 1.0!) but there are still a few corners of the code where parameter names can be improved and made clearer. We want to fix these names as soon as possible, before too many people (including us!) become reliant on them.
Here is the current list of changes I would personally like to see.

Plot.show_xaxis -> Plot.xaxis  : Sounds like a boolean when in fact it is a list of options. Also shorter.
Plot.show_yaxis -> Plot.yaxis  : Same as above.

Please add any other suggestions you may have or entirely new suggestions below! In addition,  I have submitted a feature request to param that would help make minor API changes easier:  https://github.com/ioam/param/issues/112

Sounds like something we could easily doe for version 1.0.3. Thoughts?

No one has replied so I might go ahead and make these changes before release this week. I am also considering if the figure options e.g figure_size should be shortened down to simple fig_size.

Yes, the show_ options are very confusing, and should be fixed.
I also vote for changing "sublabel_format" to just "sublabel", so that it is more easily turned off.
I've no opinion on fig_size, except that if there's a corresponding option in matplotlib with similar semantics, it's good to match the name, and if there's a corresponding option in matplotlib with different semantics, it's good not to match the name.

Ideally we'd go through all the plot options one by one and rename them more sensibly. They've mostly just accumulated without working out consistent naming schemes.

If there's a corresponding option in matplotlib with different semantics, it's good not to match the name.

I agree it is good not to match the name in such circumstances but I don't think it should stop us from using the best name we can think of (whether matplotlib has used it or not).
Matplotlib is only one possible backend and we would do out best to keep the basic plotting class API consistent for new backends. The matplotlib API is generally bad (inherited from Matlab after all!) which means we can pick whatever names we think are most suitable.

Sure. Comparisons with matplotlib are just a tie-breaker -- if there is no overwhelming reason to choose a specific variant of the name (which is common), then what I said is "good" above can help decide.
Yes, going through the plot options one by one is a great idea.  You probably don't have time before Thursday, though.

As you said we don't have time to go through all of them before this release. I just renamed show_xaxis -> xaxis and show_yaxis -> yaxis, since they were clearly misnamed and were rarely used anyway. We should keep this issue open to continue discussion on sensible naming of plot options.

Ok, good.  I wouldn't say those options are rarely used, though; they are used twice in holoviews/doc/Tutorials/Options.ipynb, and also in imagen/doc/index.ipynb, and imagen/doc/Tutorials/hegde_cc07.ipynb.  So those notebooks will need to be updated, and so will the .zip file of notebooks for HoloViews and for ImaGen.  That's a bit annoying, because it means that the .zip file for ImaGen will require a very specific version of HoloViews.  But it's better to make such fixes now than later...

This name isn't great either:
xticker = param.ClassSelector(default=None, class_=ticker.Locator, ...)

In matplotlib.ticker you have both tick formatters and tick locators. The name 'xticker' is therefore rather ambiguous and something like 'xticklocator' semantically clearer (although a lot more awkward).

The xticker, yticker and `zticker`` options are being removed for 1.3.0. I think we can close this issue as the API changes described here have been implemented.
In future, we can simply use the "API" tag for any issues regarding the API.


BrightSign: Monitoring-only incl. screenshots via built-in API

Hey @thefennecdeer (and @scroix),
I've had a simple BrightSign monitoring recipe "in my drawer" for a while which does some basic monitoring incl. Thumbnail snapshotting.
There were a few reasons I hesitated publishing it, one being I wasn't sure if there was any impact on the players taking screenshots, hence why I made them trigger on demand only. It might not be recommended having them continually take snapshotting in case they write to disk (memory card) every time, if that's what they actually do.
Also, I ran out of time to implement authentication. It only works when no authentication has been set.
If either one of you wants to make use of those code and to consolidate it into the new one you posted @thefennecdeer , please feel free.
I don't get to work with BrightSigns enough to be able to test these with different models.

I had no idea there was a built-in API. That's super valuable. I thought writing a companion Brightscript was the only method.


implement RFC 10: OARec virtual collections

Overview
Implements RFC 10: OGC API - Records virtual collections support.
Related Issue / Discussion
None
Additional Information
Contributions and Licensing
(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
[x] I have already previously agreed to the pycsw Contributions and Licensing Guidelines

Waiting for PSC to approve RFC 10 to merge


How can i get the swiper instance outside of the component?

I want to use the methods of instance outside of the component, but it seems no way to get the swiper instance in documentation and examples.
issue #20 didn't work for me, init has no argument of swiper
react-id-swiper version: 2.3.2

I had found the getSwiper method in ReactIdSwiper.tsx.
so i closed the issue.

I had found the getSwiper method in ReactIdSwiper.tsx.
so i closed the issue.

May I ask how to deal with getSwiper?

A simple solution to this:
`const [swiper, setSwiper] = React.useState(null);
let goNext = () => swiper.slideNext();
let goPrev = () => {swiper.slidePrev();
React.useEffect(() => {
var mySwiper = document.querySelector(".swiper-container").swiper;
setSwiper(mySwiper);
}, [swiper]);

`


Remove Effect

Describe the problem:
Dear Compose developer, I am an ink screen（similar to kindle） APP developer, I hope that the click effect can be removed globally,Because it affects refresh performance, I hope to get your help.
Describe the solution
Remove Effect via Theme?

@tmxdyf, for your information, the click effect in Compose is called an “indication” and the LocalIndication composition local holds an Indication object that draws the indication.
I can't test it right now, but you should be able to remove that effect by using an Indication object that draws no indication, as follows:
CompositionLocalProvider(
    LocalIndication provides NoIndication
) {
    // Contents
}

What's left now is the NoIndication object. Internally, Compose has a NoIndication object that unfortunately is not exposed — you can find it here.
You may want to just copy-paste it, for as crappy a solution as it may seem. Luckily it is very short:
private object NoIndication : Indication {
    private object NoIndicationInstance : IndicationInstance {
        override fun ContentDrawScope.drawIndication() {
            drawContent()
        }
    }

    @Composable
    override fun rememberUpdatedInstance(interactionSource: InteractionSource): IndicationInstance {
        return NoIndicationInstance
    }
}

@DavideCannizzo Thank you very much for your kind reply! Good luck.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Fix building with standalone sqlite3 >= 3.30.0

Problem
When using SQLite.swift/standalone with sqlite3 pod version >= 3.30.0, attempting to build the project results in "ambiguous use" warning in Xcode pointing here.
Solution
SQLite 3.30.0 changed the definition of SQLITE_DETERMINISTIC:
-#define SQLITE_DETERMINISTIC    0x800
+#define SQLITE_DETERMINISTIC    0x000000800

Meaning that the (older) system sqlite3 library and the pod have different definitions, even though they're the same value.
We've been importing the system SQLite3 module in SQLiteObjc.h even when linking against standalone sqlite.
I added a check to SQLiteObjc.h to import the sqlite3 pod when we're using it, instead of always importing the system module.
This leads to there being only one definition in scope.

Running into the same issue. Thank you for the solution!


erronous make test outcome

It is more an annoy outcome that confuses automatic build
all tests are passed but make reports fail
qpdf-10.2.0 on Cygwin (it seems to appear between 10.0.1 and 10.0.2 )
Overall test suite                                             ... PASSED
TESTS COMPLETE.  Summary:
Total tests: 5251
Passes: 5251
Failures: 0
Unexpected Passes: 0
Expected Failures: 0
Missing Tests: 0
Extra Tests: 0
*** ERROR: make failed

There's not enough information here for me to do anything with this.


chore: remove import Mathlib.Mathport.Rename

We remove all import Mathlib.Mathport.Rename.
This is done using
find . -name "*.lean" -exec sed -i '/^import Mathlib.Mathport.Rename/d' {} +

plus two fixing by hand of files that were missing various imports.

bors d+

bors merge


epoll_wait() usage needs to handle EINTR correctly

Applying strace(1) to a bpftrace process causes bpftrace to exit prematurely. To replicate simply start any bpftrace invocation and ion another terminal apply strace to thee process and it will terminate.
This is because epoll_wait(2) returns -1 with errno set to EINTR and we don't handle this correctly in BPFtrace::poll_perf_events().  I think to fix this we will need to do set some global state in the SIGINT handler to differentiate between the ptrace based condition that strace induces and the genuine condition of a user supplied <ctrl-c>.

I've got a basic patch to fix this, although I'm holding off on submitting it for a little bit while I test.


Rename model_mode, break up KV cache impl and test Model

Closing in favor of https://github.com/google/maxtext/pull/308


IPFS configuration instructions are truncated

Having not correctly configured my daemon to work with the new web UI, I've been provided with a very helpful message telling me I need to reconfigure it. However, some of the text is truncated via CSS:

You can copy-paste the whole lot and get the full command, but it seems like this is something you should be able to see. Happy to PR something to fix this, but I suspect @olizilla will have opinions on the best way!

It fixed! https://github.com/ipfs-shipyard/ipfs-webui/issues/849


CI builds runtime has regressed recently

The issue appears to be in the "Build CLI" part of the build. I did binary search through the commit history and it seems the jump in test execution occurred with #2172 ☹

I did some digging and the longest part is IntegrationTestHelper.StartServerWithTextAsync(dataSet.Bicep, uri, resourceTypeProvider: TypeProvider, fileResolver: new FileResolver());
Perhaps we should consider starting server only once and then just opening/closing files to test.

Also another question came into my mind: is this just a problem of the Test executor or did the PR impacted VSCode add-on performance?

I haven't noticed any perf impact on the VS code extension.
@bhsubra has a hypothesis around the root cause. It seems like the snippet provider is getting initialized every time the language server is started.
@miqm I like the idea of sharing the language server instance in these tests. I think it may also help with the O# 19 issue we're seeing in my own branch as well. (Although we don't need that change in @bhsubra's idea works out.)


Add Unknown command support for ClusterClients (API and shell)

Hello, thanks for the library.
I want to invoke the command of unknown cluster, for example:
The matter bridge enable the function of scanning subdevices.
Here's what I think:

Can command-by-id be implemented? For example:
./chip-tool any command-by-id 0x130DFC00 0x02 '{"0": 180}' 1 0

After the device is added, Unknown cluster is automatically generated for easy invocation.

Hi, ok what exactly is your usecase?
Do you need this in the matter.js shell ? Yes there this option is missing.
I had the idea of "Dynamic commandline building" also but here yargs seems to do some caching so this might not work or I need to find out how to workaround this cache and make this more device specific ... So maybe that's "harder" to implement then the "by id" command
Or do you need this in controller code? Because there it is already existing and can be used.
So please be more specific what exactly you need and for which usecase.

The Matter.js shell is designed to facilitate testing devices.
Finally, it is necessary to use the controller code to achieve, convenient to give a sample code? We couldn't find the code.
In addition, I found that reading the power consumption/power of a zigbee smart plug, the value is always 0. I don't know why.
Thank you

Ok I looked into the code.
Currently the cluster clients have support for unknown attributes and event, but not for unknown commands at all, so we need to add this. I adjusted the issue name.
So the workaround for now would be to create a schema definition for the custom cluster and register this, because then it would be no "unknown" cluster anymore :-) Do you have all details for that? Else you need to wait for the above feature to be added. Would be needed to be a file like https://github.com/project-chip/matter.js/blob/main/packages/matter.js/src/cluster/definitions/OnOffCluster.ts
In general: In Controller code when you see in log that a custom cluster was added or custom attributes these are available as objects too. You could create a matter.js schema for the relevant clusters and register this then you could use it in controller code with all the typing and also tlv parsing convenience. as all the other clusters have. This is for cases where you know the structure f the cluster. For cases where this is not the case we do the best to parse the data generically. basically use ´pairedNode.getDeviceById(2)or such to get the relevant endpoint. then usedevice.getClusterClientById(0x123345). On this object you have Attribute client objects like client.unknownAttribute_0x1234` (with the hex number of the attribute id) and same-ish for events.
Re: the question why the attribute value is always 0 can be one of two cases depending on how you connected to the node in the shell
1.) if you connected with an subscription (most likely "nodes connect 111 X Y") then the system currently expects that the values are also updated via subscription and reads are executed locally. We might need to adjust this behavior anyway with Matter 1.4, but right now it is that way. I think there is no way right now in shell to define that the values are requested always remotely. I can also consider as enhancement.
2.) If you connected without subscription (so connect call without X and Y) then the read is done remotely and then this is the response from the device. Debug log would show whats up there.
And please consider posting logs and screen infos as text and not image for copy&paste convenience. Thank you.

PS: In general a full debug log of that would be interesting because also the warnings about attribute updates received for an unknown endpoint but after this you can interact with this endpoint is weird. So a full debug log of a connect and also a "nodes log" for this node would be interesting. Thank you.-

For the debug log, please see the attached document.
console.txt
logfile.txt

Thank you, in fact the connection is done with a subscription, so basically it reads locally because it expects attributes to be reported.
You can try to connect without a subscription (kind of with just "nodes connect 111") then it should just do a read* atthe beginning and all reads will be done remotely. Does that values change then?

Thank you, I will try this method tomorrow

BTW: I also considered this with Eve energy and their custom cluster. Also there energy values are not reported on subscribe which is meehh because you need to poll it which is bad in my eyes :( maybe it changes then with the official Energy cluster soon


Make Sass components self-contained to improve performance

Summary
Right now it is impossible to import just a couple of Sass components from the library to your app without anything else. Because Sass' @import rule is used in the library, when importing a specific Sass component in a consumer app you would also need to import a number of dependent Sass files. Those dependencies are different for each component and are not specified explicitly.
We need the styling components to be self-contained and independent from each other. Importing a component file would automatically bring all the dependencies (possibly not only Sass dependencies, but also CSS variables if those are used).
Ideally a consumer app would need to go an "import-all" strategy only in case it really needs all the components at once.
This will help to reduce bundle size and improve performance – a consumer app would only load the parts of the library it needs.
What alternatives have you tried?
No response
Does this feature request involve any updates to the reference design?
No
Implementation details
Every Sass component would @use all of its dependencies. Splitting some of the existing Sass dependencies (eg colors.scss) might be necessary.

This could help consuming apps to import just what they need. Eg. right now it is not possible to @use a cvi-breakpoint-up() mixin from CVI in a consuming app because that one doesn't @use variables it depends on directly but instead just assumes they are in global scope.


Model streams in HTTP/2 as child channels

Background
The HTTP/2 implementation is currently monolithic. It only has one
Http2FrameListener and that applies to all streams. It is generally necessary
to extend Http2ConnectionHandler in order to override onStreamError. This
prohibits mixing different HTTP/2 services on the same connection, something
that HTTP/2 is intended to do.
The HTTP/2 implementation is also not "normal Netty." It doesn't use the
channel for communicating incoming data or for receiving outgoing data (instead
methods). Flow control is different than isWritable and setAutoRead (instead
methods). Streams have their own attributes and those attributes are needed
frequently because there isn't another place for per-stream variables.
It seems that much of the functionality built around Channels has had to be
re-invented or will need to be re-invented in the coming future. This has made
the API of the HTTP/2 implementation large and substantially increases its
learning curve. It is also hard to use properly and there are a lot of moving
parts that many users won't want to care about. Such an API may be better for
advanced or high performance usages, though.
It also isn't possible to implement a handler (ChannelHandler, or some
specialized class) that functions with both HTTP/1 and HTTP/2, as the two have
completely different models of operation.
High-level solution
To me, it seems like using child channels would improve the situation
dramatically. The basic idea is that every time a new stream is created, a new
Channel is created for it whose parent() is the originating channel. You would
then use normal ChannelHandlers for processing the stream.
If the child channel shares the same eventLoop as the parent, it looks to add
little overhead. Reads and writes take place immediately and effectively
function the same as they do today. The main cost I see us discussing is the
object allocation necessary for creating a message to pass to the Channel.
Note that we don't need to create a message for each WINDOW_UPDATE and PING;
these sorts of frames can be handled within the HTTP/2 codec without exposing
to the application; we only need to pass headers and data.
Proof-of-concept
I have developed a proof-of-concept (#3666), built on top of the current HTTP/2 APIs.
Unfortunately, EmbeddedChannel doesn't work well because it isn't integrated
into the event loop. Additionally, LocalChannel doesn't work well because the
implementation needs to construct a LocalServerChannel, a client LocalChannel,
and a server LocalChannel per stream, with each Channel requiring the previous
to be fully constructed before its own construction. It would also require some
way of obtaining a reference to the server LocalChannel, which seems doable but
a bit icky. Although I attempted to make a reusable Channel, it appeared that
the amount of code that could be reused was relatively small and largely
uninteresting while the price consistently increased.
Thus, the ChildChannel I've created is intended only for HTTP/2's usage. It was
based on LocalChannel and you can see it is relatively simple, but it still
looks a bit generic. I was mostly deterred by doWrite() which was surprisingly
hard to implement in a generic and extremely efficient way. Even if we are able
to reuse code now, I feared that integration with flow control would prevent us
from doing useful sharing in the future.
The ChildChannel is not using the parent Channel for communicating with the
connection (other than determining the event loop to run with, which is
actually a bit broken but easy to fix). For the moment that seemed prudent as
it would perform better, and I worried that performance would be a high
concern. We can discuss whether we think we should use message passing via the
parent Channel for communicating with the connection. It would primarily mean
the ChildChannel would have to wrap or re-package any messages sent on the
child channel to include the stream id.
Mostly-appropriate flushing is already available in the implementation,
although not very well when using different event loops. If using the same
event loop, we may want to suppress any flushes during channelReadComplete
until after all the streams have been called, as we would generally expect
every stream notified in channelReadComplete to flush.
Expectations for some future features:
Flow control: Flow control will be implemented by watching the queue between
the connection and the streams.
For the inbound direction, when a frame is read it will be considered consumed,
which is basically identical to Netty/TCP/Kernel interaction today. This does
mean flow control is more coarse than the API affords today, but the normal
case will be the same. Note that setAutoRead defaults to true, so flow control
is "disabled" by default.
For the outbound direction, I have do not yet have a clear design, but we will
provide isWritable and channelWritabilityChanged. I doubt it will be best to
implement these methods by utilizing the parent's methods, as that would easily
starve streams and is harder to calculate. I have some thoughts on the subject,
but they still require investigation.
Half-close: It seems Netty has only a weak conceptual model of half-closing, so
I'm doubting exposing stream half-closing as channel half-closing. Instead, I'm
expecting to need to include closure information in the messages sent on the
stream (like is done for HTTP).
Stream-level messages sent on the child channel: I've not yet figured out if we
should re-use the HTTP/1 messages for HTTP/2. If we choose some HTTP/2-specific
messages we could still easily create an adapter that converts between the two
message types to allow having "HTTP" handlers that work for both HTTP/1 and
HTTP/2. The main issue for using the same messages is (surprise!) headers. I do
think we only need messages for headers and data. Other than END_STREAM and
similar "details," it doesn't seem unreasonable to just send ByteBufs and
Http2Headers.
To send RST_STREAM: It may involve closing the channel, but that doesn't
provide a way to give a reason for the reset. Thus, I wouldn't be surprised if
we make "close the stream" send INTERNAL_ERROR/CANCEL and then have a message
to send the other. The errors that may look useful for an application to send
are: INTERNAL_ERROR, REFUSED_STREAM, CANCEL, ENHANCE_YOUR_CALM,
INADEQUATE_SECURITY, and HTTP_1_1_REQUIRED (heh).
Providing error details from RST_STREAM or GOAWAY to a stream: I don't have too
many ideas, but we could have an event on the child channel before closing the
child channel.
Connection-level outbound things: like GO_AWAY and creating new streams, we can
have methods on our Http2 ChannelHandler or have messages be sent to the parent
channel. I would lean toward the parent channel in general as 1) every child
stream has access to the parent channel, so PUSH_PROMISE could be easy to
initiate, 2) it provides a mechanism for intercepting such messages, and 3) the
threading requirements are easy and clear.  However, creating new streams could
benefit from a method as we could return a ChannelFuture which allows accessing
Channel immediately (even before its ID has been allocated). Having to buffer
while waiting for a Channel can be quite bothersome and it would be convenient
to avoid.
Connection-level inbound things: it seems channel events would work well, at
least for GO_AWAY and priority/weight notifications.
@nmittler, @buchgr, @Scottmitch for HTTP/2 specifics
@normanmaurer, @trustin for the general approach and soundness in concept

first! @ejona86 +1

@ejona86 I really like the idea. :+1:

@ejona86 - Sounds like a good idea to me.  As you pointed out there is a risk of object allocation and associated GC costs increasing but may be a nice compromise for a lot of use cases.

@ejona86 +1 very cool!

+1
Child handlers seem to me to be the perfect level to do this at, since they support the bidirectional flow of data and back pressure, and would allow HTTP1 applications to use HTTP2 with only a small amount of modification - nesting the channel initialiser for their app another pipeline deeper.  I'm actually surprised this wasn't done from the outset.
If done right, this would allow the reuse of Netty reactive streams at all nesting levels, for example, you could have a publisher of HTTP2 stream child channels, publishers/subscribers of HTTP 2 messages that handle each of those channels, and publishers/subscribers of the bodies of HTTP requests/responses associated with each request/response.
I think it would make a lot of sense to have the http2 data objects extend the equivalent Netty HttpObject messages, such as HttpContent and HttpMessage.  It would also help to implement the same semantics in the current HTTP support where a full http request/response is one HttpMessage followed by zero to many HttpContent, where the last message in the object is a LastHttpContent (and it is possible for the first HttpMessage to also be a LastHttpContent).  This would be necessary for applications that implement proper streaming of request and response bodies (not just aggregating them into FullHttpMessage, which obviously does not work for large bodies) to be able to use the same handler for HTTP 1 and HTTP 2.
We are interested in this for Play Framework, and are willing to help both in reviewing and in implementation, where necessary.  Let me know if there's anything I can help with.

@jroper, having the HTTP/2 data objects extend the equivalent HttpObject turned out to be cause problems, almost entirely due to how headers are done differently in HTTP/2 via Http2Headers. Converting to/from the HTTP/1 headers is possible, but not free. Thus, it seemed that HTTP/2 having its own objects and only convert when an application adds a special converting handler was appropriate.

@ejona86 but would the idea of do stuff with child channels and ChannelHandler apply ? I kind of like the idea to not need to "learn" another api :)

That surprises me because as the HTTP 2 spec states, HTTP 2 only changes the wire protocol, not the semantics of HTTP messages. And it was certainly the intention of the HTTP 2 spec authors that HTTP 2 should, to existing HTTP 1.1 apps, be a transparent upgrade. If there's something wrong with Netty's HTTP 1 data modeling that prevents it from being used with HTTP 2, then that is probably something that should be addressed by fixing the data modelling, not by making a second model.
At the end of the day, HTTP 2 is going to take many years to become ubiquitous. Even if browsers support it today, there are many, many HTTP clients out there that will take years to catch up, which means most users are going to have to support both HTTP 1 and HTTP 2 for some time. Forcing most users to essentially write their HTTP server code twice is a huge usability problem, one that would call into question Netty's suitability for HTTP 2 at all. Personally, I would rather have a compatible but maybe less optimal HTTP 2 implementation in Netty 4.x, and then address the problems caused by the existing HTTP modeling in Netty 5.x, than have something that set all apps down the path of having to write their code twice.

@normanmaurer, I thought the entire point of this issue was to do stuff in child channels and use ChannelHanlders, so I didn't reply to that part. It seemed like a continuation of the +1.
@jroper, it is true that HTTP/2 only changes the wire protocol. However, Netty works with the wire protocol (like in the Matrix :-P) and tries to have very high efficiency. So in the HTTP/2 implementation there are restrictions/tweaks that make it more efficient, such as only lowercase header names are permitted.
Netty is too low-level for the HTTP/2 support to be a transparent upgrade. However, that doesn't mean that it would have to be hard. It was a goal of mine to make it possible to easily use the same handler independent of whether HTTP/1 or HTTP/2 on the wire. The converter logic for HTTP/2 to HTTP/1 and vise versa is pretty well understood in Netty right now, so I have very few concerns about implementing such things.

@ejona86 That makes sense - indeed many wire level concerns do flow into application code when using Netty.
Perhaps this deserves a new issue - but - I can see there is some existing conversion logic, however it converts to/from FullHttpMessage.  When we put Play on top of HTTP/2, for example, we won't be able to use this, since Play must not buffer the full body because Play bodies are potentially infinitely long, could represent SSE events streams, etc.  Of course, as a framework, it is not a high cost for us to implement our APIs directly on top of both the HTTP/1 and HTTP/2 APIs.  But our preference is, for anything that may be useful for more than just us, not to do something that's just specific for us, but to implement it in a way that can be used by anyone.
So our preferred approach would be to write conversion logic that supports streaming request HTTP/2 bodies in a way that's transparent to HTTP/1 users directly in Netty.  Is this already being done or do you have plans to do this?

@jroper The translation layer between HTTP/2 and HTTP/1.x is given by a combination of HttpToHttp2ConnectionHandler and InboundHttp2ToHttpAdapter.  You can see how to construct a pipeline using these in the tiles example.

@nmittler yes I saw that, but the problem is, it buffers the entire body into FullHttpMessage objects, so is not suitable for large body handling, and is unable to send bodies that don't fit in memory.

@jroper FYI this just came up in #3946

reminder https://github.com/netty/netty/pull/4503#discussion_r54444640: as part of the outbound flow controller work we should fix the interaction with ChannelOutboundBuffer so we don't have to complete promises before they are actually written in we are not in the correct event loop.

@Scottmitch, the poor interaction with ChannelOutboundBuffer is the "No premature child promise completion (in child's doWrite)" in the checklist above. Feel free to reword it :)

@ejona86 - I see. Thanks. I just added a snippet about ChannelOutboundBuffer for more context.


Remove storage from dataset query and refactor related codebase

Removes IndexingStep from DatasetQuery as this part of the codebase will not be reachable any more.
Since we use listing / indexing from DatasetQuery a lot in tests and related code, this PR needed to refactor those as well. In those places we now use DataChain.from_storage(...)
Changes in more details:

Using DataChain.from_storage() instead of DatasetQuery listing in functions: Catalog.create_dataset_from_source() and Catalog.apply_udf()
Added new argument signal_name to Catalog.apply_udf() for naming of the new signal that will be created with UDF
Refactor dataset dependency insert functions - removed add_storage_dependency etc.
Removed DatasetQuery.changed() -> this one is not used in DataChain at all so it's "dead" code. It also uses old listing columns so it won't work anyway. We can easily return this back if there will be a need
Refactoring index_tar from builtins to work with new feature file schema. Currently hardcoded to use file prefix.
Unified DatasetQuery.subtract() and DatasetQuery._subtract() -> no need to have two
Stopped using old listings in tests - refactor listed_bucket from conftest to use new listing
Refactoring test_dataset_query.py to use new listing with DataChain.from_storage() and to use new file based schema instead deprecated columns
Refactor other tests to use new listing and file based schema
Skipping dir expansion algorithm test for now as it will be moved to application level and tests will need refactoring

That's a lot of changes in a single PR, I wish it could be broken up.

I know, but the problem  was that listing from DatasetQuery was used in a lot of places (mostly tests) so needed to change it all together

*Why don't we just remove Catalog.apply_udf()? We don't actually use it.

Removed

index_tar() only exists to support some tests, couldn't we get rid of it?

I would leave it for now and remove in separate PR as I would like to add some DataChain test with tar (without this low level index_tar thing) as well, so I don't want to extend this already too big PR.

I'm rather uncomfortable with the general approach to DatasetQuery tests: we end up with a bunch of hacks to support a random mix of old- and new-style code, and I feel that they don't really test much that we actually care about any more. I think we should rather try to port them to the new-style, using DataChain and DataModel (or discard them if we already have a `DataChain equivalent), and then use the hacks if there are any left.

DatasetQuery should ideally be agnostic about our schema, but it's more handy if we use our current file based one.
Note that the goal of this PR was to remove dead indexing / listing code from DatasetQuery that has only been used in tests so tests needed some refactoring.
I agree that we should think about removing them or porting to DataChain but that can be done in separate PR.

I agree that we should think about removing them or porting to DataChain but that can be done in separate PR.

And I think that porting or removing the tests should be done first, i.e. that this should wait for #438.

I agree that we should think about removing them or porting to DataChain but that can be done in separate PR.

And I think that porting or removing the tests should be done first, i.e. that this should wait for #438.

Just when I thought I'm done with resolving conflicts :D ... Ok, I will wait for that ..


Allow package seed configurations to apply to projects that use that package

Describe the feature
It's entirely possible that this is already a feature, but I've been playing around with it for awhile / asked about it on StackOverflow and Slack, and so far I'm thinking it isn't currently doable.
I'd like to set some seed configurations in the dbt_project.yml of my package code, and have those configurations apply to all projects that use my package. Users of my package will have the ability to create their own .csvs to load with dbt seed, but I'd like to configure those seeds once at the package level. This is already possible for seed properties, but I haven't found a way to do it for seed configurations.
One specific example is configuring column length. If a package user runs dbt seed when their column has a maximum length of 25, the data will be loaded as type varchar(25). If they then add a longer string to that column, the following dbt seed fails. I'd like to specify the following in the package-level dbt_project.yml:
seeds:
  seed_name:
    +quote_columns: false
    +column_types:
          column_name: varchar(127)

I realize that for this particular particular example, the --full-refresh flag could help. However, there are cases where the inclusion of --full-refresh isn't necessarily desirable. I'd also like to be able to set +quote_columns: false and some specific post-hooks once at the package level, and have this apply to all projects that use the package.
Describe alternatives you've considered
The main alternative that I'm aware of is for package users to copy a chunk of seed configuration code into their individual project-specific dbt_project.yml files. This creates an unnecessary duplication of code across projects and also limits the package owner's ability to control those configurations.
Additional context
Because this package is for internal use only, I'm able to make some very strong assumptions about these configurations up front. But I definitely recognize that applying package-level configs to all projects should not be the default.
Who will this benefit?
I think this would benefit package maintainers and users by making it easier for package-wide configurations to be set one time.
Are you interested in contributing this feature?
I don't think I have the right expertise, but I'd certainly be interested in the possibility of contributing depending on the implementation details.

Thanks for the thoughtful writeup @emmacooperpeterson!
The short answer is: No, it isn't possible today for a dbt package to configure objects in the root project that installs it (or in a different installed package, for that matter).
 -----       ---------       --------------
| dbt | --> | package | --> | root project |
 -----       ---------       --------------

The inclusion of resources works from left to right, and so hierarchical configuration works from right-to-left: The root project can configure its own objects and anything further back/left: the objects in any installed package or the global dbt project of builtins. A package can configure its own resources, or resources in a package that it installs, but it cannot push its configurations forward/right to the root project's objects.
So by adding a configuration to your package like:
seeds:
  seed_name:
    +quote_columns: false
    +column_types:
          column_name: varchar(127)

That rightly says, All seeds in this package will not quote their columns and will treat column_name as varchar(127). But it cannot say those things about objects in other projects, which it does not know anything about.
(This is conceptually similar to how overriding macros works, though there are a few ways they differ, mostly due to the complexity of adapter-level and dispatched macros.)
At the same time, I totally appreciate your use case: I'm all for limiting duplicated code (including configuration code) where possible.  I do think it's the exception that a package author can know and say exactly how downstream projects ought to work, though: most package maintainer is about making the package as self-contained and configurable as possible, to work with whatever root project it may meet. So I think copy-paste may be your best bet here.
One potentially wacky option is to flip the script: Create a highest-level root project (really just a dbt_project.yml file), containing just these default configurations, which then installs your team's other packages and applies/overrides these configurations once and for all.
I'm going to close this issue, because I don't think this is a change we can make given the conceptual model of dbt packages today. That said, I'm happy to keep thinking here about what the right approach for your use case is, given the constructs that exist today.

This is a super helpful response, thank you! I feel like I have a better understanding now of why this works the way that it does. I do think the idea of enabling this specific type of package where the maintainer is in a position to be super opinionated about each project is an interesting one, but there are definitely workarounds. I'm working on a script that generates the necessary config files for each project given a few inputs from the user – this solves my primary concern that the seed configuration might create confusion because my target users are not familiar with dbt.
Thanks again and have a great weekend!


[Bug Report] Public activity is not only the real public activity

As you can see on my profile: https://codeberg.org/schorsch?tab=activity there is a heatmap showing my public activity. It probably looks similar to this here

but when I am logged in I can see a different heatmap because of my private contributions

I understand that my view and the public view differs because of my private repos but why is the tab called Public Activity when its also my private one?
I also dont quite understand why the heatmap does not include private contributions on the public view? Of course the actual activity should stay private but the heatmap could be updated and just like in gitlab we could say Private activity happened here

I think it should be up to the user privacy choice which heatmap view to share. On the logged in side, it would be interesting to have a toggle between different heatmap views and in the future add other filters as well.

In GitLab is configured like that: Include private contributions on my profile. If that settings is active, the private contributions are displayed with the appropriate icons — but with no details: Made a private contribution. Also the heatmap is displayed accordingly, accounting for these.
I think that approach makes a lot of sense. Would be great if Gitea could implement something similar.


Fonts not included by default

Program doesn't work as is, require to install fonts to make it work (fonts obtained from https://github.com/ThornyFFXI/gdifonts)
Also sorry if this isn't how to submit feedback/bugs, wanted to help, thanks for the addons ♥

It is included, you should be using the release link on the right not 'download as zip'.  The gdifonts lib is a submodule, so using a proper git clone will get it, the zip will not.  This is normal, end users should always look for a release when downloading from github.

Thanks!
On Thu, Oct 31, 2024 at 6:36 PM ThornyFFXI @.***> wrote:

It is included, you should be using the release link on the right not
'download as zip'. The gdifonts lib is a submodule, so using a proper git
clone will get it, the zip will not. This is normal, end users should
always look for a release when downloading from github.
—
Reply to this email directly, view it on GitHub
https://github.com/ThornyFFXI/tTimers/issues/22#issuecomment-2451130995,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMSPR3XEUQMVNSN2CF5HZH3Z6LLLFAVCNFSM6AAAAABQ7E5ALGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGEZTAOJZGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix .editorconfig

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[ ] I have provided test coverage for my change (where applicable)

Description

{s1,s2,s3} -> matches any of the strings given (separated by commas)
*.{cmd, bat, ps1} -> space after comma, it will only apply rule to *.cmd
Please see: https://github.com/editorconfig/editorconfig/issues/176

❤


Fix Sherlock Slack alerts

Sherlock Slack alerts are still going to an old URL - needs to be updated to use the current config from vault.
Traceback (most recent call last):
  File "/wrapper_runner.py", line 30, in <module>
    slack_webhook.send(settings['slack_url'], line)
  File "/slack_webhook.py", line 14, in send
    raise ValueError(
ValueError: Request to slack returned an error 404, the response is:
channel_not_found
Starting Sherlock wrapper.
2022-08-16 08:25:32,582:INFO_:wrappe

Rebuilding the docker image appears to have fixed the problem.


Create a azure-pipelines-release.yml for release

Currentlly, all the CI builds are test signed. When it's release time, we need real signed build and publish vsix file to vsix gallery. This will be release pipeline. Only is triggered manually when we are ready to release.

Seems reasonable. I'm no YAML expert, but if this produces what we're looking for, then it looks fine.

Unfortunately, it seems almost impossible to verify YAML pipeline on local machine. Have to check in and try it out.


fix(techdoc) Path Traversal Attack Fix

Hey, I just made a Pull Request!
The AWS and Google publishers had vulnerabilities that might allow a person to access content within the bucket outside of the TechDocs directory by using specially crafted URLs.
Previously the publishers would join the root path and the content path together, without checking the content path was a child path of the root, which meant if the content path contained dot-dot-slash elements it could traverse up the directory tree and access content outside of the bucket root.
This update now checks that any paths generated a child paths of the bucket root, and throws fails the request if they are.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I tried to add tests for this but the supertest request function would normalize any URLs that contained any dot-dot-slashes so by the time it hit the function, they had been removed and the URL was just an incorrect one. i.e. '/default/component/backstage/index.html/../../secret.txt' becomes '//default/component/secret.txt'.
I thought it better to get the fix in rather than spend too long trying to find a way around that at the moment


feat: add support for .mts

Without these changes, jiti is treating .mts files as non-modules. Probably should add support for .cts, too, but I don't have a test-case for it.

I will release it shortly this week (checking some other minor issue before).


BUG: Error when launching

Environment

OS: Windows 11
Python Version: 3.10.9
Hardware: RX 6600

Issue Description:
When I start it, it gives me an this error message:
Traceback (most recent call last):
  File "D:\RVC-GUI-pkg\Applio-RVC-Fork\infer-web.py", line 60, in <module>
    from lib.modules.vc.modules import VC
  File "D:\RVC-GUI-pkg\Applio-RVC-Fork\lib\modules\vc\modules.py", line 12, in <module>
    from lib.modules.infer.infer_pack.models import (
  File "D:\RVC-GUI-pkg\Applio-RVC-Fork\lib\modules\infer\infer_pack\models.py", line 12, in <module>
    from torch.nn.utils.parametrizations import spectral_norm, weight_norm
ImportError: cannot import name 'weight_norm' from 'torch.nn.utils.parametrizations' (D:\RVC-GUI-pkg\Applio-RVC-Fork\env\lib\site-packages\torch\nn\utils\parametrizations.py)

Attempted solutions
Reinstalling and basically every option in the manager

I have to add that I updated and this started happening

I just updated as well and this has been popping up for me also. I thought it was go-applio.bat at first with it no longer sporting any of the choice entry options but nada, it goes deeper than that..

Reinstalling it for a third time worked somehow

Reinstalling it for a third time worked somehow

Glad to hear that it worked for you also! After three tries though Lol


Update metabase ownership

@amureki please find someone to take over this task for you. I think @gwassing doesn't have anything yet ;)

We'll get to that topic later.


just need to update every time i need to uninstall and install again i need to migrate my project to latest one its not good

just need to update every time i need to uninstall and install again i need to migrate my project to latest one its not good can u suggest me any other way

don't do this? :)
seriously though, last two or three versions I updated by just npm install angular-cli@latest, then ng init and it worked fine
you can even skip ng init as long as there are no braking changes in the config or dependencies

o great thank @aszechlicki. you gave me a solution


Scaffolder cli

Hey, I just made a Pull Request!
 `scaffolder-cli` for scaffolding templates locally (or in pipelines for building and testing purposes) to decrease feedback loop when making changes to a template's skeleton. More info in [issue 13668](https://github.com/backstage/backstage/issues/13668).
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip @awanlin do you have more feedback? 🙏 (I'm completely new to typescript so please don't be afraid to be nit-picky about anything)

https://github.com/backstage/backstage/issues/13668

Reopening as per ask on Discord.
Linking to https://github.com/backstage/backstage/pull/24904 which may be the ultimate goal, and ongoing experimentation in https://github.com/backstage/backstage/pull/27266

A bit unsure why this started failing in packages I have not touched after rebasing - is there some dependency that has been updated on master causing this? Or do you think it's something I've done? 😄
@Rugvip @freben

Rebased.


Documentation for GraphQL Extensions

Feature Suggestion

Document how to create GraphQL extensions to the schema
Document how to share dataloaders
Document how to call the catalog from your own extension

Not sure how far along this is but for what it's worth, Apollo Federation could be a very good solution for Backstage. Happy to talk about it if you're interested.

Hey @taras, we'd ideally like to avoid federation, and have opted for an approach like https://github.com/Urigo/graphql-modules. Currently this library is implemented, but it might not stick around. We end up creating a thinner version ourselves, and the DI might be a little overkill.

Hey @benjdlambert - I'm curious to understand the pros & cons of apollo federation vs graphql-modules myself. Would this discussion happen to be in another issue or elsewhere?

@andrewthauer its something we’ve been contemplating internally so I don’t think the docs are on public GitHub. Let me see if I can get back to you with some reasoning. Happy to discuss again if something has changed too

@andrewthauer Discussion happened on Discord, but in voice, so bleh :<
Apollo federation we think works well for stitching of large schemas where things are tied in through IDs. In this case I think we're looking more at stitching together much smaller pieces, usually individual resolvers, especially things like lookups based on annotations in entities etc.
One of the main reasons would be to share data loaders and context as well. For example, I think we want to avoid hammering the catalog with requests for individual entities for every federated piece of the schema. Using modules we can fetch an entity once from the catalog, and then provide it to all child resolvers through context.

All good. More curious than anything else at this point.


Disable obsolete cypress test

In this PR, cypress tests are disabled from the camino-wallet repository.
This is because the cypress tests that are updated are those from the camino-suite repository, so these tests that are in camino-wallet become obsolete.

@Esteban100k  if we are skipping all these tests, why don't we delete them ?


JPEG compression

Is it possible to update that tool to support jpeg compression? I have read that mentioned algorithm is not well suitable for gpu processing. do you planning to add that functionality?

Hi @balbelias ! I do not intend to implement JPEG compression in RenderScript because there is an easy way to do that using the current tools:
If you are starting with an Android Bitmap object you can just do this:
FileOutputStream fos = new FileOutputStream(Environment.getExternalStorageDirectory() + "/imagename.jpeg");
myBitmap.compress(Bitmap.CompressFormat.JPEG, 100, fos);
fos.close();

If you are starting with an NV21 image, like a frame the Camera API, you can convert it to a Bitmap from it first to use the code above:
Bitmap myBitmap = Nv21Image.nv21ToBitmap(rs, nv21ByteArray, width, height);
Or just compress directly from NV21:
FileOutputStream fos = new FileOutputStream(Environment.getExternalStorageDirectory() + "/imagename.jpg");
YuvImage yuvImage = new YuvImage(nv21bytearray, ImageFormat.NV21, width, height, null);
yuvImage.compressToJpeg(new Rect(0, 0, width, height), 100, fos);
fos.close();

I also happen to think it will be hard to beat the native code that Android provides to do the compression, as they seem to use optimized C code for this.
That said, it would be interesting to see a RenderScript implementation of JPEG compression and I would gladly welcome a pull request about it to see how it compares to the JPEG compression from the Android API.

Thanks for response!
I need JPEG compression for streaming from device camera via mjpeg protocol.
Actually I found that mentioned by you direct NV21->jpeg compression is the fastest (~50ms for my settings).
However I will try to look into possible improvement using RS. I am not very familiar with it so it could take much time :)
I will let you know if there will be valuable results.


Invariant Violation: new NativeEventEmitter() requires a non-null argument., js engine: hermes

Summary
I am creating a mobile application for Android and ios using react-native this app shows customers' product and can create accounts using Facebook and Google, I use Firebase to create accounts and save data when running the application in an Android emulator works with no error but when running using the simulator for ios shows me this error:-
**ERROR Invariant Violation: new NativeEventEmitter() requires a non-null argument., js engine: hermes LOG Running "ShoofMangment" with {"rootTag":1,"initialProps":{"concurrentRoot":false}} ERROR Invariant Violation: "ShoofMangment" has not been registered. This can happen if:
Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes info Stopping server**

I using to npx react-native start --reset-cache and using to pod install and I using this library in my project

Reproducible sample code
"@firebase/app": "^0.10.15",
"@firebase/component": "^0.6.10",
"@firebase/logger": "^0.4.3",
"@firebase/util": "^1.10.1",
"@invertase/react-native-apple-authentication": "^2.4.0",
"@react-native-async-storage/async-storage": "^2.0.0",
"@react-native-community/cli": "^15.0.0",
"@react-native-community/datetimepicker": "^8.2.0",
"@react-native-firebase/app": "^20.5.0",
"@react-native-firebase/auth": "^20.5.0",
"@react-native-firebase/database": "^20.5.0",
"@react-native-firebase/firestore": "^20.5.0",
"@react-native-firebase/storage": "^20.5.0",
"@react-native-google-signin/google-signin": "latest",
"@react-navigation/native": "^6.1.18",
"@react-navigation/stack": "^6.4.1",
"firebase": "9.6.1",
"i18next": "^23.15.1",
"react": "18.3.1",
"react-i18next": "^15.0.1",
"react-native": "^0.75.3",
"react-native-animatable": "^1.4.0",
"react-native-collapsible": "^1.6.2",
"react-native-elements": "^3.4.3",
"react-native-fbsdk-next": "^13.0.0",
"react-native-geolocation-service": "^5.3.1",
"react-native-gesture-handler": "^2.19.0",
"react-native-image-picker": "^7.1.2",
"react-native-linear-gradient": "^2.8.3",
"react-native-localize": "^3.2.1",
"react-native-maps": "^1.15.6",
"react-native-permissions": "^4.1.5",
"react-native-qrcode-scanner": "^1.5.5",
"react-native-safe-area-context": "^4.11.0",
"react-native-screens": "^3.34.0",
"react-native-share": "^11.0.4",
"react-native-swiper": "^1.6.0",
"react-native-vector-icons": "^10.2.0",
"react-native-video": "^6.6.4"

Steps to reproduce
and this is my App.js file code
import React, {useEffect, useState} from 'react';
import {NavigationContainer} from '@react-navigation/native';
import AppNavigator from './src/navigation/AppNavigator';
import './assets/i18n';
import {Settings} from 'react-native-fbsdk-next';
import auth from '@react-native-firebase/auth';
import firebase from '@react-native-firebase/app';
import AsyncStorage from '@react-native-async-storage/async-storage';
import {ActivityIndicator, View} from 'react-native';
import {BottomNavigationProvider} from './src/navigation/BottomNavigationContext'; // استيراد المزود
export default function App() {
const [isLoading, setIsLoading] = useState(true);
const [initialRoute, setInitialRoute] = useState('LoginScreen'); // تحديد الشاشة الافتراضية
useEffect(() => {
// تهيئة Facebook SDK
Settings.initializeSDK();
// التحقق من حالة تسجيل الدخول باستخدام AsyncStorage و Firebase Auth
const checkLoginStatus = async () => {
  try {
    const userInfo = await AsyncStorage.getItem('user');
    console.log('userInfo data from AsyncStorage:', userInfo);

    if (userInfo) {
      console.log('Navigating to DashboardScreen');

      // إذا كان userInfo موجودًا في AsyncStorage، توجيه المستخدم إلى Dashboard
      setInitialRoute('DashboardScreen');
      setIsLoading(false);
    } else {
      console.log('Firebase User Logged in:');

      // استخدام Firebase Auth للتحقق من حالة المستخدم
      auth().onAuthStateChanged(user => {
        if (user) {
          // المستخدم مسجل دخول
          setInitialRoute('DashboardScreen');
        } else {
          console.log('No user logged in, navigating to SplashScreen');

          // المستخدم غير مسجل دخول
          setInitialRoute('SplashScreen');
        }
        setIsLoading(false);
      });
    }
  } catch (error) {
    console.error('Error checking login status:', error);
    setIsLoading(false);
    setInitialRoute('SplashScreen');
  }
};

checkLoginStatus();

}, []);
if (isLoading) {
return (
<View style={{flex: 1, justifyContent: 'center', alignItems: 'center'}}>

);
}
return (

{/* تغليف المزود هنا */}

);
}
Expected result
max@alMacBook-Air-alkhas-bMAX ShoofMangment % npx react-native start --reset-cache
info Welcome to React Native v0.75
info Starting dev server on port 8081...
                    ▒▒▓▓▓▓▒▒
                 ▒▓▓▓▒▒░░▒▒▓▓▓▒
              ▒▓▓▓▓░░░▒▒▒▒░░░▓▓▓▓▒
             ▓▓▒▒▒▓▓▓▓▓▓▓▓▓▓▓▓▒▒▒▓▓
             ▓▓░░░░░▒▓▓▓▓▓▓▒░░░░░▓▓
             ▓▓░░▓▓▒░░░▒▒░░░▒▓▒░░▓▓
             ▓▓░░▓▓▓▓▓▒▒▒▒▓▓▓▓▒░░▓▓
             ▓▓░░▓▓▓▓▓▓▓▓▓▓▓▓▓▒░░▓▓
             ▓▓▒░░▒▒▓▓▓▓▓▓▓▓▒░░░▒▓▓
              ▒▓▓▓▒░░░▒▓▓▒░░░▒▓▓▓▒
                 ▒▓▓▓▒░░░░▒▓▓▓▒
                    ▒▒▓▓▓▓▒▒

warning: the transform cache was reset.
Welcome to Metro v0.80.12
Fast - Scalable - Integrated
info React Native v0.76.0 is now available (your project is running on v0.75.3).
info Changelog: https://github.com/facebook/react-native/releases/tag/v0.76.0
info Diff: https://react-native-community.github.io/upgrade-helper/?from=0.75.3&to=0.76.0
info For more info, check out "https://reactnative.dev/docs/upgrading?os=macos".
info Dev server ready
i - run on iOS
a - run on Android
d - open Dev Menu
r - reload app
info Opening app on iOS...
info A dev server is already running for this project on port 8081.
info Found Xcode workspace "ShoofMangment.xcworkspace"
info Found booted iPhone 15 Plus
info Building (using "xcodebuild -workspace ShoofMangment.xcworkspace -configuration Debug -scheme ShoofMangment -destination id=B1356CD1-4F1F-4773-9D1A-EFD515B02EED")
success Successfully built the app
info Installing "/Users/max/Library/Developer/Xcode/DerivedData/ShoofMangment-ghyoxmovcxtgxhcuahorjetgzmjs/Build/Products/Debug-iphonesimulator/ShoofMangment.app
info Launching "com.allaith.shoof.app"
success Successfully launched the app
BUNDLE  ./index.js
ERROR  Invariant Violation: new NativeEventEmitter() requires a non-null argument., js engine: hermes
LOG  Running "ShoofMangment" with {"rootTag":1,"initialProps":{"concurrentRoot":false}}
ERROR  Invariant Violation: "ShoofMangment" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes
info Stopping server
max@alMacBook-Air-alkhas-bMAX ShoofMangment % npx react-native start --reset-cache

Actual result

React Native Maps Version
1.0.1
What platforms are you seeing the problem on?
iOS (Apple Maps)
React Native Version
^0.75.3
What version of Expo are you using?
Not using Expo
Device(s)
iPhone 15 plus
Additional information

@Ayoubdesigner101 please learn how to use  chatgpt and stop spamming here, this has nothing to do with this repo


Missing require.main

In node.js, the require.main property points to the root (or a "main) module of the process. If I run node app.js, require.main === module within the scope of the app.js file.
Why do I want this? I like to require() my main module for running integration and acceptance tests and I distinguish between a regular run and a test run based on the require.main module.
var myAppsMain = function () { ... };

if (require.main === module) {
  myAppsMain();
} else {
  exports.main = myAppsMain;
}

This makes the myAppsMain function be called when the module is run directly and exported if require()'d.

I think it's a better idea to just have separate files instead of loading up a file with different concerns like this. This would add heft to every bundle with very marginal utility, so i don't want to support it.

Understandable, thanks for the comment.

Would this be a good reason to add―perhaps optional―support for require.main?
Is there a way I could add such a require.main.require thing myself, or at least get a string to the root of the project from anywhere?


Fix Issue 20129 - AA require with File values gives "Error: return …

…expression expected" in object.d

There's a bunch more of opAssign returning void in Phobos, may be worth looking at.


Auto Calculate Missing Speed & Direction for GPS data

When the RMC nmea sentence is not recorded, there is no record of the speed or direction. However, we can estimate these values using the lat, lon, and difference in time between each data point.
Motivation: Identify moving vs stationary periods for mobile sites.

haversine: https://github.com/TheAlgorithms/Python/blob/master/geodesy/haversine_distance.py


Add flag to check for uninitialized numbers

Closes #1340.
Companion rustc PR that implements this in the Miri engine: rust-lang/rust#88670
r? @RalfJung

By the way, let me know if/when you want me to squash :)

Yeah this looks ready, so please squash. :)

@camelid could you rebase on top of master and then also update the rust-version file so that it picks up your rustc changes?

This has no effect when -Zmiri-disable-validation is present.

Should I add a test for this, or is it not necessary?

Should I add a test for this, or is it not necessary?

I don't think that is necessary.

@bors r+

:hourglass: Testing commit 6dd10820dd207d392f58b36ff919b8b17cfa1261 with merge a8b976eb350acec83280a0cd1ca3ac99faff67bc...

Thanks again for your help with this change! ❤

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing a8b976eb350acec83280a0cd1ca3ac99faff67bc to master...

I'll try to open a submodule sync later today.


add jest.advanceTimersToNextTimer method

A contribution by me and @deanshub

Summary
When you test a code (using fake timers) with lots of timeouts/intervals, and some of the code is 3rd party, it's hard to keep track of the right amount of milliseconds you should advance timers by.
Moreover, sometimes you don't want to run all the timers, because you might want to expect something before the end of them all. (Maybe you need to assert between the second and third timeouts)
jest.advanceTimersToNextTimer will advance the timers by the correct milliseconds that make sense for the next  assertion.
The alternative is to run jest.advanceTimersByTime(1) a lot of times until you ran the right amount to make your test pass (runAllTimers will run too much timers).
We tried this new feature on our tests and they look much cleaner.
Test plan
Added tests to jest-fake-timers

@thymikee please please please?


LD14 ROS ERROR

[ERROR] [1677467993.471104037]: ldlidar point cloud data publish time out, please check your lidar device

Hi, developer, this error follow on:
● First, check whether the hardware connection is firm
● Second, connect the product to the PC host, and run the Windos version of point cloud visualization software suitable for the product to check whether point cloud information can be obtained normally。
https://github.com/ldrobotSensorTeam/ld_desktop_tool/releases/download/v3.0.6/LdsPointCloudViewer_v3.0.6_windows-x86_x64_20221017.zip

[ERROR] [1677467993.471104037]: ldlidar point cloud data publish time out, please check your lidar device

@billqualls7

你好 接到windows之后板子上面的蓝灯是熄灭状态，但是使用串口助手可以收到数据，您提供的软件无法看见点云
Hello， After connecting to windows, the blue light on the board is off, but using the serial assistant can receive data, the software you provide can not see the point cloud
------------------ 原始邮件 ------------------
发件人:                                                                                "ldrobotSensorTeam/ldlidar_sl_ros"                                                                                @.>;
发送时间: 2023年2月27日(星期一) 中午11:45
@.>;
<EMAIL_ADDRESS>主题: Re: [ldrobotSensorTeam/ldlidar_sl_ros] LD14 ROS ERROR (Issue #1)
Hi, developer, this error follow on:
● First, check whether the hardware connection is firm
● Second, connect the product to the PC host, and run the Windos version of point cloud visualization software suitable for the product to check whether point cloud information can be obtained normally。
https://github.com/ldrobotSensorTeam/ld_desktop_tool/releases/download/v3.0.6/LdsPointCloudViewer_v3.0.6_windows-x86_x64_20221017.zip
[ERROR] [1677467993.471104037]: ldlidar point cloud data publish time out, please check your lidar device
@billqualls7
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

你好 ，你所使用的产品的照片可以提供一下吗？ 我想确认一下是否是LD14激光雷达，此外使用这个上位机，需要将device选择为LDS14

你好 接到windows之后板子上面的蓝灯是熄灭状态，但是使用串口助手可以收到数据，您提供的软件无法看见点云 Hello， After connecting to windows, the blue light on the board is off, but using the serial assistant can receive data, the software you provide can not see the point cloud
…
------------------ 原始邮件 ------------------ 发件人: "ldrobotSensorTeam/ldlidar_sl_ros" @.>; 发送时间: 2023年2月27日(星期一) 中午11:45 @.><EMAIL_ADDRESS>主题: Re: [ldrobotSensorTeam/ldlidar_sl_ros] LD14 ROS ERROR (Issue #1) Hi, developer, this error follow on: ● First, check whether the hardware connection is firm ● Second, connect the product to the PC host, and run the Windos version of point cloud visualization software suitable for the product to check whether point cloud information can be obtained normally。 https://github.com/ldrobotSensorTeam/ld_desktop_tool/releases/download/v3.0.6/LdsPointCloudViewer_v3.0.6_windows-x86_x64_20221017.zip [ERROR] [1677467993.471104037]: ldlidar point cloud data publish time out, please check your lidar device @billqualls7 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

开发手册上面显示ld14波特率为115200但是我测试之后需要230400才能接受数据  在Ubuntu20.04里面使用提供的Python程序接受雷达数据也能正确打开
---Original---
From: "David @.>
Date: Mon, Feb 27, 2023 12:20 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [ldrobotSensorTeam/ldlidar_sl_ros] LD14 ROS ERROR (Issue #1)
你好 ，你所使用的产品的照片可以提供一下吗？ 我想确认一下是否是LD14激光雷达，此外使用这个上位机，需要将device选择为LDS14
你好 接到windows之后板子上面的蓝灯是熄灭状态，但是使用串口助手可以收到数据，您提供的软件无法看见点云 Hello， After connecting to windows, the blue light on the board is off, but using the serial assistant can receive data, the software you provide can not see the point cloud
…
------------------ 原始邮件 ------------------ 发件人: "ldrobotSensorTeam/ldlidar_sl_ros" @.>; 发送时间: 2023年2月27日(星期一) 中午11:45 @.><EMAIL_ADDRESS>主题: Re: [ldrobotSensorTeam/ldlidar_sl_ros] LD14 ROS ERROR (Issue #1) Hi, developer, this error follow on: ● First, check whether the hardware connection is firm ● Second, connect the product to the PC host, and run the Windos version of point cloud visualization software suitable for the product to check whether point cloud information can be obtained normally。 https://github.com/ldrobotSensorTeam/ld_desktop_tool/releases/download/v3.0.6/LdsPointCloudViewer_v3.0.6_windows-x86_x64_20221017.zip [ERROR] [1677467993.471104037]: ldlidar point cloud data publish time out, please check your lidar device @billqualls7 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

此问题是激光雷达产品与该ROS功能包不匹配导致的，我们团队内部会及时更新对最新推出产品的支持


Make Response::from_html use text/html; charset=utf-8

Response::from_html("
Hello, World! 🐱
")

Before

After

@kpcyrd could you fix formatting?

Sure, done!


About save image is it possible to realize such a function?

Your nodes can already batch load and save, but I've searched for a long time and haven't found any nodes that can save images with the original filename. if you can support that, it would be really appreciated.

Thanks for the suggestion @suede299. I will look into adding more text outputs for filenames. I think other good one would be to save the image and txt file with the same name for training data.


Refactor process task

#713  👈
main

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

Join @alanvardy and the rest of your teammates on  Graphite


S8 [User Story] Spike -- Research LDAP login token

[User Story]
Spike -- Research LDAP login token

Epic Title
#271

User Story Title
Spike -- Research LDAP login token

T-shirt size: S(mall), M(edium), L(arge)
M

Story Points - Fibonacci (1,2,3,5,8,13)
3

User Story
[Value Statement] "As a user of a certain type (specify), I want to do something (action) so I can provide business value"

Description
Find out what is returned after logging in to AD as a user. Could be a JWT token or some other token.

Business Need (the why):
from epic, if needed

Details
All the details needed to accomplish the story. Tasks - can be linked

Additional User Story
linked

Acceptance Criteria/Tests
The list of tests or criteria to be used to validate the story has been completed. May still need UAT or SIT / integration testing with other stories.

Failure
if any, fold in negative tests

Definition of Done
e.g.Passes all regression tests, Passes testing per acceptance criteria, Approved by UI team, Able to show feature in demo

UX/UI Details and/or mockups
if UI change, a mockup or description of what to change, attach image/file

Applications or Systems impacted
upstream/downstream applications/systems

Dependencies
non story dependencies

Assumptions
if any

Performance Considerations
Will the work performed as part of this story impact system performance. If there is potential for a performance hit, where would you expect it to manifest in the system.

Security Considerations
E.g., does this story involve working with Personal Identifying Information (PII)? Note any such security sensitive aspects.

QA Considerations
what might be hard to test or test setup criteria

Acrhitectural/System or Component Impacts
if we're adding something new or significantly changing architecture flag for architect review

Reverse Engineering Required
to document areas not known well by dev team

Questions/Clarifications
Create new issue (add label: question) in this repo and link to this issue

Just like all stories, this one needs an estimate.  The template at the top indicates it's a 3.  Can we go by this?
@adamgering does this need to be in S8?  Seems like it's related to the stories around AD auth but if are able to demo AD functionality now, I'm assuming we are logged in without understanding what is returned.  Why do we need this?  Thx.


SL: Review: Make Sure the form is compatible

Some dev research/review required to ensure that SL Review form will be compatible with other form/related updates functionality

Tested by @bhumin-fw and no issues found

Stella & Eddie to test on Prod

Stella does not see this ticket in prod - decision to do this testing after those other SL tickets are done: Decision about retention and how the workflow ends. Needs answers from Jodi when we are ready.

Hello @Andrew-Vargas-bcgov could you add the client team to a security group so they can access & QA the test version of SL Review form: https://digital-journeys-test.apps.silver.devops.gov.bc.ca/form/63471cbc842f42ecdafecc4b
Martha Wood
Roberta Turton
Eduardo Ramirez
Anita Bawa
Jodi Little

Form seems to be functioning properly.


Add "Abandoned" to record groups that may still exist but have bad info

This can be filled manually by looking at gemstate.io crawl log.
gemstate.io would like this so it can show abandoned groups differently to help them update or get new support.

This is now in master (abandonedDate) and I've marked off a couple groups... not all of them, for sure, but I marked off the ones that appeared to be gone from meetup.com and google.


basic E2E frame work for ingress node firewall

make sure all CRDs are installed
make sure operator deployment is running
make sure DaemonSet is running
Apply firewall rules and check actions

=== RUN   TestE2E
Running Suite: Ingress Node Firewall Operator E2E Suite
=======================================================
Random Seed:<PHONE_NUMBER>
Will run 1 of 1 specs

Ingress Node Firewall IngressNodeFirewall deploy
  should run Ingress node firewall
  /home/mmahmoud/go/src/ingress-node-firewall/test/e2e/functional/tests/e2e.go:61
STEP: checking Ingress node firewall daemonset is in running state
STEP: get nodes IPv4 with matching labels and ping their IPs
STEP: creating ingress node firewall rules
STEP: checking Ingress node firewall rules resource is created
STEP: checking ingress node firewall node list resources is created
STEP: checking Ingress node firewall rules deny ping packets
STEP: checking Ingress node firewall rules resource is deleted

• [SLOW TEST:86.370 seconds]
Ingress Node Firewall
/home/mmahmoud/go/src/ingress-node-firewall/test/e2e/functional/tests/e2e.go:29
  IngressNodeFirewall deploy
  /home/mmahmoud/go/src/ingress-node-firewall/test/e2e/functional/tests/e2e.go:30
    should run Ingress node firewall
    /home/mmahmoud/go/src/ingress-node-firewall/test/e2e/functional/tests/e2e.go:61
------------------------------

JUnit report was created: /tmp/test_e2e_logs/e2e_junit.xml

Ran 1 of 1 Specs in 86.371 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 0 Skipped

/lgtm