fix: Run ingestion-core publish on version bump

Describe your changes :

Ability to run ingestion-core publish on version bump as well.

Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

🤞  that this fixes the problem for good


Cache-Call Insertion breaks Flink iterations

With the current setup, if a workset collection is accessed more than twice, it will still be cached by the cache call insertion optimization.
Example
for (_ <- 0 until 5) {
  val cands = for {
    x <- comps
    e <- edges
    y <- comps
    if x.id == e.src
    if y.id == e.dst
  } yield LVertex(y.id, Math.min(x.label, y.label))

  comps = for {
    Group(id, cs) <- cands.groupBy(_.id)
  } yield LVertex(id, cs.map(_.label).min)
}

This is translated to (approximately)
FlinkNtv.iterate(comps)(comps => {
  val cands = for {
    x <- comps
    e <- edges
    y <- comps
    if x.id == e.src
    if y.id == e.dst
  } yield LVertex(y.id, Math.min(x.label, y.label))

  for {
    Group(id, cs) <- cands.groupBy(_.id)
  } yield LVertex(id, cs.map(_.label).min)
})

In the rewritten version, the lambda passed to FlinkNtv.iterate has a parameter comps which is accessed twice the lambda body. Because of that, a subsequent addCacheCalls transformation will insert a FlinkOps.cache(comps) call at the beginning of the lambda body.
Suggested Solution
As a first approximation, we should exclude parameter caching for lambdas passed to a FlinkNtv.iterate operator.

PS. The same applied for edges -- usages in the iterate lambda should not be considered as part of the caching criteria.


Add minion observability for segment upload/download failures

Currently, minions do not provide observability into upload or download failures. Added mechanism to log errors and bump metrics when either upload or download fails, so that operators can set alerts on these metrics to detect the inconsistent state quickly and remediate if possible.
Issue #10973

Codecov Report

Merging #10978 (6f686e5) into master (6a47311) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #10978      +/-   ##
==========================================
- Coverage    0.11%    0.00%   -0.12%
==========================================
  Files        2189     2173      -16
  Lines      118056   117638     -418
  Branches    17872    17818      -54
==========================================
- Hits          137        0     -137
+ Misses     117899   117638     -261
+ Partials       20        0      -20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration2temurin11
?

integration2temurin17
?

integration2temurin20
0.00% <0.00%> (ø)

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
0.00% <0.00%> (ø)

unittests2temurin11
?

unittests2temurin17
?

unittests2temurin20
?

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

Impacted Files
Coverage Δ

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (ø)

...ion/tasks/BaseSingleSegmentConversionExecutor.java
0.00% <0.00%> (ø)

... and 18 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

The changes look good to me. Thanks for adding them


 We should mark it broke, or fix it.

to be pedantic: Dependencies in the torch distribution are already marked as broken. One cannot currently build a top level torch derivation as it's deps are marked broken.
Does this constitute being marked broken already? Not sure.
Does being marked broken for such a long time mean we should remove it? I think so.

I mentioned this issue last week in
https://github.com/NixOS/nixpkgs/issues/56398#issuecomment-542085717. It could
be reimplemented by using the new architecture.

Not only is it broken, it's completely unmaintained and dead upstream:
https://github.com/torch/torch7#development-status

As of 2019, the Torch-7 community is close to non-existent.

It appears that the useful bits have been ported over to pytorch, and the rest left to rot.  Let's remove the remains.


FSExplorer: Clear cache when user explicitly refreshes explorer

Fixes #413
Looking at the bug and flow of code, it looks like VSCode calls stat only on refreshes, extension opens, or fileSystemProvider registration. Given the FileSystemProvider API, it looks like we would need to keep track of when to return cached results, and when we should do a re-fetch.
For blobs, there's already a policy of explicitly refreshing at every readDirectory call.

Closing due to lack of activity


TitleBar property doesn't work when using AcrylicWindow.Enabled attached property

I am using the AcrylicWindow TitleBar property after upgrading to 0.7 to get the 1903 fix. However, it doesn't work if you enable the AcrylicWindow using the AcrylicWindow.Enabled attached property.

Hi, @tim-fernico.
I fixed this issue, and released as FluentWPF 0.8.0-alpha003
Please, try it.


Support file globbing in deploy package path

Deploy package path supports minimatch pattern to find deploy package. e.g. //**/*.zip

@bishalpd,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Update README.md

@riccardoferretti Updated broken urls in README.md using relative links

thanks @darkmastermindz !


Bump cardano-ledger-shelley-test version

since it's bounds on cardano-ledger-shelley have changed from 1.3 to 1.4
Description
I'm having issues releasing to CHaP, and I think it might be because we haven't bumped this version when changing the cardano-ledger-shelley bounds (or at least one of the issues!)
Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
versioning process.
[ ] The version bounds in .cabal files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
[ ] All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages. New section is never added with the code changes. (See RELEASING.md)
[x] Code is formatted with fourmolu (use scripts/fourmolize.sh)
[x] Cabal files are formatted (use scripts/cabal-format.sh)
[x] hie.yaml has been updated (use scripts/gen-hie.sh)
[ ] Self-reviewed the diff

For cardano-ledger-api,  an upper bound to the dependency to cardano-ledger-shelley has been added (1.5)  - does this require a release?


Silent option

Hello, i would love to see a silent option to be added to the tool.  It would be nice if the tool could run without showing the popup, or at least without focussing it.

Hmm, I assume this is on Windows?
Because the program itself is a command line application. I guess Windows will open a command prompt by default, not sure if there is some option to affect it.

Hi,
yes i found a way to overrule it in the batch script i made. issue can be ignored! Thanks for the fast reply, did'nt even have time to reply with my sollution hehe.


how to deploy kuBosh on aws

Hello experts,
I'm trying to setup kuBosh on aws.
I'm referring  https://github.com/pivotal-cf-experimental/kubo-deployment/blob/master/docs/guides/customized-installation.md for the setup.
tried to generate env_config by running
bin/generate_env_config my-target-folder mybosh-name aws
but there is no configurations file available for aws.
https://github.com/pivotal-cf-experimental/kubo-deployment/tree/master/configurations
Please let me know if there is any user guide available for the installation on aws or I'm missing something.
Any inputs/insights appreciated.
Regards,
Deepak Upadhyay

Hey @cfmagnum.
Looks like we ran in the same issue. I've created an issue a bit earlier https://github.com/pivotal-cf-experimental/kubo-deployment/issues/96 :)

Hi @cfmagnum @allomov
We have plans to support AWS, but it isn't our top priority.
An issue has been created to track the progress on AWS support


Network lib ?

HI !
This looks great !
Anyway currently todo any sort of networking, like HTTP, WebSockets, JSON ?
Keep up the good work !

Hi, sorry for taking so long to get back to you. I never got a notification.
There's no way currently to do that with what runtime provides, but you can extend Ale using the FFI interface.
There's a chance I may get back to Ale soon, and this could be one of the things I look at.


extension is not working

it keeps saying ''Error!" on https://chapmanganelo.com/ and i don't know server url

You need to host the server. Follow these instructions (Server Usage Instructions (KAGGLE HOSTING, NEW)(21/11/23))

@BinitDOX i don't have accelerator option under notebook option

You need to verify your kaggle account with a phone number


Running the training benchmark with timm model produces error

I tried to run the training benchmark with a timm model. I got the following error

It seems that an index out of range error occurs trainer attempts to fetch a batch from the dataset using the data loader
steps to reproduce:

create the following configuration file (cpu_training_pytorch_timm) in the tests/configs directory

defaults:
  - backend: pytorch
  # order of inheritance, last one overrides previous ones
  - _base_ # inherits from base config
  - _training_ # inherits from training config
  - _timm_ # inherits from timm config
  - _cpu_ # inherits from cpu config
  - _self_ # hydra 1.1 compatibility

experiment_name: cpu_training_pytorch_timm

benchmark:
    training_arguments:
      use_cpu: true

run the benchmark using pytest

pytest -k "cpu_training_pytorch_timm"

or using cli
optimum-benchmark --config-dir tests/configs/ --config-name cpu_training_pytorch_timm.yaml

Expected
benchmark runs successfully

obtained
access out of bound error

