[website] update meeting times and details

What type of PR is this?
Website updates. See preview of meetings section and GAMMA page for review.
What this PR does / why we need it:
Updates meeting times to add monthly alternate time and consolidates GAMMA meetings back into main Gateway API meeting.
Which issue(s) this PR fixes:
Fixes "add to your calendar" broken link.
Does this PR introduce a user-facing change?:
NONE

Thanks @mikemorris!
/lgtm


Bug 1117931 - Removed picture gallery caching

Coverage decreased (-0.01%) when pulling e1596d012a0b4c9481d414d940b8240b74accd6f on anurag90x:Bug#1117931 into 41ccf279586a8a845a38ca27c3bf4cc27c65f0d8 on mozilla:master.


Api tests

Another test to discuss #976

Codecov Report

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

@@           Coverage Diff           @@
##           master     #980   +/-   ##
=======================================
  Coverage   63.64%   63.64%
=======================================
  Files         171      171
  Lines        9875     9875
=======================================
  Hits         6284     6284
  Misses       2801     2801
  Partials      790      790

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 b972ac0...d173c02. Read the comment docs.


There is no sound on Edge&IE

Repro steps:
• Navigate to https://makecode.adafruit.com/ on Edge & IE
• Drag block "play tone..." and put it under forever
• Click the key as below picture
Expected result:
The block should have sound
Actual result:
There is no sound

This isn't a bug. IE Doesn't support AudioContextManager so we can't produce tones.

So how about Edge?


Revert "wireshark: add wrapGAppsHook fixing open/save GUI"

Reverts NixOS/nixpkgs#147592. This change was fixing something that was already fixed in master, but due to a mistake was still merged.
cc @Ma27
(Also, it seems breaking the commit format for revert PRs is pretty common)

I don't think this should have been reverted. I've been hit by broken file chooser on Ubuntu and made https://github.com/NixOS/nixpkgs/pull/165440 to fix it.

Huh. We should probably have some kind of checkPhase to validate this somehow..


enabled dev and stable builds

enabled dev and stable builds

