compat upper-bound versions required for registration

For packages to be added to the Julia General Registry, all package dependencies must be documented in the [compat] section within Project.toml.
https://github.com/brown-ccv/Chamber.jl/blob/ce736489f6d0c450facd76ffe82c106b5a6c57ff/Project.toml#L23-L25

Deps in need of compat upper-bounds
https://github.com/JuliaRegistries/General/actions/runs/4872991121/jobs/8691940736?pr=82806#step:16:149


gulpfile.babel.js much slower than commonjs gulpfile not using babel

We've found that parsing a gulpfile that is authored in babel takes much longer than a gulpfile authored in common js. Especially on older machines.

Unfortunately, there is nothing we can do about this. You can try opening an issue on the babel tracker for performance.


How to Polygon

Does polygon only consist of four points?
What is Region class that's used in the constructor?
Many thanks for this release..

@moagrius
Can you give me a example in how to create a HotSpot Polygon?
I tried to use a Region with Path, using the method setPath(Path path, Region clip)
But the second parameter (Region clip) has not worked.

https://github.com/moagrius/TileView/blob/master/tileview/src/main/java/com/qozix/tileview/TileView.java#L592
Note that HotSpots are Regions

Thank you very much @moagrius !

:thumbsup:


add MapLight ID for Warren Davidson

Before I merge -- am I following the right protocol for additions like this @konklone @JoshData ?

Yah perfect! Sorry I could have left you to merge it but I clicked too fast. :)


🐛 Bug Report: auth.getAccessToken doesn't survive page reloads anymore

📜 Description
When calling auth.getAccesToken for the GitHub provider, it forces the user to login even if they're already logged in. This session can withstand route changes and subsequent calls, but not a page reload. The user will see a login popup again.
👍 Expected behavior
It should work as it used to where if the provider has already been authorized, a token should be returned.
👎 Actual Behavior with Screenshots
It actually props the user to login on every page load.

