[TESTING] Tadukoo Annotation Processor, Tadukoo Lang, Tadukoo Util

What change would you like to see?
Proper JUnit testing for the following modules:

Tadukoo Annotation Processor
Tadukoo Lang
Tadukoo Util

How does this change help?
Provides testing coverage for the above listed modules, that isn't currently covered. This also makes sure we're ready for Alpha v.0.1
Additional context
Here's a TODO list for all the classes in the modules to keep track:
Tadukoo Annotation Processor:

[ ] base package

[ ] AbstractAnnotationProcessor
[ ] AnnotationProcessor
[ ] AnnotationProcessorProcessor
[ ] AnnotationUtil

Tadukoo Lang

[ ] base package

[ ] AutoCloseableUtil
[ ] BooleanUtil
[ ] ExceptionUtil
[ ] FileUtil
[ ] FloatUtil
[ ] ListUtil
[ ] LoggerUtil
[ ] StringUtil

[ ] tuple package

[ ] Pair
[ ] Triple

Tadukoo Util

[ ] annotation package

[ ] ShouldBeFinal
[ ] annotation.process package

[ ] ShouldBeFinalProcessor

[ ] event package

[ ] Event
[ ] EventHandler
[ ] EventListener

[ ] functional package

[ ] NoException
[ ] functional.consumer package

[ ] ThrowingConsumer
[ ] ThrowingConsumer2
[ ] ThrowingConsumer3
[ ] ThrowingConsumer4
[ ] ThrowingConsumer5

[ ] functional.function package

[ ] ThrowingFunction
[ ] ThrowingFunction2
[ ] ThrowingFunction3
[ ] ThrowingFunction4
[ ] ThrowingFunction5

[ ] functional.predicate package

[ ] ThrowingPredicate
[ ] ThrowingPredicate2
[ ] ThrowingPredicate3
[ ] ThrowingPredicate4
[ ] ThrowingPredicate5

[ ] functional.supplier package

[ ] ThrowingSupplier

[ ] map package

[ ] HashManyToManyMap
[ ] HashMultiMap
[ ] ManyToManyMap
[ ] MultiMap
[ ] TreeManyToManyMap
[ ] TreeMultiMap

Tadukoo Annotation Processor will not have any testing, as annotation processor testing is a pain.
It should, however, have manual testing/verification that the @AnnotationProcessor annotation works correctly. This is verified currently by the fact that the ShouldBeFinalProcessor appears in the proper META_INF file in Tadukoo Util's compiled jar.
Also when new utils are added to AnnotationUtil / used by annotation processors, they should be manually tested/verified as well.
Since this problem applies to both Annotations and Annotation Processors, any annotations in other locations will need this as well. @ShouldBeFinal from Tadukoo Util has previously been manually tested to work as expected currently. In the future, its functionality will change, but for now it's working as intended.
Whenever annotations or annotation processors receive changes, they should be re-tested, but they should be fine as long as they don't receive updates.

408 total JUnit tests across Tadukoo Lang and Tadukoo Util (there are some in Tadukoo File Format, but they're old and may change in the future and weren't part of this goal).
This is why you write JUnit tests ahead of time and not wait until the end like I did. The good news is it's done now.


Files starting with a dot do not show up

On macOS, when I open the the file selector (Cmd+Shift+H), files that start with a dot (ex. .env.test) do not show up. These files are not listed in  the relativePath.ignore config or in my .gitignore file.

