Suggestion: split type into type and status in airports.csv

Right now, there are entries such as this in airports.csv:
"id","ident","type","name","latitude_deg","longitude_deg","elevation_ft","continent","iso_country","iso_region","municipality","scheduled_service","gps_code","iata_code","local_code","home_link","wikipedia_link","keywords"
...
6526,"00AR","closed","Newport Hospital & Clinic Heliport",35.6087,-91.254898,237,"NA","US","US-AR","Newport","no",,,,,,"00AR"

The problem is that closed makes us lose information regarding what type it was before it got closed. In this particular case, the name field makes it possible to figure out it was a heliport.
In other cases, such as this:
"id","ident","type","name","latitude_deg","longitude_deg","elevation_ft","continent","iso_country","iso_region","municipality","scheduled_service","gps_code","iata_code","local_code","home_link","wikipedia_link","keywords"
...
6529,"00CO","closed","Cass Field",40.622202,-104.344002,4830,"NA","US","US-CO","Briggsdale","no",,,,,,"00CO"

there's not enough information.
I'd suggest splitting this up into type (heliport, small_airport, etc.) and status (open, closed) in order to avoid losing this info and make the database more useful in the future, especially for automated processing.

Yes, that's a good idea.  I'll have more time to work on OurAirports in the new year, and I think this would be worth implementing.


Pagination doesn't work at flagrow.io

Steps to reproduce

Go to https://flagrow.io/
Use pagination to navigate extensions
It doesn't work

Expected behaviour
Pagination should show paginated results
Actual behaviour
Nothing, always show same extensions on every page.
Configuration
Browser: FireFox and Chrome

I just fixed this, will deploy the update now.


fix(create): Adding create_sotransaction support

@mrlannigan @trainerbill or @mciocchi adding this missing create_sotransaction since the library does not work for me without it.  If there is another way to accomplish that please let me know.

Hello @coalex,
Thank you for the submission!
I am currently working on a refactor of this module that matches the other SDK's that Intacct is offering in an effort to match the documentation more closely. For this reason, I would prefer to not add to the list for now.
I'm sure this is not likely what you would like to here, but there is a way to achieve your need without a new control function factory method. See below:
const { ControlFunction } = require('intacct-api');

const theFunc = new ControlFunction('create_sotransaction', { some: 'keys' });

For now I will close this and update once I have a refactor update in place.
Thank you.


Implement multi-line comments

Joy supports multi-line comments by enclosing text between (* and *).
I may want to use parentheses for something else in Rejoice, so I may use /* and */ instead.

This was implemented, and then removed.


moleculer-bull: Queue events are called several times simultaneously.

Hi,
i dont know if i do something wrong or if it is a bug, but I am currently confronted with a problem, where the events of the queue cause multiple calls.
Here is some code to reproduce it:
mixins: [
         QueueService(),
     ],

     actions: {
         start: {
             async handler(ctx) {
                 const job = await this.createJob('auto.fight', { blah: 'blah' });

                 const failed = await this.getQueue('auto.fight').on('failed', async (job, error) => {
                    this.logger.error(`An error occured in 'auto.fight' queue on job id '${job.id}': ${error.message}`);
                 });

                 const completed = await this.getQueue('auto.fight').on('completed', async (job, res) => {
                    this.logger.info(`Job with the id '${job.id}' completed.`);
                 });

                 const stalled = await this.getQueue('auto.fight').on('stalled', async (job) => {
                    this.logger.warn(`The job with the id '${job} got stalled!`);
                  });
             }
         }
     },

     queues: {
         'auto.fight': {
             async process(job) {
                 this.logger.info(`Processing queue...`);
                 const accounts = await this.broker.call('v1.users.list');
                 this.logger.info(accounts);
                 return Promise.resolve({ id: job.id, worker: process.pid });
             }
         }
     },

I also tried to use done as return return { done: true , worker: process.pid }
It seems that only the events got fired multiple times the job is processed only once.
best regards
Probber

I think you call the start action multiple times which makes the event subscriptions. Move this code part into the service started event handler, instead.


Missing backslash

on 'Syntax' [log  …] must be [/log …]

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

ID: c311f8b4-82f9-e1cb-9b6a-e284a5fff8cc
Version Independent ID: b0b9374c-2a7f-25be-d9e2-38d26c5047aa
Content: secedit:generaterollback
Content Source: WindowsServerDocs/administration/windows-commands/secedit-generaterollback.md
Product: windows-server
Technology: manage-windows-commands
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

That looks correct, the slash (as opposed to backslash) is missing in the start of the article, but present in the Secedit command.
I will open a pull request to suggest the correction.


BGP support

Following the Helm changes in https://github.com/cilium/cilium/pull/15340, update the CLI accordingly.
See commit msgs.

install: Add update ability for services/status in Operator
install: Update Agent and Operator resources for BGP

Closing in favor of #159, due to GH Actions not being able to run for PRs that are opened from forks.


angular-handsontable includes moment in bundle

Description
angular-handsontable includes moment in the bundle
Steps to reproduce
Our application already includes moment library and angular-handsontable includes another copy of moment.
Demo
https://github.com/ipassynk/handsontable-size.git
Run
yarn build
yarn bundle-report

See that moment is included twice.
Your environment

Wrapper version:
Browser Name and version:
Operating System:

Hi @ipassynk,
this is a result of the fixed version in package.json file of Handsontable.
We will talk about it inside our core team.
Moreover, you can follow this PR: handsontable/handsontable#5047


Blog post on Pricing tables

Isabel and Erin will meet on 4/27 - having their respective parts written. The goal is to have it ready for review and input by the rest of the team by 4/28.

Post is written. Will make an issue for next sprint to work with isable on markdown and a final review by the rest of the team


Could OrderBy be better specified annotatively?

Description
As a:

[ ] Application user/user of the configuration itself
[x] API user (application developer)
[ ] SPI user (container or runtime developer)
[ ] Specification implementer

...I need to be able to:
have a less lengthy and less error-prone way of defining custom repository methods
...which enables me to:
better understand the code that uses my repository methods and
reduce the opportunity for misspellings and mistakes in my repository methods.
Instead of writing queries such as the following that make the code almost unreadable,
List<Employee> findByDepartmentAndIdNotOrderByPositionDescLastNameAscFirstNameAsc(String deptNum, int empNum);

...
coworkers = employees.findByDepartmentAndIdNotOrderByPositionDescLastNameAscFirstNameAsc(deptNum, empNum);

suppose we allowed the sorting information to all be declared annotatively,
@OrderBy(value = "position", descending = true)
@OrderBy("lastName")
@OrderBy("firstName")
List<Employee> findByDepartmentAndIdNot(String deptNum, int empNum);

...
coworkers = employees.findByDepartmentAndIdNot(deptNum, empNum);

The only disadvantage I can think of is if someone wanted to duplicate the exact same query but with different ordering, but it's likely that special case would already be solved by the Sorts parameter that some of the implementations have.

For a more dynamic sort option, the vendors use a parameter.
Spring
Spring: they have a Pageable
Pageable sortedByName =  PageRequest.of(0, 3, Sort.by("name"));

On the interface method:
List<Product> findAllByPrice(double price, Pageable pageable);

Micronaut
It has a similar approach:
Pageable
Pageable sortedByName =  PageRequest.of(0, 3, Sort.by("name"));

Jakarta NoSQL
It has a Sorts
Sorts sorts = Sorts.sorts().asc("name").desc("age");

List<Product> findAllByPrice(double price, Sorts sorts);


Remove version: FreePascal.FreePascalCompiler version 3.2.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/freepascal/files/Win32/3.2.2/fpc-3.2.2.i386-win32.cross.x86_64-win64.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


Delete missing attribute results in KeyError.

Found a small issue with one of my previous changes, if you try to delete an attribute that's already deleted.

thanks!


[Feature]: Support OpenAPI 3.0.x

Tell us about your feature request
It would be nice to have the spec file published as an OpenAPI 3.0.3 file as well.
What problem are you looking to solve?
The tooling around OpenAPI spec 3.1 is quite low. Most generators have problems generating 3.1 spec files. I spent quite a while now yet to generate the a client for this spec in java.
Additional context
No response
How important is this suggestion to you?
Nice to have

I double this, an issue about supporting 3.1 spec in openapi-generator has been open for three years now (link), and I doubt we'll see it implemented any time soon.

Hey folks 👋 Thanks for your patience here. We took a good look at porting our spec back to OpenAPI 3.0 but it's not feasible right now. We rely pretty heavily on features from OpenAPI 3.1, and rolling back to 3.0 would mean a substantial overhaul and ongoing maintenance that we can't commit to.
We're keeping an eye on demand for SDKs, so I've made a note that folks here are interested in .NET and Java 👀
I'm going to close this issue as "unplanned," but feel free to pop by with any extra context or thoughts.

Amazing, so having OpenAPI 3.0 spec is a maintenance you can't commit to, at the same time we can't have a proper SDK for java/.net. That's why you'd better shift the responsibility of dealing with your API (which is 30,000 lines long in OpenAPI spec!) on your customers/merchants who have to re-implement the same piece over and over for each of them.
I don't see any benefits of choosing paddle over stripe (or recently acquired lemon squeezy by them, which is a merchant of record) with these questionable business decisions in consideration.

Hey folks 👋 Thanks for your patience here. We took a good look at porting our spec back to OpenAPI 3.0 but it's not feasible right now. We rely pretty heavily on features from OpenAPI 3.1, and rolling back to 3.0 would mean a substantial overhaul and ongoing maintenance that we can't commit to.
We're keeping an eye on demand for SDKs, so I've made a note that folks here are interested in .NET and Java 👀
I'm going to close this issue as "unplanned," but feel free to pop by with any extra context or thoughts.

I appreciate the extra overhead this may cause for the team. Any option of support only a subset of the api that does not involve any of the v3.1 features?
In absence of v3.0 support, is there any alternative that may help us support the API without custom coding it ourselves in dotnet?
Right now, I've resorted to mapping the json response examples from the docs using https://quicktype.io/csharp with very loose settings (all props are optional etc) and manually fixing some of the missing values.

Here is a file that I downgraded to v3.0 manually at 29 May. I wrote down the steps which you may repeat yourself if needed (2nd file in gist).
https://gist.github.com/ayhanap/1dd580d4d88527921b46e50f9c328c27


[SME] Introduce scalable fp32 dense schedule

