[WIP] Basic support for non-nullable references

This WIP PR adds first support for taking the new C# 8.0 non-nullable references into account in model building. This adds convention logic that interprets non-nullable references in the same way as [Required] - for both regular and navigation properties.
Some notes:

No scaffolding work done yet, we still have to decide what our scaffolding strategy is for this.
The ConfigurationSource for non-nullable references is currently Convention, as non-nullability reference nullability seems weaker than [Required] or the fluent API, and should be overridden by them (there's a test for that).
For now I integrated the nullability support into the existing RequiredPropertyAttributeConvention and RequiredNavigationAttributeConvention (which I renamed to RequiredPropertyConvention and RequiredNavigationConvention respectively). I'm not sure how fine-grained we like our conventions to be, in theory it may be worth separating them in case somebody doesn't want the new nullability support. Also, we can consider removing NavigationAttributeNavigationConvention as it's no longer used by us - although users may still use it for their own custom attributes?
Some tests are missing (for non-nullable navigation properties) as well as log message changes.
Weirdly, the tests pass from the cmdline but Reference_nullability_sets_is_nullable_correctly(NullAwareNonNullable) fails in VS2019 - this is a result of seeing a [Nullable] version from before dotnet/roslyn#30143 (VS is somehow stuck behind, not using the latest Roslyn or something)? @bricelam we briefly discussed this, if you feel like taking a quick look to make sure I'm not crazy....

Since the Milestone of the Roslyn PR you referenced is 16.1.P1, it will show up in Visual Studio 2019 Update 1 Preview 1

@vslee thanks for the comment. There's still a bit of work before this can get merged, but hopefully that will happen quite soon.

Superceded by #15499


Warning about injection attacks

Your comment in the README:

A more thorough SQL injection security review

Had given me the impression that there was preventative measures already in place to prevent injection. Thankfully I reviewed the code before deploying, and saw that the only thing sanitized is the search string. Some people, even without reading the above comment, might not think to check for injection because they're used to working with ORMs, and might make assumptions about node-datatable having similar features.
Might be overkill, but might also not hurt to have a disclaimer in the API section saying

When using user-provided values as data for any part of the query, be sure to sanitize it first

@brynnbp can you make a PR that updates the README as proposed?


Replace Bruteforce search with Hashmap lookup in block_registry

Description
Multiple functions in block_registry would do a brute force search by iterating through all blocks until it found a match. This PR would add multiple hashmaps to improve lookup times and improve performance

Testing

Tested world generation, setblock command, block placing and destroying
Compared memory usage before and after changes on windows, added ~3.2MB of memory usage

Checklist
Things need to be done before this Pull Request can be merged.

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt
[x] Code does not produce any clippy warnings cargo clippy

I feel like this could run into the same problem we had with CChunkData, with the massive amount of accesses/hashing that has to be done it could cause overhead.

LGTM. I ofc still would love to have the block! macro back (I removed it when porting to own Extractor). But still looks good.
Thanks @PrintedScript


Update for 0.16

Would this maybe be a good library to transfer to elm-community?

Possibly, if you're not interested in maintaining it.

I'm still interested in maintaining it, it's more that I'm willing to share control, so that not every change has to go through me.

Seem reasonable. And it's simpler since it's not published.

Yeah, it never made the Native whitelist.

What's so special that can't be done in Elm? Just glancing over it, is it keeping benchmarkN well-typed for everything? Or does benchmark.js do something very helpful? I'd also be okay with the "library hands you something to send out a port" solution here.

There's nothing particularly special, but it would likely require a complete rewrite. Basically, my view was that I didn't want to deal with worrying about timers, or accuracy, or sampling, so the library is really just a thin wrapper around Benchmark.js. It's what people are used to, since jsperf uses it.
I have no opposition to a pure Elm solution, but I don't know if I'm qualified to deal with the lower-level issues to write it.

You'd probably need a task-based library to get the exact current time in order to time the benchmarks.
I think it's fine how it is, might just need a few things in the README.


Disable :W hotkey

It can be easily used by typo (instead of :w) and reverts all the changes.

Fixed in https://github.com/tim2CF/ultimate-haskell-ide/commit/057490e28fd340fd8581a11569959aa9dcda7727


[TIC4002-F18-6] MyContentPlanner

MyContentPlanner is for social media managers who need to track monthly social media content posting schedules so that they won't miss out on any days without posting content for their clients. Meanwhile, they prefer to use the command line to save the ready-made content link over the GUI and have tons of content to keep track of.

Codecov Report

Merging #5 (e3ff49a) into master (4f7660a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 4f7660a...e3ff49a. Read the comment docs.


NBEATSx: Retraining with the complete data including validation set has the issue of not being able to do early stopping

Description
Issue:
When deploying the nbeatsx model and making use of all the available data up to that date as training set (which should increase performance), I do not have any other future values. So how do I know after how many training epochs to stop?
Note:
I am fairly new in the game and might be overseeing something, so happy to take on any correction.
Thank you for your consideration and continuous work on improving the nixtla univserse and, with that, the state of forecasting!
Use case
Context:
In the way early stopping is currently implemented, I do not have the possibility to make use of all available data for training. This is striking as often the most recent data (say last week) has the strongest predictive power to forecast the next period (say next week). The original nbeatsx paper here proposes a random early stopping set as a solution (being a random period over the updated training set). This could be a very useful feature to permit the deployment of nbeatsx model in production with early stopping and use of all the available data as training data.

@cchallu: as you suggested I submitted an issue for the random early stopping set here (as in the NBEATSx paper). If there are any questions please let me know and I will be happy to provide more context.

@steffenrunge I think you can use cross-validation to perform what you want. Cross-validation will at the end refit the model with the entire training data based on the best hyperparams.


[DOC-2874] Harness CI - JIRA integration

Add topic for CI Jira integration using the Jira Drone Plugin and Harness CI Jira app.
Correct description of Container Registry field in Plugin step.
Add link to new topic from Explore Plugins topic.

Preview: https://64303a2803fb0468da86d1b8--harness-developer.netlify.app/docs/continuous-integration/use-ci/use-drone-plugins/ci-jira-int-plugin

Tab structure: One for "CI by Harness for Jira" app and one for "Drone Jira Plugin" (incoming app link method).

-Visual and yaml instructions for both

Based on info I've received, I'm leaving out PIPELINE & LINK for now.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Hey, @aimurphy I think it's confusing that at the top of the page we say install the app (btw - I would say install the Harness CI Atlassian marketplace app) , and then you give two options for authentication, but only the first one actually relies on that app

@nofarb Is "Harness CI" the name of the app? There is already CI Enterprise by Harness and I don't want to use the wrong name.

For the authentication, I believe the OAuth2 option is an incoming app link, at least according to the Atlassian docs? https://confluence.atlassian.com/adminjiraserver/configure-an-incoming-link-1115659067.html

@aimurphy  we are updating the existing CI Enterprise page

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Correct wording of Import tests from NPM tab

Changed 'Migrate' to 'Import' in the mention of the tab name, as per the picture immediately below it.

@SteveBurkettNZ : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Document reasons for yanking versions from https://crates.io

Numerous versions of ahash have been yanked from https://crates.io, but it is difficult to determine why any particular version was taken down. When dependent builds fail due to a yank, it could be useful and informative to see why the yanked version of ahash was removed from the registry, especially if build artifacts that include such a version of ahash are still out in the wild.
Considering the number of yanks (46 in total at the time of this writing), perhaps a "yank log" could be introduced so users can see why a version they've depended upon must be avoided. Thoughts?

Done:
https://github.com/tkaitchuck/aHash/wiki/Yanked-versions

Awesome! Thanks for putting this together!


Implement makeWin98Image

I tested this only on x86_64-linux. Sorry if some things weren't done the best way.

You should add a repeatability test, since it will prove it works 100 times like the rest, I'm pretty impressed though.

You should add a repeatability test, since it will prove it works 100 times like the rest, I'm pretty impressed though.

Done. Do I need to run it myself or does the Hercules CI do it?

@io12 I have to bring your PR into the repo as a branch, which I just did, now the CI will start running.

It seems to not be repeatable unfortunately due to keypresses dropping. I'll try to fix it.

@io12 Yeah, one of the todos is perhaps to try xdotool to get the keypresses to stop dropping, I've found vncdo to be terrible.

Okay, I think it's more reliable now. Can you pull from the branch to run the CI again?

@io12 It appears that 10 of the instances have gotten stuck at the following step

Whereas two others are stuck at this without continuing

In testing the other windows/msdos releases in the repo that I've tested, only 1 in 1000 would fail due to what seems like runtime bugs with dosbox, which is fine. But the other failed states can be worked around. I'm happy with 1 in 1000 failure rate, maybe even 1 in 100 is good enough for the purposes of the repo.

Kind of frustrating that it's stuck at the final step 🙃. I'm not sure whether it's not fully booted yet and isn't ready to open the start menu or some other issue. Also, how did you get the screenshots?

Nix does its work in /tmp, so I just go in there and extract the screenshots, same way as the macos repeatability test defined in this repo works.

I made more repeatability improvements. Can you run the CI again?

@io12 some fail like this, maybe dosbox's fault?

Some seem to still get stuck at the end like this

@io12  Statistically, your change did have an impact, as now at least 68 in 100 attempts work, as opposed to the previous 22.

Maybe we could just kill -9 the process when it gets to the start menu instead of trying to shut down properly?

Maybe we could just kill -9 the process when it gets to the start menu instead of trying to shut down properly?

That would work but then there's a ScanDisk improper shutdown message on the next boot. I think the issue is sometimes when the start menu is open before the "welcome to windows" window, the start menu can close. I think maybe the failing cases try to select the shutdown option after this happens. I'm trying to fix this by repeating the ctrl-esc u enter combination until the desktop icons and start menu disappear.

@io12 rarely, this failure happens

Okay, I think I fixed both issues and it repeats locally 16 times. Hopefully it works all 100 times now.

@io12
Wow, the failure rate now seems to be 3 in 100.
The three that have failed, have failed in a strange way though, as they have gotten to the ### OMG DID IT WORK???!!!! ### log line. Here are their logs for you to inspect.

https://hercules-ci.com/accounts/github/MatthewCroughan/derivations/%2Fnix%2Fstore%2Fi0vaa6j8kzcp5g9jc7ph9a611kbqqy21-94-win98.img.drv/log?via-job=b500803f-1d3b-49b8-86a0-e78a7e3a3b8c
https://hercules-ci.com/accounts/github/MatthewCroughan/derivations/%2Fnix%2Fstore%2Fi0vaa6j8kzcp5g9jc7ph9a611kbqqy21-94-win98.img.drv/log?via-job=b500803f-1d3b-49b8-86a0-e78a7e3a3b8c
https://hercules-ci.com/accounts/github/MatthewCroughan/derivations/%2Fnix%2Fstore%2F70d1nhsmzybrw0wnrcg02ziyp4a56l3a-48-win98.img.drv/log?via-job=b500803f-1d3b-49b8-86a0-e78a7e3a3b8c

There are the last three screenshots that were taken by the expect script, but the screnshots are no longer being taken by the process, as their last modified time is no longer updating periodically.

Thanks! Such a rare issue is pretty tricky to debug, but I removed a code path left over from a different approach that triggers in all three and doesn't trigger normally. Even if it doesn't fix the issue it should at least give more up-to-date screenshots.

Also, I wonder how hard it would be to patch dosbox-x to fix the non-repeatability issues. If the issue comes from the timing of keypresses, maybe we can make dosbox-x take an input file that automatically triggers keypresses at specified cycle counts? I'm not sure if this would actually make things fully deterministic, but maybe worth a shot? And I don't think this is the first time Nix packages have patched build tools to make them more deterministic?

1 in 100 seems end up like this LOL!

Whereas there are one other stuck like this

@io12 Have you got a matrix or discord we can talk more actively on?

@io12 Have you got a matrix or discord we can talk more actively on?

@MatthewCroughan Yes you can send a friend request to io12_ on discord.


Please add the config in the 1.19 version

Hi, can you make the 1.19 version configurable? I just used your mod (1.1 version), and while it did make structures a lot less common, I've gone from like 30 structures to like 4 structures per region, which isn't necessarily ideal for my modpack. My modpack is stuck in 1.19 so I can't upgrade.
If it's not possible for any reason, would it be permissible within your license for users to modify the jar file themselves? If so, how can I go about it?

Thank's to #7 and @natpalmer-e4o4, this is planned!

Oh my god that is amazing!!!!

The update is now available on GitHub, Modrinth and soon on Curseforge (has to be approved first)


Delete LICENSE-CODE

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@Oldgold13 Please resolve this conflict so that this PR merge can be initiated. Thanks.

@Oldgold13 Please resolve this conflict so that this PR merge can be initiated. Thanks.


Add flat sprites as NPCs

Each region will have NPCs that can be controlled with the mouse, their speed must follow the game speed.

https://github.com/FOSS-Supremacy/cursed-world/issues/9


Wrap Digits

Is it possible to wrap each individual digit with a span?

Yup, check-out this example: http://hilios.github.io/jQuery.countdown/examples/website-launch.html
$('#clock').countdown('2020/10/10').on('update.countdown', function(event) {
  var $this = $(this).html(event.strftime(''
    + '%-w week%!w '
    + '%-d day%!d '
    + '%H hr '
    + '%M min '
    + '%S sec'));
});

@hilios, not like that, I think @picard102 needs +, for example DD-HH-MM to be:
DD-HH-MM
Is there a quick/light way to achieve this?


AOT Giving Generic Error Message when Component is declared in multiple modules

When serving a generated app with AOT enabled some errors are obfuscated behind a misleading error message at build time. This issue is specifically for the same declaration in multiple modules but I believe it happens for others as well but this is the one I have an example for.
Versions
Angular: 5.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.6.0
@angular-devkit/build-optimizer: 0.0.35
@angular-devkit/core: 0.0.22
@angular-devkit/schematics: 0.0.41
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.9.0
@schematics/angular: 0.1.10
@schematics/schematics: 0.0.10
typescript: 2.4.2
webpack: 3.10.0

Repro steps

clone repo
run npm start (command updated with --aot)
see error message

Observed behavior
non specific error message shown in terminal:
ERROR in ./src/main.ts
Module not found: Error: Can't resolve './app/app.module.ngfactory' in '/Users/dannyblue/Documents/projects/error-example/src'
resolve './app/app.module.ngfactory' in '/Users/dannyblue/Documents/projects/error-example/src'
  using description file: /Users/dannyblue/Documents/projects/error-example/package.json (relative path: ./src)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: /Users/dannyblue/Documents/projects/error-example/package.json (relative path: ./src)
    using description file: /Users/dannyblue/Documents/projects/error-example/package.json (relative path: ./src/app/app.module.ngfactory)
      no extension
        Field 'browser' doesn't contain a valid alias configuration
        /Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory doesn't exist
      .ts
        Field 'browser' doesn't contain a valid alias configuration
        /Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory.ts doesn't exist
      .js
        Field 'browser' doesn't contain a valid alias configuration
        /Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory.js doesn't exist
      as directory
        /Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory doesn't exist
[/Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory]
[/Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory.ts]
[/Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory.js]
[/Users/dannyblue/Documents/projects/error-example/src/app/app.module.ngfactory]
 @ ./src/main.ts 3:0-66
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts

Desired behavior
Terminal should display the correct error message.
In the example above the expected error message is. (This is what is shown in JIT mode)
Uncaught Error: Type ExampleComponent is part of the declarations of 2 modules: ExampleModule and AppModule! Please consider moving ExampleComponent to a higher module that imports ExampleModule and AppModule. You can also create a new NgModule that exports and includes ExampleComponent then import that NgModule in ExampleModule and AppModule.

this may be an angular core issue and not a cli issue as referenced. here https://github.com/angular/angular/issues/20292

@deebloo , have you tried Angular v5.1.1 ? Similar issue was fixed in v5.1.1

@PascalPrecht this looks a lot like what we were talking about yesterday. Maybe this is the real issue you are getting?

@Sky4CE I have not. will try now

@Sky4CE @filipesilva updating to 5.1.1 fixed it. I am now seeing the correct error message

https://github.com/angular/angular/commit/6727336 seems to be the commit that fixes it.

I am facing this same issue
.module.ngfactory.ts doesn't exist
.js        Field 'browser' doesn't contain a valid alias configuration
and seen these comments,
it seems like this line in packate.json
"@angular/compiler-cli": "^5.0.0",
need to be updated  as
"@angular/compiler-cli": "^5.1.1",
and install it.
Please confirm.
thanks,
Shobhit

Hi, I am facing same problem .module.ngfactory.ts doesn't exist
.js
Field 'browser' doesn't contain a valid alias configuration
It seems we need to update following to 5.1.1
Angular: 5.1.1
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Please let me know if that is correct.
thanks,
Shobhit


atomicparsley: 20210617.200601.1ac7c08 -> 20210715.151551.e7ad03a

and enable tests

Motivation for this change
#130662 but with a test
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

Result of nixpkgs-review pr 133312 at 97f61384 run on aarch64-linux 1

1 package skipped due to time constraints:

plex-mpv-shim

38 packages built successfully:

atomicparsley
botamusique
catt
celluloid
curseradio
get_iplayer
haskellPackages.cut-the-crap
hydrus
jellyfin-mpv-shim
jftui
kodi-cli
lollypop
minitube
mkchromecast
moodle-dl
mopidy-youtube
mpc-qt
mps-youtube
mpvScripts.mpv-playlistmanager
persepolis
play-with-mpv
python38Packages.mpv
python38Packages.pafy
python38Packages.tubeup
python38Packages.youtube-dl
python38Packages.youtube-dl-light
python39Packages.mpv
python39Packages.pafy
python39Packages.tubeup
youtube-dl (python39Packages.youtube-dl)
youtube-dl-light (python39Packages.youtube-dl-light)
qimgv
somafm-cli
spotdl
sublime-music
tartube
ytcc
ytmdl


Knmi data contains a duplicate index at 1-1-1970

Right now knmi-data is downloaded in ObsCollection.from_knmi(). KNMI-data from stations is combined to generate an equidistant series.
For some reason there is a duplicate index at 1-1-1970. I think the KNMI just sends this date twice. I think we should drop this duplicate index, to generate an equidistant series.

What code do you use to download the KNMI data? I don't get a duplicate when using this code:
stns = [344, 260] #Rotterdam en de Bilt
oc_knmi = oc.ObsCollection.from_knmi(stns=stns,
                                     meteo_vars=["EV24", "RH"],
                                     start=['1970', '2010'],
                                     end=['2015', '2015'],
                                     verbose=True)

I think you have to start before 1970 to get the double index:
stns = [344, 260] #Rotterdam en de Bilt
oc_knmi = oc.ObsCollection.from_knmi(stns=stns,
                                     meteo_vars=["EV24", "RH"],
                                     start=['1969', '1969'],
                                     end=['1971', '1971'],
                                     verbose=True)
ndup = oc_knmi.iloc[0]['obs'].index.duplicated().sum()
print('')
print(f'{ndup} duplicated indexes')

gives
Download EV24 from 344 Rotterdam
transform EV24, Potential evapotranspiration (Makkink) (in 0.1 mm); from 0.1 to 1
transform EV24, Potential evapotranspiration (Makkink) (in mm); from mm to m
station 344 has 734 missing measurements
Download EV24 from 260 De Bilt
transform EV24, Potential evapotranspiration (Makkink) (in 0.1 mm); from 0.1 to 1
transform EV24, Potential evapotranspiration (Makkink) (in mm); from mm to m
station 260 has 0 missing measurements
Download RH from 344 Rotterdam
transform RH, Daily precipitation amount (in 0.1 mm) (-1 for <0.05 mm); from 0.1 to 1
transform RH, Daily precipitation amount (in mm) (-1 for <0.05 mm); from mm to m
station 344 has 734 missing measurements
Download RH from 260 De Bilt
transform RH, Daily precipitation amount (in 0.1 mm) (-1 for <0.05 mm); from 0.1 to 1
transform RH, Daily precipitation amount (in mm) (-1 for <0.05 mm); from mm to m
station 260 has 0 missing measurements

1 duplicated indexes

I do get the double index in the Observation when starting at 1970 as well, I see now.

I do not get a double index in both cases. What package versions are using, I use:

hydropandas 0.2.7
pandas 1.1.3

I also get the duplicate index with pandas 1.1.2 but after upgrading to 1.1.4 there is no duplicate anymore....
Good idea to figure what in pandas is causing this difference...

Ah, it has to do with the normalize-method. For some reason pandas normalizes '1969-12-31 01:00:00' to '1970-01-01'. See this example:
from hydropandas.observation import KnmiObs
knmi = KnmiObs.from_knmi(260, 'EV24', startdate=pd.Timestamp('1970-1-1'),
                         enddate=pd.Timestamp('1970-1-1'), verbose=True)
print(knmi.index)
print(knmi.index.normalize())

Which gives:
transform EV24, Potential evapotranspiration (Makkink) (in 0.1 mm); from 0.1 to 1
transform EV24, Potential evapotranspiration (Makkink) (in mm); from mm to m
Download EV24 from 260 De Bilt
station 260 has 0 missing measurements
DatetimeIndex(['1969-12-31 01:00:00', '1970-01-01 01:00:00',
               '1970-01-02 01:00:00'],
              dtype='datetime64[ns]', freq='D')
DatetimeIndex(['1970-01-01', '1970-01-01', '1970-01-02'], dtype='datetime64[ns]', freq=None)

So it is a bug in Pandas, that has been fixed in the latest version. This can be closed.

It seems that all dates before 1970 are normalized to the next day in older versions of Pandas. This probably has to do with the fact that 1970 is 0 in the format in which the date is stored behind the scenes (is that english or just a dutch saying?).


Composed sync/async filters don't work

The query in this file will fail using the current mix of synchronous and asynchronous filters:
https://github.com/disco-network/rdfstore-js/commit/577f4b9a8702846cb2e0225e956e1397800cb9e1
It contains the following filter consisting of an asynchronously evaluated EXISTS expression composed with a synchronously evaluated filter ?p = 23.
FILTER (?p = 23 || EXISTS { ?x ns:price 42 })
It seems that the problem is that all the asynchronous work necessary for EXISTS clauses happens in preprocessExistentialFilters, a function only processing the filter clauses in the highest level instead of recursively processing the parts such a composed expression as above. Hence, our EXISTS clause won't be preprocessed.
Some weeks ago I managed solving issue #115 by making all filters async instead of introducing preprocessExistentialFilters to outsource all the async code - and it seems that this problem vanishes there. It shouldn't take a long time for me to update those lines of code to present them as possible fix - or do you prefer a solution with the filters working sync?

Hi @datokrat!
Thanks for finding the problem with the nested EXISTS filters.
I found the problem with these filters requiring async callbacks in f8c2127a81295d0a27d3218d591b3aa1e853f792 but I overlooked complex filter expressions.
My thinking about the issue is that EXISTS are the only filters requiring async handlers. I was thinking about trying to rewrite the filtering logic in an async fashion as you did, but it's a massive change and I would prefer saving a few calls in the stack.
I have added a fix in 618a37d1e2577e926d7d35d8596cfa9a5da8b3b0 that should extend my original solution to complex filters.
I have also added as much of your commit as possible, like the test cases (thanks!) to the project, but for the time being I really prefer this approach unless we hit a wall somewhere. Let's save your code thought just in case.
Thanks again for the time spent on this, take a look and see if you find any problems.

Okay, thank you, I just wanted to show up with a possible solution. I really like it that you're still working so actively on this repository.


Added docstrings for linesearch methods

Codecov Report

Merging #324 (71af9e9) into master (41542fb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #324   +/-   ##
=======================================
  Coverage   72.75%   72.75%
=======================================
  Files          14       14
  Lines        1960     1960
=======================================
  Hits         1426     1426
  Misses        534      534

Impacted Files
Coverage Δ

src/defs.jl
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Improve encoding performance of empty interface type

If you do deep recursive processing about interface{}, runtime.newstack will be called many times and performance will decrease, so allow you to process empty interface without consuming stacks.

Codecov Report

Merging #259 (f93d82d) into master (397a4e4) will increase coverage by 0.02%.
The diff coverage is 85.00%.

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
+ Coverage   80.97%   81.00%   +0.02%
==========================================
  Files          47       47
  Lines       14763    14845      +82
==========================================
+ Hits        11955    12025      +70
- Misses       2255     2266      +11
- Partials      553      554       +1


Introduce CountryCode validator (rework of #4142)

This is an update to #4142; I rebased the original branch and updated to include feedback on the initial prototype:

Bump copyright year + CS fixes
Remove unused resource bundle files
Removed hard-coded country names
Use static instead of self in Zend\I18n\CountryDb

There is one outstanding issue that I'm unsure of:  country codes are not locale-specific, so should this code (or at least the Validator class) be moved out of the I18n namespace?  If so, where should the resource class (Zend\I18n\CountryDb) be placed?
Thanks to @mwillbanks for the bulk of the work on this and also to those who left their feedback in the original PR (#4142)

@adamlundrigan and @mwillbanks — totally agreed; less functionality for us to support in the framework is a good thing, and this is indeed one of the motivators behind a module ecosystem (extending framework functionality).
Closing! Thanks for the work and the discussions!


crash

Every time I press start on the skin injector my league client crashes. Anyone know how to fix?

How can I guess the error if you don't give me an error message or something like crash dump.
Also, are you sure you're using the latest version I released yesterday?

Game crashed++


Add Required system param

Adds an alternative to Local without a FromWorld bound.

Looks interesting. What do you see this being used for?

Any situation where the local resource cannot be initialized automatically, and anytime say a plugin author wants to draw attention to a local resource being configurable.

Looks pretty good, however, I'm not so sure about the name Required.
Perhaps:
Configured
(can't really think of any others haha)
Or, would it be possible to overload the usage of Local either via a second generic type or trait which can statically change behavior?

My preference is definitely to override or extend Local in some way; the usage of the two APIs is identical except at initialization.
I suppose there's no way to default to this behaviour if FromWorld is missing?

I suppose there's no way to default to this behaviour if FromWorld is missing?

Specialization could probably make this work, but without it i'm pretty this is impossible if we want FromWorld to be "automatic" behavior.
We could remove the FromWorld constraint and make Local always require manual configuration, but that would be a pretty huge step down in usability.

Not a fan of having with Local and Required, or the fact that Required would fail only at runtime...
But I would love to have something that fails to compile without a .config() call...

If we're going to have another type for this, I think Configured is the best name so far. It guides the user toward the "config()" api and semi-implies that if you don't "configure" it, something bad might happen.

About naming: I like Configured. I had a hard time figuring out the meaning from Required.
About the feature: Could this help seeding multiple system local PRNGs from a single seed generator, as described in #2355?

+1 to Configured.

But I would love to have something that fails to compile without a .config() call...

I low-key assumed that was already the case when skimming it. Hm.

Could this help seeding multiple system local PRNGs from a single seed generator [...]

Not any more than Local itself could.

Would it be possible to have a const generic to specify if the Local need configuration?
struct Local<const N: bool>;

fn init<const N: bool>(test: Local<N>) {
    if N {
        println!("using default");
    } else {
        println!("need configuration");
    }
}

OK, so I just had a real-world use case for what this PR proposes. See: https://discord.com/channels/691052431525675048/742884593551802431/864264371106742312

I played with the const generic option in #2463, it works but I think it's not pretty in current Rust


Deploy new cg-cron-boshrelease

Point runtime config back to upstream cron-boshrelease. Has bug fix that will fix all cron jobs.

This is actually fairly important - some stuff currently breaks due to cron not working,


Metadata download on supported webservices when reading data

Change read_sdmx to use webservices by downloading any possible metadata we may find using the query builder.

Use detail=full & references=children. Use method get_dsd or get_dataflow.

When importing the class SdmxWebServiceConnection in api, it returns me this error because read_sdmx is used in the class:
ImportError: cannot import name 'read_sdmx' from partially initialized module 'sdmxthon.api.api' (most likely due to a circular import)
I understand the error but I don't know how to fix it or how to do it without importing that class.

Okey, we'll check it so.


FIXES #230: npm install after vue init fails if git user name has "-s

FIXES #230: npm install after vue init fails if git user name has double quotes in it
Issue URL : https://github.com/vuejs/vue-cli/issues/230

Closing in favor of #232.


init task

#8 issue: Initialize base task and generator task.
Important update:

delete init_model method in metric class. metric need to be initialized with a model argument.

TODO:

add test for task when metric is in a format of str (we don't have such metrics yet)

The conflict in this branch has been resolved, and some problems in formats have been fixed, especially the import/from part. Now most of the import/from parts of the code meet the requirements in PEP8 style.
But strangely, some of the commits have been merged before were also appeared in this branch.


WIP: Fix and update GHA workflow

Relevant issue: https://github.com/yokawasa/k8s-cronjob-prescaler/issues/2
Updates

Update CI target kubernetes versions:  https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/03e12e73b139d9d9097b3e41c0bdfd0f6c3a2d7c
Version up packages: https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/3e5bab6291a546c4f941deff12fe5aff0a349172 https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/8c1b66354fbfb44f918cfb03ebf606883a91806d https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/a5718b5ecac5584e153098f80ccfeb4301b82e45 https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/985f6d53d7cef5ab433f4284cd437bf6b7d8356a

kubebuilder: v3.1.0
kustomize: v3.8.7
kind: v0.11.1

Upgrade base image to golang:1.16 https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/7790fd619c7dc55f42b9dce82614631d4d2a1aef
Switch kind --loglevel (deprecated) to -v https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/3c6a28a4d5bf65fc17fcb75eac193fcaef9c3654
fix bug on kind-start on Makefile https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/10aac95b738876e2433b7c9f9d51abb101a546b7
update helm package: prometheus-operator to prometheus-community/kube-prometheus-stack https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/dba6e55253de209a67203d0ff039ae8433b914d3
update golang package dependencies  https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/2d859c12fbedebda3b89d8d324228b23447f978d
add kustomize as prerequistes for steps that needs kustomize in Makefile https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/90bd4b914796f92e2897df555e2b6dcda7a9641c
update helm version from 3.0.3 to 3.7.0 https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/6f9f32b0272ebd5b2685687c0dab12529c8f1fbc
remove docker.io from default image name https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/26c455b96f06badf24b82907567f19cc405782c0
fix to use kubectl of specific version (1.20.2) as default kubectl v1.16 doesn't work on k8s (v1.18+) cluster  apply https://github.com/yokawasa/k8s-cronjob-prescaler/pull/4/commits/99011b9b501e3a37d0ca46b832a2d694edc15698

it's confirmed that the following CI completed in the following k8s cluster version

v1.18.19
v1.19.11
v1.21.1
v1.22.1

https://github.com/yokawasa/k8s-cronjob-prescaler/actions/runs/1299514077

it's confirmed that the Weekly CI completed as well

https://github.com/yokawasa/k8s-cronjob-prescaler/actions/runs/1299640111


Argon2d4096 Mining Algorithm Change

Introduces Argon2d mining, replacing Skein via versionbits (bit 6) consensus.
Uses the same Argon2d parameters as Unitus and Argentum.
Activation period on testnet is from 1st Jan 2019, mainnet is 15th February.

Concept ACK, will test when able (nice to see you again).


fix for downloading policy-templete zip

removed grab package dependency for downloading policy-template zip file.
grab.Get gives error server returned 403 Forbidden
using net/http package instead.
for ref - https://accuknox.slack.com/archives/C0229TD83RA/p1694106383879899

@daemon1024 where should we store the zip?

@Prateeknandle I meant that
https://github.com/kubearmor/kubearmor-client/blob/11d6d3e1c30b15faf8a6790747540579e195c9ad/recommend/recommend.go#L108
in this line, let's not return error but just log a Warning and continue recommend with old templates


Feature Request: Cloudinary support

Hi there! As the title suggest, i don't know if this feature is already taken as part of road map, if not, it would be a nice feature to be able upload files to cloudinary service, providing the necessary config.
Postdata: Thanks a lot for this project. IT IS AMAZING!

At least for v1.0, there are no plans to add support for other storage APIs.
In the future there will be event hooks for before file upload to allow developers to intercept only this part of the model persistence and implement their own handling, but for now this a low priority.


RATIS-2015. Node.js 16 actions are deprecated

What changes were proposed in this pull request?
Upgrade GitHub actions to v4 to get Node.js 20.
https://issues.apache.org/jira/browse/RATIS-2015
How was this patch tested?
https://github.com/adoroszlai/ratis-thirdparty/actions/runs/7644612136

@szetszwo please review

Thanks @szetszwo for the review.


Segfault running test program

Trying to execute the vector add example from the githib may page results in segfault after compilation succeeds.
Installing into user space to avoid using sudo access, then build and run the test app:
$ cmake -DCMAKE_INSTALL_PREFIX=/data/nico/local/ /data/nico/code/hipSYCL/
$ make install
$ cd /tmp/foo
$ /data/nico/local/bin/syclcc-clang --hipsycl-gpu-arch=sm_52 test.cpp
$ LD_LIBRARY_PATH=/data/nico/local/lib ./test
Segmentation fault (core dumped)

LLVM/Clang environment variables are all set as suggested.
gdb backtrace says:
(gdb) run
Starting program: /tmp/foo/test
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[New Thread 0x7ffff663a700 (LWP 7740)]
[New Thread 0x7ffff4a74700 (LWP 7741)]
[New Thread 0x7ffff4273700 (LWP 7742)]
[New Thread 0x7ffff3a72700 (LWP 7743)]
[New Thread 0x7ffff3271700 (LWP 7744)]

Thread 2 "test" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffff663a700 (LWP 7740)]
0x00007ffff7b67b90 in __cudaPopCallConfiguration () from /usr/local/cuda-10.1/targets/x86_64-linux/lib/libcudart.so.10.1
(gdb) backtrace
#0  0x00007ffff7b67b90 in __cudaPopCallConfiguration () from /usr/local/cuda-10.1/targets/x86_64-linux/lib/libcudart.so.10.1
#1  0x0000000000403a69 in __hipsycl_pop_kernel_call() ()
#2  0x00000000004038bd in cl::sycl::handler::dispatch_kernel_without_offset<add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::vector_add, add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::{lambda(cl::sycl::id<1>)#1}, 1>(cl::sycl::range<1>, add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::{lambda(cl::sycl::id<1>)#1})::{lambda()#1}::operator()() const ()
#3  0x000000000040354d in std::_Function_handler<cl::sycl::detail::task_state (), cl::sycl::handler::dispatch_kernel_without_offset<add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::vector_add, add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::{lambda(cl::sycl::id<1>)#1}, 1>(cl::sycl::range<1>, add(cl::sycl::queue&, std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)::$_0::operator()(cl::sycl::handler&) const::{lambda(cl::sycl::id<1>)#1})::{lambda()#1}>::_M_invoke(std::_Any_data const&) ()
#4  0x00007ffff7946ab5 in cl::sycl::detail::task_graph_node::submit() () from /data/nico/local/lib/libhipSYCL_cuda.so
#5  0x00007ffff7947c8d in cl::sycl::detail::task_graph::process_graph() () from /data/nico/local/lib/libhipSYCL_cuda.so
#6  0x00007ffff79482a4 in std::_Function_handler<void (), cl::sycl::detail::task_graph::invoke_async_submission(std::function<void (std::vector<std::__exception_ptr::exception_ptr, std::allocator<std::__exception_ptr::exception_ptr> >)>)::$_0>::_M_invoke(std::_Any_data const&) ()
   from /data/nico/local/lib/libhipSYCL_cuda.so
#7  0x00007ffff794b006 in cl::sycl::detail::worker_thread::work() () from /data/nico/local/lib/libhipSYCL_cuda.so
#8  0x00007ffff761f1a0 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#9  0x00007ffff684a6ba in start_thread (arg=0x7ffff663a700) at pthread_create.c:333
#10 0x00007ffff6d6b41d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:109
(gdb) step
Single stepping until exit from function __cudaPopCallConfiguration,
which has no line number information.

[3]+  Stopped                 LD_LIBRARY_PATH=/data/nico/local/lib gdb ./test

For all I can tell, the issue occurs in this line (although that may not be what caused it):
cgh.parallel_for<class vector_add>(work_items,
                                   [=] (cl::sycl::id<1> tid) {

I'm experiencing the same behavior on both of the systems on which I installed hipSYCL. However, I am not entirely sure why it occurs or how I can prevent it from happening.

I know this one, it's a CUDA 10.1 issue. So far I have no idea what the root cause is, but given that it crashes in __cudaPopCallConfiguration() (an internal CUDA function related to kernel launches), it's likely that it's caused somewhere by the interaction between clang and CUDA. This issue is the reason why we recommend CUDA 10.0 over 10.1. Please try with CUDA 10.0.

Fantastic, thanks!

Okay, great to hear! I think I found something to explain the cuda 10.1 issues: Based on this, it seems that clang already emits a call to __cudaPopCallConfiguration() in the host-side kernel launch stub. In this case, we don't need to call it again (and calling it again would also explain your crash, since the pushed kernel arguments have already been popped).
Could you please check if everything behaves as expected if you remove this line: https://github.com/illuhad/hipSYCL/blob/2daf8407e49dd32ebd1c266e8e944e390d28b22a/include/CL/sycl/backend/clang.hpp#L64
(no need to recompile hipSYCL, just change your installed header files)

Can confirm this works fine!

Should be fixed on current master by @sbalint98's PR #164.


Support rest parameters & default values

"fn-args": "^2.0.0",
Two bugs in one issue:

Rest parameter. Just try this:

const fnArgs = require("fn-args");
console.log(fnArgs((...arg1) => {})); // => []
console.log(fnArgs((arg1, ...arg2) => {})); // => [ 'arg1', '...arg2' ]
This is wrong as minimum because of ambiguity of rest parameter presence.

The reason I referred to Your package (my small internal argument getter works to this moment) is to skip default values:

console.log(fnArgs((arg1 = 2) => {})); // => []
console.log(fnArgs((arg1, arg2 = 2) => {})); // => [ 'arg1', 'arg2 = 2' ]
console.log(fnArgs((arg1 = 2, arg2) => {})); // => []
Of course, this is not what we expect, really?
Yeah, this is hell - to get rid of default values. But if You want to (I hope) remove this bug, try not to parse all 'default value code', but follow the number of open/close (, ), {, } and skip to the comma or parenthesis when no parenthesis depth occurs. Nevertheless, the code will be complicated dramatically.

Yeah, those didn't really exist when I first made this module. Pull request welcome :)

Sent a pull request. Sorry for revert, I saw it only now - just a folly, I think it is not critical.


DEVOPSGUYS-2148 - Create a check for MySQL/Percona Versions to patch …

…against CVE-2016-6662
Simple bash script.
Tested on Dataloop.io

Looks good to me!


Possibility to assign keyboard shortcuts for snippets.

It doesn't seem possible to assign my snippets to the keyboard combination I'm used to from another editor. The transition to VSC would be so much easier if I could make the keyboard work the same.

@anders16000 Do you mean to have a keyboard shortcut => insert single snippet sort of thing or a keyboard shortcut to show snippets (like Ctrl-Space but only snippets)?

Closing since no more info received.


How to change the style

Is it possible to replace or add style classes in the altEditor forms? Is it possible to define a template like the original Editor from datatables?
Andreas

Currently, there is a "style" option that allows to put your own style on
elements, but there is no
"class" attribute. It should be implemented.
Il giorno ven 23 ott 2020 alle ore 12:15 Andreas<EMAIL_ADDRESS>ha scritto:

Is it possible to replace or add style classes in the altEditor forms? Is
it possible to define a template like the original Editor from datatables?
Andreas
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KasperOlesen/DataTable-AltEditor/issues/154, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLII2HCQSTRVRWPJEORVUTSMFJUZANCNFSM4S4LPL7A
.


Allow current timestamp with default iat validator

The iat claim was being validated by comparing it with the current timestamp
without allowing equality, which made validation fail when it was performed
within a second of signing the token.

Nice catch! We need a better way to test these things. I will try to look into it.
@bryanjos Perhaps it would be better if we release a patch with this?

I think you should also include a more specific test for the issue at hand like (the one you added was missing too but it may not test what you want if there is some slowness in between the calls):
	test "ensure iat validation passes for same second" do

		now = current_time()

		assert {:ok, _} = @payload
		|> token
		|> with_iat(now)
		|> with_validation("iat", &(&1 <= now))
		|> sign(hs256("secret"))
		|> verify!
	end

Looks good! I agree with Victor on the addition of the test though.
@cs-victor-nascimento yes we can release a new version with this and the changes from #115. I can put together a release and publish it.

Thanks for the input! I’ll add that test to the changes.

Thanks @lnikkila!


Autorest Regen Preview to 3.0.0-alpha.20230428.2 by Chun Yu from refs/pull/3334/head

Triggered from https://github.com/Azure/autorest.csharp/pull/3334

API change check
API changes are not detected in this pull request.


resultToAsync works with @0.10.2, but not latest version

Describe the bug
I am following the youtube lesson, https://youtu.be/O1Gu5b7rxbw?t=2885, resultToAsync works for<EMAIL_ADDRESS>version, but not 0.11.0 & 0.11.1 versions.
To Reproduce
Steps to reproduce the behavior:
import { readFile } from "fs";
import { Async, tryCatch, resultToAsync } from "crocks";

const e = e => console.error("rej", e);
const s = s => console.log("res", s);

const parse = tryCatch(JSON.parse);
const readFileAsync = Async.fromNode(readFile);
readFileAsync("config.json", "utf8")
  .chain(resultToAsync(parse))
  .fork(e, s);

Error is throw new TypeError('resultToAsync: Result returning function required

hmmm. what version of node are you using?
I am able to get this to run as expected with 0.11.1.

my node version is 10.14.2

@zhentian-wan we just released a new version. Please give it a shot and see if this does anything for you.
I'm going to close out this issue for now, if you continue to have problems, please reopen or create a new issue. Hope this helps

Hi, Congrats for new release!
I have installed and test out, looks like the problem is still here.
I found out that, if file doesn't exists, then it actually works:
rej { [Error: ENOENT: no such file or directory, open 'config.json']
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: 'config.json' }

If the file exists, then it crashes with error: TypeError: resultToAsync: Result returning function required
I can make it works when file exists by doing one extra function invoke:
const { readFile } = require("fs");
const { Async, tryCatch, resultToAsync } = require("crocks");

const e = e => console.error("rej", e);
const s = s => console.log("res", s);

const parse = tryCatch(JSON.parse);
const readFileAsync = Async.fromNode(readFile);
readFileAsync("exists.json", "utf8")
  .map(parse)
  .map(fn => fn())
  .chain(resultToAsync)
  .fork(e, s);

Hope this information helps.

Yeah so i've seen a similiar issue in exactly this, It is caused because JSON.parse is not a unary function. The output of any call to tryCatch regardless of the input is curried. This is why you need to do the extra call with JSON.parse
import unary as well from crocks and wrap parse with it and that will drop the need for the extra map
const parse = tryCatch(unary(JSON.parse))

Yeah so i've seen a similiar issue in exactly this, It is caused because JSON.parse is not a unary function. The output of any call to tryCatch regardless of the input is curried. This is why you need to do the extra call with JSON.parse
import unary as well from crocks and wrap parse with it and that will drop the need for the extra map
const parse = tryCatch(unary(JSON.parse))

Cool! Thanks for the infromation 👍

Glad i could help 😄


I want to know whether jetcd supports https ?

same with title

cc/ @lburgazzoli

See #55

@riclava it does support now via https://github.com/coreos/jetcd/pull/161.

@fanminshi thanks a lot ;-)

I still cannot activate TLS with jetcd.
Following the instructions  here, I had to add the following to my gradle dependencies:
compile group: 'io.grpc', name: 'grpc-all', version: '1.15.1'
runtime 'io.netty:netty-tcnative-boringssl-static:2.0.12.Final

Which results to java.lang.NoSuchMethodError: io.netty.internal.tcnative.SSLContext.setCipherSuite(JLjava/lang/String;Z)Z on linux.
After spending tones of time, I got this working by replacing the gradle demendencies mentioned above with
compile group: 'io.netty', name: 'netty-handler', version: '4.1.27.Final'
compile group: 'io.grpc', name: 'grpc-all', version: '1.15.1'
compile group: 'io.netty', name: 'netty-tcnative-boringssl-static', version: '2.0.12.Final'
compile group: 'com.coreos', name: 'jetcd-core', version: '0.0.2'

This works fine locally, but breaks in my bamboo server while running the tests.
The message is:
Could not find TLS ALPN provider; no working netty-tcnative, Conscrypt, or Jetty NPN/ALPN available
I worked around this by introducing an ugly if statement that wrapped the whole thing and that is not executed in tests.
The current way of dealing with TLS seems very confusing to me...
Any help or advice is most welcome.

looks like you are using incompatible versions, can you test against the latest available jetcd release ?

Sure! I will do it right now.

Ok, I think I already use the latest jetcd.
Here is my dependency:
io.etcd:jetcd-core:0.4.1

EDIT:
I upgraded the io.grpc and followed the table here to find the corresponding netty:netty-tcnative-boringssl-static version.
I ended up with the following dependencies:
    compile 'io.etcd:jetcd-core:0.4.1'
    compile group: 'io.grpc', name: 'grpc-all', version: '1.27.1'
    compile group: 'io.netty', name: 'netty-tcnative-boringssl-static', version: '2.0.26.Final'

Locally it works fine. Unit tests too.
In build server I get:
Factory method 'etcdClient' threw exception; nested exception is java.lang.IllegalStateException: Could not find TLS ALPN provider; no working netty-tcnative, Conscrypt, or Jetty NPN/ALPN available
This means that I will probably have to re-introduce the hacky if-statement that I mentioned above.

what is your build server ? maybe it is not one of the supported os

There are multiple build servers. Some of them run oracle linux and some redhat.
Yes, maybe it is not supported.
The weird thing is that we use https (for example spring's RestTemplate) without any problems nor any need for additional dependencies. How do those work?
So, any suggestion is very welcome. Giving up etcd is not an option...

It's really hard to provide an answe, we do run on test on travis and/or github actions that are bot running on ubuntu but it should work on any other system.

Ok thanks a lot for your time. It looks like I will not get https working... :(
I am not sure how others deal with https (for example, RestTemplate), but they do it great. And we, as users, do not have to isntall extra dependencies, or deal with system compatibility issues like mine.
It would be awesome if you also find a similar way of doing it.
In any case, thanks a lot for your time.


proxy: Implement prototype

First prototype for an IMAP proxy.
It can be run with:
cd proxy
RUST_LOG=proxy=trace cargo run

There is a default config file located at proxy/config.toml. It contains ports that can be used to connect with a local email client.
The change set contains some TODOs. We need to create issues for them after merging this PR.
Closes #13

The build fails because toml_edit quires Rust 1.66 but we enforce 1.65 in the CI. See https://github.com/duesee/imap-flow/blob/main/.github/workflows/build_and_test.yml#L103.
I don't have a strong opinion how to resolve this. We could just increase the MSRV? If you would like to do it here, you could also change imap-codec -> imap-flow.

If you would like to do it here, you could also change imap-codec -> imap-flow.

Done

Thanks! I added a commit to also install libssl-dev in the minimal-versions job. Not sure if it does the trick.

Actually, we don't need native-tls (and openssl) at all. Reverted and removed the dependency.


Paginate /bounties/pending

On hive, when the arbiter runs after a large amount of bounties are posted, the pending route starts hanging indefinitely. Polyswarmd hangs, arbiter crashes.

Rate limiting tracked in another ticket. This ticket is now about pagination.

@mjkoo has this been addressed? I thought portal was using this feature


Idea: Port material-ui styles

I'm thinking about porting the ink bar styles from http://www.material-ui.com/#/components/tabs to this component. What do you think?
It might be something that we can enable via a flag.

Thanks for the contribution! 🎉  Awesome work 👏 👍
Merged


Fix description of the "Support pages and views" checkbox of the Razor class library template

Description
I believe this sentence:
Select Support pages and views if you need to support views. By default, only Razor Pages are supported. Select Create.
should instead be:
Select Support pages and views if you need the library to contain pages and/or views. By default, only Razor Components are supported. Select Create.
Alternatively, remove it altogether.
It seems nitpicky but it sent me down quite a rabbit hole today. I took it at face value and didn't understand why after not selecting this checkbox and adding a page to the new project, it wouldn't compile. Only after spending an hour troubleshooting and manually "fixing" my project to compile did I read the next sentence in the doc, the one that is actually correct:
The Razor class library (RCL) template defaults to Razor component development by default. The Support pages and views option supports pages and views.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/razor-pages/ui-class?view=aspnetcore-8.0&tabs=visual-studio
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/razor-pages/ui-class.md
Document ID
fd116d00-0dc7-efaf-c714-34d77b345a05
Article author
@Rick-Anderson

Associated WorkItem - 232968

Thanks for reporting this. Any chance you could PR this? At the top right of the article, select the Edit pen icon:

Select the pen icon again.  Edit, then Save.

Here's the PR: https://github.com/dotnet/AspNetCore.Docs/pull/32231


[CI/Build]: Add Bandit security check to workflow

Currently, the project does not have a security linter integrated into its CI pipeline. This poses potential risks as security vulnerabilities in Python code can go undetected. I found many high security issues using Bandit

Integrate Bandit to scan for security issues in the codebase.
Configure Bandit to fail the workflow if any high-severity issues are found.

e.g.:
$ bandit -r . --severity-level high
[main]  INFO    profile include tests: None
[main]  INFO    profile exclude tests: None
[main]  INFO    cli include tests: None
[main]  INFO    cli exclude tests: None
[main]  INFO    running on Python 3.10.12
Working... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:12
Run started:2024-09-11 04:37:49.395569

Test results:
>> Issue: [B602:subprocess_popen_with_shell_equals_true] subprocess call with shell=True identified, security issue.
   Severity: High   Confidence: High
   CWE: CWE-78 (https://cwe.mitre.org/data/definitions/78.html)
   More Info: https://bandit.readthedocs.io/en/1.7.9/plugins/b602_subprocess_popen_with_shell_equals_true.html
   Location: ./collect_env.py:95:8
94                               stderr=subprocess.PIPE,
95                               shell=shell)
96          raw_output, raw_err = p.communicate()
97          rc = p.returncode
98          if get_platform() == 'win32':
99              enc = 'oem'

--------------------------------------------------

Code scanned:
        Total lines of code: 135058
        Total lines skipped (#nosec): 0

Run metrics:
        Total issues (by severity):
                Undefined: 0
                Low: 3887
                Medium: 33
                High: 1
        Total issues (by confidence):
                Undefined: 0
                Low: 20
                Medium: 28
                High: 3873
Files skipped (0):

rebase to main only to see if the failures in CI can be fixed.

kindly ping @simon-mo


[UserPage] Change order of Published/Private Brews

As mentioned in this Reddit comment, some users would like to be able to re-order the groups on the User Page, as they do not edit Published brews, which are listed at the top of the page. Having the ability to push this group to the bottom of the page would improve their user experience.

Or perhaps provide a widget to collapse the section?

Current state of the PR:

[x] Mousing over the title changes the pointer to a hand.
[x] Clicking the header toggles the visibility state.
[x] The state indicator to the left of the header changes to indicate when items are hidden.
[x] Visibility selections are stored in/retrieved from local storage

I haven't drafted the PR yet as there's still a lingering error in the console, but the current work is in my fork: https://github.com/G-Ambatte/homebrewery/tree/addUserPageCollapse-%231797

I think this is in a good place now, I'll raise a PR for review.

Looks good.


[Feat]:get category name

Description
This feature implements a new endpoint to retrieve all category names. It allows authenticated users to fetch a list of available product categories.
How Has This Been Tested?

Unit tests have been added to cover the new endpoint functionality.
Manual testing was performed using Postman to ensure the endpoint returns the expected response.
Integration tests were conducted to verify the interaction between the authentication middleware and the category retrieval logic.

Screenshots
With pagination

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

API Documentation
Get Category Names
Retrieves a list of all available product categories.
URL: /api/v1/categories
Method: GET
Auth required: Yes (Bearer Token)
Success Response
Code: 200 OK
Content example
{
    "status": "success",
    "status_code": 200,
    "message": "Categories fetched successfully",
    "data": [
        {
            "Name": "Fashion"
        },
        {
            "Name": "Groceries"
        },
        {
            "Name": "Appliances"
        }
    ]
}

Error Responses
Condition: If the user is not authenticated.
Code: 401 UNAUTHORIZED
Content:
{
    "status": "error",
    "status_code": 401,
    "message": "Unauthorized access"
}

Notes

This endpoint requires a valid JWT token to be included in the Authorization header.
The response includes an array of category names under the data key.
Each category is represented by an object with a Name field.

please add pagination to this, also include postman and test image

added pagination


ARM images for Sonobuoy

User @fish in #sonobuoy had requested ARM images at some point for Sonobuoy so that you could run on Raspberry Pis. Not unreasonable since k8s supports it and we want to support k8s :)

@johnSchnake What kind of images does Sonobuoy supports currently ?
I would like to collaborate for adding this enhancement, please inform if I can help in anyway.

We currently build for Windows and Linux, including linux/arm64; I'm assuming the requesting user needed armv6 or v7?
(Note that we build the binary for many more os/arch, that is trivial with Go.)
The biggest hurdle for us historically on ARM has been that we dont readily have arm devices on which to test and I'm not sure about virtualization for arm.
At some point we tried to remove arm support entirely but some people in the community did say they use it and we just took a hands off approach and said we'd publish it but relied on the community to report issues on that arch since we weren't testing it ourselves.

You can look at the os/arch supported here https://hub.docker.com/layers/sonobuoy/sonobuoy/main/images/sha256-e49fd131e727455e548efbcb0e4e57140a6751f1e46017d82fa0d3dce6bd2a61?context=explore

https://kubernetes.io/releases/download/ shows the k8s build versions which includes an arm and arm64.
GOOS supports an arm option which is a 32bit and that aligns with what the original post was regarding raspberry pi (which also is just 32bit).
We can try adding that but we would just be depending on the community for feedback.
For the time being I'm not going to pick this up but wait someone with some arm systems to reengage. I can assist but then we can get confirmation of functionality at that time.


[hotfix][test]Fix test problems caused by type enhancement

Purpose of this pull request

Fix test problems caused by type enhancement

Which issue you fix
Fixes # (issue).
Checklist:

[ ] I have executed the 'mvn spotless:apply' command to format my code.
[ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

LGTM


Add basic examples or links to basic examples to README.md

The docs button links directly to basic examples. What do you think would be a good way to make that more obvious?

The reason I don't have examples directly in the README is that it was easy for them to get out of date or out of sync. The ones in the docs can be checked by Documenter.jl

Oops, sorry, I always miss that little documentation icon.


Point release for Extension with data science changes

Codecov Report

Merging #8153 into release will increase coverage by 0.41%.
The diff coverage is 60.63%.

@@             Coverage Diff             @@
##           release    #8153      +/-   ##
===========================================
+ Coverage    58.79%   59.21%   +0.41%
===========================================
  Files          496      498       +2
  Lines        22134    22462     +328
  Branches      3562     3624      +62
===========================================
+ Hits         13014    13301     +287
- Misses        8308     8322      +14
- Partials       812      839      +27

Impacted Files
Coverage Δ

src/client/common/application/types.ts
100% <ø> (ø)
:arrow_up:

src/client/common/types.ts
100% <ø> (ø)
:arrow_up:

src/client/datascience/cellFactory.ts
89.65% <ø> (ø)
:arrow_up:

src/client/datascience/webViewHost.ts
11.11% <ø> (ø)
:arrow_up:

src/client/datascience/types.ts
100% <ø> (ø)
:arrow_up:

.../datascience/interactive-common/interactiveBase.ts
6.22% <0%> (-0.04%)
:arrow_down:

...ractive-window/interactiveWindowCommandListener.ts
60.3% <0%> (ø)
:arrow_up:

...atascience/interactive-window/interactiveWindow.ts
23.91% <0%> (-0.81%)
:arrow_down:

...ient/datascience/editor-integration/codewatcher.ts
57.21% <10%> (-0.84%)
:arrow_down:

src/client/common/utils/localize.ts
93.58% <100%> (-0.37%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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


[Feature] Structure CODEOWNERS by crate

Component
meta-crate
Describe the feature you would like
Structure ./github/CODEOWNERS by crate like in reth
https://github.com/paradigmxyz/reth/blob/main/.github/CODEOWNERS
Additional context
No response

imo this doesn't make sense here because we don't have that many reviewers


iverine-bangladesh

![riverine-bangladesh]

![riverine-bangladesh](https://cloud.githubusercontent.com/assets/19351625/15259687/9762714c-1974-11e6-91f9-eb89ce644fe8.jpg

Don't use public projects to learn how to use GitHub. Please Close This Issue.

Don't use public projects to learn how to use GitHub. Please Close This Issue And Keep It Closed!!!


Make restart in NodeStateMachine / NodeTransition optional

Follow up #71418.
Recognizing that many users are spamming travel() requests, this feature will be made optional.

Renamed to allow_transition_to_self

Just for the sake of performance, I changed it to skip the A=A process.

Okay, it's a bit easier to read now. thanks

Thanks!


Dialogic dialog fills whole screen with small resolution

The problem
Describe the bug
When playing the dialog, it covers the entire screen because of the low project resolution
To Reproduce
Steps to reproduce the behavior:

Create a dialogic
Change project resolution to a low setting for pixel art
play the scene

Expected behavior
The dialog pops up without filling the entire screen
Screenshots
Without Dialogic:

With Dialogic:

Project resolution:

System (please complete the following information):

OS: Windows
Godot Version: 3.3.2
Dialogic Version: 1.3

Any idea to fix the issue or a link to the line of code that might be the cause for this problem

You have to create a theme and set a smaller font size and dialog size


Search commits cosmetic

Gitea version (or commit ref): 1.1.0
Git version: N/A
Operating system: N/A
Database (use [x]): N/A

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL) https://try.gitea.io/Fastidious/Test-15/commits/master
[ ] No
[ ] Not relevant

Log gist:

Description
The ticked box to search All is too close to its label. I think the tick box and label should be moved to before the search field instead, and the Find button replaced with Search, to make it consistent with the rest of search boxes used.
Screenshot:

I have submitted PR #1254


feat: change link color to white

Implements # 1529 White text on sucess toast links

Nice work. One thing I noticed (unrelated to this ticket) though is the tooltip cuts off at the bottom of the screen. Maybe we should move the success toast up a bit?

Good point, yeah we can move it up some pixels. Will just use the large spacing instead


Platform independent plugin support

Add support for plugins that are platform independent on the native ports (possibly WebAssembly?)

These probably need to be wrapped (and involve a memory copy in the process):

Bitmap.buffer (pixel array) (r)
Bitmap.bufferForWrite (pixel array) (rw)
BitmapLayerTreeOwner.layerTreeOwnerInterface (VFT structure) (r)
Layer.mainImageBuffer (pixel array) (r)
Layer.mainImageBufferForWrite (pixel array) (rw)
Layer.provinceImageBuffer (pixel array) (r)
Layer.provinceImageBufferForWrite (pixel array) (rw)
Offscreen.nativeHandle
Texture.nativeHandle
VertexBuffer.lock() (float array) (rw)
VertexBuffer.nativeHandle
WaveSoundBuffer.getVisBuffer() (s16 array) (rw)
PhaseVocoder.interface (VFT structure) (r) (inserted into WaveSoundBuffer.filters)
Window.registerMessageReceiver() (callback) (r)
Window.HWND (window handle)
BasicDrawDevice.interface (VFT structure) (r)

Once support for multiple memories or memory control is implemented, it should be possible to handle the array types more efficiently without extra copies.  In the case of multiple memories, some code modification is needed (e.g. accessing the shared segment instead of the module's segment)
For message receivers, probably need to be multi thread safe.
For HWNDs and GLES native handles, it's probably not going to be very useful without corresponding Win32 and GLES APIs exposed to the modules.


layers UI could use a scrollbar (bleeding edge branch)

Is your feature request related to a problem? Please describe.
the layers UI window needs a scrollbar because too many layers overflows it
Describe the solution you'd like
a scrollbar in the layers UI window
Describe alternatives you've considered
fewer layers? but telling people to not use the feature seems like a bad plan
Additional context

Should be done on 87239d0

i'm genuinely ashamed it was literally just that
i need to add CSS to #terrible

Well, CSS is indeed a strange beast


Abstracted FFI to support multiple param-based VerkleTrie with same interface

The changes are passing the unit tests of the wrapper library. Need to check from the client side too!
Only change is in the API is verkle_trie_new(byte, byte) from verkle_trie_new().

If this works, then I will try to abstract the model code into a trait to avoid all the repetition in the codebase

I believe all the parts have been implemented now. Waiting for the https://github.com/crate-crypto/rust-verkle/pull/47 to get merged before I can include the unit tests of PrecomputeLagrange commitment based VerkleTrie(s).


ui: controlnet in linear ui

Add ControlNet to the Linear UI (Text to Image & Image to Image tabs).

add a new collapsible component to the parameters panel for ControlNet
any number of ControlNets may be added
control images are automatically processed (if not marked as preprocessed)
a mini UI displays the control image as a thumbnail, weight and begin/end step %
toggle to advanced UI to see the control image much larger and access the processor settings

Other Changes
Enhanced drag and drop

add new library dnd-kit for drag and drop, much more capable than the previous implementation with native HTML Drag and Drop API
images and be dragged from anywhere to anywhere else (except you cannot drop onto gallery)
vastly simplifies internal logic for handling dragging images around
IAIDndImage component encapsulates logic for draggable and droppable images
should fix issues raised by @GreggHelt2 and @mickr777 regarding drag and drop

Misc

add alpha colours to theme eg accentAlpha.200, okAlpha.700
handful of minor bug fixes

TODO:

support arbitrary models (currently just the basic ones are hardcoded)
auto-select processor based on model

I can verify that this PR fixes the problem with drag and drop of images in Chrome on Ubuntu. Huzzah!

I can verify that this PR fixes the problem with drag and drop of images in Chrome on Ubuntu. Huzzah!

And on chrome windows


username claim is not accessible by custom realm

I wrote a stub realm as you suggested, but the realm always returns with username as null. I have used usernameClaim="email". So the email should be available to the realm.
This isnt happening:

Once the authenticator exchanges the authoization code for the ID Token, it extracts a field from the token (a claim) used as the username in the Tomcat's Realm and looks up the user.

This is my realm code
`
package com.tfsc.mirror.realm;
import org.apache.catalina.realm.GenericPrincipal;
import org.apache.catalina.realm.RealmBase;
import java.security.Principal;
import java.util.ArrayList;
import java.util.List;
import org.apache.juli.logging.Log;
import org.apache.juli.logging.LogFactory;
public class MirrorRealm extends RealmBase {
private final Log log = LogFactory.getLog(this.getClass());

private String username;
private String password;

@Override
protected String getName() {
    return this.username;
}

@Override
protected String getPassword(String s) {
    return this.password;
}

@Override
public Principal authenticate(String username, String credentials) {
    this.username = username;
    this.password = credentials;
    /* authentication just check the username and password is same*/
    log.info("Authentication is taking place with userid: "+username);
    return getPrincipal(username);

}

@Override
protected Principal getPrincipal(String string) {
    List<String> roles = new ArrayList<String>();
    roles.add("user");  // Adding role "user" role to the user
    log.info("username is "+ this.username);
    log.info("Realm: "+this);
    Principal principal = new GenericPrincipal(this.username, this.password,roles);
    //Principal principal = new GenericPrincipal("test", "test",roles);
    log.info("Principal: "+principal);
    return principal;
}

}
`
My configuration is:
<Valve className="org.bsworks.catalina.authenticator.oidc.tomcat85.OpenIDConnectAuthenticator"
providers="[
{
name: 'OKTA',
issuer: https://somedomain.com/oauth2/something,
configurationDocumentUrl: https://somedomain.com/oauth2/something/.well-known/oauth-authorization-server,
clientId: ,
clientSecret:
}
]"
    usernameClaim="email" additionalScopes="email" noForm="true"/>

The mistake was on my side. I didnt override the right method on the RealmBase class.
public Principal authenticate(String username) - Overriding this method fixed the issue. I am closing this issue.


Setting "git.mergeEditor": false to disable new merge editor not working

Does this issue occur when all extensions are disabled?: Yes

Version: 1.73.1 (user setup)
Date: 2022-11-09T04:27:29.066Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: No

I'm trying to revert to old inline merge editor, I set "git.mergeEditor": false in user settings.json and disabled it in settings menu also. None of them worked. Tried editor.codeLens false and true. No luck

What you see there is the (old) inline merge editor.
The button stays regardless of the setting.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!

What you see there is the (old) inline merge editor.
The button stays regardless of the setting.

Well, I can't see these options, it just shows has not the buttons to work with

Oh I see. Can you try with default settings? (And/or in VS Code insiders?)

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


Vidéo split in two

Hey, after running the code, I have 2 screens of my vidéo, one in colour on the left, and one with no colour on the right, between them there is black screen. Is it normal ?

Not as normal as expected. Give me a screenshot.
Anyway, what result have you got after that? Is there a directory <current date>_distance with a CSV file inside?


Add knowledge form context selection

Adds routes for retrieving files from the local knowledge git
Adds the ability to view the file changes/additions in the native dashboard.
Enables the user to view the knowledge document, highlight the text they want to add to the context field and submit it. The highlighted text will get populated to the context field.
Allows the user to still manually enter context or knowledge file details and handles the state switching when they choose to do so.
Prevents the user from selecting context from a different commit SHA if they have already selected context from another SHA.
Lets the user toggle all files in the knowledge files repo if they wanted to use a different file/SHA for context if they want to. Otherwise, the default is the latest commit. Probably the ideal scenario would be to return the SHA from the /api/native/git/upload or whatever the route is to return the SHA that could then be referenced and present the user with an empty state if it isn't defined but that would be a lot more code to an already too busy PR.

Note: this probably breaks the container deploy for native with the changes in src/app/api/native/upload/route.ts. It's not clear to me how to rebase into that if someone could take a look. I think @vishnoianil  did that if you don't mind good sir.
Not an easy review, there are a ton of different states getting munged together here UploadYaml/AutoFillYaml/DownloadYaml/ViewYaml/ManualKnowledgeFile to name a few. Did my best to juggle all of them but will def have some bugs to fix.
Demo vid:
https://github.com/user-attachments/assets/02768290-65c9-453f-88f5-756254db546c

@nerdalert I will get to this sometime tomorrow.

Resolved the rebase issue I had by changing ENV values. Should be good to go meow.


MineDojo Crashing

Hey!
After running and it working once, now it crashes.
https://hastebin.com/cukizamuxe.sql

Hey @RocketNinja15 , thanks for reporting this! Have you double-checked that the Java version is the same as the one you used to successfully run?

Closed since the issue seems resolved. Feel free to reopen if it sill exists.


MAKES sub-graph tutorial feedback update 2

Algin params between MagDataToMakesJson.usql script and how-to-create-index-from-mag.md tutorial
Add scape for URLs in bold
Update MakesIndexResourceConfigFilePath mispell in tutorial and reference
Update trouble shooting guide to include azure quota issue and batch service not being registered issue.

Docs Build status updates of commit 614cea6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/how-to-create-index-from-mag.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/MagDataToMakesJson.usql
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-create-api-instances.md

Line 83: [Warning] Invalid file link: 'reference-makes-command-line-tool'.

academic-services/knowledge-exploration-service/how-to-create-index-from-mag.md

Line 99: [Warning] Cannot find bookmark '#Azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-Azure-account' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-azure-account'?
Line 102: [Warning] Cannot find bookmark '#Azure-Batch-service-not-registered' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-batch-service-not-registered'?
Line 125: [Warning] Cannot find bookmark '#Azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-Azure-account' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-azure-account'?
Line 135: [Warning] Cannot find bookmark '#Azure-Subscription-quota-limit-reached' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-subscription-quota-limit-reached'?

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md

Line 198: [Warning] Link 'https://docs.microsoft.com/en-us/rest/api/compute/virtualmachines/listavailablesizes' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 620fd7e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/how-to-create-index-from-mag.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/MagDataToMakesJson.usql
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-create-api-instances.md

Line 83: [Warning] Invalid file link: 'reference-makes-command-line-tool'.

academic-services/knowledge-exploration-service/how-to-create-index-from-mag.md

Line 99: [Warning] Cannot find bookmark '#Azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-Azure-account' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-azure-account'?
Line 102: [Warning] Cannot find bookmark '#Azure-Batch-service-not-registered' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-batch-service-not-registered'?
Line 125: [Warning] Cannot find bookmark '#Azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-Azure-account' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-login-failure-due-to-multiple-subscriptions-or-multiple-tenants-being-tied-to-a-single-azure-account'?
Line 135: [Warning] Cannot find bookmark '#Azure-Subscription-quota-limit-reached' in 'knowledge-exploration-service/resources-troubleshoot-guide.md', did you mean '#azure-subscription-quota-limit-reached'?

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md

Line 198: [Warning] Link 'https://docs.microsoft.com/en-us/rest/api/compute/virtualmachines/listavailablesizes' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


add barcode feature to webcam form

lookup EAN barcodes see e.g. https://opendata.stackexchange.com/questions/562/is-there-a-global-database-of-all-products-with-ean-13-barcodes
simply use amazon's main search for lookup

see https://github.com/zauberzeug/nicegui/discussions/2028


[Slack] Add ability to copy links

Extension
https://www.raycast.com/mommertf/slack
Description
Please allow copying the links to messages, channels, people.
Who will benefit from this feature?
Everyone who needs to put slack links to task trackers or documentation.
Anything else?
No response

Thanks for the feature request. Would you like to work on it? If yes, here's our guide on extension contributions so that you can open up a pull request. Feel free to contact me here or in the Slack community if you need further help.


CVEs in Netty dependency

There are multiple moderate security issues in the netty version (4.1.59.Final) brought in by the Redisson library. See https://github.com/netty/netty/security/advisories where 4.1.77.Final is the most recent version with fixed CVEs (4.1.82.Final is absolute latest right now).
This may not generally be an issue when communicating with trusted Redis servers, but brought into an application that also uses netty with other sources, it could be harder to use a safe version of netty.
Expected behavior
n/a
Actual behavior
n/a
Steps to reproduce or test case
mvn dependency:tree | grep netty
Any with 4.1.59.Final
Redis version
n/a
Redisson version
3.17.6
Redisson configuration
n/a

Redisson 3.17.6 uses netty 4.1.79


Why does sending messages with identical txIDs succeed?

https://github.com/hashgraph/hedera-hcs-sxc/blob/a5c22e557afc9d09ae3e262153666c10ac541fb3/hcs-lib/src/main/java/com/hedera/hcslib/consensus/OutboundHCSMessage.java#L124

@zoperado still an issue ?

Will test with sufficiently large message to make chunking kick in.


Fatal error while installing via NPM package

What is wrong?
Installation fails, check log file
Where does it happen?
running npm i brain.js
Expected behavior (i.e. solution)
install sucessfully
Version information
Nodejs:
v18.18.1.
 System Linux 6.5.0-14-generic
791 error gyp ERR! command "/home/martincito/.nvm/versions/node/v18.18.1/bin/node" "/media/martincito/freedom/repos/test/node_modules/.bin/node-gyp" "rebuild"
791 error gyp ERR! cwd /media/martincito/freedom/repos/test/node_modules/gl
791 error gyp ERR! node -v v18.18.1
791 error gyp ERR! node-gyp -v v9.4.1
791 error gyp ERR! not ok
792 verbose exit 1
793 timing npm Completed in 2194ms
794 verbose unfinished npm timer reify<PHONE_NUMBER>769
795 verbose unfinished npm timer reify:build<PHONE_NUMBER>917
796 verbose unfinished npm timer build<PHONE_NUMBER>917
797 verbose unfinished npm timer build:deps<PHONE_NUMBER>917
798 verbose unfinished npm timer build:run:install<PHONE_NUMBER>923
799 verbose unfinished npm timer build:run:install:node_modules/gl<PHONE_NUMBER>923
800 verbose code 1
801 error A complete log of this run can be found in: /home/martincito/.npm/_logs/2024-01-17T22_35_20_658Z-debug-0.log

it looks like your missing dependencies
apt-get update
apt-get install python3 make g++
apt-get install -y build-essential
apt-get install -y libglew-dev
apt-get install -y libglu1-mesa-dev
apt-get install -y libxi-dev
apt-get install -y  pkg-config

apt-get update
apt-get install python3 make g++
apt-get install -y build-essential
apt-get install -y libglew-dev
apt-get install -y libglu1-mesa-dev
apt-get install -y libxi-dev
apt-get install -y pkg-config

Run the above command that they sent and it should install just fine after that :blush:
These lines (see below) in particular mention that you're missing xi, which would be the libxi-dev that @apophis51 mentioned.
791 error to the PKG_CONFIG_PATH environment variable
791 error No package 'xi' found
791 error gyp: Call to 'pkg-config --libs-only-L --libs-only-other x11 xi xext' returned exit status 1 while in angle/src/angle.gyp. while loading dependencies of binding.gyp while trying to load binding.gyp


fix: increase font size for text field on commit page

Replaces #24
On [username].vue the input font size is 14px. On index.vue it is 16px.
I removed a text-sm declaration on the surrounding form-tag to set the font-size back to 16px/1rem.
edit: apologize for the formatting additions, I've disabled Prettier and enabled Eslint, so not sure why that happened.

thank you ❤


Is it possible to set default quality?

Default is "auto" quality and if we have a small player it does not make sense to force big files.
Is it possible to force 360p for example on load ?

Hello @morkov4o
Yes, that's possible. Set startLevel hls.js config param(https://github.com/video-dev/hls.js/blob/master/doc/API.md#startlevel) to a value that suits you, and pass this param in hlsjsConfig as described here https://github.com/streamroot/videojs5-hlsjs-source-handler#passing-configuration-options-to-hlsjs

Hi @ruslandinov,
thanks for your quick response!


Support different color for border and tint

I think it would be great to have different colours for the dot border and the dot fill color.

Yeah that would be awesome :)

+1


convertEasyEdaJsonToCircuitJson: improve how portHints are determined

Attempting to close https://github.com/tscircuit/easyeda-converter/issues/87

You can paste the output into snippets if it helps test!

superceded by https://github.com/tscircuit/easyeda-converter/pull/89


419 page expred after login to admin panel

Version information

Laravel: v5.8.34
Voyager: v1.2.6
PHP: 7.2

Description:
After login through the voyager login form, I get a 419 error.
Additional context
It seems to work for everyone except for me and my users. I created several of them. Initially, they work, but after logout and logging in again I get the error. I also tried to delete all browser-data and using the incognito-mode of chrome.
Thank you!
I'm really stuck with this one so every help is appreciated!

419 is indicative of a CSRF issue, either directly by configuration (i.e. missing csrf field or something on the form), or indirectly as an issue with the underlying storage mechanism (i.e. filesystem permissions issue causing session issues).  Since it's a Voyager page (I'm assuming you haven't modified it), and it works for others, I'm fairly confident saying it's not a missing csrf field on the form.  Session issues are common in laravel due to the many moving parts and configuration necessary to make it work.
Take a look into that and let us know how that goes.  You may get more a more tailored response by asking in a more generic location (e.g. Larachat slack, laravel discord, laracasts forums) and providing more details about your configuration

Closing due to inactivity.

Version information

Laravel: v5.8.34
Voyager: v1.2.6
PHP: 7.2

Description:
After login through the voyager login form, I get a 419 error.
Additional context
It seems to work for everyone except for me and my users. I created several of them. Initially, they work, but after logout and logging in again I get the error. I also tried to delete all browser-data and using the incognito-mode of chrome.
Thank you!
I'm really stuck with this one so every help is appreciated!

I am facing the same problem, how did you fix it ?

Please see my comment above.  419 is definitely a CSRF issue


How do you write .sv from UHDM?

I can use surelog to read in .sv and elaborate it, then use the UHDM python VPI / SWIG interface to interrogate it, yay!
Sorry if I'm missing something obvious, but is there a utiltiy to create a .sv (eg. verilog 2017) from the UHDM?
Follow-up: for elaborated and non-elaborated?
Thanks!

You can have a look here : https://github.com/chipsalliance/Surelog/blob/master/src/roundtrip.cpp

FWIW - You can also import the UHDM into Yosys (https://github.com/chipsalliance/yosys-f4pga-plugins/tree/main/systemverilog-plugin) and then use Yosys to write out verilog (https://yosyshq.readthedocs.io/projects/yosys/en/latest/cmd/write_verilog.html).

@mithro's suggestion is going to write out Verilog 2005.
@Thomasb81's suggestion to use @hs-apotell's rountrip executable shows the general idea to traverse UHDM but does not get you quite there.
@maxslug There is an embryonic pretty printer here that prints SV expressions from UHDM: https://github.com/chipsalliance/UHDM/blob/3852e15864f362dca23b3c20456e77f3254e41c5/templates/ExprEval.cpp#L746
"Someone" has to pick up from there and write the code to pretty print the entire design.
It could be done in Python too if you feel like it!


Shorten block range on errors

When there is a possibly transient error in processing such as a timeout, we should shorten the block range to see if that helps get through the error.

This was done.


[2.0] Add a global cache for downloaded .gem files

What was the end-user problem that led to this PR?
The problem was that bundler would need to download foo-1.0.gem files from a RubyGems server for each different ruby version installed on a user's machine. It also meant that people installing into a per-app path would need to re-download every gem for that bundle an additional time. This adds up, and makes bundle install slower than it needs to be.
Was was your diagnosis of the problem?
My diagnosis was that Bundler could keep a (per-source) cache of these .gem files, and pull from that cache instead of hitting the network whenever possible.
What is your fix for the problem, implemented in this PR?
My fix implements said cache, in a very similar way to the compact index cache (same cache slug per remote strategy, etc). This largely comes from https://github.com/bundler/bundler/pull/3983.
Why did you choose this fix out of the possible options?
I chose this fix because it is safe when used from multi-source gemfiles, it is easy to clear (rm -rf bundle cache), and it minimally interferes with the existing installation process.

This is looking great to me. Very excited about using --path but not redownloading all the gems every time. 👍

Green. Is there anything else I missed in https://github.com/bundler/bundler/pull/3983 that's relevant? (I think there were some commits related to other changes that made that diff needlessly large)

@bundlerbot r+

(gonna export the env for this and run from master on my machine until this is released, now)

@indirect bundle config --global global_gem_cache true should also do the trick

Ugh has this timed out on CI?

@bundlerbot retry

@bundlerbot retry

Rebased. @bundlerbot r=indirect


External errors not being set correctly

There seem to be a few problems with external errors that I have noticed.

If a field does not have client-side validation rules set at all then external errors will not be set.
When I fill out my form and submit it (validateForm) then external errors are not shown.

Reproduction link:
https://stackblitz.com/~/github.com/martinszeltins/regle-test-3
Screenshot 1

Screenshot 2

I'm working on this

Fixed the behaviour on 0.0.14, also added type inference for properties in externalErrors that are not in the rules declation. Still need to to it for the regle.$fields types. Also need to add tests still
Thanks for the bug reporting!

@victorgarciaesgi Thank you! 🙏🏻