feat: strongly typed css and cleaner express support

This is a complete revamp of the ory/themes repository which will completely change how the library functions!
Here we are using Vite as the bundler instead of only Rollup. This makes the bundling experience a bit easier and really fast! We can now support a variety of environments through a singular repository. For example, in this repository we already have React, Preact and HTML support.

This PR also introduces a new way of writing CSS!
Vanilla-Extract is used to strongly type the CSS, a type of CSS-in-JS library which generates a static CSS file for us when the library is built. This means we can manage our CSS and reduce a lot of typing, since it can generate the CSS classes for us.
Here is an example of vanilla-extract in action!
export const dividerStyle = recipe({
  base: {
    display: 'block',
    textAlign: 'center',
    overflow: 'hidden',
    boxSizing: 'border-box',
    border: 0,
    borderTop: `${pxToRem(4)} solid`,
    borderColor: oryTheme.border.def,
    width: pxToRem(64)
  },
  variants: {
    sizes: {
      fullWidth: {
        width: '100%'
      }
    }
  }
});

Generated JS function.
var dividerStyle = createRuntimeFn({ defaultClassName: "_3ldkmt0", variantClassNames: { sizes: { fullWidth: "_3ldkmt1" } }, defaultVariants: {}, compoundVariants: [] });

And the generated CSS classes.
._3ldkmt0 {
  display:block;
  text-align:center;
  overflow:hidden;
  box-sizing:border-box;
  border:0;
  border-top:.25rem solid;
  border-color:var(--ory-theme-border-def);
  width:4rem
}

._3ldkmt1 {
  width:100%
}

Vanilla-Extract also provides us theme variables which we can give static names. This means we can overwrite them inside the project consuming the library!
:root {
    --ory-theme-font-family: Inter;
    --ory-theme-font-style: normal;
    --ory-theme-accent-def: #3D53F5;
    --ory-theme-accent-muted: #6475F7;
    --ory-theme-accent-emphasis: #3142C4;
    --ory-theme-accent-disabled: #E0E0E0;
    --ory-theme-accent-subtle: #eceefe;
    --ory-theme-foreground-def: #171717;
    --ory-theme-foreground-muted: #616161;
    --ory-theme-foreground-subtle: #9E9E9E;
    --ory-theme-foreground-disabled: #BDBDBD;
    --ory-theme-foreground-on-dark: #FFFFFF;
    --ory-theme-foreground-on-accent: #FFFFFF;
    --ory-theme-foreground-on-disabled: #e0e0e0;
    --ory-theme-background-surface: #FFFFFF;
    --ory-theme-background-canvas: #FCFCFC;
    --ory-theme-error-def: #9c0f2e;
    --ory-theme-error-subtle: #fce8ec;
    --ory-theme-error-muted: #e95c7b;
    --ory-theme-error-emphasis: #DF1642;
    --ory-theme-success-emphasis: #18A957;
    --ory-theme-border-def: #E0E0E0;
    --ory-theme-text-def: #FFFFFF;
    --ory-theme-text-disabled: #757575;
    --ory-theme-input-background: #FFFFFF;
    --ory-theme-input-disabled: #E0E0E0;
    --ory-theme-input-placeholder: #9E9E9E;
    --ory-theme-input-text: #424242;
}

For ExpressJS the library also exports a helper function which registers all of the CSS the library produces.
import express, { Application } from 'express';
import { assignInlineVars } from '@vanilla-extract/dynamic';
import { oryTheme, Theme } from '../theme';

export const RegisterOryThemesExpress = (app: Application, theme: Theme) => {
  app.use('/theme.css', (req, res) => {
    res.header('Content-Type', 'text/css');
    res.send(
      `body {${assignInlineVars(oryTheme, {
        ...oryTheme,
        ...theme
      }).toString()}}`
    );
  });
  app.use('/', express.static('node_modules/@ory/themes/dist'));
};

Which exposes all of the relevant css files for us which we just import in our HTML page:
  <link rel="stylesheet" href="style.css"> // the default theme variables
  <link rel="stylesheet" href="theme.css"> // the overidden theme variables
  <link rel="stylesheet" href="inter-font.css"> // Inter fonts
  <link rel="stylesheet" href="brands-icons.min.css"> // fontawesome brand icons

We can then reference a component through the handlebars helper functions that return pure HTML.
{{{card}}}

{{{typography
      "Welcome!"
      "headline37"
      "foregroundDefault"
 }}}

// Render the data using a view (e.g. Jade Template):
res.render('login', {
   ...flow,
  typography: (text: string, size: any, color: any) =>
        Typography({
          children: text,
          type: "regular",
          size,
          color,
        }),
   card: ComponentWrapper(Card({
     title: "Login With Ory",
     children: Message({message: "Woah there", severity: "info"})
    }))
})

Things to do before merging:

[x] Release new @ory/client
[ ] Merge https://github.com/ory/integrations/pull/36
[x] Add missing UiNode enum types to @ory/client
[ ] Theme variable naming conventions
[ ] Component re-usability and HTML attribute exposure
[ ] Component tests
[ ] More Storybook stories
[ ] NPM multi-bundled package system

Related Issue or Design Document
Checklist

