text:
rpm: Package SELinux files

Signed-off-by: Boris Ranto branto@redhat.com

This looks OK to me. @ktdreyer can you take a look?

I've added a second commit that creates the wrapper scripts for gunicorn and celery.
@ktdreyer any other changes that you see being missed?

Thanks, I will run this through a smoke test install in "permissive" today and report back

The smoke test uncovered a couple issues, see https://github.com/b-ranto/ceph-installer/pull/1 for some of the fixes.
There were still many AVC denials when I used the API to do an install. Here's the output from audit2allow -a -M installer && cat installer.te
module installer 1.0;

require {
	type cert_t;
	type ssh_exec_t;
	type kernel_t;
	type ssh_port_t;
	type fs_t;
	type unreserved_port_t;
	type devpts_t;
	type ceph_installer_t;
	type krb5_conf_t;
	type ceph_installer_var_lib_t;
	type tmp_t;
	type devlog_t;
	type passwd_file_t;
	type ptmx_t;
	type net_conf_t;
	type file_context_t;
	type security_t;
	class key { write read view };
	class unix_stream_socket connectto;
	class chr_file { write ioctl read open getattr };
	class tcp_socket { name_bind name_connect };
	class file { write execute read open getattr execute_no_trans };
	class filesystem getattr;
	class sock_file { write link create unlink };
	class security check_context;
	class unix_dgram_socket { create connect sendto };
	class dir { search create rmdir };
}

#============= ceph_installer_t ==============
allow ceph_installer_t ceph_installer_var_lib_t:sock_file { write create unlink link };
allow ceph_installer_t cert_t:file { read getattr open };
allow ceph_installer_t devlog_t:sock_file write;

#!!!! This avc can be allowed using the boolean 'daemons_use_tty'
allow ceph_installer_t devpts_t:chr_file open;
allow ceph_installer_t devpts_t:filesystem getattr;
allow ceph_installer_t file_context_t:dir search;
allow ceph_installer_t file_context_t:file { read getattr open };
allow ceph_installer_t fs_t:filesystem getattr;
allow ceph_installer_t kernel_t:unix_dgram_socket sendto;
allow ceph_installer_t krb5_conf_t:file { read getattr open };
allow ceph_installer_t net_conf_t:file { read getattr open };
allow ceph_installer_t passwd_file_t:file { read getattr open };
allow ceph_installer_t ptmx_t:chr_file { read write ioctl open getattr };
allow ceph_installer_t security_t:file write;
allow ceph_installer_t security_t:security check_context;
allow ceph_installer_t self:key { write read view };
allow ceph_installer_t self:unix_dgram_socket { create connect };

#!!!! This avc can be allowed using the boolean 'daemons_enable_cluster_mode'
allow ceph_installer_t self:unix_stream_socket connectto;
allow ceph_installer_t ssh_exec_t:file { read execute open execute_no_trans };
allow ceph_installer_t ssh_port_t:tcp_socket name_connect;
allow ceph_installer_t tmp_t:dir { create rmdir };
allow ceph_installer_t tmp_t:sock_file { write create unlink };
allow ceph_installer_t unreserved_port_t:tcp_socket name_bind;

Want to incorporate these into ceph-installer.te?

Still need the ceph_installer_t changes above

Yep, I wanted to review them first. They should be all in, now.

Thanks! I will try another smoke-test today with these changes.

After running through a cluster set up, audit2allow -a -w is clean with this change :) Thanks @b-ranto

authors: alfredodeza b-ranto ktdreyer

repo: ceph/ceph-installer


text:
Do we really need ceph-salt grains?

(Relying on the pillar is more secure because only the master node can set it)
For targetting, we can use pillar:
salt -I ceph-salt:minions state.apply <state>

instead of grain:
salt -G ceph-salt:member state.apply <state>

So, apart from ceph-salt:execution grains that are set during execution for orchestration, do we really need any other ceph-salt grains?
(AFAIK, Suse Manager doesn't support grains)

I think we still need grains when we want to target minion by role, for instance:
salt -G ceph-salt:roles:admin test.ping

source: gharchive/issue

repo: ceph/ceph-salt

license: MIT

license_type: permissive

license_source: github-api


text:
Need help creating a compound partition key

I want to do this
CREATE TABLE pushes (
  owner_id int,
  push_month int,
  pushed_at timestamp,
  values map<text, text>,
  PRIMARY KEY ((owner_id, push_month), pushed_at)
) WITH CLUSTERING ORDER BY (pushed_at DESC);

but am having issues creating the compound partition key (owner_id, push_month), here's what I have- currently only owner_id is the partition key. I don't want to use belongs_to because Owner is actually a postgres ActiveRecord model.
class Push
  include Cequel::Record
  key :owner_id, :int
  key :push_month, :int
  key :pushed_at, :timestamp, order: :desc, index: true
  map :values, :text, :text
...

Can anyone help? Thanks

By default only the first key is added to the partition key, but you can override this behavior with the :partition option:
key :owner_id, :int
  key :push_month, :int, partition: true
  key :pushed_at, :timestamp, order: :desc, index: true
  map :values, :text, :text

Feel free to reopen if that doesn't clear things up!

authors: johnrees outoftime

repo: cequel/cequel

license: mit

license_source: bigquery


text:
fix: Kafka async produce context cancellation

Description
Fixes the context canceled audit log message when producing via the async mechanism.
I've first committed some integration tests to show sync working, followed by async failing. The second commit then shows that async test is passing.
I've used testcontainers-go so that go test can spin up a container to integrate with. Hopefully, that approach is ok with you. Normal test caching will mean these slightly slower tests will not run when they don't need to, so I've opted not to use a build tag (to optionally run these on CI).
Checklist

[x] The PR title has the correct prefix
[ ] ~PR is linked to the corresponding issue~
[x] All commits are signed-off (git commit -s ...) to provide the DCO

Failing test to show it not working as expected, as observed in our dev environments while we've been testing Kafka audit support when running with async producer...
https://github.com/cerbos/cerbos/actions/runs/4611974520/jobs/8152336438?pr=1516#step:5:57

source: gharchive/pull-request

repo: cerbos/cerbos


text:
Enhance Card On Card for better Box Shadow

Issue Description
Enhance Card to allow for an alternative box shadow when on a white background (like when in another card)
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
The card should be flexible enough to allow for a different box shadow when nested inside another card.
Current Behavior
Terra core has a css variable for box-shadow set to none, but themes may override this to have a box shadow. The ask is for the ability to also flex it to be different when placed inside another card. This is so that a different box shadow can be applied for a better visual affect.
Possibilities
A new Prop on Card
One possible way to do this is to just add a type/variant on the card that allows it to change some visuals based off that type. In this case it might be a stacked card.
A new Card Component
In order to prevent the existing card from having its props balloon out with different specific scenarios, a simple new StackedCard could be created that utilizes the existing card and overrides the box shadow.
Smart Card Body Approach
If the rule is that "A card inside a card has a different box shadow" then 1 possibility is to make the card put a class on it that scopes a css variable inside it to change the card box shadow. This would not work in ie though, so its probably a no go. Another way of doing something similar is via a descendent selector to look for another Card inside Card and to change the box shadow. This would work in IE but comes with alot of assumptions that would inevitably cause an edge case. If target only direct children, it wouldn't work if anyone puts a div or spacer around their inner card. If you target all descendents, someone might put a container inside their card with a none-white background, and then another card in that expecting the normal box shadow.
Separate the box shadow concern from the Card itself
Create a wrapper component or HOC that can be used to add a box shadow on the outer most element. This would be useful to not couple specific box shadows to card, and to allow them to be used with other components. Would only apply to the outer element of a component however. After speaking with UX a bit more about this, their guidance is that the 6 levels of box shadow wouldn't need to be applied randomly to other components, but is already encapsulated by existing components.
Cards on Light Backgrounds
So what if a card is on the background, and a client themes their background to white. After discussing with UX, this scenario shouldn't be handled by the Card trying to be smart and flex. In this case, the client would have to theme the card box shadow appropriately.
My Vote
I think the new prop on card makes the most sense for now. There are already no props on Card (other than children) so we aren't making it super custom. The new component seems a bit overkill and would require more refactoring of the internals to share code
New React Prop:

Prop
Type
Default
Description

TBD
boolean
false
Sets a class that overrides the default box shadow css property with a new value and a new css custom property

To keep it passive, the default would be false. Additionally, since terra currently doesn't have a box shadow for the card at all, I'm assuming even when true, the default value for the css custom property would still be no box shadow. Would need UX input
Proposed Names - We don't want

stacked - isn't right, since there isn't a card stack
nested - Doesn't always have to be nested
isInsideCard - ins't right, since it is for any card used on any white background,
isRaised - Doesn't seem right since we aren't truly changing the elevation, just changing the way the light hits it and makes a shadow
hasBoxShadow - Tied directly to the specific style it adds, not its usage. Also normal cards can already have box shadows

Proposed Names - That we are open to using

isOnLightBg
enbeddedCard
cardStyle/variant - A bit vague and non-prescriptive on how to use it, but not tied to any particular use case and open for themes to use it differently. Default would be baseLine, new one would be 1raised1, and in the future it could expand for other scenarios.

+1 for a new prop on card.

naming is fun as well. stacked or nested would be useful for signifying its mainly for when you have a card in a card. Or even isInsideCard.
If there are additional use cases to use a different box shadow when on other white backgrounds, we might not want to couple it to being inside a card. The different box shadows UX are using are related to different elevations for visual effect, so isRaised could work.

+1 on prop on card.
I took a look at the visuals and since the usage of this alternate card shadow has nothing tied to specific card-inside-of-card usage, or a card-stacked-ontop-of-another-card, really is just an alternate shadow for when used on white background.

isStacked isn't right, since there isn't a card stack,
isInsideCard ins't right, since it is for any card used on any white background,
isRaised doesn't seem right since it would imply changing to another elevation shadow depth within the styleguide (there are different shadow depths similar to material design and this isn't one of those depths)

Feel like the new prop should be maybe cardStyle or variant that should take a list of string options: style1, style2, etc.

I think cardStyle with style1,  style2, etc is a bit vague and doesn't really describe what its used for. It would end up being used inappropriately. The variant is another thought, but this is the only other way to use it I have heard from UX, so I'm not sure what the values would be.
what about a boolean prop not signifying that its in a card, but that its on a light background? isOnLightBg or something of the sort?

Possibly hasBoxShadow?

One thing to consider is when adding a box-shadow around the card, the parent div will need to add the appropriate spacing such that the box-shadow is not cut off.

enbeddedCard?

I think that works for the Card In Card scenario, but @neilpfeiffer was saying it should work for other situations where its just on white. I'll add it to the list at the top to keep a record of it. Thanks for helping come up with names btw!!