Oooh, this makes some/many of my bump patches obsolete, as some are looking for TARBALL_URL (for example) to even update the download URL :( @IgorTodorovskiIBM


publish to sonatype

publish to a a standard scala open source repository, such as sonatype or maven central

Done.


upload_post return None

When i tried to upload image post response return None...

Did the image upload successfully?

no, there is no action...

I'm having no issue. Will you help me to debug ?

You need to resize some image before upload that.
http://instagram.com/animemoeus_bot

response = bot.upload_post("instagram.jpg", caption="test caption")
gives the following error
site-packages\myigbot_init_.py", line 696, in upload_post
upload_id = json_data['upload_id']
KeyError: 'upload_id'


PeakAnnotationsLoader 1 - Necessary changes to TableLoader base class

Summary Change Description
I'm working from the tip of a fully working PeakAnnotationsLoader class.  Changes in that branch spanned a lot of files, making changes in necessary supporting files, but the change is large, so I am following the following process to break it up:
git checkout main
git branch peak_annot_table_loader_1_table_loader
git merge peak_annot_table_loader_rebased  # working branch
git cherry-pick -n 41ca3d434b80c10d5538c668f4308a6354ff57e7  # HEAD of peak_annot_table_loader_1_table_loader
git reset fe501ba73745e13d4d07cd66a2da33b7078e1460  # HEAD of main
git add -p
git commit  # Need to specify new/moved files manually

This first PR represents cherry-picked changes related to the TableLoader class.  The class file moved (as did table_column.py) and a few key changes were made to support the new PeakAnnotationsLoader:
Overview:

Added the ability to take "extra" headers without error (i.e. the sample headers)
Added a "warned" stat.
Added the ability to summarize errors based on a SummarizableError class so that the TableLoader code could dynamically summarize any error type without having to know the particulars about an error.

Details:

TableLoader

Location move (wherever it's imported)

move TableColumn as well
Update wherever either is imported (e.g. other loaders and tests)

Exceptions changes

SummarizableError is a superclass that effectively simplifies a chunk of TableLoader code.  It affects the following exception classes:

ConflictingValueErrors
DuplicateValueErrors
RequiredColumnValues
UnknownHeaderError

NotATableLoader - This was added to supplant a generic TypeError that I found confusing, due to the way the eventual new peak annotations loader class uses new arguments to the constructor.  Table loader will not (as a result) complain if you supply positional arguments (because it takes none and it wasn't complaining when you supplied the wrong arguments positionally).
AggregatedErrors - I added a convenience method: get_exception_types to make the eventual new code more succinct.

A new "warned" stat was added to highlight when there are warnings, because I omitted the --new-researcher flag requirement by demoting it to a warning, but I didn't want curators to miss it in the hubbub.  That caused the following loaders/files to have to be updated as well:

DataRepo/loaders/base/table_loader.py
DataRepo/loaders/lcprotocols_loader.py
DataRepo/loaders/msruns_loader.py
DataRepo/loaders/sequences_loader.py
DataRepo/loaders/tracers_loader.py
DataRepo/management/commands/load_table.py

Affected Issues/Pull Requests

Resolves #issue_number

Review Notes

See comments in-line.
Checklist

This pull request will be merged once the following requirements are met.  The
author and/or reviewers should uncheck any unmet requirements:

Review requirements

Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here

Review period: 2 days

Associated issue/pull request requirements:

[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)

Basic requirements

[x] All linters pass
[x] All tests pass
[x] All conflicts resolved

Overhead requirements

[x] New/changed method tests implemented/updated (or no change)
[ ] Updated Unreleased section of changelog.md (or no change)
[x] Migrations created & committed (or no change)

One item I thought about when looking this over was the DryRun exception. I don't feel that the management command should be raising a DryRun exception if the --dry-run flag is set. Instead, I think there should be an underlying call that raises the exception to roll things back and the command should exit successfully (without raising an exception). This isn't something I feel needs to change in this PR, but it's a general strategy I'd consider for all management commands.

Yeah.  I got confused when I was testing, because I think I'd gotten used to seeing dry run exceptions on the command line, and the old accucor data loader was raising DryRun exceptions, so when I copied and updated accucor loader tests, my tests failed because I wasn't raising the dry run exception.  It also was confusing because the status messages were green and I was like: was the dry run exception raised or not?
In general, I'm still not entirely satisfied with the way Python handles exceptions.  I think they're quite powerful and open up a ton of possibilities, but I hate stack traces.  They're ugly, I find their presentation order to be backwards, their "during" messages aren't always correct (i.e. the description preceding the sub-trace incorrectly describes the order of operations sometimes), and you can't get (a true) one if you continue to allow the script to run - but they're just so darned useful for debugging.  I really liked the more succinct custom traces I implemented in my perl work.
I guess I just got used to python exceptions, despite the things I dislike about them.  But I agree with you.  I think that it shouldn't be raised from the command script, and I think that the color change I implemented for dry run messages should suffice to clearly convey what happened.

[ ] TODO: Create an issue to NOT raise DryRun exceptions from the command scripts.


Performance in JPA reads

I have noticed when returning large pages of search results we spend a lot of time in stack traces such as the trace below.
From this I am guessing that if we're loading 20 resources for the page, we're doing 20 DB operations individually to fetch them. This could definitely be streamlined.
I do also believe if the resources have tags this further slows things down quite a bit so I'm betting that the tag loading is a subsequent DB operation as well.
My general theory is that to load a set of resources we should be able to do one DB query for the resource bodies, and one more DB query for all of their tags/profiles/security labels.
        at org.hibernate.event.internal.AbstractFlushingEventListener.flushEverythingToExecutions(AbstractFlushingEventListener
.java:85)
        at org.hibernate.internal.SessionImpl.autoFlushIfRequired(SessionImpl.java:1397)
        at org.hibernate.query.internal.AbstractProducedQuery.list(AbstractProducedQuery.java:1414)
        at org.hibernate.query.internal.AbstractProducedQuery.getSingleResult(AbstractProducedQuery.java:1463)
        at org.springframework.data.jpa.repository.query.JpaQueryExecution$SingleEntityExecution.doExecute(JpaQueryExecution.ja
va:206)
        at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:185)
        at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:212)
        at com.sun.proxy.$Proxy197.findForIdAndVersion(Unknown Source)
        at ca.uhn.fhir.jpa.dao.BaseHapiFhirDao.toResource(BaseHapiFhirDao.java:1570)
        at com.sun.proxy.$Proxy215.toResource(Unknown Source)
        at ca.uhn.fhir.jpa.dao.SearchBuilder.doLoadPids(SearchBuilder.java:1608)
        at ca.uhn.fhir.jpa.dao.SearchBuilder.loadResourcesByPid(SearchBuilder.java:1667)
        at ca.uhn.fhir.jpa.search.PersistedJpaBundleProvider.toResourceList(PersistedJpaBundleProvider.java:281)
        at ca.uhn.fhir.jpa.search.PersistedJpaBundleProvider$1.doInTransaction(PersistedJpaBundleProvider.java:141)
        at ca.uhn.fhir.jpa.search.PersistedJpaBundleProvider$1.doInTransaction(PersistedJpaBundleProvider.java:138)
        at org.springframework.transaction.support.TransactionTemplate.execute(TransactionTemplate.java:140)
        at ca.uhn.fhir.jpa.search.PersistedJpaBundleProvider.doSearchOrEverything(PersistedJpaBundleProvider.java:138)
        at ca.uhn.fhir.jpa.search.PersistedJpaBundleProvider.getResources(PersistedJpaBundleProvider.java:214)
        at ca.uhn.fhir.rest.server.method.BaseResourceReturningMethodBinding.createBundleFromBundleProvider(BaseResourceReturningMethodBinding.java:156)
        at ca.uhn.fhir.rest.server.method.PageMethodBinding.handlePagingRequest(PageMethodBinding.java:129)

James - I'd be happy to help with this one.  Let me know if I should work it.

Is the relationship between HFJ_RESOURCE and HFJ_RES_VER one to one by RES_ID and VER_ID?

Fixed the looping issue for the HFJ_RES_VER (ResourceHistoryTable). Will look for tag loading.


Fixes compiler errors

Hi,
I was compiled it from the scratch with gcc (Debian 12.2.0-14) 12.2.0 with -Wall enabled and I got some error:
/opt/src/thirdparty/etcd-cpp-apiv3/src/KeepAlive.cpp:72:52: error: unused parameter 'handler' [-Werror=unused-parameter]
   72 |     std::function<void(std::exception_ptr)> const& handler, int ttl,
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
/opt/src/thirdparty/etcd-cpp-apiv3/src/SyncClient.cpp:150:1: error: type qualifiers ignored on function return type [-Werror=ignored-qualifiers]
  150 | const bool authenticate(std::shared_ptr<grpc::Channel> const& channel,

The first one is an actual error that can lead to bugs if the ctor will be used. The handler passed to the ctor was omitted by mistake.

Merged. Thank you!


chore: update to ui-forms and ui-core@latest

Picking up the refactor for the scheduler app. I'll submit separate PRs per topic, instead of piling onto the sizeable number of commits on #23.
I'm afraid that the changes are less atomic than I'd like. But ui-core was quite far behind. And instead of updating ui-core first, and then adding ui-forms, I thought I'd skip the inbetween step to save on time. It makes for a larger PR but it's a bit quicker.
Also, I'm aware that a lot of best practices have emerged in our ecosystem since I last worked on this codebase. Don't worry, I aim to align it fully with that, it's just that I'd prefer to do that in separate PRs for reviewability.

No rush btw @Mohammer5, just tagged you for later so I don't forget.

This looks good to me.
I think we can import Field and FormSpy from ui-forms and remove react-final-forms as an explicit dependency for this app.

Yeah you're right, good catch. I've talked to Hendrik, and later on we could potentially also reexport from final-form, so that that can be removed as well. For now, I've removed the react-final-form imports in favor of those from ui-forms.


Type 'string[]' is not assignable to type 'Shapes[]'.

I'm trying
const shapes = ['square', 'ellipse']

confetti.create({
    ...
    shapes: shapes
})

I've done the same with colors and that's working, but shapes doesn't like the string[]

Fixed. Thanks for the issue.


Check failed: error == cudaSuccess (2 vs. 0)  out of memory

Hi,
I train a classifier with :
./tools/train_net.py --gpu 0 --solver models/VGG16_RstarCNN/solver.prototxt  --weights reference_models/VGG16.v2.caffemodel
and get an error.
F1007 19:40:40.235829 26752 syncedmem.cpp:66] Check failed: error == cudaSuccess (2 vs. 0)  out of memory *** Check failure stack trace: *** Aborted (core dumped)
Where can I modify the 'batch_size'?

Hi,
I've got the same trouble.Can you tell me how you solved the question?

I got the same question，and i use the same tyoe of GPU as the author，e.g.K40C，and i found that this program take almost 10G memory of the GPU which has in total 12G memory，and i worked fine when the GPU is total free ，but get the same error out of memory  when there are process on it，so you can find a GPU with enough memory。

Thanks a lot for your attention and help.I've tried to make the batch_size ,scales and max_size in config.py become smaller.Now it can run collectly.

You can do that, however note that the performance will drop if you make those changes.

Thanks for you reply.I know that,but now i don't have the GPU with enough memory.It's the helpless choice.

Lol! I sympathize with you! Good luck!


Invalid argument: model input must specify 'dims'

I have a tensorflow savedmodel and deploy it with trtis.  I didn't set any config file but I add the --strict-model-config=false option. After deployment ,The status is as blow:
id: "inference:0"
version: "1.0.0"
uptime_ns:<PHONE_NUMBER>92
model_status {
key: "inception_new"
value {
config {
name: "inception_new"
platform: "tensorflow_savedmodel"
version_policy {
latest {
num_versions: 1
}
}
max_batch_size: 1
input {
name: "input"
data_type: TYPE_FP32
dims: 299
dims: 299
dims: 3
}
input {
name: "keep_prob"
data_type: TYPE_FP32
}
input {
name: "is_training"
data_type: TYPE_BOOL
}
output {
name: "output"
data_type: TYPE_FP32
dims: 5
}
instance_group {
name: "inception_new"
count: 1
gpus: 0
kind: KIND_GPU
}
default_model_filename: "tf_savedmodel"
}
version_status {
key: 1
value {
ready_state: MODEL_UNAVAILABLE
}
}
}
}
status_stats {
success {
count: 4088
total_time_ns: 41681841
}
}
and the error is : Invalid argument: model input must specify 'dims'
Model is unavailable, How to overcome this problem ?
I know if I redefine my tensorflow graph and remove those tensor like is_training would be a solution, But it's a big cost. Is there any other good way ? If I add a config file and add reshape config will work ?

Specify dims as an array if more than one dimension ->   e.g.:  dims: [ 3, 224, 224 ]

@ligonzheng, our current model config auto-fill does not support control flags of the form keep_prob and  is_training. Your current work around would be to create a custom config file with the keep_prob and  is_training defined with dims: 1
This also means that you cannot use dynamic batching (due to the same assumptions made while parsing the config model. Once you turn off dynamic batching and define the dims: 1 it should work.
Let me know if otherwise!

I had try your method but the result is not good. I added a config.pbtxt like this:
name: "inception_new"
platform: "tensorflow_savedmodel"
version_policy {
latest {
num_versions: 1
}
}
max_batch_size: 128
input [
{
name: "input"
data_type: TYPE_FP32
dims: [299,299,3]
},
{
name: "keep_prob"
data_type: TYPE_FP32
dims: [1]
},
{
name: "is_training"
data_type: TYPE_BOOL
dims: [1]
}
]
output [
{
name: "output"
data_type: TYPE_FP32
dims: [5]
}
]
As you said ,I added additional dims for is_training and keep_prob , no dynamic batching was setted. but I got this error if I use 19.04-py3:
Internal: unable to load model 'inception_new', model configuration supports batching but first dimension of tensor 'keep_prob' expected by framework is not a variable-size batch dimension: []
I got another kind of error if I use 19.03-py3:
Invalid argument: unable to load model 'inception_new', input 'keep_prob' dims [] don't match configuration dims [1]

@ligonzheng could you try this with the latest release -> 19.05-py3 ? I believe the support for such model configurations were incorporated in that release.

@ligonzheng Your model should be supported by TRTIS if you use the master branch (or 19.05 when it is released later this week). The problem seems to be that your model requires "reshape" which is available in 19.04 but the autofill (--strict-model-config=false) support for reshape was not added until 19.05.
Can you try loading your model (using --strict-model-config=false) with 19.05 (or master) and let us know if your problem is fixed.

@deadeyegoodwin Do you mean I can  use the autofill and a config file at the same time in 19.05?  otherwise how should I add the reshape configuration with the --strict-model-config=false option

Closing. @ligonzheng 19.05 just released. If you still see a problem with 19.05 please reopen this issue.

I tried my model again with 19.04 and 19.05. Different settings were made when I generate the tf savedmodel.  The cases are as below:
case 1, Reshape is_training tensor and keep_prob tensor to [-1,1], the /api/status shows the model is available. I try to access the server, it returns "You must feed a value for placeholder tensor 'keep_prob with dtype float'".
case 2, No reshape operation for is_training and keep_prob. The trtis stopped and return "model input cannot have empty reshape for non-batching model". This case is like the title which using 19.03.
case 3, Reshape is_training tensor and keep_prob tensor to [1,], th /api/status shows the model is available but the input tensor for picture data has a dim of -1. I tried to access the server, it returns "model supports variable size for input 'input', request must specify input shape for model".
case 4,remove the keep_prob and is_training when I define the signature. The /api/status shows the model is available. I tried to access the server, it returns "You must feed a value for placeholder tensor 'keep_prob with dtype float'".
`def convert(model_path):
with tf.Session() as sess:
saver = tf.train.import_meta_graph(model_path + '.meta')
saver.restore(sess, model_path)
graph = tf.get_default_graph()
    #writer = tf.summary.FileWriter("./log_inception",sess.graph)
    #writer.close()
    input_tensor = graph.get_tensor_by_name('input:0')
    output_tensor = graph.get_tensor_by_name('InceptionV3/Predictions/Softmax:0')
    is_training = graph.get_tensor_by_name("is_training:0")
    keep_prob = graph.get_tensor_by_name("keep_prob:0")

    is_training = tf.reshape(is_training,[None,1])
    keep_prob = tf.reshape(keep_prob,[None,1])
    #tf.graph_util.remove_training_nodes(input_graph=graph)

    new_path = "inception_new/1"
    builder = tf.saved_model.builder.SavedModelBuilder(new_path)

    inputs = {'input': tf.saved_model.utils.build_tensor_info(input_tensor),'is_training': tf.saved_model.utils.build_tensor_info(is_training),'keep_prob':tf.saved_model.utils.build_tensor_info(keep_prob)}
    outputs = {'output':tf.saved_model.utils.build_tensor_info(output_tensor)}
    method_name = tf.saved_model.signature_constants.PREDICT_METHOD_NAME

    my_signature = tf.saved_model.signature_def_utils.build_signature_def(inputs,outputs,method_name)
    builder.add_meta_graph_and_variables(sess,['serve'],signature_def_map={'serving_default':my_signature})
    builder.save()

`


Remove anchor text custom metric

Was only included in one crawl (9/1).

Can merge any time, but should only pull down to the HA server after the 9/1 crawl is complete (or at least done testing) and before the 9/15 crawl begins.


The domain graphene-python.org is no longer parked by GoDaddy

@syrusakbary
Getting this error when trying to access: http://gdom.graphene-python.org/

This should be fixed now :)


Publish

<EMAIL_ADDRESS>
Background
Adding the CountryCode type
Solution
Bumping 3 versions to match the vanilla package, now that they're in sync
🎩

...

Checklist

[ ] I have :tophat:'d these changes
[ ] I have updated relevant documentation

Should we wait for Rashad's changes?

I think having 0.15 of both for countrycode and then 0.16 for the discount changes is right. If I hadn't already published 0.15 of vanilla, i'd agree that holding off and combining would be best.


Doubt

How can we make it run with AZURE_OPENAI_API_KEY instead of OPENAI_API_KEY

Since it is using LiteLLM in the background, you can try using the variables which lileLLM accepts.
For env variables:
os.environ["AZURE_API_KEY"] = "" # "my-azure-api-key"
os.environ["AZURE_API_BASE"] = "" # "https://example-endpoint.openai.azure.com"
os.environ["AZURE_API_VERSION"] = "" # "2023-05-15"

For model:
 model = "azure/<your_deployment_name>",

Let's say you already created a deployment with name gpt-4o-mini, and the following code is the way that you can pass. In this example, we define a simple knowledge graph and we want to turn the content of one of the Paul Graham essays and extract a knowledge graph out of it. I assume you have already set your Azure API key and base and API version and there's nothing much left to do. You can simply pass the following code.
os.environ["AZURE_API_KEY"] = "" # "my-azure-api-key"
os.environ["AZURE_API_BASE"] = "" # "https://example-endpoint.openai.azure.com"
os.environ["AZURE_API_VERSION"] = "" # "2023-05-15"

async def main():
    class Entity(BaseModel):
        name: str
        description: str

    class Relationship(BaseModel):
        entity1: Entity
        entity2: Entity
        description: str
        relation_type: str

    class KnowledgeGraph(BaseModel):
        entities: List[Entity]
        relationships: List[Relationship]

    extraction_strategy = LLMExtractionStrategy(
            provider = "azure/gpt-4o-mini",
            api_base=os.environ["AZURE_API_BASE"],
            api_token=os.environ["AZURE_API_KEY"],
            schema=KnowledgeGraph.model_json_schema(),
            extraction_type="schema",
            instruction="""Extract entities and relationships from the given text."""
    )
    async with AsyncWebCrawler() as crawler:
        url = "https://paulgraham.com/love.html"
        result = await crawler.arun(
            url=url,
            bypass_cache=True,
            extraction_strategy=extraction_strategy,
        )
        # print(result.extracted_content)
        with open(os.path.join(__data__, "kb_test.json"), "w") as f:
            f.write(result.extracted_content)

    print("Done")

Thanks it worked :)


