First iteration of simulation restart option

Extends JLD2 output callback to save at user-defined time intervals (previous output was at simulation final time)
Extends simulation to allow users to define a restart file-path and new simulation end-time to pick-up simulation at some intermediate timestep for which model+integrator data (via jld2 files) is available. Requires model types (including domains) to be consistent across simulation model and restart model. (Currently does not include automated directory look-up/ scan for latest checkpoint files)

Just to clarify, does this only add the restart option, or does it also add the ability for saving at user-defined time intervals (as in the description)?

I left one comment, otherwise it looks good.

LGTM, thanks! I don't know why the codecov decreases - it seems everything is included in the tests.

I think I see why; a "fail" case i.e. in the code it is designed to throw an error if the model types between restart and sim don't match.

LGTM, thanks! I don't know why the codecov decreases - it seems everything is included in the tests.

I think I see why; a "fail" case i.e. in the code it is set up to throw an error if the model types between restart and sim don't match; I think that's the line with coverage issues

Oh, I think that's fine? Feel free to squash, rebase, and merge:)

bors try

bors r+


GUI failed when cpu count < global_settings['thread_num']

Traceback (most recent call last):

  File "streamlit\runtime\scriptrunner\script_runner.py", line 556, in _run_script

    exec(code, module.__dict__)

  File "C:\Users\marth\AppData\Local\Programs\peptdeep\peptdeep\webui\main_ui.py", line 55, in <module>

    sidebar[menu]()

  File "peptdeep\webui\settings_ui.py", line 31, in show

    thread_num = st.number_input('Thread number',

  File "streamlit\elements\number_input.py", line 113, in number_input

    return self._number_input(

  File "streamlit\elements\number_input.py", line 225, in _number_input

    raise StreamlitAPIException(

streamlit.errors.StreamlitAPIException: The default `value` of 8 must lie between the `min_value` of None and the `max_value` of 4, inclusively.

We have fixed this bug on github and on github release v0.2.0 (https://github.com/MannLabs/alphapeptdeep/releases/tag/0.2.0),
but peptdeep v0.2.0 still has this bug on PyPi. It will be fixed on PyPi with peptideep v0.2.1


connection to spark sql bug

Description
connection to spark sql don't show databases and data tables
DBeaver Version
23.0.5
Operating System
MAC
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Please provide log files. You can find instructions here https://dbeaver.com/docs/wiki/Log-files/

Please provide log files. You can find instructions here https://dbeaver.com/docs/wiki/Log-files/

It is exactly the same as this mistake, but it cannot be solved. #5314

Hello @boy-xiaozhang
We still need logs. Additionally, please provide your database and driver versions.

 hi bro the above is the jdbc driver I am using，also no error log

Not all errors from the log are visible in the Error View.
Please, provide us with the log files: https://dbeaver.com/docs/wiki/Log-files/#log-files

hello I did not find

@boy-xiaozhang it's a hidden folder. Just cd into ~/Library/DBeaverData/workspace6/.metadata and you will find log files.

thanks.accustomed to ll command, executed ls -la can operate, I found the log file

According to my judgment, the jar implementation method used may not be supported. Do we need to replace the jar package? If so, do we have any good suggestions? I think some students have used kyuubi to solve this problem, but we do not have kyuubi.

I'm confused. What needs to be fixed for you? You're referring to another ticket about tables not showing, but all I can see are network-related errors. Please provide more information regarding your issue.

I'm confused. What needs to be fixed for you? You're referring to another ticket about tables not showing, but all I can see are network-related errors. Please provide more information regarding your issue.

The real mistake I saw is that the driver is wrong and there is no corresponding implementation method.

@boy-xiaozhang can you reproduce it again and send a screenshot of this error?

@boy-xiaozhang could you clarify what driver is wrong for you and what is correct?

我使用的就是dbeaver下载的默认驱动，图一是驱动，图二是错误截图

@E1izabeth  hello   is there any good solution or should I try to replace the driver or the dbeaver version

spark version 2.4.0
hive-jdbc-driver version is 1.1.0
The old version of dbeaver can be used in a certain project before

The real mistake I saw is that the driver is wrong and there is no corresponding implementation method.

If you're claiming the issue is with the driver, it has nothing to do with DBeaver. This is a third-party issue.


Mermaid: Uncaught SyntaxError

I am not able to use Mermaid. I created a sample page and copied the content from the documentation page for testing purposes. I did not use the repository but the pre-built release bundle (v0.19.4). Furthermore I used your example configuration and rendered the page the following way:
$ hugo server -D
Start building sites …
hugo v0.89.1+extended darwin/amd64 BuildDate=unknown

                   | EN
-------------------+------
  Pages            | 188
  Paginator pages  |   0
  Non-page files   |   0
  Static files     | 105
  Processed images |   0
  Aliases          |   1
  Sitemaps         |   1
  Cleaned          |   0

Built in 409 ms
Watching for changes in /Users/dom/git/test/{archetypes,content,data,layouts,static,themes}
Watching for config changes in /Users/dom/git/test/config.toml
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at //localhost:1313/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

The page looks the following but the actual diagramm was not rendered.

I observed the following error in the console in Chromium:

Any suggestions?

I'm not able to reproduce it using the described way. So far no idea what could be wrong. The root issue seems to be that the mermaid source file (mermaid-beaf30d1e1.min.js) can't be loaded. Can you open this file in an editor and check the content? It should look like a minified JS file (no need to check every line, just check if there is something in there that looks suspicious). You can also try to re-download the theme release tarball, maybe parts were corrupted somehow.

You are right. I totally missed to check if the files really exist and indeed the correct Mermaid file is missing.

I did re-download the tarball and now it works. I do not know why it missed some files in the first place since I did exactly the same. Thank you for the quick response @xoxys!


Issue with queing work orders

During this issue;
I had just updated to the new simcraft version
I have 27 personal orders, across 9 recipes. blizzard has begun grouping these in the work order ui by recipe (various treatises, some known some unknown recipes)
there were no guild orders available
there were 9 patron orders available. of which 7 I had the recipe for (one of which was a known treatise)
Replication; set work order type to personal only
Expectation: Only Personal Work Orders are qued
reality; only patron orders are qued
replication; set work order type to both patron and personal
expectation: both patron and personal are qued
reality; only patron orders are qued, but twice
replication: set work order type to all 3
expectation: both patron and personal orders are qued (no guild orders cause there are none)
reality: only patron orders are qued
replication; set type to guild & personal
expectation: personal orders are qued
reality: no orders are qued
work order types that functioned as expected [setting (result)];
guild only (empty)
patron only (patron)
guild & patron (patron)

I have since gone through 2 of the treatises, causing the blizzard to go back to listing all orders individually. All behaviour has become as expected since then.


Port suclass theme

@caesar2164 You should update the PR to now point against master. You should also rebase against origin/master.

@caesar2164 Is there more coming? Is this all we need?

This is just the beginning... 😁

sad 😢


IsEmpty feature on Signature Point

Hi natie84,
sorry probably here is the wrong place, but i wanted to contact you for ask you if your acr-xamarin-forms handle, per each point, the IsEmpty property to check discontinuous lines
If your ansewers is negative, can you suggest me where i can implement this feature in your project (if you noticed i forked your project to update it to the latest Xamarin Forms).
Thank you for any help!
Lewix

Wrong project


ome-www deployment updates

updates to resize the LVM lvs and filesystems
also configured jekyll-build with a site name

Supporting config required: https://github.com/openmicroscopy/management_tools/pull/492

Deployed successfully with @kennethgillen. Overall :+1: for the changes and using non-default deployment locations. Merging as the state of the live deployment.


Move stanc.js to a web worker

This PR has a few contributions:

we now vendor stanc.js at a fixed version, rather than getting the nightly build. This is currently 2.35.0, which is also what version of Stan our current tinystan is using in the backend.
rather than loading in a script tag, stanc.js is loaded in a worker. This gives us some degree of sandboxing because the worker thread's globals are separate from the rest of our app.
the state logic around this worker and around the errors from stanc was extracted out into a useStanc hook
this hook is tested

It is probably still worth it to debounce calls to stanc (#12), but this PR technically does resolve the issue with the UI slowing down when editing a large model.
To be honest, preparing this PR took much longer than it was probably worth, mainly on the final bullet point. There was a lot of pain associated with the fact that vitest is not really a browser, but in the end it seems to be working.

Very cool.
This all looks great to me.
(I tested it using "review in codespaces" and it was really easy... just yarn install and yarn dev. I recommend to test all the PRs this way because it does not require checking out locally)

"review in codespaces" and it was really easy

Wow, that really does feel like magic. The fact that it actually redirects localhost:3000 to another address so you can actually open it is very cool.
It looks like each user gets 60 hours/month if you are just on a standard free github plan. Pretty nice!

I confirmed that this branch does not conflict with #40

It probably conflicts with #44, but I think it's fairly trivial to resolve no matter which one got merged first

This conflicts sort of heavily with #51
It's manageable but will require some focus to get it right. Could we merge this now and then I will work on resolving the conflicts?

I was hoping @jsoules would take a look - he did already see a lot of it while it was in progress, so maybe it’s not necessary before merging

Rebased on main and made some cosmetic renaming of the types and tests


Not having Email field in Signup page

Description:
When user want to Signup or want to create account then instead of username please change the field to email because email is must for future purpose if your signup backend is there then how you can verify your account?? by using username but you can verify by using email.
Current UI:

@SwanandD121 Please review this issue and assigned to me with the labels.

@harshbhar0629 assigned


guide for running in vcluster

Doesn't full work yet, the Helm Release is not transitioning into ready:
# kubectl describe release
[...]
Status:
  At Provider:
  Conditions:
    Last Transition Time:  2022-01-07T15:53:22Z
    Reason:                Creating
    Status:                False
    Type:                  Ready
    Last Transition Time:  2022-01-07T15:58:29Z
    Message:               create failed: failed to install release: timed out waiting for the condition
    Reason:                ReconcileError
    Status:                False
    Type:                  Synced
Events:
  Type     Reason                        Age    From                                Message
  ----     ------                        ----   ----                                -------
  Warning  CannotCreateExternalResource  2m47s  managed/release.helm.crossplane.io  failed to install release: timed out waiting for the condition
  Normal   UpdatedExternalResource       2m47s  managed/release.helm.crossplane.io  Successfully requested update of external resource

# helm ls -n redis1
NAME    NAMESPACE       REVISION        UPDATED STATUS  CHART   APP VERSION


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

have to review header box, followed code as specified on the prework guide and double check lines i cant get the shadow boxes to align on web view.


Feat:  Add Password Visibility toggle on Sign-Up Page

Description:
Adding a "Password Visibility" toggle on the sign-up page to allow users to view or hide their entered password. This feature will improve user experience by reducing errors when entering complex passwords and ensuring that users can verify their input before submission.
Objective:
Enhance usability by allowing users to toggle between hidden and visible password fields.
Reduce errors due to mistyped passwords during the sign-up process.
Expected Behavior:
Users can toggle between showing and hiding the password by clicking on an eye icon next to the password input field.
The password visibility toggle will work in real-time without refreshing the page.
By default, the password will be hidden, but users can click the icon to reveal or hide it as needed.

I would like to work on this issue kindly assign it to me ASAP.

I am interested in contributing to this project as  I came across the issue titled "Feat: Add Password Visibility Toggle on Sign-Up Page" and would love to work on it.
To implement this feature, I propose adding an eye icon to the password input field on the sign-up page, allowing users to toggle between showing and hiding their password. This enhancement will significantly improve user experience by making it easier for users to verify their password entries during registration.
Here's how I plan to proceed:
Add an eye icon button next to the password input field.
Implement JavaScript to toggle the input type between "password" and "text" based on user interaction with the eye icon.
Ensure the UI design remains consistent with the overall application style.
If this solution aligns with your expectations, I would be happy to start working on it. Please let me know if there are any additional requirements or suggestions you have for implementing this feature.

I am looking forward to your response and excited to contribute to this project.


[Firebase-trigger-email] Logs too verbose, logging template loads

The logs of this extension are too verbose. If I send one email, it loads and logs each email template I have. Logging which templates have been loaded, especially ones that are irrelevant to the email at hand, is not necessary.

This is good feedback. @dackers86 can we review how we log these steps?

This appears to occur on the Templates class constructor when the function is first initialised.
The reason it logs individually is because of a series of checks on each template.
    templates.forEach((t) => {
      const tgroup: TemplateGroup = {};
      if (t.subject) {
        tgroup.subject = subjHandlebars.compile(t.subject, { noEscape: true });
      }
      if (t.html) {
        tgroup.html = htmlHandlebars.compile(t.html);
      }
      if (t.text) {
        tgroup.text = textHandlebars.compile(t.text, { noEscape: true });
      }
      if (t.amp) {
        tgroup.amp = ampHandlebars.compile(t.amp);
      }
      if (t.attachments) {
        tgroup.attachments = attachmentsHandlebars.compile(
          JSON.stringify(t.attachments),
          { strict: true }
        );
      }

      this.templateMap[t.name] = tgroup;

     // Individual log occurs here!
      templateLoaded(t.name);
    });

Perhaps this log could be moved outside of the forEach to log after the checks have been completed.
This could also be  updated to map and provide a single log to show all valid templates in one comma separated list.


Remove quarkus.otel.traces.eusp.enabled=true from quarkus_3.7.x.patch

This doesn't seem to be needed for the tests to work.
Closes: #252

@Karm PTAL.


updated debian package build configuration

Note that this also changes the installation prefix from /usr/local to /usr
Survives lintian checks with no errors.

Thanks!


Feature request: use month macros with --output-legacy-date

I prefer jan, feb, etc. instead of {01}, {02}. Could biber --tool use these macros when outputting legacy dates?

Should now be implemented in 2.16 DEV. By default, standard month macros will be output with --output-legacy-dates in tool mode unless the option --nostdmacros is used to turn off standard BibTeX macros.

First testing suggest this is working. I'll confirm over the weekend.

Yes, I believe this is working in 2.16 dev. Thanks!


Retrieve function source description from comments

If a PostgreSQL function has an SQL comment, it will try to parse as JSON and use its values to override the auto-generated TileJSON.  It is recommended to use this form when creating comments to ensure valid JSON values.
DO $do$ BEGIN
    EXECUTE 'COMMENT ON FUNCTION YOUR_FUNCTION (ARG1_TYPE,ARG2_TYPE,..ARGN_TYPE) IS $tj$' || $$
    {
      "description": "description override",
      ...
    }
    $$::json || '$tj$';
END $do$;

Partially implements #822

Hi @nyurik  Could you give me some suggestions for fixing the errors in SQL files? I've tried many times.
It's weired. The setup-database step in CI failed.

Hi @nyurik Could you give me some suggestions for fixing the errors in SQL files? I've tried many times. It's weired. The setup-database step in CI failed.
Update: Seems it's due to the case sentive issue.But I think I shouldn't double quoting every sql file.Fix it and do the method you suggested later

usually you don't need to quote any identifiers (namespaces, tables, functions and their arguments, etc) in SQL, as long as they don't use any special characters or reserved keywords. BUT, in some very rare cases, someone may have those weird cases. So in our testing, we must make sure Martin supports it properly. And for that, I have a few tests like "MixedCase" namespaces, as well as functions, etc.
Note that the generated SQL automatically always uses quotes identifiers to avoid ambiguity.

I reworked your patch in https://github.com/sharkAndshark/martin/pull/50 -- please take a look and merge if agree.
Basic idea:

tilejson from a comment is stored as a simple json value
when creating the final tilejson, it uses generated tilejson as the "base", and applies all fields from the comment's tilejson as a "patch" (override) using json-patch crate

Agreed.I realized the tests part and I'm in a conference,merge it later.

To achieve isolation, it is recommended to have one unit test for each observable behavior. Therefore, it might be necessary to remove some comments from the function. However, there is a concern that this level of granularity might be excessive? @nyurik

@sharkAndshark lets keep things simple :)  I think we need to cover these cases. And most of these cases can even be done in one or two function comments.

overriding a single field:  just set the description in a function comment -- this should override the whatever description is set from the full ID automatically
adding a new field:  set vector_layers field with an array of whatever stuff the function is producing
deleting a field - in another function comment test, set description to null. No need to set any other fields at all
One of these tests should be set on "MixedCase"."function_Mixed_Name" - to test comments work on weird named functions. The other test should be on some basic one but with different params, like public.function_zxy_query

Just doing these two tests, with essentially just 3 lines total, you should be good enough.

I hope the doc is not too bad as English is not my first language and I'm still learning it.

Thanks @sharkAndshark , i may do another round of docs cleanup later, but this is good to merge for now.  Do you perhaps want to do the same thing for tables?  I suspect it is possible to add comments to them too.

Yeah sure. I'd love do that. A issue ticket first maybe?

technically that's the same issue #822 issue - although it is more of an umbrella issue to cover all tilejson modifications by the user.  Eventually I think we should just add an extra field to the yaml config, and that field will work the same way as the function comment


Where can I find the source code of version 2.1.2

Hi, where can I get the source code of version 2.1.2 ?
https://www.nuget.org/packages/Xam.Plugin.Connectivity/2.1.2
This repo only hosts the 2.2.12 release.

You would have to go through the commits. My branches are newer concept for me.
Something like: https://github.com/jamesmontemagno/ConnectivityPlugin/commit/5a033fa241e16be80690c03094a1ccecf7f73ebc


Addition of a build system generator

I suggest to reuse a higher level build system than your current small make file so that powerful checks for software features will become easier.

CMake
Autotools

This would probably be a good idea, I'll give it a look. (I've never worked with either system so it might take me a bit of time to figure out how they work, thank you).

Does a tool like "autoscan" show any software dependencies that you find worth for further considerations?
How do you think about to support more development environments by the reuse of build system generators?

I disagree actually. This is a really small project that is inherently tied to POSIX code (so cross-platform building isn't really an issue). I do not see any significant benefit in using a build system instead of just a Makefile for this (especially not something as giant and horrid as autotools).

This is a very small project and there's no benefit from CMake or Autotools. A simple Makefile is fine.

It's simply not needed for a small project like this. For larger codebases, it's (CMake or Autotools) a large and complex tool, with many features to facilitate large projects. However, in this context, it'd be unnecessary when a Makefile would be just fine.

I wouldn't go so far as to call either autotools or cmake such things. But I will say that, for a small project, any build system is adding a dramatically larger amount of complexity than is needed.
In addition, I would say that autotools and cmake both are incredibly complex (though some of the complexity is necessary; not even remotely all of it is), to the point where I never use either. I would be much more likely to use tup or waf than either cmake or autotools. But, again, for this small of a project, a single makefile handles it just fine.

Please don't. I just had the very nice experience of being able to compile this via (GNU) make here on FreeBSD on first try, so cross-platform seems no issue not only theoretically. Don't overthrow yourself with unneeded complexity.

OK, I've given this some thought, and the popular consensus seems to agree with me that a build system is likely not needed. Closing for now.


does it work atm?

You are trying to add an unbuilt extension to Chrome. You have to build it yourself or download it from the release section following the instructions in the manual

If we rely on
Archivio ~/Scrivania/anti-random.org-master/src
You download the sources via Code -> Download ZIP and not the ready-made releases as described in the manual

thank you man!


revert: changes for canary image for release-3.12 branch

revert: change image version to canary updating the image version to push canary images for the release 3.12 branch.

Moving to Draft to prevent accidental merging before 3.12.0 is released.

@mergifyio queue


Added album art, duration, and bi-directional offset thanks to Sonos reporting!

Made headway on some of your TODOs.

Changed the port
Added album art
Added track duration
Added track offset (implemented the tracks as "resumable"
Added callback to Overcast to update the offset, so now you can sync your play position! This uses reporting features from the Sonos service, see here: http://musicpartners.sonos.com/docs?q=node/488

Hehe you started it off really well! You're right about the port I don't think I should have changed it... I'll take another look at this sometime soon but I'm too tired atm! My day job is iOS development, python is a little weird to me. One issue I will fix ASAP is the "File not found" error... it's the only thing really stopping me from fully enjoying Overcast on Sonos. I must say though, apart from that, it's great being able to pause and resume with the play position and duration... and the album art is a nice touch. Let's see where this goes :)

I addressed your comments and enabled redirects 👍

🎉  thanks again!


chore(main): release 1.1.5

:robot: I have created a release beep boop
1.1.5 (2023-08-10)
Build System

update Dockerfile to use HATH_VERSION argument (eefbc81)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cloverdefa/hath-docker/releases/tag/v1.1.5 :sunflower:


empty DNSPrefix error should exit immediately

An empty DNS prefix value seems to generate additional, unnecessary errors.

Is this an ISSUE or FEATURE REQUEST? (choose one): ISSUE

What version of acs-engine?: v0.6.0

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm) Kubernetes
What happened:
NFO[0000] Error returned by LoadContainerService: missing Properties.MasterProfile.DNSPrefix. Attempting to load container service using LoadContainerServiceForAgentPoolOnlyCluster
FATA[0000] error parsing the api model: Namespace Properties.DNSPrefix is not caught, Key: 'Properties.DNSPrefix' Error:Field validation for 'DNSPrefix' failed on the 'required' tag
Key: 'Properties.ServicePrincipalProfile.ClientID' Error:Field validation for 'ClientID' failed on the 'required' tag
Key: 'Properties.ServicePrincipalProfile.Secret' Error:Field validation for 'Secret' failed on the 'required' tag
Key: 'Properties.CertificateProfile' Error:Field validation for 'CertificateProfile' failed on the 'required' tag

What you expected to happen: I expect just the DNSPrefix error.
How to reproduce it (as minimally and precisely as possible): Generate a template using an api model without a dNSPrefix value

I think #1453 fixes this too.

Can this be closed then since #1453 was merged?

Yes, thanks for the nudge @SaaldjorMike. Please re-open if you can confirm that master (post merge of #1453) still exhibits this bug.


Add new monomers to the library

[x] Maleimide
Location in the library: CHEM
Name in the library: Maleimide
Abbreviation in the library: Mal
Attachment points/leaving groups: R1=H, R2=H
Structure: Maleimide.zip

[x] Tetraxetan
Location in the library: CHEM
Name in the library: Tetraxetan
Abbreviation in the library: DOTA
Attachment points/leaving groups: R1=OH, R2=OH, R3=OH, R4=OH
Structure: Tetraxetan.zip

[x] Tetraethylene glycol
Location in the library: CHEM
Name in the library: Tetraethylene glycol
Abbreviation in the library: PEG-4
Attachment points/leaving groups: R1=H, R2=OH
Structure: Tetraethylene glycol.zip

[x] Hexaethylene glycol
Location in the library: CHEM
Name in the library: Hexaethylene glycol
Abbreviation in the library: PEG-6
Attachment points/leaving groups: R1=H, R2=OH
Structure: Hexaethylene glycol.zip

[x] 4-formylbenzamide
Location in the library: CHEM
Name in the library: 4-Formylbenzamide
Abbreviation in the library: 4FB
Attachment points/leaving groups: R1=NH2, R2=O
Structure: 4-formylbenzamide.zip

Verified.

Versions

Ketcher Version 2.22.0-rc.2 Build at 2024-06-28; 05:30:15
Indigo Toolkit Version 1.22.0-dev.2.1-gd5637f75-wasm32-wasm-clang-12.0.0
Windows 10 Pro
Chrome Version 126.0.6478.127 (Official Build) (64-bit)


ERC20 and Cosmos coin conversion

Research which is the best and secure way to convert ERC20 tokens <-> Cosmos coin.
Food for thoughts:

Evmos docs are a good starting point to explore the problem.
Serdar comments: One of them is uint256 and the other is uint64 (or 32?), Here, we can likely use exponent metadata and treat the Erc20 as though it had the same units as ATOM. Shouldn't be too difficult but we need concrete research and a proposal

Coin Metadata ADR

Reasoning a bit I think it would make a lot of sense if by design we decide to support ERC20 decimals in the range of [6,18].

It would allow us to easily support cosmos coin -> Erc20 conversion without the need to consider the potential loss of precision in the go-side.
6 decimals sounds like a really reasonable minimum number of decimals for Erc20 tokens.

Noted that the max number of supported decimals for the uint256 would be 77.  Adding support for this because is trivial.


frugal: init at 3.14.2

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

My bad, it's been added :)

3.14.3 is now out, but I'm going to merge this to give you a head start. Welcome!


👨🎓 Fix a typo in Terraform test

This pull request:

Fixes a typo in policies, thanks @dms1981

Signed-off-by: Jacob Woffenden<EMAIL_ADDRESS>

Signed-off-by: Jacob Woffenden<EMAIL_ADDRESS>


root: [build] update list of tested branches in CI

With the new release, we adopted a new branch convention. This PR updates the corresponding list of branches in github CI

@brianfu could you take a look?

I confirm those are the branches that we should be using, yes.


Using typescript-language-features extension and plugins

(I'm not sure if this is bug or not)
I'm building an extension for a domain specific language that uses a subset of TypeScript. It uses a custom file extension, let's say it is *.sts (SmallTypeScript). I would like to reuse all IntelliSense features that VSCode supplies for TypeScript.
I've made some progress doing following:

VSCode extension manifest defines a new language.
Same VSCode extension manifest defines TypeScript server plugin with a languages field that points to my language.

I'm able to override compilation settings (I provide my own built-in functions) and get hover support for them:

The problems are following:

Although I specify jsx property in my compiler options override, TS server does not seem to pick it up (see Problems on screenshot above).
There are no suggestions/completions. If on line 6 I try typing data., I get no completion items.

Both problems are solved in the hello.tsx file I have in the same directory, it has the same content.
I've created an example repository to reproduce the problem: https://github.com/agentcooper/vscode-sts.

@mattbierner maybe you will be able to help with this, since you've worked on 4f398bafda1fde8e17ec4646371272baad6d3914?

@RyanCavanaugh since this is related to tsserver as well, any suggestions?

Update: I solved JSX by overriding getScriptKind and solved completions by providing typescript-plugins.suggest.enabled since that is used by typescript-language-features.

The additional language support for js/ts (4f398ba) was originally added for angular. However the push behind this effort sort of died off and so it seems unlikely it will ever move past prototype stage.
I think you're better off proxying everything through VS Code apis. See the built-in commands for example of this


auto load babel for non-bleeding-edge browsers

currently 'fetch' and async functions are used which are unsupported in Safari

fixed via https://github.com/lg/cloudy-gamer/commit/ba24b1c8ddd191b33a36ec87d974498de148d8d9


[BACKLOG-5269] Cannot Execute or Open Referenced Object (di-repo)

@wseyler I tested filesystem/filerep/dbrep/direpo


Add Travis badge to README

For master branch. Alternatively could change to devel.

I think the master is more meaningful for the users. Developers can always investigate deeper using Travis.

Just a comment -- please do not push any branch into the main repo.
I want to keep it only with master and devel ;)

Yes, that is clear. This one was necessary to trigger the CI correctly to see if the matrix works. Which is why I closed #634 which was a PR from a fork.


Version 1.4.0

@conda-forge-admin please rerender

Needs #3 first before proceeding further

I also tried a build forcing the definition of ITKV4_COMPATIBILITY here just in case, which gives a different set of issues.


Process

Read max pid number from /proc/sys/kernel/pid_max
List all active pid
Read process io
Read process statm
Read process stat
Read process status

In my opinion, we should split process.go into several files : its kinda huge now (826 LOC) :

process.go
process-io.go
process-stat.go
process-statm.go
process-status.go
process-pid.go

What do you think ?

I would prefer underscore rather than dash. :-)

Done in c48d2ef


Not able to be installed on Ganymede & Galileo

Not able to be installed on Ganymede & Galileo for latest code: 1.0.0-rc.12

This is regression issue.

Drop support for these two packages as Java upgradation.


CRACK-STREAMS]]```UFC 305 Live Streams Free@𝚁eddiT in Home

UFC 305 Live Streams Free@𝚁eddiT in Home
What's the bestway to watch ufc 305 (fRee) live Streams on ReDdit?, UFC 305 Main Event: Dricus Du Plessis +105 vs Israel Adesanya -125. I was of the opinion that his KO of Alex Pereira was the peak of his career, and that it would be a quick downhill slide for him after a long run of amazing fights, and he clearly didn’t look like his prime self against Strickland, and I don’t think he has much gas left in the tank.
🔴 GO LIVE==►► CLICK HERE TO WATCH UFC 305 LIVE
🔴 STREAMING==►► CLICK HERE TO WATCH UFC 305 LIVE
He’s beaten all the big names over the years, headlined countless PPV’s, and at some point there’s just not much energy left, and it’s understandable.
If he was in his prime I would pick him as his striking, patience and gas tank would all be better than DDP’s, but I don’t think that’s the Izzy we get here.
DDP is an awkward fighter with weird striking at times, and his gas tank has been an issue in the past, but since his nasal surgery it appears to have gotten better.
I don’t have much of a breakdown on fighting styles, I just think the wear and tear of all those wars, and the energy he needed to make it through all those fights has caught up, and I wouldn’t be surprised if we don’t see him fight to many more fights. DDP is the play for me.
As the boy had made a full and frank admission and this was his first offence, it was agreed that a community resolution wasthe most appropriate and proportionate outcome.
Academy graduate Anthony Elanga is emerging as a bright talent, having scored two goals in United's last three fixtures as the 19-year-old found the net against Atletico Madrid and Leeds.
According to Marco Donzelli, president of CODACONS, the Italian association for the protection of consumer rights,Juventuscould, 15 years after Calciopoli, be once again relegated and stripped of Serie A titles.
The 29-year-old striker showed there was subtlety to his game in giving Maguire the slip early in the second half before having the awareness to slide Jay Rodriguez in behind the back line for Burnley's equaliser.
Uefa said all eight clubs were sanctioned due to their non-compliance with the 'no overdue payables'
Stendel was giving evidence at the trial of former England international Barton who is accused of assaulting the German manager at the end of a match between Barnsley and Fleetwood Town, at the South Yorkshire team's Oakwell Stadium, on April 13, 2019.
The hosts had gone into the game unbeaten in three matches under interim manager Paul Ince. However, they had not won on the road in seven outings since mid-January.
Forward Dave Bennett, who scored Coventry's opening goal at Wembley, said Sillett created a special bond among the squad of players, who are still friends to this
UFC 305: du Plessis vs. Adesanya Live Stream Free The UFC 305: du Plessis vs. Adesanya Match will air on ESPN Live today at 1 p.m. ET. Ready to tune into the UFC 305: du Plessis vs. Adesanya showdown? Here's
Canada's Christine Sinclair was presented with a special award, recognising her as the highest scorer in either men's or women's international football - the 38-year-old has 188 goals in 308 games for her
Ahmad Bin Ali Stadium Its glowing facade is comprised of patterns that characterise different aspects of the
“He stayed true to the players who got him there, which was important. We didn’t lose that bond we had as a team. We had a good core of players who had that work ethic, and it was an example to everyone who came in,” he
We scored a fluke against Chelsea from the same area but apart from that, I thought the young boys did really well.

The Match you are looking for is STREAMING NOW. Follow the given link you will find what you are looking for...
📺📱🔴 LIVE STREAMING CLICK HERE TO WATCH
📺📱🔴 LIVE STREAMING CLICK HERE TO WATCH

➭
📺📱🔴 LIVE STREAMING CLICK HERE TO WATCH
📺📱🔴 LIVE STREAMING CLICK HERE TO WATCH
[


Support for Device Code Flow Authentication

As suggested at this link, I'm creating an issue for the possible implementation of the authentication feature via Device Code Flow. I noticed that this flow is not exclusive to Azure, but it comes from OAuth 2.0. In it, we have two tokens as a response of the process: a Bearer token and a refresh token (to reauthenticate when the Bearer token expires). However, I think it doesn't make sense to implement reauthentication within DuckDB, I believe it makes more sense being a user's responsability. For more information on how this flow works in Azure, please refer to this link:
https://learn.microsoft.com/en-us/entra/identity-platform/v2-oauth2-device-code

Hello,
I have take a deeper look into this at the moment I have two issues:

The azure SDK en C++ is not at the same level than the Java one so there is no DeviceCodeCredential
We have to find a way to get the code to the end user

I will try to take a look.
Regards,
Quentin

@quentingodeau I have been thinking about the cli authentication. How to do it on DuckDB? It uses some kind of access token? If so, maybe this will solve our problem.

Yes the CLI should work if you are using DuckDB in local.
Just perform

az login

Start duckduckdb

Define the flowing secretCREATE SECRET az_cli(
  TYPE azure,
  PROVIDER CREDENTIAL_CHAIN,
  CHAIN 'cli'
);

And finally perform your queries :)SELECT *
FROM 'abfss://adlsducktests.dfs.core.windows.net/qgo/tmp.parquet';

I tried to do this, but it keeps giving auth errors. I dumped my duckdb_secrets table, but it appears that the secret is empty. Is that correct? Or I should give more parameters to the CREATE SECRET command?
secrets.csv
The command I tried to execute and error I got are the following:
SELECT *   FROM<EMAIL_ADDRESS>IO Error: AzureBlobStorageFileSystem could not open file<EMAIL_ADDRESS>unknown error occurred, this could mean the credentials used were wrong. Original error message: 'Fail to get a new connection for<EMAIL_ADDRESS>Problem  with the SSL CA cert (path? access rights?)'

I tried to do this, but it keeps giving auth errors. I dumped my duckdb_secrets table, but it appears that the secret is empty. Is that correct? Or I should give more parameters to the CREATE SECRET command?
secrets.csv
The command I tried to execute was this:
SELECT *   FROM<EMAIL_ADDRESS>And I got this error:
IO Error: AzureBlobStorageFileSystem could not open file<EMAIL_ADDRESS>unknown error occurred, this could mean the credentials used were wrong. Original error message: 'Fail to get a new connection for<EMAIL_ADDRESS>Problem  with the SSL CA cert (path? access rights?)'
I have also tried this:
SELECT count(*) FROM 'abfss://⟨my_storage_account⟩.dfs.core.windows.net/⟨my_filesystem⟩/⟨path⟩/⟨my_file⟩.⟨parquet_or_csv⟩';
And I got this error:
Invalid Error: Fail to get a new connection for: https://<storage_account>.dfs.core.windows.net. Problem with the SSL CA cert (path? access rights?)

You are under Linux isn't it ?

For the error:
Invalid Error: Fail to get a new connection for: https://<storage_account>.dfs.core.windows.net. Problem with the SSL CA cert (path? access rights?)

Try to set explicitly the configuration (if not yet done):
set azure_transport_option_type = 'curl';

cf:

For the secret part:
the secret look good if you plan to used fully qualified names otherwise you may want to specified the ACCOUNT_NAME 'sa name' in the secret definition

Regards,
Quentin

Finally, it works! Thanks for all the help!


[BUG] Filter not working in product

Description
@pooranjoyb
check this out,
filter is not working after clicking on Apply button
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome

Congratulations, @dhruv8433! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@pooranjoyb
please assign me i would like to work on this

Yes fix this bug and also add routes for other pages in Hamburger Menu :) @dhruv8433

Ok Thank you @pooranjoyb


Provide a non-technical overview of Vale and the Red Hat Style

Is your feature request related to a problem? Describe.
A clear and concise description of what the problem is.
Describe the solution you'd propose
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 here.

https://redhat-documentation.github.io/vale-at-red-hat/docs/user-guide/introduction/


Sdg v0.6.0+ multiple knowledge sources fails to clone

I think I found a potential race condition specifically here (context aware chunking): https://github.com/instructlab/sdg/pull/284
Basically if there is more than 1 knowledge document for git to clone, and it happens to do multiple clones with the same second it will generate the same output dir:
document_output_dir = Path(output_dir) / f"documents-{date_suffix}"
Which causes SDG to fail since the directory already exists on the git clone.
Generating data on a single knowledge document, things works just fine. Its when we get to multiple I am seeing failures.

Thanks Kodie! At first glance it looks like you're right, and we just didn't actually hit this in any of our testing. I'm not sure that the current knowledge document clone directory has any particular semantic meaning that required it to use a timestamp with only second resolution. We should consider having this either be an entirely unique directory, or if we want the directory structure to have semantic meaning base it on something like the taxonomy leaf node path plus a timestamp. Looping in @khaledsulayman and @aakankshaduggal here for awareness or differing opinions on how we should address this.

It turns out we were always failing when 2+ knowledge taxonomy leaf nodes were in use. Working on a fix now.

The linked PR #416 fixes this. In the interim, I believe the only viable workaround is to ensure you only generate data for a single taxonomy leaf node at a time. This may require creative use of the --taxonomy-base parameter to ilab data generate and/or working with taxonomies that only have 1 knowledge leaf node. It's generally not easy to only specify a subset of your taxonomy leaf nodes per generate call without copying files into a new taxonomy tree containing only a subset of your total taxonomy or being very careful in how you layer git commits in the taxonomy repo, ensuring only a single taxonomy leaf node changes per commit.

This has been released in v0.6.1 - thanks for the report! @KodieGlosserIBM let us know if you run into any more issues around multiple knowledge doc cloning. Every git clone we do for knowledge documents is now guaranteed to be in a unique destination directory, even if we've already cloned this knowledge doc, are cloning knowledge doc repos for multiple leaf nodes in short succession, and so on.

Ahhh great find @bbrowning !! Thanks for the quick fix.


TabContainer now requires extra tab key press to advance between tabs

This is a regression, only visible in the current dev version.  Somehow, there is an additional bit in there grabbing the focus

This has been fixed in this commit, didn't think it warranted a pull request.: https://github.com/xh/hoist-react/commit/daa2ee4497a34d2d192bc446cd9b3e3407cbef50


Document Auth methods from #137

Changes proposed in this pull request:

Add docs for #137

Attention: @prkumar

Looks great, @cognifloyd!


Configure main GPU in rootston config file

Due to the way we handle WLR_DRM_DEVICES, it's not quite as simple as setting the "main" one.
You must specify all of the GPUs going to be used, and the first one you specify is the main one.


MM-22800 show keyboard shortcuts on tooltips

Summary
Show Keyboard Shortcuts on Tooltips #15024
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15024
Related Pull Requests

Has server changes (please link here)
Has redux changes (please link here)
Has mobile changes (please link here)

Screenshots

edited tooltip

Todo's

[x] switch to a specific team tooltip shortcut
[x] Direct message shortcut tooltip shortcut
[x] History arrows (Desktop app only)
- Back in history
- Forward in history
[x] Recent Mentions shortcut tooltip shortcut
[x] Upload file - show tooltip on hover of the paperclip icon in the message input

I have done only one tooltip if this approach is correct. I will correct the remaining tools as similar to them. If there is a need to extract the logic out the file in order to reduce the repetition of code, I can do it as well.

cc// @calebroseland to check on the dev implementation before we do the rest of the tooltips. Perhaps a component would be best to eliminate code duplication?

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich

Hi @sridhar02! Do you need any further assistance or help to implement the suggestions from @matthewbirtch and @calebroseland?

Removing the test server for maintenance purposes. Please add the label again if you need it. Sorry for any inconvenience.

@sridhar02 need any more help on this?

@esethna I will look into it,I having other work was not able work on it. I will complete this one in a week or two.

@sridhar02 how's this going?

@esethna  Working on it going to update in this weekend and It will be ready for the dev review.

@calebroseland I refactored the code as you addressed above but I have few things to clarify such as shortcuts_modal.js should I have to keep it as js file or convert to TS, I still need to refine styles of the tooltip.ca you suggest any other changes for the code?

Thanks @sridhar02 for the updates! It looks like the tests/build failed for the latest changes, would you be open to help take a look?

@jasonblais I have resolved the errors, you can review it.

Thank you! @esethna @matthewbirtch @calebroseland ready for your reviews

@calebroseland Need some help in the following issues:

I have done the implementation for the recent mentions and direct mentions tooltips with a new look.
Need some help for the implementation of the tooltip for the upload file?
I have implemented the sidebar team tooltip with a new look, but have some issue regarding the how to pass the order?
how I can implement History arrow tooltip from a web app?

Hey @sridhar02,

Need some help for the implementation of the tooltip for the upload file?

Looks like you'll just need to add a new <OverlayTrigger/> + <Tooltip/> around <FileUpload/> with a message and <ShortcutSequence/> (like you've done for Recent Mentions) in:

create_post
create_comment

how I can implement History arrow tooltip from a web app?

Same for <ChannelNavigator/> here

@matthewbirtch I have changed styles of the all the tooltips which I have been working on, can you review them now and let me know if there are any changes need.

Recent Mentions

Upload Mentions:

Direct Message

Team view tooltip

I am not sure how to trigger the navigation tooltips but these styles will apply to them as well.

@calebroseland I have completed most of the work but having the following issues:

Sidebar team Tooltip issue?
I have removed the shortcut_key from the themes of utils in order to get the styles as required because they are overriding my styles.
Have a few type issues, can you help me how to rectify them?
I think I need to write some of the i18en strings for the browser navigation formatted message as I am using the shortcut from the shortcuts and name as different ones?
I am not sure whether all my instances are using the same formatted message from the shortcuts as that messages are different from the message in the shortcuts modal.do you think any better way to implement these modal messages in order to maintain uniformity for all the tooltips?

@matthewbirtch I have been doing some code changes need to fix everything, then I will get back to you.

@matthewbirtch I have been doing some code changes need to fix everything, then I will get back to you.

@sridhar02 tag me when this is ready for review again

@calebroseland I tried a couple of ways to remove type errors but not able to remove them but all the shortcuts tooltips are working now, need some help to remove the type errors?

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@calebroseland would you be open to help @sridhar02 with the type errors? Thanks!

@jasonblais Yes, he DM'ed me and it's on my list. (Sorry for the delay, @sridhar02! Should get to this by tomorrow.)

Hey @sridhar02, I went through the typings and made a some changes/refactoring to get things a bit further along. Be sure to pull/sync as I pushed it directly to your branch. (I thought this would be faster than sending you a patch or a bunch of suggestions!)
I think there's still the 'DM +'/More button to do:

Write a Direct message - '+' plus icon in the LHS next to the Direct Message category header

Other than that, I think that should be about everything.
Give it all a look over and see if there's anything else outstanding.

(Everything's linting/typing correctly on this locally, not sure why CI is complaining, will work on that...)

@matthewbirtch should the shortcut for the 10th team be added to its tooltip? If I press CMD+OPT+0, it'll switch to it.

@matthewbirtch should the shortcut for the 10th team be added to its tooltip? If I press CMD+OPT+0, it'll switch to it.

If it works, I don't see why not!

/update-branch

The file .circleci/config.yml is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

@matthewbirtch done, and (..8..9..) '10' changed to '0' to match up

@calebroseland Thanks for the help, I have gone through the code, I think I made you work a lot on my issue.

Should all be wrapped up with that now, sorry about all the noise here.
I think the only thing left is:

Write a Direct message - '+' plus icon in the LHS next to the Direct Message category header

(reason for change request)
Thanks for all your work on this, @sridhar02!

Regarding the work left do you mean by the message of tooltip in Direct message or any other?

@matthewbirtch can you review from your end about the Tooltips and let me know if we need any changes?

@sridhar02 I think some of it looks like a lot because I moved a number of files into folders, many of the diffs are misleading in that regard. Also, we most certainly underestimated the complexity on this—apologies for that!

Regarding the work left do you ... any other

Ahh! I didn't see that DM tooltip (forgot to check legacy sidebar).
I was referring to the new sidebar (enabled in System Console > Features and/or Account Settings > Sidebar), I didn't see it there.

@calebroseland I have updated the tooltip for the direct message of the new sidebar layout. can you review this pr?

Thank you @sridhar02
Tested following tooltips and all look great (macOS, Windows and Ubuntu)
Switch to a specific team
History arrows on Desktop
Write a Direct message
Recent Mentions @ header icon
Upload file postbox paperclip icon
Seeing one issue with the paperclip tooltip being triggered without mouse pointer hover, see gif
Steps:

Place focus in the post box
Type ctrl+U or cmd+U to upload a file
Cancel file upload
Expected: No tooltip
Actual: Tooltip for Upload File icon is triggered without having to hover over it

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

hey @sridhar02, and thanks for your contribution! Let us know if you have any questions about Jelena's feedback above? We'd be happy to help!

@calebroseland I busy with some work so i have not worked on this PR for the past few weeks, I think the issue that was mentioned by QA can be mainly due to this extra div if there is no div on that line it is not working due to react ref issue with the overlay trigger component can you suggest me any alternative to fix this issue?
https://github.com/mattermost/mattermost-webapp/blob/c6c3b501e2cc34398885b324a7049b7b12f4f7e0/components/create_post/__snapshots__/create_post.test.jsx.snap#L297

@sridhar02 let us know if you're still working on this? If not, perhaps @asaadmahmood could help push it across the finish line?

@asaadmahmood are you open to helping get this PR across the finish line?

@esethna Sure, I can have a look at this. Looks like there are a lot of conflicts, might have to address those first.

Closing this in favor of: https://github.com/mattermost/mattermost-webapp/pull/8161


Small Improvements to Segment.com Integration

Changes
Segment.com Code

Update to Segment 4.1.0 initializer code.
Use floats instead of strings for order/shipping totals and product pricing.
Add analytics.page for Segment destination that use page events.
Include domain in full image location path.

Spree UI Bug Fix

Moved the admin link to analytics into the configurations tab: FIX Analytics Link Issue #11

Minor Code Updates

Originals added to deface overrides: FIX Add original SHA1 digest to deface overrides to suppress warnings #13.
Remove  version '~> 0.3.18' from 'mysql2' in gem spec to play nice with Travis CI.

Tests Updated For Applicable Spree Versions.
Added tests for Spree 3.5 and 3.6, and removed older ruby versions and tests for older versions of Spree. (Am I correct in thinking that this extension intended for Spree 3.5 upwards?).

Thanks @MatthewKennedy - this looks great!


shadowAppendedHost is too restrictive, breaking valid IPv6 host headers

Description:
shadowAppendedHost expects at most 1 colon in Host. It has a debug assertion if this is not correct.
per https://www.rfc-editor.org/rfc/rfc3986#section-3.2.2, IPv6 addresses are legal host headers, and all clients will set them:
$ curl [::1]:8080 -v
> Host: [::1]:8080

This triggers a debug assertion.
The assertion should be removed at minimum, and possible the function improved to handle these other use cases (though, tbh, not sure -shadow appending makes sense on an IP)

Thanks for reporting this!
I assume this is the failing assertion.
cc @paul-r-gall and @alyssawilk

Oops forgot the link, yep that is the one

Looks like this predates @paul-r-gall and he's out so I'l pick up.
as this is a longstanding bug it's unlikely to merit backports though


⚡ Improve transaction operations

Abstracts AtomicalsCoinMixin which provides height information.
Merges transaction detail handlers in sessions into a single one and puts it under the SessionManager.
Adds payment-subrealm-failed and payment-dmitem-failed operation value.
Uses stored op value instead of massive conditions.

This also addresses https://github.com/atomicals/atomicals-electrumx/pull/180#issuecomment-2104037078.


Non Uniform Grid Global Router

Supporting non-uniform grid global routing in this branch

@soneryaldiz to provide a testcase as a next step. Not an immediate priority


Jesse: Blacklist +27717785486

Jesse requests the blacklist of the number +27717785486. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


Institute a changelog?

I would find it helpful to have a list of changes as they affect me as a user of the pipeline output. An example is that in the newest runs, output is stored in "output/" rather than "output/results", and "hv_hits_putative_filtered.tsv.gz" is now gzipped. One way to do this is to start updating a NEWS.md file to record user-affecting changes (possibly it would make sense to start semantic versioning, but dates and/or short-hashes could do).

Another option would be to use code reviews.  Will could make changes as PRs, and send them to you for review.  This would have the additional benefit of you understanding more how the pipeline worked.

Code reviews could separately be a good idea, but I still think having a plain-english summary of user-facing changes will be invaluable for anyone who wants to work with the results of the pipeline, and it will be easier for the person making the changes to at least draft these.

I agree a changelog would be good. I don't think code reviews would substitute for this.

Implemented here.


Use a newer, updated source for Russian

The current source for Russian dictionaries was updated long time ago and seems to be unmaintained anymore.
Arch Linux AUR ships a newer, updated version from the Libre Office extension.
It would be great to use those dictionaries here too.

It would be nice if you could get the communities to contribute somewhere that uses Git. Libre Office is a tough place to crawl. Something like github/gitlab/etc is easier. Do you think you could ask folks?

This seems to be the source for that extension:
https://cgit.freedesktop.org/libreoffice/dictionaries/tree/ru_RU

I also find https://forumooo.ru/index.php?topic=106.135. Still, it would be rather nice if the community could maintain a dictionary in github/gitlab/etc. Do you think you could ask folks?

Will this do?
https://github.com/LibreOffice/dictionaries/tree/master/ru_RU


feat(heade): add header component

it doesn't look like it's finished.

when the sidebar is revenue, the on-page error is generated here

After:

i believe that so far, the style and basic groundwork have been completed. Next, we need to incorporate some of the business functions.
cc @CaileanSong

i think the branch is ready for review.
@CaileanSong what do you think?


V3 : Breaking change - Big Upgrade

After seen huge traffic those past months, i decided lately to get back to maintenance and upgrading the most loved project .
this commit will cover upgrading the project from older version to laravel 11 and livewire 3 .
as well as removing and upgrading methods, making less and better code.
im preparing my library of components mainly on livewire volt , i will be playing and making from what ive coded until now.
feel free to discuss the ways we could make this project adapt to every management architecture mainly Business.
Thank you for testing and giving your feedback

Hi Zakaria, cool idea
However the project should require pestphp/pest in order to run tests.

I have noticed some bugs and i have the ability to fix them, should i create a new PR from this branch or?

I have noticed some bugs and i have the ability to fix them, should i create a new PR from this branch or?

Yes sure , im still working on upgrading the project in several aspects , but ur help will be greaat . Thanx.
Copy v3 upgrade branch work on it open a pr and let me review it

Hi Zakaria, cool idea
However the project should require pestphp/pest in order to run tests.

Test was installed but all test built were not yet fix based on changes made lately.


[ENHANCEMENT] Filter Out VM Backups That Don't Have Corresponding Image On Harvester Cluster

Is your enhancement related to a problem? Please describe.
Just a general enhancement for the UI.
In the scenario of:

user has single S3 bucket (minio, s3, s3-compatible solution) for Harvester VM backups
user uses that bucket, region, access key, access secret key for:

Harvester Cluster A (v1.1.1) - 4 node bare metal
Harvester Cluster B (v1.1.2-latest) - 3 node bare metal

if both Harvester clusters are using the bucket, to put VM backups in, both Harvester clusters, will see "all" VM's available to backup, when some of those VM's that are able to be backed up, actually only apply to either Harvester Cluster A or Harvester Cluster B, but not both.

Describe the solution you'd like
A checkbox underneath the filter in the UI to allow to filter by just the current Harvester cluster.
Maybe the comparison/filtering logic would contain a check to the pvc then to the image id associated with the volume backup - after a check to look at all available images on the cluster - so that no backup would be shown that didn't have a corresponding image in the cluster "if" the filter box was checked.
Something sorta similar to, "if" the filter box was checked:
import json
from requests import request

def main():
    url = "https://<IP_ADDRESS>/apis/harvesterhci.io/v1beta1/virtualmachineimages"
    payload={}
    headers = {
    'Accept': 'application/json',
    'Authorization': 'Bearer token-lfddt:9nml7vnvf68m7g8rl52c7r8h5xtrrjcxdn8h7xct5lhqtr7rsbt7g9'
    }
    response = request("GET", url, headers=headers, data=payload, verify=False)
    body = json.loads(response.text)
    list_of_image_names = []
    for item in body["items"]:
        list_of_image_names.append(item["metadata"]["name"])
    print(list_of_image_names)
    url = "https://<IP_ADDRESS>/v1/harvester/harvesterhci.io.virtualmachinebackups"
    payload={}
    headers = {
    'Accept': 'application/json',
    'Authorization': 'Bearer token-rx2h6:d6r4p4w78q7ghklhltzl96x6x7jv2dnxjggzh2xf6bfl6kww9fkxpl'
    }
    response = request("GET", url, headers=headers, data=payload, verify=False)
    body = json.loads(response.text)
    list_of_volume_backups_to_display = []
    for item in body["data"]:
        print(item["id"])
        volume_backup_list_for_item = item["status"]["volumeBackups"]
        for volume_backup in volume_backup_list_for_item:
            if volume_backup["persistentVolumeClaim"]["metadata"]["annotations"].get("harvesterhci.io/imageId", "").split("/")[1] in list_of_image_names:
                list_of_volume_backups_to_display.append(item)
    print(list_of_volume_backups_to_display)
    return list_of_volume_backups_to_display

if __name__ == "__main__":
    main()

Additional context

loop @FrankYang0529 to further investigate this, IIUC this issue should get addressed after https://github.com/harvester/harvester/issues/2237. cc @bk201

Yeah, I think https://github.com/harvester/harvester/issues/2237 is more about supporting backing up images together. It would be nice to distinguish if a backup can be imported first.


[Experimental-Do not merge] Initial direct Azure Blob Storage support using Azure's C++ lite client

Lots of things left to do -

The reads/writes are synchronous, but can very easily be made asynchronous.
As of now, only append blobs are created, but we should be able to create page blobs for faster read access.
Add support for azure data lake gen2.
Add error checking and generally make the azure blob storage support robust.
Using directory marker files for now, we should be able to do away with this, but need to port code and tests that rely on directories getting created.

Even with this initial support, the examples on travis run with these times
real	7m18.851s
user	0m27.072s
sys	0m1.272s

Before, the same tests in the develop branch run with the following times
real	18m33.966s
user	3m33.316s
sys	0m8.136s

Looks promising!

Codecov Report

Merging #48 into develop will decrease coverage by 0.02%.
The diff coverage is 79.11%.

@@             Coverage Diff             @@
##           develop      #48      +/-   ##
===========================================
- Coverage    58.89%   58.87%   -0.03%
===========================================
  Files           42       44       +2
  Lines        13332    13551     +219
===========================================
+ Hits          7852     7978     +126
- Misses        5480     5573      +93

Impacted Files
Coverage Δ

core/include/misc/url.h
100% <ø> (ø)
:arrow_up:

core/include/storage_manager/storage_azure_blob.h
100% <100%> (ø)

core/src/misc/url.cc
100% <100%> (ø)
:arrow_up:

core/src/storage_manager/storage_manager_config.cc
71.92% <50%> (-3.55%)
:arrow_down:

core/src/storage_manager/storage_azure_blob.cc
77.27% <77.27%> (ø)

core/src/storage_manager/storage_manager.cc
39.81% <81.25%> (+0.5%)
:arrow_up:

core/src/storage_manager/storage_gcs.cc
1.85% <0%> (-85.19%)
:arrow_down:

core/src/storage_manager/storage_hdfs.cc
73.02% <0%> (-0.66%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5295b6...a632567. Read the comment docs.

@mlathara, @kgururaj, can you review this code please?


feat!(esm): provide esm outputs and continue to support cjs

What kind of change does this PR introduce?

[ ] Bug Fix
[x] Feature
[ ] Refactoring
[ ] Style
[ ] Build
[ ] Chore
[ ] Documentation
[ ] CI

Did you add tests for your changes?

[x] Yes, my code is well tested
[ ] Not relevant

If relevant, did you update the documentation?

[x] Yes, I've updated the documentation
[ ] Not relevant

Summary
Explain the motivation for making this change. What existing problem does the pull request solve?
Try to link to an open issue for more information.
This PR is a major breaking change just due to the engines field being updated to support latest Node. However this also includes the ESM imports and continues to support those who need CJS
Does this PR introduce a breaking change?
If this PR introduces a breaking change, please describe the impact and a migration path for existing applications.
Included in summary above
Other information

@rxmarbles do you still want to work on this? We'll be branching out into beta with https://github.com/react-dropzone/attr-accept/pull/75 so that we can get some pre-releases out with ESM support.

I think we can go ahead and close this one, ESM is widely supported and CommonJS support has also been dropped for other packages such as file-selector. Now that Node.js has introduced support for importing ESM from CommonJS (which will be backported to all LTS releases), this is no longer a blocker.


How to create Cyber Observable References

I'm finding a lack of documentation in cti-python-stix2 for creating object references for cyber observables as specified in the stix2 documentation. Specifically, I am trying to create a from_ref in the EmailMessage object:
`from =<EMAIL_ADDRESS>display_name="Annie")
body = [ {'content_type': 'text/plain',
'body': "\r\n\r\n\r\nYour package has been delivered.\r\n\r\n \r\n\r\nDelivery Data:  Thursday, 3/7/2019\r\n\r\n \r\n\r\nDelivery Time:  10:44 AM\r\n\r\n\r\n\r\n\r\n\r\n\r\nThis notification is to confirm that the following shipment has been delivered.\r\n\r\n \r\n\r\nTracking Number:  1Q767AZ20351136904\r\n\r\n \r\n\r\n     Visit  us on Facebook            \r\n\r\n     Download  the UPS mobile app\r\n\r\n \r\n\r\n© 2018 United Parcel Service of America, Inc. UPS, the UPS brandmark, and the color brown are trademarks of United Parcel Service of America,  Inc. All rights reserved.\r\n\r\nAll trademarks, trade names, or service marks that appear in connection with UPS's services are the property of their respective owners.\r\n \r\nPlease do not reply directly to this e-mail. UPS will not receive any reply message.\r\nFor more information on UPS's privacy practices, refer to the UPS Privacy Notice.\r\nFor questions or comments, visit Contact UPS.\r\n\r\nThis communication contains proprietary information and may be confidential. If you are not the intended recipient, the reading, copying, disclosure or other use of the contents of this e-mail is strictly prohibited and  you are instructed to please delete this e-mail immediately.\r\n\r\n \r\n\r\nUPS  Privacy Notice\r\n\r\n \r\n\r\nContact  UPS\r\n\r\n\r\n\r\n\r\n\r\n\r\n"},
{'content_type': 'text/html',
'body': '\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n<img size="4317" id="img667191" tabindex="0" style="max-width: 99.9%;'}]
email = EmailMessage(is_multipart="true", subject="Subject", body_multipart=body, from_ref=??? what goes here?)
observable = stix_obs = ObservedData(
created="2017-02-28T19:37:11.213Z",
modified="2017-02-28T19:37:11.213Z",
first_observed="2017-02-27T21:37:11.213Z",
last_observed="2017-02-27T21:37:11.213Z",
number_observed=1,
objects = {"0": email, "1": from}

`
So how do you create the reference to the from EmailAddress? I've looked at the classes in properties.py and still haven't figured it out.

Currently, there isn't an ideal way for creating references to Cyber Observables. You could define all of the Cyber Observable objects within the objects dict in your ObservedData object (and then use the number-based reference mechanism mentioned in the current stix2 documentation), but that is obviously not an elegant/practical solution.
Moving forward, the currently proposed mechanism for handling such references is to use _valid_refs, but looking at #175, this is also a work in progress. Although formal documentation is currently lacking for _valid_refs, you may be able to gain some insights from stix2/test/v21/test_observed_data.py since some of the tests there use _valid_refs.
And, I think in the context of your situation as I've understood it, the is_multipart property should be equal to True, not "true".

There is now documentation regarding the use of _valid_refs to create references to Cyber Observables. The documentation can be found in /docs/guide/creating.ipynb (towards the end of the file) starting at "Creating Cyber Observable References".
Just remember that _valid_refs is only designed to create/handle references to Cyber Observables, and is not officially meant as a mechanism for reference resolution, as per the spec.

In referencing the documentation cited, /docs/guide/creating.ipynb, the case "case 2" style did not work for me in the following example (whereas "case 1" style did):
Code: email_address_references_py.txt
Output: email_address_references_output.txt

Thanks for pointing that out; that issue only seems to arise when using python3.x, but not python2.x. (please confirm). We'll look into why that is and get back to you soon.
If you have any ideas of why this issue only appears with python3 but not python2, please let us know :)

Hi @Ksk190809, can you try again with the latest release on PyPI? We had a fix for this on master but had not yet included it in a PyPI release. Thanks!

Just leaving a comment here to let users know that this issue (specifically the use of _valid_refs) is mitigated in STIX 2.1 (updates coming very soon to the stix2.1 branch of the code base). This is because the most recent updates to the STIX specs (Working Draft 05) specify that Cyber Observable references should be made using the (new) ID property of whichever Cyber Observable object(s) is being referenced.

I believe this has been addressed and am closing the issue due to lack of activity, but if this is still an problem with the latest version feel free to reopen or create an new issue.


Fix controller deployment args

Without patch passing args to tink controller works not correctly.
  set_list {
    name  = "tink.controller.args"
    value = ["--log-level=1"]  # debug
  }

Resulted pod:
      args:
        - '--namespace=tink-system - --log-level=1'

Description
Tink Controller Chart pass args not correctly.
Why is this needed
For instance, for passing --log-level
How Has This Been Tested?
I didn't test it, but in another module use the same construction and works correctly: https://github.com/tinkerbell/charts/blob/29de47ae2be2581247573f03b613ac3214c38be3/tinkerbell/tink/templates/tink-server/deployment.yaml#L42
How are existing users impacted? What migration steps/scripts do we need?
Tink Controller Chart pass args not correctly.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @strike , thanks for this! Just need you to fix the DCO sign off, please. Reference.


The recommendation of public IP address is ambiguous.

Hi,
The recommendation summary of public IP address shows the count of public IP address which is no Zone.

But the table of public IP address shows not only no zone but also single zone as error.

Which is a correct recommendation of this workbook?

Public IP address should not be no zone. So, no zone is NG but single zone is OK.
Public IP address should be zone redundant. So, single zone and no zone are NG.

Best regards,

We discussed and adopted the following rule:

Public IP address should be zone redundant. So, single zone and no zone are NG.

The reason is that we want to be on the foolproof side. I understand PIP's redundancy depends on backend services, so choosing zone redundant isn't always the best choice. I hope there's a discussion to adopt the best architecture.

Closed with v2.10(#148).


ng update @angular/cli => Error: connect ETIMEDOUT <IP_ADDRESS>:80

[x] bug report -> please search issues before submitting
Versions
node --version: v8.9.4
npm --version: 6.0.1
Windows 10 x64
Repro steps
I'am behinde a proxy and the proxy settings in npm are fine so far, never got any problems until today.
I've local "old" angular cli installation 1.7.4 and I try to update this to 6.0.1
I've upgraded the global cli version to 6.0.1 works fine.
then I switch to the dir of my angular project with the package.json inside and try to
ng update @angular/cli

and then the following Error shows up:

I've found this pull which should be included, but I am not able to update
https://github.com/angular/devkit/pull/692
Desired functionality
easy upgrade to 6.x

I think was wrong repo to post error.
I reposted the issue here:
https://github.com/angular/angular-cli/issues/10952


Enhancement: Move connections value to env vars

An improvement so we can change/replace the following connections value in deployment without making changes into the code.
FTPServer. max_cons = FTP_MAX_CONNECTIONS
FTPServer.max_cons_per_ip = FTP_MAX_CONNECTIONS_PER_IP

Change the default OpenSSP server

What default should we use @jeremi ?


[1131] Add extra params for Withdraw

Context

https://trello.com/c/DmZEXlPZ/1131-l-add-extra-params-for-defer-and-withdraw

We weren't setting all the required fields when submitting a trainee to DTTP for withdrawal. Confirmed with Calum and Ben which fields need to be set and this PR adds them.
Changes proposed in this pull request

Extract DTTP withdrawal logic into its own service
Move PlacementAssignmentOutcome into a module PlacementOutcomes and renamed it to Qts.
Added a separate class to build params for a Withdrawal outcome. Also lives in PlacementOutcomes

Guidance to review
You'll have to test this with local DTTP access.

Prepare a trainee locally and in DTTP such that they have a TRN and are ready for a withdrawal
Run Dttp::WithdrawTrainee.call(trainee: trainee) in the console passing in the trainee to be withdrawn
Query that trainee's placement assignment in postman and assert the correct withdrawal DTTP properties (found in Dttp::Params::PlacementOutcomes::Withdrawn are set in the response

 [L] Add extra params for Defer and Withdraw

FYI duplicated code complaint by SonarCloud is from the withdraw_trainee_spec.rb being very similar to the recommend_for_qts_spec.rb. I think we decided to ignore these to keep tests explicit cc @gpeng


[SVC-2939] Include rate limit in API documentation

Motivation
It's helpful for developers to understand what rate limits are put in place. Rate limits are used to protect the developer's game more than it is to protect Rivet, so we eventually plan to allow configuring these rate limits manually.
From SyncLinear.com | SVC-2939

Out of sync with Linear


how to drill down into libs?

https://github.com/boostorg/boost/tree/master/libs
shows directories as non-clickable objects

what do I miss? how to drill down into those directories?
thanks.

A better place for questions like this is the boost-users mailing list (boost-users@lists.boost.org). For a quick answer, though - each library has it's own repo. So, you can find Boost.Any at https://github.com/boostorg/any/tree/master/

Thank you @mclow


Snowflake Connector (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[ x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @rekodus,
Thank you very much and congratulations on your new submission. Our Certification Engineers will review it within 1-2 weeks.
Please let us know if you have any questions.
Thank you very much again for working with us.

@rekodus Please provide few screenshots of the successful runs of the connector.

[[certify-connector]]

Hello @rekodus,
I hope you are doing well.
Congratulations, your connector is approved. We will move forward with the certification process. We are preparing your connector for production deployment and will queue up for the next deployment schedule. Once your connector onboards the next deployment schedule, it starts to deploy your connector in our production environments that typically takes 3-4 weeks.
Please let us know if you have any questions.
Thank you very much for working with us.

Hello @rekodus,
I hope you are doing well.
Your connector is in production deployment and queued up for the next deployment schedule. This process typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps.
Please let us know if you have any questions.
Thank you very much for working with us.

Hi Mahbub,
Does that mean the problems with database access and the proper role to be passed have been solved?
Regards,
René
From: Amjed Ayoub @.>
Sent: Monday, 5 December 2022 18:31
To: microsoft/PowerPlatformConnectors @.>
Cc: Koch, Rene @.>; Mention @.>
Subject: Re: [microsoft/PowerPlatformConnectors] Snowflake Connector (Independent Publisher) (PR #1840)
Hello @rekodushttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frekodus&data=05|01|rene.koch%40uniper.energy|d65b6b21a9514ebd51f608dad6e68261|db8e2f828a374c09b7deed06547b5a20|0|0|638058582780317644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LAygHKSxnF%2FMWlgggw2Jc7pVE5CoPObwLP9O%2BQ6hbzE%3D&reserved=0,
I hope you are doing well.
Your connector is in production deployment and queued up for the next deployment schedule. This process typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps.
Please let us know if you have any questions.
Thank you very much for working with us.
—
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerPlatformConnectors%2Fpull%2F1840%23issuecomment-1337794754&data=05|01|rene.koch%40uniper.energy|d65b6b21a9514ebd51f608dad6e68261|db8e2f828a374c09b7deed06547b5a20|0|0|638058582780317644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qgyJSJs9MTEPTVWpDCGyWE2Iq5hwGMwbBJ1QDHgi19s%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPMOMYI7IZOTRC5IORKLXTDWLYRGDANCNFSM542JQ6LA&data=05|01|rene.koch%40uniper.energy|d65b6b21a9514ebd51f608dad6e68261|db8e2f828a374c09b7deed06547b5a20|0|0|638058582780317644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MFJf7%2FqlXpA2iJLLC9d6wXsjnKMQxwT80HQL6yKQHYE%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@rekodus Congratulations! The deployment of your connector and documentation page is now complete, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program, Thanks.


CPU 100%

JSQR is requiring a lot of CPU on PC and Smartphone. I am using Google Chrome!. Please lighten this script.

@booleenter I have the same issue, did you fix that problem?


Add missing names of usages.

Currently, only the following usages have names defined:

Generic Desktop Page
Lighting and Illumination Page

The Generic Desktop page is by far the most popular usage page, and nearly all HID devices report under that. Still, we could have devices outside that page, so it would be better to add the names of usages in other pages as well.

Added page:

Consumer

Added page:

Digitizer


Rename project

Fixes #2. After merge I'll fix the gh repo
Note that this will require a database rename if you've created one locally
@nirrattner

gonna go ahead an merge this


Dexis not CSP compliant

https://github.com/dfahlander/Dexie.js/blob/0a1cbdecb5b775d2c1d6f90381f6724d78593d84/src/helpers/promise.js#L47
This line makes Dexie not really CSP (Content Security Policy) compliant because it needs adding 'unsafe-eval' within script-src, wich is not a good practice ! Could it be remplaced in order to correct this issue (Function(str) is eval) ? Thank you

The code above tries to detect if async functions are supported.
I am not an expert on this topic, but I don't understand why we need to differentiate between async functions and promises, at all.
Example:
const myAsync = async () => {
  return 'foo'
}

is basically the same as:
const myAsync = () => {
    return Promise.resolve('foo')
}

So we are testing/optimizing an implementation detail, that should be not part of this project.
So my recommendation is to handle async functions in the same way we handle Promises.
@dfahlander Please let me know, if I am on the wrong track here. 🤭

Nevertheless thanks for this awesome project. It has been a charm using it ♥.

@HaNdTriX We basically have to distinguish the built-in promise from a polyfilled one. There are two reasons that Dexie deals with this low-level stuff. First, because IndexedDB Transactions does not work with native Promises on all browsers, so Dexie bridges that incompatibility using its own promise implementation. Two years back, this was the case for all browsers except Chrome. Nowadays, it only applies to older browsers, as Firefox recently solved the issue in its very latest version. The second reason is that Dexie's API depends on a zone system that works across native async calls and we need to detect that as polyfilled Promise would give other results than an async result. We don't want the code to be transpiled, that's why it is written in a string.
So basically, native async calls will not work as expected without this code. As long as you transpile away your "async" and "await" app code, your patch will work fine.
Could fix this withing Dexie's build system instead though (by omitting transpilation of specific parts of the code).

@SlIdE42 You can apply @HaNdTriX patch if you target older browsers and transpile all your code to ES5 or ES6, or keep away from async/await.
It's a pitty that the CSP tool that warns about this doesn't see that the input to the function is a constant string - which basically means it is not vulnerable. Would be different if we passed a variable to it.

I ran into this while trying to use Dexie.js in a browser add-on (webextensions).  On Firefox I get this error:
Content Security Policy: The page’s settings blocked the loading of a resource at eval (“script-src”). dexie.js:630:15

Which appears to be the same line at the top of this issue:
return new Function("let F=async ()=>{},p=F();return [p,Object.getPrototypeOf(p),Promise.resolve(),F.constructor];")();

There is a way to prevent the error by not using the default CSP, but that might cause the add-on to be rejected.  As described on MDN's best security practices:

Use the standard extension content security policy (CSP)
The standard policy [...] disallows potentially unsafe practices such as the use of eval(). While the manifest.json key content_security_policy enables you to modify the content security policy for your extension, this isn’t recommended as the policy helps prevent extensions from inadvertently executing malicious content. If your modified CSP allows remote script injection your extension will get rejected from AMO during review.

For this use case (browser add-on) I don't need to support older browsers.  I'll try the non-default CSP, option and if that doesn't work, I'll look into using the 'patch dexie' work-around shown above, but wanted to mention this as another use case where this comes up.

And... modifying 3rd party libraries is also a red flag, per MDN's security best practices:

Do not modify third-party libraries
Modifications to a third-party library are a significant indicator that a developer is trying to hide malicious code within code that is generally known and trusted. AMO will therefore try to detect changes to third-party libraries and may disable extensions when it finds changes.

@dfahlander This CSP-offending code does not seem to exist in Dexie v3, how did you solve it there?

@dfahlander This CSP-offending code does not seem to exist in Dexie v3, how did you solve it there?

Still seems to be there as far as I can tell:
https://github.com/dfahlander/Dexie.js/blob/master/src/helpers/promise.js#L47

@dfahlander This CSP-offending code does not seem to exist in Dexie v3, how did you solve it there?

Still seems to be there as far as I can tell:
https://github.com/dfahlander/Dexie.js/blob/master/src/helpers/promise.js#L47

@birtles You're right, I thought v3 was being tracked in the next/v3 branch. As you pointed out, it's being tracked in master, and unfortunately the offending code still seems to be required.

I found a solution to distinguish polyfilled promise from the native one without breaking the CSP. Will be included in next version (will be 3.0.0-beta.1). Planning to release a stable 3.0.0 within a week or two.

Thanks so much @dfahlander! Is there an updated estimate on the release date of the stable 3.0.0?

I'll see what possibility there is to release it this week. But not later than mid November.


Stopped working 🤷

This is one of my favourite logseq plugins. Today it stopped working, I don't know why.  I tried to update but still doesn't work.  How to proceed?  Should I uninstall and try to reinstall?

Sorry, please update and try again.
On Mon, Feb 7, 2022 at 9:35 PM Fred Guth @.***> wrote:

—
Reply to this email directly, view it on GitHub
https://github.com/hkgnp/logseq-datenlp-plugin/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNXKGKFNF3HKV7AHCRFPGLUZ7DCNANCNFSM5NXPCBBQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Fixed!


Sqoop2: Fixed spelling / typo's in the docs.

Hi @ruurtjan ,
Sqoop project currently does not accept pull requests. To contribute, you will need to generate patch and upload it to the JIRA (of course if there's no corresponding JIRA issue, you will also need to create one first).
See the instructions here: https://cwiki.apache.org/confluence/display/SQOOP/How+to+Contribute
If you need more directions, I'd be happy to help!


[SPARK-25046][SQL] Fix Alter View can excute sql like "ALTER VIEW ... AS INSERT INTO"

What changes were proposed in this pull request?
Alter View  can excute sql  like "ALTER VIEW ... AS INSERT INTO" . We should throw ParseException(s"Operation not allowed: $message", ctx)  as Create View does.
override def visitCreateView(ctx: CreateViewContext): LogicalPlan = withOrigin(ctx) {
   if (ctx.identifierList != null) {
     operationNotAllowed("CREATE VIEW ... PARTITIONED ON", ctx)
   } else {
     // CREATE VIEW ... AS INSERT INTO is not allowed.
     ctx.query.queryNoWith match {
       case s: SingleInsertQueryContext if s.insertInto != null =>
         operationNotAllowed("CREATE VIEW ... AS INSERT INTO", ctx)
       case _: MultiInsertQueryContext =>
         operationNotAllowed("CREATE VIEW ... AS FROM ... [INSERT INTO ...]+", ctx)
       case _ => // OK
     }

override def visitAlterViewQuery(ctx: AlterViewQueryContext): LogicalPlan = withOrigin(ctx) {
    // ALTER VIEW ... AS INSERT INTO is not allowed.
    ctx.query.queryNoWith match {
      case s: SingleInsertQueryContext if s.insertInto != null =>
        operationNotAllowed("ALTER VIEW ... AS INSERT INTO", ctx)
      case _: MultiInsertQueryContext =>
        operationNotAllowed("ALTER VIEW ... AS FROM ... [INSERT INTO ...]+", ctx)
      case _ => // OK
    }
    AlterViewAsCommand(
      name = visitTableIdentifier(ctx.tableIdentifier),
      originalText = source(ctx.query),
      query = plan(ctx.query))
  }

How was this patch tested?
UT has been added in SparkSqlParserSuite

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #94387 has started for PR 22028 at commit de44d0d.

Test build #94387 has finished for PR 22028 at commit de44d0d.

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

Merged build finished. Test PASSed.

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

Thanks! Merged to master.


Look at Placement of Adverts (see issue #612)

E3 U3 S6 C6: P18

Unsure what's needed here, #612 is deployed

Issue #612 is in Ready for Testing, and this isn't about that issue , but the images there raise questions about the placement of adverts.
In balancing need for $$$ and UX, I'd like to suggest removing the second advert on the detailed result page, but implementing a sticky advert at the bottom here.
On On the Records pages, it would be good for legibility for the tables to take up the normal "2/3" (I know it is not 2/3!) and there to be a side advert.
What do others think?

Sorry, I meant deployed to Test3.
Thanks for the extra info, I take it that it was felt that the extra level of buttons in #612 made the layout unpleasant. I think your suggestions sound like a very good solution.

Yes, the extra layer of buttons, while excellent, and the placement above/below the census information rather together will help people remember which 'layer' things are in, that comes at the expense of 'Show me my Great Uncle!!!'

If we move the ad to the bottom then 3 separate action lines is poor; I would suggest that the printing option go with the dwelling header since we print the dwelling and its contents. The citation can then go with the individuals header. This then gives us 2 nicely separated action header

@PatReynolds The problem with putting a sticky add at the bottom could be the fact that we have the Donate box etc under the individuals

There is lots of scroll room below the invitation to donate (and  Top of Page and Report a Problem buttons),  However there would need to be more black space under the copyright statement.

Remember Top of Page and Report a Problem buttons are not there for researchers

For information, earnings last month
freecen_728x90 (top banner) £90.03
right_side_300x600 (right rectangle) £1.17
FC1 search £17.08
FC1 homepage £5.97
FreeREG comparisons are here: https://docs.google.com/spreadsheets/d/1i2YklMJScm3DGqnjf2O1GXX-DBPx0uKAVFF3PxiEMbQ/edit?usp=sharing
Personally would prefer clean pages (i.e. no adverts apart from banner at top and rectangle at site - where there is a side), but with an easily dismissed sticky at the bottom.


[README] Programmatic usage + browserify

Added information on programmatic usage and browserify bundling - took me a bit of time to figure this out and I hope it can help somebody else.

It needs to be cleaned up a bit (there are extra comments and whitespace), and the wording could be a bit more specific, but this seems good.
Why is there a section on a Browserify shim? Holder should work with Browserify out of the box.

I've removed the browserify section (I originally couldn't get it working unless I used the shim, but I just checked again now and it seems ok, so it must have been another issue).
Please let me know what other cleanup you'd like to see.

@hiddentao comments added

I've made the changes.

Excellent, thanks very much.


Rename bundle identifiers defined in project.pbxproj and Info.plist

I believe this may be fixed by #59 since I can't reproduce it locally. I'll release a version with the fix today and hopefully you can double-check.


Add range to dash config

Add range to dash config
In order to draw the indicator of the skill we should have this field in the dash skill.

This will be fixed sending the mechanics config and handling the logic in the client


Add expression bodied function support

Purpose

This PR adds support for expression bodied functions. The users will now be able to define functions as follows:

function toEmployee(Person p) returns Employee => {
    name: p.fname + " " + p.lname + " " + x.toString()
};

public function main() {
    Person p = {fname: "John", lname: "Doe", age: 25};
    io:println(toEmployee(p));
}

Fixes #20686
Approach

To properly support expression bodies, this PR refactors the way function bodies are modeled. Previously a function body was modeled using a block statement. But with a 3rd type of function body added to the grammar with expr bodies, continuing to use the block statement wouldn't cleanly separate the logic for function bodies from generic code blocks. So instead of using the block statement, this PR introduces new abstractions for the function body. Now the block function body is represented by BLangBlockFunctionBody and the expr body is modeled by BLangExprFunctionBody. For now, no changes were done to how external bodied functions are handled. But the abstractions for it are there.
After this refactoring, adding support for expr bodied functions was trivial. The expr function body is desugared to a block function body with a single return statement.

{ return expr; }

Remarks

Builds on #20910

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Add tests for taint checking

Codecov Report

Merging #20929 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #20929   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1398     1398
  Branches      214      214
=======================================
  Hits          204      204
  Misses       1178     1178
  Partials       16       16

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2eae25...f13766b. Read the comment docs.