Symfony collection added dynamically doesn't work

When we add a symfony collection form dynamically with JavaScript, the buttons (add, remove, ...) are not working anymore.
To made it work I used .arrive event to initialize the collection when it's added dynamically.
$(document).arrive('select', (element) => {
   if ($element.length) {
      /* eslint-disable */
      $element.collection({
        add: '<button type="button" class="add"></button>',
        allow_duplicate: false,
        allow_down: false,
        allow_up: false,
        remove: '<button type="button" class="remove"></button>',
        max: 500,
      });
      /* eslint-enable */
    }
});

Is this will be fixed in next update, or did I need to prepare a pull request to fix this ?

Hi @Jerome1337
I don't think this is native jQuery, and thus this won't be easily fixed.
Let me leave this issue open to look for a solution once I'll have the time to tackle it.
Have a great night,
Alain

Hi,

I don't think .arrive is native jQuery, and thus this won't be easily fixed.

.arrive is not native and form https://github.com/uzairfarooq/arrive

Let me leave this issue open to look for a solution once I'll have the time to tackle it.

No problem.
Regards.

Hi. I'm facing similar issue. I load form html using $.get and initiate this plugin afterwards. Buttons are rendered and I can "sometimes" use delete button, but other buttons don't work.
What do you recommend here, when form is being loaded through ajax? Thanks.


chore: move context.TODO to context.Background() (2)

Removing Context.TODO() from CmdWithCtx(...) by creating a CmdWithTesting(t *testing.T ....) that ties the context to the test. Plumbed through a couple of testing helpers to also take a testing.T as the first argument in the function signature, as this practice reduces the chance of someone mistaking these testing helpers as core functionality.

@phillebaba  I'm not 100% solid on removing the ability to assess errors from the actual test (caller) itself, mainly because there may be times we are expecting a certain error in the long term.


[Discussion] Rethinking Journals concept

The more I work with journals thee more I get the feeling that having 2 journal types (calendar vs interval) was not a good idea.
Partially I did it because I was working of Periodic Notes plugin. Even at that point it seemed not right but I did not see a better way at that moment.
After some considerations I plan to remove this dichotomy and have just one concept:
A journal is set of notes that you write on regular basis - daily, weekly, monthly or some custom period (like every 2 weeks).
For current concepts this means:

Calendar journal - will be split into separate journal for every period
Interval journal - will be a journal with custom period of writing

To compensate for some calendar journal functionality I plan to also add a concept of journal group (for example work or personal)  - that will allow to have focused calendar view for just work related journals for example.
As this is quite a big of a change I plan to make it a major plugin release.
Settings and data migration will be done automatically on first plugin start after update.
I would like to hear feedback. concerns or ideas from anyone using this plugin

I like the simplified view you propose: intervals are just intervals, and a calendar is just a set of intervals we've internationally agreed on.
I don't see the value for journal groups yet. wouldn't using existing tags be enough to meet the filtering requirement?

Good question
I got an idea of journal group when I was thinking on how to keep same functionality in calendar-nav and calendar-timeline blocks.
There you can get not only to current level (like daily) but to other notes within same calendar (like weekly, monthly or yearly notes) given they are active.
As there will be no such direct connection we should somehow connect journals in other way. If you are using just one set of connected journal - you might not even need groups. But for people who have work vs personal journals combining everything in code blocks might be not convenient.
Focused calendar view was a further idea on how to use groups.
In some sense  you are right - groups are similar to tags just on journal level. But I wanted to avoid confusion as it might not fully work as regular tag - for example I don't see a way to make it searchable
If you have any other idea on how to change code blocks with new journal concept - please let me know

I want to make sure I understand what you are proposing:
Option 1

A user will add a journal in the plugin settings, but instead of choosing a type, they will be able to check all the intervals they want to add:

Yearly, quarterly, monthly, weekly, daily, and/or a custom interval like every x weeks starting 2024-07-12
intervals could be stored using a format like https://en.wikipedia.org/wiki/ISO_8601#Durations

The journal ID and journal-section will then be added in the frontmatter as usual
Code blocks like the following would be used to navigate the journals

    ```calendar-timeline
    journal-section: [one of: year | quarter | month | week | day | interval]
    journal: [journal ID]
    ```

or
Option 2

A user will add a journal in the plugin settings, but instead of choosing a type, they will be able to select a single interval they want to add:

Yearly, quarterly, monthly, weekly, daily, or a custom interval like every x weeks starting 2024-07-12
intervals could be stored using a format like https://en.wikipedia.org/wiki/ISO_8601#Durations

A journal-group to connect the various intervals, as well as a journal ID and journal-section will be added to the frontmatter
Code blocks like the following would be used to navigate the journals

    ```calendar-timeline
    journal-group: [month or year or interval]
    journal: [journal ID]
    journal-section: [not used anymore?]
    ```

or something completely different?

Using option 1 would move the journal type interval as a sub-option inside the current journal configuration, instead as a sibling, e.g.

It is option 2 - for example instead one journal for daily and weekly notes there will be 2 separate journals
as for frontmatter

journal-section would be removed as obsolete (it was needed before because one journal might have several different sections active at the same time)
not sure about adding group as plugin can read it from config (but might be handy for queries probably)

code blocks will probably be renamed but ability to set what group or journal should be used will definitely be there
also as part of journal unification functionality that currently is only in interval journals will be avaliable for every journal - like start date, journal end (specific date or after some number of repetitions) and indexing

I feel that option 1 might be easier to understand for end-users, easier to design from an interface perspective, and it aligns better with the way the plugin already works, so less migration steps.
The plugin already makes provision for adding different sections to a journal, so adding a custom interval would just be another toggle switch with a configuration screen behind that. Grouping is then already built in (via the journal ID), and you'd need to do less to migrate -- less frontmatter properties to add/change.
With option two, I feel there is more risk of end-users misunderstanding how the grouping works, and additional places for user error if you have to fill in the journal-group for each journal. Then, updating the interface to somehow show the journals connected to the same group adds a layer of complexity.
Either option would need the same amount of work in the back-end, I suspect, so I'd vote for option 1 as being more user-friendly.

Interface mockup for option 1:
After adding a new journal, a toggle is added for a custom interval:

Settings for Custom interval notes already exist:

Hm, good point I did not think that way
Though for more flexibility custom intervals should be probably not as toggle but with possibility to add multiple custom intervals
And maybe changing regular intervals (daily, weekly etc) to be added not as toggle as well - in case someone wants to have multiple writing intervals but not default ones

I would favor option1

Maybe I do not get your full use case here, but:

I agree that "interval" represents calendar as well as customized intervals. Assuming that adding manual "irregular" notes are still possible (e.g. notes on your sports sessions), that covers everything I can think of.

Following the proposed ISO 8601 standard would give maximum flexibility. You could keep the "daily", "weekly" ones in the UI for better UX, but technically it wouldn't be needed.
You could even think about support for shorter intervals (e.g. hourly) to track, for example, medical symptioms. Possibly a future feature.

I don't understand why a single journal needs multiple intervals. I would most likely just create two journals then.

Wouldn't having multiple notes and linking them later be one of the principles of knowledge management with obsidian or the "Zettelkasten" approach either?

I can think of two scenarios of "grouping" journals:

I want group/split thematically, e.g. "work", "leisure" or "sports": I can focus on my work notes during worktime and on leisure notes at the other times.

I also would see the tags as a perfect fit from user perspective. It is free-form and you could also assign notes to multiple ones, if needed.
Alternatively, you could use the folder structure, of course. Less flexible, but possibly easier to understand. Not different from standalone notes.

I want to group by "interval": I want to see my "work" and "leisure" notes combined for the same day for my overall retrospective.

Personally, I usually also consider statistical analyses. Using the notes to keep track of working hours, happiness, sports metrics, number of social interactions, could also be interesting for some users. While this is some sort of independent, I would assume that using the typical "date" or "datetime" in the frontmatter as well as the "tags", it might be quite compatible with most of the other (statistical) plugins.

Thanks everyone for discussion.
I'm starting work on v2 and will be changing journal concept as described.
Journal shelves (aka groups) will be used to group journals in settings (so it resembles current UI) and in calendar view/code blocks to limit what is shown.
I created a separate issue on v2 work - https://github.com/srg-kostyrko/obsidian-journal/issues/63
and a separate project to track progress - https://github.com/users/srg-kostyrko/projects/3


Tracks are not working

No clicking possible, no information.
http://fossasia.github.io/open-event-webapp/#/tracks

I would like to fix this bug. I have seen that it is not possible to click and there is no information either. Where do I get the information to be put there and will an accordion be fine?

An array of sessions is saved in the $sessionStorage.sessions object. We
need to get sessions from there, and filter then according to the track.
On 12 September 2015 at 14:13, Saptak Sengupta<EMAIL_ADDRESS>wrote:

I would like to fix this bug. I have seen that it is not possible to click
and there is no information either. Where do I get the information to be
put there and will an accordion be fine?
—
Reply to this email directly or view it on GitHub
https://github.com/fossasia/open-event-webapp/issues/55#issuecomment-139740602
.

You mean, when I click a track, I need to show only the sessions corresponding to the track?? Something like that??

Yes exactly.
note: for later:
Also I want to revisit the way we save the data. Maybe it is not the most
efficient, but yes for now, showing the sessiosn that have the given track
X in it's "tracks" field should be shown when we click on a track
On 13 September 2015 at 00:19, Saptak Sengupta<EMAIL_ADDRESS>wrote:

You mean, when I click a track, I need to show only the sessions
corresponding to the track?? Something like that??
—
Reply to this email directly or view it on GitHub
https://github.com/fossasia/open-event-webapp/issues/55#issuecomment-139809315
.

Ok.. I will send a pull request after I do it. I will ask again if I face any problem.

I have one more doubt. As I see, the sessions have a title, a type, language and a speaker.. I don't see anything related to tracks. So what exactly do I need to match with the track? Should I add a category all together to the sessions which I can compare with the tracks?

Howdy,
The API JSON response data for a session should contain some kind of
session identifier which tells the track that the session is a part of. If
not, it should.
Hope that helps,
Duke
On Sat, Sep 12, 2015 at 9:06 PM, Saptak Sengupta<EMAIL_ADDRESS>wrote:

I have one more doubt. As I see, the sessions have a title, a type,
language and a speaker.. I don't see anything related to tracks. So what
exactly do I need to match with the track? Should I add a category all
together to the sessions which I can compare with the tracks?
—
Reply to this email directly or view it on GitHub
https://github.com/fossasia/open-event-webapp/issues/55#issuecomment-139843188
.

