Content Identification From Hash

Summary
There can be instances where it is useful to know if two pieces of content are effectively equivalent across versions.
For example, a Page Model which has many Module models which make up its content. Because of the strength of the relationship between pages and modules, it would be fair to say that two pages are equivalent iff their attributes are equivalent and all of their modules are equivalent.
To support that type of use case, we implement a hashing scheme on the tracked models and allow you to specify relationships that also identify the content.
There are a few more hoops to jump through in some circumstances. For example if the Page is Identified by its modules, the Module will also need to know that is Identifying the page, such that when the Module's state changes the Page's hash will be updated.
Also, for identifying relationships through a pivot table you will need to use the packages provided identifyingBelongsTo, identifyingMorphedByMany and identifyingMorphToMany. These exist to update the hash of Identifiable models when pivot rows are attached, detached, synced, etc...
Tests

IdentityTest

Overall this looks and seems to function pretty well! we'll def want to add a bit of documentation around using this. It could even be as simple as mirroring the required bits from the test into the readme.
Also i'm wondering if even a way to toggle it off in the future would be useful

Technically if you set the config snapshots.history.identity to null it should be disabled. I need to add some tests around that and some documentation.


Check domain expirations

SSIA

Domain provided by onamae.com would be expired on April 23rd, 2023.
As domain migration required at least 30 days before expire, could be started migration process until now.
For tracking procedures of domain migration, have newly opened issue #43, so closed this issue.


fix: fix broken js error stack

Summary
JavaScript error stack is broken as NAPI returns err.reason in the latest version.
Caused by: https://github.com/napi-rs/napi-rs/pull/1672

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

!canary


Docker platform is not specified

I got a report that cdk2-python-library-layer tried to build a Docker image for ARM64 on an x86_64 machine and impeded the CDK process. This may have been caused because the platform is not specified to the Docker process.
How about to follow what PythonLayerVersion does?
https://github.com/aws/aws-cdk/blob/ce18037b2ba2b872dd1ed21fda71fd2ee4701353/packages/%40aws-cdk/aws-lambda-python/lib/bundling.ts#L98-L104
    this.image = image ?? DockerImage.fromBuild(path.join(__dirname, '../lib'), {
      buildArgs: {
        ...props.buildArgs,
        IMAGE: runtime.bundlingImage.image,
      },
      platform: architecture.dockerPlatform,
    });

This would make things worse?

However, we should respect compatibleArchitectures anyway.
The platform incompatibility error on Ubuntu (amd64) might be worked around by installing qemu-user-static.


WriteSingleCoilAsync missing

The extension method for WriteSingleCoilAsync is missing from IModbusClientExtensions.
Some other comments on the new NModbus.
Not sure about having to pass ILoggerFactory to the connection strategy, transports and ModbusClient.
My client wrapper has an ILogger already, but for this new code pattern I have had to pass the ILoggerFactory in too.
Could this be changed to use ILogger?
Love the change to all async, even thought its a major breaking change.
Love NModbus. I have been using it for quit a few years.
Do you have any idea when a release version might be available?

The extension method for WriteSingleCoilAsync is missing from IModbusClientExtensions.

Thank you for checking out the code in general and pointing that out in particular. I will happily add that as soon as possible.

Not sure about having to pass ILoggerFactory to the connection strategy, transports and ModbusClient.
My client wrapper has an ILogger already, but for this new code pattern I have had to pass the ILoggerFactory in too.
Could this be changed to use ILogger?

I don't love how I've set up the logging bits. The history:

I started out with ILogger<T>. This made the calling code particularly annoying because having to create a typed instance was unwieldly (e.g. new ModbusClient(loggerFactory.CreateLogger<ModbusClient>).
Just passing ILogger means we would lose context - namely what was doing the logging. This might be an acceptable tradeoff, but I haven't gotten there yet.
This led to ILoggerFactory. This allows for a consistent experience when creating NModbus objects. That said, it's a departure from the past and does still put some work on the caller.
In the current version of NModbus, I use the factory pattern to get around the need for passing logging bits explicitly around. I'm trying to avoid using factories for this version as it feels too prescriptive / dogmatic to require a factory. I also wanted to enable dependency injection for users who want to go down that road.

Do you have any other ideas for how to approach this?

Love the change to all async, even thought its a major breaking change.

Yay! I'm pretty happy with it so far. I'm trying hard to keep the library 100% async as having the non-async method bloats the code quite a bit. That said, if enough users request it, I'll add it back in.

Love NModbus. I have been using it for quit a few years.

Thank you! I'm a fan too. I was using the library for quite a while when the previous maintainer went dark. It has been a struggle maintaining the project without having more test equipment, but maybe I can find a sponsor for that in the future.

Do you have any idea when a release version might be available?

Unfortunately, no. I want to get the serial bits up and running before I make a release version for a number of reasons:

I want to make sure I don't break a released interface if I need to retrofit something in to make serial work.
I'd like to do a better job separating the Modbus protocol (RTU/ASCII/MBAP) from the underlying transport (UDP/TCP/Serial).
Do lots of testing

Thank you for filing this issue and your interest in NModbus!

I also came across that ILoggerFactory thing.
What about using a factory pattern to create the necessary instances?
An IModbusClientFactory and an IModbusServerFactory could create all the instances and could itself be created using DI.
Maybe I can try that a Bit tomorrow and submit a PR.

WriteSingleCoilAsync is available as of version v4.0.0-alpha009 of the package.

Created separate issue for ILoggerFactory: https://github.com/NModbus/NModbus.v4/issues/20


Use of Django for Backend

@ImgBotApp @codervivek5 @Babunshil please assign me this issue as i could shift the codebase to django and build on the project using Django's Class Based Views and implement Authentication,Authorization,Registration

hello sir i would like to hands in this task to resolve it am experience with Django i can implement User Account section such Registration, login, Logout and much more. i'm actively gssoc'23 Contributor.

@ImgBotApp @codervivek5 @Babunshil please assign me this issue as i could shift the codebase to django and build on the project using Django's Class Based Views and implement Authentication,Authorization,Registration

no need to tag me, I am also a contributor brother , I have just contribute on this project at starting


Health check based on content presence in Virtuoso causing deployment issues in test environment

When checking https://github.com/teamdigitale/dati-semantic-kubernetes/blob/test/dati-semantic-lodview/deploymentConfig.yaml, we see that for the following properties, a check is made on /onto/CPV, which refers to the contents of Virtuoso:
readinessProbe:
            httpGet:
              path: /onto/CPV
              port: 8080
              scheme: HTTP
livenessProbe: # FIXME
            httpGet:
              path: /onto/CPV
              port: 8080
              scheme: HTTP

Essentially, if /onto/CPV is not present in Virtuoso, OpenShift does not consider the pod as "ready".
PS. This can also cause lodview to be unreachable, as happens now in test environment.

Nice spotting of the problem.
If this is blocking, could you deep the problem and find a solution?

Nice spotting of the problem. If this is blocking, could you deep the problem and find a solution?

we should change readiness and liveness paths to /, so that they don't depend on the presence of some content, rather on the service itself.

ok, now we managed to deploy in test, so I think the issue is resolved @AgostinoPurificato @Clou-dia

We can close this issue. The tests were successful. Thank you
@AgostinoPurificato @Clou-dia @gnespolino

Possiamo chiudere la issue, grazie
@AgostinoPurificato @Clou-dia @mauropratesi


Support setting client name from sentinel pool

@tgyurci this change will break backwards compatibility as it changes a current constructor. I guess it's better to create a new constructor with the new field to prevent this situation.

I have created a new constructor with same arguments and it calls the new one.

@tgyurci you're right github diff tricked me.
One another collaborator reviews it we can merge. Thanks for contributing!.

@tgyurci LGTM. It is already included to JedisPool, so adding it to JedisSentinelPool is totally make sense.

Thanks @tgyurci for contributing, I merged this to master, 2.7, 2.8 branches respectively.


Missing link with firebase crash

I just try to use sample app, and there is missing link dependencies for firebase crash.
After I added this dependency, I could see crash report from firebase dashboard.
This is error message that I could receive
com.google.firebase.crash.FirebaseCrash is not linked. Skipping initialization.

I signed it!


fix: Ignore schema casing.

This helps to not need to declare INFORMATION_SCHEMA in order to avoid alembic attempting to autogenerate it. In particular, in testing we use fakesnow, which doesn't have perfect emulation (lowercase information_schema)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall first build on dc/snowflake-ignore-schema-case at 96.668%

Totals

Change from base Build<PHONE_NUMBER>:
96.7%

Covered Lines:
2522

Relevant Lines:
2603

💛  - Coveralls


Set error status if could not get PR number

If a PR is older github might not provide a PR number reference.
Set an error status with instructions to update the branch.

Pending #41

Addressed in a58f688fb2afc1da0525ab2b2d1801269a50f0c3


Bug when using without installation

When installing dotfiles, the terminal is completely closed by using the exit command

.


Fix compilation failure on aarch64 Linux

Thanks for your contribution @ajwdev!
LGTM! Tested on aarm64 successfully :-)

Fixes #20


[Docker] Suggestion: support base path for selfhosting

Hello, I think it would be really handy to support base path so that the css can be accessed behind reverse proxy with url: xxx.xxx.xxx.xxx/themepark.

please test -e TP_URLBASE on the develop tag

Thanks, great work! Seems to be okay. However it is not yet implemented in docker mods for me and putting xxx.xxx.xxx.xxx/themepark url in TP_DOMAIN env variable resulted in sed errors.

use domain.com\/themepark

Great, it works! Thanks!

I have just noticed that when accessing the base theme css files via web, they have an @import statements with url that has http scheme despite having TP_SCHEME=https env variable set up. When I cat the css file, those same urls had https scheme, but they were missing the base path.
So: when acessing

Via web: @import url("http://xxx.xxx.xxx.xxx/themepark/css/base/bazarr/bazarr-base.css?sha=None");
Directly the css file located in /config/www/css folder: @import url("https://xxx.xxx.xxx.xxx/css/base/bazarr/bazarr-base.css?sha=None");

make sure you reverse proxy the https port

Yup, I was connecting to theme-park via http from the reverse proxy point of view. Nonetheless, it seems kind of strange. I would think that the scheme depended entirely on what I set in the env variable. Is this the intended behaviour?


Test suite on the GPU

Background and Context
Currently both of our test suites (CircleCI and Jenkins) run on the CPU. However, some bugs such as #137 are GPU only, so our continuous integration is blind to those issues right now.
Description
Have either one of the test suites run on the GPU or create a new one on the GPU, whichever is easier.

