Statements not skipped on every iteration

"...the statements between continue and the end of the for body are skipped" but not for every iteration loop. On the final 2 iterations of the for loop, when i == 9 and i == 10, the statements are not skipped. The value of i is printed to the console. This document doesn't specify this detail incorrectly, it just may be helpful to clarify this fact.

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

ID: fb5b436f-bd9b-e6c5-9859-f71cf9ee71bc
Version Independent ID: f384b21d-5bee-6a14-d0c8-aadf61691c1e
Content: continue statement - C# Reference
Content Source: docs/csharp/language-reference/keywords/continue.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @jdschwartz87
I've added this to the backlog for when we next update this article.


Multiple Instances

Hi
This is a very generic question.
Is it possible to declare multiple instances of mongo in the configuration such us..
This would be the scenario where multiple mongo instances are running on the same box

mongodb:
name: mongodb1.server.com_ 21017
host: localhost
port: 21017
databases:
- test1
mongodb:
name: mongodb1.server.com_ 22027
host: localhost
port: 27017
databases:
- test1
mongodb:
name: mongodb1.server.com_ 23017
host: localhost
port: 23017
databases:
- test1

Thanks!

Sorry guys, I will answer my own question

mongodb:

name: mongodb1.server.com_ 21017
host: localhost
port: 21017
databases:

test1
mongodb:

name: mongodb1.server.com_ 22027
host: localhost
port: 27017
databases:

test1
mongodb:

name: mongodb1.server.com_ 23017
host: localhost
port: 23017
databases:

test1


MusicBot not automatically muted when entering empty channel

When all are leaving the channel where a MusicBot is running, the MusicBot is muting itself automatically and unmuting itself as soon as someone enters the channel again.
In the config you can pre-define which channel to connect per default. If this channel is empty when the MusicBot is connecting itself to the Mumble server, the MusicBot is not unmuting itself automatically. Only when someone enters the channel, and then leaves it again, the MusicBot is muting itself.
Please add the logic that the MusicBot even on connection to the server/channel can detect if nobody is in the channel and muting itself to not use unecessary bandwith.

Should be fixed with ea9391de.
Please test and report if it works :)

This issue is solved now and working. Well done! Thanks!


Fix include order causing unwanted dllexport symbols

Fixes #493

Thanks for the PR! It's included in the current release https://github.com/RenderKit/embree/commit/bc7cf847bc6912b86cc2ebdcae58616a75183a17


Enzyme's shallow() doesn't recognise the component when decorated with both @observer and @inject

So the workaround for this was usually to use @observer(['store1', 'store2']). Since this approach is now deprecated and probably will be removed at some point, this will be quite an issue.
This is the error:
Invariant Violation: Component(...): A valid React element (or null) must be returned. You may have returned undefined, an array or some other invalid object.
As soon as I remove one of the decorators, everything works as expected.

see the recently closed issues
Op vr 9 dec. 2016 15:17 schreef Daniel Leu<EMAIL_ADDRESS>
So the workaround for this was usually to use @observer(['store1',
'store2']). Since this approach is now deprecated and probably will be
removed at some point, this will be quite an issue.
This is the error:
Invariant Violation: Component(...): A valid React element (or null) must
be returned. You may have returned undefined, an array or some other
invalid object.
As soon as I remove one of the decorators, everything works as expected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/178, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhPiusQ9PC6k1KQBLaZ62aPIGeGZXks5rGWLfgaJpZM4LI-s6
.

(on mobile) short answer use .wrappedComponent
Op vr 9 dec. 2016 16:38 schreef Michel Weststrate<EMAIL_ADDRESS>
see the recently closed issues
Op vr 9 dec. 2016 15:17 schreef Daniel Leu<EMAIL_ADDRESS>So the workaround for this was usually to use @observer(['store1',
'store2']). Since this approach is now deprecated and probably will be
removed at some point, this will be quite an issue.
This is the error:
Invariant Violation: Component(...): A valid React element (or null) must
be returned. You may have returned undefined, an array or some other
invalid object.
As soon as I remove one of the decorators, everything works as expected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/178, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhPiusQ9PC6k1KQBLaZ62aPIGeGZXks5rGWLfgaJpZM4LI-s6
.

Thank you!


BUG: launch Qwen1.5-moe failed on docker

Error message: ValueError: [address=<IP_ADDRESS>:38973, pid=80] The checkpoint you are trying to load has model type qwen2_moe but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.

The qwen1.5-moe requires installing transformers from source code.
pip install git+https://github.com/huggingface/transformers

We just installed latest transformers in docker, you need to install it on your own.

The qwen1.5-moe requires installing transformers from source code.
pip install git+https://github.com/huggingface/transformers

We just installed latest transformers in docker, you need to install it on your own.

use pip install --upgrade transformers upgrade to latest version(now is 4.40.1) is ok

Oh, transformers 4.40.0 has supported it.


Create package builder

Setup minimal builder, that provides resolver for files with new translations.

Now it uses an external tool for building, so no need to set up its own builder. So, closing as useless.


Add a task allowing to trigger an Airflow DAG run and wait for its completion

Feature description
Here is the suggested syntax for the task:
id: airflow
namespace: company.team

tasks:
  - id: run_dag
    type: io.kestra.plugin.airflow.dags.TriggerDagRun
    host: http://localhost:8080
    dag: example_astronauts
    wait: true
    checkFrequency: # same as in the io.kestra.plugin.core.flow.WaitFor task
      interval: PT30S # check every 30 seconds by default
    options: # same as in the o.kestra.plugin.core.http.Request task
      basicAuthUser: admin
      basicAuthPassword: admin
    headers:  # same as in the io.kestra.plugin.core.http.Request task
      authorization: 'Bearer <TOKEN>'
    body: # same as in the io.kestra.plugin.core.http.Request task — allow optionally to override the default conf payload sent to the API request

The implementation should be functionally equivalent to this blueprint https://kestra.io/blueprints/api/236-trigger-an-apache-airflow-dag-run-from-kestra-and-wait-for-its-completion
I recommend using this CLI to spin up Airflow for testing https://www.astronomer.io/docs/astro/cli/get-started-cli
Important aspects for the implementation:

The task can leverage the io.kestra.plugin.core.http.Request task task as a basis —  best to use options and headers to avoid reinventing the wheel for the REST API authentication
Use checkFrequency from the io.kestra.plugin.core.flow.WaitFor task for the polling implementation.
Set all sensible defaults, including adding kestra-specific metadata to the API call triggering an Airflow DAG, as shown in the blueprint:

    body: |
      {
        "conf": {
          "source": "kestra",
          "flow": "{{ flow.id }}",
          "namespace": "{{ flow.namespace }}",
          "task": "{{ task.id }}",
          "execution": "{{ execution.id }}"
          }
      }

This conf is displayed in the DAG run and is helpful for tracking:

@iNikitaGricenko follow-up before closing the issue:

icon is missing on the task
the body property must be dynamic, currently the Pebble expressions are not rendered, see below

id: airflow
namespace: company.team

tasks:
  - id: run_dag
    type: io.kestra.plugin.airflow.dags.TriggerDagRun
    baseUrl: http://host.docker.internal:8080
    dagId: example_astronauts
    wait: true
    pollFrequency: PT1S
    options:
      basicAuthUser: admin
      basicAuthPassword: admin
    body:
      conf:
        source: kestra
        flow: "{{ flow.id }}"
        namespace: "{{ flow.namespace }}"
        task: "{{ task.id }}"
        execution: "{{ execution.id }}"


MH Search typeahead is being affected on us-en ibm.com landing page

Brief description:  We have been noticing an issue with typeahead in the search box on the us-en ibm.com landing page. It appears queries are getting 'overridden?' somehow in typeahead when the search box is used.  This behavior is not happening all of the time but for me personally it's happening pretty frequently.  We did test other locales and don't see the issue we are seeing on the us-en page.  We fear it may be poorly affecting the overall Click Through Rate for masthead search as this 'bug' is changing the visitors' queries.
Is this a feature request (new component, new icon), a bug, or a general issue? Bug

Is this issue related to a specific component? Search box

What did you expect to happen? What happened instead? What would you like to see changed? Queries should not be overridden
What browser are you working in? Chrome
What offering/product do you work on? Any pressing ship or release dates we should be aware of? Search team
Steps to reproduce the issue

Go to https://www.ibm.com
Perform a search and press enter

Additional information
https://user-images.githubusercontent.com/79653961/109180114-8d840900-7758-11eb-8b04-462c8eea5627.mov

Hi @jmkraus213 ,
As an update, we believe we may have found a fix for this issue, and have kicked off a patch release process for this. The behavior of the search was to highlight the first item in the results list once results came back, which we have removed in this patch. This issue became more noticeable as the time for results to return from the typeahead API has increased.
Here is our pinned release plan which you can track. We will be publishing a patch release and announcing to the support slack channel shortly.
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5288
cc: @RaphaelAmadeu @rsebade @ljcarot @RobertaJHahn


Plugin causes "Twig instantiated before Craft is fully initialized" warning