agreed that style1 style2 was too vague, idea was more placeholder for a better enum list we could base on solidified concepts. (Let's keep the naming open for a bit longer until something makes sense.)
.

not typically putting margins/padding on the outside of components

Generally the rule we've discussed it that complex containers & views do margin appropriately outside of themselves, such they position themselves as intended within any full-width (& height as needed) space, including other containers, but don't add padding to the inside, unless it is a special type of container that can receive "basic" content as children (pure strings, etc), thus functioning like a pseudo-view (card-body and table-cell as examples). TLDR: any child should always be able to touch/reach the edge of it's parent if it needs to.
For this case of the current designs trying to be achieved (as with cards of any type) - I would assume consumer would wrap the component in a spacer or other custom layout (grid, card-collection, etc) that appropriately keeps the inner cards correctly aligned and separated from one another, and inset correctly from the outer card/page-panel.

Worried on the other end that isOnLightBg is also too specific, since if we don't look at directly at the new design being for on-white and abstract it for other themes, is really just an alternate style.
I'm still open to using cardStyle with baseline as default, and raised for new type, then list can passively grow as needed in future with raised+1, raised+2 for other elevations if needed, or flip,reveal, etc. if other functional types of cards might be created.

talked in person a bit, I think we are both fine going with a prop of cardStyle the default prop value being default to provide passivity, and the new type being raised.
@bjankord @emilyrohrbough do you have any additional thoughts?

I'd so go with variant instead of cardStyle. variant matches other terra components and cardStyle could be confused with a prop to pass inline styles to in my opinion. +1 on default and raised

if we want to go with variant for consistency that's fine. I think @neilpfeiffer was arguing that variant has an undertone of being an actual different type of component (like a utility button vs emphasis) where this is just a different style.

@bjankord I opened a PR here https://github.com/cerner/terra-core/pull/1458 using variant.
@neilpfeiffer let me know if you feel strongly about the prop being called cardStyle instead

+1 on variant. As other use-cases for cards pop up, this will allow us to easily divide component/styling properties passively. Although I agree that right now, the only difference being controlled through the variant prop is style toggling, so cardStyle would make sense, I don't know if that will necessarily be the case as develop new use-cases and contexts where card can be consumed

+1 for a new prop on card.

JIRA created

@alex-bezek This has been released in terra-card@2.8.0

sweet, thanks!

authors: PaulSolDev alex-bezek bjankord dylan-sftwr emilyrohrbough neilpfeiffer vaghashiyamukesh

repo: cerner/terra-core


text:
quay.io/jetstack/cert-manager not public

Hi community, I'm trying to install cert-manager from a Helm chart but looks like the image quay.io/jetstack/cert-manager is not public so the image pull failed.
Is this intentional? To reproduce, simply run docker pull quay.io/jetstack/cert-manager:v1.6.1

looks like the image quay.io/jetstack/cert-manager is not public

There isn't a quay.io/jetstack/cert-manager image I think. For the controller (main cert-manager component) it should be quay.io/jetstack/cert-manager-controller
docker pull quay.io/jetstack/cert-manager-controller:v1.6.1 works for me.

Ah I see, thank you for explaining :)
So it seems that quay.io would return the code of 403 for inexistent repos, and the Helm chart I used misconfigured the image name :(

authors: irbekrm lorqor

repo: cert-manager/cert-manager


text:
WIP: Deduplicate code for syncing configmaps and secrets

Currently, the code for syncing target configmaps and secrets is separate - even if the logic should be the same. This doesn't make it tempting to add new features to bundle targets. In this PR I made an attempt to duplicate some of the code. We could probably do more, but let me know what you think before I spend more time on this! ☺

/test all

/cc @inteon

Thanks @erikgb. I would like you to continue making improvements, let's do it in separate PRs.
/approve
/lgtm

authors: erikgb inteon

repo: cert-manager/trust-manager


text:
automatically download JNI jars

JOGL seems to work fine in Ceylon if I do the following things:

enable --flat-classpath, and
manually download the right platform-specific JNI -natives- jar from this maven repo and place it in the right location in my .m2 folder.

It would be nice if the additional artifacts were downloaded automatically, or if there were some way to explicitly install them.
We need to think this through.

I'd say automatically, if possible.

@xkr47 oh definitely, but we haven't found out yet why Aether (the API we use to support Maven) doesn't download them in the first place. Documentation on the subject is sorely lacking.

Perhaps it's time to write a Maven library in Ceylon ;p

Oh I think we definitely don't want to do that.
People should have fun writing Ceylon code ;)

Maven just might be fun in Ceylon..... :lollipop: :grin:

@xkr47 congratulations, you were officially assigned to the task "Rewrite Aether in Ceylon" 🎉

No but seriously, the algorithm needs to be exactly the same as in maven, including any bugs that might have slipped into the dependency resolution. And when that changes, we want the changes too.

I got the same issue with lwjgl, but having the -natives- jar downloaded was not enough, so I did a ceylon import-jar org.lwjgl.lwjgl-natives/3.1.0 ~/.m2/repository/org/lwjgl/lwjgl/3.1.0/lwjgl-3.1.0-natives-linux.jar and imported org.lwjgl.lwjgl-natives manually. JOGL might be doing something additional where lwjgl expects developers to use maven-nativedependencies-plugin.

I've run into this again with LWJGL. It seems like all I really need here is some support for Maven classifiers.
Looks like what I need to do is reproduce this POM:
<properties>
	<maven.compiler.source>1.8</maven.compiler.source>
	<maven.compiler.target>1.8</maven.compiler.target>
	<lwjgl.version>3.1.2</lwjgl.version>
</properties>

<dependencies>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl</artifactId>
		<version>${lwjgl.version}</version>
	</dependency>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl-glfw</artifactId>
		<version>${lwjgl.version}</version>
	</dependency>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl-opengl</artifactId>
		<version>${lwjgl.version}</version>
	</dependency>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl</artifactId>
		<version>${lwjgl.version}</version>
		<classifier>${lwjgl.natives}</classifier>
		<scope>runtime</scope>
	</dependency>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl-glfw</artifactId>
		<version>${lwjgl.version}</version>
		<classifier>${lwjgl.natives}</classifier>
		<scope>runtime</scope>
	</dependency>
	<dependency>
		<groupId>org.lwjgl</groupId>
		<artifactId>lwjgl-opengl</artifactId>
		<version>${lwjgl.version}</version>
		<classifier>${lwjgl.natives}</classifier>
		<scope>runtime</scope>
	</dependency>
</dependencies>

<profiles>
	<profile>
		<id>lwjgl-natives-linux</id>
		<activation>
			<os><family>unix</family></os>
		</activation>
		<properties>
			<lwjgl.natives>natives-linux</lwjgl.natives>
		</properties>
	</profile>
	<profile>
		<id>lwjgl-natives-macos</id>
		<activation>
			<os><family>mac</family></os>
		</activation>
		<properties>
			<lwjgl.natives>natives-macos</lwjgl.natives>
		</properties>
	</profile>
	<profile>
		<id>lwjgl-natives-windows</id>
		<activation>
			<os><family>windows</family></os>
		</activation>
		<properties>
			<lwjgl.natives>natives-windows</lwjgl.natives>
		</properties>
	</profile>
</profiles>

@marvelous, can you give me a bit more detail on how you worked around this?

@FroMage @quintesse what is the most-minimal, stupidest sort of support for classifiers that we could add?

@gavinking
I worked around the issue with the following imports:
import maven:"org.lwjgl:lwjgl" "3.1.0";
import maven:"org.lwjgl:lwjgl-vulkan" "3.1.0";
import maven:"org.lwjgl:lwjgl-glfw" "3.1.0";
import "org.lwjgl.lwjgl-natives" "3.1.0";
import "org.lwjgl.lwjgl-glfw-natives" "3.1.0";

The org.lwjgl.lwjgl-natives and org.lwjgl.lwjgl-glfw-natives modules are created with ceylon import-jar.

Ah, cool, I did not know about that ... so that implies that somewhere, under the covers, perhaps, we do have some sort of support for classifiers.

Try it and let me know how it works?

I don't think it would work in this case because I need two artifacts: both the one without the classifier, and the one with the classifier.
@FroMage if I added support for something like the following syntax to the typechecker, how hard would it be to somehow map it to the CMR backend:
import maven:org.lwjgl:"lwjgl":"natives-macos" "1.3.2";

@FroMage: I had tried to use overrides to get the correct classifier, but couldn't make it work because we need to depend on two modules with the same artifactId (lwjgl both without classifier and with the native platform classifier).

The org.lwjgl.lwjgl-natives and org.lwjgl.lwjgl-glfw-natives modules are created with ceylon import-jar.

OK, thanks. But what precisely did you put in the -natives modules directory? As in what artifacts and what does the module.xml look like precisely.

but couldn't make it work because we need to depend on two modules with the same artifactId

Yes, precisely. But it seems to me that it should be possible to treat org.lwjgl:lwjgl and org.lwjgl:lwjgl:natives-macos as two separate modules, if the language itself allowed one to specify more than one "artifact identifier" in the grammar, i.e.
ModuleSpecifier: Repository Module Artifact*

ceylon import-jar org.lwjgl.lwjgl-natives/3.1.0 ~/.m2/repository/org/lwjgl/lwjgl/3.1.0/lwjgl-3.1.0-natives-linux.jar
ceylon import-jar org.lwjgl.lwjgl-glfw-natives/3.1.0 ~/.m2/repository/org/lwjgl/lwjgl-glfw/3.1.0/lwjgl-glfw-3.1.0-natives-linux.jar
I didn't see any module.xml file.

it should be possible to treat org.lwjgl:lwjgl and org.lwjgl:lwjgl:natives-macos as two separate modules

Of course that leaves the problem of how to adjust the classifier based on the OS, but at worst you could do it using overrides.xml. Anyway let's cross that bridge when we come to it.

@marvelous yeah but I'm not sure what you have in the directory org.lwjgl.lwjgl-natives/3.1.0.

Just

modules/org/lwjgl/lwjgl-natives/3.1.0/org.lwjgl.lwjgl-natives-3.1.0.jar.sha1
modules/org/lwjgl/lwjgl-natives/3.1.0/org.lwjgl.lwjgl-natives-3.1.0.jar
modules/org/lwjgl/lwjgl-glfw-natives/3.1.0/org.lwjgl.lwjgl-glfw-natives-3.1.0.jar
modules/org/lwjgl/lwjgl-glfw-natives/3.1.0/org.lwjgl.lwjgl-glfw-natives-3.1.0.jar.sha1

ok, thanks.

if I added support for something like the following syntax to the typechecker, how hard would it be to somehow map it to the CMR backend?

Several days, for even just basic hack support. We're still finding bugs with the classifier now. It was quite some work to add support for groupId/artifactId.

OK.

@FroMage note that I would treat it pretty much just like a generalization of the already-existing artifactId.

@marvelous OK, you're right, it works fine with adding just the jars to a module repo.
But now I have a different problem, did you ever run into this:
GLFactory.static - Platform: Mac OS X - not available: com.sun.prism.es2.MacGLFactory
java.lang.IllegalAccessException: Class com.sun.prism.es2.GLFactory$FactoryLoader can not access a member of class com.sun.prism.es2.MacGLFactory with modifiers ""
	at sun.reflect.Reflection.ensureMemberAccess(Reflection.java:102)
	at java.lang.Class.newInstance(Class.java:436)
	at com.sun.prism.es2.GLFactory$FactoryLoader.run(GLFactory.java:108)
	at com.sun.prism.es2.GLFactory$FactoryLoader.run(GLFactory.java:100)
	at java.security.AccessController.doPrivileged(Native Method)
	at com.sun.prism.es2.GLFactory.<clinit>(GLFactory.java:97)
	at cuchaz.jfxgl.JFXGL.start(JFXGL.java:88)
	at eg.jfxgl.Main.jfxglmain(run.ceylon:67)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at cuchaz.jfxgl.JFXGLLauncher.launchMain(JFXGLLauncher.java:123)
	at eg.jfxgl.run_.run(run.ceylon:37)
	at eg.jfxgl.run_.main(run.ceylon)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)