Added support externalIPs to Stolon

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md

Please review PR
Thanks

@rtluckie @dcwangmit01
Hello!
Please review PR
Thanks


Roadmap second version

When planned release second version?

 Due by December 31, 2016
https://github.com/orocrm/platform/milestone/2

Cool :)

@tadcka by the end of the year, unless major problems will happen. We hope they won't :)

@artml I think 2016 have been long enough as it is :) let's hope there is no more major problems :)  thanks!


Not Getting the code

I entered my phone number in  international format like +989222222 but I cannot get the code , what's the issue guys ? Can anyone help me ?

maybe your system time is not synchronized


Announce SciPy 1.7.3

Announce SciPy 1.7.3

I think the news stuff has already been reorganized a bit since I last did this so let me know if there's a better way

I added a link to the release notes; I don't see the connection to forward-porting though--it seems that these links just point to the github release page, which is already available.


Here are two earlier gym trading projects.

https://github.com/AminHP/gym-anytrading
https://github.com/AminHP/gym-mtsim

Thanks for reporting them. Both of them provide available financial environments.


Add support for default arguments in utility functions

This is a rebased version of https://github.com/godotengine/godot/pull/87275 (author seems abandoned it), with some fixes (including adding an actual default argument to is_equal_approx/is_zero_approx/randfn), updating documentation etc.

Closes https://github.com/godotengine/godot-proposals/issues/4425

@dsnopek Ok, I reverted the change, only randfn is affected and since its have same number of arguments the CI will pass, so this change may be merged before the utility extension mechanizm is implemented.


[v2] add configuration pages

Added these for most helpers, needs some more time investment to fully complete and improve so I'll open an issue for that


Relax GetConfigurationResponse value to allow integer also

Changes included in this PR
Relaxes the validation schema on the GetConfigurationResponse to allow numerical values in addition to string values.
Current behavior
Currently, only strings are allowed. Some devices are returning integers, which causes validation failure.
New behavior
Integers are allowed to be a valid format for the value of the response.
Impact
Enables use without modification for devices that pass integers directly, rather than encapsulated in strings.
Checklist

[ ] Does your submission pass the existing tests?
[ ] Are there new tests that cover these additions/changes?
[ ] Have you linted your code locally before submission?

Hi @emlynmac, these JSON schemas are the official ones from the OCA and we don't have any intention of modifying them. The changes you are asking for do not comply with the specification. I would reach out to the vendor to follow the spec properly or to use the skip validation parameters to not validate against the schema.


Question about how to set coordinate of other dataset (camera pose and lidar)?

Hi @rakhimovv @TArdelean, thanks for your excellent work. Now I want to reproduce your npbgpp on RobotCar Dataset, which gets lidar dataset, RGB images and camera pose (from IMUs) from a motion car with onboard sensors. I am curious about what kind of MetaShape is used to generate point clouds, and camera poses in your work.
Note I have tried to set the camera pose and point clouds from lidar to OpenGL coordinate, but the value of PSNR is quite low (around 5), so I doubt the coordinate is not right. I appreciate any help that you can provide!

Hi @LZL-CS
you can take a look e.g. at
https://github.com/rakhimovv/npbgpp/blob/main/npbgplusplus/data/colmap_scene.py
https://github.com/rakhimovv/npbgpp/blob/main/configs/datasets/colmap_one_scene.yaml
so one option e.g. is to convert to colmap format or some other presented in the repo
the other option - write your own dataset class
point cloud can stay the same - no modification is needed
Regarding the camera pose format:
so we know that x_cam = R^T @ (x_world - camera_center) = R^T @ x_world + T
regarding notation in the code:
R_col = R^T (_col stands for matrix-(column)vec multiplication)
R_row = R

as R_col = [u^T, v^T, w^T] = [
u.x, u.y, u.z,
v.x, v.y, v.z,
w.x, w.y, w.z,
]
(u, v, w) are camera x,y,z-axis

so how finally to make conversion:
in p3d the camera format is x-axis(=u) points left, y-axis(=v) points up and z-axis(=w) points out from the image plane.
so e.g. in colmap: x-axis points right, y-axis points bottom
therefore you need change a sign before u and v in R_col, we do this by following:
R_col[:2, :] *= -1
R_col also affects T as T=-R_col@cam_center, so we need to change T too:
T[:2] *= -1
so finally you need to compare the camera axis orientation in your dataset, compared it to p3d and change the signs of the certain axes if needed

Thanks for your kind reply. I will try it.


Save text as file

Title and Issue number
Issue No-
#2540
Title: Save text as a file
Close #<#2540>

Screenshot

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented on my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy-to-understand README.md

@MohitGupta121 please check the pr.


How to change colors?

How can i change background and font colors in character sheets? I have tried setting up it in "fabulaultima\less\globals.less", but it doesnt work.

I don't have an easy way to do this. You would have to change it in fabulaultima\less\globals.less, but then you'd have to compile it, using a less compiler.

I've added some ability to change color in commit 07da929 to the beta branch. You will be able to customize the bg of the sheet and the font color, but you cannot change the tabs themselves. Changing that would be fairly involved and would require sacrifices on the default style.


Improve Method History Browser performance

The Method History Browser is very slow in my current working image (around 60MB changes file) - 10 seconds to open, then another 10 seconds as it refreshes when I reinstall an earlier method version (this wasn't the case in D7.0). The core of the slowdown appears to be the send of nextAvailable: in ChunkFileMethodScanner>>positionsOf:in:firstOnly: as it reads and decodes the requested number of UTF8 characters.
I initially implemented a modification whereby ChunkFileMethodScanner instead reads a specified number of bytes via basicNextAvailable: plus any trailing UTF8 character units (the exact number of characters read is unimportant; it's just part of the local paging scheme). The result is a String of valid character points <= the requested number, and a much improved performance.
However a much simpler solution with similar performance is to remove the local paging scheme and simply read the entire file in one hit. I'm wondering whether this would be acceptable - the likely maximum size of a changes file (100MB?) isn't that large for current machines.

I'd noticed that myself of late, although more the impact of leaving open a method history browser on subsequent editing. It is quite a heavyweight operation to rescan the entire change file every time some new stuff is added to the end!
I think just changing the nextAvailable: to basicNextAvailable: is probably fine as a short term fix. It probably doesn't matter (or at least should not make any worse) the fact that this is potentially going to return partial characters at start and end. The statement that adjusts the position at the start of the loop could (in theory) cause that to happen anyway, at least at the start of the string. Without testing it to be sure, it looks to me like it would never be an issue because of the nature of the content of the file and what is being searched for - Smalltalk identifiers can't contain multi-byte characters because the lexicon doesn't allow it. An example search string would be !Object methodsFor!. I think this means we can only start searching for the string in the middle of a character which cannot be in the string, so there is no negative impact from having a start position that isn't on a code-point boundary.
Theoretically the code that adjusts the start position could fail to move back far enough if the current stream position is in the middle of a multi-byte character that follows immediately after the search string. Again in practice I don't think this can happen because the immediately following characters will be a pair of CR/LFs, and then the start of a method selector, all of which must be ASCII characters. For correctness, it should probably step back until Utf8String>>isLeadCodeUnit: is true for the current position, and only then step back by the length of the search text.
Another simple improvement to the history browser itself would be to avoid re-scanning on method update. The method history browser knows which method it is viewing the history for, and it already ignores all method changes apart from those for the method in question. However it doesn't need to scan the change log to get the updated source since it is passed the new method in the compilation result and could grab the source directly.
As a longer term fix I think two things should be done in the categories of optimization and choosing a different algorithm. As usual the 2nd approach is the one that carries by far the biggest win, although the optimization would benefit many other scenarios too:

The performance of PositionableStream>>nextAvailable: (or certainly the operation on FileStream) should be improved. When doing the UTF work I added an implementation that is functionally correct, but it could be optimised in some way. The request is expressed in "number of code points" rather than "number of code units", so it will probably require primitive support  as there is no way to tell the number of code points between two code unit positions in a variable length UTF encoding, except by enumerating through them.
Adopt a scheme of writing the source descriptor integer of the previous edition of the method as part of the chunk header for an individual method update in the log. This would then allow the string search for existing methods to be replaced by a massively faster linked list style traversal.

Assuming you started from the current method's source descriptor, (2) wouldn't work when using the Method History Browser to recover changes after exiting an image without snapshotting. Though I guess you could do a short forward scan to check for any unsaved versions and append these to the list.
A related idea which I implemented in D5 was to not re-log the method source when loading a previous method version but to just recompile and point the new method to the previous source descriptor. This kept things a bit cleaner and allowed something akin to an Envy-style history view:

The method "timestamps" are picked up from the nearest previous image start/snapshot marker. If this is of interest I can look at a port to D7.

Yes, that's true. If you wanted to see the forward but unsaved history, it would also be necessary to scan ahead.
I like the idea of restoring the original source pointer when a old version is reloaded. That would stop the rather unfortunate behaviour of loading an old version then adding another copy to the end of the list.
Personally I don't think I would find a time-based list terribly useful as my images tend not to be very long lived, and unlike Envy it's unlikely to be useful for going very far back. I would tend to look at the git history for that. For recent changes I prefer to just scroll back quickly through the version until I find the one I want as I can generally remember the shape better than when I did things, but that doesn't mean it wouldn't be useful for others.


Add high performance macro

In the case where the type is available at macro expansion time, it should be possible to provide a high performance macro that does introspection at macro expansion time and only trivial operations at runtime.

Ah, right. A generated function is the right answer here. A macro would be at macro-expansion time instead of compile time, but that is less flexible.
To do it as a macro the type would need to be a literal type and then the macro would determine what that literal refers to, but that would be a hacky and bad solution in comparison to a generated function.


Provide lib to calculate machine image vector

Description
As discovered during implementation of the AWS Component, we need to provide a reusable lib which can calculate the machine image vector.

first version ready in "provider-aws"


IsTar Implementation

Is your feature request related to a problem? Please describe.
Implement IsTar API
Requester Info
Anh Van, NASA Goddard

Note compression routines moved to FS and GSFC app libraries.


[Backend] Added Field and FieldType entities and MetaDataResource

@maheshkkumar check the additional endpoints - you have to integrate these with your code.


Make comments flat again #798

Removed most of the complexity out of hcomments, removed unused views, templates, forms and models and unnecessary customization of the django_comments.comment model. Added tests of the changes.
All hcomments templates were removed as they were duplicates of the files in template/hcomments.

@umgelurgel there are some conflicts due to python 3 migration.  could you please take a look at those?

Rebased and working under Python3 :)

79 sounds good to me as well. :)


