How to obtain Deformable DETR pretrained model (hidden_dim_288)

I got an error when loading pretrained deformable detr weight [r50_deformable_detr_plus_iterative_bbox_refinement-checkpoint.pth]. How to obtain the pretrained weight [r50_deformable_detr_plus_iterative_bbox_refinement-checkpoint_hidden_dim_288.pth] used here?

What error did you get?

Thanks a lot!


get related objects

This PR:

implements GetRelation WithObjects
updates to pb-directory v0.20.3 and go-directory v0.20.3
updates SDK used for testing from aserto-go to go-aserto v0.20.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 15 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 42.799%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/directory/reader.go
5
15
33.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
1361

Relevant Lines:
3180

💛  - Coveralls


Install fails on fresh RPi

After re imaging an SD card with the latest Rapian OS, and allowing it to update, I attempted to re install allsky. Following the instructions on the wiki for installing the allsky (not the instructions show on the left), bullet 1 (mkdir) and 4 (git clone) fail without a sudo command, but then later the build fails to compile. I believe the issue is that the folder which the git uses needs to be owned by pi or given more liberal access permissions (that would need to be documented)

.

just a follow up to my last comment, the camera is running. running command raspistill -o bob.jpg does work.

.

Line 3 of the install tips doesnt set the permissions correct.

.

@ssagerian What's wrong with the permissions?  Doesn't it set the owner and group to "pi" and make it mode 664?

@ssagerian Steve, that file is nothing but a pain in the butt.  It is initially created empty by allsky.sh then after camera detection completes the file is populated.  The newest allsky-portal doesn't use the file anymore.
If you set the CAMERA variable in config/config.sh using nano you can run allsky.sh once to populate the file.  You should the be able to use the WebUI.

@linuxkidd @EricClaeys  The most recent instructions in the wiki install tips work, to the extend that the install scripts ran without error. When I opened the page for the first time it exited with a note indicating that CAMERA was not set and to manually update it. After manually updating the setting, and refreshing the browser the page came up correctly. Everything appears to be working correctly however I have a few more things to check.  autocam.sh is still zero length

In order to close this out, I think it still needs a mention in the wiki install tips to edit the config/config.sh file to add the camera type, as mentioned in the previous comment.

@ssagerian Steve, thanks for the suggestion.  Check out the first section in "02 Installation Tips" of the Wiki.  If it's sufficient, please close this issue.

comment in 02 is clear, thanks


Fork and update dependencies in flow-go to update badgerdb/v2 in Archive node to v3

Upgrade badgerdb to the latest version (and apply compatibility changes in flow-go) in order to possibly improve GetRegisters latency.

https://github.com/onflow/flow-archive/pull/61

Fork with required changes to run badger v3: https://github.com/koko1123/flow-go-1


[hotfix] Remove skopeo dependency

Related to the issue #402
Signed-off-by: Dmitrii Ustiugov<EMAIL_ADDRESS>

The changes in scripts/setup_system.sh are a sufficient hotfix for this issue.
cri/Makefile can therefore be discarded.

Without these changes, the CRI tests would be broken

Then proceed with the merge.

thanks @cvetkovic it's best if you approved using the button though


Add icons for .bash files

.bash files do not have icons

I don't use atom any more really. Feel free to open a PR adding these icons and I'd be happy to merge and release

has this been added @KyleGoetke  ?

@KyleGoetke @sudancovid19
Looks like this is done and this issue can be closed:
icons/bash.svg


using variable is [a literal] is wrong, and results in undefined behavior

I've seen terrible non-deterministic problems (in an unrelated project) due to a similar bug.
I converted all   is [string] into   == [string]   (in layers.py) as it should be.

Hi Yoel, thanks for the PR! Good idea.


contact trick

Please use dev as base and make the PR to dev.

I rebased the branch onto dev but github is still showing commits that should not be there anymore.
Thus I created a new PR to dev: https://github.com/Artificial-Pancreas/iAPS/pull/595
Closing this one.


Option to always orient the player body & head even if not holding a ranged item

currently when in decoupled mode, the player only turns to the crosshair when holding right mouse if they're either in reach of a block or holding a ranged item such as a shield or bow. an option to let the player turn their character regardless of this would be quite nice

Do you mean turning the player even if there is no target, or synchronizing the player rotation with the camera rotation? The former is already part of the next release and the later can be achieved by setting decoupled_camera to false.


fix update cg command to take cg uuid, etc

fix "update cg" command to allow cg-uuid
fix skip-older defaults to "0"
other minor annoyances

Coverage decreased (-0.2%) to 68.331% when pulling e2bb3a662afc793c21e4b9e20d13ac936d5ed5fa on fix-cli into 59bfb01af8f2336e28f4e01d90314030b2f3372c on master.

Coverage decreased (-0.4%) to 68.178% when pulling 3af0fde9977c13dc03fec4c07c60a89335bd71be on fix-cli into 59bfb01af8f2336e28f4e01d90314030b2f3372c on master.


is_primary_data review & revision

We are establishing a more robust mechanism to evaluate data reuse & is_primary_data curation. This has uncovered some things to review and likely some revisions...
DOIs that are not true anywhere

[x] 10.1101/2<IP_ADDRESS>3877
[x] 10.1016/j.molmet.2021.101240
[x] 10.1186/s13059-020-02210-0
[ ] 94,927 observations in 80a2c5b6 that need to have the true "original DOI" tracked down

DOIs that are true in multiple places and it's not immediately clear that it can be explained by a lack of data overlap...

[ ] 10.1016/j.jid.2017.09.045
[ ] 10.1016/j.molmet.2022.101595
[ ] 10.1038/s41591-018-0096-5
[ ] 10.1038/s41591-019-0750-6
[ ] 10.1038/s41591-020-0901-9
[ ] 10.15252/emmm.202012871

10.1101/2<IP_ADDRESS>3877

anticipated in an upcoming submission (immune aging)

10.1016/j.molmet.2021.101240

Revised 35945 obs (Dataset:Dowsett10xnuc) to is_primary_data:True

10.1186/s13059-020-02210-0

anticipated in an upcoming submission (megagut)

10.1016/j.jid.2017.09.045

Revised 2742 obs (Author:Tabib_2018) to is_primary_data:False in the human Dataset here

10.1016/j.molmet.2022.101595

No changes needed - the mouse data is the only overlap and mouse in the Collection for that DOI is not is_primary_data:TRUE

10.1038/s41591-018-0096-5

Revised 66296 obs (study:Lambrechts_Thienpont_2018) to is_primary_data:False in the LuCA extended atlas Dataset here

10.15252/emmm.202012871

Revised 7333 obs (study:Mayr_Schiller_2020) to is_primary_data:False in the LuCA extended atlas Dataset here

10.1038/s41591-020-0901-9

Revised 41573 obs (Unpublished:PMID: 32398875) to is_primary_data:False in the lone Dataset here

10.1038/s41591-019-0750-6

Revised 32850 obs to is_primary_data:False in the "full" Dataset here

94,927 observations in 80a2c5b6

Revised 94927 observations in the lone Dataset to is_primary_data:True
These are considered "original" to the DOI of their Collection. They just weren't deposited in ArrayExpress with the rest of the original data, but they do not appear in other publications/records like the other reused data in this submission.

Reran the evaluation of data reuse after the above Revisions, and no overlap or gaps were identified.


Update Biome User Management

Update the Biome User Management 0.5 docs

update content to reflect the current submodules in the biome module and the current biome REST API endpoints
add an updated diagram for the biome database tables

Should keep this an svg

New diagram is now svg


Login redirect behind proxy returns 404

I am trying to use flask-user in a Heroku App behind a chisel proxy.
When opening the website it redirects me to localhost:<port>/user/sign-in?next=/ instead of <app-url>/user/sign-in?next=/.
I tried to set flask's app.config['SERVER_NAME'] to the correct URL but then I get a 404 Not Found error. I guess this is because gunicorn only talks to the proxy so it thinks that the SERVER_NAME must be <IP_ADDRESS>: or something like that.
What would be the correct way of configuring flask-user for this scenario?

I solved this by using nginx as the proxy and then gunicorn and chisel behind that.


Add support for pandas DataFrames

Resolved #53

@robertfrankzhang I updated this PR to be compatible with your latest changes. Please let me know if you have any issues with merging this in.


Add description with full test suite instructions

These are already documented on the contribute page but would be
good to have them available at hand here as well.

I can make it part of #1215 though (new envar added )

All are just minor changes, to be squashed under a single commit.

All are just minor changes, to be squashed under a single commit.


Add Deno support

We need a separated file excluded from npm package and using deno.noColor.
What hosting should we use to server ESM file?

What about https://deno.land/x ?

Thanks. I took the name and will add Deno file with guide in docs tomorrow.

Fixed https://github.com/ai/nanocolors/commit/5452c035979e3227a6ffc550192eb8708347192b

It’s gone now? Why so?

Because Deno has color functions in stdlib.


download v0.4.0  .mac.zip and open the app, show me Error in auto-updater, something wrong?

sorry, dmg is ok

i keep getting the same error. How did you fix it?


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2761

Questions ?
Please refer to the Backport tool documentation


Improving the UI and UX of the overall app

Improving the UI and UX of the overall app  and making the app more usable

@chintan0513
pls assign this issue to me


[Forged In Dungeon] sheet updates

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

update html : add class prefix 'sheet-' at all rolltemplate section.

Character Sheet Info Roll20 Internal Use only.


Apostrophe integer field has broken filter implementation

Fix that. Also make sure our "insensitiveSort" does the sensible thing if comparing two numbers.

This was fixed.


RfHub2 is not displaying more than 100 items in left panel

RfHub2 is not displaying more than 190 items in left panel.
HTR:
run application and populate it
for i in {1..500}; do curl -u rfhub:rfhub -X POST "http://localhost:8000/api/v1/collections/" -H "accept: application/json" -H "Content-Type: application/json" -d "{\"name\":\"$i\",\"type\":\"string\",\"version\":\"string\",\"scope\":\"string\",\"named_args\":\"string\",\"path\":\"string\",\"doc\":\"string\",\"doc_format\":\"string\"}"; done

