CLI arguments that make sense

I am having a lot of trouble with command line arguments. It would be nice if A) topshelf did not die at unknown cli arguments. B) followed its own standard for cli argumets. C) made built in arguments available through the current cli integration.

The help text explains the arguments pretty well, is there something missing or a clear course of action to be taken here?

These are all issues I encountered while using topshelf on windows 8.1
Spaces in arguments do not work. If I try to wrap the arguments in quotes topshelf recognizes it as two different arguments and then dies because for some unknown reason it dies on unknown command line arguments.
If I leave the spaces windows mangles the arguments and I dont get the full value.  You also pass commands to the service using the -name value format for the top level values, but when I add my own you force the --name:value paradigm. There should be a uniform way of handling all CLI arguments. I think my original post made it pretty clear what I would like to see:
A) the service does not die when it encounters un-known command line arguments.  This at the very least should be toggle by the consumer.
B) Follow its own conventions for cli arguments: passing agruments to topshelf should be the same for all types of arguments. I don't understand what it not clear here.
C) built in arguments are available to the service: when you pass -servicename servicename you cannot parse this out using the cli argument methods provided by topshelf.
The documentation on the built in cli arguments is ok its not clear what is a switch and what is an option, but there is no documenation that I could find on defining custom arguments other than a few messages on a google group, and the processing of those arguments is very shoddy and prone to errors.

I would like to see Topshelf parse out the arguments that the library itself uses, and makes the rest, as a string[], available to the hosted service in some form.

Because of the way that services are launched by Windows custom command line arguments are tricky, will behave un-intuitively, and we're unlikely to support them. If you need configuration, use the app.config.

I cant use the app config. I am using topshelf as a wrapper to launch another (non .net) application as a windows service. I pass along configuration information via command line options. The service launched by the application changes based on the configuration so configuring the topshelf wrapper via an app config is not a possible solution since the same package will be used to launch multiple windows services all with different configurations, and forcing consumers of my package to edit the configuration file is a horrible/error prone idea. Even if you cannot pass command line options to the running process, topshelf still does dumb things like crash the process on unknown cli arguments, and does not give access to the built in cli arguments with the interface you supply.

I cant use the app config. I am using topshelf as a wrapper to launch another (non .net) application as a windows service. I pass along configuration information via command line options. The service launched by the application changes based on the configuration so configuring the topshelf wrapper via an app config is not a possible solution since the same package will be used to launch multiple windows services all with different configurations, and forcing consumers of my package to edit the configuration file is a horrible/error prone idea. Even if you cannot pass command line options to the running process, topshelf still does dumb things like crash the process on unknown cli arguments, and does not give access to the built in cli arguments with the interface you supply.

I hear your complaints about handling of the command line, but what you miss is that when the program runs as a service those command line arguments used during installation are lost.
I could imagine a world where you edit the app.config based upon the passed configuration, which we don't currently enable. That would primarily be in your hand to implement though.

I know this is an old issue, but some of the problems seems unsolved - specifically when using multiple instances of the same service, which means I can't edit the app.config on install. When installing using sc.exe I can just add the arguments inside the binpath like "MyService.exe -args:abc" and then they will be picked up on start too. Wouldn't it be possible for topshelf to append the command line arguments the binpath on install? Maybe optional... maybe as a command to customize the binpath directly.
Or am I missing something? :)


map_into doesn't work with tuples

use itertools::Itertools;

fn main() {
    // works
    let foo = vec![1u8, 2u8, 3u8].into_iter();
    dbg!(foo.map_into::<usize>().collect::<Vec<usize>>());

    // doesn't work
    /*
    let bar = vec![(0u8, 1u8), (2u8, 3u8), (4u8, 5u8)].into_iter();
    dbg!(bar.map_into::<(usize, usize)>().collect::<Vec<(usize, usize)>>());
    */
}

This isn't an issue with itertools. Your 'buggy' example can be simplified down to this, which also won't compile:
fn main() {
    let _ : (usize, usize) = (0u8, 1u8).into();
}

The rust core library simply does not provide an implementation of From<(u8, u8)> for (usize, usize).


NIFI-2754

If the Active queue is empty and the number of FlowFiles added to the queue is perfectly splitable by the current Swap size (100000 Flow Files / 20000 files per swap file = 5 with no remainder), then no FlowFiles will move to Active and all will remain in Swap.
Apart from NIFI-2754, there was also a style violation in the existing code, which I fixed.

Thanks for the PR @patricker! The proposed solution makes sense to me but I would love to get @markap14 input before signing off as I know he's much more familiar with this part of the codebase.

@patricker @mcgilman looking at the PR, I do think that this will resolve the specific use case that you're running into, Peter. I think, though, that it is addressing a very specific corner case and could actually be solved a little more generally, though. Instead of changing the number of swap files based on the specific condition of activeQueue.size() == 0 && numSwapFiles > 0 && swapQueue.size() % SWAP_RECORD_POLL_SIZE == 0 what do you think of just calling migrateSwapToActive() at the beginning of the writeSwapFilesIfNecessary()? This should also ensure that we keep with the same ordering guarantees that we have provided throughout the rest of the class.

@markap14 I changed the code per your suggestion and the test I wrote still worked as expected.
PR updated.

Thanks @patricker! Following Mark's suggestion and your update, I've gone ahead and merged this into master.


-m option not working

What steps will reproduce the problem?
1. invoke with -m<module> option

What is the expected output? What do you see instead?
error happens

What version of the product are you using? On what operating system?
latest version, OS X

Please provide any additional information below.
the variable 'name' used in function print_module in the following file should
obviously be 'module':
http://code.google.com/p/ctypesgen/source/browse/trunk/ctypesgencore/printer_pyt
hon/printer.py

Original issue reported on code.google.com by yizhang84 on 1 Oct 2011 at 8:04

Fixed with recent patch


Agreement on UI rendering elements

For NHS 111 work in Meddbase, we have taken the approach to use a simple UI approach to demonstrate what is displayed in the UI to the users.
However, the requirements for the UI rendering is not coming from the end-users, but from NHS 111 assurance process. I just wonder if we can agree on what should be displayed for the UI, I think before the adapter work, the only requirements for UI rendering is revolving around the transformation stylesheet, but with the adapter work, it is not possible to use the stylesheet anymore. But we can still use it to identify elements that need to be displayed.
For NHS adapter work, Initially, I have used the old stylesheet to identify elements that need to be output.
David waters give me the Integrated Urgent Care stylesheet, and it seems it outputs more elements.
So, before I am going to do more work, can we agree on this issue on whether
(1) the old stylesheet should be enough to identify what elements should be displayed in the UI
(so I will do no more work as that is what I have done at the moment)
(2) using the new stylesheet to identify what elements should be displayed in the UI
(3) a new agreement on what elements should be displayed as MUST (or probable categorized as MUST and SHOULD, with SHOULD can be decided by the main application), and this should be documented as part of the adapter work and feedback to the accreditation process for new suppliers using NHS111 adapter.
I think the adapter work should be in async with the whole accreditation process for NHS111, including on the application side, and thus the UI rendering should be relevant to this indirectly.
Just let me know the decision.
Attached is the output from the new stylesheet, the output from the old stylesheet can be seen in #123

We understand that you are asking is UI rendering part of the accreditation process for NHS111, which it is not. Based on this we think we can close this, please confirm. @Kusnaditjung

ok, based on the reply, my understanding is I will need to omit some of the current rendering parts, as they are not available on the adapter at the moment though they are on the original CDA. There is one element however I think it will only make sense if they are available such VersionNumber. This element and SetID are used to decide whether the CDA document is a replacement document or the original document. Can you confirm this for me with NHSD (e.g whether NHS 111 provider may send a replacement document, otherwise we may need to assume it will be always = 1)

@Kusnaditjung What do you do with the MessageHeader that the adapter outputs?

@davidwaters33 At the moment, the message header carries two ITK elements, addressList and ITK Interaction (as adapter 0.3.0, I think 0.4.0 should be the same),  the addressList is used to route the report to the correct recipient organization, while the ITK interaction is used to decide whether the report required action (if primary type interaction), or only for the information to the recipient (if copy type interaction). Please check if this is the correct usage.

@Kusnaditjung That's fine - I was more asking what the receiving organization will do with it?  Basically, if they are then going to store in a FHIR server, we can look at how the normal FHIR way of doing versions of resources works (basically, this would be a history of the Bundle) but if they aren't doing that, we may need to be more explicit in how we carry/express any version number
As an aside, this may not be necessary, as we don't believe that any supplier will ever send replacement documents, but I'm checking that, as well

@davidwaters33, ah okay, so one element (address list) is used to route by the integration engine and also stored in the recipient organization audit (as the address list may contain more information other than the ODS), and the interaction type is used in the organization inbox workflow for the report action.
For the FHIR bundle history version, we don't store them in an FHIR server, and this is an unlikely scenario for us as this is a transient FHIR (there is no actual server that outputs the NHS 111 report which is also an FHIR server), so we treat them as a mere message wrapper rather than the actual FHIR resources.
The VersionNumber I am referring to is the VersionNumber of the CDA,  if they are going to be mapped in the FHIR output, they can be in the extension (either Bundle or composition), or any other places that you think may signify them better.
I have one question as I have no idea what the answer is. Who normally implements the  NHS 111 supplier systems or call center, is it NHS or any parties can implement them based on the specification? I think if the answer is NHS then NHSD can override the specification and make that assumption, but if the supplier system can be implemented by any parties then it is futile to make that assumption, as the new supplier may not do the same as previous suppliers did unless NHSD also changes the specification at the same time.

@Kusnaditjung So, if you don't store the Bundle, what would you do with an updated report if you got one?  How would you recognise it was an update, if you didn't have the original stored?

With regard to the second question, 111 supplier systems are built by third parties

@davidwaters33 hi, sori I think you misunderstand what I mean. We don't have a special FHIR server to store the bundle, but we do store the FHIR in our audit in integration service, and more importantly, when we send the document to the main application in our format, we provide the metadata data with that, e.g equivalent to SetId and Version Number, so using this metadata we can track if there is a replacement coming in in the future.   For the second question, if the NHS 111 is built by third parties, that make this issue more important, as the third parties develop based on specs, and not necessarily follow the exact same path of the previous implementation.

Adapter will be changed to map versionid to the Bundle.identifier

@Kusnaditjung closing as discussed. This is now fixed.


ARTEMIS-149 Advisory Message Support

   Adding functions to send advisory messages.

Can you take a look @clebertsuconic ? Thanks.

I need your help with something.. right now I can't include the activemq-tests on the PRs because they are using invalid javadocs.. do you think you could fix that? remove the javadocs?

do you mean remove the javadocs in every java file in amq5 tests? I can do that.