Autocreate prompt content if entry is nil

I found it very annoying to need to hit shift-enter instead of enter. So... Now you just need to hit enter and the prompt will get created.
Removes <s-cr> binding since it's now redundant. <cr> will do the same thing.

I don't think this makes <s-cr> redundant does it? You can create_from_prompt without an empty entry with <s-cr> but switching to <cr> would now require the entries to be empty.

Fair point! I put it back.

thanks!


How to default to a login form?

I am using browserify to add a series of forms to UI, but I am unable to navigate to the initial Login form. I am using the following and expecting the default Login component to display. Is this the intended function of the example code?
<Router>
  <A path="/" />
  <B path="/b" id="42" />
  <C path="/c/:id" />
  <D default />
</Router>

Below is what I tried:
import { h, render, Component } from 'preact';
import Router from 'preact-router';

const routes = () =>
	(
		<Router>
	      <Login default />
  		</Router>
	);

render(
	<routes />, placeholder
);

Hi @donadley! The default prop is useful for things like "catch-all" or 404 pages, and your <Login> sounds like the former, so you're on the right track.
However, in your example, I notice you're using a lowercase name for <routes>.  The way JSX works means that will create a DOM element called "routes", rather than actually rendering your routes component.  The best way I know of to illustrate this is by looking at Babel's output: https://goo.gl/qcHLXb
Switching the name to <Routes> in your example seems to work just like you described:
https://jsfiddle.net/developit/56jq2wzs/

Let me know if this solves your issue! :)

That did the trick and now I can move forward. Thank you for the help.


fix hue errors in plasma_stick_rainbows

In this example the use of the monotonically increasing offset variable in the hue calculations will cause the colours to stop rotating when the repeated addition of 20 / 2000 eventually becomes too small to make a difference to the accumulated value given the limited precision of a float.    That is, at some point offset += float(SPEED) / 2000.0f becomes a no-op.
Granted it will take a long time - 265 days with the current code - but in a modified version of this demo I've been able to trigger issues far more rapidly.
This patch first makes sure that offset doesn't increase without bound so that it stays within the numerical precision range of a float, and secondly ensures that all hue values passed to RGBLED::set_hsv() are within the range [0..1)
The second change begs the question of whether the various FOO::set_hsv() methods in the library should perform this normalisation of the h parameter on entry to those methods.  It would not only resolve some issues I've seen with negative hue values, but also avoid the need for the i % 6 in the switch since with h constrained to [0..1) i will also be similarly constrained to [0..6).

Correction - I forgot to account for 60 updates per second.   The existing code should stop working after just 4 days and 10 hours, give or take.    It takes 22918664 passes through the loop for the addition of 0.01f to stop working.

Wouldn't it make more sense to just offset %= 1.0 and hue %= 1.0?

Wouldn't it make more sense to just offset %= 1.0 and hue %= 1.0?

No, you can't use modulo on floating point numbers in C / C++

Wouldn't it make more sense to just offset %= 1.0 and hue %= 1.0?

No, you can't use modulo on floating point numbers in C / C++

Right... another reason for why we need to move on from C as a society.

The proposed python fix was incomplete - it's necessary to constrain the range of the offset variable too.

Thank you for the detailed explanation and fix, this looks reasonable to me!


Fix arm64 build

Build error:
Undefined symbols for architecture arm64:
"_kCGImageSourceCreateThumbnailWithTransform", referenced from:
+[FLEXUtility
thumbnailedImageWithMaxPixelDimension:fromImageData:] in FLEXUtility.o
"_CGImageSourceCreateWithData", referenced from:
+[FLEXUtility
thumbnailedImageWithMaxPixelDimension:fromImageData:] in FLEXUtility.o
"_kCGImageSourceThumbnailMaxPixelSize", referenced from:
+[FLEXUtility
thumbnailedImageWithMaxPixelDimension:fromImageData:] in FLEXUtility.o
"_kCGImageSourceCreateThumbnailFromImageAlways", referenced from:
+[FLEXUtility
thumbnailedImageWithMaxPixelDimension:fromImageData:] in FLEXUtility.o
"_CGImageSourceCreateThumbnailAtIndex", referenced from:
+[FLEXUtility
thumbnailedImageWithMaxPixelDimension:fromImageData:] in FLEXUtility.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see
invocation)

It looks like your build is not linking to CoreGraphics.framework. This should happen automatically if you use cocoapods or have Link Frameworks Automatically set to YES in your build settings. If not, you may need to manually add it to the list of linked frameworks. Feel free to write back here if I've misdiagnosed the issue. I just tested building the example project for my 64-bit device and didn't encounter any issues.
I don't see a good reason why we would completely disable the thumbnailing functionality for arm64, so I'm closing the pull request.

Oh, sorry, thank you for the tip.

No problem! I hope the suggestions fix the error for you.


Extract text to translation reday files profile pages

Extracted user-visible text and created translation files for Page Profile and EditProfile.

@alexjball Hi Alex, please review this PR for extract translation on Page Profile and EditProfile

Hey @timblais could you take a look at this since you've been doing a lot of the translation work? Thank you!!

@tommagnusson Hi Tom, I have resolved issues you pointed out in comment. Please take a look for on the new commit again when you got time. And thank you for doing the code review for me!

@YuhaoT thanks for following up, looks great! If you wouldn't mind resolving the merge conflicts when you get a chance then I'll be happy to approve.


the link is not accessible

Unfortunately I can't edit the description here, but I'm running a clone of the project on my server:
https://demo.marpi.pl/codeology/


Scroll-zooming in Safari can move page position

Both in 0.7 and in 1.0, scrolling can move the page scroll in Safari. It's even visible on http://leafletjs.com, you can scroll down to the "Trusted by the best" part, by just doing a big scroll inside the Leaflet map.
Made an example for trying the different versions:
http://playground-leaflet.rhcloud.com/wey/1/edit?html,output
Usually it's just 10-20 px, but can be a big jump as well.
Safari 9.0.2, OS X 10.9.5

Reproducible on Chrome as well, happens when you scroll on a popup. It shouldn't propagate to the page...

Weird, as L.Popup._initLayout calls L.DomEvent.disableScrollPropagation: https://github.com/Leaflet/Leaflet/blob/master/src/layer/Popup.js#L187

That's great to have it fixed, but my issue was not related to popups. Removed the popup from the example, have a look:
http://playground-leaflet.rhcloud.com/sow/1/edit?html,output
Screen recording:
https://youtu.be/qVjqGycT0u0

Have a look at this, somehow even this can move the page (try scrolling outside, then over the container):
http://playground-leaflet.rhcloud.com/rac/edit?html,output

Let's make a clean minimal non-Leaflet example. I'd assume that this is a Safari bug we can't get around.

Not reproducible with Safari 9.0. Reproducible with Safari 9.1 and today's Webkit Nightly for Mac.

Cannot reproduce on Safari 9.0 nor on webkit nightly on a MacMini.

Just updated to Safari 9.1 on 10.9, same behaviour. I guess probably 10.9 related.

@hyperknot I saw you being able to reproduce this in a leaflet-less webpage, just <div>s and preventDefault()s. Did you open an upstream bug report?

@IvanSanchez not yet, I will do it. I think it's related to OS X 10.9, but I'm not sure.
Can someone have a look at http://playground-leaflet.rhcloud.com/sow/1/edit?html,output in OS X 10.10 or 10.11 using Safari?

Is it still worthwhile to keep this issue open? If we believe this is an OS X bug in a specific version, and we don't have a workaround, it sounds unlikely this is something we should invest time and energy in.


Replacement for the identfile module

As mentioned in #znc-dev, I'm currently working on a replacement for the identfile module that does not prevent concurrent connection attempts.
The current identfile module locks the ident file and blocks concurrent connection attempts, then directs the identd to reply with the connecting user's configured ident for all of ZNC's outgoing connections, and releases the lock when the connection is established (or the connection attempt fails).
My proposed alternative would require using oidentd, and take advantage of its ability to specify which connection a rule should apply to. For each outgoing connection, it would write a block like the following to the ident file:
to <IP_ADDRESS> fport 6667 from <IP_ADDRESS> lport 37470 {
    reply "foo"
}

This requires us to know both the local IP address and the local port. In a comment to issue #460, it was mentioned that there would be a race condition if the ident file is written after connect() is called. This problem can be avoided by always explicitly calling bind() on the socket. The port to bind to would be specified as 0 to let the OS choose a suitable port. A call to getsockname() will then tell us which port was chosen.
This works well if a bind host is configured, in which case we know all the information needed to completely specify the connection in the ident file.
Problems arise if no bind host is configured:

We need to explicitly bind() the socket to get the local port before calling connect(). However, if no bind host is configured, we need to choose a local address to bind to. This address depends on the remote host: when connecting to an IPv4 host, we would bind to <IP_ADDRESS>, and for an IPv6 host, we would bind to ::.
When using ZNC's threaded resolver, this is not a problem. The resolver will decide which remote IP address to connect to, and we can simply choose the matching IP to bind to before calling TSocketManager::Connect().
When not using ZNC's resolver, however, DNS is handled by Csocket, which would need to be modified for this approach to work.

Csocket resolves the bind host and binds to it before resolving the remote host. This means that we would need to decide on the bind address before we even know whether we are connecting to an IPv4 or IPv6 host. The "obvious" approach would be to swap the CST_DESTDNS and CST_BINDVHOST steps in Csocket, but:
If a remote host has both an IPv4 and an IPv6 address, we would need to bind() multiple times to different addresses, essentially combining CST_BINDVHOST and CST_DESTDNS/CST_CONNECT. I'm not comfortable fiddling with Csocket internals like this myself, especially since this might break other projects using Csocket.

If we bind() to <IP_ADDRESS> or ::, we don't know which local IP address was chosen until after the connect() call. This means that we must omit the from <IP_ADDRESS> part in the ident file, which could theoretically apply the ident to an unrelated connection in a very specific case if there is another outgoing connection from the same user that ZNC is running as. However, I believe this is not an issue (i.e. it cannot happen) if ZNC is run as a dedicated user.

Since I'd like to see this get merged, I wanted to ask for some feedback regarding these two points before spending too much more time on this.
Would it be acceptable to say that the new identfile module can only guarantee race-free operation when using ZNC's threaded resolver, or should I try to find a way to make it work with Csocket's resolver?
Can we simply "ignore" the second issue and tell people not to run other programs under the same user as ZNC? (Even if they disregard this, I believe the probability of this actually causing problems is exceedingly low.)

I'm currently working on a replacement for the identfile module that does not prevent concurrent connection attempts.

Hi, was there any progress?

Would it be acceptable to say that the new identfile module can only guarantee race-free operation when using ZNC's threaded resolver

Looks fine. I still hope to rewrite socket layer though.

