Intermediate representation for sessions

Research
Concepts

Three-Address Code (TAC)
Static single assignment form (SSA)
Continuation-Passing Style (CPS)

Languages

C Intermediate Language

Design
Approach
Specification ⇾ Model ⤵
                     Intermediate Representation ⇾ SPARK Model ⇾ SPARK Code
          Integration ⤴

Objectives

Represent actions of session states
Enable static analysis / verification of session model

Take into account

Contracts of states
Contracts of functions
Type constraints
Buffer sizes defined in integration file

Prove absence of runtime errors (e.g., integer overflows)
Enable mapping of issues in the intermediate representation back to the specification

Enable code generation for various languages (at least SPARK and C)
Prevent unnecessary checks in generated code

Instructions

Assignment (TAC, maybe also SSA?)

Variable
Literal
Mathematical expression (add, sub, mul, div, pow, mod)
Boolean expression (and, or)
Relation (lt, le, eq, ge, gt, ne)
Quantified expression (for all, for some)
Call expression

Variable
Constant
Literal

Assertion

Boolean expression

Convertion from Model to IR

Transform state actions into TAC format
Transform model statements into IR instructions
Infer and add assertions (e.g., preconditions of calls, checks to prevent overflows)

Mapping from Model to IR
Statements

Model Statement
IR

VariableAssignment
Assignment

MessageFieldAssignment
Call?

Reset
Call?

Append
Call?

Extend
Call?

Read
Call?

Write
Call?

Expressions (in VariableAssignment)

Model (in TAC format)
IR

T := X
T:= X

T := not X
T := not X

T := X and Y
T := X and Y

(Other boolean expressions)
(Direct mapping)

(Mathematical expressions)
(Direct mapping)

(Relations)
(Direct mapping)

(Quantified expressions)
(Direct mapping)

T := X'Size
T := Type_Specific_Size_Func (X)

(Other attributes)
(Equivalent mapping to call)

T := M.F
T := Message_Type_Specific_Field_Func (M)

T := F (X, Y)
(Direct mapping)

T := (1, 2, 3)
(Direct mapping)

T := "FOO"
(Direct mapping)

T := [for I in S if C => I.X]
(Direct mapping or call)

T := M'(F1 => X, F2 => Y)
(Direct mapping or call)

(Binding)
(INVALID)

Implementation

[ ] Create class hierarchy for IR

Separate session, statement, and expression classes
Comply with TAC
#861

Enable determining preconditions for mathematical and boolean expressions (to enable detection and prevention of runtime errors)
Enable representation of preconditions and postconditions of function calls (e.g., for reading or setting message fields)

[ ] Create transformation from existing session model to IR
[ ] Adapt SPARK code generator to IR
[ ] Create simple C code generator

Added in version 0.12.0.


Whitelist SSM user in OpenSSH module

The OpenSSH module catches the ssm-user as a false positive:
----------[Diagnostic Results]----------

module run/openssh [FAILURE] Improper configuration of one or more OpenSSH components.
-- SSH may deny access to users when improperly configured.
-- FAILURE Missing authorized key directory: /home/ssm-user/.ssh
-- Create directory: mkdir -p /home/ssm-user/.ssh.

The ssm-user should not be accessing the system via ssh, so there is no reason to perform checks for it.

Resolve with #71


DNSimple v2

Looks like the current glide dependency uses the old DNSimple v1 API written by weppos. The lego repo have been updated to DNSimple official go library.
When I tried master, it seems to have issues with the token (verified working separately)
2017/04/19 00:23:21 Error while processing certificate event: Error while obtaining certificate for new domain [omitted]: Error presenting token: DNSimple API call failed: GET https://api.dnsimple.com/v1/domains: 401 Authentication failed

I think we can update glide, and replace DNSIMPLE_API_KEY with DNSIMPLE_OAUTH_TOKEN, however the requirement for DNSIMPLE_BASE_URL might be more annoying since it will require multiple kube-cert-manager per domain.

Would updating the package actually be useful to DNSSIMPLE users ? It seems to me like xenolf/lego probably needs a way to pass in the baseURL and we need to support that

Yes we should update to v2, because DNSimple is deprecating their v1 API, and it will be no longer available as of October 1st, 2017. We can pass the baseURL via environment variable.

Closed by #57, thank you!

PR #50 clobbered over #57, we are back to DNSimple v1.


https website picture provider : Received fatal alert: handshake_failure with java 11

tested Pictures provider : MagicCardInfo (now scryfall), Scryfall
provider : MTGJson, Scryfall
jvm : java 11
first card selected OK, and after few minutes handshake failure.

need to add  -Djdk.tls.client.protocols="TLSv1" in jvm argument.


⚠ Jerry Savelle Ministries International Australia has degraded performance

In 4f775cc, Jerry Savelle Ministries International Australia (https://jsmiaustralia.org) experienced degraded performance:

HTTP code: 200
Response time: 6276 ms

Resolved: Jerry Savelle Ministries International Australia performance has improved in 5500544 after .


Support secrets stored in Hashicorp Vault

At Under Armour, we run Vault to store our secrets (like many other companies do), do you envision having an integration with a tool like Vault where a secret update in vault triggers a rolling update of deployment?
I’d be happy to contribute to the project if you think Wave can watch secrets in Vault. Let me know what your thoughts on this are and how you think this can be implemented to add value to the Wave project.

I'd be interested in knowing more about your use case.
How do you integrate Vault-based secret with Deployments?
Most solutions I know will use a separate controller to create Kubernetes Secret objects from Vault secrets which in turn then get referenced in Pod specs.
In such a scenario, I would expect that controller to ensure the Kubernetes Secrets stay in sync with Vault and Wave would work out of the box because it reacts to changes in the Kubernetes Secrets.

Closing this out as https://github.com/DaspawnW/vault-crd fit our use-case really well


Added Dynamic Deference & Allocation WG

Summary
Add new Working Group description
Details and comments
Voted by the TSZC 2022-01-28

Voted by TSC 2022-01-28


Hugo should generate an Atom feed and an RSS feed.

Now that we have support for custom output formats, I think it's time to fix the problem of Hugo generating a mixture of RSS and Atom in its feed.
I would suggest generating a pure RSS feed and a pure Atom feed, and making the existing default RSS feed a pure RSS feed going forward. That said, my main concern is having a proper Atom feed.
I've got an Atom feed template I'm using which I'm happy to contribute, but I'm no expert on templates or Hugo internals so it might be better if someone else cleans it up, removes the Atom from the RSS template, and incorporates it into the internal defaults.

Related: #3487

I would like to see RSS 2.0 removed entirely with a minor release and Atom made the default. Rather than stating my reasons for this please simply look at the Atom and RSS comparison on the Atom Wiki.

It's better to choose one not both. I would like to see RSS 2.0 removed and Atom made the default. Here's a comparison of RSS and Atom Atom showing some key differences.
Other reasons to Atom by default:

Multiple enclosures
Support for author with out Dublin Core namespace
It's a standard built to fix the issues of RSS 2.0
It supports more encoding types
Unlike JSON Feed it's an IETF standard

A new "feed" is Output Format + Media Type + Template. I would love it if this could have lived outside of Hugo core.

@bep thoughts on removing default RSS output then and leaving internal templates for each so they're optionally included by theme?

(I personally provide only Atom, and if it was up to me I'd rip out the RSS code from Hugo and support only Atom by default, but I was choosing to avoid controversy in the request.)

Given RSS is intended to support syndication of content it seems the Content Formats shouldn't necessarily be responsible for this behavior - so it seems one should be chosen and something like Atomic RSS avoided.
Another useful feature Atom has which fits well into Hugo's content-agnostic direction are Containers:

Containers (synonym: collection) are objects that hold multiple Entries for any purpose.

I totally agree that if Hugo is to include support for something it should be Atom. RSS is outdated and limited for the reasons the others have stated above.

For those looking to add Atom support here's some info shaping up: https://github.com/comfusion/after-dark/issues/32#issuecomment-312515542. Hugo makes it very easy to disable RSS, which is a blessing and very simple giving the IMHO stellar design of Output Formats. 😍

What we need in that case is something we can add cleanly to existing themes.


text: Make Draw take a GeoM argument

Some sort of GeoM equivalent for text.Draw could be useful, especially for scaling text. The Draw function could take a GeoM argument instead of an x, y coordinate pair.
This might also allow rotating text.

It seems like an option for scaling text should be very easy to add. Rotating text, however, seems more difficult to achieve.
What do you think of adding a DrawScaled function with two extra parameters: scaleX and scaleY.
func Draw(dst *ebiten.Image, text string, face font.Face, x, y int, clr color.Color)
func DrawScaled(dst *ebiten.Image, text string, face font.Face, x, y int, clr color.Color, scaleX, scaleY float64)

The scale would be passed to drawGlyph, which would then scale the glyphs after translating them.
An alternative approach would be to introduce an Options struct, and a new function called DrawOptions:
type Options struct {
    X, Y int
    Face font.Face
    Color color.Color
    ScaleX, ScaleY float64
}

func DrawOptions(dst *ebiten.Image, text string, options Options)

The advantage of this approach is that we can add new options (like Rotation) later on.

Sorry I forgot the context at the first comment.
Wouldn't it be enough to render the text on an offscreen once, and scale it, or have we already discussed about this idea?

Wouldn't it be enough to render the text on an offscreen once, and scale it, or have we already discussed about this idea?

That seems like it would be very expensive for a simple scaling of the text.
Here's my use case: I want to allow the user to configure the screen resolution. This can be achieved by scaling images to the resolution of the screen with ebiten.GeoM.Scale (this was discussed in #1335). I also need to be able to scale text for this to work properly.

Hmm, have you actually measured the performance? Of course it is less performative than having a scaling argument though.
If we adopt a new API, I prefer having an 'option' argument as you already suggested. I'd suggest this:
DrawWithOptions(dst *ebiten.Image, face font.Face, text string, op *ebiten.DrawImageOptions)

A font face is not an option and should be a regular argument.
Not only scaling, there are other useful options like Filter.

If we adopt a new API, I prefer having an 'option' argument as you already suggested. I'd suggest this:
DrawWithOptions(dst *ebiten.Image, face font.Face, text string, op *ebiten.DrawImageOptions)

This API would actually be a lot nicer! I would prefer it over the DrawScaled function mentioned earlier. I just wasn't sure exactly how it would work.


send event to save state after 'Add Project Folder' or 'Remove Projec…

Fix bug https://github.com/atom/atom/issues/12495
When we add or remove project folder,  the window's current project path will be changed, then "initialPaths" will be reload, it is the best time to save state
/cc @maxbrunsfeld @lee-dohm

This looks great to me; I left one minor comment. I would like to add a test for this behavior. @yuanwhy are you interested in adding a test for this? I think that it should be added to this test file.

@maxbrunsfeld I renamed the event name. I will appreciate it if you could add a test for this behavior, because I am not good at the test code


No suggestions when changing the topic for a message

When you edit the topic in a new message, you get a nice list of suggestions:

However, when you move a message to a new topic, you don't:

It'd be nice to give suggestions for the latter, too.

@zulipbot claim

This is a duplicate, and I think @aero31aero already has a PR for the original.
See #14033.

@timabbott: thanks, I thought this was probably reported already, but didn't find that one!


DomainJoin?

I am starting to evaluate the soution and asked myself when setting up the Connector Service on a OnPrem Windows Server, if that should be DomainJoind or not. If there are maybe dependencies like Kerberos SSO etc. maybe that could be described / mentioned in the article

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 2bfe7e36-297f-9496-8ace-8a750831c912
Version Independent ID: afe86f38-e147-1061-3196-ed3432af0596
Content: How to configure connectors for Microsoft Entra Private Access
Content Source: articles/global-secure-access/how-to-configure-connectors.md
Service: network-access
GitHub Login: @shlipsey3
Microsoft Alias: sarahlipsey

@tobiasabele
Thanks for your feedback! We will investigate and update as appropriate.

Working with the product group to improve the documentation around connectors and domain-joined scenarios.
#please-close


Add support for automix

see https://github.com/bitfocus/companion-module-soundcraft-ui/issues/72
channel

enable/disable automix, automix state (i.0.amixgroup^0), values 0, 1 (groups) and -1 (disabled)
automix weight (i.0.amix^0.5193637621023514), linear and dB

global

automix groups enable/disable, group state (automix.a.on^0), a/b, values 0/1
global response time (automix.time^0.1821943259976487)

I started with this and encountered a blocking issue: Conversion from the internal linear values to the "global response time" in milliseconds is not linear. Probably it's logarithmic, but I did not investigate this further.
We need to understand this conversion and need to implement two functions to convert these value forth and back.
Here are a few value samples (first: milliseconds value, second: linear value):
[4000, 1],
[3719, 0.976303317535545],
[3400, 0.9478672985781991],
[2568, 0.8641390205371249],
[2090, 0.8072669826224329],
[1741, 0.7598736176935229],
[1183, 0.6682464454976303],
[831, 0.5939968404423381],
[470, 0.4897314375987363],
[265, 0.40126382306477104],
[143, 0.3206951026856241],
[67, 0.23380726698262255],
[51, 0.2037914691943129],
[38, 0.17061611374407593],
[27, 0.12796208530805697],
[21, 0.08056872037914703,],
[20, 0.06477093206951037],
[20, 0]

@koalaraptor Can you try to find out how this conversion works?

What did NOT work is powers of 10:
const upper = 4000;
const lower = 20;

const x = Math.pow(10, value * 3);
return Math.round(((x / 1000) * (upper - lower) + lower) * 10) / 10;

How about chatGPT?
Here is it's answer to your examples:
The conversion from linear values to the "global response time" in milliseconds seems to be a non-linear transformation. The provided samples suggest that it may be logarithmic or some other mathematical function. In JavaScript, you can try to find out how this conversion works by using different mathematical functions and fitting them to the given data to see which one provides the best approximation.
One common non-linear function to consider for this kind of transformation is the logarithmic function. Here's a JavaScript code snippet that attempts to approximate the conversion using a logarithmic function and provides functions for both conversion directions (forward and backward):
// Forward conversion: Linear to Milliseconds
function linearToMilliseconds(linearValue) {
  if (linearValue === 0) return 0;
  // Assuming logarithmic conversion
  return Math.exp(linearValue) * 1000;
}

// Backward conversion: Milliseconds to Linear
function millisecondsToLinear(millisecondsValue) {
  if (millisecondsValue === 0) return 0;
  // Assuming logarithmic conversion
  return Math.log(millisecondsValue / 1000);
}

// Test with sample values
const samples = [
  [4000, 1],
  [3719, 0.976303317535545],
  [3400, 0.9478672985781991],
  // Add more samples as needed
];

samples.forEach((sample) => {
  const milliseconds = sample[0];
  const linear = sample[1];
  const calculatedLinear = millisecondsToLinear(milliseconds);
  const calculatedMilliseconds = linearToMilliseconds(linear);
  console.log(`Milliseconds: ${milliseconds}, Linear: ${linear}`);
  console.log(`Calculated Linear: ${calculatedLinear}, Calculated Milliseconds: ${calculatedMilliseconds}`);
  console.log('---');
});

This code uses a logarithmic transformation for the conversion and should provide approximate results. You can add more sample data to improve the accuracy of the approximation if needed.
Keep in mind that this is just one possible approach, and the actual conversion function might be more complex. If this approximation does not match the actual behavior closely enough, you may need to explore other mathematical functions or methods for fitting the data.
What do you think? might this work?

I know that it has something to do with exponential/logarithmic functions, but I did not find out the details.
This ChatGPT answer is practically useless since it only tells us that Math.exp() and Math.log() exist ;-)

Hello @fmalcher
I am afraid i don't have much knowledge to support this kind of issue, but i do know someone who used to talk directly with the soundcraft team, i will try and reach out to him if he can help out.
Out of curiosity, is it important for the implementation to work for now?
While it would be nice to have the control, it's not critical in most use cases i have encountered ✌

I know that it has something to do with exponential/logarithmic functions, but I did not find out the details. This ChatGPT answer is practically useless since it only tells us that Math.exp() and Math.log() exist ;-) So if ChatGPT has some more ideas, let me know! 🙂