Actual state:

I think this is due to limit coming from
https://github.com/pbylicki/rfhub-new/blob/master/rfhub2/ui/ui_router.py#L22
and directly from
https://github.com/pbylicki/rfhub-new/blob/b74ccf242687e6450cb86e755dd03530d07c0890/rfhub2/db/repository/collection_repository.py#L24

@pbylicki how we should proceed with this one? Should paging be added? This might be good for left panel, but start page is also suffering from this one.

@MaciejWiczk I am playing around with new, React-based version of frontend reading data from API. I would prefer to address this issue there.

@MaciejWiczk I am playing around with new, React-based version of frontend reading data from API. I would prefer to address this issue there.

I was wondering what occupied You :)


[DCJ-389] Add script to create default Broad DAA

Addresses
https://broadworkbench.atlassian.net/browse/DCJ-389
Summary
Prepopulate the system with a Broad Data Access Agreement (DAA), add it to all existing Data Access Committees (DACs), and add all users with a library card to the Broad DAA.
It is not advisable to test this script without restriction, as it will create alternate Broad DAAs.

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

looks good! out of curiousity, how does auth exactly work?

It uses the magic of AUTH_TOKEN=$(gcloud auth print-access-token) to get a valid auth token for your currently logged in gcloud user (which should typically be your Broad user).


Item_6

Added the account page and functionality. Need review.

Added an updated account page, this version is designed much better than the last.

Forcing pull request with admin perms after self review over the course of 3 days.


Python code for quick sort

can you assign me this issue

Yeah, sure!!

make sure to add programs under respective folder.

don't forget to star the repo.

Hi @Saikat2407
Can u assign me this issue
I am happy to contribute on this.

Hi @Saikat2407 Can u assign me this issue I am happy to contribute on this.

sure!

Hi @Saikat2407 Can you please assign this issue to me I want to contribute on this

sure work on it asap!!

Hi @Saikat2407 Can you please assign this issue to me I want to contribute on this

sure, star the repo first and start working on it!!

Hi @Saikat2407  can i work on this issue?


Add common name to CSR payload

The CSR payload currently doesn't contain the common name, resulting in the subject of the CSR not having a CN value defined.
Example:
Subject: C=CA, ST=British Columbia, L=Nanaimo<EMAIL_ADDRESS>Adding the common name to getCSRPayload in CertificateRequestSigner will result in:
Subject: C=CA, ST=British Columbia, L=Nanaimo, O=NA<EMAIL_ADDRESS>

Moved issue to original repo: https://github.com/acmephp/acmephp/issues/129


utf-8 decoding error

The script currently throws this error and fails:
Traceback (most recent call last):
  File "/Users/samthorley/.pyenv/versions/3.9.9/bin/class-email", line 8, in <module>
    sys.exit(main())
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/site-packages/class_email/__init__.py", line 96, in main
    for i, row in enumerate(csv):
  File "/Users/samthorley/.pyenv/versions/3.9.9/lib/python3.9/codecs.py", line 322, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x85 in position 3223: invalid start byte

I've made a fix but currently can't push the change.

Please fork the repo and make a PR from there.

