Neutron-damaged tungsten validation case from Dark et. al

Resolves #20 . You might want to change the title for the case, it's not my best work.
I might be able to display the parameter tables a bit prettier, e.g. using latex, but I think they're fine for the time being.
Also, I can add the fitting procedure too, the code is available and we have no demonstration of FESTIM's fitting capabilities. I spent a long time toying with the code btw.

Weird how it's showing those previous commits, I must've messed something up on my side, I can fix it though. The first actual commit is https://github.com/festim-dev/V-V-report/pull/42/commits/5eb1f5255296d334e7a5fa3ff8b395715b029942

Weird how it's showing those previous commits, I must've messed something up on my side, I can fix it though. The first actual commit is 5eb1f52

Rebased the commit, looks good now.

The FESTIM data is off for the 2.5 dpa case in the second figure which replicates figure 4a) from the paper. The other dpa's seem fine but I did not check thoroughly. I'll look into it, but I'd appreciate if you had a look first.

The FESTIM data is off for the 2.5 dpa case

Have you checked all the parameters compared to the scripts given with @jhdark paper?

The FESTIM data is off for the 2.5 dpa case

Have you checked all the parameters compared to the scripts given with @jhdark paper?

Yeah I've checked multiple times, thought it might be an implementation thing. I'll check thoroughly again soon.

You must have changed something because I ran the script in the repository directly and obtained the correct spectra:

@rekomodo everything is fine. On your end at least. After a few hours of investigating, @jhdark and I realised there was an issue in the paper repository. All the dpa levels were exported to one folder
https://github.com/J-Dark-PhD/Modelling_neutron_damage_effects_in_tungsten/blob/95d5cbff34364d63bb1e3dd7492fa492578a452a/section_3_model_parameter_evaluation/generate_data.py#L193
except for the 2.5 dpa where it was accidentally exported to another folder
https://github.com/J-Dark-PhD/Modelling_neutron_damage_effects_in_tungsten/blob/95d5cbff34364d63bb1e3dd7492fa492578a452a/section_3_model_parameter_evaluation/generate_data.py#L204
The 2.5 dpa case that was plotted was in fact old data that had been pushed to the repo at some point and never updated because festim was writing elsewhere.
This is the reason why one should not push generated files to a repository, and why this executable book is the way to go 🚀

Could you try tweaking the densities of the first two traps to better fit the data?


Analyzer output does not have clear structure

Far output:
Error: \perp at {x=0, ?-4-3=\any}

Odin output:
Error: Method `mutual_rec.g` is mutually recursive with method `foo.f` through the following possible code path:
mutual_rec.g -> foo.f -> mutual_rec.g
Error: Method `foo.f` is mutually recursive with method `mutual_rec.g` through the following possible code path:
foo.f -> mutual_rec.g -> foo.f

It is hard to extract valuable information from these messages. Is it possible to represent errors in some unified format?

@APotyomkin dup of #26


feat: Use SpeckleLog.Logger instead of global Log

Follows #2313
Updated all references I found.
I skipped Triangles as it seemed to be using something different

git is very confused about the reformatting in some places. but looks good to me


raidboss.js "matches" returns empty object

Description
After updating cactbot to the latest version, custom triggers I've written in my raidboss.js file are no longer working. Logging the return data, I'm seeing matches is an empty object instead of the regex capture info.
Options.Triggers = [
  {
    zoneRegex: /.*/,
    triggers: [
      {
        regex: /.*/,
        alertText: function(data, matches) {
          console.log(matches); // Logs {}
          return;
        }
      }
    ]
  }
];

What are you expecting here in matches?  Because that RegExp doesn't have any capturing groups.

Because the "matches" variable is actually the "captured groups", i.e. regex.exec(log).groups.
The regex you have written /.*/ has no capture groups, if you change it as /(?<log>.*/, then you should get an object which has a string property that contains the whole log line.

I used to use .* to capture everything, and matches would contain an array with a single index containing the full string of the capture. I've never had to use ?<log> to get the result of the regex.
Even something as simple as /.*(a|b).*/ returns an empty object, despite using a capture group.

Yeah, as far as I know, there are only "named capture groups" that would be recorded in.

The old logic before PopupText was converted to ts was to replace the RegExpExecArray with .groups if it was set but leave it intact if not, which resulted in this object getting passed to triggers:

Maybe we need a special handler for the 'None' type case which passes through this type instead?


WITH statement is not useable

This is https://github.com/adimetrius/Herschel/commit/d429e326ad0857b768f75e5555021274427bb384.
When compiling following module, errors 85 (guarding or testing type is not an extension of variable's type) are produced, but it compiles and work fine with DevCompiler.
MODULE A;
	TYPE
		PointerMsg = RECORD
			x, y: INTEGER;
			btns: SET;
		END;

		KeyMsg = RECORD
			key: INTEGER;
			isDown: BOOLEAN;
		END;

	PROCEDURE HandleMsg (VAR msg: ANYREC);
	BEGIN
		WITH
		| msg: PointerMsg (* error 85 *) DO HALT(0)
		| msg: KeyMsg (* error 85 *) DO HALT(1)
		ELSE HALT(2) END;
	END HandleMsg;

END A.

Same error with msg(PointerMsg).x := 10;.

Dynamic variables and type extension Not Implemented Yet, as mentioned in the April 2021 Update on the website.


TE-2.1, TE-4.2:   Add GRIBIPreserveOnly flag and fix refrencing issue

Hi This pull adds GRIBIPreserveOnly guard to TE-2.1 and TE-4.2. It also  fixes TE-2.1  referencing issue for deleting entries in TE-2.1. We can not delete a NHG before deleting all dependent ipv4 entries and the same holds for NH. The reason that current tests does not catch this issue is because it uses delete mode and does not verify RIB ack.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.664%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1462

Relevant Lines:
2012

💛  - Coveralls

@liulk thank you so much! closing this since it is no longer needed.


nixpkgs.calc won't install on macOS

Issue description
The nixpkgs.calc package won't install on macOS. It triggers an assertion in nixpkgs.glibc: assert stdenv.cc.isGNU;.
Steps to reproduce

On macOS run nix-env -iA nixpkgs.calc

Technical details

system: "x86_64-darwin"
host os: Darwin 17.5.0, macOS 10.13.4
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.0
channels(root): "nixpkgs-18.09pre137816.255a833e841"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixpkgs

Awesome, thanks!


Accessing the Title keyword for a property from error object

Let's say I have a schema with a property called "full_name" and in my validation response I want to construct a custom, more user friendly error message.
I want to display the words "Full Name" not the property name "full_name". In the spec it says I can put a Title keyword on the property, which would work perfectly, since the property would be called "full_name" with Title keyword "Full Name".
My question then is - how do I access this the Title keyword from the error object? It seems like I need to set the verbose flag and traverse the schema. Is this correct? or am I missing a trick ?

@byrneciaran That is correct - you will have schema and parentSchema property in each error object, title will be in parentSchema, schema is the value of keyword itself.


Add HCDocument

@isalga could you please "unblock" it? Maybe the next time it's enough to talk to me and trust me? (I can merge anyways) ;)

@isalga changed the settings so you really blocked me now


tiny fix

based #502, removed unused return
by the way: maybe we should merge Framework & Scheduler, they're almost the same now.

Codecov Report

Merging #529 into master will increase coverage by 0.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #529      +/-   ##
==========================================
+ Coverage    41.5%   41.83%   +0.32%
==========================================
  Files          47       47
  Lines       12708    12708
==========================================
+ Hits         5275     5316      +41
+ Misses       6130     6083      -47
- Partials     1303     1309       +6

ping @bbklab  need rebase

rebased， review again

LGTM


test(scorecard): multi-namespace scorecard test

Welcome to Cryostat! 👋
Before contributing, make sure you have:

[x] Read the contributing guidelines
[x] Linked a relevant issue which this PR resolves
[x] Linked any other relevant issues, PR's, or documentation, if any
[x] Resolved all conflicts, if any
[x] Rebased your branch PR on top of the latest upstream main branch
[x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[x] Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Fixes: #539
Multi-namespace scorecard test for 2.x operator

/build_test

/build_test

/build_test

/build_test


Version release script

npm-publish.js is not cohesive enough: besides building and publishing it also modifies some files.
It is better to have a separate script that will bump versions, make a commit and tag it.
scripts/bump-oss-version.js does exactly that.
This simplifies release process and allows manual release to npm if CI is not available.

@facebook-github-bot shipit


SECURESIGN-1015 | Configure ingress sharding

Configuring ingress sharding for all services as well as fixing bug when customising rekor-search-ui host name

/retest


Is this project DEAD ??

I've pick it for project in Production and made a couple of tests but then found that DoubleClick() doesn't work.
So, could anyone advice about the issue ? Please.

I've pick it for project in Production and made a couple of tests but then found that DoubleClick() doesn't work. So, could anyone advice about the issue ? Please.

Seems like it is Honestly. Can't really help, I have not started using this yet.


Property with relation and fieldName config with mongo not populating properly

Describe the bug
This issue seems present when using
import { ObjectId } from '@mikro-orm/mongodb';

to define the primary key on the entity.
For some reason I can use mongoose ObjectId and everything works
 import { ObjectId } from 'mongoose';

Example usage and result
const data = new MyEntity();
data.test = 'me';
const withRelation = new MyEntityTwo();
withRelation.entity = data.toReference();

DB_DI.orm.em.persist(data);
await DB_DI.repos.myEntity.nativeDelete({});
await DB_DI.repos.myEntityTwo.nativeDelete({});
DB_DI.orm.em.persist(withRelation);
await DB_DI.orm.em.flush();
DB_DI.orm.em.clear();
const datas = await DB_DI.repos.myEntityTwo.findAll({});
console.log(datas);

this will log
[
  MyEntityTwo { _id: ObjectId('5ff633eeb2b03e24953c665e'), entity: null }
]

To Reproduce
Steps to reproduce the behavior:

You can see a full example here: https://github.com/ryankauk/mikro-orm-request-time/commit/73ad32ba1a1f95eacaffa912a2c2c2e6c0256801
and if you hit the endpoint http://localhost:5999/myEntity/mikro-create you will see the result in the logs

Expected behavior
Should populate the entity
Additional context
I though it might be due to having mongoose installed as well and possibly a package collision or something. I removed the mongoose package and the same issue still appears.
Versions

Dependency
Version

node
v12.16.3

typescript
4.1.3

mikro-orm
4.3.4

your-driver
4.0.1

Interesting issue, you should use the one from @mikro-orm/mongodb, that way the type is correctly inferred - but due to a bug in mongo driver, it was actually working only when it was not inferred correctly.
Btw you have misconfigured entity definition, you are missing wrappedReference: true (this is automatic only when using ts-morph):
  @ManyToOne(() => MyEntity, { fieldName: 'asdf', wrappedReference: true })
  entity: IdentifiedReference<MyEntity>;

Interesting issue, you should use the one from @mikro-orm/mongodb, that way the type is correctly inferred - but due to a bug in mongo driver, it was actually working only when it was not inferred correctly.
Btw you have misconfigured entity definition, you are missing wrappedReference: true (this is automatic only when using ts-morph):
  @ManyToOne(() => MyEntity, { fieldName: 'asdf', wrappedReference: true })
  entity: IdentifiedReference<MyEntity>;


Add optional custom frontmatter meta description for SEO and Open Graph

I appreciate the new SEO merge in #143. An additional feature I thought useful would be a custom meta description frontmatter field for SEO and Open Graph purposes. I'm relatively new to Gatsby, so I hope this PR is useful and not interfering with anything under the hood. Cheers!
Changes proposed:

To src/pages/index.js and src/templates/blog-post.js add description to frontmatter query
Add short-circuit evaluation for frontmatter.description in posts mapping (index.js) and SEO component (blog-post.js)
Add description field to hi-folks/index.md to demonstrate use

Hey @nielsenjared
Thank you for opening this. We've moved our official starters over to our monorepo to help ease maintenance (https://github.com/gatsbyjs/gatsby/pull/10310). Closing this since this repo is now a read-only mirror.
Please reopen this PR against the monorepo if you'd like.


StaticArray for Pointer

I tried to initilize a StaticArray of UInt8 pointer and the compiler send me this message :
Nil assertion failed (NilAssertionError)
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
from ???
Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/crystal-lang/crystal/issues
And source code is simple :
static_parsed = uninitialized Pointer(UInt8)[3]

Hi! I can't reproduce it. It also compiles fine in play: https://play.crystal-lang.org/#/r/7j6u
Crystal version? OS?

Crystal 0.30.1 for Linux Mint

That line is the entire program that's crashing for you?

Yes but if I call it in no lib func it's fine

Can you give a minimal example that reproduces? Note since it's a compiler error the binding can be made up since it doesn't even get to the linker stage.

fun expr_parser = cr_expr_parser(str : UInt8*) : UInt8
	expr = String.new(str)
	parsed_array = [] of String

	if valid_expr(expr) != Expr_error::Good
		return 1_u8
	end
	parsed_array = cutout_expr(expr)
	static_parsed = uninitialized Pointer(UInt8)[3]
	return 0_u8
end

the principal function of my binding C

static_parsed = uninitialized Pointer(UInt8)[3]

This line raise a very very strange Nil Unknown error...I gess

This was not a reproducible example though, there were still lines you didn't gave (I commented out the function calls you didn't define)
No error here https://carc.in/#/r/7j9p
With the following code:
fun expr_parser = cr_expr_parser(str : UInt8*) : UInt8
  expr = String.new(str)
  parsed_array = [] of String

  # if valid_expr(expr) != Expr_error::Good
  #   return 1_u8
  # end
  # parsed_array = cutout_expr(expr)
  static_parsed = uninitialized Pointer(UInt8)[3]
  p! static_parsed
  return 0_u8
end

expr_parser("hh".to_unsafe)

sidenote for @asterite: would be nice to avoid having to call String#to_unsafe manually here

Thank it's fine now but I have a other probleme with StaticArray return...so can I open an other issue ?


Ability to clone data posted on Issue Tracker / Other Github data of interest

Just like the clone feature, it would be good to be able to programmatically clone other data that is associated with the repo on the Github interface. The best example would be a clone of the issue tracker. The main two motivations are (1) data preservation and (2) convenience for offline analytics for pedagogical research. I imagine it follows the same API as repomate:
repomate clone-issues -mn week-X -sf students.txt -status [all | open | closed ] -filter [regex]
Food for thought :)

You mean like list-issues but saving the files?

:) perfect! I'm off to RTFM :)

