Multiple instances of app don't appear as separate processes in Task Manager

Problem Statement
Calculator supports multiple windows today, but they are all part of a single process. This means that a crash or hang in one instance brings down all instances and that it is not possible to terminate individual instances via Task Manager.
Evidence or User Insights
Terminating one instance will shut down all.

Proposal
Instead of handling multiple views of the app within a single process, we should use the new platform APIs to enable an app to be multi-instance.  That will allow instances to be tracked in Task Manager, and if one instance crashes the other instances will remain alive.
Goals

Refactor current multi-window architecture to leverage platform APIs and enable true multi-instance Calculator
Maintain existing reliability/performance targets

Non-Goals

None

Low-Fidelity Concept
N/A
Requested Assignment
I'm just suggesting this idea.  I don't want to implement it.

There are a number of open questions we have with this one and some trade offs to consider against the current multi-window approach. I know @mcooley and @HowardWolosky have some thoughts here, so I will let them chime in.

@HowardWolosky and I talked offline about the tradeoffs involved here:
Benefits:

Data protection. If there is a bug in the app that takes down an instance of the calculator, the other instances wouldn’t go down at the same time, losing the work that users had in their other instances. This isolation would have offered us some protection against a crash we introduced recently with always-on-top mode, if we hadn't caught that bug in time.

Drawbacks:

Perf overhead of extra processes (for both calc.exe and the runtime broker). We'd need to measure to see what the actual impact is on memory usage and the time required to start up a secondary window.
Sharing state between windows is harder. For example, let's say we chose to implement settings in the app, like theme selection. It’s a better experience if it changes all the windows at once, and that’s hard if they’re multiple processes.
New concurrency issues introduced. For example, how do you make sure that a new request for the currency converter rates doesn’t get downloaded each time the user opens a new calc window?

I was not aware of this issue asking for multi-instancing when I was looking at prototyping this functionality, so I have not taken into account the discussion here around pros/cons, potential impact for future features, etc.
However, I did code it up because of other reasons, so if anyone wants to check out my prototype, provide feedback on the approach I've taken, point out where there might be dragons, use it as a template for brining the functionality into main, or simply use it for investigations into the topics identified in this thread (perf overhead, currency converter rate syncs, etc.) feel free to spelunk the following branch on my fork of Calculator: https://github.com/rkarman/calculator/tree/rkarman/multiinstancing


[Blog] Java SDK 7.0.0-alpha1 announcement

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
Blog post to announce our newest Java SDK release
Special notes for reviewers:
Additional information (if needed):

@fjtirado? 😬

I think we might add more text to the blog.
For example, given workflow definition
document:
  dsl: 1.0.0-alpha1
  namespace: default
  name: http-call-with-response-output
do:
  - getPet:
      call: http
      with:
        method: get
        endpoint:
          uri: https://petstore.swagger.io/v2/pet/{petId}
        output: response

Once you have loaded the workflow into memory, you might access the Http call info by writing
 workflow.getDo().get(0).getTask().getCallHttp()

@fjtirado we should add this to the SDK readme instead. This is just an announcement post.

@ricardozanini Ok, I mentioned that because I see we were adding some code snippep to the annoucement and just putting that one might hide the fact that we have the generated classes.


sincerely request to public the pre-trained model for PCN dataset

amazing job in self-supervised pc completion area.
but i can't get the counterapart cdl2 score in the paper for PCN dataset when i train the model with vanilla setup.
so I sincerely request the author to release the pre-trained model of the PCN dataset.

Thank you for your interest on our work.
The vanilla setting is not suitable for every category, so I did make some adjustments on the weights for different categories.
Unfortunately, I don’t have access to the PC that stores the weights for at least the next two months.
I am confident that everyone should be able to reproduce the reported scores.
Please let me know any problems you encountered, so I can help you reproduce the results.

Thank you for your interest on our work. The vanilla setting is not suitable for every category, so I did make some adjustments on the weights for different categories. Unfortunately, I don’t have access to the PC that stores the weights for at least the next two months. I am confident that everyone should be able to reproduce the reported scores. Please let me know any problems you encountered, so I can help you reproduce the results.

Appreciate your timely reply! That really dissolve my confusion.

Before you comment any specific questions, I would like to share some empirical tricks to run reproduce the results for every category in PCN.

The weight for NCC loss should be reduced if the geometry is usually complex or not flat in a category, e.g. lamp, watercraft.
Make the weight of loss for reconstruction loss low if the result lack completion capability.
To be noted, samples in the PCN dataset is much more incomplete than 3D-EPN, so we are training with complete samples from 3D-EPN as we clarified in the paper.

Before you comment any specific questions, I would like to share some empirical tricks to run reproduce the results for every category in PCN.

The weight for NCC loss should be reduced if the geometry is usually complex or not flat in a category, e.g. lamp, watercraft.
Make the weight of loss for reconstruction loss low if the result lack completion capability. For example, 1 for recon loss, 1000 for completion loss.
To be noted, samples in the PCN dataset is much more incomplete than 3D-EPN, so we are training with complete samples from 3D-EPN as we clarified in the paper. To train with complete samples, it is easy by copying all the complete samples to the same folder of partial samples.

Hello again! I tried train EPN dataset these days. Stay, cant obtain the right cd score. So, it is my problem or the configyaml in your code is not correct?

May I confirm which setting you are referring to? a. purely train the P2C on 3D-EPN dataset; b. jointly train PCN with EPN?
I have provided the pre-category weights for the 3D-EPN dataset. Even we use the same config, the results should be close to the reported results.

May I confirm which setting you are referring to? a. purely train the P2C on 3D-EPN dataset; b. jointly train PCN with EPN? I have provided the pre-category weights for the 3D-EPN dataset. Even we use the same config, the results should be close to the reported results.

NoNoNo, this time i just use the config file in the path "cfgs/EPN3D-models/P2C.yaml" train EPN dataset.
so you mean using this very config file can wrok for every single taxonomy in EPN dataset, right?


Navigation no longer working; receiving 404 errors

Hey hey.
My multirepo doc site was working fine up until yesterday, but when I checked it today, links to those docs sites in the side navigation all give 404 errors.
I've tried running workflows on the individual repos and the multirepo doc site to see if that helped, but neither of those attempts worked.
Were there any changes to how navigation is treated?

Rerun the workflow and pin the plugin version to 0.4.3. The previous version had a bug. I was able to build your site locally and all links worked for me.

I'm going to close this again since I can't reproduce it. I am running into errors with the git-revision-date plugin but when I remove that the site seems to be built fine.

Thanks for looking into this.
Looks like the documentation for the git-revision-date plugin was recently updated:

https://github.com/mkdocs-material/docs/setup/adding-a-git-repository.md

I'll do some testing later to see if updating the plugin name fixes the issue.
Appreciate your help!🙏🏻


Fix datamigration definition, model methods not available when migrating

Coverage remained the same at 88.235% when pulling 027c7ba3036540f678ea757fa20dcb46edb079dc on johngian:fix-datamigration into bce935a7ebfb2196cf5137510bb64a86472bb114 on mozilla:master.


add ServiceProvider to TestAdapter.AddUserToken

Close https://github.com/microsoft/botbuilder-dotnet/issues/2873
It seems not harmful..

Pull Request Test Coverage Report for Build 88945

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 76.482%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/Adapters/TestAdapter.cs
15
83.77%

Totals

Change from base Build 88872:
-0.3%

Covered Lines:
8065

Relevant Lines:
10545

💛  - Coveralls

@gabo, can you take a quick look at this?

Closing due to inactivity.


Arduino LCD display 1602 I2C adresse

The actual LCD display 1602 I2C models have different i2c adresses than the one that we support (old model with fixed adress)
The I2C adress should be configurable in the corresponding configuration block, just a little input field.
So the generated code from the configuration should look like
LiquidCrystal_I2C _lcd_L2(0x27, 16, 2);
where 0x27 is derived from the configuration block.

Tests are hard to do because special hardware is needed. Should work, please reopen, if not


Leonardo/users relationship impl

Description
This PR implements the users' relationships.
Close #168 .
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote unit tests.
[x] Wrote integration tests (simulation & CLI).
[ ] Updated the documentation.
[x] Added an entry to the CHANGELOG.md file.
[x] Re-reviewed Files changed in the Github PR explorer.

@RiccardoM I've made all the changes we discussed during the call.
I've not updated the docs yet, I'll wait for the approval of the code than I'll do it.


Issue/317

@walkwel sorry i didn't notice your pull request just now. I did some style reformat (indentation etc) of AnalysisDialog.js. I will resolve the conflicts on github.


Short blog post on the community trends app

This would be useful for showcasing a concrete, useful use case of the twitter community archive. And also as inspiration for developers.
General ideas / outline:

Explain that it exists and is open source: https://labs-community-archive.streamlit.app/
Explain that we can trace the origin of new terms, like Visa's "psychofauna" and see it overtaking or in relation to the standard terminology "egregore"
Example of using it to find others who talk about the same concepts I do? (like me finding that Forshaper is also doing open culture studies, that helps my career?)
Track the rise of individual people in a community by searching their name?
TODO, other examples?