Sorry, just tried to see if it was of any help...

Out of curiosity, is it important for the implementation to work for now?

You're right – it's not important for the rest of the features. However, if we add support for automix, this should be included to be feature-complete.

I figured it out. It was a weird constant that I found by approximating the value step-by-step.
Then I rearranged the equation to get the back conversion.


Deterministic findTag

https://github.com/rwjblue/git-repo-info/blob/c71c155bddb02e6540c26e4a276d600d199b83fe/index.js#L141-L142
When multiple tags are referencing the same commit the returned tag is dependent on whether it is packed or loose. Currently, the sort order is packed then loose. Each should be then alpha sorted (though readdir does not guarentee this).
I am thinking a manual sort of both loose and packed tags should do the trick.

Agreed.


Broken link in README.md

The Getting Started link in the README.md points to https://docs.cast.ai/getting-started/overview/, which shows a 404 Not Found page.

Thanks, fixed.


incompatible with "sort and permute" plugin, cant sort lines.

if cant work with other plugins,
this is an encrypted notepad.exe alike...
thanks
ps:
android + win10 64bit.
in android, the sorting no effect after applied the command "sort and permute alphabetically"
in windows, all lines become EMPTY!!!! so take care when testing.
thanks

I think its better to raise an issue with the sort and permute plugin.  I don't have control on what it does.


Update README.md

Update README.md EXAMPLES
grpc_health_probe -addr=localhost:5000

CMD error, should be
grpc-health-probe -addr=localhost:5000

According to what? The release binaries are also underscored.


Load also abstract types

It's probably rare but some WSDL files use abstract types in operation definitions. Without this change it's impossible to build messages for such operations.
I'm also bumping bundler and rake dependencies here.

some WSDL files use abstract types in operation definitions.

Interesting. It sounds like this is breaking the spec to me (whatever that really means in the SOAP world). If a type is abstract it means that other types can derive from it, but it should never be used directly.
I think for this reason we should avoid changing the existing types method and perhaps consider introducing an abstract_types method. This means that a WSDL that does not follow the specification can still be parsed, we just have to add a little more work as consumers of the Lolsoap library. WDYT?

consider introducing an abstract_types method

That is one of options that I was considering. If using and abstract type is against specs then I agree with you and I'll add that method. You're right it's safer too. Thanks!

I pushed a commit that loads abstract types into a separate hash. Unfortunately types for description of operations are handed over from WSDL class directly so that change itself wasn't sufficient and I had to add a flag :allow_abstract_types. I didn't notice it at first glance. It keeps the default behavior unchanged so risks should be limited.
Alternatively I could add a standalone class with this behavior - I have no strong preference about this.
WDYT? I'll merge it with the previous commit once it's acked.

Thanks @tstrachota

Alternatively I could add a standalone class with this behavior - I have no strong preference about this.

Do you mean a new standalone class like AbstractType instead of Type? Which would remove the allow_abstract_types  behaviour? If so, I think that's the way to do. AFAICT the parser type method is only used in Type#base_type so having a new flag which dictates whether abstract types should be allowed or not is odd, especially in the context of calling WSDL#abstract_types which would return the types regardless of the value of this flag. WDYT?

I meant rather having a standalone descendant of WSDL that would implement the behavior that comes with :allow_abstract_types => true. I might be overlooking something but I think adding a class for AbstractType wouldn't help us here.
The problem is Lolsoap::Builder checks for types and their allowed elements internally. I added WSDL#abstract_types just to complement WSDL#types but in fact only adding this method can't help with fixing the issue.
The crucial part is to allow NamedTypeReference to use abstract types here:
https://github.com/loco2/lolsoap/blob/master/lib/lolsoap/wsdl/named_type_reference.rb#L12
The current behaviour (without this patch) is that WSDL#type returns NullType for abstract types there and the builder raises an exception when it attempts to add an inner element.
I added the flag to make sure we stay in line with specs by default and only allow abstract types as opt in.
Does it make sense to you?

Thanks. I squashed the commits, merging now.


Pokemon picross froze

Hi, I tried to dump my save of Pokemon Picross with svdt several times but svdt just froze!