Made a fix myself using ASCII surrogates (https://docs.python.org/3/howto/unicode.html#files-in-an-unknown-encoding), see https://github.com/HamishWHC/class-email/commit/cec8b34acc30c92585b42c6b940ca6ee8944b40b.


AST named column (use raw expression from sqlwtf)

https://github.com/staabm/phpstan-dba/pull/570 got too messy after rebasing etc, decided to make it again from scratch

I dont think this
protected function setUp(): void
    {
        QueryReflection::getRuntimeConfiguration()->utilizeSqlAst(true);
    }

    protected function tearDown(): void
    {
        QueryReflection::getRuntimeConfiguration()->utilizeSqlAst(false);
    }

works properly, the tests are failing as if the AST wasnt enabled at all (which I double checked in QueryReflector by dumping the runtime config)
if (
                self::getRuntimeConfiguration()->isUtilizingSqlAst()
            ) {

I am not very much familiar wiht phpunit, I usually use nette/tester or codeception

I think the reason that it worked for DoctrineKeyValueStyleRuleStrictTest extends RuleTestCase
but not in this PR is the different base class DbaAstInferenceTest extends TypeInferenceTestCase.
unfortunately we cannot use a different base class here.
it seems your previous approach using a new test-suite was the right one. I have forgotten about that. sorry.


Add my pachyderm test/restart script

Per our conversation in Miami, I thought I'd check in the script I've been using to build/test Pachyderm locally, as a reference.

Closing this PR, ironically, as I've started using this branch to develop a shared tool.


Add a Flutter for web issue template

Add an issue template specific to Flutter's web support

This would prevent web issues from getting front line triage (because the way we drive what gets triage is by a search that excludes already-triaged bugs). Given that there is very little here that is unique to web issues, I'm concerned that this would actually overall be a net negative as it would mean web bugs get fewer labels and less front line processing.
That said, we can certainly add this temporarily while we wait for triage to resume. If you land this, please keep a note of all the places where you link to this template so that we can change them all to the the bug template when we are doing triage normally again.

@Hixie – so you DON'T want a template?

Happy to reuse Flutter's existing triage system. It would be very helpful to have issues get the platform-web label asap so it lands on the Web team's plate quicker. Would Flutter's triage process facilitate that?

I could go either way on this, as described above. I don't have a strong opinion. Long term I'd rather rely on front line triage. @kf6gpe tells me that will be going again soon.


Allow administrators to reset user settings to organization default

At present, admins can configure default settings for new users, but not change settings for existing users. We should also make it possible to reset non-privacy settings for existing users.
Specifically, each of the settings in the Default user settings panel should have a "Reset users to default" button (exact label TBD), that pops up a confirmation modal. The button can appear on hover over the setting (always shown on mobile devices).

Reset user configurations?
User configurations for this setting will be reset to the organization default.
Users whose configuration should be changed
[Everyone, Users who have not personally configured this setting] <- dropdown; default: Everyone
[Cancel] [Confirm]

As noted on CZO:

We can tell using RealmAuditLog which users have manually edited a given setting. (We can even distinguish if the user has edited it vs. some other user edited the setting for them).

Notification triggers
For notification triggers, we can have just one reset button for the whole section. It should reset the values for the rows shown in default settings, but it should not affect any custom configurations for particular channels.
Notes

A possibly follow-up would be to allow admins to automatically notify users whose settings were changed.

CZO thread
API discussion

@zulipbot claim

@zulipbot I am working on the issue I made some minor mistake for which I need to restart it again today

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot claim!

@kartikshukla17 Hi, i would like to work on this issue and it's been 3 weeks no movement.

@zulipbot claim

Can i get this issue assigned, if the work is not in progress ?


Fix #68, fix #69: Use packageLicenseExpression in all packages, and fix some package properties

This change fixes a problem with certain jschema packages which prevented them from being published, because they violated Microsoft package publishing policy (PPP, if you like ;-)). Opportunistically, it fixes a couple of other package property problems (PPP2, if you like).
Our build.props file sets the <PackageLicenseExpression> property to "Apache-2.0". The creation of the JsonPointer and JsonSchema NuGet packages is done by dotnet pack, and is driven by the MSBuild project file. So those packages respected the package license expression, and picked up the correct license.
But the NuGet packages for the two command line tools (ToDotNet and Validator) are created from custom .nuspec files, by directly invoking NuGet.exe. This is necessary because those packages require files that are not part of the projects that build the tools. That means that those packages don't know anything about the MSBuild project properties, so they didn't pick up the package license expression.
To fix that, we add a <license type="expression"> tag to the package metadata in the .nuspec files. That leads to two more issues:

Version 4.3 of NuGet.exe, which we were using, doesn't recognize that tag. Therefore we upgrade to the latest NuGet.exe, Version 4.9.2.

We could hard-code Apache-2.0 into the .nuspec files, but we would rather get it from the MSBuild properties, to ensure that it stays in sync with the packages that are created by dotnet pack. Therefore we add some code to BuildAndTest.ps1 to read the PackageLicenseExpression MSBuild property, and pass it on the NuGet.exe command line.

That takes care of #69.
While doing this, I noticed a couple of problems with the metadata of the other packages, the ones created by dotnet pack. This is #68.

They did not have a Title property. I fixed that by setting <Title> to $(AssemblyTitle>) in build.props.

They did not have the correct Owners property. There was in fact a bug in the setting of the <Owners> MSBuild property: the Condition was wrong. But fixing it does not fix the problem. After some research, it seems that dotnet pack does not respect Owners and always sets it to Authors, so there's nothing more I can do.

Finally, I fixed a couple of casing errors in the .nuspec files.

@michaelcfanning is added to the review. #Closed


Pass options hash to Client, Ring, and Server

Adds support for passing an options Hash from any Dalli::Client calls into the downstream Dalli::Client, Dalli::Ring and Dalli::Server methods. This allows callers to pass options like failover: false in the cache call itself, rather than having to create a separate Client or Store to enable these options.
Normally this can be achieved by creating a separate Store to get different configuration, but right now there's no way to share connections between Stores. This means if you want a particular call to failover and another one to not, the only current option is to double the open connections (one pool for each configured Store).
Currently the set, replace, get, and add operations support passing options, but the delete, incr, decr, append, prepend, cas, and touch operations don't. This PR adds options support for these latter operations.
It also adds support for passing :failover option per operation, rather than as configuration.
An alternative direction would be to allow creation of new clients that share the same underlying pool of connections, but this is a deeper change as currently the connection pool is at the Store level, as a pool of Clients.

Would be interested to hear any thoughts on this proposal... we've been running this fork in production for a week and would love to get back on mainline at some point. Thanks!

If you want to resubmit this PR, we'll take a look. Broadly, I'm not sure I like the idea of individual operations being able to override global settings but I could be convinced with a reasonable use case.

Thanks @mperham, I've rebased the original change with the latest master.
The main benefit of having options passed is that we can do things like control the failover per operation.
As an example, our normal caching operations are failover tolerant in that they simply rely on the TTL, and it doesn't matter if a memcached node goes down and we temporarily store a value in another node in the ring.
However, we have certain operations that we want to prevent failover, as we prefer stronger cache invalidation guarantees. To achieve this we pass failover: false to all reads and writes to these keys, and raise_errors: true to writes. We also have a patch applied to Dalli::Ring#server_for_key method (irrelevant code omitted):
module DalliRingWithServerOffset
  def server_for_key(key, options = nil)
...
        server = @continuum[entryidx].server
        server_offset = options[:server_offset].to_i
        server = @servers[(@servers.index(server) + server_offset) % @servers.length] if server_offset != 0
        return server if server.alive?
...
  end
end
Dalli::Ring.prepend(DalliRingWithServerOffset)

This allows us to store a key on two separate servers in the ring, which are both read and only used if equal, which gives us stronger cache key invalidation when one server is down.
We could do this with different combinations of options on the client itself, but we use quite a few different combinations of options:

failover: true, raise_errors: false
failover: false, raise_errors: false, server_offset: 0
failover: false, raise_errors: false, server_offset: 1
failover: false, raise_errors: true, server_offset: 0
failover: false, raise_errors: true, server_offset: 1

With the current gem, we'd require 5 different clients, and therefore 5 different connection pools. This is unfortunately untenable due to the large amount of connections to memcached, given Ruby's threading model.

I think this is a little too specialized for Dalli. I’m happy to entertain patches or refactorings that make a fork or patch more maintainable


Feature request: Markdown code blocks highlighting

I keep my notes in Markdown format, and it would be very useful if the plugin supported Markdown code blocks syntax highlighting.
I'm not familiar with the VSC plugin internals, however, based on a similar PR, this could be a simple job.

Hi @saveriomiroddi - apologies for taking so long to get around to this! This sounds like an interesting idea, and I'd love to look into it more.
When you have some time, would you mind providing me with an example of a note you would store in MD format? This would be very helpful in me being able to test syntax highlighting against your use case. Thanks!

Hi @saveriomiroddi - apologies for taking so long to get around to this! This sounds like an interesting idea, and I'd love to look into it more.
When you have some time, would you mind providing me with an example of a note you would store in MD format? This would be very helpful in me being able to test syntax highlighting against your use case. Thanks!

Hi @saveriomiroddi - apologies for taking so long to get around to this! This sounds like an interesting idea, and I'd love to look into it more.
When you have some time, would you mind providing me with an example of a note you would store in MD format? This would be very helpful in me being able to test syntax highlighting against your use case. Thanks!

Hello! No problem, better late than never :smile:
You can find my terraform notes here. There's plenty for you to test! :smile:

Hi @saveriomiroddi - apologies for taking so long to get around to this! This sounds like an interesting idea, and I'd love to look into it more.
When you have some time, would you mind providing me with an example of a note you would store in MD format? This would be very helpful in me being able to test syntax highlighting against your use case. Thanks!

Hello! No problem, better late than never :smile:
You can find my terraform notes here. There's plenty for you to test! :smile:

Thanks for the quick response! I've been having very limited time lately to work on this extension (hobby :)), but I will take a look when I can. Thanks again for the suggestion!

Thanks for the quick response! I've been having very limited time lately to work on this extension (hobby :)), but I will take a look when I can. Thanks again for the suggestion!

Thank you :smile:
All in all, Ruby does fit the syntax, so this is a minor feature request. However, it may be simple enough, and it would make the plugin complete (syntax-wise).
Bye :smile:

Thank you :smile:
All in all, Ruby does fit the syntax, so this is a minor feature request. However, it may be simple enough, and it would make the plugin complete (syntax-wise).
Bye :smile:


[WIP] Restructure the Paths of Guides

This should lead to more organization?
closes #103

this is a draft

Also add a disclaimer at top of every page that says "under construction"
New Important Wiki Structure Changes!
all guides are now stored under src/guides/ instead of src/pages/guides . The main reason I did this was so that the urls can be cleaner
guides/academics/majors/ -> /academics/majors/cs
I updated the how to write a guide doc, and links, but there will probably be some errors. #123 can help this , plus a #105 that creates graph


Feature Request: Search by name or wildcard search by name

Use one of the two search boxes for a search by name (e.g. name contains Sunnyside).  Limit the number of records returned by this search to 50 or some number that won't impact performance.

@ladytmarie I like this idea, but the Socrata API we're using might be limited. To do a search for containing text, we'd have to use their 'like' function: reference docs
Unfortunately, I'm not having much luck using both this, and a filter for results only in Illinois. It doesn't look like I can chain it with something else. I might just be doing it wrong, but if we can filter for Illinois results I'm open to it, but otherwise I think it might just be pulling too many.

@pjsier My understanding is that the data set only contains facilities in Illinois/Chicago.  If this is true and you can limit the results to Illinois or a fixed radius/distance from the zipcode, would it be possible?

@ladytmarie so actually the dataset is national (I think the one on the previous version was just Illinois). Here's the Medicare Dataset. So that's why I'm having trouble with filtering it. It looks like we can't use the like function and something to parse the state at the same time, unless I'm formatting it wrong

Could we just display all of provider_county_name Cook by default?

@ziegegeist I think that works! I was worried earlier about it being overwhelming, but I tested it out and it seems fine. Just pushed to master and gh-pages
@ladytmarie I think this might actually be a way to do autocomplete for nursing home names. I can load all nursing homes in Cook County, and then search against that sort of the way it's searching against zip codes. Not sure if it'll work, but going to try it because it doesn't seem like the Socrata API will let us do that

Cool. Thanks @pjsier

This is implemented in the current version, closing


Add test to verify approved aliases list

Closed #3145

We need lists of approved aliases and tests to verify the approved aliases exist in PowerShell.

Question about original tests (first Describe in the file): it looks as non-actual - should we fix it or remove at all?


Find good standard recommendations for Gitlab CI testing Docker images

I am currently using conanio:{gcc9, clang10}, but they are based on Ubuntu 19.10, so you cannot run apt update anymore and will not be able to install additional packages.

Additional problem here: In what image can coverage testing be done best? I encountered lcov-related failures with gcc >= 8. I am currently usingthe image  gcc:7, but its setup regarding sudo is slightly different from the conanio:... images which prevents code reuse for e.g. installation of upstream dependencies.

I researched this once more and again ended up with the conanio images. I updated to recent versions and wrote a note about them.


SignatureDoesNotMatch exception when using custom service endpoint that has a path component.

Here's a quick snippet that demonstrates how one can reproduce the issue; note that the service URL has a path component:
var clientConfig = new AmazonEC2Config
{
    ServiceURL = "http://somehost/somepath"
};

var ec2Client = new AmazonEC2Client(
        new BasicAWSCredentials("access_key_here", "access_key_secret_here"),
        clientConfig);

var response = ec2Client.DescribeAccountAttributes();

The root cause appears to be that when AWS4Signer goes to canonicalize the request, it strictly uses IRequest.ResourcePath, which is null most of the time; in some cases, it gets hardcoded to "/". As a result, the canonicalized request has the wrong canonical URL which ultimately leads to the request being signed incorrectly.

@PavelSafronov I'm a little confused why this is labeled as a feature request when the issue seems to be a violation of the Signature Version 4 spec:

The canonical URI is the URI-encoded version of the absolute path component of the URI—everything in the URI from the HTTP host to the question mark character ("?") that begins the query string parameters (if any).

Normalize URI paths according to RFC 3986 by removing redundant and relative path components. Each path segment must be URI-encoded. If the absolute path is empty, use a forward slash (/). In the example request, nothing follows the host in the URI, so the absolute path is empty.

Additionally, other AWS SDKs (e.g. boto3) handle this case with no issues.

Currently, the .NET SDK does not access any AWS services which use subpaths in their endpoints, so I judged that the lack of this feature was not strictly a bug but rather a feature. That being said, I have no objection to labeling this as a bug if the lack of this functionality is causing issues for you.

PR merged and updated core version <IP_ADDRESS> released, closing.

@rafd123 it looks like this commit has broken a fairly specific scenario, so I'm looking at reworking the approach we use to access a service with a path component. Can you provide some information on how you are accessing a service with a path component? I'm trying to figure out how to test your specific scenario so I can verify that my alternative approach doesn't end up breaking you.

The .NET SDK team is planning on making the following change to AWS4Signer, which may end up breaking this specific scenario. Please notify us as soon as possible if it does. Our current plan is to make and release this fix by November 17th, 2015.
protected static string CanonicalizeResourcePath(Uri endpoint, string resourcePath)
{
    if (string.IsNullOrEmpty(resourcePath))
        return Slash;

    if (endpoint != null)
    {
        var path = endpoint.AbsolutePath;
        if (string.IsNullOrEmpty(path) || string.Equals(path, Slash, StringComparison.Ordinal))
            path = string.Empty;

        if (resourcePath.StartsWith(Slash, StringComparison.Ordinal))
            resourcePath = resourcePath.Substring(1);

        resourcePath = path + Slash + resourcePath;
    }

    // split path at / into segments
    var pathSegments = resourcePath.Split(new char[] { SlashChar }, StringSplitOptions.None);

    // url encode the segments
    var encodedSegments = pathSegments
        .Select(segment => AWSSDKUtils.UrlEncode(segment, false))
        .ToArray();

    // join the encoded segments with /
    var canonicalizedResourcePath = string.Join(Slash, encodedSegments);

    return canonicalizedResourcePath;
}

The proposed code will break the scenario of a custom endpoint that has a path component. In it, if resourcePath is null or empty (which is true for EC2 calls, for example), the endpoint's path component never has an opportunity to get included as part of the canonicalized path.
I think modifying the proposed code as follows solves both issues for the version 4 signer:
    protected static string CanonicalizeResourcePath(Uri endpoint, string resourcePath)
    {
        if (endpoint != null)
        {
            var path = endpoint.AbsolutePath;
            if (string.IsNullOrEmpty(path) || string.Equals(path, Slash, StringComparison.Ordinal))
                path = string.Empty;

            if (!string.IsNullOrEmpty(resourcePath) && resourcePath.StartsWith(Slash, StringComparison.Ordinal))
                resourcePath = resourcePath.Substring(1);

            if (!string.IsNullOrEmpty(resourcePath))
                path = path + Slash + resourcePath;

            resourcePath = path;
        }

        if (string.IsNullOrEmpty(resourcePath))
            return Slash;

        // split path at / into segments
        var pathSegments = resourcePath.Split(new char[] { SlashChar }, StringSplitOptions.None);

        // url encode the segments
        var encodedSegments = pathSegments
            .Select(segment => AWSSDKUtils.UrlEncode(segment, false))
            .ToArray();

        // join the encoded segments with /
        var canonicalizedResourcePath = string.Join(Slash, encodedSegments);

        return canonicalizedResourcePath;
    }

FWIW, I looked into why 608fdcf76078abc11893dd7b4cdb6d1c10c3fbb4 regressed WP 8.1. The original intent was to sign the request with the same URL that was to be used to make the request by calling AmazonServiceClient.ComposeUrl() to ensure parity between the contents of the signature and the URL the server will ultimately use to recreate the signature. AmazonServiceClient.ComposeUrl() ends up calling AmazonServiceClient.DontUnescapePathDotsAndSlashes() in an attempt to smooth over differences in the way the Uri class behaves across .NET versions; clearly this is a no-op on WP 8.1 and of course for WP 8.1 the Uri class behaves in the "old" way by unescaping the slashes. What I don't understand is how the final URL for the request is composed before it heads out the door on WP 8.1, and how is it getting away without unescaping the slashes.

Thanks for looking into this. I'm going to test this change with the .NET SDK locally.
Is there any way we can test this change with a service where its url has a path? I don't have access to one. How are you able to test/verify this change?

@PavelSafronov  I submitted a pull request with the changes you suggested along with unit tests to document the intend.
The reason the "custom endpoint with path component" scenario is important to us is because our group is trying to wean existing users off of other hypervisor technologies and use AWS as a standard. We have an AWS adapter in front of our stack to facilitate this, but due to technical restrictions, the endpoint hosted by this adapter needs a path component. The other SDKs we've tested worked out of the box (e.g. boto, the ruby SDK); we just wanted to ensure the .NET SDK didn't pose as an impedance.
To answer you question: the ultimate way this was being tested was to point an AWS client at our custom endpoint (see example in the originating comment of this issue) and ensuring that the server didn't return a SignatureDoesNotMatch error.
Ultimately, it came down to ensuring the endpoint's path components are represented in the canonicalized URL per the signing spec.

@rafd123 thanks for working with us and helping out with the code change. The pull request has been accepted and the new binaries are available now. Closing again. :)