I did this
- this._myGlob = new Glob(this._workspacePath + "/**/*.*",
+ this._myGlob = new Glob(this._workspacePath + "/**/*",

- { ignore: this._configuration.get("ignore") },
+ { dot: true, ignore: this._configuration.get("ignore") },

and got the hidden files to work, @jakob101 do you see a problem with this change? if not I can do a PR

Feel free to do this man! 👍 👍

Just a small comment: please make sure you do get back only files, not folders. But otherwise we're good to go!


Saladin and Emir's wells are no longer switched off whenever there isn't a fire

I noticed that Saladin and Emir's wells no longer switch off whenever there isn't a fire, like they did before.
Not sure if that is a good thing, which it will allow them to respond quicker to fires, it can lead to population shortages and less recruitment (especially together with the ale farm spam issue that has been mentioned before). Of course another house could be built in their AIVs but increasing the population may impact adversely on ale supply and fear factor.
It would good to have the option to adjust whether wells are kept on or off whenever there is no fire for an AI.

My AIV are all balanced around wells being constantly manned. I'd love to know how to do that permanently for all AI because it's super frustrating to plan you AIV around a proper fire defense just to see it burn to the ground because they didn't even man them until they are completely set ablaze.

I just want to know how did you manage to make them  do this "Switching off unused peasants"

I just want to know how did you manage to make them do this "Switching off unused peasants"

It is on 'no switched off buildings mode' (which I use to stop that annoying bug that sometimes happens where AI bakeries switch off while they are still in the process of making bread) but this switching off buildings mode was not supposed to include wells (it didn't at the start whenever I first used it).

Any AI will have their wells switched on by default (even in vanilla, no patch involved) whenever they have a bit of gold reserves (above ~500 gold or sth like that).
Whenever they have their wells switched off, that usually means they are simply out of gold (or just have everything set to sleep due to being in fear during attacks or similar), which causes them to go into some sort of "saving mode" - the AI then also is less likely (or even 0% likely^^) to restock any wood, flour, wheat, etc in the stockpile, causing even more economic disruptions as a consequence, as production comes to a grinding halt.
Anyway, wells set to sleep is a sign of constant gold shortage, nothing else.
Any AIV that has insufficient housing is just badly designed; wells/waterpots are a regular part of castle population. And besides the (weak) "it just is like that, deal with it" argument, there's also proper arguments for that behaviour:

If wells are manned when a fire starts, the well workers can react a lot quicker, extinguishing a fire while it is still small, before everything burns.
Especially due to worker loss often preceding or going hand in hand with fires, and AI trying to recruit nonetheless, manning before-sleeping wells when the fire is already burning does often not work out well, making things a lot worse.
Furthermore, always-on wells allow the AI to aid allied AIs with firefighting.
More population - if properly planned for - doesn't really affect the AI... more taxes, more gold, more awesome!

It is on 'no switched off buildings mode' (which I use to stop that annoying bug that sometimes happens where AI bakeries switch off while they are still in the process of making bread) but this switching off buildings mode was not supposed to include wells (it didn't at the start whenever I first used it).

The patcher setting to disable "switching off buildings" is supposed to apply to resource processing buildings that are always set to sleep by the AI when running out of the corresponding input resources (no matter whether bakeries or weapon workshops) and having 0 units of those resource types stockpiled for a fixed (short) amount of time.
Fear-based or (for wells) gold-based setting to sleep of buildings is an entirely different behaviour independent of that patcher setting.

My AIV are all balanced around wells being constantly manned. I'd love to know how to do that permanently for all AI because it's super frustrating to plan you AIV around a proper fire defense just to see it burn to the ground because they didn't even man them until they are completely set ablaze.

@Truetobi91 Possibly a better economy will allow your AIs to always (or at least usually) have a bit of spare gold which makes them have their wells manned permanently. Of course, under some circumstances, e.g. when constant recruiting of expensive sortie troops at high rate (popularity 100) and immediately sending them into their deaths nonstop, it can be nearly impossible to create an AI-economy strong enough to be fully "immune" to gold shortage in all cases.
Also, a proper fire defense does not rely on wells being manned all the time. Manned wells certainly help and are desirable whenever feasible, but a proper fire defense uses more than "you just have to place enough wells" to work. The amount of wells (as long as there's a few wells at all) is usually less important than 2 other factors:

Campfire not burning: This allows the AI to get new peasants that can be turned into well-workers, lowers losses of workers and soldiers, and generally benefits any burning AI unless there's 200 hostile troops trying to storm the keep right now... How to achieve this: Avoid having flamable material near (in a radius of ideally 7, but at least 4-5 tiles to be effective) your campfire! Non-flamable material includes things like moat, walls, towers, gates, stockpiles, dungeons, gardens, statues, shrines, wells, water pots, empty space, killing pits, stairs (including stair5 tiles which are accessible from the ground and can be used to create functional streets if one wants to)... generally speaking, if fire from the outside cannot spread onto the campfire, you are almost golden!
Not everything else burning: This allows the AI to use their well-workers to extinguish a fire while a good chunk of their castle economy is still up and running, and especially to fight a fire that has a limited area. Larger fires are generally more dangerous/negative/whatever than small ones, obviously, but smaller fires are also considerably easier to extinguish even with relatively few well workers.
How to achieve this:

Close this pls?

we have now a fire nerf option. If further improvemetns are needed, open a new issue


enable users to override the default api endpoint path

customize the API endpoint path by providing LitServer(..., endpoint_path="/api/CUSTOM_PATH").

@aniketmaurya @williamFalcon  hi
i am curious what thoughts are around standardization. it would be nice to change the default /predict or to add new endpoints (doesnt seem to be possible now but I could be wrong).  having something like /classify or /cluster or the like might be interesting for some users. having flexibility to change the default (via decorator) might be nice

hi @eugsokolov, may I know more about your use case, please? What can /classify solve that /predict is not able to? more context will be helpful to understand.

hi @eugsokolov, may I know more about your use case, please? What can /classify solve that /predict is not able to? Some context will be helpful.

/predict can do everything /classify can do, but sometimes a model may not want to provide a prediction but instead do something else. there might be a model that can predict and classify (and do something else) as well and it might be nice to have multiple endpoints all under one LitAPI

the word predict is being taken too literally here. predict just means “do what you want now with your model”. if you want to classify, segment, summarize, etc… just do it.
it’s just a word.
so, i’m still having a hard time understanding what the problem is that you are trying to solve?
“with predict i cannot do XYZ and thus we need ABC to enable the behavior i am looking for”.
without a clear motivation, this feels purely aesthetic…
are you asking that the endpoint /predict be customizable?
are you asking for the literal “predict” method to be named something else? if this is the case, why does the name matter?
are you asking for a server to provide multiple functionalities? ie: predict AND classify? and if so, is that really a best practice? do you have a concrete example where this is used in a real world application (not just “nice” in theory?) and if that’s further true, can’t a simple if statement handle that logic for you in the predict step?
we aim to provide a standard that allows servers to interoperate and be legible by anyone. if every litserve implementation uses a different name it loses the ability to be standardized… ie: right now if you want to know what a server is doing, then simply look at the predict method. if we remove this standard, it will be hard to pinpoint exactly what a server is doing to someone not familiar with a code base.
not saying no to the request, just asking to understand the underlying problem to understand what a reasonable solution might be.

yes this is aesthetic - if i am exposing an endpoint to the public i may not want it to be called /predict if it isnt providing any sort of prediction. i have a model that is classifying geospatial data and not doing any predictions but i am forced to use the /predict endpoint (openai doesnt even have a predict endpoint, just completion and embedding). in similar vein, i name my variables to something that resembles the data they are storing.
i dont feel very strongly here and thought it would be nice to name the endpoint via a decorator instead of always needing to use the default name
as far as best practice to provide multiple endpoints, maybe i want to provide a debug or data endpoint for my users. if there isnt much flexibility here then ill create a fastapi app to provide the relevant endpoints for my users

ok. then yes, in this case it makes sense to enable
(api_path="/api/CUSTOM_PATH")

wdyt @aniketmaurya @lantiga

ok. then yes.
To summarize why i think this makes sense:
An endpoint could be made public to people who didn't create the endpoint and thus the path name should match the functionality to reduce confusion.
in this case it makes sense to enable something similar to what @aniketmaurya proposed in the beginning here
# default
LitServer(..., api_path="predict")

# overwrite it (for example to be called /classify)
LitServer(..., api_path="classify")

but the method name still remains "predict" in the server API.
wdyt @aniketmaurya @lantiga

awesome!
i'll also add that a partial requirement of "renaming" would be to add new endpoints (which of course would require a different name) and hence it may make sense to force the developer to name the endpoint. this seems like a paradigm shift from the current implementation. but i do believe that taking an example from fastapi would be useful here in allowing for developers the flexibility.
the default example can include /predict. what i am envisioning would look something like:
# server.py
import litserve as ls

# STEP 1: DEFINE YOUR MODEL API
class SimpleLitAPI(ls.LitAPI):
    def setup(self, device):
        # Setup the model so it can be called in `predict`
        self.model = lambda x: x**2

    def decode_request(self, request):
        # Convert the request payload to your model input.
        return request["input"]

    @ls.endpoint("/predict")
    def predict(self, x):
        # Run the model on the input and return the output.
        return self.model(x)

    @ls.endpoint("/classify")
    def classify(self, x):
        return self.model.classify(x)

    @ls.endpoint("/model-weights")
    def model_weights(self, x):
        return self.model.weights

    def encode_response(self, output):
        # Convert the model output to a response payload.
        return {"output": output}

# STEP 2: START THE SERVER
if __name__ == "__main__":
    api = SimpleLitAPI()
    server = ls.LitServer(api, accelerator="auto")
    server.run(port=8000)

hi @eugsokolov, thanks for all the feedback. For manually specifying endpoint, the following looks ideal to me.
# default
LitServer(..., api_path="predict")

# overwrite it (for example to be called /classify)
LitServer(..., api_path="classify")

@eugsokolov for multiple endpoints, I would suggest to use LitSpec (note that it is not 100% documented yet), we use it to provide OpenAI compatible API endpoints here.

yeah not sure about multiple endpoints on the same server. because now you need a different encode and decode function for each as well.
i don’t like building “nice to haves” for “theoretical scenarios”.  do you have a concrete, real-world example where you must have multiple methods?
i totally get it for regular web development… but i don’t see how that applies here and i’m not even sure we would even want to encourage such a thing. i can see that being fraught with problems like out of memory, collisions, etc….

Interestingly, we already have the concept of endpoint with their own path (and method) behind the scenes, which we currently use to expose /v1/chat/completion for the OpenAI spec.
The only question is, do we really need to expose it to users? Flexibility comes with cognitive overhead so we need to be super conservative. In the future, as practical needs arise, we can expose the mechanism for advanced users, but probably not before we have a real need.
For now @aniketmaurya 's solution seems to check all boxes so it's a +1 for me.


eva02 fp16 not working

The problems are the lines keras_cv_attention_models/beit/beit.py:88-89
if there we change float32 to float16 it starts working with fp16

Are you using with keras.mixed_precision.Policy("float16")? I've actually awared of this issue, but only expecting usage with keras.mixed_precision.Policy("mixed_float16")... Anyway, it should be fixed now.

Using with keras.mixed_precision.Policy("mixed_float16")

How is it after previous fix? I've replaced all those dtypes with self.compute_dtype. Is the issue still exists?

Closing, it should work now.


put functions in alphabetical order

I asked around (during one of our clean coding sessions) what people think one should do regarding the ordering of methods/functions. In the Java world at least, the 'right' way of doing it is to order them depending on what level a fucntion is on in the callgraph (I find that confusing BTW). This leaves me unsure how to resolve this issue. Regardless, I still think it would be a good idea to have some kind of ordering in the function names. I leave it up to you wether to pursue this or not.

For files that contain more than just a couple of functions it makes sense to have at least some ordering, alphabetically probably makes the most sense. I've sorted the 20 functions in util.py and the 11 functions in core.py alphabetically. See commit c4d20486219416f316fe85ba750549b655368b58.


Support Scheduling Priority and Preemption

What would you like to be added:
Support Scheduling Priority and Preemption
Why is this needed:
To further improve the packing rate and resource utilization of the cluster, we usually divide tasks into high-priority tasks and low-priority tasks, and deploy them in a mixed manner. When the cluster is idle, low-priority tasks can make full use of cluster resources, while high-priority tasks can preempt low-priority task resources to meet the needs of high-priority tasks. In a single Kubernetes cluster, the community provides the pod priority feature to implement the scheduling priority of pods, and also implements preemptive scheduling of pods in the kube-scheduler based on priority.
When we move the application deployment process to a multi-cluster level, such scenarios still exist. However, in the current Clusternet multi-cluster application management mechanism, no similar capabilities are provided.
The entire requirement can be divided into two parts: "scheduling priority" and "preemptive scheduling".
"Scheduling Priority" implementation:
Add a scheduling priority field in the subscription
Add a priority queue in the scheduler. After the scheduler listens to the subscription changes, it puts them into the priority queue and dequeues them according to priority.
"preemptive scheduling"
There are two implementation ideas for "preemptive scheduling":
Idea 1: Implement subscription preemption;

step1: Reuse the "scheduling priority" implementation;
step2: Extend the preemption logic based on the existing clusternet-scheduler provided scheduler framework;

Idea 2: Rely on single-cluster preemption logic;

step1: Enhance the clusternet-scheduler's resource prediction process's perception of pod priority, It is necessary to supplement pod priority as scheduling information in feedinventory;
step2: Enhance the cross-cluster rescheduling capability of low-priority pods;

btw: I prefer idea 1

@abstractmj Nice proposal. This is exactly what we've planned to do.

For scheduling priority implementation, the design looks good to me. I'd suggest splitting them into smaller tasks, such as adding api, priority queue, etc.

For preemptive scheduling, I agree with you on the first design.
While the second one looks more like workload re-balancing. Since v0.16.0, Clusternet has introduced a new feature gate FailOver to help migrate workloads from not-ready clusters to healthy spare clusters. And I think we could make an enhancement on top of FailOver feature to support such workload re-balancing.

Thank you for your suggestion.
As you mentioned, the second design indeed seems more like a rebalance process. I haven't learned much about FailOver before, so I will study it to see if it can replace the descheduler logic I implemented previously.
Following the idea of the first design, the priority scheduling is relatively simple. I think it can be divided into the following two tasks: (1) add a priority field in the subscription, and (2) add a priority queue in the scheduler. On this basis, we can further extend the priority queue and scheduler plugin to implement preemptive scheduling logic. If possible, I can provide a more detailed design of the preemptive scheduling logic.

@abstractmj Looks good to me. Please go ahead.

Tracking PRs:

[x] #765


concurrent tasks. Failed MultipartUploadError(error: HTTPClientError.readTimeout, completedParts: [])

Hi,
xcode Version 12.5.1 (12E507) device ios 14.7.1. AWS S3 eu-west-2
 I have an issue loading files. I have a mix of large files (mp4) and small files (webp thumbnails). When I load then with

maxConcurrentTasks: Int = 2 or greater I get the http timout error. With a maxConcurrentTasks: Int = 1 no error occurs.
One of the larger files seems to stall not always the same one.
I think it must be an issue with the task scheduler, as timeout happens before 2 min.
I tried running the operation twice in parallel  with maxConcurrentTasks: Int = 1 and that succeeds. So does not seem to be an issue really of timeout.

It could still be a timeout issue, in that if you send multiple HTTP requests at the same time you slow them all down. The idea behind the task scheduler was to only allow so many HTTP requests running at one time to avoid this problem. If you run with the logging middleware it will print out when requests are sent and responses received. This might give us an idea what is going on. Warning this will produce a lot of output
let transferManager = S3FileTransferManager(
    s3: s3.with(middlewares: [AWSLoggingMiddleware()], timeout: .minutes(2)),
    threadPoolProvider: .createNew,
    logger: Logger(label: "S3TransferTests")
)

When adding your logger I found I had added the timeout to the wrong function. So with your original suggestion it works correctly after all!
Thanks again.


Version : 1.8.8 Spigot
Jobs Version : 4.9.3v

STRIPPED material does not exist on versions 1.8, only versions higher than 1.9. You ignoring it or remove it from the the job.

You'll need to either remove those from the jobConfig file or upgrade to 1.13.2.  They are invalid for 1.8.

ty <3 but should make code ignore this block if u running in 1.8.8

No. We can not ignore it because the user uses version 1.8 and has a version 1.13 material in the file and there is no bug display on the console, so understand it works. Therefore, it is not to be ignored.

ok sorry u can remove this ticket if u want <3


check permission with multiple groups

Codecov Report

Merging #1205 (6f65d59) into main (6d46906) will decrease coverage by 0.00%.
The diff coverage is 60.00%.

:exclamation: Current head 6f65d59 differs from pull request most recent head bcf5d3c. Consider uploading reports for the commit bcf5d3c to get more accurate results

@@            Coverage Diff             @@
##             main    #1205      +/-   ##
==========================================
- Coverage   57.47%   57.46%   -0.01%
==========================================
  Files         127      127
  Lines       21356    21359       +3
==========================================
+ Hits        12274    12275       +1
- Misses       7583     7584       +1
- Partials     1499     1500       +1

Impacted Files
Coverage Δ

pkg/meta/utils.go
73.17% <50.00%> (+0.16%)
:arrow_up:

pkg/meta/base.go
74.16% <100.00%> (ø)

pkg/meta/sql.go
67.58% <0.00%> (-0.77%)
:arrow_down:

cmd/gateway.go
29.77% <0.00%> (-0.34%)
:arrow_down:

pkg/meta/tkv.go
76.33% <0.00%> (+0.74%)
:arrow_up:

pkg/meta/context.go
52.00% <0.00%> (+8.00%)
:arrow_up:

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 6d46906...bcf5d3c. Read the comment docs.


fix(connector-socketio): use common package convention

Describe the bug
All packages should follow the same conventions to avoid problems in the future. Our socketio connector should be fixed to achieve that. Use common package.json and tsconfig.json
@petermetz comments:

I recommend keeping the conventions of the other packages when it comes to declaring the paths & configuring the compiler (tsconfig.json file). That will avoid a big chunk of the issues like when some packages do not get published due to strange bugs: https://github.com/hyperledger/cactus/issues/2069 or other kinds of automation scripts failing if they make assumptions that some of these paths are uniform everywhere. If there is a specific reason for not going uniform with the package configuration properties then that's a different question of course but if it's all the same to you then I'd highly recommend changing it to match the rest of the packages.

... in general every single property in the package.json file that does not have a specific reason (like contributor list or package name/description) to be different should be the same, uniform configuration (same for tsconfig.json files).

@petermetz Can i take this one?

@petermetz Can i take this one?

@rwat17 All yours, thank you very much in advance!


Indexing should retry failures when paginating through buckets

I had to run node worker.js multiple times in order to fully index the new oin-hotosm bucket.
Something (probably the function that paginates through S3, potentially due to unbounded concurrency) fails with:
{ Error: read ECONNRESET
    at exports._errnoException (util.js:1022:11)
    at TCP.onread (net.js:569:26) code: 'ECONNRESET', errno: 'ECONNRESET', syscall: 'read' }

@mojodna Is this still an issue or was this fixed in https://github.com/hotosm/oam-catalog/pull/93?

Retrying wasn't added, but this error is much less likely to occur with the reduced concurrency. For full resilience, it still probably should be.

I've got a funny feeling this is to do with the seemingly errant db.close() I asked @kamicut about in #93. If it is, then it's fixed in the unmerged #98.

@tombh no, this was happening before the rewrite in #93.

Thanks, so then we have to assume this isn't fixed.
I think the most direct and comprehensive solution here is to move over to using a formal queuing library, such as; hapi-job-queue, mongodb-queue or monq. They have already solved the problems we are facing here. This will also further separate our concerns and conform more closely to 12 Factor Apps. Unless there are arguments against this, I will make an issue for it.

See #102 for my proposed fix.


Create .gitattributes

This way the tests folder and etc do not end up being unnecessarily downloaded into the vendor folder when users install it on their projects.

Thank you for this! I was not aware of this neat feature.
Great work!


possible length of qasm code vs. number of registers created

The possible length of a program is shorter if you create less registers, for example this code gives an error (error code 400) if you create only 1 register, but works fine with 16:
from qiskit import QuantumProgram
import Qconfig

qp = QuantumProgram()

qp.set_api(Qconfig.APItoken, Qconfig.config['url'])

qr = qp.create_quantum_register('qr', 16) #if i create only 1, the program doesn't run
cr = qp.create_classical_register('cr', 16)

qc = qp.create_circuit('qc', [qr], [cr])

def pad_QId(circuit,N,qr):
    for ii in range(N):
        circuit.barrier(qr)
        circuit.iden(qr)
    return circuit

qc.x(qr[0])
qc = pad_QId(qc, 1000, qr[0])
qc.measure(qr[0], cr[0])

result = qp.execute('qc', timeout = 1000, backend = 'ibmqx5', shots = 8192, max_credits = 5)
result.get_counts('qc')

What could be causing this?

Hi @akosbudai, nice catch! I looked around a bit but couldn't find the cause of this bug. I am guessing that it is caused by the backends converting the json-format "quantum object" code into qasm before running. I am hoping that PR#326 will fix this, by executing from the json directly. But let's keep this issue open until after that PR is merged, and revisit.

@akosbudai actually I found the source of the problem. It will definitely be fixed with the introduction of qobj (quantum object code), where we won't send QASM to the backends anymore.
If you want to fix it now, call your registers "q" and "c" instead of "qr" and "cr". :-)

ha, that's a convenient fix! :D thanks

@ajavadia how is it possible the names of the registers would affect on this?

This should be fixed once we stop sending QASM as part of Qobj.

It seems to be related to the total length of the payload accepted by the backend. Keeping the names short make the final payload shorter and so, it works.

We need to check the implication to remove QASM from the QObj to reduce the size of the payload.


fix nereids_p0/update/update_unique_table.groovy

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run p0

run buildall

也需要合入2.0


extern EasyTab global for linking. (issue #5)

My github-foo is weak. I should have just opened the pull request. This is the fix for #5

Thanks. :)


add MUL

school website https://mul.edu.pk
faculty of computer science and information technology page: https://www.mul.edu.pk/english/faculty/computer-science-and-information-technology/
thank you!

@lazy-prime First stage of verification passed, official domain is mul.edu.pk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@lazy-prime Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Travis add chisel3

Added Verilator installation to the Travis cache
Added Chisel 3 checkout and "sbt test"
Removed brittle string comparison of compiled rocket.fir and rocket-golden.v
Actual FIRRTL tests will be added in another pull request.

I'm not 100% sure how the caching works, but I think each branch has it's own cache, but all Pull Requests share a cache. This means that Verilator will be rebuilt every time someone pushes to a new branch (which is kind of annoying).
Also, the chisel tests take a little while to run because I don't think Travis gives us a lot of parallel stuff, so perhaps we should figure out how to tag tests in Scala and only run the subset of Chisel3 tests that actually touch FIRRTL


logo is too big

On any reasonable screen it prevents someone from reading any part of the readme without scrolling down first.

Resolved by #296


Wrong address with multiple firefox windows

When having multiple Firefox windows open, the Plug-In wrongly prefills the address field with the address of the first opened window.
So as an example, I have two windows (not tabs):

first: wikipedia.org
second: myloginsite.net

I have the second window up and focused, but calling the Plug-In with F8, results in wikipedia.org filled into the address field.

Fix commited and uploaded.


Addition of Unscented Kalman Filter.

It seems there are some SpotBugs warnings related to this merge (see the Orekit SonarQube report).

Right! I missed them. I'll fix the warnings.

That's fixed!


Add all files from LuaSocket to make it usable in editor_scripts

Closing because it isn't possible to extend editor_script in this way


feat: add tagFilter setting

This PR introduces a new tagFilter setting that allows users to filter out tags from the version  checking logic. In our particular case, we have semver tags like v1.2.4 and then we have auto-generated tags named v1.2.4-chart. This was causing the SemVer action logic to fail:
Run ietf-tools/semver-action@v1
Comparing against latest tag: v1.2.4-chart
[PATCH] ...

>>> Will bump version v1.2.4-chart using PATCH

Current version is v1.2.4-chart
Next version is v1.2.4

With this new setting in place, I can set tagFilter: (.+)-(chart) and then it filters that tag out and we get the next version we expect:
Run diranged/semver-action@filter
Will filter out tags that match: (.+)-(chart)
Skipping tag "v1.2.4-chart" (matches tagFilter)
Comparing against latest tag: v1.2.4
[PATCH] ...

>>> Will bump version v1.2.4 using PATCH

Current version is v1.2.4
Next version is v1.2.5

@NGPixel,
What do you think of this change?


err_remount

whenever i try to run the jailbreak on a iphone 11 on ios 13.4.1 i get that issue

How many times have you tried? If you consistently get this, can you enable log view and send a log?

I am getting the same error. Logs show: "OTA update already mounted"

Ensure that the software update in settings > device storage is deleted first and try again.


TS: I cannot move a member to an existing file

I only see an option for moving to a new file:

Tracked upstream by https://github.com/microsoft/TypeScript/issues/29988


HIVE-28598:  Fix MurmurHashExpression to handle isRepeating properly

What changes were proposed in this pull request?

MurmurHashExpressions now use the first item for every row if isRepeating is true.
Why are the changes needed?

In current implementation, MurmurHashExpression does not check isRepeating and always reads whole rows. So if one of ColumnVector has isRepeating == true, then MurmurHashExpression reads the second row and then throws NullPointerException.
Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

No
How was this patch tested?

Tested using the input data provided on HIVE-28598 JIRA page, and also by TestMiniLlapLocalCliDriver.
However, I failed to reproduce the issue with a file smaller than 11MB. I'll try to reproduce this issue with a smaller input file and update the qfile.

@okumin @ngsg @deniskuzZ @zhangbutao @zabetak
Hello.
Surprisingly, this patch works well. All NPEs are gone, and the correct results are returned. Thank you for your work!@ngsg
By the way, I'd like to understand why using hash(varchar, varchar) does not trigger an NPE when I'm not using JOIN?
Is there actually something wrong with vectorReader?

@okumin @ngsg @deniskuzZ @zhangbutao @zabetak
Hello.
Surprisingly, this patch works well. All NPEs are gone, and the correct results are returned. Thank you for your work!@ngsg
By the way, I'd like to understand why using hash(varchar, varchar) does not trigger an NPE when I'm not using JOIN?

LGTM, +1 on adding a test for repeating values, see TestMurmurHashExpression

@zabetak @deniskuzZ
I have added a qfile and unit tests for repeating values. Could you please review the added tests? Thank you.

@zabetak
The null values in BytesColumnVector come from ORC code.
In the ORC code, ORC reader reset only vector[]. ORC code1
And if the new ColumnVector has isRepeating=true, the reader updates the first item only. ORC code2
So the previously used start[] and length[] are not updated nor reset.
In my opinion, these null values are not a problem because it seems that isRepeating=true indicates that only vector[0] is valid and the other entries should be ignored.
https://github.com/apache/hive/blob/afe05b968026dfdda631de1e2b090665f0820ef3/storage-api/src/java/org/apache/hadoop/hive/ql/exec/vector/ColumnVector.java#L71-L75

@BsoBird
I don't know the exact answer to your question, but in my experience, the NPE issue did not occur in the following situation:

used hash() instead of murmur_hash(). Both of them appear as hash in query plan, but only murmur_hash causes the NPE issue.
hive.llap.io.enabled=true & small inputs. I don't know the exact reason, but I have to disable LLAP IO to reproduce this issue using qtests with a small file.
ORC file using direct encoding. If a string column uses direct encoding, then ORC reader always set isRepeating to false.
length[i] == 0 for all 1<=i<=1023. Then Murmur3.hash32 does not access vector[][] and NPE does not occur.


Update package details

addresses https://github.com/Azure/azure-functions-rabbitmq-extension/issues/54

@fabiocav  / @paulbatum  - Here are the predefined license expressions https://spdx.org/licenses/
Do you know if we should use 'MIT or 'MS-PL ?
Note: Existing extensions use licence URL  which is now deprecated

It should be MIT. We need to match all the details we have on other packages (i.e. https://github.com/Azure/azure-webjobs-sdk-extensions/blob/6085a909c2aab7d4ad20cfee9a182f2b0aa10cb3/build/common.props#L16-L26)

merging this in. Please prepare the release!


The  underscore (_) special character in Header names is currently supported, and the limitations should be updated.

[Enter feedback here]
The Limitations part of the docs should be updated since the (_) is being supported.

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

ID: aedb5a9b-70bd-506d-41c7-2ede4e12e116
Version Independent ID: b55a0075-80ff-a967-3d9d-8429e7663b8c
Content: Rewrite HTTP headers with Azure Application Gateway
Content Source: articles/application-gateway/rewrite-http-headers.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: absha

@tcsougan Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hi @tcsougan , the limitation still stands. Application Gateway v2 does allow underscores in header values, but not header names. Have you seen otherwise? If so, can you provide more logs/information?

#please-close

#please-close


Save built-in script

Create a scene with buildin script in it.
Save
Open buildin script
Fix
Save
Close godot
Open godot
Opened "fixed" script

Changes were not been saved

Is this issue still valid in the current master branch?

this has been finally fixed :)


serialization_spec.rb fails with Ruby1.9.3 if yaml loaded after rspec/mocks

Hi!
When trying to run the specs with Ruby1.9.3, the following test is failing:
1) RSpec::Mocks::Serialization with YAML loaded using Psych as the YAML engine behaves like normal YAML serialization serializes to yaml the same with and without stubbing, using YAML.dump
     Failure/Error: Unable to find matching line from backtrace
       result should not have changed, but did change from "--- !ruby/struct:RSpec::Mocks::SerializableStruct\nfoo: 7\nbar: something\n" to "--- &9513520 !ruby/struct:RSpec::Mocks::SerializableStruct\nfoo: 7\nbar: something\nmock_proxy: !ruby/object:RSpec::Mocks::Proxy\n  object: *9513520\n  name: \n  error_generator: &9510280 !ruby/object:RSpec::Mocks::ErrorGenerator\n    declared_as: Mock\n    target: *9513520\n    name: \n    opts: {}\n  expectation_ordering: &9510220 !ruby/object:RSpec::Mocks::OrderGroup\n    error_generator: *9510280\n    ordering: []\n  messages_received: []\n  options: {}\n  already_proxied_respond_to: false\n  null_object: false\n  method_double:\n    :bazz: !ruby/hash:RSpec::Mocks::MethodDouble\n      :expectations: []\n      :stubs:\n      - !ruby/object:RSpec::Mocks::MessageExpectation\n        error_generator: *9510280\n        expected_from: /home/boutil/debian/packages/ruby/ruby-rspec-mocks/ruby-rspec-mocks/lib/rspec/mocks/methods.rb:33:in\n          `block in stub'\n        sym: :bazz\n        method_block: \n        return_block: !ruby/object:Proc {}\n        actual_received_count: 0\n        expected_received_count: :any\n        args_expectation: !ruby/object:RSpec::Mocks::ArgumentExpectation\n          args:\n          - !ruby/object:RSpec::Mocks::ArgumentMatchers::AnyArgsMatcher {}\n          block: \n          match_any_args: true\n          matchers: \n        consecutive: false\n        exception_to_raise: \n        args_to_throw: []\n        order_group: *9510220\n        at_least: \n        at_most: \n        exactly: \n        args_to_yield: []\n        failed_fast: \n        args_to_yield_were_cloned: false\n        eval_context: \n"
     Shared Example Group: "normal YAML serialization" called from
     # ./spec/rspec/mocks/serialization_spec.rb:62:in `block (3 levels) in <module:Mocks>'
     # ./debian/ruby-tests.rb:4:in `<main>'

It turns out that, the way the spec file is written, rspec/mocks is required before yaml, thus rspec/mocks/extensions/psych.rb is not loaded because ::Psych is not (yet) defined, and thus, dump does not drop mock_proxy.
Exchanging requirements on spec_helper and yaml solves the problem when running this unique spec file, but not when running several because rspec/mocks/serialization.rb could have been loaded before.
Would it make sense to move "require yaml" from spec/rspec/mocks/serialization_spec.rb to lib/rspec/mocks/serialization.rb?
Thanks!
Cédric

I'm not able to reproduce this with 1.9.3 using this command: bin/rspec spec/rspec/mocks/serialization_spec.rb:61
What is the exact command you are using?

We should not need to explicitly require yaml in serialization.rb. The point at which you're running the specs is such an edge case it does not constitute a "fix". Just stick with requiring yaml before you invoke the runner. Thanks.

I just removed requiring yaml.
When you "require 'yaml'", it makes Psych available:
➜  ~  irb
1.9.3p0 :001 > Psych
NameError: uninitialized constant Psych
from (irb):1
from /Users/justinko/.rvm/rubies/ruby-1.9.3-p0/bin/irb:16:in `'
1.9.3p0 :002 > require 'yaml'
=> true
1.9.3p0 :003 > Psych
=> Psych
Note that in Ruby 1.9, YAML will only use Psych if libyaml is installed.
A dependency is most likely requiring yaml before the spec is run, which is why Psych is available.

Thanks for your comments and explanations. I regret though that the test suite cannot be run readily from the command line (I've got now 5 failing tests for Serialization).

You did bundle install --binstubs then bin/rspec?


BLEManagerImpl::SoftDeviceBLEEventCallback in nRF5 BLEManager can overflow size computation

Problem
The code looks like this:
        const uint16_t valLen     = bleEvent->evt.gatts_evt.params.write.len;
        const uint16_t minBufSize = CHIP_SYSTEM_PACKETBUFFER_HEADER_SIZE + valLen;

There is no check that the result of the addition doesn't overflow uint16_t.
Proposed Solution
Add a check.  I'm not sure whether it should emit kCHIPoBLEOutOfBuffersEvent or do something else....
@pan-apple do you know who generally deals with this code?

Obsolete — this nRF5 code is no longer in tree


add doc note about equal sign in rewrite rule's pattern

Resolves https://github.com/graphite-project/carbon/issues/620

Codecov Report

Merging #2339 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2339   +/-   ##
=======================================
  Coverage   79.55%   79.55%
=======================================
  Files          85       85
  Lines        8863     8863
  Branches     1899     1899
=======================================
  Hits         7051     7051
  Misses       1551     1551
  Partials      261      261

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 87240bf...4b6ce14. Read the comment docs.


Tag releases in git

Generally a good practice.

Good suggestion. Thanks. I've updated my release script to automatically add and push tags and have tagged all versions back to 0.2.0.


[Codelab Issue] Testing Codelab 5.1.13, Step N/A - Wrong hyperlink

Describe the problem

Official Testing Sample
Hyperlink should take the user to https://github.com/android/architecture-samples
In which lesson and step of the codelab can this issue be found?
https://codelabs.developers.google.com/codelabs/advanced-android-kotlin-training-testing-basics/index.html?index=..%2F..index#12
Right at the top, under

Learn more

Samples:

Official Testing Sample - This is the official testing sample, which is based off of the same TO-DO Notes app used here. Concepts in this sample go beyond what is covered in the three testing codelabs.

codelab: advanced-android-kotlin

Thanks, fixed in next release.


Deploy kubernetes via ACS-engine on China Azure failed.

Is this a request for help?:

Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE
What version of    ###acs-engine?:
F:\Repos\acs-engine\_output\kevinstagingmaster>acs-engine version
Version: v0.17.1
GitCommit: c8812e9
GitTreeState: clean

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
The deployment was failed with the below error from cli
Deployment failed. Correlation ID: 77e4d1b3-7882-45d8-923e-e5f187ea62cc. {
  "status": "Failed",
  "error": {
    "code": "ResourceDeploymentFailure",
    "message": "The resource operation completed with terminal provisioning state 'Failed'.",
    "details": [
      {
        "code": "VMExtensionProvisioningError",
        "message": "VM has reported a failure when processing extension 'cse-agent-0'. Error message: \"Enable failed: failed to execute command: command terminated with exit status=4\"."
      }
    ]
  }
}

What you expected to happen:
Succesful
How to reproduce it (as minimally and precisely as possible):
Follow the steps (https://github.com/Azure/devops-sample-solution-for-azure-china/tree/master-dev/acs-engine) to generate the arm template and deploy to azure via azure cli
kubernetes.json:
{
  "apiVersion": "vlabs",
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes"
    },
    "masterProfile": {
      "count": 1,
      "dnsPrefix": "kevinstagingmaster",
      "vmSize": "Standard_D2_v2"
    },
    "agentPoolProfiles": [
      {
        "name": "agentpool1",
        "count": 2,
        "vmSize": "Standard_D2_v2",
        "availabilityProfile": "AvailabilitySet"
      }
    ],
    "linuxProfile": {
      "adminUsername": "azureuser",
      "ssh": {
        "publicKeys": [
          {
            "keyData": "xxxx"
          }
        ]
      }
    },
    "servicePrincipalProfile": {
      "clientId": "xxxx",
      "secret": "xxxx"
    }
  }
}

az group create --name "kzha_k8s_rg"  --location "chinanorth"
az group deployment create --name "deployment1" --resource-group "kezha_k8s_rg" --template-file "azuredeploy.json" --parameters "azuredeploy.parameters.json"

Anything else we need to know:
See the following logs on master node
azureuser@k8s-master-35463463-0:~$ vi /var/log/azure/cluster-provision.log

Failure logs

+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.
+ '[' 1 -eq 0 ']'
+ '[' 1 -eq 20 ']'
+ sleep 1
+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.
+ '[' 1 -eq 0 ']'
+ '[' 2 -eq 20 ']'
+ sleep 1
+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.
+ '[' 1 -eq 0 ']'
+ '[' 3 -eq 20 ']'
+ sleep 1
+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.
+ '[' 1 -eq 0 ']'
+ '[' 4 -eq 20 ']'
+ sleep 1
+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.
+ '[' 1 -eq 0 ']'
+ '[' 5 -eq 20 ']'
+ sleep 1
+ for i in '$(seq 1 $retries)'
+ timeout 10 systemctl daemon-reload
+ systemctl restart hyperkube-extract
Job for hyperkube-extract.service failed because the control process exited with error code. See "systemctl status hyperkube-extract.service" and "journalctl -xe" for details.

+ '[' 1 -eq 0 ']'
+ '[' 6 -eq 20 ']'

azureuser@k8s-master-35463463-0:/var/log/azure$ vi hyperkube-extract-status.log

perkube-extract.service - kubectl and kubelet extraction
   Loaded: loaded (/etc/systemd/system/hyperkube-extract.service; disabled; vendor preset: enabled)
   Active: activating (auto-restart) (Result: exit-code) since Wed 2018-05-23 02:53:16 UTC; 5ms ago
  Process: 10862 ExecStartPre=/usr/bin/docker pull k8s-gcrio.azureedge.net/hyperkube-amd64:v1.8.13 (code=exited, status=1/FAILURE)
  Process: 10851 ExecStartPre=/bin/mkdir -p /tmp/hyperkubedir (code=exited, status=0/SUCCESS)

May 23 02:53:16 k8s-master-35463463-0 systemd[1]: hyperkube-extract.service: Control process exited, code=exited status=1
May 23 02:53:16 k8s-master-35463463-0 systemd[1]: Failed to start kubectl and kubelet extraction.
May 23 02:53:16 k8s-master-35463463-0 systemd[1]: hyperkube-extract.service: Unit entered failed state.
May 23 02:53:16 k8s-master-35463463-0 systemd[1]: hyperkube-extract.service: Failed with result 'exit-code'.

#3024

cc @andyzhangx

@huajunzhao would you specify "location": "chinanorth", in kubernetes.json, and try again:
{
  "apiVersion": "vlabs",
  "location": "chinanorth",
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes"
    },
    "masterProfile": {
      "count": 1,
      "dnsPrefix": "kevinstagingmaster",
      "vmSize": "Standard_D2_v2"
    },
    "agentPoolProfiles": [
      {
        "name": "agentpool1",
        "count": 2,
        "vmSize": "Standard_D2_v2",
        "availabilityProfile": "AvailabilitySet"
      }
    ],
    "linuxProfile": {
      "adminUsername": "azureuser",
      "ssh": {
        "publicKeys": [
          {
            "keyData": "xxxx"
          }
        ]
      }
    },
    "servicePrincipalProfile": {
      "clientId": "xxxx",
      "secret": "xxxx"
    }
  }
}

@andyzhangx yes, this works.
I was also following https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/windows.md to deploy the Windows server as agent. All looks good but only the extension on master failed.

{
  "apiVersion": "vlabs",
  "location": "chinaeast",
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes"
    },
    "masterProfile": {
      "count": 1,
      "dnsPrefix": "xxxxxx",
      "vmSize": "Standard_D2_v2"
    },
    "agentPoolProfiles": [
      {
        "name": "windowspool2",
        "count": 2,
        "vmSize": "Standard_D2_v2",
        "availabilityProfile": "AvailabilitySet",
        "osType": "Windows"
      }
    ],
    "windowsProfile": {
      "adminUsername": "azureuser",
      "adminPassword": "xxxxx"
    },
    "linuxProfile": {
      "adminUsername": "azureuser",
      "ssh": {
        "publicKeys": [
          {
            "keyData": "xxxxxx"
          }
        ]
      }
    },
    "servicePrincipalProfile": {
      "clientId": "aefa3d81-cf47-4dc8-a4cb-785df9a0f53c",
      "secret": "0c543319-0570-4fc6-b426-95fc8345d4a8"
    }
  }
}

@huajunzhao the second error you pasted is different from the first. Exit code 9 means that we weren't able to install docker... Can you try running apt-get install apt-transport-https ca-certificates iptables iproute2 socat util-linux mount ebtables ethtool init-system-helpers on the master VM that failed (in the latest deployment)? Also check /var/log/azure/cluster-provision.log on the second deployment it should show the exact error.

I also tried to deploy a windows k8s cluster on chinacloud, it also failed, master node could not connect to azure.archive.ubuntu.com, I will file a ticket to china cloud support.
azureuser@k8s-master-53578836-0:/var/log/azure$ sudo apt-get install --no-install-recommends -y apt-transport-https ca-certificates iptables iproute2 socat util-linux mount ebtables ethtool init-system-helpers -y
Reading package lists... Done
Building dependency tree
Reading state information... Done
ethtool is already the newest version (1:4.5-1).
iptables is already the newest version (1.6.0-2ubuntu3).
apt-transport-https is already the newest version (1.2.26).
ca-certificates is already the newest version (20170717~16.04.1).
init-system-helpers is already the newest version (1.29ubuntu4).
iproute2 is already the newest version (4.3.0-1ubuntu<IP_ADDRESS>).
mount is already the newest version (2.27.1-6ubuntu3.4).
util-linux is already the newest version (2.27.1-6ubuntu3.4).
The following NEW packages will be installed:
  ebtables socat
0 upgraded, 2 newly installed, 0 to remove and 4 not upgraded.
Need to get 400 kB of archives.
After this operation, 1,299 kB of additional disk space will be used.
0% [Connecting to azure.archive.ubuntu.com (<IP_ADDRESS>)]
Err:1 http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 ebtables amd64 <IP_ADDRESS>-3.4ubuntu2
  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
Err:2 http://azure.archive.ubuntu.com/ubuntu xenial/universe amd64 socat amd64 <IP_ADDRESS>-1
  Unable to connect to azure.archive.ubuntu.com:http:
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/e/ebtables/ebtables_<IP_ADDRESS>-3.4ubuntu2_amd64.deb  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out

E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/universe/s/socat/socat_<IP_ADDRESS>-1_amd64.deb  Unable to connect to azure.archive.ubuntu.com:http:

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

azureuser@k8s-master-53578836-0:/var/log/azure$ sudo apt-get update
Get:1 http://security.ubuntu.com/ubuntu xenial-security InRelease [107 kB]
Get:2 http://security.ubuntu.com/ubuntu xenial-security/main Sources [123 kB]
Get:3 http://security.ubuntu.com/ubuntu xenial-security/universe Sources [65.4 kB]
Get:4 http://security.ubuntu.com/ubuntu xenial-security/main amd64 Packages [500 kB]
Get:5 http://security.ubuntu.com/ubuntu xenial-security/universe amd64 Packages [351 kB]
Err:6 http://azure.archive.ubuntu.com/ubuntu xenial InRelease
  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
Err:7 http://azure.archive.ubuntu.com/ubuntu xenial-updates InRelease
  Unable to connect to azure.archive.ubuntu.com:http:
Err:8 http://azure.archive.ubuntu.com/ubuntu xenial-backports InRelease
  Unable to connect to azure.archive.ubuntu.com:http:
Fetched 1,145 kB in 2min 0s (9,513 B/s)
Reading package lists... Done
W: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/dists/xenial/InRelease  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
W: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/dists/xenial-updates/InRelease  Unable to connect to azure.archive.ubuntu.com:http:
W: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/dists/xenial-backports/InRelease  Unable to connect to azure.archive.ubuntu.com:http:
W: Some index files failed to download. They have been ignored, or old ones used instead.

I also found that sometimes it's an intermittent network issue, is there any way to continue the existing deployment? @CecileRobertMichon

@huajunzhao you may redeploy now, the network issue of accessing azure.archive.ubuntu.com is solved.


Flaky tests with newer Stockfish 15.1

There's an issue with flaky tests on newer Stockfish versions. Using expected moves in testing will always open up for issues with different engines.
This is a larger issue that needs some consideration and planning I think. Tests should pass on all versions of Stockfish, both old and new.

Is it appropriate to test for specific moves at all in py-stockfish? If tests cover Stockfish engine settings and assures results are returned, is it appropriate to test for the actual moves? After all, py-stockfish does no evaluation.
If testing for moves, there will always be the problem of different engine versions returning different moves. It seems an attempt was made to deal with problem by testing a move against a list of possible correct moves — but this makes the test less precise and therefore less valuable.
An attempt has also been made to expect different test results for different engine versions. This is a possible solution, but quite work-intensive.

Case in point, on Stockfish 15.1:
______________________________________________ TestStockfish.test_set_fen_position_second_argument _______________________________________________

self = <tests.stockfish.test_models.TestStockfish object at 0x103625420>, stockfish = <stockfish.models.Stockfish object at 0x103079810>

    def test_set_fen_position_second_argument(self, stockfish):
        stockfish.set_depth(16)
        stockfish.set_fen_position(
            "rnbqk2r/pppp1ppp/3bpn2/8/3PP3/2N5/PPP2PPP/R1BQKBNR w KQkq - 0 1", True
        )
        assert stockfish.get_best_move() == "e4e5"

        stockfish.set_fen_position(
            "rnbqk2r/pppp1ppp/3bpn2/4P3/3P4/2N5/PPP2PPP/R1BQKBNR b KQkq - 0 1", False
        )
>       assert stockfish.get_best_move() == "d6e7"
E       AssertionError: assert 'd6b4' == 'd6e7'
E         - d6e7
E         + d6b4

tests/stockfish/test_models.py:129: AssertionError
_______________________________________________________ TestStockfish.test_set_skill_level _______________________________________________________

self = <tests.stockfish.test_models.TestStockfish object at 0x103626320>, stockfish = <stockfish.models.Stockfish object at 0x103667550>

    def test_set_skill_level(self, stockfish):
        stockfish.set_fen_position(
            "rnbqkbnr/ppp2ppp/3pp3/8/4P3/5N2/PPPP1PPP/RNBQKB1R w KQkq - 0 1"
        )

        assert stockfish.get_parameters()["Skill Level"] == 20

        stockfish.set_skill_level(1)
>       assert stockfish.get_best_move() in (
            "b2b3",
            "d2d3",
            "d2d4",
            "b1c3",
            "d1e2",
            "g2g3",
            "c2c4",
            "f1e2",
            "c2c3",
            "h2h3",
        )
E       AssertionError: assert 'f1b5' in ('b2b3', 'd2d3', 'd2d4', 'b1c3', 'd1e2', 'g2g3', ...)
E        +  where 'f1b5' = <bound method Stockfish.get_best_move of <stockfish.models.Stockfish object at 0x103667550>>()
E        +    where <bound method Stockfish.get_best_move of <stockfish.models.Stockfish object at 0x103667550>> = <stockfish.models.Stockfish object at 0x103667550>.get_best_move

tests/stockfish/test_models.py:177: AssertionError

That's a good point that I've been thinking about as well. The test concept needs some changes. I think there is no way around specific tests, but I am thinking about working with fixtures and simulating the output of the engine itself at least partially and mocking the _put and _read_line functions. This way you could test the function of the parser independent of an actual engine. This is of course quite a lot of work and it is still necessary to always include end-to-end tests with an actual engine as an additional step, but for some functions it would at least be an approach to think about.
(Although that's certainly a point to work on, I'd think that for now we'd look at working on any acutely pending bugs and the like, and really moving all the stuff over from the old repo. Once we're through with that, we won't have to worry about duplications or merge conflicts and we'll have everything together. So it may be that we will not work on it directly, but thanks for the suggestions and ideas :smiley:).

Adding to this, using a recent version of SF (May 30), the test_get_wdl_stats function fails for me (in addition to the test_set_fen_position_second_argument one you mentioned).


Investigate filtering of duplicate solutions for intermediate edge swaps

With ComPWA/expertsystem#487 and the groupings notebook introduced by b049d3e, I generated the following transitions. Judging from the labels, there are duplicates.

particle_graphs.pdf
transitions.pdf

Originally posted by @redeboer in https://github.com/ComPWA/expertsystem/issues/482#issuecomment-774773166

Doesn't this multiplicity just stem from the fact that there is final state combinatorics?


Package Blink not found in current path:Package Blink not found in current path:

Caveats: I am NOT a developer at all. My abilities consist of very limited Matlab use and some limited Fortran back in the "submit a stack of cards" days.
Issue : What I am trying to do is install IJulia so I can run Jupyter notebook and learn what Julia is all about. I am doing this on a Mac. My abilities, such as they are, do extend to being able to poke around directories in the Terminal window.  So in the process of trying to do this, I get the message in the subject line. I do a system wide search for Blink and don't find it (it turns out there's a gif on my machine called Blink, tho...who knew?) Any help greatly appreciated. And in fact, if you think I am headed down the wrong track to learn Julia, especially as a Matlab replacement, please advise.  Thanks, Mark

It sounds like you are trying to use the Blink package (https://github.com/JunoLab/Blink.jl)?
You can install Blink in Julia by doing
using Pkg
Pkg.add(“Blink”)

IJulia itself does not use Blink, so I guess it was being used by some other code you are trying to run in Julia?
In general, you use Pkg to install Julia packages and all their dependencies. (Even easier, you can just type ] at the Julia prompt to enter package mode, at which point you can simply type add Blink)

For general Julia usage questions, I recommend asking at https://github.com/JunoLab/Blink.jl

To install IJulia itself you should just be doing Pkg.add(“IJulia”)


Use Api.ai's natural language tools in your Botkit-powered Bot!

Proposed the use of Api.ai to integrate NLP to a Botkit bot.
It's even better than wit.ai
the api.ai community made a specific middleware for Botkit which i used in the example.

Good job
+1

Thank you coach :smile:

@Raniazy
We just created a new readme file didicated to all middlewares, and by this PR #483 i proposed to move your contribution into the new file readme-middlewares.md
Thoughts ?


add -o --out to fix scripts

We have many fix scripts which don't allow users to specify an output path. Most fix scripts will output a fixed font with .fix appended to the input font path. Adding an optional -o/--out arg would be a nice touch.

If we do implement this, we must add this feature so it doesn't break existing projects.

This would be an awesome thing to have a proper fix for! I've seen hundreds of (frequently dirty) shell scripts working around this irregularity.
Orthogonality related because it causes the same genre of problem while trying to automate build pipelines: https://github.com/googlefonts/fontmake/issues/693

This is something we still need to do, but it's essentially the same issue as #136.


Negentropy sync

It would be awesome if grain supported syncing with negentropy.

Yea multiple database types are on the to-do. I need to make some sort of adapter and LMDB is the first on the list since it's the most popular. Can you link me to more docs on negentropy?

Never mind. I think I found it again. https://github.com/hoytech/negentropy
Any tips for where to begin? I might have to read this several dozen times before it sinks in 😅

How's this look? Now, where to begin with the implementation...
Core code snippets:

Encoding a Varint:

func encodeVarint(x uint64) []byte {
    var buf []byte
    for x >= 0x80 {
        buf = append(buf, byte(x)|0x80)
        x >>= 7
    }
    buf = append(buf, byte(x))
    return buf
}

Fingerprint Calculation:

func calculateFingerprint(ids []string) []byte {
    var sum big.Int
    for _, id := range ids {
        idBytes := []byte(id)
        sum.Add(&sum, new(big.Int).SetBytes(idBytes))
    }
    hashInput := append(sum.Bytes(), encodeVarint(uint64(len(ids)))...)
    sha := sha256.Sum256(hashInput)
    return sha[:16]
}

Constructing and Sending Messages:

func createMessage(version byte, ranges []Range) []byte {
    var msg []byte
    msg = append(msg, version)
    for _, r := range ranges {
        msg = append(msg, r.toBytes()...)
    }
    return msg
}

func sendMessage(ws *websocket.Conn, msg []byte) error {
    return ws.WriteMessage(websocket.BinaryMessage, msg)
}

Handling Incoming Messages:

func handleIncomingMessage(ws *websocket.Conn, message []byte) {
    // Parse the message and handle ranges based on the mode.
    version := message[0]
    if version != 0x61 {
        // Respond with the highest supported version.
        ws.WriteMessage(websocket.TextMessage, []byte{0x61})
        return
    }

    // Parse ranges and respond accordingly...
}

Processing Fingerprints and Ranges:

When receiving a Fingerprint, compare it with your computed value for the same range.
If they differ, split the range into smaller sub-ranges and request detailed IdList information.
Synchronizing Missing IDs:

For IDs that are missing in your relay, initiate additional requests to fetch the events.

I haven't implemented negentropy personally, I just use the javascript version of the code in the repository. Maybe you could use the c version he provides through FFI, or you could use one of the go implementations out there:

https://pkg.go.dev/github.com/wader/fq/format/negentropy
https://github.com/HORNET-Storage/go-negentropy

One wrinkle is that Doug has recently released v1 of the protocol, which isn't backwards compatible. So you'd have to choose which versions to support, and there may not be any existing go implementations (if it helps, Coracle/Flotilla only support v1).


Is there anyway to detect that Guest Wifi is turning on or off?

I sometimes turn the guest wifi on, then forget to turn it off. I think it will waste some resources, so I want to create an announcement when the guest mode is turn on for a period of time.

No way to detect that currently. Not sure if there is a way to get the data from the API either.


Release Google.Cloud.AppEngine.V1 version 1.4.0

Changes in this release:
New features

Add Application.service_account (commit 523e2d2)
Add client library method signature to retrieve Application by name (commit 523e2d2)
Add Service.labels (commit 523e2d2)
Add Version.app_engine_apis (commit 523e2d2)
Add VpcAccessConnector.egress_setting (commit 523e2d2)

Created release for Google.Cloud.AppEngine.V1-1.4.0


ENH: Improve itk.imread error message when the input file doesn't exist

Ref #3900. Analogous to #3901 for release branch.
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

Ah, sorry @dzenanz maybe I should have not merged #3901?

Merging #3901 was a prerequisite for this PR.


IDEA preview plugin doesn't work for compose multiplatform application

when run task:  desktop:configureDesktopPreviewJvm or common:configureDesktopPreviewDesktop, error  occurs
Some problems were found with the configuration of task ':desktop:configureDesktopPreviewJvm' (type 'AbstractConfigureDesktopPreviewTask').

In plugin 'org.jetbrains.compose.ComposeCompilerKotlinSupportPlugin' type 'org.jetbrains.compose.desktop.preview.tasks.AbstractConfigureDesktopPreviewTask' property 'idePort$compose' doesn't have a configured value.

Hello! Which IDEA and which plugin version do you use?

Hello! Which IDEA and which plugin version do you use?

IDEA 2021.2.3(Ultimate Edition) Build #IU-212.5457.46
Compose Multiplatform IDEA Support : 1.0.0
It works only in "Compose Desktop" project.

after  update to 1.0.0 , still not work.  @Preview annotation cann't be resolved

Some problems were found with the configuration of task ':desktop:configureDesktopPreviewJvm' (type 'AbstractConfigureDesktopPreviewTask').

@iptton do you run the task manually? It should be run only from Intellij plugin.

@Preview annotation cann't be resolved

Do you have a dependency on compose.desktop.currentOs?
Please note, that the preview annotation and functionality is not available in common code yet.
Here is the example of MPP project configuration https://github.com/JetBrains/compose-jb/blob/master/idea-plugin/examples/simple-preview-example/mpp-jvm/build.gradle.kts (note that in the example the preview is available only in desktopMain source set, but unavailable in commonMain source set)

@iptton do you run the task manually? It should be run only from Intellij plugin.

yes ,because i cannot see preview, i tried to run it manually

Do you have a dependency on compose.desktop.currentOs?

I am using @preview in common module androidMain/commonMain/desktopMain folders of this project: https://github.com/JetBrains/compose-jb/tree/master/examples/todoapp
seams it has configured correctly but not work.
And i saw some error in file: desktopMain/kotlin/example..../Utils.kt
Cannot access 'java.io.Serializable' which is a supertype of 'kotlin.Int'. Check your module classpath for missing or conflicting dependencies

The project's clean, not changed any thing but add code like:
@Preview
@Composable
fun Test(){

}

@iptton I had a similar problem and I just found out that Preview was not working with Java 17. In order to get previews to work I set kotlinOptions.jvmTarget = "16" in my build.gradle.kts.

I experience a similar issue on IDEA 2022.3.3: no preview.
Turns out it's just a misleading user experience for someone used compose on Android Studio.
The preview doesn't appear automatically, I need those 2 steps to get it:

run the app once, this will add a small icon in front of composable functions
click the small preview icon of the composable I want see

i renamed my preview functions to <>Preview() with the annotation and it worked

saved

@iptton I had a similar problem and I just found out that Preview was not working with Java 17. In order to get previews to work I set kotlinOptions.jvmTarget = "16" in my build.gradle.kts.

saved my life. I'd never thought it'd be the sdk's problem

Why is it still not fixed in 2023??
Java 17 is so popular today :stuck_out_tongue_closed_eyes:

issue still here

yeah still not working

it still not working

@ThuanPx  No，Fleet can't preview... , Just follow  guide , I can't preview any desktop preview ~  Fleet -1.35.115

are you using jdk 11 or 17?

@ThuanPx  No，Fleet can't preview... , Just follow  guide , I can't preview any desktop preview ~  Fleet -1.35.115

are you using jdk 11 or 17?

I am using openjdk 17 ~

kotlinOptions.jvmTarget = "16"
Excuse , I not good at gradle or fleet ，Could you tell me how to set the jdk version?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Use just created selection instead of view selection state in dragstart handler

Hi,
I am using ProseMirror within Lit components. In order to fit Lit mental model and lifecycle, we update the editor view based on the state during render:
  render() {
    this.view.updateState(this.edstate)
    return html`...`

However, while working on drag and drop, I've noticed that the default handler expects the "dispatch" calls to be synchronous. This means that updateState is expected to be called immediately, so that view.state.selection is up to date to the latest selection.
This forces us to also call updateState whenever we dispatch a transaction:
      dispatchTransaction: (tr) => {
        this.edstate = this.edstate.apply(tr);
        // important otherwise the view state will be stale immediately after a dispatch
        this.view.updateState(this.edstate)
      },

While I know the need to call updateState when dispatching is documented, this is still a bit unsettling to see code consider a dispatch to be synchronous and the state to be immediately up to date. Would this still work if I wanted to add more operations to the transaction? It's also a breach in the Lit component lifecycle, because now the view will update in-between render cycles.
This is a small PR mostly as an occasion to discuss this approach. This code should work even with an asynchronous dispatch, because it uses the new selection directly instead of the selection from the view state.

While I know the need to call updateState when dispatching is documented, this is still a bit unsettling to see code consider a dispatch to be synchronous and the state to be immediately up to date.

Why? It's how the system is designed, it shouldn't unsettle.

Would this still work if I wanted to add more operations to the transaction?

Why not?
In any case, we are definitely not moving to a design where updates are asynchronous, so this PR is not something I'm going to merge.


⚠ Bofrost.es has degraded performance

In 4950feb, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 2277 ms

Resolved: Bofrost.es performance has improved in 70a3861 after 6 minutes.


[th/validating-webhook] add validating admission webhook to check DpuOperatorConfig

use operator-sdk create webhook to add the scaffolding for a validating admission webhook
implement checks for DpuOperatorConfig

enforce name to be dpu-operator-config
check mode for valid values

https://issues.redhat.com/browse/IIC-317
Missing: run the webhook in the kind cluster. As such, the tests don't cover this (though, there is a unit test that checks the validation code a bit). Maybe make test also need to run the admission webhook (if possible).

This looks good! Very clean for implementing the webhook. The issues you had before, how did you solve them? What were the issues such that we know what can go wrong.

This looks good! Very clean for implementing the webhook. The issues you had before, how did you solve them? What were the issues such that we know what can go wrong.

The main problem is that I didn't understand how to correctly make TLS certificates for the webhook work.
One approach is to uncomment the [CERTMANAGER] parts. That took me a while to understand how to get that to work. In particular, we must install the cert-manager operator, which is available in Openshift but not enabled by default. That is a problem for using it in dpu-operator.
Compared to sriov-network-operator, that seems implemented mostly by hand (not using operator-sdk?). So what is done there does not directly translate and spread out over many commits. It was not easy to understand or follow. It supports both cert-manager and service.beta.openshift.io.
For this branch, only service.beta.openshift.io is implemented. That seems sufficient to me, it also seems to work on Microshift.

/approve

/test all

/ok-to-test

/hold
Not holding because of contents, but rather because e2e ci seems not to be checking this anymore.


Support for more file formats

Hi David,
Do you have plans to update textract to support .ppt, .xlsx, .xltx, .potx, .key, .pages, .xml? I'd also love to see support for OpenOffice file formats, like .odt, .ott, .ods, .ots, .odg, .otg, .odp, .otp.
Thanks!

xml, xlsx, ods and odt are in the v1 branch, along with a fromUrl feature.
Some of the od* ones are a bit rough.  ppt is super hard, there is a ticket for it now. #24
I'd need to look at some of the outliers.  Some may already be supported, its just a matter of adding the mime type to an existing extractor.

key and pages don't have much existing node support.  would need to figure out a ton of XML spec stuff for both formats.

Perfect, thanks!

Going to look into odg/otg and odp/otp later this evening.  My hope is, if they are anything like docs and sheets that they won't be too hard to knock out.
key, pages and ppt will have to wait.
The rest are done.
Only other thing I want to get in before I release 1.0 is some speech to text stuff via Google's API.

Added new ticket to cover key and pages.  There is one for ppt.  Rest are included in v1 branch.

Perfect, thanks.


API Scan metadata fix

@Reezaali : Thanks for your contribution! The author, @MightyPen, has been notified to review your proposed change.

@Reezaali I see your metadata change. I will merge this public PR 1743 now.  Thanks, GeneMi.


Initial conditions

How are initial conditions handled?
I can imagine some chemical system where all the initial concentrations are known,
but then only some concentrations are measured during the reaction.
I think identifiability of the reaction kinetics might depend on whether those initial conditions are known or not?
I'm not sure if it is possible to pass that information.

Right now they are not handled. However, you are completely right, this knowledge may affect the identifiability results. The problem is that the problem with fixed initial condition is quite hard on the theoretical level, to the best of my knowledge there was no sound complete solution. Here is an example of subtlety with other software.
This said, I think the functionality is important enough to implement at least some semi-heuristic methods (with appropriate warnings), I will structure this as a couple of specific separate issues after NY.

I have elaborated on the problem and possible solutions in #169 and #168


Support specifying concurrency type while creating child DataContext.

@difujia Is this ready to be merged? It looks like it should be.

@AnthonyMDev Sure, these are simple and obvious changes.


How get fromLatLngToPoint from getProjection?

I tried this:
handleMapMounted(map) {
        this._map = map;
   }
render(){
const projection = this._map.getProjection()
const var p1 = projection.fromLatLngToPoint({ lat: 17.987557, lng: -92.929147 })

But i get undefined? Any suggestion?

Feel free to submit a full repo.
Please refer to Getting Help section in the README.


Derivative for padded(forSizes:with:).

Fix: #179

@eaplatanios All the requested changes are done.


Added pricegun sound

About the PR
Fixes #33473
Adds a sound that plays when the Appraisal Tool successfully appraises an entity.
previously PR #34101 - had to recreate this PR after deleting my fork.
Why / Balance
Previously, the Appraisal Tool had no sound, this improves feedback of the tool with audio.
Technical details
added SoundPathSpecifier to "Machines/chime.ogg" to PriceGunComponent class
added PlayPredicted to SharedPriceGunSystems class in the OnAfterIntereact method.
Media
https://github.com/user-attachments/assets/c847db24-9942-4407-99d4-a021c1539469
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
None
Changelog
:cl:

add: Added appraisal tool sound.

Thats hot, brother. I like the idea


Still seeing errors when connecting through pgbouncer

2018-10-22 07:44:50 SAST [4709]: [1-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_392" does not exist
2018-10-22 07:44:50 SAST [4709]: [2-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_391" does not exist
2018-10-22 08:44:51 SAST [8360]: [1-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_393" does not exist
2018-10-22 08:44:51 SAST [8360]: [2-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_392" does not exist
2018-10-22 09:44:52 SAST [11984]: [1-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_394" does not exist
2018-10-22 09:44:52 SAST [11984]: [2-1] user=postgres,db=postgres,app=jpgAgent: ip-172-31-36-118,client=<IP_ADDRESS> ERROR:  prepared statement "S_393" does not exist

On 2nd thought, there will be other things broken by using jpgAgent with pgbouncer. We utilize persistent connections to the db, and expect the PID to remain static. This is simply due to how PGAgent was designed and being compatible with the existing system. I don't believe it'll be terribly easy to work around this.
I'd suggest connecting the main jpgAgent process directly to the database if possible.
For the all the job steps, you are able to use the annotations to connect them to your pgbouncer instance rather than directly to the DB, and this will mitigate taking more real database connections than needed.
Sorry I don't have a better solution for the moment.


Can you possibly add chips inside this widget?

Hi,
Can you possibly add chips input inside this widget?
Ex: https://i.stack.imgur.com/diyDl.gif

Hello.
Sorry for the late reply.
I would like to apply the function you suggested to this package.
However, due to personal reasons, it will be difficult to upload it as soon as possible.
I'm sorry.


Wish: support SomaticSniper paired analysis (tumor/normal)

Like MuTect and VarScan, this is yet another method which is used for analysis of paired normal/tumor samples.
Support needs to be written from scratch.

Closing this issue for now, it was addressed in #109. Thanks!


fix: update sdk core to 3.13.0

:tada: This PR is included in version 1.10.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add mssql support via sqlpackage

Added Microsoft SQL database support with sqlpackage utility.
Updated Dockerfile (add necessary dependencies).

Test has failed.

Test has failed.

Yep, we can't use empty host. I will add some logic tonight.

Fixed now. Sorry for inattention.


🐞 [BUG] - Validation issue in contact page fields

Description
In the contact page there is no front-end validation present. there should be an validation for each field so that an empty data request not get submitted to the backend.
Current behaviour
Empty values are getting submitted to the backend because there is no fields validation for contact page right now
Expected behaviour
Validations should be implemented for the contact page fields. so that empty data request not got submit to the backend.
Reproduction steps
1. Go to 'Contact page'
2. Click on 'Send button'
3. Empty data request is getting submitted to the backend

Screenshots
https://github.com/anmode/grabtern-frontend/assets/68184118/7bd96df4-9e87-44c5-883b-3f8dbd21865b
Logs
No response
Browsers
Chrome
OS
Windows
Are you contributing for a program ?
GSSoC2023
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

it has been fixed.. see the development version!


Fibonacci Search in Python

💥 Proposal
Category: Searching Algorithm
Language: Python
Fibonacci Search is a comparison-based technique that uses Fibonacci numbers to search an element in a sorted array. Similarities with Binary Search: Works for sorted arrays.
Also, it uses a divide and conquer algorithm that narrows down possible locations with the aid of Fibonacci numbers.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Added documentation for some of the newer features.

@StephenTaylor-Kx - please review.  I did not try to build the docs with these changes, but I hope I got the markdown right.  Please check that the tip box on "Extras" shows up correctly.

Looks like your text editor is set to discard trailing spaces. This is not a safe setting for our Markdown, because a trailing double space (yuk) marks a forced line break. Also generates a lot of spurious changes in the diff. I think I found your intende change and it looks OK, so merging this.

Merged locally and pushed to server

Slight fix to the admonition: the title string needed embracing
double-quotes. Thanks for this!
Best
Stephen
Stephen Taylor | Librarian | Kx | +44 7713 400852 |<EMAIL_ADDRESS>On 8 November 2017 at 18:37, Alexander Belopolsky<EMAIL_ADDRESS>wrote:

@StephenTaylor-Kx https://github.com/stephentaylor-kx - please review.
I did not try to build the docs with these changes, but I hope I got the
markdown right. Please check that the tip box on "Extras" shows up
correctly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/KxSystems/docs/pull/20#issuecomment-342916671, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AT2hgE4-1FfJxjAKMYjX6axMiqQzEDh1ks5s0fT7gaJpZM4QW1m1
.


Error using divider with ---

I'm using frontmatter in my Markdown and using dividers (---) within my text. However, the library is removing all the text between the --- tags.
I noticed there's a pull request (https://github.com/quantizor/markdown-to-jsx/pull/538) that automatically removes YAML frontmatter. I suspect this PR is causing the issue by removing all text between --- tags, and since my text contains two such dividers, it gets removed.
Using another type of divider tag (like *** or ___) works correctly.

I may be able to fix this by modifying the regex to be lazy while looking for ---. I'll try to take a look this week.


Awake Icon

Description of the new feature / enhancement
In Powertoys Awake :

It would take four different icons to easily see the setting
-- Icon1 for  :  "Keep Screen on"
-- Icon2 for  :  "Mode>Off"
-- Icon3 for  :  "Mode>Keep Awake Indefinitely"
-- Icon4 for  :  "Mode>Keep Awake Temporarily"

It's easy for you
sorry for my poor english
Scenario when this would be used?

It would take four different icons to easily see the setting

Supporting information
No response

Duplicate of #11996

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


Dragging and Dropping (New) Blocks Causes "Cannot find descendant" Error

Bug Report
I've found that dragging and dropping blocks in a blocks field under certain conditions will cause a "Cannot find descendant" error. I'm not sure if this is coming from whatever drag and drop library Payload's using, or if this is from custom code.

Text version of first error
Do note that the content in the first node here is what's in the paragraph above (not shown in video), the content in the second (that I've bolded) is in the Paragraph block I just added in the video.
Uncaught Error: Cannot find a descendant at path [0,14] in node: {"children":[{"type":"ul","children":[{"children":[{"text":"Hardness","bold":true},{"text":": a material's resistance to wear and scratching"}],"type":"li"},{"children":[{"text":"Strength","bold":true},{"text":": a material's ability to be stressed without becoming permanently deformed."}],"type":"li"},{"children":[{"text":"Ductility","bold":true},{"text":":  a material's ability to become permanently deformed under stress before it breaks."}],"type":"li"}]}],"operations":[],"selection":{"anchor":{"path":[0,14],"offset":189},"focus":{"path":[0,14],"offset":189}},"marks":null,"id":5,"history":{"undos":[{"operations":[{"type":"remove_node","path":[0],"node":{"children":[{"text":""}]}},{"type":"insert_node","path":[0],"node":{"children":[{"text":"Lorem ipsum "},{"text":"dolor","bold":true},{"text":" sit amet, consectetur adipiscing elit. Sed lacus ligula, faucibus sit amet mauris sed, dapibus viverra risus. Donec sit amet efficitur dui. Quisque id "},{"text":"volutpat","bold":true},{"text":" "},{"text":"purus","italic":true},{"text":", vel interdum dolor. Nam ornare leo nunc. Sed at eros nec nunc vehicula posuere ac at libero. Etiam sed sollicitudin nunc. Donec efficitur eros nec nibh porttitor, efficitur tempus lectus gravida. Pellentesque mollis, enim non bibendum imperdiet, "},{"text":"diam","bold":true},{"text":" sem malesuada velit, ut vestibulum nibh dolor id dui. Vivamus "},{"text":"cursus","bold":true,"italic":true},{"text":" eleifend justo pulvinar mollis. Vestibulum pretium massa et rhoncus pretium. Fusce odio dolor, tristique non ultricies eu, faucibus vitae nibh. Suspendisse potenti. Mauris tellus mi, hendrerit sed commodo vel, "},{"text":"fermentum","bold":true},{"text":" "},{"text":"cursus","bold":true,"italic":true},{"text":" mi. Integer sodales convallis feugiat. Fusce aliquam pretium urna rutrum laoreet. Fusce hendrerit, odio et facilisis vestibulum, nunc arcu congue augue, at volutpat eros odio finibus urna."}]}}],"selectionBefore":{"anchor":{"path":[0,0],"offset":0},"focus":{"path":[0,0],"offset":0}}}],"redos":[]}}
at Object.get (index.es.js:2487:1)
at Object.common (index.es.js:2349:1)
at getCommonBlock (getCommonBlock.js:7:1)
at isListActive (isListActive.js:8:1)
at ListButton (ListButton.js:45:1)
at renderWithHooks (react-dom.development.js:16305:1)
at mountIndeterminateComponent (react-dom.development.js:20074:1)
at beginWork (react-dom.development.js:21587:1)
at HTMLUnknownElement.callCallback (react-dom.development.js:4164:1)
at Object.invokeGuardedCallbackDev (react-dom.development.js:4213:1)

Steps to Reproduce

Video showing this behavior - Note that the video was too big for GitHub. This link will expire in 24hrs, so please contact me if you need another copy of it.
I haven't tried this in a fresh/clean instance, but the way I was able to reproduce it consistently is by...

Have a blocks field (mine's in a collapsible)
Add a new block of content to the array of pre-existing content.
Before saving, attempt to drag and drop to move the old block below the new content.
The above step doesn't seem to work as it should the first time (as shown by the click and drag not doing anything in the video), so...
Attempt to perform step 3. again
The screen will go blank and the errors are logged to the console.

This has happened before by dragging the new content as well.
Other Details
Payload version < v1.16.11 -> v1.16.17
Browser is the latest version of Chrome on MacOS
Let me know if you need more info, but this is causing some serious frustration for our team. They're scared their work will be lost so they've resorted to hitting save constantly like early 2000's Microsoft Word 😆

Hey @TheDunco — figured this out! Thanks for the issue and the details. That is super helpful when we go to recreate and we really appreciate it.
It ended up being a strange Slate side-effect but the fix was simple.
Releasing a patch version now!

Awesome, that's great to hear! Thanks for the quick fix, y'all rock! 🥇


Doc: replace "," with "," in the end of the sentence

💭 Description
Where: https://github.com/devtron-labs/devtron/blob/main/README.md
What: "Implements GitOps to manage the state of Kubernetes applications," ends in "," and should be a ".", according to other examples.
Screenshot:

P.S. Please apply "good first time issue" label.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#1006

@nikitakoselev I would like to work on this issue.

hello @nikitakoselev
i have fixed this issue check below pr:
https://github.com/devtron-labs/devtron/pull/2552#issue-1429301913

It has been fixed by #2550, has to be merged, so I can close this ticket.
@akshat2203, it looks like 2 people completed the fix. I suggest you make a Twitter post about it, as a funny part of our job, which is hard to avoid sometimes, even for experienced devs :)

Hi @nikitakoselev, its fixed. you can close this issue.

Thank you 🤓
On Thu, 24 Nov 2022, 07:25 SNe789, @.***> wrote:

Hi @nikitakoselev https://github.com/nikitakoselev, its fixed. you can
close this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/devtron-labs/devtron/issues/2548#issuecomment-1326053666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQWQ6XRZSFQ7DBIPNP3ZDLWJ4KABANCNFSM6AAAAAARRV7RKA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Support for vector graphics

Opening an issue for this since it has come up repeatedly in discussions.
Loading and displaying SVG and similar formats would definitely be a useful addition to the framework.

I did a bit of research today, trying to find an svg library that can target .NET Standard.
So far, the most promising candidate seems to be the appropriately named https://github.com/vvvv/SVG.
It does target .NET Framework, but with vvvv/SVG#326, vvvv/SVG#365 and several manual changes I got it to compile under netstandard 2.0. I also created a simple .NET Core test app, which runs under windows and rasterizes an svg file.
Unfortunately, the SVG library is very tightly coupled to the System.Drawing namespace, which is not available under netstandard. The first PR I mentioned uses ZKWeb.System.Drawing to add back the needed functionality, but it has since been discontinued, so I changed it to use System.Drawing.Common instead.
Please note that my fork of the library it its current state is completely hacked together and not really in a presentable state.

Just a thought:
Instead of a library with "external" rendering, how about reading the .svg file (it's XML) and "translating" that to drawables directly?
I found a small library which seems to be able to read (but not render, I think?) the .svg format:
https://github.com/huysentruitw/svglib
It could be useful for that kind of approach, and supports the .NET Standard without any modifications.

yes, that would be the only correct path forward


Implement robust user input flow on Plan For Me page

Summary
This pull request implements user input flow on Plan For Me page, involving:

[x] Adds auth-walled and responsive Plan For Me page, accessible from nav bar and mobile sidebar
[x] Adds text field and select form for city and filter input, respectively
[x] Initialize all Materialize select forms across site in constant-script.js
[x] Implements autocomplete specifically for city input so user does not need to do it manually and that Google Places API can return more details about the city (for later search of places within that city)
[x] Displays toast when user does not select city from autocomplete suggestions
[x] Implements two event listeners (one for when user selects autocomplete suggestion and when user selects filter) with shared global variables so user can input city or filter in any order and page updates accordingly

Test Plan
Run mvn package appengine:run on branch, input city and filter on Plan For Me page, and check if it properly outputs to the browser console (prints PlaceResult object of city and filter value). To test for the toast, type a city without selecting an autocomplete suggestion and press enter to trigger the toast.
Notes
Though seemingly simple, figuring out a solution that allowed the user to input a city and filter in any order and automatically update the page with new input without drastically restructuring user input flow in an unnatural or unclear order was difficult for me given my previous unfamiliarity with how Javascript runs event listeners relative to the rest of the file like regular functions and involved a lot of experimentation. This helps set up automatic page updates i.e. place suggestions when user decides to change either of their initial input.

weird, but I tried using the My Trips features on your branch and it seems broken. It looks like the my-trips.html is importing the place-suggestions.js script for some reason, even though it doesn't need it, which is probably causing the issue. Can you remove the <script src...> now that it's not needed on the My Trips page?


Using unix.EOPNOTSUPP will fail when build on windows.

Hey @buger , the last PR https://github.com/buger/goterm/pull/38 brought the change which prevented built for the Windows OS.
if errors.Is(err, unix.EOPNOTSUPP) {
	return math.MaxInt
}

I suppose, that there are should be built tags, for Linux and separate terminal_windows.go w/o Operation not supported on transport endpoint (EOPNOTSUPP) constant.
Thanks :)

@buger friendly ping 😃 https://github.com/buger/goterm/pull/41

Fixed now


ffmpeg-nightly: Fix checkver, moved to /latest builds, Update to version<PHONE_NUMBER>

[x] I have read the Contributing Guide.
Closes #795

/verify

/verify