--
Jonathan "Duke" Leto<EMAIL_ADDRESS>Leto Labs LLC http://letolabs.com
209.691.DUKE http://duke.leto.net
@dukeleto http://linkedin.leto.net https://github.com/leto

http://openevent.championswimmer.in/api/v1/event/1/sessions/page/2
Yeah currently they are all coming out to be null.
I think you this needs be an issue flagged up on
https://github.com/fossasia/open-event-orga-server
On 13 September 2015 at 21:19, Duke Leto<EMAIL_ADDRESS>wrote:

Howdy,
The API JSON response data for a session should contain some kind of
session identifier which tells the track that the session is a part of. If
not, it should.
Hope that helps,
Duke
On Sat, Sep 12, 2015 at 9:06 PM, Saptak Sengupta<EMAIL_ADDRESS>
wrote:

I have one more doubt. As I see, the sessions have a title, a type,
language and a speaker.. I don't see anything related to tracks. So what
exactly do I need to match with the track? Should I add a category all
together to the sessions which I can compare with the tracks?
—
Reply to this email directly or view it on GitHub
<
https://github.com/fossasia/open-event-webapp/issues/55#issuecomment-139843188
.

--
Jonathan "Duke" Leto<EMAIL_ADDRESS>Leto Labs LLC http://letolabs.com
209.691.DUKE http://duke.leto.net
@dukeleto http://linkedin.leto.net https://github.com/leto
—
Reply to this email directly or view it on GitHub
https://github.com/fossasia/open-event-webapp/issues/55#issuecomment-139889406
.

on updating bower and deploying the project, i get this error:
Error: [$injector:modulerr] Failed to instantiate module openevent due to:
[$injector:modulerr] Failed to instantiate module leaflet-directive due to:
[$injector:modulerr] Failed to instantiate module nemLogging due to:
[$injector:nomod] Module 'nemLogging' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.4.6/$injector/nomod?p0=nemLogging
minErr/<@http://localhost:8080/bower_components/angular/angular.js:68:12
module/<@http://localhost:8080/bower_components/angular/angular.js:1986:1
ensure@http://localhost:8080/bower_components/angular/angular.js:1910:38
module@http://localhost:8080/bower_components/angular/angular.js:1984:1
loadModules/<@http://localhost:8080/bower_components/anugular/angular.js:4388:22
forEach@http://localhost:8080/bower_components/angular/angular.js:336:11
loadModules@http://localhost:8080/bower_components/angular/angular
Can you help me out?

Fixed the issue. Needed to add angular-simple-logger as a dependency.

I am facing a problem. Though the above issue has been resolved in the above mentioned repo, but when I deploy my local repo, it still doesn't show any change.
The values have changed here http://openevent.championswimmer.in/api/v1/event/1/sessions
But when I open the website, it still shows null track. Am I missing something?

fixed here https://github.com/fossasia/open-event-webapp/pull/75

I guess this issue is fixed?

This has been fixed by @SaptakS.
Cheers!


Added warning when graph file is missing to prevent crashes

Closes #12

Refer to this link for build results (access rights to CI server needed):
http://jenkins.tribler.org//job/Graphalytics_pull_request_tester/14/
Test PASSed.


Allow aliases for @@unique and composite @@id fields

Problem
When defining a unique index or composite primary key, prisma introspects the name from the database. You can provide your own name instead, but upon reintrospecting the name will be overwritten.
Suggested solution
It would be nice to be able to map the name from the database to the prisma schema and resulting types, as this would allow us to name things to better match the domain and intent, especially when applying prisma to legacy databases.
model Thing {
  id Int
  userId Int
  customerId Int

  // Allow a new "map" key to reference the underlying db name, but can use name field for the javascript name
  @@unique([userId, customerId], name: "userAndCustomer", map: "user_customer_unique")

  // Allow a custom name for a composite id filter
  @@id([id, customerId], name: "idAndCustomer")
}

prisma.thing.findOne({
  where: {
    // friendlier to our domain AND our eslint rules for camel casing
    userAndCustomer: {
      userId,
      customerId,
   }
 }
})

Alternatives
Using the names as defined in the database.

Hi,
Thanks for reporting this! It has already been reported before, so I am going to close this as a duplicate of https://github.com/prisma/prisma/issues/3193


Add Participant name to DDS Router participants

Signed-off-by: jparisu<EMAIL_ADDRESS>

The failing test in docker_test was due to a delay in the discovery, so not all the messages (but some) arrive to the subscriber. Thus, there is not an actual failure of the tests.
Note: we should increment that timeout.


wttr.in error on load webpage

wttr.in page showing ERROR

This also happens to me when making a request from Portugal.
However, if I make my request from my VPS in the United States, everything works as expected.

It may happen periodically if our data service provider has problems. It happens. If the problem will not disappear, please let me know if it happens if you access not wttr.in but wttr.in/Berlin for example

I have still problem even on dsl or mobile network. Requesting wttr.in or wttr.in/Berlin ends with same error for now, I will see evening or tommorow :)

@chubin on my local machine, both cases (wttr.in and wttr.in/Berlin) fail, while on my remote machine both of them work fine.

I think I've fixed it. Can you please recheck it?

Its working thx :)

Same here. I think this issue can be closed.

Thank you for the help, guys!

Hi
This is still an issue, if you're in South Africa

@Kingshish Is it still an issue now?

Yes
Curl wttr.in
Returns
ERROR
On 21 Feb 2017 4:45 PM, "Igor Chubin"<EMAIL_ADDRESS>wrote:

@Kingshish https://github.com/Kingshish Is it still an issue now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chubin/wttr.in/issues/33#issuecomment-281364129, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD3rq0H8JNYTZraZ64inYtu45MhhZyoEks5revimgaJpZM4IFWWf
.

@Kingshish Please check it again. It should work now. If your location is unkown it should write the name of your location (how it was detected) and bring you to Oymakon. Please check it, if it works as expected

Still says ERROR,  I'm in Johannesburg if that helps
On 21 Feb 2017 6:26 PM, "Igor Chubin"<EMAIL_ADDRESS>wrote:

@Kingshish https://github.com/Kingshish Please check it again. It
should work now. If your location is unkown it should write the name of
your location (how it was detected) and bring you to Oymakon. Please check
it, if it works as expected
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chubin/wttr.in/issues/33#issuecomment-281395503, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD3rq8B2F5kqa_OOOJ_5yu7vwVVQkbW0ks5rexAigaJpZM4IFWWf
.

And now?

It works! Thanks man,  awesome work
On 21 Feb 2017 6:43 PM, "Igor Chubin"<EMAIL_ADDRESS>wrote:

And now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chubin/wttr.in/issues/33#issuecomment-281401094, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD3rq136Wx8pWLCuzdZ__rVrW-99xohRks5rexQjgaJpZM4IFWWf
.

@Kingshish Thank you for the reporting!

Hi @chubin
same for me, opening it in the browser or via "curl wttr.in" gives me an error, for some locations (e.g. Vienna) it just displayed a note "New feature: multilingual location....."
Thx and cheers from Austria :-)

@peter9teufel Thank you very much for the reporting! (es sollte jetzt wieder laufen und schöne Grüße aus Bayern :)

Das geht ja sprachlich gleich viel leichter von der Hand :-D
Perfekt danke!

@peter9teufel apropos: curl de.wttr.in

ITS NOT WORKING STILL GET ERROR

@vijaygalagali20 It works now. Thank you for reporting!

yup its working thank you all

ERROR Problem for me. Tried http://wttr.in/Toronto and http://wttr.in/Hamilton

@hopkins001 Fixed! Thank you for reporting


add support for windows

Tested from the browserpass-windows64.zip file produced via make release in firefox and chrome with success (built from linux)!
Both extensions were installed from the respective "stores".
This fixes #133

Nice, this surely makes installation a lot easier :) I'm curious though, why the powershell script? Wouldn't a regular old batch file be sufficient here?
Also, does this actually work without MinGW?

I like powershell script more :stuck_out_tongue:
@chrboe do you mind testing this? Just to make sure it works for more than one person :smile:
@qbit could you please also update the README.md? Just a bit, mention how exactly to run the script instead of ./install.sh, and whether there is a difference of executing this script as a normal user or as admin (on linux there is).

Sure, let's see if the old Windows 10 on my Laptop still boots :D
(It could potentially take a while for me to get to it, but I expect to get it done today)

OK, I added some install details to README and gave install.ps the ability to install for all users (as admin).

Ahh crud - I should have checked if this was merged already!
Anywa - this new version is what I should have written in the first place. Sorry for the back and forth! :P

What do you think about #136? @chrboe suggested not to ask to choose a browser, but install for all available browsers at once.
You don't have to do it now, we can handle it separately, just want to get your opinion on this.
@chrboe did you have a chance to try the script from this PR?

@qbit everything looks good to me, so just want to confirm with you, is it ready to be merged once Christoph tests this, or you want to do something else in the code?

Yeah, I feel like it's ready. :D

Sorry, I didn't have time to test it yesterday. I'm on it right now though 😃

Alright, all done. It seems to work just fine as far as I can tell :)
(I also built browserpass-windows64.zip on Linux using make release, then just unzipped and ran the script and it worked like a charm.

Merging then, thanks to both of you! 🙂


Self-compiled soyutils problem with Imported test-only code

Not sure if you can help me here but I am trying to build the soyutils myself with the google closure compiler like this:
java  -server -XX:+TieredCompilation  -jar "../closure-compiler/build/compiler.jar"  --js ../closure-library/closure/goog/ --js ../closure-templates/javascript/soyutils_usegoog.js --js soyutils-exports.js --js_output_file=soyutils-no-amd.js --manage_closure_dependencies --only_closure_dependencies --closure_entry_point soy --closure_entry_point soydata --closure_entry_point soyutils.exports --define 'goog.DEBUG=false' --compilation_level ADVANCED_OPTIMIZATIONS --output_wrapper '(function(w){ %output% w.soy=this.soy; w.soydata=this.soydata; w.goog=this.goog})(window);'
But using the generated soyutils-no-amd.js in my project leads to this error code: "Uncaught Error: Importing test-only code into non-debug environment."
Any idea / hint where this test-code comes from or how I can disable it?

It would be best to ask this on the closure-compiler google group or the closure templates google group. The issue tracker is reserved for bug reports a feature changes.


Do not enforce lat/long to be between -90/-180 and 90/180

Description
Whenever you currently want to use the geolocation feature like so:
		return index.search(searchInput, {
			sort: [`_geoPoint(${center.x}, ${center.y}):asc`]
		});

It tells you that the latitude must be between -90 and 90. While this does make sense for the "real world", this enforcement means you can not use MeiliSearch for searching something within a virtual world like a game where the lat/long could be way bigger than this.
Basic example
I'd like to not have this enforcement between values which make sense in the real world, or at least have a way to disable the warning.
Other
I'd like to know if there's any technical reason why this enforcement is made, or is it just that humans are not to always be trusted?

Hey @rbozan.
Actually, the lat and lng only describe the angular degrees from the center of the earth to the surface.
It doesn't make sense to go over 90 or below -90 since these are angles (and I don't really see how this is an issue for you, tbh)
What is absolutely wrong, though, is the distance returned when computing a geo sort because that's the part that is tied to the size of the earth specifically!
The other issue you could have with Meilisearch is that it enforces the usage of a “round” planet. If your world is supposed to be flat, it doesn't work, either.