No.. just some XDoclets that are breaking the build..
use mvn javadoc:javadoc on the folder and you will understand

btw I only get 6 warnings running mvn javadoc:javadoc. I'll fix those warnings anyway.


The syntethic workload is not working using command line

I tried to simulate batch workers, using the synthetic workload of generating 4096 bit RSA keys through the command line but I got some error messages.
I can do the same simulation using the UI successfully.
I´m using the following command:
kubectl run -i oneshot --image=gcr.io/kuar-demo/kuard-amd64:blue --restart=OnFailure -- --keygen-enable --keygen-exit-on-complete --keygen-num-to-g
en 10
Here are the events:
Events:
Type     Reason     Age               From                               Message

Normal   Scheduled  96s               default-scheduler                  Successfully assigned default/oneshot to aks-nodepool1-11892983-1
Normal   Pulled     7s (x5 over 95s)  kubelet, aks-nodepool1-11892983-1  Container image "gcr.io/kuar-demo/kuard-amd64:blue" already present on machine
Normal   Created    7s (x5 over 95s)  kubelet, aks-nodepool1-11892983-1  Created container oneshot
Warning  Failed     6s (x5 over 95s)  kubelet, aks-nodepool1-11892983-1  Error: failed to start container "oneshot": Error response from daemon: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "--keygen-enable": executable file not found in $PATH: unknown
Warning  BackOff    1s (x8 over 93s)  kubelet, aks-nodepool1-11892983-1  Back-off restarting failed container

I ran into this as well. I found this link which also mentions the issue and the fix that worked for me.
Errata: https://www.oreilly.com/catalog/errataunconfirmed.csp?isbn=0636920043874
Fix:

Changing the image to "gcr.io/kuar-demo/kuard-amd64:1" seems to solve the issue.


fix: 返回region external_id

这个 PR 实现什么功能/修复什么问题:
返回region external_id
是否需要 backport 到之前的 release 分支:

release/2.12

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #4040 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #4040      +/-   ##
=========================================
+ Coverage    6.91%   6.92%   +<.01%
=========================================
  Files         506     506
  Lines       85445   85369      -76
=========================================
  Hits         5911    5911
+ Misses      79033   78957      -76
  Partials      501     501

Flag
Coverage Δ

#aFlag
6.92% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/cloudregionresource.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/elasticcache_accounts.go
3.87% <0%> (+0.04%)
:arrow_up:

pkg/compute/models/elasticcache_skus.go
7.63% <0%> (+2.75%)
:arrow_up:

Continue to review full report at Codecov.

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

/lgtm

/approve


Question: Binary data?

Would it be possible/desirable to be able to send and receive binary data?

The JS WebSocket object supports sending binary data, so it makes sense for billstclair/elm-websocket-client to have a sendFile function to use it. I don't have any uses for that, and I'm unlikely to spend the effort on it unless somebody convinces me it would really help them, since I'm hoping that Evan will build an official package, but I'm happy to do it if so motivated, or review and include a pull request.


Too confusing

I'm confused by this and wasn't able to get 4 wild cards to work.
Can you clarify with a downloadable workflow that actually uses several wildcards effectively?
I'm at odds and think there may be a more polished tool to get wildcards working easily, but this one seemed so close by all appearances to what I need.

wildcards.json

Great! Thank you! I'd append it and label it clearly as Example Workflow. In case others have the same difficulty.


Address manifest sync mechanism in ui-plugin-charts

The syncing mechanism is the ui-plugin-charts doesn't work because of branch protection in the main branch. We should fix that in order to have the mechanism running without having to lift branch protection every time we want to sync a new extension to the official Rancher extensions helm chart

Update: created ticket to EIO team to be able to leverage Github Apps to bypass the main branch protection


Question, LibDBIcon vs LibDBCompartment

Hey @Meorawr, thanks so much for creating this library.
My question to you is, now that LibDBIcon has added support for the addon compartment natively, is there a strong reason to use LibDBCompartment over LibDBIcon? This isn't meant to be a 'gotcha' type question, I'm genuinely curious if I should migrate to yours over using LibDBIcon's compartment feature, and whether or not you plan on supporting this library in the long-term or if you plan to merge your version into LibDBIcon itself.
Thanks!

I'd certainly agree with and recommend just using LibDBIcon if you have need of both a minimap button and compartment integration. Given that Classic doesn't have the addon compartment and that users have largely been trained over ~20 years to expect minimap buttons, I'd expect a lot of addons will settle on LibDBIcon as their all-in-one solution.
There's no plans to discontinue or stop supporting this project though. Its scope is small enough that it's effectively feature-complete at this point, and I'd expect that the cost of ongoing support is minimal enough that it won't be too onerous.

Great answer, thanks!


prevent breakage with the latest molinillo gem

This is a fast fix for breakage with the latest molinillo gem
https://github.com/berkshelf/solve/issues/67
https://github.com/berkshelf/solve/issues/64

If the other already merged PR works, can close this
https://github.com/berkshelf/solve/pull/66

I've pushed solve 3.1.1 that should resolve this.

Thanks for the help!


Profiler regression test failing on WH N300 due to bad eth set up

Seems to be timing out due to eth set up failure, different problem than #5489
As per @tt-rkim comment in slack thread this seems to be an eth related issue: https://tenstorrent.slack.com/archives/C05GRJC4J4A/p1708556701673689?thread_ts=1708556300.858779&cid=C05GRJC4J4A

Example CI run:
https://github.com/tenstorrent-metal/tt-metal/actions/runs/8008815220/job/21878586915
CC: @TT-billteng @mo-tenstorrent

Hopefully we fix via: https://github.com/tenstorrent-metal/metal-internal-workflows/issues/94

most likely related to #108

@aliuTT and @vtangTT can this be closed?

Can we try with these changes? https://github.com/tenstorrent-metal/tt-metal/pull/6708

@vtangTT ping


Add turbconv into SingleColumnModel and ConstantViscosity

Ekman layer runs:
pressure gradient force and coriolis force in base model; turbulent friction in subgrid scale constatnt viscosity model.
closes #229

bors try

bors r+


Internal blueprint and generic endpoints

User Story - Business Need
We currently use pipedream.com as a temporary server for modifying requests. This ticket will use our own service as a server to receive test http requests.

[ ] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)
As a Notify developer,
I want the API to include internal endpoints
So that we can integrate with Vetext, etc.
Additional Info and Resources

This ticket is being created along with infra 775 to point Dev to these new internal endpoints for vetext forwarding.
Acceptance Criteria

[ ] New paths use a new /internal blueprint and route
[ ] POST requests to /internal/generic_one and /internal/generic_two do not require any auth/validation
[ ] Each route logs the Flask request object and returns a status code of 200

QA Considerations

[ ] work with QA for testing and validation

These routes do not need OpenAPI or Postman collection entries.

@kalbfled this needs to work for both Dev and Perf. Does this ticket need to be updated to account for both environments?

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @k-macmillan @kalbfled @MackHalliday @mchlwellman

@mchlwellman See the PR for #1853 for an example of how this could work.

REVISION:
The route should accept GET and POST requests. This is a one word change to include in the list of methods accepted by the flask route.
Logging the entire request object ended up not being what we want. After discussion with @mchlwellman we decided on the following important request attributes:
    method
    root_path
    path
    query_string
    headers
    url_rule
    _cached_data
    _trace_id

Some will be pulled directly from the request object, some make sense to use the request class properties, such as content instead of _cached_data.

PR approved and merged. Will deploy and test after standup.

Deploying to Perf

QA PASSED

[ ] POST  https://sandbox-api.va.gov/internal/foobar with {"foo": "bar"} returns a 200 and the request body
200 received

{
    "foobarbaz": {
        "foo": "bar"
    }
}

[ ] POST  https://sandbox-api.va.gov/internal/foobar with {"foo": "bar", "foobar": "baz"} returns a 200 and the request body
200 received

{
    "foobarbaz": {
        "foo": "bar",
        "foobar": "baz"
    }
}

[ ] GET https://sandbox-api.va.gov/internal/foobarbaz returns 200 and the request
200 received

GET request received for endpoint /internal/foobarbaz?

[ ] GET https://sandbox-api.va.gov/internal/foobarbaz?name=alice&location=colorado&planet=earth returns 200 and the query parameters
200 received

GET request received for endpoint /internal/foobarbaz?name=alice&location=colorado&planet=earth

[ ] There is logging for both routes

will deploy to prod with the utils api PR, closing ticket


VSCode version 1.58.2 Not able to add new imports in Typescript rush monorepos

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.58.2
OS Version: macOS Big Sur 11.4
Extension ids: [coenraads.bracket-pair-colorizer, wmaurer.change-case, streetsidesoftware.code-spell-checker, dbaeumer.vscode-eslint, file-icons.file-icons, eamodio.gitlens, esbenp.prettier-vscode, netcorext.uuid-generator, vscodevim.vim]

Steps to Reproduce:

Create an interface Foo in a new location
Create a class that depends on the Foo interface and try to auto import the definition through the VSCode auto import feature (highlight the interface and use command + . to open the suggested actions)
The only option it will give the user if nothing has been imported from this location before is to extract to a type alias
If something had been imported from the same location however, it can add the import to the existing list of imports

What should happen:
VSCode should suggest creating a new import statement for that interface at the top of the class file
What actually happens:
using command + . yields only the option to Extract to type alias.

Please share a minimal example project that demonstrates this issue