Can we simply "ignore" the second issue and tell people not to run other programs under the same user as ZNC?

This can happen already with the current identfile module


Ignore Annotations with empty /Rect-entries in the display-layer (issue 14438)

This prevents the BaseSVGFactory.create-method from throwing, and thus preventing any remaining Annotations (on the page) from rendering in corrupt documents.
Fixes #14438

/botio-linux preview

/botio-windows preview

/botio test

/botio-linux preview

/botio test

Thank you for fixing this!


Dev

Description
Please excuse my first attempts! Still learning... Feel free to trash and make fun of while I'm not around. FWIW, this version:
--Updates styled_nav_drawer.dart to (near) RFP style.
--Moves light/dark mode into a nested pop-up dialog with WYSIWYG radio buttons.
--Adds copyright info in a nested pop-up dialog with app version information also.
--Creates a new wvems_logo_light_simple.svg (that has the small text cropped out) for display on the drawer.
Type of change

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

How Has This Been Tested

[X] I followed every menu tree on my emulator.

Test Configuration:

Firmware version: emulator
Hardware: Pixel_3a_API_30_x86
Toolchain: Dart v201.9317, Flutter 53.0.1
SDK: Android 11.0 x86 (R)

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Thanks for helping!
See my comments above. Changes made as requested, updated to keep all strings in a single class, which will be far easier to maintain down the road.
Will plan to do the same re: themes, fonts soon.


Add section dependencies

so that you can mark that a section "Cannot be started" until it's dependencies have been met.

I'm happy to approve this as the functionality seems to work well, I am however getting some failing tests around 'supcharg' and 'Capybara::ElementNotFound:Unable to find field "case-log-earnings-field" that is not disabled' (for example)


RFC00003 First version of a vulnerability disclosure policy

…as well as a description of our approach to embargoes + a first draft of an Enarx Security Advisories page.
Comments and improvements most welcome.
Closes https://github.com/enarx/rfcs/issues/18.

keybase nowadays provides public links to chat
https://keybase.io//chat

Uh, isn't it a bit strange that this is PR #19, when it should have been https://github.com/enarx/rfcs/pull/4?
@haraldh That redirects to keybase://chat/hellobot, which my browser has no idea what to do with :)

@axelsimon The PR being numbered #19 is because issues also count towards numbering. We've filed ~15 issues and 4 PRs, so this ends up being #19.

@axelsimon The PR being numbered #19 is because issues also count towards numbering. We've filed ~15 issues and 4 PRs, so this ends up being #19.

Makes sense. Thanks Mark.

I made a few suggestions that help with clarity and offer a few ideas to consider.
From an outsider perspective, this RFC is straightforward and easy to understand. In particular, it is thorough in addressing points of contact, timelines, and the structure of the reporting process. As long as vulnerabilities receive attention and the security researcher reporting a vulnerability does not feel ignored I think it would work quite well.
I can also go through and make some suggestions for style and consistency if that it worthwhile - I don't want to be pedantic because the RFC is clear in its message as-is.

I've finally managed to include all suggestions (thanks a lot everyone!) as well as rebase in such a way to give credit to all of you (hopefully) and clean up the commit history down to a few logical commits that, i think, make sense.
This RFC's directory has also been moved to the root of the RFC directory, as we now have done away with the "concepts" etc. subdirectories.
Please review and hopefully we can merge this soon!

It is currently RFC 00004, but if it gets merged before others, especially Mike's currently ones going for #2 and #3, it looks like it should be renumbered. Honestly, i'm tempted to leave this one as #4 unless someone feels strongly about it.

It looks like that's causing the "name" test to fail. Is this correct, @mbestavros?

@axelsimon The name test is sequential, and it's based on your current tree.
Since the only merged RFC at this point is 00000, you'll want to update it your number to 00001 to make the tests pass (it's currently 00004). If another RFC gets merged before this one, you'll need to update the number to 00002, etc.

Thanks for the review!

I am also pretty sure that we will require the current 3 commits to be squashed to one before merging.

I thought 3 commits (creation, inclusion of improvements, move to PROPOSED state) was an accurate description of history, with neither too much useless detail nor too much simplification. Looking at the commit history, the Chat RFC had 5 commits, so i think the situation here is in line with that.


Routing table null record

Thanks for submitting an issue to gorouter. We are always trying to improve! To help us, please fill out the following template.
Issue

Failure of Gorouter routing table

while true; do curl -s http://admin:1qtyuydjdfg6574r$@localhost:8080/routes>/tmp/gorouer.jq && sleep 1s && echo "#######################$(date)########################" | tee -a /tmp/gorouter.poll&& /var/vcap/packages/jq/bin/jq -r "."machine.apps.toto.com"[0].address" /tmp/gorouer.jq | tee -a /tmp/gorouter.poll ; sleep 1s; done
<IP_ADDRESS>:60042
#######################Fri May 26 20:31:57 UTC 2017########################
<IP_ADDRESS>:60042
#######################Fri May 26 20:31:59 UTC 2017########################
null
#######################Fri May 26 20:32:01 UTC 2017########################
null
#######################Fri May 26 20:32:03 UTC 2017########################
null
#######################Fri May 26 20:32:05 UTC 2017########################
null
#######################Fri May 26 20:32:07 UTC 2017########################
null
#######################Fri May 26 20:32:09 UTC 2017########################
null
#######################Fri May 26 20:32:11 UTC 2017########################
<IP_ADDRESS>:60034
#######################Fri May 26 20:32:13 UTC 2017########################
<IP_ADDRESS>:60034
#######################Fri May 26 20:32:16 UTC 2017########################

The time of null output  matches exactly with  the Haproxy 404 HTTP error
May 26 20:32:05 localhost haproxy[25774]: 9ba9f570-2216-4024-ab45-8c4661996aba <IP_ADDRESS> - [26/May/2017:20:32:05.852] "POST /status HTTP/1.1" 404 [reqId:"9a2a0cd81b556a38771c63fd7dac614f84978bc53c9a017923d879975d0d3bf0"] "" 12 "node1-<IP_ADDRESS>"

....

May 26 20:32:16 localhost haproxy[25774]: 9ba9f570-2216-4024-ab45-8c4661996aba <IP_ADDRESS> - [26/May/2017:20:32:16.504] "POST /status HTTP/1.1" 404 [reqId:"cf2b52b773a8ba5f6a414ec05abc919b10227d7f011e8f5f1c0e2872487d6310"] "" 18 "node0-<IP_ADDRESS>"

Context
CF stress test. We run a stress test against CF diego. We have noticed some requests return 404 or 502 HTTP error code.
Steps to Reproduce
Perform a high traffic against CF, and you will see the errors...
Expected result
All requests should return 200 HTTP code
Current result
For small period of time we got a bunch of 404 HTTP error code
Possible Fix
[not obligatory, but suggest fixes or reasons for the bug]
name of issue Output Results
2017-05-24T20:06:15.617 502 Bad Gateway
2017-05-24T20:06:15.621 502 Bad Gateway
2017-05-24T20:06:15.625 502 Bad Gateway
2017-05-24T20:06:15.633 502 Bad Gateway
2017-05-24T20:06:15.636 502 Bad Gateway
2017-05-24T20:06:15.639 404 Not Found
...
2017-05-24T20:07:07.803 404 Not Found
2017-05-24T21:36:57.416 401 Unauthorized
2017-05-24T21:48:22.425 502 Bad Gateway
2017-05-24T21:48:22.474 404 Not Found
...
2017-05-24T21:49:08.451 404 Not Found
2017-05-24T22:12:27.197 502 Bad Gateway
2017-05-24T22:12:27.219 502 Bad Gateway
2017-05-24T22:12:27.239 404 Not Found
[if relevant, include a screenshot]

@fghorbel :
Thank you for filing this issue. We run performance tests against gorouter in our CI but as a standalone job instead of gorouter in CF context.
Can you give us the more details about the test setup

number of requests
concurrency
Number of cells
Number of routers
Is the client doing the stress test within the network(targetting gorouter directly) or through the Load balancer or haproxy?
What is the backend application used for the test?

Regards
Shash

@Shash
Thank you for you answer, it turned out it s out of memory issue. We didn t give the application enough memory for that reason, the container got stopped and restarted.
Thank you

@fghorbel
Thats great. If an application container is out of memory, gorouter will fail to establish any TCP connections which explains the 502s.
I am going to go ahead an close this issue for now. Feel free to open another issue if you have any problems.
Regards
Shash


Aspect Ratio of 16:9 DVD movies always displayed in 4:3

I have an issue / a question (pick one) about Vlc.DotNet.
Generic information

Vlc.DotNet version : 3.0.0 develop 261
Vlc.DotNet project used : WPF
libvlc version : 3.0.1 x86
.net version : 4.6.2
Project language : C#
Project build architecture : AnyCPU
Operating system : Windows 7 and 10 - x64

Summary
I have a lot of mkv-movies in 16:9 aspect ratio (from DVD - anamorphic 16:9 with a resolution of 720x576).
They are displayed in VLC Player and Vlc.DotNet.Forms in the correct 16:9 aspect ratio AUTOMATICALLY.
Using Vlc.DotNet.Wpf these movies are always displayed in the wrong 4:3 aspect ratio.
I've checked this wrong behaviour also with your samples applications.
Same result - with Forms correct - with WPF incorrect.
Am I doing something wrong ?
Do I have to specify in WPF some additional options for the MediaPlayer class ?
Thank you
Bye
Erwin

Hi,
Sorry for the delay, I was doing something else when I received your e-mail and totally forgot to reply.
This is a case I never tested when I implemented the Wpf control. When I try to get the file ratio, I divide the width by the height here : https://github.com/ZeBobo5/Vlc.DotNet/blob/eda62b2cec210161aa9ea7b40029185bb72dcae4/src/Vlc.DotNet.Wpf/VlcVideoSourceProvider.cs#L128
I don't know if I have any other options for that. Maybe this is related to #375 : We may need to add a AspectRatio property somewhere so that we can override the aspect ratio.
Do you have a small sample file that can demonstrate that behavior, so that I can take actions to mitigate that?

Here is a 3sec sample file.
AAA.zip
Bye
Erwin

From what I've seen, the issue is more serious than expected :
When I'm in the VideoFormat callback I know the width/height in pixels, but not the "target" aspect ratio. I think that an info is missing in libvlc to reliably get that info. We could just get the first track and get its aspect ratio, but that feels terribly wrong (there might be more than one video track).
I'll ask what VLC developers think about that, but I don't expect anything before VLC 4.0.0... In the meantime you have several options:

Use WinForms in your Wpf control
Use an Image which you set at the correct aspect ratio, and wire the things yourself, as suggested in #375 .

Hello Jeremy,
I think there must be some kind of a flag (Anamorphic-Flag) in the video-stream or libvlc. If this flag is set the video must be stretched horizontally. According to the resolution the aspect ratio is correct.
Bye
Erwin