Defender's thread, "science vs spirituality"  https://x.com/DefenderOfBasic/status/1846905536115024307
Xiq's original post: https://x.com/exgenesis/status/1846578431212802156
(this is more general but want to put it somewhere for now: Xiq's thread of app ideas/stuff people are building on top of community archive https://x.com/exgenesis/status/1835411943735140798 )

Looking at the very first usages of the word "LLM" in the archive is fascinating. Here are people in 2021

Papers of LLMs are barking on the wrong tree. If the main effort was to get all the right parameters and engineering efforts. Why doesn't it take most of the report? This is the main thing we can learn about the next LLM.

https://x.com/i/web/status/1473608713625812996
(ChatGPT release date is November 30, 2022. The "attention is all you need" paper was 2017?)

Twitter recently released a "radar" premium feature that lets you do this kind of trend analysis on real-time twitter data. This is validating that there's real value here. The "radar" tool only lets you search data up to 7 days ago. Also we can't improve it/add other data sources/do semantic search


Usage as non privileged user

How the pipx provider is expected to be used as a normal non privileged user?

The provider by default works system-wide (parameter $global = true in main class), for per-user usage just disable this funcionality, for example with this hiera snippet:
puppet_pipx::global: false

I case you want both global and per-user usage I don't think the module supports this at present. We could do it, just need to think about a feasible implementation.

Perfect, thanks, I should have checked better the code.


Ways to disable line feed in datatable?

I want to display some data, with a column called article title which can be up to 30 chars ( If it is too long, I will strip it ), but I want to enable horizontal scoll when screen width is too small to display all the chars.
I tried to add white-space: nowrap to the root node of datatable in DevTools, and it is work for my case, but when I turns to add kwargs in pc.datatable, it doesn't work.
Is the attr has been attched to a wrong DOM node? It is a Bug to fix or there are another way to do it?

Got it do you have experience with gridjs https://gridjs.io this maybe be a good issue for you to tackle if you want to get more involved with the feature development on the framework.
We really appreciate your help with the open source so far!

Not any experience yet, but will try to dig into this.

Ok awesome just as some guidance you will mostly be editing this file https://github.com/pynecone-io/pynecone/blob/main/pynecone/components/datadisplay/datatable.py. ill be free tomorrow to go over anything and would be happy to give you an introduction to the framework if you want


Add real documentation beyond the README

Let's make a nice readthedocs.io documentation

This is done!


Won't match entry if query appears after position 70 in the given data set.

Describe the bug
If the string we want to match appears after position 70, in another string which is longer than 70 chars, it won't match it.
Version
Not sure before this, but happens on 6.0.0 up to 6.2.1.
🔬Minimal Reproduction
const assert =  require('assert')
const Fuse = require('fuse.js');

const list = [
  {
    title:  'The Lock Artist',
    author: {
      name: 'Steve Hamilton',
      tags: [
        {
          value: 'The readable English, Many desktop publishing packages and web juliancito'
        }
      ]
    }
  }
];

const options = {
  includeScore: true,
  keys:         [ 'author.tags.value' ]
};

const fuse = new Fuse(list, options);

const result = fuse.search('juliancito');
assert(result.length > 0)

Not a bug.
Please check out the scoring theory.
Since you didn't specify the fuzzy-matching options, Fuse.js will use the following values:

location defaults to 0
distance defaults to 100
threshold defaults to 0.6

With the above options, for something to be considered a match, it would have to be within (threshold) 0.6 x (distance) 100 = 60 characters away from the expected location 0.
Since 70 > 60, it obviously wouldn't match.
Strongly encourage everyone to read the scoring theory for additional insights.

Thank you for the insight.
Will read. Much appreciated.


Unable to instantiate error with non literal string

On version 0.1.28 of @ts2asl/asl-lib, I was able to do (playground here):
import * as asl from "@ts2asl/asl-lib"

export const main = asl.deploy.asStateMachine((_: any) => {
  const error = {
    message: "Some Error Message"
  };
  throw new Error(error.message);
});

Now, I get the following error:

Error: error thrown must have string literal as argument

a previous version of @ts2asl/converter might have accepted this, but it would not have resulted in valid ASL as the following is not supported by AWS:
   "Assign error": {
      "Type": "Pass",
      "Result": {
        "message": "Some Error Message"
      },
      "ResultPath": "$.vars.error",
      "Next": "Throw Error"
    },
    "Throw Error": {
      "Type": "Fail",
      "Error": "Error",
      "Cause.$": "$.vars.error.message", //not supprted
      "Comment": "source: throw new Error(\"Some Error Message\");"
    }

if the intention is declare a variable once and use it multiple times you can so it using asl.deploy.evalConst:
const error = "Some Error Message";
export const main = asl.deploy.asStateMachine((_: any) => {
  throw new Error(asl.deploy.evalConst(error));
});

The intent was to create an error from a lambda function's response, for instance. I dig the actual commit where that was working again.

makes sense. i dont believe this is possible in ASL. unless you throw the error with the right message/cause from within the lambda (using asl.runtime.createError).
if you find a way to do this in ASL, let me know

@eduardomourar , can this be closed?


Add load-testing to list of valid sample slugs

azure-load-testing is an approved product slug in https://review.docs.microsoft.com/help/contribute/metadata-taxonomies?branch=main#product but is not present in the SampleMetadata.ps1 file. Adding it here as pipelines are failing due to this

/azp run azure-sdk-tools - sync - eng-common

@sima-zhu could you please help this PR through the eng/common sync process?

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

@sima-zhu It seems the pipeline failure is related to permissions in android repo. Is it possible to provide those permissions temporarily or change the reviewer to someone with access?

@Harshan01 I just added you to azure-sdk-read team which should give you enough permissions for this.

@Harshan01 I will help to move the pipeline forward.
BTW, docs also needs to the slug for the docs metadata. Could you please also update the field MSDocService with the value to your service in csv?
This is java csv: https://github.com/Azure/azure-sdk/blob/main/_data/releases/latest/java-packages.csv. The other language is under the same folder.

@sima-zhu I see for most of the languages, MSDocService column is empty. Is this expected or does it mean that those services did not use custom product slugs?

@sima-zhu I see for most of the languages, MSDocService column is empty. Is this expected or does it mean that those services did not use custom product slugs?

It is legacy issue. Most of the services do not aware that Docs team required to have accurate product slug. We have service/package name as default one in code. Docs team throws us many warning for inaccurate service name. That's why we introduced a column to override the default value. We are now pushing service to update this as many as possible.

@sima-zhu Created the PR for docs: https://github.com/Azure/azure-sdk/pull/5272


Atomic Array Wrapper

Add a wrapper that makes any array atomic by adding an array of locks

Instead of storing a dense array of locks, consider reusing the locks for blocks of locations


Added new placeholders

custom_fishing_boolean boolean
custom_fishing_status Custom string in message.

On request from:
https://github.com/Oheers/EvenMoreFish/issues/420


New settings object

This is a work in progress of creating a new settings object for asreview. It started out as a small project, but is getting a little bigger, so I thought I would make a draft pull request to get some feedback. Here are the notes I've been taking:
New ASReviewSettings object
Purpose of the current object

CLI info is stored in the settings objects
This info is then used to initialize models/reviews
This info is stored in the state object
webapp gets review info from it

Furthermore, the settings object does some type casting. There is a hard coded list of names for which it typecasts the value to a certain type.
Finally it can read settings from a config file, and read model default parameters/type check/type cast whilst doing it.
Related to the settings objects, BaseModel class has an attribute called param. This is an attempt to automatically generate the model parameters. It is only used in review/base.py and webapp/api/projects.py to initialize an ASReviewSettings object. It also does some type checking/casting and complains if the type in the signature of __init__ is different from the type of a corresponding stored attribute.
Occurences

entrypoints/simulate.py:

for webapp, load settings from state and initialize model
for simulate, load settings from cli/config and initialize model/reviewer -> could skip settings object.

review/base.py:
Review.settings:
Store the info on the settings of the review (model/n_instances/stop_if/n_prior_included/n_prior_excluded/
model_parameters)
state/legacy/dict-hdf5-json.py:
Initialize from unknown dictionary.
state/sqlstate.py:
Store the settings of the state.
webapp/api/projects.py:
Set the settings in the state.

Contents

Model info:
Classifier/Query/Balance/Feature
seperately, model_parameters.
Review settings:

n_instances
stop_if
n_prior_included
n_prior_excluded

Data object, but it is unused.

Ideas for improvement

Remove reading config file from the settings object. It should be dealt with at the entrypoints.
(Maybe have a Settings.from_json() method.)
Let model components declare settings themselves, using the principle that Model.__init__(**model.settings) = model
Let review components declare settings using the principle
Review.__init__(review.model, **review.settings) = review
Remove the param attribute from BaseModel.
Let the settings object have a to_json method so that the state can store the settings.
(And make sure Settings.from_json(settings.to_json())) == settings.)
Make sure that random state satisfies RandomState(random_state.random_seed) = random_state and
store it in the settings as it's seed. If the seed has been set for a random state,
you can retrieve it by random_state.get_state()[1][0]. We can write a small wrapper around
the numpy random state, so that you can only initialize it using a seed, and the seed is
an attribute of the state.
Make tests that check random state is always seeded.
Make tests that check Model.__init__(**model.settings) = model
Make BaseModel._settings an abstract property so that you are forced to implement it
when creating a new model component.

Progress

Random state is now always seeded, and it is easy to obtain the seed from the object.
All model components declare which settings they use.
Made tests for random_state
Made tests checking Model.__init__(**settings).settings == settings.

I will extract the useful parts from this and make separate pull requests for those.

Thanks, this is still very promising!


Private channels can run no way on multi server with top domain ?

For excample I have two server on nginx,  one server domain is "www.aaa.com",other server domain is "www.bbb.com".Then how to set laravel-echo-server.json?
I has been tried "authHost": "http://nginx",  ， but it will logs 404 like this

<title>404 Not Found</title>

404 Not Found

nginx

I has been tried "authHost": "http://aaa.com"， it logs
{ Error: connect ECONNREFUSED <IP_ADDRESS>:80
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1163:14)
  errno: 'ECONNREFUSED',
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 80 }

You should set host to null , it wil accept requests for any host. And check also Cors

the same problem: I had publishing app on server with one IP, but access through another IP on router with mapped ports. Error:
[01:27:22] - Error authenticating 1K7v1GZsKswn9oqYAAAB for presence-portal
{ Error: connect ECONNREFUSED ...:443
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1104:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '...',
port: 443 }
Error sending authentication request.


Avoid key collision for child trie

Update 11/11/19, this PR will revert to initial approach where unique id is provided by the srml storage.

Closing in favor of #4162


Application could not be started

Describe the bug
After running the installatoin script, autodesk does not want to be launched
To Reproduce
Steps to reproduce the behavior:

Run the steps as described in the readme file and get a successfull install
Run [drill@archlaptop Autodesk]$ xterm -hold -e "/home/drill/.local/share/applications/wine/Programs/Autodesk/fusion360-launcher.sh"
See error

Expected behavior
Im explecting it to launc in a manner to where i can intereact with it
Screenshots
https://cdn.discordapp.com/attachments/937093221190864896/946013862078083132/IMG_2979.jpg
https://cdn.discordapp.com/attachments/937093221190864896/946013862652690482/IMG_2978.jpg
Desktop (please complete the following information):

Version number of Autodesk Fusion 360 Not sure how to get that, its not running
Graphics card model + Graphics card driver (Version number) I use a nvidia card and have the latest version
Operating system (OpenSUSE Leap 15.3, Fedora 34, ...) Archlinux, btw
Desktop environment (XFCE 4.16, Gnome 40, ...) Qtile
DXVK- or OpenGL-Mode not sure what that is