When I enable this plugin in Craft CMS v<IP_ADDRESS> I get to see warnings telling me that Twig is instantiated too early (in the ./storage/logs/ directory or on the commandline when CRAFT_STREAM_LOG is active):
2022-08-03 11:32:08 [console.WARNING] [craft\web\View::createTwig] Twig instantiated before Craft is fully initialized. {"memory":22820336}
This seemed to be caused by the _addTwigExtensions() function that is called from the init-function in UserManual.php. This uses Craft::$app->view->twig->addExtension(new UserManualTwigExtension);, but according to https://craftcms.com/docs/4.x/extend/extending-twig.html#register-a-twig-extension should be Craft::$app->view->registerTwigExtension(new UserManualTwigExtension);
Would it be possible to fix this warning?

This has been addressed in 5.0.2!


Error running gulp build:release

Bug:
running
gulp build:release
/material2/src/lib/index.ts:7:53: Cannot find module '.corea11ylive-announcer'
What is the expected behavior?
build done
What is the current behavior?
Errors
[20:44:33] 'library:build:esm' errored after 2.63 s
[20:44:33] Error: Error at C:/wx/joujou/material2/src/lib/index.ts:7:53: Cannot find module '.corea11ylive-announcer'.
Error at C:/wx/joujou/material2/src/lib/index.ts:8:66: Cannot find module '.corecoordinationnique-selection-dispatcher'.
Error at C:/wx/joujou/material2/src/lib/index.ts:8:88: Hexadecimal digit expected.
Error at C:/wx/joujou/material2/src/lib/index.ts:9:89: Cannot find module '.coreoverlayoverlay-container'.
Error at C:/wx/joujou/material2/src/lib/index.ts:10:44: Cannot find module '.coreoverlaypositionoverlay-position-builder'.
Error at C:/wx/joujou/material2/src/lib/index.ts:11:103: Cannot find module '.coreoverlayposition♂iewport-ruler'.
Error at C:/wx/joujou/material2/src/lib/index.ts:12:89: Cannot find module '.coreoverlayscrollscroll-dispatcher'.
Error at C:/wx/joujou/material2/src/lib/index.ts:13:32: Cannot find module '.coreoverlayscrollscrollable'.
ipple-renderer'.oujou/material2/src/lib/index.ts:14:36: Cannot find module '.core
Error at C:/wx/joujou/material2/src/lib/index.ts:15:154: Cannot find module '.grid-listgrid-tile'.
at new UserError (C:\wx\joujou\tools@angular\tsc-wrapped\src\tsc.ts:33:5)
at check (C:\wx\joujou\tools@angular\tsc-wrapped\src\tsc.ts:81:11)
at Tsc.typeCheck (C:\wx\joujou\tools@angular\tsc-wrapped\src\tsc.ts:175:5)
at C:\wx\joujou\tools@angular\tsc-wrapped\src\main.ts:128:11
at process._tickCallback (internal/process/next_tick.js:109:7)
What are the steps to reproduce?
run gulp build:release
Which versions of Angular, Material, OS, browsers are affected?
fresh git clone

Actually the file material2/src/lib/index.ts doesn't even exists. That's strange, but I'm not familiar with gulp tasks.
I think @DevVersion is responsible for fixing this.

Same issue here

index.ts is generated by the build task and deleted after the build finish

@zigzag95 That is good to know, thank you. Actually multiple gulp tasks are failing with the same error message. I wonder how it's working in CI.

no problem on OSX but errors on Windows

Same thing with npm run demo-app
Win 7 x64
Node 6.9.2/7.8.0
Npm 4.4.4

Thanks. This is a known issue we all experience at the moment. The build process works fine on Mac OS, which let's us think that it is caused by an @angular/tsc-wrapped issue.
I will take a look at it again, otherwise we would need to wait for feedback from angular/angular.

FTI Replacing libs content by a test module and component works fine on w7
I suspect core a11y

Thanks fo

Thank you so much.  Bundler.ts fix worked in Win10!

I think this should be all working now.

Actually it may still be broken by https://github.com/angular/angular/issues/15403. In which case, you can follow that issue for updates.

Hi guys,
I have always the issue on windows 10. Do you have a solution ?
Cheers,

@rtrompier See #15642


Broken Markup fixed

Why:
Closes #21606

What's being changed (if available, include any code snippets, screenshots, or gifs):

Wrapped the command with reusables {% raw %}/ {% endraw %}
Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@codewithdev Thanks so much for opening a PR! I'll get this triaged for review :zap:

It looks like we prevent changes to this file for security reasons 🤔  I'm pinging in the leads for this area to see if we can override this check or need to make the change internally instead: @martin389 @dihydroJenoxide


Support for: GameJolt, EnMasse, Arc, Indigala

Please add support for connecting GOG 2.0 client with these clients if that's at all possible

En Masse game Client
Gamejolt (would be amazing if you could support its notifications as well as friend list/chat)
Arc Game Client with your game library and friend list/chat
Indigala account support to import games in the your library

Aggregated in mentioned task.


net: Finish FIONBIO default action if si_ioctl return OK

Summary
Continue the work: https://github.com/apache/nuttx/pull/6976. https://github.com/apache/nuttx/pull/8728 contain additional information.
Impact
Handle FIONBIO more correctly for socket case
Testing

It seems to work.


#43 change jar file name

@mandy-chessell  I will address them in subsequent prs. This is very much work in progress at the moment - I need to clean up a lot of the code including what you have identified.  I appreciate this early feedback


Manager & Network have inconsistent API casing

The Manager library has inconsistent API casing, half of it being camelCase and the other PascalCase. Plan to move to PascalCase & unify Loader and it's libraries API

Network also has an inconsistent API casing, using : where . is available. This will be fixed in the next update.
Update v1.2.0

Network also has an inconsistent API casing, using : where . is available. This will be fixed in the next update.
Update v1.2.0

Manager & network have both been updated respectfully in update v1.2.0.

Manager & network have both been updated respectfully in update v1.2.0.


Graphite-Carbon processes going down Abruptly with no error

I've recently installed graphite and grafana on same instance and for some reason carbon processes going down with no error. I'm installed python 2.7.5 and graphite 1.1.5 versions.
I'm using the following carbon.conf file.
carbon.txt
Current carbon process is up and running

But intermittently few of the carbon processes are going down and we're missing the metrics.

There must be something in your carbon log or system logs or coredump, etc.

Under carbon-cache-b/query.log
. During that time carbin-cache-b process went down and I didn't see any exceptions in creates.log

Nothing in your system logs, no core dumps? Only time I've seen carbon go down with nothing in it's log was due to the Linux OOM killer.

I don’t have any other system logs, if you see in the first image for carbon process(it’s showing a to h) process but abruptly few of the carbon processes are going down. Can we have a quick meeting regarding this issue.

I don’t have any other system logs

What do you mean?

I've checked the system logs. These are the logs that are created on 05/12/2019 under /var/log folder.

Under messages I've seen one error

And

yup, oom-killer as expected.

That will do it, you're trying to run 8 caches and 2 relays so you probably don't have enough memory for all that on your host.

Currently I’m using 8 core machine with 16gb ram. Could you please suggest
me what’s  the ideal cpu and RAM requirement for running 8carbon cache and
2 relays.
Or else with current 8core machine with 16gb RAM, how many carbon caches I
can run to avoid carbon downtime.
Thanks
Kondala Rao
8127748943
On Wed, May 15, 2019 at 3:46 PM Dan Cech<EMAIL_ADDRESS>wrote:

That will do it, you're trying to run 8 caches and 2 relays so you
probably don't have enough memory for all that on your host.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/graphite-project/carbon/issues/854?email_source=notifications&email_token=ADJ2UND5GXPETIR27N3SUNLPVRZCVA5CNFSM4HNDXSB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVP4WYA#issuecomment-492817248,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADJ2UNCQIKKCU4AGUZTZBJDPVRZCVANCNFSM4HNDXSBQ
.

Thanks piotr and DenCech. Currently I've carbon-cache-a to carbon-cache-h and two relays. Now I've commented carbon-cache-c to carbon-cache-h and restarted the graphite carbon. For some reason it's showing three carbon-cache-processes.

Please see the following file for carbon.conf configuration file
carbon.txt

Could you please suggest me what’s the ideal cpu and RAM requirement for running 8carbon cache and 2 relays.

There are just too many variables to give an simple answer. Disk performance, number of metrics, metric interval, storage schema's, etc.


@cds/angular not playing nice with element bindings

Describe the bug
The element bindings and attributes are not getting reflected correctly by the @cds/angular wrappers to the core elements.
How to reproduce
https://stackblitz.com/edit/angular-ivy-d3mqge?file=src%2Fapp%2Fapp.component.html
Steps to reproduce the behavior:

Start focus on the first link
Hit tab
Expect focus to move to second link, because tag is supposed to be readonly.
Change from using pure Core in the AppModule to using CdsModule.
Start focus again on first link
Hit tab
Observe focus hits tag.
Review the element and see that it does not reflect the readonly state properly into the web component.

Expected behavior
That bindings will work 1:1.
Versions
Clarity project:

[X] Clarity Core
[ ] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[X] v5.x

Framework:

[X] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
ie: Angular 11
Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

Resolved by https://github.com/vmware/clarity/pull/5932


Handle ql/v2 root endpoint