This commit adds a new scalable fp32 dense schedule that calls SME intrinsics according to the SME RFC: https://github.com/apache/tvm-rfcs/pull/107.
Currently the schedule does not make use of predication, meaning the output from the matmul compute must be copied in a subsequent compute stage. This will be removed once support for predication is added.
cc @ekalda @Anndrey24 @tqchen @cbalint13 @Lunderberg

@tvm-bot rerun

Retriggering CI since it hasn't been run for a while

thanks @Anndrey24 @leandron!


Almost read in data

Hey @AaronGullickson -- I'm so painfully close to reading in my data. It's in csv file format in my raw data folder. My only issue is RStudio seems very upset about my working directory-- I'm very sure I have it set in the right place (but I guess not)

Yes, this was a discovery about quarto. It always wants to run from the directory of the file not the working directory unless we set that as an execution parameter. I am going to fix this in the template, but the easiest solution to this problem is just to use the here package, which is best practice anyway:
cattle_pop <- read_fwf(here("data","data_raw","cattle_numbers.csv"))
cattle_loss <- read_fwf(here("data","data_raw","cattle_loss.csv"))
wolf_data <- read_fwf(here("data","data_raw","wolf_data.csv"))


[Docs]: Disable overriding the ssh key config through identityFile to avoid git auth failure

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
https://docs.appsmith.com/core-concepts/version-control-with-git/connecting-to-git-repository
Describe the problem
When user is working with local development ssh config file can have default ref for IdentityFile mapped to
~/.ssh/id_ed25519 as per latest security guidelines followed by platforms like github, gitlab etc. This creates a problem to connect the appsmithb application with git remote provider, as we generate the SSH public and private key using Jgit library and use these keys to authenticate the communication with the remote repo. To avoid overriding the keys generated by Jgit we need to comment or remove the identity file reference from the Host setting.
Describe the improvement
Sample file content of /.ssh/config:
Before:
Host *
  AddKeysToAgent yes
  UseKeychain yes
  IdentityFile ~/.ssh/id_ed25519

After:
Host *
  AddKeysToAgent yes
  UseKeychain yes
  #IdentityFile ~/.ssh/id_ed25519

@vuiets - Is this ready for doc team?

@jnikhila do you have enough to go on or do you need a draft for this?

@vuiets - A draft will be helpful.

@abhvsn hey this slipped from our radar. If this is not related to adding anything in the instance configuration files, then ideally this shouldn't reside in the product docs. Could you confirm where this setting is to be added.

Closing this as it's stale, will reopen if required.


Fix oqc qasm

It was meant to solve #95 but actually the problem was never present for the get_one_benchmark method since that method always returns the Qiskit::QuantumCircuit object. Nevertheless, the OQC error handling is improved:

It is done now immediately after creation.
The opaque function of OpenQASM is now used to avoid introducing non-native gates in the native and mapped levels.

This still fixes #95, right?

This still fixes #95, right? If so, then I'd link the issue to this PR so that it gets closed automatically once this PR is in.

Actually no, because #95 was never a problem, since the ECR problem happens only when the qasm files are saved. The get_one_benchmark method is not saving the qasm files but just returning the Qiskit/TKET Circuit object. But nevertheless, this PR improves the postprocessing since it is now done automatically after the file is created.
What do you mean by "linking"? Can you do it explicitly or is it done by just referencing somewhere to the issue?

This still fixes #95, right? If so, then I'd link the issue to this PR so that it gets closed automatically once this PR is in.

Actually no, because #95 was never a problem, since the ECR problem happens only when the qasm files are saved. The get_one_benchmark method is not saving the qasm files but just returning the Qiskit/TKET Circuit object. But nevertheless, this PR improves the postprocessing since it is now done automatically after the file is created.

Yeah, got that. I just meant that the issue can be closed (since it is not an issue after all).

What do you mean by "linking"? Can you do it explicitly or is it done by just referencing somewhere to the issue?

On the right side of the PR page here under "Development" you can link any issues that are automatically closed once this PR gets merged. This can even be accomplished automatically when the PR description contains certain closing keywords. See https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue for further explanations.


Ci cd

Adds automatic version bumping based on commit messages and automatic publishing of the package. This means we need to stick to the following commit message pattern from now on
https://github.com/angular/angular.js/blob/master/DEVELOPERS.md#commits

@glencoe I am not sure if this is correct, the Bump Version and Release (Semantic Release) Workflow fails for me with this error message:
Building elasticai.creator (0.3.8)
  - Building sdist
  - Built elasticai.creator-0.3.8.tar.gz
  - Building wheel
  - Built elasticai.creator-0.3.8-py3-none-any.whl
Uploading to artifact Repository
debug: _handle_credentials_init(ArtifactRepo(repository_name='pypi', repository_url=None, dists=[]))
error: Missing credentials for uploading to artifact repository

It looks like there missing credentials for the PyPi repo?
And as I understood what changed now, is that we need to write commit messages in the form of <type>(<scope>): <subject>
and it increases automatically the version number and publishes to the PyPi repo. Is this correct or is there a misunderstanding here?

@glencoe For example to set up the pre-commit stuff I need to have pre-commit installed (https://pre-commit.com/) and run pre-commit install. Is this correct? And what else need to be set up or take care of while developing? What is with isort? Is this automatically installed by pre-commit?

@julianhoever had to create the pr to test some of the workflows... It is not yet finished, and i'll add the corresponding documentation

@glencoe Okay. Thank you


LandmarkNamePartTable Issues

Description

For the LandmarkNamePartTable, the “NGUID” field is nullable and non-required in the geodatabase script but the standard has the field as required.

In both the geodatabase and PostgreSQL scripts for the LandmarkNamePartTable, the “LMNP_Order” field domain “LandmarkNamePartOrder” has a width of 2 and the standard calls for the field width to be 1.

For the LandmarkNamePartTable, the “NGUID” field is nullable and non-required in the geodatabase script but the standard has the field as required.
Confirmed in schema_fgdb_v2.py.
In both the geodatabase and PostgreSQL scripts for the LandmarkNamePartTable, the “LMNP_Order” field domain “LandmarkNamePartOrder” has a width of 2 and the standard calls for the field width to be 1.
The working group should probably discuss this issue. Generally, field widths only apply to character fields (e.g., VARCHAR, TEXT, etc.) and not numeric fields (e.g., SHORT, INTEGER). We used what was defined in v1 which includes a domain constraint of values between 1 and 99. Below is the field description from NENA-STA-006.2-2022, which only sets a domain of whole numbers but not an actual range. My assumption is that v1 may have determined the range. @jasonhorning - We may want to add this to our items to discuss.
5.49 Landmark Name Part Order
     Description: The order in which to concatenate Landmark Name Parts where 1
     is the first (or leftmost) Landmark Name Part, 2 is the second Landmark Name
     Part, 3 is the third Landmark Name Part, etc.
     Domain: Whole numbers starting at 1
     Example: 1; 2; 3

LandmarkNamePartOrder Domain issue is resolved.


slidesGrid values are calculated incorrectly when slidesPerView = auto

See example. You can open the left bar with a long swipe, but cannot open the right slide with the same action due to this bug.
Looks like this code produces error.
for (var i = 0; i < s.slidesGrid.length; i++) {
    if (- translate >= s.slidesGrid[i]) {
        slideIndex = i;
     }
}

http://codepen.io/anon/pen/YXLwxw

This code is correct, and i CAN open right panel with long swiper, not sure what issue did you mean?

When you pull left slide long enough it stays put. But when you pull the right one it doesn't.

Ah, i see what you mean now. It is because with long swipes it looks on the 1/2 distance of current slide (orange one), so you have to move it more than on half to switch to next (right) slide. You can tweak it using longSwipesRatio parameter

Closing as not further response & discussion


docs: add section when vulnerabilities are found with our script

Description
Document what do to when vulnerabilities are found with our script.
Tested
N/A
How others should test
N/A
Related issues

Discussed on Slack.

Backwards compatibility
Yes

@mergifyio refresh


Ruby logger device

When using this gem I was surprised for example that formatter does nothing.
Wouldn't it be better if I could do something like this:
gelf_logger = GELF::Logger.new(graylog_host, graylog_port, 'LAN', defaults)
logger = ::Logger.new(gelf_logger)
Was there any reason not to do it this way?

+1 on that. I'm going through the same issue. That's bad because you can't compose gelf logger other formatters such as ActiveSupport::TaggedLogging::Formatter.
I'll try to monkey patch here and see how it goes.

I was able to integrate with the following patch:
  class GELF::Logger
    alias_method :original_add, :add
    def add(level, message = nil, progname = nil)
      if message.nil?
        if block_given?
          message = yield
        else
          message = progname
          progname = default_options['facility']
        end
      end

      original_add(level, formatter.call(level, nil, progname, message), progname)
    end
  end

That's obviously ugly, but it works. I'll open a PR with an slightly better solution.


Fix spacing issues in various components

For detailed info on spacing in components, see this Figma file: https://www.figma.com/file/MGKwAH7UbuGUuOQjzCF9WA/ILO-Components-Development-Handover-LIVING?type=design&mode=design&t=cfUPujYWuoFk5uC3-0

[ ] Multilink card: revise all spacings in the multilink card, including top, left, right and bottom padding; before/after heading; before/after image; before/after description text
[ ] Stats card: Space between last card and CTA should be 64px on desktop and 56px on mobile

[ ] Profiles: align 'Learn more' button to name and title

[ ] Card with short description: fix space between eyebrow, title and description; fix space between image and grey line.

@inesdgomes could you please update this ticket?

@justintemps I updated this ticket. I suggest you tackle the tickets below first as they may partially resolve #469. Once the ones below are addressed, we can come back to this one.
Cards' category
Content category
Feedback category
Forms category
Media category
Navigation category
Transitions and user interface categories

@bashlk maybe i could pickup some of the tickets assigned above :)

@justintemps Can we tackle the issues above within this sprint?

@bashlk yea, I'm adding them to the next sprint.

Hey @Shashika6 could you tackle this one next?


Add a feature flag for reverse proxy metrics collection

We should add a feature flag in capsule proxy to enable/disable the collection for metrics using the metrics middleware

Hi @viveksyngh, do you still intend to implement this feature? Do you need any assistance? :)

I remember we had some issues while implementing the metrics middleware, that led us to evaluate a toggle for the metrics.
Not sure if it's still needed, tho.

Stale


Use .title for search classes or string.capitalize