Well, list-issues only lists issues currently (optionally with the body), and that does not include comments. It's mostly meant to be used for quickly checking which issues exist for just a few students/weeks, and sometimes what's written in them (for example if you want to check the feedback given to some students). It's not well suited for large queries as the output will be a bit hard to read, and the fact that the REST api requires one to make at least two queries per repo to get all issues.
There's definitely a use-case for being able to save all issues together with all related comments, in a local database (maybe just a JSON file). Two key difficulties come to mind.

With PyGithub (using the REST API), querying for all issues and comments for N different repositories requires a minimum of some 2*N queries (one query for the repo, at least one query for the issues related to it).
If we also want the comments for each issue, then that's +1 query for each issue that's found.

If you do that at a large scale (say a couple of hundred repos with a few issues each), that's gonna be thousands of queries.
This could potentially be implemented with just one or two queries to the GraphQL API, though, but that requires some extra stuff that's not particularly relevant to the core of Repomate. One way forward may be to split the data sciency functionality off into a separate GraphQL-dependent part, and extend the plugin system such that it can be seamlessly integrated with Repomate.

Yup, I have some sketchy scripts that just used the Python requests library and the API endpoints :-\ and there was a fair wait for 12 weeks of issues x 140 ish active students :)
Agree that this is not core, but the use case for analytics is still strong, and decent excuse to dive into GraphQL API. Perhaps a longer discussion on that could be good to have :)

Yes, we could definitely have a look at more use-cases. I think the gathering of issues/comments itself is a strong one. If we can find some more use-cases where there is a demand for extensive data gathering that does not involve cloning repos, then we could warrant an excursion into the GraphQL API.
I think extending the plugin system enough to allow for new top-level commands really wouldn't be too hard, so it would be feasible to create a separate data science plugin for Repomate.

This is pretty much covered by the --hook-results-file option to list-issues.
The now relevant topic is what data to fetch for issues, see #329


Can't git clone the project

I try to git the project.  I'm not sure what's the problem of the git server.

Has anyone used this command to download it recently?

Can you try with https?
git clone https://github.com/hpcn-uam/Limago.git --recursive

Can you try with https?

git clone https://github.com/hpcn-uam/Limago.git --recursive

Yes, but the result is same.
Can't remote. >_<

Dear Mariodruiz,
Oh, It's work in the morning.
Many thanks!


v0.20.3-ls28 Docker image, SPEEDTEST_SCHEDULE not working

Describe the bug
Docker run setup of the latest version does not accept the SPEEDTEST_SCHEDULE environment variable. Please see yellow markings in the screenshot below.
Without it, the Docker run command perfectly works and spins up the container. (The error message doesnt match the cause i.e. the environment variable, as it looks to me)
To Reproduce
Steps to reproduce the behavior:

Expected behavior
Spin up the container, do a speedtest hourly at 15 min past
Environment (please complete the following information):
see screenie

See if wrapping your cron in double quotes fixes it, "15 * * * *" \

Yes!
Works perfectly now.
Thank you, Alex!


Unknown compression type for exr

Godot version
4.1
System information
Manjaro
Issue description
Attempted loading an exr obtained from polyhaven: https://polyhaven.com/a/lauter_waterfall
Specifically, the 4k Varient, got the following error:
modules/tinyexr/image_loader_tinyexr.cpp:70 - Unknown compression type.
core/io/image_loader.cpp:101 - Error loading image: res://textures/lauter_waterfall_4k.exr
Error importing 'res://textures/lauter_waterfall_4k.exr'.
Steps to reproduce
Install godot from official arch repositories
download above exr file, import and observe error
Minimal reproduction project
hello-godot.zip

Image information:
$ identify lauter_waterfall_4k.exr
lauter_waterfall_4k.exr EXR 4096x2048 4096x2048+0+0 16-bit RGB 13.0176MiB 0.000u 0:00.000

$ ffprobe lauter_waterfall_4k.exr
Input #0, exr_pipe, from 'lauter_waterfall_4k.exr':
Duration: N/A, bitrate: N/A
Stream #0:0: Video: exr, gbrapf32le, 4096x2048 [SAR 1:1 DAR 2:1], 25 fps, 25 tbr, 25 tbn

I also have this problem with Godot 2.3.1 and can not use normal map files

I meet this problem, too.
My textures is provided by Poly Haven's blender plugin.

polyhaven by default now downloads .exr file type for normal maps
download i.e. .png which is supported by Godot


Remove CLS implementation

Description
Changes proposed in this pull request:

Remove CLS implementation as it's been decided not to use it.

/retest

/retest

/retest


[WIP] Implement exercise sublist

Implement Sublist Exercise

Why?
This is not currently available in the Ruby track

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (non-breaking change which adds functionality)
[ ] Breaking change (fix or enhancement that would cause existing functionality to change)
[ ] Gardening (code and/or documentation cleanup)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change relies on a pending issue/pull request
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Let us know if you have any questions or need any help @rnice01
We can also be found in the gitter chat room: https://gitter.im/exercism/ruby

How are you getting on with this @rnice01? Need any help?

Hey @Insti thanks for checking in, I think I'm ok for now. I have gotten busy with work and my family, so I'm hoping to get more progress this week.

There's been no progress on this for a while so I'm going to close it.
Feel free to open it again if you've made progress and want to continue, otherwise we'll leave it available for someone else to implement.


Uploading large images is timing out

Hello,
I'm not sure if this is a bug with guzzle or my environment.
When I upload large images it hangs at a certain number of bytes every time.
More details can be found here http://stackoverflow.com/questions/30332441/cant-seem-to-upload-large-files-with-php-guzzle
Thanks

Can you set debug to true when sending the request? If that doesn't reveal anything, then perhaps take a look at a tcpdump of the transfer.

@mikeklein13 ping


ISPN-6950: Errors thrown with timestamp index in H2.

https://issues.jboss.org/browse/ISPN-6950

@gustavonalle There shouldn't be any output on the query tests now. The h2 drop syntax was different, hence the problems when dropping the table.

Looks good.

Integrated. Thanks @ryanemerson !


Specify that the controllers developed for the API will not have network access.

