Test containers

My original idea was to make another totally unrelated change.
But I saw I could not run most test without installing every engine manually, and I
didn't like the idea.
I found this wonderful project TestContainers, used for other persistence libraries.
Basically it automatically starts and stops an instance of a docker virtual machine
with whatever database you need. I'm not docker-savvy, but it worked with docker for windows,
instructions were pretty clear and so I give it a try. And it worked.
You still need to install docker (for windows if that's your OS), but it's waaaay less work than
installing 5 database engines.
The first time a test for a specific database runs it need to download the full image
and that make take some time, but after that it reuses the same image and they are pretty fast.
161 tests run in 300 seconds on my laptop
What I did:
I migrated all tests to junit4, since is the minimal version supported by testContainers
I separated the tests in 3 categories

The quick tests that run in memory databases plus sqlite. I didn't touch them, they don't need to be installed.
The TestContainerDB category that run using test containers and cover almost every other database.
The ExternalDB category that runs with a working external DB. The only class left in that category is TestOracle,
because I couldn't make it work.

You can choose what category to run from command line, I wrote the instructions in the readme.md file.
I refactored the tests one by one, using TestContainers docker based databased and most test worked after a few changes
The changes

MySql tables and columns are case sensitive or not depending on the file system. In Windows they are case insensitive, in
linux (like docker images) are case sensitive. I edited your script for creating tables for MySql, basically making
all tables names PascalCase.
There was a bug in the method UtilsForTests.isProcedureInDatabase, fixed.
I needed to separate the two MSSQL tests in two classes, on for the MS driver, one for the jtds driver.
jtds returned a null clob when there was null data! A simple if solved the problem.
In some cases the insert call to executeUpdate throw an exception! Something like "there is a record and should not be". I solved it by changing from executeUpdate to execute.
executeCommand methods in BaseTest are now static because I needed them.
TestContainers database are empty, in order to run tests that included Pubs or Northwind databases I needed to create them first, and then reverse engineer your extra columns.
In some cases I was able to create the tables just once (and make them faster), in other cases I needed to recreate
them on every test method.

Finally I don't know If I broke oracle, I find no way to test it.
I think there is still room for improvement:

Every TestContainer DB starts form zero, so there is no need to clean up anything. That may speed things up.
If all tests inside a class can run without restarting the database, it will be faster.
Probably the logging of every creation statement is slowing down the whole thing.
Maybe create a basic battery of tests for every engine using Pubs and Northwind, and then add
specific tests for the non-standard dataypes?
there is a lot of copy/paste between Test clases, could be removed.
Every once in a while a got an OOM, or a "The forked VM terminated without properly saying goodbye. VM crash or System.exit called?"
message. I think I'm killing my machine by starting/stopping multiple database engines, but maybe there is something else.

Anyway, let me know what you think, and please try in you machine!
PS:
Persism.iml files, and the whole folders .idea and target should be included in .gitignore. There are not needed  because
idea regenerated them on demand, and are filesystem dependent. I didn't change them because I didn't want to
make more changes in the PR.
Again, let me know what you think!

Thanks! This is awesome.  I'll have a look this weekend and coordinate.

Line 79 in my todo.txt file. Great minds think alike.

Oof. Docker and VBox don't play nice together. I'll need to figure this out.

I have trouble with Docker since it now requires HyperV to be enabled or it doesn't play nice with Virtual Box. I tried running the older Docker Toolbox version but it doesn't work.
I integrated your tests.  We can tidy this up over time.
POM
changed oracle back to ojdbc8 which should be fine running under java8 or +
fixed jaybird warning
added profile for running without test containers
Added back suites to AllTests (Left @Ignore for Maven) - I still need AllTests from IDE to generate code-coverage
Added subclasses for Container versions - please test these
Marked JTDS for deprecation - I should remove it because it's no longer maintained
Added .idea to .gitignore and removed from git


Where is the info about how to use drag and drop?

back-link: ch04/index.html#mousearea-element
I looked everywhere in this chapter and expected drag and drop to either be covered under MouseArea or under Advanced Techniques (not yet written). I am having real trouble trying to find useful documentation on how to use drag and drop in the way I need to use it. It is documented well enough for Widgets/C++, but I am trying to do this with QML and having trouble figuring out the parallel operations I want.
I include below some examples of what I can't find out or what was very difficult to figure out, so that you can consider covering them when you write about drag and drop:

pixmap is not a property of Drag, so how do I set the little picture that should be part of the drag cursor? [Note: I am only copy-dragging, so the original never moves!]

I haven't got my head around the paradigm of drag and drop yet. I don't understand what must be happening in the background so that it works. So that means I am unsure of my role (as programmer) to set it up correctly. It is really hard for me to code for a paradigm I don't understand.

Most of the examples are using MouseAreas which are both draggable and include a DropArea. In other words, they are dragging onto other instances of themselves. This tends to muddy the waters - I get all mixed-up in my head about what is important for the item that is being dragged and what is important for the place where it is being dropped (and how they communicate with each other) since the code is all mixed up together in the same Item. You see, I have an Item which is more like an "icon" (e.g. representing some generic shape, such as a circle) that needs to be copy-draggable. When dragged onto a completely different type of item (sort of a canvas), it needs to create an "instance" (copy) of that generic shape when it is dropped. In my case, the instance will look and be internally quite different from the original "icon" I "dragged". I am only dragging enough information (e.g. the name of a .qml file) to enable the DropArea to generate the "instance" for me. The original "icon" is otherwise irrelevant (other than providing the name of the necessary .qml file, id, or whatever I will need to generate my "instance").

The current docu in this regard is sparse at best. For example, it tells me there is a property for storing mimeData, but does not tell me what it is, why I would need it, or how to set it up. I needed to go to multiple Widgets examples to figure out what it was for and how to set up my own mime type. I am currently storing just a string of text (e.g. the name of a .qml file) but as an application-specific mime type. Furthermore, there seems to be an implicit connection between the keys and the mime type in the mimeData. I was using a custom mime type as my key. If the keys property on both ends match, but that key didn't match the actual mime type in the mimeData (for test purposes, I was trying to use text/plain), the DropArea ignored the dragged object! What's up with that?

It is also not at all clear what the difference is between the dragTypes Drag.Automatic and Drag.Internal. All I have been able to figure out is that, depending on the example file, sometimes one works, sometimes the other, seldom both, but no clue why. My current code only works with Drag.Automatic.

Anyway, those are some of the points to consider when trying to explain the conceptual working of drag and drop. Thank you!

Thanks for the input. Drag and Drop would be a good topic for an Advanced section. No promises:-)

Added this to the chapter queue. I'm not sure if it is a chapter of its own, or just a section of an existing chapter. I'm not closing this issues as it still contains valuable info.


Update to use symfony v6 as a dependency

resolves #90

[x] Update symfony dependency to v6

@PeterOcansey does this work with laravel mail plugin? Thats the point of update here.

Ok, I missed that.....I'll run a test on it, see how it works

@fosron Yeah, install in laravel-driver works fine, I also tested the driver in L10 project and sending email works too.


The opentracing-toolkit context can not propagation across process

This issue includes:

Hostname, ipv4, ipv6 tags are not supported. Related #169
Inject and extract fail when using binary-map. Related #167
Peer.port is type of short, in some version of ot. Done in prev commits.
Refactor codes about inject and extract when using text-map. #167
Change context head-name to sw3. Lower case and shorter are better.

Coverage increased (+0.07%) to 71.309% when pulling 11a4d14d6d9d7b199ab509d1175464e2c1514539 on ascrutae:zhangxin/fix/169 into e577b791efb486a8d4fa5cb763324f54b9517556 on wu-sheng:feature/3.0.1.

Coverage increased (+0.07%) to 71.309% when pulling 3dfbb6abef7a37ce8d4aa9ffb743ba00461131a4 on ascrutae:zhangxin/fix/169 into e577b791efb486a8d4fa5cb763324f54b9517556 on wu-sheng:feature/3.0.1.

Coverage increased (+0.07%) to 71.309% when pulling 3dfbb6abef7a37ce8d4aa9ffb743ba00461131a4 on ascrutae:zhangxin/fix/169 into e577b791efb486a8d4fa5cb763324f54b9517556 on wu-sheng:feature/3.0.1.


RTMP Support

Any plans on supporting this?

@ismhdez @sampotts any reason why this was closed? Is there going to RTMP support?

You actually have it, just use another plugin with plyr.

@ismhdez, can you please clarify this. I am not sure what plugin you are referring to?

Search for HLS or something and include it on Plyr player


Design: trailing comma semantics

Current proposal:

require trailing comma for maps and non-tuple structs (which have named fields)
optional comma for tuples and tuple structs

:+1: since I was CCed here on Gitter.  This proposal looks good to me.

It looked good to me at first, but I already hear people complaining about it.

@torkleyy We can't address a vague feeling of dissatisfaction.  What were the complaints?

There were no real complaints yet (since it's not implemented / used), but @vitvakatu found it weird and I think users would find it annoying if the parser fails because of a missing trailing comma (which isn't obvious). Furthermore, I think a trailing comma is useful, but should not be enforced just because of that fact; enforcing it would make it an anti-feature again. The simple and obvious solution would be to allow it everywhere, which would make the spec easier and it's also easier to keep in mind.

That sounds good to me, for our serializer can we have it insert trailing commas for structs though?

That's already the case.


Updates for hero banner gallery

Updates to hero banner per design meetings

here's a build: https://makecode.microbit.org/app/5815523cda0fd9ce28690b493d7346eae12648ea-7b4356e8ad (updated to https://github.com/microsoft/pxt/pull/8171/commits/68bd0c128c2831a1c4b617a5850de8bff5e3e888)
to make the default hero banner show up with actions also need to add https://github.com/microsoft/pxt-microbit/pull/4077
Two other things I'll probably look at today -- supporting swipe to skip / go back, and possibly changing the transition between images from default browser crossfade to an opacity fade (but that can be surprisingly tedious to deal with / might not be worth the time right now.)
One implementation note; the reason for leaving the homeScreenHero in and not just moving complete to a gallery is because the homeScreenHero is defined in pxtarget, so it will be present immediately on load and show up in first sweep -- the gallery is loaded asynchronously so it will populate after the first load. If we were to define only a gallery and no homescreenhero banner, it would take a moment to load and then pop into view / move the contents of the screen around.

if we can ensure the images are all the same size then the crossfade thing is a nonissue also!

small thing that doesn't have to be this PR, it looks like when the description is 2 lines and the screen is short, it looks a little cramped

Ooh good catch thanks, I missed that there was a 'very short wide' setting for banner too. Fixed by swapping it to mobile view:


control Concurrent number of tasks execute per worker and per DAG