Oh, wait, I guess that problem is not related to LWJGL, but to JFXGL. Oops, my bad.

Yeah, sorry about the missing Mac files. That version of JFXGL wasn't compiled correctly for cross-platform use. That bug should be fixed in the secret 0.4-SNAPSHOT version.

That bug should be fixed in the secret 0.4-SNAPSHOT version.

Yep, already figured that out.

On branch 6638 I have added support for Maven classifiers. You can see how this helps solve the issue but looking at the example project here.

Likewise, this branch of the JOGL example project shows that the solution works with JOGL too.

I'm pretty much ready to declare victory on this issue:

adding support for Maven classifiers in module import statements does indeed seem to be a very reasonable solution to the problem here, and has minimal impact on the codebase
Tools including IntelliJ, ceylon fat-jar, ceylon assemble all seem to handle this new feature elegantly without a problem.

There are a couple of minor loose ends, however:

update the spec
update ceylon.ast (cc @lucaswerkmeister)
allow you to address a module with a classifier in overrides.xml

allow you to address a module with a classifier in overrides.xml

Oh, wait, looks like this is already done.

@gavinking thanks for the heads-up! Do you plan to add a spec for this before the next release or will it remain unspecified for now? (I’m fine either way, I just don’t want to have to update the ceylon.ast implementation within the same release cycle after a spec is added.)
Also, I assume ceylon.formatter also needs to be updated, unless you tested it and it works?

I'll probably spec it now. It's extremely trivial.
I did not test the formatter.

allow you to address a module with a classifier in overrides.xml

Oh, wait, looks like this is already done.

Well, no: the overrides.xml file format has the ability to specify a classifier, but it seems to be pretty much ignored by the actual code.
I can't find any way at all to change the classifier of a module in overrides.xml. I can't even seem to just add a new dependency with a classifier. In principle, this should work, it seems to me:
    <add groupId="org.jogamp.jogl"
         artifactId="jogl-all"
         classifier="natives-macosx-universal"
         version="2.3.2"/>
    <add groupId="org.jogamp.gluegen"
         artifactId="gluegen-rt"
         classifier="natives-macosx-universal"
         version="2.3.2"/>

But it has no effect.

Nor does this work, FTR:
    <artifact groupId="org.jogamp.jogl"
              artifactId="jogl-all"
              version="2.3.2">
        <add groupId="org.jogamp.jogl"
             artifactId="jogl-all"
             classifier="natives-macosx-universal"
             version="2.3.2"/>
    </artifact>
    <artifact groupId="org.jogamp.gluegen"
         artifactId="gluegen-rt"
         version="2.3.2">
        <add groupId="org.jogamp.gluegen"
             artifactId="gluegen-rt"
             classifier="natives-macosx-universal"
             version="2.3.2"/>
    </artifact>

Alright, I have got it sorta-working now. At least I can now add native dependencies using overrides.xml, as follows:
<overrides
        xmlns="http://www.ceylon-lang.org/xsd/overrides">
    <define name="version" value="2.3.2"/>
    <define name="mac" value="natives-macosx-universal"/>
    <add groupId="org.jogamp.jogl"
         artifactId="jogl-all"
         version="${version}"
         classifier="${mac}"/>
    <add groupId="org.jogamp.gluegen"
         artifactId="gluegen-rt"
         version="${version}"
         classifier="${mac}"/>
</overrides>

Which is great.

All done and merged. @lucaswerkmeister has opened additional issues for stuff on his side.

Formatter support done. But, honestly…
module look a:"legal":"Ceylon":"module" "descriptor" {
    import and:"a":"legal":"module" "import";
}
// “legal” in terms of the grammar

This just looks like a total mess to me. How many more components with undefined semantics are we going to add? Can’t we just say that there’s an arbitrary number of colon-separated lidentifiers-or-strings and be done with it?

Can’t we just say that there’s an arbitrary number of colon-separated lidentifiers-or-strings and be done with it?

Well, I think that ultimately the grammar should be just:
"import" (namespace ":")? packageName (":" string)* ";"

But it was a smaller change to do it this way.
If you really insist, I could do this now as a separate change.

Do you mean spec grammar or also compiler grammar?
(I’m not really insisting… there are probably other issues more worth your time. Perhaps do it the next time the module grammar is touched.)

authors: FroMage bjansen cuchaz gavinking lucaswerkmeister marvelous quintesse xkr47

repo: ceylon/ceylon


text:
Removed related posts from sidebar of imported Newsroom items

After this gets merged in the import-data script needs to be run for these pages with the overwrite flag.
@kave @rosskarchner

verified related posts are gone

authors: kave kurtw

repo: cfpb/cfgov-refresh


text:
Minor update to testing readme

Updates typo and adds more instructions for brew to the testing section
of the readme.
Additions

Adds instructions for running brew info to find chromedriver installation path.

Changes

Updates typo in somewhere text.

Review

@sebworks
@jimmynotjim

:+1:

authors: anselmbradford sebworks


text:
Redirects don’t trigger interceptors

Package
cookie_manager
Version
2.1.1
Output of flutter doctor -v
No response
Dart Version
No response
Steps to Reproduce

final dio = Dio();
final dir = await getApplicationSupportDirectory();
final cookieJar = PersistCookieJar(storage: FileStorage(join(dir.path, "cookies")));
dio.interceptors.add(CookieManager(cookieJar));
dio.get("http://site1") ; // This is a 302 redirect url. The redirected site2 url will return the set-cookie response header

Expected Result
The cookie will be saved to site2
Actual Result
The cookie is saved to the original url, ie site1, resulting in no cookie being carried in subsequent requests.
So I checked the source code cookie_mgr.dart, Should response.requestOptions.uri be changed to response.realUri?
  Future<void> _saveCookies(Response response) async {
    final setCookies = response.headers[HttpHeaders.setCookieHeader];

    if (setCookies != null) {
      final cookies = setCookies
          .map((str) => str.split(_setCookieReg))
          .expand((element) => element);
      await cookieJar.saveFromResponse(
        response.requestOptions.uri, // Should this be changed to response.realUri ?
        cookies.map((str) => Cookie.fromSetCookieValue(str)).toList(),
      );
    }
  }

This seems to be caused by the fact that the interceptor cannot intercept the redirection. Is this a BUG?

This is not a bug. Redirects don‘t trigger interceptors since redirects are handled by the underlying APIs dio is making use of.
We should document this probably better since I feel like I’ve seen this question popular multiple times.

The inspector is working as intended, but we need to evaluate if the cookie manager is behaving normally.

This is not a bug. Redirects don‘t trigger interceptors since redirects are handled by the underlying APIs dio is making use of.
We should document this probably better since I feel like I’ve seen this question popular multiple times.

So, should dio handle redirection instead of handing it to the underlying api?

So, should dio handle redirection instead of handing it to the underlying api?

While it seems to be possible, I don't think it should, since that creates a lot of complexity. What we could offer is to allow the user to handle redirects himself.

My temporary solution is
      dio.setFollowRedirects(false);
      dio.interceptors.add(RedirectInterceptor(dio2));

extension DioExt on Dio {
  Dio setFollowRedirects(bool followRedirects) {
    options.followRedirects = followRedirects;
    if (followRedirects) {
      options.validateStatus =
          (int? status) => status != null && status >= 200 && status < 300;
    } else {
      options.validateStatus =
          (int? status) => status != null && status >= 200 && status < 400;
    }
    return this;
  }
}

class RedirectInterceptor extends Interceptor {
  final Dio dio;

  RedirectInterceptor(this.dio);

  @override
  Future<void> onResponse(
      Response response, ResponseInterceptorHandler handler) async {
    if (_isRedirect(response.statusCode ?? 0)) {
      final location = response.headers.value('location');
      if (location == null) throw Exception("location is null");
      final requestOptions = response.requestOptions;
      final rawUri = requestOptions.uri.toString();

      final redirectResponse = await dio.get(
        parseHttpLocation(rawUri, location),
        options: Options(
          sendTimeout: requestOptions.sendTimeout,
          receiveTimeout: requestOptions.receiveTimeout,
          extra: requestOptions.extra,
          headers: requestOptions.headers,
          responseType: requestOptions.responseType,
          contentType: requestOptions.contentType,
          validateStatus: requestOptions.validateStatus,
          receiveDataWhenStatusError: requestOptions.receiveDataWhenStatusError,
          followRedirects: requestOptions.followRedirects,
          maxRedirects: requestOptions.maxRedirects,
          persistentConnection: requestOptions.persistentConnection,
          requestEncoder: requestOptions.requestEncoder,
          responseDecoder: requestOptions.responseDecoder,
          listFormat: requestOptions.listFormat,
        ),
      );
      return handler.next(redirectResponse);
    }
    return handler.next(response);
  }

  bool _isRedirect(int statusCode) {
    return statusCode == 301 ||
        statusCode == 302 ||
        statusCode == 303 ||
        statusCode == 307 ||
        statusCode == 308;
  }
}

IMO this does not related to the mechanism of interceptors, but related to the mentioned response.realUri.

While it seems to be possible, I don't think it should, since that creates a lot of complexity. What we could offer is to allow the user to handle redirects himself.

But cookieJar should work "out of the box".

authors: AlexV525 CaiJingLong ueman wilinz

repo: cfug/dio


text:
Suggest: add second service for long press button.

Is it possible to add second service for "long press" the button?
original service for "short press", this will double the button function?

Great idea and not too hard to implement, i think.
I will check this out :)

"Long press"-Service is included in Release 1.3.0 (https://github.com/cgiesche/streamdeck-homeassistant/releases/tag/1.3.0)

Tested, and works like a charm~~   :)

authors: cgiesche cnstudio

repo: cgiesche/streamdeck-homeassistant


text:
Put box() on figure 55

'Maximum likelihood estimates of recruitment deviations for the base model and alternative...'
For consistency.
It should change figure 15 as well.

Fixed Figure 15, 50, and 55 in commit 0c6d15a8619ee26bc02f0781bf84a1348ea29518
Ideally r4ss SSplotComparisons() would do this but doesn't for subplot = 10 so I just added box() commands into the knitr chunks.

repo: cgrandin/hake-assessment


text:
How to config?

I get an error (Failed to instantiate module app due to: Error: [$injector:unpr] Unknown provider: notify when I do something like this:
app.config( function(notify) {

})

How can I inject the notify service in the config?

You can't inject normal services into a module config method (only providers).  You'll need to use a module run method.

Okay, thanks!

Fixed by #39

authors: cgross faceleg samuelbeek

repo: cgross/angular-notify


text:
Protein function prediction

Hi, I have a question about the code. After predicting the interaction between a protein and its substrate, can it determine whether the protein catalyzes the substrate? Thanks!

that's on open question for research 😄

authors: Jiajie-Zzz arogozhnikov

repo: chaidiscovery/chai-lab


text:
Release 1.0.0

Release for 1.0.0
Marked as 1.0.0 because A) its a big jump in capabilities and B) lets get this thing on the semver train!
@logicalparadox if you're happy I'll merge and release, then @Charminbear (or someone else) can get to work putting it into Chai :wink:

Coverage remained the same at 100.0% when pulling bbd3ff14128f09538132234ee1df1423151db23e on release-1.0.0 into 6d95c818bebffcd09530e69219fa01f378cdaf3a on master.

Sounds good - im happy to see this in chai soon. :+1:

:+1:  and a bonus :100: for ReleaseNotes.md...
Ship it!

authors: Charminbear coveralls keithamus logicalparadox

repo: chaijs/type-detect


text:
Add first name field to email signup form

Users are statistically exceedingly more likely to open an email if it contains their first name
use the standard emailoctopus forms

Hey @DanGould , I would like to take on this issue.

Ok @ArtemSilka cool!
The field would need to be added to the existing one here: https://github.com/chaincase-app/website/blob/6b67673d0157f424756d3ebd5bec908a7138902f/app/views/static_pages/home.html.haml#L32-L54
here's the raw embedded form code containing the first name field from EmailOctopus (our mail list provider) it would need to be converted to haml first. maybe this tool can help:
<link rel="stylesheet"
      href="https://emailoctopus.com/bundles/emailoctopuslist/css/1.6/form.css"
>

    Subscribe to my list

  <form action="https://emailoctopus.com/lists/f0814c6a-30ea-11eb-a3d0-06b4694bee2a/members/embedded/1.3s/add"
        method="post"
        data-message-success="Thanks for subscribing!"
        data-message-missing-email-address="Your email address is required."
        data-message-invalid-email-address="Your email address looks incorrect, please try again."
        data-message-bot-submission-error="This doesn't look like a human submission."
        data-message-consent-required="Please check the checkbox to indicate your consent."
        data-message-invalid-parameters-error="This form has missing or invalid fields."
        data-message-unknown-error="Sorry, an unknown error has occurred. Please try again later."
        class="emailoctopus-form"
        data-sitekey="6LdYsmsUAAAAAPXVTt-ovRsPIJ_IVhvYBBhGvRV6"
  >

        Email address

      <input id="field_0"
             name="field_0"
             type="email"
             placeholder
             required="required"
      >

        First name

      <input id="field_1"
             name="field_1"
             type="text"
             placeholder
      >

      <input type="text"
             name="hpc4b27b6e-eb38-11e9-be00-06b4694bee2a"
             tabindex="-1"
             autocomplete="nope"
      >

      <input type="hidden"
             name="successRedirectUrl"
      >
      <button type="submit"
              style="background-color: rgb(224, 224, 224); color: rgb(0, 0, 0); font-family: inherit;"
      >
        Subscribe
      </button>

  </form>

    Powered by

      EmailOctopus

Hey @DanGould, just to clarify, how far should code. Seems like you'll need to expand db schema, since right now it only has email. Plus you probably will need to adjust controller, to be able to pass name in params.
Please let me know.
Because if you're looking only for ui,

yeah all the back end is handled on email octopus, the rails is just vestigial at this point I see the tests came through with first names. Please mark the name field optional if you can. Appreciate the ambition to help out :)

Got it! Ok, just changed firstname field to be optional.
I've created brunch and tried to push but getting access denied message
remote: Permission to chaincase-app/website.git denied to ArtemSilka.
Please let me know how should I upload everything.

You're going to want to fork the repo push your changes there and then create a pull request from there to here where we can review 😎

Got it, let me try! Thank you

closed by #16

authors: ArtemSilka DanGould

repo: chaincase-app/website


text:
fix setting parameters of random_distort

The parameters of random_distort doesn't seems work because the value is set with hard coded.
I fixed it.

Thank you for review.
I updated.

Thank you!

authors: Hakuyume peisuke

repo: chainer/chainercv


text:
"/hiro000227"エンドポイントを追加する

/hiro000227 にGetリクエストをすると、 Hello hiro000227! と表示される機能を実装しました。
Webブラウザで、 http://localhost:4567/hiro000227 にアクセスすると、動作確認ができます。

こちらのPRの送り先は、challecara配下ではなく、forkした先のチームのリポジトリとなると思います。なので、こちらのPRはcloseさせていただきます。
以下のドキュメントを読んでみてください〜
https://github.com/challecara/git-training/blob/main/doc/pull_request.md

repo: challecara/git-training


text:
Need to make a Linux installation walkthrough

There's a Mac walkthrough up on the wiki, but my only Linux install is on Heroku, which is not at all like a standard Linux install.
Ideally, somebody who knows what a Linux install should look like would take this, not someone (like me) who'd stumble through and eventually say, "Okay it works now."

I think @cohoe's changes have dealt with this issue. If there's more to do let's open new issues for the leftover bits.

repo: challfry/swiftarr


text:
Issue prefix

Issues menu is just displaying issue title and so is completely hard to distinguish issues per repository.

Will now display with the following patterns

For own repository : [reponame] Issue title
For organizations : [orgname/reponame] Issue title

Also add avatars of the submitter.

repo: chamerling/QuickHubApp


text:
Code Generation!!!

To do: To be determined...

[ ] Add printf support and ignore that undefined reference at semantic stage (or initially add it to function table)

authors: Jaredanwolfgang chanbengz


text:
Feature/volume rendering shader

Adds a shader builder to create a GLSL shader with separate invertible contrast limits for both 2D cross sections, and 3D volume rendering.
If we agree on the builder class structure, we could look to refactor the other shaders which are defined inline in json_generator.py into the builder class and add tests for those shaders too.
Also refactors a little bit to support adding this a bit easier and a few associated fixes.
The final shader would be like this:
#uicontrol invlerp contrast
#uicontrol bool invert_contrast checkbox
#uicontrol invlerp contrast3D
#uicontrol bool invert_contrast3D checkbox

float get_contrast() {
  return invert_contrast ? 1.0 - contrast() : contrast();
}
float get_contrast3D() {
  return invert_contrast3D ? 1.0 - contrast3D() : contrast3D();
}

void main() {
  float outputValue;
  if (VOLUME_RENDERING) {
    outputValue = get_contrast3D();
    emitIntensity(outputValue);
  } else {
    outputValue = get_contrast();
  }
  emitGrayscale(outputValue);
}

With the values for the contrast limits and window limits stored in the JSON state as the layer shader controls parameter

Hey @manasaV3 and @aranega, could you please review? I don't have permissions to request reviews

Hey @manasaV3, could you please check if this is what you were expecting from the public API side?
The call signature to state_generator.generate_image_layer remains unchanged, but the resulting shader now has functionality to invert both the volume rendering and regular cross section contrast limits.
I've also added a default volume rendering samples of 256 to the image shader, as neuroglancer's default 64 is a bit on the low side. Maybe we could bump this in the future to something higher, but 256 might be a good tradeoff between image quality and rendering speed if a user turns on VR in neuroglancer.

Shader for has_volume_rendering_shader = True:

Shader for has_volume_rendering_shader = False:


text:
Integrate all required ontology processing into ontology preparation

Context
The cellxgene-schema CLI downloads and processes pinned ontologies to address the schema requirements.
There is separate (distributed in the code base) ontology processing occurring for filter by metadata, filter with ontology awareness, gene expresssion, and cell census.
We should investigate integrating all required ontology processing into the ontology preparation step for the cellxgene-schema CLI  to eliminate duplication of code and ensure that all features are based on the pinned versions of the ontologies.
See Proposal: Standardization of ontology-derived process and assets of CELLxGENE Discover.

@atolopko-czi also writes:
I've attempted to capture the currently ad hoc, manual Schema Upgrade Process. This can serve as requirements for what processes and configuration updates need to be automated and simplified in order to address this story.

I've attempted to capture the currently ad hoc, manual Schema Upgrade Process. This can serve as requirements for what processes and configuration updates need to be automated and simplified in order to address this story.

Issue moved to chanzuckerberg/single-cell #446 via Zenhub

authors: atolopko-czi brianraymor

repo: chanzuckerberg/single-cell-curation


text:
服务器每次重启都会添加一个server-side-host

每次删除了之后，服务器重启server-side-host又给我添加回去了，如图，在哪里能关闭这个功能吗？

下个版本修复

服务端将不再默认支持客户端，2.0已修复

authors: RiversSong chaos-zhu

repo: chaos-zhu/easynode


text:
Renaming a few metrics

Hi everyone,
In the Common WG, we have been considering renaming a few of the time-based metrics. This wouldn't require a review or anything, just a rename:
https://github.com/chaoss/wg-common/issues/176

Closing this, as I think this has all been completed.

authors: ElizabethN germonprez

repo: chaoss/wg-evolution


text:
Publish on npm

Hello, thank for this library. Can you publish it on npm registry?

And add, please license, without it I can't use it.

I've added a license in commit 478c03b and published.


text:
Implement flake8-broken-line

flake8-broken-line
Forbids broken lines with a \.
Error Codes

[ ] N400 Found backslash that is used for line breaking

Because of compatibility with black this should be taken into account: https://black.readthedocs.io/en/stable/the_black_code_style/future_style.html#using-backslashes-for-with-statements

As of python 3.10 with statements can be parenthesized.

You are right.
Therefore the check should probably either run differently depending on the version of python that is targeted (pre vs. post 3.10) or have an option to allow backslashes in with statements.
On the other hand the special behavior is an edge case and becomes kind of obsolete in 2.5 years with 3.9 eol. So I can fully understand when the special behavior is not implemented. I do not know how much work that would be to implement. Unfortunately I currently have no time at hand to dig in to this.

authors: Cielquan conformist-mw

repo: charliermarsh/ruff


text:
README: add Windows Install Instruction (Winget)

Changes

Add Windows Install instruction for Winget.

Recently, I added the manifest for soft-serve to Winget. Once new releases are made, I will update the Winget manifest asap. I have referred to glow README for this template.

Related: https://github.com/goreleaser/goreleaser/issues/2290
We will likely wait for goreleaser to support winget before updating our docs. I know they're planning on supporting this in the near future, so will keep this open for now as a reminder. That will allow us to ensure that the winget package is automatically updated with each release. Thanks for the nudge!

It looks like we're now using goreleaser to publish our releases, so I'm happy to merge this. I'm just a bit unsure about naming and want to make sure that the package naming is consistent with what gets published to the winget manifest.
Related: https://github.com/microsoft/winget-pkgs/blob/master/manifests/c/charmbracelet/soft-serve/0.7.3/charmbracelet.soft-serve.installer.yaml#L3C20-L3C34

authors: bashbunni kbdharun

repo: charmbracelet/soft-serve


text:
Quote vSphere username to handle backslashes

Fixes https://bugs.launchpad.net/charm-kubernetes-master/+bug/1872811

LGTM

authors: Cynerva tvansteenburgh


text:
Allow unclipped scales

Feature Proposal
Starting in #10691, a dataset is automatically clipped to its scales' range. We have some custom scales that control an area for plotting but aren't intended to override the chart's normal clipping area.
Would it be worth adding an option (either via scale options or via a new scale API) to allow scales to control whether or not they're considered for clipping? If not, we can work around it locally.
Possible Implementation
No response

It could make sense for me to add an option. My 2 cents.

@joshkel @etimberg I think it could be better to have new option at scale level.I have submitted a PR https://github.com/chartjs/Chart.js/pull/11404 adding clip option to scales configuration.

I'm not sure if it's a regression but clip doesn't seem to be working on the scale level anymore.
When I set clip: false on the x and y scales level

When I set clip: false in the options

When I set clip: false in the options and clip: true in the scales x options