The root endpoint (https://ql/v2/) should not error out, maybe present the api with the different options available

maybe best solved together with https://github.com/smartsdk/ngsi-timeseries-api/issues/94

Hi , I would like to work on this issue.
Thanks

hi @iamarnavgarg you are welcome to contribute!
if you provide a short description of how you plan to handle it (it could be also through an initial pull request updating the swagger doc), we will provide you feedback before you start coding.
cheers

Hi @chicco785 ,
My approach would be as follows:

We will create a new file with api.py present at location /reporter. Here, we will return the list of it's available api's while calling http://ql-url/v2/
Moreover, we will add new path corresponding to /v2/ in the /specification/quantumleap.yml file, so that it will call the api when we hit http://ql-url/v2/ URL.

Please let me know in case of any concerns.

Hi @chicco785 ,
My approach would be as follows:
We will create a new file with api.py present at location /reporter. Here, we will return the list of it's available api's while calling http://ql-url/v2/
Moreover, we will add new path corresponding to /v2/ in the /specification/quantumleap.yml file, so that it will call the api when we hit http://ql-url/v2/ URL.
Please let me know in case of any concerns.

hi @iamarnavgarg
Sounds good. I would also include a new test for the new endpoint.


Changed version of the IPA to "beta"

Instead, leave link to b1, b2, and b3

#141 does this too for reference


Paper, table2

I am impressed with your research. Thank you for your good research.
But I have a question and would like to ask.
According to Table 2 of the paper, success and failure modes are divided.

what is the definition of success mode and failure mode?
if success mode is a successful case, it should not include false positives, because false positives are predicting something wrong as right.
ultimately, Hallucinated reasoning traces or facts are present in both success mode and failure mode. I wonder why?

Thanks!

Thanks @serotoninpm !

Success and failure are simply defined as whether the LLM is able to reach correct answers after the ReAct procedure (measured by the final response).
Therefore, the LM can reach correct answers for the wrong ReAct trajectory. We would like to distinguish such cases from those successful cases where the trajectories are also correct, therefore report "false positive" in the table.
Hallucination can happen anytime, both in success and failure cases (even the model hallucinates at some point, it may still be able to reach the correct answer for the wrong reason). Therefore in both modes we report the hallucination traces.


Tabular Data Drift By Binary Classifier

Hi,
I saw that for text & embedding there is binary classification method:
"Evidently trains a binary classification model to discriminate between data from reference and current distribution"
I want to use same method for tabular data but didn't see it. Is it supported for tabular? If not how can I implement one?
and general question about data drift, can it be run as test in test suite or just report?
Thanks!

Hi @Miriam2040,
Classifier drift detection method
The classifier method is available for embeddings (docs here https://docs.evidentlyai.com/user-guide/customization/embeddings-drift-parameters) and for text data (docs here https://docs.evidentlyai.com/user-guide/customization/options-for-statistical-tests). For text data, it includes text-specific pre-processing.
It is not implemented for tabular data.
If you want to pass a custom drift detection method, here is the explanation of how to pass a custom function: https://docs.evidentlyai.com/user-guide/customization/add-custom-drift-method
Data drift test
Yes, you can use data drift detection as a Test Suite. There is a test preset (DataDriftTestPreset()), and separate tests you can choose from: TestNumberOfDriftedColumns(), TestShareOfDriftedColumns() for the dataset and TestColumnDrift(column_name='name') for individual columns.
You can see them in the example notebooks linked here
https://docs.evidentlyai.com/examples
Or in the all tests list
https://docs.evidentlyai.com/reference/all-tests#data-drift

Great, thanks!


Added Ideal Gas Calculator for calculating various parameters using different combination of input

Fixes Issue🛠

Closes #870
Description👨💻

The Ideal Gas Calculator is a web application that helps you calculate one of the variables (Pressure, Volume, Moles, or Temperature) of an ideal gas using the Ideal Gas Law equation: PV = nRT.

Select the variable you want to find (Pressure, Volume, Moles, or Temperature).
Enter the known values for the other variables.
Supports multiple units for each variable:

Pressure: atm, Pa, kPa, mmHg
Volume: liters, ml, m³
Temperature: K, °C, °F

Perform the calculation and display the result.

Type of change📄

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

How this has been tested✅

I have been working on this calculator since last 2-3 days and have checked for calculations and cross checked its variable combination calculations manually
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
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Calculating all the parameters of an ideal gas

Various input feature

Please review and merge the PR @Rakesh9100
Thanks :)

Pls rename your folder to Ideal-Gas-Law-Calculator in this same PR by making new push commits, don't close this and create a new PR @debangi29

I have done it @Rakesh9100
please check it
Thank Youu :)


webiste width Improper

The width of website is going out of 100% window size.
image is attached for reference

additionally, i will improve
the side box will be visible when the dropdown icon is clicked besides username and all will be in the window size.
@SUGAM-ARORA  i would like to solve this issue, so assign the same to me.

@Rajput-xv Issue already exists


Migration directions: serverurl doesn't exist

Migration instructions say to remove serverurl but all I have on mine is servers . Are these the same (should I remove servers before git pull) ?

Update secnodetracker
Stop the tracker application.
Delete the following files in the secnodetracker/config folder:
nodeid, serverurl, lastChalBlock, lastExecSec, stakeaddr

Why you don’t try remove ‘servers’ and let us know?

'serverurl' was for older versions.  The 'servers' file is currently used.  It should be recreated during setup if it does not exist.

Thank you.


Update storage-quickstart-tutorial-intro-include-powershell.md

Updated this quickstart to:

Add Parameter names
Convert LONG command into a Hash Table
Converted variable names to Pascal case.

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

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

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

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

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

I think I picked them all up! :-)

Thanks @mikefrobbins! Change accepted.

Who has to approve this PR??

#sign-off

@mikefrobbins: I'm sorry - only the author of this article, @tamram, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#label:"aq-pr-triaged"


Pipelines ignoring following name fields on resolvers

Expected Behavior
Have multiple different resources resolved from a specific resolver using name field and concise remote syntax.
Actual Behavior
Following StepActions/Task/Pipeline are ignored and the first declared name is being passed to the following resources.
Steps to Reproduce the Problem
execute a TaskRun with the following:
steps:
      - name: step-1
        ref:
          resolver: some-resolver
          name: scheme://<path>/step-1
        params:
          - name: param1
            value: value1
          - name: param2
            value: value2
     - name: step-2
        ref:
          resolver: some-resolver
          name: scheme://<path>/step-2
        params:
          - name: param3
            value: value3
          - name: param4
            value: value4

Resulting in
extra params passed by Step to StepAction: [params3 params4]

Even though step-2 supports those parameters. Which means that is trying to apply params3 and params4 as if it was using step-1.
Additional Info
Proposed solution
change GetNameAndNamespace in  pkg/resolution/resource/name.go to accept *v1beta1.ResolutionRequestSpec parameter instead of just params
func GetNameAndNamespace(resolverName string, owner kmeta.OwnerRefable, name string, namespace string, req *v1beta1.ResolutionRequestSpec) (string, string, error) { ... }

cc'ing @chitrangpatel

Thanks @Aleromerog !
I agree that this should be fixed. If you have the fix handy, please submit a PR and I'm happy to review it.


Closes #75: check transfers bug

Closes #75: check transfers bug

Add tests for cases reported
Fix parsing of value string
Don't check sender and receipient for transfer calls

This is a quick fix ... we should probably refactor this a little once we have 1.0.0 ... it is not super bad, but we could probably make this nicer (there is also the todo to use enums)


Dev 6.1 Minor fixes

Dev 6.1 Minor fixes
lv_draw_arc.c -> line 218
Removed unsed fast_atan2() function which appears to have been moved to
lv_math.c as lv_atan2() function.
lv_font_heb_16.c -> line 1
Fixed inconsistent include path.
lv_label.c -> line 734
Added (char*) cast to const char* variable 'txt' to silence compiler
warning when LV_USE_BIDI is disabled.
lv_style.c -> line 93
Removed this line
lv_style.c -> line 97
Reverted this line to remove override of LV_FONT_DEFAULT defined in
lv_conf.h

Looks good thank you!


dialogflow: increase timeout for test

Fix failing build by increasing timeout. https://github.com/GoogleCloudPlatform/java-docs-samples/pull/2105

Looks like Dialogflow has some flaky tests, let me investigate.
https://source.cloud.google.com/results/invocations/489e8a59-ed54-4a51-b2ca-5c185cc870c5/targets
https://source.cloud.google.com/results/invocations/2be7950e-77d2-4404-a03c-12ff853948a0/targets


feat(storage): support AuthorityOption

If present, use the AuthorityOption to configure the Host:  header
or the .set_authority() attribute in grpc::ClientContext. By
default, configure AuthorityOption to be storage.googleapis.com,
applications can override this when initializing the storage::Client.
There is already an issue open to add per-call Options that would
provide more fine-grained control of this field.
Fixes #3317.  I am adding a node to #8164 because we will need to undo some of this work to support per-call options.

This change is

Done. Though a slightly different implementation, note that not options.has<AuthorityOption>() implies options.get<AuthorityOption>().empty().

Part of the motivation for checking .has<>() was to avoid constructing the empty string once when the option is not present. But maybe that is not worth the trouble.


User defined formats with standalone validation code

The error we're seeing is:
Error: CodeGen: "code" for formats0 not defined
Oddly this error does not occur if we use Ajv in our app, it only occurs if we try to make a standalone validator script.
What version of Ajv are you using? Does the issue happen if you use the latest version? 7.1.1 (latest)
Ajv options object
const ajv = new Ajv({
  code: {
    source: true, // this option is required to generate standalone code
  },
  removeAdditional: true,
});

JSON Schema

