Change FilterIndex rule to cover select all columns case

What changes were proposed in this pull request?
This PR extends FilterIndex rule to cover the case when all columns are selected from a relation.
Such a case happens when logical plan is "Scan -> Filter" and Project node is not present or is eliminated due to some other optimizations (for example in a select * scenario).
Why are the changes needed?
Extend FilterIndex rule to cover more potential optimization cases.
Fixing issue #16
Does this PR introduce any user-facing change?
No
How was this patch tested?
Test cases added under FilterIndexRule tests and E2E tests.

Thanks, LGTM @imback82 , @pirz


Improve README.md shipped with tools

The current README.md shipped with the tools is more of a manifest and less of a how-to document. Look at migrating as much of the portal documentation as possible to this file for future tools releases.

[master 024105c] Update the tools README to show all of the instructions (Issue #55)


Upgrade ash-molten

Resolved https://github.com/attackgoat/screen-13/issues/59.

This will get the code to build on Mac, but see #62 for updates to the examples and feature-parity work


Repetition in rosetta stone example

There are two sections called which seem to be 2 variants of the same:

Loop between index  3   and    7   (inclusive)
Loop between  3  -th and    7  -th elements (inclusive)

Rows 672 and 701 of rosetta_stone.md file.

Fixed by PR #294 .


docs: 211: Fix an inconsistency in pillar.get formatting

I was confused for a while trying to figure out why the same syntax was both correct and incorrect.

Thanks @alvarobrey for improving the documentation!


Error while running bash eval rlbench  file

Traceback (most recent call last):   File "online_evaluation_rlbench/evaluate_policy.py", line 194, in      var_success_rates = env.evaluate_task_on_multiple_variations(   File "/3d_diffuser_actor/utils/utils_with_rlbench.py", line 464, in evaluate_task_on_multiple_variations     sum(var_success_rates.values()) / ZeroDivisionError: division by zero QObject::~QObject: Timers cannot be stopped from another thread

From the error you pasted, it seems that you cannot access the test demos. Have you followed the instructions here? https://github.com/nickgkan/3d_diffuser_actor/blob/master/docs/DATA_PREPARATION_RLBENCH.md

Got it , Done.   So how much time wiil it take approx. for eval to be completed ?

Its been more than 35 minutes , but its showing the same.

almost 52 minutes still showing the same

almost 52 minutes still showing the same

It seems that your program hangs there. This is not the expected behavior, the logs are printed every few seconds. I would suggest that you try to trace where it hangs. Also, I'm suspicious there is some call trying to find the data but cannot fetch it.

why its giving retry for such a long steps after step 9 until last 24?

Probably the planner cannot reach the target pose. This happens sometimes. Can you still render the video? I think you're running it in headless mode, but video is helpful for such understanding.

after running the script , i only find .json files in eval_logs/3d_diffuser_actor/seed0/ , but how to get  video .mp4 files ?


perlPackages.AppSqitch: 1.2.1

Checklist

Latest version bump: https://github.com/NixOS/nixpkgs/commit/ef87b2112369292e334e360cb7ef6b01e5c15f11

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: legacyPackages.x86_64-linux.perlPackages.AppSqitch

current version: 1.1.0
desired version: 1.2.1
Notify maintainers

maintainers: volth but it doesn't seem like the email is tied to a GitHub account
Note for maintainers
Please tag this issue in your PR.

https://github.com/NixOS/nixpkgs/blob/e6e389917a8c778be636e67a67ec958f511cc55d/pkgs/top-level/perl-packages.nix#L919


NotFound gokvstore-bench

https://github.com/xujiajun/gokvstore-bench

hi @yangwenmai  please try again.

You set a private repos before?


Argument matching for callbacks added

This was bugging me for a while.
Say I have an trigger and not all callbacks need its arguments (especially conditions, like):
class Model(object):
  states = ['unbroken', 'broken']
  def __init__(self, is_concrete=True):
    self.is_concrete = is_concrete
    self.machine = Machine(model=self, (...))
    self.machine.add_transition('hit_hammer', 'unbroken', 'broken',
           conditions='was_hit_hard', unless='is_concrete')
  def was_hit_hard(self, force):
    return force > 50
  def is_concrete(self):
    return self.is_concrete

model.hit_hammer(force=50)

. Now the code crashes and I had to resort to changing callback signature to take (*args, **kwargs), but I would like to avoid that.

Coverage increased (+0.003%) to 99.89% when pulling 3710fe7fccf36ec2fb9800b3fbd84a5a8edf3b86 on KarolOlko:allow_callbacks_with_less_args into 23e2b29b7afb41ab9346002c6e1c6c54a195ff7c on tyarkoni:master.

Oops.. Now I realized conditions are not covered. This method needs to be moved to EventData. Let me do it in another pull request.

Moved function arguments trimming to EventData and checked conditions in unit test added.

Can you clarify what the problem behavior is? As detailed in this README section, When send_event=True, args and kwargs are stored inside the EventData object. So the expected pattern here is for callbacks to get what they need from, e.g., event.args and event.kwargs. Is there a reason this approach doesn't work for your use case?

Coverage increased (+0.003%) to 99.89% when pulling 3710fe7fccf36ec2fb9800b3fbd84a5a8edf3b86 on KarolOlko:allow_callbacks_with_less_args into 23e2b29b7afb41ab9346002c6e1c6c54a195ff7c on tyarkoni:master.

Hi, thanks for your reply. Well, it would work, but now all callbacks need an unused event (or *args or whatever) function parameter in their signature. I think client code is cleaner if you don't impose this limitation. If I don't need event data and base my callback only on internals - library should respect that.


Add hard dollars

Reflecting on this, here's where I come down. Feel free to disagree.
I'd make the case that professional usage alone should be our test for inclusion. Whether or not we agree with the recent usage of "hard dollars," the fact is that reporters are putting it in their stories, as documented by my proposed examples.
If we applied an editorial test based on our own opinion about a word's fitness, we might have cut many more terms. For instance, I don't like the loose usage of the term Super PAC, but we include that in the dictionary.

I get it, but "Super PAC" has an administrative definition. Hard money does not. I don't agree that we should be describing the bad uses of a term.

Alright, let's see if my sense of the hard-soft distinction over time matches your more experienced view. My general sense is that the Bipartisan Campaign Reform Act of 2002 and the Citizens United were watershed dividing lines that changed the parlance of our press.

Before BCRA
After Citizens United

hard money
funds contributed directly to campaigns that count against contribution limits
funds contributed directly to campaigns that count against contribution limits and, in contrast to soft money, can be spent directly on the activity of the official campaign

soft money
funds indirectly routed to campaigns via parties and other vehicles in a manner that evades contribution limits
independent expenditures made by outside groups that may benefit a campaign but cannot be coordinated with candidates or spent directly on the activity of the official campaign

How'd I'd do, professor?

Not bad; soft money was not routed to candidates but was given to national party committees and used by them for ads that were the equivalent of issue ads today. The real issue is that "guides" like this one completely misstate what is the law and what is happening. This guide is largely fiction except at the very broadest level.
Citizens United did not create independent expenditures; it broadened the scope of what kind of organizations could pay for them using particular sources (corporate treasury money, which previously could be given to parties as a soft money donation). But CU did not turn soft money into IEs. IEs have existed in law and in practice for decades, and are funded by disclosed donations with one exception, when 501c4s engage in what is known as electioneering communications. CU also did not change the definition of "hard money".

Do you think we need to tweak our existing soft money definition, then? https://moneyinpolitics.wtf/soft-money/

Yeah, if we add a hard dollars, we probably should just to reflect that definition.

So here's what I have in the first proposal:

funds donated directly to a campaign committee that are publicly disclosed and subject to contribution limits; used by candidates and parties to pay for operating expenses like staff salaries, travel and events; distinguished from “soft” independent expenditures by outside groups that cannot be put toward the day-to-day operation of a campaign

So am I correctly grokking that you'd be okay if we simply trimmed it to cut the IE language, like so:

funds donated directly to a campaign committee that are publicly disclosed and subject to contribution limits; used by candidates and parties to pay for operating expenses like staff salaries, travel and events


run_bug_detection.pl terminates with an error ClassNotFoundException: {test_class_name} logged in the {versionid}.{testid}.trigger.log files

I'm running evosuite generated tests against fixed and buggy versions of (let's say) Lang27, to check whether they are able to detect the Lang27 bug. The tests were generated for the buggy version. Following is the command I've used.
run_bug_detection.pl -p Lang -d . -o output -v 27b -D &
The test suite is Lang-27b-evosuite-branch.1.tar.bz2, and it contains both NumberUtils_ESTest.java & NumberUtils_ESTest_scaffolding.java files. NumberUtils is the buggy class in Lang27.
However, when I ran the run_bug_detection.pl ... command, it terminates with the following error.

run.gen.tests:
[echo] classes.dir: target/classes
[echo] build.home: target
[junit] Running NumberUtils_ESTest
[junit] Tests run: 1, Failures: 0, Errors: 1, Time elapsed: 0 sec
[junit] Test NumberUtils_ESTest FAILED

Further, it shows the following in the output/bug_detection,

and both 27b.1.trigger.log & 27f.1.trigger.log shows the following error.

I've also checked the /tmp/run_bug_detection.pl_*** directory and found out that NumberUtils_ESTest.java is available in evosuite-branch/ and NumberUtils_ESTest.class is available in target/gen-tests/
Here's the link to the dropbox folder with test suite and Runtime test dependency: evosuite-standalone-runtime-1.0.7-SNAPSHOT.jar used.
https://www.dropbox.com/sh/yebmblfgjp77x1o/AABdWenEHcs0paoLPXAQ4dDea?dl=0

Closing this as I found out the issue is actually because not having the test classes inside the test_suite directory as package structure.


npl/riot: update generated syscfg.h

As promised in #755: this PR updates the generated syscfg.h file for the RIOT target to the latest changes.
Before generating syscfg.h, I took the freedom of setting BLE_LL_SUPP_MAX_TX_BYTES and BLE_LL_SUPP_MAX_RX_BYTES to MYNEWT_VAL_BLE_LL_MAX_PKT_SIZE in the target's syscfg.yml, is this keeps the existing values in syscfg.h and makes it easier to change the link layer packet size from withing RIOT's makefiles...
I made sure nothing broke by verifying that RIOT/examples/nimble_gatt as well as my private NimBLE/IPv6/RPL experiments are still running as expected.

Travis failed for unrelated reasons as it seems, restarted the build.

Travis is happy after all -> go.


how to edit config and use tasks

hey, so i am trying to learn how to use tasks and advance configuration and i don't find out anywhere how. if anyone can help and yes i have look on the slack chat for help in the #help channel and still no help. i just want to know how to add tasks in the config and use advanced configuration. if you can help it would be greatly appreciated. thanks bye ^-^. using windows 10.

yes i very hard because you need enter to all files that you find in configs, in this place you can see all related with advanced configuration, config.example all in this folder

@jomarocas umm what are you talking about? because i don't get what you're referencing to.

?
https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Configuration-files

@

@liudayu i was using the master and not dev never mind.

the you need use
git fetch && git checkout dev
pip install --upgrade -r requirements.txt


Expose Credential type for DefaultAzureCredential and ChainedTokenCredential

When using DefaultAzureCredential or ChainedTokenCredential I find it useful to know which credential type is being used.  Right now I have to resort to reflection or EventSource:
using var listener = new AzureEventSourceListener((eventArgs, text) => AzureEventSourceHandler(eventArgs, text), EventLevel.Informational);

        static void AzureEventSourceHandler(EventWrittenEventArgs eventArgs, string text)
        {
            if (text.Contains("Credential") && text.Contains("succeeded"))
            {
                Console.WriteLine(text);
            }
        }

I would rather do something like this:
var credential = new DefaultAzureCredential();
credentail.GetToken();
credential.GetType();  // and this return the type used

or

credential.GetCredentialType(); // if overriding GetType() is a problem due to composition model

I understand that GetToken isn't called until a network call is needed, but a user could call GetToken() first if they really need to know the Credential type.

Related: https://github.com/Azure/azure-sdk-for-java/issues/6591

Hi @jongio, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.


License Usage

how can a global admin determine which user are not using an assigned License?

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

ID: 129e6c4a-03f2-9a84-cecd-14adf3e490b2
Version Independent ID: 65a3bb77-a63e-7375-5dbe-47c50b236cb6
Content: Periodic tasks for Project Online administrators
Content Source: ProjectOnline/periodic-tasks-for-project-online-administrators.md
Service: project-online
GitHub Login: @efrene
Microsoft Alias: efrene

@DeanGross Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @DeanGross, thank you for your feedback.
Please check these reports:
https://docs.microsoft.com/en-us/office365/admin/usage-analytics/active-user-in-usage-reports

Thanks but there is nothing on that page for the Project Online application.

Please check Office activation report on that page:
"Any user who has activated their Office 365 Pro Plus, Visio Pro or Project Pro subscription on at least one device."

@DeanGross Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


"Output GPKG file does not exist"

Hello
I have tried to follow the docker setup instructions on my synology NAS but receive an error when I try to initiate it from gpkg.
Below are screenshots of my container setup, I also mounted the /tmp/dbsync folder.  I think my configuration is able to login to both the postgres dB and my local instance of mergin.  Not sure where I am going wrong?  I have also created the schemas in postgres using pgAdmin.  Not sure if it should be in the env variables as well.

Once again, thank you in advance for the assistance.

It looks like you have a space between -- and init-from-gpkg ? It should be together --init-from-gpkg - probably that's why the option is not recognized correctly?

Thank you! Rookie error.  I now have this error, but I think I need to setup my project correctly.

It may be a good idea to remove your existing mergin-db-sync container and start a new one, in case there are some leftover files from the previous unsuccessful run...


Documentation?

Is there any documentation on the classes? I can't find any - am I overlooking it somewhere?

I'm assuming you mean C# XML Documentation like this class: https://github.com/bradyholt/cron-expression-descriptor/blob/master/CronExpressionDescriptor/ExpressionDescriptor.cs#L12 ?  If so, I have not fully implemented the XML documentation on the types and methods.  I would surely accept a pull request with the documentation added :smile:


TermDD error Event ID 50

"The RDP protocol component X.224 detected an error in the protocol stream and has disconnected the client."
I don't know if it is related to the rdpwrap but I find many of this errors in the event viewer. The strange thing is that there isn't any client connected.
Uploading TermDD EventID 50.txt…

Thanks for the reply but it doesn't solve the issue, because the automatic fix stop because the fix doens't apply to the operating system (windows 7 64-bit home premium) and the manual fix suggest to remove some registry entries, which doesn't exists.


fix(gitRawCommitsOpts): apply user opts over our opts

When releaseCount equals any value other than 0, the original code
replaces the gitRawCommitsOpts object with our own configuration.
Instead, we need to apply the user's configuration over our configuration,
replacing our configuration where it conflicts with the user's configuration.

You can push directly to this repo so that Travis can build properly.


Database Connection Issue

Description
I have an reverseproxy network called webproxy with a /16 subnet (<IP_ADDRESS>/16)
Hemmelig.app runs on ip <IP_ADDRESS>
No public ports are exposed in my compose file for security reasons, ports are accessable within the vps and can be used with nginx proxy manager to access my services via a domain.
Here the compose file i use:
version: '3'
services:
    hemmelig:
        image: hemmeligapp/hemmelig:v5.13.1 # Check the tags for what version to use
        hostname: hemmelig
        init: true
        volumes:
            - ./data/hemmelig/:/var/tmp/hemmelig/upload/files
            - ./data/database/:/root/docker/data/hemmelig/database/
        environment:
            - SECRET_LOCAL_HOSTNAME=files.vbucksgenerator.zip # The local hostname for the fastify instance
            - SECRET_PORT=3000 # The port number for the fastify instance
            - SECRET_HOST=http://<IP_ADDRESS>:3000 # Used for i.e. set cors/cookies to your domain name
            - SECRET_ROOT_USER=groot # User as the root admin user
            - SECRET_ROOT_PASSWORD=iamroot # The admin user password (change this after signed in)
            -<EMAIL_ADDRESS># The email for the admin user
            - SECRET_FILE_SIZE=4 # Set the total allowed upload file size in mb
            - SECRET_FORCED_LANGUAGE=en # Set the default language for the application
            - SECRET_JWT_SECRET=P6FJuoLQPB8qcqWSVvmrXrdPL # Override this for the secret signin JWT tokens for log in
            - SECRET_MAX_TEXT_SIZE=256 # The max text size for the secret. Is set in kb. i.e. 256 for 256kb
        networks:
            - webproxy
        restart: always
        stop_grace_period: 1m

        #healthcheck:
            #test: 'wget -O /dev/null http://<IP_ADDRESS>:3000 || exit 1'
            #timeout: 5s
            #retries: 1

networks:
    webproxy:
        external: true

Here the docker logs:
><EMAIL_ADDRESS>start
> prisma migrate deploy && node server.js
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "hemmelig.db" at "file:../database/hemmelig.db"
SQLite database hemmelig.db created at file:../database/hemmelig.db
13 migrations found in prisma/migrations
Applying migration `20230403094651_init`
Applying migration `20230403105353_init`
Applying migration `20230405075854_usergenerated`
Applying migration `20230405082548_userrole`
Applying migration `20230406072002_settings`
Applying migration `20230406095057_readonly`
Applying migration `20230409114406_disablefileupload`
Applying migration `20230409162446_cascadedelete`
Applying migration `20230411070916_disable_account_creation`
Applying migration `20230415103015_user_secret_relation`
Applying migration `20230417053342_org_email_setting`
Applying migration `20230417111804_rename_restrict`
Applying migration `20230421090823_secretid_uuid`
The following migrations have been applied:
migrations/
  └─<PHONE_NUMBER>4651_init/
    └─ migration.sql
  └─<PHONE_NUMBER>5353_init/
    └─ migration.sql
  └─<PHONE_NUMBER>5854_usergenerated/
    └─ migration.sql
  └─<PHONE_NUMBER>2548_userrole/
    └─ migration.sql
  └─<PHONE_NUMBER>2002_settings/
    └─ migration.sql
  └─<PHONE_NUMBER>5057_readonly/
    └─ migration.sql
  └─<PHONE_NUMBER>4406_disablefileupload/
    └─ migration.sql
  └─<PHONE_NUMBER>2446_cascadedelete/
    └─ migration.sql
  └─<PHONE_NUMBER>0916_disable_account_creation/
    └─ migration.sql
  └─<PHONE_NUMBER>3015_user_secret_relation/
    └─ migration.sql
  └─<PHONE_NUMBER>3342_org_email_setting/
    └─ migration.sql
  └─<PHONE_NUMBER>1804_rename_restrict/
    └─ migration.sql
  └─<PHONE_NUMBER>0823_secretid_uuid/
    └─ migration.sql

All migrations have been successfully applied.
(node:20) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.
Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy
(Use `node --trace-warnings ...` to show where the warning was created)
2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 hostname=hemmelig 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 hostname=hemmelig err={"type":"Error","message":"listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:3000","stack":"Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:3000\n    at Server.setupListenHandle [as _listen2] (node:net:1800:21)\n    at listenInCluster (node:net:1865:12)\n    at GetAddrInfoReqWrap.doListen [as callback] (node:net:2014:7)\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:110:8)","code":"EADDRNOTAVAIL","errno":-99,"syscall":"listen","address":"<IP_ADDRESS>","port":3000} 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 hostname=hemmelig 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 2023/09/28 04:58PM 2023/09/28 04:58PM 50 pid=20 hostname=hemmelig err={"type":"Error","message":"listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:3000","stack":"Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:3000\n    at Server.setupListenHandle [as _listen2] (node:net:1800:21)\n    at listenInCluster (node:net:1865:12)\n    at GetAddrInfoReqWrap.doListen [as callback] (node:net:2014:7)\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:110:8)","code":"EADDRNOTAVAIL","errno":-99,"syscall":"listen","address":"<IP_ADDRESS>","port":3000} msg=listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:3000

As you can see, it try to use <IP_ADDRESS> which is outside of the webproxy subnet so i need to figure out, how i can cange the ip it is trying to run on. (<IP_ADDRESS>/16)
Screenshots
No response
Additional information
No response

Well, I do not think this is a bug within Hemmelig. And, it is kind of hard to debug without access to such a network.
That said,
SECRET_LOCAL_HOSTNAME=files.vbucksgenerator.zip should be set to <IP_ADDRESS> (or not used at all)
SECRET_HOST you can remove http://, but you should use the domain name here (files.vbucksgenerator.zip)

Well, I do not think this is a bug within Hemmelig. And, it is kind of hard to debug without access to such a network.
That said,
SECRET_LOCAL_HOSTNAME=files.vbucksgenerator.zip should be set to <IP_ADDRESS> (or not used at all)
SECRET_HOST you can remove http://, but you should use the domain name here (files.vbucksgenerator.zip)

Thanks, i now got it up and running. little sidequestion, where do i find the html, js files from the service?

If you can work with private networks, then you should be able to figure out that one yourself. 😉


Log debug logs only on when at least one error logged in request

Hello, thank you for this great library.
I was wondering is it possible to log debug logs only when error log was triggered in request? For example if in production debug logs would be present it would help developers, but we wouldn't want to log them all the time as they are useless if request did not fail.
From what I saw it is not possible currently as logs are not buffered, but logged immediately, would appreciate if you could comment if this layer is an appropriate one for such implementation. If this makes sense I could submit a PR with a solution.
Or maybe there is already an existing option that will help with that, which I didn't see yet.
As for layer I am using nestjs-pino. So dependency goes nestjs-pino <- pino-http <- pino. So maybe you have some suggestions where this functionality could exist.
Thank you!
EDIT: Actually I see that this library takes care of request/response logging, it doesn't affect other logs, lmk if this is the case.

Thanks for reaching out. Unfortunately it's not a responsibility of this library to do it as this just instrument the server. However creating this behavior would be an amazing transport

Thanks for a quick reply Matteo, yes that makes sense.


fix(example-app): Remove dependencies to deprecated @angular/flex-layout

Usage of @angular/flex-layout was creating Conflicting peer dependency when running npm install
Removing this dependency fix that.
When removing it's usage from example-app to be able to build, no other change have been made to the html.
It's possible it may not represent the intended output, but the librairy look to be working appropriatly.
Since I'm not the creator, it's hard to access, but would be ready to make the necessary change.

@DelBiss Thanks again for your contribution, this is now merge in the beta branch , I could not merge it directly because there were some conflicts so I had to rebase your branch from a copy I did of it

As mentioned closing because this was merged to beta branch after a rebase in a copy of this branch


Ability to customize http request

pass custom headers, use cookie jar, etc.

Should be a simple addition, I will take a look.

Ability to provide custom options to request has been added in 1.1.0 (https://github.com/brendonboshell/supercrawler/commit/58dda5bbd57d23cc77d78a7e1e1d91650d64c904)
Documentation has been added to the README: https://github.com/brendonboshell/supercrawler#crawler
Example:
var crawler = new supercrawler.Crawler({
  interval: 100,
  concurrentRequestsLimit: 5,
  urlList: new supercrawler.RedisUrlList({
    redis: {
      port: 6379,
      host: '<IP_ADDRESS>'
    }
  }),
  request: {
    headers: {
      'x-custom-header': 'example'
    }
  }
});


Lua Config Error

Neovim version
NVIM 0.8.1
Operating system/version
macOS 13.0.1
Terminal name/version
iTerm2 Build 3.4.18
$TERM environment variable
xterm-256color
Branch info
main (Default/Latest)
Fetch Preferences
HTTPS (use_ssh = false)
How to reproduce the issue
when i open the any file will be opened the error message
Expected behavior

Actual behavior
No response
Additional information
No response

Have you finished the installation? Show me the output of :checkhealth please.

I solved by update libraries just now.


Update project page metadata to change page title and description according to project

On chat page, the title should change to the project title

Hey Shivank, can I take this up?

Can you close this issue, @skks1212?


Chakra failed to build with MSVC + permissive- on windows

Chakra failed with error C3861 when build with permissive- on Windows, I use latest source on master branch. Could you please help take a look at this? Noted that this issue only found when compiles with unreleased vctoolset, that next release of MSVC will have this behavior.
You can repro this issue as the steps below:

git clone -c core.autocrlf=true https://github.com/microsoft/ChakraCore D:\Chakra\src
Open a clean x86 prompt and browse to D:\Chakra\src
set CL=-wd4471 /permissive-
msbuild /m /p:Platform=x86 /p:Configuration=Test /p:WindowsTargetPlatformVersion=10.0.16299.0 Build\Chakra.Core.sln /t:Rebuild

Failure like:
d:\chakra\src\lib\common\DataStructures/BaseDictionary.h(1441): error C3861: 'ValueToKey': identifier not found [D:\Chakra\src\lib\Common\Core\Chakra.Common.Core.vcxproj]
…
lots of error C3861

We are not yet committed to being clean w.r.t. any particular implementation of the /permissive- flag but we have been looking into issues as they come up (and the team working on that feature has been contributing fixes as well). That feature is still in active development and is changing and generally becoming more strict over time.


doesn't linking file

VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:34:09.706Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions: none

Steps to Reproduce:
1.my file doesn't open when target file double click or select vs code in menu(it open if right mouse click)

#33216


IE11 ReferenceError: 'Promise' is undefined

This error occurs for me in IE11.

ReferenceError: 'Promise' is undefined
at create (https://dev7.atomiclearning.com/static/js_apps/main/node_modules/angular-flickity/dist/angular-flickity.js:161:14)
at Anonymous function (https://dev7.atomiclearning.com/static/js_apps/main/node_modules/angular-flickity/dist/angular-flickity.js:948:14)
...

Research indicates that the module might rely on Promise, which is not supported by IE11. Was it intended that a polyfill be used? Why wasn't $q used instead?

I've added a polyfill for Promises:
<script src="https://cdn.polyfill.io/v2/polyfill.min.js?features=Promise"></script>
The page loads extremely slowly, though, and the slide animation is also very slow and jerky. Possibly related is the fact that the demos don't work in IE.

I've added a polyfill for Promises:
<script src="https://cdn.polyfill.io/v2/polyfill.min.js?features=Promise"></script>
The page loads extremely slowly, though, and the slide animation is also very slow and jerky. Possibly related is the fact that the demos don't work in IE.

Hey @isherwood, when this was being built IE was not a target browser. The core Flickity library does support IE10+ so IE support would be a good addition.
I've recently started a new role so my time is extremely tight right now so I can't say if/when I could refactor this. :/ More than open to PRs though if the polyfill isn't cutting it for you.

I completely understand. Not to derail anyone, but I ended up writing a simple directive that is working well:
        .directive('myFlickity', function ($timeout) {
            return {
                restrict: 'A',
                link: function (scope, element, attrs) {

                    // wait for the ngRepeat stuff to get set up
                    $timeout(function () {
                        // register DOM listeners and update the DOM
                        var slider = element.flickity(scope.$eval(attrs.alFlickity));

                         // grab the Flickity data object for later use
                        var sliderData = slider.data('flickity');
                    });
                }
            };

+1 IE 10+ support would be great. I'll use the polyfill for now thanks :)

I had an issue with IE11 -> 'Promise' is undefined
Just used $q instead.
Worked for me


KAFKA-13715: add generationId field in subscription

Implementation for KIP-792, to add generationId field in ConsumerProtocolSubscription message. So when doing assignment, we'll take from subscription ownedPartition/generationId fields if they are both provided. Otherwise, we'll fall back to original solution to use userData.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the PR, @showuon. Is it ready for review? I will help you to get it in.

@dajac , sorry for delay. It's good for review now. Thanks.

@dajac @guozhangwang , call for review. Thanks.

Will review it this week. Thanks for the PR.

@dajac , I've addressed all comments. Please take a look when available. Thanks.

@dajac , PR updated. Please take a look when available. Thanks.

@dajac , thanks for the comments. PR updated. Thanks.

@dajac , Thanks for the comments. PR updated. Thanks.

All tests passed.


Python tests are getting skipped in GitHub actions

Describe the problem

The CI and master builds are only running Ray. This is probably because we export MODIN_ENGINE=... instead of setting it as an environment variable via env in the yml
Source code / logs
Problem lines:
https://github.com/modin-project/modin/blob/e993bf69c305377c34a6b333a44da7950c1d28c5/.github/workflows/ci.yml#L52
https://github.com/modin-project/modin/blob/e993bf69c305377c34a6b333a44da7950c1d28c5/.github/workflows/push.yml#L127

Resolved via #964


Mask Slow Performance and Inaccurate Input on Blazor Server

Bug type
Component
Component name
MudTextField
What happened?
I have experienced significant lag when using masks on the MudTextField while debugging a Blazor Server project which always produces incorrect values compared to actual keys pressed. I was able to reproduce this issue by cloning the MudBlazor repository and debugging the Blazor Server project. This is not a problem when running the Blazor WASM project. I also noticed that the site deployed to mudblazor.com is WASM which is why the Masking section in documentation works so well. I have a feeling that this is related to SignalR performance based on the current strategy of carrot manipulation to achieve the mask effect.
Expected behavior
As you press keys moderately quickly, you see the carrot move left as a character is input and attempt to move back to end. However, if you type too quickly (easy to do), the output is not accurate to what was typed. For example, when using the PatternMask "0000-0000-0000-0000", if you were to type 1234 repeatedly, the expected result would be 1234-1234-1234-1234 but can result in 1234-2314-3241-3421 (varying each attempt).
Reproduction link
https://try.mudblazor.com/snippet/GOmdYQkSLTSUAIGW
Reproduction steps

Clone MudBlazor repo.
Debug new instance of Blazor Server project.
Navigate to Features > Masking page.
In the first example for credit card number, type '1234' (not using numpad) very quickly, repeatedly.
Watch carrot lag and input be inaccurate.
...

Relevant log output
No response
Version (bug)
6.1.8
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Unfortunately MudMask doesn't working with server-side (except there is really no latency). You can think about some third party js solutions like Cleave.js or IMask.

Thanks @mckaragoz for the reply. That's a bummer, but it makes sense. I wonder if it will be possible to remedy this with Blazor's new 'united' prototype coming in .NET 8.

Couldn't help but chuckle at the "carrot", sorry, but I believe this is the same issue: #5949

Same issue with v6.2.1 and Blazor Server.  :-(

When this will be fixed, is there any roadmap kindly update. This is issue causing slow adoption of the masking feature.

When this will be fixed, is there any roadmap kindly update. This is issue causing slow adoption of the masking feature.

Hi. This is open source project, and if you know how to address the issue a PR is welcome.
This slow performance in BSS is because the Masking is implemented natively in C# and any kind of latency will affect it. Masking should be done through JS, but since writing own JS solution is costly on resources and we have policy not to use 3rd party libraries it is what it is. As @mckaragoz said we recommend to use Cleave.js, IMaskJS

Actually it's not "slow" its by design. Our C# code is lightweight, but the process of server-client communication has delay. Probably you did not encounter caret inconsistency issue in development, but on production we can see the real world-delay.
I have currently no idea how to fix this in server-side. Probably its the only (and the currently most advanced) full C# inputmask solution in blazor world.


Version 2.0 release

Is there anything blocking for version 2.0 release. It has been more than a year for beta version.

I agree. It looks like Ruby 3 support just got added so it seems like a good time for a release. Are people hand wringing over a 2.0.0 release? I feel like most people have been forced to run of a git commit instead of a release at this point anyway. It seems like any release is better than nothing at this point.

It would be great to have official release, happy to help if anything blocks it :)

@orien / @asellitt we are about to start using this gem for a project.  I was wondering if we can get a 2.0 release cut that way we do not have to pin to specific commit?

I think you're right. Looking over the recently merged PRs, it looks like theres quite a bit of change. I'll try to find some time next week to get a proper change log written up, and then see if i can remember how to release a new version

@asellitt my team is picking up this gem to use at our company.  If you want some support at getting a release out email me at<EMAIL_ADDRESS>

@orien / @asellitt any update on this?

Sorry, I no longer work at envato and cannot help you with this.

Sorry, I no longer work at envato and cannot help you with this.

We are hiring at Wonolo (remote) email me your resume if you want<EMAIL_ADDRESS>

@stevend @scottyp-env @ricobl any thoughts here?

@orien?

Sorry for the delay folks. We have this task in our backlog, I'll try and get it prioritised.

Hi @orien, is there any timeline when we can expect 2.0 release? Thanks!

Thanks for your patience everyone.
Version 2.0.0 has been released and is available on rubygems.org: https://rubygems.org/gems/double_entry/versions/2.0.0

@orien can you take a look at this PR ?
https://github.com/envato/double_entry/pull/213#issuecomment-1372645467


Docker Machine pre_window command breaks pane commands

Docker Machine requires an init command to be run in each shell session, which I've set as the Tmuxinator pre_window command:
pre_window: eval "$(docker-machine env test-vm)"

This worked great at first, but a few weeks ago I noticed that any following pane commands don't work. Strangely, they're printed in the console but not evaluated.
E.g.:
pre_window: eval "$(docker-machine env test-vm)"

windows:
  - "Test":
    panes:
      - ls

When I launch the above config, I get the following output:
$ eval "$(docker-machine env test-vm)"
ls
$

Can you get a full trace?

I can also confirm this is happening. I just switched one of my projects from boot2docker to docker-machine. If I switch back to boot2docker, it works even though the command fails. echo $? returns 0 when using this eval statement so I don't believe that's the issue.

Changing to the following also fails the same way. It runs the eval statement, but no more.
windows:
    - work:
        root: /path/to/project
        panes:
            - main:
                - eval "$(docker-machine env dev)"
                - workon myenv

I just found this issue over in the Docker Machine repo.  @pstengel's workaround fixed the issue for me:
eval "$(docker-machine env default </dev/null)"


Add prettier-plugin-tailwindcss to sort tailwindcss classes

Description
This PR adds prettier-plugin-tailwindcss to scaffold-eth-2.
By using prettier-plugin-tailwindcss, this PR aims to achieve consistency in classNames across projects using scaffold-eth-2.
Also, all necessary files that contain tailwind classes are sorted.
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Your ENS/address: portdev.eth

Getting this error when I format a file, also it takes too long to format.
`[Error - 2:17:14 PM] An unexpected error occurred:
[Error - 2:17:14 PM] Error: Cannot find module '@trivago/prettier-plugin-sort-imports'
Require stack:

C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\prettier\index.js
C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-plugin-prettier\eslint-plugin-prettier.js
<EMAIL_ADDRESS>Occurred while linting C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\pages\debug.tsx:1
Rule: "prettier/prettier"
at Module._resolveFilename (node:internal/modules/cjs/loader:1010:15)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)
at mod._resolveFilename (C:\Users\port\testrepos\scaffold-eth-2\packages\nextjs\node_modules\eslint-config-next\index.js:49:26)`

Seems like the official Tailwind CSS Intellisense vscode extension is the cause of this slowness, not ESLint. I disabled that extension and now everything works fine.
I would like other people to test this as well.

Hey @portdeveloper, thanks for pr!
I have Tailwind intellisense extension, and for me everything works and works fast. We need to figure out why it's slow for you because disabling the extension for everyone is not the best solution

Thanks for the insight. I am glad to hear that it is working out great for you! I think there might be a problem with my machine...

Tysm Port and Rinnat 🙌
I tried this on vscode with tailwind intellisense installed and it works fine for me 🙌
NOTE : My nvim lsp config is a bit messed up, so just sharing my personal problem when I was try to format in nvim it was adding some daisUI stuff at the top of the file after formatting :
https://github.com/scaffold-eth/scaffold-eth-2/assets/80153681/ae4d33f0-d6e7-4bb2-a149-7da02715702e
After researching some, I found this discussion:

Doing daisyUi.logs : false in tailwind.config.js solved the issue for me as suggested here -> https://github.com/tailwindlabs/tailwindcss/discussions/8380#discussion-4083131

Its seems that prettier-plugin-tailwindcss had some incompatibilities with @trivago/prettier-plugin-sort-imports (which we are using....port also got kind of same error https://github.com/scaffold-eth/scaffold-eth-2/pull/393#issuecomment-1614547337 ).......check out this discussion about this issue:

https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/9
Although the above issue's was fixed I found people people still using some workaround like :
https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/31#issuecomment-1448996271 also worth checking full discussion
Also regarding Port's this comment https://github.com/scaffold-eth/scaffold-eth-2/pull/393#issuecomment-1615145199  while researching I found a person who had same issue and he resolved it by uninstalling and installing tailwind extention :), I am unable to find it now but it might be somewhere hidden here ->  https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/113

One thing I noticed about prettier-plugin-tailwindcss is that it does not remove extra whitespaces in the classNames, and sometimes it just changes the places of two classes over and over again. This will hinder standardization.
I will open an issue on their github and see if they can fix it.

Sometimes it just changes the places of two classes over and over again. This will hinder standardization

Yeah, I have noticed sometimes this issue too !!
Due to some issues in the plugin and also, I went through a few discussion's on this, and seems like not everyone likes it
Obviously, this is considered as best practice to have in "tailwind zone" but I think it not worth adding at the moment in SE-2 😅 due to some issues, maybe in future in se-2-docs if we have a section regarding best-practices/ extra tools we are for sure gonna recommend this !!
But tysm @portdeveloper for this, really appreciate it !!

One thing I noticed about prettier-plugin-tailwindcss is that it does not remove extra whitespaces in the classNames, and sometimes it just changes the places of two classes over and over again.

Has anyone found a fix for this? This issue makes it really hard to use format check in CI.
At this point I've tried everything mentioned in this PR and some other things to no avail:

reinstall VSCode extensions (tailwind, prettier)
remove @trivago/prettier-plugin-sort-imports
try different prettier/prettier plugin versions with different import styles

I will open an issue on their github and see if they can fix it.

Did you find time to do this? Couldn't find any issues you authored.

One thing I noticed about prettier-plugin-tailwindcss is that it does not remove extra whitespaces in the classNames, and sometimes it just changes the places of two classes over and over again.

Has anyone found a fix for this? This issue makes it really hard to use format check in CI.
At this point I've tried everything mentioned in this PR and some other things to no avail:

reinstall VSCode extensions (tailwind, prettier)
remove @trivago/prettier-plugin-sort-imports
remove daisyUI logs
try different prettier/prettier plugin versions with different import styles

I will open an issue on their github and see if they can fix it.

Did you find time to do this? Couldn't find any issues you authored.
I failed to replicate this issue with new Next.js + DaisyUI + Tailwind + ESLint + Prettier (w/ @trivago/prettier-plugin-sort-imports and prettier-plugin-tailwindcss) even with the exact versions and configs, wondering whether it has something to do with the monorepo structure. I'll post an update here if I figure it out.

I didn't open an issue and basically forgot about it. I think there is a vscode extension that does the same thing. If one day, there exists a solution for this that could be added to a CI/CD pipeline of a project, it will be great as a lot of the devs i saw, including me, just dont care about the order of tailwind classes.

Adding this in case someone who struggles with this stumbles on this:
My issue was that I was using tailwind.config.ts instead of tailwind.config.js and in that case the prettier config needs to point the prettier plugin to the correct config with this:
"tailwindConfig": "./tailwind.config.ts"


#701 broken markdown preview

Report
What did you do?
Run danger for my custom made plugin that outputs a single warning + markdown with a section header and a table.
What did you expect to happen?
I expected to see two beautiful tables: a Danger summary and my own markdown table.
What happened instead?
Danger summary still renders pretty, but markdown (header + table) is not parsed and renders as a text instead.
Your Environment

Bitrise.io
Danger 4.2.0
Dangerfile:

synx_plugin_url = 'https://raw.githubusercontent.com/turekj/danger-synx/master/lib/synx/plugin.rb'
danger.import_plugin(synx_plugin_url)

synx.ensure_clean_structure

Issue
Looking at the comment source it seems that top section tags are broken:
 Danger prints


Binding Error - Two different mapper for same attribute used (MT_COLUMN_CONFIG)

I  run first sql  ,it's ok .  then  I run another sql after first sql (different table) , the APP prompt the error "Binding Error - Two different mapper for same attribute used (MT_COLUMN_CONFIG)"
.  pls refer to attached file

thanks for the hint. fixed with https://github.com/abap2UI5-apps/abap-sql-console/commit/5a88f5be2e563206c2fa40f66585c242d0c42fdf

thank you


Create "Provide name and email" modal

For people that need to accept the terms and conditions AND have a

missing displayname
missing email address
invalid email address
display name that's a URL

We should show a modal dialog that asks them to supply their firstname, lastname and email and accept the Terms and Conditions. This modal should work like the left part of the Sign Up dialog. All fields are required. Clicking the Terms and Conditions link should take them to the full Terms and Conditions agreement per the Sign Up dialog.

Assigning to @simong for fix

What if there are no T&C but the displayName starts with http:// ?

The T&C will be configured when this is rolled out, but we might as well show the modal (without the checkbox) if no T&C are configured

I've attached a commit that takes a stab at this. It currently looks like:

It feels like we should add a little note telling the user why we need this?

@simong given the design screenshot you posted, is there any reason not to make this more in line with the create account modal design? e.g., First Name + Last Name, have colons, etc... the way it is in the design that @nicolaasmatthijs posted?

I've finished a round of review, no functional review yet, however.

Back to @simong

It appears as though this forces the global admin to enter an email address. I suppose that is expected

@simong given the design screenshot you posted, is there any reason not to make this more in line with the create account modal design? e.g., First Name + Last Name, have colons, etc... the way it is in the design that @nicolaasmatthijs posted?

The modal will be shown when a user has no name OR no email address. When the user is missing an email address but has a name that consists of 3 parts there's no way of knowing on which bits are the first name. We decided to only use 1 field for this.

Followed up on review so far, assigning to @nicolaasmatthijs for functional review + css question

Finished a round of review. Assigning to myself for follow-up

Submitted follow-up PR at https://github.com/oaeproject/3akai-ux/pull/4019


[DataInteractor] min/max intensities steps

Working on https://github.com/medInria/medInria-public/issues/492 i enhanced the way the steps are computed for min and max intensities sliders.
Edit: this PR helps solving https://github.com/medInria/medInria-public/issues/492. It needs to be tested on an other PC however i think. On mine, this is really quicker, especially because of d->view2d->SetColorWindowLevel(window, level, layer); in medVtkViewItkDataImageInteractor::setWindowLevelFromMinMax().
:m:

Edit: this PR now solves https://github.com/medInria/medInria-public/issues/492. It needs to be tested on an other PC however i think. On mine, this is really quicker, especially because of d->view2d->SetColorWindowLevel(window, level, layer); in medVtkViewItkDataImageInteractor::setWindowLevelFromMinMax().

Improves the problem of #492 and easier / better handling of steps.
Merging


Feature: author and co-authors to be clickable links leading to the author's page on Hashnode

This PR Closes Issue
closes #
Description
I added links in the PostDetails component for the authors and co-authors images that will open in a new tab. However, I encountered an issue with special names, such as {{ MonaCodeLisa }}, where the presence of curly brackets caused incorrect ASCII formatting in the links (e.g., https://example.com/@{%7MonaCodeLisa %7D%7D). To resolve this problem and ensure proper URL formatting, I created a custom pipe. This pipe removes curly brackets and extra spaces from names, providing an extensible solution for handling special characters throughout the application
What type of PR is this? (check all applicable)

[yes] 🅰 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Mobile & Desktop Screenshots/Recordings
[Attach screenshots or recordings if applicable]
Before Adding the custom pipe  :

After I added the custom pipe :

Steps to QA
Added to documentation?

[ ] 📜 README.md
[ ] 🙅 no documentation needed

[Optional] Post-deployment tasks
[Specify any post-deployment tasks that need to be performed]
[Optional] What gif best describes this PR or how it makes you feel?
[Embed gif or describe the feeling in plain text]

Hello , please try to check the blog post where the author is "monacodeLisa" and  where there is not co-authors , there where I found the problem !

@MestiriBaha ok I saw it -

this vas a very good catch thank you for finding this :)
So what we want to do in this case - is to explore the GraphQL query,
since the values are retrieved from the Hashnode's API and ->
if one time the response is monacodelisa and another time it's {{ MonaCodeLisa }} there is some inconsistency.
I tested the queries, first:

and then -

after that I verified the query that we have in the file graphql.operations.ts and can you guess what showed up

The problem is that in our query on line 139, we have:
				author {
					name
					profilePicture
				}

when it needs to be:
				author {
					username
					profilePicture
				}

it repeats also on line 175 - just below author it still needs to be username
I do appreciate your work creating the pipe, but for this case we need to fix the root cause.
I am going to close this PR and I will write another issue soon - where I will mention everything from this conversation.
I will have to do this across all the app versions. If you would like - once the issue is created in this repo - I can assign it to you, after all you found it :)

@MestiriBaha here is the new issue - #74 if you would like to take it just leave a comment bellow the issue.
It has higher priority, so this one should be done before #61

@monacodelisa , Thank you so much for your feedback  ! I doubted in first place that the root problem can be from the Queries however because of my "little" knowledge of GraphQL syntax , I didn't dig deeper into it , but now , I want to explore the GraphQL query syntax and fix the root of the problem , I would like to take on this issue so assign me on it please !

@MestiriBaha ok so what you need to do is head over to the issue https://github.com/AnguHashBlog/angular-primeng/issues/74
and leave a comment there, because if your comment is not there, your name does not appear and I cannot assign you


[DO NOT MERGE][AUTOREST V3] generated web 2018-02 with autorest v3

Can one of the admins verify this patch?

closing because https://github.com/Azure/azure-sdk-for-python/pull/10214 is the more recent one


Add configuration setting to name query parameter enums also by operation id in c# code generation

When generating c# class from a swagger definition, enums query parameters (e.g. in GET requests) are named by their parameter name. This causes collision if the same parameter name is used across multiple endpoints.
Enums on a model are named from their model, which makes sense. Since these enums are not in any model definition, it does indeed make sense that their names are just the query parameter name.
However, the naming collision is not ideal, and so an idea could be to enable to possibility to name the enums by appending the operation id - this would (as far as I can imagine) make the enum names unique.
Example:
Except of swagger.json
[...]

"api/someobjects": {
  "get": {
    "operationId": "SomeObjects_List",
    "parameters": [
      {
        "name": "OrderBy",
        "in": "query",
        "description": "Which property to order the result by",
        "required": false,
        "type": "string",
        "enum": [
          "FieldOne",
          "FieldTwo"
        ]
      }
    ]
  },

[...]

}

[...]

The generated c# code for the enum would be something like
[...]

public enum OrderBy {
  FieldOne = 0,
  FieldTwo = 1
}

[...]

Having multiple endpoints with an OrderBy query parameter, will result in enums OrderBy, OrderBy2, OrderBy3, ...
I suggest to add a configuration parameter (for example queryParameterEnumNameFromOperationId) that when set to true would generate something like the following
[...]

public enum SomeObjects_ListOrderBy {
  FieldOne = 0,
  FieldTwo = 1
}

[...]

Another solution would be if the rather more complicated - but much more flexible - approach is implemented: The possibility to inject custom naming code, as discussed here: #2458 and here: #2459.

Can you give an example of how you preprocess/transform the spec?

Can you give an example of how you preprocess/transform the spec?

Sorry for late answer. We are in the lucky situation where we control the swagger.json, so we are able to avoid the naming collision, so we decided to do that instead of pre-processing the swagger.json.
It is still my clear view that this issue should be addressed in nswag at some point, since non-breaking changes to an api should not cause breaking changes to the generated c# code (in my opinion, at least).


User Story 8: View history of previously entered commands

As a User, I want to view history of previously entered commands so that I can trace my workflow.

Feature already implemented in initial implementation.
Issue closed.


Canvas Log Viewer Utility does not work with refactored operators

Long story short
Following the description in
https://github.com/tmforum-oda/oda-canvas/tree/main/source/utilities/canvas-log-viewer#canvas-log-viewer-utility
I installed the Canvas Log Viewer Utility.
When executing it, it fails with an error message.
Description
The documentation needs to be updated in two points:

I needed sudo permissions in the npm install: sudo npm install @lesterthomas/kubectl-canvas_logs -g
to get the component-operator the selector should now be "component-operator instead of "oda-controller": kubectl get pods --selector app=component-operator -n canvas

But that is not the issue. I installed a component and wanted to get the logs with the following command:
$ kubectl canvas-logs demo-a-productcatalogmanagement $POD

=================================================
===           ODA Canvas Log Viewer           ===
=================================================
targetComponent: demo-a-productcatalogmanagement
(node:44880) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
Error listing pods: TypeError: Cannot read properties of undefined (reading 'metadata')
    at listPods (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/index.js:23:24)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async mainFunction (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/index.js:47:28)
Canvas Operator: undefined
HttpError: HTTP request failed
    at Request._callback (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/node_modules/@kubernetes/client-node/dist/log.js:42:32)
    at self.callback (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/node_modules/request/request.js:185:22)
    at Request.emit (node:events:519:28)
    at Request.<anonymous> (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/node_modules/request/request.js:1154:10)
    at Request.emit (node:events:519:28)
    at IncomingMessage.<anonymous> (/usr/lib/node_modules/@lesterthomas/kubectl-canvas_logs/node_modules/request/request.js:1076:12)
    at Object.onceWrapper (node:events:633:28)
    at IncomingMessage.emit (node:events:531:35)
    at endReadableNT (node:internal/streams/readable:1696:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  response: <ref *1> IncomingMessage {
    _events: {
      close: [Array],
...
 req: ClientRequest {
      _events: [Object: null prototype],
      _eventsCount: 5,
      ...
	  _headerSent: true,
      _closed: false,
      _header: 'GET /api/v1/namespaces/canvas/pods/undefined/log?follow=true&tailLines=500&pretty=false&timestamps=false&container=oda-controller HTTP/1.1\r\n' +
        'Authorization: Bearer eyJh...ucWg\r\n' +
        'host: <IP_ADDRESS>\r\n' +
        'Connection: close\r\n' +
        '\r\n',
      _keepAliveTimeout: 0,
      ...
    },
    reason: 'NotFound',
    status: 'Failure'
  },
  statusCode: 404
}

I think the relevant part is:
_header: 'GET /api/v1/namespaces/canvas/pods/undefined/log?
  follow=true&tailLines=500&pretty=false&timestamps=false&
  container=oda-controller
  HTTP/1.1\r\n' +

In the same way, as the app label was changed from "oda-controller" to "component-operator" this was also done for the container name.
As I wanted to introspect the secretsmanagementoperator it would be good, if the container name would also be configurable on cli.

Created an alternative implementation using Pyhon:
https://github.com/tmforum-oda/oda-canvas/tree/feature/320-stardardize-logging-secman-depapi/source/utilities/canvas-log-viewer/python_alternative


Tags definition (doubts)

Hi guys, firstly I would like to thanks for this plugin.
Someone can provide some examples ...
How you are using these tags on yours templates ?
Thanks in advance

{reset}{center}This is a centered text! {br}
{reset}{b}And this is a bold left aligned text!{/b}

works like a charm! Thanks @giorgiofellipe


add -k switch to common.sh to resolve authentication issues

yk_loads is missing the -k switch to allow management key to be entered when loading certificates to yubikeys

thanks for the report! closed by re-write in #8


RAS Terrains come in slightly different form

Terrains can also come as a single hdf with supporting tifs in the same directory. I will create a Discussion to demonstrate this.
https://github.com/HydrologicEngineeringCenter/HEC-FDA/blob/b23441b56fe7b5da9950cfb467925fb176ebf06f/ViewModel/Watershed/TerrainBrowserVM.cs#L18

https://github.com/HydrologicEngineeringCenter/HEC-FDA/discussions/349

I left a comment on #349

I believe we decided to tackle this using maintenance money. @rnugent3 is that correct?

Correct.
Best,
R Nugent III

On Jun 3, 2022, at 1:54 PM, codymccoy @.***> wrote:

I believe we decided to tackle this using maintenance money. @rnugent3 is that correct?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

This is completed. Terrains now support .hdf import. When importing from hdf, we store the name of the hdf file in the sqlite database and we copy the hdf file and all the files at that level into our study directory.

@Brennan1994 do we want to import all of the files in the terrain directory in which we find the .hdf? For example, in the Muncie terrain folder within Muncie/Hydraulics/Outputs, the prj, tif, and vrt files are brought over too. I imagine if we don't need the .tif because we have the .hdf, we wouldn't want to add the dead weight.

@Brennan1994 I am not tracking the changes you made regarding this issue. Is there another commit I could take a look at?


mergeWithSpec

The new mergeWithKey function is pretty nice, but with its callback being ternary, it's not as useful if one has more than a single key to deal with. Inspired by the new function and by evolve, I came up with something like this:
  var mergeWithSpec = R.useWith(R.mergeWithKey, [spec => (k, l, r) => spec[k] ? spec[k](l, r) : l,  R.identity, R.identity]);
  var objects = R.repeat({ a: 1, b: 2 }, 10);
  var foldThem = R.converge(R.reduce(mergeWithSpec({ a: R.add, b: R.multiply })), [R.head, R.tail]);
  foldThem(objects);  // => {"a": 10, "b": 1024}

This is far more useful, imho. What do others think?

I know the name is a little problematic given the use of with elsewhere.

I'm wondering if this example might make it more clear:
var combine = mergeWithSpec({min: R.min, max: R.max, vals: R.concat});
var a = {name: 'foo', min: 10, max: 20, vals: [12, 17, 20, 10]};
var b = {name: 'bar', min: 13, max: 25, vals: [18, 25, 13]};
combine(a, b); //=> {name: 'bar', min: 10, max: 25, vals: [ 12, 17, 20, 10, 18, 25, 13]}

I can see the usefulness, I suppose, although I've yet to have a real need for this or for mergeWithKey myself.

isn't this just sugar for mergeWith(where(spec))?

mergeWith doesn't pass the key to the function.

I'm wondering if this example might make it more clear:

My own use case was pretty much something like this: I had "raw" entries in the db that I wanted to fold into a logical combined entries:
var foldEntries = mergeWithSpec({ values: intersection, name: concat, _id: always(null));

Personally, I view this as a replacement for mergeWithKey, rather than a companion. I think that the use case for mergeWithKey is very restrictive. Once there's more than a single key to process, a simple lamda doesn't do the trick, and cond doesn't play well either because both the condition and the consumer need to take care of all three arguments.

ok, well mergeWhere might be a better name anyways

Thanks for the example @CrossEye - i really did not get what was going on before ;)
@buzzdecafe I don't think mergeWhere would fit since this would suggest that there is some filtering on what is being merged where what actually happens is that the caller can specify how values of identical values are being merged.
Then again I just realised i am not sure if only the keys which have been specified will be merged using the provided functions) or everything.
Either way, perhaps mergeWith would be better ? I have to say though that while this seems nice I am not sure I would personally find it interesting enough to include into ramda.

The current behavior:

key only in l or r - value from l or r respectively
key in both, and also in the spec, the result of applying the spec function on the values
key in both, but not in spec, the value from l is used.

@asaf-romano Great idea! I also have a use-case, where I have a list of objects of the same shape and need to aggregate them in a non-trivial way (where R.mergeWith didn't suffice).
I came up with a somewhat different implementation, but it doesn't handle missing props the nice way your implementation does. I'm posting it below just for reference:
const argsToList = R.unapply(R.identity);
const mergeWithSpec = config => R.pipe(
  argsToList,
  R.reduce(R.evolve, config),
);


When token is selected autocorrectionType is not respected.

When a token is selected, the autocorrectionType on CLTokenInputView isn't respected. A few images that demonstrate that below, where you can see that the keyboard's autocorrection bar comes back once the token is selected.

I'm also experiencing this problem, did anyone figure out a fix? My attempts so far an unsuccessful. Thanks.

In case it helps anyone else, I found the fix for autocorrection type on selection... add these lines to the bottom of CLTokenView.m...
-(UITextAutocorrectionType) autocorrectionType
{
return UITextAutocorrectionTypeNo;
}

Hey guys! Sorry I'm so late to this. I'm a bit confused by the issue. You're saying that when a token is highlighted, it should not show any auto-correct options in the keyboard, right? If so, @epocket's suggestion fixes that issue (essentially disables auto-correct when a token is highlighted), and I'm going to add that to the code. It doesn't make sense to offer auto-correct on an already formed token anyway.
Thanks!

Should be fixed on 1.1.0


[DOC] Improve core doc and test_core

Description

python/test/unit/language/test_core.py::test_vectorization is improved to actually compare the src and dst. There was no value comparison before.

python/triton/language/core.py: The comment in the cat function has been updated to clarify that the current implementation only supports can_reorder=True.
If set False or leave as default, compilation error.
Due to
https://github.com/triton-lang/triton/blob/b0faf9b202846434bdd67d7ffec17c8924abe316/python/triton/language/semantic.py#L571

python/triton/language/core.py: The cache_modifier parameter in the load function now includes more detailed information about the accepted values and their meanings, linking to external documentation for further details.

Template
The core Triton is a small number of people, and we receive many PRs (thank
you!).  To help us review your code more quickly, if you are a new
contributor (less than 3 PRs merged) we ask that you fill out the following
checklist and include it in your PR description.
Fill out the checklist by replacing [ ] with [x].

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[ ] I have used an LLM to copyedit my PR description and and code comments.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[ ] This PR does not need a test because FILL THIS IN.
A test case is improved to be more robust.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added are "minimal" -- they contain only the
instructions necessary to exercise the bug.  (Usually running Python code
and using the instructions it generates is not minimal.)

LGTM, thank you!


Getting error while compiling with Xcode 8

Steps to reproduce:
Step 1: Integrate Version 4.0 with Xcode 8 and swift 3 environment
Step 2 : Try to compile and its not able to compile
Actual Result:
The target “Alamofire” contains source code developed with an earlier version of Swift.
Choose “Convert” to update the source code in this target to the latest SDKs. You will be given the choice to use Swift 2.3 syntax or update to Swift 3.
This action can be performed later using “Convert to Current Swift Syntax” in the Edit menu.
Expected Result : As per doc it  should be compatible with Siwft 3.0

Confirm

I was seeing this same error. After updating cocoapods per #1614 and running pod install again, this issue went away for me.

The issue got fixed when i updated cocoapods version to 1.1.0.rc.2.
current Alamofire version is 4.0.1


Create property declarations from constructor parameter properties

TypeScript has a shortcut syntax to declare properties, by using access modifiers on constructor parameters.
I was missing this so support for it is added now.
Covers:

parameter name
type
visibility
@param comment => property description

Cool! Thanks a lot!


docs: mention xdebug_ide_location setting in xdebug troubleshooting section

Xdebug troubleshooting doc update for xdebug_ide_location.
-mike

I fixed the link and added another stanza about DDEV_DEBUG=true ddev start to find out the value of host.docker.internal

Quick followup on this - I just confirmed that when using PhpStorm with xdebug set to be run via "Test frameworks" inside the DDEV web container (using the "DDEV" CLI Interpreter automatically provided by the, that the xdebug-ide-container must be set to an empty string (as described in the doc change for this issue).
The problem I faced which triggered this whole discussion is that I alternate between PhpStorm and VSCode (mainly while teaching) and I had set xdebug-ide-container=container in my ~/.ddev/global_config.yaml - which caused xdebugging of PHPUnit tests in DDEV to fail.
One thing I did notice that is important for xdebugging PHPUnit tests using my setup in PhpStorm is that the "Host" and "Name" in the "PHP | Servers" configuration are the same (and of the form name.ddev.site). This is not the way that the DDEV PhpStorm plugin configures the "server" out-of-the-box. I have documented this in the DDEV plugin issue queue.
I believe that my global setting ofxdebug-ide-container=container was a remnant of previous experiments of trying to get xdebug working from the VSCode UI via the PHPUnit Test Explorer extension without using the "Remote development" extension.


Switched paleo.climatemirror.org to France and success tag

Maybe should just be "Non-US" as I saw above

Yup. Can you change it to Non-US so I can merge it?
Thanks!

OK, all set.

Thanks. Merging.


Passing OpenTelemetry (otel) Env Vars to the Shim Runtime

What is the problem you're trying to solve
As a maintainer of runwasi project, an area I've identified for improvement is the OpenTelemetry collection from containerd-shims. Currently, containerd supports collecting tracing, metrics and logs through OTLP exporters using standard OTLP env vars (e.g. https://github.com/containerd/containerd/pull/8645 and otel config options). But it lacks a similar mechanism for collecting detailed otel telemetry directly from containerd-shims.
Describe the solution you'd like
As discussed in the community meeting at April 24, 2024, we reached a consensus that a good first step to support shim otel is to pass down the OTLP env vars from containerd to the shim.
More specifically, we may want to pass these env vars to the The start command of the Shim
const (
	sdkDisabledEnv = "OTEL_SDK_DISABLED"

	otlpEndpointEnv       = "OTEL_EXPORTER_OTLP_ENDPOINT"
	otlpTracesEndpointEnv = "OTEL_EXPORTER_OTLP_TRACES_ENDPOINT"
	otlpProtocolEnv       = "OTEL_EXPORTER_OTLP_PROTOCOL"
	otlpTracesProtocolEnv = "OTEL_EXPORTER_OTLP_TRACES_PROTOCOL"

	otelTracesExporterEnv = "OTEL_TRACES_EXPORTER"
)

Note: it seems like containerd is already passing OS env vars to the shim binary at https://github.com/containerd/containerd/blob/main/pkg/shim/util.go#L71. This proposal is to make the env vars more explictly in the shim runtime APIs.

Additional context
We have also discussed a few other approaches in the community call:

Envelope the trace span as an event to containerd
Utilize the existing ttrpc communication channels to transport telemetry data. Meaning this will define new APIs for OTEL

The above two approaches all have the same downside - exposing too much OTEL specific concepts to containerd.
I am mostly interested in the performance impact that this method might affect the performance of containerd. I'd love to gather your feedback, additional ideas, or concerns regarding the proposed method.
FYI @cpuguy83 @dmcgowan

This is completed, I think we can close.
@Mossaka Do you agree?

@cpuguy83 can you link the relevant PRs here?  Then we can close.

We were already passing through the otel env vars (which is now the only way to configure otel in containerd).
https://github.com/containerd/containerd/pull/10186 made it so the trace contexts are passed over the rpc interfaces.

yes we can close it now.


warning: lint missing_crate_level_docs has been removed

Describe the bug
A recent commit to master has resulted in this warning which prevents release builds.
warning: lint 'missing_crate_level_docs' has been removed: use 'rustdoc::missing_crate_level_docs' instead
To Reproduce
Steps to reproduce the behavior:

clone this repo locally
cargo +nightly run --release // to see build failure
cargo +nightly run  // to see warnings but still able to run

Expected behavior
A successful release build
Desktop (please complete the following information):

OS: Ubuntu 18.04.3 LTS

Additional context
This was a trivial code fix; I am putting together a PR to address it now.

If this is being handled in #249, please feel free to close this as a duplicate.


node-gyp failed to rebuild

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.2.1
Electron version
25.2.0
Operating system
macOS 13.4.1
Last known working Electron Forge version
No response
Expected behavior
The app should compile.
Actual behavior
The electron app doesn't compile properly. Here is the complete log:
> electron-forge start

✔ Checking your system
✔ Locating application
✔ Loading configuration
✖ Preparing native dependencies: 0 / 1
  › node-gyp failed to rebuild '/Users/myname/Desktop/myapp/node_modules/ca…
◼ Running generateAssets hook

› Package 'gobject-2.0' requires 'libffi >= 3.0.0' but version of libffi is 2.1
  Package 'gobject-2.0' requires 'libffi >= 3.0.0' but version of libffi is 2.1
  SOLINK_MODULE(target) Release/canvas-postbuild.node
  CXX(target) Release/obj.target/canvas/src/backend/Backend.o
  In file included from ../src/backend/Backend.cc:1:
  ../src/backend/Backend.h:3:10: fatal error: 'cairo.h' file not found
  #include <cairo.h>
  ^~~~~~~~~
  1 error generated.
  make: *** [Release/obj.target/canvas/src/backend/Backend.o] Error 1
  Error: `make` failed with exit code: 2
  at ChildProcess.onExit (/Users/myname/Desktop/myapp/node_modules/node-gyp…
  at ChildProcess.emit (node:events:513:28)
  at ChildProcess._handle.onexit (node:internal/child_process:291:12)

An unhandled rejection has occurred inside Forge:
Error: node-gyp failed to rebuild '/Users/myname/Desktop/myapp/node_modules/canvas'
at ChildProcess.<anonymous> (/Users/myname/Desktop/myapp/node_modules/@electron/rebuild/lib/module-type/node-gyp/node-gyp.js:118:24)
    at ChildProcess.emit (node:events:513:28)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12)

Steps to reproduce
I have the latest version of Electron and associated lib. The app works fine on macOS Big Sure, but not on macOS Ventura.
Do I need to globally install a package like node-gyp to make it work?
Additional information
No response

@DoneDeal0 Sorry for the delayed response. Do you have an exact repro available? This seems like an issue with the native module you're using but I'd like to confirm the setup on my end as well.

@erickzhao here is a basic repo: https://github.com/DoneDeal0/electron-build-fail
The issue comes from the package"pdfjs-dist": "3.4.120" (upgrading it doesn't change anything). When I remove it, the build works fine. Sadly, I need this lib.
I have node 18.16.1. The build fails on a machine with an I7 chip, but works fine on a M2 machine. This is very weird. The PDFJS team won't help me because I use their lib in Electron.

@erickzhao hi, have you had time to research this issue?

i have  a  Similar question

electron-forge make

✔ Checking your system
rget/fse/fsevents.o
In file included from ../fsevents.cc:6:
../../../../nan/nan.h:688:39: warning: 'IdleNotificationDeadline' is deprecated: Use…
return v8::Isolate::GetCurrent()->IdleNotificationDeadline(
^
/Users/tangsong/.electron-gyp/28.1.4/include/node/v8-isolate.h:1343:3: note: 'IdleNo…
V8_DEPRECATE_SOON(
^
/Users/tangsong/.electron-gyp/28.1.4/include/node/v8config.h:561:41: note: expanded …
define V8_DEPRECATE_SOON(message) [[deprecated(message)]]
^
../fsevents.cc:14:10: fatal error: 'src/storage.cc' file not found
#include "src/storage.cc"
^~~~~~~~~~~~~~~~
1 warning and 1 error generated.
make: *** [Release/obj.target/fse/fsevents.o] Error 1
Error: make failed with exit code: 2
at ChildProcess.onExit (/Users/tangsong/Documents/cqq/test/demo/node_modules/node-gy…
at ChildProcess.emit (node:events:513:28)
at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)
An unhandled rejection has occurred inside Forge:
Error: node-gyp failed to rebuild '/private/var/folders/6c/44_3vj9n74sdtr04s4nndkr40000gp/T/electron-packager/tmp-JTs2GE/Electron.app/Contents/Resources/app/node_modules/watchpack-chokidar2/node_modules/fsevents'
at ChildProcess. (/Users/tangsong/Documents/cqq/test/demo/node_modules/@electron/rebuild/lib/module-type/node-gyp/node-gyp.js:118:24)
at ChildProcess.emit (node:events:513:28)
at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)

Same problem for me when I installed ssh2 in my electron forge project. Node-gyp can not rebuild cpu-features which is one of the dependencies of ssh2. The environment info is:
cpu: M1 Pro
os: MacOS 14.3.1
node: v18.18.0
electron: ^29.0.1
electron-forge: ^7.3.0
ssh2: ^1.15.0

Same problem for me when I installed ssh2 in my electron forge project. Node-gyp can not rebuild cpu-features which is one of the dependencies of ssh2. The environment info is: cpu: M1 Pro os: MacOS 14.3.1 node: v18.18.0 electron: ^29.0.1 electron-forge: ^7.3.0 ssh2: ^1.15.0

For this problem, I found ssh2-electron-no-cpu-features which do not need cpu-features dependency. And this worked for me. If you using typescript, you can just install @types/ssh2 and rename node_modules/@types/ssh2 to ssh2-electron-no-cpu-features, then you can have fully annotations. Wish this can help you if you have same problem.

Open your lock file
Find the ssh2's dependencies, you can see there the cpu-features are optional, just drop this dependency from there.
my case looks like:

"ssh2@npm:^1.5.0":
  version: 1.5.0
  resolution: "ssh2@npm:1.5.0"
  dependencies:
    asn1: ^0.2.4
    bcrypt-pbkdf: ^1.0.2
-   cpu-features: 0.0.2
-   nan: ^2.15.0
  dependenciesMeta:
    cpu-features:
      optional: true
    nan:
      optional: true
      ...


use isEmpty and isNotEmpty

Follow Don't use length to see if a collection is empty rule of Effective Dart

I thought we had a lint that would catch these. Is it broken or something? How did we miss them?

LGTM but I'm concerned that the lint isn't working.

@pq, any idea why the lint didn't catch these?

@pq, any idea why the lint didn't catch these?

At first no and then I looked closer and it turns out the lint is only checking for: !foo.isEmpty and not the more obvious foo.length != 0.  (The name is actually a tip-off: prefer_is_not_empty.  Looks like we should have a prefer_is_empty too or have an uber-lint that handles both checks.)  Net-net: this is confusing.  We should fix it!

Linter tracking issue opened: https://github.com/dart-lang/linter/issues/363.


fix so that multihashes may be followed by sub-path strings

In the command-line tool you can have a hash followed by a path:
ipfs cat QmXJ8KkgKyjRxTrEDvmZWZMNGq1dk3t97AVhF1Xeov3kB4/meta.json

Using ipfs-api before this patch I would get these errors when I try to do the same thing:
not valid multihash

With this patch, I get the expected output, the same as the command-line tool.

Sweet, thank you @substack :)


coverage: rename NOT_REACHED/NOT_IMPLEMENTED to exclude them from code coverage

gcovr ignores any line containing GCOVR_EXCL_LINE without regard to syntax.
This change makes use of this to automatically exclude the NOT_REACHED and
NOT_IMPLEMENTED macros from code coverage.
Risk Level: low (macro rename)
Testing: manual
Docs Changes: n/a
Release Notes: n/a
Signed-off-by: Stephan Zuercher<EMAIL_ADDRESS>

echo2_config.cc:31:5: error: use of undeclared identifier 'NOT_IMPLEMENTED'
    NOT_IMPLEMENTED;
    ^
1 error generated.

So... I guess keep a version of NOT_IMPLEMENTED marked deprecated until I can update envoy-filter-example and then remove it?

@zuercher for envoy-filter-example, it's up to you, but we can just force merge their a patch to use the new macro, and then go back and update the submodule SHA once this lands?


Leader Lines Connecting Issues

Hi Team can u help in this issue

when ever i close the parent node the leade lines are irrelevant to the position

how can i connect those leader lines to parent node when ever i close the parent node wise versa when ever i expand parent node those need to connect back to child nodes

Hi @Allabakshu, thank you for the comment.
You have to call position method.

No reply came, then this abandoned issue is closed.


Misspelling

This line contains a spelling error:
https://github.com/ansible/ansible-kubernetes-modules/blob/master/tasks/main.yml#L4
Intall should be Install

Fixed. Thanks for catching and reporting this!


Move cursor with table of contents

If you activate the table of contents with tab, the hardware cursor isn't put on the selected item.
This makes it impossible to find your location if using a screen reader or braille display.
The cursor should be able to be moved even while still hidden.

will fix, thanks for reporting

@tspivey hi, could you check master to see if this is fixed? idk how to test on my own.

It works well here, except the cursor isn't moved back to the top after leaving the table of contents.

When rendering a page of text, the cursor is left at the bottom, is that an issue, or does the screen reader automatically wrap around?


[SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2.

What changes were proposed in this pull request?
Adds DeleteSupport mix-in for DataSourceV2. This mix-in provides a method to delete data with catalyst expressions in support of delete from and overwrite logical operations.
How was this patch tested?
No tests, this adds an interface.

Test build #90534 has started for PR 21308 at commit ffbd3cb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3161/
Test PASSed.

Test build #90534 has finished for PR 21308 at commit ffbd3cb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90534/
Test PASSed.

#21888 shows how this is used to implement DELETE FROM.

Test build #94814 has started for PR 21308 at commit db77b9a.

@rxin, I've updated this API to use Filter instead of Expression. I'd ideally like to get it in soon if you guys have a chance to review it. It's pretty small.
cc @cloud-fan

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2220/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2222/
Test PASSed.

Test build #94818 has started for PR 21308 at commit e32e6c4.

Test build #94814 has finished for PR 21308 at commit db77b9a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94814/
Test PASSed.

Merged build finished. Test PASSed.

Test build #94818 has finished for PR 21308 at commit e32e6c4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94818/
Test PASSed.

I am assuming this API was intended to support the "drop partition" use-case.  I'm arguing that adding and deleting partitions deal with a concept that is a slightly higher concept than just a bunch of records that match a filter.  Backing up this fact is the concept that partitions are defined independently of any records they may or may not contain - You can add an empty partition and the underlying state of the system will change.
Also - as an end user I would be very upset if I meant to drop a partition, but because of a transcription error accidentally started a delete process with a filter that didn't directly match a partition definition that takes a million times as long to execute.
  I am wondering if we should represent them in this API as something slightly more higher level then just a filter definition.

@rdblue what about those data sources that support record deletion and partition dropping as two semantically different operations - Kudu and Hbase being two examples.
All systems that support partitions have a different api for dealing with partition level ops. Even file based table storage systems support the different levels of manipulation. (look at the sql DDL that impala supports for parquet partition for an example - they use a filter, but the command is “this partition op applies to the partiton that is defined by this filter”, not “apply this op to all records that match this filter)”
The difference is subtle, but it is an important one, and every system that supports partitions enforces that difference for a reason.

@tigerquoll, there is currently no support to expose partitions through the v2 API. That would be a different operation. If you wanted to implement partition operations through this API, then you would need to follow the guarantees specified here: if you need to delete by partition, then the expression must match records at partition boundaries and reject the delete operation otherwise.

@rdblue Actually:  https://issues.apache.org/jira/browse/SPARK-22389.

@rdblue I think our debate is whether we should expose an API to represent direct operations on partitions in the new datasource api.

@tigerquoll, I'm not debating whether we should or shouldn't expose partitions here. In general, I'm undecided. I don't think that the API proposed here needs to support a first-class partition concept for tables, largely because partitions aren't currently exposed in the v2 API.
The issue you linked to, SPARK-22389, exposes Spark's view of partitioning -- as in repartition(col) -- which is to say data rows are grouped together. That's not the same thing as partitions in a data source that can exist independent of data rows.

@rdblue when you say "you don't think the API proposed here needs to support a first-class partition concept", are you referring to the "DeleteSupport" Interface, or to DataSourceV2 in general?
If you are referring to DeleteSupport, then do you have the same objections to a separate "DropPartition"/"AddPartition" interface?
If you mean that you don't think DataSourceV2 requires supporting partitions as a first-class concept, then how are users of spark supposed to perform operations like

adding,
altering,
removing, and
listing
partitions on those data-sources that are represented by particular instances of DatasourceV2?

@tigerquoll, I'm talking about the DataSourceV2 API in general. I'm not sure if I think there is value in exposing partitions, but I'd be happy to hear why you think they are valuable and think through how it would fix with the existing API.
I think that partitions that aren't hidden make tables much harder for users to work with, which is why Iceberg hides partitioning and automatically translates from row filters to partition filters. For Kudu, maybe it is different. Could you write up the use case with a bit more context about what empty partitions are used for, and send it to the dev list?
If we think that the v2 API should expose a partition concept, then that would definitely include a way to add or drop partitions.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5236/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6382/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15760/
Test PASSed.

The DELETE support is already merged, closing this


Add form-feedback class to avoid "jumping" forms

If I use the .(in)valid-feedback class, there is no built-in possiblity to leave space for the feedback if it is not validated so far. Because of the sibling rule, I cannot embed the .invalid-feedback into a div with a min-height.
Now, when validating a form (only .invalid-feedback, no valid-feedback) with invalid syntax the content below the input jumps down when .invalid-feedback gets active
Thus, I propose to add a form-feedback container to allow reserving some space.
Here my example css code, limited to .invalid-feedback:
/* show invalid-feedback inside form-feedback if form was validated and form control is invalid */
.was-validated .form-control:invalid ~ .form-feedback .invalid-feedback {
  display: block;
}

/* reserve some space for the embedded invalid-form */
.form-feedback {
  margin-top: 0.25rem;
  min-height: 1.25em;
}

/* margin-top is applied by .form-feedback, don' add it again */
.form-feedback .invalid-feedback {
  margin-top:0;
}

ok

Should I prepare the changes and crate a pull request?

No plans to add this ourselves as part of the form layout or feedback styles.


Additional control events for timers

Propose adding two new control events:
timed_pause: which pauses the timer for a specified amount of time, then automatically resumes.
Example is if you have a hurry up timer, but you want to pause the timer while the ball is in the pop bumpers. You want the timer to resume 1s after the last pop bumper hit, but you don't want to just +1s on every hit because that would add too much time. So it's almost a sliding window pause. (Similar to how timed switch sequences work.)
Also we need a way to add time to a timer but to respect a cap, like "add 2 but never go above 15 total" This could potentially be solved with player var limits also

Hey, this issue is pretty old, so i was wondering whether it was actually still open? If so, i might be interested at having a look at it.
Cheers

Hey @scared-fish. Welcome! Looks like this has not been implemented yet. We got pause (https://github.com/missionpinball/mpf/blob/dev/mpf/devices/timer.py#L149) but not timed_pause. So this is still open :-). There is a unit test here: https://github.com/missionpinball/mpf/blob/dev/mpf/tests/test_Timer.py.
Jan

Hi,
I have developed a rather simple approach to the first part of the feature request and was wondering if anyone could confirm that this would work and actually cover the request:
I would add a function to timer.py (timed_pause), which really just does the same thing as the already existing pause function, but instead of calling a delay.add, I would have it call a delay.reset. This way, the pause would reset every time the function is called and resume the timer x seconds after the last call.
Regards, Lars

That sounds very reasonable. Guess we have to stop the delay when disabling the timer. Also maybe on manual resume or pause.

[I sadly did not have much time lately (vacation etc.) to work on this issue much...]
I wanted to write a Unit test for the new function... But first I tried to understand the testing environment (or at least the test_timer.py). But I cannot even find a test for the standard pause() function. Am i missing something, or is this test missing? I kinda wanted to find that test in order to have something to lean on basically...


Implement Open Source Software Release Review Recommendations

Implement the recommendations of the Open Source Software Release Review.

Build is successful including automated tests. The expanded source archive is expected to build and pass tests.
DISCLAIMER file is correct.
Top-level LICENSE and NOTICE are correct for each distribution. See the Licensing How-To, plus various pages under Legal Affairs.
All source files have license headers where appropriate.
The provenance of all source files is clear (software grants). Details on ensuring that software grants and third party contributions are handled correctly can be seen in the DARPA Open Source Policy
Release consists of source code only, no binaries. It is strongly encouraged that each release contains a source package. This package may not contain compiled components (such as "jar" files) because compiled components are not open source, even if they were built from open source.

These are requirements that have been put on XDATA but will eventually apply to Memex. It is easier to start implementing them now. The biggest change here will be to add the license header to the top of each source file.

License, Notice, and Disclaimer have been added. We will need to add the license clause to the top of all source files.