I can set a max task numbers(slot) to the worker. I set to 10, it's means one worker just run 10 tasks in same time.
I have 2 DAG, every DAG can set different tasks numbers (slot)
For example:
I just have a woker.
1.I can set one DAG slot 10, another 10. 10+10, It's bigger than 10. It's right, because one worker only can run 10 tasks in same time.
2.in some time , i can set one DAG's slot to 0 , give priority to another DAG. This is a very common scenario.

类似airflow中pool和slot的概念

+1

official english,  please change title and content to english.

not supported yet

do you mean https://github.com/apache/incubator-dolphinscheduler/issues/268 ？

Similar to the concept of pool and slot in airflow

Duplicate of #268

oh . but #268 it's not my issue?

Similar to the concept of pool and slot in airflow

Duplicate of #268

yes . same as #268


passport-google-oauth20 passport.authenticate doesn't run

When I visit the Google oauth authentication page:
app.get('/user/auth/google', (req, res, next) => {
  passport.authenticate('google', {
    scope: ['profile', 'email'],
  })(req, res, next);
});

My passport about Google oauth authentication strategy:
passport.use(
    new GoogleStrstedy({
    clientID: GOOGLE_CLIENT_ID,
    clientSecret: GOOGLE_CLIENT_SECRET,
    callbackURL: `/auth/google/callback`,
    scope: ['profile', 'email'],
},(accessToken,refreshToken,profile,done)=>{... })

))

redirect page:
app.get("/auth/google/callback",passport.authenticate("google", { session: false, failureRedirect: CLIENT_IP+ "error" }),(req, res) => {
      if (req.user.success) {
                    res.redirect(CLIENT_IP);
            });
      } else {
        res.redirect(CLIENT_IP + "/Register_google?mail=" + req.user.data.mail);
      }
    }
);

When I use Google authentication with localhost:5000 as the backend server, it successfully redirects to the specified page after selecting the authentication account in Google. However, when I upload the frontend of the website to AWS S3 and configure it with CloudFront, and upload the backend to a public EC2 and also configure it with CloudFront, and access the backend server through the HTTPS link generated by CloudFront, the callback function (accessToken, refreshToken, profile, done) => {...} in Google authentication is not executed on the backend server hosted on CloudFront. Instead, the response header redirects to the authentication page: https://accounts.google.com/o/oauth2/v2/auth...
Should I adjust the backend code, Google OAuth settings, or AWS configuration
to successfully execute the authentication process?

This doesn't seem to be a passport issue.  It seems like you are not configuring your oauth callback properly, since you said the code works in your dev environment but not production.

看來不是護照問題。看來您沒有正確配置 oauth 回調，因為您說代碼可以在您的開發環境中運行，但不能在生產環境中運行。

But I only changed the environment variables used locally to the ones used for the online website, without modifying any other code.

The docs explain it.

The docs explain it.
Could you please share where in the documentation this is explained? I have been trying to figure out the answer to this issue for almost a month, but still haven't found it. Thank you.

https://developers.google.com/identity/protocols/oauth2/web-server

https://developers.google.com/identity/protocols/oauth2/web-server

I have solved the issue. My Passport Google OAuth 2.0 is not the problem. The issue lies in the deployment on CloudFront, where the cache is not accepting the req.query parameters. This caused an error in the redirected route as it couldn't retrieve the code parameter. Still, thank you very much for your help.

https://developers.google.com/identity/protocols/oauth2/web-server

where the cache is not accepting the req.query parameters. This caused an error in the redirected route as it couldn't retrieve the code parameter. Still, thank you very much for your help.

I have the same problem with you. How do you solve that ?

其中快取不接受 req.query 參數。這導致重定向路由出現錯誤，因為它無法檢索程式碼參數。不過，非常感謝您的幫忙。

我跟你有同樣的問題。你怎麼解決這個問題？
https://github.com/jaredhanson/passport/issues/989#issuecomment-1662738087

can you tell how you figure that issue and resolve exactly i have same problem my google oauth working in development but not in production


Inventories are broken with some plugins

We were running latest build from spigomc and we've noticed some of GUIs on our Creative server are broken, after building and printing windowID it seems that in some cases windowID is 0 even tho it's GUI inventory...
if (windowId != 0 && panilla.getPConfig().ignoreNonPlayerInventories)

would it make sense to remove windowId != 0 check?

How are these GUIs created? I'm not sure what causes the window ID to be 0 if it's not a player inventory.

Not sure yet I'm looking into this rn, but just
if (panilla.getPConfig().ignoreNonPlayerInventories)
doesn't fix the issue, even tho check is skipped (so config value is properly read and set to true), however removing Panilla fixes the issue, we've only noticed cause GUI clicks are cancelled (in some cases, holding shift and clicking on GUI would still work for some reason).

Looking at the code I think Inventory inventory = player.getOpenInventory().getTopInventory(); is used, then populated with GUI items etc.