It was a good learning experience. The pleasure is all mine! :)


DataTableColumnDirective Input Binding Changes

Current Behavior: When creating a table using custom column
templates via the DataTableColumnDirective, only the initial
inputs to each DataTableColumnDirective are used to create
column templates. Updating an input binding of
DataTableColumnDirective after
its creation does not result in updated column templates.
Expected Behavior: This PR introduces a service that
DataTableColumnDirectives use to communicate when changes to
input bindings have occurred. DatatableComponent then subscribes
to the service, and recreates column templates when the subscription
emits.
Resolves #1330

What kind of change does this PR introduce? (check one with "x")

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
When creating a table using custom column
templates via the DataTableColumnDirective, only the initial
inputs to each DataTableColumnDirective are used to create
column templates. Updating an input binding of
DataTableColumnDirective after
its creation does not result in updated column templates.
What is the new behavior?
This PR introduces a service that
DataTableColumnDirectives use to communicate when changes to
input bindings have occurred. DatatableComponent then subscribes
to the service, and recreates column templates when the subscription
emits.
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@marjan-georgiev @amcdnl would you be able to take a look? Thanks!

please can you guys merge this PR, this is very much needed otherwise formatting is a big mess


0.6.3 importing enums causes compile to fail

When importing an enum, compilation fails with the follow error:
[error] failed post-processing: 46:29: expected operand, found 'type' (and 10 more errors)
--validate_go_plugin_out: protoc-gen-validate_go_plugin: Plugin failed with status code 1.

Steps to reproduce:

Proto structure:

/common/apis/type/a/a.proto
/common/apis/type/b/b.proto
/path/to/non-common/proto/c.proto

a.proto looks like:

syntax = "proto3";

package common.apis.type;

option go_package = "github.com/ORG/common/apis/type/a;a";

enum A {
  A0 = 0;
  A1 = 1;
}

b.proto looks like:

syntax = "proto3";

package common.apis.type;

option go_package = "github.com/ORG/common/apis/type/b;b";

enum B {
  B0 = 0;
  B1 = 1;
}

c.proto looks like:

syntax = "proto3";
package c;
import "validate/validate.proto";
import "common/apis/type/a/a.proto";
import "common/apis/type/b/b.proto";

option go_package = "github.com/ORG/path/to/non-common/proto;cpb";

message Cs {
    repeated C items = 1;
}

message C {
    common.apis.type.A a = 0;
    common.apis.type.B b = 1;
}

service CService {
    rpc CreateBatch(Cs) returns (Cs) {};
}

Generate Go code

Expected Result: c.pb.validate.go is generated
Actual Result: c.pb.validate.go fails to generate with the error above
Notes

This does work as expected with version 0.6.2
This does work if I combine my enums into a single message and then import that message. e.g.

syntax = "proto3";

package common.apis.type;

option go_package = "github.com/ORG/common/apis/type/enums;enums";

message Enums {
    enum A {
        A0 = 0;
        A1 = 1;
    }

    enum B {
        B0 = 0;
        B1 = 1;
    }
}

And use it in c.proto like:
syntax = "proto3";
package c;
import "validate/validate.proto";
import "common/apis/type/enums/enums.proto";

option go_package = "github.com/ORG/path/to/non-common/proto;cpb";

message Cs {
    repeated C items = 1;
}

message C {
    common.apis.type.Enums.A a = 0;
    common.apis.type.Enums.B b = 1;
}

service CService {
    rpc CreateBatch(Cs) returns (Cs) {};
}

This is not an ideal solution as it forces us to combine all of our common enums into a single file, and causes some redundancies in our usage of these enums in our Go code.

👍

👍

Seeing the same issue

Seems to be caused by an assumption in the go file template that assumes enums are defined under a message type.
Following is an _ variable that tries to make sure the import is used somewhere.
	{{ range $pkg, $path := enumPackages (externalEnums .) }}
		_ = {{ $pkg }}.{{ (index (externalEnums $) 0).Parent.Name }}_{{ (index (externalEnums $) 0).Name }}(0)
	{{ end }}

The Parent.Name_Name pattern is only valid in one case, when the enum is defined under a message. Any other place an enum can be defined will break, producing different kinds of failures.
Error case 1: Enum on its own
If the enum is on its own, the Parent is a file and you get something like
// pkg1/foo.proto
enum Enum { }

// pkg2/foo.proto
import "pkg1/foo.proto"

// pkg2/foo.pb.validate.go
_ = pkg1.pkg1/pkg1.proto_MyEnum(0)

This is clearly not valid go code, and the GoFmt post-processor cannot format it correctly, hence the error.
Error case 2: enum is under a nested message (two messages deep)
OTOH, if the enum is defined under a nested message (message.message.enum), The Parent is just M2, however the enum type includes both M1 and M2. The result is you get a reference to something that does not exist.
// pkg1/foo.proto
message M1 { message M2 { enum Enum { } } }

// pkg2/foo.proto
import "pkg1/foo.proto"

// pkg2/foo.pb.validate.go
_ = pkg1.M2_Enum(0) // This does not exist, instead 'M1_M2_Enum(0)' is the correct value to use.

This causes code that generates successfully but will not compile.

Any suggestion?

We've opted to group our enums into single enums message. This is far from an ideal solution and causes other problems, but they are at least problems we can work around. Wondering if there's any update on the actual fix for this issue?

This is VERY much not a stale issue

I got the same problem too.

So my solution was to clone the repo, fix it myself, build a binary and distribute that within my org. I was going to contribute the fix but never got round to it.

hhh,  I found it was fixed in https://github.com/envoyproxy/protoc-gen-validate/pull/573.


Updated schema doc is missing enums

Intentional?
ImageSize, etc...
https://docs.microsoft.com/en-us/adaptive-cards/create/cardschema

Known issue and will be resolved soon. In the mean time you can view all those properties using the Schema explorer

Duplicate of #883


Validation immediately upon form load

Plunker
Example
Expected behaviour
When the form is loaded it is immediately validated.
Actual behaviour
The form is not validated upon load, it is not validated until the model changes and a validation is triggered by the watch on the model.
Are my expectations wrong, I'd have thought that sending the schemaFormValidate broadcast as the final action in the controller would trigger an immediate validation, showing the errors, but that's not the case. Am I doing something wrong?
@json-schema-form/angular-schema-form-lead

Also, as an aside, when the validation does take place, the first array element does not display the validation message

@AndrewIsh would love to know if anything has changed with the alpha releases regarding this? Probably not, but it would help to know.

This should be working, so I will close this until I hear otherwise.


Names throws TypeError

Sample output:
➤ xplat names --source-dir ~/temp/ --dry-run
Scan 23.pdf
Scan 24.pdf
Scan 26.pdf
Total files found = 3
No output directory specified. Rename files? [y/N]: y
Selected files will be renamed and saved to:
/Users/neil/temp
Rename 3 files of type 'None'? [y/N]: y
Dry run is active, proposed changes won't be saved.
Proposing file name change from:
/Users/neil/temp/Scan 23.pdf
  to:
Traceback (most recent call last):
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/bin/xplat", line 5, in <module>
    app()
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/neil/Library/Caches/pypoetry/virtualenvs/xplat-gXevMVG1-py3.9/lib/python3.9/site-packages/typer/main.py", line 500, in wrapper
    return callback(**use_params)  # type: ignore
  File "/Users/neil/repos/xplat/src/xplat/cli.py", line 189, in names
    rename_total = rename_list(file_list, output_dir, dryrun=dry_run)
  File "/Users/neil/repos/xplat/src/xplat/cli.py", line 78, in rename_list
    new_file_name = renamer.inet_names(f_name, output_dir, dryrun)
  File "/Users/neil/repos/xplat/src/xplat/renamer.py", line 34, in inet_names
    new_stem = new_stem.replace(" ", delim_chr)