I want to set clip: false in the options scales.x only, is there any way? Am I missing something?

authors: joshkel sbourouis stockiNail


text:
Improved object initialization

Breaking another one off of https://github.com/chartjs/Chart.js/pull/7085

The alternatives to not doing this would be either to check if these values are undefined everytime we use them which would be very unweidly or not enable the TypeScript checks which I think would be more likely to result in issues. I'm not able to think of a way in which initializing to -1 would cause issues.
Also, this PR saves a couple hundred bytes
Before:
-rw-r--r--  393084 Feb 17 14:18 Chart.js
-rw-r--r--  177982 Feb 17 14:18 Chart.min.js
After:
-rw-r--r--  392822 Feb 17 14:18 Chart.js
-rw-r--r--  177825 Feb 17 14:18 Chart.min.js

If we have a check for any of these variables not being undefined somewhere, that check would not catch the -1. Saying that we would have to check for undefined every where means we should now check for -1 every where or to make sure using that value in the computations will not result in errors, like unexoected proportion of something.
What I'm really trying to say is that reviewing this change property is not a Light task. Also would need to add the changed defaults to migration docs.
There might be no said issues, but I'm sure tsc is not going to catch those.
I'll take a deeper look, if this way is something we think is good for the project.
Te savings come from the 'undefined` every where. Could initialized one local variable to undefined and assign that to each property to gain those bytes

If we have a check for any of these variables not being undefined somewhere, that check would not catch the -1. What I'm really trying to say is that reviewing this change property is not a Light task. There might be no said issues, but I'm sure tsc is not going to catch those.

Yes, that's all true. One thing to note though is that this is done only for the scales. _updateDatasets is called after the scales have been updated, so we don't have to worry much about the controllers accessing those properties before they're initialized and just have to worry about internal usage (the scales are also minorly used in core.controller, scaleService, and layouts, but it doesn't look like there's anything there that'd be of concern).
Though we have pretty good test coverage and will have users testing in the alpha, beta, etc. which gives another layer of confidence

Saying that we would have to check for undefined every where means we should now check for -1 every where or to make sure using that value in the computations will not result in errors, like unexoected proportion of something.

That's a bit different than what I was saying. There isn't anywhere that we need to check for undefined in the code to the best of my knowledge from a logic perspective. What I was suggesting is that if we turn on strictNullChecks the compiler will force us to check whether the variable is undefined everywhere the variable is used.

An example of possibly changed behavior:
https://github.com/chartjs/Chart.js/blob/7b029aa780edaedaf9fb246420ca9ca4c91d31d1/src/core/core.layouts.js#L151
-1 || 400 is going to be -1
That might not happen, but we might have other similar checks. External libraries can have similar checks.
We can document all this and be happy about it, but is it really worth it?

Explainin a bit more, not all the code is in this library, that needs to be considered, so TSC does not have the whole picture. Things can be accessed "out of order" in plugins, extensions and other external code.
I'm not sure what we can gain from these strict checks, because of those things.
A valid point is that if an object does not mutate in its lifetime, its going to be better for optimization. So initialization in general is good for that. Its bad for the library size.
If we want to get the types from the initialization, I think a better default is 0, because at least the short-circuit operations (||) work the same as with undefined.
0 could still cause some unexpected results, but IMO a lot less likely.
If the variable is strictly internal (not used in any public methods or functions that are usable from public methods), then it is contained within this lib and TSC has the whole picture (assuming it has enough typings to know all the places where said objects are accessed) and then we could use -1 after making sure there is no path that uses that value before its set to its real value.
If we can resolve the final value in constructor, that would be optimal.

@kurkle you have some great points on where this could error. 0 seems like a safe initialization given that or maybe we shouldn't change this at all.