in src/interfaces create
bar: string,
baz: number
} ```

in src/service create
```export class MyClass {
constructor(foo: Foo) {}
}```

and see that trying to import interface `Foo` will not work

https://github.com/sgoossens/vscode-issue


Rescue Zero Subscriber Errors on bulletin send

Bug: If a single user creates a topic, and then either fails to
complete the subscription process, or later unsubscribes, we will
continue to send emails to that topic. While that topic has zero
subscriptions, govdelivery will return an error message, which will
cause sidekiq to continually retry the bulletin send until success.
This means that the first user to then subscribe to that topic, will
soon receive all emails that we were trying to send during the time the
topic had zero subscribers.
This commit captures that specific error. It has to check both error
code and message, as that error code is used for other errors by
govdelivery.

I'm currently not doing anything when we receive this error message. I think it will just silently be marked as successful. Is this OK, and if not, any proposed solution?

@james I think that outcome is okay - in our existing GovDelivery go-between we also ignore these errors as they don't tell us anything interesting.

Agree with @bradleywright.  I'll merge.


fix: Remove filters via the indicator icons

This fixes #3794.
The details bug are in that issue but essentially if you have a Filament\Tables\Filters\Filter added to a form, apply the filter, and then remove it either by clicking the icon in the filter indicator or the 'x' icon in the filter indicator row, the table does not update until the page is refreshed.
This fixes the issue by first making sure that the Filter sets a default state of false when it's using a Checkbox or Toggle component. Then when removing the filter it sets the state of the filter to the default state of the field.
I've added tests showing this fixes the issue and all other tests are passing. However I'm not sure if this breaks other filter types or filters with custom forms. I don't think it should since it only sets the default state to false for Checkbox and Toggle components and it should be null for all other components.

A few other things. I updated the checkbox() and toggle() methods of the Filter class to set the default state too false. The formComponent() method is also updated to set the default state to null which is what it defaulted to before this fix. A potential problem is that if someone registers a Filter and calls default() and then formComponent() like so:
Filter::make('confirmed')
    ->default('some default value')
    ->formComponent(CustomFormComponent::class)

it will override the default value the user explicitly set themselves. I'm not sure of the best way to get around that.

Originally, we reset to the default state, but then there was this issue - https://github.com/filamentphp/filament/pull/3599#issuecomment-1225000368
I think we just need an extra check - if the state is true, set it to false. Else, set it to null.
Also we need to make sure that default() is only called on the form component if there is a non-null default value defined on the filter.

Can you double check if my changes still fix all your issues?

@danharrin that does fix the issue.


[HXML] import other hxml look like incorrect

For reproduce use latest build of plugin.

Tooltip:

ok, will be added

FIXED in 0.9.9

If your complain not referenced to HXML runner and project updater automation...

@eliasku That comment is confusing.  I parsed it as:  "If your complaint was not about the HXML runner or project updater automation..."  What is the rest of the statement that the elipsis (...) is alluding to?  (e.g. what is the 'then' clause?)

@EBatTiVo Apologize.
@fzzr- If you have no complains about HXML runner and HXML project updater then we should close the issue.

No response from @fzzr- and this was fixed for the last release.  Closing.  Please re-open if the issue recurs.


Added find-s to include tutor name search

list-s normal command shows the whole list

if a keyword is entered after, e.g. list-s a the filter will be implemented

the search only recognises full words, meaning list-s Alice will show schedules matched with the tutor name Alice, while list-s ali will return 0 schedules

Currently, I have yet to find a solution where I can filter the schedule by tutor index, as tutor index is not unique.

For 6 maybe you can try to look at edit/delete commands and how they get the tutor, there might be a way to filter from there?

Did you try running list-s with tutor search while the list of tutors is showing? I think the Ui might not update in this case?

Did you try running list-s with tutor search while the list of tutors is showing? I think the Ui might not update in this case?

I agree since handleListDisplay in MainWindow.java is checking for the MESSAGE_SUCCESS of a list-s's CommandResult and here a list-s that is filtering would be returning MESSAGE_SCHEDULES_LISTED_OVERVIEW instead.
Can maybe just add another condition in MainWindow::handleListDisplay or add a new attribute of some sort to CommandResult to indicate which list is being displayed?

For 3 and 5, I think its fine.
find command also has the same behaviour.
For 6:
If we do wish to filter by tutor index, are we keeping this filter by name too?
If so how would the syntax format look like and how to handle if they do both at once (filter by name and index together)?

For 3 and 5, I think its fine. This behaviour is similar to find command.
For 6: If we do wish to filter by tutor index, are we keeping the filtering by name too? If so how would the syntax format look like and how to handle if they do both at once (filter by name and index together)?

I would prolly change it to smth like list-s INDEX n/KEYWORDSEARCH using a param flag. The main issue is these params are optional, not mandatory. Which is something that the codebase do not have, so I would need more time to implement the logic in.

code is updated to use find-s instead of list-s keywords for searching schedule by tutor. This should resolve handleListDisplay issue as well.


Fixture Mocking

CakePHP Testing Suite provides fixtures when testing data manipulation instead of using a real database. And a good feature to be added to CakeCeption.

Not feasible as we can mock the base model and instantiate the model to be tested.


⚠ Bofrost.at has degraded performance

In 07ac37d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2953 ms

Resolved: Bofrost.at performance has improved in 5b70927.


Access Denied when making rpc call.

Describe the bug
When invoking the supabase.rpc.execute method, an APIError is thrown from postgrest.exceptions, which denotes a permission denial for a table.
postgrest.exceptions.APIError: {'code': '42501', 'details': None, 'hint': None, 'message': 'permission denied for table xxx'}

To Reproduce
Consider my_database_function as any PostgreSQL database function with RLS enabled.
Use the following code:
from supabase import create_client
from supabase.lib.client_options import ClientOptions

# Create client
supabase = create_client(SUPABASE_URL, SUPABASE_ANON_KEY)

# Login as test user
result = supabase.auth.sign_in_with_password({'email'<EMAIL_ADDRESS>'password': '123'})

supabase = create_client(SUPABASE_URL, SUPABASE_ANON_KEY, ClientOptions(headers={'Authorization': f'Bearer {result.session.access_token}'}))

result = supabase.rpc('my_database_function', {}).execute()

Additional context
Upon deeper inspection of the library's source code, it's apparent that the supabase.rpc method, which internally relies on postgrest-py, uses the same access_token as the SUPABASE_ANON_KEY. The intricacies in the option management logic within supabase-py could be causing this anomaly.
Might be related to https://github.com/supabase-community/supabase-py/issues/590

Why are you making a second client here? The first one should use the logged-in user's access token in the rpc() call

Yes, the first client is effective. However, in certain situations, we can only obtain the user token from the session. In those instances, I'll take the initiative to manually set up the Supabase client. So in the code snippet I provided, the first client only serves as a way to get the user access_token

I created such a function in my project and tested this and it works as expected. Not sure what's going wrong in your code? it would be best to create a reproducible example repository for someone to take a look at.

Closing this out as there has been no activity on this issue.


Pod not compatible with OS X

I would assume this is just an issue with the pod spec and that its code would be general enough
to work on both platforms.
I request for this to be made universal!

hello @jhoughjr,
in my fork i support other platform as well. Please report any further issues.
https://github.com/sammy-SC/Pluralize.swift


fix avx2 detection

reword commits to make it clearer
closes #2957

As noted in #2957 I am not yet convinced that checking only the cpu capability is sufficient (the original intention must have been to handle that via the cpu model, and only do an addtional check if the OS environment actually allows avx2 to be used)

IOW my spontaneous idea is to subsitute
-  if((ebx & (1<<7)) != 0)
+  if ( (ebx & (1<<5)) != 0 && (ebx & (1<<7)) != 0 )

I don't see how checking bit 7 is related to avx2. According to https://en.wikipedia.org/wiki/CPUID#EAX=7,_ECX=0:_Extended_Features, it's the bit "Supervisor Mode Execution Prevention". I really think it was a typo originally, there is no reason that I can see to check that bit. This PR doesn't change at all checking for the xmm and ymm registers (that's the call to xgetbv), which I agree is required.

Right, sorry, must have been a typo indeed. Got confused over my initial confusion it seems :/


WIP Adjust backup schedules to user timeszones.

Adjust select times to users selected timezone.
Added some help text to clarify use.

WIP
Cant get tests to pass. I cant even see the console.log statements since everything is pushing them off my console.

Look at which window is selected in Classic compared to Beta.  Getting those to match up is the most important part of the task.

Coverage increased (+0.1%) to 68.547% when pulling 708e465ed8d2d84d7c4972d3c74c7cc4da92e9a9 on rjstires:bug-2563 into 5448ec5b1d9f43a31f86181fc242fbc27f1790aa on linode:develop.

Coverage increased (+0.06%) to 68.512% when pulling 28ab5529f069b87e1e025c385b62bd3c68e08fce on rjstires:bug-2563 into 5448ec5b1d9f43a31f86181fc242fbc27f1790aa on linode:develop.


Enable cross-browser testing using Sauce Labs

Sauce Labs has generous plans for open source projects:
https://saucelabs.com/opensauce
Simply set SAUCE_USERNAME and SAUCE_ACCESS_KEY in the environment
and run "grunt test-saucelabs" following the instructions from the
grunt-saucelabs package:
https://github.com/axemclion/grunt-saucelabs

@iangilman any luck on setting up the Travis CLI?

@leesei No, I haven't tried since then... maybe the issue I was having has gone away in the meantime!
Do you have any experience in this area?

I haven't used Travis CLI, but I can try setting up a Docker image with it installed so you can use without contaminating your environment.

Did some search and there is a 54MB image:
https://hub.docker.com/r/antonmry/travis-docker/
# install script for most Linux distro
# !! need su !!
curl -sSL https://get.docker.com | sh
docker -v  # should >= 1.10

# then follow instructions in https://hub.docker.com/r/antonmry/travis-docker/

The first -v mounts a directory on your host machine (which should be this repo) to /repo in the container.
The second -v mount the setting folder on host machine so the settings can be persisted.

Thank you! I'll have to look into this when I get a chance. I'll probably have more questions then!

Hey, I have a little spare time and decided to dust this off and try it out. @iangilman do you still have those OpenSauce credentials around? I can give it a try if so.


Adds more options to spo page add/set --layoutType. Solves #2821

Adds more options to spo page add/set --layoutType. Solves #2821

Extends 'spo page add' and 'spo page set' commands with support for more layoutTypes in addition to Article and Home
Layout types added: SingleWebPartAppPage, RepostPage, HeaderlessSearchResults, Spaces, Topic.
Closes #2821

Linked Issue
Closes #2821

Awesome! We'll review it shortly 👏

Merged manually. Thank you! 👏


Include the link for SPDX Online Validation Tool somewhere in the README

I could not find a link or reference to the SPDX Online Validation tool anywhere in the tools README or in other relevant documentation. Is there a reason this link isn't easier to find? If not, I would be happy to open a PR to add a reference for finding it.

@rnjudge It should be easy to find - please open a PR - Thanks!


Auto-detect changes the user does to the list of GameViewSizes

The big plus at the bottom of the list allows users to add, edit or delete custom GameViewSizes.
But Camera-Plane does not detect such changes

Issue dropped because it's no longer relevant after the redesign


got core dumps when keepalive deconstruct

I got core dumps when keepalive deconstruct
#include "util/election.h"
#include <thread>
#include "gtest/gtest.h"
TEST(ELECTION, ELECTION1) {
  try{
  etcd::Client etcd("http://<IP_ADDRESS>:2379");
  auto lease_resp = etcd.leasegrant(3).get();
  auto _lease_id = lease_resp.value().lease();
  auto _keep_alive = std::unique_ptr<etcd::KeepAlive>(new etcd::KeepAlive(etcd, 3, _lease_id));
  etcd.set("ass", "ccc", _lease_id).get();
  }  catch ( std::exception& ex ) {
    std::cout << ex.what() << std::endl;
  } catch(...){
  }
}

the stack
(gdb) bt
#0  0x00000001003e0003 in ?? ()
#1  0x000000000042c804 in boost::asio::detail::service_registry::shutdown_services (this=0x7f38dfffeb58) at /usr/local/include/boost/asio/detail/impl/service_registry.ipp:44
#2  0x000000000042c903 in boost::asio::execution_context::shutdown (this=0x232c080) at /usr/local/include/boost/asio/impl/execution_context.ipp:41
#3  0x000000000042c8a7 in boost::asio::execution_context::~execution_context (this=0x232c080, __in_chrg=) at /usr/local/include/boost/asio/impl/execution_context.ipp:34
#4  0x0000000000439586 in boost::asio::io_context::~io_context (this=0x232c080, __in_chrg=) at /tmp/tmp.FKwox9TpTf/include/boost/asio/impl/io_context.ipp:56
#5  0x00007f38fe4e91b2 in etcd::KeepAlive::~KeepAlive (this=0x232c030, __in_chrg=) at external/etcd_cpp_apiv3/src/KeepAlive.cpp:101
#6  0x000000000042f1b4 in std::default_deleteetcd::KeepAlive::operator() (this=0x7ffffa55f6c0, __ptr=0x232c030) at /usr/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:76
#7  0x000000000042e383 in std::unique_ptr<etcd::KeepAlive, std::default_deleteetcd::KeepAlive >::~unique_ptr (this=0x7ffffa55f6c0, __in_chrg=) at /usr/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/../../../../include/c++/5.4.0/bits/unique_ptr.h:236
#8  0x0000000000429882 in ELECTION_ELECTION1_Test::TestBody (this=0x2306df0) at util/election_test.cpp:13
#9  0x00007f38fe6438d4 in testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void> (object=0x2306df0, method=&virtual testing::Test::TestBody(), location=0x7f38fe6535eb "the test body") at external/googletest/googletest/src/gtest.cc:2433
#10 0x00007f38fe63eed5 in testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void> (object=0x2306df0, method=&virtual testing::Test::TestBody(), location=0x7f38fe6535eb "the test body") at external/googletest/googletest/src/gtest.cc:2469
#11 0x00007f38fe628698 in testing::Test::Run (this=0x2306df0) at external/googletest/googletest/src/gtest.cc:2509
#12 0x00007f38fe629009 in testing::TestInfo::Run (this=0x230d9c0) at external/googletest/googletest/src/gtest.cc:2684
#13 0x00007f38fe6296f0 in testing::TestSuite::Run (this=0x230d350) at external/googletest/googletest/src/gtest.cc:2816
#14 0x00007f38fe6355bb in testing::internal::UnitTestImpl::RunAllTests (this=0x230d0f0) at external/googletest/googletest/src/gtest.cc:5338
#15 0x00007f38fe6447d1 in testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> (object=0x230d0f0, method=(bool (testing::internal::UnitTestImpl::)(testing::internal::UnitTestImpl * const)) 0x7f38fe635190 testing::internal::UnitTestImpl::RunAllTests(),
location=0x7f38fe653f88 "auxiliary test code (environments or event listeners)") at external/googletest/googletest/src/gtest.cc:2433
#16 0x00007f38fe63fee3 in testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> (object=0x230d0f0, method=(bool (testing::internal::UnitTestImpl::)(testing::internal::UnitTestImpl * const)) 0x7f38fe635190 testing::internal::UnitTestImpl::RunAllTests(),
location=0x7f38fe653f88 "auxiliary test code (environments or event listeners)") at external/googletest/googletest/src/gtest.cc:2469
#17 0x00007f38fe633ebe in testing::UnitTest::Run (this=0x7f38fe688440 testing::UnitTest::GetInstance()::instance) at external/googletest/googletest/src/gtest.cc:4925
#18 0x00007f38fee04b3e in RUN_ALL_TESTS () at external/googletest/googletest/include/gtest/gtest.h:2473
#19 0x00007f38fee04acd in main (argc=1, argv=0x7ffffa560078) at external/googletest/googlemock/src/gmock_main.cc:63
#20 0x00007f38fb68eb35 in __libc_start_main () from /lib64/libc.so.6
#21 0x0000000000429399 in _start ()

if i build binary it works fine.
but core dumps when unit test in gtest.

Thanks for reporting. Do you work on the master branch?

I have no information about #include "util/election.h", but I have manged to make it compilable:
#include <etcd/Client.hpp>
#include <etcd/KeepAlive.hpp>

#include <memory>
#include <thread>
#include "gtest/gtest.h"
TEST(ELECTION, ELECTION1) {
  try{
  etcd::Client etcd("http://<IP_ADDRESS>:2379");
  auto lease_resp = etcd.leasegrant(3).get();
  auto _lease_id = lease_resp.value().lease();
  auto _keep_alive = std::unique_ptr<etcd::KeepAlive>(new etcd::KeepAlive(etcd, 3, _lease_id));
  etcd.set("ass", "ccc", _lease_id).get();
  }  catch ( std::exception& ex ) {
    std::cout << ex.what() << std::endl;
  } catch(...){
  }
}

And build with the following command:
g++ test.cpp -std=c++14 -letcd-cpp-api -lgtest -lgtest_main

I cannot reproduce the failure on both Ubuntu 20.04 and MacOS, with the current master version. Could you please try to upgrade the latest version?

Close as lack of response and cannot be reproduced.


Laravel 6 support

Any plans to support Laravel 6?

+1

I made a pull request for laravel6 support #36

Hi, thank you. I'll review today and tag a new version.

I should be now, 0.19.x supports Laravel 6.x, latest is 0.19.1


Fix up Setting updating with optional update params

Previously, several of the options "default" to falsey/nil
when the parameter is not provided for update. This is the
result of a simplistic edit form that did not handle the
falsey/nil value in the actual HTML form, instead defaulting
to falsey/nil when the value was not supplied. This causes
errors after the Settings update pages were split into sections.
Closes #211

Coverage decreased (-0.1%) to 84.22% when pulling 2a58f61d7bd22a0040905d7055e3e2aa8cc1e613 on bug/211 into 1b1019739e4cbd23e2651500a18926992575a94b on master.


Missing coordinates for UTR introns

eg. https://www.pombase.org/gene/SPBC725.10

what do you mean, in Artemis?
or in chado?

what do you mean, in Artemis?
or in chado?

Just in the display.  We're not showing the UTR intron coordinates.  Like here:


Updated Readme to correct the order of input of the viewUpdater Example.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #46   +/-   ##
=======================================
  Coverage   57.17%   57.17%
=======================================
  Files          32       32
  Lines        1018     1018
=======================================
  Hits          582      582
  Misses        436      436

Continue to review full report at Codecov.

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


Support 2076-2-style structures in reassignIds

Closes #181
See issue ticket for more details

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (9955fbd) 89.56% compared to head (636badf) 89.63%.

Files
Patch %
Lines

src/utilities/id_assignment.cpp
95.55%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
+ Coverage   89.56%   89.63%   +0.06%
==========================================
  Files         125      125
  Lines        5781     5826      +45
==========================================
+ Hits         5178     5222      +44
- Misses        603      604       +1

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


Load Overlap2D levels

We want to be able to load Overlap2D levels supporting:

Textures
Particles
Physics
Invisible objects
Animations
...

All using our Ashley component/system infrastructure.
Custom Overlap2D Runtime.

This is blocked on #4.

We're missing physics and lights.

@AlbertoCejas can this be considered done?

yes ;)


[HUDI-2203] When the orderingVal is the same, take the latest one

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
OverwriteWithLatestAvroPayload.
When the orderingVal is the same, select the later data
Brief change log
org.apache.hudi.common.model.OverwriteWithLatestAvroPayload.class
org.apache.hudi.common.model.TestOverwriteWithLatestAvroPayload.class
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@wangxianghu  please help review

I actually have a similar PR https://github.com/apache/hudi/pull/3267 but have no time to finish it yet, you can take it.


ExportDeclaration require string cut off on Windows

export { GCOptimizedMap } from "./src/gc_optimized_map";
export { GCOptimizedSet } from "./src/gc_optimized_set";

produces
local exports = exports or {};
do
    local __TSTL_gc_optimized_map = require("rc.gc_optimized_map");
    local GCOptimizedMap = __TSTL_gc_optimized_map.GCOptimizedMap;
    exports.GCOptimizedMap = GCOptimizedMap;
end
do
    local __TSTL_gc_optimized_set = require("rc.gc_optimized_set");
    local GCOptimizedSet = __TSTL_gc_optimized_set.GCOptimizedSet;
    exports.GCOptimizedSet = GCOptimizedSet;
end
return exports;

This issue was caused by rootDir being ./ instead of .. This still shouldn't happen, pushing through a PR.


 Or is it just the problem faced when using new domain?

Hmm i checked and I'm getting good results while using your repl site

Also which dns are you using?

I use Freenom Domains, Dns correct and recommended

the error performs with the following steps: Enter the site> The Error Happens
Expectation (Which obviously should happen): Enter the site> Get Login> Place bots
I use Repl.it
My domain is registered by freenom, my dns is what repl.it provides the steps

I've been using repl.it for a long time. Never occurred

Yea it's happening to my website too. https://pingus.tk

I found the solution.

First go to your website's CloudFlare dashboard.
Then go to SSL/TLS setting.
Currently the setting is Flexible.
Change it to Full(strict).
Done. Now your website will be OK!!

@ZonaBRDev

@xHyroM you can close the issue.

Okay

repl.it received an update and automatically offers an ssl certificate so you have to turn off proxied status on cloudflare.


"Burthorpe" misspelled on Discord Rich presence in Burthrope Games Room

"Burthorpe" is misspelled as "Burthope" in Discord's rich precense
To Reproduce
Steps to reproduce the behavior:

Go to Burthorpe Games Room in game
Click on your discord name if you have it displayed
Burthorpe is spelled wrong

Screenshots
https://i.imgur.com/M8cIrDh.png
Environment (please complete the following information):

OS: [e.g. Windows 10]
RuneLite version: 1.4.20
Launcher version: 1.6.0

I don't know if you got this but it was misspelled when in the Burthorpe Games Room as well, same misspelling

I’ll double check this when I get home

Probably should've mentioned that in original issue @ScuzzyBeta :d I will reopen the issue and edit the description then

@deathbeam Sorry about that, I didn't even notice it until it was already closed. Thanks again.


DO NOT MERGE: Introduce helm depenency

Summary
This introduces helm a dependency
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[x] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 46.718% (+0.01%) from 46.704%
when pulling 57ba5ea49e88cf43bdb72881bd09b91ce0002548 on helm-introduce
into a0d83dcf78ce6be1372c77e30cbaa6e63091f639 on main.


Build to deploy directory independent of IDEA

Now the build to deploy directory is done via IDEA build artifact which means we have a dependency on the IDE.
Also what's not good : when we add a new library in pom.xml we need not to forget to update the build artifact.
Better to move to Maven-based approach (with some hotkey to keep the existing behaviour). This will make the build cross-IDE-compatible and easy to support.

Most likely it was already done. Need double checking

The deploy has been Maven based for a long time ago. Fixed in https://github.com/lrozenblyum/chess/commit/aa73612d4e368252bdace3febd0c4bad29266480#diff-600376dffeb79835ede4a0b285078036

Migrating back to 0.1 milestone where it was in fact done


Add a tt-muncher for constructing Value

This puts the object! and array! macros from json-rust and the json! compiler plugin from json_macros to shame.

WIP because I am porting the test suite to use this. Win-win - we get test coverage for this macro and our tests become somewhat readable.

Here is the rustdoc in rendered form:

Construct a serde_json::Value from a JSON literal.
let value = json!({
    "code": 200,
    "success": true,
    "payload": {
        "features": [
            "serde",
            "json"
        ]
    }
});

Any variable or expression that implements Serde's Serialize trait can be interpolated into the JSON literal just by referring to it.
let code = 200;
let features = vec!["serde", "json"];

let value = json!({
   "code": code,
   "success": code == 200,
   "payload": {
       features[0]: features[1]
   }
});

Trailing commas are allowed inside both arrays and objects.
let value = json!([
    "notice",
    "the",
    "trailing",
    "comma -->",
]);

:umbrella: The latest upstream changes (presumably #208) made this pull request unmergeable. Please resolve the merge conflicts.

Hi @dtolnay, this is very nice.
As a micro-optimization, do you think there is any way to use Vec::with_capacity(3) for something like json!([1,2,3]) instead of Vec::new()?
I looked how vec!  does it, IIUC it "cheats" by using the builtin syntax.

Good idea @bluetech. I am busy until this afternoon but when I get back I will make it generate Value::Array(vec![...]) directly so we can benefit from any optimizations it provides.

Ah that's actually usable! What's the plan to release this?

This will be in 0.9.0. I am working as hard as I can to have it out by the end of Tuesday.

@bluetech thanks again, I made that change in #209. Now we will get Value::Array(vec![...]).

@dtolnay this is some admirable display of macro mastery :)


Chore: fix feefeed container healthcheck

Check if feefeed fetched fees instead of checking if it's up

Perhaps it would also be valuable that we add docker-compose logs -f  where feefeed is running. This would give us more visibility.
- run:
          name: Log daemon services
          command: |
            cd priv/cabbage
            docker-compose logs -f

right after
- run:
          name: Start daemon services
          command: |
            cd priv/cabbage
            make start_daemon_services-2 || (START_RESULT=$?; docker-compose logs; exit $START_RESULT;)


Normalizing associative-commutative operators

We will need to flatten (internally) certain associative arithmetic and logical operators, so that simplifications such as
(X + 1) + 2 = X + (1 + 2) = X + 3

happen automatically.
Candidate operators are:

+Int
*Int
andBool
orBool

We should also take make the integer operations aware of -Int and /Int.
See also: https://runtimeverification.slack.com/archives/CC360GUTG/p1564513723167600
CC: @yzhang90

FWIW, z3 has a simplify command which, as far as I can tell, does something similar to this.

We should also do something similar for associative operators like concatList.

This is done directly in K with simplification rules.


ES-1312: fix handling of reaching WAL archive capacity limit

Scope & Purpose
Fixed issue https://arangodb.atlassian.net/browse/ES-1312
Fix handling of reaching the WAL archive capacity limit.

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[x] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[x] Backport for 3.10: https://github.com/arangodb/arangodb/pull/17353
[x] Backport for 3.9: https://github.com/arangodb/arangodb/pull/17354
[x] Backport for 3.8: https://github.com/arangodb/arangodb/pull/17355

Related Information

[ ] Docs PR:
[ ] Enterprise PR:
[x] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/ES-1312
[ ] Design document:

I am not really a fan of duplicating the test code. :|

I will add a follow-up PR for deduplicating the test code.


avoid DATA RACE if reusing query builder

If calling Query or QueryContext on the same query builder in parallel there is a race condition in using.
This PR removes that race condition.
Example: query is built once, then reused (concurrently):
func getDelete(session gocqlx.Session) func(ctx context.Context, id string) error {
	qry := qb.Delete("tbl").Where(qb.Eq("id"))

	return func(ctx context.Context, id string) error {
		return qry.QueryContext(ctx, session).Bind(id).ExecRelease()
	}
}

@mmatczuk would it be possible to get a new version tag in the near future? That way we can pull this in without having to use a commit hash in all of our services or having to use a replace and create a version tag on our clone. That would be super helpful. 🙏

Here it is https://github.com/scylladb/gocqlx/releases/tag/v2.7.0


update outbound http docs when Spin 1.5 ships to include new self feature

Update https://developer.fermyon.com/spin/http-outbound to reflect new self feature from https://github.com/fermyon/spin/pull/1710

Fixed via #765


Ensure that $router, $route, $plugin, and cache.map are marked as non-reactive

@yonasberhe23 yes, that should also be case here


itvy.txt

@Fran12F1 First stage of verification passed, official domain is vyaqui.tecnm.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://vyaqui.tecnm.mx/index.php/directorio

@Fran12F1 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please notice that the domain was already in the trusted list.


num_samples=0 error

from e
hydra.errors.HydraException: Error calling 'torch.utils.data.DataLoader' : num_samples should be a positive integer value, but got num_samples=0
In my dataset class I am reading things inside a folder where I ran the search command is this the cause ?

i print the length of my image path list when dataset.py was instantiated i can see the length is 0
so how can I point working directory to the place where my images are located ?

fixed it just had to pass a parameter in root from yaml and then use it

Hi @ashish244co, I've encountered the same issue, but am unsure exactly how you fixed it, can you elaborate please? Thanks


SQL Notebook - cannot select a connection created after notebook has been opened

Issue Type: Bug
Open a notebook with no active connections in object explorer.
Create a new connection in object explorer
Click on "select connection" error message is as follows:
Changing context failed: No valid connection
also noticed.
When I mouse-over the error in bottom right corner the pointer turns to a hand. Clicking highlights the message box but does not perfom an action. All you can do is close the window.
Azure Data Studio version: azuredatastudio 1.6.0 (eb35dae1d1238e03f1cab3bd45e151f381b22aef, 2019-04-17T07:11:15.944Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
63.92GB (19.22GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

ssmskeymap
kev
1.1.0

sql-dw
Mic
0.0.1

sql-vnext
Mic
0.12.1

Gene please investigate - ping JRJ if needed to get exact repro steps

Contacted JRJ for repro, but unable to repro in the latest ADS bits.


Stuck loading workspace with 2024-09-19 version

Expected
I expect to be able to start the workspace in the same way as in the previous version, 2024-07-02.
Actual
I am able to start the workspace in 2024-07-02, but doing the same procedure in 2024-09-19 version does not seem to complete loading the workspace, just hangs/is stuck.
Steps to reproduce
Steps:

Put "workspace.dsl" file in an empty folder
Run old version 2024-07-02 and go to localhost:8080 to verify that workspace was loaded successfully

docker run -it --rm -p 8080:8080 -v <full-path-to-folder>:/usr/local/structurizr structurizr/lite:2024.07.02

Delete ".structurizr" folder and the "workspace.json" file
Run new version 2024-09-19 and go to localhost:8080. Notice how the workspace is stuck loading.

docker run -it --rm -p 8080:8080 -v <full-path-to-folder>:/usr/local/structurizr structurizr/lite:2024.09.19

Workspace.dsl file used:
workspace {
model {
    u = person "User"
    system = softwareSystem "Software System" {
        wa = container "Web Application" {
            tags "Application"
        }
        db = container "Database Schema" {
            tags "Database"
        }
    }

    u -> wa "Uses"
    wa -> db "Reads from"
}

views {
    systemContext system "Diagram1" {
        include *
        autolayout lr
    }

    container system "Diagram2" {
        include "->wa->"
        autolayout lr
    }

    styles {
        element "Element" {
            color white
        }
        element "Software System" {
            background #1DACD6
        }
        element "Person" {
            background #E5D3D3
            shape person
        }
        element "Database" {
            shape cylinder
            background #A39193
        }
        element "Application" {
            background #7C0A02
        }
    }
}

}
Version/build information
Using Structurizr docker version as listed here, 2024.09.19 and 2024.07.02.
https://hub.docker.com/r/structurizr/lite/tags
It's the first time I'm trying the newer version from today.
Note im using Docker API "dockerd (moby)" from Rancher Desktop version 1.14.1 which has a docker cli.
Severity
Major
Priority
I have no budget and there's no rush, please fix this for free
More information
Was following this guide:
https://docs.structurizr.com/dsl/tutorial
See video below, reproducing the issue:
https://github.com/user-attachments/assets/a26566ad-ef09-495d-9c9d-361e3fcdcc9c

I see these errors. Issue with using dark mode?
structurizr.ui.initDarkMode is not a function
structurizr.ui.isDarkMode is not a function
GET http://localhost:8080/workspace/1/images/Diagram1-thumbnail.png 404 (Not Found)
GET http://localhost:8080/workspace/1/images/Diagram2-thumbnail.png 404 (Not Found)

Try doing a hard refresh in your web browser - something is likely being cached.

Tried from incognito window, seems to load now, so must have been something cached yes.


npm latest version 1.0.17 is breaking on angular rc3 app

I upgraded angular app to rc3. But, npm latest version 1.0.17 is still rc1.
Using rc1 ng2-bootstrap in rc3 is breaking the app. Any inputs/hack on 'getting the latest'?

We know this, there are open pull requests. See #631


Annoying Babel warnings from webpack build

We are currently getting spammed with warnings during the webpack build of applications. The messages read as follows:

Though the "loose" option was set to "false" in your @babel/preset-env config, it will not be used for @babel/plugin-proposal-private-methods since the "loose" mode option was set to "true" for @babel/plugin-proposal-private-property-in-object.
The "loose" option must be the same for @babel/plugin-proposal-class-properties, @babel/plugin-proposal-private-methods and @babel/plugin-proposal-private-property-in-object (when they are enabled): you can silence this warning by explicitly adding
["@babel/plugin-proposal-private-methods", { "loose": true }]
to the "plugins" section of your Babel config.

Fixed?


Release 0.5.0 of spark-xml

Need a new release. There are many good fixes were done.

@srowen, do you have some more changes to push? If so, I can wait for another week. If you think it's mostly done, I will make a release tomorrow.

I'd like to run down one more potential issue -- following up on https://github.com/databricks/spark-xml/pull/358 In the course of that I noticed some loops have a condition like shouldStop = shouldStop && parser.hasNext. Because the flag is always false when the loop starts, this is basically doing nothing. I 'fixed' it but it causes a few tests to fail that relate to corrupt records. So maybe I should investigate for a day or two to see if we need to fix one more thing there.

Yea .  Makes sense. Let me take a look tomorrow.

I pushed

Released
made a tag v0.5.0
made a branch branch-0.5

I'm going to make a release note by the end of today (https://github.com/databricks/spark-xml/releases/tag/v0.5.0)

Release is done! and note is also done! https://github.com/databricks/spark-xml/releases/tag/v0.5.0


Add zhoujin3150's file

Created a branch, created a file and made a commit

Followed the instruction


Bugfix: Allow to compile with C++17

Could you please merge this, this enables string_view only when compiling with c++17 support.

We went through a long period of waiting and discussion about this and the decision seems to have to been to make C++17 required.
I ended up forking it and making a couple of changes: https://github.com/asmaloney/crossguid


"--ignore" and "IgnoreGroup" ignored

pacaur v4.6.4
I have several packages installed from a PKGBUILD modified by me and I have added these packages to a group (modified) to avoid upgrade from AUR/PACMAN with the parameter IgnoreGroup = modified in /etc/pacman.conf but pacaur doesn't respect it.
One of those packages (whatsie) has an update in AUR database that it has a annoying bug, because this I wan't upgrade it but pacaur does:
$ pacaur -u
:: Starting AUR upgrade...
:: resolving dependencies...
:: looking for inter-conflicts...
:: gtk-theme-arc-git latest revision -- fetching

AUR Packages  (5): filebot-4.7-1  google-chrome-50.0.2661.94-1  gtk-theme-arc-git-latest  ultra-flat-icons-1.3.4-6  whatsie-2.0.14-17

:: Proceed with installation? [Y/n]

That package is in modified group:
$ pacman -Qs whatsie
local/whatsie 2.0.13-1 (modified)
    A simple & beautiful desktop client for WhatsApp Web.

--ignore modified option doesn't work:
$ pacaur -u --ignore modified
:: Starting AUR upgrade...
:: resolving dependencies...
:: looking for inter-conflicts...
:: gtk-theme-arc-git latest revision -- fetching

AUR Packages  (5): filebot-4.7-1  google-chrome-50.0.2661.94-1  gtk-theme-arc-git-latest  ultra-flat-icons-1.3.4-6  whatsie-2.0.14-17

:: Proceed with installation? [Y/n]

This is out of scope, pacaur has never been intended to manage packages modified by the user. See also #388.

pacaur has never been intended to manage packages modified by the user.

That's what I want.

That's because AUR packages don't have groups.

The groups in AUR packages is unnecessary. But installed packages have groups:
$ pacman -Qs whatsie
local/whatsie 2.0.13-1 (modified)
    A simple & beautiful desktop client for WhatsApp Web.

That's what I want.

The you are out of luck. Either you'll need to fork, or you'll need to sponsor the feature. In the man page, you'll find a way to send me money. Loooot of money.


[ISSUE #4584] Add new persist method to update consume offset to remote server.

Make sure set the target branch to develop
What is the purpose of the change
close #4584
Brief changelog
Add new persist method to update consume offset to remote server.

Codecov Report

Merging #4692 (3a01a7a) into develop (3602b45) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #4692      +/-   ##
=============================================
- Coverage      44.59%   44.45%   -0.15%
+ Complexity      7522     7488      -34
=============================================
  Files            977      977
  Lines          67582    67584       +2
  Branches        8927     8927
=============================================
- Hits           30138    30044      -94
- Misses         33726    33807      +81
- Partials        3718     3733      +15

Impacted Files
Coverage Δ

...ocketmq/client/consumer/DefaultMQPullConsumer.java
32.75% <0.00%> (-0.58%)
:arrow_down:

...rg/apache/rocketmq/common/stats/StatsSnapshot.java
84.61% <0.00%> (-15.39%)
:arrow_down:

.../apache/rocketmq/common/stats/MomentStatsItem.java
38.09% <0.00%> (-9.53%)
:arrow_down:

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...q/namesrv/routeinfo/BrokerHousekeepingService.java
72.72% <0.00%> (-9.10%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-8.70%)
:arrow_down:

...a/org/apache/rocketmq/logging/inner/SysLogger.java
28.57% <0.00%> (-5.72%)
:arrow_down:

...va/org/apache/rocketmq/common/stats/StatsItem.java
50.00% <0.00%> (-5.00%)
:arrow_down:

...java/org/apache/rocketmq/logging/inner/Logger.java
51.67% <0.00%> (-4.31%)
:arrow_down:

...lient/impl/consumer/DefaultMQPushConsumerImpl.java
31.64% <0.00%> (-3.77%)
:arrow_down:

... and 22 more

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

Coverage decreased (-0.1%) to 48.695% when pulling 3a01a7a13b6299dbe44ec51e084439f3176a9152 on ShannonDing:pull_consumer_offset into 3602b450cfef72324404c7805be60bcd231ac24f on apache:develop.


Enhancement Request: Support for Skill VRP with Precedence Constraints

Body:
Hello,
Firstly, I would like to express my admiration for the project. It has been an invaluable resource in the field of vehicle routing problems, and the community has greatly benefited from the open-source nature of the work.
Moving forward, I believe there is an opportunity to further enhance the project's capabilities by introducing support for a Skill Vehicle Routing Problem (Skill-VRP) model that differentiates vehicles not only by skill levels but also by skill types. This would allow for a more nuanced approach to fleet management in scenarios where a variety of service types are provided.
Additionally, incorporating precedence constraints into the model would be a significant advancement. For instance, in cases where a fleet of vehicles offers services A, B, and C, and there is a requirement for customers to receive these services in a sequential order before a vehicle can return, the ability to plan routes efficiently with this in mind would be highly beneficial. The objective would be to minimize the total travel cost while respecting the service sequence.
Here are the key requirements for this enhancement:
1.Recognition of different skill types for vehicles.
2.Implementation of precedence constraints for service order.
3.Optimization algorithms to ensure minimal travel costs.
The inclusion of these features would greatly expand the project's applicability to a broader range of logistics and service provision challenges.
Thank you for considering this enhancement request. I am eager to see how it could be implemented .

Closing this issue because I think it is answered. Feel free to continue commenting on this issue if you still have related questions.


Prepare landing page for ICES website

Template structure:

@neil-ices-dk here is the issue we can use to build up the materials for the landing page of the online advice

first thing to settle, the landing page URL, will it simply be .../online-advice.aspx

@neil-ices-dk we have a winner .../advicexplorer.aspx

as it appears in the data report, was the Caps A negotiable ;)

Hi Luca - as emailed, Mark wants this version

@lucalamoni what location should the page be on ices.dk?

Following what was said during the meeting, the group suggested having a link to the page on the right menu of the latest advice page

and on the general advice page

thanks Luca, these are just links though - so not really defining the page. I will go with a page under
https://www.ices.dk/advice/Pages/xxxx.aspx which can then be linked from wherever :)

Hi Luca - as emailed, Mark wants this version

I like this, did communications have a look at it as well? I need to test if I can embed it in the navbar, in terms of colors. During the meeting, I also showed the new color palette of the app, that we developed with the graphic designer.
It will probably look more like this

or this

thanks Luca, these are just links though - so not really defining the page. I will go with a page under https://www.ices.dk/advice/Pages/xxxx.aspx which can then be linked from wherever :)

yes, sorry for the confusion. Sounds good

Hi Luca - as emailed, Mark wants this version

I like this, did communications have a look at it as well? I need to test if I can embed it in the navbar, in terms of colors. During the meeting, I also showed the new color palette of the app, that we developed with the graphic designer. It will probably look more like this  or this

Comms haven't had a look at this yet, will you take this forward with them?

Hi @neil-ices-dk, here are the two final logos that Maria worked on if you need them on the landing page. The color one will be in the adviceXplorer on top of the map

I will give you the text for the page asap.

the white space below the color logo is the negative version of logo, with a white background is invisible :)

adviceXplorer_landing_page_content_TM_MDC_LL.docx
Hi @neil-ices-dk, here is the edited text for the landing page using Terhi and Marc's suggestions. The metadata record is almost ready for publishing and Periklis is just waiting for me to deploy the app at the final URL. I'm also working on the disclaimer.

https://github.com/ices-tools-prod/disclaimers/blob/master/Disclaimer_adviceXplorer.txt

@lucalamoni and so it begins
https://www.ices.dk/advice/Pages/adviceXplorer.aspx
We're not able to place the logo graphic in the title (as you have it in the Word file), it can only go in the body or right hand side, so i have for the moment put it in the image box.
Unlike the data section, advice don't follow the same left hand navigation on each page; this means there is potential to create a navigation that equates to the top menu tabs in the shinyapp; have a think about that, otherwise we can leave it blank and change the layout to make the page wider

Hi @neil-ices-dk, looking good!
I'll have a think about the menu. In the end, you decided to avoid the app iframe in the landing page?

https://ices-taf.shinyapps.io/advicexplorer/
I deployed the app on the new URL

Periklis will publish the metadata record soon and I finalized the Disclaimer, which now is zipped together with csv files when the user downloads the data.

Hi @neil-ices-dk, looking good! I'll have a think about the menu. In the end, you decided to avoid the app iframe in the landing page?

it's just for this ICES app we need a proper landing page with context and links regardless of whether we have a page with the embedded app. I think that because the app is ICES, and is branded as such, it is not necessary to have the embedded app.

so all we need are 'call to action' buttons on the right hand side

I'm not 100% sure what needs to go in the "call to actions" buttons but on top of my head I would put:

link to the adviceXplorer
link to the github repo
link to the metadata record
link to the ASD

link to the github repo

link to the metadata record

do you have these links @lucalamoni

https://github.com/ices-tools-dev/online-advice.git
@PeriklisICES could you add here the link to the metadata record?

Github link added;
Celine is asking where this page will be linked from, so please check with Lotte where on the advice site it will sit (linkwise)

The link to the metadata is this:
https://gis.ices.dk/geonetwork/srv/eng/catalog.search#/metadata/8046f4e2-f62f-4c0b-b970-70d29602e710

thanks @PeriklisICES now implemented.
I would say the page is ready

It looks great :)
Thank you @neil-ices-dk and thank you @PeriklisICES !

@neil-ices-dk do we need to update anything in google analytics?

The standard SharePoint landing page will be fine, but we will need to do something for the shinyapp url as that has changed.

the google tag

is already in the app, so I guess is just a matter of finding the page on google analytics

have made a link from the 'assessment tools' in the data area (below). assume you will make the links from Advice with Celine/Michala?

thank you, yes, Celine should take care of the links on the advice page

suggest this ticket can be closed then 🥇


Error Certbot add DNS Challenge witch OVH

Hello,
When i wan't create a new certificat to DNS Challenge with OVH , i have this error :
CommandError: usage:
  certbot [SUBCOMMAND] [options] [-d DOMAIN] [-d DOMAIN] ...

Certbot can obtain and install HTTPS/TLS/SSL certificates.  By default,
it will attempt to use a webserver both for obtaining and installing the
certificate.
certbot: error: unrecognized arguments: --dns-ovh-credentials /etc/letsencrypt/credentials/credentials-8

    at /app/lib/utils.js:16:13
    at ChildProcess.exithandler (node:child_process:410:5)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

Nginx Proxy Manager Version
2.12.1

Operating System
Debian12/Proxmox, 8.3

I can't reproduce this however it get the following error for OVH's EU Endpoint:
`CommandError: Saving debug log to /tmp/letsencrypt-log/letsencrypt.log
Unexpected error determining zone identifier for test.gassner-it.at: HTTPSConnectionPool(host='eu.api.ovh.com', port=443): Max retries exceeded with url: /1.0/auth/time (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self-signed certificate in certificate chain (_ssl.c:992)')))
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /tmp/letsencrypt-log/letsencrypt.log or re-run Certbot with -v for more details.
at /app/lib/utils.js:16:13
at ChildProcess.exithandler (node:child_process:430:5)
at ChildProcess.emit (node:events:518:28)
at maybeClose (node:internal/child_process:1104:16)
at ChildProcess._handle.onexit (node:internal/child_process:304:5)`

Can't reproduce. For me this works fine

I have the same issue — just installed npm in LXC containter in Proxmox and had the same error message. Tried installing certbot plugins by running app/scripts/install-certbot-plugins, but installing ovh plugin fails with this error:
[Certbot  ] › ▶  start     Installing ovh...
[Global   ] › ⬤  debug     CMD: . /opt/certbot/bin/activate && pip install --no-cache-dir acme==$(certbot --version | grep -Eo '[0-9](\.[0-9]+)+') certbot-dns-ovh==$(certbot --version | grep -Eo '[0-9](\.[0-9]+)+')  && deactivate
[Certbot  ] › ✖  error     Traceback (most recent call last):
  File "/opt/certbot/bin/certbot", line 5, in <module>
    from certbot.main import main
  File "/opt/certbot/lib/python3.11/site-packages/certbot/main.py", line 2, in <module>
    from certbot._internal import main as internal_main
  File "/opt/certbot/lib/python3.11/site-packages/certbot/_internal/main.py", line 26, in <module>
    from certbot._internal import account
  File "/opt/certbot/lib/python3.11/site-packages/certbot/_internal/account.py", line 16, in <module>
    from acme.client import ClientBase  # pylint: disable=unused-import
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ImportError: cannot import name 'ClientBase' from 'acme.client' (/opt/certbot/lib/python3.11/site-packages/acme/client.py)
Traceback (most recent call last):
  File "/opt/certbot/bin/certbot", line 5, in <module>
    from certbot.main import main
  File "/opt/certbot/lib/python3.11/site-packages/certbot/main.py", line 2, in <module>
    from certbot._internal import main as internal_main
  File "/opt/certbot/lib/python3.11/site-packages/certbot/_internal/main.py", line 26, in <module>
    from certbot._internal import account
  File "/opt/certbot/lib/python3.11/site-packages/certbot/_internal/account.py", line 16, in <module>
    from acme.client import ClientBase  # pylint: disable=unused-import
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ImportError: cannot import name 'ClientBase' from 'acme.client' (/opt/certbot/lib/python3.11/site-packages/acme/client.py)
ERROR: Could not find a version that satisfies the requirement acme== (from versions: 0.0.0.dev20151006, 0.0.0.dev20151008, 0.0.0.dev20151017, 0.0.0.dev20151020, 0.0.0.dev20151021, 0.0.0.dev20151024, 0.0.0.dev20151030, 0.0.0.dev20151104, 0.0.0.dev20151107, 0.0.0.dev20151108, 0.0.0.dev20151114, 0.0.0.dev20151123, 0.0.0.dev20151201, 0.1.0, 0.1.1, 0.2.0, 0.3.0, 0.4.0, 0.4.1, 0.4.2, 0.5.0, 0.6.0, 0.7.0, 0.8.0, 0.8.1, 0.9.0, 0.9.1, 0.9.2, 0.9.3, 0.10.0, 0.10.1, 0.10.2, 0.11.0, 0.11.1, 0.12.0, 0.13.0, 0.14.0, 0.14.1, 0.14.2, 0.15.0, 0.16.0, 0.17.0, 0.18.0, 0.18.1, 0.18.2, 0.19.0, 0.20.0, 0.21.0, 0.21.1, 0.22.0, 0.22.1, 0.22.2, 0.23.0, 0.24.0, 0.25.0, 0.25.1, 0.26.0, 0.26.1, 0.27.0, 0.27.1, 0.28.0, 0.29.0, 0.29.1, 0.30.0, 0.30.1, 0.30.2, 0.31.0, 0.32.0, 0.33.0, 0.33.1, 0.34.0, 0.34.1, 0.34.2, 0.35.0, 0.35.1, 0.36.0, 0.37.0, 0.37.1, 0.37.2, 0.38.0, 0.39.0, 0.40.0, 0.40.1, 1.0.0, 1.1.0, 1.2.0, 1.3.0, 1.4.0, 1.5.0, 1.6.0, 1.7.0, 1.8.0, 1.9.0, 1.10.0, 1.10.1, 1.11.0, 1.12.0, 1.13.0, 1.14.0, 1.15.0, 1.16.0, 1.17.0, 1.18.0, 1.19.0, 1.20.0, 1.21.0, 1.22.0, 1.23.0, 1.24.0, 1.25.0, 1.26.0, 1.27.0, 1.28.0, 1.29.0, 1.30.0, 1.31.0, 1.32.0, 2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.4.0, 2.5.0, 2.6.0, 2.7.0, 2.7.1, 2.7.2, 2.7.3, 2.7.4, 2.8.0, 2.9.0, 2.10.0, 2.11.0, 3.0.0, 3.0.1)
ERROR: No matching distribution found for acme==

Got that too.
Seems like certbot-dns-ovh is not up ta date : .
Maybe that's why certbot-dns-ovh requires an acme-2 while others require an acme-3 version.
Switching to latest certbot-dns-ovh version could solve this issue.
Enjoy!

Trying to force certbot-dns-ovh version leads to this :
 . /opt/certbot/bin/activate && pip install --no-cache-dir  certbot-dns-ovh~=3.0.1  && deactivate Collecting certbot-dns-ovh~=3.0.1   Downloading certbot_dns_ovh-3.0.1-py3-none-any.whl (10 kB) Collecting dns-lexicon>=3.15.1   Downloading dns_lexicon-3.18.0-py3-none-any.whl (248 kB)      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 248.5/248.5 kB 8.5 MB/s eta 0:00:00 Requirement already satisfied: setuptools>=41.6.0 in /opt/certbot/lib/python3.11/site-packages (from certbot-dns-ovh~=3.0.1) (74.0.0) Requirement already satisfied: acme>=3.0.1 in /opt/certbot/lib/python3.11/site-packages (from certbot-dns-ovh~=3.0.1) (3.0.1) Collecting certbot>=3.0.1   Downloading certbot-3.0.1-py3-none-any.whl (407 kB)      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 407.5/407.5 kB 35.6 MB/s eta 0:00:00 Requirement already satisfied: cryptography>=3.2.1 in /opt/certbot/lib/python3.11/site-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (3.4.8) Requirement already satisfied: josepy<2,>=1.13.0 in /opt/certbot/lib/python3.11/site-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (1.14.0) Requirement already satisfied: PyOpenSSL!=23.1.0,>=17.5.0 in /opt/certbot/lib/python3.11/site-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (21.0.0) Requirement already satisfied: pyrfc3339 in /usr/lib/python3/dist-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (1.1) Requirement already satisfied: pytz>=2019.3 in /opt/certbot/lib/python3.11/site-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (2024.1) Requirement already satisfied: requests>=2.20.0 in /opt/certbot/lib/python3.11/site-packages (from acme>=3.0.1->certbot-dns-ovh~=3.0.1) (2.32.3) Requirement already satisfied: ConfigArgParse>=1.5.3 in /opt/certbot/lib/python3.11/site-packages (from certbot>=3.0.1->certbot-dns-ovh~=3.0.1) (1.7) Requirement already satisfied: configobj>=5.0.6 in /usr/lib/python3/dist-packages (from certbot>=3.0.1->certbot-dns-ovh~=3.0.1) (5.0.8) Requirement already satisfied: distro>=1.0.1 in /opt/certbot/lib/python3.11/site-packages (from certbot>=3.0.1->certbot-dns-ovh~=3.0.1) (1.9.0) Requirement already satisfied: parsedatetime>=2.4 in /usr/lib/python3/dist-packages (from certbot>=3.0.1->certbot-dns-ovh~=3.0.1) (2.6) Requirement already satisfied: beautifulsoup4>=4 in /usr/lib/python3/dist-packages (from dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (4.11.2) Requirement already satisfied: dnspython>=2 in /opt/certbot/lib/python3.11/site-packages (from dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (2.7.0) Requirement already satisfied: pyotp>=2 in /opt/certbot/lib/python3.11/site-packages (from dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (2.9.0) Requirement already satisfied: pyyaml>=3 in /opt/certbot/lib/python3.11/site-packages (from dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (5.3.1) Requirement already satisfied: tldextract>=2 in /opt/certbot/lib/python3.11/site-packages (from dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (3.5.0) Requirement already satisfied: six in /usr/lib/python3/dist-packages (from configobj>=5.0.6->certbot>=3.0.1->certbot-dns-ovh~=3.0.1) (1.16.0) Requirement already satisfied: cffi>=1.12 in /opt/certbot/lib/python3.11/site-packages (from cryptography>=3.2.1->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (1.17.0) Requirement already satisfied: charset-normalizer<4,>=2 in /opt/certbot/lib/python3.11/site-packages (from requests>=2.20.0->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (3.3.2) Requirement already satisfied: idna<4,>=2.5 in /opt/certbot/lib/python3.11/site-packages (from requests>=2.20.0->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (3.8) Requirement already satisfied: urllib3<3,>=1.21.1 in /opt/certbot/lib/python3.11/site-packages (from requests>=2.20.0->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (2.2.2) Requirement already satisfied: certifi>=2017.4.17 in /opt/certbot/lib/python3.11/site-packages (from requests>=2.20.0->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (2024.8.30) Requirement already satisfied: requests-file>=1.4 in /opt/certbot/lib/python3.11/site-packages (from tldextract>=2->dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (2.1.0) Requirement already satisfied: filelock>=3.0.8 in /opt/certbot/lib/python3.11/site-packages (from tldextract>=2->dns-lexicon>=3.15.1->certbot-dns-ovh~=3.0.1) (3.16.1) Requirement already satisfied: pycparser in /opt/certbot/lib/python3.11/site-packages (from cffi>=1.12->cryptography>=3.2.1->acme>=3.0.1->certbot-dns-ovh~=3.0.1) (2.22) Installing collected packages: dns-lexicon, certbot, certbot-dns-ovh   Attempting uninstall: dns-lexicon     Found existing installation: dns-lexicon 3.5.6     Uninstalling dns-lexicon-3.5.6:       Successfully uninstalled dns-lexicon-3.5.6   Attempting uninstall: certbot     Found existing installation: certbot 2.11.0     Uninstalling certbot-2.11.0:       Successfully uninstalled certbot-2.11.0 ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. certbot-dns-timeweb 1.0.1 requires certbot<3.0,>=2.8, but you have certbot 3.0.1 which is incompatible. certbot-dns-websupport 2.0.1 requires certbot<3.0.0,>=2.0.0, but you have certbot 3.0.1 which is incompatible. certbot-dns-porkbun 0.9.1 requires tldextract<6.0,>=5.1.2, but you have tldextract 3.5.0 which is incompatible. certbot-dns-constellix 0.2.2 requires certbot<3.0,>=2.0, but you have certbot 3.0.1 which is incompatible. certbot-dns-cloudns 0.6.0 requires acme<2.8.0,>=2.7.4, but you have acme 3.0.1 which is incompatible. certbot-dns-cloudns 0.6.0 requires certbot<2.8.0,>=2.7.4, but you have certbot 3.0.1 which is incompatible. certbot-dns-oci 0.3.6 requires certbot<=1.15.0, but you have certbot 3.0.1 which is incompatible. certbot-dns-mijn-host 0.0.5 requires acme==2.11.0, but you have acme 3.0.1 which is incompatible. certbot-dns-mijn-host 0.0.5 requires certbot==2.11.0, but you have certbot 3.0.1 which is incompatible. certbot-dns-mijn-host 0.0.5 requires pyOpenSSL==24.2.1, but you have pyopenssl 21.0.0 which is incompatible. certbot-dns-powerdns 0.2.1 requires dns-lexicon<=3.5.6,>=3.2.4, but you have dns-lexicon 3.18.0 which is incompatible. certbot-dns-dynu 0.0.4 requires dns-lexicon==3.11.7, but you have dns-lexicon 3.18.0 which is incompatible. Successfully installed certbot-3.0.1 certbot-dns-ovh-3.0.1 dns-lexicon-3.18.0
So It looks like you have to choose which certbot version you want depending on your provider.
As an enhancement, I would suggest to install only user related certbot-dns provider, by passing it as an argument to installation script. I don't see the point of having all these unused ones installed on my LXC.

Hey @Wendigogo, I was able to install OVH plugin by the command you provided, however this did not fix the original error, and I was unable to obtain a certificate. Did it work for you?

I didn't try it atm. I'm not at home and I don't want to break things now. ^_^
Maybe you should try with original version with:
. /opt/certbot/bin/activate && pip install --no-cache-dir certbot-dns-ovh~=2.11.0 && deactivate
Maybe that's why script stucks to this version.

Oh my!
I just see that you could install ONLY wanted certbot plugins in looking to the script:

// Usage:
//   Install all plugins defined in certbot-dns-plugins.json:
//    ./install-certbot-plugins
//   Install one or more specific plugins:
//    ./install-certbot-plugins route53 cloudflare

So I made a clean install (thanks proxmox) and only use ./install-certbot-plugins ovh.
No issue in installing but certbot plugins still doesn't show ovh plugin. 😭

Running certbot plugins gave me an idea to use MultiDns provider with the following config:
dns_multi_provider=ovh
OVH_ENDPOINT=ovh-eu
OVH_APPLICATION_KEY=XXX
OVH_APPLICATION_SECRET=XXX
OVH_CONSUMER_KEY=XXX

This way, I could generate a cert successfully! Thanks, @Wendigogo.

This way, I could generate a cert successfully! Thanks, @Wendigogo.
From the GUI?

I find something:
When installing plugins with /app/scripts/install-certbot-plugins there is a king of chroot and they are installed in /opt/certbot/lib/python3.11/site-packages/ instead of /usr/local/lib/python3.11/dist-packages/. That's why modules are not seen in "normal" shell.
And if you look in /usr/local/lib/python3.11/dist-packages/, the ONLY available module is certbot_dns_multi.
So MAYBE, the npm envirronement is not set correctly. I'm using an LXC in proxmox using this script for installation:.

This way, I could generate a cert successfully! Thanks, @Wendigogo.

From the GUI?
I find something: When installing plugins with /app/scripts/install-certbot-plugins there is a king of chroot and they are installed in /opt/certbot/lib/python3.11/site-packages/ instead of /usr/local/lib/python3.11/dist-packages/. That's why modules are not seen in "normal" shell.
And if you look in /usr/local/lib/python3.11/dist-packages/, the ONLY available module is certbot_dns_multi.
So MAYBE, the npm envirronement is not set correctly. I'm using an LXC in proxmox using this script for installation:.

For information I have the problem with ovh, I use the same version as you with proxmox and the lxc script

dns_multi_provider=ovh

I tested with your solution: Multi Dns provider and same problem with DNS Challenge

CommandError: usage:
  certbot [SUBCOMMAND] [options] [-d DOMAIN] [-d DOMAIN] ...

Certbot can obtain and install HTTPS/TLS/SSL certificates.  By default,
it will attempt to use a webserver both for obtaining and installing the
certificate.
certbot: error: unrecognized arguments: --dns-ovh-credentials /etc/letsencrypt/credentials/credentials-11

    at /app/lib/utils.js:16:13
    at ChildProcess.exithandler (node:child_process:410:5)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

From the GUI?

Yes, from the GUI.

So MAYBE, the npm envirronement is not set correctly. I'm using an LXC in proxmox using this script for installation:.

I also use LXC in proxmox with nginxproxymanager installed from the same script.

dns_multi_provider=ovh

I tested with your solution: Multi Dns provider and same problem with DNS Challenge

@kwetnico You sure you selected the MultiDns provider? The screenshot shows OVH.

From the GUI?

Yes, from the GUI.

So MAYBE, the npm envirronement is not set correctly. I'm using an LXC in proxmox using this script for installation:.

I also use LXC in proxmox with nginxproxymanager installed from the same script.

dns_multi_provider=ovh

I tested with your solution: Multi Dns provider and same problem with DNS Challenge

@kwetnico You sure you selected the MultiDns provider? The screenshot shows OVH.

Yes it's ok to ovh with DNSMulti !


Custom render function doesn't work as documented

The documentation is missing - or at least misleading - on "Custom Rendering (e.g. from a MongoDB database)".
One would expect that, by providing the render function in the configuration object, the template would be rendered by that function, period. However, the flow of a send goes through renderAll, which checks if there's real files on disk.
This is not documented.
Note that I'm not suggesting the documentation should change; what I'm suggesting is that the behaviour changes completely: if a custom render function is provided, renderAll shouldn't check for files at all.

@mjamado would love a PR + tests for this.  great point.

I'll try, but I can't promise (callback won't work either - I'm so sorry for this pun).
If anyone lands here, one workaround is to just create the file structure on disk anyway, with empty pug files. Highly ineffective, but gets the job done.

Published as v3.6.0 to npm, thanks @mjamado.


Empty Har files

Hi!
Thanks for writing this library, I was looking for something exactly like this. I am using jenkins and robotframework to do automated testing on a website. Unfortunately I'm not able to actually record anything in Har files using robotframework, while using jenkins or otherwise. Even the provided example .robot file does not actually record any data, just the basic outline of a har file. However, when using browsermob proxy with selenium (without robotframework), I am able to record a normal (populated with data) har file.
So what's going on? I see that this library works with Robot framework 2.6... is this problem caused by robotframework 3? Should I revert to an older robot version? BTW, I'm on ubuntu 16.4, and I am using python 2.7. Thanks in advance!

Hi,
Was there a solution found to the issue mentioned above. I seem to be having the same issue.
I updated the "New Har" keyword with the options but this did not resolve the issues.

Sorry for late response, can you please check tests I've just added. Can you please provide a code snippet to reproduce the issue.

Same here:


[tapcell] Fix invalid placements when macros are near core boundaries

I don't understand why this is breaking the regressions after I merged it. The only failure I saw in the PR was a missing ok file that I had already taken care of. Eder, can you take a look at the jenkins log?

@jjcherry56 At both openroad and fix/tapcell_overlaps branches, I'm seeing this error in Test-Flow stage:
"Error: check_power_grid -net is not a known keyword or flag."

I think this was a Jenkins problem - I cleaned the workspace and that error went away.  There is still an error in pdngen but it seems unrelated to this change.


Update identity-resolver.md

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Thanks! You will need to DCO sign your commit too, see the details of that build for more information.

Already done

Seems you accidentally rebased wrong in this instance though - now you got all of the changes of another PR along with yours.

@freben pls take a look again

Hey @Artur93gev, congrats on your first merged PR 🎉 Nice work 👍 Please let me know if we can be helpful in any way.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.4.276",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=1512e7dc5eb08b7d92a12e2bfcd9cb8c4a1ec069"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "1512e7dc5eb08b7d92a12e2bfcd9cb8c4a1ec069",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "misc/libmasm/module: Fix two use after free in ibmasm_init_one\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.4.276 by commit 1512e7dc5eb08b7d92a12e2bfcd9cb8c4a1ec069. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001482


Add MetaMask "HD Key Tree" derivation paths

This together with https://github.com/iov-one/iov-core/issues/404 allows us to generate MetaMask addresses from a mnemonic

@LucasIsasmendi this one is easier to review commit by commit


runtime: Bump Oasis SDK to 408b8c5f9d2a7aeafaba773effb4bda5f6306a6e

Yeah wanted to do that in a separate PR, but I guess it's fine :-)


Deploy a jenkins server for BSP yocto layers we currently maintain

@brownjohnf After a discussion with @petrosagg we decided to have:
yocto.resin.io (landing page - similar to resinos.io)
yocto-ci.resin.io (jenkins web interface)

Cool, I like it. I'll let you know when it's up.

Hi @brownjohnf . Any updates on this one?

Sorry @agherzan, I've gotten caught up by other things. I'll prioritize this for mon morning. The host is up, I just need to get Jenkins installed.

It was done and it's not available at https://yocto-ci.resin.io/.