@thomasyu888 Does the commit a2afab3d53790fe1ea5377dc910c144a397e3332 solves this request?

@thomasyu888 This requirement will be described on the NLP Sandbox website.


Enrollment

Name: Jau'n Wheatley
GitHub: @datwheat
Twitter: @datwheat
Website: datwheat.github.io

Completed Courses
none

Course
Files

Course Name
link

Hey, @datwheat !
Can you create your profile in that way?
It's much more easy now! :smile:
Thanks


accessing to APIs from back-end instead of  making AJAX request

I have a question about schema
I know that I can send AJAX request from front-end like:
var user = {email<EMAIL_ADDRESS>password: '1234'};
AJAX('POST /api/login/', user, function(response) {...});

and I know that this request will responsed by something in back-end like:
F.route('/api/login/', json_exec, ['*Login', 'post', 'unauthorize']);
function json_exec(id) {
        this.id = id;
	this.$workflow('exec', this.callback());
}

note: login is a schema that has email and password fields (like user variable that sends from front-end)
now, my question is how can I do same thing from back-end (instead of sending AJAX request), for example when user type some url like: "http://mysite/{email}/{password}" in browser's url bar the same thing happend (when user sends that AJAX from front-end)
now, I know that I should first write this code:
F.route('/{email}/{password}', me, ['unauthorize']);
function me(email, password){
        this.$workflow('exec', this.callback()); // I wanna the same thing like func() function and then get the result(in this $workflow I have SUCCESS(true) at the                                                                                // end  ) then show a new html page based on result
        //an else if lines of code to show new html page based on result of above (this.view('...'))
}

note: and this is login schema:
NEWSCHEMA('Login').make(function(schema) {

	schema.define('email', 'Email', true);
	schema.define('password', 'String(30)', true);

	schema.addWorkflow('exec', function(error, model, options, callback, controller) {
		var user = F.global.users.findItem('email', model.email);
		if (user && user.password == model.password.sha1()) {
			if (user.blocked)
				error.push('error-user-blocked');
			else {
				controller.cookie(F.config.cookie, F.encrypt(user.id + '|' + controller.ip + '|' + F.datetime.getTime()), '1 month');
				NOSQL('users').counter.hit('all').hit(user.id);
			}
		} else
			error.push('error-user-credentials');
		callback(SUCCESS(true));
	});
)}

thank you

Hi @tahavs,
I don't know which version of Total.js you use but this will work on the latest version:
ROUTE('/{email}/{password}', me, ['unauthorize']);

function me(email, password){
   var model = CREATE('Login');
   model.email = email;
   model.password = password;
   model.$controller(self);
   model.$workflow('exec', this.callback());
}

thank you @petersirka , that was very helpful
I have another problem too, I wanna use response from schema.
when I use AJAX, server sends me SUCCESS(true) as response, then I can decide which page should I show to user, now I wanna get that SUCCESS(true) inside me() function, then decide which page I should show to user by using this.view('htmlPage') inside an else if based on that response, something like this:
function me(email, password){
    var model = CREATE('Login');
    model.email = email;
    model.password = password;
    model.$controller(self);
    model.$workflow('exec', this.callback());
    **// else if code based on response of schema condition, then I consider which page should I show to user**
}

node: I mean this code about response:
AJAX('POST /api/login/', user, function(response) {
	SET('user.response', response);
	**response.success** && location.reload(true);
});

that because  of callback(SUCCESS(true)); line inside of login schema

function me(email, password){
    var self = this;
    var model = CREATE('Login');
    model.email = email;
    model.password = password;
    model.$controller(self);
    model.$workflow('exec', function(err, response) {

    	if (err) {
    		// invalid credentials
    		self.view('something');
    		// or self.throw401();
    		// or self.json(err);
    		return;
    	}

    	// response.success
    	self.view('htmlPage');
    });
}

thank you @petersirka, it works


matchExact does not work if text has period

I am not able to do retrieve data with a matchExact call on a username field of type text having a period in it, it return 0 entries. e.g username = "first.last"
little node js snippet show usage:
query = query.and((search) =>
search.where("username").matchExact(req.query.username)
);
if the username does not contain a period, it works well.
It seems the issue stems from the fact that the search is interpreting the dot as a nest field of the parent field. If that is the case, how do I escape characters or tag and attribute in the schema?

I suspect you have specified this field as a 'text' field and not a 'string'. 'text' fields are optimized for human-readable text. Like paragraphs of text, product descriptions, or song lyrics. As such, certain common words like a, an, the, or, etc. (i.e. stop words) are not searchable. And, punctuation is not searchable. The period is certainly being seen as punctuation in this case.
The better solution would be to set the username as a 'string'. This sets it up as a TAG field in RediSearch and will allow you to simply do this:
search.where("username").equals(req.query.username)
In future version, I plan to have Redis OM index all 'string' fields as both TEXT and TAG in RediSearch as this confusion happens quite a bit.

Newer versions of RediSearch support wildcarding. You should be able to use a * with a 'string'.
For example:
search.where("username").equals("bob*")

Understood, thanks again. Great library btw.

Thanks and glad you like it. Sorry for taking so long to respond to this. Been on the road a lot the last several weeks.

No worries at all


Gatekeeper

This pull requests add a Gatekeeper Listener co-process that is able to bootstrap machine registration without manual intervention and prompting. This is useful in a continuous integration environment where servers may scale in or out automatically.
This pull requests contains code for bootstrapping AWS machines, based on their Instance Metadata and their Instance Identity Docs. AWS  provides signed metadata information, which can be used for instance verification.
The Gatekeeper listens on "<IP_ADDRESS>:8200" by default.
Gatekeeper Commands
torus gatekeeper start -- start the Gatekeeper listener
torus gatekeeper status -- get the status of the Gatekeeper listener
torus gatekeeper stop -- stops the Gatekeeper listener
Bootstrapping Commands:
torus bootstrap -- bootstrap a machine into the Gatekeeper listener.
The torus bootstrap command must be run as root, since it writes to /etc/torus
bootstrap takes additional arguments:
--url (required): Gatekeeper URL
--auth (requied): Gatekeeper authentication type to use (valid: aws)
--org (optional): Organization to bootstrap to
--role (optional): Team/Role to bootstrap to
--machine (optional): Machine name to provision
Future Work
There is an opportunity for future work here, particularly around other cloud providers. For AWS, some additional verification that the server is running, and a comparison on instance running in a provided AWS environment would help Torus ensure the server actually exists. Additionally, SSL on the Gatekeeper may be worthwhile since it passes a machine secret to the server.
Added Dependency
Added the dependency github.com/fullsailor/pkcs7 which is used for signature validation from AWS.
Issue Dependency
This pull request relies on #235

Closing in favour of #242.


⚠ Jenkins Plugins Artifactory has degraded performance

In a2e4d69, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1487 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 343f49d after 12 minutes.


mat: simplify redundant selector expression

Please take a look.

Codecov Report

Merging #1711 (c8204df) into master (af39aeb) will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1711      +/-   ##
==========================================
- Coverage   73.26%   73.09%   -0.18%
==========================================
  Files         510      493      -17
  Lines       60132    59555     -577
==========================================
- Hits        44056    43532     -524
+ Misses      13584    13558      -26
+ Partials     2492     2465      -27

Impacted Files
Coverage Δ

internal/asm/c64/conj.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/asm/f64/scal.go

...sp/fourier/internal/fftpack/array_bounds_checks.go

internal/asm/f64/stubs_noasm.go

graph/internal/set/same_appengine.go

internal/asm/f64/ge_noasm.go

mat/offset_appengine.go

internal/math32/sqrt.go

internal/asm/c128/stubs_noasm.go

internal/asm/f64/axpy.go

... and 8 more

Continue to review full report at Codecov.

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


[question]: Why is accessing a node variable not allowed in text templating

Consider the following:
a: "asd"
b: "ggg"

#@yaml/text-templated-strings
all: "(@ .a + .b @)"

I would have liked to have access to the current document values a and b somehow to be able to use them in text templating. Why was that design choice ctaken to dissallow this? Whats the reason behind?

I cannot speak to the design, but in the meantime, if you are stuck on this, overlay.replace could help you to merge a and b into all.

Right: there is no "this" reference.
@cppforlife can elaborate on the thinking behind this specific instance.
In general, I'll say that there's a deliberate attempt to maximize expressiveness while keeping the underlying mechanisms as simple as possible.
There are a couple sources of complexity in such expressions:

it introduces a path-expression syntax that doesn't currently exist in the tool (so far, all referencing use-cases have been met with structure "expressions" (i.e. through a YAML structure itself);
in YAML, keys can be values other than strings; the syntax would need to support such cases;
such syntax would also need to support indexing into array values;
it wouldn't be hard to write cycles (e.g. a: (@= .all)); these would need to be detected or execution timed-out.

In addition, there's a "workaround" that seems reasonable (at least for this example): we're authoring a template, so we're aware that a: and b: have values that are desired elsewhere. Those values can be hoisted to variables and referenced; ensuring a DAG.

In terms of a solution, @mamachanko rightly pointed to Overlays as a way to update the value of all: with an expression. It gets a little complicated because the @overlay/replace needs to be placed on the lowest-common ancestor of the node receiving the evaluated expression and the nodes supplying values. Overlays also represent a level of indirection: it's a patch applied after the fact and there's no evidence of it in the original template; one needs to know that the overlay is being applied, somehow.
If it's possible, templating is preferred.
#@ a = "asd"
#@ b = "ggg"

a: #@ a
b: #@ b
all: #@ a + b

Notes:

when supplying a value for a map item, there's generally no need to create the string in a text template.
if a template is desired, check out string.format().
text templating is required when the key for a map item is an expression.

@gabyx — if you have any follow-up questions on this topic or have a suggestion, please feel free to comment on this issue.
In the meantime, closing.


Buffer() is deprecated due to security and usability issues

I have created a function to parse the multipart/form-data.
export function parseMultipartFormData(event) {
  return new Promise((resolve, reject) => {
    const busboy = new Busboy({
      headers: {
        'content-type': event.headers['Content-Type'],
      },
    });

    const body = {};

    busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
      body.filename = filename;
      body.encoding = encoding;
      body.mimetype = mimetype;

      const buffer = [];

      file
        .on('data', (chunk) => {
          buffer.push(chunk);
        })
        .on('end', () => {
          body.documentContent = Buffer.concat(buffer);
        });
    })
      .on('field', (fieldname, val) => {
        body[fieldname] = val;
      })
      .on('finish', () => {
        resolve(body);
      })
      .on('error', (err) => {
        reject(err);
      });

    busboy.end(event.body);
  });
}