Based sample-db #41 use a string.title function to compare if a class already exists in database or string.capitalize

closed id 2318b12ed3c24733b8def927cdd6323afb3a7eb7


Check why unlock emails aren't being sent

AE emails not being sent either.
Project invite emails do work.
May have something to do with ActiveJob not being setup properly, or with deliver_later not working as intended within forked processes.

Looks like skipping ActiveJob fixes the issue (changing from deliver_later to deliver_now), however will still look into getting ActiveJob set up correctly as well as a better fix.


[$250] Ensure newly added report actions allow threads being created on them

👋 Coming from here, we'd like someone to look into an automated solution to avoid running into this again.
We've added a bunch of report actions in recent months, and we'll be continuing to do that as we bring more functionality over into NewDot. As we've built these report actions we've intermittently shown the reply in thread button in the comment action menu on them. That created a situation where some report actions allowed threads while others didn't, leading to an inconsistent and confusing experience with no rhyme and reason to it.
@tgolen outlined some options here on how we can better catch that in development going forward to make sure we uphold this consistency for threads. That's a good place to start and we'd like someone to explore the options further to put this safeguard in place.
Thanks!
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014617a7e47bd69ebb
Upwork Job ID:<PHONE_NUMBER>887613440
Last Price Increase: 2023-11-20
Automatic offers:

s77rt | Contributor | 27807672

Switching out the Bug label now this has an Upwork job.

Taking this as C+

Done!

@dukenv0307 Thanks for the proposal. Your RCA is correct. However the suggested solution will not prevent similar bugs from occurring but only reduce them.

When we have a new action, if this action can have Reply in Thread option, no need to change anything. If not we only need to add this to shouldReplyInThread.

We could be adding a new action that should not have the reply in thread option without adding it to the shouldReplyInThread function since there is no enforcement.

@dukenv0307 This is not looking any different.

In the further when a new action is added, we only need to add this to NOT_ALLOWED_THREAD const if it doesn't has thread option

We can still add a new action and forget it to NOT_ALLOWED_THREAD

@s77rt is working through proposals, Melv.

Not overdue. Still looking for proposals

Not overdue. Still looking for proposals

Still looking for proposals

Same ^

@dukenv0307 any thoughts?

Still looking for proposals

So what's the plan here? One of Tim's suggestions was this:

Maybe better to change the logic for Reply in thread which wouldn't require it to be updated for every new action (for example, rather than checking for all the actions you CAN reply to, only check for the ones you CANNOT reply to, or vice versa, whatever)

I do feel like checking for ones you can't reply to are better than needing to remember to add a check for the ones you can reply to. I think the ones you can't reply to should be super minimal.