Regarding the example above (https://github.com/chartjs/Chart.js/pull/7114#issuecomment-587289559), there won't be a -1 because width is already initialized by setLayoutDims by that point.
I think most of these variables should be private and had a TODO to revisit that, so I sent a PR (https://github.com/chartjs/Chart.js/pull/7115) to address that part. That would leave a much smaller number of variables that we have to worry about (top, bottom, left, right, width, height). It might be possible to initialize those in the constructor, but we might end up duplicating some code between core.controller (where they're constructed) and core.layouts
We could initialize to 0 (nothing breaks, all tests pass, etc.), but if we're worried about external users, is that really better? -1 is clearly unitialized, but 0 is very often a valid value for the position variables and so I think that'd make it harder to tell that things are unitialized
There's a very small window which we need to consider. It's only in beforeUpdate where the scales have been created, but not initialized. I don't really see why anyone would be trying to get the scale layout properties before the scale has been laid out though. I just don't see a usecase for it and so it would surprise me quite a bit if people were doing it. I think once we make the other variables private it's a pretty safe change

I'm going to close this for now. I realized that this may show up in more places as we add more types. I want to consider the full impact before committing to it since we're a bit uncertain about whether it's a good idea

authors: benmccann etimberg kurkle


text:
Invalid events callbacks invocation on line label

When you set the events callbacks on a line annotation and you move the mouse on the label of the line, the events are triggered only when the mouse is on the half area of the label.
The issue has been introduced by version 1.2.2 (with version 1.2.1 it works correctly).
https://codepen.io/stockinail/pen/JjrmRBR

Closed because I forgot that labelX and labelY are the center of the label and not x,y of the box.

The issue has been introduced by version 1.2.2 (with version 1.2.1 it works correctly).

This is related to the #590 (already fixed)

repo: chartjs/chartjs-plugin-annotation


text:
Bug:  官网下载报错 公众号打不开

Chat2DB Version
3.2.4
Describe the bug

这玩意有人维护么 ，官网下载都不行啊
关注二维码就算了，你这个二维码都出不来

你好，你这边是想下载哪个版本的？windows还是mac？

Chat2DB Version
3.2.4
Describe the bug

这玩意有人维护么 ，官网下载都不行啊
关注二维码就算了，你这个二维码都出不来

开源版在这里下载：https://github.com/chat2db/Chat2DB/releases
pro版这里：https://chat2db-ai.com/zh-CN/download

authors: miahemu silent-night-no-trace tmlx1990


text:
[BUG] 使用conda安装jq库之后，仓库仍然无法使用jq库。

问题描述 / Problem Description
如题，以下是截图：

复现问题的步骤 / Steps to Reproduce

执行 conda install jq
确认安装 conda list jq
检验效果 python -c "import jq; print(type(jq))"
报错：ModuleNotFoundError: No module named 'jq'

预期的结果 / Expected Result
导入jq库，并打印：<class 'module'>
实际结果 / Actual Result
报错：ModuleNotFoundError: No module named 'jq'
环境信息 / Environment Information

langchain-ChatGLM 版本/commit 号：448c99f969f22c23abcf3ec1b8646dad73d15960
是否使用 Docker 部署（是/否）：否
使用的模型（ChatGLM2-6B / Qwen-7B 等）：ChatGLM3-6B
使用的 Embedding 模型（moka-ai/m3e-base 等）：bge-large-zh
使用的向量库类型 (faiss / milvus / pg_vector 等)： faiss-gpu
操作系统及版本 / Operating system and version: Windows 10
Python 版本 / Python version: Python 3.10.13
其他相关环境信息 / Other relevant environment information: conda 23.11.0

附加信息 / Additional Information
添加与问题相关的任何其他信息 / Add any other information related to the issue.

你确定成功安装了啊 pip 或者conda list一下安装的库

问题描述
使用conda install jq 成功后，在python中import jq时，依然报ModuleNotFoundError: No module named 'jq'

复现问题的步骤/重现步骤
执行 conda install jq
确认安装conda list jq
检验效果 python -c "import jq; print(type(jq))"
报错：ModuleNotFoundError: 没有名为“jq”的模块
预期的结果
导入jq库，并打印：<class 'module'>
实际结果
报错：ModuleNotFoundError: No module named 'jq'
环境信息
langchain-ChatGLM 版本/提交号：0.2.09
是否使用Docker部署（是/否）：否
使用的模型（ChatGLM2-6B / Qwen-7B 等）：ChatGLM3-6B
使用的Embedding模型（moka-ai/m3e-base 等）：bge-large-zh
使用的支持库类型 (faiss / milvus / pg_vector 等)： faiss-gpu
操作系统及版本 / 操作系统及版本: Windows 10
Python版本/Python版本：Python 3.10.13
其他相关环境信息 / 其他相关环境信息：conda 23.11.0
执行过程
安装conda install jq

查看'conda list'

同样的问题，似乎没有解决啊。conda install之后在conda list里面能看到jq，但是import就会报错，没有这个库

同样的问题，似乎没有解决啊。conda install之后在conda list里面能看到jq，但是import就会报错，没有这个库

搞了一下午，才明白jq是不能在win上import的，最多搞个c++编译器命令行调用。有哪位大神可以解决不能import jq的问题吗

authors: ItsFated Zhaibeiebi tomdogs1 zRzRzRzRzRzRzR

repo: chatchat-space/Langchain-Chatchat


text:
调用agent的text2sql接口，或者只使用text2sql不适用agent的api接口，得到的输出是重复的两句

messages = [
{"role": "user", "content": "赵强的生日是多少？"},  #
]
def tools_text2sql():
tools = ['text2sql']
data = {
"model": model_name,
"messages": messages,
"stream": False,
"temperature": 0.7,
# "tools": tools,
"tool_choice": "text2sql"
}
    response = requests.post(f"{base_url}/chat/completions", json=data, stream=False)
    print(response.text)

输出：{"id":"chatf27734db-caa4-4fb3-a70b-a6e3e41f13e8","object":"chat.completion","model":"qwen2-7b-instruct","created":1721206410,"status":5,"message_type":1,"message_id":null,"is_ref":false,"choices":[{"message":{"role":"assistant","content":"Thought: 我需要从数据库中查询赵强的生日信息。\nAction: text2sql\nAction Input: {\"query\": \"SELECT 生日 FROM 人员信息 WHERE 姓名 = '赵强'\"}Thought: 我需要从数据库中查询赵强的生日信息。\nAction: text2sql\nAction Input: {\"query\": \"SELECT 生日 FROM 人员信息 WHERE 姓名 = '赵强'\"}我从数据库中找到了赵强的生日信息。\nFinal Answer: 赵强的生日是1957年6月27日。赵强的生日是1957年6月27日。","finish_reason":"stop","tool_calls":[{"index":0,"id":"fe936c6f-0ff5-460e-b190-0d3530cf8713","type":"function","function":{"name":"text2sql","arguments":"{'query': \"SELECT 生日 FROM 人员信息 WHERE 姓名 = '赵强'\"}"},"tool_output":"查询结果:赵强的生日是1957年6月27日。\n\n执行的sql:'SELECT `birth_date` FROM `person_info` WHERE `name` = '赵强'\nSQLResult: [(datetime.date(1957, 6, 27),)]\n'\n\n","is_error":false}]}}]}

这个是走的默认的langchain的SQLDatabaseSequentialChain 确实有这个问题如果不设置表配置的话


text:
[백엔드/feat] JWT 토큰 검증 개발

해당 이슈는 #1 개발 전 선행되어 개발되어야함 => 유저 update 시 token 검증 필요
목표

JWT 토큰을 이용하여 전체 서비스 로직에서 유저 및 권한 검증 후 수행 될 수 있도록 수정

스펙 정의

AccessToken : 유저 AcessToken 으로 만료시간 1시간
RefreshToken : 유저 RefreshToken 으로 만료시간 1일

개발 내용

유저 로그인 시 accessToken 과 refreshToken 을 header 에 담아서 전달 => refreshToken 은 추후 토큰 재발급 검증을 위해 redis 에 저장
서비스 로직 수행 이전 token 검증
refreshToken 의 경우 acessToken 재발급 전 사용하며, redis 에 저장된 refreshToken 정보와 비교 검증 수행

소셜 로그인에도 마찬가지로 token 검증이 필요하며 추후 개발 내용을 고려하여 개발 필요

API 명세서

Note: 모든 요청에는 Bearer Token이 필수로 요청 헤더에 포함되어야 합니다.

1. 로그인 API

Endpoint: POST https://hjproject.kro.kr/chatforyouio/auth/login
Description: 사용자 로그인 요청을 수행하여 Access Token과 Refresh Token을 발급합니다.

Headers

Content-Type: application/json

Request Body
{
    "id": "string", // 사용자 이메일
    "pwd": "string" // Base64로 인코딩된 비밀번호
}

Response

Status: 200 OK

Response Headers:

AccessToken: 로그인 후 발급된 액세스 토큰
RefreshToken: 로그인 후 발급된 리프레시 토큰

Response Body:
{
    "result": "success",
    "userData": {
        "idx": 32,          // 사용자 인덱스
        "id": "sejon@test.com",  // 사용자 이메일
        "name": "sejon",     // 사용자 이름
        "nickName": "nicknameCC" // 사용자 닉네임
    }
}

2. 로그아웃 API

Endpoint: POST http://localhost:8443/auth/logout
Description: 사용자 로그아웃 요청을 수행하여 세션을 만료 처리합니다.

Headers

Content-Type: application/json
Authorization: Bearer <AccessToken>

Request Body
{
    "idx": 32,          // 사용자 인덱스
    "id": "sejon@test.com" // 사용자 이메일
}

Response

Status: 200 OK

Response Body:
{
    "result": "success"
}

3. 토큰 갱신 API

Endpoint: POST http://localhost:8443/auth/refresh_token
Description: 만료된 액세스 토큰을 갱신하여 새로운 액세스 토큰을 발급합니다.

Headers

Content-Type: application/json
Authorization: Bearer <RefreshToken>

Request Body
{
    "idx": 32,          // 사용자 인덱스
    "id": "sejon@test.com" // 사용자 이메일
}

Response

Status: 200 OK

Response Body:
{
    "result": "success",
    "accessToken": "new_access_token"
}

@junotb
JWT 토큰 발급 및 검증, 갱신 api 개발 완료하였고, 테스트를 위해 배포해두었습니다.

1차적으로 유저 정보 수정(닉네임, 비밀번호) 수정 api 에만 검증로직을 적용해두었습니다.

2가지 먼저 확인 후 다른 api 에도 적용 해두도록 하겠습니다.

유저 로그아웃 api 를 만들어두었습니다. 이쪽은 로그인 시 redis 에 저장된 유저 정보(토큰 정보, 로그인 정보 등)를 삭제하기 위함입니다. 다만 이전에 로그아웃이 부분에 대해서 뭔가 이슈가 있었다고 들어서 그게 잘 되면 이대로 적용하거나 아니면 추가 수정하도록 하겠습니다ㅎ

유저 정보 수정 api 명세서는 #1  요거 확인부탁드립니다!

@junotb 아래 api 들에 토큰 검증 로직 적용 완료했습니다.
User

유저 업데이트 : 정보 업데이트 & 비밀번호 업데이트
유저 삭제
로그인 된 유저 조회

ChatRoom

방 생성
방 정보 조회
방 정보 수정
방 삭제
유저 방 입장
openvidu 데이터 조회
openvidu connection token 조회


text:
doc: update slack bot event subscriptions

Fixes #246

@vishnu-narayanan  we might need to update this in the codebase as well?

@sojan-official I think we are not specifying any specific permissions over there. We are relying on the slack app to have the necessary permissions we need.
Correct me if I'm wrong here.
cc: @tejaswinichile

@vishnu-narayanan  my bad. makes sense.

authors: sojan-official vishnu-narayanan

repo: chatwoot/docs


text:
3ds implementation for custom UI Flow

Hi,
I am trying to integrate the iOS sdk with the custom UI as shown in the iOS Example Custom. I am looking to add the 3Ds implementation but could not find any documentation to implement it for iOS.
We have successfully implemented for the android-sdk as it's clearly documented in the git repo.
/////////////// Android Snippet ////////////////
private final PaymentForm.On3DSFinished m3DSecureListener = new PaymentForm.On3DSFinished() {
    @Override
    public void onSuccess(String token) {
        displayMessage("Result", "Authentication success: " + token);
    }

    @Override
    public void onError(String errorMessage) {
        displayMessage("Result", "Authentication failure: " + errorMessage);
    }
};

/////////////// End //////////////////////
Note : Looking for something similar for iOS.
Kindly let me know if there is any way to add the 3Ds for the iOS.
I have found the ThreedsWebViewController but on the callback methods we are not able to receive any sid_token as it is received in the android-sdk version as mentioned above in snippet.
It would be really helpful if i could get e sample to integrate.
Kindly advise me to proceed further.

Hi @gravindragupta90 The sid_token is currently not been added to the ThreedsWebViewController call back methods for iOS. This is something we are working on and will be available in our next release.

@gravindragupta90 The 3.5.0 release was made yesterday with examples on how to handle 3DS call back. Also we are only passing back the token on success case in this release. However if you feel this is really required for failure scenario please feel free to raise a new issue and we can plan to put this in our next release. Please note we highly recommend you do carry our regression testing after you upgrade to our new release and before you release your apps into production.

repo: checkout/frames-ios


text:
Ping and Traceroute should not use parse_bird_output

Version: 0.1.0 (Latest from pip)
Behavior: When use ping or traceroute on bird nodes, execution of ping or traceroute command is successful. But it will return not-found error to hyperglass.
Reproduce:

Configure hyperglass-agent to use bird.
Start a ping or traceroute from hyperglass.

Related Code:
        output += await parser(     raw=raw_output, query_data=query, not_found=params.not_found_message  )
Possible Solution:

Remove parse_bird_output when performing ping or traceroute.
Add parsers for ping and traceroute.

Hi, I wasn't able to reproduce this in version 0.1.4 (latest from PyPI as of 2020 03 21). However, I've made sure this won't happen in version 0.1.5, which I just pushed up. Please give it a try, and let me know if you run into any issues. You'll probably want to update your hyperglass version as well, as one of the fixes I implemented required a change on both ends.
Thanks,
Matt

I upgraded the version of both ends, and the problem seems to be solved.

authors: SteveCharlesYang checktheroads

repo: checktheroads/hyperglass-agent


text:
Improve documentation of data bag fallback flag

The existing documentation doesn't mention the databag_fallback flag needed in .kitchen.yml to take advantage of this very useful feature of chef_vault_item(). It appears to be documented only in the source code.

Oops, this should go on chef-vault, which led my here. Apologies.


text:
Does not work with Chef 13

Cookbook version
5.0.2
Chef-client version
13.x
Platform Details
Centos 7 on Vagrant
Scenario:
Run default recipe, fails.  Switch to chef 12 and it works.
Steps to Reproduce:
Just use the included kitchen.yml and watch it fail.
Expected Result:
Jenkins should be installed and started
Actual Result:
Chef run fails, kitchen fails

@scarolan I'm trying to reproduce this and I'm not seeing issues with Chef 13. Can you give it a try with the latest releases and get back to me with more information.

This is still an issue. Specifically, you can not connect a slave via jnlp or ssh connection method under Chef 13. There is an open issue regarding the specific error that is returned here:
https://github.com/chef-cookbooks/jenkins/issues/615
I have attached the output from my converge attempt as well as the resulting stacktrace on that thread.

This has been fixed on master and I'll be cutting a release once I fix a few other problems

authors: oberones scarolan tas50

repo: chef-cookbooks/jenkins


text:
--bootstrap-msi-

Versions:

Version of knife-vsphere: knife-vsphere (2.0.0)
Version of chef:
Chef Development Kit Version: 0.10.0
chef-client version: 12.5.1
berks version: 4.0.1
kitchen version: 1.4.2

Platform Details

Version of vSphere: 5.5
Version of ESXi:

Scenario:

I have chef client and other chef rpm located on a nfs mount. I am trying to use knife vsphere to build and then bootstrap new VMs. Is there any option to provide nfs paths with --bootstrap. Since my organization isolates linux/window machines from external network, I need to keep rpm locally on a nfs server or in pulp repo.

Hi, no, file paths won't work, but we do support --bootstrap-msi-url to pass a URI which you could point to your repo.

Hi,
Do I need to both --bootstrap and --bootstrap-msi-url together or only --bootstrap-msi-url.
As I understand --bootstrap downloads and execute install.sh , that will inture download the rpm and will install it. After that the actual "knife bootstrap" step will be executed. When bootstrap gets executed, and it uses user given in --ssh-user, how can I specify sudo ? Normally when we do bootstrap, we use --sudo, to specify that user will be using sudo.

I think --bootstrap-msi-url  doesnt works for Linux VM builds. Is there any other option for Linux machines ?

Hi, the msi options are only for Windows, and yes, you'll need --bootstrap in any case.
Sudo will be automagically enabled if your ssh_user is not root.
Linux bootstrapping without an Internet connection is a bit different. Have a look at custom templates and the --distro option. I've never used this option myself but might be able to help out if you get stuck. This feels like something we should be doing a better job on.
If it is a possibility, there's also --bootstrap-proxy that will point your node at a proxy server to get to the Internet.

repo: chef-partners/knife-vsphere


text:
chef-dk fail after chef verify

I installed the last version of Chef-DK for Ubuntu 16.04 which is chefdk_1.3.43-1
chef-client: 12.19.36
berks -v: 5.6.4
Test kitchen -v: 1.16.0
after I had done chef verify, I got this error and chef-dk failed.
Running verification for component 'berkshelf'
Running verification for component 'test-kitchen'
Running verification for component 'tk-policyfile-provisioner'
Running verification for component 'chef-client'
Running verification for component 'chef-dk'
Running verification for component 'chef-provisioning'
Running verification for component 'chefspec'
Running verification for component 'generated-cookbooks-pass-chefspec'
Running verification for component 'rubocop'
Running verification for component 'fauxhai'
Running verification for component 'knife-spork'
Running verification for component 'kitchen-vagrant'
Running verification for component 'package installation'
Running verification for component 'openssl'
Running verification for component 'inspec'
Running verification for component 'delivery-cli'
Running verification for component 'git'
Running verification for component 'opscode-pushy-client'
Running verification for component 'chef-sugar'
........
Generating cookbook example
- Ensuring correct cookbook file content
- Committing cookbook files to git

================================================================================
Error executing action `run` on resource 'execute[git-commit-new-files]'
================================================================================

Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0], but received '128'
---- Begin output of git commit -m "Add generated cookbook content" ----
STDOUT:
STDERR: *** Please tell me who you are.

Run

  git config --global user.email "you@example.com"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'aida@fxinnovation.(none)')
---- End output of git commit -m "Add generated cookbook content" ----
Ran git commit -m "Add generated cookbook content" returned 128