👟 Reproduction steps
Using backstage 1.18.3 try something like the PR plugin (https://github.com/RoadieHQ/roadie-backstage-plugins/tree/main/plugins/frontend/backstage-plugin-github-pull-requests)
I'm experiencing the same behavior on an internal techdocs addon that calls the GH api.
📃 Provide the context for the Bug.
No response
🖥 Your Environment
Chrome Version 117.0.5938.132 (Official Build) (arm64)
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Relates to https://github.com/backstage/backstage/issues/20103 which doesn't seem to be fully fixed

Can anyone else confirm that they're experiencing this issue?

@kunickiaj I updated and am encountering this issue as well, using Okta auth.

@kunickiaj I can confirm the issue.
After some investigation I think there is a problem in the session refresh handler. I see that the scope previously saved in cookie is being used to refresh a session but is missed in the response. I guess this line should be fixed, like this e.g.:
scope: authenticator.shouldPersistScopes ? scope : result.session.scope,

Not very familiar with this implementation so going to try to add some more details to what i'm observing if it helps..
When loading a page that has a plugin that wants gh repo scope (to make some gh requests) i see:
/api/auth/github/refresh?optional&scope=repo&env=production called in the browser -- which is correct afaict
it returns a response that even includes the accessToken, however scope is ""
e.g.
{
...
  "providerInfo": {
    "accessToken": "ghu_....",
    "scope": "",
    "expiresInSeconds: "28800"
  }
...
}

Once i click on the login popup i see that GH requests are made with a different ghu_ token from the one above.
I think this might confirm your suspicions that the requested scope is not being returned in the response.

nvm, was able to repro/verify fix using yarn dev on the demo backstage. will work on getting a PR together shortly.


New PR to test SignCLA at 2019-02-14 22:06:17.698782732 -0500 EST m=+7.615753004

Body of PR

unsigned-fail-staging: test succeeded


Issue PDF.js v1.1.215 (build: c9a7498) Message: Invalid PDF structure

Is probably solved when pdf.js is updated (see #13). In any case, this is an issue of pdf.js, not this AngularJS plugin.

@jasny thanks for Thanks for answering, actually i update the pdf.js from here https://mozilla.github.io/pdf.js/ but that version doesnt work with the angular-pdfjs-viewer.js, Another thing I'm programming from develop localhost:8080 I do not know if the src is wrong
"$scope.pdf = {
src: 'localhost:8080#/digitalBlog/viewer/leer/example.pdf'
};"
the example.pdf is inside in the folder of my view and controller. I hope you can help me

No, you need the patched version. Please have patient, while we work on updating our fork.

Probably not that long. There is a PR now, we have to review it. Not making any promises though.

Thank very so much, I hope to receive news soon

@theseeker16 I believe this is not an issue of pdfjs. From what I can tell from your error pdfjs is not receiving the pdf you are intending for it to open. This could occur when it actually receives some html or an error message which could be accidental (server issue, wrong pdf path, etc).
Can you check in your browsers inspector (for chrome its F12) whether or not the pdf is actually being loaded correctly? You need to check the network tab and verify that the pdf being loaded is actually of the type pdf.
For the mozilla demo you would see something like this:

Note the response content-type is set to application/pdf. Also try opening the request url in another tab to check if the pdf is actually corrupt and/or check the response message, it should contain a binary representation of the pdf.

@moesjarraf

the src of my pdf is wrong i thnk and i dont know how to solve

this is my controller i put the pdf inside the directory but i dont know how to call that

@moesjarraf

i resolver the problem he load the pdf. but he does not display content

@theseeker16 @moesjarraf Please leave the issue tracker for bugs and feature requests. For how to use questions, please use StackOverflow. Thanks


Stevedore needs /norestart flag

Slonopotamus.Stevedore
Reference PR https://github.com/microsoft/winget-pkgs/pull/54370

For the reference, here's how Chocolatey installs MSI packages:
2022-03-15 17:48:57,518 12832 [DEBUG] - Running Start-ChocolateyProcessAsAdmin -validExitCodes '0 3010 1641' -workingDirectory 'C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore\0.3.3' -statements '/i "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore\0.3.3\stevedore-0.3.3-x86_64.msi" /qn /norestart /l*v "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore.0.3.3.MsiInstall.log" ' -exeToRun 'C:\windows\System32\msiexec.exe'
2022-03-15 17:48:57,534 12832 [DEBUG] - Test-ProcessAdminRights: returning True
2022-03-15 17:48:57,942 12832 [DEBUG] - Elevating permissions and running ["C:\windows\System32\msiexec.exe" /i "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore\0.3.3\stevedore-0.3.3-x86_64.msi" /qn /norestart /l*v "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore.0.3.3.MsiInstall.log" ]. This may take a while, depending on the statements.
2022-03-15 17:49:11,408 12832 [DEBUG] - Command ["C:\windows\System32\msiexec.exe" /i "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore\0.3.3\stevedore-0.3.3-x86_64.msi" /qn /norestart /l*v "C:\Users\m.radchenko.admin\AppData\Local\Temp\chocolatey\stevedore.0.3.3.MsiInstall.log" ] exited with '3010'.
2022-03-15 17:49:11,415 12832 [DEBUG] - Finishing 'Start-ChocolateyProcessAsAdmin'
2022-03-15 17:49:11,799 12832 [INFO ] - stevedore has been installed.

So, you may see that there's /norestart flag.
Originally posted by @slonopotamus in https://github.com/microsoft/winget-cli/issues/229#issuecomment-1068076523

Err... I am the author of Slonopotamus.Stevedore package :D And it isn't yet uploaded to WinGet. https://github.com/microsoft/winget-pkgs/pull/54370 is my initial attempt to do that.

Do you want to say that I can somehow add -norestart via editing the manifest?

https://github.com/microsoft/winget-pkgs/blob/master/doc/manifest/schema/1.0.0/installer.md#custom

Just add Custom: /norestart to your InstallerSwitches and it should work just fine.

As an MSI, we can determine the switches for Silent and SilentWithProgress. You can add "custom" switches to cover the /norestart scenario.

Wrong button 😞

Thanks. So I guess I'll just handle this as part of #54370 (that has some other issues preventing it from being merged currently) and we don't need a separate issue for /norestart thing.

The same is true for WIX, we can determine which switches handle silent and silent with progress.

we can determine which switches handle silent and silent with progress.

Could you elaborate on this a bit please? When does that switch-guessing happen? If during install, then it does not work as expected for MSI we talk about. Or it was supposed to happen during manifest generation via wingetcreate? I'll recheck the procedure then...

It should be happening during install, but winget doesn’t always append the correct flags (there’s issues in the other repo about where they need to be fixed, but MSI/WiX is definitely over there). It’s best to manually add them to your manifest when it is initially created, and on updates wingetcreate will carry the switches forward.


DiagID Control packet Handling

Add support for DiagID V1, V2 and V3 Control Packet definitions and subsequent feature handling.

Also can you update the commit message to follow the standard format within the diag project? Commit 1 should be something like:
diag_cntl: Add support for diag id v2 and v3 control commands

<description of why its needed and how problem is solved>

signed-off-by<>

Second Commit should look like:
apps_cmds: Add support for get diag ID command

<description of what it is for>

signed-off-by<>

Also can you update the commit message to follow the standard format within the diag project? Commit 1 should be something like:
diag_cntl: Add support for diag id v2 and v3 control commands

<description of why its needed and how problem is solved>

signed-off-by<>

Second Commit should look like:
apps_cmds: Add support for get diag ID command

<description of what it is for>

signed-off-by<>


feat: stubs in ad-hoc ported files for linarith algebra requirements

linarith needs various things that haven't been ported yet. This PR adds various sorried stubs in ad-hoc ported files ahead of where we're currently up to.

bors merge


Publish candidate models for the 2020.2 release

I have two questions:

FP32-INT8 models are not part OMZ anymore? Published only FP16-INT8. I right understand, that they not will be added in future?
what about excluding pipelining models FP16-INT8 from package? (it is available only for encoder (decoded is not supported by POT now) and nobody validate its accuracy yet)


Configure encrypted credentials

https://github.com/etiennestuder/gradle-credentials-plugin
when private repositories are needed to be configured later

rather by using https://github.com/neva-dev/gradle-fork-plugin

Done.

If nexus creds needed just reuse value / add separate in similar way as for fileTransfer / see gradle.user.properties.peb


[TypingIndicator] Request: Log to the .json file who, and on which channel is typing.

TypingIndicator should have a setting to log to the file who, on what channel and when was typing.
User should be able to specify on what servers and channels this feature should work. Would be nice to have output in .json file like this:
[
    { "server": "<serverName>", "timeStamp": "2020-11-05T14:48:00.000Z", "user": "<userName>" }
]

If it is possible it should log only the typing events when user didn't send the message for over for example 1 min.

Feel free to do that yourself but imo that is

logging user metadata which I don't want to do, and it might even be against the betterdiscord terms
just a waste of storage space. over time the log would just grow and grow.

Feel free to do that yourself but imo that is

logging user metadata which I don't want to do, and it might even be against the betterdiscord terms
just a waste of storage space. over time the log would just grow and grow.


Fix enum serialization in rails 4.2

Fixes #798
Our test suite has one model that tests enums, PostWithStatus. Its
spec did not cover touch_with_version. Somehow, probably during
our upgrade to rails 5, we broke touch_with_version for enums.
Unlike other methods, when an enum arrives at
CastAttributeSerializer#serialize during touch_with_version, it
is already a number. We assumed it was always a string.
This is sort of a hack, just handling both strings and numbers in
#serialize. I'd rather figure out how a number got to #serialize
in the first place. However, a hack is acceptable, as it's only
for rails 4.2, and the hack will eventually be deleted when we
drop support for 4.2.

Looks like build 1162 (https://travis-ci.org/airblade/paper_trail/builds/152138082) failed because the table under test (post_with_statuses) lacked timestamp columns. I keep forgetting that touch_without_version requires timestamps. We should probably raise an error instead of failing silently.
Hmmm, still failing on AR 5 / MySQL .. not sure why. Any ideas?

Hmmm, still failing on AR 5 / MySQL .. not sure why. Any ideas? MySQL does have issues with fractional seconds precision .. but I'm not sure ..

Yeah, it was probably a MySQL fractional seconds issue. Adding a Timecop.travel 1.second.since seems to have fixed it.


Using Java Future is limiting ability to easily scale in environments like Scala Akka

As I am attempting to build out a micro service, I am running into issue of using current Java client response which returns Future  turns out that Future response is not really all that Future friendly.
CompletableFuture on the other hand is the way to go. Unlike Future there's a way to get a call back when the call is complete vs having to either Await (block) the thread or while loop while waiting for isDone() to become true which might as well be blocking.
Are there any plans to transition the client into fluent asynchronous processing vs old school Future type?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32964862-using-java-future-is-limiting-ability-to-easily-scale-in-environments-like-scala-akka?utm_campaign=plugin&utm_content=tracker%2F2398677&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2398677&utm_medium=issues&utm_source=github).

CompletableFuture is Java 8 only and Crate is still running on Java 7 (although we're phasing that out).
The CrateClient also has a second overload were you can pass a ActionListener which is callback based. Can you use that instead? I think it should be possible to create some kind of wrapper class if you really want to turn that into a CompletableFuture.
There are no immediate plans to transition the client, but I can see that it could be helpful.
Btw. this repo is for the JDBC driver. The CrateClient itself is part of the https://github.com/crate/crate repo. (I was confused for a moment because I didn't quite understand which Future you're referring too).

Jodok asked me to create 2 issues under crate-jdbc so I followed his guidelines. I'll use the ActionListener in the interim, it has the callback I was looking for :)


test_raw_probs

CHeck this unit test -- we commented it out.,

original_probs returns negative values. Working on this issue.

@pnrobinson I am working on this issue in branch update_hetrogenous_graph_code. I know what caused the error. I made some changes, but the hetnode2vec.py still needs some modficiations. Working on the tests to make sure all tests pass.

I have modified _preprocess_transition_probs_xn2v and  get_alias_edge_xn2v. I have added some more tests. Now, all tests are passed.
Justin had modified the code so that node indices were used. However, I had changed it to the old format. I need to change it back to what Justin had implemented.

Tests are passed. I think we can close this issue.


Customize the parent class of SolidQueue::RecurringJob

Introduction
Allow specifying recurring tasks just with a "command"

I find this feature very interesting.
However, there is one barrier to its use.
That is, the parent of SolidQueue::RecurringJob is fixed at ActiveJob::Base.
I have defined error handling in ApplicationJob as described below.
https://github.com/rails/solid_queue/blob/67b964d5e9d1120e0f198dc1ef538a0aff966133/README.md#error-reporting-on-jobs
This error handling does not work when using “command”.
Proposal
How about allowing customization of the SolidQueue::RecurringJob parent class?
Thinking something similar to MaintenanceTasks.parent_controller.
https://github.com/Shopify/maintenance_tasks/blob/d98544fb3de714a14082b64e6831980945db01cf/README.md#customizing-the-parent-controller-for-the-web-ui
What do you think?
Workaround
Just extend SolidQueue::RecurringJob.
module JobErrorHandling
  extend ActiveSupport::Concern

  included do
    rescue_from(Exception) do |exception|
      Rails.error.report(exception)
      raise exception
    end
  end
end

class ApplicationJob < ActiveJob::Base
  include JobErrorHandling
end

Rails.application.config.after_initialize do
  SolidQueue::RecurringJob.include(JobErrorHandling)
end

Yes, good idea! I'll get this done shortly. I could also allow providing the job class to be used in this case 🤔  Like config.action_mailer.delivery_job.

I just noticed that you already have the ability to customize a job class 👀
(I don't know if this is an official config 🙃)
Rails.application.config.after_initialize do # or to_prepare
  SolidQueue::RecurringTask.default_job_class = MyRecurringCommandJob
end

Oh wow, yes, it is! 🤣  I added this ages ago when I first started implementing this feature. Then, I had to work on other non-solid queue stuff at work, and came back to it recently. I had completely forgotten I had added that! 🤣 I also forgot to expose it or document it 😅  I think this could work! My initial idea was to also allow people to do different things with their command parameter.

So sorry for the delay! 😳 I completely forgot about this 🙈 I've just documented the default_job_class option that was completely undocumented 😅


chore: unit tests for url utilities

Overview

Writing unit tests for url utility methods.
Related Issue(s)

https://issues.newrelic.com/browse/NR-87783
Testing

Run unit tests

Codecov Report

Merging #458 (26214e3) into main (fec8c63) will increase coverage by 7.25%.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           main    #458      +/-   ##
=======================================
+ Coverage      0   7.25%   +7.25%
=======================================
  Files         0     119     +119
  Lines         0    3914    +3914
  Branches      0     966     +966
=======================================
+ Hits          0     284     +284
- Misses        0    2866    +2866
- Partials      0     764     +764

Impacted Files
Coverage Δ

src/common/url/location.js
100.00% <ø> (ø)

src/common/util/obfuscate.js
0.00% <0.00%> (ø)

src/features/metrics/aggregate/index.js
0.00% <0.00%> (ø)

src/common/url/protocol.js
100.00% <100.00%> (ø)

... and 115 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Importing this GLB + exporting as GLTF causes changes.

Ideally doing an import+export is an idempotent operation but tangents are lost and the textures look different.
https://github.com/Yzou-NaO/glTF-Model

There’s an option in the export settings to enable tangents - they’re not preserved through a round trip, I don’t think, but can be regenerated at export.

I confirm that tangents import are not yet implemented

Tangents are now imported as custom attributes


Portfolio Revision

Please have a look at my portfolio and feel free to add any comment.

you can limit the width of the container instead of using  br on some parts or text-align to move the paragraph around or flex

your project looks great.it think it is best if use section to divide part of the pages.


docs: testgrid branch builds proposal

Proposal for using TestGrid on Kurl branches. To be honest, considering the limitations here and the near-term utility of it, I'm thinking the time is better spent coming up with a strategy for supporting 3rd party addons, but I guess that's up to the team.
Thanks to @laverya for working through these ideas with me. The installer spec was all him 🥇 .

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

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


@conda-forge-admin, please re-render

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/dask-sql-feedstock#55.


Test async-index-and-search with two service workers

One for indexing and one for searching?

Make two json-files to be indexed. One small to index first and then one big that can be indexed while the user searches within content in the first one.
A service worker for the indexing part and a service worker for the query part
Make two html files so you can click back and forth and still get info on how indexing is going, i.e. a message for every 1000 documents indexed.

@fergiemcdowall Service workers looks interesting.

As web workers, they don't block the UI when doing processor heavy stuff, like indexing
For a site (or a part of a site / domain) they can run in the background even if you switch pages.
They can pull stuff from cache if up to date instead of using network. This means true offline applications

https://www.digitalocean.com/community/tutorials/demystifying-the-service-worker-lifecycle

With this we can make a offline-first and the basis for non-blocking UI for search-index, and have it framework agnostic. So easier to make a browser application and still your choice of what to use to develop the application.

https://web.dev/service-worker-mindset/

Seems only one service worker can be added to an app for what my purposes are:
https://stackoverflow.com/questions/42474908/using-multiple-javascript-service-workers-at-the-same-domain-but-different-folde

Seems the idea is to add the possibility of running a worker / sharedWorker from a service worker, but not implemented yet.
https://github.com/w3c/ServiceWorker/issues/678
https://github.com/whatwg/html/pull/5601

Hmm, seems it might actually be possible??? Will try
https://developer.mozilla.org/en-US/docs/Web/API/Client

@eklem yes, very interesting!

I'll explore =) If it works out, it could maybe take JSON input and give JSON output as a browser version of Norch.


Develop

Description
spelling fixes by @InnocentBug
Changes
Tests
Known Issues
Notes
Checklist

[ ] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[ ] I have updated the documentation to reflect my changes.

Wow. That seems like a big one.
is that the release commit?

@InnocentBug it shouldn't be, its just a regular merge from develop to main

let me close this and try it again and see if that makes a difference


Builder: Add error handling to check config files

As suggested in issue #345. I need some feedback to make sure I'm going in the right direction.
So far, if I change sources to something like sssources in sources/config.yaml, I get the following error that includes the original traceback and some more helpful error messages giving hints on how to fix the problem:
$ gftools builder sources/config.yaml

Traceback (most recent call last):
  File "/Users/Air/GF-2021/python/gftools/bin/gftools-builder.py", line 86, in <module>
    builder.build()
  File "/Users/Air/GF-2021/python/gftools/Lib/gftools/builder/__init__.py", line 145, in build
    self.build_variable()
  File "/Users/Air/GF-2021/python/gftools/Lib/gftools/builder/__init__.py", line 238, in build_variable
    for source in self.config["sources"]:
KeyError: 'sources'

Error: A key in the configuration file, typically ``config.yaml``, is likely misspelled.
Error caused by key: 'sources'
Possibly misspelled key in the configuration file:  ['sssources']

This is not currently working with some of the other keys, I'm looking into that now...

Thanks for the PR and good start!
I really like it can detect typos. However, I think it would be better if we used something like strictyaml as we do for the packager since it allows us to validate user config files against a schema. Totally happy to pair on this at some point if you'd like?

Sure thanks! I'll look into the strictyaml stuff in the packager and update this PR. I can pair on this whenever works for you.

@googlebot I consent.

@m4rc1e I pushed an update to this that is ready for review.
I will push another update to include a test but wanted to post this in case you have time to give feedback before I write the test.

This is what the output looks like now. I mostly tested this on Texturina, but will try it with a few more when I write the test.
$ gftools builder sources/config.yaml

while parsing a mapping
  in "<unicode string>", line 10, column 1:
    sstat:
    ^ (line: 10)
unexpected key not in schema 'sstat'
  in "<unicode string>", line 84, column 1:
          value: '1'
    ^ (line: 84)

ERROR ****************************************************************

A key in the configuration file, typically ``config.yaml``, is likely misspelled.

Error caused by key: ['sstat']
Possibly misspelled key in the configuration file: ['stat']

@m4rc1e The requested changes have been made and this is ready for review again at commit 9463326.
I also added one additional commit (649425f) adding a test, but I'm not very confident this is the best way to do this. Feel free to discard this commit if everything else looks good to merge. I'm also happy to pair on this at any time.
Thanks!

@googlebot I consent.

@m4rc1e I removed the regular expressions as requested. I couldn't figure out how to do this while retaining all the original functionality of the spell checker, but better done than perfect.
This is ready for review again, thank you!

Cool. I will test and merge tomorrow. Thank you for working on this!


Add undo/redo to convert cells

This PR fixes #17822

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 12 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 41.7%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/notebook/browser/models/cellEdit.ts
6
10
60.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
27280

Relevant Lines:
59366

💛  - Coveralls


Loading gpt2 from pretrained is None

model = EasyTransformer.from_pretrained("gpt2").to(device)
returns None
This was working yesterday, so might be the new update.

Rip, thanks, fixed. I added a wrapper to automatically update self.cfg.device but forgot to return the model!


Enforce Formatting

Problem Description
Small models are having a hard time adhering to the formats we're trying to enforce in our prompts.
For example, when the LLM wants to execute commands, it is supposed to format like this
COMMANDS:
- command_name(arg1="some arg",arg2="some other arg")

However, some LLMs are listing them completely differently, or just listing what commands they want to run in a pseudo code summary that is wrong and/or not detected by the back end as commands to execute.
Proposed Solution
Add Guardrails if determined to be applicable and viable for the purposes of this project.
https://github.com/ShreyaR/guardrails
Alternatives Considered

Change format of prompts to force all inputs and outputs to be in JSON format, which may confuse the smaller models.
Use a smaller model to validate formatting and reformat when the LLM didn't follow format?

Additional Context
Need to research further into Guardrails to see how it will impact the already small token limit.
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

@Josh-XT have you seen this one?
https://github.com/1rgs/jsonformer
But i think people from guardrails repo will integrate with that. Saw some issues about that.

That definitely looks more my speed than Guardrails too.  Everything JSON!  I'll look more at that as the primary idea instead of guardrails.

Work in progress in #244

Done #244


Allow color specification in non-RGB formats

The suggestion was "Would be nice to be able to specify colors as strings supporting standard color strings like "red" or "#0F0F0F""
There are probably some standard modules that can take care of this already, so it's a matter of searching.

Hi @sea-bass
I did some search and my suggestion is to use matplotlib-specifying colors functionality. It provides three classes of colors - Base, Tableau Palette and CSS. Since matplotlib is already a dependency of pyrobosim, this approach might make it easier to implement the required changes.
Let me know your thoughts. If this approach makes sense, I would like to contribute to this issue.
I tested my suggestion using the demo.py script, and here are the results:
Code Changes:
import matplotlib.colors as mcolors

....

def create_world(multirobot=False):
    """Create a test world"""
    world = World()

    # Set the location and object metadata
    world.set_metadata(
        locations=os.path.join(data_folder, "example_location_data.yaml"),
        objects=os.path.join(data_folder, "example_object_data.yaml"),
    )

    # Add rooms
    r1coords = [(-1, -1), (1.5, -1), (1.5, 1.5), (0.5, 1.5)]
    world.add_room(
        name="kitchen",
        footprint=r1coords,
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:brown"]),
        nav_poses=[Pose(x=0.75, y=0.75, z=0.0, yaw=0.0)],
    )
    r2coords = [(1.75, 2.5), (3.5, 2.5), (3.5, 4), (1.75, 4)]
    world.add_room(
        name="bedroom",
        footprint=r2coords,
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:cyan"]),
    )
    r3coords = [(-1, 1), (-1, 3.5), (-3.0, 3.5), (-2.5, 1)]
    world.add_room(
        name="bathroom",
        footprint=r3coords,
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:pink"]),
    )

    # Add hallways between the rooms
    world.add_hallway(
        room_start="kitchen",
        room_end="bathroom",
        width=0.7,
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:olive"]),
    )
    world.add_hallway(
        room_start="bathroom",
        room_end="bedroom",
        width=0.5,
        conn_method="angle",
        conn_angle=0,
        offset=0.8,
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:olive"]),
    )
    world.add_hallway(
        room_start="kitchen",
        room_end="bedroom",
        width=0.6,
        conn_method="points",
        conn_points=[(1.0, 0.5), (2.5, 0.5), (2.5, 3.0)],
        color=mcolors.to_rgb(mcolors.TABLEAU_COLORS["tab:olive"]),
    )

     ....

Results

Okay, understood. I just need one clarification: From what I understand, I need to change any [R, G, B] color values used in source code and YAML files with matplotlib color functionality, while keeping the values the same or similar. Is that correct? If I’m missing something, please let me know.

Yeah, probably just if statements on whether the value is a list of numbers or a string...

okay cool, so I will start implementing with base colors as shown in this image.
{
  'b': (0, 0, 1),
  'g': (0, 0.5, 0),
  'r': (1, 0, 0),
  'c': (0, 0.75, 0.75),
  'm': (0.75, 0, 0.75),
  'y': (0.75, 0.75, 0),
  'k': (0, 0, 0),
  'w': (1, 1, 1)
}

@kumar-sanjeeev knocked this one out! 🏅


[Feature] Remove all documents by ClassNames

Allow CMS admin to remove all of the documents for specific classes (i.e. All news pages only)
There is this feature for reindex by class name, however there it does not exist for removing documents.
I'm working on this feature for a client, I can contribute to module if going to be accepted.

Yea, nice! That sounds like a useful feature 😃


Update react-native to 0.31.0

From https://github.com/broberson/react-native-audio-streaming/commit/3169b8298e46e5f64c60017f72614ef142e2b5a1.

+1

Is it possible to update for react-native 0.32.0 ? Thanks for the great work !

Or perhaps, 0.33.0, now?

Maybe someone can, this is probably an easy task but I have really no time this week sadly.


Update dependencies

Coverage remained the same at 100.0% when pulling cec61f5f638ecf3f876a966febe7bd8e013a95b7 on update-dependencies into 1cd3ddba905bf82d62ce1000b6f98251e6c33cbf on master.

Coverage remained the same at 100.0% when pulling cec61f5f638ecf3f876a966febe7bd8e013a95b7 on update-dependencies into 1cd3ddba905bf82d62ce1000b6f98251e6c33cbf on master.


Tuya

Is your feature request related to a problem? Please describe.
Not a problem
Describe the solution you'd like
I use lightbulbs that use Tuya for their app.  Was hoping you could look at adding that API to your application.
https://www.tuya.com/
Describe alternatives you've considered
Haven't looked at other options as all my bulbs use Tuya.
Additional context
Thanks for creating this app!

I don't have access to a Tuya light, would love to see a PR to add this functionality.

I want this too.
I am not versed with C# and thus, even if I implement the change, I don't have a way to compile it.
I have a working python library - https://pypi.org/project/tinytuya/.
I will try if I can convert it to CPP or c# for someone to continue.

Closing this, @anishsane feel free to open a PR for the lights you are using.


modify efault package private level

modify default package private level
2 files updated;
ServiceExample.java
DuckDuckGo.java

Has been done earlier.


Propogate tracing flag across all subsequent requests

Refactor the code so that when new statements are constructed from
old statements, the 'tracing' flag is preserved.
Relates to: JAVA-815

@rkuris I realized that we had another pending ticket (JAVA-726) that implicitly fixes this one. It's even cleaner because it adds a copy method to Request objects, which avoids using makeRequestMessage for that purpose altogether.
Sorry for not finding this out earlier and making you work on your patch!

@olim7t No worries, glad it got fixed :+1:


Build validation does not trigger a new build automatically for draft pull requests

However, documentation does not contain any description of that. This lack enforces questions on the forums like that: https://stackoverflow.com/questions/60666212/triggering-pull-request-builds-automatically-on-draft-pull-requests-in-azure-dev

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

ID: 7cd986e0-608c-1d8c-6faf-ecdb12e10bc6
Version Independent ID: d60c996e-187e-dd6b-7f3a-dd056a1603f4
Content: Protect your Git branches with policies - Azure Repos
Content Source: docs/repos/git/branch-policies.md
Product: devops
Technology: devops-code-git
GitHub Login: @apawast
Microsoft Alias: sdanie

General question: "Was this change intentional?"
Before this change, I had the habit of making all my PRs drafts to be sure the CI is green before asking for review.
Is there an option you can set in yaml to get the old behavior?

This was always the behavior for PRs from Azure repos. The same is now true for GitHub repos as well.
https://docs.microsoft.com/en-us/azure/devops/release-notes/2020/pipelines/sprint-173-update#github-draft-pull-requests-do-not-trigger-pipelines

I only use Github, so that is why I noticed something changed recently.
How can I get the old behavior? Is there a setting?

No, there is no setting to change that. You can queue those builds manually.

So what seems odd to me about this:

I open a draft PR on Github
I comment /azp run
2 hrs later. It's green, yay!
Mark the PR "Ready for Review", bringing it out of draft.
CI triggers again

Should I file a bug? or is this expected?

I also would like to be able to configure that draft PRs should have automatic builds

For us, at least, this issue has been fixed:

Github Draft PRs trigger builds.
Bringing a PR out of draft does not trigger a build.

However, we are using DevDiv's instance of Azure DevOps -- maybe someone on the team here would know if the public releases have this?

https://docs.microsoft.com/en-us/azure/devops/release-notes/2020/sprint-176-update#configure-draft-pr-validation-for-github-repositories. This is rolling out now.


Add Upgrade button

Could we add an upgrade or update button for each container to allow user to correct the actual container without going through the full process of duplicating it after a full remove of the container ?

We already have an open issue regarding the ability to update a container: https://github.com/portainer/portainer/issues/466


Readme

Description
This PR fixes #
Pull Request for Utils package Documentation  #286
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Aisuko i have made adjustments on the spaces and blanks .
This is all i could come up with for the docs of the utils package.

@Philip-21 💪

Thanks @leecalcote


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2931

Questions ?
Please refer to the Backport tool documentation


Bug 2041351: Remove all queries and logic based on the old Provisioner CRD

Following the removal of the Provisioner CRD in the backend, this PR removes all references to it in the UI. The one major impact of this is that we no longer have a way to display the following warning when selecting target storage classes for a mapping:

This PR simply removes that warning, so we may want to follow up to reintroduce a similar warning later (https://github.com/konveyor/forklift-ui/issues/892). Relevant discussion with @mansam on Slack:

Thanks Gilles!


Update Step03 to include a save of Monitored Systems to the dataRepository

Step 03 needs to save off the monitored systems to the dataRepository for use in Step 14.  Email was sent to Eric on 10/16/2020 requesting information on the need for an update to the specification.
Step 14.pdf

Already done


Gift Entry: Sorting on Certain Columns Causes "No Such Column" Error

On the Gift Entry landing page, when I try to sort list view columns (both Batch table and Templates table) by Form Template, LastModified By, Created By, and Owner, I see this error:
"No such column {!Field Name} on entity 'npsp__DataImportBatch__c'. If you are attempting to use a custom field, be sure to append the '__c' after the custom field name. Please reference your WSDL or the describe call for the appropriate names.

Steps to Repeat

Go to the Gift Entry Tab
Go to the Batches sub-tab.
Click on the Form Template column header. Confirm the error appears.
Now switch to the Templates sub-tab.
Click on the Created By field. Similar error appears.

W-7859492

Manually closing this issue since the related PR has been merged.


Show and upload compilation metrics in the report

This change includes the compilation metrics (compile time and total dispatch size) into the benchmark report.
It also adds two kinds of benchmark series to the dashboard:
MobileNetV3Small [fp32,imagenet] (TFLite) CPU-x86_64-CascadeLake full-inference,default-flags [compilation-time]
MobileNetV3Small [fp32,imagenet] (TFLite) CPU-x86_64-CascadeLake full-inference,default-flags [total-dispatch-size]

The PR benchmark report now contains the following sections:

Regressed Latencies
Improved Latencies
Similar Latencies
Regressed Compilation Times
Improved Compilation TImes
Regressed Total Dispatch Sizes
Improved Total Dispatch Sizes
All Compilation Metrics (all compilation metrics in one table)

The compilation metrics report is only enabled for x86_64 benchmarks in this change. Tested the dashboard uploading on a local instance.

Hi @antiagainst , would you mind taking a look on the naming of metric keys? : )


Fix failing tests on develop from PR #69 merge

PR https://github.com/DataONEorg/slinky/pull/69 introduced four failing tests,
FAILED tests/test_eml201_processor.py::test_processor_extracts_top_metadata - httpx.ConnectError: [Errno 61] Connection refused
FAILED tests/test_eml210_processor.py::test_processor_extracts_top_metadata - httpx.ConnectError: [Errno 61] Connection refused
FAILED tests/test_eml211_processor.py::test_processor_extracts_top_metadata - httpx.ConnectError: [Errno 61] Connection refused
FAILED tests/test_eml_processor.py::test_processor_extracts_top_metadata - httpx.ConnectError: [Errno 61] Connection refused

I'm not sure what's going on here but I'll take a look. These types of tests shouldn't require a network connection so something is probably wrong in test setup.

Fixed unit tests in https://github.com/DataONEorg/slinky/commit/69dab0b1243b9237635b9748598654d598af7aa8 and ended up tweaking how environment variables work so we just use one env var per service (not two). See https://github.com/DataONEorg/slinky/commit/69dab0b1243b9237635b9748598654d598af7aa8 @ThomasThelen.


Add support for the "faint" graphic rendition attribute

Summary of the Pull Request
This PR adds support for the SGR 2 escape sequence, which enables the ANSI faint graphic rendition attribute. When a character is output with this attribute set, it uses a dimmer version of the active foreground color.
References
This is a partial resolution for issues #2916 and #6703.
PR Checklist

[ ] Closes #
[x] CLA signed.
[x] Tests added/passed
[ ] Documentation updated.
[ ] Schema updated.
[x] I've discussed this with core contributors already. Issue number where discussion took place: #6703

Detailed Description of the Pull Request / Additional comments
There was already an ExtendedAttributes::Faint flag in the TextAttribute class, but I needed to add SetFaint and IsFaint methods to access that flag, and update the SetGraphicsRendition methods of the two dispatchers to set the attribute on receipt of the SGR 2 sequence. I also had to update the existing SGR 22 handler to reset Faint in addition to Bold, since they share the same reset sequence. For that reason, I thought it a good idea to change the name of the SGR 22 enum to NotBoldOrFaint.
For the purpose of rendering, I've updated the TextAttribute::CalculateRgbColors method to return a dimmer version of the foreground color when the Faint attribute is set. This is simply achieved by dividing each color component by two, which produces a reasonable effect without being too complicated. Note that the Faint effect is applied before Reverse Video, so if the output it reversed, it's the background that will be faint.
The only other complication was the update of the Xterm256Engine in the VT renderer. As mentioned above, Bold and Faint share the same reset sequence, so to forward that state over conpty we have to go through a slightly more complicated process than with other attributes. We first check whether either attribute needs to be turned off to send the reset sequence, and then check if the individual attributes need to be turned on again.
Validation Steps Performed
I've extended the existing SGR unit tests to cover the new attribute in the AdapterTest, the ScreenBufferTests, and the VtRendererTest, and added a test to confirm the color calculations  when Faint is set in the TextAttributeTests.
I've also done a bunch of manual testing with all the different VT color types and confirmed that our output is comparable to most other terminals.

Btw, here are a couple of screenshots comparing how other terminals handle this. From left to right, top to bottom: XTerm, VTE (Gnome Terminal), Mintty, and then us. In each case I've got a color test pattern rendered with normal attributes first, and with a faint version below. I think XTerm and VTE are a bit brighter than us, but Mintty looks almost identical. The overall effect is about the same though.


Renaming doesn't work as expected with cyclic unions

Description:
$title.
Steps to reproduce:
Consider the following sample.
type CyclicUnion int|CyclicUnion[];

public function main() {
    CyclicUnion x = 1;
}

Renaming CyclicUnion to Union in the variable declaration renames as follows.
type CyclicUnion int|Union[];

public function main() {
    Union x = 1;
}

Changing the name in the type definition doesn't change anything else.
type Union int|CyclicUnion[];

public function main() {
    CyclicUnion x = 1;
}

Renaming the member of the union doesn't do anything.
Affected Versions:
slalpha4-SNAPSHOT

This looks like a bug in the compiler. For cyclic types, it seems to create two type symbols for the same identifier, CyclicUnion. One, the regular type symbol created for the identifier in the type def, the second, for the reference to the type def in the union type. IMO, in the union, it should have the same type symbol as the type def.
I think in the symbol enter we've intended to do this. See https://github.com/ballerina-platform/ballerina-lang/blob/52aec0fe8322fee34e02a766a76ace53a65fdb28/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/analyzer/SymbolEnter.java#L1336. Here, if it contains a cyclic reference, it creates a type symbol, define it, and would later be looked up when resolving the union.
But the issue is, this hasCyclicReference flag is getting set after the above check is executed: https://github.com/ballerina-platform/ballerina-lang/blob/52aec0fe8322fee34e02a766a76ace53a65fdb28/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/analyzer/SymbolEnter.java#L1239


Deploy Checklist: New Expensify 2023-10-05

Release Version: 1.3.78-2
Compare Changes: https://github.com/Expensify/App/compare/production...staging
This release contains changes from the following pull requests:

[x] https://github.com/Expensify/App/pull/23737
[x] https://github.com/Expensify/App/pull/26307
[x] https://github.com/Expensify/App/pull/26778
[x] https://github.com/Expensify/App/pull/27650
[ ] https://github.com/Expensify/App/pull/27836
[x] https://github.com/Expensify/App/pull/27855
[x] https://github.com/Expensify/App/pull/27923
[x] https://github.com/Expensify/App/pull/28188
[x] https://github.com/Expensify/App/pull/28218
[x] https://github.com/Expensify/App/pull/28240
[ ] https://github.com/Expensify/App/pull/28244
[ ] https://github.com/Expensify/App/pull/28350
[x] https://github.com/Expensify/App/pull/28365
[ ] https://github.com/Expensify/App/pull/28368
[x] https://github.com/Expensify/App/pull/28373
[x] https://github.com/Expensify/App/pull/28376
[x] https://github.com/Expensify/App/pull/28385
[x] https://github.com/Expensify/App/pull/28409
[x] https://github.com/Expensify/App/pull/28425
[x] https://github.com/Expensify/App/pull/28426
[x] https://github.com/Expensify/App/pull/28429
[x] https://github.com/Expensify/App/pull/28445
[x] https://github.com/Expensify/App/pull/28459
[x] https://github.com/Expensify/App/pull/28519
[x] https://github.com/Expensify/App/pull/28555
[x] https://github.com/Expensify/App/pull/28575
[x] https://github.com/Expensify/App/pull/28614
[x] https://github.com/Expensify/App/pull/28616
[x] https://github.com/Expensify/App/pull/28624
[ ] https://github.com/Expensify/App/pull/28677
[ ] https://github.com/Expensify/App/pull/28688
[x] https://github.com/Expensify/App/pull/28700
[x] https://github.com/Expensify/App/pull/28709
[x] https://github.com/Expensify/App/pull/28720
[x] https://github.com/Expensify/App/pull/28734
[x] https://github.com/Expensify/App/pull/28735
[x] https://github.com/Expensify/App/pull/28736
[x] https://github.com/Expensify/App/pull/28739
[x] https://github.com/Expensify/App/pull/28786
[ ] https://github.com/Expensify/App/pull/28787
[x] https://github.com/Expensify/App/pull/28791
[x] https://github.com/Expensify/App/pull/28809
[x] https://github.com/Expensify/App/pull/28822
[x] https://github.com/Expensify/App/pull/28881
[x] https://github.com/Expensify/App/pull/28890
[ ] https://github.com/Expensify/App/pull/28929

Deploy Blockers:

[x] https://github.com/Expensify/App/issues/28909
[x] https://github.com/Expensify/App/issues/28911
[ ] https://github.com/Expensify/App/issues/28919
[ ] https://github.com/Expensify/App/issues/28930

Deployer verifications:

[ ] I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
[ ] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
[ ] I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.78-0 :rocket:

https://github.com/Expensify/App/issues/28911 demoted to normal bug
https://github.com/Expensify/App/issues/28909 fixed in staging with this CP PR https://github.com/Expensify/App/pull/28890

Regression TR is at 50%. We are validating few PRs
#28677 failing with #28921

Regression is only 8% left.
#28677 failing with #28921

https://github.com/Expensify/App/issues/28919 fixed in staging

Regression is completed
One open blocker : https://github.com/Expensify/App/issues/28971
unchecked PR:
#28677 failing with #28921

I commented in https://github.com/Expensify/App/issues/28921, we should address the bug in the original issue - the PR was a revert so probably didn't cause the bug, the bug existed already - so I'll check off https://github.com/Expensify/App/pull/28677 from the checklist

https://github.com/Expensify/App/issues/28971 reproduced in prod, checking it off
Looking into the last 3 checkboxes before deploying!

:shipit:


Feature #318 - added Icon picker control

Q
A

Bug fix?
[ ]

New feature?
[x]

New sample?
[ ]

Related issues?
fixes #318

What's in this Pull Request?
Added property pane icon picker control.

Thank you @gautamdsheth for the contribution!
The PR has been manually merged into dev and will be included in the next release.
In a meanwhile you can test the functionality in beta version


Button is expanded when returning back to FAB screen

I have the button on HomeScreen.. when the user navigates to MainScreen by the use of the button and presses back to navigate to the HomeScreen on the top of the page, the FAB is still expanded. please check this issue

The buttons do not close automatically, you must program them to close.
Please refer to the example folder for details.


Auto-create database for UDP input

All other services operate like this, so make UDP service consistent.
This change also adds a README for the UDP service, which fixes issue #4041.

@corylanou

Makes sense.  +1

Great, thanks @corylanou

CHANGELOG conflict only, merging now.


chore: update gamedig dep to 4.3.0

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Updates the GameDig dependency from 4.2.0 to 4.3.0.
Node-GameDig 4.3.0 changelog:

Fix Post Scriptum not being on the valve protocol.
Added support for the Minecraft Better Compatibility Checker Mod.
Halo Online (ElDewrito) - Added support (by @Sphyrna-029)

Type of change
New feature (non-breaking change which adds functionality)
Checklist

[x] My code follows the style guidelines of this project
[ ] (not applicable) I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
[x] My changes generates no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

It is already in Uptime Kuma 1.23.9.


question about update

Hello, can you help me with one moment?
I have two tables:
class Employer extends Table {
  IntColumn get id => integer().autoIncrement()();
  TextColumn get surname => text().withLength(min: 1, max: 50)();
  TextColumn get name => text().withLength(min: 1, max: 50)();
  TextColumn get middleName => text().withLength(min: 1, max: 50)();
  DateTimeColumn get birthDate => dateTime().nullable()();
  TextColumn get position => text().withLength(min: 1, max: 50)();
  IntColumn get children => integer()();
}

class Child extends Table {
  IntColumn get id => integer().autoIncrement()();
  IntColumn get parentId => integer()();
  TextColumn get surname => text().withLength(min: 1, max: 50)();
  TextColumn get name => text().withLength(min: 1, max: 10)();
  TextColumn get middleName => text().withLength(min: 1, max: 50)();
  DateTimeColumn get birthDate => dateTime().nullable()();
}

My goal is update quantity of children in employer table, when i update child table.
And tables must be linked (employer.id = child.parentId).
I was looking for a similar example in tutorial but I didn't find it.
Thank you

tables must be linked (employer.id = child.parentId).

You can use REFERENCES for that:
IntColumn get parentId => integer().customConstraint('NOT NULL REFERENCES employer (id)')();

To enforce this, you need to enable foreign key constraints:
MigrationStrategy get migration => MigrationStrategy(
  beforeOpen: (details) async {
    await customStatement('PRAGMA foreign_keys = ON');
  }
);

My goal is update quantity of children in employer table, when i update child table.

This is more complicated. You could use generated columns for that, but moor doesn't support those at the moment. If it's possible, you could just remove the children column and replace with a COUNT() in queries. Otherwise you could use a trigger to automatically update that value.


[stable/graylog] provisioner job hook

Signed-off-by: Volodymyr Linevych<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR optionally installs the annotation that triggers the graylog provisioner job on chart release upgrades.
Triggering the provisioner job after release upgrade might be useful, depending on a logic of your provisioner script.
In my case, I install content packs and propagate saved searches.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @kongz

@vlinevych Should we change to .Values.graylog.provisioner.annotations? Then we can freely add more annotations without modifying chart in the future. And please update the README.md as well when you add another field on values file.

@KongZ Thank you for the suggestion. Implemented.

/ok-to-test

/lgtm

/retest


ENH: Wrap all BLAS routines

Remaining tasks are

[x] Add tests for the missing ones
[x] Wrap trsm

Closes #1915

There are two files for BLAS tests and it seems that I've chosen the wrong one. Should I move the new tests into test_blas.py or should I keep them under test_fblas.py ?

I can't detect a clear logic in the split between test_blas.py and test_fblas.py. They're about the same length and there's some overlap in functions they test. So not sure.
If you see the logic and can maintain or improve it, go for the move though I'd say:)

I think it's some leftover from the days we had both blas and cblas
wrappers, most likely not relevant any more.

I am really surprised that this test ```
def test_scal(self):
    for p in 'sd':
        f = getattr(fblas, p+'scal', None)
        if f is None:
            continue
        assert_array_almost_equal(f(2, [3, -4, 5]), [6, -8, 10])
    for p in 'cz':
        f = getattr(fblas, p+'scal', None)
        if f is None:
            continue
        assert_array_almost_equal(f(3j, [3j, -4, 3-4j]), [-9, -12j, 12+9j])
    for p in ['cs', 'zd']:
        f = getattr(fblas, p+'scal', None)
        if f is None:
            continue
        assert_array_almost_equal(f(3j, [3j, -4, 3-4j]), [-9, -12j, 12+9j])

under `TestFBLAS1Simple` class has been passing. It should have been (and does on my build) failed in the 3rd assertion. Can anyone confirm this?

`csscal` and `zdscal`, as their name imply meant for real,float scalars but here it is multiplied by a complex number.

Here is my regular console output with SciPy 0.19.1
In[15]: sp.linalg.blas.zdscal(3j,[3j, -4+0j, 3-4j])
Out[15]: array([ 0.+0.j, -0.+0.j,  0.-0.j])

In[16]: sp.linalg.blas.csscal(3j,[3j, -4+0j, 3-4j])
Out[16]: array([ 0.+0.j, -0.+0.j,  0.-0.j], dtype=complex64)

The code pasted above is not the same as in the repository:
https://github.com/scipy/scipy/blob/master/scipy/linalg/tests/test_blas.py#L180

Tests seem to be stuck but this is ready for review from my side.

@ilayn could you according to #7847 add *sycon, *hecon routines as well? Then I can apply our discussion in #7847.

Yes, I'll add as soon as I have time with another PR.

Great! Thanks!

This would be really nice to get into the next release, so if someone has the bandwidth and expertise to review in the next 24 hours, please do!

I won't be able to check and do #7847 before next week. :(

I might have time -- @ilayn if this is Fortran wrapping plus tests for it I might be able to look. Do you think it's good enough if I look, given I've done two (one accepted one in progress) PR to add LAPACK functions, or would you be more comfortable if someone more experienced looked?

@larsoner go for it I'd say! If you're not 100% about something maybe just flag it, and someone else can check post merging.

Sure @larsoner the more the merrier. It's mostly repetitive wrappers anyways. Tests are there but let me know if more is needed.

All tests pass (also checked 32-bit Linux), and positive review. So in it goes. Thanks @ilayn, and @larsoner for the review!

I'll do the release notes update.


Add Yoruba to the supported languages

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@smyja, Thanks for your suggestion and letting me know! I've just committed changes on 56af8dc. From now on, you can use the BBC News API to get news in Yoruba. This endpoint will respond with news from https://www.bbc.com/yoruba.

@smyja, Thanks for your suggestion and letting me know! I've just committed changes on 56af8dc. From now on, you can use the BBC News API to get news in Yoruba. This endpoint will respond with news from https://www.bbc.com/yoruba.

Great, you should create a new release so people can upgrade.

you

@smyja, What did you mean by 8k though?

@smyja, Thanks for your suggestion and letting me know! I've just committed changes on 56af8dc. From now on, you can use the BBC News API to get news in Yoruba. This endpoint will respond with news from https://www.bbc.com/yoruba.

Great, you should create a new release so people can upgrade. Also, how do you do paging? i need to scrape at least 8k

@smyja, What did you mean by 8k though?

I meant 8,000 pages

Seems, https://www.bbc.com/yoruba has only 1 page! Where did the 8k pages come from?

@smyja, Thanks for your suggestion and letting me know! I've just committed changes on 56af8dc. From now on, you can use the BBC News API to get news in Yoruba. This endpoint will respond with news from https://www.bbc.com/yoruba.

Great, you should create a new release so people can upgrade. Also, how do you do paging? i need to scrape at least 8k

@smyja, What did you mean by 8k though?

I meant 8,000 pages

Seems, https://www.bbc.com/yoruba has only 1 page! Where did the 8k page come from?

Categories/topics have more than 1 page. Here's one https://www.bbc.com/yoruba/topics/c340q0y3p5kt

@smyja, Thanks for your suggestion and letting me know! I've just committed changes on 56af8dc. From now on, you can use the BBC News API to get news in Yoruba. This endpoint will respond with news from https://www.bbc.com/yoruba.

Great, you should create a new release so people can upgrade. Also, how do you do paging? i need to scrape at least 8k

@smyja, What did you mean by 8k though?

I meant 8,000 pages

Seems, https://www.bbc.com/yoruba has only 1 page! Where did the 8k page come from?

Categories/topics have more than 1 page. Here's one https://www.bbc.com/yoruba/topics/c340q0y3p5kt

Oh, you meant that!? Actually BBC News API responds with the news from the main page (https://www.bbc.com/yoruba) as it is considered as latest and important! Though I beleive it is possible to respond with other pages news if wanted.


[ie/bilibili] Provide fallback for playinfo extraction

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

[site-bug]
the site has removed window.__playinfo__ from the page source. This pr adds a fallback by downloading the playinfo from the API on extraction failure. bvid and cid extraction is intentionally made fatal.
Fixes #11665
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)

I have the same issue this morning.   Everything was working fine last night.

Things would be much easier if we are assuming that window.__playinfo__ is not coming back
show patch
diff --git a/yt_dlp/extractor/bilibili.py b/yt_dlp/extractor/bilibili.py
index 71e2dbdd0..c5b7d718b 100644
--- a/yt_dlp/extractor/bilibili.py
+++ b/yt_dlp/extractor/bilibili.py
@@ -18,7 +18,6 @@
     InAdvancePagedList,
     OnDemandPagedList,
     bool_or_none,
-    clean_html,
     determine_ext,
     filter_dict,
     float_or_none,
@@ -640,34 +639,7 @@ def _real_extract(self, url):

         initial_state = self._search_json(r'window\.__INITIAL_STATE__\s*=', webpage, 'initial state', video_id)
         is_festival = 'videoData' not in initial_state
-        if is_festival:
-            video_data = initial_state['videoInfo']
-        else:
-            play_info_obj = self._search_json(
-                r'window\.__playinfo__\s*=', webpage, 'play info', video_id, default=None)
-            play_info = None
-            if not play_info_obj:
-                if traverse_obj(initial_state, ('error', 'trueCode')) == -403:
-                    self.raise_login_required()
-                if traverse_obj(initial_state, ('error', 'trueCode')) == -404:
-                    raise ExtractorError(
-                        'This video may be deleted or geo-restricted. '
-                        'You might want to try a VPN or a proxy server (with --proxy)', expected=True)
-                play_info = self._download_playinfo(initial_state['videoData']['bvid'],
-                                                    initial_state['cid'], headers=headers)
-            else:
-                play_info = traverse_obj(play_info_obj, ('data', {dict}))
-            if not play_info:
-                if traverse_obj(play_info_obj, 'code') == 87007:
-                    toast = get_element_by_class('tips-toast', webpage) or ''
-                    msg = clean_html(
-                        f'{get_element_by_class("belongs-to", toast) or ""}，'
-                        + (get_element_by_class('level', toast) or ''))
-                    raise ExtractorError(
-                        f'This is a supporter-only video: {msg}. {self._login_hint()}', expected=True)
-                raise ExtractorError('Failed to extract play info')
-            video_data = initial_state['videoData']
-
+        video_data = initial_state['videoInfo' if is_festival else 'videoData']
         video_id, title = video_data['bvid'], video_data.get('title')

         # Bilibili anthologies are similar to playlists but all videos share the same video ID as the anthology itself.
@@ -693,10 +665,9 @@ def _real_extract(self, url):
         old_video_id = format_field(aid, None, f'%s_part{part_id or 1}')
         cid = traverse_obj(video_data, ('pages', part_id - 1, 'cid')) if part_id else video_data.get('cid')

+        play_info = self._download_playinfo(video_id, cid, headers=headers)
         festival_info = {}
         if is_festival:
-            play_info = self._download_playinfo(video_id, cid, headers=headers)
-
             festival_info = traverse_obj(initial_state, {
                 'uploader': ('videoInfo', 'upName'),
                 'uploader_id': ('videoInfo', 'upMid', {str_or_none}),

I would want to try to adapt the error handling to the API call, too, though

I'm thinking of adding a param fatal to _download_playinfo and change the method to return None if the extraction has failed but fatal is set to false. Then we can handle the error in BiliBiliIE._real_extract


Commit E1603 and E1702 are the same 1aab849 breaks Home Assistant integration

1aab849b593c95b6ac95b9b40376f5163ea384ed breaks Home Assistant integration with TRADFRI control outlet, with the current change:
model: 'E1603/E1702',
in home assistant outlets turn unavailable.
reverting the change back:
model: 'E1603',
In Home Assistant outlets function as expected.

Thanks, fixed!


Traversals In Tree (PreOrder,InOrder,PostOrder)

https://www.geeksforgeeks.org/tree-traversals-inorder-preorder-and-postorder/
Hmmm, nice copy&paste...

https://www.geeksforgeeks.org/tree-traversals-inorder-preorder-and-postorder/
Hmmm, nice copy&paste...

Wow, means if you know the concept then you can't copy paste the things if you know that thing already in other language(because I do programming in C++).


I can't seem to compile properly

I've been trying to compile the build, everything seems to be fine but i cant find any .html file inside any folder
i'm not sure if this is normal result of the build but these are the files built

More than an issue im in need of help because i suck building stuff :P :<
Thanks for the help @0xFireball

Hi @ExodusTime,
I'm glad to see interest in this project!
Regarding the build, everything looks right; this is the compiled output from the app. In fact the compiled client application is in Views/, but to actually host the server, you need to run the server application, which you can do like this:
dotnet PenguinUpload.dll
That will use the dotnet runtime to run PenguinUpload on .NET Core.
Be sure to first set the configuration options mentioned here: https://github.com/0xFireball/PenguinUpload#setup-instructions-from-source
For setup you will want to enable registration and create your account, then stop the server, add your username to the admins list, and disable registration if you like.


fix(ui): social media icons alignment

Fixes #2712
Describe the changes you have made in this PR -
Before:

padding-top and margin-left property on the social-media-icons restricts it from being aligned.
After:

removed padding-top and margin-left property on the social-media-icons, added some margin to the container to align it, and lastly added padding to give some spacing to elements.
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@vedant-jain03 @tanmoysrt can you please review this ?

how about this @tanmoysrt I also decreased size of the icons a little bit with some extra spacing

 how about this @tanmoysrt I also decreased size of the icons a little bit with some extra spacing (didn't commit yet)

This looks fine !

Let's remove border from this!

Done

@theslowdeveloper Can you remove the last commit I request for border, that is fixed under #4181

@theslowdeveloper Can you remove the last commit I request for border, that is fixed under #4181

Done


Bug: Saleor performance slow

What are you trying to achieve?
We experience a sudden performance issues in Saleor.
The logs we see in Jaeger suggest response times around 30-40 seconds.
Can somebody help us to debug?
SO FAR

On Saleor level, seems like queries that take longest are graphQL.
On database level, we noticed a number of idle connections.

Steps to reproduce the problem
n/a
What did you expect to happen?
n/a
Logs
Idle connection in DB

Jaeger tracing logs
jaeger tracing_1.txt
jaeger tracing_2.txt
Environment
Saleor version: 2.10.0
Servers: AWS EC2
Deployment method: docker
Database: PostgreSQL running in AWS
Tracing: Jaeger tracing

Hey @madmonkeyworks it seems you are using old version of Saleor that is no longer maintaned we advice you upgrade

@timuric Even after upgrading the saleor core to the latest version the graphql responses are taking more time.


Add up the Numbers from a Single Number

Before jumping into Pr first comment for assign.
Create a function that takes a number as an argument. Add up all the numbers from 1 to the number you passed to the function. :
https://edabit.com/challenge/XDx6JoaumAimRHgEt :
Choose one Cateorgy - Easy
question category : Numbers

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid can I work on this?

[ ✔] Task Assigned @jayanthj737

@aayushi-droid Added addUp #53


Fix crash during message parsing (and other small fixes)

Included fixes:

Relogin delay is not parsed as a number when set from env variabile
There's a wrong type translation of Message to SaicMessage since that has been moved into the core client library
The application kept running even if one of the core tasks crashed, making It impossible to restart via docker, Kubernetes, etc.
Exception logging made it difficult to see where the application was crashing

This closes #11

Rebased on top of develop. Remaining fixes:

There's a wrong type translation of Message to SaicMessage since that has been moved into the core client library
The application kept running even if one of the core tasks crashed, making It impossible to restart via docker, Kubernetes, etc.
Exception logging made it difficult to see where the application was crashing

This fixes #22

Thank you very much for your contribution. I had a busy week at work, so I couldn't have a look earlier.
You say: There's a wrong type translation of Message to SaicMessage since that has been moved into the core client library
I don't find where this is fixed in 61abb87.

It's the message = convert(msg) that you have already fixed in 530fb42, right?

Thank's goes to @nanomad.


Programatically setting map region using animateToRegion fires onRegionChange

this.refs.map.animateToRegion(this.region, 1000)
The code mentioned above moves map to the specified region but also fires onRegionChange .
Also the prop region  for MapView does not fire onRegionChange when the prop is updated via setState.
I believe onRegionChange should be fired only when user plays with the map and both the techniques should give consistent result.

I think we're having a similar issue. Have you had any resolve to this issue or do you have any ideas?

Merging to #1620


[Feature]: Add hotkey to pin/unpin current tab

Describe the solution
An event listener for a keybind like Ctrl + Shift + x that pins the current tab.
Related issues
No response
Example usage
Press Ctrl + Shift + x with an unpinned tab selected => current tab pinned
Press Ctrl + Shift + x with a pinned tab selected => current tab unpinned
Alternative solutions
Automatically pin tabs matching x url? Nothing sustainable.
Additional context
No response
Code of Conduct

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

Duplicate of #7


BUG: Fix the error showing on signup and login .

Issue Template
Description
Fix the error showing on signup and login .
Steps to Reproduce
Please provide step-by-step instructions on how to reproduce the issue:

Go to 'signup' or login
Fill details
Submit
See error

Expected Behavior
After fill up the details correctly it should signup or login
Actual Behavior
Showing error after fillup the details .
Screenshots

hey @theashikagupta  , could you please assign it to me

Assigned to you! @Saurabhchaudhary9799

hey @theashikagupta ,  i just checked this issue , the signup page is working properly
Screencast from 2024-10-02 23-04-55.webm

@Saurabhchaudhary9799 fork repository then it will show error.

i got it.

i will fix this issue

hey @theashikagupta , it should also working on localhost after fork
new.webm
please make sure whether your server is connected or not , or this error may occur because of network issue

Okay then I closed this issue.

@theashikagupta  ,due to this issue i am not able to work on issue #44  and #26 .is it okay to wait until this issue solve.

@theashikagupta please check PR i have fixed the issue #49 of login and logout.
https://github.com/user-attachments/assets/c78fced4-0c2c-450d-886d-9f0414504ea1

hey @yashmandi @theashikagupta , Kindly check my PR

@Saurabhchaudhary9799  In my local , signup and login is not working again . Can you please check ?

@theashikagupta , yeah

@theashikagupta , there is lots of error in router.js in my local machine after pull

@theashikagupta , can you tell me what is exact error with the help of screenshots of  backend console

@theashikagupta , send me screenshots of  backend console also

https://github.com/user-attachments/assets/0b6c0b33-2ac3-42c3-91f1-27b4b05eda81

@theashikagupta , don't put quotes around string

@theashikagupta , don't put quotes around string


Remove FreeBSD build instructions since they don't work

Description of the Change
As mentioned in #17168, the current FreeBSD build instructions don't work. This removes them to prevent people from wasting time on instructions that we know are broken.
Alternate Designs

https://github.com/atom/atom/pull/17168#issuecomment-382516855

Why Should This Be In Core?
It already is.
Benefits
People will stop wasting time on broken instructions and be directed to or search for #6035, directing more attention to the effort of getting things working again.
Possible Drawbacks
People will think that Atom doesn't work on FreeBSD (which at this point, it doesn't) and give up.
Verification Process
N/A
Applicable Issues

#6035
#17168
electron/electron#3797

/cc @Arcanemagus @Ben3eeE to make sure I didn't miss anything :grinning:

@lee-dohm https://github.com/atom/atom/blob/master/docs/README.md also mentions FreeBSD.

Thanks @Ben3eeE! Done.


Saving the search criteria in the DbFull-Text search section and later listing these records and using them in the search section again.

Is your feature request related to a problem? Please describe.
Search criteria cannot be set in the Db Full-Text section and cannot be used in the search process again.
Describe the solution you'd like
To be able to save the search criteria, list them on a screen and reuse the selected criteria in the search process.
Describe alternatives you've considered
It is possible to save the search made in the search section of the interface via a button and then see it listed on a dialog screen via another button. Listed search criteria must be deletable and selectable. The search process is carried out by setting the selected criteria into the search fields.
Additional context

Thank you for the idea!
We decided to do the following:

add the search history in the dropdown
add a possibility to edit this history

Thanks for your return. We have provided a solution to this issue. I would like to present it to you.22.11.2023, 15:51, "Anya Ovsiannikova" @.***>:
Thank you for the idea!
We decided to do the following:
add the search history in the dropdown
add a possibility to edit this history
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


SSLVerifyHost: Make some functions static

These functions shouldn't be visible outside of this file. This was caught by
-Werror=missing-declarations.
Signed-off-by: Uli Schlachter<EMAIL_ADDRESS>

:+1:

cherry-picked to 1.6: https://github.com/znc/znc/commit/5f9ad5cc87cb4f5cc987b7c43d2d19a36937e53f


Modified code that deals with repositioning popover on orientation change

The code that existed after calling the delegate's willRepositionPopoverToRect method was incorrectly comparing pointers -- the code had #pragmas around it to stop the compiler warning, but it was a valid warning - the code wasn't doing what was intended.  Changed this block to initialize the local variables passed by reference to the delete method, and then correctly check values that may have been returned.  Made the assumption that if you pass in CGRectZero for the bounds, then a valid change in bounds would not be CGRectZero (seems reasonable).

:+1:


It is possible to use fsevents_to_vm with standard NFS server ?

Hello, thanks for your hard work, very much appreciated!
I have 2 questions:

Why have you choosed unfsd instead of native nfs server ? It is seems to be 30% slower than the kernel one, we are building a very complex drupal application in almost 12 minutes more (42 instead of 30).

I am trying to mount Users using the standard nfs driver (using docker-machne-nfs) it is mounted and it works fine, but trying to debug fsevents_to_vm i see that events gets sended to vm:

cli ❯ fsevents_to_vm start --debug --ssh-identity-file ~/.docker/machine/machines/dinghy/id_rsa --ssh-ip $(dinghy ip)
Watching /Users/paolomainardi and forwarding events to Dinghy VM...
touching<PHONE_NUMBER>12.22:/Users/paolomainardi/tmp/prova
touching<PHONE_NUMBER>12.37:/Users/paolomainardi/tmp/ciao
touching<PHONE_NUMBER>12.55:/Users/paolomainardi/tmp/ciao
touching<PHONE_NUMBER>12.56:/Users/paolomainardi/tmp/ciaoasdfasdf
touching<PHONE_NUMBER>12.58:/Users/paolomainardi/tmp/ciaoasdfasdf12341234
touching<PHONE_NUMBER>14.35:/Users/paolomainardi/tmp/ciaoasdfasdf12341234
touching<PHONE_NUMBER>14.39:/Users/paolomainardi/tmp/ciao

But i can't see events reach the VM:
root@c7e287e86ee3:/fstest/tmp# inotifywait .
Setting up watches.
Watches established.

Thanks!

I've answered that before but I can't find it now... I'll add something to the FAQ. There's two main reasons we didn't go with the OSX system NFS, both related to the fact that we wanted to expose the user's entire $HOME dir the same way that docker-machine does:

I wasn't able to lock down the system NFS server to the extent that I felt comfortable exposing the user's entire home directory using it.
The system NFS server doesn't allow nested exports -- in other words, if you export /Users/me, you can't also export /Users/me/some-other-project. This is a problem for us, because we have projects using Vagrant with NFS shares, and so exposing the whole home dir broke those Vagrant projects.

At first glance I don't see why your test wouldn't work. Have you confirmed that the same test works for you using Dinghy's default NFS setup?

@codekitchen thanks a lot for your reply, everything is quite crystal clear now.
I found why fsevent_to_vm doesn't work, because the container should mount the absolute path, the provided example here: https://github.com/codekitchen/fsevents_to_vm#testing is broken.
In order to test it you should do:
docker run --rm -it -v /Users/user/somedir:/Users/user/somedir ubuntu:trusty
apt-get update && apt-get install -y inotify-tools
inotifywait /Users/user/dir

Anyway, i think that having an option to just use the osx nfs server should be added, because the performance loose using unfsd is quite high, up to 30% in our case.
Is there a way to install dinghy without installing/running the unfsd server ?

Closing in favor of https://github.com/codekitchen/dinghy/issues/274


Interpolation values should support VNode

Version
3.0.9
Reproduction link
https://codesandbox.io/s/romantic-raman-hhhoi?file=/src/App.vue
Steps to reproduce
Interpolation value is a vnode
What is expected?
display as HTML
What is actually happening?
display as string.

This is intentional, text interpolation always stringifies it. If you want to render VNode in template, you need to use dynamic components, see https://codesandbox.io/s/laughing-hoover-fgipn?file=/src/App.vue


Some gogs edited scripts contains CRLF line endings

i had been noticing this for a while but i didnt know what it was . but recently i found it again when building a docker image.
when you edit some scripts in gogs it kind of restructure the script which will be throwing error when building.
but if you edit that same script in github or gitlab it builds without error.
After a long searching , this stackoverflow question helped me track the bug
http://unix.stackexchange.com/questions/56024/bash-korn-shell-script-edited-on-windows-throws-error-m-not-found
i had to use " sed -i 's/\r$//' cxStopAllServicesOnSERVER.sh " to clean the script before building.
that works for one script, but am worried if there are many affected.
To reproduce
clone this https://try.gogs.io/oganisa/CRLF_Edit_Bug
build the Dockerfile
then try to run the image , you will get an error
then try to enable the RUN sed command in the dockerfile , then the image will run without error
check my build below
root@scw-9232fa:~/CRLF_Edit_Bug# ls
Dockerfile  README.md run.sh
root@scw-9232fa:~/CRLF_Edit_Bug# docker build -t crlf-edit-bug .
Sending build context to Docker daemon 78.85 kB
Step 1 : FROM debian:jessie
---> 73e72bf822ca
Step 2 : ADD run.sh /run.sh
---> Using cache
---> 3dc339462713
Step 3 : RUN chmod 755 /*.sh
---> Using cache
---> c5da17f000fa
Step 4 : ENV PHP_UPLOAD_MAX_FILESIZE 10M
---> Using cache
---> 00a95fba3a19
Step 5 : ENV PHP_POST_MAX_SIZE 10M
---> Using cache
---> 4c20c22f8341
Step 6 : ENV SQLBUDDY_URL https://codeload.github.com/try/sqlbuddy/tar.gz/v1.0.0
---> Using cache
---> 1fb657f48442
Step 7 : ENV HTTP_DOCUMENTROOT /var/www/html
---> Using cache
---> ab2daac693b7
Step 8 : CMD /run.sh
---> Using cache
---> 24d8278529d6
Successfully built 24d8278529d6
root@scw-9232fa:~/CRLF_Edit_Bug# docker run -ti -d crlf-edit-bug
3a1daaf0b90e3839a5c05194e8e3b979a72b7337c9b88b1f60ff0c6f3a88872f
docker: Error response from daemon: Container command '/run.sh' not found or does not exist..
root@scw-9232fa:~/CRLF_Edit_Bug#

@richmahn please can you try and reproduce this error and see if you can fix it please

Thanks for the feedback!
But I have no clue without knowing version info, and this commit seems fixed it.


Unknown plugin "transform-replace-object-assign"

Hey guys, running into an error when trying to use this module. Here is what I'm getting:
ERROR in ./~/react-dates/index.js
Module build failed: ReferenceError: Unknown plugin "transform-replace-object-assign" specified in "/Users/user/Programming/workspace/PlusOne/node_modules/react-dates/.bab
elrc" at 0, attempted to resolve relative to "/Users/kendrick/Programming/workspace/PlusOne/node_modules/react-dates"

Any ideas?

(#530)
Are you babel-transforming node_modules? If so, don't - you should only be transpiling your own code.

@ljharb I've got a react native app where I'm trying to include the airbnb-prop-types package adn I'm getting this error. How do I go about changing the settings to ensure node_modules isn't being transpiled?

It’s a flaw in the React native packager. The best solution right now is to add a postinstall script that removes all babelrc files inside node_modules.


KAFKA-10559: Not letting TimeoutException shutdown the app during internal topic validation

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

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

LGTM
Looks like the Java 8 build may have stalled? The Java 11 & 15 builds both passed a while ago

Thanks for the PR @vamossagar12!
No ideal to just remove the test instead of adjusting them, but we can do this in a follow up PR. @ableegoldman told me she will take care of it.
Merged to trunk and cherry-picked to 2.7 branch.

Follow up PR: https://github.com/apache/kafka/pull/9446

Thanks @mjsax . I thought these were time out related tests so ended up removing them. Thanks for the feedback :)

Well, that are timeout related: they checked if we send the shut-down signal. After the change, they should check if the TimeoutException is thrown and only kills the local thread. :)


Consider Handling Case Where Upstream Data Hasn't Changed

For instance: https://github.com/chainguard-dev/image-comparison/actions/runs/3427973545
There wasn't any change in the data. This is a corner case that I don't expect to happen often.

nvm. Not important enough to fix, at least for now.


Avoid shedding requests on the initial load

When the initial load is applied - little_loadshedder discards a number of requests.
I was wondering if we can add some option to initially just do the measurements and only after some time or some number of requests to enable shedding.

Hmmmm, that's a reasonable request although I can't immediately see a way to implement it. My thoughts when implementing this is that when starting up a service you'd slowly ramp-up load to it via an external mechanism, but that does require a lot of other infrastructure.
One option would be to let users specify the initial values for concurrency and queue size (or something equivalent like concurrency and "average latency at concurrency". I'd want to add a nice way of extracting those values from a test instance, but it would let it start up with good guesses for the queue sizes.
The other option which you've suggested is have some sort of grace period, that';s possible but I'm not sure how it would work. How do you decide when the loadshedder should kick in? Also, it currently uses latency measured while at concurrency capacity to determine what the best value is and it's not obvious it would get good data if it wasn't limiting the concurrency.
What do you think? What would be the most useful to you?

Thanks for the response!
Yes, it's a bit tricky, I agree.
For now I've forked the little-loadshedder and initially implemented a grace period in the number of requests. While the grace period is active (e.g. first 1k requests) it just measures the latency of the inner service. But as you said it's not perfect since we're not limiting the concurrency - I am yet to test this in our production.
Gracefully ramping up the load does not nicely work in certain use cases like ours - we use DNS for load balancing and the load is applied quite steeply which leads to false positives.
Frankly I'm not sure what would be the right approach, but maybe the one that you suggested with specifying the initial values for queue/concurrency semaphores would be a good fit for most cases. I can try to implement that approach also, especially if the grace period approach won't work as intended and see how that works on real load.

That would be great! As you can probably tell I'm not maintaining this library particularly actively but I'll happily make small changes like this and review anything you throw my way.


FIX-#2313: improved handling non-numeric types at 'mean' when 'axis=1'

Signed-off-by: Dmitry Chigarev<EMAIL_ADDRESS>
What do these changes do?

We are currently checking the performance. Please, do not merge the PR for now.

The perf is okay.


Can receive packets but can't send

Hello. I followed your tutorials on YouTube, I'm working with an Arduino Nano + MCP2515 8MHz. I built your GUI, used Sandeep Mistry library with your modification on the MCP2515.cpp and .h files and open your canSniffer.ino and I'm correctly reading at 500Kbps both in the Arduino IDE serial monitor and in the GUI, the packets comes with no problem.

I already mapped some packets in my electric motorcycle Ovaobike MCR-S. The problem comes when I try to send a frame because nothing happens. I tried almost all the decoded packets to no avail. If I try to keep sending them continuously sometimes something changes on my bike screen and then it returns as before, but nothing more. On the terminal I see "wrong packet!" and sometimes it happened something like "no permission to write".
In this example you can see I'm trying to send 3 packets (I even tried with 2 or 1 only).

Here the message screen ready to push the packets. The ID 606 at D1 changes autonomously sometime so is marked as new packet.

Here I pushed the packets. Only 217 at D2 changes, then returns to normal, then changes again some seconds later, despite the 100ms send delta.

While the terminal returns errors

I have no idea how to solve this, can you help me?

I should add that I think that the system is not workind due to the constant resending of the legit frames that override my frames. I still haven't found a solution.


Does Vaadin Router support relative links ?

Hi,
I've an application on which I try to use Vaadin-Router, but I've an issue with relative paths in links.
If I'm routed on a page http://localhost/user/tom, then I click on a link alice, so the resulting location is http://localhost/alice instead of http://localhost/user/alice.
Is it a known behavior?
Thanks

Vaadin Router resolves all relative URLs relative to the base URL of the app, not relative to its current location. That is, the behavior described in this ticket is expected.
In single-page apps, when routing happens on the client side, that seems to make more sense as the default option. That allows keeping the source code on average simpler. E.g. a navigation component does not need to know the current location in order to render a list of navigation links. When it's rendered as a part of the user/tom page the link to the users view is the same as when it's rendered as a part of the alice page (./users).
In order to make a component that has a link relative to the current location, one would need to  calculate its href dynamically based on the app's base URL, and the current location, e.g. .


Utilization of GEM library for MS Thesis

During literature review for my Master's thesis, I came across your paper regarding Deep Q-Learning based DTC of PMSM. My idea was to work on motor control algorithm by using GEM library. I am trying since 2 months, but is unable to run the provided example codes because of various bugs arising due to non-compatibility of outdated libraries. I also tried to use Stable Baselines3 but it doesn't support tuple data type for observation space. My request is to kindly suggest some way out in order to proceed further with this DQ-DTC code as I am already running out of time.
Looking forward for your response. Thankyou.

Thanks a lot! I have been able to execute the code after your provided solution of updated libraries. But, the files contain only training and validation codes. Can you kindly guide me that what other code files would be required for successful verification of this research? So that I can work on this technique to implement some other motor control algorithm.
Looking forward for your early response.


Change annotations to python-36 style

Pull Request Test Coverage Report for Build 396

21 of 21 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 93.326%

Totals

Change from base Build 392:
0.06%

Covered Lines:
797

Relevant Lines:
854

💛  - Coveralls


docs(META): add links to source for each package in api reference

If this is useful, another option would be to include the source links in the index instead.

[ ] There exists an issue discussing the need for this PR
[ ] I added new tests for the issue I fixed or built
[ ] I ran make test FOO for the package FOO I'm modifying
[X] I used make commit instead of git commit

This sounds reasonable. :)

Can you drop @staltz's commit? Then I'll merge

@jvanbruegge sorry, just saw this. I think I have done what you asked but am not a git rockstar/ninja/paladin.

Perfect, thanks for the PR!!!


Add Build & Test on Pull-Requests

Checking rules

Check ESLint on node 14 (https://github.com/actions/setup-node)
Prettier (https://github.com/hexacta/prettier-check)


Fixing update script to properly upgrade CLI.

@ritwik20,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

LGTM


Add python script and cli to read values

Add quick python script to read from the sensor and print to terminal and a csv file. Should enable testing and characterizing of boards without using Bonsai.
It depends on the harp-serial branch of pyharp so we should probably clean that up and merge it before merging this one in, as it's pretty questionable to depend on a certain commit of a git repo...

It would be good to package the device yaml from the repo into the python package when installing via pip. I just copied it into the code for now to make it work, but we should do it the right way before calling this done.