VLC itself knows what is the aspect ratio and scales accordingly in other vout modules, but the "vmem" video output module doesn't pass that info to my callback, so I really don't know how to solve that issue.

Or maybe is there a bug in the vmem plugin : The dimensions passed here might need to take the aspect ratio into account?

Found this issue while trying to fix your bug in #526 , stay tuned.


Add manifest for Dns Jumper

I wrote this app manifest for myself, but I think this little and useful software belongs here (I was quite surprised it wasn't here already, actually. Is there maybe already another app that makes this one redundant?).
Also, this is one of my first attempts in writing an app manifest. I believe it's fine, but please give me a feedback if something is wrong and/or can be improved.

Nice 👍
Made some small adjustments :smile:

Thank you! However, I'm afraid that the author did not update the SHA1 on the website 😕

@benplm dang it, one time I forget to check if the installation works and it bites me. 😁


added chatbot for enhanced user experience

What does this PR do?

Fixes #45
@sharmavikas4 please review this pr thanks!
https://github.com/user-attachments/assets/abea13a9-2390-4f54-9eed-657c26366b59

Type of change

Bug fix (non-breaking change which fixes an issue)
Chore (refactoring code, technical debt, workflow improvements)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How should this be tested?

[ ] Test A
[ ] Test B

Mandatory Tasks

[ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@sezallagwal It is only showing the chatbot but it is not rendering anything else. A plain white background with chat bot only.

@sezallagwal but i am getting a white screen. Is there anything additional i have to do while setting up server and frontend

@sharmavikas4 no nothing, check the auth setup it might be a backend issue and check logs what error you're getting

There is no error i am getting

@sezallagwal i am getting a white screen only


Clippy/autofix

Codecov Report

Merging #522 (e627663) into tests/remove-globals (2ca8356) will increase coverage by 0.09%.
The diff coverage is 42.66%.

@@                   Coverage Diff                    @@
##           tests/remove-globals     #522      +/-   ##
========================================================
+ Coverage                 55.04%   55.14%   +0.09%
========================================================
  Files                       165      165
  Lines                     14574    14544      -30
  Branches                   4027     4022       -5
========================================================
- Hits                       8022     8020       -2
+ Misses                     2349     2326      -23
+ Partials                   4203     4198       -5

Flag
Coverage Δ

integration-aries-vcx
41.94% <33.57%> (+0.21%)
:arrow_up:

unittests-agency-client
23.63% <30.00%> (+0.02%)
:arrow_up:

unittests-aries-vcx
51.33% <31.57%> (+0.13%)
:arrow_up:

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

Impacted Files
Coverage Δ

agency_client/src/agency_client.rs
20.00% <0.00%> (+0.21%)
:arrow_up:

agency_client/src/api/downloaded_message.rs
0.00% <ø> (ø)

agency_client/src/api/messaging.rs
0.00% <ø> (ø)

agency_client/src/api/onboarding.rs
0.00% <ø> (ø)

agency_client/src/httpclient.rs
8.33% <0.00%> (ø)

agency_client/src/internal/messaging.rs
23.63% <ø> (ø)

agency_client/src/messages/a2a_message.rs
22.41% <ø> (ø)

agency_client/src/messages/create_key.rs
61.40% <ø> (ø)

agency_client/src/messages/update_com_method.rs
32.35% <ø> (ø)

agency_client/src/messages/update_message.rs
0.00% <0.00%> (ø)

... and 62 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Some changes and fix

add User delete & update & index tests

@amir-ys
Can you add this pr again?


Fix the path to the cargo artifact directory for third_party/

Cargo's artifacts are supposed to be in the cargo/ subdirectory of build/userspace/, not directly in build/userspace/.
----------------------
`make prtest` summary:
----------------------
git rev-parse HEAD
45f75cb6c3773ff5c865e973ed1ac42ae5fa93b5
git status
On branch tp-cargo-fix
Your branch is up to date with 'origin/tp-cargo-fix'.

nothing to commit, working tree clean

I think this is still relevant.


Cannot run the docker-compose sample

Using the docker-compose sample I have :
Pulling app (stavshamir/springwolf-kafka-example:0.1.0)...
ERROR: The image for the service you're trying to recreate has been removed. If you continue, volume data could be lost. Consider backing up your data before continuing.

Continue with the new image? [yN]y
Pulling app (stavshamir/springwolf-kafka-example:0.1.0)...
ERROR: pull access denied for stavshamir/springwolf-kafka-example, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Thanks for raising this issue. Apparently the image was indeed removed from dockerhub.
I uploaded the new version (0.1.1) which should now work according to the README and docker-compose.
Please let me know if there are any other issues.


Request for a TokenClient.RequestResourceOwnerPasswordAsync without Client Credentials

One of our scenario start with an authenticated client (the client authenticate itself with a piece of code like that):
var tokenClient = new TokenClient(disco.TokenEndpoint, clientId, clientSecret);
var tokenResponse = await tokenClient.RequestClientCredentialsAsync(scope, extra: extraParameters);

and it returns a valid JWT to the client.
At a certain point the client needs to authenticate a User using the "resource owner password" flow (yes, we know it is only for legacy & trusted apps).
In order to do that we are currently using a piece of code like this:
var tokenClient = new TokenClient(disco.TokenEndpoint, clientId, clientSecret);
var tokenResponse = await tokenClient.RequestResourceOwnerPasswordAsync(username, password, scope, extra: extraParameters);

and it works.
Since the client has a valid JWT (generated by the first RequestClientCredentialsAsync) we are wondering if there is a way to pass that JWT to the TokenClient instead of the ClientId & ClientSecret.
We know that this technique will not give any bit of security more then your approach but doing this way (generating a valid Client JWT and use it anyway instead of ClientId & ClientSecret) will allow us to reach 2 important points:

We will have one single "AllowAnonymous" Api to generate the client JWT and all the others Api will be authomatically protected by the Bearer Authorization.
We will have less "Secret"/"Password" parameters flowing through the network and it will be easier keep safe all the log & trace we are writing

Digging into the code I discovered the potential enhancement is not strictly related to the client side but also the Token Endpoint should be able to accept the JWT as an alternative form of client credentials, and, in case validate it.
I'm going to open an Issue also in IdentityServer4 linking this one

There's even a "standard" for that - you can pass a JWT like this:
https://github.com/IdentityServer/IdentityServer4.Samples/blob/712534ce4825eae93ac00180e17bcab7106dff3a/Clients/src/ConsolePrivateKeyJwtClient/Program.cs#L37-L43
IdentityServer has a parser for that - you would need to add some custom validation I guess for your scenario.
parser:
https://github.com/IdentityServer/IdentityServer4/blob/dev/src/IdentityServer4/Validation/JwtBearerClientAssertionSecretParser.cs
sample validator:
https://github.com/IdentityServer/IdentityServer4/blob/dev/src/IdentityServer4/Validation/PrivateKeyJwtSecretValidator.cs


pg-entity still requires the primary key to be the first item in the entity to be the Primary Key even with the primary key being set via deriving GenericEntity

data Project = Project
  { createdAt :: ZonedTime,
    id :: ProjectId,
    title :: Text,
    ...
  }
  deriving (Show, Generic)
  deriving anyclass (FromRow, ToRow)
  deriving
    (Entity)
    via (GenericEntity '[Schema "projects", TableName "projects", PrimaryKey "id", FieldModifiers '[CamelToSnake]] Project)

It seems pg-entity still requires the primary key to be the first item in the entity. And in some situations, it's not possible to do this, since it conflicts with the order of fields in the record, which FromRow relies on. Apparently From Row relies on field order and not field names.
So I get errors such as this: Incompatible {errSQLType = "timestamptz", errSQLTableOid = Just (Oid 80156), errSQLField = "created_at", errHaskellType = "UUID", errMessage = ""} from pg-entity trying to treat the first field as UUID (id/ProjectId), even though the first field is created_At/ZonedTime

Good catch, thanks

@tonyalaribe Can you give me precisely the code that you wrote that led to this error? :)

data Project = Project
  { createdAt :: ZonedTime,
    updatedAt :: ZonedTime,
    deletedAt :: Maybe ZonedTime,
    active :: Bool,
    id :: ProjectId,
    title :: Text,
    description :: Text,
    hosts :: Vector.Vector Text
  }
  deriving (Show, Generic)
  deriving anyclass (FromRow, ToRow)
  deriving
    (FromJSON, ToJSON)
    via DAE.CustomJSON '[DAE.OmitNothingFields, DAE.FieldLabelModifier '[DAE.CamelToSnake]] Project
  deriving
    (PET.Entity)
    via (PET.GenericEntity '[PET.Schema "projects", PET.TableName "projects", PET.PrimaryKey "id", PET.FieldModifiers '[PET.CamelToSnake]] Project)

selectProjectsForUser :: Users.UserId -> PgT.DBT IO (Vector.Vector Project)
selectProjectsForUser = query Select q
  where
    q = [sql| select pp.* from projects.projects as pp join projects.project_members as ppm on (pp.id=ppm.project_id) where ppm.user_id=? order by updated_at desc|]

when I move id out of the first position, I then this in my logs:
[SELECT] select pp.* from projects.projects as pp join projects.project_members as ppm on (pp.id=ppm.project_id) where ppm.user_id='a73c227f-38ab-4e91-9e0c-f8cd269254bc' order by updated_at desc
Incompatible {errSQLType = "uuid", errSQLTableOid = Just (Oid 82666), errSQLField = "id", errHaskellType = "ZonedTime", errMessage = ""}

Another example:
data Endpoint = Endpoint
  { createdAt :: ZonedTime,
    updatedAt :: ZonedTime,
    projectId :: Projects.ProjectId,
    id :: EndpointId,
    urlPath :: Text,
    urlParams :: AE.Value,
    method :: Text,
    hosts :: Vector.Vector Text,
    requestHashes :: Vector.Vector Text,
    responseHashes :: Vector.Vector Text,
    queryparamHashes :: Vector.Vector Text
  }
  deriving (Show, Generic, Eq)
  deriving anyclass (FromRow, ToRow, Default)
  deriving
    (PET.Entity)
    via (PET.GenericEntity '[PET.Schema "apis", PET.TableName "endpoints", PET.PrimaryKey "id", PET.FieldModifiers '[PET.CamelToSnake]] Endpoint)

endpointsByProject :: Projects.ProjectId -> PgT.DBT IO (Vector.Vector Endpoint)
endpointsByProject pid = selectManyByField @Endpoint [field| project_id |] pid

endpointById :: EndpointId -> PgT.DBT IO (Maybe Endpoint)
endpointById = selectById @Endpoint