[x] I have read the contributing guidelines and signed the CLA.
[x] I have referenced an issue containing the design document if my change introduces a new feature.
[x] I have read the security policy.
[x] I confirm that this pull request does not address a security vulnerability.
If this pull request addresses a security vulnerability,
I confirm that I got green light (please contact<EMAIL_ADDRESS>from the maintainers to push the changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation within the code base (if appropriate).

Closing this in favor of https://github.com/ory/elements/pull/1


The XSLT stylesheet file is now initialized to an env.File() Node, such that dependencies work correctly in hierarchical builds.

Re-applying a patch I submitted already in 2013 and was accepted back then. Refer to commit e63d742d2026 in the old Mercurial https://bitbucket.org/dirkbaechle/scons_docbook repo.
Somehow during upgrade and integration of the docbook this must have got lost.
I only stumbled across this because I finally upgraded from Scons 3 to 4 after all those years.
Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated CHANGES.txt (and read the README.rst)
[ ] I have updated the appropriate documentation

This sounds reasonable enough, though the bitbucket repo pointed to here is gone in their purge. @dirkbaechle ?

@hwmaier You removed the developer checklist which is in the pull request template.
I've restored it .
Please fill it out.

What breaks without this change?

Given that this change already was committed and accepted earlier, I felt it would not be necessary for those formalities.
This is the log of the original branch (which I still have a local copy of) on bitbucket:
changeset:   11:e63d742d2026
parent:      9:b0e290b5d5ef
user:        dirkbaechle
date:        Fri Oct 04 11:13:12 2013 +0200
summary:     - a patch, contributed by Henrik Maier: the XSLT stylesheet file is now initialized to an env.File() Node, such that
changeset:   9:b0e290b5d5ef
user:        dirkbaechle
date:        Mon Apr 22 23:47:53 2013 +0200
summary:     - fixed the lxml support for xincludes and xsl transformations
changeset:   8:1686cc7cdee5
user:        dirkbaechle
date:        Sun Mar 24 19:43:09 2013 +0100
summary:     - always resolve entities when reading files with libxml2
changeset:   7:0afc056ca355
user:        dirkbaechle
date:        Sat Mar 09 12:27:04 2013 +0100
summary:     - small fix: If the input file for DocbookMan doesn't exist yet (it is generated during the build), we can't derive the target name from its contents. So we have to rely on the given target name instead.
changeset:   6:0ab3bdbaab05
user:        dirkbaechle
date:        Sat Mar 09 11:53:17 2013 +0100
summary:     - changed the interface of DocbookXInclude, it now always requires a target AND a source parameter
changeset:   5:3ff5ccf6e4fc
user:        dirkbaechle
date:        Wed Nov 14 23:50:09 2012 +0100
summary:     - fix by Pawel Tomulik: changed terminator symbol for XML dependencies detection
changeset:   4:74f0af5781ec
user:        dirk
date:        Tue Jul 05 01:06:17 2011 +0200
summary:     - support for preferring xsltproc over libxml2 and lxml (environment variable DOCBOOK_PREFER_XSLTPROC)
changeset:   3:f28ed172f9aa
user:        dirk
date:        Tue Jul 05 00:52:21 2011 +0200
summary:     - added support for implicit dependencies, based on system entities in XML files
changeset:   2:41057821ac4d
user:        dirk
date:        Wed Jun 08 20:56:17 2011 +0200
summary:     - improved dependency scanning
changeset:   1:e7212442c66a
user:        Dirk Baechle<EMAIL_ADDRESS>date:        Tue Jan 04 00:53:35 2011 +0100
summary:     - added dependencies for the used XSL file to all builders
changeset:   0:e8bbba1f699e
user:        Dirk Baechle<EMAIL_ADDRESS>date:        Thu Dec 30 15:43:28 2010 +0100
summary:     - first version

You'd be incorrect.
Clearly undoing the change didn't break any tests..
So that's why we like to add tests with code changes.
All PR's go through the same process, excluding some housekeeping by the maintainer..

What breaks without this change?

Sorry, forgot to explain this.
If you have a main SConstruct file in the top level directory with this content:
SConscript('doc/SConscript', 'env')

and then a SConscript file in the doc subdir with:
env.DocbookXslt('xml/index.aml', 'xml/indexpage.xml', xsl = 'doxy2aml.xsl')

and the XSL stylesheet doxy2aml.xsl resides in doc\doxy2aml.xsl then the style sheet will not be found because the doc subdirectory is not applied to the stylesheet's file node in this hierarchical build scenario. Once the patch is applied, then the subdir is applied to the node and the style sheet is passed to the xslt processor with correct path name.

So I added a test case which I think is good anyway as it also explains better what goes wrong.
Without the patch:
C:\temp\scons>python runtest.py  test\Docbook\basic\xsltsubdir\xsltsubdir.py
1/1 (100.00%) C:\\Python39\\python.exe test\Docbook\basic\xsltsubdir\xsltsubdir.py
File doesn't exist: `C:\Users\henrik\AppData\Local\Temp\testcmd.8360.67of99er\subdir/out.xml'
FAILED test of C:\temp\scons\scripts\scons.py
        at line 578 of C:\temp\scons\testing\framework\TestCommon.py (must_not_be_empty)
        from line 42 of C:\temp\scons\test\Docbook\basic\xsltsubdir\xsltsubdir.py

with the applied patch:
C:\temp\scons>python runtest.py  test\Docbook\basic\xsltsubdir\xsltsubdir.py
1/1 (100.00%) C:\\Python39\\python.exe test\Docbook\basic\xsltsubdir\xsltsubdir.py
PASSED

I also did notice that the docbook tool (__build_lxml function, etree.parse call in this case) does not write exceptions on the console if something goes wrong internally. Not sure if that is intended, but it makes fault finding harder as the Scons will not show an error on the console and the DocbookXslt tool silently fails with simply not generating any output. If I wrap etree.parse in a try/except block and then print the exception, we see what is going wrong.

@hwmaier - Hiding the etree.parse error was most likley not intentional. If you can capture and output that, it'd be a great addition.

@hwmaier - Hiding the etree.parse error was most likley not intentional. If you can capture and output that, it'd be a great addition.
This may be more of an issue with lxml than Scons.Docbook.  I don't feel comfortable to dig deeper into this as I never used lxml and don't know the correct API usage and method or dealing with potential errors.

Merged.
Note I removed the clone from the SConscript and just set tools=['docbook'] in the Environment in SConstruct and test was still effective.

Another minor note. For next time, we ask that people add their names to CHANGES in alphabetical by last name order. Just to keep it organized.
Thanks for the fix!


Can you use netlify-recaptcha?

Your example works great! However I'm wondering how to integrate  as described here.
https://www.netlify.com/docs/form-handling/
Currently I'm getting a full page response when I try to submit the form request which includes a recaptcha. As a workaround I wanted to put one on the page up front.

@imorente , I too would like to use the explicit recaptcha option. I couldn't seem to get it to work. Any additional guidance or documentation on this specifically for a Gatsby website?

From the docs:

This will not work with React-based sites, including Gatsby. This is because React will not execute script tags injected into the DOM.

It looks like Netlify will need to do more work on this in order to allow us to use it from React.

Hopefully this is something that Netlify can provide a workaround for. Checking to see if https://github.com/appleboy/react-recaptcha will work temporarily, not sure. The Netlify Honeypot option, while helpful, definitely lets too much spam through… I just created a new site for a client, 48 hours later 120+ messages, all spam.

@coreygo
I have reached out to Netlify a couple of times about this topic. I was chatting with their support as recently as yesterday but there is no solution as of yet. They just mentioned they will look into more spam solutions but I wouldn't hold your breath.
They did however mention that if you go over the 100 submission monthly limit due to spam to reach out to them directly so that you don't get charged.
As for something like https://github.com/appleboy/react-recaptcha I don' know if that would be possible with Netlify alone. Unless I am mistaken you would need to validate the captcha response with the server somehow which I don't think you could do that.
If you find a solution that works please share with the group. 😄

I ended up using Formspree as a workaround for now, but I really would have liked to integrate with Netlify instead. Formspree works great with Gatsby and has recaptcha.

@acarl thanks for sharing! I agree though that I would prefer to use Netlify if possible. Particularly since it is a paid feature now I feel like my expectations are a bit higher for their form functionality. Good to know though that Formspree works with Gatsby.

@ryanwiemer Good to know and I definitely agree, especially if they plan to charge $19/mo. That's pricey compared to Webflow or WordPress hosting etc.
I pinged Netlify support for my client, hopefully they'll do a one time credit since only 2 of the 120+ submissions are legit.
I could swear I set the honeypot field up correctly on the client's two forms and also made some fields required but he's still getting invalid form submissions with missing fields. Might try Formspree.

Netlify announced in their newsletter yesterday the possibility to add a custom reCAPTCHA.  I tried it out using react-google-recaptcha in this example and it seems to work 🎉
I updated the README with the steps I followed in case anyone else wants to try it out.


Problem adding new message type

after i add the new message type by the instruction in the wiki (https://github.com/siemens/ros-sharp/wiki/Dev_NewMessageTypes ,Option 3) i got the following error in unity:
"The type or namespace name 'MessageTypes' does not exist in the namespace 'RosSharp.RosBridgeClient' (are you missing an assembly reference?)"
the only thing that is exist in RosSharp.RosBridgeClient now is Actionlib.
How do i get all the types and namespace that spouse to be in  RosSharp.RosBridgeClient?
the message i tried to add:
using Newtonsoft.Json;
namespace RosSharp.RosBridgeClient.MessageTypes.Sensor
{
public class Outbonnd_sv_list : Message
{
[JsonIgnore]
public const string RosMessageName = "kairos_brigde/Outbonnd_sv_list";
public RosSharp.RosBridgeClient.MessageTypes.Std.Header header;
public string[] video_mux1;
public bool veh_start;
public bool veh_enable;
public bool veh_motion;
public short joy_steer;
public short veh_steer;
public ushort veh_brake;
public ushort veh_throttle;
public ushort veh_shift;
    public Outbonnd_sv_list()
    {
        header = new Std.Header();
        video_mux1 = new string[0];
        veh_start = false;
        veh_enable = false;
        veh_motion = false;
        joy_steer = 0;
        veh_steer = 0;
        veh_brake = 0;
        veh_throttle = 0;
        veh_shift = 0;
    }
}

}

Hello @omergreenwald,
Unfortunately I cannot reproduce this issue. After downloading the RosSharp repository, have you tried getting a successful build from the solution under the folder ros-sharp/Libraries? The message types that you mentioned should be all included in the project.
With the recent changes, you do not need to have "Newtonsoft" related codes in your class. The wikipage shall be updated soon. Here is the code for the message class \Libraries\RosBridgeClient\MessageTypesOutbonnd_sv_list.cs that I was able to successfully added to the project;
namespace RosSharp.RosBridgeClient.MessageTypes.Sensor
{
    public class Outbonnd_sv_list : Message
    {
        public const string RosMessageName = "kairos_brigde/Outbonnd_sv_list";
        public RosSharp.RosBridgeClient.MessageTypes.Std.Header header;
        public string[] video_mux1 { get; set; }
        public bool veh_start { get; set; }
        public bool veh_enable { get; set; }
        public bool veh_motion { get; set; }
        public short joy_steer { get; set; }
        public short veh_steer { get; set; }
        public ushort veh_brake { get; set; }
        public ushort veh_throttle { get; set; }
        public ushort veh_shift { get; set; }

        public Outbonnd_sv_list()
        {
            header = new RosSharp.RosBridgeClient.MessageTypes.Std.Header();
            video_mux1 = new string[0];
            veh_start = false;
            veh_enable = false;
            veh_motion = false;
            joy_steer = 0;
            veh_steer = 0;
            veh_brake = 0;
            veh_throttle = 0;
            veh_shift = 0;
        }
    }
}

If you have an issue regarding getting a successful build from the RosBridgeClient project, please open another issue using the provided issue template.


Additional inference parameters for openai and genai

Added additional inference parameters for both IbmGenAiInferenceEngine and OpenAiInferenceEngine

@yoavkatz

Is it necessary on top of to_dict() inherited from Dataclass?

to_dict also returns attributes of the Artifact class, so there would be need to filter these parameters each time anyway

So it is just the immediate class parameters without its ancestor?

So it is just the immediate class parameters without its ancestor?

Yup

I like the changes - but note they are not backward compatible. We need to make the changes gradually.

So should we revert changes to main?

Do we have a solution to make it backward compatible? There are pro's and con's in keeping old code.
At the minimum, we should keep  IbmGenAiInferenceEngineParams  as a deprecated class, so  there will be a clear error message.

So I suggest we revert for and make these changes.


how can use Date-picker with Tabledit

hello
i have PHP page with Tabledit  , its work very fine
but i would like to use Date-picker
thanks a lot

thanks a lot my friend
please can you share full code ,
i try this code but not work with me
thanks again

You should put the above code in onDraw: function() {}
Please refer Example 6 javascript : https://markcell.github.io/jquery-tabledit/#examples

BindhyaKC  Thank you very much

can we use the same for drop down?

Yes. See example 6.
https://markcell.github.io/jquery-tabledit/#examples

Thank you @BindhyaKC for answer and @xbox2007 >for asking..


Rules ANN401,W605 cause autofix error

Ruff 0.0.286 (latest changes from main branch)
ruff  *.py --select ANN401,W605 --no-cache

file content:
from currency.converter import CurrencyConverterMessageFactory as _
currencies = {
        "DOP" : (_(u"Dominican Peso"), u"\RD$"),
        }

error:
error: Autofix introduced a syntax error. Reverting all changes.

This indicates a bug in `ruff`. If you could open an issue at:

    https://github.com/astral-sh/ruff/issues/new?title=%5BAutofix%20error%5D

...quoting the contents of `currencies922.py`, the rule codes E201, E203, I001, UP025, W291, W605, along with the `pyproject.toml` settings and executed command, we'd be very appreciative!

currencies922.py.zip

It must be adding the r prefix in u"\RD$" which makes it ru"\RD$" which is invalid. I guess the fix would be to either not add the r prefix if the string is unicode.

Cannot reproduce with latest version


[fix](tablet schedule) disable schedule need clear running tasks

BUG:  when disable tablet schedule, the running task still remain until timeout.  Need set them fail when disable schedule.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run feut

run feut

run buildall

run cloud_p1


HTTP(S) proxy won't work under windows platform

Describe the bug
There's no way to use proxy smoothly, even i tried:
vcpkg env
set HTTPS_PROXY=https://<IP_ADDRESS>:10809/
set HTTP_PROXY=https://<IP_ADDRESS>:10809/
vcpkg install boost:x64-windows

The vcpkg says:
Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.75.0.beta1/boostcpp.jam... Failed. Status: 4;"A requested feature, protocol or option was not found built-in in this libcurl due to a build-time decision."

Seems the curl won't use windows proxy.
Isn't there any way for you to use another download tool on Windows for simple proxy setting like
vcpkg --proxy=https://<IP_ADDRESS>:10809 ?
Environment

OS: Windows
Compiler: Visual Studio 2019 MSVC

To Reproduce
Steps to reproduce the behavior:
1.Start an HTTP proxy at 10809
2.
vcpkg env
set HTTPS_PROXY=https://<IP_ADDRESS>:10809/
set HTTP_PROXY=https://<IP_ADDRESS>:10809/
vcpkg install boost:x64-windows

Expected behavior
The vcpkg uses my HTTPS proxy to download files fast and easy.
Failure logs
Starting package 1/121: boost-build:x64-windows
Building package boost-build[core]:x64-windows...
-- Using cached E:/vcpkg/downloads/boostorg-build-boost-1.75.0.beta1.tar.gz
-- Cleaning sources at E:/vcpkg/buildtrees/boost-build/src/75.0.beta1-4371cdd5cf.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source E:/vcpkg/downloads/boostorg-build-boost-1.75.0.beta1.tar.gz
-- Applying patch fix_options.patch
-- Using source at E:/vcpkg/buildtrees/boost-build/src/75.0.beta1-4371cdd5cf.clean
-- Using cached E:/vcpkg/downloads/boost_LICENSE_1_0.txt
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.75.0.beta1/boostcpp.jam...
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.75.0.beta1/boostcpp.jam... Failed. Status: 4;"A requested feature, protocol or option was not found built-in in this libcurl due to a build-time decision."
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:184 (message):

      Failed to download file.
      If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
      variables to "https://user:password@your-proxy-ip-address:port/".
      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
  ports/boost-build/portfile.cmake:25 (vcpkg_download_distfile)
  scripts/ports.cmake:136 (include)

Error: Building package boost-build:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: boost-build:x64-windows
  Vcpkg version: 2020.11.12-nohash

Additionally, attach any relevant sections from the log files above.

Additional context
No

It seems after setting the following:
vcpkg env
set HTTPS_PROXY=http://<IP_ADDRESS>:10809/
set HTTP_PROXY=http://<IP_ADDRESS>:10809/
vcpkg install boost:x64-windows

which use HTTP not HTTPS works for me, so i think it is better to change the description of proxy setting instructions for more compatibility.

It seems after setting the following:
vcpkg env
set HTTPS_PROXY=http://<IP_ADDRESS>:10809/
set HTTP_PROXY=http://<IP_ADDRESS>:10809/
vcpkg install boost:x64-windows

which use HTTP not HTTPS works for me, so i think it is better to change the description of proxy setting instructions for more compatibility.

#15435

#15435


Accept handling VIO events

Keeping the VIO continuation set at all times to prevent connection leakage. In this case, it will cause the acceptor to log an error so we know this is happening.

Is this a candidate for 9.0.x or 9.1.x ?

From the perspective of H3/QUIC, it's a nice-to-have for 9.0, and I can wait for 9.1.


Missing element types for actions block

Hi!
I'm trying to use an actions block but the SlackActionBlockElement enum seems to be missing a few of the possible elements. In my case I'm trying to use a static_select but I think there are more missing, such as users_select.
Any chance those could be added?
Thanks for a really nice crate!

Hey,
I think they are in fact defined here:
https://github.com/abdolence/slack-morphism-rust/blob/169ca68ca1ab93d1d15f9ca8dc770393edcf8e27/src/models/blocks/kit.rs#L160
https://github.com/abdolence/slack-morphism-rust/blob/169ca68ca1ab93d1d15f9ca8dc770393edcf8e27/src/models/blocks/kit.rs#L168
Did you see some issues with those or are you looking for something else?

Ah, now I understand. SlackActionBlockElement not SlackSectionBlockElement. Probably they have been added afterwards. Let me check.

I think this will do:
https://github.com/abdolence/slack-morphism-rust/pull/171
It is merged and available in the master branch. You can use a reference to the git:
https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories
Can you help with the testing? :)
Thanks for the report!

Works great! Thanks a lot for jumping on this so quickly :)