Additional context
Add any other context about the problem here.

Please, use my newest setup wizard! 🙂


Default margins for header and footer are already added

Puppeteer version: 1.11
Platform / OS version: MacOS Mojave
URLs (if applicable):
Node.js version: 8.9.4

What steps will reproduce the problem?
Please include code that reproduces the issue.
Below is my to code to generate the PDF:
await page.pdf({format: 'A4', displayHeaderFooter: true, headerTemplate: "
Hello Header
", footerTemplate: "
Hello Footer
", margin: {top: '100px', bottom: '100px'}});
I am basically trying to set the the top margin for the headerTemplate to 0. But it seems this style is not picked.

If I am giving margin-top value as some negative value, then the header seems to move upwards but it also applies the same margin value to it's left. Weird! ( I have applied the negative value to be -15px)

What is the expected result?
Header should be placed just below the start of page without any margins when I provide margin-top value as 0px. Also, if I give some negative value to margin-top, it should not apply the same value to margin-left, right and bottom.
What happens instead?
Header has some default margins even if we provide inline margins. Is it that Puppeteer internally adds some default margins to headerTemplate and  footerTemplate ?

This is related to #4132 . Chromium sets a default padding for header and footer. You can override it by selecting #header and #footer.


Why does the email template still link to app.css after inlining?

How can we reproduce this bug?

Use foundation new --framework emails with<EMAIL_ADDRESS>to set up a new project.
Run either npm run build or npm run zip.
./dist/index.html or ./dist/index.zip/index/index.html is created.

What did you expect to happen?
Since all styles are inlined into index.html no external css files should be needed anymore.
What happened instead?
index.html still contains <link rel="stylesheet" type="text/css" href="css/app.css">. Is there a reason for that? Is the reference actually needed or should it be removed?

I was wondering the same thing. It would be nice to know.

I always delete it from the inlined emails, and from what I've tested (using all Inky tags) it doesn't affect the template in any way.

As it seems the css file is really not needed anymore, I created the pull request above to update the gulp workflow of the project created by foundation new --framework emails to automatically remove the link tag when using npm run build.

Thanks for creating the pull request in this matter. You just saved me the effort of doing the exact same thing. Yes, the inlining process should strip all redundant exterior links.

Please see my comment in the PR (https://github.com/zurb/foundation-emails-template/pull/23), just adding removeLinkTags: true in the inliner function gives you what you are looking for.

Fixed! Thanks for pointing this out! https://github.com/zurb/foundation-emails-template/commit/75d5f37749750c4b3a802e4f8e15387cf1e94da7


Add EOF testing completeness matrix

Kind request to implementation owners to double-check if mentioned revisions of tests pass and provide input on completeness (apologies if my first take is off here, also I have no accurate knowledge on erigon/geth/nethermind/EELS).
Also, if any other client is exporting fixtures like evmone, feel free to let know and add to the table (this is to enable us to find issues earlier, meanwhile work on providing coverage with EEST is ongoing)

What abot https://github.com/ipsilon/execution-spec-tests/releases/tag/eip7692%40v1.0.4%2Bipsilon_bf37249b, is this worth adding?

What about https://github.com/ipsilon/execution-spec-tests/releases/tag/eip7692%40v1.0.4%2Bipsilon_bf37249b, is this worth adding?

What abot https://github.com/ipsilon/execution-spec-tests/releases/tag/eip7692%40v1.0.4%2Bipsilon_bf37249b, is this worth adding?

Nope it's an interim release to get past CI hurdles on evmone's side exclusively. Actually, I'm wondering if maybe we shouldn't delete those releases, so that no one runs into them (see the notice on the prerelase)

Erigon is passing EOF tests for now, soon state tests should pass as well

Erigon is passing EOF tests for now, soon state tests should pass as well

@racytech can you be more specific, do you mean all 3 "kinds" of EOF tests in the versions specified in the table?

OK, I'll merge this with a few ones I'm uncertain about and please file PRs if anything is off or out-of-date


ImportError: No module named setuptools

Hi, dear authors,
When I run
 python setup.py build_ext --inplace
The error information is  ImportError: No module named setuptools

can you pls tell me why this error happened?
Thank you very much!

Try sudo apt-get install python3-setuptools.

have tried but no use.

---- Replied Message ----

From
Shuxin<EMAIL_ADDRESS>

Date
06/22/2022 18:34

To

Cc
、

Subject
Re: [microsoft/Graphormer] ImportError: No module named setuptools (Issue #125)

Try sudo apt-get install python3-setuptools.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Then I guess it's not the issue from Graphormer but your python program can not found where you install setup-tools. Maybe you can raise an issue here  to find some help.

Thank you so much!


Unable to meet conditions  ""

update  value=""    throw Uncaught SyntaxError: Unexpected token ')'

Thank you for your contribution~


Improve Styling of Search Button - Unify CSS for Consistency ✨

Issue Description:
The current styling of the search button on our website lacks consistency. We have noticed that there are two search buttons, and their CSS styles are slightly different, leading to an inconsistent user experience. This issue aims to unify the CSS for both search buttons to ensure a cohesive and polished look throughout the website.
Expected Behavior:
When users encounter the search buttons on our website, they should experience a consistent and visually appealing design. Both search buttons should have the same CSS styles applied, regardless of their placement on the website.
Steps to Reproduce:

Navigate to the website's homepage.
Observe the search button in the header.
Perform a search operation and notice the search button in the search bar.

Current Behavior:
As of now, the two search buttons appear with slightly different CSS styles. The header search button might have different padding, font size, or colors compared to the search button in the search bar.
Proposed Changes:
To enhance the user experience and create a cohesive design, we propose the following changes:

Review the existing CSS for both search buttons and identify the differences.
Unify the CSS styles for both search buttons, ensuring that they share the same font, size, color, and padding.
Implement the updated CSS to create a consistent look across the website.

Hey @Anupkjha2601. Please assign me this issue


Fails to pick up Dark Mode in Npp

Bug:
Plugin fails to apply style-dark.css when dark mode is enabled in Npp (v8.2, 32 bit)
As a workaround I just set the default stylesheet to the dark one, but it isn't detecting the switch within Npp. I'll post back after a PC restart, maybe that will fix it.
Win10 v1709

1709 has been dead for almost 3y ...

The plugin detects the dark mode with an API which was added in version 8.4.1 of NPP.
See here: https://npp-user-manual.org/docs/plugin-communication/#nppm-isdarkmodeenabled
Means dark mode support doesn't work with version 8.2
Is it possible for you to upgrade you npp version?

Is it possible for you to upgrade your npp version?
Done! All working perfectly now! Thank you! Would it be possible to add a note to the Readme, e.g. "automatic dark/light mode switching supported in NPP v8.4.1+"?

Done! All working perfectly now! Thank you!
Great!
Would it be possible to add a note to the Readme, e.g. "automatic dark/light mode switching supported in NPP v8.4.1+, earlier versions require manual switching or setting style-dark.css as default"?
Sure I will add a note to the Readme.


Why the model generates images with 'pink filter' after training in a relatively large dataset?

Thanks for the great work! I met a strange problem when I try to train a model based on IP-adapter-base-sdxl. Specifically, the input data will contain 1~4 images and is all padded to 4. The model performs well when training with ~10000 samples, but generates images with 'pink filter' when training with ~100000 samples. I wonder if you have met this issue before or know what is happening in this situation. Looking forward to your reply!

I haven't encountered similar problems. Maybe the size of training dataset is too small?

What are your training parameters


Create 2022-04-01 API

Which issue this PR addresses:
Work item: 12693000
What this PR does / why we need it:
The new API version to include new stable features directly (former feature flags)

L-series
Pull back OVN
FIPS

Test plan for issue:

existing unit tests and e2e
upstream Azure API tests

Is there any documentation that needs to be updated for this PR?

@m1kola would you mind taking a quick glance over this as well and seeing if you see anything wrong?  Would like to push up a PR to azure-rest-api-specs by the end of the week to get their feedback on this, so trying to merge this in shortly.

Hi @25region I have few minor questions. Beside that I believe we are good to go.


Exit as an error so others programs can catch it

The goal is to be able to catch invalid args as an error in automated scripts and programs.

Fair enough

Hi, I'm impressed by the reactivity ^^, quick question, how long until we get it on pip ?

Can't find comment so just commenting with image for reference.
I've already pushed it to pip, that's why I bumped the version earlier after your merge of PR.

 My bad ^^

Ah lol


Update instructions for MOM6 data

The instructions presume that some of the more fundamental MOM6 input data for the OM4_025 configuration were pre-downloaded on disk (note this is separate from the data that is archived on zenodo). Currently the .datasets symlink points to a certain user (me) who no longer has access to the system.
We need to update the instructions to explain how to download and link the inputs on a brand new platform.

@Spartee how can I help with this?


Merge feature-branches/forms into v.next

Related to issue: #kotlin/5139

Description:
Merge the latest feature form development into v.next. Looks like the commit history below is not accurate. After the merge, I will purge this branch and start fresh.
Summary of changes:

Some design updates have been added to the component in line with the new UI/UX design. Check the issue above for links to the design document.
The feature forms micro-app now has the ability to create point features.
Fixed lint errors regarding translation in the strings.xml.

Pre-merge Checklist

a vTest Job for this PR has been run

[x] link: https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/259/

Unit and/or integration tests have been added to exercise this PR's logic, and the tests are passing:

[ ] Yes
[ ] No

The vtest seems to be failing due to a different component.


Fixes build break

@gauntface - this backed out the svg-minified changes as the svg-minified directory wasn't checked in. Feel free to revert once it's checked in.


Automatic npm publish action

Refer https://github.com/marketplace/actions/publish-to-npm. Adding their instruction to this PR to make it easier for you.
When someone changes the version in package.json to 1.2.3 and pushes a commit to master with the message Release 1.2.3, the npm-publish action will create a new tag v1.2.3 and publish the package to the npm registry.
Inputs
These inputs are optional: that means that if you don't enter them, default values will be used and it'll work just fine.
tag_name: the name pattern of the new tag
tag_message: the message pattern of the new tag
commit_pattern: the pattern that the commit message needs to follow
workspace: custom workspace directory that contains the package.json file
Environment variables
GITHUB_TOKEN: this is a token that GitHub generates automatically, you only need to pass it to the action as in the example
NPM_AUTH_TOKEN: this is the token the action will use to authenticate to npm. You need to generate one in npm, then you can add it to your secrets (settings -> secrets) so that it can be passed to the action. DO NOT put the token directly in your workflow file.

Thanks!


Download functionality

Hi This is great repo. Could you please add download functionality if possible?

i don't think thats possible at the moment

but i will try to add this feature


[Bug]: [Weekly]Some operators failed reporting 'Torch not compiled with CUDA enabled'

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Some operators failed reporting 'Torch not compiled with CUDA enabled'.

link: https://github.com/towhee-io/towhee/actions/runs/4088900552/jobs/7126498094#step:5:8610
operator https://towhee.io/video-text-embedding/clip4clip
error message:

2023-02-08T16:20:40.8695357Z 2023-02-08 16:20:40.868 | DEBUG    | __main__:execute_ops:163 - Running extracted readme for /workspace/towhee_CI/hub_operator/video-text-embedding-clip4clip
2023-02-08T16:20:43.7598247Z Cloning into '/github/home/.towhee/operators/video-text-embedding/clip4clip/main'...
2023-02-08T16:22:23.0863297Z Traceback (most recent call last):
2023-02-08T16:22:23.0869729Z   File "/workspace/towhee_CI/hub_operator/testcases/test_video-text-embedding-clip4clip.py", line 4, in <module>
2023-02-08T16:22:23.0871021Z     pipe.input('text') \
2023-02-08T16:22:23.0872314Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/pipeline.py", line 113, in output
2023-02-08T16:22:23.0873233Z     run_pipe.preload()
2023-02-08T16:22:23.0874562Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/runtime_pipeline.py", line 102, in preload
2023-02-08T16:22:23.0875866Z     _ = Graph(self._dag_repr.nodes, self._dag_repr.edges, self._operator_pool, self._thread_pool)
2023-02-08T16:22:23.0877415Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/runtime_pipeline.py", line 46, in __init__
2023-02-08T16:22:23.0878348Z     self.initialize()
2023-02-08T16:22:23.0879621Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/runtime_pipeline.py", line 56, in initialize
2023-02-08T16:22:23.0880599Z     raise RuntimeError(node.err_msg)
2023-02-08T16:22:23.0883353Z RuntimeError: Node-video-text-embedding/clip4clip runs failed, error msg: Create video-text-embedding/clip4clip operator video-text-embedding/clip4clip:main with args None and kws {'model_name': 'clip_vit_b32', 'modality': 'text', 'device': 'cuda:1'} failed, err: Torch not compiled with CUDA enabled, Traceback (most recent call last):
2023-02-08T16:22:23.0885794Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/nodes/node.py", line 81, in initialize
2023-02-08T16:22:23.0886782Z     self._op = self._op_pool.acquire_op(
2023-02-08T16:22:23.0888211Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/operator_manager/operator_pool.py", line 99, in acquire_op
2023-02-08T16:22:23.0889423Z     op = self._op_loader.load_operator(hub_op_id, op_args, op_kws, tag)
2023-02-08T16:22:23.0890987Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/operator_manager/operator_loader.py", line 153, in load_operator
2023-02-08T16:22:23.0892048Z     op = factory(function, arg, kws, tag)
2023-02-08T16:22:23.0893597Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/operator_manager/operator_loader.py", line 132, in load_operator_from_cache
2023-02-08T16:22:23.0894793Z     return self.load_operator_from_path(path, arg, kws)
2023-02-08T16:22:23.0896367Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/operator_manager/operator_loader.py", line 118, in load_operator_from_path
2023-02-08T16:22:23.0898439Z     return self.instance_operator(op, arg, kws) if op is not None else None
2023-02-08T16:22:23.0900074Z   File "/opt/conda/lib/python3.8/site-packages/towhee/runtime/operator_manager/operator_loader.py", line 161, in instance_operator
2023-02-08T16:22:23.0901222Z     return op(*arg, **kws) if kws is not None else op(*arg)
2023-02-08T16:22:23.0902676Z   File "/github/home/.towhee/operators/video-text-embedding/clip4clip/main/__init__.py", line 19, in clip4clip
2023-02-08T16:22:23.0903914Z     return CLIP4Clip(model_name, modality, **kwargs)
2023-02-08T16:22:23.0905372Z   File "/github/home/.towhee/operators/video-text-embedding/clip4clip/main/clip4clip.py", line 45, in __init__
2023-02-08T16:22:23.0906562Z     self.model = clip4clip.create_model(model_name=model_name,
2023-02-08T16:22:23.0908049Z   File "/opt/conda/lib/python3.8/site-packages/towhee/models/clip4clip/clip4clip.py", line 210, in create_model
2023-02-08T16:22:23.0908983Z     model.to(device)
2023-02-08T16:22:23.0910170Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 987, in to
2023-02-08T16:22:23.0911088Z     return self._apply(convert)
2023-02-08T16:22:23.0912321Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 639, in _apply
2023-02-08T16:22:23.0913204Z     module._apply(fn)
2023-02-08T16:22:23.0914397Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 639, in _apply
2023-02-08T16:22:23.0915238Z     module._apply(fn)
2023-02-08T16:22:23.0916404Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 639, in _apply
2023-02-08T16:22:23.0917231Z     module._apply(fn)
2023-02-08T16:22:23.0918397Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 662, in _apply
2023-02-08T16:22:23.0919284Z     param_applied = fn(param)
2023-02-08T16:22:23.0920535Z   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 985, in convert
2023-02-08T16:22:23.0921746Z     return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None, non_blocking)
2023-02-08T16:22:23.0923183Z   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/__init__.py", line 221, in _lazy_init
2023-02-08T16:22:23.0924246Z     raise AssertionError("Torch not compiled with CUDA enabled")
2023-02-08T16:22:23.0925186Z AssertionError: Torch not compiled with CUDA enabled
2023-02-08T16:22:23.0925713Z
2023-02-08T16:22:23.3111063Z 2023-02-08 16:22:23.309 | ERROR    | __main__:execute_ops:167 - Fail to run readme for 37: operator https://towhee.io/video-text-embedding/clip4clip

Expected Behavior
pass
Steps To Reproduce
No response
Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS):
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

Can you look at the error? @zc277584121
Or do we need to add any configuration to the script?

Same error: https://towhee.io/video-text-embedding/drl

I have reinstall torch(1.13.1 fit for CUDA 11.7)

@NicoYuan1986
any update here?


Ensure Azure.AKS.CNISubnetSize uses CNI selector

PR Summary
Small fix. Added Azure.AKS.Azure.CNI selector to Azure.AKS.CNISubnetSize rule. Makes sense to make sure this rule just checks CNI clusters.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[ ] Unit tests created/ updated
[ ] Rule documentation created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Other code changes

[x] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Acutally I might need to modify the Azure CNI selector to also handle Microsoft.ContainerService/managedClusters/agentPools.

Is it possible to do where clauses in selectors? Wondering if I could do a where clause to check if agentpool subresources have a subnet resource.
---
# Synopsis: AKS clusters that use Azure CNI network plugin.
apiVersion: github.com/microsoft/PSRule/v1
kind: Selector
metadata:
  name: Azure.AKS.AzureCNI
spec:
  if:
    anyOf:
    - allOf:
      - type: '.'
        equals: 'Microsoft.ContainerService/managedClusters'
      - field: 'Properties.networkProfile.networkPlugin'
        equals: 'azure'
    - type: '.'
      equals: 'Microsoft.ContainerService/managedClusters/agentPools'

Actually do we even need to handle Microsoft.ContainerService/managedClusters/agentPools for this rule? Export-AzRuleTemplateData doesn't export agent pools as a separate resource. Seems to just keep them intact with in agentPoolProfiles of the AKS cluster resource.

@ArmaanMcleod Makes sense to me. It's a bug. Let's log this and add a change log so if anyone encounters this issue they know it's been fixed.

No worries, I can link an issue with this.
I've also noticed Export-AzRuleTemplateData doesn't include Microsoft.ContainerService/managedClusters/agentPools resources, which this change would affect as the selector only checks for Microsoft.ContainerService/managedClusters. Do you think that would be a problem?

/azp run

@ArmaanMcleod One cleanup suggestion. Otherwise all good to merge.

Thanks @BernieWhite. Have removed the type filter.


Y24-081 Add Program name field to MLWH

As team lead Sequencing Operations (James M) we would like to add the “name” field from the “Programs” table in the Sequencescape database to the MLWH. This will enable us to identify studies by Sanger programme type and enable efficient reporting.
Contact
jm36
Raised in RT802591

@TWJW-SANGER @stevieing this was marked as low priority in the requirement sheet Seq80 but can be pulled ahead if a quick win.

@sabrine33 the column name should be 'programme' see here https://www.sanger.ac.uk/science/programmes/
The 'program' in Sequencescape is incorrect but no one can see it, if we expose it incorrectly in MLWH I'm sure it will not be long before someone points out the error.

@sabrine33 the column name should be 'programme' see here https://www.sanger.ac.uk/science/programmes/ The 'program' in Sequencescape is incorrect but no one can see it, if we expose it incorrectly in MLWH I'm sure it will not be long before someone points out the error.

Thank you @neilsycamore for highlighting this, I will update it accordingly.


Automated cherry pick of #11216

Cherry pick of #11216 on release-7.4.
/cc  @nevyangelova
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 93c2555019348f00d56982d5c3c73e5be86b360c


Completed all the tasks for Homework 1 and 2 (Reading user input & BMI calculated and second person)

The program can read a user's input which includes the following:

The user's first name.
The user's last name.
The user's age, weight and height

This will then be printed on the console as:
"John Smith is 29 years old, their weight is 90 and their height is 189"
The second task consisted of calculating the BMI index using the user's input which has also been done.
The BMI will be then printed as follows:
"John, your BMI value is [BMI Value]"

I have realised that I have not repeated it for one other person. Will get that done by the end of today.

Task 3 has been done where the process needs to happen twice for another user


Lang Validation updates & SettingName Period issue

List supported language options in validation set. Add handling of periods in SettingName

Tested use of Language options & SettingName regex transformation


merge current_machine_options into config in connect_to_machine

I'm not sure if this is the right thing to do.
In testing the machine_file resource with chef_metal_vsphere 0.3.0.beta.1, the config parameter passed to the driver's connect_to_machine does not contain a machine_options hash.  Without the machine_options, I'm not sure where to obtain ssh_config for the machine.
#am_i_doing_it_right

I'm going to close this since the original patch isn't what we want to do.  The question of transferring passwords is open, but at the moment the answer is "transfer encrypted files, data bags, or any other secure mechanism you have; chef-provisioning will let you specify them in code on the other end, so you can use any storage Ruby can read."


only push temporaries that can be live at a gc checkpoint

I based this PR on top of the records one, so it should be merged only after that one, as both are pretty invasive on the code generator.
This PR implements an optimization where we try to push GC-able temporaries only prior to intra-statement function calls (a possible GC point). The other GC points that we have, loops, do not count, as by definition temporaries are not live across statements. I am pretty confident that I have covered all possible bases so we will not have any dangling pointers, but of course I have not formally proven that the optimization is sound. 😈
This should really cut into the stack usage of code that uses complex expressions. The next step is to do liveness analysis on local variables, so a similar optimization can be done for them too, but this can go in another PR.
By the way, I recall that the Lua test suite compiles Lua with some flag where the GC is very aggressive so dangling pointers surface more easily, I think we should do something similar in our test suite so we can have effective GC tests.

Codecov Report

Merging #217 into record-objects will decrease coverage by 0.53%.
The diff coverage is 87.5%.

Impacted Files
Coverage Δ

titan-compiler/coder.lua
94.01% <87.5%> (-1.03%)
:arrow_down:

titan-compiler/util.lua
61.81% <0%> (-7.28%)
:arrow_down:

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 7a26601...57e9404. Read the comment docs.

This branch got really broken, as it should just have the last two commits instead of duplicates of all the record-related commits. I have no intention to merge this for the time being, but it is a good idea to fix it so I can keep it tracking the master branch.

@mascarenhas Rebased the PR!

Thanks @hishamhm!

I really need to make a habit of using git rebase -i instead of just rebase, would have saved me an hour of headaches when updating this branch.

If it gets too troublesome to keep making this track master I will just drop this, we can do this optimization properly once we have a language that is actually useful and pleasant to program in so we will have time to focus solely on improving the code generator.

git rebase -i is your best friend when doing rebases.
Other tips:

when experimenting, create another branch and mess with it until happy. you can use git cherry-pick to pick commits one by one (or replace one branch with another entirely with git reset --hard (careful!))
when things get out of hand, and you realize you're probably making a mess during the rebase, use git rebase --abort to start over.
gitk really helps visualizing how things are laid out, and navigating history — a very underrated tool
if you did something you regret, git reflog allows you to go back to any previous state of your git tree, and git reset --hard (careful! :) ) reverts a branch to those commits. This is the real "undo" in git.

Thanks for the tips. This tool looks very interesting, is free for open source projects, and seems easy to write Lua backend, but the current version if Windows-only (they are working on Linux/OSX versions, though).


Getting instance index in parallel mode

Is your feature request related to a problem or a nice-to-have?? Please describe.
– I wish I had a way to get instance index in parallel mode, I will be able to use eth-gas-reporter in parallel mode then: https://github.com/cgewecke/hardhat-gas-reporter#parallelization
Describe the solution you'd like
– I want to have any way to get something like parallelMochaIndex() returning numbers from 0 to N-1.
Describe alternatives you've considered
– I am not sure what can be done to implement it easily.

Could be related: https://github.com/mochajs/mocha/pull/4813
Will try to check process.env.MOCHA_WORKER_ID

process.env.MOCHA_WORKER_ID

Yes, that's what's meant to be used 🙂


Merge mxbox into gluon.data

Refer to https://github.com/apache/incubator-mxnet/issues/7376#issuecomment-330006540
What I did:

Add transform utils to gluon.data.

Since transform handles PIL image by default, dependency Pillow is added to setup.py

Split vision dataset into separate files.

What are left:

An efficient multi-process loader
I notice there is already one single process loader. MXBox has a multi-thread loader, however due to the GIL of python, multi-thread does not help much when loading large image. Meanwhile, fetching large data from queue will cause overhead in multi-process. The best solution I have seen so far is the pin_memory in pytorch.Tensor -- which passes a reference between processes instead of copying.

CIFAR100 / SVHN  more dataesets

CIFAR100 is being added in #7892

closing due to inactive. Please see the vision branch.


How to use EslintFixAll command as formatter?

I tried a custom formatter but it didn't work

Is there any difference in running cmd and using eslint executable?
Conform expects command to be executable. E.g. eslint_d. If you don't have eslint_d, creating eslint should be straightforward.
If you still want to use EslintFixAll take a look at LazyVim eslint lsp configuration. With lsp_fallback enabled and this lsp configuration formatting should work.

Commands are not supported as formatters, due to the fact that they have a radically different interface. See #193 for more

Hi, don't completely understand what @fbuchlak is saying - this is all new to me. But is there an EslintFixAll equivalent  using conform?


Added LR tile instructions, other updates to README

The quick starts README previously didn't have any info about creating the YAML files for LR tiles (links to docs, Learning Paths, etc). This PR contains:

new Learning resource tiles instructions (HCCDOC-1859)
fixed broken PatternFly links, and corrections from 'Patternfly' to 'PatternFly' (HCCDOC-1950)
rename of 'Application & Data Services' to remove 'Data'
fixed typo on Ansible LR tile that I stumbled across

/ok-to-test


Trouble with Step 11: Run The Notebook

Hello, I hope someone can help me with this issue. Someone had this issue previously, but there was no follow-up on it.
I entered my Websocket URL into the notebook, ran each cell in order from top to bottom, but I received this error:
DEBUG:websocket:-----------------------
send: b'\x81\x97\xe7<4\xdd\x9c\x1eW\xb0\x83\x1e\x0e\xff\xa3ol\xfd\xafYU\xaf\x93~Q\xbc\x93\x1eI'
DEBUG:websocket:send: b'\x81\x97\xe7<4\xdd\x9c\x1eW\xb0\x83\x1e\x0e\xff\xa3ol\xfd\xafYU\xaf\x93~Q\xbc\x93\x1eI'
error from callback <function on_message at 0x7f4ac4764620>: 'botresponse'
ERROR:websocket:error from callback <function on_message at 0x7f4ac4764620>: 'botresponse'
File "/home/spark/shared/user-libs/python3.6/websocket/_app.py", line 346, in _callback
callback(self, *args)
File "", line 7, in on_message
for item in msg["botresponse"]["messageout"]["context"]["entities_array"]:
Unhandled exception in thread started by <function on_open..run at 0x7f4a941dbc80>
I suspect I will run into several more issues after this one is solved, so thank you in advance for the help!

It's been a while since I last interacted with this, but my advice would be to print out the full response from the websocket, if possible.
This way you can check if the error comes from structure of the msg being different for some reason -- and it can be fixed by changing msg["botresponse"]["messageout"]["context"]["entities_array"] accordingly.

How would I print the response? I am not great at Python yet.

Please start the Websocket listener first by running the last cell in the notebook and then start interacting with the UI.

@RK-Sharath  I do not know which UI you are referring to. When all the cells are run, this happens:

@Quiescitus  I have attempted to print the response using a few different methods, but none gave me results I needed. Please let me know if there is a specific way I am supposed to do it. If I cannot print the response, is it possible this information is on the Node-Red site somewhere?


fix(#890): add migrations in config only when wanted

Closes #890

@marcelom97 Can you add a test that doesn't create a migration option?

@ivan-tymoshenko for sure!!
In the issue there is also mentioned a validation for the empty dir attribute inside migrations config
Should I also add this validation here?
And if yes could you help me locate where the config schema is created?

Here is a migration dir schema, but you don't need it:
https://github.com/platformatic/platformatic/blob/4b1165e3b04af0421439db7b45f7a10c2a92e655/packages/db/lib/schema.js#L309
What you need is to add an empty string check to the resolvePath ajv keyword:
https://github.com/platformatic/platformatic/blob/4b1165e3b04af0421439db7b45f7a10c2a92e655/packages/config/index.js#L176

@ivan-tymoshenko Thanks!!


latch register not working with Docker update

[REQUIRED] Describe the bug
Latch workflow registration is failing due to Docker configuration.
[REQUIRED] To Reproduce
Seeing this on macOS with current Docker, configured to run either at /usr/local/bin (system) or ~/.docker/bin/docker (user).
latch --version
# latch, version 2.19.6
docker --version
# Docker version 23.0.5, build bc4487a
latch register fetch-and-run

❯ latch register fetch-and-run
Unable to register workflow. - printing traceback:

/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/connectionpool.py:703 in urlopen
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/connectionpool.py:398 in _make_request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1283 in request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1329 in _send_request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1278 in endheaders
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1038 in _send_output
5 frames collapsed...

--Traceback (most recent call last)--
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:976 in send
  974|         if self.sock is None:
  975|             if self.auto_open:
> 976|                 self.connect()
  977|             else:
  978|                 raise NotConnected()
  979|
  980|         if self.debuglevel > 0:

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/transport/unixconn.py:30 in connect
  28|         sock = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
  29|         sock.settimeout(self.timeout)
> 30|         sock.connect(self.unix_socket)
  31|         self.sock = sock
  32|
  33|     def putheader(self, header, *values):
  34|         super().putheader(header, *values)

FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/requests/adapters.py:486 in send
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/connectionpool.py:787 in urlopen
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/util/retry.py:550 in increment
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/packages/six.py:769 in reraise
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/connectionpool.py:703 in urlopen
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/urllib3/connectionpool.py:398 in _make_request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1283 in request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1329 in _send_request
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1278 in endheaders
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:1038 in _send_output
9 frames collapsed...

--Traceback (most recent call last)--
/opt/koopa/app/python3.10/3.10.11/lib/python3.10/http/client.py:976 in send
  974|         if self.sock is None:
  975|             if self.auto_open:
> 976|                 self.connect()
  977|             else:
  978|                 raise NotConnected()
  979|
  980|         if self.debuglevel > 0:

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/transport/unixconn.py:30 in connect
  28|         sock = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
  29|         sock.settimeout(self.timeout)
> 30|         sock.connect(self.unix_socket)
  31|         self.sock = sock
  32|
  33|     def putheader(self, header, *values):
  34|         super().putheader(header, *values)

ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/api/client.py:214 in _retrieve_server_version
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/api/daemon.py:181 in version
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/utils/decorators.py:46 in inner
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/api/client.py:237 in _get
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/requests/sessions.py:600 in get
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/requests/sessions.py:587 in request
5 frames collapsed...

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/requests/sessions.py:701 in send
  699|
  700|         # Send the request
> 701|         r = adapter.send(request, **kwargs)
  702|
  703|         # Total elapsed time of the request (approximately)
  704|         elapsed = preferred_clock() - start
  705|         r.elapsed = timedelta(seconds=elapsed)

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/requests/adapters.py:501 in send
  499|
  500|         except (ProtocolError, OSError) as err:
> 501|             raise ConnectionError(err, request=request)
  502|
  503|         except MaxRetryError as e:
  504|             if isinstance(e.reason, ConnectTimeoutError):
  505|                 # TODO: Remove this in 3.0.0: see #2811

ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/centromere/utils.py:162 in _construct_dkr_client

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/api/client.py:197 in __init__
  195|                                 str
  196|                                 ) and version.lower() == 'auto'):
> 197|             self._version = self._retrieve_server_version()
  198|         else:
  199|             self._version = version
  200|         if not isinstance(self._version, str):
  201|             raise DockerException(

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/docker/api/client.py:221 in _retrieve_server_version
  219|             )
  220|         except Exception as e:
> 221|             raise DockerException(
  222|                 f'Error while fetching server API version: {e}'
  223|             )
  224|
  225|     def _set_request_timeout(self, kwargs):

DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

/opt/koopa/app/latch/2.19.6/libexec/bin/latch:8 in <module>
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/click/core.py:1130 in __call__
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/click/core.py:1055 in main
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/click/core.py:1657 in invoke
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/click/core.py:1404 in invoke
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/click/core.py:760 in invoke
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/main.py:106 in register
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/services/register/register.py:235 in register
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/centromere/ctx.py:155 in __init__
8 frames collapsed...

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/centromere/ctx.py:152 in __init__
  150|                 )
  151|             else:
> 152|                 self.dkr_client = _construct_dkr_client()
  153|         except Exception as e:
  154|             self.cleanup()
  155|             raise e
  156|

--Traceback (most recent call last)--
/opt/koopa/app/latch/2.19.6/libexec/lib/python3.10/site-packages/latch_cli/centromere/utils.py:164 in _construct_dkr_client
  162|             return docker.APIClient(base_url="unix://var/run/docker.sock")
  163|         except docker.errors.DockerException as de:
> 164|             raise OSError(
  165|                 "Docker is not running. Make sure that"
  166|                 " Docker is running before attempting to register a workflow."
  167|             ) from de
  168|

OSError: Docker is not running. Make sure that Docker is running before attempting to register a workflow.

>> Crash report written to .latch_report.tar.gz <<
[1]    68772 segmentation fault  latch register fetch-and-run

[REQUIRED] Expected behavior
A clear and concise description of what you expected to happen.
Workflow should register successfully -- this worked with previous versions of the SDK.
[REQUIRED] Platform Information
General information about host platform + SDK

OS: macOS Ventura 13.3.1 (a)
Browser: NA
Latch Version: 2.19.6

Best,
Mike

Yeah this is a Docker issue with 23.0.5 not working by default with Python. Changing these settings fixes latch register for me:

In Docker Desktop > Settings > Advanced

Set to System, which will link Docker at /usr/local/bin/docker.
Check "Enable default Docker socket".
Check "Enable privileged port mapping".


mocha -h doesn't always display help

Pretty straightforward testcase:
zarel@Phenylalanine ~> mocha -h

  Usage: mocha [debug] [options] [files]

  [...]

zarel@Phenylalanine ~> mkdir test
zarel@Phenylalanine ~> echo "test" > test/mocha.opts
zarel@Phenylalanine ~> mocha -h
No test files found

In other words, if test/mocha.opts exists, mocha -h won't display help.

There is a pull request for a fix. The only reason it hasn't been merged yet is because Node 0.10 doesn't support the method used to test it.


[CodeGen] Clean up header includes and build dependencies

This commit cleans up some header includes and build dependencies:

Created a new target for VectorLayoutAnalysis
Dropped Codegen/Common and Codegen/Utils dependencies from IREEGPU dialect
Removed various header includes not directly used

This is a preliminary step for plumbing through a target description, to be defined in the IREEGPU dialect, for various GPU backends.

@Groverkss Good points regarding IREEGPUDialect should not depend on VectorLayoutAnalysis. I dropped the connection there. (Earlier wasn't looking into it closely.)  Regarding the separate targets, I think it's good to sperate--typically we have separate targets for analysis vs. passes in mlir.


Added simple REST-API

Added rest api on:
GET:

/locations
/reports
POST:
/add-person

This is a good foundation... we can modify the flask app end point once I group all regions together.


JWT Auth

Hi, thanks for the example!
is it possible to switch authentication from JSON auth to JWT? maybe using lexik/LexikJWTAuthenticationBundle?
Thanks

Hello @abelland!
Actually, we do not like JWT so much for various reasons.
Our method is also simplier 😄


Problem trying to create workflow on Centos7

While developing some actors, I am having some issues trying to follow the documentation about Working with Workflows. All this is being executed in a CentOS7 machine, with EPEL enabled and virtualized with Vagrant box "centos/7" and libvirt.
By default, CentOS7 has this version of Python2.7
$ /usr/bin/python2.7 --version
Python 2.7.5

Following the steps on documentation
$ cd devel/
[devel]$ virtualenv -p /usr/bin/python2.7 tut-2.7.5
(...)
[devel]$ . tut-2.7.5/bin/activate
(tut-2.7.5)[devel]$ pip install git+https://github.com/leapp-to/leapp
(...)
(tut-2.7.5)[devel]$ snactor new-project tutorial-2.7.5
New project tutorial-2.7.5 has been created in /home/vagrant/devel/tutorial-2.7.5
(tut-2.7.5)[devel]$ cd tutorial-2.7.5/
(tut-2.7.5)[tutorial-2.7.5]$ snactor workflow new Example
(tut-2.7.5)[tutorial-2.7.5]$ ls -la
total 0
drwxrwxr-x. 3 vagrant vagrant 20 May  5 02:40 .
drwxrwxr-x. 4 vagrant vagrant 45 May  5 02:40 ..
drwxrwxr-x. 2 vagrant vagrant 36 May  5 02:40 .leapp
(tut-2.7.5)[tutorial-2.7.5]$ python --version
Python 2.7.5
(tut-2.7.5)[tutorial-2.7.5]$ deactivate

Nothing seems to happen. Since I am able to create workflows in Fedora 27, my best guess is that is a problem with Python 2.7.5. So, I built Python 2.7.14 from source on the same CentOS7 machine
$ /usr/local/bin/python2.7 --version
Python 2.7.14

And know running the same steps from documentation
$ cd devel/
[devel]$ virtualenv -p /usr/local/bin/python2.7 tut-2.7.14
(...)
[devel]$ . tut-2.7.14/bin/activate
(tut-2.7.14)[devel]$ pip install git+https://github.com/leapp-to/leapp
(...)
(tut-2.7.14)[devel]$ snactor new-project tutorial-2.7.14
New project tutorial-2.7.14 has been created in /home/vagrant/devel/tutorial-2.7.14
(tut-2.7.14)[devel]$ cd tutorial-2.7.14/
(tut-2.7.14)[tutorial-2.7.14]$ snactor workflow new Example
New tag ExampleWorkflow has been created in /home/vagrant/devel/tutorial-2.7.14/tags/exampleworkflow.py
New workflow Example has been created in /home/vagrant/devel/tutorial-2.7.14/workflows/example.py
(tut-2.7.14)[tutorial-2.7.14]$ ls -la
total 0
drwxrwxr-x. 5 vagrant vagrant 49 May  5 02:48 .
drwxrwxr-x. 6 vagrant vagrant 86 May  5 02:48 ..
drwxrwxr-x. 2 vagrant vagrant 36 May  5 02:48 .leapp
drwxrwxr-x. 2 vagrant vagrant 32 May  5 02:48 tags
drwxrwxr-x. 2 vagrant vagrant 24 May  5 02:48 workflows
(tut-2.7.14)[tutorial-2.7.14]$ python --version
Python 2.7.14
(tut-2.7.14)[tutorial-2.7.14]$ deactivate

It seems to work as expected.
It is not clear to me how much of a impact this would have in real world, if it is just a problem about creating a workflow or if it also impacts running them on old versions of Python2.7. But, if it is in fact a problem with Python version, we could mention that on documentation.

Thanks for this awesome report again - It got addressed in https://github.com/leapp-to/leapp/commit/bc6963eef0d02b248da84b632dd4e9c38b06e03f and should be fixed now - If you find any other problems please let me know :-)


fix(Global): clean some codes

I don't understand the motivation for merge the PR. Why do you think the changes are so important?

I just remove some warnings and think this is looks better
I see you changed some codes to return If

I prefer not to use return with if expressions.

If you don't like I won't force you


AASX Template of Submodel Provision of 3D Models Does not match Spec

Hi when trying to implement the submodel template i was confused by submodel Element collections with strange names.
When reading the pdf i noticed the template does not match the spec and the spec is not deterministic.
For example Figure 6 implies that the SML Model3D contains an unnamed SMC that contains the SMCs File,Capability and Geometry.
Figure 7 and 8 are missing this unnamed SMC.
The Content of table 3 which goes into the details of SML Model3D also specifies that File, Capability and Geometry should be contained directly inside of the SML Model3D.
But the Provided Template contains this ominous unnamed SMC.
I hope this helps
have a great day
See pictures Provided:

One solution would be to make Model3 a SMC in the first place. Almost all SMLs in this submodel should be infact SMCs, i really dont understand why they arent and why any [nn] exists at all.

I dug a little deeper and made a proposal in #86


Fixes errors in MEP URIs

Hi,
Apparently the URIs used for the Exchange Patterns were wrong in two different ways:

First opt was wrongly worded optional (see http://www.w3.org/TR/wsdl20-additional-meps/).
Second there was an underscore instead of a dash in out-opt-in.

I don't think it will have much impact, except maybe with components such as servicemix's nmr that uses the method fromWsdlUri() to set the pattern of exchanges when converting from their classes to camel's.
Thanks!

Thanks for this PR. The code has been merged. Do you mind closing this.

Thanks!


Function Calls [WIP]

Depends on: https://go-review.googlesource.com/c/go/+/109699

The corresponding CL has been merged, this now only depends on the go1.11 support branch (#1157) and on #1208 (which needs some decisions to be made).
This implements the function call injection protocol on linux only and doesn't support a number of features that one would reasonably expect (see the TODO comments in pkg/proc/fncall.go for a list).

I think this is a great start, but we should make a note in the docs about current shortcomings.

Can't pass in new vars (e.g. call myfn(1,2))
Can't call methods
Arch / OS limitations (this only applies to native IIRC, correct me if I'm wrong)
etc...

Where do you think is the appropriate place for this? The docstring of call?

Done.


Package Information

The idea for this package is to be the main team package for Fountain
This package should in theory:

Allow multiple teams per user
Invitation system (invite, accept, decline)
Switching Team
Team member list
Supporting views for member management, invites, etc.

I would appreciate feedback on the idea and implementation of this package.


Documentation

Is there documentation on how to connect Form SG to external applications using this SDK via webhooks? Where in Form SG to configure for the submissions to point to the external application using this Form SG JavaScript SDK?

hi @yhrchan, you can add your webhook URL under the "Settings" tab in your form's admin dashboard

Thanks for the quick response. I am testing this locally on my machine but I don't see a field to fill in the webhook URL under the "Settings" tab in the form dashboard.

I see the source code has this field, is there anything special that needs to be configured in order for the webhook url field to show up or to enable this feature?

        Webhook Endpoint URL
         beta

        (optional)

@yhrchan out of curiosity, can I also check what you're using webhooks for?

@mantariksh ah thanks! the webhook url field showed up now so i'll try creating the webhook as a next step. BTW, I am not fluent in Angular so sometimes hard for me to follow the source code. I am only fluent in React but I see in your roadmap that your team is planning to migrate to React, right?
Yes I am building a CRM-like system and see the possibility to integrate FormSG as an intake component and within the CRM different workflows could be configured for downstream functionalities. Just curious, would there be a possibility to meet online to have some overview of FormSG from your perspective? I am sure I have some questions to ask in terms of future Roadmap and architecture.
Thanks for your help! =)

@mantariksh I've tried putting in an address in the webhook url field such as http://localhost:3003 but the field validation won't let me. Is there any way to test the webhook where the webhook resides locally without HTTPS?

@mantariksh so I was able to use ngrok to expose my local server publicly via a HTTPS address and seems like FormSG could establish connection to my local webhook. However, some config might not be right as I am getting a 401 Unauthorized response.
webhook source code
const POST_URI =https://address.ngrok.io/submissions
Webhook Endpoint URL
https://address.ngrok.io/submissions
Also in the webhook code, i've entered in the form secret key in the following line:
const formSecretKey = xxxxxxxxxxxxxxxxxxxxxx
HTTP Requests
POST /submissions              401 Unauthorized

@mantariksh
Ok got it to work!
Have to use set mode = test so that it uses the same public key in the webhook code
const formsg = require('@opengovsg/formsg-sdk')({
mode: 'test',
})

hey @yhrchan, sorry for the late response, we've been caught up with reviews on the main FormSG repo.
yes, we are planning to start the migration to React in the next couple of weeks actually. you can see the issues we've scoped out here.
sure, happy to meet online! I'm curious about your use case as well. feel free to drop me an email at<EMAIL_ADDRESS>and we can set it up.

@yhrchan we also introduced retries for webhooks in the main repo, if that's useful for you


Ad serving is limited

Ad serving is limited since 3 months. Google is telling invalid traffic concerns but all of my traffic are organic and come from Google Play only. I keep getting the same warning every 2 days in notification.
Not sure if this problem is due this package or my implementation. Did anyone else faced same issue?

I dont think its a plugin problem , every one those days can get admob to be limited , u need to be careful when implementing admob inside your app amd make a cpm target for your ad units at least 0.35$ (for banner ads u can leave it 0) , and frequency cap (show no more than 1 interstitial ad for user in one minute) , inside your admob account.
And use admob mediation if possible.
Following those steps i guess you can avoid admob limit.

this plugin and the official one by google have many issues needed to fixed and causes ad limit.
for example, in this plugin banner ads are still making ad requests even if the app is in background, which counted as invalid traffic according to admob policy

This is because of banner ads :(


Python requests not using host CA certificate bundle

When configuring OIDC SSO with an IdP that uses a self-signed certificate, the Python requests package does not use the host's CA certificate bundle.  See https://stackoverflow.com/questions/42982143/python-requests-how-to-use-system-ca-certificates-debian-ubuntu
To verify, the /etc/supervisor/conf.d/zulip/zulip.conf was modified to include the environment variable pointing to the host CA bundle:
...
[program:zulip-django]
command=nice -n5 /home/zulip/deployments/current/zulip-current-venv/bin/uwsgi --ini /etc/zulip/uwsgi.ini
environment=HTTP_proxy="",HTTPS_proxy="",REQUESTS_CA_BUNDLE="/etc/ssl/certs/ca-certificates.crt"
...

After reloading supervisor and restarting zulip, the SSO request succeeded.

I'm on the fence about defaulting to the system CA bundle -- requests includes its own CA bundle (by way of the certifi package) intentionally, as often system CA stores are in unusual places, or are infrequently updated.
But since we mostly control the OS, we know where the CA bundles live, and we can somewhat ensure that they're up-to-date using apt.  But I also don't see much advantage to using the OS bundles in most circumstances; for folks who need to support custom CAs, having some config option to provide an env override seems reasonable.
The [http_proxy] section doesn't seem right, though, since it affects outgoing requests even if you're not using a proxy.


adding a feature to customize the egress rule

what

Added 4 new variables egress_source_port, egress_dest_port, egress_protocol, and allowed_egress_cidr_blocks for the "aws_security_group_rule" "egress" resource.
By default, the egress rule <IP_ADDRESS>/0 will be created. If user is expected to restrict outbound traffic, they can specify the required values.

why

We propose this PR because there are scenarios where users may want to restrict outbound traffic from their DocDB instances.
if the DocDB cluster is only used internally and do not need to communicate with wide systems or network, users may want to customize the egress rule for <IP_ADDRESS>/0. By providing the option to customize the egress rule, we are giving users a control over their security posture (compliance). For example, our docdb may only connected with internal applications inside the aws eks cluster, or users may integrate their cloud resources with a third party, such as Prisma Cloud or maybe use tfsec as their security scanner, which prompts users to kindly avoid <IP_ADDRESS>/0 for security best practices.

references

This PR will help to cover the minimum egress exposure of the security group, including: #34

Thank you

/terratest

@milldr thanks Pal!


Feature/create d2 diagrams extension

Work in progress

@vsch I'm working on adding the support for d2 diagrams and am able to parse the ast correctly (D2Node should be 0 to 23, I'm going to fix it later)
Document[0, 23]
D2Block[0, 23]
D2Node[20, 23]
and this should render rendering d2 node but the render method of the D2NodeRenderer is not called even though the D2Node rendering is associated with the render method of the renderer.
public class D2NodeRenderer implements NodeRenderer {
public D2NodeRenderer(DataHolder options) {
}

@Override
public Set<NodeRenderingHandler<?>> getNodeRenderingHandlers() {
    HashSet<NodeRenderingHandler<?>> set = new HashSet<>();
    set.add(new NodeRenderingHandler<>(D2Node.class, this::render));
    return set;
}

private void render(D2Node node, NodeRendererContext context, HtmlWriter html) {
    html.tag("p");
    html.text("rendering d2 node");
    html.tag("/p");
}

public static class Factory implements NodeRendererFactory {
    @NotNull
    @Override
    public NodeRenderer apply(@NotNull DataHolder options) {
        return new D2NodeRenderer(options);
    }
}

}
I'm a bit at a loss here because the method is not called and I don't see any relevant differences between the examples and other extensions and this one. Do you see what could be a cause of this issue?


::swap will accidentally free buffer

utf8_string:.swap() allocated 'tmp', a temorary storage. Leaving it as exact copy of 'str', it might deallocate the held storage when being destructed.

Fixed in #6


[pigeon] NCAssert always triggered after upgrade to 1.0.0

After upgrading to pigeon: 1.0.0, this line was added in the generated files on iOS (i.e. in the generated file messages.m):
https://github.com/creativecreatorormaybenot/wakelock/commit/501196df07356c584e849b6464454f034486b330#diff-8db9c75e600a161c01bef8fd96ee06d2ff5509bd3b3d371a4e00e2a7a54fd8ebR130
This assert:
       NSCAssert([api respondsToSelector:@selector(toggleMsg:error:)], @"FLTWakelockApi api doesn't respond to @selector(toggleMsg:error:)");

Throws in the plugin now, i.e. the message does not work anymore. However, I do not even know what the assert means.
@gaaclarke when is this supposed to be triggered / what am I doing wrong here?

This means you haven't updated your implementation of FLTWakelockApi for the breaking changes in ObjC selector generation described in the CHANGELOG (or you typo'd the selector while doing so). Most likely you have a method called toggle:error: still instead.
@gaaclarke Probably the string api should be replaced with a %@ of the variable api (or a class-to-string) so the message contains the name of the incorrect implementation class.

This means you haven't updated your implementation of FLTWakelockApi for the breaking changes in ObjC selector generation described in the CHANGELOG (or you typo'd the selector while doing so). Most likely you have a method called toggle:error: still instead.

I do not understand - the only part I do when using pigeon is write messages.dart. Then I run flutter pub run pigeon for generation and it generates this file. So are you suggesting I need to touch the generated file? Because on the Dart side, there are no selectors 🤔

This means you haven't updated your implementation of FLTWakelockApi for the breaking changes in ObjC selector generation described in the CHANGELOG (or you typo'd the selector while doing so). Most likely you have a method called toggle:error: still instead.

I do not understand - the only part I do when using pigeon is write messages.dart. Then I run flutter pub run pigeon for generation and it generates this file. So are you suggesting I need to touch the generated file? Because on the Dart side, there are no selectors 🤔  @stuartmorgan

I do not understand - the only part I do when using pigeon is write messages.dart.

No, you also write an implementation of the interface generated by Pigeon

(Sorry for the truncated original version of the comment above; posting from my phone and got the comment button early)

I commented on the issue referenced above. Retitling for the error message change I suggested above.

@stuartmorgan oh wait, sorry I totally did not think about this.
So because I did not change the message definition on the pigeon side, I did not think about the version upgrade generating a different interface.

Yep, there was a breaking change announced in the CHANGELOG and is indicated by the 1.0.0 version number.  In other languages this should generate a compilation error, but in objc it is a warning by default.  You should be getting a warning that your class isn't implementing the interface.
You can either update the signature of you method in the objc code, or you can use the @ObjCSelector() metadata to override the generated selector to match what was previously there.
@HostApi()
abstract class Api {
  @ObjCSelector(toggle:)
  void toggle(Messsage msg);
}

(Also just a tip: when I worked on objc projects one of the first things I'd do is turn on "Treat warnings as errors".  Objc is pretty loosey-goosey by default.)


xwayland_socket_connected: avoid calling wl_event_source_remove on NULL

This should fix #2163.
I was unable to obtain more details through debug prints, because the bug has not triggered in the past two weeks. However, I believe this fix should be sufficient to prevent future crashes.

I'd prefer to understand what goes wrong before merging a fix.

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:
https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/2224


Hosting private RTD instance

We are trying to host a private RTD instance (following instructions on RTD site) and have gotten an instance up, and sample docs built, but are unable to access the docs themselves as whenever the page redirects to <project>.<domain>/en/latest we get a 404. Everything else appears to work.
Any thoughts on what could be wrong here? We've tried fronting with nginx but that didn't do anything differently. # @

If the docs are built but do not show up through http, it is a webserver configuration issue. Which has nothing to do with RTD, so you are better off asking at http://serverfault.com

@gwillem Thanks for the reply. You are correct in that I was doing something silly in my web server config.


As requested,, I'm reporting a "we should provide substanittailly more information" exception

Steps to reproduce
Buildiing conditional Widget lists
Expected results
paint my widgets on the screen
Actual results
======== Exception caught by rendering library =====================================================
The following assertion was thrown during paint():
RenderBox was not laid out: _RenderLayoutBuilder#c47d3 relayoutBoundary=up16 NEEDS-PAINT
'package:flutter/src/rendering/box.dart':
Failed assertion: line 2165 pos 12: 'hasSize'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.

Hi @richardhaven, the code sample you provided above is lacking in context so we cannot offer much help here. If you are using a Row in SingleChildScrollView, you should probably wrap it in an object that constraints the size of the Row, like a SizedBox or a ConstrainedBox.
Please provide a complete runnable minimal reproducible code sample so that we can properly investigate this.
Thank you


[feature] Is this possible?

I have 2 questions:

I'd like to be able to setup a 'vsco checker' that alerts me to new posts (image uploads) to a handful of pages; is that possible with this app?

For some other sites I'd like to NOT have my own IP show up in any of that server's logs that I'm checking, if I were to say, host it at home in a docker container; is there some way to "bounce" the requests through another IP?

Thanks!
Kacey

please read the issue template


Handle newly-introduced types.Alias type

Go 1.22 [1] introduces a proper types.Alias type for type aliases. This PR adds handling (by simply unwrapping alias node - effectively restoring previous behavior) for such a type.
Further simplification and refactors will be in future PRs.

AppendSeq

This seems to be a separate issue we should handle, will do that in a follow-up PR :)

Nice! Could you also add support for iter.Seq? The golden test is reporting internal errors.

This seems to be a separate issue we should handle, will do that in a follow-up PR :)