Actually, it seems that
    @Override
    public void checkPacketPlayInClickContainer(Object _packet) throws NbtNotPermittedException {
        if (_packet instanceof PacketPlayInWindowClick) {

            if (panilla.getPConfig().ignoreNonPlayerInventories) {
                return; // TODO test
            }

in public class PacketInspector implements IPacketInspector would fix the issue with GUIs, but I'm not sure about possible other implications...


Feature/styling/jd

Pull Request
Description
Added styling for front page and kafka cluster metrics.
Changes Made

Added styling for all components.

Checklist

[x] Code follows project style guidelines.
[x] Documentation has been updated.
[x] No breaking changes introduced.

Nicely done!


feat(aws-codecommit): New method addToPipelineStage on Repository

See #265 for the discussion about this feature.
This is a trial PR - if we agree with the direction taken here, I'll follow up with more PRs for S3, CodeBuild, CodeDeploy, etc.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

pipeline.addSource(repository.asPipelineSource({ artifactName: 'hello' }));

I'm not a huge fan of this, as it hides the Stage completely. What name will it have? This works fine for source Actions, but what if I have a Stage that needs more than one Action (an approval, manual or with integ tests)? Will we have another variant that allows you to give the stage? That might be too many ways, all similar but slightly different, of achieving the same thing...

Also, we need a solution that doesn't suck without jsii support for generics. We may have other API entry points that leverage generics to provide "reciprocate" views on those (allowing discoverability from both sides is in parts what we are trying to aim for here).
One thing, though, is you mention the pipeline.addSource(repository).configure({...}) doesn't require generics, but I actually think it does... If I'm wrong, can you please elaborate on how this works out to actually be type-safe? I'm interested in learning.

I'm not a huge fan of this, as it hides the Stage completely. What name will it have?

That could all be solved by changing the arguments slightly. Feel free to read my example as:
stage.addAction(repository.toSourceAction({ name: 'Hello' }));

Or whatever if that makes more sense. My point was more about the object model and the order of operations than the exact type of object and set of parameters involved.
Only thing I wanted to say was, I think element.addToContainer(container) is backwards, and container.addElement(element) makes more sense.
But I just wanted to provide some final alternatives, to see if they were considered. In particular:
container.addElement(elementProvider.makeAdapter({ args }));

// and
container.addElement(element).supplyCustomization({ args });

But if everyone's on board with the current API, I'm not one to stand in the way of progress.

One thing, though, is you mention the pipeline.addSource(repository).configure({...}) doesn't require generics, but I actually think it does.

You are correct.

@rix0rrr can you write out the entire signature of Stage#addAction that you use in the code stage.addAction(repository.toSourceAction({ actionName: 'Hello' })) - including the arguments and return type? Remember  that it needs to return a codecommit.PipelineSource Action.
I don't believe it can be expressed without generics.

@rix0rrr can you write out the entire signature of Stage#addAction

// Generic
interface IAction {
    public readonly name: string;
    public readonly toJSON(): any;
}

class Stage {
    public addAction(action: IAction) {
        this.actions.push(action);
    }
}

// Specific
class RepositorySourceAction implements IAction { ...elided... }

class Repository {
    public toSourceAction(props: RepositorySourceActionProps): IAction {
        return new RepositorySourceAction({
            repository: this,
            name: props.name,
            ...
        });
    }
}

The combination of IAction+RepositorySourceAction could potentially even be a single concrete class Action if we just say that class is just a type tag for an arbitrary property bag.
Maybe I'm thinking too simply about it? Tell me why this wouldn't work?

You are right of course.

Rebased and incorporated Romain's and Elad's feedback.


UpstashError: NOPERM this user has no perm issions to run the 'eval' command or its subcommand

I'm trying to rate-limit a simple netlify serverless function. The function manages to return a response but still throws an error.
Here's the code for the function:
// /netlify/handler.ts
import * as dotenv from 'dotenv';
import { Handler, HandlerEvent, HandlerContext } from "@netlify/functions";
import { Configuration, OpenAIApi } from "openai";
import { Ratelimit } from "@upstash/ratelimit";
import { Redis } from "@upstash/redis";

dotenv.config();

const configuration = new Configuration({
    apiKey: process.env.OPENAI_API_KEY
});
const openai = new OpenAIApi(configuration);

const ratelimit = new Ratelimit({
  redis: Redis.fromEnv(),
  limiter: Ratelimit.fixedWindow(10, "20 s"),
});

const handler: Handler = async (event: HandlerEvent, _context: HandlerContext) => {

  const question = JSON.parse(event.body).body;

  // check if the selected text is useless.
  if (question === undefined || question.length < 7) {
    return {
      statusCode: 205,
      body: JSON.stringify({ message: "no text" }),
    }
  }

  // Upstash rate-limiter
  const identifier = "api";
  const success = ratelimit.limit(identifier)
    .then(e => { return e})
    .catch(e => { console.log('Error in upstash fetch:', e) }) // <---- error comes from here

  if (!success) {
    return {
      statusCode: 429,
      body: JSON.stringify({ message: "Woah! Slow down." }),
    }
  }

  try {
    const res = await openai.createCompletion({
      model: "text-curie-001",
      prompt: `:Say this is a test: """${question}"""`,
      max_tokens: 100,
      temperature: 0,
    });

    return {
      statusCode: 200,
      body: JSON.stringify({ message: res.data.choices[0].text }),
    }
  } catch (e) {
    console.log("ERROR has occured", e.response);
    return {
      statusCode: 500,
      body: JSON.stringify({ message: "Error" }),
    };
  }
};

export { handler };

Here's the stack trace:
Request from ::1: POST /.netlify/functions/gpt-summarize
(node:87001) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
Error in upstash fetch: UpstashError: NOPERM this user has no permissions to run the 'eval' command or its subcommand
    at HttpClient.request (/Users/Yash/Code/summarize/node_modules/@upstash/redis/esm/pkg/http.js:93:19)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at EvalCommand.exec (/Users/Yash/Code/summarize/node_modules/@upstash/redis/esm/pkg/commands/command.js:46:35)
    at RegionRatelimit.limiter (/Users/Yash/Code/summarize/node_modules/@upstash/ratelimit/esm/single.js:88:44)
    at RegionRatelimit.value (/Users/Yash/Code/summarize/node_modules/@upstash/ratelimit/esm/ratelimit.js:62:24)
Response with status 200 in 1763 ms.

After I catch the error, it manages to give me a response though.
Let me know if this a ratelimiter error or if I report it at upstash-redis.
Cheers!

Hey @YashKarthik
sorry I didn't notice this issue, I just added an alert for myself in the future.
The error you pasted says NOPERM this user has no permissions to run the 'eval' command or its subcommand.
I don't think this is an issue with ratelimit nor the redis sdk.
Can you double check your ACL permissions?

Closing this as it looks like a configuration error, please reopen if necessary


1st Proposal for Generalising Uint serialisation and merkleisation

This PR generalises the serialisation and merkleisation for uint[8,16,32,64,128, 256] types.
Note:
At the moment, I think that only the type uint64 requires merkleisation by the beacon chain spec.
However, the SSZ and Merkleisation spec define the serialisation and merkleisation for all the uint types listed above.

The accepted solution was merged into the master branch at commit 0c810d7cfa9fe143d2893ea84b7e2f19cf15d3d8


Feedback/wishes from march '24 workshop: More illustrations

From Day 1 feedback of March 24 workshop: It may be easier for beginners to understand the new vocabulary if you have additional illustrations showing the differences/similarities between e.g. repository, branch, fork, clone etc. oo

These places could use an illustration:

Fork: https://coderefinery.github.io/git-intro/browsing/#creating-a-copy-of-the-repository-by-forking
Branch and commit: https://coderefinery.github.io/git-intro/commits/#exercise
Merge: https://coderefinery.github.io/git-intro/merging/
Clone: https://coderefinery.github.io/git-intro/local-workflow/
Publishing own work as repo: https://coderefinery.github.io/git-intro/sharing/

Awesome, I could draw possibly draw something during this week. Will add drafts for discussion .

To avoid possible accidental double-work, I am working on this.

Good luck! I didn't come up with anything useful :/


Custom signatures not working with Subclasssed Keras Model

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: n/a
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.3
Python version: 3.7
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: n/a
GPU model and memory: n/a

Describe the current behavior
When saving keras_models (checked for Subclassed Model) and providing custom signatures, keras_model.save(export_dir, signatures)  does not work and the saved model does not have the exported signature. The custom signature however works if we are using tf.saved_model.save(keras_model, export_dir, signatures).
Describe the expected behavior
tf.saved_model.save(keras_model, export_dir, signatures) and keras_model.save(export_dir, signatures) should have similar signatures.
Linked to TF Issue: https://github.com/tensorflow/tensorflow/issues/51827

Thanks for your issue. Can you provide a minimal repro example to illustrate the behavior?

@sumitbinnani Saving a TF2 keras model with custom signature


Support for attribute groups

bump

bump

Someone discuss the idea with me and if it's okay I can PR this

Someone discuss the idea with me and if it's okay I can PR this


[Event Change] Page 55 "Purch. Invoice Subform" - Field "Description" - trigger OnValidate()

------------------ Change IntegrationEvent Attribute ----------------------
IncludeSender : true, GlobalVarAccess : true
-------------------------------------------------------------------------------

[IntegrationEvent(TRUE, TRUE)]
local procedure OnBeforeOnValidateFieldDescription(var PurchaseLine: Record "Purchase Line"; var IsHandled : Boolean)
begin
end;

GlobalVarAccess was deprecated in 16.x, there is concept of protected var instead. You can list here the global vars that have to be protected and accessible from extensions.
Please put the code surrounding the place, where you expect a new event call, as a text not as a screenshot.
It helps us finding the right place quickly if you put enough code to identify the unique place.


Add example for enabled_metrics in MNG with LT

Is your request related to a new offering from AWS?
No
Is your request related to a problem? Please describe.
I didn't find an example on how to enabled_metrics ("Auto Scaling group metrics collection") while using Managed Node Group with Launch Template.
Describe the solution you'd like.
Looking for an update on terraform-aws-eks/examples/launch_templates_with_managed_node_groups/ to include enabled_metrics for ASG
Describe alternatives you've considered.
Manual enable it on console
Additional context
None

Autoscaling group is created silently by Nodegroup API. In result you cannot officially influence ASG as this is unsupported way. If you want to customize ASGs then you most go with worker_groups scenario described here


Update ReadMe

Update the docs to show you can hide the year of the date picker.

Hey, thanks for doing this, actually we can show the hide the year feature in features section. I want to show all possible options in that section. But set a yearsRange also an option. I think we can add comment in this line eg; //pass null for hide years.

Feel free to decline and we can take your approach. I don't have strong opinions, was just looking for ways to help. I appreciate your responsiveness here.


ZonamaDev Setup issues

Hi there,
I have run across several repeated issues with the ZonamaDev Environment setup process that require attention that they may be fixed.  First, Git Bash loses remote connection with the VM as soon as it goes up, and is unable to communicate with the environment within that VM.  Also, the rc.fasttrack scripts are not running at all like they are supposed to, and it is impossible to set up a new VM environment at all using the current VM setup system.  Also, the TRE.SH script cannot access the VM to copy the .tre files to the VM, although that should be obvious since the Git Bash Shell program cannot connect to the VM.  I hope this is enough detail to help you guys fix the problems, since I cannot update or create a new instance of the code at all with these problems.
Thanks a lot, and feel free to contact me if you need further details.
Sincerely,
Jeremy Marshall

Please provide the following details:

Host OS & Version
Vagrant version
VirtualBox Version
Host CPU Cores and RAM
Cores and RAM allocated to guest

Thanks for getting back to me.  Here is the info you requested:

  Host OS: Windows 10 with latest patches

  Vagrant Version: Latest as of yesterday (5-14-19) (Downloaded the updated version yesterday.)

  VirtualBox Version 6+

  Host CPU Cores and RAM: 8 Cores, 16GB Ram

  4 Cores allocated to Guest and 6 GB of RAM allocated to guest.

Hope that helps.
Sincerely,
Jeremy Marshall
From: Farelli<EMAIL_ADDRESS>Sent: May 15, 2019 5:01 AM
To: Zonama/ZonamaDev<EMAIL_ADDRESS>Cc: Coder476<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Zonama/ZonamaDev] ZonamaDev Setup issues (#60)
Please provide the following details:

Host OS & Version
Vagrant version
VirtualBox Version
Host CPU Cores and RAM
Cores and RAM allocated to guest

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Zonama/ZonamaDev/issues/60?email_source=notifications&email_token=ACUOQGQ74RZ423XJ2TQM2DTPVPNNJA5CNFSM4HM65WCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVOFJOA#issuecomment-492590264 , or mute the thread https://github.com/notifications/unsubscribe-auth/ACUOQGRCSPDAHPOXAQ6MXODPVPNNJANCNFSM4HM65WCA .  https://github.com/notifications/beacon/ACUOQGT3WEJYIQFKF423PM3PVPNNJA5CNFSM4HM65WCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVOFJOA.gif

This email has been checked for viruses by AVG.
https://www.avg.com

You might want to either decrease cores to 3 or increase RAM to 8GB on the guest. There are some processes that will max core usage trying to finish and then run out of RAM, failing to execute. This causes some extreme errors.
Latest Vagrant version possibly has issues; it has not been tested by our team. Recommend version 2.2.3 as we know it works with ZD.
VirtualBox hasn't been tested above version 6.0.4, so that could be causing an issue as well.

I just got past this same issue, I needed to change the config for SSH in the shell script.
I added a password for vagrant, and when promted in the script just typed it in... Kludgy but it works.


Render <!DOCTYPE html> instead of <!doctype html>

It doesn't matter much, but code highlighters seem to handle the uppercased version better.

hi :)
https://github.com/buxlabs/boxwood/pull/660


Necessary looks blocks

These looks blocks from Scratch should be added in:

[x] Switch to costume by number
[x] Switch to costume by name
[x] Default costume for tests
[x] Add costume
[x] Delete costume
[x] Change size by ()
[x] Set size to ()
[x] Show
[x] Hide
[ ] Go to front
[ ] Go back () layers
[x] Get costume number
[x] Get size
[x] Get costume name

Let me know of any more that might be needed, or any that should be removed.

Added some of the reporters.

Size functions now work properly with the basic test. Run it for yourself.

The costumes now work with both name and numbers, as the commits in #3 show. Pretty soon I should be able to get a deleteCostume() function working.

Deleting costumes has been added.
Also, show and hide were unchecked but I added them in now.

I added in the last few items in #3's latest commit, so I'm gonna go merge that now.

Here's the updates on the last 4 items:

Default costume was added.
Next costume is not needed because it can be workarounded for now.
The layer blocks are staying until I get feedback on what to do with them.

Hmm, layers. How big should a layer be? How many layers? Why so many questions!?!?

@Omegabyte  Layering of sprites works like it does in Scratch - one sprite on top of another.
I think the simplest way is to reorder the sprites list.

OK, so here's what we've narrowed this down to:

The last two things are layers
Layering is done by re-ordering the sprites list

Based on the conclusion we reached in #26, I'm replacing layers with z-indexes.

#31 fixes the final item - the z-indexes.
I can now close this.
It feels good.


Consistent floating point representation between tests

For example, tests/valid/exponent-part-float.json represents floats without a trailing .0, but tests/valid/float-exponent.json represents floats with a trailing .0.

@arp242 Could it be that this one got closed accidentally? The formatting is still not completely consistent. Additionally, as I already mentioned in https://github.com/toml-lang/toml/issues/585#issuecomment-644618718, it would be good if the exact representation of the floats in JSON (e.g. number of digits) could be specified.

All natural numbers are without a .0, so I figured it was fixed, but I didn't really understand why it mattered either. Anyway, we can talk about that in the PR.


Packaging improvements

Various improvements to the Python package that gets pushed to PyPI and it's display on PyPI.
Summary of changes

Enable the new native PyPI Markdown long_descriptions.
Remove the conversion of README to RST.
Remove old unsupported Python version 3.2 from classifiers.
Add Python 3.7 to classifiers (#631).
Add the project website and Gitter chat to the project URLS that are displayed on PyPI.
Ensure the LICENSE file gets included in the package metadata
Cleanup and format code.

This is similar to PR's I've created in three other Hug projects, though bit more in depth (adding project_urls and changing classifiers for Python versions).

Coverage increased (+0.2%) to 99.564% when pulling c2e7581915c7b753d2a68b0c1fdbc6eb4dd7f393 on GhostofGoes:packaging-improvements into 9cac91105b3384030221b9187d15a5754eb3fc03 on timothycrosley:develop.

Coverage increased (+0.2%) to 99.564% when pulling c2e7581915c7b753d2a68b0c1fdbc6eb4dd7f393 on GhostofGoes:packaging-improvements into 9cac91105b3384030221b9187d15a5754eb3fc03 on timothycrosley:develop.

Coverage increased (+0.2%) to 99.564% when pulling c2e7581915c7b753d2a68b0c1fdbc6eb4dd7f393 on GhostofGoes:packaging-improvements into 9cac91105b3384030221b9187d15a5754eb3fc03 on timothycrosley:develop.

Coverage increased (+0.2%) to 99.564% when pulling c2e7581915c7b753d2a68b0c1fdbc6eb4dd7f393 on GhostofGoes:packaging-improvements into 9cac91105b3384030221b9187d15a5754eb3fc03 on timothycrosley:develop.

This is great, thank you!


Add formvalidator-swift by @ustwo to Form & Settings section

Project URL

https://github.com/ustwo/formvalidator-swift
Description

Add formvalidator-swift by @ustwo to Form & Settings section
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Hey @nschucky, It was nice to meet you in person last weekend! 👍
And as always, thanks for contributing 😄 🎉

@lfarah always good to see you mate, i really wish you stay in SF for longer


Should not set the global variable "scrollMonitor" when in AMD environment

First, thanks for the awesome library. It does its job very well 👍

Just to nitpick, the current implementation defines global variable window.scrollMonitor explicitly in index.js#L10:
if (isInBrowser) {
  window.scrollMonitor = scrollMonitor;
}

This is somewhat unwanted functionality in AMD environments (especially when one is developing cross-site widgets and want to avoid conflicts with globals on host sites).
As a solution, I propose we get rid of the explicit global window.scrollMonitor definition and we let the Webpack Universal Module Definition target template handle it as you already use it:
(function webpackUniversalModuleDefinition(root, factory) {
	if(typeof exports === 'object' && typeof module === 'object')
		module.exports = factory();
	else if(typeof define === 'function' && define.amd)
		define([], factory);
	else {
		var a = factory();
		for(var i in a) (typeof exports === 'object' ? exports : root)[i] = a[i];
	}
})(this, function() {
  // the rest of the webpack bootstrap ...
  // actual scrollMonitor scripts ...

Which basically checks if in:

CommonJS environment
AMD environment
Attach to root/global, which in browserland is window
… and then initializes the library for the first environment it detects.

Good catch! I'm on vacation I'll take a look next week.
On Mon, Mar 20, 2017 at 3:38 PM Ari Palo<EMAIL_ADDRESS>wrote:

First, thanks for the awesome library. It does its job very well 👍
Just to nitpick, the current implementation defines global variable
window.scrollMonitor explicitly in index.js#L10
https://github.com/stutrek/scrollMonitor/blob/f88afe9b9b066176778169750f66369390588eb6/index.js#L10
:
if (isInBrowser) {
window.scrollMonitor = scrollMonitor;
}
This is somewhat unwanted functionality in AMD environments (especially
when one is developing cross-site widgets and want to avoid conflicts with
globals on host sites).
As a solution, I propose we get rid of the explicit global
window.scrollMonitor definition and we let the Webpack Universal Module
Definition target template handle it as you already use it
https://github.com/stutrek/scrollMonitor/blob/v1.2.3/webpack.config.js#L9-L11
:
(function webpackUniversalModuleDefinition(root, factory) {
if(typeof exports === 'object' && typeof module === 'object')
module.exports = factory();
else if(typeof define === 'function' && define.amd)
define([], factory);
else {
var a = factory();
for(var i in a) (typeof exports === 'object' ? exports : root)[i] = a[i];
}
})(this, function() {
// the rest of the webpack bootstrap ...
// actual scrollMonitor scripts ...
Which basically checks if in:

CommonJS environment
AMD environment
Attach to root/global, which in browserland is window
… and then initializes the library for the first environment it
detects.

You can view, comment on, or merge this pull request online at:
https://github.com/stutrek/scrollMonitor/pull/62
Commit Summary

Remove explicit window.scrollMonitor definition

File Changes

M index.js
https://github.com/stutrek/scrollMonitor/pull/62/files#diff-0 (4)

Patch Links:

https://github.com/stutrek/scrollMonitor/pull/62.patch
https://github.com/stutrek/scrollMonitor/pull/62.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stutrek/scrollMonitor/pull/62, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUtfzWfTRSBFCOraC2001-bFkHiEeA6ks5rnp2OgaJpZM4MimuH
.

Enjoy your vacation! 🍹
I can live with my fork (but I still think this is something that would be good to fix for others/future).

I see what you did here. Originally I had it like this, until #19. Initially I didn't implement it because, like you, I thought it was a little weird. Since then I ran into some use case myself (maybe it was testing? I can't remember) and I decided that pragmatism is the way to go: just let people use it like that. Unfortunately, now that it's like this (and I'm not saying it was a good idea), changing it back would be a major version change because someone might be using it... foolishly.
Thanks for the contribution. Keep making the world around you better.

Totally understandable! 👍  And as I said, I can live with my fork!


insight-yo.json

run yo webapp, and the err show:
Error: EACCES, permission denied '/root/.config/configstore/insight-yo.json'
You don't have access to this file.

    at Error (native)
    at Object.fs.openSync (fs.js:502:18)
    at Object.fs.readFileSync (fs.js:354:15)
    at Object.create.all.get (/usr/local/lib/node_modules/yo/node_modules/configstore/index.js:27:26)
    at Object.Configstore (/usr/local/lib/node_modules/yo/node_modules/configstore/index.js:20:44)
    at new Insight (/usr/local/lib/node_modules/yo/node_modules/insight/lib/index.js:37:34)
    at Object.<anonymous> (/usr/local/lib/node_modules/yo/lib/cli.js:130:15)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)

when I run the commandls /root/.config/configstore, the result is:
bower-github.yml  insight-bower.yml  update-notifier-bower.yml

HELP !!

That's a permission error coming from configstore. There's not much we can do about that, except throwing a better error message. For now you should be able to fix it using the chmod command in the issue.

Also, note that it's weird the file appear into /root/. This mean you've been running commands as sudo. The take away here is you should never run anything as sudo, it's very unsecure and is just creating issues. Ensure your main user have the correct permissions on your system.

@arthurvr @SBoudrias
thx~
I have resolved the problem on the root cause by using NVM, It's really really great.

Oh, yes it is! NVM isn't specifically about issues like this but it's a perfectly valid fix. Glad it's resolved!

Bump on this issue as I have been unable to solve it using permissions (the directory does not exist) or nvm
/home/projects/yo-test# yo
/usr/local/lib/node_modules/yo/node_modules/configstore/index.js:53
throw err;
^
Error: EACCES: permission denied, open '/root/.config/configstore/insight-yo.json'
You don't have access to this file.
at Error (native)
at Object.fs.openSync (fs.js:584:18)
at Object.fs.readFileSync (fs.js:431:33)
at Object.create.all.get (/usr/local/lib/node_modules/yo/node_modules/configstore/index.js:34:26)
at Object.Configstore (/usr/local/lib/node_modules/yo/node_modules/configstore/index.js:27:44)
at new Insight (/usr/local/lib/node_modules/yo/node_modules/insight/lib/index.js:37:34)
at Object.<anonymous> (/usr/local/lib/node_modules/yo/lib/cli.js:130:11)
at Module._compile (module.js:425:26)
at Object.Module._extensions..js (module.js:432:10)
at Module.load (module.js:356:32)

@looeee it is permissions issues. To change permissions, you need to have permission to change them. So you need to sudo chmod ....

As I said, the directory and files do not actually exist so this doesn't work. Nonetheless I tried actually creating the directory file and chmod 777 on it just as a test, this also didn't work (same error).
Anyway I decided to just install locally which works fine.

I'm seeing the same issue.  Fresh install, running as root.  Everything in node was installed as root (not just 'sudo', actual root), and running (in my case) yo hubot fails with a permission denied.
If I create the directory it still fails:
root@chat:/opt/hubot# ls -la /root/.config/configstore/
total 8
drwxr-xr-x 2 root root 4096 Dec  1 15:44 .
drwxr-xr-x 3 root root 4096 Dec  1 15:44 ..
root@chat:/opt/hubot# yo hubot
/usr/local/node-v5.1.0-linux-x64/lib/node_modules/yo/node_modules/configstore/index.js:53
				throw err;
				^

Error: EACCES: permission denied, open '/root/.config/configstore/insight-yo.json'
You don't have access to this file.

    at Error (native)
    at Object.fs.openSync (fs.js:584:18)
    at Object.fs.readFileSync (fs.js:431:33)
    at Object.create.all.get (/usr/local/node-v5.1.0-linux-x64/lib/node_modules/yo/node_modules/configstore/index.js:34:26)
    at Object.Configstore (/usr/local/node-v5.1.0-linux-x64/lib/node_modules/yo/node_modules/configstore/index.js:27:44)
    at new Insight (/usr/local/node-v5.1.0-linux-x64/lib/node_modules/yo/node_modules/insight/lib/index.js:37:34)
    at Object.<anonymous> (/usr/local/node-v5.1.0-linux-x64/lib/node_modules/yo/lib/cli.js:130:11)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)

@RangerRick don't install anything as sudo - ever. Doing this is a huge security risk.

I got this issue when running yo in a container, thanks to this discussion, I solved the problem by RUN useradd in Dockerfile :bow:

royling is there anything that can be done on the command line after I ssh into my docker container to fix this issue?

@dreamerkumar yes, you can add user after ssh and switch to it (su - docker), it's just equivalent to do so in the Dockerfile as below:
RUN useradd -ms /bin/bash docker
USER docker

Just in case you are installing in a container and you know what you're doing, you can also do:
sed -i -e '/rootCheck/d' "${NPM_CONFIG_PREFIX}/lib/node_modules/yo/lib/cli.js"

freedom FTW!
see also sindresorhus/root-check#1

@RangerRick  hi, i know this is years later, but am wondering how u were able to solve this issue. I am running into the same error and I have no idea what to do. I have tried several suggestions but nothing seems to be working.
I would really appreciate your help. Thank you

@iruoma Sorry, I honestly have not used this for a long time, I'm no help.

thank you so much... i eventually figured it out.
On Wed, Jul 26, 2017 at 10:34 AM, Benjamin Reed<EMAIL_ADDRESS>wrote:

@iruoma https://github.com/iruoma Sorry, I honestly have not used this
for a long time, I'm no help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yeoman/yo/issues/348#issuecomment-318109575, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUibmgwdFOgi4t5_mZkBKMFBco_J2vMKks5sR2p-gaJpZM4FUaHa
.

@iruoma So, how to fix this??

Hi,
Sorry for the late response. I figured out that yo doesn't work as a root
user. So i just created another user on linux and ran yo and it worked
without any issues. You can make use of sudo or give the new user root
priivileges.
I hope this works for you.
On Fri, Aug 25, 2017 at 8:59 PM, E.L<EMAIL_ADDRESS>wrote:

@iruoma https://github.com/iruoma So, how to fix this??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yeoman/yo/issues/348#issuecomment-325077699, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUibmj2uVfxmrWeebdT6AhMWZxggDnMYks5sb4oNgaJpZM4FUaHa
.

Do not run the yo command as root.

I'm going to remove yo due to same issue

Small machine that I don't care, so I'm using everything as root, so avoid the "do not run as root" comments as a fix, as that's not a fix, but a workaround.
I ran chmod 777 -R /root/*, and still having this problem. Pretty weird.

Just in case you are installing in a container and you know what you're doing, you can also do:
sed -i -e '/rootCheck/d' "${NPM_CONFIG_PREFIX}/lib/node_modules/yo/lib/cli.js"

freedom FTW!
see also sindresorhus/root-check#1

thx!!issue is resolved!

这个问题怎么用nvm 解决的？？

同问啊 嘿嘿

Would like to ask that
how to use nvm to slove this question ..


Update the Refaster rule build instructions

Overview
Update the Refaster rule build instructions
to match the ones on the web site. Remove
no longer needed J--add-exports compiler
options.
See also:

0ede1de
#1078
https://errorprone.info/docs/refaster

I signed it!


Text is not displayed completely

Hey @gagan3012 , just one more thing.
Although there is enough space, the text argument (in this case 'Press enter to add up to 3 keywords') is not displayed completely, it is truncated at "Press enter to add up to" as shown by the picture below

Hey @GitHunter0
I have fixed it in the latest update

Thanks man, problem solved!


fix: remove readOnly from constants

a fix to make us compatible with the vjsf component for meditor.

yes, should be patch.


sync progress to session storage

to enable at least minimal persistence

ähm, nope. keep it completely within the app


Feature / add consentManager Placeholder component

I added the consentManager's Placeholder component based on the example given on the stories.
While adding the consent manager on our project, the CI failed on accessbility tests (aXe). I took the liberty to add titleAs so we have not to stick to arbitrary H levels on the DSFR examples (which are, depending on the context, accessibility issues):

replaced consentManager title by span, which won't break title hierarchy
Placeholder title with titleAs prop (which default to h4 - same behavior as the dsfr demo)
replaced consentManager modal wrapper tag by dialog (instead of div, to resolve missing landmarks issues)

Still discussing on it on Mattermost: https://mattermost.incubateur.net/betagouv/threads/jnzhstsadbdjib9ithqhotte7h

Still having issues:

dialog seems to applies other styles (centered, thick border)
cookies banner and consent management modal have the same ID

I think I'll remove all semantic / accessbility related things in another PR.

Let me know when you want me to review it!
And thanks a lot fot this!

@garronej feel free to review it :)

Looks great, thanks a lot!

<3


⚠ SceneTime has degraded performance

In b58c510, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 7577 ms

Resolved: SceneTime performance has improved in 6f8cd9d.


duplicate import

https://github.com/pytorch-labs/gpt-fast/blob/3bcaaaf068d112d534f335ec21a17d7b8b5551bf/quantize.py#L12
Already included on line 6.

There are a few more of this and some imports that don't seem to be used. I will create a pull request to sort the imports.

Resolved by https://github.com/pytorch-labs/gpt-fast/pull/16


GetDefaultEntityMapping to get only certain columns

I'm trying to get only certain columns, but when my grid is filled, it shows all the columns. This is the code I use:
using (SQLiteConnection db = new SQLiteConnection(SQLiteConnectionString))    {       var customMapping = OrmConfiguration          .GetDefaultEntityMapping<Product>()          .Clone()          .RemoveAllPropertiesExcluding(             nameof(Producto.IDProduct),             nameof(Producto.ProductName));       var miLista = db.Find<Producto>(statement =>          statement.WithEntityMappingOverride(customMapping));       if (miLista != null)          {             grdDatos.DataSource = miLista.ToList();          }
What I'm doing wrong?
Thanks a lot

In this case you should create a new entity type, mapped to the same table, or manually restrict what the grid is displaying. Otherwise FastCrud is still gonna return your full entity, the only difference being that those properties you're excluding won't be filled.


ARROW-16417: [C++][Python] Segfault in test_exec_plan.py / test_joins

This builds on top of #13035 which is also important for avoiding segmentation faults.  On top of that there were a few more problems:

The python was using SourceNodeOptions::FromTable which is a rather dangerous method (mainly useful for unit testing) as it doesn't share ownership of the input table (even worse, it takes a const ref).  Python was not keeping the table alive and it was maybe possible for the table to deleted out from under the plan (I'm not entirely sure this was causing issues but it seemed risky).  I switched to TableSourceNode which shares ownership of the table (and is a bit more efficient).
Setting use_threads to False did nothing because _perform_join was not passing the arg on to execplan.
When fixing the above and running with use_threads=False it was creating a single thread executor but the current best practice is to pass in nullptr.
Finally, the actual bug was my improper fix in #12894 .  I had still left a small window open for End to be called between Submit and AddTask which would allow the task to be submitted but not participate in setting finished on the node.

I was able to get it to reproduce in debug mode with stress -c 32.  Then I got reasonable stack traces that showed the hash join node still working while the python was getting ready to start the next test.  That was pretty similar to the debugging I did in #12894 which fortunately was still recent enough I remember roughly what was happening.

Aha.
Hmm, I wonder if it'd be useful to somehow have Pytest also assert that Arrow's thread pools are idle in between tests. (And frankly, Googletest as well.)

Technically, we aren't quite there yet.  The async generators in the scanner are allowed to run some cleanup after the exec plan runs.  However, they strongly capture all of their state.  Once #12468 merges then I think this might be a good idea.


How to handle rust types with lifetimes?

Hello,
thank you for this cool project.
I'm working on some Rhai bindings for scraper. Specifically, I'm trying to achieve the following:
let html = document("https://google.de");
let links = html.select("a");
for element in links {
 // do something with element
}

document download and parsers the html and select runs a css query against this document. However, the underlying Select has two lifetime parameter and I assume this is why this wont work:
#[derive(Clone)]
pub struct HtmlDocument {
    pub document: Rc<Html>,
    pub map: HashMap<ImmutableString, Selector>,
}

impl HtmlDocument {
    pub fn select(&mut self, css_selector: ImmutableString) -> Select {
        let selector = Selector::parse(css_selector.clone().as_str()).unwrap();
        self.map.insert(css_selector.clone(), selector);
        self.document.select(&self.map[&css_selector])
    }
}

// in main

    engine.register_type_with_name::<HtmlDocument>("HtmlDocument");
    engine.register_fn("select", HtmlDocument::select);

I get this error:
error[E0277]: the trait bound `for<'a> fn(&'a mut HtmlDocument, ImmutableString) -> scraper::html::Select<'a, 'a> {HtmlDocument::select}: RegisterNativeFunction<_, _, _, _, _>` is not satisfied
  --> src/main.rs:10:34
   |
10 |     engine.register_fn("select", HtmlDocument::select);
   |            -----------           ^^^^^^^^^^^^^^^^^^^^ the trait `RegisterNativeFunction<_, _, _, _, _>` is not implemented for fn item `for<'a> fn(&'a mut HtmlDocument, ImmutableString) -> scraper::html::Select<'a, 'a> {HtmlDocument::select}`
   |            |
   |            required by a bound introduced by this call
   |
note: required by a bound in `rhai::api::register::<impl Engine>::register_fn`
  --> /Users/kinch/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rhai-1.15.1/src/api/register.rs:64:12
   |
58 |     pub fn register_fn<
   |            ----------- required by a bound in this associated function
...
64 |         F: RegisterNativeFunction<A, N, C, R, L>,
   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `rhai::api::register::<impl Engine>::register_fn`

I found a workaround by using a self-referencing struct:
#[self_referencing]
pub struct SelectExpression {
    pub document: HtmlDocument,
    #[covariant]
    #[borrows(mut document)]
    pub select: Select<'this, 'this>,
}

#[derive(Clone)]
pub struct SelectRef(pub Rc<RefCell<SelectExpression>>);

This workaround solves the problem for now, but I'm wondering if there's a simpler solution or if I might have misunderstood something about how to handle lifetimes with Rhai
Kind Regards,
Kinch

Unfortunately I believe this is where you'd have to do it the long way.
Rhai is sandboxed, meaning that it tries not to access/modify anything outside of itself.  Data with lifetime usually means that it is a reference to something outside, so your Rc<RefCell<...>> trick is necessary.
In Rhai, you only access the outside world via interior mutability.

Thank you for your response :)
I see. Maybe this is a good opportunity to learn how to write rust macros.
Cheers.


Port integration tests to RSpec

The integration tests for txgh currently rely on @matthewjackowski's Github and Transifex credentials and make calls to production APIs. These tests are also written with Test::Unit. This PR makes the following changes:

Replace @matthewjackowski's credentials with a set of public test ones, all tied to non-essential test accounts on Github and Transifex. These accounts only have access to test resources. If they are compromised, we'll just delete them and create some new ones. This means we can remove all the environment variables on Travis as well as the gist shim I added last week :tada: :tada: :tada:
Introduce VCR for replaying API responses without making actual API calls. This should make things faster. The VCR cassettes are easily updatable using the test credentials described above.
Port the existing integration tests from Test::Unit to RSpec.

@jdoconnor @st33b @zvkemp

@camertron Brilliant!  I have 1 question about how to handle ENV vars.


Feat: more playlist

Codecov Report

Merging #136 into development will decrease coverage by 1.14%.
The diff coverage is 91.07%.

@@               Coverage Diff               @@
##           development     #136      +/-   ##
===============================================
- Coverage        98.11%   96.96%   -1.15%
===============================================
  Files               42       52      +10
  Lines              212      264      +52
  Branches            21       22       +1
===============================================
+ Hits               208      256      +48
- Misses               4        8       +4

Impacted Files
Coverage Δ

src/components/Home/index.tsx
100% <ø> (ø)
:arrow_up:

...rvices/spotify/search/searchPlayListsByTopTrack.ts
100% <100%> (ø)
:arrow_up:

...vices/spotify/search/searchPlayListsByTopArtist.ts
100% <100%> (ø)

src/services/spotify/album/getAlbum.ts
100% <100%> (ø)

src/services/spotify/browse/getCategoryPlayList.ts
100% <100%> (ø)

.../spotify/search/searchPlayListByLastPlayedTrack.ts
100% <100%> (ø)

...spotify/search/searchPlayListByLastPlayedArtist.ts
100% <100%> (ø)

src/services/spotify/artist/getArtist.ts
100% <100%> (ø)

src/components/PlayList/PlayListByTopTrack.tsx
80% <50%> (ø)
:arrow_up:

.../components/PlayList/PlayListByLastPlayedTrack.tsx
80% <80%> (ø)

... and 13 more

Continue to review full report at Codecov.

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


[release-v1.4] pod.Status.Phase == apiv1.PodRunning is not a good way to test if pods are Running

Cherry-pick of https://github.com/knative-sandbox/eventing-kafka-broker/pull/2523
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/assign @aliok @matzew

/hold
Until https://github.com/knative-sandbox/eventing-kafka-broker/pull/2523 is merged

/unhold


Bump to aiocoap alpha

Coverage remained the same at 63.319% when pulling 2a07eb25e2a36a912e997cfd629ce5a94a5a1094 on fix_deps into<PHONE_NUMBER>d50cf25ca9edc3a824e9d0e42aacfa on master.

Release log for reference:
https://github.com/chrysn/aiocoap/blob/master/NEWS


[FEATURE]: Foreign Keys are automatically treated as relations

Describe what you want
The documentation states pretty explicitly that relations do not affect the database schema in any way and do not create foreign keys implicitly.
While I think that is correct for relations to not create foreign keys, it would be really helpful if foreign keys automatically created relations.
Migrating from Prisma to Drizzle, I need to write a ton of extra code to use the query API to add relations for every FK constraint I have defined or bear the brunt of writing the aggregations manually for every relation.
In my opinion, FKs should be automatically treated as relations in the query API to cut down on all the extra boilerplate and open up this incredible abstraction by default.
Thanks for all that you do 🙇

Because the query API is opt-in, you can choose to not use it even if FKs generate relations automatically

I also found this very counter intuitive. I expected relations to be inferred by foreign keys.

I second this. Having to make relations manually, even if you have already made a foreign key, is redundant. Foreign keys are a lot simpler to work with on one to one relationships too in my opinion.
Much less code using foreign keys
export const passwordResetToken = pgTable('password_reset_token', {
	userId: varchar('user_id', {
		length: 255
	}).notNull().references(() => user.id)
});

versus relations
export const passwordResetTokenRelations = relations(passwordResetToken, ({ one }) => ({
	user: one(user, {
		fields: [passwordResetToken.userId],
		references: [user.id],
	}),
}));

and you probably want .references(() => user.id) too anyways if you're creating a relation like this.

This is mixing two different APIs into one, I can understand it can be intuitive for some, but the team aims to separate SQL-like functionality (foreign keys) from Drizzle specific abstractions (relational queries).


Wire up countdown

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#389
#388  👈
#386
#385
#376
#371
#369
#367
#366
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jvmi7 and the rest of your teammates on  Graphite


[Bug] Completed task are marked as killed when user kills all tasks

Describe the bug
Not completely sure this is a bug. Also I think this might be a backend issue.
When I kill a task where some of the subtasks have been completed successfully they all get marked as killed. I would expect that the status of this successful tasks remains completed as that is what it is.
To Reproduce
Steps to reproduce the behavior:

Create task with subtasks and multiple nodes
When one of the subtasks complete, kill the entire task
Observe that the completed task gets marked as killed nu user

Expected behavior
I would the completed task to keep the status completed.
Screenshots

The 'main - TEST Biomeris' was completed successfully but marked as killed
Details

Version 4.3.0b3

I can confirm that this is a backend issue - the backend marks all runs in a task as killed even if they already finished.
So I'll transfer this issue to the main repo


UI: saving pipeline needs user feedback

Problem
There is no UI reaction to saving a pipeline, a notification would be nice

Resolved in PR #360


Better error message: Query interpretation error. Error for binding '0': AssertionError("Required exactly one parent ID to be present for connect query, found 0."

Originally posted by @Albert-Gao in https://github.com/prisma/prisma/issues/3488#issuecomment-684826417
my current use case is:
When use nexus framework + prisma combo, when I have an error from the backend api
{
  "errors": [
    {
      "message": "\nInvalid `ctx.db.user.update()` invocation in\n/Users/AlbertGao/codes/teamingCloud/monorepo/packages/nexus/api/graphql/entities/test.ts:9:28\n\n   5 definition(t) {\n   6   t.field('testPrisma', {\n   7     type: 'User',\n   8     resolve: (_, __, ctx) => {\n→  9       return ctx.db.user.update(\n  Query interpretation error. Error for binding '0': AssertionError(\"Required exactly one parent ID to be present for connect query, found 0.\")",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "testPrisma"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR",
        "exception": {
          "code": "P2016",
          "meta": {
            "details": "Error for binding '0': AssertionError(\"Required exactly one parent ID to be present for connect query, found 0.\")"
          },
          "stacktrace": [
            "Error: ",
            "\u001b[31mInvalid \u001b[1m`ctx.db.user.update()`\u001b[22m invocation in\u001b[39m",
            "\u001b[31m\u001b[4m/Users/AlbertGao/codes/teamingCloud/monorepo/packages/nexus/api/graphql/entities/test.ts:9:28\u001b[24m\u001b[39m",
            "",
            "\u001b[2m  \u001b[90m 5 \u001b[39mdefinition(t) {\u001b[22m",
            "\u001b[2m  \u001b[90m 6 \u001b[39m  t.field('testPrisma', {\u001b[22m",
            "\u001b[2m  \u001b[90m 7 \u001b[39m    type: 'User',\u001b[22m",
            "\u001b[2m  \u001b[90m 8 \u001b[39m    resolve: (_, __, ctx) => {\u001b[22m",
            "\u001b[31m\u001b[1m→\u001b[22m\u001b[39m \u001b[2m\u001b[90m 9 \u001b[39m      return ctx.db.user.update(\u001b[22m",
            "  Query interpretation error. Error for binding '0': AssertionError(\"Required exactly one parent ID to be present for connect query, found 0.\")",
            "    at PrismaClientFetcher.request (/Users/AlbertGao/codes/teamingCloud/monorepo/packages/nexus/node_modules/@prisma/client/src/runtime/getPrismaClient.ts:1212:15)",
            "    at processTicksAndRejections (internal/process/task_queues.js:97:5)"
          ]
        }
      }
    }
  ],
  "data": {
    "testPrisma": null
  }
}

Is there any way I can do to identify what kind of error is this without matching hardcode string from the error message?
Isn't something like ErrorCode enum or a const object make it easier? Or I am miss something here.
Thanks :)

Thanks for moving, I thought it's duplicated 😂
information that needed:

error code : PascalCase words but not number plz
error message
call to action
callstack
source code
doc url

The current one already includes a lot but not all.
Furthermore, might be better if we just separate each of them into a separate property in an error object.

This is most probably greatly improved by not. If not, please open a new issue and provide a full reproduction on how to get the problematic error message. Thank you for opening an issue back in the day AlbertGao!


Add ability to use alignment in print function

Extend print method to have text argument take either a string or an object. If an object it will be possible to specify an alignment parameter to align printed text
Similar to another PR but more backward compatible.

Nice @theill! I'll be happy to accept your contribution. You only need to add tests to ensure your feature is working and it will still working on future. Please see print.test.js

@aurium sure. my bad. I've added tests now.

@aurium let me know if more changes are needed so we can get this merged into master


Update README.md

I have added "UVERworld".
could you please contribute to my list?(research or interest keyword)
https://github.com/kuroishirai/AttractingContributors_shirai

Of course!
I added "Materials Informatics" and sent pull request.


Fixes working-directory input

This fixes the "Warning: Unexpected input(s) 'working-directory', valid inputs are ['check_name', 'commit_sha']" message.
Adds a default value for the input
Possibly closes #8.

I don't understand how this can address #8, it seems to me that ./lib is still hardcoded. What I'm asking for is being able to override which directories to analyze relative to the root of a project, this seems to only change where that root is located.

@llucax, I understand your issue now and you're right. I've removed the linked issue. Thanks.


[FEATURE] Expand audit logging with appid/username and backend roles

Is your feature request related to a problem?
We are using oidc for both dashboards and opensearch itself. Currently the appid isn't logged in the auditlogging, neither are the backend roles for the user that is i.e. doing queries. I need to guess which user did a specific action based on ip/user agent/etc.
What solution would you like?
Log the appid/username and backendroles to the auditlogs
What alternatives have you considered?
No viable alternatives
Do you have any additional context?

[Triage] Hi @ict-one-nl, thank you for filing this issue. This will require some more investigation to determine the best path forward but as a general action item we can leave this comment stating that we will first need to look into the issue before implementing the fix.

Great thanks! Will try to be at the next triage, just came back from holiday :)

@ict-one-nl Is this request specifically for FAILED_LOGIN events? In the event for failed login, OpenSearch doesn't differentiate between failed login where user exists and wrong password vs user doesn't exit. Are you hoping to log the username in cases where the user exists?
Is this request applicable to other audit log event types?

Hi Craig,
It would be helpful yes. I'd like to see whatever is supplied by the user, it's not always the user name, but in the case of openid it's the appid (which doesn't match any user, but there is a roles claim that contains the backend roles). I can image that this would be helpful for more types of events on the REST layer, yes. In general you'll want to be able to diagnose what a potential hacker has tried. Or maybe it's a configuration error where there was a typo in the backend roles.
The more info is contained in the logs the easier fault diagnosis or forensics are.
Is this helpful?

[Triage] @cwperks Please follow-up to determine if the appid in OIDC can be captured in the audit log on different event types.

Thanks Craig!

Hi @cwperks are you going to be following-up on this? Thanks

[Triage] We should not log backend roles on failed login attempts, but additional information could be added to the audit logs.

That would be great. Is @cwperks working on this? No problem to schedule a call if needed.
Thanks guys!

@ict-one-nl apologies, I have not in earnest gotten around to this issue yet. I'll take a closer look at this issue in the next few days and post any follow-up questions on this issue.

Sure no problem Craig, thanks for the update!

This should be possible to do by modifying the AbstractHTTPJwtAuthenticator to return an AuthCredentials object instead of null when a token is expired and add a new method onto AuthCredentials (maybe isValid()) that can be used to indicate if the credentials provided were valid or not.
If authCredentials is non-null then details including the username will get auditlogged here: https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/auth/BackendRegistry.java#L423
For JWT claim associated with appid (same as Client ID?) is azp according to the openid specification: https://openid.net/specs/openid-connect-core-1_0.html

Like @cwperks said, that should definitely be possible but I do want to point out one concern: previously we had fixed an issue which would allow for credential fishing by responding a different message or timing when an account was valid vs. when it was not. If we implement this change as described, I think we may run into a similar scenario.
Basically, if we have the isValid() check then we will know that the subject is a real one. I don't think we want that

Any news on this? Usage of our platform has grown, this insight is helpful!

@sandervandegeijn There is no active development on this area that I am aware of, typically signals include having someone assigned to an issue.  For features important to your platform - contributing a pull request would be a great way to move progress forward.

I started looking into this today. Added a new attribute isValid to the authcredentials like Craig suggested. I don't know much about OIDC so making sure this does what @sandervandegeijn was asking will take me a little. In the meantime, I made the extract credentials block @cwperks was referencing look like this:

private AuthCredentials extractCredentials0(final SecurityRequest request) throws OpenSearchSecurityException {

        String jwtString = getJwtTokenString(request);
        boolean isValid;

        if (Strings.isNullOrEmpty(jwtString)) {
            return null;
        }

        SignedJWT formedJwt;

        try {
            formedJwt = jwtVerifier.formJwtToken(jwtString);
        } catch (BadCredentialsException e) {
            if (log.isTraceEnabled()) {
                log.trace("Forming JWT token from {} failed", jwtString, e);
            }
            return null;
        }

        SignedJWT jwt = null;

        try {
            jwt = jwtVerifier.getVerifiedJwtToken(formedJwt);
            isValid = true;
        } catch (AuthenticatorUnavailableException e) {
            log.info(e.toString());
            throw new OpenSearchSecurityException(e.getMessage(), RestStatus.SERVICE_UNAVAILABLE);
        } catch (BadCredentialsException e) {
            if (log.isTraceEnabled()) {
                log.trace("Extracting JWT token from {} failed", jwtString, e);
            }
            isValid = false;
        }

        JWTClaimsSet claimsSet;

        try {
            if (isValid) {
                claimsSet = jwt.getJWTClaimsSet();
            } else {
                claimsSet = formedJwt.getJWTClaimsSet();
            }
        } catch ( ParseException e) {
            if (log.isTraceEnabled()) {
                log.trace("Failed to parse claims from JWT token {}", jwtString, e);
            }
            return null;
        }

        final String subject = extractSubject(claimsSet);
        if (subject == null) {
            log.error("No subject found in JWT token");
            return null;
        }

        final String[] roles = extractRoles(claimsSet);
        final AuthCredentials ac = new AuthCredentials(subject, roles, isValid).markComplete();

        for (Entry<String, Object> claim : claimsSet.getClaims().entrySet()) {
            ac.addAttribute("attr.jwt." + claim.getKey(), String.valueOf(claim.getValue()));
        }

        return ac;
    }

So we return the credentials regardless if they are valid and only send back null if the output is not computable. The one thing I am not sure is what we will do with this flag for everything else... I am adding it to all the tests so that they pass (just setting to "true"), but this seems like a waste. Maybe there is something more useful we can do with it.

Thanks Stephen!


Current Day disappears on scroll

Hello,
I have observed that the current date disappears if we use horizontal scroll
it can be seen in the demo version : https://www.angular-gantt.com/demo/
Just select :

Layout : Width
Today : Column

Then scroll, to the right, back to the left, the column will be disapear.
In fact, it disappears if we rerender the calendar (put today in column, and change scroll status, current-day will be disapear).
Did I miss something? I'm sorry if it's this.

It's a bug, thanks for reporting it ! I'll fix now.

Thanks a lot :)


3 CYCLE QUESTION

THIS IS VERY IMPORTANT QUESTION IN INTERVIEW WHICH IS SOLVED BY USING GRAPH DATA STRUCTURE.

@Abhijit2505 CAN I WORK ON THIS ISSUE

@Abhijit2505 CAN I WORK ON THIS ISSUE


Allow templating COMPONENT_NAME during package create

Is your feature request related to a problem? Please describe.
When creating reusable importable components it would be useful to have a ###ZARF_COMPONENT_NAME### template / variable that can be used to scope that component properly.
Describe the solution you'd like

Given I have a zarf.yaml with ###ZARF_COMPONENT_NAME### in a string value
When I run zarf package create on that package
Then Zarf will template that value with that component's name

Describe alternatives you've considered
We could do this on deploy time or in another way, but most scoping would be around variables or actions and this would be easier / more declarative to do create-time.
Additional context
This would make things like a reusable S3 bucket component easier to make.

Closing as completed by #1923


Remove references to Clinepidb in site text

Fixes https://github.com/VEuPathDB/web-eda/issues/1266

@bobular Seems like you specifically need to approve my changes since you submitted a change request?

@dmfalke I don't think so! I just merge in main and here's what I'm seeing (you might be seeing it too):

@dmfalke I don't think so! I just merged in main and here's what I'm seeing (you might be seeing it too):

Ah, ok. I saw a different message yesterday. Let me see what I can do.

I "dismissed" @bobular's review, so you should be able to merge now.


Updated architecture introduction diagram of gRPC-Gateway

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


Fix multi-class classification task on reddit's comments.

Updated prompt works really well, but i want to see if using langchain and then adjusting parameters like temperature could help improve results or not.


pip packaging

setup.py cleaned up (descriptions, dependencies, versions, comments)
MANIFEST.md expanded by VERSION and development dependencies
LICENSE changed
python 2.7 is the only supported version now

fixes #4
fixes #5

Coverage remained the same at 85.811% when pulling c56061347290a4e991f7a91ec1dd3e868e62245d on pip-packaging into f778e5cc1226c40e2a8afbb50fb5bd476f89ab9a on master.


Adds a windows-compatible regex for foundry ABI paths

This was tested manually with @ZePedroResende . seems to be working


Update to language around "TAC Projects"

Update to language around "TAC Projects" as approved by the LFPH Governing Board at the January 12, 2021 meeting.

LGTM

LGTM


Question On Params

I could not find any information on using path params or query params can you help on this?

@JaspervanRiet is this supported?

I'm sorry, but I don't understand your question. DuckRouter does not use URLs, so path/query params are not relevant to it. I recommend reading the README.

@JaspervanRiet is there a way to pass data then? I read the readme but don't see any reference.

Ah I understand your confusion, the README needs some improvement there. I have made #37 to fix it.
The short answer is, like this:
class DetailScreen extends Location {
  const DetailScreen({required this.recipeOverview});

  @override
  String get path => 'detail/${id}';

  final int id

  @override
  LocationBuilder? get builder => (c) {
        return DetailScreen(id: id);
      };
}

This is one of DuckRouter's most powerful concepts. These are just classes, so you can use any type, as many parameters as you want.

Thanks @JaspervanRiet


Upgrade to vscode 1.24.1 fails on Win 10 Enterprise Version 10.0.15063 Build 15063

VSCode Version: upgrade from 1.24.0 to 1.24.1
OS Version: Win 10 Enterprise Version 10.0.15063 Build 15063

Steps to Reproduce:

Select "New Code update available" in bottom left corner of vscode 1.24.0

A "Visual Studio Code 1.24.1 is being installed" message appears

" appear

Then 2 messages, 1 "Restart Visual Studio Code" and the other "Visual Studio Code 1.24.1 is being installed in the background

This lead to this install log
vscode-inno-updater.log

Does this issue occur when all extensions are disabled?: Yes
vscode-inno-updater.log
Note: Add/Remove already shows vscode at version 1.24.1

While vscode > About shows correct version 1.24.0 :
Version 1.24.0
Commit 6a6e02cef0f2122ee1469765b704faf5d0e0d859
Date 2018-06-06T17:35:40.560Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

You seem to have #47841.
Please read this comment if you can provide more detail: https://github.com/Microsoft/vscode/issues/47841#issuecomment-393101777
Please read this one to disable background updates: https://github.com/Microsoft/vscode/issues/47841#issuecomment-395017687

Thanks it works! I am now upgraded to vscode 1.24.1
As in previews ticket I also have this installed ion my work laptop:
CrowdStrike Device Control
CrowdStrike Sensor Platform
CrowdStrike Windows Sensor
Managed Windows Defender
The steps to implement this workaround on vscode 1.24.0 are:
From vscode 1.24.0 > File > Preferences > Settings then:
add a comma after the last entry and before the "}", for me it was:
"gitlens.historyExplorer.enabled": true
}
and add "update.enableWindowsBackgroundUpdates": false
now the end of the "USER SETTINGS" json file looks like:
"gitlens.historyExplorer.enabled": true,
"update.enableWindowsBackgroundUpdates": false

}


Getting Location on Android Emulator more then once

Bug Information
Version Number of Plugin: <IP_ADDRESS>
Simulator Tested On: Android 7.1.1 & Android 6.0
Version of VS: 2017 Update 2
Steps to reproduce the Behavior
When I run the Android Emulator and get the location on the first time it works just fine.
If I try to get the location for the second, third and so on times, I get an Exception:
"System.Threading.Tasks.TaskCanceledException"
(https://gist.github.com/MarcoLLeite/80fda91dd7790495f591c5305ea571dd)
When I restart the emulator it works again for the first time and then the problem comes back.
The permissions for ACCESS_COARSE_LOCATION & ACCESS_FINE_LOCATION are set.
I found someone on stackoverflow with the same problem but there is no clear solution:
(https://stackoverflow.com/questions/43307447/xamarin-xlabs-geolocator-not-working-on-android/44629595#44629595)
Expected Behavior
Get Latitude and Longitude
Actual Behavior
Exception thrown
Code
            var locator = CrossGeolocator.Current;

            locator.DesiredAccuracy = LocationPositionDesiredAccuracy; //100 is new default

            var position = await locator.GetPositionAsync(timeoutMilliseconds: LocationPositionTimeout);

            LocationLatitudeEditor.Text = position.Latitude.ToString();
            LocationLongitudeEditor.Text = position.Longitude.ToString();

You have to go into the emulators settings and press Send. It is the emulator, not the code.

Thank you 👍


WSOD when displaying content logged in (dev)

Today, I updated Hubzilla dev and some relevant Debian packages: MySQL (5.5.53), libapache2-mod-php5 (5.6.27), and php5 (5.6.27). Not sure which of the updates lead to my problem, but:
As soon as I'm logged in, I'm getting a WSOD on pages like network/, channel/, display/ (not sure which else, it seems any page that displays network content), but not on pages like apps/, admin/, events/ etc. Unread counts and notifications are shown on pages that work.
There are no error messages anywhere, not in php.out (.htconfig.php lines active), not in dbfail.out, not in the normal Hubzilla log even when set to Loglevel All, not in Apache's error logs. The only thing I see is a 500 in the Apache access logs.
It works if I'm not logged in, i.e. I can access channel/ pages with suitable access rights then without problems. Also, I can still access these same pages when I'm logged in as long as it isn't my channel, i.e. https://hubzilla.zottel.net/channel/zottelszeug is accessible whether I'm logged in or not. When I'm not logged in, https://hubzilla.zottel.net/channel/zottel correctly redirects to the profile page, logged in I get the WSOD.
Even more precise: Showing one's own network or channel doesn't work, i.e. when I switch to the channel zottelszeug, looking at channel/zottelszeug/ yields a WSOD, but when I switch back to channel zottel, channel/zottelszeug/ works again.
As I don't see anything in any logs, I don't know where else to look. Any hints what I could do to help hunt this bug down?
(Note that the PR I just openend doesn't fix this problem, it's just something else I stumbled over while looking at php.out.)

Item table is fine?

Yes, I think so. Just repaired it to be sure (still same behaviour); but I'd expect yes, anyway, as showing another channel's channel page works, but doesn't work anymore after switching to that channel.

Do you have smileybutton addon installed? (Fixing now...)

Yes, I have, though I did test with safemode turned on? Anyway, it works again now, thanks!


server-sent-events with opine

Issue
Setup:

Deno Version: deno 1.34.2 (release, x86_64)
v8 Version: <IP_ADDRESS>
Typescript Version: 5.0.4
Opine Version: 2.3.4

Details

I was trying to write a simple server sent events example with opine. Ideally the TCP connection for the HTTP request should be open and server should be able to write data chunk to the response stream. Once the whole data is sent or client is disconnected, connection should be closed.

app.get("/server-sent-events", (req: OpineRequest, res: OpineResponse) => {
    res.setHeader("Content-Type", "text/event-stream");
    res.setHeader('Cache-control', 'no-cache');
    res.setHeader('Connection', 'keep-alive');
    res.setHeader("Transfer-Encoding", "chunked");
    res.send(`data: ${currentData}\n\n`);
});

res.send will close the connection after this data is sent to the user. I wanted to use res.write, but It isn't present in opine. How do I implement this with opine?
Below is a similar example for express.

app.get("/server-sent-events", (req, res) => {
    res.header("Content-Type", "text/event-stream");
    res.header('Cache-control', 'no-cache');
    res.header('Connection', 'keep-alive');
    res.header("Transfer-Encoding", "chunked");
    res.write(`data: ${currentData}\n\n`);
});

Hi @AlayKharadi
Since the launch of NPM support in Deno this lib has gone into to “maintenance mode” where I’m not looking to evolve the API any further. I would suggest looking to use Express itself over this library in future.
You’re correct in that res.write() is not available - this is actually an API on the Node response object rather than something Express introduces, and isn’t something that was implemented for Opine.
If you’re open to submitting a PR then happy to review.
res.send() does support a Deno.Reader or ReadableStream, not sure if you might be able to workaround using these?


Deep Evolution

Dear Fingym Team!
I am delighted by the Deep Evolution Agent.
Is there a chance that a version for Forex gets published?
It should be able to do short trading, and the Environment should limit the agent to only buy, sell or  hold one unit at each step. Like this it will be easier to adopt it for Live Trading.
Best regards and keep up the great work!
Matthias

Hey Matthias,
Thanks for the feedback.
You can configure the deep evolution agent to trade one unit at a time via the config value
'max_shares_to_trade_at_once': 1

There is a backlog item to implement short selling in the environments, however, there are no plans to implement a forex environment at the moment. However, I'll add it to the backlog and see when we can get to it. And to be completely honest and transparent, I'm not sure when either of these will be done.
If you want to make one and create a pull request, I would merge it into the codebase.
To allow short selling, look at the class SpyEnv in https://github.com/entrpn/fingym/blob/master/gym/envs/spy_envs.py the _trade method needs to be modified to allow short selling.
For forex, create another environment that extends SpyEnv and override the _load_data() function to pass the forex data.


fixing the refactoring

So, I grow increasingly weary of the ever increasing slowly but surely breakage of my system when working on the refactor branch.
I come to admit that its just too much going on at the same time.
This will be my notes for the whole process, just hopefully much cleaner.
To keep myself from going ever so frustrated about this;

keep every feature in its own branch

this is to segregate it, to avoid as much breakage as possible

bump those version numbers

I cant stress this enough
this makes my life so much easier, as I can with a quick kiss U, revert to a previous working state, instead of having to rebuild 20 packages..

Goals:

[x] Switch to byacc

[x] gcc
[x] mesa
[x] libxkbcommon
[x] linux/linux-lts
[x] awk
[ ] gstreamer

[x] switch to blake3

[x] patch kiss, untill a new release is out
[x] switch all checksums to b3sums

[x] switch to pax

requires a simple patch for kiss, which already exist.
pax is indeed faster than tar, which is reason enough for me.
only package that has given me issues, was ruby, but Im not sure if its was actually pax, or a combination of pax and sbase..
[x] patch kiss for pax
[x] add a pax implementation

currently using otools, which is outdated.
need to find a more up to date implementation for this

mcf has been doing some work on pax, consider that one

[ ] switch to muon

work has already been done. so just continue from there.
I might be able to revert the merging
pkgs that needs altering:

[ ] freetype-harfbuzz
[ ] glib-networking
[ ] gst-plugins-base
[ ] gst-plugins
[ ] gstreamer
[x] libdrm
[x] libmpdclient (dropped)
[x] libxkbcommon
[x] mpc (dropped)
[ ] mpd (dropped)
[x] mpv
[x] tinyalsa
[x] xfsprogs libinih (was bundled with xfsprogs)
[ ] mesa
[ ] pango
[ ] libepoxy
[x] gdk-pixbuf
[ ] atk

thers a patch for gtk+3, but webkit will complain about atk.h is missing, even if atk is installed

[x] switch init

Ive chosen rich felker's init, as its immensly tiny.

[x] rewrite init scripts

on booting;

[x] fix /dev not being properly mounted

on powering off;

[x] properly kill everything before unmounting
[ ] properly unmount partitions when powering off

some partitions return being busy when attempting to umount on shutdown

ubase dont have umount -t flag.
could try to remount read-only instead, then unmount. still reports busy

[x] build ffmpeg statically linked

to my knowledge, ffmpeg can easily be built statically linked, while still enabling shared libs
several packages needs to be forked/altered to enable static libs
[x] alsa-lib
[x] libwebp
[x] libvpx
[x] libvorbis
[x] xvidcore
[x] libass
[x] libdrm
[x] libogg
[x] opus
[x] lame
[ ] x264? (not currently used, but I may consider enabling it)

this is more or less boils down to wether or not the performance has any noticable difference. I see not reason to use it, if there is none

[ ] x265? (not currently used, but I may consider enabling it)

this is more or less boils down to wether or not the performance has any noticable difference. I see not reason to use it, if there is none

[ ] sndio (this is a separate thing, but its related to ffmpeg none-the-less

one thing at a time.
needs a patch to build static libs, and statically link it

[ ] build mpv statically linked

mpv seems to favor shared libs when they exist, and doesnt seem to respect LDFLAGS properly. this needs more looking into.

in any case, a couple of seds on the build.ninja seems to fix this

several packages may need forking/altering
[x] lock to a certain commit to avoid issue with more recent commits

use latest release tarball instead

[x] switch to svc

scripts might be open to improvement, but they so far work sufficently
[x] ubase needs to use ps -A flag, else it wont give us the output we're looking for
[x] make ps output filename unique, to avoid issues when running svc -k
[ ] fix issue if service has mutliple processes, making svc unable to kill the service

just create a loop giving us all the PID's in a list, and that should then be fine.
ubase also has pidof, which does this, but its not POSIX

[x] build sndio statically linked

[x] patch in support for building a static lib
[ ] get sound working again with sndio in general
[ ] get sound on mpv with static sndio, and static mpv

[ ] avoid having packages have unneeded alternatives.

i.e, util-linux provides mount, this is unneccessary, and imo should be avoided unless neccessary.

[x] switch to sbase/ubase (unless something better is found, but I doubt it)

this poses alot of issues, as its lacking in tools.
needs to supplement several of them.
utils that lacks and may or may not need replacement:

[x] ip

iproute2

[x] less

can use cat with head and tail for smaller stuff
for bigger things, pg or more can also be used (util-linux have these)
in any case, theres also gnu's less which is already packaged.

[x] bc

can use gnu's bc. already packaged.

or w/e reason, when busybox owns ed, the buid will fail
not the case with sbase though

or gavin-bc

[x] ping

only used to check connectivity
can use curl -I instead

[x] top

can use ps instead

[x] diff

(otools can provide this. current provider of pax)

just use this untill I figure something else out
otools is based on openbsd 6.8, which is outdated..

[x] patch

(otools can provide this. current provider of pax)

just use this untill I figure something else out
otools is based on openbsd 6.8, which is outdated..

[ ] fsck

I belive I can get by without this.
$(fs)progs should also provide this
in any case, util-linux provides this too if needed

other possible nice to have supplements:

shadow

provides shadow utils, (adduser, addgroup etc)
these are all missing, and can be managed without.

procutil-ng

provides stuff like top, ps, free, kill etc

known issues:

tar breaks the webkit2gtk tarball.(and probably others aswell)

it truncates the outputs, thus messing up files when unpacking the tarball
not an issue with pax. so fixed

webkit2gtk uses a non-POSIX ln flag

removing the unsupported -n flag with a sed breaks the build...

sbase tr gives issues with nss package

fixed

[x] switch to netbsd-curses

[ ] figure out a more up-to-date replacement for otools utils.

cem should reportedly be working on this soon

[ ] find a better tar implementation

semi related to to the above otools section.
sbase tar has some issues when paths are really long

this is an issue mostly with webkit2gtk.

ruby cant be extracted correctly with sbase/otools fixed

[x] when using s/ubase, kiss doesnt seem to properly dependencies

if I delete a dependency of a package, then build said package, kiss will not reinstall said dep.
KISS_FORCE being set to 1, seems to interfer for some reason
set KISS_FORCE=0 instead, and if I need it, set it to 1 manually

More to come, if neccessary

most things is fixed by now. any new issues should rather be in a new issue.


Added an e2e for forceRadonly API

Can you please add a short description of why this test is being added or what this is testing? Like, are you testing that the "forceReadonly" API on container works as expected or that deltamanger emits the "readonly" event as expected.


Updated gradle version

Updated gradle version to latest one

I don't know why my commits are not getting accepted here. Can anyone help me please.
Thanks.
Sincerely,
Shreyash

@ShreyashPromact Don't worry about it, there is an internal problem with Travis configuration, not with your commits. Here's a history of builds: https://travis-ci.org/google/iosched/builds (check just to feel you're not the one with this problem ;-) ).


sitewide navigation does not meet WCAG contrast guidelines

Describe the issue
it looks like that at some point the datatracker.ietf.org nav bar contrast changed and now doesnt meet WCAG guidelines for contrast.  Here is the issue I see reported:
"Element has insufficient color contrast of 4.17 (foreground color: #0b0c0c, background color: #6c757d, font size: 15.0pt (20px), font weight: normal). Expected contrast ratio of 4.5:1
Related Node

"
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Out of curiosity, how did you get this WCAG warning? What service did you use?

I used the aXe extension for Chrome


Upgrading mongo to connect to newer mongo instances.

I did do some work on getting the tests to pass. However it looks as if the tests have been failing prior to this change and need a bit of work themselves.

Changes Unknown when pulling c22f84685a59ec8b9ff595197eead25b8f9f3868 on upgrade-mongo into ** on master**.