Released in https://github.com/abdolence/slack-morphism-rust/releases/tag/v1.6.0


⚠ Feedaty widget has degraded performance

In be0390c, Feedaty widget (https://widget.feedaty.com/public/js/feedaty.js?merchant=10213975&style_ver=2021&data-sku=36263) experienced degraded performance:

HTTP code: 200
Response time: 3269 ms

Resolved: Feedaty widget performance has improved in bed830e after 10 minutes.


Fix double buffering on egl and restore default (fixing everything else)

Restore the default that was changed by https://github.com/tomaka/glutin/pull/690 (changing defaults isn't nice)
fix double buffer requirement check
double_buffering is forced on at line 603 and so this should fail if single buffer was requested

Passing None in the requirements means "use the implementation default", which in practice means double-buffering for windows and single-buffering for pbuffers. That's exactly what we want.
There is indeed an error in the code, but it's that glutin should return an error for both Some(true) and Some(false), as enforcing double-buffering or single-buffering is unimplemented.

fine but isn't changing the default quite a serious change? changing the default without implementing some way to change it back to what works sucks majorly.
with it set to None, glium just draws black on windows if the glutin dependency targets the commit from merging #690, exposing the setting in WindowBuilder and setting it so Some(true) results in things drawing again but breaks android because of the false Some(true) = fail check

fine but isn't changing the default quite a serious change? changing the default without implementing some way to change it back to what works sucks majorly.

Well, it's not supposed to not work.
You're describing two problems:
1 - Windows shouldn't create a single-buffered window by default
2 - Glium shouldn't draw black even if the window is single buffered
I don't think we should fix these by reverting to wrong code.

the other change is not implemented to broken though...
the change to src/api/egl/mod.rs seems only right in the current state, you can ask for anything except Some(false)
also who says Windows shouldn't be single buffered. The docs for double_buffer say default is Some(true)

So my reasoning for this change is that changing the default is bad. Everything out there has already been able to assume that double buffering is the default and right now there's no way to disable that if you use a WindowBuilder. Hence Glium breaks, the default changed underneath it. Bad thing to do.
However the default was changed because Some(true) is erroneously being thrown out as bad despite that being the reality. That is the real fix, not changing the default.

It has only been since #677, merged less than a month ago, that the user is allowed to specify whether double buffering or simple buffering is required. Before this PR you couldn't even choose and you would just get an implementation-defined behavior (double buffering for windows, single buffering for pbuffers).
In #677 the default value for double_buffer was set to Some(true). This was definitely a mistake, since pbuffers aren't supposed to be double-buffered. This change may have broken some stuff. Changing that to None restores the behavior as it was one month ago, when you weren't able to choose this.
If changing the value to None breaks Windows, then that means there was another problem in #667 that was only revealed when switching to None.
About rejecting Some(true), as I said above both Some(true) and Some(false) should be rejected for now, until the EGL implementation is complete.

I see the wgl version only sets double buffer if requested. If i do a PR that makes it double unless false is set will that be ok ?

Sure!

I just had a look at changing that around https://github.com/tomaka/glutin/blob/master/src/api/wgl/mod.rs#L386 and https://github.com/tomaka/glutin/blob/master/src/api/wgl/mod.rs#L544 but it doesn't feel right either =/

I think the problem is that these functions (ChoosePixelFormat and wglChoosePixelFormat) can return multiple formats, but glutin just takes the first one. Maybe it's better if glutin took the last one instead, as they are often sorted from worse to best.
For the record, before #677 the formats were enumerated manually and picked manually.


Does not support php8.0？

composer create-project
Your requirements could not be resolved to an installable set of packages.
Problem 1
- specialtactics/l5-api[v3.0.0-beta1, ..., v3.0.1] require php ^7.3 -> your php version (8.0.3) does not satisfy that requirement.
- Root composer.json requires specialtactics/l5-api ^3.0 -> satisfiable by specialtactics/l5-api[v3.0.0-beta1, v3.0.0, v3.0.1].

Hey there, I think it's easier to add PHP 8 support with laravel 9, the main reason is that there are upstream repos that are not compatible with php 8, I think the jwt-auth, however that was taken over recently by a new php group, so I will take a look to see if I can do it with the current version of the boilerplate.

Hey @xinfox-ltd PHP 8 support has been added to the dependency package.


How can I get the text in a textbox?

Hi.
this library is very cool.
but I dont know how to get text in textbox.
I try this code below:
target

<input type="text" size="40" value="" readonly ng-model="data">

agouti
/*XPath to target element*/
fmt.Println(page.FindByXPath("/html/body/div/div/div[2]/section/section[3]/div/table/tbody/tr/td/div[3]/div/input").Text())

I think you're looking for the value attribute: selection.Attribute("value")

Thank you so much!


master/slave terminology is inappropriate and unnecessary

Expected behavior

Actual behavior

”slave” has already been replaced with worker. Please update to 1.0.


Move the code copied from a Wai library to Commence

See the bottom of src/Prototype/Exe/Server.hs.
Also, try to upstream it, or at least expose more of the library to reduce the amount of copied code.

This was done in the commit cee35ebfe62816e55ba382ab5679d1a5fc5d518b.


Invalid buffer id: 4 Errors while backspacing in insert mode.

Description
Hey all, I've been noticing this error while leveraging the plugin. Any ideas what this means?
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Press ENTER or type command to continue
E5108: Error executing lua ....vim/plugged/nvim-autopairs/lua/nvim-autopairs/utils.lua:110: Invalid buffer id: 4
Mapping  bug
1.If you report a bug about indent. Please remember that plugin doesn't do anything about indent.
It just trigger the indent of your vim config so if you have wrong indent config then it will do wrong indent.
You can check by select a block of code and press ==
2. provide result of command :verbose imap <cr>.
Steps to reproduce
When I'm in insert mode and I decide to backspace.
Video
Minimal config
vim.cmd [[set runtimepath=$VIMRUNTIME]]
vim.cmd [[set packpath=/tmp/nvim/site]]
local package_root = '/tmp/nvim/site/pack'
local install_path = package_root .. '/packer/start/packer.nvim'
local function load_plugins()
  require('packer').startup {
    {
      'wbthomason/packer.nvim',
      {
        'windwp/nvim-autopairs',
      },
      -- ADD PLUGINS THAT ARE _NECESSARY_ FOR REPRODUCING THE ISSUE
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. '/plugin/packer_compiled.lua',
      display = { non_interactive = true },
    },
  }
end
_G.load_config = function()
  require('nvim-autopairs').setup()
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing nvim-autopairs and dependencies.")
  vim.fn.system { 'git', 'clone', '--depth=1', 'https://github.com/wbthomason/packer.nvim', install_path }
end
load_plugins()
require('packer').sync()
vim.cmd [[autocmd User PackerComplete ++once echo "Ready!" | lua load_config()]]

please test  it with the minimal config above

Also facing this issue, but so far it seems like it only happens after I restore a session with auto-session.

Thanks @zachy-ho for the hint! I'll dig around. I notice when I delete my ~/.local/share/nvim/view/* files this error goes away. Now to figure out why nvim is storing these files to begin with.

@dky do you also use auto-session or are your files in ~/.local/share/nvim/view/* created by something else? I couldn't be bothered figuring it out so I stopped using the auto-session plugin 🤷♂. If you want sessioning functionality from a plugin, I just found out vim-startify also does saving/loading sessions and it doesn't (or hasn't) cause any complications with nvim-autopairs.
Hope you can figure out what's happening with your config~! Let me know if you find something promising.

@zachy-ho I don't use the auto-session plugin so it must be some other plugin or some other configuration stanza that's creating those files. I do use vim-autosave and wonder if that could be it. I'm going to try to figure this out.

looks like options or localoptions causing this error. when set, some global vars are saved, but wrong buffer nr or buffer list is restored?
set sessionoptions=folds,tabpages,winsize,localoptions

I've re-opened the issue and submitted a potential fix for review. I've been the buffer check fix on my own for several months with issue not re-occuring.

This issue is due the following line that is stored in the Session.vim file when localoptions is enabled.
inoremap <buffer> <expr> <BS> v:lua.MPairs.autopairs_bs(111)

Here 111 is the buffer number at the time of :mksession
On subsequent loads of the session file, since buffer number is not the same, this results in the annoying series of errors.
This is definitely nvim-autopairs fault, since this also happens when you are not using any session plugin at all. Just native neovim :mksession with localoptions enabled.

@windwp
This issue is due to this line.
https://github.com/windwp/nvim-autopairs/blob/9fa996123031b4cad100bd5afad04384a622c8a7/lua/nvim-autopairs.lua#L305
Any reason you are passing bufnr here to autopairs_bs?
https://github.com/windwp/nvim-autopairs/blob/9fa996123031b4cad100bd5afad04384a622c8a7/lua/nvim-autopairs.lua#L336
This already ensures that current buffer is used.
You can just replace the line with, 'v:lua.MPairs.autopairs_bs()', and the issue with go away. I don't see any drawbacks.

I just ran into this too with auto-session. Current using:
vim.o.sessionoptions = "blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"

I've been running into this issue dozens of times a day (when pressing Backspace in insert mode).
@windwp If #299 fixes the problem, I'd certainly be interested in seeing it merged!


ReadMe里头的ProGuard内容是不是颠倒了？

原文如下显示：
ProGuard
If you use Glide as your image engine, add rules as Glide's README says.
And add extra rule:
-dontwarn com.squareup.picasso.**

If you use Picasso as your image engine, add rules as Picasso's README says.
And add extra rule:
-dontwarn com.bumptech.glide.**

新手尝试用这个库，说错勿喷

It's not. See https://github.com/zhihu/Matisse/pull/102.


roadmap entry for canister groups

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

The path should not be roadmap/1_core-protocol/upcoming/1_canister-groups/canister-groups.md but roadmap/1_core-protocol/upcoming/canister-groups.md. Please also make sure things look good on the preview that is generated for each PR (currently it's not actually shown due to the path issue).


Update index.js to set a default layout which does not cause white screen issue after loading

When loading the plugin for the first time, after setLoading is false, the screen goes blank because setLayouts can be null by default. So initializing it with empty values solves the problem.

Looks good, thanks! I'll add this to master forgot to update the origin branch 👌


Webcrack can't find string array of old obfuscator version

Describe the bug
When trying to deobfuscate this file, it seems like webcrack can't find the string array.
Expected Behaviour
I expect webcrack to find the string array.
Code
https://gist.github.com/g0dzcsgo/fa6e0402b54718a77656a28affd9c396

Logs
webcrack:transforms prepare: started +0ms
  webcrack:transforms prepare: finished with 4339 changes +461ms
  webcrack:transforms deobfuscate: started +0ms
  webcrack:deobfuscate String Array: no +0ms
  webcrack:transforms deobfuscate: finished with 0 changes +620ms
  webcrack:transforms transpile, unminify: started +0ms
  webcrack:transforms transpile, unminify: finished with 39226 changes +472ms
  webcrack:transforms self-defending, debug-protection, jsx, jsx-new: started +0ms
  webcrack:transforms self-defending, debug-protection, jsx, jsx-new: finished with 0 changes +139ms
  webcrack:transforms merge-object-assignments: started +1ms
  webcrack:transforms merge-object-assignments: finished with 2 changes +438ms

https://js-deobfuscator.vercel.app/ seems to work for older versions


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

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

Questions ?
Please refer to the Backport tool documentation


Update supported Fedora versions

cc/ @andrewhsu
Relates to #5482

F27 builds are currently blocked by https://github.com/rpm-software-management/rpm/issues/367.


Uniqueing arrays and publishers by identifiable ID

I am not happy with the names.
"scan" should be kept, for the sake of consistency with the implementation and publisher return type, but all the rest "unique blah blah blah" seems a bit odd, but I couldn't think about better namings.


Stuck on loading assets screen

Running in Firefox on an iPad Air 4 iOS 14.4. Running vanilla mc server 1.16.5 offline mode.
Running web-minecraft latest commit in dev mode.
I suspect something funky is happening with WebKit html rendering over something like Chromium. In the web-minecraft logs I can see that my character is connecting, and in the minecraft server logs I see that the client connects successfully.
Let me know what other information you would like from me, I am happy to help in debugging all that I can, I absolutely love this project idea!!

Ok, I opened up an old Mac computer to try running it in WebKit but it worked fine in the desktop version of Safari/WebKit. I then plugged in an iPad Air version 1 and connected to its console like this. I think I found the underlying issue:

It appears as though the function to lock in the mouse pointer for gameplay doesn't work on (this particular) mobile browser. I never even considered the fact that you may need to build in a touch screen interface. Regardless, in the case of my iPad Air 4, I was using it with a keyboard and mouse, but I suppose at this time Apple doesn't support those native javascript functions for this device. I wonder if there is some sort of workaround?

There are no controls for touchscreen devices, and I doubt a bypass for pointer lock will be created before those controls are created. @NexusNull prove me wrong!

I think now it will not be worked on. I will focus on pc-like devices.

I think we should add controls for touchscreen devices @NexusNull  and @michaljaz
I cna work on with KaffinPX

can

But in this moment, we must focus on bugs/glitchs.


Functions added: single.groupBy(), summary.isString().

New functionality:

single.groupBy();
summary.isString();
Stream.groupBy().

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
432

Relevant Lines:
432

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
432

Relevant Lines:
432

💛  - Coveralls


added definitions of 2.2.5 and 2.3.1

https://www.ruby-lang.org/en/news/2016/04/26/ruby-2-2-5-released/
https://www.ruby-lang.org/en/news/2016/04/26/ruby-2-3-1-released/

:octocat:

🤘
On Tue, Apr 26, 2016 at 07:39 SHIBATA Hiroshi<EMAIL_ADDRESS>wrote:

Merged #940 https://github.com/rbenv/ruby-build/pull/940.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rbenv/ruby-build/pull/940#event-641411048


Invalid OSSKU

Describe the bug
Cannot create a WINDOWS node pool using Terraform. Can create using Portal GUI or az-cli using same parameters.
To Reproduce
Steps to reproduce the behavior:

Setup pipeline as (include yaml configuration or screenshots of classic ui editor)

resource "azurerm_kubernetes_cluster_node_pool" "winxxx" {
availability_zones    = [1, 2, 3]
enable_auto_scaling   = true
kubernetes_cluster_id = azurerm_kubernetes_cluster.aks_cluster.id
max_count             = 3
min_count             = 1
mode                  = "User"
name                  = "win01"
orchestrator_version  = data.azurerm_kubernetes_service_versions.current.latest_version
os_disk_size_gb       = 128
os_type               = "Windows" # Default is Linux, we can change to Windows
vm_size               = "Standard_D2s_v3"
priority              = "Regular" # Default is Regular, we can change to Spot with additional settings like eviction_policy, spot_max_price, node_labels and node_taints
node_labels = {
"nodepool-type" = "user"
"environment"   = var.environment
"nodepoolos"    = "windows"
"app"           = "dotnet-apps"
}
tags = {
"nodepool-type" = "user"
"environment"   = var.environment
"nodepoolos"    = "windows"
"app"           = "dotnet-apps"
}
}

Execute pipeline

See error

/usr/local/bin/terraform apply -auto-approve /home/vsts/work/1/terraform-manifests-out/test-27.out
azurerm_kubernetes_cluster_node_pool.winxxx: Creating...
Error: creating/updating Managed Kubernetes Cluster Node Pool "win01" (Resource Group "terraform-aks-test"): containerservice.AgentPoolsClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code="InvalidOSSKU" Message="OSSKU='Ubuntu' is invalid, details: Windows does not allow OSSKU selection"
│   with azurerm_kubernetes_cluster_node_pool.winxxx,
│   on 10-aks-cluster-windows-user-nodepools.tf line 3, in resource "azurerm_kubernetes_cluster_node_pool" "winxxx":
│    3: resource "azurerm_kubernetes_cluster_node_pool" "winxxx" {
##[error]Terraform command 'apply' failed with exit code '1'.
##[error]╷
│ Error: creating/updating Managed Kubernetes Cluster Node Pool "win01" (Resource Group "terraform-aks-test"): containerservice.AgentPoolsClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code="InvalidOSSKU" Message="OSSKU='Ubuntu' is invalid, details: Windows does not allow OSSKU selection"
│
│   with azurerm_kubernetes_cluster_node_pool.winxxx,
│   on 10-aks-cluster-windows-user-nodepools.tf line 3, in resource "azurerm_kubernetes_cluster_node_pool" "winxxx":
│    3: resource "azurerm_kubernetes_cluster_node_pool" "winxxx" {
Finishing: Terraform Apply
Expected behavior
A clear and concise description of what you expected to happen.
should have created a windows node pool with 1 node
Screenshots
If applicable, add screenshots to help explain your problem.
Pipeline Logs
Include logs that help demonstrate the problem. Please make sure to redact any sensitive info such as secrets.
Agent Configuration

OS: ubuntu
Hosted in Azure DevOps
Terraform version used (Default for hosted agent is acceptable). 1.0.6
AzureCLI version used (Default for hosted agent is acceptable or N/A)

Additional context
Add any other context about the problem here.
this is a class example: from https://www.udemy.com/course/azure-kubernetes-service-with-azure-devops-and-terraform/learn/lecture/23644074 at 4:28
the class was record some time after 11/4/20 so this appears to have been functional in an
earlier version of the plugin OR an earlier version of Terraform
azurerm.log

This appears to be an issue with terraform itself (most likely the way the templates are configured) rather than this task. I would suggest directing your question to the azurerm terraform provider.


Add support for clusters

I think we can handle this by accepting a list of URIs in Client#initialize and assigning them at random in the connection pool block.
Thinking something like this:
    # Keep this method signature for backwards-compatibility
    def self.new(uri : URI)
      max_idle_pool_size = uri.query_params.fetch("max_idle_pool_size", "25").to_i
      new([uri], max_idle_pool_size)
    end

    def initialize(uris : Array(URI) = [URI.parse("redis:///")], max_idle_pool_size : Int = 25)
      @pool = DB::Pool.new(
        max_idle_pool_size: max_idle_pool_size.to_i32,
      ) { Connection.new(uris.sample) }
    end

I can't remember if this is a thing with Redis, but we may also need to adjust to network topology after we connect in case we don't have all the IP addresses or servers are added/removed after we connect, etc. This is common with a lot of clustered resources, but I don't know if Redis provides this information to clients.

Cluster support is added in ca206a7a88361c206fd89724e0a9c0e327858600 but it doesn't handle plain replication (single primary shard, many replicas). Redis doesn't call this "clustering" and none of the CLUSTER commands are available in this mode, so I'm not sure what the client abstraction for this particular deployment configuration should be.
And, realistically speaking, do Redis clusters exist in the wild that do simple replication without cluster-mode enabled?
I'm still also unsure that the current abstraction split between Client and Cluster is ideal. The Client maintains a connection pool but the Cluster maintains multiple connection pools. Should the Cluster instead maintain a collection of Clients rather than messing with its own connection pools? 🤔

Just wanted to add to this. Our redis env is using cluster mode. https://github.com/stefanwille/crystal-redis/issues/127 There's a docker-compose example in this issue that could be used for some local testing on this.

@jwoertink Thanks for jumping in here. If I’m reading that issue correctly[1], this shard already supports that particular type of Redis clustering. This issue is still open only to handle the plain-old-replication form of clustering (via REPLICAOF), which Redis confusingly doesn’t call a “cluster”. This shard does support reading from replicas in sharded clusters, though.
If you use this shard on your Redis cluster (using Redis::Cluster instead of Redis::Client), does it work? For example:
cluster = Redis::Cluster.new(REDIS_URL)

pp cluster.lrange(“thing”, “0”, “-1”)

(pardon the smart quotes in that code, I’m writing on an iPad)

[1] The MOVED error is raised when the client talks to the wrong Redis server in the cluster for that key, and is solved by the Redis::Cluster class running .

oh nice! I'll have to give that a shot and let you know how it goes. That's for the info :rocket:


[e2e]: maven workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8531084129
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.maven.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.maven.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Apr  3 00:10:47 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8541798951
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.maven.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.maven.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Apr  3 16:03:08 UTC 2024
Tests are passing now. Closing this issue.


Added custom cartridge properties multi-line text field to load cartridge

Added custom cartridge properties multi-line text field to load cartridge to support custom properties
e.g.
sonar.projectKey=adop
openshift.projectName=adop-dev
Cartridges can then parse this text field and be configured according to the provided properties. Cartridges that adopt approach will have greater  re-usability.
The library to consume these custom properties can be found here: https://github.com/RobertNorthard/adop-dsl-custom-cartridge-properties
I have also updated the skeleton cartridge as an example of how to use the library:
https://github.com/Accenture/adop-cartridge-skeleton/pull/6
Signed-off-by: Northard, Robert A<EMAIL_ADDRESS>

@RobertNorthard Thanks for the PR, happy to merge! Can you squash your commits please?

This is dependent on https://github.com/Accenture/adop-platform-management/pull/55

@RobertNorthard Can you rebase on master so latest changes that have gone in for the Jenkins upgrade are included?

@dsingh07 rebased.

@dsingh07 tested and ready to merge


update arcadia 2024 schedule

should be reviewed and merged after #78

I created my branch from his branch so i could link to his lesson in the overview. that allowed me to serve his lesson locally and check for formatting while i was reviewing his PR.


Add grpc reflection to grpc_json_transcoder.

Previously, the grpc_json_transcoder filter required a statically
generated set of protobuf descriptors to perform translation.
This commit adds the ability to fetch protobuf descriptors via grpc
reflection.
Note that, per discussion on #1182, this change entails a fairly major
restructuring of the filter's code.
In addition to unit tests, this PR was tested manually using greeter_server from grpc examples in the grpc repository.
Signed-off-by: Henry Qin<EMAIL_ADDRESS>

Open source code, at lease Envoy project,  requires code quality. It is NOT me that insisting on the way I want to do,  It is the code quality.
There is a big problem that I have commented and you did not reply,  the freeing of the callbacks.   Let me try to find that comment or re-comment it again.

BTW,  I like to see separate unit-test xyz_test.cc for the two new classes,  ProtoDescriptorBuilder and AsyncReflectionFetcher.   If they have dependency, mock their dependency.
Thanks

Open source code, at lease Envoy project, requires code quality. It is NOT me that insisting on the way I want to do, It is the code quality.

For the record, I disagree with the Envoy project's definition of code quality.
I believe understandability is more important than modularity, and subscribe to John Ousterhout's philosophy of deep abstractions to reduce complexity.
However, I will update the code to conform for consistency's sake.

There is a big problem that I have commented and you did not reply, the freeing of the callbacks. Let me try to find that comment or re-comment it again.

Thanks for finding it again and re-commenting. I also did not find the original one, maybe GitHub lost it?

BTW, I like to see separate unit-test xyz_test.cc for the two new classes, ProtoDescriptorBuilder and AsyncReflectionFetcher. If they have dependency, mock their dependency.
Thanks

Yes, I think you pointed this out in your comments on my original draft PR, so I separated it out in this one.
If there's something else I didn't separate out that you asked for, please remind!

@qiwzhang @adisuissa It'll probably take me quite a while to address all the review comments.
Should I close this PR or make it a draft or keep it open in the interim?

@qiwzhang @adisuissa It'll probably take me quite a while to address all the review comments, since some of the changes @qiwzhang requested are structural (Note that this is a personal project so I have limited time to work on it).
Should I close this PR or make it a draft or keep it open in the interim while I address the comments?

As long as the PR keeps the same spirit, and is not completely different, then we can keep it open in "waiting" state for a while. This will keep the previous comments.
If you think it is going to be significantly different, then you can open a new PR.
/wait

@qiwzhang I am still waiting on your opinion on this question before this work on this PR can proceed further.
I agree with you that it is an important problem, as you mentioned.

@hq6 I replied it.  Thanks.

@hq6 Overall this pr is good. Thanks for the hard work.  It just needs a bit of restructure in classes ProtoDescriptorBuilder and AsyncReflectionFetcher with proper unit-tests.   I don't see a big changes from this pr, so we can continue to use this pr.
I can sense some defensive tone in your comment.
Anyway,  this is a good feature to have.  Thanks for the hard work.

I can sense some defensive tone in your comment.

No offense taken, although I do object to any claims that there's one globally accepted definition of code quality today.
I currently believe that a majority of software developers today hold as a basic tenet that modularity is more important than ease of understanding, and I am part of the minority that disagrees.
I simply wanted to make it clear that this contribution to Envoy does not mean I endorse Envoy's school of thought about what code quality means. :)

Is this merge stuck? when is going to be released?

This PR cannot be merged until comments are addressed, and code coverage is up to the required level.
I've been busy with my day job, and haven't yet had time to address the comments.

@hq6 are you still working on this? Or has this been cancelled?

@jeffsawatzky I'm not actively working on this due to other commitments.
I was planning to try to get to it after tax season, but if someone else wants to take over the PR by addressing the remaining comments and fixing code coverage, I am not opposed.
For coordination reasons, it'd be good for such a person to comment on this PR or the related issue to avoid duplicated work.

@hq6 awesome work keep it up. This would be an extremely useful feature!


Handles ArrayList when parsing JSON

There's a (todo) in the source about not being able to parse JSON arrays because we use merge! to merge the JSON and the destination. Well, my new input source has an array as the base type, so I kind of need that functionality now :D.
Questions

Are there instances where parsed.kind_of?(Java::JavaUtil::ArrayList) would fail? Maybe because we weren't running inside of JRuby?
Is it okay to raise an error message like that? Still gets the normal parse failure tag and you get a little more insight as to why it's failing.

@jrgns event.to_hash.merge! did work.

@tebriel answered your question re: JRuby

@suyograo fixed and specs still pass, thanks!!
cmoultrie@Sauron ~/G/logstash-filter-json> bundle exec rspec
Using Accessor#strict_set for specs
Run options: exclude {:redis=>true, :socket=>true, :performance=>true, :couchdb=>true, :elasticsearch=>true, :elasticsearch_secure=>true, :export_cypher=>true, :integration=>true, :windows=>true}
............

Finished in 0.184 seconds (files took 0.883 seconds to load)
12 examples, 0 failures

Randomized with seed 41454

LGTM


Add reheat and ability to represent dehumidification process

Proposing to add commands, sensor, and equipment classes to support the dehumidification process in the Brick ontology. Specifically, we are introducing the following:

Reheat Command
Coiling Coil Supply Air Temperature (SAT) Sensor
Dehumidification Command

Why These Additions are Necessary
Reheat Command
The reheat command is needed for distinguishing the heating command needed to reheat the supply air post-dehumidification. Without a dedicated reheat command, it isn't possible to decouple the reheat operation from other heating operations within an HVAC system.
Coiling Coil SAT Sensor
The Cooling Coil SAT Sensor is used for monitoring the temperature of the supply air immediately after the cooling coil. This is used to modulate the amount of cooling used dehumidify the air before it is reheated to the target supply air temperature setpoint.
Dehumidification Command
Dehumidification Command is used to initiate the control sequence using the introduced concepts listed above.

I accidentally added Reheat SAT sensor. I corrected it with "Cooling Coil Supply Air Temperature Sensor"

This looks good for the most part, but I want to talk a little more about the Cooling Coil SAT Sensor. In Brick we are trying to get away from having the names of the equipment inside the point names. If the SAT sensor is immediately after the coil, then we should find a way of representing that explicitly within the model, e.g. by relating it to an instance of the cooling coil.
Doing this properly means we'll have to put a Sensor Equipment right after the Cooling Coil. Maybe the way we capture this concept is with a shape?
bsh:Cooling_Coil_Supply_Air_Temperature_Sensor a sh:NodeShape, owl:Class ;
  sh:class brick:Supply_Air_Temperature_Sensor ;
  sh:property [
    sh:qualifiedMinCount 1 ; sh:qualifiedMaxCount 1 ;
    sh:path brick:isPointOf ;
    sh:qualifiedValueShape [ sh:node bsh:CoolingCoilDownstreamSensor ]
]
  sh:rule [
    a sh:TripleRule ;
    sh:subject sh:this ;
    sh:predicate rdf:type ;
    sh:object brick:Supply_Air_Temperature_Sensor ;
 ]
.

bsh:CoolingCoilDownstreamSensor a sh:NodeShape, owl:Class ;
  sh:class brick:Sensor_Equipment ;
  sh:property [
    sh:qualifiedMinCount 1 ; sh:qualifiedMaxCount 1 ;
    sh:path brick:isFedBy ;
    sh:qualifiedValueShape [ sh:class brick:Cooling_Coil ]
] ;
.

In your model, you could instantiate it as follows:
:ahu a brick:AHU ;
  brick:hasPart :cc, :se .

:cc a brick:Cooling_Coil ;
  brick:feeds :se .

:se a brick:Sensor_Equipment ;
  brick:hasPoint :sat .

:sat a bsh:Cooling_Coil_Supply_Air_Temperature_Sensor.

What do you think? The CCSAT Sensor has a rule which adds the correct Brick point class to the instance


Ability to stop Listerners based on Binding Target

Hello,
We have a use case where we have multiple binding on the same destination using different target name. Would it be possible to enhance changeState to take target name and stop the listeners ?

This has already been addressed in 2.2.0. Next time please specify which version you are referring to.

I am currently on ..
springBootVersion = '2.1.6.RELEASE'
springCloudVersion = 'Greenwich.SR1'.
What do i need to do, to get the new version so i can start using requested feature ?

Simply override spring-cloud-stream version to 2.2.0.RELEASE

Thank you for the details. I am not seeing what I am expecting. May be my questions was not clear to begin with.
Below is my rabbit mq binding definition, which listens to same exchange twice - each listener has specific header. When I try to stop the listener - I can only do it by name - which is always the destination / exchange. I am requesting a change to BindingEndpoint to allow stop/start by Binding Target in addition to Binding Name.
Can you help me with  making sure this is the feature added to 2.2 and how to achieve this behavior ?

  cloud:
    stream:
      bindings:
        consumer-input-primary:
          destination: exchange.test
          content-type: application/json

        consumer-input-secondary:
          destination: exchange.test
          content-type: application/json

Can someone help with the question ? If more details are needed, I can gladly write detailed explanation and share more details on coding.

Can you please look at this and let me know what do you need/expect beyond what's described in the issue. Sorry, but I am not following.

Yes. This is exactly what I am looking for.
Here is what I have in bindings endpoint. When I try to run command curl -d '{"state":"STOPPED"}' -H "Content-Type: application/json" -X POST http://localhost:8081/actuator/bindings/consumer-input-secondary  - it does not seems to stop the listener. Am missing something ?
[
{
"bindingName": "consumer-input-secondary",
"name": "federated.test",
"group": null,
"pausable": false,
"state": "running",
"input": true,
"extendedInfo": {
"bindingDestination": "RabbitConsumerDestination{queue=federated.test.anonymous.iA9PCUZUSoWFALRFGUIdqg, binding=Binding [destination=federated.test.anonymous.iA9PCUZUSoWFALRFGUIdqg, exchange=federated.test, routingKey=#]}",
"ExtendedConsumerProperties": {
"autoStartup": true,
"concurrency": 1,
"instanceCount": 1,
"maxAttempts": 3,
"backOffInitialInterval": 1000,
"backOffMaxInterval": 10000,
"backOffMultiplier": 2,
"defaultRetryable": true,
"extension": {
"exchangeType": "topic",
"declareExchange": true,
"exchangeDurable": true,
"exchangeAutoDelete": false,
"delayedExchange": false,
"queueNameGroupOnly": false,
"bindQueue": true,
"bindingRoutingKey": null,
"ttl": null,
"expires": null,
"maxLength": null,
"maxLengthBytes": null,
"maxPriority": null,
"deadLetterQueueName": null,
"deadLetterExchange": null,
"deadLetterExchangeType": "direct",
"declareDlx": true,
"deadLetterRoutingKey": null,
"dlqTtl": null,
"dlqExpires": null,
"dlqMaxLength": null,
"dlqMaxLengthBytes": null,
"dlqMaxPriority": null,
"dlqDeadLetterExchange": null,
"dlqDeadLetterRoutingKey": null,
"autoBindDlq": false,
"prefix": "",
"lazy": false,
"dlqLazy": false,
"overflowBehavior": null,
"dlqOverflowBehavior": null,
"transacted": false,
"acknowledgeMode": "AUTO",
"maxConcurrency": 1,
"prefetch": 1,
"txSize": 1,
"durableSubscription": true,
"republishToDlq": false,
"republishDeliveyMode": "PERSISTENT",
"requeueRejected": false,
"headerPatterns": [
"*"
],
"recoveryInterval": 5000,
"exclusive": false,
"missingQueuesFatal": false,
"queueDeclarationRetries": null,
"failedDeclarationRetryInterval": null,
"consumerTagPrefix": null,
"frameMaxHeadroom": 20000,
"containerType": "SIMPLE",
"anonymousGroupPrefix": "anonymous.",
"requestHeaderPatterns": [
"*"
]
}
}
}
},
{
"bindingName": "consumer-input-primary",
"name": "federated.test",
"group": null,
"pausable": false,
"state": "running",
"input": true,
"extendedInfo": {
"bindingDestination": "RabbitConsumerDestination{queue=federated.test.anonymous.Ajo1GHWaR-W2az8MfcqMwA, binding=Binding [destination=federated.test.anonymous.Ajo1GHWaR-W2az8MfcqMwA, exchange=federated.test, routingKey=#]}",
"ExtendedConsumerProperties": {
"autoStartup": true,
"concurrency": 1,
"instanceCount": 1,
"maxAttempts": 3,
"backOffInitialInterval": 1000,
"backOffMaxInterval": 10000,
"backOffMultiplier": 2,
"defaultRetryable": true,
"extension": {
"exchangeType": "topic",
"declareExchange": true,
"exchangeDurable": true,
"exchangeAutoDelete": false,
"delayedExchange": false,
"queueNameGroupOnly": false,
"bindQueue": true,
"bindingRoutingKey": null,
"ttl": null,
"expires": null,
"maxLength": null,
"maxLengthBytes": null,
"maxPriority": null,
"deadLetterQueueName": null,
"deadLetterExchange": null,
"deadLetterExchangeType": "direct",
"declareDlx": true,
"deadLetterRoutingKey": null,
"dlqTtl": null,
"dlqExpires": null,
"dlqMaxLength": null,
"dlqMaxLengthBytes": null,
"dlqMaxPriority": null,
"dlqDeadLetterExchange": null,
"dlqDeadLetterRoutingKey": null,
"autoBindDlq": false,
"prefix": "",
"lazy": false,
"dlqLazy": false,
"overflowBehavior": null,
"dlqOverflowBehavior": null,
"transacted": false,
"acknowledgeMode": "AUTO",
"maxConcurrency": 1,
"prefetch": 1,
"txSize": 1,
"durableSubscription": true,
"republishToDlq": false,
"republishDeliveyMode": "PERSISTENT",
"requeueRejected": false,
"headerPatterns": [
"*"
],
"recoveryInterval": 5000,
"exclusive": false,
"missingQueuesFatal": false,
"queueDeclarationRetries": null,
"failedDeclarationRetryInterval": null,
"consumerTagPrefix": null,
"frameMaxHeadroom": 20000,
"containerType": "SIMPLE",
"anonymousGroupPrefix": "anonymous.",
"requestHeaderPatterns": [
"*"
]
}
}
}
}
]

I can't seem to reproduce it, so can you please create a small sample and push it to github so one of us can take a look by running it locally.

https://github.com/kishorekkota/test-mq
Here is the repo.
It has an exchange 'hello-exchange' listening on the local rabbit mq.
If try to stop using binding name - does not work.
curl -d '{"state":"STOPPED"}' -H "Content-Type: application/json" -X POST http://localhost:8080/actuator/bindings/second_listener
Only exchange name works...
curl -d '{"state":"STOPPED"}' -H "Content-Type: application/json" -X POST http://localhost:8080/actuator/bindings/hello-exchange
actuator URL http://localhost:8080/actuator/bindings
I could not try to get it working. Please suggest how i can enable this option.

Thanks a lot.


Manual segment refresh should be done by oro:message-queue

Hi Oro Team,
Since refreshing a manual segment can be a long running task, we think it should be processed by a background task with oro:message-queue:process
We already have an implementation ready but we are missing a small key piece. We need to be able to notify the end-user that the message queue has finished.
How can we send a flashBag notification to the user from the message processor ?
Related forum post :  https://forum.oroinc.com/orocrm/orocrm-how-do-i-questions/topic/send-message-from-message-queue-processor-to-ui
Thanks,
Samuel

Hey @sadortun
The Async DatagridExport sends an email to the user.
https://github.com/oroinc/platform/blob/master/src/Oro/Bundle/ImportExportBundle/Async/Export/PostExportMessageProcessor.php#L137
But maybe it's similar like the outdated content flash message?
https://oroinc.com/oroplatform/doc/current/dev-cookbook/websockets/content-outdating-notifications#dev-cookbook-system-websockets-content-outdating-notifications
Eric

Thanks @RocKordier,
Indeednelwenuse the email example to transmit data from the app to the queue.
For the other direction ( queue to app ), how do you see the usage of the websocket ? We look into it but I dont see how to use that from the message queue.
Thanks,
Samuel

Here is an example shared by Mario Prud'homme at the ORO community slack some time ago:
 $connectionChecker = $this->getContainer()->get('oro_sync.client.connection_checker');
 $websocketClient = $this->getContainer()->get('oro_sync.websocket_client');

 var_dump($connectionChecker->checkConnection());
 var_dump($websocketClient->publish('oro/ping', ['test' => 123]));


docs: fix version notice

Minor documentation change.
Change the version compatibility notice.

Closing. Most likely out of date.


Running the plutus benchmark fails due to dependency conflicts

Summary
I am trying to run the plutus benchmark described in plutus-benchmark. The suggested cabal command fails due to conflicting dependencies.
Steps to reproduce the behavior

cabal bench plutus-benchmark:validation --minimize-conflict-set

Actual Result
➜  plutus git:(master) ✗ cabal bench plutus-benchmark:validation --minimize-conflict-set
Warning: Requested index-state 2022-11-15T00:00:00Z is newer than
'hackage.haskell.org'! Falling back to older state (2022-11-14T23:53:01Z).
Warning: Requested index-state 2023-02-23T00:00:00Z is newer than
'cardano-haskell-packages'! Falling back to older state
(2023-02-22T19:09:40Z).
Resolving dependencies...
cabal: Could not resolve dependencies:
[__0] trying: plutus-core-<IP_ADDRESS> (user goal)
[__1] rejecting: plutus-core:-with-inline-r (constraint from config file,
command line flag, or user target requires opposite flag selection)
[__1] trying: plutus-core:+with-inline-r
[__2] next goal: inline-r (dependency of plutus-core +with-inline-r)
[__2] rejecting: inline-r-<IP_ADDRESS>.<IP_ADDRESS> (conflict: pkg-config package
libR>=3.0, not found in the pkg-config database)
[__2] rejecting: inline-r-1.0.0, inline-r-0.10.5, inline-r-0.10.4,
inline-r-0.10.3, inline-r-0.10.2, inline-r-0.10.1, inline-r-0.10,
inline-r-0.9.2, inline-r-0.9.1, inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>,
inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>,
inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>, inline-r-<IP_ADDRESS>,
inline-r-<IP_ADDRESS> (conflict: plutus-core +with-inline-r => inline-r>=<IP_ADDRESS>)
[__2] fail (backjumping, conflict set: inline-r, plutus-core,
plutus-core:with-inline-r)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: plutus-core,
plutus-core:with-inline-r, inline-r

Expected Result
Benchmark runs as intended
Describe the approach you would take to fix this
Fix the dependencies?
System info
OS: Ubuntu 22 LTS
Commit: https://github.com/input-output-hk/plutus/commit/b77e82c1a42e48b098627e2c2274c107ac3078da

Cabal is telling you the problem:

conflict: pkg-config package libR>=3.0, not found in the pkg-config database

We recommend using Nix to avoid these kinds of problem.


Multiplayer support

What about doing something with RealtimeMultiplayerNodeJs?
If it sounds like a good idea, I'm up for doing some/all of the work.
FYI: I've cross-posted a similar issue just to get some more feedback.

https://github.com/olahol/melody-jsnes :)


Add a way to specify blackbox modules on the surelog command line

For hierarchical synthesis it would be useful to specify certain modules as black boxes.
Something like surelog -blackbox a b c d ...

@QuantamHD, please try #3022.
My suspicion is that it is not the whole solution. We probably have to have the interface to the blackbox (ports, parameters) dump in the UHDM db to be able to do the stitching. But let me know.

Open another issue if need be. But this looks like this is fixed in the current use case.


Linked notion page in README.md

After reading about AWS-vault, I think it's pretty self explanatory and there isn't much more that I can add besides a link to their github


Include ineffassign in CI, fix ineffassign in code

Adds the ineffassign target to our Makefile and circle tests, indirectly part of #815
Signed-off-by: Riyaz Faizullabhoy<EMAIL_ADDRESS>

LGTM!  Thank you for adding this!

Non-blocking, but wondering if it'd be worth adding a full-lint endpoint that just combines all of these?  That way we can run all the lints that are run on the CI with one command.

Agree on adding a full-lint (could probably just be lint ??)
LGTM though. Merging


Fade shimmer

Hi @IbrahimTabba , thanks for your PR. You can provide example codes for your work and the screenshot for it in README as well.


Allow spaces in plugin path

This necessary if you (for example) have your vim config symlinked to a DropBox path where one of the folder names has a space in it.

Thanks @adamlogic.


Build(deps-dev): Bump @babel/core from 7.17.8 to 7.17.9 (#3293)

Pull request opened by the merge tool on behalf of #3293

Merged in e9f72636c8902e33513e91d45a6546d424f990bd


Example doesn't work

Hi!
I tried using the example code to fetch arrival times at Christopher St. station with this code:
var Mta = require('./');
var mta = new Mta({
  key: '{{my key here}}', // only needed for mta.schedule() method
  feed_id: 1                  // optional, default = 1
});

mta.schedule(133, function (err, result) {
  console.log( err, result );
});

This produces
> $ node example.js
[TypeError: Cannot read property 'build' of null] undefined

I notice if I remove this line, it works. The file lib/data/proto/nyct-subway.proto is included in the project, so I wonder if the reference to node_modules/mta-gtfs is an artifact?
Love the package btw! :smile:

Ah — okay.
The node_modules/mta-gtfs bit is there so that the mta module can be used by another node project.
I think we could use __dirname here, which will give the absolute folder path of the script being executed, rather than expecting the package to be used as a module, therefore inside of the node_modules/mta-gtfs folder.
var file = path.resolve( __dirname + '/../data/proto/nyct-subway.proto' );

Fixed! Thanks @ericandrewlewis for finding this!


Variable scope vs Thread/Runspacepool

Describe the Bug
Hi,
we got some weird issues when we activate the -Threads parameter with value greater than 1, when starting Pode Server. After searching, we identified the problem to be related to local scope variable use. With runspacepool, the variable goes from initialized to unitialiazed to back to and old value. The issue is explained very well in this article https://learn-powershell.net/2018/01/28/dealing-with-runspacepool-variable-scope-creep-in-powershell/
I search in the pode src code but haven't found how to handle this as of now.
Steps To Reproduce
Steps to reproduce the behavior:

Import a module
In your module, define a function with a local variable in the begin section and assign a value
In the process, use this variable and change the value. Write the value to the shell and see how the content of the variable change from initialized to unitialized depending witch thread handle the call...

Expected Behavior
Would like to have the option to respect the UseLocalScope if possible

I was often concerned this kind of a bug existed, and thanks for the link!
This hopefully looks like something quick to resolve by changing https://github.com/Badgerati/Pode/blob/aa0548338f52104940fe66f76d3ce3a9a9eaaa72/src/Private/Helpers.ps1#L642
to:
$ps.AddScript($ScriptBlock, $true) | Out-Null

🤔

Hi Badgerati,
I tested with your suggestion but it's not working. It's a lot worse. I mean, pode start but nothing works. A simple request just never reach the routes as if the whole server is not listening at all. It looks like Pode code has something not compatible with the uselocalscope = $true parameter.
I dig futher and did a simple test to search for a solution and that's very strange. I use ActiveDirectory cmdlet in one of my route (trying to return a simple user information). The cmdlet Get-Aduser $username -properties mail return the mail property on the first call/thread. The subsequent call return the aduser without the mail property. And when I get back on the first thread (recall until you get back on the first thread), the mail property is filled again. But on all thread the distinguishedname is there and all right.
Thread 1: VERBOSE:  ad user CN=username,OU=OrgUnit,DC=contoso,DC=com [mail=<EMAIL_ADDRESS>found!
Threads X: VERBOSE:  ad user CN=username,OU=OrgUnit,DC=contoso,DC=com [mail=] found!
I also notice you have the -Now parameter on Import-PodeModule function, but I tried to use this parameter with no difference. I see in the Import-PodeModule that you load the module with -Global with that parameter. Thought that maybe it may load the module differently.
If I try with another module function of our own. The first thread is working but all other threads have no result return... very strange. It behave like ad module.
If you have some idea, let me know. Thanks
Danny

Hi Danny,
That might make some sense actually, the UseLocalScope could be blocking variables set within the Session State of the Runspaces - if these variables can be set as global then it could work.
On the -Now, that just loads the module into the current scope as well - rather than just the runspace scopes.
As for Get-Aduser $username -properties mail, that is a weird one! I can't actually see why that would work on one thread, and then not work on others 🤔 It doesn't seem like it could be the thread-scoping issue, but possibly something else.
Would you be able to put together a simple script that can reproduce that? Something I can test with to try and fix.
Thanks

Hi Matthew,
My test pode server with TestDirectory module
the url for the test:
GET http://server.fqdn.com:8080/api/user/useridentity
Accept: application/json
pode_bug_513.zip
output of multiple request:
[PS] ps01->C:\util\psh\danny> cd test
[PS] ps01->C:\util\psh\danny\test> pode start
Listening on the following 1 endpoint(s) [4 thread(s)]:
- http://localhost:8080/
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=<EMAIL_ADDRESS>found!
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=<EMAIL_ADDRESS>found!
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=  cip=] found!
VERBOSE:  revert cip to identity [cip:danny9301]
VERBOSE: Get-LDAPPersonne danny9301
VERBOSE: Get-AdUser danny9301
VERBOSE:  ad user CN=danny9301,OU=OrgUnit,DC=contoso,DC=com [mail=<EMAIL_ADDRESS>found!
VERBOSE: Get-LDAPPersonne danny9301
Terminating... Done
Thanks :-)

Oh by the way. I'm running it on Windows Server 2019 with Powershell 5.1

Thanks!
I had to use CN to test, but was able to reproduce. I've a feeling this one could be an AD Module issue, as the second I added the following to your config.ps1 the issue went away:
Import-PodeModule -Name ActiveDirectory
Import-PodeModule -Name TestDirectory

Does that do the same for you?

Hi,
Yes it's working. So simple solution. I understand that I need to explicitly load built in modules...
Will continue doing more test tomorrow but sound like it's all fine.
Thanks for your help.

It works Thank again


can't render position fixed element in nodeTreeToSketchGroup

filter function enter 2nd, 3rd arguments to the callback.
https://developer.mozilla.org/ko/docs/Web/JavaScript/Reference/Global_Objects/Array/filter
So, the isNodeVisible in nodeTreeToSketchGroup didn't work correctly.

@kdzwinel
Please review and merge.😭

I need this PR..😭

This was released with 4.0.1 . Thank you ❤


[EN-7031]: feat: deleted user cannot login

On retourne maintenant le champ deletedAt?
Le test ne passe plus du coup


Modal Window

![screen shot 2016-01-20 at 12 42 52 am]
(https://cloud.githubusercontent.com/assets/11501546/12425492/ff23a1da-bf0f-11e5-9134-75d58bad34ce.png)
:target pseudo-class
z-index property

Ugly solution http://codepen.io/jsstrn/pen/Rrjdzr

http://codepen.io/lanbau/pen/eJeoYP?editors=110

Simple Solution: http://codepen.io/sevresbabylone/pen/adVavg

http://codepen.io/tongrhj/pen/BjmbGQ


Display user score on profile page

#114 fix
User can view their score on their profile page now

@just-ary27 Done with the changes .


Update readme

This info could prevent some recurring issues.

@itzsaga Thanks! bower install is actually in the bin/setup script so I'm wondering if it isn't working on windows atm? Do you know?
See here: https://github.com/nielsenramon/chalk/blob/master/bin/setup#L15
Running bin/setup should be enough.

It didn't work for me on my machine. This is how I had to do it on my Windows 10 machine.

@itzsaga Added differently on the master branch. Thanks for bringing it up!


Sprint 18 functional testing

Test the improved View API backed, which now has a tabbed interface, on nightly.apinf.io.
Definition of done

[x] Go throught the different views: Details, Metadata, Feedback, Backlog, optionally also Export. (If possible, in more than one browser)
[x] If there is an API you have added and hence you are the manager, add metadata for that API.
[x] If there is an API you have added and are the manager addone  backlog item for that API.
[x] Select another API (one which you are not manager for) and create one feedback item for it.

@ashakunt Any updates on Apinf functional testing?

Blocked by #836, i.e. nightly being down.

@bajiat @saralavanip nightly.apinf.io up again.

@jykae Thanks for fixing the nightly!

Please find below issues reported:
https://github.com/apinf/api-umbrella-dashboard/issues/852
https://github.com/apinf/api-umbrella-dashboard/issues/851
https://github.com/apinf/api-umbrella-dashboard/issues/853
https://github.com/apinf/api-umbrella-dashboard/issues/854
https://github.com/apinf/api-umbrella-dashboard/issues/855
https://github.com/apinf/api-umbrella-dashboard/issues/856
https://github.com/apinf/api-umbrella-dashboard/issues/857
https://github.com/apinf/api-umbrella-dashboard/issues/865
https://github.com/apinf/api-umbrella-dashboard/issues/867


spec: fixed another flaky test

Fixes #1010
Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

This hopefully fixes another flaky test :wink:

The failure doesn't seem related to this change, but I'll investigate later.

In those days I remember that Travis CI was having some problems. Now it all works. I've triggered a re-build multiple times and everything works. I'd said it's good to go :smile:


Added IRecurring interface, Builder methods for DateTime & tests

I added an IRecurring interface to allow me to build up a RecurrenceType without knowing what type it is and still be able to call Next(DateTime) on it.
This could be used when starting with an EveryBuilder but then calling Months() or Weeks() depending on some other variable. Then you get a LINQ-y syntax because you don't have to declare the instance type to be able to call Next after calling Build.
I also added a method OnDay(DayOfWeek day) to the WeeksBuilder and OnMonth(DateTime dateTime) to the YearsBuilder to give more integration with DateTime objects, which is what I'm finding I have on hand when I'm using this library.
Finally I changed the On(Day day) method for the WeeksBuilder to be OnDays which brings it in line with the naming conventions for the other Builders.
I updated the tests accordingly.

I updated the test.
Thanks for your work on this - it's definitely helped me in a project I've been working on and I hope some other people get some use out of it.

No worries and thanks for your contributions :+1:


Unable to open file

i followed the exact instructions as on README. I am running OSX.
The adapData is empty for some reason so it coming as undefined while running

fixed, can you retry?
can you check whether you can download the problem list at http://uhunt.felix-halim.net/api/p


EDIF backend produces a file with multi-driven net.

Steps to reproduce the issue
I have a SoC design with a VexRiscv core that when synthesized with the latest Yosys (Yosys 0.9+1706 (git sha1 0ec97144, clang 8.0.0 -fPIC -Os) fails in Vivado during EDIF parsing:
Parsing EDIF File [./design.edif]
WARNING: [EDIF 20-100] Multiple nets with the same name 'GND_NET' specified. Folding connectivity of these nets into a single net. [/work/test2/zephyr_broken_edif/build/design.edif:27427]
ERROR: [EDIF 20-80] Cannot connect net 'IBusCachedPlugin_predictionJumpInterface_payload[0]' to pin '$auto$simplemap.cc:420:simplemap_dff$7519/Q' in cell 'VexRiscv'. This pin is already connected to net '_zz_115_[0]'. The new connection will be ignored. [/work/test2/zephyr_broken_edif/build/design.edif:68684]
Finished Parsing EDIF File [./design.edif]

I attached a test case:
zephyr_broken_edif.zip
Simply run with make yosys
I ran a git bisect on the current master and it pointed me to the following commit:
  commit 50d70288d0f1b516dc04f38494eea94612104974
Author: Claire Wolf<EMAIL_ADDRESS>Date:   Wed Jan 29 14:07:11 2020 +0100

    Preserve wires with keep attribute in EDIF back-end

    Signed-off-by: Claire Wolf<EMAIL_ADDRESS>
Expected behavior
Vivado should successfully parse the EDIF and continue with P&R.
Actual behavior
When inspecting the design.edif I can clearly see that we have a multi-driven net:
...
          (net (rename id09476 "IBusCachedPlugin_fetchPc_pcReg[0]") (joined
            (portRef (member io_cpu_fetch_pc 31) (instanceRef IBusCachedPlugin_cache))
            (portRef D (instanceRef id04838))
            (portRef Q (instanceRef id05980))
            )
          )
...
          (net (rename id09507 "_zz_115_[0]") (joined
            (portRef (member io_cpu_decode_pc 31) (instanceRef IBusCachedPlugin_cache))
            (portRef D (instanceRef id05515))
            (portRef Q (instanceRef id04838))
            )
          )
...
          (net (rename id12281 "IBusCachedPlugin_predictionJumpInterface_payload[0]") (joined
            (portRef Q (instanceRef id04838))
            )
          )

Your test case is broken:
ERROR: Can't open input file /home/tmichalak/Development/U-Ray/prjuray2/minitests/litex/zephyr/zcu104/src/top.v' for reading: No such file or directory`

@mwkmwkmwk just in case I updated the original test case
zephyr_broken_edif.zip

Trying my reduced example in ISE (after fixing up the attributes) to get some idea what to do:
WARNING:Xst:638 - in unit top Conflict on KEEP property on signal a and b b signal will be lost.

So it seems that the correct solution to multiple aliased signals with the keep attribute could be "don't do that".


Functions to specify colors in themes

Themes now can define using a string with a function, for example, hsb(210, 1, 0.3) or rgb(255, 0, 218), the methods available are rgb, hsl, hsv, hsb and its respective alpha variants.
Also, a change is introduced in the gradients API and the coordinates of the gradients are made relative to the path.

It is related to https://github.com/redox-os/orbtk/issues/174 ?
You should mention it.

It is related to #174 ?
You should mention it.

Nope, this is only preparing the ground, I am dividing my work into multiple PRs and organizing them by parts, this has nothing to do with that issue, this only allows to use a few functions to add colors, of course in future PRs those functions can be expanded to add things like gradients, but here there's none of that except the fact that gradients are made relative


Support For SPM

As SPM is officially part XCode 13+, Is there any timeline for the same

Hi @ajjumehra ,
Thanks for raising this issue. As we can develop application using Swift language we have introduced Contentstack-swift SDK that you can include through the SPM.
Please, let us know if this is helpful and works for you.


Status check to endpoint failing

I am running via docker container.
After adding the end point status check is failing:

But if TTY into the container i am able to issue ping to the endpoint.

Which docker container did you download, the OSS or Pro version?

Also, what is the end point type, windows or linux?

I used this image quay.io/ortelius/ortelius. Is this correct for OSS?
Endpoint is a linux RHEL vm.

Go ahead and download the latest container image.  We had found a similar issue in the Pro version that was fixed but not pulled into the OSS version.  I pulled the changes into OSS.  Also, the .ssh keys will be accessible from /home/omreleng/.ssh/.  They get copied from the host to /keys in the container and then into /home/omreleng/.ssh in the container.
Let me know the results.

I took the latest image and it's still failing,
Though i had found a workaround, when i run the docker container like this:
docker run -u root <rest_of_command>
then i am not facing this issue.

Another issue has popped up after updating to latest image, base directory check is failing.

What type of credential are you using to access the remote end point?   I tested my side with a public/private key credential and it was successful.  Send me a screen shot of the end point definition (ie general tab).

I am using a username/password based login. The VM is in a private cloud and is accessible from inside the container.

What was the result that you were getting when you ran the docker container without -u root?  Also, is your docker daemon running as root or a different userid?

Without -u root all the status checks are failing.
Docker daemon is running as root.

I will need to debug why -u root is needed on a web meeting on your machine since I cannot recreate it.

Updated the Status check to show the underlying OS error.  Please pull the latest mage.
docker pull quay.io/ortelius/ortelius

Error:
Base directory "/opt" is invalid or inaccessible /opt/deployhub/engine/lib/sshexec: line 84: //.ssh/known_hosts.unique: No such file or directory /opt/deployhub/engine/lib/sshexec: line 85: //.ssh/known_hosts: No such file or directory cat: //.ssh/known_hosts: No such file or directory /opt/deployhub/engine/lib/sshexec: line 88: //.ssh/known_hosts.unique: No such file or directory /opt/deployhub/engine/lib/sshexec: line 89: //.ssh/known_hosts: No such file or directory grep: //.ssh/known_hosts: No such file or directory /opt/deployhub/engine/lib/sshexec: line 97: //.ssh/known_hosts.unique: No such file or directory /opt/deployhub/engine/lib/sshexec: line 98: //.ssh/known_hosts: No such file or directory Could not create directory '/.ssh'.

Hi, issue has been fixed by reinstalling Docker.

Do you still need to pass with -u root after reinstalling Docker?

No, not required anymore.


feat: fr translations

i18n: fr translations.

Preview and edit are mixed up - could become very confusing for French sidekick users :)

Nice catch! Your french is pretty sharp ;)

:tada: This PR is included in version 3.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Async exec

Uses an async queue to process haml files. It shoud be good enough for our use case, although it stores all file contents in memory while waiting in the queue (better than a fork bomb I guess).
I added a non-standard parameter civisConcurrency just in case we want to fine-tune it, but I think 2 ruby processes is enough.

Oops, this doesn't belong here