When I execute this function I am getting a warning,
(node:8) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Currently I am using the version ^0.3.1
Could you please suggest me how to solve it? I have tried using a stream directly still I'm getting this warning.

#223

This was fixed in https://github.com/fastify/busboy

This is now solved in two ways:

dicer has updated its dependency on streamsearch
busboy v1.0.0 will no longer depend on dicer


Add startRound code.

This code is responsible for starting a new prediction round and recording its details in the rounds data structure. The StartRound event can be used to notify external parties or interfaces about the beginning of a new round.
event StartRound(uint256 indexed epoch);

function _startRound(uint256 epoch) internal {
    Round storage round = rounds[epoch];
    round.startTimestamp = block.timestamp;
    round.closeTimestamp = block.timestamp + 5 minutes;
    round.epoch = epoch;
    round.totalAmount = 0;

    emit StartRound(epoch);
}

#32


[COR-831] Add integration tests for Trunk Registry

Add integration tests for publish, download, and all other registry routes.
From SyncLinear.com | COR-831

I was able to add a test for token generation, but ran into an issue while testing publish. Testing publish requires sending a Multipart request, and it looks like that is not possible with the testing framework we're using: https://github.com/actix/actix-web/discussions/2512
I'll reach out in their discord channel to get more info. In the meantime, we'll write simpler tests for each route. We'll need to come back to this and improve on these when we know more about using Multipart with TestRequest.

A few potentially useful links:

https://docs.rs/actix-multipart-rfc7578/latest/actix_multipart_rfc7578/client/multipart/struct.Form.html
https://discord.com/channels/771444961383153695/1071055130734104577/1071802300697423973

Added some simpler tests in https://github.com/CoreDB-io/trunk/pull/132. Moving this to 'To Do' for now.


StyletronProvider issue with TypeScript 2.4.1

After upgrading to TypeScript 2.4.1 (from 2.3), I started getting this error:
ERROR in [at-loader] ./src/framing.tsx:43:9
    TS2605: JSX element type 'StyletronProvider' is not a constructor function for JSX elements.
  Types of property 'setState' are incompatible.
    Type '{ <K extends never>(f: (prevState: void, props: { styletron: StyletronServer | StyletronClient; }...' is not assignable to type '{ <K extends never>(f: (prevState: {}, props: any) => Pick<{}, K>, callback?: (() => any) | undef...'.
      Types of parameters 'f' and 'f' are incompatible.
        Types of parameters 'prevState' and 'prevState' are incompatible.
          Type 'void' is not assignable to type '{}'.

The code that it comes from is fairly mundane:
import {StyletronProvider} from 'styletron-react';
// ...
const styletron = new Styletron();
const root = (
    <StyletronProvider styletron={styletron}>
        {elem}
    </StyletronProvider>
);

Based on my understanding of TS 2.4, I think this should be updated like this:
declare namespace StyletronReact {
-  export class StyletronProvider extends React.Component<{styletron: StyletronServer | StyletronClient}, void> {}
+  export class StyletronProvider extends React.Component<{styletron: StyletronServer | StyletronClient}, {}> {}