Here, I move Id out of the first position, and if I call endpointById, I get the following:
[SELECT] SELECT endpoints."created_at", endpoints."updated_at", endpoints."project_id", endpoints."id", endpoints."url_path", endpoints."url_params", endpoints."method", endpoints."hosts", endpoints."request_hashes", endpoints."response_hashes", endpoints."queryparam_hashes" FROM apis."endpoints" WHERE "created_at" = 'c70409e2-bcb2-4902-a3f7-802946116569'
SqlError {sqlState = "22007", sqlExecStatus = FatalError, sqlErrorMsg = "invalid input syntax for type timestamp with time zone: \"c70409e2-bcb2-4902-a3f7-802946116569\"", sqlErrorDetail = "", sqlErrorHint = ""}

In this example, it seems pg-entity is using created_at as the primary key in selectById.
But surprisingly, endpointsByProject works with no issues.

Okay perfect, thank you. :)

@tonyalaribe You can use commit 656edce and see if this fixes your problem. :)

I tested it and it works :) . Thanks!!

 Thanks again for hitting those cases, it's very helpful. :)


[Snyk] Upgrade @loopback/rest from 14.0.0 to 14.0.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @loopback/rest from 14.0.0 to 14.0.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2024-04-09.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Open Redirect SNYK-JS-EXPRESS-6474509
305/1000  Why? CVSS 6.1
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: @loopback/rest

14.0.1 - 2024-04-09

14.0.0 - 2024-03-04

    from @loopback/rest GitHub release notes

Commit messages

Package name: @loopback/rest

2bdb95f chore: publish release
5839f5e refactor(sequelize): use flag on SequelizeDataSource for JSON parsing
de82454 chore: remove redundant assertion
26b1277 test(sequelize): add test coverage for JSON parsing for mysql
6b0d0fb refactor(sequelize): restrict the JSON parse to mysql dialect
7b8e193 fix(sequelize): parse JSON columns to objects for non-Postgres dialects
d129a5f chore: update dependency @ types/node to ^16.18.95
c818e38 chore: update lint packages to ^7.6.0
d79e95d chore: update dependency loopback-connector-soap to ^8.0.5
c5dc9e3 chore: update dependency loopback-connector-postgresql to ^7.1.0
f259203 chore: update dependency swagger-ui-dist to v5.14.0
b82b1ea chore: update github/codeql-action action to v3.24.10
0f139e1 chore: update dependency loopback-connector-mysql to ^7.0.9
33a1961 chore: update dependency path-to-regexp to ^6.2.2
2c459f7 chore: update dependency eslint-plugin-eslint-plugin to ^5.5.1
24fc8dd chore: update dependency loopback-datasource-juggler to ^5.0.8
b192cf1 chore: update dependency sequelize to ^6.37.2
96c127a chore: update dependency @ types/node to ^16.18.94
71c11a6 chore: update dependency express to ^4.19.2
b226e7d chore: update dependency @ types/serve-static to v1.15.7
cd25c58 chore: update dependency mocha to ^10.4.0
d13aa15 chore: update dependency testcontainers to ^10.8.1
211d601 chore: update dependency webpack to ^5.91.0
14051e4 chore: update dependency glob to ^10.3.12

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Part of https://github.com/sourcefuse/loopback4-ratelimiter/pull/175


Unable to promisify localStorage

Hi,
I want to promisify the localStorage object.
using like this  -
const get = promisify(localStorage.getItem)
Can you tell me what i am doing wrong ?
Thank you

Hi @paras-ziggletech , localStorage functions are synchronus methods, so there is no need to promisify them.
Any function which accepts a next callback as latest argument, can be promisified.
Use case:
await asyncFunction();
const value = localStorage.getItem("key");
// other calls here...


remove internalQueryArgs

This would address #154 and #155 in a more broad manner.
Also, I added endpointDefinition to serializeQueryArgs and renamed endpoint to endpointDefinition or endpointName everywhere to be more consistent.
Hope I didn't change a public API with it :sweat_smile:  (apart from serializeQueryArgs)
@msutkowski could you take a look at this? Also, @gfortaine, can you please test if this would solve your problem?

Same :sweat_smile:

Same :sweat_smile:

@phryneas We do confirm that it solves our problem 👍 : https://codesandbox.io/s/rtk-query-demo-forked-hd08r?file=/src/app/services/posts.ts

@phryneas We do confirm that it solves our problem 👍 : https://codesandbox.io/s/rtk-query-demo-forked-hd08r?file=/src/app/services/posts.ts


Use httplug instead of old adapter

Relink this library to new interface of httplug
Should we rename all "adapter" to client ?

Thanks, I will try this out with one of the existing adapters, once it is updated.

Should we rename all "adapter" to client ?

No, I don't think it is necessary. They are still adapters of other, existing clients.

It's currenlty running against https://github.com/joelwurtz/socket-adapter-temp (with the last commit)

Ah, cool. Not just cool, awesome.

Actually i was having random errors, i have to force the content length header is the tests (to have valid requests) otherwise the php server will throw random errors (for my library)

Hm, what kind of errors?

400 Bad request on the PHP server launched by the script
It is mainly due to my adapter which write body by using a buffer (so i do not write the full request on the same call).
So when the php server read the request it may considered it finish but it is not.
By setting the content-length header we told to the server to wait for X bytes before considering the request finish (which is the correct behavior) the other solution would be to send the request as a chunked one but this is another subject ^^

Great. I don't think it is a hack anyway, so I am fine with having this in the tests by default.


A11y_AzureToolsStorageExplorer_Use a SAS URL_Attach with SAS URL_ScreenReader: Voiceover does not read the title of the dialog when select the Use a shared access signature (SAS) URI radio button and next.

“Check out Accessibility Insights! Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11y_AzureToolsStorageExplorer; #A11yMAS; #A11YTCS; #DesktopApp; #WCAG1.3.1;#WCAG2.4.2 #VoiceOver; #Mac;
Environment Details:
Application Name: Azure Tools Storage Explorer
Application Version: 1.10.0
MAC Version 10.14.6 Beta(18G29g)
Repro Steps:

Launch Storage Explorer
Navigate to "Open Connect Dialog" pane present on left side of screen and hit enter.
Select "Use a SAS URL" radio button and hit enter on Next button.
"Attach with SAS URL" screen gets opened.
Start VoiceOver and using Tab key and check whether the Voiceover reads the title of the dialog when opened.

Actual:
Voiceover does not read the title of the dialog when select the Use a shared access signature (SAS) URI radio button and next.
Same issue is observed for the below screen.
Attach with Azure AD

Launch Storage Explorer
Navigate to "Open Connect Dialog" pane present on left side of screen and hit enter.
Select "Add a resource via Azure Active Directory(Azure AD)" radio button and hit enter on Next button.
"Attach with Azure AD" screen gets opened.

Expected:
VoiceOver should read the title as Microsoft Azure Storage Explorer Connector Attach with SAS Url dialog .
UserImpact:
Screen reader users unable to know the which dialog opened if it will not read the title.
Attachment for Reference:
1975_A11y_AzureToolsStorageExplorer_Use a SAS URL_Attach with SAS URL_ScreenReader.pptx

The above logged issue is observed for all popup present in entire application.

@nirmalsahoo and @Suraj1200499 I'm confused by this issue. When I use VoiceOver (see attached), there are no problems with speaking that the current window is the connect dialog. As you can see in my recording, VoiceOver says: "Microsoft Azure Storage Explorer, Microsoft Azure Storage Explorer - Connect, window, URI:, edit text".

@nirmalsahoo and @Suraj1200499 I'm confused by this issue. When I use VoiceOver (see attached), there are no problems with speaking that the current window is the connect dialog. As you can see in my recording, VoiceOver says: "Microsoft Azure Storage Explorer, Microsoft Azure Storage Explorer - Connect, window, URI:, edit text".

@nirmalsahoo and @Suraj1200499 can you please follow up?

@MRayermannMSFT
When Screen gets Opened VoiceOver should say the exact title of the dialog.
For eg. possibly, VoiceOver should announce, "Microsoft Azure Storage Explorer - Connect, Attach with SAS URL window".
Same issue is Observed in the entire application for all the dialogs.

@suraj1200499 , that is not the title of the dialog. The title is "Connect". Which in my gif you can see VoiceOver speaking. Attach with SAS URL is the title of the current page.

@nirmalsahoo and @Suraj1200499 can you please follow up?

@nirmalsahoo and @Suraj1200499 can you please follow up?

We have verified the issue and is still reproduced. Voiceover should read the title as Attach with SAS URI.This issue is reproduced throughout the all dialog of the application.

I understand that you still see the behavior, what I'm trying to say is that the behavior is by design. The title of the dialog is "Connect". The title of the page is "Attach with SAS URI" That information is available to all users with the page title being printed on the page.

@MRayermannMSFT
The bug is By design, As per our process By design bug will be validated by our SME's. So I had put this bug in their note. Once I get a reply from them, I will update the comment.

@nirmalsahoo, I went ahead and joined office hours this week with Ryan. He agrees that the title of each page does not need to be added to the title of the dialog itself. Also, fyi, for the other 2 bugs you and I have been disagreeing on (#2069 and #2093), I'm going to take one final look at those.

Closing as by design.

GithubTags:#Closed;

GitHubTags:#BM-TCS-StorageExplorer-Mac-Sep2019;


Add meeting links to SEF handbook Engineering page

Purpose

The purpose of this PR is to fix https://github.com/sef-global/sef-site/issues/1373
Goals

Adding meeting links to SEF handbook Engineering page
Approach

Added meeting links to SEF handbook Engineering page
Screenshots

Checklist

[ ] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[ ] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

Learning

@anjula-sack @Piumal1999 @jayasanka-sack  how about this?


Add error handling reference to README

PR summary
Add a section about error handling.
Link Cloudant API docs error handling section.
Note: The referred link content will be visible only for those who has Cloud subscription.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the
Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

No behaviour change.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Thank you for reviewing @bessbd  and @eiri ! I am about to merge this!


error: Sequence: readUint64: unexpected character: \xff

Code:
type Event struct {
	Operation OpCode              `json:"op"` /* OpCode is an int */
	Sequence  uint64              `json:"s"`
	Type      string              `json:"t"`
	RawData   jsoniter.RawMessage `json:"d"`
}
...
	var e *Event
	decoder := jsoniter.NewDecoder(reader)
	if err = decoder.Decode(&e); err != nil {
		return errors.Wrap(err, "decoding message")
	}

Error:
time="2017-09-04T11:00:57+03:00" level=error msg="Event handler failed" error="decoding message: discord.Event: Sequence: readUint64: unexpected character: \xff, parsing 15 ...null,\"s\":n... at {\"t\":null,\"s\":null,\"op\":10,\"d\":{\"heartbeat_interval\":41250,\"_trace\":[\"gateway-prd-main-wkd0\"]}}"

Note, code works 100% fine with stdlib json Decode.
I also tried the stdlib config in jsoniter, without any success.

Fixed it by changing Sequence to a pointer of uint64, still seems to be a deviation from standard library as such I will keep this open, it's up to you guys how you want to keep it, feel free to close if intended.

can you provide a sample json input?

should be fixed now


REPRODUCIBLE.md fullclean produces error

idf.py fullclean all yields:
Executing action: fullclean
Directory '/builds/blockstream/jade/build' doesn't seem to be a CMake build directory. Refusing to automatically delete files in this directory. Delete the directory manually to 'clean' it.
idf.py clean all works (I realize they are very different based on idf.py --help).

These  issues disappear if idf.py and pre-reqs (cmake, etc.) are installed in the host; just wasn't obvious because I jumped right to REPRODUCIBLE.md.


Add the ability to upload more than one file via the file browser

We should give the ability to a user to select one or multiple files for upload when browsing the content of the volume or a host when using an agent setup.

Even more, it would be very nice if you could:

List folders and files in a volume you are browsing
Edit file contents in a volume  (i.e.: showing to the users a command line as a vi/nano or and textarea)

List folders and files in a volume you are browsing

You can already do that when using the Portainer agent.

@deviantony Listing volumes' files is not possible in the standalone Portainer. Am I right?

+1
Having the possibility to create new folders would be really useful too

@mhemrg that's right. Browse volume works if you're using the Agent. Doesn't work for the Portainer local endpoint.
Just in case you've been wondering about that all these years.

@jamescarppe is this anywhere in the docs? it feels like a pro/con for agent vs not :)