TypeError: replace() argument 2 must be str, not bool

Not resolved. Today's error:
➤ xplat names --source-dir ~/Dropbox/cadent/Projects/Erlene/text/convert/ --ext docx
POST #6.docx
Total files found = 1
No output directory specified. Rename files? [y/N]: y
Selected files will be renamed and saved to:
/home/neil/Dropbox/cadent/Projects/Erlene/text/convert
Rename 1 file of type 'docx'? [y/N]: y
Converting file name:
/home/neil/Dropbox/cadent/Projects/Erlene/text/convert/POST #6.docx
  to:
Traceback (most recent call last):
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/bin/xplat", line 5, in <module>
    app()
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/neil/.cache/pypoetry/virtualenvs/xplat-uC98EnBH-py3.10/lib/python3.10/site-packages/typer/main.py", line 500, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/neil/repos/xplat/src/xplat/cli.py", line 222, in names
    rename_total = rename_list(file_list, output_dir, dryrun=dry_run)
  File "/home/neil/repos/xplat/src/xplat/cli.py", line 79, in rename_list
    new_file_name = renamer.inet_names(f_name, output_dir, dryrun)
  File "/home/neil/repos/xplat/src/xplat/renamer.py", line 36, in inet_names
    new_stem = new_stem.replace(" ", delim_chr)
TypeError: replace() argument 2 must be str, not bool


Duplicate symbol pthread_atfork when compiling Android release

Checklist

[x]  I've looked through the issues and pull requests for similar reports

Describe your issue
Building https://github.com/MarcelCoding/zia for target aarch64-linux-android fails to link. The linker fails because of "duplicate symbol pthread_atfork".
What target(s) are you cross-compiling for?
aarch64-linux-android
Which operating system is the host (e.g computer cross is on) running?

[ ]  macOS
[ ]  Windows
[x]  Linux / BSD
[ ]  other OS (specify in description)

What architecture is the host?

[x]  x86_64 / AMD64
[ ]  arm32
[ ]  arm64 (including Mac M1)

What container engine is cross using?

 And it builds fine for aarch64-linux-android if you do the debug build, and not the release build.
Do you know any solution for this?
It is very strange, that only the release version is affected....
It does not matter, if I do it in GH actions, or locally, using the same commands. Same error.

Going to ask the same question here as I asked in #1222, what version of rust is +nightly? cargo +nightly -V
this might be a wrong assumption done for the new ndk
also, did this recently break (e.g has it worked before)? We did a fresh build of the images on :main here, it might've broken something we don't check for

cargo +nightly -V:

sorry, the information-required label is for why pthread_atfork is defined in the zia-client cgu
I'll remove the label :)

Ok, no worries, at least now you have a little more info :wink:
But yes, that was a little misleading :)

Removing these lines solves the problem: https://github.com/MarcelCoding/zia/blob/main/Cargo.toml#L8-L17 I'll further investigate which property exact is the problem.

Removing jmalloc MarcelCoding/zia@9583a8d (#42) is also solving the problem. I don't know why I didn't consider that a problem, sorry for that.

Seems to work with the changes.
@Emilgardis If you want to, you can close this, the workaround is enough. If you want to fix it, to prevent future issues: My replicate script above is version pinned, so you can still use that script.

It seems to be related to this issue, and it has been fixed.


Synchronous mode crashes the provided training script

After #8 the example_sb3_dqn_training.py script crashes when running with synchronous mode enabled. I don't think the patch is erroneous, but rather that before #8 synchronous mode never was enabled because it got reset to False once you loaded a map and the original setting wasn't reapplied. I verified this prior to creating #8 by printing the world settings after loading a map, which showed that synchronous mode was not turned on.
The script crashes in get_data() for the GNSS class in sensors.py with the error message "AttributeError: 'NoneType' object has no attribute 'latitude'"

I don't think that's the case but I won't remove all the cards on the table. Now i won't be able to look at the problem, but monday I'll focus on fixing those problems once and for all. Is that ok?

Fixed the synchronization with the sensors!

It works!


Is it possible debug oracle procedures??

Hello everyone.
I would like to know if it is possible to debug oracle procedures with the community version.
Regards

No, it is not possible.


[BUILD] When building with haddock enabled, build fails

Folloiwing comments breaks the build if build is enabled with haddock using GHC 8.10.2, have not tried to replicate with 8.6.5
https://github.com/input-output-hk/cardano-rt-view/blob/707566aaadc1cda4cd033bf866fb4ab3c0af7f6f/src/Cardano/RTView/NodeState/Updater.hs#L234-L237
Possible workarrounds:

Apply this patch: remove-haddock-problematic-comment.txt
Disable haddock entirely e.q. using RPM macro %ghc_lib_build_without_haddock

Thanks @mark-stopka, fixed.


Release a new version

For someone has ownership

Thank you for your review @shirts I'll get back to your comments


Hide or remove old instances

We still need to implement hiding/removal of old/unseen instances, as described here:
https://docs.google.com/document/d/1p16jeodYUYF9tp2mh9nDVIHlKagzHZLYVHRKZLZkspw/edit#heading=h.kp9aiwk4zvam

Closing as this is not a priority right now


Filter by a range of TrustThreshold values

Closes: #2798
Description

Changes the logic of FilterPolicy::control_client such that it filters out clients that exhibit a TrustThreshold < 1/3 or > 2/3.

PR author checklist:

[x] Added changelog entry, using unclog.
[x] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[x] Linked to GitHub issue.
[x] Updated code comments and documentation (e.g., docs/).
[x] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

I am on it


Adds .Net Publish Build Process Configuration RFC

Summary

Readable
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

I can't think of any reason not to implement this. It looks like all of the other dotnet commands that run an implicit restore aren't commands we support so this is the only buildpack we'd have to change. I'm for it!


Optional args for getProductPlans, getPlanUsers and getPlanPayments

Closes #34
Changed typescript file to make args optional and marked them as option for the JSDoc header too.

@vespasianvs Do you mind just rebasing this branch on top of the latest commits and regenerating the docs again?

Branch rebased and docs regenerated (after some messing about by me!)

Done

:shipit: in v2.5.0


Completed assignment 1

TITLE: UofT-DSI | Shell - Assignment 1
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Create new directories, move data, copy data, copy specific data, delete specific data, and creating a txt file with the list of files and directories, all by using CLI in bash.

What did you learn from the changes you have made?

Learned how to do all of the above locally in bash, test, commit, and push it my Github branch for verification and submission.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I tried different approaches based on the coding I know until I got it right with the feedback from the auto grader checks. I used an iterative method to get used to the bash commands, wildcards and syntax that is thought in the course one assignment question at a time.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
-Followed an iterative approach to target each assignment question until I got it right one by one. Used slack for help and followed the assignment submission guides to figure out the flow.
How were these changes tested?

After the iterative approach of targeting each question one at a time, I created a clean environment by copying the content to the new folder via bash and ran a copy of the final script. Verified that the script was executed successfully by checking the content of the the folder for the expected outcome.

A reference to a related issue in your repository (if applicable)

None

Checklist

[X] I can confirm that my changes are working as intended

Thank you Anjali, will keep mkdir -p in mind for the next time I create multiple directories.


Show dates default to en-GB?!

Eek. I can define show dates just fine in shows.yml but since I'm in Seattle, they need to be defined one day in the future since all dates are being represented as en-GB locale strings.
Can this be fixed?

I’m having the exact same problem on multiple websites. Hope this is fixed soon.

I am also experiencing this issue on https://thomashindsmedia.com/

I wonder if Component Shadowing could be used to work around this problem?  Find/Replace all instances of en-GB with en-US?  https://www.gatsbyjs.org/blog/2019-04-29-component-shadowing/

I've confirmed that simply shadowing "show-item.js" locally and changing en-GB to en-US does not fix the problem. Apparently it's more complex than that.

I have no idea what that means...
On Fri, Jan 3, 2020, 3:59 PM Charles Hood<EMAIL_ADDRESS>wrote:

I've confirmed that simply shadowing "show-item.js" locally and changing
en-GB to en-US does not fix the problem. Apparently it's more complex than
that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekafyi/gatsby-theme-musician/issues/5?email_source=notifications&email_token=AM6HKI77J7YTBDMDZOGZGR3Q36RMTA5CNFSM4J7BE6Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEICCGWA#issuecomment-570696536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6HKI5T36NLQOUGY5CC5SLQ36RMTANCNFSM4J7BE6ZQ
.

Sorry, you could ignore that – – I was supposed to be logged in as me when I posted that bug report.
Thanks,
~C.H.

On Jan 3, 2020, at 4:51 PM, thmshinds<EMAIL_ADDRESS>wrote:
I have no idea what that means...
On Fri, Jan 3, 2020, 3:59 PM Charles Hood<EMAIL_ADDRESS>wrote:

I've confirmed that simply shadowing "show-item.js" locally and changing
en-GB to en-US does not fix the problem. Apparently it's more complex than
that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekafyi/gatsby-theme-musician/issues/5?email_source=notifications&email_token=AM6HKI77J7YTBDMDZOGZGR3Q36RMTA5CNFSM4J7BE6Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEICCGWA#issuecomment-570696536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6HKI5T36NLQOUGY5CC5SLQ36RMTANCNFSM4J7BE6ZQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Okay, first of all, Thanks to @TheRobBrennan for putting me on the right track! I did not implement the fix the same way he did, mine is more of a hack -- but I was able to use component shadowing, which is something I've been wanting to try. I created a src/gatsby-theme-musician/components/ folder and put my modified copy of show-item.js in there. (Again, based heavily on @TheRobBrennan 's work, but I am the lazy so I hardcoded mine to en-US. Here is the new show-item.js:
/** @jsx jsx /
/ eslint-disable camelcase */
// eslint-disable-next-line no-unused-vars
import React from "react"
import { Styled, jsx } from "theme-ui"
const ShowItem = ({ id, name, date, location, info_url, map_url, locale }) => {
return (

<time
dateTime={date}
aria-label={new Date(date).toLocaleDateString("en-US", {
day: "numeric",
month: "long",
year: "numeric",
timeZone: 'UTC',
})}
className="GtmShowItem__date"
sx={{ variant: "components.show.date" }}
>

{" "}
{new Date(date).toLocaleDateString("en-US", {
month: "short",
timeZone: 'UTC',
})}{" "}

{new Date(date).toLocaleDateString("en-US", {
day: "numeric",
timeZone: 'UTC',
})}

{location}
{map_url && (
<>
{" "}
·{" "}

</>
)}

{info_url ? (

RSVP

) : (

No RSVP info

)}

)
}
export default ShowItem


docker image xd009642/tarpaulin:latest is stale

My instance of tarpaulin on CI has started complaining about my compile-fail tests. Here is an example log.
I understand that these types of tests have an inherent flakiness to them, but they're useful for catching proc-macro bugs. Could the xd009642/tarpaulin:latest tag be updated to be built on the latest rust image?

I've just triggered a rebuild in docker hub so once that job is processed it should be up. It's a bit of a pain because it's the latest rust version at the time the image is built and for subsequent updates I have to retrigger it. If you have any docker hub tips to solve this it would be appreciated. However, I'm not holding my hopes too high docker hub often spuriously fails to build without giving any logs...

Well the builds all completed successfully so all the tagged builds (including latest) are now using the latest rust base image

Yep my issue is resolved! Thanks for your help


ListWheel broken since #9c6dd0de

https://github.com/flutter/flutter/commit/5142238c857c9a7f6af0926ea8a8860d9f54bb69
https://github.com/flutter/flutter/commit/9c6dd0de231d2f16352419da6a193c2f9dbad31c

Repro code:
Copied from the "Center child is magnified" test from list_wheel_scroll_view_test.dart but replaced const Placeholder() with a colored container.
void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Flutter Demo',
      home: Directionality(
          textDirection: TextDirection.ltr,
          child: RepaintBoundary(
            key: const Key('list_wheel_scroll_view'),
            child: ListWheelScrollView(
              useMagnifier: true,
              magnification: 2.0,
              itemExtent: 50.0,
              children: List<Widget>.generate(10, (int index) {
                return Container(color: Colors.white),
              }),
            ),
          ),
        )
    );
  }
}