const demoSchema = {
  $id: 'demoSchema',
  additionalProperties: false,
  properties: {
    test: hexCodeValidator,
  },
};

Sample data
N/A - bug occurs during compilation of standalone Ajv.
Your code
Repro
Are you going to resolve the issue?
I cannot resolve the issue.

Custom format functions support with standalone code is limited.
If simply using regular expression (not a function) works for your case - it works for the example, but maybe you have a more complex function, - then passing regular expression directly to addFormat would make it work.
If you do need a function then there is an instance option (see Options in api.md) to which you need to pass a “code snippet” that would result into an object with all formats - see ajv-formats here for the example how it can be done:
https://github.com/ajv-validator/ajv-formats/blob/master/src/index.ts#L55
An obvious improvement is to allow defining custom formats individually and pass “code snippets” for standalone code in their definitions.
It’s not well documented,

@epoberezkin thanks for your speedy reply!
I think regex alone would work for us but I'm not clear on the syntax in that case. I took a look at the docs and I tried the following:
  ajv.addFormat('hex-code', hexCodeRegex);`

and
  ajv.addFormat('hex-code', {
    validate: hexCodeRegex,
  });

but both result in the same error. Any thoughts?

I expected this to work: ajv.addFormat('hex-code', hexCodeRegex);
(assuming hexCodeRegex is RegExp instance)
need to have a look why it’s failing

I made a new repro but it throws an error about unexpected token instead of producing the bug.
The JS validates though... Seems like a Runkit thing?
https://runkit.com/slapbox/603d6ad7a58cfe001a9e648e

@epoberezkin any thoughts on this? Either on the original issue at hand or why the repro won't run?

@epoberezkin I moved the code to CodeSandbox where it doesn't have that error. The resulting error now is: CodeGen: "code" for formats0 not defined
https://codesandbox.io/s/relaxed-bhaskara-5bisx?file=/src/index.js
Any thoughts?

I just updated to 8.0.1 in our actual project to try in a more well known environment and it generates the same error, CodeGen: "code" for formats0 not defined.
We're hoping to push a new release in a couple weeks and this issue is a blocker for us. I'd love to be of more help resolving it but I think all I can do is report findings unfortunately. If there's anything I can do to help, let me know!

The required minimal change is to support formats defined as regular expressions without the need to set code.formats option - the change would be in /lib/vocabularies/format/format.ts - the PR is welcome.
More complex change that I wasn’t planning at all would be to define code snippet per each format, not only on the instance level, to support formats defined with “validate” function. That can be also done for user defined keywords (those that use “validate” function) - this is not in scope of this issue.
The workaround “should be” quite straightforward, as I suggested above - you need to put all your formats in a separate file and pass code snippet with require of this file to Ajv instance via code.formats option - similar to how ajv-formats is doing it - see the link above.

The required minimal change is to support formats defined as regular expressions without the need to set code.formats option - the change would be in /lib/vocabularies/format/format.ts - the PR is welcome.

I'm not following when ajv-formats would be needed for RegExp support versus what ajv should handle out of the box?
Just linking to the file here for convenience: https://github.com/ajv-validator/ajv/blob/master/lib/vocabularies/format/format.ts

Oops, submitted that early and accidentally closed.
I'm not really able to follow what would need to be done for a PR solution unfortunately, and I'm also not the most proficient in TypeScript which makes it a bit harder still to jump right into the code. I'll peruse the docs a bit and see if I can get a more firm understanding of the code.formats option and related functionality.

No worries.

I'm not following when ajv-formats would be needed for RegExp support versus what ajv should handle out of the box?

Ajv should handle regexps out of the box.

code.formats option

As an example you could have a file with all your formats, say formats.js:
module.exports = {
  "hex-code": /.*/,  // or anything you need
  // and you can mix-in ajv-formats here as well, if needed
}

Then when you initialise ajv you would:
const Ajv = require("ajv")
const {_} = Ajv
const ajv = new Ajv({code: {formats: _`require(${path_to_your_formats_module})`}})

This path should be relative to the location of your compiled validation code module - so it would depend on where you place it - it most cases it would not be a variable.
Hope it makes sense

this is not documented really - needs to be added to the docs

Yes - adding to docs would be great. I would amend/amend code example in https://ajv.js.org/guide/formats.html#formats-and-standalone-validation-code (I actually forgot there was something there:) and reference it from options: https://ajv.js.org/options.html#code

@Slapbox RegExp format support in standalone code added in v8.0.2
Still worth amending the docs, for the formats defined as functions.

So the new code example would be like this, right? The current code example doesn't include source even though it's discussing standalone validation, so I've added that and some comments. Let me know if you think the comments might be better mentioned elsewhere instead of the comments.
const { default: Ajv, _ } = require('ajv');

const ajv = new Ajv({
  formats: require('./my_formats'),
  code: {
    source: true, // Generate standalone validation code
    formats: _`require("./my_formats")` },  //  This path must be relative to the compiled standalone validation code at runtime
});

And then on https://ajv.js.org/options.html#code just mention to see the page https://ajv.js.org/guide/formats.html#formats-and-standalone-validation-code for more information on how to use formats? Or link specifically to the sub-heading? https://ajv.js.org/guide/formats.html#formats-and-standalone-validation-code
Could you point me to an example of a non-RegExp custom format? Since our use case is now supported without the extra steps, it wouldn't make for a great example. Is it now basically just formats which provide custom validate functions?

So the new code example would be like this, right?

yes

Or link specifically to the sub-heading? https://ajv.js.org/guide/formats.html#formats-and-standalone-validation-code

I think so - to point to the example there (that would be extended).

Is it now basically just formats which provide custom validate functions?

Yes. Some string formats can be easier to write as a function than regexp.


Update registration methods to use V2 API

Description
Uses the registration V2 API to create , delete, get a registration.
Adds additional features:

update registration
get all registrations for a workspace
get all registrations for an org ( with pagination )

References: https://developer.adobe.com/events/docs/api/#tag/Registrations
Motivation and Context
All APIs were using the V2 model of projects and workspaces except for registrations. This PR migrates the registrations to use the V2 version.
How Has This Been Tested?
E2E and unit tests
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@sangeetha5491 changes look good. Just one concern, these look like breaking changes to aio-lib-events as methods are getting renamed etc.
Will this impact usage of the lib here - https://github.com/adobe/aio-cli-plugin-events ?
Also for the same reason this will have to be a major release for the lib so users don't update to this latest change when they install the lib as npm dependency.

@sandeep-paliwal yes these are breaking changes and will impact its use in the CLI.


Criteria.count doesn't respect limit

Behavior was definitely different in 2.0.2 from 2.4.0.
require "spec_helper"

class Foo
  include Mongoid::Document
end

describe "Test" do

  before do
    [ Foo ].each(&:delete_all)
  end

  describe "works" do

    it "sometimes" do
      2.times { Foo.create! }
      Foo.count.should == 2
      Foo.limit(1).to_a.length.should == 1
      Foo.limit(1).count.should == 1 # fails, returns 2
    end

  end

end

Am I missing something? How do I get the count of the objects for a criteria with a limit without actually executing the full query that returns the documents?

Note that this breaks pagination information with Kaminari.

This might be helpful, the ruby driver count takes a boolean to take limit into consideration.

Mongoid count takes a boolean as well to respect limit:
https://github.com/mongoid/mongoid/blob/master/lib/mongoid/contexts/mongo.rb#L89

Thanks @durran, I can see why you want the default behavior the same as the Ruby driver. Although IMO this was just broken in the Ruby driver and the option was a hack.
For those running into this problem with Kaminari, see https://github.com/amatsuda/kaminari/pull/194

@dblock Yeah I am in agreement there, but just wait until you see what we release in the next couple of weeks... You won't need to use the 10gen driver anymore... :)

Do yo intend to change the need for an unexpected boolean?
It feels really buggy :)

While we're still on the 10gen Ruby Driver, yes... In 3.0 when we are using Moped then it will be gone.

Ok, thanks for the update


Cosmetic changes to rpgdice

Changed text colour to grey while bouncing, changing to white on final bounce except for critical rolls. Critical hit is red text on black, critical miss is black text on red.

Thanks! Please could you just modify apps.json to update the version number as well?

Sorry about that, updated accordingly.

Thanks!


Wishlist for next version: maintain sort selections for re-sorting

Not a priority, would be for next version. It would be nice to maintain sort selections for an image set until it is closed. So if I sort by label, depth, and then recorded timestamp and then clean up localizations, would be nice to re-sort with same parameters that I used without having to reselect those in same order again ...not sure if this is possible:)

Got it, I think this is captured in #49.

Implemented in v0.9.0


Add layer 'Klassifizierte Gewässer Kreis Viersen'

Quickly tested with Firefox and mobile android. Works!
I would suggest to call the layer "Kreis Viersen Klassifizierte Gewässer" so that it corresponds to the other layers called that way.

Thanks @jimmyhenne !

I would suggest to call the layer "Kreis Viersen Klassifizierte Gewässer"

Good point, will do.

Good point, will do.

This would be a breaking change (existing permalinks would not work anymore) and likely the layer will be removed in the near future. We will keep the current name for now.


CRASHING!!!!!!! (My device is a DOPO)

Plz, close this report because, were spending more time dealing with reports than fixing Blocklauncher.
Thx!

Please do not post duplicate issues.


cant edit description on clusters created with rke templates

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

create cluster and select an existing rke template
after cluster is provisioned, edit cluster, modifying description
click save

Result:
the description doesnt get updated or saved. custom clusters work fine, its just when you use rke templates, the description change is omitted.

UI doesnt seem to preserve annotations added via the UI as well.

Reproduced the issue with v2.4.3.

Created a RKE template "template1"
Create a cluster using "template1"
Edit the cluster and add description, labels and annotations and save
The description/labels/annotation do not get updated.

Validated the fix with master-head build (dea919fb2)

Create a RKE template "template1"
Create a cluster using "template1"
Edit the cluster and add description, labels and annotations and save
The description, labels and annotations are updated correctly.
API also reflects the description, labels, annotations.

created": "2020-05-21T00:32:57Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "user-nd9k6",
"defaultClusterRoleForProjectMembers": null,
"defaultPodSecurityPolicyTemplateId": null,
"description": "testdesc",
"desiredAgentImage": null,
"desiredAuthImage": null,
"dockerRootDir": "/var/lib/docker",
"driver": "rancherKubernetesEngine",
"enableClusterAlerting": false,
"enableClusterMonitoring": false,
"enableNetworkPolicy": false,
"id": "c-zpq5b",
"internal": false,
"istioEnabled": false,
"labels": {
"cattle.io/creator": "norman",
"l1": "l2"
},

"annotations": {
"a1": "a2",
"authz.management.cattle.io/creator-role-bindings": "{\"created\":[\"cluster-owner\"],\"required\":[\"cluster-owner\"]}",
"lifecycle.cattle.io/create.cluster-agent-controller-cleanup": "true",
"lifecycle.cattle.io/create.cluster-provisioner-controller": "true",
"lifecycle.cattle.io/create.cluster-scoped-gc": "true",
"lifecycle.cattle.io/create.mgmt-cluster-rbac-remove": "true",
"provisioner.cattle.io/ke-driver-update": "updated"
},


Bug with usage of DataColumn as DataRow index in GroupBy key

wow that is weird indeed.
key["x"] // fine

val x by column<String>()
key[x] // fine

key[df.x] // wrong

One thing that could be relevant is that df.x and x inside groupBy are not the same column. That said, a column of the same type and name exists, so in theory it should just work.

Another hint at what's going on:

Narrowed it down to ColumnReference.resolveSingle(context) being overridden by DataColumn which ignores the df inside context. Thus ColumnReference<C>.resolveFor(df: AnyFrame) simply fails here.
Why is this library so overly complex sometimes...


it need stringio module

I tried to use your source code with python3.5 but it seems it runs on python2. you should give proper description with your source code.

@faststare yes I will add support for higher python versions and description for the code as well

thanks in advance sir.


Enable Istio auth in the bookinfo demo application

@andraxylia @mandarjog Can either of you add me as a member of this project and assign this issue to me?


Add GNU Property Types

Adds GNU Property Type constants.
We are using them in Rust PR to emit notes.
A previous CL added a property constant CL.

let pr_data: u32 = 3; // Program property descriptor

Can you add constants for this value as well (it appears to be architecture specific, for example GNU_PROPERTY_X86_FEATURE_1_IBT | GNU_PROPERTY_X86_FEATURE_1_SHSTK).

let pr_padding: u32 = 3;

It seems unusual to have a non-zero value for padding.

let pr_data: u32 = 3; // Program property descriptor

Can you add constants for this value as well (it appears to be architecture specific, for example GNU_PROPERTY_X86_FEATURE_1_IBT | GNU_PROPERTY_X86_FEATURE_1_SHSTK).

yup, done.

let pr_padding: u32 = 3;

It seems unusual to have a non-zero value for padding.

It also worked with '0' padding so I changed it to that.

Hi, I want to learn more about how crates get updated.  When can we expect this code to be merged into upstream Rust?

Released in 0.31.1

Cool! I'm going to try to incorporate this in and work on a PR. Unless you already had one started?

There's already https://github.com/rust-lang/rust/pull/111413 for updating object.


door sensors and motion sensors, with official integration disappeared

With TUYA v2 I had all the door sensors and motion sensors, with this new official integration everything disappeared, only the switches remain.
There are only 32 sensors...
All devices are there in the IoT, but they don't pass the sensors to the Home Assistant
It's too early to install, I had to go back to TUYA v2

I have Tuya a Window sensor that was working fine with 1.5 but now with 1.6 it's not working like it did before.
It loads but will not see updates in window status (ie open/close).  If I reload the integration it detects the current state.  My TUYA Android app sees the changes in state so why not this integration?
I am running the integration from HACS

The bit that confuses me is
I was assuming that the Tuya V2 in HACS was a beta test.  I then assumed
that the official HA Core version would be the Tuya V2 moving from beta to
production.
Obviously my assumption was wrong but why was it so?
On Fri, 22 Oct 2021 at 00:03, Franck Nijhof @.***>
wrote:

With TUYA v2 I had all the door sensors and motion sensors, with this new
official integration everything disappeared, only the switches remain.
The Home Assistant Core version was released without support for (binary)
sensors, this has now been added and will become available with Home
Assistant 2021.11
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tuya/tuya-home-assistant/issues/576#issuecomment-948650517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALFUCPQPYCVK7BLGWQ4MY6DUIAMU3ANCNFSM5GAB2DLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
CheersRT

@ronjtaylor sorry for the inconvenience. The official HA Core version will catch up with the Tuya v2 version in the Home Assistant 2021.11 release (November 3rd).
After discussing with the Home Assistant team, in order to provide Home Assistant users a great experience using Tuya products, we decided to stop maintain the Tuya v2 version in this repository and only provide the Tuya HA Core version. Trying to maintain the Tuya v2 integration alongside the version in Home Assistant Core is a waste of resources for us.
Please help to create your pull request or issues in the Home Assistant Core repository directly.


update r4ss tests so that they use the latest release instead of .18

Tested running on the r4ss main branch, and the job passed, so closing!


Problem with implementation of setState functions in Media.IdealGases.Common.MixtureGases

Reported by julien.bonifay on 23 Jan 2012 17:25 UTC
This ticket is in relation with some points mentioned in #85 (Re-Design and simplification of Modelica.Media) but only a small modification of the current Modelica.Media seems here to be enough to solve the problem.
The fixedX=true option for mixture media (Media.IdealGases.Common.MixtureGases) is sometimes very useful in fluid models to switch off some performance costing mass fraction equations or simply to not have to write them. Unfortunately the setState functions are not adapted to this eventuality.
The setState functions are often used in fluid models with ports data as inputs. If fixedX is set to true the mass fraction vector at ports will have a size null. However if we look inside the setState functions this case is not being investigated:
state := if size(X,1) == nX then ThermodynamicState(p=p,T=T, X=X) else  ThermodynamicState(p=p,T=T, X=cat(1,X,{1-sum(X)}));
A way to solve this seems simply to test if the size of vector X is null and if yes use then the reference_X vector:
state := if size(X,1) == 0 then ThermodynamicState(p=p,T=T, X=reference_X) else if size(X,1) == nX then ThermodynamicState(p=p,T=T, X=X) else  ThermodynamicState(p=p,T=T, X=cat(1,X,{1-sum(X)}));
In the attachements you will find one test with a simple fluid model using an original mixture medium (extended from Media.IdealGases.Common.MixtureGases) with fixedX=true (not working) and one test with the same fluid model but using a modified mixture medium as suggested above (working).
A correction of this problem could be done before Modelica.Media is entirely revised as suggested in ticket #85. It’s furthermore not easy for a Modelica user to bypass this problem without copying the all Media.IdealGases.Common.MixtureGases package as the setState functions are not replaceable anymore.

Migrated-From: https://trac.modelica.org/Modelica/ticket/689

Comment by dietmarw on 24 Jan 2012 07:12 UTC
duplicate of #691

Modified by dietmarw on 24 Jan 2012 07:12 UTC


Amino-json signing not compatible with cosmjs

PR #44 (shipped in version 1.0.3) changed the amino types path from cosmos-sdk to govgen. This sounded legitimate because govgen.gov is not the same module as cosmos.gov. Unfortunately, cosmjs is not aware of these new amino types and still generates them with the cosmos-sdk prefix, for instance when it needs to generate the bytes-to-sign for the amino-json signing mode (typically when a legder is used). This makes any ledger signature impossible, because the generated bytes-to-sign are differents between the client and the node.
Fortunately most (all?) Govgen validators are still running the govgen node using the version 1.0.1, where the amino types are not renamed. So today, cosmjs is able to sign a tx with a ledger for govgen, but things will stop to work as soon as validators will run version 1.0.3.
@clockworkgr Is there a way to tell cosmojs to use a different type name for amino-json encoding? I looked at their code and it seems to be set in stone. Look for instance at cosmos-sdk/MsgDeposit (which should be govgen/MsgDeposit in govgen 1.0.3)
If compatibility with cosmjs can't be achieved, we should revert #44. The same change should be applied to the AtomOne gov module, and json-signer will also need to be updated.

Simple fix but must ensure all are on 1.0.3....can't have 2 different versions of the same lib

The place for the fix in cosmjs/telescope : https://github.com/atomone-hub/govgen-types/blob/main/scripts/codegen.js (thx @clockworkgr for the info)
But I don't think this needs to be changed now, as we are not going to update the govgen chain. But for AtomOne, where the amino types have been renamed similarly to govgen (but with atomone prefix), we'll have to update them.
For govgen, I think we need to remove/disable the v1.0.3 release, to ensure no validator will upgrade. #44 was a state breaking change, so it should have been shipped in a v1.1 release or something (cc @giunatale).

Finally we decided to go with a new release that revert the change made in #44. So this issue is fixed by #51


[5.7🍒] Add missing function conversion for member access to preconcurrency

Cherry-pick of https://github.com/apple/swift/pull/60521
resolves rdar://96309577

@swift-ci please test


[docs] Make it clear about when to add constraints for generic type parameters

Closing this one as it will no longer be relevant with Java Direct Interop #647.


Fifteen thousand vscode-tscancellation* temp files created

Issue Type: Bug
This is a bit of a weird bug. I noticed that in my Windows %Temp% directory, I have 15,612 separate empty files named vscode-tscancellation-([0-9a-f]+).sock([0-9]{2}):
$ ls -al ~/AppData/Local/Temp/vscode-tscancellation-*.sock* | wc -l
15612

They date back to when I installed VS Code on my computer, and appear to be created during the hours I actively use VS Code. Sometimes over 20 will be created in the course of a minute. I write Python, JavaScript, JSX, and other things in VS Code.
I'm assuming that some process is failing to cleanup old temp files. It looks like the sock([0-9]+) values are sequential, so something is probably creating them, incorrectly thinking the creation failed, trying to recreate it, finding the file already exists, then incrementing the end number so it can be created - and repeating infinitely.
Anyone have an idea what could be causing this?
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:51:32.889Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
7.89GB (0.55GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

python
ms-
2018.6.0

Maybe related to this?
https://github.com/Microsoft/vscode/blob/533c6deb7901fba48df4db8c7adfbd42a6d2fe7a/extensions/typescript-language-features/src/typescriptServiceClient.ts#L977

Looks like it comes from here:
https://github.com/Microsoft/vscode/blob/533c6deb7901fba48df4db8c7adfbd42a6d2fe7a/extensions/typescript-language-features/src/typescriptServiceClient.ts#L798
The seq number just keeps incrementing and so new files are touched over and over again at this LOC.
Not sure what this tscancellation file does, I'm assuming some other process checks if it exists and if so terminates?

This is by-design. It's just a empty file we use to signal out of process the typescript server that a request as been canceled
533c6deb7901fba48df4db8c7adfbd42a6d2fe7a moved them to their own directory

That helps somewhat, but won't there still be thousands and thousands of orphaned files? I read #53423, it sounds like the assumption is that the OS will clean up these files, but Windows so far has not done that.

@mjbvz It looks like Windows doesn't clean out this temp directory on restart.
ls -al ~/AppData/Local/Temp/vscode-* | wc -l gave me 1,600 files before and after restart. However the files only go back a week. It might be a time based thing?


non-proper quoting in ON-condition

I use these settings:
squel.useFlavour('postgres');
var quoteOptions = {
autoQuoteTableNames: true,
autoQuoteFieldNames: true,
nameQuoteCharacter: '"',
tableAliasQuoteCharacter: '"',
fieldAliasQuoteCharacter: '"'
};
var queryStr = squel.select(quoteOptions)
.from('tableOne', 'tOne')
.field('tOne.topic')
.field('tOne.mediaDate')
.join('tableTwo', 'tTwo', 'tOne.id = tTwo.tOneId')
.limit(10)
.toString();
output:
SELECT "tOne"."topic", "tOne"."mediaDate" FROM "tableOne" "tOne" INNER JOIN "tableTwo" "tTwo" ON (tOne.id = tTwo.tOneId) LIMIT 10
The elements within the ON() parenthesis must also be quoted: ON("tOne."id" = "tTwo"."tOneId").
Thanks for your effort!

sorry, I created the issue too fast. Of course I control that myself in:
.join('tableTwo', 'tTwo', 'tOne.id = tTwo.tOneId')
So everything is fine. Thanks a lot.


Store locale in Arrow object so it can be used by format

https://github.com/crsmithdev/arrow/issues/309

Codecov Report

Merging #467 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #467   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines        3189   3202   +13
  Branches      229    230    +1
=====================================
+ Hits         3189   3202   +13

Impacted Files
Coverage Δ

arrow/factory.py
100% <100%> (ø)
:arrow_up:

arrow/arrow.py
100% <100%> (ø)
:arrow_up:

tests/factory_tests.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @eezomancer, thanks for the contribution!
I feel this can be accomplished a different way that wouldn't require changes to Arrow.  Two options come to mind, either creating a subclass in your project:
class FrCaArrow(arrow.Arrow):
  def format(self, fmt='YYYY-MM-DD HH:mm:ssZZ', locale='fr-ca'):
    return super().format(fmt, locale)

or using a function (this would be my preferred solution):
def fr_ca_format(arw):
  return arw.format(locale='fr-ca')

@Syeberman, hi! I made this PR in response to this issue. If my understanding is correct, the goal is to make the locale persist beyond Arrow method calls so it can be used with python's String format.
A use case would be along the lines of:
>>> import arrow
>>> arw = arrow.get(locale='it_it')
>>> dict = {'name': 'Joe', 'today': arw}
>>> print("Hi, I'm {name}! Today is {today:dddd}.".format(**dict))
Hi, I'm Joe! Today is venerdì.

I suppose one could just change the dict['today'] value to arrow.get().format(fmt='dddd', locale='it_it')  though. Wdyt?

Closing so I can properly set up my local environment first. I'd still like to know if a PR for this issue is still worth pursuing.


(SE) SLES15 optimization for Github Actions

Tier 1 finished successfully but took too long, ~1 hour for almost each task(you can compare with previous Tier 1 runs). I had to do a few cylc related change in gmao_ci account to ~/bin/cylc and ~/cylc/global-workflow.yaml files, which are one time changes. Another change I had to do was ~/bin/cylc file. It chooses the correct Cylc installation depending on the OS, and ~/bin is added to $PATH.
See output here:
https://github.com/GEOS-ESM/swell/actions/runs/11633945304
Here are the steps I took to modify test_swell.yml to be able to run Test CI Applications Action:

Update CI-Workflows :
Modify following file:
GEOS-ESM/CI-workflows/.github/workflows/test_swell.yml
in feature/test_swell_application branch

In Swell -> Actions -> Test CI Applications and run any Swell branch (say we are testing different SLURM configs). Test CI only runs a particular CI-Workflows branch linked below:

https://github.com/GEOS-ESM/swell/blob/7812c4114de3c587a026cc8c7453ad3e6b7e4528/.github/workflows/test_ci_application_discover.yml#L12
To run this you need to be in @jardizzo's nams_check.py file.
The slowdown is caused partly due to these two lines for variational tasks, since now there are 126 cores available in Milan nodes (I personally request 100 ntasks-per-node):
https://github.com/GEOS-ESM/swell/blob/7812c4114de3c587a026cc8c7453ad3e6b7e4528/src/swell/utilities/slurm.py#L49-L50
However I'm not not sure about why hofx suite would be slow, so there must be combination of Discover filesystem being slow + Swell SLES15 SLURM settings.
Default ntasks-per-node is defined here for different platforms:
https://github.com/GEOS-ESM/swell/blob/develop/src/swell/deployment/platforms/nccs_discover_sles15/slurm.yaml
@rtodling and I can help with the proper node/ntasks combination but filesystem could be resolved with $TSE_TMPDIR implementation?

This is a bit urgent as our Tier 1 or 2 tests won't work since we updated runners to SLES15.

The time between submitting a job on a head node and its start is much longer compared to the previous SLES12 runner setup. To give an example, in this action run for 3dfgat_atmos, 20211212T0000Z/GetObservations-geos_atmosphere task takes 6 minutes from submission to start running. Is it due to runners not being able to handle the number of requested tasks? Compute node tasks don't seem to have this issue.

3dfgat_atmos wasn't working until I made this small fix to use more ntasks-per-node on Milan: https://github.com/GEOS-ESM/swell/compare/develop...feature/tasks_per_node

3dvar still fails, I have no idea why but it always fails on the same task, GenerateBClimatology, for the same reason claiming missing files but they do exist. This task requires compute nodes. Exact same setup works on my local submission and 3dvar suite runs on a 5-degree setup so the compute requirements are minimal.

@jardizzo, could you update swell-tier1_application_discover.yml in develop with .github/workflows/test_swell.yml in feature/test_swell_application branch? That's what I've been testing with.


Display name get/set methods

Looks like this was superseded by #11?

Partially, The set methods will be going in another PR.


Add Wellknown data type support for ProtobufNativeSchema

This fixes #194
Currently, ProtobufNativeSchema does not support handling data with DateTime type. This PR will add Wellknown type(include System.DateTime, System.TimeSpan, System.Guidand System.Decimal) support for ProtobufNativeSchema. But currently only DateTime is supported in this PR.
Currently, this PR is only a preliminary change, the specific implementation solution still needs to be discussed. I am wondering if there is a better solution so that we don't need to introduce the content of proto files(such as /google/protobuf/timestamp.proto) in our project. Any ideas on this ? @Lanayx  Thanks!

Hi, @RobertIndie ! I think we should be able to load proto files content from the protobuf-reflection library, as soon as they are already loaded as embedded resources

Can you try using this approach instead of hardcoding the content?

Great! I will try.

Hi @Lanayx . This PR is ready for review. Thanks!

Thanks, @RobertIndie ! I'll do a little cleanup and publish a version soon


Add code signing

Adds code signing of the artifacts with the .NET Foundation certificate

@onovotny,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

NOTE: This is identity signing the assemblies so if you right -> click properties you can verify that the binaries were produced by us. This is not strong naming and is completely unrelated.

Changes Unknown when pulling 491d87df77c117b43add55401433f4fd337edb6b on add-code-sign into ** on develop**.

Changes Unknown when pulling cbf8a2bd7f204023c3ac006fac20febc8e788b69 on add-code-sign into ** on develop**.

Changes Unknown when pulling 729645fb26fed6593528d6c4e4db290c261fb7a9 on add-code-sign into ** on develop**.

Changes Unknown when pulling 53c5b3c922a2e5d16d1dc4eb7af19a771eafe3e6 on add-code-sign into ** on develop**.

Needed to change a couple things:

Retargeted to master
Failing to sign (due to environment variable missing) is now throwable and will fail a build
Signing is not run on pull-request builds as the variable isn't set (for security reasons)
Renamed some variables/locations for 💄 reasons

Apart from that thank-you once again Oren.


Update package verification fields for DPKG

This PR implements the suggested improvements from https://github.com/anchore/syft/issues/371#issuecomment-815932176, specifically:

add conffile listing to the files field, combining information from both the status file and the info/conffile
normalize the md5 field to digest, with subfields value and algorithm
updated JSON schema with DpkgMetadata changes (which is a partially breaking change)
discovered that the artifacts.metadata anyOf clause was being generated incorrectly, resulting in an always-passing validation --this was fixed as well.

Partially addresses #371

@alfredodeza created https://github.com/anchore/syft/issues/377


ci: use internal worker for release job

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 22.39%. Comparing base (fd1286c) to head (d4a396d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1328      +/-   ##
==========================================
- Coverage   22.46%   22.39%   -0.07%
==========================================
  Files          54       54
  Lines        4514     4514
==========================================
- Hits         1014     1011       -3
- Misses       3401     3403       +2
- Partials       99      100       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Feature Request] Q Ensembles

Motivation
Twin Q/ensemble Q functions are used in many RL algorithms and mitigate Q overestimation. My understanding is that TorchRL only deals with ensembles in the loss functions. This is fine for actor/critic methods since we only use the critics to compute actor loss. But for critic-only methods (e.g. DQN), we need the Q ensemble at sample collection time. Doing so would also simplify the loss functions for DDPG/SAC/REDQ/etc.
Solution
I would like to add ensemble Q function support to TorchRL, but I'm not sure on the best way to do this. I was thinking of creating a  TensorDictModuleEnsemble in tensordict_module.py that could be used for more than just Q functions. The issue is that we essentially need two forward functions: one at sample-time to compute some reduce operation like min over the ensemble outputs, and one at training time that does not reduce, but rather does something like
self.ensemble(tensordict.expand(tensordict.shape, ensemble_size))

so we can compute the loss for all Q functions. I'm not sure if there is a good way to tell a TensorDictModule whether it is in "sampling" or "training" mode.
I think it also makes sense to provide an option to keep separate datasets for each model/Q function, e.g.
tensordict.set("ensemble_idx", tensordict.get("ensemble_idx", torch.randint(ensemble_size, 1)))
for i in range(ensemble_size):
  sub_td = tensordict[tensordict["ensemble_idx"] == i]
  self.models[i](sub_td)
...

I'd like to avoid the for-loop here if possible, but I'm not sure how.
Additional context
Related to https://github.com/pytorch/rl/issues/876
Checklist

[x] I have checked that there is no similar issue in the repo (required)

Thanks for raising this
Seems like an interesting problem to tackle.
Regarding basic implementation, I think we should keep a contiguous stack of params and use vmap over the Q-Value network, that should be faster than looping over the networks / params.
Next you're raising the question of how to select or not the min value. We do that in a very custom fashion in the losses, but I agree that we should find a more generic way of doing so. If we get a table of state-action values like
[[v_00, v_01],
[v_10, v_11]]

for row-based actions 0, 1 and column-based networks 0, 1, how do you select an action using the min operator (knowing that you want the action with the maximum value)?
IMO we could build the backbone that produces the table and append an action selection strategy like we do for QValue networks (i'm trying to move away from wrappers as wrapping quickly puts things in super nested structures where you don't really know where your original module lives). So we'd have something like
policy = TensorDIctSequential(
  EnsembleStateActionValue(...), # uses vmap, writes a table of values in the output tensordict
  EnsembleQValueActor(...), # selects the action given some heuristic
)

In the loss function, we can either pass the entire policy, which will work since TensorDIctSequential keeps track of intermediate values in the output tensordict (but risky: one could overwrite the actions in the tensordict) or just pass the EnsembleStateActionValue and let the loss deal with it.
Happy to sketch a solution in a notebook if that helps!

Do you think it makes sense to break this into three modules, so we can utilize the default QValueActor?
policy = TensorDictSequential(
  Ensemble(in_keys=['observation'], out_keys=['ensemble_state_action']),
  Reduce(in_keys=['ensemble_state_action'], out_keys=['state_value_action'] reduce_fn=lambda x: x.min(dim=-1)),
  QValueActor(env.action_spec)
)

This keeps ensemble/reduce more general as they could be useful outside of Q functions.

I think that could work
Here's how I would go about the vmap module
from tensordict import TensorDict
from tensordict.nn import TensorDictModuleBase, TensorDictModule
from torch import nn
import torch

net = nn.Sequential(nn.Linear(10, 128), nn.Tanh(), nn.Linear(128, 128), nn.Tanh(), nn.Linear(128, 128), nn.Tanh(), nn.Linear(128, 128))

module = TensorDictModule(net, in_keys=["in"], out_keys=["out"])

class VmapParamModule(TensorDictModuleBase):
    def __init__(self, module, num_copies):
        super().__init__()
        params = TensorDict.from_module(module)
        params = params.expand(num_copies).to_tensordict()
        self.in_keys = module.in_keys
        self.out_keys = module.out_keys

        self.params_td = params
        self.params = nn.ParameterList(list(params.values(True, True)))
        self.module = module

    def forward(self, td):
        return torch.vmap(self.module, (None, 0))(td, self.params_td)

vmap_module = VmapParamModule(module, 2)
td = TensorDict({"in": torch.randn(10)}, [])
vmap_module(td)

This gives you a td:
TensorDict(
    fields={
        in: Tensor(shape=torch.Size([2, 10]), device=cpu, dtype=torch.float32, is_shared=False),
        out: Tensor(shape=torch.Size([2, 128]), device=cpu, dtype=torch.float32, is_shared=False)},
    batch_size=torch.Size([2]),
    device=None,
    is_shared=False)

Great thanks. One last thing: I'm struggling with a nice way to reinitialize the copied parameters. Calling reset_parameters on each module individually would be ideal, but not all torch modules have this. For example, nn.Sequential does not expose reset_parameters.
Furthermore, by disconnecting the modules from the parameters via TensorDict.from_module, we can no longer call reset_parameters on each copy of the parameters. We could pass parameter_init_function to VMapParamModule, but this will become ugly fast. For example, consider a MLP with a special final layer init. The linear bias init requires fan_in which depends on the associated weight. The final layer of the MLP should do something like nn.init.normal_(weight, 0, 1e-4), bias.zero_(). How do we determine which weights in the ensemble get which init functions?
It's ugly, but perhaps something like the following is the best we can do?
modules = [deepcopy(module) for _ in range(num_copies)]
[user_defined_reset_weights_fn_(m) for m in modules]
params_td = TensorDict({str(k): modules[k] for k in range(num_copies)})

Another solution would be to add reset_parameters to TensorDictModuleBase, but this would be a lot of work. We'd need to write custom reset_parameters for CNN, LSTM, MLP, etc.


add some documentation about jurst and managing legacy maps

Related #349
I think there's some language which is not quite right ("translate vjass"?) Let me know what you think.

Please try to keep one sentence per line. It is easier to read diffs that way.

the Wurstpack World Editor will parse jass/vjass, in addition to compiling your Wurst code.

It will only parse jass unless you enable the vjass compiler.

translating existing vJass to Jurst automatically

There is no automatic translation. The command just extracts the trigger code as it is and puts it into files.

I'll not have time anymore to make the required changes. @Frotty can you take this over please?

@peq I've fixed this up.

ty


[BACKLOG-40097] - new interfaces and calling points for Bowls and met…

…astores
Have RunConfigurations use the Bowl metastore at runtime

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 49m 3s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 5395, Failures: 0, Skipped: 14    Test Results

:information_source:  This is an automatic message

SonarQube didn't like the volatiles, so I checked Effective Java, and found a slightly better and recommended version.

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 49m 47s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 5395, Failures: 0, Skipped: 14    Test Results

:information_source:  This is an automatic message

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 46m 20s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 5179, Failures: 0, Skipped: 10    Test Results

:information_source:  This is an automatic message

:x: Build failed in 1m 51s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

:x: Build failed in 15m 24s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 1277, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message

:x: Build failed in 15m 18s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 1922, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message

:x: Build failed in 18m 11s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 1928, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message

:x: Build failed in 25s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

Could not find credentials matching jfrog-pentaho-service-cicd

:information_source:  This is an automatic message

:x: Build failed in 13m 53s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 1487, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message

:x: Build failed in 24s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

Could not find credentials matching jfrog-pentaho-service-cicd

:information_source:  This is an automatic message

:x: Build failed in 18m 32s
Build command:
mvn clean verify -B -e -Daudit -amd -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 2405, Failures: 0, Skipped: 2    Test Results

:information_source:  This is an automatic message

:x: Build failed in 35m 21s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 5392, Failures: 0, Skipped: 15    Test Results

:information_source:  This is an automatic message

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 52m 53s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox

:ok_hand: All tests passed!
Tests run: 5392, Failures: 0, Skipped: 15    Test Results

:information_source:  This is an automatic message


Add pytest test execution

add "testlaunch.py" to TestExecutor when using pytest

Support Cancellation
handle different test options

Run All
Run Test
Run Multiple Tests
Debug options

Any news on the May 2019.1 milestone due today?


sidebar-sm-show

When I leave the default <body class="app header-fixed sidebar-fixed aside-menu-fixed sidebar-lg-show"> everything works great. However I want the sidebar to appear by default on sm or larger. So I followed the instructions(https://coreui.io/docs/components/sidebar/) to change it
<body class="app header-fixed sidebar-fixed aside-menu-fixed sidebar-sm-show">
This appears to work at first glance. However, unlike sidebar-lg-show, for sidebar-sm-show the sidebar toggle button does not work. If I resize the window to larger or smaller, then the toggle works and I can go back to sm size and the toggle works. But onload the toggle does not work on sm. This isn't the same behaviour on sidebar-lg-show where the toggle works onload.
This seems like a bug to me. I have tried to search for this but did not find anything. I'm using Chrome(Version 68.0.3440.84 (Official Build) (64-bit)) on Windows 10. I downloaded the latest version of CoreUI 5 minutes ago to try it on a new project.

hr @rharor
try to modify sidebar toggler button from data-toggle="sidebar-lg-show" to data-toggle="sidebar-sm-show" and its display property accordingly (as well as mobile sidebar toggler button display property, to make sure they not interfere)


Add more metadata

For minimal apis, i miss methods to add descriptions
app
        .DefineApi("People")
        .HasMapping(api =>
        {
            api.MapGet("/api/v{version:apiVersion}/people/{id:int}", (int id) => "Some response")
              .Produces(response => response.Body<WeatherForecast>())
              .Produces(404)
              .HasApiVersion(1.0)
              ;

        });

Produces:
{
    "x-generator": "NSwag v<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))",
    "openapi": "3.0.0",
    "info": {
        "title": "Minimal API",
        "version": "1"
    },
    "servers": [
        {
            "url": "https://localhost:7174"
        }
    ],
    "paths": {
        "/api/v1/people/{id}": {
            "get": {
                "tags": [
                    "people"
                ],
                "operationId": "GetApiV1People",
                "parameters": [
                    {
                        "name": "id",
                        "in": "path",
                        "required": true,
                        "schema": {
                            "type": "integer",
                            "format": "int32"
                        },
                        "x-position": 1
                    }
                ],
                "responses": {
                    "200": {
                        "description": "",
                        "content": {
                            "application/json": {
                                "schema": {
                                    "$ref": "#/components/schemas/WeatherForecast"
                                }
                            }
                        }
                    },
                    "404": {
                        "description": "",
                        "content": {
                            "application/json": {
                                "schema": {
                                    "type": "string"
                                }
                            }
                        }
                    }
                }
            }
        }
    },
    "components": {
        "schemas": {
            "WeatherForecast": {
                "type": "object",
                "additionalProperties": false,
                "properties": {
                    "date": {
                        "type": "string",
                        "format": "date-time"
                    },
                    "temperatureC": {
                        "type": "integer",
                        "format": "int32"
                    },
                    "temperatureF": {
                        "type": "integer",
                        "format": "int32"
                    },
                    "summary": {
                        "type": "string",
                        "nullable": true
                    }
                }
            }
        }
    }
}

In this example i want to be able to add paths.['/api/v1/people/{id}'].get.description and paths.['/api/v1/people/{id}'].get.responses.['200'].description which is currently (6.0.0-preview.2) not possible

This will be reverted in Preview 3, which would alleviate this problem. It should be published - soon. 😉 I'll leave this unresolved until you can confirm you're getting the behavior/features you expected.

Ahh what?
You will remove features? 😵
How would this solve the issue? 💩

🤣 HA! No, not quite. After a long discussion and some refactoring, the mechanisms by which you provide intrinsic metadata will be reverted back to how they were/are. You can see what it will look like here in the updated example.
It's less than ideal, but probably the best that can be done for 6.0. Once MapGroup is available in 7.0, then I'm hoping to pair things down even more. 🤞🏽

I'm going to call this a 🐞 simply because it didn't do what you expected. As I mentioned, there is a significant refactor in Preview 3 that will address this. Effectively, the API Versioning-specific code for this has been removed and things should now behave the same as any other Minimal API.
Preview 3 is now available and contains the fix for this issue. Review the release notes and updated examples to see how things have changed.

6.0 is officially released and has the same setup as in Preview 3. This should feel natural and coherent with how the metadata is applied, even without API Versioning. In .NET 7.0, some improvements will come to streamline things even further. Thanks for reporting the issue.


Upload failing with error: In-Adequate Token Approval or Balance

Upload failing with 400 error: Sending response with status: 400, message: In-Adequate Token Approval or Balance
This is cause by the allowance's check: https://github.com/oceanprotocol/dbs_filecoin/blob/main/src/api/upload/upload.controller.js#L48
We need to check also DBS.js and DBS.

Are you sure you have enough tokens in your wallet? I don't see anything wrong with the checks

Yes, I do have enough funds.

But somehow on the logs I keep getting this:

The issue here is that filecoin service needs to give approval to the deposit contract rather than the dbs address. I'll fix this from dbs.js: https://github.com/oceanprotocol/dbs.js/issues/46


cardano-cli conway governance action create-constitution: rework hash arguments

Remove the metadata word from
--constitution-anchor-url TEXT
( --constitution-anchor-metadata TEXT
| --constitution-anchor-metadata-file FILE
| --constitution-anchor-metadata-hash HASH
)

it should be
--constitution-anchor-url TEXT
( --constitution-anchor-text TEXT
| --constitution-anchor-file FILE
| --constitution-anchor-hash HASH
)

[edit]
Update from a discussion on Slack:

We will have --constitution-anchor-url TEXT --constitution-anchor-hash HASH,
and a command cardano-cli conway governance hash --file --text to create the HASH

I'm nearly done with this. In the end I propose we have three modes in the conway governance hash command, like b2sum:

--text which takes the text to hash directly
--text-file which takes a file and treats it as text, and hashes its content
--binary-file which takes a treats it as binary, allowing PDF files, compressed files, etc; and hashes its content.

If we go with only --text and --file, we don't clearly state how we treat files, and will end up with unhappy users. Test-wise, those flags will allow us to be platform-agnostic and avoid failures on Windows.

https://github.com/input-output-hk/cardano-cli/pull/442#pullrequestreview-1722472440

I went for --text, --file-text, and --file-binary in the end, as it looked more consistent with existing flags, that tend to differentiate different behaviors by the suffix of the flag, rather than the start of the flag.

@CarlosLopezDeLara> another side effect of the PR is that I harmonized the flags of anchors of vote create with all the other commands:

--vote-anchor-url becomes --anchor-url
--vote-anchor-metadata-hash becomes --anchor-data-hash


Find container repositories with Pyxis GraphQL APIs

The LightBlue class is updated to use Pyxis GraphQL APIs to find
container repositories, this is just a partial change so there are still
a lot of variables, functions in this class have not been changed
accordingly.
JIRA: CWFHEALTH-1708

Close this one, all commits are included by #194, please review that one instead.


Clean way to optional enable/disable extensions

Now that we are adding more tests for the linalg extension, and we will soon have other extensions as well, we need to have a clean way for modules to not implement these extensions and not fail the test suite. Some test files mix extension and non-extension tests, e.g., the signature tests currently test all signatures.
This is closely related to #20.

Bare minimum it would be nice if we automatically skipped tests if required attributes don't exist, like:
@pytest.mark.skipif(hasattr(xp, "linalg"), reason="extension not found")

This could be a more immediate thing we do. Could be refactored to something like mark.requires_xp_linalg. We could create a custom option and use a hook in conftest.py down the line if we think its still useful to folks.


Remove the vibrate service and replace w/ notifications

We shouldn't be using vibrate like we do in the app anyhow... replace it with notifications. This was also causing stability issues.

Done, 4.0 release


Github/Gitlab Pages

Feature Description
It's 2024 and we need alternative to Github/Gitlab Pages feature, which is as easy to use. Gitea tries to be like Github and it's actions are compatible but i do not understand why does it take for so long to have feature as useful as that.
https://docs.gitea.com/help/faq?_highlight=pages/help/faq#does-gitea-have-a-githubgitlab-pages-feature
Screenshots
No response

Duplicate of #302. We will not embed the pages feature unless the possible security problem can be resolved.


Error with starting the bot

I hate no idea what it is but here is a screenshot. Also at the top it says "No module named "Discord"

Fixed it.


Ready to merge - addresses #115

@cm Travis says this branch has conflicts that must be resolved. Should I hold off on merging this branch or ignore that and merge anyway?

I think this can be closed.