I think the root cause is that there is a mismatch between the column name generated by the dataset 'pixel_values' and the expected input name by the timm model 'x'.
Before starting to train the model, The trainer class checks the model's signature looking for its input name and removes the unused columns from the dataset. Therefore, it removes the 'pixel_values' because it thinks it wont be needed by the model.
The issue does not occur in the inference benchmark because the inference benchmark calls the function prepare_inputs which maps pixel_values to x before running the model
https://github.com/huggingface/optimum-benchmark/blob/d2d1e620c6ebb3a7b7ec7adc2f538d447c7ab2ca/optimum_benchmark/benchmarks/inference/benchmark.py#L108

currently we're using transformers trainer, which is not compatible with timm models.


TFclient Error Message about data loss

Hello
Recently, I tried reverb.TFClinet as below
'''
table = reverb.Table(name='experience', sampler=reverb.selectors.Prioritized(0.8), remover=reverb.selectors.Fifo(), max_size=2048, rate_limiter=reverb.rate_limiters.MinSize(1))
reverb_server = reverb.Server(tables=[table], port=8003)
reverb_client = reverb.TFClient(f"localhost:{self.reverb_server.port}", shared_name="tfc", name="tclient")
'''
However, I  got the below messages when used reverb_client.insert() method
'''
[reverb/cc/writer.cc:391] Error when stopping the confirmation worker: DATA_LOSS: Item confirmation worker were stopped when 1 unconfirmed items (sent to server but validation response not yet received).
[reverb/cc/writer.cc:387] Unable to confirm that items were written.
'''
I used tensorflow 2.11, reverb 0.10, ubuntu 20.04, python 3.10

Hi,
Thanks for reporting this. I would like to start off by encouraging you to avoid using the tf ops if possible. Using the TrajectoryWriter or the StructuredWriter should be preferred when possible and you should prefer TrajectoryDataset for sampling.
That being said, this error message isn't as scary as it seems. What it really means is that the client didn't receive the confirmation from the server that the item has been successfully inserted into the table before the connection was closed. Note that this doesn't mean that the item wasn't inserted, it just means that the client didn't get the confirmation of the insertion.
I have sent a fix though which ensures that the insert op blocks until the confirmations are received which will make this messages go away. It will be available in nightly in the next few days and then part of the next stable release once tf 2.12 is available.
Cheers,
Albin


Add the schematics for the control circuitry of the circuit

based on:
https://github.com/ebastler/zmk-designguide/tree/main

for the battery management
USB C connector

and using a BT40 fanstel module for the nRF52840 microcontroller

[ ] Add the reset button from the Nice!Nano schematic

[ ] Calculate the battery chargng current

[ ] Change resistors to 0603

[ ] Find correct battery connector

[ ] Add power switch

[ ] Add gpio controlled LED.


[F000458] Rep. Stephen Fincher

Rep. Stephen Fincher:
http://fincher.house.gov
Likely contact form URL to start with:
http://fincher.house.gov/contact-me/email-me

YAML file lost. I am working on it.


SILCombine: fix keypath optimization with optional chaining and classes

For optional chaining a swift_enum is created.
If the sub-projection is ending a begin_access in the some-branch, it also needs to be ended in the none-branch.
Fixes a compiler crash and/or a miscompile.
https://bugs.swift.org/browse/SR-14534
rdar://77224220

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 0335efef8e4568e9aa9119cb8311917fd32cddf5

@swift-ci test linux

@swift-ci smoke test macOS

@swift-ci smoke test macOS


Race conditions being detected with -race for requests

Hit a few race conditions in the SDK's protocol/rest build logic. Need to investigate why these race conditions are being hit, and what impact they have.  Probably a small refactor needed to prevent the race conditions.
https://gist.github.com/jasdel/b0858b5c06cbd88f6a6b
Noticed in #190

Fixed in 1a0d8fa04


How to get page source of displayed screen like appium

How to get page source of displayed screen like appium

Can I get page source for only displayed view on screen

https://github.com/google/EarlGrey/blob/master/docs/faq.md

I think you'd have to write a custom method if you want to filter out the invisible elements.

Thanks, @bootstraponline I have added custom method to do this.
Closing this issue as currently there is no way to handle this using EarlGrey.

Consider posting your custom method, it may be useful to others. 😄
I'm hoping that EarlGrey gets better tooling in this area. If we had a websocket that'd enable an inspector tool similar to uiautomatorviewer or appium desktop.

Ohhh, I really happy to post this method but the method is not written on the iOS side.
I have captured string hierarchy using GreyElementHierarchy and dump it into the file. And added a ruby script to apply the filter for visible elements. As this is our requirement.

I think a lot of iOS teams are using Ruby with Fastlane & Cocoapods, might still be useful.

I'm hoping that EarlGrey gets better tooling in this area. If we had a websocket that'd enable an inspector tool similar to uiautomatorviewer or appium desktop.
@bootstraponline  can you please elaborate this or share any link if you have how to integrate.

Checkout how the WebDriverAgent inspector works:

https://github.com/facebook/WebDriverAgent/tree/master/Inspector

This doesn't exist today in EarlGrey, however I'm hoping to persuade the team eventually. 🙂


Do we want to enforce maintainers to have packaging configuration?

I'm trying to setup omero-test-infra in a repository that is not set up to be a python package.
Currently, omero-test-infra requires to be set-up in a repository that contains pyproject.toml or setup.py. Is that a feature or a bug?
It might be good to make project maintainers to setup packages if they want to use this setup, but I'm wondering if that's what we want to do.

It's a bit specific and a bit uncommon I think, but I'm creating an extension for OMERO that uses ontop.
And then got prompted about needing a config file for Python, which didn't make sense in my case, but then I imagined it might be other users since there's a big overlap of this ecosystem with Java, for example.

Which "stage" are you passing to the test-infra srv and dev do not run Python commands

It might well be that the ontop use case (https://github.com/German-BioImaging/omero-ontop-mappings) needs another stage, @marimeireles like service. i.e., you're going somewhere where no one has gone before ;)
That could possibly then use the docker-compose.yml in your repository as an extension of the main one. That might help to get rid of some of the diff in https://github.com/ome/omero-test-infra/compare/master...marimeireles:omero-test-infra:master
cc: @cfgrote


Radial scale point label backdrop color

Resolves #5085

Size Change: +225 B (0%)
Total Size: 231 kB

Filename
Size
Change

dist/chart.esm.js
66.8 kB
+80 B (0%)

dist/chart.js
84.5 kB
+76 B (0%)

dist/chart.min.js
60.2 kB
+69 B (0%)

ℹ View Unchanged

Filename
Size
Change

dist/chunks/helpers.segment.js
18.8 kB
0 B

dist/helpers.esm.js
1.08 kB
0 B

Added the tick backdrop padding changes


Release 6.0.9

Questions
Answers

Description?
Release 6.0.9

Type?
bug fix / improvement / new feature / refacto / critical

BC breaks?
yes / no

Deprecations?
yes / no

Fixed ticket?
Fixes PrestaShop/Prestashop#{issue number here}.

How to test?
Please indicate how to best verify that this PR is correct.

Hello @Hlavtox
Thanx for the PR !
I tried to check the PR , But when I install it,  I see that there is some conflicts

Could you please check it !
Waiting for your feedback !
Thank you

Merge conflicts are probably from #153 and #155 in config.xml. Both PRs change version number to 6.0.9.

There should be no conflict now, I tested the merge on my fork and it went fine with no conflicts.
@sallemiines Can you retest it? We need the release ASAP because it's making troubles, please.

Hello @Hlavtox
Thanx for the feedback , But unfortunately I re-test the Pr , and I find that the conflict still exist !

@kpodemski could you please check ? Github says there is no conflict, but when we try to install the PR, conflicts appears!
Thank you !

Hi @Hlavtox @kpodemski ,
Given the content of the release, I think we indeed can release it without functional tests, it can be merged :)

Ping @kpodemski for merge 👍

Thanks @Hlavtox


[Help needed] CallbackAddress clarification