We made it like that because that's what everyone was asking for 😅
Tbh you're the first one I've seen that try to use meilisearch with the geo search feature on something other than Earth

Hey @rbozan.
Actually, the lat and lng only describe the angular degrees from the center of the earth to the surface.
It doesn't make sense to go over 90 or below -90 since these are angles (and I don't really see how this is an issue for you, tbh)
What is absolutely wrong, though, is the distance returned when computing a geo sort because that's the part that is tied to the size of the earth specifically!
The other issue you could have with Meilisearch is that it enforces the usage of a “round” planet. If your world is supposed to be flat, it doesn't work, either.

We made it like that because that's what everyone was asking for 😅
Tbh you're the first one I've seen that try to use meilisearch with the geo search feature on something other than Earth

Haha it seems indeed my case is a bit of a nice. I just have a 2d world on a regular flat plane where I'd like to be able to search for things on the plane. It's indeed not lat/long but just x/y actually. But because these are unable to get a distance (I guess?) I'm trying with _geo instead.

Yeah, that's what I thought.
Currently, I don't think we have any nice solution to handle that.
Which operation do you need?

For my use case I basically want to sort the search results by distance from the center of the viewport of the game.
Just imagine you have objects with x+y coordinates and I'd want to do a search on these objects sorted by distance.
It's this operation according to the docs:
client.index('restaurants').search('', { sort: ['_geoPoint(48.8561446, 2.2978204):asc'], })

Hello @rbozan and @irevoire
Thank you for your feedback
Pinging our PMs @macraig and @gmourier for your information

Hi @rbozan , as this is currently a feature request I'll close this issue and I've opened a product discussion for it in our Product repo: https://github.com/meilisearch/product/discussions/676
Feel free to add more to that feature proposal issue!


Hello, how to change the size of the font, has been looking for such a method.

[ ] I have verified there are no duplicate active or recent bugs, questions, or requests
[ ] I have verified that I am using the latest version of Material Dialogs.
[ ] I have given my issue a non-generic title.
[ ] I have read over the documentation (before asking questions on how to do something).

Include the following:

Material Dialogs version: 0.x.x.x
Device OS version: 7.1.1
Device Manufacturer: Google
Device Name: Pixel XL

Also, please wrap Java with correct syntax highlighting.
System.out.println("Hello, world!");

Reproduction Steps

Expected Result
Actual Result

The material specs are used.


Error Logging in FastHttpUser

Discussed in https://github.com/orgs/locustio/discussions/2936

Originally posted by swaalt October 10, 2024
Hi, I am using FastHttpUser and when I receive a 404 the fail is registered like this, this happens because the geventhttpclient saves in the exception the badstatuscode text the url of the request I made, which is dynamic and appears separated by each one,
I need that either I take the “name” or simply do not place anything, just place the badstatuscode 404, can this be addressed with the locust as it is now or do I have to modify something?

PR:s welcome (I'll look at it myself, but no guarantees on the timing)

I would like to work on this issue. Please assign me this issue.

Ideally, make it work the same as HttpUser.

Interesting. But maybe even better/easier to subclass the ConnectionError class (in locust) and override the __repr__ method there? Such a PR would be most welcome!

Would that be okay?
https://github.com/swaalt/locust/commit/c9192ac0a0ec7a58c70d57b5bc112b3c7e9f9152

Looks great! PR it? Maybe add a test case for it? (ignore any other errors, the tests are a bit flaky atm)

Attach the PR : https://github.com/locustio/locust/pull/2944

maybe this solution will solve your problem click


Humbug unexpectedly ends up with "Tests must be in a fully passing state..."

Sometimes humbug does this
Humbug running test suite to generate logs and code coverage data...

  118 [==========================================================] 11 secs

<warning>Tests must be in a fully passing state before Humbug is run.</warning>
<warning>Incomplete, skipped or risky tests are allowed.</warning>
<warning>Stdout: \n    > 1..120
    >
    > Generating code coverage report in PHP format ... done
    > \n</warning>

(Yes, tags are printed, not converted)
Tests are really ok, and the strange thing is that after manually running phpunit (and ensure that tests are really ok) this error disappears, otherwise you will get this error every time. Looks like some cache issue.

It looks like an issue processing the TAP output. The stdout is actually truncated at the moment (had issues on bigger libraries with many thousands of tests creating a memory sink). Will look into it.

I add more of a safety net. The TAP parsing appears correct, but I may be racing against the initial process closing (if open just long enough, the exit code would be null, not 0 or 1 as expected). Explicitly closing it should do the trick so will consider this one fixed!


fix definition-type-error

Codecov Report

Merging #277 (46d3646) into master (ec04a0f) will decrease coverage by 0.11%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master     #277      +/-   ##
==========================================
- Coverage   67.97%   67.86%   -0.12%
==========================================
  Files          20       20
  Lines        6826     6842      +16
==========================================
+ Hits         4640     4643       +3
- Misses       2186     2199      +13

Impacted Files
Coverage Δ

stix2elevator/utils.py
89.90% <ø> (ø)

stix2elevator/test/test_idioms.py
73.28% <64.00%> (-1.33%)
:arrow_down:

stix2elevator/convert_stix.py
81.25% <73.91%> (-0.46%)
:arrow_down:

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 ec04a0f...46d3646. Read the comment docs.


Add new MultiQC modules before release

nf-core/viralrecon pull request
Many thanks for contributing to nf-core/viralrecon!
Please fill in the appropriate checklist below (delete whatever is not relevant).
These are the most common things requested on pull requests (PRs).
PR checklist

[ ] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/viralrecon branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[ ] CHANGELOG.md is updated
[ ] README.md is updated

Learn more about contributing: CONTRIBUTING.md

Tests passing here and quite uncontroversial changes so going to merge this so I dont have to think about it over the weekend!


Investigate image post failures

Maybe phantom js executable permissions

it was in the wrong directory. made a symlink.


Remove version: Jackett.Jackett version 0.20.3063

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.3063/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


[BUG]: Preserve copied content formatting

Currently, there is a bug where the formatting of copied content is not being preserved. This includes line breaks, which can result in the content being difficult to read. This issue should be fixed to ensure that the copied content retains its original formatting.

I will close this issue since after testing this I no longer see the problem surface


Added missing symbol "&" for matching end of code in "GenerateCSharpMethodBody" method.

Thanks, we did it already today.


Error in file that is presented for function.json

This:
{
    "bindings": [
    {
        "type": "httpTrigger",
        "name": "req",
        "direction": "in",
        "methods": [ "get" ],
        "route": "products/{category:alpha}/{id:int?}"
    },
    {
        "type": "http",
        "name": "res",
        "direction": "out"
    }
    ]
}

for python should be:
{
    "bindings": [
    {
        "type": "httpTrigger",
        "name": "req",
        "direction": "in",
        "methods": [ "get" ],
        "route": "products/{category:alpha}/{id:int?}"
    },
    {
        "type": "http",
        "name": "$return",
        "direction": "out"
    }
    ]
}

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

ID: 5b6cd6a0-5364-65e6-b088-cc00093f836f
Version Independent ID: 3c90d8fb-0fea-3496-e73e-0020ff33adef
Content: Azure Functions HTTP trigger
Content Source: articles/azure-functions/functions-bindings-http-webhook-trigger.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Thanks for your feedback! We will review and update this doc accordingly.

Thanks for your feedback! We will review and update this doc accordingly.

Hello @EasyAsABC123,
We were able to replicate the issue mentioned, so we have made the changes in the doc, you will able to see the changes in next 24 hours. Please let us know if you have any further concerns. Thank you!

Hello @EasyAsABC123,
We were able to replicate the issue mentioned, so we have made the changes in the doc, you will able to see the changes in next 24 hours. Please let us know if you have any further concerns. Thank you!


Let TIs store notes on their clients

TIs sometimes need to record some notes about their clients, such as "Client needs childcare help" or "Client wants to apply for program X, but I need to get their W-2 first". Let TIs do that directly within the CiviForm interface so that the information they need is always at hand when they're working with a client.
Doesn't have to be too fancy -- just a free-text input field that can support Markdown or something similar.
This is a lightweight way of enabling case management / CRM functionality for TIs.

Hi Holly. Can you me to this pls?

Could be a good item for feedback at next CBO user session. Matt to draft PRD for design. @msprenke

Check with privacy!

Combined this with https://github.com/civiform/civiform/issues/4956

As this is combined with design for 4956 I am closing this tracking item.


Write-Stream throws lots of spurious errors on missing variables

While running hundreds of jobs which return a mix of output, I've encountered issues when doing Receive-RSJob which look like this:

Write-Verbose : Cannot bind argument to parameter 'Message' because it is null.
At C:...\PoshRSJob\Private\WriteStream.ps1:17 char:107

... Job | Select -ExpandProperty Verbose | ForEach { Write-Verbose $_ } }

                                                               ~~

CategoryInfo          : InvalidData: (:) [Write-Verbose], ParameterBindingValidationException
FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Microsoft.PowerShell.Commands.WriteVerboseCommand

Which in turn (appears to) trigger these:

WriteStream : Exception calling "EndInvoke" with "1" argument(s): "The property 'Arguments' cannot be found on this object. Verify that the property exists."
At C:...\PoshRSJob\Public\Receive-RSJob.ps1:102 char:18

        $_ | WriteStream

             ~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,WriteStream

WriteStream : Exception calling "EndInvoke" with "1" argument(s): "The WriteObject and WriteError methods cannot be called from outside the overrides of the BeginProcessing, ProcessRecord, and EndProcessing methods, and
they can only be called from within the same thread. Validate that the cmdlet makes these calls correctly, or contact Microsoft Customer Support Services."
At C:...\PoshRSJob\Public\Receive-RSJob.ps1:102 char:18

        $_ | WriteStream

             ~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,WriteStream