@trjExpensify That's what @dukenv0307 proposed as well however it's not full proof as it only "inverses" the bug. The full proof solution is to use a configuration file (the 4th suggestion from Tim's list).

I think the ones you can't reply to should be super minimal

They are not, but the new report actions that we may add are more likely to be thread-enabled.
cc @tgolen for visibility

Yeah, but I think given that by and large almost all reportActions should be thread-enabled we vastly reduce the likelihood of introducing the inconsistency again. You have to be explicit to make it not threadable, and for that, you need to have a reason to consider doing it.

Makes sense. Let's go with @dukenv0307's proposal.
🎀 👀 🎀 C+ reviewed
Link to proposal

Sounds good! Assigning @dukenv0307

Woof, good question! I can think of these two off the top of my head for the [Deleted message] parent comment and the split action preview. Perhaps the "uploading attachment..." action as well?

Err.. no idea why Melv did that. Dropping @anmurali off!

Okay, so no regression checklist for this. It's a follow-up improvement in the code so we can hopefully minimise the amount of non-threadable report actions that get added.
Confirming payments as follows:
@s77rt - $250 for the C+ review
@dukenv0307 - $250 for the fix

@dukenv0307 - paid!
@s77rt - looks like you denied the offer. Anything to note there, or should I send you a fresh one?

@trjExpensify Can you please send a new offer. I thought Melvin would send two offers when @dukenv0307 got assigned. I declined the first one as it had the wrong role.

Sent!

Accepted! Thanks!

Paid!


feat(emacs)!: update script to latest spec

Updates the pacscript to the latest PKGBUILD spec
https://aur.archlinux.org/cgit/aur.git/commit/?h=emacs-git&id=b9eb3b405717f8df37b38217dbb7d361b8ffca08
Closes #1303

@JonBoyleCoding Please test if this works.

Also @Henryws this is a breaking change for the pacscript as a lot of deprecated features have been removed from it

@Henryws The CI doesn't work, jitterbit fails.

Sorry I haven't had the time to test this out. I won't have time until end of next week to do anything on the side - if you still need some help with this by then I can have a look then.

@wizard-28 can you look at the reviews?

For Ubuntu:
emacs : Depends: libgtk-3-dev but it is not going to be installed
         Depends: libwebp6 but it is not installable

For Debian:
 emacs : Depends: libjpeg-turbo8 but it is not installable
         Depends: libwebp6 but it is not installable

libwebp7 seems to be present in ubuntu:latest, but not libwebp6
libgtk-3-dev is present in ubuntu:latest, but not in ubuntu:devel and ubuntu:rolling

@Henryws what to do?

Maybe use incompatible

Maybe use incompatible

Should I add OS-specific dependencies?

Sure


'va_copy' not declared with --std=c++03

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Ubuntu 17.04 64bit
g++ (Ubuntu 6.3.0-12ubuntu2) 6.3.0 20170406
Issue description:
Build error:
core/io/logger.cpp: In member function 'virtual void RotatedFileLogger::logv(const char*, __va_list_tag*, bool)':
core/io/logger.cpp:181:28: error: 'va_copy' was not declared in this scope
   va_copy(list_copy, p_list);
                            ^
core/io/logger.cpp: In member function 'virtual void CompositeLogger::logv(const char*, __va_list_tag*, bool)':
core/io/logger.cpp:235:28: error: 'va_copy' was not declared in this scope
   va_copy(list_copy, p_list);
                            ^

Steps to reproduce:
Run scons with CCFLAGS=--std=c++03

cc @endragor


Bound check for operations with memref

Is your feature request related to a problem? Please describe.
Currently, elements accessing operations on memref do not provide any bound check, which I think there should be some check. For example, even this example below can successfully pass lowering/translate/run:
// Use global memref to ensure they are continuous in memory, for more
// straight forward to show the result.
memref.global "private" @region_gv : memref<4xi32> = dense<[0, 1, 2, 3]>
memref.global "private" @canary_gv : memref<12xi32> = dense<202>
// 202 == 0xCA, means canary

func.func private @printMemrefI32(memref<*xi32>)

func.func @main() -> i32 {
  %region = memref.get_global @region_gv : memref<4xi32>
  %canary = memref.get_global @canary_gv : memref<12xi32>

  // ===================== test code ============================//
  %c1 = arith.constant 1 : i32
  %c5 = arith.constant 5 : index

  %region_print = memref.cast %region : memref<4xi32> to memref<*xi32>
  %canary_print = memref.cast %canary : memref<12xi32> to memref<*xi32>

  // NO ANY warning here! This type of access should be obvious for compiler.
  %val = memref.load %region[%c5] : memref<4xi32>
  vector.print %val : i32

  memref.store %c1, %region[%c5] : memref<4xi32>
  func.call @printMemrefI32(%region_print) : (memref<*xi32>) -> ()
  func.call @printMemrefI32(%canary_print) : (memref<*xi32>) -> ()

  %ret = arith.constant 0 : i32
  return %ret : i32
}

I have written more examples for this problem on my forked repo.
Describe the solution you'd like
A warning should be given when trying to lower/translate/run files contains out of bound access when an analysis is possible.
Describe alternatives you've considered
AFAIK, there are three ways we can do better about this:

A pass may be added for bound checking. It seems easy to at least analyze the simplest cases (constant out-of-bound index for fixed length memref). And by keeping as long as the layout info inside or even between functions, we may also be possible to find out erased bound info like this examples.
A pass to insert runtime bound check for every load/store or other operation on dynamic length memref. This is possible becasue striped layout dynamic length memref will be lowering with lengths in each dimension.
Just leave it over, and let LLVM or the lower-level compiler check this. They should be able to detect gep or pointer out-of-bound operations.

Option 1 seems great, but it may "reinvent the wheel" by re-implementing the common check which is usually done at LLVM IR level. Option 2 is very costly in runtime, but having such a mode in debugging seems great. Option 3 will delay the time that such errors are detected, and make it harder to map the error back to the source language.
Additional context
Since most MLIR are created by compilers with higher level languages that may already ban unsafe memory operation, the bound check may be pushed up to be done before lowering to memref dialect or pushed down to be done after lowering to LLVM. The necessity of bound check in memref dialect level may need a deeper discussion.
In the current implement, the global memref will be continuous in memory, so it is the easiest way to see the effect of out-of-bound accesses. Stack allocated or normal memory allocated memref is not continuous in memory, which can be observed by reading printMemRefI32's output by mlir-cpu-runner.

I think this problem should be solved at its core i.e inside the memref dialect itself. Did you discuss this in official mlir forums? Changes related to core mlir dialects (like memref dialect) are out of scope for this project.

@Origami404 , what is the status of this issue? Can we safely close it?

No progress so far, can be closed now.


Update run_integration_tests.yml

Fixes #
Test Plan

How do we know the code works?

.
Checklist

[ ] Documented
[ ] Unit tested
[ ] Integration tests updated

My comment

another comment


fix: reset unmounted state for the component

When Excalidraw is used as a dynamic component with useEffects it gets unmounted and mounted back multiple times. componentWillUnmount sets this.unmounted to true which breaks the functionality because it blocks syncActionResult. This fix ensures that this.unmounted is reseted to false

cc @ad1992 as it related to the embed component.

@rulikrulit since this is related to the package, can you also update changelog under Fixes ?

merging 🎉, thanks @rulikrulit for fixing this!


[bugfix] Remove "based on" from READMEs

What
I noticed this while reviewing #87
We're not very good at maintaining the "Based on XXX image" messages in the
READMEs of each of our containers. I don't think there's much value in
providing this, since the correct information is the in the Dockerfile and
people should look there before deciding whether a container is appropriate
for their use.
➜  paas-docker-cloudfoundry-tools git:(master) for i in */README.md; do ack Based ${i} && ack FROM $(dirname $i)/Dockerfile; echo; done
Based on [alpine](https://hub.docker.com/_/alpine/) image.
FROM governmentpaas/curl-ssl

Based on the [ruby:slim](https://hub.docker.com/_/ruby/) image.
FROM ruby:2.2-alpine

Based on the [upstream image](https://hub.docker.com/r/relintdockerhubpushbot/cats-concourse-task/), which is
FROM ubuntu:trusty

Based on the [wheezy](https://hub.docker.com/_/debian/) image.
FROM alpine:3.3

Based on the [ruby:slim](https://hub.docker.com/_/ruby/) image.
FROM ruby:2.2-alpine

Based on [alpine](https://hub.docker.com/_/alpine/) image.
FROM alpine:3.3

Based on [alpine](https://hub.docker.com/_/alpine/) image.
FROM alpine:3.3

Based on the [ruby:alpine](https://hub.docker.com/r/library/ruby/tags/) image.
FROM ruby:2.3-alpine

Based on [alpine](https://hub.docker.com/_/alpine/) image.
FROM alpine:3.3

Based on the [wheezy](https://hub.docker.com/_/ruby/) image.
FROM ruby:2.2-slim

How to review
Confirm that the documentation changes make sense.
Who can review
Not @dcarley

I remember once updating README to indicate that some image was now based on alpine... And I wondered then already, why need for that if it's in the Dockerfile...


Check for kubectl.exe on Windows

On the various flavors of bash for Windows kubectl won't resolve as the binary is kubectl.exe.
Simple aliasing doesn't seem to work. So test for both otherwise fail with a not found error.

I signed it!

This looks good, let's hold off until I fully understand why bash isn't able to find  kubectl.exe. I'll probably move the logic outside main() (to the beginning of the script).

Bash does find it if it is fully specified. Here is a scenario.
Docker Desktop is installed on Windows 10 and docker and kubectl are exe binaries installed in some path on Windows. The user is choosing to run Bash in WSL and append the Windows path so that they don't have to do the hack where they also have to install docker and kubectl in the WSL side and point them back to the Windows side (via copying kube config or setting DOCKER_HOST).
In a WSL bash shell, kubectl.exe is available kubectl isn't because the executable is in fact namedkubectl.exe on the windows side. On the windows side, the cmd or PowerShell take care of not having to add the .exe. The way I hack this now is to hijack bash to make symbolic links so that something named kubectl is available in the path. The specifics of that hack are here: https://github.com/ahmetb/kubectx/issues/97#issuecomment-435277728

Tested, looks good. Thanks a lot.


Openshift/bump130224

Align with upstream
align the crds
add description to the crds
update the provider to Red Hat
Merge upstream

/lgtm

Only the frr-k8s test failed, which is not relevant
/override ci/prow/operator-e2e

/override ci/prow/operator-e2e


Fixed gitter badge

Changes

Fixed gitter badge on README

@hijarian I fixed gitter badge. Could you check it when you have time?

@is2ei Thank you for a contribution! 👍


SDL2: 2.0.14 -> 2.0.20

Motivation for this change
https://github.com/libsdl-org/SDL/releases/tag/release-2.0.20
Additionally, for reasons not entire clear to me, this change fixes vulkaninfo for me on NixOS master + Intel graphics.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

See also #136166.

There is already another PR for the update: https://github.com/NixOS/nixpkgs/pull/136166


Left Sidebar changes

closes #61
closes #58
closes #59

This will make the left sidebar background grey, add a border and highlight deeply linked pages.

@gtema
I now removed the greyish look (I also didn't like it, but meeting set we want that) and just added a border.


[CP] Fix Android platform view creation flow

issue_link
https://github.com/flutter/flutter/issues/107297
Commit Hash
ab23233
Target
beta
pr_link
https://github.com/flutter/flutter/pull/109544
Impacted Users
All clients of Android plugins using Hybrid Composition (including the 1P google_maps_flutter)
Impact Description
Assertion failures, performance issues, and possible crashes on platform view creation due to double-create.
Workaround
Only to for plugin authors to stop using Hybrid Composition, which causes other potential issues.
Risk
medium
Test Coverage
yes
Validation Steps

Check out the flutter/plugins repository.
Edit packages/google_maps_flutter/google_maps_flutter_android/lib/src/google_maps_flutter_android.dart to replace PlatformViewsService.initSurfaceAndroidView with PlatformViewsService.initExpensiveAndroidView. (This is a change we will make and publish to the plugin shortly to work around a regression in Flutter 3.0, but are holding off on until this is fixed on beta).
cd packages/google_maps_flutter/google_maps_flutter_android/example
Launch an Android device or simulator.
flutter run on that device/simulator.
Look for (lack of) assertion failures about futures that have already completed.

Because it changes some logic around the creation flow of Android platform views, it could potentially regress other aspects of Android platform view creation (including for non-hybrid-composition plugins). For instance, the first version of the PR broke display of the google_mobile_ads plugin, which was caught by google3 testing but not Flutter testing.

That is fair, thank you for clarifying.  If we get this into this weeks beta, I think we will have sufficient signal from google3 before landing.

I approve


Starting timer using todoist extension, ending with desktop app issue

Environment details

Operation system: OSX
Extension version: 1.9.0

Expected behavior
Stopping timer using desktop app should remove "stop timer" button from todoist
Actual behavior
"Stop timer" button persists in todoist after tracking stopped using desktop app.
Pressing "stop timer" button in todoist overwrites the end time of the entry with the current time.
Steps to reproduce the behavior
Start a timer using the todoist extension, stop the timer using the desktop app
Note the end time of the entry.
Notice the todoist "stop timer" button is still present
Wait 60 seconds or more.
Press todoist "stop timer button"
The previous end time for the entry will be overwritten with the current time
Additional details
In the screenshot, notice the timer is stopped, but the todoist button persists

screenshots or screencapture

duplicated of https://github.com/toggl/toggl-button/issues/300


Travis: fix builds on mac os

Coverage decreased (-0.5%) to 92.116% when pulling 70feebec7de3b09a6f088fabac2b08bcf3f0a683 on iMichka:python3fix into d4ad0ad71418fd9c728dc75228a7c3b6ef800666 on gccxml:develop.

Fixed by a9157dde7ce5da91f5ba40f400320945e303ef47


Step is missing while running the application

During the local application's execution, it offers the sign-in option only which is not mentioned in the documentation.
Additionally, kindly add the screenshot illustrating the starting point of the sign-up flow.

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

ID: 1a40d334-e5fc-a38d-a7d8-40b3f41688b7
Version Independent ID: 9a0b5dcb-4532-3842-8125-455cbdb42e09
Content: Sign in users in a sample vanilla JavaScript single-page application - Microsoft Entra
Content Source: articles/active-directory/external-identities/customers/sample-single-page-app-vanillajs-sign-in.md
Service: active-directory
Sub-service: ciam
GitHub Login: @OwenRichards1
Microsoft Alias: owenrichards

@shwetamathur7 Thanks for the feedback - have addressed and merged in the following PR:
https://github.com/MicrosoftDocs/azure-docs-pr/pull/248553
If there are any other issues, or anything I may have missed - please feel free to let me know!

@OwenRichards1 thank you for addressing this feedback. I'll go ahead and close the issue now.
#please-close


Add API to view FxOS Add-ons in the review queue (bug 1196225)

https://bugzilla.mozilla.org/show_bug.cgi?id=1196234
Bonus: also start implementing publish/reject APIs (bug 1196234)

Thanks, docs are fixed now.

r+wc/t

Added CORS and CORS tests


python lstm.py always needs a mode argument

https://github.com/mila-udem/platoon/blob/master/example/readme.txt says that after I ran the first lstm.py as lstm.py init, I should run other workers. I does not say with which mode, and from the code it seems even lstm.py abacaba works.
I can make a PR that would make mode handling more consistent.

Sure if you want to do it.  I made it as experimental code so there are some rough edges.

This should be addressed by #19 .

PR #29 fixed this.


Add AArch64 Neon implementation of MCTF motionErrorLumaFrac8

Add a new Arm specialization for MCTF including initialization, and wire up the m_motionErrorLumaFrac8[1] member to a new Neon kernel.
Running a video encoding job on a Neoverse V2 machine using the --preset=fast setting, this shows a ~5.2% improvement in reported FPS.

5% fps improvemets sounds amazing. Please feel free to include yourself in the AUTHORS.md list for this quite substantial contribution. I'll give 2-3 days for you to consider and than merge.
Thanks

5% fps improvemets sounds amazing. Please feel free to include yourself in the AUTHORS.md list for this quite substantial contribution. I'll give 2-3 days for you to consider and than merge.
Thanks

Added myself to AUTHORS.md, Thanks!


[BugFix] New anltr parser not support broker load without broker

Signed-off-by: xyz<EMAIL_ADDRESS>What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #9847
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

In #9049, we support broker load without broker in old cup parser,
after changing to new anltr parser in #8965, this grammar is not support.
In this pr, we add support for broker load without broker in anltr

run starrocks_admit_test


Update to Unbound 1.17.1 RC1 (potentially)

The first Release Candidate for Unbound <IP_ADDRESS> is out, with some 'minor' impact, focusing mainly on:

Bug Fixes
Additional config options that do not impact the normal running of existing configs (a.o. an option to keep cache upon Unbound reload)

Detailed message from the NLnetlabs team:

Hi,
Unbound 1.17.1rc1 pre-release is available:
https://nlnetlabs.nl/downloads/unbound/unbound-1.17.1rc1.tar.gz
sha256 4686cd98222cd7580606a07f4f20c15d531634e1ae34e9dad767e4e12609b61f
pgp https://nlnetlabs.nl/downloads/unbound/unbound-1.17.1rc1.tar.gz.asc
This release fixes a number of bugs. There are also new configuration options that by default do not change the existing behaviour of Unbound.
With statistics-inhibit-zero the printout of zero values by stats can be controlled. Similarly with max-sent-count and max-query-restarts the iterator behaviour can be controlled. The maximum CNAME chain length that is accepted can be changed by increasing the max-query-restarts number. This takes more time to follow those elements.
The keep-cache option allows reloads to change configuration whilst keeping the cache memory intact, making the cache hot for good response times after the change has completed.
Features

Expose 'statistics-inhibit-zero' as a configuration option; the
default value retains Unbound's behavior.
Expose 'max-sent-count' as a configuration option; the
default value retains Unbound's behavior.
Merge #461 from Christian Allred: Add max-query-restarts option.
Exposes an internal configuration but the default value retains
Unbound's behavior.
Merge #569 from JINMEI Tatuya: add keep-cache option to
'unbound-control reload' to keep caches.

Bug Fixes

Merge #768 from fobser: Arithmetic on a pointer to void is a GNU
extension.
In unit test, print python script name list correctly.
testcode/dohclient sets log identity to its name.
Clarify the use of MAX_SENT_COUNT in the iterator code.
Fix that cachedb does not store failures in the external cache.
Merge #767 from jonathangray: consistently use IPv4/IPv6 in
unbound.conf.5.
Fix to ignore tcp events for closed comm points.
Fix to make sure to not read again after a tcp comm point is closed.
Fix #775: libunbound: subprocess reap causes parent process reap
to hang.
iana portlist update.
Complementary fix for distutils.sysconfig deprecation in Python 3.10
to commit 62c5039ab9da42713e006e840b7578e01d66e7f2.
Fix #779: [doc] Missing documention in ub_resolve_event() for
callback parameter was_ratelimited.
Ignore expired error responses.
Merge #720 from jonathangray: fix use after free when
WSACreateEvent() fails.
Fix for the ignore of tcp events for closed comm points, preserve
the use after free protection features.
Fix #782: Segmentation fault in stats.c:404.
Add SVCB and HTTPS to the types removed by 'unbound-control flush'.
Clear documentation for interactivity between the subnet module and
the serve-expired and prefetch configuration options.
Fix #773: When used with systemd-networkd, unbound does not start
until systemd-networkd-wait-online.service times out.
Merge #808: Wrap Makefile script's directory variables in quotes.
Fix to wrap Makefile scripts directory in quotes for uninstall.
Fix windows compile for libunbound subprocess reap comm point closes.
Update github workflows to use checkout v3.

Best regards, Wouter

Happy New Year!

Sorry I didn't get back to you, I've been swamped at work. I don't see the need to update to an RC release personally, but i could be convinced otherwise if there is good reason.

No worries @klutchell , I closed as today RC2 was released, which made me consider that indeed there is no rush - so why not just wait for the final version to become available.


add unit_test_framework proposal

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Can you add one real test case with this framework

Can you discuss this pr with a real case at regular meeting?

/lgtm
/approve

/lgtm
/approve


VAPOR disappears upon closing notice board

To reproduce:

Open VAPOR on macOS by right click -> open
Click "not check for notice" on the notice board.
Click "close" on the notice board.
VAPOR disappears.

https://drive.google.com/file/d/1J-2l26_pdAFEQ8Qb7NTt-VjCk4yWf9eq/view?usp=sharing

What do you mean by "disappears"? Does it crash, does the window get hidden, etc.? I can't reproduce.

Hmm, I'm not sure if it crashes or quits normally, it just disappears, as the video shows. I'm clicking icons on MacOS so there's not terminal output to help determine its status.
Dunno what factors in its reproducibility, but I cannot use VAPOR on my MacOS.

You can check the process table to see if it is still running. Works for me on MacOSX 11.4 using the <IP_ADDRESS>ad55a6ac installer.

Good point. I don't see it in top results, so it must quit itself.
Here I'm using the same VAPOR installer version, and macOS 11.5. Macbook Pro 2019 16-inch.
Report status on source build:

If source built using apple compiler and openmp disabled, the program runs well without the reported disappearing issue.
if source built using standard LLVM (described here), VAPOR fails to launch with the following message:

shaomeng@cisl-watson VAPOR-git/build % ./bin/vapor.app/Contents/MacOS/vapor
dyld: Library not loaded: @rpath/librender.dylib
  Referenced from: /Users/shaomeng/Git/VAPOR-git/build/./bin/vapor.app/Contents/MacOS/vapor
  Reason: image not found
zsh: abort      ./bin/vapor.app/Contents/MacOS/vapor

To reproduce:

Open VAPOR on macOS by right click -> open
Click "not check for notice" on the notice board.
Click "close" on the notice board.
VAPOR disappears.

https://drive.google.com/file/d/1J-2l26_pdAFEQ8Qb7NTt-VjCk4yWf9eq/view?usp=sharing

@stasj can you reproduce this?

To reproduce:

Open VAPOR on macOS by right click -> open
Click "not check for notice" on the notice board.
Click "close" on the notice board.
VAPOR disappears.

https://drive.google.com/file/d/1J-2l26_pdAFEQ8Qb7NTt-VjCk4yWf9eq/view?usp=sharing

@StasJ can you reproduce this?

I cannot reproduce this. I tested with the pre-release installer on macOS Catalina and on macOS Big Sur.
https://user-images.githubusercontent.com/2772687/127063815-eceed608-0db0-4a21-b08d-40fa300edc88.mov

I can only reproduce this on my development laptop

CISL-FARMINGTON (Cattaliina 10.15.7)

I can't reproduce this on the following.  The notifier has correct behavior.

CISL-ACWORTH (Catalina, 10.15.7)
CISL-TRIANGLE (BigSur, 11.4)
Ubuntu20 test machine
Ubuntu18 test machine

Unfortunately, the notifier does not show up at all on the following machines.  It was enabled in all tests.

Windows10 test machine
CentOS test machine
2 personal laptops (Windows 10)

@shaomeg will try to build with/without openMP to see if that is a related issue.
@sgpearse will make a debug build and get a trace, and try to fix

Update on non-OpenMP build on Sam's Macbook:
I'm able to build and successfully launch the program with OpenMP OFF.
However, there is no notice board showing upon opening the app. The command line has this output though:
shaomeng@cisl-watson VAPOR-git/build-apple % ./bin/vapor.app/Contents/MacOS/vapor
qt.qpa.window: <QNSWindow: 0x7fc014b74c90; contentView=<QNSView: 0x7fc014b74500; QCocoaWindow(0x7fc014b743e0, window=QWidgetWindow(0x7fc014b71df0, name="QGLWidgetClassWindow"))>> has active key-value observers (KVO)! These will stop working now that the window is recreated, and will result in exceptions when the observers are removed. Break in QCocoaWindow::recreateWindowIfNeeded to debug.

Fixed with the changes introduced in #2839.


Bump gnatsd

We have run across an issue when multiple gnatsd servers are restarted around the same time.
A recent fix to gnatsd is potentially the issue we are seeing.  What we do know is that the clients are sending NATS messages but receiving very few to none.
https://github.com/nats-io/gnatsd/commit/b62835363a08d6b14dfee655389261575baca8ee

Hey @fraenkel
Sorry for the delayed response, vacation...
Have you tried bumping, does it fix your problem (without causing other problems)?  Sounds like you're saying it's potentially a fix, but not sure.  If you do find a SHA that works, would you be inclined to submit a PR for the bump?

Hey @fraenkel any thoughts?

I did try bumping. I don't know if it fixed our problem since any restart as long as it's not within a similar time frame will prevent the issue as well. I did not see any issue with the update.

Hmmm, doesn't sound too compelling at this time.  Feel free to re-open or submit PR if you're confident it fixes your problem.


post_filter for python module

What is the correct way to pass a filter by hashtag for Location download in the module of Python?
This is an example:
L = instaloader.Instaloader()
L.download_location('362629379', max_count=30, post_filter="'cute' in caption_hashtags")
I get an error by setting the post_filter like this.
Thanks.

It seems that works with this:
L.download_location('213270076', max_count=10, post_filter= lambda post: "dog" in post.caption_hashtags)
But I get the exception 429 for too many requests being made. Is there a way to avoid this? I guess no, because the filter is only to download, so at filtering by reading the post captions requests are being made and it reaches the limit. =(


nom7

Update to nom 7

Updated @nickelc

@jethrogb Anything blocking this?

@bors r+

bors r+

Travis CI is no longer free, this should probably be migrated to GH Actions

@jethrogb so, we can't push without bors to master, can you unprotect the branch (maybe temporarily) so I can get CI working?

Do you not have access to project settings? I change the branch protection rule to exclude the master branch for now.

I do not. Thanks!


Fix addition for #396

Initializes Location values and adds safe return in the event of an index failure which is less likely now.

This causes a new test failure resulting in a different number of results. Will have to evaluate.
I'm not sure that returning an incorrect position is better than an exception, I think null might be more appropriate.

Will not merge this change.

I don't think silently throwing this exception away is correct.
The underlying issue should have been resolved by the previous fix, doing an empty catch here will just prevent us from detecting any similar problems in the future. Returning 1,1 is incorrect if the position cannot be found.


impossible to connect

Hello,
I try https://github.com/iamseth/oracledb_exporter
when I do this call
export DATA_SOURCE_NAME=USER/PASSWORD@localhost/DATABASE
./oracledb_exporter

I got the information and the database is considered as UP
When i try with this exporter with the configuration file
connections:
 - connection: USER/PASSWORD
   database: DATABASE
   instance: DATABASE
   alertlog:
    - file: /oracle/diag/rdbms/....log
      ignoreora:
       - ORA-00001
       - ORA-01033
       - ORA-01041
       - ORA-01089
       - ORA-01555
       - ORA-28
       - ORA-235
       - ORA-609
       - ORA-3136
   queries:
    - sql: "select 1 from dual"
      name: sample1

the database is down but I didn't see anything.

Hi,
i think your problem is the missing TNS Connection
connections:

connection: USER/PASSWORD@TNS


[feature] redirect to root [OSF-6133]

Purpose
Visiting https://api.osf.io/ return 404 File Not Found when it should redirect to https://api.osf.io/v2/.
Changes
Add url pattern to redirect to root if / route is visited.
Side effects
None
Ticket
https://openscience.atlassian.net/browse/OSF-6133

Looks great. Could you add a test? And I presume you tested manually.

:penguin:

Test has been added (is the name okay?). And yes, I did test it manually :)

Perfect. When tests pass, this is ready to merge. :tada: :penguin: :tada:

The api test that is failing is failing on the release branch so even though it seems like it could be related to this PR it is not:
======================================================================
FAIL: test_redirect_throws_501_for_non_implemented_views (api_tests.guids.views.test_guid_redirect.TestGuidRedirect)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/laurenbarker/GitHub/osf.io/api_tests/guids/views/test_guid_redirect.py", line 74, in test_redirect_throws_501_for_non_implemented_views
    assert_equal(res.status_code, 501)
AssertionError: 302 != 501


Added safety definition

Hi @zhongjun2
Thanks for the PR. A few comments:

Could you add a DCO sign-off for the PR
Could you place the markdown file in a /definition folder
I think you just include the metric and link to the metric in the Metrics in the Model section

Hi @zhongjun2
Thanks for the PR. A few comments:

Could you add a DCO sign-off for the PR
Could you place the markdown file in a /definition folder
I think you just include the metric and link to the metric in the Metrics in the Model section

Done

Hi @zhongjun2
Thanks for the PR. A few comments:

Could you add a DCO sign-off for the PR
Could you place the markdown file in a /definition folder
I think you just include the metric and link to the metric in the Metrics in the Model section

Done

@zhongjun2 Does this model include data insight?

@zhongjun2 Does this model include data insight?

The document does not yet present the data and associated analysis

Hi @zhongjun2 -- Thanks for continuing to move this forward. A few more suggestions:

Could you simply provide a link to the metric used in the metric model? I don't think that we need the rich text below the link to the metric.
You could likely follow the suggestions from @eyehwan to remove the headers that are not being used. We can return to include them when we have a sample community to talk about.

Hi @zhongjun2 -- Thanks for continuing to move this forward. A few more suggestions:

Could you simply provide a link to the metric used in the metric model? I don't think that we need the rich text below the link to the metric.
You could likely follow the suggestions from @eyehwan to remove the headers that are not being used. We can return to include them when we have a sample community to talk about.

Added link to the metrics in the doc. Removed the headers that are not being used

Looks good to me!


chore(ci): remove unused orb definition

This orb is no longer referenced in the CircleCI config.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Check for updates window is too small

Check for updates window is too small and text is clipped

fixed


[Question][JIRA PLugin] JIRA Issue and commit link pattern

Question
I am trying to setup JIRA issue and github commit remote link pattern but anything and everything I try keeps on giving me below error message
missing {namespace} (400).
I have tried both options of directly specifying pattern and using development panel but got same error.
Wondering if anyone has managed to set this up and can share a working example please ?
Screenshots
If applicable, add screenshots to help explain.
Additional context
Add any other context here.

@dosu can you share any real example of this pattern working please.

@dosu both of the above doesn’t work and still says namespace not found

I think the conditions in the code are wrong.
https://github.com/apache/incubator-devlake/blob/main/backend/plugins/jira/api/scope_config_api.go#L331

@kulnagar Why do you say so? Would you like to elaborate?

@klesh I have tried this with different combination.Below code snippet expects a string which is {namepsace} not the actual value.
If you put actual value of your github org and/or repository name the UI validation keeps on failing and complaining about missing namespace.
However if you put hard coded string "{namespace}" which isn't correct then UI validation works but regex generation fails(as expected).
Hence this UI validation seems incorrect where instead of actual org value it's looking for string "{namespace}".
if !strings.Contains(input, "{namespace}") {
		return errors.BadInput.New("missing {namespace}")
	}


stats / analyses

we might want to add additional / more advanced analyses
eg. looking at harmonics

so, i think this is the next and hopefully last crucial step to finish the tutorial.
what is your take on this: anything meaningful we can and want to show with this toy data?
my take:

we could keep the ROI stuff, imo, because it should nicely correspond with the topo plot (and together it could nicely visualize how easy it might be to p-hack ;) - jokes apart, there could be a mini discussion of preregistering ROI's.
we could add quantification of harmonics, even though it doesnt add much with these data

any more meaningful analysis might be done in a subsequent loop "you got an impression, now lets do some stats":

e.g. across subjects comparison or group level analysis (quite tasteless with n=2 but who cares)
or comparing 12 vs 15 hz response. in the auditory domain, there is at least one paper that investigated which stim freq elicits maximal response - we might find this for the visual domain as well

Hi, @dominikwelke! Sorry for disappearing yet again: I missed the comment above completely.

ROI stuff - yes. I would also introduce it earlier and substitute ROI averaging for whole-head averaging in one of the plots.
note on double-dipping not being OK - yes
across subjects - I'd rather not if we can compare trials of one subject. It is much cooler to do it on a single-participant basis. It is partially the point of using SSVEP, I think.
12 vs 15 at the same frequency, that is: can we discriminate the conditions? - yes. The result is kind of obvious from the plots but makes sense as a prior hypothesis.
12 at 12 vs 15 at 15 - I am not sure if it makes sense in this case. Generally, I think this comparison is too specific for a tutorial.

closed by ce6902f104da08c2ef1dcc6cad8e0a0317a6dd04


Random wallet crashes in the Send and Receiver screens.

Description
I had some crashes when I did some actions on the receiver screens. Those actions which lead to crashes:

The wallet was crashed when I chose an offline token.

The wallet was crashed when I sent a max privacy transaction.

The wallet was crashed when I press on the back button on the Receiver screen.

The wallet was crashed when I press on the back button on the Send screen.

Attention: The problem is inconsistent. The same actions didn't lead to failure when I tried to reproduce it.
Attachments
There are two wallets with those issues:
Wallet 1:
Crashes_wallet 1.zip
password=1
Wallet 2:
Crashes_wallet 2.zip
password=1
There is another wallet with some other crashes.
Wallet 3.zip
password=1

stack overflow

Beam Wallet Masternet.exe!__chkstk() Line 109	Unknown
Beam Wallet Masternet.exe!QTextEngine::itemize(void)	C++
Beam Wallet Masternet.exe!QTextEngine::attributes(void)	C++
Beam Wallet Masternet.exe!QTextLine::layout_helper(int)	C++
Beam Wallet Masternet.exe!QTextLine::setLineWidth(double)	C++
Beam Wallet Masternet.exe!QQuickTextPrivate::setLineGeometry(class QTextLine &,double,double &)	C++
Beam Wallet Masternet.exe!QQuickTextPrivate::setupTextLayout(double * const)	C++
Beam Wallet Masternet.exe!QQuickTextPrivate::updateSize(void)	C++
Beam Wallet Masternet.exe!QQuickText::geometryChanged(class QRectF const &,class QRectF const &)	C++
Beam Wallet Masternet.exe!QQuickItem::setSize(class QSizeF const &)	C++
Beam Wallet Masternet.exe!QQuickGridLayoutItem::setGeometry(class QRectF const &)	C++
Beam Wallet Masternet.exe!QGridLayoutEngine::setGeometries(class QRectF const &,class QAbstractLayoutStyleInfo const *)	C++
Beam Wallet Masternet.exe!QQuickGridLayoutBase::rearrange(class QSizeF const &)	C++
Beam Wallet Masternet.exe!QQuickLayout::geometryChanged(class QRectF const &,class QRectF const &)	C++


Add prettier recomended to eslint config

Landed in https://github.com/metarhia/metautil/commit/b85fe618eda3b078aae1253a0e5d034d16478dc4


rumble.com

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[ ] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: blu

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

rumble.com

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client:

Version:

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:
Why is it blocked?

Thank you for making Energized Protection great, with your kind help!
A project by Ador with ❤

https://www.thegatewaypundit.com/2020/12/watch-mypillow-ceo-slams-fox-news-election-coverage-makes-shocking-claim-video/


Day295

Resolves #601.

Pull Request Test Coverage Report for Build 1123

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1113:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


llamafactory会考虑支持 Online DPO 吗

Reminder

[X] I have read the README and searched the existing issues.

System Info
无
Reproduction
无
Expected behavior
No response
Others
https://huggingface.co/docs/trl/main/online_dpo_trainer

+1


egui-gizmo should not react to input events which occur outside the currently active viewport

Hello,
I am using egui-gizmo in a multi-viewport context (an editor with multiple 3D viewports using egui_dock). Drawing the gizmos works fine (the gizmo shows up in each viewport in the right position) but there is a problem with manipulation.
If calling gizmo.interact(...) for each viewport, the object would jump around if I tried for example to translate it in a second viewport (two viewports arranged horizontally). The problem was that in that case, the hover position we got from ui.input(|i| i.pointer.hover_pos())?; was outside the viewport, so the calculated x was outside the range of -1 to 1. I guess the root problem is egui-gizmo attempts to manipulate the object in relation to both viewports when the mouse position should only be valid in one viewport at a time.
The problem could be fixed by adding this before calculating x and y in pointer_ray():
if !viewport.contains(hover) {
    return None;
}

In this case, manipulation would only occur in the active viewport.
Thanks in advance,
J-S

Hi, I suspect this issue may no longer be valid, as the crate has undergone quite a few changes. If you happen to update to the transform-gizmo-egui and this problem still persists, feel free to re-open this.


Fix start command detection for spdlog

:notebook_with_decorative_cover: Description
When configuring spdlog we do different things for client and server. We distinguish between the two by checking whether the current subcommand we're running is start.
However, this returned a false positive for a plugin that had a command <plugin> start. This commit changes to checking whether the full name of the command we're running is start.
:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
n/t

I assume you have already checked that the full_name is just start and not vast start in the server case?

Yes. I verified that vast start still writes to the vast.db/server.log file.


MSME Applications endpoints (FI User)

Get the list of applications in the status list not in Pending, Declined, Accepted, Lapsed see #39
List only applications associated with the FI of the logged user.

Closing as completed


EarlGrey: adc_ctrl_core.sv doesn't work in Verilator

I created the test of this module: https://github.com/chipsalliance/UHDM-integration-tests/pull/564
It throws an error:
%Error: Member selection of non-struct/union object 'SEL' which is a 'PACKARRAYDTYPE'
And 8 warnings like
%Warning-WIDTH:adc_ctrl_core.sv:101:14: Operator LTE expects 10 bits on the LHS, but LHS's CONST '1'h0' generates 1 bits.

caused probably by the same bug

https://github.com/chipsalliance/UHDM-integration-tests/pull/565 reproduces the error, which is caused by the same problem as warnings: https://github.com/chipsalliance/Surelog/issues/2095


When the on-screen keyboard is open, NavigationBar does not maintainBottomViewPadding in Edge-to-Edge mode

Steps to reproduce

Create this NavigationBar example:

Alternatively you can clone https://github.com/VinhNgT/navbar-edge2edge-bug and skip step 1 and 2.

flutter create --sample=material.NavigationBar.3 mysample

Enable Edge-To-Edge mode by adding these to void main():

WidgetsFlutterBinding.ensureInitialized();
SystemChrome.setSystemUIOverlayStyle(
    const SystemUiOverlayStyle(systemNavigationBarColor: Colors.transparent));
await SystemChrome.setEnabledSystemUIMode(SystemUiMode.edgeToEdge);

Press "Push/list" -> "Push/text", then tap on the text field to open the on-screen keyboard.
Notice every time you close/open the keyboard, the NavigationBar moves up and down.

Expected results
The NavigationBar should lock in place when the keyboard is open. Similar to SafeArea.maintainBottomViewPadding
Actual results
The NavigationBar follows padding instead of viewPadding, which causes it to move when the keyboard closes/opens.
Code sample
Code sample
https://github.com/VinhNgT/navbar-edge2edge-bug

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/6505380a-4f9c-4cb1-849d-626aa63b8921

Logs
No log
Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.24.5, on Microsoft Windows [Version 10.0.22631.4460], locale en-US)
    • Flutter version 3.24.5 on channel stable at F:\tools\fvm\versions\3.24.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (13 days ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at C:\Users\victo\AppData\Local\Android\sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)
    • All Android licenses accepted.

[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.11.5)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.11.35327.3
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2024.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)

[√] VS Code (version 1.95.3)
    • VS Code at C:\Users\victo\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.100.0

[√] Connected device (4 available)
    • Redmi Note 8 (mobile) • <IP_ADDRESS>:42415                      • android-arm64  • Android 14 (API 34)
    • Redmi Note 8 (mobile) • adb-139ed888-lNT5l9._adb-tls-connect._tcp • android-arm64  • Android 14 (API 34)
    • Windows (desktop)     • windows                                   • windows-x64    • Microsoft Windows [Version 10.0.22631.4460]
    • Edge (web)            • edge                                      • web-javascript • Microsoft Edge 130.0.2849.80

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thanks for the report. I was able to replicate this using latest versions and seems specific to edge to edge. I tried with manual and immersive with which the navigationBar doesn't jump as shown below:
https://github.com/user-attachments/assets/1269fd1d-2d4d-45fd-88b8-22aede851dcd
Verified on Android S10+ device and also on emulator.
stable : 3.24.5
master : 3.27.0-1.0.pre.628


Add some getter for the "open" property of each node

The open property (well, collapsed actually) is part of my controlled data.
I want it to be like this so that the open or closed state is persistent.
I've tried using the onToggle property to keep my open state in sync with react-arborist's open, and it seems to work fine. The problem is that with the initial state they become desynchronized. openByDefault to true or false would not solve my problem, because some nodes should be able to be initialized open and others closed.
Two ideas have occurred to me:

Add an openAccessor or getOpen prop that works as a getter and use onToggle as a setter.
Add two properties getOpen and setOpen. It probably makes sense to deprecate onToggle in this case.

Something I don't like about onToggle is that instead of passing the node, it passes the id, and I have to do a lookup of my data to find it and access it.
Edit: I just discovered the initialOpenState property that I had overlooked.
With this I have been able to solve it, although it is a bit of a boilerplate since I have to go through my entire data structure and form an OpenMap, not ideal.

You're right. This is not ideal. I am considering changing the api to make this easier in the next major version. Glad you were at least able to create a workaround.

Edit: I just discovered the initialOpenState property that I had overlooked. With this I have been able to solve it, although it is a bit of a boilerplate since I have to go through my entire data structure and form an OpenMap, not ideal.

Could you show a snippet how you solved it, i think i need to solve the same problem.

any update on this ?

I would be also appreciated for the change the API to: onToggle?: (node: NodeApi<T>) => void


CockroachDB: Retrying mechanism

(Adding as an issue as it appears not to have been raised as one before?)
CockroachDB recommend the use of retries should we see a SQL error code 40001, which represents a lock wait timeout. We are adding a pair of classes:

DefaultExecutionStrategy - which simply fires a SQL call at the db once
CockroachDBRetryingStrategy - which attempts retries up to a set limit if the above code is received from the database.

This should be straightforward to extend if other databases need a similar approach (or if people want to fork and try their own strategies).

Released in 6.2.2


Error 401, Server blocking and others

Hello, I'm trying the SW out (Windows standalone executable) but I've had a few issues.

.\instaloader PROFILE. It seems like it proceeds with a first 12 attempts, then if these have already been DL, it goes by 50 up to those still to DL. After a couple of days where it seemed to work fine (except after some massive # of DL - like after DLing 1000 posts, IG blocked it), today it just downloads the first 12 posts and then it's like IG rejects the connections (error 401 as per the attachment). Retrying doesn't work.

Using the --login=ID option with my personal account to DL like 60 private posts, IG blocked the DL halfway through and required a change of password. So I guess it's not safe at all unless you want to risk getting blocked/banned.

Is it a known problem? Is IG targeting the too fast connections? Is there a plan for an option to create a (random) delay between DLs for the standalone executable?
Thanks.

I haven't download anything for 2 days to see whether or not it will reset itself but it didn't and still giving me this error after 12th post and hashtags.

I haven't downloaded anything for 2 days to see whether or not it will reset itself but it didn't and still giving me this error after 12th post and hashtags.

Guess IG wins this... it's time for 4.9.6 :D

Happens almost every time for me too. Waiting a few days doesn't seem to solve the problem either.

I used to had similar error, but after adding a proper user agent, it continues to work

I used to have similar error, but after adding a proper user agent, it continues to work

And could you share what that proper UA "value" is ?
Indeed seeing other people still asking questions I guess not everyone is modifying the SC to bypass IG's firewalls and there's something easier to do...

Proper UA value would be Instagram App UA

I used to have similar error, but after adding a proper user agent, it continues to work

And could you share what that proper UA "value" is ?
Indeed seeing other people still asking questions I guess not everyone is modifying the SC to bypass IG's firewalls and there's something easier to do...

Proper UA value would be Instagram App UA

Excuse me I'm not a programmer but how do you change the ua? Can you give me an example?

Yep, with Instagram UAs it works again, thanks! But I guess it's just a matter of time before IG blocks all these fast connections indipendent of the UA...

I used to have similar error, but after adding a proper user agent, it continues to work

And could you share what that proper UA "value" is ?
Indeed seeing other people still asking questions I guess not everyone is modifying the SC to bypass IG's firewalls and there's something easier to do...

Proper UA value would be Instagram App UA

Excuse me I'm not a programmer but how do you change the ua? Can you give me an example?

It's in the documentation: use the --user-agent parameter. Possible values can be found online under IG App UAs.

Yep, with Instagram UAs it works again, thanks! But I guess it's just a matter of time before IG blocks all these fast connections indipendent of the UA...

You will still hit a wall even with IG's UA, but at least it's not 401. Just add abort flag on certain errors

I used to have similar error, but after adding a proper user agent, it continues to work

And could you share what that proper UA "value" is ?
Indeed seeing other people still asking questions I guess not everyone is modifying the SC to bypass IG's firewalls and there's something easier to do...

Proper UA value would be Instagram App UA

Excuse me I'm not a programmer but how do you change the ua? Can you give me an example?

It's in the documentation: use the --user-agent parameter. Possible values can be found online under IG App UAs.

i did  the user agent thing couple of days ago. it's much much better but still time to time it gives me the error 401. I delete the session or use vpn for a while to fix it. I don't know how it works but it works

Yep, with Instagram UAs it works again, thanks! But I guess it's just a matter of time before IG blocks all these fast connections indipendent of the UA...

You will still hit a wall even with IG's UA, but at least it's not 401. Just add abort flag on certain errors

Hello! I am facing the same problem, I am trying to download photos and their comments from a list of profiles and every time I download 12 I get a 401 error. I have tried changing the user agent to some of the ones I found on this page - > https://user-agents.net/applications/instagram-app , but it doesn't change anything. Do you know if I have to use some specific IG user agents and if there are other solutions to avoid this problem? Thank you for your time

Yep, with Instagram UAs it works again, thanks! But I guess it's just a matter of time before IG blocks all these fast connections indipendent of the UA...

You will still hit a wall even with IG's UA, but at least it's not 401. Just add abort flag on certain errors

Hello! I am facing the same problem, I am trying to download photos and their comments from a list of profiles and every time I download 12 I get a 401 error. I have tried changing the user agent to some of the ones I found on this page - > https://user-agents.net/applications/instagram-app , but it doesn't change anything. Do you know if I have to use some specific IG user agents and if there are other solutions to avoid this problem? Thank you for your time

Make sure you are not logged in on multiple device

Also what I do:

Change UA from time to time and use it for public profiles only
Give up for the day at the first 401 (OFC with the abort parameter as well on 401)
Use it during high traffic time for IG servers (in Europe it's 6PM to 24PM): it may be contrary to logic but it seems like I get blocked less often... they may implement less strict policies to avoid stressing the servers? I don't know...

Still, after a while (Can be the 1st account after 12 pics, can be 10k items downloaded and depending on the day) I get blocked. Not really ideal and hopefully a new version that solves these problems will be released soon...

I actually hitting 403 lately, not 401. I might need to stop scraping for a while


Add microphysics parameters to toml file

Let's go with this for now to test things out in thermo

bors r+


Register default extensions first (#989)

Register default extensions first
This changes so default extensions are loaded
before custom extensions. This means custom
extensions can override things like default templates
from the default extension
Addresses #989

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


Add new Flowchart Symbol

Hello Mr. Adrai
I'm really appreciate your work, It's amazing, but I have some problem, the symbol is not complete
I've try to look your js but I'm not very well with js.
Is there another way to add some symbol?
Glad to reply my issue, thanks for all your support

Which Symbol do you miss?

document, multiple document, connector, database, etc --> https://www.smartdraw.com/flowchart/flowchart-symbols.htm
thanks for answer Mr. Adrai


(feature-ui-signup):To sign up

This allows the user to create an account with the restaurant
[Delivers #162102369]

This pull request adds a signup page to the site


Fix : selected subcategory in the sidebar

Fixes Issue
#1687

Changes proposed
Made the Selected category in sidebar highlighted by adding extra adding a path  variable
Screenshots

Note to reviewers
Please check
@Anmol-Baranwal @CBID2

@Anmol-Baranwal can you add hacktoberfest-accepted in this?

@Vijaykv5
Is this PR not showing in your dashboard?
The hacktoberfest topics are added to the LinksHub repo, so there is no need of this.
If it isn't there, let us know. It would be a great clarification.

Yeah, it will update.
This is for the case where the PR gets reverted or something unexpected occurs in a week. Here, it won't matter.


Logic App infrastructure and workload provisioning separation

I am considering to separate out the infrastructure part of a logic app from the workload part of a logic app. Comparable to how one would separate out app service infrastructure from workload.
But after some exploration, in particular the key is found in this example file:
https://github.com/Azure-Samples/media-services-video-indexer/blob/master/AIEnrichmentPipeline/infra/logicapp/logicapp.tf
It seems that there is very little separate infrastructure:
resource "azurerm_logic_app_workflow" "logicapp" {
  location            = var.shared_env.rg.location
  resource_group_name = var.shared_env.rg.name
  tags                = var.shared_env.tags

  name = var.workflow_name
  lifecycle {
    ignore_changes = [
      # Ignore changes to parameters as otherwise we will break the $connections
      parameters,
      tags
    ]
  }
}

The workload seems to contain all infrastructure details as well. I.e.
https://github.com/Azure-Samples/media-services-video-indexer/blob/master/AIEnrichmentPipeline/logicapp/digitaltextfileworkflow/armtemplate.json
specifies the full set of logic app resources.
Why would you keep this terraform resource declaration?
At least one reason I see, is that terraform provisions the tags towards the logic app resource.
And what about your app lifecycle. Will you run terraform to deploy an updated version of your logic app workload? Is there any suggestion you can make on versioning? Analogous to how one would separate out app service infrastructure from workload package versioning through e.g. Azure Artifacts.
In the past I was involved in a setup where we provide building blocks to use case teams. I.e. we provide a logic app building block and a connection bulding block and the use case team applies these building blocks in their pipelines.
Analogous, you could separate out from the arm template:

the workload json; reference it from the logic app resource inside terraform
the connection declaration into terraform

Are there any considerations here?

This code sample was removed,
closing the thread.


usage of operator without internal user realm

i admittedly haven't tried the operator yet, but based on the code it seems that it does everything with an admin user from the internal user realm (using basic auth with username + password).
are there any plans to support other modes? namely, we'd like to use the operator but, if possible, run without the internal user realm and instead only use JWT.
i currently see two options here (more might exist):

the operator uses an admin certificate to manage the cluster (and can organise the certificate on its own; e.g. using #141)
the operator provides a callback which can be implemented to get a valid auth token (which can be the basic auth header, a JWT header, a certificate, etc.) and then uses that

note: we currently don't use OpenSearch Dashboards, but i've also raised https://github.com/opensearch-project/OpenSearch-Dashboards/issues/970 for this a while ago.

You are correct that currently the operator uses the internal admin user with basic auth for interacting with opensearch clusters.
There are currently no concrete plans to support anything else, but I can see the certificate option being viable. We already use an admin certificate to apply the securityconfig with securityadmin.sh so that would be the same direction.


Error in release 1.0.2-alpha

I'm using release 1.0.2-alpha with Alteryx version 2021.2.35394. I attempted to use the SFTP Downloader in an otherwise blank workflow and I get the message "You have found a bug. Replicate, then let us know. We shall fix it soon."
I replicated the settings (host, port, username, etc.) in a WinSCP session and was successfully able to connect/FTP files.

This is an issue with the Python SDK that we have with several plugins and that seems to be connected to the handling of metadata for the outgoing connection. In our experience, this issue only arises in the Designer when building the workflow and does not impact running the workflow. Did the workflow run properly for you?


Improve Connection::_foreignKeyData()

This PR improves the method for retrieving foreign keys. It aligns all DBMS to use the same naming convention. It adds some more information such as delete, update, and match rules. The returned dataset uses a new structure which uses one row per foreign key with an array of fields - useful for composite foreign keys.
This is the first part of https://github.com/codeigniter4/CodeIgniter4/pull/6430
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

The returned dataset uses a new structure which uses one row per foreign key with an array of fields

Can you add the structure in the user guide?

@sclubricants Thank you for this PR. This is easy to understand!
The new PHPStan reports new errors. We've fixed. Please rebase to get rid of the errors.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#updating-your-branch
There are a few GitHub Action check failures. Also please check.

There are two PHPUnit / PHP 7.4 - MySQLi (pull_request) tests. Not sure why. Not sure why one of them is failing.
ErrorException: Undefined property: stdClass::$constraint_name
Connection::_foreignKeyData()
        $sql = '
                SELECT
                    tc.constraint_name,
                    ....
                    tc.table_name = ' . $this->escape($table);

        if (($query = $this->query($sql)) === false) {
            throw new DatabaseException(lang('Database.failGetForeignKeyData'));
        }

        $query   = $query->getResultObject();
        $indexes = [];

        foreach ($query as $row) {
            $indexes[$row->constraint_name]['constraint_name']       = $row->constraint_name;

The only way I can see this failing is if MySql isn't returning the fields in lower case. I had this problem with Oracle in upper case.

There are two PHPUnit / PHP 7.4 - MySQLi (pull_request) tests. Not sure why. Not sure why one of them is failing.

The last one is MySQL 8.0.

https://github.com/codeigniter4/CodeIgniter4/pull/6468/commits/6bfd0bec2c1e71e3c18f8fcd9ad8fc7dad85a577

Fix MySql _foreignKeyData() SQL upper case

Please write why you changed to uppercase in the commit message.
We can easily know that this commit changed the column names to uppercase,
but it is difficult to know why this change was needed from the commit.

https://github.com/codeigniter4/CodeIgniter4/pull/6468/commits/1b03ee7a7fcdf8c21ea8d9e425cb71dfc5c58a88

Using a prefix causes problems when using dropForeignKey().

What's the problems?

What's the problems?

When you call dropForeignKey() the table parameter gets prefixed by table prefix. I mentioned it elsewhere here..

Please write why you changed to uppercase in the commit message.

Good question.. basically cause it worked. Its something I've never seen with MySql. MySql has always returned the same letter case as the sql is typed in my experience but for some reason the server you are running that test on returns upper case no matter what.

basically cause it worked.

It seems MySQL 8 changed the behavior.
what are the changes in mysql 8 result rowset case? - Stack Overflow
https://stackoverflow.com/questions/54538448/what-are-the-changes-in-mysql-8-result-rowset-case

When you call dropForeignKey() the table parameter gets prefixed by table prefix.

Oh, we can't drop any key that does not begin with the DB prefix!
https://github.com/codeigniter4/CodeIgniter4/blob/a8072a51bb8c1ea05998d9a045ce61d2c78e32bc/system/Database/Forge.php#L466-L470

Oh, we can't drop any key that does not begin with the DB prefix!

We definitely want to have the prefix. Someone might have a different connection on same database with duplicates of tables but with different prefixes. This way we avoid name collision.

We definitely want to have the prefix. Someone might have a different connection on same database with duplicates of tables but with different prefixes. This way we avoid name collision.

Do we need the prefix for foreign key name?

FYI, if we allow setting the name manually this won't apply to SQLite. SQLite does not really support naming the constraint. You can name it when you create table but you cannot retrieve the name with the foreign keys. My guess this is because you can't drop or alter the foreign keys so it doesn't need a name. Allowing people to specify a name should through an error. They will need to use the generated name that we assign to it.

FYI, if we allow setting the name manually this won't apply to SQLite. SQLite does not really support naming the constraint.

Yes, I'm aware of this. Since we had never any controls over constraint names in SQLite I don't recognize it as a problem. Adding a friendly note in the user guide about SQLite should be enough.
I was wondering if throwing an error when we define a name in SQLite would be a good choice... simply from a practical perspective. SQLite is frequently used during tests in smaller projects and throwing such an error may be frustrating for some use cases (tests on SQLite3 and production on MySQL). But probably throwing an error is the right thing to do here.

@sclubricants Please rebase to resolve conflicts.
It seems ready! Thank you for your cooperation.

@sclubricants @michalsn Thank you!
This is a great improvement.

@kenjis said it best! Thanks to everyone for the work on this 🥳


Upgrade to Testcontainers 1.12.3

Also, use reusable containers.

See also testcontainers/testcontainers-java#1781.


Delete ContentLaunch::SupportedProtocolsBitmap::kWebRTC

This had overlapping bitmap values which is invalid. It also is not part of the current spec and we expect this feature to be part of 1.4 only (and with changes due to sharing with the camera TT).
Removing for now.

No backwards compatibility: bitmap value is not part of the spec, it is not released.


ConfigMaps Seem To Not Be Created With v2 Of The Template

First off, thank you so much for all of your work!
The v2 template is very nice :)
I have run into an issue using configmaps, though.
Using the example from the values file here, I have this yaml block:
configMaps:
  config-yaml:
    enabled: true
    labels: {}
    annotations: {}
    data:
      file.yaml: |
        data goes here

However, in the new version of the template, a configmap is not created.
Am I missing something here?

I just played with it a bit more, and I am thinking it was whitespace error.
Thank you, again!


How to troubleshoot

I tried it in my application, but the listener is never called. How can I troubleshoot it?

Found the issue (using lambda so I need to pass lifetimeobject)


get_class_ind error in Python 3.6

In python 3.6, the get_class_ind function (c / self.k) in magnet_tools.py may return float not integer value, which leads the following sq_dists[...]=np.inf invalid.

Can someone please explain this? @liuyuyuil did you find a solution?


add global modules so running is easy

I am a n00b and these directions help

Technically you don't need to install it globally but it does make it easy for development


Add a Music Kit App

Develop a Javascript {Music-Kit} app.
Description 📜
Creating a music kit app feature using JavaScript involves building a web application that allows users to interact with and manipulate music. Also operated by a Keyboard
Requirements 🛠
Drum Kit Interface: The app must provide a user-friendly interface that resembles a drum kit, including drum pads that users can tap or click.
Sound Samples: Include a set of predefined sound samples for different drum instruments, such as snare, kick, hi-hat, tom, and cymbals.
Interactive Drumming: Users should be able to trigger drum sounds by clicking/tapping on the drum pads or using keyboard keys that correspond to each drum instrument.
Volume Control: Include a volume control/slider that allows users to adjust the volume of the drum sounds.
Can-Haves:
Customization: Allow users to customize the drum kit by choosing different drum sounds for each instrument from a selection of presets or by uploading their own audio samples.
Mobile Responsiveness : Ensure that the app is responsive and works well on mobile devices, making it accessible to a wider audience.
Any number of people can work on a single issue 👨💻👨💻
assign me to work on it.
👨🏻💻 Happy Coding 👩🏻💻

I want to work on it Assign me!

pls assign it to me

Please assign this issue to me