Fixed by https://github.com/flutter/engine/pull/12610. Going to try relanding https://github.com/flutter/flutter/pull/39919 with an additional test.


TypeError: Cannot read properties of null (reading 'registered')

Hi Eric!
I would like to integrate my calendar into my app and I would like to use your solution, but I have a problem. When I run the application for the first time it works fine but after a refresh a nd later on, I got the following issue:
warn  -<EMAIL_ADDRESS>Module not found: Can't resolve './locale' in 'C:\git\MyRepo\node_modules@ericz1803\react-google-calendar\dist'
TypeError: Cannot read properties of null (reading 'registered')
at C:\git\MyRepo\node_modules@ericz1803\react-google-calendar\dist\react-google-calendar.umd.js:53:954
at C:\git\MyRepo\node_modules@ericz1803\react-google-calendar\dist\react-google-calendar.umd.js:52:2177
at renderWithHooks (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:5658:16)
at renderForwardRef (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:5842:18)
at renderElement (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:6005:11)
at renderNodeDestructiveImpl (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:6104:11)
at renderNodeDestructive (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:6076:14)
at finishClassComponent (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:5688:3)
at renderClassComponent (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:5696:3)
at renderElement (C:\git\MyRepo\node_modules\react-dom\cjs\react-dom-server.browser.development.js:5943:7)
Do you have any idea whats the problem?
This is my packet.json dependencies:
"dependencies": {     "@emotion/react": "^11.11.1",     "@ericz1803/react-google-calendar": "^4.3.1",     "@material-ui/core": "4.11.4",     "@material-ui/icons": "^4.11.3",     "classnames": "2.3.2",     "moment": "2.29.4",     "next": "12.2.5",     "nouislider": "14.7.0",     "prop-types": "15.8.1",     "react": "^18.2.0",     "react-datetime": "3.1.1",     "react-dom": "^18.2.0",     "react-slick": "0.29.0",     "sass": "1.66.1",     "styled-components": "5.3.5",     "webpack": "5.88.2"   }

This worked for me: try to create loaded state and change it to true when a component is rendered using useEffect.
function YourComponent() {
    const [loaded, setLoaded] = useState(false);

    useEffect(() => {
        setLoaded(true);
    }, []);

    if(!loaded) return
loading...

    return (

          {loaded && <Calendar/>}

    )
}

@dmitrydnch have you have able to get the tooltip to work for you. if so, how did you implement thru code or is it supposed to work automatically when an event is clicked. Thanks for any help on this time-sensitive issue im having

Just fixed the build process in v4.3.2 which should fix this issue.

@ericz1803 I am still experiencing the issue reported here. Any pointers of what could be done to help :) ?
Cheers and thanks for the project!

Would it be possible to turn this component into a SSR, I could help if provided with some pointers :)
Cheers!


Firmware v14, changelog/release notes, it's not possible to revert to older version after update to v14

Hello. I'm using two different UPS Plus EP-1036 (rev. 5.4 2020-01-05 PCB03a) and (rev. 5.0 2020/12/23 PCB01b) both on Firmware v14. Because I can not revert to Firmware v10. OTA_firmware_upgrade.py failed to download any other Firmware version ("File not found").
Is there a changelog or some release notes available?
I tried the OTA_firmware_upgrade.py Scipt, which allows me to update both UPS-Plus to the latest Firmware v14. It's not possible to download any older Versions ("File not found").
I've downloaded a file from this forum with v10, but I'm concerned to use it to flash v10.
Is a solution/script available to select the firmware version to download/flash it? Thanks a lot!

Hi jens-roth，
Older versions do not support rollback; it is recommended to use the latest version, which is compatible with all hardware and can be used with confidence.


To fix before documenting

Fix the edge case where the initial state is completely filled.
Update tile type counts with the given state tiles at the start.
Change weighting:
3.1 don't multiply cost w/ entropy.
3.2 make the extra weight the avg boundary entropy ( or some other, potentially more useful measure ).

addressed in commit 02a9112f36ff35499999f4eaa48bd5d1573e3ca4.
keeping old costs w/ high random weights at high temperatures is not ideal, but changing this behavior requires a custom search implementation or hijacking the py_search priority queue.
I am not keen on changing the behavior; especially without comparing against normal wfc implementations first.
It may be something to explore in the future, but, for now, this issue is closed.


UI for the ' About ' section on the Landing Page