This is on PowerShell v5, PoshRSJob <IP_ADDRESS>, with Set-StrictMode -Version Latest and $ErrorActionPreference = "Stop". I always do this to catch scripting errors.
I've modified the code to do this:

               "Verbose" { $IndividualJob | Select -ExpandProperty Verbose | Where { $_.Message } | ForEach { Write-Verbose $_ } }
              "Warning" { $IndividualJob | Select -ExpandProperty Warning | Where { $_.Message } | ForEach { Write-Warning $_ } }
              "Error"   { $IndividualJob | Select -ExpandProperty Error | ForEach { Write-Error $_ } }
               "Output"  { $IndividualJob | Select -ExpandProperty Output }

This avoids a lot of the errors. But it changes the semantics of Error because it discards a lot of info, and this still isn't working perfectly (I think).

Errors are a little fun in runspaces, especially when we are trying to capture them from a runspace. Do you happen to have some code to add to this that I use to duplicate the issue? It sounds like you have a partial solution that I can test with and possibly integrate into the module.

I believe that this should be fixed in v<IP_ADDRESS>

@codykonior Any chance that you might be able to revisit this and see if anything is better?

I'm just not sure why you are selecting out only some of the properties and then rewriting only some of the properties to the host.
Why not just return the full object? The outcome is the same except that the user can do what they like with the objects and all the other information they contain instead of, say, only seeing the message property.

Sorry, I'm drawing a blank on this. Can you point me to the file/line that you are seeing this on? It may better help to remember why I did it the way that I did...or make me realize that I should do it differently.

I believe that I saw you fixed this (whole object vs. just Message property) in your latest PR. Are things looking better now?

I checked WriteStream and it's still your original version - I only expanded the existing names. I do have a private version that does the full objects but I didn't merge it as I wanted to keep that PR just for fixes.

Got it. For some reason I thought I saw otherwise, but going back I apparently wasn't paying attention. It's all good as I will work that into the next update.

No longer an issue as far as I know.


Fix isOpApplyDg to accept non-ref args.

@Geod24 Just a quick verification, if there is anything subtle that would preclude non-ref args.

The specs (kinda) says they are ref http://dlang.org/statement.html#ForeachStatement , but from experience, non-ref is just fine. Actually, if you look at the specs, they are even more permissive with templated opApply.
Did that triggers any bug ?

Yeah, it used to be the case that ref was required and the examples are probably just still from that time. I'll try to find out when this was changed.
The problem is that usually the compiler allows to use non-ref variables in the foreach statement, even if opApply only takes ref arguments. However, when a template type is used for the delegate, it tries to match a delegate without ref, so that the isOpApplyDg failed. I need to dig up the actual example I had, though, since the unit tests do seem to pass...
BTW, I just noticed now that the unit test changes in rest.d fail for 2.065. I'll fix that and rebase.


sbctl: 0.13 -> 0.14

Description of changes
A go mod tidy was needed for some reason.
https://github.com/Foxboron/sbctl/compare/0.13...0.14
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 314592 run on x86_64-linux 1

1 package built:

sbctl


"invalid argument passed" during test provision

I'm trying to make a test provision with command:
mainflux-cli provision test
and get the following error:
invalid argument passed

@cowboylux how did you install mainflux-cli ?
You must re-build it, we made some small changes, just do make cli from <project_root>, which will build mainflux-cli in <project_root>/build folder.
Later if you want you can copy mainflux-cli to your $GOBIN or just use it from <project_root>/build directory as discrabed in doc

Can I download a binaries somewhere?

@cowboylux Unfortunately not, because it's not a release and we don't publish the latest artifacts.
Do you have any blockers to do a build?


Can I not draw

I guess if I do not draw the pictures, the program can run faster. but I do not know how to not draw.
Thank you!

@wangliang1989 just set 'number_of_plots' to 0

Thank you


IOTDB-1125 encoding set regular, flush and select failed due to data is not consistent

reproduct:
create timeseries root.db_0.tab0.salary with datatype=INT64,encoding=REGULAR ;
insert into root.db_0.tab0(time ,salary) values(2021-01-25T17:36:01.000+08:00,1200);
insert into root.db_0.tab0(time ,salary) values(2021-01-25T17:36:02.000+08:00,1100);
insert into root.db_0.tab0(time ,salary) values(2021-01-25T17:36:03.000+08:00,1000);
insert into root.db_0.tab0(time ,salary) values(2021-01-25T17:36:04.000+08:00,2200);
flush;
reason:
when create timeseries, set  encoding=REGULAR. so data need RegularDataEncoding in flush. but data is not constant frequency.
when read data,  regular data decoding... so  this too.
solution：
if a timeseries data encode failed, will do not write data to chunk, and WAL will delete（filter Dirty data）.  give a warn in log.if a timeseries error-data lead to all timeseries flush failed, which will be a problem...

https://issues.apache.org/jira/browse/IOTDB-1125

https://issues.apache.org/jira/browse/IOTDB-1125

hi @HTHou @qiaojialin @jixuan1989 , could you please review this ?
Thanks

hi @HTHou @qiaojialin @jixuan1989 , could you please review this ?
Thanks

Hi, I am confused by your description...

Hi, I am confused by your description...

Hi, I am confused by your description...

定频编码要求数据为定频数据，存在几种情况：

一次memtable 中数据不是定频数据，在flush的时候encoding会报错，直接导致本次flush的所有序列都会失败。（这是我测试的问题，本PR已解决，解决方法：将编码错误的数据过滤掉，不影响其他序列flush）
单次memtable是定频的，但是两个tsfile文件中加起来的数据不是定频的，经过合并后，会导致在读取数据解码的时候，发生错误。（本PR已解决）（这是iotdb-1125中的问题）
如果一旦flush失败，会设置成只读模式，没有恢复机制，我提了一个优化单（IOTDB-1136），将在这个单子里面解决。
Constant-frequency encoding requires constant-frequency data. There are several cases:
The data in the memtable is not constant-frequency data. During flushing, an error is reported during encoding. As a result, all sequences fail to be flushed. (This is a problem in the test and has been resolved in this PR.)
A single memtable is fixed-frequency, but the sum of data in the two TSfile files is not fixed-frequency. After the combination, an error occurs during data decoding. (This issue has been resolved in this PR.) (This issue is in iotdb-1125.)
If the flush fails, the system sets the data to the read-only mode and does not have a recovery mechanism. I have submitted an optimization ticket (IOTDB-1136) to resolve this issue.

Hi, I am confused by your description...

定频编码要求数据为定频数据，存在几种情况：

一次memtable 中数据不是定频数据，在flush的时候encoding会报错，直接导致本次flush的所有序列都会失败。（这是我测试的问题，本PR已解决，解决方法：将编码错误的数据过滤掉，不影响其他序列flush）
单次memtable是定频的，但是两个tsfile文件中加起来的数据不是定频的，经过合并后，会导致在读取数据解码的时候，发生错误。（本PR已解决）（这是iotdb-1125中的问题）
如果一旦flush失败，会设置成只读模式，没有恢复机制，我提了一个优化单（IOTDB-1136），将在这个单子里面解决。
Constant-frequency encoding requires constant-frequency data. There are several cases:
The data in the memtable is not constant-frequency data. During flushing, an error is reported during encoding. As a result, all sequences fail to be flushed. (This is a problem in the test and has been resolved in this PR.)
A single memtable is fixed-frequency, but the sum of data in the two TSfile files is not fixed-frequency. After the combination, an error occurs during data decoding. (This issue has been resolved in this PR.) (This issue is in iotdb-1125.)
If the flush fails, the system sets the data to the read-only mode and does not have a recovery mechanism. I have submitted an optimization ticket (IOTDB-1136) to resolve this issue.

OK.  Got it.
But IMO, I think it is a bug of "regular" encoding implementation, rather than solving it by rollback the WAL.
see https://issues.apache.org/jira/browse/IOTDB-1136?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&focusedCommentId=17274179#comment-17274179

OK.  Got it.
But IMO, I think it is a bug of "regular" encoding implementation, rather than solving it by rollback the WAL.
see https://issues.apache.org/jira/browse/IOTDB-1136?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&focusedCommentId=17274179#comment-17274179

Ses the design https://lists.apache.org/thread.html/f612326ca8708981b5a2da29f52b826105bbb55dc840281c5bf6af4f%40<dev.iotdb.apache.org>

Ses the design https://lists.apache.org/thread.html/f612326ca8708981b5a2da29f52b826105bbb55dc840281c5bf6af4f%40<dev.iotdb.apache.org>

OK. Got it.
But IMO, I think it is a bug of "regular" encoding implementation, rather than solving it by rollback the WAL.
see https://issues.apache.org/jira/browse/IOTDB-1136?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&focusedCommentId=17274179#comment-17274179

Thanks ， I will try to alter regular" encoding...  if can  not , I will give reason... we discuss again...

OK. Got it.
But IMO, I think it is a bug of "regular" encoding implementation, rather than solving it by rollback the WAL.
see https://issues.apache.org/jira/browse/IOTDB-1136?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&focusedCommentId=17274179#comment-17274179

Thanks ， I will try to alter regular" encoding...  if can  not , I will give reason... we discuss again...

Thanks. look forward to making the encoder really available~

Thanks. look forward to making the encoder really available~

@jixuan1989 @jialin  hi, could you please review this code ?

Hi, Current description of the PR is clear.
But IMO,  the regular encoding is more suitable for the timestamp column... how do you think? (is there some cases that the value column obeys the pattern?)

sample: 1000，1100，1800，1400，1500...
current algorithm be do not use...
1800 is a error point, we should identity error point, revise data.
revise data should be : 1000，1100，1300，1400，1500

This is ... dangerous, as we modify the raw data directly and do not tell the user in time.

Hi, Current description of the PR is clear.
But IMO, the regular encoding is more suitable for the timestamp column... how do you think? (is there some cases that the value column obeys the pattern?)

sample: 1000，1100，1800，1400，1500...
current algorithm be do not use...
1800 is a error point, we should identity error point, revise data.
revise data should be : 1000，1100，1300，1400，1500

This is ... dangerous, as we modify the raw data directly and do not tell the user in time.

@jixuan1989  Thanks review.
（
是的，定频编码适合时间戳。
目前因为一个时间戳数据错误就导致整个ts flush失败，从而开启只读模式。
我觉得我们应该做一些事保证一个服务的稳定性。
因为

我们不能确定用户指定了定频编码，就能按照时间戳数据顺序插入。
2.如果插入这种数据，需要有容错能力，最起码不能影响同一个flush中的其他序列。有以下几个方法：
（2.1） 要么修改这个数据。
（2.2） 要么删除这个数据。
（2.3） 或者报错，但是此时数据已经写了wal,和内存，只是在flush的时候出错，所以是否需要清理该数据wal.

我目前的解决办法是：