In several CN request messages it is defined mandatory field called callbackAddress, like
"dspace:callbackAddress": "https://..."
My question is:
is this value defined somewhere by the specification or this is left to the connector implementation, and receiver will use value provided, and send response to that address, without any processing.
For example:
if connector receives dspace:ContractRequestMessage with callbackAddress=https://igor.connector/listen/response
this means that receiving connector will return response to https://igor.connector/listen/response
This url is not defined anywhere in the specification. It is up to the implementation of the connector to provide valid logic for processing response under this API.
Why this ask for help:
https://docs.internationaldataspaces.org/ids-knowledgebase/v/dataspace-protocol/contract-negotiation/contract.negotiation.binding.https#id-2.2-the-negotiations-request-endpoint-provider-side
says:
Callback messages will be sent to paths under the base URL as described by this specification.
If added here Callback request message will be sent ...
this means (IMHO) that callback is referred only in Prerequisite section scope - for receiving requests, both on provider and consumer side.
callbackAddress will be used as-is, without any  url replacing.
Thanks

Hi @IgorBalog-Eng
not sure if I understood the question. Does this explanation help?

All callback paths are relative to the callbackAddress base URL specified in the Contract Request Message that initiated a CN. For example, if the callbackAddress is specified as https://consumer.com/callback and a callback path binding is negotiations/:consumerPid/offers, the resolved URL will be https://consumer.com/callback/negotiations/:consumerPid/offers.

https://docs.internationaldataspaces.org/ids-knowledgebase/v/dataspace-protocol/contract-negotiation/contract.negotiation.binding.https#id-3.1-prerequisites
Also the table here should help:
https://docs.internationaldataspaces.org/ids-knowledgebase/v/dataspace-protocol/contract-negotiation/contract.negotiation.binding.https#id-3-consumer-callback-path-bindings
--
Matthias Binzer

Hello @matgnt
There was misunderstanding (lost in translation or overthinking) from our side, when callbackAddress are mentioned.
We were thinking that callbackAddress were used to send response (ContractNegotiation in case of ContractRequestMessage request send by consumer) and such requests/responses are async, and there are no endpoints defined to receive such document/object.
But it was "different": requests are synchronous, sending response ContractNegotiation, and once decision is made what to do with ContractRequestMessage, next message in flow will be sent to callback binding (accepting propper message, Agreement, Offer or Termination) in combination with callbackAddress from CRM.
This is just one simple example, but logic applies to other decision making requests, that does not need prompt decision.
I cannot promise but I will try to update existing negotiation diagram with callback endpoints, and share it with community (if it adds value, we can keep it, otherwise we learned something)

yes, correct, the contract request is a http request/response with 200 OK and THEN, the async part is a completely separate http request/response from the provider to the (consumer's) callbackAddress... and so on...
Sounds to me like we could close the issue.
--
Matthias Binzer

Closing issue.
Clarification of help requested in comments


Move flatten operations to normalize/simplify transformation

we should take the optimisations of nested 'and' and implications of ands etc and move them to the normalize transformation (perhaps specifically the simplify_boolean operation). That will significantly simplify the flatten code.
that will be most powerful after push down negation and decompose is preceding this transfo; so we can do those first

https://github.com/CPMpy/cpmpy/pull/291/files#r1203689588

I can take that. However, currently simplify boolean is only used to removes boolean constant from all CPMpy expressions. If we move these optimizations to simplify_booleans then the purpose of the function changes. I still think though it should be done, I will create a PR for this.


WARN users if there are background processes running at end of script

If a user runs a process in the background, raise a WARN if the process is still running at the end of the script

This is a bit old but were you thinking of something like ps T and detecting the number of processes associated with the terminal before ending the script?
    PID TTY      STAT   TIME COMMAND
  48855 pts/1    Ss     0:00 bash
  61494 pts/1    S      0:00 sleep 10
  61553 pts/1    R+     0:00 ps T


Internal method invocation style

See https://github.com/tc39/ecma262/issues/91. Currently I paper over it using DefaultInternalMethod which seems somewhat reasonable.

I like it. I'd call it "Ordinary" instead of "Default" and remove the term default from "default ordinary object internalMethod internal method"

In that other thread it seems Allen values having both "default" and "ordinary". Happy to rename the abstract operation though.
The other thing that's still a bit outstanding is that if you do this, are the internal methods this default ordinary object internal method invokes, also of the default ordinary kind?

Turns out this is not a problem as far as I can tell. Leaving this as-is for now.


Better dependency on server using node modules

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


Flight broken

Flight seems to be broken in fabric 1.20.1, you can activate the flight fine but once it teleports most buttons become non-functional and you cant get the door open. Some things can still be pressed/interacted with, but anything relating to the fight seems to break

Hello! For temporary solution you can break the console unit and put it again (for checking which controls are usable now you can use F3+B, that hotkeys shows entity hitboxes). I will try to recreate your problem and fix it. What mods are you using alongside dwm?

Hi i tried it with no mods and it still had that error, i am using fabric api 0.83.1 if that causes any problems

@JibberishGulp what says the "Materialized" line on the console screen? Looks like you didn't materialize. Maybe the last time when you started flight, you ran out of fuel and the TARDIS didn't find a safe coordinate to make an emergency landing on.
In this case, when you are not materialized, you cannot use most of the levers and buttons, that's right. If this is the problem, it must be confirmed and then we can think about how to get out of this situation inside the game.
To make sure it's not a technical issue, could you send a F3+B screenshot like this?

Yeah, i tried updating the fabric api, mod and architerctury, and I followed the video exactly but it refuses to materialize after the flight. Also, i dont see it on the page anywhere but the fabric version of the mod does say "fabricconfigapiport" is required for it to run. Considering everything else works fine it might be a small issue with the port, because otherwise the mod seems to work perfectly

@JibberishGulp yes, ForgeConfigApiPort is required and it must be listed in dependencies of Fabric file on CurseForge. Also the game must refuse to launch if that api-mod was not found, so i doubt the problem is in that
I'm trying to recreate your problem, but I always materialize after a flight. I hope sending your logs (from {MINECRAFT_PATH}/logs) right after completing a flight helps. Maybe there is some crash in landing destination calculation or something

I attached the latest log file, it seems like it un-materalizes but the flight itself just never starts. If it helps i can attach a video on the procress im doing, could just be user error as the video seems slighty wrong (the handle level doesnt flick, only the up and down button does)
latest.log

@JibberishGulp yes, attach the video please from the beginning, it will be just great! Logs seems clear

Here it is, i kinda just messed around in the video but from what i understand should be everything you have to do for it to work. Im sorry if this has been some kind of stupid user error this whole time
https://youtu.be/NerrUbbsphc

@JibberishGulp Thank you for the video, i think i got it.
The fact is that levers and sliders have different clicking logic for the left and right mouse buttons. For example, the coordinate lever with left mouse button click will decrease the value, the right click will increase it. Such logic have: coordinate levers, vertical scanning lever, scaling lever, flight start lever, handbrake lever, materialization slider button. This was done in order to prevent miss clicks
This is not your omission, it only means that the interface is not entirely clear. Give it a try, if that's really the case, I'll think about how I can improve this point: maybe a simple description of this logic on the page will be enough, or I'll change the behavior

Hi yes it worked perfectly,  i cant believe i didnt discover that this entire time. I dont think the system is bad but because most mods break with you mine them i cant say i wouldve ever tried it without reading it somewhere, if your thinking about changing it maybe an interact button might be more clear. Before I end this ticket is there a config file to edit the spawn rate of the tardis?

@JibberishGulp Great!
The spawn rate can be customized by creating your custom datapack (this wiki might help). The file "(DoctorWhoMod-1.20.1-fabric-1.0.2.jar as archive)data/dwm/worldgen/placed_feature/abandoned_tardis_{surface|underground|nether}.json" must be overridden in the datapack. I think changing the "chance" parameter for minecraft:rarity_filter is what you need now, but in the future I plan to change the TARDIS generation a little or completely redo the approach, because now there are too many of them in the world. But it only plans for now :)

Thank you for the cool mod, sorry about the trouble man. Take care


Add calculations to character sheet

Since it is a spreadsheet, it should be able to automatically calculate things like health.

Resolved via: https://github.com/ekultails/simpler_things_extras/commit/287b76704922775653793bd19a94a76e77938277

Resolved via: https://github.com/ekultails/simpler_things_extras/commit/287b76704922775653793bd19a94a76e77938277


How to submit PR for twig-phpstan-compiler and phpstan-twig-rules?

The READMEs say to submit it here, but I just cloned the monorepo to work on tests and realized that the twig-phpstan-compiler and phpstan-twig-rules packages are not included here. Or maybe I'm just blind?
I'd appreciate some guidance since I already got quite a few hours invested in this 😁