Resource Declaration:
---------------------
# In /opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-dk-1.3.49/lib/chef-dk/skeletons/code_generator/recipes/cookbook.rb

137:     execute('git-commit-new-files') do
138:       command('git commit -m "Add generated cookbook content"')
139:       cwd cookbook_dir
140:     end
141:   end

Compiled Resource:
------------------
# Declared in /opt/chefdk/embedded/lib/ruby/gems/2.3.0/gems/chef-dk-1.3.49/lib/chef-dk/skeletons/code_generator/recipes/cookbook.rb:137:in `from_file'

execute("git-commit-new-files") do
  action [:run]
  retries 0
  retry_delay 2
  default_guard_interpreter :execute
  command "git commit -m \"Add generated cookbook content\""
  backup 5
  cwd "/tmp/d20170510-14208-1atys7d/example"
  returns 0
  user nil
  declared_type :execute
  cookbook_name :code_generator
  recipe_name "cookbook"
end

Platform:
---------
x86_64-linux

ERROR: Chef failed to converge: execute[git-commit-new-files] (code_generator::cookbook line 137) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '128'
---- Begin output of git commit -m "Add generated cookbook content" ----
STDOUT:
STDERR: *** Please tell me who you are.

Run

  git config --global user.email "you@example.com"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'aida@fxinnovation.(none)')
---- End output of git commit -m "Add generated cookbook content" ----
Ran git commit -m "Add generated cookbook content" returned 128
Caused by: (Mixlib::ShellOut::ShellCommandFailed) execute[git-commit-new-files] (code_generator::cookbook line 137) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '128'
---- Begin output of git commit -m "Add generated cookbook content" ----
STDOUT:
STDERR: *** Please tell me who you are.

Run

  git config --global user.email "you@example.com"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'aida@fxinnovation.(none)')
---- End output of git commit -m "Add generated cookbook content" ----
Ran git commit -m "Add generated cookbook content" returned 128

..
---------------------------------------------
Verification of component 'test-kitchen' succeeded.
Verification of component 'chefspec' succeeded.
Verification of component 'rubocop' succeeded.
Verification of component 'knife-spork' succeeded.
Verification of component 'openssl' succeeded.
Verification of component 'delivery-cli' succeeded.
Verification of component 'opscode-pushy-client' succeeded.
Verification of component 'berkshelf' succeeded.
Verification of component 'chef-client' succeeded.
Verification of component 'fauxhai' succeeded.
Verification of component 'inspec' succeeded.
Verification of component 'chef-sugar' succeeded.
Verification of component 'tk-policyfile-provisioner' succeeded.
Verification of component 'chef-provisioning' succeeded.
Verification of component 'kitchen-vagrant' succeeded.
Verification of component 'git' succeeded.
**Verification of component 'chef-dk' failed.**
Verification of component 'generated-cookbooks-pass-chefspec' succeeded.
Verification of component 'package installation' succeeded.