Games released after a certain date (I don't know which) can't be dumped.
I think it is because a different encryption is used, but I don't really know.
Supposedly this might be fixed when smealum updates ninjhax.
In any case, this is not related to svdt, but rather it is a current limitation of ninjhax 2.5.

im trying to do this with picross

Still no solution for userland, ninjhax 2.6 doesn't adress this issue. This is not a problem related to svdt.

so pokemon vc(yellow/red/blue)'s issue is because of same reason?

Yes. Smea is working on 9.6+ support for ninhax 2.7, so expect the games to work when that gets released

Thank you. I just tried a latest fork from njnjhax 2.7 (http://ihaveamac.github.io/ninjhax2.x/) and that problem has been fixed (svdt works, but hans doesn't. formal 2.7 should have fixed that.).
Pokemon Picross works and Pokemon VC yellow works as well. I tested both of them.
Thank you all developers involved.

Oh, that's great news!

Hi, I am on FW 11, no custom firmware. I am trying to use SVDT on freakyhax, and it still froze. Can anyone offer any advice please?
Thank you

https://github.com/ihaveamac/9.6-dbgen-xmls/blob/master/README.md


cf ssh not creating app from manifest

I am deploying this app on cloud foundry, and wanted to view the production console.
I ran this command: cf create-app-manifest dolores
using this version of the CLI: 6.12.3-c0c9a03
and this error occurred:
runtime error: invalid memory address or nil pointer dereference
and this stack trace:
goroutine 1 [running]:
	main.generateBacktrace(0x0, 0x0)
		/Users/cffadmin/go-agent/pipelines/Mac-OSX-Unit-Tests/src/github.com/cloudfoundry/cli/main/main.go:117 +0xc7
	main.handlePanics(0xe1b270, 0xc208116700)
		/Users/cffadmin/go-agent/pipelines/Mac-OSX-Unit-Tests/src/github.com/cloudfoundry/cli/main/main.go:101 +0x1a4
	github.com/cloudfoundry/cli/cf/commands.(*CreateAppManifest).createManifest(0xc208214c60, 0xc208817b90, 0x24, 0xc208829d16, 0x7, 0xc208501f80, 0x32, 0x0, 0x0, 0x0, ...)
		/Users/cffadmin/go-agent/pipelines/Mac-OSX-Unit-Tests/src/github.com/cloudfoundry/cli/tmp/cli_gopath/src/github.com/cloudfoundry/cli/cf/commands/create_app_manifest.go:91 +0x85
	github.com/cloudfoundry/cli/cf/commands.(*CreateAppManifest).Execute(0xc208214c60, 0xe1cbf0, 0xc20821cd80)
		/Users/cffadmin/go-agent/pipelines/Mac-OSX-Unit-Tests/src/github.com/cloudfoundry/cli/tmp/cli_gopath/src/github.com/cloudfoundry/cli/cf/commands/create_app_manifest.go:87 +0x4d2
	main.main()
		/Users/cffadmin/go-agent/pipelines/Mac-OSX-Unit-Tests/src/g

2016/01/11 12:29:49 USAGE: cf-ssh -f manifest.yml
Any clue what is going on?

Hi @gemfarmer
That's a known issue with that release. Please update your CLI.
Regards,
Dies Koper
CF CLI PM


Experience company link added

Hi @arifszn
Description:
Experience section's company name clickable added to redirect to company page is it has in new tab
Please add hacktoberfest-accepted label
Issue: #195
Checklist:

readme updated
Link made optional
updated props
updated types/index.d.ts

Thanks for the PR. Will review it soon.

@jsvigneshkanna please fix the linting error.
https://github.com/arifszn/gitprofile/actions/runs/3227038117/jobs/5330702374

@arifszn ,
I have fixed the prettier linting errors. please check now
BTW, i would suggest you to add
'prettier/prettier': ['error', { endOfLine: 'auto' }], under rules section in .eslintrc.js as we are seeing many false errors like Delete cr

Thanks. Noted.

@arifszn Can you explain whats the issue in github action

npm run prettier:fix should solve it.

@arifszn ,
Should I need to run that cmd?


List.saveOutline / loadOutline

It would be nice if you could save/load a List as an OutlineSavedState + literal values array. This is convenient for searching data at rest, and in the benchmarks, it looks like gzipping those two separately will be smaller than gzipping the ListSavedState.
I'm imagining the following methods on List:
saveOutline(): OutlineSavedState;

loadOutline(savedState: OutlineSavedState, values: T[]): void;

where you can save the values using list.slice().

Once we have this, it's possible to convert between (saved states for) List, LexList, and Outline. This should be documented in a "Compatibility" section of the readme.

I’m probably missing something obvious in the docs, but why does load take the return of the save function AND  a list of values? What would those values be?

@scottmessinger The idea behind these methods is to let you save the CRDT metadata (encoded by the OutlineSavedState) separately from the actual list values, so that you can store the values in your own format. E.g., store text data as a normal text file, so that it is easy to search. But then when you reconstruct the List<T> later, it needs both the CRDT metadata and the values.


Troubles using simple Input/InputSkin components with newest webpack 4 loader configuration

Hello to maintainers of react-polymorph code.
First, thanks to share this react framework to the community.
I face a (probably obvious) problem using the simplest components (as described in the official readme).

to be able to compile (react-polymorph) successfully in my project i had previously to do the following

yarn add node-sass react-css-themr lodash classnames -D

adapt my webpack 4 config with the following loader sections (maybe my problem comes from there?)

(note the postfix "-loader")
{
        test: /\.scss$/,
        use: [
          'style-loader?sourceMap',
          'css-loader?sourceMap&modules&localIdentName=[name]_[local]&importLoaders=1',
          'sass-loader?sourceMap',
        ],
      }

my test project is a simple react-router starter basic example

where i:

separated all components in different files
added the following code to the About.jsx component

import React from 'react';
// import EB from './ErrorBoundary';
import Input from 'react-polymorph/lib/components/Input';
import InputSkin from 'react-polymorph/lib/skins/simple/InputSkin';

export default () => (

About

    <Input skin={InputSkin} />

);

But I end up with the following errors (i have attached a screenshot) when i visit the About section:
warning.js:33 Warning: Failed prop type: Invalid prop `skin` of type `function` supplied to `Input`, expected a single ReactElement.

and
Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

Actually I already saw that error and went to google but could not manage to debug this time(((.
Any suggestions or help would be welcome.
Thanks for reading me
(i attach my webpack config, package.json and screenshot errors

package.txt
webpack.config.txt
)

@chisNaN
Thanks for your interest in react-polymorph! If you are still interested in using the library for your project I would recommend using the develop branch. The library has undergone many changes over the past few months and the next will release will introduce a new API in the master branch. I recommend taking a look at the stories available for each component. After cloning the repo and checking out the develop branch, run npm install and then npm run storybook. Take a look at the stories directory for code examples.

Thanks sir for the reactivity and indications

@chisNaN as the error message indicates you need to install Xcode (the whole app, not just the command line tools) 😉

@chisNaN
After you install Xcode: to run the component stories via storybook, please either pull the latest changes from the master branch and reinstall dependencies using yarn, or if you're using react-polymorph as a dependency in another project, please use version 0.7.0 as this is the latest release that's been made available since we last spoke.

THanks to you both will try that ASAP
Sorry I miss the part saying full xcode to be installed.


Tests are hanging on Node.js >= v18

Running npm test runs the tests, but the process never exists. It just hangs.

Fixed by #1606


how to add my repo to an organizations?

such as
http://github.com/gmarik/Vundle.vim
https://github.com/VundleVim/Vundle.vim
when i use the above url will go to the next one

i have find how to do it


Saga being called multiple times

version: 0.11.0
i have exactly same issue explained here #471 but his solution didn't work on my case.
My code
function* submit(action) {
  console.log('saga submit');
}

export function* watchSubmit() {
  yield* takeEvery(SUBMIT_FORM, submit);
}

My scenario:

First time full refresh browser - click the submit button which dispatches action once & saga once.
Change the location using react router then go back to previous route click the submit button again - which dispatches action once & saga twice. So every time do this saga will executed increasingly.

So console log look like:

Any help or suggestion most appreciated
Thanks

Crucial here is how and when do you run watchSubmit? Most likely you run it on every route change, so with each change you fire up thistakeEvery.

@Andarist Thanks. That's exactly what was happened.
So i made some research found this solution https://github.com/mxstbr/react-boilerplate/pull/390


feat: Improved Module Support for Fortran Intellisense

Is there an existing request for this?

[X] I have searched the existing issues

Feature Request
In issue #175 I requested the ability to control where the Fortran modules are placed and this has since been added, which is great.  However, you might note this comment.
This is still the behaviour today.  In order to get rid of the red squiggles on the use module_name statements you have to go and load the code that contains the module module_name so that the .mod file is generated into the output directory.  It's working but you need to click on them one by one.
It would be much better if the .mod files are generated on the fly when you load a particular source file.
This is an issue on both Windows, when configured with Intel Fortran, and Linux/MacOS when configured with gfortran.

This is essentially a build system request, which would be very useful. A basic attempt to solve this has been added to the pre-release which I would strongly advise you to try. The option is under the linter settings fortran.linter.initialize. A better solution would involve using `

fpm` to get the file order with which files should be linted and then run through the vscode linter
get fpm to generate the .mod files and emit any possible warnings
write/or use an existing dependency graph generator, and query the nodes (corresponding to a source file) to generate linting messages

All three are not that simple to implement, especially since vscode does not have an elegant way of opening and closing files.
The only realistic solution, if working on a very large project is to build the project a-priori and then point the linter to the location of the .mod files.

I think this won't make it in the next release, which has been finished but remained in a pre-release stage for some time now, but I would like to include it in the next year's roadmap.

I tried fortran.linter.initialize but it tries to lint all workspace files in the order it receives them from getFiles, so in a large project all these linter invocation mostly fail because Fortran modules must be compiled in a specific order.
I think it would make more sense to allow fortran.linter.initialize to point to a specific VSCode task. This way the task could precompile the project such that all modules are created in the correct order. This was suggested in https://github.com/fortran-lang/vscode-fortran-support/issues/175#issuecomment-666007292.


SortBy Implementation

Based on the issue #182 i have created a sortBy function based on the current behavior of Lodash and Ramda.
Please feel free to reply with feedback and i will be pending of it in order to make the changes that you think that have to be made.
Documentation and unit tests cover the case uses.
Note: I am also planning to add a sortWith which will accept the sorter and reimplement sortBy using sortWith.

Coverage increased (+0.2%) to 96.345% when pulling 8c00b3aaa789ee524dad2614849f19769cbf2015 on Ivykp:feature/sort-by into 2d870b14a01e222493c686a7644181185f859558 on rtfeldman:master.


METAMODEL-1099 elasticsearch factories

This PR adds some factories for the two ElasticSearch modules.

Merging as per lazy concensus.


Automated backport of #1973: Change default image type

Backport of #1973 on release-0.12.
#1973: Change default image type
For details on the backport process, see the backport requests page.

:warning: We should make sure the main PR goes in before the backport :warning:


Post-processing: Add annotate stage

The annotate stage writes text onto the top of the image and supports
the same % directives as the FrameInfo class (hence also matching the
--info-text option). It is implemented using OpenCV.
Signed-off-by: David Plowman<EMAIL_ADDRESS>

@naushir @RaspiShonak


Cargo fmt strips if let pattern field attributes.

Cargo fmt strips if let patterns annotations/meta (in this case a feature gate).
So this code:
    pub fn endpoint<F>(self, func: F) -> Response<hyper::Body>
    where
         /*stripped*/
    {
        let Endpoint {
            global,
            request,
            type_hint: _,
            #[cfg(feature="test-functionality")]
            mock_workspace,
        } = self;

        /**stripped*/
    }

Becomes
    pub fn endpoint<F>(self, func: F) -> Response<hyper::Body>
    where
        /*stripped*/
    {
        let Endpoint {
            global,
            request,
            type_hint: _,
            mock_workspace,
        } = self;

        /**stripped*/
    }

Which then fails to compile if you do so without the test-functionality feature...

This is a duplicate of #3759. The issue has been fixed on the latest nightly.


Migrate config-time consts to contract

A follow-up will add configurable testing now that we have this.
closes #51
I refactored to make functions in utils.sw that formerly depended on config-time consts from the manifest instead have these values passed as function params, which is much cleaner anyway.

Just curious, why did you move the constants in the helper functions into parameters for the function instead? Do you know how this impacted the code size?


Enable checkstyle && fix issues in entity/agent

Pull request checklist

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Code compiles correctly with all tests are passed.
[ ] I've read the contributing guide and followed the recommended practices.
[ ] Wikis or README have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?
If this introduces a breaking change for Hydra Lab users and requires a migration process, please describe the impact and migration path for existing applications below.

[ ] Yes
[ ] No

Pull Request Description
Enable checkstyle in common module, wave 1
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Build related changes
[ ] Refactoring (no functional changes, no api changes)
[x] Code style update (formatting, renaming) or Documentation content changes
[ ] Other (please describe):

A few words to explain your changes:

Does this close any currently open issues? Issue ID: #
How you tested it
Please make sure the change is tested, you can test it by adding UTs, do local test and share the screenshots, etc.
Feature UI Screenshots or Technical Design Diagrams
If this is a relatively large or complex change, kick off the discussion by drawing the tech design with PlantUML and explaining why you chose the solution you did and what alternatives you considered, etc...

Could you modify the doc contributing guide and provide a list there to list all the enabled rules? Or if there are too many, you may briefly summarise the goal of this change in the doc, and attach some official doc link of checkstyle in the contributing doc to provide clarity.


Can't open a Reminder or the Reminders app programmatically

Tried to open specific reminder with x-apple-reminder://{REMINDER_GUID}
Also tried variants with x-apple-reminder: and variants without the reminder ID
Logs error even though LSApplicationQueriesSchemes has been set:Failed to open URL x-apple-reminder://618A7A9F-4BCD-40D9-BCCC-48BE7E34B902:
Error Domain=NSOSStatusErrorDomain Code=-10814 "(null)"
UserInfo={_LSLine=279, _LSFunction=-[_LSDOpenClient openURL:fileHandle:options:completionHandler:]}

Initial scan suggests this might be undocumented/unsupported?

Apparently, this changed to x-apple-reminderkit in iOS13. It also doesn't allow passing the ID any longer. So, the app's implementation will be changed to just open the Reminders app itself.


build(deps-dev): add tape

Used in tests but not explicitly defined as dependency:
https://github.com/pinojs/thread-stream/blob/7135302d32a7306443c3c6d7b38fa28bc1e698ca/test/indexes.js#L3

I have a feeling that file is not run at all. Could you switch to tap instead?

Is it worth just removing the file? Have ran the tests and you are right, it's not used.

It's actually a useful test, maybe make it run?


Start EIA-176 pipelines: company data

PR Overview

This introduces the basic extraction components for EIA-176, starting with reading company data. We can extend this pattern to process all EIA-176 data, but that is not included in this PR.
This also includes integration with Dagster, where the assets materialized successfully.

PR Checklist

[ ] Merge the most recent version of the branch you are merging into (probably dev).
[ ] All CI checks are passing. Run tests locally to debug failures
[ ] Make sure you've included good docstrings.
[ ] For major data coverage & analysis changes, run data validation tests
[ ] Include unit tests for new functions and classes.
[ ] Defensive data quality/sanity checks in analyses & data processing functions.
[ ] Update the release notes and reference reference the PR and related issues.
[ ] Do your own explanatory review of the PR to help the reviewer understand what's going on and identify issues preemptively.

I have an extractor I can run locally that populates the company table associated with EIA-176 (screenshot). Next I'll make sure there's sufficient test coverage and graduate this out of a draft so it doesn't become too big. I'll aim to do that over the next week. I think the next iterations would be making sure to integrate with Dagster properly, populating the tables for other data, and any refactoring around the repeated patterns between the CSV-related objects and DBF-related objects.

I was able to slim down the CSV extraction classes and tests a bunch. I also materialized the assets successfully in Dagster (screenshot).

I was able to materialize in Dagster and my updated integration test passed for this latest iteration.

The integration tests failures appear to be due to Zenodo flakiness + outdatedly including /draft in the download URLs they return. Should get handled if we merge https://github.com/catalyst-cooperative/pudl-archiver/pull/192 and recreate some data packages on Zenodo with better URLs.
The zenodo-cache-sync failed on git pull? Which seems funky. And the notification failed because we're missing some Slack webhook secret, which I can set up.

@davidmudrauskas  I think if you merge dev one more time, we should be able to try re-running the integration tests. Sorry that took so long 😬
We also took a look at the zenodo-cache-sync errors from a different PR - the root cause is some credential stuff that isn't particularly important to fix now, so we can just ignore those checks.

FYI test_filter_for_freshest_data failed in my last build's unit tests:
Unreliable test timings! On an initial run, this test took 558.62ms, which exceeded the deadline of 400.00ms, but on a subsequent run it took 47.77 ms, which did not. If you expect this sort of variability in your test timings, consider turning deadlines off for this test by setting deadline=None.

You can reproduce this example by temporarily adding @reproduce_failure('6.91.0', b'AXicY2BgYGBhYFjAwMDIzMDAycjAzMB4moGRkQEOgEwglx1IMTGC1DCygIRsgTKrQICR4R0DszgjEwODD7MUyKgjLiD1UQxIAABbSwgI') as a decorator on your test case
== 1 failed, 284 passed, 1 skipped, 9 xfailed, 8 warnings in 71.09s (0:01:11) ==

It did fine locally, so hopefully it's more consistent and the above was an anomaly. Running a new build with latest dev.

I think you're good to merge! Might need to pull our latest dev into your branch. I see a button for me to do it but not sure if that will mess with your flow:

Hey @davidmudrauskas I'm sorry I didn't realize that when I got rid of the dev branch it would close all the PRs from forks of the repo. Can you re-open this against the main branch?

Hey @davidmudrauskas I'm sorry I didn't realize that when I got rid of the dev branch it would close all the PRs from forks of the repo. Can you re-open this against the main branch?


Unable to Activate Google Lookup

Hi, it seems like latest version of Obsidian v1.6.3 break this plug in. I am not able to find the plug in in command palette. However, it is found in Hotkeys settings. I assigned a hotkey to try to activate the plug in but no success. Nothing happen, no error. Please help to check.

the plugin commands are only visible if you are active inside a notes window.  i.e. if you hit a key on the keyboard, you see it in the note.
if your focus is in the folder view, for example, or any other pane in Obsidian, then the commands do not show up in the palette.
can you confirm you are in a notes view?  I'm on 1.6.3 btw and haven't noticed any changes or issues myself

the plugin commands are only visible if you are active inside a notes window. i.e. if you hit a key on the keyboard, you see it in the note. if your focus is in the folder view, for example, or any other pane in Obsidian, then the commands do not show up in the palette.
can you confirm you are in a notes view? I'm on 1.6.3 btw and haven't noticed any changes or issues myself

Thanks. It works now.


Fix wrong URL in S3 HTML template

Remove the duplicated itms-services://?action=download-manifest&url=

Coverage increased (+0.006%) to 66.945% when pulling 3c18340cc856cbf067b352587df4c1341c914570 on thii:s3_html_template into 424e8137c3438474b5290880a0cfc2d89259c169 on fastlane:master.

Thanks for the PR @thii! Do you have an example of how this was failing? Or a test to confirm it is working as expected now? Would you mind creating an issue and linking this PR with a bit more description of the problem?

There was an issue here #743 but it was closed without being fixed.

@thii I know of multiple teams successfully using the action. If we would merge this, it would probably break their setup. Do you know why this is working for some people?

@KrauseFx Probably they (and I think most people) use custom template.

@thii You're right, according to https://stackoverflow.com/questions/25562693/ios-enterprise-deployement-clicking-on-itms-services-link-results-in-cannot-co we're currently using the wrong format.
I'm really surprised this hasn't been raised or fixed before. Let's see if everything works as expected now.
Thanks for the pull request @thii 🚀


NanNew doesn't work with promise resolvers

The following line of code doesn't compile:
::v8::Local<::v8::Promise::Resolver> promiseResolver = NanNew<::v8::Promise::Resolver>();

Not sure if I'm doing something wrong or Nan simply doesn't support this use case.
According to [the docs][1], ::v8::Promise::Resolver should be equally as easy to construct as any other object.
[1] https://v8docs.nodesource.com/node-0.12/d5/d76/classv8_1_1_promise_1_1_resolver.html

The compile error is the following:
../node_modules/nan/nan_new.h:245:1: note: candidate: template<class T, class A2> typename NanIntern::Factory<T>::return_t NanNew(NanFunctionCallback, v8::Handle<v8::Value>, A2)
 NanNew( NanFunctionCallback callback
 ^
../node_modules/nan/nan_new.h:245:1: note:   template argument deduction/substitution failed:
../node_modules/nan/nan_new.h: In substitution of ‘template<class T, class A2> typename NanIntern::Factory<T>::return_t NanNew(NanFunctionCallback, v8::Handle<v8::Value>, A2) [with T = v8::Promise::Resolver; A2 = <missing>]’:
../src/misc.h:244:96:   required from here
../node_modules/nan/nan_new.h:245:1: error: invalid use of incomplete type ‘struct NanIntern::Factory<v8::Promise::Resolver>’
In file included from ../node_modules/nan/nan.h:82:0,
                 from ../src/database-wrap.h:36,
                 from ../src/database-wrap.cpp:30:
../node_modules/nan/nan_new.h:43:30: note: declaration of ‘struct NanIntern::Factory<v8::Promise::Resolver>’
 template <typename T> struct Factory;
                              ^

There's no support for ES 6 features. If it's not doable on Node 0.8, it is likely not doable with NAN. You have to use raw V8 for these parts.

Oh, I see your point. There are no promises in the V8 version of Node 0.8 and 0.10 either. Sorry for opening this issue then! :)

No problem. Just use Promise::Resolver::New directly. Do not that its signature has changed in V8 4.3 to return a MaybeLocal<Resolver>, so you'll have to do conditional compilation if you want to support that.

Is there a reason we can't add a v0.12+-only overload for NanNew?  I suppose the biggest issue is that the C++ Promise API is still experimental-ish and going through some changes because of the Local/MaybeLocal switch.

Only major reason is that I like to keep the API uniform. In time, as interest grows and fewer use 0.8 or 0.10, I imagine this can be loosened, but I don't want a torrent of inane issues about that kind of functionality not being available on some old version.

I was personally thinking about creating a custom, promise-like object for use in versions < 0.12 - do you think such a thing would be welcome in Nan?

Only if it is a working port of V8's promises, with the same API, the same
features and consistent behavior on all supported platforms. That will be a
ton of work if it works at all.
If you were thinking of something more lightweight, you're probably better off
writing an addon offering the intended functionality.
On Friday 29 May 2015 03:38:41 Timur Krist�f wrote:

I was personally thinking about creating a custom, promise-like object for
use in versions < 0.12 - do you think such a thing would be welcome in Nan?

Reply to this email directly or view it on GitHub:
https://github.com/nodejs/nan/issues/366#issuecomment-106769524

Hm. There is one more thing I'm not sure about: performance-wise, is it worth the effort to pursue promises on the C++ side, or is it equally as good if I just choose a JavaScript-based promise library (Q or kew come to mind) and write a JS wrapper around my C++ add-on?

Each transition between C++ and javascript has a fixed cost, so if you do work
that requires less time than this, it will be faster to run it in javascript.
V8's promise implementation has not yet been optimized for performance, which
is why bluebird beats it in tests. You're probably better off doing the
promises in JavaScript. It will at least be much easier to get it right.
On Friday 29 May 2015 03:49:53 Timur Krist�f wrote:

Hm. There is one more thing I'm not sure about: performance-wise, is it
worth the effort to pursue promises on the C++ side, or is it equally as
good if I just choose a JavaScript-based promise library (Q or kew come
to mind) and write a JS wrapper around my C++ add-on?

Reply to this email directly or view it on GitHub:
https://github.com/nodejs/nan/issues/366#issuecomment-106771699

These benchmarks are totally contrived, but it looks Lie could be faster than Bluebird (and both still completely smash V8):

http://jsperf.com/promise-comparisons/155
https://github.com/calvinmetcalf/lie

@jokeyrhyme What do you mean, contrived? Btw, I've found this comparison: http://complexitymaze.com/2014/03/03/javascript-promises-a-comparison-of-libraries/ - according to this guy, lie, q and bluebird are in the middle and kew is the fastest.

@Venemo nice comparison, especially in terms of features and size, although the benchmark (actually an older version of the one I linked) uses old versions of these libraries. By "contrived", I mean the jsPerf is a bit unrealistic: it just immediately resolves Promises with no work, no errors, no chaining, no .all()ing, etc.

@jokeyrhyme To be honest, I agree and don't think these benchmarks have any actual meaning in real-world applications (which are more likely to be I/O bottlenecked), but it never hurts to look around to find the most performant choice. :) Your link doesn't contain a benchmark for kew, btw.


ShawnCampbellPixart

Comfort 1.5
Completeness 1.5
Getting a slightly understanding of click events and functions. I will keep working out code options and find solution. I spent about 4 hours on this which is hard to see, but I am picking more things up as I try to google ways to implement my pseudo code.

Let me know if you want to talk about this assignment. Based on our 1:1 today, it sounds like you've reviewed the things covered here. But nevertheless, happy to do so.
Also strongly suggest not to submit the solution as your homework assignment. It's far more helpful to see code you wrote yourself, even if it does not work. That way I can identify what we need to work on.
:+1:


Initializer lists fail on nested structs

struct MyStruct {
  int b;
};

struct A {
  struct MyStruct d;
};

struct A f() {
  struct MyStruct e;
  return (struct A){.d = e};
}

unexpectedly fails with:
test.c:11:19: error: initializing 'int' from incompatible type 'struct MyStruct'
  return (struct A){.d = e};
                  ^

Duplicate of  #189

Oops. Thanks!


Update pylint to version 3.0

Remove leading _ from type alias (since it is type annotations it is effectively a public name)
Disable the cyclic import checker. By its definition, we do have cyclic imports. pylint seems not to find them when using multiple processes but it does find them when using a single process.

I want a newer version of pylint for #1285 where I want to use it to replace some of the benefits we were getting from QiskitTestCase.

By the way, I ended up not needing this for #1285. I was going to replace some of the checking that QiskitTestCase did with a custom pylint checker, but I decided that it was not necessary.


make some more linktypes and make them implicit

Design document mentions a number of implicit Links (child of, parent of etc) make the relevant changes so that the correct linktypes get assigned when docs are exported from the DB
ensure when Docs are added to the DB lineage is preserved

closed by #33


Feature/codelist UI

if this was a migration thas was previously merged, you will need to create a new migration to update the old data


CFG Branch Logistics Mistake

Faizaan the logic of your code needs to be updated. If you press the remove component button when no componenets exist. The code will then take 2 add component presses to add a new component.
I will fix it.

Resolved

Thank you for fixing this.


Update make_prots and BBH functions to play nice with NCBI

Right now, it just uses the default gene NCBI gene IDs, which look like lcl|NC_000913.3_prot_NP_414542.1_1. However, we want to use the locus tag, which is hidden in that header string. Example code shown below:
records = SeqIO.parse(fasta_file,'fasta')
result = []
id_list = []
for record in records:
    record.id = re.search(r'\[locus_tag=(.*?)\]',record.description).groups()[0]
    # Ignore duplicate sequences
    if record.id not in id_list:
        id_list.append(record.id)
        result.append(record)
new_fasta_file = os.path.join(blast_dir,org+'.fna')
SeqIO.write(result,new_fasta_file,'fasta')

Fixed in #70


QA display fixes as per the doc.

Rebased to master. Had to resolve a conflict on lines 375-389 within ui-customizations. We recently changed the showcase list to use material-ui Avatars instead of basic images, so the changes to image formatting in this section were no longer needed.

:+1:

:+1:


External DNS failing in pods when IPv6 is used

Describe the bug
Building a java service that deploys into k8s via orbstack.
Java service deploys fine and runs.
A request is generated that causes the java service to attempt a DNS lookup for an external service.
The /etc/resolve.conf has both IPv4 and IPv6 nameservers.
The external DNS lookup fails with the following errors
Caused by: io.netty.resolver.dns.DnsResolveContext$SearchDomainUnknownHostException: Failed to resolve 'rxnav.nlm.nih.gov' [A(1), AAAA(28)] and search domain query for configured domains failed as well: [diagnosis-service-test.svc.cluster.local, svc.cluster.local, cluster.local]
 Caused by: io.netty.resolver.dns.DnsNameResolverTimeoutException: [3493: /[fd07:b51a:cc66:a:8000:0:0:a]:53] DefaultDnsQuestion(rxnav.nlm.nih.gov.diagnosis-service-test.svc.cluster.local. IN AAAA) query '3493' via UDP timed out after 5000 milliseconds (no stack trace available)
Forcing java to prefer IPv6 or IPv4 does not resolve the issue.
To Reproduce

Deploy java service that uses netty to do dns lookups on an IPv6 enabled network.
Perform DNS query over IPv6.

The current work around was to add to the entrypoint of the container to remove any IPv6 nameservers in the /etc/resolve.conf but that is a hack that requires root access which we don't grant to our pods.
Expected behavior
DNS resolution is successful.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.8.2
Commit: 44cf8269710cd29802bfee176a7cec65b55cc822 (v1.8.2)
System info:
macOS: 15.1.1 (24B91)
CPU: arm64, 10 cores
CPU model: Apple M1 Max
Model: MacBookPro18,4
Memory: 64 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-11-25T11-44-27.875167Z.zip

Screenshots and additional context (optional)
No response

Fixed for the next version.

Released in v1.9.0.


Default Instruction implementation allows invalid CreditReceiver allocation

Hello, I think there's an issue on CreditReceiver allocation.
The way is implemented allows the following scenario:
--
--  INSTRUCTION:
--
--            bank
--  sender ---------> receiver
--
--
--  ACCOUNTS:
--
--              bank
--           /         \
--    sender         receiver
--
--  bank allocates with CreditReceiver
--
--

For instance:
-- Allocate instruction
  (cashInstructionCid, _) <- submit bank do
    exerciseCmd cashInstructionCid Instruction.Allocate with
      actors = Set.singleton bank; allocation = CreditReceiver

  -- Approve instruction
  cashInstructionCid <- submit receiver do
    exerciseCmd cashInstructionCid Instruction.Approve with
      actors = Set.singleton receiver; approval = TakeDelivery receiverAccount

  -- Settle batch
  submitMultiMustFail [bank] [publicParty] do
    exerciseCmd batchCid Batch.Settle with actors = Set.singleton bank

https://github.com/digital-asset/daml-finance/blob/66495d084ed9307e126fc6bd0e87e9ce900ad0a6/src/main/daml/Daml/Finance/Settlement/Instruction.daml#L116
 CreditReceiver -> do
   allMustAuthorize $ Set.singleton routedStep.custodian
   pure allocation

I would say that the code bellow might fix the issue:
 CreditReceiver -> do
   allMustAuthorize $ Set.singleton routedStep.custodian
   mustBe this Custodian routedStep.sender
   pure allocation

Hi @kindofdev,
Thank you for bringing this up! I agree that it would indeed be a nice enhancement to add the sanity check for the
CreditReceiver allocation, namely:
mustBe this Custodian routedStep.sender

This is something we can consider including in a future release.
That said, we currently don't see it as a high-priority change for the following reasons:

The sanity check (^) is already performed during the execution of the instruction. If an instruction is "wrongly" allocated with CreditReceiver, it can't be executed —similar to how an unallocated instruction would fail when executed.
Custodians are expected to know how to correctly allocate instructions. In particular, they could avoid allocating with CreditReceiver unless they are the sender in the instruction.

We appreciate your feedback and will keep this suggestion in mind for upcoming updates.
Thanks again,
Johan


dragNodes plugin doesn't work correctly when zoomed.

When zoomed into a graph, the dragNodes plugin seems to freak out and repositions the dragged node seemingly randomly.
Replicate by using the examples/drag-nodes.html file and zooming into the graph until it stops.  Then try to move a node.  It will not follow the mouse cursor.
Any ideas?

drag-nodes plugin really strange. On every mouse move calls:
 function nodeMouseMove(event) {
      if(navigator.userAgent.toLowerCase().indexOf('firefox') > -1) {
        clearTimeout(timeOut);
        var timeOut = setTimeout(executeNodeMouseMove, 0);
      } else {
        executeNodeMouseMove();
      }

But User-Agent is CONSTANT! We need test it only once. JS already toooooo slow.


Removing worldspawn brush (no entities!)

I strongly ask you to add a function to remove individual worldspawn brushes (no entities!). For example, boxes, containers, etc. I'm in the middle of porting maps, and I really need this function in order to avoid decompilation. Porting maps from dod, tfc, csczsd to cs 1.6.

Remove... sounds like more easy that adding)

This is impossible, short of decompiling and recompiling. In a sense, the entire worldspawn is a single giant brush. Everything is superglued into a single piece and it's not possible to take them apart at the seams because everything not on the surface has been stripped during compilation.
Not to mention how binary space partitioning has sliced the map up into nodes and leafs along the seams of the world geometry, containers, boxes and all. Removing those world "objects" actually mean "adding" BSP volumes that weren't there during compilation so there would be no BSP leaf info, no visibility info, and everything would appear and not appear at the same time. It'd be the same as standing in the void.


stacking bricks

I am stacking "bricks" in a matter.js model. The bricks are overlapping like a classic wall for stability. However I can't get the wall stable enough to remain standing.

I found the below properties of Body in the documentation (I have only added what I found relevant.)
// Brick properties (Body defaults)
options = {
    isStatic: false,
    density: 0.001,
    slop: 0.05,
    restitution: 0,
    sleepThreshold: 60,
    friction: 0.1,
    frictionAir: 0.01
};

I have no success in altering any of these parameters. I expected especially friction to prevent the wall from sliding apart, but no, not even when setting it to 1.
Here is a jsfiddle. Any ideas what im missing?
http://jsfiddle.net/1L0dek1t/1/

First use the master build:
https://github.com/liabru/matter-js/blob/master/build/matter.js
Which already improves it a lot. Then I'd suggest increasing iterations:
var engine = Engine.create(document.body, {
positionIterations: 10,
velocityIterations: 10
});
I think you should also add small bricks on the ends to ensure there's no overhang.
Even then I still think it may eventually collapse. You should also make the blocks larger (2x would be good) which will help reduce the error, you can then simply render them smaller.
If you enable sleeping this should prevent it collapsing, but sleeping has a few drawbacks in the current implementation, but it's worth a shot.

First use the master build:
https://github.com/liabru/matter-js/blob/master/build/matter.js
Which already improves it a lot. Then I'd suggest increasing iterations:
var engine = Engine.create(document.body, {
positionIterations: 10,
velocityIterations: 10
});
I think you should also add small bricks on the ends to ensure there's no overhang.
Even then I still think it may eventually collapse. You should also make the blocks larger (2x would be good) which will help reduce the error, you can then simply render them smaller.
If you enable sleeping this should prevent it collapsing, but sleeping has a few drawbacks in the current implementation, but it's worth a shot.


Remove jackson-databind and jackson-annotations dependencies now coming from core

Signed-off-by: Sicheng Song<EMAIL_ADDRESS>Description
jackson-databind and jackson-annotations were added as api dependencies in server of opensearch core. The security plugin now gets these dependencies transitively and does not need direct dependencies on the jars.
PR in core where the dependencies are added: https://github.com/opensearch-project/OpenSearch/pull/5366
Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance
Issues Resolved
This PR will resolve https://github.com/opensearch-project/ml-commons/issues/634 and complete campaign https://github.com/opensearch-project/opensearch-plugins/issues/191
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #651 (2ddea3f) into 2.x (ec8a997) will increase coverage by 4.19%.
The diff coverage is 47.61%.

@@             Coverage Diff              @@
##                2.x     #651      +/-   ##
============================================
+ Coverage     79.86%   84.06%   +4.19%
- Complexity      927      970      +43
============================================
  Files            92       92
  Lines          3522     3540      +18
  Branches        324      326       +2
============================================
+ Hits           2813     2976     +163
+ Misses          586      427     -159
- Partials        123      137      +14

Flag
Coverage Δ

ml-commons
84.06% <47.61%> (+4.19%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...h/ml/action/unload/TransportUnloadModelAction.java
79.68% <47.61%> (+62.29%)
:arrow_up:

...ain/java/org/opensearch/ml/task/MLTaskManager.java
60.20% <0.00%> (ø)

...arch/ml/action/forward/TransportForwardAction.java
90.90% <0.00%> (+9.09%)
:arrow_up:

.../ml/action/syncup/TransportSyncUpOnNodeAction.java
87.87% <0.00%> (+69.69%)
:arrow_up:

...ml/action/load/TransportLoadModelOnNodeAction.java
94.44% <0.00%> (+75.55%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Are constraints on the CostVar possible?

This is a cross-post from stack exchange: https://stackoverflow.com/questions/58809525/how-can-i-search-for-a-solution-with-a-lower-cost
I have found a solution with assignment.objectiveValue() equal to 4400. I would like to search for solutions with a lower cost than 4400. My first attempt is:
cpsolver.addConstraint(cpsolver.makeLess(routingModel.costVar(), 4400));

But java informs me that:
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007fe9dfa76227, pid=29481, tid=0x00007fea3a35a700
#
# JRE version: OpenJDK Runtime Environment (8.0_222-b10) (build 1.8.0_222-8u222-b10-1ubuntu1~18.04.1-b10)
# Java VM: OpenJDK 64-Bit Server VM (25.222-b10 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# C  [libortools.so+0x6ca227]  operations_research::Solver::MakeSum(operations_research::IntExpr*, long)+0x17

My hunch is that I can not add constraints that involve the costVar(). Is that true?

Answered on S.O.


Relations to identify equals objects

We have to check this image and think how this will affect our rules.

The CAEX paths need to be considered and added to the rules

Criteria to define different elements.

Criteria to identify set differences, which is supported in PSL

So should i create a new File with difference.txt or add these with 0 ?

I added negative with values with 0


Generate forecasting information that combines CHIME and IHME

The goal of this issue is to create an easy-to-consume way to generate forecasting data from IHME and CHIME data.
Currently IHME is only by-state, so this combination will be at the state-level until #79 or IHME releases county level data.
The dataset should have a normalized format - for instance, CHIME produces day counts while IHME has dates.
Documentation should be made to describe the most comparable columns between the two datasets.

Re: integrating CHIME model and running it on regional levels, @lossyrob  is making headway on this at

#79
https://github.com/covidcaremap/covid19-healthsystemcapacity/blob/master/notebooks/00_getting_started/02_Working_with_demand_data.ipynb
https://github.com/CodeForPhilly/chime/pull/272

@jsfenfen is interested in running the CHIME model at CBSA and State levels too so looping in here.


Feature/stash indicator

I implemented an indicator for the current stash status as requested in issue #31.
This pull request includes

stash indicator functionality for bash and zsh
new color model was adapted
documentation was added in Readme and command prompt help

The code has been thoroughly tested in bash shell, but not yet in zsh as I don't have acces to one.
But given the code, I hope it should just work in zsh as well.

I just realized how the images in README.md work.
@michaeldfallen If you are planning to merge this, could you create another image for this?

This looks awesome, just pulled it to check it out and I'm liking. good job.
Apologies but I merged the #65 Format string changes first so this branch no longer merges. Seems it's not the first time I've done that to you.
Since it's so old I'll bring it up to scratch and ask you to merge it once I'm ready, then we'll merge this. Closing for now.


Global definitions hiding lint errors

My application has to import .svg files. When importing an .svg Typescript report missing type declarations error.

To handle this error I created a global module declaration
declare module '*.svg' {
  const content: string
  export default content
}

But with this global declaration if the file path is wrong, TS doesn't report any errors.
Example
Folder structure
app
 |
 ├ img
 |  └─ top-view.svg
 |
 └─index.ts

index.ts

No errors, but the correct path is ../img/top-view.svg

declare module is intended to be used when a file with that name isn't present. This is like turning your volume to zero and then expecting to hear music when you press "Play".

Hello @RyanCavanaugh
OK, but if the file path is correct I'm getting the error Cannot find module (first image).
If declare module is not the correct way to handle not recognized files could you explain how could be do?
Thanks

There isn't a mechanism that says "This is how to treat files that are on disk but not TS/JS files"

I see, this issue affects many projects. Do you know if in next TS versions there is something in backlog to handle that?
If I could give a suggestion, compiler could still returning missing type declarations for not TS/JS files. But always check if import path results in a file and return an error if not, even if file is not an TS/JS.

dude its not working as intended the intended way is as if you suggest some thing else or always verify import paths
if you know any way else for module augmentation or any thing els please help others
example fix might look like "you could inerface it so it can validate path and know what its return looks like
example interface "*.svg"{
ReactComponent: JSX.Element
}
"


What would you do for a LazyVStack ?

Hi!
Amazing helper for iOS 13 support! thank you, appreciated here!
I was wondering, I have performance issues since i have a lot of data. I wanted to use a LazyVStack but I am starting to have issues like this:
ID (sb-l2-000021) not found, make sure to add views with `.id(_:scrollView:)`. Did find: [AnyHashable("sb-l2-000004"): (0.0,<PHONE_NUMBER>000001,<PHONE_NUMBER>333333, 168.0), AnyHashable("sb-l2-000005"): (0.0,<PHONE_NUMBER>000001,<PHONE_NUMBER>6666663, 196.0), AnyHashable("sb-l2-000008"): (0.0, 1322.4, 371.0, 140.0), AnyHashable("sb-l2-000003"): (0.0,<PHONE_NUMBER>000001,<PHONE_NUMBER>6666663, 112.0), AnyHashable("sb-l2-000001"): (0.0,<PHONE_NUMBER>000001,<PHONE_NUMBER>6666663, 112.0), AnyHashable("sb-l2-000006"): (0.0, 1086.4,<PHONE_NUMBER>6666663, 140.0), AnyHashable("sb-l2-000009"): (0.0, 1482.4, 365.0, 84.0), AnyHashable("sb-l2-000013"): (0.0, 2010.4, 374.0, 140.0), AnyHashable("sb-l2-000011"): (0.0, 1830.4,<PHONE_NUMBER>6666663, 56.0), AnyHashable("sb-l2-000012"): (0.0, 1906.4, 374.0, 84.0), AnyHashable("sb-l2-000002"): (0.0,<PHONE_NUMBER>000001, 374.0, 84.0), AnyHashable("sb-l2-000007"): (0.0, 1246.4, 374.0, 56.0), AnyHashable("sb-l2-000010"): (0.0, 1586.4, 374.0, 224.0)]

I believe the fact that I am using a LazyVStack, it hasn't load the Text() with their respective ids. What do you guys recommend to do here?
Thanks for any future help!

I'm unsure how the SwiftUI 2.0 implementation works for that so I can't build that into this package.
I would use the SwiftUI 2.0 ScrollViewProxy if you use lazyvstack

I'm unsure how the SwiftUI 2.0 implementation works for that so I can't build that into this package.
I would use the SwiftUI 2.0 ScrollViewProxy if you use lazyvstack

I see, I am not able to use the ScrollViewProxy from Swift2.0 since i have to build for iOS 13 :/
Thanks for your help!

I see, I am not able to use the ScrollViewProxy from Swift2.0 since i have to build for iOS 13 :/
Thanks for your help!

But LazyVStack is also SwiftUI 2.0?

But LazyVStack is also SwiftUI 2.0?

 But it doesn't show an error for it but for the scroll to it says that I need to be iOS 14.  But it doesn't show an error for it but for the scroll to it says that I need to be iOS 14. :/

Uhhm, not sure what you mean but Apple docs say that LazyVStack is also iOS 14+
https://developer.apple.com/documentation/swiftui/lazyvstack

Uhhm, not sure what you mean but Apple docs say that LazyVStack is also iOS 14+
https://developer.apple.com/documentation/swiftui/lazyvstack

Weird, will check that again! Thanks!

Weird, will check that again! Thanks!

I had a custom implementation of LazyVStack this is why!
Totally forgot about it 😅

I had a custom implementation of LazyVStack this is why!
Totally forgot about it 😅

Oh cool, I'm interested how that would work, do you have a package for this?

Oh cool, I'm interested how that would work, do you have a package for this?

It is just a wrapper that will fallback on VStack if LazyVStack is unavailable
Sorry i expressed it wrong 😂

It is just a wrapper that will fallback on VStack if LazyVStack is unavailable
Sorry i expressed it wrong 😂

Ah, well you can make the same for ScrollViewReader

Ah, well you can make the same for ScrollViewReader

TempTemp.zip
I was able to reproduce in a more isolated and sharable project.

TempTemp.zip
I was able to reproduce in a more isolated and sharable project.

Quick note, this doesn't work with a List either. I believe it is related to the lazy loading too, since List (based on UITableView) is also lazy loading the content.


Allow installing Symfony 4 components

Javier - one of the reps you bumped is not Symfony. And we need to update the Travis config to test 4 (and does the bundle expose in services that should be made public?)

Could we please merge this and tag a new stable release? Thanks.

@javiereguiluz I think you can close here. FOSCKEditorBundle is where development happens now.


added tests for jump instruction and attempted bug fix for abs jump

Bug fix
Bug fix for error: running instruction: pc update: absolute jump: memory value is not an address
Test Jump Instructions:
Test relative jumps (where the operand represents an offset from the current pc) and absolute jumps and tests update for TestUpdatePcJump
Unconditional jumps.
jmp abs <address>
jmp rel <offset>

Conditional jumps.
jmp rel <offset> if <op> != 0

#117

@Tomi-3-0 remember to close PRs you are no longer going to work on


Event consumers must be robust against duplicates

Improves clarity and provided context information.
It is important to create a clear understanding of event duplicates, that event producers support deduplication, and event consumers ensure to be robust against duplicates to avoid business processing failures or analytical data quality issues  (see internal (!) paper: https://docs.google.com/document/d/1lGfKcKqncX04QTmL0moq9RRefDpcm-itJoJLuN4CrhA/edit#)

:+1:

👍


Fix typo in comment in Py style guide.

iff is not a typo.  it's short for "if and only if".
https://en.wikipedia.org/wiki/If_and_only_if


Tuners moved to GOLEM

PipelineTuner (now SimultaneousTuner) and SequentialTuner were generalized for domain graphs and moved to GOLEM
to customise search space one can use PipelineSearchSpace instead of SearchSpace

Изменила версии питона, чтобы прошли проверки


Tweak 12A0 for ClimaRad HVAC

parser, regex, TypedDict with inherit/moved, logs

I have refactored the typed dicts - you'll have to rebase.

Convinced by your example parsing. Made the regex stricter by requiring 1 or more of these elements in the packet.
Added a humidity_element parser that can handle the name of indoor and outdoor (based on index as extracted from the packet list).
What do we need to do here - or in ramses_cc - to make sure Home Assistant understands the results and maps them to typical HRU properties?


assimp: 5.3.1 -> 5.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/assimp/assimp/releases.
meta.description for assimp is: A library to import various 3D model formats
meta.homepage for assimp is: https://www.assimp.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.4.0 with grep in /nix/store/s2n35flq70bhkq3xcr9lmig1a0yv01nf-assimp-5.4.0
found 5.4.0 in filename of file in /nix/store/s2n35flq70bhkq3xcr9lmig1a0yv01nf-assimp-5.4.0

Rebuild report (if merged into master) (click to expand)

37 total rebuild path(s)

37 package rebuild(s)

First fifty rebuilds by attrpath
assimp
comic-mandown
crocoddyl
cutter
cutterPlugins.rz-ghidra
hpp-fcl
kdePackages.falkon
kdePackages.full
kdePackages.qt3d
nitrokey-app2
normcap
onedrivegui
onionshare
onionshare-gui
openusd
pinocchio
pioneer
python311Packages.crocoddyl
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.pinocchio
python311Packages.pyside6
python312Packages.crocoddyl
python312Packages.example-robot-data
python312Packages.hpp-fcl
python312Packages.pinocchio
python312Packages.pyside6
qt6.full
qt6.qt3d
qt6Packages.full
qt6Packages.qt3d
retool
smb3-foundry
streamdeck-ui
syncplay
trenchbroom
vkmark

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s2n35flq70bhkq3xcr9lmig1a0yv01nf-assimp-5.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A assimp https://github.com/r-ryantm/nixpkgs/archive/403fe0f349d39fb826705a4eced3675edb65ec64.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/403fe0f349d39fb826705a4eced3675edb65ec64#assimp

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s2n35flq70bhkq3xcr9lmig1a0yv01nf-assimp-5.4.0
ls -la /nix/store/s2n35flq70bhkq3xcr9lmig1a0yv01nf-assimp-5.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

17 packages failed to build:

comic-mandown
comic-mandown.dist
crocoddyl
nitrokey-app2
nitrokey-app2.dist
normcap
normcap.dist
pinocchio
python311Packages.crocoddyl
python311Packages.example-robot-data
python311Packages.pinocchio
python312Packages.crocoddyl
python312Packages.example-robot-data
python312Packages.pinocchio
sbclPackages.classimp
sbclPackages.classimp-samples
sbclPackages.clinch-classimp

38 packages built:

assimp
assimp.dev
assimp.lib
cutter
cutterPlugins.rz-ghidra
hpp-fcl
hpp-fcl.dev
hpp-fcl.doc
kdePackages.falkon
kdePackages.falkon.debug
kdePackages.falkon.dev
kdePackages.full
kdePackages.qt3d
kdePackages.qt3d.dev
onedrivegui
onedrivegui.dist
onionshare
onionshare-gui
onionshare-gui.dist
onionshare.dist
openusd
pioneer
python311Packages.hpp-fcl
python311Packages.hpp-fcl.dev
python311Packages.hpp-fcl.doc
python311Packages.pyside6
python312Packages.hpp-fcl
python312Packages.hpp-fcl.dev
python312Packages.hpp-fcl.doc
python312Packages.pyside6
retool
retool.dist
smb3-foundry
streamdeck-ui
streamdeck-ui.dist
syncplay
trenchbroom
vkmark

Maintainer pings
cc @ehmry for testing.

@ehmry : I think I can be added to the maintainer list here, if it can help.


feat(forge): add getCode cheatcode

This addresses #326. I've got a basic version working which allows users to fetch the contract creation code, but have a couple of questions.

This only fetches the creation code so it is not compatible with etch. I think this is probably the best way to go since most people will be wanting to go through the normal deployment process. However, the UX for taking the creation code and deploying it is quite bad, as it requires some inline assembly. Would it be better to have the cheatcode handle the whole deployment process, or should we instead leave that to some sort of standard library?
I was not able to write automated tests for this. This is because the current testing setup for the cheatcodes is not really compatible with this kind of cheatcode, which assumes that we are running the test in a standalone forge project.

Ok I think this version is much cleaner. I autodetect the artifacts path using ProjectPathsConfig::find_artifacts_dir. If you pass just a solidity filename in, it assumes that the contract name is the same as the filename. However, it still supports specifying the contract using the form ContractFile.sol:ContractName. This tests are failing here but I think it is a fluke since they are running fine on my machine and it seems CI is unstable.
I looked into passing artifacts all the way down to cheatcode_runner, but it started to get ugly pretty quickly. If we think its worth it to continue to explore that direction I'd be happy to but I might need some clarification on how best to pass the field all the way down.

I went through and made the suggested changes. Let me know if we need anything else before we merge.

cc @libevm re: getting arbitrary artifacts bytecode


Move to file based catalog & containerized build process

The SQLite-based catalog format, including the related CLI commands, is a deprecated feature. So we are moving to the latest file-based catalog.
The process of building images(building of binary) is now wholly containerized. And the old index & appregistry methods are removed.

/hold
I needs a lot of work on openshift-ci side to make this feasible. So this will take time. Putting on hold till then.


Registration failed: The application identifier is missing

I'm getting this issue but I flowed all steps that you are mentioned in official documentation.
Device token error: PlatformException(PUSHY ERROR, Registration failed: The application identifier is missing. Are you using ProGuard? Please contact support., null, null)
please let me know if I have to additionally add something except from Doc

Hi @nj9666-xcelore,
Thanks for reaching out. We'd be glad to assist.
Flutter comes preconfigured with ProGuard. Please make sure the following lines are present in your project's android/app/proguard-rules.pro file (please create this file if it doesn't exist yet):
-dontwarn me.pushy.**
-keep class me.pushy.** { *; }
-keep class androidx.core.app.** { *; }
-keep class android.support.v4.app.** { *; }

Please give it a try, and let us know if you need any further assistance.


Add desktop shortcuts to qgis

Need to add the start file to https://github.com/CryoInTheCloud/hub-image/blob/26eb812a9999525f8fd2d3ded5d0170f52a50ab5/.github/workflows/build.yaml#L7-L10 and https://github.com/CryoInTheCloud/hub-image/blob/26eb812a9999525f8fd2d3ded5d0170f52a50ab5/.github/workflows/test.yaml#L5-L8 so that the docker build is triggered.
In fact, I should probably have added all of the config files listed under https://repo2docker.readthedocs.io/en/latest/config_files.html#configuration-files while doing #10...

So QGIS 3.22 couldn't be installed at bbd75cca43c65029622a0ff90b21d79147d526b5 because of cartopy pinned to 0.20.2, see https://github.com/CryoInTheCloud/hub-image/actions/runs/3571789674/jobs/6003992007#step:5:4030.
- package cartopy-0.20.2-py310h902574e_0 requires geos >=3.10.1,<<IP_ADDRESS>a0, but none of the providers can be installed

Tried to use the newer cartopy=0.21.0 locally, and then it gave this error:
package qgis-3.22.11-py310heaafcf0_0 requires pdal >=2.4.3,<2.5.0a0, but none of the providers can be installed

So need to wait for things to sort out in PDAL, I think at https://github.com/conda-forge/pdal-feedstock/pull/190.

@weiji14 do you know why this might still be failing? I have removed all changes to qgis

@weiji14 do you know why this might still be failing? I have removed all changes to qgis

I'm slowly working through the dependency pins. Something with the recent GDAL mayhem. Give me an hour or so...

Ok, I think I've found a good combination that works with the mamba solver. Need to bump up versions of cartopy, rasterio, hdf5 and qgis. See https://github.com/CryoInTheCloud/hub-image/pull/14/commits/90fe9f603fbb3df66a5cb1a9663e7e6807d451f0.

@weiji14 perfect! So I'll just wait for that PR to land and then rebase this one.

Thanks @weiji14! And yes re: tagging. Not sure how to automate that...

Hmm, there was some tagging stuff in the CryoCloudWebsite repo we could copy over I think? I do feel like we're slowly moving in the direction of https://github.com/pangeo-data/pangeo-docker-images 😂

@weiji14 how do you feel about extracting the conda-lock stuff from here into an action by itself? Then we can use that action in pangeo-docker-images, thus removing duplicate work...


Kekw

Sup
I think u made a mistake in the readme where “kewk” should be replaced by “kekw”

Fixed


Bug: diff now showing

Self Checks

[X ] I'm using the latest lualine.
[X ] I didn't find the issue in existing issues or PRs.

How to reproduce the problem

Expected behaviour
lualine to show diff.
Actual behaviour
lualine is not showing diff as per default configuration.
Minimal config to reproduce the issue
--[lualine]--
require('lualine').setup({
options = {disabled_filetypes = {'packer', 'NvimTree'}},
extensions = {'fugitive'}
})

Additional information
I set sections = {lualine_b = {'branch', 'b:gitsigns_status', 'diagnostics'}}, now because i'm using the plugin.

I set sections = {lualine_b = {'branch', 'b:gitsigns_status', 'diagnostics'}}, now because i'm using the plugin.

Can you show what's in the rest of the sections? In the above you aren't setting the diff, are you?

I set sections = {lualine_b = {'branch', 'b:gitsigns_status', 'diagnostics'}}, now because i'm using the plugin.

Can you show what's in the rest of the sections? In the above you aren't setting the diff, are you?

That's the only section i configured, i'm using b:gitsigns_status because diff is not working.
Complete config:
use {
    'nvim-lualine/lualine.nvim',
    requires = {'nvim-tree/nvim-web-devicons', opt = true},
}

--[lualine]--
require('lualine').setup({
    options = {disabled_filetypes = {'packer', 'NvimTree'}},
    sections = {lualine_b = {'branch', 'b:gitsigns_status', 'diagnostics'}},
    extensions = {'fugitive'}
})

you've configured lualine not to show diff.
add diff to your lualine_b section
lualine_b = {'branch', 'diff', 'diagnostics'}

if it's still not showing use the gitsigns source shown here

I was not using 'diff' because it was not working "alone".
Using gitsigns source let it work, thank you.
Out of curiosity, why 'diff' alone was not working?


Github-user-finder

Related Issue
issue #580
Description
it is simple github username finder, where developers can access any user
it provides description of searched user as contributions, prs, issues raised etc.
Screenshots

Thankyou sir. One more doubt can I add more projects ?

Thankyou sir. One more doubt can I add more projects ?

yeah sure


Installation without internet access

As a system administrator, on a CF appliance that doesn't have internet access, I would like to install this project.
For example by downloading a .tgz/.zip to my laptop, then uploading via a vpn, then running Ansible.

Thank you for the suggestion. I have modified the README as well as the installation playbook to accept offline installs.

Thank you.


README.md has errors in documentation

The headlines in the README.md do not match the actual topic they are describing. In addition throughout the document (and in some comments within the project as well), the RFC3164 has a typo (spelled RFC3613)

Fixed by https://github.com/CloudBees-community/syslog-java-client/pull/13


backend task not registering

@magland I restarted kachery daemon and sortingview backend but the tasks are not registered

output from sortingview backend

This has been resolved. Closing.


Project 📃: Hangman Game with Illustration

WebMasterLog Repository (Proposing new issue)
:red_circle: Project Title : Hangman Game with Illustration
:red_circle: Aim : It helps to increase the general knowledge by guessing answers of riddles.
:red_circle: Tech Stack Used : HTML, CSS, JS.
:red_circle: Category :  Vanilla-JS-Projects
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Damini Chachane
GitHub Profile Link :  https://github.com/Damini2004
What is your participant role? GSSOC24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Kindly assign this issue to me under GSSOC24 with suitable level.

@Damini2004 in Issue creation guidelines which is pinned above all the issues, it is mentioned more than 1 issue is not allowed and no games are allowed in this repository.
For further communication, join discord server https://discord.gg/tSqtvHUJzE


Todo tasks

Hi, I'm interested in contributing to the project, I'm new to open source development and know my way around NodeJS. I'd like to jump in and do some work to complete the wrapper.
Would it be possible to put some todo's on the issues list so I can start contributing?
Thanks!

I'm glad you want to contribute! I should have the main functionality done in the next day or two. I would love help with the documentation or tests. I started on the documentation in the readme to give you an idea if you want to work on that.


make the global default sendAsJson configurable

We prefer to be explicit rather than implicit, and we'd like for that to be enforced globally. Could you make sendAsJson globally configurable so we could set it to false?

I'll happily accept a Pull request for this. Ideally by means of a configure() method that accepts an options object so it's extensible if need be in future

Addressed in v5 https://github.com/wheresrhys/fetch-mock/pull/110


Bug or hidden feature - overwriting model.Blog.Title with post title?

In line 80 of the BlogController there is this line:
model.Blog.Title = model.Post.Title;

https://github.com/blogifierdotnet/Blogifier/blob/d930cece91c9dc95b62dd3d57e2d7e8d53d37c06/src/App/Controllers/BlogController.cs#L80
Any reason?

I figured that out when I saw it was being overwritten multiple times. I would go the other way.
My scenario was ... main blog title being used in the "main menu" to be displayed on all pages.
So I have changed this for my blog ... used BlogEngine before so this was nice to finally get it upgraded :-)


IndexError: list index out of range

Hi guys,
Is this message familiar?
STEP 4: Define strain-specific gene-families presence/absence (1,-1,-2,-3 matrix, option --o_idx)
[W] No DNA 1,2,3 index file has been written because no strain was detected.
STEP 5: Get presence/absence of gene-families (1,-1 matrix, option --o_matrix)
Traceback (most recent call last):
File "/shared/homes/12705859/miniconda3/envs/panphlan_env/bin/panphlan_profiling.py", line 763, in
main()
File "/shared/homes/12705859/miniconda3/envs/panphlan_env/bin/panphlan_profiling.py", line 738, in main
sample2family2presence = get_genefamily_presence_absence(sample2family2dnaidx, sample_stats, avg_genome_length, args)
File "/shared/homes/12705859/miniconda3/envs/panphlan_env/bin/panphlan_profiling.py", line 574, in get_genefamily_presence_absence
families = sample2family2dnaidx[dna_samples[0]].keys()
IndexError: list index out of range
my command was:
panphlan_profiling.py -p /shared/homes/12705859/panphlan/Blautia_wexlerae/Blautia_wexlerae_pangenome.tsv -i map/ --o_matrix ./matrix_out/profile_Blautia_wexlerae --min_coverage 1 --left_max 1.70 --right_min 0.30
I have run panphlan_profiling.py before and never had this problem. Only difference is the way I downloaded the pangenomes (via panphlan_download earlier, while now via browser). Mapping worked fine. Files have content (from the size, over 20Mb each), which is why I wonder why no strain was detected
Thank you
Dany

Hello,
The warning in step 4 tells you that no sample in your input passed the thresholds you provided (--min_coverage 1 --left_max 1.70 --right_min 0.30 ). That does not specifically mean that you species is absent in the sample, but either that PanPhlAn limits are kind of reached. You could lower again the coverage thresholds (like --min_coverage 0.9 --left_max 2 --right_min 0.10 for example), but then your profiling results should be analyzed with care.
Also, keep in mind that depending on the species and the sample, a significant part could not pass the PanPhlAn analysis. I sometimes have more than half of the samples mapped that do not pass profiling.
Out of curiosity how many samples do you have in your input folder ?

Hi!
Yes I imagined that was the case... over 500 input samples (metagenomic).
Do you think the fact that I used single reads (only forward reads) instead
of concatenated forward and reverse (as was suggested) could have impacted
this substantially?
On Sat, Oct 3, 2020, 00:14 Léonard Dubois<EMAIL_ADDRESS>wrote:

Hello,
The warning in step 4 tells you that no sample in your input passed the
thresholds you provided (--min_coverage 1 --left_max 1.70 --right_min 0.30
). That does not specifically mean that you species is absent in the
sample, but either that PanPhlAn limits are kind of reached. You could
lower again the coverage thresholds (like --min_coverage 0.9 --left_max 2
--right_min 0.10 for example), but then your profiling results should be
analyzed with care.
Also, keep in mind that depending on the species and the sample, a
significant part could not pass the PanPhlAn analysis. I sometimes have
more than half of the samples mapped that do not pass profiling.
Out of curiosity how many samples do you have in your input folder ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SegataLab/panphlan/issues/16#issuecomment-702758620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTFBBD5I5RU6EIBDGF3RKDSIXN5XANCNFSM4SBUEJVA
.

Hi !
Yes indeed, it's better to concatenate all the reads. Otherwise it could prevent a sample to pass the min_coverage threshold thus discarding the sample


Not being able to view remote image url via iTerm2's imgcat

Hi guys,
In the default mailcap file, there is a pre-defined command to view remote image url via iTerm2's imgcat as such image/*; bash -c '[[ "%s" == http*  ]] && (curl -s %s | ~/.iterm2/imgcat) || ~/.iterm2/imgcat %s' && read -n 1;     needsterminal. Does anyone who has imgcat installed be able to use this command to display remote image url?

Yes, it works for me.

@eyemyth  did you do any configuration to make it work?

It goes to a blank page in your browser?

@eyemyth blank page as in my terminal view. the imgcat process should have displayed it in-line if it's working correctly. Yours is being displayed in the terminal view or browser?

Yep, in the terminal.
I'd start by adding some logging to rtv or imgcat or both.
By the way, this project is dead. There's a fork that's actively maintained: https://gitlab.com/ajak/tuir

Thanks for the pointer to the new fork. I'll keep digging to figure out why my config is not displaying image. Thanks.


Cannot copy or view ordinals receive address (copied text is 'undefined')

When I copy my ordinals receive address, the clipboard text says 'undefined'. When I copy the Bitcoin Stamp address, it works. The Receive modal only shows the partial taproot address. I uninstalled and re-installed the Hiro wallet extension with the same issue. Brave on macOS

This was fixed


Preview11: RouteId change in the routes config model is not breaking

Describe the bug
I updated my application to the latest version 1.0.0-preview.11.21223.5. I tried running it without making any changes to the routes config model and I was surprised to see that everything was working. It turns out that the RouteId defined in my configuration is not used but the index of the array will now be my RouteId
To Reproduce
Use the old config:
{
  "RouteId": "route1",
  "ClusterId": "cluster1",
  "CorsPolicy": "default",
  "Match": {
    "Path": "/route1/{**catch-all}"
  },
  "Transforms": [
    { "PathRemovePrefix": "/route1" }
  ]
},

This will result in something similar to:

Further technical details
I would've expected a warning or error telling me that I am missing the correct RouteId as this could cause subtle bugs if your application is dependant on the RouteId

Include the version of the packages you are using 1.0.0-preview.11.21223.5
The platform (Windows)

Thanks for reporting this, definitely wasn't intended.
This issue applies to clusters & destinations as well.
@Tratcher can correct me here, but from what I understand, configuration is effectively flattened to a list like
{
    "Clusters:myCluster1:Destinations:destinationA:Address": "foo",
    "Clusters:myCluster2:Destinations:destinationB:Address": "bar",

    "Clusters:0:Destinations:destinationA:Address": "foo",
    "Clusters:1:Destinations:destinationB:Address": "bar",
}

So the IConfiguration abstraction can't distinguish between an array and a dictionary with keys like "0", "1", ....
Short of not accepting IDs that are also integers (or specifically banning "0" as being used), I'm not sure we can detect this.

I hadn't anticipated it would fail quite like that. @MihaZupan's assessment looks accurate.  Luckily the route id isn't used for much so it should work as is for most people.
We don't usually add mitigations for breaking changes between previews, only release notes. Since the failure is not obvious in this case we may want to add a temporary check that the obsolete "RouteId" label is no longer present on a route config object.
Here we could throw if section["RouteId"] returns a non-empty value:
https://github.com/microsoft/reverse-proxy/blob/bc0399b08a6f511ed638c690a2636ca7e3b4c023/src/ReverseProxy/Configuration/ConfigurationConfigProvider.cs#L138-L142

Clearing the milestone for triage review.

Triage: Take the fix suggested by Tratcher in https://github.com/microsoft/reverse-proxy/issues/1029#issuecomment-849843294