  export function styled<TProps>(
    base: React.StatelessComponent<TProps & ClassNameProp>,
    style: Style<TProps>): Component<TProps>;

Based on React's type definitions, it would seem that state cannot be void.
I haven't tested whether this change would negatively impact TypeScript 2.3 or earlier.

I'm getting the same issue with 2.5.8 and TypeScript 2.4.1. Tried downgrading to TypeScript 2.3 and got the same issue.

It's due to some changes in latest @types/react. It's actually patched in master for the next major release. But we should definitely patch this for v2 too. This version is working for me though<EMAIL_ADDRESS>

@faddee Those types work for you on which version of TypeScript and styletron?

<EMAIL_ADDRESS>and<EMAIL_ADDRESS>There's a fix coming soon for v2.

Released in<EMAIL_ADDRESS>Thanks @faddee!


Don't mount /tmp as host volume in example pod

We're encouraging using kubectl cp in the meantime, using host mounts
was a temporary measure.
This fixes an issue where the documented kubectl cp command is
bringing in tarballs from previous runs.

/lgtm

oh I don't have permissions to stamp 🤷♀

oh wait nvm, there just isn't a bot to recognize /lgtm I guess

b/c we don't have turnkey validation in place I'll say ok-to-self merge on test verification.


Bug: List Log GUI default behaviour [Look at description]

This may or may not be a bug depending on how you look at it. When logs is used to list out the logs, the default behavior is that no session logs are focused on and users have to manually have to click on one of the logs to get it focused before being able to use the arrow keys [UP and DOWN] to move.
I think that ideally, we should have the first indexed log to be focused so that users do not have to use the mouse to click on the logs before navigating it with arrow keys.

Closed, listed as known issues in UG.


Do not show full changelog in package control update summary

As title says, do not show changelog for all versions when updating plugin, it's a pretty big list

Why does it matter that it's a big list?

First, annoying a little, I didn't remember what the previous version was, also there are sometimes other changelogs under, and I need to scroll to them. Is this hard to do?


offline mode: when I navigate to another route, it shows nothing, but when I refresh then it shows the page from cache

Help needed!
Am facing some issues,
when I switched to offline via network tab
(1) when I navigate to another route, initially it shows the blank page, it works only on refresh
(2) Images are not showing, they are not from assets but hosted on another domain, like for ex - https://images.delta.com/name_of_image.jpg
and my configurations looks like this
pwa: {
   strategies: 'generateSW',
   injectRegister: 'auto',
   registerType: 'autoUpdate',
   manifest: {
     name: "Testing app",
     short_name: "Testing app",
     start_url: "/?pwa",
     scope: "/",
     icons: [
         {
             src: "maskable_icon.png",
             sizes: "384x384",
             type: "image/png",
             purpose: "maskable"
         },
         {
             src: "android-chrome-192x192.png",
             sizes: "192x192",
             type: "image/png",
             purpose: "any"
         },
         {
             src: "android-chrome-256x256.png",
             sizes: "256x256",
             type: "image/png"
         },
         {
             src: "icon.png",
             sizes: "512x512",
             type: "image/png"
         }
     ],
     theme_color: "#245BA7",
     background_color: "#000000",
     display: "standalone"
   },
   workbox: {
     navigateFallback: '/',
     globPatterns: ['**/*.{js,css,html,png,PNG,svg,jpg,JPG,jpeg}']
   },
   devOptions: {
     enabled: true,
     type: "module",
     navigateFallback: '/',
   }
 }

also I pre-rendered the nitro routes
// pre-render the routes
async 'nitro:config'(nitroConfig) {
   if (nitroConfig.dev) { return }
   const getMenuItem = useMenuNavigation()
   const response = await fetch(`${process.env.INTERNAL_API_URL}/api/globalMenuItems`)
   const { main_menu } = await response.json()
   if (nitroConfig.prerender && nitroConfig.prerender.routes) {
     nitroConfig.prerender.routes?.push(...main_menu.map((item) => getMenuItem(item).link))
   }
   console.log(nitroConfig.prerender, 'nitroConfig.prerender')
 }

Sorry, if something I missed or wrong, Can anybody please help ?

If using SSR app, you will need to prerender index.vue page (that's the fallback page /). Check https://github.com/vite-pwa/sveltekit/issues/65#issuecomment-1734213190 .

@userquin
did you mean pre-render the / page ?
prerender: [
    routes: ['/']
  ]

yes

@userquin I already added the dynamic routes, so it will be pushed to prerender.routes
async 'nitro:config'(nitroConfig) {
   if (nitroConfig.dev) { return }
   const getMenuItem = useMenuNavigation()
   const response = await fetch(`${process.env.INTERNAL_API_URL}/api/globalMenuItems`)
   const { main_menu } = await response.json()
   if (nitroConfig.prerender && nitroConfig.prerender.routes) {
     nitroConfig.prerender.routes?.push(...main_menu.map((item) => getMenuItem(item).link))
   }
   console.log(nitroConfig.prerender, 'nitroConfig.prerender')
 }

okay @userquin I will make it
if I wanted it to redirect to simple offline.vue file which normally says( your are offline now ) instead of default offline page
what should I do for that ?
it would be very helpful if you can answer it
my client needs it urgently for a quick fix
Much thanks @userquin

If you want to redirect to custom page (pages/offline.vue, you need to prerender the route, otherwise you will get infinite redirection when offline. In the runtime caching just redirect to that page route (/offline)

@tamanna-makkar check https://github.com/vite-pwa/vite-plugin-pwa/discussions/734


There are no strings.h in MSVC.

strings.h used in abnf2pegtl.cpp example, but it absent for MSVC.

True, we didn't build the examples on MSVC so far. I'll enable them and see what other issues come up and how to fix them.

All examples now build cleanly on Windows.


insensitive search for asset

Intended to provide a fix for #18
This might have some performance issues. The other way we could look at this is by making sure that asset lowercases the fields that are searchable by default. We could also look at asset creating a case insensitive index such as mentioned here https://docs.atlas.mongodb.com/schema-suggestions/case-insensitive-regex/

Yeah this is all very ropy anyway and needs a proper review of how to do this in future.


CM-85 Remover validacion de parametros vacios en busqueda de bovinos

Refer to this link for build results (access rights to CI server needed):
http://utils.brazilsouth.cloudapp.azure.com:8080/job/ci/job/continuous-integration-bovine/28/


[Feat]: Making the contributors section dynamic.

Is your feature request related to a problem? Please describe.
Making the contributors section dynamic in Readme.md
Describe the solution you'd like.
The svg will look something like this with limit upto 54 contributors & will move from right to left.
Describe alternatives you've considered.

Show us the magic with screenshots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

\assign

/assign


"404 Not Found" returned on JS download attempts of recently compiled patches

The requested js files do not seem to exist on the server under the provided URL.
The only other patch I have was compiled about 2 months ago and download behaves as expected, so this issue only appears to apply to the recently transpiled patches but irrespective of which download method I try:

using the provided javascript

@tamasmarquetant thanks for letting us know. That issue has been fixed. Try uploading the patch again.


Add compat for older functors version

Codecov Report
Base: 84.75% // Head: 84.75% // No change to project coverage :thumbsup:

Coverage data is based on head (42f29d3) compared to base (da8ff23).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   84.75%   84.75%
=======================================
  Files           8        8
  Lines         433      433
=======================================
  Hits          367      367
  Misses         66       66

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


java.lang.NoSuchMethodError when calling .toList() on CoroutinePublisher

When I try to get a list of items from collection.find() method on a CoroutineCollection instance, I'm getting an exception:
Exception in thread "scheduledTasks" java.lang.NoSuchMethodError: kotlinx.coroutines.channels.LinkedListChannel: method 'void <init>()' not found
backend_1         | 	at kotlinx.coroutines.reactive.SubscriptionChannel.<init>(Channel.kt:51)
backend_1         | 	at kotlinx.coroutines.reactive.ChannelKt.openSubscription(Channel.kt:31)
backend_1         | 	at kotlinx.coroutines.reactive.ChannelKt.openSubscription$default(Channel.kt:30)
backend_1         | 	at org.litote.kmongo.coroutine.CoroutinePublisherKt.toList(CoroutinePublisher.kt:63)
backend_1         | 	at org.litote.kmongo.coroutine.CoroutinePublisher.toList(CoroutinePublisher.kt:48)
backend_1         | 	at personik.core.scheduledtasks.ScheduledTaskRepositoryMongo.getAndUpdateActiveTasks(ScheduledTaskRepository.kt:38)
...

The code that triggers the error looks like this:
return client.getDatabase(dbname).getCollection<ScheduledTask>("scheduledTasks").find(ScheduledTask::jobId eq jobId).toList()

be sure do use the same coroutine lib version than KMongo https://github.com/Litote/kmongo/blob/master/pom.xml#L78

Actually, this only manifests with kotlinx.coroutines version 1.4, when I roll back to 1.3.8 everything works fine.

next release of kmongo will use 1.4, for now you have to downgrade to 1.3.9 or lower

Have any update anymore there? So I need to use kotlin 1.3.9?

Have any update anymore there? So I need to use kotlin 1.3.9?

@sinhpn92  It should be ok with latest kmongo version - kmongo 4.2.3 uses coroutine 1.4.2

@sinhpn92  It should be ok with latest kmongo version - kmongo 4.2.3 uses coroutine 1.4.2


[Translate ru] Resources -> widgets

Check out a review request at GitLocalize

@agektmr Reviewed

:tada: This has been pushed live to https://developers.google.com/web/


Get pull request(s) API ignores includeCommits and includeWorkItemRefs parameters

https://docs.microsoft.com/en-us/rest/api/azure/devops/git/pull requests/get pull request?view=azure-devops-rest-5.1
I tried various combinations of "true", 1 and just leaving the parameter with no value, but the response from the API doesn't change.

Hi
Sorry to be sarcastic, but I found it amazing that it takes one year to mark this as a bug and almost four years without any update.
BTW, the same results for all API's versions, including 7.2.

I thought I was doing something completely wrong, but am amazed to see this bug has been here forever... Can confirm OPs observations. Please either adapt your documentation and remove the parameters or make them work.


brightbox: kubernetes bootstrap example

This is an example of Kubernetes cluster bootstrapping on Brightbox using Systemd sysext Ignition configuration.
:warning: This is really for demo purposes but it can serve as a foundation (for example don't pass the admin configuration through HTTP for workers to join)

[x] documentation
[x] get the kubeconfig as an output
[x] add outputs (IPs, etc.)

$ terraform apply
$ scp core@<IP from the output>:/home/core/.kube/config ~/.kube/config
$ kubectl get nodes
NAME                          STATUS     ROLES           AGE   VERSION
srv-cruzw.gb1.brightbox.com   NotReady   <none>          55s   v1.29.2
srv-fltor.gb1.brightbox.com   NotReady   control-plane   72s   v1.29.2
srv-gvzhx.gb1.brightbox.com   NotReady   <none>          59s   v1.29.2
srv-mipnf.gb1.brightbox.com   NotReady   <none>          60s   v1.29.2

⚠ This is really for demo purposes but it can serve as a foundation (for example don't pass the admin configuration through HTTP for workers to join)

This could be part of the README.md


Dice loss

Dear author,
Thank you for this excellent work. I have seen that you mentioned dice loss in the Presentation, but why is it not reflected in this repo? Looking forward to your reply.

Hi @tracob-github,
This repository is for the implementation of our paper in MICCAI:

Conditional Deformable Image Registration with Convolutional Neural Network
Tony C. W. Mok, Albert C. S. Chung
MICCAI 2021. eprint arXiv:2106.12673

In the original paper, the training is fully unsupervised; therefore, the dice loss is not included in this repository.

Thank you for your reply. I found that dice loss was added to Task1 and Task3 in your presentation. Did you add dice loss in the final implementation?

Looking forward to your reply.

Hi @tracob-github,
We did use dice loss during the learn2reg challenge in order to maximize the performance. For more details of our approach in the challenge, you may refer to our challenge paper at https://link.springer.com/chapter/10.1007/978-3-030-97281-3_23.

Thank you for your patient reply. And is it convenient for you to provide the source code with dice loss? I want to learn about weakly-supervised registration. Looking forward to your reply.

Hi @tracob-github,
Example of the dice loss https://github.com/voxelmorph/voxelmorph/blob/793b90fbddfd93f31019ecbb8c036454c6ee0720/voxelmorph/torch/losses.py#L79

Thank you very much. It helped me a lot.


[LA-315] Fix travis race condition (#2309)

If closing/reopening a PR fast, the reference of the PR stays
the same, but the merge commit sha will be different.
We don't use the fetched merged commit, but the TRAVIS_COMMIT,
which is sometimes out of date. Here, we'd fetch and checkout
the correct reference in PRs.

recheck_cit_xenial
recheck_cit_swift
recheck_cit_ceph

rebased

recheck_cit_all

CIT/ceph
CIT/swift
CIT/trusty-leapfrogupgrade-ceph
CIT/xenial-leapfrogupgrade-swift
are expected to fail. Some are addressed with this: https://github.com/rcbops/rpc-gating/pull/232.


Improve compatibility with Woodpecker CI

[ ] Updated Unreleased section in CHANGELOG or it's not notable changes.

https://woodpecker-ci.org/docs/next/usage/environment

Thanks👍

Hi, @lafriks! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨
We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.
Thanks again!


Level the playing field for navigation libraries

The "Navigation Between Screens" documentation was biased towards React Navigation. As per my chat with @cpojer, I’ve rewritten it to give all navigation libraries a fair crack of the whip. To include a new navigation library you add a row to the feature table at the top. I’ve picked features that give an architectural overview of navigation libraries.

Here's a screenshot:

@grahammendick I'll review the content but is it possible to make the table layout better?

@osdnk would you be able to review this as well?

Also, I see that these ticks and crosses are not black or white:
e.g. RNN does not work with Expo? How about detaching?
RN does not work for web? PR is pending and it's matter of few weeks
RN does not use Native API? How about react-native-screens?

Thanks @rickhanlonii  and @osdnk  for the great feedback. I wanted the documentation to be unbiased but I didn’t want it to be unopinionated. For example, I wanted to make it clear that React Router Native doesn’t pass the most basic test and so shouldn’t be used. But I can see that’s going to be too controversial.
Let’s change the format to match your suggestions. We can have a section for each library and reach out to library maintainers for their pitch/anti-pitch paragraphs. Are we all happy with that?

Good idea. I believe that the most important part is to explain motivations of these libraries, different approaches and problems solved with them.

I'm afraid it's not a good place for this discussion. Can you, @grahammendick, open RFC in https://github.com/react-native-community/discussions-and-proposals/issues and sum up some thoughts. I'll happily follow up with come more general comments and thoughts, but now I'm feeling that every comment here is considered as a "review" for your changes. Let's talk about overview of navigation patterns with any bias.

And for now, can we close this PR? Don't get me wrong, @grahammendick, but I'm afraid we're not ready to merge in current state even with every of suggestions here included.

No problem @osdnk. Thanks all for the suggestions


Get-PnPClientSidePage behaves differently in different tenants

I get an error while running the command  $page = Get-PnPClientSidePage -Identity Startsida
Get-PnPClientSidePage : After parsing a value an unexpected character was encountered: F. Path 'description', line 1, position 224.
At line:1 char:9

$page = Get-PnPClientSidePage -Identity Startsida

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : WriteError: (:) [Get-PnPClientSidePage], JsonReaderException
FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.ClientSidePages.GetClientSidePage

Whereas, the same command in another tenant works fine.

@NishaRaghu - I've tried on 3 different tenants with v1.5.25 and I have no issue at all.
Can you retry, or is there any difference between your tenants? Thanks.

@veronicageek Did you have the same permissions on all 3? If so what were yours roles in each? Are there any features that should be enabled? Or any other settings to consider before running the command? I have these modules installed:

Does it depend on the modules installed?

I was a Global Admin for testing but you may be able to do it with a SharePoint admin (haven't tested). However, you're using the old (and not maintained anymore) SharePoint PnP Module. The new module (this repo) is PnP.PowerShell.
documentation to install and authenticate are here: https://pnp.github.io/powershell/articles/installation.html

@veronicageek Extremely sorry for a delayed reply.
My client has a classic team site and the command did not work for a page in that environment. I tried running the same command connecting to the client's modern site collection and it worked.
I have uninstalled the old  version and then installed PnP.PowerShell. I haven't run the Register-PnPManagementShellAccess prior to connecting and I use -UseWebLogin while Connecting. The 'F' character is the outcome of the command Get-PnPClientSidePage.

@NishaRaghu

I have uninstalled the old version and then installed PnP.PowerShell. I haven't run the Register-PnPManagementShellAccess prior to connecting and I use -UseWebLogin while Connecting

I guess when you said "I tried running the same command connecting to the client's modern site collection and it worked" it was not with the PnP.PowerShell (new module) then? Can you confirm?
If this is correct, can you do your steps with the PnP.PowerShell module after running Register-PnPManagementShellAccess which is mandatory for this module. We need to make sure that it's installed/configured properly to be able to troubleshoot the issue because I have no problem on my side.
Thanks

@veronicageek I ran the Register-PnPManagementShellAccess before connecting to the client site and tried running the Get-PnPClientSidePage -Identity Startsida but unfortunately i get the same error. Some points i would like to make it clear:

The commands works in my development tenant environment when run against a modern team site collection
The command works in my clients environment if its a modern team site collection
The command does not work in my clients environment if its a classic team site with a modern homepage. Including any subsite.

@NishaRaghu - I know it's been a while, but can you provide an update please? Thanks.

Closing as no update.


is it possible to get layers activations using theano backend when traning?

i know theano backend actually creates gradient function for the backpropogation mechamism and the forward pass is also a function for the gradient. the forward pas functions inputs are actually placeholders not really real numbers. so one can only get layers activations after training by k.function and inputs real numbers into it. however since on each training batch there must have real numbers inputs flows back and forth in the gpu calculations for the backpropogation to work then why cannot get the layers activations on traning using callback fuction in theano backend?
appreciates!
Please make sure that the boxes below are checked before you submit your issue. Thank you!

[ ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

The keras backend has a function called K.gradients which will get you the gradient of a tensor with respect to a loss. Perhaps this what you're looking for.  examples/deep_dream.py demonstrates how to use this function.


tree-sitter/py-tree-sitter 0.22 API changes

The initialiser for tree-sitter languages no longer supports loading of object files, i.e. only Python bindings are supported going forward. In the short term, I suggest pinning the pip dependency for tree-sitter to 0.21.

Resolved after 0.5 release


Add a new CommitmentId field to the RD object for Elevate integration

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
New Field for Future Use:

npe03__Recurring_Donation__c.CommitmentId__c

Deleted Metadata

**lurch attach W-039004

Tracking W-039004


Change Crosswalk Version?

Hi,
thanks for this great plugin.
I use crosswalk for a while and test now your integration.
After playing around I ask me, how to change the Crosswalk version? I wanna use:
12.41.296.9
Is it possible to change this version for myself?
Regards

Changing 11+ with 12+ in this file libs/xwalk_core_library/xwalk.gradle should do the trick. (didn't tried it)
Also you can check the repo here https://download.01.org/crosswalk/releases/crosswalk/android/maven2/org/xwalk/ ...from what I can see/understand from the repo & xwalk.gradle file, the cordova-plugin-crosswalk-webview installs a beta version of the library, why?

Thanks for the hint!

FYI - just published a new version (v1.1.0) that sets the default version to crosswalk 13, and that makes the crosswalk version configurable via a <preference> (refer to README.md)

I've opened a feature request to crosswalk to made also canary version available
https://crosswalk-project.org/jira/browse/XWALK-3986
Vote it up please :)

@agrieve
Thats great! Thx!

Seems like we had the same idea. See #24 and https://crosswalk-project.org/jira/browse/XWALK-3987

My question about xwalk library version is still up!
To make it clear: the production ready version is the stable one that can be found here (11.x), right?
So, if someone installs this plugin and immediately runs cordova build android, then the app will use the latest beta version of the crosswalk webview (which is 13 in 1.1.0).
I think that the plugin releases should use only the latest stable version of crosswalk which is listed here.
I think that this setup may confuse some users that doesn't pay attention to other details when they build their apps.
Instead of doing this:
<!-- These are all equivalent -->
<preference name="xwalkVersion" value="org.xwalk:xwalk_core_library_beta:13+" />
<preference name="xwalkVersion" value="xwalk_core_library_beta:13+" />
<preference name="xwalkVersion" value="13+" />
<preference name="xwalkVersion" value="13" />

... I propose something like this:
<preference name="xwalkChannel" value="CHANNEL" />
<preference name="xwalkVersion" value="VERSION" />

... where channel possible values are: stable, beta, canary and version - the version number.
Default values: channel - stable & version - latest stable version (11 for now).

@TanaseButcaru - best answer and best "way to go" :+1:

Also, some little issues:

Right now when running cordova plugin add cordova-plugin-crosswalk-webview I get a "Not found" result.
If the project has more than one platform (ie: android & ios) the plugin is installed for all:
Uninstalling cordova-plugin-crosswalk-webview from android Uninstalling cordova-plugin-crosswalk-webview from ios

@TanaseButcaru , this should be an own ticket, we should not mix up everything now

@Hirbod you're right. I'll open another issue. :)


feature/SKED-2325: In IE it's impossible to upload the same image twice as profile picture

On hold, wait for changes in #1 to be approved.


merge from konsum-frontend-svelte,template-svelte-app

package.json

chore(deps): add ^9.0.1 remove ^9.0.0 (devDependencies.@semantic-release/commit-analyzer)
chore(deps): add ^10.0.2 remove ^10.0.1 (devDependencies.@semantic-release/release-notes-generator)

:tada: This PR is included in version 1.29.26 :tada:
The release is available on:

v1.29.26
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pr/41

Added inline and readonly features. Cleanup, bugfixing and version bump to 0.4.0.

Here goes the new features with the improved interface. Thanks again for the code @iyedb!


Add temperature and top-p

Doing this since Negin requires it for UnnaturalInstructions.
Also to get an idea of the best way to do #62 and #74.
This is just a stop gap before I get to those two issues.

Test with
from bespokelabs.curator import Prompter
from datasets import Dataset

dataset = Dataset.from_dict({"prompt": ["write me a poem"] * 10})

prompter = Prompter(
    prompt_func=lambda row: row["prompt"],
    model_name="gpt-4o-mini",
    response_format=None,
    temperature=TEMP,
)

dataset = prompter(dataset)
print(dataset.to_pandas())

Temperature 0.0
0  In the hush of dawn's embrace,  \nWhere whispe...
1  In the hush of dawn's embrace,  \nWhere whispe...
2  In the hush of dawn's embrace,  \nWhere whispe...
3  In the hush of dawn's embrace,  \nWhere whispe...
4  In the hush of dawn's embrace,  \nWhere whispe...
5  In the hush of dawn's embrace,  \nWhere whispe...
6  In the hush of dawn's embrace,  \nWhere whispe...
7  In the hush of dawn's embrace,  \nWhere whispe...
8  In the hush of dawn's embrace,  \nWhere whispe...
9  In the hush of dawn's embrace,  \nWhere whispe...

Temperature 1.5
0  In the hush of morning's light,  \nWhere whisp...
1  In the quiet hush of dawning light,  \nThe wor...
2  In a quiet grove where violets dance,  \nBenea...
3  In twilight’s hush, when shadows play,  \nThe ...
4  In a garden where the blossoms sing,  \nBeneat...
5  Beneath the sky, in twilight hues,  \nA whispe...
6  In the hush of dawn's embrace,  \nA gentle whi...
7  In the garden where the wildflowers sway,  \nU...
8  In a quiet glade where shadows dwell,  \nWhisp...
9  In the quiet of the dawn, when shadows blend w...


Scratch-style toolbox

Vertical toolbox
Horizontal toolbox is the same as before for simple, and with categories looks funny right now
Small problem with differing scale between block menu and workspace

Discussed offline! Although this code is still a heroic work-in-progress, we decided that we'll merge it now to start exposing bugs early in the process.

🎉 🐱 🍰 💥


name loggers

This pull request moves logging into loggers that are specific to the current __name__. This allows code that uses the API client to set the logging level for the client without affecting other logging happening on the root logger.

Please squash commits with each round of code review.

@jonparrott, what do you think of this?

This is fine, but we might need to attach a nullhandler to the oauth2client logger ala urllib3:
https://github.com/shazow/urllib3/blob/master/urllib3/__init__.py#L23

What dark sorcery is that?

It's not sorcery, it's just the right thing to do:
https://docs.python.org/2/howto/logging.html#configuring-logging-for-a-library

Huh. Well that seems... curious.
Should @emmett9001 amend his commit to satisfy that guidance? (I suspect yes; @emmett9001, what do you think?)

Yes, please.

Just to clarify - are you suggesting that this code should be added somewhere in my existing diff? It's not clear where this should go.

Yes, it should go in googleapiclient/__init__.py.

See my updated diff for the NullHandler logic.

Looks good, except you this line: https://github.com/shazow/urllib3/blob/f0d98555ce6e3eeb379a8a659dff9d7f9ff42348/urllib3/init.py#L53
Once that's in and you've squashed commits, this LGTM.

I messed up the history by rebasing and merging master, so I've reopened a clean version of this pull request as #206. Please comment further there and close this one if you'd like.

In the future please keep to one pull request for a desired change to the codebase; it makes for better forensics to have all the code review recorded in one place.

Will do. My previous pull request was corrupted by a bad merge - I'll be
sure to take care next time.
On Tuesday, March 22, 2016, Nathaniel Manista<EMAIL_ADDRESS>wrote:

In the future please keep to one pull request for a desired change to the
codebase; it makes for better forensics to have all the code review
recorded in one place.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/google/google-api-python-client/pull/174#issuecomment-200111176


OBSWebSocket is not a constructor error

Type of Issue:

e.g. Bug / Enhancement / Suggestion / Development Trouble
bug

General Description:

Details.
OBSWebSocket is not a constructor error when using code.

fixed! :)
By using var OBSWebSocket = require('obs-websocket-js').OBSWebSocket;
Documentation suggested a different way.
gg

Yeah whoops. I don't node good.


Added prometheus metrics for email analytics event processing

ref https://linear.app/ghost/issue/ENG-1508/add-custom-metrics-for-email-analytics-jobs

Added two new metrics for the email analytics jobs:

email_analytics_events_processed — a counter incremented each time an event is processed. Sometimes the event has already been processed in the past, so this doens't always result in a new event being stored in the DB.
email_analytics_events_stored — a counter incremented each time an event is stored in the DB. For example, if an email is opened 3 times by the same recipient, this counter will only be incremented once.

The metrics also have a label for the event type, so we can split out opened events from delivered events.

Closing to re-open on upstream branch


Shadow Update rejected: code 400 (Missing required node: state)

Hello,
I've integrated release 1.4.0 in my project, but I'm not able to update shadow documents.
For debugging, I reduced the content of the doc to minimum:
          shadowDeviceDoc = _pShadow->GetEmptyShadowDocument();
          std::unique_lock<std::mutex> block_handler_lock(_sync_action_response_lock);
          {
              _pShadow->UpdateDeviceShadow(shadowDeviceDoc);

              _pShadow->PerformUpdateAsync();

              _sync_action_response_wait.wait_for(block_handler_lock, shadow_action_timeout);

              ResponseCode rc = _sync_action_response;
              if (ResponseCode::SHADOW_REQUEST_REJECTED == rc) {

                  qDebug() << __FUNCTION__ << "Sync Action Failed";
              }
          }

AWS Console shows this error:
Shadow Update rejected {   "code": 400,   "message": "Missing required node: state",   "clientToken": "empty" }
Console output of doc content using rapidjson writer shows this -> doc DOES include "state":

{"state":{"desired":{},"reported":{}},"version":0,"clientToken":"empty","timestamp":0}"

Any ideas what I'm missing? Maybe there is an issue with encoding?

closing pending more info from OP.


Properly set sky_cover property of sky_material to Nil in setter

issue 74259 seems to be caused by being the value being set to "RID()" in the setter for the sky_cover property. I was able to fix this by setting the sky cover property back to Variant::Nil when it is reset in the editor.
I have tested this locally by running the engine before the fix, seeing the bug is present, then running it after the fix and seeing it works as expected.

My understanding of the engine is limited. If it is meant to be the case that RID() behaves as a nil value then I can make that change too

Noticed the formatting caused one of the tests to fail so I fixed it

Alright, I have updated the other sky materials, sorry it took a while for me to get back to this.

Looks great! The final step before merging is to squash all the commits together so that the whole PR only contains 1 big commit with all your changes. We like to merge one commit at a time to keep the git history clean and navigable.
If you don't know how to do that, we have a helpful tutorial in the official documentation https://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#the-interactive-rebase

Alright, The commit has been squashed

Cherry-picked for 4.0.4.


more downstream cost tests

added a irange in iarange test. while at it i also fattened the test_compute_downstream_costs_big_model_guide_pair test a bit

what would be the point of such a change? afaict the downstream formulation is more efficient in this case

what would be the point of [refactoring in terms of local_cost * upstream_grad?

No point, I'm just curious what it would look like. I thought that if both TraceEnum_ELBO and DiCE use the local_cost * upstream_grads formulation it might be nice to explore how that would work in TraceGraph_ELBO, but if there's no benefit then indeed we should avoid unnecessary change.


[WIP] Add YUIDoc to server.js

Woops, sorry I let this slip. If you rebase I can merge this in (even if incomplete).
Thanks!

bc8831f34cc803b8882df1f1444a91ff7750bc16 closes


Added AggregationVisitor, similar to QueryVisitor

This is awesome :+1: thank you @jonyadamit

Great! :)


Move block functions into sstable/block package

Purpose
This is the first of a series of PRs to organize the code into packages organized by problem domain. Similar to some of the work done here https://github.com/slatedb/slatedb/pull/346 but starting with sstable. Just as json module solves the problem domain of marshalling JSON into a []byte so the sstable package solves the problem of marshaling tables into sstable.
Implementation

Prefer package.Decode to Struct.Decode as it makes the package API clear when looking at the package structure.
Prefer passing in the item to be decoded like this package.Decode(*Target, []byte) as it is consistent with other marshalling libraries in golang like json.UnMarshall(*Target, []byte) and it can allow the golang compiler to avoid allocating the Target on the heap in some situations, thus reducing GC heap collection.
Added assert package as an experiment, if we don't like it, we can change it
Renamed SizeOfUint16InBytes to SizeOfUint16 as the sizeof is always referring to the size of bytes. (Same as in Rust, and C++, etc..)
Fixed panic in bloom filter if the filter is empty, added test TestEmptyFilter

TODO

Add some more tests to block package.

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


Two vars result in three allocations if captured in closure

Describe the bug
Two mutable variables allocates 3 times, while referencing only one mutable variable with a tuple of the same types allocates only once.
Two mutable variables calls swift_allocObject three times: https://godbolt.org/z/be86bEoEj
func makeClosureWithTwoVariables() -> () -> () {
    var a = 1
    var b = 2
    return {
        a += 1
        b += 1
    }
}

One mutable variables calls swift_allocObject one time: https://godbolt.org/z/63PTPeKE1
func makeClosureWithOneVariable() -> () -> () {
    var value = (1, 2)
    return {
        value.0 += 1
        value.1 += 1
    }
}

From looking at SIL & LLVM IR I think this is happening because:

each variable is allocated as a separate object
if we have only one variable, the one variable object can be passed directly to the closure
if we have two variables, we need a separate object that holds a references to the two variable objects as only one object can be passed to the closure

To Reproduce
Godbolt: https://godbolt.org/z/be86bEoEj
Expected behavior
The three allocations should be optimised into just one.
Environment

OS: macOS 12.4
Xcode Version: 13.3.1
also tested with nightly from May 27, 2022

rdar://125990589

I'm surprised there's any allocations at all given that a and b aren't actually used.

rdar://125990589


beforeChange event; event.preventDefault() to block slide change

Related to #1138

+1

Unfortunately this breaks #1179 as it seems to prevent events from propagating down into children slick sliders.

But events don't propagate down. I'll reply on #1179.

+1

Oh, this looks like a good addition...
@adam-lynch are you still intertested in having it merged? -- I can review it if you could rebase, and also create a quick JSFiddle demonstrating the functionality with a few  options / breakpoints? :) (use contributing.md one as base)

@adam-lynch I'll give you one last nudge, because this is highly requested.
If you can't do a rebase and add a JSFiddle, I'll have to close the PR :smile:

It's not that I can't, I don't want to. Will be moving away from slick.
Feel free to close :)
On Sat, Jan 23, 2016, 13:47 Simon Goellner<EMAIL_ADDRESS>wrote:

@adam-lynch https://github.com/adam-lynch I'll give you one last nudge,
because this is highly requested.
If you can't do a rebase and add a JSFiddle, I'll have to close the PR [image:
:smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/kenwheeler/slick/pull/1139#issuecomment-174187722.

Sure, that's a shame :disappointed:

:(

The missing jsfiddle and documentation was the only problem with this PR? We also need this feature and I am willing to reopen one in a proper way including the missing stuff.
@simeydotme If you have any other suggestions please let me know and I add them.

yes @cfoehrdes .. but I pulled @adam-lynch 's fork in to my own branch, and began testing and found many bugs regarding other options such as "asNavFor" ... I gave up with it because it was too buggy :(

Ok, I will give it a try. I will let you know if I was successful


fix(warning): added warnings_logger to display warnings

Fixes GitHub Issue - #902
Added warning logger in __init__.py file to display warnings to the console.
Alternative Solution:
Another approach for resolving the same issue is instead of using warnings.warn, use a logger object. Below is the snippet for the alternative.

I recommend using a warnings_logger and warnings module over the use of a simple logger. Please let me know if you think otherwise.

If you have no issue in using logger, then I am also fine with that. I will update the PR with logging instead of warnings.

Plus I have also noticed that the warning looks better with logging.

Closing this PR to keep the commit history clean. Raised PR #908 as discussed here.


How to verify ID token from SPA sent backend service?

Issue
I have the sample application working with your SPA and your API backend. Getting a validated Access bearer token.... all good.
The thing is that the UI is getting an id_token which is great b/c it has certain groups that my user belongs to. I'd like to send this token to my node api layer, and grant certain permissions based on this group association.
The problem is I have no idea how to verify this ID token.... if I pass it as bearer, it will not validate.... am I thinking about this the wrong way? I would think I would need to validate the ID Token's signature on the node side to make sure that it was a valid token... otherwise someone could forge the ID token (assuming they have a good access token) and get elevated privs.
I've read https://docs.microsoft.com/en-us/azure/active-directory/develop/id-tokens several times but it is unclear to me how to validate this ID token. Any help/guidance is appreciated to clarify my thinking.
This issue is for the sample

    - [ ] 1-1) Sign-in with Azure AD
    - [ ] 1-2) Sign-in with Azure AD B2C
    - [ ] 2-1) Acquire a Token and call Microsoft Graph
    - [X] 3-1) Protect and call a web API on Azure AD
    - [ ] 3-2) Protect and call a web API on Azure AD B2C
    - [ ] 4-1) Call a web API that calls Microsoft Graph on-behalf-of user
    - [ ] 4-2) Call a web API that calls another web API on-behalf-of user
    - [ ]   5) Deploy to Azure Storage and App Service