Hi, this package was moved to https://github.com/revealphp/, to keep all templates-related PHPStan mechanics in one repository.
You can contribute to the rules there :+1:


[KED-519] Support template variables in configuration files

Description
As a user of kedro I would like to be able to parameterize/template my config files, in order to be able to change configurations of lots of datasets, accross config files, by changing only one variable in a centralized place.
This would allow me to for instance:

Create flexible folderpaths for all (or a large subset of) Datasets, allowing me to

Quickly switch root directories from a dev/test environment to a prod environment
Work with data iterations
Have pipelines for different countries run using the same config files

Reduce the size of my config files by templating common parts (for instance I could parameterize the 'save_args' or 'load_args' of a Spark Dataset)

Context
User stories/ideas from current users have mainly focussed on the templating of the folderpath.
Some more advanced ideas have inlcuded:

Allowing for boolean logic inside the template and the values to allow for easier switching (e.g. one could have a parameter dev=True, and when this gets switched to False, all the conifg that depends on this boolean switches. This would be usefull if for instance both the folderpath and the load_args change when running in dev vs prod, in this case only one parameter would have to be changed)
Allowing for templating inside the template values. This would reduce the chance of errors in complex config folders, that all have a similar root but might have multiple parameters in them. Example:

	raw_employees:
	  type: kedro.contrib.io.pyspark.SparkDataSet
	  filepath: s3a://${s3_bucket}/${country}/${raw_folder}/employees.csv
	  file_format: csv
	  load_args:
	    delimiter: ";"
	    header: true

with values.yml:
	s3_bucket: 'bucketname'
	country: 'uk'
	raw_folder: '00_raw'

you could have:
	raw_employees:
	  type: kedro.contrib.io.pyspark.SparkDataSet
	  filepath: ${raw_root}/employees.csv
	  file_format: csv
	  load_args:
	    delimiter: ";"
	    header: true

with values.yml:
	s3_bucket: 'bucketname'
	country: 'uk'
	raw_folder: '00_raw'
	raw_root: s3a://${s3_bucket}/${country}/${raw_folder}

Lots of people I spoke to have created their own version of this on kedro projects (mostly the simple one without templating inside the values.yml). It would be good to make a solution for it in the kedro core, incorporating best-practices and unit-tests.
Possible Implementation
You can make a class that inherits from ConfigLoader with an extra class called resolve that is similar to the .get method with the only difference being that it applies the default value dict to config before returning it.
One way this could be done is using the string Template class in python. Otherwise, some logic needs to be written to replace template variables with defaults.
Checklist
Include labels so that we can categorise your issue:

[ ] Add a "Component" label to the issue
[ ] Add a "Priority" label to the issue

Resolved in #41


npm error while running build script

Hi,
If you download then run npm install then npm run build you have this error
You have mistakenly installed the `babel` package, which is a no-op in Babel 6.
Babel's CLI commands have been moved from the `babel` package to the `babel-cli` package.

    npm uninstall babel
    npm install babel-cli

See http://babeljs.io/docs/usage/cli/ for setup instructions.