Done in #288


News Item Page

There's a sharp line at the edge of the container. Might need to revisit your html structure.

Browser metadata
Path:      /news-template/
Browser:   Chrome 91.0.4472.106 on Mac OS 10.15.7
Viewport:  1680 x 914 @2x
Language:  en-US
Cookies:   Enabled

View on BrowserStack

Open Deploy Preview · Mark as Resolved

@asasmith that is the container from the carousel, it's showing up because of the drop shadow. See below without shadow:

@asasmith that is the container from the carousel, it's showing up because of the drop shadow. See below without shadow:

What about changing the overflow?

Still seeing a very sharp and very defined line here

Browser metadata
Path:      /news-template/
Browser:   Chrome 91.0.4472.106 on Mac OS 10.15.7
Viewport:  1676 x 914 @2x
Language:  en-US
Cookies:   Enabled

View on BrowserStack

Open Deploy Preview · Mark as Resolved


DllNotFoundException: FirebaseCppApp-11_3_0 assembly: type: member:(null)

DllNotFoundException: FirebaseCppApp-11_3_0 assembly: type: member:(null)
at (wrapper managed-to-native) Firebase.AppUtilPINVOKE+SWIGExceptionHelper.SWIGRegisterExceptionCallbacks_AppUtil(Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate,Firebase.AppUtilPINVOKE/SWIGExceptionHelper/ExceptionDelegate)
at Firebase.AppUtilPINVOKE+SWIGExceptionHelper..cctor () [0x000ef] in :0
Rethrow as TypeInitializationException: The type initializer for 'SWIGExceptionHelper' threw an exception.
at (wrapper managed-to-native) System.Object.__icall_wrapper_mono_generic_class_init(intptr)
at Firebase.AppUtilPINVOKE..cctor () [0x00000] in :0
Rethrow as TypeInitializationException: The type initializer for 'Firebase.AppUtilPINVOKE' threw an exception.
at Firebase.AppUtil.SetLogFunction (Firebase.LogUtil+LogMessageDelegate arg0) [0x00001] in :0
at Firebase.LogUtil..ctor () [0x0000f] in :0
at Firebase.LogUtil..cctor () [0x00011] in :0
Rethrow as TypeInitializationException: The type initializer for 'Firebase.LogUtil' threw an exception.
at Firebase.FirebaseApp..cctor () [0x00051] in :0
Rethrow as TypeInitializationException: The type initializer for 'Firebase.FirebaseApp' threw an exception.
at RegisterManager.Start () [0x00000] in :0