尝试修复定频数据，并打印警告日志。
2.如果不能修复，输入的不是定频数据，那么我们抛出警告异常，然后删除错误数据，并且在日志中打印出来。）
Yes, constant frequency coding is suitable for timestamps.
However, the entire TS flush cannot fail because of a timestamp data error. In this case, the read-only mode cannot be enabled.

Because

If you cannot determine whether the code is specified by the user, the data can be inserted in the sequence of timestamp data.
If such data is inserted, the error tolerance capability is required.
(2.1) Modify the data.
(2.2) Delete the data.
(2.3) An error is reported. However, the data has been written to the wal and memory, and an error occurs only during flushing. Therefore, do you need to clear the wal?

My current solution is:

Restore the constant-frequency data.
If the fault cannot be rectified and the input data is not constant-frequency data, a warning exception is thrown, the error data is deleted, and the error data is printed in the log.

@jixuan1989 @Jialin hi, could you please review this code ?

Just FYI, you guys might tag the wrong person for review :)


Give private key contents

Is it possible to pass in the private key's contents to setPrivateKey?
$wrapper->setPrivateKey('/path/to/private/key');
Thanks.

@craftpip Did you mean:
$wrapper->setPrivateKey(file_get_contents('/path/to/private/key'));

For your example?

@cpliakas I think you close to quickly, I just fix the example.
@craftpip Would like to path the path of the key instead of the content.


[checkpoint]  Shard saved checkpoint need to be compatible with the naming format of hf checkpoint files

📌 Checklist before creating the PR

[ yes ] I have created an issue for this PR for traceability
[ yes ] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ yes ] I have added relevant tags if possible for us to better distinguish different PRs

🚨 fixed #3463

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?
fix shard saved checkpoint files to be compatible with hf file naming format

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

💥 Checklist before requesting a review