Current package.json
"devDependencies": {
    "babel": "^6.5.2",
    "babel-preset-babili": "0.0.3",

Modified version works better
"devDependencies": {
    "babel-cli": "^6.16.0",
    "babel-preset-babili": "0.0.3",

:cupid:

Yeah I fixed this yesterday... sorry about that.


Cannot convert value of type

Hello
I was try use your Pod (thanks for this), but when i put block of code in AppDelegate - i have an error.
(http://prntscr.com/dd4wwg - this is a screenshot)
My MainViewController type - UIViewController. What's wrong with him?
Please, help me.

Can anyone help me?

I'm having the same error. Any updates?

@felstell What's your swift version? 2.3 or 3.0? And what's your SlideMenuController version?

@Pluto-Y Swift 3.
But i find solve for this problem:
I did this Container View

And i this LeftViewController

Then i'm add only  "import SlideMenuControllerSwift" in my AppDelegate

In storyboard i have one Container View, and two Navigation Controller with Storyboard ID:

And it's work! :)
I hope this help anyone with his trouble


Reuse PyEnumerable, removing need for PyKeyValuePairEnumerable

PyKeyValuePairEnumerable<,> had almost an identical implementation to PyEnumerable<> except for the conversion of the PyObject value. The former calls PyObject.As<,>:
https://github.com/tonybaloney/CSnakes/blob/b9a6b8d76c5de5c3beb2b46d1639b06253e4002d/src/CSnakes.Runtime/Python/PyKeyValuePairEnumerable.cs#L42-L43
whereas the latter calls PyObject.As<>:
https://github.com/tonybaloney/CSnakes/blob/b9a6b8d76c5de5c3beb2b46d1639b06253e4002d/src/CSnakes.Runtime/Python/PyEnumerable.cs#L42-L43
That's a lot of duplication of maintain, sync, test and cover. This PR consolidates the duplication so there's only one bulk implementation. It does this by adding another type parameter (called TImporter) to PyEnumerable<> that determines how the PyObject is imported (into presumably a managed type):
internal class PyEnumerable<TValue, TImporter> : IEnumerable<TValue>, IEnumerator<TValue>, IDisposable
    where TImporter : IPyObjectImporter<TValue>
{
    // omitted for brevity
}

The importer has a static member, which bears zero cost (that is, no allocation and no virtual dispatch needed):
interface IPyObjectImporter<out T>
{
    static abstract T Import(PyObject pyObj);
}

The two import strategies, PyObject.As<> and PyObject.As<,>, are then embodied into implementations of this interface. The import strategy is chosen at the time PyEnumerable<,> is instantiated. A PyEnumerable<> with a single type parameter still exists, but which uses the default strategy of PyObject.As<> so most of the code is untouched except in PyDictionary<,>.GetEnumerator.

Love it! this is a lot cleaner


How can I output real-time progress when I'm pushing? Instead of printing all the progress at once after the push

ReceivePort receivePort = ReceivePort();
sendPort.send(receivePort.sendPort);
receivePort.listen((params) {
  ...
  try{
  	...
    var keypair = Keypair(username: username,pubKey: sshKey.publicKeyPath ,privateKey: sshKey.privateKeyPath,passPhrase: sshKey.passPhrasePath);
    var callbacks = Callbacks(credentials: keypair,
        sidebandProgress: (progress){
          sendPort.send(progress);
          print(progress);
        }
    );
    remote.push(refspecs: ['refs/heads/master'],callbacks: callbacks);
    print('push to ${remote.url} success !');
    remote.free();
    sendPort.send('success');
  }catch(e){
    print('push error:$e');
    ...
  }
});

Hi, what does stdout.write(progress) instead of print(progress) produce?

The result is the same .
By the way, This code runs on iOS devices. Run asynchronously using Isolate.
I'm trying to show the real-time progress of push on the screen.

Is progress displayed in "real-time" when you don't use isolate?

Can you confirm, please, that for example, using that sidebandProgress callback in Repository.clone(...) provides you the result you want?


Invalid multinomial distribution (sum of probabilities <= 0)

After training a new 3 x1024 model (dropout=0.1, batchnorm=1) for a couple of days, I got the following error:
$  th mysample.lua -checkpoint /nas/doc/nn/checkpoint_200.t7.reset.t7 -length 2000 -gpu -1
Loading 	/nas/doc/nn/checkpoint_200.t7.reset.t7
Loaded
/home/alekz/torch/install/bin/luajit: bad argument #2 to '?' (invalid multinomial distribution (sum of probabilities <= 0) at /home/alekz/torch/pkg/torch/lib/TH/generic/THTensorRandom.c:120)
stack traceback:
[C]: at 0x7ffad932e1d0
[C]: in function 'multinomial'
./LanguageModel.lua:196: in function 'sample'
mysample.lua:43: in main chunk
[C]: in function 'dofile'
...lekz/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00405d70
What can be the problem?

Apparently it's related to https://github.com/jcjohnson/torch-rnn/issues/65


Install<EMAIL_ADDRESS>and fix<EMAIL_ADDRESS>as otherwise cucumber-html does not work

https://github.com/gkushang/cucumber-html-reporter/pull/197#discussion_r320247815

LGTM, but we should revert this after: https://github.com/gkushang/cucumber-html-reporter/pull/197


About INatureList17 acc

I want to know how to set batch size and lr on INatureList17，Does anyone know?

@jpoberhauser Good, thank you very much


chore: make run_samples_test file executable

Good catch!

Safe to close when #204 is merged.
Kokoro - Test: Sample Java 11 passes but 8 keeps failing with this error:
notifications {
      status: ERROR
      commit_status_url: KOKORO_BUILD_URL
      description: "Infrastructure error. Please apply label \'kokoro:force-run\' to rerun the build."
    }

duplicated of #204


Interactive Dashboard Tile - Using schema name.

The interactive dashboard tile seems to be using the schema name, which looks a little odd. See my example attached.
Would be great to have more control over the dashboard like setting colours, what values are used, whether or not the view pop-out for the tiles is used (or using it like a filter).

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

ID: 04e12c55-01ce-12e9-a958-fde2d8c07ede
Version Independent ID: 10e4c17b-f352-e139-220d-96518eb673d6
Content: Configure interactive experience dashboards in Dynamics 365 for Customer Engagement
Content Source: ce/customize/configure-interactive-experience-dashboards.md
Service: crm-online
GitHub Login: @Mattp123
Microsoft Alias: matp

@skoofy5 - Thank your for your feedback. For product-related feedback, visit https://experience.dynamics.com/ideas/.
Closing this issue.


[Storage] Fix #16872: az storage blob now (2.19) requires login even if connection-string is provided

Description

Fix #16872
Testing Guide

az account clear
az storage blob sync with account key or connection string
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage

Do azcopy.remove have the same issue? I remember in #16519 both remove and sync are modified due to permissions.
https://github.com/Azure/azure-cli/blob/bc4d70c7b98065c8132fda465d549da0ba924d9d/src/azure-cli/azure/cli/command_modules/storage/operations/azcopy.py#L57-L60

because the fix

Do azcopy.remove have the same issue? I remember in #16519 both remove and sync are modified due to permissions.
https://github.com/Azure/azure-cli/blob/bc4d70c7b98065c8132fda465d549da0ba924d9d/src/azure-cli/azure/cli/command_modules/storage/operations/azcopy.py#L57-L60

Good point! updated and added test.


wip: social logins

Why

[x] Login with apple
[ ] Login with google
[ ] Login with twitter

How

Test Plan

Commenting social login buttons for now.


Offline Route: Listen for task to load before accessing

@tschie Can you look this over?
Sample was not working for me and had to wait for task to load to fix problem.

Thanks for finding that @john0005. I just added 1 more commit just moving around some of the view logic. Could you review and make sure it works for you now?


Sort defintions more carefully

 Issue by nomeata
Thursday Sep 28, 2017 at 15:24 GMT
Originally opened as https://github.com/antalsz/hs-to-coq/issues/15

We have to do a topological sort to mix instance declarations and value declarations, and also to get instance declarations in the right order. Currently, this uses topoSortSentences. I want to see if I can make this function, or another function, a bit better at preserving the original order, so that we have to emit less order edits when things are in the right order in the source code already. examples/successors is an example for that.

 Comment by nomeata
Wednesday Oct 11, 2017 at 19:21 GMT

Would also fix the problem that the notations for operators are in the wrong order.

@Lysxia I remember that you did something relevant to this?

Indeed one of my PRs partially addressed this: https://github.com/antalsz/hs-to-coq/pull/171
The main missing piece is to track dependencies through type class dictionaries. There probably is or should be an issue about that so we can close this one.


i am a  qualified member.but now click the leaves ,and i don't qualify..please help me..

i am a  qualified member.but now click the leaves ,and i don't qualify..please help me..

Hello keybase i am complete the 3 keys..The 3 active device or paper key and then the phone is succeful and the big smile..But when i click see if im qualified..The phone number has x..Why?


Response::new does not send grpc-status code correctly

Bug Report
Grpc-web won't work with tonic. It does not send the grpc-status header correctly when only using Response::new
which leads on the js client side to the error "Error: Response closed without grpc-status (Headers only)"
Version

tonic v0.11.0
tonic-async-interceptor v0.11.1

Platform
Windows 11 64bit
Description
I tried to run Response::new(someType) it returns the data correctly but won't set the header.
However if I send Status::invalid_argument("Name cannot be empty")
Things work out just as intended.
My question is: Is this defined behavior? Because the client expects a status code. If this is not needed the client library should ignore this case then.
Here is a screenshot from the problem:

It seems like "Response::new" sets the status code 2 => which means unknown.
Client libraries expect 0 => 0K which does colide here.
Is it possible to set the code manually within the response?


feat: implement search system

I don't know why, but this onClick handler does not work....
https://github.com/turtton/volglass/blob/56923c56cf43507b0ad933b4530ce78a1e23291d/src/components/Search.tsx#L80-L82

I don't know why, but this onClick handler does not work....
https://github.com/turtton/volglass/blob/56923c56cf43507b0ad933b4530ce78a1e23291d/src/components/Search.tsx#L80-L82

fixed in 04e69a4


Cross Origin Requests

Thank you for writing this.
I'm wondering if postgraphql could enable cross origin requests by default.  As it's not currently possible to talk to it from a Relay application without setting up a proxy or the like.
I see this ( accidental ) pull request. https://github.com/calebmer/postgraphql/pull/79/commits/4211c773da0d83e244a4e55804f94e93e9630362
Wondering if this would be a good approach to take

I think it would be a useful feature to have cross origin support built in so that you dont also need to have a proxy for it. Rather than being enabled by default perhaps it would be better to have an option/cli flag to enable it so that people aren't exposing that without knowing it?

Any plan when this will be merged and released?

@d-winter I was waiting for a LGTM, I’ll merge :wink:

Released in 1.8.0 👍

@calebmer awesome timing 👍
Thanks a lot!

Just looked into this for my needs so sharing the results ...
While this feature seems like a good idea at first (no proxy needed), you will add a network overhead on each request.
The first overhead is that you will need those headers on each request (not so bad).
The second overhead is that you will have preflight request for EVERY request.
Caching preflight requests does not really work http://stackoverflow.com/questions/12013216/how-to-apply-cors-preflight-cache-to-an-entire-domain
Do a bit more work and add the proxy (a real one like nginx, not a nodejs script) and you will make your app faster.

@ruslantalpa does this change with HTTP2 and/or hosting on the same domain?
Unless preflight requests seriously slow down request performance, I’m comfortable with leaving CORS on by default to simplify the beginner experience. But that’s a good optimization to be aware of, thanks so much 👍

I don't know about the http2
Same domain means no need for CORS ... so no preflight (OPTIONS) requests, which means the headers are not needed at all and they just  add (unused) bytes to traffic.
About slowdown .. yes they do, it's all nice when it's on localhost and the OPTIONS takes 5-10ms, but move this to production over https and suddenly a requests that would take 100ms in normal conditions, turns into 2 sequential  requests, each taking 100ms so in total 200ms. Mix in Relay with it's preference for splitting requests into multiple requests to node endpoint and suddenly you have a flood.

For anyone else who struggles to understand... the CORS headers are now disabled by default, and enabled on the command line by --cors.

Hi @estaub can you please tell me the command, for how to enable cors

https://github.com/postgraphql/postgraphql/blob/master/docs/cli.md
postgraphql --cors -c postgres://...

Since I use expressjs as my custom server, so I just add npm install cors --save as a middleware to expressjs and done.

You can just use "enableCors" from the library settings (it's what "--cors" maps to) but using the cors middleware will give you more control 👍

I'm trying to use it in Docker, and behind traefik in order to setup a domain name and manage ssl, but it can only be accessed though its IP, I'm using --cors and -n <IP_ADDRESS> and I can't access it though a revers proxy :(

Could you post more details?

Thanks a lot for your time, I have uploaded the testing environment to github, if you can check it out, I would be really grateful.
https://github.com/celvin/postgraphile-docker
I'm using traefik as a reverse proxy in order to use domain names to access different containers mapping their ports
Basically you just need to run docker-compose up --build and then you can navigate to the traefik dashboard to see the deployment details:
http://localdev.zlivio.com
and the postgraphile service (it's already routed to /graphiql) is using:
http://pgpgraph-local.zlivio.com/
and to enter to the graphile container:
docker exec -it postgraphile-nexxtel bash

I can't spin your env up right now, but what's the exact error you're getting?

There is no error in the log, and when I navigate directly to the container's IP, it works
but when I try to get it through a domain name managed by traefik (reverse proxy) it fails under timeout message.
If I post another node app in the same container it works well, so I think must be something related with allowing cors or not

So a timeout implies it's not a CORS issue (those errors are explicit and typically immediate); sounds like it's a connectivity issue in your setup.

Mmm everything is possible, the thing is the same settings even the same container running another app works well through traefik but postgraphile doesn't...

I'm not familiar enough with Docker to be able to advise you there, sorry. Are you correctly exposing the port?

Thanks a lot for your support,
Yep I'm doing it well, I have had a similar issue with timeout trouble as well but with an angular 6 app, and I managed it adding --disable-host-check and listening <IP_ADDRESS> that's why I thought maybe the --cors option wasn't working well I'm postgraphil
For now I'll use it through IP but without ssl
Thanks for your time, this is a great project.

Are you using -n <IP_ADDRESS>? We have this by default in our Dockerfile which you can install from graphile/postgraphile on Docker Hub.

Yep I have checked all the doc and that Docker file as well.
I have tried this:

With connection plugin:
postgraphile -n <IP_ADDRESS> --cors -c postgres://$POSTGRES_USER:$POSTGRES_PASSWORD@$PGHOST:$DB_PORT/$POSTGRES_DB  -a -j -M --append-plugins /plugins/connection-filter/index.js

without
postgraphile -n <IP_ADDRESS> --cors -c postgres://$POSTGRES_USER:$POSTGRES_PASSWORD@$PGHOST:$DB_PORT/$POSTGRES_DB

Sorry, can't really help you - if it's a timeout then it's most likely a connectivity issue - the packets are literally not getting through (or getting back) - hence the timeout. Unless you think the server is hanging for some reason, but I've not known PostGraphile to hang except if you feed it a ridiculously complex GraphQL query 😈

haha just to get the UI so nop, hehe
thanks a lot I'll try to get all debug that I can I will read carefully everything, thanks again.

I've found the trouble, the domain name wasn't working properly, my mistake, thanks it is working just perfect, I will keep alive that docker-compose wrapper for postgraphile, in case somebody wants to use it

👍 Thanks for reporting back

I am trying to use graphql playground with postgraphile. Both are running on different origins (I am running postgraphile docker container on port 5000 and graphql-playground + my custom graphql api (using graphql-yoga) on port 4000.
The error I am having is that the playground sends OPTIONS requests to postgraphile, which seems to fail as the CORS rule is  set to only allow HEAD, GET and POST.

Let me know if you need me to create a seperate issue.

Has the issue been created for the last comment: https://github.com/graphile/postgraphile/issues/84#issuecomment-404582793
I have got the same problem.

So the bug is actually that we're sending Access-Control-Request-Method rather than Access-Control-Allow-Methods; not sure how this has never been spotted before! Raising a PR now.

https://github.com/graphile/postgraphile/pull/838

So the issue actually seems to be that they're setting an X-Apollo-Tracing header. I'm going to add that to the PR too, seems harmless.


Some project-level useful actions (add/remove tags, list with ignore tags)

There are some useful actions at project-level that are not yet supported by Watson and could add nice extra features.
PROBLEM 1: after some time, there are so many projects that a watson projects command returns too many rows to be useful (takes a lot to go through the list of projects). Moreover, some projects are no longer active. Although they must be kept for obvious reason (we want to keep the history of our activities), there is no reason to keep them in the list of active projects. There should be the possibility to close or hide a project, so that the frames still appear in logs/reports, but not in the list of projects.
SOLUTION TO 1: add a flag, e.g., --ignore-tag(s) to the projects command to exclude the visualization of projects whose frames have a specific tag; in this way, a tag such as +closed or +hidden could be added to the frames and prevent the visualization of the corresponding project.
PROBLEM 2: tags are associated to frames, not to projects. Therefore, a special command to add/remove a tag to/from ALL the frames of a project is necessary.
SOLUTION TO 2: add a new command, e.g., watson tag <project> <tag> to add a tag to all the frames of a project, and watson untag <project> <tag> to untag (or watson tag --remove <project> <tag>)
PROBLEM 3: during the "refactoring" of the organization of projects and tags, sometimes we want to add/remove a tag to every frame of a project.
SOLUTION TO 3: same solution as for problem 2. So that feature would address both problems.
I would be happy to contribute a patch if these ideas would be deemed as interesting.

Thanks for your report @toolleeo :pray: It appears that the solution to all aforementioned problems is to add a new command that batch adds or removes tags from a selection of frames (as you suggested). Instead of adding a new command why not adding a --batch option along with selectors to the edit command? An example usage would be:
$ watson edit --batch -p myproject --add-tag archived

WDYT?

I would be happy to contribute a patch if these ideas would be deemed as interesting.

:heart:

It appears that the solution to all aforementioned problems is to add a new command

Not to all the problems. Solution to PROBLEM 1 requires an option to exclude the listing of the projects where frames have some given associated tag.
Regarding the proposed solution, I see that you don't like the proliferation of new commands :-)
I agree on the idea of keeping the list of commands as compact as possible.
However I think the user would expect that the edit command would do what it says: editing the content, which means opening the editor and making changes. Which is the current behavior.
What we are actually doing it is a kind of "update", thus I suggest one of the possible options:
$ watson update project <myproject> --add-tag archived
$ watson update -p <myproject> --add-tag archived

The former format is similar to the rename command, to keep some coherence.
But if you don't like it, I am not gonna fight and would accept the edit --batch solution, which is better than nothing :-)
I can't say when I could start looking at the implementation: this is busy period :-/

I have a small part already as edit command working, and my solution for mass-marking with single tag was a watson log with the required time and project filters, and the output is piped to grep " to " | awk {' print $1 '} | xargs -n1 watson edit --add-tag/--remove-tag because the format could be easily matched by the "from-to" lines starting the frame ids. But the next step would be to do the same matching if the batch switch enables the same filtering as it is available in log command

Hi, since there are no such commands yet and I need a remove a tag, I would like to ask:
Could I simply do search&replace for the tag name in the file ~/.config/watson/frames, given
I pay attention only to replace the correct fields?
Or could anything break because of this because the data is also stored/cached elsewhere?

@mcrot in fact there are edit swutches for adding and removing tags and for log and report there is an ignore-tag switch in use by me since some time.

@mcrot somehow i did not push te commit button, but right after you commented, i replied.
The edit command incorporates switches to add and remove a tag, also without specify a tag, the "billed" tag can quickly added or removed, the log and report commands include an ignore-tag switch, this fork is used by me for quite lot of time. The complete cleanup of a specific tag from all projects could be done by a log command grepped for that tag, and using all frame-id-s lopped an edit commend to remove that tag.
But you already seen it: this frames file is kind of a key-less array of values in a fixed order, as such you can edit it, as long as it follows the rules of a json array

@gnanet Thanks! I've seen your branches, great, that it works. Have you thought about creating a pull request to get it into the original project?
Until then I'm going with just editing the frames file, if I need to. I only had one tag which I didn't need any more, and removing it was simple this way.

@gnanet Thanks! I've seen your branches, great, that it works. Have you thought about creating a pull request to get it into the original project? Until then I'm going with just editing the frames file, if I need to. I only had one tag which I didn't need any more, and removing it was simple this way.

It would require to resync the base code and reapply my changes to have them properly PR ready, and some of my changes are not QA fit for the projext.


Remove obsolete files to build raw VM docker instances

We could ignore the test failures for this PR. As this one is not merging from a forked branch, the build_dashboard will run on this PR, which is sure to fail. (There is some cloud_sql_proxy issue in there, but as we are going to remove the build dashboard workflow from github soon, I didn't fix it).


HOT_FIX: Change flag so plain RocksDB instance returned

This PR fixes the issue found in the soak testing cluster.  Once merged this will get cherry-picked to 2.2
I've added a test that fails without this fix
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @vvcephei @ableegoldman for reviews

cherry-picked to 2.2


Table cannot be created in non-default database, if the database is set as default in Initialization

System information:

Operating system (distribution) and version Win10
DBeaver version 7.3.5 EA

Connection specification:

Database name and version PostgreSQL

Steps to reproduce, if exist:

set a database as default in connection settings>Initialization
set another database as default in DB Navigator tree
try to create a table in the database (mentioned as default in connection settings>Initialization)

Expected: the table is created
Actual result: the cannot be created and error appears "schema doesn't exists"
Workaround: make the database default.

Cannot be reproduced anymore

It can be easily reproduced by using the steps you've provided above.

The problem is reproducible. More than that, after steps 1 and 2 navigator tree starts to lie.

verified


Removing Year from birthday in profile card

ahh, you don't have to pr on alpha, do it on master + remember the commit message convention plssss.


fix: change container port to 9200 for health check and remove args

Signed-off-by: ankitm123<EMAIL_ADDRESS>

/approve


UserWarning in ktk.Player

Describe the bug
UserWarning [/Users/felix/Documents/git/kineticstoolkit/kineticstoolkit/player.py:361] frames=None which we can infer the length of, did not pass an explicit save_count and passed cache_frame_data=True.  To avoid a possibly unbounded cache, frame data caching has been disabled. To suppress this warning either pass cache_frame_data=False or save_count=MAX_FRAMES.
To Reproduce
import kineticstoolkit.lab as ktk

markers = ktk.read_c3d(ktk.doc.download("kinematics_tennis_serve.c3d"))[
    "Points"
]

ktk.Player(markers)

Expected behavior
We shouldn't see this warning.
Python installation
conda-forge
Operating system

OS: macOS

Duplicate of #167


Build aarch64-unknown-linux-musl and add to Go package

Hi,
This builds on #1535 to add a build job for aarch64-unknown-linux-musl, which is added to the library artifacts as libpolar-Linux-arm.a. I've only propagated this to the Go package as it's relevant to my use-case.
Unfortunately, it's not quite there yet – the built artifact works fine if I use it in an alpine-based container on an m1 mac, but when running it in an ubuntu-based container on an m1 mac I get spooky FFI errors that I'm unsure how to debug: https://github.com/connec/oso/runs/5478560216?check_suite_focus=true
I suspect I'm missing some particular incantation of RUSTFLAGS and/or cgo... The errors typically seem to relate to malloc, so I'm guessing it's perhaps something like it's using glibc's in Go and musl's in Rust, or something like that? I'm not sure why this one struggles when the x86 MUSL version seems to "just work" on ubuntu-latest. Perhaps someone with more FFI experience will spot something obvious.
⚠ Depends on #1535.

Commits: (excluding the first 2 since they're part of #1535)

4db2c3ce Make build cache depend on target rather than OS
This should lead to less churn in the cached artifacts.

ebc26d9f Add a static library build for ARM Linux (musl)

b7fa218a Fix path creation in build_go
Since they already existed this went unnoticed.

5de3c5a4 Prepare Go package for Linux ARM
This introduces the architecture indirection for AMD64, in the same
style as aleady exists for MacOS.

b1cc8082 Add Linux ARM64 support to Go package
This requires a Linux ARM64 runner to validate the package, which can
most easily be achieved by running a container on an m1 mac.

For reference, I've been testing the build with a runner in an ubuntu container on an m1 mac, built with the following Dockerfile.

Dockerfile

FROM ubuntu:latest

RUN apt-get update && apt-get install -y \
  build-essential \
  curl \
  libdigest-sha-perl \
  && rm -rf /var/lib/apt/lists/* \
  && curl -o go1.17.8.linux-arm64.tar.gz -L https://go.dev/dl/go1.17.8.linux-arm64.tar.gz \
  && tar -C /usr/local -xzf go1.17.8.linux-arm64.tar.gz

RUN groupadd -r runner && useradd --no-log-init -r -m -g runner runner

USER runner
WORKDIR /actions-runner
RUN \
  curl -o actions-runner-linux-arm64-2.288.1.tar.gz -L https://github.com/actions/runner/releases/download/v2.288.1/actions-runner-linux-arm64-2.288.1.tar.gz \
  && echo "9fbb98fc64f57d48e86b641bd3dd54d870123afc32886b5ca1c33bdc10afb6ee  actions-runner-linux-arm64-2.288.1.tar.gz" | shasum -a 256 -c \
  && tar xzf ./actions-runner-linux-arm64-2.288.1.tar.gz \
  && /usr/local/go/bin/go install golang.org/dl/go1.16@latest && ~/go/bin/go1.16 download \
  && /usr/local/go/bin/go install golang.org/dl/go1.17@latest && ~/go/bin/go1.17 download

USER root
RUN DEBIAN_FRONTEND=noninteractive ./bin/installdependencies.sh

USER runner
RUN echo './config.sh --url "https://github.com/$RUNNER_REPO" --token "$RUNNER_TOKEN" --name "$RUNNER_NAME" && ./run.sh' > config-and-run.sh

ENV RUNNER_REPO=osohq/oso
CMD ["/bin/bash", "config-and-run.sh"]

You can run the built image with, e.g. docker run --env RUNNER_TOKEN='...' --env RUNNER_NAME=whatever.

hey @connec
thank you for this and #1535 and also for your patience.
RE: changing the build workflow to use a matrix. We actually used this pattern previously and found that the performance was poorer than expected. While in theory breaking out each job should allow us to chew through them much faster, in practice grouping by target as we have done here ends up being quicker. We would definitely love to add aarch64 linux targets to the mix, and I think the changes here could be easily appended to the build_linux_libs  task
RE: shipping this Go aarch64. We very much hope to support aarch64, the primary issue is that we currently lack an ARM64 test environment to validate these builds properly. We have some plans to establish some ARM64 linux build and test action environments, but until then we've opted to hold back on this work. We're conscious that with growing popularity and availability of ARM targets that we'll need to revisit this in the near future, however I unfortunately can't offer a strict timeline on when this will be shipped as we are currently heads down as a team on a complimentary product sprint.

No problem. I'll close this then as it's not suitable in its current form, and blocked as you say.


Generate MountReals for standalone AdditionalSigns

Generate MountReals also for standalone AdditionalSigns.
Refs LIIK-161

Codecov Report

Merging #110 into master will decrease coverage by 0.07%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
- Coverage   97.54%   97.46%   -0.08%
==========================================
  Files          96       96
  Lines        5936     5810     -126
  Branches      232      233       +1
==========================================
- Hits         5790     5663     -127
  Misses        129      129
- Partials       17       18       +1

Impacted Files
Coverage Δ

...management/commands/generate_mount_real_objects.py
96.55% <90.00%> (-3.45%)
:arrow_down:

azure_client_ip/middleware.py
68.75% <0.00%> (-1.84%)
:arrow_down:

traffic_control/views/_common.py
93.33% <0.00%> (-0.22%)
:arrow_down:

traffic_control/models/traffic_sign.py
98.72% <0.00%> (-0.01%)
:arrow_down:

traffic_control/models/mount.py
99.17% <0.00%> (-0.01%)
:arrow_down:

traffic_control/views/mount.py
100.00% <0.00%> (ø)

traffic_control/views/barrier.py
100.00% <0.00%> (ø)

traffic_control/views/signpost.py
100.00% <0.00%> (ø)

azure_client_ip/test_middleware.py
100.00% <0.00%> (ø)

traffic_control/views/road_marking.py
100.00% <0.00%> (ø)

... and 23 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 7b5faf8...99e4d85. Read the comment docs.


feat: add ddl task service

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

Removes Create in the Router
Adds DDL task service

Checklist

[ ]  I have written the necessary comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

PTAL @MichaelScofield @fengjiachun


Blacklist with exceptions, possible?

1 . Some sites use /pc_search and /search in the search result. The redirects are undesirable so I put this in the blacklist.
\/(pc_)?search

However this will also ignore redirect links in Firefox addons site whose addons' names start with search. For example,
https://outgoing.prod.mozaws.net/v1/3949e3fa1f8684e62e12b49707af540aec3f7eef13e37789e3ab33c8ae3ab05a/https%3A//github.com/andreicristianpetcu/Search-and-Switch-Containers
https://outgoing.prod.mozaws.net/v1/409bf55345207cc65f09ea6142e4b4cd069323e76f635076bfc139da6d46de4d/https%3A//github.com/dessant/search-by-image
The format of redirect links which I still want to execute:
https://outgoing.prod.mozaws.net/vX/XXXXXXXXXX/https%3A// (domain name)/...
It would be great if I can specify something like this:
\/(pc_)?search | except at mozaws.net

2 . Is positive/negative lookbehind/lookahead supported?
Thank you.

\/(pc_)?search= :wink:
Do you have any sample URLs to share? (not the Mozilla ones, but the opposite)

https://www.google.com/search?q=
https://image.baidu.com/n/pc_search?queryImageUrl=https%3A%2F%2Funitedwithisrael.org%2Fwp-content%2Fuploads%2F2014%2F11%2Fhappy.jpg&fm=index&uptype=paste

How many search sites do you use? Why not just be specific and blacklist www.google.com/search? Otherwise, something like this: https://stackoverflow.com/questions/6851921/negative-lookahead-regular-expression could work?

Google is not just one of them. I found there are various sites which uses /search/. It seems to be quite common for search queries.
Thanks for the pointer. This regex works.
http(s)?:\/\/(?!outgoing\.prod\.mozaws\.net).*\/(pc_)?search

BTW you might consider putting this in the default list.
/searchbyimage


Removed contract trait.

No longer a logical link between Contract and Order.

Coverage decreased (-0.02%) to 98.969% when pulling 02bed6d2611b6f92d249396271ae5b8c8420e8db on remove-contract into d60e4fd7d9d46991f1befb217d6cb6f1ff1fcbeb on develop.


capact ti get may cause gateway to be oomkilled

Description
capact ti get may cause gateway to be oomkilled. We pull all TypeInstance informations from the API. After running few actions I've got almost 30MB of a raw yaml.
When this happens, user cannot see any data:
$ capact ti get
Error: while executing query to list TypeInstances: All attempts fail:
#1: decoding response: invalid character '<' looking for beginning of value

Ingress is returning
2021/07/29 13:04:56 >> headers: map[Accept:[application/json; charset=utf-8] Content-Type:[application/json; charset=utf-8]]
2021/07/29 13:05:00 <<
<title>502 Bad Gateway</title>

502 Bad Gateway

nginx

Because Gateway was killed.
There are few options to solve it:

Short term: Increase gateway resources so it's not oomkilled
Mid term: Change CLI to query only required data. Fot table view it's only few fields. Problem persists when yaml output is used
Long term: All above plus something like pagination so we don't download all

AC

Increase gateway resources so it's not oomkilled
Change CLI to query only required data. Fot table view it's only few fields. Problem persists when yaml output is used

It will be also good to think about some cache layer.

Testing
I run such test to verify if Gateway component is OOMKilled.

Create 8 TypeInstances with 1MB payload under spec.value via capact ti stress -r 8 -s 1 (see https://github.com/capactio/capact/pull/443).
Build CLI from the latest main (https://github.com/capactio/capact/commit/608679fdce7c449adf7aa82f0b67356fd2c29c25)
Execute capact ti get resultRoundTrip time: 31.028609749s Request size: 3536 Response byte size: 32799137

So I was able to fetch 32MB, I also was able to fetch 41MB after adding more TI. The problem was that I had to manually change our HTTP timeout as for 41MB it took 1m5s to download all this data.
Additional problem is that  by adding one TypeInstance with 1MB payload the response body on capact ti get is increased by 4MB as we duplicate the value multiple times for each latestResourceVersion, firstResourceVersion, resourceVersions, and resourceVersion(resourceVersion: 1). If there will be a new version then it will be duplicated 5times as the previousResourceVersion will be also populated.

Iterations:

From 14mb to 18mb
From 18mb to 22mb
From 22 to 25mb
From 25 to 29mb (first deadline, took more that ~30sec)
From 29 to 33mb
From 33 to 37mb (took ~60s)
From 37 to 41mb (took ~1m5s)

But the gateway was not OOMKilled even though  the memory was significantly over the limit for a few secs.

Memory
Currently, we have such settings:

gateway: (Change it to 320mb)
    resources:
      limits:
        cpu: 300m
        memory: 128Mi
      requests:
        cpu: 100m
        memory: 48Mi

hub-public:
    resources:
      limits:
        cpu: 400m
        memory: 512Mi
      requests:
        cpu: 200m
        memory: 128Mi

hub-local:
    resources:
      limits:
        cpu: 400m
        memory: 512Mi
      requests:
        cpu: 200m
        memory: 128Mi

We can update Gateway

There are a few options how we can handle

use single method to narrow-down request fields: https://github.com/capactio/capact/blob/608679fdce7c449adf7aa82f0b67356fd2c29c25/pkg/hub/client/public/iface_options.go#L21-L22
but it doesn't scale well, as we will need to have methods for all possible permutations.
use struct where all fields can be specified: https://github.com/shurcooL/graphql#simple-query. IMO it's hard to use
Allow passing raw query - it's hard to use and we can and up with a lot of duplication, it's also hard to validate
Use func options - this case quite verbose methods

Finally, I decided to borrow the idea of bitmasking:
        //  used e.g. by standard lib
	OpenFile(name, O_RDWR|O_CREATE|O_TRUNC, 0666)

	// our usage
	preset := TypeInstanceRootFields | TypeInstanceTypeRefFields | TypeInstanceUsedByIDField | TypeInstanceUsesIDField | TypeInstanceLatestResourceVersionField
	hubCli.ListTypeInstances(ctx, local.WithCustomFields(preset))

It has one more benefit, it's typed, so you cannot use other flags e.g. from Interface in Implementation query.
To make it more robust:

Add Templates, and modify struct to have info under which property it should be attached. So thinks like UsedByIDField and UsedByNameField are not exclusive but can be set together. Currently, we don’t have such problem, but in the future it may be problematic.# Because both has `usedBy` they are exclusive, if we will use templating then `usedBy` will be
# map/struct and can be used together and `usedBy` will be populated both with id and name.
var UsedByIDField: `usedBy{
  id
}`
var UsedByNameField: `usedBy{
  name
}`

This will be good to implement too. We can use Go template for that.
Other things that I found

Investigate if we can transfer data in chunks
Currently we have inconsistency capcat hub interfaces get prints revisions. capact ti get prints type instances and typeinstances revisions. Maybe we should change CLI from ti get: to

ti get for TypeInstances and only revisionsVersions (int)
tir get for TypeInstanceRevisions - here you can as for first/latest/previous/given rev/all revs

capact TypeInstanceRevision get  (—resourceVersion=@all)
capact TypeInstanceRevision get —resourceVersion=@latest
capact TypeInstanceRevision get —resourceVersion=@first
capact TypeInstanceRevision get —resourceVersion=@previous
capact TypeInstanceRevision get —resourceVersion=13 = 13th revision
Or capact TypeInstanceRevision get 13/@latest/@first/@previous/@all

nit: sort table result e.g. by date.
Unify all client interfaces, add options to specify which fields should be fetched

As we have right now, by default ListInterfaces returns only “root fields” (all primitive fields - name, path, prefix ) other fields can be requested via options

ListInterfaces(ctx, WithLatestIfaceRevision())- to additionally get all field from LaterInterfaceRevision “complex” property.
or  ListInterfaces(ctx, WithLatestIfaceRevision(IfaceRevMetadataFields))  - to request only data fields from LaterInterfaceRevision  property.


JavaScript TDD Todo List App

A step-by-step complete beginner example/tutorial for building a Todo List App (TodoMVC) from scratch in JavaScript following Test Driven Development (TDD) best practice.
https://github.com/dwyl/javascript-todo-list-tutorial

Hi i am a fresher looking for some opensource contribution can u help me out i want to contribute to your project and can u assign me this issue:)

Hi i am a fresher looking for some opensource contribution can u help me out i want to contribute to your project and can u assign me this issue:)

Just make a PR.

Hi i am a fresher looking for some opensource contribution can u help me out i want to contribute to your project and can u assign me this issue:)

Just make a PR.

So basically i need to make a  repo like https://github.com/dwyl/javascript-todo-list-tutorial which has complete tutorial and explanation on how to make a todo list using JavaScript

Hi i am a fresher looking for some opensource contribution can u help me out i want to contribute to your project and can u assign me this issue:)

Just make a PR.

So basically i need to make a repo like https://github.com/dwyl/javascript-todo-list-tutorial this which has complete tutorial and explanation on how to make a todo list using JavaScript

https://github.com/firstcontributions/first-contributions

@enkeyz  i have opened a PR please check it


How to open popup

Hello all,
As soon as i drop my source to destination, i want to open pop up box or may be alert box then how i can do that, where to call alert or popup box function.
Please put  your suggetion.
Thanks!

It sounds like this demo does what you're asking: http://angular-ui-tree.github.io/angular-ui-tree/#/drop-confirmation.
Please reopen this issue or ask in chat if this doesn't solve your problem.