Thanks for bringing this to our attention, @johntitor08. Does the issue occur when using the editor or while running the build? Also, could you provide the following information from our template related to your implementation so that we could analyze what's causing this to happen:

Unity editor version
Source you installed the SDK: _____ (.unitypackage or Unity Package Manager)
Problematic Firebase Component: _____ (Auth, Database, etc.)
Other Firebase Components in use: _____ (Auth, Database, etc.)
Additional SDKs you are using: _____ (Facebook, AdMob, etc.)
Platform you are using the Unity editor on: _____ (Mac, Windows, or Linux)
Platform you are targeting: _____ (iOS, Android, and/or desktop)
Scripting Runtime: _____ (Mono, and/or IL2CPP)


Explore supporting masking

Following up on a discussion following a talk on Tiled and Tiled-Uproot with @jpivarski @dylanmcreynolds and others. (Feel free to tag anyone interested in commenting on the IRIS-HEP side.)
Consider this use case:
A user uploads an array to Tiled that we want to use as a boolean mask. Support that mask lives at
https://.../api/v1/x

The user wants to apply that mask to a request for the masked region of another dataset, y. Perhaps this is supported like
https://.../api/v1/array/full/y?mask=/x

I believe this would be compatible with slicing. If the masked region were still quite large and/or N-dimensional, one might request a sub-slice of the masked region:
https://.../api/v1/array/full/y?mask=/x&slice=0