UI for the ' About ' section on the Landing Page has to be built as per the figma file provided (attachment About.png provided as well)
[https://www.figma.com/file/OiU7Tl4k1YFfqZQXxLaRrl/Reach4Help?node-id=6859%3A2]

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Already completed by @leadersheir 's team


fix(automation): add tar zip step to release script

This resolves a few bugs with the automation script that were due to the idiosyncratic setup I had locally. We don't need sudo to run the cibuild script, and since tar-ing the minified JS doesn't happen by default, that needs to happen within this script.

:shipit:


[TrustedLaunch] Default --enable-secure-boot and --enable-vtpm to true and optional flags in relation to --security-type TrustedLaunch

Related command
az vm create
az vmss create
Is your feature request related to a problem? Please describe.
Today if a customer does not provide the values for --enable-vtpm and --enable-secure-boot, the values are not defaulted to true. This will result in a VM that does not take advantage of the full features of Trusted Launch
Describe the solution you'd like
We would like to make EnableVtpm and EnableSecureBoot as optional flags in relation to SecurityType, and have the EnableVtpm and EnableSecureBoot flags set to true in the background. This will lower the bar of entry even further, provide the customer the full set of Trusted Launch features by default, as well as provide the Guest Attestation extension by default functionality.
Describe alternatives you've considered
n/a
Additional context
az vm create -n "s1win1" -g  sirlinazsub_1 --location japaneast --size Standard_D2s_v3 --image MicrosoftWindowsServer:WindowsServer:2019-Datacenter-Gensecond:latest --admin-username "player1" --admin-password "supersecurepw" --security-type TrustedLaunch
az vmss create -n s1vmss1 -g sirlin5 --location westcentralus --instance-count 5 --data-disk-sizes-gb 2 --os-disk-size-gb 128 --image SUSE:SLES-15-SP2:GEN2:latest --admin-username "player1" --admin-password "supersecurepw" --security-type TrustedLaunch
Both scenarios listed above should result in secureboot enabled, vtpm enabled, and MSI enablement, and Guest Attestation extension installation.

Compute feature

@sirfamelin Please note that if we directly modify the default value of those parameters, it will cause breaking change to users.
I suggest that when the --security-type used by the VM/VMSS creation is TrustedLaunch, we print the warning log to recommend users to specify the --enable-secure-boot True and --enable-vtpm True first.
Since CLI has bi-annual breaking change releases coinciding with Microsoft Build and Ignite. Until the next breaking change window (Ignite Event) of CLI, please submit a new feature request one month before that to change the default value, then we will change the default value in that sprint to avoid the big impact of breaking change on users.

Interesting, I did not realize that --enable-secure-boot and --enable-vtpm were defaulted to false and optional already. It seems like we don't really have choice but to wait until Ignite. Does that mean I should raise the request in the beginning of October?
In the meantime, a warning message will do. Something like "Please set --enable-secure-boot to True and --enable-vtpm to True in order to receive the full suite of security features that comes with Trusted Launch."

@zhoxing-ms just curious about the breaking changes that are allowed. How "breaking" can breaking changes be during the bin-annual releases? I am thinking to potentially change the --enable-secure-boot and --enable-vtpm to switch parameters.

Does that mean I should raise the request in the beginning of October?
In the meantime, a warning message will do.

Yes, we only add warning logs to inform users of migration usage first and then wait until the next breaking change release Ignite to modify the default behavior, this is the best solution I can think of that will have the least impact on users at present.
For more details about the CLI breaking change process, you can refer to this draft guideline doc link

Okay. For the change to add warning logs, can we target the August release or is it too late?

@sirfamelin We can target the August release, @PARADISSEEKR please be aware

@zhoxing-ms , I just got around to checking the PR, I see the message to encourage users to enable vtpm and secureboot, but I don't see the warning message of the upcoming breaking change.
On that note, I would like to revisit enabling vtpm as a breaking change. By enabling vTPM by default, there's no impact or disruptions to customer workloads. Secureboot is a breaking change, I agree, but I think vTPM is not indicative as a breaking change.

@sirfamelin I'd like to ask why enabling vTPM will not cause breaking change to users? Could you guarantee that the vTPM will only add new feature to users, but not affect the existing features of users?

By the way, since this question was raised late and we will launch the release of the new CLI version tomorrow, I suggest that the warning message of breaking change and the behavior of enabling vTPM by default be added in the next sprint (09-06), do you think it's acceptable?

@zhoxing-ms, that is correct. Vtpm will not cause breaking changes as it will not affect existing software. The vTPM provides cryptographic functions and key storage. It should not impact customer workloads.

that is correct. Vtpm will not cause breaking changes as it will not affect existing software. The vTPM provides cryptographic functions and key storage. It should not impact customer workload

@sirfamelin Got it, thanks~

Is 09/06 the release date?

Yes


path.extname is deprecated.

Argument to path.extname must be a string
path.extname (/usr/share/atom/resources/app.asar/src/electron-shims.js:20:10)
ATPOutputView.getCwd (/home/buca/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:1874:17)
BuiltinVariables.parseFull (/home/buca/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:188:77)
BuiltinVariables.parseHtml (/home/buca/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:111:16)
ATPOutputView.parseTemplate (/home/buca/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:664:36)
ATPPanel.updateStatusBar (/home/buca/.atom/packages/atom-terminal-panel/lib/atp-panel.coffee:190:50)

I am new to linux and atom and attempting to set up atom for C++ development on this platform.
atom-terminal-panel deprecation warning with no updates or replacement for the plugin.
is there another plugin I can use in its place.


[Marker.io] please link "view all" links present in different sectors on sectors page

go to sectors page
scroll down and click "view all" links for different sectors.

they are currently not configured. please configure them before going live.

Reported by: vinod<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://kensway.webflow.io/sectors
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1366 x 768OSWindows 10 (10.0.0)Viewport Size1366 x 641Pixel Ratio@1x

they are now configured but they take to projects page without filtering.
e.g: if we go to residential section on "sectors" page and click the "view all" link, it takes to "projects" page but shows all the projects( they are not filtered by "residential" )
please check this with the team or client. if current behaviour is correct, this can be ignored.


Content Test - contains check

@benjaminDC suggested to extend the content test validation with a "contains" check.
The current "value" check for content test, does a strict check verifying if the value matches. The "contains" check would do a fluid check, verifying if the string is present in the value to test.
Example:
{
"contentTests": [
      {
        "openApiOperationId": "post-oauth-request-token",
        "responseBodyTests": [
          {
            "key": "token_type",
            "value": "Bearer"
          },
          {
            "key": "scope",
            "contains": "broadcast:push"
          }
        ]
      }
    ]
}

Where the "scope" would return a whole range of scopes, like "send:sms send:email broadcast:push"

This will be part of the 1.10 release.

Closing this request, since the feature is included in the 1.10.0 release


fix(bi-link): isRelativePath invalid in vuepress

Rebase / Merge is required.

As suggested in https://github.com/nolebase/integrations/pull/361#discussion_r1872462228

We should write the comments to tell the later coming developers why these properties, and perhaps we can file up an issue to VitePress to ask them whether this is the unexpected behavior.

Or at least, link this PR in the comment.


[v1.0.4, macOS] Recursive opens only top level

On macOS, running v1.0.4, I ensured appropriate extension settings then attempted to recursively open a folder.  Only the one file opened.  Attached video shows:

the file layout
the extension version
the extension settings
the failure to open more than one file.

https://user-images.githubusercontent.com/160164/182007967-7cb4abb8-8adc-4044-a46c-0171fcf2af9b.mov

A very, very similar extension here (which almost definitely looks like it may have been originated from your work, but with all credit to you removed) works as expected on my machine.

Really appreciate the detailed issue, and notifying me of the other project, it's a shame they didn't just submit a PR to help fix it but to each their own!
I see in the 'forked' version from @EuroCent82 that it's just interpolating the filepath, which I feel would be broken on windows so want to narrow down the best way to handle it with VS api.
Yesterday I was sorting windows pathing, so not sure if thats where it broke on macOS for you (I might just get a windows VM so I can fix it on both without breaking 🙃).
Good news though! Whilst I wasn't able to reproduce in a similar way to you, playing around this morning noticed some clear issues, I just published v1.0.5 which I think should be a lot better, switched from the node fs library to vscode.fs which is what they recommend (didn't know this at the time) - so I'm hoping this should be better.

I apparently have v1.0.7 now so I can't specifically comment on the v1.0.5 mentioned above, but I can confirm that the current version is working perfectly.  Many thanks!

I apparently have v1.0.7 now so I can't specifically comment on the v1.0.5 mentioned above, but I can confirm that the current version is working perfectly.  Many thanks!

Awesome thanks for checking back in! I switched over to using vscode fs library as its their recommended approach now - might look to add more features going forwards for things like filtering file patterns


rails-dev-boost incompatiblity

Hey guys,
sorry, if the issue tracker isn't the best place for this.
I'm the maintainer of the rails-dev-boost (a Rails plugin that speeds up development mode by reloading only the changed ruby files in between requests). I have users reporting incompatibility with mongoid here: thedarkone/rails-dev-boost#27.
Basically the only thing that should be able to trip rails-dev-boost up are "class-level" references between reloadable constants, the plugin already ships with custom ActiveRecord-handling code: see here.
I went over mongoid's codebase looking for class-level caching behaviour and only found 3 things:

Mongoid::Relations::Metadata's @klass and @inverse_klass instance vars,
Mongoid::Document::ClassMethods's @_types cache.

Here's my current mongoid-specific patch: see code here.
Users are reporting that this is not enough, this means I must have missed something. Can any of the mongoid's devs help me out?
Thanks!

Hey there, @thedarkone. Does this occur with all Mongoid models, or just those that inherit from other models?
Is model preloading enabled?

Hey @nickhoffman, I don't use mongoid myself, so I don't know anything beyond what has been reported in that ticket. I'll repost your questions.
The model preloading code as seen here seems to be pretty harmless, it Dir.globs for *.rb files and then issues a bunch of require_dependency calls, it does not keep any references to the models' constants.
What I'm looking for is the code that stores references to other mongoid models' classes like the 3 examples I posted above. You can also have a look at the rails-dev-boost's README under the "Class-level reference examples" section.
Thanks!


Issue with k8s.io/docs/tasks/administer-cluster/kubelet-config-file/

This is a Bug Report

Problem:
Pointing people at types.go for a configuration reference is a fairly useless way to describe a configuration file since it requires people to be basically familiar with reading Go and understanding the structure.  It's unclear how to generate the correct structured YAML from that source file.
Proposed Solution:
Add an example config.yaml to the page that somebody can use to derive their own.  The docs as they exist assume that all users are going to run kubeadm init.
Page to Update:
https://kubernetes.io/...

page https://kubernetes.io/docs/tasks/administer-cluster/kubelet-config-file/
/kind feature

/priority backlog


To Do

[x] Move to NeoVim
[x] Add current ZSH theme
[x] Break Up vimrc & nvimrc
[ ] Find all recent Homebrew additions
[ ] Use Oceanic Next Colorscheme Everywhere
[x] Use iTerm 2 Beta

[ ] get oceanic next for iterm in the svn call
[ ] configure symlinks for dotfiles
[ ] installer for zsh config

The Neovim executable is called nvim. To use your existing Vim
configuration:
 ln -s ~/.vim ~/.config/nvim
 ln -s ~/.vimrc ~/.config/nvim/init.vim
See :help nvim for more information on Neovim.

init script
bash -c "$(curl -LsS https://raw.github.com/charlespeters/dotfiles/master/scripts/init.sh)"