Yep, this is mentioned in a few places, notably https://docs.portainer.io/v/ce-2.9/user/docker/volumes/browse :

This feature is only available when running Docker Swarm or the Portainer Agent on the environment.

Would also love to be able to upload folders... as far as I know, you can't upload a directory via volumes.


storefront:watch broken since June

Description
Since June ./psh.phar storefront:watch is broken due to this commit:
https://github.com/shopware/storefront/commit/0bf72847bba83968ce6f83e11cae00f0fdbc5dc6#diff-7fca9b4ec42e199993d4074fef47a6643235268c9e4464ddbf01fdbec6a6db3d
Possible Solution
Revert the deleted line

you can use storefront:hot-proxy instead

you can use storefront:hot-proxy instead

you can use storefront:hot-proxy instead

Your answer is a workaround which does not fix the issue. And this is a bug report and not a shopware forum topic asking for help.
Unless you as an official Shopware employee want to say that the storefront:watch functionality is no longer supposed to work?

you can use storefront:hot-proxy instead

Your answer is a workaround which does not fix the issue. And this is a bug report and not a shopware forum topic asking for help.
Unless you as an official Shopware employee want to say that the storefront:watch functionality is no longer supposed to work?

I've created a pull request for it: https://github.com/shopware/platform/pull/1614

I've created a pull request for it: https://github.com/shopware/platform/pull/1614

hey @audaki
sorry that I tried to help you here, so you might could go on with your work :slightly_smiling_face:
I'm not sure about the intention of the removal. we will clarify that in your PR. thanks for this one, btw :+1:

hey @audaki
sorry that I tried to help you here, so you might could go on with your work :slightly_smiling_face:
I'm not sure about the intention of the removal. we will clarify that in your PR. thanks for this one, btw :+1:

hey @audaki
I spoke to my colleague and indeed the command was removed intentionally. We will remove the regarding PSH command. You can find some more information here: https://issues.shopware.com/issues/NEXT-13399

hey @audaki
I spoke to my colleague and indeed the command was removed intentionally. We will remove the regarding PSH command. You can find some more information here: https://issues.shopware.com/issues/NEXT-13399


Physical flows - remove unused observable (freshness) indicators

Description

Remove observed indicator from Waltz UI code
Remove server side observed (aka 'freshness') indicators from DAO level and Waltz DB.

These indicators were introduced to support a proof of concept (PoC) which has been discontinued.  Removing this code keeps the codebase clean.
Resourcing
We intend to implement this task

NB: Database change:  Removing tableName="physical_flow" column ="freshness_indicator"

Backing out drop column change as it seems to be failing.   Will try it again in 1.42


cloudinary_fileupload() called upon script inclusion

Please see this commit:
https://github.com/cloudinary/cloudinary_js/commit/05cd1ec379f69719e3a01404de813a32351fe8dd
In this way cloudinary_fileupload() is called as soon as jquery.cloudinary.js is loaded which is far too early for initializing blueimpFileupload (at least in case the you wish to customize the options passed like in my case). At this point I haven't initialized api_key and cloud_name or other binding callbacks.
I want to pass more options so I call later on:
$(".cloudinary_fileupload").cloudinary_fileupload(myCustomOptions);

Could the maintainer explain what is the correct usage in case I want to pass my own options such as maxFileSize, dataType, headers?

Closing this issue due to the time elapsed.
Please feel free to contact our support http://support.cloudinary.com or create a new ticket if you have any additional issues.


Fix to allow query arguments in script src

In reference to https://github.com/sethladd/dart_to_js_script_rewriter/issues/13

Thanks! I just merged another patch that fixed this. I simply saw that before this one :)


Bumping staging versions for patch release v0.19.1

Description of changes:
Bumping staging versions for patch release v0.19.1
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approved


Automate instance creation at OpenStack level

Give us a shout if some example Ansible scripts would help.

This is now done in https://github.com/lsst-uk/lasair-deploy


WikiEduDashboard Support and Settings page added.

Description
Fixes #47 Add an option in settings to change between the 2 servers. [Support for Wiki Education Server]
Feedback Email ID<EMAIL_ADDRESS>Terms and conditions link: https://wikiedu.org/terms-of-service/
Privacy Policy Link: https://wikiedu.org/privacy-policy/
Licenses Link: https://creativecommons.org/licenses/by-sa/3.0/
Screenshots:


[SE-NNNN] [WIP] [stdlib] Rename Sequence.elementsEqual

A PR to support a pending proposal to rename Sequence.elementsEqual.

Resolves SR-6102.

SE-0203 is rejected.


VASP parser regression: ValueError('Cannot normalize HDF5 value without context.')

testcase: single_point.zip
I assume this is related to cases with charge density (uploading just the vasprun.xml works fine) and this case this specific case was parsed fine in the NOMAD in the past (it doesn't parse neither with the stable nor staging servers now, but it worked during the latest mass reprocessing)
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/nomad/processing/data.py", line 1507, in parsing
    parser.parse(
  File "/usr/local/lib/python3.11/site-packages/nomad/parsing/parser.py", line 463, in parse
    self.mainfile_parser.parse(mainfile, archive, logger)
  File "/usr/local/lib/python3.11/site-packages/electronicparsers/vasp/parser.py", line 2336, in parse
    self.parse_configurations()
  File "/usr/local/lib/python3.11/site-packages/electronicparsers/vasp/parser.py", line 2274, in parse_configurations
    sec_density.value_hdf5 = np.reshape(
    ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/nomad/metainfo/metainfo.py", line 1083, in __setattr__
    return super().__setattr__(name, value)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/nomad/metainfo/metainfo.py", line 3325, in __set__
    obj.m_set(self, value)
  File "/usr/local/lib/python3.11/site-packages/nomad/metainfo/metainfo.py", line 1208, in m_set
    value = quantity_def.type.normalize(value, section=self)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/nomad/metainfo/data_type.py", line 726, in normalize
    return self._normalize_impl(value, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/nomad/datamodel/hdf5.py", line 118, in _normalize_impl
    raise ValueError('Cannot normalize HDF5 value without context.')
ValueError: Cannot normalize HDF5 value without context.

Might be related to https://github.com/nomad-coe/electronic-parsers/pull/214 so cc @ladinesa

@TLCFEM this is kind of strange that the archive does not have a context.

Do this:
                    sec_density = Density()
                    sec_scc.density_charge.append(sec_density)
                    sec_density.value_hdf5 = np.reshape(
                        np.array(charge_density, np.float64), grid
                    )

Thanks a lot for the quick fix.


Add FunPtr support

I've added support for FunPtr for passing haskell functions as arguments to the Rust FFI

LGTM! I can merge this one and publish a new version on crates.io. After that, you could update your other two PRs so that they can pass the CI tests :)


Air Sonos does not seem to find any UPnP Devices

I have the following system.
Add-on: AirSonos
AirPlay capabilities for your Sonos (and UPnP) devices.
Add-on version: 3.5.1
You are running the latest version of this add-on.
System: Home Assistant OS 9.5  (amd64 / qemux86-64)
Home Assistant Core: 2023.2.5
Home Assistant Supervisor: 2023.01.1
When I start the server it does not seem to ever detect any UPnP Devices anymore, it used to work fine, but I have no clue when it stopped working.
I installed the following HCAS integration - https://github.com/rytilahti/homeassistant-upnp-availability which confirms that my Sonos Speakers are broadcasting their UPnP, so I am not sure what is missing with this Plug-In.

Thank you for the awesome work

In an older thread where someone had similar issue the recommend action is to see if mDNS is working. I can confirm that mDNS is working by reaching the .local name of devices in the network. I also confirmed using Air Sonos from a linux VM, as in, not from the plugin, it works as expected. This seems to be a specific issue with the plugin. Any advice on what I can do to continue troubleshooting this.


Integrating Apollo

Adds Apollo to fit our API's GraphQL and shorten the load time of the site.
Closes #79.

You got some regressions.

Huge chunk of whitespace for recommended media (http://localhost:3002/features/9-11/share-your-story-the-legacy-of-9-11)

Also large chunk of whitespace in October Terror Attack section (http://localhost:3002/10-31-terror-attack)

Error while searching

No summary or preview thumbnail for recommended articles

Also site straight up doesn't load on firefox

White space is because I only seed media for a couple of articles, but, in reality, there is always at least one article in any section that has media.

Nah this is from a db dump of prod
On Tue, Feb 6, 2018 at 16:02 Jason Kao<EMAIL_ADDRESS>wrote:

White space is because I only seed media for a couple of articles, but, in
reality, there is always at least one article in any section that has media.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stuyspec/client-app/pull/80#issuecomment-363563256,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHBFpzcgD4E1HmTxpktScdDxyK0zr9Iwks5tSL3NgaJpZM4RVd-k
.


tools: SCION v2 wireshark plugin

Updated Wireshark dissector plugin for the new SCION header format.
Extensions are not yet supported.

This change is

/rebase


Restructuring: Creating an own repository for VoCol and the other planned initiatives.

Not sure what this means – VoCol already has a repository of its own. Or do you mean a GitHub organization of its own? This is possible. One could then even move the mobivoc/vocol repository to new_organization/vocol. But this will break the old repository URL, so let's not do it while our paper is still under review.

The problem is: I do not want to set up the two others repos as part of the mobivoc project. I would propose to create a new organization with the two new inivatives (lucid, shopfloor) and we are going to move mobivoc to that place later.

Yes, makes sense. So, let's have a "vocol" organization, and under that organization repositories "vocol", "mobivoc", "shopfloor", etc. But only after the ESWC notification.

I talked also with Sören about it and he proposed the exact same thing :) I created for now the other repositories. The renaming will be done after the ESWC notification

The organisation was now renamed.