Similarly, for large arrays with chunked access:
https://.../api/v1/array/block/y?mask=/x&slice=0&block=1,7,3

This would require:

Adding a mask query parameter to the appropriate routes, and resolving that path to a TIled node. I think that can reuse the exact same code (the same FastAPI "Dependency")  that we use to resolve the {path} parameter, and apply the same access controls.
Adding logic to the route which, conditional on mask being set, reads that data and passes it down to the Adapter.
Adding an option mask argument to the relevant Adapters, which could either apply the mask after reading the data or, where possible, further push it down into the underlying I/O

I don't think it would be too much of a lift to prototype a PR branch we could play with.
Question: Clearly there is interest in applying this to the /awkward/full/{path} and /awkward/buffers/{path} routes too. Would the mask sometimes be Awkward? (Is that even a coherent question?) Or are the masks always straightforward strided arrays?

@lgray and @nsmith- are interested in this.


fix #4181

try to fix #4181

Fix looks good but tests are needed.

I'm working on it, but first family weekend 😉

Something strange has happened with this PR most likely when rebasing it. According to the tab on the top, there are changed to 162 files which obviously cannot be true.

created a new PR #4206 with only the changes for the fix
please reject this PR

New PR merged!


x/tools/gopls: jump to definition on an import spec should offer multiple locations