Note: I`m new in chef, so please give me your suggestion to understand what this error means.
Thank you

you need to have git configured in order to run chef verify, the new version of chef-dk explicitly warns end users against running it since there's prerequisites and edge conditions like this

authors: aidda lamont-granquist

repo: chef/chef-dk


text:
Rename ORGNAME_global_admins groups to ORGNAME_read_access_group

The ORGNAME_global_admins group has one purpose inside the Chef
Server. To provide READ access to users who join an organization.  Its
name, however, implies great power. This change renames the group to
something more appropriate by:

changing the organization creation code to create a
ORGNAME_read_access_group rather than a global_admins group,

providing a migration for existing global_admins groups,

changing oc_chef_authz to transparently handle either group

cc @chef/lob
Note this is based off of ssd/global-admins-migration, as it is a bit pointless until that is merged.

Looks reasonable. I'll have to modify fixie to handle this.

To confirm - we're going to hold off on this until 12.1 RC is out?

Much more descriptive name :+1:

:+1:

for posterity, this code was merged as part of #376

authors: joedevivo manderson26 marcparadise stevendanna tylercloke

repo: chef/chef-server


text:
chef-client (local mode) on Windows will sometimes fail to set ohai node domain attribute

Description
Hey.  I'm working on a Windows automation stack using chef-client in local mode and I've noticed that every so often that the automatic ohai node for domain doesn't populate correctly.  As a consequence node['fqdn'] only has the hostname set.
From what I've seen it happens fairly rarely.  But I've seen it several times today over numerous runs.  We depend on the domain parameter for quite a few things so it has an impact.
Chef Version
14.1.12 now but also saw it happening in whatever chef-client version is in ChefDK 2.5.3
Platform Version
Windows Server 2016
Replication Case
Create a simple recipe that logs out the node['domain'] and node['fqdn'] variables on Windows and run it multiple times.  Hopefully you'll see them not set correctly at least once.
Client Output
Just had the issue happen there with the following recipe:
%w(platform platform_version ipaddress macaddress fqdn hostname domain).each do |node_attr|
  log "node['#{node_attr}'] is #{node[node_attr]}"
end

The output was:
* log[node['platform'] is windows] action write
* log[node['platform_version'] is 10.0.14393] action write
* log[node['ipaddress'] is 10.177.17.228] action write
* log[node['macaddress'] is 00:50:56:97:5C:99] action write
* log[node['fqdn'] is qa-test-win-server] action write
* log[node['hostname'] is qa-test-win-server] action write
* log[node['domain'] is ] action write

Domain should've have been lab.local.

node['fqdn'] requires DNS to work correctly, if the DNS lookup fails, it'll fail.  for that reason it isn't a good idea to use that and to instead use node['hostname'] or node['machinename'] instead.

The application I'm setting up needs the full address at points.  Isn't it weird that it only fails sometimes?  Is there anything I can do to make the DNS more hardy?

making DNS more hardy is well of out of scope of what we can do.
that is part of the lost art of system engineering.

authors: Ryuzavi lamont-granquist

repo: chef/chef


text:
Property: Reorder comparison with NOT_PASSED.

The original comparison value == NOT_PASSED ends up calling .<=> on
value under the hood. This relies on the implementation of <=>/cmp
not being broken in that class.
Unfortunately, some classes like OpenSSL::X509::Name have broken
implementations of <=> that raise TypeError instead of returning nil
for mismatched types. https://github.com/ruby/openssl/pull/264
As a result, currently, if you attempt to set a resource property to an
OpenSSL::X509::Name object, the result is a TypeError.
>> require 'openssl'
>> require './lib/chef/constants.rb'
>> name = OpenSSL::X509::Name.new

>> name == Chef::NOT_PASSED
TypeError: wrong argument type Object (expected OpenSSL/X509/NAME)
>> Chef::NOT_PASSED == name
=> false

With this change, we use the implementation of comparison from the
NOT_PASSED object instead, which we know will work no matter what.
Signed-off-by: Andy Brody git@abrody.com
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

There's probably at least a couple dozen more cases of this and somehow this needs at least a comment so it doesn't get forgotten that this is why it is compared in this order.

Thanks @ab for the solid fix and a great explanation

Another option would be to use NOT_PASSED.equal?(value). The Object#equal? method checks for exact object equality, and is unlikely to be overridden by subclasses regardless of which order it's written in. That also makes it more obvious that the order is significant.

authors: ab lamont-granquist tas50


text:
Run chef-client from node application

I have to run chef-client from node.js application. Is their any specific command to run chef client in node.js.

This is a question that's best answered on our community Slack. We use GitHub for filing issues with the product. As for running chef-client via node-js your best bet it probably just to shell out to chef-client via node.

repo: chef/knife-push


text:
修复文件格式问题

修复格式问题：
修复格式: lib/browser/isMobileDevice.js 文件, export 末尾加分号

收到~

repo: chenbimo/yidash


text:
🛑 Homepage is down

In 787da9d, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in 5d25b72.

repo: chendachao/uptime


text:
🛑 Homepage is down

In c15438e, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in fbada63.


text:
🛑 Homepage is down

In 2478851, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in 567ca2e.


text:
兄台博客的SSL证书已过期

建议更新证书，并添加letsencrypt的自动更新功能。

感谢提醒，此前设置了自动更新来着，我去检查一下 :)

用你的配置文件成功在戴尔 Precision T3610 (C602/X79) 引导10.13.5 （CCC备份的系统）。
处理器是 Xeon 1607 v2，但 DSDT 用的是你配置完毕的 Xeon 1620 v2，正常进入系统。
特此感谢！

恭喜呀。
现在正在被Ryzen搞得焦头烂额中，不得不说，民用主板对虚拟化技术的支持还很不成熟，坑多得很。
我微信是cheneyveron

感谢！这是我第一次在戴尔的机器上折腾这个系统，对 AMD 的系统就更无发言权了。
现在暂时的问题是：处理器总是保持最高频率 (Intel Power Gadget) 中 （正常情况下会频率上下波动）。如果兄台有思路和类似的解决方案，还望请教。
平时用邮件较多，不过等我回去即加微信，再交流请教。
再次祝好！

重新配置了ssdtPRGen ，并加上了我的CPU具体清单。现在CPU的电源管理功能正常。
还有其他小问题要修修补补，例如在开机会看到Overcurrent （USB 口电流过载）等等问题。
我们再交流。

authors: YIN-Renlong cheneyveron


text:
🛑 tt is down

In efe50f1, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 504
Response time: 10080 ms

Resolved: tt is back up in 20bb83f.

repo: chenhaonanniubi/upptime


text:
🛑 tt is down

In ff4ff44, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 8887 ms

Resolved: tt is back up in bc12f4b.


text:
🛑 tt is down

In 426f5b8, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 2691 ms

Resolved: tt is back up in a8ad421 after 12 minutes.


text:
🛑 tt is down

In e5c116d, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 2479 ms

Resolved: tt is back up in a5fed48.


text:
🛑 tt is down

In c102ddd, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 3009 ms

Resolved: tt is back up in bd61149 after 21 minutes.


text:
🛑 tt is down

In 35aefdb, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 3679 ms

Resolved: tt is back up in 4c8db1a after 1 hour, 8 minutes.


text:
🛑 tt is down

In 8bd5dab, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 5368 ms

Resolved: tt is back up in cfda807 after 37 minutes.


text:
🛑 Blog is down

In 43ccbaa, Blog (https://blog.chenishr.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Blog is back up in abe74d5 after 10 minutes.

repo: chenishr/upptime


text:
Vitality meter

Provide a Vitality meter. Enhance the existing Frostfall 2.6 meter framework to account for more screen resolutions and provide additional flexibility.

I have pivoted on this; not going to do a vitality meter since it's not a moment-to-moment concern. Provide other feedback instead.


text:
[BUG] AI Assistant throws 404 before making any network calls

Hi,
I am currently trying to set up AI assistant. I have watched your guide and have set up AI assistant accordingly. Once I select a prompt (template) it throws a 404 error immediately. I have tried with both Ollama and OpenAI. I investigated by opening the network tab in the console, and it appears that it is not even making a request before returning the error. I do not think the error is with Ollama, since I am successfully using it with other AI tools and it is probably an issue with my quicadd configuration. Any help would be appreciated. Thanks

I'm encountering the same issue.

Hi @rossoj85 and @alejandroarmas
Your Ollama host should be http://localhost:11434/v1.
For example:

authors: alejandroarmas chhoumann rossoj85

repo: chhoumann/quickadd


text:
chore(deps): update scalafmt-core to 3.7.12

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.7.11 to 3.7.12
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:n:3

Superseded by #245.

repo: chief-of-state/chief-of-state


text:
[Data] Automatically check for CTA-observed holidays

Spinout from: https://github.com/chihacknight/chn-ghost-buses/pull/37#discussion_r1003846071
In compare_scheduled_and_rt.py we have a hard-coded list of holidays in a few places (ex: https://github.com/chihacknight/chn-ghost-buses/blob/main/data_analysis/compare_scheduled_and_rt.py#L99) and ideally that would be handled more automatically.
There is a holidays library in Python: https://github.com/dr-prodigy/python-holidays, which could help us. The thing is that we do not want to check for generic US (or even Chicago / Cook County) holidays, we only want to check for the specific holidays on which the CTA runs Sunday Service.
At time of posting, that is:

Our services operate on a Sunday schedule on New Year’s Day, Memorial Day, July 4th (Independence Day), Labor Day, Thanksgiving Day and Christmas Day.

Unfortunately, I was unable to set up your project to run locally but I wrote up a little script that can parse the holidays out of the CTA site with the holidays package you suggested. This currently assumes that holidays are correctly named but seems to do the job right now. One could also add some word likeness checking but that doesn't seem necessary as is. Let me know if this is of help!
https://gist.github.com/adrianleh/a5c532fd693cdf19791f5a0382405e6f

Thank you @adrianleh! We are on a bit of a hiatus this week for the holiday but I will take a look soon!

authors: adrianleh lauriemerrell

repo: chihacknight/chn-ghost-buses


text:
On Mac you cannot see a sideways scrollbar for the power plants

I am on mac and I can't tell that these can be scrolled sideways

@SuragNuthulapaty - we can't fix the scrollbar not being visible (that's a thing Mac does) but maybe we can put a box-shadow or something to make it clearer it scrolls?

Is this still an active issue? I can take a stab at it if you would like.

Hi @clairenied! DecarbMyState hasn't been in active development for a bit, so I wouldn't suggest jumping on this. If you're looking for a more active project, our group at Chi Hack Night is mostly working on Electrify Chicago!

Awesome!! I will take a look.

authors: SuragNuthulapaty clairenied vkoves

repo: chihacknight/decarbonize-my-state


text:
Support failable initializers

Related to #235.
This PR supports failable initializers.
Something like:
// Rust side
#[swift_bridge::bridge]
mod ffi {
    extern "Rust" {
        #[swift_bridge(Equatable)]
        type FailableInitType;

        #[swift_bridge(init)]
        fn new() -> Option<FailableInitType>;
    }
}

// Swift side
let failableInitType = FailableInitType()
if failableInitType == nil {
    ~
} else {
   ~
}

See: Swift Documentation - Failable Initializers

Hello, hello. Still planning to land this?

Sorry. I'll address your review this weekend if possible.

authors: NiwakaDev chinedufn

repo: chinedufn/swift-bridge


text:
Crash if internet connection isn't available

Hello,
great job on the Sdk and the documentation!
Just a small issue, if you run your node server and you have no connectivity, SDK will crash inside apiCall.js
/** Manage response from Chino API
 *
 * @param error
 * @param response
 */
function responseHandler(error, response) {
  if (error) {
    this.reject(response.body || error);
  }
  else {
    this.resolve(response.body);
  }
}

Response will be undefined, could you just add another check on this?
Thank you!

This repository has been migrated to https://gitlab.com/chinoio-public/sdk-chino-nodejs and will no longer be maintained on GitHub. Please reach out to tech-support@chino.io for any question, or if you wish to contribute to the SDK development.

authors: FRizzonelli aarighi

repo: chinoio/chino-nodejs


text:
Incorporate changes to TensorFlow syntax

TensorFlow has updated the syntax for sub and mul to subtract and multiply in order to be consistent with NumPy syntax.
https://github.com/tensorflow/tensorflow/issues/7032

Thanks for the pull request. This chatbot was written using TF 0.12, and I'd love to update it to TF 1.0, but then I'll have to update more than just tf.mul and tf.sub. I'll do it when I have time :-)

I've updated the repo to work with TF's changed syntax. Thanks @jamiekin

authors: chiphuyen jamiekin

repo: chiphuyen/tf-stanford-tutorials


text:
Remove switch

after #2069
Contributor Checklist

[ ] Did you add Scaladoc to every public function/method?
[ ] Did you add at least one test demonstrating the PR?
[ ] Did you delete any extraneous printlns/debugging code?
[ ] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[ ] Did you state the API impact?
[ ] Did you specify the code generation impact?
[ ] Did you request a desired merge strategy?
[ ] Did you add text to be included in the Release Notes for this change?

Type of Improvement

API Impact

Backend Code Generation Impact

Desired Merge Strategy

Release Notes

Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (3.2.x, 3.3.x, 3.4.x, 3.5.0) ?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

Why do you want to remove switches? Switches should be different from .elsewhen. Switches should be unique cases, but elsewhen has an implicit priority. Switch-generated hardware should use less than priority-based .elsewhen. (I know that this is not anymore the case in Chisel 3, but hopefully might come back at some point. It was the case in Chisel 2).

authors: schoeberl sequencer


text:
Fix some low-hanging fruit reducing global static initialization.

Fixed some constructors to be constexpr so that static const
objects can be changed to static constexpr.
However, the biggest culprits are still ABSL_FLAG and
VERILOG_REGISTER_LINT_RULE.
Signed-off-by: Henner Zeller h.zeller@acm.org

Codecov Report

Merging #945 (fcb8e18) into master (e0fc965) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #945      +/-   ##
==========================================
- Coverage   92.76%   92.76%   -0.01%
==========================================
  Files         316      316
  Lines       20731    20728       -3
==========================================
- Hits        19231    19228       -3
  Misses       1500     1500

Impacted Files
Coverage Δ

common/formatting/align.h
100.00% <ø> (ø)

verilog/formatting/align.cc
94.51% <ø> (-0.01%)
:arrow_down:

common/analysis/matcher/matcher_builders.h
100.00% <100.00%> (ø)

common/text/symbol.h
100.00% <100.00%> (ø)

verilog/formatting/verilog_token.cc
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter hzeller

repo: chipsalliance/verible


text:
Failed to connect to server

Command:
DistantLaunch ssh://myserver.com

Failed to connect to server
Caused by:
    Deserialize failed: IO error while reading marker: failed to fill whole buffer
Exited unexpectedly: exit code 1

+1

Same problem
Exited unexpectedly: exit code 1

Okay, there are two things clear to me from this:

We need better error reporting within neovim to figure out what happens when something goes wrong.
We need to finish the migration to a pure-Rust ssh, as errors with ssh keep happening due to poor C library integrations. See the issue about moving to russh and the branch with progress towards better supporting plugins and using russh.

Me too.
☁ bin ./distant launch ssh://root@192.168.8.7
Failed to launch server
Caused by:
Failed to spawn server: ''distant server listen --daemon --host ssh
Last login: Wed Dec 13 11:40:25 2023 from riley
]0;root@Centos:~[?1034h[root@Centos ~]# distant server listen --daemon --host ssh
-bash: distant: command not found
]0;root@Centos:~[root@Centos ~]#
]0;root@Centos:~[root@Centos ~]# ''

@rileychc for your specific issue, the error just after distant server listen --daemon --host ssh makes me believe that either
a. You don't have distant installed on your server
b. When distant on your host machine logs into your server via ssh, it doesn't have distant in its path to execute it.
If you do have distant on your server, I would recommend including -- distant /path/to/distant as part of your launch arguments, where the path is the absolute path to the distant binary on your server. For example:
# If there was a bin directory within your home directory of your user, and distant was in that bin directory
distant launch ssh://root@192.168.8.7 --distant /home/riley/bin/distant

authors: carand chipsenkbeil heaveless iliutaadrian


text:
Create a new PR

lkhglk

sdad


text:
(FEATURE REQUEST) Re-list for FUTBIN while bot searches

A cool feature that could be implemented is the ability for the bot to re-list for futbin price instead of its original price for expired items when the bot is running!

This will mostly result in futbin ban

license: ISC


text:
Getting undefined from list

I need help on my code since I'm getting undefined when I select an item from my list. What I wanted to do is select an item from the list and once I press enter key, it will display the value I selected.
Here is my code
var blessed = require('blessed');

// Create a screen object.
var screen = blessed.screen();

// Create a box perfectly centered horizontally and vertically.
var outer = blessed.box({
  fg: 'blue',
  bg: 'default',
  border: {
    type: 'line',
    fg: '#ffffff'
  },
  tags: true,
  content: '{center}{red-fg}Hello{/red-fg}{/center}\n'
         + '{right}world!{/right}',
  width: '50%',
  height: '50%',
  top: 'center',
  left: 'center'
});

// Append our box to the screen.
screen.append(outer);

// Create a child box perfectly centered horizontally and vertically.
var inner = blessed.box({
  parent: outer,
  top: 'center',
  left: 'center',
  width: '50%',
  height: '50%',
  border: {
    type: 'line',
    fg: '#ffffff'
  },
  fg: 'white',
  bg: 'magenta',
  content: 'Click {bold}me{/bold}!',
  tags: true,
  hoverEffects: {
    bg: 'green'
  }
});

// If our box is clicked, change the content.
inner.on('click', function(data) {
  inner.setContent('{center}You clicked {red-fg}me{/red-fg}.{/center}');
  screen.render();
});

// If box is focused, handle `enter` and give us some more content.
inner.key('enter', function() {
  inner.setContent('{right}You pressed {black-fg}enter{/black-fg}.{/right}\n');
  var item = list.getItem();
  inner.setLine(1, 'Value from list is ' + item); // I'm getting undefined
  outer.setLine(2, 'Random number is ' + Math.random() * (10 - 1) + 1);
  // inner.insertLine(1, 'foo');
  screen.render();
});

// menu list
var list = blessed.list({
  parent: inner,
  width: '50%',
  height: '50%',
  top: 'center',
  left: 'center',
  align: 'center',
  fg: 'blue',
  border: {
    type: 'line'
  },
  selectedBg: 'green',

  // Allow mouse support
  mouse: true,

  // Allow key support (arrow keys + enter)
  keys: true,

  // Use vi built-in keys
  vi: true
});

list.setItems([
  'one',
  'two',
  'three',
  'four',
  'five',
  'six',
  'seven',
  'eight',
  'nine',
  'ten'
]);

list.prepend(new blessed.Text({
  left: 2,
  content: ' My list '
}));

// Allow scrolling with the mousewheel (manually).
// list.on('wheeldown', function() {
//   list.down();
// });
//
// list.on('wheelup', function() {
//   list.up();
// });

// Select the first item.
list.select(0);

// Quit on Escape, q, or Control-C.
screen.key(['escape', 'q', 'C-c'], function(ch, key) {
  return process.exit(0);
});

// Focus our element.
inner.focus();

// Render the screen.
screen.render();

If you want to get name of a selected item, you can try to use list.ritems[list.selected]:
var item = list.ritems[list.selected];

I figured out a solution the same day I posted this. This is the code I'm using now
theList.getItem(theList.selected).getText();
I'll try your code as well @hypersad
You can see the tool I wrote in action here - https://twitter.com/v1rtu0s1ty/status/929622051698216961
Thanks!

repo: chjj/blessed