Jenkins Test suite is now working on GPU. The script is dynamic enough to check if any GPU is available to run the tests on it.


Is 'role' required when specifying an account in a TestPlanConstellationNode?

The code throws an exception if it is not given.
The comment says it's optional.

It seems it's not required.


Strategy language updates

Includes:

[BugFix] No longer forget the current term constraint when applying "secret" #STUCK() rule.
#STUCK() is defined as a KLabel instead of an Attribute.
Add file builtin/Rules.java which is analogous to builtin/KLabels.java for "secret"/"magic" rules (eg. the #STUCK() rule).
Emit a STUCK_RULE event on StateLog when that rule is applied.
Replace BASIC-STRATEGY with STRATEGY-ABSTRACT, separate out RULE-TAG-SYNTAX to make parsing definitions with #RuleTag variables easier.

If STRATEGY doesn't import RULE-TAG-SYNTAX, DEFAULT-STRATEGY definitely needs to, or you /will/ break things.


Allow publishing multiple video segments

[x] Update the message parsing
[x] Update the speakers' corner and LRC docs

Should be good to go, checking if everything works smoothly.


Craftable Power Generator From Trolley Cart Parts

Inherit from PowerGenerator class and create two new classes:

PowerGenerator_Unfinished_Gas

Attachment Slot: Electric Motor

PowerGenerator_Unfinished_Electric

Attachment Slot: Gas Motor

Recipe to create a power generator should be:
1x Trolley Cart tank + 1x electric or gas motor.
If gas motor is used, produce a PowerGenerator_Unfinished_Gas object
If electric motor is used, produce a PowerGenerator_Unfinished_Electric object
Player should use a wrench on PowerGenerator_Unfinished_Gas with an electric motor equipped in the attachment slot to make a PowerGenerator
Player should use a wrench on PowerGenerator_Unfinished_Electric with an gas motor equipped in the attachment slot to make a PowerGenerator

basic template for this.... and is in use now.   However there is the possibility for an infinite electric motor glitch, hence the idea of creating intermediary "unfinished" powergenerators.
class DayZPlayersClubCraftGenerator extends RecipeBase
{
	override void Init()
	{
		m_Name = "craft Generator";
		m_IsInstaRecipe = false;
		m_AnimationLength = 2;
		m_Specialty = 0.02;

		m_MinDamageIngredient[0] = -1;
		m_MaxDamageIngredient[0] = 3;

		m_MinQuantityIngredient[0] = 1;
		m_MaxQuantityIngredient[0] = -1;

		m_MinDamageIngredient[1] = -1;
		m_MaxDamageIngredient[1] = 3;

		m_MinQuantityIngredient[1] = 2;
		m_MaxQuantityIngredient[1] = -1;

		InsertIngredient(0,"Spur_Kart_FuelTank");

		m_IngredientAddHealth[0] = -60;
		m_IngredientSetHealth[0] = 0;
		m_IngredientAddQuantity[0] = 0;
		m_IngredientDestroy[0] = true;
		m_IngredientUseSoftSkills[0] = true;

		InsertIngredient(1,"Spur_Kart_Engine");

		m_IngredientAddHealth[1] = 0;
		m_IngredientSetHealth[1] = -1;
		m_IngredientAddQuantity[1] = 0;
		m_IngredientDestroy[1] = true;
		m_IngredientUseSoftSkills[1] = false;

		AddResult("PowerGenerator");

		m_ResultSetFullQuantity[0] = false;
		m_ResultSetQuantity[0] = 1;
		m_ResultSetHealth[0] = -1;
		m_ResultInheritsHealth[0] = -1;
		m_ResultInheritsColor[0] = -1;
		m_ResultToInventory[0] = -2;
		m_ResultUseSoftSkills[0] = false;
		m_ResultReplacesIngredient[0] = -1;
	}
	override bool CanDo(ItemBase ingredients[], PlayerBase player)
	{
		return true;
	}

	override void Do(ItemBase ingredients[], PlayerBase player,array<ItemBase> results, float specialty_weight)
	{
		Debug.Log("Recipe Do method called","recipes");
	}
};


Error is raised for problematic rows

In some of the fastq files that I have, there are some problematic rows, such as this:
AAAAAEEEE...
@NB552437:11:HY5W2BGXG:1:11101:12897:1306 1:N:0:GAATTCGT+GTCAGTAC

+

@NB552437:11:HY5W2BGXG:1:11101:22688:1308 1:N:0:GAATTCGT+GTCAGTAC
CAGCGAGGG...
+
AAAAAEEEE...
@NB552437:11:HY5W2BGXG:1:11101:9713:1313 1:N:0:GAATTCGT+GTCAGTAC
TTTTCTTGA...

These problematic rows crash the code and the error is:

Expected '+', got
ERROR: '+' expected

I tried just deleting the empty rows but it still doesn't work.
Trimmomatic seems to be able to bypass this issue (it doesn't crash)

Hello. I am having the same problem, which seems to be identical to #534.
"Expected '+', got CCACAGTGAGCCGCCGATTTGTTCCATGCCTGCAC...
It seems like it causes the run to terminate at the problematic read rather than continuing. @TomerAntman are the quality threshold parameters you're using for trimmomatic the same for the fastp run?

@nickgladman yes they are as far as I remember. I solved this the other day by running seqkit sana on the files and then seqkit pair. It solved the issue for most of my files

Could you please upload a piece of your data so that I can reproduce this issue?

------------------ Original ------------------
From: @.>;
Date:  Sun, Jun 2, 2024 03:04 AM
To: @.>;
Cc: @.***>;
Subject:  Re: [OpenGene/fastp] Error is raised for problematic rows (Issue #560)

Hello. I am having the same problem, which seems to be identical to #534.
"Expected '+', got CCACAGTGAGCCGCCGATTTGTTCCATGCCTGCAC...
It seems like it causes the run to terminate at the problematic read rather than continuing. @TomerAntman are the quality threshold parameters you're using for trimmomatic the same for the fastp run?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Flutter Web - Mobile Browser Bubble viewing problem

I can't see bubbles on my phone in flutter web application. Do you know any suggestion about it ?

Same here

Same here, anyone to help with this bug?


work around test failute on v0.5-dev (#20)

Seems to be related to https://github.com/JuliaLang/julia/issues/17170.

Huh? Because position is used as a function name somewhere else maybe? Very odd, sounds like a bug.
(Why isn't Travis running here?)

The behavior is unpredictable to me. So, I renamed all position variables that may collide with Base.position to p. I think this should be fixed in the upstream.
I don't know why Travis won't run. Maybe I need to kick off it again.

try closing and reopening the issue? that works for me sometimes

@tkelman Thanks. Nice tips.

[Current coverage][cc-pull] is 61.13%

Merging [#22][cc-pull] into [master][cc-base-branch] will decrease coverage by 20.68%

@@             master        #22   diff @@
==========================================
  Files             5          5
  Lines           286        283     -3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            234        173    -61
- Misses           52        110    +58
  Partials          0          0

Powered by Codecov. Last updated by [ec31f4e...b3175f5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/BioJulia/BufferedStreams.jl/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/BioJulia/BufferedStreams.jl/compare/ec31f4e8924f94de90e90d681621b837ee2b16a0...b3175f530adb3c3c0e2c8c0c20a40f7e56cfd3cc?src=pr
[cc-pull]: https://codecov.io/gh/BioJulia/BufferedStreams.jl/pull/22?src=pr

The great decrease of code coverage would be not related to this pull request.

This hasn't been tagged yet, has it?

Not yet. I'll tag it soon.

thanks. oh, and did the issue get reported on base julia? was it equivalent to something already open?

Yes, I guess this is the same: https://github.com/JuliaLang/julia/issues/17170.


Monitor the health of your Azure virtual machine by collecting and analyzing diagnostic data  Exercise - Set up a VM with boot diagnostics

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --identity

Expected Behavior
Environment Summary
Linux-4.15.0-1089-azure-x86_64-with-debian-stretch-sid
Python 3.6.10
Installer: DEB

azure-cli 2.7.0

Additional Context

hello @mgoular could you pls share more detail about the ask and repro steps? Do you need a doc on how to monitor health of vm by setup VM with boot diagnostics? Or did you meet any error during the operation? Pls share more detail infoo to help investigate, thanks.

No response for long time, closed.


FancyZones Explorer/Shell Apps Sizing Stuck

Microsoft PowerToys version
0.64.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Initial setup done with 2 monitors, laptop and 49" ultrawide. Laptop 1 zone, UW 3 zones.
Using laptop with no attached monitor, open Explorer or Control Panel via any mechanism.
✔ Expected Behavior
Explorer and Control Panel open to the size they should be (whatever it is), and remain in position. without being minimized in size and shoved into the upper left of the monitor.
❌ Actual Behavior
Explorer and Control Panel  (possibly other apps) open and immediately transition to the upper left corner of the screen at the minimum size possible (meaning they cannot be reduced in size manually).

They can be resized via dragging, so they are not hung in size.
If I temporarily disable FancyZones, the apps open to the expected sizing and do not move.
This is repeatable across restarts.
I do not have my ultrawide currently, but I will borrow a 1080p monitor from a user at my current location, test again, and post a follow-up.
Other Software
No response

Can you please also attach a /bugreport and some hints how can I reproduce this issue?

Upon attaching a different monitor than my ultrawide, the issue resolved. I am now back on the UW, and the issue has not returned. I have attached the bug report file.
PowerToysReport_2022-12-04-13-37-43.zip


Accepting files in MVC on Linux (kre beta3) corrupts those files

Here's an example project which returns the same file back to you:
https://natori.aspbooru.de/Files/public/thisbreaks.zip
On my CentOs box, KRE beta3, everything else beta3, this should return the same file but it actually returns a corrupted version, a little larger. On Windows this works with any server (Kestrel, IIS, Listener...), it only breaks on Linux/Mono.
I just realized: Sorry for putting this issue up in home, I should have put it somewhere else, though I am actually not sure where.
This is the controller action summing the error up:
    [HttpPost]
	public Task<ActionResult> Add(IFormFile file)
	{
		return new FileStreamResult(file.OpenReadStream(), "application/octet-stream");
	}

The form's encoding is multipart/form-data.

I uploaded an image and posted the result here:

@sinni800 Can you upload your repro project as a github repository instead?

/cc @ajaybhargavb @Tratcher

@davidfowl https://github.com/sinni800/aspnet_errordemonstration
You will need to set the $TEMP variable (to /tmp for example) for it to work. Otherwise it will throw a DirectoryNotFoundException.

Additional info: I tried using a bleeding edge version of Mono (mono-20150311171222) instead of the 3.12 release and it seems to work...

Mono bug then?

@davidfowl I am unsure. Could be in this case but it would have to be tested with old ASP.NET probably. Maybe this bug only creeps up in vNext though.

If you can repro with the latest bits reopen this bug


use camelCasedObjectMapper in DocsController  for endpoint /swagger.json

This PR resolves #82.
I checked that it works as intended, by adding SecurityScheme for the sample app.

Looks great, thanks! Will try to get a new build kicked off today


(maint) Fix --hypervisor vagrant_libvirt

Previously the options[:hypervisor] did not end up in the hosts' config.  This refactors the hypervisors in a common method which always copies the original name into the resulting host config. It still prefers the explicit host hypervisor over the command line option.

Can one of the admins verify this patch?

Note I did take the liberty to refactor some internal code to reduce duplication. I did consider changing the get_platform_info to handle multiple hypervisors instead of doing that in the hypervisor class itself.


Change Timelines for Alexander

The current style of the alexander timelines does not match the rest (contains X uses skill Y etc instead of :X:SkillID:)
see #986

This is almost done, other then a7s.  Because of the branching nature, I need some more Network log files of pulls that see the whole fight.


Problem with stepper styling

Problem description
Hello! I am using material-ui with MERN.
I try to implement stepper.
Everything works great but it looks like styling or autoprefixing is broken

Take a look at that code:

I found it is because wrongly prefixed display property.
When I fixed it like this:

everything looks pretty.
I also wonder what is this mui-prepared...
Steps to reproduce

Use MERN.
Install material-ui
Use code for vertical linear stepper

Versions

Material-UI: 0.15
React: 15.0.1
Browser: Chrome 50.0.2661.102

@maciejmyslinski
I THINK this is some combination of the new rendering method in React and how the inline prefixing library works.

@nathanmarks what can I do about it? Do you have any advice for me?

I had something like this:
const muiTheme = getMuiTheme({}, {
  userAgent: 'all',
});

and changed it to
const muiTheme = getMuiTheme({}, {
  userAgent: 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.82 Safari/537.36',
});

it works now, and I am able to develop the app further but certainly I have to fix it before going to production. Please help me to figure out proper inline prefixing with react.
BTW with
const muiTheme = getMuiTheme({}, {
  userAgent: 'false',
});

it didn't work also.

@maciejmyslinski The issue unfortunately is not with material-ui, we use an external library for prefixing. Can you please post here for help? https://github.com/rofrischmann/inline-style-prefixer

@nathanmarks @maciejmyslinski It is actually not possible to use the prefixer without a userAgent (which defaults back to inline-style-prefix-all), because React 15+ does not allow passing fallback strings as inline style.
This actually breaks inline styles as they can't be used to support compatible CSS output, but they're already fixing this by introducing a new helper https://github.com/facebook/react/pull/6701.
The prefixer API (at least the prefix-all by now) is already updated to return an array e.g. ['-webkit-box', '-moz-box', '-ms-flexbox', '-webkit-flex', 'flex'] which material-ui must either pass to the new helper soon or transform to css by doing sth like prefixes.join(';' + dashCase(property) + ':').
Therefore this is not really a bug within the prefixer but rather an issue with inline styles right now.

Thank you for this research @rofrischmann!

For anyone facing this issue with userAgent: 'all' and doing server side rendering.
Here is my hack-around:
const muiTheme = getMuiTheme();

if (process.env.PLATFORM === 'server') {
  const prepareStyles = muiTheme.prepareStyles;

  muiTheme.prepareStyles = (style) => {
    style = prepareStyles(style);

    if (typeof style.display === 'object') {
      style.display = style.display.join(';display:');
    }

    return style;
  };
}

...

<MuiThemeProvider muiTheme={muiTheme}>

big thanks @oliviertassinari. Was hitting this issues in a Meteor SSR app. Works great with this hack.


add run-length-encoding

nice, thanks

There were no objections in 48 hours (and a few more). It is time to merge. Thank you for your contribution!


All imgproxy instances become unhealthy at the same time

Hi!
We've had quite some trouble with our Imgproxy instances the last week. For some reason all 3-4 ec2 instances, managed by beanstalk, become unhealthy at the exact same time. Digging through the logs i found an "oom" comment in the docker logs indicating they were out of memory. However there was no load in the period so there was 2gb free.
All trails end and there are no clear indications on why all imgproxy docker images shut down or restart.
We're running docker-compose in ELB with 3-4 t4g.small instances.
I'm no devops expert an I'm not sure how to best go about this. They are only running Imgproxy and the fact that they all go red the same time, for me, is odd. Also - they have been running smoothly for the last 6 months or so - nothing has changed on our end.
Have you changed anything in verifying/limiting running threads for pro users?
Our compose:
version: "3.9"
services:
  imgproxy:
    container_name: imgproxy
    image: us-docker.pkg.dev/imgproxy-pro/v3/imgproxy:latest-ml
    restart: always # Added to as an attempt to midigate
    deploy: # Added as an attempt to avoid OOM
      resources:
        limits:
          cpus: "2.0"
          memory: "1.5G"
        reservations:
          cpus: "1.5"
          memory: "1G"
    ports:
      - 80:8080
    environment:
      - DEBUG=true
      - LOG_LEVEL=debug
      - IMGPROXY_DOWNLOAD_TIMEOUT=30
      - IMGPROXY_ENABLE_VIDEO_THUMBNAILS=true
      - IMGPROXY_ENABLE_WEBP_DETECTION=false
      - IMGPROXY_ENFORCE_WEBP=false
      - IMGPROXY_MAX_ANIMATION_FRAMES=9999
      - IMGPROXY_MAX_SRC_RESOLUTION=500
      - IMGPROXY_NEW_RELIC_APP_NAME=Imgproxy-shared
      - IMGPROXY_NEW_RELIC_KEY=***
      - IMGPROXY_QUALITY=100
      - IMGPROXY_REPORT_DOWNLOADING_ERRORS=true
      - IMGPROXY_S3_REGION=eu-central-1
      - IMGPROXY_USE_S3=true
      - IMGPROXY_TTL=31556952
      - IMGPROXY_VIDEO_THUMBNAIL_SECOND=0
      - IMGPROXY_WRITE_TIMEOUT=30

Hey @schonert!
Sorry, I don't quite understand if it was imgproxy container that OOM'ed or Docker itself. And was it a single incident or a regular one? The fact that all the instances went down at the same time makes me believe that was something on the AWS side.
We changed nothing in Pro limits. The only thing that could affect RAM usage that comes to mind is that we now warm up object detection models at launch. But those models would use their memory sooner or later anyway, and we actually decreased memory consumption by ML stuff in the latest release. So this shouldn't affect a thing.

From the docker it indicated that the container went OOM. This was before i declared resources. Not seen the OOM since - they do, however, still regually all go red at the same time.
If nothing has changed on your side - I'll go bother someone else. Thanks for the fast response!
Also - If you have any suggestions on how to go about investigating this I'll appreciate any input.


Bump mongo-client.version from 4.3.4 to 4.6.0 BOT TEST PR

Bumps mongo-client.version from 4.3.4 to 4.6.0.
Updates mongodb-driver-sync from 4.3.4 to 4.6.0

Release notes
Commits

Updates mongodb-driver-core from 4.3.4 to 4.6.0

Release notes
Commits

Updates mongodb-driver-legacy from 4.3.4 to 4.6.0

Release notes
Commits

Updates mongodb-driver-reactivestreams from 4.3.4 to 4.6.0

Release notes
Commits

Updates bson from 4.3.4 to 4.6.0

Release notes
Commits

updated-dependencies:

dependency-name: org.mongodb:mongodb-driver-sync
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-name: org.mongodb:mongodb-driver-core
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-name: org.mongodb:mongodb-driver-legacy
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-name: org.mongodb:mongodb-driver-reactivestreams
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-name: org.mongodb:bson
dependency-type: direct:production
update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>

I forget the context of this, but it seems we can definitely close it. Was it a rampaging bot?


docker-proxy-close-dangling-sockets

Make docker-proxy more resilient to CLOSE_WAIT and FIN_WAIT2 socket states.
In case of unexpected connection terminations, do not wait indefinitely until both ends are closed.
Signed-off-by: Jason Kim<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:meermonster10/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

+1

+1

Note we have migrated this codebase over to github.com/moby/moby/libnetwork.
We are not accepting PR's on this repo anymore except for backports to be included in moby 20.10


Knowledge panel lite: Subject card not styled well and does not show "view full record"

The item at catalog/11019769 has the subject "Civil War, 1861-1865".  The knowledge panel does not show the full record link and has styling issues with information placement.

Regarding the full record link not showing up: The knowledge panel gets the authority html from /browse/info.  The browse info view (https://github.com/LD4P/blacklight-cornell/blob/dev/app/views/browse/info.html.erb#L239) checks for an XHR request and will add the full record link at the bottom, and this is the one the code replaces with the link we want instead.  What is happening in the Civil War example is that the HTML returned from the browse info ajax request does NOT have a full record link in it.  There is some logic around the heading type (which is probably coming from Solr) and parameter passed to the page for type that does not show the link.
One method to always display the record link for an authorized heading would be to not rely on the original html and just generate the link and include it independently.

I set up a custom controller and view for the knowledge panel which gets data from the browse subject and author indices.  This enables displaying information in a controlled fashion and including the view full record link where required.


Add option to select Ruling option for under 30 with Master's degree

Hi,
thanks for the page and for the latest improvements. I think that the only thing missing is a check box for selecting 30% ruling option for under 30 with master's degree.
Regards and thanks again!

yes, it's there Young employee with masters


Does not support IE11 [defineProperty: Unsupported property or method ]

Object.defineProperty(exports, '__esModule', {
    value: true
});

I use the latest clappr which is located in "http://cdn.clappr.io/latest/clappr.min.js"

"

@nxtreaming is still happening?


Boogie on .NET 5.0 ?

I couldn't find a duplicate of this, though I figure it has been asked before.  Currently, it seems like you need dotnet-runtime-3.1 to run Boogie on Linux.  Otherwise, you end up with an error message like this:
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '3.1.0' was not found.
  - The following frameworks were found:
      5.0.7 at [/usr/share/dotnet/shared/Microsoft.NETCore.App]

You can resolve the problem by installing the specified framework and/or SDK.

It seems to me that modern Linux distro's support .NET 5.0 and it seems to generally be the default choice.  So, I'm always either manually installing dotnet-runtime-3.1 or asking someone to do it for me.  Anyway, I don't know what the trade-offs are here.

I think it is a good idea to move Boogie to .NET 5.0.
@RustanLeino, @zvonimir , @bkragl :  Any opinions?

I think it is a good idea to move Boogie to .NET 5.0.

I agree. I've put up at PR.

Great --- thanks!


Pass parameter in f7-link

Hi!
I'm using version 2.2.1 of framework7-vue.
This is my app initialization
...
new Vue({
  el: '#app',
  template: '<app/>',
  framework7: {
    id: 'io.framework7.testapp',
    name: 'Framework7',
    theme: 'auto',
    routes: Routes,
  },
  components: {
    app: App
  }
});
...

And routes are defined like this:
export default [
...
  {
    path: '/customer/edit/:customerId/',
    component: CustomerEdit,
  },
...

And in the Customer-List I include a f7-link like this:
<f7-link :href="patient | toUrl">Edit</f7-link>

with filter
filters: {
      toUrl(patient) {
        return "/customer/edit/" + customer._id + "/";
      }
    },

How can I access the parameter customerId in CustomerEdit-Component? The property customerId is not shown in this.$f7.router.params.

In your toUrl filter, where did you get custom._id from?

Sorry, I was replacing the real code into example code and forgot it.
I have a v-for-loop and customer is the list element.

What I also see it must be  $f7.route.params.customerId instead of $f7.rouerR


No module named "modules.utils"

Traceback (most recent call last):
File "C:\Users\vladi\Desktop\comic-translate\comic.py", line 3, in
from modules.utils.download import get_models, models_data
ModuleNotFoundError: No module named 'modules.utils'; 'modules' is not a package
Already downloaded all req, downloaded utils but not working

Uninstall python. Run the installer again but make sure to tick
"Install Python to PATH" during the setup
Let me know if that works

Using python before for study, kind of prog and for stable diffusion (python 3.10 installed for it). Because of that path is confirmed (also add some kind of it to path myself with windows)

are you running "python comic.py" directly/as is or from some other folder ?

directly from folder

in the comic.py file, after line 2 (import os), add
import sys
print(f'Running from {os.getcwd()}')
sys.path.insert(0, os.getcwd())
See if this works

#move folder from desktop to disk

BTW, found something. when i renamed folder modules to modls and change it also in comic.py and pipeline.py it's work...
I think my python was trying to find real modules.py from python folder in appdata

And now it's error responses...

Okay so the Gui loads now?
What progress does the error start at ? i.e 'forming textblocks', 'Translating' etc

Gui working. This settings and now it's translate i t
hink somehow? But it's translate to [] and some text is not translated

Change your Font to Notosans Medium. animereg does not support Russian.


Add configuration to set base_dir

A map's base_dir is what's used to resolve relative paths used to reference external resources.
When reading the map from standard input the base_dir is also taken as the map's filename, and as such printed togheter with error messages by Mapnik.
Currently it is set to the dirname of the current working directory at server startup time.
It may have implications for https://github.com/Vizzuality/grainstore/issues/4

I see no need to implement this, closing as has been opened for a while now.


Error while

Hi! I'm using WAL-E (it's awesome because is just the right tool we need) and I'm getting the next error while fetching a backup from S3:
lzop: Inappropriate ioctl for device: <stdin>
wal_e.retries WARNING  MSG: retrying after encountering exception
        DETAIL: Exception information dump:
        Traceback (most recent call last):
          File "/usr/local/lib/python3.5/dist-packages/wal_e/retries.py", line 62, in shim
            return f(*args, **kwargs)
          File "/usr/local/lib/python3.5/dist-packages/wal_e/worker/s3/s3_worker.py", line 78, in fetch_partition
            TarPartition.tarfile_extract(pl.stdout, self.local_root)
          File "/usr/local/lib/python3.5/dist-packages/wal_e/tar_partition.py", line 284, in tarfile_extract
            cat_extract(tar, member, relpath)
          File "/usr/local/lib/python3.5/dist-packages/wal_e/tar_partition.py", line 219, in cat_extract
            copyfileobj.copyfileobj(fp, pl.stdin)
          File "/usr/local/lib/python3.5/dist-packages/wal_e/copyfileobj.py", line 15, in copyfileobj
            shutil.copyfileobj(src, dst, BUFSIZE)
          File "/usr/lib/python3.5/shutil.py", line 73, in copyfileobj
            buf = fsrc.read(length)
          File "/usr/lib/python3.5/tarfile.py", line 698, in readinto
            buf = self.read(len(b))
          File "/usr/lib/python3.5/tarfile.py", line 689, in read
            raise ReadError("unexpected end of data")
        tarfile.ReadError: unexpected end of data

        HINT: A better error message should be written to handle this exception.  Please report this output and, if possible, the situation under which it arises.

My installation is:

ubuntu_x64==16.04
python==3.5.2
appdirs==1.4.3
boto==2.46.1
gevent==1.2.1
greenlet==0.4.12
packaging==16.8
pyparsing==2.2.0
six==1.10.0
wal-e==1.0.3

Maybe some of these are wrong :persevere:

This one was before #333, but maybe one could close this as duplicate?

Yep, Thanks @tschwaerzl.


DRAFT: refactor Utility namespace

BigIntegerSerialization was unused so let's
remove it.

@aarani what's left to do here?

@aarani what's left to do here?

The PR title is misleading because I never actually did "refactor Utility namespace". There's currently only one commit that removes an unused file.

So?

So?

Since I don't want to refactor Utility namespace for now, I synced the PR title with the only commit msg it had.


std::merkle functions break for path array of length >254

Aim
I tried to implement a sparse merkle tree using std::merkle::compute_root_from_leaf and std::merkle::check_membership. However, the compiler gives a strange error when I try to use std::merkle::check_membership:
main.nr:
mod smt;

fn main(start_root: pub Field, value: pub Field, path: [Field; smt::SMT_SIZE]) -> pub Field {
  smt::insert(start_root,_ value, path)
}

smt.nr:
use dep::std;

global SMT_SIZE: Field = 256;

fn insert(start_root: Field, value: Field, path: [Field; SMT_SIZE]) -> Field {
  constrain std::merkle::check_membership(start_root, 0, value, path) == 1;
  std::merkle::compute_root_from_leaf(1, value, path)
}

fn delete(start_root: Field, value: Field, path: [Field; SMT_SIZE]) -> Field {
  constrain std::merkle::compute_root_from_leaf(1, value, path) == start_root;
  std::merkle::compute_root_from_leaf(0, value, path)
}

Substituting check_membership for compute_root_from_leaf also does not work but results in a different error and works with SMT_SIZE up to 253 but breaks when trying to get the full size of 256
Expected behavior
I expected the code to compile with no errors.
Bug
For check_membership I get the following error when SMT_SIZE is 253 or below:
The application panicked (crashed).
Message:  internal error: entered unreachable code: invalid input: Const(Constant { id: NodeId(Index { index: 775, generation: 0 }), value: 0, value_str: ""
, value_type: NativeField })
Location: crates/noirc_evaluator/src/ssa/acir_gen/operations/intrinsics.rs:188

This is a bug. Consider opening an issue at https://github.com/noir-lang/noir/issues/new?labels=bug&template=bug_report.yml

For both variants compute_root_from_leaf(...) == root and check_membership(...) == 1 the compiler breaks when the SMT_SIZE is set to above 253.
To reproduce

nargo new your-project
Create main.nr:

mod smt;

fn main(start_root: pub Field, value: pub Field, path: [Field; smt::SMT_SIZE]) -> pub Field {
  smt::insert(start_root, value, path)
}

Create smt.nr:

use dep::std;

global SMT_SIZE: Field = 256;

fn insert(start_root: Field, value: Field, path: [Field; SMT_SIZE]) -> Field {
  constrain std::merkle::compute_root_from_leaf(0, value, path) == start_root;
  std::merkle::compute_root_from_leaf(1, value, path)
}

fn delete(start_root: Field, value: Field, path: [Field; SMT_SIZE]) -> Field {
  constrain std::merkle::compute_root_from_leaf(1, value, path) == start_root;
  std::merkle::compute_root_from_leaf(0, value, path)
}

Run nargo compile

Installation method
Binary
Nargo version
nargo 0.3.2 (git version hash: 29b1f7df4d563849a62e64c533cb62932188135b, is dirty: false)
@noir-lang/noir_wasm version
No response
@noir-lang/barretenberg version
No response
@noir-lang/aztec_backend version
No response
Additional context
No response
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

Since the field size is 254 bits large my sparse merkle tree would need a proof path size of at least 254 :/

@TomAFrench thx for explaining.
Is this something that can be fixed i.e. allowing Field.to_le_bits to decompose to 254 bits or is this a limitation we'll have to live with?
Couldn't to_le_bits just additionally constrain that the value is being decomposed into the lower (non-overflowing) representation if size >= 254 since that parameter has to be known at compile-time anyway?
Thinking about it depending on the situation it may be safe to just use 253 bits rather than field's full bit size of 254 if the user can't directly choose the index value as it's computationally infeasible to find a collision in 126.5 bits (halved bc of birthday attack) for now.

Is this something that can be fixed i.e. allowing Field.to_le_bits to decompose to 254 bits or is this a limitation we'll have to live with?
Couldn't to_le_bits just additionally constrain that the value is being decomposed into the lower (non-overflowing) representation if size >= 254 since that parameter has to be known at compile-time anyway?

Yes, we'd need to think about the most efficient way of doing it however. In the worst case scenario, you can always decompose the index into a bit array inside noir itself however this is quite inefficient which is why we have this optimized version.
Using 253 bits for now seems fine imo while you're experimenting but we'll want to address this issue (along with the errors when using the check_membership function.)
Flagging this issue up to @joss-aztec as he's working on a SMT implementations as well so may be able to provide some insight.

Hey @Philogy, just wanted to let you know that this has been addressed in #1312. I've tested your code again and got it compiling.*

It seems like there's a bit of an issue with passing constant values into compute_merkle_root so your code doesn't compile exactly as it is due to the hardcoded leaf values but that shouldn't be an issue in real usage.

I've opened an issue for it here #1345 and am going to close this issue as resolved.


[Go Live][31 May]

Push to live: https://github.com/MicrosoftDocs/msteams-docs/pull/3520

Docs Build status updates of commit 7ced923:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/get-started/first-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/action-commands/respond-to-task-module-submit.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/add-configuration-page.md
:white_check_mark:Succeeded

msteams-platform/messaging-extensions/how-to/search-commands/respond-to-search.md
:white_check_mark:Succeeded
View

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Tidy up some magic numbers

For example, Help allows the parent component to set its position, but randomly adds 6 pixels:
if (this.props.position) {
    helpStyle = {
        bottom: this.props.position + 6
   }
}

Due to the position of the help component, it is influence by the height of the form__control container and the padding/margin around it. Ideally, it should be positioned purely with CSS. An alternative should be implemented perhaps using the examples as benchmark.


Add model name to the summary method

Our models could have names, but the current version of summary method (for both, Sequential and Functional models) doesn't support model name displaying.
The PR should include:

[ ] model name support for both, Functional and Sequential model
[ ] the missed model name case is handled correctly
[ ] SequentialCompilationTest.summary should be updated for validation model name
[ ] Test for the Functional model summary is added (you could use here model like ToyResNet)
[ ] (Optional) Refactoring of both summary methods (in Functional and Sequential models)

I could do this one :)

@zaleslaw to what extend do both summary methods have to be refactored? E.g. does the method signature have to change at all?

I've changed the ticket description regarding refactoring: the refactoring will be optional, the API is not under stability guarantees and could be changed.
The refactoring is just an idea, looks like it could be refactoring, but it's your choice as a contributor: propose the refactoring (with minor changes in the signature that doesn't affect all examples and tests) because there is some copy-paste there and common logic, but PR could be merged and without refactoring

@zaleslaw do the above commits cover the first 3 points in this issue sufficiently? Or is this not what you meant. And I can look into the refactoring yes.

I've added the Contributing Guidelines. Please check this.

You solution is simple and solves the problem

Maybe the refactor part and writing a test is better suited as a part of #135 than this issue @zaleslaw ?

I suggest do things step by step, the final form of model summary is unclear and there are a few moments related to logging, jupyter integration and so on.
I suggest to finish this ticket and if the form of summary will be changed, we will not ignore the modelname


Typo in 12.1

The very last analysis in 12.1, where the model without level 3 is being compared to the full three-level model using anova(), reads anova(full.model, model.l2.removed) when it should read anova(full.model, model.l3.removed). As written, the code is just repeating the previous anova. [thanks for the great book!]

I've submitted a pull request to fix this.


Remove redundant coords assignment; test for dims and coords

I don't know why this fixes this index dim on some systems, but it seems like it does. I added tests to ensure that only obs and traj are in dims and coords.
@selipot @philippemiron please check on your ends.
Closes #323.

Ok index is gone.
But why not having like for the GDP the coordinates time(obs) and ids(traj) instead of obs(obs) and traj(traj)? I find this not really comprehensible. The coordinate you have currently called obs is effectively a time variable and the traj is a unique identification string.

Works for me too. Only comment: should we put a message to notify the user when the file already exists in ~/.clouddrift/data/?

Yes, let's discuss the naming convention.
My preference may be time(time) and id(id). Unclear to me why name coords differently from dims, as they map 1-to-1 in our case.

And I agree with Shane. I'm not sure I understand the addition of obs and traj as coordinates either.

As discussed on the call, we will rename obs(obs) and traj(traj) coordinates to time(obs) and id(traj) respectively. Dimension names remain the same.


Nest controllers

This is the approach at organizing controllers I have decided on.  This
will match the URL structure that is provided from the Pinterest API and
will feel more fluent than the previous approach I was taking.

Coverage remained the same at 100.0% when pulling 94a3e9dc9150d5c4ecd2c39cc88ac4cc4be3f862 on br.nested-controllers into 7437e046ce506aad8a6c1a1e096815d140068c51 on master.


suffix filenames (reports, collected show commands) with date and time

Can be solved by using following syntax:
python scripts/check-devices.py --inventory .personal/avd-lab.yml --catalog .personal/ceos-catalog.yml --json --tags dc1 --save "demo-$(date).txt"


Unexpected Inputs behaviour in switch, not linked mode

Input states randomly change its's state accroding to output state within 1-2 hours.
Toggle mode also not working in switch, not linked.

recreate:
Flash E18 chip with firmware, connect to zigbee.
Wait 5-15 min, change some output states to true from zigbee.
Wait 1-2 hours. Inputs with the same numbers change their state to 1 from 0.
Firmware ini attached.
DM1_test.zip
Last version tested <IP_ADDRESS>

Problem appears while connected to sputhub, solved by correcting template.


[Feature request] USelectMenu option types

Description
Sometimes in the SelectMenu custom elements are useful to be rendered instead only options.
Dividers:
When in options there is an option like: { divider: true } it should render:

Header:
When in options there is an option like: { header:  'Text' } it should render:

{{ option.header }}

More element types could be added but these are the most common, for example: Link1 Link 2
Thanks!.
Additional context
No response

Any update?

This has been implemented in v3 already.


Derequires build to be friendly with browserified projects

Thanks for your hard work. Your library has been very useful. I'm running into a problem when using your module within a project that also uses browserify. It's unfortunate that browserified dependencies don't play well within a browserified project! The parser looks for the require calls in all the modules (including external dependencies) and tries to resolve them.  That's an old issue that as far as I know hasn't been solved yet. There's a discussion going here: https://github.com/substack/node-browserify/pull/1151
On your side, you can run your build through derequire to make it easier to consume. derequire renames the require calls to _dereq_ so browserify ignores them. No functionality is affected. The only downside is that your build file will take a tiny bit longer to generate.

Hmm... this seems like a real hack.
Is there some better JS module system? Maybe a good time to look into ES6 modules?

This is a pretty common approach to handle this. The real problem is that Browserify doesn't properly deal with this problem by itself. If you have any other idea I will update the PR

A first option could be to just not use browserify to create the build. Closure Compiler does a good job (from my experience) handling CJS modules – and it will get rid of all require() calls in the output, resolving the issue.

Is there some better JS module system? Maybe a good time to look into ES6 modules?

As this project is targeted exclusively at browsers, and is never going to run on a server, I have the impression that using synchronous CJS as module format might not be the best choice. Going for ES6 seems like a good idea; but, it would require yet-another-extra-step during both development time, and for the build.
A second option that comes to my mind is to switch to AMD module format. The main benefit of this would be that no build step and watchers were needed during dev time (i.e. get rid of the watchify dependency). Do your changes to the code, reload in the browser, done. With regards to the build, Closure Compiler can also handle AMD modules well (again, just from my experience).
In any case, i’d wrap the resulting build in a UMD wrapper to make sure ppl can consume it the way they like; but, that is something else, as it might mean that using make isn't sufficient anymore and one would need to switch to Grunt or Gulp.
If there is interest, I could give one or both options a shot over the weekend.

@dmarcos sorry for delay. Looking at this again. Can you link to how you use webvr-polyfill with require?

I have nothing special in my config. The problem is very easy to reproduce:

Create an empty directory
Copy inside the webvr-polyfill dist file
Create a main.js file with just this line require('./webvr-polyfill.js');
Run the following command (this would be the browserify step within my project that would require my own files)
```browserify ./main.js -o ./myLib.js````
Do you see the error?

Now try to run your file through derequire (npm install -g derequire) and try again:

derequire webvr-polyfill.js > webvr-polyfill.js
```browserify ./main.js -o ./myLib.js````
Now it should work

Have you considered using https://www.npmjs.com/package/browserify-derequire ?

Yes, this is what I'm doing and it's fine. This PR makes a preemptive derequire on webvr-polyfill side so users are not surprised (as I was) and have to eventually realize that you're also using browserify and need to include a derequire step. It's unfortunate that browserify doesn't handle this in a more graceful way because it's a common use case.

Thanks, sorry for delay. Merged.


google/material-design-icons update

Hi,
Do you plan to update to the latest version of google/material-design-icons? The 2.1 update says that 96 new icons have been added. One of the forks even have the update: https://github.com/dresslerit/angular-material-icons/tree/google-icon-v2.1-update
Thanks!

This is a great icon pack. I would also like to know when this will be updated. I'm waiting to use some of the new icons and like using the angular directives to accomplish this.

Hi, I've created an pull request (#43) some time ago. I am just waiting until the request is merged.

Thank you, @dresslerit, I have merged your pull request.


Update aws-refarch-wordpress-04-web.yaml

EFS Current ID format has changed since October 2021
to identifier plus 17-character ID
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Changes approved


Add flags to pepr init to allow creation of --deployment / --library modules.

Is your feature request related to a problem? Please describe.
pepr init currently creates what I'm thinking of as a "deployment" module (i.e. a Pepr module intended for direct deployment to a cluster) -- it uses a pepr.ts to build & does not override npm package entrypoint within the package.json dev:build script.
In contrast to that, the pepr-keycloak-authsvc project uses a slightly modified layout that is both:

hand-rolled, and
a modification of the standard pepr build process to produce a Pepr module that is intended not for direct deployment but instead for reuse via NPM import  (a.k.a. what I'm thinking of as a "library" module) -- it uses an index.ts to build & overrides npm package entrypoint within the package.json dev:build script.

This second pattern is useful and I'd like to see it formalized.
Describe the solution you'd like

Given an updated pepr command
When  I run pepr init or pepr init --deployment
Then I should see the "deployment" pattern in use (i.e. just the same as running a naked pepr init does now)

But When  I run pepr init --library
Then I should see the "library" pattern in use (i.e. similar to the way the pepr-keycloak-authsvc project does now)

And I should be able to navigate to some docs page that describes module type differences / usage.

Describe alternatives you've considered
Haven't really -- just trying to formalize the pattern we're already using / make it easier / nicer to use the "--library" module pattern.
Additional context

pepr-keycloak-authsvc pepr.ts vs index.ts / package.json dev:build script.

No longer needed after discussion with team.


build failure on Mac Pro with Apple M2 Chip.

Describe the bug
ld emits warning:
ld: warning: ignoring file /Users/tpa/oqs-provider-git-repo/oqs-provider/.local/lib/libcrypto.dylib, building for macOS-x86_64 but attempting to link with file built for macOS-arm64
and subsequent link step fails due to not finding entry points...
Undefined symbols for architecture x86_64:
"_ASN1_item_d2i_ex", referenced from:
_oqsx_d2i_X509_PUBKEY_INTERNAL in oqs_decode_der2key.c.o
"_ERR_new", referenced from:
_OSSL_provider_init in oqsprov.c.o
_oqsx_key_op in oqsprov_keys.c.o
_oqsx_key_new in oqsprov_keys.c.o
_oqsx_key_fromdata in oqsprov_keys.c.o
_oqsx_key_recreate_classickey in oqsprov_keys.c.o
_oqsx_key_new_from_nid in oqsprov_keys.c.o
_oqsx_import in oqs_kmgmt.c.o
...
etc, etc.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/open-quantum-safe/oqs-provider.git
cd ors-provider
scripts/fullbuild.sh -F

Expected behavior
Expected build to complete without error.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: MacOS Ventura 13.3.1
OpenSSL version:  OpenSSL 3.2.0-dev  (Library: OpenSSL 3.2.0-dev )
Version [e.g. 0.4.0]

Additional context
it appears that openssl is being built for arm64 and oqs-provider is trying to build for X86_64, but I have no idea how to fix this.
note that I get the same result, whether I use the "fullbuild.sh" script, or if I manually build & install openssl and liboqs.
Those components build and run, but when I attempt to use them to build oqs-provider, I get the arch mismatch problem.
I expect this is NOT an issue on an older Mac that is using an intel chip.

Thanks for this report. It's puzzling indeed and I have no M2 to test this on. Build on M1 following the 3 steps outlined above works without problems. Also there is no reference to "x64" anywhere in the code of oqsprovider.
So the best thing I can offer is to review the output of your cmake run: Please share everything output by fullbuild.sh: Maybe there's a wrong compiler setting visible somewhere... In addition, can I ask you to run make run_tests in the liboqs/_build subdirectory created as part of the oqsprovider build? This just to test that cmake works OK for liboqs.

I just compiled on my M2 with macOS 13.4.1 and did not get any errors. I wonder if it has something to do with the OpenSSL version.  How did you install OpenSSL 3.2.0-dev?  For me, cmake finds OpenSSL 1.1.1u via brew.  I do have OpenSSL 3.1.1 installed via brew but the build scripts apparently find the 1.1.1 installation.

Thank you for following up.  I will go back and re-check my steps from
OpenSSL install/build, liboqs install/build (making sure to run the “test”
target), and the oqs-provider build and let you know.
On Wed, Jul 5, 2023 at 6:40 PM Douglas Stebila @.***>
wrote:

I just compiled on my M2 with macOS 13.4.1 and did not get any errors. I
wonder if it has something to do with the OpenSSL version. How did you
install OpenSSL 3.2.0-dev? For me, cmake finds OpenSSL 1.1.1u via brew. I
do have OpenSSL 3.1.1 installed via brew but the build scripts apparently
find the 1.1.1 installation.
—
Reply to this email directly, view it on GitHub
https://github.com/open-quantum-safe/oqs-provider/issues/199#issuecomment-1622790460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6WIS65CTZFGT4QQFVYRVTXOYJPPANCNFSM6AAAAAAZWRUSUM
.
You are receiving this because you authored the thread.Message ID:
@.***>

but the build scripts apparently find the 1.1.1 installation.

This should be fixed by now courtesy https://github.com/open-quantum-safe/oqs-provider/pull/212.
@tpa, please retry with latest code.

Will do, thanks!
On Fri, Jul 14, 2023 at 2:22 AM Michael Baentsch @.***>
wrote:

but the build scripts apparently find the 1.1.1 installation.
This should be fixed by now courtesy #212
https://github.com/open-quantum-safe/oqs-provider/pull/212.
@tpa https://github.com/tpa, please retry with latest code.
—
Reply to this email directly, view it on GitHub
https://github.com/open-quantum-safe/oqs-provider/issues/199#issuecomment-1635566947,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6WIS3PGLHBPWK4OXRCSR3XQEFVBANCNFSM6AAAAAAZWRUSUM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Closing assuming this is fixed. Please re-open if not, providing new logs.


cf restage fails with unrelated error when authentication has expired

Command
$ cf restage our-app
Restaging app  in org our-org / space our-space as<EMAIL_ADDRESS>Warning: error tailing logs
Timed out waiting for connection to Loggregator (wss://doppler.run.pivotal.io:443).

What occurred
We immediately asked for recent logs, and got the good old expired auth failure message:
$ cf logs our-app --recent
FAILED
Authentication has expired.  Please log back in to re-authenticate.

What you expected to occur
We would have expected to see the auth expired error message when running the restage command.
CLI Version
cf version 6.23.1+a70deb38f.2017-01-13
CC API Endpoint Version
API endpoint:   https://api.run.pivotal.io
API version:    2.75.0
Platform & Shell Details
Mac OS X 10.12.3
iTerm2

Apologies - I failed to update this issue.
Thank you for your report! We fixed this in cf CLI 6.29.0. You should now see:
D:\>cf restage testapp
Restaging app testapp in org dies-test / space dev as admin...
The token expired, was revoked, or the token ID is incorrect. Please log back in to re-authenticate.
FAILED

Regards,
Dies Koper
CF CLI PM


Does not propose updates for dependent gems?

From @rdunlop on September 21, 2017 15:54
In my project, I have capybara in my Gemfile, which depends upon nokogiri >= 1.3.3.
Currently, I have nokogiri (1.8.0) installed.
Why hasn't dependabot prompted me to install nokogiri 1.8.1?
Doing bundle update nokogiri locally does succeed in creating a new Gemfile.lockwith an updated version ofnokogiriandmini_portile2`
Copied from original issue: dependabot/feedback#22

Hey @rdunlop,
I've been wondering when this was going to come up! At the moment, Dependabot doesn't bump sub-dependencies, except when bumping the parent dependency (which means #21 is the other side of this issue).
We decided to start out with the above approach because we were worried about the sheer number of PRs we'd be creating when bumping sub-dependencies. By their nature, they're things your less interested in than your primary dependencies, so they were a good candidate to ignore.
However, I can see arguments for updating them, too - the nokogiri example is a good one - 1.8.1 is a security patch, so you definitely want it.
I think the best compromise would be for Dependabot to continue to create single-dependency PRs for primary dependencies, but create an additional "update sub-dependencies" PR each day when required, which would combine many updates. Would that make sense to you? It's quite a bit of work on our side, so won't happen immediately, but I'm keen to get clarity on the best possible solution.

Makes sense. I want to keep thinking on this one, so I'm not going to ship anything immediately, but it's really high up my list of problems to solve.
FWIW, we've tried to avoid having a configuration page because we quite like the interface for Dependabot to be entirely pull request based. That won't necessarily last forever, but I don't want to add a configuration page / file before thinking through all the other options.

From @rdunlop on September 22, 2017 12:7
Hi @greysteil,
I do like the idea of having ALL of my gems updated to the latest version, all the time. If I installed a gem fresh, all of its sub-dependencies would be "latest"...which is my goal for my projects...so I think that this is a good feature.
I currently really like the fact that each PR only contains a single gem in it. If you're proposing that the "sub-dependencies" PR would combine many unrelated gem version changes, doesn't feel right to me.
I am warming to your idea of batching them together though...because it sounds right that we care about certain gems more than we do others..... Perhaps we could do sub-dependencies after all of the other ones are done? (ie: if I have less than 5 PRs tonight...look for gems in the sub-dependencies group?).
Alternatively, perhaps it will require configuration/options to be allowed on a per-project basis. e.g. "check for updated dependencies of gems" option? This configuration page might become very useful, if you can use it to show those gem PRs which are currently open, closed/not-merged, etc.
I know I'm probably mixing many features together.

From @rdunlop on September 28, 2017 13:10
Hi @greysteil I may be seeing a related (or unrelated) issue. I would like your opinion on whether it's the same issue, or a new bug.
Basically, it was my understanding that dependabot was doing the equivalent to bundle update <gem> for each of the gems specified in my Gemfile...and if it succeeded in resolving a gem update, it would open a PR. Is that a fairly accurate summary of the intended dependabot behavior?
I was expecting a PR last night, but I didn't get one:
On rdunlop/unicycling-registration my Gemfile specifies "gem 'carrierwave-aws'".
Currently, my Gemfile.lock says
    carrierwave-aws (1.2.0)
      aws-sdk (~> 2.1)
      carrierwave (>= 0.7, < 2.0)

Yet, last night I did not get a PR for upgrading that from 1.2.0 to 1.3.0?
Doing "bundle update carrierwave-aws" on my command-line results in
git diff
diff --git a/Gemfile.lock b/Gemfile.lock
index 93b0cfa8..eb938c74 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -105,16 +105,16 @@ GEM
     ast (2.3.0)
     autoprefixer-rails (<IP_ADDRESS>)
       execjs
-    aws-sdk (2.10.47)
-      aws-sdk-resources (= 2.10.47)
-    aws-sdk-core (2.10.47)
+    aws-sdk (2.10.54)
+      aws-sdk-resources (= 2.10.54)
+    aws-sdk-core (2.10.54)
       aws-sigv4 (~> 1.0)
       jmespath (~> 1.0)
     aws-sdk-rails (1.0.1)
       aws-sdk-resources (~> 2)
       railties (>= 3)
-    aws-sdk-resources (2.10.47)
-      aws-sdk-core (= 2.10.47)
+    aws-sdk-resources (2.10.54)
+      aws-sdk-core (= 2.10.54)
     aws-sigv4 (1.0.2)
     axiom-types (0.1.1)
       descendants_tracker (~> 0.0.4)

My settings:
bundle -v
  Bundler version 1.14.6
gem -v
  2.6.13

Thoughts? thanks.

From @rdunlop on September 25, 2017 10:49
@greysteil Please take your time to think through this one. We have other means to track sub-dependencies right now, so this is not pressing...just would be really nice to have.
I can understand your desire to avoid a configuration page. I'll keep thinking about it too.

@rdunlop that's definitely related. You're right that Dependabot does the equivalent of bundle update <gem>, but it also has some logic to check whether the version of the dependency it's updating changes. If it doesn't (e.g., in this case), it doesn't generate a pull request, so you'll never get a "Bump sub-dependency of carrierwave-aws" pull request.
Having a general strategy for dealing with sub-dependencies would definitely fix the above - if they were getting updated anyway a PR like the above wouldn't ever be needed.
Unrelatedly, the reason you didn't get a carrierwave-aws PR is because the latest version requires aws-sdk-s3 ~> 1.0, which in turn requires aws-sdk-core ~> 3 (yes, the AWS team have put a ~> specifier on a major version. 🤷♂), which conflicts with the version required by the version of aws-sdk-rails which is in your Gemfile.lock (and is in your Gemfile, so Bundler doesn't automatically bump it). The solution would be for Dependabot to be able to bump multiple dependencies at a time when required, which I'd really like to add but haven't had time to (yet).
To get to carrierwave-aws update locally, try bundle update carrierwave-aws aws-sdk-rails.

From @rdunlop on September 28, 2017 14:1
@greysteil I knew you'd have an explanation for it.
I'm looking at the Gemfile.lock changes that I posted above....and I'm now seeing that it doesn't say what I thought it said. Oops.
You're right, this is totally explainable by #21.
I've seen people put gem "rails", "~> 4" recently.... 🤷♂

From @jalada on January 15, 2018 9:46
Ruby 2.5.0 introduced some bugs in our code that required sub-dependent gems to be updated (specifically around ruby_parser) but because haml-rails is the dependency, dependabot didn't notify us of them.
Just an extra datapoint for this feature; I think it would definitely be valuable.

From @regentgal on January 31, 2018 18:16
I would expect sub-dependent gems to be updated with the parent. The strict 1 gem per PR rule is nice for isolating changes but updating a gem without updating its explicitly stated dependencies (and theirs), you end up with the issues listed here by @rdunlop and @jalada. I think moving to a 1 parent gem and its sub-dependencies per PR rule would be a nice way to limit the number of PRs and still keep changes isolated.

From @rdunlop on January 31, 2018 19:45
@greysteil If haml-rails doesn't need an update, but its dependent gem ruby_parser does need an update, I think that it would be great if we had a PR opened which was something like:
`haml-rails`'s dependent gems update:
- ruby_parser - from version 1 to version 2
- other sub-gem - from version 2.3 to 2.4

Which is the result when you run bundle update haml-rails. Even though haml-rails itself doesn't end up being changed, some other gems in my Gemfile.lock are updated.
I like this idea because it also helps to identify which parent-gem is using these dependencies, so that I can focus my manual testing/review before releasing changes to prod, if I deem them risky/worrisome.
This is not "creates PRs for sub-dependencies on their own", in that it's framing the new PR in terms of the parent/Gemfile-referenced gem.
My 2c.

Hey @regentgal, Dependabot does update sub-dependencies when it's updating a top-level dependency 🎉 . The issues above are because Dependabot never creates PRs for sub-dependencies on their own - in @jalada's case, for example, Dependabot would have included a ruby_parser update if it was updating haml-rails, it's just that haml-rails didn't need an update.
Hope that helps clarify!

Interesting, thanks @rdunlop. Might see some movement on this one in the next couple of weeks as I’m hoping for some progress on #53 (meeting GutHub about it next week)

👍. At this point it would be really straightforward for me to add a setting for this, so I'll do that this week.

From @rdunlop on March 27, 2018 14:9
@greysteil #53 appears to be focusing on updating sub-gems which have security alerts from github.
I agree that that's valuable work, but I still would like to have other sub-gems updated through automatic means if possible.
If we were to concentrate on security warnings as the goal for doing gem updates, here's some thoughts:

Keeping gems updated, means that when a security-warning comes up, and we need to update the gem, we aren't many versions behind the tip, so we are less likely to have issues
There aren't always "security warnings" posted for gems which actually have security issues discovered/resolved. It highly depends on the specific gem community/reach/perspective

In general, I just like keeping all my libraries up to date, and dependabot is doing a fantastic job doing that for most of them...but when I have to manually go through and create nokogiri, loofah, rack-protection, rails-html-sanitizer PRs...I just wish it was doing those for me too.
(I'd be happy with #53, I'd be thrilled with #22)

@rdunlop - I've just shipped the backend for this. If you ping me an email on<EMAIL_ADDRESS>with the repos you'd like it set up on I'll do that now (same goes for anyone else who would like to receive individual PRs for sub-dependencies.
Assuming all goes well I'll add a front-end for this available to everyone in the next couple of weeks.


Visual Code terminal takes too long to abort nodemon with CTRL+C

When you press CTRL + C when running nodemon your-file.js, Visual Code takes several seconds to show prompt again.
Not sure if this is related to mongodb open connection while running the server. Packages used:
{
...
  "engines": {
    "node": "8.2.1"
  },
  "dependencies": {
    "body-parser": "^1.17.2",
    "express": "^4.15.3",
    "lodash": "^4.17.4",
    "mongodb": "^2.2.30",
    "mongoose": "^4.11.5"
  },
  "devDependencies": {
    "expect": "^1.20.2",
    "mocha": "^3.5.0",
    "nodemon": "^1.11.0",
    "supertest": "^3.0.0"
  }
}

VSCode Version: 1.14.2
OS Version: Microsoft Windows [Version 10.0.15063]

Steps to Reproduce:

Run nodemon your-file.js
node app will be running (on a port such as 3000)
click on terminal and press CTRL+C
Visual Code takes too long to cancel nodemon execution
There is no UI info showing if terminal (bash.exe) is processing this cancel request
The prompt will be shown after a big delay youruser@YourPC MINGW64 ~/YourFolder/your-node-rest-api (master)

Reproduces without extensions: Yes/No

Tracking these issues in https://github.com/Tyriar/node-pty/issues/7


When importing files to Coordinate Conversion, persist fields from input

Background
This is useful because all fields used from the input are lost once they are imported to CC. Not a big deal for fields that contain simple coordinate formats, as those can be added back in upon export by the CC pane, however those fields that are lost may contain non-coordinate attribute information, or highly customized coordinate notation.
Current behavior
Coordinate conversion imports from select file types. Only the coordinate fields are used by the CC pane. All other fields whether they are other coordinates or attribute data are lost.
Expected behavior
Coordinate Conversion holds all fields associated with a row/entry in memory while that point is in the Coordinate Conversion list.
[design doc tbd]

ArcGIS Pro 2.2-2.3
Arc Map 10.3.1 -10.6.1
CC import fields design.pptx

@saip  has asked when importing values from fields, what should be done about field type. because my understanding is that there is not a reliable one to one connection between excel data types and arcgis data types,  i have given the answer that converting everything to strings will be ok for our first release/mvp

Can we mimic the behavior of (or reuse code from) the "Table to Table" GP tool?
Seems as though Excel tables usually have a header row with text, so we could also look at the values further down the table to infer data type.
The Incident Analysis sample data is in XLS format, and if you use Table to Table to import it to a GDB table (with default settings, no field mapping), it matches numeric fields to Double, Text to Text, and Dates to Date.
IncidentsCopy.zip
MyProject85.gdb.zip

Tested this in ArcMap and ArcGIS Pro. Good to merge PR. Should be added as What's New

@saip i just noticed that importing a list with fields works properly. If you manually add to the list using the map point tool (so one point would have no details) and then export the list, none of the imported fields persist to the exported file

Doc changes visible in What's new, CC, and MT Release page on STG.


2016.2.1 on red hat has weird formatting (maybe other versions too)

prime example, iostat parsing is missing timestamps...

96370db fixes this for iostat, i didn't really find another example for now. closing


I tried a few:
4.8.0-dev,
4.7.0-beta
On Fri, May 13, 2022 at 4:39 PM Matt Bierner @.***>
wrote:

What TypeScript version is at that path?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/149414#issuecomment-1126579975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRLM5O74DOKPX3J7YURNYTVJ3RZXANCNFSM5VZ6U55A
.
You are receiving this because you authored the thread.Message ID:
@.***>

What are you trying to refactor? Please share steps and code to reproduce the issue.

function foo(a,b){|
if (2<3){
 console.log(a+b);
}
return a+b;
}

the cursor is where the '|' is in the code above. then I select the
contents of the function and trigger refactor.
I see no refactor options in the context menu.
On Mon, May 16, 2022 at 3:41 PM Matt Bierner @.***>
wrote:

What are you trying to refactor? Please share steps and code to reproduce
the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/149414#issuecomment-1128207142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRLM5I5MWDGRWPC3G64FCLVKLFIPANCNFSM5VZ6U55A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do other language features work?

Yes,
Intellisense, syntax highlighting etc work.
[Sent from mobile.   i-typos; I-Apologize.]

On May 16, 2022, at 3:51 PM, Matt Bierner @.***> wrote:

Do other language features work?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Full IntelliSense or just basic word+snippet suggestions? If the refactor doesn't show up at all, it suggests the built-in JS/TS extension is disabled

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Update json.py to accomodate JSONIFY_PRETTYPRINT_REGULAR

Without this change, the pretty output of JSON only works when in debug mode.  With this correction, it now matches the logic that  is found in the flask package, in file flask/json/init.py allowing a developer to dynamically set this switch while in production to do pretty output of the JSON.

@joshfriend any work on this PR? I would like to take this forward

Hi Josh,
It’s a simple fix, but I was having problems getting the test scripts to run successfully for older versions as I recall.  In the meantime, I got pulled into something else so I’ve not had time to work on this.  If you are able to take it forward that would be great for everyone and much appreiated.
Best regards,
Art.
From: onlinejudge95<EMAIL_ADDRESS>Sent: Saturday, February 15, 2020 11:32 AM
To: flask-restful/flask-restful<EMAIL_ADDRESS>Cc: Arthur Lampitt<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [flask-restful/flask-restful] Update http://json.py to accomodate JSONIFY_PRETTYPRINT_REGULAR (#848)

NOTICE: ***This email came from a source outside of Diplomat and is not a trusted sender. Please be cautious opening any attachments or clicking on links. ***

WARNING: HR never sends email with the "Outside of Diplomat" banner
@joshfriendhttps://github.com/joshfriend any work on this PR? I would like to take this forward
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/flask-restful/flask-restful/pull/848?email_source=notifications&email_token=AKB3UUT3GZVTBDJJGWDFYL3RDARIDA5CNFSM4JQVFNH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL3SQGQ#issuecomment-586623002, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKB3UUV4W6NGTJW6PZ42YCTRDARIDANCNFSM4JQVFNHQ.

Pretty printing json is pretty wasteful of bandwidth in a production environment which is not something I really want to encourage. Furthermore, tools like HTTPie and CharlesProxy will prettyprint json responses clientside.
If you really want to do this in your app, you can do it quite easily by overwriting the json representation for your flask-restful api: https://flask-restful.readthedocs.io/en/latest/extending.html#response-formats


Adjust paddings to make history list edge to edge

Adjusts some paddings to make the history list more "edge to edge" removing some whitespace cutting of history items half way.
Before:

After:

Makes sense 👍🏼


Improve logging in dispatcher

Dispatcher logging are really verbose and not useful!
We should improve logging

Let's improve it


Add GitHub Task-Support

GitHub flavored Markdown supports the definition of tasks:
- [ ] not yet done
- [X] done

which is rendered as follows:

[ ] not yet done
[X] done

It would be convenient, if LiaScript could support the same feature, which could be helpful to users to mark their progress, some properties, or something else. Furthermore, this should be combined with the LiaScript feature of attached scripts:
- [ ] not yet done
- [X] done
<script>
let task = @input // substituted by [false, true]
if (task[0] && task[1])
    alert("super")
else if (task[0] || task [1])
    alert("it is okay")
else
    alert("that is absolutely not sufficient")
</script>

Such attached scripts are not mandatory, but they allow to trigger a certain action, if the input value changes, which might be or might not be useful ;-)

Just tested it and it seems to work. Issue can be closed I guess.
#35 didn't close the issue automatically because the keyword (closing) was wrong. See here for reference: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword

Tranks for hint .... I am closing :smile:


Getting rid of event-stream as devDependency

Can we get rid the dependency for event-stream? There is a recent malicious attack on this dominictarr/event-stream#116. We can lock the version to<EMAIL_ADDRESS>

Published as 1.0.2, with a locked event-stream and updated Mocha, and no audit errors.
Thanks for the report!


tour: Exported names, bug  cannot refer to unexported name math.pi

Context: https://tour.golang.org/basics/3
Click on button RUN response
prog.go:9: cannot refer to unexported name math.pi
prog.go:9: undefined: math.pi

Replace on fmt.Println(math.Pi)


Bump DSN parser for space parsing bug fix

https://github.com/ManageIQ/pg-dsn_parser/pull/16

Backported to oparin in commit 7a5f332dcdf62174de5f4ab624d9a98480bbc64c.
commit 7a5f332dcdf62174de5f4ab624d9a98480bbc64c
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Mon Aug 29 17:31:36 2022 -0400

    Merge pull request #22083 from jrafanie/bump_dsn_parser_for_space_parsing_bugfix

    Bump DSN parser for space parsing bug fix

    (cherry picked from commit e9db4fbfe76ae6a99d2c04e468a6eac151bb1ea0)


Backend id not extracted resulting in error on publish

created a new api using azure function in the backend. it has a function key
when running the publisher i get an error like this
System.InvalidOperationException: REST API call failed. Status code is 400, response content is '{"error":{"code":"ValidationError","message":"One or more fields contain incorrect values:","details":[{"code":"ValidationError","target":"set-backend-service","message":"Error in element 'set-backend-service' on line 4, column 4: Backend with id 'ahmsdemofunc' could not be found."}]}}'.
Need more docs/clarity on what needs to be done so I don't get such an error?

More info. See https://github.com/MicrosoftDocs/azure-docs/issues/48421 and
https://stackoverflow.com/questions/58879254/azure-api-management-and-app-function-backend-backend-with-id-foo-could-not-b
in short the policy for an APIM has this
[2:21 PM] Ahmed Salijee

****

and this backend id needs to extracted and/or published.. Not clear how do this .

Currently, the backend object isn't being extracted by the extractor. We would need to extract the backend object. The reference to the backend in the API and/or the policy is a reference and should work. We will triage this for a fix. If you create the backend object it works right?

This has been released as part of Release v1.0.0


feat: ast-based helpers

feat: add ast helpers + build script
build: add python tests

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

This is mostly a proof of concept. The idea being that the python test runner can import the astHelper string, evaluate it and then import the new helper functions.
https://github.com/freeCodeCamp/freeCodeCamp/pull/53125 contains a simple example of how they can be used.

Also, I assume in another PR we would want to add python/*.py to .gitignore?

Why? It's source code.

What is the point of adding the mock-script for Jest? Why not just ignore everything in python/ when running the tests?

How, though? Jest tries to import python/py_helpers.py and throws an error because it doesn't know how to deal with python code. Is there a way to make Jest not do that?

Why? It's source code.

Is it not just used in the build, and then imported as a JS string?


It is possible to create in migration tinyint?

geolocation_status tinyint(4) DEFAULT '0'
is_update tinyint(1) DEFAULT '0'

Yes, pass :tinyint as the type. Or :tinyint, size: 4.


Reduce max pin on run export

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This reduces the run_exports max_pinning so the upstream packages does not need to be rebuilt each time this package has a patch version change.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[FEATURE] document loader web Cheerio

Hello good evening, thank you for this project, it's great
I would like to propose the inclusion of the following feature
document loader Cheerio
In our company it is necessary to read the content of a web page and be able to ask questions
https://js.langchain.com/docs/modules/data_connection/document_loaders/integrations/web_loaders/web_cheerio

is not necessary currently


Fix race on MockZipkinCollector HttpListener

Trying a minimal fix for issues that I found running on Windows and I suspect it also to be related to Linux Azure Pipeline not finishing. This is basically a quick workaround on the limited HttpListener type, that said there is no good and simple alternative to it to use in tests.

Would you be able to describe the race condition?  It's not immediately clear to me what your changes are fixing.

Would you be able to describe the race condition? It's not immediately clear to me what your changes are fixing.

Added a few comments, let me know if that is not clear.


cmake: use add_definitions instead of add_compile_definitions

add_compile_definitions() requires CMake 3.12, but Ubuntu-18.04 ships CMake 3.10. This fixes the build for old CMake versions.
Noticed this on one of the packaging bots that failed to build WPEBackend-fdo after updating it

@clopez Do we want this merged to the wpebackend-fdo-1.8 release branch?

@clopez Do we want this merged to the wpebackend-fdo-1.8 release branch?

Yes, please merge it if is possible.
Thanks!

Done, merged in the wpebackend-fdo-1.8 branch as commit 88bb93c05fd7283e6d1695fa958b09d51a588dfe


NativeScript variables not saved with ResourceSaver-saved scene

Godot version:
3.0.4.stable
OS/device including version:
Windows 10 64-bits
Issue description:
When I try to save a runtime-created scene with ResourceSaver, nodes having a NativeScript attached with export variables are not correctly saved. In the saved scene, the node properties are written in the following order:

[script variables]
script = ExtResource( 1 )

If I do the same using GDScript, the order is as follows:

script = ExtResource( 1 )
[script variables]

When the scene is reloaded, in the case of NativeScript, it will find first the script variables, but since it doesn't know yet there is a script attached, it will ignore them. So these variables are effectively not saved, while in the case of GDScript the scene can be saved and reloaded correctly, with the export variables keeping their values.
The problem I can find is the order of the node properties in the SceneState. Scripts should be first always, for both GDScript and NativeScript. Note that when loading the ResourceSaver-saved scene in the editor, after modifying the NativeScript export variables and saving, the file is correctly written. So this happens only when saving with ResourceSaver and not from the editor.
Steps to reproduce:
Make a NativeScript including and export variable
Load the script into a node, like: var gdnative_node = load("path/to/script").new()
Change the value of the export variable
Pack the node into a PackedScene
Save the PackedScene with ResourceSaver
Minimal reproduction project:
I add a project which shows how GDNative can save the value of export variables and NativeScript fails.
ResourceSaver.zip

I had the problem too, but after applying the pr of https://github.com/godotengine/godot/pull/19748 on custom build works ok

Nice, thanks


SFU Port

2 example outputs are included. myexperience is not imported in main.ts but was tested with it imported. Squashing commits advised.
There is also currently an (unrelated) error of the output folder needing to exist or else saving to file fails with ENOENT.
output.zip

LGTM

I don't know how to reference it, but should the return value of src/extractor/sfu-myexperience/index.ts on line 216 be changed and if so, what should it be changed to?
  private async extractPostingDetails(     page: Page   ): Promise<Omit<Posting, "statsRatings"> | PostingError> {

I don't know how to reference it, but should the return value of src/extractor/sfu-myexperience/index.ts on line 216 be changed and if so, what should it be changed to?
  private async extractPostingDetails(
    page: Page
  ): Promise<Omit<Posting, "statsRatings"> | PostingError> {

Since statsRatings doesn't exist in Posting, the return type should just be Promise<Posting | PostingError>

I didn't want to inline because the functions are already quite long. You can do so if you'd like. Everything else should be resolved.


[1.21][Neoforge] Crafteting terminal still not reconising inventoryu for available crafts recipes

neoforge: 21.0.95
toms_storage-1.21-2.0.6
emi-1.1.10+1.21+neoforge
The available crafting window in EMI is only showing items that are in my inventory, and also, the items are not even craftable as I do not have half of the required items in those recipes.

As you can see, the items in red show that I lack that item, I don;t even have in inside the terminal

Fixed in the latest release
I added a smart toggle under the search setting, which toggles if the terminal contents are included in EMI's Craftables item list (can cause frame lag with 10k+ items)


Blocks: Add functionality to hide posts

Allows hiding the posts of people the user has blocked
Allows hiding the user's posts from people they have blocked
Allows temporarily disabling the first, but not the second with a url param (manually added only, currently)

TODO: modify flash warning on blocks index

Currently blocked on scope issues


Conflict with another triangle.py (LinearNDInterpolator)

Hi-  I noticed that triangle-plot conflicts with another triangle.py which seems to be necessary for the LinearNDInterpolator routine (scipy.interpolate.LinearNDInterpolator())
http://dzhelil.info/triangle/
I'm not exactly sure what to do in this case... I need the LinearNDInterpolator to perform the calculations, but the corner plots would be really nice as well...  Do you know of an easy way to alias the package name?  (Sorry, my ignorance of the guts of python is showing!)

I'm having this problem too. I renamed triangle.py to triangle_plot.py and adjusted imports


TipTap removes  elements

Filament Version
3.2.57
Plugin Version
3.2.25
PHP Version
8.3
Problem description
Tip does not allow <span> tags as it removes them when i add it in "Source code".
It also removes  <div> and <iframe> tags.
Expected behavior
Allow  tags
Steps to reproduce
All  tags in "Source code" editor
Reproduction repository
No response
Relevant log output
No response

This is part of the underlying Tiptap package and how Prosemirror works under the hood.
You would have to make custom Tiptap extensions to handle them.


Some fixes I've found using REST client

It seems that these files are generated, so this PR can be used as reference fixing the generator hopefully.

Thanks very much for this, @benjaminr-ps!  We'll get to it ASAP.

We cannot modify generated files manually. However, recently we've re-generated them and I hope most of the issues have been resolved. However, LogicalOperations class still looks ugly. I'll investigate what I can do.
Thanks,
Alex

The serialization of the LogicalOperations class's (as any other) members is defined by its declaration in the server REST API controllers. IMO, serialization rules are case insensitive and the ugle look of the current state of the LogicalOperations class shouldn't cause any issue for vso-httpclient-java. However, I agree that it would be nice if the server team improves this class definition on the server. I've notified the team responsible for the REST clients' generation about this request. Thank you.
Alex


Ballot for the last period should always be the ballot we manipulate

Currently, we use the branch's current vote period value from the contract to determine which period's ballot to display, but you can never vote on any period except for branch (currentPeriod - 1). votePeriod is really only useful for determining whether to call dispatch.

Folded into #259.


Refator methods to handle multiple input types

Refator methods to handle multiple input types converting them to numpy.ndarray if applicable.
For example correct handling of any Iterable or Sequence sent to methods.

Every public method which accepts np.ndarray should also accept pd.DataFrame and convert it to ndarray