[ yes ] I have linked my PR to an issue (instruction)
[ yes ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ yes ] I have performed a self-review of my code
[ yes ] I have added thorough tests.
[ yes ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[ ] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

LGTM.


End-To-End tests expected tests pass / fail ?

In part 17, the end to end tests, it says to expect some failures:

This could take 18 hours or more! There are several thousand tests in the suite. The number of tests run and passed will be displayed at the end. Expect some failures as it tries tests that aren't supported by our cluster, e.g. mounting persistent volumes using NFS.

If one followed the instructions on the provided Vagrant cluster, what would a successful test output be? How many tests should it run, how many falures are expected ?
For example here is the output I got
Ran 1140 of 6971 Specs in 77497.301 seconds
FAIL! -- 1009 Passed | 131 Failed | 0 Pending | 5831 Skipped
--- FAIL: TestE2E (77498.52s)
FAIL

Ginkgo ran 1 suite in 21h31m38.605375964s
Test Suite Failed
2023/04/02 11:28:16 process.go:155: Step './hack/ginkgo-e2e.sh ”--ginkgo.focus=[Conformance]”' finished in 21h31m38.851260918s
2023/04/02 11:28:16 main.go:328: Something went wrong: encountered 1 errors: [error during ./hack/ginkgo-e2e.sh ”--ginkgo.focus=[Conformance]”: exit status 1]

Is that good or bad?
It would be useful if the lab notes set the expectation.

Sadly it is unknown exactly how many tests are expected to pass. None of us are members of the Kubertnetes testing SIG.
Put it this way. If the cluster wasn't built correctly, none of the tests would pass.


Monitoring

https://github.com/quakeservices/master/pull/108

Free third party accounts to check availability like Checkly?
Prometheus for metrics gathering internally?

Initially I thinking of exporting metrics to CloudWatch and generating alarms from those. Prometheus would be a good option as well. Depends on the pricing model in AWS. I figure the interface would be similar to how Storage is implemented so it's relatively simple to swap between services.
Synthetic tests (e.g. something sending a ping and expecting an ack) was another route I was planning with the client CLI. Probably spawned from a scheduled AWS Lambda or something.
Tracing would be awesome (X-Ray/Jaeger) but definitely something for later on. Need to divide all the options up into the minimum required for now,  and required for if enough people start relying on the service.

My day job is as an SRE/DevOps, so let me know what you think would work out best.  Running an internal Prometheus container could fit the bill to keep tabs on utilization, and external synthetics for user-facing uptime/performance, integrate them into an alerting service for specific triggers, etc.


First pass test matches ES6 modules as CommonJS modules making namedExports unusable

From the source the first time test detects using an simple regular expression could be a CommonJS module, trying to find expressions like "require" and "exports". As it's a simple test sometimes it matches those expressions in strings and comments (example: "required" from Apache License), making it try to transform the file as a CommonJS module.
When transforming the file it ends up not finding any exports and normally it returns null, except when namedExports is used. Maybe it can be changed to a warning and/or the first pass test me improved in some way to ignore strings and comments.

Thanks! Was a simple typo – fixed in 2.2.1.


subuid and subgid check?

I quicly read through the tasks, and this caught my eye: If /etc/sub[gu]id files don't list the user, podman will fail. I had the trouble in my role. Not all the tools add user to the files, like anaconda and if authorized by freeipa/IdM.
It might make sense to somehow at least check and notify about it. Better yet, add the line for the user.
Here's what I mean: https://github.com/ikke-t/podman-container-systemd/blob/master/tasks/check_subid.yml

ok - we'll have to do this in a loop for every user mentioned in any of the list of kube specs passed in
also - what to do about     line: "{{ container_run_as_user }}:165536:65536" - do we need a different, unique number for each unique user instead of hardcoding 165536:65536?

it should be different. Otherwise they all end up using the same space. I don't know how useradd does it. Is there a mechanism to ask useradd or any other command to generate the line?

The use cases that I've run into this recently is that when I set up our lab to use IdM (freeipa) with central auth management, the system doesn't create the sub stuff when you just login to host. I don't know if there was pam module to do the entries in such case.
Also in RH edge boxes do miss that. That's likely quite typical to run podman in edge boxes. One of the likely places to get the request from one day. The user creation there is likely done by anaconda or image builder customization. They use /usr/lib/passw instead od /erc/passwd, so user creation is a bit different anyhow.

if there are no additional IDs available, Podman can only use one single ID so you'd have to set to ignore chown errors in the storage.conf file.  Said so that is not optimal configuration since all IDs in the container are squashed to the same one.
freeipa supports remote assignment of additional IDs to the users and podman/new?idmap would pick them through libsubid.  Could that be used?

if there are no additional IDs available, Podman can only use one single ID so you'd have to set to ignore chown errors in the storage.conf file. Said so that is not optimal configuration since all IDs in the container are squashed to the same one.
freeipa supports remote assignment of additional IDs to the users and podman/new?idmap would pick them through libsubid. Could that be used?

If that isn't a suitable solution - what can/should we do in the podman role?  Should we implement something like https://github.com/ikke-t/podman-container-systemd/blob/master/tasks/check_subid.yml?

@richm, a check like that would improve the user experience.  I think it's worth doing it and guide the user to resolve the issue.  It's only required for rootless users.

@richm, a check like that would improve the user experience. I think it's worth doing it and guide the user to resolve the issue. It's only required for rootless users.

ok - let me see if I understand using pseudo code
for user in rootless_users:
  if user not in /etc/subuid:
    add user to /etc/subuid - $user:$sub_uid_num:$sub_uid_count
for grp in rootless_groups:
  if grp not in /etc/subgid:
    add grp to /etc/subgid - $grp:$sub_gid_num:$sub_gid_count

my problem is - how to I get the values to use for $sub_uid_* and $sub_gid_*?

You can use podman info --format for that:
~ $ podman info --format "{{.Host.IDMappings}}"
{[{0 1000 1} {1<PHONE_NUMBER>6}] [{0 1000 1} {1<PHONE_NUMBER>6}]}

If the output shows empty lists, I'd throw an error/warning.

From what I understand based on the RHEL 8.6 release notes, "Podman now has ability to source subUID and subGID ranges from IdM".  So for a IdM connected system, it seems like we would not want to have the role add subUID/GID information locally.
useradd on RHEL 8.6 automatically adds lines to the subUID/GID file
The "ansible.builtin.user" module also appears to automatically add lines to the subUID/GID file (just tried this on RHEL 8.6 with Ansible Core 2.12. )
It also appears that the user I created during a RHEL 8.6 interactive install with Anaconda was automatically added to the subUID/GID file (I would need to do another RHEL install to make 100% sure of this)
I tried useradd on a RHEL 7.9 system, and it also adds the entries to subUID/GID, so presumably systems originally built on RHEL 7 and upgraded to RHEL 8/9 would also have subUID/GID setup.
What circumstances to we anticipate that a user exists on the system but doesn't have the subUID/GID entries setup?


Mouse flicker if some options are enabled

Hey,
I get some mouse flickering if I enable the option:
"Disable Mod API statistics"
"Cache compiled mods"
(not 100% nailed down, but it occurs then)
Could also be because of an update from<PHONE_NUMBER> RichHudFramework
The flicker is inGame and inMenue after tabbing out of game.
I tried to get a original screenshot but it is hard to get. So I made a short clip
https://github.com/user-attachments/assets/38f7956f-7aff-4366-bf91-ea0be9b9b3fd
Recorded at a cryo pod without inventar interface open.
I get this at our torch server with some plugins and around 94 mods.
Give me some time to list all mods / plugins and settings here

RichHudMaster
Update: 17. Juli um 15:37
Version: <IP_ADDRESS>: Cursor Flicker Hotfix


feat: Get stream of CIDs from .forest.car.zst

https://github.com/ChainSafe/forest/issues/3565

Nice.


docs: use s2-react

👀 PR includes

✨ Feature

[ ] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[ ] Solve the issue and close #0

🔧 Chore

[ ] Test case
[x] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
s2 =》 s2-react

将 SheetComponent 的 引用从 s2 改为 s2-react
去除 s2 需要依赖的 antd 等说明

🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

应该还要在gatsby-browser.js里面加上s2-react配置


Track pilot sessions (Old 211)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:07
Track all pilot sessions.
Each flight should be an entry in the flights table.
Each 2 minute position report, stored in another and linked correctly.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:17
mentioned in issue #283

In gitlab by @A-Lawrence on Aug 31, 2015, 01:17
mentioned in issue #284

In gitlab by @A-Lawrence on Dec 20, 2015, 18:52
Removed ~98469 label


Switch to using URLCredentialStorage to securely store tokens in the keychain

Tokens are currently stored in the UserDefaults dictionary, which is highly discouraged to store sensitive data such as authentication tokens.
Switch to storing tokens using URLCredentialStorage (which stores permanent credentials in the keychain) or the keychain itself to store tokens.

That's something I will definitely want to enhance, thank you for mentioning that!


sys/linux: run make extract

Split out deprecated descriptions from socket_netlink_route_sched.txt. Adjust include definitions.
The only remaining big problem is kvm_dev.txt:
dev_kvm.txt: KVM_ASSIGN_SET_MSIX_NR is unsupported on all arches (typo?)
dev_kvm.txt: KVM_DEASSIGN_DEV_IRQ is unsupported on all arches (typo?)
dev_kvm.txt: KVM_DEASSIGN_PCI_DEVICE is unsupported on all arches (typo?)
dev_kvm.txt: KVM_ASSIGN_DEV_IRQ is unsupported on all arches (typo?)
dev_kvm.txt: KVM_ASSIGN_SET_MSIX_ENTRY is unsupported on all arches (typo?)
dev_kvm.txt: KVM_ASSIGN_SET_INTX_MASK is unsupported on all arches (typo?)
dev_kvm.txt: KVM_ASSIGN_PCI_DEVICE is unsupported on all arches (typo?)

I've tried all of the following, but none seems to work:
include <tools/include/uapi/linux/kvm.h>
include <../tools/include/uapi/linux/kvm.h>
include <../../tools/include/uapi/linux/kvm.h>
include <../../../tools/include/uapi/linux/kvm.h>

What was the intended way to do these includes?
A number of BPF consts disappeared, though they do not seem to be present even in Linux 5.10. Maybe we shouldn't care about them anymore?

@dvyukov the PR is obviously not ready yet, but I'd like to get some early feedback.

Okay, syz-extract includes the kvm file, but discards it because of errors. It feels like KVM headers are just broken in 6.8.
The consts we're interested in are now in tools/, but the common header file mentions struct kvm_vcpu_init:
https://elixir.bootlin.com/linux/v6.8-rc1/source/tools/include/uapi/linux/kvm.h#L1655
Which is arm64-only
https://elixir.bootlin.com/linux/v6.8-rc1/A/ident/kvm_vcpu_init
Are we supposed to always include arm64 headers?
But if I also include tools/arch/arm/include/uapi/asm/kvm.h, I get tons of errors like
In file included from /syzkaller/kernel/tools/include/uapi/linux/kvm.h:15,
                 from <stdin>:13:
/syzkaller/kernel/arch/x86/include/uapi/asm/kvm.h:111:8: error: redefinition of 'struct kvm_regs'
  111 | struct kvm_regs {
      |        ^~~~~~~~
In file included from <stdin>:11:
/syzkaller/kernel/tools/arch/arm/include/uapi/asm/kvm.h:61:8: note: originally defined here
   61 | struct kvm_regs {
      |        ^~~~~~~~

I don't think we are supposed to include any of the headers from tools/.
As a workaround, maybe we could remove some includes from the descriptions and redefine the missing structs?

Yes, we shouldn't use tools/, these are intended only for user-space tests and user-space tools. They usually duplicate official kernel headers.
We can either drop the removed ioctls, or define the consts manually with define and keep them for a bit.

For KVM, I've defined those consts in the descriptions -- they have been deleted not so long ago. Now it should be working.
PTAL


Javax Validator Auto Provide

javax I guess, should we even merge this?

I can pull it locally and release it from there without merging it into master.
I'll comment back into here when I've done that.

the jakarta one is still not on mvn central

hmm wait a minute, on my local it doesn't work

did we get this pushed?

did we get this pushed?

No, I've just done it now ... so version 2.9 is on it's way to maven central now.


Clean up test deprecation warnings for google-cloud-core

Logging integration test failure was quota related.


Making change of ipmi state easier

Softlayer builds fail when ipmi state is probe so we always change to absent before deploy. This makes it easier to set in defaults without having to edit code and worrying about committing it :smile:
Also threw baud_rate in to the role defaults since it looked lonely by itself in the task

This looks fine to me.

LGTM, merging.


Querying ranges of dates via the API

Just wondered if there were any plans to allow querying between a range of dates for the views that already allow selection by date.
I'm writing a python wrapper for the api part of which I want to be able to query for a range, but as it stands either it seems a little inefficient retrieving the whole timeline, or requires repeated calls per month. Might also make testing/writing tutorials a little more robust as you could ensure each person would get the same result even after more data is added.
I've got an example here that I haven't been able to test yet, though I'm not a fan of how the dates are input as it's a little ambiguous whether it's a list or a range.

Hi there, thanks for this.
We're not planning to add support for more dates, as internally we now use BigQuery for analysis on this dataset, and the API now only really drives the website. You might want to look at that instead.
Happy to give you access to our BigQuery dataset if that's of interest - just let us know a Google account name. We just ask that you cite us if you produce anything using it.

That would be much appreciated, though full disclosure I'm only a medical student, so can't guarantee any publications etc. If that's still ok I think glfharris should work.
Cheers!

@glfharris just given you access - you should have an email. "prescribing" is the dataset of interest.
It's fine to poke around and run queries, but please don't start scripting anything, because it'll get expensive fast :) if you want to do that then please let us know and you can fork the dataset - Google will give you a monthly quota to run queries for free.

I'm going to close this now, but shout if you object!


f90nml 1.3

Describe the bug
f90nml 1.3's string splitting (https://github.com/marshallward/f90nml/pull/120/files) can cause MAGICC running to break if you try to write a long list of string arrays. For example, f90nml will now write output like the below, which can't be parsed by MAGICC (because the commas are in the wrong place).
out_dynamic_vars = 'DAT_SURFACE_TEMP', 'DAT_SURFACE_MIXEDLAYERTEMP',
'DAT_TOTAL_INCLVOLCANIC_ERF', 'DAT_TOTAER_DIR_ERF', 'DAT_CO2_ERF', 'DAT_,  # this line is wrong
CO2PF_EMIS', 'DAT_CH4PF_EMIS'

Failing Test
TBD
Expected behaviour
The output should be like the below
out_dynamic_vars = 'DAT_SURFACE_TEMP', 'DAT_SURFACE_MIXEDLAYERTEMP',
'DAT_TOTAL_INCLVOLCANIC_ERF', 'DAT_TOTAER_DIR_ERF', 'DAT_CO2_ERF',
'DAT_CO2PF_EMIS', 'DAT_CH4PF_EMIS'

Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

OS: Linux
Python and Pymagicc version Python 3.7, Pymagicc 2.0

Additional context
It's unclear to me if this is a pymagicc bug or an f90nml bug. One for further investigation.

@lewisjared and @rgieseke fyi

I opened https://github.com/marshallward/f90nml/issues/131 the other day so a fix is in the works. I've been pinning 1.2 locally to resolve this

Legend

I've updated PyPI to 1.3.1 which fixes the errant comma but and also defaults back to the old behaviour.  I've also added an array string test so this should be detected in the future.

Thank you so much for your work! :-)
Used by NASA and in IPCC climate model pipelines ... quite a userbase!

Legend thanks @marshallward !


New Frontend Project: Drum Game

Project title
Drum Kit
Project Description
The Drum Kit game is a simple but fun game that allows you to play drums on your computer. The game is built using HTML, CSS, and JavaScript, and it is fully responsive, meaning that it works on all devices.
Project Working
To play the game, simply press the keys on your keyboard to hit the drums. The different keys correspond to different drums, so you can create your own beats and rhythms.
Doc Domain

[x] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ? SSOC'23

Hey @Vikash-8090-Yadav, it would be great if this project(Audio-based new type) is added in your repository.
Please assign if you like it.

Can i also try to work on this game @Vikash-8090-Yadav ??
looking for my first contribution in SSOC'23

@Vikash-8090-Yadav  I am almost done with making of this game.
Will create a merge request by today.


Add 1016 to spikeglx

Requested via slack

What's the difference between 1015 and 1016 for NP? (ie, would it be better to test against or do your current tests already cover it's broad features?)

From @jenniferColonell: "it's a short NHP probe with the linear site configuration, 960 sites, 1.0-like gain and referencing."

1016 is really a twin of 1015. It has a different part number because it's sterilized.


Fix Request::ip() ipv6 padding.

I noticed that some webservers remove padding on ipv6, which causes issues as the model primary key is the fully padded ipv6 (39 chars).
As a result I created a custom Req(uest) class in \App\Hub that extends the Illuminate Request Facade so we can pad ipv6 at the source.
Most controllers are using the new Req, however the maps still need to be fixed.

Following needs to be fixed:

app/Http/routes.php
resources/views/profile.blade.php

Fixed in #30


After runinng influxd upgrade command successfully I still see Get Started page

Steps to reproduce:

sudo apt install influxdb2
influxd upgrade --username='admin' --password='adminpass1234' --org='Rudik' --bucket='Kubik' --force"

Upgrade successfully completed. Start the influxd service now, then log in","login_url":"http://<IP_ADDRESS>:8086\
Expected behavior:
I should see influxdb dashboard page
Actual behavior:
When I open http://<IP_ADDRESS>:8086 link I see Welcome page and there is a button "Get Started". After clicking on the button it redirects me to the "Setup Initial User" page where I need to fill all data again which was already provided in this command influxd upgrade --username='admin' --password='adminpass1234' --org='Rudik' --bucket='Kubik' --force" .
Environment info:

Linux 4.9.0-9-amd64 x86_64
Old version: influxdb InfluxDB v1.8.10 (git: 1.8 688e697c51fd)
New version: InfluxDB version: InfluxDB v2.2.0 (git: a2f8538837) build_date: 2022-04-06T17:36:40Z

Config:
bolt-path = "/var/lib/influxdb/influxd.bolt"
engine-path = "/var/lib/influxdb/engine"

http-bind-address = "<IP_ADDRESS>:8086"
reporting-disabled = true

I have the same issue, any advice / resolution?

Some here when do we resolve this, can we get triage on that?

We have found a solution that resolved it for us here:
https://community.influxdata.com/t/data-lost-after-upgrade/21686

Closing in favor of the community forum solution, please file a new issue if there are any further issues. Thanks!


error="failed to query source: no nodes identified within pool"

Hello,
im currently testing the new os-nova plugin on openstack. I have near to no experience with autoscaling and have created a configuration from your readme.
I can start the autoscaler, the plugin is detected but I get the above mentioned error (which seems to come from nomad as I haven't found it in the plugin code.
Here's my "policy":
scaling "worker_pool_policy" {
  enabled = true
  min     = 1
  max     = 2

  policy {
    cooldown            = "2m"
    evaluation_interval = "1m"

    check "cpu_allocated_percentage" {
      source = "nomad-apm"
      query  = "percentage-allocated_cpu"
      strategy "target-value" {
        target = 70
      }
    }

    target "os-nova" {
      dry-run = false

#     evenly_split_azs   = true
      stop_first         = true
      image_id           = "15f26180-7806-474a-8981-cdeb03ef9acf"
      flavor_name        = "2C-2GB-20GB"
      pool_name          = "nom-pool"
      name_prefix        = "nom-"
      network_id         = "9e7bd5ca-eba3-49b6-a522-9d2ea6cc52a5"
#     user_data_template = "user-data.gotxt"
      security_groups    = "sg_nomcl"
      availability_zones = "az1"
      tags               = "nom-pool,ubuntu-minimal"

      node_class                    = "wrkr-test"
      node_drain_deadline           = "1h"
      node_drain_ignore_system_jobs = false
      node_purge                    = true
      node_selector_strategy        = "least_busy"
    }
  }
}

And here's my autoscaler config:
nomad {
  address = "http://<IP_ADDRESS>:4646"
  region  = "stage"
}

http {
  bind_address = "<IP_ADDRESS>"
  bind_port    = 9999
}

apm "nomad-apm" {
  driver = "nomad-apm"
}

target "os-nova" {
  driver = "os-nova"
  config = {
    auth_url    = "https://my.openstack.api:5000/v3"
    username    = "myname"
    password    = "mysecretpassword"
    domain_name = "mydomain"
    project_id  = "a66cd244776d4602bf88eb6a663c287a"
    region_name = "RegionOne"
  }
}

I don't know, whether they're even valid. Nomad autoscaler docs are not as comprehensive as other hashicorp docs.
Any hints?

Hi @neuroserve! It's great that you're trying to use the plugin!
I think that the error you're getting it's because there're no nodes with the specified node_class.
In the nomad-autoscaler code (in the nomad APM plugin).
	// Perform our node filtering so we are left with a list of nodes that form
	// our pool and that are in the correct state.
	nodePoolList, err := scaleutils.FilterNodes(nodes, id.IsPoolMember)
	if err != nil {
		return nil, fmt.Errorf("failed to identify nodes within pool: %v", err)
	}
	if len(nodePoolList) == 0 {
		return nil, errors.New("no nodes identified within pool")
	}

I think that the IsPoolMember function verifies nodes matching the node_class. So you should change that parameter to class already present in the pool of nodes.
I hope this helps.

Thanks. It works now. As the Nomad APM docs state, you cannot scale down to zero when using that APM. As soon as I had a Nomad Node with the defined node_class, the scaling started and launched its first instance.

That's great! Let me know if you need anything else. I'll close this issue.


[Feat] Improve Linear and Binary Search Page

Description
Description:
Adding Content and Improving Linear and Binary Search Page
https://frontendfreaks.vercel.app/batch/dsa/search

I want to work on this issue, Can I take this up, @Vishal-raj-1 ?

I want to work on this issue, Can I take this up, @Vishal-raj-1 ?

Assigned to you - complete it ASAP

Hey @ranitasahaa1230 Any Update?

if its not started then you can assign it to me sir.

Hi @Vishal-raj-1, I have started and will raise the PR by Sunday. Is there any format or anything that the content should look like?

Hey @ranitasahaa1230 Any Update?

Hi @Vishal-raj-1, I have started and will raise the PR by Sunday. Is there any format or anything that the content should look like?


Possible innerJoin and eager bug

Hi guys,
I think I've encountered a possible innerJoin / eager bugs.
Here is my setup:
DB
Person table

Person_Movie table

Movie table

Express route
app.get('/person/movie/:name', function (req, res, next) {
    Person
      .query()
      .eager('movies')
      .joinRelation('movies')
      .where('movies.name', 'like', req.params.name)
      .debug()
      .then(function (actors) { res.send(actors); })
      .catch(next);
  });

Results for '/person/movie/Wolverine':
[
  {
    "id": "1",  //duplicate ids
    "parentId": null,
    "firstName": "Hugh",
    "lastName": "Jackman",
    "age": null,
    "address": null,
    "personId": "1",
    "movieId": "1",
    "name": "Wolverine",
    "movies": [
      {
        "id": "1",
        "name": "Wolverine"
      }
    ]
  },
  {
    "id": "1", //duplicate ids
    "parentId": null,
    "firstName": "Ryan",
    "lastName": "Reynolds",
    "age": null,
    "address": null,
    "personId": "2",
    "movieId": "1",
    "name": "Wolverine",
    "movies": [
      {
        "id": "1",
        "name": "Wolverine"      //where is Deadpool?
      }
    ]
  }
]

As you can see above as soon as I use innerJoin('movies') my IDs are all the same. I've tested it with more data and I was experiencing the same issue.
Another thing that I noticed is that I'm not receiving Deadpool movie for second actor when using eager('movies') relation. Bellow you can see that when I search using Deadpool I'm receiving corresponding actor with all his Movies.
/person/movie/Deadpool
[
  {
    "id": "2",
    "parentId": null,
    "firstName": "Ryan",
    "lastName": "Reynolds",
    "age": null,
    "address": null,
    "personId": "2",
    "movieId": "2",
    "name": "Deadpool",
    "movies": [
      {
        "id": "1",
        "name": "Wolverine"
      },
      {
        "id": "2",
        "name": "Deadpool"
      }
    ]
  }
]

It seems to me that as soon as I start getting multiple results IDs get weird and eager is not returning all related data.
These may be 2 different bugs but I've decided to post them together. Hopefully that's ok.
Thanks!
EDIT:
Models were taken from example/express.

Forgot to paste debug SQL queries
select *
from "Person"
  inner join "Person_Movie" as "Person_Movie_rel_movies" on "Person_Movie_rel_movies"."personId" = "Person"."id"
  inner join "Movie" as "movies" on "Person_Movie_rel_movies"."movieId" = "movies"."id"
where "movies"."name" like 'Wolverine';

select "Movie".*, "Person_Movie"."personId" as "objectiontmpjoin0"
from "Movie"
  inner join "Person_Movie" on "Person_Movie"."movieId" = "Movie"."id"
where "Person_Movie"."personId" in (1)

After further looking into this... I think eager part of my issue is not an issue.... But confirmation is needed.
Thanks

Actually this is not a bug. Both the Person and Movie have the column id and since you don't select anything the default select * is left in the query and the id colum gets mixed. The eager fetch is then made using the id columns of the fetched models and it gets messed up also.
This should fix your problem:
app.get('/person/movie/:name', function (req, res, next) {
    Person
      .query()
      .select('Person.*')
      .eager('movies')
      .joinRelation('movies')
      .where('movies.name', 'like', req.params.name)
      .debug()
      .then(function (actors) { res.send(actors); })
      .catch(next);
  });

That makes sense. Should this example/scenario be in documented in docs?

Maybe.. This is how SQL joins work, but maybe it could be made clearer that joinRelation doesn't alter the select clause.

I'll just leave it here for now. People always search through past issues before posting new one :)
Thanks!


Creating variable outside handler function in SSR code

First, this library rocks.  Super impressive and I'm excited to get going with it.
To adopt, I will need a server-side cache in the lambda across lambda invocations.  This is easy in a lambda.  Just define a variable outside your handler function.  How would I achieve it?
let html; // this is reused between invocations
const loadHtml = async () => {
  if (!html) {
    // load HTML from somewhere
    // this is only run during cold start and then cached
  }
  return html
}
module.exports.handler = async (event) => {
  return {
    statusCode: 200,
    body: await loadHtml(),
    headers: {
      'content-type': 'text/html; charset=UTF-8'
    }
  }
}

Hi @bxgrant. Thank You for your interest in my project!
Yes, it certainly is possible to implement caching in this way. However, as every SSR request is handled by the same lambda in my implementation (this includes POSTs and PUTs etc), you would have to implement your own way to determine if you wish to cache a request and derive its cache key yourself.
This could be done by having your html variable be an object, keyed by some derivative of the request, only taking into account the headers, cookies, and query params you are interested in caching. I would envisage the flow, in this case, to be as follows;

Request comes in, a cache key is derived from it, and that keys existence is then checked for in the global object containing all requests cached by that lambda.
If the key exists, return the cached HTML. If not, render the response.
Before responding to the user with the rendered response, check for a cache header indicating you want the lambda to cache this response, if present store the rendered html in the global object against the key derived in step 1.
If you care about TTL, you would also need a system to handle that manually as well.

However, this seems overly fiddly and, as you may know, will not persist between concurrent invocations. May I suggest you look at API gateway caching. I'm not using an API gateway for this adapter because for most use cases it's just an unnecessary added cost in the chain now Amazon has added support for Lambda function urls. This approach will give you all the benefits of server-side caching (as API gateway caching is done by a dedicated provisioned node rather than at the edge) but as a managed service with data sharing between concurrent functions. The downside of course is cost.
If you're interesting, this is the serverless plugin I've used in the past to config this https://www.serverless.com/plugins/serverless-api-gateway-caching. You will also need to rejig your serverless.yml file to use an API gateway, this is fairly simple to do and you can see this old version of the adapter to see how it could be done.
Before I embark on any of these tho, I would try testing with just the edge caching CloudFront provides, and see how you get on, with the right TTL and cache policy (only caching headers, cookies, and query params that are strictly necessary) you can achieve very high cache hit rates without having to resort to any of these methods.
Hope this helps
James


Fix forcing HDMI port 1 to 1024x768 mode.

Closing this as the HDMI2USB-jahanzeb-firmware is no longer under development.
All active development is now on the HDMI2USB-misoc-firmware which supports both the Atlys and Opsis boards and has loads of other new features!


Komorebi widget does not update when changing workspace to one with no windows

Expected Behavior
Say you have windows open on workspaces 1 and 2, but not 3. When switching from 1 to 2, the widget updates. But when switching from 1 to 3, the widget is stuck on 1 even though the workspace changes.
Current Behavior
Widget does not update when switching to workspace with no current windows open.
Context (Environment)
Windows 10, komorebi latest release 0.1.21.

Occurs for me as well on windows 11, komorebi 0.1.21

I just updated komorebi to the latest version 0.1.22 and the issue seems to be resolved. @Zerogaku


[💡 FEATURE REQUEST]: Endure, singnal when all plugins are started

Plugin
ref: https://github.com/roadrunner-server/roadrunner/issues/1654
I have an idea!
Add an additional blocking method call to the endure container that would notify that all plugins have been successfully started.

Done


Wrong equation numbering position if page width is set to auto

Description
When using #set page(width: auto) the position of the equation numbering is not the expected position.
In this case the numbering is right next to the equation and therefor not vertically aligned.
My expectation is that the numbering is at the right of the page and vertically aligned with all the other equation numbering.
This is the behavior when using a set page width.
I asked about this behavior in the discord, and it was recommended to open an issue here.
Discord question
Reproduction URL
https://typst.app/project/r5yqqwpud7upZfbIkY52qP
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

I'm think this is a WAI. A page's width is Smart::Custom(Paper::A4.width().into())) by default so this issue doesn't manifest unless the page width is specifically set to auto.
Conceptually, if the page width is auto, "the pages will grow to fit their content on the respective axis" according to the doc, so Typst doesn't know where the put the equation number -- it wants to put it at the end of the X axis, but there's no "end" due to the auto width.
Practically, if the page width is auto, the region's X size is set to Abs::inf(): https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/layout/page.rs#L370 Therefore, when Typst lays out the equation number, it has no other clue to determine the row's width this equation should take, other than relying on the width of the equation itself: https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/math/equation.rs#L475-L479

I think this is a WAI. A page's width is Smart::Custom(Paper::A4.width().into())) by default so this issue doesn't manifest unless the page width is specifically set to auto.
Conceptually, if the page width is auto, "the pages will grow to fit their content on the respective axis" according to the doc, so Typst doesn't know where the put the equation number -- it wants to put it at the end of the X axis, but there's no "end" due to the auto width.
Practically, if the page width is auto, the region's X size is set to Abs::inf(): https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/layout/page.rs#L370 Therefore, when Typst lays out the equation number, it has no other clue to determine the row's width this equation should take, other than relying on the width of the equation itself: https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/math/equation.rs#L475-L479

I think this is a WAI. A page's width is Smart::Custom(Paper::A4.width().into())) by default so this issue doesn't manifest unless the page width is specifically set to auto.
Conceptually, if the page width is auto, "the pages will grow to fit their content on the respective axis" according to the doc, so Typst doesn't know where the put the equation number -- it wants to put it at the end of the X axis, but there's no "end" due to the auto width.
Practically, if the page width is auto, the region's X size is set to Abs::inf(): https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/layout/page.rs#L370 Therefore, when Typst lays out the equation number, it has no other clue to determine the row's width this equation should take, other than relying on the width of the equation itself: https://github.com/typst/typst/blob/c4dd6fa06227b8c4a04cf488f371f52d5379ef9b/crates/typst/src/math/equation.rs#L475-L479

You're explaining why the result looks like this currently, but surely this cannot work as intended. There is no real reason why the equation number couldn't be right-aligned.

Hey @Enivex I explained why in the "Conceptually" paragraph before the implementation details.

Hey @Enivex I explained why in the "Conceptually" paragraph before the implementation details.

I get that, but I don't agree with the reasoning. It doesn't have to work like that even if the page width is auto.


Website 🧑💻 : Remove the Extra spacing from the sidebar

At the moment the website sidebar has a lot of extra spacing, which is not looking good on desktop screens.
Current sidebar  :

Expected result :

Additional info
It will be great if we are able to manage more info in that extra space that is left over after fixing on sidebar spacing and utilize that space by adding more info around the docs, also it will be great if we are able to achieve responsiveness around different screen sizes ( suggestion:  to cover responsiveness in a separate issue)

cc  : @joewxboy , @johnwalicki

Hi I'm new to open source and i will be glad to make this my first contribution, can you guide me on how to make the contribution?

Hi I'm new to open source and i will be glad to make this my first contribution, can you guide me on how to make the contribution?

ya sure @Rasheedatj, it will be great if you first setup the project locally by following the contributing guide, also join the org's community channel here on the matrix in case you get questions or doubts regarding the setup and project in general.

Hey @joewxboy can you assign this to me?


Configuration for Internal Styles

New Feature Proposal
For more information, see the CONTRIBUTING guide.
Description

When you're using the modern isolation setting in piral.json , a <piral-component> is used to wrap the pilets in, which is nice. However, is there a setting to prevent the inline style (display: contents) being set on that wrapper element? in our case there are content security policies in place which prevent us to use inline styling unfortunately.

We can have a setting for the Piral-given styles in the piral.json called internalStyles.
There could be 3 settings for "internalStyles":

"inline" (default; as of today)
"sheet" (puts an "internal" / "piral-core" included stylesheet into the bundling process)
"none" (essentially puts you in charge of defining / or leaving out those styles; could also be named "custom", but "none" could be a better name as it means "nothing applied from the framework)

Background
This has been discussed / brought up in the Discord.
Discussion
Anything else to consider?

Landed in the preview - works.


feat(config): load local deno.json file as import map

What kind of change does this PR introduce?
feature
What is the current behavior?
Currently each edge function shares a global import_map.json by default.
To use one particular import map per function the user has to define it manually in config.toml
What is the new behavior?
We now automatically load deno.json and deno.jsonc under the supabase/functions/<slug> directory.
Each function should be treated as a different project with its own set of dependencies.
Additional context
import_map.json disappeared from Deno's doc since as of v1.19.2: https://deno.land/x/manual@v1.19.1/npm_nodejs/import_maps.md?source
Imports should now be declared as part of deno's config file deno.json.
We might want to deprecate import_map.json, especially when we will ship deno v2 support.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 59.829%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>7:
0.03%

Covered Lines:
6382

Relevant Lines:
10667

💛  - Coveralls

As per the additional context in the description can a global deno.json now be used instead of an import_map.json with this change?


Bug in last example

The request body in the last example is invalid. The changeId property has two 'beforeTime' entries instead of one 'beforeTime' and one 'afterTime'

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

ID: 733fcd9b-e234-8765-a7e0-498ded659ddc
Version Independent ID: bd115a3e-37c9-66c9-da11-68b46b563275
Content: Get resource changes - Azure Resource Graph
Content Source: articles/governance/resource-graph/how-to/get-resource-changes.md
Service: resource-graph
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

#assign:DCtheGeek

@jtuliani Thanks for the feedback and catching this. I'll get it fixed right away! When the fix merges, this issue will automatically close.  Thank you again for catching it and pointing it out!
@MicrosoftDocs/azure-cxp-triage Please label 'doc-bug' and #in-progress.

@DCtheGeek Added 'doc-bug' and 'in-progress' labels.


No option to create container in a bridged network

So I've had trouble with trying to follow this guide.
http://docs.docker.com/engine/userguide/networking/get-started-overlay/
Most of the commands works, but falls over when creating the swarm hosts. Basically the port 3376 is closed due the container being created using a shared interface.
The docker-machine gets to last stage then reports
Setting Docker configuration on the remote daemon...
Error creating machine: Error running provisioning: Unable to verify the Docker daemon is listening: Maximum number of retries (5) exceeded

looking further into it, when using eval $(docker-machine env --swarm mhs-demo0)
I get
"<IP_ADDRESS>:3376": dial tcp <IP_ADDRESS>:3376: getsockopt: connection refused

Which makes sense because there is no port forwarding for this port, but each swarm host 'needs' this port. So manually setting the port forwarding only works for one host, which is pointless.
Is there an option to create the host with docker-machine's parallel driver specifying the network type and adapter? If not is there any future plans in resolving this issue? As the above case is pretty useful for trying out a swarm deployment before it's deployed onto real servers.

@jahead Seems like you've made a mistake. The port of Docker daemon is 2376, not 3376.
And also, while creating the Swarm hosts via parallels driver, you should advertise the port on interface eth0 instead of eth1, for example:
$ docker-machine create -d parallels \
    --swarm \
    --swarm-discovery="consul://$(docker-machine ip mh-keystore):8500" \
    --engine-opt="cluster-store=consul://$(docker-machine ip mh-keystore):8500" \
    --engine-opt="cluster-advertise=eth0:2376" \
  mhs-demo0

Considering these notes, I've managed to create Swarm cluster from the "Get started guide" (https://docs.docker.com/engine/userguide/networking/get-started-overlay/)  without any issues:
$ docker run -it --rm --net=my-net --env="constraint:node==mhs-demo1" busybox wget -O- http://web

Connecting to web (<IP_ADDRESS>:80)
<!DOCTYPE html>
<html>
<head>
<title>Welcome to nginx!</title>
...

Closed.


MongoDB - Bulk insert

To insert bulk of Documents to a Collection.

PR# https://github.com/kdhrubo/db2rest/pull/555


DD-312: Extend unit tests for NARCIS classification mapping

Fixes DD-312
Description of changes

Convert toBasicInformationBlockSubjectCv test to a TableDrivenPropertyChecks style test to test the other audience classifications.

How to test
Related PRs
(Add links)
*
Notify
@DANS-KNAW/dataversedans

See my previous comment.

See my previous comment.


Add multithreaded pattern/fp generator

This is a simple addition that allows faster creation of libraries.
holder = rdSubstructLibrary.CachedSmilesMolHolder()
for smi in pdb_ligands:
      holder.AddSmiles(smi)

lib = rdSubstructLibrary.SubstructLibrary(holder)
rdSubstructLibrary.AddPatterns(lib, numThreads=-1)

This comes with a more full wrapping of the substructlibrary

@greglandrum If this compiles, I think it's ready.


lint php files on travis builds

this makes the travis build break when syntax errors are contained in *.php files.
to prevent errors like https://github.com/Microsoft/tolerant-php-parser/pull/62

Hi @staabm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@staabm :+1:  but it should be restricted to the src directory only

as /tools and /test etc. also cotains php sources I just included all the php files to make sure we dont miss anything.
its a matter of a few seconds anyway, so no need to complicate things.

the last missing error will be fixed by https://github.com/Microsoft/tolerant-php-parser/pull/65.
so after #65 landed this should get a green build.

@staabm thanks for the contribution! Any chance you could also update to disclude syntax-visualizer/example? (also left a comment in #65 to explain why that's there)
For reference, the directories that should be included are:

src
tests (except subdirectories)
experiments
tools
validation (except subdirectories)
syntax-visualizer/server/src/*

fixed per comments and rebased


[Teaser] The image authored in the Teaser react core component is not rendered in the webpage

Describe the bug
I configured an image in the teaser react core component but the image is not being rendered in the webpage. The teaser core component model is not returning the imagePath value and hence the teaser typescript code is failing the render the image.
Package version
AEM Cloud
Core Components version 2.11.1
JRE version Java(TM) SE Runtime Environment (build 11.0.6+8-LTS)
To Reproduce
Steps to reproduce the behavior:

Go to 'http://localhost:4502/editor.html/content/aem-react-core-spacomponents-example/library/teaser.html'
Author the teaser example component
Drop the asset from DAM
Image is not displayed in the webpage

Expected behavior
When the image is authored in teaser component, it has to be displayed in the webpage.
Screenshots

This seems to be an issue with the Sling Model of the Teaser Core Component. Please open an issue in [0]
[0] https://github.com/adobe/aem-core-wcm-components