Forked from https://github.com/microsoft/vscode-go/issues/2614.
Jumping to the definition of an import spec should give you an option to jump to any file in the package, not just the longest one.

Duplicate of https://github.com/golang/go/issues/32993


update search_api_solr to 3.8

What does this Pull Request do?
Updates search_api_solr to version 3.8
What's new?

Updates search_api_solr to version 3.8
Per the install.md - adds a patch for Drupal Core
Per the install.md - uses drush command to generate the default solr config files
Reorders the playbook.yml so that solr.yml runs after webserver.yml because the solr config setup requires the output of the drush command

How should this be tested?

check out this PR, vagrant up
check to see that you got a running solr instance with a core called ISLANDORA
check to see that you have a default solr server in the drupal search api config (http://localhost:8000/admin/config/search/search-api/)
check that http://localhost:8000/admin/config/search/search-api/server/default_solr_server says the schema is set to drupal-8.3.8-solr-7.x
check that you can create an object and that it is indexed in solr and searchable

Additional Notes:
I was concerned about conflating the solr host and the webserver host so that is why I implemented the "Unarchive solr config" as a request to localhost instead of directly from a file path like /var/www/...
Interested parties
@Islandora-Devops/committers

Results in a warning: "Solarium 5.0.2 is too old and has known issues. "
To alleviate this we can update the composer task to be:
- name: Add a patch for symfony-event-dispatcher per the INSTALL.md docs on search_api_solr:3.8
  composer:
    command: require
    arguments: symfony/event-dispatcher:'4.3.4 as 3.4.99' solarium/solarium:^5.1.4 drupal/search_api_solr
    working_dir: "{{ drupal_composer_install_dir }}"

I'm building a new instance with this revised task right now.

awesome, thanks @seth-shaw-unlv

So, this works!

HOWEVER, it breaks the existing islandora_search configs. With the updated search_api_solr module they appear to assume solr will be accessible @ '/solr/' so the linked config line turns http://<IP_ADDRESS>:8983/solr/#/ISLANDORA into http://<IP_ADDRESS>:8983/solr/solr/#/ISLANDORA. I don't know why they did that, but there it is.
So, @elizoller , if you can add that solarium bit to the composer command in this PR we can merge it. I'll create a new PR on islandora_defaults to purge the path config.

Spinning up another instance to double check before merge.

This build completed successfully for me and no more solarium warning

I had forgotten to pull in the Grok playbook fix for CentOS so my build died. 🤦♂ It is building again and should be done in a little while.