bash <(curl -s https://raw.github.com/charlespeters/dotfiles/master/scripts/init.sh)


The JSON file contents should get included when using a loop in a flow of a scenario

The JSON contents does not get included when using a loop construct in a flow, example code:
config:
  target: http://targetmachine.com
  plugins:
    json-include: {}
scenarios:
  - flow:
      - loop:
          - post:
              url: "/route"
              json: "!include(relativePath.json)"
        count: 1

As soon you remove the loop level, it works.
UPDATE: the same issue for a parallel item. Note that this construct is technically undocumented, however it can be used. See here for more info about that.
Example with a parallel construct:
config:
  target: http://targetmachine.com
  plugins:
    json-include: {}
scenarios:
  - flow:
      - parallel:
          - post:
              url: "/route1"
              json: "!include(relativePath.json)"
          - post:
              url: "/route2"
              json: "!include(relativePath.json)"

I'll have to dig onto that, no clues of why it don't work


Suggest clean up of usage for #1410

Signed-off-by: Eric Sage<EMAIL_ADDRESS>

Why not include some sample output?  It would save time in reviewing the PR.  Generally I'm kind of skittish about changing the help text.

Sure, I will post some sample output. Looking through the current descriptions, they seem to augment Usage, as the use of Usage doesn't really provide much in the way of useful information. By interpreting Usage as the unix style sample usage, and the description as describing what the command does, they have a better separation of concerns in my eyes. I think it would be even better to have an additional field in the template, but I wanted to go ahead and throw this out there. It seems to be difficult to add fields to the struct that gets applied to the templates without making some changes to codegansta/cli.
./machine

Commands:
  active		Print the active machine.
  config		Print the specified machine's connection configuration.
  create		Create a machine with the specified name.
  env			Display the commands to setup up the environment for the Docker client.
  inspect		Inspect information about a machine..
  ip			Get the ip(s) of one or more machines.
  kill			Kill one or more machines.
  ls			List machines.
  regenerate-certs	Regenerate TLS Certificates for a machine.
  restart		Restart one or more machines.
  rm			Remove one or more machines.
  ssh			Establish a secure connection to the machine, optionally supply a command to run on the target.
  scp			Copy files securely from host to the target machine, or vice-versa.
  start			Start one or more machines.
  stop			Stop one or more machines.
  upgrade		Upgrade one or more machines to the latest version of Docker.
  url			Get the url of a machine.
  help, h

./machine help ssh

Usage: docker-machine ssh <machine-name> [command]

Description:
   Establish a secure connection to the machine, optionally supply a command to run on the target.

Cool, thanks.
@hairyhenderson Having done some work on the help text before, WDYT?

Hmm... This is good in principle, but the whole Usage vs. Description thing is a little weird. The way the code stands right now, the auto-generated help command is missing its description, which looks a little odd.
Maybe if there's a way to get the help command's Description field populated, it would work...
It does seem that there's some use cases that aren't fully supported in codegansta/cli, so probably would be good to issue some PRs there too.

@ericsage would you be willing to look at codegangsta/cli to see if we can address there instead of this way as @hairyhenderson suggested?

I'm going to close this -- IMO it deviates too much from the prescribed codegangsta/cli style.  Please feel free to open an issue to discuss if you are adamant.


UI/UX: Company dashboard clean up

It's time that the company gets a more uniform look. This shouldn't be as expansive as the content mod one. Just minor touch ups.

Closed in #232


How are you implementing the navigation bar header?

I'm learning, so bear with me. :-)
In all your Exponent examples there is a header bar with a screen title.
But, when I go to an example I don't see how the header (the section above the tabs) is created.
I'm looking to do this as recommended and need a little help.
Thanks!

It's here https://github.com/react-native-community/react-native-tab-view/blob/master/example/main.js#L162


Move scores calls out of profile

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
#1351
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
Moved Score Calls out of profile page. They are not needed there.
Moved to where they are used.
NOTE: Profile displaying data needs to be tested. No changes around updating data, only displaying in profile and subsequent profile screens like Contribution Score Screen and Citizen Upgrade Screen
👯♀ Paired with
Solo

@RaulUrtecho  pls review this code since u wrote it

Code looks fine just verify the ScoresViewModel.
BTW it would have been great you update those Blocs to 7.2v

Agree, ill update the blocs


Week 1 exercise

The Week 1 exercise is a bit heavy on the text and needs more interactive parts.

Closed with #31


Add parameters to disable HostPorts for gateways, leafnodes and websockets

PR related to https://github.com/nats-io/nats-operator/issues/300
Restructured args for natsPodContainer() a bit and introduced a struct of bools to control if HostPorts should be used or not.
I also added a check against AdvertiseExternalIP which ignores these new parameters if that is true. It makes no sense to advertise external IP without using a HostPort?
Tested in my environment with success.

I updated the master branch after the other PR. Does this look ok?


Run processes in parallel and wait

It would be nice if basic parallel processing was added. Consider this target:
dev-watch game:
	#!/bin/bash
	just dev-game-assets {{game}} &
	just dev-client {{game}} &
	wait

I'm using bash to run these two targets in parallel and then wait for them both. In this case the processes don't terminate on their own (they are watching files and updating builds).  By using wait I'm able to press Ctrl+C to terminate all child processes.
As backgroung execution is already supported, I think the addition of wait might be enough -- as a keyword to indicate that the wrapping just should wait for all children to complete (and of course kill all children if interrupted).

I think this is a dupe of #626, commenting there.


Import SVG?

Would like to know if anyone would find this feature useful.

In my case it's useful because i work in a mixed team programmer and designer so when i found makerjs and i start to experiment i looked up in the api a function to load the svg. So i  can reuse work made by designer and in some case i could be more fast in my work.

For this to happen, will depend on Bezier support. https://github.com/Microsoft/maker.js/issues/96

Closing this issue for now.
There's a lot of SVG (such as fills, gradients, etc.) that does not have a similar concept in Maker.js which only deals with outlines.
I've added a feature to import just SVG path data: http://microsoft.github.io/maker.js/docs/api/modules/makerjs.importer.html#fromsvgpathdata


Odd behaviour between Conductor, Rx and Mosby

Hey, I just posted this question on SO because I'm struggling with screen rotation and navigating away from my first screen.
I explained well on SO, so to not be repetitive I'll keep this issue small.
The project I mentioned on SO is : https://github.com/leonardo2204/ConductorSample
Thanks for the great work !

Hi, I will checkout your example in the evening.

or rotate, the problem described starts.
rotation should work, that might be a bug

navigate away from the FirstScreen or rotate the device, after Rx onNext callback, the isViewAttached()
This actually is the correct behaviour as for now, but I might change that in Mosby 3.0  / mosby-conductor 1.0. The problem is that if you Navigate away, the view will be destroyed. If there is no View what should the Presenter update then? So this is more a philosophical question I know.
My inital thought was,  that if you want to do so Conductors setRetainViewMode(RETAIN_DETACH) mode would be an option, but I'm not sure if this will leave to memory issues in big apps.

Therefore I thought, it would be better to go the simple way, if there is no view then Presenter can't do nothing and should unsubscribe from observables in presenter.detachView(false), then once the user comes back to your first screen presenter. public void subscribe(@NonNull UseCase useCase, @Nullable Bundle bundle, final boolean pullToRefresh) will be called again.
feedback is very welcome.

First of all thanks for the reply !
I'll wait for some news on the 'likely' rotation bug.
As per the Rx issue, the problem is that I'm back on the view that calls the Rx, so in my mind it makes no sense that the isViewAttached() returns null. I guess it will be easier to see the code !
Again thanks for the help and the excellent lib !

Is your sample project up to date? I only see a white screen when starting the app ...

Sure, no problem.
To reproduce:
First click on the text on the toolbar (the contact list will be updated). The text means to filter contacts with number only, or not.
Rotate the device.
If you click again nothing will happen, because the Rx onNext callback never gets called, since onViewAttached() is false.

I found the issue: This is not a Mosby-Conductor problem. this issue has been caused by dagger and the way you inject the presenter.
You are creating and injecting a new Presenter instance on every screen orientation change in your onViewBound() method. That means every screen orientation a new presenter will be created.
Moreover, since you are declaring your own field for presenter you are not using MvpLceViewStateController field for Presenter. See: https://github.com/sockeqwe/mosby-conductor/blob/master/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/conductor/MvpController.java#L19
Mosby internally works as follows:
if (getPresenter() == null) {
     setPresenter(  createPresenter() );
}

In MvpController (the base class of MvpViewStateLceController you are extending from getPresenter() and setPresenter() are using MvpController.presenter field.
https://github.com/sockeqwe/mosby-conductor/blob/master/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/conductor/MvpController.java#L19
There are two solutions:

Don't declare your own presenter field but rather do something like this (then it will use  MvpController.presenter field):

 @Override
    public FirstScreenPresenter createPresenter() {
        return DaggerFirstScreenComponent.builder()
            .activityComponent(((RootActivity) getActivity()).getActivityComponent())
            .firstScreenModule(new FirstScreenModule())
            .build().presenter();
    }

This ensures that a presenter instance will be created only once since createPresenter() will only be called once.

If you want to use your own field presenter then you also have to override setPresenter() and getPresenter() in your controller that extends from MvpLce

public class FirstScreen extends BaseController<RecyclerView, List<Contact>, FirstView, FirstScreenPresenter> implements FirstView {

    private static final String MUST_HAVE_NUMBER = "must_have_number";
    @Inject
    protected FirstScreenPresenter presenter;

   @Override
    public FirstScreenPresenter getPresenter(){
        return presenter;
    }

    @Override
    public void setPresenter(FirstScreenPresenter p){
       this.presenter = p;
    }

because of the way Mosby internal works it calls getPresenter(), createPresenter()  and setPresenter(). But still, you have to ensure not to create and inject a new Presenter in your onViewBound() method.
Btw. you will run into  same "problem" if you use Mosby with Activity / Fragments

First of all, I really appreciate all your explanation and help, I'll dive deeper into your code to learn more.
I think I solved the problem by creating a onCreate method as seen on Conductor #34:
@NonNull
    @Override
    protected View onCreateView(@NonNull LayoutInflater inflater, @NonNull ViewGroup container) {

        if(!created){
            onCreate();
            created = true;
        }

        View v;
        v = inflateView(inflater, container);
        ButterKnife.bind(this, v);
        onViewBound(v);

        return v;
    }

And use like this:
@Override
    protected void onCreate() {
        DaggerFirstScreenComponent.builder()
                .activityComponent(((RootActivity) getActivity()).getActivityComponent())
                .firstScreenModule(new FirstScreenModule())
                .build().inject(this);
    }

I tested and it works. So, is that a good solution ? Should I save the created variable on saveInstance ?
Thank you so much again !

You are welcome. No you don't have to save that in onSaveInstanceState() ...
For now forget everything I mentioned before about
onViewStateInstanceRestored() ... you only run into this if you use hot
observables like sqlbrite, then you have to write 2-3 lines of code to
solve that manually, but for now it seems like you were using rxjava as
async task replacement and in that case everything should work out of the
box...
Leonardo Ferrari<EMAIL_ADDRESS>schrieb am So., 22. Mai 2016,
19:07:

First of all, I really appreciate all your explanation and help, I'll dive
deeper into your code to learn more.
I think I solved the problem by creating a onCreate method as seen on Conductor
#34 https://github.com/bluelinelabs/Conductor/issues/34:
@NonNull
@Override
protected View onCreateView(@NonNull LayoutInflater inflater, @NonNull ViewGroup container) {
    if(!created){
        onCreate();
        created = true;
    }

    View v;
    v = inflateView(inflater, container);
    ButterKnife.bind(this, v);
    onViewBound(v);

    return v;
}

And use like this:
@Override
protected void onCreate() {
DaggerFirstScreenComponent.builder()
.activityComponent(((RootActivity) getActivity()).getActivityComponent())
.firstScreenModule(new FirstScreenModule())
.build().inject(this);
}
I tested and it works. So, is that a good solution ? Should I save the
created variable on saveInstance ?
Thank you so much again !
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/sockeqwe/mosby/issues/136#issuecomment-220843662