This issue is for a

    - [ ] bug report -> please search issues before submitting
    - [X] question
    - [ ] feature request
    - [ ] documentation issue or request

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior
A (server side) verified ID token
Library version
latest
Browser and version
node (web api), chrome browser
Mention any other details that might be useful

Thanks! We'll be in touch soon.

Hi @chadmott,
We don't send the id-token to another application to be validated. The id-token is only meant for the app that acquires it. In this case, you will send the access token to API to validate your permissions. The access token will include the groups that the user belongs to. Please check this sample single-page application calling Node.js web API using Security Groups to implement Role-Based Access Control for more information.


Double check hover and focus states on all links and buttons

Before we release we should double check all hover and focus states in the site to make sure we're meeting a11y needs and they are working as intended.

Things to make sure of:

[ ] does pink work for link color and contrast/color blindness issues
[ ] focus states, need to make sure they work for keyboard navigation
[ ] active state, do we want a press down type thing or something else?

We need to figure out links in the tools section:

Results of an a11y audit from Chrome Dev Tools:

[ ] Warning on contrast for the blue in the footer for the Github repo link
[ ] Add titles to the nav links
[ ] Add lang = en to the <html> tag
[ ] The pink links on the gray bring up a contrast warning (in the body of method details)

Alright, here are things that are done with some screenshots:

The pink now has a 2px underline and it has a small hover effect of a background color (the same pink just not as opaque)

The tools stuff is fixed up as well, the bold text looked a bit odd to me on my screen so I just mimicked the pink links but used white instead, not sure if the white background works or not for hover here tho:

I added a border on the cards for the methods on the home page, but wasn't sure if that works or not? But just thought a hover, focus, and active effect might be nice:

The buttons for phases are getting darker backgrounds when you hover and then a bit darker still when you press:

And all links, no matter where they are, when you click them have an active state like bocoup.com where they move down by 1px.

We're still getting a11y warnings about the contrast on the pink in general links and the blue in the footer for the link to the repo, but wasn't sure if we thought adding the border on the pink was enough? Thoughts welcome.

I think I'm worried about the pink links in these scenarios:

I have a hard time reading that on my crappy external monitor.

What if we used bolded black and pink underlines?
Sent from my iPhone

On Sep 1, 2016, at 11:46 AM, Susan Robertson<EMAIL_ADDRESS>wrote:
I think I'm worried about the pink links in these scenarios:
I have a hard time reading that on my crappy external monitor.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Black and bold with pink underline:

I really like the black bold and pink on hover:

Ooo, +1!

I think we're good here? If not let me know @iamjessklein.


[TECH ED] Join Codewars

Link to the coursework
https://www.codewars.com/join
Why are we doing this?
When you begin JS1 you will also begin Codewars, if you have not done so already. Prepare now by signing up to Codewars.
You will complete at least three kata a week for the rest of the course.

Read the CYF tutorial and code along with the example kata.
Link your account with Github, and join the MigraCode clan, so we can find you on the clan board here: https://www.codewars.com/users/Ali Ashraf/following
Add your Codewars username to the trainee tracker, so you can start building your milestone. If you don't add your username to the tracker, it can't track your progress.

By the end of JS1, you need to have completed nine kata to meet your milestone. Codewars is not just to practice JavaScript, it's to practice time management and study skills.
This has to be completed by the end of this module.
Maximum time in hours
1
How to submit
Add your username to your trainee tracker.

Kamino closed and cloned this issue to DianaRCruz/Coursework-Planner

Kamino cloned this issue to kostiantynkovalchuk/Coursework-Planner

Kamino cloned this issue to FabioMedinaD/Coursework-Planner


Can sombody show me how to apply my own TURN server?

Thank you Kevin Shen for sharing this great app source
I am making a Metaverse contents with SkyOffice code
But I stuck on Video streamming
Client don't show video on local IPs
Solving this issue, I wanna use coturn server
Can you teach me how to apply coturn server in SkyOffice code?

Finally, I succeeded to setup the coturn server
But.. sadly, even I setup the coturn server, Video and Audio don't work
Some guy told me to use HTTPS protocol for Video and Audio
How can I modify SkyOffice source for using HTTPS ?

I solved it myself
After changing HTTP to HTTPS, viseo and Audio works
Actually, we don't need to setup a TURN server


Updating okta_auth.py with debug information on verification title

In the case, the application title doesn't have the text 'Extra Verification' add a debug option to print to the console to figure out what it's being displayed instead.
I realized my app access was "locked". Adding a logging statement here helped me understand what was trying to be parsed. I think adding this could be a better error message rather than just "No Extra Verification", but let me know what you think! :)
It might help other users figure out what's wrong in their system since the error message is a bit generic: https://github.com/okta-awscli/okta-awscli/issues/197

Please update it using double quotes and will merge. Thanks for contribution

Please update it using double quotes and will merge. Thanks for contribution

Updated!


[docs] Provide Custom Component demo without styled-components

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/system/getting-started/custom-components/
Kind of issue
Other
Issue description
Installing @mui/system tells you that Emotion is the default styling engine. It is weird to me how a few demoes later on the Custom Components page, the demo is then written with Styled Components instead.
There should be more consistency in what is recommended, and the docs should adhere to that. Demoes using other tools can (and should of course) still be included, but right now it feels like the documentation tells you "here is what we recommend, and here is why you were wrong to follow our recommendation because now we'll do it the other way".
Context 🔦
No response

Hello, I'm having some hard times understanding this issue.
Emoticon is the default package for icons, I can not really find any text where @mui/system refers to it as the default styling engine.
Nor do I really see any issues with the custom components page, it uses styled components, which is what you should do if you wish to customize mui components.
Is there perhaps something that I am missing / don't understand? Thank you

Emoticon is the default package for icons

@Z3rio I think you have misread Emotion as "Emoticon"

Emoticon is the default package for icons

@Z3rio I think you have misread Emotion as "Emoticon"

Ah thank you, I will see if I can do something about it

@Studio384 Thanks for reporting this – I agree the title "Custom components" could be a bit misleading

If you want to use sx with fully custom (non-MUI) components...

Maybe we could be more specific about the use-case? For example: "If you want to add the sx prop to an existing styled-components project..."
Also the "Using standalone system utilities" section seems like it would be better suited for the "The sx prop" page


add KMT test washer

What does this PR do?
Add KMT support for Test Washer (added in https://github.com/DataDog/datadog-agent/pull/25015)
Motivation
Better handling of flaky tests
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge -c

/merge


Remove denominator in character counter

At present, when a message is close to or above the character limit, we display a character counter like this:

Instead, we should:

[ ] Display the number of characters remaining, or the number of characters over the limit (as a negative number). So if the user has typed N characters, we'd show the number 10000-N. We will no longer show the character limit.
[ ] Add a tooltip to the counter: "Maximum message length: 10000 characters"
[ ] Update the counter colors to match the new UI design (see #24878). Source code on Repl.it. Here is the decomposed source code archive:
compose-decomposed.zip; compose box design in Figma.

It's OK to submit a PR covering just the first two points, or just the third point.
There are no changes in the logic of when to show the counter, or whether it's yellow or red.
CZO discussion

@zulipbot claim

@alya I want to try this issue, can u assign this to me.

@alya I think this is the desired behaviour

@zulipbot claim

@zulipbot claim

hey @alya , if the issue is not resolved ,i can work on this


Short solution needed: "How to join in Rust" (rust)

Please help us write most modern and shortest code solution for this issue:
How to join in Rust (technology: rust)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to join all futures in vector in Rust

How to join array in Rust

How to join array into string in Rust

How to join bytes in Rust

How to join chars to string in Rust

How to merge enums in Rust

Example of join_all in Rust

How to join file path in Rust

How to merge files in Rust

How to join hashmaps in Rust

How to merge hashmaps in Rust

How to merge hashsets in Rust

How to join iterator of string in Rust

How to merge iterators in Rust

How to merge json objects in Rust

How to join list of strings in Rust

How to join two strings in Rust

How to join two arrays in Rust


feat!: make SnOsRunner more configurable

Issue Number: N/A
Type

[X] feature
[ ] bugfix
[ ] dev (no functional changes, no API changes)
[ ] fmt (formatting, renaming)
[ ] build
[ ] docs
[ ] testing

Description
This PR allows the configuration of SnOsRunner with more that one property at the same time.
Also, not sure why some properties must be private. In this proposal, the properties are made public.
Breaking changes?

[X] yes
[ ] no

Hey, runner will have to undergo deep refactoring, it does not make sense to try to improve it incrementally now.


provide stop state descriptions in device tree

It'd be good to expose some description of what the stop states mean for a particular CPU through /sys/devices/system/cpu/cpu*/cpuidle/state*/desc
To do that we need to first expose a description from OPAL, then a small kernel patch to tie them together (rather than the current kernel code just repeating the stop state name as the description)

Possibly should be related to https://github.com/open-power/skiboot/issues/177


nixos-generate-config seemed to set efiSysMountPoint to /boot isntead of /boot/efi in inappropriate situations

Rephrasing/summary of https://github.com/NixOS/nixpkgs/issues/275365.
Describe the bug
One day, I woke up and couldn't rebuild my config. It would complain that it could not find the bootloader. This seemed to be due to a failing regex search over the output of a bootctl query.
Eventually I set my hardware config's boot.loader.efi.efiSysMountPoint = "/boot/efi"; and the problem went away.
I made a little effort to figure out whether nixos-generate-config still produces this issue, and it seems like it does, but I identified a mysterious commented out line that used to set efiSysMountPoint to /boot/efi as required, which gives me hope that someone, at some point, understood what might have gone wrong here.
The problem might reside in generate-config, or it might just require making that regex a bit more flexible.
Additional context
See the prior issue https://github.com/NixOS/nixpkgs/issues/275365 where I was more confused for whole investigation.
Notify maintainers
I initially tried to notify @obadz, who was git blamed for the commented out line in generate-config, but they didn't respond.
@lheckemann might understand this aspect of generate-config's responsibilities.
@waldheinz is blamed for the systemd-boot regex.
I hope there's enough information there to make a decision, if not feel free to ask me to fetch more.

Add a :+1: reaction to issues you find important.

run into this error after installing nixos 23.11 via graphical installer. i've selected delete entire disk and install nixos.
error
{ config, lib, pkgs, modulesPath, ... }:

{
[...]
  fileSystems."/" =
    { device = "/dev/disk/by-uuid/e11e255e-c419-45fa-99fa-cc1a819b8d82";
      fsType = "ext4";
    };
[...]
}

$ lsblk
NAME   MAJ:MIN RM   SIZE RO TYPE MOUNTPOINTS
sda      8:0    0 931,5G  0 disk
├─sda1   8:1    0 897,1G  0 part /nix/store
│                                /
└─sda2   8:2    0  34,4G  0 part [SWAP]

$ sudo nixos-rebuild switch --flake .#
[...]
File system "/boot" is not a FAT EFI System Partition (ESP) file system.
systemd-boot not installed in ESP.
No default/fallback boot loader installed in ESP.
Traceback (most recent call last):
  File "/nix/store/62rk7fv5m3rbi5jiwlhs45iahw1ghjjg-systemd-boot/bin/systemd-boot-builder", line 343, in <module>
    main()
  File "/nix/store/62rk7fv5m3rbi5jiwlhs45iahw1ghjjg-systemd-boot/bin/systemd-boot-builder", line 331, in main
    install_bootloader(args)
  File "/nix/store/62rk7fv5m3rbi5jiwlhs45iahw1ghjjg-systemd-boot/bin/systemd-boot-builder", line 269, in install_bootloader
    raise Exception("could not find any previously installed systemd-boot")
Exception: could not find any previously installed systemd-boot
warning: error(s) occurred while switching to the new configuration

haven't tested with nixos-24.05 installer since wi-fi drivers won't come up.
possible solution
fresh install with manual partitioning.

@grerrg that looks to me like you haven't created an EFI System Partition, which is required for booting a system via UEFI (and systemd-boot only supports EFI). Try adding an ESP or switching to GRUB.

@grerrg you must set NIXOS_INSTALL_BOOTLOADER=1 in order to install systemd-boot. nixos-install sets this by default.


Add new gemma license

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

No, they are using other. We'll need to update their repos, currently:
license: other
license_name: gemma-terms-of-use
license_link: https://ai.google.dev/gemma/terms

yep i meant for license_name i.E. i would have used gemma-terms-of-use as the id here. No big deal though